Skip to content

PR: Show contents of Numpy string arrays in the Variable Explorer #24150

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

Merged
merged 1 commit into from
Apr 25, 2025

Conversation

jitseniesen
Copy link
Member

Description of Changes

Show the contents of arrays of strings in the Variable Explorer.

This PR incorporates spyder-ide/spyder-kernels#535 and nothing more.

string-array

Issue(s) Resolved

Fixes #22570

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:
Jitse Niesen

@jitseniesen jitseniesen marked this pull request as ready for review April 24, 2025 07:15
@jitseniesen
Copy link
Member Author

I do not understand what is happening with the Mac tests.

@ccordoba12
Copy link
Member

I do not understand what is happening with the Mac tests.

I decided to simply skip the failing tests on PR #24294.

Also, after that PR is merged and when you have rebased spyder-ide/spyder-kernels#535, please rebase this branch, remove the commit you pushed to it, and then resync our subrepo again with the changes in your Spyder-kernels PR.

@ccordoba12
Copy link
Member

@jitseniesen, please resync our subrepo again now that I merged your Spyder-kernels PR.

…r-kernels.git external-deps/spyder-kernels

subrepo:
  subdir:   "external-deps/spyder-kernels"
  merged:   "910936cee"
upstream:
  origin:   "https://github.com/spyder-ide/spyder-kernels.git"
  branch:   "master"
  commit:   "910936cee"
git-subrepo:
  version:  "0.4.9"
  origin:   "https://github.com/ingydotnet/git-subrepo"
  commit:   "30db3b8"
@jitseniesen
Copy link
Member Author

@ccordoba12 Subrepo resync'ed, tests pass, so this is ready.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jitseniesen!

@ccordoba12 ccordoba12 changed the title PR: Show contents of string arrays in Variable Explorer PR: Show contents of Numpy string arrays in the Variable Explorer Apr 25, 2025
@ccordoba12 ccordoba12 merged commit 585dfd9 into spyder-ide:master Apr 25, 2025
20 checks passed
@jitseniesen jitseniesen deleted the string-array branch April 28, 2025 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Array of strings not displayed in Variable Explorer pane
2 participants