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

GI 갈아엎기 / VCT 다시 구현 #38

Closed
Jin02 opened this issue Sep 10, 2016 · 1 comment
Closed

GI 갈아엎기 / VCT 다시 구현 #38

Jin02 opened this issue Sep 10, 2016 · 1 comment
Assignees
@Jin02
Labels
Done

Comments

@Jin02
Copy link
Owner

Jin02 commented Sep 10, 2016 •
edited

코드도 일관성이 없고
버그가 나도 어디에서 나는지 확인하기도 힘들다.

더군다나 설계하지 않고 그냥 손가는대로 구현하다보니 구조도 뒤죽박죽이다.
중복되는 코드또한 상당히 많다.
#29 #2 #7 #16 등.. 전부 VCT 구현하겠다고 삽질한 이슈들이다.

지금 현재 있는걸 다시 지우고 새롭게 구현하는게 옳다고 생각된다.
#15 도 염두에 두고 설계해야한다.
@Jin02 Jin02 self-assigned this Sep 10, 2016
@Jin02 Jin02 mentioned this issue Sep 10, 2016
Closed
@Jin02 Jin02 changed the title VCT 다시 구현 GlobalIllumination 갈아엎고 다시 구현 Sep 10, 2016
@Jin02 Jin02 changed the title GlobalIllumination 갈아엎고 다시 구현 VCT 다시 구현 Sep 10, 2016
@Jin02 Jin02 changed the title VCT 다시 구현 GI 갈아엎기 / VCT 다시 구현 Sep 11, 2016
Jin02 added a commit that referenced this issue Sep 24, 2016
@Jin02
VXGI 전체적으로 수정 #38
29663fe
Jin02 added a commit that referenced this issue Sep 26, 2016
@Jin02
MipmapVoxelMap -
1272a5b 
1. MipmapVoxelTexture -> MipmapVoxelMap 으로 이름 변경
2. AnisotropicMipmap 추가

#38
Jin02 added a commit that referenced this issue Sep 26, 2016
@Jin02
VoxelViewer - Init 수정 #38
bb65db2
Jin02 added a commit that referenced this issue Sep 26, 2016
@Jin02
Voxelization - Clear 부분 수정 #38
b021f43
Jin02 added a commit that referenced this issue Sep 26, 2016
@Jin02
VCT 갈아엎음 #38
64aa46b
@Jin02 Jin02 added Done and removed Accept labels Sep 26, 2016
@Jin02 Jin02 closed this as completed Sep 26, 2016
@Jin02 Jin02 mentioned this issue Sep 26, 2016
Closed
Jin02 added a commit that referenced this issue Sep 27, 2016
@Jin02
vct - 테스트 중 #38
44e9a6b
Jin02 added a commit that referenced this issue Sep 27, 2016
@Jin02
Merge branch '#37' into #38
0696308
Jin02 added a commit that referenced this issue Sep 27, 2016
@Jin02
Merge branch '#38'
3c91347
@Jin02 Jin02 mentioned this issue Oct 7, 2016
Closed
@Jin02 Jin02 mentioned this issue Sep 9, 2018
Closed
@Jin02
Copy link
Owner Author

Jin02 commented Sep 9, 2018

This issue was moved to TeamSOC/SOCEngine#38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Jin02 Jin02

Labels
Done
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Jin02