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

Paper notebooks #492

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

Paper notebooks #492

wants to merge 5 commits into from

Conversation

PGijsbers
Copy link
Collaborator

Added notebooks that:

  • helped identify/classify errors
  • cleaned the data, performed imputation for baselines
  • produced some of the figures and tables for our new paper

and updated the readme to refer to our new preprint.

@sebhrusen
Copy link
Collaborator

@PGijsbers nice additional material. thanks! I'll just try to run them and see if they're easily adaptable to user's results before approving.

It's not currently on the new website, so we refer the old readme instead.
docs/README.md Outdated Show resolved Hide resolved
docs/README.md Outdated Show resolved Hide resolved
Co-authored-by: Qingyun Wu <qw2ky@virginia.edu>
@PGijsbers PGijsbers added this to the 2.1 milestone Mar 3, 2023
@PGijsbers PGijsbers modified the milestones: 2.1, 2.2 May 30, 2023
@PGijsbers
Copy link
Collaborator Author

We'll release updated notebooks after we finish our revision that will be better documented, but I figured I'd update the PR since I noticed it is incomplete: the box_plot function was missing. Now the notebook successfully runs top-to-bottom and reproduces the plots. I also included some installation instructions that may help people install the right version of dependencies (as newer versions break parts of the notebook).

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.

None yet

3 participants