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

[feature] support monocle time with multiple timestamps and relaxed input validation #53

Open
digizeph opened this issue Mar 5, 2024 · 0 comments
Open

[feature] support monocle time with multiple timestamps and relaxed input validation #53

digizeph opened this issue Mar 5, 2024 · 0 comments
Labels
enhancement New feature or request time Issue related to time subcommand
Milestone
Better paralleliz...

Comments

@digizeph
Copy link
Member

digizeph commented Mar 5, 2024

Multiple timestamps

Should support monocle time 1640995200 1641081600

Relaxed input validation

Would be great to support non-RFC3339 inputs with some reasonable default values:

  • 2022-01-01 -> 2022-01-01T00:00:00Z
  • 2022-01-01T01 -> 2022-01-01T01:00:00Z

humantime's parse_rfc3339_weak could be used here.

In short, treat any string with non-numeric characters as time string and try to parse it as much as possible.
@digizeph digizeph added enhancement New feature or request time Issue related to time subcommand labels Mar 5, 2024
@digizeph digizeph mentioned this issue Apr 30, 2024
Merged
@digizeph digizeph added this to the Better parallelization milestone Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request time Issue related to time subcommand
Projects
None yet
Milestone
Better parallelization
Development

No branches or pull requests
1 participant
@digizeph