-
Notifications
You must be signed in to change notification settings - Fork 27
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
More statistics functions #115
Merged
Merged
Changes from 4 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
3ec58b3
function to save z, dz
chrisbrahms c8810d0
t and ω energyfunctions
chrisbrahms 948dbb0
testing t and ω energy-functions
chrisbrahms 59bf810
Merge branch 'master' into stats
chrisbrahms 5ef705d
t and ω energy function for modes
chrisbrahms 13c1229
energy stat
chrisbrahms 395a2a7
test script
chrisbrahms cea4a67
Merge branch 'master' into stats
chrisbrahms 92e6dd8
energy functions
chrisbrahms 577d714
change MemoryOutput call signature
chrisbrahms 202a00a
improve hilbert transform, add plan_hilbert
chrisbrahms d764aa3
load/save FFTW wisdom
chrisbrahms 35aef02
some more stats
chrisbrahms e50c732
docstrings
chrisbrahms f699edf
Merge branch 'master' into stats
chrisbrahms 2cde7cb
switch back to separate transforms
chrisbrahms 1732406
docstring, clutter
chrisbrahms 4c5437e
FWHM, energy in window
chrisbrahms d78efe5
switch to window for energy_λ to fix envelope
chrisbrahms 72f090c
rename energy functions
chrisbrahms ffebd5c
make ADK function type-stable
chrisbrahms ace7626
fix electron density, typo
chrisbrahms 6eb07f0
add FWHM with linear interpolation
chrisbrahms 19f6d41
docstrings
chrisbrahms 7ed5fca
docstrings
chrisbrahms 1330b9d
typo
chrisbrahms c947685
Merge branch 'master' into stats
chrisbrahms 5c1ea66
Merge branch 'master' into stats
chrisbrahms 70d2e16
improve FWHM
chrisbrahms d9c1010
compact printing
chrisbrahms 4ef1141
save simulation type
chrisbrahms 8c8f742
Merge branch 'master' into stats
chrisbrahms d26aa68
electron density for multimode
chrisbrahms ddf9e37
radial FWHM statistic
chrisbrahms 546962a
docstrings, combine functions
chrisbrahms 8ce0161
Merge branch 'master-saved' into stats
chrisbrahms e4c1ba2
Merge branch 'master' into stats
chrisbrahms 572e780
switch to BSpline in FWHM with spline method
chrisbrahms 3672faf
save transform as string
chrisbrahms aad6ce8
tests pass
chrisbrahms 9762587
string representation for modes in output
chrisbrahms ae64861
fix npol
chrisbrahms 499f70a
more roots for BSpline FWHM
chrisbrahms eb2385c
peak intensity
chrisbrahms e43eb45
docstring
chrisbrahms 59119b8
add modal stats tests
chrisbrahms 2baafb1
mode-averaged peak intensity
chrisbrahms f6a67e5
docstring
chrisbrahms 41cd416
correct docstring for TransFree
chrisbrahms c1b5d8b
add dumps of linop and transform to output
chrisbrahms 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 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 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 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 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 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 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 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
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.
For horrible functions the number of roots could be very high, we probably need to increase the argument to the
Dierckx
roots function. See here:https://github.com/kbarbary/Dierckx.jl/blob/master/src/Dierckx.jl#L378
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.
any feeling for how many might be enough?
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.
Well for a "noise burst" it could be thousands. That might come with a performance hit though. To be clear, passing a high
maxn
to Dierckx doesn't cost anything if there are not many roots. But if there are thousands then it will find all of them.This isn't just academic, when looking for pulse convergence in say modelocking (or a FOPO!), a key metric is if the min and max FWHM match. We could be doing a lot of that quite soon. I suppose in that case we'd specify a different method here, and only use the spline for more precise requirements.
It might be worth adding a "noise burst" test though.
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.
With this pulse:
it finds around 600 roots, FWHM max still works, and it still only takes 1.5 ms with 2^14 points
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.
funnily, the warning in
Dierckx.jl
when more roots thanmaxn
were found is now an exception, becausewarn
no longer exists. Medium to long term it would be good to no longer depend on that package.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.
Agreed, but Dierckx (the fortran part) is by far the highest quality and most tested spline package I've found so far. And the wrapper code is rather minimal, so we can help maintain it. If I had sufficient free time I'd translate it, but alas I can;t see that happening very soon.