Skip to content

VPTS CSV 1.1#50

Open
adokter wants to merge 3 commits into
mainfrom
vpts-csv-1.1
Open

VPTS CSV 1.1#50
adokter wants to merge 3 commits into
mainfrom
vpts-csv-1.1

Conversation

@adokter

@adokter adokter commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

add optional height_reference column, addressing issue #49

@adokter adokter self-assigned this Nov 20, 2025
@adokter adokter requested a review from peterdesmet November 20, 2025 21:29
Comment thread vpts-csv-table-schema.json Outdated

@bart1 bart1 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

just a quick think I noticed as now I need to round down radar_height

Co-authored-by: bart1 <1662852+bart1@users.noreply.github.com>
@adokter

adokter commented Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

@peterdesmet @bart1 I will start using this updated format in the IRAC conference workshop, so might be good to merge this

@peterdesmet

Copy link
Copy Markdown
Member

@adokter Is any software relying on this being merged?

I'd prefer to discuss this at our pre-IRAC meeting first. Maybe there are other changes we want to make to VPTS-CSV and I think it's better to limit the number of releases.

@adokter

adokter commented Jun 10, 2026 via email

Copy link
Copy Markdown
Contributor Author

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