cnumr / cnumr/GreenIT-Analysis
Question sur measures.responsesSize
- Dominant language
- JavaScript
- Stars
- 161
- Forks
- 38
- PR merge metrics
- No merged PRs in 30d
Description
Bonjour,
J'essaye tant bien que mal de reproduire le calcul, de mon côté manuellement, et je crois comprendre que measures.responsesSize provient d'une addition des entry.response._transferSize puis au final measures.responsesSize est converti en Ko grâce à un Math.round(measures.responsesSize / 1000)
Ma question, c'est, est-ce qu'il n'y aurait pas une erreur, ou est-ce que je suis mal documenté/j'ai raté une ligne, mais selon moi, entry.response._transferSize est en bytes, alors que derrière, on considère cela comme étant en octet, ne devrait-on pas convertir _transferSize en octet avant de le convertir en Ko puis de l'afficher / l'utiliser pour calculer l'ecoindex ?
Note:
1 bit = 0,125 octet
Si ce que je dis est correcte, le correctif serait alors:
measures.responsesSize += entry.response._transferSize / 8;
Contributor guide
No contributing guide indexed for this repository
Research direction
Search the JavaScript code for measures.responsesSize and entry.response._transferSize, then verify the units used in the accumulation and the later Math.round conversion. Confirm whether the proposed conversion is correct and check any existing calculation or regression tests; done means the measurement and EcoIndex input use the intended unit.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- performance
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100