Skip to content

Normal map color wheel not working as intended #3015

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

Closed
Louis-Julien opened this issue Oct 20, 2021 · 6 comments
Closed

Normal map color wheel not working as intended #3015

Louis-Julien opened this issue Oct 20, 2021 · 6 comments

Comments

@Louis-Julien
Copy link

Problem:
When selecting a color with the color picker on the image of a normal map sphere, produced outside of Aseprite, the colors selected do not match the one displayed on the color picker. The color selected on the color wheel seem to not match the desired orientation when tested in Unity with default 3D lights.
I tried to use the normal color wheel to select colors between 2 already known and working colors, but it doesn't work at all.

Steps to reproduce:

  • import a normal map sphere (the one from the wikipedia page for example) in aseprite
  • select a color with the color picker, preferably not at the center
  • with the normal map palette selected, try to select the same color by clicking at the spot highlighted with a small circle on the color wheel.
  • The new color is different from the previous one.

This capture shows the difference between the 2 colors, the first one (in the red circle), actually on the image, and the second, produced with the color wheel:
https://gyazo.com/e115b949781c6c0b77a3ca19bfe563fb

As per my tests, the color produced with Aseprite normal color wheel are not correct when the texture is imported in a program to light them.

As an added request, while testing some other programs to edit normal map I found one that shows a wireframe 3D square around the color selected on the wheel, clearly giving the information of the direction of the normal. That would be very helpful to wrap your head around what a color will really do when lighted.

If you need any feedback or to test fixes, don't hesitate to contact me.

Aseprite and System version

  • Aseprite version: 1.2.29-x64, steam version
  • System: W10
@dacap dacap added this to the v1.2.31 milestone Oct 29, 2021
@dacap dacap self-assigned this Oct 29, 2021
@dacap
Copy link
Member

dacap commented Oct 29, 2021

I'll check this bug later, maybe related to #2900 where we could just simplify the color selector and replace all color selectors with a (png) image and render that image.

@Louis-Julien
Copy link
Author

Loading an external png to use as a palette could solve the issue, as it's what I do but it's pretty finicky to go from one file to another, even with split display. As long as the selected color is highlighted on the png, that would be awesome for us.

@dacap dacap assigned Gasparoken and unassigned dacap Nov 15, 2021
@Louis-Julien
Copy link
Author

The issue is preventing us from making significant progress in our current project, but we could provide help to resolve it. With some indications from your part, one of our dev @as_csauveur could look into the issue and see if there is a solution. Would that be okay with you?

@dacap dacap assigned dacap and unassigned Gasparoken Nov 30, 2021
@dacap
Copy link
Member

dacap commented Nov 30, 2021

I'm reassigning this issue to me, in case you @Louis-Julien can check some of the code, please check:

@Louis-Julien
Copy link
Author

Thank you, I've forwarded the links to our dev, I'll keep you in touch.

@cs-altshift
Copy link
Contributor

Thanks for the links into the codebase. I'm starting looking at this issue today.

@dacap dacap modified the milestones: v1.2.31, v1.2.32 Jan 12, 2022
@dacap dacap modified the milestones: v1.2.33, v1.2.34 Feb 8, 2022
@dacap dacap modified the milestones: v1.2.34, v1.2.35 Mar 14, 2022
@dacap dacap modified the milestones: v1.2.35, v1.x-bugs Jun 1, 2022
@dacap dacap removed their assignment Sep 15, 2022
@dacap dacap modified the milestones: v1.x-bugs, v1.3.0 Nov 16, 2022
@dacap dacap closed this as completed in 20902e3 Nov 16, 2022
@dacap dacap self-assigned this Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants