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.

Sign up for GitHub

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

Jump to bottom

Statistics (aggregate per day (run/lift) - Total time skiing (moving) #69

Open
beingtranquil opened this issue Mar 15, 2024 · 2 comments
Open

Statistics (aggregate per day (run/lift) - Total time skiing (moving) #69

beingtranquil opened this issue Mar 15, 2024 · 2 comments
Assignees
@Sharanyu @divawalazeel @beingtranquil
Labels
Enhancement New feature or request Group 9 Label for group 9 Medium Priority Medium Priority UI It is a UI feature

Comments

@beingtranquil
Copy link
Collaborator

Describe briefly your feature.
Create a function to determine the total time skiing and create a UI element for displaying this statistic.

Describe the solution you'd like
From the outcome of Group #4-6 - Statistics (service functions, Ski statistics), we will have distance skied, average speed, so for calculating time, we can divide distance by speed.

Describe alternatives you've considered
There is no alternative for this.
@beingtranquil beingtranquil added Enhancement New feature or request Group 9 Label for group 9 Medium Priority Medium Priority UI It is a UI feature labels Mar 15, 2024
@beingtranquil beingtranquil self-assigned this Mar 15, 2024
@Sharanyu Sharanyu mentioned this issue Apr 9, 2024
Closed
@Sharanyu
Copy link
Collaborator

Sharanyu commented Apr 9, 2024

Update:

@Sharanyu Sharanyu mentioned this issue Apr 9, 2024
Merged
Sharanyu added a commit that referenced this issue Apr 9, 2024
@Sharanyu
Created the UI Element for the total time skiing (#204)
4087a57 
Changes for Issue 69 by Group 09

Description

- Updated the UI element, defined a dummy function to return the total
distance skiied.
- created our own Java class for calculating the daily ski statistics
src/main/java/de/dennisguse/opentracks/ui/aggregatedStatistics/daySpecificStats/DailySkiStatistics.java.
- Partially completing the issue due to merge conflict and dependancy
from group 6.

Link to the the issue
#69
@beingtranquil
Copy link
Collaborator Author

I updated the feature after getting the functionality from group 6 and I have merged the following PR:

#206

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sharanyu Sharanyu

@divawalazeel divawalazeel

@beingtranquil beingtranquil

Labels
Enhancement New feature or request Group 9 Label for group 9 Medium Priority Medium Priority UI It is a UI feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Sharanyu @divawalazeel @beingtranquil