Opened 7 years ago

Last modified 6 years ago

#2738 new task

Re-enable avatars (buddy photos)

Reported by: smcv Owned by: daf
Priority: normal Milestone: Future Release
Component: presence-service Version:
Keywords: Cc: eben
Blocked By: #3297 Blocking: #6910
Deployments affected: Action Needed:
Verified: no

Description

This should not be done until Gabble 0.5.13 (or at least, my patch that fixes #1967) has landed. There seems to be a bug in that code that's causing fd.o#11920, so it's a bad idea to pull it in right now.

(We're doing upstream stabilization/review work on Gabble at the moment, so it may be better to wait for that to happen before pulling in the upstream versions in any case. If it becomes urgent, please ping smcv.)

Change History (11)

comment:1 Changed 7 years ago by marco

There was also performance issues with the buddy cache right? Please don't re-enable until those are fixed. And we need to (re) add to the intro screen a way to get the photo, otherwise the feature is pretty useless.

comment:2 Changed 7 years ago by jg

  • Milestone changed from Untriaged to Trial-3

comment:3 Changed 7 years ago by morgs

  • Milestone changed from Trial-3 to V1.1

Unlikely we can do this for FRS

comment:4 Changed 7 years ago by kimquirk

  • Owner changed from morgs to daf

comment:5 Changed 7 years ago by daf

Currently, the presence service stores buddy information in a dict which is pickled to disk on every change. Moving to SQLite ought to improve performance.

comment:6 Changed 7 years ago by daf

  • Cc eben added

Eben suggests a 256x256 resolution. The UI would most likely involve taking a picture on first boot, and allow changing it via the Sugar control panel.

comment:7 Changed 6 years ago by marco

  • Keywords 9.1.0:? added
  • Milestone changed from Future Release to 9.1.0

comment:8 Changed 6 years ago by Eben

  • Blocked By 3297 added

(In #3297) The hope here is to distribute avatar info in a pull fashion only, which should make the network implications much less severe.

comment:8 Changed 6 years ago by Eben

The first-boot photo is covered by #3297. We should ensure that this gets into the release at the same time that does.

comment:9 Changed 6 years ago by marco

  • Keywords 9.1.0:? removed
  • Milestone changed from 9.1.0 to Future Release

comment:10 Changed 6 years ago by daf

  • Blocking 6910 added

(In #6910) #2738 address re-enabling avatar support in the presence service.

Note: See TracTickets for help on using tickets.