fix report command #3375
fix report command #3375
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| # Command to run - can be overridden for different scripts | ||
| command: ["uv", "run", "python", "-m", "softmax.dashboard.report", "report", "--push"] | ||
| command: ["uv", "run", "python", "-m", "softmax.dashboard.report", "--push"] |
There was a problem hiding this comment.
The CLI in softmax.dashboard.report registers report as the only subcommand (@app.command() in softmax/src/softmax/dashboard/report.py). By dropping the literal "report" here, the CronJob now runs uv run python -m softmax.dashboard.report --push, which Typer treats as a missing command and fails with No such option --push. This means the scheduled job will exit before collecting or pushing metrics, so the dashboard will never update.
Useful? React with 👍 / 👎.
Asana Task