Thx. I’ll look into it
I believe I have fixed the bug, it would be great if someone could confirm this is true.
Technically, I just made the note-sorting take note-on/off into consideration.
Oh, and nice code to step in and fix. Props to Conner 
Wow, thanks! One comment, current code has
DBUG_MODE = true
COROUTINE_MODE = false
It should be reversed for release.
It should be reversed for release.
OK, a few downloads later…,I’ll release it then 
PS: how about your git repository?
https://github.com/connerbw/xrnx/tree/master/com.renoise.MidiConvert.xrnx
PS: how about your git repository?
https://github.com/connerbw/xrnx/tree/master/com.renoise.MidiConvert.xrnx
I suggest:
Fork & pull
The_fork & pull model_lets anyone fork an existing repository and push changes to their personal fork without requiring access be granted to the source repository. The changes must then be pulled into the source repository by the project maintainer. This model reduces the amount of friction for new contributors and is popular with open source projects because it allows people to work independently without upfront coordination.
More info: https://help.github.com/articles/using-pull-requests/
Hey guys
Still getting that same error message as above:
‘C:\Users\Mikael\AppData\Roaming\Renoise\V3.0.1\Scripts\Tools\com.renoise.MidiConvert.xrnx\main.lua’ failed in one of its notifiers.
The notifier will be disabled to prevent further errors.
Please contact the author (Dac Chartrand, Marvin Tjon) for assistance…
Unknown Lua interpreter error occurred!
Never had any problems until switching to a 64bit system.
Any sollutions on the horizon?!
I think this is quite an important tool, and would be happy to fix any bugs… but:
Still getting that same error message as above:
I’m a bit confused here - which error message areyou pointing to?
The bug report at the top of the page was fixed a couple of years ago
If would be great if you could somehow reproduce the bug, describe exactly what steps you take to trigger it.
Although, I’m pretty sure running a 64bit system shouldn’t affect things (but who knows ^_^)
Hey
Well
I have two computers, both with latest versions of Renoise and this very clever little tool installed. Only difference is that when I “export song to midi”, I get
the error message I posted on my 64bit system. I’ve tried several tracks, and there is no problem when converting on the 32bit system.
I get this message:
‘C:\Users\Mikael\AppData\Roaming\Renoise\V3.0.1\Scripts\Tools\com.renoise.MidiConvert.xrnx\main.lua’ failed in one of its notifiers.
The notifier will be disabled to prevent further errors.
Please contact the author (Dac Chartrand, Marvin Tjon) for assistance…
Unknown Lua interpreter error occurred!
and that’s it.
Just out of curiosity, has anyone tried the simple xrns file that satobox posted? I’ll attach the file I’m talking about here. With the danoise amendment to the function ‘export_compare’ I get an error message along the lines of: ./export.lua:66 attempt to index local ‘a’ (a nil value) blah blah… This is on a Linux 64-bit system 
Edit: Looking a tiny bit closer, I’m not sure that this amendment to the script:
-- Used to sort a table in export_midi()
function export_compare(a, b)
if (a[1] == b[1]) then -- this line
return (string.sub(a[2],0,2)~="On") and true or false -- and this
else
return a[1] < b[1]
end
end
Is going to quite do what you want with the lua sort function danoise
I’m not sure that this amendment to the script:Is going to quite do what you want with the lua sort function danoise
The sort algorithm is quite simple: boolean value determines if item should bubble up or not.
I just modified it so that in the case that two events arrive on the same line, a note-off event will take precedence.
But then it becomes really strange, because now it seems that in this particular case the table will now contain a “nil” entry.
This simply shouldn’t happen. The source table never contains a nil value, and the sort method always return a boolean.
Very strange, but at least it gives us a particular spot to start investigating…
Oh, and the behavior seems consistent across 64 and 32bit systems according to my quick test (both Renoise 3.0.1 runningon Win7).
@Mikael: are you sure you are using the same song data, and the same version of the tool?
But then it becomes really strange, because now it seems that in this particular case the table will now contain a “nil” entry.
This simply shouldn’t happen. The source table never contains a nil value, and the sort method always return a boolean.
Very strange, but at least it gives us a particular spot to start investigating…
Yup, I agree sir that a ‘nil’ does not appear in the ‘sort_me’ table. However (I’m just speculating here BTW)…I think you may find that you can’t put your ‘equal if’ test in the comapre sort function. Lua doesn’t like it. It seems to fall off the end of the table when it goes through its sort algorithim, hence the funny nil result. Now I’m not a Lua expert or anything but I think that Lua doesn’t like the situation when ‘a’ and ‘b’ are the same. One way of looking at it is that becuase you are returning a boolean (either true or false) in that situation it can confuse Lua. By the looks of it it doesn’t know what to do. You might be saying ‘IF ‘a’ and ‘b’ are the same THEN ‘a’ comes before ‘b’ AND ‘b’ comes before ‘a’’!
Silly I know, but we didn’t implement the sort function in Lua
One way of looking at it is that becuase you are returning a boolean (either true or false) in that situation it can confuse Lua.
A boolean is really the only allowed return value for the sort method, in case of the “plain” compare it’s evaluating the expressiona[1] < b[1].When they are the same, the result would simply be false.
But really, you should be able to implement as complicated a comparison as you would want - as long as there is a boolean result, even a “wrong” sorting algorithm should not result in nil entries, just that the entries would end up having the wrong order.
This is what confuses me. Hmm, maybe I should pop a question on the lua mailing list and see what the gurus have to say.
Technically if I was looking at it from the Lua point of view, if both ‘a’ and ‘b’ are the same then ignore (irrespective of what boolean the user returns from the compare function.) No swap would need to occur.
Again I agree that you should be able to implement as complex a sort method as you like. But alas Lua doesn’t play ball. Trouble is I think it has made things much more awkward for you 
Oh, and the behavior seems consistent across 64 and 32bit systems according to my quick test (both Renoise 3.0.1 runningon Win7).
@Mikael: are you sure you are using the same song data, and the same version of the tool?
Actually… I was mistaken. I was using ver 0.9 on my 32bit computer. I uninstalled the latest version (ver 0.92 was it?) from my 64bit system and installed ver0.9 on it… and voila.
Works as a charm
Hi…
Just thought I’d let you know I’m getting the same error as above too…
"C:\Users\Mikael\AppData\Roaming\Renoise\V3.0.1\Scripts\Tools\com.renoise.MidiConvert.xrnx\main.lua’ failed in one of its notifiers.
The notifier will be disabled to prevent further errors.
Please contact the author (Dac Chartrand, Marvin Tjon) for assistance…
Unknown Lua interpreter error occurred!"
I’m running Win 7 Pro 64 Bit with Renoise 64Bit - V3.0.1 (Built Oct 13 2014) and Midi Convert v0.91.
I’ve gone back to v0.9 and that’s working fine. 
Hmm, first of all this page -> http://www.renoise.com/tools/midi-convert shows midi convert v0.91. However I click on that to download and it gives me v0.92. v0.92 of this tool won’t work due to the way Lua works its custom sort function. (Just as a side note I don’t think danoises quick fix would’ve completely fixed the problem anyway even if Lua would’ve accepted his sort function…but don’t tell danoise that 
)
Let me just clarify for you what the problem (that you could run into) is here with Midi Convert…(I’ll assume you’re using another version rather than 0.92.)
If you have this simple situation…(my best module I’ve ever written :D)
Attachment 5435 not found.
If you ask Midi Convert to export that and you then reload the resulting midi file back into Renoise you’ll probably get something along the lines of:
Attachment 5436 not found.
That tripped Midi Convert up. Cut notes because…well unfortunately just dumping all the columns/tracks notes and note offs into a table and then sorting that table isn’t quite a good enough solution. It will kinda work about 80-90% of the time, but not quite all of the time
New version, 0.93 should fix the issue above AND the one introduced in 0.92:
http://www.renoise.com/tools/midi-convert
For the technically curious, it turned out that the comparison case was ambiguous after all.
But to my surprise, lua supports “>” and “<” operators on strings, so the new version is using just that.
Hmm, first of all this page → http://www.renoise.com/tools/midi-convert shows midi convert v0.91. However I click on that to download and it gives me v0.92.
Oops, I forgot to update the title of the download. Fixed with 0.93
For the technically curious, it turned out that the comparison case was ambiguous after all.
But to my surprise, lua supports “>” and “<” operators on strings, so the new version is using just that.
Oh of course :rolleyes:, I knew that Lua could do a lexi comparison on strings, but the mistake I made was how I was returning the boolean comparison from the custom sort function. Lua isn’t really all that kind on its sort function danoise 
Anyway, good show sir
Oops, I forgot to update the title of the download. Fixed with 0.93
Committed to GitHub. Thanks!
https://github.com/connerbw/xrnx/commit/d2af3b58d5fa0ed9c97db43c3fc19522047a0b79
What this tool needs is the option to choose whether or not you want to keep the MIDI routings that are assigned on to sampler channels, as for the latest version of both Renoise (3.01) and Midi Convert Tool (v0.93), I didn’t find the MIDI export at all useful. All the stuff I had in my project are reduced to multiple tracks of piano notation - just like a bunch of users have reported previously.
Upon exporting to MIDI, I’d seriously like to keep (if I want to):
- The channel properties of each sample/instrument (transpose, volume, midi channel/patch/bank)
- All channel/bank/program information etc
- Preferably you could choose your export method before exporting (what parameters are included and what are not)
Otherwise, upon exporting to MIDI, I’d have to go thru all that stuff again if I decide to migrate the project to another DAW. And that’s a pain in the a–. Since I’m new to Renoise and I’ve been using it as the “sequencer brains” for hardware synthesizers that I have in my studio rig, I’d really like to see the MIDI export/import options implemented more thoroughly in the future.
As for now, my best bet would be to record the MIDI coming from Renoise through my MOTU Midi Timepiece (a rackmount midi-matrix) to another soundcard that’d record all the MIDI data that’s being sent realtime.
IMHO, one of the best programs ever made available for tracker modules to MIDI conversion was the Impulse Tracker-interfaced zTracker, which is sadly no longer being developed. However, the source code is available and whoever is developing the MIDI export tool / such features would probably find the program interesting. https://github.com/cmicali/ztracker
Thanks.