search

ReactiveBayes / GraphPPL.jl

DSL for probabilistic models specification and probabilistic programming.
MIT License
27 stars 4 forks source link

Migration guide #198

Closed wouterwln closed 3 months ago

wouterwln commented 3 months ago

Let me know if you miss anything

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.11%. Comparing base (aaa8261) to head (f2ded5a). Report is 5 commits behind head on dev-4.0.0.

:exclamation: Current head f2ded5a differs from pull request most recent head 3cab18c. Consider uploading reports for the commit 3cab18c to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev-4.0.0 #198 +/- ## ============================================= + Coverage 90.89% 92.11% +1.21% ============================================= Files 15 15 Lines 1802 1915 +113 ============================================= + Hits 1638 1764 +126 + Misses 164 151 -13 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

bvdmitri commented 3 months ago

Bert had a good point of also making this visible in README. There are people who are using the current version of GraphPPL.jl in their experiments. We must explicitly mention the major breaking change in README, point them to the Migration Guide, but also make a small paragraph on how to keep using the old version. To use the old version people must explicitly set [compat] bound in their Project.toml. Also we should mention that the old version will not see any updates or maintenance.