stan-dev / stan-dev/cmdstanpy

[Stan 2.34] Utilize argument save_cmdstan_config

Open
#714 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
198
Forks
81
PR merge metrics
No merged PRs in 30d

Description

Summary:

Added in https://github.com/stan-dev/cmdstan/pull/1204, save_cmdstan_config outputs the arguments provided to cmdstan as a JSON file. This should remove most of the need to parse the CSV comments, besides perhaps for optional validation

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing where CmdStanPy parses CSV comments and review CmdStan PR #1204 to understand the save_cmdstan_config JSON output. Done means the JSON arguments are used instead of most CSV-comment parsing, with only any needed optional validation retained.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.