Quansight / lsst_dashboard

LSST Dashboard https://quansight.github.io/lsst_dashboard/
BSD 3-Clause "New" or "Revised" License
8 stars 3 forks source link

Rework overview plot to only show available tracts. #155

Open dharhas opened 4 years ago

dharhas commented 4 years ago

We need to rework overview plot to only show available tracts.

@timothydmorton I assume we still want the ability to select a subset of tracts.

@brendancol, @philippjfr is there a way to disable the skyplot and detail view until a selection is made on this tab?

timothydmorton commented 4 years ago

Yes, we will need ability to select subset of tracts.

I'm not sure what you mean above by "only show available metrics." The metrics being displayed on the overview plot are a completely different set of quantities from what we call "metrics" later (which are just raw table columns).

Regarding the first point here, let's continue to use the single overview dataset that we have, as there is not yet a mechanism to produce that generically from the LSST pipeline. Probably just having a load_overview_data() function that can be updated by us in the future to accommodate a new repo-specific dataset of the same format should be fine. This can still for now be loaded upon selection of a data directory, even if it stays a generic thing for now.

timothydmorton commented 4 years ago

The other point to add here is to make sure that the load-data action that happens upon tract selection doesn't happen until the tract selection is finished (e.g., not ever tap action should trigger a new load). Also, add the option box-select on the polygons, if that isn't already done.

This might involve a "finalize tract selection" button or something to that effect.

dharhas commented 4 years ago

@timothydmorton meant to say 'tracts' not 'metrics'