Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a JSON output format #323

Closed
TomMD opened this issue Jun 25, 2020 · 2 comments · Fixed by #334
Closed

Add a JSON output format #323

TomMD opened this issue Jun 25, 2020 · 2 comments · Fixed by #334
Assignees
Labels
enhancement New feature or request

Comments

@TomMD
Copy link

TomMD commented Jun 25, 2020

Stan lacks a machine-parsable output format, such as JSON. We should add one!

@chshersh chshersh added the enhancement New feature or request label Jun 26, 2020
@chshersh
Copy link
Contributor

If the goal is to have a machine-parseable output format then TOML is a more favourable choice, since we already depend on the tomland library. Bringing new dependencies to the project will make the installation process slower for all Stan users, even if they don't need machine-parseable output.

@TomMD
Copy link
Author

TomMD commented Jun 27, 2020

OK, thanks for the direction. Making a full TOML is a bit more work both because tomland affords less automation and encourages a bidirectional codecs, while the envisioned JSON would be lossy.

I hacked up a JSON that will suffice until we get the toml and I also get a good toml/json translation external to stan. Will keep the toml task on my radar.

@vrom911 vrom911 self-assigned this Jul 4, 2020
vrom911 added a commit that referenced this issue Jul 4, 2020
chshersh pushed a commit that referenced this issue Jul 4, 2020
* [#323] Add a JSON output format

Resolves #323

* Update CHANGELOG

* Fix Stack build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants