remove MPIRequest.wait()

[mtar]

Description

Removes the wait method of MPIRequest such that it does not interfere with the lowercase non-blocking MPI functions of mpi4py.

Issue/s resolved: #671

Changes proposed:

• delete MPIRequest.wait()
• fix calls using wait() to Wait()

Type of change

• Bug fix
• Breaking change

Due Diligence

• All split configurations tested
• Multiple dtypes tested in relevant functions
• Documentation updated (if needed)
• Updated changelog.md under the title "Pending Additions"

Does this change modify the behaviour of other functions? If so, which?

no

[codecov]

Codecov Report

Merging #672 into master will increase coverage by 0.01%.
The diff coverage is 98.55%.

@@            Coverage Diff             @@
##           master     #672      +/-   ##
==========================================
+ Coverage   97.44%   97.46%   +0.01%     
==========================================
  Files          87       87              
  Lines       17099    17093       -6     
==========================================
- Hits        16662    16659       -3     
+ Misses        437      434       -3     

Impacted Files	Coverage Δ
heat/core/communication.py	90.35% <ø> (+0.47%)	⬆️
heat/core/linalg/qr.py	99.28% <75.00%> (ø)
heat/core/arithmetics.py	99.29% <100.00%> (ø)
heat/core/dndarray.py	96.71% <100.00%> (ø)
heat/core/linalg/basics.py	94.46% <100.00%> (ø)
heat/core/manipulations.py	99.22% <100.00%> (ø)
heat/core/tests/test_communication.py	97.96% <100.00%> (ø)

---

Continue to review full report at Codecov.

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

[mtar]

rerun tests

[coquelin77]

why do we need to remove wait? what was it interfering with? isnt it something that we should have?