Skip to content

Conversation

@alexfikl
Copy link
Contributor

@alexfikl alexfikl commented Jul 15, 2025

Similarly to neurophysik/jitcdde#63, there were some line numbers in the examples that didn't quite match (my fault again :(. This fixes them and some other small errors shown by Sphinx.

@alexfikl
Copy link
Contributor Author

The tests failures seem unrelated and are in the Lyapunov test:

  File "/home/runner/work/jitcode/jitcode/tests/test_transversal_lyap.py", line 124, in <module>
    assert ODE1.y[i]==ODE1.y[j], message
AssertionError: The dynamics left the synchronisation manifold when grouped by variables with coupling {'k': -0.1, 'sign': 1}. If this fails, this is a problem with the test and not with what is tested or any software involved.

@Wrzlprmft
Copy link
Contributor

Thanks for fixing this.

The tests failures seem unrelated and are in the Lyapunov test:

Indeed. It seems like the same problem as this one. It came up during the monthly test run, but I hadn’t looked at it yet. Anyway, I implemented similar catches to avoid this, which should fix this.

@Wrzlprmft Wrzlprmft merged commit f9a8622 into neurophysik:master Jul 15, 2025
2 of 6 checks passed
@alexfikl alexfikl deleted the fix-docs-lines branch July 15, 2025 16:07
@alexfikl
Copy link
Contributor Author

Indeed. It seems like the same problem as this one. It came up during the monthly test run, but I hadn’t looked at it yet. Anyway, I implemented similar catches to avoid this, which should fix this.

Yeah, I knew it looked familiar! Thanks for fixing it.

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