Alan Race

Results 16 comments of Alan Race

It sounds like this issue is related to one I am currently having, so thought I would just add this here. Zooming in/out of a plot (holding `Ctrl` and using...

@Mec-iS Thanks for supplying the test - I am guessing there was a problem due to row-major vs column-major storage of `DenseMatrix`? Swapping the number of rows and columns in...

Maybe I am misunderstanding, but `predict_probs` does return a `DenseMatrix`. It looks like `DenseMatrix::from_vec` (which is called from `DenseMatrix::from_array` as part of your test) assumes that the given vector is...

Hopefully this should now resolve the previous ambiguity. I decided that the presence of app segments should set `color_transform` rather than overwriting `fallback_color_transform`, but that's a simple change if you...

As this pull request is dealing with colour transforms, I had a look at solving https://github.com/image-rs/jpeg-decoder/issues/249 too. I have rewritten the logic for selecting the colour transform based on the...

@Shnatsel This should now be resolved in the master branch - can you check that this hasn't had any unintended consequences for your large set of JPEGs?

I have also encountered this issue. It seems that the problem occurs when `get_output_contour_id()` returns `-1` as this (`i32`) is then converted to a `usize` and causes this problem. https://github.com/21re/rust-geo-booleanop/blob/7198a0321c3274de6f21aa5ee7456a4763151338/lib/src/boolean/connect_edges.rs#L176-L180...

@micahsnyder yes, and the same with https://github.com/image-rs/jpeg-decoder/pull/255. It seems that https://github.com/image-rs/jpeg-decoder/pull/254 is now merged, so just waiting on https://github.com/image-rs/jpeg-decoder/pull/255 now.

Yeah I realised that after I posted, I was just trying to put something simple together. In my actual code the function signature is closer to this: ```rust fn add(...

And the R wrappers for the `Person` example are as follows: ```R Person