These are chat archives for arenanet/api-cdi

25th
Sep 2017
Dominic
@DomGrieco
Sep 25 2017 13:55
@lye how's the api repairs coming
Darqam
@Darqam
Sep 25 2017 14:00
don't think he's at work yet
that should be in ~2h iirc
BoyC
@BoyC
Sep 25 2017 14:39
something like that yeah
Enno G.
@SchoolGuy
Sep 25 2017 17:00
@lye Thanks for your efforts to bring the API up as fast as possible :)
David Reeß
@queicherius
Sep 25 2017 21:16
@lye Thanks <3
h-hold up I haven't run the integration suite yet
everything should work mostly
darthmaim
@darthmaim
Sep 25 2017 21:17
:heart:
Quaggan BooOOoot
@QuagganBooOOoot
Sep 25 2017 21:17
:heart:
inventories might still be broken
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:17
@lye but the colors endpoint
it's like
updated the code to account for larger bags
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:17
the most important
colors are dead until I figure out how the new hotfix process works
David Reeß
@queicherius
Sep 25 2017 21:18
Yeah /account/characters is deaded
awesome, it worked on stage :(
David Reeß
@queicherius
Sep 25 2017 21:18
I just saw a spark of light and got excited
also regenerating tiles
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:18
Dead?
image.png
hmm yeah.
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:19
dat error
crap this might take some time to fix
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:20
works on my machine, all that matters :+1:
does it?
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:20
one sec, lemme make a key
something specific to que
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:21
@queicherius Do you have any of the new, bigger bags?
David Reeß
@queicherius
Sep 25 2017 21:21
Does that key nothave the "inventory" permissions?
it doesn't
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:22
oh
David Reeß
@queicherius
Sep 25 2017 21:22
No i'm a scrub, i dont have that
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:22
well it works on my full access key too
it's either (1) whether or not you've gotten a large bag affects the storage format, or (2) the largest bag currently equipped
I couldn't find the bit in the game server logic that does it
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:22
oh
David Reeß
@queicherius
Sep 25 2017 21:23
And it breaks for pretty much every key i can find
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:23
no it does not work for me
whoopsie
it gets the list just fine, it's when it parses inventory, yeah
more error code if it helps
yeah I've got the call stack
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:24
weird that it's failing on my thief's inventory
she's not the first character chronologically or alphabetically
David Reeß
@queicherius
Sep 25 2017 21:25
On the plus side, thats the only error I can find
as an additional plus, I can maybe fix that with just javascript
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:26
What does it mean, anyway? "inventory not emptied"
it's mapping from a sparse array to unsparse arrays
e.g.
[{slot:1, item:x},{slot:16,item:y}]
// to
[ x, null, null, null, ..., y ]
"inventory not emptied" means there are some items left over in the source array that it couldn't find things for
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:27
huh
how would that happen?
:shrug:
I submitted a config change to disable that codepath
so inventories won't be returned, but it'll clear up the errors (so some alarms stop going off)
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:28
Weird, now it's showing my ele's inventory
Question, does this array include equipped items?
Nope
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:29
Well, uh..
Equipment is stored elsewhere.
UHHH
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:29
It's showing me slots 133 and 134
image.png
those aren't slots as in slot slots
those are inventory positions
treating all your bags as a single array of things
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:31
right, but
the max size is 120
(until PoF)
I might be wrong I don't remember
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:31
she certainly doesn't have 130 slots
probably does
since each bag is treated as a max-slot bag
(for storage purposes)
you just can't use the slots beyond what it stores
anyway inventory bits disabled, hopefully that clears the alert
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:32
Hmm. So, it would jump from slots 1-20 to 33?
yeah
presumably
oh, hmm, there's another case I hadn't considered
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:33
let me actually look at her inventory real quick..
I really like how I totally tested the inventories with new bags and they worked on my machine :tm:
@Eearslya you've logged into all of your characters right?
like since-pof
that might be the issue here
if ... yeah that'd be pretty bad
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:36
no I have not
oh noes
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:36
I can see where you're going with that
let's try
yeah two different inventory formats
well I turned off the broken js so there's no clean way to test
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:36
welp
hmm, I can try with my account; pulling the raw data from the backend server before/after logging in
anyway the rest of the stuff might work maybe
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:37
Yeah, I remember hearing that data changes like this only happen on logins
yeah it's really looking like that's the case
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:37
that'll be...fun
it does only happen on login, but the backend server applies all the data migrations to it's read-only copy
this kind of insanity shouldn't happen
unless the bit that upgrades the inventory storage doesn't use the normal data migration path
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:39
hmm
the item that the error says is in slot 134, is actually in 125
that's way too low if it was changing everything to 32-slots
I just created a new char on my local stack
worked without adding bags, worked after adding old bags, worked after adding new bags
windwarrior
@windwarrior
Sep 25 2017 21:40
did you spent at least 30m in character creation
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:41
but wouldn't you need to create it with the old bag data format?
I can't create it with the old bag format :D
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:41
exactly
that's the gap in testing
@Eearslya is my live account in your friends list? What's my account ID?
I thought it was Lawton.1694 but that's giving me "account not found"
Ah 1697, I was close.
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:42
1697
For funsies, this is what the backend server returns: NO GITTER BAD
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:44
gitter \o/
given that has slots 21+22 filled..... looks like it's using 20 slots/bag as the count
gonna log in and see if that changes the output
if it does... :vomit:
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:45
the suspense
I haven't run the live client in a long time
so this might take awhile to patch :|
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:46
oh boy
I think we have patch servers on-prem though so it should be pretty quick
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:46
even AWS I can get 10/11MB/s
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:47
rip
it's not even hitting 2MB/s anymore
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:47
go yell at ops
darthmaim
@darthmaim
Sep 25 2017 21:47
tell us your password and we can login for you :P
David Reeß
@queicherius
Sep 25 2017 21:47
Are there new item categories? o.o
uhh
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:48
uh oh
if there are they might not be serialized correctly
windwarrior
@windwarrior
Sep 25 2017 21:48
who makes bags >20 slots then
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:48
@windwarrior any of the armor crafters
windwarrior
@windwarrior
Sep 25 2017 21:49
nah just commenting on how such an assumption was sane
@queicherius are you seeing type:"Unknown" for items?
That's what would be serialized for any new item types
I'm not seeing any new types in the enumeration though
David Reeß
@queicherius
Sep 25 2017 21:50
I am seeing "Key", which is new
        { "Key",                ITEM_TYPE_KEY               },
that's basically all I know
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:51
looks like the wiki's gonna need some updating
David Reeß
@queicherius
Sep 25 2017 21:51
I am mapping the strings to an int and it's missing in the map and dying, that's why I was asking :D
Key's been in there since 2015
So... I guess it just started being used? I've no idea what it is
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:52
What's the value for access for free accounts? PlayForFree?
uhh maybe
honestly your guess is as good as mine
I haven't spent very much time with this in the last few months
I think free accounts will simply not have HeartOfThorns or PathOfFire
Best way to check access is to look at the presence of the expansion unlocks
Eearslya Sleiarion
@Eearslya
Sep 25 2017 21:53
But would they have GuildWars2?
They ... might
David Reeß
@queicherius
Sep 25 2017 21:53
Yeah thats the only new category, everything else is in there. I guess someone used it instead of "Gizmo -> Container Key". It's also only used on one item. :D
lul
JC123
@JC123
Sep 25 2017 21:58
@lye can you check out achievement 3901?
progress def for the act 2 meta specifies it but its not in the act 2 category?
darthmaim
@darthmaim
Sep 25 2017 21:59
/v2/achievements is returning unknown region for mastery points
{
"type": "Mastery",
"region": "Unknown",
"id": 384
}
David Reeß
@queicherius
Sep 25 2017 22:04
I also seem to only see 18 PoF achievements, but I assume that's cause of whitelisting
(At least they are missing in the "achievements" array in "/v2/achievements/categories")
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:09
the forest fire is over and now the smaller flames are revealed
@lye did you ever get patched?
it's still patching
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:12
ouch
200KB/s whooo
Archomeda
@Archomeda
Sep 25 2017 22:16
i suppose the api has missed the whitelisting calls when it was down :( (if that's how it works)
nah only the frontend was turned off
the whitelist queue is brimming with things for me to bless
Archomeda
@Archomeda
Sep 25 2017 22:16
ah
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:19
...huh
@lye here's one you can probably pawn off to a lackey
image.png
free account
https://account.arena.net/content/download/undefined/undefined/undefined
Archomeda
@Archomeda
Sep 25 2017 22:20
hahaa
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:21
{
text: "no game account"
}
huh
that one's new
you sure you didn't get banned or something
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:21
it's my play for free account
you get that message when you're banned
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:21
I dunno if it's ever logged in
can't imagine why it would have been
ha ha haaa logging into my character did change the backend's output format
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:23
oh goodie
how badly?
I mean it changed the slots-per-bag from 20 to 32
so those items in slot 21 are now in slot 33
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:24
but
I can "fix" this by switching off the last modified field I think.
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:24
why would that break the sparsifying?
wouldn't that just..make more nulls?
because it changes the meaning of the slots?
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:24
oh
oh right
                for(idx = 32 * i; idx < 32 * i + bag.size; idx += 1) {
specifically it changes where the frontend stops reading
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:25
riiight, I've got it
hmm when did we turn on expac2
was it 9am pst friday?
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:25
3 days ago?
yeah
darthmaim
@darthmaim
Sep 25 2017 22:26
2017-09-22T16:00:47.000Z
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:26
image.png
is that the ban message?
uhh
no idea
:|
@darthmaim thanks
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:27
weird
windwarrior
@windwarrior
Sep 25 2017 22:28
47s late, oh damn
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:29
Hmm. I guess maybe the ANet account is separate from the GW2 account? I always thought they were implicitly linked
windwarrior
@windwarrior
Sep 25 2017 22:30
hmm?
they're linked but different accounts
windwarrior
@windwarrior
Sep 25 2017 22:31
also, can you find a max bag size somewhere, feels kinda brittle to fix it on 32 now
sounds like a recipe for disaster :P
future lye can deal with that
I'll have to spend a few hours reading the new inventory server code
David Reeß
@queicherius
Sep 25 2017 22:31
I have a feeling future lye hates past lye
past lye is always a dick
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:32
Weeeiiird. I can't register a GW2 account.
windwarrior
@windwarrior
Sep 25 2017 22:32
I have the feeling that the temporal ordering of various lyes prevents massacre
BoyC
@BoyC
Sep 25 2017 22:32
yay, 257 mastery points reached. now to take a breather. :D
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:32
This whole account is just..screwed XD
rip
windwarrior
@windwarrior
Sep 25 2017 22:33
we just took a full buffed party to do bounty bosses
some of them are actually difficult
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:33
image.png
my username is a GUID
so that's nice
that happens with you migrate a gw1 account or something
there's some historical reason for that that I don't recall
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:34
no gw1 account linked with this
I'm like, 90% sure I made this just to have a F2P account and see the API/in-game differences
and now it can't log in
windwarrior
@windwarrior
Sep 25 2017 22:35
oh rip my heatmap died
Error: Cannot set property 'count' of undefined
is that from the API? which endpoint
windwarrior
@windwarrior
Sep 25 2017 22:36
probably my handling
I'm gonna have to go through the errlogs at some point -- really not looking forward to that :(
windwarrior
@windwarrior
Sep 25 2017 22:36
let me get to the bottom of it first ^^
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:40
welp, does anyone happen to have a free account that isn't fucked?
darthmaim
@darthmaim
Sep 25 2017 22:41
2509D27C-EF6E-A142-B4E5-C58098EC4E8FC13601D0-8A28-470B-91D2-A4270299E6CC
or CB0EA922-28D1-F745-9253-2A4D0AD38EAAC2928255-2CD1-457F-A328-38373ED49C35
Eearslya Sleiarion
@Eearslya
Sep 25 2017 22:41
access: [
"PlayForFree"
],
perfect
thanks :D
darthmaim
@darthmaim
Sep 25 2017 22:42
:+1:
If you upgraded an old p4f account before core became free you'll have "GuildWars2" in there as well I think
I'm not sure about p4f accounts after core became free; I'm assuming you'll have e.g. ["PlayForFree","PathOfFire"]
oh you know what
there's a bagMaxSize here that's stored in the blob
cool it's unconditionally set to 32 after saving
so I can just ignore it maybe
what's the new mastery region called?
CrystalDesert?
Eearslya Sleiarion
@Eearslya
Sep 25 2017 23:06
I guess so
could just do Desert
windwarrior
@windwarrior
Sep 25 2017 23:08
I think my heatmap thing is crashing because of a non-whitelist ID
82118
or rather, an incosistency between /v2/account/materials which does list that ID, but /v2/items/that doesnt
ah let me go through the whitelist now
Eearslya Sleiarion
@Eearslya
Sep 25 2017 23:13
windwarrior
@windwarrior
Sep 25 2017 23:18
I am not really sure how such an error is not checked by my code
but I can also really not read the sh that I produced a year ago
oh my
I think I liked promises last year, and wasn’t feeling like writing comments
Archomeda
@Archomeda
Sep 25 2017 23:23
inb4 we get another desert in the future
okay finished playing clicker hero with the whitelist
@Archomeda "Desert2"
I can make this work
windwarrior
@windwarrior
Sep 25 2017 23:24
Yeah I silently assume that all items of /v2/materials?ids=all are also on the api
they should presumably start appearing soon
I just whitelisted like 2000 things
I am very good at clicker hero
windwarrior
@windwarrior
Sep 25 2017 23:27
If someone could plug in an api key into http://windwarrior.github.io/GW2MaterialHeatmap/ I would be grateful
1B410E2E-DECA-D248-9354-82724F218D753D7EAAC2-BBC8-4051-9AEB-FD3DEC6BD0FF
windwarrior
@windwarrior
Sep 25 2017 23:29
Eh i’ll check tomorrow, phone won’t let me paste
Eearslya Sleiarion
@Eearslya
Sep 25 2017 23:30
seems to work
only error in the console is 404 on favicon
windwarrior
@windwarrior
Sep 25 2017 23:30
Cool thanks!
okay important question time: should I update the Server header to be Choyas instead of Quaggans
windwarrior
@windwarrior
Sep 25 2017 23:32
When do we get /v2/choyas
when someone gives me a pile of choya concept art
Eearslya Sleiarion
@Eearslya
Sep 25 2017 23:32
Choyaggans
windwarrior
@windwarrior
Sep 25 2017 23:33
Chaya’s
Choyas try to kill me tho and wuaggans
Don’t
Really phone :(
I give up, gn
Archomeda
@Archomeda
Sep 25 2017 23:40
nah, just have it return those choya philosophies, e.g. don't embed
omg... i just noticed: "deaths": 8496,
Archomeda @Archomeda sighs
Archomeda
@Archomeda
Sep 25 2017 23:46
@lye, Path of Fire: Act 2 Mastery isn't whitelisted yet i think
plus, i still think there's a hidden achievement that counts for the meta achievement as well... or some people can't count to 19
why are there hidden achievements that count for the meta achievement? :(
or rather, are required for the meta achievement... :/
JC123
@JC123
Sep 25 2017 23:59
its not hidden