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

WIP::Speedup tabix histogram #1322

Closed
wants to merge 2 commits into from
Closed

WIP::Speedup tabix histogram #1322

wants to merge 2 commits into from

Conversation

nathandunn
Copy link
Contributor

Submitting a pull request

Please base your pull request based on the "dev" branch of JBrowse

The default branch is "master" but this contains the last stable release, and new pull requests should be based off of "dev"

Then set the base branch to "dev" on GitHub also

@nathandunn nathandunn self-assigned this Feb 28, 2019
@ghost ghost added the in progress currently being worked on label Feb 28, 2019
@nathandunn nathandunn changed the base branch from master to dev February 28, 2019 22:06
@nathandunn nathandunn requested a review from cmdcolin February 28, 2019 22:27
@nathandunn
Copy link
Contributor Author

@cmdcolin This works (definitely gives different results, but they are what I want for this).

I'm trying to test it on large data within Apollo to see if it actually makes any difference.

@nathandunn
Copy link
Contributor Author

For 28Mbps . . . performance is almost identical.

@nathandunn
Copy link
Contributor Author

nathandunn commented Mar 1, 2019

There are a few other issues, not really related to this, that I think are more performant worrying, mostly that they needs to be asynchronous calls to count things (instead of bring back anything).

screen shot 2019-02-28 at 3 52 12 pm

  1. provide an async fetch with callback for the histograms
  2. add countline method provide a countLines method tabix-js#16
  3. not caching

@nathandunn nathandunn changed the title Speedup tabix histogram WIP::Speedup tabix histogram Mar 1, 2019
@nathandunn nathandunn closed this Apr 21, 2019
@ghost ghost removed the in progress currently being worked on label Apr 21, 2019
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.

1 participant