Power Profiler current measurements showing 0 on 100ms scale.
markb-EE opened this issue · comments
I was unable to reproduce this. If this was also done with 4.0.0-beta1: Could you first check whether this problem still exists with the latest version? Does it only happen at 100ms, not above or below? Could you export a run to a .ppk file and share it?
Average calculation is still incorrect in version v.4.0.0-beta3. The average current cannot be greater than the max current.
ppk-20231229T142044.zip
Ah, yes, I am now able to reproduce this with your PPK file by navigating to one of the peaks and then zooming in. Seems the further I zoom in, the further the values are off.
Also interesting that in your image the location in the minimap is wrong, since it shows you to be left of the peak.
Thanks for the PPK file!
@kylebonnici: Do you have an idea what causes this?
![image](https://private-user-images.githubusercontent.com/260705/293930407-3210e72c-a53f-4a46-929d-f700df2e8cf5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE4ODUxMzcsIm5iZiI6MTcyMTg4NDgzNywicGF0aCI6Ii8yNjA3MDUvMjkzOTMwNDA3LTMyMTBlNzJjLWE1M2YtNGE0Ni05MjlkLWY3MDBkZjJlOGNmNS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzI1JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcyNVQwNTIwMzdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0yOThjM2IwODYwNjY1NGY4ODI0NTJhODg4ZjIzNGY0OGI5ZjI4YWIyMzYzOGNlODIyMmFkYmUxMWNlYmEwYWJlJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.MNkINgfqtQF4mrzP7dXXoyQq4_3zcaKoQ0l9BhjVrsI)
Issue has been found and a PR will follow soon the issue is that the average is off by a factor of 1000 as we have change the unit in the chart and omit changing unit in the Window average box
Fixed by: #378
Will look at the other issue where values are zeros
UPDATE:
The issue where the window values are zeros is fixed by #379
Hi @markb-EE and @KonstKonovalov version 4.0.0-beta4 is out and should address these issues