saas, ну она будет примерно такой шороховатой после применения bump mapping'а, нормал мапа в комплекте с котиком имеется, осталось шейдер допилить немного.

Ревьюшка: https://bitbucket.org/mkbkompas/engine/pull...-to-master/diff

Зачем в texture.cpp #pragma once?

saas, комментарии можно оставлять прямо на битбакете, для этого нужно залогиниться/зарегистрироваться под своим аккаунтом, открыть ревью и кликнуть мышкой на нужной строчке

refuse, ну ладно, написал ещё и на битбакете.

saas, спасибо, исправил. Там есть кнопочка approve, если замечаний нет больше нажми ее пожалуйста.

refuse, у меня нет, но я не сильно в C++11 разбираюсь.

Погоди-ка, а кто тогда аппрув поставил? Я похоже в ревьюверы еще кого-то добавил с таким же ником как у тебя

Заехал пул реквест в мастер, спасибо всем

Замечаний нет. Кнопочку я нажал.

Нашел математику от товарищей из гугла, выглядит вроде неплохо, SIMD оптимизация под различные платформы, включая мобильные http://google.github.io/mathfu/

В общем посмотрел на эту библиотеку, и на некоторые другие, и решил что в проэкт из нескольких классов не стоит затаскивать зависимость с несколькими сабмодулями и кучей разных систем сборки. Пусть пока останется наивная реализация, ибо как сказал один известный человек: "Premature optimization is the root of all evil".

Сейчас обдумываю чем бы заняться дальше, пока на примете такие варианты:
1. Прикрутить логирование и обработку ошибок.
2. Сделать редактор материалов и углубиться в реализацию эффектов и свойств материала.
3. ...


Сообщение отредактировал refuse - 24.12.2017, 12:40

Всё, я его собрал Первая неважная попытка освещения кота:


refuse, а у тебя какая версия винды? У меня и на Win7 и на XP была ошибка из за ReadFile.
https://msdn.microsoft.com/en-us/library/wi...v=vs.85%29.aspx


Алсо компилятор GCC ругается на векторы в анонимных союзах(в классах матриц).

У меня win 10, ошибки не было, однако в документации по этому кейсу действительно все однозначно определено. Можешь пул реквест с исправлением создать в ветку develop?


А вот тут надо уточнять что в стандарте С++11 по поводу анонимных структур говорится, у тебя какая версия GCC ?

А что делать если количество считанных байт != size? Возвращать false?


Последняя из Mingw-w64, 7.2.0.

Сообщение отредактировал saas - 24.12.2017, 18:29

По идее если функция вернула код успешного завершения операции, то все ок, так что если нам вернули буфер меньшего размера чем мы попросили я бы просто ворнинг в лог писал, а там пускай оно в другом месте валится с другой ошибкой. Но ты на свое усмотрение делай, ревью еще abramcumner будет смотреть.


Ok. Поправь пожалуйста и это тоже.

Выглядит так, словно bump mapping работает, однако из диффузной карты цвет не применился

Сообщение отредактировал refuse - 24.12.2017, 19:04

refuse, диффузная карта на скриншоте не используется. Почему-то после BindHardwareTexture карта нормалей оказывалась в обоих семплерах.

saas, Прокомментировал. Добавь пожалуйста abramcumner'а в ревьюверы, пусть тоже посмотрит.

refuse, добавил, на комментарий ответил.

saas, заапрувил, подождем abramcumner'a и можно мержить.