Apparent C-syntax compiling errors about micropython-ulab HOT 6 CLOSED

Update: I can now compile successfully for the unix, pybd_sf6 and pybv11 ports. The issue was on my end (I finally understood that I need to do edit the mpconfigport.h inside each port directory. Sorry for the noise!

from micropython-ulab.

Thanks for posting the fix, too! Your original post indicates that the manual was not clear enough. I will extend the section on compilation.

from micropython-ulab.

No problem. Constructive criticism: speaking as an inexperienced micropython compiler, it was a bit disorienting that the link to compilation instructions for micropython-ulab points to a section in the middle of the micropython-usermod documentation: it was not really clear how far back up I had to read the documentation if all I wanted to do was compile ulab.

In my opinion it would really help if the readme page for micropython-ulab just listed a simple series of steps to follow, e.g., (1) git clone micropython; (2) git clone micropython-ulab; (3) edit mpconfigport.h in the micropython/ports/xxx directory; (4) install cross-compiler; (5) run make command...

PS - IMHO ulab is the best improvement to micropython in years, and the quality of the micropython-ulab and micropython-usermod documentation makes a big difference for many users. Thanks again!

from micropython-ulab.

No problem. Constructive criticism: speaking as an inexperienced micropython compiler, it was a bit disorienting that the link to compilation instructions for micropython-ulab points to a section in the middle of the micropython-usermod documentation: it was not really clear how far back up I had to read the documentation if all I wanted to do was compile ulab.

In my opinion it would really help if the readme page for micropython-ulab just listed a simple series of steps to follow, e.g., (1) git clone micropython; (2) git clone micropython-ulab; (3) edit mpconfigport.h in the micropython/ports/xxx directory; (4) install cross-compiler; (5) run make command...

This is a fair point. I will update the README on the front page. By the way, I am more than happy to accept pull requests;)

from micropython-ulab.

from micropython-ulab.

On Nov 4, 2019, at 19:59, Zoltán Vörös @.***> wrote: This is a fair point. I will update the README on the front page. By the way, I am more than happy to accept pull requests;)
I'm not truly familiar with the whole git pull/push/merge workflow. If I wanted to propose edits to the readme, I assume I need to git clone, make edits, git commit, and then send a push request to your micropython-ulab repo? I'll read up on that again and make an attempt.

You would have to create a fork, which is then yours. Then you implement changes on that fork (you would probably pull that, modify the files on your computer, and then push it back to the server), and once you are satisfied with the modifications, you press the pull request button, which will then notify the original copy (i.e. v923z/micropython-ulab). I can then simply accept the changes, which then become part of my branch.

But i have already updated the README. Let me know if you would still like to extend it. And thanks for the comments, I really appreciate them!

from micropython-ulab.