The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/database/myisam-to-innodb](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/database/myisam-to-innodb)
- [docs/articles/sites/errors-and-server-responses](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/errors-and-server-responses)
- [docs/articles/sites/migrate/wordpress-site-networks](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/migrate/wordpress-site-networks)
- [docs/articles/sites/quicksilver](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/quicksilver)
- [docs/articles/wordpress/configuring-wp-config-php](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/configuring-wp-config-php)
- [docs/articles/wordpress/hyperdb](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/hyperdb)
- [docs/articles/wordpress/site-networks/getting-started](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/site-networks/getting-started)


I tried playing around a bit but that was the cleanest solution I could come up with. If you've got any other ideas I'm all ears ;)


delete this line


:laughing:


found it!


What's wrong for checking directly for Private, instead of not the other two?

Also these three sections here are almost identical code. Please make a lambda or something and remove the code duplication.


<deathRespawnDied>¡Has muerto!</deathRespawnDied>


This is way smoother than I thought it would be. Nice job!


Wie wird dies verhindert? Oder meinst du, dass dies nicht getan werden **soll**?


@katherineng no problem! Always glad to give back to the community!


tests/docs (as noted in first review) for this?


what is leanModal() ?


Good organization and no obvious issues. I would just recommend if you're using relative measurements like `vh` and `vw` then use relative measurements like `em` instead of `px`. Not a huge deal, and you get more fine grain control with px, but just know that pixels will be different across devices depending on pixel density and resolution


Since it's exported, and to match the other functions in this module,  ```get_test_dir()``` should be just a regular function, not a class method.


What is this supposed to hold exactly? Could you comment on this line, please.


You are right, `host->name` must not be NULL after `curl_win32_idn_to_ascii()` has been called.

I think that `curl_win32_idn_to_ascii()` should really return 0 if it has failed to convert the host name. I have created https://github.com/curl/curl/pull/637 for this.


Your SQL wrapper seems to be vulnerable against SQL Injections. Use `prepared statements` as described in  Guideline 3-2 on <a href="http://www.oracle.com/technetwork/java/seccodeguide-139067.html">Secure Coding Guidelines for Java SE </a>.


Wie heißen diese Ordner?


> It is possible that curl_win32_idn_to_ascii() returns 1 and ace_hostname still be NULL

That's the issue to address. Discussion moved to #637


Wow, does `comments_` hold all the comments of all translation units??? Isn't there a way to only keep the comments of the current translation unit?


```assert_using_test_db``` might be a better name, since it's not validating input, and it dies if the assertion fails.


Check your spelling.


Both methods have an authority parameter, naming one "WithAuthority" suggests the other doesn't. Maybe "buildUriWithBase"?


If I was to checkout the codebase at this commit, then your codebase would be broken since jq_ajax.php doesn't exist in the repo till four commits later:   https://github.com/swash99/ims/commit/24683690c6c36eab6d7520458c926516699d9091

You should have added jq_ajax.php and switched the sql_common.php calls to jq_ajax.php in the relevant places in *one* commit.

Currently, not only are you pushing out hundreds of commits, they are also not in any logical order or structure making your commit history a disaster. I'm wondering if you even know that you can add multiple files in one commit? Each of your commit only ever has one file even if its related to five other commits/files.


In welchem Ordner? Wo muss das Steuerprogramm erweitert werden? In welcher Zeile muss was eingefügt werden?


When you write a function name in prose, convention is to not write the parentheses.


:like:


This commit would be best broken into smaller chunks. There's a lot going on in here.


The program fails to run because of this line.


Pasted the docker response when no Exposed Port is set. Note the relevant field. Docker, however unfortunate, is not consistent with when it wants to return an empty list/map and when it wants to return null.

```
"Config": {
        "Hostname": "92e5c83152bb",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": false,
        "AttachStderr": false,
        "ExposedPorts": null,
        "PublishService": "",
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": null,
        "Cmd": [
            "/hello"
        ],
        "Image": "hello-world:latest",
        "Volumes": null,
        "VolumeDriver": "",
        "WorkingDir": "",
        "Entrypoint": null,
        "NetworkDisabled": false,
        "MacAddress": "",
        "OnBuild": null,
        "Labels": null
    }
```


You can remove comments here


TeamCity pull requests :: netty-tcnative [Build 58](https://garage.netty.io/teamcity/viewLog.html?buildId=5481&buildTypeId=pulls_netty_tcnative&guest=1) is now running


It seems like there are too many style rules being applied to IDs in here. ID styles should be used sparingly.


Removing this dead code as part of the clean-up.


not OS replace `/` in `\`? Oh...


You're alive? =O


Why are these values hardcoded? Each method (e.g. `dbInfoInsert`) should accept a `table` parameter instead.


Huzzah for outlining code into separate functions. :)


Windows batch is horrible


TeamCity pull requests :: netty-tcnative [Build 58](https://garage.netty.io/teamcity/viewLog.html?buildId=5481&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:44


109658788_LowTemRadSys (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


This change should've been made in one commit along with the function name change you made 6 commits later: https://github.com/swash99/ims/commit/ec09a760631b41d863d00f39fc1e87814a92703f

Your codebase is now broken for the next 6 commits since you seemingly randomly changed the function name to something which does not currently exist.


Fixed it on commit https://github.com/FoysalOsmany/loopback-connector-mssql/commit/2f87f507d072fb8581279f807ba4d791f565117b


Needed to make line 100 and line 103 mimic the same functionality that was in the account edoc. This change has been verified locally.


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


no need for the `else` here. If the `$item` isn't an array then just use it :)


@MasterOfDesaster Im Klassendiagramm sollen die  privaten Methoden  nicht aufgeführt werden. Es fehlt auch noch der Verweis vom GParserTest zum GParser. Von Main aus gibt es keine Assoziation zu Download und CopyThread?


Good job! In the head tag we can add <meta name="viewport" content="width=device-width, initial-scale=1"> to make our page responsive. This sets the scale of relative measurements to fit within the device's viewport. Look it up online, but it's a good technique to use on every website to make sure things look the way you expect.


@obeliskos yeah @VladimirTechMan we're starting to question you exist. Say it, it's skynet isn't it. It's real.


ye


I did not think it was necessary because this comes straight from the BIOS, but I guess we can't be too careful in this case. I'll use `g_strstrip()` which removes both leading and trailing whitespaces, and we still need to remove the terminating `\n` first, as none of these functions take care of this (please correct me if there is a better way to do so).


The switch to jq_ajax.php should have been made in one commit along with the addition of jq_ajax.php. I'm repeating this in all your commits so you understand the significance of the problem.

The aim of a commit is to make logically related changes in one go -- it doesn't necessarily mean you need one commit for each file.


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


We talked about this in class. Please fix this!


The Problem with TSpline3 is that it does qubic(?) interpolation. It
therefore "overshoots" sometimes. I mean when it interpolates between
(x1,y1) and (x2,y2), sometimes the interpolated curve between these two
values is higher than max(y1,y2) or smaller than min(y1,y2). That means you
cannot really rely on the interpolation to be at least as good as no
interpolation at all and you have to check it visually every time. Does
this make sense?

On Fri, Feb 5, 2016 at 5:30 PM Max Schlupp <notifications@github.com> wrote:

> Can you remove the TMVA dependence? We do not need it here. You should be
> able to use TSpline3 instead of TSpline1. I think there is no fundamental
> need for linear interpolation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gammacombo/gammacombo/commit/b7877491fb9217d5d94af90469a41a8229a82461#commitcomment-15918495>
> .
>


Why is the vendor code 0x**2**eb37e91 (Mackapar Media) when on the official DCPU spec and the M35FD it is actually `0x**1**eb37e91(Mackapar) ? Is there a specific reason or is it an error?


Ha, force of habit!  The scripts actually do pull the source strings from en-GB, FYI.


This switch to jq_ajax.php should have been made along with the addition of jq_ajax.php file in one commit


extra space


Refactor-Rich-CoolingTower1Spd (energy-plus) - x86_64-MacOS-10.9-clang: OK (1879 of 2051 tests passed, 161 test warnings)

 * 331 tests had: AUD diffs.
 * 329 tests had: BND diffs.
 * 195 tests had: MDD diffs.
 * 212 tests had: MTD diffs.
 * 232 tests had: RDD diffs.
 * 168 tests had: Table big diffs.
 * 10 tests had: ESO small diffs.
 * 10 tests had: MTR small diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.

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


I like this file. Things make more sense now.


Please fix the typo :)


Yes. That's the pre-existing behavior.


Hi,
I don't think this change is necessary - you can produce mass lists for all your scans, and chromatogram builder will run just fine.


Yeah, I'll clean that up in both content and source.


nice, this fixes the build issue in eclipse also


yikes youre right.


Just noticed: there shouldn't be the `/docs/XX/YYY/` part in the URIs in the ToCs anymore. Please rebase on top of master.


TeamCity pull requests :: netty-tcnative [Build 59](https://garage.netty.io/teamcity/viewLog.html?buildId=5482&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Yup.


Thank you !


This project is to process large set of the date to meet the requirement of bank problem


Waren das alle Änderungen?


To be honest, I'm not totally sure what that would look like yet. Hoping we can create a full example via https://trello.com/c/I7EujTnt/59-try-creating-masonry-files-for-a-real-application.


TeamCity pull requests :: netty-tcnative [Build 59](https://garage.netty.io/teamcity/viewLog.html?buildId=5482&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:56


names inverted.


oh yeah, duh!


Jay, free node class in Santa Monica tomorrow -
http://nodeschool.io/los-angeles/

Can you share with Tommy and Jenny? I don't have their emails.

Thanks,

PEH2

On Fri, Jan 22, 2016 at 6:06 PM, JayJohnson1 <notifications@github.com>
wrote:

> Thanks for the quick reply Phil. I'm going to research Bootstrap snippets
> now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kingluddite/emerging-technologies/commit/d1b972161194b4506336ab65250e18d5948dab21#commitcomment-15627377>
> .
>

--
Philip Howley - Web Developer Consultant


Your SQLite database holds sensitive data (such as private keys) in clear-text. You should encrypt it. Don't put this as high priority though.


Please make sure to include new routes.php file in this directory too.


This change would break the current behavior for most users. Why don't we make it conditional? If peak.getDataFile().getDataRTRange(1) returns non-null Range, let's use it. If it returns null, let's use peak.getRawDataPointsRTRange()


Но ведь он останется при выборе любого из пунктов select


Hi @gusmanb,
Thank again for the commit. I have reviewed it and I have committed an update https://github.com/pgstath/Sharp.Xmpp/commit/24445c75426bcd7626558099a35a2a1c367c369d at the dev-GCM-pull14 branch.
Some changes, in relationship to your commit:

1. There was  a bug during connection for StartTls connections (no stream was created)
2. Some CodeMaid automatic code styling (e.g. addition of private)
3. I would prefer to name UseRoster, as RetrieveRoster in order to be inline with XMPP terminology. I have changed the variable name, but if this a blocker for you let me know.
4.  Last but not least, I was having a look at the BodylessMessage event. Is there a need to define a new event, additional to the Message event? Essentially your BodylessMessage event should replace the Message event. What do you think?

Please test https://github.com/pgstath/Sharp.Xmpp/commit/24445c75426bcd7626558099a35a2a1c367c369d  against your codebase. If it is OK I will merge it to main, and mark it as a new version, since XmppClient constructor is not backwards compatible now.


Why are SQL query executions implemented in these if blocks? Don't equivalent functions already exist in sql_common.php?

The idea of a function is to pull out frequently used functionality into logical blocks. This hides the implementation details and makes your code more readable and easier to understand. Example:

if (isset(POST[newRole]) {
       return update_user_role($_POST["newUserName"], $_POST["newRole");
}

Isn't the above a lot simpler?


lodash


follow up, do we need a TODO to hook in our checkretry behavior?


I'd just use the Groovy elvis operator:

    String exposedPorts = container.config?.exposedPorts ? container.config.exposedPorts : '[]'
    logger.quiet exposedPorts


add transition easing


Add Hover state and transition ease


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [source/_partials/search_box.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_partials/search_box.html)
- [source/_views/changelog.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_views/changelog.html)
- [source/_views/guide.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_views/guide.html)
- [docs/articles/sites/database/myisam-to-innodb](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/database/myisam-to-innodb)
- [docs/articles/sites/errors-and-server-responses](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/errors-and-server-responses)
- [docs/articles/sites/migrate/wordpress-site-networks](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/migrate/wordpress-site-networks)
- [docs/articles/sites/quicksilver](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/quicksilver)
- [docs/articles/wordpress/configuring-wp-config-php](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/configuring-wp-config-php)
- [docs/articles/wordpress/hyperdb](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/hyperdb)
- [docs/articles/wordpress/site-networks/getting-started](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/site-networks/getting-started)
- [source/docs/search.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/docs/search.html)


The more comments the merrier ;-)


This change seems to be unrelated. Why did you remove that ?


lgtm


Perhaps I don't completely understand how this is working...but I believe the aim is to asynchronously update the user role via AJAX and be done. Where is this 'return' returning to?


I don't think this is right -- are you sure you aren't passing floats/complexes or some other type inside a pmt::blob/u8vector when it could be natively passed in a f32vector? see the following experiment

pdu::make_pdu_vector maybe should probably be type specific here if it is not already --

In [17]: import pmt
In [18]: import numpy as np
In [19]: a = pmt.to_pmt( np.array( [1,2,3], dtype=np.float32 ) )
In [20]: pmt.is_f32vector(a)
Out[20]: True
In [21]: pmt.length(a)
Out[21]: 3L


@azertyfun Good catch, I missed that completely. I'd like to imagine I would of caught it when implementing... Updated @6e16ce0b


what are you doing with singleClass.students after this


is o(n^2) the best we can do here?


Yep, missed that as well.


just spacebar


Hi, when I compile this toolkit, I met some problems below. can you give me some advice to solve it?

=-----------------------------------------------------------=
creating /home/jren/Repos/CNTK/build/release/.build/Source/Readers/LMSequenceReader/SequenceReader.o for with build type release
mpic++ -c Source/Readers/LMSequenceReader/SequenceReader.cpp -o /home/jren/Repos/CNTK/build/release/.build/Source/Readers/LMSequenceReader/SequenceReader.o -D_POSIX_SOURCE -D_XOPEN_SOURCE=600 -D__USE_XOPEN2K -DCPUONLY -DUSE_ACML -DNDEBUG -msse3 -std=c++0x -std=c++11 -fopenmp -fpermissive -fPIC -Werror -fcheck-new -Wno-error=literal-suffix -g -O4 -ISource/Common/Include -ISource/Math -ISource/CNTK -ISource/ActionsLib -ISource/ComputationNetworkLib -ISource/SGDLib -ISource/SequenceTrainingLib -ISource/CNTK/BrainScript -ISource/Readers/ReaderLib -I/opt/acml5.3.1/ifort64_mp/include -I/home/jren/opencv-3.0.0/include -MD -MP -MF /home/jren/Repos/CNTK/build/release/.build/Source/Readers/LMSequenceReader/SequenceReader.d
cc1plus: all warnings being treated as errors
Makefile:555: recipe for target '/home/jren/Repos/CNTK/build/release/.build/Source/Readers/ReaderLib/ReaderShim.o' failed
make[1]: *** [/home/jren/Repos/CNTK/build/release/.build/Source/Readers/ReaderLib/ReaderShim.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: Leaving directory '/home/jren/Repos/CNTK'
Makefile:6: recipe for target 'all' failed
make: *** [all] Error 2


+ will automatically coerce track._id to a string no need to toString()


I thought vector is supported as well?


I actually have no clue what happened here LOL


I would rather use either `_id` or `_thread_id` here. But it's your call :-)


That sounds definitely like @tobibeer :)


If a linker script has already been created the rule to create a linker script will not be executed. In that case the whole build would fail needlessly. If on the other hand the linker script does not exist the build will fail anyways because DEVICE is not defined even if this is just a warning. This warning is created no matter if the linker script is built.


You don't actually need to do this, `Comment.new` is fine. In the end, it doesn't really make a difference in this case, since you only need the `Pin#id` to setup the route for this form, and Rails already has it from the `@pin` instance variable. When you create a `[@pin, Comment.new]` resource, you're basically telling Rails: this is a form, using the POST method (since the last item in the array isn't persisted in the database), to the `/pins/:pin_id/comments` route.


It would be more useful to explain what function the if block is performing instead of where it will be called. You may end up in a situation where you utilize the same if block in multiple files.


inline style.. tsk tsk


Yes. those are in the next patch, as you already noticed.


Is this a work in progress, or why is it ignored?


It'd be a different case if the route wasn't nested, in which case you'd need to pass the `Pin#id` inside the form, as a hidden field:
```
<%= form_for @pin.comments.build do |f| %>
  <%= f.hidden_field :pin_id %>
```


Begrüdung für die Resitroerimg der Domain qsc.ffhh

Neues Handle für  **Q**uantum **S**erver **C**omputing . ffhh


muhh


I believe 'newUserName' is incorrect. You are not updating the username, just the role for the existing user. Programming is all about logic and things making sense. You could just do username: username


@bmuschko already tried that. It's not the config object which is null but the exposedPorts, which in turn is a method call, which calls another object that may/may-not be null. Pasted the relevant code snippet below from docker-java so that it will make more sense. Perhaps I can access the private property and then test from there. I generally don't like doing that but it may make things more cleaner.

```
    @JsonProperty("ExposedPorts")
    private ExposedPorts exposedPorts;

    ...

    @JsonIgnore
    public ExposedPort[] getExposedPorts() {
        return exposedPorts.getExposedPorts();
    }
```


What was causing this to happen?


This should fix issue #50 related to animation inconsistencies.


I don't see the point of having these static builders.
Why don't we just use appropriate constructors directly?
(I know we have these patterns throughout the code but I consider those redundant too.)


Do you have an object named models in your workspace that is not created in
all.r?  I restarted R and tried again after merging in the most recent
version, without any success.  Aaron is also experiencing the same problem.

On Fri, Feb 5, 2016 at 9:44 AM, andrew-edwards <notifications@github.com>
wrote:

> Seems to work for me, and we look to have the same latest commits looking
> at your network graph (I’ve added a couple of minor things).
>
>
>
> You did source(“all.r”) again, yes?? That’s all I can think of,
>
>
>
> Andy
>
>
>
> PS You may get this email twice as I CC’ed you, and not exactly sure if a
> simple reply would work...
>
>
>
> --
>
> Dr. Andrew Edwards
>
> Andrew.Edwards@dfo-mpo.gc.ca <mailto:Andrew.Edwards@dfo-mpo.gc.ca>
>
> http://www.chebucto.ns.ca/~english <http://www.chebucto.ns.ca/~english>
>
> Marine Ecosystems and Aquaculture Division, Pacific Biological Station,
>
> Fisheries and Oceans Canada, Government of Canada, 3190 Hammond Bay Road,
>
> Nanaimo, British Columbia, V9T 6N7, Canada. Tel: +1 250-756-7146. Fax: +1
> 250-756-7053.
>
>
>
> La station biologique du Pacifique, Division des ecosystemes marins et de
> l’aquaculture,
>
> Pêches et Océans Canada, Gouvernement du Canada, 3190 chemin Hammond Bay,
>
> Nanaimo, Colombie-Britannique, V9T 6N7, Canada. Tél: +1 250-756-7146. Fax
> : +1 250-756-7053.
>
>
>
> From: allanhicks [mailto:notifications@github.com]
> Sent: February-05-16 9:17 AM
> To: cgrandin/hake-assessment
> Subject: Re: [hake-assessment] Merge remote-tracking branch
> 'cgrandin/master' (b6fa67d)
>
>
>
> A variable models was added to all.r, but when I try to build, R cannot
> find models. It was introduced here, and I don't understand the code enough
> to figure out a fix. See this list(models[[base.model.ind]]) in sensitivity
> section.
>
> Currently, I cannot run all.r.
>
> —
> Reply to this email directly or view it on GitHub <
> https://github.com/cgrandin/hake-assessment/commit/b6fa67d8c9fd19ab86f8820aae82aea090f5159f#commitcomment-15919753>
> . <
> https://github.com/notifications/beacon/AGp2pv_oTAmEZ49rcf0F0UGVuy8VaGPaks5phNB7gaJpZM4HUc9C.gif>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cgrandin/hake-assessment/commit/b6fa67d8c9fd19ab86f8820aae82aea090f5159f#commitcomment-15920385>
> .
>


Minor:  'create' and 'new' are redundant. When you create something, it will always be new. create_message() would be sufficient.

Code is simply logic and your thought process written down. When you read/write code, ask yourself if it makes sense or if you can do it in a different and better way.


ng-style? you would use this if you want to set a style conditionally based on the result of an expression or tie it to a scope variable. if you're just assigning strings then you can use an inline style, or css class


Doesn't that require adding ratpack-groovy?


is this from upstream?


yify.is API is faster


Your `javadocs` are nearly empty. You need to make sure you fill at least a short description.


I think we should open an issue for Docker Java on this. The method should guard for `null`.


112796613_DesicDehum (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2060 of 2060 tests passed, 1 test warnings)

 * 7 tests had: AUD diffs.
 * 1 test had: Table small diffs.

![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)


Received. I will have a look at it as soon as I have some time

Le 05/02/2016 21:10, Massimo Santini a écrit :
>
> You should have received it from BB, but the repo is
> https://bitbucket.org/mapio/pygraphviz-anim/ it's absolutely not
> commented and not ready to be distributed (even less than this
> project). You can get the idea looking the |copy_with_layout| function.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/remi-pr/GraphvizAnim/commit/5adf133110b7df9ee749d64765282f8c81784261#commitcomment-15923921>.
>


Don't know what you mean. Maybe the class methods confused you? https://github.com/AlchemyCMS/alchemy_cms/blob/f857d93ba40eab10ff789ebec07de4b17999ed9b/app/models/alchemy/cell.rb


oh, we already are....hm, are we ok with adding `ratpack-groovy` to the Java version of this project?


look into using a filter here


todo, or background?


as above


Minor:  If you updated the lines above, you could make these lines into one PHP block as well! :)  Your decision though.


Anti-pattern to eval user input.  Suggest something like:
```python
forever = raw_input("> You want to live forever? (y/n)") == "y"
```


So a child item will scroll off screen.


filter!


are all those defines here wanted or did they slip in for some reason? (looks like something for the xcconfig file if even...


I don't believe this part is necessary:

> ... and an ansible.cfg file at ansible/ansible.cfg.

I have never touched an ansible.cfg file, but regularly override global roles with local ones. I can't find the documentation on it though...


hmm blank line crept in here?


Instead of directly doing all this logic, I broke it out into a createTableRow() function for possible reuse later (independent of creating the table). This could be useful if say you need to add a new row of data but don't want to refresh the whole table.


The change to jq_ajax.php should have been made throughout the codebase in one commit along with the addition of jq_ajax.php. This line of code and whatever functionality it performs would be broken from the point you moved functionality out of sql_common.php to jq_ajax.php, until this commit.


tu pourrait te rebase ta branch d3d12 et ajouté ces commit dessus aussi stp ? ^^


Just one small fix (sorry, I haven't noticed it before): without `yields`, in other examples we just write output as is.


Hmm, why did you add this? It could potentially mask undesirable cases of nil err, nil value


Instead of directly calling createTable() at this point, we'll add an event listener/callback for when the window tells us everything is 100% loaded on the page. That way we can be sure that the html elements are all there and ready to be wired up with interactivity


is it by intention that those are nowhere deleted?


oops


Yeah, that's not supposed to be called as a class method, so I just now added another commit for `Genome/VariantReporting/Framework/TestHelpers.pm`.

The name of the package under test still needs to be sent as the first argument though so that `_data_dir()` can determine the name for the test data directory.


timestamp = datetime(year, month, day)


I moved the scripts up into the head of the document for clarity/loading purposes


where'd all these magic numbers come from?


This division with an id "tables" will be the container we add the generated tables into, instead of directly appending them to the body. This could give us more control/flexibility if you were adding stuff around the tables later


Agreed.  I changed the name.


Not quite understand the logic here, could you explain more why we need `loadNextPage ` here?


When these ```close()``` fail with throwing IOException, the following lines will not be executed and not be closed.
We should ```try...catch``` individually and accumulate error and throw.


yes


This was a fixed typo in the file.


The deletion of all these if blocks should have been made in the same commit where you added jq_ajax.php. Then its obvious that you moved the functionality out of this file into another file. Not to mention that you had duplicate code in between the ~7 commits where jq_ajax.php and this logic existed together.


You could also post to Confluence (as an attachment if necessary) with page restrictions, but I think that's probably a worse choice than DocuShare (but might be better than a private GitHub repo).


This was just for code consistency. :+1:


Wouldn't it be `catch { foo()?.bar()?.baz()? }`?


@bberenz FYI


All interpolations are assumptions and you can always construct scenarios, where one is better than the other.
However, I can only see scenarios where this gets relevant, when you have not enough toys  for a smooth 1-CL curve. In any occasion, when the interpolation between the scan points has a big impact, the scan if poorly configured! Therefore I prefer cubic splines, because it looks better.

TMVA is not needed anyways. And this is the crucial point. Linear interpolation can be performed via `TGraph::Eva(x)`, I guess this function also is able to do cubic interpolations with something like `TGraph::Eva(x,3,"")`


are you suggesting i let it panic, or t.Errorf instead?


change of behavior - doesn't match the commit message - just to be sure - you know about git commit -p to group chunks in one file into multiple commits do you?


manic mode :) I think a lot of those can get removed. we just have not gotten around to it. topshelf was a bitch to get working right.


Found it:

http://docs.ansible.com/ansible/intro_configuration.html#roles-path

> The roles path indicate additional directories beyond the ‘roles/’ subdirectory of a playbook project to search to find Ansible roles.

My reading of that is that the roles subdirectory of a playbook is the first place that is checked for a role. That supersedes what is set in roles_path. Since that's where you are recommending the roles be placed, I think the documentation pertaining to ansible.cfg can be removed.


Let it panic, probably.


"Επιλέξτε και καταχωρήστε μία τιμή για κάθε πεδίο εισόδου και πατήστε  «Υποβολή».


Good change -- the foreign keys were necessary. Make sure to go through the codebase and add any other necessary foreign key relationships.


TeamCity pull requests :: netty [Build 1222](https://garage.netty.io/teamcity/viewLog.html?buildId=5483&buildTypeId=pulls_netty&guest=1) is now running


follow up - can we use DefaultConvert with some flag to ignore the mismatched type to avoid hand-copying all the matching fields?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.494.0](http://build01:8888/viewLog.html?buildId=14115&buildTypeId=ConfigInjector_CI) is now running


that one can get nuked. was added in noise blip search and slipped in.


this is https://godoc.org/github.com/coreos/etcd/client#CancelableTransport, copied so i can set Transport.TLSClientConfig


TeamCity pull requests :: netty [Build 1222](https://garage.netty.io/teamcity/viewLog.html?buildId=5483&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:29


TeamCity pull requests :: netty [Build 1223](https://garage.netty.io/teamcity/viewLog.html?buildId=5484&buildTypeId=pulls_netty&guest=1) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.494.0](http://build01:8888/viewLog.html?buildId=14115&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:35


Thanks!


surprised the commit checker allowed this


Could this one word break our Travis test?


TeamCity pull requests :: netty [Build 1223](https://garage.netty.io/teamcity/viewLog.html?buildId=5484&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:26


Yeah, entirely dropping it might be a good idea. Because, like you said,
nobody's using it.
However, Socorro is open source and if you're reasonably good at coding
Vagrant we'd be totally up for some contributions. Meet us on #breakpad on
irc.mozilla.org and we can chat.

On Fri, Feb 5, 2016 at 2:48 PM, Vishnu Mohan <notifications@github.com>
wrote:

> Yeah, that worked. Thanks. But I noticed that there was a whole bunch of
> dependency errors I had to work through to make it finally work. When I
> selecting the Vagrant development option I was expecting a painless
> experience. But it ended up being worse than setting it up on a fresh linux
> box. Maybe you guys should get rid of the Vagrant option if no one is
> updating it.
>
> On 4 February 2016 at 18:45, Peter Bengtsson <notifications@github.com>
> wrote:
>
> > @vishnumohanm <https://github.com/vishnumohanm> Hmm... I wasn't aware of
> > that. We're not using 2.6 any more in production and I don't use Vagrant.
> > The whole point of pipstrap is to make sure you get pip8 installed. If
> you
> > can comment out the python tools/pipstrap.py and replace it with pip
> > install -U pip, does the rest of things work?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mozilla/socorro/commit/e376bb92a45e89a5532d14a34c2c5527a48708fd#commitcomment-15885425
> >
> > .
> >
>
>
>
> --
> Vishnu Mohan
> Computer Science and Engineering (2008-2012)
> Govt. Model Engineering College
> Cochin-21
> +919916582948
> vishnumohanm@gmail.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla/socorro/commit/e376bb92a45e89a5532d14a34c2c5527a48708fd#commitcomment-15923328>
> .
>

--
Peter Bengtsson
Mozilla Tools & Services


come on - this is xcconfig stuff - how should i backport that xcode mess


Your commit message doesn't say why this was deleted. Also, this delete should have been part of the commit where you added ims_schema.sql. Again, the aim being that all related changes and logic are grouped in one commit.


TeamCity pull requests :: netty [Build 1224](https://garage.netty.io/teamcity/viewLog.html?buildId=5485&buildTypeId=pulls_netty&guest=1) is now running


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


TeamCity pull requests :: netty [Build 1224](https://garage.netty.io/teamcity/viewLog.html?buildId=5485&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:28


sounds like i forgot to change the click listener....


OK, this is a nicer algorithm (and more efficient) but the exploding at https://github.com/SpiritLevel/silverstripe-docsviewer/commit/e443e9fa0b93012949be2a1fa4d719a6235f32ed#diff-31fd09995298dd94e6a11820a8a4cee5R323 doesn't work ideally in all cases. I've tried a preg_split using the main three regexs but there are problems recapturing the delimiter.


Iniciamos con el proyecto


can this be formatted more nicely possibly?


Ah. Hm. Perhaps a comment to that effect?


Fixes #1


humm, never thought of that, thx.


dont wanna use sass? if you have the time you can refactor this to be smaller


Up in smoke


Thank you very much!

I have noticed a lot of problems that were caused because of leftovers on custom/Extension/modules/MODULE/ for customizing fields that once were there but currently were deleted.

When that happens it's also very difficult to track where the problem is exactly.

I should have reported this as a bug before but, well, did not.

Thank you again.

This will make Suitecrm more resilient.


still incorrect, newest firmware 5.7.2, now prints:

* Beyond Legacy Code, P1.0 (Zbigniew)_445E83FCA6544B93B5203A17EF443B89.azw3: EXTRACTING COVER:

and hangs, then print disappears but program still deadlocked


I'd think that building with --enable-single-host would always test more things.  Do you really have to disable single-host to run clang?  Is running clang so important?  I had to emphasize to the students in my class to pre-install gcc on Mac OS X and build Gambit with it because clang couldn't build Meroon in any reasonable amount of time.

So unless people want to report and follow up with clang errors with Gambit-generated code, it doesn't seem worth it to me.


The Tools directory will always exist because init-tools.cmd, which is called by clean.cmd, creates it if it doesn't already exist.


> follow up - can we use DefaultConvert with some flag to ignore the mismatched type to avoid hand-copying all the matching fields?

We should pursue this: https://github.com/deads2k/origin/pull/22 instead.  Any solution like the one I introduced here is limitted to same group, same version.


Good catch! I pushed an updated commit (aa01225) with this fixed.


should be..?

```go
if got != nil {
  continue
}
```

test 2 in the table returns nil, err


this also looks weird


Great! Tanks


You could try loading in the models again from scratch - i.e. answer Y to all questions, except N to smart load? If we've finalised models then this won't need to be done again. I will send a .pdf shortly...


'strict' is not a field of account_lines.


sgtm


ouch. it turns out that now I cannot read the data files on windows anymore (does fopen default to text when I read? stdio is the worst!)


The RPC::accountFromString default param value of strict is false.


good job


From http://www.open-std.org/jtc1/sc22/wg14/www/docs/tc1.htm:

> Where an argument declared as size_t n specifies the length of the array for a function, n can have the value zero on a call to that function. Unless explicitly stated otherwise in the description of a particular function in this subclause, pointer arguments on such a call shall still have valid values, as described in 7.1.4.

I don't accept that you've identified a real issue here. Sorry.


проверено


added double footer


Same as above.


I fixed the mentioned issues and added some initial tests. I discussed with Pascal about the right way to add the extra row/column when the upsampling ratio is even. The current approach may not be correct. I need to fix that and then add the tests to verify it.


chartTypeMap[chartType]()


oh that makes me sad. My beloved tutorial. Not longer in markdown.


I don't think a `toggleClass` method exists on DOM elements, does it?


"Για να απομακρύνετε τις τιμές που έχουν εισαχθεί πατήστε «Επαναφορά» (reset)"


@codephillip Thanks man.


Missed the start and end of line anchors and that allowed some regex patterns to be broken


"Λάθος τιμή εισόδου"


So um... is this just attempting to implement Knockout?


Please send me the problematic file to "listy at blaut dot biz". I have to deeply test this case...


this.resize = setTimeout.bind(window, anonFuncBefore);

//later
resize(200), resize(100);


One thing to consider is that clang is *necessary* to embed objective-c code to interface the mac using cocoa. So even though it is painful to compile gambit using clang, it is necessary for some programs.


'strict' is not part of the command.


@wifelette Thanks, this was on my todo list for today, just hadn't gotten to it yet :-/


oh lol.


Is this comment intended to be here?


Just a question out of curiosity? Why remove rmse? (it was the main loss function for our experiment ;p)


I did more investigations on this.
The documentation says *64-bit signed integer*. Xrdp has a bug and send 32 bits unsigned, this was my mistake.


They're stable, or else the entire class must be marked draft. There is no
way to use zlistx without these methods.

There is no value IMO in zproject current bindings trying to support old
versions of CZMQ. If you need a new stable release, we make it.

On Fri, Feb 5, 2016 at 2:59 PM, Patrik Wenger <notifications@github.com>
wrote:

> @hintjens <https://github.com/hintjens> They haven't been released yet. I
> marked them as draft so the Ruby binding generator of zproject will
> generate code that gracefully handles the absence of these functions, which
> is the case for anyone using a stable release of CZMQ.
>
> With your change, CZTop can't support the current stable release of CZMQ.
>
> What do you suggest? Handle the absence of any function gracefully,
> independently of its declared state?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zeromq/czmq/commit/99b2b7daef9afdc3cc65e583c6bec7faac38e6b8#commitcomment-15914329>
> .
>


From the aforementioned 7.1.4 as of http://www.open-std.org/JTC1/SC22/WG14/www/docs/n1570.pdf:

> If an argument to a function has an invalid value (such as ... a null pointer ...)


I see what you are saying. This does not necessarily demand that all entities use the same DB so much as it would demand that all DBs be able to interact with every possible identity, but you are right that this could be onerous and unnecessary.

The one issue with what you have here is that the CRUD interface would involve functions acting on specific types; it may be best just to assume that entity backends will have to specify their own CRUD operations.


Do you still have a large .RData file in doc/r/  ?
If so then load that in, and it should contain your last 'models'. If you're restarting R from scratch then models won't be in there.
Chris is fixing it, but this would be a good first step.


Like in the `ElementViewHelper` the options are not meant to be passed to the view helper. So I will just mark the argument as not used


fixes arkenthera/ChiikaIssues#4


testeee


it's to fire off the initial load when the adapter is registered with the recyclerview


i dont like your file


Seems incorrect issue referenced


The help page as it is I think needs to have javascript.


oh this newline can be dropped


Thanks.  Will fix.


I think this how it changed after pandoc. I guess it is trying to put 72 chars/line. But I'll correct this


can you use `this.$$(".urth-widget-error")` instead?


I didn't know about the latter point. Of course it's stupid to have the
tmva dependency and I will fix that.

Personally I'd prefer linear interpolation because it's simpler and
therefore easier to understand and less error-prone. But if you would like
to insist on cubic splines I'll implement that.

On Fri, Feb 5, 2016, 21:58 Max Schlupp <notifications@github.com> wrote:

> All interpolations are assumptions and you can always construct scenarios,
> where one is better than the other.
> However, I can only see scenarios where this gets relevant, when you have
> not enough toys for a smooth 1-CL curve. In any occasion, when the
> interpolation between the scan points has a big impact, the scan if poorly
> configured! Therefore I prefer cubic splines, because it looks better.
>
> TMVA is not needed anyways. And this is the crucial point. Linear
> interpolation can be performed via TGraph::Eva(x), I guess this function
> also is able to do cubic interpolations with something like
> TGraph::Eva(x,3,"")
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gammacombo/gammacombo/commit/b7877491fb9217d5d94af90469a41a8229a82461#commitcomment-15924942>
> .
>


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


Este es el Vouter que deberíamos tener en el bundle HatueySoftSecurityBundle... Este que contiene todas las reglas generales. Y el que está en HatueySoftSecurityBundle sería el que hay que pasar hacia acá.


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2092 of 2092 tests passed, 1 test warnings)

 * 7 tests had: AUD diffs.
 * 1 test had: Table small diffs.

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


It wasn't working for me, I didn't see any input and keyup events?


again


@cala I have good DM West and DM North data. In fact, I've started respawning DM West from scratch a bit over a year ago or so, similar to what I did with DM East. But I ran into a few core limitations - mostly related to pooling and formations - so that's on pause because I'm refusing to work around the core limitations and spend all that time only to get a so-so result. In all honesty, it would still be a serious improvement and many people wouldn't even see the remaining shortcomings ingame, but I just can't get myself to continue like that.

Note that I'm not complaining here. It would be (too) easy to criticize the core, but that wouldn't help us, nor would it be appropriate after all the hard work done by so many and given to us for free. Let's just say I'm about to do something crazy.


Wouldn't [`$groupsAndSelf.addAll($groups)`](https://docs.oracle.com/javase/7/docs/api/java/util/List.html#addAll%28java.util.Collection%29) work?


Cuidar los espacios en blanco innecesarios.


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [source/_partials/search_box.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_partials/search_box.html)
- [source/_views/changelog.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_views/changelog.html)
- [source/_views/guide.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/_views/guide.html)
- [docs/articles/sites/database/myisam-to-innodb](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/database/myisam-to-innodb)
- [docs/articles/sites/errors-and-server-responses](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/errors-and-server-responses)
- [docs/articles/sites/migrate/wordpress-site-networks](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/migrate/wordpress-site-networks)
- [docs/articles/sites/quicksilver](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/quicksilver)
- [docs/articles/wordpress/configuring-wp-config-php](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/configuring-wp-config-php)
- [docs/articles/wordpress/hyperdb](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/hyperdb)
- [docs/articles/wordpress/site-networks/getting-started](http://1243a11y-static-docs.pantheon.io/docs/articles/wordpress/site-networks/getting-started)
- [source/docs/search.html](https://github.com/pantheon-systems/documentation/commit/2cce9a5255e0d5bbe03ddcc84fd9fb258a37e986/source/docs/search.html)
- [source/_partials/footer_content.html](https://github.com/pantheon-systems/documentation/commit/a836429ba9a5e0d6a9f224ac8a75314895efbc10/source/_partials/footer_content.html)
- [source/_views/article.html](https://github.com/pantheon-systems/documentation/commit/a836429ba9a5e0d6a9f224ac8a75314895efbc10/source/_views/article.html)
- [source/_views/landing.html](https://github.com/pantheon-systems/documentation/commit/a836429ba9a5e0d6a9f224ac8a75314895efbc10/source/_views/landing.html)


Although, the second argument was optional anyway and even if someone (very unlikely!) passed a second argument to that helper nothing ever happened. Yes, this will blow up for such people (I predict the count is 0!), but YOLO!


perhaps `innerHTML=""` instead of looping with `removeChild`?


Wow. ELB really can't handle `204`?


Ok, I removed the link from here :)


Not that this needs much documentation but, it would greatly help if you'd explain or provide a small example of what the function does.
Also, please:
* terminate statements with a dot, "."
* use "The message bundle object representation", i.e. preferably use an article when referring to things.


Yes, and that is what the commit subject says (and that makes sense), yet the patches do not test the pointer, at all. So I've no clue what the intention here is.


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (2063 of 2063 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

![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)


and here


The `this` value in this function is relevant and should be noted. You can use the JsDocs 3 `@this` tag.


and her


I haven't been closing my R windows, but just have done. When I reopen R (through Emacs and Emacs Speaks Statistics) it automatically loads in an .RData file (I hadn't realised that, because I don't normally save stuff as .RData). If you're using Rstudio or something else, maybe it doesn't automatically load an .RData file.


phew...


i.e. here too


"Μεγάλα θραύσματα προσκρούουν στην επιφάνεια της Γης και μπορεί να δημιουργήσουν μια περιοχή σπαρμένη από κρατήρες. Μια πιο προσεκτική ανάλυση των παραμέτρων εισόδου του ---------- στην ατμόσφαιρα, είναι απαραίτητη για τον ακριβή υπολογισμό του μεγέθους και της κατανομής συχνότητας των θραυσμάτων του μετεωρίτη καθώς και για την πρόβλεψη του αριθμού και του μεγέθους των κρατήρων που θα σχηματιστούν."


Really. :-(


Well written code-  Grade:A


Oh why didn't you submit a pull-request?


Because `typeof null === "object"`, this might not be testing what you think it is.


Hey! Saw that you created this repo and I'm happy to see that you're in the PHP class! We should definitely partner sometime during the PHP course. See ya on Monday!


Looking at this makes me wonder how this logic affects parameters used in this function.  Do parameters (`?`) typically have a null value for their datatype?  (If not, then when is the datatype null?)

It seems like a user might reasonably want to do something like `DWITHIN(pt, pt, ?)`, which seems like it should work without a cast.

Seems like we should have tests for parameters in all one or more arguments to this function, do we have that somewhere?


yeah. Think so too.


Also, if the statement says the object is empty, you should probably check it is empty :-)


Don't hesitate to submit a pull-request if you'd like your changes to be released!


I am not sure why the tabbing is weird but this is what it looks like to me in Coda 2.

![Tabs](https://cldup.com/poh0EtpX0p.png)


By intention until we understand it more. topshelf is a iOS binary extension, it might get called even before the app starts up.


On 02/05/2016 04:15 PM, Guillaume Cartier wrote:
> One thing to consider is that clang is /necessary/ to embed objective-c
> code to interface the mac using cocoa. So even though it is painful to
> compile gambit using clang, it is necessary for some programs.

Ah, so you can't use the Objective-C front end of gcc with Cocoa?

Brad


Seems to work. Thanks.

However, regarding It the -h or --help I don't get any help screen.  The app simply flashes and closes.  (under Win7)

Similarly, the
`qWarning() << qPrintable(tr("The -t/--table option requires an argument"));`
message is never shown if -t is specified but without a table name


cool.


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


[oh crap.](https://github.com/KC3Kai/KC3Kai/blob/dev-lodger/src/pages/strategy/tabs/overlodger/overlodger.js#L945-L946)
it still waiting for merge though


Debug output


On 02/05/2016 04:15 PM, Guillaume Cartier wrote:
> One thing to consider is that clang is /necessary/ to embed objective-c
> code to interface the mac using cocoa. So even though it is painful to
> compile gambit using clang, it is necessary for some programs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/feeley/gambit/commit/bfa0a19835e2d0af418b103cde450634084c3292#commitcomment-15925285>.
>
Perhaps one could make it with the combinations

1.  gcc and --enable-single-host

2.  clang and --disable-single-host

Brad


Thanks for the updates and helpful graphics stuff, SharAI!

BTW your (shar)AI is very impressive given how little we’ve got in place so far. I love the sleeping spider.    But boy does it have a mouth on it.

Everyone else, just a note that some big changes are coming in the code.  I have refactored a bunch of stuff and will be making it easier for you to access the underlying phaser sprites and phaser game object.  Actually the latter is available now as a global variable.     If you are interested in what is possible have a look at the Phaser examples page

http://phaser.io/examples

We use “arcade” physics but most of the stuff on that page (besides “arcade physics”) is accessible to us.   I made a stalker bot which stalked Yang and the navigation is quite smooth.

The upshot is a bunch of stuff will have to be changed after I make my changes.  I’ll wait until after Monday when the homework is due and then help with the transition.

Oh, also Sharai a few other things:

- You should remove the extra graphics stuff you  have in there from the repository.

- I suggest converting the variables you have on lines 12-14 (which are global variables) to sharAI… object properties, which wont conflict with anything else.   Sublime will make the change easy (ctrl-D….) I’ll talk more about the topic of global variables vs. object properties  next Tuesday.

Fun stuff!

- jeff

On Feb 4, 2016, at 8:52 PM, Insectoid <notifications@github.com<mailto:notifications@github.com>> wrote:

[faust]<https://cloud.githubusercontent.com/assets/16771559/12838281/0a0306f4-cb81-11e5-82c7-a5ae2dd75165.gif>

Your bot was moving butt-forwards, so here's a quick edit I made to it's sprite. Just replace the old sprite with this one and it should face the proper direction when moving

—
Reply to this email directly or view it on GitHub<https://github.com/jyoshimi/phil_190_2016/commit/e7b1b075dcc87308e4b8df4968678074e088a4f5#commitcomment-15905190>.


Can we set the Socket error here before returning?


YOLO! Scrap it :)
Am 05.02.2016 22:22 schrieb "Thomas von Deyen" <notifications@github.com>:

> Although, the second argument was optional anyway and even if someone
> (very unlikely!) passed a second argument to that helper nothing ever
> happened. Yes, this will blow up for such people (I predict the count is
> 0!), but YOLO!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AlchemyCMS/alchemy_cms/commit/f857d93ba40eab10ff789ebec07de4b17999ed9b#commitcomment-15925428>
> .
>


While running the libzmq test suite, the only time those pointers are ever NULL is when the size is 0. I assume the pointers are never NULL when the size is larger anywhere (they definitely aren't in the test suite), because then they would segfault for everybody.

Would you prefer something like this instead? Seems more complicated to me.
```
if (pointer == NULL) {
  assert(size == 0);
} else {
  memcpy(pointer, ...);
}
```


Newline removed


Keep up your good work! Nice to see that you are still developing the quiz.


/jkbvhiyuctydxirzeuyws


Use MSE instead. The reason I removed it is that it is an unreliable
indicator to monitor, since it computes RMSE per batch and shows you the
average of these measures. The correct way to compute this is to compute
MSE for each batch, average, then take the square root. Indeed, `sqrt(
average(mse(x) + mse(y)) ) != average( sqrt(mse(x)) + sqrt(mse(y)) )`

On 5 February 2016 at 13:16, AntreasAntoniou <notifications@github.com>
wrote:

> Just a question out of curiosity? Why remove rmse? (it was the main loss
> function for our experiment ;p)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fchollet/keras/commit/65b5899d068daadb9101025dd8b8a7e4ceecec35#commitcomment-15925313>
> .
>


No empecemos no empecemos :smiley:


it would be a new xcconfig file as topshelf is a iOS binary extension and is a separate project.


I guess you mean DISTANCE *or* DWITHIN here.  But since you know which function the user entered, you could just put it in the message?


At this point the dictionary people should just pronounce the new correct spelling to be referer and be done with it.


This is going to make the build order potentially differ based on the user's current locale. That seems like a very bizarre thing to do.


A little fail 0.0


Unit test


any reason for the switch from `'` to `"`?


wow so much green


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


The format method is also static and while we don't need to repeat all the tests, cause it is the exact same code, we could should assert it is there.


Esto podríamos haberlo desactivado solamente, y dejarlo pendiente para más adelante posiblemente adicionar una configuración que permita activar o desactivar el uso de las ACL de Symfony.


Last time I checked there was still a major bug logged in gcc regarding supporting the new objective-c features that the mac now uses. It looked like major work with no one really interested in tackling the task.


Typo fixed in 653595dcd749f6408497454e823bba8d4f893806


@calvinatintel looks good please merge


In the class' description we should also mention that tags can also be words.


I was wondering if below you think there could be any perf hit for always specifying the requestHeaders. I'm wondering if we should only specify them when they change.


Todas las reglas debemos generarlas de forma dinámica... puesto que para las conguraciones más adelante sería necesario mostrar unas o otras no, aunque sean aquellas por defecto.


What is "blockchainstorage.synctex.gz" and why was it added?


Igual que el comentario anterior...


The constructor `source` argument could be more properly documented as `Object.<string, string>` as is the case of the `source` property.


++ I haven't decided if these should move under preferences.
Also open to suggestions for setting name formatting. Currently they don't have a hierarchy.
We could name the constants instead to adopt a format like:
`SCOPE.SCOPE2.VALUE1`


This seems oddly placed. It looks like gamelabPreload gets called during reset. Would it make more sense to detach in `GameLab.prototype.reset` instead?


ok thanks :)


Materialize says it's what's used to open the modal using a trigger/in our case when you press on the donate button.  http://materializecss.com/modals.html


sofa occupied


this email domain is used by all the high-schools in morocco and is provided by microsoft (www.taalim.ma)


// change 1


Removed


The 'std::string' is verbose and can be removed.

Consider raw string literals. eg.
R"({"account": ")" + alice.human() + R"("})"


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.680.f82e4744eb1ae375b6f925e271659d5e99f9ef0f](http://build.palaso.org/viewLog.html?buildId=68926&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:07:16


>  However, FormID lists should not be sorted for Skyrim or Fallout 4.

Hmm - this sorts th elist (?)


Hi,
I just forgot to send it as a pull request! Thanks for remember me! Its not fully tested, but I think tis working!
I will do the PR now.


Should this have just been a 200 ok


Este método se debería refactorizar para limpiar el arreglo de "rules" de todas aquellas reglas diferente de VIEW, EDIT, DELETE. Pero igual que uno de los comentarios anteriores, esto pertenece a la lógica que se usaba anteriormente. Más adelante, teniendo una opción para activar o desactivar las ACL de Symfony si sería muy útil este método.


http://docs.aws.amazon.com/ElasticLoadBalancing/latest/DeveloperGuide/elb-healthchecks.html:
> An HTTP or HTTPS GET request is issued to the instance on the ping port and the ping path. If the load balancer receives any response other than "200 OK" within the response timeout period, the instance is considered unhealthy. If the response includes a body, your application must either set the Content-Length header to a value greater than or equal to zero, or specify Transfer-Encoding with a value set to 'chunked'.

also https://forums.aws.amazon.com/message.jspa?messageID=325760


Missing tests for when `get` is called with a spec object, with or without a default text afterwards.


Esta es el Vouter que me refiero debe estar en el bundle del Taller. Y el que esta en el Taller que contiene todas las configuraciones por defecto venir a parar al HatueySoftSecurityBundle


Also, this method's doclet is missing a couple of examples showing how to use the variable number of arguments signature.


Generalmente debemos velar por poner siempre un "enter" como fin de línea...


I'm sure the protocol didn't change. I can still join with the same protocol.


if NULL is used you need to include stdlib.h. Thus nullptr is preferred.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.670.f82e4744eb1ae375b6f925e271659d5e99f9ef0f](http://build.palaso.org/viewLog.html?buildId=68925&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:11:36


did you mean to log these to a file or print them?


:sparkling_heart: :+1:


Please, rewrite the `structured` doclet to something like "Gets the hierarchical object representation of the message bundle.".


True, but I think one could argue that these are the only outward-facing variables of the model that are sometimes fixed, and sometimes variable.


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


Please add a doclet to the prototype, `format` method. You can direct users to the static version.


This is strange: I made this commit with `git commit package.json -m '...'`, but these files were all somehow still included...


I want to redo the Java one to use Rocker templates. Will do that this weekend.

> On 6 Feb 2016, at 06:50, John Engelman <notifications@github.com> wrote:
>
> oh, we already are....hm, are we ok with adding ratpack-groovy to the Java version of this project?
>
> —
> Reply to this email directly or view it on GitHub.
>


f73cba3 contains a slimmed down version of the reproduction case that triggers the same problem, but doesn't require an internet connection.


BTW, this was eventually done. [semver 1.0.0-beta](http://semver.org/spec/v1.0.0-beta.html) was created to map to what we originally had. But I actually went back and fixed all the versions we had that didn't have a hyphen, so it should all be consistent with v1.0.0 now.


There is ";" sign here


TCP pings ain't a real workaround...it's often not enough to merely know that something responds to a handshake...


Heh is `~/.my.cnf` no longer working?


theoretical question really for all clients in general --> are we going to show an example to just use pd.request(path, method='get', data=None, files=None, **kwargs) for flexibility in keeping up w/ the code-ashians (keeping with changes in our api and documentation) ?


anything needing an upgrade of the code on existing installations? the commit doesn't explain if this is critical or not :)


TypeError: execute is not a function
    at GameServer.<anonymous> (C:\Users\Downloads\Ogar-master2\Ogar-master2\src\GameServer.js:143:20)
    at Server.g (events.js:260:16)
    at emitNone (events.js:72:20)
    at Server.emit (events.js:166:7)
    at emitListeningNT (net.js:1263:10)
    at nextTickCallbackWith1Arg (node.js:444:9)
    at process._tickCallback (node.js:366:17)
    at Function.Module.runMain (module.js:432:11)
    at startup (node.js:141:18)
    at node.js:980:3

Error


dope


@FranDias could you give me a hint as to how I'd get the ninja balls to fall down rather than bounce around?


TeamCity ReOpenLDAP :: ps-stable [Build 109](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1792&buildTypeId=ReOpenLDAP_PsStable&guest=1) is now running


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1271.31786926686b8bf55357bfa51d561026a6e6476a](http://build.palaso.org/viewLog.html?buildId=68932&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 890, ignored: 7 Build time: 00:08:54


Closes #68


Fair enough. Care to open a PR?


And here is ";" too


TeamCity ReOpenLDAP :: ps-stable [Build 109](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1792&buildTypeId=ReOpenLDAP_PsStable&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:31


112796613_DesicDehum (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (960 of 960 tests passed, 0 test warnings)

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


I see. This is done to be able to mock the repository.


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


You’re preaching to the choir. :-)


It works for me, maybe your not useing the latest version


Please move branches to gh-pages


Thanks. Fixed it. If you find more mistakes please correct them [directly in our translation system](https://translations.launchpad.net/backintime/trunk/+pots/back-in-time/pt_BR/+translate). It's writable for everyone.


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1508 of 1508 tests passed, 0 test warnings)

![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)


FWI the 11113 patch had some updates that you probably want to pull in.


whart the hell


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


**seriusly**


`why is it so big??`


- ### aaahhhhh


Please do not use the localtime stuff.....


Back up


Should the default radius be 1 not 0 as almost all consumers are likely to want to query immediately adjacent tiles and any that don't can explicitly specify 0?


whoopsie


Haven't seen you on


Fixed in https://github.com/trotzig/import-js/commit/6084bf1f9e8e2279f1b7601aa32896a523880854


height: auto is default. is it necessary to include?


There's no one I *have* to explain anything to - contributors, yes, maybe - you and anyone else on the other hand, not. It is nice that others are using the library but you are mistaken to think that I or any of the contributors *have* to think about other non-contributing users when making any kind of changes.

I've checked with contributor(s), they were fine with the upgrade, I am working with Json.Net 8.0.2 and I knew Json.Net hadn't supported PCL Profile259 until *recently* and 8.0.2 does/did... and that was it.

That being said, I just went back and tried a couple of versions and it appears 7.0.1 was the first version that supported Profile259. I am currently pushing/building the updated version - if you want to invest more time into it, you're gladly invited to do so and send over a PR.


I did some cleanup in https://github.com/trotzig/import-js/commit/29646adb95dfca6b6790929847612d7864f4fab6


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


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


@FaridSafi with the css change or the scrollToBottom()?


Css change


Oh yeah, this sounds great! (typo: treat → treated)


I made some changes in https://github.com/trotzig/import-js/commit/971aeae85ed9159f23991de54aabe87d06cade2d related to this. It doesn't completely fill all holes, but it makes it better.


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [source/_partials/footer_content.html](https://github.com/pantheon-systems/documentation/commit/dcfd49ec89a36fefbee7911a1dae3c30106004b4/source/_partials/footer_content.html)
- [source/_partials/landing-index.html](https://github.com/pantheon-systems/documentation/commit/dcfd49ec89a36fefbee7911a1dae3c30106004b4/source/_partials/landing-index.html)
- [docs/articles/drupal](http://1243a11y-static-docs.pantheon.io/docs/articles/drupal)
- [docs/articles/organizations](http://1243a11y-static-docs.pantheon.io/docs/articles/organizations)
- [docs/articles/sites/code/email](http://1243a11y-static-docs.pantheon.io/docs/articles/sites/code/email)
- [source/docs/assets/images/by-sa88x31.png](https://github.com/pantheon-systems/documentation/commit/dcfd49ec89a36fefbee7911a1dae3c30106004b4/source/docs/assets/images/by-sa88x31.png)
- [docs/changelog/2015-01-01-January](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-01-01-January)
- [docs/changelog/2015-03-01-March](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-03-01-March)
- [docs/changelog/2015-04-01-April](http://1243a11y-static-docs.pantheon.io/docs/changelog/2015-04-01-April)


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


maybe squash this into previous commit?


This should be documented as returning type `string` :-).


jlkafjlakdjf;alkdsfja


lolno


@MrApocalypse I know this is an inappropriate place to ask, but do you guys have a gerrit? I've kind of searched around to no avail.  Figured I'd ask


@FaridSafi Interesting. I have yet to experience an issue after a bunch of uses. The scrollToBottom() also was necessary for me since new messages are set by state change. The scrollview does not scroll to the bottom automatically.


Sorry I do not understand anything about this commit suggestion....
What do you try to reach with the extra menu button?


This commit looks good. I just merged it.


Please use double-quotes by default.


it gave me the wraning but it takes an awful lot of time for it to reset


fix for this landed in https://github.com/nodejs/node/commit/878bcd43f8f9d3ec0d3403cb1bbe1ba6201a6d84


Should the default radius be 1 not 0 as almost all consumers are likely to want to query immediately adjacent tiles and any that don't can explicitly specify 0?


@I-am-Reinvented No gerrit, we're not big enough for that yet, lol


I think that's a good idea.

On Fri, Feb 5, 2016 at 1:47 PM, Kurt Kraus <notifications@github.com> wrote:

> theoretical question really for all clients in general --> are we going to
> show an example to just use pd.request(path, method='get', data=None,
> files=None, **kwargs) for flexibility in keeping up w/ the code-ashians
> (keeping with changes in our api and documentation) ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pokitdok/pokitdok-python/commit/da07740910c7fdb1c2010af1c97d3b8c80f68c0f#commitcomment-15926000>
> .
>


Can we output a deprecation warning when using `--filename`?


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.582.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68927&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:23:07


what syntax is this??


Oh, it is. Point is: it never worked before, I only noticed it's broken when combining #134 with #100. You can test it by introducing a deliberate failure in the examples -- they are all guarded with `if (mysqlHasDefault())` (at least in master) and won't be run if there's no connection.

If you like, you can figure out how to set up Travis CI to work with this. I haven't figured out how to replicate the Travis environment locally. Without it, it's just too much trial and error. Setup instructions are a mess of Chef cookbooks and rather loose instructions. People have tried, too... Have you? Compare this with the `wercker build` command, if only they took care of the GitHub permissions they require (https://github.com/wercker/support/issues/20).


not really, I made it so you set the time in gameserver.ini. the only wait is one minute because I made the minimum be 1 min and the warning to happen 1 min before restart


@rmacklin What I want to do here is to check the tags I got when I click into edit page is correct. But this line seems ugly because the real value is deeply embedded into the element. Is there a way to make it clearer?


Hi @DigitalFlow

Do you use VS2012 to build the project? It's ok for this pull request but I was wondering because i'd like to work on issue #21 where I'll replace all linked files by a new Shared Project structure, which is only available after VS2013 Update 2.
I'm personally using VS 2015 now to build it, and we don't need the Enterprise one, just the community one is enough.

What do you think?


Would you consider letting this be optional, or do you defend the need to enforce this?


Thanks for this... the signed/unsigned differences are a major source of headaches.


Sorry, I was still trying to figure out how to run the unit tests in my dockerized Jenkins setup, and as soon as I could I was going to do a pull request!


Hey Jelle,
Gechrashed = gecrasht :)


I recommend not hard coding these values. Consider the following instead.
```RPC::missing_field_error(jss::account)[jss::error_message]);```


@MrApocalypse haha okay, cool :) I didn't get scolded when I pushed that little commit to frameworks/base the other day so I figured this was the case 😎 I'll try and contribute more. This is the only ROM I use now.


This looks fancy, is it a new undocumented provider on Travis to automatically handle that?


I recommend not hard coding these values. Consider the following instead.
RPC::missing_field_error(jss::account)[jss::error_message]);


@wkz We don't really need a static buffer here, right?


Nevermind, just saw your PR here: https://github.com/travis-ci/dpl/pull/401


Fixes #2


Perfect!


I know that this would have been very useful for myself, being very new to contributing.

> On 5 Feb 2016, at 22:03, Brenden Blanco <notifications@github.com> wrote:
>
> Would you consider letting this be optional, or do you defend the need to enforce this?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/iovisor/bcc/commit/87d2f69a591b29c254f6ce1974a8347d746bfe9f#commitcomment-15926318>.
>


@wkz ... or here, or am I missing something?


@jippi - If you 1) use incremental repair (which most people should, but some people may not) and 2) issue a major compaction (`nodetool compact <ks> <table>`) - in that case, all of the repaired data gets merged with all of the unrepaired data, causing your next repair to have to do far more work.

@JeremiahDJordan - thanks for the note - I did see that Marcus had updated the ticket, I'll pull over his additions as well.


Hey, gitter is being piece of shit and can't use it. I will check this tomorrow.


just print is fine


Now the polyglot and lang globals get updated when the user model language is updated (be it directly or via fetch). So, we shouldn't have to modify those directly at all. If there is a lang change, we just update the model.


We could, though both stdout and stdin is occupied with the current implementation. `stdout` should be used only for the resulting file content. `stderr` needs to be a JSON object.


I guess we could add a message to the JSON blob.


Let me check, that might be an issue in there.


Looks good. For compactness, notes that don't require formatting can be put all on one line. But it doesn't affect output. Thanks!


I'm actually in the process of modifying iOS App Signer to sign .appex files as well as change the CFBundleIdentifier when needed


I'm pretty sure one is able to use zlistx without these two methods. See [zlistx API page](http://api.zeromq.org/czmq3-0:zlistx). They've only been added in August 2015, long after the zlistx class has been added (November 2014). I'd say the class is usable without them, as one can use `zlistx_next()` and `zlistx_last()`, which are stable AND released.

What do you mean by old versions? I'm referring to the current stable release of CZMQ. I consider supporting that quite important, as it lowers the barrier. I assume that not everybody compiles CZMQ from master.

I don't personally require support for it, though. I guess I could adapt the Ruby binding generator to treat the absence of any function gracefully, independently of its state.


Refactor-Rich-CoolingTower1Spd (energy-plus) - Win64-Windows-7-VisualStudio-12: OK (1885 of 2057 tests passed, 161 test warnings)

 * 334 tests had: AUD diffs.
 * 329 tests had: BND diffs.
 * 10 tests had: ESO small diffs.
 * 195 tests had: MDD diffs.
 * 212 tests had: MTD diffs.
 * 232 tests had: RDD diffs.
 * 168 tests had: Table big diffs.
 * 12 tests had: MTR small diffs.
 * 6 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.

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


Круто сделано.Только вот я мало чего поняла из этого кода...Есть 2 невидимых блока input и div .select-text.Ты привязал невидимые input  к названиям сортировки и в этому случае при нажатии на определенный пункт на сервер отправляется значение как сортировать. Но если это форма почему не в теге form или это не обязательно, и данные здесь отправляются по нажатию на пункт а не кнопке с type submit...так можно?В коде CSS вообще много непонятного. Я так понимаю там есть JS код?Не мог бы ты прокомментировать в своем примере что делается в блоках. И как вообще можно додуматься куда двигаться для создания такого меню. Я вот увидела что выпадающее меню-сразу подумала что select. А оказалось и вообще не Select!


Minor issue: I know it's consistent with ET_Client, but man is that an ugly(and not conventional) interface name


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


@danmar Would it be possible to distinguish this case from one on line 124?


Does the interface actually need these properties available?


need comma here


can't have comma here


that's fine... I just want the hand-written field-by-field copies to go away.. do we fuzz test config conversions? if not, I'm worried we'll drop a field and not know it


The following doc(s) have been deployed to the [1243a11y](http://1243a11y-static-docs.pantheon.io/docs) Multidev environment:
- [source/docs/objects/index.html](https://github.com/pantheon-systems/documentation/commit/be5c33237b79e7fa4a3469d479087bd1f819f530/source/docs/objects/index.html)


`path.parse` does not exist in node 0.10.x, use `path.basename(file, path.extname(file));` instead


needs to be in the `default` case. at the moment this always triggers

```
switch {
case "foo":
default:
    return nil, fmt.Errorf(...)
}
```


kj;alkjdfadfa


should be `options.tags.hasOwnProperty`


FUCK THIS DB CALL


Si dans perso.txt j'ajoute des **

   **%import;tools;**
   <style type="text/css">

Alors en recopiant le code source généré par le navigateur dans pspad ou
notepad++, j'ai un caractère invisible au milieu des ** **
alors que ce caractère n'apparait pas dans le code source du navigateur !!!
Ce serait le caractère 65279 $FEFF selon pspad.

****
<style type="text/css">
.col1{width:57%;min-width:290px;

Je ne trouve pas ce caractère dans tools.txt !

Quoi qu'il en soit, avec ce commit, il n'y a pas de caractère dans le
head, et donc, je n'est pas d'erreur html me disant que le la balise de
fin /head est isolée.

Pour t'en rendre compte, ajoute seulement *** dans le head en dehors de
toute balise et tu verras le saut de ligne que tu cherches, et l'erreur de
/head avec.

Il reste aussi une erreur sur le /body me disant "balise fermante body
rencontrée mais des éléments non fermés sont présents." Je ne sais pas
d'où çà vient.

Il y a des espaces et des lignes en trop dans js_perso_accesskey.txt.

Dans mon css.css j'avais écrit #perso padding-top:2em; actuellement c'est
0; donc, c'est normal que le titre se trouve collé en haut de page.


Since booleans default to false if unspecified, can we make this "skipCertVerification" so it defaults to checking the cert?

Also, we're probably going to need a way to specify a trusted CA chain (don't know if the ldap library supports this).


The reported message includes name of the argument and function both along with error.

If the provided distance argument to DWithin() is non-numeric, reported message say "incompatible data type in operation: input type DISTANCE to DWITHIN function must be non-negative numeric value" .

Can update the message to: "incompatible data type in operation: input type DISTANCE to DWITHIN function must be non-negative numeric value", if that makes it more reasonable?


I've got no problem with anyone contributing. You'll never see me scolding anyone with contributing changes that actually make sense, lol. Glad you like the ROM!


same notes as comment above. this is currently invalid JSON due to commas


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 142](https://teamcity.labkey.org/viewLog.html?buildId=301289&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 43 Build time: 01:23:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

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

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

```


`update_attributes` works because it's an alias for `update`, but in Rails 4 the preferred method to use is `update`.


Can we set _lastError here before returning?


It's minor, but we could reduce the duplication between the branches of this if/else statement.


Looks fine by me


@MrApocalypse how would you feel if I pushed a commit enabled -O3 instead of -O2 in the build/core/combo folder? Can I at least push it for you to test?


Looking into it, it looks like that file can't access the MisoPropertyExporter that I'd usually use to test for the property. I'll look into this more on Monday.


Spelling error here.


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


This file should be called _edit_form rather than _editform


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1020.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=68930&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:17:39


Should this match the name of the resource to better support possible future resources and just be consistent in general? Ie: `TestAccAWSCloudFrontWebDistribution_basic`.


Oh, I read the error message wrong.  See "DISTANCE" in all caps makes me think that it's a function name.  Consider making it lower case I guess?  I might say something like, "input type of distance argument to DWITHIN must be non-negative numeric value."


Lines 1-11 here are not part of the "form". So I would probably move those out of this partial and into the edit template.


Took me a while to find this link, read the docs on how to disable devtools in production in a google cache and was wondering whether they were outdated.
links like these are easily missed in a lenghtier README and full text search didn't lead me to the walkthrough page.
next thing i usually check then is the wiki, which was empty.

can understand that READMEs shouldn't get too lengthy and see why splitting them up can make sense.
not urging to put the instructions back to the README, just wanted to share how splitting up the infos can confuse people.

how about adding more text around the link to the walkthrough and a link from "Be careful to strip its code in production!" to he walkthrough?


Spelling error here as well.


Minor, but we typically call this button "Cancel".


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


Is there a reason to wait here? If not, then this could greatly speed up the deployment of this resource.


It would be better if we could separate these changes into atomic commits.


sure.


Good to know and I'll fix it.


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


change curDepth itself will cause MaxValue go to different layer in next loop!


Actually, what I'll probable do is add the logging data to the test input and spit the whole thing out as a json string. Busy weekend ahead!


小伙子可以的 玩的飞起:joy:


Thank you!


Mmm... If I add references to CallActor messages in BackendActor... we are going to have (again) circular dependency... and I wouldn't like to have in the code :)... so we can define a message (wrapper) in BackendActor.


The following doc(s) have been deployed to the [905deployb](http://905deployb-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/deploybot](http://905deployb-static-docs.pantheon.io/docs/articles/sites/deploybot)


Do you mean it should be like this?
 `redirect_to @picture, notice: @picture.update(picture_params) ? 'Picture was successfully updated.' : "Failed to update picture. (#{@picture.errors.full_messages.join('. ')})"`


I will fix this.


True, fixed.


Well that is easier! Glad you caught it.


i tried making the files uppercase and that did not work


Sure.


Sure and I also update the label in create form to 'Cancel' too.


Yep. Will do.


So you want something like:

```csharp

int size = this.GetSizeOfCategory(...);
if(size > 200){

}


Maybe we should expose this as an coap option as well.


rwewerwer


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.174.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68937&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:40:59


Introduction

This second programming assignment will require you to write an R function that is able to cache potentially time-consuming computations. For example, taking the mean of a numeric vector is typically a fast operation. However, for a very long vector, it may take too long to compute the mean, especially if it has to be computed repeatedly (e.g. in a loop). If the contents of a vector are not changing, it may make sense to cache the value of the mean so that when we need it again, it can be looked up in the cache rather than recomputed. In this Programming Assignment you will take advantage of the scoping rules of the R language and how they can be manipulated to preserve state inside of an R object.

Example: Caching the Mean of a Vector

In this example we introduce the <<- operator which can be used to assign a value to an object in an environment that is different from the current environment. Below are two functions that are used to create a special object that stores a numeric vector and caches its mean.

The first function, makeVector creates a special "vector", which is really a list containing a function to

set the value of the vector
get the value of the vector
set the value of the mean
get the value of the mean
makeVector <- function(x = numeric()) {
        m <- NULL
        set <- function(y) {
                x <<- y
                m <<- NULL
        }
        get <- function() x
        setmean <- function(mean) m <<- mean
        getmean <- function() m
        list(set = set, get = get,
             setmean = setmean,
             getmean = getmean)
}
The following function calculates the mean of the special "vector" created with the above function. However, it first checks to see if the mean has already been calculated. If so, it gets the mean from the cache and skips the computation. Otherwise, it calculates the mean of the data and sets the value of the mean in the cache via the setmean function.

cachemean <- function(x, ...) {
        m <- x$getmean()
        if(!is.null(m)) {
                message("getting cached data")
                return(m)
        }
        data <- x$get()
        m <- mean(data, ...)
        x$setmean(m)
        m
}
Assignment: Caching the Inverse of a Matrix

Matrix inversion is usually a costly computation and there may be some benefit to caching the inverse of a matrix rather than computing it repeatedly (there are also alternatives to matrix inversion that we will not discuss here). Your assignment is to write a pair of functions that cache the inverse of a matrix.

Write the following functions:

makeCacheMatrix: This function creates a special "matrix" object that can cache its inverse.
cacheSolve: This function computes the inverse of the special "matrix" returned by makeCacheMatrix above. If the inverse has already been calculated (and the matrix has not changed), then cacheSolve should retrieve the inverse from the cache.
Computing the inverse of a square matrix can be done with the solve function in R. For example, if X is a square invertible matrix, then solve(X) returns its inverse.

For this assignment, assume that the matrix supplied is always invertible.

In order to complete this assignment, you must do the following:

Fork the GitHub repository containing the stub R files at https://github.com/rdpeng/ProgrammingAssignment2 to create a copy under your own account.
Clone your forked GitHub repository to your computer so that you can edit the files locally on your own machine.
Edit the R file contained in the git repository and place your solution in that file (please do not rename the file).
Commit your completed R file into YOUR git repository and push your git branch to the GitHub repository under your account.
Submit to Coursera the URL to your GitHub repository that contains the completed R code for the assignment.
Grading

This assignment will be graded via peer assessment.


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


DRY way to produce icon selectors, whether you'd like to implement via the `@extend` directive or as selectors in the markup. Load the fonts into your font library, and then enter values like this:

```
$icon-symbols: (
  ico-close: '\e900',
  ico-menu:  '\e901',
  ico-arrow-down: '\e902',
  ico-arrow-right: '\e903'
); $icons: map-keys($icon-symbols);

@each $icon in $icons {
  .#{$icon} {
    &:before {
      content: '#{map-get($icon-symbols, $icon)}';
      @include icomoon;
    }
  }
}
```


and some of them load


> I'm going to wait till the weekend when I have more time. I was going to do a trial run on their test server.

I see, then it's better not, at least, to push, wait until everything
has tested out, or you might need to push another fix release; or
better, not even commit since you don't need that to upload to PyPI.


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


This is really the wrong place for this.


it is giving errors at tile classes that dont have getTextureName() in it


Did you try `elements.first[:value]`? (Note that I'm also suggesting renaming the block variable to the plural form `elements` because its a collection, albeit one with a single element.)


Why isn't ScaleTwo in a different API version?


can you guys release a gem update with this change? because stripe blows up if 0.0 is passed: Invalid Positive Integer error


(nitpicking)  Unless they are simple declaration, I find it easier to read when they are in separate lines. i.e.
let yLabel = get(flds, ['y.label'], yName);
let yUnit = get(flds, ['y.unit'], this.getUnit(yName));


ya i fixed it


So you suggest I use MSE for training and sqrt(MSE) as a realistic indicator of what is happening? What about mae? Would you reccomend that over MSE or not?


@bartbutler i get the point that the you need this operation for practical reason, i'm only saying that this is surely not managed in a standard way as  decyrpt(encrypt($a)) != $a for certain payloads

if you pass "antani\r\nantani" to openpgp then the encrypted payload you get "antani\n" that is an altered version of the initial string that i guess is not standard and there is no way out to know at descrypt time that in some places you require to read \r to get the original.

and here is a scenario that proves the rilevancy even of having back an \r\n that may clarify you where thingns can fail.

suppose a openpgp user want to encrypt:
```
long_text
crc_of_long_text is: something
```

the user will decypt the message, but supposing long_text was containing \r\n, it wont no more match the CRC


Thanks! I made changes to get rid of CSS overrides of material design lite as well as make some things configurable that were hard coded. Unfortunately I don't know if I'll have to time to separate that work from the work of actually building my blog in order to submit PRs... :disappointed:


i will commit the fix when i done


You cannot test multiple actions in a controller test. So you'll need separate test cases for the `update` action. If you want to test the complete flow, you can write a flow test with capybara.


Line 21/23 in kernel/interrupts/idt.c was the damned one!


It looks like they're not being used anywhere (nor printed).


Yes, shouldn't ScaleTwo not even be in the extensions group at all?


The action being tested in this test is `edit`, not `update`.


yes, nevermind.


I think we have a few untested cases here.


Cool, it works!


bbbbbbbbb bbbbbb


Thanks for testing @tonypdmtr. :grin:  Looks like it needs a bit more development to get that sorted as well then.


Could we maybe add a sentence to say "This is useful in cases where X" or something like that? I feel like this isn't going to make 100% sense to a brand new Drone user.


This test is mainly testing functionality provided by the `ActsAsTaggableOn` gem. I'd be okay with omitting it, but you can leave it if you want.


I have started importing PropType directly.  It makes the propTypes a little easier to read.
`import React, {Component,PropTypes} from 'react';`


OK.  Not ideal, but this is important.  Can we please log the error - at least the first time (I don't know if there's a way to do that easily though)?  Once option might be to move this logic into getSelfAwsInstance.

I'm fine with just logging the error every time though.


If you put something like the following.
`var {alignment, labelWidth, fieldKey= props; `
it make if a little easier to see how you are using props. I have adopted this and it appears to be become more of a idiom among react blogers


I added another cases where the new tags are empty in next commit. Then those are all the cases came into my mind...


Submit a PR to my branch dude and I'll merge it in....


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


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


this is just for ruby 1.9.2


Did this get published to npm?