Skip to content

Commit

Permalink
update
Browse files Browse the repository at this point in the history
  • Loading branch information
@jschlyter
jschlyter committed Mar 4, 2022
1 parent 0a9e29f commit 06e93b1
Showing 1 changed file with 512 additions and 390 deletions.

6 comments on commit 06e93b1

@elden1337
Copy link

Choose a reason for hiding this comment

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

Hi, It's probably me being a noob in the python-world. But am unable to install v 1.5.0.

_Building wheel for ciso8601 (pyproject.toml) ... error
error: subprocess-exited-with-error

× Building wheel for ciso8601 (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [12 lines of output]
running bdist_wheel
running build
running build_py
package init file 'ciso8601_init_.py' not found (or not a regular file)
creating build
creating build\lib.win-amd64-3.10
creating build\lib.win-amd64-3.10\ciso8601
copying ciso8601_init_.pyi -> build\lib.win-amd64-3.10\ciso8601
copying ciso8601\py.typed -> build\lib.win-amd64-3.10\ciso8601
running build_ext
building 'ciso8601' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
[end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for ciso8601
Failed to build ciso8601
ERROR: Could not build wheels for ciso8601, which is required to install pyproject.toml-based projects_

@jschlyter
Copy link
Member Author

Choose a reason for hiding this comment

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

Hi, It's probably me being a noob in the python-world. But am unable to install v 1.5.0.

Did you use pip3 install chargeamps ?

@elden1337
Copy link

Choose a reason for hiding this comment

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

pip install chargeamps==1.5.0 was the cause of the above.
I've also tried to manually change the version in the HA-implementation manifest.json, but no luck there (could ofc be for other reasons but feels as if it should import the packages if working?).

HA is the vanilla impl on rpi.
Error in HA logs is:

Unable to install package chargeamps==1.5.0: ERROR: Cannot install chargeamps==1.5.0 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

@elden1337
Copy link

Choose a reason for hiding this comment

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

Ok so managed to get it working on win, but required the c++ mentioned. On my ubuntu it works. But in HA it does not fyi.
and I have absolutely no idea why, otherwise id ofc make a pull request for it.

@jschlyter
Copy link
Member Author

Choose a reason for hiding this comment

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

@elden1337 I'm actually using the same iso8601 parser as home assistant, but it does indeed require some binary code. Are you using this module stand alone or via the home assistant integration? The latter is now updated to use the latest version.

@elden1337
Copy link

Choose a reason for hiding this comment

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

Using it via ha-integration.
the thing was that i felt that i wanted to help out testing directly and just swapped the package.
but since ive seen your update at it now seems to work swell.

Tack!

Please sign in to comment.