Anderson Reyes

Results 12 comments of Anderson Reyes

Old issue but Ii've been trying some proof of concepts on how this would work exactly. Using the docker sdk I was able to get papermill to proxy the commands...

Also this could also very well be not needed anymore since it's easy to just create a docker file now a days with papermill installed and you just install papermill...

One solution for python might be to use autopep8 as a module. https://github.com/hhatto/autopep8/blob/master/README.rst#use-as-a-module Now this is very specific to python. Idk how it work for other kernels, subprocess call maybe?...

@timkpaine How you think this should be handled? via a different config list of repos like `c.JupyterTemplates.git_repos = []` or or use the same template_dirs list but deetect git repos...

@dbczumar is this still up for grabs? I’ll be down to start brainstorming. couple thoughts 1. If mlflow supports OR operator, it would also have to support parenthesis operator generally...

> Hi @AndersonReyes , absolutely! This is still up for grabs! Welcome back :) > > I agree that supporting parentheses is critical here, and I think recursive processing of...

got little hack working here https://github.com/AndersonReyes/mlflow/blob/FR-6075/mlflow/utils/search_utils.py#L1218 to output a list of comparators including for OR, And, and nested parenthesis. next step is to use this to build file store matcher/filterer...

@dbczumar Actually was able to just use the existing code patterns to support Or clause and parenthesis with recursion. https://github.com/mlflow/mlflow/compare/master...AndersonReyes:mlflow:FR-6075 lmk what you think. Added tests to make sure changes...

> Hi @AndersonReyes , your approach from [#6075 (comment)](https://github.com/mlflow/mlflow/issues/6075#issuecomment-1399551648) sounds great to me. Is this still something you're able to contribute? Hey sorry I've been unresponsive, (got a little weird...

@dbczumar https://github.com/mlflow/mlflow/pull/7972