Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Hieu Hoang
you're quite right, i've added a check https://github.com/moses-smt/mosesdecoder/commit/982d52e5b657f4c1fa7369e577cfd75a8af16543 However, that the the problem I'm having on OSX. It opens but it crashes on loading. I suspect one of the datatypes has slightly different size on clang/OSX compared

Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Jeroen Vermeulen
On 10/13/2015 04:59 PM, Hieu Hoang wrote: > you're quite right, i've added a check > > https://github.com/moses-smt/mosesdecoder/commit/982d52e5b657f4c1fa7369e577cfd75a8af16543 > However, that the the problem I'm having on OSX. It opens but it crashes > on loading. > > I suspect one of the

Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Marcin Junczys-Dowmunt
Hi, yes, definitely wrong turn, all code should be in CompactPT. I am not sure this is actually a code bug, is it working with g++ on macOS? W dniu 2015-10-13 12:50, Jeroen Vermeulen napisał(a): > On 10/13/2015 04:59 PM, Hieu Hoang wrote: > >> you're quite right, i've added a check >>

Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Jeroen Vermeulen
On 10/13/2015 06:05 PM, Marcin Junczys-Dowmunt wrote: > yes, definitely wrong turn, all code should be in CompactPT. Ah, I'd missed that this is a template. So what's being loaded there is an array of float. Not that much that can go wrong there, within one CPU architecture... My next stab in

Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Hieu Hoang
i'll take a closer look when I have time. I think it's been happening for a while but I've ignored it. btw, i've pulled unblockpt into master Hieu Hoang http://www.hoang.co.uk/hieu On 13 October 2015 at 12:05, Marcin Junczys-Dowmunt wrote: > Hi, > > yes, definitely wrong

Re: [Moses-support] Compact lex reordering table on OSX/clang

2015-10-13 Thread Jeroen Vermeulen
On 10/12/2015 11:15 PM, Hieu Hoang wrote: > I'm not sure if anyone else encounters it but the compact lexical > reordering table crashes for me on OSX/clang during loading. > > The stack trace i have for this is > LexicalReorderingTableCompact::LexicalReorderingTableCompact >

[Moses-support] Compact lex reordering table on OSX/clang

2015-10-12 Thread Hieu Hoang
I'm not sure if anyone else encounters it but the compact lexical reordering table crashes for me on OSX/clang during loading. The stack trace i have for this is LexicalReorderingTableCompact::LexicalReorderingTableCompact LexicalReorderingTableCompact::Load line 180