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

Populate "BF Channel" list with presets containing keywords #287

Merged
merged 6 commits into from
Nov 29, 2022

Conversation

talonchandler
Copy link
Collaborator

Before this PR, the "BF Channel" list would be populated with all MM presets, which would lead to confusing defaults like this:
image

After this PR, the "BF Channel" list will only contain MM presets that have any of the following case-insensitive keywords as a substring:
["bf", "brightfield", "bright", "labelfree", "label-free", "lf", "label"]

I checked these keywords against the latest MM configs on CompMicro_MMConfigs and tested in person on hummingbird and falcon. Feel free to suggest/add other keywords.

On hummingbird, the startup menus now look like this (similar on falcon):
image

@codecov-commenter
Copy link

codecov-commenter commented Nov 24, 2022

Codecov Report

Merging #287 (f0f7791) into main (4af72d3) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main    #287      +/-   ##
========================================
- Coverage   4.38%   4.37%   -0.01%     
========================================
  Files         22      22              
  Lines       4974    4979       +5     
========================================
  Hits         218     218              
- Misses      4756    4761       +5     
Impacted Files Coverage Δ
recOrder/plugin/main_widget.py 0.00% <0.00%> (ø)
recOrder/compute/qlipp_compute.py 0.00% <0.00%> (ø)

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

@deprecated-napari-hub-preview-bot
Copy link

deprecated-napari-hub-preview-bot bot commented Nov 24, 2022

Preview page for your plugin is ready here:
https://preview.napari-hub.org/mehta-lab/recOrder/287
Updated: 2022-11-29T01:38:17.535859

@ieivanov
Copy link
Contributor

Another good keyword may be phase

@talonchandler talonchandler merged commit f5978a6 into main Nov 29, 2022
@talonchandler talonchandler deleted the bf-preset-keyword branch December 3, 2022 18: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.

3 participants