fix: master branch is failing precompilation

[sathvikbhagavan]

Getting an error:

julia> using Surrogates
[ Info: Precompiling Surrogates [6fc51010-71bc-11e9-0e15-a3fcc6593c49]
ERROR: LoadError: syntax: invalid interpolation syntax: "$$"
Stacktrace:
 [1] top-level scope
   @ ~/oss/Surrogates.jl/src/Radials.jl:37
 [2] include(mod::Module, _path::String)
   @ Base ./Base.jl:457
 [3] include(x::String)
   @ Surrogates ~/oss/Surrogates.jl/src/Surrogates.jl:1
 [4] top-level scope
   @ ~/oss/Surrogates.jl/src/Surrogates.jl:7
 [5] include
   @ ./Base.jl:457 [inlined]
 [6] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::Nothing)
   @ Base ./loading.jl:2049
 [7] top-level scope
   @ stdin:3

Same error in the CI build on master - https://github.com/SciML/Surrogates.jl/actions/runs/5554601185/job/15046314727

This should fix that

[ai-maintainer]

AI-Maintainer Review for PR - master branch is failing precompilation

Title and Description 👍

The Title and Description are clear and concise

The title and description of the pull request are clear and concise. They effectively communicate the purpose of the changes, which is to fix the precompilation failure on the master branch. The error message and the link to the CI build where the error occurred provide useful context.

Scope of Changes 👍

The changes are narrowly focused

The changes in this pull request are narrowly focused on resolving a specific issue. The modifications to the Radials.jl file are directly related to fixing the error mentioned in the description. There is no indication that the author is trying to resolve multiple issues simultaneously.

Docstrings 👍

All new functions, classes, or methods have appropriate docstrings

The diff does not show any new functions, classes, or methods being added. Therefore, there are no missing docstrings in the changes made.

Testing 👎

Testing methodology is not described

The description does not provide information about how the author tested the changes. It would be helpful for the author to provide details about the testing methodology to ensure the changes have been thoroughly validated.

Suggested Changes

Please provide information about how you tested the changes. This could include unit tests you ran, manual testing you performed, or any other relevant testing methodology. This will help reviewers understand how you ensured the changes are effective and do not introduce new issues.

Reviewed with AI Maintainer

[sathvikbhagavan]

@ChrisRackauckas can we merge this?

[ChrisRackauckas]

This also fails?

[sathvikbhagavan]

Hmm, its a different error in CI / test (Core 1.6)

✓ Surrogates
✓ CUDA
✓ cuDNN
✓ NNlibCUDA
✓ Flux
133 dependencies successfully precompiled in 157 seconds
   Testing Running tests...
 Resolving package versions...
ERROR: LoadError: Unsatisfiable requirements detected for package Statistics [10745b16]:
Statistics [10745b16] log:
├─possible versions are: 0.0.0 or uninstalled
├─Statistics [10745b16] is fixed to version 0.0.0
└─found to have no compatible versions left with Zygote [e88e6eb3]
 └─Zygote [e88e6eb3] log:
   ├─possible versions are: 0.1.0-0.6.63 or uninstalled
   └─restricted to versions 0.4-0.6 by Surrogates [6fc51010], leaving only versions 0.4.0-0.6.63
     └─Surrogates [6fc51010] log:
       ├─possible versions are: 6.5.1 or uninstalled
       ├─restricted to versions 6 by SurrogatesAbstractGPs [78aa1720], leaving only versions 6.5.1
       │ └─SurrogatesAbstractGPs [78aa1720] log:
       │   ├─possible versions are: 0.1.0 or uninstalled
       │   └─SurrogatesAbstractGPs [78aa1720] is fixed to version 0.1.0
       └─Surrogates [6fc51010] is fixed to version 6.5.1

Surrogates is getting precompiled though. I am not sure why the error is coming though.

[ChrisRackauckas]

Latest Turing doesn't allow v1.6: is this related?

[sathvikbhagavan]

Surrogates doesn't use Turing.jl afaik

Can you trigger the workflow for the latest julia version - it worked for me locally (although tests were failing - unrelated reason) and it was cancelled in the previous run.

[sathvikbhagavan]

Why does CI / test (Core, 1) job gets cancelled?

[ChrisRackauckas]

Because v1.6 fails.

[sathvikbhagavan]

Ok, Surrogates does precompile on both but I am not sure why we are getting a version conflict on 1.6. Any ideas to fix this?

[ChrisRackauckas]

Some package requires >v1.6, something in the GP stack.

[sathvikbhagavan]

One small thing I tried was adding this line in runtests:

Pkg.add(name="Statistics", version=VERSION)

and this worked if I did ]test and did not error out with the package resolution issue in julia 1.6

[KristofferC]

I'll try figure out what is going on here.

[KristofferC]

The following fixes the issue on 1.6 Pkg:

diff --git a/src/Operations.jl b/src/Operations.jl
index 0644e0ee7..6ac5a48c4 100644
--- a/src/Operations.jl
+++ b/src/Operations.jl
@@ -281,7 +281,7 @@ function collect_project!(ctx::Context, pkg:
:PackageSpec, path::String,
         pkg.version = project.version
     else
         # @warn("project file for $(pkg.name) is missing a `version` entry")
-        pkg.version = VersionNumber(0)
+        pkg.version = VERSION # VersionNumber(0)
     end
     return
 end

Since

Pkg.add(name="Statistics", version=VERSION)

seems to work I would go with that on 1.6 though as a workaround.

[ChrisRackauckas]

okay thanks let's do that.