Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

added data feeder API #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mbuerki
Copy link

@mbuerki mbuerki commented Apr 28, 2014

No description provided.

@mbuerki
Copy link
Author

mbuerki commented Apr 28, 2014

code as I use it right now.

if you are interested in adapting this, I can make some changes, remove some non-generic aspects (i.e. I load four images in the data feeder thread because I know I have four cameras, stuff like that)

@ahundt
Copy link
Collaborator

ahundt commented Apr 28, 2015

What does this do? I just made a lot of changes to improve cross platform support and I'm going to submit another pull request.

@hengli
Copy link
Owner

hengli commented Apr 29, 2015

Sorry that I have not addressed this for so long. The pull request by mbuerki makes it easier for users to use the CamOdoCal library; after placing the text files with odometry/GPS data and image files in the specified directory, they simply call an executable which automatically reads the data, runs the calibration, and generates calibration data. If you are interested in working on making the proposed changes more changes, do feel welcome to do so.

@ahundt
Copy link
Collaborator

ahundt commented Jul 2, 2015

Hey @mbuerki, thanks to @hengli I'm a library maintainer now. If you were still interested in merging the changes with the fixes to the non-generic aspects of your pull request I'd be happy to help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants