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

REMIND: Check for summation errors and report in already existing Mif column of getRunStatus #94

Merged
merged 4 commits into from
Mar 8, 2024

Conversation

dklein-pik
Copy link
Member

@dklein-pik dklein-pik commented Mar 8, 2024

For REMIND model: if a REMIND_generic<scenname>_summation_errors.csv exists in the individual run folder, report sumErr in the Mif column of getRunStatus. This will be displayed also in rs2 and added to the summary message of the AMTs.

Folder                                            Runtime      inSlurm   RunType      RunStatus          Iter              Conv                   modelstat              Mif     AppResults 
default-AMT_2024-03-01_22.08.46                   NA           no        nash         Abort 12R*5*Infes  5/100             566555550555           6: Intermed Infes      no      no
SDP_MC-NDC-AMT_2024-03-02_06.06.48                NA           no        nash         Abort 10R*5*Infes  5/100             000500000050           5: Locally Infes       no      no
SDP_MC-NPi-AMT_2024-03-01_22.13.42                6 hours      no        nash         Normal completion  53/100            converged              2: Locally Optimal     sumErr  yes

@dklein-pik dklein-pik requested a review from orichters March 8, 2024 15:02
Copy link
Contributor

@orichters orichters left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor question, but as you make some decision later based on the existence of the mif file and consider sumErr to be an indicator of the mif file existing, it might be relevant.

R/getRunStatus.R Outdated Show resolved Hide resolved
@dklein-pik dklein-pik merged commit cbe65c2 into pik-piam:master Mar 8, 2024
2 checks 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