qojulia / QuantumOptics.jl

Library for the numerical simulation of closed as well as open quantum systems.
http://qojulia.org
Other
518 stars 101 forks source link

Rename derivative functions #312

Closed david-pl closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #312 (6f8ae1c) into master (ecbbd3a) will increase coverage by 0.29%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #312      +/-   ##
==========================================
+ Coverage   97.94%   98.24%   +0.29%     
==========================================
  Files          16       16              
  Lines        1315     1308       -7     
==========================================
- Hits         1288     1285       -3     
+ Misses         27       23       -4     
Impacted Files Coverage Δ
src/steadystate.jl 91.17% <ø> (ø)
src/master.jl 100.00% <100.00%> (+2.50%) :arrow_up:
src/mcwf.jl 100.00% <100.00%> (ø)
src/schroedinger.jl 88.88% <100.00%> (ø)
src/semiclassical.jl 96.62% <100.00%> (ø)
src/steadystate_iterative.jl 97.61% <100.00%> (ø)
src/stochastic_base.jl 100.00% <100.00%> (ø)
src/stochastic_master.jl 97.18% <100.00%> (ø)
src/stochastic_schroedinger.jl 100.00% <100.00%> (ø)
src/stochastic_semiclassical.jl 98.03% <100.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ecbbd3a...6f8ae1c. Read the comment docs.

Krastanov commented 2 years ago

Will these new dsomething! functions be made public API and/or exported?

david-pl commented 2 years ago

@Krastanov They will be part of the API (and also of the documentation). As far as exporting goes, you'll have to pre-fix them with the submodule you're using (e.g. timeevolution.dschroedinger!), just as for the standard timeevolution functions.