gpython icon indicating copy to clipboard operation
gpython copied to clipboard

Do you need type conversion of struct?

Open wetor opened this issue 1 year ago • 6 comments

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

wetor avatar Mar 13 '23 15:03 wetor

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

ncw avatar Mar 14 '23 10:03 ncw

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.

wetor avatar Mar 14 '23 11:03 wetor

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.

:-)

ncw avatar Mar 15 '23 16:03 ncw

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

wetor avatar Mar 16 '23 09:03 wetor

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.

ncw avatar Mar 16 '23 11:03 ncw

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.

wetor avatar Mar 18 '23 06:03 wetor