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

main (submission) & petric (backend) framework #20

Merged
merged 8 commits into from
Jun 27, 2024
Merged

Conversation

casperdcl
Copy link
Member

@casperdcl casperdcl commented Jun 27, 2024

  • exposes the petric.py backend (user changes to it will be ignored/overwritten)
    • get_data() reader
    • Dataset class passed to submitted algorithms
    • MetricsWithTimeout excluding time to run our callbacks
  • updated main.py with a BSREM example (fixes main.py needs updating #19)
  • updated SIRF_data_preparation/BSREM_*.py with new framework

@casperdcl casperdcl self-assigned this Jun 27, 2024
@casperdcl casperdcl added the enhancement New feature or request label Jun 27, 2024
@casperdcl casperdcl force-pushed the main-backend branch 4 times, most recently from 6e83afa to 5d9f3d5 Compare June 27, 2024 13:04
@casperdcl

This comment was marked as resolved.

main.py Show resolved Hide resolved
petric.py Show resolved Hide resolved
README.md Show resolved Hide resolved
petric.py Outdated Show resolved Hide resolved
petric.py Outdated Show resolved Hide resolved
@KrisThielemans KrisThielemans mentioned this pull request Jun 27, 2024
@casperdcl casperdcl force-pushed the main-backend branch 3 times, most recently from faf4a00 to 0be0da2 Compare June 27, 2024 21:56
main.py Show resolved Hide resolved
petric.py Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@casperdcl casperdcl merged commit a8d3155 into main Jun 27, 2024
2 checks passed
@casperdcl casperdcl deleted the main-backend branch June 27, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

main.py needs updating
2 participants