KechrisLab / multiMiR

Development repository for the multiMiR database's R API
Other
19 stars 3 forks source link

Katerina's revisions #16

Closed mmulvahill closed 7 years ago

mmulvahill commented 7 years ago
  1. Send to Kevin as well, to keep him in the looop

  2. Can you reference our paper (provide link to PubMed) in the Vignette? Unless there is a rule about that. Maybe in the introduction when you introduce the webserver?

  3. You know more about making vignettes, but what is the rationale for some output that is displayed but not others? For example, should the output for other "db.ver", "db.tables", "apply(db.count[,-1], 2, sum)" etc. commands in the "Getting to know ..." section be displayed? You show output for other sections (like the examples).

  4. For the "Luciferase" and "cisplatin" searches, can the function handle case insensitive?

Minor comments:

  1. Some language edits in the "Getting to know ..." section, please us this instead

Among the 14 external databases, eight contain predicted miRNA-target interactions (DIANA-microT-CDS, ElMMo, MicroCosm, miRanda, miRDB, PicTar, PITA, and TargetScan), three have experimentally validated miRNA-target interactions (miRecords, miRTarBase, and TarBase) and the remaining three contain miRNA-drug/disease associations (miR2Disease, Pharmaco-miR, and PhenomiR). To check these categories and databases from within R, we have a set of four helper functions:

  1. Take out extra "}" for the "List miRNAs ..." header
mmulvahill commented 7 years ago

Done e9e068f