search

flatironinstitute / stan-playground

Run Stan models in the browser
Apache License 2.0
36 stars 1 forks source link

Allow partial parse for SamplingOpts #125

Closed jsoules closed 4 months ago

jsoules commented 4 months ago

As mentioned in comments for PR #124

Currently we insist on a full parse for a SamplingOpts object to be considered successfully deserialized. Going forward it will likely be more resilient and flexible to populate any missing values with defaults instead.

WardBrian commented 4 months ago

In particular, this issue should be considered a blocker for adding any new entries to SamplingOptions. Before that point, it's just a "nice to have"