BabylonJS / Exporters

Exporters for Babylon.js and gltf file formats
Other
597 stars 307 forks source link

Log file data not showing in tab unless window is oversized ... #1097

Closed kmerino closed 10 months ago

kmerino commented 11 months ago

I am using the latest release of exporter (Release20230731.2) and am seeing issues with the display of the log file data. When the Export window is at default size, no log file is displayed. If the window is scaled up considerably, the data frame starts to show. See screen captures.

I tried it in the latest versions of Max 2024 and 2020 and it behaves the same way. Win 10 - 22H2. Screenshot 2023-08-13 060152 Screenshot 2023-08-13 060238

bghgary commented 10 months ago

I'm guessing there is something up with DPI. What is your DPI setting?

kmerino commented 10 months ago

I am running 3 monitors 1920x1080 @60Hz using Nvidia RTX 2080 Super.

bghgary commented 10 months ago

Hmm, maybe it's the 3 monitors that is causing it. Can you see if it repros with just one monitor with your setup?

kmerino commented 10 months ago

I can try that, but I went back to a previous build I had been using (Release20221031.2) w/ Max 2022 and it works as it use to. Screenshot 2023-08-14 172030

kmerino commented 10 months ago

Going to single monitor did not change the display of the log data (Release20230731.2 | Max 2024).

bghgary commented 10 months ago

Interesting, that must mean a PR broke it. I suspect it might be this one: https://github.com/BabylonJS/Exporters/pull/1089. @kelilife Do you have any thoughts?

PatrickRyanMS commented 10 months ago

@bghgary, here is a file to test with where nothing shows on the log: https://github.com/BabylonJS/Exporters/files/12178219/riggedMeshTest.zip

SergioRZMasson commented 10 months ago

@bghgary , I was able to verify that the issue was introduced by this PR: https://github.com/BabylonJS/Exporters/pull/1089, I will be working to see what the problem was.

SergioRZMasson commented 10 months ago

Found the issue, the size of the log tab was set to zero by PR: #1089 , Just created a PR that reverts that particular change:

https://github.com/BabylonJS/Exporters/pull/1101

kmerino commented 10 months ago

Thanks to all involved for your efforts on resolving this!