-
Notifications
You must be signed in to change notification settings - Fork 550
Initial commit of C# port - does not build #184
base: develop
Are you sure you want to change the base?
Conversation
This is awesome! However I feel like it's better to put this in a separate repo :-? |
You might be right, but I hope to persuade people otherwise:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comments for atleast the public properties would be awesome.
I can't figure out what the most of them are for by just looking at the code.
Don't worry - I can't either. :-) I'm running the Python project in one Visual Studio debugger and am porting statements to C# in another as I encounter them. I haven't actually confirmed TensorFlow.net can successfully load the GPT-2 model into memory yet. Once that is done, I'll rework this into something more product-shaped than proof-of-concept-shaped. Multiple people on Discord were asking to see what I had so far, so that's why this is here (despite its current sorry state). |
Ahh didn't know that xD |
insane |
If need be, you could just run the code in Iron Python, which is a .NET python interpreter. |
What is the overall goal of porting to C#? |
Anyone interested in writing a C# port might find this useful: https://github.com/losttech/Gradient-Samples/tree/master/GPT-2 (Gradient is Tensorflow's .NET binding) |
🎉 Issues resolved:
🧪 How to Test