@physikerwelt Thanks for the code review. I agree with all of your comments. I was working on something else just now but will start making the changes you asked for right now. Thanks again for the help!


Where do these scopes come from? I can see `scopeName` under `atom.grammars.grammars.#index#`


Type C-x 3 to split your window horizontally. That'll make things easier. Make one of your windows switch to your *scratch* buffer. Go to the other buffer and type M-x isend-assoc<TAB>
If installed correctly the autocompletion should write for you "isend-associate" otherwise there is something not quite right with your installation.
If autocompletion does fine type <ENTER> that will get you in "isend-mode". If asked for a buffer, type `*scratch*`.
M-x describe key
then:
C-enter
Notice that isend-mode works only between two buffers so if you switch to a third there will be no magic.
Notice also you can reassociate two buffers at any time.


@dave-malone  I remember the build failed. This can explain the deploy failure, but can not explain the build failure


HBIRE_vector_take2 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


"assholes"...  while possibly true, this may be a bit harsh :)


bad name, this isn't installing anything.  It's merely chaining the handlers together


Yeah, that's great. I'll just review the other commit, but I think this is good to go now.


@schleichardt I still didn't check the transaction script pattern, but you can already check the iterative version. In order to test it comes properly sorted I had to sleep the thread for a total of 10ms. Unless found an alternative, I think it's almost unnoticeable.


I wrote a simple script:

```javascript
scopes = [];
for (var i in atom.grammars.grammars) {
  g = atom.grammars.grammars[i];
  scopes.push([g.name, g.scopeName])
}
JSON.stringify(scopes, undefined, 4);
```

Here's what I got:

```json
[
    [
        "Null Grammar",
        "text.plain.null-grammar"
    ],
    [
        "Hyperlink",
        "text.hyperlink"
    ],
    [
        "TODO",
        "text.todo"
    ],
    [
        "C++",
        "source.cpp"
    ],
    [
        "Clojure",
        "source.clojure"
    ],
    [
        "CoffeeScript (Literate)",
        "source.litcoffee"
    ],
    [
        "C",
        "source.c"
    ],
    [
        "CSS",
        "source.css"
    ],
    [
        "CoffeeScript",
        "source.coffee"
    ],
    [
        "GitHub Markdown",
        "source.gfm"
    ],
    [
        "Git Rebase Message",
        "text.git-rebase"
    ],
    [
        "Git Config",
        "source.git-config"
    ],
    [
        "Git Commit Message",
        "text.git-commit"
    ],
    [
        "Go",
        "source.go"
    ],
    [
        "Go Template",
        "source.gotemplate"
    ],
    [
        "HTML (Go)",
        "text.html.gohtml"
    ],
    [
        "HTML",
        "text.html.basic"
    ],
    [
        "JUnit Test Report",
        "text.junit-test-report"
    ],
    [
        "Java Properties",
        "source.java-properties"
    ],
    [
        "JavaServer Pages",
        "text.html.jsp"
    ],
    [
        "JavaScript",
        "source.js"
    ],
    [
        "Regular Expressions (JavaScript)",
        "source.js.regexp"
    ],
    [
        "JSON",
        "source.json"
    ],
    [
        "Java",
        "source.java"
    ],
    [
        "Makefile",
        "source.makefile"
    ],
    [
        "SQL (Mustache)",
        "source.sql.mustache"
    ],
    [
        "Objective-C++",
        "source.objcpp"
    ],
    [
        "LESS",
        "source.css.less"
    ],
    [
        "Strings File",
        "source.strings"
    ],
    [
        "HTML (Mustache)",
        "text.html.mustache"
    ],
    [
        "Objective-C",
        "source.objc"
    ],
    [
        "Perl",
        "source.perl"
    ],
    [
        "Property List (Old-Style)",
        "source.plist"
    ],
    [
        "PHP",
        "text.html.php"
    ],
    [
        "Perl 6",
        "source.perl6"
    ],
    [
        "Python Console",
        "text.python.console"
    ],
    [
        "Python Traceback",
        "text.python.traceback"
    ],
    [
        "Property List (XML)",
        "text.xml.plist"
    ],
    [
        "HTML (Ruby - ERB)",
        "text.html.erb"
    ],
    [
        "Regular Expressions (Python)",
        "source.regexp.python"
    ],
    [
        "Python",
        "source.python"
    ],
    [
        "HTML (Rails)",
        "text.html.ruby"
    ],
    [
        "Ruby",
        "source.ruby"
    ],
    [
        "Ruby on Rails (RJS)",
        "source.ruby.rails.rjs"
    ],
    [
        "SQL (Rails)",
        "source.sql.ruby"
    ],
    [
        "JavaScript (Rails)",
        "source.js.rails source.js.jquery"
    ],
    [
        "Sass",
        "source.sass"
    ],
    [
        "Shell Session",
        "text.shell-session"
    ],
    [
        "Ruby on Rails",
        "source.ruby.rails"
    ],
    [
        "SCSS",
        "source.css.scss"
    ],
    [
        "TOML",
        "source.toml"
    ],
    [
        "SQL",
        "source.sql"
    ],
    [
        "Shell Script",
        "source.shell"
    ],
    [
        "Plain Text",
        "text.plain"
    ],
    [
        "XML",
        "text.xml"
    ],
    [
        "XSL",
        "text.xml.xsl"
    ],
    [
        "YAML",
        "source.yaml"
    ],
    [
        "Regular Expressions (Babel)",
        "source.regexp.babel"
    ],
    [
        "CoffeeScript (JSX)",
        "source.coffee"
    ],
    [
        "Babel ES6 Javascript",
        "source.js.jsx"
    ],
    [
        "TSS",
        "source.tss"
    ],
    [
        "Diff",
        "source.diff"
    ],
    [
        "HTML (Angular)",
        "text.html.angular"
    ],
    [
        "Handlebars",
        "text.html.handlebars"
    ],
    [
        "C# Script File",
        "source.csx"
    ],
    [
        "NAnt Build File",
        "source.nant-build"
    ],
    [
        "HTML (Swig)",
        "text.html.swig"
    ],
    [
        "JavaScript Template",
        "text.html.js"
    ],
    [
        "C#",
        "source.cs"
    ],
    [
        "Spacebars",
        "text.html.spacebars"
    ],
    [
        "Rust",
        "source.rust"
    ],
    [
        "Regular Expressions (jsx)",
        "source.jsx.regexp"
    ],
    [
        "JSX",
        "source.jsx"
    ],
    [
        "Marko",
        "text.html.marko"
    ],
    [
        "Jade (Python)",
        "source.pyjade"
    ],
    [
        "SWIG",
        "source.swig"
    ],
    [
        "Jade",
        "source.jade"
    ],
    [
        null,
        "preview"
    ],
    [
        "Swift",
        "source.swift"
    ]
]
```


`basicHandlerInstaller`?  `defaultHandlerInstaller`?


Great


name the arg and return for readability


Probably worth explaining that DATASOURCE here is simply a link to a `countries.json` file. It's not, itself, saying where the actual data lives, it's simply selecting the map to use. (This can either be a path to a local file on disk, or a remote URL)


Because I don't like my builds to break under me because somebody down the line accidentally published a breaking change.


I'd prefer an `interface` over a `func` because I'm old and stodgy.


Although it shows how the option feature, I dislike this particular change for two reasons:

- here we are repeating a magic number (5) twice;
- it is a reasonable change to go from a constant to something set with a dial ("5 goal game or 10 goal game"), and the options is not good for that, we would move it back to a port.


what about textareas and selects ?


This looks a lot like an `http.Handler`. Why do I have this?


sounds good to me


@mikejolley done :)


Yes ! No problem !


Sure, please use the full org name ("juridisk person")


You're right. I'm still trying to wrap my head around this node stuff. In any event, did some more cleanup, added a sleep function to make sure theres adequate time for the response to go through.


rax retry tests


@coderhaoxin
```
//=include
//= include
```
I think so.


fixes 14


Good catch, thanks for this. Not sure how it the build didn't fail with the error in there.


Ok, I definitely don't like this because it allows the names of the chaining methods to be different.  Make this an interface.


Check out L4966: https://github.com/MarlinFirmware/Marlin/commit/97ec224d72247537ed1c498f1660e4eee4c58709#diff-1cb08de130a6ece2d1b5b9c37bcfef48R4966

You forgot to change those `target`s to `destination`s, it should say:
```
plan_buffer_line(destination[X_AXIS],destination[Y_AXIS],destination[Z_MAX_ENDSTOP_INVERTING],current_position[E_AXIS],AUTO_FILAMENT_CHANGE_FEEDRATE/60,active_extruder);
```

Use the `destination` identifier here, otherwise you will get a compilation error saying target wasn't declared in the scope.

I suggest editing your config headers to enable the feature so that you can test against it prior to merging. In this case you should have defined `ULTIPANEL` and `FILAMENTCHANGEENABLE`.

I have edited my commit to include these changes here: https://github.com/MarlinFirmware/Marlin/pull/2186


Great! I've merged it into my master and deleted the branch. Feel free to
make a new branch for the next file.

It looks like there are some other failing tests to resolve based on one of
Francoi's checkins so I'm going to look into what's going on there.

On 1 July 2015 at 01:10, Thomas McColgan <notifications@github.com> wrote:

> Done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tleeuwenburg/keras/commit/a50ff36f859be543ff5f7f0513a3c5d716e5a2ac#commitcomment-11933920>
> .
>

--
--------------------------------------------------
Tennessee Leeuwenburg
http://myownhat.blogspot.com/
"Don't believe everything you think"


fixes #14


Looks good to me


I don't think we have any of these..


I understand why this is here (makes the diff easy for me, thank you), but add a TODO to move near its owning type.  We'll do that just before merging.


Did you mean to leave this commented out?  If so, consider removing it completely.


HVACTemplateTransition (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1673 of 1676 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)


Thanks for the contribution. The number of Nano-based boards currently appearing is interesting.

I think it would be a good idea to replace the AIO names with their DIO counterparts when used as a digital pin. The comment in arduino_168_328.h, line 113 might be a bit misleading here.

One question I can't answer myself: does this board really have a Debug LED? If not, this pin should be commented out.

I'll happily do both changes, but need an answer for the second question.


Not to say it shouldn't be fixed, but it's not really a critical issue right now :)


I understood from the talk that "safepoints" are instrumented at method returns and loop continues, not arbitrary.
http://blog.ragozin.info/2012/10/safepoints-in-hotspot-jvm.html


Have these return a struct that has the correct methods for `http.Handler`.  That struct ought to contain things like `authenticator` and `authorizer`


When i start a game com do not show up and i see these errors which look bad:
[f=0000000] Error: LuaRules::RunCallIn: error = 2, Initialize, [Internal Lua error: Call failure] error = 2, LuaRules/gadgets.lua, error = 2, LuaGadgets/setupdefs.lua, [string "LuaGadgets/setupdefs.lua"]:18: attempt to perform arithmetic on field 'energyCost' (a nil value)
stack traceback:
	[C]: in function 'Include'
	[string "LuaRules/main.lua"]:53: in main chunk
[f=0000000] Error: LuaUI::RunCallIn: error = 2, Initialize, [Internal Lua error: Call failure] error = 2, LuaUI/setupdefs.lua, [string "LuaUI/setupdefs.lua"]:18: attempt to perform arithmetic on field 'energyCost' (a nil value)
stack traceback:
	[C]: in function 'Include'
	[string "LuaUI/utils.lua"]:53: in function 'include'
	[string "-- $Id: camain.lua 3171 2008-11-06 09:06:29..."]:38: in function 'chunk'
	[string "luaui.lua"]:55: in main chunk


Looks like it works when form is empty but brakes when some 'data' comes from backend in different order


"through analyze on" => "thorough analysis of" ?
it's => its


Haw about this: `int(status) >= 200 and 204 != int(status) != 304`


This will save the execution time as we don't need to pull/unzip/delete the data set separately for every test (as we no longer modify the contents of the dataset itself)

![Image of histogram](https://lh3.googleusercontent.com/IvpmK3Cr1ntq9MtEI3Kq2Q2IFggEs3TN8vAc4GbbYn8=w870-h418-no)
[Execution times interactive histogram](http://codepen.io/rplevka/pen/yNpdXv?editors=001)
..pretty neat, isn't it


@boneskull, i need to mock external modules, because i don't want to test them. for example the code use "fs" to delete a file. when i test this code i don't want to delete anything, but i want to check fs.unlink was called. the code contain
```javascript
var fs=require('fs');
```
and sandboxed-module helped me to mock fs. (and i use sinon to create a stub for fs.unlink)


modulegi is typo


Might still be worth calling `getContext().dispose()` followed by `System.exit(0)` then, to give the context a chance to properly clean up resources before quitting the JVM.


So I am wrong, `source.html` is definitely not it. However under `HTML` grammar it has `text.html.basic` for `scopeName`. Not sure what is correct here. Does `text.html` retrieve all nested options, such as `text.html.basic`, etc?


Happy to put this out of scope, I agree it would be a theano thing.

On 1 July 2015 at 01:14, Thomas McColgan <notifications@github.com> wrote:

> Thanks for your feedback, I found it very helpful! I think I have
> addressed everything except for the question you raised here about float
> precision. I just put 1e-7 in there out of a gut feeling to have something
> small, not sure how we would systematically test for that kind of precision
> issue. On the other hand, if such an issue does arise, it is most likely
> coming from theano, so maybe it is out of scope for us?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tleeuwenburg/keras/commit/2e84eb5a6b06b55809b1772461e86b360c2e8379#commitcomment-11934105>
> .
>

--
--------------------------------------------------
Tennessee Leeuwenburg
http://myownhat.blogspot.com/
"Don't believe everything you think"


`mongo.base.MongoKeyValueStore` is used by only the Old Mongo modulestore.  In order for your unit test to work with Split as well (when we add a nightly test to test all our unit tests with other modulestores), can you mock `modulestore.mixed.get_course` instead?


interested in


Oops. What's this for?


please implement `db.js` as it is implemented here: https://github.com/hoodiehq/pouchdb-hoodie-api/blob/master/tests/utils/db.js

That way we don't need `uuid`


Il y a déjà une règle sur mobile dans kanban_view.less, donc je ne pense pas que ce soit ça qui ajoute la barre.
        .o_kanban_record {
            .o-flex(1, 1, auto);
            width: 320px;
            @media (max-width: @grid-float-breakpoint) {
                margin: 0 0 1px 0;
                width: 100vw;
                box-shadow: none;
            }
        }


FYI, `jtx::Account` is implicitly convertible to `AccountID`.


definetly typo


remove console.log


```bash
$ nosetests -m test_reimport_us test_import.py -v
@test: Try to Import all users from the ... ok

----------------------------------------------------------------------
Ran 1 test in 58.759s

OK
```

This test examines the scenario of repetitive user import


it's => its


@AdrianGaudebert This should address everything you mentioned above...


Wtf just return `(into above-current-plus-rest-of-current every-line-below-current-line)`


I don't get what you're trying to do - why can't we make a normal call?


I know the test existed before, but it's wrong. `api.getLocalChanges()` does not create the db. `new PouchDB(name)` does. Let's remove the test altogether


You should document the user interface, so not _nd_grid, but ogrid and mgrid.


A `CachedView` is not necessary here (since we don't care about caching) and you will have one less dependence on the call to `getApp`. Just pass `*env.ledger` where you are currently passing `view` to `forEachItem`.


This looks wrong...


> from Wallacoloo/boopity-boppity

:laughing:


On Mon, Jun 29, 2015 at 02:21:07AM -0700, Carl Friedrich Bolz wrote:
> why is the results checking commented out?

I just adjusted it so that it returns True or False like the versions
for other languages. See line 316.

--
Best Regards
Edd Barrett

http://www.theunixzoo.co.uk


I would indent line 47 one extra tab stop, but that's just me.


Thank you!

-e.

On Jun 30, 2015, at 11:38 AM, Lasse Fister <notifications@github.com> wrote:

> changes send by @EbenSorkin via email 06/29/2015
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Tarobish/Jomhuria/commit/6711a1b27fc24e5715b86179152c5b61298e3793


@marcus-herrmann
Für sowas in Zukunft bitte immer die neuen Internationalisierungs-Styleguides beachten. Vor Allem wenn man zwei Punkte beachtet, dann ergeben sich viele Fehler schon gar nicht:
1. Innerhalb von views einfach die t() Funktion benutzen, nicht I18n.t()
2. Den Scope-String mit einem Punkt beginnen (z.B. `= t '.map.enlarge_map'`)
Wenn es dann nicht funktioniert, weißt du, dass die locale an der falschen Stelle steht.

Brauchst es in diesem speziellen Fall nicht nachträglich korrigieren. Ich baue auf einem anderen Branch eh die Suche um, daher ist es mir aufgefallen. Nur für die Zukunft zur Referenz.


Crazy thought?

```ruby
          case length
          when Fixnum, Symbol, Range, Hash
            send("rel_length_#{length.class.downcase}_error_message", length)
          else 'should be a Symbol, Fixnum, Range or Hash'
          end
```

And you'd need to implement `#rel_length_fixnum_error_message`, but that would reduce the cyclomatic complexity.  Also maybe it could be an `if` statement with `length.class.in?([Fixnum, Symbol, Range, Hash])`.  I don't know if that loses us some useful class hierarchy checking, though.


1679-DualDuct (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)


@jdandoy


just a chore, not a fix, right?


:clap:


Интересно получилось, но зачем в сообщениях время черным по серому писать, не видно же ни фига.


I believe its worthwhile writing a "directory iterator" as a substitute for `forEachItem`. If we have `directory_iterator`, we could write this free function:
```
boost::iterator_range<
    directory_iterator, directory_iterator>
directory (uint256 const& key, BasicView const& view);
```

Usage:
```
void visitAccounts (AccountID const& account)
{
    for (auto const& item :
        directory(keylet::ownerDir(account)))
    {
        // use `item` here
    }
}
```


Für obiges Beispiel würde diese Internationalisierung nicht gefunden werden, da es keinen "map" view gibt. Es müsste unter `offers.index.*` genested werden.


コメント


> It seems that it is common in emscripten to named the mem file: ".mem.js"

Is it?  AFAICT by default emscripten seems to spit out `test.js` and `test.js.mem`.


Well, I was not sure whether there are some in Packagist. I indeed cannot think of a place with one though (I don't think we have lots of forms)


I had thought about it too!
I finally decided it was a form of "over-optimization" and was not very KISS...

I think `length.class.in?` will definitely lose class hierarchy checking ^^


Problème avec la 2.6?


I am not sure whether gradle do it itself or not, these are generic lines that we put in any travis file.


I found out there was a problem with one of the part 1 tests, but it's
fixed now.

On Tue, Jun 30, 2015 at 10:29 AM, Caroline Larsen <animals4all@gmail.com>
wrote:

> All the part 1 tests passed!
>
> On Mon, Jun 29, 2015 at 10:31 PM, Mipmap <notifications@github.com> wrote:
>
>> Looks 100% correct to me, nice job!
>>
>> Did all the part 1 tests past the unit tests?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/animals4all/Exercise04/commit/b86fa7d6f731bcd630eabfbe4ad46f23f1efbe59#commitcomment-11923098>
>> .
>>
>
>


Le 30/06/2015 11:21, Rudi Ihle a écrit :
> Just a note: phpinfo() does work here with php.ini-production. And it
> avoids the 'It is not safe to rely on the system's timezone' warning.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenBricks/openbricks/commit/989f06b117d1fb6ca9c27f0f4e6561808245ec10#commitcomment-11927331>.
>
Don't remember why it didn't work (blanck screen, I guess)
It's great it works now.


Is this really Dominican Republic? Thanks for pointing to the package - love it!


These 3 ctors could have all been handled with a single templated ctor


I'm observing a 'double free', too, and will try to fix this for the upcoming release, FWIW.


I still observed this problem on 12.04, cmake 2.8.12.1, when installing iDynTree some .so files have been installed in $INSTALL_DIR/x86_64-linux-gnu, so I had to move them manually to the right folder to make everything work.


Done


yes, please.


Verstehe. Ich wusste nicht, dass die Ordnung + Hierarchie in de.yml fest mit den Views und Methoden der Models verbunden ist


Wo finde ich diesen Styleguide?


That looks correct to me. I have a local example running with data coming through as well.

One thing to check is that the build you are using includes these changes. If you installed via bower, I don't believe these changes are included, though you said it works when the form is empty so I assume this isn't the case.

Can you provide the full JSON you're using that breaks it?


When I was running the test case it was throwing null pointer when toBoolean is called, to fix the null pointer I updated the code.


You can use `print` wherever you can use `echo`, but not the other way around. For consistency and predictability I have used `print` everywhere in new code instead of mixing both. It makes code search and changes a little bit easier. The performance difference doesn't really matter.


can you put this part into a dedicated print function so we keep functionality seperate from output?


Not a problem dude, never a bad thing to have more eyes looking over the code!


Do we have a Trello todo for this?


The prints ?


@ll911 We're about to switch to 2.3, will updating this update ckan on deply or will we have to modify some jenkins scripts?


On my system python does not accept the character percentage (%) for comments, only sharp (#) or three apostrophe ('''). I wonder why it works on your system @traversaro


Hmm, I'm trying out a new text editor (Atom) and it decided to collapse some double line spaces on its own. I'll try to clean this up and issue a new PR.


Closes #9


signature would be kind of `print_stuff(test_nr, test_count, skipped, message)` just with a better function name


It must be done by Idea while optimizing import statements.


For an instruction taking multiple lines the three dots do not work on my computer, the backslash (\) instead works.


確認しました。


Yes, its moved to build.gradle and I have updated the version there as well


What is this field supposed to do? Right now it's a copy of existing "info" field (L27)


Not yet, I'll get those created shortly.


> Is it? AFAICT by default emscripten seems to spit out test.js and test.js.mem.

https://github.com/kripken/emscripten/search?utf8=%E2%9C%93&q=%22.mem.js%22&type=Code


unneeded backslash


Ok, I'll check it :)


:+1: Awesome :)


@iyee it really depends on what you are trying to do. If you just want to integrate against the OAuth2 API of UAA, then there are much better oauth libraries out there. And if you are looking for something that acts as a client for the more general UAA API, then you can check out https://github.com/pivotal-cf-experimental/warrant, but it is still pre-1.0 and is subject to major API revisions.


OK


I'm still working on it.


According to [this](https://atom.io/docs/v0.186.0/advanced/scopes-and-scope-descriptors#scope-selectors) these selectors work like CSS selectors so I believe it should match text.html.basic and all of the other text.html options. I am glad you are finding this useful :)


@seelabs I tweaked existing code to test a successful transaction where delivermin is greater than the destination amount. Thanks for the suggestion!


Ok


ALtextarea などはmacroと呼ばれ、よく使われるhtmlのパターンを登録しておいてそれを呼び出す処理を行っています。
出力されるhtmlは
https://github.com/aipocom/aipo/blob/master/war/src/main/webapp/WEB-INF/templates/vm/GlobalMacros.vm
このへんのファイルに書いてあります。


@MaruthiPrasadA Please do not post under commit threads unless they are directly related to the code that was committed. Please post all issues related to the project here:https://github.com/bchavez/Coinbase/issues

Please follow proper GitHub usage. This is the last warning.


here is VM, good catch.


おそらくここで

    mvn install

の時にエラーになるかと思います。


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 94](https://teamcity.labkey.org/viewLog.html?buildId=258516&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 24 Build time: 00:32:24
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

```


this will break FW rules for sure


https://github.com/clarat-org/clarat/tree/develop/config/locales
Ich weiß, ist etwas lang. Aber wenn du die beiden Sachen beachtest dann wirst du quasi gezwungen in den meisten Fällen gezwungen, es richtig zu machen


HBIRE_vector_take2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1680 of 1682 tests passed)

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


ack


Ah, good, can you commit the fix so I can review it?


fsdasfda


oic. No, better rollback changes in this file.


Add a `julia 0.3` (if that makes sense) next time around


```
Utils.nonNull(v,"...")
```


roll back too


Mhh... I have the impression that an intermediate version between the python and the matlab one was committed. I will check!


It looks like you forgot to remove one in flower.js:
url: url_prefix() + '/api/task/timeout/' + taskname,


If anyone is relying on the check, we should probably cure them of that.


These additions are supposed to be Events, not Telemetry...


cool, I planned to do this one i could look for the current issue with duplicates.


Why aren't `magma_sources`, `jit_sources` and `backend_sources` included here?


We probably could, yes! I was thinking about this yesterday evening.


hehe, your tests are devilish !


fix space


Yep, let's fix that up in a subsequent diff.


German (Austria)


3.2, 3.3 and 3.4
And no windows ?


Nevermind, i see the reason now.


German Austria


this brings a fair number of other changes:
https://github.com/finnlabs/awesome_nested_set/compare/v2.1.6...finnlabs:2-1-rails4


I don't know why I keep having this spaces problem, I don't know if it is the rebase who is miscalculating this since I am paying attention in both editors I use to this (vim and atom), mmmm....


haddock string of `getRqMod` needs an update, i think.


nicer, good.


@changtengfei I'd like to make sure change like this end up in the main repo of OpenWSN. Do you see a way to merge your changes into the main OpenWSN repo, and use a conditional scons switch `gd=1` to set a couple fo `#define`s?


Utils.nonNull on source, target and probably e.


good catch, total copy/paste error.


HVACTemplateTransition (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1679 of 1682 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)


Could you fix it?


String.format better... since is just one value a single line concat statements is acceptable too.


FacilityTabGridView (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)


coming from static languages I always feel weird about this filename not been defined before the if/else, do you think this can have any implication ?


[optional, fyi] i would have implemented a function that takes a sid and a thentosconfig and returns a maybe proxy.  that would take some complexity out of this function and could have a clear and easy-to-grasp haddock string.

(also, i suspect that we may be able to think of a nicer way to configure proxies in the future, then we would only have to change that smaller function.)


HBIRE_vector_take2 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1167 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-69.81%25-green.svg)


Wow...Now we'll have to choose between economy and safety.... ^^'


I've always had a bad time getting the ipython cluster interface to work well on LSF. Jobs seem to take an unreasonably long time to complete - possibly due to something about LSF array jobs and ipython zmq messaging.  Even when the jobs get scheduled immediately and then immediately fail with an error, it takes 10+ minutes for the ipython cluster to shut down and for the errors to appear in the logs. I switched to a script that pipes grabix vcf chunks through vt decompose | vt normalize | vep | gemini load_chunk, and then calls gemini merge_chunks directly.


will try :)


@Sasikanth3 is the size option the number of rows to import at a time? Also how can we set various settings on in the db driver to turn off autocommit and secondary indexes? The current database is several dozen gigabytes and inserts are taking longer and longer.  We will test this code on the actual db and provide feedback.


- webkit and moz border-radius prefixes applied.


ah inline, no, no windows yet.


Oh I see nrom(df) so size is number of rows.


Looks like you want to add 1 cup


Missing new-line at end of file.


= 1 postgres ругается, поставил true все ок )


prefix order updated.


You see the red circle and arrow on the last line of the file? That's because there should be a clear linespace at the end of each file, since end-of-line character !== end-of-file character.


i took some time to get this use int, kinda hacky but useful I admit.


I think it looks better with height: auto; padding: 5px;


Please be a bit more verbose. Why does Pyinstaller need this if Python does not? Does it trigger some hidden import mechanism?


Область видимости переменной `mainWindow` ограничена этой анонимной функцией.
После отрисовки окна в `mainWindow.loadUrl()` его *относительно скоро* грохнет [сборщик мусора](https://developer.mozilla.org/ru/docs/Web/JavaScript/Memory_Management), потому что после отработки `app.on('ready')` на переменную `mainWindow` не останется *ссылок*.


- [ ] @alexeyk ```this.layout.form.editors.root.toggle_button```


Instead of `echo '<some markup...>', do `?><some markup...><?php`. That way, it will be syntax highlighted correctly, making it easier to spot typos.

Some goes for the relatively simply label below.


Hi John,

I just fixed it. sorry about that! the cluster was so slow that I had to push it before leaving.

PoYuan
________________________________
From: John Blischak [notifications@github.com]
Sent: Tuesday, June 30, 2015 7:13 AM
To: jdblischak/singleCellSeq
Cc: Po-Yuan Tung
Subject: Re: [singleCellSeq] correct the inter-individual variance (9ec998d)

Doesn't this variable name throw an error?

—
Reply to this email directly or view it on GitHub<https://github.com/jdblischak/singleCellSeq/commit/9ec998dc8b06e2616abd4a285e1a939bdd29ab03#commitcomment-11930219>.


@saintedlama yep... not a bad place to hack when it's cold as hell in my home country ;)


Whoops, I thought I switched to the master branch first :o. How do I cherry-pick a commit to the master?

(Other than that I believe the conversion branch is ready to be merged anyway)


argh.  (-:

we should have just used hex encoding...


Here's exact example.
```javascript
{
	"schema":{
		"title":"Title",
		"properties":{
			"prop1":{
				"title":"Property 1",
				"type":"string"
			},
			"prop2":{
				"title":"Property 2",
				"type":"string"
			},
			"prop3":{
				"title":"Property 3",
				"type":"string"
			},
			"prop4":{
				"title":"Property 4",
				"type":"string"
			}
		},
		"type":"object"
	},
	"data":{
		"prop1":"",
		"prop2":"",
		"prop3":"",
		"prop4":""
	},
	"options":{
		"fields":{
			"prop1":{
				"order":"3"
			},
			"prop2":{
				"order":"2"
			},
			"prop3":{
				"order":"1"
			},
			"prop4":{
				"order":"0"
			}
		}
	}
}
```
Looks like it breaks when there's more than 2 or 3 items and items with highest order come first after serialization/desirialization


`<p><i><?php _e( 'Primary...', 'genesis-featured-image' ); ?></i></p>`


Hello, Zarkonnen!

In fact as I see you don't need to change version in seleniumbuilder/install.rdf, because makefile script is doing that, according to the version taken from seleniumbuilder/chrome/content/html/js/loader.js (don't know why is it made so, but just to make you know).


The commit message should better read:
```
Add: hook for Logilab

Logilab has a hidden ConfigParser import (it's imported from six). This hook provides this hidden import. This makes tests/libraries/test_pyline.py pass.
```


remove this statement


`parts =`


Well, I still have problems :(

With your new .classpath I don't have the src folder in the build path so the Action1 is not available.
Adding the src folder to the new .classpath the problem remains the same as reported in the issue.
If you want I can submit another commit.
Here some screenshot...

*With your .classpath*
![image](https://cloud.githubusercontent.com/assets/4881389/8435479/b3dad724-1f51-11e5-9611-fca0494a9b32.png)

![image](https://cloud.githubusercontent.com/assets/4881389/8435517/ed23b7ee-1f51-11e5-8f51-076de3760054.png)
*error*
![image](https://cloud.githubusercontent.com/assets/4881389/8435611/a3fdc086-1f52-11e5-8eb7-3810c3db5c76.png)

*Adding the src folder*
![image](https://cloud.githubusercontent.com/assets/4881389/8435646/cb49653c-1f52-11e5-82b0-acc9ec2cc85d.png)
*error*
![image](https://cloud.githubusercontent.com/assets/4881389/8435694/0cc428e4-1f53-11e5-8b84-fb66836708bf.png)


`parts = [... for option_string in ...]`


There is no event in Amsterdam on July 31st. I was merely pointing out to Jacqui that the NewsReader people are in Amsterdam.


Yup :smile:, this provides a default argument to the entrypoint so that it will run the executable 'logstash' as the user 'logstash' (ie `exec gosu logstash logstash`).


No tabs please, should be spaces.


Hey guys, I just made another wiki page, now showing how to setup your eclipse workspace.
@CircuitLord I ended up setting you up wrong, you dont have any of forge or minecraft's code setup in it.
You will have to run gradlew commands after all. You can just skip to the step where you close eclipse and navigate to your git folder.


Was there a reason for moving the order of the proxy and kubelet?


yes, this 0 is superfluous.


No, all of them need be positive numbers.


Scope is important. :sweat_smile:


That sounds like a reasonable plan to me.  Although it may be wise to keep the dracut code separate.  Or at least structure such that we _could_ push it upstream in a way that would be acceptable to the dracut maintainers.  I;d love to shed this code from the ZoL tree but we would need someone willing to spend the time to shepard it upstream.


would you be to accomplish the effects of using `<br>` in css instead?


The length needs to be 1 or greater.


Now that have a few contributors I think we should all start using geopkg when we publish :) Then we can start making nice maps. @watson could you add it to devdeps and add a `npm run geotag` script?


cc @sorribas o/


@truenorth I believe that we need a `return` here otherwise when we call extend we get `undefined` back.


What's the status of this patch?


I don't understand why Pyinstaller needs this while Python doesn't. There is some import magic that happens at the end of pygments.formatters (see https://bitbucket.org/birkenfeld/pygments-main/src/652e08e410f4f4f3a61add91f75ce5ed9d134711/pygments/formatters/__init__.py?at=default#cl-114):

```
oldmod = sys.modules[__name__]
newmod = _automodule(__name__)
newmod.__dict__.update(oldmod.__dict__)
sys.modules[__name__] = newmod
del newmod.newmod, newmod.oldmod, newmod.sys, newmod.types
```

Effectively, Pygments uses the `_automodule` class (defined in `pygments.formatters` to rename its sub-modules. While this is convoluted, I don't know Pyinstaller's internals well enough to understand why the sequence `import pygments.formatters; from pygments.formatters import HtmlFormatter;` works, while the bare `from pygments.formatters import HtmlFormatter` fails -- I would expect both to work or both to fail. Any thoughts?


@thegreatgazoo yes you're right, this comment could be clearer.   It would be great if you could open a pull request to clarify the comment.  If you do make sure you update the `zfs-module-parameters(5)` man page as well.


I committed the fix. The delete_third_and_seventh function wasn't working.

I'm also having trouble with part 2 - some of the functions aren't working.
When I print the input and results of the function, it shows the correct
results, but the functions aren't passing the test. Could I please have
help with it?

On Tue, Jun 30, 2015 at 11:57 AM, Mipmap <notifications@github.com> wrote:

> Ah, good, can you commit the fix so I can review it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/animals4all/Exercise04/commit/b86fa7d6f731bcd630eabfbe4ad46f23f1efbe59#commitcomment-11935261>
> .
>


On Tue, 2015-06-30 at 09:13 -0700, Jeffrey Bean wrote:
> Was there a reason for moving the order of the proxy and kubelet?

Only to keep it in blocks by program.  Do all of proxy setup in one
place. Do all of kubelet setup in one place.

Instead of doing a little proxy, little controller manager, little
apiserver until eventually they are all finished.

Maybe it was better the old way, since it was grouped by the type of
activity. You tell me....


no hardcoded versions please


@DavidCaswell whoops. I misunderstood. Ok, I'll have to arrange for a visit sometime then.


```
ido-ubiquitous.el:816:21:Warning: `format' called with 2 args to fill 1 format field(s)
```


you are correct, mind opening a new issue (or even better a follow up PR)

Be sure to submit it against the stable branch


Ja, war er, der ist ja seit 0.23.5 dabei ;)

2015-06-30 17:46 GMT+02:00 AllanScheSar <notifications@github.com>:

> Utility übersetzt
>
> @ThomasKerman der GrapplingDevice war bereits übersetzt oder?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/KerbalSpaceProgramDe/KSP-Deutsch-Patch/commit/1231775d59fef300c5bea32eeb80705eecbc51b6


reack


Thanks. I'll fix it right now.


"after a database transaction is successfully committed"?


The string-like constructors parse the string and `Json::Value` constructor moves the value into `jv_`. I don't think I can combine all three.

The `char*` constructor is needed because both `Json::Value` and `std::string` can take a `char*` and a call of the form `json j("hi");` would be ambiguous.

It would be possible to combine the two string-line constructors using a template, but as is I use a delegating constructor and there isn't much code duplication.


thanks!


Fix for issue #3


FacilityTabGridView (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)


No, it will be 1 or greater.


Completed 500 Internal Server Error in 30ms (ActiveRecord: 3.6ms)

NameError (uninitialized constant RailsWorkflow::Uuid::Guid):
rails_workflow (0.3.1) app/concerns/rails_workflow/uuid.rb:10:in `generate_guid'

I had to add **require 'guid'** at top of **app/concerns/rails_workflow/uuid.rb** file in order to fix this


Dialog revision, added combo box and named it correctly.


No, the SymmetryVerifyingDataHandle only works for the symmetric interfaces. For everything else you should get an assertion. Did you run this in debug mode? You have to make sure that the macro `NDEBUG` is not defined.


btw I know changing copy to deepcopy on those was unnecessary, just being defensive.


Indeed the height has no effect at the moment and could be removed.
But `padding: 5px;` makes the last line more stretched, I think it doesn't look better:
![schedule](https://cloud.githubusercontent.com/assets/1684826/8436011/49655488-1f55-11e5-9fd3-80a7a5e9e94e.png)

But I'm not a fan of inline input boxes anyway. :-)


duration calculation null control has been added.


No I agree with your orginization. The only thing i have seen that might not go so well is when some services or configuration depend on another. In Kubernetes so far everything is nice and modular so it works well.


Oops. That message should've been for spot-report-servant. I cleaned up some code and removed some code that was implemented in spot-report-servant


If there is a way of returning "NULL" back to JS, that would be great.


Is this variable still used after your patch?


the following would be more compact:
```
responses.add(responses.GET,
              "https://maps.googleapis.com/maps/api/elevation/json?locations=41.889040454306752%2C12.525333445447737",
              body=json.dumps({
                  "results" : [
                     {
                        "elevation" : 42.70062255859375,
                        "location" : {
                           "lat" : 41.88904045430675,
                           "lng" : 12.52533344544774
                        },
                        "resolution" : 4.771975994110107
                     }
                  ],
                  "status" : "OK"
              }),
              match_querystring=True,
              content_type='application/json')
```


Kudos!


:D

On Tue, Jun 30, 2015 at 5:23 PM, espitall <notifications@github.com> wrote:

> "// fuck the rules !!!!"
>
> Ah ben bravo ! Je vous met un avertissement de fairplay.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robotter/eurobot/commit/f803bf59c91c04a2d3a0cf5108e843d5ffb7fd70#commitcomment-11934415>
> .
>


@betanalpha I don't understand how the following code would work:

```
#ifndef STAN__IO__SCALAR_UPPER_BOUND_HPP
#define STAN__IO__SCALAR_UPPER_BOUND_HPP

#include <stan/io/transforms/base_transform.hpp>

#include <stan/math/prim/scal/fun/ub_constrain.hpp>
#include <stan/math/prim/scal/fun/ub_free.hpp>

namespace stan {
  namespace io {

    template <double UpperBound, typename T>
    class scalar_upper_bound {
      static void constrain(T input, vector<T>& output) {
        output.push_back(stan::prob::ub_constrain(input, UpperBound))
      }

      static void unconstrain(T input, vector<T>& output) {
        output.push_back(stan::prob::ub_free(input, UpperBound));
      }

      static int constrained_dim() { return 1; }
      static int unconstrained_dim() { return 1; }
      static std::string base_type() { return "scalar"; }
    }

  }  // io
}  // stan

#endif
```

I don't think you can have double template arguments, and even if you could instantiate the templates, we have the runtime vs. compile-time problem, no?


Thanks for finding this! This is a great argument for adding the "strip-trailing-slashes" flag.

Could you create an issue requesting this flag? This will help make this visible to everyone.


It is fixed+squashed+pushed now.

(Thanks a lot for your patience, I'm not a native english speaker and these things are not always obvious to me!)


Since the value is only set when read from the file, setting it multiple times should not be a problem. So, a lock is nice to have but not really necessary.


why a new line?


This test examines the scenario of repetitive user import
```bash
$ nosetests -m test_reimport_us test_import.py -v
@test: Try to Import all users from the ... ok

----------------------------------------------------------------------
Ran 1 test in 63.699s

OK
```


Can't you use the cogwheel from the "in editor" animation?


This will save the execution time as we don't need to pull/unzip/delete the data set separately for every test (as we no longer modify the contents of the dataset itself)

![Image of histogram](https://lh3.googleusercontent.com/IvpmK3Cr1ntq9MtEI3Kq2Q2IFggEs3TN8vAc4GbbYn8=w870-h418-no)
[Execution times interactive histogram](http://codepen.io/rplevka/pen/yNpdXv?editors=001)
..pretty neat, isn't it?


Comma is missing


What is the advantage of removing this option and explicitly checking the return value?


commit 2ad31227e513a69f8f4ce7ab0ac838dbe65274f5


reack


r+


this new logo is awesome! :+1:


Using the gcloud version of kubectl will cause problems for developers, who may have version incompatible versions of the server code installed in their clusters.


Ca fait beaucoup de fucks tout ça


@deads2k I'm not clear what you mean here - `AuthenticationFilter` has exactly what it needs to implement `http.Handler`


Looks good!


Its not really unknown. Its a sprite mixed with a mask to get the feathered edges. A simple example of this is on the ride construct selection window each of the ride pictures are drawn with this function.


markdown


This is the first file created by anumbajoe


Oh...fair enough


gggg


In order to be able to differentiate the different handlers to identify them? Is this not what you asked for?


how to handle commas in a csv file? Thanks


Is image2 the mask?


lets also add a note inside of the .md file for layout


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


merging jedmccaleb/stellar-core/master = 1e3ad178 into auto


jedmccaleb/stellar-core/master = 1e3ad178 merged ok, testing candidate = 6e0f6dd8


> @deads2k I'm not clear what you mean here - AuthenticationFilter has exactly what it needs to implement http.Handler

Make an `AuthenticationFilter` struct that confirms to the `http.Handler` interface and see how it falls out.


Also 3.2 is repeated twice instead of 3.3


I'm aware of --> at the end, but my regexp skills does not go so deep :-)

The problem is that inside updateReferences all lines in the block are matched and without --> the 2nd line gets matched as attbs


Yes, the digest-name can be made optional.

All the available options can see found in the below file. Look for "const EVP_MD *".
https://github.com/jxcore/jxcore/blob/f7dcfdb39225a0d0773dcb930d1d2c18f640eab6/deps/openssl/openssl/crypto/evp/evp.h

The list can also be see at https://www.openssl.org/docs/crypto/EVP_MD_CTX_md.html.


uninitialized constant RailsWorkflow::ErrorWorker::Sidekiq (NameError)
from rails_workflow-0.3.1/app/workers/rails_workflow/error_worker.rb:3:in `<module:RailsWorkflow>'

uninitialized constant RailsWorkflow::ErrorWorker::Sidekiq (NameError)
from rails_workflow-0.3.1/app/workers/rails_workflow/operation_worker.rb:3:in `<module:RailsWorkflow>'

I had to add **require 'sidekiq'** at top of files
1. app/workers/rails_workflow/error_worker.rb
2. app/workers/rails_workflow/operation_worker.rb
in order to fix this


Ah - I missed this too.  Removing from back to front is a good thing to remember.  Looks good.


This will always pass because the `$` isn't escaped, so `$color` will be expanded before it is even passed to the outer test.


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2183 of 2213 tests passed)

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


Perhaps you should also mention this in the `index.html.haml` newsbar? It seems a bit outdated in general (“Don’t miss the UK JUG tour in February!”) :)


Yeah I think so. Had a look at the new_ride image and image 1 is definitely a mask.


saw approval from bkerley
at https://github.com/basho/riak_kv/commit/66a2c1b6b85192e23450fb076ef1cc0f7406d063


merging basho/riak_kv/temp/timeseries/ajs = 66a2c1b6 into borshop-integration-1147-temp/timeseries/ajs


basho/riak_kv/temp/timeseries/ajs = 66a2c1b6 merged ok, testing candidate = 2731cdd8


Hi Nico,

Can you please fix the code style before I merge it.

```php
}
        else if (!isset($this->_defaultRatingIdByStore[$storeId])) {
```
should be
```php
}  else if (!isset($this->_defaultRatingIdByStore[$storeId])) {
```

Thx,


ok


Fix for delete_third_and_seventh looks good, left a code review comment for that commit.

I'll come by and we can look at part 2.


yes - but in other chunk of code which I did not touch as unrelated to issue #404


Thanks :)


Let's clean this function first. There are many hacks that I noticed:

* You can use `vec_basic`
* Use the C++11 for loop
* There are too many pointers, I think you put one more star in there than you wanted
* the `array` argument should not be const if you are using it as a return value
* use the `RCP_cast` macro in this file to cast from C to C++

So here is the polished function:
```c++
void basic_get_args(const basic a, basic_struct **array, int *size)
{
    SymEngine::vec_basic args = (*RCP_const_cast(a))->get_args();
    *size = args.size();
    *array = (basic_struct*)malloc((*size)*sizeof(basic_struct));
    int index = 0;
    for (auto &a: args) {
        *RCP_cast(&((*array)[index++])) = a;
    }
}
```

Try it and let me know if it works.


thanks, committed a fix for this in the master repo at eclipse..can't bring into jetty.project on github, sorry


为什么安卓的，在点击微博登录授权时，直接提示停止运行退出app了。？


Ack


@betanalpha nvm, see you changed this code in the last commit on the branch.


appended two queries before writing to DB
dbSendQuery(mydb, 'SET unique_checks=0;')
dbSendQuery(mydb, 'START TRANSACTION;') (this is same as setting auto_commit to 0)


Been messing around with this also, would this also require a change in ./kalite/settings/__init__.py?

It has the old settings line of without armv7l...
CONFIG_PACKAGE = getattr(local_settings, "CONFIG_PACKAGE", "RPi" if (platform.uname()[0] == "Linux" and platform.uname()[4] == "armv6l") else [])


Well, after we upgraded to EF6.1.3 a couple of weeks ago, we noticed a couple of queries got slower.  It turned out that they had fixed a bug where EF would generate an INNER JOIN when it should have generated an OUTER JOIN.  In our case, the bug worked in our favor in a few spots.  So, now we are keeping an eye out for slow queries and looking at the generated SQL.  If it contains OUTER JOINS that we wished where INNER JOINs, we re-write the Linq to do an explicit .Join.  In this particular case, the block was slow for a couple of reasons, and the INNER JOIN vs OUTER JOIN was just one of them.

BTW, to see the generated SQL, I usually can just look at the ToString() value of the Queryable, or use Rock.Context.Database.Log... to print the SQL to the Debug Output screen


Pretty sure that's not how Copyright works


laul la bonne joke


This recursion is very inefficient but seems that this is only used for debugging, so guess is alright to leave as it is; I just needed to point it out.


1) I think just keep the encoding parameter in string adaptors for different memory layouts, like for NumPy or C-style fixed-size string buffers. Not sure what you mean by raw text, do you mean text encoded in values of the `bytes` type? I think for encode/decode we should do pretty much what Python 3 does.

2) Putting such a flag in the evaluation context seems dangerous, because it's an inherited thing, and that would mean other string code couldn't rely on the behaviour of basic string functionality without ensuring the eval context is set as they expect. I agree it's a potentially tricky thing, but offering the ability to switch between both approaches seems worse to me.

3) Yeah, to keep fixed_string. It can still include other encodings, I think that's valuable for interop. In interoperating with NumPy, you want ASCII and/or LATIN1 for the byte-level strings, and UTF32 for the unicode. Having fixed_string be an expression type whose value type is string might be what we want to do.


Your are welcome!


Thanks a lot! I wasn't sure whether to use C++11 for that, that's why I did that. I am trying this out.


Game Designed With SPL library under the cs50 Course


{} x 2


FacilityTabGridView (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)


even if it is not going to cause bugs because we use rewritten queries for caching, it would be nice to have good equals/hashcode on this class


But I would like to take the opportunity to advertise my [`auto-compile`](https://github.com/tarsius/auto-compile) :-)


Listo! Ya cambié a CST. (Era CST en vez de CT)


The names go away automatically for me...


ack


It can't really compare two queries because we miss the index reader from the search context, but we could eg. add a new field to this class: `private final Object identity = new Object();` and then we would use this object (only) in equals and hashcode.


Can you make it a hard exception?


```
width = (width + 15) & ~0xF;
```

depending on the data type.


No... The GitIgnore was added beforehand.


The comment says "grater (sp) than 5.6", but the code has greater than 5.7.  The comment is correct here, and it should be:  `int(version[1]) <= 7)` or `int(version[1]) < 6`.


I'm sorely tempted to call this a bug, see http://melpon.org/wandbox/permlink/FEgFTDWW8BzRdFKP. Clang flavors compile the program if `move_only` is an aggregate, but require a default constructor if it is not. GCC flavors don't require a default constructor for either case. I'm fairly positive - without having actually traced the behavior through the standard - that the initialization in `non_aggregate na[2]{non_aggregate(1),non_aggregate(2)};` and ` new non_aggregate[2]{non_aggregate(1),non_aggregate(2)};` is equivalent; the `new` case should be well-formed if the non-`new` case is.


Not 100% sure if this is to blame, but at present, machined seems to think any nspawn machines have terminated if you do a "systemctl daemon-reload".

Reverting this patch makes it work.

Will need to do more tests but I think this patch is the root cause at present.


Error: inflate failed [too many length or distance symbols] at file offset [5616172]
imgdiff only supports gzip kernel compression, did you try CONFIG_KERNEL_LZO?
failed to break apart source image
make: *** [/home/cuzz1369/source/candy5/out/target/product/ls990/obj/PACKAGING/recovery_patch_intermediates/recovery_from_boot.p] Error 1
make: *** Waiting for unfinished jobs....

Only happens on ls990, other variants compile just fine. Changed to LZO in 990_defconfig and compiled fine.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2150


To cherry-pick the commit to the master branch (don't forget to make sure your local master branch is up to date)...
```
git checkout master
git cherry-pick eec95af
```
...and to remove the commit and its changes from the Conversion_final branch, so long as there isn't any else commited immediately after it...
```
git checkout Conversion_final
git reset HEAD^
git checkout -- gui/pregame/backgrounds/hellenes.*
```
...which will require a force-push to GitHub.


-  [ ] let's have all args on one line


- [ ] same


確認しました。


hello


Thanks for noticing.
Have removed it now.


oops, by one after I meant one after the #, so "  # "


I do! Was curious what the new release brought with it.


Right! Thanks, done.


heyheyy!


Yeah, I'm working on that now. Sorry premature push to my own branch

On Tue, Jun 30, 2015 at 9:36 AM, richardhansen <notifications@github.com>
wrote:

> This will always pass because the $ isn't escaped, so $color will be
> expanded before it is even passed to the outer test.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grondo/sharness/commit/5459db845a613048c6ab1f27c203e9ae8ec51b9e#commitcomment-11936061>
> .
>


This is to minimal as it doesn't even have a autoload section which would be good for TYPO3 CMS 7.3+

Anyway, I removed the version information from the composer.json provided in my PR. Thanks for the comment.


No estás guardando los constructores en el objeto. De hecho estas guardando `undefined`, ya que los constructores llamados sin `new` no devuelven nada, no tienen `return`.


@nairdo do you think that how to view the SQL is worth adding to the Wiki?
On 30 Jun 2015 17:42, "Mike Peterson" <notifications@github.com> wrote:

> Well, after we upgraded to EF6.1.3 a couple of weeks ago, we noticed a
> couple of queries got slower. It turned out that they had fixed a bug where
> EF would generate an INNER JOIN when it should have generated an OUTER
> JOIN. In our case, the bug worked in our favor in a few spots. So, now we
> are keeping an eye out for slow queries and looking at the generated SQL.
> If it contains OUTER JOINS that we wished where INNER JOINs, we re-write
> the Linq to do an explicit .Join. In this particular case, the block was
> slow for a couple of reasons, and the INNER JOIN vs OUTER JOIN was just one
> of them.
>
> BTW, to see the generated SQL, I usually can just look at the ToString()
> value of the Queryable, or use Rock.Context.Database.Log... to print the
> SQL to the Debug Output screen
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SparkDevNetwork/Rock/commit/6516a0f19905560fa6d7153800a033fe89fcdbc6#commitcomment-11936169>
> .
>


no comment


> @ckhroulev: might be something for stable.

OK.


Nice :)


The `cwrapper.cpp` can (and should) use modern C++. The file `cwrapper.h` can only use C. So the arguments to the functions must be raw C, but inside it should use C++.


Doesn't filtering these keys mean there will a problem if table has field of same name and you are trying to filter by it?


Guess I'll keep doing these (https://github.com/rgbkrk/atom-script/commit/7f899016abb55112a656ff6ed111bb07cab052cc). I used to not be as diligent with the CHANGELOG, hopefully it keeps on keeping on here.


And btw. thanks much for that PR - I don't directly see the issue in the code that this fixes - did you PR this fix also upstream to the plymouth-lite people? I think they will also be happy.


H-m. I'd rework this class to use the mentioned technique, but we can do that with the separate JIRA, e.g. https://jira.spring.io/browse/INT-1174
WDYT?


sdfdsfdsfdsf


nice, +2


Lexicographical order please


Passing value of current allele as a string to the coding function (more efficient)


probably want to include both schemas


Great, thanks!


Ah, got it! Awesome. I accepted your proposal on Elance. Looking forward to working with you!


Looks good.


This is what happens when there is no constructor but bits and pieces of code doing the same thing over and over and there's no tests to see that a clone actually does anything sensible.


So it will place a multi-row widget in the *first* available row that doesn't overlap another widget, and that is not the last row of the _current player widget_ ?


The type being move-only is not significant, http://melpon.org/wandbox/permlink/7HPkPwPOzrk6VlQj has the same problem with implicitly-declared copy/moves.


Actually, could you test latest branch version and tell me if it is a bit faster.

Then I add a define to only support blue color in plugins/GSdx/res/glsl_source.h. I don't think I will enable it because some games uses others channel... Still I'm curious of the perf impact.


Did you tried to add
gem 'sidekiq'
gem 'guid'

to your Gemfile?


Sorry I've missed that. Was a bit in rush. I'll fix via next push. Because I've missed another point in this review: method ReplaceRequiresWithAssumInMoveNext should be commented out, actually.


xD @DeicPro lol thanks, i havnt contributed cause im busy learning C ;-; soz


You are right. We can remove the exception and just clean up (by calling HMAC_CTX_cleanup(&hmac)). If exception is needed, it is better to clean up first.


Great, but can you also add assertion that checks that the database has actually been renamed? Maybe fetch via the API to ensure that the name has been updated


add
gem 'guid'
to your application Gemfile


Hi @felixhayashi the talk is available online at http://tiddlywiki.com/talkytalky. Stupidly, it's not linked from tiddlywiki.com; we really ought to integrate it properly as a presentation edition:

#1853


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


please fix spelling issue "Analizer" to "Analyzer"


Correction: *not* a bug in the compiler, a bug in the language. https://llvm.org/bugs/show_bug.cgi?id=22924.


@akbertram here are assertions injected directly in rename method


This take a while to run on `vagrant up`. Shouldn't this only run when

```
    ssl:
      enabled: true
```
?


Add generic to class ResultSetAnalyzer<T>


This change is causing the sass issue. It seem that css files which start by a `_` are not included in the `stylesheets`.


good choice!


OK


You're right, strange ;-)


HVACTemplateTransition (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1679 of 1682 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)


LGTM


extract switch to factory method and use it.


Merged as instructed


I think this entire paragraph could be removed, with some minor changes to the next.


What's the replacement for this function now? Removal has broken my pagerank algorithm :(


It was documented functionality, but there were no tests for it. I know that sucks. Good call on binding the IIFE. Shoot me a PR, if I don't get to it first.


fast-forwarding master to auto = 6e0f6dd8


I thought about this, but IMHO because it's a one time operation and if you have multiple site, It would be complicated to check globally if you have at least one ssl site. So I decided to not make it optional.


@miguelportilla Thanks for writing that test. However, I'm clearly lost. We set up a transaction to pay bob $5 and delivermin $50. Bob was paid $5 and even tho thats < delivermin the transaction succeeds.

I was expecting to see some weird transaction where we used the same currency - say "USD" - and the receiver valued two issuers of "USD" differently. But a vanilla transaction with the same issuer can specify a delivermin > amount paid and that's OK? This isn't your fault, but I'm just lost. I need to look at the code again to see what `delivermin` does.


Is the project moving to Zend? If so I guess I need to start learning it.
Have fun and try not to let it hurt your head, cause it gave me a headache.
I noticed that the $fake_register_globals=false; and $sanitize_all_escapes=true; were removed in places. Are they not needed with Zend?
Terry      From: Brady Miller <notifications@github.com>
 To: teryhill/openemr <openemr@noreply.github.com>
Cc: teryhill <teryhill@yahoo.com>
 Sent: Tuesday, June 30, 2015 3:07 AM
 Subject: Re: [openemr] fix for export to windows (be5229d)

k, will look at them when have time (hopefully over the weekend). Have another massive review that am planning to start soon for MU2 stuff:
https://github.com/zhhealthcare/openemr/commits/CareCoordinationV3—
Reply to this email directly or view it on GitHub.


However, the barrier functionality is already tested in `AggregatorParserTests`, but I suppose we could add a test with timeout here.


As i suspected this testcase fails https://gist.github.com/ADmad/3ac625fa04a7cb163f91  This is an unacceptable limitation IMHO. I actually have table with column `group`.


Is there a reason this (and the face above it) are in group "helm" instead of "helm-faces"?


hold on a second!


No problem man :). studies need take a lot of time to get good results ;), sure you are the best of your class m8, so better i don't wish luck because you don't need b8 lol


I wonder if it will work like we expected to replace it.
Anyway: are you going to do that right now or do we leave that work to that JIRA ?


I would change deliver to 'run' or 'execute'. I think the 'three ways' here is more consistent with the rest of the doc and does a good job of illustrating the types of benchmarks in charms.


In both GCC and Clang's standard libraries, `<iostream>` transitively includes `<string>`, but you can't (at least, shouldn't) rely on this!  It's better to explicitly state your dependencies and add a `#include <string>` at the top; if someone modifies `<iostream>` to not include `<string>` (or you switch to C++17 modules, which we mentioned in passing in the lesson), your code will break!  This is a problem with the current model—lack of _isolation_.

For what it's worth, it's not likely that this specific code will break, because `<iostream>` really does need to include `<string>` to work.  But it's still a little bit to subtle to rely on.

There is actually a tool to help you check this, and it helps especially in larger projects.  It's called [include-what-you-use](http://include-what-you-use.org/), and it can detect a number of these violations.


@marutosi thanks. I will use your patches in https://github.com/opf/openproject/pull/3194 - 0b0dc1ee11e48376ca4b52fb05c3508a8519eabd


should be 'pip install -U charm-benchmark'


@mafintosh done: 772dff2d569ec343416318f942617d34dd65df3b

I usually like to add the coordinates to the same commit as the version bump - I don't know if this is good or bad - but anyways, this means that my current publish workflow is something like this:

```
$ npm version patch
v1.0.1
$ npm run geotag
$ git add -A
$ git commit -C HEAD --amend
$ git tag v1.0.1 -f  # ensure the tag now points to the new ref
$ npm test && git push && git push --tags && npm publish
```

It's a bit cumbersome, but works for me. But I'm looking into improving it :stuck_out_tongue_winking_eye:


yes, PR on its way


Je pense que tu peux supprimer ce fichier vu que tu as une redirection en principe il ne sert a rien


You broke the blog feed, man ;)


`SC.MODULE_INFO` is not populated anymore. I guess it comes from this removal.


idem que create


yep, PR on its way


Es mejor hacer render sólo en las áreas que fueron modificadas, no sobre todo el lienzo


I couldn't figure out why that `""s` syntax was working without you explicitly `using namespace std::literals;`.  It turns out that `std::literals` is what we call an _inline namespace_, which means when you `using namespace std;`, all the things in `std::literals` also get brought in.  We'll talk about these in several weeks, and I don't want to get into them now.  I don't know whether I like not `using namespace std::literals;` here or not, but it's guaranteed to work. :smile:


Ok but current code only tells first toolkit version that was used to edit not the current one.


> I guess for the time being we just throw our hands in the air and only check new[] in DefaultConstructible.

I suppose so. I couldn't find a core issue for this, FWIW. I might ping Richard.


As expected, this the previous commit broke a large number of tests. Trying a more conservative approach here. https://github.com/finnlabs/awesome_nested_set/compare/v2.1.6...v2.1.6-rails4


when you get primitive is OK to use int but if you have String use java.lang.String


The rewording of this line carries much less information than the previous wording did.


As with above, `EXIT_SUCCESS` is [defined in `<cstdlib>`](http://en.cppreference.com/w/cpp/utility/program/EXIT_status), so it's a good idea to explicitly include that header.

The link I gave just there is good for figuring out which header to include where, because it has those nice handy “Defined in header `<cstdlib>`” lines.


(IMO effort is better spent here on upgrading to awesome_nested_set 3.0, but that is definitely out-of-scope for this PR - https://community.openproject.org/work_packages/20672)


je ne suis pas sur que ce index de booking soit pertinent. on aura jamais une vue avec tout tout tout les bookings. Mais on va avoir des tableaux, avec par exemple tout les bookings au dejeuner pour le lundi 23 septembre.
je sais pas si c'est clair.... et je ne sais pas du tout comment faire ca !


Other JIRA


A noter que ceci sera une modal


It would have to take a maybeSid because of the case where there is no Service header (in which case the mDefaultProxy is consulted). I think it's reasonable to have two functions because these two use cases (no sid, default proxy vs. sid as per header) are quite different. But I might change the code to just pass the whole config in and let the lookup functions handle the rest.


fichier à supprimer


This is solved by the `jsx-uses-react` rule of eslint-plugin-react.


@coderanger that seems correct, though I haven't thought through the implications. Do you have an example use case in mind @coderanger?

Also cc'ing folks like @danielsdeleo and @lamont-granquist who might have thoughts on the impact of making the resource guard fully emulate the execution semantics of a resource when multiple actions are specified.


OK. Merging after the lunch!


Now we do it in the same way as the A3 backend, which is not bad.


This makes a very good point: when the input has exactly the `sha1BlockSizeInBytes` we don't need to do anything. ;)


Este proyecto está perfecto para un framework de automatización en el que estoy trabajando. Deberías incluir un README de como hechar andar este proyecto 👍


The previous wording was carefully precise, and covered the full behavior including savepoints. At any given point in your code where you call `on_commit`, that hook will be run if and when a database write at that same point in your code would be committed.

The new wording is imprecise (which database transaction? Any database transaction?) and inaccurate in the case of a rolled back save point. It is possible to call `on_commit` and have that hook not be called the next time a transaction is committed, if a savepoint is rolled back in the meantime that was active when `on_commit` was called.


It doesn't make any sense to me.
There are a few reasons sol_flow_send_boolean_packet() may fail:
  * you've done an error while writing the code, passing a null node or invalid out port - in this case you certainly don't want to send an error packet saying so =D
  * no memory to alloc the packet (not sure if it's an issue for boolean packets specifically) - but probably you'll fail to alloc the error message later to send an error packet anyway.

So I agree with Gustavo, you're mixing things. Error packets weren't designed for this purpose.

-2 for this pr


What happened to the license?


Great !


Can you point to where it is broken in your code? Does `v'*M` or `M*v` work for you as a replacement for `v*M`?


You should handle situation when resultSet.next() return false in this case we don't need to perform other steps. We can return null this mean that table is empty.


LGTM.


This should be an assignment r31 := constant(instruction->endAddr()).


This is the way I told you to do the input, which is great.  But what happens if the user types in `Patrick Niedzielski`?  What's the output of your program?  Is this within the specification or not?  A separate question: is it okay behavior or not?

You don't need to change this, but it's an interesting line of questions to ask.  Without looking at the documentation for `cin`, describe what features you would want from it so you could change your program to take the input `Patrick Niedzielski` and output `Hi Patrick Niedzielski!`.  If you were to change the program in that way, are there bogus inputs that would be allowed?  Is this okay or not, in your opinion?


@miguelportilla Ah, I looked at the code. That test succeeds because the actual delivered amount == requested amount and the delivermin check is bypassed. You're going to hate me, but I don't think that tests the case we want. Can we have a test where the check is exercised (actual amount != delivered amount) and delivermin > requested amount.


can you add a comment that we can't check for actual equality since we miss the index reader that we will fetch from the context when rewriting?


What about adding geopkg as npm `preversion` script?


Lot of text :wink:, but the interesting part is already in section 2


Actually, this move and the copy of operand0 on the previous line are not ordered, so, you have no guarantee that operand0 first will not be moved and then copied.


@miguelportilla Also, I think the current test that is there should fail, not succeed. But check with David for sure.


it appears to search all descendants. using a mashup with `mda.streams`, i can do:
```r
> query_item_in_folder(text="ts_wtr_nwis", folder=locate_folder("sites"), limit=3)
```
and get
```
        title                       id
1 ts_wtr_nwis 5580b18ce4b023124e8f0255
2 ts_wtr_nwis 55843be7e4b023124e8f54b0
3 ts_wtr_nwis 55844af2e4b023124e8f5680
```
where those returned children are 2 levels down.


Please replace all tabs with spaces.


Is it grammatically correct? I couldn't parse the second half of the sentence. That's all I asked to be revised.


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2180 of 2216 tests passed)

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


Someone made a comment that the translation is not quite accurate, please see here:
https://www.phpbb.com/community/viewtopic.php?p=14127336#p14127336
Would you have time to update your fork with the master (I made some structural changes), and redo the translation? I guess in particular the last text is a bit off. The hidden text cannot be revealed by only logging in, one must also leave a reaction in the topic to unhide the text. As it turns out, the translation tells otherwise?
Thanks :) .


este


1174 is a different beast...

>allows an asynchronous receiver to pull messages out of it.

In that case, it's up to the consumer to decide when to check if the group is releasable rather than when the last message arrives (or group timeout).

So, maybe we shouldn't deprecate?


We haven't learned enough about CMake to fix this, but when you type `make test`, it waits for user input, so it's not completely automated.  Of course, `echo "Patrick" | make test` would work, but it makes a little more sense for the test case to supply its own input, not the person executing the test, right?


yes i can, and i'm learning about httr as we go. see what you think of my update to this PR in a sec


Yeah i had similar problems.  The default behavior is trailing, where repeated calls resets the timer, and only _after_ the timer has expired (calls have ceased for a long enough period of time) does the specified function get called.  Leading is different, in that when called, it executes the function immediately, and then similarly starts a timer which is reset by subsequent calls.  Only when the timer has expired will the function be available to fire again.

But, it seems that specifying leading behavior does __not__ inherently disable the default trailing behavior.  This caused a bug, which pat found, where the event was firing both at the start and after the timer expired, causing double events.  Apparently it needs to be explicitly disabled :|  Note to future debouncers!


This expression designates the value of size 4 bytes located at address zero in the global memory.
Is this really what you want?


Irnos going to ve swallowed by the query object anyway. None of those fields can possibly make it to the options array


The gallery page doesnt have the big pink spinning asterisk on the left of the page


@Robbie1977
I'd added the iframe with the google group vfb_community to the FAQ page some time ago, forgot to mention at the meeting.
You have added a VFB support group, from the helpmail, under the tab Support forum. I'm not sure if you filter these, or if it is showing all helpmail. But I'm not sure if it is good policy to make all helpmail public when people are not aware this will happen, and might not want to. What do you think?


Cool. Might just be useful to note in documentation that it recursively searches sub-folders as well.


Why we need "new String" we already have field Value String?


The following program would also pass the test:

```cpp
#include <iostream>
#include <string>

int main() {
  using namespace std;

  string name;
  cin >> name;
  cout >> "Hi !\n"s;
}
```

You don't have to change this, because we'll soon be abandoning this simple way of specifying tests in CMake.  But what attribute am I demonstrating about the test case here?


**getattr** --- функция, которая обрабатывает запросы вида wordgraph.attr, где attr --- имя атрибута. При такой реализации нельзя вызвать никакие атрибуты, кроме **longestwords**. То есть все остальные методы класса используются только внутри программы, но не могут быть вызваны извне класса.


Awesome! Thanks!!


@pelme no need for apologies, it's mostly my writing and not yours that you're being asked to fix up here :)


Sorry!


Actually you can spare the 'cat', use $1 as grep arg. Can you change it slightly ? Then I could really credit you for the patch :)

PS: re-fetch the repo, I've added a test for this feature, which as test-based programming requires, fails. It should then work once you implemented the feature.


aici e un ș


OK so realized that arm-eabi-6.0 is not arm-linux-androideabi but was bundled in with the commit, was able to build this so no need to go back and change the context of the commit, IMHO


current implementation triggers an error, change to Process.parse_run_options(*args) will do the job unless you intend something different here?!


Чтобы потом писать в коде a & b, а не a.intersect(b). __and__ перегружает бинарный оператор "и".


LGTM


Should this be `MONGODB_BSON_NAMESPACE`?

/me ducks


Why we need this new class we can add this functionality to ReflectionAnalyzes and reduce many classes. What do you think?


OK cool.

-e.

On Jun 30, 2015, at 11:54 AM, Lasse Fister <notifications@github.com> wrote:

> I'm still working on it.
>
> —
> Reply to this email directly or view it on GitHub.
>


Adoptă un ț.


I'm trying M' * v as a workaround - this works in my older version (passes tests) but the latest nightly is building right now.

Issue is here: https://github.com/JuliaGraphs/LightGraphs.jl/blob/master/src/centrality/pagerank.jl#L18


ok


looks like we have some tests failing on this.  did you git pull origin master to make sure you branched from  the latest code where tests were passing?


Great ! :) (don't forget to link this commit to the issue)


@timgraham you mean is it correct to use "is successfully committed" in reference to a hypothetical database write? Not sure; that could be changed from "is" to "would be" I guess? Is that the part that's causing you parsing trouble?


In this function, I was using an array of `const basic_struct *`, that's why the extra `*`. Using an array of `basic_struct` should also work. Since I was getting a vector of `RCP<const Basic>`, as a return value of `get_args()`, I tried to preserve the constness and hence the `const` qualifier, but that didn't make the variable `array` const. Upon using this as the function
```cpp
void basic_get_args(const basic a, basic_struct **array, int *size)
{
    SymEngine::vec_basic args = (*RCP_const_cast(a))->get_args();
    *size = args.size();
    *array = (basic_struct*)malloc((*size)*sizeof(basic_struct));
    int index = 0;
    for (auto &a: args) {
        *RCP_cast(&((*array)[index++])) = a;
        //debugging functions
        char* str = basic_str(&(*array)[index-1]);
        printf("%s ",str );
        basic_str_free(str);
    }
}
```
and this as test
```cpp
//basic array is declared as: basic_struct *basic_array;
//Tests basic_get_args
    integer_set_si(e, -123);
    basic_pow(e, x, e);
    int size = 0;
    char* l = basic_str(e);
    basic_get_args(e, &basic_array, &size);
    printf("basic_get_args(%s) :\n", l);
    int i = 0;
    for (i = 0; i < size; i++) {
        char *str;
        str = basic_str(basic_array+i);
        printf("%s ", str);
        basic_str_free(str);
    }
    printf("\n");
```
I get the following result on executing the test
```
Symbol : x
Integer : 123
Basic : y*(456 + x)/z
Basic : -y*(456 + x)/z**2
Rational : 100/47
Is_a_Symbol 100/47: 0
Is_a_Rational 100/47: 1
Is_a_Integer 100/47: 0
integer_get_ui 123: 123
integer_get_si -123: -123
integer_get_mpz(e): 123
*** Error in `./test_cwrapper': malloc(): corrupted unsorted chunks 2: 0x0000000001983e20 ***
Aborted (core dumped)
```


2ț: https://www.youtube.com/watch?v=YyF0yvDX2ww


Perhaps you could try an `is_first_row` approach, as the current stuff seems to work for cards that are 2 rows, but not cards that are 3. Good progress though, further than I got in a day for certain!


These entries are already available in rails_workflow/Gemfile. Do I need to add it to my_rails_app/Gemfile as well ?


Ok nice it will do the trick :+1:


That would explain it. ;)


I'm feeling bad about asking @pelme to go back and forth committing and squashing over our wording disagreements though :) I'm fine if you commit like this and then I may propose some clarifications in a separate PR?


I'm ok with this. The only situation where this would work is if there user doesn't specify any assetName or assetType, expecting it to publish to an existing asset, but the asset type wouldn't match task type.

That's why the piece of code was there in the first place.
* Find user specified asset
* Else: Find asset that matches default assetName and assetType (derived from task)
* Else: try to find any asset on the task
* if we still didn't find anything, create new asset.


Another call of that method was not fixed with that commit.

Is fixed in commit ddb726ba28afcd8228318f7ec266a23c8393048c


Right, but in that case how the custom find works needs to rethought.


good call. i don't know if the implementation is truly recursive - could be some neat SB pointer magic - but i agree with the idea. so i just now added "and any subfolders" to the doc.


I'm ok with this, however one situation where this wouldn't work is if user doesn't specify any assetName or assetType, expecting it to publish to an existing asset, but the existing asset type wouldn't match task type.

That's why the piece of code was there in the first place.

Find user specified asset
Else: Find asset that matches default assetName and assetType (derived from task)
Else: try to find any asset on the task
if we still didn't find anything, create new asset.


Cool, just fix the indentation in this file and put the new chrpath dep and we're gold.


Looks good!


Method getObjectFromResultSet should return Object or also should be generic?


You need to add a ton more. :earth_americas:


You should never have a space between the method name and parentheses. This is the source of a bug. `my_method (arg1, arg2)` and `my_method(arg1, arg2)` are not the same thing. The former will blow up.


Remove this line?


You mean in he cake core?


Hi to all at cyanogenmod
is there anyone who can fix the problem with frequency scaling  issue ive posted in issues.. in this repo..
never saw any response


tagged the issue in the wrong repo, was supposed to be for

https://github.com/chendaniely/multidisciplinary-diffusion-model-experiments/issues/79


Use your street name


I guess pagerank is mainly on real data, but the equivalent to the old method is `M.'x`.


Consider using "++" and "--" instead. For branch you could use "+-" or "//"


great outline.. a heading to show what it does it would be nice ,something like //this is how tables are created


Should it be "no-contracts" or "contracts"?


the corresponding issue in mann: https://github.com/chendaniely/multi-agent-neural-network/issues/114


How is this related?


You can also do `expect(assigns(:user)).to be_a_new(User)`. This will verify that it is a new object of type `User`.


Not sure what this is about?


can you please remove this third line?


Can you include your name?


No, this behavior :)


New logo - much better! New 3D-crystal - much worse! The old one has a crystal clear vibe to it. This one looks like something dragged out of a coal mine in Wolfenstein.


How was the coffee? Can you add an update to the code to add the best coffee shop?


can u remove third line


Should include what to order: Whole-wheat deep dish Savage.


We could expect all filters to be nested inside another key on the options array.


second line remove


For eg. you can pass the search query as `find('search', ['searchQuery' => $this->request->query]);` to avoid this issue.


old name conflicted with stdio.h


Shouldn't this be Myles Dyson?


resolved implicit declaration warning


don't use two names for something! (like `energy` and `action_points` or `position` and `location`)


line note


Can we rename this to `api/services/sequelize.js`?


bug fixed


not every page has this. to mix things up, each page has an asterisk somewhere on it, at varying scales. on the gallery page it's in the upper right corner.


Sorry, @andreasnoack - is `M.'x` the same as `M' * x`?


Let me know what you see. I was using cmftStudio to generate my test images, but DDS in particular is much too deep a format to support all of so I definitely missed some permutations.


Or `api/services/database.js`


testing new comment on commit


Did you mean #519?


built fine here =/


LGTM


This is interesting.
This is the faster version so far. No doubt. Probably due to avoid using mxArray structures within the data wrangling. To do this... only slows everything down. I shall remember this. mxArray only input and outputs... as long as its possible.


Since the project has nothing to do with signing in, I would not have that logic here. I would recommend having a `set_active_project` method instead.


This is not correct add a #


I am terrible at markdown


Questo me l'ha aggiunto da solo Eclipse, si toglierà al prossimo commit


hello @allisonchau cc: @ajschumacher @jonhanke
Good job on the python exercises!


@ozra the old logo is pornographic, showing everything. The new one is erotic.


use the following details in the application


Randy Morris <notifications@github.com> writes:

> Is there a reason this (and the face above it) are in group "helm" instead of "helm-faces"?

No they should be indeed in helm-faces.
Done, thanks.

--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997


I am sorry for nit-picking, but why not add prefixes to `self._lm_states_names` instead?


commenting on this file


commenting on this line of code


Check out my Bio


This illuminate/support version does not work with php version 5.4! You should change it for require the illuminate/support 5.0


You have to run it in gdb to figure out where it segfaults. However, obviously you can't call the `basic_get_args` as follows:
```
    basic_get_args(e, &basic_array, &size);
```
Because you are assigning to the location (calling malloc inside). You need to pass in a pointer.


can you remove this seccond line?


@snoyberg I'm running a warp server that sends EKG metrics to Carbon using https://github.com/ocharles/ekg-carbon. The `defaultShouldDisplayException` function made hard for me to debug a `ResourceVanished` exception thrown by [`flushSample`](https://github.com/ocharles/ekg-carbon/blob/c6e5f8d2351d7f5f269dc1f24463b1778246077e/src/System/Remote/Monitoring/Carbon.hs#L109). Wouldn't it be more useful to just print all kinds of exceptions?


can you remove this third line?


@phucnguyen60749 Có tý xíu vậy cũng sai nữa trời @_@ 2 tháng = 60 ngày. Code để ý giùm em.


This re-opens #842 because `autoFocus` isn't set to `false` on the ModalStatic.js example.


jkicillof: With regards to the adjectives - I'd definitely label them in reverse. Or rather , the new is sleazy, the old is erotic.


hello @Audryc cc: @ajschumacher @jonhanke
You're right, lambdas Are Awesome!
Good work on these exercises: 3-6


I dont think it makes sense for us to remove this second line.


Thanks for the PR.  The developer who owns this area is out on vacation right now, I will have him take a look and validate when he gets back.


Included what to order.


Commenting on line3


ya will do that


@mmc46 I think it's probably fine to have that stuff open – we can move anything that doesn't belong.

We should try to use this forum ourselves when we have questions about how to use the site.

Best, Greg.


The array allocation requirements in Copy/MoveConstructible also indirectly require the target constructor to be non-explicit, since array elements are copy-initialized, which I think is a useful property. My best idea to check syntactically that a move/copy constructor is non-explicit is to require argument passing to work, but it's a but ugly.
```
// MoveConstructible
requires (T t) {
  [](T){}(move(t));
}
// CopyConstructible
requires (const T& t) {
  [](T){}(t);
}
```


renamed


Oh yes, sorry my bad. I am correcting it and trying with gdb.


I really don't like the indentation system of github. This code should still work disregarding the strange indentation.


Can you add an empty line in between?


Hi @juleskk cc: @ajschumacher @jonhanke
Good job on the python exercises.  There is one item that is not complete.  Let me know if you have any questions on it.

Explain list comprehensions. Give examples and show equivalents with map and filter. How do their capabilities compare? Also demonstrate set comprehensions and dictionary comprehensions.

REPLACE THIS TEXT WITH YOUR RESPONSE


There is a need for new output.


const removed, reference removed?


Why?


I'm with @FHenry, there are different ways to use projects; in my case millestones is the best one.


Great! I digg it.


merge Emre from Master.


why


No. `M.'x` is the same as `M.'*x`. The point is that the elements of `M` are not conjugated when using `.'`. For matrices with real elements, there is no difference between the two.


Thanks! I'll do this slightly differently, RunThreaded should just do nothing for an empty list instead of throwing an error.

See https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/355 .


Oh, I see. It's moved to sub-makefile.


nice work


Correct, just being consistent.


Everything in part 2 is working except for the reverse function. The
assignment wants you to create a custom version of the reverse() function
and reverse the list in place without returning it, but this means that the
reversed list isn't saved when the program exits the custom function. The
function passes the test if you just assign the input list to reverse(input
list).

On Tue, Jun 30, 2015 at 12:38 PM, Mipmap <notifications@github.com> wrote:

> Fix for delete_third_and_seventh looks good, left a code review comment
> for that commit.
>
> I'll come by and we can look at part 2.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/animals4all/Exercise04/commit/b86fa7d6f731bcd630eabfbe4ad46f23f1efbe59#commitcomment-11936101>
> .
>


doh


да я бы вообще фразу заново перевел :) И начал фразу с императива "вы должны", а не с condition term Ж)


@mpreisler :+1: Thanks for the contribution! We've pulled it into [GerritHub](https://review.gerrithub.io/238125) for formal review.


Да, согласен.


Yup. Pas encore trouvé pourquoi. Je vais sûrement P/R `eslint-plugin-react` demain.


there's something funky about this code.. level is never updated (infinite loop?) and idx is being reset to zero at the beginning of the loop despite being set to newIdx later on.


@ozra when you say crystal and draw a crystal there's not much room for imagination. When you see a shape with something that looks like a hole but ends up being a glitter you have the opportunity of completing the idea. That was what I was trying to explain with those adjectives. Anyway thanks for the critics.


Alternate fix is merged as of https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/commit/3418284a6a50c531b04e79e62c99af93083b5227 .


At first, I was uncertain about using Wet/Dry as well. So, I started to see if that was used elsewhere and what some alternatives would be when I learned that it appears to be fairly standard terminology in the audio signal processing world. Do you know what it does in the implementation? It mixes the processed output with the original signal. 100%, the default, means only the processed output. It is kind of like saturation. Therefore, I do not think boost/cut makes sense with positive and negative values because that implies 50% is the normal/default, and I do not believe that is normal in audio processing. Maybe "level" but even that feels a little strange. I can see how a log action with dB units would work and make some sense. What kind of tangible information/evidence about the right thing to do here do you have to share? The current behavior derives from the JACK-RACK GUI tool, which was written by someone with an audio background, and I felt more comfortable following his lead than trying to guess a better way.


What should be the return when my array has only one element?

```
> new TiTao().tail(['vitti'])
[]
```


I delete some of the values of the beginning and the end of the analytic signal. so that phase differences are not taken into account here. This will mitigate Hilbert transform's border effect.


This is committed here: https://github.com/voldemort/voldemort/commit/b9b21f462ede254dd3a485e0a0f0ffa6cd4463b3


Thankyou, yes I did!


but in switch we have to use only int, string or enum


@waleedjaffar that's correct.


need to delete readme edit


I see what your sentence means now. I came up with, "The function you pass in will be called immediately after a hypothetical database write made where on_commit() is called would be successfully committed." or we could keep your original version. What do you think? I don't mind amending the wording when committing this.


thank you. I'll give this a try now.


Why 10 and not 5? Add an argument to indicate the number explicitly and rewire the calling code to use that one.

I would call it ```refSourceNeighborhood```.


@jkicillof - mean no offence, beauty is one of those things obviously in the eye of the beholder. Once again great work on the logo :-)


Do you need to check that vertices does not contain nulls?


did previously the code in the brackets got executed, no matter what packagesExist(IlmBase) yielded?


@ulferts I'm only seeing this issue when running specs locally `rspec spec/models/work_package/work_package_rebuild_nested_set.rb`


@alexmarsev: I'm trying to finish this patch though I'm pretty busy IRL those days. I need to pass the currently used DLL file to the crash reporter so that I can load the right language in case of a crash. To do that I need to notify the crash reporter of language changes.

We have an event for that. Do you think it's better to have the main frame doing something like:
`if (CrashReporter::IsEnabled()) CrashReporter::Enable(langDllPath);` or to register the crash reporter to receive the event directly? It must be noted that the `CrashReporter` is just a name space so it is pretty basic currently. Additionally the first language "change" (the crash reporter is created before the settings are loaded) does not trigger an event so it has to be handled manually anyway.


ok


ack :) that's good


disappointed that this came in as one big commit and not a list of sequential commits. Wanted to parse through what exactly was updated.


Is this for the galaxy s5 fingerprint scanner?


Ah ok I see what you mean. That's a possibility, yeah.

I personally like that we can say "this store is at version 234" rather than "this store is at version 1.4 billion", but you're right that it would be nice to have consistent versions across clusters.

I would rather not change this as part of this commit, but we can discuss it...


@ozra thanks! The new logo also looks better on tshirts.


Get request should be a public method: it should allow people to use the method with endpoints we don't explicitly support--like the rates endpoint. My suggestion is to remove the get_rates method and the unsigned tokens_request method, keep the unsigned_get_request as a public method and have it take a path as a parameter so that unsigned_get_request("/rates") and unsigned_get_request("/tokens", payload) both work.


If you rebase on top of my latest branch, you don't need to make this `startAnimating` call. The spinner will animate automatically.


Fix the place as city not a country


Committed here: https://github.com/voldemort/voldemort/commit/b9b21f462ede254dd3a485e0a0f0ffa6cd4463b3


The Python interpreter ensures that reading and writing basic data types (such as integers or strings) are atomic, so concurrent writes of the same value should be fine.
However, I would like to add a lock in the future because changing this variable to some other type may cause it to lose its atomicity.


Nie mialem jak zrobic znaku paragrafu :D


Cocoa prefers method arguments being named, so believe it or not, a better name for this might be:

```objc
- (void)mapView:(MGLMapView *) didFinishRenderingFrameFully:(BOOL)fullyRendered;
```

Since you're not actually passing the frame.

/cc @1ec5


{}. Can't it just call addAll?


@saintedlama Then npm unfortunately just complains that the "Git working directory not clean."


Same.


I want to print which command failed, which is the main purpose of the run_command() function.


ok


-> "listening for the app"


Hi @benvanik! What was the IE issue here?


oops!!


+2 go on...


Is this named after `-[MKMapView mapViewDidFinishRenderingMap:fullyRendered:]`?


Good catch. This was because we had to pass the number as an object with the `performSelector:withObject:afterDelay:` previously.


I never got a chance to test the tweaks to made to it, did they not work very well?
I tried it over the weekend with

    var splitSpeed = cell.getSpeed() + 80;
    split.setMoveEngineData(splitSpeed, 45, 0.9);
and it seemed to look half-decent...


This also worked for us. Thanks !


This script should run on the target cluster and not on the user's workstation. Are you saying that the default "kubectl" in the target cluster may be incompatible with the underlying Kubernetes?