FlailingFog / KK-Blender-Porter-Pack

Plugin pack for exporting Koikatsu characters to Blender.
310 stars 29 forks source link

Vertex on upper/bottom lip merged together #82

Closed poisenbery closed 1 year ago

poisenbery commented 2 years ago

Screenshot (366)

Center vertex on lips seems to be merged together. Relatively consistent error. I'm pretty sure it's happening on all of my models but I need to test further to see. EDIT: It's on all of my models. I tried importing with a larger scale size to see if it was a merge by distance thing, but no luck.

naminode commented 2 years ago

Just noticed this. Not sure how the exporter works. I guess the only solution for now is to unsigned the mixed vertices from the vertex group.

Mouth_Up image

Mouth_Low image

Mouth_R image

MediaMoots commented 2 years ago

What blender version?

FlailingFog commented 2 years ago

I see what you mean for mouthup and mouthlow. Happens for 2.93 and 3.0.0 on a CATS import or mmd tools file menu import. For Mouth_R though, the weight for the stray vertices are zero, so it should be okay. They're just assigned to the group for some reason

poisenbery commented 2 years ago

What blender version?

I tested on 3.0 and I believe the prior LTR to make sure it wasn't a 3.0 issue, so the same as @FlailingFog

MediaMoots commented 1 year ago

Is this fixed?

poisenbery commented 1 year ago

With the CATS import, there was a workaround where the scale could be set to something pretty high and it would mitigate the issue, but now that CATS is out of the import process, scale can't be manually set before importing and now this card has the problem again. Screenshot (904) KK_109194

It's not on all cards though. Screenshot (905)

It's caused by the "Fix Seams" merging the verts together. Screenshot (906)

EDIT: Looks like the vertex indices are 1096 and 1200. I tested across a couple of models [non-headmodded] and these seem to be consistent for the lip vertices that are closest together.

poisenbery commented 1 year ago

@FlailingFog @MediaMoots It might be a better idea to "Select all by trait - Non Manifold" and merge with those vertices, instead of performing the operation on the entire mesh.

MediaMoots commented 1 year ago

@poisenbery Just checking, is this fixed?

poisenbery commented 1 year ago

AFAIR, yes

On Sat, Dec 17, 2022, 00:46 MediaMoots @.***> wrote:

@poisenbery https://github.com/poisenbery Just checking, is this fixed?

— Reply to this email directly, view it on GitHub https://github.com/FlailingFog/KK-Blender-Porter-Pack/issues/82#issuecomment-1356123331, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASWU22DGKXUUAPIEUVF4THLWNV4VLANCNFSM5KIBCETA . You are receiving this because you were mentioned.Message ID: @.***>

MediaMoots commented 1 year ago

Cool! Thanks for your help!