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

Quilt links are not downloadable via curl #28

Open
pgarrison opened this issue Oct 22, 2024 · 1 comment
Open

Quilt links are not downloadable via curl #28

pgarrison opened this issue Oct 22, 2024 · 1 comment

Comments

@pgarrison
Copy link
Collaborator

This issue is about the work-in-progress branch for colony mask inference reproducibility, sm_colony_mask_pred_test_v0.

Issue

The instructions ask the user to download a file from the following link.

https://open.quiltdata.com/b/allencell/tree/aics/emt_timelapse_dataset/supplemental_files/cytodl_checkpoints/all_cells_mask_seg_model_checkpoint.ckpt

Users may attempt to download via curl https://open.quiltdata.com/b/allencell/tree/aics/emt_timelapse_dataset/supplemental_files/cytodl_checkpoints/all_cells_mask_seg_model_checkpoint.ckpt, but this gets the HTML of the Quilt website.

Fix: Option 1

Update the instructions to clarify that the user should navigate to that link in the browser and then click the "Download File" button.

The drawback of this approach is that it is harder to automate.

Fix: Option 2

Replace the link with one of the following.

  • Single immutable version: https://allencell.s3.amazonaws.com/aics/emt_timelapse_dataset/supplemental_files/cytodl_checkpoints/all_cells_mask_seg_model_checkpoint.ckpt?versionId=ejf07rBiw5slyx1zQyurfX6.zpSJ92JM
  • Latest version: https://allencell.s3.amazonaws.com/aics/emt_timelapse_dataset/supplemental_files/cytodl_checkpoints/all_cells_mask_seg_model_checkpoint.ckpt
    The drawback of this approach is that the "single immutable version" link may get outdated if we re-train the model, and conversely the data at the "latest version" link may change, creating inconsistency.
@smishra3
Copy link
Collaborator

I think we can go ahead with both options and letting the user decide based on their preference. In future, if we update the checkpoint path, we need to update the README to provide the correct immutable version.

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

No branches or pull requests

2 participants