Skip to content

Conversation

@Sleepyhead01
Copy link

  1. Added init.py files in the necessary directories (e.g. picotron/picotron/context_parallel) to treat them as Python subpackages
  2. Replaced absolute imports (e.g., picotron.utils import print) with relative imports (e.g., from .utils import print)
  3. Checked setup.py with find_packages() to confirm all subpackages (picotron, picotron/context_parallel, etc.) are included during installation

@3outeille
Copy link
Member

helloooo, i am going to review the PR now (sorry for the delay 🙏 )

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.

2 participants