-
Notifications
You must be signed in to change notification settings - Fork 3
Feature #52: Increase MAX_SNAPSHOT_IMAGE_SIZE to 32768 #53
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
base: dev
Are you sure you want to change the base?
Conversation
This commit increases the MAX_SNAPSHOT_IMAGE_SIZE constant in llviewerwindow.h from 8192 to 32768 (2^15). This change addresses user requests for the ability to capture snapshots at extremely high resolutions, primarily for virtual photography and large-format printing, as outlined in Issue #52. **Motivation:** To provide advanced users and virtual photographers with the capability to generate ultra-high-resolution imagery within Aperture Viewer. **Risks and Considerations:** - **High Memory Usage:** Rendering at resolutions approaching 32768x32768 will consume substantial VRAM and system RAM (approx. 4GB for the framebuffer alone at 32k RGBA, plus viewer/OS overhead). - **Stability Concerns:** There is a significant risk of viewer crashes, system slowdowns, or out-of-memory errors, especially on systems not equipped with high-end GPUs and ample RAM. The previous limit of 8192 (originally 8096) was set due to observed instability at higher values. - **Performance Impact:** Snapshot generation at these sizes will be very time-consuming. - **Hardware Limitations:** Individual GPU capabilities may impose lower practical or hard limits on render target dimensions. **Disclaimer:** This feature is considered experimental. Users attempting to utilize snapshot dimensions near the new maximum do so entirely at their own risk. Aperture Viewer development is not responsible for any resulting viewer crashes or system instability. It is highly recommended to save all work before attempting such snapshots and to test incrementally. **Testing (Initial by Committer):** - Snapshot UI (e.g., "Save to Disk" floater) correctly reflects the new maximum allowable dimension when custom size is selected. - Basic snapshot functionality tested at moderate high resolutions (e.g., 10240x10240) on a capable system to ensure the mechanism isn't immediately broken by the constant change itself. Further extensive testing regarding stability limits across different hardware and scenarios is required on this branch as outlined in Issue #52. Related: #52
Test Case ID: REG-PR53-HIRES-SNAPSHOT-VERIFICATION Test Category: Functional, Stress, Performance, Visual Fidelity Preconditions:
|
Eira Juliesse Findings: Preset Used: Screen Space Reflections 8192 x 4367 snapshot resolution
12287 x 6550 snapshot resolution
16384 width snapshot resolution
Observations:
|
Eira Juliesse Findings: Preconditions PR: Feature #52: Increase MAX_SNAPSHOT_IMAGE_SIZE to 32768 #53 10240x5404
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Aperture Viewer.log has this error repeating 2025-05-30T12:44:49Z WARNING # llimage/llimage.cpp(705) LLImageBase::allocateData : LLImageBase::allocateData: bad size: 429293568
This PR implements Feature Request #52, increasing the
MAX_SNAPSHOT_IMAGE_SIZE
constant inllviewerwindow.h
from8192
to32768
.Purpose:
To allow users, particularly virtual photographers, to capture snapshots at extremely high resolutions (up to 32768x32768 pixels) for purposes such as large prints or detailed post-processing.
Commit Included:
MAX_SNAPSHOT_IMAGE_SIZE
.Important Considerations & Risks (as detailed in Issue #52):
USER DISCLAIMER:
This is an experimental feature intended for advanced users with very high-end hardware. Users attempting to render snapshots at extreme resolutions (approaching 32768x32768) do so ENTIRELY AT THEIR OWN RISK. Aperture Viewer development is not responsible for any viewer crashes, system instability, data loss that may occur or any other software or hardware consequence. It is strongly advised to save all work and test with incrementally larger resolutions.
Testing Performed by Committer (Locally on this branch):
Further Testing Required (On this branch, before merging to
dev
):Related Issue: Closes #52