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

About target encoder #89

Open
ZhaoYuTJPU opened this issue Apr 5, 2023 · 3 comments
Open

About target encoder #89

ZhaoYuTJPU opened this issue Apr 5, 2023 · 3 comments

Comments

@ZhaoYuTJPU
Copy link

First of all, thanks for your work.
I have a question about the parameter update of the Target encoder.
As shown in the figure, the function will be executed in each forward process, and then the parameters will be copied from the Online encoder instead of always using the EMA update. I don't understand it very well.
I look forward to your reply.
image
image

@lc17721825
Copy link

I have the same confusion about the projector in online_encoder.
The projector is created every time in the forward() of online_encoder.
Have you figured it out?

@ZhaoYuTJPU
Copy link
Author

Hello.
I think I've solved the problem.
The function _get_target_encoder has a decorator on it, which makes it run only once, not every time.
You can see this process with single-step debugging.
I hope this solves your problem.

@lc17721825
Copy link

Thanks for your reply.
The projector follows the same situation and uses a decorator.

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