Skip to content

Conversation

@alejandrodnm
Copy link
Collaborator

No description provided.

@alejandrodnm alejandrodnm temporarily deployed to internal-contributors June 4, 2025 13:56 — with GitHub Actions Inactive
@alejandrodnm alejandrodnm temporarily deployed to internal-contributors June 5, 2025 14:49 — with GitHub Actions Inactive
@alejandrodnm alejandrodnm changed the title Adn/pydoc md chore: add generated docs for python API Jun 5, 2025
@alejandrodnm alejandrodnm temporarily deployed to internal-contributors June 5, 2025 15:04 — with GitHub Actions Inactive
UV_PROJECT_ENVIRONMENT=/usr/local/
WORKDIR /build
COPY --from=ghcr.io/astral-sh/uv:0.5.20 /uv /uvx /bin/
COPY --from=ghcr.io/astral-sh/uv:0.7.10 /uv /uvx /bin/
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@JamesGuthrie should I change this as well?

Copy link
Member

Choose a reason for hiding this comment

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

Yes!

@alejandrodnm alejandrodnm marked this pull request as ready for review June 5, 2025 15:06
@alejandrodnm alejandrodnm requested a review from a team as a code owner June 5, 2025 15:06
Copy link
Member

@JamesGuthrie JamesGuthrie left a comment

Choose a reason for hiding this comment

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

TBH the vectorizer api documentation is not useful. Especially when compared with what we have in https://github.com/timescale/pgai/blob/main/docs/vectorizer/api-reference.md.

IMHO we would need to tweak it quite a bit in order to provide the same level of experience as what we have for the sql API.

Add the command `just pgai generate-docs` to update the docs.

The `just install-commit-hook` will also install a hook that updates the
docs automatically, if any `projects/pgai/pgai/*.python` file changes.
@alejandrodnm alejandrodnm temporarily deployed to internal-contributors June 6, 2025 09:20 — with GitHub Actions Inactive
@alejandrodnm alejandrodnm requested a review from jgpruitt June 6, 2025 10:48
@alejandrodnm alejandrodnm enabled auto-merge (rebase) June 6, 2025 10:48
@jgpruitt jgpruitt removed their request for review July 24, 2025 14:00
@alejandrodnm alejandrodnm force-pushed the main branch 2 times, most recently from 35e80a5 to f15011e Compare October 14, 2025 14:27
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