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

Stable items on S3 for software testing purposes #80

Closed
gwaybio opened this issue Mar 7, 2024 · 4 comments
Closed

Stable items on S3 for software testing purposes #80

gwaybio opened this issue Mar 7, 2024 · 4 comments

Comments

@gwaybio
Copy link
Member

gwaybio commented Mar 7, 2024

Hi All! Responses to prompts below:

Is your question related to a problem? Please describe.

We had an issue with pycytominer tests failing, and it seems due to an updated path in the JUMP S3 bucket. See cytomining/pycytominer#374 (comment) for complete details.

Describe the solution you'd like

It would be ideal if these paths were stable and we could build tests using them. If paths will update in the future, is it possible to keep a dedicated folder of test files that will not change?

Describe alternatives you've considered

As I describe in cytomining/pycytominer#374, we can delete the S3 integration with cyto_utils.cell_locations

@ErinWeisbart
Copy link
Member

Hi Greg!
It looks like you're pointing your tests to s3://cellpainting-gallery/test-cpg0016-jump/?
This is not a stable folder.

In general, one can consider stable all S3 paths to objects within prefixes named in our README. In your example this would be s3://cellpainting-gallery/cpg0016-jump/.

(This comes with the caveat that right now there are some things moving around in JUMP as we prepare for final data release and several publications, but this is an exception to the rule.)

@gwaybio
Copy link
Member Author

gwaybio commented Mar 7, 2024

great thanks Erin, this is super helpful! pinging @d33bs so he is aware of this insight.

It seems like an easy option is to update the s3 path (remove test-) and brace for potential additional shake-ups prior to final data release/publications.

@ErinWeisbart - after this shake-up, do you anticipate additional changes to files? Might it still be worth to create something in the gallery dedicated for testing purposes?

@ErinWeisbart
Copy link
Member

No, we anticipate things staying stable.
That being said, given its scale and complexity, JUMP is the most likely to have shakeups if there are future shakeups that we don't have in-progress, so I would suggest pointing to any other dataset in the gallery for tests.

@gwaybio
Copy link
Member Author

gwaybio commented Mar 8, 2024

so I would suggest pointing to any other dataset in the gallery for tests.

Oh great point - @d33bs, I think we have enough info to proceed on this fix.

Thanks again Erin!

@gwaybio gwaybio closed this as completed Mar 8, 2024
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