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

Unable to load Diablo save game due to invalid monster animation #7509

Open
AJenbo opened this issue Nov 4, 2024 Discussed in #7508 · 1 comment
Open

Unable to load Diablo save game due to invalid monster animation #7509

AJenbo opened this issue Nov 4, 2024 Discussed in #7508 · 1 comment
Labels
vanilla Bugs found also in the original Diablo 1.09b release
Milestone

Comments

@AJenbo
Copy link
Member

AJenbo commented Nov 4, 2024

Discussed in #7508

On dlvl 7, there is an Illusion Weaver that is walking northeast, but its animation info is invalid (ticksPerFrame is zero). It's causing a divide-by-zero error when trying to convert the level.

https://github.com/user-attachments/files/17621051/Saves.zip

@AJenbo AJenbo added the vanilla Bugs found also in the original Diablo 1.09b release label Nov 4, 2024
@AJenbo AJenbo added this to the 1.5.4 milestone Nov 4, 2024
@StephenCWills
Copy link
Member

The debugger stops on this line.

offset = monster.position.CalculateWalkingOffset(monster.direction, monster.animInfo);

Ronb1238 said that the save was created in vanilla Diablo, and that he only downloaded DevilutionX to transfer the save to Hellfire. This is just speculation, but it could be that this is a vanilla edge case for invisible Hidden-type monsters that wouldn't occur in saves created by DevilutionX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vanilla Bugs found also in the original Diablo 1.09b release
Projects
None yet
Development

No branches or pull requests

2 participants