Skip to content

Session.virtualfile_to_dataset: Add 'header' parameter to parse column names from table header - #3117

Merged
seisman merged 9 commits into
mainfrom
gmtdataset/header
Apr 18, 2024
Merged

Session.virtualfile_to_dataset: Add 'header' parameter to parse column names from table header#3117
seisman merged 9 commits into
mainfrom
gmtdataset/header

Conversation

@seisman

@seismanseisman commented Mar 18, 2024

Copy link
Copy Markdown
Member

Address #2730 (comment).

Add the header parameter to parse column names from table header. It's behavior is similar to the pd.read_csv's header parameter.

@seismanseisman changed the title POC: Parse column names from data headerGMT_DATASET: Add 'header' parameter to parse column names from table headerMar 30, 2024
@seisman
seisman marked this pull request as ready for review March 30, 2024 11:56
@seismanseisman changed the title GMT_DATASET: Add 'header' parameter to parse column names from table headerSession.virtualfile_to_dataset: Add 'header' parameter to parse column names from table headerMar 30, 2024
@seismanseisman added enhancement Improving an existing feature needs review This PR has higher priority and needs review. labels Mar 30, 2024
@seismanseisman added this to the 0.12.0 milestone Mar 30, 2024
@seismanseisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Apr 7, 2024
@seisman

Copy link
Copy Markdown
MemberAuthor

/format

@seisman

Copy link
Copy Markdown
MemberAuthor

/format

@seisman
seisman merged commit 1746c04 into mainApr 18, 2024
@seisman
seisman deleted the gmtdataset/header branch April 18, 2024 12:20
@seismanseisman removed the final review call This PR requires final review and approval from a second reviewer label Apr 18, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementImproving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@seisman@michaelgrund@actions-bot