Skip to content

Conversation

@webdevred
Copy link
Owner

@webdevred webdevred commented Sep 19, 2025

  • Decrease required test coverage threshold from 80% to 70% in CI workflow.
  • Add SupportVertex module for identifying support vertices based on beam connectivity.
  • Update cabal and package.yaml to include new modules and test specs.
  • Add and update comprehensive tests for new and existing functionality..
  • Refactor transformation logic:
    • Introduce support vertex detection using beam connection counts.
    • Implement sorting and naming for support vertices.
    • Refactor related functions for improved modularity.

@webdevred webdevred force-pushed the support-vertex-detection branch 7 times, most recently from 2f8bb25 to d5fcdd9 Compare September 20, 2025 07:03
- Added special-case renaming logic for SupportTree in `renameVertexId`
- Updated `buildTreeForType` to return support vertices separately
- Introduced helpers:
  * addSupportVertex / addSupportVertices
  * sortSupportVertices
- Modified `addVertexTreeToForest` to insert and sort support vertices
- Simplified `moveVerticesInVertexForest` by removing SupportTree special casing
- Unified tree ordering with SupportTree as first-class: [Left, Middle, Right, Support]
@webdevred webdevred force-pushed the support-vertex-detection branch 2 times, most recently from 3e68810 to 58b9ca2 Compare September 20, 2025 09:43
@webdevred webdevred force-pushed the support-vertex-detection branch from 58b9ca2 to dc25f99 Compare September 20, 2025 09:45
@webdevred webdevred marked this pull request as ready for review September 20, 2025 09:46
@webdevred webdevred merged commit 9daeecd into master Sep 20, 2025
13 checks passed
@webdevred webdevred deleted the support-vertex-detection branch September 20, 2025 10:00
This was referenced Oct 2, 2025
webdevred added a commit that referenced this pull request Oct 5, 2025
Updated examle files to showcase how the recent changes to Transformation code affect jbeam. The major changes to transformation code are in #69

The changes to transformation which are related to this new example files are the following:

- Transformation reimplemenation: before errors were handled less safely and meta groups interrupting vertex groups was not supported #2
- Transformation refactoring: split Transformation into different files and added improved how comments and metadata is handled when moving vertices #41
- Support vertex detection: implemented detection of support vertices and automatically moving them to their own tree  #52
- Associate comments without newline before with prior node #56
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.

2 participants