Implement full MeshIO (OBJ/PLY) by csparker247 · Pull Request #17 · educelab/libcore

csparker247 · 2026-03-25T15:50:25Z

Summary

This PR implements OBJ and PLY mesh IO for libcore — read and write, with three tiers (positions only; positions + UV map; positions + UV map + texture paths). A case-insensitive read_mesh / write_mesh facade dispatches by file extension. The mesh IO track (mesh-io_20260323) is complete.

MeshIO pipeline

io/MeshIO.hpp — facade with case-insensitive extension dispatch (.obj, .OBJ, .Ply all work)
io/MeshIO_OBJ.hpp — read/write OBJ + MTL with per-wedge UVs, multi-chart support via usemtl
io/MeshIO_PLY.hpp — read/write ASCII PLY, read binary little-endian PLY, per-wedge texcoord list + backward-compat per-vertex s/t
Vertex traits (has_normal, has_color) drive compile-time inclusion of normals and colors in both readers and writers
UVMap traits (has_chart) drive multi-chart OBJ output

Supporting additions

utils/MeshUtils.hpp — expand_at_seams for formats needing per-vertex UVs
Mesh: num_vertices, num_faces, clear
String.hpp: zero-allocation split(string_view, vector&, pred) predicate overload, to_string / to_string_view (charconv-based, locale-independent, buffer-reusing), per-type charconv fallbacks
cmake/CheckCharconvFP.cmake replaces CheckToNumericFP.cmake — probes from_chars and to_chars independently for float, double, long double, emitting per-type EDUCE_CORE_NEED_* definitions

Hardening

Binary PLY: stream state checked after every read; truncated files throw instead of returning garbage
OBJ: face vertex indices bounds-checked before insert_face
PLY header: element count capped at 500M; malformed element / property lines throw instead of being silently skipped; property before any element throws
PLY face records: texcoord and unknown list property counts capped at 1024 (vertex_indices still capped at 256)
PLY colors: stored in their native precision (U8C3 for uchar, U16C3 for ushort, F32C3 for float/double) instead of being forced through a lossy uchar cast
OBJ mtllib with no argument now throws instead of dereferencing out of bounds
Write-error detection on every write_obj / write_ply tier (catches silent I/O failures like disk full)
Stream error check after OBJ read loop distinguishes I/O errors from normal EOF

Tests

55 cases in tests/src/TestMeshIO.cpp covering round-trips (positions, normals, colors, UVs, textures, multi-chart, N-gons, empty meshes), facade dispatch, and all hardening cases: truncated binary PLY, out-of-bounds indices, excessive element / texcoord / list counts, malformed PLY headers, bare mtllib, case-insensitive extensions, native-precision color storage.

…ring Extends String.hpp with utilities needed for locale-independent mesh IO serialization and efficient whitespace parsing of OBJ/PLY files. - split(): add predicate overload (single O(n) pass); no-arg overload now splits on any whitespace via the predicate path; string-delimiter overload fast-paths single-char delimiters through the predicate overload - to_string_view(buf, val): caller-supplied buffer variant of to_chars with no heap allocation; intended for hot paths (e.g. per-vertex coordinate output in a write loop) - to_string(val): convenience wrapper over to_string_view for one-off conversions; allocates a std::string Replaces CheckToNumericFP.cmake + CheckToCharsFP.cmake with a unified CheckCharconvFP.cmake that probes from_chars and to_chars separately. EDUCE_CORE_NEED_CHARCONV_FP is now set only by the from_chars probe (the only path with a compile-time fallback). Updates README and conductor artifacts for the mesh-io_20260323 track. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Replace split_ws references with split() (no-arg whitespace form) - Update to_string_view usage to caller-provided buffer pattern: to_string_view(buf, val) instead of to_string_view(val) - Record preparatory String.hpp commit (2f67926) in plan overview - Bump updated date to 2026-03-25 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…h-io_20260323) Add compile-time detection traits has_normal<V>, has_color<V>, and has_chart<UVMapT> in detail/MeshTraits.hpp using the std::void_t detection idiom. IO functions use these via if constexpr to conditionally read/write normal, color, and chart data without extra overloads. Register TestMeshIO.cpp and add MeshTraits.hpp to installed headers. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Add read_obj / write_obj across all three tiers in MeshIO_OBJ.hpp. Write uses to_string_view(buf, val) for zero-allocation numeric output. Read uses split() + parse_face_ref() for locale-independent parsing. Detection traits (has_normal, has_color, has_chart) drive all conditional IO via if constexpr. Multi-chart write enforces has_chart via static_assert. Normals are written/read per-vertex; UVs are per-wedge. MTL texture paths are recovered from map_Kd in material-declaration order. Also adds Mesh::num_vertices() / num_faces() required for iteration. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…0260323) - Add MeshUtils.hpp with expand_at_seams: deduplicates (vertex, uv-pool) pairs across face corners, splitting seam vertices and returning a flat per-vertex UV array suitable for PLY output - Add MeshIO_PLY.hpp with write_ply (Tiers 1–3) and read_ply (Tiers 1–2); supports ASCII write, ASCII and binary-little-endian read, per-vertex s/t UV properties, and comment TextureFile header (MeshLab convention) - Add expand_at_seams and PLY round-trip tests (Tasks 3.1, 3.3) covering no-seam, single-seam, binary PLY, UV expansion, texture path round-trip, and missing-file error cases - Register MeshIO_PLY.hpp and MeshUtils.hpp as installed public headers Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…-io_20260323) Add MeshIO.hpp with read_mesh and write_mesh overloads (Tiers 1–3) that dispatch to OBJ or PLY IO by file extension; throw std::runtime_error for unsupported extensions. Add facade dispatch tests for both formats and all three tiers. Register MeshIO.hpp as an installed public header. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…20260323) Security: - parse_face_ref: throw on zero or negative OBJ face index - read_ply_impl: cap per-face vertex count at 256; validate face indices against n_vertices; throw on binary big-endian format Performance: - String::split(): thread_local static locale eliminates per-call mutex - Add split_into() overload; OBJ/PLY ASCII parsers reuse token vector - read_obj_impl: hoist face_verts/face_vts/face_vns outside loop - expand_at_seams: std::unordered_map with documented Fibonacci pair hash Architecture: - write_obj: extract write_obj_vertices/faces helpers; Tiers 1-3a share them - write_ply: extract write_ply_header/data helpers; all tiers share them - Add read_ply(path, mesh, uvmap) Tier 2 overload - read_mesh Tier 2 calls new read_ply Tier 2 directly (no throwaway vector) Testing: - Unique temp dirs via this-pointer hex suffix in all three fixtures - Full 3x3 position verification in all position round-trip tests - Verify all three vertex colors in color round-trip tests - Write-failure tests for write_obj, write_ply, write_mesh - PLY UV coordinate value verification - v/vt/vn combined face format test - PLY malformed header test - NCMesh (normals+colors) OBJ round-trip test - BinaryBE throw test - expand_at_seams empty-mesh test Platform: - CheckCharconvFP.cmake: add long double to_chars probe - String.hpp: to_string_view<long double> fallback via double cast when EDUCE_CORE_NEED_TO_CHARS_LONG_DOUBLE_FALLBACK is defined (macOS-26) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Replace the single EDUCE_CORE_NEED_CHARCONV_FP flag with 6 independent per-type probes (from_chars × {float,double,long double} and to_chars × {float,double,long double}). Each fallback specialisation is now gated on its own definition, so a platform where only long double is missing does not regress float/double paths. to_string_view fallbacks use snprintf (float: %.9g, double: %.17g); long double casts to double (same representation on Apple Silicon). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

csparker247 · 2026-03-26T00:09:16Z

@copilot Review the PR implementation

Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/d7bca254-3f1d-4647-abdb-4eb0f9586a21

…_obj Tier 3b Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/d7bca254-3f1d-4647-abdb-4eb0f9586a21

Copilot · 2026-03-26T00:20:06Z

@copilot Review the PR implementation

Reviewed the implementation and fixed issues found:

color_to_rgb (OBJ) and color_to_u8c3 (PLY) — Added missing support for U16C1, U16C3, U16C4, U8C4, and F32C4 color types (4c51c7c)
Duplicate using Vertex declaration in write_obj Tier 3b removed (658cdb5)

All 39 tests pass.

…tive dispatch, and edge-case tests Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/50bb342d-a00b-4bf5-9c2b-7c2af9957968

Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/50bb342d-a00b-4bf5-9c2b-7c2af9957968

… these implementations.

…h-io_20260323) Collapse the three identical `if (back=='\\r') pop_back()` + empty-check idioms in the ASCII read paths of MeshIO_PLY.hpp into a single `trim_right_in_place(line)` call from String.hpp. No behaviour change; trim_right already treats '\\r' as whitespace via std::isspace. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ispatch Pre-compute PropRole enum for every PLYProp in parse_ply_header based on element context (vertex vs face). Replace the if-else name-comparison chains in both binary and ASCII vertex/face reader inner loops with switch statements on the pre-computed role, eliminating O(properties) string comparisons per vertex/face. No behavior change; 17/17 tests pass. (mesh-io_20260323)

…reader Pre-compute per-property byte offsets from the PLY header so the binary vertex loop makes one istream::read() per vertex (O(vertices)) instead of one read per property per vertex (O(properties × vertices)). Adds read_ply_prop_from_buf<T> alongside the existing stream-based read_ply_binary_prop<T>; unknown/skipped properties are read into the buffer and their bytes are simply ignored, preserving correct stream positioning without extra istream::ignore() calls. Closes task 5.7 (mesh-io_20260323) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ly_impl The face-element branch of read_ply_impl contained ~80 lines of nested binary + ASCII parsing logic inside the element loop. Extract into two inline detail functions: - read_ply_face_binary: parses one binary face record from an istream - read_ply_face_ascii: parses one ASCII face record from a token vector Both take a reusable face_indices buffer and texcoords vector as out-params, clearing them before filling. read_ply_impl now delegates to these helpers and constructs the Face from the returned index buffer, reducing the nesting depth and isolating per-face parsing logic. Pre-compute load_texcoords = uvmap != nullptr && has_texcoord once outside the face loop; both helpers receive it as a plain bool. Closes task 5.8 (mesh-io_20260323) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…_20260323) Tasks were completed in Phase 5 (cb5ee97, 6b9d7f5); carry-forward references in Phase 6 were not ticked at the time.

@brief

- PLYProp, PLYElement, PLYHeader and their members/enums: add @brief + inline ///< comments so EXTRACT_PRIVATE builds cleanly - ply_type_bytes, read_ply_binary_prop, read_ply_impl: add @brief - Fix broken \ref read_ply(path,mesh) (used param names, not types) - MeshIO_OBJ.hpp: escape <stem> as <stem> to suppress Doxygen xml/html tag warning

- product.md: add UVMap, IO, and trait detection to Key Goals - tech-stack.md: add UVMap, MeshUtils, MeshIO, MeshIO_OBJ, MeshIO_PLY to component table - README.md: add MeshIO header-only entries; add Mesh IO usage section

csparker247

@copilot Review the comments.

csparker247 · 2026-04-22T02:32:15Z

+4. Provide standard Mesh IO (OBJ and PLY read/write) including UV maps, texture
+   paths, and compile-time vertex trait detection (`has_normal`, `has_color`)


Suggested change

4. Provide standard Mesh IO (OBJ and PLY read/write) including UV maps, texture

paths, and compile-time vertex trait detection (`has_normal`, `has_color`)

csparker247 · 2026-04-22T02:36:20Z

+auto cross(const T1& a, const std::initializer_list<T2>& b) -> T1
+{
+    if (std::size(a) != 3 or std::size(b) != 3) {
+        throw std::invalid_argument("Inputs have mismatched dimensions");


Suggested change

throw std::invalid_argument("Inputs have mismatched dimensions");

throw std::invalid_argument("Inputs must be 3-dimensional");

csparker247 · 2026-04-22T02:39:19Z

+            chars.push_back(d[0]);
+        }
+        split(s, tokens, [&chars](char c) {
+            return std::find(chars.begin(), chars.end(), c) != chars.end();


Is this correct?

csparker247 · 2026-04-22T02:45:23Z

+#### A note on `to_numeric` and `to_string` compilation
+
+`to_numeric` uses `std::from_chars` for string-to-number conversion.
+`to_string` and `to_string_view` use `std::to_chars` for number-to-string
+conversion. Both were introduced in C++17 but floating-point support arrived
+later and is gated on the runtime library version (e.g. macOS 13.3+).
+
+CMake probes for both at configure time and reports the results:
+
+- **`CXX_CHARCONV_FP_FROM_CHARS`** — whether `std::from_chars` supports
+  `float`. If not, `EDUCE_CORE_NEED_CHARCONV_FP` is defined and `to_numeric`
+  falls back to `std::stof` / `std::stod` / `std::stold`.
+- **`CXX_CHARCONV_FP_TO_CHARS`** — whether `std::to_chars` supports `float`.
+  `to_string_view` requires this unconditionally; no fallback is provided.
+
+When linking against the `educelab::core` CMake target, `EDUCE_CORE_NEED_CHARCONV_FP`
+is propagated automatically. If using the library header-only, check the CMake
+cache variable and set the definition manually:


Is this accurate?

csparker247 · 2026-04-22T02:45:40Z

-# Conditionally add the to_numeric compiler definition
-if(EDUCE_CORE_NEED_TO_NUMERIC_FP)
-    target_compile_definitions(foo PRIVATE EDUCE_CORE_NEED_TO_NUMERIC_FP)
+# Propagate the to_numeric fallback definition if needed
+if(EDUCE_CORE_NEED_CHARCONV_FP)
+    target_compile_definitions(foo PRIVATE EDUCE_CORE_NEED_CHARCONV_FP)


Is this accurate?

csparker247 · 2026-04-23T18:25:31Z

+                } else {
+                    // ASCII: skip '#'-comment lines.
+                    while (std::getline(file, line)) {
+                        trim_right_in_place(line);


Why not stringview trim_right or trim? Especially since we're using a string_view for split later.

csparker247 · 2026-04-23T18:27:49Z

+                        if (color_type == "uchar" || color_type == "char") {
+                            mesh.vertex(new_vi).color = Color::U8C3{
+                                static_cast<uint8_t>(r),
+                                static_cast<uint8_t>(g),
+                                static_cast<uint8_t>(b)};
+                        } else if (
+                            color_type == "ushort" || color_type == "short") {
+                            mesh.vertex(new_vi).color = Color::U16C3{
+                                static_cast<uint16_t>(r),
+                                static_cast<uint16_t>(g),
+                                static_cast<uint16_t>(b)};
+                        } else {


Suggested change

if (color_type == "uchar" || color_type == "char") {

mesh.vertex(new_vi).color = Color::U8C3{

static_cast<uint8_t>(r),

static_cast<uint8_t>(g),

static_cast<uint8_t>(b)};

} else if (

color_type == "ushort" || color_type == "short") {

mesh.vertex(new_vi).color = Color::U16C3{

static_cast<uint16_t>(r),

static_cast<uint16_t>(g),

static_cast<uint16_t>(b)};

} else {

if (color_type == "uchar") {

mesh.vertex(new_vi).color = Color::U8C3{

static_cast<uint8_t>(r),

static_cast<uint8_t>(g),

static_cast<uint8_t>(b)};

} else if (

color_type == "ushort") {

mesh.vertex(new_vi).color = Color::U16C3{

static_cast<uint16_t>(r),

static_cast<uint16_t>(g),

static_cast<uint16_t>(b)};

} else {

Don't include the signed types char and short in these conditions. That could be an incorrect cast if the values are negative.

csparker247 · 2026-04-23T18:34:24Z

+- **`CXX_CHARCONV_FP_FROM_CHARS`** — whether `std::from_chars` supports
+  `float`. If not, `EDUCE_CORE_NEED_CHARCONV_FP` is defined and `to_numeric`
+  falls back to `std::stof` / `std::stod` / `std::stold`.
+- **`CXX_CHARCONV_FP_TO_CHARS`** — whether `std::to_chars` supports `float`.
+  `to_string_view` requires this unconditionally; no fallback is provided.
+
+When linking against the `educelab::core` CMake target, `EDUCE_CORE_NEED_CHARCONV_FP`
+is propagated automatically. If using the library header-only, check the CMake
+cache variable and set the definition manually:


Is this still accurate? What compile definitions are added now? Update if this is wrong.

csparker247 · 2026-04-23T18:34:55Z

+# Propagate the to_numeric fallback definition if needed
+if(EDUCE_CORE_NEED_CHARCONV_FP)
+    target_compile_definitions(foo PRIVATE EDUCE_CORE_NEED_CHARCONV_FP)


Same. Update if needed.

Addresses all unresolved review comments on the MeshIO implementation PR. ## `MeshIO_PLY.hpp` - **`PLYType` enum** replaces `std::string` for `PLYProp::type` and `list_count_type`; `parse_ply_type()` converts header tokens at parse time. `ply_type_bytes`, `read_ply_binary_prop`, and `read_ply_prop_from_buf` now dispatch via `switch` instead of sequential string comparisons - **Color-type conditions** now match only `PLYType::UChar` / `PLYType::UShort`; removed signed `char`/`short` cases that could produce incorrect casts for negative values - **`trim_right` (string_view)** replaces `trim_right_in_place` in all three ASCII read loops; the trimmed `string_view` is passed directly to `split`, avoiding a redundant cast ## CMake charconv probes - `CheckCharconvFP.cmake` now appends needed definitions to `EDUCE_CORE_CHARCONV_DEFS` instead of calling `add_compile_definitions` - `CMakeLists.txt` applies them via `target_compile_definitions(core PUBLIC ...)`, so they propagate automatically to any downstream target via `target_link_libraries` ```cmake # No manual propagation needed anymore target_link_libraries(foo PRIVATE educelab::core) ``` ## Miscellaneous - `Math.hpp`: `cross()` error message → `"Inputs must be 3-dimensional"` - `String.hpp`: comment at predicate-dispatch call site clarifying it routes to the `is_invocable_r_v`-constrained overload, not back into the variadic template - `README.md`: charconv section updated with correct per-type definition names (`EDUCE_CORE_NEED_FROM/TO_CHARS_{FLOAT,DOUBLE,LONG_DOUBLE}`) and revised cmake example reflecting automatic propagation - `conductor/product.md`: trimmed goal #4 per suggestion --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com>

Flip if/else blocks with a small false branch into early continue/return/skip, hoisting the main body out one indent level. - PLY: read_ply_face_ascii inner loop (matches read_ply_face_binary) - PLY: skip_binary_prop lambda - OBJ: read_obj_impl MTL parse block (early return) - OBJ: write_obj Tier 3b chart-grouping loop Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

csparker247 and others added 8 commits March 25, 2026 06:38

chore: mark Phase 1 complete (mesh-io_20260323)

9442655

chore: mark Phase 2 complete (mesh-io_20260323)

2f6612d

chore: mark Phase 3 complete (mesh-io_20260323)

57bb3b0

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

csparker247 commented Mar 25, 2026

View reviewed changes

Comment thread include/educelab/core/io/MeshIO_OBJ.hpp

csparker247 and others added 5 commits March 25, 2026 15:27

chore: mark Phase 4 and track complete (mesh-io_20260323)

3788b89

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

chore: mark Phase 5 tasks 5.1-5.5 complete (mesh-io_20260323)

6ced8c7

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Copilot started work on behalf of csparker247 March 26, 2026 00:09 View session

Copilot AI and others added 2 commits March 26, 2026 00:17

fix(MeshIO): support all Color types in color_to_rgb and color_to_u8c3

4c51c7c

Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/d7bca254-3f1d-4647-abdb-4eb0f9586a21

fix(MeshIO_OBJ): remove duplicate 'using Vertex' declaration in write…

658cdb5

…_obj Tier 3b Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/d7bca254-3f1d-4647-abdb-4eb0f9586a21

Copilot finished work on behalf of csparker247 March 26, 2026 00:20

Copilot started work on behalf of csparker247 March 26, 2026 00:25 View session

Copilot AI and others added 2 commits March 26, 2026 00:35

fix(MeshIO): add stream error checks, bounds validation, case-insensi…

28f3ec3

…tive dispatch, and edge-case tests Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/50bb342d-a00b-4bf5-9c2b-7c2af9957968

style: restore section separators lost during edit

cba7923

Co-authored-by: csparker247 <1434526+csparker247@users.noreply.github.com> Agent-Logs-Url: https://github.com/educelab/libcore/sessions/50bb342d-a00b-4bf5-9c2b-7c2af9957968

Copilot AI changed the title ~~Feature/mesh io 20260323~~ Harden mesh IO: stream validation, bounds checks, case-insensitive dispatch Mar 26, 2026

Copilot finished work on behalf of csparker247 March 26, 2026 00:41

csparker247 added 5 commits March 26, 2026 10:05

Remove detail/MeshTraits

ca18909

Rename split_into -> split and define new vector versions in terms of…

b0b63d4

… these implementations.

String: docs and tests

1d12044

OBJ: Review

149d3f0

Fix regressions

e52159b

csparker247 and others added 17 commits April 6, 2026 11:54

Handle TODOs

1fa2b71

Merge branch 'develop' into feature/mesh-io_20260323

b841b72

obj_io: Error reporting. math: fix init list hack

f53d54f

Cleanup and comments

0c1b94a

chore: mark task 6.3 complete (mesh-io_20260323)

087491a

chore: mark task 5.6 complete (mesh-io_20260323)

066da48

chore: mark task 5.7 complete (mesh-io_20260323)

e1abe9a

chore: mark task 5.8 complete (mesh-io_20260323)

a069c58

chore: mark Phase 6 carry-forward tasks 5.7 and 5.8 complete (mesh-io…

44bc4fa

…_20260323) Tasks were completed in Phase 5 (cb5ee97, 6b9d7f5); carry-forward references in Phase 6 were not ticked at the time.

chore: mark Doxygen verification complete (mesh-io_20260323)

ab3e4fe

chore: mark track mesh-io_20260323 complete

a5ebb00

csparker247 changed the title ~~Harden mesh IO: stream validation, bounds checks, case-insensitive dispatch~~ Implement full MeshIO (OBJ/PLY) Apr 23, 2026

claude: Harden

4cf40fc

csparker247 commented Apr 23, 2026

View reviewed changes

Copilot started work on behalf of csparker247 April 23, 2026 18:35 View session

Copilot AI and others added 2 commits April 23, 2026 15:01

Tweak README

3b00d82

csparker247 mentioned this pull request Apr 23, 2026

Remove floating-point to_numeric specializations #1

Open

		4. Provide standard Mesh IO (OBJ and PLY read/write) including UV maps, texture
		paths, and compile-time vertex trait detection (`has_normal`, `has_color`)

	throw std::invalid_argument("Inputs have mismatched dimensions");
	throw std::invalid_argument("Inputs must be 3-dimensional");

Conversation

csparker247 commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

MeshIO pipeline

Supporting additions

Hardening

Tests

Uh oh!

Uh oh!

csparker247 commented Mar 26, 2026

Uh oh!

Copilot AI commented Mar 26, 2026

Uh oh!

csparker247 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

csparker247 commented Mar 25, 2026 •

edited

Loading