Skip to content

Commit d29a9dc

Browse files
committed
Ruff format
1 parent 908d0b7 commit d29a9dc

12 files changed

+46
-51
lines changed

ffcx/codegeneration/C/__init__.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
11
"""Generation of C code."""
2-
from ffcx.codegeneration.C import form, expressions, integrals, file # noqa
2+
3+
from ffcx.codegeneration.C import form, expressions, integrals, file # noqa
4+
35
suffixes = (".h", ".c")

ffcx/codegeneration/codegeneration.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,9 @@ class CodeBlocks(typing.NamedTuple):
3737
file_post: list[tuple[str, str]]
3838

3939

40-
def generate_code(ir: DataIR,
41-
options: dict[str, int | float | npt.DTypeLike]) -> tuple[CodeBlocks,
42-
tuple[str, str]]:
40+
def generate_code(
41+
ir: DataIR, options: dict[str, int | float | npt.DTypeLike]
42+
) -> tuple[CodeBlocks, tuple[str, str]]:
4343
"""Generate code blocks from intermediate representation."""
4444
logger.info(79 * "*")
4545
logger.info("Compiler stage 3: Generating code")

ffcx/codegeneration/cpp/__init__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
from . import form, expressions, integrals, file # noqa
1+
from . import form, expressions, integrals, file # noqa
22

33
suffixes = (".hpp", None)

ffcx/codegeneration/cpp/cpp_implementation.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@
3333
"bessel_j": "std::cyl_bessel_j",
3434
"conj": "std::conj",
3535
"real": "std::real",
36-
"imag": "std::imag"}
36+
"imag": "std::imag",
37+
}
3738

3839

3940
def build_initializer_lists(values):
@@ -49,6 +50,7 @@ def build_initializer_lists(values):
4950

5051
class CppFormatter:
5152
"""C++ formatter."""
53+
5254
def __init__(self, scalar) -> None:
5355
"""Initialise."""
5456
self.scalar_type = "T"

ffcx/codegeneration/cpp/expressions.py

+16-26
Original file line numberDiff line numberDiff line change
@@ -41,14 +41,12 @@ def generator(ir, options):
4141
d["tabulate_expression"] = CF.c_format(parts)
4242

4343
if len(ir.original_coefficient_positions) > 0:
44-
d[
45-
"original_coefficient_positions"
46-
] = f"original_coefficient_positions_{ir.name}"
44+
d["original_coefficient_positions"] = f"original_coefficient_positions_{ir.name}"
4745
n = len(ir.original_coefficient_positions)
4846
originals = ", ".join(str(i) for i in ir.original_coefficient_positions)
49-
d[
50-
"original_coefficient_positions_init"
51-
] = f"static int original_coefficient_positions_{ir.name}[{n}] = {{{originals}}};"
47+
d["original_coefficient_positions_init"] = (
48+
f"static int original_coefficient_positions_{ir.name}[{n}] = {{{originals}}};"
49+
)
5250

5351
else:
5452
d["original_coefficient_positions"] = "NULL"
@@ -78,9 +76,9 @@ def generator(ir, options):
7876
if len(ir.coefficient_names) > 0:
7977
names = ", ".join(f'"{name}"' for name in ir.coefficient_names)
8078
n = len(ir.coefficient_names)
81-
d[
82-
"coefficient_names_init"
83-
] = f"static const char* coefficient_names_{ir.name}[{n}] = {{{names}}};"
79+
d["coefficient_names_init"] = (
80+
f"static const char* coefficient_names_{ir.name}[{n}] = {{{names}}};"
81+
)
8482

8583
d["coefficient_names"] = f"coefficient_names_{ir.name}"
8684
else:
@@ -90,9 +88,9 @@ def generator(ir, options):
9088
if len(ir.constant_names) > 0:
9189
names = ", ".join(f'"{name}"' for name in ir.constant_names)
9290
n = len(ir.constant_names)
93-
d[
94-
"constant_names_init"
95-
] = f"static const char* constant_names_{ir.name}[{n}] = {{{names}}};"
91+
d["constant_names_init"] = (
92+
f"static const char* constant_names_{ir.name}[{n}] = {{{names}}};"
93+
)
9694
d["constant_names"] = f"constant_names_{ir.name}"
9795
else:
9896
d["constant_names_init"] = ""
@@ -103,9 +101,7 @@ def generator(ir, options):
103101
# FIXME: Should be handled differently, revise how
104102
# ufcx_function_space is generated (also for ufcx_form)
105103
for name, (element, dofmap, cmap_family, cmap_degree) in ir.function_spaces.items():
106-
code += [
107-
f"static ufcx_function_space function_space_{name}_{ir.name_from_uflfile} ="
108-
]
104+
code += [f"static ufcx_function_space function_space_{name}_{ir.name_from_uflfile} ="]
109105
code += ["{"]
110106
code += [f".finite_element = &{element},"]
111107
code += [f".dofmap = &{dofmap},"]
@@ -123,24 +119,18 @@ def generator(ir, options):
123119
for (name, _) in ir.function_spaces.items()
124120
)
125121
n = len(ir.function_spaces.items())
126-
d[
127-
"function_spaces_init"
128-
] = f"ufcx_function_space* function_spaces_{ir.name}[{n}] = {{{fs_list}}};"
122+
d["function_spaces_init"] = (
123+
f"ufcx_function_space* function_spaces_{ir.name}[{n}] = {{{fs_list}}};"
124+
)
129125
else:
130126
d["function_spaces"] = "NULL"
131127
d["function_spaces_init"] = ""
132128

133129
# Check that no keys are redundant or have been missed
134130
from string import Formatter
135131

136-
fields = [
137-
fname
138-
for _, fname, _, _ in Formatter().parse(expressions_template.factory)
139-
if fname
140-
]
141-
assert set(fields) == set(
142-
d.keys()
143-
), "Mismatch between keys in template and in formatting dict"
132+
fields = [fname for _, fname, _, _ in Formatter().parse(expressions_template.factory) if fname]
133+
assert set(fields) == set(d.keys()), "Mismatch between keys in template and in formatting dict"
144134

145135
# Format implementation code
146136
implementation = expressions_template.factory.format_map(d)

ffcx/codegeneration/cpp/file.py

+1-3
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,7 @@ def generator(options):
2929
extra_includes = []
3030
if "_Complex" in options["scalar_type"]:
3131
extra_includes += ["complex"]
32-
d["extra_includes"] = "\n".join(
33-
f"#include <{header}>" for header in extra_includes
34-
)
32+
d["extra_includes"] = "\n".join(f"#include <{header}>" for header in extra_includes)
3533

3634
# Format declaration code
3735
code_pre = (

ffcx/codegeneration/cpp/integrals.py

-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ def generator(ir, options):
2121
logger.info(f"--- type: {ir.expression.integral_type}")
2222
logger.info(f"--- name: {ir.expression.name}")
2323

24-
2524
factory_name = ir.expression.name
2625

2726
# Format declaration

ffcx/codegeneration/numba/__init__.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
from ffcx.codegeneration.numba import form, expressions, integrals, file # noqa
1+
from ffcx.codegeneration.numba import form, expressions, integrals, file # noqa
22

33
suffixes = (None, "_numba.py")

ffcx/codegeneration/numba/expressions.py

+4-6
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,7 @@ def generator(ir, options):
8383
# FIXME: Should be handled differently, revise how
8484
# ufcx_function_space is generated (also for ufcx_form)
8585
for name, (element, dofmap, cmap_family, cmap_degree) in ir.function_spaces.items():
86-
code += [
87-
f"static ufcx_function_space function_space_{name}_{ir.name_from_uflfile} ="
88-
]
86+
code += [f"static ufcx_function_space function_space_{name}_{ir.name_from_uflfile} ="]
8987
code += ["{"]
9088
code += [f".finite_element = &{element},"]
9189
code += [f".dofmap = &{dofmap},"]
@@ -103,9 +101,9 @@ def generator(ir, options):
103101
for (name, _) in ir.function_spaces.items()
104102
)
105103
n = len(ir.function_spaces.items())
106-
d[
107-
"function_spaces_init"
108-
] = f"ufcx_function_space* function_spaces_{ir.name}[{n}] = {{{fs_list}}};"
104+
d["function_spaces_init"] = (
105+
f"ufcx_function_space* function_spaces_{ir.name}[{n}] = {{{fs_list}}};"
106+
)
109107
else:
110108
d["function_spaces"] = "NULL"
111109
d["function_spaces_init"] = ""

ffcx/codegeneration/numba/file.py

-1
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
# old implementation in FFC
99
"""Generate file output for numba."""
1010

11-
1211
import logging
1312
import pprint
1413
import textwrap

ffcx/codegeneration/numba/form.py

+2-4
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ def generator(ir, options):
2929
d["num_coefficients"] = ir.num_coefficients
3030
d["num_constants"] = ir.num_constants
3131

32-
orig_coeff = ', '.join(str(i) for i in ir.original_coefficient_positions)
32+
orig_coeff = ", ".join(str(i) for i in ir.original_coefficient_positions)
3333
d["original_coefficient_position"] = f"[{orig_coeff}]"
3434

3535
cnames = ir.coefficient_names
@@ -59,9 +59,7 @@ def generator(ir, options):
5959
# Check that no keys are redundant or have been missed
6060
from string import Formatter
6161

62-
fields = [
63-
fname for _, fname, _, _ in Formatter().parse(form_template.factory) if fname
64-
]
62+
fields = [fname for _, fname, _, _ in Formatter().parse(form_template.factory) if fname]
6563

6664
for f in fields:
6765
if f not in d.keys():

ffcx/codegeneration/numba/numba_implementation.py

+12-3
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
"""Numba implementation for output."""
2+
23
import ffcx.codegeneration.lnodes as L
34

45

@@ -15,6 +16,7 @@ def build_initializer_lists(values):
1516

1617
class NumbaFormatter:
1718
"""Implementation for numba output backend."""
19+
1820
def __init__(self, scalar) -> None:
1921
"""Initialise."""
2022
self.scalar_type = scalar
@@ -185,9 +187,16 @@ def format_symbol(self, s):
185187

186188
def format_mathfunction(self, f):
187189
"""Format a math function."""
188-
function_map = {"ln": "log", "acos": "arccos", "asin": "arcsin",
189-
"atan": "arctan", "atan2": "arctan2", "acosh": "arccosh",
190-
"asinh": "arcsinh", "atanh": "arctanh"}
190+
function_map = {
191+
"ln": "log",
192+
"acos": "arccos",
193+
"asin": "arcsin",
194+
"atan": "arctan",
195+
"atan2": "arctan2",
196+
"acosh": "arccosh",
197+
"asinh": "arcsinh",
198+
"atanh": "arctanh",
199+
}
191200
function = function_map.get(f.function, f.function)
192201
args = [self.c_format(arg) for arg in f.args]
193202
if "bessel" in function:

0 commit comments

Comments
 (0)