Re: [new] databases/py-influx-client

On 2022/06/28 07:48, Lucas Raab wrote:
> On Mon, Jun 27, 2022 at 02:46:55PM +0100, Stuart Henderson wrote:
> > On 2022/06/27 08:39, Lucas Raab wrote:
> > > On Sun, Jun 26, 2022 at 06:15:47PM +0100, Stuart Henderson wrote:
> > > > On 2022/06/25 15:53, Lucas Raab wrote:
> > > > > Hello,
> > > > >
> > > > > Here's a new port to connect to InfluxDB instances running 1.8,
> > > > > 2.0+, and Cloud. This is in contrast to the existing databases/py-influxdb
> > > > > which is only for InfluxDB 1.7 or less.
> > > > >
> > > > > Lightly tested against an InfluxDB Cloud instance running some
> > > > > queries. If anyone has 1.8/2.0 instances to try this with, that'd
> > > > > be great.
> > > > >
> > > > > Other note: there's asynchronous support with aiohttp, but only
> > > > > with 3.8. 3.7.4 is in the tree currently and I haven't looked at
> > > > > updating it. Other optional dependencies are numpy and pandas if
> > > > > you wanted to use DataFrames for query/data import. I left those
> > > > > out of RUN_DEPENDS since they're marked as extra, but put a note
> > > > > in DESCR.
> > > >
> > > > "Optional dependencies are:
> > > > math/py3-pandas
> > > > math/py3-numpy"
> > > >
> > > > I prefer to use package rather than port names for this, when somebody
> > > > wants to install them they'll normally be using pkg_add and may not have a
> > > > ports tree available
> > > >
> > > > It might be helpful to explain what they're useful for too?
> > > >
> > > > Otherwise OK
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > > Other thoughts and feedback?
> > > > >
> > > > > Thanks,
> > > > > Lucas
> > > >
> > > >
> > >
> > > Something like this?:
> > >
> > > These optional dependencies can be used to upload or query data using pandas
> > > DataFrame objects:
> > > py3-pandas
> > > py3-numpy
> >
> > Hmmm actually, py-pandas depends on numpy anyway (of course), and you're
> > not going to expect to do anything with pandas objects unless you have
> > pandas installed anyway...so I guess this can just be omitted
> >
> Fair enough. Good to import after removing that part?

yep