cnumr / cnumr/GreenIT-Analysis

Question sur measures.responsesSize

Open
#73 3 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.