This sizing is still incorrect since _total_num_dofs is refernce,

const PostprocessorValue & _total_num_dofs;


Development of functions setWindowLevel and setGPIO


There's a better texture in PixelBOX, FYI.


@dunnbydesign nice job on assigning the array to a variable.  Much cleaner and more reusable.


@RosaChoi For support hit up our [gitter room](https://gitter.im/lodash/lodash).


with this commit is this code snippet still necessary?

https://github.com/rubysolo/dentaku/blob/master/lib/dentaku/ast/identifier.rb#L30-L32
```ruby
      def dependencies_of(node)
        node.respond_to?(:dependencies) ? node.dependencies : []
      end
```


Well, not *really* much explained here :P


@Star55555 please following the steps in the repository's README.md file underneath the section "Api Key Is Required"

Here's a link to it.
https://github.com/udacity/Sunshine-Version-2/blob/sunshine_master/README.md

Hope that helps.


exactly


I'd suggest naming this context 'without args' or maybe 'without args, receives 2 resizes'


@pmros I am not able to reproduce this issue.
Environment: electron 0.34.1, darwin

![image](https://cloud.githubusercontent.com/assets/1622515/10805421/87ee6134-7df3-11e5-9911-7460735d010f.png)


@preetgujral nice!  bonus point achieved!


Jan is correct that the managed debugging support is the only other coreclr pal caller of SetConsoleCtrlHandler and it adds handlers to the head of the list so last added first called so the system console code should be called first on ctrl-c then the debugger whether the coreclr's SetConsoleCtrlHandler is used or System.Native hooks the SIGINT signal directly and chains if not the ctrl-c isn't wanted.


On Wed, Oct 28, 2015 at 09:02:38AM -0700, Mehdi Amini wrote:
> Well you're going to give a tutorial at LLVM Dev about "Building, Testing and Debugging a Simple out-of-tree LLVM Pass". It's too bad that the audience that is using windows can't compile/run the code.

The issue is with LLVM that does not support out-of-tree pass for
windows ;-)


Or better, 'without args, renders defaults'


the first commit


FYI, not sure about Stockfish but I have definitely measured it in other code.


Nice.  But you should be able to take advantage of rspec-given to DRY it up a bit:

```ruby
When { expect_to_statement(match_argument) }

context 'without args, renders default viewports'
   Given(:match_argument) { nil }
   Then { expect(@driver).to have_received(:resize).with(1024, 768) }
   Then { expect(@driver).to have_received(:resize).with(480, 320) }
end

context 'choose a different viewport for a match statement' do
    Given(:match_argument) { { viewport: :wide} }
    Then { expect(@driver).to have_received(:resize).with(1440, 990) }
end

# etc.
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2753](https://teamcity.mesosphere.io/viewLog.html?buildId=65549&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:11:12


Sadly I never really figured out what was causing this.  The only way I was able to fix it from my end was by resizing the ground LOD textures from 128x128 to 64x64.  It is something that popped up literally the day before I submitted my code.  However, that issue went away when I resized some of the textures.  Which branch are you working with?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2755](https://teamcity.mesosphere.io/viewLog.html?buildId=65551&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 169 Build time: 00:04:09


Protege + Github's diff make a real mess of tracking changes. Here's an actual change: the superclass of 'clinical history taking'


Params is a dict, but here you set it to a string... that can't be right.


Here's a real change: moved 'diagnostic process' under 'health care process'


It works OK now.
I guess commit 8da5797 solve this issue.


Hmm, funky Rails-ism :( Thanks for catching this


Nesse capítulo eu pretendo falar mais sobre o conceito de sistema de controle de versão e a maneira mais básica de fazer. Pretendo falar de git no próximo.


This seems to assume that your cluster will only ever be a locator and server on a single host?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2754](https://teamcity.mesosphere.io/viewLog.html?buildId=65550&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 169 Build time: 00:09:16


Where does this user come from?


How is this created?


You want help on this?


@pmros Good to know. thanks :smiley:


As the parameter called ```parent``` is now actually the used ```section```, I would rename it.


Hey @ImadSalimi, is this was intended?, because I don't think so.


What params? The params that used to exist in line 238 was overwritten and this variable does not exist.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2756](https://teamcity.mesosphere.io/viewLog.html?buildId=65553&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 169 Build time: 00:04:20


Author information is not needed


This script seems to assume a lot - such as the user having been created and the bits installed in a particular place, JDK version, etc. It seems like it should be part of a package (RPM) build rather than isolated as it is right now.


[![Hunting Coat](http://api.artsmia.org/images/114602/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114602)


[![The Eternal Feminine](http://api.artsmia.org/images/123752/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123752)


[![Selim and Zuleika](http://api.artsmia.org/images/123767/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123767)


[![Ice bucket with cover](http://api.artsmia.org/images/40260/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=40260)


[![The Exchange Bridge](http://api.artsmia.org/images/8228/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8228)


[![Phil / Fingerprint](http://api.artsmia.org/images/7695/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=7695)


[![The Goldweigher's Field](http://api.artsmia.org/images/8833/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8833)


[![The Exchange Bridge](http://api.artsmia.org/images/9186/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9186)


[![Un Corps de Garde](http://api.artsmia.org/images/63126/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=63126)


[![The Goldweigher's Field](http://api.artsmia.org/images/41906/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=41906)


[![Un Corps de Garde](http://api.artsmia.org/images/63127/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=63127)


[![Effet de Pluie](http://api.artsmia.org/images/105335/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=105335)


[![Phil / Fingerprint](http://api.artsmia.org/images/116570/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116570)


[![Nocturne:  Palaces](http://api.artsmia.org/images/120316/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120316)


[![Stallion and Jack Fighting](http://api.artsmia.org/images/108171/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=108171)


[![Effet de Pluie](http://api.artsmia.org/images/120643/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120643)


[![Ice gun](http://api.artsmia.org/images/6949/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=6949)


[![Hunting Coat](http://api.artsmia.org/images/114602/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114602)


[![The Eternal Feminine](http://api.artsmia.org/images/123752/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123752)


[![Selim and Zuleika](http://api.artsmia.org/images/123767/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123767)


How does the 'product' get to this location?


Looks like a very nice and clean solution to the problem described in #138. I am not sure whether the algorithm works for all thinkable corner cases. However, it does for all we can think of up to now and for that reason it is definitively a better solution than the current one :).


Directly on master


What? It’s had that since forever AFAIK, we just didn’t define tags before.


Good point. These assertions shouldn't fail, but I wanted to be able to debug easily if they do. I'll switch to require.


I'll change it.


Are you certain? @thradec?


Yeah, I think I'll switch to that anyway.  This was just a quick way to get it working yesterday.


Again, no author information is needed. It is part of the Apache Geode, not individual team members.


I added a test for `ShardRecorder`


I will revert back to per thread counters if/when I see evidence that a global counter does not scale. I have measured it on my machine, even with many threads, and could not measure anything.


The `tagged` annotation has existed (in some form) since M2, I assume ceylondoc has supported it for almost as long.


Please review your IDE settings, lines are being wrapped at 80 characters. Don't use a limit or at least use a big one.


Ah, understood. I think that does sound right, mainly because I recall having pain debugging CORS issues due to something like that in the past. Okie dokie.


Most of the lines from 20-24 should be in a geode sysconfig file, instead of directly in the script


The next update should include the correction of the app crashing when user enters a letter and also adding an icon to the app


I was getting errors like
```shell
fatal: [openshift-node-lqt99v6i.os.muc.drebes.org] => SSH Error: ControlPath too long
```
which proved to be due to the $HOME variable not being correctly set:
```shell
<openshift-node-lqt99v6i.os.muc.drebes.org> EXEC ssh -C -tt -vvv -o ControlMaster=auto -o ControlPersist=60s -o ControlPath="/root/openshift-ansible/$HOME/.ansible/cp/ansible-ssh-%h-%p-%r" -o StrictHostKeyChecking=no -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o User=cloud-user -o ConnectTimeout=10 openshift-node-lqt99v6i.os.muc.drebes.org /bin/sh -c 'mkdir -p $HOME/.ansible/tmp/ansible-tmp-1446013646.94-234983501034813 && echo $HOME/.ansible/tmp/ansible-tmp-1446013646.94-234983501034813'
```
this new commit fixes that as well.


What about the `dist` directory?


> The tagged annotation has existed (in some form)

Sure.

> I assume ceylondoc has supported it for almost as long.

I don't think so.


Most of the lines from 20-26 should be in a geode sysconfig file, instead of directly in the script, if they are going to be default properties


Haha, it was a was a terrible catch but I figured a man of your dedication would appreciate the nudge. Thanks for the on going efforts over the years. Will you be at Mage Titans again this year?

Rich

Sent from my iPhone

> On 28 Oct 2015, at 14:11, Vinai Kopp <notifications@github.com> wrote:
>
> I like living in the future.
> Actually, uh, thanks, good catch! :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Thank you for letting me know.

I knew I was making a mess but did not want to wake you. :bow:


It would be nice to add the `Order` to each tree for debugging purposes


I seem to recall `ceylon doc` supporting it at some later time.


https://github.com/ceylon/ceylon-compiler/commit/6905eaa7a467174fd44ed2811e28a97072eeba92, ceylon/ceylon-compiler#381


typo!


@runjak didn't we wanted to use ace for transcribing the files and make the text-field only appear after an area of interest has been selected. Currently there is no way for the user to actually transcribe the object.


OK fine.


TeamCity pull requests :: netty [Build 486](https://garage.netty.io/teamcity/viewLog.html?buildId=2732&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4792, ignored: 21 Build time: 00:36:18


this literally is giving me a boner


But it’s definitely worth pointing this out in the release notes, since it seems to be an underused feature…


Most XenCert tests are run in batches. I am not sure an interactive prompt is the way to go. Perhaps add a command line option that can be passed in when the test is run?


Hey Spencer, did this ever work for reactive subscriptions?


if I try to run this
it { should validate_presence_of(:password_confirmation) }
in spec/models/admin_user_spec.rb it gives this error:
![screen shot 2015-10-28 at 21 10 28](https://cloud.githubusercontent.com/assets/8008067/10805889/70ba931a-7db8-11e5-93ea-96cb7e3fed53.png)


does that mean that in `MONO_DEBUG=bla mono ./compare.exe ...` the benchmark wouldn't see `MONO_DEBUG`? is that what we want?


tests https://github.com/stefanpenner/broccoli-persistent-filter/pull/39


Fix #49


should be
```
java version 1.8 (needed for es 1.7.2)
es version 1.7.2 (yay)
```


I agree with Jens, the two scripts are only a very small part of the full solution. Also, we should track this work as a JIRA so the community can decide whether to add these as files in the product or simply helpful scripts for launching locators and servers.


GF leads to GemFire, maybe should be geode_locator?


GF leads to GemFire, maybe should be geode_server?


@runjak after this commit the selection for an area of interest via the button isn't working anymore. You can't toggle back to the non-selective state in openLayer.


Made public for convenience of those wanting to grab the exact logger by class instead of by name.


EPIC


Please fix the indentation?


this readme update looks awesome.
if the branch name is any suggestion, are you trying to add ruby integration tests within the project? if so I strongly recommend that we place integration tests in the https://github.com/socrata/integration-tests/ cucumber repo.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2757](https://teamcity.mesosphere.io/viewLog.html?buildId=65554&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:27


Yes


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2758](https://teamcity.mesosphere.io/viewLog.html?buildId=65555&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:31


Using logback instead of log4j because:

* Somewhat useful to test with different frameworks then we do in our core modules.
* Easier to create an Appender that builds a string.


Yep! Everything up to this point is working


This is important as otherwise pax-logging is used, this allows us to use another logging framework instead.


s/active/activate


Going crazy... Not working for me... POST, PUT, DELETE working like a charm but nothing to do with GET.

I really can't understand it. The case 'GET' do the same as below? Repeated again? I'm feeling so stupid but I'm not beeing able to run get command.

Thank you guys for fast fixing. Can you take a look?

If you have any plans to improve code, I can contribute as many as I can :)


Indeed


Separated out mailboxBundle from defaultOptions as new tests don't use it.


Some code duplication here between tests but didn't seem like refactoring a utility method was needed.


Es una idea excelente, la envie mi solicitud para entrar en el grupo, esta pendiente, espero que me acepten, yo tambien quiero aportar todo lo que pueda, asi no muere rosadin, desconozco otro programa que me funcione, (es el primero en mi vida que me funciono de verdad), ya me instale kodi, y no funciona el programa, ni siquiera arranca, no dejemos que este programa tan genial muera!!!!

De todas maneras, cualquier cosa que pase, agradezco mucho al autor que se dio el trabajito de buscar periodicamente links que funcionen, espero no llegue a ser la ultima vez. Muchas gracias.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2759](https://teamcity.mesosphere.io/viewLog.html?buildId=65556&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 169 Build time: 00:08:24


I'm writing smoke-tests in Cetera-lipstick and then I'm looking into sbt integration tests based on those. I don't think it's sensible to introduce another language dependency. Also, FE has fallen out of love with Cucumber. It's very sad, one could write a song about it. Cucumber does the laundry.


I am installing from your was video. should I uninstall plugins with ver 2.0.0? I get this error after completing your previous readme steps...

[root@ip-10-0-0-20 elasticsearch]# Exception in thread "main" java.lang.IllegalStateException: Unable to initialize plugins
Likely root cause: java.nio.file.NoSuchFileException: /usr/share/elasticsearch/plugins/bigdesk/plugin-descriptor.properties
	at sun.nio.fs.UnixException.translateToIOException(UnixException.java:86)
	at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:102)
	at sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:107)
	at sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:214)
	at java.nio.file.Files.newByteChannel(Files.java:317)
	at java.nio.file.Files.newByteChannel(Files.java:363)
	at java.nio.file.spi.FileSystemProvider.newInputStream(FileSystemProvider.java:380)
	at java.nio.file.Files.newInputStream(Files.java:108)
	at org.elasticsearch.plugins.PluginInfo.readFromProperties(PluginInfo.java:86)
	at org.elasticsearch.plugins.PluginsService.getPluginBundles(PluginsService.java:306)
	at org.elasticsearch.plugins.PluginsService.<init>(PluginsService.java:112)
	at org.elasticsearch.node.Node.<init>(Node.java:144)
	at org.elasticsearch.node.NodeBuilder.build(NodeBuilder.java:145)
	at org.elasticsearch.bootstrap.Bootstrap.setup(Bootstrap.java:170)
	at org.elasticsearch.bootstrap.Bootstrap.init(Bootstrap.java:270)
	at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:35)
Refer to the log for complete error details.


Will fix manually. Thanks.


should you just do ``transformers=LMS_COURSE_TRANSFORMERS``. Is there a reason someone might pass ``None`` on purpose?


Nice!


Done in 4dd67405903b2fa597c378fb0dba10a232ad26ae, also for the other occurrences where we are dealing with classes instead of native types. Thanks!


khkh


Offhand and probably under-informed comment: There's a reason that the disintegration paper defines a variable to be hnf only if it's not bound in the heap. In case 87 for example, if v1' or v2' were a variable bound in the heap, then the "factor" inserted on the right-hand side would be ill-scoped.


does this mean that the index still holds the data for this node? I came here on my search on how to "delete node from spatial index". Any hints? Thank you :-)


I actually would try this using the "files" key instead of "classmap".  I think the former will force the file to load, the latter just tells Composer to map the class name in that file.


i added new methods to ExampleController for testing, pull the latest client-connect.php and change the variable $method to PUT, DELETE OR GET. should work now


Perhaps I'm not understanding the question correctly, but the destination ID mentioned here is literally an ID, not the sensor's address. For instance, if you want to send a packet to sensor 2, just add `to_id = 2` to the packet. The code then finds sensor 2's address using https://github.com/timvandermeij/drone-tomography/blob/master/settings.json#L95-L99. The first item (0) is always the ground sensor, the second item (1) corresponds to sensor 1, et cetera.

In short, this code provides the abstraction from XBee device addresses and allows external code to just address a sensor by its ID. The only prerequisite is that the sensors list in the settings JSON file is correct, but that only has to be done once for the entire network. It is also possible to do a node discovery, but that takes quite some time and does not find all sensors in the worst case, so I decided against doing that.


@glima this is the expected behaviour,  if. config has changed it means the current setup is not valid anymore,  having that sad,  the opts for example must be regenerated and since we don't know -  or we don't have any means to inspect based on the new opts what must be rebuilt -  we need to rebuild every thing.


@cam5 that's what it's all about!


How does this help?  It it just a lazy copy?


eggcelent


That's awesome. That minimongo-cache setup flow is confusing. Now need to work on with.


But we shouldn't be rebuilding things between 2 make all calls...  But,  if that is the problem then the problem is not here.


Following Jeremiah's fix to deal with this issue:  https://github.com/tesseradata/trelliscope/issues/124, matching the change he made here:  https://github.com/tesseradata/datadr/compare/master...jrounds:master


s/with/auth/g


da pra colocar um lambda ao invés desse foreach


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2761](https://teamcity.mesosphere.io/viewLog.html?buildId=65562&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:14


Interesting. We have two options here - use some other API that does not throw, but perhaps results in equally invalid string. Or catch an exception here and produce an error.

This is a prototype at this point so does not need to be hardened for this, but thanks for noticing. If this goes into production, we need to deal with such scenario and have tests.


Development is on pause. We were considering switching to Python (I think)


não deve retornar XML, apenas uma List<Jogo>.
XML deve ficar somente dentro da classe que trabalho com os dados.


Nos métodos de busca, o ideal é converter o resultado da busca para uma lista de objetos Jogo.


Sorry but I didn't understand. How do you use or not use the Malonzo at runtime?


i can't get the point, it will fire only if it gets some damage and i would avoid a loop for so long


1141 does need to be removed too.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2762](https://teamcity.mesosphere.io/viewLog.html?buildId=65569&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 169 Build time: 00:04:08


Aqui tem que retornar um List<Jogo>


[unused parameter 'map' [-Wunused-parameter]](https://travis-ci.org/TrinityCore/TrinityCore/builds/87984248#L793)


It seems the comment should be removed or the constants redefined as vars?


Typo


Somehow my timer tests stopped working in reactor-pipe :( going to debug tomorrow...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2763](https://teamcity.mesosphere.io/viewLog.html?buildId=65567&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:03


Ha ? the Lite waiting strategy is maybe not a fit ? I was noticing high peak of CPU but it might be from something else (global timer never cancelled)


Fixed.


Wrong id on this and `/0323-translate-an-article.yml`


28 thousand commits


That is the worst idea i've heard  all day....
On 28 Oct 2015 23:38, "V0idSt4r" <notifications@github.com> wrote:

> Development is on pause. We were considering switching to Python (I think)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Torch-Box/Torch-Box/commit/a4923dee936fdc73e0f5d49e4a275510496d2f2c#commitcomment-14055065>
> .
>


This seems to break `make; make install` with LLVM 3.7 .....

`usr/bin/julia` works right after `make` but `make install` fails with

```
LLVM ERROR: Program used external function 'julia_write_22503' which could not be resolved!
```

After that, both `usr/bin/julia` and `usr/bin/julia-debug` fails with similar errors.

Bisect log.

```
d61a09ad8ae6c5d3f9f2a9902f638f3eafdae938 is the first bad commit
commit d61a09ad8ae6c5d3f9f2a9902f638f3eafdae938
Author: Jameson Nash <vtjnash@gmail.com>
Date:   Sun Oct 25 22:13:24 2015 -0400

    implement simple OrcJIT adaptor for Julia

:040000 040000 70a7e3f931790e402bdd3e72184479c1595b9c10e53fd8256403e0b0b6bb6b7e95bbfde26eba5392 M        src
yyc2:~/projects/julia/master
yuyichao% git bisect log
git bisect start
# bad: [344fdc2e379e0a33b35291d2970d159f180dbea4] Merge pull request #13784 from JuliaLang/anj/umf
git bisect bad 344fdc2e379e0a33b35291d2970d159f180dbea4
# good: [37dad15e254405ed7e6dbad9647d1a77b970854d] remove some uses of `svec` in Base
git bisect good 37dad15e254405ed7e6dbad9647d1a77b970854d
# good: [df6105d2125c5ec7dbfa3ec529cd25482be80020] update manpage
git bisect good df6105d2125c5ec7dbfa3ec529cd25482be80020
# bad: [ae903954636cc7544713d09f5af0a0b1b971ece2] block compilation of lambda objects that are not valid for compilation
git bisect bad ae903954636cc7544713d09f5af0a0b1b971ece2
# good: [d594954e299617fdf72af6a896a5ff89c5d831f1] Fix Makefile typo I introduced in my last commit
git bisect good d594954e299617fdf72af6a896a5ff89c5d831f1
# bad: [4e09bbc5be0a8d7bbe2b77d31a818259d1293b58] in --compile=all mode, try to optimize the pattern f{T<:Union{...}}(...)
git bisect bad 4e09bbc5be0a8d7bbe2b77d31a818259d1293b58
# bad: [5287930780545fba8a05a53970f5ddf49cd1e053] Merge pull request #13764 from JuliaLang/jn/orcjit
git bisect bad 5287930780545fba8a05a53970f5ddf49cd1e053
# bad: [d61a09ad8ae6c5d3f9f2a9902f638f3eafdae938] implement simple OrcJIT adaptor for Julia
git bisect bad d61a09ad8ae6c5d3f9f2a9902f638f3eafdae938
# first bad commit: [d61a09ad8ae6c5d3f9f2a9902f638f3eafdae938] implement simple OrcJIT adaptor for Julia
```


Fixed 3 cases of wrong ids.


you should drop this file. if you need it, add it to your gitignore so it doesn't get uploaded to Github


Master is red :(

https://travis-ci.org/rejectedsoftware/vibe.d/jobs/87871836

@MartinNowak FYI


Thanks.


consider moving this logic into a method of the Course class.


Why?


use path helper (courses_path?)


Did you look at the Travis build failure ? https://travis-ci.org/rejectedsoftware/vibe.d/jobs/87862921 it seems totally unrelated (and weird)


The next challenge :smile:


Yes, that build error was present yesterday too, more than 22 hours before this commit.
It happened outside of your commit https://github.com/TrinityCore/TrinityCore/pull/15749#discussion-diff-43210422


Might be... trying to debug / fix but falling asleep :)


Agreed. We felt it would be useful to get a first-cut init script to work from.


Yes, we are optimizing for "time to market" here by lowering early. :-)

This part of implementaton is likely to change. We will likely introduce a special BoundUtf8String node and have a special emit for it. And we will need to make a bunch of mostly trivial modifications in lowering, flow analysis, semantical info, etc so that those are not confused by a new node.


patch-1 (pachi) - i386-Windows-7-VisualStudio-12: OK (2337 of 2365 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.82%25-yellow.svg)


No, I don't think it will change anything for easy moves also.  For easy move to fire you need BestMoveChanges close to 0, so  unstablePVFactor is close to  1, thus available() is always less than maximumTime


let's refactor this partial on our call tomorrow. A partial should be concerned with displaying a single course object


Might be better if we moved this into a matching sysconfig file, as Jens suggested.


Do the new lines between list items render properly?


The user provisioning should likely be covered in the install/deploy docs. This init script should ultimately be updated to reflect the chosen username convention, similar to how httpd adheres to an agreed-upon username convention.


I'd like the overall design of the feature to finalize before making this kind of change. There are alternative designs that could drastically change how syntax of this feature looks like, so I did not want to commit to a particular shape of a bound node.


Yeah it's definitely not the right skill id.


Good point, the matching sysconfig for this init script should parameterize for the general case of deployment options.


We need to also add each route to the list of matches where all the remaining segments could be optional. Then sort matches by precedence - that will resolve the issues with defaults and catchalls.

`/api/Products/search/{price}` is a match for `/api/Products/search` and `/api/Products/search/7` - so it should appear at depth 3 & 4 in the matches list.


Why the array here ?


Is the `>` character right here? Instead of `|`?

http://stackoverflow.com/a/21699210/858553


What's the motivation for this change ? The original version didn't allocated, this one does.


Sounds good. It would be nice if these assumptions were captured in the code as TODO comments.


You realize you're commenting a commit that was merged into master 4 months ago, do you?


@vihari We're only reading 500K lines of Dbpedia, not the whole file?


You can use atan2(y,x) here.


https://github.com/torvalds/linux/commit/3cf521f7dc87c031617fd47e4b7aa2593c2f3daf CVE patches


We can use a simpler class here LinkedList is heavy. The main benefits of using a LinkedList arise when you re-use existing iterators to insert and remove elements.

Since that is not our purpose here - we should use simpler alternatives like List, ArrayList.


Correction, calling open will only work if it's the first command used when running the shell.
Might have something to do with the globalexitstatus?? I'm not sure


Update translate.oc.tc (1)


It should be a persistent link.


If you just want to get rid of the warnings, you can remove the variable names in the line above (or better put them in comments so that you still know what these variables are, in case you ever want to change something and then use them) and then you don't need the following lines.


@tdwong I've updated the README file with the two dependencies that I needed to install; hopefully that helps!

I figured this out by running `Xournal.app/Contents/MacOS/Xournal` in a terminal; this tells you if anything is missing.


Better to use a string builder here.. take a look at the setUserAgent() code.


Rad I'll look into it :)
On Oct 27, 2015 10:50 PM, "Eric Hebert" <notifications@github.com> wrote:

> Maybe just a map of the UMBC campus, which are already available in high
> resolution.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ayebear/polygons/commit/e281e2ecd524320fa6f1616a901326e6eb38f148#commitcomment-14029199>
> .
>


I am getting `a syntax error` because there is no such parameterless constructor in [RequiredClass](https://github.com/DevelopersGuild/AndroidAssist/blob/parsing/app/src/main/java/com/assist/dg/androidassist/RequiredClass.java#L14). @JH72 @keerthisb

```java
public RequiredClass(String name, RequiredClass[] prereqs, float numUnits,
                          boolean take, String univEq)
    {
       formalClassName = name;
       prerequisites = prereqs;
       units = numUnits;
       taken = take;
      univEquivalent = univEq;
      qualif();
    }
```


Raycasts output is not affected by the type of mesh.


Zach and I were able to make a create user page, that does indeed create users. However, they can't log in afterwards because their password is not encrypted. In looking at the users created through the form and the users created in admin, there is a difference on their password. It's at least nice to know that Django recognizes a password created this way and treats it as an invalid password.


@gchiu The same rule as for the divide needing "coverage" applies to multiply.  But getting that coverage can be done with a refinement just as with there.  (An error seems like a good idea as a default.)

I don't know if /ONLY is the most communicative name for getting the "bad behavior".  But if it gets called /ANY then it won't line up very well with divide because divide was already willing to divide money.  :-/


I am pretty sure there is a way to just work with the set in this function, and in the end convert it by a clever function to a vector and return the vector? I am speculating here, but maybe: `std::vector<DoFHandler_active_cell_iterator> patch(cells_done.begin(),cell_done.end());` at the end of the function would do the job.


This revision is clean.


`sort_decreasing_order` is not a good function name. It is not to sort something but just to test the order.
Since this function is just a simple compare, I think we can remove it.


`String` keys of a `Hash` are always frozen.

~~~ruby
a = {"foo"=> 1}
a.keys[0].frozen? #=> true
~~~


얼추 정리된건가?? 1st 2nd 3rd 차이점이 뭔지랑 내가 어떤걸로 사용하면 좋을지, 그리고 현재까지 상황에 대해서 간단히만 설명해줄 수 있어?ㅎㅎ


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2658](https://teamcity.mesosphere.io/viewLog.html?buildId=65243&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:09


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1947 of 1956 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.54%25-yellow.svg)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2659](https://teamcity.mesosphere.io/viewLog.html?buildId=65244&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 188 Build time: 00:03:11
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Show de bola @grupolanches .
Att.


These comments are nice, but I would expect them in the header file.


I think those codes will get in trouble when `dim==3`.
Add  `if(dim == 2)` or ` Assert(dim==2, ...)`


Need one more blank line.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2660](https://teamcity.mesosphere.io/viewLog.html?buildId=65245&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:37


See my note about capital letters in variable names in StokesSolution.cc


Okay


[DOCTYPE html.docx](https://github.com/Monse-Nolasco/Nowvit/files/22170/DOCTYPE.html.docx)


Somehow this indentation is broken


At the moment, it is not really clear what the error estimate is and what you do with it. It would be nice to have some comments here that describe what you do, i.e. compute the error estimator that is then used to mark cells for refinement.


This line is to aviod user rate 0 for one movie and dividing by 0 is meaningless.


Hey y'all. Please check my code and contribute any changes if needed!


@chu575 No it's fine, ```i``` is defined outside the for loops. While you are right that the first time ```i=0```, for the others iterations of the ```outside``` for loops ```i``` may be non-zero


When converting leniency to reputation, we use absoluate value. So here is no metter whether is ```rr.score - movie.temp_score``` or ```movie.temp_score - rr.score```.


You can't `return` from a closure though.


Here the function is doing something beyond what suggested by its name `generate_mesh`.
If you always have to do this after setting up the mesh, you can call this function `initialize()` and document what it is designed to do in header file.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2662](https://teamcity.mesosphere.io/viewLog.html?buildId=65251&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 188 Build time: 00:02:53
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Use ```Assert``` to speed-up the code in release mode. It's sufficient to check this in debug mode.


Internal error is not a very explicit.


What does ```cell_tt``` mean?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2661](https://teamcity.mesosphere.io/viewLog.html?buildId=65250&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:08:24


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2664](https://teamcity.mesosphere.io/viewLog.html?buildId=65253&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:18


Looks great, can't wait to see it in action next week!


```else``` and ```for``` on the same line is pretty strange.


gitignoreいいですね！
アップされないようになりました！


What does ```cell_ttt``` mean? What's the difference with ```cell_tt```?


But where will it lead us?


So why not call them `i_cell` and `i_vertex`?


セクシーですね！
短く書けるところは短く、読みやすく書けるところは読みやすく記述するようにしましょう！


this is good.

i do think we should remove margin from headings by default and apply them specifically where they are needed (eg: body of marketing copy, blog post, etc.)

default for "application" usage should be no margin


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2665](https://teamcity.mesosphere.io/viewLog.html?buildId=65254&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:07


You are still checking if a primitive is null. This is never going to happen.


initial address 1CDSr9UTM1J6QYxzVBsoBodT7T5Yiszgwn
output address 1GXj62bo8bvzj9MTUEBwFHTyzW6RArvQcT


ここのコメントアウトはなんでしょうか？


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1424 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.4%25-green.svg)


このテストファイルも削除しましょう！


Ah, thanks. I did not know that. I see the reasoning behind it.

---------------------------------------------
Von: Bruno Turcksin
Gesendet: 27.10.2015 21:48
An: Arezou-gh/CodeReview
Cc: Rene Gassmöller
Betreff: Re: [CodeReview] The CodeReview comments-suggestions took in edditing-rewritting the code (0cf07a1)

> Bruno Turcksin <notifications@github.com> hat am 28. Oktober 2015 um 03:48 geschrieben:
>
>
> @gassmoeller It works like usual. It's just easier to read pointer and reference from the right to the left. For example, ```int const* ``` is a pointer to a constant int, ```int* const``` is constant pointer to an int, and ```int const* const``` is a constant pointer to a constant int. It's easier to read than ```const int* const``` where there is no order.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Arezou-gh/CodeReview/commit/0cf07a18070e288f61b0f5330e395305b82e13a1#commitcomment-14029178


Can't use lombok as we need to return `this` on setter method.


ahahahah


Oops


This is a test showing that we can add custom fields to the api_key and that we can read them back and even set it as an NGINX variable, if that variable has been previously defined in NGINX config.


@sercharpak ahi ya tienes una primera serie de cambios. Quedo pendiente de una nueva iteracion tuya con mas info en la introduccion y en la metodologia.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2663](https://teamcity.mesosphere.io/viewLog.html?buildId=65252&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 171 Build time: 00:07:47


Please add messages to your asserts so you anyone can know what the test is trying to do.
(Both if it fails and when reading it.)


We decided to set the value from Service class? to handle the `validFrom` and `validTo`.


What's the difference between ```active_patch_cell``` and ```act_patch_cell```?


Why is this code commented?


Declare them immediately before the `for` loop.


According to sdkassettypes, the DATA block for both Mesh and Particle have a KV3 encoding:
```
symbol m_DataBlockEncoding		= RESOURCE_ENCODING_KV3
```
And this lines up with [Dingf's implementation of it](https://github.com/Dingf/Source-2-Decompiler/blob/master/kv3.cpp#L48-L81) which appears to be some sort of decompression which would help explain why when you open a vpcf_c in a hex editor (or notepad++) all the strings that you would expect from a NTRO readable DATA block are deformed.


Why don't you use ```act_end_patch_cell```?


Same thing as before.


Same thing as above.


1. Good call. The `#{nil}` part went in before nil existed on the JS side, or at least before I knew about it. :-)
2. `||` also short-circuits. If the lefthand side is true, the right hand side is irrelevant. :-)


Declare `repeat_vertex` here.


See also: clutchski/coffeelint#526


You've still got an unneeded "sprite" reference in the examples section, where it says:

> This will fetch icon1.svg and icon2.svg and add two symbols to the sprite.


Why do you use ```dofs_per_cel``` instead of ```dofs_per_cell```?


# Source, Executable, Includes, Library Defines
INCL   = loop.h defs.h
SRC    = moo.c foo.c bar.c baz.c dood.c shazbot.c mop.c woot.c
OBJ    = $(SRC:.c=.o)
LIBS   = -lgen
EXE    = moolicious

# Compiler, Linker Defines
CC      = /usr/bin/gcc
CFLAGS  = -ansi -pedantic -Wall -O2
LIBPATH = -L.
LDFLAGS = -o $(EXE) $(LIBPATH) $(LIBS)
CFDEBUG = -ansi -pedantic -Wall -g -DDEBUG $(LDFLAGS)
RM      = /bin/rm -f

# Compile and Assemble C Source Files into Object Files
%.o: %.c
       $(CC) -c $(CFLAGS) $*.c

# Link all Object Files with external Libraries into Binaries
$(EXE): $(OBJ)
       $(CC) $(LDFLAGS) $(OBJ)

# Objects depend on these Libraries
$(OBJ): $(INCL)

# Create a gdb/dbx Capable Executable with DEBUG flags turned on
debug:
       $(CC) $(CFDEBUG) $(SRC)

# Clean Up Objects, Exectuables, Dumps out of source directory
clean:
       $(RM) $(OBJ) $(EXE) core a.out


Extra blank line


Missing blank line.


There are weekly and monthly history present as well. Seems will not work for some scenarios.


Comment should be one line lower.


Extra blank line.


ohhhh i see ok gotcha


(Y)


jquery-moonflowers


Declare `end_cell` with `const`.
It's a good practice to declare a variable as constant when you do not intend to change it.
This can prevent you from introducing bugs accidentally and make the code easier to read.


What about Lombok's `@Builder` annotation (or similar)?


the `docker exec` commands to attache a shell the containers were failing because the containers were not yet ready.
the delay gives my PC a chance to complete spinning up all of the containers, which is very resource intensive


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2666](https://teamcity.mesosphere.io/viewLog.html?buildId=65255&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:30


@VictorJSV por favor revisar el cambio de @Edux87, al parecer no hay gran cambio por favor revisalo para hacer el pull request. Gracias

c.c. @andru255


Guess the word: XXXXX

Guess a letter: ?????
?3
Letters only please

Guess a letter: ?????
?4
Letters only please

Guess a letter: ?????
?6
Letters only please

Guess a letter: ?????
?a

Your guesses:
 A

Guess a letter: A??A?
?a
'A' Was already guessed. Try another letter.

Guess a letter: A??A?
?s

Your guesses:
 A S

 o

Guess a letter: A??A?
?r

Your guesses:
 A S R

Guess a letter: ARRA?
?y

Your guesses:
 Y A S R

Word: ARRAY

Congratulations!!! You guessed my word.

Play again? (y/n) y
Guess the word: XXXXXXX

Guess a letter: ???????
?c

Your guesses:
 C

 o

Guess a letter: ???????
?g

Your guesses:
 C G

Guess a letter: ????G??
?q

Your guesses:
 C G Q

 o
/

Guess a letter: ????G??
?w

Your guesses:
 C W G Q

 o
/|

Guess a letter: ????G??
?e

Your guesses:
 C W G Q E

Guess a letter: ???EGE?
?r

Your guesses:
 C W G Q E R

Guess a letter: ???EGER
?t

Your guesses:
 C W G Q E R T

Guess a letter: ??TEGER
?y

Your guesses:
 C W G Y Q E R T

 o
/|\

Guess a letter: ??TEGER
?u

Your guesses:
 C W G Y Q U E R T

 o
/|\
 |

Guess a letter: ??TEGER
?o

Your guesses:
 C W G Y Q U E R T O

 o
/|\
 |
/

Guess a letter: ??TEGER
?p

Your guesses:
 C W G Y Q U E R T O P

 o
/|\
 |
/ \

Sorry - you have run out of guesses ;(
The correct answer was: INTEGER

 ...GAME OVER...

Play again? (y/n) n
Press any key to continue . . .


I tried to test this, but think I did not test right; I got "unsupported parameter for module: creates" internally.  I need a clue about how to test a fix to 1.9.  Here is a precis of what I did:

bash-3.2$ pushd ~/Documents/ansible/
~/Documents/ansible ~/Documents/mjs-infra/playbooks

bash-3.2$ git clone https://github.com/ansible/ansible.git --recursive
Cloning into 'ansible'...

bash-3.2$ cd ansible/

bash-3.2$ pushd lib/ansible/modules/core/
~/Documents/ansible/ansible/lib/ansible/modules/core ~/Documents/ansible/ansible ~/Documents/mjs-infra/playbooks

bash-3.2$ git checkout remotes/origin/stable-1.9

bash-3.2$ pushd ../extras/
~/Documents/ansible/ansible/lib/ansible/modules/extras ~/Documents/ansible/ansible/lib/ansible/modules/core ~/Documents/ansible/ansible ~/Documents/mjs-infra/playbooks

bash-3.2$ git checkout remotes/origin/stable-1.9

bash-3.2$ pushd +2
~/Documents/ansible/ansible ~/Documents/mjs-infra/playbooks ~/Documents/ansible/ansible/lib/ansible/modules/extras ~/Documents/ansible/ansible/lib/ansible/modules/core

bash-3.2$ git checkout f8cac24cb23ff85f24f2a35f35572cf5aeca406f

then I test


Also I moved the supporting classes to SensorState inside SensorState, to avoid namespace pollution while keeping access across packages easy.


Doing a visual scan of the changes here, which look fine, by and large.

Imports in the XSL: I think `mathbook-html.xsl` is overkill, since maybe all it is bringing in (I suspect) is the `internal-id` template, which is in `mathbook-common`.

Enable the `extract-identity.xsl` template and list it first (then common, then pg, then base64).  It is like a "do-nothing" template.  Then  make

```%xsl
<xsl:apply-templates select="//webwork[descendant::image[@pg-name]]"/>
```

into simply

```%xsl
<xsl:apply-templates />
```

Can you test that?


BasementDomainNumCounterBug (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1960 of 1960 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


We can have the examples differ so that we show the variety. I have added setContentType() to the exampe.


I was able to some testing and was able to connect and issue commands from the IRC bot. But there is one failure when I issued a "bbtest: force build runtests test build":

```
2015-10-27 20:35:25-0700 [IrcStatusBot,client] irc command force
2015-10-27 20:35:25-0700 [IrcStatusBot,client] Unhandled Error
	Traceback (most recent call last):
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/Twisted-15.4.0-py2.7-linux-x86_64.egg/twisted/words/protocols/irc.py", line 2579, in handleCommand
	    method(prefix, params)
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/Twisted-15.4.0-py2.7-linux-x86_64.egg/twisted/words/protocols/irc.py", line 1944, in irc_PRIVMSG
	    self.privmsg(user, channel, message)
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/buildbot-0.9.0b4_23_g222878c-py2.7.egg/buildbot/status/irc.py", line 97, in privmsg
	    d = contact.handleMessage(message)
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/buildbot-0.9.0b4_23_g222878c-py2.7.egg/buildbot/status/words.py", line 981, in handleMessage
	    d = defer.maybeDeferred(meth, args.strip())
	--- <exception caught here> ---
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/Twisted-15.4.0-py2.7-linux-x86_64.egg/twisted/internet/defer.py", line 150, in maybeDeferred
	    result = f(*args, **kw)
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/buildbot-0.9.0b4_23_g222878c-py2.7.egg/buildbot/status/words.py", line 691, in command_FORCE
	    waited_for=False)
	  File "/home/wink/bb/venv/lib/python2.7/site-packages/Twisted-15.4.0-py2.7-linux-x86_64.egg/twisted/internet/defer.py", line 1265, in unwindGenerator
	    gen = f(*args, **kwargs)
	exceptions.TypeError: addBuildset() got an unexpected keyword argument 'builderNames'
```


Law of demeter


@theskumar
Why we are not using slugify available inside django utils ?

from django.utils.text import slugify


`Jump_est_per_cell` -> `jump_est_per_cell`


and BOOM
 missed one


it's a wrong description not "Hex".


I would still initiliaze to default in initialization list and set up custom values here.


This `2` is a magic number. I guess it should be `dim`?


That would be cool and provide UI's some information.

On Wed, 28 Oct 2015 12:37 pm Stephen Lyons <notifications@github.com> wrote:

> Um, re: bug 1427364 <https://bugs.launchpad.net/mudlet/+bug/1427364>,
> shouldn't this be _url.contains("https") NOT _path.contains("https") [image:
> :confused:]
>
> Also: there is a specific QUrl::scheme() to pick the relevant bit out of
> a QUrl if we bothered to create a local QUrl instance which we could use
> QUrl::isValid() on before even trying to create the QNetworkRequest. I am
> enhancing and refactoring code in this area to add more error checking and
> validation and also clean-up afterwards - one is supposed to delete() (or
> actually deleteLater() in the slot connected to the replyFinished()
> *SIGNAL*) the QNetworkReplay when you have finished with it!
>
> Also the existing code-base does not provide any record of past downloads
> in a session - you MUST have an event handler in place to catch the
> TEvents created - as they happen. As well as various fixes, I am coding
> up a downloadFilesStatus() function as I type this which will provide
> something a bit like the info that FireFox's *Downloads* does to track
> what files have been downloaded, the fully encoded representation of the
> Url that was used, the state of the download (active/completed/several
> error states), and if successful the byte count so far obtained (if active)
> or written out (if completed)... look for a PR soon. [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vadi2/Mudlet/commit/0053fd1e82ed04bd489f25fb97a64d7e50b9d346#commitcomment-14029054>
> .
>


Yes, it's a law of demeter violation.  As the guy who *created* the law of demeter article on Wikipedia, I also don't think it's a problem here.

The issue here is that @project.user.name goes through "user" to get the name, but project.user.name actually a pretty clean way to get it; creating an extra method to eliminate it won't provide any maintainability gain.


Hey man, this is really great! I appreciate it.


lösenorden ska senare förvaras krypterade i databasen


Huzzah!


This command is failing for me.
I renamed the directory that I previously `curl`'d and `make install`'d in from `rustc-1.3.0` to `. rustc-ios`, hoping to fix it.
I get `multirust: no rustc in custom toolchain at '/Users/asb/.rustc-ios/bin/rustc'`

I also tried simply `multirust update ios`, but I'm told:
```
multirust: installing toolchain 'ios'
rustup: gpg available. signatures will be verified
rustup: downloading toolchain for 'ios'
rustup: command failed: curl -s -f -O http://static-rust-lang-org.s3-website-us-west-1.amazonaws.com/dist/rust-ios-x86_64-apple-darwin.tar.gz.sha256
rustup: couldn't download checksum file 'http://static-rust-lang-org.s3-website-us-west-1.amazonaws.com/dist/rust-ios-x86_64-apple-darwin.tar.gz.sha256'
```


BasementDomainNumCounterBug (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1960 of 1960 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I think you missed the point. We need that if check. In safari, there are some nodes in the `elements` which are not DOM nodes. That's what the problem.

Before you send the PR, try whether it's working on safari. Don't just run tests. Try using a real app.


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1961 of 1961 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


The idea is that `make install` will install into `~/.rustc-ios` automatically so that you don't have to

Is `make install` succeeding?


So I tried testing a different way, still got the same complaint when using the args syntax, no complaint when using the run-in syntax.  Following is a precis of what I did.

bash-3.2$ cd ~/Documents/ansible/
bash-3.2$ git clone -b stable-1.9 https://github.com/ansible/ansible.git --recursive
Cloning into 'ansible'...

bash-3.2$ cd ansible/

bash-3.2$ source ./hacking/env-setup

(Here I test, verifying that I am getting the old behavior.)

bash-3.2$ pushd
~/Documents/ansible/ansible ~/Documents/mjs-infra/playbooks
bash-3.2$ git branch -v
* stable-1.9 8b5588f Note the script and raw yaml dict fix
bash-3.2$ git checkout f8cac24cb23ff85f24f2a35f35572cf5aeca406f

(test the fix)


BasementDomainNumCounterBug (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Looks nice, let's see it in action!


That's an odd way to name your functions. Maybe change them to be a bit more user-friendly? They'd also look nicer on the wiki if added.


Does the v1.1.1 pcap work with smash version 1.1.2? If not, will this be updated in the future?


@aswenson -- i know you worked on this. any "insights" to add?


OK, now I feel even dumber, apparently stable-1.9 has already been updated to include this change.  But my testing of stable-1.9 showed the old behavior.  Maybe I need a clue about how to test, this is my first time using ansible from source; see above for some approaches I tried.


@dhaworth is there a cool diagram of these pieces somewhere?


@BillyOnWiiUIn144p Do you know what metamethods are?


@aswenson, @dhaworth, @csquire : decided to start transferring our outline to words. let me know what you think of the format. also feel free to add some notes/ideas/sentences/paragraphs, please.


Make it more clear as `velocity_divergences`.


@dale76uk It is merged in 4.3-rc7. Better wait for 4.3 become stable.


My mistake, I wasn't looking as closely when I installed.

`make -j$(sysctl hw.ncpu | sed -e 's/^.*: //')` fails with:
```sh
make[1]: *** No rule to make target `build_lib_static'.  Stop.
make: *** [x86_64-apple-darwin/rt/jemalloc/lib/libjemalloc_pic.a] Error 2
make: *** Waiting for unfinished jobs....
```


I read the overscan specs on Hybridcast TV.
Hybridcast TV can overscan when three conditions are fullfilled.

* Playing the full screen video broadcasted.
* The broadcasting video is placed at upperleft position (top:0px, left:0px).
* The size of broadcasting video is same as HTML5 application's one.

So, I think Hybridcast TV will not overscan, because the Web Platform Test runner has no broadcasting video object.


patch-1 (pachi) - Win64-Windows-7-VisualStudio-12: OK (2335 of 2365 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.73%25-yellow.svg)


This is confusing because it adds to the existing Golang section and does not explain why the first image cannot work on ARM.


What was outdated about them?


I'm a little out of my comfort zone, I've tried taking a look at the Rust project and their issues for anything about the `build_lib_static` target to no avail.

I also don't really understand what `make -j$(sysctl hw.ncpu | sed -e 's/^.*: //')` is doing. Stream editors and the kernel state is lower level than I'm used to.


@arunoda I understand. There is also length and NodeList prototype. Since they are not iterable objects, they won't be processed with for-of.

I just checked in the real app with blaze and FR. Everything works as expected.


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1376 of 1376 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.37%25-green.svg)


这一刻，心里好难受


`$(sysctl hw.ncpu | sed -e 's/^.*: //')` evaluates to the number of CPUs on your Mac. For me, it's `4`. It's the same as typing `make -j4`.

Investigating the `make` issue


Okay. Then we can use this. Add a comment on why we used it.
I'll also check it.

On Wed, Oct 28, 2015 at 9:51 AM Vlad Shcherbin <notifications@github.com>
wrote:

> @arunoda <https://github.com/arunoda> I understand. There is also length
> and NodeList prototype. Since they are not iterable objects, they won't be
> processed with for-of.
>
> I just checked in the real app with blaze and FR. Everything works as
> expected.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kadirahq/meteor-dochead/commit/c0ce85d573e5dbf8cfa0ae02fa7f777743059276#commitcomment-14029910>
> .
>


Looking at a guide from Big Nerd Ranch on building an iOS app in Rust. May be able to use their instructions to build rust.
Will edit this guide with my learnings to make installation easier for others.


I don't know why this works. I only found function `get_function_laplacians` that takes `std::vector< Vector<> > ` as parameter in deal.II doxtgen.


RubyGems has an API, it's quite simple to query for the latest version of a gem: https://rubygems.org/api/v1/versions/jekyll/latest.json


不错。不过用记忆式搜索，写成类似如下的形式会更简单一点：
int getValue(int pos) {
  if (dp[pos] == -1) {
    dp[pos] = xxxx;
  }
  return dp[pos];
}


I'll fix this after accepting -- actually the test as designed before was correct: if inPlace is True then nothing should be returned (just m21 style).  But you've done enough changes; it's not a show stopper.  Thanks for this.  It'll be great!


Just a random thought, but I think we might be at the point that we can use a runtimeIC as tp_iter.  (I think the thing that was missing before was the ability to throw CAPI exceptions from an IC.)


把求max分成单独的一步？


The API for Led actually has not changed. The static methods `Led_*` are [created](https://github.com/rhempel/ev3dev-lang-python/blob/5c5fc3e025092cadc85c340cacc10982e8642a08/ev3dev.py#L1666-L1671) outside of the `Led` class but then [added](https://github.com/rhempel/ev3dev-lang-python/blob/5c5fc3e025092cadc85c340cacc10982e8642a08/ev3dev.py#L1672) to the class.


This works fabulously. :)


This else can be removed. There is no the third possibility except `true` and `false`.


I think probably should now also start actively blocking < VS2012 or possibly < VS2013?


Use sublime text 3. it will show the difference easily.


Use English for all communcation.


@elecnix , The first project was created a long time ago since we didn't have ARM builders. There is a known issue between QEMU and Go, so we couldn't cross-build Go application on x86_64 build servers and that's why the project can't work on ARM.
I will ping my work mate to update the project since we have native ARM build servers now.

Sorry for this, We'll update our docs to make it clear.
Next time could you please report your issue to Resin official repository here: https://github.com/resin-io/docs, so we can provide you better support


On it. It was a problem in `AC::TestCase`.


This should be done with attribute-set and it should be extended to cover all page-sequences.


Looks good! Thanks for those modifications.


I discovered the iOS 8 part 😔. Weird about peeking
On Tue, Oct 27, 2015 at 16:19 montehurd <notifications@github.com> wrote:

> It also crashes on iOS 8 again :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wikimedia/wikipedia-ios/commit/cd08e3aa9769686c884717c5fbdc25031be228e4#commitcomment-14026442>
> .
>


BasementDomainNumCounterBug (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (886 of 886 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-24.56%25-green.svg)


Đất nền Bình Phước giá rẻ Mua bán - ký gửi nhà đất giá rẻ tại Bình Phước
Đất nền Bình Phước chỉ từ 130 triệu/nền
Nhà Đất Bình Phước giá rẻ chỉ từ 350 triệu/căn
Bao sổ, công chứng ngay
Holine: 0977.58.57.68
Website: http://datnenbinhphuoc.com


yeah defaults are a good idea


This should be a "enter a zip code" I believe?


Thanks for the info, i just tried with the above config, and it still dosent work for me, when i run it on the zed board. I am creating an issue of the same on git with all details below.

https://github.com/ucb-bar/rocket-chip/issues/23


The welcome-file-list in core's web.xml lists index.html, index.form, index.htm, and index.jsp last. Spring will pick up index.form and index.htm before the container/JspServlet picks up index.jsp. But @dkayiwa can confirm.


it's just where i have mw installed, this is meant to be configurable.  split in half because i figure we'll need to know the parent and the relative submodule path in order to pull the right branch and update the pointer.  there's probably a better way, very preliminary


Thanks, I've applied it to the master branch. The next commit requires more work unfortunately.


`REJECT_ALL_IF_INVALID_RECORD_EXISTS` is not actually doing anything yet, correct?


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1961 of 1961 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Last time, I was working with `django.utils.text.slufify` it didn't play well with unicode. I'm not sure of the current status, so went ahead with what I trust.

That said, the [current implementation](https://github.com/django/django/blob/master/django/utils/text.py#L413-L427) does looks like it handle unicode. I'll do some checks and replace it with inbuilt one if satisfactory. Nice catch!


Ok, agreed


Big Nerd Ranch the same instructions you gave. I'll defer to the results of your investigation


delegating to isNoncodedDrug()..


Btw.: You're still not listed as a contributor although you've contributed quite some patches already. Please make sure that your e-mail address from commits is associated with your GitHub account so that GitHub can find you.


Fixed in a05f410


a nghĩ là sửa thành find_course đi cho đồng bộ. vì mọi cái nhóm làm đang là find


ooops :)


Good eye, I will change it in the next commit.


Hi Rob,

That breaks it for several reasons, which is why this is the way it was.
But hopefully you can diagnose the issues better and find a better solution.

Without

<xsl:apply-templates select="//webwork[descendant::image[@pg-name]]"/>

then the "Python triple", which should be just:
('base64 for the one problem','123567890','webwork-15')

ends up having a bunch of text before that first parenthesis. Specifically,
it's the PG code for every other webwork problem in the sample article,
interspersed with the `<wbwk />` output from the article (the shortcut for
WeBWoRK). So of course the script breaks. It means a bunch of templates are
applied for some reason, and the select specification stops them all.

If I go back to specifying

select="//webwork[descendant::image[@pg-name]]"

then the script all executes fine, but the problem is that the content of
the PG that was converted to base64 is now some kind of strange subset of
the PG that it should be. Earlier in this thread you back-translated two
base64 strings, and could see this comparing the two versions of the PG
problem. While I can't explain it, setting up the imports the way I have
them appears to make it fully give the PG problem. It makes the base64
identical to what the webwork-html.xsl gives for base64 (except the
timestamp part is different).

Any ideas why?

Alex

On Tue, Oct 27, 2015 at 8:48 PM, Rob Beezer <notifications@github.com>
wrote:

> Doing a visual scan of the changes here, which look fine, by and large.
>
> Imports in the XSL: I think mathbook-html.xsl is overkill, since maybe
> all it is bringing in (I suspect) is the internal-id template, which is
> in mathbook-common.
>
> Enable the extract-identity.xsl template and list it first (then common,
> then pg, then base64). It is like a "do-nothing" template. Then make
>
> <xsl:apply-templates select="//webwork[descendant::image[@pg-name]]"/>
>
> into simply
>
> <xsl:apply-templates />
>
> Can you test that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Alex-Jordan/mathbook/commit/754ab58f16fe65b6c5a1ef286d4d993aeb4afa0d#commitcomment-14029687>
> .
>

--
Alex Jordan
Mathematics Instructor
Portland Community College


checking if the nonCoded Value is not blank..


In fact I was trying something myself. My own app ID.


Using StringUtils.isNotBlank to check whether drugNonCoded is blank


By the way, accept the bitter truth that I haven't modified the map's collision architecture.


my thinking is something like updating a submodule can involve a couple relevant directories so cwd might not be enough information.


yep


Replacing the logic to use hasSameOrderableAs() .. Changing the exception message as per suggestion


Thanks, fixed in 0091820.


got this from @ejegg


Adding nonCoded check to see if it has either drug or nonCoded


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2669](https://teamcity.mesosphere.io/viewLog.html?buildId=65266&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:08:15


Cool. I was just curious what the health bar was looking like so I noticed that while messing around with it and thought I would mention it. Pretty cool though, looks nice :D


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2668](https://teamcity.mesosphere.io/viewLog.html?buildId=65265&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:09:07


Removing the method checkDrugOrdersForDiscontinuing() and using hasSameOrderableAs instead..


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-71.05%25-red.svg)


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1947 of 1956 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.54%25-yellow.svg)


checking if the concept uuid is not blank..


adding "the" concept..


first checking if the drug is required..


It has nearly same code for paralyze


indentation


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.49](https://pixelated.design/build/viewLog.html?buildId=172&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 275 Build time: 00:00:14


why use both cost_price_cur and cost_price?


empty lines for what?


the unsupported parameter error probably means that the playbook syntax is slightly off.  Here's the playbook I tested with:
```
---
- hosts: localhost
  tasks:
    - script: /bin/cat /etc/passwd
      args:
        creates: /etc/passwd
```

Also -- it appears that I left out part of my fix... I must either have put it in a different checkout or it got dropped when I stashed or rebased or something..... Luckily it's easy enough to recreate.... I'll have it pushed in a little bit.


jadi ceritanya begini

-> bahasa C adalah bahasa prosedural
-> Bahasa prosedural artinya bahasa itu dijalankan urut dari atas ke bawah, sesuai pemanggilan fungsi int main(void){}
-> Arduino memakai bahasa C

Jadi klo dalam bahasa C, contoh seperti ini tidak bisa djalankan, karena saat fungsi main() dipanggil, fungsi1() belum di kenal:

===============================================

int main(void){
 fungsi1();
}

void fungsi(){
 statement1;
 statement2;
}

===============================================

Nah,,kalau dalam bahasa C, lebih baik fungsi 1 di deklarasi dahulu, baru dibelakang ada referensi,,contoh

===============================================

void fungsi1();

int main(void){
 fungsi1();
}

void fungsi(){
 statement1;
 statement2;
}

===============================================

atau sekalian fungsi1 dideklarasi dan direferensi bersamaan,,seperti yg tak revisi kmaren

===============================================

void fungsi(){
 statement1;
 statement2;
}

int main(void){
 fungsi1();
}

===============================================

Nah seperti itu kaidah prosedural dari bahasa C


Ok, melhorado :).


String.join wouldn't exist on openjdk7 - rather than that..  We already use Guava dependency so we can use their helper Joiner - http://docs.guava-libraries.googlecode.com/git-history/release/javadoc/com/google/common/base/Joiner.html


If concept is DRUG OTHER
1) drug is null.. then drug non coded is required..
2) drug not null.. then error saying cannot set both drug and drugNonCoded


Hi @wakes :)

Cheers for giving this some attention. I was also thinking about this .. in my humble opinion. This change can be slightly disruptive. Specially for people who have already used that configuration parameter. Maybe I kindly recommend that we keep 'default_config_path' configuration parameter and introduce the two new parameters as you have i.e module_scripts_dir and dropzone_script_file.

I agree with your objective of being able to load them outside of dropzone module directory. Right now, I have just been loading it using the following in my controller.

```
	public function init(){
		parent::init();
		Requirements::javascript(DROPZONE_DIR.'/javascript/dropzone.js');
		Requirements::javascript(DROPZONE_DIR.'/javascript/file_attachment_field.js');
		Requirements::css(DROPZONE_DIR.'/css/file_attachment_field.css');
	}

Another delima I have is, what happens if I copy the contents of config.yml that sits inside dropzone in to mysite/_conf/config.yml. Would I be able to overwrite dropzones configuration? I also don't see any default configuration setter methods ... That is another reason why I second your pull request to move this configuration out of dropzone module domain.


If concept is  not DRUG OTHER
1) drug is null and not nonCoded.. then drug is required..
2) drug not null and isNonCoded .. then error saying cannot set both drug and drugNonCoded


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


often have to approve things in sequence to get sha for submodule pointer or composer.lock for the next patch, which is why i thought auto +2 would be useful


correcting the test.. If the concepts match and drug non coded is empty.. hasSameOrderableAs() should return true..


reverting the change..


i can look into it, but worth mentioning that this is a separate un/pw combo from the ldap one.  it is defined in the http-auth section of the web ui


reverting the change


este metodo tiene que estar mal no pueden ser 3 colecciones con cosas parecidas


tapi,,disisi lain,, Arduino memakai bahasa C yang sudah dimodifikasi (simplified)

sisi prosedural C seperti penggunaan fungsi int main(void) di "hide" oleh Arduino IDE

jadi ana ndak bisa pastikan apakah kaidah prosedural deklarasi-referensi tetep sama atau tidak perlu se "disiplin" bahasa C dasar

kuatir ndak jalan,,makanya tak revisi kmaren

emang antum nulis fungsi dengan referensi fungsi di belakang itu da yg ajari kah?

klo iya,,pakai itu ndak apa kalau emang diketahui bisa jalan


changed the exception and corresponding test..


changed the global property name..


101162510-SubsurfaceOutwardAngleWarning-5016 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1902 of 1912 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.48%25-yellow.svg)


@MikeSpreitzer okay, the second part of the fix is now in the stable-1.9 branch.


testing discontinue of nonCoded drug order if it is exists and not discontinued


+2s the merge invoked by calling code's "git review" (not yet implemented)


Я добавил второй параметр в `getFormattedNowDatePlusDays()`. Таким образом, вызов этого метода стал выглядеть громоздко. Поэтому, решил добавить map. В будущем хочу сделать замену через regexp.


@philmill I see. Thank you!


testing discontinue of nonCoded drug order where the previous order and new order donot match


Nothing, it's empty...


you are looking into outdated code


testing the revise if nonCoded order where orders donot match


testing the revise of existing nonCoded order


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2670](https://teamcity.mesosphere.io/viewLog.html?buildId=65267&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:47


correcting the test with right error


could there be another reason the merge would conflict where it wouldn't be a good idea to commit -a afterward?


adding the nonCoded concept and global property for tests


Hey I want you to know how amazing I think this add on is. The work you do is amazing and I want you to know that what you are doing is making a positive difference. I gave this add on to my cousin who has been battling cancer for 2 years. He has no energy so he mainly stays inside watching all his favorite movies and tv shows on your add on. I just wanted you to know how much happier he is to be using Genesis. I can't thank you enough for the work you do.

P.S.- If there were a way to mark a movie as having Korean/foreign subtitles in the link list, that would help a lot! A lot of older HD links have Korean subtitles and it's hard to find the legit versions. I think people would really love that! Again thank you for all that you do!


r=jedmccaleb


but better for *whom* exactly??


а нужен ли вообще этот класс?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/da228f162b0b006e165d47ef7ca8bd56260857d3


merging MonsieurNicolas/stellar-core/partialValidator = da228f16 into auto


MonsieurNicolas/stellar-core/partialValidator = da228f16 merged ok, testing candidate = 7325ecd1


@MissingHandle wait for it...


Added test to check if the validation passes with out drug and nonCoded when drug is not required


Для тестов нужен


My earlier review had added the word 'possible', because the reason that an app doesn't allow you to check crypto key fingerprints isn't because it's closed source.


A good comment in this case could be: "Expected empty sim bus, as no modules have been attached."


DARY?
http://media3.popsugar-assets.com/files/2014/03/24/845/n/1922283/ba786af3014f9ad7_legendairy.gif.xxxlarge/i/Ultimately-Barney-most-legen-wait-dary-guy-around-well-miss-him-very-much.gif


Let me know if you have any other comments, so that i can avoid sending multiple pushes.


A message here would be useful. I'm not sure what the expected state is...


ユーザーがサインインしていなくてもプロトタイプの詳細ページが見れるようにしました。サインインしていないときのlikeマークは灰色で押してもパスが走らないようにしました。


Also, what should be the value at 0L or 60L? Good job checking the upper boundary (120), but the lower boundary and middle category should be tested as well


Created a bash script that encodes and hosts the given file.


Roughly or literally, PGP is data encryption software; I think its first historical use was in BBSes and Usenet newsgroups before spreading to e-mail. But it's definitely email-centric, so this is probably fine.


biome code? Are you talking about extending biome id range?


方法重载。


у тебя теперь неизменяемые значения в таблице


I'd almost prefer to call OpenPGP an open standard instead of an open protocol, but the words are nearly interchangeable so whatever.


The `either` dependency is still there?

I guess this has been discussed IRL, but we could also have left `orDie` where it was and make `x-eithert` a dependency on `x-optparse`?  It seems safe enough, `x-eithert` should always be pretty light in terms of deps (`transformers` only).  `orDie` seems a bit heavy for `x-eithert`/`x-either`


test


What's meant by the WoT having a single point of failure? Keyservers that propagate signatures are generally centralized, but distributed in pools. So if you're referring to PGP keyservers, that's not a "single" point if we're being literal.


test log


This should be named "testEnableAllMods"

We should also consider adding a test case to check the state when one mod is enabled and one mod is disabled


I have been considering it, however there are limits on how large we can make these things. I am not particularly well knowledge in the area of Biomes, however if you had more information on this, such as the current maximum, I would gladly look into it.


This should be named "testDisableAllMods"


Hello colin, my bad, i was giving the wrong configuration while building, the changes mentioned by you have worked, also I'm closing the issue created by me, sorry for the inconvenience


Yeah you're right, this should be removed.

I think we can't leave `orDie` there because it will have the wrong type (namely, the `EitherT` from the `either` package).


if we remove `either` from `x-optparse`, and add `x-eithert` to the deps, can't we change `orDie` to returnt eh `EitherT` from `x-eithert`?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1023.e392c276c5663def4323a695e00b75d6cdd2c45b](http://build.palaso.org/viewLog.html?buildId=64913&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 694, ignored: 6 Build time: 00:13:29


Currently there are just 256 biome ids available. It's just very very short range (For example if you have ExtrabiomesXL, BiomesOPlenty and Highlands in the pack, the limit will be already overfilled). Would be cool if you can extend them for example to 4096, it's should be enough for all biome mods. If you need more info or maybe testing just ask. I know some people which can help with testing including me :smile:


We could, but then it means that if you upgrade `x-optparse` you have to switch to `x-eithert`.

Which I guess is ok? I don't really have a strong opinion on this one. @markhibberd what are your thoughts about having `x-optsparse` depend on `x-eithert` instead of moving `orDie`.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/87825896


There is some fragmentation in the versions throughout the skins and other files. The correct version is 0.7.0-d.2.


why was the else condition removed? this seems unrelated to the tab width change..


shouldn't this be 3 instead of 8 to preserve the default behavior?


My last point would be that you only recently used the ease with which you can switch to `x-eithert` to justify something else, which I totally agree with.  I don't see why it should stop us here either,

unless there is an issue creating deps between non-core `x` libs.  `x-eithert` should remain quite close to a `core` like lib though i think


I will look into it, but that doesn't sound very encouraging. The skill required to expand an id range within a mod is actually fairly low, so if it were possible, you would likely see it implemented into those mentioned mods already. I believe there may be some limiting factors within the game that prevent expanding the array, for example you can only expand enchantment ids from 256 to 4096, and potion ids from 32 to 256.   I will do some digging and let you know.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2671](https://teamcity.mesosphere.io/viewLog.html?buildId=65268&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:04:36


Дата подставляется в файл https://github.com/JuniorsJava/itevents/blob/issue47-2/restservice/src/test/resources/dbunit/MailFilterUtilTest/MailFilterUtilTest_initial.xml


fixed, thanks


fast-forwarding master to auto = 7325ecd1


BasementDomainNumCounterBug (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1424 of 1424 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.4%25-green.svg)


这里防止走位是什么意思？？


防止走位是什么意思？


pull


I guess to expand those ids you need to ASM into the Minecraft code. Mentioned mods prefer to not ASM into the code. For the potion ids, isn't you actually can extend it to whatever limit? See https://github.com/lumien231/Extended-Potions for example. From the Curseforge page:
"This mod goes further and allows you to make the id limit as high as you want, by default it is increased to 4096 which should really be enough for everyone^^. Obviously if you increase it to something really high you are going to notice a slight increase in memory usage." Anyway, I'll wait for news about biome ids. Hopefully it's possible to extend them.


The commit causes bugs.

1. The new property `headerHeight` is unnecessary because there is an `originalHeight` already.
2. It doesn't change the real height of the header.
3. Adjustment for `contentOffset` and `contentInset` is unnecessary for UITableView.

I'll send a PR later to fix the issue.


101163166-AirLoopHVACNodeNameNotUnique-4407 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (769 of 769 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-18.84%25-green.svg)


JS 模块排除以后没有排除相应的样式文件。

可以使用 http://amazeui.org/customize


issue


no


fix table concat bug


So, I still think it will be relatively straightforward to switch to `x-eithert`, even in spite this. You only need to make some minor changes to imports in the case that you are using `orDie` or `bracketEitherT'`.

I think the major issue we really want to avoid is that you must upgrade to `x-eithert` in order to use `x-optparse` or `x-exception`. This could cause a problem where something way downstream needs `x-optparse` and that forces us to upgrade everything in between.

Unfortunately I had to add a dependency on `text` for `x-eithert` to support `orDie`, which is not ideal, but `text` is still fairly lightweight.


ok no probs fair enough


just a test


a


Building them now @SubJunk


Idiot code...= ="


Excellent!


Зачет


> If you use an escape character, just escape the escape too, and it should not matter which escape character is chosen, no?

@ClemensRobbenhaar that's what I said too (below)

> Are you sure you cannot escape the escape char by doubling it? That's what I did and it worked. So actually I don't think we need to forbid any character in the document name.


As I am sure you already know, this project is not shy to adding ASM hooks. The main issue I have seen from everyone who has commented on the subject, is that it would involve rewriting how Vanilla writes the save format, which is fairly discouraged. The minecraft world format uses a byte to save ids, which structurally limits the game to 256. From there, you would have to make changes to chunks and world providers. A lot of custom dimension mods also limit their worldgen to 256 biome ids. And then there is the issue of getting biomes above that list to actually generate. The best solution to this would be to switch over to a namespace system, similar to how items and blocks currently function. As for potion ids and enchantments, our current implementations make use of basic reflection, which allows us to change minor things in parts of the game. To do virtually unlimited amounts of potion effects, as shown by that project, you need to use a considerable amount of ASM to rewrite packeting and other things.

I am not saying we wont do it, however it is fairly ambitious, and I don't know if our small team is up to the task right now.


can be implemented using `while srclen >= 4` + add condition when 3 bytes rest after that loop.

For amd64, after main loop, you may write smaller loop like `while (bytes_rest >=3) ...` doing not-so-optimal conversion before cases with 1 and 2 bytes rest.

please test it about correctness in that corner cases


Here is a log of me going home and restoring the app multiple times, then exiting:

https://drive.google.com/file/d/0B57Ioy26LWegTjE2VDRPU1dPTjQ/view?usp=sharing


I like this *a lot*.


Ok, got it. Thanks for the detailed response.


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1961 of 1961 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Here is a 2nd log file with the added logging in ```VidExt_GL_SwapBuffers```:

https://drive.google.com/file/d/0B57Ioy26LWegeXltS2lfbnNSUUk/view?usp=sharing


LGTM


PEP 8 (E303)


PEP 8 (E128)


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


tenhle preg je neplatnej, první skupina čísel je ID projektu, to přeleze 3 cifry za pár míst, druhá skupina je ID tokenu, to je teď 5místný a taky brzy přeleze


thanks! they were leftovers from my experiments as you can tell ;)

I don't mind stalkers as long as they improve my code hehe


zašifrovaný tokeny maj vždy dýlku 108 znaků? co když ten token před zašifrováním bude delší?


PEP 8 (E231)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2672](https://teamcity.mesosphere.io/viewLog.html?buildId=65269&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 193 Build time: 00:07:10


commit


I think this may be a better approach (and no need for active low level stuff):

```
def createFreedString(c_pointer):
        if c_pointer is not None:
            char_star = ctypes.c_char_p(c_pointer)
            string = str(char_star)
            free(char_star)
            return string
        else:
            return None

CWrapper.registertype("string", createFreedString)
```
Now you can use string as a return type for alloc_field_init_file
I think the free statement could easily be (without prototyping):
```
ENKF_LIB.free(char_star)
```
Or if you prefer:
```
ENKF_LIB.util_free(char_star)
```


Well, we kind of need to choose something, heh.


co tomu tady v nepovinném parametru forcnout encryptor, co má použít? tady přesně víš, co chceš použít, je to na jenom centralizovaném místě

`$configEncryptor->decrypt($job->getToken()['token']), 'syrup.encryptor');`


lol


Malloc is already a part of ctypes:
```
array_of char = ctypes.c_char * 10
array_of_int = ctypes.c_int * 100
```
And I believe these are automatically garbage collected.


map=[[0,0,0,0,0], #map synatx is first sub-list levcel is a row
     [0,0,0,0,0], #each item in a sub-list level is a cell's state
     [0,0,0,0,0], #0 means a cell is white, 1 means black
     [0,0,0,0,0],
     [0,0,0,0,0]]
ants=[[3,3,"Up"]]
switchwhite={"up":"right", "right":"down", "down":"left", "left":"up"}
switchblack={"up":"left", "left":"down", "down":"right", "right":"up"}
times=8
for l in range(0, int(times)):
    newmap=[] #The next steps map
    for i in range (0,len(ants)):
        print(i)
        if i==len(ants):
            if [map[(ants[i])[0]],map[(ants[i])[1]]]==0:
                (ants[i])[3]=switchwhite((ants[i])[3])
            else:
                (ants[i])[3]=switchblack((ants[i])[3])
            ##DOTHISmap[(ants[i])[0]],map[(ants[i])[1]]
    map=newmap #Update map
for item in map:
    print(item)
print(map)


@tonict : Thank you very much for the bug report! I posted the fix [here](https://github.com/free-jqgrid/jqGrid/commit/25eba794ae03a038d98b524b359af2d9835029fb#diff-5ffa8604b646bea9aa928a8a9891e303L137). Please test the new code and confirm that the new code works correctly in your project. Please continue reporting any error which you could find in the future.


But ultimately Peter realized that, after all his labors, one fatal flaw
remained: his /admin principal did not exist.
On Oct 27, 2015 11:15 PM, "Felix Wong" <notifications@github.com> wrote:
>
> lol
>
> —
> Reply to this email directly or view it on GitHub.


I would really like to see an auto complete functionality for the required keyword.


done


any build instruction?


Your contribution on the ComMotion motor driver is really appreciated. Still to date finding Arduino code on the ComMotion is difficult and the letsmakerobots link leads to a 404 error.

Would appreciate it if you could clarify the following:
1) When uploading the sketch - I noticed that the Vin jumper needs to be open, else the sketch will not load to the Arduino - pls confirm that this needs to be so.
2) Beyond successfully loading the sketch I am unable to get Arduino to talk to the ComMotion board. There is no motor motion from the the ComMotion board. It does go into the default demo scamper mode (tune and all) with three of my four motors spinning as expected (scamper has 3 wheels?). Opening the Serial Port does give the expected lines received e.g. "Starting ComMotion3 ...", "50% forward for 5 sec." etc. Would like to know what the status of the Vin jumper and RST jumper should be - shorted or not or doesn't matter? Would also like to know if the MCU Program Selection Switch should be in which position,  1 or 2 or NC? Anything else I am missing in terms of hardware to get the Arduino to talk to the ComMotion Board?

Thanks very much.


Guess you can also count the lines between the `.` and the selected, which is probably a bit more efficient.


This comment is off by one space. Quite tiny, but still...


Can you explain how this is supposed to work?

The `put` method used to mean: map `expression` to the translated symbol `symbol` so that anywhere `expression` used to appear, it should be replaced with the given symbol. E.g., if computing an aggregation like `sum(a * b)`, a pre-projection step would first compute `x = a * b` and the mapping `a * b => x` would be recorded in the translation map. The aggregation step would rewrite the expression `sum(a * b)` using the translations so that it cam be computed in terms of the output of the pre-projection. Thus, `sum(a * b)`, given `a * b => x`, is translated as `sum(x)`.

I'm confused about the semantics of having an extra `columnReferences` argument in the `put` method.


DRY. Może utwórz mixins, jeśli potrzebujesz operować na czasie?


Is this a spelling mistake? I think this should be "Canola oil"


👈🏼🤖👉🏼 sounds good
On Tue, Oct 27, 2015 at 8:24 PM Spencer Hurst <notifications@github.com>
wrote:

> this is good.
>
> i do think we should remove margin from headings by default and apply them
> specifically where they are needed (eg: body of marketing copy, blog post,
> etc.)
>
> default for "application" usage should be no margin
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pivotal-cf/pivotal-ui/commit/6e71f255e4c7267f428ca199a795057d5c38b934#commitcomment-14029489>
> .
>


It exists now.

On 10/27/2015 11:18 PM, kpengboy wrote:
> But ultimately Peter realized that, after all his labors, one fatal flaw
> remained: his /admin principal did not exist.
> On Oct 27, 2015 11:15 PM, "Felix Wong" <notifications@github.com> wrote:
> >
> > lol
> >
> > —
> > Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ocf/puppet/commit/a7396fa404c3efc9d5de4bc26f9086d2a4194b5b#commitcomment-14030903>.
>


Thanks for pointing this. Can you send a pull request against `master` branch?


建議應該要習慣在綠燈的時候進行重構，避免一些 side effect


ハッシュとは違うので、()の前後のスペースは消してしまいましょう！
どこ参照の元このような記法にしましたか？


just fyi: window.open oder au de angular wrapper $window.open wirded vom browser blockiert, wenns ned imene user-generated event ufgrüefe wird. i dem fall wos du bruchsch passt das, wills hindereme click listener isch (also de user klickt und darum dörf es popup ufga). wenns aber eifach so, irgendwo usm code ufrüefsch, denn wirds gad gstoppt vom popupblocker. just fyi :wink:


Don't mix the independent changes into single GIT commit.


Variable que obtiene el parametro lang desde la barra de busqueda del navegador o desde la operación generada desde angular js


題外話，一般 C# naming convension, _books 應該是給 field 用的，不是給 argument 用的 :)


Since this adds a function rather than breaking the old API, this should not need a major version release (at least, not as far as I am aware), so this could even be a small version release that could be made very quickly (hopefully).


@bbatsov I like working on OSS projects which have very high standards for code quality, and yours is one of them. Please keep up the rigorous code review. I will fix this.


alles klar. thx for the hint


falls mer denn irgendeinisch wieder es ``iframe`` bruched miteme pdf oder bild dinne, dörfsch das also gern wieder ine tue :smiley:  hesch doch so froid dra gha :wink:


Why not simply tune this using SPSA on the framework? That would give you good values within 20K iterations and a lot more efficient..


如果是要 override Equals() 與 GetHashCode(), 建議 Book 可以直接實作 IEqualityComparer<T>，會顯得更加清楚。


Sorry, isn't that what the code already does? I think I'm failing to understand your point.


@SubJunk This is no longer accurate even though zipped logfiles are still not permitted. Log files could be posted as ```.txt``` files though. https://github.com/blog/2061-attach-files-to-comments


ハッシュは`{ key: value }` のようにスペースをあけるのでしたね。

いくつかあるので修正しましょう！


Let's leave serilog out of this, it's a message template


I should probably have squashed the commits before submitting the PR, sorry about that. The SET_PERI stuff here has been superseded with https://github.com/kanflo/esp-open-rtos/commit/a0a712d92444ce5691809c4df173e812b14cdbeb


Hm, there is some logical flaw here I think:

```js
{
  baseUrl: '/todomvc/'
}
```
This currently requires: `href="/todomvc/active"`

But:
```js
{
  onlyHash: true,
  baseUrl: '/todomvc/'
}
```
Only requires: `href="/#/active"`. It should be: `href="/#/todomvc/active"`

That is a logical flaw I think. BaseUrl should mean the same thing for both hash and regular urls. Or we should discourage use of hyperlinks all together, just remove the general onClick hook on it. We can do this since signals are bound to urls.

To be logical baseUrl can only mean two things:

1. Only handle url changes on "this" path. Which requires hrefs to give the full path

2. Make baseUrl the relative path. Hrefs uses the same path as defined in the route definition

I think point 2 is the right way to go if we are HREF focused. The reason is that you use the same URL definition on your hyperlinks as you do in the router. The problem with this approach of course is that you can not do a normal location change to a different path using hyperlinks.

So to summarize, I think we should decide between the following:

1. Discourage use of hyperlinks and remove the "hook" logic from the codebase. Signals are a lot more powerful and expresses intent better in the code anyways

2. We change the "hook" logic of hyperlinks to inject the baseUrl automatically when using a relative url. This allows users to think about hyperlinks as route definitions. `/active` -> `/todomvc/active`. And `/#/active` will work the same way

Okay, those are my thoughts :-)


スペース


lexically it is tottally wrong


I misled you about `WT_UNUSED` in `WT_ASSERT` -- what we actually have in non-diagnostic builds is:

```c
#define WT_ASSERT(session, exp)                                         \
        WT_UNUSED(session)
```

That doesn't help at all with variables used in the expression, it only avoids warnings from functions that take a `session` parameter and only use it for assertions.


101163020-SetpointAirTempWorksOneWay-4627 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (887 of 887 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-24.57%25-green.svg)


P.S.  And I don't think there's much we can do about it -- we don't want to evaluate `exp` in non-diagnostic builds, it could be a call to a function that is `#ifdef`'d out.


TDD的過程，切記一次只新增一個失敗的測試案例，然後接著就要想辦法通過它。

也就是正常的狀況，測試總管最多只會有一個 failed 的 test case，而且那一個就是剛剛新增的。


ファイルの末尾は改行を入れましょう。

gccのコンパイルをするC言語の名残みたいなものです。気になるのであれば調べてみてください。


ここも改行。


Currently "gw" is being set to the address of the network and Neutron rejects this when creating a subnet. So we should either change this or not set the "gw".


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

<?php

$people = array(
   0 => array('id'=>1, 'first_name'=>'John', 'last_name'=>'Smith', 'email'=>'john.smith@hotmail.com'),
   1 => array('id'=>2, 'first_name'=>'Paul', 'last_name'=>'Allen', 'email'=>'paul.allen@microsoft.com'),
   2 =>array('id'=>3, 'first_name'=>'James', 'last_name'=>'Johnston', 'email'=>'james.johnston@gmail.com'),
   3 =>array('id'=>4, 'first_name'=>'Steve', 'last_name'=>'Buscemi', 'email'=>'steve.buscemi@yahoo.com'),
   4 => array('id'=>5, 'first_name'=>'Doug', 'last_name'=>'Simons', 'email'=>'doug.simons@hotmail.com')
);

$i = 0;

foreach ($people as $key)
{

	echo '
';
	echo''.$key['id'].'	';
	echo "".$key['first_name']."	";
	echo "".$key['last_name']."	";
	echo "".$key['email']."	";
	echo "
";
	echo '';
	echo '
<button onclick = "myFunction()">Display Info</button>
';

	$i = $i++;
}

?>


重構的時機點，通常是在所有測試案例都是綠燈的情況下進行重構。
否則開發人員自己還需要去記憶，這次重構有沒有影響到原本運作正常的結果。

這就是為什麼TDD的循環是紅燈，綠燈，重構。而且一次只新增一個失敗的測試案例，就要用最簡單的方式通過它。


改行


You can use the modulus operator to avoid having to reset it at the end - look into that https://en.wikipedia.org/wiki/Modulo_operation

Outside of this, good job :D. Fix the modulo thing and then we'll move on to another one.