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

Effect Shader does *NOT* work anymore #24

Closed
elmkni opened this issue Mar 10, 2020 · 5 comments
Closed

Effect Shader does *NOT* work anymore #24

elmkni opened this issue Mar 10, 2020 · 5 comments

Comments

@elmkni
Copy link

elmkni commented Mar 10, 2020

Hi Michalis,

it's me, Elmar (again) ...

I've just tested the latest snapshot of View3DScene.exe

view3dscene-3.18.0-win64-x86_64.zip | 2020-03-10 06:09 | 4.1M | Latest Windows version (64-bit)

and got following complaint by view3dscene when I use an 'Effect' shader:

VRML/X3D: Plug name "texture_apply" not declared (in shader type "Fragment")

although this file worked perfectly with an older view3dscene snapshot from 2020-02-20!

it should look like this:
BackFog_animated fx

It does work when I use a complete 'ComposedShader' --- am I missing something?

I'll attach the test files (both: Effect shader and ComposedShader version) for you to test.

Please help!

With best regards,

Elmar

view3dscene_issue_24.zip

@michaliskambi
Copy link
Member

The plug name changed to main_texture_apply. Unfortunately I had to change it, the previous name was really bad (it does not affect all textures). I didn't yet announce this change, but it's related to recent PBR and material/lighting improvements on https://castle-engine.io/wp/2020/03/07/physically-based-rendering-and-lots-of-other-lighting-materials-upgrades-in-castle-game-engine-and-x3d-4-0/ .

@elmkni
Copy link
Author

elmkni commented Mar 10, 2020

Hi Michalis,

thanks for the quick answer!

So I have to change all my files ...

with best regards,

Elmar

@michaliskambi
Copy link
Member

Hm, let me implement an upgrade path actually. I can make the texture_apply working again, just deprecated. This should be easy, I'll do it ASAP.

@michaliskambi michaliskambi reopened this Mar 10, 2020
@michaliskambi
Copy link
Member

I pushed a simple fix to enable PLUG name texture_apply for backward compatibility.

In about 15 minutes, latest view3dscene snapshot from http://michalis.ii.uni.wroc.pl/view3dscene-snapshots/ should have it working :)

@elmkni
Copy link
Author

elmkni commented Mar 11, 2020

Hi Michalis,

thanks a lot for the 'PLUG_main_texture_apply' work-around!

In the meantime I had already updated my effect files ;)
but it is always good to have backward compatibility!

With best regards,

Elmar

P.S.:
I've checked your test-suite concerning the new X3D v4.0 stuff (especially PBR and unlit material), and it looks pretty awesome! I am really looking forward to all the possibilities X3D v4.0 will bring!

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

No branches or pull requests

2 participants