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

Hand Rendering #909

Merged
merged 60 commits into from
Nov 16, 2021
Merged

Hand Rendering #909

merged 60 commits into from
Nov 16, 2021

Conversation

maximumpower55
Copy link
Contributor

@maximumpower55 maximumpower55 commented Oct 2, 2021

closes: #349

@Kichura
Copy link

Kichura commented Oct 2, 2021

mc.mp4

Seems to be working properly in some places but with other shaders, it causes items to be transparent or hand becomes uglified from reflections. otherwise not bad.

@coderbot16
Copy link
Member

@Kichura what exactly is the context of the video? The whole world turning black a few seconds in seems like a pretty critical issue

@IMS212
Copy link
Member

IMS212 commented Oct 2, 2021

Doesn't seem like you're accounting for non-70 FOV's.

@IMS212
Copy link
Member

IMS212 commented Oct 2, 2021

It also seems to clip through blocks

@Kichura
Copy link

Kichura commented Oct 2, 2021

@Kichura what exactly is the context of the video? The whole world turning black a few seconds in seems like a pretty critical issue

The fading is from continuum itself and i did not make the changes to it, might have to look through the config. - as for video, was trying to show that hand's lighting exposure is working properly when aiming at a specific angle unlike everything. (changing camera angle makes hand look uglified)

edit: caused by "camera mode" when set to auto, meaning it's intended.

@maximumpower55 maximumpower55 marked this pull request as draft October 2, 2021 18:38
@maximumpower55
Copy link
Contributor Author

maximumpower55 commented Oct 2, 2021

Converting this to draft and starting to work on fixing the major issues

@maximumpower55
Copy link
Contributor Author

Fixed Most Major issues

this is some screenshots of it using Complementary v4.2.1

2021-10-02_13 02 53
2021-10-02_13 02 07
2021-10-02_13 02 14
2021-10-02_13 02 32

@IMS212
Copy link
Member

IMS212 commented Oct 2, 2021

Can you test Sildur's just to make sure

@maximumpower55
Copy link
Contributor Author

Sure one second

@maximumpower55
Copy link
Contributor Author

For some reason everything is black on sildurs vibrant medium

Can you test it on normal iris to make sure its not a issue with iris it self

@IMS212
Copy link
Member

IMS212 commented Oct 2, 2021

It's not, I had the same issue when previously attempting hand rendering which made me stop working on it. Not sure what's up with it.

@maximumpower55
Copy link
Contributor Author

More Debugging for me i guess

@maximumpower55
Copy link
Contributor Author

Looks like its a issue with the way i fixed the clipping issues

Im going to try to find a other way to fix them

@maximumpower55
Copy link
Contributor Author

Cant find a way to fix it

Im going to make this ready for review

If this gets merged

@maximumpower55 maximumpower55 marked this pull request as ready for review October 2, 2021 20:16
@maximumpower55
Copy link
Contributor Author

maximumpower55 commented Oct 2, 2021

Through out testing with other shaderpacks this is my collection of issues

Complementary 4.2.1 works with out any issues
Continuum 2.0.4 works with some issues mainly the clouds move with the camera
BSL 8.1.01 works with out any issues
Oceano 3.0.1 is broken
SEUS Renewed 1.0.1 the shadows are broken and the hand is transparent and there is a weird white overlay over the screen
Every Version of Sidurs Vibrant Shaders is broken everything is black but the water

@maximumpower55
Copy link
Contributor Author

@coderbot16 how is this looking for you

@coderbot16
Copy link
Member

I'm pretty sure clearing the depth buffer is going to cause a lot of chaos. I'll try to figure out how ShadersMod handled hand clipping.

@maximumpower55
Copy link
Contributor Author

ok

@maximumpower55
Copy link
Contributor Author

maximumpower55 commented Oct 2, 2021

give me all the feedback you have because im a first time contributor to iris

@Kichura
Copy link

Kichura commented Oct 2, 2021

image

Tested complementary here with a gold sword and now it looks worse than before, BSL however doesn't seem to show this issue here.

@maximumpower55
Copy link
Contributor Author

@Kichura what version of complementary

@Kichura
Copy link

Kichura commented Oct 3, 2021

@Kichura what version of complementary

v4.2.1 without modifications applied.

Copy link
Member

@coderbot16 coderbot16 left a comment

Choose a reason for hiding this comment

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

This is looking much nicer now, good work. I still have some things that I'd like resolved but it's definitely possible this could make it in to the next release.

src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
src/main/java/net/coderbot/iris/pipeline/HandRenderer.java Outdated Show resolved Hide resolved
@coderbot16
Copy link
Member

Once MixinItemBlockRenderTypes is addressed this should be ready for merge.

Copy link
Member

@coderbot16 coderbot16 left a comment

Choose a reason for hiding this comment

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

Thanks! The lack of hand rendering is one of the bigger issues affecting Complementary Shaders. I wasn't looking forward to implementing this myself, so it's great that you both took it on. Looking forward to shipping it in the next update!

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.

Unimplemented: "gbuffers_hand" program
5 participants