Skip to content

gh-103000: Optimise dataclasses asdict/astuple for common types #103005

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.

Sign up for GitHub

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

Merged
AlexWaygood merged 15 commits into from
Apr 10, 2023
Merged

gh-103000: Optimise dataclasses asdict/astuple for common types #103005

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
0af5e1d
Special case 'atomic' types that are not deepcopied
DavidCEllis Mar 20, 2023
2c76ef8
Special case dict_factory=dict
DavidCEllis Mar 20, 2023
c58017c
Add comment explaining _ATOMIC_TYPES
DavidCEllis Mar 20, 2023
8fdbdb2
Merge branch 'python:main' into faster_dataclasses_serialize
DavidCEllis Mar 21, 2023
4730a13
Merge branch 'python:main' into faster_dataclasses_serialize
DavidCEllis Mar 24, 2023
3a650a1
Reorder atomic types and clarify intention
DavidCEllis Mar 24, 2023
b6d8a6f
Update Lib/dataclasses.py
DavidCEllis Mar 24, 2023
bf320be
Update Lib/dataclasses.py
DavidCEllis Mar 24, 2023
6280f2e
Convert _ATOMIC_TYPES to frozenset
DavidCEllis Mar 24, 2023
7343fe6
Correction - complex and bytes are not JSON serializable (by default …
DavidCEllis Mar 24, 2023
b77c17e
Remove dict special case for now
DavidCEllis Mar 24, 2023
0209f3c
Revert function call skip for _asdict_inner
DavidCEllis Mar 24, 2023
986371c
Revert function call skip for _astuple_inner
DavidCEllis Mar 24, 2023
801af82
📜🤖 Added by blurb_it.
blurb-it[bot] Mar 24, 2023
903b2fb
Update Misc/NEWS.d/next/Library/2023-03-24-20-49-48.gh-issue-103000.6…
AlexWaygood Apr 10, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove dict special case for now
  • Loading branch information
@DavidCEllis
DavidCEllis committed Mar 24, 2023
commit b77c17ed417bb3621bf970a16a1ae6a88547ae54
23 changes: 7 additions & 16 deletions Lib/dataclasses.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1316,22 +1316,13 @@ class C:

def _asdict_inner(obj, dict_factory):
if _is_dataclass_instance(obj):
if dict_factory is dict:
result = {}
for f in fields(obj):
value = getattr(obj, f.name)
if type(value) not in _ATOMIC_TYPES:
value = _asdict_inner(value, dict_factory)
result[f.name] = value
return result
else:
result = []
for f in fields(obj):
value = getattr(obj, f.name)
if type(value) not in _ATOMIC_TYPES:
value = _asdict_inner(value, dict_factory)
result.append((f.name, value))
return dict_factory(result)
result = []
for f in fields(obj):
value = getattr(obj, f.name)
if type(value) not in _ATOMIC_TYPES:
value = _asdict_inner(value, dict_factory)
result.append((f.name, value))
return dict_factory(result)
elif isinstance(obj, tuple) and hasattr(obj, '_fields'):
# obj is a namedtuple. Recurse into it, but the returned
# object is another namedtuple of the same type. This is
Expand Down