RobotecAI / o3de-genai-gems

Core features and interfaces enabling use of modern AI with O3DE for variety of purposes.
Other
1 stars 0 forks source link

Add registered name getter for async python requests #97

Closed arturkamieniecki closed 3 months ago

arturkamieniecki commented 3 months ago

I've added a getter for the registered name of the model configuration component currently used in the async python requests. Example in readme. Resolves #95.

arturkamieniecki commented 3 months ago

I've stumbled upon 2 issues:

1. Model configuration seems to not be active until either selection of model config is made in layouting assistant via combo box or the prompt is sent to the model.

2. After removing the model configuration asking for active configuration results in segfault. Also after readding the config there is a segfault.

I couldn't manage to recreate the segfault. Could you provide me with instruction steps that lead to the crash?

pawel-kotowski commented 3 months ago

I've stumbled upon 2 issues:

1. Model configuration seems to not be active until either selection of model config is made in layouting assistant via combo box or the prompt is sent to the model.

2. After removing the model configuration asking for active configuration results in segfault. Also after readding the config there is a segfault.

I couldn't manage to recreate the segfault. Could you provide me with instruction steps that lead to the crash?

I create the configuration and provider, select the configuration in layouting assistant (then it's active) and the configuration is returned correctly. The I just remove the configuration and try to get it again which results in segfault.

arturkamieniecki commented 3 months ago

The segfault has been fixed

arturkamieniecki commented 3 months ago

The first issue is fixed in https://github.com/RobotecAI/ai-scene-generation-gem/pull/70.

arturkamieniecki commented 3 months ago

I've implemented the second half of the interface.