[jira] [Commented] (SANDBOX-402) Add GraphML importer

2012-03-03 Thread Thomas Neidhart (Commented) (JIRA)

[ 
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

2012-03-03 Thread Simone Tripodi (Commented) (JIRA)

[ 
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

2012-03-03 Thread Thomas Neidhart (Commented) (JIRA)

[ 
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

2012-03-03 Thread Simone Tripodi (Commented) (JIRA)

[ 
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