POSYDON-code/POSYDON

`find_nearest` in `gridutils`

Offen

#465 geöffnet am 05.12.2024

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (35 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (42 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Finde die `find nearest` Funktion im Code, verstehe ihre aktuelle Implementierung mit manuellen Schleifen und ersetze sie durch `np.argmin(np.abs(array value))`, um die Leistung zu verbessern und die Anforderung von numpy Arrays zu entfernen.
Tech Stack
python
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75