qutip / qutip-cupy

CuPy linear-algebra backend for QuTiP
BSD 3-Clause "New" or "Revised" License
19 stars 10 forks source link

pow_dense #29

Closed MrRobot2211 closed 3 years ago

MrRobot2211 commented 3 years ago

This implements pow and creates a class that should be inside QuTiP proper to do the testing.

We can afterwards add the test class to qutip and inherit here in future PRs.

This in pursuit of having the most functionality available on the "first" release.

This is ready to merge after extra_functions

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 1093767054

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details


Totals Coverage Status
Change from base Build 1078504429: 0.4%
Covered Lines: 192
Relevant Lines: 229

💛 - Coveralls
AGaliciaMartinez commented 3 years ago

The class PowTest is now merged in dev.major. :)

MrRobot2211 commented 3 years ago

@hodgestar I think this is ready then, I'll inherit from the newly minted QuTiP tests later on.

hodgestar commented 3 years ago

@MrRobot2211 Approved! Happy for you to merge once the test run passes.

hodgestar commented 3 years ago

@MrRobot2211 Could you fix up the silly pep 8 errors resulting from my merging with GitHub's merge interface? Apologies for leaving you with the mess but I am stepping away from my computer now and don't have the branch checked out to do the clean up quickly.