Distance Oracle
close #10 (closed)
Merge request reports
Activity
assigned to @tuxmain
added 8 commits
-
fa1291d4...e714b76c - 7 commits from branch
master
- 327b2ece - wip
-
fa1291d4...e714b76c - 7 commits from branch
mentioned in issue #10 (closed)
added 7 commits
-
4012f146...14ee9635 - 6 commits from branch
master
- 77efb022 - wip
-
4012f146...14ee9635 - 6 commits from branch
added 26 commits
-
2b9fc019...e063a6fd - 25 commits from branch
master
- e4509c08 - wip
-
2b9fc019...e063a6fd - 25 commits from branch
added 6 commits
-
e4509c08...29d2c0fd - 5 commits from branch
master
- a840fd8e - wip
-
e4509c08...29d2c0fd - 5 commits from branch
added 37 commits
-
d9acc281...5a3f0654 - 36 commits from branch
master
- 3d7ec21e - wip
-
d9acc281...5a3f0654 - 36 commits from branch
added 6 commits
-
8b5c107c...5d8ba754 - 5 commits from branch
master
- b04cc3f6 - wip
-
8b5c107c...5d8ba754 - 5 commits from branch
added 2 commits
added C-binary C-doc C-runtime RN-binary RN-runtime T-end2endtests labels
changed milestone to %runtime-600
requested review from @HugoTrentesaux
- Resolved by Pascal Engélibert
@c-geek pourrais-tu relire au moins la partie calcul de la distance ? (
distance-oracle/src/lib.rs
)Les tests end2end ne marchent pas car en mode manual seal (blocs créés sur demande via RPC) les extrinsics correspondant aux inhérents ne sont pas créés (même si les inhérents sont créés par les inherent providers). La seule parade que je vois est de créer l'extrinsic dans les tests.
Il faut aussi que j'ajoute des tests dans la palette, pour ça j'essaierai de copier ceux de la branche de cgeek.
Il faudrait enfin comparer les résultats avec ceux de Duniter v1. @c-geek as-tu un outil particulier pour ça ?
added 1 commit
- 75b06534 - fix(distance): add distance pallet to gtest, g1
added 1 commit
- 2dc0ffb8 - fix(distance): max_depth, compute min_certs_for_referee
- Resolved by Pascal Engélibert
- Resolved by Pascal Engélibert
- Resolved by Pascal Engélibert
Comment est généré le fichier
resources/metadata.scale
? Ce serait bien de l'écrire dans le fichierresources/README.md
.Car quand j'essaye de lancer
cargo cucumber -i 'distance*'
j'obtiens :✘ When alice requests distance evaluation for dave Step failed: ./cucumber-features/distance.feature:13:9 Captured output: Step panicked. Captured output: Metadata: Pallet Distance Call Distance has incompatible metadata
Je suppose que c'est à cause de ce fichier.
- Resolved by Pascal Engélibert
Notez que je veux bien pousser du code pour les points que j'ai soulevé, comment préférez-vous que je procède ? MR ?
- Resolved by Pascal Engélibert
@tuxmain : la variante
DistanceStatus::Pending
n'est pas exploitée actuellement, pourtant il y a bien des appels àIdentitiesDistanceStatus::<T, I>::remove
, notamment un suivi de<T as Config<I>>::Currency::slash_reserved
laissant penser que cette variante y était attendue.Est-ce le cas ? J'aurais bien vu l'instruction
IdentitiesDistanceStatus::<T, I>::insert(idty_index, DistanceStatus::Pending);
positionnée juste avant le retourOk(())
dansdo_evaluate_distance
.Edited by Cédric Moreau
added 9 commits
-
1f42850a...81f4a877 - 2 commits from branch
master
- 03d35e6a - feat: distance
- 635e558c - test(distance): WiP end2end test
- 8149734e - fix(distance): add distance pallet to gtest, g1
- e5142fec - fix(distance): max_depth, compute min_certs_for_referee
- a50ded0e - fixes & working end2end tests
- f9262fbe - integration tests, expiration
- b96b83dd - feat: distance
Toggle commit list-
1f42850a...81f4a877 - 2 commits from branch