kubeedge/ianvs

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

Aperta

#14 aperta il 9 ago 2022

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (1 assegnatario)Python (168 fork)auto 404
good first issuekind/cleanup

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca tutte le occorrenze di `initial model url` e `base model url` nel repository. Determina quale nome è preferito dal progetto, quindi aggiorna codice, test e documentazione di conseguenza. Controlla la configurazione `algorithm.yaml` e `test algorithms/basemodel.py` per il contesto.
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
GitPython
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80