changed existing_entities list into set datastructure for performance…

jamesscottbrown / pyyed

A simple Python library to export networks to yEd

BSD 3-Clause "New" or "Revised" License

82 stars 38 forks source link

changed existing_entities list into set datastructure for performance… #29

Closed bockor closed 4 years ago

bockor commented 4 years ago

… improvement

existing_entities list accumulates multiple duplicate elements when edges are build using the add_edge method of the Graph class. Sets do not take duplicates.

Attached you can see the difference.

existing_entities_list_vs_set.txt

jamesscottbrown commented 4 years ago

The duplication of elements has been fixed by https://github.com/jamesscottbrown/pyyed/pull/33 / https://github.com/jamesscottbrown/pyyed/pull/36.