search

kotelnik / plasma-applet-weather-widget

Plasma 5 applet for displaying weather information from yr.no server.
GNU General Public License v2.0
89 stars 42 forks source link

Reload the weather make the whole plasma desktop crash. #56

Closed trmdi closed 7 years ago

trmdi commented 7 years ago

When I click the Weather Widget icon on the panel then click the Reload button, the whole Plasma desktop crashes. Maybe the CPU is heavily used, I have to restart the computer.

Thank you for this very cool widget.

kotelnik commented 7 years ago

Thanks! Is this happening only when you click on the reload button? Does this happen when the widget is reloading automatically? Does this happen witl yr.no provider as well as Open Weather Map?

loutr commented 7 years ago

I have the same problem on my computer under Debian Testing. The informations are still displayed on hover and refreshed, but when I click on the reload button it crashes. It does crash only when trying to display graphs for locations chosen with OWM. This issue randomly appears after booting, but can be "resolved" by changing cities' order in the settings.

kotelnik commented 7 years ago

So far I'm quite sure this issue is related to a memory safety problem in XmlListModel component of Qt. I will investigate and possibly file a bug in Qt's Jira as soon as I manage to create a usable minimal example code.

kotelnik commented 7 years ago

I may have found a logical problem in my QML code (regarding OpenWeatherMap provider handler) and possibly fixed it. This seems to fix the crashing. I still think it should be fixed in Qt but in any case it should not happen since latest commit. Please reopen this issue if you still encounter this crashing.