Displacement feature is incomplete and questionable design choices #18
Comments
|
Hello! Thank you for test. Regarding to About the speed. I the VMF you loading doesn't contain About the fix you made with Also. Could you provide the VMF you tried to import? |
Thanks, that would be amazing. For me it's quite a daunting task but your dedication to this is fascinating.
gameInfoPath case sensitivityI'm assuming you mean the .to_lower() thing? Basically, when I encase it in brackets or take out but without encasing them, I get these messages: What this basically means is that materials will not load because your addons are looking for paths on my system that do not exist, because my paths have upper case letters in them. The one simple fix that I did managed to fix it for me, but for windows, this may be a bad thing? (I don't know) I also had to change my config to not end in a / (Slash)
Otherwise, I'd get messages like which you'd think the Original code (VMTManager.gd)Static func getTexture Static func getTextureInfo but instead do this: My editsStatic func getTexture Static func getTextureInfo and set my config to DemonstrationI would demonstrate that this works, but vtflib is crying at me right now because of the space in "Godot Engine" telling me that "Unknown argument: Engine/VMFTest/mesa/materials/tools/" but because I care, I'll move mesa to my Development folder to demonstrate. Just gotta change my gameInfoPath to
Must be some sort of argument error, tried running it myself and it thinks the space is a new argument. Models import too, but their textures seemingly do not.
The VMF FileThe VMF I tried to import can be acquired by going to https://gamebanana.com/mods/122542 and converting the bsp to vmf by using BSPSrc. Github will not let me upload vmf files, I would upload to pastebin but it is so extremely long and you might need the custom resources for extensive testing. You can get BSPSrc from here: https://github.com/ata4/bspsrc/releases Just drag in the surf mesa file, or click add and find it. Make sure you enable this setting to get the folder
|
|
@Lachrymogenic UPD: Forgot to add your fix regarding to path. I'll do it later. |
Absolutely! Gonna avoid mistakes like last time and just create a new folder and copy over my config... Alas, we are one step closer, my friend! Absolutely phenomenal work! |
|
In regards to materials, it got stuck on creating a few because it was missing a directory,
Guessing the rest of it is just textures I have yet to unpack or substitute, but the models use their own custom textures, which doesn't seem to be working. I should probably go unpack game files for testing purposes. There are so many new files inside of my Counter Strike Source installation now, more textures have loaded in, except those rocks.
|
|
Looks good! |
|
Sure! First attempt loading stuff in and all the models are here, with their textures which is sweet. However, nothing else loaded in? The lava that was there is now not, and no matter what I do, putting the files somewhere else that isn't uppercase or removing .to_lower() they are not loading in, which is strange. There's no error messages or warn messages telling me that VMTs are missing either.
I'm not entirely sure why that's happening, even when changing Perhaps this is intentional as you are only testing model imports? |
|
@Lachrymogenic
|
|
I tried with this config, I pressed "Reload Config" it's not working: Still just white. Is there an issue with my config? |
|
@Lachrymogenic {
"entitiesFolder": "res://examples/entities",
"gameInfoPath": "/home/lachrymogenic/Development/BSPSrc/surf_mesa_d/",
"instancesFolder": "res://examples/instances",
"materialsFolder": "res://examples/materials",
"mdl2obj": "/home/lachrymogenic/Development/Godot Engine/GodotVMF-feature-VTFTool/mdl2obj",
"modelsFolder": "res://examples/models",
"nodeConfig": {
"defaultTextureSize": 512,
"fallbackMaterial": null,
"generateCollision": true,
"generateCollisionForModel": true,
"ignoreTextures": [
"TOOLS/TOOLSNODRAW"
],
"importModels": true,
"importScale": 0.025,
- "overrideModels": true,
+ "overrideModels": false,
- "textureImportMode": 1
+ "textureImportMode": 2
},
} |
|
Yep, works!
Though, I'm not sure if theres a way you can fix this but there might be memory leaks This kind of ram usage comes from multiple reimporting though, here is what it is on 1 reimport. Subsequent imports are showing an increase in ram usage.
|
|
Yea. Definitely it needs to be fixed. Thank you. I guess i know where the issue is |
|
Not sure if it's related to the issue, but I am noticing an increase in error messages like this:
|
|
@Lachrymogenic And there are some breaking changes regarding to config. Check the installation docs again :) |
|
I've tried it and things are definitely looking a lot better this time around, but I'm not exactly sure what I should be testing for, here. So, I'll try loading and reloading the same map to see if ram is negatively affected or not. After doing so, ram was not affected so much, started at about 4400 MB and is now at 4560 MB with no signs of increasing on further reimports After reimporting one more time it's now 4600 MB, another reimport, still 4600 MB and when importing a different map, 4300 MB. So definitely no memory leaks this time around, if there are any, then they are very minor.
You might find improvements to your project by testing more maps, but that's completely up to you, ofcourse.
As to whether this is BSPSrc's fault or the mapper's fault, I have no idea (I think it's the mapper's fault), but it makes loading assets for other maps a pain, because you would assume that mappers keep everything all lower cased and for the most part they do, but it seems like not always. Overall, great update, I'm happy to see that RAM Usage isn't as bad as it was before! |
|
That's good! Thank you for test! |
First and foremost
I'm somewhat of an experienced developer, so I thought about fixing stuff myself, but I'm not too confident in knowing how GodotVMF works yet and I don't feel like making commits to the project if my use case is going to be extremely specific, so I thought I'd let you know about these kinds of things, because for the past few days, I've been trying to get a custom map, particularly a surf map, to load for testing by using GodotVMF.
Before update 1.2.0 (Displacement) I was able to get the map to load correctly, although the models were missing their textures and everything else was too because I hadn't extracted the vpk. A full map and no textures is completely fine because I can just substitute the missing textures with my own materials, but as of right now, importing the map doesn't seem to work.
First, though, I need to talk about my setup, here is my config:
My config
just incase you're wondering, yes, I put vtflib in a completely different location and yes it is a shell file but it does indeed work, it runs wine and passes all the arguments to the wine executable.
Attempting to study how the code works, I set my gameinfo folder to the folder that BSPSrc produces when you extract a map with the "Extract embedded files" option on. I moved this folder to the project folder. On the old version, it works and imports in custom materials from that map. I also had to change the code because materials and other stuff wouldn't load in general because for some reason the code tries to apply
.to_lower()to the path files, for what reason is somewhat unknown. Simply encasing code likewith
is what allows me to use my gameinfo path, which includes uppercase letters. I've often had to restart and reload the project multiple times in order to get it working correctly.
Everything was fine in the old version except for an issue where I cannot load custom model materials. But as of right now, my output is spamming this: `Vertex amount (8) must be a multiple of the amount of vertices required by the render primitive (3)' and I'm not sure why though I can only conclude that the Displacement feature is unfinished and other commits have broken functionality of the importer, such as the merging of "develop" into the main branch. This is what it looked like before and after
Before
(I had to redo the before image and models take too much time to load and materials are being a pain rn so I didn't load them in, just pretend that they're there and some textures on the wall are actually there in this image etc.)
After
With all this being said, I'm not sure if there is anything else that I can provide to help diagnose the issue, maybe I'm the issue, because in trying to revert back to the old version, I have managed to screw myself over and now absolutely NO entities are rendering... except they are now for some reason?
Reverted to commit eedac7d
I reverted to the version I was using 3 days ago and moved some parts of my config from res://assets (my own folder) to res://examples and renamed the name of the map to use all lower case, turn off import models in the config and loaded up a completely different map and I've managed to get entities to import again
Very weird, I'm going to attempt to study more about how this code works and then modify it to work with my use case, get it to load model materials, but I still thought I should let you know and also that there may be things you might want to tweak, like using all lower case stuff for file paths is a pretty bad idea when most people have capitals in their file paths, I don't know if Windows disregards it or something but Linux for sure doesn't like that.
To be honest, I should have studied it first instead of blindly trying to load in decompiled BSPSrc maps and expecting it to just run perfectly, but if I do manage to learn enough about your project that I'm able to fix some things, I still might be apprehensive to commit due to the potential possibility that whatever works for you breaks for me and vice versa, though I am really keen on using and potentially helping with this project.
To conclude
.to_lower()is there sometimes.I don't expect others to do all the work for my issues but what I'm worried about is that the devs of this project have real working model support and displacement support that just works for them but not for me and by optimizing it to work for me, it won't work for them. This might be related to the fact that I am using decompiled maps instead of maps made with Hammer++
The text was updated successfully, but these errors were encountered: