The Wayback Machine - https://web.archive.org/web/20220411230732/https://github.com/SFTtech/openage/issues/1371
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

Speed up converter texture packing with Cython #1371

Open
3 tasks
heinezen opened this issue Jan 22, 2021 · 0 comments
Open
3 tasks

Speed up converter texture packing with Cython #1371

heinezen opened this issue Jan 22, 2021 · 0 comments
Labels
assets good first issue improvement python
Projects

Comments

@heinezen
Copy link
Member

@heinezen heinezen commented Jan 22, 2021

Required skills: Cython

Difficulty: Medium

Animation frames from AoE2 graphics files are packed into a texture atlas by the openage converter. We use bin packing to find the optimal arrangement (= smallest atlas dimensions) of the frames in the atlas. Bin packing becomes computationally intensive if a lot of frames are packed (looking at you Definitive Edition), so we want to optimize the algorithm as much as possible.

The goal of this issue is to speed up the bin packing implemented here using Cython. The most important methods to cythonize are pack() and factor(), since they are responsible for the packing logic. Ideally, these functions should contain as little Python overhead as possible.

Additional optimization might be possible such as:

  • Currently pack() gets passed a list of frame objects, but technically only width, height and index of a frame are needed.
  • Most of the games have a fixed number of frames per animation (e.g. 50 frames for a unit animation in AoC). These could be hardcoded as lookups into factor()
  • texture_merge.pyx, the function initiating the binpacking, could also use optimization for merging the individual frames into the atlas using the packer settings.

You can test your changes with the singlefile media converter.

Further Reading

@heinezen heinezen added improvement python assets good first issue labels Jan 22, 2021
@TheJJ TheJJ added this to conversion in convert Jan 22, 2021
@heinezen heinezen added the hacktoberfest label Sep 30, 2021
@heinezen heinezen removed the hacktoberfest label Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assets good first issue improvement python
1 participant