Deprecation warning for xlsx

[aloukina]

We get the following deprecation warning when using xlsx:

python3.7/site-packages/xlrd/xlsx.py:266: PendingDeprecationWarning: This method will be removed in future versions. Use 'tree.iter()' or 'list(tree.iter())' instead.

[mulhod]

@aloukina Can you point me to a situation where this happens. So far, I don't see it happening anywhere when I run the tests and I don't even see it happening when I open an IPython session and try to use read_excel directly to trigger a warning.

[aloukina]

@mulhod
[edited since we established that the warnings were not aligned with test names due to parallelization]

[mulhod]

It's strange. I don't see any warnings when I run the tests. For example:

$ nosetests --nologcapture tests/test_reader.py:TestDataReader.test_add_containers_duplicate_keys
.
----------------------------------------------------------------------
Ran 1 test in 0.156s

OK

[mulhod]

Would the deprecation warning be expected to show up in the CI build? https://travis-ci.org/EducationalTestingService/rsmtool/jobs/643938322?utm_medium=notification&utm_source=github_status

I don't think I see them there either.

[desilinguist]

@mulhod are you using the same version of xlrd as @aloukina ?

[mulhod]

@desilinguist Yes. I am:

$ conda list -p ../rsm_env | grep xlrd
xlrd                      1.2.0                    pypi_0    pypi

[aloukina]

This might be relevant: pandas-dev/pandas#28547

[aloukina]

We are calling read_excel and to_excel, not calling xlrd directly.

[desilinguist]

yeah, I realized that as soon as I posted and then deleted my message :)

[aloukina]

Closing because we are not seeing this warning in tests anymore.