allegroai/clearml-docs

task.upload_artifact() example does not work

Closed this issue · 3 comments

AjeelAhmed1998 commented

Link to the exact section: here

task.upload_artifact('/path/to/preprocess_data.csv', name='data')

The above code will not work because the first positional argument and the keyword argument name both are trying to fill in the name variable in the function. You can see the function in the codebase here.

AjeelAhmed1998 commented

I can not tell by the codebase which argument is supposed to be used as name. If someone can let me know I'll create a PR.

shomratalon commented

Hi @AjeelAhmed1998 ,

I think you are right, the usage should be as in https://github.com/allegroai/clearml/blob/master/examples/reporting/artifacts.py#L39 -

task.upload_artifact(name='Numpy Eye', artifact_object=np.eye(100, 100))
ainoam commented

Thanks for noticing @AjeelAhmed1998. Fixed in #351

  • 🎉1