Skip to content

Conversation

@mzenk
Copy link
Collaborator

@mzenk mzenk commented Jun 18, 2021

This PR includes updates to the readme as well as additional python scripts for testing singularity containers.
The readme now describes how to get access to the container submission platform and how to test containers on sample data locally.

@mzenk mzenk requested review from FabianIsensee and dzimmerer June 18, 2021 08:01
dzimmerer
dzimmerer previously approved these changes Jun 18, 2021
Copy link

@dzimmerer dzimmerer left a comment

Choose a reason for hiding this comment

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

Looks good
(only not sure with first come first serve policy and unlimited # submissions...)

@mzenk
Copy link
Collaborator Author

mzenk commented Jun 18, 2021

The timestamp of the last submission will determine the order of evaluation. Do you think that's a problem? Should I clarify this?

Copy link
Collaborator Author

@mzenk mzenk left a comment

Choose a reason for hiding this comment

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

minor

@mzenk mzenk requested review from dzimmerer and sarthakpati and removed request for FabianIsensee June 18, 2021 19:42
@mzenk
Copy link
Collaborator Author

mzenk commented Jun 18, 2021

@sarthakpati only if you have a minute. I pushed a tiny commit after David's review...

Copy link
Member

@sarthakpati sarthakpati left a comment

Choose a reason for hiding this comment

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

Hey @mzenk, apologies, this somehow slipped through my notifications. Anyway, this LGTM!

@mzenk mzenk merged commit a63583d into FeTS-AI:main Jun 19, 2021
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.

3 participants