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

Formatting python scripts with black auto-formatter #73

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

Conversation

pavanstrand
Copy link
Collaborator

Formatting changes with black auto-formatter.

abs(x[1] - pos_),
abs(min(x[1] + len(x[2]), pos_ + len(ref_)) - max(x[1], pos_))) <= max_dist)), record_set))) > 0:
chrom_, pos_, ref_, alt_ = push_left_var(ref_fasta, chrom, pos, ref, alt)
if (

Choose a reason for hiding this comment

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

@pavanstrand the formatting seems to have gone crazy here. Would be better with a bit of refactoring here.

):
records[loc].append(
[
chrom,

Choose a reason for hiding this comment

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

Would the formatter have options to control for such formatting? For example, the number of variables to put on one line (here, it seems to be 1)?

@marghoob
Copy link

@pavanstrand thanks for this. Did you run this branch on the data to ensure that the output VCF is identical to output before formatting?

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.

None yet

2 participants