Gareth Hancock

Results 117 comments of Gareth Hancock

Hi @kevinlinxc I'm not familiar with React CRA but after a little scanning a couple of things jump out. The error you mentioned appears to occur when the module is...

Hi @kevinlinxc are you able to describe your usecase in a little more details? I wonder if you might be better placed to use something else rather than CRA but...

Hi @Geekz45679 could you please confirm the versions of VS build tools and python that you are using?

Thanks @Geekz45679 Could you check if you have the VS build tools included in your Path environment settings? In my case while I have both 2019 and 2017 installed only...

> @GazHank Unfortunately that didn't seem to do the trick. > > Here is how I have it set up: > > ![image](https://user-images.githubusercontent.com/22227801/134722435-c4286b6c-e3eb-4ee3-bd39-e1a1f2407544.png) > > Hmm, since you have python...

Here are links to some potentially associated "issues", in case it helps: - [New Binding Interface](https://github.com/serialport/node-serialport/issues/1679) - [PR: Unify API between WebSerial and this](https://github.com/serialport/node-serialport/issues/2238) - [Web Serial Spec reference implementation](https://github.com/serialport/node-serialport/issues/1669)

Thanks for raising this @tcbennun it looks like the lowlatency fix needs some more work. I'm glad the the resequencing of the logic worked, so that only the low latency...

Thanks @tcbennun, it's disappointing to be correct :-( We previously tried to use `#if defined(__linux__) && defined(ASYNC_LOW_LATENCY)` to protect environments which didn't understand (/were incompatible with) the low latency mode,...

> Maybe enable the feature during open and don’t mess with it if it’s > disabled. Makes sense, although at the moment I think LL is only an option during...

Sounds good. I'll see if I can draft something over the next few days. I'd consider including this in with the N-API changes (to avoid duplication) but am not sure...