-
Notifications
You must be signed in to change notification settings - Fork 58
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
Chromatic noise updates #242
base: master
Are you sure you want to change the base?
Conversation
merging master in
…l for dm_gp and chrom_gp
Adding option to set dt
tagging @Hazboun6 and @blarsen10 since Jeff coded most of these changes originally.
|
This reverts commit 33d2b26.
@AaronDJohnson do you recognize the errors that are happening in the |
This also now adds support for the SW GP basis & kernel from Nitu et al 2024 |
@blarsen10 and @jeremy-baier Can we have a look at the errors here ASAP. From what I saw it's just not passing the linting rules. It would be good to get this merged this week. |
@jeremy-baier and @blarsen10 I think that the reason the tests are still failing is that GitHub CI uses the |
Add the limit on the `scipy` version
Sorry for the delay on all of this. Feel free to ping me again if you don't get a reply quickly next time! I did know exactly what was happening, and I've merged a fix. If @jeremy-baier pulls from upstream/main, the Mac tests should be fixed now. The new runners only support ARM architectures, so we have to go back one version on the CI runners so that they run an x86 architecture instead. |
I've also fixed the |
This PR contains many various updates associated with chromatic noise modeling efforts over the past several years.
Notable changes include:
model_log_weights.json
)