| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004488 | The Dark Mod | GUI | public | 05.03.2017 14:20 | 23.04.2021 07:14 |
| Reporter | AluminumHaste | Assigned To | stgatilov | ||
| Priority | normal | Severity | normal | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Platform | x86-64 | OS | Windows | OS Version | 7 |
| Product Version | TDM 2.05 | ||||
| Fixed in Version | TDM 2.09 | ||||
| Summary | 0004488: Image previews are sometimes black | ||||
| Description | Some missions shots are completely black. Checking the console shows that it can't load previewshot_gatehouse_2014-01-29_19.24 Checked the _missionshots folder and the name is: previewshot_gatehouse_2014-01-29_19.24.32859.jpg The filename parsing is completing on the first period. | ||||
| Steps To Reproduce | Open TDM Download Missions Select Gatehouse Click "More..." button Screenshots show black. Check console shows WARNING: Couldn't load image: fms/_missionshots/previewshot_gatehouse_2014-01-29_19.24 | ||||
| Additional Information | The ideal fix would be a code change in filename parser, as this might affect other parts of the engine elsewhere. Otherwise, all screenshots that were uploaded to mission database would have to be downloaded, renamed, re-uploaded. | ||||
| Tags | No tags attached. | ||||
 |
See screenshot.png for some idea of what is happening. The images are downloading from the server correctly, but not being rendered properly. Going to check some other missions where the images load correctly and see if there is any difference with what is downloaded. Full Moon Fever works correctly, and it saves the image to same location (__missionscreenshot.temp). So it is something happening between downloading the temp page and displaying it on the GUI Also I have a really bad keyboard which doesn't let me open the console (even with rebound keys in the config file... New one should arrive on Monday and will help debugging) |
|
|
Got the console working with this new keyboard, wish the warning message would tell me which file it was from. But I will look at this again today |
|
|
More progress: https://pastebin.com/icncMgbf Starts off with the stack of where FAILED TO LOAD IMAGE is called from Currently debugging the Script_Set function, but not easy |
|
|
Screenshot 2 shows the image failing to load when the extension is forced on it Screenshot 3 shows the image failing to load when the absolute path and extension are forced on it. Maybe it's something inside R_LoadImageProgram not liking all the periods in the filename |
|
|
Screenshot 4 shows where the lexer is reading the filename incorrectly! We are getting there! Screenshot 5 shows it reading correctly (from Exhumed FM) |
|
|
R_ParseImageProgram_r calls idLexer::ReadToken twice when 'More' is pressed for The Gatehouse Works correctly the first pass Works incorrectly the second pass src.buffer (\n replaced by actual newlines to increase readability) material fms/_missionshots/previewshot_gatehouse_2014-01-29_19.24.32859 // IMPLICITLY GENERATED { \t{ \tblend blend \tcolored \tmap \"fms/_missionshots/previewshot_gatehouse_2014-01-29_19.24.32859\" \tclamp ... The difference is the quotes around the filename in the one that fails ReadName is being called for the first one ReadString is being called for the one wrapped in quotes |
|
|
May have gotten ReadName and ReadString the wrong way around. I have added LEXFL_ONLYSTRINGS to R_LoadImageProgram as the filename has hyphens in. This nearly solves the problem, but it's still having trouble with the extension |
|
|
The think the next problem is DefaultFileExtension in R_LoadImage due to the periods in the files |
|
|
I DID IT!!! Going to clean up code and send patch This took way longer than expected Fix is slightly hacky though |
 |
Patch? |
|
|
Ping |
 |
I sent him a PM on the forums, hopefully he sees it. |
 |
So we're waiting for AlexDiru to provide us his 4-month-old 'slightly hacky' patch to fix this problem? |
|
|
Looks like. |
|
|
I will look at this problem. |
|
|
Gatehouse's preview picture filenames have a different format than those for other missions. I.e. here's a Gatehouse picture: previewshot_gatehouse_2014-01-29_19.23.20420.jpg And here's one from NHAT: previewshot_anoott_4143.jpg If there's any number of dots before the file extension, it screws up loading the picture. It looks like the correct fix is to fix the filenames. |
|
|
This problem will be fixed by changing the names of the preview pictures to remove the extraneous dots. No point in futzing with low-level TDM code, trying to accommodate the dots. |
 |
Indeed, we should not change the common image loading code in the engine. Too much depends on it, and too many obstacles to make it work with bad names. Instead, I simply added some code to sanitize screenshot filenames in svn rev 8892. For example, here is a name of screenshot file saved for Gatehouse: previewshot_gatehouse_2014_01_29_19_24_32859.jpg As you see, all dots and hyphens (and other bad chars) are replaced with underscores. Followed by minor refactoring in svn rev 8893. |
