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

abbreviations: DNN, CNN, RBM, DBN, etc #345

Closed
alxndrkalinin opened this issue Apr 24, 2017 · 8 comments
Closed

abbreviations: DNN, CNN, RBM, DBN, etc #345

alxndrkalinin opened this issue Apr 24, 2017 · 8 comments

Comments

@alxndrkalinin
Copy link
Contributor

Couldn't find any guidelines on abbreviations - are we going to introduce common ones like CNN in the beginning of the manuscript and use further on? Is there journal policy on this?

@agitter
Copy link
Collaborator

agitter commented Apr 24, 2017

Good question. We haven't decided anything yet. Defining the most common neural network abbreviations like those in the issue title in the introduction makes sense. Others can be defined at first use.

@cgreene let us know if you have thoughts.

@cgreene
Copy link
Member

cgreene commented Apr 24, 2017

NDC -> National Drug Codes (came up in #339)

ICD -> International Classification of Diseases

I figure we'll need to go through and define acronyms the first time they appear. If you encounter some, feel free to note them here.

@enricoferrero
Copy link
Contributor

enricoferrero commented Apr 25, 2017

NLP: natural language processing (#342) - Plan to introduce it in Categorize/EHR, I can see it's also used in Study/Metagenomics.
GWAS: genome-wide association study (#342) - Also used later on.

@agitter
Copy link
Collaborator

agitter commented May 10, 2017

RF -> random forest

May decide to always write it out in full, but we should be consistent throughout.

@agapow
Copy link
Contributor

agapow commented May 10, 2017

What I've spotted that appear to occur more than once:

EHR: electronic health record
RNN: recurrent NN
TF: transcription factor
MRI / fMRI: Magnetic resonance imaging
DBM: deep boltzman machine
DBN: deep belief network
MTL: multi-task learning

@agitter
Copy link
Collaborator

agitter commented May 14, 2017

AUC: area under the curve
ROC: receiver operating characteristic

Needed for #438 among others. This is also called auROC in parts of the review.

@agitter agitter mentioned this issue May 23, 2017
@agitter
Copy link
Collaborator

agitter commented May 23, 2017

I'm going through these in #498 and adding 👍 once I cover them.

@agitter
Copy link
Collaborator

agitter commented May 23, 2017

Instead of abbreviating random forest as RF, which I don't care for, I'm using square brackets to write it out in a quote. This will show up as a warning during builds, but we can ignore it.

agitter added a commit to agitter/deep-review that referenced this issue May 23, 2017
dhimmel pushed a commit that referenced this issue May 23, 2017
This build is based on
ff129e9.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/deep-review/builds/235172530
https://travis-ci.org/greenelab/deep-review/jobs/235172531

[ci skip]

The full commit message that triggered this build is copied below:

Consistency (#498)

* Reflow after proofreading

* Dashes

* auROC auPRC

* Standardize abbreviations, closes #345

* Fix Study header
dhimmel pushed a commit that referenced this issue May 23, 2017
This build is based on
ff129e9.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/deep-review/builds/235172530
https://travis-ci.org/greenelab/deep-review/jobs/235172531

[ci skip]

The full commit message that triggered this build is copied below:

Consistency (#498)

* Reflow after proofreading

* Dashes

* auROC auPRC

* Standardize abbreviations, closes #345

* Fix Study header
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

5 participants