-
Notifications
You must be signed in to change notification settings - Fork 226
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
x64 build crashes everytime I press F9 #311
Comments
Can confirm. Happens to me too. But the problem is that it doesnt happen in Release build or the one that we download from github releases. In debug it happens every time. |
I can't reproduce this crash in either release or debug builds, so it would help if you could debug it for me. Thanks! |
I'm not at my pc now.
I wrote "build" in the title, but now I don't know if it's correct.
What I downloaded and tested are the x86/x64 zip files under "Assets".
The 3 or 4 previous versions I tried never showed this bug.
|
Ok that's unfortunate. I am still new to the code but I will try to debug to the best of my abilities Dr. Schweiger. |
|
Thank you very much for the detailed debugging instructions Dr. Schweiger. Will debug and figure out what might be causing the problem. |
After setting Planetarium=0, I can confirm that enabling any of the grids always crashes Orbiterx64 (and Orbiterx86, only in D3D9). |
OK. Are you in a position to debug (or at least compile) the code? |
Also, I just noticed from the log file you posted that you have redirected your TextureDir and HightexDir locations. Any particular reason for this? For the development builds this is probably not a good idea since you will be missing any additional textures added during development. If you didn't copy the new gridlabel.dds file from the local Textures directory to your Orbiter2016\Textures directory, it will not be found, which may well be the reason for the crash. What happens if you don't redirect these folder locations? The only directory that should be redirected is the PlanetTexDir folder, since that doesn't exist in the development build. |
…ssage is written to the log, and grid labels are not displayed. D3D9 client no longer crashes.
…ssage is written to the log, and grid labels are not displayed. D3D9 client no longer crashes. (#315)
I have now pushed a PR to main which should deal with a missing grid label texture file more gracefully. Instead of crashing, it should just not display the labels and leave an error message in the log file instead. Does this make any difference to the observed behaviour for both of you? |
The only directory that should be redirected is the PlanetTexDir folder, since that doesn't exist in the development build. |
That was it! Redirecting only PlanetTexDir and leaving the other dirs at their default results in NO crashes whatsoever. |
Thanks for the feedback. Can you also confirm that with the latest build you don't get crashes anymore even if you still redirect the TextureDir and HightexDir locations? |
Yes, confirmed, no more crashes with TextureDir and HighTexDir redirection, both x86 and x64, BUT now solar panels flicker when in external visuals (default DG docked at ISS scenario). |
Sorry, are you saying that this latest problem has only started happening with the latest commit (111577b) and only if you redirect TextureDir and HighTexDir? If that isn't what you are saying, can you please open another issue for that problem, since it doesn't seem to be connected to what was discussed here so far. Also, please provide as much details as possible (which clients, which screen modes, and ideally the commit hash when this started to happen). Thanks! |
Hi, the new dawn took the flickering away. |
No worries. Closing this issue. If it happens again, just open another issue. |
...Or if I check the corresponding box in the Options window -> Planetarium.
Logs_Orbiter-x64.zip
While reading the log, I noticed that the new PlanetTexDir entry is not logged.
The text was updated successfully, but these errors were encountered: