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

More robust server handling #368

Open
crasanders opened this issue Jul 29, 2024 · 0 comments
Open

More robust server handling #368

crasanders opened this issue Jul 29, 2024 · 0 comments

Comments

@crasanders
Copy link
Contributor

crasanders commented Jul 29, 2024

The AEPsych server receives data from clients through "tell" messages and fits a model to that data to recommend trials that it sends back to the client through "ask" messages. There are a number of things that could go wrong in this process (all of these issues should be recorded in the logs with sufficient detail to understand what happened):

  • The server cannot process ask, tell, or other messages before it has received a setup message. Right now it just crashes if this happens, but it should have a "setup required" response.
  • The server could receive ill-formatted messages. It should be able to fail gracefully, and wait to receive another message.
  • The server could fail to fit the model after an "ask" message. It should fallback to sending a Sobol trial.
  • The server could lose connection to the client. It should recognize this and wait for a new connection.
  • There are surely lots of other scenarios that the server should be prepared for.

Unit tests should be written to test all these cases and to see where the cracks are in the current error-handling.

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

1 participant