Resolve "Restructure prior module"

Closes #444 (closed)

changed milestone to %1.0.0

added Priors Refactoring labels

added 9 commits

• 589180ee - Moved prior.py into its own module
• 0b54cc72 - First step at restructuring the prior module
• 6b0d6b58 - Resolved some circular dependencies
• 1741fc0f - Resolved all circular dependencies
• 88a80b0d - Fixed a typo
• c48dc917 - Adapted module path to new structure
• 4030b99a - Fixed remaining issues to make the tests work
• 338809a1 - PEP8 fixes
• 5ad6ace0 - Roll back changes

Compare with previous version

added 1 commit

• 3d5f178f - PEP8 fixes

Compare with previous version

added 1 commit

• cb6b06df - Fixed PEP8 violations

Compare with previous version

added 1 commit

• 72f64bfd - Added new prior package structure to setup.py

Compare with previous version

added 1 commit

• e4b63f96 - Added import to basic tests

Compare with previous version

added 1 commit

• c4180813 - Made gw prior loading work with new package structure

Compare with previous version

unmarked as a Work In Progress

mentioned in merge request !689 (closed)

Looks nice, thanks for doing this. Just to confirm, this was a direct copy/paste? There are no substantive changes to the code?

@colm.talbot There were some issues coming up along the way.

• I had to make some changes to the prior file reading so that it is backwards compatible. See, line 166 and in dict.py and 416 in base.py. There are also some redundancies in the code that should be fixed, but I will work on this in another MR.

• I added an attribute _is_fixed to the Prior classes. This is set to True except for constraints and delta functions. I had to do this to avoid circular dependencies between the modules. I think it is better practice anyway.

mentioned in merge request !693 (closed)

resolved all threads

approved this merge request

changed milestone to %0.6.4

mentioned in merge request !651 (merged)

approved this merge request

mentioned in commit e567db4c

merged