Skip to content

Conversation

@EZheln
Copy link
Collaborator

@EZheln EZheln commented Jan 7, 2026

📝 Description

Refactored the graph node and arrow logic to remove the dependency on @d4c/numjs.
Added dynamic validation to check connections between nodes and arrows, making the validation more reliable across different graph scales and UI rendering variations.


🛠️ Changes Made


✅ Checklist

  • I have given the PR a well-structured title describing the domain and the specific change that was made
  • I tested the changes in the browser (locally or via preview build)
  • I confirmed that existing tests pass
  • I added or updated unit / integration tests (if needed)
  • I checked that this change doesn’t introduce new console warnings or lint / formatting errors
  • I updated the relevant Jira ticket with the appropriate details and status

🔗 References

  • Related ticket / issue:
  • Figma / design spec:
  • Documentation:

🚨 Potentially Breaking Changes

  • Yes
  • No

Includes DRC change

  • Yes
  • No

If yes -> requires bump NPM version


🔍 Additional Notes


📸 Screenshots / Demos


@EZheln EZheln force-pushed the QA-tests-1.11.0-rc12 branch from b537a89 to 5824581 Compare January 7, 2026 11:56
@muli-cohen muli-cohen merged commit a16d13a into mlrun:development Jan 7, 2026
3 checks passed
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.

3 participants