8000 Fix label conversion with proj mode by brisvag · Pull Request #7095 · napari/napari · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix label conversion with proj mode #7095

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

Merged
merged 3 commits into from
Jul 22, 2024
Merged

Conversation

brisvag
Copy link
Contributor
@brisvag brisvag commented Jul 15, 2024

References and relevant issues

Fixes #7092

Description

Ignore incompatible projection modes when converting between layer types. If no corresponding mode exists, a warning is now raised and the mode is ignored.

@brisvag brisvag added the bugfix PR with bugfix label Jul 15, 2024
@brisvag brisvag added this to the 0.5.1 milestone Jul 15, 2024
@brisvag brisvag requested review from andy-sweet and jni July 15, 2024 10:13
@Czaki
Copy link
Collaborator
Czaki commented Jul 15, 2024

I think that we should add a warning and a test.

Copy link
codecov bot commented Jul 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.86%. Comparing base (157b144) to head (0c72139).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7095      +/-   ##
==========================================
- Coverage   92.95%   92.86%   -0.10%     
==========================================
  Files         618      618              
  Lines       56486    56504      +18     
==========================================
- Hits        52509    52471      -38     
- Misses       3977     4033      +56     

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

@github-actions github-actions bot added the tests Something related to our tests label Jul 15, 2024
Copy link
Member
@psobolewskiPhD psobolewskiPhD left a comment

Choose a reason for hiding this comment

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

I like this and it works perfectly locally.
❤️

@jni jni added the ready to merge Last chance for comments! Will be merged in ~24h label Jul 16, 2024
@jni jni merged commit 041c581 into napari:main Jul 22, 2024
39 checks passed
@github-actions github-actions bot removed the ready to merge Last chance for comments! Will be merged in ~24h label Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix PR with bugfix tests Something related to our tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Converting image to labels breaks if projection_mode='mean'
4 participants
0