Because I wanted to release sooner. Having to balance the BC jetpacks was the only thing holding it back and my time is quite limited these days.


You might want to consider something along the lines of #53 for returning samples.
That way, you still always only return an ERP object, but different inference algorithms can augment their returned marginalized ERPs in different ways (e.g., particle filters can return their constituent particles as well. Will also make merging with and across dev easier.)


For some reason, it appears this change has broken how findElements() works. I now get the following error when the driver is unable to find any elements:

```
WebDriverCurlException: Curl error thrown for http POST to /session/a7aaee84ed92404fbf66824157b13e69/elements with params: {"using":"css selector","value":"div.someclass"}

Operation timed out after 30004 milliseconds with 0 bytes received
```

I'm expecting no error to be thrown and to get back an empty array if the driver times out. I attempted to set the request timeout when creating a new instance of RemoteWebDriver, but that throws back the same error with the adjusted timeout. I'm not entirely positive what's going on, but if I revert this change everything works just fine again. This problem is occurring on both OSX Yosemite with PHP 5.4.33 + cURL 7.37.1, and CentOS 6.2 with PHP 5.4.34 + cURL 7.19.7


I guess before u don anything u must request then u execute or whoever is in charge of executing.
If branches are made, they are there only to search its purpose or to hold base for certain parts of code or people. Lastly, every action has a reason/comment for documenting purposes. it keeps mind of everything u written/deleted.


um "an" is correct...


sorry about the split commit, i just forgot some files in the first one, and it didn't seem important for this purpose.


(copied from above comment)

should probably be fs_notifications to match other files in os/, like pty_process, pipe_process, and uv_helpers.


agreed, this is a quasi-backwards incompatible change if you previously expected the total tax to exclude shipping tax.


##THIS IS A MARKDOWNFILE


`process.release` looks very messy. Why can't we have `process.config.variabes.node_name` or something like this....

What is the _number approach_ you dislike? (didn't understand)


@egilburg Thoughts?


http://www.ecenglish.com/learnenglish/lessons/vowel-sounds-when-use-a-and-an

:^)


TeamCity Akka.NET :: Akka.NET PR Build [Build 115](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=180&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll, D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Remote.Tests\bin\Release\Akka.Rem... Build time: 00:07:52
### Failed tests
```
Akka.Remote.Tests.dll: Akka.Tests.dll: Akka.Tests.Actor.InboxSpec.Inbox_have_a_default_and_custom_timeouts: <no details avaliable>

Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


TeamCity Akka.NET :: Akka.NET PR Build [Build 116](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=181&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Why do we need both?
I mean Suppress and the ReadCommitted?


I figured having the 'rely_to' be the initiator would mean the wording is all still fine.


@gmacdougall We spent a bit more time with this approach. While it does a nice job of retrieving all the records efficiently, the JSON generation grinds the request to a halt. It turns out that Rabl doesn't do so well with eager loading. This is apparently a known issue (https://github.com/nesquena/rabl/issues/558). There's a recent commit to the Rabl project that suggest they might have resolved this. Testing this tomorrow, and I'll report back.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2215](http://teamcity.monogame.net/viewLog.html?buildId=5632&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Issue discovered in non-nesting attributes forms. Currently investigating. This gem really needs tests


不使用.sh文件，而且如果用文件也是execFile方法


这个函数的名字叫main似乎更好


fast-forwarding develop to borshop-integration-466-bugfix/17-gitignore = 5a9df0ce


dont need returns, invoke, map, filter?


@Kagami I think that jaded-brunch works with `jade-ngtemplates-brunch` as well. Any specific reason why it wouldn't?


Why is this change needed?


It's a little creepy that this claims to "extend" the type when it's narrowing the type. Since this is a backwards-incompatible change maybe we should call this `restrict` or something like that.

I know it's goofy to invent the IDL as we go along, but that never stopped me before. ;P


looks like test modification


I will try your patch too, thanks. Could you also Look at https://github.com/TrinityCore/TrinityCore/issues/14310 ?


89812206_radiance_fix (rpg777) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Nit: "a parser"

Also, maybe change "parser" to "implementation" since this spec isn't just for parsers.


30411/Grigoras C. Alexandru Mihai/Assignment 1.txt

It will work because when you have 1 nod in the list, then the head and the
tail are the same. I know what changes i must make, but i did this in a
hurry

În data de luni, 9 martie 2015, Cosma Alex <notifications@github.com> a
scris:

> same as above
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grigorasmihai/DSA-lab/commit/d93cf695095e3ecd581180ee7c7582497330ac54#commitcomment-10109129>
> .
>


Why `-1`?
Isn't possible for two revisions to have the same timestamp?
(e.g. in situations where an edit conflict is likely to happen)


Good evening @burmanm , I overlooked this part when I merged the PR. Is there a reason not to put generated stuff in `${basedir}` instead of `${project.build.directory}` directory?


You can propose them with this approach, but it looks lengthy `process.config.variabes.node_name` (too nested IMO).

> What is the number approach you dislike? (didn't understand)

When node.js will move to v1, it will break all the packages relying on version number.


user = ParkingTickets


Такое оформление списков лучше задавать прицельно.
Если клиент будет вставлять свои списки, то они будут без буллетов.


Получилось выкачать страничку?

Кстати, обычно папки от IDE (например, `.idea`) не коммитят


Thanks :) I like this one too!

for(var child in sprite) {
 ....
}


Looks like the linter is requiring a space after each comma in the ['fun', 'jid', 'user'] list.


Agreed: Not a big deal (if it doesn't make it to the main branch), but it's best to keep "extra" info as an extra field on the ERP object, in order to preserve the marginalization interface.


You need to delete the unrelated commits. This will involve using the
rebase command.  You can find instructions in the textbook folder.

On Mon, Mar 9, 2015 at 3:01 PM, Michael Uy <notifications@github.com> wrote:

> Hi Mike,
>
> What do I have to change to make my pull request acceptable? I left my Vim
> ec grade unchanged, I just realized too late that the file was still not
> updated on my own repo. It is still "-/10" though.
>
> Thanks,
> Michael
>
> On Mon, Mar 9, 2015 at 1:43 PM, Mike Izbicki <notifications@github.com>
> wrote:
>
> > I can't merge your commit with this here :)
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/MichaelUy627/ucr-cs100/commit/e7a64fdc7d97c3d96a56b9d3e7b1ae2e3be07170#commitcomment-10108256
> >
> > .
> >
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/MichaelUy627/ucr-cs100/commit/e7a64fdc7d97c3d96a56b9d3e7b1ae2e3be07170#commitcomment-10109827>
> .
>


search = serve*


Please use `ParsleyUtils.warnOnce`.


fender, you're such a nerd


HEY WAIT. NO. THIS IS WRONG. WRRRRROOOONG.
The constructor UUID is the document UUID so it shout be random since we don't have documents right now. But I *should* be calling `claim.setUser()` and `item.setClaim()`.


Look that I've found another reason for the non admin author blank.
The thing is that those kind of types are recognized like post lists (probably because they fails all the other checks) [looking at the pull request], and the problem is just 1, tc_set_post_list_hooks called too much early (the same problem which affects the custom templates with custom loops ). I've just found that hooking it to get_header makes the author profile and the custom templates work + also the inline style works (which didn't work when hooked to wp_head, in my previous attempts). I suggest you to check this (hook tc_set_post_list_hooks to 'get_header') 'cause will solve also that other issue.


can't even use a and an correctly :(


TeamCity Akka.NET :: Akka.NET PR Build [Build 116](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=181&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Tests\bin\Release\Akka.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHelper.xUnit(FSharpFunc`2 s... Build time: 00:08:50
### Failed tests
```
Akka.Tests.dll: Akka.Tests.Routing.ResizerSpec.DefaultResizer_must_grow_as_needed_under_pressure: <no details avaliable>

```


TeamCity Akka.NET :: Akka.NET PR Build [Build 117](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=182&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Is this really better than importing and instantiating `sun.net.spi.DefaultProxySelector` directly?


If we don't -1, we'll get the same revision!

Revisions take 2-3 seconds to save if you are sitting in the datacenter, so this is only possible in the case of a multi-tab or automated situation.


Добавь документации к методам, особенно к перегруженным операторам.


You'll have to use `expectWarning` in that test.


I know this doesn't help much. I tried to annotate the case-to-case scale/shift values for each text to render.


The constructor UUID isn't the document ID; it's the parent ID. Notice that ds.addUser() doesn't take an argument, because there's no parent document.


@asiekierka is the main BC author? He is working on it, but not very heavily. BC is mainly powered by the community by now. We all are focused on some big projects in the future...


I'm not sure this is Ok, as the user might want to change `options.trimValue` later on...


Не зубдь Нормалайз.


Такое оформление списков лучше задавать прицельно.
Если клиент будет вставлять свои списки, то они будут без буллетов.


@kalashnikovisme изменения же нет, новый объект-декоратор создается


Could this logic be in setter rather than getter? Ideally getter would just return the existing var, and setting do all the mutation


Why are you no tpassing the profiles along too?


Looks like you've misspelled "canola"


mhhh...
This makes Single pass lighting crash actually.
DiffuseSum2 is never declared. With or without color ramp enabled.


> Thanks! BitHub has sent payment of $7.85USD for this commit.

@WhisperBTC what's the deal with these commits? Of the last 5 commits that have payouts:
* [Only one (1) was actually code contributed towards the project.](https://github.com/WhisperSystems/TextSecure-Server/commit/d2dbff173a3826b9d2d648974bfd1aea4b5ca382)
* [One (1) was paid $5.97USD for "Remove /docs. It was obsolete before it was complete".](https://github.com/WhisperSystems/whispersystems.org/commit/556f3845e5050fe30f945f5945b80fd9ccca8c54)
* [One (1) was paid $8.11USD for "Edited post for typos, and made a few small changes to improve flow".](https://github.com/WhisperSystems/whispersystems.org/commit/11e8c2f783a7b7352c66c2f980564e9647297b1b)
* [One (1) was paid $8.02USD for a new blog post "My first git pull request".](https://github.com/WhisperSystems/whispersystems.org/commit/112f4c44709dbf53cf7d6bbcd81a986ca9f1ed97)
* [This one (1) was $7.85USD for "Fix typos in Jessy's blog"](https://github.com/WhisperSystems/whispersystems.org/commit/ed2fc249a0b97fe7a7d83fce3188f039d22e9cfa)

I was really excited about working on this project with the BitHub experiment, but commits like this getting $7.85USD for "fixing typos" in a blog seem like they're gaming the system...


Ah, you are right of course. I need to learn a bit more a bit `gyp`/`node-gyp`/`pangyp` to contribute.

Do I understand it correctly: when http://github.com/TooTallNate/node-gyp accepts TooTallNate/node-gyp#564 there is no longer need for http://github.com/rvagg/pangyp ?


FYI, there is also a reference to your twitter earlier in the file @markhibberd


Oh, wait, yeah. You're right.


Maybe we should. Solar would probably disagree but that's not necessarily a reason to keep them.


created an issue
https://github.com/jMonkeyEngine/jmonkeyengine/issues/239


Fixed in my PR #43.


ds.addUser() doesn't have an argument because ds manages the documents behind the scenes, doesn't it?


Ok, thanks. Someone asks me a question, but I don't have the time or real desire to participate - so figured it was best to just make it easier for people looking for help.


Confirming request


Yes pangyp is just a transient package. You can actually propose any idea to io.js even without learning that stuff. There are many enthusiasts who might be willing to implement it. :)


Yeah, you're right.
On Mar 9, 2015 4:23 PM, "Braedy" <notifications@github.com> wrote:

> ds.addUser() doesn't have an argument because ds manages the documents
> behind the scenes, doesn't it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CMPUT301W15T07/TravelTracker/commit/1d8bea3af3f52aac5196ee60be8ffd9b64fe4949#commitcomment-10110141>
> .
>


Ok, there's another issue. For whatever reason certain keys have been deleted from the master. Attempting to restore them doesn't seem to work. A support ticket has been lodged with transifex, so I'll wait until they get back to me.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2215](http://teamcity.monogame.net/viewLog.html?buildId=5632&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:13:56


postal_code = Postal_code.find(33771)


@bajtos This test passes if `block: false` is added to the created account.


1297-ExteriorLightsSchedule (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2159 of 2183 tests passed)

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


@mowchan  made download button pop more


exactly has exaactly one a ;-)


I'm a little worried about this. [Create pathspec](https://libgit2.github.com/libgit2/#HEAD/group/pathspec/git_pathspec_new) doesn't take in a string but an array of strings.

Can we add a test to confirm that this works?


@bajtos
`act.block` is undefined here, even if you call `related` with `refresh=false`.  Seems to be a buggy test.


Don't need quotes, I think


TeamCity Akka.NET :: Akka.NET PR Build [Build 117](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=182&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 823, ignored: 47 Build time: 00:09:29


I'll make that change in `master`.


Такое оформление списков лучше задавать прицельно.
Если клиент будет вставлять свои списки, то они будут без буллетов.


Yes! looks perfect
Really good catch Rocco.


It makes reviewing changes a lot easier if you submit white space changes in a separate commit.


Tu powinno być jeszcze super(true);
Wiesz czemu? ;>


Perhaps even make the distinction explicit - have an `extra` field for an ERP that can itself have multiple subfields (normalization constant, samples, etc.). This makes little to no programmatic difference, but might be saner semantically.


Для клинера лучше создать отдельный класс


well, we do have the filter for that

is: arc, etc


@kalashnikovisme  AttributeAccess.find_or_initialize_by( params[:attribute_access].except(:access) )


agaain


this commit really deserved a travis build ...


Ska


aand again


I don't eat dairy anymore so now I mostly just dream of pizza.


I'm fine with that. Probably a good idea tbh. I'll do that now


Buen aporte ;)


Same for tc_set_grid_hooks no ? what do you think ?


Made a comment here https://github.com/nodegit/nodegit/commit/0180b9f2337c4f403759a89f03fba9734a0a7c14#commitcomment-10110182


@s0undt3ch I think opensuse jobs are still disabled on jenkins.saltstack.com so could you start a run and see if npm and bower are satisfied with the repo?


to exploits


There are more hereaabouts (I'm running out of words to use).


I think so, but still keep bbpress compatibilty for tc_set_grid_hooks, 'cause there the problem is with the pre_get_posts.
Though the same (hook to get_header) should be done with headings : https://gist.github.com/eri-trabiccolo/14ade3c986e9f3086843#file-custom_post_lists_comp-php-L66
I used here wp_head, but as I said before, get_header should be better, if you want I can test it, give me 2 minutes.
Last thing, when you will use the get_header hook for the bb author, you should remove the compatibility which forces the single post content, otherwise the author profile will be shown twice.


Should this be onChange, not onClick?


Well, `extend` is extending interface of `AssignmentExpression` here, and it's not meant to be recursive (applied to fields), so I guess it's fine to override / narrow field types inside.


class Book < ActiveRecord::Migration
  def change
    create_table :new_book |t|
      t.string :title
      t.string :author
      t.date :publication_year
      t.integer :pages

      t.timestamps null: false
  end
end


In the new version of plotly-test-table that I wrote today, it writes data/*/*.log files for every download attempt from the plotly server.

the new ./git-add.sh script adds the log files along with the png and html files. https://github.com/ropensci/plotly-test-table/blob/gh-pages/git-add.sh


Good suggestion.


Oh, yeah, I think it should be.


Great idea!


personally I think this function would make more sense in `luigi.tools`. I prefer to separate executables from packages (feel free to disagree though, happy to discuss)


showing twice issue : yes I have already fixed that.
I'm still thinking that wp_head is still a good candidate tough. With a very high priority like 9999.
Because in the future, I plan to allow user to get rid of the header in some contexts. It will never happen with wp_head.
What was the issue with wp_head exactly ?


Don't remove this file because people may depend on it. Keep something that imports the new location (and optionally throws a deprecation warning)


see comment below


all tests pass:
success: http://build.servo.org/builders/linux1/builds/976
success: http://build.servo.org/builders/linux2/builds/975
success: http://build.servo.org/builders/android/builds/1011
success: http://build.servo.org/builders/mac1/builds/1191
success: http://build.servo.org/builders/mac2/builds/1192
success: http://build.servo.org/builders/gonk/builds/162


Thanks guys, fixed this in c6bfd5d01ce52d794dde893cc7410d0814a6fd5e


Not sure you understand by "their branch is on the right", isn't web-tutorial the branch we're all working on? Do they need to know this?


tah nyaa udah :)


heh you are missing `:`


ditto


this (copied from the pull request about the content/excerpt filter):
The problem using "wp_head" hook is about tc_user_options_style filter which is "fired" on wp_add_inline_style (in wp_enqueue_scripts hook callback) so ... if "we" want to allow users to use custom templates in which force stuff which requires user options style, then "we" have to change this logic and don't use wp_add_inline_style, but write an inline css hooked to wp_head with a lower priority .. :/


oh


sorry i'm dumb


You may want to name some of these sub-expressions to make it clearer


Ticket at https://github.com/silverstripe/silverstripe-framework/issues/3992


Snap, of course. Thanks, will fix. (We can't change idx there, as we rely on it later on, but we can access xt_reg->idx or otherwise rearrange the function a bit.)

Regarding EPS[DEBUGLEVEL], I started implementing this early on and then got confused by the fact 'PS' was directly accessible and ended up leaving it for later. Do I understand right that when the CPU halts and enters OCDHaltMode, PS is saved to EPS[DEBUGLEVEL]. Then while halted, 'PS' holds a value that's specific to the debug state only. Then when resuming, PS is restored from EPS[DEBUGLEVEL]. Is that right?


'Unpermitted parameters: first_name' means you need to override the devise strong params implementation inside your ApplicationController.


fast-forwarding master to auto = 2df4dd9e


Should move up with the first set of assertions.


do we want this to be public / expose it? ... I think it is ok just want everyone to think about it


1297-ExteriorLightsSchedule (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@brianburridge
I have provided new answers to the questions you marked wrong.  My new answers are below your original comments.


thats good :)


nit: spacing should be:

```
int cinderLogLevelToSysLogLevel( Level cinderLogLevel )
```

(note spaces within function argument body)

Other thing that might be nice, is to add a comment noting why we made the choice to route both `LEVEL_INFO` and `LEVEL_VERBOSE` to syslog's `LOG_NOTICE`, since this might not be obvious to future developers wishing to work on this code.


This throw the following error after checking out last master version of admin-bundle:

```
PHP Fatal error:  Declaration of Sonata\AdminBundle\Admin\Admin::validate() must be compatible with Sonata\AdminBundle\Admin\AdminInterface::validate(Sonata\AdminBundle\Validator\ErrorElement $errorElement, $object) in /projects/fork/sonata-sandbox/vendor/sonata-project/admin-bundle/Admin/Admin.php on line 48
PHP Stack trace:
PHP   1. {main}() /projects/fork/sonata-sandbox/app/console:0
PHP   2. Symfony\Component\Console\Application->run() /projects/fork/sonata-sandbox/app/console:34
PHP   3. Symfony\Bundle\FrameworkBundle\Console\Application->doRun() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:126
PHP   4. Symfony\Component\HttpKernel\Kernel->boot() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Console/Application.php:70
PHP   5. Symfony\Component\HttpKernel\Kernel->initializeContainer() /projects/fork/sonata-sandbox/app/bootstrap.php.cache:2344
PHP   6. Symfony\Component\DependencyInjection\ContainerBuilder->compile() /projects/fork/sonata-sandbox/app/bootstrap.php.cache:2565
PHP   7. Symfony\Component\DependencyInjection\Compiler\Compiler->compile() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/ContainerBuilder.php:614
PHP   8. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->process() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/Compiler/Compiler.php:117
PHP   9. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->processDefinition() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:64
PHP  10. class_exists() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:110
PHP  11. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:110
PHP  12. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:0
PHP  13. require() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151
PHP  14. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:17
PHP  15. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:0
PHP  16. require() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151
PHP  17. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:23
PHP  18. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:0
PHP  19. require() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151

Fatal error: Declaration of Sonata\AdminBundle\Admin\Admin::validate() must be compatible with Sonata\AdminBundle\Admin\AdminInterface::validate(Sonata\AdminBundle\Validator\ErrorElement $errorElement, $object) in /projects/fork/sonata-sandbox/vendor/sonata-project/admin-bundle/Admin/Admin.php on line 48

Call Stack:
    0.0003     240984   1. {main}() /projects/fork/sonata-sandbox/app/console:0
    0.0315    3276216   2. Symfony\Component\Console\Application->run() /projects/fork/sonata-sandbox/app/console:34
    0.0345    3479544   3. Symfony\Bundle\FrameworkBundle\Console\Application->doRun() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Console/Application.php:126
    0.0345    3479768   4. Symfony\Component\HttpKernel\Kernel->boot() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Bundle/FrameworkBundle/Console/Application.php:70
    0.0455    4190536   5. Symfony\Component\HttpKernel\Kernel->initializeContainer() /projects/fork/sonata-sandbox/app/bootstrap.php.cache:2344
    0.2845   11548120   6. Symfony\Component\DependencyInjection\ContainerBuilder->compile() /projects/fork/sonata-sandbox/app/bootstrap.php.cache:2565
    0.2870   11603632   7. Symfony\Component\DependencyInjection\Compiler\Compiler->compile() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/ContainerBuilder.php:614
    7.4842   33911384   8. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->process() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/Compiler/Compiler.php:117
    7.6325   42803768   9. JMS\AopBundle\DependencyInjection\Compiler\PointcutMatchingPass->processDefinition() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:64
    7.6325   42803816  10. class_exists() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:110
    7.6325   42804144  11. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:110
    7.6325   42804208  12. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/jms/aop-bundle/JMS/AopBundle/DependencyInjection/Compiler/PointcutMatchingPass.php:0
    7.6326   42807104  13. require('/projects/fork/sonata-sandbox/vendor/sonata-project/user-bundle/Admin/Entity/UserAdmin.php') /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151
    7.6326   42807416  14. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:17
    7.6326   42807472  15. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:0
    7.6332   42909464  16. require('/projects/fork/sonata-sandbox/vendor/sonata-project/user-bundle/Admin/Model/UserAdmin.php') /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151
    7.6332   42909760  17. spl_autoload_call() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:23
    7.6332   42909808  18. Symfony\Component\Debug\DebugClassLoader->loadClass() /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:0
    7.6363   43438392  19. require('/projects/fork/sonata-sandbox/vendor/sonata-project/admin-bundle/Admin/Admin.php') /projects/fork/sonata-sandbox/vendor/symfony/symfony/src/Symfony/Component/Debug/DebugClassLoader.php:151
```

I think is AdminInterface fault: https://github.com/sonata-project/SonataAdminBundle/blob/master/Admin/AdminInterface.php#L21

Please see PR #2793


Hopefully this won't affect the mobile app. The timezone is currently +0000 so the mobile app probably ignores the timezone anyway. If it causes problems for the app, skip this commit.


spacing in argument body (`cinderLogLevelToSysLogLevel( level );`)


Any wishes on its formatting? Two if stamements or should I keep it as it is? Or should I make a method out of it :P


spacing (should be `mAppend( appendToExisting )`)


+1

On Mon, Mar 9, 2015, at 03:27 PM, Adam Becker wrote:
> I'll make that change in `master`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/dobtco/knowledge_base/commit/1e6078d8c46df1be5f6acfe5a42c2e83eabaaa0b#commitcomment-10110193


@patriciogonzalezvivo I eliminated some redundancy here. There is some downside since the pattern becomes a little more abstract, but it saves 100 lines of code.


psst...

```
(recur (f result (first remaining)) (rest remaining)))))
```


ditto for spacing


Being a SASS noob, I'm not sure why this token changed when this was generated nor if it matters, just leaving this comment here as a reminder to check this on deployment.


only poll if it has active tasks?


ditto for spacing here and in the syslog call below too.


The system is functioning as intended. All PR's are equal in the eyes of Bithub.


> Even to places that weren't really changes I made.

Apologies @vanzin. I probably should have limited my comments to code added by your patch (as opposed to just edited). :disappointed:

Nonetheless, thank you for cleaning up this stuff.


I belive you are using one of the outdated sun jogl 2 pre-releases.
please download the latest maintained JOGL from http://jogamp.org

isGL3core got added back in 16 jul 2013
http://jogamp.org/git/?p=jogl.git;a=commit;h=5dafc958385da595160dc0d3c843c8253334c3c5

JogAmp JOGL 2.2.4 contains isGL3core()
http://jogamp.org/deployment/v2.2.4/javadoc/jogl/javadoc/javax/media/opengl/GLBase.html#isGL3core%28%29

JogAmp JOGL 2.3.0 (with javax.media.opengl moved to com.jogamp.opengl) also contains isGL3core()
http://jogamp.org/deployment/archive/master/gluegen_846-joal_576-jogl_1374-jocl_1034-signed/javadoc/jogl/javadoc/com/jogamp/opengl/GLBase.html#isGL3core%28%29


All of the above reasoning totally makes sense. I'm worried about the overhead of JSON.stringify to create the histogram table keys on every ERP call (for me, this amounts to serializing giant mesh objects and is totally meaningless), but I'll profile it before jumping to any conclusions.


This is actually because OpenTSDB is returning an error with a Status Code of 400 and the error in the body.

The original variable "error" is null. I really should probably use a different variable here and perhaps add a comment to clarify for the future so this doesn't get "fixed".


@jasonsturges Interesting, very interesting. I'll make use of this once I create a first release :)


Are these collision values equal?


This is actually the case where an error occurred with the request.


black? -bolder sounds like a better name.


Not sure that we should check for state classes.


[nitpick]
```scala
if (playerAssessments.exists(predicate)) Some(avg) else None
````
or even:
```scala
playerAssessments.exists(predicate) option avg
```
the former is standard scala, the latter is a lichessism.


Yeah so it looks like you need to pass in a variable to navigator.geolocation.getCurrentPosition.


spacing


Just do a pull request and I'll merge. Cheers and thanks!


spacing x3


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-MacOS-10.9-clang: Tests Failed

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


`auto` is fine in this context, it's only in `for each` that there can be problems and then it's only in certain more complex scenarios.


[nitpick] are you sure you need to typehint `a`? wouldn't `sfAvgGiven(_.blurs > 70)` work?


again, get from configuration file.


state class


If I remember properly googleapis do not send proper caching headers making fonts blink or something. we should move that in house? can you check if a request is made for that font at every page load?


`<b>` is so 2005. Please use `<strong>`.


Already mentioned this in a previous commit, but the indentation is messed up, making your code hard to read !
Your editor must be inserting space characters instead of tab character when tabbing.


i think we might be deprecating this. finding that it's harder to type out that variable name than the number. maybe we should convert this to some sort of dictionary in the html for reference?


requires to use os.path.join() I think.


When I call hintStaticallyElided, I get no "$type":

```
{"name":"foo","type":"jar","extension":"jar","classifier":null,"configurations":[],"url":null,"extraAttributes":{}}
```

That's not what I want.


@gyst This merge has deactivated the display of preview images by @reinhardt which I merged shortly before. I am not familiar enough to understand what needs to be adapted but I both merges need to get consolidated. I can confirm that the image preview still worked with 4cd2e0d1299a3b8d8f95ac9c8b85ef550d412a17


Hopefully this won't affect the mobile app. The timezone is currently +0000 so the mobile app probably ignores the timezone anyway. If it causes problems for the app, skip this commit.


But I see that this is only covered from the tests coming later... I need to make an extra test for this case


use complete name: computeOption, no cOptions


`i => i` == `identity`

`a.fold(false){i => i}` == `~a` (lichessism)


The same binary may exist in several `bin\<project name>\Debug` directories.  That is actually preferential though.  Putting them all in the same directory causes MSBuild headaches because it often fights with itself writing files to the same location.  Additionally if the tools end up depending on diff versions of the same assembly then outputting to the same path will result in random runtime errors.


79453838-AutomateSomeDocumentation (Myoldmopar) - 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-68.81%25-red.svg)


kdate pillo amigo


I can't find the moveToExileZone() method. Did you mean moveToExileZoneWithInfo()?


Which version of PHP are you using?

I suspect PHP 5.3 doesn't like the assignment within the call to empty(), It might be OK with some more brackets or moving the assignment out to the line above.


GridViewBugFixes (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This commit shall be reverted and feature branch created. Also missing issue describing this commit. Or pull request with some description.


isn't normal the default value? I feel like we should force a height / line-height here because browsers might have different sizes for buttons.


@bors: retry


if the background color and border color are both blue, then can we remove the border?


I think this is a good thing. It allows you to specify that no views in this region should get destroyed when they are swapped out, I think the use case shown in the docs would be fairly common.


Good to know.  I just wanted to confirm because we have been outputting everything to bin\ up to this point when using build.cmd (even before the build.props change).  Your change makes a lot of sense though.


Hey,

It's solved!

The .getcurrentposition() required a callback function as a param -- needed
to set self=this to stay within scope (not sure if that's the right
verbiage) and then pass this.storePosition.

Thanks again!
Edna

I'm moving on to storing a second marker using the location coordinates ---
then drawing a route between between.

Woooo

On Monday, March 9, 2015, KaboomFox <notifications@github.com> wrote:

> Yeah so it looks like you need to pass in a variable to
> navigator.geolocation.getCurrentPosition.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ednacao/SecretSpaces/commit/38fda3ee3837544d130093008439c12ba89c99e4#commitcomment-10110437>
> .
>


An unbreakable space, might break the event


79453838-AutomateSomeDocumentation (Myoldmopar) - 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)


We were speaking of design patterns, right? ;)


Agora tudo "funfou" que é uma beleza.
Funcionou como um relógio
 :+1:


saw approval from saneyuki
at https://github.com/aweinstock314/servo/commit/7803f2b216f736ac703e55010bfbd70e4dfc0cad


merging aweinstock314/servo/master = 7803f2b2 into auto


aweinstock314/servo/master = 7803f2b2 merged ok, testing candidate = 8739015c


GridViewBugFixes (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2092 of 2184 tests passed)

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


Much oddness


@kamcio96 Masz rację, ten event powinien asynchroniczny ;)


All those should be added to the styleguide html.


Actually, I haven't tested this yet, but maybe we should also update build.cmd not to set `$(OutDir)` explicitly


why are those two images part of the styleguide?


I was already aware of that extra space on [/giveaways/new](http://www.steamgifts.com/giveaways/new) when I made the last comment. What I meant to tell you is that the space goes away after you open the datepicker, as demonstrated [here](http://imgur.com/a/Lf4hH). `.ui-datepicker { margin-top: 5px }` (in the website stylesheet) is the actual problem and, since it appears to bother you, I'll fix it on the next update.

Btw, sorry for the late response xD


The purpose of this method is to fetch the annotations from the symbol. Annotations carry the information on how to reify the code.

Maybe we should remove the comment from the PR.


check


The first configuration looks correct. Based on the error, I'm guessing you're loading it via `pannellum.htm?tour=config.json`, but that syntax is only for tours; use `pannellum.htm?config=config.json` instead.


:pizza:


I documented `stream_await`, `curl_multi_await`, and `HH\Asio\curl_exec`...

I think these `<function>` tags with namespace //might// work as-is now. But we should accept this pull request and see :)


Will need to delete the existing muhammad culture (central_semitic)


aweinstock314/servo/master = 7803f2b2 merged ok, testing candidate = 083752d8


you should use the emoji in your tests.


if it is zero-rated, do we still show message?


Does not seem to work.
Neither this nor the commit:  galaxyfreak/device-sony-yukon@8156b05
I tried with timekeep instead of time_daemon and with "mkdir /data/time/ 0700 system system" removed as in the commit above.
And, it did not persist.


Yeah, this is something I wanted to do as well. Since activate() is called each time you leave the previous mode, connection would be done each time.


PR?

On Mon, Mar 9, 2015 at 3:57 PM, Visnu Pitiyanuvath <notifications@github.com
> wrote:

> you should use the emoji in your tests.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/huned/better-initialize/commit/1cd3c0adbac72fd91b12baab524e4addda0f1060#commitcomment-10110610>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2216](http://teamcity.monogame.net/viewLog.html?buildId=5635&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


You gave up?! Cannot believe it!


[nitpick] use `fuccess` instead of `Future` to avoid unusual context switches


I compared the initial commits with all the tags from the freescale repository.

The nearest it matches is tag rel_imx_3.0.35_4.0.0 (54750ff96a725ed78c057bc386d0f210c34bc53c)

    $ git diff rel_imx_3.0.35_4.0.0..5d7c0ac5748e357084b13df7325e2a331c6fdde8 --shortstat
     3820 files changed, 48146 insertions(+), 566 deletions(-)

Seems that most changes from tag rel_imx_3.0.35_4.0.0 to the initial commit of this repository are not needed (removing of .gitignore files or including binary built products on the repo :\ ).

This is patch for the initial commit of this repo http://sprunge.us/HijB reapplied on top of freescale kernel repository tag rel_imx_3.0.35_4.0.0 after removing that unneded changes.

    $ curl -s http://sprunge.us/HijB|diffstat|tail -1
     51 files changed, 13840 insertions(+), 14 deletions(-)

Also openned issue #13 to encourage udoo developers to properly use the kernel git repository.


I think you can:
```
import reflect.runtime.universe._
object A {object C}
showRaw(reify(A.C))showRaw(reify(A.C))
```


saw approval from saneyuki
at https://github.com/aweinstock314/servo/commit/7803f2b216f736ac703e55010bfbd70e4dfc0cad


merging aweinstock314/servo/master = 7803f2b2 into auto


aweinstock314/servo/master = 7803f2b2 merged ok, testing candidate = f4a36272


mostly because Frackend routes '/images' to this styleguide repo. it looks like the intention was to not have the retail project to not serve any images directly. shall we change that?

```
app.use(brackend.static('app/lib/styleguide/styles/images', '/images'));


I'll preface this with the obligatory warning to "Ignore if this is a naive comment", but is this supposed to be "PRIMARY_KEY_COLUMNS = {" rather than "PRIMAY_KEY_COLUMNS = {"

If so the typo is carried forward in at least three lines in this file.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1146 of 1146 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)


If these functions (filterContentByIncluded/Excluded) are only used for the "show only content which has been included/excluded" filters, then you can remove the 'groups' key from this pick.


This is different in future commits.


REPL says it can. It would be great to fix this!


Congrats!


Yup. border is removed in the last commit of this pull request.


Shouldn't we have a nicer message here?


#96 and #98.


Well, Ogre::Real can be a double on Windows, right?


I have another idea, but I think I will wait this morning to implement it (based on subprocesses), I reverted commits to avoid troubles


Could you elaborate on this?


do this instead: /^\s*can (i|we) pong\s*\?*\s*$/


Interesting change. I'll keep that one in mind.


Thanks @springmeyer, I’ve implemented this in f56ee8d0 :)


This change has currently broken the format bar @zgryw.

When testing a simple text block it will work fine, however if you have a page that scrolls slightly, it's positioning starts to calculate incorrectly. The more it scrolls, the further away it appears from the selection.

The issue lies in how coords.top is calculated. The values scrollTop and topOffset are always zero in the project example as for this.el.parentNode (document.body).


Line ending missing here.


I managed to re-insert the hyphens on this attempt and still got it about 50 lines shorter.


Both `Client` and `Server` record implement these methods the same way: both pass through to an (untyped!) JSON blob.  Is there a reason to keep these separate?


This not set to permissive .... This change the policy from old app to server for new cmhw


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Have you tried reading the license?

>1. You may copy and distribute verbatim copies of the Program's
source code as you receive it, in any medium, **provided that you
conspicuously and appropriately publish on each copy an appropriate
copyright notice** and disclaimer of warranty;


Suggest naming it add_kflow


Of course, it should be separate from the macro it self. You can look in scala-yinyang how [`LanguageVirtualization`](https://github.com/scala-yinyang/scala-yinyang/blob/master/components/core/src/transformers/LanguageVirtualization.scala) is done.


Shouldn't the culture be bedouin_arabic ? Otherwise he'll get foreigner malus with everyone !
The occlusion can be applied via set_graphical_culture = muhammad_culture


I think we can work on the rest tomorrow. Looks good!


Historically we differentiated failure (HTTP response) from error (local exception).  Is this an oversight or an intentional change?


There is now equivalent functionality for this in the upstream repo.


multiline strings with substitutions!
```scala
s"""[AUTOREPORT]
Cheating Games: ${cheatingSum}
Likely..."""
```


@graemerocher FYI, I made this change in grails-core
https://github.com/grails/grails-core/commit/a3f29790
and revised it with https://github.com/grails/grails-core/commit/8282ada9 so that grails.env system property will be always followed .
I hope that's the correct way to do it.

grails3-functional-tests were failing locally without making that change.


Because otherwise I get complaints from FunctionGraph that the variable is in use in another graph.


:clap:


Sure.

—
Sent from Mailbox

On Mon, Mar 9, 2015 at 3:20 PM, Eugene Gilburg <notifications@github.com>
wrote:

> Could this logic be in setter rather than getter? Ideally getter would just return the existing var, and setting do all the mutation
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/steakknife/rails/commit/b1332a1b9df2db80375e34a0899340409e0ecb41#commitcomment-10110098


due to


grep didn't find any other call sites. maybe rename it to `get_kwargs` and move the `search query` handling into it?


Oh, I guess I didn't test it correctly. Could you create an issue for that? Will try to fix it as a first thing.


whops sorry for overwriting and woohoo! \o/


Suggest naming it delete_kflow


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2216](http://teamcity.monogame.net/viewLog.html?buildId=5635&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:14:03


These 2 images were moved to sub-folder gfx/traits/WtWSMS/ to avoid confusion between WtWSMS and VIET stuff.


GridViewBugFixes (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2165 of 2184 tests passed)

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


Duplicate with WtWSMS_portraits_muhammad.gfx ?


Nice !!


this is the same as the one in `twitter`. maybe just reuse that one?


looks good


Observe that this is `get𐐀`, not `get𐐨` (which would be `…\udc28`)


Duplicated with WtWSMS_settings.csv ?


Missing period at the end.


+1, looks good!


not high priority...but ideally we should parameterize `API_SEARCH_URL` on `count` like the other API URLs, and only use 100 when we're searching for mentions.


remove numPlayers(numPlayers) for initialization list and initialize it in the body of the constructor. Right now in the body of the constructor we check to see if the input is valid, but when we do the assignment to 2 or 5 players, it cannot because numPlayers is a constant and has already been set


test


I'm having trouble trying to find the request for the Roboto font in chrome devtools? Hmm...


At this part of the flow — the record delegate — there's no real local exception that can occur. So this is just "did we get an IOException" versus "we got an unexpected HTTP response". Both are failures, really.


No good reason


You'll need to basically rename 'excluded_ids' to 'included_ids' in this case, as the items we want to get here are the ones which are excluded in the query we're building.


Thanks for catching this!
Can you do this before I merge:
1) pull the code at line 395 into a utility function and use that instead to output true or false.
2) probably remove the check against kCSharp, since a future language is more likely to support these keywords than not.
3) ensure that your code uses 2 spaces for an indent.


Code Review 13: teacher test

this is a test from teachers panel

Hey, could someone look through this and give me some feedback conserning this?
Sincerely @hfurubotten

---------
@hfurubotten, follow this tread for feedback.


Req. ACID.
http://paste2.org/kg62p1wI


yay thanks! great addition! can't wait to see woodwind and others use this.

...just for my own memory, we should probably also update:
* file and method docstrings (`get_activities_response()`, activitystreams.py file, etc)
* web UI
* add these links to the readme, probably where we say we implement the opensocial API:
  http://www.opensearch.org/Specifications/OpenSearch/1.1#OpenSearch_URL_template_syntax
  http://opensocial.github.io/spec/2.5.1/Core-Container.xml#rfc.section.11.2
  http://opensocial.github.io/spec/2.5.1/Core-Gadget.xml#OpenSearch

you're welcome to either grab those or file issues and i can get to them.

thanks again!


This interface seems incomplete.  I feel like this should take a timestamp.


```
             pos = getPosition(msg);
             break;
```
no got error?


True, will change that.


This is my first repository


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1143 of 1143 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)


Nice work, please see my review comments !
In the future don't hesitate to split huge commits in smaller chunks (Islam / migration / ...) for easier review :smile:


Be explicit: for each record, one of `onSuccess` or `onConflict` is called.  How frequently might the others be called?


Awesome.


I've changed .single-event-tile to .show-header, as it seems most show pages will share this header


Fixed


Oh, I see now that Cursor.autoscroll is a private API... 8-(


Esti que ça arrivera jamais...


Oops, I forgot to remove this. Todo: Delete this line.


This is what I'm seeing currently:
![screenshot 2015-03-09 16 23 39](https://cloud.githubusercontent.com/assets/422381/6566588/be2c0e96-c678-11e4-99a3-7e27bb1d8be6.png)


Why not just make methods derive from OdcmType?


Nice work - good to be consistent


that is cool


Don't have a separate `initImpl()`. Keep it simple, one `init()` method is enough


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (604 of 604 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-2.91%25-green.svg)


LGTM


Urgh, hand-mapped?


```scala
val sfAvgLowVar = sfAvgGiven(a => a.mtSd.toDouble / a.mtAvg < 0.5)
```


Don't use an inline synchronized block here. Synchronize the method. Much simpler and the runtime cost is a single synchronized boolean check.


Don't double check - if you follow comment above then this is no longer necessary.


should be canola oil


Newlines at end file, please!  You can set up your editor so that it automatically inserts one every time you save if there isn't one already present.


Be clear that this is called once per record.


See issue description at matplotlib/matplotlib#4207.


Make the method simpler by exiting if already initialized.


1297-ExteriorLightsSchedule (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Thank you :)


Isn't this completely not thread safe? It is probably not a good thing to use in a libary that can be linked in other applications.

May I suggest this instead?

```C
    double sass_atof(const char *str)
    {
      char separator = *(localeconv()->decimal_point);
      if(separator != '.'){
        // The current locale specifies another
        // separator. convert the separator to the
        // one understood by the locale if needed
        const char *found = strchr(str, '.');
        if(found != NULL){
          // substitution is required. perform the substitution on a copy
          // of the string. This is slower but it is thread safe.
          char *copy = strdup(str);
          if(copy == NULL){
            fprintf(stderr, "Out of memory.\n");
            exit(EXIT_FAILURE);
          }
          *(copy + (found - str)) = separator;
          double res = atof(copy);
          free(copy);
          return res;
        }
      }

      return atof(str);
    }
```


That felt like a stretch to me. I had a hard time convincing myself that a Method is a Type. Your call, though. Happy to make the change...


Use something like `dotenv-rails` or Rails' built-in `secrets.yml` feature to manage stuff like this.


Read the [Configuration Rails Applications](http://guides.rubyonrails.org/configuring.html) guide for how to handle configuration stuff like this.  The [Twilio blog](https://www.twilio.com/blog/2014/10/twilio-on-rails-part-2-rails-4-app-sending-sms-mms.html) has a good, Twilio-specific example.

You can (and should) use something like `dotenv-rails` or Rails' built-in `config/secrets.yml` feature for this.


What can I say, you can't improve on perfection


looks like `parent != null` should be checked for this case too


For consideration, elemental triangle swatches in the corner.
![screenshot 2015-03-09 16 34 27](https://cloud.githubusercontent.com/assets/422381/6566698/3bf9cd62-c67a-11e4-9a99-d10dc5facd8c.png)


+   name: Greg Walker
+   image: https://pbs.twimg.com/profile_images/530080518449020928/XBmB6_Vn.jpeg
+   twitter: gregaloha
+   github_username: gregaloha


@kalashnikovisme find это не find_by, грохнется. и попробуй всё же
AttributeAccess.find_or_initialize_by( params[:attribute_access].except(:access) )


Ohhhhh, never knew about `git grep`, thanks.

Ok, seeing your example I definitely agree. Would the same kind of rules apply to single line comments?

````js
// HACK: We're using a magic number here.
// We need to figure out how to get to this number instead of just using it.
````

I know it's strange but would it be good for consistency?

Also. I'd need to adjust the script a bit to account for multiple notes per comment. Right now your example would result in:

````js
TODOs:
* Clean up code. This is ugly, Things are all over the place. Check the following coding standards. Also the for loop has a bug in it. We need to figure out a better approach to this logic. TODO: Add full name property to the Author object.
````

So, I'm going to take a crack at that tomorrow or something.


Sorry about that. I always forget I'm supposed to use `make publish` which compiles the sources before publishing. Remove the makefile and switched to npm scripts so that doesn't happen again in 9131a19a3bcc527dde96c4406dc7663814dd2275. Should be fixed in v0.3.1.


I'd define a simple wrapper for `Twilio::REST::Client` that encapsulates my application-specific logic, e.g.,

```
class SweetTwilioClient
  def initialize(account_sid, auth_token)
    @account_sid = account_id
    @auth_token = auth_token
  end

  def create_message(opts = {})
    client.messages.create(opts)
  end

  private

  def client
    @_client ||= Twilio::REST::Client.new(@account_sid, @auth_token)
  end
end
```


GridViewBugFixes (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Why are you doing this?  `URIBuilder` can add query arguments from a `key, value` pair already.


Ah, for specialized cases where the support is hairy, you can look at using [hashmap](https://github.com/flesler/hashmap). It allows you to hash arbitrary javascript objects in a map. The default ends up stringifying elements depending on some typing, but you can specify your own hash function on objects to alleviate that pain. (I use it for some distance/locality based hashing for my stuff).
So, if your mesh objects have unique id, or some such thing, you can save the hassle of having to serialize them.


This is built into the Heroku cli: https://devcenter.heroku.com/articles/heroku-postgresql#pg-pull


Aurelio parece que o uso do "sed -n l" tem um efeito colateral inesperado.
Como exemplo, aparece isso:
```
Meu par\303\241grafo, com <strong>negrito</strong> e <em>it\303\241li\
co</em>.$
```

em vez disso:
```
Meu parágrafo, com <strong>negrito</strong> e <em>itálico</em>.$
```


I know. I said a plugin generator can make the things that you're going to make.


Additionally, I'd define a `twilio_client` method in `ApplicationController` that works like

```
class ApplicationController < ActionController::Base
  def twilio_client
    twilio_account_sid = ENV.fetch("TWILIO_ACCOUNT_SID")
    twilio_auth_token = ENV.fetch("TWILIO_AUTH_TOKEN")

    @_twilio_client ||= SweetTwilioClient.new(twilio_account_sid, twilio_auth_token)
  end
end
```

This will make `twilio_client` available to every controller.  Using the `@_twilio_client ||= ...` trick means you don't initialize multiple copies of the `SweetTwilioClient` class in a single request.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2217](http://teamcity.monogame.net/viewLog.html?buildId=5638&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Wow lgtm


If it's not clear, that means you can now do this in `MessagesController#create`:

```ruby
class MessagesController < ApplicationController
  def create
    twilio_client.create_message(from: ..., to: ..., body: ...)
  end
end
```


Hi there sfroment42!
Thanks for share your solution for credit slip with voucher.

I add all the codes in my prestaversion 1.6.0.9, except the line 873, 874 and the corrected 883 ('amount' => isset($amount) ? $amount : $order_detail->unit_price_tax_incl * $full_quantity_list[$id_order_detail],)

In version 1.6.0.9 -- $product_list = array(); --  and the part of -- foreach ($full_product_list as $id_order_detail) -- dosn't exist.

Can you maybe help me?

Thanks in advance!


:unamused:


bla


Don't commit empty files. :smiley:


Newlines at end of file


What changes are to made on our side for this? If I change protected_resource_url to "https://graph.facebook.com/v2.0/me" I get a OAuthConnectionException.


newline at end of file


You're right of course. Didn't check and thought its another extension method that does the same check. But it is not. Corrected.


Feel free to remove this boilerplate commentary.


Does this line throw an error when you run your specs?


You're now using `pg` in every environment, which is good, but make sure to remove the `sqlite3` gem from the `Gemfile` (and re-run `bundle install`) if you're no longer using it.


Why not do this inside zfs_inactive() and avoid duplicating the code in call sites?


79453838-AutomateSomeDocumentation (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Hehe, you are right ^_^.

Now it works on Firefox but in Chrome I get this:
Your browser does not have the necessary WebGL support to display this panorama.

Thx! :)


Note that the same change is also part of add/Unpickling. Different commit, but the delta is the same. It's probably easier to let it go in with this one.


Just tried it on my iPad. This is the error I was getting:

2015-03-09 19:31:45.015 AppSalesMobile[285:26334] *** Terminating app due to uncaught exception 'CALayerInvalidGeometry', reason: 'CALayer position contains NaN: [35.5 nan]'

I set an Exception breakpoint in Xcode to find where it was and did a hack to fix it. See code here:

https://github.com/dave256/AppSales-Mobile/commit/4fb1d1d48eda7a2415f49bfffe9f071ade1b0b44

> On Mar 9, 2015, at 5:50 PM, RBConsulting <notifications@github.com> wrote:
>
> Thanks Dave
> I have been using App Annie but I really like AppSales better.
>
> Please let know if you figure it out.
>
> Thank...Bob Beroza
> Sent through my iPhone 6
>
> > On Mar 9, 2015, at 1:59 PM, Dave Reed <notifications@github.com> wrote:
> >
> > It still works on my iPhone 6, but stopped working on my iPad a while ago (sometime last fall I think).I haven't looked into why.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


Para a Mupi, não faz sentido o usuário só poder postar 1 resposta.


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2158 of 2183 tests passed)

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


Well, we treat methods as types all the time.

    var x = () => { doSomething(); };
    x();


We need to ensure that the `Defaults` for later enabled features are also called, and u need to do this in a loop


The method doesn't seem to be necessary, the return value is always `field.symbol` except when we are getting a member value in which case the symbol seems to equal `field.symbol` anyways. I replaced the whole implementation with `field.symbol` and all tests passed, at least.

We can look into this better tomorrow.


See my previous comment about encapsulating this stuff in an application-specific class.  The environment variable should really be something like `TWILIO_SENDER_NUMBER`.


@pjf there's still an issue with pressure feeding that I have to fix when I get home, alas.


79453838-AutomateSomeDocumentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Estou tendo problemas com a exibição das mensagens de erro apenas no teste.
O uso normal sai como deveria mais o retorno dentro do teste dá erro, como esses:
```
-Opção inválida --foo
 1
+ção inválida --foo
```
```
-O número máximo de colunas é 20
 1
+número máximo de colunas é 20
```
```
-A largura máxima é de 500
 1
+largura máxima é de 500
```
```
[FAILED #1, line 2] zzascii a        ;echo $?
Perl regex '^Uso:.*\n1' not matched in:
1
o: zzascii [colunas] [largura]

```


Newline at end of file


The method doesn't seem to be fetching the annotation, it only verifies that the annotation exists and (with this commit) provides an error message if the annotation is missing.


I think this does need a little refactor. Defintely race condition material. I also don't understand (or remember) the need for the while loop.


Looks good besides the one comment.


can you also bump the version of google-auth-library here to 0.9.3 ?


The annotation is fetched in the `reify` method, see [this line](https://github.com/olafurpg/directembedding/blob/2eb883f3be334572b02f61236e79351b2fec7463/directembedding/src/main/scala/ch/epfl/directembedding/DirectEmbedding.scala#L49)


If you just want "dummy" text in the inputs, use the `placeholder` attribute instead of the `value` attribute.  That's assuming the second argument to things like `ratio_button_tag`, etc. are the actual form field value.  See the [input tag documentation on MDN](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Input#attr-placeholder).


Consider using parentheses here to clarify your intention for this line.

```ruby
(user_id != favorite.user_id) && favorite.user.email_favorites?
```

would make it clearer to you when you come back to read this in the future.


Hello guys! where can I find information about which flags to use? Been searching on Google but cannot find the info I wanted. I would like to learn how to configure clang and use flags correctly. Could you point me out to the right direction? Thanks


This pull request enables tcpip_ipv6_output() to send the first of the (possibly multiple) NS messages for a new IPv6  neighbor immediately after this neighbor is added to the neighbor list.

Under the current implementation, tcpip_ipv6_output() prepares a new NS packet, places it in the uip buffer, but returns, before sending it, relying on the periodic calls of the IPv6 engine to send the pending frame. However, if a packet arrival from the underlying MAC layer (e.g. sicslowpan) occurs before the following call of uip_ds6_periodic, the pending NS frame is lost (sicslowpan overwrites the uip_buf and uip_len may be set to 0 after processing the incoming frame). This pull request will force the NS transmission before tcpip_ipv6_output() releases the control.


Is there a migration for this attribute too?


doing -> don't?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1146 of 1146 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)


haha. yep


I initially tried `[STRING, NUMBER]`, I figured that would work, but I got the same error. @hegemonic, can you clue me in as to why it didn't work?


Agreed. existing code that looks related to the root cause of #228


owwww


develop (mjwitte) - 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)


De novo no testador tem erro:
Se feito o comando fora, o resultado é perfeito, mas no testador há uma problema aparentemente na codificação, como exemplo da uma linha que deveria ser e como fica:
na linha ``` [FAILED #11, line 101] zzascii 4 60 ```
```
   161 A1 241 ¡    185 B9 271 ¹    209 D1 321 Ñ    233 E9 351 é
```
```
161 A1 241 \xC2\xA1 185 B9 271 \xC2\xB9 209 D1 321 \xC3\x91 233 E9 351 \xC3\xA9
```


This is what I ran:
```scala
@ val universe: scala.reflect.runtime.universe.type = scala.reflect.runtime.universe
universe: scala.reflect.api.JavaUniverse = scala.reflect.runtime.JavaUniverse@2845377e
@ import universe._
import universe._
@ val x = q"foo.bar"
x: cmd3.universe.Select = foo.bar
@ x.symbol.isModule
res6: Boolean = false
```


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2217](http://teamcity.monogame.net/viewLog.html?buildId=5638&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 441, ignored: 6 Build time: 00:13:54


Will this URL work for non eu-west-1 buckets?


I don't understand what this "add download from Wigner to the MT2 cfg " change is about. Maria, can you explain it?


Ill fix that right now, must have missed that!


u can just make IDB diff and get all smsg opcodes for 6.1.0 ( 85-90% will be right )


This looks like a fine thing to do to me.


I personally would prefer to avoid statements like this because they can be confusing when assigning and evaluating an expression in the same line. To improve readability, could we do this over two lines please?


If you haven't noticed this already the open gl framebuffer fail to setup so it it is defualting to the cpu renderer
renderer_opengl.cpp:RendererOpenGL::InitOpenGLObjects:306: Framebuffer setup failed, status 8CD6


Huh, I didn't know that was in, ill go ahead and revert that.


Yup. Schemas differ, and have changed; I'd rather do this than have another client migration and change the mapping code!


Bah! It's still failing ... https://jenkins.saltstack.com/job/salt-linode-opensuse13/1052/console


Thanks @agorrod good catch.  I'm not sure how we ended up with two of them in there.  Let's hope this solves the problem, but I suspect it will.


rax retry tests


can merge.


saw approval from larsbergstrom
at https://github.com/saneyuki/servo/commit/27dfbb69c1911c45714632fc3081dabfccbb3963


merging saneyuki/servo/ignore = 27dfbb69 into auto


saneyuki/servo/ignore = 27dfbb69 merged ok, testing candidate = a234cc3e


miss a '/' here?


This commit is implicated in this issue
https://github.com/facebook/hhvm/issues/4979
What's weird (see the issue) is that I only have this problem on some, but not all, of my ubuntu 14.04 and 14.10 build/devel/debug systems.  I don't understand how the different platforms differ.

I did a git bisect on ubuntu 14.04 system compiled with debugging (just -ggdb), and then a linear search of commits just before this was committed to end up at this commit.  Each step of the discovery process I used a private script to clean out all(?) build products and rebuild from scratch.


please indent the "<<" operator to be aligned with the first one in line 3098.


spelling of 'the'


Also 'synthesizer', not 'generator'


Dave...
Obtaining the binaries is pretty easy to script (I would probably leave it to an external shell script, but it can be done with cmake custom commands as well), what's going to be a PITA is that once we remove the binaries from the history, all the consequent hashes will naturally change, which will make people not contributing changes back upstream less than happy...
Anyway, I'm definitely in favor of doing it as it will save a lot of space, it just has to be done at some good moment, probably just before the first 4.1 beta.

Pavel


ah you have just dropped it :package:


main.js already loads jquery ui


Shouldn't this be "isNaN" instead of "isNan"?


tested by installing it locally


I actually meant this line :smile:


The project name is libCellML.


You don't need to use the file extension in the `partial:` option here.

```ruby
<%= render partial: 'comments/form' ...
```

would work.


Fixes #9


BTW. In general, print statements as debugging is frowned upon. Debugging-type code need not be committed. If more information is needed, then we should put it in the exception body.


These 3 lines for comments will cause you problems when you try to view them, as the comments controller needs a `Post` object to be able to render correctly, and these routes won't provide that.


@nihco2 @slyg
You are breaking the design of the placeholder, You can see an offset of 5px. (this issue is visible on all legacy page within palceholders)
What is a goal behind it? Can you do some specific adjusmtents into your own app?


/*
 *  2015.3.3 by shimeng
 *  弹出框基础组件
 *  最后修改于3.6
 *  参数：width      长度
          height      宽度
          title      标题行html
          body      内容html
          bottom      底部html（不写默认为确认按钮和取消按钮）
          textStyle      body和底部 的容器的样式
          showParm      dialog进行show()方法时传入的参数
          onshow      当show()时的回调函数
          onhide      当hide()时的回调函数
          onclose     当close()时的回调函数
          afterRender      渲染后的自定义事件(参数为当前容器)
          isClickBgClose   是否点击遮罩层关闭
          hideTime         自动隐藏时间
 */
define(['template'], function (template) {
    //默认的层级从1000开始
    var zIndex = 1000;
    //相当于id
    var index = 0;
    var jqWIN = $(window);
    var _html = '
\

\
        {{if title}}\

{{#title}}
\
        {{/if}}\
        X\

\

\
		   {{#body}}\
			{{if !bottom }}\
				<button id="dialog_confirm_{{index}}" class="btn btn-success">确定</button>\
				<button id="dialog_cancel_{{index}}" class="btn">取消</button>\
			{{else}}\
				{{#bottom}}\
			{{/if}}\

\

';

    var _dialog = function (opt) {
        opt = opt || {};
        this.opt = opt;
        var _index = index++;
        //模板生成html
        var render = template.compile(_html);
        //模板生成html，传入数据
        var html = render({
            width: opt.width||400,
            height:opt.height||200,
            title: opt.title,
            body: opt.body || '',
            bottom: opt.bottom || '',
            index: _index,
            zIndex: zIndex++,
            textStyle: opt.textStyle
        });
        $('body').append(html);
        this.el = $('#dialog_' + _index);//模态框
        this.elbg = $('#dialog_bg_' + _index);//遮罩层
        //没有底部的结构就不需要事件绑定
        if (!opt.bottom) {
            this.cancelButton = $('#dialog_cancel_' + _index);
            this.confirmButton = $('#dialog_confirm_' + _index);
        }
        this.isClickBgClose = opt.isClickBgClose || false;
        this.hideTime = opt.hideTime || 0;
        this.showParm = opt.showParm;//当初始化dialog的时候自定义参数（如：需要传入某些回调函数）
        this.onshow = opt.onshow || function () { };
        this.onhide = opt.onhide || function() { };
        this.onclose = opt.onclose || function () { };
        this.closeButton = $('#close_' + _index);
        this.bind().show();

        var ar_el = this.el;
        ///渲染后可以自己定义一些自定义事件
        setTimeout(function () {
            if (typeof opt.afterRender === 'function') opt.afterRender(ar_el);//此处参数为整个容器
            //opt.afterRender && opt.afterRender();
        }, 0);
    };

    _dialog.prototype = {
        bind: function () {
            var _self = this;
            this.cancelButton && this.cancelButton.click(function () {
                _self.hide();
            });

            this.closeButton.click(function () {
                _self.hide();
            });
            if (_self.isClickBgClose) {
                _self.elbg.click(function () {
                    _self.hide();
                });
            }
            this.confirmButton && this.confirmButton.click(function () {
                _self.opt.onconfirm();
                return false;
            });

            $(window).resize(function () {
                _self._reposition();
            })
            return this;
        },

        show: function (d) {//这里参数是由外部函数调用传入的参数
            this.elbg.show();
            var _self = this;
            function bgHeight() {
                var bodyHeight = $(document).height();
                var windowHeight = $(window).height();
                if (windowHeight < bodyHeight) {
                    _self.elbg.height(bodyHeight);
                } else {
                    _self.elbg.height('100%');
                }
            }
            bgHeight();
            $(window).resize(function () {
                bgHeight();
            });
            this.el.show();
            this._reposition();
            this.onshow(d || this.showParm);//因为初始化dialog自动show()和dialog.show()参数得保持一致，所以当初始化没有参数的时候就会传入自定义参数。
            if (this.hideTime) {
                setTimeout(function () {
                    _self.hide();
                }, this.hideTime);
            }
        },
        _reposition: function () {
            this.el.css({
                top: (jqWIN.height() - this.el.height()) / 2 + jqWIN.scrollTop(),
                left: (jqWIN.width() - this.el.width()) / 2
            });
        },

        hide: function () {
            this.el.hide();
            this.onhide();
            this.elbg.hide();
        },

        close: function () {
            this.el.remove();
            this.onclose();
            this.elbg.remove();
        }
    }
    return _dialog;
});


[Code style] : Use braces for all `if / else / for`, even if they are single line.


Actually yes. That was the plan. But I can do something different if you want me to have it possible for certain boss fights or quests. If I changed it to @experience >= EXPERIENCETABLE[@level] that would fix that edge case, right?


ghtreat :+12


😛 @kyleterry


No, the runnerCore is the liveRunner core. its what the EngineController refers to


`var` keyword is missing for `values


Done


Thanks for your comment, @jyegerlehner! In this commit, we test the consistency of PReLU and ReLU. In 9527ad8, we test if we get the identical results from non-in-place PReLU and in-place PReLU. Is that good enough?


Do the two commits in this branch totally screw the way you're handling the logic to move stuff around, @SunburnedGoose?


Not triplets. I closely inspected the midi file file.mid -- every note is ever so slightly shorter than they should be (by maybe a hundredth of a second). It introduced little mini-rests between each note that JMusic couldn't handle.


RuntimeData is part of The DSExecutable. Its considered part of the exe needed by the engine to run. This in part was done to minimize chagnes to DSExecutable during the refactor


Resolves #50


Added a task here to merge that: http://adsk-oss.myjetbrains.com/youtrack/issue/MAGN-6193


I think this should be:
```html
<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
```


Any particular reason the response is passed as string here? In my experience so far Uber's response is a JSON object and should go through JSON.parse.


@aashish24 this was my first pass it fails with wrong headers. What we have now picks up the correct headers/libraries from mac


I was expecting those to be merged.  I was avoiding it at the moment to keep the emit the same to avoid scripting issues with main.js and bunige.js.  If you got everything to play together, that's great.


Closes #54


This commit does not appear to work for me. As soon as I import `fmt` package, I see this in JS console:

![image](https://cloud.githubusercontent.com/assets/1924134/6549544/6168df3c-c5d2-11e4-9b87-8a87cef45b4c.png)


Please rename to ydn-db - and tag this commit at 0.8.2


keren, juragan.web.id :D


Ive modified it for listDllTypesToLoad


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/968


Hey @zikani03 I've gone with 4 spaces instead of tabs for this repo. I fixed it here so just git pull when you can :)


请follow simple-requirejs的惯例，不用require，而是define name, dependencies, (dependencises)->


commit readme file


为何不直接用jQuery的event？


Things are starting to come together:
![screenshot 2015-03-08 20 36 41](https://cloud.githubusercontent.com/assets/422381/6549563/e4089d6a-c5d2-11e4-8a09-b8a431126cea.png)

It could do with some abstraction that is beyond my Angular skills to do quickly. I'm currently printing code manually for each item type:
![screenshot 2015-03-08 20 38 38](https://cloud.githubusercontent.com/assets/422381/6549578/4c608bfc-c5d3-11e4-8129-7bf887ae804d.png)

And I didn't pull out store-equipped, just got rid of the call to it. Needs cleaning up.


이거 이제 필요없지?


Comment is wrong, meant to be "// 1 minute". But it doesn't matter enough to push another commit right now.


`# Ignore paperclip attachments.`


Woot Got that sum bitch


아마 나는 커밋 메시지를 `Add Status#photo` 라고 썼을 거 같음


아마 나는 이 커밋을 앞 커밋과 합쳤던지, 커밋 메시지를 `Update/remove photos in status#update` 라고 썼을 거 같음


Once this is resolved, I'd like to find out why the CI tests passed despite a seemingly critical issue preventing even basic functionality.

https://circleci.com/gh/gopherjs/gopherjs/384

Depending on what it turns out to be, perhaps we can improve CI tests to be better at catching such issues.


mm i was wrong.


![d contexto](https://cloud.githubusercontent.com/assets/8311103/6549688/dd71f3e8-c5dc-11e4-8080-c0953fa8e6c3.png)


얘는 엄밀히 말하면 독립적인 작업 단위라고 생각해서, 3a78d35 로 빼내었음


Hello, where can I find the latest schema diagram ?


![d contexto](https://cloud.githubusercontent.com/assets/8311103/6549725/8a94c8b6-c5dd-11e4-9901-9634a53216a0.png)


I think they're still useful with dev-env.sh.  I typically don't have all the requirements installed on the host OS.


hi


This isn't really the right place to put css links


Hi @tnarihi. I think that covers the correctness of PReLU. The one remaining possibility is that ReLU doesn't give identical results to PReLU in in-place computation scenario, due to a defect in ReLU. But I guess that is out-of-scope for your PReLU PR. Perhaps I'll write a test like that for ReLU as a separate matter.


Seriously is, I think it's because we use two different editors, so our spacing is different.
What do you use?


Hmm, it seems i should look more carefully. Sorry


Thanks, Jim!


As there's some interesting changes in there, I may have a go at giving the main scripts a parse at some point too


f


You can now run into a NPE here when the detector is disabled.


![screenshot from 2015-03-09 17 03 08](https://cloud.githubusercontent.com/assets/9469469/6549792/a353bc76-c67e-11e4-8f42-f616aee9902b.png)


Ahh.. you create the thread, but don't start it.


Hi Mattt,

This fix will cause 'allowInvalidCertificates' parameter to be ignored.

If pinning mode is 'AFSSLPinningModeNone', it will return true for both valid and invalid serverTrust,  whatever 'allowInvalidCertificates' is. Since 'AFSSLPinningModeNone' is the default mode, this will cause serious security issue, for it will let all invalid serverTrusts pass the test by default. I don't think this is what developers expect.


You make your HTML and I'll refactor if needed.  If the directive gets in your way, let me know and we'll remove the roadblock to the logical representation.


Use this annotation to builds a Equals and HashCode excluding the annotated field.


FIXME to many )


Did this work?


Added error **messages**, not actual errors... at least, I don't think I did....


Good catch. I wonder if that has anything to do with the problems we were
having. Good work this weekend Allison.
On Mar 8, 2015 6:01 PM, "alliecatstanz34" <notifications@github.com> wrote:

> i just pushed this but this computer is logged in as Cody on eclipse
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neobots2903/Neobots2015/commit/6e4f5ae0ef85e14eb042b37bbcf366372569cb6f#commitcomment-10091706>
> .
>


toi我在3.18和19使用一切正常 麻烦问下是哪里出了问题


good job


没读论文儿的时候，感觉DAC跟AC是一样的；看了之后，怎么觉得就那么不一样呢，不懂的太多了...DAC，糊涂中...新概率的计算方法，总算是弄明白了，但又卡死在码率上面。。。。。。


> so LDAP and non-LDAP users will can login

s/will can/can/


@windy [todolist](https://github.com/windy/lina#todolist) ActiveRecord Validator Helper ( TODO ) means what?


I imagine this will be very hard to read


@bors: retry


Los siguientes alumnos faltan de reportarse en github:

Juan Carlos Mora (j2dd)
Verny Mora (2k15)
Joan Piedra (2k15)
Anthony Fuentes (qwerty)
Esteban Retana (qwerty)

Favor de escribir su nombre en el archivo colaboradores.txt en su grupo, ya que lo tomo en cuenta para evaluar su participacion en el curso.


Maybe you want, max(attack - defense, 0)?


I'll rework the tests for this. These tests are only asserting what the default values are in the config file. They are pretty much useless. A test should assert on behavior or state after an action occurs.


It works!? Awesome


Unused imports: Hashable, Iterable.


Why change this? do/end is preferred for multiline AFAIK.


Actually, __setitem__ uses Hashable; but should probably use a similar fix to this as well.


Already fixed (your first email alerted me).  Cheers!

On 9 March 2015 at 14:31, Jackson Gatenby <notifications@github.com> wrote:

> Actually, *setitem* uses Hashable; but should probably use a similar fix
> to this as well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSSE1001/MyPyTutor/commit/779ff8d086f708c5dbb850544b23e7c4f253c2b3#commitcomment-10092690>
> .
>


Documentation says SetDoubleBuffer takes effect only if current platform supports this. I have no linux installation now to take a test, but I do remember that it was no notably visible flickering of controls there. So I separeted this code just in case not to do it worth. Furthermore, this code is not a good idea at all becouse it results visible extra CPU usage when redrawing list/resizing window. Will try to do something with it.


We need to remember to set the source attribute whenever we create or save an entity so that the EAV records are created.


bug fi"x"


Samu has inconsistent indentation. This caused errors with autocompletion. So I fixed it. Also, netbeans.


Not following this change either. unless form is shorter and more readable since there's no else clause. The intent is: unless this option is set, do it this way.


missing `root_url 'https://juliabottles.s3.amazonaws.com'` ?


nice catch! Can you add some test coverage to https://github.com/mapnik/mapnik/blob/master/tests/cpp_tests/image_io_test.cpp#L46?


@shuckins-r7 Yeah actually @kernelsmith did actually change this to do/end, but Tod wasn't very comfortable about this change so I ended up changing it back. Please see: https://github.com/rapid7/metasploit-framework/pull/4823#discussion_r25559963

I can change it back to do/end again, please let me know.


I applied this patch and ran the resulting `configure`. The relevant part of `include/HsUnixConfig.h` became:

   /* If the system defines fdatasync */
   /* #undef HAVE_FDATASYNC */

    /*  Define to 1 if you have the `fsync' function. */
    #define HAVE_FSYNC 1

I think this patch has fixed the problem.


为啥不在 route 里面写？


Yeah there is a short story behind that, could you please read this and let me know what you think?  https://github.com/rapid7/metasploit-framework/pull/4823#discussion_r25559975


Sure thing @wchen-r7, should have checked that first, reading up.


RowMajor (DeadParrot) - 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)


Wow, yes indeed.  Fixed in 5eca198


Thanks.


A comment should be added here to avoid future pain: Boolean true versus truthy check required here; RPC responses such as ```{ "error" => "Here I am" }``` and ```{ "error" => "" }``` must be accommodated.


@mnapoli you changed submodule to one that's one month old... I think you forget to run `./console git:pull` (which will pull the submodules as well) before committing the new submodules...


Did not work.


Everything is done and committed. However, I'm not sure if I got everything in correctly.


mendingan yang html di dalam php diganti aja rud jadi php pakai echo. Jd sintaks html dimasukin dala php pakai fungsi echo


Yes, I wanted a more polished report as well. The thing that held me back is that the runner script is written in Bash, not Ruby, and I'm not an expert in Bash so I always hestitated to add features. Well, runner.sh is already the most buggy part of gettc.

The advantage of using Bash is that you don't even need Ruby to run things, which is helpful sometimes, but maybe not worth the trouble. I'll probably improve this in future releases.


This returns an image value, which you are ignoring. You'll want to combine them all together with overlay


Uninitialised value != memleak


oke  mat di coba dulu


Looks like you misspelled canola! :)


Remove the target directory. Need to update git ignore file.


add a class comment


Zusk, great changes! But for many different reasons we don't want 476 to be the main bookmark.


Comment added. Thanks.


Also, changelog time.


please help 20745 05154 enquiry number please help


I agree with @kernelsmith here, I don't see the precedence issue mattering in this case. It's not being chained and is pretty long, so if I am reading https://github.com/bbatsov/ruby-style-guide right I think do/end is the better way.


.irssi in znc container?


Ok, I will change back to do/end. Thanks!


explain the exception


@FlowerWrong translate ActiveRecord Validator to JSON Schema column.
e.g.

```ruby
class Post
  validates_numericality_of :width, greater_than: 10
end
```

```ruby
puts Lina::Helper.translate(:post, :width)
```
will be translated to:

```ruby
{
width: {
  type: 'number',
  minium: 10,
}
}
```

This feature is a little complex, there are many variousness in Active Record Validators.

I have working on defining its behave.


Another thing yoiu will have to revert: kemetic_pagan which should be egyptian_pagan due to ACR and CK2+ using that name.


Why throwing runtime exception here. Shouldn't it be custom exception or specific java exception


explain all params


Try

```ruby
def paginate(page: 0, per_page: 10)
```


Throwing generic exception is not a good practice


explain all method parameters


"Zusk, great changes! But for many different reasons we don't want 476 to be the main bookmark."

Could you explain this to me?


benerkeun


Open PMS.Web.Sln. For now, focus only on PMS.Call.Web - windows desktop project.


嗨,

hjwdzh@gmail.com 想關注您

****** hjwdzh@gmail.com 是您的朋友嗎？ ******
好/是: http://invites.flipmailer.com/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=1b464ec2-6084-4e24-bd0c-58a430687b08&from=hjwdzh@gmail.com&src=txt_yes

不/否: http://invites.flipmailer.com/signup_e.html?fullname=Hjwdzh/physbam-drive-pc&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=1b464ec2-6084-4e24-bd0c-58a430687b08&from=hjwdzh@gmail.com&src=txt_no

按一下這裡，將您自己從所有這類電子郵件移除
http://invites.flipmailer.com/uns_inviter.jsp?email=reply@reply.github.com&iid=1b464ec2-6084-4e24-bd0c-58a430687b08&from=hjwdzh@gmail.com


Right. Although that is out-of-scope of this PR, it would be meaningful to write tests for in-place computation for layers that allows in-place computation. It would be nicer that we abstract my in-place computation test and apply to other layers. Another issue I feel is that the current implementation/documentation of Caffe does not provide a good way us to know which layers can be used for in-place computation.


use Http response codes without hard coding the values.


![screenshot 2015-03-08 22 03 55](https://cloud.githubusercontent.com/assets/422381/6550189/2ba11182-c5df-11e4-8c31-fd421810ca8c.png)


Don't throw generic exceptions


:tada:


It's still early, but congrats for this one!


Well, due to many different reason: the mod is labeled as "480 AD" and therefor we considered it as the main bookmark. Another reason was that the 476 always ended up with the WRE surviving and had some issues in terms of characters and balance (byzantine empire). We did not, at that point want the new players to play in a bookmark that felt unbalanced and a bit more bland than the 480 one. Although, I would be fine with having 476 as the main bookmark, it should be more balanced and rome should pretty much fall 9/10 sooner or later in the hands of the aI:


You can check out how I'm refactoring these tests from here:
https://github.com/ghostsquad/Graphite-PowerShell-Functions/commit/dbe03fe83dc1a1d374190c3191dc78d2b0beb67f

TODO:
Complete behavior of sunny day behavior of each config value
Complete behavior of rainy day behavior of each config value
Complete expected exception message for rainy day scenarios


wat is a thios plz to halp wear iz comppooot rr


This could then become

```ruby
limit(per_page).offset(page * per_page)
```


Please explain bit more on the scenarios under test


test cmt


Calling it a night. I'll be back at it tomorrow evening.


Need to read admin user name though automationContext.


Oops, I'll get on that.


Alright, I'll get on that.


This method should return a hash, which could look like:

```ruby
{
  results: results,
  total: count,
  page: page
}
```


I would note that, now with Odoacer being fixed most of the time WRE falls. My historical events make the bookmark more balanced then 480 as in the 476 bookmark you have events that simulate the diplomacy between the two nations, for example, after the WRE falls the ERE can ask their foedarati the Ostrogoths to invade as they did historically. Additionally, Byzantium can hire the Avars to invade the Gepids to the north in the same way.

I have about 20 historical events for the 476 start, from gameplay experience it is indeed the more balanced start date. Go ahead and try it out if you want.


Tod's point is a good catch; this is a happy medium that I think is more readable:
```
unless opts['RunInForeground'] == true
```
What do you think @wchen-r7?


I added this simple stand-in so we can test these intermediate commits on all supported devices.


If I am not wrong, I think there is a typo where "can" should be "can't".
So line 13 should be:
You reach out to grab a handful of the marshmallow wall, but you can't seem to take the handful away from the wall.
Please correct me if I am wrong. Thanks


OK. Sounds fair. But for the Byzantines, did you look at the event troops of the war at start?


Forgot to edit the commit message, should be "Added Ritwik's personal site instead of GitHub"


I re-implemented this in a simpler way ;)


Those event troops for Zenon's rebellion are disbanded when they peace out, they are supposed to be big.


It seems the first three CRCs are already at this file anyway, which means you didn't compile and try this patch yourself - or otherwise it would show a "duplicate CRC" error on the console.


```unless opts['RunInForeground'] == true``` sounds brilliant and I will change to that. Thanks for the suggestion.


They should be around 30k, enough to easily defeat the current emperor.


I separated out only the navigation drawer implementation from your original commit.  The floating action button will come in part 2 as part of the ROM search/extract overhaul.


Updated. Thanks again for reviewing.


Initialising values in the @BeforeClass would be a good practice. If any exceptions occurred at the time of patch construction, then we can't handle it.


same as above


FYI; this seems to be breaking light maps. https://github.com/mrdoob/three.js/issues/6201
Implementing a path for my next PR. Will ping you on this


You can use HTTP response codes directly. No need define our own constants.


why can zip files but not pants


You had originally moved a whole bunch of lines here.  To keep the diff easier to read, I just referenced the original code instead.  We can do some housecleaning commits after the dust settles to make the final code (rather than the diff) easy to read.


I added this conditional so that the menu button also closes the drawer.


Nice.  Looks like something we can re-use in the other activities :+1:


I defined a locale to squelch a lint warning.


63k, that might be a -bit- too high, ill lower it down right now.


RowMajor (DeadParrot) - x86_64-MacOS-10.9-clang: Tests Failed

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


RowMajor (DeadParrot) - 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-77.23%25-red.svg)


RowMajor (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (558 of 559 tests passed)

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


You had originally implemented this block inside an OnLayoutChangeListener, which requires minSDK 11.  I just moved the implementation inside the activity's onCreate method and all seems well.

The one difference to note is that the width is now provided by the DisplayMetrics variable, rather than passed as `left` and `right` arguments in the onLayoutChange callback.  AFAIK the dimension in both cases is provided in pixels rather than dip, so there shouldn't be any difference.


really cool - but we might want to create an options menu to toggle this... this could be a bit of information/visual overload

eventually we should have a mouseover that would provide similar info.

come to think of it, we should eventually also show the items perks + add them to the search filters


Add the libs. This is useless without them.


very good cash for IOException
when ignoreISOException option set true, fix problem when channel disconnect in send response in permanent connection.


I noticed that the use of `metrics.density` was removed in this commit.  Was that change intentional?  It seems we are mixing pixels and dip in some of the calculations above.  For example, width was previously in dip.  Now it is calculated using a mix of pixels (`metrics.widthPixels`) and dip (`galleryHalfSpacing`).


this should probably be named `clearValidators` for clarity


this should probably be renamed to `hasValidators` for clarity


Hm, that might be why the gallery spacing is a little off. I never figured out why. I designed it so the padding on the left and right of each row should be identical to the spacing between the items in each row, but the spacing between the items is slightly larger. The pixels vs. dpi difference would explain that since my device is using something like a 1.4x multiplier.


@rhc54 could you please review this ?
new_hostname can be NULL (cutoff modex) so it cannot be passed to strcmp
that being said, should plist[i]->super.proc_hostname be updated with NULL in this case ?


Hey @stackArmor - I've noticed you're adding these ```::SA.AWS.IAM.ACC.``` sections to a number of the auditors.  I'm not familiar with the format.  I'd love to learn more about what you're adding.

We have a gitter.im chatroom if you'd like to join:

[![Gitter](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/Netflix/security_monkey?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge)


变量数值交换


At least you got the name-mangling bit!


My first repo


我觉得是不是最好让用户自己在 poopup 中进行选择设置？


Remove `solo_` ?


RowMajor (DeadParrot) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


gh-pages commit


Shouldn't it be Palatino Linotype?


que pasó aquí?


Good, and for the changelog, you can do it later, but before the next release (will be in a while, no need to stress) it should be done.


First version. Haven't tested it yet.


could be host.host || host


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2212](http://teamcity.monogame.net/viewLog.html?buildId=5626&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Hi, does this fix solve this issue ?? http://bbs.aprbrother.com/2983-beacon-enters-region-and-also-exits-immediately/p1#p4391 ??


I'm worrying about if this will disable some usage that should be allowed.


@sueloverso You made this change as part of  the initial commit in #1696, it looks wrong to me (and Coverity). Maybe I'm missing a subtlety about a case where we don't want to set the slot error though?


Understood

On Sun, Mar 8, 2015 at 10:49 PM, Loup <notifications@github.com> wrote:

> Good, and for the changelog, you can do it later, but before the next
> release (will be in a while, no need to stress) it should be done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rickinator9/WTWSMS/commit/d9ca7d6738fcd3da4564c0c2f8517685c8c2500c#commitcomment-10093058>
> .
>


@appsbychris I looked over your implementation. That's so nice and simple. Thanks!