storch icon indicating copy to clipboard operation
storch copied to clipboard

Hi, sbrunk please review my code add many feature

Open mullerhai opened this issue 3 months ago • 6 comments

HI Sbrunk:

    Now  STorch I have extend many feature , it gain so many line code, I cannot say the detail. 

mullerhai avatar Sep 09 '25 09:09 mullerhai

This is so amazing @mullerhai!

sbrunk avatar Oct 20 '25 20:10 sbrunk

Let's get CI running again so we can run the tests there. I think we need to update the workflows a bit as the actions are outdated.

sbrunk avatar Oct 20 '25 20:10 sbrunk

I've started going through the PR. A few initial remarks:

  • storch-core has a lot of dependencies right now like storch-pandas. Ideally I'd like to keep it very small instead and let the other modules depend on storch-core if possible or create integration modules.
    • Related, do we want to keep the storch-* modules all in separate repositories or does it make sense to move some of them into this repo as submodules?
  • The current Scala version is 3.6.3, while LTS, which is recommended for libraries is currently 3.3.7. Can we use the LTS version to improve compatibility?

sbrunk avatar Oct 28 '25 22:10 sbrunk

I've started going through the PR. A few initial remarks:

  • storch-core has a lot of dependencies right now like storch-pandas. Ideally I'd like to keep it very small instead and let the other modules depend on storch-core if possible or create integration modules.

    • Related, do we want to keep the storch-* modules all in separate repositories or does it make sense to move some of them into this repo as submodules?
  • The current Scala version is 3.6.3, while LTS, which is recommended for libraries is currently 3.3.7. Can we use the LTS version to improve compatibility?

Hi @sbrunk , I have remove storch-pandas storch-safe-tensors storch-sklearn , the scala3 version I think because we need use jupyter notebook, the almond.sh scala kernel is support scala3.6.3 ,it could work normally . if use scala3 3.3.7 I don't know the compatiable with jupyter notebook , it just could make decision by youself , I am not care of the scala version . just could work is ok

mullerhai avatar Oct 29 '25 04:10 mullerhai

Hi @sbrunk , I have remove storch-pandas storch-safe-tensors storch-sklearn , the scala3 version I think because we need use jupyter notebook, the almond.sh scala kernel is support scala3.6.3 ,it could work normally . if use scala3 3.3.7 I don't know the compatiable with jupyter notebook , it just could make decision by youself , I am not care of the scala version . just could work is ok

almond should work fine with Scala 3.3 LTS, but we also need to make sure that the libraries storch depends on are also compatible.

When storch and its dependencies are compiled with Scala 3.3 LTS, then other libraries using Scala 3.3/3.4,3.5,3.6,3.7 can use it but when it's compiled with 3.7, all libraries that depend on it need also be compiled to 3.7.

I created https://github.com/bytedeco/storch-numpy/pull/2 to move storch-numpy to 3.3, could you look at the PR?

sbrunk avatar Nov 02 '25 15:11 sbrunk

HI @sbrunk , I have some idea ,to make more people quickly search storch and try to use storch. we need renamed storch as PyTorch too, how do you think?

mullerhai avatar Nov 04 '25 02:11 mullerhai