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

tests: Convert more tests to np.testing.assert_allclose #2633

Merged
merged 4 commits into from
Apr 14, 2023

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Apr 14, 2023

Implement per_item mode for the derivative variant of SoftMax.
Add missing assertion checks to "heterogeneous variable" tests.
Adjust expected shape in LCControlMechanism and FitzHughNagumoIntegrator tests.

@jvesely jvesely added bug Should work but doesn't tests Test changes labels Apr 14, 2023
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

Increase precision of expected results and use
np.testing.assert_allclose.
Use multiple elements in per-item derivative test inputs.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
The difference between LLVM and Python is caused by shape mismatch in
output port values. The default is 1D (per port), but the Python
returned values are 2D per port.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
…d on execution mode

Apply reduced precision only in single precision compiled mode.

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
@github-actions
Copy link

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely merged commit c3fc203 into PrincetonUniversity:devel Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Should work but doesn't tests Test changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant