feature: add use of map server for collision data #764
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When attempting to run this bot on windows 11 I kept encountering an error when generating the collision map.
After tracing through and attempting to run the commands directly I found it was returning the somewhat well known but still mysterious
ACCESS VIOLATION (0xc0000005)exception when the map toolkoolo-map.exeattempts to load the game.Since the tool was already based off the existing diablo2 tool` I attempted to use the map server instead of the included tool. I was able to successfully generate the collision data using the tool, presumably because it starts the game inside the container using wine instead of under windows.
Finally, it appears the included tool has modifications to the output, mainly a
roomscollection that is required for the bot to function properly.With all of this information I copied the included tool in place of the binary included in the referenced map generator's Dockerfile.
This provides a number of advantages: