Skip to content

Updated README.md #85

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

DanielaSchacherer
Copy link
Contributor

updated name of notebook

updated name of notebook
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@DanielaSchacherer
Copy link
Contributor Author

@fedorov I added here the code for exporting ANN objects to GeoJSON. It's not recommendable to do with this collection, the files are mostly > 2GB, but I still included it as it can be useful in case of smaller files.

@fedorov
Copy link
Member

fedorov commented Mar 3, 2025

@DanielaSchacherer can you please instead of adding images to the repository add them as release attachments? I don't want to pollute the repo with binary data.

@DanielaSchacherer
Copy link
Contributor Author

DanielaSchacherer commented Mar 4, 2025

@fedorov can I do that? Because if I do it in a fork, then I would link to my release and you would have to do the same again for the original repository, wouldn't you?

@fedorov
Copy link
Member

fedorov commented Mar 14, 2025

Sorry for the delay ... I added the image here so you can reference: https://github.com/ImagingDataCommons/IDC-Tutorials/releases/tag/0.20.0

Can you please remove it from this PR, and also squash the commits so it is not in the history?

@DanielaSchacherer DanielaSchacherer force-pushed the bulk_annotations_notebook branch from 81c3c8b to 315c181 Compare March 17, 2025 10:59
@DanielaSchacherer
Copy link
Contributor Author

DanielaSchacherer commented Mar 17, 2025

Hi Andrey,
should be done. Not sure why I seem to have replicated previous commit from you, but the code looks fine to me.
Let me know, if there is still something to be fixed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants