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

initial commit, simple, separated models #1

Merged
merged 1 commit into from
Mar 17, 2016

Conversation

snurkabill
Copy link
Contributor

not sure, how this repository is planned to be. so this is just first attemt. feel free to criticise it

@martinwicke
Copy link
Member

Sorry, my notification settings were messed up. I have to fix some CLAbot settings, and then I'll look at your PR in detail.

Thanks!

@martinwicke
Copy link
Member

Can you say something? I'm checking that the CLA bot works.

@snurkabill
Copy link
Contributor Author

something. :)

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@snurkabill
Copy link
Contributor Author

Why do I need sign some CLA? I don't want to fill in my mobile number atc... #justWhyGoogle

@martinwicke
Copy link
Member

The CLA is the only way for us to ensure that the code in this repository is actually licensed under Apache2. If we didn't have CLAs, it would be nominally Apache2, but if someone finds their code here, or thinks they do, our defense may be weak. Therefore we cannot accept contributions without a CLA -- it protects not only Google, but also everybody who uses this code.

Our CLA is very reasonable. I'd love to accept your contribution, but I cannot without the CLA.

@snurkabill
Copy link
Contributor Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

@snurkabill
Copy link
Contributor Author

here you go. but only because you are Google, guys :P

@snurkabill
Copy link
Contributor Author

would you be interested in more sophisticated, objected oriented model? Or are these simple separated models great as PoC?

I can add some more, parametrized models.

@@ -0,0 +1 @@
autoencoder/MNIST_data/*
Copy link

Choose a reason for hiding this comment

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

If you are adding .gitignore cosider adding *.pyc there and remove the pyc files from this PR (and squash it, so the never make it to the repo).

@martinwicke
Copy link
Member

These models are great. Can you remove the pyc as Jan said, then we can merge.

@ilblackdragon, this is using sklearn "bare", maybe down the road we can merge with some of your examples?

@martinwicke martinwicke self-assigned this Mar 14, 2016
@snurkabill
Copy link
Contributor Author

sure, I will do it, sorry for late responses

On 14 March 2016 at 18:47, Martin Wicke notifications@github.com wrote:

These models are great. Can you remove the pyc as Jan said, then we can
merge.

@ilblackdragon https://github.com/ilblackdragon, this is using sklearn
"bare", maybe down the road we can merge with some of your examples?


Reply to this email directly or view it on GitHub
#1 (comment).

@snurkabill
Copy link
Contributor Author

What about now?

@martinwicke
Copy link
Member

Looks good. Can you rebase and squash?

@snurkabill
Copy link
Contributor Author

sure, done 👍

martinwicke added a commit that referenced this pull request Mar 17, 2016
initial commit, simple, separated models
@martinwicke martinwicke merged commit a894b6e into tensorflow:master Mar 17, 2016
@martinwicke
Copy link
Member

Thank you!

yuyu2172 pushed a commit to yuyu2172/models that referenced this pull request Jul 25, 2018
MarkDaoust pushed a commit that referenced this pull request Aug 16, 2018
Add graphs of percents for the prediction array
pooyadavoodi pushed a commit to pooyadavoodi/models that referenced this pull request Apr 11, 2019
Add example image classification inference script and readme
guptapriya added a commit that referenced this pull request Jun 3, 2019
guptapriya added a commit that referenced this pull request Jun 3, 2019
guptapriya added a commit that referenced this pull request Jun 3, 2019
thekevinscott pushed a commit to thekevinscott/laugh-tracks that referenced this pull request Aug 10, 2019
tf-models-copybara-bot pushed a commit that referenced this pull request Sep 23, 2019
tf-models-copybara-bot pushed a commit that referenced this pull request Mar 17, 2020
DavidHanle added a commit to DavidHanle/models that referenced this pull request Jul 8, 2020
lable map for 'Sebastian Houben, Johannes Stallkamp, Jan Salmen, Marc Schlipsing and Christian Igel, 
"Detection of Traffic Signs in Real-World Images: The German Traffic Sign Detection Benchmark",
IEEE International Joint Conference on Neural Networks (submitted), 2013'
ID's are original ID's+1 for a pbtxt-map always starts with tensorflow#1
@DavidHanle DavidHanle mentioned this pull request Jul 8, 2020
9 tasks
tf-models-copybara-bot pushed a commit that referenced this pull request Jan 25, 2021
Fixed Code Execution on tensorflow/models
tf-models-copybara-bot pushed a commit that referenced this pull request May 12, 2021
tf-models-copybara-bot pushed a commit that referenced this pull request Sep 8, 2021
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.

None yet

4 participants