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.

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

[Feature request] Retrieve column letter #2091 #2305

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

philipdp123
Copy link

This PR aims to resolve #2091.

I noted that you had a function that achieves this task. My PR contains a reworked version of xw.utils.col_name inclusive of a few test using pytests to ensure that the function works as expected.

@mullimanko
Copy link

This is great, thank you. When will this be implemented?

@fzumstein
Copy link
Member

I am not quite sure about the motivation of this PR as the function is already implemented (I didn't remember this, so forgot to mention it in #2091). Also keep in mind that xlwings.utils is also been more of an internal API.

The new implementation is almost twice as slow as the current implementation, so not likely an improvement?

Screenshot 2023-09-24 at 2 23 17 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature request] Retrieve column letter
3 participants