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.

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

코드 정리 및 미구현된 항목들 정리 #19

Closed
Jin02 opened this issue Jun 18, 2016 · 6 comments
Closed

코드 정리 및 미구현된 항목들 정리 #19

Jin02 opened this issue Jun 18, 2016 · 6 comments
Assignees
Labels

Comments

@Jin02
Copy link
Owner

Jin02 commented Jun 18, 2016

  1. class 선언부분에서 변수 내리고 함수 올리는 작업들?
    • 당장 급한건 아니고, 굳이 보기 힘들거나 그런건 아니기에 나중에 해도 괜찮을 것 같다.
  2. Texture와 Map의 용어를 통일할것, shader 내 코드 정리들
    • 완료
  3. 그 외 코드 정리 등등..

현재 지내는 환경이.. 환경이다보니.. 이런 작업 위주로 진행하는게 좋을듯 싶다.
굳이 디버깅이 필요없는 작업들이니깐 말이다.

@Jin02 Jin02 added the Accept label Jun 18, 2016
@Jin02 Jin02 self-assigned this Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
USE_OUT_ANISOTROPIC_VOXEL_TEXTURES -> USE_OUT_VOXEL_MAP #19
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
그리고 코드 정리
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
Jin02 added a commit that referenced this issue Jun 18, 2016
@Jin02
Copy link
Owner Author

Jin02 commented Jun 18, 2016

g_tGBufferAlbedo_occlusion
이런 이름도 다 바꾸자

g_t 이런 수식어는 전부제거하고 앞을 대문자로 설정하자.
예를들면,
g_tGBufferAlbedo_occlusion -> GBufferAledo_occlusion 이런 식으로..
그렇게 고치면 될 듯 싶다.

Depth와 BlendedDepth 이걸
GBufferDepth, GBufferBlendedDepth 이걸로 변경해야함

완료

@Jin02
Copy link
Owner Author

Jin02 commented Jun 19, 2016

PhysicallyBased_GBuffer_Common - 완성이 덜 됬다.

현재 GBuffer가

  1. albedo(3), occlusion(1)
  2. motion(2), height(1), metallic(1)
  3. normal(3), roughness(1)
  4. emission(3), specularity(1)

이다.
여기서, 현재 구현자체 시작을 못한건 motion이다.

그리고.. 여기서 materialID를 가진 버퍼 하나를 만들어 내야한다.
height를 없애고 materialID를 넣는게 좋을 듯 싶다.
motion.xy가 unorm 형식이라고 해도 크게(?) 영향을 끼칠 것 같진 않고 materialID는 1byte면 충분하다. 그리고.. metallic도 2byte나 먹을 필요가 없기에 변경하면 될 것 같다.

heightMap은 displacement mapping에 쓰려고 해둔거다.
그래서 굳이 저렇게 뺄 이유가 없다. 다른 단계에서 저걸 읽어와서 쓰겠지.

처리 완료
#24 와 연관된다.

@Jin02
Copy link
Owner Author

Jin02 commented Jun 19, 2016

BRDF.h 에서 fresnel0은?

@Jin02
Copy link
Owner Author

Jin02 commented Jun 19, 2016

아, 작업은 #16_0 에서 작업한다.
Voxelization 방식을 새롭게 바꿔야해서 이런식으로 가려고 한다.

@Jin02 Jin02 changed the title 코드 정리 코드 정리 in Army Jun 19, 2016
@Jin02 Jin02 added Incompleted and removed Accept labels Jun 19, 2016
@Jin02 Jin02 closed this as completed Jun 19, 2016
@Jin02 Jin02 changed the title 코드 정리 in Army 코드 정리 및 미구현된 항목들 정리 Jun 19, 2016
@Jin02 Jin02 mentioned this issue Jun 19, 2016
@Jin02 Jin02 added this to the 신병 휴가동안 작업해야 할 목록 milestone Jul 1, 2016
@Jin02 Jin02 reopened this Jul 1, 2016
@Jin02 Jin02 modified the milestone: 신병 휴가동안 작업해야 할 목록 Jul 2, 2016
@Jin02 Jin02 mentioned this issue Jul 2, 2016
@Jin02
Copy link
Owner Author

Jin02 commented Jul 6, 2016

굳이.. 1번 작업이 필요하긴 할까?
그냥 이거 때문에 이슈 열어놓는 것 보단 나중에 언급하는게 좋다고 생각한다.

@Jin02
Copy link
Owner Author

Jin02 commented Sep 9, 2018

This issue was moved to TeamSOC/SOCEngine#19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant