post-kerbin-mining-corporation / DynamicBatteryStorage

Resolves issues with electrical power handling in Kerbal Space Program and provides user interface planning tools for heat and electrical generation.
17 stars 12 forks source link

Weather Dependant Solar Panel Support is broken #101

Open ChrisAdderley opened 2 months ago

ChrisAdderley commented 2 months ago

Curent, WDSP support is broken with the merged PR. As I don't have access to True Volumetric Clouds, I can't effectively debug this. See https://forum.kerbalspaceprogram.com/topic/186207-112x-system-monitor-electrical-planning-and-timewarp-compensation-september-1/?do=findComment&comment=4419122

@Aebestach .

Aebestach commented 2 months ago

If you need, I can send you a TVC for testing. In DBS 2.3.2, I merged my PR, tested it, and everything is OK. I will check v2.3.3 soon. The implementation logic of WeatherDrivenSolarPanel is the same as that of the new KOP.

Aebestach commented 2 months ago

I also responded to this in the forum. https://forum.kerbalspaceprogram.com/topic/186207-112x-system-monitor-electrical-planning-and-timewarp-compensation-september-1/?do=findComment&comment=4419255

This problem is mainly caused by single star mode.

Aebestach commented 2 months ago

The problem is half solved. I would like to ask you, in https://github.com/post-kerbin-mining-corporation/DynamicBatteryStorage/issues/91#issuecomment-2288219309, how do you deal with the extra invalid text? @ChrisAdderley