✨ Use PLMC amount in Evaluation - #484
Closed
lrazovic wants to merge 1 commit into
Closed
Conversation
Member
Author
This was referenced May 8, 2025
lrazovic
changed the base branch from
05-08-feat_macros_update_and_documentation
to
graphite-base/484
May 9, 2025 15:11
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 9, 2025 15:11
a1105c5 to
3fe6433
Compare
lrazovic
changed the base branch from
graphite-base/484
to
05-09-feat_simplify_the_acceptable_assets
May 9, 2025 15:11
lrazovic
force-pushed
the
05-09-feat_simplify_the_acceptable_assets
branch
from
May 9, 2025 15:42
359ca8a to
15d088b
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
2 times, most recently
from
May 9, 2025 15:44
8520fc9 to
27f9807
Compare
lrazovic
force-pushed
the
05-09-feat_simplify_the_acceptable_assets
branch
from
May 9, 2025 15:44
15d088b to
67962d0
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 9, 2025 16:52
27f9807 to
fd6baea
Compare
lrazovic
changed the base branch from
05-09-feat_simplify_the_acceptable_assets
to
graphite-base/484
May 9, 2025 20:20
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 9, 2025 20:20
fd6baea to
01f9671
Compare
lrazovic
changed the base branch from
graphite-base/484
to
05-09-chore_remove_useless_clone
May 9, 2025 20:20
Merged
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 9, 2025 20:33
01f9671 to
324253e
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 12, 2025 08:47
a514ee5 to
1e5802c
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 12, 2025 08:47
324253e to
8f88296
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 12, 2025 08:49
1e5802c to
a4afbb3
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
2 times, most recently
from
May 12, 2025 08:58
20c73fc to
67b7d3f
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 12, 2025 08:58
a4afbb3 to
f6c04d2
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
2 times, most recently
from
May 12, 2025 09:46
33189e7 to
94a27e5
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 12, 2025 11:36
f6c04d2 to
7e9c5ce
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
2 times, most recently
from
May 12, 2025 11:43
aa724ab to
fce9db1
Compare
This was referenced May 12, 2025
lrazovic
marked this pull request as ready for review
May 16, 2025 14:44
dastansam
approved these changes
May 19, 2025
dastansam
left a comment
Contributor
There was a problem hiding this comment.
looks ok, but no integration tests (?)
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 19, 2025 06:15
fce9db1 to
f9cceec
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
2 times, most recently
from
May 19, 2025 06:48
8b86a93 to
b244067
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 19, 2025 06:48
f9cceec to
2ffd8f6
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 20, 2025 06:21
b244067 to
e9c508f
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 20, 2025 06:21
2ffd8f6 to
0c2cd35
Compare
dastansam
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 23, 2025 11:43
0c2cd35 to
8c809b8
Compare
dastansam
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
May 23, 2025 11:43
e9c508f to
1e2f49d
Compare
Closed
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 28, 2025 13:35
8c809b8 to
7714174
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
2 times, most recently
from
May 28, 2025 13:38
093bd7c to
bda4556
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
May 28, 2025 13:38
7714174 to
909a092
Compare
dastansam
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
July 9, 2025 06:38
909a092 to
ac45b29
Compare
dastansam
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
July 9, 2025 06:38
bda4556 to
18ad867
Compare
lrazovic
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
July 25, 2025 14:19
18ad867 to
f6c2185
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
July 25, 2025 14:19
ac45b29 to
5559689
Compare
dastansam
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
July 25, 2025 21:21
5559689 to
ac45b29
Compare
dastansam
force-pushed
the
05-09-chore_remove_useless_clone
branch
from
July 25, 2025 21:21
f6c2185 to
18ad867
Compare
lrazovic
changed the base branch from
05-09-chore_remove_useless_clone
to
graphite-base/484
August 4, 2025 07:56
lrazovic
force-pushed
the
graphite-base/484
branch
from
August 4, 2025 07:59
18ad867 to
25a8367
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
August 4, 2025 07:59
ac45b29 to
7ba84ad
Compare
lrazovic
force-pushed
the
05-08-draft_use_plmc_amount_in_evaluation
branch
from
August 4, 2025 07:59
7ba84ad to
8f206da
Compare
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
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.

This pull request refactors the evaluation logic in the
fundingpallet, transitioning from using USD-based calculations to PLMC-based calculations in the evaluation round. The changes streamline the handling of evaluation bonds, improve consistency, and update related tests and utilities to align with the new approach.Core Refactoring: Transition to PLMC-based Calculations
usd_amountwithplmc_bondin thedo_evaluatefunction and related methods, ensuring all evaluation logic now operates using PLMC values. (pallets/funding/src/functions/2_evaluation.rs: [1] [2] [3]EvaluationParamsstruct to useplmc_amountinstead ofusd_amount, reflecting the shift to PLMC-based evaluations. (pallets/funding/src/instantiator/types.rs: pallets/funding/src/instantiator/types.rsL100-R126)Utility and Helper Updates
generate_evaluations_from_total_usdtogenerate_evaluations_from_total_plmc, ensuring compatibility with PLMC-based logic. (pallets/funding/src/instantiator/calculations.rs: pallets/funding/src/instantiator/calculations.rsL356-R393)pallets/funding/src/instantiator/calculations.rs: pallets/funding/src/instantiator/calculations.rsL33-R33)Tests and Mock Adjustments
pallets/funding/src/tests/2_evaluation.rs: [1] [2]HOURSfrom the mock configuration to clean up the codebase. (pallets/funding/src/mock.rs: pallets/funding/src/mock.rsL282-L283)Miscellaneous Improvements
BlockProviderFor<T>as an alias for the block number provider, improving readability and consistency. (pallets/funding/src/lib.rs: pallets/funding/src/lib.rsR139)iter::zip, to streamline the code. (pallets/funding/src/instantiator/mod.rs: pallets/funding/src/instantiator/mod.rsL21-R21)