Skip to content

Commit df05f8d

Browse files
committed
Post final QA fixes
1 parent 78dc8f3 commit df05f8d

File tree

9 files changed

+26
-57
lines changed

9 files changed

+26
-57
lines changed

python-namedtuple/database.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
def get_column_names(table):
2+
if table == "passenger":
3+
return ("id", "first_name", "last_name", "class")
4+
raise ValueError(f"unknown table {table}")
5+
6+
7+
def get_passenger_by_id(passenger_id):
8+
if passenger_id == 1234:
9+
return (1234, "John", "Doe", "Business")
10+
raise ValueError(f"no record with id={passenger_id}")

python-namedtuple/employees.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import csv
22
from collections import namedtuple
33

4-
with open("employees.csv", "r", encoding="utf-8") as csv_file:
4+
with open("employees.csv", mode="r", encoding="utf-8") as csv_file:
55
reader = csv.reader(csv_file)
66
Employee = namedtuple("Employee", next(reader), rename=True)
77
for row in reader:
88
employee = Employee(*row)
9-
print(employee.name, employee.job, employee.email)
9+
print(employee)

python-namedtuple/namedtuple_dataclass_memory.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,5 @@ class PointDataClass:
1717
dataclass_memory = asizeof.asizeof(PointDataClass(x=1, y=2, z=3))
1818
gain = 100 - namedtuple_memory / dataclass_memory * 100
1919

20-
print(f"namedtuple: {namedtuple_memory} bytes ({gain:.2f}% lower)")
20+
print(f"namedtuple: {namedtuple_memory} bytes ({gain:.2f}% smaller)")
2121
print(f"data class: {dataclass_memory} bytes")

python-namedtuple/passenger.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
from collections import namedtuple
2+
3+
from database import get_column_names
4+
5+
Passenger = namedtuple("Passenger", get_column_names("passenger"), rename=True)

python-namedtuple/performance.py

Lines changed: 0 additions & 51 deletions
This file was deleted.

python-namedtuple/person_dataclass.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,8 @@ class Person:
1111

1212
def __iter__(self):
1313
return iter(astuple(self))
14+
15+
16+
jane = Person("Jane", 25, 1.75, 67)
17+
for field in jane:
18+
print(field)

python-namedtuple/requirements.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
pympler

python-namedtuple/subclass.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,15 +11,15 @@ class Person(BasePerson):
1111

1212
__slots__ = ()
1313

14-
def __repr__(self):
14+
def __str__(self):
1515
return f"Name: {self.name}, age: {self.age} years old."
1616

1717
@property
1818
def age(self):
1919
return (date.today() - self.birthdate).days // 365
2020

2121

22-
Person.__doc__
22+
print(Person.__doc__)
2323

2424
jane = Person("Jane", date(1996, 3, 5))
2525
print(jane.age)

python-namedtuple/typed_namedtuple_time.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ class PointTypedNamedTuple(NamedTuple):
3434
typed_namedtuple_time = average_time(
3535
PointTypedNamedTuple(x=1, y=2, z=3), time_structure
3636
)
37-
gain = typed_namedtuple_time / namedtuple_time
3837

3938
print(f"namedtuple: {namedtuple_time:.2f} ns")
4039
print(f"typing.NamedTuple: {typed_namedtuple_time:.2f} ns")

0 commit comments

Comments
 (0)