Tweak another error message

Signed-off-by: Mike Knepper <mknepper@nvidia.com>

Author: mikeknep

packages/data_designer_nemo/src/data_designer_nemo/seed.py

@@ -22,8 +22,10 @@
     "referencing fileset paths). Upload your data to the Files service, adjust your config, and try again."
 )
 _DATAFRAME_SEED_TYPE = "df"
-_DATAFRAME_SEED_TYPE_MESSAGE = (
-    "Dataframe seed sources (seed_type=df) are not supported on the NeMo Platform. TODO: more detail here!"
+LOCAL_DATAFRAME_SEED_ERROR_MESSAGE = (
+    "Dataframe seed sources are not supported on the NeMo Platform. "
+    "Save your data to a file or directory and update your config before trying again. "
+    "If you intend to run this same workload remotely, upload the file or directory to the Files service."
 )
 
 
@@ -156,7 +158,7 @@ def _validate_seed_type_for_execution_context(seed_type: str, *, is_local: bool)
     """Raises if a seed source type is unsupported in this execution context."""
     if is_local:
         if seed_type == _DATAFRAME_SEED_TYPE:
-            raise NDDInvalidConfigError(_DATAFRAME_SEED_TYPE_MESSAGE)
+            raise NDDInvalidConfigError(LOCAL_DATAFRAME_SEED_ERROR_MESSAGE)
         return
 
     if seed_type not in _SUPPORTED_SEED_TYPES:

plugins/nemo-data-designer/tests/integration/test_preview_local_cli.py

@@ -8,6 +8,7 @@
 import pytest
 from data_designer.cli.utils.sample_records_pager import PAGER_FILENAME
 from data_designer.config.analysis.dataset_profiler import DatasetProfilerResults
+from data_designer_nemo.seed import LOCAL_DATAFRAME_SEED_ERROR_MESSAGE
 
 pytestmark = pytest.mark.integration
 
@@ -113,7 +114,7 @@ def load_config_builder() -> dd.DataDesignerConfigBuilder:
     # errors into ``ValueError`` so Pydantic wraps them properly; this test
     # asserts the resulting user-visible message.
     assert result.exit_code != 0
-    assert "Dataframe seed sources (seed_type=df) are not supported on the NeMo Platform" in result.output
+    assert LOCAL_DATAFRAME_SEED_ERROR_MESSAGE in result.output
     assert "Field required" not in result.output
     assert "No such file" not in result.output
 
@@ -152,8 +153,7 @@ def load_config_builder() -> dd.DataDesignerConfigBuilder:
     # ``CreateRenderer.on_error`` runs the message through Rich, which line-wraps
     # to the terminal width, so we assert on fragments rather than the full
     # ``"Dataframe seed sources (seed_type=df) are not supported..."`` substring.
-    assert "Dataframe seed sources" in result.output
-    assert "seed_type=df" in result.output
+    assert LOCAL_DATAFRAME_SEED_ERROR_MESSAGE in result.output.replace("\n", "")
     assert "Field required" not in result.output