AMICI expects rdata not dict about pypesto HOT 5 CLOSED

Yes, will have to add more versatile creation of RData instances for other purposes anyways.

from pypesto.

I don't think there are any simple conversion functions as a ReturnDataView is a relatively complex class with caching ane attributes that are C pointers. Wouldn't it be sufficient to create a simple dataclass (instead of the dict) that mocks ReturnDataView attributes?

from pypesto.

I don't think there are any simple conversion functions as a ReturnDataView is a relatively complex class with caching ane attributes that are C pointers. Wouldn't it be sufficient to create a simple dataclass (instead of the dict) that mocks ReturnDataView attributes?

Yes that is essentially what was done before. I will do a simple AttributeDict then. Works but won't be very nice, as the expected data type is ignored.

from pypesto.

I don't think there are any simple conversion functions as a ReturnDataView is a relatively complex class with caching ane attributes that are C pointers. Wouldn't it be sufficient to create a simple dataclass (instead of the dict) that mocks ReturnDataView attributes?

Yes that is essentially what was done before. I will do a simple AttributeDict then. Works but won't be very nice, as the expected data type is ignored.

Agreed. But it also shouldn't be too difficult though to simply insert those columns into a copy of the measurement_df, right?

from pypesto.

I don't think there are any simple conversion functions as a ReturnDataView is a relatively complex class with caching ane attributes that are C pointers. Wouldn't it be sufficient to create a simple dataclass (instead of the dict) that mocks ReturnDataView attributes?

Yes that is essentially what was done before. I will do a simple AttributeDict then. Works but won't be very nice, as the expected data type is ignored.

Agreed. But it also shouldn't be too difficult though to simply insert those columns into a copy of the measurement_df, right?

Should be possible, but implementing that functionality here could lead to issues, as the logic has been moved to AMICI. Long-term, either pyPESTO should keep/construct actual AMICI RData, or AMICI's function be made more generic. But fine for the moment I think.

from pypesto.