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

Stubs for FlyingObjects implementation #6507

Closed
wants to merge 18 commits into from

Commits on Oct 6, 2023

  1. RemoveDuplicateVertices Implementation for TriangleMesh.

    Functionality to remove duplicate vertices and update
    other vertex attributes and triangle indices is implemented.
    The C++ and python tests are also written to test the code.
    
    On branch sganjugu/remdup2
    Changes to be committed:
    modified:   cpp/open3d/t/geometry/TriangleMesh.cpp
    modified:   cpp/open3d/t/geometry/TriangleMesh.h
    modified:   cpp/pybind/t/geometry/trianglemesh.cpp
    modified:   cpp/tests/t/geometry/TriangleMesh.cpp
    modified:   python/test/t/geometry/test_trianglemesh.py
    intelshashi committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    4535bb2 View commit details
    Browse the repository at this point in the history
  2. Updated CHANGELOG.md

    intelshashi committed Oct 6, 2023
    Configuration menu
    Copy the full SHA
    b56ca5f View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2023

  1. Configuration menu
    Copy the full SHA
    158d965 View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2023

  1. Duplicate vertex attributes shrunk using IndexGet

    Previously, I was doing a manual copy and shrinking the vertices.
    Instead in this checkin all vertex attributes, including the
    coordinates are shrunk using IndexGet method using a vertex mask.
    
    Further, the triangle index remapping is similar to what was done
    earlier, with some simplications. One thing to note, is that
    we cannot use utility::InclusivePrefixSum to remap vertex indices
    because the duplicates can occur in any position and may be duplicates
    of any earlier vertex.
    
    For e.g., suppose there were 9 vertices to start with, and 8th (index 7, starting from 0),
    was a duplicate of 2nd (index 1, starting from 0).
    
    So, the vertex mask would look like this:
    Vertex indices: [0, 1, 2, 3, 4, 5, 6, 7, 8]
    Vertex mask:    [1, 1, 1, 1, 1, 1, 1, 0, 1]
    Prefix sum:   [0, 1, 2, 3, 4, 5, 6, 7, 7, 8]
    This gives an incorrect index map for 8th vertex, which is mapped to
    index 7 instead of 1.
    
    On branch sganjugu/remdup2
    Your branch is up to date with 'origin/sganjugu/remdup2'.
    Changes to be committed:
    modified:   ../cpp/open3d/t/geometry/TriangleMesh.cpp
    modified:   ../python/test/t/geometry/test_trianglemesh.py
    intelshashi committed Nov 15, 2023
    Configuration menu
    Copy the full SHA
    a7664c8 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    9155acf View commit details
    Browse the repository at this point in the history

Commits on Nov 24, 2023

  1. Configuration menu
    Copy the full SHA
    90b6b39 View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2024

  1. Configuration menu
    Copy the full SHA
    d50ec35 View commit details
    Browse the repository at this point in the history

Commits on Jan 17, 2024

  1. Fix .desktop entry when installing in a UNIX system (isl-org#6599)

    * Fix .desktop entry when installing in a UNIX system
    
    * Fix the path to the default Open3D application when installing to a custom path.
    
    * Update CHANGELOG.md for PR#6599
    
    ---------
    
    Co-authored-by: Ewing Kang <ewing_kang@glorymakeup.com>
    EwingKang and Ewing Kang committed Jan 17, 2024
    Configuration menu
    Copy the full SHA
    0cf605f View commit details
    Browse the repository at this point in the history

Commits on Jan 19, 2024

  1. Configuration menu
    Copy the full SHA
    ee65b6c View commit details
    Browse the repository at this point in the history
  2. Bump jinja2 from 3.1.2 to 3.1.3 in /docs (isl-org#6597)

    Bumps [jinja2](https://github.com/pallets/jinja) from 3.1.2 to 3.1.3.
    - [Release notes](https://github.com/pallets/jinja/releases)
    - [Changelog](https://github.com/pallets/jinja/blob/main/CHANGES.rst)
    - [Commits](pallets/jinja@3.1.2...3.1.3)
    
    ---
    updated-dependencies:
    - dependency-name: jinja2
      dependency-type: direct:production
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] committed Jan 19, 2024
    Configuration menu
    Copy the full SHA
    4214a0d View commit details
    Browse the repository at this point in the history

Commits on Feb 3, 2024

  1. Configuration menu
    Copy the full SHA
    f4bcc48 View commit details
    Browse the repository at this point in the history

Commits on Feb 4, 2024

  1. Fixed vertex mask and handle case of triangle indices being absent.

    - Made vertex mask type bool.
    - Check if mesh has triangle indices before getting them.
    - Modified some errors to warnings.
    
    On branch sganjugu/remdup2
    Your branch is up to date with 'origin/sganjugu/remdup2'.
    Changes to be committed:
    modified:   cpp/open3d/t/geometry/TriangleMesh.cpp
    shashigk committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    cbf75f8 View commit details
    Browse the repository at this point in the history
  2. Fixed code style

    shashigk committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    db3f353 View commit details
    Browse the repository at this point in the history
  3. Used DISPATCH_FLOAT_INT_DTYPE_TO_TEMPLATE macro.

    Instead of using if statements explicitly, used DISPATCH_FLOAT_INT_DTYPE_TO_TEMPLATE
    to dispatch to RemoveDuplicateVerticesWorker function appropriately.
    
    On branch sganjugu/remdup2
    Your branch is up to date with 'origin/sganjugu/remdup2'.
    Changes to be committed:
    modified:   ../cpp/open3d/t/geometry/TriangleMesh.cpp
    shashigk committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    b98ace5 View commit details
    Browse the repository at this point in the history
  4. Merge remote-tracking branch 'intelshashi/sganjugu/flyingobject2' int…

    …o sganjugu/flyingobject2
    shashigk committed Feb 4, 2024
    Configuration menu
    Copy the full SHA
    519f376 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2024

  1. Configuration menu
    Copy the full SHA
    18231d3 View commit details
    Browse the repository at this point in the history
  2. Merge remote-tracking branch 'intelshashi/sganjugu/flyingobject2' int…

    …o sganjugu/flyingobject2
    shashigk committed Feb 9, 2024
    Configuration menu
    Copy the full SHA
    33f1272 View commit details
    Browse the repository at this point in the history

Commits on Feb 10, 2024

  1. Configuration menu
    Copy the full SHA
    df02cb2 View commit details
    Browse the repository at this point in the history