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

Postprocessing failure summary file only includes last measure #234

Open
ChristopherCaradonna opened this issue Oct 17, 2024 · 1 comment · May be fixed by #239
Open

Postprocessing failure summary file only includes last measure #234

ChristopherCaradonna opened this issue Oct 17, 2024 · 1 comment · May be fixed by #239
Assignees
Labels
bug Something isn't working postprocessing PR improves or adds postprocessing content

Comments

@ChristopherCaradonna
Copy link
Contributor

The failure summary created during postprocessing only includes the baseline and 1 upgrade (the last one processed), regardless of the number of upgrades. This is because the files are created individually per measure, with the same name and file path, so they overwrite each other.

We want a single file to be produced with the failure summary for the baseline and all upgrades. This is preferred over individual files per upgrade so that a user doesn't have to open potentially dozens of files to understand the run failures.

@ChristopherCaradonna ChristopherCaradonna added bug Something isn't working postprocessing PR improves or adds postprocessing content labels Oct 17, 2024
@wenyikuang wenyikuang linked a pull request Oct 22, 2024 that will close this issue
29 tasks
@wenyikuang
Copy link
Collaborator

Fixed this issue, but I think there is somethign wrong in the meta data check. Could we have a meeting to run a sample?

Thanks,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working postprocessing PR improves or adds postprocessing content
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants