Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: fix variable name typo in run experiments doc #4249

Merged
merged 2 commits into from
Aug 15, 2024

Conversation

chrishart0
Copy link
Contributor

The ds variable is never defined. This should have been df which is defined on line 32.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 15, 2024
Copy link
Contributor

github-actions bot commented Aug 15, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@chrishart0
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@chrishart0
Copy link
Contributor Author

recheck

github-actions bot added a commit that referenced this pull request Aug 15, 2024
@mikeldking mikeldking changed the base branch from main to docs August 15, 2024 18:40
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Aug 15, 2024
@mikeldking mikeldking changed the base branch from docs to main August 15, 2024 18:41
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:XL This PR changes 500-999 lines, ignoring generated files. labels Aug 15, 2024
@mikeldking mikeldking changed the title FIx variable name typo in run experiments doc docs: fix variable name typo in run experiments doc Aug 15, 2024
@mikeldking
Copy link
Contributor

Thanks for the catch @chrishart0 !!

@mikeldking mikeldking merged commit 9745754 into Arize-ai:main Aug 15, 2024
2 checks passed
fjcasti1 added a commit that referenced this pull request Aug 16, 2024
* main: (58 commits)
  Update OpenAPI spec (#4260)
  Default to adding span annotations to queue (#4259)
  docs: Add human feedback notebook tutorial (#4257)
  refactor: daemon task abstraction (#4255)
  feat(auth): create system key (#4235)
  chore: add vision fixture (#4250)
  docs: fix variable name typo in run experiments doc (#4249)
  build(deps): bump certifi (#4147)
  chore(main): release arize-phoenix 4.24.0 (#4225)
  chore(main): release arize-phoenix-evals 0.15.0 (#3920)
  feat(experiments): ability to specify concurrency in run_experiment and evaluate_experiment (#4189)
  ci(evals): limit `mistralai` below version 1.0 (#4241)
  feat(auth): user / system api key resolvers (#4232)
  feat(auth): add user role, exclude system in user lists (#4229)
  docs: Add multimodal image reasoning tutorial with llama index (#4210)
  chore(main): release arize-phoenix 4.23.0 (#4216)
  docs: No subject (GITBOOK-818)
  build(deps): bump streamlit (#4214)
  fix: Propagate span annotation metadata to examples on all mutations (#4195)
  feat(auth): users table in settings (#4221)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants