Skip to content

Wind parameters from .metaversefile - #1686

Closed
skaljac wants to merge 1 commit into
masterfrom
wind_hair_params
Closed

Wind parameters from .metaversefile#1686
skaljac wants to merge 1 commit into
masterfrom
wind_hair_params

Conversation

@skaljac

@skaljac skaljac commented Oct 28, 2021

Copy link
Copy Markdown
Contributor

I have change everything commented in previous PR.

Comment thread rig.js


function initWindParameters(rig) {
var hairPhysics = rig.app.getComponent('hairPhysics');

@avaer avaer Nov 3, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"Hair physics" is not the best name, since all avatars should have hair physics and not having this component won't change that. This is about wind.

Comment thread rig.js
return;
}

if (!hairPhysics.hasOwnProperty('windDirectionX')) hairPhysics.windDirectionX = 0.0;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't the wind direction be gotten from the component?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Components have a value which you can get look at.

@skaljac

skaljac commented Nov 30, 2021

Copy link
Copy Markdown
Contributor Author

I have created new PR with fresh copy of webaverse:

#1893

That is the reason for closing this PR

All requests are addressed there.

@skaljac skaljac closed this Nov 30, 2021
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

Successfully merging this pull request may close these issues.

2 participants