Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FIX: remove simplification step Abs of mass #482

Merged
merged 6 commits into from
Mar 31, 2023
Merged

FIX: remove simplification step Abs of mass #482

merged 6 commits into from
Mar 31, 2023

Conversation

redeboer
Copy link
Member

Closes #479

@redeboer redeboer added 🐛 Bug Something isn't working 📝 Docs Improvements or additions to documentation labels Mar 28, 2023
@redeboer redeboer added this to the 0.4.11 milestone Mar 28, 2023
@redeboer redeboer self-assigned this Mar 28, 2023
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@redeboer redeboer changed the title FIX: set non-negative assumptions on mass symbols FIX: remove simplification step Abs of mass Mar 31, 2023
@redeboer redeboer marked this pull request as ready for review March 31, 2023 13:15
@redeboer redeboer enabled auto-merge (squash) March 31, 2023 13:15
@redeboer redeboer merged commit ccad808 into ComPWA:main Mar 31, 2023
@redeboer redeboer deleted the fix-mass-substitution branch March 31, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@redeboer redeboer

Labels
🐛 Bug Something isn't working 📝 Docs Improvements or additions to documentation
Projects
None yet
Milestone
0.4.11
Development

Successfully merging this pull request may close these issues.

Problem with mass substitution in amplitude analysis notebook
1 participant
@redeboer