pandas/conftest.py refactoring
See original GitHub issueAs stated in this review comment from jreback, we should address some refactoring issues in pandas/conftest.py
:
- create the indices fixture using
tm.all_index_generator
as suggested by jbrockmendel in this comment - all fixtures and functions should have a docstring
- add sections, i.e. “some text markers so reading the file is easy”
- remove/avoid imports from
conftest.py
Issue Analytics
- State:
- Created 4 years ago
- Comments:15 (15 by maintainers)
Top Results From Across the Web
Refactoring for Scalable Python Code With Pandas
This post shows a design pattern for reusable and low maintenance code when data-wrangling with Pandas. Objectives of Post. Build a metrics pipeline...
Read more >Refactoring Python Applications for Simplicity
In this step-by-step tutorial, you'll learn how to refactor your Python application to be simpler and more maintainable and have fewer bugs.
Read more >python2-pandas-0.23.4-bp153.1.19 RPM for x86_64 - RPMFind
pandas is a Python package providing flexible and expressive data structures for working with "relational" or "labeled" data.
Read more >refactoring python function that uses pandas dataframes
Consider using pd.DataFrame.eval with pd.DataFrame.pipe : def refactored(df, oldcol, dftest, replace, mod_col, series_col): # ...some ...
Read more >Refactoring with tests in Python: a practical example
Refactoring with tests in Python: a practical example. By Leonardo Giordani - 21/07/2017. Programming OOP pytest Python Python3 refactoring TDD testing.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
+1 in adding commented sections but agree split not needed yet
agree with @jorisvandenbossche for now. adding sections to conftest.py to address https://github.com/pandas-dev/pandas/pull/31701#discussion_r379705368 may help make the case for splitting in the future