Question: Can modality row selection be moved before the output projections?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 42/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Attiva
- Stack tecnologico
- python
- Ambito
- distributed-systems, machine-learning
Direzione di ricerca
Inizia da MiniMaxH3Transformer3DModel.forward e traccia il modo in cui norm_out, proj_out, audio_proj_out, video_indices e audio_indices interagiscono con la raccolta degli output context-parallel. Verifica se gli indici sono locali o globali e se l'inferenza distribuita o le predizioni delle righe di riferimento dipendono dall'ordine corrente; il lavoro è completato quando viene stabilita la validità dell'ottimizzazione e vengono documentati eventuali vincoli.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Question
In MiniMaxH3Transformer3DModel.forward, the output heads are currently applied to the entire packed sequence before selecting the video and audio rows:
hidden_states = self.norm_out(hidden_states, temb, timestep_indices)
video_output = self.proj_out(hidden_states).index_select(1, video_indices)
audio_output = self.audio_proj_out(hidden_states).index_select(1, audio_indices)
Since both output projections operate independently on the last feature dimension, would the following be mathematically equivalent?
hidden_states = self.norm_out(hidden_states, temb, timestep_indices)
video_hidden_states = hidden_states.index_select(1, video_indices)
audio_hidden_states = hidden_states.index_select(1, audio_indices)
video_output = self.proj_out(video_hidden_states)
audio_output = self.audio_proj_out(audio_hidden_states)
In other words:
index_select(Linear(H), I) = Linear(index_select(H, I))
This would avoid applying proj_out and audio_proj_out to rows that are not needed by the corresponding output head.
Is there any reason why the current order is required, such as:
- context-parallel output gathering;
- global versus local sequence indices;
- distributed inference constraints;
- numerical or checkpoint compatibility;
- or the need to preserve predictions for all reference rows?
I would appreciate any clarification about whether selecting the rows before the modality-specific output projections is a valid optimization.
- Lingua principale
- Python
- Stelle
- 9.8k
- Fork
- 738
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di MiniMax-AI/MiniMax-H3
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 5/100
MiniMax-AI/MiniMax-H3#89 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 5/100
MiniMax-AI/MiniMax-H3#88 ·
-
،herAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 10/100
MiniMax-AI/MiniMax-H3#87 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
MiniMax-AI/MiniMax-H3#86 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 74/100
MiniMax-AI/MiniMax-H3#85 ·
Tutte le issue di MiniMax-AI/MiniMax-H3
Issue simili
-
vis.image() reports "Unsupported number of image channels: 224" for an HxWxC imageForse già presa @piyush182004 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
fossasia/visdom#1997 · 3 commenti ·
I maintainer di solito rispondono entro 6 giorni
-
Byse new domainAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
Gujal00/ResolveURL#1525 ·
I maintainer di solito rispondono entro 1 giorno
-
Serialisation fails with pydantic 2.14: wrap serialisers pass SerializationInfo as index_keyForse già presa @hugohills-regnosys l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
finos/rune-python-runtime#45 ·
-
[Bug]: LangGraph Store sync put bypasses namespace validationForse già presa @ktz03 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
volcengine/OpenViking#6050 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
hummingbot/condor#283 ·
I maintainer di solito rispondono entro 1 giorno