Skip to content

/ aut

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

Jump to bottom

Add graphml output to DomainGraphExtractor #435

Closed
ruebot opened this issue Apr 8, 2020 · 0 comments
Closed

Add graphml output to DomainGraphExtractor #435

ruebot opened this issue Apr 8, 2020 · 0 comments
Assignees
@ruebot
Labels
DataFrames Scala feature

Comments

@ruebot
Copy link
Member

@ruebot ruebot commented Apr 8, 2020

The DomainGraphExtractor currently only exports as TEXT and GEXF. I'm not 100% certain the rationale for the GEXF output since our workflow in auk is graphml to graphpass, which produces the gexf. So, at the very least, we should add a GRAPHML option.
@ruebot ruebot self-assigned this Apr 8, 2020
@ruebot ruebot mentioned this issue Apr 8, 2020
Closed
ruebot added a commit that referenced this issue Apr 8, 2020
@ruebot
Check-in #435 work. Blocked by #436.
Verified
This commit was signed with a verified signature.
ruebot Nick Ruest
GPG key ID: 417FAF1A0E1080CD Learn about signing commits
895616c
This was referenced Apr 8, 2020
Merged
Merged
ianmilligan1 pushed a commit that referenced this issue Apr 11, 2020
@ruebot
Add graphml output to CommandLineApp and DomainGraphExtractor. (#438)
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
943cf92 
* Resolves #435
* Adds GRAPHML option to CommandLineApp
* Adds DataFrame method to DomainGraphExtractor
* Updates CommandLineApp, and WriteGraphML tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ruebot ruebot

Labels
DataFrames Scala feature
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Add graphml output to CommandLineApp and DomainGraphExtractor.
1 participant
@ruebot
You can’t perform that action at this time.