-
Notifications
You must be signed in to change notification settings - Fork 26
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
Different results compared to MerrySky #20
Comments
Currently briefsky can't distinguish Light Rain from Rain for PirateWeather: https://github.com/vsergeev/briefsky/blob/v1.4.0/src/providers/PirateWeatherProvider.ts#L5-L16 , because Light Rain isn't enumerated in the API for Icon: https://docs.pirateweather.net/en/latest/API/#icon . If it is a valid icon state coming back from PirateWeather, it should be easy to extend that mapping. If it's inferred from the other parameters, it may take a bit more logic to replicate MerrySky's conditions. Next step would be to double check the data coming back from the API to see where the light qualifier is coming from. |
I think the response payload between merrysky and briefsky is the same for pirateweather so i think there is additional processing happening to distinguish light precipitation vs heavy vs simple precipitation. Maybe this logic is being used and can be added to briefsky as well |
Also @vsergeev is there a way to add the scroll functionality on the weather bar similar to the merrysky? I like how you can scroll on the weatherbar and the weather forecast changes. That would be a cool feature to add. |
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 (Jan 26) shows
![image](https://private-user-images.githubusercontent.com/20840348/299437698-3648a54f-7548-4be1-9ccb-c3724ba3ac5e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk1ODIwMzMsIm5iZiI6MTczOTU4MTczMywicGF0aCI6Ii8yMDg0MDM0OC8yOTk0Mzc2OTgtMzY0OGE1NGYtNzU0OC00YmUxLTljY2ItYzM3MjRiYTNhYzVlLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDAxMDg1M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTkwOWVhMDQ0NWFkODI5ZDQxMTNiMTA3MDU4YzIwNzhjYTA0MWM5NTA1MjNhMGVhNGQyNTZiOTBmMDY5YjljZmQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.lOk38uwcWqGQKxRH6DS5HTjcmrXSg5SyRfglF7vna38)
where as briefsky shows
![image](https://private-user-images.githubusercontent.com/20840348/299437808-7658b46c-e17c-4649-b07a-23ae4ded8b48.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk1ODIwMzMsIm5iZiI6MTczOTU4MTczMywicGF0aCI6Ii8yMDg0MDM0OC8yOTk0Mzc4MDgtNzY1OGI0NmMtZTE3Yy00NjQ5LWIwN2EtMjNhZTRkZWQ4YjQ4LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDAxMDg1M1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWU1OTUxNjkwNGFjYTNhODg5ZGU2MTdlMjZlZTllNmI2NmRhZDNjZWY5OWYwNGU4OWFkYmNiNmI0ZGI4MTQ1ZTAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.kmqHhsnEaQnzYAXd_NUYf_3Z-eJgLjY8Tc3Q5vc0zg4)
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?
The text was updated successfully, but these errors were encountered: