Updating Compat Section

[hammy4815]

See https://github.com/gridap/GridapMakie.jl/issues/62#issuecomment-1938923425

My working dependency versions:

  [13f3f980] CairoMakie v0.11.8
  [5789e2e9] FileIO v1.16.2
  [1a297f60] FillArrays v1.9.3
  [e9467ef8] GLMakie v0.9.8
  [5c1252a2] GeometryBasics v0.4.10
  [56d4f2e9] Gridap v0.17.23
  [ee78f7c6] Makie v0.20.7

[hammy4815]

I think the issues above were caused from trying to compile the docs (which calls GridapMakie and then Gridap dependencies) with Julia v1.6 instead of 1.9, so I just pushed an update to try and fix the issue.

[HelgeGehring]

Yay, great to see an update :partying_face: Could we directly add julia 1.10 as well?

[fverdugo]

Hi @hammy4815,

the ci failed for this reason:

"This request was automatically failed because there were no enabled runners online to process the request for more than 1 days."

This is the first time I see this problem. Any idea?

[HelgeGehring]

We probably just need to update ubuntu-18.04 which is not supported anymore. In #71 I made a pull request to update it to ubuntu-latest.

[fverdugo]

Hi @hammy4815, the PR #71 has been merged. Can you pull this changes to your fork, and then update this PR?

[hammy4815]

The only tests that are failing are:

fig, _ , sc = plot(Γ, uh, colormap=:algae)

fig, _ , plt = plot(Λ, jump(n_Λ⋅∇(uh)))

Because of the same error:

can't splice MakieCore.ShadingAlgorithm into an OpenGL shader. Make sure all fields are of a concrete type and isbits(FieldType)-->true

Once I have this issue fixed, the tests should pass.

[hammy4815]

There seems to be some sort of bug specifically with how the shaders for GLMakie are being called, and strangely this only happens for SkeletonTriangulations and BoundaryTriangulations. The bug does not exist for CairoMakie however, so the tests work now with CairoMakie...but it's worth noting this issue for GLMakie.

[hammy4815]

@fverdugo ,

Do you have any thoughts regarding the GLMakie vs CairoMakie situation? Otherwise, could we see if it passes the tests this time?

[fverdugo]

Hi @hammy4815,

Thanks for your new attempt!

Since you never contributed to the project I need to manually accept the execution of the tests. Could you please do some dummy change in the repo in a separate PR? This will provably speed up the debugging of this issue.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.66%. Comparing base (0df6796) to head (5673b55). Report is 31 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
- Coverage   93.57%   92.66%   -0.92%     
==========================================
  Files           3        3              
  Lines         218      218              
==========================================
- Hits          204      202       -2     
- Misses         14       16       +2     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[dnuy]

The tests also pass with Gridap v0.18.2, so I suggest to update the Gridap version in the compat section.

(I also tried using the latest Makie but that does not work.)

[JordiManyer]

This has been out-done by PR #73 . Closing this.