| 05:39.24 | *** join/#riftuidev purl (~purl@rikers.org) |
| 05:39.24 | *** topic/#riftuidev is http://pastebin.com for code snippets || PIL: http://www.lua.org/pil || Rift AddOn Policy: http://t.co/7xXd4LL || AddOn Wiki: wiki.riftui.com/ || RiftUI.com || rift.curse.com || this channel is logged, via purl || Rift goes F2P, June 12th, http://www.riftgame.com/free |
| 07:05.19 | Thorarin | I've not made sense about the coin lock logic yet.. |
| 07:05.49 | Thorarin | I get locked out at home sometimes with a static IP and no unusual transactions as you call it |
| 07:06.46 | Thorarin | I invariably get coinlocked when I log in from somewhere else tho, that part is okay |
| 08:10.55 | *** join/#riftuidev Kheldar (~Kheldar@217.108.40.234) |
| 08:12.49 | Kheldar | Hi |
| 08:36.08 | Thorarin | Heya |
| 08:39.06 | *** join/#riftuidev Adelea (~martin@109.224.129.157) |
| 08:51.29 | Kheldar | Hey Adelea |
| 08:51.47 | Kheldar | Did you see my answer about Corout yesterday ? |
| 08:51.59 | Adelea | yes |
| 08:52.22 | Adelea | though - my ponderings/implementation were based on the early watchdog behaviour, where the time was shared |
| 08:52.35 | Adelea | now, when each addon gets its own slice, its probably less of an issue |
| 09:01.47 | Kheldar | Ah, i'm not old enough in rift to know that :p |
| 09:03.01 | Kheldar | I'm think i will add a isRunning() function or something like that |
| 09:03.22 | Kheldar | so you can check if you're in a corout or not |
| 09:03.55 | Kheldar | and then avoid the errors throws by corout.check() when you're not |
| 09:04.50 | Kheldar | i'm just wondering if i should automatically check it in corout.check(), but i guess not |
| 09:05.06 | Kheldar | errors are useful sometimes |
| 09:05.45 | Adelea | you need to pass any errors back to the addon using the coro |
| 09:05.59 | Kheldar | yes of course, this is done |
| 09:06.48 | Kheldar | Oh, well you're right it's the same in this case |
| 09:07.09 | Kheldar | so i will just let people the ability to prevent them |
| 10:38.54 | Ivnedar | I'm definitely getting quicker at calibrating this each time I log in Adelea :D |
| 10:39.13 | Adelea | yeah, it gets quicker for sure. |
| 10:39.34 | Adelea | is it too much of a pain in the ass to have to do? or do you trhink ppl can live with it, for the functionality it offers? |
| 10:39.53 | Ivnedar | It's fiddly getting the mousewheel to apply to the buttons instead of the map |
| 10:40.09 | Ivnedar | I think people will complain about having to do it more than once |
| 10:40.23 | Adelea | if they dont change map scale, they wont have to =P |
| 10:40.29 | Adelea | or at least they shouldnt |
| 10:40.34 | Ivnedar | hrrm, ok |
| 10:40.37 | Adelea | you think making the buttons bigger would help ? |
| 10:40.42 | Adelea | although that said |
| 10:40.46 | Ivnedar | whenever I log in, the green calibration target is top left |
| 10:40.50 | Adelea | i do often have to do it once i log in |
| 10:41.02 | Adelea | that is just a display problem - remember this is still a beta =) |
| 10:41.07 | Ivnedar | so I open the map and click calibrate, then line them up, try to not move the mouse when scrolling or it starts zooming the map |
| 10:41.14 | Ivnedar | moving the buttons outside of the map ui may help |
| 10:41.26 | Ivnedar | so having it slightly off doesn't zoom the map and make me strat over |
| 10:41.35 | Ivnedar | I know :D |
| 10:41.41 | Ivnedar | sometimes I log in and the green thing isn't there, and it's all happy |
| 10:42.04 | Ivnedar | Just passing on observations in case any are useful :) |
| 10:42.34 | Adelea | with the green thing in the top left, just opening and closing the map should remove it - as the map is forced then to update what is displayed |
| 10:42.43 | Ivnedar | ah, right |
| 10:42.44 | Adelea | i just need to fix it being displayed at all sometimes =) |
| 10:43.22 | Adelea | so: If I make the buttons a bit bigger, and then attach them to the bottom of the map, with an offset, that would help? |
| 10:43.29 | Ivnedar | I think so |
| 10:43.30 | Ivnedar | It'd help me |
| 10:43.41 | Adelea | be less clumsy =P |
| 10:43.52 | Adelea | i know what you mean though, it does happen to me too from time to time |
| 10:44.02 | Ivnedar | When I mouse over one, the icon is either the cursor or the resize icon |
| 10:44.18 | Adelea | i could also add some visual feedback that your mouse is over a button, say changing color to green from white ? |
| 10:44.27 | Ivnedar | If it's resize, it stays resize even when mousing over the icon, and if I try resizing while it's like that, it zooms the map instead |
| 10:45.20 | Ivnedar | Though now it isn't doing it when trying again |
| 10:45.28 | Ivnedar | I'll try to pay more attention when logging in next |
| 10:45.40 | Adelea | i think it depends on which direction you come from onto the buttons, and how quick you move |
| 10:45.50 | Ivnedar | Yeah, that's what I'm playing with now |
| 10:46.07 | Adelea | at least if ppl find it a pain in the ass, they can just click the button in the top left and turn it off =P |
| 10:47.43 | Ivnedar | I had raretracker data being shown on redar's hud at one point, felt way too clunky to push out |
| 10:48.23 | Adelea | how do you mean ? |
| 10:49.00 | Ivnedar | on a fresh zone, the hud was covered in icons around in a circle since I was clipping their distance |
| 10:49.11 | Ivnedar | wasn't too bad when I only had 1-2 to go in a zone |
| 10:49.42 | Ivnedar | maybe cluttered is a better word to use than clunky |
| 10:56.08 | Adelea | If you change lines 281-283 of LibMapNotesUI.lua to : |
| 10:56.09 | Adelea | LM.UI.calibratescale[3]:SetPoint("TOPRIGHT", UI.Native.MapMain, "BOTTOMRIGHT", -10,4) |
| 10:56.10 | Adelea | LM.UI.calibratescale[2]:SetPoint("TOPRIGHT", LM.UI.calibratescale[3], "TOPLEFT", -10,0) |
| 10:56.10 | Adelea | LM.UI.calibratescale[1]:SetPoint("TOPRIGHT", LM.UI.calibratescale[2], "TOPLEFT", -10,0) |
| 10:56.15 | Adelea | does that work any better ? |
| 10:58.28 | Ivnedar | much better |
| 10:58.47 | Ivnedar | for me anyway |
| 11:02.39 | Adelea | OH! ive just got what you meant earlier - you had imported rare data into the redar display. i thought somehow that the data was overlaying somehow |
| 11:02.48 | Ivnedar | oh, no, sorry :) |
| 11:03.02 | Adelea | what you said was exactly right - just my interpretation =) |
| 11:03.21 | Ivnedar | I was pulling in the coords from rare tracker's data, showing them on the redar hud and clipping them at a max distance from the centre |
| 11:05.02 | *** join/#riftuidev Kheldar_ (~Kheldar@ANantes-557-1-143-184.w2-1.abo.wanadoo.fr) |
| 12:33.08 | *** join/#riftuidev bry (51694aad@gateway/web/freenode/ip.81.105.74.173) |
| 12:33.37 | bry | Hey can anyone help me with what I assume is a quick problem please? |
| 12:39.36 | Ivnedar | ask away |
| 12:41.47 | bry | Well im trying to make a simple personal addon for logging XP per session but before I've even got started I can't really seem to find the correct way to log XP gained? |
| 12:42.05 | Adelea | http://portal.wildtide.net/RiftAPI/Category/Experience |
| 12:42.20 | Adelea | Event.Experience.Accumulated fires each time you gain xp |
| 12:43.43 | bry | Aaaah, thank you, I have probably every page on that site open currently except that, couldn't see for looking, cheers! :) |
| 13:42.00 | Kheldar | Adelea: i'm thinking of removing the label functionnality on corout, cause it's actually quite useless |
| 13:42.43 | Kheldar | the problem is it represents a breaking change with the parameters function, so i don't know if it's still a good idea |
| 13:43.24 | Kheldar | but on another hand, there is only 2 downloads of corout at the moment, so it's not awful either |
| 13:43.37 | Kheldar | 2 including you ;) |
| 13:45.39 | Kheldar | And removing it will be easier to use for every single corout so it represent some % of code needed |
| 13:46.30 | Adelea | im not sure how accurate the download counts are.. it will be static for hours, then jump suddenly - so the updates could be delayed |
| 13:48.22 | bry | I understand the event returns a table, which index actuallys contains the integer with gained xp in? [1] and [2] return a nil value |
| 13:48.54 | Kheldar | yes and i can't update the new version before testing it, so not before something like 3 hours |
| 13:50.16 | Kheldar | bry: you need to look at the Members part of the API |
| 13:51.05 | Kheldar | for example for your event, you will need to query result["accumulated"] |
| 13:51.54 | Adelea | take a look at another addon that handles experience, and see what they do - that may help explain whats going on |
| 13:52.01 | Adelea | f/ex the xpbar in gadgets |
| 13:52.14 | Adelea | since that can handle normal xp, pa xp, prestige etc |
| 13:54.37 | bry | Ah I was looking at RiftMeter earlier to try deduce where to find the XP gain, ok, I do have a function that accepts the accumulated xp and returns it for the ui function, if I set no index it tells me I cant use a table in the text window, fair enough, try to extract the value and I get nil value, I but ill go check that addon best way to learn :) |
| 13:59.45 | Adelea | How is your coro library intended to be used Kheldar? |
| 14:00.06 | Adelea | is it, every time you have a job that could run a while, you package it into a function and run it on its own |
| 14:00.15 | Adelea | so after a while you may have started several jobs |
| 14:00.40 | Adelea | or create a single function, which you run and the lib. takes care of waking it up and checking for dead etc ? |
| 14:02.04 | Kheldar | I have a single function witch care of waking and and properly end the coroutines |
| 14:07.56 | Adelea | i didnt mean it like that =) let me try again! |
| 14:07.56 | Kheldar | well, it exactly works like that : create the coroutine, attach event, call the corout |
| 14:09.58 | Kheldar | then the corout do it jobs |
| 14:10.05 | Kheldar | and when it's done i detach the event, treat the error, etc |
| 14:18.10 | Adelea | this is how i do it : http://pastebin.com/nz3Q2w9a |
| 14:18.10 | Kheldar | so i was mainly asking a label for the event, but it's a pain for nothing since i can easily produce a unique label for each corout |
| 14:18.10 | Adelea | so i creat one 1 coro, which stays running the whole life of the addon, and I set some status flags when i need it to wake up |
| 14:18.11 | Adelea | or in the case of using your lib, would I fire up a new coro instead, each time I had something I wanted to run |
| 14:18.13 | Adelea | or, could i do it my way, using your lib? which since its in a while true loop, would never end, so wouldnt be cleaned up ? |
| 14:18.14 | Adelea | the only reason i did it my way, is so that i wouldnt have to manage dozens of coro's.. .which if the lib will do for me anyway isnt a concern |
| 14:18.21 | Kheldar | so... your aim is to run your loop each time you want ? |
| 14:18.22 | Adelea | when it needs it |
| 14:18.26 | Kheldar | if you use corout(JB.UpdateHandler), it will works in the same way |
| 14:18.27 | Kheldar | but i just figured out i didn't tried to use coroutine.yield() in the function |
| 14:18.28 | Kheldar | although it should works |
| 14:18.42 | Kheldar | but i wonder why you're not removing your loop and the yield |
| 14:19.10 | Kheldar | oh, got it |
| 14:19.43 | Kheldar | to be able to run it again during the same process ? |
| 14:19.51 | Adelea | yeah. |
| 14:20.01 | Adelea | the things that i want to run in a corout can happen over and over |
| 14:20.22 | Adelea | so all i do is set the relevant flag, and then the corout will run it next frame - and i dont need to worry about creating yet another coro and managing it |
| 14:21.03 | Adelea | i figure while true do if a elseif b elseif c end coro.yield() end is pretty cheap.. at least i hope so =P |
| 14:22.33 | Kheldar | but you just could call your function out of a coro, without the loop and the yield, isn't it ? |
| 14:22.47 | Kheldar | i mean your flags looks to be globals |
| 14:23.03 | Adelea | the flags are yes, but the work the functions do, needs to yield |
| 14:23.19 | Adelea | whenever i start see watchdog warnings, then the task gets put into a coro |
| 14:23.46 | Adelea | and coroutine.yield() when you arent in a coro is bad =P |
| 14:25.10 | Kheldar | So you just want JB.UpdateHandler() to be in a coro to have the function executed in a coro ? |
| 14:25.53 | Kheldar | the other functions* |
| 14:26.02 | Adelea | kind of. nothing that JB.UH() calls is ever called outside of JB.UH(), so the functions that called are all in a coro so can yield |
| 14:26.32 | Adelea | ive seen some of them take 8 frames to complete in the past |
| 14:28.19 | Kheldar | i think my library is actually perfect to avoid the need of your function |
| 14:28.56 | Adelea | i could certainly move parts of it out, thats for sure. as i said, the ony reason i did the single handler is to avoid having to maintain multiple coro's |
| 14:29.30 | Kheldar | it's understandable |
| 14:29.53 | Kheldar | so you're checking inspect.Time.Watchdog in each functions ? |
| 14:30.24 | *** join/#riftuidev Imhothar (miranda@ppp-188-174-176-90.dynamic.mnet-online.de) |
| 14:31.57 | Kheldar | and yes i think it was quite a pain to handle noe coro, so i wouldn't handle more of then, so i write the library ^^ |
| 14:35.16 | Adelea | idd! next time i need to play with coro's ill try your lib.. not sure i want to try and retrofit it to an existing addon though =) |
| 14:37.01 | Kheldar | eheh ^^ |
| 14:37.31 | Kheldar | well, for this example, to adapt it to the lib |
| 14:37.41 | Kheldar | you should do |
| 14:38.03 | Kheldar | corout(functionToBeInACoro) |
| 14:38.21 | Kheldar | and in your function, each time you check the watchdog |
| 14:38.27 | Kheldar | corout.check() |
| 14:38.36 | Kheldar | to be the best use |
| 14:40.16 | Kheldar | you can also do corout(JB.UpdateHandler) but in this case the only thing interesting is to remove the second part of your pastebin |
| 14:41.19 | Kheldar | so this is not profitable |
| 14:43.20 | Kheldar | One thing you could win using corout will be that shorter task will finish earlier |
| 14:43.45 | Kheldar | cause each coro can take some time |
| 14:44.20 | Kheldar | so shorter ones wil finish quickly even if they are started later in the code than the heavy ones |
| 14:45.00 | Kheldar | but maybe you are sorting your's :) |
| 14:45.52 | Thorarin | Hmm, trying to execute multiple coroutines sounds strange to me |
| 14:46.54 | Kheldar | Well, i wasn't sure how it would works, so i tried with two different coro |
| 14:47.15 | Kheldar | and each have the reset timer of the watchdog |
| 14:48.53 | Thorarin | Instead of executing two of them, executing the most important one would generally make more sense |
| 14:49.36 | Thorarin | Of course I don't know exactly what you want to achieve, it may be valid |
| 14:50.13 | Kheldar | You're making a good point |
| 14:50.22 | Kheldar | in my case it was only for tests |
| 14:50.30 | bry | when I try to set the text now, it sets it as a "Error: Incorrect function usage. Parameters: (Text: XPLogger.xpbox.0x201c5508), 252025" |
| 14:50.53 | Adelea | :SetText(tostring(num)) ? |
| 14:51.07 | bry | my function is .. http://pastebin.com/GmD89zxP |
| 14:51.18 | Kheldar | anyway, you will have 0.1sec for each coro, so even trying to exec only one won't change something imo |
| 14:51.38 | Adelea | yeah, you prob need to tostring() the numeric value |
| 14:51.46 | Thorarin | 0.1 for each? |
| 14:51.50 | Kheldar | indeed |
| 14:51.51 | Imhothar | http://hackertyper.com/ |
| 14:52.04 | Thorarin | Uhh, no? :) |
| 14:52.17 | Adelea | ROFL. |
| 14:52.19 | Kheldar | ohhh |
| 14:52.27 | Thorarin | Unless you circumvent the watchdog, which is a very bad idea |
| 14:52.29 | Kheldar | ok i just undertood that it was the next frame |
| 14:52.30 | Kheldar | nvm |
| 14:53.02 | Thorarin | If you execute 1 each frame, but rotate them, I guess |
| 14:53.23 | Kheldar | i'm probablu overheating ^^ |
| 14:54.20 | Kheldar | hum the strange thing is corouts were executing like that : |
| 14:54.52 | Kheldar | coro1 - coro1 - coro1 - coro2 - coro2 - coro2 - coro2 - coro1 |
| 14:54.54 | Kheldar | etc |
| 14:55.10 | Kheldar | i wonder why |
| 14:56.25 | Imhothar | wonder why they call the item "REX: Rift Exchange" instead of just "Rift Exchange" or "REX" |
| 14:56.33 | Imhothar | that's like redundancy all over |
| 14:58.35 | Kheldar | it was REX initially and they add Rift Exchange later to avoid confusion, isn't it ? |
| 14:58.48 | Ivnedar | because no one could find them on the AH |
| 14:59.12 | Ivnedar | They were originally called REX in all the promotional stuff and Rift Exchange was the name of the actual item |
| 14:59.18 | Ivnedar | search on the ah for REX and you get nothing back |
| 14:59.33 | Imhothar | sigh |
| 14:59.38 | Imhothar | the human factor again |
| 15:12.30 | *** join/#riftuidev bry (51694dab@gateway/web/freenode/ip.81.105.77.171) |
| 15:12.42 | bry | Accumulated seems to show how much XP I've gained in total so far is there a way to see just the last XP gain? do I have to take it away from the total? |
| 15:14.12 | bry | All I wish to do is basically press a button record how much xp I get from say an aoe pull then stop the counter and see how much that pull just netted me |
| 15:15.01 | Kheldar | well, you just ehave to get the accumulated xp at start and a end |
| 15:15.07 | Kheldar | and do end - start then ;) |
| 15:18.38 | bry | blar its the concept of it I cant get my head around, thanks I'll soldier on :) |
| 15:21.11 | bry | Is there an Frame typ thats just an invisible container? |
| 15:21.47 | bry | RiftWindow has all the UI stuff, I know I can use text but I want to contain curr + gained in one place |
| 15:22.17 | bry | or is that treating it too much like css and divs? |
| 15:22.18 | Imhothar | "Frame" is just an empty invisible container where you put other frames into |
| 15:22.36 | Adelea | well - rather than track it all, if you are doing just start/end, then Inspect.Experience() at the start. Record the results. Inspect.Experience() at the end, and compare. |
| 15:22.46 | Imhothar | however "Mask" is the only UI type which restricts its children itnto its client area |
| 15:23.29 | Adelea | though if you level up, the 2nd val. will be lower, but i think there is a required value, so you could do required - current, to get the amount required to level |
| 15:23.32 | Imhothar | bry: css and divs are fundamentally different than typical UI frameworks |
| 15:25.32 | bry | Not saying you're wrong because this is just something to learn in my spare time and I know noooothing :P, but setting height and width on certain frame ID's seems just like CSS in that sense |
| 15:25.53 | Imhothar | that's because everything needs a size |
| 15:26.35 | Imhothar | but there are no classes or automatic styles or whatever |
| 15:26.47 | Imhothar | every frame is an isolated object for which you have to set all properties yourself |
| 15:27.08 | Imhothar | and there is no automatic layout, you have to SetPoint() all frames manually as well |
| 15:27.30 | Imhothar | once you did that however, frames keep their spatial relation to each other |
| 15:44.30 | bry | so confused now ._. |
| 16:04.09 | bry | Can I put current inside Inspect.Exp to get current xp? |
| 16:04.27 | bry | eg currxp = Inspect.Experience(current) |
| 16:06.40 | *** join/#riftuidev Kheldar (~Kheldar@ANantes-557-1-143-184.w2-1.abo.wanadoo.fr) |
| 16:08.22 | bry | or is it currentxp = Inspect.Experience() currentxp[1] = currXP |
| 16:15.24 | Imhothar | http://portal.wildtide.net/RiftAPI/Details/Inspect_Experience |
| 16:15.25 | bry | could anyone tell me if Im doing this right? http://pastebin.com/HvjHRQ6Y Im finding it really hard to find a bit more beginner friendly documentation |
| 16:15.41 | bry | Yes, that to me doesn't say much |
| 16:16.35 | Imhothar | the call returns a table and the table has the members listed on that page |
| 16:16.57 | bry | For example how do I get the members? Ive tried as you would with a table and supplying Inspect.Experience(current) not sure how else, looking at lua tuts not that useful |
| 16:17.26 | Imhothar | the table is the function's return value |
| 16:17.59 | bry | but I cant then pull the current accumulated XP from that, or atleat *I* cant lol |
| 16:18.58 | bry | I could using the vent |
| 16:18.58 | Imhothar | local xp = Inspect.Experience() |
| 16:18.59 | Imhothar | print(xp.accumulated) |
| 16:19.21 | bry | ooooooooh thats how you use members, thank you! |
| 16:19.34 | bry | Its frustrating knowing im looking at what I need but no idea how to use it |
| 16:20.27 | Imhothar | http://www.lua.org/pil/contents.html |
| 16:20.41 | Imhothar | it's for Lua 5.0 and we're using 5.1 but only most of it still aplies |
| 16:20.50 | Imhothar | -only |
| 16:20.59 | bry | can feel about 20- minutes frustration going aaaaah |
| 16:21.12 | bry | http://www.lua.org/pil/2.5.html Was what I was looking at, thank you either way :) |
| 16:21.41 | Imhothar | quote: |
| 16:21.41 | Imhothar | print(a.x) -- same as print(a["x"]) |
| 16:22.07 | Kheldar | if you want a well written introduction to lua, you can check this too : http://nova-fusion.com/2012/08/27/lua-for-programmers-part-1/ |
| 16:22.21 | Kheldar | found it very usefull when i strated |
| 16:23.33 | bry | Thanks man I'll have a look, ill try not to be too annoying! |
| 16:31.53 | Kheldar | Thorarin: i juste tested the each corout have 0.1sec thing for the same addon |
| 16:32.18 | Kheldar | and it effectively run each corout 0.1sec if needed |
| 16:33.52 | Kheldar | of crouse, the game is just very slow, so it's not a good idea to abuse of it |
| 16:38.28 | Thorarin | Like I said, not a good idea :) |
| 16:50.36 | *** join/#riftuidev seebs (~seebs@home.seebs.net) |
| 16:50.37 | *** mode/#riftuidev [+o seebs] by ChanServ |
| 17:00.16 | bry | When is the best time to take their xp after log in, Addon.Load.End or another? |
| 17:10.33 | Thorarin | Not sure, it might require full player unit availability |
| 17:30.59 | bry | My update function seems to throw 'attempt to index 'updateXP' a number but I cant seem to get to convert to a string and re-insert into the table |
| 17:31.00 | bry | function update (hEvent, accumulated) local updateXP = accumulated sessionXP["currentxp"] = tostring(updateXP.accumulated) end |
| 17:40.50 | seebs | Try tostring(updateXP). |
| 17:40.58 | seebs | Oh, wait. *thinks* |
| 17:41.48 | seebs | What's calling this update function? Is it an event handler? If so, the "accumulated" name is confusing, I'd rename it something else just to make it easier to read. |
| 17:43.01 | bry | Event.Experience.Accumulated triggering is calling the update function :) |
| 17:43.40 | bry | http://pastebin.com/37FtQqS6 - What I've got so far there |
| 17:44.07 | bry | Everything works even the XP gained since logon just the update function that doesn't, Its not important I'd just like to know what im doing wrong |
| 17:45.33 | Imhothar | first you should decide whether you want to use the a.x or a["x"] syntax and keep it consistent |
| 17:45.47 | Imhothar | that may make the code easier to read |
| 17:46.39 | bry | Sorry haha the way I do it now just feels right |
| 17:47.21 | Imhothar | line 34f: |
| 17:47.22 | Imhothar | local updateXP = accumulated |
| 17:47.22 | Imhothar | sessionXP["xp"] = updateXP.accumulated.tostring() |
| 17:47.39 | Imhothar | accumulated is a number, therefore updateXP is a number as well |
| 17:47.42 | Imhothar | you can't index numbers |
| 17:47.57 | bry | im trying to turn it into a string . |
| 17:48.02 | Imhothar | tostirng(x) |
| 17:48.13 | bry | doesnt work tried too |
| 17:48.37 | bry | sessionXP["xp"] = tostring(updateXP.accumulated.tostring) for exmaple |
| 17:48.55 | Imhothar | updateXP is still a number you can't index |
| 17:49.12 | Imhothar | sessionXP.xp = tostring(accumulated) |
| 17:49.50 | Imhothar | accumulated is of type "number", you cannot write a.x if a is a number |
| 17:51.44 | bry | but accumulated is a table |
| 17:52.23 | Imhothar | quote: attempt to index 'updateXP' a number |
| 17:53.45 | Imhothar | because of "local updateXP = accumulated" in line 34 "updateXP" has the same type as "accumulated" and since Lua is telling you "updateXP" is a number so must be "accumulated" |
| 17:58.54 | bry | right, thanks, spending too long looking and not enough thinking |
| 18:34.47 | bry | Hey again...I've tried setting a global variable to true eg "on = true" then do a if on == true then update else --do nothing-- end but it doesn't seem to be working? Xp still gets recorded |
| 18:35.38 | seebs | It's possible that something else is hitting it, maybe? Because, well, global. |
| 18:35.54 | Imhothar | Did I smell the word "global'? |
| 18:36.03 | seebs | I would tend to recommend a local. Assuming your addon is a single file, a "local on = true" near the top of the file will give you a variable which is accessible throughout your addon. |
| 18:36.08 | bry | well its local to the file |
| 18:36.14 | bry | it has local infront of it |
| 18:36.17 | bry | its just not in a function |
| 18:36.23 | bry | sorry probably used wrong term |
| 18:36.44 | seebs | Is the declaration of it before any code that refers to it? |
| 18:36.58 | seebs | e.g.: function foo() on = true end; local on = false; foo() |
| 18:37.01 | bry | its right atop, the function that uses its is last |
| 18:37.11 | seebs | This would not set the "local on" to true. |
| 18:37.45 | bry | "local on = false " near top before function |
| 18:37.53 | bry | function recordXP () on = true end |
| 18:38.05 | bry | function to turn it to on is still before the If statement too |
| 18:38.48 | bry | spent all this time getting it to update the gained xp now I want to make it stop...technology! |
| 18:40.39 | bry | http://pastebin.com/FrHh0X4R all in one in case im being a confusing nub |
| 18:43.07 | Imhothar | recordXP and stoprecord are never called |
| 18:43.14 | bry | Ha just noticed another problem, but I can fix that one! |
| 18:43.29 | bry | Yeah, but by default after load up it shouldnt be recording XP |
| 18:43.43 | bry | not until I do the slash command, of which Im to sort out |
| 18:44.53 | Imhothar | some error messages would really help |
| 18:45.09 | Imhothar | "doesn't work" just isn't enough |
| 18:45.44 | bry | thats the thing there is none |
| 18:46.16 | bry | It should'nt be recording the xp because i've set the variable to true, so why is it stil counting xp? |
| 18:46.48 | seebs | bry: One thing to do for things like this is add print statements so you have more details. |
| 18:46.50 | bry | should it not be going straight to the update function else, which does nothing? |
| 18:50.29 | bry | Imothar, I'm not trying to have someone do this for me, nor am I trying to annoy you, I try to say whats wrong to the best of my ability to describe it, Seebs I will try have it print what the value equates to on load up, thanks :) |
| 18:50.47 | seebs | Hmm. sessionXP is a global, btw. You should probably make it local. |
| 18:51.32 | seebs | Oh, also. Your command.slash.register stuff will work now, but it'll stop working around 2.5ish because of changes to the event model. |
| 18:51.54 | bry | Ok will the updated API be released? |
| 18:51.59 | seebs | It is already. |
| 18:52.04 | seebs | There's two event APIs now. |
| 18:52.09 | seebs | I see what's happening. |
| 18:52.16 | seebs | Command.Event.Attach(Event.Experience.Accumulated, showxpGained, "updating gaiend XP") |
| 18:52.16 | seebs | Command.Event.Attach(Event.Experience.Accumulated, update, "updating current XP in sessionXP table") |
| 18:52.18 | bry | ah so i've just done it the crap way, Ill fix that then :) |
| 18:52.46 | seebs | Both of these will get run on every event. |
| 18:53.01 | seebs | So update() is checking whether on == true. |
| 18:53.26 | seebs | But showxpGained() isn't, and I think that's what's running all the time. |
| 18:53.44 | bry | Aaaaah |
| 18:53.45 | bry | Yes |
| 18:53.46 | bry | I see |
| 18:53.48 | bry | thank you |
| 18:54.02 | bry | Its updating UI but not the table its stored in |
| 20:49.13 | *** join/#riftuidev Imhothar_ (miranda@ppp-188-174-151-231.dynamic.mnet-online.de) |
| 21:24.59 | *** join/#riftuidev Solsis02 (~Solsis00@ip98-186-89-123.fv.ks.cox.net) |
| 21:27.17 | *** join/#riftuidev ZorbaTHut (tocohl@maximilian.pavlovian.net) |
| 21:27.17 | *** mode/#riftuidev [+o ZorbaTHut] by ChanServ |
| 22:07.54 | *** join/#riftuidev Noshei (~quassel@c-24-9-171-99.hsd1.co.comcast.net) |
| 22:59.47 | *** join/#riftuidev Noshei (~quassel@c-24-9-171-99.hsd1.co.comcast.net) |
| 23:21.39 | *** join/#riftuidev seebs (~seebs@97-92-17-178.dhcp.stcd.mn.charter.com) |
| 23:21.40 | *** mode/#riftuidev [+o seebs] by ChanServ |