Warnings in Unit Test

[rogerkuou]

There is a warning not in unit test. We need to solve this by either:

1. Fix in the implementation, or
2. Fix in the unit test, or
3. Fix by update the sample csv file

======================================= warnings summary ========================================
tests/test_io.py::TestFromCSV::test_readcsv_dims
tests/test_io.py::TestFromCSV::test_readcsv_vars
tests/test_io.py::TestFromCSV::test_readcsv_output_chunksize
tests/test_io.py::TestFromCSV::test_readcsv_custom_var_name
tests/test_io.py::TestFromCSV::test_readcsv_list_coords
tests/test_io.py::TestFromCSV::test_readcsv_custom_coords
tests/test_io.py::TestFromCSV::test_readcsv_timevalues
tests/test_io.py::TestFromCSV::test_readcsv_timevalues
tests/test_io.py::TestFromCSV::test_readcsv_dtypes
  /home/oku/miniforge3/envs/mobyle/lib/python3.11/site-packages/dask/dataframe/core.py:3828: UserWarning: Dask currently has limited support for converting pandas extension dtypes to arrays. Converting string to object dtype.

[vanlankveldthijs]

Today Ou and me discussed the warnings that occur when running the unit tests.

Firstly, the warnings mentioned above don't occur (anymore).

There are three types of warnings that occur now.
Two are related to LAMBDA.py:

• Warnings about non-escaped "\" characters in some docstrings
• A warning that numpy.matlib is deprecated and should be replaced by ndarray.
  These should all be fixed when LAMBDA.py is either updated, or phased out.

Finally, there are some warnings about the Datasets created in test_classification.py.
Apparently, xarray Datasets no longer like storing time variables in non-nanosecond precision. However, the warning mentions "This behavior can eventually be relaxed in xarray".
We decided to not take action on these warnings, because an xarray update should eventually resolve them.

[SarahAlidoost]

@rogerkuou and @vanlankveldthijs To see which warnings are still showing up, you can check the workflow results—for example, from this PR. The warning mentioned in this issue UserWarning: Dask currently has limited support for converting pandas extension dtypes to arrays. Converting string to object dtype is still there. The other warnings related to LAMBDA.py is for depsi_group environment, not the stmtools environment defined in this repo.

[rogerkuou]

Thanks @SarahAlidoost. Sorry I overlooked this and thought this was about the DePSI repo when I was discussing with Thijs. Indeed it is about stmtools. I will reopen it.

@vanlankveldthijs I plan to have a look into stmtool tomorrow by myself. I suspect this is caused by the irregular fields in the sample csv data in the unit test.

[rogerkuou]

@SarahAlidoost, as we discussed yesterday, if you have time please feel free to take over this issue.