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

Upstreaming v21.1 #27

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

Upstreaming v21.1 #27

wants to merge 5 commits into from

Conversation

quozl
Copy link
Contributor

@quozl quozl commented Aug 1, 2020

Some changes in the Python 2 branch are potentially useful for the Python 3 branch.

  • a double update is fixed,
  • audio file is lengthened slightly,
  • tick pipeline is now paused after sounding,
  • tick pipeline end of stream handler is no longer required.

Display was being updated twice a second, because the timeout was
scheduled twice, because the active property was set to True by the
toolkit twice.

The toolkit avoids this by not processing a second active True.

Ignore property changes that have no effect.
On OLPC XO-4 laptop, opening the audio device causes a click, and this
makes the ticking clock noisy.

Existing logic was to play a 200 ms sound file every second, wait for
end of stream, rewind, and play again.  This closed and opened the audio
device.

Now, rewind and play a 1200 ms sound file and pause it after 450 ms.
End of stream is never reached.  This keeps the audio device open.
On OLPC XO-1, if the ticking clock is already enabled, a traceback
occurs;

Traceback (most recent call last):
  File "/home/olpc/Activities/Clock.activity/clock.py", line 467, in _ticking_toggled_cb
    if self._pathetic:
AttributeError: 'ClockActivity' object has no attribute '_pathetic'

Caused by toggling the button before the hardware has been identified.

Move the identification of hardware higher up in the initialisation.
On OLPC XO-1 rewinding the GStreamer pipeline yields a random volume
level.
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.

1 participant