-
Notifications
You must be signed in to change notification settings - Fork 2
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
calibrate_pitot: cal_time_range is not being used #44
Comments
Similarly, looks like the default pitot speed estimate is 30s instead of 10min? Is this intentional? |
your are right cal_range is not used anymore... there is no reason to keep it I guess... the 30 sec, was me beeing lacy and not wanting to change it anytime I pull a fresh version :) |
I've been dealing with this by having a branch that's particular to each "experiment" for experiment-specific default settings. And then I have instrument specific branches. E.g. "master -> rama -> rama13-526" with each branch building on top of each other. It makes things complicated but it's relatively easy to ensure that all RAMA chipods have the same set of defaults. |
That's smart.. |
Hey it looks like cal_time_range is not used anymore? But that variable still exists. I'm getting merge conflicts so wondering why...
The text was updated successfully, but these errors were encountered: