Allow propcache objects to be utilized in cython#130
Allow propcache objects to be utilized in cython#130Vizonex wants to merge 32 commits intoaio-libs:masterfrom
Conversation
for more information, see https://pre-commit.ci
…into cython-support
for more information, see https://pre-commit.ci
CodSpeed Performance ReportMerging #130 will not alter performanceComparing Summary
|
Codecov Report✅ All modified and coverable lines are covered by tests. ❌ Your project status has failed because the head coverage (97.44%) is below the target coverage (98.20%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #130 +/- ##
==========================================
+ Coverage 97.63% 97.67% +0.03%
==========================================
Files 17 18 +1
Lines 847 861 +14
Branches 44 44
==========================================
+ Hits 827 841 +14
Misses 12 12
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…into cython-support
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
|
I'm going to close this for now I might rethink about this in the future but for now my desire for cython to have this is kinda dead. However if someone needs this in cython I might just let people know that if you want it copying it off should be doable and to give citations as to where it came from. |
What do these changes do?
After doing a bit of research and digging I've discovered that it is possible to use propcache's member descriptors in Cython and I even wrote a little code snippet to prove it works. I think some low level programmers may take interest in this feature and I am looking to use it myself in my own projects but this might be the last thing needed if we were looking to embed yarl's URL object more deeply in Cython (Which is one of my goals which was to cimport the URL object to Cython to improve performance of aiohttp's http_parser, not to mention the new Multidict CAPI that I am really excited to start using in Cython).
Are there changes in behavior for the user?
Cython Programmers can now utilize propcache. This would benefit projects like yarl or other low level libraries that may use cached properties. I'll see about documenting these features here in a bit if we decide this is something new to explore. I don't want to flush my work down the toilet after writing documentation if this idea is rejected.
Related issue number
Checklist