kubeedge/ianvs

Inconsistent with interface name 'initial_model_url' and environment variable 'base_model_url'

Offen

#14 geöffnet am 09.08.2022

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (168 Forks)auto 404
good first issuekind/cleanup

Repository-Metriken

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

Beschreibung

What would you like to be added/modified: While configuring algorithm.yaml, the pre-trained model url is specified as initial_model_url, however in test_algorithms/basemodel.py it is parsed and renamed as base_model_url.

Why is this needed: This inconsistency might be confusing for algorithm developers. May I possibly recommend revising these two names to the same one rather than keeping this inconsistency.

Contributor Guide

Research-Richtung
Durchsuche das Repository nach allen Vorkommen von `initial model url` und `base model url`. Entscheide, welcher Name vom Projekt bevorzugt wird, und aktualisiere dann Code, Tests und Dokumentation entsprechend. Überprüfe die `algorithm.yaml` Konfiguration und `test algorithms/basemodel.py` für den Kontext.
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
GitPython
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80