Skip to content

Convert to httr2 - #59

Merged
lshep merged 7 commits into
Bioconductor:develfrom
lshep:httr2
Apr 22, 2025
Merged

lshep merged 7 commits into
Bioconductor:develfrom
lshep:httr2

Conversation

@lshep

@lshep lshep commented Jan 13, 2025

Copy link
Copy Markdown
Contributor

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

lshep commented Jan 13, 2025

Copy link
Copy Markdown
Contributor Author

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

@LiNk-NY LiNk-NY left a comment •

Copy link
Copy Markdown
Contributor

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

@lshep
lshep merged commit 8b9877d into Bioconductor:devel Apr 22, 2025
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