Cli improvements

Closes #53

See merge request !84

At some point this mode stopped producing plots.  This fixes that by
updating the code execution call to InteractiveShellEmbed.

Also only imports ipython opportunistically, so it's not a hard dependency.

This allows for saving budgets in multiple formats simultaneously, without
having to recalculate noises.

This will speed up some operations in the CI.

test look for upstream master by default, and minor fixes

See merge request !85

This changes the test to look for the upstream master in the user
current config and use that as the test reference by default.  This
seems like a more sensible default in general, and it's easy enough to
fall back to HEAD if need be ('-g HEAD').

This also fixes some things from the last commit:

* fixes the MR CI pipeline
* adds better command line documentation to the test
* fixes command help
* fixes bug in test cache pruing

simplify testing by removing unnecessary reference

See merge request !83

The assumption should really be that the HEAD of the repo is the
current noise reference.  There is therefore no need to store a
special reference hash for the tests.  This simplifies things quite a
bit, in particular by not requiring submitters to update a reference.

This commit drops the reference and modifies the test to just compare
the current code against HEAD by default.  The option to compare
against an arbitrary ref is preserved.

The CI is updated for these simplifications.  The "test" job is now
simple generation of the budgets.  The merge request approval job is
renamed 'review:noise_change_approval`, and it's logic remains mostly
the same: it compares the current budgets against the head of the MR
target branch, and requires the same MR approval to pass.  The overall
MR approval process therefor doesn't change.

This change got added accidentally from an earlier version of the
"test compare git hash" series during a rebase.

This reverts commit 16b9ae40.

Revise suspension model

See merge request !80

Upper joint temperature comes from the stage above
Drop support for BQuad suspension type
Use the fiber end cross-section consistently in wireTELoss()
Remove the "questionable approximation" in continuumWireKh
Correct the spring constant calculation in continuumWireKv

MR approval check for noise changes

See merge request !82

tests compare against git hash, eliminate cached .h5

See merge request !81

Cli improvements

Closes #19

See merge request !68

This adds a CI special job that runs only on merge requests that update
the test git ref, indicating that a noise has been changed.  The MR will
show present a report of the changed noises, and will block on MR approval.
Once approval has been given, the check_approval job can be re-run, which
should now pass, allow the MR to proceed.

If the noises change but the reference has not been updated, then the CI
pipeline will fail as usual.

This updates the test infrastructure to store a git hash of the reference
code, instead of generated .h5 files.  The test infrastructure creates a
cache of reference .h5 files from the specified git hash if it doesn't
already exist, and then compares against that cache.

Comparisons against arbitrary git hashes can be specified manually.
The cache curves are generated from a shell script so that it uses the
checkout of the python code at the reference commit.  Multiple caches based
on git references are stored, and old caches are automatically pruned to
keep just the latest five.

This gets rid of the need for git-lfs and the cached .h5 files, requiring
instead the code be checked out from git to run the tests.

This updates the test infrastructure to store a git hash of the reference
code, instead of generated .h5 files.  The test infrastructure creates a
cache of reference .h5 files from the specified git hash if it doesn't
already exist, and then compares against that cache.

Comparisons against arbitrary git hashes can be specified manually.
The cache curves are generated from a shell script so that it uses the
checkout of the python code at the reference commit.  Multiple caches based
on git references are stored, and old caches are automatically pruned to
keep just the latest five.

This gets rid of the need for git-lfs and the cached .h5 files, requiring
instead the code be checked out from git to run the tests.

Some cleanup and updates on the CI, including:
* creating a common docker image for all jobs
* expose test failure report to merge requests

Use overlaid dashes for the head traces.

Also do comparison in preferred IFO order.

These changes improve the modeling of suspensions that have a
temperature gradient along the fibers.  Each stage is split into an
upper and a lower joint, which can have different material properties
assigned in the ifo structure.  The temperature of a joint is that of
the mass to which it is attached.  (However, temperature gradients are
not yet supported within the top or bottom stages.)  The average of
the upper and lower joint dissipation is used to compute the noise.

minor numpy import cleanup

See merge request !76

using --ifo option.

closes #19

Just use a single argument of the form 'flo:npoints:fhi', instead of
three separate arguments.

no functional change, just limits individual function imports to non-array
operations.

Added Mac file .DS_Store to .gitignore

See merge request !75

Anchal Gupta authored 4 years ago and Anchal Gupta committed 4 years ago

(cherry picked from commit b253c97e)

Removed normalization of sinc_sqr in dhdl because it is one at DC. Previously the normalization was set by the first frequency point, which is wrong if only high frequencies are considered

Add SQL noise type

It's sometimes helpful to have an SQL noise curve on noise budgets.

See merge request !73