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

residues test fails on Fedora #80

Closed
susilehtola opened this issue Jun 26, 2022 · 5 comments · Fixed by #86
Closed

residues test fails on Fedora #80

susilehtola opened this issue Jun 26, 2022 · 5 comments · Fixed by #86

Comments

@susilehtola
Copy link
Collaborator

build log: https://kojipkgs.fedoraproject.org//work/tasks/2687/88772687/build.log

 21/114 Test  #26: residues .........................***Failed    0.17 sec
          MOPAC Job: "SITE_Individual_residues.mop" ended normally on Jun 26, 2022, at 20:09.
WARNING: output file size mismatch, 616 vs. 66
Traceback (most recent call last):
  File "/builddir/build/BUILD/mopac-22.0.3/tests/run_test.py", line 32, in <module>
    compare_mopac_out_file(out_line, out_list, ref_line, ref_list)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/mopac-22.0.3/tests/compare_output.py", line 225, in compare_mopac_out_file
    assert ref == out, f'ERROR: string mismatch between {ref} on reference line {ref_line0} and {out} on output line {out_line0}'
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: ERROR: string mismatch between ******************************************************************************* on reference line 1 and END on output line 2
@godotalgorithm
Copy link
Collaborator

Can you provide more information about this? Is this happening on only one machine in the Fedora tests? Is there anything especially distinctive about this machine?

This looks like an input parsing error: the calculation seems to terminate almost immediately for the failed test. In examining the test, it's using a keyword line extender feature that I'm no longer officially supporting, and I'll adjust the test to use the supported version of the feature. Hopefully the test will be more robust after the switch ...

@susilehtola
Copy link
Collaborator Author

The fail occurs on a x86_64 builder. The builders tend to be pretty big machines, with many processors and memory.

There are also test failures for pdb_compare2 and pdb_compare3 on aarch64:

 21/114 Test  #13: pdb_compare2 .....................***Failed    0.49 sec
          MOPAC Job: "Compare_PDB_Bacteriorhodopsin_1AP9_and_2AT9.mop" ended normally on Jun 26, 2022, at 20:10.
WARNING: output file size mismatch, 5677 vs. 66
Traceback (most recent call last):
  File "/builddir/build/BUILD/mopac-22.0.3/tests/run_test.py", line 32, in <module>
    compare_mopac_out_file(out_line, out_list, ref_line, ref_list)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/mopac-22.0.3/tests/compare_output.py", line 225, in compare_mopac_out_file
    assert ref == out, f'ERROR: string mismatch between {ref} on reference line {ref_line0} and {out} on output line {out_line0}'
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: ERROR: string mismatch between ******************************************************************************* on reference line 1 and END on output line 2

and

 23/114 Test  #14: pdb_compare3 .....................***Failed    0.51 sec
          MOPAC Job: "Compare_PDB_Bacteriorhodopsin_1AP9_and_1C3W.mop" ended normally on Jun 26, 2022, at 20:10.
WARNING: output file size mismatch, 4877 vs. 66
Traceback (most recent call last):
  File "/builddir/build/BUILD/mopac-22.0.3/tests/run_test.py", line 32, in <module>
    compare_mopac_out_file(out_line, out_list, ref_line, ref_list)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/builddir/build/BUILD/mopac-22.0.3/tests/compare_output.py", line 225, in compare_mopac_out_file
    assert ref == out, f'ERROR: string mismatch between {ref} on reference line {ref_line0} and {out} on output line {out_line0}'
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError: ERROR: string mismatch between ******************************************************************************* on reference line 1 and END on output line 2

@godotalgorithm
Copy link
Collaborator

The pbe_compare tests are another concurrency issue - I didn't notice that they are all acting on the same file. I think I'll go back and adjust my previous concurrency fix, since there are now many tests using the same file. I'll copy all auxiliary files needed by the tests to the testing subdirectory of the build directory before running any tests.

Is the "residues" test failure intermittent as with these other concurrency-driven test failures?

@susilehtola
Copy link
Collaborator Author

Is the "residues" test failure intermittent as with these other concurrency-driven test failures?

It may be...

@godotalgorithm
Copy link
Collaborator

Ok, the "residues" test depends on a file that is regenerated by another test (IONIZE.arc), so it is just a concurrency issue, but a more subtle one.

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 a pull request may close this issue.

2 participants