obs_sequence object aware whether it has assimilation results or not #28
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added two bits of information to the obs_sequence class:
self.has_assimilation_info = False
self.has_posterior = False
An obs_sequence object is then aware of whether it has assimilation info (prior only or prior and posterior)
Added decorator for class methods that require assimilation to check info is present.
Select by qc and possible vs used functions now class methods and use the decorator (because they need assimilation info)
Now have a posterior bias and sq error column (note I've not renamed bias & sq error to prior_bias, prior sq error)
Added tests for requires assimilation_info, init method changed so you don’t need a file for testing the data frame routines.
Added an early exit if no file is given to obs_sequence::init. This is so you can test the data frame methods without worrying about having a file to give data.
fixes #25
fixes #27
fixes #19