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

Family.ByName Deletes Loaded Families #1832

Closed
winterweken opened this issue Oct 27, 2017 · 7 comments
Closed

Family.ByName Deletes Loaded Families #1832

winterweken opened this issue Oct 27, 2017 · 7 comments
Labels

Comments

@winterweken
Copy link

Dynamo version

1.3.2.2480

Revit version

2017

Operating system

win 10

What did you do?

Using Family.ByName to compare an external list with loaded families. Once node is removed from graph, it also removes the families it 'touched'

What did you expect to see?

For it to leave the Families in my project as they were.

What did you see instead?

All 'touched' families removed from my project.

More info here.

@mjkkirschner
Copy link
Member

@kronz related to the bug @mrahmaniasl saw?

@mjkkirschner
Copy link
Member

I believe this is an easy fix in the constructor for the ByName method.
My guess would be to change this line:
https://github.com/DynamoDS/DynamoRevit/blame/Revit2018/src/Libraries/RevitNodes/Elements/Family.cs#L128

to use Family.FromExisting instead of creating a wrapper with the internal constructor.

@kronz
Copy link

kronz commented Oct 30, 2017

@mjkkirschner sounds related to what we saw, athought I was not able to delete Families with only Family.ByName (needed to also use the Family.Types AND it needed to be in a custom node).

@DanBoghean
Copy link

DanBoghean commented Nov 2, 2017

I'm having the same issue when I use the Element.SetParameterByName node.

I am currently trying to use dynamo to set a Family Type parameter to switch out door panels in a door.

Let’s say I have 3 doors all set to Door Panel “2P-G”. If I change all three doors to a new panel type “F” so that no doors use Door Panel “2P-G” any more using the dynamo script, Door Panel 2P-G gets deleted.

Problem does not occur if I run the script through Dynamo Player

I forgot to add... I am also using a Family.ByName node in the workflow and that's what seems to be causing it.

@mjkkirschner
Copy link
Member

@mrahmaniasl can you share your workaround here?

@QilongTang
Copy link
Contributor

#1848 should have fixed this issue in the coming release RC1.3.3 and RC2.0,0. Closing

@WspDev
Copy link

WspDev commented Nov 28, 2017

@QilongTang Just wanted to report that this bug also happens when this node is frozen then unfrozen.

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

No branches or pull requests

6 participants