These are chat archives for arenanet/api-cdi

29th
Jan 2016
windwarrior
@windwarrior
Jan 29 2016 00:00
what language doesnt have cURL bindings
Archomeda
@Archomeda
Jan 29 2016 00:01
The following query conditions could not be considered due to the wikis restrictions in query size or depth huh, pretty restrictive are we :(
smiley
@codemasher
Jan 29 2016 00:01
lol
maybe you might need a bot account
Archomeda
@Archomeda
Jan 29 2016 00:04
just wow
my query can just be as long as matching pages with THREE skills
that will bring me down to 400 requests, from 1200
at first i just got a generic 404 page
new patch incoming @Iye?
is there
Archomeda
@Archomeda
Jan 29 2016 00:08
well, the api is timing out for me
looks like the backend servers are patching
I need to make those timeouts re-use the old cached values so it's less obvious
Archomeda
@Archomeda
Jan 29 2016 00:10
i don't really care about an incoming patch being obvious or not, it's just annoying that the api throws me a 502 :P
windwarrior
@windwarrior
Jan 29 2016 00:10
so lets have it throw 400, and call it your fault then
Archomeda
@Archomeda
Jan 29 2016 00:10
actually, why is the api throwing a 502 in HTML instead of JSON?
Pat Cavit
@tivac
Jan 29 2016 00:11
because the API isn't responding, so the load balancer barfs out a 502
Wouldn't the API get ErrTimeout?
Or does it not get a timeout because there's no providers for the content service?
Pat Cavit
@tivac
Jan 29 2016 00:12
the API's timeout value is prooooooobably the same as the load balancers
(which we should fix)
I'll have to adjust that when I make it re-use the cache
yeah
Archomeda
@Archomeda
Jan 29 2016 00:13
oh well, can't wait for that then :)
Archomeda
@Archomeda
Jan 29 2016 00:45
huh, apparently one skill has been added
wait what
which one?
Archomeda
@Archomeda
Jan 29 2016 00:57
i have no clue, i mentioned earlier that i was iterating over 1267 skills, and now my output tells me 1268
dang :/
Archomeda
@Archomeda
Jan 29 2016 00:57
or there might be something wrong
eh
Pat Cavit
@tivac
Jan 29 2016 00:58
time to start writing diffing logic into the API servers lmao
Archomeda
@Archomeda
Jan 29 2016 00:58
i did add a prototype to an array a while ago, maybe that is counted in .length too
Js2Diff4U
.prototype is not enumerable unless you added it to the wrong place
I always get .prototype and .__proto__ confused though so don't look at me
Archomeda
@Archomeda
Jan 29 2016 00:59
well, i added a function to Array.prototype
two actually
that wouldn't affect .length
Archomeda
@Archomeda
Jan 29 2016 01:01
then either i'm just blind and it was 1268 all along, or a skill has been added
given Array.prototype.foo = "bar";, [].foo is effectively an alias for [].__proto__.foo
Archomeda
@Archomeda
Jan 29 2016 01:01
but i don't have a previous version
wasn't there just a patch or something
I wouldn't doubt a skill was added
or some bug in my code was introduced
introduced/exposed by changed content
Pat Cavit
@tivac
Jan 29 2016 01:02
content :ghost: :boom:
basically by using the API you have entered my personal castle of madness and waive all claims to sanity
Archomeda
@Archomeda
Jan 29 2016 01:02
hahaha
I should get legal to add that to the TOU
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:03
LOL
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:09
here's your new skill (probably)
Mesmer
Feedback: Fixed a bug that caused this skill to use ground targeting while submerged. When used underwater, this skill will now cast on your current target.
seperate version for underwater and land
yep, that would do it.
Archomeda
@Archomeda
Jan 29 2016 01:10
ah, sweet patch notes
Bugfixes:
  • Items
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:10
lol
Archomeda
@Archomeda
Jan 29 2016 01:10

Fixed an issue that allowed players to kill the Mossman in the Fractals of the Mists with underwater combat.

hahaha, this was all over reddit

Edgar Doiron
@Coffee4cr
Jan 29 2016 01:10
yeah noticed that
uh oh, I wonder which route they took for that mossman fix
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:11
@Archomeda the funny thing is, that was fix a long time ago and came back recently, no?
Archomeda
@Archomeda
Jan 29 2016 01:11
yup
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:11
doesn't matter
now it's on top of the tree
Archomeda
@Archomeda
Jan 29 2016 01:12
how many times has mossman been fixed to prevent unintended methods to kill him? XD
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:12
next time will be.
"Mossman insta dies when he sees a player"
windwarrior
@windwarrior
Jan 29 2016 01:13
itemizing items
that is very meta
"Mossman pulses a 2k AoE that apples 10 stacks of agony every time he's hit and can't find a path to player"
windwarrior
@windwarrior
Jan 29 2016 01:13
and once again my client lost all credentials
fighting him on final destination right
Archomeda
@Archomeda
Jan 29 2016 01:14
don't forget that he strips players from armor and AR like with the underwater fractal when he eats you
that's basically an instakill :p
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:15
that's been fixed
Archomeda
@Archomeda
Jan 29 2016 01:15
yeah, i know, still funny :D
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:15
but by fixed, it means they moved the bug feature elsewhere
no strikethrough?
Archomeda
@Archomeda
Jan 29 2016 01:15
strikethrough
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:15
:(
Archomeda
@Archomeda
Jan 29 2016 01:15
works for me
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:15
how?
windwarrior
@windwarrior
Jan 29 2016 01:15
its ~, not -
why does no one implement markdown correctly, I don't get it
Archomeda
@Archomeda
Jan 29 2016 01:16
rofl, those edits
windwarrior
@windwarrior
Jan 29 2016 01:16
dancy dash dancy dash yourtext dancy dash dancy dash
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:16
@lye everyone has their own markdown :(
hmm, actually, Gruber's original spec didn't include strikethrough: https://daringfireball.net/projects/markdown/syntax#em
so I forgive gitter.
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:17
we should have a markdown showdown!
windwarrior
@windwarrior
Jan 29 2016 01:17
no, thats github's flavour of markdown
showdown is already the name of a markdown parser IIRC
windwarrior
@windwarrior
Jan 29 2016 01:18
those last few files are big :O
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:18
lol
wtf? 128mb patch?
windwarrior
@windwarrior
Jan 29 2016 01:18
well, it has items
what did you expect
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:18
lol
Archomeda
@Archomeda
Jan 29 2016 01:18
are you certain you are not downloading 1 file over and over again?
has happened to me before
windwarrior
@windwarrior
Jan 29 2016 01:19
dont know?
Archomeda
@Archomeda
Jan 29 2016 01:19
if i had to believe the client, that 1 file would've been over 1GB
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:19
let's wait, maybe that_shaman will datamine something new
windwarrior
@windwarrior
Jan 29 2016 01:19
I got my file!
Archomeda
@Archomeda
Jan 29 2016 01:19
then again, during that time the CDN was acting up
windwarrior
@windwarrior
Jan 29 2016 01:19
I have gigabit internet
Archomeda
@Archomeda
Jan 29 2016 01:19
wasn't that_shaman on a break?
windwarrior
@windwarrior
Jan 29 2016 01:19
but it still took me minutes to download the patch #firstworldproblems
@windwarrior that's not how pipes work
Archomeda
@Archomeda
Jan 29 2016 01:20
i still haven't downloaded the last 4 patches completely, yay for content streaming
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:20
@Archomeda that's what I thought too, but he's on a ????forget the name of that cat??? break
windwarrior
@windwarrior
Jan 29 2016 01:20
sorry?
when there's too many people putting trucks into one end of the pipe, it can take weeks for them to reach their destinations
windwarrior
@windwarrior
Jan 29 2016 01:21
right
(sorry, this is an old american joke)
This message was deleted
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:21
@Archomeda Neko Atsume
Archomeda
@Archomeda
Jan 29 2016 01:21
@Coffee4cr yeah, he was referring to something like that, but i seriously didn't know what he was talking about, so i assumed he was just on a break
Edgar Doiron
@Coffee4cr
Jan 29 2016 01:22
I'm glad he isn't, it's the next best thing to patch days :D
Archomeda
@Archomeda
Jan 29 2016 01:22
haha
windwarrior
@windwarrior
Jan 29 2016 01:22
he made a joke I believe
he reclarified
but we shall see
Archomeda
@Archomeda
Jan 29 2016 01:24
hmm, alright
Archomeda
@Archomeda
Jan 29 2016 01:43
i still don't know what to make of it, i think i'm missing the joke actually regarding that cat
but oh well, time to head to bed
parsing the wiki for information regarding skills, and trying to match them with the api is not as straightforward as i would've thought
the API data needs a lot of love
I'm gonna try to start working through those skill issues tomorrow
Archomeda
@Archomeda
Jan 29 2016 01:45
i found a few more minor things
i think some transformation skills are missing
that ... wouldn't surprise me
Archomeda
@Archomeda
Jan 29 2016 01:45
and some skills don't have a profession
to get those I'm reading the actual skill scripts :/
uhh what
that seems totally broken
Archomeda
@Archomeda
Jan 29 2016 01:46
some ele and necro skills i believe
I'll have to look at it tomorrow
I wonder why my unit tests aren't catching that
Archomeda
@Archomeda
Jan 29 2016 01:47
i had hoped to get some more insight by matching the api data with the wiki, but that's not finished yet :P
i'm not going through 1268 skills manually
but i'll take a look at it again tomorrow
cya :)
David Reeß
@queicherius
Jan 29 2016 05:35
@Nabrok Yeah the line breaks are the different tiers. And the gist should be public now. Didn't realise it wasn't. D:
smiley
@codemasher
Jan 29 2016 10:28

the API data needs a lot of love
:point_up: 29. Januar 2016 02:44

oh yes...

mapdata cough
codemasher @codemasher is annoyed by markdown
dammit @codemasher I've been reading both commitstrip and cube drone from the start for a day now :clap:
smiley
@codemasher
Jan 29 2016 13:44
lol
Edgar Doiron
@Coffee4cr
Jan 29 2016 14:52
I can just feel my productivity plumetting today
haha
Ryan Mast
@nightlark
Jan 29 2016 19:06
I have the same problem as @Coffee4cr
@Archomeda, if you happen to have a frontend-only tool that grabs and renders everything from /v2/skills I'd definitely be interested in using that locally for verification
If not I'll probably end up writing one anyway :>
Archomeda
@Archomeda
Jan 29 2016 19:09
I've been working on that, while also on the same time pulling things from the wiki
Lines of code: 678 and increasing... haha
heh
Archomeda
@Archomeda
Jan 29 2016 19:10
Mostly wiki stuff that's being annoying
I can trim out the wiki stuff for now, and post a gist
woot
right now I'm just reading the json
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:11
after a while it's like reading a book right
Archomeda
@Archomeda
Jan 29 2016 19:11
i haven't done traited_facts yet
I'm actually more confident in traited_facts than the rest of the data
Archomeda
@Archomeda
Jan 29 2016 19:13
also, a question, but i've never seen a damage fact with an actual value
is that also intended?
Archomeda
@Archomeda
Jan 29 2016 19:18
just let me implement traited_facts, should be done in a few minutes
yeah, the damage is kind of hard to compute
I punted on it :<
at some point I should probably put it in -- want to double-check with a higher pay-grade that it's okay. since the formula has been pretty much reverse-engineered at this point, I don't think it'll matter too much if we expose the raw values
Archomeda
@Archomeda
Jan 29 2016 19:23
hmm
oh, some traited fact doesn't contain a status... that crashed my code...
a Buff traited fact that is
hmm, that's kind of odd. That should probably be defaulted to "" since it's a human-readable string
Archomeda
@Archomeda
Jan 29 2016 19:25
let me find it... it's something with a duration of 10, a description of " feet", an apply_count of 25, that requires trait 1233
doesn't seem to be correct, that traited_fact links to image https://render.guildwars2.com/file/6408C6EF002E2B19187735AFA7DCF616A4BFC5B3/102972.png
oh... thanks gitter
O_o that trait isn't exposed
let's see what that is
Archomeda
@Archomeda
Jan 29 2016 19:28
https://api.guildwars2.com/v2/skills/5990 has the same required trait
hahaha, okay, well
it's the removed thief trait Descent of Shadows
rip sanity
it's also ref'd by daredevil steal
Archomeda
@Archomeda
Jan 29 2016 19:31
lol
you can create a fresh html file by running it with nodejs
woot, thanks for this
Archomeda
@Archomeda
Jan 29 2016 19:31
but i've also added a html file that i rendered just now
there are some checks in it, like facts having no icon, skills that are referenced but don't exist, etc
i hope i got everything
working on the icons now -- need to adjust my unit tests so they require an icon for all facts
Eearslya Sleiarion
@Eearslya
Jan 29 2016 19:33
I never did mess with the skills API much; I have a page that renders all traits, but not skills
the skills are still really rough around the edges
hopefully we'll be able to get all the issues sorted so it's a bit more ... consistent at least
Archomeda
@Archomeda
Jan 29 2016 19:34
if the icon warnings are annoying you, you can comment out L284-291
sorry about the cyan background color though :D
Eearslya Sleiarion
@Eearslya
Jan 29 2016 19:35
Mine's got a nice dark theme :D http://gw2.sleiarion.net/
Archomeda
@Archomeda
Jan 29 2016 19:36
mine is basically just a dump of all skills
didn't mean for it to be useful besides double checking
Eearslya Sleiarion
@Eearslya
Jan 29 2016 19:37
Mine checks /v2/specializations and organizes them into their professions/specs..and then it fetches all traits and complains in the console if any traits don't fit somewhere
Archomeda
@Archomeda
Jan 29 2016 19:37
btw, yesterday's patch did indeed add the underwater feedback skill: https://api.guildwars2.com/v2/skills?ids=10302,34326
now there are two feedbacks in the api
that are exactly the same XD
we are missing an underwater property or something like that
haha nice
Eearslya Sleiarion
@Eearslya
Jan 29 2016 19:39
My site's not good for much except checking bank/mats/characters yet XD
Archomeda
@Archomeda
Jan 29 2016 19:40
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:42
I think your page is killing /v2/skills
haha
Archomeda
@Archomeda
Jan 29 2016 19:42
nah, that page is static
Nabrok
@Nabrok
Jan 29 2016 19:42
not just /v2/skills
that usually means a patch incoming
because I suck at writing good code
Archomeda
@Archomeda
Jan 29 2016 19:42
it's rendered only once when you run node ./build-api-table.js
oh great, i have guild missions in 15 minutes
and then we get a patch :(
spoilers
Eearslya Sleiarion
@Eearslya
Jan 29 2016 19:43
SAB time?
Archomeda
@Archomeda
Jan 29 2016 19:43
guild mission instances are still a bit wonky
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:43

SAB time?

don't play with my emotions

Archomeda
@Archomeda
Jan 29 2016 19:44
still can't tag up in those instances D:
lol, if only SAB were to come back
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:44
you need to tag up before entering
Archomeda
@Archomeda
Jan 29 2016 19:44
that infinite continue coin is still sitting on a character
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:44
@Archomeda it will, cause they won't refund the infinite coin (that I bought(
Archomeda
@Archomeda
Jan 29 2016 19:44
yeah i know, but it's frustrating
although, now with the new icons, it should be less frustrating
Nabrok
@Nabrok
Jan 29 2016 19:45
We have always gone in with a tag already up
Archomeda
@Archomeda
Jan 29 2016 19:45
SAB... well the question is, when
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:45
when it's ready
Nabrok
@Nabrok
Jan 29 2016 19:45
My biggest frustration with missions right now is the queue for pvp missions.
Archomeda
@Archomeda
Jan 29 2016 19:46
anyhow, i have to prepare for guild missions, so if there's any issue with that nodejs script i wrote, you can fix it yourself :D
windwarrior
@windwarrior
Jan 29 2016 19:46
your problem now
that is how this sounds :P
Archomeda
@Archomeda
Jan 29 2016 19:46
XD
well, at least it creates a static file, so you can still view it when the api spits out a 502
windwarrior
@windwarrior
Jan 29 2016 19:47
oh it actually produces an HTML file?
I was wondering what it did
Archomeda
@Archomeda
Jan 29 2016 19:47
yeah, couldn't be bothered by using javascript in the browser
since i wanted to link the skills against the wiki data
at that just takes too long
so i wrote it with nodejs
this is pretty much perfect for verifying that I'm using the right icons
windwarrior
@windwarrior
Jan 29 2016 19:50
so @Archomeda, did you add steal_all_sab_spoilers.js
Edgar Doiron
@Coffee4cr
Jan 29 2016 19:56
oh thanks gitter :(
Archomeda
@Archomeda
Jan 29 2016 19:57
@windwarrior lol
@Coffee4cr at least it doesn't autoplay, that would scared the living crap out of me
@Iye i'm glad that works for you :D
windwarrior
@windwarrior
Jan 29 2016 19:59
It doesnt check against the wiki yet right?
Archomeda
@Archomeda
Jan 29 2016 19:59
this code doesn't no
windwarrior
@windwarrior
Jan 29 2016 19:59
and wow this game has loads of skills
Archomeda
@Archomeda
Jan 29 2016 19:59
i've stripped that out since it wasn't finished yet
also, guild hall instances are freaking acting up now, is that an issue with the incoming patch?
windwarrior
@windwarrior
Jan 29 2016 20:00
I head home instances are also acting up
Archomeda
@Archomeda
Jan 29 2016 20:00
when new instances are made, people get kicked out of the guild hall
smiley
@codemasher
Jan 29 2016 20:00

because I suck at writing good code
:point_up: 29. Januar 2016 20:42

at least you're honest, @lye ;)

Archomeda
@Archomeda
Jan 29 2016 20:04
rip gw2
i'm stuck in a loading screen
STATE_SERVER_WAIT
Archomeda
@Archomeda
Jan 29 2016 20:06
noooo... blazeridge is rip
there goes our guild trek target
blazeridge steppes instance on 206.127.159.134 is rip D:
the shatterer shattered the instance
Archomeda
@Archomeda
Jan 29 2016 20:32
also, our party chat is broken in squad
can't link stuff XD
windwarrior
@windwarrior
Jan 29 2016 21:07
Fancy, if you hover over the trumpet on gw2's subreddit a new menu pops up
my dev tracker is in there :D
Archomeda
@Archomeda
Jan 29 2016 21:08
wow gz :D
Pat Cavit
@tivac
Jan 29 2016 21:46
that's awesome
now I just need a repo popular enough to warrant that :cry:
smiley
@codemasher
Jan 29 2016 21:47
oh, we could make you use it... :D
Archomeda
@Archomeda
Jan 29 2016 22:09
well, the api repo is pretty popular so now and then :)
@Archomeda, so I've been thinking. It may be useful to provide a per-profession list of skill ids for each skill slot in addition to the current "lol here's all the skills" endpoint
something like /v2/skills/profession/engineer would return
{
  weapon : {
    pistol : [ 1, 2, 3, 4, 5 ],
    shield : [ 14, 15 ]
  },
  heal : [ 6, 7, 8 ],
  utility : [ 9, 10, 11, 12, 13 ],
  elite : [ 16, 17 ]
}
(there's also profession skills, downed skills, etc, but it's kind of the same)
The idea is that it gives you a view of which skills are available, and from there you can drill down into flip_skill or transform_skills and whatnot
Might want to do the same thing with /v2/traits, now that I think about it.
Internally each profession has a datastructure which contains all this information -- might be reasonable to just provide /v2/professions which serializes all that out, I dunno.
so /v2/professions/Engineer would provide the above blob in the .skills field, maybe a .icon, maybe a .specializations and such
Archomeda
@Archomeda
Jan 29 2016 23:33
that would be awesome, it's better than grab all the skills or traits and filter afterwards
the main reason is that I'm not sure if I can make /v2/skills not be a total disaster for end-users
Archomeda
@Archomeda
Jan 29 2016 23:34
at least i'm glad that /v2/skills?ids=all returns all the skills without having to paginate
windwarrior
@windwarrior
Jan 29 2016 23:34
how would that work with chains?
chains should be encoded on the skills (but might be broken)
Archomeda
@Archomeda
Jan 29 2016 23:35
i saw some chains in the current /v2/skills yeah, haven't checked whether they are correct or not
so in the above example, skill #1 is engineer pistol_1 and has a .chain : 18 which in turn has a .chain : 19
I'm pretty sure there's at least a few bugs
Archomeda
@Archomeda
Jan 29 2016 23:36
i think it was encoded as next_chain and prev_chain
maybe, it isn't in the documented examples >_<
Archomeda
@Archomeda
Jan 29 2016 23:36
i've implemented some checks in the gist that checks for chain consistency
I'll have to port those over to my version; I've butchered the original pretty hard already, heh
Archomeda
@Archomeda
Jan 29 2016 23:37
hahaha :D
the code is a mess anyway
story of my life
but I'm not sure I can fix the "Skill is not assigned to any profession" warning
those are usually on bundle skills which have no profession -- e.g., magnetic shield can be picked up and used by any profession
and tornado is a transform that can technically be used by engie and ele (even though the skills that trigger the transform are ele/engie-only)
(but in the tornado example, the transform can technically be "wielded" by any profession, if that makes sense)
Archomeda
@Archomeda
Jan 29 2016 23:39
hmm
that totally breaks the "pull all skills and filter them based on the profession" thing -- because content says tornado_1 can be used by a warrior but realistically it can't
(conceptually that's what /v2/professions is intended to solve)
Archomeda
@Archomeda
Jan 29 2016 23:40
all those bundles, transformations, kits and whatsnot really make a mess of the current skills endpoint
yeah they do, but they need to be exposed since they're referenced by other skills
you know this might hurt your mind
but buffs are also skills
Archomeda
@Archomeda
Jan 29 2016 23:41
what
welcome to my castle :>
windwarrior
@windwarrior
Jan 29 2016 23:42
does your character automatically cast boons every tick, how would this work...
a skill is basically some common fields, then a union of buff-specific and ability-specific bits
windwarrior
@windwarrior
Jan 29 2016 23:43
yep this doesnt make sense
oh waauw
Archomeda
@Archomeda
Jan 29 2016 23:43
ooookaay...
common sense would make buffs a separate thing
at least separate from skills
anyhow, did you manage to find the source of that skill with the odd required trait with the ground target image?
pulled it up in the editor
Archomeda
@Archomeda
Jan 29 2016 23:52
and?
the thumper turret also has a traited fact that requires the same non-existent trait XD
it said it required a thief trait? might be misunderstanding the question
Archomeda
@Archomeda
Jan 29 2016 23:53
you said something like that earlier too, but i meant what that traited fact actually means
if you know if yourself of course
Ah, there are multiple ways for a trait to affect the skill -- some things I encode as "facts" aren't actually facts (e.g., range, recharge, etc) so for a trait to affect them it has to swap the skill to a copy of the skill with different parameters
For facts that are actual facts a trait can add or overwrite existing facts (which pretty much works in the same way as the API serializes it)
There's a bug in the serialization code wherein the traitable skills (that are replaced with a copy of the skill) require nonsensical traits; that's what's causing the strange dupes
Anyway, got a meeting to run to :>
Archomeda
@Archomeda
Jan 29 2016 23:56
okay :D
thanks for the explanation :D
windwarrior
@windwarrior
Jan 29 2016 23:57
yeah, you rock