Revert partially constructed segments on-error in segment init function

[dagardner-nv]

Description

• All nodes and edge holders now can optionally be assigned a name for ease of debugging.
• Explicitly release edges from ingress/egress ports and manifolds.
• Ensure that the inner_edge held by the ForwardingEgressProvider is disconnected.

Fixes #360
Fixes #400
Fixes #401

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[codecov]

Codecov Report

Merging #407 (4636694) into branch-23.11 (de48cb1) will decrease coverage by 0.20%.
The diff coverage is 85.45%.

❗ Current head 4636694 differs from pull request most recent head 67840a2. Consider uploading reports for the commit 67840a2 to get more accurate results

Additional details and impacted files

@@               Coverage Diff                @@
##           branch-23.11     #407      +/-   ##
================================================
- Coverage         74.03%   73.84%   -0.20%     
================================================
  Files               394      385       -9     
  Lines             13955    13710     -245     
  Branches           1053     1033      -20     
================================================
- Hits              10332    10124     -208     
+ Misses             3623     3586      -37     

Flag	Coverage Δ
cpp	69.84% <75.46%> (-0.68%)	⬇️
py	43.29% <81.52%> (+0.66%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
cpp/mrc/include/mrc/edge/edge_builder.hpp	86.53% <100.00%> (+4.13%)	⬆️
...pp/mrc/include/mrc/manifold/composite_manifold.hpp	88.88% <100.00%> (+2.22%)	⬆️
cpp/mrc/include/mrc/manifold/interface.hpp	100.00% <ø> (ø)
cpp/mrc/include/mrc/node/generic_sink.hpp	63.63% <100.00%> (-3.04%)	⬇️
cpp/mrc/include/mrc/node/generic_source.hpp	60.00% <100.00%> (+7.05%)	⬆️
cpp/mrc/include/mrc/node/queue.hpp	100.00% <100.00%> (ø)
cpp/mrc/include/mrc/node/rx_sink.hpp	97.77% <100.00%> (+0.15%)	⬆️
cpp/mrc/include/mrc/node/rx_sink_base.hpp	100.00% <100.00%> (ø)
cpp/mrc/include/mrc/node/rx_source.hpp	94.73% <100.00%> (+0.98%)	⬆️
cpp/mrc/include/mrc/node/rx_source_base.hpp	83.33% <100.00%> (+10.60%)	⬆️
... and 20 more

... and 70 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de48cb1...67840a2. Read the comment docs.

[dagardner-nv]

Closing in favor of #434