Create a cross-section function for Coulomb collision under collisions.py. This should effectively be sigma under collision_frequency. Replace instance of sigma with this new function.
Note that the sigma ~ pi * b_perp ** 2 definition of the Coulomb collision cross-section is only valid for large angle collisions (e.g. 90 degree collisions based on b_perp).
sigma = pi * (2 * b_perp) ** 2 instead of sigma = pi * b_perp ** 2Note that the
sigma ~ pi * b_perp ** 2definition of the Coulomb collision cross-section is only valid for large angle collisions
Just wanted to clear some confusion. This definition matches the one in the docs but is there a reason for (2 * bPerp) in the source code?
@ritiek Note that sigma ~ pi * b_perp ** 2 is using ~ (similar) rather than = (equals). When we use ~ in physics the tendency is to keep variables but drop coefficients, and equations written in this form are there to give an idea of what are the important effects/parameters rather than to be precise.
sigma = pi * (2 * b_perp) ** 2 is correct, and it follows the general definition for a collisional cross-section between two particles. b_perp is essentially the radius of a particle, but one must account for the radii of both particles in the interaction, as illustrated here.
@ritiek we can of course replace the equation in the docs with the more precise version, to eliminate any ambiguity.
EDIT: the docs are wrong, because they use an equality. This needs to be changed. I've added this to the TO DO list.
If we have time, we should do this for 0.1 (as it is quite easy), if not it can easily be put off until later.
This is what I actually closed in #388.
Holy wow, we're at 389 already?!
Most helpful comment
This is what I actually closed in #388.
Holy wow, we're at 389 already?!