kubeedge/ianvs

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

Ouverte

#14 ouverte le 9 août 2022

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (1 personne assignée)Python (168 forks)auto 404
good first issuekind/cleanup

Métriques du dépôt

Stars
 (178 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Recherchez dans le dépôt toutes les occurrences de `initial model url` et `base model url`. Déterminez quel nom est préféré par le projet, puis mettez à jour le code, les tests et la documentation en conséquence. Consultez la configuration `algorithm.yaml` et `test algorithms/basemodel.py` pour le contexte.
Stack technique
python
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPython
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80