Skip to content
This repository was archived by the owner on Apr 7, 2022. It is now read-only.

Data: Wrangle for ImpactEd#3

Merged
avisionh merged 7 commits intowrangle-final-datafrom
data/wrangle
Jan 14, 2021
Merged

Data: Wrangle for ImpactEd#3
avisionh merged 7 commits intowrangle-final-datafrom
data/wrangle

Conversation

@avisionh
Copy link
Contributor

Summary

This branch wrangles the data so that it is in a format suitable for ImpactEd. In particular, they want it in a wider format so they can conduct their own analysis on it.

A lot of this code is inspired by @nacnudus approach in wrangle-final-data branch, indeed there is some duplication (we worked on our branches in parallel). I think the main difference is that his one formats it in a tidy manner whereas my one formats it so it's tailored to ImpactEd's data structure.

Checklists

General checks

  • Code runs
  • Developments are secure and ethical)
  • You have made proportionate checks that the code works correctly
  • Test suite passes
  • Assumptions, and caveats log updated (see docs/aqa/assumptions_caveats.md), if necessary
  • Minimum usable documentation written in the docs folder

Project-specific checks

These are additional checks determined necessary according to the analytical quality assurance plan
(see docs/aqa/aqa_plan.md).

  • Code runs.
  • The data is in wide format.
  • The logic for randomly allocating rank when we have duplicate rows but different response is sound (L25-31 in src/make_data/02_link_data.R)

This is so we can link up the data.
This is so we can get in a format ImpactEd are expecting.
This is inspired by Duncan's code.
This is so we can pivot wider without creating lists.

Also, export to csv for sharing.
This is to update packages used in scripts.
This is because the second script cannot call the first script (causes RStudio to abort). One suspects it is to do with Google Authorisation not being able to apply across scripts.

Move function to separate script so it can be tested more easily and to modularise code.
@avisionh avisionh mentioned this pull request Jan 10, 2021
8 tasks
@avisionh avisionh merged commit b04198e into wrangle-final-data Jan 14, 2021
@avisionh avisionh deleted the data/wrangle branch January 14, 2021 09:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant