Dan Rose

Results 564 comments of Dan Rose

GitHub CI just deprecated python 2 so our CI workflow is now failing: https://github.com/actions/setup-python/issues/672

Here's some things I noticed. Some of this might be in scope for #829. 1. There is some documentation that needs to be updated to remove support for python 2:...

> Let's keep the pull requests minimal as [experience tells me](https://github.com/ArduPilot/pymavlink/pulls?q=is%3Apr+author%3Acclauss) that review cycles can be really long on this repo. Agreed. That's why I commented on this issue instead...

A more complete answer to 2, which is helpfully explained [here](https://github.com/ArduPilot/pymavlink/blob/f41a355596a363b0855093268b9d6a157261fc67/generator/mavgen.py#L252-L256). While type hints were [introduced in 3.5](https://peps.python.org/pep-0484/), variable annotations weren't [introduced until 3.6](https://peps.python.org/pep-0526/). The mavgen-generated code for python3 includes...

The *most important* thing is to set expectations. Python 3.5, 3.6, and 3.7 are all end-of-life. @hamishwillee: 1. For each of these out-of-support versions, when is PyMavlink dropping it as...

> My take is "so what" - it isn't like you [can't upgrade easily on that platform](https://www.itsupportwale.com/blog/how-to-upgrade-to-python-3-10-on-ubuntu-18-04-and-20-04-lts/). That's half of it. The other half is why do you feel compelled...

I think it's a bug that the web frontend and `mavlink2rest` only appear to be listening on IPv4. e.g. `wget -6 localhost:80` fails e.g. `wget -6 localhost:6040` fails Some other...

I dislike the workaround and don't love the idea of enshrining it. It's superficial (e.g. it shouldn't but does allow `x = defineEmits`) and can't provide many of even more...

So my initial intent here was so that I could install nightly builds that *don't* have a stable versioning scheme, e.g. canary builds like: https://nodejs.org/download/nightly/v22.0.0-nightly202311184e23d6904c/ The other intent was to...

> I think for this kind of stuff, we can use prerelease or build metadata I strongly disagree. 1. URLs allow getting a package from a fork or other resource...