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

Remove redundancy caused by the back face of cards. #39

Closed
drwhut opened this issue Feb 16, 2021 · 0 comments
Closed

Remove redundancy caused by the back face of cards. #39

drwhut opened this issue Feb 16, 2021 · 0 comments
Labels
feature New feature or request
Milestone

Comments

@drwhut
Copy link
Owner

drwhut commented Feb 16, 2021

Is your feature request related to a problem? Please describe.
The way cards are imported is such that each card has one texture, with both the front and back faces on the texture. However, the fact that most card packs in general almost always have the same back face, means that the back face is being copied unnecessarily in every texture file. If the back face can't compress well, this can lead to huge folder sizes!

Describe the solution you'd like
Change how cards are imported so that each texture is just the front face of the card - then, in the config.cfg file, the back face can be set with a new property:

[*]
; Set the back face of every card in the pack.
; The BackFace.png file is also in the folder.
back_face = "BackFace.png"

[BackFace.png]
; Optional, but if this wasn't here, you would
; have another card with both the front and
; back faces being this texture.
ignore = true

This way, the back face only takes up one file, and the redundancy is gone! This also means that the front and back faces could potentially use different formats, so e.g. the front face can be a vector image while the back face is a pixel image.

Describe alternatives you've considered
Keeping the current system as is, but I can't sleep at night knowing the cards/ folder is twice the size it needs to be!

Additional context
Ideally, the new cards shouldn't have two separate meshes for each face, but rather one mesh with two surfaces that can have two separate materials.

@drwhut drwhut added the feature New feature or request label Feb 16, 2021
@drwhut drwhut added this to the v0.1.0 milestone Feb 16, 2021
@drwhut drwhut closed this as completed in 29ceaf2 Mar 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant