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

EpiAware specific predict method to handle prediction edge cases #380

Closed
wants to merge 25 commits into from

Conversation

seabbs
Copy link
Collaborator

@seabbs seabbs commented Jul 17, 2024

This PR was spun out of #369 and implements a custom predict method and predict context. See that PR for discussion

@seabbs seabbs changed the base branch from main to optimise-mvnormal-scan July 17, 2024 17:50
@seabbs seabbs changed the title Hack predict EpiAware specific predict method to handle prediction edge cases Jul 17, 2024
@seabbs
Copy link
Collaborator Author

seabbs commented Jul 17, 2024

I'm still stuck on this but tempted to pause work on it as the fix in Turing seems to be gaining traction (TuringLang/Turing.jl#2239) and so maybe we can just wait on that.

@SamuelBrand1
Copy link
Collaborator

I'm still stuck on this but tempted to pause work on it as the fix in Turing seems to be gaining traction (TuringLang/Turing.jl#2239) and so maybe we can just wait on that.

I think thats a good idea.

Base automatically changed from optimise-mvnormal-scan to main July 18, 2024 09:21
@seabbs
Copy link
Collaborator Author

seabbs commented Jul 25, 2024

It seems like progress on this on the Turing side might be quite slow so we might need to implement something here sooner rather than later.

@SamuelBrand1
Copy link
Collaborator

How is this going?

@seabbs
Copy link
Collaborator Author

seabbs commented Aug 28, 2024

it was passed to wait for Turing. Do you think we need it soon?

@seabbs seabbs closed this Oct 24, 2024
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