Pipeline Response Changes #282
Conversation
| fields.extend([f"{k}={repr(v)}" for k, v in self.additional_fields.items()]) | ||
| return f"PipelineResponse({', '.join(fields)})" | ||
|
|
||
| def __contains__(self, key: str) -> bool: |
There was a problem hiding this comment.
above my comment regarding making this dict like object also applies here.
| from aixplain.enums.asset_status import AssetStatus | ||
|
|
||
|
|
||
| # def test_list_pipelines(): |
There was a problem hiding this comment.
All these tests are disabled somehow. Is that by purpose?
There was a problem hiding this comment.
The data is not being returned properly, so I commented some out to debug. Any idea why I could be getting success status, while no data is being returned?
There was a problem hiding this comment.
I have really no idea on that.
| start = time.time() | ||
| try: | ||
| response = self.run_async(data, data_asset=data_asset, name=name, **kwargs) | ||
| if response["status"] == "FAILED": | ||
| if response["status"] == Status.FAILED: |
There was a problem hiding this comment.
Here, are we sure the value of "status" in the matching enum type or a string only?
There was a problem hiding this comment.
The enum makes sure that if it's passed in as string it turns it into enum object
| end = time.time() | ||
| response = self.__polling(poll_url, name=name, timeout=timeout, wait_time=wait_time) | ||
| status = Status(polling_response["status"]) |
There was a problem hiding this comment.
Regarding above my comment, for example here the value of "status" not seems to a be enum. Best to make sure of it.
There was a problem hiding this comment.
Okay I'll check again
No description provided.