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

Replace Array{T, 1} with Vector{T} and Array{T, 2} with Matrix{T} #56922

Open
LilithHafner opened this issue Dec 30, 2024 · 0 comments
Open

Replace Array{T, 1} with Vector{T} and Array{T, 2} with Matrix{T} #56922

LilithHafner opened this issue Dec 30, 2024 · 0 comments
Labels
docs This change adds or pertains to documentation help wanted Indicates that a maintainer wants help on an issue or pull request

Comments

@LilithHafner
Copy link
Member

There are still usages of Array{T, 1} throughout the codebase. In some rare cases they are intentional but in many cases this is a holdover from very old versions of Julia. Especially in example output we should pretty much always be using Vector and Matrix rather than Array{T, 1} and Array{T, 2}.

Tagging with the docs label because it is most important to fix this for documentation and less important (if at all) to make the change in source code.

@LilithHafner LilithHafner added docs This change adds or pertains to documentation help wanted Indicates that a maintainer wants help on an issue or pull request labels Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation help wanted Indicates that a maintainer wants help on an issue or pull request
Projects
None yet
Development

No branches or pull requests

1 participant