NanoComp / meep

free finite-difference time-domain (FDTD) software for electromagnetic simulations
GNU General Public License v2.0
1.26k stars 629 forks source link

improve meep code quality #2163

Open joamatab opened 2 years ago

joamatab commented 2 years ago

Improve quality code:

Keep up quality code

joamatab commented 2 years ago

may require rebasing existing PRs

662

ianwilliamson commented 2 years ago

meep.adjoint.* had the yapf formatter applied to it, but I am not sure if subsequent PRs kept this up. See https://github.com/NanoComp/meep/pull/1590#issue-910874343

joamatab commented 2 years ago

We applied black to python code and clang to the c++

Now it runs with pre-commit hook