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

Validation-set #15

Merged
merged 5 commits into from
Jul 31, 2024
Merged

Validation-set #15

merged 5 commits into from
Jul 31, 2024

Conversation

samgregson
Copy link
Owner

@samgregson samgregson commented Jul 26, 2024

Two changes:

  • move validation set to separate file and create placeholder for "golden" output, see notebooks/validation_set.py.
  • added model as input for each pipe function so that model control is moved up to pipeline function

commit b618814
Author: Sam Gregson <[email protected]>
Date:   Fri Jul 26 11:17:24 2024 +0200

    Update README.md (#14)

    update status badge

commit 4a198d8
Merge: ad1a67a 8d2b17d
Author: Nick Franklin <[email protected]>
Date:   Thu Jul 25 13:50:16 2024 -0700

    Merge pull request #13 from samgregson/Integrate_GPT4o

    gpt 4o mini integration

commit 8d2b17d
Author: NicolasFranklin-MM <[email protected]>
Date:   Thu Jul 25 12:46:44 2024 -0700

    removing gpt40 variable in pipeline line179

commit d9738a4
Author: NicolasFranklin-MM <[email protected]>
Date:   Tue Jul 23 17:13:48 2024 -0700

    gpt 4o mini

commit ad1a67a
Merge: cbd2fbe 5fb4b22
Author: Nick Franklin <[email protected]>
Date:   Wed Jul 17 08:33:03 2024 -0700

    Merge pull request #12 from samgregson/fix_getk_nearest_2

    changed k from 5 to 3

commit 5fb4b22
Author: NicolasFranklin-MM <[email protected]>
Date:   Wed Jul 17 08:27:52 2024 -0700

    changed k from 5 to 3

commit cbd2fbe
Merge: 3e65a38 744f250
Author: Nick Franklin <[email protected]>
Date:   Wed Jul 17 08:21:44 2024 -0700

    Merge pull request #11 from samgregson/semantic_examples_6

    adding first pass example embeddings logic
Copy link
Collaborator

@NicolasFranklin-MM NicolasFranklin-MM left a comment

Choose a reason for hiding this comment

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

all good

@samgregson samgregson merged commit 291ce4d into main Jul 31, 2024
1 check passed
@samgregson samgregson deleted the validation-set branch July 31, 2024 14:35
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.

2 participants