Wasim Lorgat
Wasim Lorgat
We chatted on Discord about extending this prototype to use plum everywhere, not just for Transform. I'm still working on that. I'll make this a draft PR again. Sorry for...
@hamelsmu @jph00 this is ready for review. I suggest reviewing #424 and #425 first.
@hamelsmu Done!
@jph00, **TL;DR:** Switching to plum would be a bet that multiple dispatch could lead to powerful use-cases in future. It wouldn't improve much today. Someone with experience using both fastai...
@jph00, thanks for the kind words and the thoughtful suggestions! `plum.function` is compiled with Cython[^cython]. I'm not sure how much of a performance boost this gives `fastai` since it's a...
@jph00 Sorry, I totally missed your reply! Double-checking that my understanding is correct: - [x] This PR's code moves to [`fastdispatch`](https://github.com/fastai/fastdispatch) - [ ] As much of `fastdispatch` functionality as...
Thanks for the issue write-up! The first one is definitely a bug, we'll track that in the corresponding nbdev issue you created. For the second, do you perhaps have a...
Oh no! Could you please share your macOS version? I'm unable to reproduce this on my side, but I have an idea of what might be going wrong if you...
I'm not able to repro this in Positron Version: 2024.07.0 (Universal) build 125. @isabelizimm @nstrayer could you please try again?
@ntluong95 have you used VSCode's "Run cell by line" feature? If that were available in Positron, would that work for you instead?