uber / causalml

Uplift modeling and causal inference with machine learning algorithms

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

installs via environment files failing

ras44 opened this issue · comments

Describe the bug

This is a follow up on the work being done in #778 to list and address a variety of ongoing issues related to the pre-defined environment files in envs/:

question re defining causalml version

  • should env files contain a relative reference to the current source? ... as shown in envs/environment-py39.yml:
    image

  • or a fixed version, as shown in envs/environment-py38.yml:
    image

  • I would vote in favor of a fixed version

    • follow up: should we update to the latest release 0.15.1 ?

pytest failures

See failure details at: https://github.com/ras44/causalml/actions/runs/9391720812/job/25864510171

  • pytest -vs tests/ --cov causalml/ fails for:
    • environment-py37.yml
      image
    • environment-py38.yml
      image
    • environment-py39.yml
      image

The failures in our Github workflows largely appear related to:
image

Building on other platforms like Ubuntu on Windows, we see other failures, likely related to the build environment and cython.

environment build failures re tensorflow dependencies

  • dependency issues in the tensorflow environments that look like:

image

To Reproduce
See example results of current build at:
https://github.com/ras44/causalml/actions/runs/9391720812/job/25864510171

Expected behavior
Successful builds for all included environment files.

Screenshots
N/A

Environment (please complete the following information):
See build logs

Additional context
N/A

closing, as we will likely remove support for environment files via #780