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.

Sign up for GitHub

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

Jump to bottom

Fix logic error in nvidia frequency calculation #2018

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Fix logic error in nvidia frequency calculation #2018

merged 1 commit into from
Aug 27, 2024

Conversation

w0j0pl
Copy link
Contributor

@w0j0pl w0j0pl commented Aug 25, 2024 •
edited
Loading

Made a suggested change in logic from issue #1177

Checklist

  • I have described the changes
  • I have linked to any relevant GitHub issues, if applicable
  • Documentation in doc/ has been updated
  • All new code is licensed under GPLv3

Description

  • Describe the changes, why they were necessary, etc
  • Describe how the changes will affect existing behaviour.
  • Describe how you tested and validated your changes.
  • Include any relevant screenshots/evidence demonstrating that the changes work and have been tested.

@w0j0pl
Update nvidia.cc
b0a5f6a 
Made a suggested change in logic from issue brndnmtthws#1177
@github-actions github-actions bot added sources PR modifies project sources nvidia Issue or PR related to nvidia cards labels Aug 25, 2024
@netlify Netlify
Copy link

netlify bot commented Aug 25, 2024 •
edited
Loading

✅ Deploy Preview for conkyweb canceled.

Name Link
🔨 Latest commit b0a5f6a
🔍 Latest deploy log https://app.netlify.com/sites/conkyweb/deploys/66cb29209dce5d000805f17d

@brndnmtthws brndnmtthws changed the title Update nvidia.cc Fix logic error in nvidia frequency calculation Aug 27, 2024
@brndnmtthws brndnmtthws merged commit c428663 into brndnmtthws:main Aug 27, 2024
39 checks passed
@brndnmtthws brndnmtthws mentioned this pull request Aug 27, 2024
Closed
@brndnmtthws brndnmtthws added the bug Bug report or bug fix PR label Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brndnmtthws brndnmtthws brndnmtthws approved these changes

Assignees
No one assigned
Labels
bug Bug report or bug fix PR nvidia Issue or PR related to nvidia cards sources PR modifies project sources
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@w0j0pl @brndnmtthws