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.

Sign up for GitHub

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

Jump to bottom

Ben plan #20

Closed
wants to merge 74 commits into from
Closed

Ben plan #20

wants to merge 74 commits into from

Conversation

benrayden37
Copy link
Collaborator

work in progress

chendaniely and others added 30 commits March 18, 2020 11:26
@chendaniely
Initial commit
a45cb3c
@chendaniely
short blurb about dan
b11306f
@gitchrissi
added Chrissi Taylor
1772af5
@chendaniely
fix Chrissi's markdown syntax for her name
e387ba6
@geckko123
Added to readme
795e69b
@kelsieking23
work in progress - webscrper to download ligand pdb files
ba5121b
@kelsieking23
requirements for scrape.py
3c3d596
@Makhsuda
Add Makhsuda in README.md file
bda2d20
@Makhsuda
Edit text for user Makhsuda
3794773
@sjain1701
Added Somya's bio
221b375
@chendaniely
move @kelsieking23 scrape.py into bbl project template folder
492089d
@chendaniely
create data folders
039d10b
@mitchdolby
Mitch Dolby
acd5408 
Freshman at Virginia Tech majoring in geography. Interested in geospatial sciences.
@mitchdolby
Mitch Dolby
b26b254 
Freshman at Virginia Tech majoring in geography. Interested in geospatial sciences.
@mitchdolby
Update README.md
9ae8476
@benrayden37
Added my name and major to readme
f21a42f
@chendaniely
create environment files for pip and conda
0f255b9
@chendaniely
Merge branch 'master' of github.com:databridgevt/covid19
b0ff066 
* 'master' of github.com:databridgevt/covid19:
  Added my name and major to readme
  Update README.md
  Mitch Dolby
  Mitch Dolby
@chendaniely
link to GH iss how to remove conda prefix
f3b844d
@chendaniely
add kaggle package (from conda-forge)
6ad2193
@LoveishSarolia
Update README.md
5ebc034
@chendaniely
readme to data links + makefile to download kaggle
66e56ac
@chendaniely
Merge pull request #2 from databridgevt/get_kaggle
b0040ee 
Readme to data links + Makefile to download kaggle
@chendaniely
remove linux specific conda packages
ddd4ee8
@sjain1701
Initial question about risk factors
d870153
@sjain1701
initial question
08b401f
@sjain1701
question updated with how to identify risk factors
17e420a
@chendaniely
manual list of packages instead of exporting all of conda env
ac7105b
@chendaniely
Merge pull request #11 from databridgevt/simplify_env_yml
c3c66b9 
manual list of packages instead of exporting all of conda env
@chendaniely
fix pyprojroot verion, add make to environment
d79374b
@chendaniely
Copy link
Member

Cool thanks or the PR! now you just gotta actually write something :D

chendaniely
chendaniely reviewed Apr 7, 2020
View reviewed changes
analysis/db/ben/coronavirus_V1.Rmd Outdated

```{r}
library(jsonlite)
yelp <- fromJSON("/Users/benrayden/Desktop/CDCResearch/CORD-19-research-challenge/comm_use_subset/comm_use_subset/0a00a6df208e068e7aa369fb94641434ea0e6070.json")
Copy link
Member

Choose a reason for hiding this comment

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

This is a hard coded path. and needs to be changed to a relative one for this project

chendaniely and others added 10 commits April 8, 2020 22:38
@chendaniely chendaniely mentioned this pull request Apr 9, 2020
Closed
6 tasks
@chendaniely
Copy link
Member

chendaniely commented Apr 9, 2020 •
edited
Loading

@benrayden37 have you written R functions and/or R packages before? might be worth converting your work into a more general tool for the lab.

@chendaniely
Copy link
Member

chendaniely commented Apr 23, 2020 •
edited
Loading

Are you planning to use this PR for #32 as well? If so:

  1. The README.md file doesn't contain a research question or focus, it's just listing things you're planning to do
    • E.g., why are you plannig to run all those methods?
  2. The Rmd document has a non-working data load call and still has the default boilerplate text

@chendaniely chendaniely mentioned this pull request May 22, 2020
Closed
@chendaniely
Copy link
Member

chendaniely commented May 22, 2020 •
edited
Loading

@benrayden37 I messed something up while fixing up some git things (see #53):

can you run this command in your repo on your computer

git log --oneline --graph --decorate --all

and send me the results?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chendaniely chendaniely chendaniely left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@benrayden37 @chendaniely @gitchrissi @geckko123 @kelsieking23 @Makhsuda @sjain1701 @mitchdolby @LoveishSarolia