Skip to content

Added Loopy Pro template for Midi Follow#976

Merged
sapphire-arches merged 3 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/add_loopy_pro_template
Jan 16, 2024
Merged

Added Loopy Pro template for Midi Follow#976
sapphire-arches merged 3 commits into
SynthstromAudible:communityfrom
soymonitus:monitus/add_loopy_pro_template

Conversation

@soymonitus

Copy link
Copy Markdown
Collaborator

It is setup to send and receive on channel 15, when Deluge is connected via USB, so it is detected as “Deluge Port 1”

It is setup to send and receive on channel 15, when Deluge is connected via USB, so it is detected as “Deluge Port 1”
@sapphire-arches

Copy link
Copy Markdown
Collaborator

It might be good to reduce the size of the image a bit, as it's currently larger on disk than the actual template zip (I assume the .zip format is what's required to actually feed it to loopy pro?) Otherwise looks fine to me

@seangoodvibes

Copy link
Copy Markdown
Collaborator

It might be good to reduce the size of the image a bit, as it's currently larger on disk than the actual template zip (I assume the .zip format is what's required to actually feed it to loopy pro?) Otherwise looks fine to me

It doesn't need to be a zip no. The actual loopy pro template is compressed inside the zip. I think it should be uploaded without the zip.

@soymonitus

Copy link
Copy Markdown
Collaborator Author

If you unzip it, it is like a container folder with several other files. And the size is quite bigger. Are you ok with that? The image snapshot is for reference purposes, to include it in the docs or webpages. If it is not the place to put images let me know and I can delete it. Or do you want me to make the size smaller?

@sapphire-arches

Copy link
Copy Markdown
Collaborator

Given the size and how people will probably interact with this template (downloading just the file from GH), it seems like we should keep the project compressed. I was hoping the zip had only 1 file or stuff we could rip out but that doesn't seem to be the case.

@sapphire-arches sapphire-arches left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, @seangoodvibes if it also looks OK to you we can merge.

@sapphire-arches sapphire-arches added this pull request to the merge queue Jan 16, 2024
Merged via the queue into SynthstromAudible:community with commit 316279c Jan 16, 2024
@soymonitus soymonitus deleted the monitus/add_loopy_pro_template branch January 16, 2024 21:15
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