Test: delete threshold files in the examples with warnings and update the result.ref files
#5402
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WHUweiqingzhou
changed the title
threshold files in the examples with warnings and update the result.ref files
Collaborator
Author
|
Two examples were found in CI tests, which indicates they cannot reach the threshold: |
mohanchen
added
the
Compile & CICD & Docs & Dependencies
mohanchen
approved these changes
Nov 21, 2024
Fisherd99
pushed a commit
to Fisherd99/abacus-BSE
that referenced
this pull request
Mar 31, 2025
…te the `result.ref` files (deepmodeling#5402) * delete threshold and update the ref * recover some unnecessary changes * recover some unnecessary changes * recover some unnecessary changes * recompile and update the ref * add threshold for two unstable tests * add threshold for two unstable TDDFT tests * update threshold for two unstable TDDFT tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #4615
What's Changes?
After #4493, warnings are not allowed anymore in CI tests. However, #4493 only changed the threshold to make CI pass in 64 examples with warnings. This PR deletes the threshold files in 61 CPU_examples, and updates the corresponding
result.ref. Please notice that 3 GPU examples still have the threshold files.61 CPU examples:
3 GPU examples: