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

Reasoner challenge #63

Open
rubensworks opened this issue Apr 11, 2022 · 6 comments
Open

Reasoner challenge #63

rubensworks opened this issue Apr 11, 2022 · 6 comments

Comments

@rubensworks
Copy link
Member

@jeswr Have you seen this reasoning challenge pass by?
https://gist.github.com/justin2004/f9d07adf4e7c2c422be3e0ba92f278d2

Might be interesting to try out in your implementation(s).

@jeswr
Copy link
Member

jeswr commented Apr 11, 2022

Haven't seen this yet @rubensworks - definitely worth a try - I suspect the performance issues in the AsyncIterator package need to be released before these components will be competitive in it

@jeswr
Copy link
Member

jeswr commented Apr 11, 2022

I just tested the following segment

const { Parser, Store } = require('n3');
const fs = require('fs');

const parser = new Parser();
const store = new Store();

const now = performance.now();
parser.parse(fs.createReadStream('./tbox.ttl'), (e, quad) => {
  if (!quad) {
    const end = performance.now();
    console.log(end - now, store.size);
    return;
  }
  store.add(quad)
})

it takes 21s (and if we don't load it into the store it is 5s), so the time to parse and add data in the N3 package still needs to be improved in order to be competitive with loading times accounted for @rubensworks @RubenVerborgh

@rubensworks
Copy link
Member Author

it takes 21s (and if we don't load it into the store it is 5s),

So that's just for parsing and loading, right?

Based on the loading times reported in that gist (6s, 36m, 2m), that seems quite competitive though.

@jeswr
Copy link
Member

jeswr commented Apr 11, 2022

So that's just for parsing and loading, right?

Yep

that seems quite competitive though.

True (I was only comparing to the 6s one) - though it would be nice if it could be brought closer to the 1sec state-of-the-art of RDFOX

@RubenVerborgh
Copy link

True (I was only comparing to the 6s one) - though it would be nice if it could be brought closer to the 1sec state-of-the-art of RDFOX

This is JavaScript though; we don't have the low-level access that they do. I suspect most of the time is storage, not parsing.

@rubensworks
Copy link
Member Author

There's also the hidden variable of different system capabilities to take into account here. Good comparisons will require running all systems on the same machine.

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

No branches or pull requests

3 participants