Skip to content

Move inmap to data loader#8

Open
crizCraig wants to merge 3 commits intokarpathy:mainfrom
crizCraig:cq/mv_inmap
Open

Move inmap to data loader#8
crizCraig wants to merge 3 commits intokarpathy:mainfrom
crizCraig:cq/mv_inmap

Conversation

@crizCraig
Copy link
Copy Markdown

Hey Andrej! Cool repo 👍 This is a small change that effectually computes model(x - 0.5) in visualize.ipynb. Otherwise model(x) - 0.5 looks like this
before_mv_inmap
After the change we get
after_mv_inmap
And since we always want to normalize (e.g. before training, validating, or visualizing), I just moved the normalization/inmapping back to the data loader.

This leads to blurry images but that's to be expected with only thousands of possible images. Hoping this will work for planning.
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.

1 participant