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

Roll HSGPPeriodic into HSGP #7470

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Roll HSGPPeriodic into HSGP #7470

wants to merge 2 commits into from

Conversation

maresb
Copy link
Contributor

@maresb maresb commented Aug 21, 2024

Description

Not much here yet, but the intention is to eliminate HSGPPeriodic in favor of HSGP as a first step to implementing boundary conditions.

Related Issue

  • Closes #
  • Related to #

Checklist

Type of change

  • New feature / enhancement
  • Bug fix
  • Documentation
  • Maintenance
  • Other (please specify):

📚 Documentation preview 📚: https://pymc--7470.org.readthedocs.build/en/7470/

@maresb
Copy link
Contributor Author

maresb commented Aug 21, 2024

Ping @bwengals @AlexAndorra

@maresb
Copy link
Contributor Author

maresb commented Aug 21, 2024

TODO: why is it sqrt_psd in HSGP and psd in HSGPPeriodic? Which is it, or is the psd just not square rooted in the periodic version?

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.17%. Comparing base (68a7578) to head (5362113).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7470      +/-   ##
==========================================
+ Coverage   87.49%   92.17%   +4.67%     
==========================================
  Files         103      103              
  Lines       17215    17220       +5     
==========================================
+ Hits        15063    15872     +809     
+ Misses       2152     1348     -804     
Files Coverage Δ
pymc/gp/hsgp_approx.py 88.70% <100.00%> (+0.31%) ⬆️

... and 8 files with indirect coverage changes

@bwengals
Copy link
Contributor

TODO: why is it sqrt_psd in HSGP and psd in HSGPPeriodic? Which is it, or is the psd just not square rooted in the periodic version?

The goal was a direct implementation appendix B from this paper

@maresb
Copy link
Contributor Author

maresb commented Aug 23, 2024

A GP model with a periodic covariance function does no fit in the framework of the HSGP approximation covered in this study

I'm pretty sure it fits in with the manifold HSGP framework outlined in ... that paper. So I think that if we cite that as well we could remove the comment about it not being a real HSGP.

@maresb
Copy link
Contributor Author

maresb commented Aug 23, 2024

Oh!!! Sorry @bwengals, I wasn't reading that closely enough. They're not quite doing what I expected. I'm 99% sure it's not qualitatively different from what I had in mind, but I need to read up and figure exactly how what they're doing differs from the canonical manifolds-based approach.

@maresb
Copy link
Contributor Author

maresb commented Aug 23, 2024

I just did a rebase because I was accidentally working from an old version of the main branch. There are no new changes relative to the rebase.

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