ISISComputingGroup/genie

load_script type checking - allow `np` datatypes in `cset` / `PVValue`

Aperta

#59 aperta il 9 ott 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Python (0 fork)auto 404
good first issue

Metriche repository

Star
 (1 stella)
Metriche merge PR
 (Merge medio 7g 10h) (3 PR mergiate in 30 g)

Descrizione

As a user on MERLIN I would like the following snippet to pass type checking:

import numpy as np
from genie_python import genie as g

def runscript():
    for psi in np.arange(30,130,10):
        g.cset("Rot", psi, lowlimit=psi-0.1, highlimit=psi+0.1, runcontrol=True)

It currently fails with:

E: Argument of type "signedinteger[Any]" cannot be assigned to parameter "args" of type "PVValue"

PVValue should also include np types

Acceptance criteria

  • Above snippet passes type checking

Guida contributor

Direzione di ricerca
Esamina l'annotazione di tipo per `PVValue` nel codice genie e aggiornala per accettare i tipi numerici numpy (ad esempio `np.signedinteger`, `np.floating`) in modo che la funzione `cset` possa accettare tali valori.
Tech stack
pythonnumpy
Dominio
backend
Tipo issue
Bug
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonNumPyType hints
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75