Qiskit-Extensions / circuit-knitting-toolbox

Tools for knitting quantum circuits with Qiskit
https://qiskit-extensions.github.io/circuit-knitting-toolbox/
Apache License 2.0
69 stars 23 forks source link

Add note in INSTALL regarding Docker installation #611

Closed caleb-johnson closed 1 month ago

caleb-johnson commented 1 month ago

This note is at least useful for developers who will commonly have large .tox directories in their local repository, along with other caches which speed up local CI.

This could also be useful for someone running a local install and using the repository as a dumping grounds for results.

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 9308881765

Details


Totals Coverage Status
Change from base Build 9306464756: 0.0%
Covered Lines: 2401
Relevant Lines: 2401

💛 - Coveralls
garrison commented 1 month ago

This note is at least useful for developers who will commonly have large .tox directories in their local repository, along with other caches which speed up local CI.

Are you seeing this with .tox specifically? .tox/ is listed in .dockerignore, but maybe we need to list **/.tox instead.

garrison commented 1 month ago

(I am not opposed to adding a note, but I do want to fix the problem to the extent possible that we can.)

caleb-johnson commented 1 month ago

No, you are right. This came up developing in another repo. We are covered here. Closing