Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inclusão dos métodos __parse_float_cols e __parse_pct_cols #10

Merged
merged 16 commits into from
Feb 12, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Feb 6, 2024

📑 Detalhes do Pull Request:

Este PR contempla a inclusão dos método __parse_float_cols e __parse_pct_cols da classe Fundamentus presente no módulo fundamentus. O intuito dos novos métodos é proporcionar uma experiência diferenciada aos usuários que desejarem obter seus respectivos indicadores financeiros com os tipos primitivos do DataFrame resultante previamente tratados.

Além dos novos métodos inclusos na classe, o método coleta_indicadores_de_ativo foi adaptado através da inclusão do parâmetro parse_dtypes (cujo default está configurado para False) com o objetivo de guiar a conversão dos tipos primitivos do DataFrame resultante.


🔨 Atividades/features contempladas:

  • Inclusão do método __parse_float_cols para conversão de atributos do scrapper Fundamentus com características numéricas
  • Inclusão do método __parse_pct_cols para conversão de atributos do scrapper Fundamentus com características percentuais
  • Adaptação do método coleta_indicadores_de_ativo para validar a conversão dos tipos
  • Validação de proposta de nova documentação da solução

🔗 Issues vinculadas

Este PR eventualmente está atrelado às seguintes issues:

  • Nenhuma issue vinculada
  • #

Checklist:

  • As novas parcelas de código seguiram as boas práticas de desenvolvimento
  • Novas funcionalidades foram testadas adequadamente
  • Espera-se que as novas funcionalidades não comprometam a solução como um todo

Copy link

codecov bot commented Feb 6, 2024

Codecov Report

Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (fc20b8b) 72.56% compared to head (4e15e28) 66.66%.
Report is 2 commits behind head on main.

Files Patch % Lines
pynvest/scrappers/fundamentus.py 25.00% 18 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   72.56%   66.66%   -5.90%     
==========================================
  Files           7        7              
  Lines         164      186      +22     
==========================================
+ Hits          119      124       +5     
- Misses         45       62      +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ThiagoPanini ThiagoPanini self-requested a review February 12, 2024 02:17
@ThiagoPanini ThiagoPanini self-assigned this Feb 12, 2024
@ThiagoPanini ThiagoPanini added the doc 🖍️ Alterações ou aprimoramentos na documentação label Feb 12, 2024
@ThiagoPanini ThiagoPanini changed the title pr(main): refs/heads/v0.1.x -> main Inclusão dos métodos __parse_float_cols e __parse_pct_cols Feb 12, 2024
@ThiagoPanini ThiagoPanini added new-function 💎 Desenvolvimento de novas funções ou pequenos blocos de código and removed doc 🖍️ Alterações ou aprimoramentos na documentação labels Feb 12, 2024
@ThiagoPanini ThiagoPanini merged commit b380e21 into main Feb 12, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-function 💎 Desenvolvimento de novas funções ou pequenos blocos de código
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant