search

allegroai / clearml

ClearML - Auto-Magical CI/CD to streamline your AI workload. Experiment Management, Data Management, Pipeline, Orchestration, Scheduling & Serving in one MLOps/LLMOps solution
https://clear.ml/docs
Apache License 2.0
5.69k stars 655 forks source link

Correct spelling, code formatting, add deprecation warnings to logger methods #1052

Closed Make42 closed 1 year ago

Make42 commented 1 year ago

This patch corrects spelling, a little code formatting, but mainly adds deprecation warnings to logger methods.

Make42 commented 1 year ago

Since I am on it, @pollfly and @jkhenning, I corrected the query_tasks docstring. The description on how "__$not" is to be used was erroneous - I tested it. But, I am not sure if it is right now.

Particularly, I suspect that "__$all" is actually deprecated, considering https://clear.ml/docs/latest/docs/clearml_sdk/task_sdk/#tag-filters. I was not able to find anything in the code and suspect that this is implemented in the back-end. If "__$all" is deprecated, I would like to remove this part in the docstring. Please tell me: Is "__$all" deprecated?

Also, I am only "90%" - so to speak - confident about my description in general. Please check!