Quick patch to check that Dataset Keys contain non-None Values

mosaicml / llm-foundry

LLM training code for Databricks foundation models

https://www.databricks.com/blog/introducing-dbrx-new-state-art-open-llm

Apache License 2.0

3.99k stars 525 forks source link

Quick patch to check that Dataset Keys contain non-None Values #1228

Closed KuuCi closed 4 months ago

KuuCi commented 4 months ago

A bug arose where user set remote to be None, which led the parse_uri to attempt to parse None, which throws an error. This change checks for values so that None vales will never hit parse_uri

KuuCi commented 4 months ago

working version mcli logs test-KvZifh