search

ITensor / NamedGraphs.jl

Extension of `Graphs.jl` to graphs with named vertices.
MIT License
6 stars 3 forks source link

Allow partitionedge and partitionedges to accept pairs #47

Closed JoeyT1994 closed 9 months ago

JoeyT1994 commented 9 months ago

A quick PR which allows partitionedge(pg, edge) and partitionedges(pg, edges) to accept a Pair or Vector{Pair} alongside AbstractEdge and Vector{AbstractEdge} types.

mtfishman commented 9 months ago

Thanks, could you add some tests?

codecov-commenter commented 9 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (569e829) 78.54% compared to head (cb69407) 78.41%. Report is 1 commits behind head on main.

Files Patch % Lines
src/Graphs/partitionedgraphs/partitionedgraph.jl 0.00% 2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #47 +/- ## ========================================== - Coverage 78.54% 78.41% -0.14% ========================================== Files 31 31 Lines 1184 1186 +2 ========================================== Hits 930 930 - Misses 254 256 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

JoeyT1994 commented 9 months ago

Sure I added a few tests.

mtfishman commented 9 months ago

Looks good, thanks!