Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 9, 2025

The ROM usage report was missing the embedded modules.dmp file, which is included in flash during build but not tracked in the linker map file.

Changes

  • scripts/memory_report.py: Check for modules.dmp in build directory and include its size in ROM calculations
    • Add modules_dmp_size parameter to get_rom_usage() and get_total_rom()
    • Add modules.dmp as separate line item in ROM table when present
    • Handle file access errors gracefully

Example

With modules.dmp present:

Found modules.dmp: 2.14 KB

============================================================
                  ROM (Flash) Memory Usage                  
============================================================
Component                                 Size          %
------------------------------------------------------------
dmlog                                  8.50 KB      34.1%
main                                   6.25 KB      25.1%
modules.dmp                            2.14 KB       8.6%
...
------------------------------------------------------------
Used                                  24.89 KB     100.0%

Without modules.dmp:
No change to existing behavior—file is simply not listed and total excludes it.

Original prompt

This section details on the original issue you should resolve

<issue_title>Dodać do podsumowania ROM rozmiar modules.dmp</issue_title>
<issue_description>Aktualnie w podsumowaniu zużycia ROM brakuje rozmiaru modules.dmp który też jest we flash </issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 9, 2025 20:00
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Co-authored-by: JohnAmadis <17320783+JohnAmadis@users.noreply.github.com>
Copilot AI changed the title [WIP] Add ROM size for modules.dmp to summary Include modules.dmp size in ROM usage summary Dec 9, 2025
Copilot AI requested a review from JohnAmadis December 9, 2025 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dodać do podsumowania ROM rozmiar modules.dmp

2 participants