search

thenineteen / Semiology-Visualisation-Tool

Data driven 3D brain visualisation of semiology. Semiology to anatomy translator based on over 4600 patients from 309 peer-reviewed articles.
MIT License
9 stars 6 forks source link

PendingDeprecationWarning: Use 'tree.iter()' or 'list(tree.iter())' instead #188

Closed thenineteen closed 4 years ago

thenineteen commented 4 years ago 
PendingDeprecationWarning: This method will be removed in future versions.  Use 
'tree.iter()' or 'list(tree.iter())' instead.
  for elem in self.tree.iter() if Element_has_iter else self.tree.getiterator():

This warning comes up during testing. Similar to #179 I can't find where this occurs during debugging. Any help would be appreciated.

thenineteen commented 4 years ago

one function that returns this is:

all_localisations()

thenineteen commented 4 years ago

@sadeghs how does one go about chasing the functions that result in the above warning? I can't figure what the best debug method is when the call stack doesn't trace this warning

sadeghs commented 4 years ago

If it's not in your solution then it's probably in your dependencies. If you can search your deps, if not then it's not the end of the world, when you update your deps the warning should go away hopefully.

thenineteen commented 4 years ago 
 maybe a year and a half ago we only had xlrd as a reader, so it was a non-issue. But we've since added openpyxl, pyxlsb, and odfpy as readers, I think without using machinery similar to the writer side for respecting options.

from

https://github.com/pandas-dev/pandas/issues/34252

and

https://stackoverflow.com/questions/60288732/pandas-read-excel-returns-pendingdeprecationwarning