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

Timeline / interest for not yet implemented foverlaps() functionality #3002

Open
sritchie73 opened this issue Aug 20, 2018 · 3 comments
Open

Timeline / interest for not yet implemented foverlaps() functionality #3002

sritchie73 opened this issue Aug 20, 2018 · 3 comments

Comments

@sritchie73
Copy link
Contributor

@sritchie73 sritchie73 commented Aug 20, 2018

There are a number of arguments in the foverlaps() function that are not yet implemented:

  • type = "equal" - #3416
  • maxgap
  • minoverlap

Along with a few requested features:

  • #979 non integer support for minoverlap
  • #1355 foverlaps() should work without setkey() beforehand
  • #1975 return length of overlap
  • #2645 foverlaps() could work with factor levels

There will be a lesson on the foverlaps() function in the DataCamp course on joins, so I'm wanting to gauge whether its worth trying to implement these features before the course is released. Are there obvious technical barriers that have prevented these features from being implemented? Or has there been a lack of interest from the community and developers, e.g. given the availability of GenomicRanges?

@msummersgill
Copy link

@msummersgill msummersgill commented Aug 21, 2018

Related: #3007 foverlaps error message could be more informative

@arunsrinivasan
Copy link
Member

@arunsrinivasan arunsrinivasan commented Feb 16, 2019

@sritchie73 thanks for this. I wrote foverlaps in about 3-4 hrs (for EARL conference) and have not since looked at the features that've been left out .. Will take a look.

@Atrebas
Copy link

@Atrebas Atrebas commented Feb 16, 2019

Some potentially helpful other references: bedops, bedtools, tidygenomics.

@MichaelChirico MichaelChirico mentioned this issue Feb 19, 2019
6 of 45 tasks complete
@MichaelChirico MichaelChirico added the High label May 30, 2020
@jangorecki jangorecki removed the High label Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.