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

[WIP] Issue 1647 rendered banner poi #1735

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DragonDev1906
Copy link

@DragonDev1906 DragonDev1906 commented Mar 2, 2020

Working on #1647

I have implemented code to render a banner using it's NBT data (see first commit).
When trying to use it from genPOI I have the following problems / questions:

  • For rendering we need access to a Textures object. This is never created for genPOI. Is the creation I've used ok?
  • Is the Textures object thread save or do we need a lock when it is used (only load_image is used)
  • Can we just pass the Textures object as an additional parameter to the threads?
  • What do you think is the best way for the filenames. I was thinking of having a global counter but I'm not sure where to place it. I haven't done much multiprocessing in python yet.

Note: Because the render_banner function requires a Textures object placing the code in the filter itself would probably not work (no access to the config)

The second commit contains some of my thoughts (partially commented). Maybe you can help me with the questions above.

The config I used (including the Filters): config.txt

Example for the Banner Rendering:
2020-03-02_22:15:27

Jens added 2 commits March 2, 2020 19:24
- renders a banner using it's base color and the NBT data
- supports rendering the backside, too
- currently not used and only tested manually
@Tirpitz93 Tirpitz93 deleted the issue-1647-rendered-banner-poi branch March 31, 2020 21:46
Atten007 added a commit to Atten007/Minecraft-Overviewer that referenced this pull request Apr 12, 2022
Atten007 added a commit to Atten007/Minecraft-Overviewer that referenced this pull request Apr 12, 2022
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

1 participant