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

Socket Closure Errors in simulink_gym Wrapper #15

Open
parisarnj opened this issue Nov 20, 2024 · 3 comments
Open

Socket Closure Errors in simulink_gym Wrapper #15

parisarnj opened this issue Nov 20, 2024 · 3 comments

Comments

@parisarnj
Copy link

Hello,

I'm currently training an agent using the simulink_gym wrapper with a custom Simulink model. During the training process, at the end of each episode, I receive the following messages:

INFO: Something went wrong while closing socket (('127.0.0.1', 55322), 42312)
INFO: Something went wrong while closing socket (('127.0.0.1', 55320), 42313)

I also noticed the following comment in the close function of the CommSocket class:

This catches an error appearing after some time in the training

process. It seems that the socket used to send data to the

Simulink model is closing before its close() method is called.

The reasons have to be investigated (#TBD).

The training continues despite these messages, but I'm concerned about whether this might negatively impact the training process.

  • Does this comment indicate that the error I'm encountering is related to the simulink_gym wrapper code?
  • Should I be concerned about how these socket errors might affect the training performance of my agent?
  • Do you have any suggestions on how I can solve it?

Thank you in advance for your assistance.

@johbrust
Copy link
Owner

johbrust commented Dec 3, 2024

Hi,
this should not impact your training process since it is only occuring after the episode when the socket is to be closed anyway.

@parisarnj
Copy link
Author

Thank you so much for your response.

Just to confirm, can we say that there won't be any interruption in transferring data between Python and MATLAB during the actual training process?

@johbrust
Copy link
Owner

johbrust commented Dec 4, 2024

Yes, that should be the case.

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