Your comments
We fixed all things discussed in this thread - will be available in upcoming release 6.0.8
Hi again,
I'm actively working on integrating your changes into the codebase right now. While I'm at it, we've also made some improvements to the connection handling (making it non-blocking) and enhanced the documentation for the S7 interface.
I'll send you an updated version of the interface later today for testing. Would you be able to test it on your setup when you get it? Your real-world testing would be incredibly valuable to ensure everything works smoothly.
One quick note: I've marked your post as "private" to keep the code snippets exclusive to our customers only. I hope that's okay – just want to make sure we're protecting the value for everyone who has purchased the product.
Thanks again for your patience and collaboration on this!
Best regards,
Thomas
Hi there,
Thank you so much for taking the time to share your detailed feedback and for sending over your updated S7Interface.cs file – we really appreciate it!
I want to sincerely apologize if our previous communication gave you the impression that we weren't taking your feedback seriously. That's absolutely not the case, and I'm sorry if it came across that way. We genuinely value your input and have been carefully reviewing the issues you've highlighted.
You're right that there are some areas we need to improve, and your code contributions are incredibly helpful. Since we support over 20 different interfaces, we can't claim to be deep experts in every PLC protocol, which is why detailed feedback from users like you is so valuable to us.
To make sure we're on the same page, let me confirm what I understand:
- You've identified data type issues in the AreaReadWriteMode functionality
- You've created an updated version of S7Interface.cs that addresses these issues
- You'd like us to review and incorporate these fixes
A couple of questions to help us move forward:
- Were you able to test the interface version we sent you yesterday? We'd love to hear how that worked for you.
- Can you please take some time to explain us more - concerning the XML Import - what we should change or add on your optinion
We're committed to getting this right, and your expertise is helping us improve the product for everyone. Please don't hesitate to reach out if you need anything else!
Best regards,
Thomas
Hi, thanks for all the hint and advices. For 1-4 please check attached update of the S7Interface.
For 6 - if i understand it correctly - your proposal is to do a new importer which is also able to read dbs? Please explain us a little bit more what we should do based on your discovery.
Also you have your own implementation of S7 protocoll which is faster - if i understand it correctly?
Best regards
Thomas
S7TCPIP.unitypackage
Hi, are you sure that the gizmos are turned on - can you show us a screenshot of the demo scene where one drive (e.g. Axis1) is selected in Hierarchy?
Shoe sorter is fixed in the beta - will be available in upcoming release next week.
Is solved - will be available next week in V6.0.7 update / patch
Is solved in current beta - will be available in upcoming Simulation 6.0.7 Release
Customer support service by UserEcho
Hello,
thank you for sharing your issue.
We only support Unity 6 LTS versions. You need to open your project with a Unity 6000.0.x version. The Sentis package should default to version 2.1.3 and resolve the issue