-
Notifications
You must be signed in to change notification settings - Fork 9
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
CORDIC in Filament #330
Comments
Extremely hype! I think it would be cool to spend some time thinking what kinds of design trade-offs we'd like to expose using parameters and what capabilities they can express. Once we have that, let's write up a proposal for implementation and get started! |
Very true, that is a good idea. I think I'll start with writing a simple unparameterized version of it to get an idea of what makes sense to expose/parameterize. I can write up a proposal after that. Does that sound good? |
Sure! The pre-reading suggestion was because it might be good to understand if there are any opportunities for parameterization in the design or not. If CORDIC doesn't have any opportunities, then we might prefer something else for the sake of more interesting exploration. |
Ah, good point. I think it would be cool to do something that could show off existential parameters - I'll see if they can be applied here |
@gabizon103 are you working on #332 instead of this? If so, let's close this issue for now |
Okay, closing this for now but @gabizon103 feel free to reopen if you end up working on it. |
We want to implement more designs in Filament (ideas listed in #124), so I'll take a shot at CORDIC. Will list some resources/other implementations here
The text was updated successfully, but these errors were encountered: