-
Notifications
You must be signed in to change notification settings - Fork 0
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
Spec requirements #1
Comments
Got an API key for Met office. I did some queries to the API and it feels less friendly to use than open weather map. Can find sites through this call: http://datapoint.metoffice.gov.uk/public/data/val/wxfcs/all/json/sitelist?key= To get weather info, I need to use the location id after the data type along with a res parameter that only accepts 3hourly or daily.
|
Additional parsing would be required to match the rep names numbers to their actual meaning. |
The regional forecast call is useful because it outputs a description for what the weather will be like for the day night and next day:
finding the region id is required first for the call |
I think that having the summary is nice to have. Open weather API has subscriptions for certain things. Not too much of a difference for the free version because you will get access to 3 hour period forecasts. You will have to supply a long and lat though. On the other hand, I can make a call to the met office for the sites and set the default one as a cached var. Additional command called "List" which outputs all of the sites. The sites call doesn't provide the regional id so a separate call will need to be made to match the site name with the id. |
From a UX pov, having the output of all sites will be not all that friendly because there are over 5000. They wouldn't all fit on the screen. |
I should have the commands either accept a location or nothing. If the user passes through nothing, check the var defaultLocationID. If the user inputs the regional area, then output the sites for that area and tell them to use one of the listed sites for the command. |
Only the region name is returned which is a pain because I'll have to make another separate call to the region site list end point to get the id for it if I would want to query for the regional forecast summary |
Could find a better way to set the region and site id. I am making two requests and looping through all of the returned values to match what the user has supplied. Not necessarily the best solution but it works. |
Met office API doesn't have an end point for the current weather conditions. Parameters that it outputs are for daily and 3 hour intervals. Not sure how openweathermap is getting the current data and even hourly because they get their info from the met as their source. The time stamp is are for 3hourly or daily time stamps in the future. |
My thinking right now is to make a new bonzai cmd for "forecast". The commands could be:
|
Current thinking for the commands should be:
The weather API options are openweathermap or met office
Noticed that open weather map queries the met office anyway for their data
The text was updated successfully, but these errors were encountered: