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

Siamese network tutorial #453

Merged
merged 1 commit into from
Dec 10, 2015
Merged

Siamese network tutorial #453

merged 1 commit into from
Dec 10, 2015

Conversation

gheinrich
Copy link
Contributor

No description provided.


To create a train database of 100000 pairs of images into a folder called `siamesedb`:
```sh
$ cd $DIGITS/examples/siamese
Copy link
Member

Choose a reason for hiding this comment

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

This envvar is referred to as $DIGITS_HOME elsewhere in the docs.

@lukeyeager
Copy link
Member

There's a typo in the title for feature_clusters.png - should be "MNIST".


# save test images (one for each label)
testImageSameClass = None
testImageDifferentClass = None
Copy link
Member

Choose a reason for hiding this comment

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

Can we create a few more test images? They're really hard to create without this script, and I find myself wanting to play with a few more examples.

@lukeyeager
Copy link
Member

Sweet tutorial! It worked without a hitch for me. I've added several comments above, but they're all pretty minor.

@gheinrich
Copy link
Contributor Author

Thanks for the review. I changed the Python script to generate 10 test images of each type (same and different classes).


![Test Different Class Channels](different-class-channels.png)

Now run the same test using image `train_test_same_class.png`. In this case this maps two different representations of
Copy link
Member

Choose a reason for hiding this comment

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

I think you mean val_test_same_class_0.png

@lukeyeager
Copy link
Member

Awesome. If you can address the little typo I mentioned, let's go ahead and merge this.

lukeyeager added a commit that referenced this pull request Dec 10, 2015
@lukeyeager lukeyeager merged commit 95ee936 into NVIDIA:master Dec 10, 2015
@gheinrich gheinrich deleted the dev/siamese branch December 12, 2015 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants