Hi, @daichi1207, thank you for releasing ReMoRa.
I evaluated the released checkpoint from the exact upstream commit
3a456f7
and found a possible issue in the evaluation data flow.
Possible bug
The GOP evaluation code computes MV magnitude maps and attempts to store them in
pending_mv_maps:
However, I could not find an initialization or consumer for pending_mv_maps,
so the guarded assignment appears to be a no-op.
Instead, the RMR path reads:
pending_mv_seqs
pending_i_indices
pending_gop_boundaries
Here:
These values are then passed to the two-stage compressor:
I do not see these raw-MV fields being populated by the released evaluation
script. Consequently, motion_vectors, gop_boundaries, and
i_frame_indices appear to reach the compressor as None.
I added an opt-in fix that stages these fields without changing the existing
magnitude/GOP logic. I also added two causal ablations:
- Zero MV: RMR receives zero input, producing
RMR(0) + 0.
- Zero RMR: the complete RMR output is zeroed before GOP fusion.
Patch and full results:
Results
All arms used the released checkpoint, complete five-dataset evaluation
(33,915 questions), GOP loading, and the repository's prompts, decoding, and
scoring.
| Dataset |
Paper |
No fix |
Real MV + fix |
Zero MV |
Zero RMR |
| LongVideoBench |
60.8 |
58.19 |
58.04 |
57.89 |
58.04 |
| MLVU |
72.1 |
69.78 |
69.60 |
69.50 |
69.60 |
| NExT-QA |
84.2 |
82.38 |
82.39 |
82.37 |
82.43 |
| PerceptionTest |
67.7 |
66.91 |
67.16 |
67.24 |
67.32 |
| Video-MME |
64.4 |
62.96 |
62.67 |
62.70 |
62.52 |
| Macro |
69.84 |
68.04 |
67.97 |
67.94 |
67.98 |
Against the fixed real-MV arm:
| Ablation |
Macro change |
95% video-clustered CI |
McNemar p |
| Zero MV |
−0.031 pp |
[−0.182, +0.120] |
.651 |
| Zero RMR |
+0.009 pp |
[−0.151, +0.170] |
.127 |
Removing the MV input or the complete RMR output therefore causes no measurable
accuracy decrease. The interventions change only about 1.1–1.2% of predictions.
Could you clarify:
- How were the raw-MV fields populated during the paper evaluation?
- Was another evaluation script or configuration used for Table 1?
- Did you perform a similar causal MV/RMR ablation? Do these results suggest
that the released model has learned to mostly ignore the MV signal, or is
there another intended MV path missing from the released evaluation code?
- Are additional checkpoint files or preprocessing steps required to reproduce
the reported scores?
Thank you!
Hi, @daichi1207, thank you for releasing ReMoRa.
I evaluated the released checkpoint from the exact upstream commit
3a456f7and found a possible issue in the evaluation data flow.
Possible bug
The GOP evaluation code computes MV magnitude maps and attempts to store them in
pending_mv_maps:infer.py, lines 356–373However, I could not find an initialization or consumer for
pending_mv_maps,so the guarded assignment appears to be a no-op.
Instead, the RMR path reads:
pending_mv_seqspending_i_indicespending_gop_boundariesHere:
llava_arch.py, lines 445–460These values are then passed to the two-stage compressor:
llava_arch.py, lines 313–323I do not see these raw-MV fields being populated by the released evaluation
script. Consequently,
motion_vectors,gop_boundaries, andi_frame_indicesappear to reach the compressor asNone.I added an opt-in fix that stages these fields without changing the existing
magnitude/GOP logic. I also added two causal ablations:
RMR(0) + 0.Patch and full results:
Results
All arms used the released checkpoint, complete five-dataset evaluation
(33,915 questions), GOP loading, and the repository's prompts, decoding, and
scoring.
Against the fixed real-MV arm:
Removing the MV input or the complete RMR output therefore causes no measurable
accuracy decrease. The interventions change only about 1.1–1.2% of predictions.
Could you clarify:
that the released model has learned to mostly ignore the MV signal, or is
there another intended MV path missing from the released evaluation code?
the reported scores?
Thank you!