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

Update latest features for UI testing #130

Merged
merged 146 commits into from
Dec 10, 2023
Merged

Update latest features for UI testing #130

merged 146 commits into from
Dec 10, 2023

Conversation

thisisWooyeol
Copy link
Contributor

UI 테스팅을 위한 최신 커밋 업데이트입니다

  • Single navigation host를 반영해 navigation testing을 추가할 예정입니다

2018JunyoungLim and others added 30 commits December 1, 2023 19:34
@2018JunyoungLim
partial implementation of logout signout feature
ed2d871
@dawitfamanu
feat: reaction API connection
d5aabae
@thisisWooyeol
fix: fix typo & additional log message
f73338e
@2018JunyoungLim
feat: logout, signout
a34e34a
@thisisWooyeol
Merge branch 'develop' into feat/android-logout-signout
5012666
@yangchanhk98
Merge pull request #92 from snuhcs-course/feat/android-logout-signout
25fc0ca 
Feat/android logout signout
@peng-u-0807
fix: fix crashing for EmojiPlayView & TransformVideoView
5bad372
@thisisWooyeol @yangchanhk98
fix: permit Get posts, emojis without jwt token
89520a8
@2018JunyoungLim
feat: add sorting mode in emoji tab
ac8308e
@yangchanhk98
Merge pull request #93 from snuhcs-course/fix/android-emoji-crash
75f1ee9 
Fix crashing for `EmojiPlayView` & `TransformVideoView`
@yangchanhk98
Merge pull request #94 from snuhcs-course/feat/back-permit_authentica…
9ba63da 
…tion

fix: permit Get posts, emojis without jwt token
@2018JunyoungLim
fix: build error, button design
051c24c
@dawitfamanu
Reaction usecase, repo, api module binding
683be1f
@dawitfamanu
fix: match reaction API call parameter with server side
0f11d0b
@dawitfamanu
feat: call upload reaction API when emoji is selected
6bfaf59
@thisisWooyeol @yangchanhk98
fix: add emoji unicode of each reaction in post database
c26929d
@yangchanhk98
fix: collect error messages in ErrorType.kt
f299e95
@dawitfamanu
Merge pull request #96 from snuhcs-course/feat/back-error_message
8ef4b62 
Feat/back error message
@dawitfamanu
fix: remove Dialog on add reaction success
9813b77
@dawitfamanu
fix: match deleteReaction API's param with server implementation
0c3131d
@yangchanhk98
fix: collect all string values in StringValue.kt
7692cba
@thisisWooyeol
fix: resolve EmojiViewModelTest failure
cb8243f
@thisisWooyeol
refactor: make consistency on emoji fetch mode as sortByDate
49a1fc9
@yangchanhk98
Merge pull request #95 from snuhcs-course/feat/android-emoji-sort
361f1ba 
feat: add sorting mode in emoji tab
@dawitfamanu
Merge branch 'develop' into feat/android-reaction-API
f5db599
@yangchanhk98
feat: feat getMe API
53c6bf9
@thisisWooyeol
fix: fix import error Log
f45d5c5
@2018JunyoungLim
Merge pull request #98 from snuhcs-course/feat/back-getMe
6324afa 
Feat/back get me
@peng-u-0807
feat: disable LoginButton if username or password isEmpty
7a6e76b
@2018JunyoungLim
Merge pull request #97 from snuhcs-course/feat/android-reaction-API
cfff903 
Android - Reaction API Connection
thisisWooyeol and others added 23 commits December 9, 2023 01:41
@thisisWooyeol
feat: classname_to_unicode file only for 10 classes (for debugging, a…
0a422bc 
…s hugging face inference api only support 10 classes)
@thisisWooyeol
refactor: rename log tag
3965626 
refactor: rename emojiRepository so that it can choose X3dRepository or ClipRepository
@thisisWooyeol
refactor: refactor duplicated codes & fix typos
41cbcdd
@thisisWooyeol
feat: set up network connection with build config and proguard
b0a01a8
@thisisWooyeol
test: implement test script for MediaDataSourceImpl & ClipRepositoryImpl
f14a9dc 
refactor: fix test script of X3dRepositoryImpl & EmojiUseCaseImpl
@thisisWooyeol
refactor: fix typos & formatting
255df01
@thisisWooyeol
fix: remove redundant test call on github workflow
c74ac6a 
fix: add clip local.properties to github workflow
@peng-u-0807
fix: fix non-user dialog not showing issue
518e6d1
@peng-u-0807
fix: enable auto-play and remove buttons
8e7ccda
@thisisWooyeol
Merge pull request #126 from snuhcs-course/feat/clip-dl-api
c4cb555 
Use CLIP model for video -> emoji inference
@peng-u-0807
feat: add background to video player
2024000
@peng-u-0807
fix: suppress lint
d8746f9
@peng-u-0807
fix: change CustomBottomSheet button UI
b264478
@peng-u-0807
feat: apply VideoPlayer UI change to PlayEmojiView
b6ac27e
@thisisWooyeol
Merge pull request #128 from snuhcs-course/fix/android-transform-UI
ad800a4 
Minor UI changes for `TransformVideoPage`
@thisisWooyeol
Merge pull request #127 from snuhcs-course/fix/android-non-user
50a1177 
Fix non-user dialog not showing issue
@dawitfamanu
Merge pull request #123 from snuhcs-course/test/reaction-unit-test
62cf027 
Test: reaction unit test
@thisisWooyeol
refactor: make log directory for hagrid
a95a163
@thisisWooyeol
checkpoint: [GCP] efficient_x3d_s finetuning on whole dataset
5ff41fe 
config:
    trainable params: s5.pathway, head, projection layers
    optimizer: AdamW with lr: 3e-5
    total_epochs: 1
    batch_size: 48
-> version_15에 이어서 연속 2회 진행
@thisisWooyeol
train results of 231107 finetuning
a60ad40
@thisisWooyeol
tools to merge version 15, 16,17 training logs
040ad3c
@thisisWooyeol
cumulated train results of version 15,16,17, until 231210
6bcfcc9
@thisisWooyeol
.pt model for emojihub app
d5686eb
yangchanhk98
yangchanhk98 reviewed Dec 10, 2023
View reviewed changes
Copy link
Contributor

@yangchanhk98 yangchanhk98 left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@yangchanhk98 yangchanhk98 merged commit 580fae7 into release Dec 10, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yangchanhk98 yangchanhk98 yangchanhk98 left review comments

@dawitfamanu dawitfamanu Awaiting requested review from dawitfamanu

@peng-u-0807 peng-u-0807 Awaiting requested review from peng-u-0807

@2018JunyoungLim 2018JunyoungLim Awaiting requested review from 2018JunyoungLim

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

Successfully merging this pull request may close these issues.
5 participants
@thisisWooyeol @yangchanhk98 @2018JunyoungLim @dawitfamanu @peng-u-0807