select_points / session-ful marking about astrowidgets HOT 3 CLOSED

If we want to support "sessions", then we also need to support merging/splitting/deleting/saving/loading sessions? Or am I over-thinking this?

from astrowidgets.

I'm also inclined at this point to switch from making self.is_marking a settable property to adding a method called start_marking for a few reasons:

• I'm encountering cases where it would be handy to:
  • name different sets of markers that are added interactively. (.e.g by labeling them "bad comparison star" or something).
  • set the marker properties along with the name
• Better symmetry with stop_marking, currently part of the API. Right now one could do either .is_marking = False or .stop_marking().

The first point requires either switching to a method to initiate marking or creating some kind of Marking object for specifying marker metadata.

The second point could be addressed by removing stop_marking but that method has an option to clear the markers when marking is stopped.

@pllim -- if you give this change a thumbs up I'll implement it this morning.

from astrowidgets.

@mwcraig , since you have a real use case for this, 👍 from me. Thanks!

from astrowidgets.