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

Music() function with tempo parameters #1193

Closed
Evendream opened this issue Sep 29, 2020 · 6 comments
Closed

Music() function with tempo parameters #1193

Evendream opened this issue Sep 29, 2020 · 6 comments
Assignees
Labels
API Issues related to TIC80 API

Comments

@Evendream
Copy link

This function has the form [music(id)], but you can add a parameter that sets the initial tempo of the composition, reducing the function to the form [music(id, tempo)].

@nesbox nesbox added this to To do in dev version 0.90 via automation Sep 29, 2020
@nesbox nesbox added the API Issues related to TIC80 API label Sep 29, 2020
@Evendream
Copy link
Author

Also, you can add a channel mute function. Unless, of course, it overloads the system.

@nesbox nesbox moved this from To do to In progress in dev version 0.90 Apr 15, 2021
@nesbox nesbox self-assigned this Apr 15, 2021
nesbox added a commit that referenced this issue Apr 15, 2021
@nesbox
Copy link
Owner

nesbox commented Apr 15, 2021

Added music tempo/speed parameters to the RAM here 8c1e0ae (the wiki will be updated after 0.90 release)
so, music tempo, speed can be poked by 0x14E04/0x14E05 addr now

+-----------------------------------+
|           96KB RAM LAYOUT         |
+-------+-------------------+-------+
| ADDR  | INFO              | BYTES |
+-------+-------------------+-------+
           ...
| 14E04 | MUSIC PARAMETERS  | 2     |
           ...
+-------+-------------------+-------+

Also, you can add a channel mute function. Unless, of course, it overloads the system.

Try to poke a sound register volume to mute the channel https://github.com/nesbox/TIC-80/wiki/RAM#sound-registers

Thank you.

@nesbox nesbox closed this as completed Apr 15, 2021
dev version 0.90 automation moved this from In progress to Done Apr 15, 2021
@joshgoebel
Copy link
Collaborator

(the wiki will be updated after 0.90 release)

I've been trying to keep it updated as we go and just adding notes on versioning when appropriate. Wrong idea?

@nesbox
Copy link
Owner

nesbox commented Apr 15, 2021

Not at all, good idea :)
This is just a reminder for me to check the wiki before the release...
Thank you.

@nesbox
Copy link
Owner

nesbox commented May 5, 2021

I reverted tempo/speed parameters in the RAM because it breaks compatibility with some carts, this one for example https://tic80.com/play?cart=479
160b29a
And I'm going to add speed/tempo to the music() API, reopening...

@nesbox nesbox reopened this May 5, 2021
dev version 0.90 automation moved this from Done to To do May 5, 2021
@nesbox nesbox moved this from To do to In progress in dev version 0.90 May 5, 2021
@nesbox
Copy link
Owner

nesbox commented May 5, 2021

Added tempo/speed parameters to the music(track=-1 frame=-1 row=-1 loop=true sustain=false tempo=-1 speed=-1) api 39db700

@nesbox nesbox closed this as completed May 5, 2021
dev version 0.90 automation moved this from In progress to Done May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Issues related to TIC80 API
Projects
No open projects
Development

No branches or pull requests

3 participants