Skip to content

Conversation

@silvandeleemput
Copy link
Contributor

As requested by Justin Kirby this PR:

  • Adds the full data citation to the TCIA Pancreas-CT dataset to the model card.
  • Adds a link (shared-cart URL) to the exact 80/82 cases used from the TCIA Pancreas-CT dataset for evaluating the model.

@silvandeleemput
Copy link
Contributor Author

@LennyN95, is it possible to merge this? Only the metadata file has changed. From what I can see, the tests fail because the earlier provided test data for the model is expected to be in a default folder. If necessary, I can try reuploading the test data I provided earlier in the new expected folder structure.

@LennyN95
Copy link
Member

Thank you @silvandeleemput. This is unexpected since the referenced test zip contains a top-level dir named default- so this should not be an issue. I will look into this!

@LennyN95
Copy link
Member

@silvandeleemput I fixed the underlying issue in the compliance check automation. I'll temporarily close & reopen the PR to trigger the automation.

@LennyN95 LennyN95 closed this Feb 17, 2025
@LennyN95 LennyN95 reopened this Feb 17, 2025
@LennyN95 LennyN95 self-requested a review February 17, 2025 12:36
Copy link
Member

@LennyN95 LennyN95 left a comment

Choose a reason for hiding this comment

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

Thank you @silvandeleemput for adding the TCIA dataset to the metadata.

@LennyN95 LennyN95 merged commit 0beda94 into MHubAI:main Feb 17, 2025
1 of 2 checks passed
@silvandeleemput silvandeleemput deleted the m-gc-nnunet-pancreas-meta-update branch February 26, 2025 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants