This is the new home of the egghelp.org community forum.
All data has been migrated (including user logins/passwords) to a new phpBB version.


For more information, see this announcement post. Click the X in the top right-corner of this box to dismiss this message.

DragnLord's weather script

Support & discussion of released scripts, and announcements of new releases.
Post Reply
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

* Bug Fix *

Post by DragnLord »

3.6-a released to fix country code bug
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

teeny tiny fixes/suggestions

Post by rosc2112 »

Just a note to point out a typo or 2, and a suggestion.

In the top comments, you're still missing the [-s] option in the list, should be:

## Usage: .wz [-d] [-o] [-u imc] [-s][-t] [-p] [-short] [help] <city,state|zip> [country]

and further down, where -s is explained, sunset is spelled wrong =)

Also, some responses are sent by NOTICE, perhaps these should use the wz_msg_reply type that is defined? I usually change all the NOTICE to PRIVMSG to make it consistent (although it looks the same in my client, I don't use mirc :)

Once again and as always, thank you very much for keeping this script alive and doing a fine job :)
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

oops, where's canada? =)

Post by rosc2112 »

Ohh, I just noticed, canada is missing from the country list? Do we hate them for hockey, or for Anne Murray? <grin>
c
cache
Master
Posts: 306
Joined: Tue Jan 10, 2006 4:59 am
Location: Mass

Post by cache »

Might be helpful if you go on Dragons server and show them.

First post states 'The script contains the channel and IRC network to report bugs and requests to.'
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

Post by rosc2112 »

Mea culpa
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

Canada

Post by DragnLord »

Script updated, wrong list was included.
No version change.
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

multiple matches prob?

Post by rosc2112 »

Well, I don't know if the bots on #weather @ shell-net does logging for submitting bug reports, and a written record here prolly won't hurt, so I'm posting this here as well =)


With the new additions to the ctlist/countries for australia, if I try to do like, !wz sydney au, it says there's multiple matches. Being that I'm american and we're stupid about geography, I don't know australia's regions and provinces, so perhaps the script should accept the 1st match of 'au' unless a more specific string is given, (eg, AU-SA)?

The change confused a few other people on the channel too, since we're used to using <cityname, au>
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

Post by DragnLord »

just need to add an -exact to the match
for an easy fix, remove the listings with extended code for now
User avatar
Alchera
Revered One
Posts: 3344
Joined: Mon Aug 11, 2003 12:42 pm
Location: Ballarat Victoria, Australia
Contact:

Post by Alchera »

rosc2112 wrote:The change confused a few other people on the channel too, since we're used to using <cityname, au>
rosc2112 wrote:Being that I'm american and we're stupid about geography, I don't know australia's regions and provinces.
NSW, VIC, ACT, SA, NT, QLD, WA & TAS (all States as we're a Federation). :)

Anyway, I am endeavouring to downgrade this script to a version that actually once worked! Last version to function correctly was weather3.5.9-a.
Last edited by Alchera on Fri May 05, 2006 11:01 pm, edited 1 time in total.
Add [SOLVED] to the thread title if your issue has been.
Search | FAQ | RTM
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

Post by DragnLord »

Alchera wrote:
<Almy> Sorry no matches for usa
<Almy> Sorry, us is not supported. Try !country <country> or /msg Almy !country <country> to find your 2 letter country code.
"RTFM" :roll:
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

for those having trouble reading how to use the script

Post by DragnLord »

Code: Select all

Usage: .wz [-d] [-o] [-u imc] [-t] [-p] [-short] [help] <city,state|zip> [country]
      or /msg bot .wz [-d] [-o] [-u imc] [-t] [-p] [-short] <city,state|zip> [country]

     state should be the two letter state code
     country defaults to US and is not needed for US weather
Don't complain that the use of "us" as a country code does not work, it's in the instructions not to use it.
User avatar
Alchera
Revered One
Posts: 3344
Joined: Mon Aug 11, 2003 12:42 pm
Location: Ballarat Victoria, Australia
Contact:

Post by Alchera »

Read my *edited* post! :wink:

I am aware the script does not use "usa/us" even though wunderground.com apparently does.

I was part way through editing/posting when I got sidetracked re-testing last two script releases before settling on using 5.9-a (again); at least we can once more do:

Code: Select all

!wz sydney au
.. etc :)
Last edited by Alchera on Fri May 05, 2006 11:10 pm, edited 1 time in total.
Add [SOLVED] to the thread title if your issue has been.
Search | FAQ | RTM
User avatar
DragnLord
Owner
Posts: 711
Joined: Sat Jan 24, 2004 4:58 pm
Location: C'ville, Virginia, USA

Post by DragnLord »

downgrade if you want to, new script has been posted for a while (before your post about "us" user errors)
3.6-b has no problems with Aussie requests.
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

Post by rosc2112 »

DragnLord wrote:just need to add an -exact to the match
for an easy fix, remove the listings with extended code for now
I'm not sure where the -exact option would go, I only see one lsearch in the script and that already uses the -exact option. But in the meantime I did remove the extended codes.
User avatar
rosc2112
Revered One
Posts: 1454
Joined: Sun Feb 19, 2006 8:36 pm
Location: Northeast Pennsylvania

Post by rosc2112 »

Alchera wrote: NSW, VIC, ACT, SA, NT, QLD, WA & TAS (all States as we're a Federation). :)
Well, that doesn't tell me what cities are in which State.
Alchera wrote: Anyway, I am endeavouring to downgrade this script to a version that actually once worked! Last version to function correctly was weather3.5.9-a.
This version works, just have to remove the extra States/Regions for the moment. Or learn Oz geography I guess ;)
Post Reply