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

chore: Wrap VRCSDK dependencies with AAO_VRCSDK3_AVATARS #627

Merged
merged 19 commits into from
Oct 25, 2023

Conversation

kaikoga
Copy link
Contributor

@kaikoga kaikoga commented Oct 19, 2023

Related Issue: #607

This PR wraps everything VRCSDK specific inside #if AAO_VRCSDK3_AVATARS and remove any VRCAvatarDescriptor usage outside them, until

This PR assumes all components would be supported without VRCSDK, excluding:

  • MergePhysBone: depends on PhysBone
  • Clear Endpoint Position: depends on PhysBone

@kaikoga kaikoga marked this pull request as ready for review October 22, 2023 14:02
Copy link
Owner

@anatawa12 anatawa12 left a comment

Choose a reason for hiding this comment

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

ありがとうございます

Editor/ObjectMapping/ObjectMappingContext.cs Outdated Show resolved Hide resolved
Editor/AvatarGlobalComponentEditorBase.cs Show resolved Hide resolved
Editor/Processors/TraceAndOptimize/AnimatorParser.cs Outdated Show resolved Hide resolved
Editor/Processors/TraceAndOptimize/AutoFreezeBlendShape.cs Outdated Show resolved Hide resolved
Editor/Utils/Utils.cs Outdated Show resolved Hide resolved
Test~/TestUtils.cs Outdated Show resolved Hide resolved
@anatawa12
Copy link
Owner

anatawa12 commented Oct 23, 2023

conflict resolveしました

@kaikoga
Copy link
Contributor Author

kaikoga commented Oct 23, 2023

対応していきます

@kaikoga
Copy link
Contributor Author

kaikoga commented Oct 23, 2023

we found errors in commit message of 796fe80bf0e1eaf1dd39e3cd92ac0e1990a6fbbf
  unknown header type: cosme

cosme なかったかーーーー

Runtime/AvatarTagComponent.cs Outdated Show resolved Hide resolved
Copy link
Owner

@anatawa12 anatawa12 left a comment

Choose a reason for hiding this comment

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

ありがとうございます

@anatawa12 anatawa12 merged commit 4bf2125 into anatawa12:master Oct 25, 2023
5 checks passed
@kaikoga kaikoga deleted the remove_vrcsdk_deps branch October 26, 2023 19:44
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.

None yet

2 participants