ComPWA / ComPWA-legacy

[deprecated] C++ back-end for the Common Partial Wave Analysis framework

https://compwa.github.io/legacy

Other

9 stars 9 forks source link

fix: serialize fit result correctly #300

Closed spflueger closed 4 years ago

spflueger commented 4 years ago

Fix bug not setting bounds of the FunctionTree::FitParameter correctly, causing wrong output when saving a fit result to file and errors when reading the fit results again.
Fix bug not serializing all members.
Also add test for fit result serialization.

codecov[bot] commented 4 years ago

Codecov Report

Merging #300 into master will decrease coverage by 0.47%. The diff coverage is 63.88%.

@@            Coverage Diff             @@
##           master     #300      +/-   ##
==========================================
- Coverage   37.05%   36.58%   -0.48%     
==========================================
  Files          88       88              
  Lines        7356     7380      +24     
==========================================
- Hits         2726     2700      -26     
- Misses       4630     4680      +50