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

[ML] Allowing more job config settings to be updated on reset jobs #77080

Open
droberts195 opened this issue Aug 31, 2021 · 5 comments
Open

[ML] Allowing more job config settings to be updated on reset jobs #77080

droberts195 opened this issue Aug 31, 2021 · 5 comments
Assignees
Labels
>enhancement :ml Machine learning Team:ML Meta label for the ML team

Comments

@droberts195
Copy link
Contributor

We only allow a subset of job configuration settings to be updated after the job is created. This is because updating some of the settings would cause the job config to be out-of-sync with the results and/or state. But in the case of a job that has been reset (or never opened in the first place) there are no results or state for the config to be out-of-sync with.

Therefore, in the case of a job that has been reset (or never opened in the first place) it would be nice to allow all fields within the job config to be updated.

The remaining complication is the interaction between the job config and its associated datafeed config. Some datafeed config fields are validated against values in the associated job config, for example datafeed frequency and job bucket span. It would be easier to resolve this difficulty if the job and datafeed configs were a single object, and while this is the long-term goal, it will take a while to get there. In the short term we will need to disallow job config updates that would cause the validation of the associated datafeed to fail.

@droberts195 droberts195 added >enhancement :ml Machine learning labels Aug 31, 2021
@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Aug 31, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@EmekaMomodu
Copy link

Hello @droberts195, is this still open ? I would like to make my first contribution through this.

@EmekaMomodu
Copy link

Currently working on this.

@sharubitsi
Copy link

Hello i am willing to start my open source software development, is it possible to me work on this issue?

@droberts195
Copy link
Contributor Author

@sharubitsi yes, if you'd like to tackle this then that would be great.

@droberts195 droberts195 removed the good first issue low hanging fruit label Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement :ml Machine learning Team:ML Meta label for the ML team
Projects
None yet
Development

No branches or pull requests

5 participants