Skip to content

Tut28 - 29 preliminary code#37

Draft
blurrypiano wants to merge 6 commits into
mainfrom
wip28-29
Draft

Tut28 - 29 preliminary code#37
blurrypiano wants to merge 6 commits into
mainfrom
wip28-29

Conversation

@blurrypiano
Copy link
Copy Markdown
Owner

Rough draft of code for tutorials 28 and 29 covering per frame descriptors, game object manager and textures

@Jark5455
Copy link
Copy Markdown

Can we use an implementation of the transitionImageLayout function that is more flexible and supports more image layouts instead of hardcoding every specific situation like it is now? Maybe something along the lines of this? Link

Comment thread src/lve_game_object.hpp
LveGameObject(id_t objId, const LveGameObjectManager &manager);

id_t id;
const LveGameObjectManager &gameObjectManger;
Copy link
Copy Markdown

@ghost ghost Dec 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo here

Comment thread src/lve_texture.cpp

void LveTexture::createTextureImage(const std::string &filepath) {
int texWidth, texHeight, texChannels;
// stbi_set_flip_vertically_on_load(1); // todo determine why texture coordinates are flipped
Copy link
Copy Markdown

@imbobz imbobz Oct 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants