facebookresearch/fmmax

Better type checking in `eigensolve_isotropic_media`

Aperta

#53 aperta il 31 ott 2023

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Jupyter Notebook (15 fork)auto 404
enhancementgood first issue

Metriche repository

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

Descrizione

Currently, if the wavelength parameter within eigensolve_isotropic_media is passed as a float, rather than a jax array, then it throws an obscure error (not directly pointing to the wavelength parameter itself).

It would be good to either

  • explicitly allow for float values and do the necessary conversions under the hood.
  • assert a type up front and throw an error describing why this needs to be the case.

Guida contributor

Direzione di ricerca
Ispeziona la funzione `eigensolve isotropic media` per individuare dove viene utilizzato `wavelength`. Aggiungi un controllo di tipo per asserire che `wavelength` sia un array jax, oppure converti automaticamente un float in un array jax per evitare errori oscuri.
Tech stack
python
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.
Meno di 1 ora
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
Python
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80