Skip to content

Fixes needed for nucleon-nucleus corpora curation - #14

Merged
beykyle merged 1 commit into
mainfrom
nn-corpora
Aug 27, 2026
Merged

Fixes needed for nucleon-nucleus corpora curation#14
beykyle merged 1 commit into
mainfrom
nn-corpora

Conversation

@beykyle

@beykyle beykyle commented Aug 26, 2026

Copy link
Copy Markdown
Owner
  • reaction.py: drop stray literal "f" from reaction_string/str for product-type reactions (produced e.g. "Ca-48(P,N)fSc-48")
  • distribution.py: add "average" statistical_err_treatment, for EXFOR asymmetric error pairs where both columns store positive magnitudes; the existing "difference" treatment yields negative errors there
  • distribution.py: fix duplicated "+DATA-ERR" test that meant (+DATA-ERR, -DATA-ERR) pairs were never detected
  • distribution.py: implement EnergyDistribution.plot, which previously inherited Distribution.plot and raised NotImplementedError
  • requirements.txt: add pandas, imported by distribution.py

- reaction.py: drop stray literal "f" from reaction_string/__str__ for
  product-type reactions (produced e.g. "Ca-48(P,N)fSc-48")
- distribution.py: add "average" statistical_err_treatment, for EXFOR
  asymmetric error pairs where both columns store positive magnitudes;
  the existing "difference" treatment yields negative errors there
- distribution.py: fix duplicated "+DATA-ERR" test that meant
  (+DATA-ERR, -DATA-ERR) pairs were never detected
- distribution.py: implement EnergyDistribution.plot, which previously
  inherited Distribution.plot and raised NotImplementedError
- requirements.txt: add pandas, imported by distribution.py

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LSfNX9V6VEYiSsSmaJPyq5
@beykyle
beykyle merged commit 1ff4fe5 into main Aug 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant