Skip to content

Commit

Permalink
gltfpack: Update help to account for vtf/vnf and remove vtn
Browse files Browse the repository at this point in the history
On further reflection, vtn was added prematurely: it only allows setting
the texture coordinate compression mode to default which isn't actually
useful until we add automatic modes, and once we do a lot of things will
change.

This was not very obvious when only vpf and vtf existed, but with vnf as
well it's definitely not a good idea to also add vnn...

Removing this should hopefully be fine since it was a no-op argument that
wasn't documented or advertised anywhere but the command line.
  • Loading branch information
zeux committed Nov 17, 2023
1 parent 8bac922 commit f95a0bb
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions gltf/gltfpack.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1249,10 +1249,6 @@ int main(int argc, char** argv)
{
settings.pos_float = true;
}
else if (strcmp(arg, "-vtn") == 0)
{
settings.tex_float = false;
}
else if (strcmp(arg, "-vtf") == 0)
{
settings.tex_float = true;
Expand Down Expand Up @@ -1499,7 +1495,7 @@ int main(int argc, char** argv)
fprintf(stderr, "\t-si R: simplify meshes targeting triangle/point count ratio R (default: 1; R should be between 0 and 1)\n");
fprintf(stderr, "\t-sa: aggressively simplify to the target ratio disregarding quality\n");
fprintf(stderr, "\t-slb: lock border vertices during simplification to avoid gaps on connected meshes\n");
fprintf(stderr, "\nVertices:\n");
fprintf(stderr, "\nVertex precision:\n");
fprintf(stderr, "\t-vp N: use N-bit quantization for positions (default: 14; N should be between 1 and 16)\n");
fprintf(stderr, "\t-vt N: use N-bit quantization for texture coordinates (default: 12; N should be between 1 and 16)\n");
fprintf(stderr, "\t-vn N: use N-bit quantization for normals and tangents (default: 8; N should be between 1 and 16)\n");
Expand All @@ -1508,9 +1504,9 @@ int main(int argc, char** argv)
fprintf(stderr, "\t-vpi: use integer attributes for positions (default)\n");
fprintf(stderr, "\t-vpn: use normalized attributes for positions\n");
fprintf(stderr, "\t-vpf: use floating point attributes for positions\n");
fprintf(stderr, "\nTexture coordinates:\n");
fprintf(stderr, "\t-vtn: use normalized attributes for texture coordinates (default)\n");
fprintf(stderr, "\nVertex attributes:\n");
fprintf(stderr, "\t-vtf: use floating point attributes for texture coordinates\n");
fprintf(stderr, "\t-vnf: use floating point attributes for normals\n");
fprintf(stderr, "\nAnimations:\n");
fprintf(stderr, "\t-at N: use N-bit quantization for translations (default: 16; N should be between 1 and 24)\n");
fprintf(stderr, "\t-ar N: use N-bit quantization for rotations (default: 12; N should be between 4 and 16)\n");
Expand Down

0 comments on commit f95a0bb

Please sign in to comment.