João Carlos Bastos Portela

Results 16 comments of João Carlos Bastos Portela

Is there any ambition to get this PR merged? As far as I can see the current master still have the same issue while loading data type definitions. If there...

> Also generally, this send to be a quote ugly hack. Exceptions should only be raised and catched in case of errors. Here you are using exceptions to avoid sorting...

> We can leave that PR open, but I would rather not merge that. It make understanding code even more complicated. > Also I am surprised I never got that...

> https://github.com/FreeOpcUa/opcua-asyncio/blob/a69566c5ae926233e85be3b77c4f5480f148614e/asyncua/ua/uatypes.py#L842 > > should be "if len(dims) >= 1: " i guess Those 2 things pointed out seem indeed to be needed, however not sufficient I think. It solves...

> @jcbastosportela red or blue pill? Without breaking Change: #690 With breaking Change but cleaner: #691 > > feedback welcome! I was just looking the changes in #691. I think...

> but I just made a PR: #508 Very good man! Very fast. I will try it as soon as possible.

I was using 0.9.0, and when I checkout this commit many new things are coming I believe. With this version I get an exception on `load_data_type_definitions()`. Opened a new issue...

So, yesterday after finding the issue I have tried some dirty hacks to get to try this functionality. Basically I have made similar changes (commit 4fb0d26b265b55adaebcc340f9ba366ed09640cc), catching same exceptions, but...

This is all I see when selecting the datatype: ![image](https://user-images.githubusercontent.com/15647115/112625528-596f3700-8e2f-11eb-93d4-c264a3776558.png) NOTE: I can normally get python objects out of OPC UA datatypes from the server. both with `get_ua_class(TYPE_NAME)()` and `ua.TYPE_NAME()`....

> OK. If your server support spec 1.4 Then you can use the load_data-type_definition() (return nothing) and load_custom_struct() (rturn the struct). > IF your server only support the old custom...