engine_components icon indicating copy to clipboard operation
engine_components copied to clipboard

Picking colums in FloorPlans views

Open RASRC opened this issue 2 years ago • 6 comments

Describe the bug 📝

I'm working in the Floor Plan viewer, and noticed that you cannot pick columns, even when I added the category using the classifier like in the documentation c0nlons3yjtdp0x9gv59hmzyiafa I can't select column elements by clicking. Instead, it selects the slab. Otherwise, I have no problems selecting any wall element by clicking. 72nva7rh953pqge1307imnes4a09 jsfhuxwgczwra8hk8st018u83s2f Thanks!

Reproduction ▶️

No response

Steps to reproduce 🔢

No response

System Info 💻

openbim-components

Used Package Manager 📦

npm

Error Trace/Logs 📃

No response

Validations ✅

  • [X] Read the docs.
  • [X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • [X] Make sure this is a IFC.js components issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
  • [X] Check that this is a concrete bug. For Q&A join our Community.
  • [X] The provided reproduction is a minimal reproducible example of the bug.

RASRC avatar Oct 18 '23 19:10 RASRC

I don't think it is a bug. It is more of an additionnal feature. Basically, the plan view is just a combination of a clipper and a camera position. When you clip object, it is in fact empty, even though filling style gives the inverse impression. So, when you click, the raycasting doesn't reach column. It instead reaches the slab.

Maozerhouni avatar Oct 19 '23 10:10 Maozerhouni

Hi @Maozerhouni! Thanks for the comment.

I understand your point but I can pick Walls without problems, wherever they are placed in the floorplan. In line with your explanation, they couldn't be pickable...

RASRC avatar Oct 19 '23 17:10 RASRC

Oh ok. You're right. That s a bit odd. For the sake of experimentation, have you tried to tag walls as columns and vice versa ? I find it very weird that category Plays any role in this.

Maozerhouni avatar Oct 19 '23 17:10 Maozerhouni

Indeed, this is something we haven't implemented yet, but shouldn't be hard. I can bind the faces of the Edges Clipper to the IFC items to which they belong, thus enabling to pick clipped elements from above. I'll take care of it!

agviegas avatar Nov 03 '23 13:11 agviegas

Hi @agviegas ! Did you can implement the picking columns functionality for the Floor Plan viewer? Thanks.

RASRC avatar Mar 25 '24 14:03 RASRC

Hi! Not yet, sorry 😳 we are 3 people maintaining the whole thing plus also making the new features. I expect that next month we will have time to clean up the issues in this respository. Feel free to open a PR if it's an urgent matter.Thanks in advance for your patience! 🙏

agviegas avatar Mar 26 '24 19:03 agviegas

Progress

https://github.com/ThatOpen/engine_components/assets/56475338/58f5846f-6472-45d5-8a0d-422f3d62d4d3

agviegas avatar Jun 21 '24 15:06 agviegas

Done! Now we can select clipped elements. Available from @thatopen/[email protected]. Docs also updated, you can try it out in the plan tutorial example. Cheers!

agviegas avatar Jun 22 '24 11:06 agviegas