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.

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

Add proton synchrotron computation #134

Open
zblz opened this issue Jan 26, 2017 · 6 comments
Open

Add proton synchrotron computation #134

zblz opened this issue Jan 26, 2017 · 6 comments

Comments

@zblz
Copy link
Owner

zblz commented Jan 26, 2017

This is a relatively simple modification that would require an additional argument to the Synchrotron class and setting the particle mass to either m_e or m_p in the gyroradius calculation.

@XiaonaS
Copy link

XiaonaS commented Jan 27, 2017

Thank you very much for your reply, Zabalza. I modified it, and now I can use the proton synchrotron.
Now I can enter the code "radiative.py" from my local path, but I don't know how to put it in the installed Naima path, and enter it from naima. Sorry for your time because of my limit knowledge of python.

@wrijupan
Copy link

Hello,

Thanks for the suggestion. I was looking into the Synchrotron class to modify it for proton synchrotron. Apart from modifying the gyroradius calculation, the Synchrotron class also uses _elec and _gam functions from the base class BaseElectron for _spectrum calculation. In this base class also the electron mass appears. So we have to modify the base electron class as well right ? Or just define mec2 = (m_p * c**2).cgs globally in the proton synchrotron code? Please forgive me if I am wrong.

Regards

@wrijupan
Copy link

@XiaonaS did proton synchrotron work for you by only modifying the Synchrotron class or did you touch the BaseElectron class as well? Your suggestions are much appreciated.

Thank you in advance.

@XiaonaS
Copy link

XiaonaS commented May 23, 2017

Yes, I have changed all the m_e to m_p in "radiative.py " when I calculate the proton synchrotron.

Best

@wrijupan
Copy link

Thank you!

Regards

@zblz zblz mentioned this issue Jul 16, 2017
2 tasks
@Uxuee
Copy link

Uxuee commented Aug 15, 2020

@XiaonaS
Hello,
Apart from the change of the mass, what is the "additional argument to the Synchrotron class" that @zblz mentions in the first message?
Kindly Regards

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

No branches or pull requests

4 participants