Skip to content

Support ragged arrays - #119

Merged
leonlan merged 3 commits into
mainfrom
ragged-array
Jun 21, 2024
Merged

Support ragged arrays#119
leonlan merged 3 commits into
mainfrom
ragged-array

Conversation

@leonlan

@leonlanleonlan commented Jun 21, 2024

Copy link
Copy Markdown
Member

This PR closes#108.

N-Wouda
N-Wouda reviewed Jun 21, 2024
View reviewed changes
Comment threadtests/parse/test_parse_vrplib.py
@leonlan
leonlan marked this pull request as ready for review June 21, 2024 07:32
@leonlan

leonlan commented Jun 21, 2024

Copy link
Copy Markdown
MemberAuthor

This is just a quick patch to support ragged arrays because it's handy for testing PyVRP with site-dependent VRP instances. After this, I think we should only add support #117 before starting the big rehaul #114.

N-Wouda
N-Wouda approved these changes Jun 21, 2024
View reviewed changes

@N-WoudaN-Wouda left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@leonlan
leonlan merged commit a7ff515 into mainJun 21, 2024
@leonlan
leonlan deleted the ragged-array branch June 21, 2024 07:55
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@N-WoudaN-WoudaN-Wouda approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Ragged arrays

2 participants

@leonlan@N-Wouda