Recent content by ikzann

On the theory that until we can consistently reproduce the bug, we are not going to be able to fix it, I spent some time this morning trying to write a test to reproduce the behavior, with no luck. My experience of the bug is some version of running a script in Mafia, closing my laptop without...

We've fully spaded the correlation between familiars and the Grafted intrinisic, so we wouldn't even necessarily need to track that separately.

https://github.com/kolmafia/kolmafia/pull/2522 Includes everything except "recently expired effects" - which I think comes from the mood and is currently accessible via get_mood_list or something like that.

Sorry. I changed the branch name without thinking about how it would break existing installs. Let me just undo that

Here is a wiki page documenting the API. https://wiki.kolmafia.us/index.php?title=Browser_JSON_API

Maybe it was a little inaccurate to say it will be used by "relay scripts." It can be used by any client-side code running in the relay browser. Whether client-side code is generated by an ASH or JS mafia relay script, the code that runs client-side is all javascript by necessity, as that's the...

Hi folks - didn't see this thread until now, apologies. For some reason, the pinging feature that is supposed to go to my email is not working. I will figure that out. The basic purpose of this change was to provide an efficient way for relay scripts to make large and/or frequent requests for...

LOL. That's the last time I rely on what someone tells me about the code... Never mind!

I've been working on some clan dungeon collaboration scripts, and it would be useful to have the scripts be able to use the dungeon chat channels to communicate between players. In particular, when a player gets to the I Refuse! adventure in The Heap, it is helpful for one player to let everyone...

Thanks everyone for all the help. After the PRs I've submitted, the hot spots are still mostly the same, but overall execution time seems to have been reduced (hard to measure exactly). I think there is probably some work that could be done to the maximizer, but that may be a more in-depth...

Gausie merged my changes this morning, so I ran the 4:1 ratio benchmark against both r26905 and r26907. I still saw a 4-5x speedup or so as the old code also slows down substantially with the corrected benchmark. That's good enough for me. This is also only CPU time - obviously folks with...

Thanks. I'll rerun the tests.

That's fair. It is true that a big part of the "problem" (to the existent there is one) is saveSettingsOnSet - there are now quite a few preferences that update every combat. I am going to explore turning it off, or triggering a write once per adventure, in some of the scripts I work on. I...

Okay, after the work this weekend there is still one major outstanding question I'd like to ask folks: should we switch preferences from using a sorted TreeMap to an unsorted HashMap-style? Mafia currently spends a nontrivial amount of time doing its in-memory preference lookups - maybe 8-10%...

I have a couple PRs in now to at least ameliorate some of these issues. Still looking at a few other candidates for low-hanging fruit for improvement.