Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change to the CSV export #861

Closed
CaptainBrad opened this issue Jan 6, 2023 · 4 comments
Closed

Change to the CSV export #861

CaptainBrad opened this issue Jan 6, 2023 · 4 comments
Assignees
Labels
ToExplore Label for internal use only

Comments

@CaptainBrad
Copy link

Currently, if you ask for 'one row per file', any additional identified formats are added as extra columns - but they are not given a column header. Can this be included, e.g. PUID2 etc. The missing header makes the csv difficult to use in Excel and also for scripting where you have to work around the missing headers (e.g. Python pandas read_csv). I'm using 6.5.2.

@CaptainBrad CaptainBrad added HIGH Priority: High and removed HIGH Priority: High labels Jan 6, 2023
@ross-spencer
Copy link

@CaptainBrad this looks like it's connected to: #118 there's some additional discussion there.

@CaptainBrad
Copy link
Author

Thanks for the heads up @ross-spencer , we will look into 118.

@CaptainBrad CaptainBrad added the ToExplore Label for internal use only label Jan 13, 2023
@paulwellsagilej
Copy link
Contributor

I understand the problem now and I have investigated and started making changes to add the missing header values. The number of columns will vary from one CSV file to another, but each file will be valid within itself. The number of values in each row will match the number of values in the header row.

@paulwellsagilej
Copy link
Contributor

Existing tests pass now. Working on new tests to check new behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ToExplore Label for internal use only
Projects
None yet
Development

No branches or pull requests

4 participants