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

Tuned LDR and HDR env settings for included maps #4

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

Jacobdeanr
Copy link

  • Environment entities like lighting, fog, and tonemap values has been fixed/tuned for default maps, for LDR and HDR.
  • Updated default maps to utilize the hdr variant of their chosen skybox (falls back to ldr variant if client is not using HDR).
  • sb_field and sb_field_night have a new spawn platform.
  • increased lightmap resolution on all maps
  • Compiled demo maps, and built cubemaps.
  • Including .ain to prevent node graph message on spawn (no ai nodes in any of the included maps)

andrewmcwatters and others added 6 commits November 22, 2024 16:23
environment entities like lighting, fog, and tonemap values has been tuned for default maps.
sb_field and sb_field_night have a new spawn platform.
Compiled demo maps, and built cubemaps.
Including .ain to prevent node graph message on spawn (no ai nodes in any of the included maps)
@andrewmcwatters
Copy link
Member

@Jacobdeanr I want to merge this, but you have no commit history notes and your work is squashed to a single commit.

@andrewmcwatters andrewmcwatters added the invalid This doesn't seem right label Dec 1, 2024
@andrewmcwatters
Copy link
Member

OK, yep, perfect. I'll schedule this for merge.

@andrewmcwatters andrewmcwatters added enhancement New feature or request and removed invalid This doesn't seem right labels Dec 1, 2024
@Jacobdeanr
Copy link
Author

@andrewmcwatters I've updated all maps to have individual comments on feature changes.

@andrewmcwatters
Copy link
Member

Yeah, this is semantically correct now. Thanks, I realize it's tedious! But it's technically correct and makes reviews easier, and also easier for us individually to git rebase when we have to alter PRs.

@andrewmcwatters
Copy link
Member

@Jacobdeanr I think this is ready to be rebased on the latest master now that #8 has been merged.

Accidentally removed vproject from the gitignore. Re-added
@andrewmcwatters
Copy link
Member

@Jacobdeanr Do you want us to make GitHub Actions build runners compile maps on .vmf changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants