TweakScale/Companion_FS

Weird issue with SXT parts using FSBuoyancy

Closed this issue · 13 comments

Lisias commented

1)This is the part unmodified (defaultScale was set to 1.25 because of an error in the TS patch in SXT, but shouldn't matter) - the max floatBuoyancy is 50:

2)This is the part scaled up to 2.0x - note the mysterious new value for floatBuoyancy, which does not persist when launching the craft. Upon launching, right clicking on the part reveals floatBuoyancy maxed out at 102.4 or so, but adjusting floatBuoyancy changes the range back to 0-50.

3)Then, this is the part scaled back down to 1.0x - the floatBuoyancy is now more than it was at 1.25x, for some reason:

  1. Finally, here is the part scaled BACK up to 2.0 after the previous step where I scaled it down to 1.0 - new value for floatBuoyancy again:

https://forum.kerbalspaceprogram.com/index.php?/topic/179030-141151-tweakscale-under-new-management-2018-1127/&do=findComment&comment=3488699

Lisias commented

It works for me!

screen shot 2018-11-16 at 23 26 29
screen shot 2018-11-16 at 23 46 33

I need more information:

  1. KSP version used (are you using Making History�?)
  2. SXT version used
  3. I'm assuming the latest orthodox TwekScale version.

For the recored, my test was made using:

  1. KSP 1.4.5 + MH
  2. "My" fork of SXT (just because I'm testing it before a proper release - unlikely I had "fixed" something by accident….)
  3. This Version of TweakScale (Experimental, for logs)
Lisias commented

I just tested it with KSP 1.5.1 + MH. Exactly the same add-ons (I just copied from one KSP into another).

Same results: works for me.

Lisias commented

I realized that I didn't properly reproduces all the steps. Will redo the tests

Lisias commented

The behaviour was confirmed

  1. Take two equal parts with FSBuyoancy: screen shot 2018-11-17 at 13 50 45
  2. Test 1:
    1. Scale up one of them: screen shot 2018-11-17 at 13 51 06
    2. Scale down it two times (to 1.0): screen shot 2018-11-17 at 13 51 39
      • Whoopsy… The Buoyancy value is wrong!!
  3. Test 2:
    1. Scale down directly: screen shot 2018-11-17 at 13 52 12
      • Whoopsy… The Buoyancy value is wrong!!
    2. "Fly" the thing: screen shot 2018-11-17 at 13 54 10
      • Whoopsy… The Buoyancy value is wrong!!

E agora, José? — The fraction values are rounding errors, but the buoyancy values are completly messed up when you scale the thing. Why the values are rollback on launch?

Lisias commented

Another issue: This is not a round part. The 1.260 scale shouldn't be applied.

This appears to be like the parachutes, check what was done on the chutes and replicate it here.

Lisias commented

This can be related (or not) to issue TweakScale/TweakScale#11 , [see this comment](url
TweakScale/TweakScale#11 (comment)).

Lisias commented

Possible problems with MH parts too. Investigate

Playing around with rescaling other Structual Tubes looks like the scale factors are just off for all of them.  For example, if I scale down a short T-37 3.75m tube to 1.875m it visually looks to be the same size as the T-18 tube, but weighs .544t vs .075t for the short size T-18 tube. Likewise scaling up a T-18 to 3.75m weights .3t vs .6t for a stock size T-37.

https://forum.kerbalspaceprogram.com/index.php?/topic/179030-141151-tweakscale-under-new-management-2018-1127/&do=findComment&comment=3502132

Lisias commented

It appears to be a bug. "Promoting" this to a proper status.

Lisias commented

TweakScale 2.4.0.7 will "solve" this issue by dropping support for parts using FSBuoyancy. Sorry.

Proper support will be implemented in the new code tree.

Lisias commented

Since this commit kinda of "solve" the problem, I'm closing this.

TweakScale/TweakScale@3becb45

Lisias commented

This issue was transfered to TweakScaleCompanion_FS as it's a Firespitter related problem

Lisias commented

I'm reopening this issue as it will be correctly tackled down on TweakScaleCompantion_FS

Lisias commented

This issue was closed in release 0.0.1.0. #HURRAY