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

Tab MISC | 'Enable the use of Master-pitch' Diodes has slave behavior #2612

Closed
musikBear opened this issue Feb 26, 2016 · 13 comments
Closed
Labels
Milestone

Comments

@musikBear
Copy link

musikBear commented Feb 26, 2016

PR 17. feb. g23d284 win32

To reproduce:
Insert any number of presets or sample tracks in B&B-editor
Add some notes.

Open top-track
Go to MISC.
Toggle the diode for MASTER-PITCH
Open any track below the top track
The diode for MASTER-PITCH will have changed to the value of the top-track as a slave.

Does current windows Master (still) have this behavior?

Demofile: https://lmms.io/lsp/download_file.php?file=8152&name=masterpitchErrors.mmp

Edit: Similar issue closed as duplicate: #2613

@zonkmachine
Copy link
Member

There's absolutely something fishy here. Is this only in a Beat track?

@musikBear
Copy link
Author

@zonkmachine ..dont know why i did not test that too 🔎
No, song-editor has the same bug.

@zonkmachine zonkmachine self-assigned this Aug 2, 2016
@zonkmachine
Copy link
Member

zonkmachine commented Jan 18, 2017

I got a crash while probing this that could be related. 🍺

backtrace
bt full

@zonkmachine zonkmachine removed their assignment Apr 11, 2017
@zonkmachine
Copy link
Member

@curlymorphic This issue is related to #1593 and #1597 . Turning "Enables the use of Master Pitch" on/off affects other instrument tracks than the intended in an unpredictable way.

@zonkmachine
Copy link
Member

It looks like this could be related to song-global automation.

@musikBear
Copy link
Author

@zonkmachine #403 -i vote remove

@PhysSong
Copy link
Member

@musikBear Can you still reproduce it?

@zonkmachine
Copy link
Member

Well, I could a month ago.

@PhysSong
Copy link
Member

@zonkmachine What was the revision?

@zonkmachine
Copy link
Member

Replicated on stable-1.2 b3054fd with the test project in the first post.

@PhysSong
Copy link
Member

@zonkmachine Can you test with latest master?

@zonkmachine
Copy link
Member

Tested on latest master. Bug still around.

@zonkmachine
Copy link
Member

Closed. Fixed by @serdnab in #3753 and backported to stable-1.2 in 94646e2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants