Ryan Curtin

Results 312 comments of Ryan Curtin

@riaash04 sure, anyone is welcome to work on this. :+1:

@abernauer that seems like a reasonable approach to me. You can take a look through the existing PRs that have already been merged to help identify any situations where the...

Sure, feel free; if you open a PR, I'd suggest picking only one small bit of the code at a time (to make the review quicker).

In some cases, I think this is still not applied everywhere, so maybe we should leave this open as a good first issue for people looking to get involved?

Hey @Yashwants19, good luck on the Mtech entrance! I am sure it will go well. So I think there are several issues that would need to be figured out. But...

Yes, sorry, haven't had a chance to look into this yet.

Just checking, it sounds here like on the mlpack side we don't need to do anything, and things will work when R 4.2.2 is released. Is that correct?

Hey @benz0li, is there any way to see if this problem still exists? mlpack 4.0.0 just got pushed to CRAN, and I wonder if some of the changes we made...

Awesome, great to hear! :+1:

@Vasniktel agreed, I'm sorry I haven't managed to review this. Things have been crazy. :(