-
Notifications
You must be signed in to change notification settings - Fork 13
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
Comments
Hi Greg! In general, one can consider stable all S3 paths to objects within prefixes named in our README. In your example this would be (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.) |
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 @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? |
No, we anticipate things staying stable. |
Oh great point - @d33bs, I think we have enough info to proceed on this fix. Thanks again Erin! |
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
The text was updated successfully, but these errors were encountered: