Check args default values and consistency

[Bisaloo]

• Please check if the PR fulfills these requirements

• [ ] I have read the CONTRIBUTING guidelines

• [ ] A new item has been added to NEWS.md

• [ ] Tests for the changes have been added (for bug fixes / features)

• [ ] Docs have been added / updated (for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

• What is the current behavior? (You can also link to an open issue here)

• What is the new behavior (if this is a feature change)?

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

• Other information:

This branch should be rebased on main once develop has been merged and we switch to working directly with PR to main.

[Karim-Mane]

@Bisaloo kindly review this when you have time. thanks

[Karim-Mane]

@Bisaloo kindly review this PR when you have time. Note that there are commits that I made to fix conflicts with main

[Bisaloo]

Work will continue in #62 because this branch history was mixed with main and I don't want to force push in case any important change was added during conflict resolution.