Skip to content
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.

Sign up for GitHub

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

fix hard coded return_per_1000 #1015

Open
wants to merge 12 commits into
base: devnet-ready
Choose a base branch
from
+2 −1
Open

fix hard coded return_per_1000 #1015

Changes from 10 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
7b9b3e0
fix hard coded return_per_1000
open-junius Nov 25, 2024
65c0b24
Update chainspecs
open-junius Nov 25, 2024
66f4e42
fix clippy
open-junius Nov 25, 2024
fb17cbf
Update chainspecs
open-junius Nov 25, 2024
5251506
merge devnet ready
open-junius Nov 27, 2024
a59fb30
regenerate spec local
open-junius Nov 27, 2024
6755853
Update chainspecs
open-junius Nov 27, 2024
3fdccee
regenerate spec
open-junius Nov 27, 2024
c2ad312
trigger spec check
open-junius Nov 27, 2024
42e19dd
merge target branch
open-junius Dec 5, 2024
8710131
Merge branch 'devnet-ready' into fix-return_per_1000
open-junius Dec 6, 2024
fcd3220
use into for type convert
open-junius Dec 6, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion pallets/subtensor/src/rpc_info/delegate_info.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,8 @@ impl<T: Config> Pallet<T> {

let return_per_1000: U64F64 = if total_stake > U64F64::from_num(0) {
emissions_per_day
.saturating_mul(U64F64::from_num(0.82))
.saturating_mul(U64F64::from_num(u16::MAX.saturating_sub(take.0)))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use .into() here to be more idiomatic?

.saturating_div(U64F64::from_num(u16::MAX))
.saturating_div(total_stake.saturating_div(U64F64::from_num(1000)))
} else {
U64F64::from_num(0)
Expand Down
Loading