Skip to content
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

DynamoRevit dealing badly with 3D view templates #1636

Open
andydandy74 opened this issue Mar 28, 2017 · 5 comments
Open

DynamoRevit dealing badly with 3D view templates #1636

andydandy74 opened this issue Mar 28, 2017 · 5 comments

Comments

@andydandy74
Copy link

andydandy74 commented Mar 28, 2017

Dynamo version

1.2.3.592

Revit version

2017.1

Operating system

Win 7

What did you do?

Ran All Elements of Type on View3D types.

What did you expect to see?

A list of all 3D views, hopefully including view templates

What did you see instead?

3dviewtemplatesreturnedasnull
The last two items are 3D view templates and return null (to reproduce run these nodes on the basic architectural sample that ships with Revit).
The same thing happens when running All Elements of Category on category Views, btw.
Admittedly, this is better than the original bug (DynamoDS/Dynamo#2970), but it would obviously be better if these nodes wouldn't return null and even better yet if they would actually return 3D view templates as well (view templates for all other view types are returned).

@ksobon ksobon mentioned this issue Jun 1, 2017
5 tasks
@ksobon
Copy link
Contributor

ksobon commented Jun 1, 2017

@andydandy74 we would have to do a small refactor of the 3dView class internally to make this work but it's possible:

image

@mjkkirschner I had to rename the original 3d view class to Abstract3DView since PerspectiveView and AxonometricView both inherit from it. I also created it's own 3DView wrapper so that we can handle things that are 3d view but are not either perspective nor axonometric: 3d view template. Also changed a few things around with the ToDSType() call to stop skipping view templates.

Let me know what you think. Cheers!

@andydandy74
Copy link
Author

Nice work

@ksobon ksobon mentioned this issue Apr 11, 2018
5 tasks
@ksobon
Copy link
Contributor

ksobon commented Aug 13, 2018

@kronz can we close this issue? Fix was merged. Thanks!

@kronz
Copy link

kronz commented Aug 14, 2018

FYI @AndyDu1985

@AndyDu1985
Copy link
Collaborator

We would merge the 3dview-wrapper commits after version 3.x. Track it with link: #2290 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants