Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TEMPERATURE] Adds sensors_excluded_subfeatures config. option #108

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

HorlogeSkynet
Copy link
Owner

closes #107

Description

See #107 for rationale.

Reason and / or context

Some chipset subfeatures may return inaccurate values when probed.

How has this been tested ?

Unit test.

Types of changes :

  • Bug fix (non-breaking change which fixes an issue)
  • Typo / style fix (non-breaking change which improves readability)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist :

  • [IF NEEDED] I have updated the README.md file accordingly ;
  • [IF NEEDED] I have updated the test cases (which pass) accordingly ;
  • [IF BREAKING] This pull request targets next Archey version branch ;
  • My changes looks good ;
  • I agree that my code may be modified in the future ;
  • My code follows the code style of this project (PEP8).

@HorlogeSkynet HorlogeSkynet added the enhancement ⬆️ Implements a new feature, fixes or improves existing ones label Jan 15, 2022
@HorlogeSkynet HorlogeSkynet self-assigned this Jan 15, 2022
@HorlogeSkynet HorlogeSkynet added this to the v4.13.4 milestone Feb 7, 2022
@HorlogeSkynet
Copy link
Owner Author

Merging due to age.

@HorlogeSkynet HorlogeSkynet merged commit 2a12cdc into master Feb 8, 2022
@HorlogeSkynet HorlogeSkynet deleted the feature/sensors_excluded_subfeatures branch February 8, 2022 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement ⬆️ Implements a new feature, fixes or improves existing ones
Projects
Status: DONE
Development

Successfully merging this pull request may close these issues.

Temperature invalid on AMD platform
1 participant