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

Add a button for downloading the airway prediction in NIfTI format #58

Closed
tonycwk opened this issue Jun 21, 2024 · 4 comments · Fixed by #59
Closed

Add a button for downloading the airway prediction in NIfTI format #58

tonycwk opened this issue Jun 21, 2024 · 4 comments · Fixed by #59
Assignees
Labels
enhancement New feature or request

Comments

@tonycwk
Copy link

tonycwk commented Jun 21, 2024

The airway prediction in NIfTI format is useful for model performance evaluation.

The current application produces the airway prediction in NIfTI format, however, it is not accessible through the web interface.

Therefore, it is very useful to add a button for downloading this file.

@andreped andreped self-assigned this Jun 21, 2024
@andreped andreped added the enhancement New feature or request label Jun 21, 2024
@andreped
Copy link
Member

Thanks for the great suggestion, @tonycwk! :]

I can add support for this feature now. Will let you know when the PR has been merged.

@andreped
Copy link
Member

andreped commented Jun 21, 2024

@tonycwk Demo should now have a download button. You will need till inference is completely finished, before clicking download.

Test the updated deployment at:
https://huggingface.co/spaces/andreped/AeroPath

@andreped
Copy link
Member

@tonycwk Sadly, there is a bug with Docker build. So deployment on Hugging Face is now broken.

This was because I upgraded Gradio to the latest version in the aforementioned PR.

I will make a bugfix for this now!

@andreped
Copy link
Member

I will make a bugfix for this now!

Made the fix for it in 1a46c84 and now deployment works.

Found another issue causing the download button not to work in Hugging Face. Made a fix for it now, so I believe the feature is completely ready for use in the deployed instance, @tonycwk :]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants