Skip to content

Conversation

AdvancedImagingUTSW
Copy link
Collaborator

No description provided.

AdvancedImagingUTSW and others added 7 commits October 9, 2024 17:22
The log x axis creates some very annoying behaviors with the tick marks... I'm hoping that this is okay now.
Decided I'll simplify the histogram and only plot the values with log y and linear x...
May ultimately adjust the position of the histogram to a new tab on the bottom so that it can be built out further in the future.
Tried putting it in a tab, in its own notebook, which would be nice, but the size of the app is entirely off now.
@AdvancedImagingUTSW AdvancedImagingUTSW linked an issue Oct 13, 2024 that may be closed by this pull request
Copy link

codecov bot commented Oct 13, 2024

Codecov Report

Attention: Patch coverage is 90.75145% with 16 lines in your changes missing coverage. Please review.

Project coverage is 54.07%. Comparing base (a69c034) to head (0fbb4f7).

Files with missing lines Patch % Lines
...c/navigate/controller/sub_controllers/histogram.py 86.00% 7 Missing ⚠️
src/navigate/controller/controller.py 54.54% 5 Missing ⚠️
src/navigate/controller/configurator.py 25.00% 3 Missing ⚠️
.../navigate/controller/sub_controllers/keystrokes.py 94.44% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1014      +/-   ##
===========================================
+ Coverage    53.87%   54.07%   +0.20%     
===========================================
  Files          178      181       +3     
  Lines        20202    20263      +61     
===========================================
+ Hits         10883    10957      +74     
+ Misses        9319     9306      -13     
Flag Coverage Δ
unittests 54.07% <90.75%> (+0.20%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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.

Add Histogram for Visualization

2 participants