Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testing regarding data type mapping definitely needs to be improved. #592

Open
eitsupi opened this issue Dec 10, 2023 · 3 comments
Open

Testing regarding data type mapping definitely needs to be improved. #592

eitsupi opened this issue Dec 10, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@eitsupi
Copy link
Collaborator

eitsupi commented Dec 10, 2023

Originally posted by @eitsupi in #589 (comment)
@eitsupi eitsupi added the enhancement New feature or request label Dec 10, 2023
@eitsupi eitsupi mentioned this issue Dec 10, 2023
Merged
@eitsupi
Copy link
Collaborator Author

eitsupi commented Dec 10, 2023

Related to #425

@etiennebacher
Copy link
Collaborator

@eitsupi can you clarify a bit what you mean by this? I could take a look at some point if I have more info (unless you wanna do it)

@eitsupi
Copy link
Collaborator Author

eitsupi commented Jan 17, 2024

It means that it has not been tested for the mapping of R and Polars types. For example, in arrow and nanoarrow, we can see the corresponding type in the infer schema functions, but currently in this package type conversions are hard-coded in each function, making it difficult to guess how they are actually converted.

@eitsupi eitsupi mentioned this issue Feb 12, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@eitsupi @etiennebacher