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

Add _pixelbuf support #42

Closed
wants to merge 12 commits into from
Closed

Add _pixelbuf support #42

wants to merge 12 commits into from

Conversation

rhooper
Copy link
Contributor

@rhooper rhooper commented Dec 2, 2019

No description provided.

.pylintrc Outdated
@@ -18,6 +18,7 @@ ignore-patterns=
#init-hook=

# Use multiple processes to speed up Pylint.
# jobs=1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there is a bunch of commented out changes; just make sure to remove them before you're done.

@siddacious
Copy link
Contributor

@rhooper Since pypixelbuf and the API changes were both merged, is this no longer WIP and ready to merge? I took a look and it's good from what I can tell but it may be wise to have someone more familiar with the code. In the meanwhile I'll do some testing

@rhooper
Copy link
Contributor Author

rhooper commented Jan 10, 2020

This is no longer a wip

@makermelissa makermelissa changed the title WIP: Add _pixelbuf support Add _pixelbuf support Jan 10, 2020
@siddacious
Copy link
Contributor

Before someone else merges, might be worth mentioning this issue in the NeoPixel repo that popped up:
adafruit/Adafruit_CircuitPython_NeoPixel#65
I'm not sure how similar the DotStar implementation is to the NeoPixel one but perhaps DotStar has the same issue?

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple things. Otherwise this looks very cool!

@@ -15,7 +15,7 @@
'sphinx.ext.viewcode',
]

# autodoc_mock_imports = ["digitalio", "busio"]
autodoc_mock_imports = ["pypixelbuf"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this needed? Can't the real version be installed from pypi?

`Feather M4 Express <https://www.adafruit.com/product/3857>`_ and
`DotStar FeatherWing <https://www.adafruit.com/product/3449>`_.

.. code-block:: python
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this example would be better to have in a separate example file. The example in the readme should be short and simple. Other examples can do fancier things.

@rhooper rhooper closed this Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants