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

Support for Forward_Open vs. Large_Forward_Open requests #6

Open
akgoins opened this issue Feb 8, 2016 · 3 comments
Open

Support for Forward_Open vs. Large_Forward_Open requests #6

akgoins opened this issue Feb 8, 2016 · 3 comments

Comments

@akgoins
Copy link

akgoins commented Feb 8, 2016

The Session::createConnection() creates a Large_Forward_Open connection (0x5B) and uses the connection::createForwardOpenRequest() which creates a large forward open connection request which is a bit misleading. Some devices do not support a large forward connection, so for the drivers to be generalizable there needs to be two functions in the Session class. One to create a normal connection and another to create a large connection. Or else, there needs to be a parameter passed in to specify which connection is made.

@pbeeson
Copy link

pbeeson commented Mar 16, 2017

@akgoins

Did you ever get this working with regular Forward Open. Like replacing 0x5B with 0x54? If so, I'd be extremely anxious to get your Yaskawa PLC code, as I'm banging my head to use this with a Yaskawa PCL too.

@akgoins
Copy link
Author

akgoins commented Mar 16, 2017

Yes, I was able to get it running. I had to do a lot of trouble shooting using wireshark to make sure that the data was being properly sent. I will send you an example that should help.

@pbeeson
Copy link

pbeeson commented Mar 16, 2017

That would be great. I got basic reading working with OpENer, but there are large dropouts and I still haven't figured out writing.

Using this seems simpler, but so far it doesn't get past connecting as you had found.

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