DiamondLightSource/dodal

Remove conversion from string to DetectorSizeConstants in DetectorParams

Open

#717 geöffnet am 29. Juli 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Python (12 Forks)auto 404
buggood first issuelow priority

Repository-Metriken

Stars
 (4 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We used to need to be able to accept a string as DetectorSizeConstants as it was a parameter from GDA but this is now used as a constant in Hyperion so we can remove the need to derive it from a string.

Acceptance Criteria

  • Remove ability to specify DetectorSizeConstants as a string
  • DetectorParams still converts to json and back ok

Contributor Guide

Research-Richtung
Entfernen Sie die Konvertierung von String zu DetectorSizeConstants in DetectorParams und stellen Sie sicher, dass die JSON Serialisierung weiterhin funktioniert.
Tech Stack
python
Domain
backendapi
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