Re: [sqlalchemy] Issue with type signature of DBAPI Connection

2023-11-07 Thread Mike Bayer
there does seem to be an extra Sequence on the outside that should be removed, 
please open an issue.


On Tue, Nov 7, 2023, at 9:46 PM, William Hakim wrote:
> I was recently using the DBAPI Cursor, and it seems to me that the type 
> signature of the `executemany()` function is incorrect:
> 
> https://github.com/sqlalchemy/sqlalchemy/blob/main/lib/sqlalchemy/engine/interfaces.py#L194
> 
> The given type signature for the second argument to `executemany()` is ` 
> "Sequence[Sequence[Sequence[Any]] | Sequence[Mapping[str, Any]]]"`, however 
> this doesn't appear right (and doesn't actually work) - it seems like it 
> should instead be `Sequence[Sequence[Any]] | Sequence[Mapping[str, Any]]`.
> 
> Happy to open a Pull Request if this is indeed an issue.
> 
> Will
> 
> 
> -- 
> SQLAlchemy - 
> The Python SQL Toolkit and Object Relational Mapper
>  
> http://www.sqlalchemy.org/
>  
> To post example code, please provide an MCVE: Minimal, Complete, and 
> Verifiable Example. See http://stackoverflow.com/help/mcve for a full 
> description.
> --- 
> You received this message because you are subscribed to the Google Groups 
> "sqlalchemy" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to sqlalchemy+unsubscr...@googlegroups.com.
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/sqlalchemy/048482ea-0c27-46e3-9acf-6257d685f855n%40googlegroups.com
>  
> .

-- 
SQLAlchemy - 
The Python SQL Toolkit and Object Relational Mapper

http://www.sqlalchemy.org/

To post example code, please provide an MCVE: Minimal, Complete, and Verifiable 
Example.  See  http://stackoverflow.com/help/mcve for a full description.
--- 
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sqlalchemy+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sqlalchemy/4898aaa6-94da-4bfc-baab-d537688ced68%40app.fastmail.com.


[sqlalchemy] Issue with type signature of DBAPI Connection

2023-11-07 Thread William Hakim
I was recently using the DBAPI Cursor, and it seems to me that the type 
signature of the `executemany()` function is incorrect:

https://github.com/sqlalchemy/sqlalchemy/blob/main/lib/sqlalchemy/engine/interfaces.py#L194

The given type signature for the second argument to `executemany()` is ` 
"Sequence[Sequence[Sequence[Any]] | Sequence[Mapping[str, Any]]]"`, however 
this doesn't appear right (and doesn't actually work) - it seems like it 
should instead be `Sequence[Sequence[Any]] | Sequence[Mapping[str, Any]]`.

Happy to open a Pull Request if this is indeed an issue.

Will

-- 
SQLAlchemy - 
The Python SQL Toolkit and Object Relational Mapper

http://www.sqlalchemy.org/

To post example code, please provide an MCVE: Minimal, Complete, and Verifiable 
Example.  See  http://stackoverflow.com/help/mcve for a full description.
--- 
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sqlalchemy+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/sqlalchemy/048482ea-0c27-46e3-9acf-6257d685f855n%40googlegroups.com.