Skip to content

Commit

Permalink
Merge pull request #125 from JuliaGaussianProcesses/freeze-deps
Browse files Browse the repository at this point in the history
Clean up and fix tests
  • Loading branch information
simsurace committed Jan 23, 2024
2 parents 2e6ccab + 064bb53 commit 6023595
Show file tree
Hide file tree
Showing 8 changed files with 16 additions and 20 deletions.
10 changes: 5 additions & 5 deletions Project.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name = "TemporalGPs"
uuid = "e155a3c4-0841-43e1-8b83-a0e4f03cc18f"
authors = ["willtebbutt <wt0881@my.bristol.ac.uk> and contributors"]
version = "0.6.6"
version = "0.6.7"

[deps]
AbstractGPs = "99985d1d-32ba-4be9-9821-2ec096f28918"
Expand All @@ -20,10 +20,10 @@ Zygote = "e88e6eb3-aa80-5325-afca-941959d7151f"
AbstractGPs = "0.5.17"
Bessels = "0.2.8"
BlockDiagonals = "0.1.7"
ChainRulesCore = "1"
FillArrays = "0.13.0 - 0.13.7, 1"
ChainRulesCore = "1.0.0 - 1.16.0"
FillArrays = "0.13.0 - 0.13.7"
KernelFunctions = "0.9, 0.10.1"
StaticArrays = "1"
StaticArrays = "1.0.0 - 1.6.5"
StructArrays = "0.5, 0.6"
Zygote = "0.6.65"
Zygote = "0.6.65 - 0.6.65"
julia = "1.6"
9 changes: 9 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,15 @@ TemporalGPs.jl is registered, so simply type the following at the REPL:
```
While you can install TemporalGPs without AbstractGPs and KernelFunctions, in practice the latter are needed for all common tasks in TemporalGPs.

## Note !!!

This package is currently not guaranteed to work with all current versions of dependencies. If something is not working on the current release of TemporalGPs,
please try out v0.6.7, which pins some dependencies in order to circumvent some of the problems. You can do so by typing instead:
```julia
] add AbstractGPs KernelFunctions TemporalGPs@0.6.7
```
Please report an issue if this work-around fails.

# Example Usage

Most examples can be found in the [examples](https://github.com/JuliaGaussianProcesses/TemporalGPs.jl/tree/master/examples) directory. In particular see the associated [README](https://github.com/JuliaGaussianProcesses/TemporalGPs.jl/tree/master/examples/README.md).
Expand Down
5 changes: 2 additions & 3 deletions test/gp/lti_sde.jl
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,7 @@ using KernelFunctions: kappa
using ChainRulesTestUtils
using TemporalGPs: build_lgssm, StorageType, is_of_storage_type, lgssm_components
using Test
include("../test_util.jl")
include("../models/model_test_utils.jl")

_logistic(x) = 1 / (1 + exp(-x))

# Everything is tested once the LGSSM is constructed, so it is sufficient just to ensure
Expand Down Expand Up @@ -226,7 +225,7 @@ println("lti_sde:")

# Just need to ensure we can differentiate through construction properly.
if isnothing(kernel.to_vec_grad)
@test_broken "Gradient tests are not passing"
@test_broken false # "Gradient tests are not passing"
continue
elseif kernel.to_vec_grad
test_zygote_grad_finite_differences_compatible(
Expand Down
3 changes: 0 additions & 3 deletions test/models/lgssm.jl
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,6 @@ using LinearAlgebra
using StructArrays
using Zygote, StaticArrays

include("model_test_utils.jl")
include("../test_util.jl")

println("lgssm:")
@testset "lgssm" begin

Expand Down
3 changes: 0 additions & 3 deletions test/models/linear_gaussian_conditionals.jl
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
using TemporalGPs: posterior_and_lml, predict, predict_marginals
using Test

include("../test_util.jl")
include("../models/model_test_utils.jl")

println("linear_gaussian_conditionals:")
@testset "linear_gaussian_conditionals" begin
Dlats = [1, 3]
Expand Down
3 changes: 0 additions & 3 deletions test/models/missings.jl
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@ using Random: randperm
using ChainRulesTestUtils
using Zygote: Context

include("../test_util.jl")
include("../models/model_test_utils.jl")

@info "missings:"
@testset "missings" begin

Expand Down
2 changes: 0 additions & 2 deletions test/space_time/pseudo_point.jl
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,6 @@ using TemporalGPs:
Separable,
approx_posterior_marginals
using Test
include("../test_util.jl")
include("../models/model_test_utils.jl")

@testset "pseudo_point" begin

Expand Down
1 change: 0 additions & 1 deletion test/util/chainrules.jl
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ using TemporalGPs: time_exp, _map, Gaussian
using FillArrays
using StructArrays
using Zygote: ZygoteRuleConfig
include("../test_util.jl")

@testset "chainrules" begin
@testset "StaticArrays" begin
Expand Down

2 comments on commit 6023595

@simsurace
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JuliaRegistrator register()

@JuliaRegistrator
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Registration pull request created: JuliaRegistries/General/99368

Tip: Release Notes

Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.

@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.

Tagging

After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.

This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:

git tag -a v0.6.7 -m "<description of version>" 60235954cf892b6afe197f531641a2de36beaf8e
git push origin v0.6.7

Please sign in to comment.