Splitting the Installation Script + Fix for Fan Speed in pico_status #3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 4 commits
February 4, 2017 08:53
No need to mix up daemon installation and firmware update preparation step.
This is the script for firmware update preparation steps.
This script is for returning the OS to original state and undo all steps in firmware_update.sh.
Author
|
renamed the firmware_update.sh to pre_firmware_update.sh |
Better to use percentage for fan speed as the original script unable to parse 60% fan speed (0x3c) and throws this error below:
UPS PIco FAN Speed....:
Traceback (most recent call last):
File "./pico_status.py", line 263, in <module>
print " ","UPS PIco FAN Speed....:",fan_speed(),"RPM"
File "./pico_status.py", line 201, in fan_speed
return int (float(data) * 100)
ValueError: invalid literal for float(): 3c
Crescendo-BLYAT
changed the title
Author
|
Also if you can (coz its beyond my capability), please also add in config_for_pico.sh script to comment out this: in /lib/udev/hwclock-set to enable the real-hwclock. |
added 2 commits
February 4, 2017 10:51
include daemon installation...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I split the script as there's no need to mix the daemon installation with pre-firmware update steps.
So the original script is only for daemon installation.
The other 2 is for firmware update preparation and post firmware update to return the OS to original state.
Futher: the installation script also include daemon installation & steps to enable dtoverlay needed for activating RTC as mentioned in: AdaFruit RTC Installation
I also changed the fan speed displaying from RPM to % coz the current script unable to parse fan speed 60% (i2cget as 0x3c).
It throws error: