python-snap7

python-snap7 copied to clipboard

In the DB class, there is no canonical way to read or write the entire DB

3

When using the `DB` class, you provide initial data, and then you're able to provide a new underying `bytearray` to refresh the data. While `DB_Row` allows to write to and...

vmsh0

In the DB class, there is no canonical way to iterate on all rows when id_field is not None

2

When `id_field` is set to something that is not `None`, there is no canonical way or public API to iterate on all rows or even to obtain a list of...

vmsh0

we support arm 64-bit wheel installation, but not arm 32-bit yet.

gijzelaerr

Use black?

6

We could decide to use black. How to proceed: * Run black on the source code, see if the changes are good and commit. * Add a github actions job...

gijzelaerr

Use correct typing

3

You use typing -> Great You return the wrong Type -> Not so Great You use strange typing -> okey but not Great Examples: wrong use: util get_byte returns int,...

debauer

we get a warning during `build-push-image` actions step: ``` Warning: Input 'repository' has been deprecated with message: v2 is now available through docker/build-push-action@v2 Run docker/build-push-action@v1 ```

gijzelaerr

how to recover from an intermittent TCP timeout exception

8

What is the proper way to recover from a snap7 exception ISO : An error occurred during recv TCP : Connection timed out? If I place a db_read() into a...

tmcolby

gijzelaerr

gijzelaerr

switch to CFFI

5

CFFI is an easier method to create bindings. It also simplifies the binary wheels creation process, greatly simplifying the installation procedure. https://cffi.readthedocs.io/en/latest/

gijzelaerr