B1' derived ef

[jordidj]

PR description

Added the derivative of B1 to the derived eigenfunctions to allow for computation of Delta' for tearing modes.

New features

Legolas

• New derived ef: dB1

Checklist

• all tests pass (and for new features, new tests are added)
• documentation has been updated (if applicable)

[jordidj]

To do:

• regenerate derived ef tests
• bump version number
• add LaTeX form for dB1 to pylbo

[n-claes]

Poke me if this is ready and I'll give it a review. Failing tests can be simply fixed by bumping the expected number of derived eigenfunctions for the unit tests and regenerating the regression answers for the Couette and Taylor-Couette cases.
You can do this through

pytest test_taylor_couette.py --generate

and then it will ask you to confirm when the tests run. You can add the --keep-files flag as well so it doesn't automatically remove them after testing for visual inspection.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5f8c87a) 94.93% compared to head (f22a762) 94.94%.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #156   +/-   ##
========================================
  Coverage    94.93%   94.94%           
========================================
  Files          161      161           
  Lines         9138     9155   +17     
========================================
+ Hits          8675     8692   +17     
  Misses         463      463           

Flag	Coverage Δ
legolas	93.20% <100.00%> (+0.01%)	⬆️
pylbo	98.13% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

[n-claes]

Looks good -- quick question, for testing purposes I assume that (at least) one of the regression tests you regenerated include a case in which dB1 is non-zero? By this I mean a case where dB1 properly behaves.

Merge when ready :)

[jordidj]

dB1 is nonzero and well-behaved in the MRI test case.