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

[guntamatic] Initial contribution: Binding for Guntamatic Heating Systems #12011

Merged
merged 12 commits into from
Jan 11, 2022

Conversation

MikeTheTux
Copy link
Contributor

@MikeTheTux MikeTheTux commented Jan 9, 2022

Initial contribution: Binding for Guntamatic Heating Systems (Biostar, Powerchip, Powercorn, Biocom, Pro, Therm)

I have implemented a Binding that can be used to monitor and control Guntamatic Heating Systems (https://www.guntamatic.com/en/).

The Guntamatic Binding was developed and tested using Guntamatic Biostar 15kW Pellets Heating System running Firmware 3.2d.
It should work for all other Guntamatic Heating Systems as well, that support the same web interface:

  • Biostar (tested via 15kW, firmware 3.2d)
  • Powerchip (untested)
  • Powercorn (untested)
  • Biocom (untested)
  • Pro (untested)
  • Therm (untested)

Forum topic:

Test Build:

Final Build:

@MikeTheTux MikeTheTux requested a review from a team as a code owner January 9, 2022 18:25
@MikeTheTux
Copy link
Contributor Author

MikeTheTux commented Jan 9, 2022

@lolodomo please find here the new PR, started from current OH3.3.0 SNAPSHOT HEAD.
Previous PR: #11598

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo
Copy link
Contributor

lolodomo commented Jan 9, 2022

I submitted a PR to your repo including the default translation properties.

@jlaur jlaur added the new binding If someone has started to work on a binding. For a new binding PR. label Jan 9, 2022
Default translation properties
@MikeTheTux
Copy link
Contributor Author

MikeTheTux commented Jan 9, 2022

Bad news: i18n still fails

D:\openhab-addons-guntamatic\bundles\org.openhab.binding.guntamatic>mvn i18n:generate-default-translations
[INFO] Scanning for projects...
[WARNING] The POM for org.eclipse.m2e:lifecycle-mapping:jar:1.0.0 is missing, no dependency information available
[WARNING] Failed to retrieve plugin descriptor for org.eclipse.m2e:lifecycle-mapping:1.0.0: Plugin org.eclipse.m2e:lifecycle-mapping:1.0.0 or one of its dependencies could not be resolved: Failure to find org.eclipse.m2e:lifecycle-mapping:jar:1.0.0 in https://repo1.maven.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced
[WARNING] The POM for org.openhab.core.tools:i18n-maven-plugin:jar:3.3.0-SNAPSHOT is missing, no dependency information available
[WARNING] Failed to retrieve plugin descriptor for org.openhab.core.tools:i18n-maven-plugin:3.3.0-SNAPSHOT: Plugin org.openhab.core.tools:i18n-maven-plugin:3.3.0-SNAPSHOT or one of its dependencies could not be resolved: Failure to find org.openhab.core.tools:i18n-maven-plugin:jar:3.3.0-SNAPSHOT in https://repo1.maven.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced
Downloading from central: https://repo1.maven.org/maven2/org/codehaus/mojo/maven-metadata.xml
Downloading from central: https://repo1.maven.org/maven2/org/apache/maven/plugins/maven-metadata.xml
Downloading from openhab-release: https://openhab.jfrog.io/openhab/libs-release/org/codehaus/mojo/maven-metadata.xml
Downloading from openhab-release: https://openhab.jfrog.io/openhab/libs-release/org/apache/maven/plugins/maven-metadata.xml
Downloaded from central: https://repo1.maven.org/maven2/org/codehaus/mojo/maven-metadata.xml (20 kB at 41 kB/s)
Downloaded from central: https://repo1.maven.org/maven2/org/apache/maven/plugins/maven-metadata.xml (14 kB at 28 kB/s)
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  5.152 s
[INFO] Finished at: 2022-01-09T19:29:34+01:00
[INFO] ------------------------------------------------------------------------
[ERROR] No plugin found for prefix 'i18n' in the current project and in the plugin groups [org.apache.maven.plugins, org.codehaus.mojo] available from the repositories [local (C:\Users\mike\.m2\repository), central (https://repo1.maven.org/maven2), openhab-release (https://openhab.jfrog.io/openhab/libs-release)] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/NoPluginFoundForPrefixException

@MikeTheTux
Copy link
Contributor Author

I submitted a PR to your repo including the default translation properties.

reviewed and merged ...

@lolodomo
Copy link
Contributor

lolodomo commented Jan 9, 2022

@MikeTheTux : I submitted a very last PR to your repo with improvements for your thing handler.
What is required is the additional exception catching in parseAndJsonInit.
Using constants from CoreItemFactory is better.
All the other changes are more suggestions to you.

lolodomo and others added 3 commits January 9, 2022 23:50
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Improvements in the thing handler
Signed-off-by: Michael Weger <weger.michael@gmx.net>
@MikeTheTux
Copy link
Contributor Author

@MikeTheTux : I submitted a very last PR to your repo with improvements for your thing handler. What is required is the additional exception catching in parseAndJsonInit. Using constants from CoreItemFactory is better. All the other changes are more suggestions to you.

reviewed, merged and updated

Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

Code LGTM

@lolodomo
Copy link
Contributor

@MikeTheTux : do you want time for retesting or do you give me the go for merging?

@MikeTheTux
Copy link
Contributor Author

@MikeTheTux : do you want time for retesting or do you give me the go for merging?

Please let me do some final testing before the merge.

@MikeTheTux
Copy link
Contributor Author

Code LGTM

go for it 🚀

thanks for all your effors. I owe you one 🍺

@lolodomo lolodomo merged commit 7b0ba30 into openhab:main Jan 11, 2022
@lolodomo lolodomo added this to the 3.3 milestone Jan 11, 2022
@lolodomo
Copy link
Contributor

@MikeTheTux : thank your for contributing this new binding.

Now, you could add your binding's logo to the openHAB website. See https://www.openhab.org/docs/developer/bindings/#add-your-binding-s-logo-to-the-openhab-website

@MikeTheTux
Copy link
Contributor Author

@MikeTheTux : thank your for contributing this new binding.

Now, you could add your binding's logo to the openHAB website. See https://www.openhab.org/docs/developer/bindings/#add-your-binding-s-logo-to-the-openhab-website

openhab/openhab-docs#1733

moesterheld pushed a commit to moesterheld/openhab-addons that referenced this pull request Jan 18, 2022
…tems (openhab#12011)

Signed-off-by: Michael Weger <weger.michael@gmx.net>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
…tems (openhab#12011)

Signed-off-by: Michael Weger <weger.michael@gmx.net>
NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Apr 27, 2022
…tems (openhab#12011)

Signed-off-by: Michael Weger <weger.michael@gmx.net>
Signed-off-by: Nick Waterton <n.waterton@outlook.com>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
…tems (openhab#12011)

Signed-off-by: Michael Weger <weger.michael@gmx.net>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
…tems (openhab#12011)

Signed-off-by: Michael Weger <weger.michael@gmx.net>
Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new binding If someone has started to work on a binding. For a new binding PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants