Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Weather Fix #46

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Weather Fix #46

wants to merge 5 commits into from

Conversation

Drgabi18
Copy link

@Drgabi18 Drgabi18 commented Mar 8, 2019

pls dad

@Drgabi18
Copy link
Author

Drgabi18 commented Mar 8, 2019

image
Location might be a bit... borked since I wasn't able to find more parameters for the link to give shorter stuff, and the equally worse
Example if you wanna see the json
http://wxdata.weather.com/wxdata/weather/local/USVA0944?cc=*&unit=f&dayf=6&locale=en_US

@Drgabi18
Copy link
Author

Drgabi18 commented Mar 8, 2019

I just realized the current WeatherLocation variable will always return error because you need to specify a Location Code and not your city name, will get to fix it once I'm not bored

; Measure to get the weather data and properly parse it
[MeasureWeather]
;FindWeatherLocation
[FindWeatherLocation]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't seem to work properly.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Honestly, I'd just guide the user to find their weather code themselves. Otherwise you'd have to search for it every time Rainmeter starts.

@Revadike
Copy link

Honestly, I'd just guide the user to find their weather code themselves. Otherwise you'd have to search for it every time Rainmeter starts.

@Drgabi18
Copy link
Author

Drgabi18 commented Mar 24, 2019

Rainmeter stops halfway downloading the weather data and to fix that and also make it automatic like the original I added the abundance of updates and enablemeasures
I think I detalied in earlier edits that you can find the other locations by just going to wxdata's link and replace the last part with their city and blah blah. I guess I'll just make them find the exact name of the city, not the LocId cause that's boring for me.
Good thing I haven't done anything with ip geolocations hehe

@BoscoDomingo
Copy link

This has worked wonderfully for me. Literally changed a couple lines by hand in the way @Drgabi18 did and it is fully functioning. It fixes the Yahoo Weather problem!

@xiankai
Copy link

xiankai commented Jul 17, 2022

Sadly, the weather API used here is not working anymore.

The API returns "This service is deprecated as of 01/15/2020" when you try to access it directly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants