Skip to content
This repository has been archived by the owner on Aug 7, 2018. It is now read-only.

Issue #452 - Exibir a porcentagem da variância explicada pelas duas dimensões do mapa de votações #464

Merged

Conversation

lopes05
Copy link
Contributor

@lopes05 lopes05 commented Apr 27, 2018

Criado campo de informações estatísticas avançadas contendo o valor da variância explicada fornecida pelo json.

@leonardofl
Copy link
Member

Me parece que não funcionou aqui. Poderiam mandar, por favor, um print da tela de vocês funcionando?

@leonardofl
Copy link
Member

Obs: eu fiz um commit no arquivo analise.html, que é onde vcs mexeram nesse PR. Então façam o rebase primeiro, plz.

@lopes05
Copy link
Contributor Author

lopes05 commented Apr 29, 2018

Print da tela:

Irei fazer o rebase

Co-authored-by: Jefersonalves <ferreira.jefersonn@gmail.com>

Criado campo de informações estatísticas avançadas que mostra a variância explicada pelos componentes

Co-authored-by: Jeferson Alves <ferreira.jefersonn@gmail.com>
Co-authored-by: Thiago Nogueira <thiagonf10@gmail.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 949

  • 0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 82.327%

Totals Coverage Status
Change from base Build 948: 0.0%
Covered Lines: 3913
Relevant Lines: 4753

💛 - Coveralls

@lopes05
Copy link
Contributor Author

lopes05 commented Apr 30, 2018

Rebase feito, se não aparecer tenta abrir em uma aba anônima, a gente teve bastante problema com cache e só conseguia ver os resultados do código ou apagando o histórico de cache ou com aba anônima.

@diraol
Copy link
Member

diraol commented May 2, 2018

@leonardofl confirma?

@lopes05 uma dica para não ter problema com cache é abir a "barra de dev do navegador" (aquela que tem o "console"), ir na parte de configurações e setar a flag "Desabilitar cache enquanto a barra de desenvolvedor estiver aberto" (ou algo assim). Tem tanto no firefox quanto no chrome.

@leonardofl leonardofl merged commit cd12b4a into radar-parlamentar:master May 3, 2018
@leonardofl
Copy link
Member

OK! Tanto usando a guia anônima quando dando refresh com ctrl+F5 deu certo!!!

Valeus!

@diraol : sobre a questão de não ter commit de merge... Esse commit 167c561 é um problema ou o problema era mais os "merge master" na master?

@diraol
Copy link
Member

diraol commented May 3, 2018

Oi @leonardofl , eu acho que esse commit de merge não é tão problemático não.

Eu tenho uma tendência pessoal a não tê-lo, mas também não acho o fim do mundo ele por lá rs.
Já o "merge master on master" ai fica bizarro... rs

@felipe72 felipe72 deleted the issue-#452 branch May 8, 2018 20:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants