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

[Feat] 간단한 테스트용 DNN Model 구현 #28

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

[Feat] 간단한 테스트용 DNN Model 구현 #28

wants to merge 17 commits into from

Conversation

22ema
Copy link
Collaborator

@22ema 22ema commented Jan 8, 2025 •
edited
Loading

What is this PR 🔍

  • 간단한 DNN model 구현 및 DNN model 실행 가능하도록 BaseModel 수정 진행하였습니다.

Changes 🔑

  • fix basemodel.py
  • create dnn_model.py
  • create train_sample.py

To Reviewer 🙇‍♂️

  • 시간관계상 basemodel.py 의 수정사항도 함께 포함시켰습니다. dnn_model 학습 실행 파일도 함께 업로드 하겠습니다.🙇‍♂️

22ema added 15 commits January 3, 2025 15:14
@22ema
[feat] DNNModel() 클래스 생성자 구현
13f4977
@22ema
[feat] DNNModel 클래스 구현시, 필요한 라이브러리 import
6e5929e
@22ema
[feat] DNNModel 클래스 주석 및 타입힌트 추가
11b24d5
@22ema
[feat] l2 정규화 코드 추가
3ec0e4c
@22ema
[feat] 코드 컨벤션 수정
46a0f88
@22ema
[feat] sparse feature와 dense feature 분리 코드와 embedding 테이블 생성 코드 추가 작성
7034839
@22ema
[feat] 코드 컨벤션 수정
6cd4b16
@22ema
[feat] DNNModel.forward() 메서드 구현
84b8b9f
@22ema
Merge branch 'main' into feat_dnn_model
d567822
@22ema
[fix] BaseModel와 중복되는 코드 제거
e74b8b0
@22ema
[fix] BaseModel 내 불필요한 linear_model 제거
f992eec
@22ema
[fix] BaseModel 내 미구현된 validation 관련 코드 삭제
f7622cf
@22ema
[fix] 사용하지 않는 VarLenSparseFeat 삭제
7e28cae
@22ema
[fix] BaseModel.add_regularization_weight() 함수 수정
a597ec6
@22ema
[fix] DNNModel 테스트를 위한 샘플 데이터 추가
ed2c701
@22ema 22ema requested a review from f-lab-owen January 8, 2025 08:30
@22ema 22ema self-assigned this Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@f-lab-owen f-lab-owen Awaiting requested review from f-lab-owen

At least 1 approving review is required to merge this pull request.

Assignees

@22ema 22ema

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@22ema