Skip to content

Conversation

ViralBShah
Copy link
Contributor

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

Add any other context about the problem here.

@ViralBShah
Copy link
Contributor Author

ViralBShah commented Sep 2, 2025

@ChrisRackauckas This doesn't seem to work for x86 and we could just not support that.

It also seems to have issues on windows, which should be fixed before any new release is tagged.

@ViralBShah ViralBShah mentioned this pull request Sep 2, 2025
5 tasks
@ChrisRackauckas
Copy link
Member

ERROR: LoadError: could not load symbol "SUNContext_Create": that sounds like an issue with the binary?

@ChrisRackauckas
Copy link
Member

This doesn't seem to work for x86 and we could just not support that.

I don't think Sundials ever supported. It looks like it's trivial for Claude though since it's just some Int conversions.

@ChrisRackauckas ChrisRackauckas merged commit 920b525 into master Sep 2, 2025
6 of 10 checks passed
@ChrisRackauckas ChrisRackauckas deleted the ViralBShah-patch-1 branch September 2, 2025 11:21
@ViralBShah
Copy link
Contributor Author

The windows is puzzling. I wonder what is going on. Maybe need to check the build logs.

@ViralBShah
Copy link
Contributor Author

ViralBShah commented Sep 3, 2025

Windows works fine for me. I think there are some static .a files that are in lib that may be confusing matters, and there could be a new build of Sundials that gets rid of them.

The first thing though is to be able to reproduce this issue, but I have no issues precompiling and running tests on Windows (in UTM on my mac). I am seeing test failures though for which I will open a new issue.

@ViralBShah
Copy link
Contributor Author

I have a feeling that the runners for windows and linux x86 are broken. Is there a way to fix these or reinstall them or something?

@ViralBShah
Copy link
Contributor Author

In https://github.com/SciML/Sundials.jl/actions/runs/17417995538/job/49450603795, the depot points to /home/chrisrackauckas/.julia. Is that how this is supposed to be?

@ChrisRackauckas
Copy link
Member

That's for the runners that are running on deepsea. It's "sufficient", we should probably do some docker thing or something but it works well enough.

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