Skip to content

fixed the issue where x and y column would be added regardless of the columnMode while exporting to csv#3215

Merged
j9ac9k merged 1 commit intopyqtgraph:masterfrom
virgile-hernicot:export-csv-fix-xyyy
Mar 24, 2025
Merged

fixed the issue where x and y column would be added regardless of the columnMode while exporting to csv#3215
j9ac9k merged 1 commit intopyqtgraph:masterfrom
virgile-hernicot:export-csv-fix-xyyy

Conversation

@virgile-hernicot
Copy link
Copy Markdown
Contributor

@virgile-hernicot virgile-hernicot commented Jan 8, 2025

Issue

fixed the issue where x and y column would be added regardless of the columnMode while exporting to csv.

Basically fixes #2747.

Tested environment(s)

PyQtGraph version: pyqtgraph-0.13.7
Qt Python binding: PyQT6
Python version: 3.12.7
NumPy version: 2.1.2
Operating system: Windows 11

@virgile-hernicot virgile-hernicot changed the title fixed the issue where x and y column would be added regardless for the columnMode while exporting to csv fixed the issue where x and y column would be added regardless of the columnMode while exporting to csv Jan 8, 2025
Copy link
Copy Markdown
Member

@ixjlyons ixjlyons left a comment

Choose a reason for hiding this comment

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

Looks good to me.

It'd be nice to have a test - would you be interested in adding a test with the (x,y,y,y) option to tests/exporters/test_csv.py?

@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Mar 24, 2025

Closing and Opening to trigger a new CI run.

@j9ac9k j9ac9k closed this Mar 24, 2025
@j9ac9k j9ac9k reopened this Mar 24, 2025
@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Mar 24, 2025

Thanks for the PR @virgile-hernicot !

@j9ac9k j9ac9k merged commit 4559ed1 into pyqtgraph:master Mar 24, 2025
56 of 64 checks passed
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.

CSVExporter (x,y,y,y)

3 participants