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

PEP 558: Update PEP for implementation changes and PEP 667 #2060

Merged

Conversation

ncoghlan
Copy link
Contributor

@ncoghlan ncoghlan commented Aug 26, 2021

  • address remaining review comments from the July threads
  • Rationale section renamed to Motivation
  • Design Discussion section renamed to Rationale and Design Discussion
  • kind enum is guaranteed to be at least 32 bits
  • fast refs mapping is stored on the underlying frame
  • delay initial cache refresh for each proxy instance to the
    first operation that needs it
  • be specific about which operations always update the cache, and which
    update it if it hasn't been updated by this proxy instance
  • eliminate more mentions of the old "dynamic snapshot" terminology
  • add new rationale/discussion section covering PEP 667 (including
    how the PEP 558 implementation could be turned into a PEP 667
    implementation if desired)
  • make it clearer that proxy instances are ephemeral (lots of
    stale phrasing with "the" dating from when they stored on the frame)

* address remaining review comments from the July threads
* Rationale section renamed to Motivation
* Design Discussion section renamed to Rationale and Design Discussion
* kind enum is guaranteed to be at least 32 bits
* fast refs mapping is stored on the underlying frame
* delay initial cache refresh for each proxy instance to the
  first operation that needs it
* be specific about which operations always update the cache, and which
  update it if it hasn't been updated by this proxy instance
* eliminate more mentions of the old "dynamic snapshot" terminology
* add new rational/discussion section covering PEP 667 (including
  how the PEP 558 implementation could be turned into a PEP 667
  implementation if desired)
* make it clearer that proxy instances are ephemeral (lots of
  stale phrasing with "the" dating from when they stored on the frame)
pep-0558.rst Outdated Show resolved Hide resolved
@ncoghlan ncoghlan self-assigned this Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants