somaticio / tensorflow.rb

tensorflow for ruby
BSD 3-Clause "New" or "Revised" License
829 stars 63 forks source link

[WIP] Adding Constant and Variable #32

Closed arafatkatze closed 8 years ago

arafatkatze commented 8 years ago

@zverok Sorry for the confusion, but I was trying various things to understand why circle ci is failing. I wanted to make a few changes to see if things work and then squash all the un necessary commits later. The current commit offers no changes to the gem but it is still failing by circle ci, so this needs to be examined.

zverok commented 8 years ago

Ah, ok, sorry to intercept in a random moment :)

nethsix commented 8 years ago

@Arafatk This branch has no circle.yml file to tell CircleCI how to perform CI, thus CircleCI tried to be smart and infer where the spec files are, and how to run the specs. Naturally this will fail because CircleCI did not build all the dependencies, e.g., Tensorflow, etc.

You can do any of the following:

  1. Ignore this and proceed to merge if you have done your rspec testing locally
  2. Merge the master (which already has the circle.yml) with this branch, and the re-push. Note: this probably won't run the CircleCI automatically because it is turned off for now since we were still testing how CircleCI can work best (see https://github.com/Arafatk/tensorflow.rb/pull/37)
  3. Copy the circle.yml from master into this branch

Option 2 is probably best.

nethsix commented 8 years ago

@Arafatk I notice that the CI is failing. Let me try to fix it. This may have something to do with what @chrhansen mentioned to me about the branch name but didn't ring a bell back then.