Skip to content

PGVector: add some changes to readme#45636

Merged
Aldo Gonzalez (aldogonzalez8) merged 4 commits into
masterfrom
aldogonzalez8/destination/pgvector/some-changes-for-readme
Sep 23, 2024
Merged

PGVector: add some changes to readme#45636
Aldo Gonzalez (aldogonzalez8) merged 4 commits into
masterfrom
aldogonzalez8/destination/pgvector/some-changes-for-readme

Conversation

@aldogonzalez8
Copy link
Copy Markdown
Contributor

What

Some changes to readme related to embedding models and enabling extension.

How

Update docs

Review guide

User Impact

Better docs to read

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 23, 2024 5:43pm

@aldogonzalez8
Copy link
Copy Markdown
Contributor Author

Aldo Gonzalez (aldogonzalez8) commented Sep 23, 2024

/approve-regression-tests

Check job output.

✅ Approving regression tests

@aldogonzalez8
Copy link
Copy Markdown
Contributor Author

Aldo Gonzalez (aldogonzalez8) commented Sep 23, 2024

/approve-regression-tests

Check job output.

✅ Approving regression tests

)
default_schema: str = Field(
...,
default="public",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In Pydantic, ... is used to indicate that a field is required, while default is used to provide a default value.

These two are mutually exclusive. If a field is required (...), you cannot also provide a default value, as it creates a contradiction: the field cannot be both required and have a default.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just commenting if somebody is curious as I was getting TypeError: Field() got multiple values for keyword argument 'default'

@aldogonzalez8 Aldo Gonzalez (aldogonzalez8) deleted the aldogonzalez8/destination/pgvector/some-changes-for-readme branch September 23, 2024 19:57
Mike Saffitz (msaffitz) added a commit to Encore-Post-Sales/airbyte-magnify that referenced this pull request May 29, 2025
* commit '59e981a72915dc57461250c62b60d3a8a04b5b0b': (6181 commits)
  docs(source-intercom): Update outdated URLs in docs (airbytehq#45858)
  🤖 Cut version 5.7.1 of source-declarative-manifest
  🤖 patch bump Python CDK to version 5.7.1
  fix(airbyte-cdk): add snappy to file-based deps (airbytehq#45864)
  🐛 Source HubSpot: fix infinite loop when iterating through search results (airbytehq#44899)
  Destination Iceberg: cleanup cloud spec to only support S3 & Glue (airbytehq#45861)
  Load CDK Clean up Tmp File; SpillToDisk/ProcessRecords Unit Tests (airbytehq#45841)
  docs: Release notes for 1.0 (airbytehq#45693)
  Load CDK TaskLauncher Refactor + Unit Tests (airbytehq#45350)
  PGVector: add some changes to readme (airbytehq#45636)
  docs: Update homepage and product summary (airbytehq#45689)
  docs: What are credits? (airbytehq#45713)
  chore(source-zendesk-support): Update `ticket_activities` record (airbytehq#45849)
  chore: add docs for ai assist (airbytehq#45845)
  🐙 source-typeform: run up-to-date pipeline [2024-09-21] (airbytehq#45495)
  🐙 source-bing-ads: run up-to-date pipeline [2024-09-21] (airbytehq#45512)
  🐙 source-retently: run up-to-date pipeline [2024-09-21] (airbytehq#45513)
  🐙 destination-chroma: run up-to-date pipeline [2024-09-21] (airbytehq#45553)
  🐙 source-smartsheets: run up-to-date pipeline [2024-09-21] (airbytehq#45722)
  🐙 source-gridly: run up-to-date pipeline [2024-09-21] (airbytehq#45723)
  ...
Mike Saffitz (msaffitz) added a commit to Encore-Post-Sales/airbyte-magnify that referenced this pull request May 30, 2025
* commit '59e981a72915dc57461250c62b60d3a8a04b5b0b': (6181 commits)
  docs(source-intercom): Update outdated URLs in docs (airbytehq#45858)
  🤖 Cut version 5.7.1 of source-declarative-manifest
  🤖 patch bump Python CDK to version 5.7.1
  fix(airbyte-cdk): add snappy to file-based deps (airbytehq#45864)
  🐛 Source HubSpot: fix infinite loop when iterating through search results (airbytehq#44899)
  Destination Iceberg: cleanup cloud spec to only support S3 & Glue (airbytehq#45861)
  Load CDK Clean up Tmp File; SpillToDisk/ProcessRecords Unit Tests (airbytehq#45841)
  docs: Release notes for 1.0 (airbytehq#45693)
  Load CDK TaskLauncher Refactor + Unit Tests (airbytehq#45350)
  PGVector: add some changes to readme (airbytehq#45636)
  docs: Update homepage and product summary (airbytehq#45689)
  docs: What are credits? (airbytehq#45713)
  chore(source-zendesk-support): Update `ticket_activities` record (airbytehq#45849)
  chore: add docs for ai assist (airbytehq#45845)
  🐙 source-typeform: run up-to-date pipeline [2024-09-21] (airbytehq#45495)
  🐙 source-bing-ads: run up-to-date pipeline [2024-09-21] (airbytehq#45512)
  🐙 source-retently: run up-to-date pipeline [2024-09-21] (airbytehq#45513)
  🐙 destination-chroma: run up-to-date pipeline [2024-09-21] (airbytehq#45553)
  🐙 source-smartsheets: run up-to-date pipeline [2024-09-21] (airbytehq#45722)
  🐙 source-gridly: run up-to-date pipeline [2024-09-21] (airbytehq#45723)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/destination/pgvector

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants