-
Notifications
You must be signed in to change notification settings - Fork 4
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
componentfunction #91
base: main
Are you sure you want to change the base?
Conversation
lines(data) | ||
vlines!(evts.latency, color = (:gray, 0.5)) | ||
xlims!(0, 500) | ||
current_figure() |
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.
[JuliaFormatter] reported by reviewdog 🐶
current_figure() | |
current_figure() |
axislegend(ax) | ||
f | ||
|
||
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` |
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.
[JuliaFormatter] reported by reviewdog 🐶
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` | |
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` |
# No way to find out what size it is without actually generating first... | ||
Base.size( | ||
design::Union{<:SequenceDesign,<:SubselectDesign,<:RepeatDesign{<:SequenceDesign}}, | ||
) = size(generate_events(design), 1) |
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.
[JuliaFormatter] reported by reviewdog 🐶
) = size(generate_events(design), 1) | |
) = size(generate_events(design), 1) |
|
||
|
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.
[JuliaFormatter] reported by reviewdog 🐶
end | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,10}")) == 10 | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,10}B")) == 11 | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 |
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.
[JuliaFormatter] reported by reviewdog 🐶
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 | |
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #91 +/- ##
===========================================
- Coverage 72.70% 62.57% -10.14%
===========================================
Files 10 11 +1
Lines 359 529 +170
===========================================
+ Hits 261 331 +70
- Misses 98 198 +100 ☔ View full report in Codecov by Sentry. |
lines(data) | ||
vlines!(evts.latency, color = (:gray, 0.5)) | ||
xlims!(0, 500) | ||
current_figure() |
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.
[JuliaFormatter] reported by reviewdog 🐶
current_figure() | |
current_figure() |
axislegend(ax) | ||
f | ||
|
||
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` |
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.
[JuliaFormatter] reported by reviewdog 🐶
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` | |
# Voila - the inter-onset intervals are `20` samples longer for condition `B`, exactly as specified.` |
# No way to find out what size it is without actually generating first... | ||
Base.size( | ||
design::Union{<:SequenceDesign,<:SubselectDesign,<:RepeatDesign{<:SequenceDesign}}, | ||
) = size(generate_events(design), 1) |
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.
[JuliaFormatter] reported by reviewdog 🐶
) = size(generate_events(design), 1) | |
) = size(generate_events(design), 1) |
|
||
|
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.
[JuliaFormatter] reported by reviewdog 🐶
end | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,10}")) == 10 | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,10}B")) == 11 | ||
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 |
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.
[JuliaFormatter] reported by reviewdog 🐶
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 | |
@test length(UnfoldSim.sequencestring(StableRNG(1), "A{10,20}")) >= 10 |
Allows to specify a (function,maxlength) tuple instead of an Array for LinearModel & LinearMixedModel.
As the function gets the design as input, this makes modifying the shape of a response depending on the design trivially easy :)
add functionality
add tutorial
add docstring
add unit-test
FormulaOnsets
initial sequence tryout
fix bug in predef_eeg
forgot the end
half way through to success for sequence designs or something
everythinig except sequencelength seems to be working now
added string sequence tests
small doc update
added jitter to the '_' trial divisor
generalized LinearModelComponent to arbitrary functions instead of vectors
bugfix with endless loop due to multiple dispatch
function component for multi-subject