Add Mixed Repeat/Nonrepeat Instrument Support

[rsh52]

Description

This PR seeks to add a new parameter to read_redcap() which will allow users to override the check in place that stops data exports for REDCap projects where instruments are detected to be both repeating and nonrepeating. The background rationale for not allowing this was to keep to tidy data principles.

For the purposes of this PR and documentation, here are my definitions for some things:

• Mixed Structure: One of the supertibble columns is called structure, and has only ever been "nonrepeating" / "repeating" for values. I wanted to keep this terminology so some of the updated functions use "structure" / "mixed structure"
• Mixed Instrument: An instrument that has a mixed structure

~Keeping this in draft for now until TODOs are addressed.~ ✅

Proposed Changes

List changes below in bullet format:

• Add param enable_repeat_nonrepeat to read_readcap() and lower-level function clean_redcap_long()
• Move shared logic in check_repeat_and_nonrepeat() to new function get_mixed_structure_fields()
• Wrap get_mixed_structure_fields() in handler function for converting nonrepeat parts of mixed structure instruments to repeating ones with a single instance via convert_mixed_instrument()
• Update tests
• Update test_creds.R (We now have a new database where a new env variable is needed: REDCAPTIDIER_MIXED_STRUCTURE_API)
• Update NEWS and version (currently considering this a minor update to 1.1.0)
• Update Diving Deeper article with new section on mixed structure instruments
• Update structure column in supertibble to say "mixed" for mixed structure instruments

Remaining TODOs:

• [x] ~Determine if we should add a new category to structure in the supertibble~
  • Currently this is either "repeating" or "nonrepeating" but we could open this further to say "mixed" or "repeating/nonrepeating"
• [x] ~Update Diving Deeper vignette/article (holding off until we're happy with logic choices and syntax)~

Issue Addressed

Addresses #126 Addresses #169

PR Checklist

Before submitting this PR, please check and verify below that the submission meets the below criteria:

• [x] New/revised functions have associated tests
• [x] New/revised functions that update downstream outputs have associated static testing files (.RDS) updated under inst/testdata/create_test_data.R
• [x] New/revised functions use appropriate naming conventions
• [x] New/revised functions don't repeat code
• [x] Code changes are less than 250 lines total
• [x] Issues linked to the PR using GitHub's list of keywords
• [x] The appropriate reviewer is assigned to the PR
• [x] The appropriate developers are assigned to the PR
• [x] Pre-release package version incremented using usethis::use_version()

Code Review

This section to be used by the reviewer and developers during Code Review after PR submission

Code Review Checklist

• [ ] I checked that new files follow naming conventions and are in the right place
• [ ] I checked that documentation is complete, clear, and without typos
• [ ] I added/edited comments to explain "why" not "how"
• [ ] I checked that all new variable and function names follow naming conventions
• [ ] I checked that new tests have been written for key business logic and/or bugs that this PR fixes
• [ ] I checked that new tests address important edge cases

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1206635877717263

[rsh52]

I think this is ready for review now, if I can get @ezraporter to review code (and the rest 😄 ) and @skadauke to review the vignette / glossary updates and the error message:

(note that the "Mixed Structure Instruments" hyperlink doesn't work yet until we've published this)

[skadauke]

I think this is ready for review now, if I can get @ezraporter to review code (and the rest 😄 ) and @skadauke to review the vignette / glossary updates and the error message:

(note that the "Mixed Structure Instruments" hyperlink doesn't work yet until we've published this)

I like the glossary, vignette, and error message updates! 👍

[rsh52]

Run our timing benchmarks to make sure the latest change didn't mess things up. (Can we also add identifiers to the microbenchmark_results.csv so it's easy to line things up when we add new redcaps?)

Updated run results and added column outputs for the database description and source (ouhsc / redcaptidier).

[ezraporter]

Run our timing benchmarks to make sure the latest change didn't mess things up. (Can we also add identifiers to the microbenchmark_results.csv so it's easy to line things up when we add new redcaps?)

Updated run results and added column outputs for the database description and source (ouhsc / redcaptidier).

Thanks!