Assertion failed: [Viewport._setContentBounds] bounds.height must be greater than 0
404mike opened this issue · comments
UV version:
universalviewer@4.0.16
I'm submitting a:
- bug report => please fork one of these codesandbox examples with a repro of your issue and include a link to it below
- feature request => please use the user stories repo
- support request => Please do not submit support requests here, use stackoverflow
Current behavior:
UV throws Assertion failed: [Viewport._setContentBounds] bounds.height must be greater than 0 errors on version 4.0.16. See https://codesandbox.io/s/uv-4-0-16-q5tj5k
CORS error not related to UV.
Expected behavior:
It looks like this issue was introduced in 3a01c62 as the previous commit works fine, see https://codesandbox.io/s/uv-4-0-6-4j863r?file=/index.html
Steps to reproduce:
Related code:
// insert any relevant code here
Other information:
Thanks for pinning this down, @404mike! Given that there aren't many actual code changes in the commit you highlighted, I suspect this is actually a problem introduced in either manifold 2.0.18 or manifesto 4.2.15, so we might have to investigate those projects and fix the problem over there. And actually, since manifesto's requirement was changed from the broad ^4.2.14 to the pinned 4.2.15, I suppose there's also a possible scenario where a bug fixed in a later version of manifesto is reintroduced by forcing a rollback here.
@edsilv, what do you think? I'm unfortunately very short on time this week due to a conference and can't dig much deeper right now, but if any of my theories seem particularly likely or unlikely, let us know and it might help direct the course of further investigation.