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

kalibr_bagcreater can now handle short filenames. #13

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Commits on Sep 3, 2014

  1. CMake finds correct Eigen config in Ubuntu 14.04.

    The installation tutorial instructs to install Eigen3.  A standard
    Ubuntu installation has no FindEigen.cmake, but only FindEigen3.cmake.
    Therefore, I instructed CMake to require Eigen3 instead of Eigen.
    Patrick de Kok committed Sep 3, 2014
    Configuration menu
    Copy the full SHA
    87df29a View commit details
    Browse the repository at this point in the history
  2. kalibr_bagcreater can now handle short filenames.

    Previously, filenames had to be at least 10 character, without filetype
    extension.  When a filename was shorter, an empty string would be passed
    to int(), which results in a ValueError.  With this version, when an
    empty string is encountered, secs is set to 0.
    Patrick de Kok committed Sep 3, 2014
    Configuration menu
    Copy the full SHA
    e80faaa View commit details
    Browse the repository at this point in the history
  3. Merge branches 'indigo-compatibility' and 'eigen-ubuntu' of github.co…

    …m:pkok/kalibr
    Patrick de Kok committed Sep 3, 2014
    Configuration menu
    Copy the full SHA
    4ddcf2d View commit details
    Browse the repository at this point in the history

Commits on Sep 9, 2014

  1. Grayscale images sometimes problem in calibration.

    When the sensor_msgs/Image is converted to its numpy `ndarray`
    representation, it sometimes has the shape `(n, m, 1)`.  This gives an
    error even though it is a `sensor_msgs::image_encodings::MONO8`.
    
    The underlying algorithm only supports 2D arrays, and the images were
    stored as 3D objects.  Error:
    
    ```
    TypeError: Conversion is only valid for arrays with 1 or 2
    dimensions. Argument has 3 dimensions
    ```
    
    Transforming the image with a reshape if and only if it has a dimension
    of lenght 1 solves this problem.
    Patrick de Kok committed Sep 9, 2014
    Configuration menu
    Copy the full SHA
    2c209c1 View commit details
    Browse the repository at this point in the history
  2. Merge branch 'kalibr_target_extractor'

    Patrick de Kok committed Sep 9, 2014
    Configuration menu
    Copy the full SHA
    e4d3db2 View commit details
    Browse the repository at this point in the history