Search results

Thank you Veracity! This is great, exposing this to scripters will both help them and help us when they point out areas that it is missing. I think that the most use for scripts is going to be from validate2 (and validate1? I haven't read to see if you need to run both. Also maybe we should...

I had to read this quite a few times to understand it but are you saying that the environments stop being checked after the last consult? If so that's annoying because I've probably named this pref too widely. Noncombats are not tracked by design. The cellar is an odd one - what should it be...

I think we could consider providing similar functionality in core mafia

The change can be applied to mirror, it's not really a defined interface that we need to not change at all

Thanks V. We could consider changing the MafiaState.ABORT to MafiaState.ERROR but I'm also heartened somewhat that the change surfaced some actual bugs that we didn't know about and has lead to them being fixed.

I think https://github.com/kolmafia/kolmafia/pull/936 may help with this one a little bit.

Thanks pya. It would help me get to fixing it if you could you drop a DEBUG log for this happening?

I defer triage to a Windows user :)

https://github.com/kolmafia/kolmafia/pull/936/files

I will fix this but I must say that not checking you have an item before using it and then not checking you're in a choice before attempting to choose that choice is silly.

https://github.com/kolmafia/kolmafia/pull/935

But why do you need html for that

I'm fine with changing mirror to more exactly copy the GCLI output but I must ask why on earth you want it

Seems like that could be refactored to be a list, or even a query on the item database sorted by +weight

It's a simple PR, please do make it!

If we were to do something like this I'd be interested in adding a new column to defaults.txt that describes when a pref is reset "r,a" could be rollover and ascension and would sit next to existing prefs with a leading underscore

I would be interested in an _ascensionsToday pref, PRs welcome. Potato alarm clock should be fixed on the mafia end - we can problably give it a _ exception to only apply on true rollover.

You can implement this yourself by maintaining a pref with the day and resetting all prefs with a certain prefix when you see that that date is old.

Ok as I flesh out the further support, I realise the ash function I added is probably not quite right - it might be nicer to have item [int] expected_cold_medicine_cabinet(); where the int key is the choice and the item is the item you'd get from that choice. That is once we have estimation in...