-
Notifications
You must be signed in to change notification settings - Fork 15
Get rid of OrderedDicts #233
Comments
I could do that, if no one has started yet |
Welcome @cedric-audy, feel free to take it, no one is working on this atm. |
@cedric-audy are you working on this? or may I begin working on it now? |
hi there, @sr-gi I searched the codebase for the term |
Hi @ritikramuka, happy to see you're showing interest to contribute to teos. The only instances of I wanted to let you known, though, that this codebase will be deprecated in the upcoming weeks, in favor of https://github.com/sr-gi/rusty-teos (the Rust port of it). I'm planning on tackling some issues in this Python version and leave it at a final stable version, but all future development may only take place in Having said that, you're welcome to tackle this if you'd like to get familiar with the codebase, and more than welcome to contribute to |
Hi @sr-gi I tried to install all the requirements listed in CONTRIBUTING.md but running pytest gives the below
then I tried installing requirements.txt which contains plyvel package but got this error
|
The minimum required Python version will be bumped to 3.7 with release v0.1.1.
With Python 3.7,
dicts
preserve inclusion order, so there will be no need forOrderedDicts
anymore within the codebase (given that the former are much faster).The text was updated successfully, but these errors were encountered: