Closed ajhollid closed 4 days ago
The pull request modifies the InfrastructureDetails
component in index.jsx
, primarily focusing on CPU core calculations and temperature data handling. The initialization of numCores
is set to 1
, enhancing the baseline for calculations. Additional conditions are added to manage CPU temperature data, ensuring defaults are set when data is missing. The buildTemps
function is adjusted to consistently return structured data. Furthermore, checks are implemented to prevent rendering charts without data, improving the component's robustness against empty datasets.
File Path | Change Summary |
---|---|
Client/src/Pages/Infrastructure/Details/... | - Adjusted numCores initialization from 0 to 1 . - Expanded checks for CPU temperature data. - Modified buildTemps function to return consistent structure. - Updated logic for maximum y-domain in temperature chart. - Added checks to prevent rendering area charts with no data. |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Metrics that are missing are dispalyed as a zero value. In the previous implementation temperature was removed entirely if it was missing, this PR updates that to replace empty data with a zero value to be consistent with the other metrics