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

Download zarrs from AICS FMS #244

Open
BrianWhitneyAI opened this issue Sep 23, 2024 · 1 comment
Open

Download zarrs from AICS FMS #244

BrianWhitneyAI opened this issue Sep 23, 2024 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@BrianWhitneyAI
Copy link
Contributor

          We have decided to forgo individual local file tracking for now. In the situation that we use a /allen path and want to download locally we should use the FmsFile class's logic for determining cloud paths for all zarrs so that the existing download tracking functions correctly. This might prove more complicated as right now we do not support download paths for paths like `s3://allencell/`

Originally posted by @BrianWhitneyAI in #243 (comment)

@BrianWhitneyAI
Copy link
Contributor Author

Right now we only support S3 downloads and local copying on desktop. The current download path we are using allows for rendering thumbnails but not for download (since it doesnt meet our current S3 method). The cloudpaths we have for those files are permission blocked so we cant use that for download either. If we want to support download from AICS FMS we need to either change permissions of the cloudpaths or look into downloading other formatted paths (RN its a http). This could just point to a s3 file and be easy but would be another step.

@BrianWhitneyAI BrianWhitneyAI changed the title Resolve /allen zarrs to use S3 Download path Download zarrs from AICS FMS Sep 25, 2024
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