Skip to content

Misc changes - #59

Merged
gvanrossum-ms merged 11 commits into
mainfrom
release
Oct 29, 2025
Merged

Misc changes#59
gvanrossum-ms merged 11 commits into
mainfrom
release

Conversation

@gvanrossum-ms

@gvanrossum-ms gvanrossum-ms commented Oct 29, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Latest to earliest commit:

  • cd4e870 Mark two more tests as really_needs_auth
  • 08f928c Fix pyright 1.1.406, since 407 has a regression (1.1.407 breaks the use of pydantic.dataclasses attributes pyright#11060)
  • 10b8b69 Reformat vectorbase.py
  • 969d130 Cleanup deps
  • a2a3ff3 Default auto_extract_knowledge to True
  • 707e8b8 Tweak contents and initialization of TextEmbeddingIndexSettings
    • Remove retry_max_attempts, retry_delay
    • Add max_retries
    • More sensible initializations
    • Pass max_retries to AsyncEmbeddingModel
    • Default batch_size to 10
  • a53e873 ingest_vtt.py: take batch size from settings
  • d157257 Fix some embeddings issues
    • add max_retries arg, passed to openai.AsyncOpenAI
    • Fix typos in model_to_embedding_size_and_envvar keys (missing -3- in names)
  • be77839 Upgrade dependencies, including pyright
  • fcce50f Update TODO list
  • 8999559 Use black.Mode instead of obsolete black.FileMode

gvanrossum-ms and others added 8 commits October 28, 2025 10:15
@gvanrossum-ms
Use black.Mode instead of obsolete black.FileMode
8999559
@gvanrossum-ms
Update TODO list
fcce50f
@gvanrossum @gvanrossum-ms
Upgrade dependencies, including pyright
be77839
@gvanrossum-ms
Fix some embeddings issues
d157257 
- add max_retries arg, passed to openai.AsyncOpenAI
- Fix typos in model_to_embedding_size_and_envvar keys (missing -3- in names)
@gvanrossum-ms
ingest_vtt.py: take batch size from settings
a53e873
@gvanrossum-ms
Tweak contents and initialization of TextEmbeddingIndexSettings
707e8b8 
- Removeretry_max_attempts, retry_delay
- Add max_retries
- More sensible initializations
- Pass max_retries to AsyncEmbeddingModel
- Default batch_size to 10
@gvanrossum-ms
Default auto_extract_knowledge to True
a2a3ff3
@gvanrossum-ms
Cleanup deps
969d130
@gvanrossum-ms
gvanrossum-ms marked this pull request as draft October 29, 2025 00:10
@gvanrossum-ms

gvanrossum-ms commented Oct 29, 2025

Copy link
Copy Markdown
Contributor Author

Whoops, there are type checking errors that I need to fix later.

@gvanrossum
Guido van Rossum (gvanrossum) marked this pull request as ready for review October 29, 2025 02:33
robgruen
robgruen approved these changes Oct 29, 2025
View reviewed changes
Comment thread typeagent/aitools/vectorbase.py
@gvanrossum-ms
gvanrossum-ms merged commit cfc3955 into main Oct 29, 2025
21 checks passed
@gvanrossum-ms
gvanrossum-ms deleted the release branch October 29, 2025 19:59
@gvanrossum Guido van Rossum (gvanrossum) mentioned this pull request Oct 29, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gvanrossum Guido van Rossum (gvanrossum) gvanrossum approved these changes
@robgruen robgruen robgruen approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gvanrossum-ms @gvanrossum @robgruen