search

microsoft / pxt-brainpad

Microsoft MakeCode editor for the GHI Brainpad
https://makecode.brainpad.com/
Other
3 stars 11 forks source link

I2C issue updated? #81

Closed greg-norris closed 6 years ago

greg-norris commented 6 years ago

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 commented 6 years ago

The changes are in beta with the latest c++ bits but I haven’t tested them.

From: greg-norris notifications@github.com Sent: Wednesday, May 16, 2018 7:16 AM Subject: [Microsoft/pxt-brainpad] I2C issue updated? (#81) To: Microsoft/pxt-brainpad pxt-brainpad@noreply.github.com Cc: Subscribed subscribed@noreply.github.com

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.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-brainpad%2Fissues%2F81&data=02%7C01%7Cjhalleux%40microsoft.com%7Ce23bfac18bc145aa239308d5bb37a3b0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636620770000924581&sdata=aDsJYvUMb0k%2BbXxSRr%2BxTPrAsrob5vfgoIf%2BGrKwpIs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KWp24_zPUgiLD5_otZO_qecbZCA5ks5tzDTGgaJpZM4UBZku&data=02%7C01%7Cjhalleux%40microsoft.com%7Ce23bfac18bc145aa239308d5bb37a3b0%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636620770000924581&sdata=O2DFbQ2ZeB1AAoyz5kKlbtuXEWI75zqTomAw92O2Qyc%3D&reserved=0.

gus-ghielec commented 6 years ago

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

greg-norris commented 6 years ago

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

pelikhan commented 6 years ago

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

greg-norris commented 6 years ago

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.

https://github.com/lancaster-university/codal-core/issues/26

pelikhan commented 6 years ago

Make sure to clean your build folder when updating a tag.

pxt clean

From: greg-norris notifications@github.com Sent: Friday, May 18, 2018 6:20 AM To: Microsoft/pxt-brainpad pxt-brainpad@noreply.github.com Cc: Peli de Halleux jhalleux@microsoft.com; Comment comment@noreply.github.com Subject: Re: [Microsoft/pxt-brainpad] I2C issue updated? (#81)

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

lancaster-university/codal-core#26https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Flancaster-university%2Fcodal-core%2Fissues%2F26&data=02%7C01%7Cjhalleux%40microsoft.com%7C9070ed89c8124a0f9a1008d5bcc21324%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636622464090905985&sdata=Ly7dcUsbjA0T3rHXoqLHZxHMG7TlRCTLICClKwXsD7I%3D&reserved=0

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt-brainpad%2Fissues%2F81%23issuecomment-390204494&data=02%7C01%7Cjhalleux%40microsoft.com%7C9070ed89c8124a0f9a1008d5bcc21324%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636622464090905985&sdata=qe05a9vQMWgYo%2BiOWtL%2Fg465idYW%2B6Do2hrUMVn4IYs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KdzFhJyDZ9ZPqn4_BWc4-4UBaM5Yks5tzsqHgaJpZM4UBZku&data=02%7C01%7Cjhalleux%40microsoft.com%7C9070ed89c8124a0f9a1008d5bcc21324%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636622464090916002&sdata=Rz%2FjzAsNmxVsZgqVEapovKL6Z5eBcWQD1jhQGEbk2fc%3D&reserved=0.

greg-norris commented 6 years ago

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

pelikhan commented 6 years ago

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