regolith-labs / ore-cli

Command line interface for ORE cryptocurrency mining.
919 stars 443 forks source link

added fee estimation if RPC supports it #57

Open pennicoat opened 2 months ago

pennicoat commented 2 months ago

adds --estimate-fees and --priority-level <min/low/medium/high/very-high/unsafe-max/default>

takes the fee estimation stuff from #44 and removes the Helius requirement, though I think Helius is the only RPC that supports it

Milo123459 commented 2 months ago

just a thought, could it be good to dynamically adjust the priority fee? perhaps fetching an estimate every X minutes (if a transaction is taking a long time maybe)

pennicoat commented 2 months ago

just a thought, could it be good to dynamically adjust the priority fee? perhaps fetching an estimate every X minutes (if a transaction is taking a long time maybe)

That adds quite a bit of complexity to this solution. Could be useful in the future when the network isn’t congested, but right now I don’t think many people are landing transactions without priority fees.

Milo123459 commented 2 months ago

I think would be worth it in the long run, and would make ore mining more cost efficient