Skip to content

Fix docs, packaging, and minor code issues in gnm.shape#23

Closed
manumishra12 wants to merge 1 commit into
google:mainfrom
manumishra12:fix/docs-typos-packaging-cleanup
Closed

Fix docs, packaging, and minor code issues in gnm.shape#23
manumishra12 wants to merge 1 commit into
google:mainfrom
manumishra12:fix/docs-typos-packaging-cleanup

Conversation

@manumishra12

Copy link
Copy Markdown
Contributor
  • README: correct viewer filename (gnm_colab_viewer.py ->
    gnm_jupyter_viewer.py) and model data filename (gnm.npz -> gnm_head.npz)
  • pyproject: include data/**/*.txt so the landmark definition file (head_sparse_68.txt) is bundled on non-editable installs
  • gnm_jupyter_viewer.py: add the missing Apache-2.0 license header
  • gnm_utils: reference the existing 'variant' attribute instead of the non-existent 'model_type' in the region error message; fix "invidual" typo
  • gnm_data_loader: drop the stale commented-out import; fix "combinating" typo
  • semantic_sampler: ExpressionSampler docstring no longer claims to sample identities
  • gnm_tensorflow: correct attribute name in the GNM docstring (bone_aligned_template_joint_orientations)

- README: correct viewer filename (gnm_colab_viewer.py ->
  gnm_jupyter_viewer.py) and model data filename (gnm.npz -> gnm_head.npz)
- pyproject: include data/**/*.txt so the landmark definition file
  (head_sparse_68.txt) is bundled on non-editable installs
- gnm_jupyter_viewer.py: add the missing Apache-2.0 license header
- gnm_utils: reference the existing 'variant' attribute instead of the
  non-existent 'model_type' in the region error message; fix "invidual" typo
- gnm_data_loader: drop the stale commented-out import; fix "combinating" typo
- semantic_sampler: ExpressionSampler docstring no longer claims to sample
  identities
- gnm_tensorflow: correct attribute name in the GNM docstring
  (bone_aligned_template_joint_orientations)
@manumishra12

Copy link
Copy Markdown
Contributor Author

Superseded by #34. This PR accidentally included __pycache__/*.pyc files; force-pushing to remove them auto-closed it (unrelated-history close, which can't be reopened), so I re-raised the cleaned-up change as #34.

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.

1 participant