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

Multiscale: Support project files containing an integer working_scale #2816

Merged
merged 1 commit into from Feb 20, 2024

Conversation

btbest
Copy link
Contributor

@btbest btbest commented Feb 19, 2024

All project files created with ilastik1.4.1b13 break after switching to string-indexed scale handling. While fixing that, I also implemented handling projects with Precomputed datasets in them

It may be worth looking into version compatibility testing. It should be possible on CI; could be costly though in terms of pipeline time.

@btbest btbest marked this pull request as ready for review February 19, 2024 17:06
@btbest btbest force-pushed the support-projects-with-int-scale branch 2 times, most recently from 1e6c376 to 0f77f94 Compare February 19, 2024 17:25
Copy link

codecov bot commented Feb 19, 2024

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (efaee7e) 55.76% compared to head (6f88900) 55.76%.
Report is 1 commits behind head on main.

Files Patch % Lines
ilastik/applets/dataSelection/opDataSelection.py 16.66% 10 Missing ⚠️
...utility/io_util/RESTfulPrecomputedChunkedVolume.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2816      +/-   ##
==========================================
- Coverage   55.76%   55.76%   -0.01%     
==========================================
  Files         530      530              
  Lines       62191    62204      +13     
  Branches     8500     8502       +2     
==========================================
+ Hits        34683    34689       +6     
- Misses      25761    25773      +12     
+ Partials     1747     1742       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@k-dominik k-dominik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Glad this is fixed.

All project files created with ilastik1.4.1b13 break otherwise
@btbest btbest force-pushed the support-projects-with-int-scale branch from 0f77f94 to 6f88900 Compare February 20, 2024 09:36
@btbest btbest merged commit 14e6469 into ilastik:main Feb 20, 2024
14 of 16 checks passed
@btbest btbest deleted the support-projects-with-int-scale branch February 20, 2024 10:30
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

Successfully merging this pull request may close these issues.

None yet

2 participants