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

[BUG] Sometimes, downloading packages from Bioconductor fails. #4

Closed
MrHedmad opened this issue Sep 21, 2022 · 2 comments
Closed

[BUG] Sometimes, downloading packages from Bioconductor fails. #4

MrHedmad opened this issue Sep 21, 2022 · 2 comments
Labels
bug Something isn't working needs triage This issue needs to be triaged

Comments

@MrHedmad
Copy link
Member

Describe the bug
Sometimes, the download of packages from Bioconductor (e.g. in the prep modules) fails, and the container crashes.

To Reproduce
Steps to reproduce the behavior:

  1. Install biotea
  2. Run the analysis presented in the paper
  3. When downloading pd.hugene.1.0.st.v1, sometimes the download cannot be run, an the resulting timeout causes a crash.

Expected behavior
Expected to download the package as normal.

Desktop:

  • OS: Arch Linux
  • BioTEA Version (Run biotea info biotea): 1.0.0
  • Docker engine version (Run docker --version): 20.10.17, build 100c70180f
  • BioTEA container version (if applicable): latest (1.0.1)
@MrHedmad MrHedmad added bug Something isn't working documentation Improvements or additions to documentation labels Sep 21, 2022
MrHedmad added a commit that referenced this issue Dec 19, 2022
@MrHedmad MrHedmad reopened this Dec 19, 2022
@MrHedmad MrHedmad added info A question was asked to the authors needs triage This issue needs to be triaged and removed documentation Improvements or additions to documentation info A question was asked to the authors labels Feb 3, 2023
@MrHedmad
Copy link
Member Author

MrHedmad commented Feb 3, 2023

It is unclear why, or how, the package download fails. Is it just an issue of bioconductor? Of our office internet? Of something else entirely?

Someone, please triage this!

@MrHedmad
Copy link
Member Author

I've taken another look at the packages involved and, in short, there is not much we can do. We read package data from (e.g.) read.celfiles and it automatically fetches the data that it needs to parse that specific model of array. If bioconductor dies, we cannot really change the internal function call.

If and when this becomes a really bad issue, we might think of something (download all data packages locally?), but not right now. Closing as wontfix.

@MrHedmad MrHedmad closed this as not planned Won't fix, can't repro, duplicate, stale Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs triage This issue needs to be triaged
Projects
None yet
Development

No branches or pull requests

1 participant