IRC log for #arl on 20150531

01:29.14TorhalRessy: pingage
01:29.48Ressyhais
01:29.50Ressywhats up
01:30.05TorhalWhat do you do to run the flag checks?
01:30.14Ressyvisit the trainer
01:30.15Ressy:)
01:30.20Ressyor the vendor
01:30.22TorhalFoof
01:30.23TorhalOk
01:30.33TorhalThis code is terribad.
01:30.44TorhalI just continued to tack shit on instead of rewriting it.
01:31.14RessySorry. >.<
01:31.20TorhalGetting rid of all the "let's find this recipe by name instead of USING THE FUCKING RECIPE OBJECT THAT WE ALREADY HAVE HOLD OF" stuff.
01:31.28Ressylol
01:32.29TorhalIf I didn't use the PTR on occasion, I'd uninstall it. I'm getting tired of waiting for it to update before I can start the live client.
01:32.37TorhalIn what universe was that ever considered a good idea?
01:32.46RessyYou can cancell the PTR update.
01:32.52RessyIts all in BDA settings
01:33.08Torhal"Oh, hey! There's a Diablo III update! BET YOU WISH YOU COULD PLAY WOW, HUH?!?"
01:33.26RessyChange it to Game specific settings
01:33.36Ressyand you can set it to update PTR when you want it to.
02:04.24*** join/#arl Ressia (~Ressy@WoWUIDev/WoWAce/ARL/Ressy)
02:04.24*** mode/#arl [+v Ressia] by ChanServ
02:06.22RessiaThat was fun
02:06.31Ressiaattempting to test, and I lose connection to everything
02:06.33Ressiabut netflix
02:07.29Ressyoooh.. Its not doing the Missing Vendor, Extra vendor thing
02:09.07RessyTorhal...
02:09.08RessyOdd issue.
02:09.14RessyDid a scan of leatherworking.
02:09.34Ressy"ProfessionScan(): output is empty."
02:09.39Ressypops up in /arl debug
02:10.50TorhalThat means there were no issues, unless something went horribly wrong. Other professions give output.
02:11.28TorhalThere is a problem with detecting skill levels, though, since trainers only give the recipe name - no ID available from the client API.
02:13.26RessyI've never had that pop up before.
02:13.58Ressyoooh
02:14.10RessyEnchanting is popping up ok now it seems.
02:15.15RessyGonna change some wording...
02:15.17TorhalOh? No data, or correct issues detected?
02:15.24Ressycorrect issues detected
02:15.28TorhalGood deal.
02:15.32Ressy"Recipe 47051 (Greater Dodge): Has Trainer filter flag, but also has a recipe item (35756)."
02:15.40RessyGonna change that to Spell 47051
02:15.54TorhalYeah. Basically a warning to perhaps check to see if something has changed.
02:15.58Ressynods
02:15.59TorhalHeh
02:22.45RessyProblem.
02:23.06RessyIts pulling "Greater Dodge" 47766.. and confusing it with 47051
02:24.37Ressydid a scan of an Enchanting trainer
02:25.39Ressysaying that 47051 is trainer.
02:25.44Ressywhen its not.
02:25.49Ressywe had that it is..
02:25.51Ressyits really really not
02:27.17TorhalHuh?
02:27.25TorhalShit
02:27.31RessyIts somehow confusing 2 spells of the same name
02:32.41TorhalRessy: New push.
02:34.55Ressyuh
02:34.56Ressydidn't fix
02:35.06RessyBoth of the spells are enchanting
02:36.01TorhalSonofabitch.
02:36.10TorhalCan't help, there.
02:36.18TorhalTrainers only report the skill name.
02:36.25TorhalFucking Blizzard.
02:36.53Ressythen how does Wowhead pull the spellids?
02:37.45TorhalDBC files.
02:38.00TorhalGood old reverse-engineering.
02:39.11Ressydamnit
02:39.31RessyOK.. any chance made to enchanting will require a dump then a scan & manual fixing. lol
02:40.08TorhalOnly for those specific trainers, for those specific spells, right?
03:02.17RessyI'll do a dump then a scan.. Then check for flags of recipe & trainer..
03:02.20Ressyand remove trainer
06:24.57sidjimoin
12:18.49*** join/#arl pompy (~Mike@c-73-33-19-204.hsd1.nj.comcast.net)
12:18.49*** mode/#arl [+v pompy] by ChanServ
12:19.55*** join/#arl pompy1 (~Mike@c-73-33-19-204.hsd1.nj.comcast.net)
14:10.36*** join/#arl Ressy (~Ressy@WoWUIDev/WoWAce/ARL/Ressy)
14:10.36*** mode/#arl [+v Ressy] by ChanServ
15:04.27*** join/#arl Ressia (~Ressy@WoWUIDev/WoWAce/ARL/Ressy)
15:04.27*** mode/#arl [+v Ressia] by ChanServ
16:07.16sidjimoin
16:46.16RessySo Torhal... The scan just isn't working.
16:47.10RessyProfessionScan()
16:47.32RessyI wiped out all the No_Role_Flag stuff for Alchemy..
16:47.45RessyThen did a scan..
16:48.23RessyAnd its still showing "ProfessionScan(): output is empty."
18:13.08RessySo Torhal... How do I revert your commits locally so I can actually work on ARL.. >.>
19:39.28TorhalI'm going to set up a trigger for "So Torhal..." that makes me answer with "It is of no cern to me, Ressy..."
19:39.28Torhal>.>
19:39.29Torhal<.<
19:39.35Torhal*concern
19:39.39TorhalDamn typos.
19:40.09TorhalJust woke up from my nap. Should probably actually eat something today.
19:59.09Ressyyes, eat
20:24.35*** join/#arl pompy (~Mike@c-73-33-19-204.hsd1.nj.comcast.net)
20:24.35*** mode/#arl [+v pompy] by ChanServ
20:53.28TorhalRessy: Anyway, I don't see how my changes would break things; instead of passing a recipe name and performing a lookup to get an ID, I'm passing the recipe itself.
20:53.59TorhalI'll look at it though.
20:54.22TorhalReverting locally isn't feasible really since I removed the code that performed the name-to-ID-to-recipe lookups.
21:07.16RessyThen... Can we revert the changes? >.>
21:07.30RessyAlso.. Trying to add support for TSM3.  I have beta access...
21:07.39RessyI just need to know what to look for.
21:11.05TorhalYou want me to undo what I did yesterday, which actually fixed things, instead of fixing whatever is currently wrong?
21:11.06TorhalO.o
21:11.07Torhalo.O
21:11.17TorhalMinecraft is beginning to actually sound sane.
21:11.18Torhal:D
21:11.39TorhalJust look for scan_button in Core.lua
21:12.13RessyOk. Fix the problem then. lol
21:12.18RessyI just figured you didn't have the time. :)
21:12.38RessyAs for the TSM...
21:12.46RessyI meant what do I look for in the TSM addon.:)
21:15.55Ressyso I can hook the button into the TSM3 crafting frame. :)
21:16.18TorhalI have no idea.
21:17.54TorhalI mean, there will obviously be a frame where everything is displayed. I have no idea what it will be called, though. The current TSM entry in ARL may provide a clue.
21:18.14Ressyyeah
21:19.19RessyOH
21:19.21RessyI think..
21:19.28RessyI think I might have found it.
21:19.29Ressyor how to
21:34.30Ressyheh.. and another one.
21:35.09TorhalHmm?
21:35.50RessyAnother TSM3 error which causes problems with ARL.
21:35.51Ressy:)
21:39.45TorhalI blame Ackis.
21:43.16RessyI blame Sapu
21:43.27RessyHe's trying to fix.
21:43.49Ressy[17:43:29] <@Sapu> well i found another issue, but doubt it's related
21:44.13RessyHe built his own error handler in tsm.
21:49.15RessySo yeah..
21:49.42RessyARL's button isn't showing up on the guild profession panel after opening the personal profession panel with tsm_crafting running.
21:54.12TorhalHAWT
21:54.28TorhalThat's an ARL issue.
21:54.39TorhalOnly one scan button, only parented/anchored once.
21:54.48Ressyorly?
21:54.56Ressydamn
21:55.00Ressywas working fine with 2.0
21:55.03TorhalUnless I'm misremembering, which could be.
21:56.22TorhalDoes TSM have its own frame for that?
21:56.45TorhalBecause I don't see how it would work. InitializeScanButton() in Core.lua is what parents/anchors it
21:56.52TorhalFires on TRADE_SKILL_OPEN
21:58.54Ressyit worked with 2.0..
21:59.37TorhalI don't see how :D
21:59.58RessyI mean getting the ARL button on the guild profession panel.
22:00.04Ressywith TSM2.
22:00.22Ressyshrugs
22:00.47Ressyand as soon as I disable crafting, or just not open the personal crafting, it works and shows up fine.
22:01.19TorhalOh
22:01.42TorhalI see the issue: TSM3 might have a separate frame for that.
22:02.06Ressy[17:51:11] <@Sapu> once it's shown for the personal one
22:02.07Ressy[17:51:46] <Ressy> But it worked with 2.0
22:02.07Ressy[17:52:05] <@Sapu> maybe TSM_Crafting showed for the guild profession in 2.0
22:02.07Ressy[17:52:06] <@Sapu> i don't remember
22:02.07TorhalTSM2 uses the same frame regardless of whether or not it's your profession or a guildie's
22:02.10TorhalThat's why it works.
22:02.33TorhalSo not an ARL issue.
22:02.43TorhalI don't do guild shit, so I didn't know it was the same frame.
22:02.51TorhalIt is for the stock UI as well, apparently.
22:03.41Ressy[18:02:52] <Ressy> Are you touching the guild profession panel with 3.0?
22:03.41Ressy[18:03:19] <@Sapu> no
22:04.54TorhalAnyway, don't care about TSM until he releases it. Trying to concentrate on figuring out why ARL is locking up on a scan.
22:05.17TorhalDoing a profession scan locks the progress bar on "Gloves of Spell Mastery" every time.
22:06.38TorhalBut yeah - the guild professions frame is the same frame as the personal professions frame in the default WoW UI and TSM 2.
22:06.45TorhalSo no idea why he wouldn't still be doing that.
22:06.57TorhalSame format, same data, different toon.
22:07.06Ressyand if he reverted to default guild profession frame for 3.0?
22:07.31RessyTorhal: Its related to the recipe item.
22:07.38Ressycomment out that line, and it runs fine.
22:07.44Ressyalong with the next spell as well.
22:07.46TorhalWouldn't work. Also, as I said, I don't know why he wouldn't use the frame for both since the ONLY difference is the player's GUID.
22:11.17RessyI ran into that issue before.
22:11.27RessyI didn't report it cuz you were already busy with so many other things
22:18.29TorhalI mean, I don't use a separate frame for ARL when you scan a guildie's profession...
22:18.30Torhal:D
22:18.55TorhalThere's no difference except for the data available from WoW's API.
22:21.35RessyYou don't scan a guildy's profession..
22:21.56Ressyyou can't
22:22.11RessyHowever theres an option in the guild panel, View Professions.
22:22.18TorhalExactly
22:22.53TorhalAnd when you click on that profession, it populates the TradeskillFrame with the data from that toon.
22:23.02Ressyno.
22:23.13Ressyit populates with aggregated data from all toons in guild with that profession.
22:24.00Torhalhttps://www.dropbox.com/s/xnki9cew504t7xh/Screenshot%202015-05-31%2017.23.51.png?dl=0 <-- Looks rather specific to me.
22:24.17Ressywe're lookign at separate things.
22:25.22TorhalIt's broken with TSM2 as well.
22:25.39TorhalScan button doesn't appear if you've opened TSM first.
22:26.12TorhalAnd doesn't appear on TSM if you open the guild equivalent first.
22:26.26TorhalNeat.
22:26.39Ressyworks fine in TS2.
22:26.46Ressyhttps://www.dropbox.com/s/cwpjig8952btqqm/Screenshot%202015-05-31%2018.26.07.png?dl=0
22:26.56Ressymiddle screen is live with TSM3.
22:27.02Ressyright screen is PTR with TSM2.
22:27.06TorhalI just reloaded my UI, opened the guild version, had a scan button. Opened TSM, did not have a scan button.
22:27.22TorhalReloaded again, did the reverse. The reverse was true.
22:27.33Ressyhm
22:27.46TorhalThere is only ONE scan button. Can't have it on both places :D
22:27.52Ressylovely.
22:28.13TorhalNot a high priority since this isn't end-user stuff.
22:28.22RessyThis is what end users will run into.
22:28.45Ressybut essentially it means I disable TSM3 to do any sort of guild data mining.
22:30.03TorhalOr just don't use it before doing so...
22:30.05TorhalAnd reload to use it.
22:52.05*** join/#arl pompy (~Mike@c-73-33-19-204.hsd1.nj.comcast.net)
22:52.05*** mode/#arl [+v pompy] by ChanServ
23:02.34*** join/#arl pompy1 (~Mike@c-73-33-19-204.hsd1.nj.comcast.net)
23:02.55TorhalRessy: Ugh. Either Ackis or myself fucked up hard. We were trying to add class filters BY NAME.
23:03.02TorhalRessy: https://www.dropbox.com/s/b461le6o6gznmzc/Screenshot%202015-05-31%2018.02.30.png?dl=0
23:07.45Ressyo.O
23:12.04Ressywooo
23:12.08Ressythings look to be working again
23:12.13TorhalIndeed.
23:12.32Ressyreal test is enchanting. >.>
23:13.10Ressymkay.
23:13.26RessyJust warning..
23:13.35RessyI'm going to start the no role flag things from scratch. :)
23:13.47RessyThe list is huge, and after the previous issues, I don't trust'em. >.>
23:14.32Ressyyes/no?
23:15.03TorhalDon't care. Was a pompy1 thing he said helped.
23:15.11Ressylol
23:15.26Ressyyeah. it keeps things from popping up.

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.