fix/fix_future_warnings

[JasonYu1]

Description

I encountered the following future warnings

FutureWarning: iteritems is deprecated and will be removed in a future version. Use .items instead.
  for i, x in scene_files_nunique.iteritems():

I applied the suggested fix which is to use .items

Pull request recommendations:

• Name your pull request your-development-type/short-description. Ex: feature/read-tiff-files
• [?] Provide relevant tests for your feature or bug fix.

Thanks for contributing!

[evamaxfield]

This looks fine to me! Thanks for contributing!

Assuming all tests pass, will merge.

[codecov-commenter]

Codecov Report

Base: 94.00% // Head: 94.00% // No change to project coverage 👍

Coverage data is based on head (2b1129b) compared to base (25d561e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #453   +/-   ##
=======================================
  Coverage   94.00%   94.00%           
=======================================
  Files          46       46           
  Lines        4021     4021           
=======================================
  Hits         3780     3780           
  Misses        241      241           

Impacted Files	Coverage Δ
aicsimageio/readers/tiff_glob_reader.py	86.77% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ianhi]

Test failures thus far are seemingly unrelated:

=================================== ERRORS ====================================
_ ERROR collecting aicsimageio/tests/readers/extra_readers/test_bioformats_reader.py _
ImportError while importing test module 'D:\a\aicsimageio\aicsimageio\aicsimageio\tests\readers\extra_readers\test_bioformats_reader.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
aicsimageio\readers\bioformats_reader.py:34: in <module>
    import jpype
E   ModuleNotFoundError: No module named 'jpype'

[evamaxfield]

Test failures thus far are seemingly unrelated

they are indeed unrelated but I am wondering why they are getting picked up by the test suite at all. Need to investigate before merge now. Sorry.

[evamaxfield]

I think its the new tox major version release. https://pypi.org/project/tox/

Can you try pinning tox==3.27.1 in setup.py and pushing

[evamaxfield]

@JasonYu1 you pinned tox>=, can you pin tox==

[evamaxfield]

Thanks! Looks like that was the issue. Will wait for second approval but good from me!