-
Notifications
You must be signed in to change notification settings - Fork 46
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
[Fud2] Cider is the default for --to dat
computations
#2208
Comments
I think defaults are decided based on the order that they are added. I think that the right solution to this problem is being able to explicitly declare priorities of operators. So that it doesn't depend on something rather opaque. |
Yeah I agree with that approach |
We definitely need a way to do something about this so that the default paths are more obvious & predictable. The tracker (#1878) mentions priorities as a way about this, and it links to #1873 for further discussion. I will link the tracker to this issue as well as a specific instance of the prioritization problem, which we should perhaps fix even if we don't have a general scheme yet. In general, however, I'm not 100% sure integer priority scores are the way… they are a little bit brittle and fiddly to manage. Maybe there's another way that we can ensure a deterministic order of op registration (including from Rhai files) that would allow this kind of control without scoring. |
The downside is that this will require fiddling with Rhai for users who only care about using |
I think I understand what you're saying: the the user should also be able to specify what to prioritize. That wasn't what I was thinking of here; it was just that the fud2 developer should have a way to predictably induce the default that they want. Separately, yeah, I guess it could also make sense to let users pick whether they want to behave like |
It should probably be verilator for the sake of matching the original
fud
and for overall stabilityThe text was updated successfully, but these errors were encountered: