Skip to content
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

Sanity Check: duplicated properties Support page. #34

Open
Lisias opened this issue Mar 31, 2019 · 52 comments
Open

Sanity Check: duplicated properties Support page. #34

Lisias opened this issue Mar 31, 2019 · 52 comments
Assignees
Labels
enhancement New feature or request support support (no bug)
Milestone

Comments

@Lisias
Copy link

Lisias commented Mar 31, 2019

Due this stunt, properly explained here, a new Sanity Check needs to be implemented.

This stunt essentially shoves Free Scaling in every part on the GameDatabase that has "Cone" in the name!This not only mangles the prefab, but also corrupts craft files and savegames, as they are "tainted" by this misconfiguration  Once you delete the Add'On and Module Manager rebuilds the cache (and the GameDatabase), all your TweakScaled parts gets rescaled terribly wrong!

So, when you load your craft and savegames after installing such Add'On, a lot of parts are wrong (There're more patches like this! The thing is way "greedy" on this). So you "fix" them, and now your parts are incompatible with everybody that doesn't have that Add'On too. And when you uninstall the thing, your crafts and savegames became "corrupted", as the original "type" is back to be effective and Kraken knows what would happen - one of my crafts got the Cones so big that the craft ended up inside an 'egg" - unflyable. And this includes the savegames. Yeah, I need to fix them all on my installment due this.  

It worth to mention that this same stunt when applied on a non visual feature (as resource consumption and mass!) would summon the Sacred Rage of the Krakens on your KSP. This has potential to be very nasty if you don't use the latest TweakScale hat withdraw itself from such parts!

This check should:

  • Detect any duplicated properties on each Part's TweakScale module from prefab
  • Check against a "whitelist" where (hypothetical) properties that can be duplicated would be ruled out (currently none)
  • And then withdraw itself from the Part if non whitelisted properties are duplicated.
    • An error entry should be added to the log file, pinpointing to this issue.

After Math:

Since this have a highly destructive potential, as Stock parts (between others) that used to work suddenly became UnTweakScaled, this has a huge potential to ruining savegames.

problem described

problem described synthetic

The user should be warned and ideally the game must be finished. Whatever it was installed by last, it must be rolled back.

@Lisias Lisias self-assigned this Apr 2, 2019
@Lisias Lisias added bug Something isn't working Not My Fault Not my Fault! I'm innocent! :) question Further information is requested labels Apr 2, 2019
@Lisias Lisias added this to the 2.4.3.0 milestone May 5, 2019
@Lisias
Copy link
Author

Lisias commented Jun 8, 2019

Implemented on commit eea6046

@Lisias
Copy link
Author

Lisias commented Jun 9, 2019

Yep. It works… =P

182 Fatals on my main KSP installment. This is going to be interesting...

Screen Shot 2019-06-09 at 14 48 56

@Lisias
Copy link
Author

Lisias commented Jun 14, 2019

Interesting. My old Career instalment (yet on 1.4.3!!) got "only" 12 Fatalities. Way better than the "Company" installment, this one updated to 1.7.0 + MH.

The Career has a bit less Add'Ons than Company, but yet it has a good basket of them.

On a blind guess, this appears to be something that was always there, but only started to bite hard in the last year (from 1.5 ahead, perhaps?)

Screen Shot 2019-06-13 at 22 10 32

@Lisias
Copy link
Author

Lisias commented Jul 10, 2022

The Scale Migration feature implemented in the 2.4.6.x series and described on issue #236 mitigated the problem to a point this is not necessary anymore.

Uncontrolled patching is still a terrible thing, but this is not going to kill the savegame anymore - most of the time, as I can't prevent some idiot from shoving corrupted exponents into parts.

But double patching is not a FATAL anymore, so I'm closing this definitively.

@Tonkyboi1
Copy link

Help me with this error, Please?
Line in log: [LOG 21:02:16.060] [TweakScale] ERROR: FATAL Part bdWarheadSmall (Small High Explosive Warhead) has a fatal problem due having duplicated properties - see issue #34. at error:0

@Lisias
Copy link
Author

Lisias commented Dec 5, 2022

Hi, @Tonkyboi1

Help me with this error, Please?
Line in log: [LOG 21:02:16.060] [TweakScale] ERROR: FATAL Part bdWarheadSmall (Small High Explosive Warhead) has a fatal problem due having duplicated properties - see issue #34. at error:0

Please send me the whole KSP.log, as well the GameData\ModuleManager.ConfigCache and the Log\ModuleManager\MMPatch.log ones (yeah, they are scattered on the file system - I hate this too). I need these files in order to diagnose the source o the problem!

@Lisias Lisias reopened this Dec 5, 2022
@helpmetweakscale
Copy link

KSP.log

@helpmetweakscale
Copy link

MMPatch.log

@Lisias
Copy link
Author

Lisias commented Dec 11, 2022

KSP.log

Hi, @helpmetweakscale ! Sorry the delay, I'm kinda buried on issues (both here and on Real Life™).

Your problem is due "BladeTweaks". Remove the file `GameData/BladeTweaks/Squad-TweakScale.cfg" and you will be fine.

@343GuiltySpark-04
Copy link

i'm getting it for these

https://pastebin.com/1r0GnA84

@343GuiltySpark-04
Copy link

which searching the files reveals is stock parts..... well **@#@)

@Lisias
Copy link
Author

Lisias commented Dec 16, 2022

Hi, @343GuiltySpark-04 !

I need the full KSP.log in order to see who is badly patching these parts. Adding the Logs/ModuleManager/MMPatch.log will help too, as sometimes the KSP.log misses some data related to patching (as when loading from the ModuleManager.ConfigCache.

Cheers!

@TheLocust911
Copy link

TheLocust911 commented Dec 23, 2022

KSP.log
MMPatch.log

I've got 4 fatal errors related to duplicate properties, help me find root cause? My mods are currently installed via Curse Forge. (on that note it seems like CKAN seemed to do a better job of preventing conflicts lol)

log attached.

@343GuiltySpark-04
Copy link

I unfortunately don't have the log but I know it was in the prerelease version 😅 I downloaded it instead by accident.

@Lisias
Copy link
Author

Lisias commented Dec 23, 2022

@343GuiltySpark-04 ,

I unfortunately don't have the log but I know it was in the prerelease version 😅 I downloaded it instead by accident.

The problem is that without the KSP.log, I can only do blind guess…

Looking on the historical diagnosis, I found this one.

Apparently, you have some oldies lingering around. Completely remove the GameData/TweakScale folder and install TweakScale again. I think this may solve your problem, as the parts affected in your rig are the same from the post I linked above.

@Lisias
Copy link
Author

Lisias commented Dec 23, 2022

@TheLocust911 ,

I've got 4 fatal errors related to duplicate properties, help me find root cause? My mods are currently installed via Curse Forge. (on that note it seems like CKAN seemed to do a better job of preventing conflicts lol)

You definitively have the same problem I had linked on the previous post. The fix is to completly remove the GameData/TweakScale and install TS again from scratch!

@343GuiltySpark-04
Copy link

@343GuiltySpark-04 ,

I unfortunately don't have the log but I know it was in the prerelease version 😅 I downloaded it instead by accident.

The problem is that without the KSP.log, I can only do blind guess…

Looking on the historical diagnosis, I found this one.

Apparently, you have some oldies lingering around. Completely remove the GameData/TweakScale folder and install TweakScale again. I think this may solve your problem, as the parts affected in your rig are the same from the post I linked above.

I'm currently troubleshooting a unrelated issue with my modlist but once that's sorted out I'll try the prerelease version again and pull the log if it happens.

@Lisias
Copy link
Author

Lisias commented Dec 23, 2022

@343GuiltySpark-04 ,

I'm currently troubleshooting a unrelated issue with my modlist but once that's sorted out I'll try the prerelease version again and pull the log if it happens.

Just to clarify, you don't need to reinstall the pre-release, you can reinstall the current one.

The problem is that it's expected that you completely remove all files from the GameData/TweakScale folder when installing new versions, but since most people don't do it, the Release versions try to handle the mess by adding dummy files to overwrite the older ones that are not needed anymore, preventing old patches from lingering around.

Such precaution is not feasible when installing the Pre-Release (Beta) version, and then installing back the Release (mainstream) one.

By completely removing the TweakScale's folder you have now, and then reinstalling whatever the TweakScale version you wanna run, you will be sure that there will be no lefties lingering around, screwing up the patches.

@343GuiltySpark-04
Copy link

I did the crash was when I was running the prerelease version by accident.

@TheLocust911
Copy link

Yuppers, worked like a charm

@aussiemilitary
Copy link

aussiemilitary commented Jul 6, 2023

ive gotten 9 fatal errors, theyre all from stock parts
log: https://drive.google.com/file/d/1nCZ4WDj3Q-tN9yF-D8sQSv9shAAouDyb/view?usp=drive_link

ive already reinstalled tweakscale like 3 times

@Lisias
Copy link
Author

Lisias commented Jul 6, 2023

Hi, @aussiemilitary !

ive already reinstalled tweakscale like 3 times

Reinstalling TweakScale will never solve these problems, because they are caused by 3rd Parties - so TweakScale is just a messenger for something that it's wrong on your rig.

ive gotten 9 fatal errors, theyre all from stock

Found the problem:

[LOG 18:48:05.071] Applying update LShipParts/LShipPartsRequired/LBP_tweakableComplete/@PART[small*]:NEEDS[TweakScale] to Squad/Parts/Cargo/CargoContainers/smallCargoContainer.cfg/PART[smallCargoContainer]

Unfortunately, not all patches are written the best way, and some had aged pretty badly. This is one of them.

You need to install https://github.com/TweakScale/Companion_SMCE/releases or, even better, https://github.com/TweakScale/Companion/releases (with everything and the kitchen's sink).

Additionally, there's a file on LShipPartsRequired that need to be replaced. You need to copy this file into the GameData/LShipPartsRequired directory, replacing the one that it's there. This is highly irregular, but it's the way I found to fix the problem, as the patches on that replaced file are really, really overeaching parts from everybody else's add'ons (Stock included).

Please note that your LShipPartsRequired is wrongly installed. You need to put this directory directly into GameData, and not inside any other!

Cheers!

@Lisias Lisias added this to the 2.4.7.7 milestone Mar 24, 2024
@Lisias Lisias modified the milestones: 2.4.8.0, 2.4.8.x Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request support support (no bug)
Projects
None yet
Development

No branches or pull requests