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

Modality change, fixed 250 epochs, and seeded splits logic #10

Merged
merged 10 commits into from
Nov 8, 2024

Conversation

brandon-edwards
Copy link
Collaborator

Modality is now fixed for the new initial model
Val and training batch counts are set at 50 and 250 as is done in NNUnet (with timouts still in place)
Train_completion dampener is set to 0.0 in the trianing_config I provide here (can bring over other plan of course but set this the same if so).
Splits logic is seeded to be reproducible

Splits logic tested independently with different seeds to see difference and reproducibility

to 50 and 250 respectively. Timeouts will still apply to stop early if
it is taking too long. Amount of training completed will be computed off
of these new maxes, i.e. training_completed of 1.0 will mean all of the
250 batches were trained. With this change we will want model updates to
be counted according to local data size, therefore it is important that
the training config set the train_completion_dampener at the value: 0.0.
…s related to his new initial model:

order = t1 t2 flair t1c
@brandon-edwards brandon-edwards temporarily deployed to testing-external-code November 8, 2024 04:57 — with GitHub Actions Inactive
@brandon-edwards brandon-edwards temporarily deployed to testing-external-code November 8, 2024 05:01 — with GitHub Actions Inactive
@brandon-edwards brandon-edwards temporarily deployed to testing-external-code November 8, 2024 05:02 — with GitHub Actions Inactive
@hasan7n hasan7n temporarily deployed to testing-external-code November 8, 2024 17:24 — with GitHub Actions Inactive
@hasan7n hasan7n merged commit f150ac5 into fl-poc Nov 8, 2024
2 of 7 checks passed
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