I am trying to write a script to let python to help to get panels in a row.
This script could be useful when you have thousands of panels.
The strategy is to provide the first panel in the row and specify a vector for selection then let python to select the rest on the row from the panel collection.
It is almost getting there but I got an error saying" Iteration over non-sequence of type NoneType".
I am not sure what does it mean but I am pretty sure it occur due to I have remove a panel from the list of panels.
Appreciate if someone can tell me why it would be a problem – why can’t I remove a panel? I thought a list should be dynamic and you can always modify it…
does not move the point closer to the panels, try it using rs.AddPoint(pt) in your loop. Would it be possible to just pick a line like in below example ?
import Rhino
import rhinoscriptsyntax as rs
def SelectPanelsNearLine():
ids = rs.GetObjects("Select all panels", 8, True, True, False)
if not ids: return
pt_start = rs.GetPoint("Pick point on start of row")
if not pt_start: return
pt_end = rs.GetPoint("Pick point on end of row", pt_start)
if not pt_end: return
line = Rhino.Geometry.Line(pt_start, pt_end)
curve = line.ToNurbsCurve()
params = curve.DivideByCount(100, True)
points = [curve.PointAt(param) for param in params]
panels = set()
for pt in points:
pco = rs.PointClosestObject(pt, ids)
if pco: panels.add(pco[0])
if len(panels) != 0: rs.SelectObjects(list(panels))
SelectPanelsNearLine()
I can’t quite pin point where the error comes from but the iteration seems to go through the entire length of ‘panels’. Your centroid may move out of the length of the row pretty quickly. I’d use a while loop and make it stop when it reaches the end of that row.
I can not test your code as I’m on my phone.
But I’d like to point out that placing a debug break at line 25 will allow you to step into the code and scrutinise what is being passed for the arguments pt and panels.
My suspision is that the error is caused by all panels being popped out of the panels list . So the closest point search has no objects to search.
If you place a print panels between line 24 and 25 you can check the command line after the crash to see what panels was before the crash.
But got a minor query: understand set() can remove duplicated panel ids but will it also jitter the panel sequence in the list since it sorts the sequence base on the ids’ alphabet / number? untitled.py (1.5 KB)
your comments added above are correct. To answer the questions, the set does not allow duplicates and it may not keep the order of the elements added to it while it removes the duplicates. The order in which the objects are added are along the line but it may change, to keep the order, you might try using an OrderedSet or use a list which you parse later to remove consecutive duplicates. I’ve used 100 division points to make sure no panel is missed.
Do you need the previous order of the panels in the final list ?
Thanks for your reply. Yes, I would like the panels maintain a certain sequence. OrderedSet( ) seems to be a class. Haven’t look into it yet…
I have modified the script a bit, seems it works. But just curious is there any other neat way(like a method) can eliminate duplicate items in a list but also maintain its original sequence.
you can maintain the order in the panel ids by not adding the panels but only its indices to a list. Then iterate over the sorted list to select the panels:
panel_indices = []
for pt in points:
pco = rs.PointClosestObject(pt, ids)[0]
index = ids.index(pco)
if index not in panel_indices:
panel_indices.append(index)
for i in sorted(panel_indices):
rs.SelectObject(ids[i])
rs.Sleep(50)
This does not require additional classes and creates no duplicates.
