[Upd] replace() instead of replaceWith() in GraphView

Description of replaceWith()

The replaceWith() member function works as follow:

1. Select a set of Nodes to replace
2. Create a GraphView from this set. (let's call it g)
3. Select a set of new Nodes and solve their input connections (let's call it newNodes)
4. run g.replaceWith(newNodes)
5. Nodes are replaced in every GraphView pointing at them all

Issue

1. It may be confusing to create a new GraphView from the set of Nodes to replace and replace it by another set of Nodes and not a GraphView
2. Input connections are not handled at all
3. Output connections are handled and authorized only if obvious

Solution

Change replaceWith() by replace()

GraphView::replace(std::set<std::shared_ptr<Node>> oldNodes, std::set<std::shared_ptr<Node>> newNodes);

1. symetric handling of old/new Nodes
2. automatically recreate input AND output connections if obvious (one connection or every input pointing to the same Tensor)

changed milestone to %v0.1.0

added 1 deleted label

assigned to @pineapple

I think you should remove replaceWith or make it call the replace method. I would opt for the latest as some of our tutorials depend on replaceWith.

Furthermore, I think we should add a deprecation system in Aidge @olivierbichler , @pineapple

changed the description

added 2 commits

• f646e90d - [Upd] simplify removeView() member function in Node.hpp
• 4b3540a3 - [Upd] replace() member function for tiling handling

Compare with previous version

added 2 commits

• 422ac1ce - [Depreciated] replaceWith()
• 7299fadf - [Upd] recipies use replace() instead of replaceWith()

Compare with previous version

added 1 commit

• 5b97a6a5 - [Upd] replace() python binding

Compare with previous version

added 8 commits

• 5b97a6a5...daec3e64 - 6 commits from branch main
• 835e17e7 - Merge remote-tracking branch 'origin/main' into fix/replaceWith
• 2e9f0013 - [Fix] replace() binding

Compare with previous version

added 1 commit

• ad28a0e8 - [Fix] replace() member function to run python tests successfully

Compare with previous version

resolved all threads

requested review from @cmoineau, @thibaultallenet, and @vtemplier

changed the description

mentioned in issue aidge#32 (closed)

resolved all threads

changed the description

added 1 commit

• 458c4ae9 - [MIN] make FuseMulAdd more resilient to edge cases

Compare with previous version

approved this merge request

added 12 commits

• 458c4ae9...c8c16f2e - 10 commits from branch main
• d076d8df - Merge remote-tracking branch 'origin/main' into fix/replaceWith
• ff19c397 - Completely remove replaceWith() member function from GraphView

Compare with previous version

enabled an automatic merge when the pipeline for ff19c397 succeeds

This MR should be coupled with an Aidge bundle MR to update tutorials.

merged

mentioned in commit 50f86506