Skip to content

Improve CI texture image support - #944

Open
LucretiaArc wants to merge 3 commits into
HackerN64:develop/2.4.0from
LucretiaArc:ci-fix
Open

Improve CI texture image support#944
LucretiaArc wants to merge 3 commits into
HackerN64:develop/2.4.0from
LucretiaArc:ci-fix

Conversation

@LucretiaArc

Copy link
Copy Markdown

The current CI texture image implementation has two issues:

  • Additional Python packages are required
  • Only the RGBA16 texel format is supported

Fast-64/fast64#719 adds an option to export CI texture images with texel format information in the file name, and n64graphics already supported CI textures, so this PR solves both problems at once.

Also:

  • Removed the guard in n64graphics against CI word swapped textures.
  • Removed n64graphics_ci, which wasn't used anywhere. It does support palette reduction using exoquant, which can't be said of the n64graphics implementation, so that might be worth implementing for n64graphics in future.

@gheskett gheskett left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait n64graphics already supported CI this entire time? What on earth was that script for lol

@arthurtilly

Copy link
Copy Markdown
Collaborator

i always used binpng for ci textures. okay but literally who has ever used a non-rgba16 ci texture is that a thing

@gheskett

gheskett commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

ia16 apparently 🤷

@arthurtilly

Copy link
Copy Markdown
Collaborator

who has ever used ia16 ever

@gheskett

gheskett commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

People that like transparent gradients I guess

@arthurtilly

Copy link
Copy Markdown
Collaborator

ia8 has 4 bit alpha

@gheskett

gheskett commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

Yeah but 4 bits of not alpha isn't very smooth

@arthurtilly

Copy link
Copy Markdown
Collaborator

still, i cant think of any situation where you wouldnt either wanna multitex with rgba or could just use i8

Comment thread Makefile
@LucretiaArc
LucretiaArc requested a review from gheskett August 10, 2026 06:53
Comment thread Makefile
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.

3 participants