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.

Sign up for GitHub

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 · Fixed by #438
Closed

Add graphml output to DomainGraphExtractor #435

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

Comments

@ruebot
Copy link
Member

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.
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)
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 feature Scala
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add graphml output to CommandLineApp and DomainGraphExtractor. archivesunleashed/aut
1 participant
@ruebot