Skip to content

Update public/content/developers/docs/gas/index.md#14934

Closed
AvulaRamSwaroop wants to merge 1 commit into
ethereum:devfrom
AvulaRamSwaroop:patch-2
Closed

Update public/content/developers/docs/gas/index.md#14934
AvulaRamSwaroop wants to merge 1 commit into
ethereum:devfrom
AvulaRamSwaroop:patch-2

Conversation

@AvulaRamSwaroop
Copy link
Copy Markdown
Contributor

At line 102:
Incorrect Formula For refund :
max fee - (base fee + tip)

Corrected Formula For refund:
(gas limit−gas used)×(maxfee−(base fee+tip))

Reason:
1.max fee - (base fee + tip) gives the per-unit refund, but the
actual refund depends on how much gas was unused.
2.gas unused = gas limit- gas used .

At line  102:
  Incorrect Formula For refund :
   max fee - (base fee + tip)

  Corrected Formula For refund: 
    (gas limit−gas used)×(maxfee−(base fee+tip))

  Reason: 
   1.max fee - (base fee + tip) gives the per-unit refund, but the 
    actual refund depends on how much gas was unused.
   2.gas unused = gas limit- gas used .
@github-actions github-actions Bot added the content 🖋️ This involves copy additions or edits label Feb 18, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 18, 2025

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 7adb59b
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/67b4e1c1c3d52d0008eb9622
😎 Deploy Preview https://deploy-preview-14934--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 46 (🔴 down 7 from production)
Accessibility: 95 (no change from production)
Best Practices: 89 (🔴 down 7 from production)
SEO: 98 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@wackerow wackerow changed the title Update index.md Update public/content/developers/docs/gas/index.md Feb 18, 2025
@wackerow
Copy link
Copy Markdown
Member

Hey @AvulaRamSwaroop, thanks for the suggestion.. I see where you're coming from, just not sure this is really necessary? I think the calculation that is there is simple enough to read (in "per-unit" terms) and it gets the point across that the excess is returned. Feel free to push back; small change so not a hill I would die on

@AvulaRamSwaroop
Copy link
Copy Markdown
Contributor Author

AvulaRamSwaroop commented Feb 19, 2025

Hi ,As a beginner I felt little confused for a While cause the line is stated as :

any gas not used in a transaction is returned to the user (i.e. max fee - (base fee + tip) is returned).

So I later came to known that ,this must be multiplied by gas unused units .

if it is per-unit terms then no need to modify it .

Thanks for your Time.

@AvulaRamSwaroop
Copy link
Copy Markdown
Contributor Author

Hi @wackerow.Can I close this Issue ?

@AvulaRamSwaroop
Copy link
Copy Markdown
Contributor Author

HI there. anyone @corwintines @wackerow @pettinarip @minimalsm Can I close this One?

@ethereum ethereum deleted a comment from Afeter8 Mar 25, 2025
@ethereum ethereum deleted a comment from Afeter8 Mar 25, 2025
@github-actions
Copy link
Copy Markdown
Contributor

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions Bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Apr 25, 2025
@wackerow
Copy link
Copy Markdown
Member

wackerow commented Oct 6, 2025

Hey @AvulaRamSwaroop, apologies for the delays... Yes, these are in per-unit terms. Going to close this out for now but appreciate the idea. If you'd like, feel free to open a separate suggestion to improve wording if you feel it's necessary ❤️

@wackerow wackerow closed this Oct 6, 2025
@github-actions github-actions Bot added the abandoned This has been abandoned or will not be implemented label Oct 6, 2025
@wackerow
Copy link
Copy Markdown
Member

wackerow commented Oct 6, 2025

@all-contributors please add @AvulaRamSwaroop for ideas

@allcontributors
Copy link
Copy Markdown
Contributor

@wackerow

I've put up a pull request to add @AvulaRamSwaroop! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

abandoned This has been abandoned or will not be implemented content 🖋️ This involves copy additions or edits Status: Stale This issue is stale because it has been open 30 days with no activity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants