-
Notifications
You must be signed in to change notification settings - Fork 20
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
TypeError: Cannot read property 'filter' of undefined #2
Comments
Hey, Could you send me the link to the midi file ? Midi parsing is still a WIP, Some files might not be understood by the parser. But if you give me the midi file to reproduce I might be able to help ! |
Hello again, This is my file: https://raw.githubusercontent.com/nicobrinkkemper/Gher/master/nginx/gher.mid I think it has something to do with the request headers not being "audio/mid", but I could be mistaken. Will try again tomorrow. Regards, Nico |
Cool ! will give it a shot asap and let you know :) |
Hey, It looks like the midi file is corrupt, I couldn't open it with 3 different midi players. How did you generate it ? |
I think react orchestra can only handle specific midi files. I've tried a couple but they all fail. I believe because they have multiple tracks inside them. Could you tell me how you generated your files on mac? Thanks. Works: Wont work: |
Hey! Sorry for the late reply :) I want to rewrite the midi parsing for the library as soon as I get the time, I don't think it's production ready yet. I'm thinking of using @grimmdude's Midi parser and writer in future versions. Meanwhile, if you want to use midi I think as long as you're using one track only the midi should behave as expected. Cheers ! |
When I build my own version of this repo, everything works fine. I got it working trough docker on a windows machine. However, when I change the url to the .mid file, it fails. I'm not doing anything different, I just changed the url and allowed cross origin on the host. Perhaps you can help me out with this? It could be that something went wrong with my build.
Which points to line:
The text was updated successfully, but these errors were encountered: