vod-converter icon indicating copy to clipboard operation
vod-converter copied to clipboard

Updates to facilitate registration/upload to PyPI

Open monocongo opened this issue 4 years ago • 2 comments

In order to register and upload this to PyPI there should be a setup.py etc. in order to facilitate the process, as described here.

The short version:

  1. Get the setup.py file in shape as described here
  2. $ rm -rf dist
  3. $ python setup.py sdist bdist_wheel
  4. $ twine check dist/*
  5. $ twine upload --repository-url https://test.pypi.org/legacy/ dist/*
  6. $ twine upload dist/*

If this is useful for the project I can make the updates and submit a PR but I don't think I have permission to upload the package to PyPI.

monocongo avatar Sep 03 '19 23:09 monocongo

Hi James, sorry I missed this. As you've probably guessed, I hadn't been actively maintaining this, and am no longer working in the FCAV lab, but given this project is referenced by the KITTI website and seems to be useful, I'll try to start keeping an eye on requests like this.

So I welcome your pull requests and will take a look!

krosaen avatar Oct 03 '19 11:10 krosaen

Thanks, Karl. The code is certainly useful, I appreciate your effort and for making the code available here. I'm incorporating it into a dataset creation script and it's helpful to be able to call the conversion functions from my script rather than using the provided command line interface. There are several ways to skin that cat but what I'm familiar with is to install the package into my environment and import the module(s) for use within my script. I'm cooking up a setup.py script to facilitate this and hopefully, it won't take too much more work to get everything in place for a PyPI registration to allow for an easy install via pip install vod_converter.

Once I submit a PR for this please make sure that I've provided the correct author attribution(s), license restrictions, reasonable version number, etc. Thanks again for your help!

monocongo avatar Oct 03 '19 14:10 monocongo