Skip to content

Conversation

@dariaivanciucova
Copy link
Contributor

@dariaivanciucova dariaivanciucova commented Nov 10, 2023

Purpose

A third batch of documentation help files for T-Spline nodes.
The In-Depth description follows this guidance.
This batch includes In-Depth, Example image, and .dyn files for the following 63 nodes:

Topology

  • TSplineTopology.DecomposedEdges : 7LMFKLQNCV53W7KLS5QWD3E27NGGA33QPHSXMUGH323WVXWJY3GQ
  • TSplineTopology.DecomposedFaces : VIA3XNZWZWW3XHWL222NGHWM22VLSA4QXMZCPWZ6JO6G3P7H2WGA
  • TSplineTopology.DecomposedVertices : GNCPW2XCPOITIUUDNZKISNP347TEGTDWVI5SQTBJX3JWFZTVRXGA
  • TSplineTopology.EdgeByIndex
  • TSplineTopology.FaceByIndex
  • TSplineTopology.VertexByIndex
  • TSplineTopology.BorderEdges
  • TSplineTopology.BorderFaces
  • TSplineTopology.BorderVertices
  • TSplineTopology.EdgesCount
  • TSplineTopology.FacesCount
  • TSplineTopology.InnerEdges
  • TSplineTopology.InnerFaces
  • TSplineTopology.InnerVertices
  • TSplineTopology.NGonFaces
  • TSplineTopology.NonManifoldEdges
  • TSplineTopology.NonManifoldVertices
  • TSplineTopology.RegularFaces
  • TSplineTopology.Regularvertices
  • TSplineTopology.StarPointVertices
  • TSplineTopology.TPointVertices
  • TSplineTopology.VerticesCount

T-Spline Surface

  • TSplineSurface.CreateMatch(tSplineSurface, tsEdges, curves) : 6ICXLN4V6DNK5KMYTY5LPCJBE27IRW5VOBKCCVFQGO3HST752ZNQ
  • TSplineSurface.CreateMatch(tSplineSurface, tsEdges, brepEdges) : BUMI5UR5LLKRXP5CUH46L62SN6YIVFGB6FU2PUTKTVDJMTXWXI5Q
  • TSplineSurface.DeserializeFromTSM
  • TSplineSurface.DuplicateFaces
  • TSplineSurface.EnableSmoothMode
  • TSplineSurface.ExportToTSM
  • TSplineSurface.ExportToTSS
  • TSplineSurface.ExtrudeEdges : 5MRWZW5S4JJMDCAQNQWK32MPFTU5RFV6GLUOOMEN6FLMBO2FOFGQ
  • TSplineSurface.ExtrudeEdgesAlongCurve
  • TSplineSurface.ExtrudeFaces
  • TSplineSurface.ExtrudeFacesAlongCurve
  • TSplineSurface.FillHole
  • TSplineSurface.FlattenVertices(vertices) : WSNXB54TDQEZRTUNRFCTUDK74DHVPQPXBXEYJUZJPW6T2F7XERDQ
  • TSplineSurface.FlattenVertices(vertices, parallelPlane) : XGSWLBVZ2TGT6X7FZRUWJDGDSBL7JWKDBQYWEJWO7VZOJPJI7OWQ
  • TSplineSurface.FlipNormals
  • TSplineSurface.ImportFromTSM(filePath, inSmoothMode) : SEK72377I2YNUG2LJNOJ6APT3WYI3HJW6FDSVZL3LX2XEMYIA4OQ
  • TSplineSurface.ImportFromTSM(file, inSmoothMode) : H7WNBOQXEC5RUVELW464ZP2E7CYHSPZ22TJHQJO4AKA4LHFB6N3A
  • TSplineSurface.ImportFromTSS(filePath, inSmoothMode) : ZBLSFRQNVI3VPC3KTS3XZI6VXSAJAON2ETVE46OLR7NMYOY6INXA
  • TSplineSurface.ImportFromTSS(file, inSmoothMode) : GD5OJ5UNM2WTBIO7EKGVKBLXL5UMKVOXBJKW2YFKOPVHUIHDCIFA
  • TSplineSurface.Interpolate
  • TSplineSurface.MakeUniform
  • TSplineSurface.MergeEdges
  • TSplineSurface.MoveVertices
  • TSplineSurface.PullVertices
  • TSplineSurface.Repair
  • TSplineSurface.SerializeAsTSM
  • TSplineSurface.SlideEdges
  • TSplineSurface.Standardize
  • TSplineSurface.SubdivideFaces
  • TSplineSurface.Thicken(distance, softEdges) : UHLOMXPCNY3C36FQ45G3HQGKIZLSUE2QX4N7FY7ZCCOEN7F7Q6YA
  • TSplineSurface.Thicken(vector, softEdges) : USR6ESCX7ACJGZV2YVVIIF7437ZNDU23SUQ6IAHAIPM2YY2FPFGA
  • TSplineSurface.ToBRep
  • TSplineSurface.ToMesh
  • TSplineSurface.WeldCoincidentVertices
  • TSplineSurface.WeldVertices(vertices, newPosition, keepSubdCreases) : WO4HBR4YYKPQOTXPPFJWGC6B66KH4HDONFO6W7XXS7TACWL7P6YA
  • TSplineSurface.WeldVertices(firstGroup, secondGroup, keepSubdCreases) : UVMMX2H7AW6KY7XACJHBCRTDYLOGHPZHWREHFNQUMZ6SK33XMHHQ
  • TSplineSurface.IsClosed
  • TSplineSurface.IsExtractable
  • TSplineSurface.IsInBoxMode
  • TSplineSurface.IsStandard
  • TSplineSurface.IsWaterTight

Declarations

Check these if you believe they are true

  • The codebase 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
  • All tests pass using the self-service CI.
  • Snapshot of UI changes, if any.
  • Changes to the API follow Semantic Versioning and are documented in the API Changes document.
  • This PR modifies some build requirements and the readme is updated
  • This PR contains no files larger than 50 MB

Release Notes

Renamed nodes are submitted with their short names.

For the TSplineSurface.SubdivideFaces the short definition is wrong:
Subdivide given faces into four faces each in exact or simple mode depending on given flag value.
For NGon faces the subdivision has a different number of faces (can be 3 or n).

For the TSplineSurface.CreateMatch(tSplineSurface, tsEdges, curves) the tooltip for curves is wrong:
Closed curve loop to create match with
The curve does not have to be a closed loop.

For TSplineSurface.CreateMatch(tSplineSurface, tsEdges, brepEdges) the continuity does not currently support G1 & G2 continuity.

Reviewers

FYIs

@dnenov @Amoursol

@github-actions
Copy link

github-actions bot commented Nov 10, 2023

Bin Diff Issue Resolved.
(♻️ Updated: 2024-02-01-04:09:20)

@dariaivanciucova dariaivanciucova marked this pull request as ready for review November 16, 2023 13:31
Copy link
Collaborator

@hwahlstrom hwahlstrom left a comment

Choose a reason for hiding this comment

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

LGTM with minor fixes

@QilongTang QilongTang added this to the 3.1 milestone Dec 12, 2023
@github-actions
Copy link

github-actions bot commented Feb 1, 2024

UI Smoke Tests

Test: success. 2 passed, 0 failed.
TestComplete Test Result
Workflow Run: UI Smoke Tests
Check: UI Smoke Tests - net8.0

@QilongTang QilongTang merged commit 2bb51dc into DynamoDS:master Feb 1, 2024
QilongTang pushed a commit that referenced this pull request Feb 6, 2024
* DYN-6535 DynamoRevit Improve Search Fix 2

The specific case "element set parameter" was failing on Revit 2025 (not finding anything), then I've modified the code so that check if the first term is a category if that is the case it will create a specific query using the first term as category and the next terms as node name.

* DYN-6535 DynamoRevit Improve Search Fix 2 Code Review

Updating Linq query so that is executed only once.

* Squashed commit of the following:

commit 3893c85
Author: Ashish Aggarwal <ashish.aggarwal@autodesk.com>
Date:   Mon Feb 5 12:53:49 2024 -0500

    Fix retain package structure flow where nested directory at root level were lost (#14918)

    * optimize

    * comments and a test

    * add subdir test and comments

    * add child test

    * test updates

commit d0bd96e
Author: Aaron (Qilong) <173288704@qq.com>
Date:   Fri Feb 2 15:56:03 2024 -0500

    Fix API breaking changes (#14916)

    * Fix API breaking changes

    * Update OpenFileCommand

    * update

commit 8f123fe
Author: Daria Ivanciucova <43077096+dariaivanciucova@users.noreply.github.com>
Date:   Fri Feb 2 03:20:48 2024 +0000

    TSpline rename hashed nodes  (#14921)

    * renamed 2 nodes

    TSplineSurface.BySphereCenterPointRadius
    TSplineSurface.ByCylinderPointsRadiu

    * TSplineSurface.ByPlaneOriginNormal

    * TSplineSurface.ByQuadballCenterRadius

    * TSplineSurface.BySphereFourPoints

    * TSplineSurface.ByPlaneOriginNormalXAxis

    * TSplineSurface.ByPlaneOriginXAxisYAxis

    * TSplineVertex.FunctionalValence

    * TSplineSurface.ByConeCoordinateSystemHeightRadii

    * TSplineSurface.ByPlaneBestFitThroughPoints

    * TSplineSurface.ByPlaneLineAndPoint

    * TSplineSurface.ByPlaneThreePoints

    * TSplineSurface.ByTorusCenterRadii

    * TSplineSurface.ByTorusCoordinateSystemRadii

    * TSplineTopology.TPointVertices

    * TSplineSurface.AddReflections

    * TSplineSurface.DeleteVertices

    * TSplineSurface.CompressIndices

    * TSplineSurface.RemoveReflections

    * TSplineSurface.ByNurbsSurfaceUniform

    * TSplineSurface.UnweldVertices

    * TSplineSurface.BridgeFacesToEdges

    * TSplineReflection.SegmentsCount

    * TSplineSurface.ByNurbsSurfaceCurvature

    * TSplineSurface.BridgeEdgesToFaces

    * TSplineInitialSymmetry.IsRadial; TSplineInitialSymmetry.RadialSymmetryFaces

    * TSplineReflection.SegmentsAngle

    * .TSplineSurface.BridgeFacesToFaces

    * TSplineSurface.BridgeEdgesToEdges

    * TSplineInitialSymmetry.ByRadial

    * SplineSurface.ByCombinedTSplineSurfaces

    * TSplineSurface.UncreaseVertices

    * BuildFromLines

    * TSplineSurface.CreaseVertice

    * TSplineTopology.RegularVertices

    * TSplineTopology.VertexByIndex

    * TSplineSurface.DeserializeFromTSM

    * TSplineSurface.SerializeAsTSM

    * TSplineTopology.BorderVertices

    * TSplineSurface.EnableSmoothMode

    * TSplineTopology.NonManifoldVertices

    * TSplineTopology.VerticesCoun

    * TSplineSurface.DuplicateFaces

    * TSplineSurface.ExtrudeEdgesAlongCurve

    * TSplineTopology.StarPointVertices

    * TSplineTopology.NonManifoldEdges

    * TSplineSurface.WeldCoincidentVertices

    * TSplineSurface.SubdivideFaces

    * TSplineSurface.ExtrudeFacesAlongCurve

    * TSplineSurface.ByConeCoordinateSystemHeightRadius

    * TSplineSurface.ByCylinderRadiusHeight

commit 2bb51dc
Author: Daria Ivanciucova <43077096+dariaivanciucova@users.noreply.github.com>
Date:   Thu Feb 1 17:43:53 2024 +0000

    TSpline Nodes Documentation - Third set (#14598)

    * FillHole, IsClosed, IsInBoxMode

    * BorderFaces, BorderEdges, BorderVertices, DuplicateFaces

    * EnableSmoothMode, ExportToTSS, ExportToTSM

    * ImportFromTSM (2)

    * ImportFromTSS(2)

    * ExtrudeFaces, Interpolate, SubdivideFaces, FlattenVertices (2)

    * IsStandard, Standardize, Thicken (2)

    * Decomposed (3), EdgeByIndex, EdgeCount, InnerEdges, InnerFaces, InnerVertices, NGonFaces, NonManifoldEdges, NonManifoldVertices

    * RegularFaces, RegularVertices, StarPoints, TPoints

    * Repair, Serialize, Deserialize

    * ExtrudeEdges, ExtrudeEdgesAlongCurve, ExtrudeFacesAlongCurve

    * FlipNormals, IsExtractable

    * MoveVertices, FaceByIndex, VertexByIndex

    * Is Watertight, FacesCount, VerticesCount, SlideEdges, MergeEdges, WeldCoincidentVertices, WeldVertices(2)

    * ToBrep, ToMesh

    * Update WSNXB54TDQEZRTUNRFCTUDK74DHVPQPXBXEYJUZJPW6T2F7XERDQ.md

    * CreateMatch (2), PullVertices

    * MakeUniform

    * updated to better method for highlighting edges

    * spell check

    * Update 6ICXLN4V6DNK5KMYTY5LPCJBE27IRW5VOBKCCVFQGO3HST752ZNQ.md

    * added TSM, TSS files, typos

    * Helena's comments

    ---------

    Co-authored-by: Deyan Nenov <dnenov@archilizer.com>
    Co-authored-by: Aaron (Qilong) <173288704@qq.com>

commit 9136633
Author: Daria Ivanciucova <43077096+dariaivanciucova@users.noreply.github.com>
Date:   Thu Feb 1 16:59:41 2024 +0000

    TSpline Nodes Documentation - Second set (#14550)

    * TSplineInitialSymmetry nodes

    * TSplineReflection nodes documentation

    * two more Reflections nodes

    * 7 more nodes

    - Bevel Edges
    - Crease Edges
    - Crease Vertices
    - Uncrease Edges
    - Uncrease Vertices
    - Unweld Edges
    - Unweld Vertices

    * ByNurbs and ByCombinedSurfaces

    - ByNurbsSurfaceCurvature
    -ByNurbsSurfaceUniform
    - ByCombinedTSplineSurfaces

    * 4 Bridge nodes

    * Compress Indexes, BySweep, ByRevolve, BuildFromLines

    * DeleteEdges, DeleteFaces, DeleteVertices

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BridgeEdgesToEdges.md

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BridgeEdgesToFaces.md

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BridgeFacesToEdges.md

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BridgeFacesToFaces.md

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineSurface.BuildFromLines.md

    * added missing TSplineSurface.Reflections node

    * Helena's comments

    ---------

    Co-authored-by: Deyan Nenov <dnenov@archilizer.com>
    Co-authored-by: Aaron (Qilong) <173288704@qq.com>

commit 99cf281
Author: Deyan Nenov <dnenov@archilizer.com>
Date:   Thu Feb 1 16:58:27 2024 +0000

    binding issues resolved (#14682)

    - fixed null exception when trying to assign click handler on a context menu button when the parent button
    - also fixed another incorrect binding issue

commit d2e57b5
Author: Daria Ivanciucova <43077096+dariaivanciucova@users.noreply.github.com>
Date:   Thu Feb 1 15:58:29 2024 +0000

    TSpline Nodes Documentation - First set (#14549)

    * TSEdge and TSFace nodes documentation

    * TSpline Vertex nodes documentation

    * TSpline.UVNFrame nodes documentation

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineVertex.IsStarPoint.md

    * TSpline Box docs

    * TSplineSurface.ByPlane nodes docs

    * TSplineSurface.ByCone docs

    * TSplineSurface - Cylinder docs

    * TSplineSurface.ByQuadball nodes

    * replaced with renamed files for ByBox nodes

    * BySphere nodes

    * TSplineSurface.ByTorus nodes

    * BuildPipes and ByExtrude

    * Update Autodesk.DesignScript.Geometry.TSpline.TSplineVertex.Index.md

    * corrected the description for Torus nodes

    * Helena's comments

    ---------

    Co-authored-by: Deyan Nenov <dnenov@archilizer.com>
    Co-authored-by: Aaron (Qilong) <173288704@qq.com>

* Revert "Squashed commit of the following:"

This reverts commit c182c1b.
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

Successfully merging this pull request may close these issues.

4 participants