Skip to content
This repository was archived by the owner on Sep 17, 2019. It is now read-only.
This repository was archived by the owner on Sep 17, 2019. It is now read-only.

does do_compound work / general performance improvement? #8

Description

@mikkokotila

I did some testing comparing the results with diff for begining of do_compound and end of do_compound, and it seems that at least with the test data (some snippets of Rinchen Terdzo) there was no substantial difference. do_compound takes almost half of processing time though. So the question I have is if do_compound have been tested properly in the sense of quantifying the way it changes the output. Because of the change is marginal, but performance is significant, it might be something to think about.

In case the current implementation yields significant functional value, then I think it would be important to go through the code and see how it can be optimized. I found some very quick wins such as not doing len() inside list comprehensions etc. I did not get that far yet with the code, but generally, if there are a list comprehensions for comparing two list of strings, set intersection comparison is generally much faster.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions