Removed padding from conv and pool and added Pad operator

I think padding should be removed from Conv and Pool operators. From N2D2 experience, padding adds a lot of complexity to the operator implementations. Third party conv and pool implementations do not always support padding or all types of padding (e.g. CuDNN only supports symmetric padding).

We could have a PaddedConv meta-operator to take advantage of implementations that support padding inside conv. It could be a good first use case for meta-operator, that I will try to include in this MR.

• Remove padding from operators
• Add Pad implementation
  • Add Constant mode implementation
  • Handle other modes
• Add PaddedConv and PaddedPooling meta-operators
  • Add meta-operator prototype
  • Requires ordered inputs for GraphView @pineapple => for now, specify ordered inputs/outputs in the meta-op
  • Requires generateScheduling() to be callable iteratively @cmoineau => should be OK
• Update previously padded layers unit-tests

If we agree on having a separate Pad operator, I still have some questions :

• Should it be one or several operators? For example, should we have a specific ZeroPad operator for 0-padding only? Or a little more general ConstantPadding for padding with a given value?

  Use a single Operator.

• What should be the parameters of this operators? Like constant, replicate, reflective... to handle different type of classical padding in computer vision.

  Current list is Constant, Replicate, Reflect, Wrap, which are pretty standard.

added 1 commit

• 72fa1b79 - Removed padding from binding

Compare with previous version

assigned to @olivierbichler

requested review from @cmoineau and @pineapple

added 1 commit

• 30df0333 - Missing removal in MaxPooling binding

Compare with previous version

changed the description

changed the description

changed the description

changed milestone to %v0.1.0

added 1 deleted label

Since ZeroPad is basically Pad({dims}, "constant", 0.0), default values could already be set to provide a ZeroPad function. So I think this function is not needed.

changed the description

added 1 commit

• 9b3c6ec2 - Added first MetaOperator prototype

Compare with previous version

changed the description

mentioned in merge request aidge_backend_cpu!7 (merged)

added 4 commits

• 48ce14d2 - Changed GraphView inputs() and dataInputs() behavior
• 3e041bf9 - Fixed Pad op
• a35bdf17 - If forward() calls generateScheduling(), then mStaticSchedule should be cleared
• d376906b - Make MetaOperator work for PaddedConv

Compare with previous version

changed the description

added 127 commits

• d376906b...71763994 - 126 commits from branch main
• ef778f09 - Merged with main

Compare with previous version

changed the description