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

Fix battles run mappings in chaos dungeons #500

Merged
merged 2 commits into from
Nov 13, 2023

Conversation

RINGULARITTY
Copy link
Contributor

@RINGULARITTY RINGULARITTY commented Nov 11, 2023

There are issues in Battle log runs. In this exemple I did Giant AH, Dragon AH, Necro AH, Spirit AH, Steel Fortress B10 and Punisher B10 :

2023-11-11 12:02,Unknown,Win,1:05,11304,0,0,Rune,6*,13801,Swift,48.21,5,Rare,HP +360,,SPD +6,ATK 6%,,,Akhamamir,Savannah,Galleon,Veromos,Laika
2023-11-11 12:04,Unknown,Win,1:27,11366,0,0,Rune Piece x12,,,,,,,,,,,,,Verdehile,Veromos,Poseidon,Loren,Spectra
2023-11-11 12:07,Unknown,Win,1:44,10710,0,6,Symbol of Chaos x8,,,,,,,,,,,,,Abigail,Raoq,Icaru,Astar,Shamann
2023-11-11 12:09,Unknown,Win,1:46,10428,0,0,Unknown Drop,,,,,,,,,,,,,High Elemental (Fire),Raoq,Randy,Spectra,Shaina
2023-11-11 12:10,Steel Fortress B10,Win,0:44,11160,0,2,Artifact,,,Attribute (Fire),,,Hero,DEF +100,,Damage Received from Water -5%,Additional Damage by 7% of DEF,SPD Increasing Effect +10%,Damage Received from Fire -4%,Sath,Raoq,Mellia,Mellia,Tatu
2023-11-11 12:12,Spiritual Realm B10,Win,1:01,12034,0,2,Artifact,,,Archetype (Support),,,Rare,HP +1500,,Single-target skill CRIT DMG 11% on your turn,Life Drain +6%,Skill 2 Recovery +6%,SPD Increasing Effect +5%,Verdehile,Icaru,Raoq,Kro,Eirgar

As you can see, mappings are wrong. I've checked full_logs to get correct ids and I correct mapping.js.
For Abyss dungeons, It will be displayed "<dungeon name> Abyss B<stage>" where stage is 1 or 2.

@Xzandro
Copy link
Owner

Xzandro commented Nov 13, 2023

Thank you. Seems like mappings weren't necessarily wrong, they were just not updated/complete. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants