microsoft / knossos-ksc

Compiler with automatic differentiation
Other
45 stars 10 forks source link

Remove procedure language #1061

Open tomjaguarpaw opened 2 years ago

tomjaguarpaw commented 2 years ago

Shall we get rid of the procedure language stuff? It's irrelevant given that we have "BOG-style AD for ksc" (which is better).

ghost commented 2 years ago

CLA assistant check
All CLA requirements met.

tomjaguarpaw commented 2 years ago

cc @awf

awf commented 2 years ago

If you get a chance, could you merge with master, and then try again?

Thanks!

tomjaguarpaw commented 2 years ago

Sure. Could you please add this account as a collaborator?

awf commented 2 years ago

Thanks, and done.

awf commented 2 years ago

Let's retry when #1064 is merged. The new scenario here is PR from a fork, which we should support, so using this as a test case.

Thanks for your patience!