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.

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

feat: automatically adjust fee rate #999

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

blckngm
Copy link
Contributor

@blckngm blckngm commented Feb 23, 2023

and automatically reset submission txs when a block cannot be confirmed for an extended period of time.

Manually tested by altering ckb-miner parameters in kicker.

@Flouse Flouse requested review from jjyr and magicalne March 16, 2023 06:19
@Flouse Flouse marked this pull request as ready for review March 18, 2023 10:22
Copy link
Collaborator

@Flouse Flouse left a comment

Choose a reason for hiding this comment

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

Please do a rebase and try to pass the CI?

magicalne
magicalne previously approved these changes Mar 22, 2023
and automatically reset submission txs when a block cannot be confirmed
for an extended period of time.
@jjyr jjyr requested review from Flouse and magicalne March 24, 2023 02:38
@jjyr jjyr added this pull request to the merge queue Mar 24, 2023
Merged via the queue into godwokenrises:develop with commit 8a4c70a Mar 24, 2023
@@ -180,14 +181,25 @@ pub struct PscConfig {
pub submitted_limit: u64,
/// Minimum delay between blocks. Default is 8 seconds.
pub block_interval_secs: u64,
pub min_fee_rate: u64,
pub max_fee_rate: u64,
pub fee_rate_pid: Option<Pid<f64>>,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you have a tested config example for fee_rate_pid?
@sopium

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The sim program outputs PID params on exit, e.g.:

setpoint = 0.6
output_limit = 1.0
kp = -0.8
ki = -0.08
kd = 0.0
p_limit = 1.0
i_limit = 1.0
d_limit = 0.0
integral_term = -1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants