[gpz]

I've added a 3 line modification to fullfill this request into the today's unstable release. Could you guys please test it? (I've added this to my otherwise unrelated glTF PR, because I needed to make some modifications there to where this was added, and I couldn't have been able to create a separate PR without arising a code conflict. Code reviewers won't like it though...)

[eric from trainsim]

gpz, on 04 June 2023 - 09:50 AM, said:

I've added a 3 line modification to fullfill this request into the today's unstable release. Could you guys please test it? (I've added this to my otherwise unrelated glTF PR, because I needed to make some modifications there to where this was added, and I couldn't have been able to create a separate PR without arising a code conflict. Code reviewers won't like it though...)

Still getting grey track:

Version = U2023.06.05-0807
Warning: Ignored missing texture file c:\orts\routes\str_v2\textures\acleantrack1.dds
Warning: Ignored missing texture file c:\orts\routes\str_v2\textures\acleantrack2.dds
Warning: Ignored missing texture file c:\orts\routes\str_v2\textures\acleantrack2.dds
Warning: Ignored missing texture file c:\orts\routes\str_v2\textures\acleantrack1.dds

I think Line 173 may be wrong...

                        var pngPath = Path.ChangeExtension(path, ".png");
                        if (File.Exists(pngPath))
                            return Get(GraphicsDevice, pngPath);
                        var acePath = Path.ChangeExtension(path, ".ace");
                        if (File.Exists(pngPath))
                            return Get(acePath, defaultTexture, required, extensionFilter);
                    } 
                    if (File.Exists(path))
                        Trace.WriteLine(new FileLoadException(path, error));

Shouldn't it be ----> if (File.Exists(acePath))

[gpz]

Ah, sure, of course, my bad... Try again please, when a new version appears, shortly probably.

[eric from trainsim]

All good. You have no idea how many times I've made that same copy/paste error, which is why I knew to look for it. ;)

[eric from trainsim]

OK, I just fired up the new version, and copied over a legacy XTracks route into my TrainSimulations folder.... no DDS textures anywhere in the route.

It appears to work. ACE track textures are now showing instead of DDS.

[eric from trainsim]

Did this get uncommitted at some point?

I just installed the MLT Michigan Iron Ore route, and it's all grey tracks, with a warning "ignored missing texture file acleantrack1.dds" and "ignored missing texture file acleantrack2.dds"

Yes, we want to enable newer content formats, but I have a problem forcing end users to have to install new default track textures that override what the original route creator specified.