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

Optimise time taken to get raw graph #2312

Merged
merged 2 commits into from
Jun 6, 2017

Commits on Jun 2, 2017

  1. Optimise time taken to get raw graph

    The main change is the restructure of the variable
    `SuiteConfig().edges`. It is now a dict. Each key is a unique sequence,
    and each value is a set of `(left, right, suicide, conditional)` tuples.
    This change allows `get_graph_raw` to look at each unique sequence once
    instead of multiple times. There is also caching at various points to
    reduce the number of expensive calls to regular expression matching and
    date-time functionality.
    
    Other minor changes include:
    * Remove unused variables.
    * Use compiled regular expressions where possible.
    * Reduce usage of complex regular expressions.
    * Remove edge and graphnode objects, which were not cheap to initialise.
    * New GraphNodeParser object to manage and cache graph node parsing.
    * The `recursive_replace` internal function is replaced with a loop at a
      level up.
    matthewrmshin committed Jun 2, 2017
    Configuration menu
    Copy the full SHA
    74d946e View commit details
    Browse the repository at this point in the history

Commits on Jun 6, 2017

  1. Configuration menu
    Copy the full SHA
    8c2444b View commit details
    Browse the repository at this point in the history