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.6k stars 649 forks source link

Print error when using links in the wrong function #1332

Open morzahavi opened 5 days ago

morzahavi commented 5 days ago

Proposal Summary

I would suggest adding an informative error when using links inside dataset.add_files() instead of using the appropriate dataset.add_external_files

Motivation

It took me some time to figure out whats wrong since I wasn't aware of the function for external files, and chat GPT was not clever enough to suggest it even though I used links in my question. The notification I got was something like "failed to add files..."

Related Discussion

If this continues a slack thread, please provide a link to the original slack thread.

ainoam commented 4 days ago

Thanks for suggesting @morzahavi - Would you care to put in a PR? any addition to streamline development is wholeheartedly welcome.

morzahavi commented 4 days ago

Thanks for suggesting @morzahavi - Would you care to put in a PR? any addition to streamline development is wholeheartedly welcome.

Added -> 50ad7b09a11ffeccf0729e939f1e1b8babcf8e18

LMK if not clear