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

expose exportBlankForGrayFormStatus parameter in redcap_read_* function #386

Closed
greg-botwin opened this issue Mar 4, 2022 · 2 comments
Closed

Comments

@greg-botwin
Copy link

Hello, Thank you all for your support in building a high value package!

I am happy to open a PR for what I think is a fairly simple addition, but I wanted to post the issue here first.

Would it be possible to expose the exportBlankForGrayFormStatus parameter directly from theredcap_read_oneshot function?

Per the REDCap API Documentation

exportBlankForGrayFormStatus: TRUE, FALSE [default] - specifies whether or not to export blank values for instrument complete status fields that have a gray status icon. All instrument complete status fields having a gray icon can be exported either as a blank value or as "0" (Incomplete). Blank values are recommended in a data export if the data will be re-imported into a REDCap project

Thanks for your support and feedback

@wibeasley
Copy link
Member

wibeasley commented Jul 23, 2022

Thanks @greg-botwin for adding that code and then waiting for a few months. We just updated the server and can now test this parameter. Are you still willing to write 3 tests? If so, here are the three form statuses for the five subjects in the standard, non-longitudinal dataset. (I don't understand the "gray icon" thing. Will this dataset work?):
image

So for each function (ie, redcap_read_oneshot(), redcap_read(), redcap_read_oneshot_eav()) that you added the parameter, write test passing the non-default parameter. Tell me if you have questions or have lost interest.

Then please

@wibeasley
Copy link
Member

@greg-botwin, I was doing stuff in the package and just wrapped up this PR. Please add any additional person info to the DESCRIPTION file you'd like (eg, email, orcid). Thanks again.

I added a new fake-data project to test this:

image

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

No branches or pull requests

2 participants