Skip to content

Tuned LDR and HDR env settings for included maps#4

Open
Jacobdeanr wants to merge 20 commits into
Planimeter:masterfrom
Jacobdeanr:master
Open

Tuned LDR and HDR env settings for included maps#4
Jacobdeanr wants to merge 20 commits into
Planimeter:masterfrom
Jacobdeanr:master

Conversation

@Jacobdeanr

Copy link
Copy Markdown
  • 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Author

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

@andrewmcwatters

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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