-
Notifications
You must be signed in to change notification settings - Fork 3
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
Hi! I think there might be a detail missing in the code #6
Comments
Hi!Thank you for pointing this out! The sample_gt has already been scaled to its valid ratio in (see here). I will check the part of the code after the deadline of CVPR! Thank you very much! |
hello, I meet some errors when installing the 1D Grid Sampling and RoI Align operators, the error message is 'roi_align/src/roi_align_kernel.cu:12:10: fatal error: ATen/ceil_div.h: No such file or directory |
hi! could you please try to uncomment the /roi_align/src/roi_align_kernel.cu in the line 7-9 and all other commented lines like: dim3 grid(std::min(THCCeilDiv((long)output_size, 512L), 4096L)); That is because the pytorch1.9 replace the c++ operations from the THC library to the ATEN library. After replace all operations, please setup the extension again. I have not tried to build the code below the pytorch v1.9, so if you have any other questions, feel free to contact me. Thank you. |
In the React/thumos_dataset.py, and in the getitem () function, if it is training mode now, and when the self.provide_contrastive_data is True, we will get sample_gt as well as pos_sample_segment, neg_sample_segment and candidated_segments.
When we get pos_sample_segment, neg_sample_segment and candidated_segments, as the code shows:
pos_sample_segment = pos_sample_segment * (len(pos_feat) / self.clip_len)
neg_sample_segment = neg_sample_segment * (len(neg_feat) / self.clip_len)
candidated_segments = candidated_segments * (gt_len / self.clip_len)
Their start and end points are scaled to a value relative to 256.
However, the start and end of sample_gt are still relative to its own snippet_num.
Should a similar scaling operation be done for sample_gt here?
The text was updated successfully, but these errors were encountered: