Need to vectorize simple functions such as `node_lonlat_rad_to_xyz`

UXARRAY / uxarray

Xarray-styled package for reading and directly operating on unstructured grid datasets following UGRID conventions

https://uxarray.readthedocs.io/

Apache License 2.0

142 stars 31 forks source link

Need to vectorize simple functions such as `node_lonlat_rad_to_xyz` #744

Closed erogluorhan closed 3 months ago

erogluorhan commented 3 months ago

For instance, grid.coordinates.node_lonlat_rad_to_xyz(node_coord) has an implementation that is straightforward to be vectorized. Because it is not vectorized, there are for loop or map-based calls to it, which is not ideal.

philipc2 commented 3 months ago

Related to #346

erogluorhan commented 3 months ago

Thanks for catching that! I missed it as I thought this was more a performance&scalability milestone as the calls to such functions are almost always for-loop-based. Anyways, let's close this then.