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 to MTK 9 #128

Merged
merged 9 commits into from
Jul 17, 2024
Merged

Update to MTK 9 #128

merged 9 commits into from
Jul 17, 2024

Conversation

termi-official
Copy link
Contributor

Updates to new MTK version and hot-fixes master CI.

test/noble_1962.jl Outdated Show resolved Hide resolved
test/noble_1962.jl Outdated Show resolved Hide resolved
@termi-official
Copy link
Contributor Author

I cannot reproduce the downgrade issue locally with https://github.com/julia-actions/julia-downgrade-compat/blob/main/downgrade.jl .

@ChrisRackauckas
Copy link
Member

I wouldn't worry too much about downgrade.

Is the only failure now Aqua?

@termi-official
Copy link
Contributor Author

termi-official commented Jul 17, 2024

Yes. Aqua only fails in CI. It does not fail locally. I will try to dig into this later.

@termi-official
Copy link
Contributor Author

termi-official commented Jul 17, 2024

The last commit should fix the compat issue. However, I could not find any way to reproduce the CI failure locally.

Edit: Can now (thanks to Fredrik Ekre)

@termi-official
Copy link
Contributor Author

Okay the Aqua failure is essentially SciML/ModelingToolkit.jl#2827. I think I can fix this.

@termi-official
Copy link
Contributor Author

CI fails now locally with a different error using the SymbolicUtils.jl fix. But the Aqua issue seems to be resolved at least.

@termi-official
Copy link
Contributor Author

I think I fixed all issues now.

@ChrisRackauckas ChrisRackauckas merged commit 78cec21 into SciML:master Jul 17, 2024
7 of 10 checks passed
@termi-official termi-official deleted the do/mtk-update branch July 17, 2024 18:29
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.

2 participants