generated from readthedocs/tutorial-template
-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Precipitation Types #43
Labels
Comments
Closed
This issue was probably missed so will bump for visibility. Probably more of a long-term issue here so adding the keep label would be useful. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue is a continuation of the discussion from this issue Pirate-Weather/pirate-weather-ha#78 and since the discussion about adding other precipitation types.
I know that DarkSky is pretty limited when it comes to precipitation types (as well as icons) and I am aware that you want to keep things compatible with DarkSky and using flags/API options instead of breaking things. I know that WeatherKit has added in more icons/precipitation types to their API so maybe it's worth breaking things in this case?
Maybe the following could be added as
precipTypes
:freezingrain
orice
hail
mixed
unknown
orprecipitation
I thought about adding in
thunderstorm
but since it isn't a form of precipitation I decided to exclude it. I was thinking about thunderstorms today and it would be a nice addition to the API. I know there are ways to forecast using Lifted Index/CAPE/precipRate/wind speed/etc. and using the Blitzortung (#6) to show in the currently section.I would show the icon only if
precipType
is rain and it would always come after theprecipType
so if we use today as an example it would show Freezing Rain and Thunderstorms.EDIT: I might be misunderstanding the Categorical Ice/Rain/Snow/Sleet that HRRR offers. During precipitation only one of those four values will be 1 and the rest 0? If that's the case then maybe we can not bother adding in
mixed
as aprecipType
The text was updated successfully, but these errors were encountered: