search

actionml / harness

Harness is a Machine Learning/AI Server with plugins for many algorithms including the Universal Recommender
Apache License 2.0
283 stars 49 forks source link

separation of engine ops and validation of params #298

Open pferrel opened 3 years ago

pferrel commented 3 years ago

we should add better synchronous validation when engine ops happen now that they are async.

pferrel commented 3 years ago

With etcd we have moved the validation of engine params into the deferred processing of the engine-add notification. In the past validation was synchronous with the engine-add command and there was no notification. The effect of this separation of validation means virtually no validation happens synchronous with the angine-add REST call. We can refactor this is to support more validation.

engine add, and update are affected

pferrel commented 3 years ago

only applies to Harness 1.0+