[jira] [Commented] (SANDBOX-402) Add GraphML importer
[ https://issues.apache.org/jira/browse/SANDBOX-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13221635#comment-13221635 ] Thomas Neidhart commented on SANDBOX-402: - ok, now I understand your intention. yes this looks very good imo. > Add GraphML importer > > > Key: SANDBOX-402 > URL: https://issues.apache.org/jira/browse/SANDBOX-402 > Project: Commons Sandbox > Issue Type: New Feature > Components: Graph >Reporter: Thomas Neidhart > > There is already an GraphML exporter and it would be nice to have an importer > too, but for reasons of testing and convenience. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (SANDBOX-402) Add GraphML importer
[ https://issues.apache.org/jira/browse/SANDBOX-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13221630#comment-13221630 ] Simone Tripodi commented on SANDBOX-402: yes that is what I mean maybe poorly expresses, given that {{Graph}} is a data structure wich generics, the importer should understand wich {{Vertex}}/{{Edge}} type has to deserialize. Under the fluent APIs PoV, I would suggest something like: {code} Graph g = CommonsGraph.import( "path/to/gml/file.xml" ).usingGraphML(); // or DotNotation {code} maybe with few overloads: {code} import( "path/to/gml/file.xml" ).usingGraphML(); import( new File( "path/to/gml/file.xml" ) ).usingGraphML(); import( URI.create( "http://my.rst.server/graph.xml"; ) ).usingGraphML(); {code} and so on... WDYT? > Add GraphML importer > > > Key: SANDBOX-402 > URL: https://issues.apache.org/jira/browse/SANDBOX-402 > Project: Commons Sandbox > Issue Type: New Feature > Components: Graph >Reporter: Thomas Neidhart > > There is already an GraphML exporter and it would be nice to have an importer > too, but for reasons of testing and convenience. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (SANDBOX-402) Add GraphML importer
[ https://issues.apache.org/jira/browse/SANDBOX-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13221593#comment-13221593 ] Thomas Neidhart commented on SANDBOX-402: - I do not understand your point. The idea is not to import a GraphML file into an existing graph (which may be a valid operation too), but rather to create a new graph from an GraphML file, something like: {noformat} Graph g = CommonsGraph.importFromGraphML( "path/to/gml/file" ); {noformat} > Add GraphML importer > > > Key: SANDBOX-402 > URL: https://issues.apache.org/jira/browse/SANDBOX-402 > Project: Commons Sandbox > Issue Type: New Feature > Components: Graph >Reporter: Thomas Neidhart > > There is already an GraphML exporter and it would be nice to have an importer > too, but for reasons of testing and convenience. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (SANDBOX-402) Add GraphML importer
[ https://issues.apache.org/jira/browse/SANDBOX-402?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13221583#comment-13221583 ] Simone Tripodi commented on SANDBOX-402: It would be nice having the general one, what it's not clear to me is how binding parsed elements to target graph/vertex/edge, it would require factories, maybe... > Add GraphML importer > > > Key: SANDBOX-402 > URL: https://issues.apache.org/jira/browse/SANDBOX-402 > Project: Commons Sandbox > Issue Type: New Feature > Components: Graph >Reporter: Thomas Neidhart > > There is already an GraphML exporter and it would be nice to have an importer > too, but for reasons of testing and convenience. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira