Skip to content

Conversation

@SendellX
Copy link
Member

@SendellX SendellX commented Jun 6, 2025

HAPPY B-DAY TO OUR LOVELY REMAKE PROJECT! 🥳
As a little celebration, I tried to extract the star shader from the Dome level through AI-assisted ✨vibe coding✨ I did my best to understand how it works, but I'm obviously still not a programmer, so I'm counting on your expertise to check if it makes any sense or if there's a better way to approach it!

If it's total nonsense, feel free to discard it entirely – I’ll just commit the 3D models instead 🤣
Speaking of which, this PR also includes all the missing models needed to complete the Francos Island asset replacement, yay! 🎉

Here's the screenshot of the very clean globe I added the stars to 🤭
shiny globe yay

Layouts 000, 001, 010, 011.
Layouts 009 and 012.
Added custom keepMat materials to these and the padded armchair.
Moved brick in Franco's Island's bar.
New Layout added:
Layout 083.

Fixed a positioning issue with Layout 073.
@netlify
Copy link

netlify bot commented Jun 6, 2025

Deploy Preview for lbaremake failed.

Name Link
🔨 Latest commit bc82237
🔍 Latest deploy log https://app.netlify.com/projects/lbaremake/deploys/6844729a60f849000865e201

@xesf
Copy link
Member

xesf commented Jun 6, 2025

Awesome stuff!!!
I will need some time to review it, it has been a long time :).

Attempt to extract and adapt the twinkling star shader from the Dome level. Added two of those around Mr. Kurtz's golden globe in scene 173 (entry 117).

Fix: Reversed vertex index order so the front side of the triangles would be drawn facing the camera. That way, the star doesn't need to be double sided ✌️
@SendellX
Copy link
Member Author

SendellX commented Jun 7, 2025

Awesome stuff!!! I will need some time to review it, it has been a long time :).

Thanks 💙 I hope it's been a good long time at least 😊

I made some small changes suggested by agrande so it seems the code wasn't too bad 🥳
But if you ever feel like taking a look at it, I'll gladly accept the review 🤗 Take care!

@agrande agrande merged commit add4620 into LBALab:master Jun 7, 2025
2 of 6 checks passed
@agrande
Copy link
Member

agrande commented Jun 7, 2025

Hmm, looks like the netlify deployment failed. Some issue with a missing config file.
Did we change something to the deployment setup recently?

@xesf
Copy link
Member

xesf commented Jun 7, 2025

Not that I remember. There were a few bug fixes releases that worked well. I wonder their service change and we need to upgrade something. I can have a look at that.

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