GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | ||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | ||||||||
GeezerVison is showing github.com — bpo-1635741: port pickle to multi-phase init (PEP 489) by koubaa · Pull Request #23188 · python/cpython Conversation
|
|
@vstinner There are more types to convert to heap types. Even so I think these changes are too much and they should be staged in multiple PRs. I'm wondering how you think I should go about dividing it. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
711f040 to
72614d6
Compare
|
This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 5 days |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
https://bugs.python.org/issue1635741 is closed. What is the status of this PR? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
The change is still relevant, but should use a new issue number. Moreover, the SC asked to put the conversion of static types to heap types on hold. @encukou and @erlend-aasland wrote https://peps.python.org/pep-0687/ which may unblock the situation but it's still a draft. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@koubaa, PEP 687 was accepted. Are you going to follow up this PR? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@erlend-aasland I'd love to - but I doubt I will get time in the near term. Others could use my PRs as a reference in case they find it useful towards doing pep 687 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
All right, I'll pick it up. Thanks for all your work on isolating extension modules. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I'm closing this in order to split it up into multiple PRs. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.