Skip to content

Conversation

mennodekker
Copy link

Added the variable labels to the dataframe
For meta data I added simple descriptions, variables get their own label and the f.variables for codelists get the codelist name.

Extra metadata
ProtocolID / Site ID
Date of birth or Year of birth
Person ID
Secondary ID
Status for Subject, Event & CRF
Event start/end date + location
Interviewer name and date for CRF

Fixed the partial date
It was output as a number, shopwing essentially just the year

@mennodekker mennodekker changed the title Added variable labels, extra metadata, fixed partial date R export: Added variable labels, extra metadata, fixed partial date Jun 25, 2019
@mennodekker
Copy link
Author

I will commit some improvements later, so please don't merge yet.

@lindsay-stevens
Copy link
Owner

Thanks for these. I have a couple of minor requests - could you please:

  • split reformatting to a separate commit - it's hard to see what is actually changed in 8a9868f.
  • add an example before / after e.g. as is done for the sas_singular XSL, to show what's fixed or changed in terms of output. Although I think the sas_singular example file doesn't have any PDATE items.

@mennodekker
Copy link
Author

Thanks for the feedback. Should I move to a new folder like the sas_singular so I can separate the example and include a readme? Suggestion: r_with_factors

@lindsay-stevens
Copy link
Owner

Sounds good, thanks 👍

@GerbenRienk
Copy link

Hi Menno and Lindsay,
Great work!
I'm looking into the possibility of using the r-xslt in LibreClinica and I see that Menno's pull-request hasn't been merged. Any particular reason that I should know of?
Gerben Rienk

@lindsay-stevens
Copy link
Owner

Hi @GerbenRienk, I haven't merged this mainly because there was no comment to say it was finished or at least ready for another look. Given the long pause, this branch may well be ready to use. I will probably not have time to check that it works for a while.

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.

3 participants