Added extra text options #267
Merged
Conversation
This file contains 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 new options for - The Extra textfile, the test to display on the image - The max age of the extra text file before its not used - Added an option to specify the text line height - Fixed a few issues with name and std library clashes (time) - Added a debug option for running in foreground - Updated readme
|
@Alex-developer Do you mind taking a look at this issue? #271 There is an issue when the extratextage is set to 0. The text doesn't display on the image. |
Thomas Sorry only just seen this, yes seen the issue and found the bug will sort it later today |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Adds three new fields to the camera config (To be used in conjunction with the PR on the allsky-portal repo)
extratext - The full path to a file containing the extra text to be displayed on the images. The file can contain multiple lines, each will be displayed.
extratextage - If the file above is not updated within this timeframe then the contents of the text file will not be added to the images, handy for when the process updating the text file crashes. To disable this check set the value to 0
textlineheight - Allows the line height of the text to be changed, saves people manually editing and recompiling capture.cpp
Fixed a few issues with name and std library clashes (renamed time to showTime)
Fixed a bug with the command line parser if a text value was missing
Added a debug option (-debuglevel) for running in foreground. If set to 0, the default then the exiting text display is maintained. This option has to be manually added and is not set by the camera config
Added a few missing values to the startup output
Updated readme
Did a full reinstall on a new Pi to test and seems to work ok, had to modify the gui install script to pull from my branch for the test but I am sure thats ok :-)