viiccwen opened a new issue, #1183: URL: https://github.com/apache/mahout/issues/1183
ref: #1058 ### What Increase coverage for `qdp/qdp-core/src/readers/torch.rs`. Current reported coverage for the file is still zero in the main llvm-cov run: - Function coverage: `0.00% (0/4)` - Line coverage: `0.00% (0/41)` ### Why We already have `qdp/qdp-core/tests/torch_io.rs`, but the current coverage still suggests that constructor and reader state transitions in `readers/torch.rs` are not being exercised in the main coverage workflow. This leaves basic reader behavior under-tested. ### How Add or adjust tests so that the main coverage run hits the reader implementation directly. Suggested coverage targets: - `TorchReader::new()` succeeds for an existing `.pt` file - `TorchReader::new()` fails for a missing file - `read_batch()` rejects a second read with `Reader already consumed` - `get_sample_size()` and `get_num_samples()` are asserted after a successful read - if the coverage job excludes the `pytorch` feature, add coverage for the non-feature `NotImplemented` path or document that gap clearly in the test approach This issue is done when `readers/torch.rs` shows real coverage improvement in `cargo llvm-cov`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
