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

[solarforecast] Fix ForecastSolar Plane default values in documentation. #16814

Merged
merged 1 commit into from
May 27, 2024

Conversation

pre-martin
Copy link
Contributor

@pre-martin pre-martin commented May 27, 2024

As can be seen in ForecastSolarPlaneConfiguration.java there are default values of 0.25 for dampAM and dampPM. But the documentation states "N/A" which is not correct.

Edit: Updated link. The previous one was for a different ticket. Now the link points to the correct code.

Signed-off-by: Martin Renner <martin+github@planet-renner.net>
@pre-martin pre-martin requested a review from a team as a code owner May 27, 2024 18:21
@jlaur jlaur changed the title Fix ForecastSolar Plane default values in documentation. [solarforecast] Fix ForecastSolar Plane default values in documentation. May 27, 2024
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks

@lsiepel lsiepel merged commit b83bdbc into openhab:main May 27, 2024
5 checks passed
@lsiepel lsiepel added this to the 4.2 milestone May 27, 2024
@pre-martin pre-martin deleted the fix/forecastsolar-damp-doc branch May 27, 2024 20:00
@jlaur
Copy link
Contributor

jlaur commented May 27, 2024

@weymann - can you add yourself as code owner for this binding? I missed this when reviewing #13308.

@weymann
Copy link
Contributor

weymann commented May 28, 2024

I think I set these values during development in order to test these config parameters.
Seeing this now I suggest to set default value to 0. There shouldn't be any suggestion provided as default damp factor.

psmedley pushed a commit to psmedley/openhab-addons that referenced this pull request Jun 15, 2024
Signed-off-by: Martin Renner <martin+github@planet-renner.net>
Signed-off-by: Paul Smedley <paul@smedley.id.au>
pgfeller pushed a commit to pgfeller/openhab-addons that referenced this pull request Sep 29, 2024
Signed-off-by: Martin Renner <martin+github@planet-renner.net>
Signed-off-by: Patrik Gfeller <patrik.gfeller@proton.me>
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
Signed-off-by: Martin Renner <martin+github@planet-renner.net>
matchews pushed a commit to matchews/openhab-addons that referenced this pull request Oct 18, 2024
Signed-off-by: Martin Renner <martin+github@planet-renner.net>
cipianpascu pushed a commit to cipianpascu/openhab-addons that referenced this pull request Jan 2, 2025
Signed-off-by: Martin Renner <martin+github@planet-renner.net>
Signed-off-by: Ciprian Pascu <contact@ciprianpascu.ro>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants