Moving GMT to microjoules #932
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Eco-CI Output:
❌ CO2 Data: |
…esolution to phase_stats
…Reworked transformation of values and units
…ergy; added new tests for SR
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
47 file(s) reviewed, 39 comment(s)
Edit PR Review Bot Settings | Greptile
There was a problem hiding this comment.
45 file(s) reviewed, 15 comment(s)
Edit PR Review Bot Settings | Greptile
There was a problem hiding this comment.
11 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings | Greptile
There was a problem hiding this comment.
LGTM
4 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
ArneTR
added a commit
that referenced
this pull request
Jan 6, 2025
* main: (58 commits) (Data): Added more metric mappings for nice display in frontend (Fix): Made functions that return data non-async and functions that operate independently async Added phase stats uJ migration Updated Cloud Energy Deactivated test for now Moving GMT to microjoules (#932) Measurement table split (#1039) (fix): end measurement can appear twice in notes, which is superflous No need for strict null check Bump orjson from 3.10.12 to 3.10.13 (#1037) Bump anybadge from 1.14.0 to 1.15.6 (#1038) Metric importer refactor (#1035) (improvement): Added ICMP, Ehterframe, QUIC and TCP-Streams to TCPDump [skip ci] (#1036) Removed the email column from the runs table (change): Changed default resolution of powermetrics to a non-underflow resolution Bump uvicorn-worker from 0.2.0 to 0.3.0 (#1034) (fix): lmsensors was not reacting to skip_check (Tests): pytest running as default with -vv (fix): Tests. Locator adapted to new default table sorting of CI repos (style): Better error message ...
ArneTR
added a commit
that referenced
this pull request
Jan 7, 2025
* main: (58 commits) (Data): Added more metric mappings for nice display in frontend (Fix): Made functions that return data non-async and functions that operate independently async Added phase stats uJ migration Updated Cloud Energy Deactivated test for now Moving GMT to microjoules (#932) Measurement table split (#1039) (fix): end measurement can appear twice in notes, which is superflous No need for strict null check Bump orjson from 3.10.12 to 3.10.13 (#1037) Bump anybadge from 1.14.0 to 1.15.6 (#1038) Metric importer refactor (#1035) (improvement): Added ICMP, Ehterframe, QUIC and TCP-Streams to TCPDump [skip ci] (#1036) Removed the email column from the runs table (change): Changed default resolution of powermetrics to a non-underflow resolution Bump uvicorn-worker from 0.2.0 to 0.3.0 (#1034) (fix): lmsensors was not reacting to skip_check (Tests): pytest running as default with -vv (fix): Tests. Locator adapted to new default table sorting of CI repos (style): Better error message ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@greptileai
Greptile Summary
This PR implements a significant database schema change and standardizes energy measurements to use microjoules (uJ) across the Green Metrics Tool.
/tests/lib/test_phase_stats.pywith sampling rate validationMICROJOULES_TO_KWHin/tests/lib/test_phase_stats.py/tests/data/metrics/network_io_procfs_system.logfor network I/O metrics validation/tests/metric_providers/test_metric_providers.pyfor improved precision/migrations/2023_07_10_indices.sql