-
Notifications
You must be signed in to change notification settings - Fork 24
[update] v0.0.45-50 #96
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 44 commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
4c271c8
update versions
patrick-ogrady 187c6dc
progress
patrick-ogrady f2a37f2
cleanup block setup
patrick-ogrady 103591c
progress
patrick-ogrady acf1735
cleanup testing in lib
patrick-ogrady b92d62b
wasm parse seed
patrick-ogrady e6f79b2
cleanup wasm
patrick-ogrady 1ac06d2
progress on client
patrick-ogrady 746d35b
update client
patrick-ogrady 13f1c08
implement reporter
patrick-ogrady 44cb98a
remove reporter from application
patrick-ogrady b1b9176
progress
patrick-ogrady 5738779
more progress
patrick-ogrady 899d02f
fix seed call
patrick-ogrady 8772410
progress
patrick-ogrady 2514e1a
cleanup client interface
patrick-ogrady 7b21928
syncer compiles
patrick-ogrady c062399
engine compiles
patrick-ogrady cf0d7c3
lib.rs compiles
patrick-ogrady 73c71aa
setup.rs compiles
patrick-ogrady 3900113
validator compiles
patrick-ogrady c3f9eab
incorporate buffered
patrick-ogrady 3eff59a
fix inspector compilation
patrick-ogrady 66bd51d
unify api
patrick-ogrady 01009b4
better integrate buffer
patrick-ogrady adf726e
don't operate over bytes
patrick-ogrady bfe376d
cleanup refresh
patrick-ogrady 5ce082c
use deployer const for metrics
patrick-ogrady e1817b5
add README deque size
patrick-ogrady eb04295
use rwlock with archive
patrick-ogrady 33103cb
use rwmutex for view tracking
patrick-ogrady b1cc99a
break last view to force us to fix
patrick-ogrady 066aa29
use AtomicU64
patrick-ogrady c905a08
progress
patrick-ogrady 456a22c
Refactor finalizer (#97)
patrick-ogrady cd61cd5
add more
patrick-ogrady c3c26a5
0.0.47 update (#98)
patrick-ogrady 9324392
add order comment
patrick-ogrady 6121b72
use proper digest decode
patrick-ogrady d777971
nit comment
patrick-ogrady 65f84fb
remove buffered cloning
patrick-ogrady 5aa54c1
remove useless clone
patrick-ogrady 3457c73
nit
patrick-ogrady b3cf7c2
[chain] Cleanup `Syncer` enum (#102)
patrick-ogrady f9ea94a
update dockerfile
patrick-ogrady 99bf10d
remove link
patrick-ogrady 8b33457
use hosts
patrick-ogrady 9728344
deployer default features off
patrick-ogrady 33f911b
update versions
patrick-ogrady 0841b73
add v4 feature
patrick-ogrady 820a3e2
update dashboard
patrick-ogrady d91dc75
catch panics
patrick-ogrady 326a5fa
[chain] Run Local Network (#103)
patrick-ogrady af9d7ff
fix validator init
patrick-ogrady 957fa70
update block
patrick-ogrady c5de223
syncer compiles
patrick-ogrady bc818e7
tests passing
patrick-ogrady 46e3426
add local metrics
patrick-ogrady 9410a56
fix readme
patrick-ogrady 0bb3408
fix paths
patrick-ogrady 71f19af
actually run consensus tests
patrick-ogrady 6eb43c1
update versions
patrick-ogrady 0d0f724
add pending writes
patrick-ogrady 4cf6e79
don't allow pending writes
patrick-ogrady 34fe8ba
add pruning log
patrick-ogrady 5516283
add tip multiplier
patrick-ogrady File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| use commonware_consensus::{ | ||
| threshold_simplex::{Context, View}, | ||
| Automaton as Au, Committer as Co, Proof, Relay as Re, | ||
| threshold_simplex::types::{Context, View}, | ||
| Automaton, Relay, | ||
| }; | ||
| use commonware_cryptography::sha256::Digest; | ||
| use futures::{ | ||
|
|
@@ -26,14 +26,6 @@ pub enum Message { | |
| payload: Digest, | ||
| response: oneshot::Sender<bool>, | ||
| }, | ||
| Prepared { | ||
| proof: Proof, | ||
| payload: Digest, | ||
| }, | ||
| Finalized { | ||
| proof: Proof, | ||
| payload: Digest, | ||
| }, | ||
| } | ||
|
|
||
| /// Mailbox for the application. | ||
|
|
@@ -48,7 +40,7 @@ impl Mailbox { | |
| } | ||
| } | ||
|
|
||
| impl Au for Mailbox { | ||
| impl Automaton for Mailbox { | ||
| type Digest = Digest; | ||
| type Context = Context<Self::Digest>; | ||
|
|
||
|
|
@@ -97,7 +89,7 @@ impl Au for Mailbox { | |
| } | ||
| } | ||
|
|
||
| impl Re for Mailbox { | ||
| impl Relay for Mailbox { | ||
| type Digest = Digest; | ||
|
|
||
| async fn broadcast(&mut self, digest: Self::Digest) { | ||
|
|
@@ -107,21 +99,3 @@ impl Re for Mailbox { | |
| .expect("Failed to send broadcast"); | ||
| } | ||
| } | ||
|
|
||
| impl Co for Mailbox { | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is now handled by the new |
||
| type Digest = Digest; | ||
|
|
||
| async fn prepared(&mut self, proof: Proof, payload: Self::Digest) { | ||
| self.sender | ||
| .send(Message::Prepared { proof, payload }) | ||
| .await | ||
| .expect("Failed to send notarized"); | ||
| } | ||
|
|
||
| async fn finalized(&mut self, proof: Proof, payload: Self::Digest) { | ||
| self.sender | ||
| .send(Message::Finalized { proof, payload }) | ||
| .await | ||
| .expect("Failed to send finalized"); | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now handled by the integrated telemetry stuff.