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

arg fish effect and other shaders #52

Merged
merged 59 commits into from
Dec 16, 2023
Merged

arg fish effect and other shaders #52

merged 59 commits into from
Dec 16, 2023

Conversation

w1n7er87
Copy link
Contributor

sending this from my phone xdx 😨 , I put the effect ✨ in hiyori prefab, on a separate object 🏒, and everything should be good to go πŸ‘, just build the assets 😊.
I did my best πŸ’ͺ to provide some explanation πŸ€” on other effects if they might be useful. also, my tri inspector was on a different commit for some reason 😞, but I hope it's not a big issue (Clueless) . I might not be able to make it to pr-review, so lmk if anything needs to be changed.

@w1n7er87
Copy link
Contributor Author

w1n7er87 commented Dec 3, 2023

the fact that most of the effects are hard coded and assigned at vfxMaterialHolder, causes all the images and etc. used by materials to be included in assets, I should probably do something about that

@w1n7er87 w1n7er87 marked this pull request as draft December 4, 2023 04:49
@Alexejhero Alexejhero self-assigned this Dec 6, 2023
@Alexejhero Alexejhero self-requested a review December 8, 2023 05:04
@w1n7er87 w1n7er87 marked this pull request as ready for review December 8, 2023 10:00
Unity/Assets/Scripts/SCHIZO/Utilities/T2DArray.cs Outdated Show resolved Hide resolved
Unity/Assets/Scripts/SCHIZO/VFX/VFXComponent.cs Outdated Show resolved Hide resolved
Unity/Assets/Scripts/SCHIZO/VFX/SchizoVFXStack.cs Outdated Show resolved Hide resolved
Unity/Assets/Scripts/SCHIZO/VFX/SchizoVFXStack.cs Outdated Show resolved Hide resolved
Unity/Assets/Scripts/SCHIZO/VFX/MatPassID.cs Outdated Show resolved Hide resolved
@Alexejhero
Copy link
Owner

Let me know when this PR is ready and I will merge it

@w1n7er87
Copy link
Contributor Author

should be good

@Alexejhero Alexejhero merged commit 49ba2f2 into Alexejhero:main Dec 16, 2023
2 checks passed
@w1n7er87 w1n7er87 deleted the uuh branch May 15, 2024 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Archive
Development

Successfully merging this pull request may close these issues.

2 participants