Skip to content
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

fix Clb_converged, don't remove 'b' from folder names #83

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

orichters
Copy link
Contributor

@orichters orichters commented Oct 30, 2023

  • since a while, calibrations show no Clb_converged anymore, but just converged. This misleads AMT runs into thinking some run did not converge
  • Did we have input_02.gdx etc. until ~ September 2022? The amount of input_xx.gdx files was used to determine whether a converged iteration was a successful calibration. Now, look for the corresponding fulldata_xx.gdx which did not seem to exist earlier.
  • fix a small bug if you manually pass a folder that contains a b (with rs2 SSP2-base`), that was sometimes removed.

@orichters orichters requested a review from dklein-pik October 30, 2023 11:51
@orichters orichters merged commit 20e347f into pik-piam:master Nov 6, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants