Zome/trust graph #28
Closed
Zome/trust graph #28
Conversation
This file contains 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
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 13, 2022
Comment on lines
38
to
39
| let rollup = trust_atom::create( | ||
| trust_atom.target_entry_hash, trust_atom.label, None, algo, trust_atoms_collection |
There was a problem hiding this comment.
In my reading of this we only create one TrustAtom, regardless of how many targets there are, of my atoms, and my (agent)atoms' atoms.
What we want is to end up with one rollup TrustAtom for every one of the targets of my atoms, and the targets of my (agent)atoms' atoms.
I added to the example in #25 to try to clarify:
eg : given my TAs:
me -[holochain, 0.99]-> zippy
me -[holochain, 0.80]-> robert
me -[holochain, 0.99]-> HIA
me -[engineering, 0.88]-> Telos
zippy -[holochain, 0.99]-> HIA
robert -[holochain, -0.99]-> HIA
robert -[engineering, 0.99]-> Ethereum
then rollup atoms will be:
me -[rollup, holochain, 0.98]-> HIA // actual value is TBD
me -[rollup, engineering, 0.99]-> Ethereum // actual value is TBD
me -[rollup, engineering, 0.88]-> Telos // actual value is TBD
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 13, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 15, 2022
harlantwood
reviewed
Mar 22, 2022
zomes/hc_zome_trust_atom/src/lib.rs
Outdated
| pub mod trust_graph; | ||
| pub use crate::trust_graph::*; | ||
| pub(crate) mod utils; | ||
| pub(crate) mod test_helpers; |
There was a problem hiding this comment.
can sweetest still use this?
There was a problem hiding this comment.
tried using pub(super) got error: "too many leading super keywords"
going to leave it public for now
harlantwood
reviewed
Mar 22, 2022
zomes/hc_zome_trust_atom/src/lib.rs
Outdated
| pub content_full: Option<String>, | ||
| pub content_starts_with: Option<String>, | ||
| pub value_starts_with: Option<String>, | ||
| } | ||
|
|
||
| // ZOME API FUNCTIONS | ||
|
|
||
| #[hdk_extern] | ||
| pub fn create_rollup(filter: Linktag) -> ExternResult<Vec<TrustAtom>> { | ||
|
|
harlantwood
reviewed
Mar 22, 2022
Comment on lines
26
to
27
| pub source_entry_hash: EntryHash, | ||
| pub target_entry_hash: EntryHash, |
harlantwood
reviewed
Mar 22, 2022
trust graph algorithm, rollup tests one zome, trust_graph, rollup, agents Notes Notes alt strategy for agent, slver & gold rollup method alt strategy for agent, slver & gold rollup method alt strategy for agent, slver & gold rollup method Comment bin/run (cargo-release) version 0.0.6 (cargo-release) start next development iteration 0.0.7-dev Docs on latest version Remove HC patches, no longer needed Upgrade to HDK 125 / HC 130 Remove hc version from holonix (cargo-release) version 0.0.7 (cargo-release) start next development iteration 0.0.8-dev Iterate
harlantwood
force-pushed
the
zome/trust-graph
branch
from
83bac9a to
69996b9
Compare
harlantwood
reviewed
Mar 27, 2022
|
|
||
| for agent in agents { | ||
| let agent = EntryHash::from(agent); | ||
| let links = get_links(agent.clone(), Some(LinkTag(filter)))?; |
There was a problem hiding this comment.
this should find TAs only with prefix rollup, right?
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
#24 #25
fixes #29 by using fn get_latest