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

Refactor tests #189

Draft
wants to merge 67 commits into
base: master
Choose a base branch
from
Draft

Refactor tests #189

wants to merge 67 commits into from

Conversation

jonasleitner
Copy link
Contributor

@jonasleitner jonasleitner commented Jan 7, 2025

  • Unified pyscf HF data generation
  • New adcman and adcc test data generators
  • Restructured test data:
    • MP data is separated from ADC data to avoid duplication.
    • Data for all reference cases (cvs, fc, gen, ...) is collected in a single file
  • Metadata for test cases (reference cases, geometry, charge, ...) is collected in TestCase "structs"
  • Adapted test data cache and tests
  • Tried to unify tests when possible. Since no test cases where dropped, this strongly increased the overall number of tests

Todo:

  • Fix CI
  • Contemplate on the number of states for which ADC test data is dumped. At the moment 2 states are consistently dumped.
  • Reduce the amount of test data and the number of tests:
    • Remove redundant systems and reference cases
    • Remove systems, methods and/or reference cases in specific tests
  • Decide where to store the new test data
  • Add some helper script to only remove e.g. the "cvs" test data in certain files and therefore allow their regeneration.

AdrianLDempwolff and others added 30 commits November 15, 2024 09:39
…yscf hfdata

dumps and used in generated Q-Chem input files.
* Fixed multiplicity dump which was incorrect in original dump_pyscf script.
@jonasleitner jonasleitner changed the title Test Refactoring Refactor tests Jan 7, 2025
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