Skip to content

Iago/h8 ticket medio#25

Open
iagonmic wants to merge 15 commits intodevfrom
iago/h8_ticket_medio
Open

Iago/h8 ticket medio#25
iagonmic wants to merge 15 commits intodevfrom
iago/h8_ticket_medio

Conversation

@iagonmic
Copy link
Copy Markdown
Collaborator

@iagonmic iagonmic commented Sep 3, 2024

No description provided.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esse diretório de libs não deveria estar aqui. Com o parâmetro self-contained = TRUE deveria fazer com que as libs fossem adicionadas direto no arquivo HTML. Testa por favor trocar o parâmetro de self-contained por embed-resources = TRUE pra ver se essa pasta libs desaparece? Valeu! @iagonmic

@AlanAugustin
Copy link
Copy Markdown
Collaborator

Minha única sugestão é adicionar o parametro decimal=',' no read csv para não precisar tratar a troca de ponto por vírgula com regex.
A análise ficou muito boa, parabéns pelo trabalho!

@iagonmic
Copy link
Copy Markdown
Collaborator Author

iagonmic commented Oct 3, 2024

Pronto! Realizei ambas as alterações.

Copy link
Copy Markdown
Collaborator

@AlanAugustin AlanAugustin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tudo certo! Adicionou o parâmetro decimal = ','. A lógica do código e as escolhas dos testes de hipóteses já estavam corretas.

@wrprates
Copy link
Copy Markdown
Owner

wrprates commented Oct 15, 2024

@iagonmic Vi que o embed-resources não fez com que a pasta libs fosse removida. Vamos tentar add ela ao gitignore, de forma semelhante ao que sugeri ao Alan, aqui: #28 (comment)

@iagonmic
Copy link
Copy Markdown
Collaborator Author

Pronto @wrprates, fiz o commit com a pasta libs excluida 👍

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.

H8 - O ticket médio (por cliente por compra) aumenta em datas especiais/comemorativas

3 participants