Skip to content

Remove kwargs in user classes and remove verbose from init

Philip Loche requested to merge remove_kwargs into main

This MR changes two things in the user facing classes:

  1. The verbose option is removed from the init method. It was always duplicated since it existed in the run and the init method

  2. The **kwargs option is removed. This prevents silently unchanged option with typos. For example if you use density=charge instead dens=charge in the density module the module was still calculating mass densities. Now this will produce an error.

Due to this change I found many inconsistency and not renamed parameters in the tests as well the examples...

Edited by Philip Loche

Merge request reports