Rotate crop box of multiple views

Paul_Wintour · July 29, 2019, 5:27am

Hi

I’m trying to rotate a list of views based on a list of rotations (in degrees) by modifying the code in this post. @Konrad_K_Sobon @Mark.Ackerley @Dimitar_Venkov

This is what I have but I am getting an error “unexpected token ‘v’”

import clr

clr.AddReference('RevitServices')
from RevitServices.Persistence import DocumentManager
from RevitServices.Transactions import TransactionManager
doc = DocumentManager.Instance.CurrentDBDocument

clr.AddReference('RevitAPI')
from Autodesk.Revit.DB import *

rot = IN[0]
viewports = UnwrapElement(IN[1])

def GetViewCropBoxElement(view):
	doc = DocumentManager.Instance.CurrentDBDocument
	TransactionManager.Instance.ForceCloseTransaction()
	tGroup = TransactionGroup(doc, "Temp to find crop box element")
	tGroup.Start()
	trans1 = Transaction(doc, "Temp to find crop box element")
	trans1.Start()
	view.CropBoxVisible = False
	trans1.Commit()
	
	shownElems = FilteredElementCollector(doc, view.Id).ToElementIds()
	
	trans1.Start()
	view.CropBoxVisible = True
	trans1.Commit()
	
	cropBoxElement = FilteredElementCollector(doc, view.Id).Excluding(shownElems).FirstElement()
	tGroup.RollBack()
	return cropBoxElement
	
def RotateCropBox(view, cropBox, angle):
    doc = DocumentManager.Instance.CurrentDBDocument
    bbox = view.CropBox
    center = 0.5 * (bbox.Max + bbox.Min)
    axis = Line.CreateBound(center, center + XYZ.BasisZ)
    TransactionManager.Instance.EnsureInTransaction(doc)
    ElementTransformUtils.RotateElement(doc, cropBox.Id, axis, angle)
    TransactionManager.Instance.TransactionTaskDone()
    return view

For v in viewports:
   	crpbx = GetViewCropBoxElement(v)
    crpbxRt = RotateCropBox(v, crpbx, rot)

OUT = crpbxRt

Any ideas?

Thanks

503_View rotation.dyn (8.5 KB)

Alban_de_Chasteigner · July 29, 2019, 5:56am

Can you try instead ? for v in viewports:
(For in lowercase )

import clr

clr.AddReference('RevitServices')
from RevitServices.Persistence import DocumentManager
from RevitServices.Transactions import TransactionManager
doc = DocumentManager.Instance.CurrentDBDocument

clr.AddReference('RevitAPI')
from Autodesk.Revit.DB import *

rot = IN[0]
viewports = UnwrapElement(IN[1])

def GetViewCropBoxElement(view):
	doc = DocumentManager.Instance.CurrentDBDocument
	TransactionManager.Instance.ForceCloseTransaction()
	tGroup = TransactionGroup(doc, "Temp to find crop box element")
	tGroup.Start()
	trans1 = Transaction(doc, "Temp to find crop box element")
	trans1.Start()
	view.CropBoxVisible = False
	trans1.Commit()
	
	shownElems = FilteredElementCollector(doc, view.Id).ToElementIds()
	
	trans1.Start()
	view.CropBoxVisible = True
	trans1.Commit()
	
	cropBoxElement = FilteredElementCollector(doc, view.Id).Excluding(shownElems).FirstElement()
	tGroup.RollBack()
	return cropBoxElement
	
def RotateCropBox(view, cropBox, angle):
    doc = DocumentManager.Instance.CurrentDBDocument
    bbox = view.CropBox
    center = 0.5 * (bbox.Max + bbox.Min)
    axis = Line.CreateBound(center, center + XYZ.BasisZ)
    TransactionManager.Instance.EnsureInTransaction(doc)
    ElementTransformUtils.RotateElement(doc, cropBox.Id, axis, angle)
    TransactionManager.Instance.TransactionTaskDone()
    return view

for v in viewports:
   	crpbx = GetViewCropBoxElement(v)
    crpbxRt = RotateCropBox(v, crpbx, rot)

OUT = crpbxRt

Paul_Wintour · July 29, 2019, 6:06am

Thanks. That fixed that error but now I get:

File “”, line 46, in
File “”, line 40, in RotateCropBox
TypeError: expected float, got list

Any ideas how to fix? I believe it is because I am feeding in a list of angles (which should actually be in radians).

Mark.Ackerley · July 29, 2019, 7:59am

Hey, sorry I can’t check it right now, but I would maybe try a zip rather than for…? So you’re iterating through both lists together? Something like…

for v, r in zip(viewports, rot):
   	crpbx = GetViewCropBoxElement(v)
    crpbxRt = RotateCropBox(v, crpbx, r)

Hopefully that is helpful

Mark

Paul_Wintour · August 2, 2019, 5:13am

Thanks @Mark.Ackerley. Yep that was pretty much it. Just had to append the output…

import clr

clr.AddReference('RevitServices')
from RevitServices.Persistence import DocumentManager
from RevitServices.Transactions import TransactionManager
doc = DocumentManager.Instance.CurrentDBDocument

clr.AddReference('RevitAPI')
from Autodesk.Revit.DB import *

#rotations to be in radians
rot = UnwrapElement(IN[0])
viewports = UnwrapElement(IN[1])
views = []

def GetViewCropBoxElement(view):
	doc = DocumentManager.Instance.CurrentDBDocument
	TransactionManager.Instance.ForceCloseTransaction()
	tGroup = TransactionGroup(doc, "Temp to find crop box element")
	tGroup.Start()
	trans1 = Transaction(doc, "Temp to find crop box element")
	trans1.Start()
	view.CropBoxVisible = False
	trans1.Commit()
	
	shownElems = FilteredElementCollector(doc, view.Id).ToElementIds()
	
	trans1.Start()
	view.CropBoxVisible = True
	trans1.Commit()
	
	cropBoxElement = FilteredElementCollector(doc, view.Id).Excluding(shownElems).FirstElement()
	tGroup.RollBack()
	return cropBoxElement
	
def RotateCropBox(view, cropBox, angle):
    doc = DocumentManager.Instance.CurrentDBDocument
    bbox = view.CropBox
    center = 0.5 * (bbox.Max + bbox.Min)
    axis = Line.CreateBound(center, center + XYZ.BasisZ)
    TransactionManager.Instance.EnsureInTransaction(doc)
    ElementTransformUtils.RotateElement(doc, cropBox.Id, axis, angle)
    TransactionManager.Instance.TransactionTaskDone()
    return view
    
for v, r in zip(viewports, rot):
	crpbx = GetViewCropBoxElement(v)
	crpbxRt = RotateCropBox(v, crpbx, r)
	views.append(crpbxRt)

OUT = views

bvs1982 · December 11, 2019, 1:24pm

Thanks for the Python @Paul_Wintour . But i don’t get the expected rotation.
Can you explain what i am missing?

EDIT
Solved! I totally missed

tjxme · July 7, 2020, 5:45pm

awesome, it works perfectly.
I am just wondering how to change “for” to make the list writing with same number of angles ?
like i want to rotate all my views for .26 radiant, but i dont have to create a list of the same number repeating itself.

Mark.Ackerley · July 8, 2020, 9:01pm

Hey,

I guess you mean this bit?

for v, r in zip(viewports, rot):
    crpbx = GetViewCropBoxElement(v)
    crpbxRt = RotateCropBox(v, crpbx, r)
    views.append(crpbxRt)
    OUT = views

you can replace it with this… (untested!)

for r in (rot):
    crpbx=GetViewCropBoxElement(view)
    crpbxRt = RotateCropBox(view, crpbx, r)
    views.append(crpbxRt)
    OUT = views

Hopefully that’s right

Mark

tjxme · July 8, 2020, 9:48pm

I don’t think repeat r would be the case. I think we should repeat append views in the list with the same “rot”

Mark.Ackerley · July 9, 2020, 10:40am

Hey, yes you’re right, my bad, but you get the idea… ‘for’ instead of ‘zip’

crawdude · September 24, 2020, 8:32pm

Has anyone gotten this to work for rotating the section boxes? I came across this post and this one by Jeremy, but I have been unable to get the section box to actually rotate the way that I need it to.

bvs1982 · September 25, 2020, 2:15pm

What goes wrong?

linderoth · June 5, 2021, 4:41pm

Hi!
I’m trying to rotate a view and the crop box (by selecting a wall and getting the angle between this and a 0,1,0 vector) I’ve tried this python code but it gives me an error:

The code in this post works partially, rotates the view but not the crop box…

Any ideas?? Thanks!!

linderoth · June 9, 2021, 5:35pm

Anybody? Please!!
@Konrad_K_Sobon @Mark.Ackerley @Alban_de_Chasteigner

Any help will be much appreciated!!

Mark.Ackerley · June 10, 2021, 9:21am

Hi,

The warning is telling you that you can’t iterate over the view… This means it is expecting a list (mostly), so if you add a ‘create list’ before you feed into IN[1] it will help matters. If you could post your .dyn it would make it easier to debug

I suspect the problem you are getting in other post is because you have a custom crop box, i’m not sure they obey the same commands. To test, if you click on it in Revit and ‘reset’ (a button on the top ribbon on the right), you can re-run the python. I don’t know if there is a way to edit the sketch lines of the crop box.

Hope that is useful,

Mark

linderoth · June 10, 2021, 10:31am

RotaciónVistas0.1.dyn (34.9 KB)
Thanks Mark for replying! You are right, addind a List.Create before the python node worked (for both inputs), but the crop region is not rotating. I’m still very noob at python.

The crop region is created automatically with another script I created before, which creates a cropped view for each flat type present in the model. There are different buildings with different angles, so some flats are not north-orientated. So I need the view and the region rotates together to be vertical…

I think there must be a way to do this, because in the other script I said, I had a node (Rhythm or archilab Package) that creates the crop region…

truevis · July 14, 2021, 3:58pm

Is there a way to do it without all the Transaction Starts and Commits, etc.? I am doing it to hundreds of views and want to optimize speed.

I tried to remove them in def GetViewCropBoxElement but it didn’t work. The script does work as is, thank you.

Topic		Replies	Views
Rotate View cropbox Revit revit	6	2991	December 11, 2019
Rotating (free angle) cropviews with Python Developers revit , python , dynamo	13	881	August 13, 2018
Rotate ''cropbox or view'' via Id Revit python	1	118	August 16, 2023
Get Crop Box Rotation	1	729	January 8, 2019
Get Crop views Developers revit , python , dynamo , views	2	594	August 8, 2018

Rotate crop box of multiple views

Related Topics