NanoComp / meep

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

Various improvements to docstrings and readability of adjoint-solver module #2579

Closed oskooi closed 1 year ago

oskooi commented 1 year ago

Some minor tweaks and fixes mainly to improve readability.

codecov-commenter commented 1 year ago

Codecov Report

Merging #2579 (26863fd) into master (cfa6163) will not change coverage. The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master    #2579   +/-   ##
=======================================
  Coverage   73.90%   73.90%           
=======================================
  Files          18       18           
  Lines        5293     5293           
=======================================
  Hits         3912     3912           
  Misses       1381     1381           
Impacted Files Coverage Δ
python/adjoint/objective.py 92.38% <ø> (ø)
python/adjoint/filter_source.py 87.17% <100.00%> (ø)