Revert "Collector Fixes"

[patryk-przybysz]

Reverts #34

[Copilot]

Pull request overview

This PR reverts the prior “Collector Fixes” change set (#34), restoring earlier collector behavior for RAPL (CPU package power) and NVML (GPU metrics) and simplifying timestamp generation in several collectors.

Changes:

• Replaces the RAPL collector’s sysfs powercap-based implementation with an MSR-based implementation.
• Refactors the NVML collector from multi-GPU enumeration to a single GPU (index 0) handle/id.
• Changes several collectors’ timestamps from explicit duration_cast to time_since_epoch().count().

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
sources/agent/src/collectors/rapl_collector.h	Removes prior zone-based powercap state; introduces MSR constants/state.
sources/agent/src/collectors/rapl_collector.cc	Replaces powercap discovery/sampling with MSR reads for RAPL energy.
sources/agent/src/collectors/ram_collector.cc	Simplifies timestamp creation in RAM metrics.
sources/agent/src/collectors/proc_stat_collector.cc	Simplifies timestamp creation in CPU utilization metrics.
sources/agent/src/collectors/nvml_collector.h	Switches from per-device storage to single-handle/single-id fields.
sources/agent/src/collectors/nvml_collector.cc	Switches from multi-GPU enumeration/loop to only GPU 0 sampling; simplifies timestamp creation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[varev-dev]

lgtm