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

Update process results workflow for new website #919

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

lazappi
Copy link
Collaborator

@lazappi lazappi commented Feb 3, 2025

Describe your changes

  • Add a new component to combine results output
  • Update the results processing workflow to output combined results
  • Update results format to new JSON schemas

TODO:

  • Output metric execution info from get_results
  • Update generate_qc to new schema
  • Update combine_output to new schema
  • Add schema validation to components
  • Create bibliography

Checklist before requesting a review

  • I have performed a self-review of my code

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Bug fixes
    • Documentation
  • Proposed changes are described in the CHANGELOG.md

  • CI Tests succeed and look good!

@lazappi
Copy link
Collaborator Author

lazappi commented Feb 3, 2025

This should produce a combined tasks.json file in the format required by the new website. However, parts of that file (datasets etc.) will be in the v2 format which the website may not expect.

@lazappi lazappi marked this pull request as draft February 10, 2025 09:29
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.

1 participant