-
Notifications
You must be signed in to change notification settings - Fork 19
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
Advise user if pickle requested, but pickled changes are older. #3
Conversation
and generate a very loud and informative message in that case.
I made an issue for it, so that the problem I think I'm solving can be made clear. 😄 #4 |
don't merge this --- it does not check the *.yaml config file; needs an edit |
should be good now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good. Default behavior is preserved with this change and just adds a small little warning on load only.
~ Joe G.
I reverted this merge @cuantar . Thankful noticed several issues that appeared when the change got added. Also, we had a detailed discussion and are unsure if this is necessary, but we can iterate on it today. |
add convert_tcb2tdb=False to all parameters, fix naming of LAMBDA
The code is simple and has been tested as part of a toy model; but it needs someone experienced at using the
timing_analysis
in practice to verify that it does what is expected.