Skip to content
This repository was archived by the owner on Oct 16, 2023. It is now read-only.

Refactor Softmax #112

Merged
merged 1 commit into from
Sep 8, 2023
Merged

Refactor Softmax #112

merged 1 commit into from
Sep 8, 2023

Conversation

daemyung
Copy link

@daemyung daemyung commented Sep 8, 2023

🙏 Describe the pull request

Softmax를 리펙토링 했습니다.

✅ Checklist

  • Code follows the project's coding conventions and style.
  • Tests have been added or updated to cover the changes.
  • Documentation has been updated, if necessary.

@daemyung daemyung added the enhancement New feature or request label Sep 8, 2023
@daemyung daemyung self-assigned this Sep 8, 2023
@daemyung daemyung force-pushed the softmax branch 2 times, most recently from 3479cbb to 05b1632 Compare September 8, 2023 06:19
Copy link
Contributor

@mejai1206 mejai1206 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@daemyung daemyung force-pushed the softmax branch 4 times, most recently from e4dada2 to 45773b6 Compare September 8, 2023 07:50
Copy link
Contributor

@kakao-steve-ai kakao-steve-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리팩토링 & 디버깅 감사합니다

@daemyung daemyung merged commit db48e03 into main Sep 8, 2023
@daemyung daemyung deleted the softmax branch September 8, 2023 08:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants