Address ruff warnings

[nick-greenwald]

The ruff and flake8 jobs for anms-core, aricodec, and transcoder all show multiple warnings related to whitespace, unused variables, unordered imports, and other benign issues. These should all be addressed. Many can be fixed by simply running ruff check --fix, others will have to be manually fixed. See #299 (comment) for a summary of the ruff findings when it was initially added to the CI.

[BrianSipos]

If these are truly formatting and not functional changes, then this can be included in the release milestone and regression tested.

[nick-greenwald]

If these are truly formatting and not functional changes, then this can be included in the release milestone and regression tested.

In anms-core:

• The E711 warnings say to use is None instead of == None. The only way changing that would be functional is if any classes we use override the __eq__ method and handle None values differently. There are only two instances where we override the __eq__ method and that's in deps/dtnma-ace/src/ace/ari.py, but those overridden methods handle None values as expected.
• All of the F401, F841, and F811 warnings are about unused code, either unused variables, imports or unused redefined methods. Removing dead code won't change the functionality but there is a small chance they could be false positives or could be indicative of a bug (where we should be using the variable/import/method.
• The F403 and F405 warnings are related to star imports, with ruff not being able to enumerate the imports, so those should changes shouldn't affect any of the functionality.
• The remaining W and I errors are whitespace and import ordering and would have no impact on functionality
• Running ruff format to address the ruff format --check failures should have no impact on functionality (there's a small chance it could mess up strings though)

In aricodec:

• There are only a handful of F401 warnings, it'd be easy to verify if the unused imports are actually unused or indicative of a bug
• The remaining W and I errors are whitespace and import ordering and would have no impact on functionality
• Running ruff format to address the ruff format --check failures should have no impact on functionality (there's a small chance it could mess up strings though)

In transcoder:

• There are some F401, F481, and F811 warnings, but it should be easy to verify if the unused variables/imports/method are actually unused or indicative of a bug
• The remaining W and I errors are whitespace and import ordering and would have no impact on functionality
• Running ruff format to address the ruff format --check failures should have no impact on functionality (there's a small chance it could mess up strings though)

If we want to include it in this milestone, running and checking the output of ruff format, as well as running through all the safe ruff check --fix changes should all be formatting changes only. We can change the name of this issue accordingly and create another follow-up issue for unsafe ruff auto-fixes.

[BrianSipos]

Please prepare a PR for these non functional changes and create a new issue for the others, as you recommend.