| 00:02.21 | *** join/#wowace pentium166 (~pentium16@iski.es) |
| 00:27.13 | *** join/#wowace Xuerian (~core-q@162.216.97.82) |
| 01:05.54 | Repo | adibuttonauras: 03rainrider 07master * 1.9.6-beta1-6-gf044ec2 (3 files in 3 directories): [+6 commits] (2 truncated) |
| 01:05.54 | Repo | f044ec2: Merge branch 'master' of github.com:Adirelle/AdiButtonAuras |
| 01:05.54 | Repo | 8f7324a: core: fix ElvUI breakage |
| 01:05.54 | Repo | aa1c0db: Merge pull request #190 from myth384/master |
| 01:05.54 | Repo | ece98ef: Added GetSpellCharges to GLOBALS |
| 01:50.12 | *** join/#wowace Ackis (~Ackis@WoWUIDev/WoWAce/ARL/Troll/Ackis) |
| 01:50.12 | *** mode/#wowace [+o Ackis] by ChanServ |
| 01:58.27 | Repo | weakauras-2: 03Stanzilla 07master * 2.1.0.7-67-gc6a4610 Locales/Localization-itIT.lua: [+1 commit] add italian locale file |
| 02:36.02 | Stanzilla | nebula: is that a valid complaint http://vgy.me/VmOd3Y.png ? |
| 02:36.29 | Stanzilla | well ok, probably is |
| 02:36.36 | Stanzilla | that lib has globals all over the place |
| 03:16.16 | nebula | Stanzilla: how would i know? also the param is compressed, the var that triggered it is uncompressed |
| 03:45.36 | znf | Stanzilla, what ide is that? |
| 03:50.01 | *** join/#wowace Megalon (~wig0r@62-116-51-189.adsl.highway.telekom.at) |
| 04:27.40 | *** join/#wowace Elmoe (~Elmoe@c-67-185-160-99.hsd1.wa.comcast.net) |
| 04:34.21 | Repo | archy: 03Torhal 07master * 6.2.0.3-4-g49bb8f0 (5 files in 1 directory): [+4 commits] |
| 04:34.21 | Repo | 49bb8f0: Remove internal record-keeping for zone IDs and names - these are now handled by HereBeDragons. |
| 04:34.21 | Repo | 27c9d5f: Replace the unmaintained Astrolabe with HereBeDragons-1.0 |
| 04:34.21 | Repo | 640576c: Cleanup. |
| 04:34.21 | Repo | 831d436: Remove unnecessarily nested if block. |
| 04:35.42 | Megalon | nests in Torhal's beard |
| 04:36.09 | Torhal | detonates Megalon. |
| 04:36.59 | Megalon | how did you know I'm the sex bomb |
| 04:38.19 | Trela | Torhal: DO YOU WANT DRILLED?! |
| 04:38.23 | Trela | I mean, do you want that drill? |
| 04:39.11 | Torhal | Trela: Aye, I should probably stop my dishwasher falling forward soon. |
| 04:39.32 | Trela | I am here and awake. :D Whenever you want to come by. |
| 04:39.33 | Megalon | oh belieave me Trela, Torhal wants the drill |
| 04:39.34 | Megalon | ( ͡° ÍÊ Í¡Â°) |
| 04:39.43 | Megalon | believe :< |
| 04:39.44 | Torhal | incinerates Megalon. |
| 04:39.50 | Torhal | Trela: Will you be around tomorrow? |
| 04:40.05 | Trela | I should be. Probably be asleep until 12PM though. |
| 04:40.14 | Torhal | Yeah, same here. |
| 04:40.14 | Torhal | Hehe |
| 04:40.19 | Torhal | I'll ping you. |
| 04:40.29 | Trela | Okay! |
| 04:40.42 | Megalon | rude :( |
| 04:46.04 | Trela | pets Megalon, it will be okay. |
| 04:46.54 | Megalon | well, he'll be pinging you |
| 04:58.32 | *** join/#wowace ShadniX (dagger@p5DDFF914.dip0.t-ipconnect.de) |
| 05:20.18 | *** join/#wowace Elmoe (~Elmoe@c-67-185-160-99.hsd1.wa.comcast.net) |
| 05:39.03 | *** join/#wowace Yoshimo (~anything@unaffiliated/yoshimo) |
| 05:56.13 | *** join/#wowace Yoshimo (~anything@unaffiliated/yoshimo) |
| 06:17.14 | Repo | gathermate2_data: 03kagaro 04v27.8 * r437 : GatherMate2_Data: Data update |
| 07:05.38 | nevcairiel | Torhal: you didnt remove the hard-embedded astrolabe :D |
| 07:06.15 | Torhal | Yeah, I meant to. S'ok - I didn't do a release yet. Have more stuff to clean up. |
| 07:06.59 | Torhal | And need to rework the whole arrow when the icon is on edge thing. It doesn't show at all if I don't make the icon float, and if I do it pulses in and out. |
| 07:07.26 | nevcairiel | really? it seems to work fine for tomtom |
| 07:07.57 | Fisker | hey nevcairiel |
| 07:07.58 | Fisker | e |
| 07:07.58 | Fisker | y |
| 07:08.28 | Torhal | It's the way the arrow is handled in Archy. Worked fine with the current code and Astrolabe, not so much now. But I know what to do to fix it. |
| 07:08.44 | nevcairiel | but if you use a arrow-on-border thing like tomtom, then thats what floatOnEdge is for, otherwise it just gets hidden when you go out of range |
| 07:08.56 | Torhal | Huh. I did remove the Libs directory in my local copy, but for some reason it didn't propagate to Git. |
| 07:09.09 | nevcairiel | need to do a git rm |
| 07:09.18 | nevcairiel | normal git adds probably wont track that |
| 07:09.24 | Torhal | Yeah, just tried that and got fatal: pathspec 'Libs' did not match any files |
| 07:09.31 | nevcairiel | weird |
| 07:09.53 | Fisker | fucking bullshit mouse control in hearthstone Torhal |
| 07:10.02 | Torhal | Stab it Fisker |
| 07:21.37 | Torhal | nevcairiel: Just re-cloned into a fresh directory and there's no Libs |
| 07:21.42 | nevcairiel | odd |
| 07:21.49 | nevcairiel | a pull should've deleted it, shouldnt it |
| 07:21.55 | Torhal | Should have, yeah. |
| 07:22.23 | nevcairiel | maybe its weird because Libs is in .gitignore |
| 07:22.37 | Torhal | Hmm, maybe. |
| 07:22.51 | nevcairiel | guess it may not delete things in ignore, so local stuff doesnt get lost |
| 07:23.00 | nevcairiel | oh well its gone now |
| 07:24.16 | nevcairiel | no more astrolabe in my interface folder, yay :d |
| 07:24.25 | Torhal | Hehe |
| 07:33.29 | Torhal | nevcairiel: Tag HandyNotes? CC is complaining that it requires Astrolabe. |
| 07:33.46 | nevcairiel | I'll probably do it a bit later |
| 07:33.52 | Torhal | Alright |
| 07:34.06 | nevcairiel | didnt release tag any of the things yet |
| 07:34.55 | nevcairiel | because when I do, i would technically also need to tag all of Ethan's HN plugins, or users might start seeing errors, and I wanted to ask him first |
| 08:50.04 | *** join/#wowace Elmoe (~Elmoe@c-67-185-160-99.hsd1.wa.comcast.net) |
| 09:17.07 | Repo | archy: 03Torhal 07master * 6.2.0.4 (2 files in 1 directory): [+2 commits] |
| 09:17.08 | Repo | d4f2402: Cleanups - use HereBeDragons instead of map manipulation hacks. |
| 09:17.08 | Repo | be8fd09: Remove code for altering survey node distance coloration on enable - this was resetting the color to cyan when the player moved, and is properly handled elsewhere. |
| 09:17.16 | Repo | archy: 03Torhal 046.2.0.4 * 6e7e298 /: [new tag] |
| 09:19.32 | Repo | herebedragons: 03Nevcairiel 041.00-release * a68579b /: [new tag] Tag as 1.00-release |
| 09:20.20 | nevcairiel | i wonder if i should do something about the garrison nodes in gathermate before i tag |
| 09:20.49 | nevcairiel | maybe i should just delete them all |
| 09:24.47 | Torhal | Herbs aren't a big deal since they're concentrated, but the ores are all over the place. |
| 09:25.04 | nevcairiel | yeah in a perfect world it wouldnt even have collected them |
| 09:25.13 | nevcairiel | but astrolabe gets extremely confused in that cave |
| 09:25.19 | nevcairiel | so it flips the map around all the time |
| 09:25.24 | nevcairiel | .. which in turn confuses gathermate |
| 09:25.46 | Torhal | Urk. |
| 09:28.34 | nevcairiel | interestingly mine show on the shadowmoon map instead of the garrison map because of that bug |
| 09:29.16 | Kalroth | http://mynoise.net/NoiseMachines/catPurrNoiseGenerator.php :O |
| 09:39.56 | Repo | herebedragons: 03Nevcairiel 07master * 1.00-release-1-g2a24ffd HereBeDragons-1.0.lua: [+1 commit] Additionally return "isMicroDungeon" from GetPlayerZone* |
| 09:45.52 | Repo | gathermate2: 03Nevcairiel 07master * 1.34-5-g6da2452 (2 files in 1 directory): [+2 commits] |
| 09:45.52 | Repo | 6da2452: Purge Garrison Herb/Mining data, and don't collect any new data in the garrison |
| 09:45.52 | Repo | 24d9105: Use HereBeDragons to determine the current position when gathering, instead of the map |
| 09:47.09 | Repo | herebedragons: 03Nevcairiel 041.01-release * 00722f6 /: [new tag] Tag as 1.01-release |
| 09:48.01 | Repo | gathermate2: 03Nevcairiel 041.35 * cf8e438 /: [new tag] Tag as 1.35 |
| 09:48.47 | Repo | routes: 03Nevcairiel 04v1.5.2 * 5cf89f5 /: [new tag] Tag as v1.5.2 |
| 09:51.42 | Repo | herebedragons: 03Nevcairiel 041.02-release * 6a8cfaf /: [new tag] Tag as 1.02-release |
| 09:51.42 | Repo | herebedragons: 03Nevcairiel 07master * 1.02-release HereBeDragons-1.0.lua: [+1 commit] Remove debug print which managed to sneak in accidentally |
| 09:53.05 | Repo | gathermate2: 03Nevcairiel 041.35.1 * ebc493e /: [new tag] Tag as 1.35.1 |
| 09:54.58 | Repo | routes: 03Nevcairiel 04v1.5.2a * 968f165 /: [new tag] Tag as v1.5.2a |
| 10:04.19 | *** join/#wowace MysticalOS (~Myst@WoWUIDev/WoWAce/DBM/MysticalOS) |
| 10:59.52 | Repo | gathermate2: 03Nevcairiel 07master * 1.35.1-1-g22a80b3 Collector.lua: [+1 commit] nil check coordinates |
| 11:02.53 | Repo | libsink-2-0: 03funkydude 04r109-release * r109 : Tagging as r109-release |
| 11:06.13 | Repo | handynotes: 03nevcairiel 04v1.4.0 * r254 : Tag as v1.4.0 |
| 11:09.35 | Fisker | hey Torhal |
| 11:09.39 | Fisker | why are you not asleep? |
| 11:42.51 | Stanzilla | znf: atom |
| 11:44.18 | Fisker | Hey Stanzilla |
| 11:45.04 | Stanzilla | hi |
| 11:45.55 | Stanzilla | need to go to family party stuff today ugh |
| 11:46.03 | nevcairiel | don't do it |
| 11:46.04 | Fisker | :( |
| 11:46.08 | Fisker | I want pizza |
| 11:46.14 | nevcairiel | they'll make you socialize with people |
| 11:46.28 | Stanzilla | have to if I want to avoid The Wrath of the Grandma |
| 11:47.12 | Fisker | Grandma khan? |
| 11:47.20 | Stanzilla | nevcairiel: can you do me a favor and check http://www.wowace.com/addons/libcompress/repositories/mainline/ any maybe fix the globals in there, some of that looks insane |
| 11:53.57 | nevcairiel | that wasnt too hard |
| 11:53.59 | Repo | libcompress: 03nevcairiel * r62 LibCompress.lua: Fix leaked globals |
| 11:54.21 | Stanzilla | thanks |
| 11:54.25 | Stanzilla | g2g no, cya laters |
| 11:54.30 | Stanzilla | *now |
| 11:56.06 | Fisker | Bye |
| 11:56.11 | Fisker | Love you |
| 11:57.06 | nevcairiel | for some reason libcompress didnt start packaging however |
| 11:58.01 | Repo | libcompress: 03nevcairiel * r63 : Remove svn:externals |
| 11:58.58 | Repo | libcompress: 03nevcairiel * r64 LibCompress.toc: Update TOC for 6.2 |
| 11:59.48 | *** join/#wowace EthanCentaurai (~EthanCent@host-92-21-132-64.as13285.net) |
| 12:10.55 | Repo | libcompress: 03nevcairiel * r65 LibCompress.lua: Make use of upvalues more consistent |
| 12:10.55 | Repo | libcompress: 03nevcairiel 04r65-release * r66 : Tagging as r65-release |
| 12:12.23 | Repo | handynotes_azerothstoptunes: 03EthanCentaurai 04v1.1 * 9682ec5 /: [new tag] Tagging as v1.1. |
| 12:21.24 | Repo | handynotes_lunarfestival: 03EthanCentaurai 07master * v1.2.5-4-g62cad45 HandyNotes_LunarFestival.toc: [+1 commit] Bump Interface number to 60200. |
| 12:22.41 | Repo | handynotes_lunarfestival: 03EthanCentaurai 04v1.3 * 43c092d /: [new tag] Tagging as v1.3. |
| 12:23.34 | *** join/#wowace Ressy (Ressy@WoWUIDev/WoWAce/ARL/Ressy) |
| 12:26.56 | Repo | handynotes_summerfestival: 03EthanCentaurai 04v1.2 * 38dccba /: [new tag] Tagging as v1.2. |
| 12:27.22 | EthanCentaurai | hai Ressy |
| 12:28.40 | Ressy | hey |
| 12:44.28 | Repo | handynotes_hallowsend: 03EthanCentaurai 07master * v1.1.2-4-gfe961cd (3 files in 1 directory): [+3 commits] |
| 12:44.28 | Repo | fe961cd: Update README. |
| 12:44.28 | Repo | 88c7891: Bump Interface number to 60200. |
| 12:44.28 | Repo | 6387108: Dropped Cartographer support. |
| 12:47.18 | znf | it looks like nev is doing some summer-end cleaning |
| 12:49.52 | EthanCentaurai | just need TomTom cleaned and I can get rid of this dumb library |
| 12:50.22 | nevcairiel | i did update tomtom, but jnwhiteh is on vacation, or so Stanzilla said |
| 12:59.55 | Fisker | hey Ressy |
| 13:06.44 | Repo | herebedragons: 03Nevcairiel 07master * 1.02-release-1-gf3d30bf HereBeDragons-Pins-1.0.lua: [+1 commit] Fix translation of icons with level = 0 to other zones |
| 13:34.55 | Repo | herebedragons: 03Nevcairiel 041.03-release * c91aa33 /: [new tag] Tag as 1.03-release |
| 13:35.37 | Repo | handynotes: 03nevcairiel 04v1.4.1 * r255 : Tag as v1.4.1 |
| 13:36.52 | nevcairiel | there now continent icons also work again in handynotes |
| 13:45.51 | nevcairiel | anyone with a horde char can test something for me? |
| 13:50.05 | nevcairiel | sbu: go be useful :D |
| 14:00.38 | nevcairiel | damnit preset chars on the ptr dont have a garrison |
| 14:12.21 | *** join/#wowace Megalon (~wig0r@62-116-51-189.adsl.highway.telekom.at) |
| 14:14.07 | *** join/#wowace Gethe (~quassel@cpe-24-55-4-32.austin.res.rr.com) |
| 14:29.22 | nevcairiel | .. man carbonite is evil, it hijacks a bunch of map API and then users wonder why shit breaks |
| 14:30.19 | Funkeh` | astrolabe did the same at one point |
| 14:30.56 | Funkeh` | I remember having issues checking zone back in cata when loading dragon soul content |
| 14:32.10 | Funkeh` | it's a shame you can't grab untainted copies of functions like blizz can |
| 14:32.26 | nevcairiel | i wish there was a inverse dependency flag, ie "doesnt work with this crap installed" |
| 14:32.38 | Funkeh` | heh |
| 14:32.42 | nevcairiel | is there a "load before" flag? :d |
| 14:33.22 | Funkeh` | !!HereBeDragons |
| 14:33.26 | Funkeh` | :p |
| 14:33.33 | nevcairiel | its an embedded library, so meh |
| 14:33.50 | Funkeh` | a library not appended with lib!? |
| 14:34.32 | nevcairiel | I'll just call it unsupported |
| 14:34.40 | nevcairiel | not dealing with addons that hijack blizzard api and break it |
| 14:35.04 | Funkeh` | just local the function and hope for the best? |
| 14:35.09 | Funkeh` | that's what I did :x |
| 14:35.18 | nevcairiel | C is unfortunately pretty high in the load order =p |
| 14:35.55 | Funkeh` | which function is it btw? |
| 14:36.07 | nevcairiel | both SetMapToCurrentZone and SetMapByID |
| 14:36.10 | Repo | herebedragons: 03Nevcairiel 07master * 1.03-release-1-g8b04990 HereBeDragons-1.0.lua: [+1 commit] Check that garrison data exists before trying to access it |
| 14:36.18 | nevcairiel | the fun fact is that me using these functions actually breaks carbonite |
| 14:36.31 | nevcairiel | well, and HBD is unable to gather any zone info |
| 14:36.50 | nevcairiel | because i check the return value of SetMapByID |
| 14:36.57 | nevcairiel | which is always "nil" when carbonite hooks it |
| 14:37.33 | nevcairiel | i wonder if i can somewhere get the original function |
| 14:40.24 | Fisker | hey nevcairiel |
| 14:47.08 | *** join/#wowace EthanCentaurai (~EthanCent@host-92-21-132-64.as13285.net) |
| 14:47.30 | nevcairiel | ..it wouldnt be too bad if he at least returned the same return values as the original function |
| 14:47.55 | Funkeh` | try contact him? |
| 14:57.40 | nevcairiel | i posted on t heir forum, but judging from the users wondering where all the feedback is, i dont have high hopes =p |
| 15:00.45 | *** join/#wowace Yoshimo (~anything@unaffiliated/yoshimo) |
| 15:01.21 | nevcairiel | interestingly this hook isnt even that old, it was added in december |
| 15:02.06 | quiescens | gasp |
| 15:09.37 | Repo | skada-graphs: 03luckycire 041.1-4 * r35 : Tagging as 1.1-4 |
| 16:50.31 | *** join/#wowace Yoshimo (~anything@unaffiliated/yoshimo) |
| 16:53.11 | *** join/#wowace EthanCentaurai (~EthanCent@host-92-21-132-64.as13285.net) |
| 16:54.05 | Stanzilla | nevcairiel: I'm here now, do you still need horde testing? |
| 16:54.16 | nevcairiel | nah |
| 16:54.21 | Stanzilla | thanks for libcompress btw, I havent restarted my pc so my svn is still broken |
| 16:57.37 | Fisker | hey Stanzilla |
| 16:57.39 | Fisker | How was it? |
| 16:57.43 | Stanzilla | hey Fisker |
| 16:57.47 | Stanzilla | it was ok |
| 16:57.47 | Fisker | e |
| 16:57.47 | Fisker | y |
| 16:57.49 | Stanzilla | lots of food |
| 16:57.50 | Fisker | damnit |
| 16:57.53 | Fisker | boo |
| 16:58.51 | Stanzilla | nice bbq |
| 17:00.58 | Fisker | nice |
| 17:01.52 | Fisker | I hope android pay is available worldwide |
| 17:02.02 | Fisker | So I can get more contactless payment |
| 17:02.16 | Repo | weakauras-2: 03Stanzilla 07master * 2.1.0.7-69-g178abe0 (2 files in 1 directory): [+2 commits] |
| 17:02.17 | Repo | 178abe0: load LibCompress by the lua, not the xml |
| 17:02.17 | Repo | d05fdcf: alter toc info |
| 17:05.30 | Fisker | btw |
| 17:05.40 | Fisker | was the fix for th2 rolled out on dev yet Stanzilla ? |
| 17:08.56 | Repo | bagslots: 03phyber 07master * 60100-1-1-g7ea09b0 BagSlots.toc: [+1 commit] Update TOC to 60200 |
| 17:09.05 | Repo | bagslots: 03phyber 0460200-1 * bddbed5 /: [new tag] Tagged as 60200-1 |
| 17:09.10 | Repo | broker_bagfu: 03phyber 07master * 60100-1-1-gfe3c138 Broker_BagFu.toc: [+1 commit] Update TOC to 60200 |
| 17:09.16 | Repo | broker_bagfu: 03phyber 0460200-1 * 4da60ef /: [new tag] Tagged as 60200-1 |
| 17:09.21 | Repo | broker_digicam: 03phyber 07master * 60100-1-1-g77bc3c0 Broker_Digicam.toc: [+1 commit] Update TOC to 60200 |
| 17:09.27 | Repo | broker_digicam: 03phyber 0460200-1 * 108699b /: [new tag] Tagged as 60200-1 |
| 17:09.37 | Repo | broker_moneyfu: 03phyber 07master * 60100-1-1-gf9b68ef Broker_MoneyFu.toc: [+1 commit] Update TOC to 60200 |
| 17:09.43 | Repo | broker_moneyfu: 03phyber 0460200-1 * f228a06 /: [new tag] Tagged as 60200-1 |
| 17:09.54 | Repo | broker_restfu: 03phyber 07master * 60100-1-1-ga62fd8e Broker_RestFu.toc: [+1 commit] Update TOC to 60200 |
| 17:09.59 | Repo | broker_restfu: 03phyber 0460200-1 * 9686c12 /: [new tag] Tagged as 60200-1 |
| 17:10.05 | Repo | cosplay: 03phyber 07master * 60100-1-1-g525b836 Cosplay.toc: [+1 commit] Update TOC to 60200 |
| 17:10.10 | Repo | cosplay: 03phyber 0460200-1 * 3be9170 /: [new tag] Tagged as 60200-1 |
| 17:10.21 | Repo | cromulent: 03phyber 07master * 60100-1-1-g9fca116 Cromulent.toc: [+1 commit] Update TOC to 60200 |
| 17:10.26 | Repo | cromulent: 03phyber 0460200-1 * 5ebe6d9 /: [new tag] Tagged as 60200-1 |
| 17:10.32 | Repo | fizzle: 03phyber 07master * 60100-1-1-g6a002cb Fizzle.toc: [+1 commit] Update TOC to 60200 |
| 17:10.37 | Repo | fizzle: 03phyber 0460200-1 * 798fca8 /: [new tag] Tagged as 60200-1 |
| 17:10.53 | Repo | liquorice: 03phyber 07master * 60100-1-1-g2e7e692 Liquorice.toc: [+1 commit] Update TOC to 60200 |
| 17:10.58 | Repo | liquorice: 03phyber 0460200-1 * f4b4757 /: [new tag] Tagged as 60200-1 |
| 17:11.09 | Repo | manaperc: 03phyber 07master * 60100-1-1-g8259992 ManaPerc.toc: [+1 commit] Update TOC to 60200 |
| 17:11.14 | Repo | manaperc: 03phyber 0460200-1 * e8236e9 /: [new tag] Tagged as 60200-1 |
| 17:11.22 | Stanzilla | Fisker: nope |
| 17:11.25 | Repo | xpbarnone: 03phyber 07master * 60100-1-1-gf11752b XPBarNone.toc: [+1 commit] Update TOC to 60200 |
| 17:11.30 | Repo | xpbarnone: 03phyber 0460200-1 * 6f2a5e5 /: [new tag] Tagged as 60200-1 |
| 17:11.49 | Fisker | I want it |
| 17:11.58 | Fisker | I want it like I want these commits from phyber gone |
| 17:12.06 | Fisker | (no offense phyber) |
| 17:12.10 | Stanzilla | looks like the packager is broken |
| 17:12.15 | Stanzilla | alert a Torhal |
| 17:12.20 | Fisker | Never alert a Torhal |
| 17:12.22 | Fisker | maybe hey a Torhal |
| 17:12.26 | Fisker | ease him down a bit |
| 17:12.33 | nevcairiel | seems to be working Stanzilla |
| 17:12.34 | Fisker | shank him in while he's not looking |
| 17:12.46 | phyber | Fisker: heh, sorry. I resubbed and thought I'd run my script to update crap ;) |
| 17:13.20 | Ressy | Packager seems ok... Its working through digicam now. |
| 17:13.28 | Fisker | Yeah I'm just joking around phyber |
| 17:13.32 | Stanzilla | it skipped WA |
| 17:13.36 | Fisker | I'm totally like the joker |
| 17:13.39 | Stanzilla | got scammed by pacakger :( |
| 17:19.55 | EthanCentaurai | I broke it with my HandyNotes stuff :V |
| 17:38.29 | *** join/#wowace Higdur (~nike@81-230-241-38-no243.tbcn.telia.com) |
| 17:47.58 | Stanzilla | THANKS EthanCentaurai |
| 17:48.11 | EthanCentaurai | YOU'RE WELCOME Stanzilla |
| 17:48.20 | Stanzilla | steak? |
| 17:49.33 | quiescens | o.o |
| 17:49.43 | Megalon | da platzt mir der sack! |
| 17:49.45 | quiescens | gives EthanCentaurai a cookie |
| 17:49.47 | Stanzilla | wow quiescens you had one job |
| 17:49.53 | Stanzilla | the correct response would have been "moo" |
| 17:49.56 | Stanzilla | you fucked up |
| 17:50.00 | quiescens | ): |
| 17:50.01 | quiescens | so mean |
| 17:50.10 | Stanzilla | ./me dies |
| 17:50.23 | Megalon | /me dies ? |
| 17:50.26 | EthanCentaurai | yay cookie! |
| 18:20.12 | Torhal | nevcairiel: http://www.wowace.com/addons/archy/tickets/574-here-be-dragons-error/ |
| 18:21.07 | Torhal | Is that me trying to remove something that wasn't set, or the lib going spastic? |
| 18:22.17 | Stanzilla | Torhal: can you requeue wa2 for me, pls |
| 18:22.22 | nevcairiel | Torhal: hm yeah |
| 18:22.25 | nevcairiel | i should nil check that |
| 18:23.48 | Torhal | Stanzilla: Queued. |
| 18:23.57 | Stanzilla | Torhal: thanks <3 |
| 18:25.19 | nevcairiel | Torhal: so yes, it was trying to remove something that wasnt actually added |
| 18:25.27 | nevcairiel | but that shouldnt error, i guess |
| 18:25.54 | Repo | herebedragons: 03Nevcairiel 07master * 1.03-release-2-g9592f11 HereBeDragons-Pins-1.0.lua: [+1 commit] Don't error when removing an icon that wasn't added |
| 18:25.57 | Torhal | Sounds like I was making assumptions based on Astrolabe behavior. |
| 18:29.38 | Torhal | Stanzilla: You have a new alpha. |
| 18:29.46 | Stanzilla | yay |
| 18:30.03 | Stanzilla | new globe round |
| 18:31.55 | Stanzilla | https://www.townlong-yak.com/globe/#h:4ccf04e305666c0336387b1bf345f38e |
| 18:32.02 | Stanzilla | pretty clean! |
| 18:50.31 | Repo | libdbicon-1-0: 03funkydude * r47 LibDBIcon-1.0/LibDBIcon-1.0.lua: tweaks |
| 18:52.36 | Repo | herebedragons: 03Nevcairiel 07master * 1.03-release-4-g223b8c9 HereBeDragons-Pins-1.0.lua: [+1 commit] Even more nil checks |
| 18:53.35 | Stanzilla | I really need to remove this stupid hack |
| 18:53.45 | Stanzilla | but I have no idea what this is |
| 18:53.47 | Stanzilla | test = MAX_NUM_TALENTS and "select(5, GetTalentInfo(%d)) == true" or "select(4, GetMoPTalentInfo(%d)) == true" |
| 18:53.58 | Stanzilla | when function GetMoPTalentInfo(t) return GetTalentInfo(ceil(t/3), (t-1)%3 +1, GetActiveSpecGroup()) end |
| 18:54.49 | nevcairiel | MAX_NUM_TALENTS is nil in WoD/MoP, so just inline the mop version instead of the check |
| 18:56.42 | Stanzilla | like that? |
| 18:56.43 | Stanzilla | test = "select(5, GetTalentInfo(%d)) == true" or "GetTalentInfo(ceil(t/3), (t-1)%3 +1, GetActiveSpecGroup())" |
| 18:57.48 | nevcairiel | no |
| 18:57.49 | nevcairiel | :D |
| 18:57.54 | Stanzilla | :( |
| 18:59.33 | Stanzilla | well with d instead of t |
| 18:59.57 | Repo | big-wigs_common-auras: 03funkydude * r280 (2 files in 1 directory): tweaks |
| 19:00.13 | nevcairiel | no |
| 19:00.29 | nevcairiel | remove the first two parts, up to and including to the "or" |
| 19:00.40 | nevcairiel | then replace getmoptalentinfo ith the body of the function |
| 19:02.30 | Stanzilla | est = "select(4, GetTalentInfo(ceil(%d/3), (%d-1)%3 +1, GetActiveSpecGroup()) == true" |
| 19:02.33 | Stanzilla | ?`:D |
| 19:08.57 | Stanzilla | well it requires a first arg |
| 19:09.06 | Stanzilla | but if MAX_NUM_TALENTS is always nil, i can just "nil and" |
| 19:09.50 | Stanzilla | but that doesnt make sense |
| 19:11.27 | nevcairiel | you are weird |
| 19:11.33 | Stanzilla | true story |
| 19:11.39 | nevcairiel | that looks fine though |
| 19:11.42 | nevcairiel | well, lacking one ) |
| 19:12.41 | nevcairiel | or just keep the getmoptalent thingy and only remove the conditional |
| 19:13.34 | Stanzilla | I don't really want to keep it backwards compatible anyway |
| 19:13.38 | Stanzilla | so I thought I can just remove it |
| 19:13.59 | Stanzilla | I think the function name might be misleading |
| 19:14.11 | Stanzilla | and I should actually keep the first part of the original |
| 19:14.54 | Stanzilla | as in test = MAX_NUM_TALENTS and "select(5, GetTalentInfo(%d)) == true" |
| 19:15.47 | Stanzilla | nooope |
| 19:15.50 | Stanzilla | doesn't work either |
| 19:16.31 | Repo | bug-grabber: 03funkydude * r213 (2 files in 1 directory): tweaks |
| 19:17.14 | nevcairiel | i'll give you a hint: its very old code |
| 19:17.27 | nevcairiel | so view "mop" in reference to that |
| 19:18.20 | Repo | libdbicon-1-0: 03funkydude 04r48-release * r48 : Tagging as r48-release |
| 19:18.21 | Stanzilla | yeah from I remember GetMopTalentInfo was added to keep it working on 5.x |
| 19:18.51 | Repo | bug-grabber: 03funkydude 04r214-release * r214 : Tagging as r214-release |
| 19:18.52 | nevcairiel | yes, but during cata |
| 19:18.59 | nevcairiel | so getmoptalentinfo is the "new" way |
| 19:19.10 | Stanzilla | but the note said 6.0 compat |
| 19:19.13 | Stanzilla | that confused me |
| 19:19.28 | Repo | big-wigs_common-auras: 03funkydude 04r281-release * r281 : Tagging as r281-release |
| 19:19.29 | Stanzilla | "patch 6.0 compatibility quick fix" |
| 19:19.37 | nevcairiel | MAX_NUM_TALENTS is nil, that already gives you the answer which of the two is being used |
| 19:19.59 | Stanzilla | well but |
| 19:20.12 | Stanzilla | nil and "select(4, GetTalentInfo(ceil(%d/3), (%d-1)%3 +1, GetActiveSpecGroup())) == true", |
| 19:20.15 | Stanzilla | didn't work |
| 19:21.02 | Stanzilla | and when I remove the nil an it complains about a format error |
| 19:24.23 | Stanzilla | oh fuck me, the same hack is in another file, too |
| 19:25.53 | Stanzilla | if not MAX_NUM_TALENTS then GetTalentInfo = function(t) return select(2, _G.GetTalentInfo(ceil(t/3), (t-1)%3 +1, GetActiveSpecGroup())) end end |
| 19:25.55 | Stanzilla | yeah ok |
| 19:28.24 | Stanzilla | so i'd actually have to rewrite the talent checking |
| 19:30.38 | Stanzilla | so talentName, talentIcon = GetTalentInfo(talentId) works |
| 19:31.08 | EthanCentaurai | stalentzilla |
| 19:31.21 | Stanzilla | talentName, talentIcon = select(2, GetTalentInfo(ceil(talentId/3), (talentId-1)%3 +1, GetActiveSpecGroup())) |
| 19:31.28 | Stanzilla | this could actually work |
| 19:32.51 | Stanzilla | ok doesnt |
| 19:33.02 | Stanzilla | man this is horrible |
| 19:34.42 | Torhal | Stanzilla: Why are you using select() all over the place? |
| 19:34.53 | Stanzilla | because it was there |
| 19:34.57 | Stanzilla | I don't even know if I need it |
| 19:35.00 | Torhal | Doesn't mean it should stay. |
| 19:35.04 | Stanzilla | yeah |
| 19:35.28 | Torhal | It's like this: select(2, whatever) grabs the 2nd index. Instead, you could: |
| 19:35.35 | Torhal | local _, whatIWant = whatever |
| 19:35.53 | Torhal | You don't incur the overhead of a function call that way, and it's just as readable. |
| 19:35.56 | Torhal | Arguably moreso. |
| 19:36.18 | Stanzilla | I guess I should just rewrite that function against the up to date version of GetTalentInfo |
| 19:36.24 | Stanzilla | which I hope is this http://wowprogramming.com/docs/api/GetTalentInfo |
| 19:37.09 | Torhal | Might be this, instead: http://wow.gamepedia.com/API_GetTalentInfo |
| 19:37.23 | Torhal | Check with /dump |
| 19:38.40 | Stanzilla | yeah wowpedia was updated to wod |
| 19:38.42 | Stanzilla | so ok, use that then |
| 19:41.05 | Stanzilla | so you don't even know the id before calling it now |
| 19:41.08 | Stanzilla | ughghghg |
| 19:44.02 | Torhal | nevcairiel: I found a fishing node that I hadn't fished yet, and Gathermate didn't add it when I did. |
| 19:44.41 | nevcairiel | fishing is weird, hmkay |
| 19:45.35 | Torhal | Yeah, reloaded and still no record. |
| 19:46.52 | Stanzilla | I' |
| 19:47.33 | Megalon | you? |
| 19:47.46 | Stanzilla | http://vgy.me/g05Rvu.png |
| 19:47.58 | Stanzilla | I need to change the tier and column stuff in the while loop too |
| 19:48.02 | Stanzilla | but not sure how |
| 19:48.10 | Megalon | what are you trying to do |
| 19:48.22 | Stanzilla | build a cache, look at the screenshot |
| 19:48.26 | Megalon | oh cache talents |
| 19:48.44 | Stanzilla | do I have to like, do a loop for every row? |
| 19:49.44 | Megalon | no |
| 19:49.54 | Megalon | you need to do the column = xx in the loop to |
| 19:50.01 | Megalon | and tier = yy |
| 19:50.01 | Stanzilla | I could just make my own gettalentinfo |
| 19:50.06 | Stanzilla | like it used to be |
| 19:50.12 | Stanzilla | but not global and retardedly named |
| 19:50.17 | Megalon | else it will always use the one you define above |
| 19:50.23 | Stanzilla | yeah I know |
| 19:50.29 | Stanzilla | that's why I said I know I have to change it in the loop |
| 19:50.31 | Stanzilla | but not how |
| 19:50.37 | Stanzilla | I could just +1 it always |
| 19:50.52 | Repo | libchatanims: 03funkydude * r21 LibChatAnims/LibChatAnims.lua: remove an unused variable |
| 19:50.52 | Megalon | you alread loop from 1..21 |
| 19:51.02 | Stanzilla | yes |
| 19:51.09 | Megalon | you just copy the things into the loop to |
| 19:52.06 | Megalon | http://i.imgur.com/PtNkAWX.png |
| 19:52.11 | Megalon | there |
| 19:52.15 | Megalon | my paint skills are amazing |
| 19:52.17 | Stanzilla | yeah that's what I just did |
| 19:52.25 | Megalon | and it's not working? |
| 19:52.26 | Repo | libchatanims: 03funkydude 04r22-release * r22 : Tagging as r22-release |
| 19:52.38 | Stanzilla | nop |
| 19:52.45 | Stanzilla | wait |
| 19:53.27 | Stanzilla | nop |
| 19:53.33 | Stanzilla | always only shows 1 talent in the dropdown |
| 19:53.36 | Stanzilla | the first ever |
| 19:53.47 | Megalon | is the cache empty? |
| 19:53.58 | Stanzilla | probably not |
| 19:54.04 | Megalon | or does the dropdown code just display one element? |
| 19:54.08 | Stanzilla | no |
| 19:54.38 | Megalon | dump the cache, then you'll at least know if that part worked :D |
| 19:55.50 | Stanzilla | yeah broken with clean cache too |
| 19:57.49 | Stanzilla | I kinda know why |
| 19:58.19 | Stanzilla | ok no, I dont |
| 19:58.42 | Megalon | me_irl |
| 19:58.59 | Stanzilla | http://vgy.me/ULnJhw.png |
| 19:59.10 | Stanzilla | sanity check pls |
| 20:00.27 | Megalon | you have a loop that uses the cachce inside the loop you build the cache? |
| 20:01.07 | Stanzilla | that was already there before |
| 20:02.43 | Stanzilla | I only have to fix the talent part |
| 20:03.34 | Megalon | hm |
| 20:04.12 | Megalon | the line with GetTalentInfo |
| 20:04.18 | Megalon | you are setting talentId |
| 20:04.56 | Megalon | perhaps better do two loops, 1..7, 1..3 |
| 20:05.16 | Megalon | or are you guaranteed that the talentId is not changed around at that point? |
| 20:05.25 | Stanzilla | that is the question |
| 20:05.29 | nebula | why is that a while loop D; |
| 20:05.45 | Stanzilla | because someone else thought it might be a good idea |
| 20:05.45 | Megalon | my guess would be no one knows |
| 20:05.54 | Megalon | and by now they are too afraid to change |
| 20:05.58 | Stanzilla | suggestions? |
| 20:06.11 | Megalon | for talId=1,21 do |
| 20:06.37 | Stanzilla | that doesnt fix the original problem though |
| 20:07.11 | Megalon | you basically use talentId = GetTalentInfo(talentID*x, talentID*y) |
| 20:07.19 | Megalon | are you sure that doesn't change talentId? :P |
| 20:07.22 | nebula | instead of that silly compat shit, why don't you use for tier = 1, MAX_TALENT_TIERS do for column = 1, NUM_TALENT_COLUMNS do |
| 20:07.53 | Stanzilla | Megalon: no, changed it, tooltip works now |
| 20:07.59 | Stanzilla | nebula: because I didN#t know about it |
| 20:10.33 | Megalon | nebula: i refuse to use them if they don't start with LE_* |
| 20:10.36 | Megalon | ;/ |
| 20:10.49 | Stanzilla | nebula: http://vgy.me/YOV2TI.png |
| 20:10.51 | Stanzilla | that better now? |
| 20:11.09 | Megalon | now you need talentId instead of _ |
| 20:11.13 | nebula | Megalon: ... but they're not enums |
| 20:11.29 | Stanzilla | :E |
| 20:11.34 | Megalon | delete the tier =, column = xy stuff |
| 20:11.41 | Megalon | because you are setting them in the for loop already |
| 20:11.47 | Stanzilla | yes |
| 20:11.50 | Stanzilla | already had it commented out |
| 20:12.16 | Megalon | btw i hate you for using screenshots :P |
| 20:12.29 | Stanzilla | no problem |
| 20:13.02 | Stanzilla | problem is |
| 20:13.09 | Stanzilla | the settings are now keyed by id not name |
| 20:13.12 | Stanzilla | right? |
| 20:13.18 | Stanzilla | because all settings just lost their value for me |
| 20:13.23 | nebula | it's a cache |
| 20:13.30 | nebula | kill and it start over with something not retarded |
| 20:13.46 | Stanzilla | still have to migrate user settings? |
| 20:13.57 | nebula | ... it's a cache? |
| 20:14.03 | nebula | it shouldn't be used by settings |
| 20:14.19 | nebula | just update the code that references it |
| 20:14.35 | Megalon | Stanzilla i know why it fucked up haha |
| 20:14.38 | Stanzilla | T_T |
| 20:14.45 | Stanzilla | Megalon: pls tell me it's something retarded |
| 20:14.46 | Stanzilla | and easy |
| 20:15.01 | Megalon | the talentId returned by GTI |
| 20:15.09 | Megalon | i think those are spellid's |
| 20:15.18 | nebula | they're unique to talents, but yes |
| 20:15.19 | Stanzilla | ... |
| 20:15.25 | Megalon | at least my druid talents rage from 18569 to 21194 |
| 20:15.29 | Megalon | not 1..21 :P |
| 20:15.35 | Stanzilla | now what |
| 20:15.36 | Megalon | so yea that line fucked it up |
| 20:15.45 | Megalon | do you need it to be 1..21? :) |
| 20:16.04 | Stanzilla | if I can avoid migrating user settings? Yes I think so |
| 20:16.36 | Stanzilla | can't i just set talent id |
| 20:16.39 | Megalon | then don't use the talentId from GTI but instead calculate it from tier/colum |
| 20:16.41 | Stanzilla | after the gettalentinfo check |
| 20:17.27 | Megalon | talendId = (tier-1)*7+colum |
| 20:17.38 | nebula | *3 |
| 20:17.45 | Megalon | erg *3 |
| 20:17.48 | Megalon | yea |
| 20:17.59 | Megalon | my inner retard got the upper hand |
| 20:19.02 | Stanzilla | probably have to remove the talentId=talentId+1, too? |
| 20:19.18 | Megalon | yes |
| 20:20.06 | nebula | yes, and remove the class_type loop, that was for when you could get other classes' talents |
| 20:20.19 | Stanzilla | http://pastebin.com/uMzQ9ii6 |
| 20:20.21 | Stanzilla | sanity check |
| 20:20.33 | nebula | well, modify it to only affect the player class i guess |
| 20:22.38 | Stanzilla | http://pastebin.com/gt62mh72 |
| 20:22.41 | Stanzilla | nebula: like that? |
| 20:23.11 | nebula | don't need the if, you just set what you're checking |
| 20:23.27 | Repo | handynotes_hallowsend: 03EthanCentaurai 07master * v1.1.2-5-g33543e5 Core.lua: [+1 commit] Use the calendar API to detect when the holiday is active. |
| 20:23.46 | Stanzilla | well for the defaults? |
| 20:24.11 | Stanzilla | well ok if you cant put your own data in the cache |
| 20:24.15 | Stanzilla | something else is broken anyway |
| 20:24.35 | nebula | loading talents for other classes has been broken since 6.0 |
| 20:25.57 | *** join/#wowace Trela (~Erza@2607:f298:6050:5f75:69b8:c565:e3f7:1ec) |
| 20:26.01 | Stanzilla | ok |
| 20:26.02 | Stanzilla | so http://pastebin.com/TL0QgGAu |
| 20:28.12 | nebula | looks fine other using talentId twice, could dump the locals above the loop and just do local _, talentName, talentIcon = GetTalentInfo(tier, column, GetActiveSpecGroup()) \ local talentId = (tier-1)*3+column |
| 20:28.43 | EthanCentaurai | nevcairiel, pins in Outland aren't showing up any more on the continent map but the zones themselves are okay |
| 20:29.04 | nevcairiel | update =p |
| 20:29.08 | Torhal | EthanCentaurai: Still better than Astrolabe. |
| 20:29.10 | Torhal | >.> |
| 20:29.19 | nevcairiel | unless its only outlands |
| 20:30.12 | EthanCentaurai | it's only Outland, every other continent is fine |
| 20:30.45 | nevcairiel | oh i see it now |
| 20:30.48 | Stanzilla | I think the icon cache is still broken |
| 20:30.56 | nevcairiel | for some reason outlands is being mapped into azeroth |
| 20:31.00 | Stanzilla | the config only shows placeholder icons |
| 20:31.19 | nevcairiel | why would it do that o.o |
| 20:31.43 | Stanzilla | talking about WA, nev .D |
| 20:31.51 | nevcairiel | i'm not talking to you |
| 20:31.54 | Stanzilla | oh ok |
| 20:33.09 | EthanCentaurai | EthanCentaurai, causing awkwardness in IRC since 2007 |
| 20:33.41 | Torhal | That was...awkward. |
| 20:34.04 | nebula | yea, seriously EthanCentaurai, put on some pants |
| 20:34.09 | EthanCentaurai | that awkward moment when you realise you are being awkward by causing awkwardness |
| 20:34.17 | EthanCentaurai | pants are for scrubs |
| 20:39.05 | Stanzilla | maaan |
| 20:39.08 | Stanzilla | why is this shit so annoying |
| 20:39.45 | Megalon | did you just describe weakauras? |
| 20:39.46 | Megalon | >:D |
| 20:39.54 | Stanzilla | yes |
| 20:40.02 | Stanzilla | tell me why it displays placeholder icons in the config :< |
| 20:40.13 | nebula | placeholder? |
| 20:40.26 | Stanzilla | the ? icon |
| 20:41.22 | Stanzilla | http://pastebin.com/cdjZtLzB current code |
| 20:42.08 | Stanzilla | except it's player_class instead of class in the last bit |
| 20:42.22 | nebula | well you don't need the lookups, replace the bits with [class] with the vars directly |
| 20:42.43 | nebula | talentName / talentIcon |
| 20:43.30 | Stanzilla | WeakAuras.talent_types_specific[class][talentId] = "|T"..talentIcon or "error")..":0|t "..talentName or "error"); |
| 20:43.33 | Stanzilla | like so? |
| 20:44.36 | Stanzilla | WeakAuras.talent_types_specific[class][talentId] = "|T"..(talentIcon or "error")..":0|t "..(talentName or "error"); |
| 20:44.38 | Stanzilla | without syntax fail |
| 20:46.40 | Stanzilla | I think the class name is wrong |
| 20:46.44 | Stanzilla | it was a loop before |
| 20:47.18 | *** join/#wowace Gewitters (~Gewitters@cpe-172-248-232-165.socal.res.rr.com) |
| 20:49.40 | nebula | don't need or error |
| 20:49.56 | nebula | it should be player_class |
| 20:51.06 | Stanzilla | does not display an icon though :E |
| 20:51.58 | znf | beats Stanzilla with a stick |
| 20:58.23 | Stanzilla | talentCache is always empty |
| 21:05.50 | Repo | herebedragons: 03Nevcairiel 07master * 1.03-release-5-g5813ef5 HereBeDragons-1.0.lua: [+1 commit] Fix logic in zone transform check |
| 21:05.56 | nevcairiel | EthanCentaurai: this fixes outland, i think |
| 21:06.50 | nevcairiel | i'm surprised transforms worked at all before, the logic must've been weirdly broken |
| 21:06.56 | nevcairiel | it worked for all zones it was meant to work on |
| 21:07.01 | nevcairiel | just additionally for outlands somehow |
| 21:07.01 | nevcairiel | :D |
| 21:07.40 | EthanCentaurai | downloads. |
| 21:08.34 | nevcairiel | i'll probably tag the library tomorrow and tag a b unch of the addons as well to spread the update |
| 21:08.59 | znf | yay |
| 21:09.01 | znf | nev is fixing wow |
| 21:09.42 | nevcairiel | ah well screw it, lets tag the lib now |
| 21:09.50 | Repo | herebedragons: 03Nevcairiel 041.04-release * 7e5efb3 /: [new tag] Tag as 1.04-release |
| 21:10.03 | Megalon | IT'S HAPPENING |
| 21:10.07 | EthanCentaurai | PANIC |
| 21:10.14 | Megalon | PARNIC |
| 21:10.26 | Repo | archy: 03Torhal 07master * 6.2.0.5 Digsite.lua: [+1 commit] Fixes for digsite minimap arrow. |
| 21:10.26 | Repo | archy: 03Torhal 046.2.0.5 * 8d2ea59 /: [new tag] |
| 21:10.51 | nevcairiel | good for archy to a lso grab it r ight away =p |
| 21:12.28 | Torhal | Heh, that was totally coincidental. |
| 21:12.29 | Stanzilla | nebula: help :( |
| 21:13.11 | nevcairiel | Torhal: http://www.wowace.com/addons/handynotes/ relationships still lists astrolabe :( |
| 21:13.25 | nevcairiel | oh its set on the website |
| 21:13.26 | nevcairiel | duh |
| 21:13.30 | nevcairiel | nevermind! |
| 21:13.33 | Torhal | o.O |
| 21:13.38 | Megalon | hihi |
| 21:13.56 | nevcairiel | i forgot this default relationship function existed |
| 21:15.17 | Repo | handynotes: 03nevcairiel 04v1.4.2 * r256 : Tag as v1.4.2 to pull in new HereBeDragons-1.0 |
| 21:19.01 | EthanCentaurai | yay, thanks nev :D |
| 21:20.32 | nevcairiel | now lets see why fishing nodes wouldnt be working |
| 21:20.55 | nevcairiel | which zone was this in Torhal |
| 21:21.15 | Torhal | Nagrand, near River Post (unless you already have this node) |
| 21:21.24 | nevcairiel | nah i barely fish |
| 21:21.39 | nevcairiel | but i was mostly interested in tanaan vs. not tanaan |
| 21:22.19 | Torhal | I mostly fish just to mark the nodes so I don't constantly go "Oh, new node!" and run to it only to find it isn't an herb. |
| 21:22.35 | nevcairiel | just turn off fish tracking? |
| 21:22.35 | nevcairiel | :D |
| 21:23.39 | Torhal | Heh, could do that as well. |
| 21:23.44 | Stanzilla | http://pastebin.com/QNB4hwSv < someone please tell me why talent_cache in the sv would be empty? |
| 21:24.53 | Funkeh` | cuz bad |
| 21:25.04 | nevcairiel | actually in that riverside post area is a bit of weird phasing going on |
| 21:25.05 | Stanzilla | that is a constant that did not change |
| 21:25.07 | nevcairiel | lets see |
| 21:25.29 | nevcairiel | i occasionally had a tree there which i couldnt harvest because it despawned when i got near it |
| 21:25.46 | Torhal | Yeah, I think they fixed that tree. I don't remember being pissed off there in some time now. |
| 21:26.32 | znf | Why does Torhal piss on trees? |
| 21:26.43 | nevcairiel | what do you piss on? |
| 21:27.03 | Funkeh` | bushes |
| 21:27.10 | Funkeh` | no trees in romania |
| 21:29.09 | nevcairiel | it seems to work in general, one school i found on the way added fine |
| 21:29.22 | Torhal | I'll fly back out there and see what happens. |
| 21:29.43 | nevcairiel | i'll go there and see if some phasing os throwing it off |
| 21:33.49 | nevcairiel | hm nah all fine |
| 21:34.14 | Torhal | nevcairiel: Yeah, works now on the same node it didn't before. Lib issue that was somehow fixed? |
| 21:34.40 | nevcairiel | i wouldnt know through what |
| 21:34.56 | nevcairiel | oh well |
| 21:34.56 | Torhal | Weird. |
| 21:36.00 | nevcairiel | if you happen to be running carbonite it'll blow up, but i doubt you are =p |
| 21:36.15 | nebula | carbonite still exsits? |
| 21:36.20 | nevcairiel | sadly, yes |
| 21:36.30 | nevcairiel | and it butchers map api |
| 21:36.54 | Torhal | I get bug reports asking me to support it in NPCScan and I forget what else. |
| 21:36.56 | nebula | curse should mod that shit until he fixes it |
| 21:37.24 | nevcairiel | the map api shit is even "new", from december |
| 21:37.39 | nevcairiel | that may at least be a reason that he will also fix it agian |
| 21:39.14 | Stanzilla | nebula: heeelp |
| 21:39.21 | nebula | what'd you do |
| 21:39.30 | Stanzilla | it does not write the cache |
| 21:39.47 | nebula | add some debug and invoke it manually |
| 21:39.49 | Stanzilla | talentcache is always empty |
| 21:40.43 | Stanzilla | can I do dump(db.talent_cache) |
| 21:40.46 | Stanzilla | in code? |
| 21:41.20 | Stanzilla | no I cant |
| 21:41.40 | nebula | you can print what you're setting and check that it's actually set afterward |
| 21:41.44 | nevcairiel | DevTools_Dump(var) |
| 21:41.55 | Stanzilla | I already checked the vars |
| 21:41.58 | Stanzilla | and they were fine |
| 21:42.39 | Stanzilla | nevcairiel: attempt to call global 'DevTools_Dump' (a nil value) |
| 21:42.47 | nevcairiel | wfm =p |
| 21:42.48 | Megalon | /dump once |
| 21:42.57 | Megalon | so the devtools get loaded :P |
| 21:42.57 | nebula | Stanzilla: you don't even need that table, it's never referenced anywhere else, lol |
| 21:42.58 | nevcairiel | yeah may need to load the addon |
| 21:43.08 | nebula | only talent_types_specific is used |
| 21:43.28 | Stanzilla | why does that not write into the sv |
| 21:46.36 | Stanzilla | I mean it calls the function |
| 21:46.39 | Stanzilla | the vars are fine |
| 21:46.42 | Stanzilla | it just does not want to write |
| 21:47.40 | nebula | thats unfortunate, except it doesn't matter at all |
| 21:48.30 | Stanzilla | eh |
| 21:48.33 | nebula | also, considering spec-based talents, it's gonna be wrong as well |
| 21:48.41 | nebula | <nebula> Stanzilla: you don't even need that table, it's never referenced anywhere else, lol |
| 21:48.45 | Stanzilla | yes |
| 21:49.02 | Stanzilla | I wanted to say that the talent_types specific one is never assigned to db. |
| 21:49.06 | Stanzilla | could that be it? |
| 21:49.27 | nebula | it's not, it's generated on load |
| 21:49.33 | Repo | herebedragons: 03Nevcairiel 07master * 1.04-release-1-g8511aa2 HereBeDragons-1.0.lua: [+1 commit] Wipe data on upgrade |
| 21:50.08 | Stanzilla | ok then I guess I need to change something in the config code so it reads that table right? |
| 21:50.12 | Stanzilla | I mean it used to work before.. |
| 21:50.49 | Stanzilla | is that format even correct? http://vgy.me/lr67sp.png |
| 21:50.59 | Stanzilla | I cant copy the text because chat addon escapes shit |
| 21:51.57 | nebula | looks right |
| 21:52.19 | Stanzilla | http://pastebin.com/YkNZpjs3 |
| 21:52.20 | Stanzilla | current code |
| 21:52.52 | Stanzilla | and I think we need it in the sv so the icons show up when on a different class? |
| 21:54.03 | nebula | no |
| 21:54.19 | nebula | on a different class it'll just say tier x - column y or something |
| 21:54.33 | Repo | herebedragons: 03Nevcairiel 07master * 1.04-release-2-g65f47d1 HereBeDragons-1.0.lua: [+1 commit] Always wipe transform table, as its appended to, and only wipe master data table on upgrade |
| 21:54.51 | Stanzilla | I just realized |
| 21:54.55 | Stanzilla | that I could very well be retarded |
| 21:55.01 | nebula | yup |
| 21:55.08 | nebula | i've known that for some time |
| 21:55.22 | Stanzilla | the whole time, the code was for the icon and text in the dropdown, yeah? |
| 21:55.22 | Megalon | but it's always good to get them to find out themself |
| 21:55.51 | nebula | or the multiselect list, yes |
| 21:56.09 | Repo | gathermate2: 03Nevcairiel 041.35.2 * 5118306 /: [new tag] Tag as 1.35.2 |
| 21:56.09 | Stanzilla | ..and I always thought about the ones in the aura list |
| 21:56.10 | Stanzilla | http://vgy.me/aUr7T9.png |
| 21:56.12 | Stanzilla | :( |
| 21:56.35 | Repo | routes: 03Nevcairiel 04v1.5.2b * d4c6d42 /: [new tag] Tag as v1.5.2b |
| 21:56.49 | nebula | you can condense that down into http://pastebin.com/DnXVkUDm |
| 21:57.57 | Stanzilla | you forgot the class check from unitclass though? |
| 21:58.04 | Stanzilla | I mean, the call to UnitClass |
| 21:58.09 | nebula | oh, yea |
| 21:58.46 | Stanzilla | wonder why he ever had that sv based cache there, too |
| 21:58.49 | Stanzilla | if it was never used |
| 21:59.45 | nebula | probably was used before they removed getting other class talents |
| 21:59.56 | nebula | <PROTECTED> |
| 22:02.22 | Repo | handynotes_hallowsend: 03EthanCentaurai 07master * v1.1.2-6-g0fd4b50 Core.lua: [+1 commit] Disable debug. |
| 22:02.40 | nebula | should also probably add http://pastebin.com/QfukZzhX |
| 22:02.46 | Stanzilla | nebula: did you also forget the local tier/column or is that not needed? |
| 22:03.18 | nebula | they're already local to the for |
| 22:03.24 | Stanzilla | ok |
| 22:03.46 | Stanzilla | and yes in prototypes I have to work on the test that GetMopTalentInfo involved |
| 22:03.51 | Stanzilla | I guess that is why icons are not loading right |
| 22:04.52 | nebula | interesting, i still have all the talents caches in my sv so they show for other classes |
| 22:05.14 | Stanzilla | yeah |
| 22:05.21 | Stanzilla | that's why I thought that still works |
| 22:05.45 | nevcairiel | so you have a read-only cache now? =p |
| 22:06.42 | nebula | could put all the ids in a table and create it yourself to keep that |
| 22:07.17 | znf | Torhal, https://i.imgur.com/Pe08TXF.jpg |
| 22:08.13 | Stanzilla | nebula: getting that with your function http://pastebin.com/tnsDud6q |
| 22:09.35 | Stanzilla | and yes I think we should just create a table that gets all talent stuff added when you log onto that class |
| 22:09.37 | nebula | http://pastebin.com/DnXVkUDm |
| 22:10.57 | Stanzilla | ok |
| 22:11.21 | Stanzilla | if we just reuse the talentCache we already had in the old code |
| 22:11.24 | Stanzilla | would that work? |
| 22:12.22 | Stanzilla | probably a bad idea to do that on login though.. |
| 22:13.44 | znf | I wonder where the hell is the coupon code for nvidia game thingie |
| 22:14.06 | znf | oh, the Witcher 3 offer has expired |
| 22:14.09 | Stanzilla | znf: you have to enter your receipt number on the website of the place you bought it, then you get a code via email |
| 22:14.39 | znf | I don't want metal gear solid :-/ |
| 22:14.50 | Stanzilla | give it to me then, znf |
| 22:15.04 | znf | but I didn't actually "buy" the card... :-| |
| 22:15.25 | Stanzilla | ... |
| 22:15.36 | znf | I got it as a deal from nVidia |
| 22:15.59 | znf | wait, you have to sign up with GOG? |
| 22:16.01 | znf | no steam? |
| 22:16.02 | znf | fuck that |
| 22:16.13 | nevcairiel | yeah they do their deals through gog |
| 22:16.23 | Stanzilla | I got batman on steam though |
| 22:16.33 | znf | Not going to play it anyway |
| 22:16.44 | znf | Also, the box for the card is in the basement, not going there for a free game, I can just pirate it |
| 22:17.09 | Stanzilla | classic romanian |
| 22:17.54 | znf | if it was on steam... I'd consider it |
| 22:18.01 | znf | but there's no difference between pirating and gog :D |
| 22:19.30 | Stanzilla | so I guess I am back to test = nil and "select(4, GetTalentInfo(ceil(%d/3), (%d-1)%3 +1, GetActiveSpecGroup())) == true", not working :E |
| 22:31.36 | Torhal | znf: Yeah, but not all vapers are obnoxious "I vape" assholes in the same way not all Linux users are "Dude, my neckbeard is glorious." schmucks. |
| 22:31.56 | Torhal | There just happen to be a large population of both. |
| 22:32.10 | nebula | Stanzilla: http://pastebin.com/UNdM6ZVm |
| 22:33.01 | Stanzilla | nebula: let me guess, because WA runs that code, i can't use %d in there so it has to be handled by a function? |
| 22:33.26 | nebula | assume it loadstrings it for some reason |
| 22:33.30 | Stanzilla | yes |
| 22:33.36 | Stanzilla | and because of that I can't use %d, right? |
| 22:33.44 | nebula | huh? |
| 22:34.07 | Stanzilla | because this did not work |
| 22:34.07 | Stanzilla | "select(5, GetTalentInfo(%d)) == true" or "select(4, GetMoPTalentInfo(%d)) == true" |
| 22:34.12 | Stanzilla | eh |
| 22:34.13 | Stanzilla | wrong |
| 22:34.24 | Stanzilla | test = nil and "select(4, GetTalentInfo(ceil(%d/3), (%d-1)%3 +1, GetActiveSpecGroup())) == true" |
| 22:34.24 | Stanzilla | this |
| 22:34.38 | Stanzilla | because I dont get the function returns in the string that it loadstrings |
| 22:34.41 | nebula | it only passes the key once, so the second %d is nil |
| 22:34.48 | Stanzilla | aha! |
| 22:35.20 | Stanzilla | that makes sense, thank you |
| 22:35.31 | Torhal | sees all of those select() calls and instantly thinks Darwinism. |
| 22:35.32 | Stanzilla | so I have to use a function because I only get the D once |
| 22:35.50 | nebula | well you can assign a var to it |
| 22:36.05 | Torhal | Did you try %%d? |
| 22:36.18 | Torhal | Not sure it's applicable in your case, but... |
| 22:36.21 | nebula | definitely not what he wants |
| 22:36.26 | Torhal | Alrighty. |
| 22:38.47 | nebula | wa saves checks as strings through the code then concats them together and loadstrings it to see if it passes |
| 22:39.04 | Stanzilla | I think your function has to be global nebula |
| 22:39.07 | Stanzilla | or it won't work |
| 22:39.14 | nebula | oh, yea, probably |
| 22:39.33 | nebula | so WeakAuras. it |
| 22:40.24 | Stanzilla | yes already did |
| 22:40.30 | nebula | god i hate weakauras more every time i look at it's code |
| 22:41.34 | Stanzilla | WeakAuras. ing wont work either I think :D |
| 22:41.46 | Stanzilla | really has to be global |
| 22:42.05 | nebula | WeakAuras is a global, though |
| 22:43.05 | Stanzilla | it errors though |
| 22:43.12 | Stanzilla | 9x [string "return function(event, combat, petbattle, n..."]:3: attempt to call global 'CheckTalentByIndex' (a nil value) |
| 22:43.25 | Stanzilla | oh maybe |
| 22:43.28 | nebula | .. did you change the string |
| 22:43.55 | Stanzilla | :D |
| 22:44.12 | Stanzilla | OK WORKS, NOTHING HAPPENED |
| 22:44.39 | Stanzilla | the icons in the config are still broken though |
| 22:44.48 | Stanzilla | so something we changed in the cache creator was not right |
| 22:46.08 | Torhal | The fact that you keep saying cache while referring to something in SVs makes me twitch. |
| 22:46.35 | Stanzilla | Torhal: well if I tell you that said SV is always 10mb |
| 22:46.39 | nebula | Torhal: wa caches everything to the sv |
| 22:46.40 | Stanzilla | would that change something? |
| 22:46.48 | nebula | INCLUDING THE SPELL ICON FOR EVERY ID |
| 22:47.23 | Torhal | A cache is something to increase runtime performance. Which is very different from a SavedVariable. |
| 22:47.40 | Torhal | Then WA is fucktarded. |
| 22:47.55 | Torhal | "Let's save shit that could be generated to memory at load time." |
| 22:47.58 | Torhal | Ick. |
| 22:48.09 | Megalon | cache into sv sounds |
| 22:48.13 | Megalon | dirty |
| 22:48.50 | Torhal | I'm not even sure I'd want to refactor something like that. Nuke from orbit and start over? |
| 22:48.50 | Torhal | >.> |
| 22:48.54 | Torhal | <,< |
| 22:48.59 | nebula | pretty much |
| 22:49.04 | nebula | it does everything backward, anyway |
| 22:49.14 | Megalon | na Torhal |
| 22:49.19 | Megalon | we could cache the sv on load |
| 22:49.29 | Megalon | ( ͡° ÍÊ Í¡Â°) |
| 22:49.34 | Torhal | ... |
| 22:50.31 | nebula | https://github.com/WeakAuras/WeakAuras2/blob/master/Options/WeakAurasOptions.lua#L391 |
| 22:50.42 | nebula | so you can get icons from names ALWAYS |
| 22:51.31 | Stanzilla | but nothing about that changed, so why doesnt it display shit :< |
| 22:51.44 | nebula | because using localized shit is a great idea |
| 22:51.55 | nebula | people need to post wa strings for each language |
| 22:51.57 | Torhal | Holy fuck. |
| 22:53.03 | Megalon | it just cycles to call getspellinfo on everything? |
| 22:53.06 | Megalon | :D |
| 22:53.41 | nebula | but don't worry, it only does it once, then saves that 10mb table in your saved var |
| 22:54.07 | Torhal | You could totally move that outside of the SVs by generating it once and storing it all in the fucking AddOn itself. |
| 22:54.24 | Torhal | Still nasty, but way better than SVs. |
| 22:55.42 | Torhal | Something akin to ARL's developer dump for recipes. Generate the stuff in-game, dump it to something you can copy/paste into the code directly. |
| 22:55.57 | Torhal | Then reference that, instead of killing your users' SVs. |
| 22:57.10 | Torhal | DAT upvalue list, though. |
| 22:57.15 | nebula | or just drop it entirely since the sole purpose is to show you a bunch of spellid/icon "matches" when you type a name into the aura trigger |
| 22:57.45 | Torhal | Wait - all of this data is just for fancy display purposes? |
| 22:57.46 | Torhal | Dafuq? |
| 22:57.47 | nebula | yes |
| 22:57.49 | nebula | in options |
| 22:57.53 | Torhal | Jesus. |
| 22:59.01 | nebula | pretty sure phanx made a acegui-sm-widget that did something similar for grid |
| 22:59.06 | nebula | eventually killed it, though |
| 23:00.31 | Megalon | lol |
| 23:00.42 | Megalon | WA sv 300kb, WAOptions sv 10mb |
| 23:01.17 | nebula | yar |
| 23:02.00 | Repo | weakauras-2: 03Stanzilla 07master * 2.1.0.7-70-g08f83cf (3 files in 2 directories): [+1 commit] Remove parts of pre6.0 compat |
| 23:02.16 | Stanzilla | if you install that, are your icons still fine? |
| 23:06.50 | Stanzilla | Megalon be my test dummy |
| 23:07.06 | Megalon | i just opened the options sv |
| 23:07.21 | Megalon | and collapsed iconCache and idCache |
| 23:07.27 | Megalon | oh, the editor is usable again |
| 23:07.47 | Stanzilla | get a better editor, sublime handles it like a champ |
| 23:07.55 | Megalon | http://i.imgur.com/vIEyeH9.jpg |
| 23:08.18 | Megalon | ultraedit looks up a few sec when collapsing 250k line of nested tables |
| 23:08.27 | Stanzilla | wow super old version |
| 23:08.39 | Megalon | indeed :D |
| 23:09.55 | Megalon | well |
| 23:10.20 | Megalon | http://i.imgur.com/s0F3IVx.jpg |
| 23:10.23 | Megalon | that? :P |
| 23:10.50 | Stanzilla | expand a group that has lots of icon auras |
| 23:10.55 | Stanzilla | and tell me if they show |
| 23:11.24 | Megalon | lots of them have custom |
| 23:11.52 | Megalon | oh icons of aura |
| 23:12.31 | Megalon | http://i.imgur.com/SMviMk0.jpg ? |
| 23:12.37 | nebula | automatic icons of anything for not my class are question marks |
| 23:12.44 | nebula | but that's pretty much how it's always been |
| 23:13.00 | Stanzilla | and for me, automatic icons for my class are, too |
| 23:13.03 | Stanzilla | and that's the problem |
| 23:13.36 | nebula | works fine |
| 23:13.41 | Stanzilla | http://vgy.me/BOUU3h.png |
| 23:13.51 | Stanzilla | yes I did not commit the createiconcache change |
| 23:13.58 | nebula | thats because it's not an aura trigger |
| 23:14.06 | nebula | shit's always been like that |
| 23:14.22 | Stanzilla | are you sure? |
| 23:14.25 | nebula | yes |
| 23:14.36 | Stanzilla | Hrm |
| 23:16.09 | *** join/#wowace harl (harl@unaffiliated/harl) |
| 23:16.15 | Stanzilla | meh I'm pretty sure those used to have icons :< |
| 23:16.25 | nebula | nope. |
| 23:20.09 | *** join/#wowace jleclanche (~jleclanch@unaffiliated/adys) |
| 23:20.36 | Stanzilla | nebula: https://youtu.be/jZ5gErd9dhI?t=73 |
| 23:21.07 | nebula | thats great, clear out the name and type it in again |
| 23:21.42 | Stanzilla | well that means that a) I just fucked my cache or b) it used to work? |
| 23:22.05 | nebula | no, on load it's never worked, but updating the aura will update the icon |
| 23:22.22 | Stanzilla | well it doesn't here, so something is broken? |
| 23:22.28 | Stanzilla | (with the changes I havent commited) |
| 23:24.32 | Stanzilla | nebula: https://dl.dropboxusercontent.com/s/0ty9p36g4yv23m9/spellcachepatch.diff?dl=0 |
| 23:26.56 | Stanzilla | guess |
| 23:27.09 | Stanzilla | because it even does not show an icon when I make a new aura |
| 23:27.38 | Torhal | Y'all need Jesus. And to learn how to use development branches. :D |
| 23:28.09 | Stanzilla | was faster to upload the patch than to create the branch but yea I agree |
| 23:29.32 | nebula | no, i guess it doesn't update the icon at all unless it's an aura now? |
| 23:32.40 | nebula | nothing to do with the "cache" just some other code path was changed |
| 23:36.39 | nebula | which is impossible to see because of changing the whitespace on every line when you first started on github |
| 23:37.53 | nebula | ok, well not impossible, but i'm too lazy to check further |
| 23:40.40 | Stanzilla | hrm |
| 23:55.05 | Megalon | Boston police stop men with guns from attacking the Pokemon World Championships |
| 23:55.14 | Megalon | lol wtf |
| 23:55.52 | Stanzilla | woah |
| 23:56.26 | Megalon | http://www.myfoxboston.com/story/29860467/2-men-arrested-with-several-guns-near-pokemon-world-championships |
| 23:57.04 | Megalon | the pokemon championship seems to be quite the serious business :s |
