-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update complexity controls. #79
Conversation
**BREAKING**: Check output `format` parameter. Must be omitted, falsey, or "application/n-quads".
- Remove advanced "useNative" example. - Add common N-Quads input example.
- Add `maxWorkFactor` to calculate `maxDeepIterations` based on non-unique blank nodes. Default to `1`. - If `maxDeepIterations` is >= 0 then use it explicitly. - Remove default timeout signal. - Add docs. - Add tests. - Update async and sync versions. - Update tests to handle `computationalComplexity` test paramter and map it to `maxWorkFactor` adjustments. - Improve negative test handling.
Codecov Report
@@ Coverage Diff @@
## add-signal-support #79 +/- ##
======================================================
+ Coverage 93.59% 94.97% +1.38%
======================================================
Files 10 10
Lines 609 637 +28
======================================================
+ Hits 570 605 +35
+ Misses 39 32 -7
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Add support for generalized RDF `BlankNode` predicate during N-Quads serialization.
fb7d15c
to
029a88c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM modulo a couple of comments. Thanks!
Co-authored-by: Dave Longley <dlongley@digitalbazaar.com>
- Fix maxWorkFactor === 0 use case. - Improve error if N-Quads input not a string. - Fix calls without options. - Test error messages to ensure correct errors are thrown. - Remove default options not needed for testing. - Add various tests for complexity controls, edge cases, and coverage. - Add simple duplicate quads test.
c58827b
to
f3ade06
Compare
- Rename `URDNA2015.js` to `RDFC10.js`. - Rename 'URDNA2015` class to `RDFC10`. - Rename `URDNA2015Sync.js` to `RDFC10Sync.js`. - Rename 'URDNA2015Sync` class to `RDFC10Sync`.
maxWorkFactor
parameter.