codecov[bot]
commented
6 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 80.91%. Comparing base (
59cc60d) to head (90f02fb).:exclamation: Current head 90f02fb differs from pull request most recent head e8ece98. Consider uploading reports for the commit e8ece98 to get more accurate results
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #958 +/- ## ======================================= Coverage 80.90% 80.91% ======================================= Files 195 195 Lines 14889 14948 +59 ======================================= + Hits 12046 12095 +49 - Misses 2843 2853 +10 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Doesn't actually resolve #930 but does resolve bugs - previously representation was seemingly working by luck.
Currently any models within a collection that share priors are considered separately. We need to be careful about how we represent a collection of models in a way that does not obfuscate what is going on.
For example, if we used a concise representation of a collection of models which all shared a prior it would in the current representation look the same as a collection of models which all had an independent prior of with the same name and parameters.