UnrealKaraulov / newbspguy

Goldsrc map viewer/editor without decompiling. Also can view .MDL models.
The Unlicense
59 stars 11 forks source link

[enhancement] Semi-ripent (instead of No Ripent option) when merging #99

Open GarompaEstomper opened 5 months ago

GarompaEstomper commented 5 months ago

I really love what you did to bspguy, it must have been a lot of work, and now merging works just fine again :) I haven't found newbspguy crashing or anything so far, so I believe the bugs have been fixed.

Edited this text since it was too long. Right now, if you use -noscript alone, the entities get replaced by others, breaking stuff, as bspguy merging assumes you want to have squadmakers instead of monster entities. And if you add -noripent to stop it from replacing the entities, it stops it from replacing the targetnames, targets and their multi_manager references too, creating problems across merged maps as you have entities with the same names in different maps.

I would like to suggest adding a semi-ripent/rename-only option that does exactly the same as using the -noripent and -noscript option but only renames the entities adding an incremental suffix across all merged maps. Example > map1: "targetname" "dude1", map2: "targetname" "dude1" , turns into map1: "targetname" "dude1_1", map2: "targetname" "dude1_2"

So far, there is no third party tool that I have found to do this, at least not free, so this has to be done manually which is a REAL pain in the ass or you merge maps without any parameter so it adds both the scripts and renames entities, and then ignore the scripts or delete them.

Thanks for reading