Closed cthoyt closed 1 year ago
Looks all good, can you add a separate test for the from_df
and from_tuples
so that each read path (edgelist, df, tuples) are tested separately?
Will do! The from_df is transitively tested by both so it really only needs a test for from_tuples
This PR packages most of the functionality from
read_edgelist()
into a private function so it can be reused for other methods that construct pandas dataframes. This PR then adds a new functionfrom_tuples
to make it easier to generate a CSRGraph from tuples in memory