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

Added stats: total path lenght, on time, motors on time #52

Merged
merged 2 commits into from
Aug 31, 2021
Merged

Added stats: total path lenght, on time, motors on time #52

merged 2 commits into from
Aug 31, 2021

Conversation

texx00
Copy link
Owner

@texx00 texx00 commented Aug 25, 2021

Stats are saved in: server/saves/stats.json

The file can be improved in the future. Can also be intersting to show them in the interace in some way.

@texx00
Copy link
Owner Author

texx00 commented Aug 25, 2021

Commit f33706b should also have fixed a problem with eta estimatio (was saving with = a recursive object instead of creting a copy)

@texx00 texx00 added Backend Something related to the backend enhancement New feature or request labels Aug 25, 2021
@texx00
Copy link
Owner Author

texx00 commented Aug 31, 2021

Stats are working.

There is a bug with the total path lenght for Scara and polar tables, thus that value is reliable only for cartesian devices (#54). Needs to fix it later (#53)

@texx00 texx00 merged commit 799ea60 into alpha Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Backend Something related to the backend enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@texx00