@tulip sleet the BLE lib still uses "bleio" not "_bleio" so nothing works as is -- I s this just waiting on your PR to be merged?
#circuitpython-dev
solar whale
idle owl
Hooray closing an issue that was over 1k issues ago! 😄
tulip sleet
@solar whale yes, do you want to review it? Scott didn't have time.
tulip sleet
i am going to rerun the check because now the sphinx error should go away
solar whale
ok -- was just going to ask
tulip sleet
@solar whale ok build passed now that _bleio is in rtd latest for circuitpython
solar whale
ok -- just downloaded -- eddystone beacon works!
also tested central/periph demo -- looks good !
@tulip sleet should the current time stuff work with the normal BLE repo or is atha still only in your fork -- having some issues with it.
but my tests may be obsolete
tulip sleet
it should work - I tested it, but maybe different test program
i was having trouble with the SSD1306 test, so I changed it to just print
you may need to forget the device in the phone, though I don't think that should be an issue. Or maybe cycle Bluetooth off/on
solar whale
that works! Thanks
OK -- all my currrent tests still work -- I have not tried HID, but I'll be happy to approve the PR without testing that if thats OK
@tulip sleet would you like me to merge?
tulip sleet
Thank you! Yes, so it will be correct. HID is a work in progress. It does work, but I need to add more frills
i'm adding boards to circuitpython.org
solar whale
OK - merged
crimson ferry
@tulip sleet I'm having trouble with SSD1306 as well. 5.0-alpha gets solid green LED with OLED FeatherWing attached to M4 Feather, but my code doesn't start up, nothing on serial/REPL. Tried several working OLEDs. Was working fine with a pre-release S3 build from last week. Everything works fine w/o the OLED.
solar whale
When I tried an old prog using ssd1306 on CP5.0alpha, it just hung....
import displayio
import adafruit_displayio_ssd1306
displayio.release_displays()
i2c = board.I2C()
display_bus = displayio.I2CDisplay(i2c, device_address=0x3c)
display = adafruit_displayio_ssd1306.SSD1306(display_bus, width=128, height=32)
``` causes the board to hang
manic glacierBOT
tulip sleet
@solar whale yes I had similar problems; not sure if it's a library or a core problem
solar whale
OK -- I opened the issue in the core, but feel free to close if you want it on the library.
solar whale
@gilded cradle any idea why this is hapenning now -- I have run this programr many time ins the past but not in the past few weeks. ```
Press any key to enter the REPL. Use CTRL-D to reload.
Adafruit CircuitPython 5.0.0-alpha.1 on 2019-08-30; Adafruit Feather M4 Express with samd51j19
import slideshow_fs
cat_rug_320x240.bmp Size: 153.7 KB
cats_320x240.bmp Size: 163.3 KB
sibs_320x240.bmp Size: 153.7 KB
skye320x240.bmp Size: 153.7 KB
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "slideshow_fs.py", line 51, in <module>
File "adafruit_slideshow.py", line 200, in init
File "adafruit_slideshow.py", line 323, in advance
AttributeError: 'ILI9341' object has no attribute 'wait_for_frame'
it is a simple use of adafruit_slideshow
crimson ferry
DIdn't wait_for_frame() and refresh_soon() get replaced by refresh() in 5.0?
solar whale
maybe ....
gilded cradle
I believe @crimson ferry might be right. I know Scott kept changing it up until he came up with exactly what he wanted.
solar whale
so -- which library has to be updated?
crimson ferry
examples code, certainly
crimson ferry
There are a lot of displayio library examples and Learn guides that will need update when alpha/beta settles down
gilded cradle
Ok, cool @solar whale
solar whale
@gilded cradle may take some time -- I found the offending line in adafruit_slideshow.py -- now have to figure out what the new way of doing it is.... reading
hmmm -- just removing https://github.com/adafruit/Adafruit_CircuitPython_Slideshow/blob/master/adafruit_slideshow.py#L323 makes the problem go away -- the slideshow works --- not sure this is the most elegant fix ... I'll keep readin ghte changes and see if taht makes sense.
gilded cradle
Maybe you could put it inside a try/except block @solar whale
solar whale
ok -- looks like it is just not needed anymore in CP5 -- I'll use try/except and put in a PR
solar whale
@gilded cradle PR submitted -- thanks -- also @crimson ferry thanks for the pointer to the CP5 changes.
gilded cradle
Thanks @solar whale
onyx hinge
I've run the "playground sound meter" on an Express and a Bluefruit, and it seems to work a lot better on bluefruit. With express it seems like there are frequently noises detected even when the room is quiet. The Bluefruit is also a lot faster at updates, but that is less of a surprise. The spurious noise detections make me wonder if there's a bug of some sort
onyx hinge
and then I gave away that cpx to an aspiring young person who's considering comp sci but hasn't programmed yet. what?! how is that a thing!?
solar whale
hmm -- that metro_nrf52840_express sounds like fun 😉
tulip sleet
@crimson ferry @solar whale doc is updated, e.g. https://circuitpython.readthedocs.io/en/latest/shared-bindings/displayio/Display.html#displayio.Display.refresh
solar whale
💤
tulip sleet
you're welcome! yes, I think so, but there are only 699 assets in the github asset list as opposed to 783 for 4.1.0. Did you adjust .bin vs .uf2 or anything like that? Just finished adding boards to circuitpython.org. You could approve https://github.com/adafruit/circuitpython-org/pull/265 if limor doesn't get to it in a minute: it's just some typo fixes after the bigger new boards PR that she approved
slender iron
will look at that soon. I bet we're missing some
tulip sleet
rate limit exceeded 😦
slender iron
ya, it was a whirlwind but I chatted with all the folks I hoped so it was good
I'll poke those now to see if they work
s3 should have all of the assets already
tulip sleet
can you just space out the uploads a bit in "Upload to Release" or is it per hour or something?
slender iron
it's per hour I think
1,000 afaik
by swapping keys we can get it to 4,000 I think though
manic glacierBOT
I connected a J-Link to the feather_nrf52840 and OLED -- and rebuilt 5.0alpha with DEBUG=1
The code appers to be executing, that is, it does not hard Fault but the display is blank and
it won't respond to any keyboard input (COntrol-C) does not work
Here is a screenshol of the call stack after loading the oled.py test progrram
this was the REPL
...well that feels like an hour wasted, still haven't gotten gdb to my metro m4 going, using segger's software or openocd 😦 time to take a break then use print-debugging like a peasant.
I'm certain I did this in the past, but if I did I destroyed the evidence
onyx hinge
yay finally got it working. breakfast helped 🙂
raven canopy
@onyx hinge working on the PDM noise disparity?
onyx hinge
@raven canopy no, I am looking at https://github.com/adafruit/circuitpython/issues/1992
it's weird.
every statement the reporter made where I said "they MUST be mistaken about that" has been true when I checked it. For instance, it actually IS dependent on the samples themselves, even though nothing can possibly depend on that.
lone sandalBOT
unreal marsh
I'm using MicroPython on my ESP32 and would like to use https://github.com/adafruit/Adafruit_CircuitPython_Motor with MicroPython. Is that possible?
solar whale
@unreal marsh the Adafruit_CircuitPython_Motor library accepts "pwm" objects that are created with "pulseio" which I don't think you have access to in MicroPython
raven canopy
@onyx hinge crawling through the DAC setups between cirpy and arduino, is it possible that this is "simply" missing a 1 (12MHz vs 2MHz)? https://github.com/adafruit/circuitpython/blob/master/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h#L84
unreal marsh
@solar whale r/ihadastroke, but thanks
onyx hinge
@raven canopy hm I don't see any uses of that macro though
I suppose I can change it, rebuild, and see if it makes some difference unexpectedly 🙂
raven canopy
its the initial config for the DAC
i needed a distraction from researching intrinsic CMSIS functions. 😄
onyx hinge
OK, let me try it
solar whale
@unreal marsh I don't think I understand your comment, I may be incorrect so I hope someone else will comment and correct me if I am.
onyx hinge
@raven canopy sadly, didn't seem to make a difference.
raven canopy
😦
timber mango
Hello..... I am back... I got successful on sound reactive on Neotrellis M4 board. Simply copied the code from https://learn.adafruit.com/sound-reactive-neopixel-peace-pendant/circuitpython-code and edited to suite the board. mic_pin = AnalogIn(board.MICOUT) and n = int((mic_pin.value / 65536) * 1000). The only problem is cannot hear from headphone, just speak into microphone as pixel colors show. I added the line "audio = audioio.AudioOut(left_channel=board.A1, right_channel=board.A0)" ... No luck. How do I get the sound in headphone?
manic glacierBOT
cerulean sparrow
How would I build Adafruit_CircuitPython_PyPortal for circuit python 5.0 ?
onyx hinge
@cerulean sparrow you mean the accompanying .mpy files? I am not sure, but I'll try to help.
something I know about generally is that build instructions are in the ".travis.yml" files, so I started by looking at https://github.com/adafruit/Adafruit_CircuitPython_PyPortal/blob/master/.travis.yml
It looks like the key steps are to get the git source, install the dependencies, and then run the circuitpython-build-bundles command
- pip install circuitpython-build-tools Sphinx sphinx-rtd-theme
...
- circuitpython-build-bundles --filename_prefix adafruit-circuitpython-pyportal --library_location .
Just for now, could I just replace the adafruit_pyportal.mpy with an updated adafruit_pyportal.py and skip the compile
onyx hinge
In general, yes, you can remove a .mpy and put a .py in its place. The main thing to be aware of, it can increase memory (RAM) usage.
and it looks like on some systems you will have to use "pip3" not "pip", in case "pip" refers to python2
manic glacierBOT
I updated the initial comment with new code and an oscope grab. And just when you got things weird enough, I got this one too:
onyx hinge
@cerulean sparrow fwiw I followed those instructions and got a file called bundles/adafruit-circuitpython-pyportal-4.x-mpy-3.1.2.zip -- this file should work on 5.0.0-alpha as well.
I was on a linux computer, it looked like the steps involved compiling software which might only work on linux systems.
cerulean sparrow
@onyx hinge Thanks for the help, I am getting ImportErrror: cannot import IO_HTTP when I run on device even after changing the refresh_soon lines in PyPortal.
onyx hinge
You might have to update another bundle called adafruit_io, or go back to an earlier version of adafruit_pyportal.py. I'm 100% guessing about this, but lines referring to IO_HTTP were added recently, on July 23. from adafruit_io.adafruit_io import IO_HTTP, AdafruitIO_RequestError and maybe the copy of adafruit_io on your pyportal is older.
raven canopy
as a note, there are pre-compiled versions of mpy-cross available in this release: https://github.com/adafruit/circuitpython/releases/tag/3.1.2 (it still works with CircuitPython 4.x [and i expect with 5.x])
onyx hinge
ugh, almost daily on this computer, subsequent to a circuitpython hard fault, the whole computer stops seeing usb hot plug events and I have to restart. (linux, some old AMD box I had laying around). kernel messages implicate efi.
and it's happened again, so a reboot is in store before I can click anything
exotic pumice
yeah bad usb can ruin your day
onyx hinge
it makes me wish for a PS2 mouse (my keyboard is a PS2 "Model M" still plugged in on that vintage connector)
exotic pumice
nice
I'm using an ibm ps2 keyboard but not a model m lol
a govt surplus near me has boxes of old mice and keyboards
you could probably find a ps2 mouse easily enough at a thrift store or something
manic glacierBOT
solar whale
@prime flower I have the ESP_ATcontrol library now working with adafruit_requests. All of the examples work and I have tested some of the AdafruitIO examples to without any problems (http/simpletest and weather work). I have also modified the ATcontrol examples to only have the Argon configurations. I think any other boards should use the ESP32SPI firmware. I think I'll go ahead an PR this later to day after a few more tests. I also want to add the get_MAC function, but that can be a separate PR. Next, I was going to take stab of adding ESP_ATcontrol to the miniMQTT library. I'll let you know how it goes.
idle owl
@tulip sleet We need to create a 5.x bundle now that 5 is released. Not sure what goes into that.
tulip sleet
I'll look at that sometime soon, maybe today
idle owl
@tulip sleet Sounds good. Please let me know when you do, I have a guide waiting on it. (No rush, would simply like to know.)
raven canopy
@tulip sleet @idle owl we could possibly extend the VERSIONS dict to include an option for pre-release, and pass that into the build script: https://github.com/adafruit/circuitpython-build-tools/blob/master/circuitpython_build_tools/target_versions.py
idle owl
Oh hmm. Yeah.
raven canopy
oh...and this was already done: https://github.com/adafruit/circuitpython-build-tools/pull/47 😆
just needs a merge.
idle owl
@raven canopy 😆
raven canopy
as it stands though, that won't include the pre-release BLE library (just to clarify).
afk: its waffle house time with the kiddo & nephew
idle owl
@raven canopy Delicious!
raven canopy
made a quick issue before leaving: https://github.com/adafruit/circuitpython-build-tools/issues/49
tulip sleet
@idle owl how do you feel about dropping the 3.x bundles as done in https://github.com/adafruit/circuitpython-build-tools/pull/47?
idle owl
@tulip sleet I wasn't sure if we should wait until 5.0 is actually released or not. We plan to support "one previous version" but since 5.0 is still alpha, I'm not sure it count yet.
tulip sleet
the latest wave of I2C changes still work on 3.x, right?
idle owl
Oh I don't know. Scott makes a good point on them not working.
I'd say let's dump it. I already changed all the warnings to "latest" instead of a particular version number.
We're going to dump it in a month or so anyway.
LET'S BREAK EVERYTHING.
tulip sleet
i'll ask again after you've had lunch.
idle owl
😆
We'll need to create the Troubleshooting/FAQ section for CircuitPython with static links to the last set of builds.
tulip sleet
we could also add that to circuitpython.org/libraries
tulip sleet
i kind of agree that 5.0 alpha is barely a version yet
but we do want to encourage testing
idle owl
That was my thought. On both counts.
tulip sleet
I think having easy access to the last 3.x bundle is a good compromise, for people who need to update but can't move on to 4.x yet for whatever reason (like they have 30 CPX's in a classroom or whatever)
tulip sleet
OK, so steps:
- accept #47 to start 5.x bundles and stop 3.x bundles
- Notify support.
- Put links to last bundle in circuitpython.org/libraries (PR) and guide (edit)
the order to do might not be that order
i have to go out for a few hours; should we try to do this today or not?
the 5.x bundle will be imperfect (like wrong BLE), but that's a hazard of being an alpha tester and the solution is not easy
idle owl
@tulip sleet I think we can do it today. I'm on a train until 13:15 and then again from 14:20-18:20 so we can look at it later and figure out everything.
slender iron
Hi all, just a reminder that our weekly meeting is a day later than normal because Monday is a US holiday. So, the meeting will be on Tuesday September 3rd at 11am Pacific / 2pm Eastern here on Discord. Everyone is welcome to attend. If you are unable to make it and want to drop us notes, the notes doc is here: https://docs.google.com/document/d/1r-F78z5lRl91G4kM6VzajSn2i7hdUUyp-jCiegHs7MA/edit?usp=sharing Thanks! <@&356864093652516868>
timber mango
Hi All, thank you to this channel for the help with my latest project, a LoRa remote running CP. I posted it on Reddit with all of the details, like the current source, 3D printing files, etc. Thank you to the entire Adafruit Discord group- this is an awesome community! https://www.reddit.com/r/3Dprinting/comments/cy3c72/i_made_a_3d_printed_lora_remote_control_details/
onyx hinge
hmm are my expectations for the debugger too high? (gdb) p $1->DACCTRL[0] $60 = {bit = {LEFTADJ = 1, ENABLE = 1, CCTRL = 1, FEXT = 0, RUNSTDBY = 0, DITHER = 0, REFRESH = 2, OSR = 0}, reg = 519} (gdb) p $1->DACCTRL[0].bit.CCTRL = 0 $61 = 0 (gdb) p $1->DACCTRL[0] $62 = {bit = {LEFTADJ = 1, ENABLE = 1, CCTRL = 1, FEXT = 0, RUNSTDBY = 0, DITHER = 0, REFRESH = 2, OSR = 0}, reg = 519} (gdb) I can't seem to modify CCTRL. some register accesses definitely happen, such as writing to the DAC register directly $1->DATA[0].reg = ###...
though attempting to read back DATA[0].reg also results in an unchanging value
raven canopy
try disabling the DAC, changing the val, then re-enabling the DAC? sheet says that the register is enable-protected.
onyx hinge
oh that makes sense
raven canopy
only knew to look because i've fought it before 😄
manic glacierBOT
onyx hinge
Okay! CCTRL does affect the weird stair-steppy thing
CCTRL=0
CCTRL=1
CCTRL=2
Switching to bandgap reference reduces the number of steps, they each have the same absolute (voltage) height
raven canopy
ahh! i was missing a link with my DAC frequency suggestion yesterday. it should match the CCTRL range. 🤦
onyx hinge
DAC under arduino (should be same time scale)
yeah same time scale
so something must be wrong that is fixable in CP
raven canopy
the only difference i could really see was that frequency setting (2MHz vs 12). both use DFLL as the source...
onyx hinge
where is the code that actually USES that #define you pointed out? Is it getting called / used at all in CP?
onyx hinge
OK, is that a part of the source code where grepping for that identifier should have found it? That's how I usually find things, and came up empty..
or is it something that is generated in some other way?
raven canopy
DAC0.CCTRL: https://github.com/adafruit/circuitpython/blob/master/ports/atmel-samd/asf4_conf/samd51/hpl_dac_config.h#L54
DAC1.CCTRL: https://github.com/adafruit/circuitpython/blob/master/ports/atmel-samd/asf4_conf/samd51/hpl_dac_config.h#L93
DAC GCLK FREQ: https://github.com/adafruit/circuitpython/blob/master/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h#L84
onyx hinge
after changing that last file, is there something beyond a normal "make" to do?
raven canopy
not that i know of.
onyx hinge
when I made this change, CP still built...! ```diff --git a/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h b/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h
index 41efca755..08eec2663 100644
--- a/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h
+++ b/ports/atmel-samd/asf4_conf/samd51/peripheral_clk_config.h
@@ -81,7 +81,7 @@
- \brief DAC's Clock frequency
*/
#ifndef CONF_GCLK_DAC_FREQUENCY
-#define CONF_GCLK_DAC_FREQUENCY 2000000
+#define CONF_GCLK_DAC_FREQUENCY "I don't believe sommersoft"
#endif
// <y> EVSYS Channel 0 Clock Source
yes
raven canopy
🤔 maybe that's all unused now?
onyx hinge
I can confirm that file is still included in places
and some similar #defines are used common-hal/analogio/AnalogOut.c: _gclk_enable_channel(DAC_GCLK_ID, CONF_GCLK_DAC_SRC);
GCLK5 is 48MHz/24 = 2MHz
raven canopy
there are two config locations. atmel-samd/asf4_config & atmel-samd/asf4/config. i could've sworn that the first is used in lieu of the second.
onyx hinge
#define CONF_GCLK_GEN_5_DIV 24
asf...such a layer cake. 😄
onyx hinge
Only LEFTADJ and CCTRL registers (within the DAC) differ
raven canopy
scrolled past this like 6 times: https://github.com/adafruit/circuitpython/blob/master/ports/atmel-samd/Makefile#L76 so its weird that "I don't believe sommersoft" didn't throw an error.
onyx hinge
well if I change say the CONF_GCLK_GEN_5_DIV line to say "I do believe summersoft", it gives an error where the macro is used.
but setting that divisor from 24 (48/24 = 2MHz) to 4 (48/4 = 12) didn't make a difference either
time to take another break from this 😓
raven canopy
breaks are good.
raven canopy
assembly math is not as fun as it looks. especially when you only have a rudimentary understanding of binary. thank the deities for mp_printf.
hmm. well, 16bit unsigned multiplication is working. but the amplitude is higher than its signed counterpart. progress, i suppose.
tulip sleet
@onyx hinge @raven canopy other people's DAC difficulties: https://www.avrfreaks.net/forum/help-getting-dac-working-samd51 also maybe drag @simple pulsar into this ^
tulip sleet
@onyx hinge @raven canopy don't feel constrained to using the ASF library. If you want to code it in straight registers, that is fine. We'd like to gradually move away from ASF
raven canopy
@tulip sleet you got a moment? i have a "is it worth it" question wrt to audiomixer & asm/CMSIS functions...
tulip sleet
sure
raven canopy
ok. so, i've got all of the addition functions using intrinsic CMSIS. but, for the multiplication, there really aren't any options for non-16bit & unsigned. since i have the C functions working, is it worth it for me to push on [learning] the asm equivalents? (they could conceivably be used with arm-v7 and arm-v6)
tulip sleet
when is multiply used? I think it's not important to do the asm stuff now, and you can also ignore the C functions for M0, since we aren't planning to enable mixer for M0
raven canopy
multiply is used to adjust the level/gain of an individual voice.
tulip sleet
it's pretty easy to check that multiply is out of range and limit it, right? You are always multiplying by a positive number, so there's less sign-checking
raven canopy
yeah, they have boundary checks, according to signedness.
using <limits.h>, so SHRT_MAX, SHRT_MIN, etc...
tulip sleet
The q is just whether to mess with
asm volatile("smulwb %0, %1, %2" : "=r" (lo) : "r" (mul), "r" (val));
asm volatile("smulwt %0, %1, %2" : "=r" (hi) : "r" (mul), "r" (val));
asm volatile("ssat %0, %1, %2, asr %3" : "=r" (lo) : "I" (bits), "r" (lo), "I" (shift));
asm volatile("ssat %0, %1, %2, asr %3" : "=r" (hi) : "I" (bits), "r" (hi), "I" (shift));
asm volatile("pkhbt %0, %1, %2, lsl #16" : "=r" (val) : "r" (lo), "r" (hi)); // pack
kind of stuff?
raven canopy
right. that one, 16bit signed, has a armv7 shortcut (SMULWx) . but there's nothing for 16bit unsigned, or 8bit anything.
i have 16bit unsigned "working"...but its super loud. 😄
tulip sleet
I think it's fine to omit the asm stuff. You could comment it out as "not working on M4" or something if you don't want to lose the info
raven canopy
which is ironic, since the space/time savings benefit M0 more. hehe
tulip sleet
yes, but we can barely do audio in M0, and we are just running out of space constantly
raven canopy
alright. i'll remove audiomixer from M0 altogether then. i was only chopping off the overruns board-by-board.
and combing armv6 instructions...i have less hope:
The only multiply instruction supported in ARMv6-M performs a 32x32 multiply that generates a 32-bit result, see MUL on page A6-143. The instruction can operate on signed or unsigned quantities
in the long run these functions might be in their own library (not Python library, but a separate file), or even a separate repo.
I'm not sure if you can use these: https://gcc.gnu.org/onlinedocs/gcc/Integer-Overflow-Builtins.html
whether you can use 16-bit types or not with those
whether type1, type2, type3 can be <32 bits, but I think so
@raven canopy this compiles with arm-none-eabi-gcc:
#include <stdint.h>
uint16_t f(uint16_t a, uint16_t b) {
uint16_t c;
__builtin_add_overflow(a,b,&c);
return c;
}
so you can use those functions, check if they return true or false. If true there's been an overflow and you can range-limit
i don't know how efficiently they are implemented, but hopefullly it's good
raven canopy
i imagine the add/sub compile closely to the armv7 instructions. e.g.:
uint32_t __UHADD8 ( uint32_t val1,
uint32_t val2
)
This function enables you to perform four unsigned 8-bit integer additions, halving the results.
Parameters
val1 first four 8-bit summands.
val2 second four 8-bit summand
the muls will be the possible benefits. i can def give it a whirl.
tulip sleet
you could try them for the addition cases too, but the built-in saturation operations should be a lot faster, since there's no checking. But these would work on M0 with checking if we ever wanted that.
raven canopy
been reading. thanks for the pointers and discussion, @tulip sleet!
onyx hinge
well this comment seems wrong``` // SAMD21: This clock should be <= 12 MHz, per datasheet section 47.6.3.
// SAMD51: This clock should be <= 350kHz, per datasheet table 37-6.
_gclk_enable_channel(DAC_GCLK_ID, CONF_GCLK_DAC_SRC);
manic glacierBOT
solar whale
@tulip sleet will the dotstar fix go into master soon? Just curious if I should do it locally or wait for merge.
tulip sleet
@solar whale I was going to prepare a merge soon - was going to check if there was anything else to add to 4.x and then merge back
solar whale
Thanks -- no rush -- just curious about the plans -- Nice job - finding/fixing that! 👍
manic glacierBOT
manic glacierBOT
Obviously I feel quite uneasy about this and wish I had more answers.
Here's how the DAC looks with the sequence [0,65535] repeated forever:
Something's obviously still quite wrong with how the DAC or its clocking is set up.
onyx hinge
hum now what do I do? After returning from Arduino to CircuitPython, my files are gone (OK, not a surprise) and storage.erase_filesystem() isn't giving me a sane filesystem to work with (ouch) ```Press any key to enter the REPL. Use CTRL-D to reload.
Adafruit CircuitPython 5.0.0-alpha.0-282-ge7c3017c9-dirty on 2019-09-02; Adafruit Metro M4 Express with samd51j19
import storage
storage.erase_filesystem()
cu: Got hangup signal
Disconnected.
jepler@soba:~/src/circuitpython/ports/atmel-samd$ cu -l /dev/ttyACM0
Connected.
Auto-reload is on. Simply save files over USB to run them or enter REPL to disable.
Press any key to enter the REPL. Use CTRL-D to reload.
Adafruit CircuitPython 5.0.0-alpha.0-282-ge7c3017c9-dirty on 2019-09-02; Adafruit Metro M4 Express with samd51j19
import os
os.listdir('.')
['\x02\x0b\x0b\x10\x1b\x0b\x11\x10.\x10\x18\x10']
Any suggestions?
hm, flashing back to 4.1.0 got me a CIRCUITPY device. hmmmm
something to do with my clocking changes? sigh/argh.
something generally wrong in master branch, ditto
afk
tulip sleet
@onyx hinge running an Arduino program on an Express board should not generally disturb the SPI flash chip, so yes, something else. It may have gotten corrupted earlier.
orchid basinBOT
subtle sun
@tulip sleet Considering that _bleio shouldn't be used directly anymore should a function be added to adafruit_ble to return the MAC address?
tulip sleet
@subtle sun sure, please file an issue in the library
i expect there will be other such things
subtle sun
Will do
indigo wedge
Seeing an interesting bug, need to investigate more:
>>> import board, busio
>>> uart = board.UART()
>>> uart2 = busio.UART(rx=board.RX2, tx=board.TX2)
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
ValueError: Invalid pins
** reset **
>>> import board, busio
>>> uart2 = busio.UART(rx=board.RX2, tx=board.TX2)
>>> uart = board.UART()
ah i see, so uart2 can only be on 1 sercom but uart can be on 2, so if i init uart first then it uses the sercom that uart2 can use but the other way around uart just uses the second one
hope that made sense heh
so it can't really be fixed cause it's kinda board specific
like could reorder sercom lookup table but then it might break other boards that use a different pin config
manic glacierBOT
Had some cycles, so dug a little deeper into item one of #2021.
In looking at the html source, the docs/static/customstyle.css wasn't being used, which explains why the override workaround wasn't working. Verified this with a local build.
This change produces an inclusion of the custom CSS in the html head:
tulip sleet
@indigo wedge limor calls it "pin sudoku" (also applies to choosing which pins to expose on a board with limited pins)
slender iron
@minor plume @plucky flint is working on a cli updater. repo for it is here: https://github.com/ntoll/circup
minor plume
@slender iron I will take a look next week after my move, and see if it is something I can contribute to. No point reinventing the wheel.
slender iron
we've got lots of things we could use help on so no worries there 🙂
minor plume
I’m looking forward to getting more involved, once I have this move behind me. 🙂
slender iron
Sounds good. 🙂
indigo wedge
Sorry for not being up to date, but when is CP V release planned? So I can make sure my board gets in there 😄
plucky flint
@slender iron @minor plume please don't hesitate to give feedback, critique and ideas. Thanks! 🙂
plucky flint
Apologies to those module maintainers for whom I've created a bunch of __version__ and __repo__ metadata issues and PRs on GitHub. Please feel free to ping me with questions if that would be helpful. Thanks!
onyx hinge
good morning! I hope I'll make it to the weekly meeting today, but it depends if somebody at DAY_JOB thinks they already have me scheduled at that hour.
umbral dagger
I don't see a 5.0.0alpha1 built for PyPortal (I just see one for the titano)
tulip sleet
@umbral dagger the uploading of builds to S3 got rate-limited, so there are some missing build. You can try a newer build of master; it should be fine
@indigo wedge there is no schedule for CPy 5: we have some goals we are shooting for (like BLE workflow) that are a stretch. But submit your board when you can and you should probably have a usable build with the new displayio updates, since the instability will be in areas that you probably aren't relying on.
indigo wedge
yeah i just pulled master yesterday and flashed that, worked without problems 🙂
oh you merged, nice, thanks 🙂
manic glacierBOT
onyx hinge
hah just spotted this in the draft meeting notes. jealous, @solar whale "@solar whale (not able to attend -- off in a canoe)"
slender iron
@stuck elbow stage q on the forum: https://forums.adafruit.com/viewtopic.php?f=60&t=155961
stuck elbow
@slender iron thanks
plucky flint
doh... https://github.com/ntoll/circup/issues/7 (I've fixed this now)
meager fog
@onyx hinge hihi lemme nkow when ur around
onyx hinge
@meager fog what's on your mind?
onyx hinge
@meager fog let me get signed in and look...
@meager fog You mean, do I have suggestions for new items to add?
those both sound like good ones so far
onyx hinge
I went ahead and added "M4 DAC Weirdness". Not sure if you were following the progress, but I started to find some difference between Arduino and CP and fixing them made CP better...
not sure how far down the road you want me to look, but I think there are some things around time measurement and timekeeping that would be good to look into
raven canopy
my yet to be posted meeting notes include the exact term "M4 DAC weirdness". coincidence? 😆
onyx hinge
No, it's an unseen hand guiding us both
@meager fog just based on my time availability between now and mid-november, these 3 items might account for it all
onyx hinge
Should I feel free to add items to this list, then?
meager fog
@onyx hinge yes! its for u!
onyx hinge
OK, I figured it was for you
meager fog
its for both of us 🙂
so for the M4 weirdness... if the same weirdness happens in Arduino its probaly cause the DAC itself is weird
because it is a little weird. its got errata
onyx hinge
Actually under Arduino with AnalogOut, the DAC seems to work pretty well
meager fog
ok
do you want to try to keep hacking on that for the next few days?
i put in a few other dac/adc-y issues
onyx hinge
the specific "stair step when setting DAC from 0 to full scale" doesn't happen on Arduino and it is a cause of at least one other issue (audio synch)
onyx hinge
I actually wanted to switch gears to I2S Out as soon as I got the hardware, because I banged my head on the DAC all weekend and didn't get very far
though I did start to make some progress, as I mentioned, it's still bleeping weird
@meager fog would you prioritize the dac bugfixing above nrf I2SOut, assuming I was sure I could bring one or the other to a conclusion before my travel starts late this month?
meager fog
@onyx hinge 👍
onyx hinge
back!
in the context of the samd51 errata, what does 'efp' mean?
(errata are divided into 'efp' and 'non-efp')
raven canopy
enhanced floating point? 🤷
plucky flint
@slender iron @meager fog @idle owl quick question about naming conventions for release tags for CircuitPython module repositories... such tags will always be valid semver values..?
onyx hinge
excessively fiddly procedures
idle owl
@plucky flint They should be yes, if they are not, they should be fixed.
plucky flint
....and somehow (how? CI presumably) the __version__ in the actual source file is updated from 0.0.0-auto.0 to the value of the semver tag.
tulip sleet
right, it's in the bundle building python scripts
plucky flint
@idle owl 👍
pearl notch
Hi all. I won’t be fully on the call today, but I want to throw out an idea about a directory of the CircuitPython community. Maybe an online spreadsheet with: who’s working on what, relationship to Adafruit, general bio, professional and personal, twitter handle, YouTube channel, LinkedIn profile and the like, geographical location. Opt-in, community-maintained. To help a newcomer like me get oriented and make connections for collaborating, contributing, and the like.
plucky flint
Aha... @tulip sleet where are these scripts (just so I can check how it's done, so the result is correctly handled by circup)..?
tulip sleet
defer to @raven canopy here
raven canopy
@plucky flint for version specifically: https://github.com/adafruit/circuitpython-build-tools/blob/master/circuitpython_build_tools/build.py#L90
errant grail
lurking today
plucky flint
@raven canopy stonking! Thanks!
onyx hinge
I keep being impressed by the toolng around CP that I didn't even realize existed...! @raven canopy
slender iron
gonna go grab more water
idle owl
brb as well.
crimson ferry
(lurking,)
raven canopy
water refill. good idea...
tulip sleet
meeting starting momentarily
plucky flint
@raven canopy cool... it's nice and simple... 👍
onyx hinge
🚰
timber mango
lurking
raven canopy
@onyx hinge i've mosly only added to stuff. Scott did most of the original stuff akaik
turbid radish
Hi all
sterile bronze
lurking
turbid radish
I'll be lurking today also
sterile bronze
and hello
gilded cradle
We can hear you @slender iron
inland tusk
@slender iron It turned out that the storm stalled out over the bahamas so it is not going to directly hit us in wpb
onyx hinge
hurries to finish bowl of crunchy cereal
marble hornet
Do you think it would speed up cirpy if it had type hinting?
tidal kiln
** lurking ** (but will decloak for in the weeds)
slender iron
turbid radish
Go to adafruitdaily.com to subscribe
raven canopy
@marble hornet with my understanding of type annotations/hinting, no. they aren't actually "used" by CPython to enforce static typing. they're more for allowing people & IDEs to help identify type mismatches.
idle owl
Meet Device Simulator Express:
https://www.microsoft.com/en-us/garage/blog/2019/08/adafruit-and-microsoft-garage-interns-team-up-to-create-device-simulator-express/
https://www.microsoft.com/en-us/garage/profiles/device-simulator-express/
https://blog.adafruit.com/2019/08/29/meet-device-simulator-exress-pythonsim-a-msftgarage-project-built-by-garage-interns-that-makes-it-easier-to-program-the-adafruit-circuit-playground-express-in-python-with-or-without-a-physica/
marble hornet
@raven canopy i see what you mean. I’m sure the people at pypy make great use of it too
pearl notch
Lurking. And getting excited about this Device Sim Express.
turbid radish
Awesome Jeff
I'll be writing up some more on device sim in the next day or so
Dave
pearl notch
Great, Mike. I look forward to it. I was gonna install Mu Editor for my classes, but perhaps VSCode with the new plugin is the way to go.
plucky flint
@slender iron I'll need to leave in 10mins (it's 7:30pm here and I need to pick up kids from places at 8pm)
+1
thank you
wild flare
Funnily, I'm kind of liking Mu Editor.
onyx hinge
plucky flint
@onyx hinge that's the one... thank you!
turbid radish
Ethernet! Great work @prime flower
prime flower
@turbid radish Blocked on it! But once it gets unblocked, it'll be finished pretty quickly 🙂
turbid radish
All good!
plucky flint
@wild flare thank you..! 🙂
tidal kiln
@onyx hinge thought i heard "M4" and "DAC" oddness?
plucky flint
👋 ok folks, I have to shoot off for dad taxiing duties. Hopefully next time I'll be around longer..! Take care folks.
tulip sleet
@ionic elk I am also working on a general PR for turning USB and USB HID devices on and off for a particular build. Hope to submit this soon.
motivation is to support a particular gamepad for assistive technology
pastel panther
tulip sleet
not done yet (need to do some more work on more flexible definition of HID report buffers, and removing Report ID when there's only a single device
ionic elk
@tulip sleet gotcha! That should be handy I'll take a look.
We should also check in with Thach about the status of USB on the 412
slender iron
@idle owl please run for a bit
idle owl
@slender iron Already on it.
onyx hinge
nRF datasheet: "Important: For more information on Standard drive, see GPIO — General purpose input/output on page 155. Low frequency I/O is signals with a frequency up to 10 kHz." wow that really is less than ideal
tulip sleet
it's not that those pins don't work at higher frequencies - it's that they add noise to radio reception
pastel panther
ah, interesting
tulip sleet
so it's a recommendtaion
they are too close to the antenna pins or internal circuitry or something
pastel panther
right, that's what I was thinking
tidal kiln
** IN THE WEEDS **
- NeoPixel related:
- remove bpp?
- is the automagic white channel thing good?
tulip sleet
you mentioned sequential in a port but if bit-bang, doesn't need to be sequential, right?
tidal kiln
@pastel panther have you seen the arduino multi-tasking learn guides?
pastel panther
@onyx hinge llol 🚀
pastel panther
@pearl notch 👀
tidal kiln
hang on
raven canopy
🐿
meager fog
@onyx hinge hi back for 30 min if u have any questions
pearl notch
Good place for the directory? Google Drive? Github requires approvals and merging, might be too much overhead?
idle owl
@pearl notch GitHub would be ideal, and would create a situation where we have an approval process, but we could also add collaborators to it so it's not on one person to merge everything. Google Drive would also work though - probably the simplest option for now.
stuck elbow
Actually Japanese works on gameboy
they just have around 90 characters for it
hiragana only
idle owl
Also GitHub would require understanding Markdown or something to format the info.
pearl notch
For Hack the Future we create a Drive sheet and mentors signing up for an event add a row with their info, filling in whatever they’re comfortable with. Works well. Has versioning.
pastel panther
@pearl notch as kattni sez, anywhere would be a good start but something with tie-ins to something else and a approval process to prevent defacement would be nice
stuck elbow
the trick is to spell out the kanji characters in hiragana
pearl notch
We’ve had no abuse at HTF, but this is a more visible community.
idle owl
@pearl notch If you're feeling fancy, could do a Google web form or whatever they call it that dumps into a spreadsheet.
pearl notch
That’s nice, I use those with students. Have to deal with duplicate rows. Let’s consider.
idle owl
@pearl notch You're starting it, you get to decide on where to start 🙂
onyx hinge
4180 bytes free in flash out of 188160 bytes ( 183.75 kb ). (english)
1676 bytes free in flash out of 188160 bytes ( 183.75 kb ). (zh_Latn_pinyin)
pearl notch
Adafruit should create the document under its account (rather than mine).
onyx hinge
msgid " File \"%q\", line %d"
msgstr " Wénjiàn \"%q\", dì %d xíng"
@pearl notch I don't see this getting so unwieldy that would prevent manual curation/merging, at least not anytime soon
pearl notch
Good point, @pastel panther
idle owl
And ideally by the time it does, we have a lot of collaborators on it who are willing to help.
pearl notch
@idle owl maybe you could create a Drive folder under an Adafruit Google account, and give me access, to create the form and the sheet within.
stuck elbow
we can use a different character encoding, like shift-jis for the japanese
utf8 is very bad for Japanese and Chinese
idle owl
@pearl notch I'll look into it. Please send me an email with the details.
slender iron
can we compress it away anyway? @stuck elbow
stuck elbow
sure, but with such short strings I'm not sure how well it would work
slender iron
right
stuck elbow
arguably using a different encoding is a kind of compression ;-)
slender iron
true true
pearl notch
@idle owl if you prefer I’ll create it under my account, and we can transfer ownership later (if the experiment succeeds).
pastel panther
bye everyone!
idle owl
@pearl notch No I think it makes sense to begin it on an Adafruit account. Simply a matter of which one.
raven canopy
thanks everyone! have a great week!
onyx hinge
@meager fog thanks but I'm already pushing my day job schedule today. I have plenty to keep me busy for now and we can figure more out down the road if that's okay with you.
meager fog
@onyx hinge yep all good
idle owl
@slender iron The basic example for NeoTrellis (the neopixel 4x4, not the Express) fails with Traceback (most recent call last): File "code.py", line 49, in <module> File "adafruit_neotrellis/neotrellis.py", line 89, in sync File "adafruit_seesaw/keypad.py", line 74, in count File "adafruit_seesaw/seesaw.py", line 372, in read8 File "adafruit_seesaw/seesaw.py", line 383, in read File "adafruit_seesaw/seesaw.py", line 383, in read File "adafruit_bus_device/i2c_device.py", line 96, in readinto OSError: [Errno 5] Input/output error
Failing on the trellis.sync() call. If you add a red LED lighting up to the main loop, it still lights up even if it's after the sync call, implying to me (I think) that it's not failing on the first time through the loop. Which feels familiar as a bug we dealt with before, but I can't place it. Ring any bells for you?
slender iron
no bells here
idle owl
blergh. Ok.
I thought I2C failing on subsequent loops was a bug we ran into with something.
Faint recollection anyway.
slender iron
ya, maybe
exotic pumice
need me to get back to work on 
busio.i2c? 😛
ruby lake
woo cp5...does it have timer intr callback? 😉
exotic pumice
jk, my first attempt is a direct translation of the C which probably has few if any safety benefits
idle owl
@slender iron I had JP try it on 3.1.2, the code works. It fails on 4.x and 5.x. Do you think it's a lib bug or a CP bug? Trying not to do what usually do and make assumptions here.
slender iron
@ruby lake no, no concurrency
ruby lake
mm
slender iron
@idle owl could be a CP since it works on 3.x
onyx hinge
@slender iron is it because my PR is from before the switch to gitlab actions (has a .travis.yml file)?
slender iron
ah, ya. could be
river quest
hello
ok, so this worked out before, we got "code + community = CircuitPython" ... so let's this again!
@idle owl ...
idle owl
🙂
river quest
these will be for Circuit Playground Bluefruit
everyone else feel free to chime in 🙂
Get the BLUES! Circuit Playground Bluefruit 🙂
gilded cradle
Adafruit + Bluetooth = Bluefruit + Adatooth?
river quest
hah
this is for the banner we'll do, so we need to say what it is and maybe something fun with the fact it's wireless, BLE
gilded cradle
👍
gilded cradle
Fun without boundaries?
slender iron
coding untethered
tulip sleet
@river quest BLE is on a roll; it's a Bluefruit Playground;Bluetooth at the Playground; Bluetooth on the Playground [or BLE or Bluefruit]; BLE Merry-Go-Roundl BLE Carousel
stuck elbow
@slender iron any idea what might have broken stage this time?
slender iron
@tulip sleet the oled still isn't happy. found it with the hidden testing
@stuck elbow which pr broke it?
tulip sleet
i didn't test it; i thought you probably had done so
stuck elbow
the displayio overhaul, I assume
slender iron
I tested that it no longer hanged 🙂
but it doesn't correctly show anything. I got a negation wrong
stuck elbow
I didn't test every PR, I will try to narrow it down tomorrow, but I was hoping you would have an idea
slender iron
@stuck elbow I was hoping my changes were enough to keep it working but obviously didn't test it
stuck elbow
obviously
slender iron
which display are you using and what bus?
stuck elbow
ST7735 4wire
if you don't have a suggestion, I will do a proper debugging session with a logic analyzer tomorrow
stuck elbow
it keeps displaying the console instead of the things I'm sending to it
so it's similar to how it was last time
slender iron
ah, how do you turn the console off?
stuck elbow
as if the display didn't get any commands
slender iron
well auto refresh will keep sending the console to the display unless you turn it off
stuck elbow
I did turn it off
slender iron
how?
stuck elbow
1.0f, // brightness (ignored)
true, // auto_brightness
false, // single_byte_bounds
false, // data_as_commands
false, // auto_refresh
20); // native_frames_per_second
but wouldn't there be at least blinking?
stuck elbow
I'm more suspicious about:
display->core.send(display->core.bus, DISPLAY_COMMAND, CHIP_SELECT_TOGGLE_EVERY_BYTE, &display->write_ram_command, 1);
what is that CHIP_SELECT_TOGGLE_EVERY_BYTE for?
slender iron
it's for some displays that need a cs toggle every command byte
I believe we were doing that for everything already
stuck elbow
so shouldn't it depend on the display, and not be hardcoded?
stuck elbow
then why is it a parameter?
crimson ferry
@slender iron Yeah, I2C is funky in latest S3 build e92b5f7. OLED works with terminalio but seems to break when displayio kicks in. Also seemed to mess up my I2C RTC. I can isolate and test more if you need.
stuck elbow
@slender iron I will debug it tomorrow, it's already too late today, thank you
slender iron
@stuck elbow k, sleep well. sorry for breaking it
stuck elbow
you can make an omelette etc.
slender iron
I love omelettes
@crimson ferry this is the fix: https://github.com/adafruit/circuitpython/compare/master...tannewt:displayio_hidden?expand=1#diff-09099cb6669a91a8dc5e4495e0a35e7dL100
crimson ferry
OK, cool thanks. I'll check it at next S3 build.
manic glacierBOT
tulip sleet
@slender iron https://github.com/adafruit/circuitpython/pull/2115 is not running checks at all - might this be due to #2112??
raven canopy
@tulip sleet was #2112 running when you pushed #2115? is the job rate limit what you estimate kept #2115 from starting?
tulip sleet
no, the check is not even pending
#2112 changed a token, but maybe it did something else?
raven canopy
i just pushed to Mixer. job started right up...
2115 is an enigma. it would make sense to me that there were already 20 jobs running on the repo, which would keep 2115 from starting. maybe GH is still working out the bugs on the queue system... 🤷
Usage limits
Exceeding usage limits may result in jobs queueing, failing to run, or failing to complete. Limits are subject to change.
- You can execute up to 20 workflows concurrently per repository.
- You can run up to 20 jobs concurrently per repository across all workflows.
@raven canopy it should still be queued, that's the weird thing. I can understand it might be wiating, but there's no box about checks in the "Conversation" tab
If you click on the "Checks" tab, it's not quite blank, but there's nothing there to try to restart or monitor.
raven canopy
yeah, was just about to mention the "Checks" tab showing your commit...and blankness. that's kind of what nudges me to GH's queue system being the culprit.
it also doesn't show up in the "Actions" tab at the top. its just...everywhere and nowhere at the same time.
and unfortunately, the only way to "manage workflows" is in the "Actions" tab.
tulip sleet
I can try closing and submitting a new PR with the same branch.
raven canopy
or try the old empty commit trick?
tulip sleet
I tried that, I just made a trivial change; that is the d3cb103 commit.
raven canopy
ahh. commit message makes total sense now. 😄
tulip sleet
let me try to close and resubmit
tulip sleet
@raven canopy, ok I merged from master and re-pushed, and now it's going
Maybe because this PR was started before the switch to GitHub actions.
raven canopy
Mixer only has 9 jobs still running, so no "proof positive" on my theory. 😦 i'm not sure the Actions switch would be the cause. Mixer switched mid-stream, as well.
at any rate. its running now! 🎉
raven canopy
the wording on that new github "feature" that dismisses reviews when changes are subsequently pushed could better. its so rude... hehe
prime flower
@onyx hinge that'd be an interesting way to do something rosie-like
But it doesn't support other libraries - only cpx
onyx hinge
so it's a very high-level simulation I guess
I didn't look into it very far at all
When I was interviewing at verypossible.com they mentioned another ARM simulator that is Python powered, now I can't remember the name. Commercial product.
ionic elk
Hi @gentle bronze, was wondering if adding F412 USB support is still on your radar. It'd be very nice to have that for adding QSPI flash support, since the F411 Discovery doesn't have a QSPI chip.
I know you've got a lot on your hands with the BLE stuff.
gentle bronze
@ionic elk I am going to do it next, was fixing an bug with qspi flash on Arduino 🙂
plucky flint
Hi folks, if anyone would like to try out, break and give feedback for circup I'd be very grateful. Just follow the "Developer Setup" instructions here: https://github.com/ntoll/circup
Sneak peak of updating a CPX. Thanks to @tulip sleet for a really cool approach that simplified and sped things up. 👍
Right... I'm off to play my tuba in an orchestral rehearsal this evening. Catch you tomorrow. 👋 🎵
idle owl
@plucky flint Have a good evening! I'll take a look today 🙂
tulip sleet
yay!
plucky flint
Thank you!
Please give feedback via GitHub issues (forgot to mention)
I've tested it on Linux and Windows, if you have a Mac it might be helpful to check too. Thanks... (what does it do..? Once installed just type circup to get the details)
-> AFK
slender iron
@tulip sleet what android do you use for testing?
tulip sleet
checking...
android 9 on a samsung tablet, SM-T510, reasonably priced and the latest android available when I bought it at Best Buy
ionic elk
@tulip sleet took a look at your HID PR. Looks like a much cleaner solution than my cdc_and_msc_only flag - should I simply remove that from my current PR, and then we merge yours?
tulip sleet
that sounds fine, yes, the idea was to generalize the possibilities
ionic elk
An additional note that should be considered is that the MSC endpoints need to change to support low endpoint devices. @slender iron, you were correct that setting both MSC endpoints to 0 results in issues on the SAMD21 (51 is fine).
slender iron
👍
ionic elk
I was going to add a flag operation there to sort it out, but dan's PR will mess that up, so I'd suggest we
- merge my PR, with all endpoint stuff removed, accepting that it'll break the STM32 port for a short while
- merge Dan's PR,
- I can integrate dan's code into the MSC endpoints
ionic elk
Do you want STM32 included in that? I don't figure there are many people using it, but the F411 won't work unless the MSC stuff is properly configured.
I'm fine leaving it broken but it's up to you
slender iron
nah, lets shoot for the next alpha
ionic elk
Ok. It should be done pretty quick
slender iron
np, I want to pick the release pace back up
ionic elk
@slender iron if you could take a quick look at my port.c comments, we should be done for this PR.
slender iron
yup, have it queued up in my tabs
will look in the next hour tops. I'm in "get through email" mode atm
orchid basinBOT
tulip sleet
@ionic elk #2065 is not testing because it needs to get merged from tip of master to pick up the new CI settings:
git fetch adafruit # or upstream if that's what you call the remote
git merge adafruit/master
# resolve conflicts
git push
@tulip sleet getting rejection on the push ! [remote rejected] f411re-feature-buildout -> f411re-feature-buildout (refusing to allow an integration to create or update .github/workflows/build.yml)
tulip sleet
did git merge adafruit/master say there were merge conflicts?
ionic elk
no
tulip sleet
@ionic elk it's odd, it worked for me, do you have a circuitpython/.github/workflows/build.yml locally?
ionic elk
I believe I do, am I not supposed to? I'm seeing similar problems from other projects that say you have to delete that the rejected workflow file from git in order to push. Is that something that should be excluded during the merge process in some way?
tulip sleet
i don't know; I did exactly the above and it worked, but the message clearly implies not
are you using git: or https urls for github?
I am using https
stuck elbow
@slender iron it does not
ionic elk
https
even deleting the file is resulting in multiple other yml files pushing the same issue
slender iron
@stuck elbow ok thanks, removing it to free up space for .hidden
ionic elk
apparently it might be able to be solved by pushing to a git url?
tulip sleet
that's what they said, but I'm not sure, and I never use those. I wonder if it's because I have admin privs on the repo.
orchid basinBOT
tulip sleet
@ionic elk still looking @slender iron are you following this PR push problem?
ionic elk
Ok, I deleted a couple files and it seems to have pushed now, but I don't know if that's going to cause a conflict
slender iron
@tulip sleet I don't have any brilliant ideas. I've never seen it before. I'd probably rebase the whole branch on latest master
ionic elk
it appears those files are required for it to run tests as well
tulip sleet
the whole idea of removing those files seems wrong
@ionic elk I guessed it might be a security issue about uploading possibly malicious actions (see https://github.com/renovatebot/config-help/issues/192), but these issues are old, from March or earlier, so I don't see how to fix this problem
tulip sleet
i am not sure that has to do with it; the basic issue is that GitHub doesn't want people pushing malicious .github/workflow/*.yml files in a PR, but that seems to conflict with the idea of triggering an action on a PR push
@ionic elk https://github.com/desktop/desktop/issues/7079 notice it was reopened yesterday 😦
ionic elk
Yikes
tulip sleet
are you pushing from linux or windows?
ionic elk
Mac
This rebase is really awful, should I force revert back to before the merge?
We shouldn't have issues since basically all the changes were approved by travis, but I don't know if that's against protocol
tulip sleet
ok, do that for now. I really don't think the rebase is going to fix anything - it's not a merge issue, it's a policy issue we're running into
meanwhile I am going to keep reading. @slender iron has an inside helper at github who responds to twitter queries, and we can ask him once we understand it clearly
@ionic elk are you using github desktop or just cli?
ionic elk
cli
Yeah the PR only affects two files outside the port and both of them were tested back when we had travis
tulip sleet
and how are you authenticating to github?
ionic elk
It should be safe to merge
Https, I guess? However it sets itself up?
Wait hang on I think it's probably osxkeychain helper
tulip sleet
this appears to be the clearest explanation of what's going on, and it's a lot more recent: https://github.community/t5/GitHub-Actions/Can-I-edit-the-main-workflow-github-actions-file-locally/m-p/26265
@ionic elk maybe turn off the keychain helper for a bit and have the cli prompt you for a password. But we can defer this and just merge the PR as is, and test all this later
ionic elk
I'll see if that works.
I think @slender iron is all set so we should be good to go once you're done.
slender iron
merge party!
slender iron
lunchtime for me
orchid basinBOT
tulip sleet
@ionic elk could you try a very simple PR: get your repo up to date (reclone if necessary and get all the submodules up to date), and then make a new PR that adds the two stm32f4 boards to the build. That can test whether a new PR from you with the workflow stuff is pushable or not. I will add details in the basecamp to-do
if there is still a github problem we'll pout that on hold and you can work on the pin table stuff. How does that sound for the next small step?
ionic elk
Sure. By adding to the build you mean the github action build?
tulip sleet
right, there are several files to modify, because this is a brand new port. I'm just writing it up.
ionic elk
I think what I actually need is a new fork
since I can't push anything to my current one
tulip sleet
i don't think it's the particular fork, it's that this restriction about changing the workflow files is tricky
when you used the osxkeychain manager, did you type your actual github password in or download a token from github? I am using a linux credentials manager and I think I had to get a token
as opposed to some OAuth thing like your Google login?
ionic elk
But if I have a new fork, it'll sidestep the issue - I can't make alterations additions or removals of workflow files, but if I just start from a new fork, I won't need to
tulip sleet
you can make all the changes but the actual build.yml, maybe
do you have 2FA on github?
tulip sleet
which worked, a new fork?
ionic elk
I thought I'd been able to modify it but deleting the credential worked
I can push the changes now
ionic elk
that top link is a bit broke
tulip sleet
@ionic elk how are the endpoints allocated right now on the '411? Do you need MSC to be only one endpoint at the moment?
ionic elk
Yes, if it takes up two, the way it is now, we don't have enough for HID, MSC and CDC at the same time
Or wait, not even HID I think. Let me double check that
tulip sleet
0 is control, 1 and 2 need to be bidirectional CDC contorl and data, that leaves only one more bidi eiether for MSC or HID, or drop CDC and have MSC and HID
HID is less important - people can use other boards for that
ionic elk
Yes that's right. Just checked as well
tulip sleet
ok, i will add the flag to my PR, just wanted to make sure it was needed right now
did you see the notes I added to basecamp about what files to change for the automated build?
idle owl
@tulip sleet The capacitive touch on the nRF isn't built into the chip, right? How is it done?
ionic elk
Yes I did. I've been temporarily evicted from my desk due to construction but I'll be right on it after
tulip sleet
@idle owl the code sets a DigitalInOut to out to 1, and then switches it to input, and sees how long it takes to decay to zero. The must be a 1 megaohm pulldown resistor on the pin
idle owl
@tulip sleet hmm.... ok. need to translate that to beginnerese. Thanks!
onyx hinge
@raven canopy hey #2080!
raven canopy
@idle owl "On Cortex-M4 chipsets (SAMD51, nRF52840), touchio does not use the built-in touch controller. It uses a 1MOhm resistor as a pulldown to detect capacitive touch." better? 😄
idle owl
@raven canopy well done
onyx hinge
"To function properly, a 1MOhm resistor must be connected from the IO pin to GND"?
raven canopy
^^^^
onyx hinge
(I assume these are built on on CPB?)
idle owl
Yeah all built in
tulip sleet
it treats the pin and whatever it's conencted to as a capacitor. It charges the capacitor, and sees how long it takes to discharge. It needs a 1MOhm resistor connected to ground to provide a discharge. It is mearusing the size of the capacitor by measuring the disharge time.
idle owl
so... WAY too much info there
tulip sleet
Something touching the pin increases the capacitance
idle owl
This is for the "guided tour" page. It's a "here's the basics of your CPB." type thing
raven canopy
hehe. "To understand touchio on the CPB, please proceed to the nearest University and acquire an electrical engineering degree. k thanks!"
idle owl
hah!
onyx hinge
"Pins A1, ... A6 can be used as touch sensors. Although the principle is slightly different from CPE, they function the same in CircuitPython programs"
tulip sleet
"CIrcuitPython can do capacitive touch on any nRF pin that supports DigitalInOut, but because there is not built-in circuitry for this, the pin must be connected to a 1MOhm resistor whose other end is connected to ground."
onyx hinge
also, looks like they're 330kOhm on the CPB
marking says 310 but I was measuring around 330k
tulip sleet
@onyx hinge latest schematic has 1M
onyx hinge
okay
tulip sleet
maybe you are measuring leakage through the chip? try reversing the probes and see if its' the same
onyx hinge
.365M and .335M measuring from GND to A1 on my CPB.
"310" marking doesn't even make sense to me, but I get about that same .3XX M reading across one of them
onyx hinge
OK, 01E would be 100 x 10000
maybe the fault lies with my multimeter then
not sure I care enough to track down a bare 1Mohm resistor right this second, sorry for the distraction!
tulip sleet
i am getting 1M with mine 🙂
onyx hinge
adds a new meter to the shopping list
manic glacierBOT
ionic elk
I see it yells at me for not including the F412zg board. What would I do if I wanted a board to be intentionally excluded from test?
I don't recall entering that board into any other config file. Does it search through and insist that every single board is included?
slender iron
ya, it insists. why not build it?
<@&356864093652516868> any objections to an alpha 2?
idle owl
Nope.
tulip sleet
no, the HID/USB cmpile-time stuff can wait
gilded cradle
None from me
pastel panther
newp
crimson ferry
None
tidal kiln
i never got a chance to try alpha 1 😦
slender iron
k, will kick it
stuck elbow
@slender iron no, assuming there will be an RC
stuck elbow
awesome
simple pulsar
RCS - never forget
onyx hinge
@simple pulsar oh my, I'd rather forget Revision Control System, assuming that's the RCS you mean.
@slender iron a bit late, but go for it!
slender iron
👍
stuck elbow
there are worse things than RCS, Team Foundation Server anyone?
orchid basinBOT
onyx hinge
Happily, I never used any version control software from Microsoft
slender iron
904 release assets!
pastel panther
yowsers
slender iron
@raven canopy @idle owl either of you run adabot regularly to set Travis credentials?
tulip sleet
@slender iron ok, yah SAMD21 is flaky with MSC endpoints on same pair. It works after reset but not when first plugged in. Still trying to find where we wrote this up or found it on the web.
slender iron
I think I just moved on
tulip sleet
you can merge #2120 unless you want sommersoft's review too
tulip sleet
@ionic elk re needing two endpoint pairs for MSC on SAMD21, did I tell you that or did you find it out elsewhere? I just want to make the right choice for the non-SAMD21 boards. a single endpoint pair seems to work for all boards except SAMD21, but I have yet to find clear doc of that
raven canopy
@slender iron i do not. would it even work with my credential?
manic glacierBOT
idle owl
@slender iron How do you mean?
raven canopy
@idle owl run adabot with -t/--travis-github-token argument, so she sets the Travis token if it isn't set.
slender iron
@raven canopy I'm happy to give the password to you if you like
plucky flint
morning 👋
exotic pumice
middle of the night 👋
stuck elbow
afternoon 👋
plucky flint
Heh.... as an old time IRC user, my timezone is always UGT (http://www.total-knowledge.com/~ilya/mips/ugt.html) 😉
onyx hinge
as an old time IRC user, I never heard of UGT before now
plucky flint
@onyx hinge heh... I can't remember how I heard about it, but once it was explained to me (perhaps 20+ years ago), it made prefect sense. I've been an advocate ever since -- simply because it's fun and rather silly in a friendly sort of a way. 😉
plucky flint
Hi folks, so I've done a bunch of updates to the docs for circup (see the README here: https://github.com/ntoll/circup/ -- or better clone the repos, install the deps into a virtualenv and make docs to generate what'll end-up on ReadTheDocs). All feedback most welcome..! Note, since I'm working in London today (I'm due to run a code dojo in a couple of hours) I've only got my Linux laptop with me. I don't think any of the changes I've made should break it working on Windows or OSX, but if they do, please ping me. I'll be testing x-platform tomorrow.
But right now, I'm off for a (very) late lunch. I'll be online later to pick up any feedback if need be. Thank you!!! 🙂
stuck elbow
good luck with the dojo!
umbral dagger
I see a few forks of the AMG88xx driver. Is anyone working on supporting settable rotation?
ionic elk
@tulip sleet @slender iron just saw your messages in livebroadcast - my testing yesterday found that there was no impact of a single MSC endpoint on the SAMD51, but that it prevented code writing on the SAMD21 (though no other USB features were inhibited). I don't have a nordic board so I wasn't able to test that.
tulip sleet
@ionic elk Ah, ok, I didn't realize you tested it thoroughly from scratch. I think Scott discovered this independently when he was first doing MSC for SAMD21. There are hints of the problem in various Atmel forum posts, but nothing definitive. The latest version of my PR does the right thing. If you're willing to finish the review for that we can get it in.
are you finished with the stm32f4 build PR? I can look at that.
ionic elk
I will switch right back to that. The stm32f4 build PR is done, the latest check just failed for some internal reason (unrelated to code) so I had to restart it. An earlier build that differed only by a redundant comma passed just fine so it should be set to review.
manic glacierBOT
ionic elk
@tulip sleet Could you take a look a the build PR check error? It's throwing some git thing, related to Run for exp in *.exp;?
tulip sleet
sure
tulip sleet
@ionic elk it looks like the test build checks out the version to test in a different way than the board builds. This causes the test build to fail, I think maybe whenever a new merge has happened while the test is in progress. This has happened to me a couple of times. If I merge from upstream and push, then it works. I'll look at the workflow .yml flows
@slender iron ^^ looking at this
ionic elk
What would you suggest for this test? Do I need to update anything or should I just restart one more time?
tulip sleet
Ideally I can fix it. Or as a temp fix, fetch from adafruit, and merge adafruit/master, and push again
I think that cause it to work unless there's another PR merged before this one is
go ahead and try that and may vet my theory
they both use actions/checkout@v1, hmm...
ionic elk
fetched and merged... let's see what happens
tulip sleet
let's mark your to-do as done after it's merged
slender iron
@tulip sleet https://github.com/actions/checkout/issues/27
tulip sleet
@slender iron yup, that looks like it, so maybe just a re-run is good enough, as long as master is quiescent during that time
tulip sleet
it's a bug that should be fixed, no comment yet from github folks
slender iron
tulip sleet
ok, yet another repo to keep track of. I was looking around for the right place to search, didn't know it was github/actions
slender iron
ya the actions/checkout is actually a repo name
tulip sleet
@v1 is a tag, iguess
tulip sleet
but if it's buggy, then we will need to advance to @v2? or should we leave the tag off?
so we pick up any fixes?
or will they advance the tag?
🤷
slender iron
I'd expect a v2