Skip to content

Add MethodOfLines to Downstream #1974

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

Merged
merged 2 commits into from
Jan 26, 2023
Merged

Conversation

ChrisRackauckas
Copy link
Member

No description provided.

@ChrisRackauckas
Copy link
Member Author

@xtalax what is this failure?

@xtalax
Copy link
Member

xtalax commented Nov 28, 2022

Haven't seen this before, this passed on the previous CI run. I will investigate.

@xtalax
Copy link
Member

xtalax commented Nov 28, 2022

as of MTK v8.29.1 this error is not present
...
And I do not reproduce as of 8.30, something in MOLs dependencies is limiting MTK to this version

@xtalax
Copy link
Member

xtalax commented Nov 28, 2022

Got it, I think this comes from the default solution interface change, just requires changing order of indexing in test - this only breaks the old solution retrieval method that nobody should be using anymore

@xtalax
Copy link
Member

xtalax commented Jan 26, 2023

@ChrisRackauckas please cycle this and I will fix what comes up

@xtalax
Copy link
Member

xtalax commented Jan 26, 2023

please also add the burgers group as this fails due to a regression from upstream

@ChrisRackauckas ChrisRackauckas merged commit 04d2c6b into master Jan 26, 2023
@ChrisRackauckas ChrisRackauckas deleted the ChrisRackauckas-patch-5 branch January 26, 2023 19:21
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