[1.12.x] KSP Part Volumes

linuxgurugamer · January 27, 2022

I have a new beta for this:

0.0.3.1 BETA

Added missing line to config file (was not needed, but nice to have) for processManipulableOnly option
Added tank volume calculations
Added code to clamp window on screen
Now hides the button if no changes are detected
Thanks to user @yalov for these:
- add stackable support: parts, that is fitted at least two times in maxStackCommonVolume, have stackableQuantity, but no more than maxPartsInStack
- add packedVolume to ModuleCargoPart Bypass reasons info
- change patch, so an index of ModuleCargoPart is not changed
- support blacklistRegexPattern, and move KerbalEVA there
- update settings window (checkbox for all fillers)

https://www.dropbox.com/s/ia29si4kdgjzssw/KSP_PartVolume-0.0.3.1.zip?dl=0

The feature to hide the button is experimental. Please let me know if you think I should leave it in or not

flart · January 27, 2022

I have found some problems in the beta:

enable "process maniputable-only parts" and "stock parts"

after every restart and without any changes, the restart window will be appearing

This problem was also on the previous version.

New problem: with the settings as above, the patch looks like that

// Squad/Parts/Electrical/z-4kBattery/z-4kBattery/batteryBankLarge
// Dimensions: x: 2.63, y: 0.25, z: 2.50
// Bounding Box Size: 1645.653 liters
// Volume adjustment: 10%
//
@PART[batteryBankLarge]:HAS[@MODULE[ModuleCargoPart]]:Final
{
    @MODULE[ModuleCargoPart]
    {
        %packedVolume = 1811
    MODULE
    {
        name = ModuleCargoPart
        packedVolume = 1811
        %KSP_PartVolume = true
    }
}

linuxgurugamer · January 27, 2022

34 minutes ago, flart said:

enable "process maniputable-only parts" and "stock parts"

You might need to restart 2x

35 minutes ago, flart said:

New problem: with the settings as above, the patch looks like that

That's a problem, thanks. Please check the first again, restarting 2x without making any changes

flart · January 28, 2022

10 hours ago, linuxgurugamer said:

You might need to restart 2x

I have restarted 3 times, the restart message still appears

linuxgurugamer · January 28, 2022

2 hours ago, flart said:

I have restarted 3 times, the restart message still appears

Strange, I did test that, but will look into it. Besides, the file is wrong with that option anyway, so that does need fixing.

Edit: The syntax issue was an easy fix, was a result of the massive merge I did, and when I had to fix conflicts, I missed one set of duplicated lines

Edited January 28, 2022 by linuxgurugamer

linuxgurugamer · January 28, 2022

New beta, fixes both the syntax error and the constant asking to restart game:

Fixed parts having ModuleInventoryPart getting a cargo module added
Added file versioning, with automatic deletion of old file versions
Fixed having manipuable parts constantly getting told to restart game

https://www.dropbox.com/s/hvvcgww1wawsejf/KSP_PartVolume-0.0.3.2.zip?dl=0

flart · January 28, 2022

6 hours ago, linuxgurugamer said:

New beta, fixes both the syntax error and the constant asking to restart game:

looks good,
what are the Calculated tank volume and Calculated tankVol (max x min) volume

// ----------------------------------------------------------------------
//      Calculated tank volume: 20978.8
//      Calculated tankVol (max x min) volume: 7556.8
// Squad/Parts/Aero/airlinerWings/MainWing/airlinerMainWing
// Dimensions: x: 11.33, y: 7.82, z: 0.92
// Bounding Box Size: 81664.4 liters
// Volume adjustment: 10%
//
//   Bypass reasons:
//      size exceeds largestAllowablePart: 64000
//
// ----------------------------------------------------------------------

linuxgurugamer · February 4, 2022

New release, 0.0.3.1

Added missing line to config file (was not needed, but nice to have) for processManipulableOnly option
Added tank volume calculations
Added code to clamp window on screen
Now hides the button if no changes are detected
Thanks to user @yalov for these:
- add stackable support: parts, that is fitted at least two times in maxStackCommonVolume, have stackableQuantity, but no more than maxPartsInStack
- add packedVolume to ModuleCargoPart Bypass reasons info
- change patch, so an index of ModuleCargoPart is not changed
- support blacklistRegexPattern, and move KerbalEVA there
- update settings window (checkbox for all fillers)
Fixed parts having ModuleInventoryPart getting a cargo module added
Added file versioning, with automatic deletion of old file versions
Fixed having manipulable parts constantly getting told to restart game

wrdiffin · April 26, 2022

I just wanted to make RestockPlus Oscar-A Fuel Tanks into Cargo Parts like the larger stock Oscar-B tanks, and against my better judgement installed this mod. It aggressively interfered with the Main Menu and kept telling me to restart the game, but otherwise apparently did nothing at all after running the game three times, so I uninstalled it. Only after uninstalling the mod, I found that nearly every single item was now a Cargo Item, including giant fuel tanks, in spite of the mod already being uninstalled. Oscar-A Fuel Tanks even took up more volume than Oscar-B Fuel Tanks twice their size. I tried deleting the partVolume.cfg file and restarting to fix the problem, but to no avail.

So now I've had to uninstall the whole game, delete everything in my Kerbal Space Program folder, reinstall the game, and reinstall all of my mods. In fact installing this mod practically wasted two days of my playing time what with all of the KSP restarts and downloading and installing I've had to do. And so I just thought I'd share my experience here for the benefit of other players. Thanks.

MoonstreamInSpace · May 2, 2022

Would it be possible to add a button to ignore if the part has an inventory? It would be annoying to whitelist every one. (I'm hoping to use this with Sandcastle)

linuxgurugamer · May 2, 2022

On 4/26/2022 at 5:59 PM, wrdiffin said:

I just wanted to make RestockPlus Oscar-A Fuel Tanks into Cargo Parts like the larger stock Oscar-B tanks, and against my better judgement installed this mod. It aggressively interfered with the Main Menu and kept telling me to restart the game, but otherwise apparently did nothing at all after running the game three times, so I uninstalled it. Only after uninstalling the mod, I found that nearly every single item was now a Cargo Item, including giant fuel tanks, in spite of the mod already being uninstalled. Oscar-A Fuel Tanks even took up more volume than Oscar-B Fuel Tanks twice their size. I tried deleting the partVolume.cfg file and restarting to fix the problem, but to no avail.

So now I've had to uninstall the whole game, delete everything in my Kerbal Space Program folder, reinstall the game, and reinstall all of my mods. In fact installing this mod practically wasted two days of my playing time what with all of the KSP restarts and downloading and installing I've had to do. And so I just thought I'd share my experience here for the benefit of other players. Thanks.

If you has asked for help, I would have been able to tell you the one file you needed to delete. By design it leaves the file behind, and should be documented in the OP. I'll check that and add a note if necessary .

Edit: this behavior is documented in the OP, I just added a second note to point out that it needs to be deleted to fully delete the mod

Edited May 2, 2022 by linuxgurugamer

linuxgurugamer · May 2, 2022

40 minutes ago, MoonstreamInSpace said:

Would it be possible to add a button to ignore if the part has an inventory? It would be annoying to whitelist every one. (I'm hoping to use this with Sandcastle)

What kind of inventory? Stock inventory?

MoonstreamInSpace · May 3, 2022

20 hours ago, linuxgurugamer said:

What kind of inventory? Stock inventory?

Yes. The partVolume_v2.cfg file said that they were blocked due to having ModuleInventoryPart.

aat · May 4, 2022

A few questions I didn't find a complete answer looking through the topic:

What "Process manipulable-only parts" does exactly?
What "Allow stackable parts" does? What is a stackable part in the first place?
Does "Include stock parts" re-calculate the volume of stock parts, or only calculates the volume for stock parts with missing ModuleCargoPart?

Edited May 4, 2022 by aat

flart · May 4, 2022

12 hours ago, MoonstreamInSpace said:

The partVolume_v2.cfg file said that they were blocked due to having ModuleInventoryPart.

the game does not allow to put an inventory into inventory, so it is stock limitation

5 hours ago, aat said:

What "Process manipulable-only parts" does exactly?

What "Allow stackable parts" does? What is a stackable part in the first place?

Does "Include stock parts" re-calculate the volume of stock parts, or only calculates the volume for stock parts with missing ModuleCargoPart?

the checkbox add parts to the partVolume_v2.cfg, that could be manipulated in the construction mode, but could not be put in the inventory (internally they have volume -1)
small parts could be in stack - several items in one cell in the inventory grid
only calculates the volume for stock parts with missing ModuleCargoPart

linuxgurugamer · May 4, 2022

16 hours ago, MoonstreamInSpace said:

Yes. The partVolume_v2.cfg file said that they were blocked due to having ModuleInventoryPart.

So what's the problem, then? The mod itself already ignores parts which have an inventory

MoonstreamInSpace · May 5, 2022

On 5/4/2022 at 8:32 AM, linuxgurugamer said:

So what's the problem, then? The mod itself already ignores parts which have an inventory

On 5/4/2022 at 4:28 AM, flart said:

the game does not allow to put an inventory into inventory, so it is stock limitation

I see the problem now. Sorry about that.

Edited May 5, 2022 by MoonstreamInSpace

LEC · May 31, 2022

On 12/17/2021 at 3:28 AM, boCash said:
I'm having trouble getting this mod to apply ModuleCargoPart to certain parts. This mod finds them and they aren't being bypassed/commented out in partVolumes.cfg. I think all of the offending parts already have ModuleInventoryPart and I see that they're all yielding 'packedVolume = -1' in the .cfg. I even tried hand-editing the .cfg on one part to 'packedVolume = 1000' in a hail mary to make something work but I cannot for the life of me get the gosh-darn ModuleCargoPart to apply. I'm not very familiar with how MM does its magic so please bear with my uninformed self and let me know if there's any additional data that would help or if I'm being really dumb and failed to read something in this thread.

Log

partVolumes.cfg

An offending part: \PlanetaryBaseInc\BaseSystem\Parts\Utility\Garages\garage_adapter_g.cfg

(12/22) Update after another couple hours of tinkering and learning:

True to my avatar, I failed to read something in this thread. Or rather, to process it in any meaningful way. Any time I had it set to 'Process Manipulable-only Parts', I encountered the same restart loop that DPOHbl4 mentioned. I tried the fix Gordon suggested but I still can't get ModuleCargoPart to stick. Is it something to do with the order of that and ModuleInventoryPart that was mentioned upthread? Hand-editing the part.cfg itself to add ModuleCargoPart works.

(12/23) Update after more tinkering and learning:

Yes, the issue is caused by the order of ModuleInventoryPart and ModuleCargoPart. Deleting ModuleInventoryPart before adding ModuleCargoPart solved the problem. I also readded ModuleInventoryPart, albeit with a static volume for all parts. I then removed KSP_PartVolume from the GameData folder so my precious hand edited file wouldn't be harassed.

Using NP++ and the ToolBucket plugin for multiline find+replace, I ran the following on PartVolumes.cfg:

Find:
    MODULE
    {
        name = ModuleCargoPart
        packedVolume = -1
        KSP_PartVolume = true
    }
Replace:
    !MODULE[ModuleInventoryPart] {}
    MODULE
    {
        name = ModuleCargoPart
        packedVolume = -1
        KSP_PartVolume = true
    }
    MODULE
    {
        name = ModuleInventoryPart
        InventorySlots = 4
        packedVolumeLimit = 400
    }

I tested this strange phenomena, and it seems that ModuleCargoPart has to come before ModuleInventoryPart to make this work. So I hacked some extra lines into the mod to make it output in this order for all the instances containing just ModuleInventoryPart instead of ignoring them... Only to find out after I finished writing that ModuleManager has this nifty little trick that lets you index the order of how MM does the edits.

@PART[*]:HAS[@MODULE[ModuleInventoryPart],!MODULE[ModuleCargoPart],!MODULE[KerbalEVA]]:FINAL
{
	MODULE,0
	{
		name = ModuleCargoPart
		packedVolume = -1
	}

}

This little hack works like charm for me (unless, on an unrelated note, I have NodeHelper installed which causes EVA Construction to throw whenever I try to detach a part). If it works for you I might throw it in a nice package and shoot it up to the dock.

Aelfhe1m · June 1, 2022

@LEC Be careful when inserting modules at the start. There are some stock modules that use index numbers to link to each other, so inserting something at the beginning can break those functionality. e.g. the Mobile Processing Lab:

PART
{
	...
	MODULE
	{
		name = ModuleScienceContainer
		...
	}
	MODULE
	{
		name = ModuleScienceLab
		containerModuleIndex = 0 // <==== NOTE
        ...
    }
	...
}

There are other examples, animations for instance.

It might be better to first move the ModuleInventoryPart to the end (clone new copy - $MODULE[ModuleInventoryPart] {}, then delete first copy - !MODULE[ModuleInventoryPart],0 {}), before inserting the ModuleCargoPart at the penultimate position (index = -2)

LEC · June 3, 2022

On 6/1/2022 at 3:40 AM, Aelfhe1m said:
@LEC Be careful when inserting modules at the start. There are some stock modules that use index numbers to link to each other, so inserting something at the beginning can break those functionality. e.g. the Mobile Processing Lab:
PART
{
	...
	MODULE
	{
		name = ModuleScienceContainer
		...
	}
	MODULE
	{
		name = ModuleScienceLab
		containerModuleIndex = 0 // <==== NOTE
        ...
    }
	...
}
There are other examples, animations for instance.

It might be better to first move the ModuleInventoryPart to the end (clone new copy - $MODULE[ModuleInventoryPart] {}, then delete first copy - !MODULE[ModuleInventoryPart],0 {}), before inserting the ModuleCargoPart at the penultimate position (index = -2)

@boCash

Noticed after playing today that there were some weirdness going on, so I did another hack instead that gives the same results without causing issues for the lab.

@PART[*]:HAS[@MODULE[ModuleInventoryPart],!MODULE[ModuleCargoPart],!MODULE[KerbalEVA]]:FINAL
{

	MODULE
	{
		name = ModuleCargoPart
		packedVolume = -1
	}
	MODULE
	{
		name = ModuleInventoryPart		
		InventorySlots = #$/MODULE[ModuleInventoryPart]/InventorySlots$
		packedVolumeLimit = #$/MODULE[ModuleInventoryPart]/packedVolumeLimit$
	}
	!MODULE[ModuleInventoryPart] {}
}

linuxgurugamer · June 5, 2022

On 6/2/2022 at 8:07 PM, LEC said:
@boCash

Noticed after playing today that there were some weirdness going on, so I did another hack instead that gives the same results without causing issues for the lab.
@PART[*]:HAS[@MODULE[ModuleInventoryPart],!MODULE[ModuleCargoPart],!MODULE[KerbalEVA]]:FINAL
{

	MODULE
	{
		name = ModuleCargoPart
		packedVolume = -1
	}
	MODULE
	{
		name = ModuleInventoryPart		
		InventorySlots = #$/MODULE[ModuleInventoryPart]/InventorySlots$
		packedVolumeLimit = #$/MODULE[ModuleInventoryPart]/packedVolumeLimit$
	}
	!MODULE[ModuleInventoryPart] {}
}

Nice, I’ll incorporate this into the next release

linuxgurugamer · June 8, 2022

New release, 0.0.3.2

Updated original patch to only add ModuleCargoPart if no ModuleInventoryPart is there
Thanks to forum user @LEC for this patch
- Added patch to add ModuleCargoPart and make sure ModuleInventoryPart is after ModuleCargoPart for those parts where there was a ModuleInventoryPart
Updated file version to 3

Zelda · June 9, 2022

23 hours ago, linuxgurugamer said:

New release, 0.0.3.2

Updated original patch to only add ModuleCargoPart if no ModuleInventoryPart is there

Thanks to forum user @LEC for this patch
Added patch to add ModuleCargoPart and make sure ModuleInventoryPart is after ModuleCargoPart for those parts where there was a ModuleInventoryPart

Updated file version to 3

Thanks for the update! I did encounter an error immediately after updating though, and it appears that a part in Universal Storage II is to blame. The cargo wedge config doesn't have a packedVolumeLimit value defined under ModuleInventoryPart, so it fails trying to parse the variable. I know that's a bug in Universal Storage II and I will be filing a bug report there, but I wanted to mention it here too in case others come here to report, since it wasn't apparent until this mod was updated.

jimmymcgoochie · June 28, 2022

I'm getting the following error when loading with the latest version of this mod, it seems to be similar to what Zelda reported above but relates to a different mod:

Quote

[LOG 19:06:21.052] Applying update KSP_PartVolume/AddToAllParts/@PART[*]:HAS[!MODULE[ModuleCargoPart],@MODULE[ModuleInventoryPart],!MODULE[KerbalEVA]]:FINAL to Delivery/parts/fuselage/cygnus/_cygnus_fuselage_s1p5_1.cfg/PART[cygnus_fuselage_s1p5_1]

[WRN 19:06:21.052] Cannot find key packedVolumeLimit in MODULE

[ERR 19:06:21.052] Error - Cannot parse variable search when inserting new key packedVolumeLimit = #$/MODULE[ModuleInventoryPart]/packedVolumeLimit$

The mod in question is Special Delivery - Stockalike Cygnus.

Edited June 28, 2022 by jimmymcgoochie

wreckreation · July 2, 2022

I had the same issue as above:

On 6/28/2022 at 11:17 AM, jimmymcgoochie said:

Cannot find key packedVolumeLimit in MODULE

for the Ares Cockpit from mod The Martian For KSP.

Fixed it by changing

@MODULE[ModuleInventoryPart]

to

MODULE[ModuleInventoryPart]

in line 14 in AddToAllParts.cfg.

KSP now reports no errors, and looking in Module Manager config cache shows Ares Cockpit now has (apparently) correctly configured ModuleCargoPart and ModuleInventoryPart entries:

		MODULE
		{
			name = ModuleInventoryPart
			InventorySlots = 9
		}
		MODULE
		{
			name = ModuleCargoPart
			packedVolume = 4140
		}

To summarize, fixed by removing the @ sign in front of "MODULE[ModuleInventoryPart]" in line 14 of AddToAllParts.cfg.

[1.12.x] KSP Part Volumes

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation