-
Notifications
You must be signed in to change notification settings - Fork 4
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
Do scaling runs for mode_appt_constraints = 2 too #1335
Comments
Question from meeting with @tamuri, @giordano, @matt-graham and @willGraham01
Seems like we're leaning towards former. |
@marghe-molaro / @tbhallett - we're working on adding
|
Hi @tamuri,
Especially the latter two, however, are not crucial. If it were much simpler to run in a simple
@tbhallett, what do you think? |
I think the run is only from 2010-2019, and the target period for calibration is 2010-2019. Perhaps, as the main purpose of this is to flush out bugs when running in this mode, we can keep the original scripts for now, and we can later raise a PR that brings in additional scripts to look at time trends. Is that ok, Margherita -- or do you think longer runs now and new scripts for analysis would be necessary for the first step? |
Yes I agree! |
To @tamuri's point above, we could make a function |
Ah sorry I missed that, yes I can issue a PR for a get_parameters_for_mode2_run function (once I've submitted the module order tests) Update: this now included in PR #1398 |
In order to weed out errors before large jobs in this mode are submitted, ensure scaling runs also consider the mode_appt_constraints = 2 set-up.
The text was updated successfully, but these errors were encountered: