Skip to content

chore: add Liva citizenIDs to ids csv #256

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.

Sign up for GitHub

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

Open
K-Beicher wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore: add Liva citizenIDs to ids csv #256

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
228e2e5
chore: add Liva citizenIDs to ids csv
K-Beicher Oct 15, 2026
7349496
chore: ✏️ automatic pre-commit hook fixes
pre-commit-ci[bot] Oct 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 25 additions & 25 deletions raw/ids.csv
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,25 +1,25 @@
participant_id;record_id_s;id_number_pre;myfood24_id
001;1;i1001;01-4
002;2;i1007;02-4
003;3;i1027;03-4
004;4;i1020;04-4
005;5;i1021;05-4
006;6;i1042;06-4
007;7;i1004;07-4
008;8;i1028;08-4
009;9;i1025;09-4
010;10;i1045;10-4
011;11;i1057;11-4
012;12;i1052;12-4
013;13;i1053;13-4
014;14;i1034;14-4
015;15;i1073;15-4
016;16;i1033;16-4
017;17;i1086;17-4
018;18;i1074;18-4
019;19;i1096;19-4
020;20;i1077;20-4
021;21;i1092;21-4
022;22;i1115;22-4
023;23;i1102;23-4
024;24;i1113;24-4
participant_id;record_id_s;id_number_pre;myfood24_id;liva_citizenid

@signekb signekb Oct 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
participant_id;record_id_s;id_number_pre;myfood24_id;liva_citizenid
participant_id;record_id_s;id_number_pre;myfood24_id;liva_citizen_id

@K-Beicher K-Beicher Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm trying to keep to the original from the source. In my eyes this is just good data practice, as this is meant to be the table that will help identifying the link IDs from a variety of systems.

@signekb signekb Oct 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

And maybe even change id_number_pre to pre_number_id, so all ID's end with _id? If you agree, you can add another PR with that since that's unrelated to adding ID's from Liva 😊

@K-Beicher K-Beicher Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Same as above, the first couple of IDs comes directly from REDCap, actually, maybe we should prefix them with exactly that...

@lwjohnst86 lwjohnst86 Oct 16, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

During processing we can rename them to all end in _id. And in the end all of these ids will be consolidated into a single participant_id, regardless of data source.

1;1;i1001;01/Apr;104521
2;2;i1007;02/Apr;104537
3;3;i1027;03/Apr;104762
4;4;i1020;04/Apr;104764
5;5;i1021;05/Apr;104765
6;6;i1042;06/Apr;106523
7;7;i1004;07/Apr;104767
8;8;i1028;08/Apr;104768
9;9;i1025;09/Apr;104769
10;10;i1045;10/Apr;106454
11;11;i1057;11/Apr;106520
12;12;i1052;12/Apr;106524
13;13;i1053;13/Apr;106525
14;14;i1034;14/Apr;106526
15;15;i1073;15/Apr;107606
16;16;i1033;16/Apr;108557
17;17;i1086;17/Apr;109298
18;18;i1074;18/Apr;109677
19;19;i1096;19/Apr;111818
20;20;i1077;20/Apr;114308
21;21;i1092;21/Apr;114442
22;22;i1115;22/Apr;132334
23;23;i1102;23/Apr;120152
24;24;i1113;24/Apr;122214
Loading