unitxt icon indicating copy to clipboard operation
unitxt copied to clipboard

possible full import needed in unitxt.dataset

Open sam-data-guy-iam opened this issue 1 year ago • 1 comments

@matanor unitxt/src/unitxt/dataset.py currently has from .from .dataset_utils import get_dataset_artifact I just newly cloned both fm-eval and unitxt, and rebuilt the envs. For me, when I try running the basic run_text2text.py task from fm-eval, I get an error that dataset doesn't have the parse method. Should this statement be instead from .dataset_utils import *? This fixes it for me.

sam-data-guy-iam avatar Jan 23 '24 12:01 sam-data-guy-iam

I think its ok, you can see in other files that this is how imports go within src/unitxt.

matanor avatar Feb 22 '24 16:02 matanor