IRC log for #arl on 20120918

00:23.59TorhalLovely.
00:24.36RessyI've got the spell numbers added to core & scanner
00:24.41RessyI just have no idea how to make it work.
00:24.46RessyYou want me to push or no?
00:26.43TorhalScanner?
00:26.45TorhalHuh?
00:26.48TorhalWHat you do?
00:26.50Torhalo.O
00:27.00RessyIn my copy only..
00:27.05Ressyhaven't committed or pushed.
00:27.19RessyI added the spell ids for the new 'way of the' specialties
00:27.36RessyI have no clue how to get it working, so i'm not screwing with that.
00:27.36TorhalThey're actually specialties as in - you can only know one at a time?
00:27.40Ressyno.
00:27.44RessyYou can know all of'em.
00:27.46TorhalThen they're not speialties.
00:27.53TorhalAnd we don't care about them.
00:28.11Ressyo.O
00:28.12TorhalWith Alchemy you can only know Elixir or Potion mastery.
00:28.15RessyI can't scan'em.
00:28.22TorhalBut we don't care because all that does is proc.
00:28.30TorhalCan you scan 'em now?
00:28.33Ressyno
00:28.35TorhalAfter you did whatever you did?
00:28.37Ressyno
00:28.59TorhalThen they're just spells you need to learn in order to learn others, from what I'm reading
00:29.26TorhalHrm.
00:29.32Ressyway of the.. is a spell like cooking is.
00:30.23TorhalSo are all the recipes, so I'm not convinced these should be treated as specialties (especially since you can learn them all)
00:31.13Torhallogs into Beta
00:39.56TorhalRessy: I can load smelting just fine...
00:41.54Ressyeven with auto scan trainer turned on..
00:42.12RessyI now need to do a full smelting scan before I can do a trainer scan for smelting
00:42.20Ressywhich is totally opposite of the other professions
00:50.38TorhalShit shit shit shit shit shit
00:52.02TorhalAll of them are completely fucked.
00:52.12TorhalI'll have to hardcode the NPC IDs.
00:52.36TorhalGetTradeSkillLine() is supposed to return the profession name and your current profession level
00:52.54TorhalFor all of them at The Halfhill Market, it returns "UNKNOWN, 0"
00:54.20TorhalNevermind, I'm losing it.
00:55.58TorhalOk, all I need to do is add a mapping to Cooking (like I do for Smelting to Mining)
01:01.19TorhalWell, not really. It's more involved.
01:01.23Ressy>.<
01:02.01RessyI really need to learn lua.
01:02.05TorhalBasically, /dump GetTrainerServiceSkillLine(1) should return "Cooking" but it returns "Way of The Grill" etc
01:02.11Ressyah
01:02.15TorhalSo I need to map all of these TO cooking.
01:03.22Ressythats... gonna suck
01:04.00Ressyquestion: How do I get map #  Pandaria zones need to be mapped in one of the main .lua files
01:06.10Torhalo.O
01:06.14TorhalOh
01:06.44Torhal<PROTECTED>
01:07.09TorhalCase doesn't matter
01:07.20Torhal"oRgRiMmAr" will work, for example
01:09.07Ressy2 map ids for jade forest....
01:09.08Ressy>.<
01:09.37Ressy1 old, 1 new?
01:09.50TorhalJust pick one. All we care about is the name being localized.
01:10.16Ressyah. k
01:11.03TorhalOk, do you have a list of all of the "Way of The X"?
01:13.54TorhalNevermind - got 'em
01:13.54RessyBrew: Spell 125589
01:13.56Ressyk
01:13.58Ressyheh
01:20.50RessyZone name for Kun-Lai Summit, isn't "Kun-Lai Summit" "Kunlai summit" "Kunlai"
01:20.52Ressy....
01:20.59TorhalOk, pushed.
01:21.32Ressyawesome
01:21.33Ressyty
01:22.15TorhalThat push also allows you to auto-ad trainers to the localization file.
01:22.22Torhal<PROTECTED>
01:22.45Ressynice
01:22.46TorhalThere I go again not typing the second letter. add*
01:23.35RessyThoughts on how to get zone name. :\
01:24.22TorhalSure.
01:24.27TorhalJust use the word "summit"
01:25.05TorhalThe reason for multiple zone maps is likely scenerios.
01:25.10Ressyoh wow...
01:25.14Ressythats awesome
01:25.15Ressyty
01:25.21Torhal:D
01:27.16RessyScanner's not working.
01:27.26TorhalHmm?
01:27.44Ressyhttp://www.wowace.com/paste/6257/
01:28.21TorhalWhat the actual fuck?
01:29.16TorhalWhat did you do to get that?
01:29.35Ressyerr...
01:29.39Ressylemme check something
01:30.16Ressyffs. nm.
01:30.27Torhalmumbles something about local modifications to Constants.lua
01:30.29Torhal:D
01:31.13RessyI forgot to copy over Constants.lua to my beta install
01:31.20Ressyarg
01:31.22Ressysry
01:31.26TorhalBwaha!
01:32.21TorhalSo in this case it was a non-modification that did it.
01:32.24TorhalHehe.
01:33.55Ressyya
01:34.11RessyI'm just copying my updated files to live, then i'mma kill my beta copy, and recopy the entire thing over.
01:34.42Ressyonly.... my live cooking doesn't work
01:34.49TorhalHuh?
01:35.09RessyAckis Recipe List: nil profession passed to InitializeProfession()
01:35.10RessyAckis Recipe List: Recipe database not loaded.  Please scan the tradeskill first then try the datamining.
01:35.42TorhalLet's ping Ackis some more.
01:35.43Torhal:D
01:35.55Ressylol
01:36.23Ressyalchemy does the same "nil" line
01:36.50TorhalYou have weird shit going on.
01:36.55Ressynods
01:36.56TorhalMine works just fine (on live)
01:37.00Ressyblah
01:37.10TorhalThen again, I have the same exact files for live and Beta
01:37.16TorhalSymbolic linking FTW.
01:37.23RessyI killed my beta install.
01:37.35TorhalKilled?
01:37.47Ressydeleted
01:37.58TorhalBefore testing on live? The madness!
01:38.06RessyI hadn't done much.
01:38.10TorhalSeriously, though - hope you didn't lose anything.
01:38.20Ressymap numbers, 1 recipe in, a few vendors. *shrugs*
01:38.40RessyImma reimport the entire thing.
01:43.35TorhalOh, no wonder I don't have trainers auto-add recipes - all that's available is the name.
01:44.32RessyYou actually also pull the npc id...
01:44.42Ressyand you COULD pull the map name.
01:45.01Ressyfor coords, you can hit a script...
01:45.08TorhalIndeed.
01:45.16Ressywowhead uses /wl loc for example..
01:45.25Ressyall that'd be left is alliance/neutral/horde
01:45.27TorhalThough this was all added before the LibBabbleZone deprecation and I didn't think to add it there.
01:45.30Ressybut.. That'd make us lazy
01:45.43TorhalI already grab coords
01:45.48Ressytrue
01:50.49pompy1it auto adds trainers to the loc phrases, but it doesnt for vendors?
01:52.15TorhalI'll look.
01:54.40pompy1so how well and how exactly does the switch command in git work.
01:58.00TorhalHuh?
02:01.40Ressyok.. I recloned the entire folder, after deleting everything in it.
02:01.45Ressyand I'm getting this error still
02:03.15Ressycooking & alchemy trainers.
02:03.22TorhalRessy: You're fine. My bad.
02:03.30TorhalI have no idea why I even thought that code would work.
02:03.39RessySo I'm not goign crazy? lol
02:03.49RessyI was gonna kick my comp.
02:03.55Ressylol
02:04.52TorhalPushed fix.
02:06.27Ressyawesome. ty :)
02:08.27RessyMy friend is so distracting.
02:09.56Ressythank god she logged.
02:10.13pompy1torhal: the "git switch" command
02:13.20Torhalpompy1: I think you mean the checkout command.
02:13.38TorhalIf you're referring to branches.
02:14.13pompy1they are sorta similar but not
02:14.15pompy1its odd
02:14.23Torhalpompy1: If that's the case, read this - http://git-scm.com/book/en/Git-Branching-Basic-Branching-and-Merging
02:14.29pompy1http://www.syntevo.com/smartgit/documentation.html?page=commands-branch#switch
02:14.40TorhalYeah, non-standard
02:14.59Torhalsorrowthorn:AckisRecipeList callahan$ git switch
02:14.59Torhalgit: 'switch' is not a git command. See 'git --help'.
02:15.46pompy1hm, "Use this command to switch your working tree to a different branch."
02:17.32TorhalSounds like "git checkout <branch name>"
02:18.11pompy1yeah probably.. also in the smartgit docs .. it says
02:18.13pompy1Check Out Use this command to switch the working tree to a certain commit. If you select a commit where local branches point to, you will have the option to switch to these branches.
02:19.59pompy1bc i dont want things to get dirty, but i want the coll 2.0 branch
02:20.42pompy1also, Torhal, this is the dialog for it in my client - https://dl.dropbox.com/u/35894760/gitswitch.png
02:22.16TorhalLooks to me like it's an "Oops! I fucked up! I meant for these changes to be on this branch!"
02:22.19TorhalDon't do that.
02:22.33TorhalIt will then attempt to merge (read what it says)
02:30.40pompy1thats not what i had in mind.. i just thought that would switch it to a diff branch ...... for ex i wanna switch to collectinator 2.0 branch.
02:41.41TorhalYo: http://o7.no/S1f1Kg
02:41.55Ressywhoa
02:42.20Ressythat.. totally.. rocks.
02:42.40TorhalInstead of doing it from the trainer, I do it from the scan itself.
02:42.58TorhalThe TradeSkillFrame has ALL The profession data - so the spell ID is available to me.
02:44.00Ressywhen you committing this change? :D
02:44.32TorhalAs soon as it works with the profession dump
02:44.41Ressyoh. lol
02:45.13pompy1lol you are weird torhal.
02:45.22Torhal…why?
02:45.24pompy1jk
02:45.24Ressyget our hopes up why don't cha. :P
02:45.39pompy1pretty meat
02:45.41pompy1neat
02:45.46Ressyvery
02:46.29Torhalhttp://o7.no/S1fo7x
02:46.51TorhalNeed to fix though
02:46.57TorhalV.CATA be wrong, y0.
02:47.43Ressydefault = mop?
02:49.28TorhalNo: _G.GetExpansionLevel() + 1
02:49.37Torhal(0 is Classic)
02:49.48TorhalThat's why it said V.CATA instead of V.MOP
02:49.59TorhalI didn't have the +1 at first
02:50.30AckisI think I tore a muscle in my arm :'(
02:50.39pompy1eh in a week you can remove the +1
02:50.42pompy1probably
02:51.10Ressynice
02:52.13Torhalpompy1: No I can't
02:52.45TorhalLua arrays start at 1 instead of 0 (which is what most languages use)
02:52.52TorhalThe expansion levels start at 0
02:52.56TorhalSo I ALWAYS have to add 1
02:53.18Ackisadds 2
02:53.20pompy1oh
02:53.35pompy1you are comparing that lookup with the number hte gameversion is associated with
02:53.37pompy1i guess
02:53.59Ackiscomments his adding 2 with -- Add 2
03:06.08pompy1Ackis:  max level?
03:12.27pompy1BL2 unlocking in 1 hour on Steam.. guess they are going by Midnight Eastern for the US .. odd, but cool
03:12.35pompy1even though i will not be playing it right at midnight lol.
03:12.36pompy1><
03:32.07Torhalpompy1, Ressy: Yo.
03:32.09TorhalJust pushed.
03:32.22TorhalWhat you do is scan the profession first. Then scan the trainer.
03:32.31TorhalKerplow.
03:32.40Ressywoots
03:32.52TorhalDump the profession, and if applicable dump the trainer and phrase
03:33.11TorhalAckis: You're a bastard.
03:33.26TorhalFuck you for asking me to add LibQTip to ARL all those years ago.
03:33.31Torhal:D
03:38.46Ackis:)
03:41.02TorhalAlso, Ressy - I must have been stupid tired when I wrote the initial version of the scanner. There will be no more "unconfirmed" levels.
03:41.05TorhalThey'll just get fixed.
03:41.55Ressycool
03:42.08pompy1theyve alweays been fixed
03:42.36Torhalpompy1: No, there were cases where it would say unconfirmed.
03:43.03pompy1Ackis: max level?
03:43.31Ackispompy1: nein, level 13ish only
03:44.00Ackisgoing back to being afk
03:44.09TorhalIn fact, the whole logic for the trainer scane was lolwut?
03:44.13TorhalAckis: Fucker.
03:44.21Torhals/scane/scans/
04:35.12pompy1oh really
04:35.36TorhalSure.
04:35.39TorhalWhy not?
04:36.02pompy1referring to you possibly uncovering the ????????? cyrillic russian issue
04:37.43TorhalWhenever someone pastes CYrillic into the WoWAce forums, all anyone sees is ????
04:45.25pompy1not talking about the forums lol
04:45.52Ressyheaded to bed.. night
04:46.38TorhalI know that
04:46.58TorhalBut if WE see ??? when Cyrillic is used, it stands to reason that THEY would see the same fucking things when English was used.
04:47.02Torhal:P
04:51.54pompy1ok
05:10.18pompy1I know IRC isn't due for looong messages, not trying to pull an Amarande :P, but I really wish I didn't have to say stuff like this, but it gets to meee....
05:10.27pompy1Ok, I appreciate all the work that Ressy is doing, I do.  When I first saw her starting to work on it, I was noticing and thinking that it was just to fill in all the Alliance vendors since I do not have a high level Alliance.  I did not expect all this other stuff to be done...  I'm purrrfectly aware mop arriving sept 25..I would be totally able to get it all done by then.  I had mentioned to Ressy that I will be working on the professions, and she acknowledged, bu
05:10.30pompy1I'm minimizing this for the rest of the night, see ya laters.
12:06.41*** join/#arl Ressy (~Ressy@unaffiliated/ressy)
12:35.01*** join/#arl lua_bot (~lua_bot@li37-84.members.linode.com)
12:51.34Ressypompy1: Is it erroring in wow live leaving in mop trainer/recipes?  Is that why you're commenting everything out?
12:52.10RessyJust trying to figure it out, cuz if we leave it in (without commenting), we can pull the info into a dump, and don't have to do everything by hand.
12:55.38*** join/#arl pompy (~Mike@68.36.135.0)
12:55.38*** mode/#arl [+o pompy] by ChanServ
13:11.24RessyI'm really surprised at the number of people who run alpha versions of arl
13:26.52TorhalRessy: It will show "Unknown" and the spell ID.
13:27.09TorhalBut hey - it's a fucking Alpha.
13:27.30TorhalGo ahead and re-add it, since you can't do a fucking dump otherwise.
13:27.39Torhalgrumps
13:54.26pompyTorhal ahh sooo
13:54.58pompyi scanned the trainer and it just added the trainer flag .. did you intentionally not have it add the crafteditemid? i thought it was supposed to do that from the trainer bc that id is aavail
13:55.48pompyand you cant get the item bind from scanning the tt? and the role flags?!
13:57.03Torhalpompy: It should and if it doesn't it might be because it had no prior data and I didn't think to account for that because we never had recipe skeletons
13:57.17TorhalI'll look at it after work.
13:57.24TorhalSpeaking of - time to  get ready.
13:57.27pompyill rescan the trainer and see if it adds more
13:57.28pompykk
14:02.40RessyTorhal:
14:03.02RessyYou know how you changed Trainers to use Z.<whatever> instead of "whatever"...
14:03.09RessyI did a dump, updated, /rl'd
14:03.15Ressyand on scan, get an error
14:03.26Ressy1x Database\Trainers\Alchemy.lua:10: attempt to index global "Z" (a nil value)
14:03.26RessyDatabase\Trainers\Alchemy.lua:10: in function "InitializeAlchemyTrainers"
14:25.58TorhalRessy: I told you what to add :)
14:26.19Torhallocal Z = private.ZONE_NAMES
14:26.42TorhalDirectly under: local FOLDER_NAME, private = ...
14:28.01Ressyokies
15:08.46Torhal|workRessy: Working now?
15:08.59Ressyyes.
15:09.01Ressyas is pompy
15:09.04Ressywe're working on diff things.
15:09.13RessyHe's working on recipes, I'm working on updating trainers
15:09.13Torhal|workCoordination. I like it.
15:43.48pompyTorhal|work: i made a fresh 85 premade but the arl window wont show when i sacn my p[ersonal professions
15:43.57pompyressy created one and they opened for her apparentl;y
15:44.03pompybut it cant be anmything on my end
15:44.49Ressynot like the dump window, right?
15:44.58pompy:O
15:45.00pompyno
15:45.11Ressyok.
15:45.13pompypersonmalprofes! the scan using the sacn button.......
15:45.31pompydamnit torhal, i hate how "scan" means 2 diff things.. you should havea created separate terminology
15:46.03pompypersonal professions**
15:46.18pompythat is what you tried on your premade?
15:47.27Ressythe lil scan button on your profession.
15:48.05Torhal|workTradeSkill scan versus Profession Scan.
15:48.16Torhal|workThe latter being the copy/pasta window.
15:48.33Torhal|workBut yah, it's fucky.
15:48.34pompyyeah profession scan is used pretty much for your personal ones.. blehhh haha
15:48.36pompyyup
15:48.56pompybut yeah, idk why an arl scan window wont come up doing a TRADESKILL SCAN ! lol <>
15:48.57Torhal|workNo, TradeSkill Scan is when you hit the Scan button on the TradeSkillFrame
15:49.03Torhal|workOh
15:49.11pompythe phrase, not the meaning
15:49.11pompyhaha
15:49.12Torhal|workDunno.
15:49.47Torhal|workI'd be able to fix it if I could reproduce
15:49.49Torhal|workBut I can't.
15:51.06pompyyoure logge dinto the beta?
15:51.10RessyWhen I go to drump trainers, I get a bug sometimes.
15:51.32RessyI have to manually add trainers
15:51.35Ressyerr, dump.
15:52.01Torhal|workWhat's the bug?
15:52.14Torhal|workbug/error/meteorite shower/
15:54.27Ressy2x AckisRecipeList-@project-version@\Scanner.lua:575: attempt to index field "name" (a boolean value)
15:54.28RessyAckisRecipeList-@project-version@\Scanner.lua:575: in function <AckisRecipeList\Scanner.lua:545>
15:54.28RessyAckisRecipeList-@project-version@\Scanner.lua:596: in function "DumpTrainers"
15:54.28Ressy<string>:"ARL:DumpTrainers("jewelcrafting")":1: in main chunk
15:58.07pompyRessy: because you forgot to add some trainers when you scanned and dumped them
15:58.17pompyprobably the reason
15:58.26Torhal|work?
15:58.45pompynothing torhal, we have it under control
15:59.02Torhal|workYou shouldn't have to add trainers is why I asked WTF
15:59.19pompyshe didnt dump the trainer list after scanning the trainer
15:59.20RessyI get that when I try to do the dump
15:59.26pompyyeah
15:59.39pompyduring a new session tho
15:59.47pompynot during the same one you added the IDs in!
15:59.58Ressyis /rl new session?
16:00.00pompyim guessing thats why its complaining
16:00.02Ressyprolly.
16:00.05pompyyeah
16:00.16Torhal|workYou reload and all data not in the DB is wiped.
16:00.23pompyexactly
16:00.56Ressyah
16:00.58Ressygotcha
16:01.10pompyRessy: commit your local changes the best you can, im about to push..then pull mine, if there are any conflicts, try to fix em before pushing!
16:01.27RessyI'm only touching trainers.
16:01.35Torhal|workFuck it, push the conflicts too. That'll teach those bastards that insist on using alphas.
16:01.38Ressyshould be ok, but doing as you said.
16:02.47pompyloll Torhal|work a russian on the beta... character name abbove its head is ???????????
16:02.58pompyyet the cyrillic showed fine in the chatframe
16:03.10Torhal|workHah
16:03.18Torhal|workIt's teh font
16:03.25Torhal|workThe font used for names doesn't support it.
16:03.52Ressy1 note for those doing trainer dumps in future: the all-in-one profession trainers get put into the trainer dumps, and you have to manually remove them, or it complains.
16:04.00Torhal|workYup
16:05.49pompyand yeah torhal.. apparently even ScanProfession() wont do the mop trainable recipes.
16:06.02pompyits not adding the missing bind data, or the role flags
16:06.04pompyherm
16:06.33pompyon beta fyi
16:07.23Ressycommitted
16:07.31Ressyjust the changes to Z.<zonename>
16:09.13pompywell you'll have to commit all your local changes to pull mine, or all hell will more than likely break loose from git.
16:09.22RessyI have committed.
16:09.23Ressy:)
16:09.30Ressyhaven't pushed
16:09.58pompyni gentlepaw is a trainer and a vendor?
16:10.37Ressyjust vendor
16:10.42pompy:O
16:10.48Ressyi think
16:10.50Ressylemme check
16:10.52pompynah'
16:10.58pompyhis id is on addTrainer() functions
16:11.02pompyto recipes
16:11.18Ressyyes, both
16:11.30Ressyerr, just trainer
16:11.55Ressystaring at her offerings now in-game
16:11.57pompyhe's <Alchemy Supplies>
16:12.00pompythats a vendor?
16:12.02Ressyyup
16:12.05Ressygo figure
16:12.15Ressycarries crystal vials
16:12.17Ressyas supplies
16:12.23Ressystupidity at its finest
16:12.28pompyand offers to train recipes?
16:12.36Ressyyup
16:12.41Ressyjust trainer
16:12.58Ressyno recipes on it, just vials
16:15.51Ressyare you gonna push, and I pull, or do you want me to push, and you pull?
16:16.00pompyill tell you when
16:16.02Ressyk
16:16.17Torhal|workWow, this isn't rocket surgery.
16:16.32Ressylol
16:16.46Torhal|workSeriously: You shouldn't have to negotiate who does what in order to use Git :)
16:21.32pompyok, have fun....... tell me now if you can get the arl window to show up doing a TRADESKILL SCAN
16:25.28Ressysob
16:25.31Ressymerge conflicts
16:26.08pompyprobably just in one of the trainer files
16:26.12Ressy3 files
16:26.20RessyVendor & 2 trainer files
16:26.45pompyyou had local changes to vendor file lol?
16:27.02RessyI added a vendor.
16:27.12pompyi didnt touch that file, sooo.
16:27.21pompyand only touched 1 trainer file
16:27.40RessyVendor, Inscription Trainer, Alchemy trainer
16:30.37pompyjust overwrite those 3 with your changes if you can, not much was changed in my end that i did.
16:31.28pompycant remember how to resolve conflicts haha
16:32.29Ressyso...
16:32.31RessyI've resolved.
16:33.00Ressydo I try to pull again, or just push
16:33.20RessyI assume just push
16:33.42pompypush
16:33.52pompyi think
16:33.53pompy!
16:39.03Ressypushing
16:39.07pompypush that and try to do a tradeskill scan noww
16:42.05Ressyok. I opened my Enchanting, hit the red Scan button, and up popped the ARL screen
16:42.44RessyScan profession works as well.
16:58.03Ressyimported localization strings
17:03.22RessyI guess the next non-alpha version goes live on Tuesday next week?
17:03.39pompythat or wednesday
17:03.59Ressyk
17:12.05pompyok, FFS, nothing shows trying to do a tradeskill scan on my elvel 85 premade
17:16.19Torhal|workWhat professions does it have?
17:17.12Ressyisn't everything still commented out?
17:17.39Torhal|workOh FFS, I forgot to remove my authenticator before I fucking upgraded my iOS
17:17.52Ressyeek
17:17.59pompyRessy: um no? i reverted the commit, remember
17:18.10pompyTorhal|work: they have enchanting and jc
17:20.35Torhal|workI'm going to have to go home.
17:20.36Torhal|workMeh.
17:22.04pompylive servers are up
17:22.10pompyand i logged in and scanned and it worked
17:22.19pompyso its def something to do with the beta/premade char
17:25.46Ressyso you're just opening enchanting, and scanning your own profession?
17:32.03pompyyes..
17:32.28pompyeither of the two profs
17:33.07Ressyand the screen doesn't pop up at all?
17:33.11pompynop
17:33.16Ressyweird...
17:33.38Torhal|workpompy: /eventtrace
17:33.42Ressymissing any libraries?
17:33.44Ressyor that.
17:33.59pompy;look for what torhal
17:35.44Ressybunch of TRADE_SKILL_UPDATE, and UNIT_SPELLCAST_SUCCEEDED
17:36.31pompybe back in a bit.. going to the doctors
21:05.32*** join/#arl purl (~purl@rikers.org)
21:05.32*** topic/#arl is Ackis Recipe List 2.4 released. Get it http://bit.ly/dLEgc or http://bit.ly/39MrIW | ARL Stats! Claim you commits: http://bit.ly/4ABNbD | WoWInsider review: http://bit.ly/HwKXO | Try out Collectinator for your pet/mount needs: http://bit.ly/2Jdiez
21:06.19Ackis~lart pompy
21:06.19purldrops a humongous exploding nuke on pompy
21:12.23pompytorhal: is local DUMP_FUNCTION_FORMATS = { missing A.REPUTATION for a reason?
21:13.09RessyHA! Wow's site is showing drop locals for the new patterns!
21:15.58pompyhuh?
21:16.06Ressyhttp://us.battle.net/wow/en/item/86382#tab=dropCreatures
21:16.36RessySo anything that's not a vendor, we know where it drops now. :D
21:17.08pompyso its actual accurate info?
21:17.16RessyIts straight from Blizzard themselves.
21:17.30pompybut its blizzard!
21:17.32pompynice.
21:18.08Ressylol
21:18.20Ressyarg
21:18.26RessyWhat do I name the custom...
21:18.32RessyINTRO MISTS HEROIC RAIDS
21:18.33Ressy?
21:18.49Ressycan drop from any mob in heroic raids
21:20.52RessyL["INTRO_MISTS_HEROIC_RAIDS"] = "Recipes drop randomly from any mob in Heroic Heart of Fear, Terrace of Endless Spring and Mogu'shan Vaults ."
21:22.04pompyUm
21:22.18Ressyrename?
21:22.39pompyis there a custom string for cata dungeons?
21:22.40pompyhm
21:23.22RessyL["HYJAL_RANDOM"] = "Random drop from Hyjal Summit trash/bosses."
21:23.57RessyL["INTRO_MISTS_HEROIC_RAIDS"] = "Random drop from Heroic Heart of Fear, Terrace of Endless Spring and Mogu'shan Vaults trash/bosses."
21:24.03pompythat BC
21:25.17Ressyrecipe:AddCustom("FIRELANDS_RANDOM")
21:25.26RessyL["FIRELANDS_RANDOM"] = "Random drop off of Firelands bosses/trash."
21:28.54pompyL["MOP_RAIDS_RANDOM"] = "Random drop off of mobs in Heroic Heart of Fear, Terrace of Endless Spring and Mogu'shan Vaults."
21:29.14Ressyk
21:29.18RessyI'll add that instead then
21:30.10Ressydone
21:30.24RessyNow.. I just gotta figure out why I can't dump tailoring
21:30.29Ressyor even scan it
21:31.52Ressy13x AckisRecipeList-@project-version@\Recipe.lua:457: table index is nil
21:31.52RessyAckisRecipeList-@project-version@\Recipe.lua:457: in function "?"
21:32.05pompyIM FIXING IT
21:32.08pompycaps
21:32.11Ressyoh
21:32.12Ressyawesome
21:32.21pompywtf, tina fellclaw is a faction?
21:32.28Ressyo.O
21:32.29pompymudclaw*
21:32.33pompytina mudclaw
21:33.06Ressyohh.. yeah
21:33.11pompyhttp://mop.wowhead.com/faction=1280
21:33.12pompylol
21:33.22Ressydoesn't give anything tho
21:33.36pompya pet
21:33.38pompyunrelated to arl
21:33.42pompyjust colledctinator
21:33.50pompybut yeah dont think all these ar prof related
21:33.54pompyprolly will refine later
21:34.00Ackisthey added a new type of rep grind in MoP, "friends"
21:34.08pompyintesreting
21:34.16Torhal|workpompy: No, it's missing because I'm a total failure.
21:34.22Ressyo.O
21:34.24pompyand yay for convert to uppercase in np++ .. ctrl+shift+u
21:34.29pompyekk Torhal|work
21:34.35pompywell glad i pointed it out then
21:34.48pompyit may be missing in a few other places in that file as well.
21:34.50pompyunsure though
21:34.51RessyI'm just glad someone's gonna fix it, cuz I sure as hell can't yet. heh
21:34.53Ressy<3
21:34.55Torhal|workif acquire_type == A.REPUTATION then
21:34.55Torhal|workfor rep_id, rep_info in pairs(acquire_info) do
21:34.55Torhal|worklocal faction_string = private.FACTION_STRINGS[rep_id]
21:34.57Torhal|workif faction_string then
21:34.57Torhal|workfaction_string = ("FAC.%s"):format(faction_string)
21:34.57Torhal|workelse
21:34.59Torhal|workfaction_string = rep_id
21:34.59Torhal|workaddon:Printf("Recipe %d (%s) - no string for faction %d", self.spell_id, self.name, rep_id)
21:34.59Torhal|workend
21:35.03pompyyo man
21:35.06Torhal|workIt isn't simple, hence it's non-inclusion.
21:35.18Torhal|workAll the others follow the same format
21:35.33pompyif you didnt add it on purpose, why you saying total failure?
21:36.40Ressycan I push the tailoring changes?
21:36.47Torhal|workBecause you have this tendency to "find" shit in the code that is pure imagination because you don't look for the other information. If you'd have looked for DUMP_FUNCTION_FORMATS you'd have found where A.REPUTATION was (and A.VENDOR, which you didn't mention)
21:38.54AckisRessy: yeah it won't affect files Torhal|work is working on if any (or me)
21:38.57pompyRessy: PUSHED
21:39.00pompythat should do i hope
21:39.06pompybe back in 20
21:39.35Ressyum, worse worse
21:39.52Ackishehehe my inscription skill is 420
21:40.25Ressy43 errors now after pompy's pull
21:40.41Ressy2x AckisRecipeList-@project-version@\Database\Vendor.lua:28: Usage: GetAddon(name): "name" - Cannot find an AceAddon "nil".
21:40.41Ressy<in C code>
21:40.57Torhal|workHold on
21:40.59Ressy2x AckisRecipeList-@project-version@\Database\Seasonal.lua:30: Usage: GetAddon(name): "name" - Cannot find an AceAddon "nil".
21:40.59Ressy<in C code>
21:41.11Ackisyou got this Torhal|work or want me to?
21:41.24Torhal|workI want to be even more pissed than I already am, so I'll do it.
21:41.24Torhal|work:D
21:41.43Ressyo.O
21:45.11Torhal|workPushed.
21:45.42Torhal|workNot fixed though
21:45.55Torhal|workWHat were you doing to get the error?
21:45.57Ressynope.
21:46.10RessyOpened my profession (tailoring), and hit the scan button
21:46.18Torhal|workI just fixed the last commit pompy did, I didn't even look at the error messages
21:46.19Torhal|workHah
21:46.20Torhal|workOk
21:49.12*** part/#arl Ressy (~Ressy@unaffiliated/ressy)
21:49.17*** join/#arl Ressy (~Ressy@unaffiliated/ressy)
21:49.19Ressyoops
21:49.40Torhal|workErm, was gonna say - works fine for me (once I updated the babble libs)
21:49.50Ressyoh?
21:50.01Torhal|workAt least for Alchemy
21:50.58Ressytry tailoring
21:51.10Torhal|workI need to get in a guild to do that, or get on Live.
21:51.17Torhal|workOh, nevermind - I copied my tailor.
21:52.12Torhal|workWorks fine.
21:52.30Ressyarg. so its something I did in my update
21:53.40Torhal|workHow many errors and what are they?
21:55.00Ressywhen I open my tailoring profession, and hit the red scan button, I get this
21:55.01Ressy15x AckisRecipeList-@project-version@\Recipe.lua:457: table index is nil
21:55.01RessyAckisRecipeList-@project-version@\Recipe.lua:457: in function "?"
21:55.01RessyAckisRecipeList-@project-
21:55.21Torhal|workThat's in BugSack?
21:55.25Ressyyeah
21:55.44Ressy457 acquire[faction_id] = {}
21:56.02Ressyso.. what faction_id am I screwing up
21:56.12Torhal|workRessy: Go to line 449 and insert this:
21:56.36Torhal|workprint(("Recipe: %s (%d)"):format(self.name, self.spell_id))
21:56.40Ressyohh
21:56.57Ressygod I love debug msgs
21:57.05Torhal|work:D
21:57.38RessyIts every single rep vendor recipe.
21:57.44Torhal|workNono
21:57.52Torhal|workThe one with the issue is the last one to print.
21:58.06Torhal|workOh
22:00.11Torhal|workShow me your AddRepVendor line or just paste the whole recipe
22:00.17Ressy-- Spelltwister's Gloves -- 125548
22:00.17Ressyrecipe = AddRecipe(125548, V.MOP, Q.COMMON)
22:00.17Ressyrecipe:SetRecipeItemID(86369)
22:00.17Ressyrecipe:SetCraftedItemID(82438)
22:00.17Ressyrecipe:SetSkillLevels(600, 600, 600, 600, 600)
22:00.18Ressyrecipe:SetItemFilterType("TAILORING_HANDS")
22:00.18Ressyrecipe:AddFilters(F.VENDOR, F.REPUTATION, F.GOLDENLOTUS)
22:00.19Ressyrecipe:AddRepVendor(FAC.GOLDENLOTUS, REP.HONORED, 59908)
22:01.46RessyFaction strings was updated again
22:01.50Ressyarg
22:02.41Torhal|workHmm?
22:03.03RessyIts the faction string
22:03.13Torhal|workYes I know
22:03.16Torhal|workI just fixed it
22:03.16RessyI had a wrong one in..
22:03.19Torhal|workPompy had a fucking underscore
22:03.24RessyThen you & pompy went back and forth with another one.
22:03.31RessyI got it now I think
22:03.42Torhal|workNo
22:03.45Torhal|workYou don't :)
22:05.38RessyI found the other problems.. Hopefully I found'em all.
22:06.15Torhal|workPushed
22:06.28Ressyty.
22:06.31Ressyadded the filter?
22:06.40Torhal|worko.O
22:06.41Ressyerr, 2?
22:06.52Torhal|workWhat did you do?
22:06.53Torhal|worko.O
22:06.53RessyAugust Celestials & Golden Lotus?
22:06.58Ressyme?
22:07.04Torhal|workYEs.
22:07.08Torhal|workYou said you found the problem
22:07.12Torhal|workSo what did you do? :)
22:07.34RessyRecipe 'Spelltwister's Grand Robe' (spell ID 125547): Attempting to assign non-existent filter flag.
22:08.03RessyF.GOLDEN_LOTUS
22:08.04Torhal|workOk, I'm confused. That's not the error you had befiore.
22:08.08Torhal|workNo
22:08.10Torhal|workDon't do that
22:08.18Torhal|workpompy fucked up twice when he added those
22:08.21Torhal|workYou were right before
22:08.22Ressyo.O
22:08.32Torhal|workI fixed the strings.
22:08.37RessyI'm just following previous reputations, like thorium brotherhood.. they had their own filters.
22:08.38Ressyoh
22:08.39Torhal|workGOLDENLOTUS was correct
22:08.40Ressyso..
22:08.42Ressyk
22:08.47pompyseriously torhal
22:09.46Torhal|workYou don't know what you're doing? Don't fucking touch it. Seriously.
22:09.47pompysorry i missed a fucking comma
22:09.58Torhal|workNo, you added commas where they had no business being.
22:10.01pompyi was in a rush sorrry.. i forgot to remove a comma
22:10.06Torhal|workAnd you arbitrarily named factions.
22:10.08pompyI DIDNT ADD i forgot to remove it
22:10.11pompyit was copy and paste
22:10.14Torhal|workYes.
22:10.45pompywhat? i fucking c/p what ackis added and turned them into the faction_strings
22:10.51pompywtf
22:11.43Torhal|workThe faction strings have to match the filter flags.
22:11.49Torhal|workAnd they never have spaces in them.
22:12.17Ressywhere can I find the filter flags?
22:12.24pompyi didnt add a space
22:12.25Torhal|workConstants.lua
22:12.26pompyanywhere
22:12.31Ressycuz..
22:12.36RessyAugust celesetials might be a problem
22:12.40Torhal|work[1271], "ORDER OF THE CLOUD SERPENT",
22:12.41RessyAUGUSTCELESTIALS= 0X00004000,-- 15
22:12.59Torhal|work[1269], "GOLDEN_LOTUS",
22:13.00Ressy[1341]=  "THE_AUGUST_CELESTIALS",
22:13.03Torhal|workThe filter flag is GOLDENLOTUS
22:13.57Torhal|workChange it to AUGUSTCELESTIALS
22:14.16Torhal|workAckis names shit weird, but the faction names ocrrespond directly to the filter flags.
22:14.27Torhal|workSo when they aren't right, shit goes boom.
22:14.31Ressylorewalkers vs the lorewalkers
22:14.43Torhal|workI only fixed what I saw.
22:14.50Torhal|workSo fix anything else that's incorrect
22:15.04Torhal|workI'm going home.
22:15.04Ressyk. I wasn't sure that was even a problem. lol
22:15.10Torhal|workRessy:
22:15.21Torhal|workWell, if it's in private.REP_FLAGS_WORD2 = {
22:15.31Torhal|workAnd it's different there than in the otehr table
22:15.35Torhal|workFix the other table.
22:17.14*** part/#arl pompy (~Mike@68.36.135.0)
22:18.40*** mode/#arl [+v Ressy] by ChanServ
22:18.59Ressyarg
22:19.03Ressystill coming up with this.
22:19.05RessyImma take a break
22:19.45Ressy3x AckisRecipeList-@project-version@\Database\Recipes\Tailoring.lua:4455: attempt to call method "?" (a nil value)
22:20.23RessyThats.. odd
22:20.29Torhal|workHrm?
22:20.31RessyIt doesn't like the createditem id
22:20.38Torhal|worko.O
22:20.51Torhal|workWhat's line 4455?
22:21.04Ressywas recipe:SetCreatedItemID82444
22:21.06Ressyerr
22:21.08Torhal|worklol
22:21.09Ressywiwth brackets
22:21.12Ressyblah
22:21.14Ressyfixing
22:21.17Torhal|workCraftedItemID
22:21.23Ressyyeah
22:21.24Ressyrealized
22:22.14Ressyblonde day
22:25.11Ressyhooray. I tracked down and fixed the million and one problems. >.<
22:25.41RessyI didn't mean to cause problems with pompy. :\
22:25.48Ackisno it's not your fault
22:26.08Ackishe's off the projects now however
22:26.17Ackisso hopefully you stay on :P
22:28.13Ressypushing
22:28.32AckisI just did a ToC update
22:28.43AckisI put my name in front of Torhal's kekekek
22:29.21Ressy:P
22:29.46RessyYou pulled him off? why?
22:30.21Ackishe pulled himself off and I'm not playing daddy/babysitter, I told him a week ago or so that he's an adult and he needs to make his choice, he made it
22:30.44Ressyomg
22:32.31Ackis?
22:33.39Ressysurprised as hell
22:33.51Ackisthis has been a long time coming
23:07.06RessyTorhal...
23:07.15Ackishe's afk
23:07.16RessyThat line you had me add to Recipes.lua
23:07.17Ressyoh
23:07.18Ackisdriving home or something
23:07.20Ressyits awesome
23:07.27Ressyit prints out every single mistake that there is.
23:07.37Ressyso I can just go through it, and fix it all in a single reload.
23:07.47Ressy<33333
23:07.58Ressyless of me going "uh, why am I broken now!"
23:25.16AckisI am in a foul mood
23:25.28Ressy:(
23:25.39Ressyhow come?
23:26.03Ackispompy bullshit
23:26.10AckisI'm off work on stress leave and I get to deal with this
23:26.19Ressy:\
23:28.18*** join/#arl Torhal (~torhal@c-71-207-224-241.hsd1.al.comcast.net)
23:29.12*** mode/#arl [+o Torhal] by ChanServ
23:52.15AckisRessy: I told him a bout your compliment
23:52.20Ackishe started blushing like a new bride
23:52.37TorhalAnd then I pissed in Ackis's shoes.
23:52.56Ressylol
23:53.04RessyBut yeah, that line is just amazing.
23:53.37Ressytells me exactly what the problem is
23:54.15Torhal:D
23:55.40Ressynow, how do I get a custom to show up in 'miscellaneous'
23:56.11TorhalRessy: Example?
23:57.20Ressyarg.
23:57.26RessyAddCustom("FIRELANDS_RANDOM", Z.FIRELANDS)
23:57.33Ressydoes AddCustom accept multiple zones?
23:58.27TorhalNo, but I can change that.
23:58.39Ressyoh?
23:58.45TorhalSure.
23:59.37Ressylooks like it already takes zone & coords
23:59.46TorhalJust one.
23:59.47RessyAddCustom("DM_TRIBUTE", Z.DIRE_MAUL, 59.04, 48.82)
23:59.54TorhalOne of each.

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