KSP-RO / ProceduralParts

A continuation of StretchySRBs, which is a continuation of StretchyTanks
72 stars 79 forks source link

Procedural Xenon Tank always has Default Capacity #355

Open bdleitner opened 2 months ago

bdleitner commented 2 months ago

Apologies for the double-post if improper, I realized after posting in the forum that maybe it should just be a bug report:

I've been noticing odd behavior before with the procedural xenon tank, like if I revert a launch the tank stays the same size but ends up nearly empty.

I'm using KSPIE (full mod list attached) and just starting to experiment with beamed power so I'm experimenting with warping ships into position just to check out my understanding of how things work before I go launch for real. With my latest craft I noticed that Kerbal Engineer was saying I'd have ~10k dV but then when I moved it into position I only had 91 dV! After some experimenting I was able to determine the following:

I'm not sure if this is caused by another mod interfering, or if there's a bug in the procedural xenon tank when computing capacity outside the VAB such that it always uses the default regardless of tank size.

I uploaded my mod list and KSP.log to this github repo.

My educated guess based on observed behavior would be that when a ship is loaded, the values for amount and maxAmount are recalculated but the capacity is just using the default and not taking the part size into account.

siimav commented 2 months ago

I would assume that resource amounts are handled by InterstellarFuelSwitch. Not at all familiar with that modstack and unless the issue can be replicated in a minimal stockalike install, there's pretty much 0 chance of anyone from PP maintainers investigating this.

siimav commented 2 months ago

However what you could try is installing an older version of PP, like 2.5.0.0.

bdleitner commented 2 months ago

Good idea to try that, so I tried v2.5.0.0 and reloaded the save with the "fixed" values, but the same thing happened. I filed an issue with KSPIE to see if they might know if there's a conflict with InterstellarFuelSwitch.