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

[Discussion] LinearAlgebra extension #6

Open
lkdvos opened this issue Dec 12, 2024 · 1 comment
Open

[Discussion] LinearAlgebra extension #6

lkdvos opened this issue Dec 12, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@lkdvos
Copy link
Contributor

lkdvos commented Dec 12, 2024

It is not entirely clear to me if the extensions for LinearAlgebra belong in a package extension, or should just be a part (possibly submodule) of the main module. For now, it would be loaded by default anyways, as LinearAlgebra is still an stdlib that is always included, but it looks like that might not always be the case in the future.

On the other hand, there is probably a large part of the package that actually does depend on implementations in LinearAlgebra, so I would guess that just having this in the main package is the best way forwards, considering that in typical usecases, projects will have LinearAlgebra loaded one way or another.

@lkdvos lkdvos added the enhancement New feature or request label Dec 12, 2024
@mtfishman
Copy link
Member

I think I was just undecided about that so for the sake of keeping the option open of creating a package extension I organized it in that way to make the dependency structure of the codebase clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants