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

Error when reading string #42

Open
smarth23 opened this issue May 21, 2018 · 2 comments
Open

Error when reading string #42

smarth23 opened this issue May 21, 2018 · 2 comments

Comments

@smarth23
Copy link

smarth23 commented May 21, 2018

Error I am getting,
pycomm.cip.cip_base.DataError: local variable 'data_type' referenced before assignment.

I don't see it where the variable 'data_type' is referred before a it is assigned. Also the code throws the error above once I run it, but works fine when I try to run it again.

Thank you.

@Alex-Atkinson
Copy link

@smarth23,
pycomm does not include functionality for reading strings in its default read_tag function. There is some functionality for reading a string shown in the readme file, but when checking the clx.py file there is no matching function present. Issue #17 shows some work done in supporting a string read, but this doesn't appear to have gone anywhere. Might be time for somebody to create a branch and look into this.

@Alex-Atkinson
Copy link

Did a little more checking on this today after getting ready to add this functionality to a new fork and found that the read_string function is present in GitHub but not in the PyPi version. Looks like the new version just needs to be pushed over to PyPi. According to the pycomm page there has not been an update to the PyPi version since November 1, 2015.

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

No branches or pull requests

2 participants