dwavesystems / dwave-system

An API for easily incorporating the D-Wave system as a sampler, either directly or through Leap's cloud-based hybrid samplers
https://docs.ocean.dwavesys.com/
Apache License 2.0
90 stars 64 forks source link

Use default resource class for macos builds on CircleCi #523

Closed randomir closed 5 months ago

randomir commented 5 months ago

Support for macos.x86.medium.gen2 is soon to be dropped.

See: https://discuss.circleci.com/t/macos-intel-support-deprecation-in-january-2024/48718

codecov[bot] commented 5 months ago

Codecov Report

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

Project coverage is 86.70%. Comparing base (9fe827d) to head (84f3fc6).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #523 +/- ## ========================================== - Coverage 89.65% 86.70% -2.96% ========================================== Files 24 24 Lines 1760 1760 ========================================== - Hits 1578 1526 -52 - Misses 182 234 +52 ```

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

randomir commented 5 months ago

Looks like tests.test_virtual_graph_composite.TestVirtualGraphWithMockDWaveSampler.test_smoke now segfaults on python 3.8-3.10 running on M1.

The minimal segfault code is:

from dwave.system.composites import VirtualGraphComposite
from dwave.system.testing import MockDWaveSampler

child_sampler = MockDWaveSampler()
sampler = VirtualGraphComposite(child_sampler, {'a': [0]}, flux_bias_num_reads=1)
arcondello commented 5 months ago

Seems to be an issue with SciPy. I tested a few versions and while scipy~=1.8.0 fails, scipy==1.9.0 succeeds.

randomir commented 5 months ago

@thisac, good catch. I've updated macos jobs (for consistency if for nothing else), although it doesn't seem to make that much of a difference here (maybe it's the new mac? or new OS? homebrew update used to be excruciatingly slow on circleci, but maybe they are caching it on their infra now :shrug:).