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

MaterialXView clamps requested window dimensions #2021

Open
kwokcb opened this issue Sep 18, 2024 · 1 comment
Open

MaterialXView clamps requested window dimensions #2021

kwokcb opened this issue Sep 18, 2024 · 1 comment

Comments

@kwokcb
Copy link
Contributor

kwokcb commented Sep 18, 2024

Issue

It seems if you set the screen dimensions on MaterialXView for interactive and command line they are sometimes set properly.
It seems that if the size is not too small (like 64) and not too large(1024) then it will work.

Example

This never seems to work:

MaterialXView  --screenWidth 1024 --screenHeight 1024

versus

MaterialXView  --screenWidth 1024 --screenHeight 1024 --captureFileName x.png

512 x 512 works, but 64 x 64 does not.

From brief testing it seems the size used to capture with is fixed and if the screen capture size somehow fits with this size it will capture properly. Otherwise the screen size is ignored.

This was only tested on Windows at SHA: e0d72e7

@kwokcb kwokcb changed the title MaterialXView capture dimensions not set properly (Regression) MaterialXView capture dimensions not set properly (Regression?) Sep 18, 2024
@jstone-lucasfilm jstone-lucasfilm changed the title MaterialXView capture dimensions not set properly (Regression?) MaterialXView capture dimensions not set properly Oct 31, 2024
@jstone-lucasfilm
Copy link
Member

@kwokcb I believe this is a limitation of the Windows environment, rather than of MaterialXView/NanoGUI, where Windows will clamp requested sizes for application windows that are too large or too small to display.

On a Windows PC with a 5K monitor, requesting MaterialXView --screenWidth 1024 --screenHeight 1024 works perfectly, but I've seen Windows reject the same request on a laptop with a smaller display.

Given that background, this may not be an issue that we can address within the MaterialX project, but I'll leave this open in case you have thoughts on how to improve this.

@jstone-lucasfilm jstone-lucasfilm changed the title MaterialXView capture dimensions not set properly MaterialXView clamps requested window dimensions Dec 7, 2024
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

No branches or pull requests

2 participants