Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Path Heavy Rains issues combo tracker #198

Open
taltamir opened this issue Nov 30, 2019 · 6 comments
Open

Path Heavy Rains issues combo tracker #198

taltamir opened this issue Nov 30, 2019 · 6 comments
Labels
bug Something isn't working Multiple Issues

Comments

@taltamir
Copy link
Contributor

This is a ticket for keeping track of various heavy rains issues that currently exist.

@taltamir
Copy link
Contributor Author

taltamir commented Nov 30, 2019

Ongoing Issues

subissue 4

OBSOLETE. we no longer rely on switching CCS. as such this mafia bug is no longer relevant to us...
Heavy rains is failing to switch CCS to autoscend_null, resulting in it erroring out until i manually switch it over.

customCombatScript is set to unrecognized 'rain', should be 'autoscend_null'
Stack trace:
  at handlePreAdventure (auto_pre_adv.ash:18)
  at handlePreAdventure (auto_pre_adv.ash:6)
  at main (auto_pre_adv.ash:360)
Stack trace:
  at rainManSummon (auto_heavyrains.ash:503)
  at rainManSummon (auto_heavyrains.ash:525)
  at hr_initializeDay (auto_heavyrains.ash:150)
  at initializeDay (autoscend.ash:2176)
  at auto_begin (autoscend.ash:15150)
  at safe_preference_reset_wrapper (autoscend.ash:15277)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
printStackOnAbort => false
customCombatScript is set to unrecognized 'HCCS', should be 'autoscend_null'
Stack trace:
  at handlePreAdventure (auto_pre_adv.ash:18)
  at handlePreAdventure (auto_pre_adv.ash:6)
  at main (auto_pre_adv.ash:360)
Stack trace:
  at rainManSummon (auto_heavyrains.ash:503)
  at rainManSummon (auto_heavyrains.ash:525)
  at hr_initializeDay (auto_heavyrains.ash:150)
  at initializeDay (autoscend.ash:2176)
  at auto_begin (autoscend.ash:15150)
  at safe_preference_reset_wrapper (autoscend.ash:15277)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
  at safe_preference_reset_wrapper (autoscend.ash:15284)
printStackOnAbort => false

This might possibly be related to this mafia bug:
https://kolmafia.us/showthread.php?24886-Some-mafia-settings-don-t-actually-change-unless-you-change-them-in-the-UI&p=157033#post157033
However, it might be an us bug instead. because mafia can successfully change CCS via command given by a script. So either we are failing to call it in heavy rains, or mafia can change CCS via script command except from heavy rains where it can't.

@Malibu-Stacey Malibu-Stacey added the bug Something isn't working label Dec 1, 2019
@Malibu-Stacey
Copy link
Member

I'd suggest putting hippy war issues in their own issue or appending them to #167 as they're not specific to Heavy Rains.

@taltamir
Copy link
Contributor Author

taltamir commented Apr 15, 2020

Good point. Reorganized

@taltamir
Copy link
Contributor Author

taltamir commented Apr 15, 2020

Universal Issues

These issues were determined to be universal and not specific to heavy rains.
As such even if they are not yet resolved they should not block this ticket from closing.
Those issues should all be spun off into their own independent ticket

subissue 2

Issue #271

subissue 5

Issue #270

subissue 6

issue #71

@taltamir
Copy link
Contributor Author

taltamir commented Apr 16, 2020

Already Fixed Issues:

subissue 1

PR #316

The final boss https://kol.coldfront.net/thekolwiki/index.php/The_Rain_King
could be optimized for its own unique rules
IIRC it lost a battle against it first time i did a run, but my second heavy rains autoscend run won the battle against it on the first try just now. Albeit a very inefficient win.
Main thing is to grab lots of cheap buffs against it (it removes 1 a round instead of all at once like naughty sorceress does). And grab any passive damage you can.
Also at begining of battle it strips all your gear except main hand and off hand.
And it has a hard cap of 40 damage per source. So you need to use multi sourced stuff, an offhand mateorb is good here. As well as all passive or retaliatory damage sources.

subissue 3

PR #316

At the end of the day I got

[INFO] - Copies left: 0
[INFO] - Pulls remaining: 6
[INFO] - If you have the Frat Warrior Fatigues, rain man an Astronomer? Skinflute?
[INFO] - You have a rain man to cast, please do so before overdrinking and run me again.

Instead of telling me, it should automatically use it. In my case the best choice would have been lobsterfrogman. although skinflute or ghost were valid too.

subissue 7

PR #316

Issue #194

subissue 8

Various univeral hippy related L12 war issues
merge #265
A heavy rains specific issue remained where rainman will only copy a fratboy for the frat war outfit regardless of which side it fights for.
This was fixed in subsequent merge

subissue 9

fixed: hole handling was universally a bit broken and I have since fixed it. And testing shows that the general fix works in heavy rains

The skill https://kol.coldfront.net/thekolwiki/index.php/Rain_Man
is used to acquire the components of the star key in order to skip the hole in the sky.
As a result autoscend does not bother to unlock the hole in the sky.
Which results in shen's quest stalling on the 3rd task of fighting a monster in the hole in the sky (since back in the day it would have gone the pirate route).
So I had to manually unlock it and then finished the palindrome quest on my own... I am not sure if it would have been able to finish that quest on its own or not had i reran autoscend as soon as i unlocked the hole in the sky

taltamir added a commit to taltamir/autoscend that referenced this issue Apr 16, 2020
To address loathers#198 I needed to split the function that determines if we are done for the day into 2.
one that calculates how many adv we want to keep, and one to determine if we reached that point.
When doing so I noticed I forgot a .to_int() in one spot which I believe is the cause of the discord reported issue (it was very lacking in details so I can't be sure)
@taltamir taltamir mentioned this issue Apr 16, 2020
4 tasks
taltamir added a commit to taltamir/autoscend that referenced this issue Apr 17, 2020
To address loathers#198 I needed to split the function that determines if we are done for the day into 2.
one that calculates how many adv we want to keep, and one to determine if we reached that point.
When doing so I noticed I forgot a .to_int() in one spot which I believe is the cause of the discord reported issue (it was very lacking in details so I can't be sure)
@taltamir taltamir changed the title Heavy Rains Issues Path Heavy Rains issues combo tracker Apr 18, 2020
taltamir added a commit that referenced this issue Apr 18, 2020
* ed can't funksling
* Issue #194 - Dr. Aquard is a puzzle boss
* Heavy Rains skill Thunder Bird fixed. it would debuff bosses 31 times then lose.
* issue #198 subissue 3
* fix to reserve adventures calculation
* after bear verb orgy is used to get wand of nagamar actually fight sorceress
* Heavy Rains final boss [The Rain King] handling. prep and combat strategy.
* rollover adv maximizer now considers Left-Hand Man
* don't assume success when adventuring in bandit crossroads (fantasy realm IOTM) to prevent infinite loops
* community service. small fix, see PR #317 for discussion about it
* various small fixes
@taltamir
Copy link
Contributor Author

taltamir commented Apr 19, 2020

TODO

  1. assaultFamiliar function that sets best damage familiar for boss fight. can be used for both regular sorceress fight and her replacement in heavy rains, probably a few other bosses in heavy rains... and actually everywhere where warbear assault drone is used.

  2. I made an error when preparing to fight the rain king. it equipped the green lantern momentarily but then unequipped it before the battle. switching to a maximizer result
    the result of which was weapon: something or another fan that deals prismatic damage, offhand lightsaber
    it should have been offhand green-lantern with weapon being maximized for spell damage percent
    also should add 1handed when optimizing spell damage percent. and pulling meteorb if appropriate.

  3. buffMaintain should be using
    maximize("-mana cost, -tie", false);
    before casting a buff to reduce the buff costs. maybe it already is?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Multiple Issues
Projects
None yet
Development

No branches or pull requests

2 participants