XBCF icon indicating copy to clipboard operation
XBCF copied to clipboard

Set up CI for python build (#2)

Open kklein opened this issue 3 years ago • 4 comments

  • Set up CI for python build via github action.
  • Fix bug due to [0] - a function parameter was removed and arguments were only removed in R client code.

[0] https://github.com/socket778/XBCF/commit/e0c0de290e537c446acb7088da83750479e092cb

kklein avatar Aug 05 '22 07:08 kklein

Hi @socket778 !

We figured it could be useful to have build the Python package by virtue of CI - making the current state of building transparent. What do you think of this?

kklein avatar Aug 11 '22 16:08 kklein

Hi @kklein -- thanks for the contribution! I need to read about CI and then I'll merge the PR/get back to you. I'm currently on vacation, so I'll probably be in touch at some point next week.

socket778 avatar Aug 12 '22 18:08 socket778

Hi @socket778 ! Just let me know if I can be of help. :)

kklein avatar Aug 25 '22 08:08 kklein

Hi @kklein, sorry about the delay in responding. We are currently discussing possible merge of XBCF back into original XBART repo; continuous integration was originally set up for XBART python using Travis, although not sure if it was actively used. Could you please send me an email (krantsevich at asu dot edu) so we can talk in more detail what your interest in XBCF is and how we can best accommodate it (we can also schedule a Zoom call if it's easier)? Your input may also help us decide if we keep XBCF as a separate package for R and python. Thanks for your interest in the package!

socket778 avatar Aug 29 '22 04:08 socket778