ILGPU
ILGPU copied to clipboard
Added metapackage containing all ILGPU libraries.
Not sure if we want this meta-package or not, but might be useful. Also not sure about the name.
Great idea @MoFtZ!
For the name, I would suggest renaming ILGPU
to ILGPU.Core
, and naming the meta-package ILGPU
. This would mean a breaking change and thus something we would plan for 2.0.
Thoughts?
@MoFtZ I really like this idea 👍
I have to admit that I prefer @jgiannuzzi's idea of creating a "new" ILGPU
package instead of an ILGPU.All
package.
@jgiannuzzi @m4rs-mt
I agree that the name ILGPU.All
is not great. However, I was trying to avoid a breaking change. Also, it matched some existing names like Microsoft.AspNetCore.All
.
I'll put this PR on hold until we decide on a timeframe (and name?).
Something else to consider, if we rename ILGPU
to ILGPU.Core
, are we only renaming the nuget package? Or all the files/folders?
That would include:
-
Src/ILGPU
toSrc/ILGPU.Core
-
Src/ILGPU.Tests
toSrc/ILGPU.Core.Tests
- etc
Renames the ILGPU nuget package to ILGPU.Core. Adds a new ILGPU metapackage.
Depends on #718.
Abandoning this PR.
In the future e.g. v2, it will be easier to deprecate ILGPU.Algorithms
and release the ILGPU
package with ILGPU.Algorithms
baked in.