vyperlang/vyper

Improve error message for `math` namespace collision

Open

#4,579 opened on Apr 14, 2025

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Python (4,710 stars) (849 forks)batch import
Easy Pickingsbug - UXhelp wanted

Description

The following snippet will throw with

from snekmate.utils import math
import math

vyper.exceptions.NamespaceCollision: 'math' has already been declared as a ModuleInfo(module_t=venv/Lib/site-packages/snekmate/utils/math.vy, alias='math', ownership=<ModuleOwnership.NO_OWNERSHIP: 'no_ownership'>, ownership_decl=None)

  contract "test.vy:2", line 2:0
       1 from snekmate.utils import math
  ---> 2 import math
  -------^
       3

You must run it via vyper test.vy --enable-decimals.

Contributor guide

Tech stack
python
Domain
developer experiencetooling
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Vyper language basicsPython error handling
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
85
Research direction
Examine the error handling in vyper/exceptions.py for the NamespaceCollision exception. The current message includes internal details like ModuleOwnership. Improve the message to clearly indicate that the collision is due to reusing a name that is already imported, and suggest renaming the local import. Reference the GitHub issue for expected format.