Skip to content

style: apply ruff formatting fixes#167

Open
ShuhaoZhangTony wants to merge 1 commit into
mainfrom
main-dev
Open

style: apply ruff formatting fixes#167
ShuhaoZhangTony wants to merge 1 commit into
mainfrom
main-dev

Conversation

@ShuhaoZhangTony

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings January 9, 2026 14:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request applies automated ruff formatting fixes to Python files across the codebase, focusing primarily on standardizing import statement ordering according to PEP 8 conventions. The changes reorganize imports to group standard library imports first, followed by third-party library imports, and then local/relative imports, with alphabetical sorting within each group. Additionally, the PR removes duplicate import statements and normalizes blank lines between import groups.

Key Changes:

  • Reorganized import statements to follow PEP 8 conventions (stdlib → third-party → local)
  • Removed duplicate imports (e.g., duplicate FontProperties imports in multiple files)
  • Standardized blank line spacing between import groups
  • Applied alphabetical sorting within import groups

Reviewed changes

Copilot reviewed 262 out of 262 changed files in this pull request and generated 557 comments.

Show a summary per file
File Description
tools/data_manager.py Reordered pathlib import after standard library imports
benchmark/scripts/*/groupLine.py Standardized matplotlib import ordering, removed duplicate FontProperties
benchmark/scripts//groupBar.py Alphabetized matplotlib.ticker imports (LinearLocator, LogLocator)
benchmark/scripts/*/drawTogether.py Comprehensive import reorganization with proper grouping
benchmark/scripts/*/OoOCommon.py Removed duplicate imports and reorganized all imports
benchmark/scripts/downstream_combine/*.py Reorganized imports with proper stdlib/third-party/local grouping
benchmark/scripts/PyAMM/tutorial.ipynb Reordered torch/PyAMM imports in notebook cells
benchmark/scripts/DownstreamCCA_static_lazy/*.ipynb Standardized import ordering in notebook code cells

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

import pandas as pd
import pylab
from matplotlib.font_manager import FontProperties
from autoParase import *

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.

Copilot uses AI. Check for mistakes.
import pandas as pd
import pylab
from matplotlib.font_manager import FontProperties
from autoParase import *

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.

Copilot uses AI. Check for mistakes.
import pandas as pd
import pylab
from matplotlib.font_manager import FontProperties
from autoParase import *

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.

Copilot uses AI. Check for mistakes.
import pandas as pd
import pylab
from matplotlib.font_manager import FontProperties
from autoParase import *

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.

Copilot uses AI. Check for mistakes.
import pandas as pd
import pylab
from matplotlib.font_manager import FontProperties
from autoParase import *

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.
Import pollutes the enclosing namespace, as the imported module autoParase does not define 'all'.

Copilot uses AI. Check for mistakes.
import os

import matplotlib
import matplotlib.patches as patches

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import of 'patches' is not used.

Copilot uses AI. Check for mistakes.
import os

import matplotlib
import matplotlib.patches as patches

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import of 'patches' is not used.

Copilot uses AI. Check for mistakes.
import os

import matplotlib
import matplotlib.patches as patches

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import of 'patches' is not used.

Copilot uses AI. Check for mistakes.
import os

import matplotlib
import matplotlib.patches as patches

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import of 'patches' is not used.

Copilot uses AI. Check for mistakes.
import os

import matplotlib
import matplotlib.patches as patches

Copilot AI Jan 9, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Import of 'patches' is not used.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants