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

namespace clash in ChargeUpPower #214

Open
URARUA opened this issue Aug 4, 2023 · 7 comments
Open

namespace clash in ChargeUpPower #214

URARUA opened this issue Aug 4, 2023 · 7 comments
Assignees
Labels
bug Something isn't working

Comments

@URARUA
Copy link

URARUA commented Aug 4, 2023

How to reproduce

  1. start a new game with marisa
  2. get full charged in battle
  3. crash

Crash log

15:02:08.436 ERROR core.CardCrawlGame> Exception caught
java.lang.ArrayIndexOutOfBoundsException: 2
at marisa.powers.Marisa.ChargeUpPower.updateDescription(ChargeUpPower.kt:52) ~[MarisaContinued.jar:?]
at com.megacrit.cardcrawl.actions.common.ApplyPowerAction.update(ApplyPowerAction.java:255) ~[?:?]
at com.megacrit.cardcrawl.actions.GameActionManager.update(GameActionManager.java:179) ~[?:?]
at com.megacrit.cardcrawl.rooms.AbstractRoom.update(AbstractRoom.java:325) ~[?:?]
at com.megacrit.cardcrawl.dungeons.AbstractDungeon.update(AbstractDungeon.java:2532) ~[?:?]
at com.megacrit.cardcrawl.core.CardCrawlGame.update(CardCrawlGame.java:876) ~[?:?]
at com.megacrit.cardcrawl.core.CardCrawlGame.render(CardCrawlGame.java:423) [?:?]
at com.badlogic.gdx.backends.lwjgl.LwjglApplication.mainLoop(LwjglApplication.java:225) [?:?]
at com.badlogic.gdx.backends.lwjgl.LwjglApplication$1.run(LwjglApplication.java:126) [?:?]
Controllers: removed manager for application, 0 managers active
Game crashed.
Version Info:

  • Java version (1.8.0_144)
  • Slay the Spire (12-18-2022)
  • ModTheSpire (3.30.3)
    Mod list:
  • basemod (5.52.3)
  • stslib (2.10.0)
  • LazyManKits (1.4.10)
  • Anm2Player (1.3.1)
  • VUPShionMod (7.1.0)
  • actlikeit (1.2.7)
  • Gensokyo (4.0.0)
  • keinemod (0.4.1)
  • MarisaContinued (3.2.0)
  • Library of Ruina (1.1.6)
  • Library of Ruina Extra (1.1.1)
  • intentgraph (0.2.1)
  • ALocalizationUpdate (0.1.0)
  • betterAltar (1.4)
  • Better_Animation (0.2.0)
  • BetterBeginnings (1.4.0)
  • betterLab (1.0)
  • betterMatch (1.1)
  • betterNote (1.3)
  • betterStone (1.1)
  • betterSkull (1.2)
  • betterThird (1.5)
  • betterrewardsmod (4.5.2)
  • neoe_growUp (0.1)
  • loadout (0.9.4)
  • mintyspire (2.5.5)
  • OptimizeTheSpire (1.1.0)
  • neoe_removeCardDiscount (0.1)
  • uncapped (1.1.1)
  • gm-reskin (1.0.1)
  • AKDsMoreRelics (1.4.8)
  • ThisIsPowerMod (1.0.1)
  • StSModEnergizedSpire (1.14.0)

Mods in stacktrace:

  • MarisaContinued (3.2.0)
    Cause:
    java.lang.ArrayIndexOutOfBoundsException: 2
    at marisa.powers.Marisa.ChargeUpPower.updateDescription(ChargeUpPower.kt:52)
    at com.megacrit.cardcrawl.actions.common.ApplyPowerAction.update(ApplyPowerAction.java:255)
    at com.megacrit.cardcrawl.actions.GameActionManager.update(GameActionManager.java:179)
    at com.megacrit.cardcrawl.rooms.AbstractRoom.update(AbstractRoom.java:325)
    at com.megacrit.cardcrawl.dungeons.AbstractDungeon.update(AbstractDungeon.java:2532)
    at com.megacrit.cardcrawl.core.CardCrawlGame.update(CardCrawlGame.java:876)
    at com.megacrit.cardcrawl.core.CardCrawlGame.render(CardCrawlGame.java:423)
    at com.badlogic.gdx.backends.lwjgl.LwjglApplication.mainLoop(LwjglApplication.java:225)
    at com.badlogic.gdx.backends.lwjgl.LwjglApplication$1.run(LwjglApplication.java:126)
@URARUA URARUA added the bug Something isn't working label Aug 4, 2023
@scarf005
Copy link
Owner

scarf005 commented Aug 5, 2023

sorry, i wasn't able to reproduce the issue:

English

image

Simplified chinese

image

Traditional chinese

image

could you explain in more detail how the bug happened?

@scarf005
Copy link
Owner

scarf005 commented Aug 5, 2023

maybe it could be due to namespace collision like in #211 (comment)

@scarf005
Copy link
Owner

scarf005 commented Aug 5, 2023

could you update to version v4.0.0 on steam and check if the issue still persists? as this update properly adds namespace to resources/ directory, it may solve the issue.

related: https://github.com/scarf005/Marisa/releases/tag/v4.0.0

@scarf005 scarf005 added the info needed this issue requires more information to be correctly addressed. label Aug 5, 2023
@URARUA
Copy link
Author

URARUA commented Aug 5, 2023

could you update to version v4.0.0 on steam and check if the issue still persists? as this update properly adds namespace to resources/ directory, it may solve the issue.

related: https://github.com/scarf005/Marisa/releases/tag/v4.0.0

sorry 4 delay, i'll test later

@URARUA
Copy link
Author

URARUA commented Aug 5, 2023

could you update to version v4.0.0 on steam and check if the issue still persists? as this update properly adds namespace to resources/ directory, it may solve the issue.

related: https://github.com/scarf005/Marisa/releases/tag/v4.0.0

update to v4.0.1 still crashed, i'll disable some mods and try again

12:12:46.863 INFO patches.RunStartEndPatches> Run Started
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:Constricted-缠绕
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:GrandCrossPower-增幅
Shion Flash Power:ChargeUpPower-充-充电时间
Shion Flash Power:ChargeUpPower-充-充电时间
12:13:22.605 ERROR core.CardCrawlGame> Exception caught
java.lang.ArrayIndexOutOfBoundsException: 2
	at marisa.powers.Marisa.ChargeUpPower.updateDescription(ChargeUpPower.kt:52) ~[MarisaContinued.jar:?]
	at com.megacrit.cardcrawl.actions.common.ApplyPowerAction.update(ApplyPowerAction.java:255) ~[?:?]
	at com.megacrit.cardcrawl.actions.GameActionManager.update(GameActionManager.java:179) ~[?:?]
	at com.megacrit.cardcrawl.rooms.AbstractRoom.update(AbstractRoom.java:325) ~[?:?]
	at com.megacrit.cardcrawl.dungeons.AbstractDungeon.update(AbstractDungeon.java:2532) ~[?:?]
	at com.megacrit.cardcrawl.core.CardCrawlGame.update(CardCrawlGame.java:876) ~[?:?]
	at com.megacrit.cardcrawl.core.CardCrawlGame.render(CardCrawlGame.java:423) [?:?]
	at com.badlogic.gdx.backends.lwjgl.LwjglApplication.mainLoop(LwjglApplication.java:225) [?:?]
	at com.badlogic.gdx.backends.lwjgl.LwjglApplication$1.run(LwjglApplication.java:126) [?:?]
Controllers: removed manager for application, 0 managers active
Game crashed.
Version Info:
 - Java version (1.8.0_144)
 - Slay the Spire (12-18-2022)
 - ModTheSpire (3.30.3)
Mod list:
 - basemod (5.52.3)
 - stslib (2.10.0)
 - LazyManKits (1.4.10)
 - Anm2Player (1.3.1)
 - VUPShionMod (7.1.0)
 - actlikeit (1.2.7)
 - Gensokyo (4.0.0)
 - keinemod (0.4.3)
 - MarisaContinued (4.0.1)
 - Library of Ruina (1.1.6)
 - Library of Ruina Extra (1.1.1)
 - intentgraph (0.2.1)
 - ALocalizationUpdate (0.1.0)
 - betterAltar (1.4)
 - Better_Animation (0.2.0)
 - BetterBeginnings (1.4.0)
 - betterLab (1.0)
 - betterMatch (1.1)
 - betterNote (1.3)
 - betterStone (1.1)
 - betterSkull (1.2)
 - betterThird (1.5)
 - betterrewardsmod (4.5.2)
 - neoe_growUp (0.1)
 - loadout (0.9.4)
 - mintyspire (2.5.5)
 - OptimizeTheSpire (1.1.0)
 - neoe_removeCardDiscount (0.1)
 - gm-reskin (1.0.1)
 - AKDsMoreRelics (1.4.8)
 - ThisIsPowerMod (1.0.1)
 - StSModEnergizedSpire (1.14.0)
 - Koishi514_Mod (1.1.2)
 - Saigyouji_Yuyuko_mod (1.2.1)

Mods in stacktrace:
 - MarisaContinued (4.0.1)
Cause:
java.lang.ArrayIndexOutOfBoundsException: 2
	at marisa.powers.Marisa.ChargeUpPower.updateDescription(ChargeUpPower.kt:52)
	at com.megacrit.cardcrawl.actions.common.ApplyPowerAction.update(ApplyPowerAction.java:255)
	at com.megacrit.cardcrawl.actions.GameActionManager.update(GameActionManager.java:179)
	at com.megacrit.cardcrawl.rooms.AbstractRoom.update(AbstractRoom.java:325)
	at com.megacrit.cardcrawl.dungeons.AbstractDungeon.update(AbstractDungeon.java:2532)
	at com.megacrit.cardcrawl.core.CardCrawlGame.update(CardCrawlGame.java:876)
	at com.megacrit.cardcrawl.core.CardCrawlGame.render(CardCrawlGame.java:423)
	at com.badlogic.gdx.backends.lwjgl.LwjglApplication.mainLoop(LwjglApplication.java:225)
	at com.badlogic.gdx.backends.lwjgl.LwjglApplication$1.run(LwjglApplication.java:126)```

@URARUA
Copy link
Author

URARUA commented Aug 5, 2023

sorry, i wasn't able to reproduce the issue:

English

image

Simplified chinese

image

Traditional chinese

image

could you explain in more detail how the bug happened?

hello i found the reason
it got conflicted with Library of Ruina

@scarf005
Copy link
Owner

scarf005 commented Aug 5, 2023

image

thanks for the info. the problem is that both marisa and LOR uses same name ChargeUpPower, which lead to the mod loading LOR descriptions incorrectly. to fix it, mod id needs to be appended to all translation strings. will try handling it within this week as this is a big change.

@scarf005 scarf005 removed the info needed this issue requires more information to be correctly addressed. label Aug 5, 2023
@scarf005 scarf005 changed the title Crashed while get power full charged namespace clash in ChargeUpPower Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

No branches or pull requests

2 participants