Lightning-AI/lightning-thunder

Make `numel` a method of TensorProxy not a property attribute

Open

#925 aperta il 5 ago 2024

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Python (114 fork)github user discovery
good first issue

Metriche repository

Star
 (1460 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

🐛 Bug

PyTorch uses tensor.numel() while Thunder's TensorProxy and in the codebase it's used as a property tensor.numel. The property attribute makes more sense, but it conflicts with setting PyTorch as the default language context (https://github.com/Lightning-AI/lightning-thunder/pull/924).

Related PR: https://github.com/Lightning-AI/lightning-thunder/pull/278

We should decide what to do with numel and how to avoid the discrepancy with PyTorch.

Guida contributor

Direzione di ricerca
Investigare l'uso di `numel` nel codice di lightning thunder, in particolare in TensorProxy e sue sottoclassi. Determinare se la modifica da proprietà a metodo è fattibile e coerente con l'API di PyTorch. Esaminare le PR correlate #278 e #924 per il contesto.
Tech stack
pythonpytorch
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonPyTorch
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75