Skip to content

BUGFIX: import error#248

Merged
Stamp9 merged 1 commit intochains-project:mainfrom
adrianthees:main
Feb 8, 2026
Merged

BUGFIX: import error#248
Stamp9 merged 1 commit intochains-project:mainfrom
adrianthees:main

Conversation

@adrianthees
Copy link
Contributor

@adrianthees adrianthees commented Feb 6, 2026

When trying to run the tool installed via pypi it throws an error almost immediately:

Traceback (most recent call last):
  File "/path/.venv/bin/dirty-waters", line 4, in <module>
    from tool.main import main
  File "/path/.venv/lib/python3.12/site-packages/tool/__init__.py", line 5, in <module>
    from .main import main
  File "/path/.venv/lib/python3.12/site-packages/tool/main.py", line 27, in <module>
    from tool_config import DEFAULT_ENABLED_CHECKS
ModuleNotFoundError: No module named 'tool_config

Steps to reproduce:

  1. pip install dirty-waters
  2. dirty-waters -p INRIA/spoon -pm maven

This patch fixes that error.
Tested by applying the patch in an editable pypi install locally.

@Stamp9
Copy link
Collaborator

Stamp9 commented Feb 6, 2026

Hi @adrianthees, nice catch! Thanks for the info and patch!
Could you please also try cd tool and run python main.py -p INRIA/spoon -pm maven to see if it works? Thanks!

@adrianthees
Copy link
Contributor Author

@Stamp9 I just tried it and it works as well.

@monperrus
Copy link
Collaborator

monperrus commented Feb 6, 2026 via email

@adrianthees
Copy link
Contributor Author

Like this #249 ?

The fix in this patch would remedy that problem.

@Stamp9 Stamp9 merged commit 66f450e into chains-project:main Feb 8, 2026
1 check passed
@Stamp9
Copy link
Collaborator

Stamp9 commented Feb 8, 2026

@adrianthees thanks for the patch!

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.

3 participants