Skip to content

Conversation

@AndersonTsaiTW
Copy link
Contributor

@AndersonTsaiTW AndersonTsaiTW commented Oct 12, 2025

Pull Request Guidelines

Before Submitting

  • Code builds without errors or warnings
  • Changes have been tested manually
  • Commit messages are clear and descriptive
  • No unnecessary files are included (build artifacts, IDE files, etc.)

PR Description Should Include

Add Windows network monitoring: shows bytes sent/received (KB/s) and the related funny moods.

  • What: Add Windows network monitoring and related moods
  • Why: This feature is listed in the “awaiting contributions” section, and monitoring network speed is also a fundamental part of system resource tracking.
  • How: Uses GetIfTable, 3s sampling, 64-bit accumulation + wrap-around handling.
  • Testing: use g++ -Iinclude src/main.cpp -o SysMood.exe -liphlpapiand run the .exe. file

!! Important: You must compile with g++ -Iinclude src/main.cpp -o SysMood.exe -liphlpapi because the network module depends on the iphlpapi library. (CMake already links Iphlpapi automatically on Windows.)

Closes #7

image image

@RayBreeze RayBreeze added enhancement New feature or request hacktoberfest labels Oct 12, 2025
@RayBreeze RayBreeze self-requested a review October 12, 2025 11:14
@RayBreeze
Copy link
Owner

Hi @AndersonTsaiTW. Apologize me, actually I have made some minor changes in main branch before reviewing your PR. Actually it was working completely fine ago with no conflicts, but it needed to be adhere to the new main branch which I actually changed after you have already submitted your PR. Sorry for inconvenience caused but can you rebase as per the latest main branch then it would be very helpful for me to review it and successfully merge it. I am again sorry for the inconvenience caused.

Regards,
Samman

@RayBreeze RayBreeze self-assigned this Oct 12, 2025
@AndersonTsaiTW
Copy link
Contributor Author

Because this branch is kind of dirty, I built a new one and added the new feature again.
Please check the new branch. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request hacktoberfest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add network stats to display bytes sent/received

3 participants