Remove dowhy_causal_discovery_example from tests (#643)

* Remove dowhy_causal_discovery_example from tests

Remove dowhy_causal_discovery_example.ipynb from advanced tests that are run daily.  It is failing due to a cdt install problem.  Temporarily disabling since we are switching CD implementations.

Signed-off-by: emrekiciman <emrek@microsoft.com>
Signed-off-by: Emre Kiciman <emrek@microsoft.com>

* Update test_notebooks.py

removed spaces that were causing a formatting error

Signed-off-by: emrekiciman <emrek@microsoft.com>
Signed-off-by: Emre Kiciman <emrek@microsoft.com>

* added code for ignoring specified notebook tests

Signed-off-by: Emre Kiciman <emrek@microsoft.com>

Signed-off-by: emrekiciman <emrek@microsoft.com>
Signed-off-by: Emre Kiciman <emrek@microsoft.com>
This commit is contained in:
emrekiciman 2022-09-28 00:38:09 -07:00 коммит произвёл GitHub
Родитель db953a63aa
Коммит 385d916e6a
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 10 добавлений и 3 удалений

Просмотреть файл

@ -15,8 +15,6 @@ advanced_notebooks = [
"dowhy_refutation_testing.ipynb", "dowhy_refutation_testing.ipynb",
"dowhy_lalonde_example.ipynb", "dowhy_lalonde_example.ipynb",
"lalonde_pandas_api.ipynb", "lalonde_pandas_api.ipynb",
# requires Rpy2 for causal discovery
"dowhy_causal_discovery_example.ipynb",
# will be removed # will be removed
"dowhy_optimize_backdoor_example.ipynb", "dowhy_optimize_backdoor_example.ipynb",
# applied notebook, not necessary to test each time # applied notebook, not necessary to test each time
@ -35,6 +33,13 @@ advanced_notebooks = [
"gcm_401k_analysis.ipynb", "gcm_401k_analysis.ipynb",
] ]
ignore_notebooks = [
# requires Rpy2 for causal discovery
# daily tests of dowhy_causal_discovery_example.ipynb are failing due to cdt/rpy2 config.
# comment out, since we are switching causal discovery implementations
"dowhy_causal_discovery_example.ipynb"
]
# Adding the dowhy root folder to the python path so that jupyter notebooks # Adding the dowhy root folder to the python path so that jupyter notebooks
# can import dowhy # can import dowhy
if "PYTHONPATH" not in os.environ: if "PYTHONPATH" not in os.environ:
@ -86,7 +91,9 @@ def test_confounder_notebook():
""" """
parameter_list = [] parameter_list = []
for nb in notebooks_list: for nb in notebooks_list:
if nb in advanced_notebooks: if nb in ignore_notebooks:
continue
elif nb in advanced_notebooks:
param = pytest.param(nb, marks=[mark.advanced, mark.notebook], id=nb) param = pytest.param(nb, marks=[mark.advanced, mark.notebook], id=nb)
else: else:
param = pytest.param(nb, marks=[mark.notebook], id=nb) param = pytest.param(nb, marks=[mark.notebook], id=nb)