Bioconductor / ExperimentHub

Client to access ExperimentHub resources
https://bioconductor.org/packages/ExperimentHub
9 stars 13 forks source link

Documentation for `...` in ExperimentHub constructor missing? #16

Closed LiNk-NY closed 3 years ago

LiNk-NY commented 3 years ago

Hi Lori @lshep, Martin @mtmorgan

Is the use of the elipsis (...) documented somewhere? I don't see it in ?ExperimentHub::ExperimentHub? Is it ignored?

Thanks!

lshep commented 3 years ago

I have a sticky to go over creating hub packages and adding resources to the hubs I'll make it more generally to review all documentation.
The ... is passed down into .Hub() and then tonew()` . The ExperimentHub and AnnotationHub are extensions of the Virtual Hub class. I think it is currently ignored since all values of the hub class get defined but is needed for the placeholder value of .db_index that gets defined/overwritten later.

lshep commented 3 years ago

https://github.com/Bioconductor/ExperimentHub/commit/e6a4b73961a7d4aaef955e924164961c2e40b7e8