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

Enhancement for /modbus_tk/defines.py #25

Closed
GoogleCodeExporter opened this issue Apr 17, 2015 · 4 comments
Closed

Enhancement for /modbus_tk/defines.py #25

GoogleCodeExporter opened this issue Apr 17, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

Added FC7 (READ_EXCEPTION_STATUS), FC8 (DIAGNOSTIC) and FC23 
(READ_WRITE_MULTIPLE_REGISTERS)

Original issue reported on code.google.com by petar.e...@gmail.com on 12 Oct 2011 at 11:33

Attachments:

@GoogleCodeExporter
Copy link
Author

Hello Petar,
Thanks for submitting this implementation for the missing function codes. I've 
included it to the current dev version.
I don't have time currently to unit-test it. If you a little more time taht 
would be perfect.
Are you interested in joining the project maintainers? Just mail if you are ok.
Best
luc

Original comment by luc.jean@gmail.com on 12 Oct 2011 at 12:18

  • Changed state: Fixed
  • Added labels: Type-Enhancement
  • Removed labels: Type-Patch

@GoogleCodeExporter
Copy link
Author

Issue 24 has been merged into this issue.

Original comment by luc.jean@gmail.com on 12 Oct 2011 at 12:19

@GoogleCodeExporter
Copy link
Author

Hi Luc,
I also don't have time for thorough testing. I implemented those function codes 
for the needs of my own project and they just work. So I assume they should 
work more or less for other people too.
I'm not sure I have the competence to participate in the project and or now I 
prefer to submit bug reports or additions which arise in the course of my work. 
I'm glad you offered though!
Best regards
Petar

Original comment by petar.e...@gmail.com on 12 Oct 2011 at 1:02

@GoogleCodeExporter
Copy link
Author

OK, I'll try to find a little time for unit-testing but if you have tested it 
on your project, it is ok.
Thanks for your contribution in any form you like :)
Best
luc

Original comment by luc.jean@gmail.com on 12 Oct 2011 at 1:46

  • Changed state: Verified

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

No branches or pull requests

1 participant