00:23.59 | Torhal | Lovely. |
00:24.36 | Ressy | I've got the spell numbers added to core & scanner |
00:24.41 | Ressy | I just have no idea how to make it work. |
00:24.46 | Ressy | You want me to push or no? |
00:26.43 | Torhal | Scanner? |
00:26.45 | Torhal | Huh? |
00:26.48 | Torhal | WHat you do? |
00:26.50 | Torhal | o.O |
00:27.00 | Ressy | In my copy only.. |
00:27.05 | Ressy | haven't committed or pushed. |
00:27.19 | Ressy | I added the spell ids for the new 'way of the' specialties |
00:27.36 | Ressy | I have no clue how to get it working, so i'm not screwing with that. |
00:27.36 | Torhal | They're actually specialties as in - you can only know one at a time? |
00:27.40 | Ressy | no. |
00:27.44 | Ressy | You can know all of'em. |
00:27.46 | Torhal | Then they're not speialties. |
00:27.53 | Torhal | And we don't care about them. |
00:28.11 | Ressy | o.O |
00:28.12 | Torhal | With Alchemy you can only know Elixir or Potion mastery. |
00:28.15 | Ressy | I can't scan'em. |
00:28.22 | Torhal | But we don't care because all that does is proc. |
00:28.30 | Torhal | Can you scan 'em now? |
00:28.33 | Ressy | no |
00:28.35 | Torhal | After you did whatever you did? |
00:28.37 | Ressy | no |
00:28.59 | Torhal | Then they're just spells you need to learn in order to learn others, from what I'm reading |
00:29.26 | Torhal | Hrm. |
00:29.32 | Ressy | way of the.. is a spell like cooking is. |
00:30.23 | Torhal | So are all the recipes, so I'm not convinced these should be treated as specialties (especially since you can learn them all) |
00:31.13 | Torhal | logs into Beta |
00:39.56 | Torhal | Ressy: I can load smelting just fine... |
00:41.54 | Ressy | even with auto scan trainer turned on.. |
00:42.12 | Ressy | I now need to do a full smelting scan before I can do a trainer scan for smelting |
00:42.20 | Ressy | which is totally opposite of the other professions |
00:50.38 | Torhal | Shit shit shit shit shit shit |
00:52.02 | Torhal | All of them are completely fucked. |
00:52.12 | Torhal | I'll have to hardcode the NPC IDs. |
00:52.36 | Torhal | GetTradeSkillLine() is supposed to return the profession name and your current profession level |
00:52.54 | Torhal | For all of them at The Halfhill Market, it returns "UNKNOWN, 0" |
00:54.20 | Torhal | Nevermind, I'm losing it. |
00:55.58 | Torhal | Ok, all I need to do is add a mapping to Cooking (like I do for Smelting to Mining) |
01:01.19 | Torhal | Well, not really. It's more involved. |
01:01.23 | Ressy | >.< |
01:02.01 | Ressy | I really need to learn lua. |
01:02.05 | Torhal | Basically, /dump GetTrainerServiceSkillLine(1) should return "Cooking" but it returns "Way of The Grill" etc |
01:02.11 | Ressy | ah |
01:02.15 | Torhal | So I need to map all of these TO cooking. |
01:03.22 | Ressy | thats... gonna suck |
01:04.00 | Ressy | question: How do I get map # Pandaria zones need to be mapped in one of the main .lua files |
01:06.10 | Torhal | o.O |
01:06.14 | Torhal | Oh |
01:06.44 | Torhal | <PROTECTED> |
01:07.09 | Torhal | Case doesn't matter |
01:07.20 | Torhal | "oRgRiMmAr" will work, for example |
01:09.07 | Ressy | 2 map ids for jade forest.... |
01:09.08 | Ressy | >.< |
01:09.37 | Ressy | 1 old, 1 new? |
01:09.50 | Torhal | Just pick one. All we care about is the name being localized. |
01:10.16 | Ressy | ah. k |
01:11.03 | Torhal | Ok, do you have a list of all of the "Way of The X"? |
01:13.54 | Torhal | Nevermind - got 'em |
01:13.54 | Ressy | Brew: Spell 125589 |
01:13.56 | Ressy | k |
01:13.58 | Ressy | heh |
01:20.50 | Ressy | Zone name for Kun-Lai Summit, isn't "Kun-Lai Summit" "Kunlai summit" "Kunlai" |
01:20.52 | Ressy | .... |
01:20.59 | Torhal | Ok, pushed. |
01:21.32 | Ressy | awesome |
01:21.33 | Ressy | ty |
01:22.15 | Torhal | That push also allows you to auto-ad trainers to the localization file. |
01:22.22 | Torhal | <PROTECTED> |
01:22.45 | Ressy | nice |
01:22.46 | Torhal | There I go again not typing the second letter. add* |
01:23.35 | Ressy | Thoughts on how to get zone name. :\ |
01:24.22 | Torhal | Sure. |
01:24.27 | Torhal | Just use the word "summit" |
01:25.05 | Torhal | The reason for multiple zone maps is likely scenerios. |
01:25.10 | Ressy | oh wow... |
01:25.14 | Ressy | thats awesome |
01:25.15 | Ressy | ty |
01:25.21 | Torhal | :D |
01:27.16 | Ressy | Scanner's not working. |
01:27.26 | Torhal | Hmm? |
01:27.44 | Ressy | http://www.wowace.com/paste/6257/ |
01:28.21 | Torhal | What the actual fuck? |
01:29.16 | Torhal | What did you do to get that? |
01:29.35 | Ressy | err... |
01:29.39 | Ressy | lemme check something |
01:30.16 | Ressy | ffs. nm. |
01:30.27 | Torhal | mumbles something about local modifications to Constants.lua |
01:30.29 | Torhal | :D |
01:31.13 | Ressy | I forgot to copy over Constants.lua to my beta install |
01:31.20 | Ressy | arg |
01:31.22 | Ressy | sry |
01:31.26 | Torhal | Bwaha! |
01:32.21 | Torhal | So in this case it was a non-modification that did it. |
01:32.24 | Torhal | Hehe. |
01:33.55 | Ressy | ya |
01:34.11 | Ressy | I'm just copying my updated files to live, then i'mma kill my beta copy, and recopy the entire thing over. |
01:34.42 | Ressy | only.... my live cooking doesn't work |
01:34.49 | Torhal | Huh? |
01:35.09 | Ressy | Ackis Recipe List: nil profession passed to InitializeProfession() |
01:35.10 | Ressy | Ackis Recipe List: Recipe database not loaded. Please scan the tradeskill first then try the datamining. |
01:35.42 | Torhal | Let's ping Ackis some more. |
01:35.43 | Torhal | :D |
01:35.55 | Ressy | lol |
01:36.23 | Ressy | alchemy does the same "nil" line |
01:36.50 | Torhal | You have weird shit going on. |
01:36.55 | Ressy | nods |
01:36.56 | Torhal | Mine works just fine (on live) |
01:37.00 | Ressy | blah |
01:37.10 | Torhal | Then again, I have the same exact files for live and Beta |
01:37.16 | Torhal | Symbolic linking FTW. |
01:37.23 | Ressy | I killed my beta install. |
01:37.35 | Torhal | Killed? |
01:37.47 | Ressy | deleted |
01:37.58 | Torhal | Before testing on live? The madness! |
01:38.06 | Ressy | I hadn't done much. |
01:38.10 | Torhal | Seriously, though - hope you didn't lose anything. |
01:38.20 | Ressy | map numbers, 1 recipe in, a few vendors. *shrugs* |
01:38.40 | Ressy | Imma reimport the entire thing. |
01:43.35 | Torhal | Oh, no wonder I don't have trainers auto-add recipes - all that's available is the name. |
01:44.32 | Ressy | You actually also pull the npc id... |
01:44.42 | Ressy | and you COULD pull the map name. |
01:45.01 | Ressy | for coords, you can hit a script... |
01:45.08 | Torhal | Indeed. |
01:45.16 | Ressy | wowhead uses /wl loc for example.. |
01:45.25 | Ressy | all that'd be left is alliance/neutral/horde |
01:45.27 | Torhal | Though this was all added before the LibBabbleZone deprecation and I didn't think to add it there. |
01:45.30 | Ressy | but.. That'd make us lazy |
01:45.43 | Torhal | I already grab coords |
01:45.48 | Ressy | true |
01:50.49 | pompy1 | it auto adds trainers to the loc phrases, but it doesnt for vendors? |
01:52.15 | Torhal | I'll look. |
01:54.40 | pompy1 | so how well and how exactly does the switch command in git work. |
01:58.00 | Torhal | Huh? |
02:01.40 | Ressy | ok.. I recloned the entire folder, after deleting everything in it. |
02:01.45 | Ressy | and I'm getting this error still |
02:03.15 | Ressy | cooking & alchemy trainers. |
02:03.22 | Torhal | Ressy: You're fine. My bad. |
02:03.30 | Torhal | I have no idea why I even thought that code would work. |
02:03.39 | Ressy | So I'm not goign crazy? lol |
02:03.49 | Ressy | I was gonna kick my comp. |
02:03.55 | Ressy | lol |
02:04.52 | Torhal | Pushed fix. |
02:06.27 | Ressy | awesome. ty :) |
02:08.27 | Ressy | My friend is so distracting. |
02:09.56 | Ressy | thank god she logged. |
02:10.13 | pompy1 | torhal: the "git switch" command |
02:13.20 | Torhal | pompy1: I think you mean the checkout command. |
02:13.38 | Torhal | If you're referring to branches. |
02:14.13 | pompy1 | they are sorta similar but not |
02:14.15 | pompy1 | its odd |
02:14.23 | Torhal | pompy1: If that's the case, read this - http://git-scm.com/book/en/Git-Branching-Basic-Branching-and-Merging |
02:14.29 | pompy1 | http://www.syntevo.com/smartgit/documentation.html?page=commands-branch#switch |
02:14.40 | Torhal | Yeah, non-standard |
02:14.59 | Torhal | sorrowthorn:AckisRecipeList callahan$ git switch |
02:14.59 | Torhal | git: 'switch' is not a git command. See 'git --help'. |
02:15.46 | pompy1 | hm, "Use this command to switch your working tree to a different branch." |
02:17.32 | Torhal | Sounds like "git checkout <branch name>" |
02:18.11 | pompy1 | yeah probably.. also in the smartgit docs .. it says |
02:18.13 | pompy1 | Check 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.59 | pompy1 | bc i dont want things to get dirty, but i want the coll 2.0 branch |
02:20.42 | pompy1 | also, Torhal, this is the dialog for it in my client - https://dl.dropbox.com/u/35894760/gitswitch.png |
02:22.16 | Torhal | Looks to me like it's an "Oops! I fucked up! I meant for these changes to be on this branch!" |
02:22.19 | Torhal | Don't do that. |
02:22.33 | Torhal | It will then attempt to merge (read what it says) |
02:30.40 | pompy1 | thats 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.41 | Torhal | Yo: http://o7.no/S1f1Kg |
02:41.55 | Ressy | whoa |
02:42.20 | Ressy | that.. totally.. rocks. |
02:42.40 | Torhal | Instead of doing it from the trainer, I do it from the scan itself. |
02:42.58 | Torhal | The TradeSkillFrame has ALL The profession data - so the spell ID is available to me. |
02:44.00 | Ressy | when you committing this change? :D |
02:44.32 | Torhal | As soon as it works with the profession dump |
02:44.41 | Ressy | oh. lol |
02:45.13 | pompy1 | lol you are weird torhal. |
02:45.22 | Torhal | …why? |
02:45.24 | pompy1 | jk |
02:45.24 | Ressy | get our hopes up why don't cha. :P |
02:45.39 | pompy1 | pretty meat |
02:45.41 | pompy1 | neat |
02:45.46 | Ressy | very |
02:46.29 | Torhal | http://o7.no/S1fo7x |
02:46.51 | Torhal | Need to fix though |
02:46.57 | Torhal | V.CATA be wrong, y0. |
02:47.43 | Ressy | default = mop? |
02:49.28 | Torhal | No: _G.GetExpansionLevel() + 1 |
02:49.37 | Torhal | (0 is Classic) |
02:49.48 | Torhal | That's why it said V.CATA instead of V.MOP |
02:49.59 | Torhal | I didn't have the +1 at first |
02:50.30 | Ackis | I think I tore a muscle in my arm :'( |
02:50.39 | pompy1 | eh in a week you can remove the +1 |
02:50.42 | pompy1 | probably |
02:51.10 | Ressy | nice |
02:52.13 | Torhal | pompy1: No I can't |
02:52.45 | Torhal | Lua arrays start at 1 instead of 0 (which is what most languages use) |
02:52.52 | Torhal | The expansion levels start at 0 |
02:52.56 | Torhal | So I ALWAYS have to add 1 |
02:53.18 | Ackis | adds 2 |
02:53.20 | pompy1 | oh |
02:53.35 | pompy1 | you are comparing that lookup with the number hte gameversion is associated with |
02:53.37 | pompy1 | i guess |
02:53.59 | Ackis | comments his adding 2 with -- Add 2 |
03:06.08 | pompy1 | Ackis: max level? |
03:12.27 | pompy1 | BL2 unlocking in 1 hour on Steam.. guess they are going by Midnight Eastern for the US .. odd, but cool |
03:12.35 | pompy1 | even though i will not be playing it right at midnight lol. |
03:12.36 | pompy1 | >< |
03:32.07 | Torhal | pompy1, Ressy: Yo. |
03:32.09 | Torhal | Just pushed. |
03:32.22 | Torhal | What you do is scan the profession first. Then scan the trainer. |
03:32.31 | Torhal | Kerplow. |
03:32.40 | Ressy | woots |
03:32.52 | Torhal | Dump the profession, and if applicable dump the trainer and phrase |
03:33.11 | Torhal | Ackis: You're a bastard. |
03:33.26 | Torhal | Fuck you for asking me to add LibQTip to ARL all those years ago. |
03:33.31 | Torhal | :D |
03:38.46 | Ackis | :) |
03:41.02 | Torhal | Also, 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.05 | Torhal | They'll just get fixed. |
03:41.55 | Ressy | cool |
03:42.08 | pompy1 | theyve alweays been fixed |
03:42.36 | Torhal | pompy1: No, there were cases where it would say unconfirmed. |
03:43.03 | pompy1 | Ackis: max level? |
03:43.31 | Ackis | pompy1: nein, level 13ish only |
03:44.00 | Ackis | going back to being afk |
03:44.09 | Torhal | In fact, the whole logic for the trainer scane was lolwut? |
03:44.13 | Torhal | Ackis: Fucker. |
03:44.21 | Torhal | s/scane/scans/ |
04:35.12 | pompy1 | oh really |
04:35.36 | Torhal | Sure. |
04:35.39 | Torhal | Why not? |
04:36.02 | pompy1 | referring to you possibly uncovering the ????????? cyrillic russian issue |
04:37.43 | Torhal | Whenever someone pastes CYrillic into the WoWAce forums, all anyone sees is ???? |
04:45.25 | pompy1 | not talking about the forums lol |
04:45.52 | Ressy | headed to bed.. night |
04:46.38 | Torhal | I know that |
04:46.58 | Torhal | But 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.02 | Torhal | :P |
04:51.54 | pompy1 | ok |
05:10.18 | pompy1 | I 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.27 | pompy1 | Ok, 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.30 | pompy1 | I'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.34 | Ressy | pompy1: Is it erroring in wow live leaving in mop trainer/recipes? Is that why you're commenting everything out? |
12:52.10 | Ressy | Just 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.24 | Ressy | I'm really surprised at the number of people who run alpha versions of arl |
13:26.52 | Torhal | Ressy: It will show "Unknown" and the spell ID. |
13:27.09 | Torhal | But hey - it's a fucking Alpha. |
13:27.30 | Torhal | Go ahead and re-add it, since you can't do a fucking dump otherwise. |
13:27.39 | Torhal | grumps |
13:54.26 | pompy | Torhal ahh sooo |
13:54.58 | pompy | i 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.48 | pompy | and you cant get the item bind from scanning the tt? and the role flags?! |
13:57.03 | Torhal | pompy: 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.17 | Torhal | I'll look at it after work. |
13:57.24 | Torhal | Speaking of - time to get ready. |
13:57.27 | pompy | ill rescan the trainer and see if it adds more |
13:57.28 | pompy | kk |
14:02.40 | Ressy | Torhal: |
14:03.02 | Ressy | You know how you changed Trainers to use Z.<whatever> instead of "whatever"... |
14:03.09 | Ressy | I did a dump, updated, /rl'd |
14:03.15 | Ressy | and on scan, get an error |
14:03.26 | Ressy | 1x Database\Trainers\Alchemy.lua:10: attempt to index global "Z" (a nil value) |
14:03.26 | Ressy | Database\Trainers\Alchemy.lua:10: in function "InitializeAlchemyTrainers" |
14:25.58 | Torhal | Ressy: I told you what to add :) |
14:26.19 | Torhal | local Z = private.ZONE_NAMES |
14:26.42 | Torhal | Directly under: local FOLDER_NAME, private = ... |
14:28.01 | Ressy | okies |
15:08.46 | Torhal|work | Ressy: Working now? |
15:08.59 | Ressy | yes. |
15:09.01 | Ressy | as is pompy |
15:09.04 | Ressy | we're working on diff things. |
15:09.13 | Ressy | He's working on recipes, I'm working on updating trainers |
15:09.13 | Torhal|work | Coordination. I like it. |
15:43.48 | pompy | Torhal|work: i made a fresh 85 premade but the arl window wont show when i sacn my p[ersonal professions |
15:43.57 | pompy | ressy created one and they opened for her apparentl;y |
15:44.03 | pompy | but it cant be anmything on my end |
15:44.49 | Ressy | not like the dump window, right? |
15:44.58 | pompy | :O |
15:45.00 | pompy | no |
15:45.11 | Ressy | ok. |
15:45.13 | pompy | personmalprofes! the scan using the sacn button....... |
15:45.31 | pompy | damnit torhal, i hate how "scan" means 2 diff things.. you should havea created separate terminology |
15:46.03 | pompy | personal professions** |
15:46.18 | pompy | that is what you tried on your premade? |
15:47.27 | Ressy | the lil scan button on your profession. |
15:48.05 | Torhal|work | TradeSkill scan versus Profession Scan. |
15:48.16 | Torhal|work | The latter being the copy/pasta window. |
15:48.33 | Torhal|work | But yah, it's fucky. |
15:48.34 | pompy | yeah profession scan is used pretty much for your personal ones.. blehhh haha |
15:48.36 | pompy | yup |
15:48.56 | pompy | but yeah, idk why an arl scan window wont come up doing a TRADESKILL SCAN ! lol <> |
15:48.57 | Torhal|work | No, TradeSkill Scan is when you hit the Scan button on the TradeSkillFrame |
15:49.03 | Torhal|work | Oh |
15:49.11 | pompy | the phrase, not the meaning |
15:49.11 | pompy | haha |
15:49.12 | Torhal|work | Dunno. |
15:49.47 | Torhal|work | I'd be able to fix it if I could reproduce |
15:49.49 | Torhal|work | But I can't. |
15:51.06 | pompy | youre logge dinto the beta? |
15:51.10 | Ressy | When I go to drump trainers, I get a bug sometimes. |
15:51.32 | Ressy | I have to manually add trainers |
15:51.35 | Ressy | err, dump. |
15:52.01 | Torhal|work | What's the bug? |
15:52.14 | Torhal|work | bug/error/meteorite shower/ |
15:54.27 | Ressy | 2x AckisRecipeList-@project-version@\Scanner.lua:575: attempt to index field "name" (a boolean value) |
15:54.28 | Ressy | AckisRecipeList-@project-version@\Scanner.lua:575: in function <AckisRecipeList\Scanner.lua:545> |
15:54.28 | Ressy | AckisRecipeList-@project-version@\Scanner.lua:596: in function "DumpTrainers" |
15:54.28 | Ressy | <string>:"ARL:DumpTrainers("jewelcrafting")":1: in main chunk |
15:58.07 | pompy | Ressy: because you forgot to add some trainers when you scanned and dumped them |
15:58.17 | pompy | probably the reason |
15:58.26 | Torhal|work | ? |
15:58.45 | pompy | nothing torhal, we have it under control |
15:59.02 | Torhal|work | You shouldn't have to add trainers is why I asked WTF |
15:59.19 | pompy | she didnt dump the trainer list after scanning the trainer |
15:59.20 | Ressy | I get that when I try to do the dump |
15:59.26 | pompy | yeah |
15:59.39 | pompy | during a new session tho |
15:59.47 | pompy | not during the same one you added the IDs in! |
15:59.58 | Ressy | is /rl new session? |
16:00.00 | pompy | im guessing thats why its complaining |
16:00.02 | Ressy | prolly. |
16:00.05 | pompy | yeah |
16:00.16 | Torhal|work | You reload and all data not in the DB is wiped. |
16:00.23 | pompy | exactly |
16:00.56 | Ressy | ah |
16:00.58 | Ressy | gotcha |
16:01.10 | pompy | Ressy: 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.27 | Ressy | I'm only touching trainers. |
16:01.35 | Torhal|work | Fuck it, push the conflicts too. That'll teach those bastards that insist on using alphas. |
16:01.38 | Ressy | should be ok, but doing as you said. |
16:02.47 | pompy | loll Torhal|work a russian on the beta... character name abbove its head is ??????????? |
16:02.58 | pompy | yet the cyrillic showed fine in the chatframe |
16:03.10 | Torhal|work | Hah |
16:03.18 | Torhal|work | It's teh font |
16:03.25 | Torhal|work | The font used for names doesn't support it. |
16:03.52 | Ressy | 1 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.00 | Torhal|work | Yup |
16:05.49 | pompy | and yeah torhal.. apparently even ScanProfession() wont do the mop trainable recipes. |
16:06.02 | pompy | its not adding the missing bind data, or the role flags |
16:06.04 | pompy | herm |
16:06.33 | pompy | on beta fyi |
16:07.23 | Ressy | committed |
16:07.31 | Ressy | just the changes to Z.<zonename> |
16:09.13 | pompy | well 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.22 | Ressy | I have committed. |
16:09.23 | Ressy | :) |
16:09.30 | Ressy | haven't pushed |
16:09.58 | pompy | ni gentlepaw is a trainer and a vendor? |
16:10.37 | Ressy | just vendor |
16:10.42 | pompy | :O |
16:10.48 | Ressy | i think |
16:10.50 | Ressy | lemme check |
16:10.52 | pompy | nah' |
16:10.58 | pompy | his id is on addTrainer() functions |
16:11.02 | pompy | to recipes |
16:11.18 | Ressy | yes, both |
16:11.30 | Ressy | err, just trainer |
16:11.55 | Ressy | staring at her offerings now in-game |
16:11.57 | pompy | he's <Alchemy Supplies> |
16:12.00 | pompy | thats a vendor? |
16:12.02 | Ressy | yup |
16:12.05 | Ressy | go figure |
16:12.15 | Ressy | carries crystal vials |
16:12.17 | Ressy | as supplies |
16:12.23 | Ressy | stupidity at its finest |
16:12.28 | pompy | and offers to train recipes? |
16:12.36 | Ressy | yup |
16:12.41 | Ressy | just trainer |
16:12.58 | Ressy | no recipes on it, just vials |
16:15.51 | Ressy | are you gonna push, and I pull, or do you want me to push, and you pull? |
16:16.00 | pompy | ill tell you when |
16:16.02 | Ressy | k |
16:16.17 | Torhal|work | Wow, this isn't rocket surgery. |
16:16.32 | Ressy | lol |
16:16.46 | Torhal|work | Seriously: You shouldn't have to negotiate who does what in order to use Git :) |
16:21.32 | pompy | ok, have fun....... tell me now if you can get the arl window to show up doing a TRADESKILL SCAN |
16:25.28 | Ressy | sob |
16:25.31 | Ressy | merge conflicts |
16:26.08 | pompy | probably just in one of the trainer files |
16:26.12 | Ressy | 3 files |
16:26.20 | Ressy | Vendor & 2 trainer files |
16:26.45 | pompy | you had local changes to vendor file lol? |
16:27.02 | Ressy | I added a vendor. |
16:27.12 | pompy | i didnt touch that file, sooo. |
16:27.21 | pompy | and only touched 1 trainer file |
16:27.40 | Ressy | Vendor, Inscription Trainer, Alchemy trainer |
16:30.37 | pompy | just overwrite those 3 with your changes if you can, not much was changed in my end that i did. |
16:31.28 | pompy | cant remember how to resolve conflicts haha |
16:32.29 | Ressy | so... |
16:32.31 | Ressy | I've resolved. |
16:33.00 | Ressy | do I try to pull again, or just push |
16:33.20 | Ressy | I assume just push |
16:33.42 | pompy | push |
16:33.52 | pompy | i think |
16:33.53 | pompy | ! |
16:39.03 | Ressy | pushing |
16:39.07 | pompy | push that and try to do a tradeskill scan noww |
16:42.05 | Ressy | ok. I opened my Enchanting, hit the red Scan button, and up popped the ARL screen |
16:42.44 | Ressy | Scan profession works as well. |
16:58.03 | Ressy | imported localization strings |
17:03.22 | Ressy | I guess the next non-alpha version goes live on Tuesday next week? |
17:03.39 | pompy | that or wednesday |
17:03.59 | Ressy | k |
17:12.05 | pompy | ok, FFS, nothing shows trying to do a tradeskill scan on my elvel 85 premade |
17:16.19 | Torhal|work | What professions does it have? |
17:17.12 | Ressy | isn't everything still commented out? |
17:17.39 | Torhal|work | Oh FFS, I forgot to remove my authenticator before I fucking upgraded my iOS |
17:17.52 | Ressy | eek |
17:17.59 | pompy | Ressy: um no? i reverted the commit, remember |
17:18.10 | pompy | Torhal|work: they have enchanting and jc |
17:20.35 | Torhal|work | I'm going to have to go home. |
17:20.36 | Torhal|work | Meh. |
17:22.04 | pompy | live servers are up |
17:22.10 | pompy | and i logged in and scanned and it worked |
17:22.19 | pompy | so its def something to do with the beta/premade char |
17:25.46 | Ressy | so you're just opening enchanting, and scanning your own profession? |
17:32.03 | pompy | yes.. |
17:32.28 | pompy | either of the two profs |
17:33.07 | Ressy | and the screen doesn't pop up at all? |
17:33.11 | pompy | nop |
17:33.16 | Ressy | weird... |
17:33.38 | Torhal|work | pompy: /eventtrace |
17:33.42 | Ressy | missing any libraries? |
17:33.44 | Ressy | or that. |
17:33.59 | pompy | ;look for what torhal |
17:35.44 | Ressy | bunch of TRADE_SKILL_UPDATE, and UNIT_SPELLCAST_SUCCEEDED |
17:36.31 | pompy | be 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.19 | Ackis | ~lart pompy |
21:06.19 | purl | drops a humongous exploding nuke on pompy |
21:12.23 | pompy | torhal: is local DUMP_FUNCTION_FORMATS = { missing A.REPUTATION for a reason? |
21:13.09 | Ressy | HA! Wow's site is showing drop locals for the new patterns! |
21:15.58 | pompy | huh? |
21:16.06 | Ressy | http://us.battle.net/wow/en/item/86382#tab=dropCreatures |
21:16.36 | Ressy | So anything that's not a vendor, we know where it drops now. :D |
21:17.08 | pompy | so its actual accurate info? |
21:17.16 | Ressy | Its straight from Blizzard themselves. |
21:17.30 | pompy | but its blizzard! |
21:17.32 | pompy | nice. |
21:18.08 | Ressy | lol |
21:18.20 | Ressy | arg |
21:18.26 | Ressy | What do I name the custom... |
21:18.32 | Ressy | INTRO MISTS HEROIC RAIDS |
21:18.33 | Ressy | ? |
21:18.49 | Ressy | can drop from any mob in heroic raids |
21:20.52 | Ressy | L["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.04 | pompy | Um |
21:22.18 | Ressy | rename? |
21:22.39 | pompy | is there a custom string for cata dungeons? |
21:22.40 | pompy | hm |
21:23.22 | Ressy | L["HYJAL_RANDOM"] = "Random drop from Hyjal Summit trash/bosses." |
21:23.57 | Ressy | L["INTRO_MISTS_HEROIC_RAIDS"] = "Random drop from Heroic Heart of Fear, Terrace of Endless Spring and Mogu'shan Vaults trash/bosses." |
21:24.03 | pompy | that BC |
21:25.17 | Ressy | recipe:AddCustom("FIRELANDS_RANDOM") |
21:25.26 | Ressy | L["FIRELANDS_RANDOM"] = "Random drop off of Firelands bosses/trash." |
21:28.54 | pompy | L["MOP_RAIDS_RANDOM"] = "Random drop off of mobs in Heroic Heart of Fear, Terrace of Endless Spring and Mogu'shan Vaults." |
21:29.14 | Ressy | k |
21:29.18 | Ressy | I'll add that instead then |
21:30.10 | Ressy | done |
21:30.24 | Ressy | Now.. I just gotta figure out why I can't dump tailoring |
21:30.29 | Ressy | or even scan it |
21:31.52 | Ressy | 13x AckisRecipeList-@project-version@\Recipe.lua:457: table index is nil |
21:31.52 | Ressy | AckisRecipeList-@project-version@\Recipe.lua:457: in function "?" |
21:32.05 | pompy | IM FIXING IT |
21:32.08 | pompy | caps |
21:32.11 | Ressy | oh |
21:32.12 | Ressy | awesome |
21:32.21 | pompy | wtf, tina fellclaw is a faction? |
21:32.28 | Ressy | o.O |
21:32.29 | pompy | mudclaw* |
21:32.33 | pompy | tina mudclaw |
21:33.06 | Ressy | ohh.. yeah |
21:33.11 | pompy | http://mop.wowhead.com/faction=1280 |
21:33.12 | pompy | lol |
21:33.22 | Ressy | doesn't give anything tho |
21:33.36 | pompy | a pet |
21:33.38 | pompy | unrelated to arl |
21:33.42 | pompy | just colledctinator |
21:33.50 | pompy | but yeah dont think all these ar prof related |
21:33.54 | pompy | prolly will refine later |
21:34.00 | Ackis | they added a new type of rep grind in MoP, "friends" |
21:34.08 | pompy | intesreting |
21:34.16 | Torhal|work | pompy: No, it's missing because I'm a total failure. |
21:34.22 | Ressy | o.O |
21:34.24 | pompy | and yay for convert to uppercase in np++ .. ctrl+shift+u |
21:34.29 | pompy | ekk Torhal|work |
21:34.35 | pompy | well glad i pointed it out then |
21:34.48 | pompy | it may be missing in a few other places in that file as well. |
21:34.50 | pompy | unsure though |
21:34.51 | Ressy | I'm just glad someone's gonna fix it, cuz I sure as hell can't yet. heh |
21:34.53 | Ressy | <3 |
21:34.55 | Torhal|work | if acquire_type == A.REPUTATION then |
21:34.55 | Torhal|work | for rep_id, rep_info in pairs(acquire_info) do |
21:34.55 | Torhal|work | local faction_string = private.FACTION_STRINGS[rep_id] |
21:34.57 | Torhal|work | if faction_string then |
21:34.57 | Torhal|work | faction_string = ("FAC.%s"):format(faction_string) |
21:34.57 | Torhal|work | else |
21:34.59 | Torhal|work | faction_string = rep_id |
21:34.59 | Torhal|work | addon:Printf("Recipe %d (%s) - no string for faction %d", self.spell_id, self.name, rep_id) |
21:34.59 | Torhal|work | end |
21:35.03 | pompy | yo man |
21:35.06 | Torhal|work | It isn't simple, hence it's non-inclusion. |
21:35.18 | Torhal|work | All the others follow the same format |
21:35.33 | pompy | if you didnt add it on purpose, why you saying total failure? |
21:36.40 | Ressy | can I push the tailoring changes? |
21:36.47 | Torhal|work | Because 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.54 | Ackis | Ressy: yeah it won't affect files Torhal|work is working on if any (or me) |
21:38.57 | pompy | Ressy: PUSHED |
21:39.00 | pompy | that should do i hope |
21:39.06 | pompy | be back in 20 |
21:39.35 | Ressy | um, worse worse |
21:39.52 | Ackis | hehehe my inscription skill is 420 |
21:40.25 | Ressy | 43 errors now after pompy's pull |
21:40.41 | Ressy | 2x AckisRecipeList-@project-version@\Database\Vendor.lua:28: Usage: GetAddon(name): "name" - Cannot find an AceAddon "nil". |
21:40.41 | Ressy | <in C code> |
21:40.57 | Torhal|work | Hold on |
21:40.59 | Ressy | 2x AckisRecipeList-@project-version@\Database\Seasonal.lua:30: Usage: GetAddon(name): "name" - Cannot find an AceAddon "nil". |
21:40.59 | Ressy | <in C code> |
21:41.11 | Ackis | you got this Torhal|work or want me to? |
21:41.24 | Torhal|work | I want to be even more pissed than I already am, so I'll do it. |
21:41.24 | Torhal|work | :D |
21:41.43 | Ressy | o.O |
21:45.11 | Torhal|work | Pushed. |
21:45.42 | Torhal|work | Not fixed though |
21:45.55 | Torhal|work | WHat were you doing to get the error? |
21:45.57 | Ressy | nope. |
21:46.10 | Ressy | Opened my profession (tailoring), and hit the scan button |
21:46.18 | Torhal|work | I just fixed the last commit pompy did, I didn't even look at the error messages |
21:46.19 | Torhal|work | Hah |
21:46.20 | Torhal|work | Ok |
21:49.12 | *** part/#arl Ressy (~Ressy@unaffiliated/ressy) |
21:49.17 | *** join/#arl Ressy (~Ressy@unaffiliated/ressy) |
21:49.19 | Ressy | oops |
21:49.40 | Torhal|work | Erm, was gonna say - works fine for me (once I updated the babble libs) |
21:49.50 | Ressy | oh? |
21:50.01 | Torhal|work | At least for Alchemy |
21:50.58 | Ressy | try tailoring |
21:51.10 | Torhal|work | I need to get in a guild to do that, or get on Live. |
21:51.17 | Torhal|work | Oh, nevermind - I copied my tailor. |
21:52.12 | Torhal|work | Works fine. |
21:52.30 | Ressy | arg. so its something I did in my update |
21:53.40 | Torhal|work | How many errors and what are they? |
21:55.00 | Ressy | when I open my tailoring profession, and hit the red scan button, I get this |
21:55.01 | Ressy | 15x AckisRecipeList-@project-version@\Recipe.lua:457: table index is nil |
21:55.01 | Ressy | AckisRecipeList-@project-version@\Recipe.lua:457: in function "?" |
21:55.01 | Ressy | AckisRecipeList-@project- |
21:55.21 | Torhal|work | That's in BugSack? |
21:55.25 | Ressy | yeah |
21:55.44 | Ressy | 457 acquire[faction_id] = {} |
21:56.02 | Ressy | so.. what faction_id am I screwing up |
21:56.12 | Torhal|work | Ressy: Go to line 449 and insert this: |
21:56.36 | Torhal|work | print(("Recipe: %s (%d)"):format(self.name, self.spell_id)) |
21:56.40 | Ressy | ohh |
21:56.57 | Ressy | god I love debug msgs |
21:57.05 | Torhal|work | :D |
21:57.38 | Ressy | Its every single rep vendor recipe. |
21:57.44 | Torhal|work | Nono |
21:57.52 | Torhal|work | The one with the issue is the last one to print. |
21:58.06 | Torhal|work | Oh |
22:00.11 | Torhal|work | Show me your AddRepVendor line or just paste the whole recipe |
22:00.17 | Ressy | -- Spelltwister's Gloves -- 125548 |
22:00.17 | Ressy | recipe = AddRecipe(125548, V.MOP, Q.COMMON) |
22:00.17 | Ressy | recipe:SetRecipeItemID(86369) |
22:00.17 | Ressy | recipe:SetCraftedItemID(82438) |
22:00.17 | Ressy | recipe:SetSkillLevels(600, 600, 600, 600, 600) |
22:00.18 | Ressy | recipe:SetItemFilterType("TAILORING_HANDS") |
22:00.18 | Ressy | recipe:AddFilters(F.VENDOR, F.REPUTATION, F.GOLDENLOTUS) |
22:00.19 | Ressy | recipe:AddRepVendor(FAC.GOLDENLOTUS, REP.HONORED, 59908) |
22:01.46 | Ressy | Faction strings was updated again |
22:01.50 | Ressy | arg |
22:02.41 | Torhal|work | Hmm? |
22:03.03 | Ressy | Its the faction string |
22:03.13 | Torhal|work | Yes I know |
22:03.16 | Torhal|work | I just fixed it |
22:03.16 | Ressy | I had a wrong one in.. |
22:03.19 | Torhal|work | Pompy had a fucking underscore |
22:03.24 | Ressy | Then you & pompy went back and forth with another one. |
22:03.31 | Ressy | I got it now I think |
22:03.42 | Torhal|work | No |
22:03.45 | Torhal|work | You don't :) |
22:05.38 | Ressy | I found the other problems.. Hopefully I found'em all. |
22:06.15 | Torhal|work | Pushed |
22:06.28 | Ressy | ty. |
22:06.31 | Ressy | added the filter? |
22:06.40 | Torhal|work | o.O |
22:06.41 | Ressy | err, 2? |
22:06.52 | Torhal|work | What did you do? |
22:06.53 | Torhal|work | o.O |
22:06.53 | Ressy | August Celestials & Golden Lotus? |
22:06.58 | Ressy | me? |
22:07.04 | Torhal|work | YEs. |
22:07.08 | Torhal|work | You said you found the problem |
22:07.12 | Torhal|work | So what did you do? :) |
22:07.34 | Ressy | Recipe 'Spelltwister's Grand Robe' (spell ID 125547): Attempting to assign non-existent filter flag. |
22:08.03 | Ressy | F.GOLDEN_LOTUS |
22:08.04 | Torhal|work | Ok, I'm confused. That's not the error you had befiore. |
22:08.08 | Torhal|work | No |
22:08.10 | Torhal|work | Don't do that |
22:08.18 | Torhal|work | pompy fucked up twice when he added those |
22:08.21 | Torhal|work | You were right before |
22:08.22 | Ressy | o.O |
22:08.32 | Torhal|work | I fixed the strings. |
22:08.37 | Ressy | I'm just following previous reputations, like thorium brotherhood.. they had their own filters. |
22:08.38 | Ressy | oh |
22:08.39 | Torhal|work | GOLDENLOTUS was correct |
22:08.40 | Ressy | so.. |
22:08.42 | Ressy | k |
22:08.47 | pompy | seriously torhal |
22:09.46 | Torhal|work | You don't know what you're doing? Don't fucking touch it. Seriously. |
22:09.47 | pompy | sorry i missed a fucking comma |
22:09.58 | Torhal|work | No, you added commas where they had no business being. |
22:10.01 | pompy | i was in a rush sorrry.. i forgot to remove a comma |
22:10.06 | Torhal|work | And you arbitrarily named factions. |
22:10.08 | pompy | I DIDNT ADD i forgot to remove it |
22:10.11 | pompy | it was copy and paste |
22:10.14 | Torhal|work | Yes. |
22:10.45 | pompy | what? i fucking c/p what ackis added and turned them into the faction_strings |
22:10.51 | pompy | wtf |
22:11.43 | Torhal|work | The faction strings have to match the filter flags. |
22:11.49 | Torhal|work | And they never have spaces in them. |
22:12.17 | Ressy | where can I find the filter flags? |
22:12.24 | pompy | i didnt add a space |
22:12.25 | Torhal|work | Constants.lua |
22:12.26 | pompy | anywhere |
22:12.31 | Ressy | cuz.. |
22:12.36 | Ressy | August celesetials might be a problem |
22:12.40 | Torhal|work | [1271], "ORDER OF THE CLOUD SERPENT", |
22:12.41 | Ressy | AUGUSTCELESTIALS= 0X00004000,-- 15 |
22:12.59 | Torhal|work | [1269], "GOLDEN_LOTUS", |
22:13.00 | Ressy | [1341]= "THE_AUGUST_CELESTIALS", |
22:13.03 | Torhal|work | The filter flag is GOLDENLOTUS |
22:13.57 | Torhal|work | Change it to AUGUSTCELESTIALS |
22:14.16 | Torhal|work | Ackis names shit weird, but the faction names ocrrespond directly to the filter flags. |
22:14.27 | Torhal|work | So when they aren't right, shit goes boom. |
22:14.31 | Ressy | lorewalkers vs the lorewalkers |
22:14.43 | Torhal|work | I only fixed what I saw. |
22:14.50 | Torhal|work | So fix anything else that's incorrect |
22:15.04 | Torhal|work | I'm going home. |
22:15.04 | Ressy | k. I wasn't sure that was even a problem. lol |
22:15.10 | Torhal|work | Ressy: |
22:15.21 | Torhal|work | Well, if it's in private.REP_FLAGS_WORD2 = { |
22:15.31 | Torhal|work | And it's different there than in the otehr table |
22:15.35 | Torhal|work | Fix 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.59 | Ressy | arg |
22:19.03 | Ressy | still coming up with this. |
22:19.05 | Ressy | Imma take a break |
22:19.45 | Ressy | 3x AckisRecipeList-@project-version@\Database\Recipes\Tailoring.lua:4455: attempt to call method "?" (a nil value) |
22:20.23 | Ressy | Thats.. odd |
22:20.29 | Torhal|work | Hrm? |
22:20.31 | Ressy | It doesn't like the createditem id |
22:20.38 | Torhal|work | o.O |
22:20.51 | Torhal|work | What's line 4455? |
22:21.04 | Ressy | was recipe:SetCreatedItemID82444 |
22:21.06 | Ressy | err |
22:21.08 | Torhal|work | lol |
22:21.09 | Ressy | wiwth brackets |
22:21.12 | Ressy | blah |
22:21.14 | Ressy | fixing |
22:21.17 | Torhal|work | CraftedItemID |
22:21.23 | Ressy | yeah |
22:21.24 | Ressy | realized |
22:22.14 | Ressy | blonde day |
22:25.11 | Ressy | hooray. I tracked down and fixed the million and one problems. >.< |
22:25.41 | Ressy | I didn't mean to cause problems with pompy. :\ |
22:25.48 | Ackis | no it's not your fault |
22:26.08 | Ackis | he's off the projects now however |
22:26.17 | Ackis | so hopefully you stay on :P |
22:28.13 | Ressy | pushing |
22:28.32 | Ackis | I just did a ToC update |
22:28.43 | Ackis | I put my name in front of Torhal's kekekek |
22:29.21 | Ressy | :P |
22:29.46 | Ressy | You pulled him off? why? |
22:30.21 | Ackis | he 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.44 | Ressy | omg |
22:32.31 | Ackis | ? |
22:33.39 | Ressy | surprised as hell |
22:33.51 | Ackis | this has been a long time coming |
23:07.06 | Ressy | Torhal... |
23:07.15 | Ackis | he's afk |
23:07.16 | Ressy | That line you had me add to Recipes.lua |
23:07.17 | Ressy | oh |
23:07.18 | Ackis | driving home or something |
23:07.20 | Ressy | its awesome |
23:07.27 | Ressy | it prints out every single mistake that there is. |
23:07.37 | Ressy | so I can just go through it, and fix it all in a single reload. |
23:07.47 | Ressy | <33333 |
23:07.58 | Ressy | less of me going "uh, why am I broken now!" |
23:25.16 | Ackis | I am in a foul mood |
23:25.28 | Ressy | :( |
23:25.39 | Ressy | how come? |
23:26.03 | Ackis | pompy bullshit |
23:26.10 | Ackis | I'm off work on stress leave and I get to deal with this |
23:26.19 | Ressy | :\ |
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.15 | Ackis | Ressy: I told him a bout your compliment |
23:52.20 | Ackis | he started blushing like a new bride |
23:52.37 | Torhal | And then I pissed in Ackis's shoes. |
23:52.56 | Ressy | lol |
23:53.04 | Ressy | But yeah, that line is just amazing. |
23:53.37 | Ressy | tells me exactly what the problem is |
23:54.15 | Torhal | :D |
23:55.40 | Ressy | now, how do I get a custom to show up in 'miscellaneous' |
23:56.11 | Torhal | Ressy: Example? |
23:57.20 | Ressy | arg. |
23:57.26 | Ressy | AddCustom("FIRELANDS_RANDOM", Z.FIRELANDS) |
23:57.33 | Ressy | does AddCustom accept multiple zones? |
23:58.27 | Torhal | No, but I can change that. |
23:58.39 | Ressy | oh? |
23:58.45 | Torhal | Sure. |
23:59.37 | Ressy | looks like it already takes zone & coords |
23:59.46 | Torhal | Just one. |
23:59.47 | Ressy | AddCustom("DM_TRIBUTE", Z.DIRE_MAUL, 59.04, 48.82) |
23:59.54 | Torhal | One of each. |