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

PNG Textures do not work #395

Closed
griestopf opened this issue Dec 13, 2021 · 7 comments
Closed

PNG Textures do not work #395

griestopf opened this issue Dec 13, 2021 · 7 comments

Comments

@griestopf
Copy link
Member

griestopf commented Dec 13, 2021

PNG Textures have different effects when used as albedo color input. Sometimes they result in a bluish albedo with otherwise correct lighting calculation, sometimes they result in bright white.
ScreenshotBlenderFusee

@wrestledBearOnce
Copy link
Member

wrestledBearOnce commented Dec 13, 2021

see: #316 (comment)
(Broken png files)

error: dotnet/runtime#55866

newest issue (6 days ago): SixLabors/ImageSharp#1879

conclusion: not our depeartement, waiting for a new version

@wrestledBearOnce
Copy link
Member

wrestledBearOnce commented Dec 13, 2021

TODO:
Update SixLabors from myget (not nuget!) (use latest pre-release with fix): https://www.myget.org/feed/sixlabors/package/nuget/SixLabors.ImageSharp.Drawing

  • Add myget as cache in nuget.json
  • Pull/use latest version
  • Update to latest API
  • Test error with latest version

@wrestledBearOnce
Copy link
Member

wrestledBearOnce commented Dec 15, 2021

@griestopf Please test with: https://github.com/FUSEEProjectTeam/Fusee/tree/feature/395-update-latest-sixlabors-fix-png

or provide test data, please.

Edit: Does not work, needs re-writing of FontImp in order to work properly. New API.

@griestopf
Copy link
Member Author

Blender file:
BrickCubeBlend.zip

Fusee file:
BrickCubeFus.zip
Warning: Setting the absolute path as a program parameter to FUSEE player does not work with the above .fus file since the asset manager currently does not try to find texture assets in the fus-files directory (another bug).

@griestopf
Copy link
Member Author

Action: Wait for next Pre-Release containing the PNG fix and adjust FUSEE to breaking API changes.

@RedImp1470
Copy link
Member

Should be fixed with #466

image

@wrestledBearOnce
Copy link
Member

Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants