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

Initial app bootstrapping with cookie cutter #2

Open
wants to merge 9 commits into
base: master
from

Conversation

@mscosti
Copy link

mscosti commented Oct 14, 2019

I finally might have some time for projects and wanted to get the ball rolling here on this library.

This is 99% Generated from the Cookie Cutter project, following the learn guide for what parameters I should be using. Please let me know if I misunderstood any of the parameters, especially around attribution.

The other 1% is an incredibly pared down version of this example from the ESP32SPI lib, just to kick things off. I picture for some foreseeable time the library code and API will be in flux to allow for quicker development and changing ideas until we land in a spot where we all feel comfortable with the API and structure.

Add pared down version of wsgi application from esp32 example to start
@ladyada

This comment has been minimized.

Copy link
Member

ladyada commented Oct 14, 2019

hiya i dont think ya wanna be committng the docs folders, thats generated for you by sphinx (i think?)

@mscosti

This comment has been minimized.

Copy link
Author

mscosti commented Oct 14, 2019

Yeah, I just noticed that as well. It looks like I'm missing some files from cookie cutter (like a .gitignore). Looking into that now. I Installed it from pip, do you know if the latest build is available there? cookiecutter -V is saying I have version 1.6.0

@ladyada

This comment has been minimized.

@ladyada

This comment has been minimized.

@brentru

This comment has been minimized.

Copy link
Member

brentru commented Oct 14, 2019

@mscosti You may want to test this (and your code in ESP32SPI, too) with the latest version of nina-fw (v1.4.0) on your ESP32.

We've updated the ESP-IDF version which builds nina-fw using ESP-IDF v3.3 (we were previously using ESP-IDF v3.2).

@mscosti

This comment has been minimized.

Copy link
Author

mscosti commented Oct 14, 2019

Looks like I accidentally deleted all the dot files after running cookie cutter, d'oh!

  • Re ran cookie cutter to get the following files back: .gitignore, .pylintrc, .readthedocs.yml, and .travis.yml

  • Also removed the generated _build folder from docs.

  • Look into travis

@brentru Neat! I Will look into upgrading firmware soon, happy to see the guide has been updated to include instructions for external breakout boards :)

mscosti added 6 commits Oct 14, 2019
@mscosti

This comment has been minimized.

Copy link
Author

mscosti commented Oct 19, 2019

@brentru Upgraded firmware on my pyPortal to v1.4 and tested it with no issues on new firmware.

I also added a starter basic simpleTest example.

I think it might be good to go to at least start us off, pending any other comments. @ladyada @brentru

@siddacious

This comment has been minimized.

Copy link

siddacious commented Oct 22, 2019

@mscosti I'll take a look at this tomorrow

@siddacious

This comment has been minimized.

Copy link

siddacious commented Nov 7, 2019

@mscosti I copied your fork over to my Metro M4 + Airlift shield, along with a fresh copy of Adafruit_CircuitPython_ESP32SPI and get the following output and error when trying to run the example application. Looking at the code I'm honestly unsure what it's complaining about; as far as I can tell you're just setting an instance attribute?

It's been a while since I worked with this code so it's entirely possible that I'm overlooking something or set up something incorrectly.

image

@mscosti

This comment has been minimized.

Copy link
Author

mscosti commented Nov 15, 2019

thanks for giving it a try! were you in this branch or my master branch? my master branch has some more development done that will be in a future PR. ill plan on looking into reproducing this error this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.