CLOSER-Cohorts / archivist-utilities

Tools for users of archivist
0 stars 0 forks source link

Names of exported files #14

Closed HayleyMills closed 3 months ago

HayleyMills commented 4 months ago

The DV and TV now contain the dataset names in the filenames, as there could be more than one, for consistency, can we include the questionnaire prefix in the name (-ccs01) for the QV and TQ files?

ollylucl commented 4 months ago

Do you mean, that for the QV and TQ files, we should generate a file for every unique questionnaire prefix, and the title of those files should be a concatenation of the questionnaire prefix,'-ccs01', and '_qv.txt' , e.g. heaf_17_fup_4-ccs01_qv.txt?

HayleyMills commented 4 months ago

So it would be heaf_17_fup_4_qv.txt

ollylucl commented 4 months ago

Why do you have (-ccs01) in the opening message on this issue? That's confusing me.

HayleyMills commented 4 months ago

Ah sorry it wasn't clear it was meant to be like minus the ccs01 which I think was in the excel files, but may not need to be see #15

ollylucl commented 4 months ago

I think I've implemented what you asked for, the qv and tq data are now in files named after the questionnaire prefixes. If there are multiple unique values for questionnaire prefixes in the input Excel file, there will be a text file generated for each one.

HayleyMills commented 4 months ago

Tested and working as expected.

ollylucl commented 3 months ago

When I use this file as input, no files are generated, because there's no questionnaire or dataset prefix columns present.

Can I assume these columns will be present, and that this file is not correctly formatted?

[sws_covid_pq1_all_mappings_wip.xlsx](https://github.com/user-attachments/files/16175213/sws_covid_pq1_all_mappings_wip.xlsx)

HayleyMills commented 3 months ago

When I use this file as input, no files are generated, because there's no questionnaire or dataset prefix columns present.

Can I assume these columns will be present, and that this file is not correctly formatted?

[sws_covid_pq1_all_mappings_wip.xlsx](https://github.com/user-attachments/files/16175213/sws_covid_pq1_all_mappings_wip.xlsx)

Yes