Skip to content

Include entire sam3 package instead of just sam3 and sam3.model#327

Closed
lematt1991 wants to merge 3 commits intomainfrom
mattle/fix-package
Closed

Include entire sam3 package instead of just sam3 and sam3.model#327
lematt1991 wants to merge 3 commits intomainfrom
mattle/fix-package

Conversation

@lematt1991
Copy link
Contributor

@lematt1991 lematt1991 commented Dec 10, 2025

there are several imports within the sam3.model package that reference other packages within sam3 other than sam3 and sam3.model (for example here). This fixes the package structure so that you can pip install the package and import sam3

there are several imports within the `sam3.model` package
that reference other packages within `sam3`.  This fixes
the package structure so that you can `pip install` the package
and `import sam3`
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Dec 10, 2025
This way we can include it as package data
@meta-codesync
Copy link

meta-codesync bot commented Dec 11, 2025

@lematt1991 has imported this pull request. If you are a Meta employee, you can view this in D88950127.

@meta-codesync
Copy link

meta-codesync bot commented Dec 11, 2025

@lematt1991 merged this pull request in b26a5f3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants