Resolve "Improve PowerSpectralDensity structure"
Closes #177 (closed)
Merge request reports
Activity
changed milestone to %0.3
added 15 commits
- f2eaa593 - Fixed a bug
- 193b6f15 - Introduced a bunch of factor class methods
- cdc92f0d - Replaced old setter methods with new fancy classmethods
- 76da7080 - Redid the __init__
- 52f4ead1 - Made amplitude_spectral_density_file a property that automagically does the…
- 20a2f167 - Made power_spectral_density_file a property that automagically does the check…
- 51ea4a07 - Moved two methods for code readability
- 1ae61217 - Refactored some names for readability and consistency
- 1fd78245 - Removed default to aLigo psd_array
- 0de52c12 - Refactored some code that checks that one correctly set from asd/psd_files
- 41ffa55e - Updated the PSD class docstring
- efe7ebc0 - Moved importing of the asd/psd_file into the respective setters
- 49ca96b8 - Did some minor refactoring
- 995b9be7 - PSD:
- 1388fe15 - Did some final refactoring
Toggle commit listadded 6 commits
- ad2378da - Removed Interferometer tests that actually test the PSD
- 1d63256b - Fixed some issues that came up in testing
- 394b943c - Removed a bug that came up in testing
- f0400b14 - Removed a bug that removed the interpolated PSD when setting from __init__
- 226dd945 - PSD: Moved file checks to the respective properties
- 6e2006c9 - Created tests for the PSD
Toggle commit list@gregory.ashton @colm.talbot @paul-lasky I made some significant changes to the PowerSpectralDensity class. Unfortunately, this will change the syntax for users (but I think for the better. Also, I was able to add tests this way.
Instead of having something like:
psd = PowerSpectralDensity() psd.set_from_power_spectral_density_array()
We now only have:
psd = PowerSpectralDensity.from_power_spectral_density_array(psd_array)
And it also works this way:
psd = PowerSpectralDensity(psd_array=psd_array)
added 1 commit
- c6457075 - Fixed an issue related to the different file structure in the CI
- Automatically resolved by Moritz Huebner
- Automatically resolved by Moritz Huebner
mentioned in commit 37acb467