search

Bioconductor / BioC2020

BioC2020: Where Software and Biology Connect
http://bioc2020.bioconductor.org/
12 stars 11 forks source link

Docker link does not agree with workshop DESCRIPTION #76

Closed LiNk-NY closed 4 years ago

LiNk-NY commented 4 years ago

Hi Mikhail @mdozmorov and Lorena @lpantano

The Docker link in the workshops page for Multi-omic Integration and Analysis of cBioPortal and TCGA data with MultiAssayExperiment does not agree with the DESCRIPTION file in the workshop:

https://github.com/waldronlab/MultiAssayWorkshop/blob/0501f784d83eeb236cc06e2e3bbd86bb647ea4ed/DESCRIPTION#L49

DockerImage: mr148/multiassayworkshop:latest

For some reason, the link points to waldronlab/multiassayworkshop. Thank you for your help. -Marcel

csoneson commented 4 years ago

We have the same issue for our workshop (iSEE: Interactive visualization of SummarizedExperiment objects), where the GitHub username != the Docker Hub username.

Workshop: https://github.com/iSEE/iSEEWorkshop2020 Docker image: iseedevelopers/iseeworkshop2020:latest

Thanks Charlotte

mdozmorov commented 4 years ago

Thank you for pointing that out. This is because the table is manually created. The current pull request fixes both issues, as well as corrects titles. But in retrospect, we should have a package that would pull the correct information from the DESCRIPTION files and generate the table automatically. Can this be an idea for a hackathon?

Mikhail

On Mon, Jun 8, 2020 at 12:48 PM Charlotte Soneson notifications@github.com wrote:

We have the same issue for our workshop (iSEE: Interactive visualization of SummarizedExperiment objects), where the GitHub username != the Docker Hub username.

Workshop: https://github.com/iSEE/iSEEWorkshop2020 Docker image: iseedevelopers/iseeworkshop2020:latest

Thanks Charlotte

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Bioconductor/BioC2020/issues/76#issuecomment-640745951, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGTFMPGY6EINMHZ3X6V6IDRVUI7JANCNFSM4NYOHVWQ .

mdozmorov commented 4 years ago

Pull request #77 closes it, when merged

LiNk-NY commented 4 years ago

Hi Mikhail, @mdozmorov I'm not sure a package is necessary. Perhaps it can be done using an .Rmd file to generate the .md file. I can give it a try some time today. -Marcel

mdozmorov commented 4 years ago

Thank you, Marcel. I'm not sure about the best way, have been thinking about a function that accepts a list of GitHub URLs, pulls workshop information from the DESCRIPTION files, and compiles the Markdown table. I can't get to it this week, please, help if you can. Mikhail