| 00:10.25 | Ressy | getting some odd bugs |
| 00:10.35 | Ressy | 1x AceLocale-3.0-6: Ackis Recipe List: Missing entry for "Steeg Haskell" |
| 00:10.36 | Ressy | AckisRecipeList-@project-version@\Database\Vendor.lua:409: in function "InitVendor" |
| 00:10.36 | Ressy | AckisRecipeList-@project-version@\Core.lua:756: in function "InitializeLookups" |
| 00:10.36 | Ressy | AckisRecipeList-@project-version@\Scanner.lua:64: in function "LoadAllRecipes" |
| 00:10.36 | Ressy | AckisRecipeList-@project-version@\Scanner.lua:754: in function "ScanVendor" |
| 00:10.36 | Ressy | AckisRecipeList-@project-version@\Core.lua:476: in function "?" |
| 00:10.36 | Ressy | Ace3-Release-r1061\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:147: in function <Ace3\CallbackHandler-1.0\CallbackHandler-1.0.lua:147> |
| 00:10.37 | Ressy | <string>:"safecall Dispatcher[1]":4: in function <string>:"safecall Dispatcher[1]":4 |
| 00:10.37 | Ressy | <in C code> |
| 00:10.38 | Ressy | <string>:"safecall Dispatcher[1]":13: in function "?" |
| 00:10.38 | Ressy | Ace3-Release-r1061\CallbackHandler-1.0\CallbackHandler-1.0-6.lua:92: in function "Fire" |
| 00:10.39 | Ressy | Ace3-Release-r1061\AceEvent-3.0\AceEvent-3.0-3.lua:120: in function <Ace3\AceEvent-3.0\AceEvent-3.0.lua:119> |
| 00:10.51 | Ressy | blah. should've just posted to wowace paste. |
| 00:14.10 | Ackis | just woke up |
| 00:14.37 | Ackis | let me pull |
| 00:15.42 | Ackis | I think I know what's wrong miss Ressy |
| 00:16.06 | Ressy | oh? |
| 00:16.35 | Ackis | you didn't add that vendor into the vendor db |
| 00:16.37 | Ressy | removed him from vendor list.. |
| 00:16.41 | Ackis | or removed him |
| 00:16.42 | Ressy | oh I did |
| 00:16.52 | Ressy | I removed him, and I don't get that error anymore |
| 00:16.54 | Ressy | I get a diff one |
| 00:17.44 | Ressy | http://www.wowace.com/paste/6193/ |
| 00:18.14 | Ackis | okay so he's a guild vendor that you just added? |
| 00:18.48 | Ressy | Torhal and I added one last night, cuz it was causing an error like that, until we DID add him. lol |
| 00:20.37 | Ackis | did you remove the recipes that he was associated with? |
| 00:20.42 | Ackis | the cooking/alchemy ones? |
| 00:21.00 | Ackis | I've never seen that error tbh |
| 00:24.10 | Ackis | only errors I get on my side are missing locale ones |
| 00:25.31 | Ressy | its.. possible |
| 00:25.41 | Ressy | there was a edit conflict earlier |
| 00:25.51 | Ressy | I tried pulling, it said there was nada, so I tried pushing.. |
| 00:25.55 | Ackis | so right now your local copy is the the same as what pushed? |
| 00:25.58 | Ressy | then it went "oh wait" |
| 00:26.03 | Ressy | It should be.. I pulled |
| 00:26.16 | Ackis | I'm not getting any errors |
| 00:26.25 | Ackis | other than acelocale ones |
| 00:27.52 | Ressy | I'm pulling, but nada |
| 00:27.59 | Ressy | How do I get it to essentially reinstall everythintg |
| 00:28.23 | Ressy | and, the data thats collected, where is it temporarily saved? |
| 00:28.34 | Ackis | that I don't know |
| 00:28.43 | Ackis | I only know how to revert to the lastest pull |
| 00:28.49 | Ressy | which is? |
| 00:28.52 | Ackis | you'd get better luck asking in wowuidev |
| 00:29.50 | Ressy | how to revert to latest pull? |
| 00:30.44 | Ackis | delete the entire directory and clone it again |
| 00:30.51 | Ackis | but that means you'd lose all your changes right? |
| 00:31.00 | Ressy | okies. |
| 00:31.12 | Ressy | Thats fine. I can run around and go nuts again |
| 00:31.45 | Ackis | whihc ones have you been updating? everything? |
| 00:32.08 | Ressy | neutral & alliance |
| 00:32.14 | Ressy | neutral before talking to you & pompy. |
| 00:32.23 | Ressy | but yeah, everything |
| 00:34.46 | Ressy | I'd rather start from scratch than fuck up the db. |
| 00:35.25 | Ressy | there we go. |
| 00:35.34 | Ressy | Before, I only saw my last push |
| 00:35.42 | Ressy | I couldn't see the following 2 for some reason |
| 00:36.00 | Ackis | okay do a pull now |
| 00:36.14 | Ackis | do you see the changes to blacksmithing/alchemy? |
| 00:37.01 | Ressy | I see your "Update blacksmith recipes" |
| 00:37.11 | Ackis | and I just pushed an alchemy one |
| 00:37.53 | Ressy | nope |
| 00:37.53 | Ressy | nada |
| 00:38.10 | Ackis | Repo[WoW] arl: Ackis master * 2.4-43-g5406694 Database/Recipes/Alchemy.lua: [+1 commit] Update alchemy - Apothecary Bressa. |
| 00:38.23 | Ressy | don't see it |
| 00:38.48 | Ackis | even with another pull? |
| 00:39.30 | Ressy | "Why isn't there a pull command in git gui's remote menu? |
| 00:39.30 | Ressy | I got lazy and didn't write support for it. Before today nobody has |
| 00:39.30 | Ressy | really asked me why it was missing so it seemed to be unimportant. |
| 00:39.30 | Ressy | > Am I supposed to fetch and then merge? |
| 00:39.30 | Ressy | Yes, that works. " |
| 00:39.31 | Ressy | ffs |
| 00:39.35 | Ressy | lazy ass bastages |
| 00:40.14 | Ackis | ahhh fetch/merge |
| 00:40.28 | Ressy | local branch I guess. |
| 00:40.29 | Ackis | http://stackoverflow.com/questions/1628088/how-to-reset-my-local-repository-to-be-just-like-the-remote-repository-head |
| 00:41.05 | Ackis | git is a beast |
| 00:41.19 | Ackis | I don't use a gui for it, I'm a command line person :D |
| 00:41.27 | Ressy | ... |
| 00:41.34 | Ressy | Yeah, not used to this yet |
| 00:42.16 | Ackis | want an unsolicited piece of advice? :) |
| 00:42.50 | Ackis | do one vendor/trainer/etc -> dump -> local commit |
| 00:42.57 | Ackis | rinse/repeat for about 5-10 (your call) |
| 00:42.59 | Ackis | push |
| 00:49.58 | Ressy | ah lovely. |
| 00:50.00 | Ressy | where you really need the command line |
| 00:50.00 | Ressy | One common task that is needlessly complex in both the GUIs is git pull. In a properly setup repository that has been cloned from the project master or something like that, getting the latest changes is just a matter of typing git pull. In git gui, described above, this takes 4 steps, although there is a reason for that complexity :-) |
| 00:50.19 | Ressy | yeah.. fuck me. |
| 00:50.21 | Ressy | >.< |
| 00:51.11 | Ressy | oh well.. looks like I'm learning the bash |
| 00:53.22 | Ackis | what client are you on? |
| 00:54.22 | Ressy | was using git gui |
| 00:54.27 | Ressy | switched to the bash now. |
| 00:54.37 | Ressy | found a site that has cheatsheets and such |
| 00:54.39 | Ressy | crash course |
| 00:54.40 | Ackis | so a linux/mac machien? |
| 00:54.44 | Ressy | windoz |
| 00:57.50 | Ressy | If I clone, can I move the folder afterwards without things breaking? |
| 00:58.08 | Ackis | should be able to |
| 00:58.31 | Ressy | howd you do it? |
| 00:59.18 | Ackis | copy/paste the entire folder |
| 00:59.30 | Ressy | okies |
| 01:00.06 | Ackis | nothing special is needed |
| 01:08.26 | Ressy | ah. there we go. |
| 01:08.53 | Ressy | there. Bash & GUI hooked up. |
| 01:09.05 | Ressy | now I see your alchemy update |
| 01:09.15 | Ressy | and since bash is hooked up, easy pull |
| 01:10.02 | Ressy | Sorry.. I haven't used git before last night. >.< |
| 01:10.16 | Ackis | no need to be sorry |
| 01:10.23 | Ackis | git is a beast |
| 01:10.32 | Ressy | quite |
| 01:10.33 | Ackis | just remember my last piece of advice: do things in small batches |
| 01:11.38 | Ressy | lol |
| 01:11.40 | Ressy | gotcha |
| 01:12.02 | Ressy | and stick to Alliance. :P |
| 01:12.14 | Ressy | can I hit up neutral too if I see'em? |
| 01:12.24 | Ackis | yeah |
| 01:55.04 | *** join/#arl Ressy (~Ressy@108.162.157.134) |
| 01:55.04 | *** join/#arl Ressy (~Ressy@unaffiliated/ressy) |
| 02:03.50 | Ackis | Ressy: http://a8.sphotos.ak.fbcdn.net/hphotos-ak-snc6/226597_5913686685_1876_n.jpg |
| 02:04.11 | Ressy | oh yeah.. I remember you posting that one |
| 02:12.55 | Ressy | I dunno how you guys wanna deal with how Blizz is doing the profession item upgrades for yourself.. |
| 02:13.54 | Ressy | http://www.wowhead.com/search?q=fur+lining+-+stamina |
| 02:14.00 | Ressy | You buy it at 400, it auto-upgrades |
| 02:14.19 | Ackis | we have functionality for that already |
| 02:14.23 | Ressy | k |
| 02:14.31 | Ackis | was quite annoying |
| 02:14.44 | Ressy | Was just wondering cuz I keep seeing it in the logs when I'm gathering data |
| 02:14.45 | Ressy | heh.. |
| 02:14.50 | Ressy | enchanting, lw. |
| 02:14.56 | Ressy | guessing discovery? |
| 02:15.27 | Ackis | discovery? |
| 02:16.01 | Ressy | MoP uses discovery almost exclusively for alchemy. |
| 02:16.10 | Ressy | you learn literally 2 recipes from the trainer |
| 02:16.15 | Ackis | ah yeah |
| 02:16.18 | Ressy | everything else is discovered from those 2 and ones you discover |
| 02:17.04 | Ackis | I don't know how we datamine discoveries |
| 02:17.36 | Ressy | *shrugs* |
| 02:17.44 | Ackis | I think it's manual tbh |
| 02:17.52 | Ressy | that'll be fun. lol |
| 02:18.03 | Ackis | that's how I did it originally =/ |
| 02:18.11 | Ackis | every recipe |
| 02:18.20 | Ressy | http://www.wowhead.com/spells=11.171?filter=cr=20:16;crs=1:5;crv=0:0 |
| 02:18.33 | Ressy | top 2 are trained |
| 02:18.36 | Ressy | The rest.... |
| 02:18.41 | Ackis | only 33, that's not too bad |
| 02:24.12 | Ackis | I'm trying not to touch ARL until Col 2.0 is done :D |
| 02:24.36 | Ressy | oooh |
| 02:26.50 | Ressy | I really like how Blizzard's adding in the easy-levelling for cooking |
| 02:27.00 | Ressy | seriously, its about bloody time |
| 02:27.16 | Ackis | I don't quite get it, because I didn't bother reading up on it :D |
| 02:27.34 | Ressy | oh |
| 02:27.44 | Ressy | They made it so you could make lvl 80 water, at 1 cooking |
| 02:27.49 | Ressy | with pandaria items |
| 02:28.02 | Ackis | nice, so you could level all the way up to 500ish with level 80 water? |
| 02:28.15 | Ressy | yup |
| 02:28.22 | Ressy | different waters of course |
| 02:28.29 | Ressy | Each is good for 30-32 levels |
| 02:28.38 | Ressy | err, 30-32 skill points |
| 02:29.17 | Ressy | http://www.wowpedia.org/Cooking_recipes#Apprentice_Recipes_.280_.E2.86.92_75.29 |
| 02:29.22 | Ressy | 1 30 32 35 Sliced Peaches MoPTrainer, for 1s. |
| 02:29.34 | Ressy | I updated the entire cooking thing.. |
| 02:29.40 | Ressy | Thats how you power level your cooking now |
| 02:29.56 | Ackis | nice |
| 02:31.37 | Ackis | make sure you're pushing your changes btw, it's been a while hasn't it? |
| 02:32.19 | Ressy | I've just been collecting in IF |
| 02:32.28 | Ressy | and Kharanos |
| 02:32.45 | Ackis | save, commit, push :D |
| 02:32.58 | Ressy | I haven't even done a single dump yet. |
| 02:33.00 | Ackis | especially when learning about it small commits/pushes are better |
| 02:33.02 | Ackis | ah okay |
| 02:33.11 | Ressy | just been flying around scanning npcs |
| 02:38.44 | Ressy | okies.. time to start the scans & dumps. |
| 02:40.24 | Ackis | :D |
| 02:40.33 | Ackis | pompy: can you add the darkmoon faire rabbit to collectinator please? |
| 02:41.39 | Ackis | (he's at work right now) |
| 02:43.11 | Ackis | I wished Torhal was around though :P |
| 02:43.14 | Ressy | can always tell how big a dump's gonna be, when it freezes you for almost a full minute. |
| 02:44.08 | Ackis | I have that problem IRL too |
| 02:44.19 | Ressy | tmi |
| 02:45.00 | Ackis | :D |
| 02:45.05 | Ackis | your son might be more mature than me |
| 02:46.13 | Ressy | Do you go smashing your wife in the face with a toy when she says its bedtime? |
| 02:46.47 | Ackis | maybe |
| 02:46.58 | Ressy | uh huh |
| 02:48.32 | pompy | yeah sure ackis |
| 02:48.41 | pompy | did you kill that elite and get it? |
| 02:49.05 | Ackis | nope but I know people who have |
| 03:04.19 | Ressy | "(blah) is missing a recipe id" |
| 03:04.21 | Ressy | o.O |
| 03:05.03 | pompy | Thats actually something i am working on right now ;) |
| 03:05.11 | Ressy | ohh |
| 03:05.21 | Ressy | I've seen a bunch in leatherworking. |
| 03:06.30 | Ressy | I'm almost done what I'll be committing |
| 03:06.30 | pompy | Yessum, theres no API to get the recipe item id associate with that recipe, thats why it kinda sucks. |
| 03:06.54 | Ressy | you do that manually? |
| 03:07.03 | pompy | pretty much |
| 03:07.43 | pompy | elseif recipe:HasFilter("common1", "VENDOR") or recipe:HasFilter("common1", "INSTANCE") or recipe:HasFilter("common1", "RAID") or recipe:HasFilter("common1", "MOB_DROP") or recipe:HasFilter("common1", "WORLD_DROP") then |
| 03:07.43 | pompy | self:Debug("Recipe %d (%s) is missing a recipe item ID.", spell_id, recipe.name) |
| 03:07.53 | pompy | if it has any of those flags and doesnt have it set, thats how the scanner tells us |
| 03:08.08 | pompy | doesnt have a recipe item id set ** |
| 03:08.29 | pompy | if we have one set, we map that to the spell id |
| 03:11.26 | pompy | Ressy: lemme know when you have committed so i can do my thing, so i can just avoid the merging stuff and what not. |
| 03:11.49 | *** join/#arl Torhal_ (~torhal@c-71-207-224-241.hsd1.al.comcast.net) |
| 03:11.49 | *** mode/#arl [+o Torhal_] by ChanServ |
| 03:11.52 | Ressy | ok |
| 03:12.03 | Ressy | I'm just about done |
| 03:12.47 | Ressy | I only hit IF, Kharanos. |
| 03:13.02 | Ressy | Theres still new vendors in other places. heh |
| 03:13.10 | pompy | dont have to say only lol. those two places are big. |
| 03:13.17 | pompy | and it doesnt get done in the snap of a finger |
| 03:14.14 | Ressy | oh, and the lil waystation headed out of dun morough. Theres nothing in the starter area now. |
| 03:14.54 | Ackis | did you get the duder that trains all professions? |
| 03:15.26 | Ressy | yeah, but I already trained profs, so I could only scan the ones I'd trained. |
| 03:15.34 | Ressy | and pushed |
| 03:15.52 | pompy | you can scan a trainer without being trained in it.. |
| 03:15.58 | pompy | are you in a guild? |
| 03:16.01 | Ressy | yeah |
| 03:16.04 | pompy | not that it matters |
| 03:16.13 | Ackis | it does |
| 03:16.16 | pompy | but yeah, you dont hav eto be a blacksmith to scan a blacksmith trainer! |
| 03:16.17 | Ackis | guild vendors have recipes |
| 03:16.27 | Ressy | 3 |
| 03:16.27 | pompy | i think she did them already |
| 03:16.33 | pompy | but yeah |
| 03:16.34 | Ressy | I only hit IF's so far. |
| 03:16.40 | Ressy | Theres one in every major city |
| 03:16.50 | pompy | resa1983, that you? |
| 03:16.52 | pompy | Yup |
| 03:16.52 | Ressy | yup |
| 03:19.03 | Ressy | taking screenshots of NPCs needing to be added. heh |
| 03:38.11 | pompy | Ressy: it didnt crash logging into my Alliance.. nicee. |
| 03:38.19 | pompy | (I hope i dont speak too soon!) |
| 03:40.15 | pompy | I just dunno what could have caused that corruption before >< |
| 03:54.18 | Ressy | it was fairly wide-spread |
| 04:47.43 | Ressy | I feel like death. I'm headed to bed. night guys |
| 12:08.49 | *** join/#arl Ressy (~Ressy@unaffiliated/ressy) |
| 12:45.00 | Ressy | go figure, I'm doing scanning, and adding those new all in 1 prof trainers, and wowhead is what bugs out. |
| 12:50.29 | Torhal | Stab it, Ressy! |
| 13:13.24 | Ressy | I did. :P |
| 13:13.28 | Ressy | go go acp. |
| 13:13.36 | Ressy | disable with just a reloadui |
| 13:13.55 | Ressy | figured out how to scan those new all-in-one trainers too. |
| 13:13.58 | Ressy | Added another one |
| 13:29.05 | Ressy | Question. |
| 13:29.13 | Ressy | Those new all-in-one trainers.. |
| 13:29.27 | Ressy | Do I have to put them in each trainers page, or just one. |
| 13:29.49 | Ressy | cuz i put this one into each, and now on opening a trainer, I get |
| 13:29.50 | Ressy | "Ackis Recipe List: Duplicate lookup: 47384 - Lien Farner." |
| 13:33.27 | Torhal | Those should just go into Trainer.lua |
| 13:33.34 | Torhal | The one with Runda and the other new one. |
| 13:38.17 | Ressy | k |
| 13:38.28 | Ressy | manually added I assume |
| 13:48.11 | Torhal | Yar :/ |
| 13:54.00 | Ressy | got it all sorted out. :) |
| 14:25.15 | Ressy | adding recipe ids for things that complain as well.. :) |
| 15:13.28 | Ressy | Pushing SW/Elwynn |
| 15:38.52 | Ressy | So how do I deal with new recipes |
| 15:45.03 | Torhal | Define new recipes? :) |
| 15:49.18 | Ressy | 2 deleted, replaced with something that was mailed to us. |
| 15:49.26 | Ressy | I think its one of those 'auto-update to highest level' ones |
| 15:49.32 | Torhal | o.O |
| 15:49.55 | Torhal | Which two were deleted? And a recipe item was mailed in replacement? |
| 15:50.00 | Ressy | yeah |
| 15:50.25 | Ressy | Flask of the North & Flask of Enhancement were deleted. Replaced with "Alchemist's Flask" which auto-upgrades itself based on your profession level |
| 15:50.43 | Ressy | like the tailoring, leatherworking tsuff does |
| 15:51.07 | Torhal | I'm assuming I never learned those since I never got a mail :/ |
| 15:51.22 | Ressy | Its an alchemy one. |
| 15:51.28 | Torhal | Yah, I'm an alchemist :) |
| 15:51.31 | Ressy | ah |
| 15:51.49 | Ressy | yeah.. they mailed an Alchemist's flask |
| 15:51.52 | Ressy | which is pretty cool |
| 15:53.30 | Torhal | Hrm. |
| 15:53.42 | Ressy | pushed. |
| 15:53.51 | Torhal | I'd hold out on that one for now - if it isn't obtainable via vendor or trainer yet... |
| 15:53.52 | Torhal | :/ |
| 15:53.56 | Ressy | It is |
| 15:53.58 | Ressy | via trainer |
| 15:53.59 | Torhal | Oh! |
| 15:54.02 | Ressy | nods |
| 15:54.11 | Torhal | Then a dump should just put it where it belongs. |
| 15:54.11 | Ressy | They replaced both the 2 training recipes, with the 1. |
| 15:54.24 | Ressy | it wasn't, so I added a minimum. |
| 15:54.38 | Ressy | spellid is over 110k |
| 15:54.58 | Torhal | Added a minimum? |
| 15:55.01 | Ressy | anyways. Added manually, and hit up 2 trainers to test, and it worked. |
| 15:55.12 | Torhal | Weird. It should have added it. |
| 15:56.28 | Torhal | It added the ALCHEMY_FLASK though, and the CraftedRecipeID, after the dump? |
| 16:04.53 | Ressy | no. I did that |
| 16:08.29 | Ressy | lawlz. Just looked at ARL on my alchemist. Only missing 14 recipes of 289 |
| 16:19.25 | Ressy | finding some trainers don't offer some recipes anymore. |
| 16:21.37 | Ackis | holy shit Torhal is ALIVE |
| 16:21.47 | Torhal | Liar |
| 16:21.55 | Ackis | msn time |
| 16:21.59 | Torhal | Nein. |
| 16:22.25 | Ressy | What exactly is being done about the auto-upgrade ones |
| 16:23.00 | Ressy | "Fur Lining - Stamina" |
| 16:23.13 | Ressy | http://www.wowhead.com/search?q=Fur+Lining+-+Stamina |
| 16:23.27 | Torhal | Ressy: recipe:SetPreviousRankID(57690) |
| 16:24.00 | Ressy | is it in use anywhere already that I can see |
| 16:24.11 | Torhal | Directly from that recipe :) |
| 16:24.41 | Ressy | k. 3 ranks. |
| 16:25.59 | Ressy | and its only spellids |
| 16:26.41 | Ressy | spellids: 124553,85007,57690 |
| 16:27.52 | Ressy | will that setpreviousrankId accept multiple ids? |
| 16:27.54 | Ressy | or just 1 |
| 16:29.22 | Ressy | also, skill levels will auto adjust? |
| 16:30.16 | Torhal | It's just one, the previous rank spell ID |
| 16:30.29 | Ressy | Theres 3 ranks tho |
| 16:30.36 | Torhal | They should auto-adjust, yes, if you mean the trivial, etc |
| 16:30.48 | Torhal | Yah, they should all three be in there unless they were missed. |
| 16:30.50 | Ressy | So, 3 entries per fur lining? |
| 16:30.59 | Torhal | No |
| 16:31.16 | Ackis | one entry with 2 previous rank |
| 16:31.18 | Torhal | 124553 would have 57690 listed |
| 16:31.32 | Ressy | k |
| 16:31.35 | Torhal | Er |
| 16:31.36 | Torhal | No: |
| 16:31.44 | Torhal | Each higher lists its immediate lower. |
| 16:31.46 | Torhal | That's all/ |
| 16:31.56 | Ackis | so three separate entries like she said? |
| 16:32.01 | Ackis | or did I misread her? |
| 16:32.10 | Torhal | I read it as three listings per recipe. |
| 16:32.12 | Ressy | Yeah.. there'll be 3 sep entries |
| 16:32.16 | Torhal | Ah. |
| 16:32.31 | Ressy | its just weird seeing this: |
| 16:32.31 | Ressy | Recipes with possible incorrect skill levels - unable to confirm: |
| 16:32.32 | Ressy | 85008 (Fur Lining - Agility): Skill set to 500; trainer reports 400. |
| 16:33.20 | Torhal | Yah, we're not sure what the actual rank is. |
| 16:33.43 | Torhal | I'd have to look at the code for it again to see if I can determine now. |
| 16:34.07 | Ressy | Its scanning the 2nd, as thats all thats available right now. |
| 16:34.19 | Ressy | but I'm not trained, so trainer correctly shows 400 |
| 16:35.01 | Torhal | Ok, that's why it can't be confirmed - three ranks all with the same name. |
| 16:35.07 | Torhal | We don't know which is which. |
| 16:35.28 | Ressy | trainer will only ever show the first |
| 16:35.37 | Ressy | the other 2 are auto-learned |
| 16:37.08 | Torhal | Yah, but we don't know which is being shown. |
| 16:37.16 | Torhal | So we can't confirm that the trainer isn't lying :) |
| 16:37.27 | Ressy | Hm. I can check with my lw |
| 16:38.25 | Ressy | found it |
| 16:38.34 | Ressy | Only level 1 is shown when not trained |
| 16:38.47 | Ressy | checking lw |
| 16:40.54 | Torhal | Ok, but we don't know which level he's showing at that point. Like I said, I could look at the code and see if I can determine maybe based on the ID |
| 16:42.21 | Ressy | They're all different skills. |
| 16:42.32 | Ressy | level 1 starts at 400, level 2 at 500, level 3 at 500 |
| 16:42.54 | Ressy | the error alone says which one the game thinks its scanning |
| 16:43.00 | Ressy | <PROTECTED> |
| 16:44.35 | Torhal | I'll look at the code. I know there's a reason I did that but hell if I remember what it was. |
| 16:44.38 | Ressy | heh |
| 16:47.34 | Ressy | Getting the hang of this.. |
| 16:48.11 | Ressy | staged a commit, unstaged the lw page, commited, pushed, and then killed the lw change |
| 16:50.43 | Ressy | gosh they removed a bunch of recipes from this trainer |
| 16:52.36 | Torhal | It's insane how much changes :) |
| 16:54.30 | Ressy | yeah |
| 16:54.43 | Ressy | as I'm going along, I'm learning quite a bit on how to fix these thigns. :) |
| 16:56.55 | Torhal | :) A lot to take in, and certain special cases (as you've seen) |
| 17:14.58 | Ressy | I'm not gonna remove the engineering ones it says the trainer doesn't have.. I think they're being hidden cuz I'm the wrong class. |
| 17:17.17 | Ressy | question |
| 17:17.22 | Ressy | -- Titanium Plating -- 62202 |
| 17:17.27 | Ressy | makes the same item. |
| 17:17.35 | Ressy | 2 different plans tho. A vs H again |
| 17:18.07 | Ressy | can you set 2 recipeids for a single item? |
| 17:28.28 | Torhal | o.O |
| 17:28.43 | Torhal | There's the spell ID, which is 62202 |
| 17:28.55 | Torhal | Then there's the recipe ID for those who have an actual drop |
| 17:29.01 | Torhal | Then there's teh crafted item ID |
| 17:29.05 | Torhal | s/teh/the/ |
| 17:30.44 | Ressy | 2 separate recipes make the same item |
| 17:30.48 | Ressy | same crafted item |
| 17:31.15 | Ressy | purchased from respective vendors |
| 17:31.19 | Ressy | alliance or horde quartermaster |
| 17:34.57 | Torhal | Oh. |
| 17:35.20 | Torhal | No, the way this was originally coded it only supports one recipe ID |
| 17:35.31 | Torhal | There are so few like that that we just choose one or the other. |
| 17:35.36 | Torhal | Doesn't really matter overall |
| 17:37.37 | Torhal | Just pushed some code changes, so pull before you push. |
| 17:40.47 | Ressy | gotcha |
| 17:42.24 | Ressy | so this change.. |
| 17:42.35 | Ressy | trainer scanning. |
| 17:42.43 | Ressy | less Lag on opening trainer? :) |
| 17:47.19 | Torhal | It should, aye :) |
| 17:47.29 | Torhal | If it's an alchemy trainer, it only loads the Alchemy recipes. |
| 17:47.52 | Torhal | Then it uses the list of ONLY Alchemy recipes instead of iterating over all recipes |
| 17:48.04 | Ressy | cool |
| 17:48.13 | Ressy | can't do anything about the dumps tho, huh |
| 17:48.21 | Ressy | blacksmithing tends to freeze me for minutes on end. |
| 17:49.43 | Torhal | Nope, it analyzes everything about each and every recipe then does the formatting that the code can read |
| 17:49.51 | Ressy | gotcha. |
| 17:50.03 | Ressy | Since pompy's at work, I'm going to hit all of Northrend quickly. |
| 17:50.04 | Torhal | kk, naptime (I hope) |
| 17:51.51 | Ressy | Ackis Recipe List: Recipe database not loaded. Please scan the tradeskill first then try the datamining. |
| 17:51.54 | Ressy | um, what? |
| 19:50.47 | Ressy | push. Northrend |
| 21:06.48 | Torhal | Ressy: Ping. |
| 21:07.14 | Ressy | yup |
| 21:07.34 | Torhal | You uncovered some obviously erroneous entries in Jewelcrafting.lua |
| 21:07.49 | Torhal | Look for recipe:AddVendor(28721) |
| 21:07.50 | Ressy | oh? |
| 21:08.18 | Torhal | You'll see a bunch where they have her as the only vendor, are marked F.RETIRED and have recipe:AddCustom("REMOVED_FROM_GAME") |
| 21:08.22 | Torhal | :D |
| 21:08.46 | Ressy | ohh. |
| 21:09.04 | Torhal | Hehehe |
| 21:09.06 | Ressy | so, remove the entire thing, or remove the vendor line |
| 21:09.21 | Ressy | entire thing since its removed from game. |
| 21:09.25 | Torhal | Nah, ARL discovered that she sells it so it's obviously in game |
| 21:09.34 | Ressy | she used to maybe |
| 21:09.55 | Torhal | You didn't add it manually did you? |
| 21:10.04 | Ressy | no |
| 21:10.13 | Torhal | If not, then ARL saw it in her list of things she sells. |
| 21:10.32 | Torhal | So when you did the dump, it inserted her into their entries. |
| 21:10.49 | Torhal | So the entries were wrongly marked as gone, |
| 21:11.05 | Ressy | lemme check manually |
| 21:11.08 | Torhal | We only physically remove entries when their ID makes ARL say "WTF is this?" |
| 21:11.25 | Torhal | Otherwise, we mark 'em as removed from game and slap the retired flag on 'em |
| 21:11.57 | Ackis | no nap for torhal? |
| 21:12.07 | Torhal | Ackis: I said nap, not hibernation. |
| 21:12.10 | Torhal | :D |
| 21:12.28 | Ackis | want to help me with Col 2.0? I can't figure out why the database entry isn't being marked as found |
| 21:12.51 | Torhal | Lemme clone the repo |
| 21:13.40 | Ressy | Its not on that npc |
| 21:13.50 | Torhal | o.O |
| 21:13.58 | Torhal | Dafuq? |
| 21:14.43 | Torhal | Design: Jagged Forest Emerald isn't on Tiffany Cartier? |
| 21:15.01 | Ressy | it is. |
| 21:15.19 | Torhal | Then you're confusing me |
| 21:15.24 | Ressy | ? |
| 21:15.25 | Torhal | Ackis: I'mma beat you. |
| 21:15.31 | Torhal | Ressy: You said it isn't. |
| 21:15.39 | Ressy | Jagged Forest Emerald is. |
| 21:15.39 | Torhal | Well, I don't know what "it" is I suppose. |
| 21:15.45 | Ressy | Sparkling Chal isn't |
| 21:16.12 | Ressy | the recipe itself has been removed |
| 21:16.16 | Ressy | http://www.wowhead.com/item=39927 |
| 21:16.45 | Ressy | Its on Timothy Jones now |
| 21:17.03 | Ressy | according to wowhead |
| 21:17.06 | Torhal | How did we even get to that recipe? |
| 21:17.06 | Torhal | lol |
| 21:17.25 | Ressy | yup. manually checked |
| 21:17.30 | Ressy | its on Timothy Jones |
| 21:18.12 | Torhal | Ok, good find but how did you get to that recipe? :) |
| 21:18.27 | Torhal | [16:07:48] <@Torhal> Look for recipe:AddVendor(28721) |
| 21:18.39 | Torhal | That isn't on Sparkling, hence my confusion :) |
| 21:18.49 | Ressy | .... |
| 21:19.08 | Ressy | ah. stormy. |
| 21:19.11 | Torhal | lol |
| 21:19.12 | Ressy | going blind tonight |
| 21:22.51 | Ackis | eat more carrots |
| 21:23.47 | Torhal | Ackis: I am seriously going to beat you. |
| 21:24.10 | Ackis | I'm in a really foul mood |
| 21:24.17 | Torhal | You took EVERYTHING out of Collectinator except for all old bad frame code and the rubbish Colour.lua file |
| 21:24.29 | Torhal | None of the ARL code is even there |
| 21:24.30 | Ackis | I haven't got that far yet |
| 21:24.42 | Torhal | That isn't how you said you were handling it >.< |
| 21:24.46 | Ackis | are you cloning the 2.0 branch? |
| 21:24.58 | Torhal | Maybe not |
| 21:25.00 | Ackis | there shouldn't even been a Colour file |
| 21:25.05 | Torhal | Still half asleep |
| 21:25.06 | Torhal | :D |
| 21:25.22 | Torhal | Oh, there are no Clones. |
| 21:26.03 | Ackis | remember you told me to do it in a branch? |
| 21:26.09 | Ackis | to retain previous history |
| 21:26.23 | Torhal | Vaguely. |
| 21:26.25 | Torhal | Meh. |
| 21:27.28 | Ackis | I've basically been goinig file by file and gutting out all the ARL code that we don't need and then modifying the existing functions to work with pets/mounts |
| 21:32.19 | Torhal | Ackis: Ok - why did you change the parameter name from spell_id to coll_id? |
| 21:33.42 | Ackis | for future use, because potentially we could add in tabards and that would not be a spell, it'd be an item |
| 21:34.24 | Torhal | We'd use another method of adding them then. |
| 21:34.51 | Torhal | Especially since we use name = _G.GetSpellInfo(col_id), |
| 21:34.59 | Torhal | Can't get spell info on an item :) |
| 21:35.07 | Ackis | I was going to deal with that part later :P |
| 21:35.16 | Torhal | Uh-huh. |
| 21:35.17 | Ackis | I was working on getting the one item in the database checked |
| 21:35.29 | Ackis | seriously I was :P |
| 21:35.38 | Torhal | You also left in profession_recipe_list |
| 21:35.49 | Torhal | And num_profession_recipes. |
| 21:36.00 | Ackis | because I don't know what they're used for just yet |
| 21:36.54 | Torhal | In ARL I added that so I didn't have to iterate through all recipes and keep doing "Is this recipe from the profession I care about?" - I just iterate through the profession-specific list |
| 21:37.53 | Torhal | Ok, since I asked and had answered, what am I doing? |
| 21:39.10 | Ackis | Core.lua -> 547 is not recognizing what's been added in to the pet DB (one entry right now) |
| 21:40.05 | Torhal | Collectinator is actually refusing to work for me since I don't have LibPetJournal-2.0 and it's complaining rather loudly about missing localizations. |
| 21:40.24 | Ackis | there should only be 5 missing localizations which I don't give a damn about right now :D |
| 21:40.41 | Ackis | but yeah you need lpj |
| 21:41.00 | Ressy | I uh... Guess you guys aren't working on arl then? :) I'mm hit up 2 more alliance cities |
| 21:41.23 | Ackis | Ressy: just talking about collectinator 2.0, I hit a wall and not sure what's going on so I wanted an expert opinion |
| 21:41.55 | Ackis | my goal is to get Col 2.0 up to the ARL code base so it actually works sexily |
| 21:42.00 | Ressy | crap. missed my boat...... screw this. Mage time |
| 21:42.18 | Ackis | lol |
| 21:42.58 | Torhal | Ressy: I'd work on ARL but Doc Brown here has me on a mad errand. |
| 21:43.42 | Ressy | shit.. I haven't assigned any of my talents for my mage. |
| 21:43.43 | Ressy | >.< |
| 21:44.30 | Torhal | lol |
| 21:53.34 | Ackis | I'm waiting to get yelled at :( |
| 21:53.40 | Torhal | Ackis: GetCompanionInfo() is the old fucking API :P |
| 21:54.40 | Ackis | what line is that on? |
| 21:55.21 | Torhal | Core.lua line 520 |
| 21:55.32 | Torhal | Plus, where is private.Player.mounts populated? |
| 21:55.39 | Torhal | Not in Player.lua |
| 21:55.53 | Torhal | Nor Core.lua |
| 21:55.56 | Ackis | mounts aren't even touched yet, my test case is with one pet |
| 21:55.59 | Ackis | the zergling |
| 21:56.39 | Torhal | Then why am I being spammed by "Not in db" which is obviously for mounts |
| 21:56.42 | Torhal | fixes. |
| 21:57.22 | Ackis | because I've got the scanning working for mounts, but I wanted to get the db integration working with pets... I did scanning for both to start and that works |
| 22:01.35 | Torhal | Just saying: If you're testing pets, don't have mounts spamming your ChatFrame :) |
| 22:01.49 | Ackis | but they don't spam my chat frame when I scan for pets |
| 22:07.28 | Torhal | Ok. |
| 22:07.53 | Torhal | You're looking for a pet in private.Player.critters |
| 22:08.06 | Torhal | private.Player.critters is always an empty table. |
| 22:08.17 | Torhal | I don't know why that table even exists. |
| 22:08.26 | Torhal | That's the reason for the flag KNOWN |
| 22:08.27 | Torhal | :D |
| 22:08.56 | Torhal | Lemme fix then commit/ |
| 22:09.08 | Ackis | please |
| 22:09.16 | Ackis | how do you do diffs in git? |
| 22:26.02 | Ackis | Torhal: are you fixing more than just that? :( |
| 22:26.14 | Torhal | git-gui |
| 22:26.16 | Torhal | And yes. |
| 22:26.19 | Torhal | had to. |
| 22:33.05 | *** join/#arl Torhal (~torhal@c-71-207-224-241.hsd1.al.comcast.net) |
| 22:33.06 | *** mode/#arl [+o Torhal] by ChanServ |
| 22:38.16 | Torhal | Ackis: Cleaned up, now tackling issue. |
| 22:38.27 | Ackis | sighs |
| 22:38.46 | Ackis | that's a lot of commits |
| 22:41.19 | Torhal | Mostly renaming |
| 22:41.34 | Torhal | A collection is a set of things, whereas a collectable is something that belongs in a collection. |
| 22:42.01 | Ackis | yeah my naming conventions were a bit wonky |
| 22:42.02 | Ackis | :P |
| 23:01.20 | Torhal | Ackis: Seriously, fuck Blizzard with WoW 5.0.4 |
| 23:01.51 | Ressy | ? |
| 23:01.55 | Torhal | http://o7.no/S1fdti |
| 23:02.04 | Torhal | "Flags removed" never prints |
| 23:02.15 | Torhal | Nor does "Iterating pet IDs" |
| 23:02.20 | Torhal | Lua is so fucking broken. |
| 23:02.38 | Torhal | No errors. |
| 23:02.46 | Torhal | It just says "You know what? Fuck this. Not processing." |
| 23:04.16 | Ressy | Have you looked at PetCounter? |
| 23:04.24 | Ressy | it works, and tells you how many of each you have. |
| 23:04.43 | Ressy | I installed it til Col 2 comes out. :) |
| 23:04.52 | Torhal | That's beside the point. |
| 23:04.57 | Torhal | Lua shouldn't ever do this :) |
| 23:05.21 | Torhal | I knew something was wonky so I just started adding print statements to see where it was dying. |
| 23:05.40 | Torhal | It prints "Removing flags" and then gives me the finger. |
| 23:05.43 | Ressy | lol |
| 23:05.45 | Ressy | i do that too |
| 23:07.17 | Torhal | Scumbag Ackis - asks for help, walks away. |
| 23:08.15 | Ackis | that's what I was doing |
| 23:08.18 | Ackis | print statements |
| 23:08.25 | Torhal | Now it "works" |
| 23:08.25 | Torhal | http://o7.no/Ru6JQq |
| 23:08.43 | Torhal | But it should be printing whether something was found or not BEFORE it says "Finished iterating" |
| 23:08.48 | Torhal | headdesks. |
| 23:08.57 | Ackis | so you're still where I was stuck eh? |
| 23:09.33 | Torhal | Except for the part where I'm actually looking at data instead of an empty table. |
| 23:10.03 | Ackis | but it's still not saying "lol I found a pet" :D |
| 23:10.11 | Ackis | you might want to add another pet to the DB btw |
| 23:10.16 | Ackis | the one I was using is a CE pet |
| 23:10.24 | Torhal | Doesn't matter |
| 23:10.40 | Torhal | It should say "I found something" or "I found nothing" |
| 23:10.47 | Torhal | Instead it just stares at me blankly |
| 23:11.08 | Torhal | Again, no Lua errors. |
| 23:11.16 | Torhal | Just "Fuck you" |
| 23:11.29 | Torhal | Oh |
| 23:11.31 | Torhal | Wait |
| 23:11.36 | Torhal | The library might be fucked. |
| 23:12.00 | Torhal | It would print nothing if LPJ:IteratePetIDs() had nothing to iterate |
| 23:15.31 | *** join/#arl Kaelten (~Kaelten@WoWUIDev/WoWAce/WoWIFA/CurseStaff/kaelten) |
| 23:16.43 | Ackis | I'm going afk for a bit Torhal (I was here the entire time just tabbed out on reddit :P) gonna lay down for a few hours or something |
| 23:17.53 | Torhal | Ackis: NOOO |
| 23:17.56 | Torhal | I think I have it |
| 23:21.31 | Torhal | Ackis: http://o7.no/S1hvZk |
| 23:21.55 | Torhal | Why it's ocurring twice, though. |
| 23:21.57 | Torhal | Meugh. |
| 23:22.32 | Torhal | Also, I had to scan twice. |
| 23:51.12 | Ressy | gotta love how dumps for JC & Blacksmithing knock me off battle.net |
| 23:52.12 | pompy | must be your connection eh? |
| 23:52.16 | pompy | doesnt happen to me. |
| 23:53.24 | Ressy | still in wow, just lose connection to the chatserver |
| 23:53.27 | Ressy | doesn't matter tho |
| 23:53.31 | Ressy | cuz my engine is frozen |
| 23:53.32 | Ressy | lol |
| 23:55.24 | pompy | yup.. gotta go through like nearly 600 recipes |
| 23:57.10 | Ressy | pushed. |
| 23:57.10 | Ressy | done |
