-
Notifications
You must be signed in to change notification settings - Fork 16
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
Remove old seed deprecation warning #258
Conversation
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.
Yes, it's a warning that has been there for a while. Perhaps the option to use "seeds" instead of "rms_seeds" should also be removed ? (line 240) This namechange came when we also introduced to possibility to set seed value for fmudesign, and it was chosen to use "rms_seeds" and "dist_seed" for the two different types of seeds.
eb4d32c
to
28b220a
Compare
Make sense to remove the if statement dealing with "seeds" I think, thanks. |
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.
Not sure how this is normally done. But I expect some project still have a general_input sheet where they have "seeds" instead of "rms_seeds". Would it be better if "seeds" is found in general_input sheet, to let it fail with the message that they should change this to "rms_seeds"?
28b220a
to
9149180
Compare
I think that makes sense and have added this: elif "seeds" in generalinput[1]:
raise ValueError(
"The 'seeds' parameter has been deprecated and is no longer supported. "
"Use 'rms_seeds' instead"
)
`` `
Thanks. |
No description provided.