Skip to content

[WIP] Add node for creating polycurve from random points#85

Merged
johnpierson merged 2 commits intomasterfrom
copilot/add-polycurve-random-points-node
Jan 21, 2026
Merged

[WIP] Add node for creating polycurve from random points#85
johnpierson merged 2 commits intomasterfrom
copilot/add-polycurve-random-points-node

Conversation

Copy link
Contributor

Copilot AI commented Jan 21, 2026

Implementation Plan for PolyCurve.ByRandomPoints Node

  • Explore the repository structure and understand existing patterns
  • Create a new PolyCurve.cs file in src/RhythmCore/Geometry directory
  • Implement ByRandomPoints method that:
    • Takes number of points as input
    • Generates random points in a bounded area
    • Creates a PolyCurve from those points
  • Follow existing code patterns from other Geometry classes
  • Verify no compilation errors in the new code
  • Run code review
  • Run security checks with CodeQL
Original prompt

This section details on the original issue you should resolve

<issue_title>PolyCurve.ByRandomPoints</issue_title>
<issue_description>What is the problem that needs solving?
Want a node in RhythmCore for creating a polycurve from a random distribution of points. The inputs should be number of points and a random distribution of points is made in the code.</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: johnpierson <15744724+johnpierson@users.noreply.github.com>
@johnpierson johnpierson marked this pull request as ready for review January 21, 2026 16:38
@johnpierson johnpierson merged commit 4a862b1 into master Jan 21, 2026
Copilot AI requested a review from johnpierson January 21, 2026 16:39
Copilot stopped work on behalf of johnpierson due to an error January 21, 2026 16:39
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.

PolyCurve.ByRandomPoints

2 participants