Add update for dseg/rtstruct [WIP] #99
Conversation
Signed-off-by: Suraj <[email protected]>
|
Looks good; ultimatively, we'd ditch the json file and use the value outputs. I've a branch with preparations for sth. like |
|
@LennyN95 This would be cool to do in the long run! For now, I will make updates for rest of the workflows and complete the PR with latest test updates |
|
@LennyN95 Will update the tests on zenodo as well |
|
@LennyN95 Added files for all workflows! I think it's good to merge from my end. Supports a wide range of tested workflows now |
|
Hi @LennyN95 Could you look into merging this? |
| mask_array = sitk.GetArrayFromImage(mask) | ||
| unique_values = np.unique(mask_array) | ||
| print(f"Unique values: {unique_values}") |
There was a problem hiding this comment.
We do this only to check the metadata, right?
|
Great! Thank you @surajpaib. I'll run the tests next week and then we can merge. |
|
FYI; I will close & reopen this PR to re-run our compliance checks. |
|
@surajpaib the test fails and the model cannot be updated. Can you check if the sample/reference data is correct and that the mhub.toml file points to the correct zip ressource? |
@LennyN95 PR'ing early for feedback