go-python/gpython

Do you need type conversion of struct?

wetor opened this issue · 6 comments

wetor commented

When I use gpython as a plugin parser, I use reflect and recurrence to achieve a more general type conversion because of the trouble of type conversion.

Support mutual conversion of go type and gpython type including struct and map

Of course, the performance is not very good

https://github.com/wetor/AnimeGo/blob/feature-update_plugin/pkg/plugin/utils.go
and https://github.com/wetor/AnimeGo/blob/feature-update_plugin/pkg/utils/utils.go line 65, 76

ncw commented

How do you think gpython should be helping here? Can you write an example? Thank you.

wetor commented

Based on my recent usage experience, I believe that gpython's embedded use lacks a convenient "bridge" for exchanging data with Go, such as the functionality in goja (https://github.com/dop251/goja) that exports types and methods from the VM as native Go types and functions. Currently, I do not have a specific plan for implementing this "bridge". However, I was wondering if you have any plans to develop gpython in the direction of an embedded script interpreter?

Furthermore, I noticed this issue because I found basic type conversion code in py/util.go, and I thought I could improve it. I will provide examples later.

Also, thank you for your contributions to this project. It's a great project.

ncw commented

Based on my recent usage experience, I believe that gpython's embedded use lacks a convenient "bridge" for exchanging data with Go, such as the functionality in goja (https://github.com/dop251/goja) that exports types and methods from the VM as native Go types and functions.

I see what you mean.

Currently, I do not have a specific plan for implementing this "bridge". However, I was wondering if you have any plans to develop gpython in the direction of an embedded script interpreter?

gpython is used like that so it seems like a good idea to make it easier for the user.

Furthermore, I noticed this issue because I found basic type conversion code in py/util.go, and I thought I could improve it. I will provide examples later.

Great. Yes that code could certainly be expanded.

Also, thank you for your contributions to this project. It's a great project.

:-)

wetor commented

example here https://go.dev/play/p/aT2zebez5zC?v=goprev

Some uint types and pointer types are not supported. It's easy to add them, you know.
Currently, all complex types are basically supported (there may be bugs)

If you think it is useful, I will add some test cases and submit a push request

ncw commented

Nice code!

This converts Go structs into python dicts.

Do you think converting them into a python object would be more pythonic? So

Perhaps if we made a base class GoStruct or something like that it would make the process easier.

So

struct {
 X int
 Y string
}

Would become an instance of GoStruct with members X and Y as if it had been defined something like

a = GoStruct()
a.X = 1
a.Y = "hello"

Potentially this would allow python to call the methods on the struct too.

wetor commented

I also think it's better to convert to an object type. I just used it for data transfer, so I converted it to a dictionary. Perhaps we do need a similar 'GoStruct' type.