stlehmann / pyads

Python wrapper for TwinCAT ADS
MIT License
265 stars 96 forks source link

String and string array fixes #419

Open RobertoRoos opened 2 months ago

RobertoRoos commented 2 months ago

Trying to improve string handling to solve some of the open issues.

Frankly I'm a bit lost what the issues are exactly, I'm trying to go through them slowly. EDIT: Okay, I think this is about it for now. I think this resolves all string issues?

Resolves:

Should resolve (but I haven't reproduced the issue before):

Obsoletes other PRs:

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 10849949250

Details


Totals Coverage Status
Change from base Build 10633721828: 0.05%
Covered Lines: 1729
Relevant Lines: 1820

💛 - Coveralls
RobertoRoos commented 2 months ago

Okay, I think I'm done. @chrisbeardy whadda think? @stlehmann maybe?