😶🌫️ Mystery Factor in Gas Estimation Script
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- typescript
- Área
- blockchain, documentation
Línea de trabajo
Start with packages/gas-estimation/scripts/exec.ts, especially the gas formula documentation around lines 29–40 and the l1EstimatedPrice calculation near line 92. Trace the source and units of the factor of 16 before choosing a descriptive constant. Done means the constant and its purpose are documented in gasEstimator() and the tutorial remains understandable.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
📄 Summary
following refers to
arbitrum-tutorials/packages/gas-estimation/scripts/exec.ts
After a thorough review of the gas-estimation script tutorial, including an analysis of the commit history and an organization-wide search, I have been unable to locate a clear explanation for why the following variable is multiplied by a factor of 16:
const l1EstimatedPrice = gasEstimateComponents.l1BaseFeeEstimate.mul(16);
I have examined the detailed documentation (👏) related to the gas formula and the section titled How do we get all parts of that equation. However, the documentation repeatedly states, "and multiply by 16" without providing a precise explanation for this multiplication.
🔎 Interpretation
This factor likely pertains to unit conversion, such as converting from units of size to units of gas. However, neither the tutorial nor the repository offers a clear and specific explanation for this multiplication.
💊 Remedy
To improve clarity, I suggest the following steps:
- Define a constant with a descriptive name for this factor.
- Provide sufficient documentation explaining the purpose and usage of the new constant.
- Update the detailed documentation within
gasEstimator()to reference the factor of 16 and clearly explain its purpose.
- Lenguaje dominante
- JavaScript
- Estrellas
- 508
- Forks
- 314
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de OffchainLabs/arbitrum-tutorials
-
destAddress in the eth-deposit-to-different-address could be an argument to pass to the script instead of being hardcodedPosiblemente ocupada @mahsamoosavi la tomó hace 997 días. Abierto
OffchainLabs/arbitrum-tutorials#115 · 1 asignado ·
-
add typescript config and convert js scripts to typescriptQuizá libre de nuevo @mahsamoosavi la tomó hace 1403 días y no hay ningún pull request abierto. Abiertoenhancement
OffchainLabs/arbitrum-tutorials#88 · 1 comentario · 1 asignado ·
Todos los issues de OffchainLabs/arbitrum-tutorials
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
solana-foundation/solana-web3.js#4015 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
gramps-project/gramps-web#1604 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue hacktoberfest help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
BinaryBaconLabs/todo-app#18 ·
-
Cache airport lookupsAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
sutantyo/flight-watch#32 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día