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 forecastWith #188

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

Add forecastWith #188

wants to merge 1 commit into from

Conversation

turion
Copy link
Owner

@turion turion commented Sep 23, 2022

No description provided.

@turion
Copy link
Owner Author

turion commented Sep 23, 2022

@reubenharry Maybe this is something that you can work with for the time being, regarding #186 (comment)? You can set the rhine version to this commit and rhine-bayes to #186. If it is useful to you, I'll merge and eventually release.

@reubenharry
Copy link

Ah, very cool, I think this will be useful! What do the a1, a2... represent here? I'm not quite sure I understand exactly what the function does.

@turion
Copy link
Owner Author

turion commented Sep 26, 2022

Since it's a signal function, it will need some input. You need to supply the forecasted input as well. For a pure signal, you'll have a = (), so you can supply [((), ts1), ((), ts2), ...].

@turion
Copy link
Owner Author

turion commented Sep 27, 2022

This doesn't build because stack and nixpkgs don't know about the new version of time-domain yet.

  • Increase time-domain version bound
  • Either wait until stack & nixpkgs are updated or overwrite versions

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.

2 participants