Skip to content

Resolved FileNotFoundErrors encountered in pymapconv.py - #37

Open
DriesDD wants to merge 1 commit into
Beherith:masterfrom
DriesDD:patch-1
Open

Resolved FileNotFoundErrors encountered in pymapconv.py#37
DriesDD wants to merge 1 commit into
Beherith:masterfrom
DriesDD:patch-1

Conversation

@DriesDD

@DriesDD DriesDD commented Mar 6, 2024

Copy link
Copy Markdown
  • 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

DriesDD commented Mar 6, 2024

Copy link
Copy Markdown
Author

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

Beherith commented Mar 26, 2024

Copy link
Copy Markdown
Owner

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