Code Review: pn_evolutions_opt.py
Minor comments:
-
Fix Me!
on this line. Add check on initial omega0? Backwards integration is enabled though? -
The documentation states that backend (str, optional): The backend to use for ODE integration. Defaults to "solve_ivp".
but the function specifiesbackend: str="DOP853"
and the default method forsolve_ivp
isRK45
. So perhaps want to clarify that this is the method passed tosolve_ivp
and notsolve_ivp
itself? Or did I misunderstand this?
Edited by Geraint Pratten