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

[Bug 🐞]: KPBS Smelter not heating #63

Closed
histalonia opened this issue Mar 5, 2022 · 4 comments · Fixed by #68
Closed

[Bug 🐞]: KPBS Smelter not heating #63

histalonia opened this issue Mar 5, 2022 · 4 comments · Fixed by #68
Assignees
Labels
bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code issue: compatibility/patch issue type

Comments

@histalonia
Copy link

histalonia commented Mar 5, 2022

Brief description of your issue

When using this with Kerbal Planetary Base Systems (KPBS), I found everything worked great except the Smelter. The smelter was not heating, and had 2 heat core settings. (thus only 6% production)

I found a workaround by modifying the smelter.cfg file that comes with KPBS and changed
line 82 from:
name = ELCoreHeat
to:
name = ModuleCoreHeat

And adjusting the value on lines 61, 75, and 83 from:
1873
to:
993

It now heats to 993, but the target efficiency goal is still 1000. I suspect with more tweeks I can make a new smelter.cfg that will work and that could be turned into a module manager patch.

Steps to reproduce

When making changes to smelter.cfg, the module needs to be removed from the craft design and then a new one readded, otherwise the changed settings will not take effect.
The smelter file can be found at:
GameData\PlanetaryBaseInc\ModSupport\Parts\Extraplanetary Launchpads\smelter.cfg

Expected behavior

The KPBS smelter should heat and operate at an optimal temperature.

Actual behavior

The smelter was not heating, and had 2 heat core settings. (thus only 6% production)

Environment

KSP 1.12.3  w/Breaking Ground and Making History

Mods: 
Kerbal_Planetary_Base_Systems-1.6.14
ExtraplanetaryLaunchpads_v6.99.1
SimpleConstruction-4.0.99.1-prerelease   (overlayed on top of EL)

I also have (Heat_Control-0.6.1) but removing it didn't change any of the smelter's behavior.

My KSP install and mods are at c:\F\KSP1203\GameData

How did you download and install this?

SpaceDock (manual install)

@github-actions
Copy link

github-actions bot commented Mar 5, 2022

Thank you. Kindly read contributiing.md, code_of_conduct.md and styleguide.md. These are boilerplate.

@histalonia histalonia changed the title [Bug 🐞]: [Bug 🐞]: KPBS Smelter not heating Mar 5, 2022
@zer0Kerbal zer0Kerbal added bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code issue: compatibility/patch issue type labels Mar 6, 2022
@zer0Kerbal
Copy link
Owner

zer0Kerbal commented Mar 6, 2022

@histalonia Forum: histalona thank you for the bug report. This is a seriously hot topic... (pun intended)
image

I believe this is the file and section of that file that needs to be addressed, especially since it probably is the culprit along with the fact that I can effect change on it but not on the KPBS file. 👍

@PART[KKAOSS_Smelter]:NEEDS[PlanetarySurfaceStructures,SimpleConstruction]:AFTER[PlanetarySurfaceStructures]
{
@description = The smelter can be used to process ore into useful metal. Do not touch! HOT!
-MODULE[ELConverter]{}
-MODULE[ELConverter]{}
-RESOURCE[MetalOre]{}
-RESOURCE[ScrapMetal]{}
RESOURCE
{
name = Ore
amount = 0
maxAmount = 50
}
MODULE
{
name = ModuleResourceConverter
ConverterName = Metal
StartActionName = Start Metal Refining
StopActionName = Stop Metal Refining
AutoShutdown = true
TemperatureModifier
{
key = 0 100000
key = 750 50000
key = 1000 10000
key = 1250 500
key = 2000 50
key = 4000 0
}
GeneratesHeat = true
DefaultShutoffTemp = .8
ThermalEfficiency
{
key = 0 0 0 0
key = 500 0.1 0 0
key = 1000 1.0 0 0
key = 1250 0.1 0 0
key = 3000 0 0 0
}
UseSpecialistBonus = true
SpecialistEfficiencyFactor = 0.2
SpecialistBonusBase = 0.05
Specialty = Engineer
EfficiencyBonus = 1
INPUT_RESOURCE
{
ResourceName = Ore
Ratio = 0.4
}
INPUT_RESOURCE
{
ResourceName = ElectricCharge
Ratio = 48
}
OUTPUT_RESOURCE
{
ResourceName = Metal
Ratio = 0.2
DumpExcess = false
}
}
MODULE
{
name = ModuleOverheatDisplay
}
MODULE
{
name = ModuleCoreHeat
CoreTempGoal = 1000 //Internal temp goal - we don't transfer till we hit this point
CoreToPartRatio = 0.1 //Scale back cooling if the part is this % of core temp
CoreTempGoalAdjustment = 0 //Dynamic goal adjustment
CoreEnergyMultiplier = 0.1 //What percentage of our core energy do we transfer to the part
HeatRadiantMultiplier = 0.05 //If the core is hotter, how much heat radiates?
CoolingRadiantMultiplier = 0 //If the core is colder, how much radiates?
HeatTransferMultiplier = 0 //If the part is hotter, how much heat transfers in?
CoolantTransferMultiplier = 0.01 //If the part is colder, how much of our energy can we transfer?
radiatorCoolingFactor = 1 //How much energy we pull from core with an active radiator? >= 1
radiatorHeatingFactor = 0.01 //How much energy we push to the active radiator
MaxCalculationWarp = 1000 //Based on how dramatic the changes are, this is the max rate of change
CoreShutdownTemp = 4000 //At what core temperature do we shut down all generators on this part?
MaxCoolant = 500 //Maximum amount of radiator capacity we can consume - 50 = 1 small
}
}

@zer0Kerbal
Copy link
Owner

@histalonia tracking the root cause - one is above, the other is was being patched twice - once from Compatibility/ISRU.cfg and again by Compatibility/KPBS.cfg

@zer0Kerbal
Copy link
Owner

@histalonia incoming changes seem to have taken:

parentUrl = PlanetaryBaseInc/ModSupport/Parts/Extraplanetary Launchpads/Smelter.cfg
	PART
	{
		name = KKAOSS_Smelter
		module = Part
		author = Nils277
		scale = 1
		rescaleFactor = 1
		node_stack_top = 0, 1.5580, 0, 0, 1, 0, 1
		node_stack_bottom = 0, -1.5580, 0, 0, -1, 0, 1
		node_stack_front = 0, 0, -0.73494, 0, 0, -1, 0
		node_stack_back = 0, 0, 0.6266, 0, 0, 1, 0
		TechRequired = advConstruction
		entryCost = 40000
		cost = 18000
		category = Utility
		subcategory = 0
		title = K&K Smelter
		manufacturer = K&K Advanced Orbit and Surface Structures
		description = The smelter can be used to process ore into useful metal. Do not touch! HOT!
		attachRules = 1,0,1,1,0
		mass = 2
		dragModelType = default
		maximum_drag = 0.2
		minimum_drag = 0.2
		angularDrag = 2
		crashTolerance = 15
		maxTemp = 2000
		bulkheadProfiles = PlanetaryBase
		tags = smelter metal ore launchpad planetary base
		MODEL
		{
			model = PlanetaryBaseInc/ModSupport/Parts/Extraplanetary Launchpads/Smelter
		}
		RESOURCE
		{
			name = Metal
			amount = 0
			maxAmount = 25
		}
		MODULE
		{
			name = ModuleResourceConverter
			ConverterName = Metal
			StartActionName = Start Metal Refining
			StopActionName = Stop Metal Refining
			AutoShutdown = true
			GeneratesHeat = true
			DefaultShutoffTemp = .8
			UseSpecialistBonus = true
			SpecialistEfficiencyFactor = 0.2
			SpecialistBonusBase = 0.05
			UseSpecialistHeatBonus = true
			SpecialistHeatFactor = 0.1
			ExperienceEffect = ConverterSkill
			EfficiencyBonus = 1
			resourceOutputName = Metal
			TemperatureModifier
			{
				key = 0 50000
				key = 750 25000
				key = 1000 5000
				key = 1250 2500
				key = 2000 2500
				key = 4000 0
			}
			ThermalEfficiency
			{
				key = 0 0 0 0
				key = 500 0.9 0 0
				key = 1000 1.0 0 0
				key = 1250 0.9 0 0
				key = 1500 0.5 0 0
				key = 3000 0.0 0 0
			}
			INPUT_RESOURCE
			{
				ResourceName = Ore
				Ratio = 1.0
				FlowMode = STAGE_PRIORITY_FLOW
			}
			INPUT_RESOURCE
			{
				ResourceName = ElectricCharge
				Ratio = 96
			}
			OUTPUT_RESOURCE
			{
				ResourceName = Metal
				Ratio = 1.0
				DumpExcess = false
				FlowMode = STAGE_PRIORITY_FLOW
			}
		}
		MODULE
		{
			name = ModuleOverheatDisplay
		}
		MODULE
		{
			name = ModuleCoreHeat
			CoreTempGoal = 1000
			CoreToPartRatio = 0.1
			CoreTempGoalAdjustment = 0
			CoreEnergyMultiplier = 0.1
			HeatRadiantMultiplier = 0.05
			CoolingRadiantMultiplier = 0
			HeatTransferMultiplier = 0
			CoolantTransferMultiplier = 0.01
			radiatorCoolingFactor = 1
			radiatorHeatingFactor = 0.01
			MaxCalculationWarp = 1000
			CoreShutdownTemp = 4000
			MaxCoolant = 500
		}
		RESOURCE
		{
			name = Ore
			amount = 0
			maxAmount = 50
		}
	}
}

zer0Kerbal added a commit that referenced this issue Mar 7, 2022
# Version 4.0.99.2-prerelease - A Little too Spicy

## This is PRE-RELEASE - and WILL break saves

* 06 Mar 2022
* Release for Kerbal Space Program [KSP 1.12.3]

## DO A CLEAN INSTALL: DELETE EXISTING THEN RE-INSTALL

## WARNING: Only [CurseForge][CRSFG:url] version will include .dll’s

### Updated

* [KPBS.cfg]
  * renamed to [KerbalPlanetaryBaseSystems.cfg] v1.0.2.0
  * [KKAOSS_Smelter]
    * split out into own file [KKAOSS_Smelter] v1.0.2.0
    * -MODULE[ELCoreHeat]{}
    * [ModuleResourceConverter]
      * [TemperatureModifier]
        * OLD:
          * key = 0 100000
          * key = 750 50000
          * key = 1000 10000
          * key = 1250 500
          * key = 2000 50
          * key = 4000 0
        * NEW:
          * key = 0 50000
          * key = 750 25000
          * key = 1000 5000
          * key = 1250 2500
          * key = 2000 2500
          * key = 4000 0
      * [ThermalEfficiency]
        * OLD:
          * key = 0 0 0 0
          * key = 500 0.1 0 0
          * key = 1000 1.0 0 0
          * key = 1250 0.1 0 0
          * key = 3000 0 0 0
        * NEW:
          * key = 0 0 0 0
          * key = 500 0.9 0 0
          * key = 1000 1.0 0 0
          * key = 1250 0.9 0 0
          * key = 1500 0.5 0 0
          * key = 3000 0.0 0 0
      * add [UseSpecialistHeatBonus] = true
      * add [SpecialistHeatFactor] = 0.1
      * add [resourceOutputName] = #SimpleConstruction_Metal_displayname // Metal]
      * change [Specialty] = Engineer to [ExperienceEffect] = ConverterSkill
      * INPUT_RESOURCE
        * [Ore] ratio was 0.4, now is 1.0
        * [ElectricCharge] ratio was 48, now 96
      * OUTPUT_RESOURCE
        * [Metal] was 0.2, now is 1.0
      * Localized
  * [KKAOSS_Workshop]
    * split out into own file [KKAOSS_Workshop] v1.0.2.0
    * [ModuleResourceConverter]
      * Localized
* [ISRU.cfg] v1.1.0.0
  * @part[*ISRU*,Smelter*] --> @part[*ISRU*,Smelter*,!KKAOSS_Smelter]
  * with KPBS installed, Smelter was being hit twice without winner winner chicken dinner
  * closes #63 - [Bug 🐞]: KPBS Smelter not heating
  * [ModuleResourceConverter]
    * Localized
    * INPUT_RESOURCE
      * [Ore] ratio was 2.5, now is 0.5
      * [ElectricCharge] ratio remains the same
    * OUTPUT_RESOURCE
      * [Metal] was 0.125, now is 0.25

### Backend

* [__deploy] v1.1.3.0
* [_gitPullIssues] v1.1.0.1
* [SimpleConstruction.version]
  * removed [KSP_VERSION_MAX], not longer needed
* removed graphics/ folder; duplicate of img/
* removed bin/ folder; empty

### Localization

* [ScienceLabs.cfg] v1.0.2.2
  * ConverterName = #SimpleConstruction_RocketParts_displayname = RocketParts
  * StartActionName = #EL_EventGUIName_StartPartProduction // Start Printing Parts
  * StopActionName = #EL_EventGUIName_StopPartProduction // Stop Printing Parts
* [KKAOSS_Workshop.cfg] v1.0.2.0
  * ConverterName = #SimpleConstruction_RocketParts_displayname = RocketParts
  * StartActionName = #EL_EventGUIName_StartPartProduction // Start Printing Parts
  * StopActionName = #EL_EventGUIName_StopPartProduction // Stop Printing Parts
* [KerbalPlanetaryBaseSystems.cfg] v1.0.2.0
  * [KKAOSS_Storage_Metal]
    * -title {}
    * title = #KBPS-Storage-Metal-titl // K&K X1200 Metal Storage
  * [KKAOSS_Storage_RocketParts]
    * -title {}
    * title = #KBPS-Storage-RP-titl // K&K X2400 Rocket Part Storage
  * [KKAOSS_Storage_MetalOre]
    * -title {}
    * title = #KBPS-Storage-Ore-titl // K&K X1200 Ore Storage
    * -description {}
    * description = #KBPS-Storage-Ore-desc
* [KKAOSS_Smelter.cfg] v1.02.0
  * ConverterName = #SCON-Metal-ConverterName // Metal
  * StartActionName = #SCON-Metal-StartActionName // Start Metal Refining
  * StopActionName = #SCON-Metal-StopActionName // Stop Metal Refining
  * resourceOutputName = #SimpleConstruction_Metal_displayname // Metal
* [ISRU.cfg] v1.1.0.0
  * ConverterName = #SCON-Metal-ConverterName // Metal
  * StartActionName = #SCON-Metal-StartActionName // Start Metal Refining
  * StopActionName = #SCON-Metal-StopActionName // Stop Metal Refining
  * resourceOutputName = #SimpleConstruction_Metal_displayname // Metal
* [en-us.cfg]v1.2.0.0
  * KerbalPlanetaryBaseSystems
    * #KBPS-Storage-Ore-titl = K&K X1200 Ore Storage
    * #KBPS-Storage-Ore-desc = A storage for ore. You can process the ore in a smelter into metal.
    * #KBPS-Storage-Metal-titl = K&K X1200 Metal Storage
    * #KBPS-Storage-RP-titl = K&K X2400 Rocket Part Storage
    * #KBPS-Smelter-desc = The smelter can be used to process ore into useful metal. Do not touch! HOT!

### Documentation

* update [readme.md] v1.6.5.9
* update [ReleaseLayout.md] v1.3.1.
* update [changelog.md] v1.4.1.0
* update [Attribution.md] v1.1.2.1
* [docs]
  * [LegalMumboJumbo]
  * [Part-Invoice]
  * [404]
  * [Localizations]
  * [ManualInstallation.md]
  * [resourceFlow.md]
    * [4.0.99.2]
* closes #66 - 4.0.99.2 Update Documentation
* closes #65 - 4.0.99.2 Verify Legal Mumbo Jumbo
* closes #67 - 4.0.99.2 Social Media

### Status

* Issues/Milestones
  * closes #64 - SimpleConstruction! (SCON) 4.0.99.2-prerelease A Little Too Spicy
* Pull Requests
  * closes #48 - 4.0.99.2-prerelease A Little Too Spicy

* [ELWorkshop]
  * newly discovered KSPFields
  * bool FullyEquipped;
  * bool - IgnoreCrewCapacity = true;
  * float - ProductivityFactor = 1f;
  * float - UnmannedProductivity;
zer0Kerbal added a commit that referenced this issue Mar 7, 2022
# Version 4.0.99.2-prerelease - A Little too Spicy

## This is PRE-RELEASE - and WILL break saves

* 06 Mar 2022
* Release for Kerbal Space Program [KSP 1.12.3]

## DO A CLEAN INSTALL: DELETE EXISTING THEN RE-INSTALL  

## WARNING: Only [CurseForge][CRSFG:url] version will include .dll’s

### Updated
  
* [KPBS.cfg]
  * renamed to [KerbalPlanetaryBaseSystems.cfg] v1.0.2.0
  * [KKAOSS_Smelter]
    * split out into own file [KKAOSS_Smelter] v1.0.2.0
    * -MODULE[ELCoreHeat]{}
    * [ModuleResourceConverter]
      * [TemperatureModifier]
        * OLD:
          * key = 0 100000
          * key = 750 50000
          * key = 1000 10000
          * key = 1250 500
          * key = 2000 50
          * key = 4000 0
        * NEW:
          * key = 0 50000
          * key = 750 25000
          * key = 1000 5000
          * key = 1250 2500
          * key = 2000 2500
          * key = 4000 0
      * [ThermalEfficiency]
        * OLD:
          * key = 0 0 0 0
          * key = 500 0.1 0 0
          * key = 1000 1.0 0 0
          * key = 1250 0.1 0 0
          * key = 3000 0 0 0
        * NEW:
          * key = 0 0 0 0
          * key = 500 0.9 0 0
          * key = 1000 1.0 0 0
          * key = 1250 0.9 0 0
          * key = 1500 0.5 0 0
          * key = 3000 0.0 0 0
      * add [UseSpecialistHeatBonus] = true
      * add [SpecialistHeatFactor] = 0.1
      * add [resourceOutputName] = #SimpleConstruction_Metal_displayname // Metal]
      * change [Specialty] = Engineer to [ExperienceEffect] = ConverterSkill
      * INPUT_RESOURCE
        * [Ore] ratio was 0.4, now is 1.0
        * [ElectricCharge] ratio was 48, now 96
      * OUTPUT_RESOURCE
        * [Metal] was 0.2, now is 1.0
      * Localized
  * [KKAOSS_Workshop]
    * split out into own file [KKAOSS_Workshop] v1.0.2.0
    * [ModuleResourceConverter]
      * Localized
* [ISRU.cfg] v1.1.0.0
  * @part[*ISRU*,Smelter*] --> @part[*ISRU*,Smelter*,!KKAOSS_Smelter]
  * with KPBS installed, Smelter was being hit twice without winner winner chicken dinner
  * closes #63 - [Bug 🐞]: KPBS Smelter not heating
  * [ModuleResourceConverter]
    * Localized
    * INPUT_RESOURCE
      * [Ore] ratio was 2.5, now is 0.5
      * [ElectricCharge] ratio remains the same
    * OUTPUT_RESOURCE
      * [Metal] was 0.125, now is 0.25
  
### Backend

* [__deploy] v1.1.3.0
* [_gitPullIssues] v1.1.0.1
* [SimpleConstruction.version]
  * removed [KSP_VERSION_MAX], not longer needed
* removed graphics/ folder; duplicate of img/
* removed bin/ folder; empty

### Localization

* [ScienceLabs.cfg] v1.0.2.2
  * ConverterName = #SimpleConstruction_RocketParts_displayname = RocketParts
  * StartActionName = #EL_EventGUIName_StartPartProduction // Start Printing Parts
  * StopActionName = #EL_EventGUIName_StopPartProduction // Stop Printing Parts
* [KKAOSS_Workshop.cfg] v1.0.2.0
  * ConverterName = #SimpleConstruction_RocketParts_displayname = RocketParts
  * StartActionName = #EL_EventGUIName_StartPartProduction // Start Printing Parts
  * StopActionName = #EL_EventGUIName_StopPartProduction // Stop Printing Parts
* [KerbalPlanetaryBaseSystems.cfg] v1.0.2.0
  * [KKAOSS_Storage_Metal]
    * -title {}
    * title = #KBPS-Storage-Metal-titl // K&K X1200 Metal Storage
  * [KKAOSS_Storage_RocketParts]
    * -title {}
    * title = #KBPS-Storage-RP-titl // K&K X2400 Rocket Part Storage
  * [KKAOSS_Storage_MetalOre]
    * -title {}
    * title = #KBPS-Storage-Ore-titl // K&K X1200 Ore Storage
    * -description {}
    * description = #KBPS-Storage-Ore-desc
* [KKAOSS_Smelter.cfg] v1.02.0
  * ConverterName = #SCON-Metal-ConverterName // Metal
  * StartActionName = #SCON-Metal-StartActionName // Start Metal Refining
  * StopActionName = #SCON-Metal-StopActionName // Stop Metal Refining
  * resourceOutputName = #SimpleConstruction_Metal_displayname // Metal
* [ISRU.cfg] v1.1.0.0
  * ConverterName = #SCON-Metal-ConverterName // Metal
  * StartActionName = #SCON-Metal-StartActionName // Start Metal Refining
  * StopActionName = #SCON-Metal-StopActionName // Stop Metal Refining
  * resourceOutputName = #SimpleConstruction_Metal_displayname // Metal
* [en-us.cfg]v1.2.0.0
  * KerbalPlanetaryBaseSystems
    * #KBPS-Storage-Ore-titl = K&K X1200 Ore Storage
    * #KBPS-Storage-Ore-desc = A storage for ore. You can process the ore in a smelter into metal.
    * #KBPS-Storage-Metal-titl = K&K X1200 Metal Storage
    * #KBPS-Storage-RP-titl = K&K X2400 Rocket Part Storage
    * #KBPS-Smelter-desc = The smelter can be used to process ore into useful metal. Do not touch! HOT!

### Documentation

* update [readme.md] v1.6.5.9
* update [ReleaseLayout.md] v1.3.1.
* update [changelog.md] v1.4.1.0
* update [Attribution.md] v1.1.2.1
* [docs]
  * [LegalMumboJumbo]
  * [Part-Invoice]
  * [404]
  * [Localizations]
  * [ManualInstallation.md]
  * [resourceFlow.md]
    * [4.0.99.2]
* closes #66 - 4.0.99.2 Update Documentation
* closes #65 - 4.0.99.2 Verify Legal Mumbo Jumbo
* closes #67 - 4.0.99.2 Social Media

### Status

* Issues/Milestones
  * closes #64 - SimpleConstruction! (SCON) 4.0.99.2-prerelease A Little Too Spicy
* Pull Requests
  * closes #68 - 4.0.99.2-prerelease A Little Too Spicy

* [ELWorkshop]
  * newly discovered KSPFields
  * bool FullyEquipped;
  * bool - IgnoreCrewCapacity = true;
  * float - ProductivityFactor = 1f;
  * float - UnmannedProductivity;
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 An issue with the system. Something isn't working... KSC... we have an error code issue: compatibility/patch issue type
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants