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.

Sign up for GitHub

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

Jump to bottom

App eps to KIN #11

Merged
merged 1 commit into from
Jul 7, 2022
Merged

App eps to KIN #11

merged 1 commit into from
Jul 7, 2022

Conversation

st9007a
Copy link
Collaborator

@st9007a st9007a commented Jul 7, 2022 •
edited
Loading

As the discussion of #8 , add eps as an init argument:

  1. To be compatible with the previous version, the default value of eps was set to zero.
  2. The implementation was aligned to the equation mentioned in the official instance normalization.
  3. All unit tests were passed.

@st9007a st9007a requested a review from Kaminyou July 7, 2022 06:55
@Kaminyou Kaminyou merged commit da235cf into main Jul 7, 2022
@st9007a st9007a deleted the Add-eps-to-KIN branch July 10, 2022 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kaminyou Kaminyou Awaiting requested review from Kaminyou

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@st9007a @Kaminyou