sagreine / homeGym

homeGym mobile app
2 stars 0 forks source link

graph PR progress over time #147

Open sagreine opened 3 years ago

sagreine commented 3 years ago

sync with the other tabs (by passing in a variable)

sagreine commented 3 years ago

need to deal with dark mode color scheme for bar chart

sagreine commented 3 years ago

1) keep the colors constant if not updating? use seeding

2) order the values correctly. might only be an issue in the over time bar chart but might be true for squat too (that might just be a test data problem though) - 
definitely an issue here.
rep maxes seem fine aside from ordering, but weight maxes aren't using the right dataset - hard to say what dataset they are using?
also an ordering issue in the

2) reset when you click next

sagreine commented 3 years ago

bar charts appear finished! on to line data for all of this, should we have a From and To date? or at least a From date? could animate along that slider for bar chart I'd want to show that in my line chart / bar chart you're probably not really setting PRs all that often.. not like you'll have thousands of them or anything we're already pulling it all in anyway..

sagreine commented 3 years ago

to barcharts: add a "total" stacked bar chart? as of a date. maybe just for e1RM or 1RM or something. similarly, line chart should have a "all lifts" page and include a total line, again maybe just for 1RM.

sagreine commented 3 years ago

barebones per-lift line chart is in but need to fix titling. may want to mess with line options. a bit tricky to store max e1RM, no? or will have to loop every dataset and calculate it..... definitely store it on the way in but annoying. the thinking here is that people aren't going to be doing 1RM tests all that often and want to see their progress.

Need to add a "current day = last lifted PR" Pr to each list? to show this is still your PR...

next features: timeline (start, end dates for inclusion) pinch zoom

sagreine commented 3 years ago

titling is in. a messsssss though. just being stupid about it.

sagreine commented 3 years ago

zoom stuff exists, but would need to override swipes and do it custom.

sagreine commented 3 years ago

need to add combined line graph for e1rm or just actual 1rm

sagreine commented 3 years ago

graph is now a stepline chart. still need e1rm / actual 1rm hmmmmmm

sagreine commented 3 years ago

i'm not sure what to show for 1RM. do people actually test it directly that often?

e1RM is .... how/where/why would we track that? I guess we could do it on the way in and separately calculate and sort it, then show the max for a given day if it is greater than our max so far? 'touch the sky' kind of appraoch.

should we use the 5/3/1 lift values? and track those to show here?

sagreine commented 3 years ago

it should be easy enough to do this on the way in, for all lifts. most of the code is already there, probably. just a matter of doing it...

also, would show stacked bar chart, with total as the value so could just have "All lifts" in rotation.