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

Authority on information about evaluations/experiments #1250

Closed
sondreso opened this issue Dec 7, 2020 · 3 comments
Closed

Authority on information about evaluations/experiments #1250

sondreso opened this issue Dec 7, 2020 · 3 comments

Comments

@sondreso
Copy link
Collaborator

sondreso commented Dec 7, 2020

We need a higher level concept than the ensemble evaluator to control what evaluations are running, and provide information about this to other modules and clients.

In particular, we are running into issues because we are unable to differentiate between the evaluation not having started yet and the evaluation already being done.

@markusdregi
Copy link
Contributor

markusdregi commented Jan 28, 2021

This is a later in the process, but still kind of related to #1340

@oysteoh
Copy link
Contributor

oysteoh commented Jun 29, 2021

Dealing with sockets and ports, i have a "finding" i can add to this issue.

Due to the evaluator using an instance of the config with host / port to acquire a socket we need to bind with reuse_addr flag. This because the binding could happen multiple times towards the same port. Without the flag we could experience flakyness due to the fact we never know how long it will take the server to shut down the socket and make it available for binding again. With reuse_addr the socket can be bound to even if it is in TIME_WAIT state.

@oyvindeide
Copy link
Collaborator

Will mark this as closed to BaseRunModel (experiment) is now responsible for forwarding events from both EE and the updates.

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

Successfully merging a pull request may close this issue.

5 participants