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

Magic AI Storybook python packaging #2594

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

Conversation

anotherthomas
Copy link

@anotherthomas anotherthomas commented Aug 18, 2023

The installation of the magic ai storybook software is quite complex and manual.
Packaging the software allows to
a) automatically download dependencies and
b) create entry points to start the software like a regular program.

Installing pyaudio via pip requires portaudio19-dev, otherwise you can install the debian package. See #2581

@makermelissa makermelissa changed the title 2581 python packaging Magic AI Storybook python packaging Aug 29, 2023
@makermelissa
Copy link
Collaborator

Shortly after you made this, bookworm was released and the code no longer worked. I have just updated to run on bookworm, so I can now test this out.

@makermelissa makermelissa changed the base branch from main to TheKitty-patch-6 March 29, 2024 21:14
@makermelissa makermelissa changed the base branch from TheKitty-patch-6 to main March 29, 2024 21:14
@makermelissa
Copy link
Collaborator

Ok, tested this today. It seems to install correctly and with a venv active, it installs to the venv bin folder. However, when running it, nothing displays on the screen. I don't know if you are wanting to update/test this yourself. If not, we can leave the guide as is because installing the packages manually isn't much more than copying/pasting the commands at the moment.

Copy link
Collaborator

@makermelissa makermelissa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please test/update for bookworm and using a venv. Thanks.

@anotherthomas
Copy link
Author

I don't have the screen here at the moment, will do so as soon as possible

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