Search results

Sorry. Not sure how to generate the right format. :( Don’t worry about it. ;)

Patch to track drops Here’s a patch that tracks drops for the two familiars.

Sorry, I forgot most people don’t know how to use patches. :P

Mafia r11618 changes the names of the basement monsters to match the names given in the Monster Manuel. This patch updates BatBrain accordingly.

My code replaces the existing matcher nam line (line 8). I think ckb’s code replaces the block of lines following if (find(nam)) { monname = group(nam); }.

Only one copy. But I just tried redownloading it and now it works. Go figure.

Every time I try to run the script, I get this: I’m running the newest BatBrain.ash, SmartStasis.ash, zlib.ash and WHAM.ash. I’ve also tried it with your BatBrain.ash, with no luck. I can’t actually find this 3‐argument version of enqueue anywhere. What am I doing wrong?

Just noticed that the script has issues with monster names that contain accented characters. This is easily fixed: matcher nam = create_matcher("(?<=(id='monname'>))([^<]+)",results); string monname = "";

Ah I see. In that case, both the \bs seem like unnecessary, unwanted, undesired and unwarranted repetition & redundancy to me. :p EDIT: It would be nicer to use Bale’s suggestion too: ^\s*(?:a |an |the |some )?(.+)$. I suppose this is a moot point, however, given ckb’s post. hey ckb what if...

Sorry for the delay. I was asleep. I have a confession to make: I’ve never quite understood how | works with regard to spaces. It’s probably something simple, but I’ve never actually looked it up, so I always over‐specify it using an inner group. Your way would likely work too, and more...

Nice script. Small problem: the checks for a/an/the/some cause errors if the monster’s name is too short. A simple way to fix this is to remove those checks and instead handle qname like this: if (find(nam)) { monname = group(nam); } matcher stripped_name =...

This version seems to have the same problem my first attempt did: if the list of effects is empty, it fails to parse the mood due to a missing br. Changing line 2771 to read as follows fixes it: endwith = "]</font>";

I managed to fix it by changing line 2897 to read: endwith = "</a>]</font><br>"; EDIT: This is slightly off. It breaks when you have no effects. Fixed version follows. Change line 2897 of charpane.ash to read: endwith = "</a>]</font>";

Well, I personally advocate code reuse wherever possible, as people tend to reinvent the wheel often, but I can see your point for a small script. That said, I looked at the code again and several other Zlib functions could be of use too: have_item() (wraps the item_amount and equipped_amount...

Very useful script—thank you! I have a few suggestions: Maximizing for just item drops left me holding a bottle-rocket crossbow as a TT, which meant I couldn’t hit anything. Maybe this would be better: if (my_primestat() == $stat[muscle]) { maximize("item +melee -tie", false)...

Sorry. That seems to be an artifact of Mercurial’s diff format. If I have any more enumification patches, should I post them here? Or would somewhere else be more appropriate?

Oh, sorry, didn’t know that was significant. :p

(Sorry for the sorta necro. I wasn’t sure where this should go.) I saw that roippi’s been converting things into enums. Thought I’d help out since it’s fairly tedious. Here’s a patch that replaces KoLConstants.ABORT_STATE, etc. with an enumeration. The vast majority of it was just searching...

Actually, that keeps going back and forth for me. Some revisions it automatically goes to the max, others it remains at zero (and therefore doesn’t adventure anywhere). It’s been in the second mode for a month or two now. :confused:

I see. Didn’t realize people automated choiceadvs in the Relay Browser like that. :)