Skip to content

[codex] fix bin entrypoint imports#80

Draft
vvv214 wants to merge 1 commit into
google:mainfrom
vvv214:codex/fix-bin-cli-entrypoints
Draft

[codex] fix bin entrypoint imports#80
vvv214 wants to merge 1 commit into
google:mainfrom
vvv214:codex/fix-bin-cli-entrypoints

Conversation

@vvv214

@vvv214 vvv214 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Fix public dpsynth/bin CLI entrypoints so they start from the open-source package.

Changes:

  • fix the indentation error and missing internal imports in run_data_generation.py
  • remove the missing fancyflags dependency from main.py and comparison.py
  • lazily import sdmetrics when comparison report generation runs
  • fix run_tabular_eval.py local CSV execution (csv flag value, enum values, dataclass output)
  • add bin CLI smoke coverage for all public entrypoints and a local CSV eval run

Validation:

  • PYTHONPATH=/tmp/dpsynth-test-stubs:. JAX_ENABLE_X64=True /tmp/dpsynth-py312/bin/python -m compileall -q dpsynth/bin tests/bin
  • PYTHONPATH=/tmp/dpsynth-test-stubs:. JAX_ENABLE_X64=True /tmp/dpsynth-py312/bin/python -m pytest tests/bin -q
  • git diff --check
  • git diff origin/main...HEAD --check

@vvv214 vvv214 force-pushed the codex/fix-bin-cli-entrypoints branch from 482d97f to a037d29 Compare July 3, 2026 07:53
@vvv214 vvv214 force-pushed the codex/fix-bin-cli-entrypoints branch from a037d29 to 51ec83b Compare July 3, 2026 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant