Untested Character hair customization (+ basics for expansion)

[Kibbelz]

I'm glad to see this feature is getting some stability. I've written one myself for Nightmare. I think a really cool feature you could add is a "remove hair render" checkbox for helmets and other hair based equipment paperdolls in the item editor. This stops layering related issues with various hairstyles and helmets clashing when rendering them both ingame.

 

 

[Cheshire]

18 minutes ago, Kibbelz said:

I'm glad to see this feature is getting some stability. I've written one myself for Nightmare. I think a really cool feature you could add is a "remove hair render" checkbox for helmets and other hair based equipment paperdolls in the item editor. This stops layering related issues with various hairstyles and helmets clashing when rendering them both ingame.

 

 

I've written it so that the hair replaces a configurable paper doll layer. If no paper doll render hair. 

 

There's some jank I'd like to clean up. But that's for later. 

[Processinho]

Does anyone have a build already made with this patch? Honestly, I can't understand how to apply this to the client.

[Justn]

11 hours ago, Processinho said:

Does anyone have a build already made with this patch? Honestly, I can't understand how to apply this to the client.

What errors are you getting?

[Processinho]

12 hours ago, Justn said:

What errors are you getting?

For now, none because I don't know how to install any patches. I saw the tutorial right here on the forum but I stuck it in the first step.

[AisenArvalis]

18 minutes ago, Processinho said:

For now, none because I don't know how to install any patches. I saw the tutorial right here on the forum but I stuck it in the first step.

 

Get the Github Desktop Application.
Clone the Intersect Repo - https://docs.freemmorpgmaker.com/dev/start/github.html
From here, you have your own copy of the engine source.

Now you can use the patch method
https://docs.freemmorpgmaker.com/dev/sharing/applying.html

[Cheshire]

I feel like this is more a discussion for the questions and answers board than this topic.

[buu]

@Joyce would this work with the new animation frames? Can we do "Hair_ idle" for example?

[wishy]

11 minutes ago, buu said:

@Joyce would this work with the new animation frames? Can we do "Hair_ idle" for example?

It does

[Cheshire]

4 hours ago, buu said:

@Joyce would this work with the new animation frames? Can we do "Hair_ idle" for example?

Yes, the first post even says and shows it does.

[buu]

8 hours ago, Joyce said:

Yes, the first post even says and shows it does.

Yea I might have skimmed over it. Thanks

[Shigetsu]

Hey,

It seems like this patch doesn't work with latest update from the dev branch. Could you please update it?

 

Spoiler

Applied it on a new branch with no edits.

 

[Cheshire]

Shouldn't be too hard to fix with a three way apply. I'll have a look later.

 

Ironically it's my own changes that broke it. 😜

 

EDIT: Updated for Build 0.6.2.419

 

Download

[kies~]

Please upgrade to a new version ... 21

[Cheshire]

Does that mean it doesn't work?

[kies~]

Yes, The patch is not applying, I did it manually line by line and it still didn't work.

 

[Cheshire]

Should still apply fine if you do git apply -3 <patchname>

 

Problem seems to be that the 6.2 branch changed, rather than any actual code breaking.

[kies~]

Now this appeared

 

[Cheshire]

Yup. Should work though, did on my end anyway.

 

The problem is that it's not the same git branch anymore now that they changed the GitHub structure. So it's telling you it's missing commits or seeing commits it did not expect.

 

Once 6.2 becomes stable or something happens code wise I'll fix the patch. But if a few meaningless messages show up and it still works I'd call that functional for the time being.

 

 

[Engelion]

Hello, I use 6.2, is work with animation attack?

[Beefy Kasplant]

47 minutes ago, Engelion said:

Hello, I use 6.2, is work with animation attack?

 

Try it 

[Engelion]

9 minutes ago, Beefy Kasplant said:

 

Try it 

If I ask is because I need know for make my hair sprites or not lol.

[Cheshire]

It works with them yes. As the first post states.

[Vallorz]

Heya, I am also getting patch failed for this patch on build 6.2 429 however I was not able to bypass this with "git apply -3 name.patch" as when I go to compile it in visual studio, I get a bunch of errors as follows: 

 

I try to fixed the merge conflicts by removing the old code to replace it with your new code from the patch as shown here: 

Quote

<<<<<<< ours
        protected Dictionary<string, IAsset> mFaceDict = new Dictionary<string, IAsset>();
=======
        protected Dictionary<string, GameTexture> mHairDict = new Dictionary<string, GameTexture>();

        protected Dictionary<string, GameTexture> mFaceDict = new Dictionary<string, GameTexture>();
>>>>>>> theirs

 

but once I get rid of all the merge conflicts, I end up with this error when trying to compile: 

 

Any help would be appreciated <3

[Cheshire]

This was for a very old build of 6.2, I'll update it when that officially releases. Until then it's supported for the build listed on the first post.

 

Otherwise I'd be updating it every other day again. Lol