-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
[FR] (Multiple user-support HR->PWR) #333
Comments
I like the idea! I'll put it on the list but it could be quite a while before I get around to it. In the meantime if you'd like to check into it, I'd recommend just creating an interface to select user profiles in: Adding a vector parameter to the physicalWorkingCapacity Class to hold the list of user names or numbers and the index of the selected one. SmartSpin2k/include/SmartSpin_parameters.h Lines 163 to 169 in 1f8d202
Then refactoring the physicalWorkingCapacity::loadFromSPIFFS() function in SmartSpin_parameters.cpp to save this info. SmartSpin2k/src/SmartSpin_parameters.cpp Lines 277 to 309 in 1f8d202
Then refactoring the calculateInstPwrFromHR() function to use the new data formats above. SmartSpin2k/src/BLE_Server.cpp Lines 557 to 588 in 1f8d202
|
Is your feature request related to a problem? Please describe.
I have not yet built SS2K, but since I had an esp32 lying around, I would give it shot for testing. I've noticed that there isn't support for multiple users (When using HR for power calculation)
Describe the solution you'd like
Add the ability to save and select hr->pwr settings for different users
Describe alternatives you've considered
Re-enter hr->pwr settings each time before use
The text was updated successfully, but these errors were encountered: