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

Adjust failing nightly tests for #653 #671

Closed
khaeru opened this issue Nov 11, 2022 · 1 comment · Fixed by #689
Closed

Adjust failing nightly tests for #653 #671

khaeru opened this issue Nov 11, 2022 · 1 comment · Fixed by #689
Assignees
Labels
bug Doesn't work as advertised/unintended effects ci Continuous integration

Comments

@khaeru
Copy link
Member

khaeru commented Nov 11, 2022

Since about 2022-11-08, nightly test jobs have failed with messages like the following. See logs 1, 2.

FAILED message_ix/tests/test_nightly.py::test_scenario[CD_Links_SSP2_baseline] - AssertionError: assert False
 +  where False = functools.partial(<function isclose at 0x7f465a8d6170>, rtol=3e-06)(3870157.5, 3869529.75)
 +    where functools.partial(<function isclose at 0x7f465a8d6170>, rtol=3e-06) = eval('partial(np.isclose, rtol=3e-6)')
FAILED message_ix/tests/test_nightly.py::test_scenario[CD_Links_SSP2_v2_1000] - AssertionError: assert False
 +  where False = functools.partial(<function isclose at 0x7f465a8d6170>, rtol=3e-06)(3358441.5, 3359089.0)
 +    where functools.partial(<function isclose at 0x7f465a8d6170>, rtol=3e-06) = eval('partial(np.isclose, rtol=3e-6)')

Per this Slack thread (private):

May be related to #667, #668, or #669.

@khaeru khaeru added bug Doesn't work as advertised/unintended effects ci Continuous integration labels Nov 11, 2022
@khaeru
Copy link
Member Author

khaeru commented Nov 11, 2022

@gidden @OFR-IIASA @behnam-zakeri —I assigned all three of you, but please remove 1 or 2 when you decide who in particular will address this.

If that's to be done through #669, please remember basic housekeeping:

  • Assign the PR to the person responsible.
  • Add it to the "MESSAGEix team" project tracker.
  • Apply any relevant labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Doesn't work as advertised/unintended effects ci Continuous integration
Projects
None yet
4 participants