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

Issue105 sort process metadata merge report so comparing reports is easier #110

Conversation

JohanKJSchreurs
Copy link
Contributor

Added some sorting to make it easier to compare the reports of merging process metadate, for example for #105.

Allowing to compare parameters that have a lot of fields set to `null` or `0.0`
which were probably supposed to be left out entirely.
Work in progress.

#105
…rant for incorrect defaults.

This was not reliable yet, and we would need a cleaner solution.
First make the tests work again.

Issue #105
@JohanKJSchreurs JohanKJSchreurs marked this pull request as draft April 28, 2023 08:33
Updated test's expected result: missing params and extra params are now a sorted list

Issue #105
@JohanKJSchreurs JohanKJSchreurs marked this pull request as ready for review April 28, 2023 10:03
@soxofaan soxofaan merged commit 60f2de4 into master May 2, 2023
@soxofaan soxofaan deleted the issue105_sort-process-metadata-merge-report-so-comparing-reports-is-easier branch May 2, 2023 09:54
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