Simplify or remove `assert_*_equal` wrappers around `pandas.testing` functions
Mantenedores costumam responder em até 1 dia
@dgowdaan-cmyk já está trabalhando nisso.
Desde 8/7/2026.
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 62/100
Direção de pesquisa
Os wrappers de assert_frame_equal, assert_series_equal e assert_index_equal ficam em conftest.py, onde verificam a versão do pandas em relação a 1.1.0 e ajustam os kwargs. Comece lendo esses três wrappers e os testes que os chamam. Está concluído quando a verificação de versão e a ramificação de kwargs forem removidas, os wrappers continuarem passando na suíte de testes existente e a versão mínima do pandas for confirmada como superior a 1.1.0. Substituir os wrappers por chamadas diretas ao pandas é uma questão maior e separada, e é melhor deixá-la para uma decisão de um maintainer.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
We have our own wrappers in conftest.py for the three pandas assert_X_equal functions. We originally added those because the default tolerances of those functions changed in pandas 1.1.0 (see #1018). Our wrappers check the version of pandas and do some simple kwarg manipulation based on whether the installed pandas is greater or lower than 1.1.0.
Now that our minimum pandas is greater than 1.1.0, the version checking is no longer needed and we can simplify those functions.
It is also worth looking into whether we can/should get rid of those wrappers entirely and just use the pandas functions directly, but that change might require a lot of work (https://github.com/pvlib/pvlib-python/issues/1018#issuecomment-668631386).
- Linguagem predominante
- Python
- Estrelas
- 1.7k
- Forks
- 1.2k
- Merge médio
- 3d 18h
- PRs com merge (30d)
- 13
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de pvlib/pvlib-python
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
pvlib/pvlib-python#2903 · 2 comentários · 2 reações ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 92/100
pvlib/pvlib-python#2902 ·
Mantenedores costumam responder em até 1 dia
-
Add an example or documentation on using 'bhi' from ERA5Talvez livre de novo @anshurajbisoyi98-ctrl assumiu há 14 dias e não há nenhum pull request aberto. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
pvlib/pvlib-python#2880 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Use "long-wave" rather than "infra-red" to refer to thermal radiation.Talvez livre de novo @Rahi-padwal assumiu há 251 dias e não há nenhum pull request aberto. Abertaapi easy
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
pvlib/pvlib-python#2653 · 6 comentários ·
Mantenedores costumam responder em até 1 dia
-
question
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 72/100
pvlib/pvlib-python#2637 · 10 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de pvlib/pvlib-python
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
NousResearch/hermes-agent#140958 ·
Mantenedores costumam responder em até 1 dia
-
Bisect issue233186Abertabisect
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
dtcxzyw/llvm-bisect-service#548 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
sallowayma-git/IELTS-practice#248 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 65/100
anthropics/claude-plugins-official#6763 · 10 comentários · 10 reações ·
Mantenedores costumam responder em até 3 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
Mantenedores costumam responder em até 1 dia