BUG: fix construction of Series / Index from dict keys when "str" dtype is specified explicitly

[tasfia8]

• This PR fixes BUG (string): contruction of Series / Index fails from dict keys when "str" dtype is specified explicitly #60343 @jorisvandenbossche and is a continuation of my earlier PR Bug Fix: #60343 Construction of Series / Index fails from dict keys when "str" dtype is specified explicitly #60383.

The default behavior (pd.Index(d.keys())) worked correctly, but explicitly setting dtype="str" raised a ValueError. The issue stemmed from dict_keys not being converted to a proper array-like structure before being passed to StringDtype, which couldn't handle such inputs.

To fix the issue:

• KeyView was introduced to identify and preprocess dict_keys before passing them to Pandas internals. The keys are now converted to a list for compatibility.
  -Handles Existing Test Cases and fixes previous PR fix issues.
  After the fix both the default (pd.Index(d.keys())) and explicit (pd.Index(d.keys(), dtype="str")) cases work:

[tasfia8]

@jorisvandenbossche Can you check now please and if possible merge? I considered your comments in this updated PR and added the generic if. Now besides the dict.keys(), it will handle other iterable such as dict.values() and addressing other similar types.

My IDE is throwing some unintentional formatting issues which I can't seem to undo, but in this commit, I only changed the if condition in the construction.py and nothing else

[github-actions]

This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this.

[mroeschke]

Thanks for the pull request, but it appears to have gone stale. If interested in continuing, please merge in the main branch, address any review comments and/or failing tests, and we can reopen.

[jorisvandenbossche]

@tasfia8 apologies for the flow response. Your main commit eb11a27 looks good, but there are still a bunch of unrelated changes. I will push a change to this branch to clean that up.

In addition, we also still need to add a test.

[jorisvandenbossche]

@tasfia8 thanks for the initial contribution!

[lumberbot-app]

Owee, I'm MrMeeseeks, Look at me.

There seem to be a conflict, please backport manually. Here are approximate instructions:

1. Checkout backport branch and update it.

git checkout 2.3.x
git pull

2. Cherry pick the first parent branch of the this PR on top of the older branch:

git cherry-pick -x -m1 84bf1ef82912ebf497a304b0ffd90914bfc41ea9

3. You will likely have some merge/cherry-pick conflict here, fix them and commit:

git commit -am 'Backport PR #60436: BUG: fix construction of Series / Index from dict keys when "str" dtype is specified explicitly'

4. Push to a named branch:

git push YOURFORK 2.3.x:auto-backport-of-pr-60436-on-2.3.x

5. Create a PR against branch 2.3.x, I would have named this PR:

"Backport PR #60436 on branch 2.3.x (BUG: fix construction of Series / Index from dict keys when "str" dtype is specified explicitly)"

And apply the correct labels and milestones.

Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon!

Remember to remove the Still Needs Manual Backport label once the PR gets merged.

If these instructions are inaccurate, feel free to suggest an improvement.

[jorisvandenbossche]

Manual backport -> #60793