[Draft] Support for multiple return values from go function to python#282

Draft

nishantsharma wants to merge 21 commits into

go-python:masterfrom

rudderlabs:nisshar/MultipleReturns

Commits on Jun 19, 2022

Merge pull request #1 from rudderlabs/nisshar/VariadicFunctions
nishantsharma
authored

Commits on Jul 4, 2022

Go module path updated so that it can be installed.
nishant-rudderstack
committed
Replacing gopy URL so that it works in forks too.
nishant-rudderstack
committed
go package to test is at rudderlabs/gopy
nishant-rudderstack
committed
Added test code for multireturn.
nishant-rudderstack
committed
First stage changes made. Next step is to change the loop to reflect original design.
nishant-rudderstack
committed
Python code being generated sensibly.
nishant-rudderstack
committed
Correcting npyres computation.
nishant-rudderstack
committed
Verfied generated python code, atleast for smaller ret-counts.
nishant-rudderstack
committed
Code to handle multiple return values as tuples now logically complete, even though it doesn't work currently.
nishant-rudderstack
committed
Both go and python code is now compiling. Automatic Test are failing.
nishant-rudderstack
committed
Working quite a bit as expected. But, some memory corruption is causing un-expected memory freeing and crashes.
nishant-rudderstack
committed
Tests passing once we let memory leak. Fix for memory leaks to be carefully enabled.
nishant-rudderstack
committed
Ignoring functions returning functions.
nishant-rudderstack
committed
__err was getting reused in a way which declarations difficult. Fixed.
nishant-rudderstack
committed
Fixed a few code generation error for methods.
nishant-rudderstack
committed
Python None now mapped to GoLang nil.
nishant-rudderstack
committed
TestGovet and TestGofmt now passing.
nishant-rudderstack
committed
Crash in gen command fixed.
nishant-rudderstack
committed
gopyerrors is not applicable any more.
nishant-rudderstack
committed
Test failure bug fixed.
nishant-rudderstack
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Draft] Support for multiple return values from go function to python#282

[Draft] Support for multiple return values from go function to python#282
nishantsharma wants to merge 21 commits into
go-python:masterfrom
rudderlabs:nisshar/MultipleReturns

Commits on Jun 19, 2022

Merge pull request #1 from rudderlabs/nisshar/VariadicFunctions

Commits on Jul 4, 2022

Go module path updated so that it can be installed.

Replacing gopy URL so that it works in forks too.

go package to test is at rudderlabs/gopy

Added test code for multireturn.

First stage changes made. Next step is to change the loop to reflect original design.

Python code being generated sensibly.

Correcting npyres computation.

Verfied generated python code, atleast for smaller ret-counts.

Code to handle multiple return values as tuples now logically complete, even though it doesn't work currently.

Both go and python code is now compiling. Automatic Test are failing.

Working quite a bit as expected. But, some memory corruption is causing un-expected memory freeing and crashes.

Tests passing once we let memory leak. Fix for memory leaks to be carefully enabled.

Ignoring functions returning functions.

__err was getting reused in a way which declarations difficult. Fixed.

Fixed a few code generation error for methods.

Python None now mapped to GoLang nil.

TestGovet and TestGofmt now passing.

Crash in gen command fixed.

gopyerrors is not applicable any more.

Test failure bug fixed.