Textures Manager #729

vklachkov · 2022-01-14T00:50:08Z

Менеджер текстур

Цель

Данный PR нацелен на создание менеджера текстур для упрощения рендеров и избавления от дублирующего кода

Мотивация

Впервые с необходимостью в отдельном менеджере я столкнулся при переключении рендера на лету. Перезагрузка ресурсов, в частности, текстур -- краеугольный камень для смены рендера без перезапуска движка. В своём черновике для принудительного обновления текстур были применены грязные хаки с флагами и отключением важных проверок. Желая избавится от них была начата работа в этом направлении

Перспектива

Менеджер преимущественно разрабатывается на перспективу.
В недалёком будущем планируется:

Вынос системы патчей материалов из ref_vk в менеджер ресурсов
Реализовать переключение рендеров. Meta issue с обобщением опыта будет в скором времени
Доработать менеджер текстур до полноценного менеджера ресурсов

План

Работу планирую разделить на два глобальных этапа

Этап 1

Этап нацелен на создание дублирующего слоя: менеджера текстур, работающего полностью идентично текущему

Этап 2

Этап нацелен на фиксы и облегчение ref_gl. Этот рендер стабилен, реализует все возможности движка и идеально подходит для экспериментов

Удалить хранение текстур
Удалить генерацию текстур-заглушек

Второй этап может выполняться параллельно с первым. Но потребует чужой помощи. Я не разбираюсь в рендерах. И патчинг ref_gl займёт у меня времени больше, чем у знающего человека

That's make menu less glitchy

That's fix most of glitches in menu. Add some logs

SNMetamorph · 2022-01-14T00:53:24Z

Это как-то повлияет на рендеры, реализованные внутри клиентской игрокой библиотеки через RenderAPI?

vklachkov · 2022-01-14T00:55:59Z

Это как-то повлияет на рендеры, реализованные внутри клиентской игрокой библиотеки через RenderAPI?

~~Возможно, повлияет. Не думал, не тестировал. Если требуется, протестирую и поддержу~~

Нет, не повлияет. Никаких изменений в render api я не вношу принципиально. Сейчас он будет работать некорректно: проксирование методов мной не реализовано. Это сделаю позднее, как дойду до работ в рендере

0x4E69676874466F78 · 2022-01-14T01:00:39Z

@SNMetamorph рендеры наверное зовут что-то типа ref.dllFuncs.GL_LoadTexture? тогда менеджер текстур по идее это перехватывает.

vklachkov · 2022-01-14T03:53:11Z

Эксперименты в mainui попали в данный PR случайно

vklachkov · 2022-01-14T04:09:31Z

Дополнительно отмечу, что где-то утекает ref_api.h. Если его подключить несколько раз подряд, случается такое на несколько экранов:

a1batross · 2022-01-14T10:15:25Z

Лучше за основу как раз брать ref_gl, а ref_vk уже подпиливать. Я в целом пока рекомендую забить на существование ref_vk, неизвестно насколько корректно он реализован.

Mod_ProcessRenderData исключительно рендероспецифичная. То что она наполовину закомментирована ничего страшного -- значит ничего делать. Вызывается при загрузке и выгрузке модели.

vklachkov · 2022-01-14T14:54:17Z

Лучше за основу как раз брать ref_gl, а ref_vk уже подпиливать. Я в целом пока рекомендую забить на существование ref_vk, неизвестно насколько корректно он реализован

За основу взят общий код для обоих рендеров

Mod_ProcessRenderData исключительно рендероспецифичная. То что она наполовину закомментирована ничего страшного -- значит ничего делать. Вызывается при загрузке и выгрузке модели.

Понял. Как мне лучше поступить с ней? Четыре сценария изложил выше

a1batross · 2022-01-15T00:08:27Z

Ничего не делать. Это загрузка моделей, она не связана с текстурами.

Возможно, лучше завести новый коллбэк, который будет оповещать рендер что пора загрузить текстуру в видеопамять и присвоить ей рендероспецифичный handle.

vklachkov · 2022-01-15T00:28:16Z

Ничего не делать. Это загрузка моделей, она не связана с текстурами.

Вы не правы. При загрузке моделей происходит загрузка и текстур через функцию R_SpriteLoadFrame (см. gl_sprite.c:50). Происходит это в обход движка. И именно это я пытаюсь решить. То ли хачить пока не поправлю рендер, то ли прокидывать менеджер ресурсов в рендер

Возможно, лучше завести новый коллбэк, который будет оповещать рендер что пора загрузить текстуру в видеопамять и присвоить ей рендероспецифичный handle.

Да, так и будет сделано, как буду вырезать управление текстурами из рендера. Сейчас я этого не делаю для упрощения отладки

a1batross · 2022-01-15T00:33:58Z

Ничего там в обход не происходит. У тебя уже есть LoadTexture в движке, пусть он движковый и вызывает.

It's isn't beautiful method, but in current situation... Need to implement loading textures from buffer to fix more glitches

Hypothetically broken stub textures, practically broken lightmaps

a1batross · 2022-08-08T14:14:27Z

А ты движок вообще собираешь после своих коммитов?

Потому что я ребейзнул и ничего не собирается. По крайней мере под Linux.

a1batross · 2022-08-08T14:29:52Z

Разреши коммитить в твою ветку мейнтейнерам. На гитхабе есть где-то галочка.

Пока залил мелкие исправления в resman_texture в нашем репозитории. НЕ МЕРЖИ ИХ, ПОТОМУ ЧТО ЭТО РЕБЕЙЗ!!11111111

vklachkov · 2022-08-08T16:00:43Z

А ты движок вообще собираешь после своих коммитов?

Потому что я ребейзнул и ничего не собирается. По крайней мере под Linux.

У меня на Linux точно так же не собирается по какой-то причине. Жалуется на TRUE/FALSE и incompatible pointers. А на Windows всё хорошо, собираю и тестирую на оном

vklachkov · 2022-08-08T16:02:11Z

Разреши коммитить в твою ветку мейнтейнерам. На гитхабе есть где-то галочка.

Пока залил мелкие исправления в resman_texture в нашем репозитории. НЕ МЕРЖИ ИХ, ПОТОМУ ЧТО ЭТО РЕБЕЙЗ!!11111111

Да я и сам хотел ребейзнуть, но случайно разнёс ветку в попытках починить. Думал отложить синхронизацию с мастером до лучших времён...) Я руками там восстановил утерянные изменения в gl_image.c, а больше ничего критичного вроде бы и нет

vklachkov · 2022-08-08T16:17:20Z

Основную пачку разгрёб.

Осталось...

Вернуть R_ShowTextures в ref_api
Поправить оформление в texturelib
Прокомментировать счётчик текстур в texturelib
Понять, зачем мы храним оригинал текстуры и нужно ли его оставлять в ref

Ohhhhh...

a1batross · 2022-08-08T17:10:31Z

Возьми мою ветку, там под линукс собирается теперь.

Эту просто перепиши форспушем. :)

vklachkov · 2022-08-09T00:23:26Z

Возьми мою ветку, там под линукс собирается теперь.

Какую твою?

Эту просто перепиши форспушем. :)

Эт шо, мне переносить всё здесь напатченное в 30+ файлах рученьками переносить на другую ветку? Ий историю коммитов потеряю, и долгое

a1batross · 2022-08-09T00:31:52Z

@zgdump resman_texture. https://github.com/FWGS/xash3d-fwgs/tree/resman_texture

Зафетчи fwgs remote, чтобы ветка была.

Эт шо, мне переносить всё здесь напатченное в 30+ файлах рученьками переносить на другую ветку? Ий историю коммитов потеряю, и долгое

Нет. Закоммити что ты сделал в свою, возьми мою ветку, черрипикни в неё то что ты накоммитил в своей и форспушни поверх.

a1batross · 2023-03-27T01:09:48Z

@zgdump any news on this? I really would like to see this upstreamed.

0x4E69676874466F78 · 2023-04-23T09:55:27Z

Последняя его TODO:

1. Вернуть R_ShowTextures в ref api
2. Поправить оформление в texturelib
3. Прикрутить текстуры неба
4. Протестировать текстуры-заглушки
5. Решить, стоит ли оставлять оригинал текстуры в ref_ и вообще для чего оно использовалось раньше

Похоже zgdump не в состоянии уже закончить, но возможно это будет по пути @w23.

vklachkov added 11 commits January 13, 2022 06:28

resman: Implement texture methods stubs

09093ae

resman: Minimal implementation of texture loading

28bd6b9

resman: Pass ref_interface_t to resource manager, save texture in mem

406dbe2

resman: Check texture name

f25ae90

resman: Implement cache

b55c92a

That's make menu less glitchy

resman: Generate stub entries for internal textures

bcbfb7c

That's fix most of glitches in menu. Add some logs

resman: Implement RM_FindTexture

378aa88

resman: Set correct names for first six textures

067c534

resman: Restore strange flags code

812af16

resman: Stab more methods

697f3b3

refgl: Dirty hack for get backtrace at texture n 121

c0e7394

vklachkov changed the title ~~Resource Manager~~ Textures Manager Jan 14, 2022

This comment was marked as outdated.

Sign in to view

This comment was marked as off-topic.

Sign in to view

Checkout mainui t master branch

5074761

vklachkov added 5 commits January 15, 2022 03:50

ref_gl: Remove dirty debug hack

1edd2f7

ref: Add RM_LoadTexture to ref_api_t

ee6669b

ref_gl: Load some textures via engine, fix sprite glitches

6a1e1c3

It's isn't beautiful method, but in current situation... Need to implement loading textures from buffer to fix more glitches

resman: Implement RM_LoadTextureFromBuffer, fix more glitches

3c65550

Hypothetically broken stub textures, practically broken lightmaps

ref_gl: Load texture for sky via engine

54fc8a0

vklachkov added 3 commits August 8, 2022 15:19

ref_soft: Minor changes, restore most of comments

d981fe7

ref_soft: Restore R_TextureList_f

4ce3d59

ref_soft: Find standard textures in the engine

b2d1283

a1batross force-pushed the master branch from 84c0de6 to aa3a0fa Compare January 3, 2023 01:02

a1batross force-pushed the master branch 4 times, most recently from db40812 to a610b15 Compare February 5, 2023 00:45

a1batross force-pushed the master branch from df7f100 to 098c4c0 Compare March 21, 2023 02:16

a1batross force-pushed the master branch 8 times, most recently from 8135f83 to 6c62136 Compare March 28, 2023 19:42

This was referenced Apr 23, 2023

materials: new format w23/xash3d-fwgs#470

Open

materials: loading textures even when they are not needed w23/xash3d-fwgs#372

Closed

a1batross force-pushed the master branch from 914ae68 to 5c6b9d3 Compare July 22, 2023 01:50

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Textures Manager #729

Textures Manager #729

vklachkov commented Jan 14, 2022 •

edited

SNMetamorph commented Jan 14, 2022

vklachkov commented Jan 14, 2022 •

edited

0x4E69676874466F78 commented Jan 14, 2022

vklachkov commented Jan 14, 2022

This comment was marked as outdated.

This comment was marked as off-topic.

vklachkov commented Jan 14, 2022

a1batross commented Jan 14, 2022

vklachkov commented Jan 14, 2022

a1batross commented Jan 15, 2022

vklachkov commented Jan 15, 2022 •

edited

a1batross commented Jan 15, 2022

a1batross commented Aug 8, 2022

a1batross commented Aug 8, 2022 •

edited

vklachkov commented Aug 8, 2022

vklachkov commented Aug 8, 2022

vklachkov commented Aug 8, 2022

a1batross commented Aug 8, 2022

vklachkov commented Aug 9, 2022

a1batross commented Aug 9, 2022

a1batross commented Mar 27, 2023

0x4E69676874466F78 commented Apr 23, 2023

Textures Manager #729

Are you sure you want to change the base?

Textures Manager #729

Conversation

vklachkov commented Jan 14, 2022 • edited

Менеджер текстур

Цель

Мотивация

Перспектива

План

Этап 1

Этап 2

SNMetamorph commented Jan 14, 2022

vklachkov commented Jan 14, 2022 • edited

0x4E69676874466F78 commented Jan 14, 2022

vklachkov commented Jan 14, 2022

This comment was marked as outdated.

This comment was marked as off-topic.

vklachkov commented Jan 14, 2022

a1batross commented Jan 14, 2022

vklachkov commented Jan 14, 2022

a1batross commented Jan 15, 2022

vklachkov commented Jan 15, 2022 • edited

a1batross commented Jan 15, 2022

a1batross commented Aug 8, 2022

a1batross commented Aug 8, 2022 • edited

vklachkov commented Aug 8, 2022

vklachkov commented Aug 8, 2022

vklachkov commented Aug 8, 2022

a1batross commented Aug 8, 2022

vklachkov commented Aug 9, 2022

a1batross commented Aug 9, 2022

a1batross commented Mar 27, 2023

0x4E69676874466F78 commented Apr 23, 2023

vklachkov commented Jan 14, 2022 •

edited

vklachkov commented Jan 14, 2022 •

edited

vklachkov commented Jan 15, 2022 •

edited

a1batross commented Aug 8, 2022 •

edited