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

joss/editing #102

Closed
soodoku opened this issue Nov 23, 2020 · 9 comments
Closed

joss/editing #102

soodoku opened this issue Nov 23, 2020 · 9 comments

Comments

@soodoku
Copy link

soodoku commented Nov 23, 2020

"The main goal of this model is to apply machine learning techniques to make projections of future areas at risk." -> expand to convey what risk means here, how it is measured, etc.

i couldn't easily find how the model is validated, any results we may see, etc. i would recommend explicitly linking a bunch of that via the readme.rst

@JannisHoch
Copy link
Owner

@soodoku I have updated the readme.md (9c702d7) .

Let me know if this clarifies (some of) your questions.

@JannisHoch
Copy link
Owner

additional contextual content in the notebooks was added in ed3fd13

@soodoku
Copy link
Author

soodoku commented Nov 28, 2020

  1. fix the spellings etc. "employes" -> "employs" by using something like Grammarly
  2. it is good to provide some additional details of the models: what are the Xs (what units), how does train/test split happen, how does hyper-param tuning happen, etc. and useful to give baseline results from your core example model, linking to the data.

@JannisHoch
Copy link
Owner

JannisHoch commented Nov 30, 2020

grammarly used to fix typos in paper, docs, and readme in commit 120f1c5 and b279c48

@JannisHoch
Copy link
Owner

the readme.md file was updated in 0936a8a.

it contains now a link to the example data stored on Zenodo. The ROC curve based on the example data (as computed in the notebooks) is shown in the readme.md now.

@JannisHoch
Copy link
Owner

i have put more contextual info and explanations about the models in the notebooks with c458f92.

@soodoku
Copy link
Author

soodoku commented Dec 1, 2020

thanks much @JannisHoch. I think it makes the package more usable more quickly. I am going to ping you again as I run my first model.

here's one portion that i am struggling w/ in the JOSS ms:

"CoPro is a novel, fully open, and extensible tool to combine the inter-disciplinary expertise required to make long-term projections of conflict risk associated with climatic and environmental drivers. "

I think that line is a bit vague. What is the precise value copro is providing is really useful to convey and something I have struggled to articulate. My hunch = "it is a python packages that makes it easier to build and validate models of how climate change predicts conflict locally. it makes it easier by providing some pre-packaged models that can work as starting points, some visualization functions, and ...." see if you can finish that and then put it in the paper and the readme.

@JannisHoch
Copy link
Owner

hi @soodoku, i have tried to make it clearer without being to specific on the technical details. see commit a31963e.

@soodoku
Copy link
Author

soodoku commented Dec 3, 2020

looks good

@soodoku soodoku closed this as completed Dec 3, 2020
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

No branches or pull requests

2 participants