tethne icon indicating copy to clipboard operation
tethne copied to clipboard

Documentation for 0.8 incorrectly specifies what's returned in a direct_citation graph

Open serenalotreck opened this issue 1 year ago • 0 comments

I'm looking at the most recent documentation, but it does not correspond to what I'm observing in my code. It says that "The networks.papers.direct_citation() method generates both a global citation graph, which includes all cited and citing papers, and an internal citation graph that describes only citations among papers in the original dataset."

In pervious iterations of the documentation, it clearly states in both the function documentation and example that two networkx Graph objects are returned. However, when I try and catch the output of direct_citation() into two variables in tethne 0.8, I get the following:

Traceback (most recent call last):
  File "build_citation_network.py", line 46, in <module>
    main(args.wos_file, args.out_loc, args.out_prefix)
  File "build_citation_network.py", line 18, in main
    igraph, ograph = direct_citation(papers)
ValueError: too many values to unpack (expected 2)

This is consistent with the source code, as the multipartite function which direct_cutation() calls only returns one Graph object.

I'd like to clarify which is the expected behavior; does the documentation need to be changed, or does the source code?

EDIT: If the capacity to generate in-dataset and out-of-dataset citation networks separately has been removed, can it be put back? It's a very valuable differentiation for my use case, and it seems odd to completely remove that functionality.

serenalotreck avatar Apr 04 '23 17:04 serenalotreck