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

make a new 0.4.1 bug-fix release #142

Closed
sezero opened this issue Jun 25, 2016 · 31 comments
Closed

make a new 0.4.1 bug-fix release #142

sezero opened this issue Jun 25, 2016 · 31 comments

Comments

@sezero
Copy link
Contributor

sezero commented Jun 25, 2016

We should make a 0.4.1 followup bugfix release:
One bug I fixed is commit 335dc04.
Bug #141 stands out, which is waiting for @chrisisonwildcode

@sezero sezero added this to the 0.4.1 - Crank it up to 11 milestone Jun 25, 2016
@psi29a
Copy link
Member

psi29a commented Jun 25, 2016

Sure, when @chrisisonwildcode fixes that we should do a 0.4.1

I'm awaiting debian/ubuntu to flood us with bugs as well.

@psi29a
Copy link
Member

psi29a commented Jul 18, 2016

After #141 is fixed, should we do a 0.4.1 release or wait a bit more?

@sezero
Copy link
Contributor Author

sezero commented Jul 19, 2016

Don't have strong opinion for it.

@chrisisonwildcode
Copy link
Contributor

Wait a bit more.

I've looked through the code and saw nothing obvious, as in the code paths for selecting a song within a multi-song file show no obvious errors. I'll be looking at the files that were sent later today and if they show nothing we've missed then I'll be gdbing the thing and tracing what's going on.

At the very least I'll have an answer today, at most a fix.

Sent from my iPhone

On 19 Jul 2016, at 4:22 PM, sezero [email protected] wrote:

Don't have strong opinion for it.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@psi29a
Copy link
Member

psi29a commented Jul 20, 2016

BTW: afritz1/OpenTESArena#38 (comment)

Most of this was answered by Chris/Kcat (OpenAL-Soft dev), please give it a once over to see if he was correct? It implies that XMI support is only partial because it doesn't support a translation of XMI's looping mechanism.

There are at least 2 files that seem to go on forever. One with 4~5m pause and another with an hour pause.

Apparently freepats has missing instruments which might cause problems for other users of WildMIDI.

@chrisisonwildcode
Copy link
Contributor

I'm going to look into this. I'm sure we can support xmi loops converted into custom midi instructions ... Aka meta events ... Or even custom controller everts.

If it can be pulled off the way I'm thinking then XMI's converted to MIDI via wildmidi would playback converted xmi loops as loops. Of course we'd need an option to ignore/enable loops for sanity.

Thoughts?

Sent from my iPhone

On 20 Jul 2016, at 4:37 PM, Bret Curtis [email protected] wrote:

BTW: afritz1/OpenTESArena#38 (comment)

Most of this was answered by Chris/Kcat (OpenAL-Soft dev), please give it a once over to see if he was correct? It implies that XMI support is only partial because it doesn't support a translation of XMI's looping mechanism.

There are at least 2 files that seem to go on forever. One with 4~5m pause and another with an hour pause.

Apparently freepats has missing instruments which might cause problems for other users of WildMIDI.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@chrisisonwildcode
Copy link
Contributor

I can add support for xmidi controllers 116 & 117 and have WildMIDI support those from its midi output. But we're going to need a "gate keeper" for the midi output due to those controllers being undefined in midi and therefore ignored by default to avoid undesirable behaviours. We could add a text meta event "XMIDI" to be that gate keeper at the start of the file to denote xmidi controllers 116 & 117 be enabled.

Again thoughts

@psi29a
Copy link
Member

psi29a commented Jul 25, 2016

Possibly related? #146

Looks like there are some unimplemented (looping based) MIDI messages/events.

@psi29a
Copy link
Member

psi29a commented Aug 8, 2016

From doing my work with OpenBSD, it told me the following:

Scanning dependencies of target wildmidi
[ 90%] Building C object src/CMakeFiles/wildmidi.dir/wm_tty.c.o
[ 95%] Building C object src/CMakeFiles/wildmidi.dir/wildmidi.c.o
[100%] Linking C executable ../wildmidi
CMakeFiles/wildmidi.dir/wildmidi.c.o: In function `open_oss_output':
/home/psi29a/WildMIDI/wildmidi/src/wildmidi.c:1052: warning: warning: strcpy() is almost always misused, please use strlcpy()
/home/psi29a/WildMIDI/build/libWildMidi.so.2.0: warning: warning: vsprintf() is often misused, please use vsnprintf()
/home/psi29a/WildMIDI/build/libWildMidi.so.2.0: warning: warning: strcat() is almost always misused, please use strlcat()
/home/psi29a/WildMIDI/build/libWildMidi.so.2.0: warning: warning: sprintf() is often misused, please use snprintf()
[100%] Built target wildmidi

Something to worry about? Good advise? Will we break something?

@sezero
Copy link
Contributor Author

sezero commented Aug 8, 2016 via email

@psi29a
Copy link
Member

psi29a commented Aug 16, 2016

Great news: https://buildd.debian.org/status/package.php?p=wildmidi

0.4.0 is finally uploaded to debian and it is built on all known platforms! (note, no tests were performed other than being built)

@sezero
Copy link
Contributor Author

sezero commented Aug 16, 2016

On 8/16/16, Bret Curtis [email protected] wrote:

Great news: https://buildd.debian.org/status/package.php?p=wildmidi

0.4.0 is finally uploaded to debian and it is built on all known platforms!
(note, no tests were performed other than being built)

Are packages depending on libwildmidi proted properly too?
The big endian target will be affected most importantly.

@psi29a
Copy link
Member

psi29a commented Aug 16, 2016

Slomo (Debian) knows of the API changes, so that has been taken care of for us.

To be fair, this is a downstream issue, but it is good to let them know ahead of time. ;)

@sezero
Copy link
Contributor Author

sezero commented Oct 2, 2016

Should we make a 0.4.1 release? None of the open bug reports are regressions, including #141, already.

@chrisisonwildcode
Copy link
Contributor

Sorry for delays, life and all that comes with it.

I'll be looking into features and bugs this week as I will have plenty of time on my hands. Hope to resolve at least #146 #149 #158

Chris

Sent from my iPhone

On 2 Oct. 2016, at 6:20 pm, sezero [email protected] wrote:

Should we make a 0.4.1 release? None of the open bug reports are regressions, including #141, already.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@psi29a
Copy link
Member

psi29a commented Oct 3, 2016

rejoice! :)

@psi29a
Copy link
Member

psi29a commented Oct 3, 2016

I'm looking to, with @sezero blessing, begin work on on unsf integration so that we have some support for sf2. This will likely be for 0.4.2 unless 0.4.1 is still not yet released by the time I'm ready to merge. ;)

@sezero
Copy link
Contributor Author

sezero commented Oct 3, 2016

On 10/3/16, Bret Curtis [email protected] wrote:

I'm looking to, with @sezero blessing, begin work on on unsf integration so
that we have some support for sf2. This will likely be for 0.4.2 unless
0.4.1 is still not yet released by the time I'm ready to merge. ;)

My blessing (oh boy..) is irrelevant, but there is an inherent
issue with unsf as it is, which you know too: it wants to extract
the whole thing into files, bu we want to get the only samples
we want in a memory area. unsf shall need modifying heavily
for it.

@psi29a
Copy link
Member

psi29a commented Oct 3, 2016

I was just trying to be funny. ;)

Three things:

  1. once the whole soundfont is extracted, we don't need to do it again. This is why I implemented the -o flag, so that we can dump it to /tmp (or other directory dependent on OS). This creates the cfg file that will be read into wildmidi.

  2. specifically asking for a patch/sample is on my todo list, but as an additional API call that gives a memory buffer location. So we would then explictely need to call open and close APIs. This is a lot more work, but worth it.

  3. unsf isn't a replacement for native support for sf2, just a stop-gap. it was always my intention, at least in the beginning, to unpack the whole soundfont.

@sezero
Copy link
Contributor Author

sezero commented Nov 24, 2016

Any chance that we'll see those bug fixes? Or should we make a bugfix release from what we already have?

@psi29a
Copy link
Member

psi29a commented Nov 24, 2016

@chrisisonwildcode I think he was referring to #146 #149 #158

@chrisisonwildcode
Copy link
Contributor

chrisisonwildcode commented Nov 24, 2016 via email

@sezero
Copy link
Contributor Author

sezero commented Nov 26, 2016

#149 and #158 are more important. #146 is a new feature which everybody can live (and have been living) without.

@chrisisonwildcode
Copy link
Contributor

I am dedicating this week to get stuff happening. I must point out that fixing devtest has been a priority due to it having timing issues making it harder to debug formats.

Plan is to:

  • Fix devtest (fix timing issue to make debugging formats easier)
  • Fix playback of xmi loops
  • Implement midi event callback (note this will trigger on the event but before it is processed by WildMIDI)
  • Fix playback of other formats

Any other request for the next release then please speak up.

@sezero
Copy link
Contributor Author

sezero commented Dec 15, 2016

Do we have an ETA yet?

@sezero
Copy link
Contributor Author

sezero commented Jan 8, 2017

Will this take forever?

@chrisisonwildcode
Copy link
Contributor

chrisisonwildcode commented Jan 8, 2017 via email

@psi29a
Copy link
Member

psi29a commented Jan 9, 2017

Gives me time to sneak in some stuff. ;)

I'll keep myself busy with unsf. I want to get to the point where we can ask unsf for specific patches instead of dumping all of them at once.

@sezero
Copy link
Contributor Author

sezero commented Mar 17, 2017

Nothing is happening. Should we release what we have as 0.4.1 ?

@chrisisonwildcode
Copy link
Contributor

chrisisonwildcode commented Mar 17, 2017 via email

@sezero
Copy link
Contributor Author

sezero commented Mar 17, 2017

Made the 0.4.1 release; also created a new "0.4.2" milestone. Closing this one.

@sezero sezero closed this as completed Mar 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants