Skip to content
New issue

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

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

Already on GitHub? # to your account

[doc] Improved documentation for UsdGeomMesh #1254

Closed
wants to merge 1 commit into from

Conversation

barfowl
Copy link
Contributor

@barfowl barfowl commented Jun 27, 2020

Description of Change(s)

Changes here improve the documentation generated for UsdGeomMesh, including:

  • the Detailed Description was expanded to include more about terminology and topology
  • the term "vertex" was specifically avoided in favor or "point" or "face-vertex"
  • a list describing primvar interpolation behaviors specific to meshes was added
  • descriptions of subdivision options were corrected and aligned more closely with OpenSubdiv

Updates were applied to usdGeom/schema.usda and remaining files generated with usdGenSchema.

Fixes Issue(s)

Hopefully reduces confusion and errors previously encountered when using UsdGeomMesh.

    - description expanded to include more about terminology and topology
    - use of "vertex" specifically avoided in favor of "point" or "face-vertex"
    - new list describing primvar interpolation behaviors specific to meshes
    - descriptions of subdivision options expanded:
        - aligned descriptions more closely with OpenSubdiv with links
        - added missing face-varying interpolation options
    - external links modified to use https
@sunyab
Copy link
Contributor

sunyab commented Jul 2, 2020

Closing this out. This PR was merged in f0c39c5 but was accidentally rebased so the merge commit was lost.

@sunyab sunyab closed this Jul 2, 2020
@barfowl barfowl deleted the geom_mesh_docs branch July 7, 2020 21:22
kohakukun added a commit to autodesk-forks/USD that referenced this pull request Oct 21, 2024
…os#1254)

- Rollback part of the side module changes
- Skip timestamp for emscripten due to outdated headers
in emscripten
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants