direct_citation creates undirected graph with no node attributes about tethne HOT 11 CLOSED

Yikes, that's odd. Looking into it.

from tethne.

@aosingh Can you take a look at this today?

from tethne.

@erickpeirson
ok, will pick this today.

from tethne.

@erickpeirson,
The method, as shown below, returns a Graph object. The class nx.Graph is the base class for undirected graphs.

return _generate_graph(nx.Graph, pairs, node_attrs=node_type,
min_weight=min_weight)

We can modify it to return a DiGraph.

from tethne.

Yes, we should change that to a DiGraph

https://asu.academia.edu/ErickPeirson
Postdoctoral Researcher
Arizona State University

On Fri, Mar 18, 2016 at 1:24 PM, Abhishek Omprakash Singh <
[email protected]> wrote:

@erickpeirson https://github.com/erickpeirson,
The method, as shown below, returns a Graph object. The class nx.Graph is
the base class for undirected graphs.

return _generate_graph(nx.Graph, pairs, node_attrs=node_type,
min_weight=min_weight)

We can modify it to return a DiGraph.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#137 (comment)

from tethne.

Fixed and tested it. Also changed the test case validation to self.assertIsInstance(g, nx.DiGraph).
Please find below an execution trace for wos corpus.

wos_citation_graph = papers.direct_citation(wosCorpus)

type(wos_citation_graph) <class 'networkx.classes.digraph.DiGraph'>

from tethne.

Awesome; can you raise a PR?

from tethne.

PR raised.

from tethne.

@KHoover2009 Can you confirm that this fixes the problem?

Be sure to install the latest version of tethne:

$ pip install -U tethne

from tethne.

Confirmed

Kevin Hoover
[email protected]

From: Erick [email protected]
Sent: Wednesday, May 18, 2016 4:24:07 AM
To: diging/tethne
Cc: Hoover, Kevin; Mention
Subject: Re: [diging/tethne] direct_citation creates undirected graph with no node attributes (#137)

@KHoover2009https://github.com/KHoover2009 Can you confirm that this fixes the problem?

Be sure to install the latest version of tethne:

$ pip install -U tethne

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com//issues/137#issuecomment-219986356

from tethne.

Fantastic; thanks!

Erick Peirson
Postdoctoral Scholar
ASU-SFI Center for Biosocial Complexity
Arizona State University

On May 18, 2016, at 5:41 PM, KHoover2009 [email protected] wrote:

Confirmed

Kevin Hoover
[email protected]

---

From: Erick [email protected]
Sent: Wednesday, May 18, 2016 4:24:07 AM
To: diging/tethne
Cc: Hoover, Kevin; Mention
Subject: Re: [diging/tethne] direct_citation creates undirected graph with no node attributes (#137)

@KHoover2009https://github.com/KHoover2009 Can you confirm that this fixes the problem?

Be sure to install the latest version of tethne:

$ pip install -U tethne

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHubhttps://github.com//issues/137#issuecomment-219986356
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

from tethne.