POSYDON-code/POSYDON

`find_nearest` in `gridutils`

Aperta

#465 aperta il 5 dic 2024

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (35 fork)auto 404
enhancementgood first issue

Metriche repository

Star
 (42 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

find_nearest doesn't use: find_index_nearest_neighbour, which is available.

Additionally, it requires the input to be numpy.array's. Can we change the code function to np.argmin()? And np.abs?

Guida contributor

Direzione di ricerca
Trova la funzione `find nearest` nel codice, comprendi la sua attuale implementazione con loop manuali, e sostituisci con `np.argmin(np.abs(array value))` per migliorare le prestazioni e rimuovere il requisito di array numpy.
Tech stack
python
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75