-
Notifications
You must be signed in to change notification settings - Fork 194
Weighted sem #754
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
Weighted sem #754
Changes from all commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
f2f83cf
Add weighted `sem`
46be952
add weighted sem method
158fb35
Correct missing method
43c7419
bug fix
e770708
Bug fix
e5a03a8
Update src/scalarstats.jl
82e4de2
Update src/scalarstats.jl
5eba9d4
Update src/scalarstats.jl
2ef18eb
change μ to mean
1307200
Update src/scalarstats.jl
7aafe38
broadcast weights
aed6531
Update src/scalarstats.jl
e49430c
Work with arbitrary iterators, fuse loops
99d8505
Update src/scalarstats.jl
5cd2e4d
Update src/scalarstats.jl
43e354f
Apply review comments
44d0b0e
RealArray instead of Array{<:Number}
094c30c
Improve citation
a3c7e61
Apply code review suggestions
5d25c7a
whitespace
59f8b8e
Update src/scalarstats.jl
73cfb64
Update src/scalarstats.jl
af78ffa
Update src/scalarstats.jl
9264d0a
Update src/scalarstats.jl
50ee853
Update src/scalarstats.jl
c83beea
Update src/scalarstats.jl
4f25268
Update src/scalarstats.jl
9ef970a
Apply suggestions
1ea8ef6
formatting
3f2361f
Update src/scalarstats.jl
b882c49
Drop real requirement
18cf255
Update src/scalarstats.jl
8b556bc
Test mean keyword with FrequencyWeights
dcff0b1
Update src/scalarstats.jl
162631b
Update src/scalarstats.jl
d213c93
Update src/scalarstats.jl
a93a15d
Use original `sem` implementation
74d2c3c
use original SEM implementation
adaf472
original sem implementation
acf81ea
original sem implementation
303c514
test fix
5bd4a91
Dispatch on mean
8d904b5
remove internal method for weights
f762e37
Bug
529cd21
Apply code review
8afc950
Update src/scalarstats.jl
989a383
Fix tests
b469d6d
Return NaN for FrequencyWeights
e25c4b0
Type instability fix
d56a98f
Fix array version to return NaN instead of error
22c8cc0
Remove useless tests
6b90e77
Update src/scalarstats.jl
f3c9714
Update src/scalarstats.jl
nalimilan cfba9a1
Update test/scalarstats.jl
nalimilan d024475
Update src/scalarstats.jl
b64d44d
Add test
e31748a
Update src/scalarstats.jl
72d1e48
Fix type instability
6ceeac4
Bug fix
5779144
Test for type stability
a520d20
Fix type instability
7b40f1c
mean type fix
4683ec4
test for type inference
ab29aa7
Update src/scalarstats.jl
1f32003
Update src/scalarstats.jl
632e212
Update src/scalarstats.jl
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
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.
Uh oh!
There was an error while loading. Please reload this page.