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

Allow SINGLE rotation value for TextNote.ByStringAndPosition #76

Closed
buddyd16 opened this issue Nov 4, 2015 · 6 comments
Closed

Allow SINGLE rotation value for TextNote.ByStringAndPosition #76

buddyd16 opened this issue Nov 4, 2015 · 6 comments

Comments

@buddyd16
Copy link

buddyd16 commented Nov 4, 2015

I am unable to get the TextNote.ByStringAndPosition node to work I have tried everything I can think of even using individual code blocks to each of the inputs with the same result (see attached jpg).

any help would be great...normally would put this on the dynamo forum but its in the middle of a transition.

Revit 2014
Dynamo 0.8.2.2392
Clockwork for Dynamo 0.8.2

capture

@buddyd16
Copy link
Author

buddyd16 commented Nov 4, 2015

I was able to get the underlying python code of the node to work as its own python node and switching out rotation with the x and z vectors in a list that matches the size of my string and point lists.

@andydandy74
Copy link
Owner

In its current state, the node expects a list of angles for the rotation input. I can see that for a lot of cases it would make sense to have this as an optional feature instead of as a default. I'll redesign the node accordingly.

@andydandy74 andydandy74 changed the title TextNote.ByStringAndPosition Issues Allow SINGLE rotation value for TextNote.ByStringAndPosition Nov 8, 2015
@buddyd16
Copy link
Author

buddyd16 commented Nov 9, 2015

Andy,

thanks for the info. Can the same be said for the "width" input is it also looking for a list of widths rather than a single width input.

Also for my own education where is the best place to get documentation on each of the nodes you have provided, not very familiar with git hubs layout.

Thank you

@andydandy74
Copy link
Owner

:-) I may have been a bit overenthusiastic when making this node. I think I will revert the width input to expect a single item instead of a list.

Regarding documentation:
There are samples for the operation of almost all of the Clockwork nodes in this directory: https://github.com/CAAD-RWTH/ClockworkForDynamo/tree/master/package_samples/0.7.x
Not all of this is up-to-date, though, and it's highly unlikely that I will manage to update the samples before next summer.

@buddyd16
Copy link
Author

buddyd16 commented Nov 9, 2015

Ha no worries.... the obviously hard work in creating all these supplementary nodes is much appreciated.

Thanks for the link to the sample files as well.

@andydandy74
Copy link
Owner

@buddyd16 - I will close this for now, but I plan to rewrite this node as a Zero Touch node in 2016 (#85) which would make it a lot more flexible in terms of feeding list or single elements into any of the inputs.

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

No branches or pull requests

2 participants