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

move extract and observe to page #329

Merged
merged 6 commits into from
Dec 23, 2024
Merged

move extract and observe to page #329

merged 6 commits into from
Dec 23, 2024

Conversation

kamath
Copy link
Contributor

@kamath kamath commented Dec 23, 2024

why

Follows the pattern of #326 to move extract and observe to the Page object

what changed

  • stagehand.extract and stagehand.observe are now deprecated in favor of stagehand.page.extract and stagehand.page.observe
  • readme

test plan

evals. need a fast-follow PR like #328 for evals moving to page

Copy link

changeset-bot bot commented Dec 23, 2024

🦋 Changeset detected

Latest commit: 2dac663

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@browserbasehq/stagehand Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kamath kamath changed the base branch from main to v2 December 23, 2024 01:23
@kamath kamath changed the base branch from v2 to anirudh/add-act-evals December 23, 2024 01:24
@kamath kamath requested a review from seanmcguire12 December 23, 2024 01:40
@kamath kamath marked this pull request as ready for review December 23, 2024 01:43
Base automatically changed from anirudh/add-act-evals to v2 December 23, 2024 02:41
@kamath kamath merged commit 7b22b83 into v2 Dec 23, 2024
@kamath kamath deleted the anirudh/move-extract-to-page branch December 23, 2024 02:41
kamath added a commit that referenced this pull request Dec 24, 2024
* Use CI on v2 branch

* branch

* add docs, move scoring functions to scoring.ts, move experiment naming to utils.ts

* add initStagehand.ts

* break up index.evals.ts and utils into smaller files

* export LogLineEval

* typing

* follow StagehandConfig pattern

* choose api key based on model name

* stagehand.act -> page.act (#326)

* need to actually move to act to page now

* move act -> page

* fix e2e

* fix tests

* readme

* changeset

* package json and changeset

* don't fail on combo evals

* Add act evals on `stagehand.page` (#328)

* move act evals to stagehand.page

* add basic act and make act necessary in type

* move extract and observe to page (#329)

* move act evals to stagehand.page

* add basic act and make act necessary in type

* move extract and observe

* example

* changeset

* More playwright tests (#330)

* add docs, move scoring functions to scoring.ts, move experiment naming to utils.ts

* add initStagehand.ts

* break up index.evals.ts and utils into smaller files

* export LogLineEval

* typing

* follow StagehandConfig pattern

* choose api key based on model name

* Use CI on v2 branch

* branch

* stagehand.page tests

* dont run on BB

* prettier

* pls dont fail

* headless

---------

Co-authored-by: Anirudh Kamath <[email protected]>

* add extract evals for stagehand.page (#331)

* add extract evals for stagehand.page

* fix typign

* smh i didn't actually run extract

* add observe page evals (#332)

* change stagehand.observe to stagehand.page.observe in evals

* changeset

* Browsercontext playwright tests (#334)

* add docs, move scoring functions to scoring.ts, move experiment naming to utils.ts

* add initStagehand.ts

* break up index.evals.ts and utils into smaller files

* export LogLineEval

* typing

* follow StagehandConfig pattern

* choose api key based on model name

* Use CI on v2 branch

* branch

* BrowserContext tests

* file path

---------

Co-authored-by: Anirudh Kamath <[email protected]>

* changeset minor

* ci yml

---------

Co-authored-by: seanmcguire12 <[email protected]>
Co-authored-by: Sean McGuire <[email protected]>
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