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

Update zenodo sample testing for FMCIB #107

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

surajpaib
Copy link
Contributor

No description provided.

@surajpaib
Copy link
Contributor Author

@LennyN95 Updated the zenodo ref

@LennyN95
Copy link
Member

LennyN95 commented Mar 1, 2025

@surajpaib I prepared sample annotations for all workflows and added exemplary descriptions to some for you to review and extend on. Can you give me write access on your fork repo?

@surajpaib
Copy link
Contributor Author

Great @LennyN95! You should have access already I think?

- add file structure annotation for all workflows
- add example descriptions for default, from_centroids, and from_json workflow
@LennyN95
Copy link
Member

LennyN95 commented Mar 4, 2025

I opened a new code space, and now it works. Previously, I encountered a permission error.

@surajpaib, I added the sample annotation structure for all workflows. Could you please:

  • Check the descriptions
  • Add descriptions where they are missing
  • For the workflow where the dicomseg file and the dicom files are in the same folder, include the name of the dicomseg file with an appropriate description, and perhaps two exemplary slices of the dicom image?

@surajpaib
Copy link
Contributor Author

@LennyN95 Looks all good to me! Added some info as you requested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Submitted Implementations
Development

Successfully merging this pull request may close these issues.

2 participants