-
Notifications
You must be signed in to change notification settings - Fork 173
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
Duplicating pattern yields "unknown" as name in Song Editor #1827
Comments
Hmm.. That's strange. I can't reproduce it. Those anything show up in the log when opening Hydrogen from the command line using Is by any chance your hard disk completely full or your |
My HDD is not full and the /tmp folder is writeable. :) |
But i found some info in the logs:
Edit: |
Nice. Could you upload one of the bricked patterns from |
I tried on both Devuan Chimaera and a clean Fedora 38 and couldn't reproduce it yet. |
Here you go :) Funny thing i noticed: The name shows up correctly in the file. |
This also happens to me on AV Linux 21.3 (based on Debian 11 Bullseye). It only happens sometimes though, as you've noted no guaranteed recipe.. |
That's really weird. This pattern looks just fine, validates the corresponding XSD file, and I'm even able to open it. Could you append
Okay. I'll try again on one of those systems (just duplicated the pattern you posted 40 times without an issue). |
I'm afraid I need some more details in order to reproduce this issue. I run several live Linux OSs and installed AVLinux, compiled the 1.2.1 version of Hydrogen and duplicated various patterns. But I didn't encounter the problem at all. Do you experience it on fresh installs using new songs too? Does it maybe occur after using Hydrogen quite a lot or just for large song in terms of patterns? What's the frequency of failing duplication? |
Hi, This has happened to me working on a new song a couple of weeks ago, I don't have time for beat making this week but next time I do I will screencapture the session and see if this issue pops up while I'm working and perhaps that will give a clue if it's a sequence of operations thing.. |
Alright. I did just that. Attached .h2pattern to the filename. Opened it via Open Pattern. The result is the same - the pattern name is "unknown". |
I tried with a fresh song, duplicated the pattern and did not experience this problem. |
Thanks a lot! This one did it. I already located the bug: the pattern has "too much" notes. Whenever there are more than 100 notes in a pattern Qt fails to validate against our XSD schema. But just Qt as the XSD defines a max occurrence of I'll fix it |
There was a limitation in the XSD file that set the maximum number of allow notes within a pattern to `1000`. This number can easily be reached by setting resolution to `off` and filling all notes for a couple of instruments. Curiously, Qt treated this number wrong and also deemed paterns containing more than `100` notes as invalid. Once considered invalid, Hydrogen thinks this pattern is of a legacy format. But since it is not, loading partially fails. I removed the constraint for a maximum number of notes. fixes hydrogen-music#1827
There was a limitation in the XSD file that set the maximum number of allow notes within a pattern to `1000`. This number can easily be reached by setting resolution to `off` and filling all notes for a couple of instruments. Curiously, Qt treated this number wrong and also deemed paterns containing more than `100` notes as invalid. Once considered invalid, Hydrogen thinks this pattern is of a legacy format. But since it is not, loading partially fails. I removed the constraint for a maximum number of notes. fixes hydrogen-music#1827
@PatDyn could you compile and install the changes in #1842 and check whether this fixes things at your end? Alternatively, you could just change line 74 in <xsd:element ref="h2:note" minOccurs="0" maxOccurs="1000"/> into <xsd:element ref="h2:note" minOccurs="0" maxOccurs="unbounded"/> |
Ahhh that makes sense! I think in hindsight it was in pretty busy patterns that this issue popped up. |
Closed with #1842 |
There was a limitation in the XSD file that set the maximum number of allow notes within a pattern to `1000`. This number can easily be reached by setting resolution to `off` and filling all notes for a couple of instruments. Curiously, Qt treated this number wrong and also deemed paterns containing more than `100` notes as invalid. Once considered invalid, Hydrogen thinks this pattern is of a legacy format. But since it is not, loading partially fails. I removed the constraint for a maximum number of notes. fixes #1827
Hydrogen version * : 1.2.1-1, installed via pamac
Operating system + version : Manjaro and Linux Mint 21 Vanessa
Audio driver + version : Jack, 1.9.22
How to reproduce:
Edit: Confirmed also on Linux Mint 21
The text was updated successfully, but these errors were encountered: