DiamondLightSource/dodal

Check and fix device arguments

Open

#1 024 ouverte le 27 janv. 2025

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Python (12 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (4 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

From #1021

the default behaviour of @device_factory is that the name of the device is the name of the factory

Turns out that eg the zebra device has name as a required argument, coming before prefix. We should go through all devices and check that prefix is always before name, and that name is optional.

Acceptance Criteria

  • Specific criteria that will be used to judge if the issue is fixed

Guide contributeur

Direction de recherche
Examinez chaque fabrique de périphériques pour vous assurer que l'argument 'prefix' précède 'name' et que 'name' est facultatif. Commencez par le périphérique zebra comme exemple.
Stack technique
python
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60