Slay the Spire

Slay the Spire

The Hierophant: Rebalanced
Zu 17 Nov, 2022 @ 10:38am
Bugs & Oversights
Separate thread seems appropriate at this point.

- Is anathema meant to deal double or triple piety damage? I took it preupgraded and it dealt double but claims to deal triple.
- Zeal is not affected by enlightenment.
Last edited by Zu; 17 Nov, 2022 @ 9:48pm
< >
Showing 1-8 of 8 comments
Mirli  [developer] 17 Nov, 2022 @ 5:30pm 
I will look into all of these. I have no idea what is wrong with Anathema or Zeal, I didn't touch either of those.
Zu 17 Nov, 2022 @ 9:53pm 
I think I get it now. Smite accurately deals half piety + modifiers, but the brackets show what it would deal if it counted piety fully.

Apropos, minor visual unclarity: Piety does bypass intangible but Nemesis' health bar doesn't reflect this. Idk if there's a workaround.

PS: Unrelated but my opinion of purify has gone up. It scales rather well with fervor. Maaaybe you could push it to 22 though.
Mirli  [developer] 18 Nov, 2022 @ 1:48am 
Originally posted by Zu:
I think I get it now. Smite accurately deals half piety + modifiers, but the brackets show what it would deal if it counted piety fully.

Apropos, minor visual unclarity: Piety does bypass intangible but Nemesis' health bar doesn't reflect this. Idk if there's a workaround.

PS: Unrelated but my opinion of purify has gone up. It scales rather well with fervor. Maaaybe you could push it to 22 though.

Ah okay, I think I may be able to fix that Smite visual bug quite easily. The piety bar thing with nemesis is way outside my expertise, when you start getting into GUI stuff I immediately lose what coding knowledge I have.

As for the other bugs you reported, I have no idea why Zeal isn't affected by Enlightenment. I did a bit of minor keyword shuffling and hopefully that will work, if not *shrug*

Anathama was in fact factoring the wrong number. It should be accurate to what the text says now. I think I originally changed it to 2 rather than 3 for balance reasons and never updated the card text, but I think I'll keep it at 3 for now for testing.

New build should be live today at some point. Just gotta do some testing first.
Zu 18 Nov, 2022 @ 10:08pm 
You'll be happy to know that zeal properly benefits from enlightenment. :cupup:
Now you just gotta do the same thing for repentance.

Unintuitive buff interaction:
If you have fervor before gaining strength, strength gets added as is and fervor doesn't multiply it. Kindle causes this unfortunate order by itself. If you do have strength already, luckily kindle does not reverse the order, so that's good.

More smite woes:
Unexpected results with strength.
Math from today's run with smite+:
4 strength: Expected 4+4= 8 damage, dealt 7. "Does smite+ think it still gains 6 piety?"
60 fervor: Expected 4+24= 28 damage, dealt 21. "Sure seems that way."
4 str, 50 fvr: Expected 4+4+40= 48 damage, dealt 42. "May I confirm my hunch?"
What I would try just for testing:
See what happens if you make smite deal damage equal to piety, then gain some.

Low-priority stuff:
Church coffers reads exhaustive 0.
Ornate javelin reads both exhaust and exhaustive.
Solar flare still has an odd empty line after tithe.
Cards that generate wounds or doubloons could show a preview when you mouse over. Even vanilla cards do this now, like wild strike, blade dance, alpha, etc.
Last edited by Zu; 19 Nov, 2022 @ 8:41pm
Mirli  [developer] 18 Nov, 2022 @ 10:27pm 
Originally posted by Zu:
You'll be happy to know that zeal properly benefits from enlightenment. :cupup:
Now you just gotta do the same thing for repentance.

Unintuitive buff interaction:
If you have fervor before gaining strength, strength gets added as is and fervor doesn't multiply it. Kindle causes this unfortunate order by itself. If you do have strength already, luckily kindle does not reverse the order, so that's good.

More smite woes:
Unexpected results with strength and it ignores vulnerable.
Math from today's run with smite+:
4 strength: Expected 4+4= 8 damage, dealt 7. "Does smite+ think it still gains 6 piety?"
60 fervor: Expected 4+24= 28 damage, dealt 21. "Sure seems that way."
4 str, 50 fvr: Expected 4+4+40= 48 damage, dealt 42. "May I confirm my hunch?"
What I would try just for testing:
See what happens if you make smite deal damage equal to piety, then gain some.

Low-priority stuff:
Church coffers reads exhaustive 0.
Ornate javelin reads both exhaust and exhaustive.
Solar flare still has an odd empty line after tithe.
Cards that generate wounds or doubloons could show a preview when you mouse over. Even vanilla cards do this now, like wild strike, blade dance, alpha, etc.

Oh boy, more stuff to fix with Smite. That card is turning into a real pain.
I'm not sure what's going on with strength, that one may be a bit of a project to fix.

Church Coffers was fixed, update the mod (put out a quick hotfix)
Yeah Ornate Javelin I messed up. I'll fix it tomorrow, easy fix.
Solar Flare's blank line is intentional.
Generation cards is something I'll have to look into, I'm working on an old version of the code that was made before that became a major feature.
Mirli  [developer] 19 Nov, 2022 @ 5:05pm 
Originally posted by Zu:
You'll be happy to know that zeal properly benefits from enlightenment. :cupup:
Now you just gotta do the same thing for repentance.

I think repentance probably shouldn't scale with Enlightenment regardless. It's a special interaction and gaining 50% more would be extremely broken. For now I'll just keep it as is.
Zu 19 Nov, 2022 @ 10:16pm 
Sorry, smite does interact properly with vulnerable and weak.

I was confused by an actual bug present in smite+. I'm now most confident smite+ does not recognize its upgrade, so even though you do gain 9 piety, the damage scales off 6.

Furthermore, the bonus it gets from enlightenment is not added to its damage either. Punish+ into smite yields 27 piety but smite only deals 12 damage since 3 bonus piety are ignored.

Kinda funny. Both of these can happen together. Smite+ with enlightenment for 13 piety, but it only deals 3 measly damage.
Last edited by Zu; 19 Nov, 2022 @ 10:27pm
Mirli  [developer] 19 Nov, 2022 @ 10:30pm 
Huh, now I'm even more confused. Time to stare at that code for a while
< >
Showing 1-8 of 8 comments
Per page: 1530 50