Encontrar parlamentar no gráfico PCA

[leonardofl]

Possibilitar o destaque de um parlamentar de interesse do usuário.

[brenddongontijo]

Opa pessoal, gostaria de saber mais sobre essa funcionalidade de destacar um parlamentar de interesse ao usuário.

Estava pensando em adicionar um campo de "Pesquisar Parlamentar" na lista de Gênero (Página Inicial). Dentro desta nova página haveria uma busca por nome que resultaria nos dados do parlamentar.

Estou certo?

[leonardofl]

Opa!
Então, esta issue faz referência ao gráfico de proximidade dos parlamentares:
http://radarparlamentar.polignu.org/analises/analise/cmsp/

Do tipo, dentre as centenas de bolinhas, queria ver a bolinha do cara em quem eu voto.

Mas se essa ideia de "encontre seu parlamentar" se aplicar a outros gráficos, podem mandar bala! (nesse caso, podem abrir uma issue específica plz).

Valeu!

[brenddongontijo]

Opa pessoal, começamos a realizar esta issue e já conseguimos alguns avanços:

• Buscar o nome do parlamentar estritamente igual.
• Expandir o primeiro partido encontrado com o nome deste parlamentar.

Como estamos no fim do semestre da disciplina de MES não será possível terminar esta issue.
Poderíamos deixar em uma branch do radar o que já fizemos, é possível?

[diraol]

Oi Brenddon,

que legal que conseguiram avançar com essa issue!

E sim, seria recomendadíssimo deixar essa contribuição parcial numa branch
no repositório! Se puderem, só peço para deixarem bem documentado o que
fizeram e, se possível, quais seriam os próximos passos, para facilitar
quem for continuar a implementação.

Eu acabei de criar um branch "issue272" à partir da branch master para
que vocês possam fazer o pull-request para lá. Só tentem sincronizar essa
branch de vocês com a master atual antes de fazer o pull-request para ter
certeza que vai dar tudo certo com o PR.
​

[leonardofl]

Humm... de volta à polêmica. Esse código pode demorar pra ser retomado. Deixar isso numa branch a parte é muito arriscado. Quando alguém for retomar isso no futuro pode ter muitos problemas pra fazer o merge. Acho melhor deixar no master, mas de alguma forma ocultar essa funcionalidade inacabada pro usuário final. Daí aqui na issue vcs registram oq o desenvolvedor tem q fazer para "reativar" a funcionalidade na interface.

Um exemplo de inibição é simplesmente comentando o input e o botão da funcionalidade no HTML. Algo mais sofisticado e melhor pra não deixar código comentado é criar uma configuração booleana no settings pra ativar/desativar a funcionalidade (feature toggle!).

http://martinfowler.com/bliki/FeatureToggle.html

[paulohtfs]

@leonardofl tentei utilizar o toggle, mas sem sucesso, então coloquei como comentário msm.

Vou colocar aqui como "documento" da issue pra facilitar quem for continuar ela.

Atualmente a funcionalidade de encontrar o parlamentar no gráfico esta sendo feita da seguinte forma:

• Buscar o parlamentar pelo nome. O nome deve ser exatamente igual ao contido no gráfico. Ex: "Ricardo" > não encontra, "Ricardo Teixeira" > encontra.
• Expande apenas o primeiro partido com o nome do parlamentar encontrado. Sendo assim, se houver outro partido com um parlamentar com o mesmo nome não é expandido.
• O partido expandido vale para todos os períodos, ou seja, quando o parlamentar é encontrado no partido ele expande independente do período. No caso da busca por "Ricardo Teixeira", por exemplo, o partido PSDB é expandido, mas o parlamentar não aparece no período de 2013 a 2014 apenas nos anteriores.
• Os arquivos modificados foram: analise.html e radar_plot.js.

O deve ser feito:

• Adicionar mensagem de erro caso não encontre o parlamentar.
• Procurar por substring. Ex: "Ricardo" expande todos os partidos contendo os parlamentares com o nome de "Ricardo".
• Adicionar toltip após a expansão nos parlamentares encontrados.
• Encontrar todos os parlamentares com o nome procurado.

Para facilitar, em todo o arquivo radar_plot.js tem comentários "// Issue 272" que são códigos que fazem parte dessa issue.

[leonardofl]

Legal, valeu!
Além disso, fiquei na dúvida sobre se o escopo da busca não deveria se restringir ao período exibido... fica estranho expandir o partido e o carinha não aparecer.

[LauraSuh]

Oi galera!
Então, o grupo de MES 2016/02 resolveu a parte inicial desta issue, nós destacamos o parlamentar pesquisado e estamos mandando uma mensagem de "Parlamentar não encontrado" quando é o caso.
Para maior granularidade e em uma conversa com o @leonardofl , resolvemos quebrar o resto desta issue nas novas #382 e #383
Após toda a equipe revisar o novo código estaremos mandando o pull request, qualquer coisa estamos aí!

[diraol]

Bacana Laura! Achei a proposta boa!

---

Diego Rabatone Oliveira
http://blog.diraol.eng.br
diraol(arroba)diraol(ponto)eng(ponto)br
Twitter: @diraol

Em 6 de setembro de 2016 19:12, Laura Barros Martins <
[email protected]> escreveu:

Oi galera!
Então, o grupo de MES 2016/02 resolveu a parte inicial desta issue, nós
destacamos o parlamentar pesquisado e estamos mandando uma mensagem de
"Parlamentar não encontrado" quando é o caso.
Para maior granularidade e em uma conversa com o @leonardofl
https://github.com/leonardofl , resolvemos quebrar o resto desta issue
nas novas #382 #382 e
#383 #383
Após toda a equipe revisar o novo código estaremos mandando o pull
request, qualquer coisa estamos aí!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#272 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALwvpbmi1mB2d-u1TeUOlQ3XH5Ps-Z6ks5qneVlgaJpZM4EFYHp
.