Do we really need both `J3_HN_HA_from_trajectory` and `J3_HN_HA_from_phi` to be public APIs? How about only having this one and making the other private, or just merging them together. Then, this could be renamed `compute_J3_HN_HA` for consistency.