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

Count trajectories: let user choose start/end timestamps by seconds #62

Closed
JuhoErvasti opened this issue Nov 1, 2024 · 1 comment
Closed
Labels
enhancement New feature or request information needed Further information is needed so that this can be implemented.

Comments

@JuhoErvasti
Copy link
Collaborator

JuhoErvasti commented Nov 1, 2024

Currently the seconds seem to be determined from the time you open the algorithm (not 100% sure), either way they can vary a lot leading to seemingly random results, which is why in #63 I changed it so that the seconds are always set to 0.

See TODO here.

        # TODO: Currently the datetime widget doesn't allow the user to set the seconds
        # so normalize them to 0 seconds, because they are being set seemingly
        # randomly leading to odd results. We need to check if selecting the time by
        # minutes is OK and if not we might need to write a custom widget
@JuhoErvasti JuhoErvasti added the bug Something isn't working label Nov 1, 2024
@JuhoErvasti JuhoErvasti self-assigned this Nov 1, 2024
@JuhoErvasti JuhoErvasti added enhancement New feature or request and removed bug Something isn't working labels Nov 1, 2024
@JuhoErvasti JuhoErvasti removed their assignment Nov 1, 2024
@JuhoErvasti JuhoErvasti added the information needed Further information is needed so that this can be implemented. label Nov 1, 2024
@JuhoErvasti
Copy link
Collaborator Author

Minute precision should be sufficient - closing as completed in 7ba46e9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request information needed Further information is needed so that this can be implemented.
Projects
None yet
Development

No branches or pull requests

1 participant