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

Fixing incompatibilities with newer libraries versions #15

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

limacarvalho
Copy link

@limacarvalho limacarvalho commented Apr 17, 2024

Hi @frasermarlow, @yuhan and community!

First of all, great work on this super helpful solution built on top of dagster. I am sure this is a great help and a starting point for lots of people who are interested in analyzing the authenticity of a particular GitHub repository.

I decided to follow the README instructions to bring up the fake-star-detector, both by building from scratch and using the docker compose up, unfortunately, both failed for me.

Let me describe a bit more what happened:

What


When following the provided instructions the docker container finishes up building just fine and remains up without any issues, however when accessing the web UI via http://localhost:3000 it is possible to see something is wrong.

The following error is shown:
error_screen_1
error_screen_2

More info available in the error log/message:
error_message.log

Why


Without doing a proper root cause analysis, it may not be possible to identify the exact issue, however, I feel this can be related to newer python packages' versions being installed (compared to the ones installed before while testing) thus causing conflicts with other packages that are not frequently updated.

How


To work around the current issue and make sure this solution is still installable/usable at the current date, I hard-coded some packages' versions in the setup.py file. This seems to work around the issue, at least temporarily, and I was able to run the tool without any problems.
Of course, this would be a temporary solution, and implementing a proper fix using the latest dependencies versions would be the ideal/proper fix.

How to replicate the current issue


How to work around the issue

import glob

from setuptools import find_packages, setup

setup(
    name="fake_star_detector",
    packages=find_packages(exclude=["fake_star_detector_tests"]),
    # package data paths are relative to the package key
    package_data={"fake_star_detector": ["../" + path for path in glob.glob("dbt_project/**", recursive=True)]},
    install_requires=[
        "dagster==1.2.2",
        "dagster-cloud[serverless]==1.2.2",
        "dagster-dbt==0.18.2",
        "PyGithub",
        "Pydantic<2",
        "pandas==1.5.3",
        "matplotlib",
        "nbconvert",
        "nbformat",
        "ipykernel",
        "jupytext",
        "dbt-core==1.4",
        "dbt-bigquery==1.4",
        # packaging v22 has build compatibility issues with dbt as of 2022-12-07
        "packaging<22.0",
        "pendulum<3.0",
    ],
    extras_require={"dev": ["dagit==1.2.2", "pytest"]},
)

Thanks a lot! Cheers 🍻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant