Skip to content

Commit

Permalink
Restrict check for 1V in Krook collisions to moment-kinetic cases
Browse files Browse the repository at this point in the history
2V Krook collisions are now implemented for full-f simulations.
  • Loading branch information
johnomotani authored Sep 25, 2023
1 parent 9b21e09 commit cf8a956
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/krook_collisions.jl
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ Currently Krook collisions
function krook_collisions!(pdf_out, fvec_in, moments, composition, collisions, vperp, vpa, dt)
begin_s_r_z_region()

if vperp.n > 1
error("Krook collisions not implemented for 2V case yet")
if vperp.n > 1 && (moments.evolve_density || moments.evolve_upar || moments.evolve_ppar)
error("Krook collisions not implemented for 2V moment-kinetic cases yet")
end

# Note: do not need 1/sqrt(pi) for the 'Maxwellian' term because the pdf is already
Expand Down

0 comments on commit cf8a956

Please sign in to comment.