Repository: mir-evaluation/mir_eval
Issue: io.load_jams_range: obscure exception
Root cause: Validation Gap

I'm getting the not-so-helpful exception:

> Error: could not open /home/bmcfee/git/olda/data/truth/JAMS/SALAMI_608.jams (JAMS module not installed?)

which obscures down-stream errors from jams.  I suggest redoing this with a decorator that first verifies that the jams import succeeded (raising the above exception if not), and then the loader proceeds with no try-catch so that errors are passed upstream.


I will do that, since we've changed jams a lot and this function is not
updated accordingly yet.

On Wed, Apr 16, 2014 at 2:41 PM, Brian McFee notifications@github.comwrote:

> I'm getting the not-so-helpful exception:
> 
> Error: could not open
> /home/bmcfee/git/olda/data/truth/JAMS/SALAMI_608.jams (JAMS module not
> installed?)
> 
> which obscures down-stream errors from jams. I suggest redoing this with a
> decorator that first verifies that the jams import succeeded (raising the
> above exception if not), and then the loader proceeds with no try-catch so
> that errors are passed upstream.
> 
> —
> Reply to this email directly or view it on GitHubhttps://github.com/craffel/mir_eval/issues/41
> .


It looks like jams support has vanished completely? O_o  

Is this a temporary thing, or the result of a discussion that I missed out on?


I don't think it's an intentionally permanent thing ... just a by-product
of concurrent development. Uri and I should try to sync up on that and
split the workload.

I say that recognizing full well that he's packing and leaving the country
Monday; meanwhile, I'm in new england for easter before leaving the country
Tuesday. Yeehaw.

On Sat, Apr 19, 2014 at 12:18 PM, Brian McFee notifications@github.comwrote:

> It looks like jams support has vanished completely? O_o
> 
> Is this a temporary thing, or the result of a discussion that I missed out
> on?
> 
> —
> Reply to this email directly or view it on GitHubhttps://github.com/craffel/mir_eval/issues/41#issuecomment-40873478
> .


I removed it temporarily, since it was not compatible anymore with the
latest version of jams. I will rewrite it in the near future so that it's
compatible with the (hopefully) stable jams version.

On Sat, Apr 19, 2014 at 12:40 PM, Eric J. Humphrey <notifications@github.com

> wrote:
> 
> I don't think it's an intentionally permanent thing ... just a by-product
> of concurrent development. Uri and I should try to sync up on that and
> split the workload.
> 
> I say that recognizing full well that he's packing and leaving the country
> Monday; meanwhile, I'm in new england for easter before leaving the
> country
> Tuesday. Yeehaw.
> 
> On Sat, Apr 19, 2014 at 12:18 PM, Brian McFee notifications@github.comwrote:
> 
> > It looks like jams support has vanished completely? O_o
> > 
> > Is this a temporary thing, or the result of a discussion that I missed
> > out
> > on?
> > 
> > —
> > Reply to this email directly or view it on GitHub<
> > https://github.com/craffel/mir_eval/issues/41#issuecomment-40873478>
> > .
> 
> —
> Reply to this email directly or view it on GitHubhttps://github.com/craffel/mir_eval/issues/41#issuecomment-40874062
> .


So, Brian (or anybody), were you using this function? I thought I was the
only one...

On Sat, Apr 19, 2014 at 1:54 PM, Oriol Nieto oriol.nieto@gmail.com wrote:

> I removed it temporarily, since it was not compatible anymore with the
> latest version of jams. I will rewrite it in the near future so that it's
> compatible with the (hopefully) stable jams version.
> 
> On Sat, Apr 19, 2014 at 12:40 PM, Eric J. Humphrey <
> notifications@github.com> wrote:
> 
> > I don't think it's an intentionally permanent thing ... just a by-product
> > of concurrent development. Uri and I should try to sync up on that and
> > split the workload.
> > 
> > I say that recognizing full well that he's packing and leaving the
> > country
> > Monday; meanwhile, I'm in new england for easter before leaving the
> > country
> > Tuesday. Yeehaw.
> > 
> > On Sat, Apr 19, 2014 at 12:18 PM, Brian McFee notifications@github.comwrote:
> > 
> > > It looks like jams support has vanished completely? O_o
> > > 
> > > Is this a temporary thing, or the result of a discussion that I missed
> > > out
> > > on?
> > > 
> > > —
> > > Reply to this email directly or view it on GitHub<
> > > https://github.com/craffel/mir_eval/issues/41#issuecomment-40873478>
> > > .
> > 
> > —
> > Reply to this email directly or view it on GitHubhttps://github.com/craffel/mir_eval/issues/41#issuecomment-40874062
> > .


`mir_eval` isn't using jams until there's a stable public release, for sanity.  This is no longer relevant.