mariuszhermansdorfer / SandWorm

Augmented Reality Sandbox for Grasshopper & Rhino
MIT License
20 stars 11 forks source link

Add color-based token recognition #99

Open philipbelesky opened 2 years ago

philipbelesky commented 2 years ago

This is very much a draft/tracking PR, the code below is just the first stage in integrating a standalone prototype that produces edges and center-points based on color recognition.

Since #25 was initially opened, I'm assuming the preference is for this functionality to be encapsulated into the main component. My initial thoughts would be that this would require:

I guess the new input/outputs could be shown/hidden based on if this analysis type is engaged. But there may not be much need given the component ends up being very tall in practice due to the sliding panels.

As previously mentioned, the intent is for this to be merged after a proper 2.0 release. Also, let me know if there's anything else to do on the 2.0 release front?

mariuszhermansdorfer commented 2 years ago

FYI @philipbelesky, I'm on holidays for the next 2 weeks and won't be able to look into this. Will get back to you in the first week of March.

mariuszhermansdorfer commented 2 years ago

Agreed - let's focus on the 2.0 release and add this later. I will prepare the Yak package and publish it as pre-release. Would appreciate it if you could test it before we roll it out to the community.

mariuszhermansdorfer commented 2 years ago

@philipbelesky, thanks for testing it with V2.

Could you please rebase this to master? It's updated now and it would be good to start with a clean slate. Also, let's open a dedicated issue to discuss the direction.