Merged
Conversation
…puts build readme md from Rmd
add Jimbo's batch API quarto to dev_docs
update tests export ant_build_messages_request add schema tests for ant_build_messages_request
… this to vignette too) add more tests to ant_build_messages_request
bump version update news and _pkgdown.yml
add the .extract_api_error function
push planning doc for refactor
and refactor chunk function to avoid confusing batch_size naming
fix small bug in the calling of .handle_output_filename inside hf_embed_chunks/df update todos/news update _pkgdown.yml with hf_embed_chunks
pass output_file to hf_embed_chunks from inside hf_embed_df to fix the filenmae issue
…r debugging people's code/errors (including my own)
…fy and later hf_embed
add max_length to hf_embed_df and hf_embed_df
move hf_classify_df over to hf_classify_chunks not hf_classify_batch remove old comments from hf_embed_df
… is to turn on 'AUTO_TRUNCATE' in the Set up of the endpoint
Build README Update NEWS.md
…s to the hf_*_df functions.
build rd files add comma in embed test add test/dev docs to .Rbuildignore
…to files with the origianl variable names.
switch to output_dir = for oai_complete_chunks & oai_complete_df add id_col_name to oai_complete_chunks fix docs
output_dir arg switch to chunk vocab
add oai_embed_chunks to _pkgdown.yml update README and build
…s well as the .error and .error_msg, for easier filtering Can't simply put into .error because we can't have numeric with FALSE (when no .error) due to single-typed vecs in R
Merge remote-tracking branch 'origin/dev' into feature/anthropic_api # Conflicts: # DESCRIPTION # NEWS.md # R/hf_classify.R # R/hf_embed.R # R/openai_completions.R # README.Rmd # README.md # _pkgdown.yml # man/oai_complete_chunks.Rd # man/oai_complete_df.Rd # tests/testthat/test-hf_classify.R # tests/testthat/test-hf_embed.R # todos.qmd
…actor across oai_* funcs and hf_* funcs with the changes to basee_request (which ant doesn't use due to auth differences)
…ncs which check the httr2 request was successful/not
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fairly hefty re-factor with a big rebase for 3 braches/features rolled together