Skip to content

feat(cli): add convert and inspect subcommands (re-land of #6) - #14

Merged
marksverdhei merged 1 commit into
mainfrom
reland/markutils-6-cli-convert-inspect
Jun 27, 2026
Merged

marksverdhei merged 1 commit into
mainfrom
reland/markutils-6-cli-convert-inspect

Conversation

@marksverdhei

Copy link
Copy Markdown
Owner

Draft re-land of #6 — hai-pilgrim's PR was 3 months stale and its commit had a Co-Authored-By: Claude trailer (can't ship per repo convention). Cherry-pick was clean — adds tests/test_cli.py (new file) and modifies cli.py/checkpoints.py. Also fixed 2 ruff F401s in the test file (json, sys imported but unused).

Summary

  • markutils convert <input> <output> — convert between .csv/.tsv/.json/.jsonl/.parquet using markutils.utils.data.load_dataframe/save_dataframe
  • markutils inspect <safetensors-file> — print tensor shapes from a .safetensors checkpoint using markutils.checkpoints.print_state_dict_shapes

Test plan

  • full suite → 40 passed (6 new CLI tests)
  • uvx ruff check clean
  • CI / your review

- convert: load/save DataFrames between csv/tsv/json/jsonl/parquet
  using markutils.utils.data.load_dataframe/save_dataframe
- inspect: print tensor shapes from a safetensors checkpoint
  using markutils.checkpoints.print_state_dict_shapes

Re-land of #6 (hai-pilgrim's PR had a Co-Authored-By: Claude trailer
that can't ship per repo convention).
@marksverdhei
marksverdhei marked this pull request as ready for review June 27, 2026 14:59
@marksverdhei
marksverdhei merged commit 35d554d into main Jun 27, 2026
2 checks passed
@marksverdhei
marksverdhei deleted the reland/markutils-6-cli-convert-inspect branch June 27, 2026 14:59
marksverdhei added a commit that referenced this pull request Jun 27, 2026
…16)

README was the stock python-template placeholder ('Hello from markutils')
and didn't mention either subcommand from #14 or the load/save helpers
that have been there longer.

Rewritten as: tool description first, then install, then develop.
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