Dynamo Node in depth description, Node help docs steve de witt Round 1 by stevecbc · Pull Request #3196 · DynamoDS/DynamoRevit

stevecbc · 2025-07-02T21:10:44Z

Please Note:

Before submitting the PR, please review How to Contribute to Dynamo
Dynamo Team will meet 1x a month to review PRs found on Github (Issues will be handled separately)
PRs will be reviewed from oldest to newest
If a reviewed PR requires changes by the owner, the owner of the PR has 30 days to respond. If the PR has seen no activity by the next session, it will be either closed by the team or depending on its utility will be taken over by someone on the team
PRs should use either Dynamo's default PR template or one of these other template options in order to be considered for review.
PRs that do not have one of the Dynamo PR templates completely filled out with all declarations satisfied will not be reviewed by the Dynamo team.
PRs made to the DynamoRevit repo will need to be cherry-picked into all the DynamoRevit Release branches that Dynamo supports. Contributors will be responsible for cherry-picking their reviewed commits to the other branches after a LGTM label is added to the PR.

Purpose

(FILL ME IN) This section describes why this PR is here. Usually it would include a reference
to the tracking task that it is part or all of the solution for.

Declarations

Check these if you believe they are true

The code base is in a better state after this PR
Is documented according to the standards
The level of testing this PR includes is appropriate
User facing strings, if any, are extracted into *.resx files
Snapshot of UI changes, if any.

Reviewers

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

Testing

…le file logic, maybe some tips and maybe a hyper link to Autodesk help

…shared within help.

Mikhinja · 2025-07-03T10:14:51Z

doc/distrib/NodeHelpFiles/New Text Document.txt

This seems to be an accidental file

@stevecbc This file needs to be removed.

Copilot

Pull Request Overview

This PR adds comprehensive node help documentation for 27 Dynamo Revit nodes, focusing on wall creation, view management, sheet operations, group handling, family operations, and parameter utilities. The documentation includes both detailed markdown descriptions and working example files to help users understand how to use these nodes effectively.

In-depth explanations and usage examples for wall, view, and sheet creation nodes
Documentation for group management, family operations, and parameter utilities
Working .dyn example files demonstrating practical node usage

Reviewed Changes

Copilot reviewed 46 out of 70 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
Revit.Elements.Wall.ByCurveAndLevels.md	Adds wall creation documentation with level constraints
Revit.Elements.Views.*.md (8 files)	Documents view operations including crop box, section views, and schedule filters
Revit.Elements.Views.Sheet.*.md (6 files)	Comprehensive sheet creation and duplication documentation
Revit.Elements.Group.*.md (4 files)	Group creation, placement, and attachment documentation
Revit.Elements.FamilyType.*.md (2 files)	Family creation from geometry documentation
Revit.Elements.FamilyInstance.*.md (5 files)	Family instance placement documentation
Revit.Elements.Element.GetChildElements.md	Child element retrieval documentation
Revit.Elements.Parameter.Unit.md	Parameter unit type documentation
*.dyn files (13 files)	Working example files for documented nodes

doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.View.SetCropBox.md

...ib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByNameNumberTitleBlockViewsAndLocations.md

...trib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByNameNumberTitleBlockViewAndLocation.md

doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByNameNumberTitleBlock.md

doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.ScheduleView.ClearAllFilters.md

doc/distrib/NodeHelpFiles/Revit.Elements.Views.View.SetCropBox.dyn

achintyabhat · 2025-08-07T13:26:50Z

doc/distrib/NodeHelpFiles/Revit.Elements.FamilyInstance.ByHostAndPoint_img.jpg

@stevecbc Group needs a title

achintyabhat · 2025-08-07T13:28:10Z

doc/distrib/NodeHelpFiles/Revit.Elements.FamilyType.ByGeometry_img.jpg

@stevecbc Image is zoomed out and not clear to read. Suggest using Watch3D node to show the geometry, that way can zoom in on the graph for better visibility.

achintyabhat · 2025-08-07T13:28:22Z

doc/distrib/NodeHelpFiles/Revit.Elements.FamilyType.VoidByGeometry_img.jpg

@stevecbc Image is zoomed out and not clear to read. Suggest using Watch3D node to show the geometry, that way can zoom in on the graph for better visibility.

achintyabhat · 2025-08-07T13:29:20Z

doc/distrib/NodeHelpFiles/Revit.Elements.Views.ScheduleView.ClearAllFilters_img.jpg

@stevecbc Monocle icons are seen in the image, can you please fix the image as it is not available OOTB and can confuse users.

achintyabhat · 2025-08-07T13:29:33Z

doc/distrib/NodeHelpFiles/Revit.Elements.Views.SectionView.ByBoundingBox_img.jpg

@stevecbc Group needs a title

achintyabhat

@stevecbc Overall looks good, just a few minor things here and there! Can you please make the minor changes to description and images as requested in the comments here.

….dyn Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…iew.ClearAllFilters.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…c/DynamoRevit into NodeHelpDocs-SteveDeWitt

…ameNumberTitleBlock.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ameNumberTitleBlockViewAndLocation.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ameNumberTitleBlockViewsAndLocations.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…ropBox.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

this was an error

…c/DynamoRevit into NodeHelpDocs-SteveDeWitt

achintyabhat · 2025-08-11T22:05:18Z

doc/distrib/NodeHelpFiles/Revit.Elements.Group.ByElements_img.jpg

@stevecbc This image has been cut-off on the right side.

achintyabhat

One image needs to be updated.

achintyabhat

LGTM now

ZbynekHanak-RWS · 2025-08-13T10:14:05Z

Hi @Mikhinja, seems the en-US .md files were posted in [https://github.com/DynamoDS/DynamoRevit/tree/master/doc/distrib/NodeHelpFiles/en-US] instead of [https://github.com/DynamoDS/DynamoRevit/tree/master/doc/distrib/NodeHelpFiles/en-US/fallback_docs] folder like the rest of NodeHelpFiles sources for localization. Thank you!

stevecbc added 4 commits July 1, 2025 09:33

Added Text file.

5f15ecc

Testing

First In-Depth description of node functionality, description of samp…

7119542

…le file logic, maybe some tips and maybe a hyper link to Autodesk help

First upload of Dynamo for Revit sample graph. This will be used and …

16915b7

…shared within help.

First image upload for sample graphs.

fd20726

achintyabhat self-requested a review July 2, 2025 21:12

Mikhinja reviewed Jul 3, 2025

View reviewed changes

stevecbc changed the title ~~Dynamo Node in depth description, Node help docs steve de witt~~ Dynamo Node in depth description, Node help docs steve de witt Round 1 Jul 23, 2025

johnpierson requested a review from Copilot July 23, 2025 14:36

Copilot AI reviewed Jul 23, 2025

View reviewed changes

achintyabhat reviewed Aug 7, 2025

View reviewed changes

doc/distrib/NodeHelpFiles/Revit.Elements.FamilyInstance.ByHostAndPoint_img.jpg

Copy link

achintyabhat Aug 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevecbc Group needs a title

achintyabhat reviewed Aug 7, 2025

View reviewed changes

doc/distrib/NodeHelpFiles/Revit.Elements.Views.SectionView.ByBoundingBox_img.jpg

Copy link

achintyabhat Aug 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevecbc Group needs a title

achintyabhat requested changes Aug 7, 2025

View reviewed changes

stevecbc and others added 14 commits August 8, 2025 09:17

Update doc/distrib/NodeHelpFiles/Revit.Elements.Views.View.SetCropBox…

de6e413

….dyn Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.ScheduleV…

7d0c384

…iew.ClearAllFilters.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update Revit.Elements.FamilyInstance.ByHostAndPoint_img.jpg

276361e

Merge branch 'NodeHelpDocs-SteveDeWitt' of https://github.com/stevecb…

86dbeac

…c/DynamoRevit into NodeHelpDocs-SteveDeWitt

Update doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByN…

f1c4d13

…ameNumberTitleBlock.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByN…

8c4bafb

…ameNumberTitleBlockViewAndLocation.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.Sheet.ByN…

1eb8edb

…ameNumberTitleBlockViewsAndLocations.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Update doc/distrib/NodeHelpFiles/en-US/Revit.Elements.Views.View.SetC…

c209cac

…ropBox.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Delete doc/distrib/NodeHelpFiles/New Text Document.txt

88d96de

this was an error

Update Revit.Elements.FamilyType.ByGeometry_img.jpg

3a6b112

Merge branch 'NodeHelpDocs-SteveDeWitt' of https://github.com/stevecb…

b43b3d8

…c/DynamoRevit into NodeHelpDocs-SteveDeWitt

added 3d watch node and zoomed in to full view of graph

dd39a5f

image update

ba16cb0

remove the grouping

7cfec85

achintyabhat reviewed Aug 11, 2025

View reviewed changes

doc/distrib/NodeHelpFiles/Revit.Elements.Group.ByElements_img.jpg

Copy link

achintyabhat Aug 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stevecbc This image has been cut-off on the right side.

achintyabhat requested changes Aug 11, 2025

View reviewed changes

update cut off image

03ba027

achintyabhat approved these changes Aug 12, 2025

View reviewed changes

Mikhinja approved these changes Aug 13, 2025

View reviewed changes

Mikhinja merged commit c2a8dfd into DynamoDS:master Aug 13, 2025
9 of 10 checks passed

Mikhinja mentioned this pull request Aug 13, 2025

move .md files to fallback_docs subfolder #3209

Merged

Conversation

stevecbc commented Jul 2, 2025

Purpose

Declarations

Reviewers

FYIs

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

achintyabhat left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

achintyabhat left a comment

Choose a reason for hiding this comment

Uh oh!

achintyabhat left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ZbynekHanak-RWS commented Aug 13, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants