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

Skybox stuff #677

Closed
3 of 6 tasks
Tracked by #708
0x4E69676874466F78 opened this issue Dec 1, 2023 · 1 comment
Closed
3 of 6 tasks
Tracked by #708

Skybox stuff #677

0x4E69676874466F78 opened this issue Dec 1, 2023 · 1 comment
Labels
enhancement New feature or request materials ray tracing refactoring Improvement of the code base

Comments

@0x4E69676874466F78
Copy link
Collaborator

0x4E69676874466F78 commented Dec 1, 2023

  • в оригинале текстуры неба располагаются в gfx/env, в рендере почему-то env, я сходу не въехал.

  • рендер грузит небо по схеме имя_сторона, хотя в игре и модах это как имясторона

    if( loadname[len - 1] == '_' )

  • не работает перезагрузка текстур через перезагрузку материалов

  • хотелось бы HDR скайбокс

@w23
Copy link
Owner

w23 commented Dec 18, 2023

  • в оригинале текстуры неба располагаются в gfx/env, в рендере почему-то env, я сходу не въехал.

Я тоже не помню, почему мы так решили. Всегда можно легко поменять на что угодно.

* [ ]  рендер грузит небо по схеме `имя_сторона`, хотя в игре и модах это как `имясторона`
  https://github.com/w23/xash3d-fwgs/blob/6fda8bd977da55b23c9fee2cd00cf9e5b027fc81/ref/vk/r_textures.c#L936-L936

У нас тоже так было -- мы пробовали оба варианта.
С последними изменениями мы переехали на то, что делает imagelib у себя внутри. А именно: опять пробует два (или даже три?) варианта.
Но, мы также умеем загружать теперь кубмапу целиком из одного КТХ2 файла.

* [ ]  не работает перезагрузка текстур через перезагрузку материалов

Сделали в #708.

* [ ]  хотелось бы HDR скайбокс

Сделали в #708 ;)

  * [ ]  с настройкой (mat файл) где можно регулировать "яркость" неба (зависимость [Adaptive tone mapping (dynamic/auto exposure) #193](https://github.com/w23/xash3d-fwgs/issues/193))

Сделали в #708 ;))

  * [ ]  https://github.com/KhronosGroup/glTF-IBL-Sampler

?

@w23 w23 moved this from 🆕 New to 👀 In review in HLRT Dec 18, 2023
w23 added a commit that referenced this issue Dec 18, 2023
Skybox improvements (E346-E349)

- [x] #706 
  - [x] Do not load skybox if map doesn't have any `SURF_DRAWSKY` surfaces.
  - [x] Cache: do not reload the same skybox.
  - [x] Enable loading packed KTX2 cubemaps as skyboxes in engine/imagelib
- [x] Reuse existing imagelib cubemap loading routines, remove custom skybox loading code
- [x] Allow hiding `SURF_DRAWSKY` surfaces via `"_xvk_remove_all_sky_surfaces" "1"`, #579 
- [x] #677 
  - [x] Add skybox exposure control for HDR skyboxes
  - [x] Allow reloading skyboxes when reloading patches
@w23 w23 closed this as completed Dec 18, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in HLRT Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request materials ray tracing refactoring Improvement of the code base
Projects
Status: Done
Development

No branches or pull requests

2 participants