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

E2E improve: col_to_ndx() #2678

Closed
Tracked by #2672
jadudm opened this issue Nov 1, 2023 · 0 comments · Fixed by #2803
Closed
Tracked by #2672

E2E improve: col_to_ndx() #2678

jadudm opened this issue Nov 1, 2023 · 0 comments · Fixed by #2803
Assignees
@phildominguez-gsa

Comments

@jadudm
Copy link
Contributor

jadudm commented Nov 1, 2023 •
edited by phildominguez-gsa
Loading

col_to_ndx()

This may need to be better.

For any column number greater than 26, this might break.

We need to map through A...Z, AA...AZ, etc.

A function that generates the Excel column references, and then numbers them, would be a good, quick fix.

We may want this elsewhere later. Having that is going to be useful.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@phildominguez-gsa phildominguez-gsa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replacing col_to_ndx() with column_index_from_string() GSA-TTS/FAC
2 participants
@jadudm @phildominguez-gsa