Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to record time for an exercise that normally records reps #12

Open
quinncomendant opened this issue Sep 5, 2016 · 4 comments
Open

Comments

@quinncomendant
Copy link

I'm not able to record a time for an exercise that normally records reps, for example wrist exercises. Recording time does work for those that expect to record time (body-line exercises). Example:

unable to record time-2

@mazurio
Copy link
Owner

mazurio commented Sep 6, 2016

Thanks for raising.

This is not a bug. There are two types of exercises "weighted" and "timed", all reps/weight based exercises are "weighted" whereas "timed" exercises contain only timer (e.g. Plank).

"weighted" exercises cannot log time whereas "timed" exercises cannot log weight/reps.

This is both a design decision and a feature.

Wrist Mobility Exercise is the only one that's really tricky. It should be timed exercise but it was put as a "weighted" from the beginning.

I will look into whether that can be changed now without messing future graphs/data in the DB.

@quinncomendant
Copy link
Author

If an exercise cannot be both "timed" and "weighted" then I wonder if it would be good to force the user behavior by allowing only timer or reps counter. Why offer a timer when it's recommended to measure reps instead, and vise versa?

@mazurio
Copy link
Owner

mazurio commented Sep 6, 2016

You are right, it shouldn't be there. The only reason why it was there at the beginning was to make the transition from "timer only" to "reps logger" smoother as it was a new feature at the time.

Another reason was that some people use timer as a "rest timer" which is not implemented yet.

For Wrist Mobility I use timer to do the exercise for 2 minutes but then log 5 reps which is bit annoying but works :)

@quinncomendant
Copy link
Author

It's even questionable to log any of the warmup at all. I'm new to BWF, so maybe I'm wrong (I'm using the app as a learning aid). So far as I understand, warm up isn't progressive and won't increase (e.g., why would you ever do more than 2 min wrist mobility?). I'll probably be following Antranik's warmup flow, and log nothing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants