tvm icon indicating copy to clipboard operation
tvm copied to clipboard

[Runtime] Compatibility with dmlc::Stream API changes

Open Lunderberg opened this issue 1 year ago • 4 comments

This commit updates TVM implementations of dmlc::Stream. With https://github.com/dmlc/dmlc-core/pull/686, this API now requires the Write method to return the number of bytes written. This change allows partial writes to be correctly handled.

Lunderberg avatar May 14 '24 16:05 Lunderberg

This PR is marked as a draft, until the upstream dmlc changes land. At that point, this PR will be updated to also include a version bump for 3rdparty/dmlc-core.

Lunderberg avatar May 14 '24 16:05 Lunderberg

Upstream dmlc-core changes have landed, and this PR updates the submodule version to include the changes.

Lunderberg avatar May 22 '24 13:05 Lunderberg

@tvm-bot rerun

tqchen avatar May 28 '24 13:05 tqchen

@tvm-bot rerun

(Rerunning to use CI fix from https://github.com/apache/tvm/pull/17040.)

Lunderberg avatar May 29 '24 16:05 Lunderberg