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: Rename _indsstring to indsstring and allow packages to define methods for it #40059

Open
jishnub opened this issue Mar 16, 2021 · 0 comments

Comments

@jishnub
Copy link
Contributor

jishnub commented Mar 16, 2021

The function _indsstring is used to convert an axis to a string representation in array summaries, and has special methods defined for Base.IdentityUnitRange and Base.Slice. The leading underscore seems to indicate that this is for internal usage. However this is a useful function that other packages that define custom axis types might use too, eg. OffsetArrays. Would it make sense to rename it to indsstring and allow packages to extend this? A search on juliahub indicates that currently this is not being used by any package.

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

No branches or pull requests

1 participant