Skip to content

Commit

Permalink
Removing pandas-profiling from post-setup checks
Browse files Browse the repository at this point in the history
Reasons:

- Pandas-profiling has been remamed ydata-profiling.
- We removed it in Update apple_silicon.txt #272 from the Apple Silicon requirements, because of incompatibility with scipy
- It is a non-essential package anyway, and is used in only one challenge (04-01-03 Exploratory Analysis), which includes instructions to pip install it.
  • Loading branch information
julesvanrie committed Apr 5, 2024
1 parent 7e8cb75 commit c60d7a1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions checks/pip_check.sh
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
REQUIRED=('pytest' 'pylint' 'ipdb' 'PyYAML' 'nbresult' 'autopep8' 'flake8' 'yapf' 'lxml' 'requests' 'beautifulsoup4' 'jupyterlab' 'pandas' 'matplotlib' 'seaborn' 'plotly' 'scikit-learn' 'tensorflow' 'nbconvert' 'xgboost' 'statsmodels' 'pandas-profiling' 'jupyter-resource-usage')
REQUIRED=('pytest' 'pylint' 'ipdb' 'PyYAML' 'nbresult' 'autopep8' 'flake8' 'yapf' 'lxml' 'requests' 'beautifulsoup4' 'jupyterlab' 'pandas' 'matplotlib' 'seaborn' 'plotly' 'scikit-learn' 'tensorflow' 'nbconvert' 'xgboost' 'statsmodels' 'jupyter-resource-usage')
PACKAGES=$(pip freeze)
PACKS=()
while read -r line; do
Expand All @@ -14,7 +14,7 @@ elif [ "${arch_name}" = "arm64" ]; then
arch_name='m1'
fi
if [ $arch_name = 'm1' ]; then
REQUIRED=('pytest' 'pylint' 'ipdb' 'PyYAML' 'nbresult' 'autopep8' 'flake8' 'yapf' 'lxml' 'requests' 'beautifulsoup4' 'jupyterlab' 'pandas' 'matplotlib' 'seaborn' 'plotly' 'scikit-learn' 'tensorflow-macos' 'nbconvert' 'xgboost' 'statsmodels' 'pandas-profiling' 'jupyter-resource-usage')
REQUIRED=('pytest' 'pylint' 'ipdb' 'PyYAML' 'nbresult' 'autopep8' 'flake8' 'yapf' 'lxml' 'requests' 'beautifulsoup4' 'jupyterlab' 'pandas' 'matplotlib' 'seaborn' 'plotly' 'scikit-learn' 'tensorflow-macos' 'nbconvert' 'xgboost' 'statsmodels' 'jupyter-resource-usage')
fi
for r in ${REQUIRED[@]}; do
echo $r
Expand Down

0 comments on commit c60d7a1

Please sign in to comment.