Skip to content
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

ar-irls documentation #26

Open
DemianVera opened this issue Jan 12, 2023 · 1 comment
Open

ar-irls documentation #26

DemianVera opened this issue Jan 12, 2023 · 1 comment

Comments

@DemianVera
Copy link

Is your feature request related to a problem? Please describe.

Hi all!

I have a small question about the documentation of the ar-irls function and the article cited there (Barker et al. 2013). The documentation of this function says: "The code should work well with 10-30 Hz data", but Barker's article shows the advantages of applying iteratively reweighted autoregressive least squares (AR-IRLS) even though the data used were sampled at 4 Hz.
The documentation may have been updated, but the paper cited remained the same.

No solution needed, just an answer.

Thanks!

P.S.: I think in the last update something was changed in the function change_stimulus_duration, because when applied to data from Dr. Hupperts tutorial, it throws some errors (line 45).

@alkvi
Copy link

alkvi commented Apr 12, 2023

I would also be curious if there's additional details about the AR-IRLS implementation regarding relationship to >=10Hz data, other than increased model order as mentioned in the article.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants