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

Questions about TDC #16

Open
nizhezhiwei opened this issue Apr 6, 2023 · 2 comments
Open

Questions about TDC #16

nizhezhiwei opened this issue Apr 6, 2023 · 2 comments

Comments

@nizhezhiwei
Copy link

I'm having trouble with the TDC module,the paper explains that it uses the TDC module proposed by AUTOHR.However, I don't think CDC_T code implements the formula for TDC in AUTOHR.Why use out_normal-06*out_diff, and out_diff is to first calculate the combination of the weights of the t0 fragment and the t2 fragment in the convolution kernel, and then use the 3D convolution to volume, I am very confused, can you explain,thank you

@KegangWangCCNU
Copy link

KegangWangCCNU commented Jul 3, 2023

self.conv.weight[:, :, 0, :, :].sum(2).sum(2) + self.conv.weight[:, :, 2, :, :].sum(2).sum(2) is a processing of the convolution kernel. Since the size of the convolution kernel is 3x3x3 and the channel is OIDHW, where the third channel is Depth which means time. Two sum (2) means accumulating on H and W channels, which equals to taking average in a 3x3 space. Then add up averages from first frame and third frame for each step of convolution operation, then calculate difference with second frame.

According to the original text, the output at the current spatio-temporal position in the feature map (that is, the central position of the convolution kernel) should subtract from adjacent outputs (i.e., previous and next frames). I think this writing should be consistent with the original text.

I reproduced this model and made a fair comparison with other models on public datasets. If interested, check out https://github.com/KegangWangCCNU/PhysBench

@nizhezhiwei
Copy link
Author

Thank you very much for your reply.

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

2 participants