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

Add vqa playground notebook #81

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

eihli
Copy link
Contributor

@eihli eihli commented Mar 6, 2024

Notebooks don't show up pretty in the Github diff viewer. You can see the pretty notebook here: https://github.com/eihli/NEKO/blob/vqa-eval-exploration/dev_notebooks/vqa_playground.ipynb

Explores the difference between training and evaluation runs. Trying to find why evaluation loss doesn't decrease.

Here's something that I found curious. The evaluation generates the correct target answer. But it includes some additional text after the correct text.

See the last screenshots.

Target answer: breakfast
Predicted answer : breakfast Haw

Target answer: breakfast
Predicted answer : breakfaststasy

etc...

The evaluation

image

image

image

Screenshot from 2024-03-06 11-36-23-1

image

eihli added 4 commits March 6, 2024 11:32
Explore the difference between training and evaluation runs.
Trying to find why evaluation loss doesn't decrease.
@bhavul
Copy link
Contributor

bhavul commented Apr 20, 2024

I can see notebook being useful, but why are there two of them?

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