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

[fix] Update svtrnet.py #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[fix] Update svtrnet.py #11

wants to merge 1 commit into from

Conversation

gaozhun
Copy link
Contributor

@gaozhun gaozhun commented Aug 6, 2024

Changed bias parameter from None to True in ConvBNLayer to ensure consistency between Paddle (where bias=None is equivalent to bias=True) and PyTorch (where bias=None is equivalent to bias=False). This avoids unexpected behavior when switching between frameworks.

Changed bias parameter from None to True in ConvBNLayer to ensure consistency between Paddle (where bias=None is equivalent to bias=True) and PyTorch (where bias=None is equivalent to bias=False). This avoids unexpected behavior when switching between frameworks.
@Topdu
Copy link
Owner

Topdu commented Aug 6, 2024

Could you please set bias to be controllable through the config file

@gaozhun
Copy link
Contributor Author

gaozhun commented Aug 6, 2024

Indeed, it is necessary to set the bias through the config file; otherwise, it will not be possible to load previously trained models.

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

Successfully merging this pull request may close these issues.

2 participants