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

test GPT layers #899

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

christinakopi
Copy link
Collaborator

No description provided.

@christinakopi christinakopi requested a review from JulienVig March 18, 2025 08:26
@tharvik tharvik self-requested a review March 31, 2025 15:47
@tharvik tharvik force-pushed the NAN-layers_tests-christinakopi branch from d6b7ad1 to 05ce66f Compare April 1, 2025 15:10
Copy link
Collaborator

@tharvik tharvik left a comment

Choose a reason for hiding this comment

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

haa thanks for shedding some light on the dark corners of GPT! some style comments but nothing blocking.
note that I wasn't able to check for the layers implementations as it is still magical to me but as long as the "gpt-tfjs can overfit one sentence" test is green, sounds good!

also, I rebased your branch to update it with the latest develop changes (which also fixes the CI error)

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