Skip to content
This repository has been archived by the owner on Jun 6, 2023. It is now read-only.

Decouple test-vector types from the vm #1455

Open
ZenGround0 opened this issue Jun 15, 2021 · 0 comments
Open

Decouple test-vector types from the vm #1455

ZenGround0 opened this issue Jun 15, 2021 · 0 comments

Comments

@ZenGround0
Copy link
Contributor

Currently the specs-actors reference vm implements its own types for generating and serializing test-vectors. We did this to get around the circular dependency (on lotus) that would be introduced by using the test-vectors repo types directly. These types and their methods make certain assumptions that are not good for general purpose use (i.e. https://github.com/filecoin-project/specs-actors/pull/1432/files#r647942788).

This is not ideal but ok as long as the actors types are conceptually coupled to the vm and not being used externally. However we are hoping to take on the responsibility of maintaining the format itself and there will be an issue describing the work around this soon. If that work materializes as expected the actors vm vector types will be strictly redundant and we should use the generally useable test-vectors types.

This issue tracks the work for moving the specs-actors reference vm from its own internal test vectors types to out-of-package general use types.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant