can you not just get the submodule version from the gitmodules file? - Silk.NET

© 2026 Hedgehog Software, LLC

More

Communities Docs About Terms Privacy

khitiara

K

khitiaraOP•10/18/23, 7:00 PM

i thought that is kept up to date

Perksey

P

Perksey•10/18/23, 7:00 PM

the SHA is checked into the repo in the form of the submodule pointer, but we still need the history to get the date

Perksey

P

Perksey•10/18/23, 7:01 PM

which involves cloning

khitiara

K

khitiaraOP•10/18/23, 7:01 PM

oh right yeah

khitiara

Kkhitiara can you not just get the submodule version from the gitmodules file?

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:01 PM

khitiara

K

khitiaraOP•10/18/23, 7:01 PM

right yeah its stored elsewhere and also not the date

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:01 PM

jeez even a shallow clone takes a million years

khitiara

K

khitiaraOP•10/18/23, 7:01 PM

if there was some way to get a bare clone of the submodules thatd be nice

khitiara

K

khitiaraOP•10/18/23, 7:02 PM

like just the git stuff no checkout

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:02 PM

thats what i thought git submodule init did

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:02 PM

but nope

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:03 PM

shallow clone not recursive of Silk.NET is 4.3GiB

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:04 PM

that should be fine

Lyris the Kitori

L

Lyris the Kitori•10/18/23, 7:04 PM

i would add this to build.ymlbuild.yml right?

Perksey

P

Perksey•10/18/23, 7:05 PM

yup

Curin

C

Curin•10/18/23, 8:04 PM

PR up for the pointer generation, took a little longer than expected. I forgot about adding the extension methods, but it is all in now.

Curin

C

Curin•10/18/23, 11:50 PM

@Perksey 2 questions regarding your comments:
For the single dimensional generics being generated with IL-Weaving. Initially I assumed IL weaving would be necessary, but if we store ref T instead of ref byte it should be easy enough. Is that how you would prefer it?

As for the public InteriorRef constuctor, there isn't any need for it to be public. Would you prefer it to be private?

Curin

C

Curin•10/19/23, 12:25 AM

Also several MutTests break when we remove string conversion for mut types, not a big deal but those tests are no longer valid.

Curin

CCurin @Perksey 2 questions regarding your comments: For the single dimensional generic...

Perksey

P

Perksey•10/19/23, 12:37 AM

yeah storing the ref in its "true form" is best where possible

Perksey

P

Perksey•10/19/23, 12:37 AM

technically even when storing it as a byte you could just use Unsafe.As, but it's far easier to just use the true ref

Curin

C

Curin•10/19/23, 12:38 AM

great, I've already started work on changing that and fixing up some more fallout I found after the other changes.

Perksey

P

Perksey•10/19/23, 4:09 AM

sorry i haven't been working on silk much, been feeling crappy today

Lyris the Kitori

L

Lyris the Kitori•10/19/23, 4:11 AM

tis all good, take all the time you need

Lyris the Kitori

L

Lyris the Kitori•10/19/23, 4:11 AM

ive been busy with a bad case of "writing unit tests" all day

dfkeenan🍍🍕

D

dfkeenan🍍🍕•10/19/23, 5:20 AM

Not unit tests?!

dfkeenan🍍🍕

Ddfkeenan🍍🍕Not unit tests?! 😱

Lyris the Kitori

L

Lyris the Kitori•10/19/23, 5:21 AM

unit tests.

Perksey

P

Perksey•10/19/23, 5:36 PM

actually before we make the decision on SilkX should we acutally just ship SilkX as SilkX and encourage people to try it out?

Perksey

P

Perksey•10/19/23, 5:37 PM

that way we get some insights into how it actually feels

Maxine

M

Maxine•10/19/23, 5:37 PM

sure

Maxine

M

Maxine•10/19/23, 5:37 PM

please just give me something to use

Maxine

M

Maxine•10/19/23, 5:37 PM

/lh

khitiara

K

khitiaraOP•10/19/23, 5:38 PM

honestly yes

khitiara

K

khitiaraOP•10/19/23, 5:38 PM

also idk what /lh means lmao

Maxine

M

Maxine•10/19/23, 5:38 PM

lighthearted

khitiara

K

khitiaraOP•10/19/23, 5:38 PM

but yeah having it as a preview thing people can try is good

khitiara

K

khitiaraOP•10/19/23, 5:38 PM

ah ok

Perksey

P

Perksey•10/19/23, 6:12 PM

the thing is with super bleeding edge previews is noone really uses them, but as long as some people try them out we'll have some backing to some of the things we want to present to the WG

Perksey

P

Perksey•10/19/23, 6:13 PM

the bleeding edge preview will likely only have GLFW and OpenGL though

Perksey

PPerksey actually before we make the decision on SilkX should we acutally just ship SilkX...

Aqua

A

Aqua•10/19/23, 6:19 PM

i think this is a decent idea. as you know i am currently heavily opposed to the ptr structs idea however its hard to say concretely until you actually try it out, so some sort of preview could be quite beneficial

Perksey

P

Perksey•10/19/23, 6:19 PM

yeah exactly

Perksey

P

Perksey•10/19/23, 6:19 PM

a lot of people have principle-based arguments against what we're doing, and i don't really care about them so long as it's actually enjoyable to use in the field

Aqua

A

Aqua•10/19/23, 6:21 PM

yep

Aqua

A

Aqua•10/19/23, 6:22 PM

its the development experience that matters
... as well as everything else like speed and whatnot but ultimately it doesn't matter how fast something is if no one likes developing with it then no one will use it

Aqua

A

Aqua•10/19/23, 6:23 PM

i'm trying to think objectively about this, subjectively my opinions are somewhat different, but the opportunity to actually use something rather than just see it written will offer an opportunity to those both for and against to see if it will actually work out or not

Perksey

P

Perksey•10/19/23, 10:19 PM

okay so maybe something like OpenTK PAL2 isn't the worst idea. again I wouldn't go for their implementation, and I still intend to go with the libsilklibsilk idea, but I'm just thinking about how I want the user experience to be and it feels like it does indeed necessitate something in Silk.NET.Core that looks similar to PAL2.

e.g. consider:

// If a "root surface" has already been created, we will use Surface.Current internally to create a "subsurface"
ISurface surface = Surface.CreateNew() // returns ISurfaceBuilder
    // requests in order of preference
    // equivalent to .RequestSingleton<T>(...) where T is a IComponentBuilder containing a HluComponentType that we use to discriminate (e.g. HluComponentType.Graphics)
    .RequestGraphics<IVkSurfaceBuilder>(
        ctx => ctx.WithVersion(1, 3)
            .WithSupportedInstanceExtensions<Silk.NET.Vulkan.ExtDebugUtils>() // in addition to the ones required to create the surface
            .Build()
    )
    .RequestGraphics<IGLContextBuilder>(
        ctx => ctx.WithVersion(4, 3)
            .WithProfile(OpenGLProfile.Core)
            .Build()
    )
    .Build();
// equivalent to .Singleton(HluComponentType.Graphics)
surface.Graphics() switch
{
    IGLContext ctx => Console.WriteLine("Using OpenGL"),
    IVkSurface surf => Console.WriteLine("Using Vulkan"),
    _ => throw new("no requests were able to be fulfilled")
};

// If a "root surface" has already been created, we will use Surface.Current internally to create a "subsurface"
ISurface surface = Surface.CreateNew() // returns ISurfaceBuilder
    // requests in order of preference
    // equivalent to .RequestSingleton<T>(...) where T is a IComponentBuilder containing a HluComponentType that we use to discriminate (e.g. HluComponentType.Graphics)
    .RequestGraphics<IVkSurfaceBuilder>(
        ctx => ctx.WithVersion(1, 3)
            .WithSupportedInstanceExtensions<Silk.NET.Vulkan.ExtDebugUtils>() // in addition to the ones required to create the surface
            .Build()
    )
    .RequestGraphics<IGLContextBuilder>(
        ctx => ctx.WithVersion(4, 3)
            .WithProfile(OpenGLProfile.Core)
            .Build()
    )
    .Build();
// equivalent to .Singleton(HluComponentType.Graphics)
surface.Graphics() switch
{
    IGLContext ctx => Console.WriteLine("Using OpenGL"),
    IVkSurface surf => Console.WriteLine("Using Vulkan"),
    _ => throw new("no requests were able to be fulfilled")
};

Perksey

P

Perksey•10/19/23, 10:20 PM

btw this would all look god tier with MS Hosting (cc @kaijellinghaus)

khitiara

K

khitiaraOP•10/19/23, 10:34 PM

oooo making things that work good with ms hosting

khitiara

K

khitiaraOP•10/19/23, 10:34 PM

I use it way too much on every project so I'm interested in that

dfkeenan🍍🍕

D

dfkeenan🍍🍕•10/20/23, 12:46 AM

If the bleeding edge has webgpu might with it.

Still no idea why I am playing with webgpu.

Lyris the Kitori

L

Lyris the Kitori•10/20/23, 12:47 AM

webgpu is great

// If a "root surface" has already been created, we will use Surface.Current internally to create a "subsurface"
ISurface surface = Surface.CreateNew() // returns ISurfaceBuilder
    // requests in order of preference
    // equivalent to .RequestSingleton<T>(...) where T is a IComponentBuilder containing a HluComponentType that we use to discriminate (e.g. HluComponentType.Graphics)
    .RequestGraphics<IVkSurfaceBuilder>(
        ctx => ctx.WithVersion(1, 3)
            .WithSupportedInstanceExtensions<Silk.NET.Vulkan.ExtDebugUtils>() // in addition to the ones required to create the surface
            .Build()
    )
    .RequestGraphics<IGLContextBuilder>(
        ctx => ctx.WithVersion(4, 3)
            .WithProfile(OpenGLProfile.Core)
            .Build()
    )
    .Build();
// equivalent to .Singleton(HluComponentType.Graphics)
surface.Graphics() switch
{
    IGLContext ctx => Console.WriteLine("Using OpenGL"),
    IVkSurface surf => Console.WriteLine("Using Vulkan"),
    _ => throw new("no requests were able to be fulfilled")
};

// If a "root surface" has already been created, we will use Surface.Current internally to create a "subsurface"
ISurface surface = Surface.CreateNew() // returns ISurfaceBuilder
    // requests in order of preference
    // equivalent to .RequestSingleton<T>(...) where T is a IComponentBuilder containing a HluComponentType that we use to discriminate (e.g. HluComponentType.Graphics)
    .RequestGraphics<IVkSurfaceBuilder>(
        ctx => ctx.WithVersion(1, 3)
            .WithSupportedInstanceExtensions<Silk.NET.Vulkan.ExtDebugUtils>() // in addition to the ones required to create the surface
            .Build()
    )
    .RequestGraphics<IGLContextBuilder>(
        ctx => ctx.WithVersion(4, 3)
            .WithProfile(OpenGLProfile.Core)
            .Build()
    )
    .Build();
// equivalent to .Singleton(HluComponentType.Graphics)
surface.Graphics() switch
{
    IGLContext ctx => Console.WriteLine("Using OpenGL"),
    IVkSurface surf => Console.WriteLine("Using Vulkan"),
    _ => throw new("no requests were able to be fulfilled")
};