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

Remove psiresp.testing #39

Open
7 tasks
lilyminium opened this issue Nov 14, 2021 · 0 comments
Open
7 tasks

Remove psiresp.testing #39

lilyminium opened this issue Nov 14, 2021 · 0 comments
Labels
documentation Improvements or additions to documentation technical

Comments

@lilyminium
Copy link
Owner

lilyminium commented Nov 14, 2021

psiresp.testing is a bad hack for patching QCFractal behaviour, making psiresp not quite independent.

Once the following issues are resolved, I need to check why I had to replace the storage in FractalSnowflake

I had to replace this because FractalSnowflake calls TemporaryPostgres calls PostgresHarness. So we are all clear to remove once issues are gone.

Steps to remove psiresp.testing:

  • delete the file
  • update the tests
  • update the examples and documentation (!!)
  • update environment pins

Issues:

@lilyminium lilyminium changed the title Remove psiresp.testing and investigate sockets Remove psiresp.testing Nov 14, 2021
This was referenced Nov 15, 2021
@lilyminium lilyminium added documentation Improvements or additions to documentation technical labels Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation technical
Projects
None yet
Development

No branches or pull requests

1 participant