-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add histogram #49
Comments
If you want to assign this to me @tlambert03 I'm happy to keep hacking along 😄 I had some further discussions with @marktsuchida and @nicost yesterday, which I'll summarize below: Additional Wants:
Backend Considerations:
my current plans (up for debate/revision):
|
thanks @gselzer! I do want you to keep hacking along, but I do really want to organize my work in https://github.com/tlambert03/napari/tree/histogram for you to have a look at as well. As mentioned, this is something that I spent a good month on at one point and so I'd like to make sure that all the stuff learned back then is accounted for here as well. I started to do this in https://github.com/tlambert03/ndv/tree/histogram (see the |
I had this working in napari/napari#675 (comment)
these are all great, but should live in the statistics calculating thing (not the histogram PR). Let's just make a histogram component, and then think about how it lives within a more complex composite statistics display. The histogram widget basically just needs to take bins and show them. So, if you want to open an independent PR to discuss a (non-GUI) stats engine, that would be great too. Basically, the GUI object should be disconnected from the calculation object. |
another thing that that previous PR had was a shared mechanism for line scans: napari/napari#675 (comment) so, it should be possible to share a lot of logic between a plot widget and a histogram widget (namely, keeping axes aligned with data and pan-zoom stuff) |
Yes, definitely important, let me know when you feel like things are in a good spot for review and I'll be happy to read over things.
I agree on multiple PRs, I was partially using this comment as a way to pin down all of my thoughts in the open before the action items slipped from my memory 😅 |
Also worth noting these operations are rather interconnected with the existing ROI interoperability. We may want to track a separate issue for expanding ROI capabilities (adding additional ROIs like lines, ellipses, polygons, etc., as well as signals to keep track of their changes that the statistics engine could listen for). |
ok @gselzer, have a look at https://github.com/tlambert03/ndv/tree/histogram it's the bones of "plot" widget, where |
opening an issue to discuss adding a histogram (which everyone agrees is a must-have feature)
@gselzer has done a lot of work here, currently in https://github.com/gselzer/ndv/tree/interactive-clims
I had a similar vispy-backed histogram that controlled the napari LUTs back in napari/napari#675 ... at one point, I had put a lot of work/consideration into it, and that branch is still at https://github.com/tlambert03/napari/tree/histogram
backend considerations
we have a few options for tools we could use to create the actual histogram graph
wants
here's a partial list of things that one might want in a histogram. not all of these are mandatory for an initial implementation.
The text was updated successfully, but these errors were encountered: