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

Interesting display issue #1

Open
theo-armour opened this issue Sep 9, 2019 · 1 comment
Open

Interesting display issue #1

theo-armour opened this issue Sep 9, 2019 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@theo-armour
Copy link
Member

@macumber

I am building a new version of the Spider gbXML Viewer designed to be embedded in the next version of OpenStudio. This requires creating a file titled "embeddable_gbxml_editor.html " that contains all the source code in a single file. The script used to create the file is called "Fetch Assemble Save". Everything is going according to plan and working as expected. Expect for a single huge deficiency - as will be displayed below. Note that at no time is an error or warning displayed i the JavaScript console.

Fetch Assemble Save - the file that builds and tests embeddable_gbxml_editor.html . All good. Note windows are OK.

image


Same gbXML file imported into OpenStudio 2.8. All good.

image


Same file imported to OpenStudio using embeddable_gbxml_editor.html as created above. Note that surfaces with openings are not displaying,

image


Now we are in the Chromium R45 browser running a standalone version of embeddable_gbxml_editor.html. Note windows are OK.

image


Again Chromium 45, but now with the same Fetch Assemble Save running in an iframe. Note again the absence of surfaces with openings.

image


The code required to build surfaces with openings is complicated. It requires conjugating quaternions and the like. So bugs like this are understandable. The issue for me was trying to find out whether the problem came from my coding or from issues with the browser. The above is a record of my process in sorting this out.

Now let's get this thing working properly!

@theo-armour theo-armour added bug Something isn't working good first issue Good for newcomers labels Sep 9, 2019
@theo-armour
Copy link
Member Author

OMG: Kittens, ponies and unicorns have all arrived!

image

It looks like being very careful about removing any let and const anywhere in the process - even with "use strict" on - may be the answer. Fingers crossed more testing to be done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant