other bug in add_simplices_from when specifying ids

[maximelucas]

Running

edges = [({0, 1, 2}, 1), ({1, 2}, 0), ({2, 3, 4}, 2)]
S = xgi.SimplicialComplex()
S.add_simplices_from(edges)

we get

# -> UserWarning: uid 2 already exists, cannot add simplex {2, 3, 4}.
S._edge
# -> {1: frozenset({0, 1, 2}),
#     2: frozenset({0, 1}),
#     3: frozenset({0, 2}),
#     4: frozenset({1, 2})}

which is wrong for two reasons:

1. {2, 3, 4} should've been added but was not because its ID was taken by subfaces of {0, 1, 2}
2. {1, 2} was assigned ID 4 even though the input specified 0

Both these issues can be solved by making add_simplices_from first add all input simplices and then add all their subfaces, rather than adding each input simplex and their subfaces immediately.

[maximelucas]

fixed in #247