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

Fixed missing space between GB #5

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

stewie0056
Copy link
Contributor

No description provided.

@Drewster727
Copy link
Owner

@stewie0056 what's up with the space between the G and B?

@stewie0056
Copy link
Contributor Author

@Drewster727 hddtemp identifies the drive name with a space so the definition must match exactly. Not sure why but as long as it works i'm not going to dig into it further.

On a side note, this Reddit post (section 3-5) suggests that we may not need to define each drive model or size individually but could cover a entire range with a generic line like:
"Samsung SSD 860 EVO" 190 C "Samsung SSD 860 EVO"
This would cover all sizes of 860 EVOs assuming all are have temp on 190. I have not tried this yet but it may make your copy of the database more universal for SSD support.

https://www.reddit.com/r/unRAID/comments/9rnp82/howto_get_ssd_temp_monitoring_working_with_hddtemp/

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.

2 participants