aws-cqc / aws-cqc/DeviceLayout.jl
Store CurvilinearPolygon curves as Vector{Segment{T}}
Open
2.0
enhancement
- Dominant language
- Julia
- Stars
- 67
- Forks
- 14
- Avg merge
- 1d 19h
- Merged PRs (30d)
- 24
Description
Right now CurvilinearPolygon has a `curves` field that's `Vector{<:Segment}`. This is awkward for multiple reasons. It should probably be `Vector{Segment{T}}`, which is technically a breaking change.
Contributor guide
Research direction
Start by locating CurvilinearPolygon and its curves field, then inspect the usages that depend on the current Vector{<:Segment} type. Update the affected representation and verify that the package’s existing behavior and tests remain valid after the breaking change.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- design
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 52/100