Pebaz / nimporter

Compile Nim Extensions for Python On Import!
MIT License
824 stars 33 forks source link

documentation #1

Closed dullbananas closed 4 years ago

dullbananas commented 4 years ago

we need documentation

dullbananas commented 4 years ago

Also, how did you get this any stars

Pebaz commented 4 years ago

I agree! I am currently working on content for the wiki pages. In regards to the Stars, I posted on the Python and Nim Subreddits so people could see my project!

dullbananas commented 4 years ago

I think i’m going to do a test to compare the speed of Cython and Nimporter.

Pebaz commented 4 years ago

Nice project! Do it! I'd be interested in seeing the results. They should be comparable since Nim compiles to C and Cython is a Python to C compiler. However, since Cython makes use of interpreter calls and Nim does not, the performance may be different.

dullbananas commented 4 years ago

I use PythonAnywhere and I have been unable to install Nim on there because I don't have enough storage

dullbananas commented 4 years ago

I have now been using Repl.it instead. It kinda worked good, but Repl.it has been super buggy and there is a file that is broken. https://repl.it/@shamshung/Performance-Comparison-of-Cython-and-Nimporter

Pebaz commented 4 years ago

Repl.it never loads for me. It shows your project's folder structure, but the code never loads. It may be easier to distribute your results if you create a GitHub repo with the code, run the tests locally and then post the results in the readme. Kind of like how I did here.

dullbananas commented 4 years ago

Maybe I can use GitHub Actions

On Thu, Dec 12, 2019 at 12:25 PM Pebaz notifications@github.com wrote:

Repl.it never loads for me. It shows your project's folder structure, but the code never loads. It may be easier to distribute your results if you create a GitHub repo with the code, run the tests locally and then post the results in the readme. Kind of like how I did here https://github.com/Pebaz/wrap.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Pebaz/nimporter/issues/1?email_source=notifications&email_token=ALL7CZYEVNZQU2PGZZO5BJ3QYKF37A5CNFSM4JXDJFH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGXXY5A#issuecomment-565148788, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALL7CZ6ZRKCGO2CNJFSTWXDQYKF37ANCNFSM4JXDJFHQ .

Pebaz commented 4 years ago

Does Nimporter not work on your local development machine? If so, can you provide your machine details (OS, arch, Python version, etc.) so I can try to replicate?

dullbananas commented 4 years ago

I use an iPad Pro, so I use PythonAnywhere for development. It runs Ubuntu and Python 3.8. I can't install Nim on it because there's not enough storage space, and I don't want to upgrade for more storage.

Pebaz commented 4 years ago

I'm glad there is no issue with Nimporter! ;) I definitely feel you though. Perhaps you could look into AWS Workspaces. Also, I am marking this issue as closed since I am actively adding documentation.

dullbananas commented 4 years ago

I definitely feel you though. Perhaps you could look into AWS Workspaces.

is this free?

Pebaz commented 4 years ago

No, but it would cost around the same per month. An alternative would be to use AWS EC2 to spin up a T2.micro or T2.nano instance and SSH into it. It would give you all the same features of Pythonanywhere without the hubbub and unlimited freedom!

On Thu, Dec 12, 2019, 9:53 PM Dull Bananas notifications@github.com wrote:

I definitely feel you though. Perhaps you could look into AWS Workspaces https://aws.amazon.com/workspaces/.

is this free?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/Pebaz/nimporter/issues/1?email_source=notifications&email_token=AGV4FZ7IEXQGVBVIWK3EPELQYL2MBA5CNFSM4JXDJFH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGYXQ7A#issuecomment-565278844, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGV4FZ7H647XYMYS7LKZKLDQYL2MBANCNFSM4JXDJFHQ .

dullbananas commented 4 years ago

I have successfully been able to run a performance test. Results are here: https://github.com/dullbananas/nimporter-vs-cython/commit/ea52215fa0bde8cee2134afb5eaa48b18f538860/checks?check_suite_id=357999486

Unfortunately, Cython ran about 5 times faster.