search

INRIA / scikit-learn-mooc

Machine learning in Python with scikit-learn MOOC
https://inria.github.io/scikit-learn-mooc
Creative Commons Attribution 4.0 International
1.12k stars 516 forks source link

Remove notebooks #775

Closed f-hafner closed 7 months ago

f-hafner commented 7 months ago

when running make from the main branch, the error is raised:

python build_tools/convert-python-script-to-notebook.py python_scripts/01_tabular_data_exploration.py notebooks/01_tabular_data_exploration.ipynb
cp python_scripts/matplotlibrc notebooks/
python build_tools/sanity-check.py python_scripts notebooks
Traceback (most recent call last):
  File "/home/flavio/repositories/teaching/scikit-learn-mooc/build_tools/sanity-check.py", line 30, in <module>
    raise RuntimeError(
RuntimeError: Inconsistency between folder python_scripts and notebooks
Only in folder python_scripts: set()
Only in folder notebooks: {'cross_validation_grouping', 'metrics_regression', 'cross_validation_train_test', 'metrics_sol_01', 'cross_validation_sol_01', 'linear_models_ex_02', 'parameter_tuning_parallel_plot', 'trees_sol_02', 'feature_selection_limitation_model', 'cross_validation_stratification', 'linear_models_ex_01', 'ensemble_ex_02', 'parameter_tuning_grid_search', 'metrics_ex_02', 'ensemble_sol_04', 'cross_validation_nested', 'feature_selection_ex_01', 'ensemble_sol_01', 'logistic_regression', 'parameter_tuning_manual', 'cross_validation_validation_curve', 'cross_validation_time', 'ensemble_sol_02', 'ensemble_sol_03', 'cross_validation_ex_02', 'linear_regression_in_sklearn', 'linear_models_ex_04', 'trees_dataset', 'datasets_bike_rides', 'ensemble_ex_01', 'dev_features_importance', 'linear_models_sol_01', 'trees_ex_02', 'trees_sol_01', 'cross_validation_learning_curve', 'metrics_ex_01', 'feature_selection_introduction', 'parameter_tuning_sol_03', 'linear_models_regularization', 'parameter_tuning_randomized_search', 'linear_models_sol_04', 'cross_validation_baseline', 'parameter_tuning_ex_03', 'trees_hyperparameters', 'ensemble_ex_04', 'ensemble_bagging', 'linear_models_ex_03', 'trees_classification', 'ensemble_adaboost', 'ensemble_random_forest', 'ensemble_introduction', 'ensemble_ex_03', 'linear_models_sol_02', 'parameter_tuning_nested', 'metrics_classification', 'linear_regression_non_linear_link', 'ensemble_hist_gradient_boosting', 'linear_models_sol_03', 'trees_ex_01', 'cross_validation_ex_01', 'linear_regression_without_sklearn', 'linear_models_feature_engineering_classification', 'feature_selection_sol_01', 'cross_validation_sol_02', 'trees_regression', 'datasets_blood_transfusion', 'parameter_tuning_sol_02', 'ensemble_gradient_boosting', 'ensemble_hyperparameters', 'metrics_sol_02', 'parameter_tuning_ex_02'}
make: *** [Makefile:29: sanity_check_notebooks] Error 1

this is because the old notebooks also need to be removed, which is what this PR does.

f-hafner commented 7 months ago

wrong repository, sorry!