search

fossasia / susi_api_wrapper

SUSI AI API Wrapper http://susi.ai
Apache License 2.0
1.48k stars 31 forks source link

Cut a release? #64

Open felixonmars opened 6 years ago

felixonmars commented 6 years ago

It would be nice to have a versioned release which an initial PyPI upload and distribution packaging could start with.

felixonmars commented 5 years ago

I understand that development is still ongoing and could change very often - but releases can really help start all the packaging bits. If you have trouble with PyPI I can help with all the uploading & updating etc given there are git tags.

The project status field in setup.py could always be used to indicate that it's still in early stage, just in case you have concern in releasing unstable things.

I would like to kick start the packaging bits in this year's GSoC, and this is one of the bottom-level dependencies of susi_linux tree.

mariobehling commented 5 years ago

Please go ahead with packaging. Would be great!

hongquan commented 5 years ago

Honestly, I'm not consent about the design of this piece of software (susi_api_wrapper). I think it is better to reconstruct it before doing packaging.

I'm sorry that I don't have time to do it yet.

cseas commented 5 years ago

Can I work on creating a PyPi package for susi_python?

norbusan commented 5 years ago

@hongquan I agree with you that the design/layout is not optimal, in particular because the current repository combines in principle api wrappers for multiple languages.

I would suggest the following;

@cseas What do you think is needed there? A simple

cd susi_api_wrapper.git/python_wrapper
python3 setup.py bdist_wheel

already produces .whl packages. What else are you planning to do?

hongquan commented 5 years ago

@norbusan Agree!

norbusan commented 5 years ago

@norbusan Agree!

Can you create a susi_python repo on github? Then I will move the current files over to preserve the history, and restructure it.

cseas commented 5 years ago

@norbusan I wasn't aware of the wheel generation. The README doesn't mention any wheel generation so I thought we just download the repository and run the program inside it by importing susi_python.

I was thinking of releasing a PyPi package for this so the module can be install with pip install susi_python from any terminal. That would make it easier for people to start using and experimenting with it.

norbusan commented 5 years ago

@cseas Yes, that is for sure the idea. I haven't seen any releases so we could very well create a 0.0.1 release, make a whl package, and upload it to the fossasia pip repository.

norbusan commented 5 years ago

@hongquan we could also physically rename the repository, but this is a bit brutal ... and would need adaption of the current raspi image generation (not mine, but the one that generates the current images by @Orbiter)

cseas commented 5 years ago

@norbusan That sounds good. Should I look into creating the PyPi package and see if it works well?

norbusan commented 5 years ago

@cseas that is for sure a good idea. Things that need to be checked is whether e.g the RPi image with the normal susi_python removed from SUSI.AI folder, but installed via pip, then the susi_linux app still is working.

cseas commented 5 years ago

@norbusan Okay, I'll check it out.

hongquan commented 5 years ago

@norbusan Added the repo susi_python.

norbusan commented 5 years ago

@hongquan I have moved the files with history, and added also Debian packaging. We should probably remove this repository or archive it?

hongquan commented 5 years ago

@norbusan Archive it.