aim icon indicating copy to clipboard operation
aim copied to clipboard

Add rm query option

Open fferflo opened this issue 3 years ago • 6 comments

Implemented the feature request from #188 since I was producing lots of superfluous dummy runs when testing my training code.

  • Add a --query option to aim experiment rm that removes all runs matching the QL expression
  • Add a --dry option to aim experiment rm to not delete anything and just print the deletion steps
  • Add run.duration and no_metrics to QL to easily find and delete dummy runs

Example usage:

aim experiment rm -n experiment_name1 -q "run.duration < 1800 and run.process.finish"
aim experiment rm -q "experiment == experiment_name1 and no_metrics" --dry

fferflo avatar Apr 17 '21 12:04 fferflo

Hi @fferflo,

Thanks for the contribution. Great enhancement! Will review the PR soon 😊

gorarakelyan avatar Apr 17 '21 18:04 gorarakelyan

@fferflo sorry for the late reply. The thing is, we are currently redesigning query language - mainly statements and we are going to add explicit delete statement for deleting the tracked entities(runs, metrics, steps etc). That will affect some cli changes as well. QL "expressions" will not be changed and I will make sure to come back and integrate this PR into the new interface right after the alpha version is ready. Thanks for your time and contribution!

gorarakelyan avatar May 06 '21 08:05 gorarakelyan

@gorarakelyan maybe it makes sense to leave this PR On hold before we release the updated version?

SGevorg avatar May 07 '21 08:05 SGevorg

@SGevorg yup, makes sense

gorarakelyan avatar May 07 '21 08:05 gorarakelyan

Sure, I'm fine with anything :)

fferflo avatar May 07 '21 10:05 fferflo

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jul 16 '22 19:07 CLAassistant

closing the PR as it is based on the legacy code. with the upcoming Aim v4 the mentioned functionality is available out-of-the-box!

gorarakelyan avatar Apr 28 '23 14:04 gorarakelyan