Skip to content

Add adaptive sampling bed files in grch37 and hg38 projections#127

Merged
Kdreval merged 9 commits into
masterfrom
adaptive_sampling_bed
Jul 2, 2026
Merged

Add adaptive sampling bed files in grch37 and hg38 projections#127
Kdreval merged 9 commits into
masterfrom
adaptive_sampling_bed

Conversation

@gbanco4

@gbanco4 gbanco4 commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

This PR adds the BED panel of adaptive sampling targets to GAMBLR.data. Both genome builds (GRCh37 and hg38) have a version of this file.
The actual .bed files have been added to inst/extdata/
.rda files were created using usethis::use_data()
Loader blocks for these files were place in DATASET.R
data.R was updated with documentation for these objects
.Rd Roxygen2 documentation files were created for these objects inside man/

Comment thread R/data.R Outdated
#' A data frame with 313 rows and 6 columns.
#' \describe{
#' \item{chromosome_name}{The chromosome the region resides on, without chr prefixes}
#' \item{start_position}{Start coordinate for the region ([0-based])}

@Kdreval Kdreval Jul 2, 2026

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.

Both documentations for grch37 and hg38 have a hyperlink that points to nothing. The embedded link is not supposed to be missing target. If this is not supposed to be a link, the square brackets should be removed.
Here is more details about the error this is causing: https://github.com/morinlab/GAMBLR.data/actions/runs/28611065680/job/84843373897#step:6:122

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Should be fixed in this new push

@Kdreval
Kdreval merged commit 3ebb502 into master Jul 2, 2026
1 check passed
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