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

add_dep_boot #59

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

colintbowers
Copy link
Contributor

Adds DependentBootstrap as a dependency to Bootstrap, and wraps the block bootstrapping methods in DependentBootstrap with new sampling methods StationarySampling, MovingBlockSampling, CircularBlockSampling, and NoOverlapBlockSampling. Relevant issue #45

@juliangehring
Copy link
Owner

Thank you, this looks very nice! It might take some time to test and review the changes - I'll try to finish it as soon as I can.

@colintbowers
Copy link
Contributor Author

No worries, take your time. I certainly did :-) (sorry about that BTW, I had a few unexpected work things come up)

@00krishna
Copy link

Hey Folks, I was wondering what the latest was on this PR? I was hoping to use some Dependent bootstrapping for time series analysis.

@colintbowers
Copy link
Contributor Author

Not sure, I haven't heard anything. However, I can say with certainty that the test suite I added as part of this PR will fail on the latest Julia version since the internals of rand have changed. I've been meaning to try and come up with a test procedure that doesn't depend on random number generation for a while now but haven't found the time.

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

Successfully merging this pull request may close these issues.

3 participants