jiangyi15 / tf-pwa

Partial Wave Analysis using TensorFlow.
MIT License
27 stars 13 forks source link

Fixed zero phsp #123

Closed jiangyi15 closed 10 months ago

jiangyi15 commented 10 months ago

fixed nan when m=0 in

generate_phsp(5.0, [1.0, 1.0, 0.0])

May cause some small effect on toy generation for specific seed.

codecov[bot] commented 10 months ago

Codecov Report

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

Comparison is base (4c7bb9d) 76.99% compared to head (06b8e49) 77.01%. Report is 1 commits behind head on dev.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #123 +/- ## ========================================== + Coverage 76.99% 77.01% +0.01% ========================================== Files 111 111 Lines 16452 16456 +4 Branches 3100 3101 +1 ========================================== + Hits 12668 12674 +6 Misses 3074 3074 + Partials 710 708 -2 ``` | [Flag](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29) | `77.01% <100.00%> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29) | Coverage Δ | | |---|---|---| | [tf\_pwa/config\_loader/config\_loader.py](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29#diff-dGZfcHdhL2NvbmZpZ19sb2FkZXIvY29uZmlnX2xvYWRlci5weQ==) | `70.14% <100.00%> (+0.14%)` | :arrow_up: | | [tf\_pwa/fit.py](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29#diff-dGZfcHdhL2ZpdC5weQ==) | `23.24% <100.00%> (+0.24%)` | :arrow_up: | | [tf\_pwa/phasespace.py](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29#diff-dGZfcHdhL3BoYXNlc3BhY2UucHk=) | `96.28% <100.00%> (+0.03%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/jiangyi15/tf-pwa/pull/123/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Jiang+Yi+%28%E8%92%8B%E8%89%BA%29)

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