Skip to content

Conversation

rchateauneu
Copy link
Contributor

The performance test does:
g.parse("orkg.nt", format="nt")

Windows:
14.71s => 12.20 : 17% faster
Linux: 13.12 => 12.45: 5% faster.

Please note that about 15% of time is spent in codecs.readline, implying UTF-8 conversions due to opening the file in "rb" mode.

@rchateauneu
Copy link
Contributor Author

I mistakenly added extra commits in this PR. Only 9b562bc and c1377a0 are relevant.

@white-gecko
Copy link
Member

But 9b562bc and c1377a0 do not look very relevant

@rchateauneu
Copy link
Contributor Author

Yes:

  • 9b562bc is simply the merge.
  • c1377a0 is the removal of an assert, but it does not harm.

@rchateauneu rchateauneu marked this pull request as draft May 16, 2021 13:03
@rchateauneu
Copy link
Contributor Author

This change is halted for the moment because their might be interferences with:

However, some of optimizations it suggests, are now split into several small PRs, which are ready for merging, independently:

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.

2 participants