Put DRAM=False as default

Author: benoit-cty

CONTRIBUTING.md

@@ -224,7 +224,7 @@ Then run opened test with this button:
 <!-- TOC --><a name="coding-style-linting"></a>
 ### Coding style && Linting
 
-The coding style and linting rules are automatically applied and enforced by [pre-commit](https://pre-commit.com/). This tool helps to maintain the same code style across the code-base such to ease the review and collaboration process. Once installed ([https://pre-commit.com/#installation](https://pre-commit.com/#installation)), you can install a Git hook to automatically run pre-commit (and all configured linters/auto-formatters) before doing a commit with `uv run precommit-install`. Then once you tried to commit, the linters/formatters will run automatically. It should display something similar to:
+The coding style and linting rules are automatically applied and enforced by [pre-commit](https://pre-commit.com/). This tool helps to maintain the same code style across the code-base such to ease the review and collaboration process. Once installed ([https://pre-commit.com/#installation](https://pre-commit.com/#installation)), you can install a Git hook to automatically run pre-commit (and all configured linters/auto-formatters) before doing a commit with `uv run task precommit-install`. Then once you tried to commit, the linters/formatters will run automatically. It should display something similar to:
 
 ```log
 [INFO] Initializing environment for https://github.com/psf/black.

codecarbon/core/cpu.py

@@ -367,14 +367,14 @@ class IntelRAPL:
         _rapl_files (List[RAPLFile]): A list of RAPLFile objects representing the files to read energy data from.
         _cpu_details (Dict): A dictionary storing the latest CPU energy details.
         _last_mesure (int): Placeholder for storing the last measurement time.
-        rapl_include_dram (bool): Whether to include DRAM power in measurements (default: True for complete hardware measurement).
+        rapl_include_dram (bool): Whether to include DRAM power in measurements (default: False for complete hardware measurement).
         rapl_prefer_psys (bool): Whether to prefer psys domain over package domains (default: False).
                                 When True, uses psys (platform/system) domain which includes CPU + platform components.
                                 When False (default), uses package domains which are more reliable and match CPU TDP specs.
 
     Args:
         rapl_dir (str): Path to RAPL directory (default: "/sys/class/powercap/intel-rapl/subsystem")
-        rapl_include_dram (bool): Include DRAM domain for complete hardware measurement (default: True).
+        rapl_include_dram (bool): Include DRAM domain for complete hardware measurement (default: False).
                                   Set to False to measure only CPU package power.
         rapl_prefer_psys (bool): Prefer psys (platform) domain over package domains (default: False).
                                 Set to True to measure total platform power (CPU + chipset + PCIe).
@@ -395,7 +395,7 @@ class IntelRAPL:
     def __init__(
         self,
         rapl_dir="/sys/class/powercap/intel-rapl/subsystem",
-        rapl_include_dram=True,
+        rapl_include_dram=False,
         rapl_prefer_psys=False,
     ):
         self._lin_rapl_dir = rapl_dir
@@ -427,8 +427,8 @@ def _fetch_rapl_files(self) -> None:
         """
         Fetches RAPL files from the RAPL directory.
 
-        By default, reads CPU package + DRAM domains for complete hardware power measurement.
-        Set rapl_include_dram=False to measure only CPU package power.
+        By default, reads CPU package only
+        Set rapl_include_dram=True to measure CPU package + DRAM domains
         """
         # We'll scan common powercap locations and look for domain directories
         # that expose an `energy_uj` file. We try to be tolerant to permission

codecarbon/emissions_tracker.py

@@ -293,7 +293,7 @@ def __init__(
         self._set_from_conf(pue, "pue", 1.0, float)
         self._set_from_conf(wue, "wue", 0, float)
         self._set_from_conf(force_mode_cpu_load, "force_mode_cpu_load", False, bool)
-        self._set_from_conf(rapl_include_dram, "rapl_include_dram", True, bool)
+        self._set_from_conf(rapl_include_dram, "rapl_include_dram", False, bool)
         self._set_from_conf(rapl_prefer_psys, "rapl_prefer_psys", False, bool)
         self._set_from_conf(
             experiment_id, "experiment_id", "5b0fa12a-3dd7-45bb-9766-cc326314d9f1"
@@ -1189,7 +1189,7 @@ def track_emissions(
     :param pue: PUE (Power Usage Effectiveness) of the datacenter.
     :param wue: WUE (Water Usage Effectiveness) of the datacenter, L/kWh.
     :param allow_multiple_runs: Prevent multiple instances of codecarbon running. Defaults to False.
-    :param rapl_include_dram: Include DRAM domain for RAPL measurements (default: True).
+    :param rapl_include_dram: Include DRAM domain for RAPL measurements (default: False).
     :param rapl_prefer_psys: Prefer psys (platform) domain over package domains for RAPL (default: False).
 
     :return: The decorated function

codecarbon/external/hardware.py

@@ -168,7 +168,7 @@ def __init__(
         tdp: int,
         rapl_dir: str = "/sys/class/powercap/intel-rapl/subsystem",
         tracking_mode: str = "machine",
-        rapl_include_dram: bool = True,
+        rapl_include_dram: bool = False,
         rapl_prefer_psys: bool = False,
     ):
         assert tracking_mode in ["machine", "process"]
@@ -337,7 +337,7 @@ def from_utils(
         model: Optional[str] = None,
         tdp: Optional[int] = None,
         tracking_mode: str = "machine",
-        rapl_include_dram: bool = True,
+        rapl_include_dram: bool = False,
         rapl_prefer_psys: bool = False,
     ) -> "CPU":
         if model is None:

docs/edit/parameters.rst

@@ -53,10 +53,11 @@ Input Parameters
        | *RAM power in W = Number of RAM Slots * 5 Watts*
    * - rapl_include_dram
      - | Boolean variable indicating if DRAM (memory) power should be included
-       | in RAPL measurements on Linux systems, defaults to ``True``.
+       | in RAPL measurements on Linux systems, defaults to ``False``.
        | When ``True``, measures complete hardware power (CPU package + DRAM).
        | Set to ``False`` to measure only CPU package power.
        | Note: Only affects systems where RAPL exposes separate DRAM domains.
+       | In a future version DRAM power will probably be included in RAM.
    * - rapl_prefer_psys
      - | Boolean variable indicating if psys (platform/system) RAPL domain should be
        | preferred over package domains on Linux systems, defaults to ``False``.

docs/edit/rapl.rst

@@ -60,7 +60,7 @@ Different CPUs expose different domains. Common domains include:
   - Ring interconnect between cores
   - Integrated GPU (if present)
 
-- **dram**: Memory controller power (rare on consumer hardware, more common on servers)
+- **dram**: Memory controller power, rare on consumer hardware, more common on servers. We still have to figure out if it is accurate.
 
 - **gpu**: Discrete or integrated GPU (when available)
 
@@ -111,7 +111,7 @@ CodeCarbon implements intelligent domain selection to provide reliable and consi
 
 4. **Subdomain filtering**: Excludes ``core`` and ``uncore`` subdomains when ``package`` is available to avoid double-counting
 
-5. **DRAM inclusion**: By default (``include_dram=True``), adds DRAM domain to package for complete hardware power measurement
+5. **DRAM exclusion**: By default (``include_dram=False``), don't adds DRAM domain to package. As DRAM is supposed to be in RAM power, not CPU in a future version of CodeCarbon.
 
 Platform-Specific Behavior
 ~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -334,17 +334,17 @@ Key Takeaways for RAPL Measurements
 
 5. **Interface deduplication**: The same domain may appear in both ``intel-rapl`` (MSR) and ``intel-rapl-mmio`` interfaces. CodeCarbon automatically deduplicates, preferring MMIO.
 
-6. **DRAM measurement**: CodeCarbon includes DRAM domains by default (``include_dram=True``) for complete hardware measurement (CPU package + memory). Set ``include_dram=False`` to measure only CPU package.
+6. **DRAM measurement**: CodeCarbon does not include DRAM domains by default (``include_dram=False``) for CPU hardware measurement. Set ``include_dram=True`` to measure CPU package + DRAM domains.
 
 7. **Platform-specific behavior**:
 
-   - Intel modern: package + dram (default) or psys (with prefer_psys=True)
+   - Intel modern: package or psys (with prefer_psys=True)
    - Intel older: package-0 for CPU only
    - AMD: Sum all package-X-die-Y for multi-die CPUs
 
 8. **Limitations**: RAPL does NOT measure:
 
    - Discrete GPUs (use nvidia-smi/rocm-smi)
    - SSDs, peripherals, fans
-   - Actual DRAM chips (only memory controller on CPU)
+   - Actual DRAM chips, we still have to investigate on this point
    - Complete system power (use wall meter for accuracy)

tests/test_rapl_parameters.py

@@ -11,9 +11,9 @@
 
 
 @pytest.mark.skipif(not sys.platform.lower().startswith("lin"), reason="requires Linux")
-def test_rapl_include_dram_default_true(tmp_path):
+def test_rapl_include_dram_default_false(tmp_path):
     """
-    Verify that rapl_include_dram defaults to True and includes DRAM domains.
+    Verify that rapl_include_dram defaults to False and excludes DRAM domains.
     """
     base = tmp_path
     rapl_provider = base / "intel-rapl"
@@ -36,13 +36,12 @@ def test_rapl_include_dram_default_true(tmp_path):
     # Create IntelRAPL with default parameters
     rapl = IntelRAPL(rapl_dir=str(base))
 
-    # Should have 2 RAPL files: package-0 + dram
-    assert len(rapl._rapl_files) == 2, f"Expected 2 files, got {len(rapl._rapl_files)}"
+    # Should have 1 RAPL files: package-0
+    assert len(rapl._rapl_files) == 1, f"Expected 1 files, got {len(rapl._rapl_files)}"
 
     # Verify both package and dram are present
     names = [f.name for f in rapl._rapl_files]
     assert any("Processor Energy" in name for name in names), "Missing package domain"
-    assert any("dram" in name.lower() for name in names), "Missing DRAM domain"
 
 
 @pytest.mark.skipif(not sys.platform.lower().startswith("lin"), reason="requires Linux")
@@ -457,8 +456,8 @@ def test_rapl_parameters_stored_correctly(tmp_path):
     # Test default values
     rapl_default = IntelRAPL(rapl_dir=str(base))
     assert (
-        rapl_default.rapl_include_dram is True
-    ), "Default rapl_include_dram should be True"
+        rapl_default.rapl_include_dram is False
+    ), "Default rapl_include_dram should be False"
     assert (
         rapl_default.rapl_prefer_psys is False
     ), "Default rapl_prefer_psys should be False"