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

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

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

Gilles (Jira)
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

       
Reply | Threaded
Open this post in threaded view
|

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

Gilles (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

       
Reply | Threaded
Open this post in threaded view
|

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

Gilles (Jira)
In reply to this post by Gilles (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

       
Reply | Threaded
Open this post in threaded view
|

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

Gilles (Jira)
In reply to this post by Gilles (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

       
Reply | Threaded
Open this post in threaded view
|

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

Gilles (Jira)
In reply to this post by Gilles (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