Fixed incorrect variable name in pixel_tokens_with_pos_embed reshaping operation wherenum_h_patches_per_window was used instead ofnum_w_patches_per_window for the width dimension.

Type of change

Bug fix (non-breaking change which fixes an issue)

Changes Made

Change:num_h_patches_per_window →num_w_patches_per_window in the fourth dimension of reshape operation
File:rfdetr/models/backbone/dinov2_with_windowed_attn.py
Line: ~315

Code Comparison

Before:

windowed_pixel_tokens=pixel_tokens_with_pos_embed.reshape(batch_size*num_windows,num_h_patches_per_window,num_windows,num_h_patches_per_window,-1)**After:**```pythonwindowed_pixel_tokens=pixel_tokens_with_pos_embed.reshape(batch_size*num_windows,num_h_patches_per_window,num_windows,num_w_patches_per_window,-1)## Impact-Improvescodereadabilityandconsistency-Nofunctionalchanges

fix: correct variable name in pixel_tokens_with_pos_embed reshape ope…

1d88124

…ration

lixiaolei1982 requested review fromMatvezy,SkalskiP,isaacrob-roboflow andprobicheaux ascode owners

November 11, 2025 13:00

Copy link

CLAassistant commentedNov 11, 2025

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign ourContributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let usrecheck it.}