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

Fix import logic for VRAM compressed texture formats #42118

Open
wants to merge 1 commit into
base: 3.x
Choose a base branch
from

Conversation

DrMoriarty
Copy link
Contributor

@DrMoriarty DrMoriarty commented Sep 16, 2020

Bugsquad edit: Fixes #38040, fixes #35851.

Compressed stream texture now saves right image dimensions for every compression format.
Also it allows to compressor resize image on it's side to make it more optimized.
So ETC1 format becames smaller then before. Pvrtc format always saves square texture.

Compressed stream texture now saves right image dimensions for every compression format.
Also it allows to compressor resize image on it's side to make it more optimized.
So ETC1 format becames smaller then before. Pvrtc format always saves square texture.
@akien-mga
Copy link
Member

I force pushed a rebase as this included a CHANGELOG update unrelated to this PR (due to me force-pushing the CHANGELOG to fix the 3.2.3 release date).

I also amended the commit message to be clearer about what it does, and removed the references to the issues which are handled in #38076.

@akien-mga akien-mga changed the title Ultimate fix for PVRTC (part 2) Fix import logic for VRAM compressed texture formats Sep 23, 2020
@DrMoriarty DrMoriarty requested a review from a team as a code owner March 12, 2021 12:26
Base automatically changed from 3.2 to 3.x March 16, 2021 11:11
@akien-mga akien-mga modified the milestones: 3.2, 3.3 Mar 17, 2021
@akien-mga akien-mga modified the milestones: 3.3, 3.4 Mar 26, 2021
@fire
Copy link
Member

fire commented Apr 26, 2021

Are you able to review your changes.

Master added etcpak and some things have been shifted.

Edited:

I forgot this was 3.x. Please ignore.

@Chaosus Chaosus modified the milestones: 3.4, 3.5 Nov 8, 2021
@akien-mga akien-mga removed this from the 3.5 milestone Jul 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants