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

Merge missing changes by finepointcgi and I (continued his fork) #29

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

Conversation

ParkingLotGames
Copy link
Collaborator

No description provided.

Fixed all compiler warnings (some warnings suppressed and need addressing (replace WWW().texture with UnityWebRequest())
Added Blurred (Most inefficient blur ever) Skybox shader to remove the background sphere that was interferring with RT GI
Scrapped React based UI due to bugs and lack of documentation on ReactUnity Project
Started working on new UnityDynamicPanels based UI (https://github.com/yasirkula/UnityDynamicPanels)
added headers in hierarchy and tessellated quad packages
Moved Post-processing and misplaced scripts to Scripts > Postprocessing to be replaced later on
Moved misplaced textures to Textures
Removed Collab package that was erroring (was not affecting anything, just couldn't load)
CommandListExecutor has unreachable code, need to rollback changes made to MainGui version that has documentation or transfer documentation to current version, then enable switch statement in  CommandListExecutor
need to test build and check if the async changes allow saving files noproblem
@ParkingLotGames ParkingLotGames changed the title Merge missing changes by maikramer Merge missing changes by finepointcgi and I (continued his fork) Jan 16, 2023
@maikramer
Copy link
Owner

Can you try to solve the conflicts?Thanks

@ParkingLotGames
Copy link
Collaborator Author

Can you try to solve the conflicts?Thanks

will do, might take a while but I'll update you

@maikramer
Copy link
Owner

I have no time to continue this, then now @ParkingLotGames is a member of this repository.

@maikramer
Copy link
Owner

I will add @finepointcgi also, then we can make it together.

@maikramer
Copy link
Owner

I think the conflicts wont be easy to fix, but, would be good if you do.

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.

None yet

3 participants