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

I2C issue updated? #81

Closed
greg-norris opened this issue May 16, 2018 · 8 comments
Closed

I2C issue updated? #81

greg-norris opened this issue May 16, 2018 · 8 comments

Comments

@greg-norris
Copy link
Collaborator

Has the I2C issue been resolved concerning the screen? I would like to work on a Display tutorial, but we would also like to get the BrainPad to a known state? Our engineers don't want to flood the site with issues from our end with things that are only temporarily down or changing.

@pelikhan
Copy link
Member

pelikhan commented May 16, 2018 via email

@gus-ghielec
Copy link
Collaborator

Do we have a way of testing it locally? with PXT not codal I mean.

@greg-norris
Copy link
Collaborator Author

The current local version, I'm running of PXT-BrainPad display currently doesn't work.

@pelikhan
Copy link
Member

Yes you can test locally. I've added a note in the README on how to update the CODAL tag (66609fc). Once the tag is updated, you can write a test program in libs/brainpad/test.ts and run pxt deploy.

@greg-norris
Copy link
Collaborator Author

greg-norris commented May 18, 2018

It appears to still be grabbing an older version of mbedos from git. When tested locally have to manually add missing functions in I2C to get it to work.

lancaster-university/codal-core#26

@pelikhan
Copy link
Member

pelikhan commented May 18, 2018 via email

@greg-norris
Copy link
Collaborator Author

When using the beta site the display doesn't work. The tutorials are all correct as you noted in the call.

@pelikhan
Copy link
Member

You should be able to repro the display issue locally. The format changed. The relevant C++ line is here: https://github.com/Microsoft/pxt-brainpad/blob/master/libs/screen/screen.cpp#L31

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

No branches or pull requests

3 participants