Add spacecraft position and velocity to all ENA pointing sets

[greglucas]

We added s/c velocity to the Lo pointing set, but it is desired to also add position now too. Additionally, other ENA instruments (and SPDF) may find this information useful, so it might be worth adding to all pointing set products.

I think it would be updating this function to include adding position as well.

imap_processing/imap_processing/ena_maps/utils/corrections.py

Line 430 in da24adf

def add_spacecraft_velocity_to_pset(

Then if this is added to all pointing sets, it could likely be removed from the L2 code that is using it for CG corrections.

NOTE: There is a noticeable difference on CG results depending on whether the beginning, middle, end of a pointing is used for the calculations. This should be discussed with instrument teams to make sure the time used is appropriate and noted.

[maxinelasp]

GLOWS already includes this info.

[ahotasu]

One way to do this is to add position to this function. This function is already called by Hi and Lo and could also be called by Ultra.

[ahotasu]

Nthan is begging me to get this implemented for Lo. I can either copy the function I linked earlier into a Lo-specific routine, make updates, and open a ticket to (eventually) resolve this duplicity, or I can just update the function, which will impact Hi and Ultra (I believe) by adding s/c position to their PSETs.

@subagonsouth and @lacoak21, can you please let me know if I can just update the function as-is (renaming it to position and velocity, updating related test cases, etc) instead of creating more technical debt? Also, can you comment on the preference of HAE coordinates vs. some other coordinate set?