forked from stevencox/roger
-
Notifications
You must be signed in to change notification settings - Fork 4
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
annotator modules added by passing config val #90
Merged
YaphetKG
merged 58 commits into
pipeline_parameterize_restructure
from
rebased-annotator-plugin
Jan 29, 2024
Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
eac8e6a
annotator modules added by passing config val
braswent 3880dbc
Merge branch 'pipeline_parameterize_restructure' into rebased-annotat…
YaphetKG 2480001
fix merge conflict
YaphetKG b0028c5
following same pattern as parsers , modify configs
YaphetKG 5113953
fix to dug config method
YaphetKG c59bfb0
fix old dug pipeline for backward compatiblity
YaphetKG e0dcd93
correct default annotator type
YaphetKG ae48080
reflective changes
YaphetKG 5fd9168
typo extra quotes
YaphetKG 5ae2d3a
annotator type not being picked up from config
YaphetKG 9ca1e38
remove annotate simple , log env value for lakefs enabled
YaphetKG 348be81
testing lakefs off
YaphetKG 528768d
add more logging
YaphetKG df89638
add more logging
YaphetKG 683e35b
post init for config to parse to boolean
YaphetKG 6428075
put back task calls
YaphetKG ec54cc8
revert some changes
YaphetKG d557c2f
adding new pipeline
YaphetKG 83815ab
lakefs io support for merge task
YaphetKG 80ddf59
fix name
YaphetKG 93dcbe9
add io params for kg tasks
YaphetKG 3676cc9
wire up i/o paths for merge
YaphetKG 0a8be3e
fix variable name
YaphetKG 5a010d8
print files
YaphetKG 1e6a9a2
few debug logs
YaphetKG c1ae51a
few debug logs
YaphetKG eb7fdc1
treat path as path not str
YaphetKG 6c49a0c
few debug logs
YaphetKG 7fdf08a
some fixes
YaphetKG 07c5bd9
logging edge files
YaphetKG 999d4a6
bug fix knowledge has edge
YaphetKG 5a8f629
re-org graph structure
YaphetKG c5d2b0e
adding pathing for other tasks
YaphetKG 96c8ff5
pagenation logic fix for avalon
YaphetKG 62e8a0c
update lakefs client code
YaphetKG 9f03265
fix glob for get kgx files
YaphetKG 749deba
fix up get merged objects
YaphetKG f4adf0d
send down fake commit id for metadata
YaphetKG ce4c84f
working on edges schema
YaphetKG dcfd3db
bulk create nodes I/O
YaphetKG e43b5f1
find schema file
YaphetKG db58018
bulk create edges I/O
YaphetKG 44ab91a
bulk create edges I/O
YaphetKG 27fc0e4
bulk load io
YaphetKG 2319a46
no outputs for final tasks
YaphetKG 6429caa
add recursive glob
YaphetKG 3c84f6a
fix globbing
YaphetKG f8db982
oops
YaphetKG 8ab1a5c
delete dags
YaphetKG ce59d53
pin dug to latest release
YaphetKG 3434d1a
cruft cleanup
YaphetKG 60f90ba
re-org kgx config
YaphetKG 6f2c0cc
add support for multiple initial repos
YaphetKG eef984e
fix comma
YaphetKG f90f13f
create dir to download to
YaphetKG 4fd8ae2
swap branch and repo
YaphetKG 0c057c1
clean up dirs
YaphetKG 9ac704a
fix up other pipeline 👌
YaphetKG File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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.
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.
Seems like if we're getting rid of the default of monarch, the docstring should change too. (I may have missed this getting replaced elsewhere tho)
EDIT: okay not sure if that change was in the PR, GitHub decided to show me a diff where this was relevant but now it's applying this comment to a different block, not sure what happened there, this may be moot.