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

Added an effect named which involves native ui picker #16

Merged
merged 1 commit into from
Oct 7, 2020

Conversation

distroinfinity
Copy link
Contributor

@distroinfinity distroinfinity commented Oct 7, 2020

Addresses issue #11 . You can simply choose from 4 glasses options available in Picker UI to try one at a time.

Copy link
Member

@lopeselio lopeselio left a comment

Choose a reason for hiding this comment

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

files ahve been compressed

Comment on lines +1 to +328
"byteStride": 16,
"name": "floatBufferViews",
"target": 34962
}
],
"buffers": [
{
"byteLength": 26352,
"uri": "scene.bin"
}
],
"materials": [
{
"doubleSided": true,
"emissiveFactor": [
0,
0,
0
],
"name": "Bluelen1Mtl",
"pbrMetallicRoughness": {
"baseColorFactor": [
0.0111261,
0.086901300000000001,
0.493616,
1
],
"metallicFactor": 0,
"roughnessFactor": 0
}
},
{
"doubleSided": true,
"emissiveFactor": [
0,
0,
0
],
"name": "Middle1Mtl",
"pbrMetallicRoughness": {
"baseColorFactor": [
0.891262,
0.89938499999999999,
0.89938499999999999,
1
],
"metallicFactor": 0,
"roughnessFactor": 1
}
},
{
"doubleSided": true,
"emissiveFactor": [
0,
0,
0
],
"name": "Redlen1Mtl",
"pbrMetallicRoughness": {
"baseColorFactor": [
0.56049899999999997,
0.016988099999999999,
0.0077510299999999999,
1
],
"metallicFactor": 0,
"roughnessFactor": 0
}
}
],
"meshes": [
{
"primitives": [
{
"attributes": {
"NORMAL": 1,
"POSITION": 0,
"TANGENT": 2
},
"indices": 3,
"material": 0,
"mode": 4
}
]
},
{
"primitives": [
{
"attributes": {
"NORMAL": 5,
"POSITION": 4,
"TANGENT": 6
},
"indices": 7,
"material": 2,
"mode": 4
}
]
},
{
Copy link
Member

Choose a reason for hiding this comment

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

reviewed the GLTF

Copy link
Member

@lopeselio lopeselio left a comment

Choose a reason for hiding this comment

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

requirements met as requested

Comment on lines +329 to +393
"primitives": [
{
"attributes": {
"NORMAL": 9,
"POSITION": 8,
"TANGENT": 10
},
"indices": 11,
"material": 1,
"mode": 4
}
]
}
],
"nodes": [
{
"children": [
1
],
"name": "RootNode (gltf orientation matrix)",
"rotation": [
-0.70710678118654746,
-0,
-0,
0.70710678118654757
]
},
{
"children": [
2
],
"name": "RootNode (model correction matrix)"
},
{
"children": [
3,
4,
5
],
"name": "3dGlass.obj.cleaner.materialmerger.gles"
},
{
"mesh": 0,
"name": ""
},
{
"mesh": 1,
"name": ""
},
{
"mesh": 2,
"name": ""
}
],
"scene": 0,
"scenes": [
{
"name": "OSG_Scene",
"nodes": [
0
]
}
]
}

Copy link
Member

Choose a reason for hiding this comment

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

meets requirements

@lopeselio lopeselio added ready-to-merge requirements met and able to be merged approved requested changes have been approved labels Oct 7, 2020
@lopeselio lopeselio merged commit 91db8d0 into ACM-VIT:master Oct 7, 2020
@lopeselio lopeselio added hacktoberfest-merged and removed approved requested changes have been approved labels Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-merged ready-to-merge requirements met and able to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants