-
Notifications
You must be signed in to change notification settings - Fork 188
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
FaceAnalysisDisplay.ByViewFacePointsAndValues node only works on single surface #2304
Comments
@Lexiko Sorry for the delay in reply. I think this might be a limitation of the Revit API which Dynamo is doing of good job of un-masking :( I have to check the API documentation, but I believe only one analysis display object is allowed in the view at one time, and Dynamo is creating an analysis display object for each surface. I need to investigate. |
Yeah, that’s what my gut is telling me too. @ksobon has a node in archi-lab which creates a colored surface view(it seems to work with two surfaces: http://archi-lab.net/colored-surface-analysis-display-with-mantis-shrimp-and-ladybug-continued/). He uses it with mantis shrimp to bring in surfaces from grasshopper but I presume it should work the same with geometry out of revit. I tried to use it in my graph (see image in original post) but I keep getting a “null” result. I’m afraid I don’t know any programming syntax to figure it out myself :P |
Had a quick look at @ksobon 's implementation. The SpatialFieldManager allows for multiple surface references: However Dynamo's implementation clears the "buffer" before adding each new surface: Most likely it was done because it would be difficult to know if the newly added data is an update to an existing face reference or an entirely new instance? |
@dimven I can't believe you actually got my nodes to work. I thought they were just old and broken but at the same time I haven't used that particular node in a while. One issue I always had with these nodes was remapping of the UVs from Rhino UVs to Revit UVs as imported Rhino surfaces would get randomly flipped. Also, flat surfaces are a bit of a pickle for Revit when it comes to mapping UVs. For some reason it always kept the original domain of the surface rather then remapping it to 0-1 as it does with all other surfaces (nurbs). I am glad you got it working though. |
@ksobon This is a parallel topic. Any ideas of why this is failing and any suggestions on better workflows? |
@ZiyunShang not sure how complicated it is, is this a matter of changing the Surface input in FaceAnalysisDisplay.ByViewFacePointsAndValues to accept a list instead of a single item? Having a conversation right now with a customer that is using this functionality |
@ZiyunShang following @kronz message above, we are trying to generate a set of heatmaps between different surfaces. Both the reference and the report should be a list of surfaces (if possible). It would be also very nice to be able to setup:
|
Hi @kronz @cesarecaoduro , I just investigated this issue. As @dimven pointed out, that node will clear SpatialFieldManager every time, so only one surface has analysis display result. |
Basically the idea is to have a base surface (or surfaces) and another set of surfaces and calculate di distance in between. The images below are a very simple example of two surface to be reported against a single surface. |
This looks fixed in Revit 2023.1, perhaps even 2023.0 |
Dynamo version
0.9.2
Operating system
Windows 7
What did you do?
Run FaceAnalysisDisplay.ByViewFacePointsAndValues and FaceAnalysisDisplay.ByViewSurfaceData
What did you expect to see?
Coloured surfaces in Revit relating to analysis results
What did you see instead?
Node outputs more than one Revit.Display and only one surface gets colorized in Revit
![archilab](https://cloud.githubusercontent.com/assets/18419461/14601235/6a8adf6c-05a4-11e6-8436-6234812c66e7.png)
I've also started a thread here:
http://dynamobim.org/forums/topic/face-analysis-display-by-view-face-points-and-values-node-bug/
The text was updated successfully, but these errors were encountered: