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

Should I add FY6300 support? #7

Open
jeremytrimble opened this issue Mar 21, 2022 · 2 comments
Open

Should I add FY6300 support? #7

jeremytrimble opened this issue Mar 21, 2022 · 2 comments

Comments

@jeremytrimble
Copy link
Contributor

Hi there. Thanks for making fygen available for the world, @mattwach !

I just got a FY6300 device (fy.get_model() returns 'FY6300-50M'), which seems to not be supported currently. The front panel says ("14Bit", "250MSa/s").

I'd be glad to take a shot at adding support for this model -- I just was wondering if anyone else has attempted adding support for the FY6300 already. If you are aware of anyone who's working on this (or tried and ran into problems), I'd love to hear about it. Otherwise I'll go ahead and try to add support myself.

Thanks again!

@mattwach
Copy link
Owner

Hello!

I don't know of anyone who is working on the FY6300 although the project has quite a few forks so maybe one of those would work. In any case, feel free to work on the FY6300 and I'll try and provide as much support as I can without having the hardware. If we get it working, I'd be happy to pull the changes in.

Have a nice evening!

@jeremytrimble
Copy link
Contributor Author

I've added support in #8 -- take a look and let me know if you have any questions. For some reason the version I have seems to represent frequency differently than the others, but I was able to make it work as-expected.

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

No branches or pull requests

2 participants