Skip to content

Fix item export resolution not scaling with size parameter#36

Merged
rubensworks merged 3 commits into
master-26from
copilot/bug-fix-image-resolution-issue
Apr 8, 2026
Merged

Fix item export resolution not scaling with size parameter#36
rubensworks merged 3 commits into
master-26from
copilot/bug-fix-image-resolution-issue

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 8, 2026

  • Understand root cause: GuiRenderer.prepareItemElements() uses 16 * guiScale for item atlas slot size; existing code only scaled blit position via pose matrix, not atlas resolution
  • Fix: temporarily set windowRenderState.guiScale = scaleImage/16 before rendering each item/fluid so the atlas rebuilds at scaleImagepx resolution; use natural 16×16 GUI unit draw size (no pose scaling)
  • Code review improvements: rename newGuiScaleexportGuiScale, capture windowRenderState once before loops
  • Verify build compiles successfully

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 8, 2026

Coverage Report for CI Build 24142281603

Coverage decreased (-0.1%) to 7.568%

Details

  • Coverage decreased (-0.1%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 32 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

32 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
org/cyclops/iconexporter/client/gui/ScreenIconExporter.java 32 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 314
Covered Lines: 26
Line Coverage: 8.28%
Relevant Branches: 56
Covered Branches: 2
Branch Coverage: 3.57%
Branches in Coverage %: Yes
Coverage Strength: 0.32 hits per line

💛 - Coveralls

Copilot AI changed the title [WIP] Fix item size not increasing actual image resolution Fix item export resolution not scaling with size parameter Apr 8, 2026
Copilot stopped work on behalf of rubensworks due to an error April 8, 2026 14:13
Copilot AI requested a review from rubensworks April 8, 2026 14:13
Copilot AI and others added 2 commits April 8, 2026 14:46
…ndowRenderState once

Agent-Logs-Url: https://github.com/CyclopsMC/IconExporter/sessions/6001cadd-a522-48b8-8f45-57605a0bf0eb

Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>
@rubensworks rubensworks marked this pull request as ready for review April 8, 2026 15:07
@rubensworks rubensworks merged commit a33f12e into master-26 Apr 8, 2026
3 checks passed
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.

Providing item size doesn't increase actual image resolution

3 participants