Skip to content

WIP: New plugin - smartcar#451

Open
gpaulbot wants to merge 5 commits intomasterfrom
add-smartcar
Open

WIP: New plugin - smartcar#451
gpaulbot wants to merge 5 commits intomasterfrom
add-smartcar

Conversation

@gpaulbot
Copy link
Copy Markdown

@gpaulbot gpaulbot commented Jul 9, 2021

nymea-plugins pull request checklist:

  • Make sure the pull request's title is of format "Plugin name: Add support for xyz" or "New plugin: Plugin name"

  • Did you test the changes on hardware, if not (e.g. absence of required hardware), please mention a person to confirm it has been tested.

  • Did you update the plugin's README.md accordingly?

  • Did you update translations (cd builddir && make lupdate)?

  • If you added a new plugin, should it be added to nymea-plugins-all or nymea-plugins-maker?

@nymea-jenkins
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jul 9, 2021

CLA assistant check
All committers have signed the CLA.

Comment thread smartcar/requirements.txt Outdated
@mzanetti mzanetti added the experimental Branches with this tag will be built in the experimental track label Jul 9, 2021
@mzanetti
Copy link
Copy Markdown
Contributor

mzanetti commented Jul 9, 2021

Also, please add the dpkg files so it will be packaged by CI:

  • add package in debian/control
  • add debian/nymea-plugin-smartcar.install.in

Not sure if we need to add it to any meta package (last point from checklist), probably not as the vast majority of users out there won't have a supported car yet.

nymea-jenkins added a commit that referenced this pull request Jul 9, 2021
nymea-jenkins added a commit that referenced this pull request Jul 9, 2021
nymea-jenkins added a commit that referenced this pull request Jul 10, 2021
nymea-jenkins added a commit that referenced this pull request Jul 11, 2021
nymea-jenkins added a commit that referenced this pull request Aug 2, 2021
nymea-jenkins added a commit that referenced this pull request Aug 2, 2021
nymea-jenkins added a commit that referenced this pull request Aug 3, 2021
nymea-jenkins added a commit that referenced this pull request Aug 5, 2021
nymea-jenkins added a commit that referenced this pull request Aug 5, 2021
nymea-jenkins added a commit that referenced this pull request Aug 6, 2021
nymea-jenkins added a commit that referenced this pull request Aug 7, 2021
nymea-jenkins added a commit that referenced this pull request Aug 7, 2021
nymea-jenkins added a commit that referenced this pull request Aug 9, 2021
nymea-jenkins added a commit that referenced this pull request Aug 9, 2021
nymea-jenkins added a commit that referenced this pull request Aug 9, 2021
nymea-jenkins added a commit that referenced this pull request Aug 12, 2021
nymea-jenkins added a commit that referenced this pull request Aug 13, 2021
nymea-jenkins added a commit that referenced this pull request Aug 16, 2021
nymea-jenkins added a commit that referenced this pull request Aug 17, 2021
nymea-jenkins added a commit that referenced this pull request Aug 17, 2021
nymea-jenkins added a commit that referenced this pull request Aug 17, 2021
nymea-jenkins added a commit that referenced this pull request Aug 17, 2021
nymea-jenkins added a commit that referenced this pull request Aug 18, 2021
nymea-jenkins added a commit that referenced this pull request Aug 18, 2021
nymea-jenkins added a commit that referenced this pull request Sep 28, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Sep 30, 2021
nymea-jenkins added a commit that referenced this pull request Oct 6, 2021
nymea-jenkins added a commit that referenced this pull request Oct 12, 2021
nymea-jenkins added a commit that referenced this pull request Oct 12, 2021
nymea-jenkins added a commit that referenced this pull request Oct 13, 2021
nymea-jenkins added a commit that referenced this pull request Oct 15, 2021
nymea-jenkins added a commit that referenced this pull request Oct 17, 2021
nymea-jenkins added a commit that referenced this pull request Oct 17, 2021
nymea-jenkins added a commit that referenced this pull request Oct 21, 2021
nymea-jenkins added a commit that referenced this pull request Oct 21, 2021
nymea-jenkins added a commit that referenced this pull request Oct 21, 2021
nymea-jenkins added a commit that referenced this pull request Oct 21, 2021
nymea-jenkins added a commit that referenced this pull request Oct 22, 2021
nymea-jenkins added a commit that referenced this pull request Oct 22, 2021
nymea-jenkins added a commit that referenced this pull request Oct 22, 2021
nymea-jenkins added a commit that referenced this pull request Oct 23, 2021
nymea-jenkins added a commit that referenced this pull request Oct 24, 2021
Comment thread debian/control
Architecture: any
Depends: ${shlibs:Depends},
${misc:Depends},
nymea-plugins-translations,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Latest nymea-plugins doesn't have a single translations package any more but every plugin will ship its own translations within the pluign package.

@@ -0,0 +1,3 @@
smartcar/integrationpluginsmartcar.json usr/lib/@DEB_HOST_MULTIARCH@/nymea/plugins/smartcar/
smartcar/integrationpluginsmartcar.py usr/lib/@DEB_HOST_MULTIARCH@/nymea/plugins/smartcar/
smartcar/requirements.txt usr/lib/@DEB_HOST_MULTIARCH@/nymea/plugins/smartcar/
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

here, the .qm files need to be added

"displayName": "Vehicle",
"id": "77ddd9f7-7c16-4bf3-bff0-c3a2208c8c66",
"createMethods": ["auto"],
"interfaces": ["battery"],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

an "electricvehicle" interface exists nowadays which should be implemented in this one.

pollTimers[thing.id] = threading.Timer(value, pollService, [thing.id])
pollTimers[thing.id].start()

def createPollTimer(thing):
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nymea's python api gained support for nymea.PluginTimer. No need to start own threads any more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants