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

Current precipitation unit #482

Closed
1 task done
jeroenterheerdt opened this issue Aug 26, 2024 · 3 comments
Closed
1 task done

Current precipitation unit #482

jeroenterheerdt opened this issue Aug 26, 2024 · 3 comments
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@jeroenterheerdt
Copy link
Owner

What happened?

Shouldn’t the unit of measurement be “mm/h” instead of “mm”?

How to reproduce

see above

Relevant log output

No response

Which version are you running?

2024.8.2

Diagnostics file

  • I have attached a diagnostics file

Additional information

No response

@jeroenterheerdt jeroenterheerdt added the bug Something isn't working label Aug 26, 2024
@jeroenterheerdt jeroenterheerdt self-assigned this Aug 26, 2024
@darek-margas
Copy link

Well, I have sensor with mm/m so maybe it should be with various times?

@jeroenterheerdt
Copy link
Owner Author

Well, I have sensor with mm/m so maybe it should be with various times?

Time is easy to convert using a template...

@jnsflschr
Copy link

Yes I agree. The naming "current" implicates some time relation because it iss not possible that it is instantaneous. Is this a mistake in the naming or in the description and usage? If the naming is correct, the expected aggregation should also be changed to "last".

@jeroenterheerdt jeroenterheerdt added the documentation Improvements or additions to documentation label Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants