Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Follow PEP8 standard on library imports. import has higher `precede… #75

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HarmonicReflux
Copy link
Contributor

For the pedantic and the ones who follow PEP8.

I modified the top-level import statements for the section containing the Python code.

@HarmonicReflux HarmonicReflux self-assigned this Oct 29, 2024
Copy link
Collaborator

@dalonsoa dalonsoa left a comment

Choose a reason for hiding this comment

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

Well, it would be for the PEPdantic 😆 . Jokes aside, it is important to honour standards when they exist, so good catch.

Copy link
Contributor

@AdrianDAlessandro AdrianDAlessandro left a comment

Choose a reason for hiding this comment

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

I've only just seen this PR, but it got approved and never merged

Comment on lines 288 to +291
from dataclasses import dataclass
import numpy as np
from scipy.stats import norm
from typing import Callable, Tuple
import numpy as np
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, this should be slightly different. import x ... comes before from x import ... but also, in-built packages should be imported in a block before external ones and the blocks should be separated by a newline.

from dataclasses import dataclass
from typing import Callable, Tuple

import numpy as np
from scipy.stats import norm

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.

3 participants