-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Make VI (posterior) mean and std accessible as a structured xarray #6387
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #6387 +/- ##
==========================================
- Coverage 94.79% 84.92% -9.87%
==========================================
Files 148 148
Lines 27730 27785 +55
==========================================
- Hits 26287 23597 -2690
- Misses 1443 4188 +2745
|
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.
The variational tests were moved into tests/variational. I am not sure where this one should go, perhaps @Armavica can weigh in?
I think |
@fonnesbeck Any update on this? |
OK, tests moved over. |
@ricardoV94 are you happy with this? |
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.
There is a deprecated method and the use of the |
operator for dicts which is not supported in our oldest Python dependency (test is failing)
Otherwise LGTM
@@ -977,7 +978,7 @@ def symbolic_random(self): | |||
|
|||
@pytensor.config.change_flags(compute_test_value="off") | |||
def set_size_and_deterministic( | |||
self, node: Variable, s, d: bool, more_replacements: dict | None = None | |||
self, node: Variable, s, d: bool, more_replacements: dict = None |
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.
self, node: Variable, s, d: bool, more_replacements: dict = None | |
self, node: Variable, s, d: bool, more_replacements: Optional[dict] = None |
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.
Unfortunately, the pre-commit hook reverts this back to dict | None = None
. A little too agressive!
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.
You can commit with --no-verify to skip the pre-commit
Co-authored-by: Michael Osthege <michael.osthege@outlook.com>
Co-authored-by: Michael Osthege <michael.osthege@outlook.com>
Co-authored-by: Michael Osthege <michael.osthege@outlook.com>
Co-authored-by: Michael Osthege <michael.osthege@outlook.com>
What is this PR about?
Moved from #6086 so that merge conflicts could be fixed. See original PR for details.