Description

kushalkolar

opened on Jun 25, 2024

Issue body actions

currently to get the histogram LUT tool to use it manually the import is: from fastplotlib.widgets.histogram_lut import HistogramLUT. Let's expose this at top level, hopefully will be stable in a few months. And it's really not a widget but a "tool", maybe we should a new subpackage for tools?

@clewis7 @apasarkar thoughts?

Activity

1. clewis7 commented on Jun 26, 2024

   

   clewis7

   on Jun 26, 2024

   Member

   More actions

   I think if you do subpackage for tools that you would also need to move selectors there…they are more of tools also
2. kushalkolar commented on Jun 26, 2024

   

   kushalkolar

   on Jun 26, 2024

   MemberAuthor

   More actions

   Makes sense. If they're all top level it doesn't change the API.
3. apasarkar commented on Jul 18, 2024

   

   apasarkar

   on Jul 18, 2024

   Collaborator

   More actions

   Hmm. Do we have some concrete situations where it is used on its own (i.e. without ImageWidget)? If so, then I think this idea makes sense. I've added some related thoughts in #557
4. kushalkolar commented on Jul 19, 2024

   

   kushalkolar

   on Jul 19, 2024

   MemberAuthor

   More actions

   Hmm. Do we have some concrete situations where it is used on its own (i.e. without ImageWidget)? If so, then I think this idea makes sense. I've added some related thoughts in #557

   yes I was doing it for some stuff where I needed more control that imagewidget gives 😅
5. kushalkolar commented on Oct 1, 2024

   

   kushalkolar

   on Oct 1, 2024

   MemberAuthor

   More actions

   this is already done in #571
6. kushalkolar

   closed this as completedon Oct 1, 2024

Issue actions