D29144: [applets/weather] Improve UX for selecting a weather station

spstarr requested changes to this revision. spstarr added a comment. This revision now requires changes to proceed. Hoping to get back into working on the backend as for how many, well depends how many weather providers will provide free API keys to use, I guess this is ok for now then. +1

D29144: [applets/weather] Improve UX for selecting a weather station

spstarr added a comment. What happens if we have more Weather Providers won't the dialog box be cluttered? REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D29144 To: ngraham, #vdg, #plasma, broulik Cc: spstarr, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev,

D25509: [applets/Weather] Port config UI mostly to QQC2 and away from Plasma components

spstarr added a comment. For the applet side, I didn't rewrite this, the original applet was non-QtQuick even, but even so, this looks sane to me. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D25509 To: ngraham, spstarr, #plasma Cc: plasma-devel, LeGast00n,

D25413: [Weather] Make compact representation temperature text match size of clock text

spstarr added a comment. as for the change, it looks good to me.. REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D25413 To: ngraham, #vdg, #plasma Cc: spstarr, filipf, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot,

D25413: [Weather] Make compact representation temperature text match size of clock text

spstarr added a comment. Probably could also say the same for, winds/visibility those could be rounded in some places depending on the unit(s). REPOSITORY R114 Plasma Addons REVISION DETAIL https://phabricator.kde.org/D25413 To: ngraham, #vdg, #plasma Cc: spstarr, filipf, plasma-devel,

D25413: [Weather] Make compact representation temperature text match size of clock text

spstarr added a comment. I wonder, some users might not want to see the temperature decimal points, maybe an option to hide it? The reason I exposed that level of temperature in the engine was when you live in cold climates the exact decimal temperature might help to tell you if there was

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. When those land I'll have a PR for EnvCan provider/ion to support those too REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau, #plasma, spstarr Cc: spstarr,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. Awesome! REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau, #plasma, spstarr Cc: spstarr, plasma-devel, #vdg, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. I note for Environment Canada they have these Night Blowing snow: https://weather.gc.ca/weathericons/40.gif Day Blowing snow: https://weather.gc.ca/weathericons/25.gif Day and night(?) windy: https://weather.gc.ca/weathericons/43.gif Blowing smoke (day and

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. Now I'm wondering if I can get a blowing snow icon ;) REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau, #plasma, spstarr Cc: spstarr, plasma-devel, #vdg, LeGast00n,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. Still looks good to me REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau, #plasma, spstarr Cc: spstarr, plasma-devel, #vdg, LeGast00n, The-Feren-OS-Dev, jraleigh,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. We do need another icon, Wind would be generic for Dust storm, or Blowing smoke, Blowing snow in this case. REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. 'breez' or breeze' just curious here REPOSITORY R120 Plasma Workspace BRANCH windy (branched from master) REVISION DETAIL https://phabricator.kde.org/D24483 To: ngraham, kossebau, #plasma, spstarr Cc: spstarr, plasma-devel, #vdg, LeGast00n, The-Feren-OS-Dev,

D24483: [Weather data engine] Handle windy conditions in NOAA backend

spstarr added a comment. No degree, but I have been following weather for years and on some mailing lists with meteorologists. 'Wind' can be a weather condition especially if its a dust storm. If NOAA lists it as a weather condition. I think this is fine. In Canada we do have Wind

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr closed this revision. spstarr added a comment. Merged into master, Plasma/5.16. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D23307 To: spstarr, #plasma, ngraham, kossebau Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr added a comment. In D23307#516304 , @kossebau wrote: > Can you push this to the 5.16 branch? No new string for translators, so should be fine to improve for stable. Sure thing, ill do in morning REPOSITORY R120 Plasma

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr added a comment. Ideally, this list shouldn't be so long since the XML schema http://dd.weatheroffice.ec.gc.ca/citypage_weather/docs/current_conditions_icon_code_descriptions_e.csv lists them all, except thats not what the XML shows :/ I should ask EC why the discrepancy REPOSITORY

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr added a comment. Thats the XML however, it is matching 'rainshowers' REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D23307 To: spstarr, #plasma, ngraham Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen,

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr added a comment. Current XML shows (until the storm ends) Thunderstorm with heavy rainshowers REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D23307 To: spstarr, #plasma_workspaces Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr added a comment. Will merge once reviewed. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D23307 To: spstarr, #plasma_workspaces Cc: plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf,

D23307: [weather] [envcan] Add additional current condition icon mappings

spstarr created this revision. spstarr added a reviewer: Plasma: Workspaces. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. spstarr requested review of this revision. REVISION SUMMARY [weather] [envcan] Add additional current condition icon mappings TEST PLAN Built

Re: Review Request 108798: Fix most of the compile issues due to wrong header used and comment out in shell/CMakeLists.txt testapplet which doesn't exist

/CMakeLists.txt ecbfad9 Diff: https://git.reviewboard.kde.org/r/108798/diff/ Testing --- Able to build things, but didn't look into why proper KDE libraries not being linked in yet (editing the generated CMakefiles/component.dir/link.txt works around this for now) Thanks, Shawn Starr

Building kde-workspace (plasma parts) for Plasma 2.x?

Hello folks, I've got KDE Frameworks 5 / Plasma 2.x built now, but kde-workspace still is using Qt4/KDE4 cmake configurations. When do we plan to move kde-workspace into plasma-frameworks? Thanks, Shawn ___ Plasma-devel mailing list

Review Request 108798: Fix most of the compile issues due to wrong header used and comment out in shell/CMakeLists.txt testapplet which doesn't exist

, but didn't look into why proper KDE libraries not being linked in yet (editing the generated CMakefiles/component.dir/link.txt works around this for now) Thanks, Shawn Starr ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman

Re: Review Request 108798: Fix most of the compile issues due to wrong header used and comment out in shell/CMakeLists.txt testapplet which doesn't exist

--- Able to build things, but didn't look into why proper KDE libraries not being linked in yet (editing the generated CMakefiles/component.dir/link.txt works around this for now) Thanks, Shawn Starr ___ Plasma-devel mailing list Plasma-devel

Request - Amend GPLv2+ license - Exemption link clause - for KDE weather engine/ions (weather related only) for BlackBerry 10/QNX

Hello folks, I need to get approvals from those who made some changes to the engine code, the change is to append the exemption clause to GPLv2+ so that I can link the binaries to QNX's libc / and RIM's cascades libs, as these are considered core to the OS. I also want to know if we can dual

Re: Request - Amend GPLv2+ license - Exemption link clause - for KDE weather engine/ions (weather related only) for BlackBerry 10/QNX

On Thursday, January 17, 2013 09:47:27 PM Matthew Dawson wrote: On January 17, 2013 08:06:05 PM Shawn Starr wrote: Hello folks, I need to get approvals from those who made some changes to the engine code, the change is to append the exemption clause to GPLv2+ so that I can link

Re: Request - Amend GPLv2+ license - Exemption link clause - for KDE weather engine/ions (weather related only) for BlackBerry 10/QNX

On Thursday, January 17, 2013 11:46:49 PM Matthew Dawson wrote: On January 17, 2013 09:51:45 PM Shawn Starr wrote: On Thursday, January 17, 2013 09:47:27 PM Matthew Dawson wrote: On January 17, 2013 08:06:05 PM Shawn Starr wrote: Hello folks, I need to get approvals from those

Re: Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

On Tuesday, September 11, 2012 08:05:50 AM Martin Gräßlin wrote: Am 11.09.2012 05:29, schrieb Shawn Starr: Hi Luís, I appreciate you making changes to the code, but it would have been nice to been told about this since I am about to replace this with QML porting. Please discuss

Re: Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

up graphic elements is however. Thanks, Shawn. On Tue, Sep 11, 2012 at 5:55 AM, Marco Martin notm...@gmail.com wrote: On Tuesday 11 September 2012, Shawn Starr wrote: Hi Luís, I appreciate you making changes to the code, but it would have been nice to been told about

Re: Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

. Thanks, Shawn. On Tue, Sep 11, 2012 at 8:15 AM, Shawn Starr shawn.st...@rogers.com wrote: On Tuesday, September 11, 2012 07:58:16 AM Luís Gabriel wrote: Hi Shawn, I don't see the point of revert the changes. Sorry about the misunderstanding but as you see in the previous replies

Re: Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

On Tuesday, September 11, 2012 01:30:50 PM David Edmundson wrote: On Tue, Sep 11, 2012 at 1:05 PM, Shawn Starr shawn.st...@rogers.com wrote: On Tuesday, September 11, 2012 08:30:50 AM Luís Gabriel wrote: You can see some screenshots here: https://git.reviewboard.kde.org/r/106225

Changes to Plasma Weather - Git revision 5a09cd815edd7723800c0ba43a8be9f67d28e011

Hi Luís, I appreciate you making changes to the code, but it would have been nice to been told about this since I am about to replace this with QML porting. Please discuss with me what changes you would like to make as I am in the process of porting to QML for the applet. My goals for '2.0' are

Re: KDE Weather engine

On Tuesday 19 January 2010 17:05:19 Thomas Richard wrote: Hey Shawn, When looking at the SVN source code i found out you are the original writer of the weather engines currently in KDE. Great work! Unfortunately the weather station close to my city is not included in any of the engines.

Re: Yahoo! Weather ion dataengine review

On Tuesday 12 January 2010 02:06:04 Aaron J. Seigo wrote: On January 11, 2010, Amos Kariuki wrote: Marco Martin wrote: On Monday 11 January 2010, Aaron J. Seigo wrote: On January 10, 2010, Amos Kariuki wrote: Is there a need to use both kdereview (svn) and reviewboard or should I

Re: Yahoo! Weather ion dataengine review

On Monday 11 January 2010 05:26:12 Marco Martin wrote: On Monday 11 January 2010, Aaron J. Seigo wrote: On January 10, 2010, Amos Kariuki wrote: Is there a need to use both kdereview (svn) and reviewboard or should I just submit a patch for review in reviewboard and ignonre kdereview?

Plasma applet API breakage for URL clicking?

I've noticed that you can no longer click text links in a plasmoid? Can anyone else confirm this? I'm currently fixing issues as I slowly ramp back into development. Thanks, Shawn. ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Forecast-only weather ion

On October 9, 2009 07:11:38 pm Thilo-Alexander Ginkel wrote: On Thursday 01 October 2009 01:48:29 Shawn Starr wrote: Now that my Weather Ion is mostly feature-complete, how does the further process look like? Should I submit a review request? Should it first go to the playground? (Feel

Re: Forecast-only weather ion

On September 29, 2009 09:14:42 pm Thilo-Alexander Ginkel wrote: Hello everyone, I am currently developing a Weather Ion for the German wetter.com weather data provider to improve the station coverage for Germany, Austria and Switzerland. Unfortunately, wetter.com only provides forecast

Re: Forecast-only weather ion

On September 30, 2009 03:14:01 pm Thilo-Alexander Ginkel wrote: On Wednesday 30 September 2009 19:20:55 Shawn Starr wrote: Unfortunately, wetter.com only provides forecast data through its free web service API, so the Ion actually has no access to current observation data to provide

Re: Review Request: Use sunrise and sunset times from the Environment Canada weather service to set weather icon to day/night, accordingly.

On 2009-05-03 16:42:23, Shawn Starr wrote: It seems to be working so far, it's already committed. Beat Wolf wrote: i can't find this code in svn, is it really commited? can somebody mark this as submitted? Its been redone. But we still dont have proper sunrise/sunset. - Shawn

Re: Review Request: Patch to add a 7-day (up-to) forecast to the NOAA weather Ion

/ --- (Updated 2009-09-13 14:04:24) Review request for Plasma and Shawn Starr. Summary --- The patch uses the REST API provided by NOAA at http://www.weather.gov/forecasts/xml/rest.php to obtain the forecast for the location coordinates

Re: Review Request: Convert weather code to use KUnitConversion (kdebase)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1529/#review2262 --- Ship it! Looks good to me, SHIP IT! - Shawn On 2009-09-07

Re: Review Request: Convert weather code to use KUnitConversion (kdeplasma-addons)

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1530/#review2263 --- Ship it! Looks fine, SHIP IT! - Shawn On 2009-09-07 09:37:35,

Re: Review Request: Condition icons for the NOAA weather ion

: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1418/ --- (Updated 2009-08-27 05:27:46) Review request for Plasma and Shawn Starr. Summary --- Possible patch to allow the NOAA weather

Re: Review Request: Condition icons for the NOAA weather ion

for Plasma and Shawn Starr. Summary --- Possible patch to allow the NOAA weather ion to determine the weather condition icon as defined at http://www.weather.gov/xml/current_obs/weather.php This allows it to work with the weather-wallpaper plasmoid as well as display the condition

Re: Review Request: Moonrise/set for time dataengine

On Monday 24 August 2009 13:12:58 John Layt wrote: On Monday 24 August 2009 08:56:57 Petri Damstén wrote: Moonrise/set patch using new TimeSource in time dataengine. This also changes all sun and moon phase calculation code so same code can be used for all the calculations. This is too

Re: Tokamak 3 for those who won't be there

On Thursday 30 July 2009 19:28:53 Nuno Pinheiro wrote: A Quinta, 30 de Julho de 2009 23:39:54 Aaron J. Seigo você escreveu: hi ... at the last tokamak we had a twitter group; this time it will be an identi.ca group ;) but beyond that, what can we do to make it better/easier/more

Review Request: Add clear() method to Plasma::TabBar widget

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1146/ --- Review request for Plasma, Aaron Seigo and Marco Martin. Summary --- Add a

Re: Review Request: Moonrise/set for time dataengine

On Thursday 16 July 2009 14:04:09 Petri Damstén wrote: On Thursday 16 July 2009 20:25:53 Aaron Seigo wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/680/#review1627

Re: Translations for Weather condition/forecast strings

On Tuesday 07 July 2009 20:59:40 Albert Astals Cid wrote: A Dimecres 08 Juliol 2009 02:19:51, Aaron J. Seigo va escriure: On Tuesday 07 July 2009, Burkhard Lück wrote: Am Mittwoch 08 Juli 2009 00:24:59 schrieb Albert Astals Cid: Any reason you create malformed english messages instead

Re: Review Request: Moonrise/set for time dataengine

On 2009-05-09 17:44:20, Aaron Seigo wrote: putting things in TimeSource was a way to avoid constant allocations. on the one hand, i suppose/hope that people won't be asking for to-the-second accurate sun/moon positions from the time engine, but this still seems like sort of a step

Re: planning for 4.4

On Saturday 27 June 2009 13:40:05 Aaron J. Seigo wrote: On Friday 26 June 2009, Aaron J. Seigo wrote: i'd like to invite everyone to a planning for Plasma in 4.4 irc meeting. it would be great to do it this weekend, as Akademy is next week. i know some of our people are at linuxtag this

Re: Review Request: Fix the 0600GMT=sunrise, 1800GMT=sunset bug in the BBC Weather Ion

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/684/#review1140 --- Ship it! For the BBC ion, It looks fine. But I think maybe we should

Re: Review Request: Use sunrise and sunset times from the Environment Canada weather service to set weather icon to day/night, accordingly.

--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/665/#review1045 --- Ship it! It seems to be working so far, it's already committed. -

Re: Review Request: weather: adds a space between value and unit. (100.20KPa - 100.20 KPa)

On Sunday 29 March 2009 16:30:59 Aaron Seigo wrote: --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/465/#review701 --- diff isn't

Re: Hi David

but I was stuck there for 4.5 hours :-( Well, hang in there, I'm thinking about you, Shawn. - Original Message - From: Shawn Starr shawn . starr @rogers.com To: David dlarochel...@comcast.net Sent: Sunday, March 29, 2009 3:06:47 PM GMT -05:00 US/Canada Eastern Subject: Hi David How

Data source connection/disconnection and force update

I have some issues with Plasma Dataengine API with regard to force updating a source. The use case is this: I load an applet and set the default update interval to every 30 minutes. However, being impatient, I decide I want to get the refreshed information quicker, the applet has a 'Update

popup tooltip API broke?

Aaron, I noticed some changes were made in trunk to the API, this now causes the tooltip to once again appear when the Plasma::PopupApplet is displayed :-) Shawn. ___ Plasma-devel mailing list Plasma-devel@kde.org

Re: Weather applet of playground

On Saturday 20 December 2008 13:55:09 Percy Camilo Triveño Aucahuasi wrote: Hi guys, first of all I want to thank you for plasma, is a wow work ;-) I would like to ask you include the weather applet of playground in your KDE 4.2 series. I have used it enough time to check that is very stable.

Re: Next Tokamak

From: Alexis Ménard [EMAIL PROTECTED] To: plasma-devel@kde.org Sent: Thursday, November 6, 2008 12:36:40 PM Subject: Re: Next Tokamak 2008/11/6 Aaron J. Seigo [EMAIL PROTECTED] On Thursday 06 November 2008, Alexis Ménard wrote: Why? Plasma in KDE 4.3

Re: Next Tokamak

On Thursday 06 November 2008 20:35:30 Alexis Ménard wrote: I know that bug, everybody talk to me about that. But the thing is : i will be *very* interessted by a test case. Actually i tried to reproduce it and i can't. And i don't want to patch Qt blindly without understanding why it happen.

Plasma sucking up huge cpu seems to be a repainting problem

gdb break: Seems something is causing too many repaints when you *just* have a Panel, no widgets, if you remove the panel the CPU drops to nothing.. Program received signal SIGINT, Interrupt. 0x00132416 in __kernel_vsyscall () (gdb) bt #0 0x00132416 in

FolderView and spinners

It would be nice if that could go into libplasma and maybe have it theme-able as part of a Plasma theme? That looks nice! I could certainly use a spinning for the weather plasmoid when it fetches data instead of a blank square to look at :) Shawn.