payload: {"push_id":4039364046,"size":10,"distinct_size":10,"ref":"refs/heads/local-photon-heating","head":"348dd74dcc16e0b93dda5b9b8e4fcf930a1dd700","before":"dc3ab07cedf867dc2c870d2ddb547221c83d713f","commits":[{"sha":"2c965ef7f810c1945cbe379852922bf0efe9832a","author":{"name":"Andrew Johnson","email":"c95d7c637c6c5a7d8787b3211e06a06d960d0506@protonmail.com"},"message":"Use defaultdict(dict) when weighting fission yields\n\nThe Chain.form_matrix method looks for information and\nfission yields for all isotopes with a fission reaction.\nHowever, the AveragedFissionYieldHelper and FissionYieldCutoffHelper\nare not guaranteed to return yields to all isotopes with\nfission reactions. Instead, the union on two sets of yields\nare returned:\n1) yields from isotopes with a single set of fission yields\n2) weighted yields computed for isotopes with both multiple sets of\n   fission yields **and** non-zero densities.\n\nThis later item can cause some isotopes that the Chain anticipates\nhaving fission yields, since they have a fission reaction, to not\nexist in the yields library. An isotope with multiple sets of\nyields but that is not physically present in the problem will not\nhave reaction rates and also not have fission yields computed\nby these helpers. This will cause a KeyError during Chain.form_matrix\nwhen fission yields for these isotopes are requested.\n\nThis commit changes the return type from weighted_yields on\nConstantFissionYieldHelper, AveragedFissionYieldHelper, and\nFissionYieldCutoffHelper to be defaultdict(dict). This resolves\nthe above issue by returning an \"empty\" set of yields for isotopes\nthat potentially *should* have fission yields, but have zero density\nacross the problem, and therefore do not have tallied reaction\nrates nor fission yields.\n\nDocumentation for the changes to these classes have been updated","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/2c965ef7f810c1945cbe379852922bf0efe9832a"},{"sha":"1a4bbb90685cab9a674f4b460d05595f9949da2c","author":{"name":"Andrew Johnson","email":"c95d7c637c6c5a7d8787b3211e06a06d960d0506@protonmail.com"},"message":"Chain.get_thermal_fission_yields -> Chain.get_default_fission_yields","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/1a4bbb90685cab9a674f4b460d05595f9949da2c"},{"sha":"99dfa1f05304b4e41b6f58a5ec3ed489ad54c790","author":{"name":"Andrew Johnson","email":"c95d7c637c6c5a7d8787b3211e06a06d960d0506@protonmail.com"},"message":"Return defaultdict(dict) from Chain.get_default_fission_yields","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/99dfa1f05304b4e41b6f58a5ec3ed489ad54c790"},{"sha":"7b8709e1a1461463dd3743396f9e1be8c2eee85c","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Merge pull request #1343 from drewejohnson/default-dict-weighted-yields\n\nUse defaultdict when weighting fission yields","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/7b8709e1a1461463dd3743396f9e1be8c2eee85c"},{"sha":"bbb0b7838331341dd38841ba31d6aa5204fbedf0","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Rename openmc.capi -> openmc.lib","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/bbb0b7838331341dd38841ba31d6aa5204fbedf0"},{"sha":"67de50e4fc6e45279aa79a41ef328a7f7fa0e61f","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Rename two tests","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/67de50e4fc6e45279aa79a41ef328a7f7fa0e61f"},{"sha":"8a580284692a30a13060b5d92b3bde558e5ec4ab","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Mention openmc.capi -> openmc.lib name change in release notes","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/8a580284692a30a13060b5d92b3bde558e5ec4ab"},{"sha":"c76b75ce33a5d23dbdb2d58a45a99902590e32ae","author":{"name":"Sterling Harper","email":"b82bab184d4643acfba0302dce01e215324f23b0@mit.edu"},"message":"Merge pull request #1345 from paulromano/capi-to-lib\n\nRename openmc.capi to openmc.lib","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/c76b75ce33a5d23dbdb2d58a45a99902590e32ae"},{"sha":"85f80f6b331109a3ced98a56d42cd4a5e6fa9102","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Make sure Nuclide::reaction_index_ is big enough for MT=901","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/85f80f6b331109a3ced98a56d42cd4a5e6fa9102"},{"sha":"348dd74dcc16e0b93dda5b9b8e4fcf930a1dd700","author":{"name":"Paul Romano","email":"a436801bae4f27296736e9545e95ef593e23fd3e@gmail.com"},"message":"Merge branch 'develop' into local-photon-heating","distinct":true,"url":"https://api.github.com/repos/paulromano/openmc/commits/348dd74dcc16e0b93dda5b9b8e4fcf930a1dd700"}]}
other: {"actor":{"display_login":"paulromano"}}