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

Revert the missing fix and add some logs #12367

Merged
merged 2 commits into from
Apr 9, 2022
Merged

Revert the missing fix and add some logs #12367

merged 2 commits into from
Apr 9, 2022

Conversation

deltakosh
Copy link
Contributor

No description provided.

@azure-pipelines
Copy link

Please make sure to tag your PR with "bug", "new feature" or "breaking change" tags.

@deltakosh deltakosh enabled auto-merge (squash) April 9, 2022 01:40
@azure-pipelines
Copy link

Snapshot stored with reference name:
refs/pull/12367/merge

Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/index.html

To test a playground add it to the URL, for example:

https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/index.html#WGZLGJ#4600

To test the snapshot in the playground itself use (for example):

https://playground.babylonjs.com/?snapshot=refs/pull/12367/merge#BCU1XR#0

@azure-pipelines
Copy link

Visualization tests for webgl1 have failed. If some tests failed because the snapshots do not match, the report can be found at

https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/testResults/webgl1/index.html

If tests were successful afterwards, this report might not be available anymore.

@azure-pipelines
Copy link

Visualization tests for webgl2 have failed. If some tests failed because the snapshots do not match, the report can be found at

https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/testResults/webgl2/index.html

If tests were successful afterwards, this report might not be available anymore.

@deltakosh
Copy link
Contributor Author

Cc @sebavan as it seems the iridescence tests are failing

@sebavan
Copy link
Member

sebavan commented Apr 9, 2022

Cc @sebavan as it seems the iridescence tests are failing

Can you try to merge master in ? I do not understand why master is not failing :-(

It is like iridescence is not available in your branch but the config is used from master... weird

@sebavan
Copy link
Member

sebavan commented Apr 9, 2022

I updated it to see, still no explanations so far...

@azure-pipelines
Copy link

Please make sure to tag your PR with "bug", "new feature" or "breaking change" tags.

@azure-pipelines
Copy link

Snapshot stored with reference name:
refs/pull/12367/merge

Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/index.html

To test a playground add it to the URL, for example:

https://babylonsnapshots.z22.web.core.windows.net/refs/pull/12367/merge/index.html#WGZLGJ#4600

To test the snapshot in the playground itself use (for example):

https://playground.babylonjs.com/?snapshot=refs/pull/12367/merge#BCU1XR#0

@deltakosh deltakosh merged commit 38d9bae into master Apr 9, 2022
@deltakosh deltakosh deleted the revert-fix branch April 9, 2022 16:47
@sebavan
Copy link
Member

sebavan commented Apr 9, 2022

@RaananW would be great if you could have a look, I wonder why the config was from master but none of the iridescence code was available ? but at least it is all in :-)

@RaananW
Copy link
Member

RaananW commented Apr 11, 2022

@sebavan - I will look into that. The config is taken from the branch it's in, nothing should be taken from master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants