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

Resolved FileNotFoundErrors encountered in pymapconv.py #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DriesDD
Copy link

@DriesDD DriesDD commented Mar 6, 2024

  • minimapthread subprocess.Popen now uses the shell to resolve FileNotFoundError
  • ensure that Temp directories are created to prevent errors like FileNotFoundError: [Errno 2] No such file or directory: 'temp\thread2\temp6.BMP'
  • Fixed another FileNotFoundError related to nvdxt.exe by rewriting the path code

Resolved FileNotFoundErrors encountered at runtime on Windows.
- minimapthread subprocess.Popen now uses the shell to resolve FileNotFoundError
- ensure that Temp directories are created to prevent errors like FileNotFoundError: [Errno 2] No such file or directory: 'temp\\thread2\\temp6.BMP'
- Fixed another FileNotFoundError related to nvdxt.exe by rewriting the path code
@DriesDD
Copy link
Author

DriesDD commented Mar 6, 2024

This is my fix to use the script by fixing errors in the order I encountered them.
However, I am not familiar with the project so this could use testing/review.

Also, there is one final error from line 966 with minimap.dds not being found that I could resolve by changing the code to look for minimap.bmp which is generated correctly, and while the script now runs to completion it still doesn't generate a working minimap so I don't want to commit that. I guess the .dds extension is not a mistake but the file is not created properly.

@Beherith
Copy link
Owner

Beherith commented Mar 26, 2024

The minimap dds binary blob gets embedded into the .smf output file, and is thus required for successful compilation.

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