Timestamp define: not be strongly coupled with the time in tendermint

[DaviRain-Su]

Summary

Problem Definition

ibc-rs/crates/ibc/src/timestamp.rs

Line 25 in da6653e

pub struct Timestamp {

Can the definition of timestamp here not be strongly coupled with the time in tendermint.

Proposal

Acceptance Criteria

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate milestone (priority) applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[hu55a1n1]

We previously used the chrono crate but moved away from it (see informalsystems/hermes#1639 and linked issues/PRs for more info). It would be helpful if you could describe your requirements/ideas so we can evaluate potential solutions.

[DaviRain-Su]

Thank you for your explanation, this is just my personal understanding, for the third party light client implementation has Time type in the ConsensusState module, I think the third party light-client implementation should not include the type defined in the tendermint. This is my understanding.

[dzmitry-lahoda]

So seems fix is cp paste wrapper. And make tendermint into from where needed. Right? It will fix ics20 types tendermint dep removed at least.

i am now 816000 bytes, against 819000 contract limit, so i am still ok.