Add histogram bin adjustment ui #455
Conversation
CyclingNinja
force-pushed
the
histogram_ui
branch
from
9bda4ba
to
5a57128
Compare
| @@ -18,3 +18,5 @@ def __init__(self, viewer_state): | |||
| self.glue_state = viewer_state | |||
|
|
|||
| link_glue_choices(self, viewer_state, 'x_att') | |||
|
|
|||
| def axis_to_bins(self): | |||
There was a problem hiding this comment.
Have you tried including a vue_ prefix on this method?
There was a problem hiding this comment.
So I know I need to, but there's nothing behind the button just yet
|
|
||
| def vue_axis_to_bins(self): | ||
| # self.glue_state. | ||
| MatplotlibHistogramMixin._update_axes(self) |
There was a problem hiding this comment.
You shouldn't need to inherit from or use this mixin - there should be a method on the histogram viewer state class for syncing the bins to the current limits.
CyclingNinja
force-pushed
the
histogram_ui
branch
from
d47c9af
to
c726cda
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #455 +/- ##
=======================================
Coverage 86.04% 86.04%
=======================================
Files 90 90
Lines 5224 5232 +8
=======================================
+ Hits 4495 4502 +7
- Misses 729 730 +1 ☔ View full report in Codecov by Sentry. |
CyclingNinja
force-pushed
the
histogram_ui
branch
from
1878679
to
f8bdab8
Compare
|
|
||
| def vue_axis_to_bins(self, *args): | ||
| self.glue_state.update_bins_to_view() | ||
| print('Triggered axis to bins') |
There was a problem hiding this comment.
Can you remove the print statement?
| <glue-float-field label="x-max" :value.sync="glue_state.hist_x_max" />--> | ||
| </div> | ||
| <div> | ||
| <v-toolbar title="Toolbar"> |
There was a problem hiding this comment.
Is there any way to make the toolbar less imposing vertically?
|
|
||
| <v-btn variant="outlined" small @click="axis_to_bins"> | ||
| Fit Axes to Bins |
There was a problem hiding this comment.
I think this should be Update bins to view - it's the bins that are being updated not the axes. And this would then match the glue-qt wording.
Adds user interface to handle bin adjustments and fixing the axis to new bin ranges