Johannes Terblanche

Results 293 comments of Johannes Terblanche

Yes I agree. The bad case is if the names are variable. The functions will compile a new method for each name.

It will be a tradeoff between compile time and runtime. Every new NamedTuple will require a new method to be compiled.

This tip is relevant here: https://docs.julialang.org/en/v1/manual/performance-tips/#The-dangers-of-abusing-multiple-dispatch-(aka,-more-on-types-with-values-as-parameters)

I checked v0.28 and it was failing intermittently there as well, so think we can safely eliminate the recent `ArrayPartition` and Manifolds v0.8 upgrade.

From #1580, it looks like this will only be done after #1010

Ok, just saw it is caused by `Prior(I)` I'm changing the issue to the question: What is the correct way of defining the mechanics in a mixture? For now the...

Also, should we perhaps use the nullhypo (spreadNH) case in initialization: ```julia certainidx, allelements, activehypo, mhidx = IIF.assembleHypothesesElements!(mh, N, 1, 3, Bool[0, 0, 1]) #currently: #samples for 0: 0.0%

I didn’t miss it in the discussion. I understand that part. The issue is even if you give the probability of 0.999-0.001 it wil go to 33% nullhypo and 66%...

This subfg from a range only example. ![image](https://user-images.githubusercontent.com/6612981/98226718-11727500-1f5f-11eb-8d2d-0fcaa8d5785c.png) Results in these 2 factors after deconvolution. ![image](https://user-images.githubusercontent.com/6612981/98226761-23541800-1f5f-11eb-991a-25fcfc3024af.png) x0 and x1 are connected by 2 range only factors and the differential factor...

For now `useMsgLikelihoods=false` should be used for better performance `useMsgLikelihoods=true` ![image](https://user-images.githubusercontent.com/6612981/98228083-cb1e1580-1f60-11eb-9093-7b14219d77d0.png) `useMsgLikelihoods=false` ![image](https://user-images.githubusercontent.com/6612981/98228042-be99bd00-1f60-11eb-8669-93072d87f937.png)