Coordinates conversion helpers overloads

[hongyuchen1030]

Overview

Enable node_xyz_to_lonlat_rad, node_lonlat_rad_to_xyz, normalize_in_place take np.array() as input and output

Modify the current node_xyz_to_lonlat_rad, node_lonlat_rad_to_xyz, normalize_in_place helper functions
such that: 1. it takes np.array() or list as input. 2. The output is in the np.array() format

Note:

Disable the numba @njit for all helpers.py since numba doesn't support the isinstance function

[rajeeja]

Note:

Disable the numba @njit for all helpers.py since numba doesn't support the isinstance function

We can use type ducking type or Union type (python 3.10+) instead of isinstance they are supported by numba.

[hongyuchen1030]

Note:

Disable the numba @njit for all helpers.py since numba doesn't support the isinstance function

We can use type ducking type or Union type (python 3.10+) instead of isinstance they are supported by numba.

sounds good. I will leave my implemented algorithm in the multi-precision branch. And feel free to adapt to any better delivery codes

[philipc2]

@hongyuchen1030

I have changed the base of this branch to point towards multi_precision instead of main

[hongyuchen1030]

@hongyuchen1030

I have changed the base of this branch to point towards multi_precision instead of main

Thank you very much!

@aaronzedwick Can you help merge the redesign into this branch and merge it into the multi_precision branch please?

[aaronzedwick]

@hongyuchen1030 I tried merging everything, but I am a bit confused about how to do it since this is your own fork. Should I check out your fork, and merge the main branch from the fork into this branch from the fork? And then push into your branch? Or is there some other way?

[aaronzedwick]

I got everything merged into your branch, do you want me to push it directly?

[hongyuchen1030]

I got everything merged into your branch, do you want me to push it directly?

Yes, you can directly commit to this branch. Thank you for your help.

[hongyuchen1030]

@aaronzedwick This branch shouldn't be considered as multiprecision since it just overloads the normal coordinate conversion with the numpy.array() and it should be merged into main