-
Notifications
You must be signed in to change notification settings - Fork 12
Feature/result cast #989
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Feature/result cast #989
Changes from all commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
4b8a365
new implementation for computing latent variables
rhayes777 ad86a76
integrated new latent variables implementation
rhayes777 c2eab67
docs
rhayes777 66d26df
unused import
rhayes777 aaa2ebe
first refactor, placing samples summary in result
Jammy2211 73e9df6
nows uses samples summary max likelihood
Jammy2211 c3975b4
median PDF added to samples summary and used for prior linking
Jammy2211 ad888c0
remove covariance matrix
Jammy2211 9eac1af
fix median pdf
Jammy2211 e3d2112
result now only uses samples summary for prior linking
Jammy2211 01cfbe5
remove sky save samples
Jammy2211 56a2ef9
fix creation of Samples Summary so it doesnt use covariance matrix
Jammy2211 a3d4b3f
samples summary no longer has model
Jammy2211 e652312
result_via_completed_fit clean and documented
Jammy2211 e7cf73b
clean up perform update
Jammy2211 fb1cb19
mockSamplesSummary to fix some test
Jammy2211 e36dd0e
fix unit tests by using MockResult
Jammy2211 ce0de40
test_abstract_search uses MockSamplesSummary
Jammy2211 4e4c59d
fix result json
Jammy2211 7af128a
hack to fix emcee
Jammy2211 52fd89e
docs
Jammy2211 62b2a71
docs refactoring
Jammy2211 7cd2299
added latent variable output.yaml parameters
Jammy2211 f2732b7
latent_variables.csv -> latent.csv
Jammy2211 e8bab2f
docs
Jammy2211 e8c90e5
added likelihood check to fitness function
Jammy2211 ace74cf
likelihood check now works
Jammy2211 186d337
update all Fitness classes with paths
Jammy2211 987e490
division by zero warning in inv_transform surpressed
Jammy2211 a19357a
fix DeprecationWarning: invalid escape sequence '\i' warning
Jammy2211 b7c0ea2
test specific warning caught
Jammy2211 1757da6
update docstring and make paths for fitness optional
Jammy2211 676f2c6
Merge branch 'feature/likelihood_check' into feature/keep_the_noise_down
Jammy2211 c94297b
surpress enviroment variable warnings
Jammy2211 8c78ceb
transform surpress
Jammy2211 9e5d3b2
fix eniroment warning unit test
Jammy2211 25fcab2
silence sneaky map test warning
Jammy2211 3dd3fb2
close matplotlib figures in graph plots to avoid noise
Jammy2211 96a394b
move dynesty suprression
Jammy2211 069a511
more transform warnings
Jammy2211 4ffc4f7
more quieting
Jammy2211 9243665
another thing quiet
Jammy2211 b0145ed
fixed a test
rhayes777 2213f60
fixing tests...
rhayes777 d718d08
fixed a test
rhayes777 bc82d0c
fixed a test
rhayes777 b2b93a6
fixed a test
rhayes777 d39c0ca
fixed test
rhayes777 79d56e3
skip computing median pdf for generic samples summary
rhayes777 80a4361
ensure ModelAnalysis.make_result conforms to super
rhayes777 e999d99
mock result with default mock samples summary
rhayes777 70a3e00
more fixes
rhayes777 4b7af3f
fixes
rhayes777 00e659b
fix more tests by moving arguments to MockSamplesSummary
rhayes777 3d867d5
another fix by providing median pdf sample
rhayes777 30ebbc2
small fixes
rhayes777 65639b1
mock samples generates kwargs
rhayes777 c19f273
fixed a test
rhayes777 612762d
sample summary first grid search results
rhayes777 46352ae
format
rhayes777 e83f23b
fix missing kwargs in grid sensitivity tests
rhayes777 2a7afd8
praise the lord another test fixed
rhayes777 ae8f276
remove print statement
rhayes777 fb02b94
consolidate fixture
rhayes777 54b3dbe
fix exception
Jammy2211 8cf85e6
unit test fixed
Jammy2211 7a0c007
MockSeardh can output Samples for aggregator tests
Jammy2211 c8a5c3f
fix sensitivfty test
Jammy2211 91f5c0c
Merge branch 'feature/samples_summary_in_result' into feature/keep_th…
Jammy2211 c72a4bd
silence output warning
Jammy2211 e1f356e
silence graphical warnings
Jammy2211 8afd38f
corner plot only when pdf ocnverged to reduce noise
Jammy2211 84eba6b
more silencing noise
Jammy2211 c67e568
merge
Jammy2211 b37256a
all tess pass
Jammy2211 f50dd4f
use getattr trick to disable Result lint errors
Jammy2211 6d6bd43
Result API added to example
Jammy2211 59c2418
fix latent bug
Jammy2211 e0809da
docs
Jammy2211 c97242c
revieew
Jammy2211 2fce21c
more review
Jammy2211 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -151,6 +151,7 @@ def draw_graph( | |
| **kwargs | ||
| ) | ||
| plt.show() | ||
| plt.close() | ||
|
|
||
| @property | ||
| def plates(self): | ||
|
|
||
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just return this line rather than defining an ephemeral variable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is for the example module, which users might read, so intentionally simple.