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

Weather Dependant Solar Panel Support is broken #101

Open
ChrisAdderley opened this issue Sep 2, 2024 · 3 comments
Open

Weather Dependant Solar Panel Support is broken #101

ChrisAdderley opened this issue Sep 2, 2024 · 3 comments
Labels

Comments

@ChrisAdderley
Copy link
Collaborator

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
Copy link
Contributor

Aebestach commented Sep 3, 2024

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
Copy link
Contributor

Aebestach commented Sep 3, 2024

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
Copy link
Contributor

The problem is half solved. I would like to ask you, in #91 (comment), how do you deal with the extra invalid text?
@ChrisAdderley

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants