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

Gpumenu #158

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Gpumenu #158

wants to merge 6 commits into from

Conversation

ksjfhor
Copy link
Contributor

@ksjfhor ksjfhor commented Apr 8, 2024

Added changeable World Environment to Lobby and the Lab, took the one from the lobby and put it into scene management for easy reuse.
Added preset graphics options, range from potato to ultra maschine.
The presets do not set the 2D and 3D AA, for now.

So far, I uncommented the directional light on the shadow size, because it all would crash without a directional light in the main menu.

Feel free to edit and to tell me what you think !

Added changeable World Environment
now it is really okay
Copy link
Contributor

Choose a reason for hiding this comment

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

I dont know if this is the case here, but for me i noticed this file always changes when i boot up the game even if i haven't made any changes, if that's the case I think it's good practice to not commit these changes in PRs unrelated to the theme file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your feedback !
In fact, I changed nothing (at least not on purpose) on the theme.
As you may see in other PRs, this theme get updated quite often, so I remove this and try to keep this in mind :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I can confirm that Cogito_Theme_A.tres gets file changes if you even look at it weird.

Nothing was changed, so there is no need to push.
@ksjfhor ksjfhor mentioned this pull request Apr 8, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you deleted the file as a whole instead of removing the changes from the branch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Indeed.

@Phazorknight
Copy link
Owner

Just adding a comment here that I've provided some feedback and thoughts in the Issue thread here: #55

- Celling in the Lab made thicker and reduced the spotlight by the desks, moved the sunlight outside the building.
- Created a DirectionalLight group to handle the change of non existing DirectionalLight in the Scene
- Renamed a few nodes for a better and more similar reading
- Added a Headbobble option to the gameplay tab
- Commented the glow and sdfgi options plus saves and loads of the cfg
Copy link
Contributor Author

@ksjfhor ksjfhor left a comment

Choose a reason for hiding this comment

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

Not sure about the headbobble, somehow something git ed up.

@ksjfhor
Copy link
Contributor Author

ksjfhor commented Apr 16, 2024

So the Headbobble mismatch came from PR#161.
I do not know why, but I cannot pull anything into this PR, every adjustments from now on would I have to do from hand.
sooo .... @Phazorknight what do you think now ? #55

-swapped out the margin for a grid in the audio tab
-added in PR#161
-More chances to apply changes :)
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.

4 participants