-
-
Notifications
You must be signed in to change notification settings - Fork 29
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
Differnce between MerrySky and BriefSky #142
Comments
From what I can tell MerrySky does their own processing on the hourly data to categorize them as Light Rain/Rain/etc. whereas BriefSky is showing the data directly from the API. I also see you created this issue on BriefSky which I'll link here as well. There is already an issue open to implement text summaries/descriptions (#48) but it seems to be more of a long-term goal so I wouldn't expect it as part of the V2 update. I did mention in that issue that maybe getting the translations/descriptions working would be a good first step and then the summaries can come later. Another thing to note is that MerrySky does have caching enabled which is why there may be differences between the two outputs as it seems like BriefSky uses your own API key and gets the most up-to-date data available. |
I looked at the dark sky translation repo. I am not sure if i understand the concept. Seems like their API was producing structured expressions that were then used to get a human readable summaries from the translation files (en.json) for example. Would pirate weather need to create the structured expression for this to work? |
Yes, DarkSky used the translation repo in their app to produce the summaries/descriptions. We don't know how they used the repo in their API since the code that produced the API isn't open source. @alexander0042 would need to write the code in order to produce the human readable summaries using the translation repo. I believe he mentioned that he was working on creating the human readable summaries before he started work on V2 of the API. |
I see, Do you know if there is a way to run pirate weather locally or self host it with the complete architecture? Maybe if that was possible we could have other people look into it |
Unfortunately there is no way to self-host or run the API locally at the moment. There is this issue #11 and it's part of the V2 road map. |
@alexander0042 any plans to use https://github.com/blaylockbk/Herbie? seems like this would help with processing the grib files? |
He did submit a PR in the Herbie repository so it looks like he is intending to use it in V2 of the API. We're starting to get a bit off-topic here so unless the dev of BriefSky wants to implement his own processing you'd have to wait until #48 is solved for the inconsistencies end. If you'd like to discuss about open-sourcing/self-hosting you can do so here #11 Unfortunately, I cannot provide any more information as I don't have any access to the back-end code so you'll have to wait for @alexander0042 to reply with more info. He usually tries to respond to items on GitHub a couple of times a week depending on how busy he is. |
Hey, thanks for opening this issue, and also cluing me into Briefsky, which I hadn't seen before! To answer your initial question- I suspect the reason they're different is because Merrysky always calls the midnight forecast, which is a clever workaround to the daily high/ daily low bug; however, sometimes leads to differences in data throughout the day. V2 addresses this (including using the wonderful Herbie), so should fix this problem soon! The self hosting question is a long running one, My goal here is to make the code open so people know where their weather data is coming from; hpowever, since everything is so tied into AWS actual self hosting is going to be very tricky to make possible. To be honest, there are some really cool AI forecasting products coming online now, and I think that's a better approach for anyone that wants to self host something. I can't think of a good use case of self hosting a weather API of the model data is still being provided by the cloud. And text descriptions are high on the list, just need to code it! It'll be after V2, but no too far after |
@alexander0042 Thanks for the input! I will look into how to all the midnight forecast to address the bug until V2 is out. Also the only missing part from the open source is how to extract data from the processed file when a query for a certain latitute and longitude comes in. I dont know how to how that process is being done and it would be cool to learn. |
There has been no activity on this issue in the last ninety days and it will automatically close in seven days. Leave a comment on this issue to prevent it from closing automatically. |
This issue has been automatically closes as there has been no further activity in seven days. Feel free to re-open this issue or you can create a follow-up issue. |
Describe the issue
When querying the weather info between two apps (MerrySky and BriefSky) I can see the two apps showing different conditions for a given day. For example, the merry sky ui for Friday shows
where as briefsky shows
![image](https://private-user-images.githubusercontent.com/20840348/299437808-7658b46c-e17c-4649-b07a-23ae4ded8b48.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzMTkzNTksIm5iZiI6MTczOTMxOTA1OSwicGF0aCI6Ii8yMDg0MDM0OC8yOTk0Mzc4MDgtNzY1OGI0NmMtZTE3Yy00NjQ5LWIwN2EtMjNhZTRkZWQ4YjQ4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEyVDAwMTA1OVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTFmMTg5YTQ2ZWVmNTM3NDczODgwNTUwNDA5M2RhNzY0OWMyMWI1MGQyNjk1OGM0MDllMTNkODE5MjVjMGJlZTkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.hkoVwv54DmLi9Tcn7n729O2rYc4BnaYr7GvqjouK6NE)
As you can see that merrysky distinguishes between light rain and rain throughout the day. However briefsky doesnt.
Looking at the response for the hourly data at 30097 (no lat long provided for privacy reasons) i dont see any summaries that have light rain so not sure how merry sky is accomplishing this?
Acknowledgements
The text was updated successfully, but these errors were encountered: