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

Thematic analysis output #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

ritazagoni
Copy link
Contributor

No description provided.

@ritazagoni ritazagoni requested a review from as2388 March 29, 2019 11:50
docker-run.sh Outdated Show resolved Hide resolved
src/lib/pipeline_configuration.py Show resolved Hide resolved
src/lib/pipeline_configuration.py Outdated Show resolved Hide resolved
src/auto_code_surveys.py Outdated Show resolved Hide resolved

# Output messages for thematic analysis
IOUtils.ensure_dirs_exist(icr_output_dir)
for plan in PipelineConfiguration.SURVEY_CODING_PLANS:
Copy link
Member

Choose a reason for hiding this comment

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

Why does this need to be incremental?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The file output, you mean? Because we needed to start thematic analysis before the data collection concluded. Does that answer your question?

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, on other projects we have been able to regenerate all of the outputs every time the pipeline has re-run (e.g. for the production files on which thematic analysis was done previously, we just overwrote the existing file with a new one that contained the newest data).

I don't think I know enough about this project to comment, @lukechurch

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The incremental data was coming from one survey. Due to delivery timelines, we started thematic analysis on Saturday of data from the survey sent on Friday, adding new data on Sunday.

@ritazagoni ritazagoni requested a review from as2388 April 9, 2019 07:59
@as2388 as2388 removed their request for review December 19, 2019 12:53
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.

2 participants