openframeworks / openframeworks/openFrameworks

ofMesh Box top and bottom switched

Open
#3,747 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug prelim-analysis section-3D
Dominant language
C++
Stars
10.4k
Forks
2.6k
Avg merge
1d 21h
Merged PRs (30d)
9

Description

Hey,

normally it wouldn't matter but when working with the sides of ofBoxPrimitives the strides and vertices have the wrong indexes for top and bottom mesh.

This is due to that the bottom face is created before the top face in ofMesh. ( comment of code is right, but calculation switched ).

I'm not sure if it effects other parts of OF code when just switching it back to the right order in ofMesh.
If not, I can create a PR.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting the ofMesh code that creates the bottom and top faces, then trace how the sides of ofBoxPrimitives use their strides and vertex indexes. Check whether reversing the calculation affects other openFrameworks code. Done means the top and bottom indexes match the documented face order and related mesh behavior remains correct.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.