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

Faster attributes #64

Closed
wants to merge 324 commits into from
Closed

Faster attributes #64

wants to merge 324 commits into from

Conversation

mmatera
Copy link
Contributor

@mmatera mmatera commented Nov 13, 2021

This PR contains the first part of the changes in the logic included in PR #61

rocky and others added 30 commits August 14, 2021 15:20
Other small changes to wording.
Mark modules that don't get documented.
Add timing decorator that can be used in profiling
* Put builtins in alphabetic order
* Add summary text for some built ins
* No hard line breaks on doc text
* Move non-showing test code in Rationalize out to a pytest
Rationalize result that are an integer stay int
All of this will be important in fixing ReadList
@TiagoCavalcante
Copy link
Contributor

@mmatera I think all these changes except the last can the combined with attributes-bitset (I'm not sure about the last).

@mmatera
Copy link
Contributor Author

mmatera commented Dec 1, 2021

@TiagoCavalcante I think so. Let's try to merge it with this one into another branch to see how they get on together.

@mmatera
Copy link
Contributor Author

mmatera commented Jan 17, 2022

This PR is deprecated.

@mmatera mmatera closed this Jan 17, 2022
@mmatera mmatera deleted the faster_attributes branch January 17, 2022 02:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants