OpenPTV / openptv

This is our central repository, holding only the community approved code and documentation
www.openptv.net
GNU Lesser General Public License v3.0
37 stars 28 forks source link

Open for contribution #202

Open shredderbvek opened 2 years ago

shredderbvek commented 2 years ago

Volunteering for contribution as per the linkedin post by Prof. Alex Liberzon.

alexlib commented 2 years ago

Hi @shredderbvek thanks for writing.

Let's first follow the installation for developers instructions and see that you have a working version on your computer. Any issue or problem that raises during the installation - is great for us to learn and make the process simpler for the new developers. Please report the process, maybe as a short blog post or a wiki report. Thanks Alex https://openptv-python.readthedocs.io/en/latest/developers.html

alexlib commented 2 years ago

the first issue to resolve is to improve this documentation :)

https://github.com/openptv/docs/blob/master/docs/source/developers.txt

prast367 commented 2 years ago

Hi Prof. Alex!

I am also in for volunteering as per your LinkedIn post.

Best regards, Prakhar Rastogi

alexlib commented 2 years ago

Great. @prast367 Let's follow the same path : installation for developers and documentation first.

We all work with forks - fork a software to your account and after changes ping us to test it and review it or send pull request If these tasks are difficult - let us meet online to have a live demo.

prast367 commented 2 years ago

Hi Prof. Alex!

Can I start by next week? Due to some health issues, I won't be able to start right now?

Best regards,

Prakhar

alexlib commented 2 years ago

Hi Prof. Alex!

Can I start by next week? Due to some health issues, I won't be able to start right now?

Best regards,

Prakhar

everyone is free to work on his/her own pace.

CleisonManriqueAguirre commented 2 years ago

Hi . Prof. Alex I am a volunteering because of your LinkedIn post . Thanks for the opportunity. What is the first goal aside of installing Open PTV ?

alexlib commented 2 years ago

Hi . Prof. Alex

I am a volunteering because of your LinkedIn post . Thanks for the opportunity. What is the first goal aside of installing Open PTV ?

Hi @C73150N great to see you. The first goal is to fix some bugs. In C or in Python. Another option is to write tests. See in py_bind

Whenever you are ready with the installed software, we can chat together and check which you like most

CleisonManriqueAguirre commented 2 years ago

Hi . Prof. Alex

I am a volunteering because of your LinkedIn post . Thanks for the opportunity. What is the first goal aside of installing Open PTV ?

Hi @C73150N great to see you. The first goal is to fix some bugs. In C or in Python. Another option is to write tests. See in py_bind

Whenever you are ready with the installed software, we can chat together and check which you like most

Sure . I will let you notice . Do you recommend to use Windows or Ubuntu for better performing off OpenPTV ?

alexlib commented 2 years ago

Ubuntu is simpler. Windows if you must

On Mon, 14 Mar 2022 at 19:30 Cleison Armando Manrique Aguirre < @.***> wrote:

Hi . Prof. Alex

I am a volunteering because of your LinkedIn post . Thanks for the opportunity. What is the first goal aside of installing Open PTV ?

Hi @C73150N https://github.com/C73150N great to see you. The first goal is to fix some bugs. In C or in Python. Another option is to write tests. See in py_bind

Whenever you are ready with the installed software, we can chat together and check which you like most

Sure . I will let you notice . Do you recommend to use Windows or Ubuntu for better performing off OpenPTV ?

— Reply to this email directly, view it on GitHub https://github.com/OpenPTV/openptv/issues/202#issuecomment-1067096033, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFWMZUPVFBYYM6QNI7VYMLU75ZRVANCNFSM5QBOTY6Q . You are receiving this because you commented.Message ID: @.***>

CleisonManriqueAguirre commented 2 years ago

Hi. I am having some issues while installing it but they seem to be solvable . Have you tried to install the C compiler through MSYS2 ? Or its not recommendable

CleisonManriqueAguirre commented 2 years ago

Ubuntu is simpler. Windows if you must

Hi . I already installed OpenPTV and it all goes fine at the moment . Whats will be next ?

alexlib commented 2 years ago

Ubuntu is simpler. Windows if you must

Hi . I already installed OpenPTV and it all goes fine at the moment . Whats will be next ?

  1. Please try to run the software according to the tutorial videos https://openptv-python.readthedocs.io/en/latest/tutorial.html
    you can use https://github.com/OpenPTV/test_cavity for comparing with the video

  2. Can you run the tests? There are two types of tests - C using libcheck library and Python, using pytest. We'd like to have all the tests which are presently in C - also in Python. for instance in python:

cd py_bind
python setup.py prepare
python setup.py install
cd tests
nose

should run 61 tests.

CleisonManriqueAguirre commented 2 years ago

Sure I can . I will do step 1 first . Thanks . I forgot to mention that in the installation instruction there is missing some steps that I followed from google groups page specially the adding of pathdata.py file .

CleisonManriqueAguirre commented 2 years ago

Ubuntu is simpler. Windows if you must

Hi . I already installed OpenPTV and it all goes fine at the moment . Whats will be next ?

  1. Please try to run the software according to the tutorial videos https://openptv-python.readthedocs.io/en/latest/tutorial.html you can use https://github.com/OpenPTV/test_cavity for comparing with the video
  2. Can you run the tests? There are two types of tests - C using libcheck library and Python, using pytest. We'd like to have all the tests which are presently in C - also in Python. for instance in python:
cd py_bind
python setup.py prepare
python setup.py install
cd tests
nosetests

should run 61 tests.

The nose command gives me this error while trying the tests // AttributeError: module 'collections' has no attribute 'Callable' , the cython==0.29.6 version is not available as the requirements.txt says to install it ... maybe its due to the versions

alexlib commented 2 years ago

Let's have a TeamViewer session - I have never seen such an error. Write me to email and we find some convenient time