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

Solve related issues #107

Merged
merged 5 commits into from
Sep 29, 2022
Merged

Conversation

gselzer
Copy link
Collaborator

@gselzer gselzer commented Sep 27, 2022

This PR was originally designed to solve one issue, but it's actually going to solve (at least) two things when all's said and done.

I was also debating solving #102 in this PR as well, as I had to make a couple changes to the functionality underlying layer selection during transfer. But I think those changes are correct so we can solve it in a separate PR.

TODO before merge:

  • Wait for pom-scijava update of imagej-common. @ctrueden do you have an estimate of when this will be ready?
  • Test the changes.
  • Clean up.

Closes #100, #9

@gselzer gselzer added bug Something isn't working enhancement New feature or request labels Sep 27, 2022
@gselzer gselzer added this to the 0.1.0 milestone Sep 27, 2022
@gselzer gselzer self-assigned this Sep 27, 2022
@codecov-commenter
Copy link

Codecov Report

Base: 91.89% // Head: 91.47% // Decreases project coverage by -0.41% ⚠️

Coverage data is based on head (6944c23) compared to base (d10e739).
Patch coverage: 73.25% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   91.89%   91.47%   -0.42%     
==========================================
  Files          41       41              
  Lines        3280     3342      +62     
==========================================
+ Hits         3014     3057      +43     
- Misses        266      285      +19     
Impacted Files Coverage Δ
src/napari_imagej/types/mappings.py 100.00% <ø> (ø)
src/napari_imagej/types/converters/images.py 68.96% <66.03%> (-31.04%) ⬇️
src/napari_imagej/widgets/menu.py 86.69% <81.81%> (+0.33%) ⬆️
src/napari_imagej/java.py 94.61% <85.71%> (-0.68%) ⬇️
tests/types/test_converters.py 99.61% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

This also allows us to add colormap support for this Python -> Java
conversion
@gselzer gselzer force-pushed the 100-convert-imagedisplay-and-datasetview branch 2 times, most recently from ea6da6d to add77a3 Compare September 29, 2022 15:27
@gselzer gselzer force-pushed the 100-convert-imagedisplay-and-datasetview branch from add77a3 to d1c6d91 Compare September 29, 2022 15:40
@gselzer gselzer force-pushed the 100-convert-imagedisplay-and-datasetview branch 2 times, most recently from 860cbb3 to 8b002d4 Compare September 29, 2022 17:26
@gselzer gselzer marked this pull request as ready for review September 29, 2022 17:30
@gselzer gselzer force-pushed the 100-convert-imagedisplay-and-datasetview branch from 8b002d4 to 36383aa Compare September 29, 2022 18:10
@gselzer gselzer force-pushed the 100-convert-imagedisplay-and-datasetview branch from 36383aa to 9b2b699 Compare September 29, 2022 18:21
@gselzer gselzer merged commit a2bbae3 into main Sep 29, 2022
@gselzer gselzer deleted the 100-convert-imagedisplay-and-datasetview branch September 29, 2022 18:33
gselzer added a commit that referenced this pull request May 31, 2023
It was added with #107, but
imagej-common now has converters such that the above converter should
always win out
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for conversion of ImageDisplay and DatasetView Associate napari's ColorMap with ImageJ's LUTs
2 participants