Skip to content

Conversation

@ea-rus
Copy link
Collaborator

@ea-rus ea-rus commented Sep 19, 2024

Fixed mind.add_datasource

  • it creates datasource if not exist and the passed only name to rest api

fix: https://linear.app/mindsdb/issue/BE-169/add-datasource-to-mind

@linear
Copy link

linear bot commented Sep 19, 2024

BE-169 Add datasource to Mind


from minds.client import Client
from minds.datasources import DatabaseConfig

client = Client("token", "https://mdb.ai/")

postgres_config2 = DatabaseConfig(
    name='my_datasource1',
    description='Customers Telecom Churn Data',
    engine='postgres',
    connection_data={
      'user': 'demo_user',
      'password': 'demo_password',
      'host': 'samples.mindsdb.com',
      'port': 5432,
      'database': 'demo',
      'schema': 'demo_data'
  },
    tables=['customer_churn']
)

mind.add_datasource(postgres_config2)

print(mind.datasources[0].name)

Error:

Traceback (most recent call last):
  File "/home/zoran/MyProjects/env/minds/mindz.py", line 50, in <module>
    mind.add_datasource(postgres_config2)
  File "/home/zoran/MyProjects/env/minds/vminds/lib/python3.10/site-packages/minds/minds.py", line 59, in add_datasource
    self.api.post(
  File "/home/zoran/MyProjects/env/minds/vminds/lib/python3.10/site-packages/minds/rest_api.py", line 43, in post
    _raise_for_status(resp)
  File "/home/zoran/MyProjects/env/minds/vminds/lib/python3.10/site-packages/minds/rest_api.py", line 7, in _raise_for_status
    raise requests.HTTPError(f'{response.reason}: {response.text}', response=response)
requests.exceptions.HTTPError: Bad Request: {"title": "Something wrong", "detail": "Something wrong", "timestamp": "2024-09-16 15:25:20.737636"}

@ZoranPandovski ZoranPandovski merged commit e3fa8c7 into main Sep 25, 2024
@ea-rus ea-rus deleted the exceptions branch October 1, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants