-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Calculate height of section levels #31
Calculate height of section levels #31
Conversation
Please note main.test.js and flags.test.js have been temp removed to allow push. Need to resolve.
Given the height of the tallest level in IFC file, we estimate a camera height location considering by also utilizing a screen size coefficient
Heya, I'm happy to review. While you still have tests excluded please leave in draft. I just flipped it. Are there sample screens? |
Thanks! No graphics in this one yet, just calculating coordinates. |
Got it checked out with I uncomment calSecLevels and run:
But no output to out.ifc. |
The script doesn't output an ifc. It outputs a simple array of height values. To see the values, please uncomment line 31: and run: for example:
|
Hi Mohammed. Sorry, I lost track. Everything's running locally for me, though you have the existing tests deleted. Just remove them from this PR. What were you blocked on for testing? |
My block is I can't run main.test.js and flags.test.js, and hence cannot push the code without removing them. |
Ah, I see. Not sure how to fix this. The version of ifclib that's being referenced is somehow being confused. The main branch is ahead of your branch and has fixed those reference problems. I tried updating tool/package.json to depend on the upstream version, 5.3.3, and then doing yarn install, but that doesn't seem to fix it:
However:
and indeed tests still fail with the bad ref to Exception which was indeed missing from from 5.3.1 |
Calculates z value for a section plane for each level + a camera location based on IFC model height and a screen size Coefficient. Please note main.test.js and flags.test.js have been temp removed to allow push. Need to resolve.