-
Notifications
You must be signed in to change notification settings - Fork 7
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
Update test dataset #240
Comments
Once this is in place, I can update the examples we have so that they produce the new output and we can just update the zenodo dataset with a new version (already did this once for one of them). |
For testing, this is currently taken care of with a workaround in a fixture: e3690ef |
(of course this is the way to go..) |
@tcompa I updated the main 2 datasets on Zenodo. Because the data changed, they also get a new identifier. They can now be found here: Tiny: https://zenodo.org/record/7674545 I ran the tiny at higher cellpose labeling resolution than before. So if you update this in the test, check whether it still matches well (are we testing for getting the exact same data actually? Could anyway be tricky, not sure whether cellpose is pixel-perfect reproducible). I used those workflows to generate the datasets: |
Thanks! To do:
|
For the record: we do not use the multiplexing dataset in the CI.
We aren't. The zarr dataset are mostly used as a valid starting point for additional processing steps, but we do not check that tasks produce exactly the same output. |
While working on #239, I though a few updates of the test datasets would be useful.
The first one is that we should put a new zarr on zenodo, complying with the new rules (that is, the omero channels should always include also a
wavelength_id
attribute).The text was updated successfully, but these errors were encountered: