search

NNPDF / pineappl

PineAPPL is not an extension of APPLgrid
https://nnpdf.github.io/pineappl/
GNU General Public License v3.0
12 stars 3 forks source link

Increase code coverage of the CAPI #243

Closed cschwan closed 8 months ago

cschwan commented 10 months ago

Since #242 was merged we have code coverage of the CAPI, which currently stands at roughly 35%. We should write more C programs using the remaining functions, which could help with #141, too.

cschwan commented 8 months ago

Commits 965c94a2f01b2a39b1b159c5d295a74b0c2b03ce, ed3fc6ad5c20bd4987e8e17ea1a84f9d655f7707, a917fbce958d0caff7b9f4d88ffb9cb13fe810ae after dd327a40fab6fed00d25c2da3594b366f36e65a1, a6aa5314fc618f5a364fbe8a91703531973032b9, 3511ac85b57074acf1da079f19dd3c8e67c0fba7 and 23f130ddbe51a3112265f683e09d82b49c6f27b0 improve code coverage slightly.

cschwan commented 8 months ago

Commits da6af32ca835d205349d757624f17e7a0b42c71e, 188a10da481864a18174b0b74028c498daa6a135, 98b6426bc6df4b98b0acb7ecdc3c5096342a4989, 752a0b2b429217df161abfe7350b7fff7bb4c023, 7197edae342170956196c317e94ac2eb9b34fd70 and 2507af8e04ed891b21df1a47cdb3eeb59eff2dad add more examples, and commits 8d12ba460017237f70842e0eee0bf2710204e741 and 150a4f7cb8361ab66114c4f8d7c81e1b9902cdcf soft-remove functions that were only used in the exporters/importers that now have long been migrated to the CLI written in Rust.

cschwan commented 7 months ago

Commit 3b5147fb7ea6b4bc4c64666196960c179561b016 tests a few missing lines and commit 9bbe8d69e8a658c9508b05b4a0f42ff46a5fce22 removes the obsolete functions completely, as allowed by semver.