WIP: Prior clean up

Requires !688 (merged)

Some code improvements for the prior package:

• Cleaned up a bunch of warnings
• removed if/else-clauses for prob/rescale/ln_prob/cdf methods that distinguish between float and array input. It now always uses the array logic and then checks the type to return a float if it is a float input
• Introduced @valid_rescale_check and @consistent_type_use decorators to reduce redundant code throughout the package
• Restructured the prior tests into its own package
• Assured that prior saving/loading is still working the same way.
• Got rid off the check_range_nonzero argument in favour of dynamic testing with properties

changed milestone to %1.0.0

added Refactoring label

added 1 commit

• 31fa550a - Fixed PEP8 violations

Compare with previous version

added 1 commit

• 93ae62fc - Fixed inconsistent return of floats and arrays from rescale methods

Compare with previous version

added 10 commits

• 498c618e - Implemented a decorator to assure consistent use of float/arrays in the prior packages
• dc4560ca - Made use of a decorator to enforce consistent type use in priors
• 8219aff9 - Enforced more consistent type use
• 9816c21b - Enforced more consistent type use
• 4b39f5a6 - Made consistent type wrapper work with tests
• d8d232a4 - Added decorator to check valid rescaling
• da76f025 - Enforced Consistent use of arrays in all prob/ln_prob/recale/cdf methods
• 6b5a0389 - Made docstrings more consistent
• 4bbfa85e - Simplified some expressions
• 0e1090a5 - Fixed the rescale check decorator

Compare with previous version

marked as a Work In Progress

changed the description

added 2 commits

• e9ee021e - Some reformatting
• b2dbfdc3 - Refactored redundant code into a new utils module

Compare with previous version

added 1 commit

• 5839387b - Removed unused import

Compare with previous version

approved this merge request

unapproved this merge request

added 29 commits

• 5839387b...29a34a34 - 28 commits from branch master
• 8c0da7bd - Merge remote-tracking branch 'origin/master' into prior_clean_up

Compare with previous version

added 2 commits

• 4dae981e - Added option not to check rescaling
• f64e0169 - Manually added Isobel's changes back in

Compare with previous version

resolved all threads

added 9 commits

• dbbcddd4 - Removed Python 2 specific handling in repr test
• 4d05cdbd - Fixed a typo introduced when manually merging
• 03a6c405 - changed no_rescale_check -> rescale_check
• b52b861d - Simplified some expressions
• 2eb8eb91 - Made IS_FIXED a public class attribute
• 28288be5 - Added new prior feature information
• d1c8fbb4 - Added `rescale_check` property to `PriorDict` class
• 96454c32 - Added information
• f3cb717e - Added check when setting rescale_check that object is in fact a `Prior`

Compare with previous version

added 2 commits

• 7e384ab9 - Fixed PEP8 issues
• 5b5d9c36 - Restructured prior files

Compare with previous version

added 1 commit

• 2978a19a - Adapted path to prior files

Compare with previous version

changed milestone to %0.6.5

added Priors label

added 1 commit

• d1ea3b85 - Removed `check_range_nonzero` in favor of a more dynamic test

Compare with previous version

changed the description

added 2 commits

• ab9c9750 - Added test for non-zero range check
• 2d248ed8 - Restructured cosmological prior minimum/maximum handling to be compatible when check ranges

Compare with previous version

unmarked as a Work In Progress

changed the description

added 1 commit

• a03ed53f - extracted some redundant code

Compare with previous version

added 1 commit

• f82c36a8 - Fixed cosmological prior bound creation

Compare with previous version

added 2 commits

• e5bb7786 - Improved cosmological prior boundary logic
• 8ccd07b2 - Reversed variable assignment order to prevent exception

Compare with previous version

added 1 commit

• 1cb4b330 - Fixed a minor bug that occurs if chirp mass is already in the CBCPriorDict

Compare with previous version

marked as a Work In Progress

changed milestone to %0.6.4

changed milestone to %0.6.5

changed milestone to %0.6.6

changed milestone to %0.6.7

changed milestone to %0.6.8

added 116 commits

• 1cb4b330...0140d1e7 - 115 commits from branch master
• 7054599f - Merge remote-tracking branch 'origin/master' into prior_clean_up

Compare with previous version

added 1 commit

• 7592a25a - Fixed an issue that arose during merging

Compare with previous version

changed milestone to %Future

closed