spaghetti-open-data / twitAntonio

TweetYourMep fork for the Italian 2013 Elections
http://www.twitantonio.it
GNU Affero General Public License v3.0
16 stars 10 forks source link

Twitter Avatar pictures #18

Closed paolomainardi closed 11 years ago

paolomainardi commented 11 years ago

Potremmo usare direttamente gli avatar di Twitter, piuttosto che cercare una foto del candidato da qualche altra parte.

Con le API è possibile farlo, ho trovato questa piccola app su Github che fa proprio questo ed ha un interfaccia REST: https://github.com/remy/twivatar/

Potremmo riutilizzare questo codice: https://github.com/remy/twivatar/

dottorblaster commented 11 years ago

Prendiamo la 48x48 e ci standardizziamo su quella?

paolomainardi commented 11 years ago

Forse la 48x48 è un po' piccola meglio andare sulla 73x73. @andypanix tu che ne pensi ?

nelsonmau commented 11 years ago
per me d'accordo su 73, al max si poi si scalaIl 24/01/2013 11:55, Paolo Mainardi ha
  scritto:

  Forse la 48x48 è un po' piccola meglio andare sulla 73x73. @andypanix tu che ne pensi ?

    —
    Reply to this email directly or view
      it on GitHub. 

-- 

Andrea Nelson Mauro - Datajournalist

http://www.dataninja.it http://twitter.com/nelsonmau

andypanix commented 11 years ago

Sì, credo che 48x48 sia piccola, ma anche 73x73 lo è pensando nella struttura attuale. Tenete conto che ora le immagini sono a 210 di larghezza. A questo punto conviene quindi ripensare la struttura della singola scheda.

Con Paolo abbiamo preparato un mockup. Che ne dite?

twitantonio

paolomainardi commented 11 years ago

@andypanix Credo che lo voterò questo Denis Sboron, ha una media del 50% di risposte....non male! :+1:

Mi piacciono molto, sentiamo anche il parere degl'altri e poi invia di implementazione. Very good work man!

dottorblaster commented 11 years ago

Implementazione, si ma come? Avevamo scelto quel layout proprio perché così con Bootstrap ci veniva rapido. Se avete indizi su come realizzare una cosa del genere anche responsive, ben venga :)

paolomainardi commented 11 years ago

Alessio,@andypanix può aiutarci ad implementarlo, è lui il mago del frontend.

lucacorsato commented 11 years ago

siete dei Grandi!: denis sboron lo trovo eccezionale. sulle dimensioni partite da 210 e poi scalate. Questo dovrebbe permettere poi al css di adattarsi per gli schermi grandi, lasciando il 210 per monitor superiori ai 1900x1200 (anche se poi hai problemi con gli schermi retina dell'ipad...)

2013/1/24 Paolo Mainardi notifications@github.com

@andypanix https://github.com/andypanix Credo che lo voterò questo Denis Sboron, ha una media del 50% di risposte....non male! [image: :+1:]

Mi piacciono molto, sentiamo anche il parere degl'altri e poi invia di implementazione. Very good work man!

— Reply to this email directly or view it on GitHubhttps://github.com/spaghetti-open-data/twitAntonio/issues/18#issuecomment-12648224.

stefanomainardi commented 11 years ago

Denis Sboron FTW. Ottimo lavoro ragazzi, mi unisco ufficialmente alla ciurma per dare il mio supporto :)

lucacorsato commented 11 years ago

ho dato un'occhiata ai profili tw... quando scelgo l'immagine è in 210, ma poi appare sempre in 73x73 (densità 28 px)... rispetto anche a tyMEP si potrebbe dare un riquadro massimo di 150 e inserire al centro solo l'avatar a 73x73

paolomainardi commented 11 years ago

@gaspa Andrea appena hai novità su questa parte facci sapere, il layout adesso è quasi pronto #11

gaspa commented 11 years ago

si', sto studiacchiando ancora un po' come fare le cose, non ho avuto molto tempo in questi giorni. Stasera mi metto in modalità operativa. :)

gaspa commented 11 years ago

@paolomainardi , o chi ne sa di nodejs: qualcuno mi spiega perchè in questo pezzo: https://github.com/gaspa/twitAntonio/blob/b2f02a5360555efe4eb7a312c29bd98a93ac75a0/import/import-avatar.js

se commento da 29 a 31 e decommento le righe sopra, non mi viene eseguito "user.save()", nonostante passi per "console.log("ok")" ??

A parte questa cosa, lo script fa già la cosa giusta (c'è da sistemare la directory in cui salvare, e (ma anche no) l'estensione dei file... in un primo momento puo' pensarci il browser a riconoscere png/jpg)

seralf commented 11 years ago

l'unica cosa che mi viene in mente (a parte typos che non vedo) a colpo d'occhio è che magari vada in errore alla linea 25? forse un null o simili?

Il giorno 26 gennaio 2013 02:13, Andrea Gasparini notifications@github.comha scritto:

@paolomainardi https://github.com/paolomainardi , o chi ne sa di nodejs: qualcuno mi spiega perchè in questo pezzo:

https://github.com/gaspa/twitAntonio/blob/b2f02a5360555efe4eb7a312c29bd98a93ac75a0/import/import-avatar.js

se commento da 29 a 31 e decommento le righe sopra, non mi viene eseguito "user.save()", nonostante passi per "console.log("ok")" ??

A parte questa cosa, lo script fa già la cosa giusta (c'è da sistemare la directory in cui salvare, e (ma anche no) l'estensione dei file... in un primo momento puo' pensarci il browser a riconoscere png/jpg)

— Reply to this email directly or view it on GitHubhttps://github.com/spaghetti-open-data/twitAntonio/issues/18#issuecomment-12728446.

gaspa commented 11 years ago

Questo:

        console.log("ok");
        usrhnd.mep_epFotoUrl = config.base_path + localurl;
        usrhnd.save();
        console.log("oook");

mi scrive entrambi i log a video :\ Ho provato pure a cambiare gli scope (messo user locale, spostato la funzione dentro db.once.... boh. Sono un po' niubbo riguardo a node.js, sorry :P

gaspa commented 11 years ago

anzi, meglio, se nell'ultimo log metto: "console.log("usr: "+usr);" vedo che c'è la cosa giusta: mep_epFotoUrl: '/imgs/users/510301b8da0b765c1100000a.sbraf', Pero' nel DB (o se lancio nodejs app.sj) il valore è quello precedente. "mep_epFotoUrl" : "/imgs/users/510301b8da0b765c1100000a.asdasd",

insomma,boh. Attendo consigli (se non me li da' google,prima :P )

gaspa commented 11 years ago

no, trovato \o/

paolomainardi commented 11 years ago

Ciao Andrea, riaprio questa issue perchè con il dataset di adesso di 680 candidati ieri non riuscito ad importare correttamente (in locale) errore EMFILE: too many files opened, inoltre non credo che il flusso async tra Mongoose e Request lo stiamo gestendo in maniera corretta.

Ho fatto delle prove con la libreria "async" per cercare di rendere i processi separati, ma non ho avuto grandi risultati (considerando gli orari dei commit :)).

Puoi vedere qui: https://github.com/spaghetti-open-data/twitAntonio/blob/master/import/import-avatar.js#L39

gaspa commented 11 years ago

@paolomainardi chiudiamo qui? o c'è ancora qualcosa da fare?

paolomainardi commented 11 years ago

Chiudiamo qui, grazie Andrea Il giorno 26/gen/2013 09:47, "Andrea Gasparini" notifications@github.com ha scritto:

no, trovato \o/

— Reply to this email directly or view it on GitHubhttps://github.com/spaghetti-open-data/twitAntonio/issues/18#issuecomment-12732783.