-
Notifications
You must be signed in to change notification settings - Fork 169
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
Suffix all doubles with d. Node batch reduced to 40 #257
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
krlawrence
requested review from
amazon-neptune-secrets-rotator and
michaelnchin
and removed request for
amazon-neptune-secrets-rotator
February 7, 2022 15:23
michaelnchin
previously approved these changes
Feb 7, 2022
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.
Looks good to me.
michaelnchin
approved these changes
Feb 8, 2022
charlesivie
added a commit
that referenced
this pull request
Feb 22, 2022
* added RDF ML notebooks * cleaning up the code a bit * completed regression and classification for SPARQL ML * added RDF ML notebooks * cleaning up the code a bit * completed regression and classification for SPARQL ML * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Rename gremlin profile args for clarity (#249) Co-authored-by: Michael Chin <[email protected]> * Add --results-per-page query option (#242) * Add -r option * Replace interval value * Add upper limit Co-authored-by: Michael Chin <[email protected]> * Pin ipython<7.17.0 to patch vulnerability (#250) Co-authored-by: Michael Chin <[email protected]> * Disable root logger output (#248) Co-authored-by: Michael Chin <[email protected]> * Fix OC Bolt metadata (#255) * Fix OC Bolt query metadata output * Add OC metadata tests Co-authored-by: Michael Chin <[email protected]> * Add groupby raw node result option (#253) * Add --group-by-raw query option * Add unit tests Co-authored-by: Michael Chin <[email protected]> * Add Gremlin group-by-depth (#251) * Add --group-by-depth to Gremlin, modify OC with same explicit param * Add tests * PEP8 fixes Co-authored-by: Michael Chin <[email protected]> * addressing comment left in PR #252 related to better conditional statements * Update ChangeLog.md * Suffix all doubles with d. Node batch reduced to 40 (#257) * Suffix all doubles with d. Node batch reduced to 40 * Update changelog for seed file changes * Update ChangeLog.md * Update ChangeLog.md Co-authored-by: Michael Chin <[email protected]> * Convert Decimal type results to float for Gremlin (#256) * Convert Decimal type values in GremlinNetwork * replace star import * Revert temp test change * Convert decimal type edge properties * Update Changelog Co-authored-by: Michael Chin <[email protected]> * removed hard coded genre deletes, and moved them into the notebook. * removed hard coded genre deletes, and moved them into the notebook. * removed hard coded ciritcScore deletes, and moved them into the notebook. * removed hard coded link prediction retractions, and moved them into the notebook. * completed the Getting Started notebook with the pretrained models. * completed the Getting Started notebook with the pretrained models. * added RDF ML notebooks * cleaning up the code a bit * completed regression and classification for SPARQL ML * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Complete notebooks for SPARQL Classification - Regression and Link Prediction * added RDF ML notebooks * cleaning up the code a bit * completed regression and classification for SPARQL ML * Complete notebooks for SPARQL Classification - Regression and Link Prediction * Complete notebooks for SPARQL Classification - Regression and Link Prediction * addressing comment left in PR #252 related to better conditional statements * removed hard coded genre deletes, and moved them into the notebook. * removed hard coded genre deletes, and moved them into the notebook. * removed hard coded ciritcScore deletes, and moved them into the notebook. * removed hard coded link prediction retractions, and moved them into the notebook. * completed the Getting Started notebook with the pretrained models. * completed the Getting Started notebook with the pretrained models. * tests passing for new SPARQL ML notebooks * Added detail fo new SPARQL/RDF ML notebooks to the change log * fixed bugs in link prediction, amde small improvements after review * updated location of pre-trained models Co-authored-by: Michael Chin <[email protected]> Co-authored-by: Michael Chin <[email protected]> Co-authored-by: Kelvin Lawrence <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Related to #254
Description of changes:
-Xss
modifier applied to change the thread stack size.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.