Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

why set image_width = 1280 and image_height = 720? #7

Open
nianfd opened this issue Feb 16, 2025 · 3 comments
Open

why set image_width = 1280 and image_height = 720? #7

nianfd opened this issue Feb 16, 2025 · 3 comments

Comments

@nianfd
Copy link

nianfd commented Feb 16, 2025

Great jobs!!
The image in dataset is 19201080, why set image_width = 1280 and image_height = 720 in this code?
I tried it and the effect was very poor when using other resolutions (e.g., when I set to 224
224, the accuracy is 0). Can you tell me which parameters need to be modified if using other resolutions?

@dk-kim
Copy link
Owner

dk-kim commented Feb 17, 2025

Thank you for your interest.
Our evaluation code assumes that the input resolution is 1280x720 (you can find in gt_tracks_24.txt file), so I think all of your box localization fails if you use different resolution.
Moreover, even if that's not the case, if the input resolution is too small, the feature map resolution will also be too low, making it difficult to extract proper human features. So, in my opinion, achieving good performance would be quite challenging.

@remember9069
Copy link

remember9069 commented Feb 22, 2025

Thank you for your interest. Our evaluation code assumes that the input resolution is 1280x720 (you can find in gt_tracks_24.txt file), so I think all of your box localization fails if you use different resolution. Moreover, even if that's not the case, if the input resolution is too small, the feature map resolution will also be too low, making it difficult to extract proper human features. So, in my opinion, achieving good performance would be quite challenging.

Hello, and thank you for your outstanding work. I have a question related to this issue, specifically concerning this part

self.roi_align = RoIAlign(crop_height=self.crop_size, crop_width=self.crop_size)

I notice that spatial_scale and sampling_ratio are positional arguments.

Could you please clarify how the original import statement (from roi_align.roi_align import RoIAlign) handles these parameters without explicitly specifying them?

Thank you for your assistance.

@dk-kim
Copy link
Owner

dk-kim commented Feb 24, 2025

Hi,
The RoIAlign code we used is from https://github.com/longcw/RoIAlign.pytorch, and as far as I know, it does not include spatial_scale and sampling_ratio as positional arguments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants