uw-ipd / tmol

TMol
Apache License 2.0
31 stars 3 forks source link

Fix pip installs #259

Closed fdimaio closed 1 year ago

fdimaio commented 1 year ago

Pip installs now copy all necessary files.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (98d1046) 95.13% compared to head (313d200) 95.13%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #259 +/- ## ======================================= Coverage 95.13% 95.13% ======================================= Files 327 327 Lines 20290 20290 ======================================= Hits 19303 19303 Misses 987 987 ``` | Flag | Coverage Δ | | |---|---|---| | _shrug_Testing_CPU | `89.68% <ø> (ø)` | | | _shrug_Testing_CPU_w_o_jit | `91.62% <ø> (ø)` | | | _shrug_Testing_CUDA | `92.31% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uw-ipd#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

kleinhenz commented 1 year ago

I think using find_packages is slightly more idiomatic than putting everything in the manifest.

from setuptools import setup, find_packages

setup(
    name='myPackage',
    packages=find_packages(),
)
fdimaio commented 1 year ago

I think using find_packages is slightly more idiomatic than putting everything in the manifest.

Ok, will update this.

fdimaio commented 1 year ago

OK, fixed.