Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add rudimentary read support for Data General "2F" format #528

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pkooiman
Copy link

@pkooiman pkooiman commented Mar 4, 2025

#526 I managed to get conversion from flux image working. Unfortunately I don't understand Greaseweazle internals well enough to create a complete (and sane) implementation but in case it is helpful, here is my attempt.

@keirf
Copy link
Owner

keirf commented Mar 4, 2025

Oh that's pretty good. Is the MFM mode also valid, or just copied over from existing code but defunct here?

@keirf
Copy link
Owner

keirf commented Mar 4, 2025

Ohhhh this is probably FM only and worked out from the doc and flux file linked in ticket #526? Nice job if so, I can put together Data General 2F support from the same resources plus your code, which probably only needs some cleanup and redundant code deleted.

@pkooiman
Copy link
Author

pkooiman commented Mar 4, 2025

Correct, FM only and put together from the flux file and the docs. I only worked on reading, the writing code is still a 1:1 copy from Northstar ;.)

@keirf
Copy link
Owner

keirf commented Mar 4, 2025

Aha! Okay, I will take all the available info and cook up something from your prototype. Thanks.

@keirf keirf force-pushed the master branch 2 times, most recently from c6e4897 to 5cc6fb0 Compare March 10, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants