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

Convert to httr2 #59

Open
wants to merge 5 commits into
base: devel
Choose a base branch
from
Open

Convert to httr2 #59

wants to merge 5 commits into from

Conversation

lshep
Copy link
Contributor

@lshep lshep commented Jan 13, 2025

Convert from httr to httr2

lshep added 5 commits January 3, 2025 11:28
 - download, head calls updated
 - TODO: need to pass proxy through properly to internal functions. This
    previously was not needed because set the proxy globally which is no longer
    an option
creat a helper function to get proxy information
@lshep
Copy link
Contributor Author

lshep commented Jan 13, 2025

If testing with hubs should use the changes and PR
Bioconductor/BiocFileCache#57
#59
Bioconductor/ExperimentHub#45

Copy link
Contributor

@LiNk-NY LiNk-NY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. I was able to download the resource behind a proxy:

### docker run -d --rm --name squid -p 3128:3128 ubuntu/squid
library(AnnotationHub)
proxy <- "http://localhost:3128"
setAnnotationHubOption("PROXY", proxy)
ah <- AnnotationHub()
# Assuming valid proxy connection through 'http://localhost:3128'
#  If you experience connection issues consider using 'localHub=TRUE'
# [ truncated ...]
ah[["AH69303"]]
GRanges object with 324754 ranges and 4 metadata columns:
                 seqnames      ranges strand |   source       type     score     phase
                    <Rle>   <IRanges>  <Rle> | <factor>   <factor> <numeric> <integer>
       [1] OOHO01000004.1   5896-6324      - |  ensembl transcript        NA      <NA>
       [2] OOHO01000004.1   5896-6324      - |  ensembl exon              NA      <NA>
       [3] OOHO01000004.1 10788-13607      - |  ensembl transcript        NA      <NA>
       [4] OOHO01000004.1 13277-13607      - |  ensembl exon              NA      <NA>
       [5] OOHO01000004.1 12737-12866      - |  ensembl exon              NA      <NA>
       ...            ...         ...    ... .      ...        ...       ...       ...
  [324750] OOHO01000042.1   3682-3884      + |  ensembl       exon        NA      <NA>
  [324751] OOHO01000042.1   5691-5804      + |  ensembl       exon        NA      <NA>
  [324752] OOHO01000042.1   7630-7716      + |  ensembl       exon        NA      <NA>
  [324753] OOHO01000042.1   9517-9625      + |  ensembl       exon        NA      <NA>
  [324754] OOHO01000042.1 11770-11805      + |  ensembl       exon        NA      <NA>
  -------
  seqinfo: 69 sequences from fAnaTes1.1 genome; no seqlengths

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

Successfully merging this pull request may close these issues.

2 participants