ctmm-initiative / ctmmweb

Web app for analyzing animal tracking data, built upon ctmm R package
http://biology.umd.edu/movement.html
GNU General Public License v3.0
34 stars 22 forks source link

Time sub-setting - improvement #30

Closed vestlink closed 7 years ago

vestlink commented 7 years ago

I was using the app with my data, and was missing the opportunity to be able to define a time-span (from date - to date), when in the time-subsetting pane.

Maybe two input fields in the "Current Time Range"?

xhdong-umd commented 7 years ago

So you feel it's easier to use a date picker instead of dragging on the histogram? Is it because the mouse selection is difficult to get specific dates? You can drag a range first, then move the range left and right, or only move one end a little bit.

Currently a lot of things will update when the mouse selection on histogram changed. Ideally I will want the selection on histogram to sync with date picker so the app will be consistent and the changes are minimal. However Shiny doesn't support setting selection programmatically yet.

If we still add the date picker, then we have two sources of truth for the date range. It's difficult to determine which one is the wanted input when both exist.

We can add a button for date picker, so clicking button will override the mouse selection. Though I'll need more manual management code to deal with all the corner cases. Previously this is all automatically handled by Shiny. Now I have to manually manage them when explicit override was introduced.

@chfleming Did you find it's easier to use a date picker?

chfleming commented 7 years ago

I think its very likely that some users will want to input specific dates (e.g., hunting season).

xhdong-umd commented 7 years ago

I added a manual date picker. After dates are selected, click the set button to apply. I didn't add hour/minutes because I guess date is enough.

screen shot 2017-09-07 at 2 29 40 pm