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

Update benchmark.yml to use lts instead of 1.6. #443

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

asinghvi17
Copy link
Contributor

No description provided.

@rafaqz
Copy link
Collaborator

rafaqz commented Nov 13, 2024

Huh

  GDALError (CE_Failure, code 1):
  	OGRFieldDefn::SetDefault() not allowed on a sealed object

@evetion
Copy link
Collaborator

evetion commented Nov 13, 2024

I think some tests are just broken on master, might be a recent gdal release?

@rafaqz
Copy link
Collaborator

rafaqz commented Nov 13, 2024

Yes it must be. But looks like some fixing is needed

@yeesian
Copy link
Owner

yeesian commented Nov 14, 2024

Is the failure in https://github.com/yeesian/ArchGDAL.jl/actions/runs/11822349442/job/32939196358?pr=443 expected with this change?

@evetion
Copy link
Collaborator

evetion commented Nov 14, 2024

Is the failure in yeesian/ArchGDAL.jl/actions/runs/11822349442/job/32939196358?pr=443 expected with this change?

The error stems from m-j-w/CpuId.jl#62. I think that's because the latest macos runner has changed infrastructure? Probably best to pin the architecture to x64 (and maybe ubuntu instead of macos?)

@yeesian yeesian merged commit 75fae34 into yeesian:master Nov 14, 2024
1 of 11 checks passed
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.

4 participants