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

Handling large BLAST results #682

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Commits on Sep 22, 2023

  1. Update package-lock.json

    Just ran npm install and it complained the lockfile format is old
    and updated it.
    tadast committed Sep 22, 2023
    Configuration menu
    Copy the full SHA
    efd945a View commit details
    Browse the repository at this point in the history

Commits on Sep 27, 2023

  1. Warn user if BLAST results are big

    Browsers might struggle to render large result sets. There's no
    reliable way to know, so pick an arbitrary size and warn the user,
    giving them a choice of downloading tabular data for local analysis
    or attempting to render the large result.
    tadast committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    3880ce4 View commit details
    Browse the repository at this point in the history
  2. Make report generation lazy

    Invoking a method that has side-effects (calculating and assigning values,
    generating files etc) as a part of the initializer makes the objects
    much harder to test and work with. When working with reports, make
    result generation explicit, rather than directly invoking it as soon
    as the object is initialized.
    tadast committed Sep 27, 2023
    Configuration menu
    Copy the full SHA
    e98a644 View commit details
    Browse the repository at this point in the history