-
Notifications
You must be signed in to change notification settings - Fork 69
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.
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
Acceleration #327
Acceleration #327
Conversation
Thanks for your comments, I updated the spelling + coding style accordingly. |
Hi Tobi, it is OK from my side. However, I checked it on a technical level only, not equations and physics, yet I assume you test this carefully. |
I am still testing this. I will merge the PR once I'm done (probably early next week). |
@TobiasWinchen I have finished reviewing the PR and testing the module. I have no more comments. |
Ready to be merged. |
…eed up development
2b591d2
to
5ffdcd0
Compare
Implementation of first and second order Fermi acceleration.
The code is documented in the new notebooks in the doc directory:
https://github.com/TobiasWinchen/CRPropa3/blob/acceleration/doc/pages/example_notebooks/acceleration
While integrating the examples I slightly restructured and streamlined the existing documentation.