libplanet

libplanet copied to clipboard

Streamline and simplify `Swarm<T>` startup process

1

Current scheme involves the following steps: - `Swarm()` for instantiation. - `Swarm.BootstrapAsync()` for connecting to network. - `Swarm.PreloadAsync()` for syncing blocks before normal operations. - `Swarm.StartAsync()` for starting normal operations....

greymistcube

Provide a sensible way to inject `IBlockPolicy<T>` to `Agent`

1

Using [Planet Clicker] as a reference, if we were to provide a simpler interface such as `LibplanetUnity.Agent`, additional options should be easily accessible by game developers. I would assume game...

greymistcube

Ignore known expired `Transaction<T>`s.

1

Currently, `VolatileStagePolicy` only ignores `Transaction` that does not follow `IBlockPolicy`. However, "ignoring" to mine a `Transaction` that has an "expired" timestamp according to `IStagePolicy`, which isn't part of a consensus...

greymistcube

Define a better `Message` protocol

1

Still learning more about messaging patterns and discovering mounds of complexities required for a reliable [publisher-subscriber] pattern implementation. Seems like the harder option mentioned in [this comment] is much harder...

greymistcube

Add analogous models to `DataModel` for `IAccountStateDelta.GetState()` and `IAccountStateDelta.SetState()`

1

Something like ```csharp StateModel state = states.GetState(address); // Do stuff. states.SetState(address, state.Encode()); ``` might be nice to have. 🙄 This is more complicated than `DataModel` as `Bencodex` parameter type for...

greymistcube

Add compression to `DataModel`

1

If human readability of encoded data structure isn't much of a concern, it is possible to encode a `DataModel` instance to a `Bencodex.Types.List` instead of `Bencodex.Types.Dictionary`. With some consideration, it...

greymistcube

Prevent public setter for `DataModel` properties

1

As a safety caution, it might be a good idea to not allow public setter for properties of `DataModel`.

greymistcube

Ignore static properties for `DataModel`

1

I only gave this a brief thought. I think there are pros and cons either way. 🙄

greymistcube

Add `null` support to `DataModel`

1

Initial implementation of `DataModel` in #1935 is missing `null` support. Depending on how game logic and its accompanying data model is designed, this might be useful. From a technical standpoint,...

greymistcube

Implement attributes to ignore certain properties for `DataModel`

1

Currently as implemented in #1935, `DataModel.Encod()` encodes **all public properties**. This may not be desirable in all cases and probably hampers with data modeling where one might want a public...

greymistcube