-
Notifications
You must be signed in to change notification settings - Fork 0
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
TotalSeg configs and supporting code #8
Conversation
init_LR: 0.001 | ||
project: "ct_fm_quick_class_eval" | ||
in_channels: 1 | ||
embedding_dim: 512 | ||
format: "$'suprem' if 'suprem' in @CONSTANTS#name else 'lighter'" | ||
format: "$'suprem' if 'suprem' in @vars#name else 'lighter'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
potentially problematic for new models
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this won't be missed as newer models we'll have to figure out input configurations again and edit this as well. I'll move it over to a dict or so then
let me know when i can merge this, will add head abnormality classification then following the same structure |
@ibro45 You can merge it already if you like. But I will add a script file for TS with specifics on how to run it. Some changes I made,
|
I've tested all the configs once and looked added several improvements to the pipeline for TS
|
Going to run the comparison against Merlin now (end-to-end) to see if the workflow is good. I'll also move on to MSD and hope to finish that asap |
@ibro45 Time to merge? |
No description provided.