Should this be updated?

https://github.com/gradle/gradle/blob/9cc662044619e99dc2f46c8e12af7aaa9bb3e5e5/config/checkstyle/suppressions.xml#L39-L40


Myoldmopar-patch-1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2223 of 2223 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)


I'm about to write a forum post about it to get people to try out the release. I think we can improve this message before the final release, lets see what kind of issues come up.


@vnugent Yes, change is here. https://github.com/jkandasa/cfme_tests/commit/d5d3ede9ce0b19629dbc4c53899fbc6db0975142


Just a precaution in case the folder exists. That said, I agree - lets rely on GUID uniqueness to enable this to work.


Can we just remove this flag entirely?


:beer:


@peaseloxes Gekkigheid


Да ну ёп тваю мать, Жень... =/


COUNT(ID)


Guy, in MediaTekRIL.java, getting some errors:
/MediaTekRIL.java:299: error: cannot find symbol
		Operators init = new Operators ();
                 ^
---------------------------------------
/MediaTekRIL.java:299: error: cannot find symbol
		Operators init = new Operators ();
                                                    ^
----------------------------------------
MediaTekRIL.java:819: error: cannot find symbol
	    Operators init = new Operators ();
             ^
-----------------------------------------
MediaTekRIL.java:819: error: cannot find symbol
	    Operators init = new Operators ();
                                              ^
-----------------------------------------
I am using other device tree but I have just copy-pasted the MediaTekRIL.java from your tree but I get errors


КОСТЫЛЬ, НЕ НАДО 95


We can go ahead and remove it

On Wed, May 4, 2016 at 11:41 AM -0700, "Matt Robenolt" <notifications@github.com> wrote:

Can we just remove this flag entirely?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub


No problem - Done it many times before!


dgfdfgdfg


hhg


@mikeferguson instead of setting push URLs for every repo you maintain, you can just put the following in your `~/.gitconfig`:
```
# Always use ssh for github, even if the remote URL uses
# https or git
[url "git@github.com:"]
  insteadOf = git://github.com/
[url "git@github.com:"]
  insteadOf = https://github.com/
```


Myoldmopar-patch-1 (Myoldmopar) - 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)


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#BLmgySJlQzu8Ng-oETzZfA


Why ? We need it for when the bot is running.


Ve, acho que não poderia fazer para apenas ter uma classe desse tipo (fazendo com o getInstance), porque cada andar vai/pode ter um arrayList de apartamentos diferentes...


Otherwise I would have made it so X and Y coordinates were shown when the bot is off


Forgot to add in the commit message, but I'm using Gradle Daemon for all gradle tasks now. It's really fast.


#include <stdio.h


Forgot to add this in the commit, but I'm using the Gradle Daemon now for all gradle tasks.


@jianqiu see my changes and placeholder. Please update when you have a chance. Thanks.


Hoi my name is Sr. Hector David


`Localization.GetSafeJSString` is NOT sufficient for this and in point of fact, should be retired or just made a pass-though for the correct implementation in `HttpUtility.JavaScriptStringEncode`.  See comment below for justification.


Mesmo caso que falei na classe Apartamento... acho q não poderíamos fazer com o que apenas tivessemos um arrayList desse tipo, porque cada obra vai/pode ter um edifício diferente..


Inclusão de Diagrama de Classes e Diagrama de Sequência de Sistemas(DSS).


That's all.
Feel free to merge with the fix for that typo :smile:


I am trying to understand the difference between "Memory" and  "Materialized". You explained MemoryToMaterialized is to Deserializes from Stream to an actual object of type  T. Looks like the "Memory" means memory stream, and "Materialized" means some memory objects, right? If that is true, I mean you might want to call "Memory" as "MemoryStream" because Materialized is also in the memory.


A small but useful tip!  :sunglasses:


mano, não acha melhor fazer uma classe Andar, e fazer seu arraylist em um RepositorioAndar ? assim como inserir entraria na parte dos Controladores ..


Hi Edy.

Thank you so much for your help. It was very useful and didatic.

Could you be so kind to provide more examples or even a link which explains more about good practices in CSS coding?

Thanks in advance and I'm waiting to hear from you again.

Best Regards,
Wellington Braga


Well, there `cabal test --test-option=--network=no`. I'll try whether that's going to work. It will pass the flag to *all* tests, though. Maybe that's okay?


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/5d06eceed9a02e8f326ac4df5e97bef11e8d56ff


merging basho/yokozuna/zl-better_dt+crdt_check_for_siblings_permitted = 5d06ecee into borshop-integration-630-zl-better_dt+crdt_check_for_siblings_permitted


basho/yokozuna/zl-better_dt+crdt_check_for_siblings_permitted = 5d06ecee merged ok, testing candidate = b7b3c82a


@chelmuth, @nfeske: please add to staging


YouTubeChannel is a class that represents a YouTube channel.  You need to call init() method in order to initialise it.

I think you have a very basic knowledge of Android, right?  If that is the case, then read the book named The Busy Coder's Guide To Android Development (version 5.0).  It's legally available online for free.


I agree about "more places => more potential bugs", different line styles work best with different zoom levels and for different purposes. E.g. when zoomed out and when printing, the boldest lines work best, but they look a bit ugly when zoomed in to a normal/default zoom level - so I would at least keep it for now and see what users use/like. It is not the only config option, and once there are some config options each next one is not adding that many special places to introduce new bugs.

(I also personally prefer to have this option since I work a lot with pedigrees and I prefer to see the old line style :)


To start with, it passes down 2 DVs from one parent, and 2 DVs from the other parent. Which DVs come from which parent is random. Then, it has a chance to keep those DVs as they are, but will usually overwrite 2 of them with new random DVs. Then to prevent weirdness after recalculating its stats, it sets their current HP equal to their new Max HP, so you don't end up with a Pokémon that has 21/20 HP or something.

As a placeholder until I add the ability to keep 2 Pokémon in the Day Care, it has DVs listed for the Day Care Man's Ditto to keep it consistent. His Ditto is shiny. If you also leave a shiny Pokémon in the Day Care, you have a much higher chance of breeding another shiny as a result of this.


I tried to get recovery versioning working using Tassadars script. Kept getting bad recovery images.

I thought I referenced the issue in my comment. My bad.

I found a way tp label the images though, JollaMan has a script to do it.


fixed in a forced rebased commit


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#DPHXMqBJSsiNPfj76Bj93A


Interesting..


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#O8DAuZ7tTP6CKXJH1b2eVA


Well, it's not really correct to start a sentence with i.e. at all, and I.e just looks weird (as does E.g.)  will change the line above to a semicolon


R+


@hhovsepy It will be good to have in similar format. In some cases we may need to use filter values from that particular class. Here we have to use deployment name and server name to get particular deployment data from mgmt. However it will be good to have a meeting with team and we will plan.


Looks like this broke the build!


Notes:
If you ever sudo gem install something, you always have to be root to use it: `sudo chown -R rtass001:staff /usr/local`


The following doc(s) have been deployed to the [ttrowellte](https://ttrowellte-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/temp-files](https://ttrowellte-static-docs.pantheonsite.io/docs/temp-files)


@hhovsepy Yes, I will figure out  how to use `verify_stats`. Thank you!


I love this commit


Oh, cool. I think it should be ok, or if not we can figure out something to make it work.


Delete Send Method.


@indhupriya you need to write code in this class.

For example see -
https://github.com/apache/tika/blob/master/tika-parsers/src/main/java/or
g/apache/tika/parser/geo/topic/gazetteer/GeoGazetteerClient.java

Add more modules in pom.xml if you need external jar. Ensure the are in sync with Tika.


Thanks @dmaloney-r7


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


Figured you may want these two as well

https://github.com/CyanideL/android_packages_apps_Settings/commit/831aa23dfb5151c8bed28ca063350be0bd999f09

https://github.com/CyanideL/android_frameworks_base/commit/d221b1bc540985edd0a4e1cb0d89b555022831a7


Could you explain, please? Or update **README.md** file with your changes description.
Did you implemented feature **"Check the bluetooth state on the app start"**?
Thanks in advance!


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1640


Nice spacing bro!


nah dude this spacing sucks Im not gonna commit it


Se realiza cambio en el archivo persistence.xml y en el readme


We need these to support “I want to add/delete a stop at drop/pickup 12 to trip 34 (stamped from route A), and also to all future trips on route A, and I also want to add/delete a drop/pickup 12 ↔ route A association”.  If we're handling association/deletion through the Beehive UI for now, that's fine.  The backend will just be partially compliant.  But I think we want to keep them in the API because we'll add support for them soon (and they don't seem like they'd be that hard to support, once you have PUT propagation working).


Forum thread: https://forums.meteor.com/t/1-3-3-beta-w-ddp-batching-now-available/22644


Right, ok.


Because new elements can impact later truth maintenance.  See https://github.com/WilliamParker/clara-rules/blob/b538d27dee134a5151682b2f8010102bdfcab0d5/src/test/clojure/clara/test_rules.clj#L3872


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#f-kjPfnsSOWFPXwoYnR9Zw


Really bad commit, sorry.


“If persist is set in addition to this flag…” seems like a typo in `persist`'s own docs.  I'm still wrapping my head around the new docs here, so I don't have a concrete suggestion for a fix yet.


But now isn't the `typealias AbstractString String` wrong on 0.3?


👍 Thanks!


We should: https://github.com/go-lang-plugin-org/go-lang-idea-plugin/issues/2503

I removed extra checking


Myoldmopar-patch-1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1077 of 1077 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-32.87%25-green.svg)


= 10% of all WebGL-enabled phones.

Plus we also need 32-bit float support which ~ 20% of these devices don't support. Both features must be present at the same time. Overlap is likely but by no means guaranteed - some will only have one or the other. So we're talking about something like dropping support for 1/4th of smartphones out there.


$(this).html('new text here')


Use a colon, not a semicolon, to separate fonts in a font stack declaration. A semicolon means the end of  the rule, stuff after it would be parsed as the next declaration.


```javascript
$('ul li').eq(1).wrap ...
```


This won't compile due to the semicolons. Take a look at the natural gift table to see that each entry corresponds with one semicolon at the end of the statement.


```javascript
$(inputs).eq(i).val(Math.random() + i);
```


Avoid leaving _stuff that do nothing_ in your commit.


h


ON WHAT PLANT WAS THAT TEST EASY!!!!!!!!!!1!


Ah, true. It relies on floating point textures too... Never mind then! 😇


good observations, didn't know all this stuff


1) Это класс
2) Так конструктор это же и есть сама эта функция
3) Потому что они в том же контексте


Perfect! I was going to report this. Thanks a million.


When quoting attributes values, use double quotation marks.
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks

Omit the protocol from embedded resources.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Protocol#Protocol

Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


Thank goodness!


thanks, I'll add them shortly.


@NeighborhoodInfoDC/review Could someone else try running this on their machine? I am getting an error about record length for the MFIS data. In the meantime, I'll investigate whether they changed the data formatting.


Nicenice


Running the test suite with shared libraries enabled fails:

~~~~
Linking dist/build/darcs-test/darcs-test ...
dist/build/darcs/darcs: error while loading shared libraries: libHSdarcs-2.12.0-GnW9XstJOoDLTbaelYK46m-ghc7.10.3.so: cannot open shared object file: No such file or directory
~~~~~

Afterward, the process runs forever, apparently doing nothing. :-( I'll try again without shared libraries.


e-mail in git, m8y. `git config --global user.email`


missing `break`. If a node is collapsed, no need to process other entries in `gep`


Ah crap. I didn't really look into the 0.3 situation so much. Will fix.


`(self, actual_objective, s=[], w=[], M=100, alpha=1.1)`

`s = PosyArray(s)`
`w = np.array(w)`


[LayoutDesign.java - Copy.txt](https://github.com/JeromeAbim/ExpenseApp/files/249429/LayoutDesign.java.-.Copy.txt)


Pay attention to indentation. _img_ is inside _body_, so it should be indented 1 level more.


Why did you delete that?


Pay attention to indentation. _article_ is on the same level as _img_, so they should have the same indentation.


Note that right after the function returns.

        if (NodeH.getNode()->getSize() < requiredSize){
          if (!(NodeH.getNode()->isArrayNode())) {
            NodeH.getNode()->growSize (requiredSize);
          } else {
            assert (NodeH.getNode()->getSize() > 0);

            NodeH.getNode()->foldNodeCompletely();
            NodeH.getNode();
            Offset = 0;
            // break;
          }
        }

        // Add in the offset calculated...
        NodeH.setOffset(NodeH.getOffset()+Offset);

        // Check the offset
        DSNode *N = NodeH.getNode();
        if (N) N->checkOffsetFoldIfNeeded(NodeH.getOffset());

        // NodeH is now the pointer we want to GEP to be...
        setDestTo(GEP, NodeH);
        return; <----------------------
    }

On Wed, May 4, 2016 at 12:20 PM, Arie Gurfinkel <notifications@github.com>
wrote:

> missing break. If a node is collapsed, no need to process other entries
> in gep
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/seahorn/dsa-seahorn/commit/0ff87149aef4d3424450540af7e31e4a062f50a0#commitcomment-17357598>
>


Do not turn off encapsulation unless you really need to. Keep your component isolated as much as possible


This commit seems to break bundling for us in https://github.com/cockroachdb/cockroach/tree/master/ui/next. The presence of this change prevents all our typings from loading:
```
$(npm bin)/jspm bundle app
     Building the bundle tree for app...
TypeScript transpiling to CommonJS, consider setting module: "system" in typescriptOptions to transpile directly to System.register format
TypeScript file:///Users/tamird/src/go/src/github.com/cockroachdb/cockroach/ui/next/app/containers/cluster.tsx:1:24
TypeScript Cannot find module 'react'. (TS2307)
TypeScript file:///Users/tamird/src/go/src/github.com/cockroachdb/cockroach/ui/next/app/containers/cluster.tsx:8:12
TypeScript JSX element implicitly has type 'any' because the global type 'JSX.Element' does not exist. (TS2602)
TypeScript file:///Users/tamird/src/go/src/github.com/cockroachdb/cockroach/ui/next/app/containers/cluster.tsx:8:12
TypeScript JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists (TS7026)
....
```

Unfortunately, there is a change in 4.0.9 (#123) which we need, so we're at an impasse. It is also impossible to revert this on master as the underlying code has changed.


> e-mail in git, m8y. git config --global user.email

I guess you didn't read my last message?

> My gmail IS used in my github account


Unnecessary space between opening _p_ and opening _em_ tags


@NeighborhoodInfoDC/review Scratch that - there was a column deleted in the new data as of 04/30/16. "Premise ID" is no longer in the data, but it looks like it is usually blank. I'll add an issue to fix that in the MFIS update procedure.


Unnecessary space between opening _strong_ and opening _a_ tags


Wprowadzenie sztucznej inteligencji


GitHub and git (the cli tool) installed on your mac is different things and they have different settings, but maybe everything is good on your side and it's just Github doing shit. I don't know, but just try to set your e-mail for git with the command I wrote :)


Header and footer components should not be part of the welcome screen. Move them to more generic level.


Pay attention to matching opening and closing tags. Missing closing tag for _p_.


Maybe the Javadoc should be amended to say that the _primary_ symbol must be a valid Java identifier, but that non-identifiers may be used as aliases.


This breaks an install where autoconf or automake has to be installed from source.
autoconf ends up implicitly depending on autoconf via the inherit:autconf (I think).
Anyway pybombs -vvvvvv install automake shows the infinite recursion.


Pay attention to matching opening and closing tags. starting _h1_ and closing _1_ tag don't match.


Everything is correct.


Yes, I was deliberately changing this to work based off the converted value.  Not operating based on the value used downstream seems like a clear bug to me.


No. Everything is wrapped in a for-loop without braces, and it needs a
break.
Took me a while to understand why they have a break in the other place a
node is collapsed :)

The structure of the code is

{
for (...)
   if (Struct)  { }
   else if (Array) { }
   esle if (Pointer) {  /* your code is here */ }

DO STUFF
return
}

On Wed, May 4, 2016 at 3:21 PM, caballa <notifications@github.com> wrote:

> Note that right after the function returns.
>
> if (NodeH.getNode()->getSize() < requiredSize){
> if (!(NodeH.getNode()->isArrayNode())) {
> NodeH.getNode()->growSize (requiredSize);
> } else {
> assert (NodeH.getNode()->getSize() > 0);
>
> NodeH.getNode()->foldNodeCompletely();
> NodeH.getNode();
> Offset = 0;
> // break;
> }
> }
>
> // Add in the offset calculated...
> NodeH.setOffset(NodeH.getOffset()+Offset);
>
> // Check the offset
> DSNode *N = NodeH.getNode();
> if (N) N->checkOffsetFoldIfNeeded(NodeH.getOffset());
>
> // NodeH is now the pointer we want to GEP to be...
> setDestTo(GEP, NodeH);
> return; <----------------------
> }
>
> On Wed, May 4, 2016 at 12:20 PM, Arie Gurfinkel <notifications@github.com>
> wrote:
>
> > missing break. If a node is collapsed, no need to process other entries
> > in gep
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/seahorn/dsa-seahorn/commit/0ff87149aef4d3424450540af7e31e4a062f50a0#commitcomment-17357598
> >
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/seahorn/dsa-seahorn/commit/0ff87149aef4d3424450540af7e31e4a062f50a0#commitcomment-17357629>
>


I know what the command is :)  This is probably a better conversation to have in IRC I think.


Pay attention to matching opening and closing tags. Missing closing tag for _p_. For all the _p_-s.


- fixed RPG7 and AT4 hitting brush trigger_ entities
- updated RPG7 welements
- melee attack now interupts reload
- updated fire rates on Mosin and M40
- 250 rnds mags for DOI Lmgs
- removed broken bazooka smokescreen
- changed M1 Carbine rof to 600rpm
- changed SKS rof to 600rpm
- fixed few init lua errors in mp
- fixed melee attack anims in MP


Yeah for sure :)


Is this case required at all? Seems like the foreach would be skipped in this case and finally the default return would result in the same return value?


The following doc(s) have been deployed to the [ttrowellbe](https://ttrowellbe-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/scope-of-support](https://ttrowellbe-static-docs.pantheonsite.io/docs/scope-of-support)


Use a space between the last selector and the declaration block.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Declaration_Block_Separation#Declaration_Block_Separation


inherit from `SignomialConstraint`


jest


TeamCity LabKey :: LabkeyModules15.2 :: CDS postgres 9.4 [Build 56](https://teamcity.labkey.org/viewLog.html?buildId=321875&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4 (2 new), passed: 21 Build time: 00:34:29
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

```


unneeded after inheritance


Whats the benefit of hiding the ArtaxClient behind this HttpClient? It makes it less explicit


Core - master, MatlabRenderer - slave. It doesn't ever need to know about core. Instead, core should have to know it has smth which can render sketches and it may ask such component to draw smth.


Use a space after a property name’s colon.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Property_Name_Stops#Property_Name_Stops


Really? Why not just style?


Tutaj chodzi o przeformatowanie danych które przychodzą z zewnętrznego programu. No ale rzeczywiście, można dać stałe :D


Replaces the recursive function `NU.branching.times`. Much faster.
```
> phy <- read.tree("RAxML_fastTreeSH_Support.SHTEST.tre.rr.pr.treepl");
> phy;

Phylogenetic tree with 5018 tips and 5017 internal nodes.

Tip labels:
	Frankeniaceae_Frankenia_gypsophila, Frankeniaceae_Frankenia_corymbosa, Frankeniaceae_Frankenia_laevis, Tamaricaceae_Reaumuria_soongarica, Tamaricaceae_Reaumuria_turkestanica, Tamaricaceae_Reaumuria_cistoides, ...
Node labels:
	20, 96, 88, 100, 100, 75, ...

Rooted; includes branch lengths.
> system.time(xx <- NU.branching.times.old(phy));
   user  system elapsed
  4.328   0.000   4.326
> system.time(yy <- NU.branching.times(phy));
   user  system elapsed
  0.004   0.000   0.004
> identical(xx,yy);
[1] TRUE
> system.time(ww <- NU.branching.times.old(phy, "begin.end"));
   user  system elapsed
  4.284   0.000   4.283
> system.time(vv <- NU.branching.times(phy, "begin.end"));
   user  system elapsed
  0.004   0.000   0.004
> identical(ww,vv);
[1] FALSE
> all.equal(ww,vv);
[1] TRUE
```
This last result suggests there is some rounding errors accumulating in the recursive version of the function.


Too many 2-s. Hexadecimal color format should look like #rrggbb


No, to taki troche januszowy kawałek jest ;) Ale też taka januszowa sytuacja z tymi płciami ;)


patrz, nigdy wszystkich nie uda mi się wywalić przed commitem ;)


Inherit everything but `as_gpconstr` from `SignomialConstraint`


czy takie skrócenie jest dla Ciebie bardziej czytelne? Bo ja tu się o czytelność kodu staram :)


dwie różne rzeczy tutaj sprawdzam


replace with `self._unsubbed.sub(self.substitutions).posy_negy()`


Should this `>= 1` constraint be done here or in the objective?


I think argv is string and data is list?


a potem nam serwisy siadają bo jakiś tam monolog zużywa gigabajty przestrzeni dyskowej ;)


flexbox helps here


probably, it is better to use rem/em here?


może to moje stare przyzwyczajenia, żeby nie robić foreachów na zmiennych w obiektach jakoś mnie popchnęły w tą drogę aby przekonwertować JSON'a na tablice...


@palmerj ,We can build only against official Ubuntu repos and our PPA. None of them contains PostGIS 2.2. Package should work with both, than we will build with 2.1 and can install with both 2.1 and 2.2


```python
        k22c1 = PosynomialInequality(posy, "<=", negy+1)
        # s above 1 makes the approximated side easier
        k22c2 = PosynomialInequality(negy+1, "<=", self.s*posy.mono_lower_bound(x0))
```


styles from other components should not be here


Why does using interfaces make it less explicit? Why is it hidden?


Hi Fabio, agreed. Originally I simply missed your implementation (or to phrase it more correctly didn't realize that it already does what I wanted without many further work, also as I wanted to have a "fast" implementation for getting datacards including these SF on a short timescale).  Upon closer look you are right, you producer is exactly what I was trying to implement. I will switch to using it in the MSSM configs as well and take care of cleaning up the rest to prevent possible future confusion.


@palmerj , fixed in 1f85fa96104aaaa3a7bf7ca87e7a26b6b125eac0


This would compute to 0, so it's easier to use 0 instead. Use 'auto' margins for horizontal margins and centering only. Using auto otherwise can get complicated. See this scary part in the CSS specification: https://www.w3.org/TR/CSS2/visudet.html#Computing_heights_and_margins


Didn't test it but maybe it works:
PRODUCT_PACKAGES += \
 	FmRadio \
	FMRecord \
	libfmradio.$(BOARD_FM_DEVICE) \
	com.broadcom.bt


`dep.name.dup` would be enough


extInt_issue5126 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2158 of 2170 tests passed, 463 test warnings)

 * 475 tests had: AUD diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 1 test had: RDD diffs.
 * 11 tests had: Table big diffs.
 * 7 tests had: EIO diffs.
 * 4 tests had: ESO big diffs.
 * 3 tests had: MTR big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.
 * 3 tests had: ESO small diffs.
 * 2 tests 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-99.45%25-yellow.svg)


missing indentation


```python
self._unsubbed.sub(self.substitutions).posy_negy()
k22c1 = PosynomialInequality(negy, "<=", posy)
# s above 1 makes the approximated side easier
k22c2 = PosynomialInequality(posy, "<=", self.s*negy.mono_lower_bound(x0))


You must use my frameworks_opt_telephony repo.
Am 04.05.2016 20:43 schrieb "Preet Patel" <notifications@github.com>:

> Guy, in MediaTekRIL.java, getting some errors:
> /MediaTekRIL.java:299: error: cannot find symbol
> Operators init = new Operators ();
> ^
>
> /MediaTekRIL.java:299: error: cannot find symbol
> Operators init = new Operators ();
> ^
>
> MediaTekRIL.java:819: error: cannot find symbol
> Operators init = new Operators ();
> ^
>
> MediaTekRIL.java:819: error: cannot find symbol
> Operators init = new Operators ();
> ^
>
> I am using other device tree but I have just copy-pasted the
> MediaTekRIL.java from your tree but I get errors
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/fire855/android_device_kingzone_k1-turbo/commit/ad9617b2a63074b897c73786fa8512300e784f55#commitcomment-17356985>
>


It's just trying to invoke the darcs binary - does that work at all?


Hmm, when I run the test suite with shared libraries disabled, there is no error but it appears to hang anyway. The test suite starts up and then just sits there, apparently doing nothing. This is very odd.


Awesome @LeMagnesium thank you ! :)

@HybridDog the "sed" tool can do this in less than 1 minute, yes. But, maybe Mg wants to do this himself in case of some exception, like you can see on some files


Setting max-width on a text container is usually more useful if you set it relative to the maximum legible line length, eg, _700px_, than if you set it relative to the viewport with, eg. 50%. This could result in half-iphone-width lines and other funny situations.


If just `python ex14.py` will show that is
Traceback (most recent call last):
  File "ex14.py", line 3, in <module>
    script, user_name = argv
ValueError: need more than 1 value to unpack
JoiesR8:python joie$ python ex14.py
Traceback (most recent call last):
  File "ex14.py", line 3, in <module>
    script, user_name = argv
ValueError: need more than 1 value to unpack

If type John Smith (With a space between John and Smith)
will shows Traceback (most recent call last):
  File "ex14.py", line 3, in <module>
    script, user_name = argv
ValueError: too many values to unpack

But if I type John_Smith and the file can run it like I type the exercise demo answer like zed.


```python

self._unsubbed.sub(self.substitutions).posy_negy()
k24c1 = negy <= self.s*posy.mono_lower_bound(x0)
# s above 1 makes the approximated side easier
k24c2 = posy <= self.s*negy.mono_lower_bound(x0)


Looks good!


Indent by 2 spaces at a time.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Indentation#Indentation


It worked, thanks!


Both are external types and there is no 2nd impl of the interface, is it?


此处存疑


oh, no good?


Paso de prueba


When quoting attributes values, use double quotation marks.
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks

Omit the protocol from embedded resources.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Protocol#Protocol

Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


Fixed errors


Avoid empty lines inside declaration blocks.


My best guess is "~2.2 || ~2.3" using this calculator:
http://semver.npmjs.com/


тест


sorry, nie zauważyłem


I would recommend leaving the image height at _auto_, that way it would keep its aspect ratio.


Sor for mistake in title... should be:
Hillsbrad Foothills and Loch Modan Overhaul and Implement Dynamic Movement

;)


monolog nie tylko do plików potrafi wrzucać dane ale np do bazy danych czy logstasha, który jest zaprojektowany do tego, poza tym ma coś takiego jak elastic search :)


fast-forwarding feature-solr-batching-rebased to borshop-integration-630-zl-better_dt+crdt_check_for_siblings_permitted = b7b3c82a


Merge sha 025d797f is stale.


merging basho/riak_shell/fix_gg_README_errors = 660742d5 into borshop-integration-36-fix_gg_README_errors


basho/riak_shell/fix_gg_README_errors = 660742d5 merged ok, testing candidate = 346ddedc


I really love your work and this is exactly what I've been looking for. Unfortunately, it isn't working for me... After executing mactelnet.py and successful authentication, the replies seem to come twice, so i see two mikrotik's welcome screen. But more importantly, the command "/system identity print" doesn't seem to be executed and I'm also not getting any prompt where I could type in commands. This is what I get:

`Connecting to MAC: 00:0c:29:3a:7c:f6
Trying if: eth0 192.168.1.114 00:0c:29:b5:56:35
Trying if: eth1 192.168.116.135 00:0c:29:b5:56:3f

0d0d0a0d0d0a0d0d0a0d0d0a0d0d0a0d0d0a0d0d0a0d0a0d20204d4d4d2020202020204d4d4d202020202020204b4b4b202020202020202020202020202020202020202020202020202054545454545454545454542020202020204b4b4b0d0a0d20204d4d4d4d202020204d4d4d4d202020202020204b4b4b202020202020202020202020202020202020202020202020202054545454545454545454542020202020204b4b4b0d0a0d20204d4d4d204d4d4d4d204d4d4d202049494920204b4b4b20204b4b4b202052525252525220202020204f4f4f4f4f4f202020202020545454202020202049494920204b4b4b20204b4b4b0d0a0d20204d4d4d20204d4d20204d4d4d202049494920204b4b4b4b4b2020202020525252202052525220204f4f4f20204f4f4f2020202020545454202020202049494920204b4b4b4b4b0d0a0d20204d4d4d2020202020204d4d4d202049494920204b4b4b204b4b4b202020525252525252202020204f4f4f20204f4f4f2020202020545454202020202049494920204b4b4b204b4b4b0d0a0d20204d4d4d2020202020204d4d4d202049494920204b4b4b20204b4b4b202052525220205252522020204f4f4f4f4f4f202020202020545454202020202049494920204b4b4b20204b4b4b0d0a0d0d0a0d20204d696b726f54696b20526f757465724f5320362e33322e332028632920313939392d3230313520202020202020687474703a2f2f7777772e6d696b726f74696b2e636f6d2f0d0a0d0d0a
<<

  MMM      MMM       KKK                          TTTTTTTTTTT      KKK
  MMMM    MMMM       KKK                          TTTTTTTTTTT      KKK
  MMM MMMM MMM  III  KKK  KKK  RRRRRR     OOOOOO      TTT     III  KKK  KKK
  MMM  MM  MMM  III  KKKKK     RRR  RRR  OOO  OOO     TTT     III  KKKKK
  MMM      MMM  III  KKK KKK   RRRRRR    OOO  OOO     TTT     III  KKK KKK
  MMM      MMM  III  KKK  KKK  RRR  RRR   OOOOOO      TTT     III  KKK  KKK

  MikroTik RouterOS 6.32.3 (c) 1999-2015       http://www.mikrotik.com/

>>

  MMM      MMM       KKK                          TTTTTTTTTTT      KKK
  MMMM    MMMM       KKK                          TTTTTTTTTTT      KKK
  MMM MMMM MMM  III  KKK  KKK  RRRRRR     OOOOOO      TTT     III  KKK  KKK
  MMM  MM  MMM  III  KKKKK     RRR  RRR  OOO  OOO     TTT     III  KKKKK
  MMM      MMM  III  KKK KKK   RRRRRR    OOO  OOO     TTT     III  KKK KKK
  MMM      MMM  III  KKK  KKK  RRR  RRR   OOOOOO      TTT     III  KKK  KKK

  MikroTik RouterOS 6.32.3 (c) 1999-2015       http://www.mikrotik.com/

0d0d0a524f5554455220484153204e4f20534f465457415245204b45590d0d0a2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d2d0d0d0a596f7520686176652032316834366d20746f20636f6e6669677572652074686520726f7574657220746f2062652072656d6f74656c792061636365737369626c652c0d0d0a616e6420746f20656e74657220746865206b65792062792070617374696e6720697420696e20612054656c6e65742077696e646f77206f7220696e2057696e626f782e0d0d0a5475726e206f6666207468652064657669636520746f2073746f70207468652074696d65722e0d0d0a536565207777772e6d696b726f74696b2e636f6d2f6b657920666f72206d6f72652064657461696c732e0d0d0a0d0d0a43757272656e7420696e7374616c6c6174696f6e2022736f667477617265204944223a20473657552d535a36330d0d0a506c656173652070726573732022456e7465722220746f20636f6e74696e7565210d0d0a
<<

ROUTER HAS NO SOFTWARE KEY
----------------------------
You have 21h46m to configure the router to be remotely accessible,
and to enter the key by pasting it in a Telnet window or in Winbox.
Turn off the device to stop the timer.
See www.mikrotik.com/key for more details.

Current installation "software ID": G6WU-SZ63
Please press "Enter" to continue!

>>

ROUTER HAS NO SOFTWARE KEY
----------------------------
You have 21h46m to configure the router to be remotely accessible,
and to enter the key by pasting it in a Telnet window or in Winbox.
Turn off the device to stop the timer.
See www.mikrotik.com/key for more details.

Current installation "software ID": G6WU-SZ63
Please press "Enter" to continue!

`
I tried this bot on physical and virtual devices. Strangely, when on virtual PC with virtual MT's I get output as above... But on physical PC and physical MT I get everything twice as well, but with "system identity print" being executed (twice as well) and also something that seems to be unprocessed output.

Is there any way to make this work? This is exactly what I've been looking for for a very long time. Thans.


Right, but you can easily mock the interface for tests.


its always LIT :)


暂时没想到替换词，如果强调为孩子定制，可掉换顺序，将之放在取源头活水前面
> 宝儿妈妈惊喜地发现，“陪孩子读诗app”，专为孩子采集诗歌源头的活水。


docker pull hortonworks/cloudbreak-web:1.2.5
Pulling repository docker.io/hortonworks/cloudbreak-web
Tag 1.2.5 not found in repository docker.io/hortonworks/cloudbreak-web


The best argument I could come up with for PEP8 is that it's the standard coding style for Python. It would be a bit easier for new people to contribute because they don't have to fiddle with editor settings and most of tools assume by default that you follow PEP8 or want to be.

TL;DR: ¯\\\_(ツ)_/¯


 在集团里唯一一个女孩子，那是只有对迷恋的对象依赖这种没出息的话。

此处存疑


You could consider a section type element instead of a div type element with a class that suggests it's a section. See https://developer.mozilla.org/en/docs/Web/HTML/Element/section


@matthew-andrews I've removed it in favour of changing `o-comments` packages to use `config.js` instead of `config.json` with `module.exports`. :-) idea credit: @ironsidevsquincy


@SC73171  is 10/10 lit all day errday


Is this the bug where the killaura could only rotate 55 degress?


It appears that the darcs binary doesn't know how to find it's own shared library. `ldd` says:

~~~
libHSdarcs-2.12.0-GnW9XstJOoDLTbaelYK46m-ghc7.10.3.so => not found
~~~

All other libraries show up correctly, and the binary does work once installed into the proper location. Apparently, the build directory is not listed in the linkers rpath.


- [ ] 强烈反省被冲走的形式。

?


its too LIT im speechless


- [ ] 是哪个循环?


@SC73171  so swessed


> Is this the bug where the killaura could only rotate 55 degress?

Where it wouldn't bypass if it rotated any further, yes.


The following doc(s) have been deployed to the [ttrowellbe](https://ttrowellbe-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/scope-of-support](https://ttrowellbe-static-docs.pantheonsite.io/docs/scope-of-support)


Setting max-width on a text container is usually more useful if you set it relative to the maximum legible line length, eg, 700px, than if you set it relative to the viewport with, eg. 50%. This latter could result in half-iphone-width lines and other funny situations.


I think what was happening is that there was a non-printable new line character in the text file that the addword method picked up. It seemed to always add words that I added into the program correctly, but always had an extra character when reading from the text file. I discovered this when I had it add the text file words twice and the error was still present.


small indeed


Damn dude, you're lame as fuck......

You should of gave credit where it was due
https://github.com/DirtyUnicorns/android_device_oppo_msm8939-common/commit/4c1a0d3ed09e4d14a7135ffe17e4df316f5703b2

Guess we see your real side come out


You could also use `frame['OBSTYPE'] != 'CATALOG'`


Ok thats a good point, indeed


Submit a pull request and I'll merge it with the main code.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#YKC5G-qjQI6LwDbllI7jKw


When quoting attributes values, use double quotation marks.
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks

Omit the protocol from embedded resources.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Protocol#Protocol

Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


Wenn das Problem in der Master behoben ist kannst du dich denn ganzen Punkt rausnehmen ?!


Working code without periodic boundaries


+1 to that.


The advantage of adding the `try...catch` there is that if there's an unexpected exception when handling the queue, the rest of the chain will execute normally (basically, other server requests in L220). If we rely only in the `catch` currently in L96-101 we have to send a rather arbitrary response.


How about you go back to just cherry picking and maintain authorship?


105015426-DefVAVTUMinMax (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1081 of 1083 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-99.82%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-32.9%25-green.svg)


Agreed, I just left them in because we presumably will have a templated "switch" later for the pull request.


good


added labelled tweets with 'labelledTweets.txt' filename


Oh, I thought you had used it...Thank you very much


we have our first model!


I'm already working on this, no need for an issue. I was working on the merge from release => dev for this change, but this merge got pushed before I got mine in. There were no merge conflicts - so it wouldn't have been obvious the merge needed work, but the docs didn't match the dev API.


Hola,

Por alguna razón no logro abrirlo en Android Studio. El archivo descargado y descomprimido es de 146 kb, por favor revisa que esté bien subido aquí en Github.


I was about to send a PR https://github.com/joshwcomeau/panther/issues/3 :expressionless:


f = es


This is going to be a challenging project, partly because of the technical challenges natural language processing presents. That said, you'll learn some valuable (and transferrable) skills by taking on this challenge. It sounds like if you have 12 events X 60K cases, that you should have enough features to get a useful classifier, even if it is not super precise. I am excited to see what discoveries you make!


on React Native, this gives me:
> Can't find variable: File


Thanks ;)


But you might have known about them if you had read about them in the docs. And I can't help people discover stuff if they don't read the docs. :)

I know there's been stuff I've missed for years because I didn't notice it in the docs, or wasn't ready for it, yet, or something.


Have eclipse format your source code before committing, eg reindent


This is confusing. We are not getting high-level documents here.


Also, don't be afraid to delete large blocks instead of commenting them out, since old code is preserved in the repo history


Why is it checking during canWrench if tStats.isCrowbar(); and not tStats.isWrench(); ?


Limit should be customizable with a default value in an argument.


yes they are :)
https://github.com/melonjs/melonJS/blob/master/src/vendors/tween.js#L152


What are these `includes`?


ahh okay :)


Good start so far! 👍
Killings
#1 An alternative method is: df.rename(columns={'raceethnicity':'race', 'lawenforcementagency':'agency'})
#8 You could also do: killings.state.value_counts().head(5)
#13 Awesome! You are the _only one_ that figured out how to show the months in order.
Majors
#4 You can use .map('${:,.2f}'.format) on the corresponding series
Tasty
#1 I like how you chained your merges here; it looks more elegant.
Keep going with this. The Instagram api challenge is really fun to try. :)


@Potrimpo ah yeah, maybe you have a prefetch or preload setting enabled in chrome advanced settings somewhere. (loads page in background)


@npjohnson what was this in reply to?


thanks for doing this


What `Int32` is being referred to?

Also, this statement seems to be incorrect:

> allocates a new int32 value


You should delete your `.idea` folder and make sure you add it to your `.gitignore` file.


sorry for the late reply, tremendous log of unread emails/notifications here :)


This endif should be at the end of your file, not at the top of it. Your code should be between your `#define` and the last `#endif` this is called the header guards. Read about it here: https://en.wikipedia.org/wiki/Include_guard

/cc @yangyulin


Same thing here, you code should be inside your header guards.


The field 'separatorChar' is a static or class field (there is a single value that belongs to the class), as opposed to an instance field (where each object or instance of the class has its own value for that field), so referring to it as 'belonging' to a particular instance doesn't make sense (but is still legal). Under the hood the compiler just ignores the 'f.' object.


All your header files should be inside a folder called "include" not in the root of the project.
Added it to your cmake here, just create the directory and move it in there.
https://github.com/rpng/mast_project/commit/a2f1e378fb9d00163baf33d7ea05cae39621a78a


dude i had to use git push --force cause weird stuff was happening. if anything weird happens later ill fix it.
the conventions in this comment are flawless. dont correct me.


The diff seems to strange. The correct is the below. Does current GitHub have bug?

```diff
diff --git a/helm-qiita.el b/helm-qiita.el
index 958dc93..38931dc 100644
--- a/helm-qiita.el
+++ b/helm-qiita.el
@@ -278,3 +278,4 @@ Argument EVENT is a string describing the type of event."
   (helm-qiita:set-timer))

 (provide 'helm-qiita)
+;;; helm-qiita.el ends here
```


This header file needs a header guard. And implementations should be in your `Corresponder.cpp`


Overall good stuff. Make sure you are making your method names are of the same type, either CamelCase or underlined_methods. Don't mix and keep the same style.


ProPic


Good work so far; keep it up! 🏃
**Killings**
#6-#7 You need to change to killings.armed == 'No' since you do want to count those that were unarmed
#7 One possible solution: "{0:.0f}%".format(killings[killings.armed == 'No'].count() / killings.count())
This solution using python string formatting to display the answer as a percentage
#8 Your solution is correct
#11 pass sharey=True, sharex=True so that your plots are easier to compare
#13 One possible solution (courtesy of Dieuwe):
some_months = killings.month.value_counts()
missing_months = pd.Series({'July':0, 'August':0, 'September':0, 'October':0, 'November':0, 'December':0}, name='month')
pd.concat([missing_months, some_months])
month_num = pd.Series({'January':1, 'February':2, 'March':3, 'April':4, 'May':5, 'June':6, 'July':7, 'August':8, 'September':9, 'October':10, 'November':11, 'December':12}, name='num')
pd.concat([month_num, pd.concat([missing_months, some_months])], axis=1).set_index('num').sort_index().plot(kind='bar')
**Majors**
#3 You need to add tail(10) so that you fetch only the top ten
#4 One possible solution (courtesy of Martina):
top10 = majors[['Major','Median']].sort_values(by='Median',ascending=False).head(10) #sort_index(by='Median').tail(10)
top10.plot(kind='bar',x='Major',title='Top 10 highest paying majors')
plt.xlabel('Major')
plt.ylabel('Median Salary ($)')
#5 To show as dollars, you could use .map('${:,.2f}'.format)
#6 Instead of rank, you could use sort_values and then tail(5)
#8 One possible solution:
majors.Median.hist(by=majors.Major_category,sharex=True,sharey=True, figsize=(15,10))


First Commit


Actually this is a .cpp file, not sure why you have a header guard here, and the `#pragma once`. Remove these.


I've created an issue to fix this behavior in the Checker Framework: https://github.com/typetools/checker-framework/issues/700


In Python 2 `isinstance(i, str)` may return `False` for unicode strings. I think the easiest way to make neurolearn compatible is to use [six](https://pypi.python.org/pypi/six) as in http://stackoverflow.com/a/23692403/318359


No language: haskell?


@gtirloni I didn't mean to commit directly to this repository, I thought I was committing to my fork with the intention of sending a PR. I can remove it from the commit log and open a PR if required but I did test the changes successfully with Fedora 22 and 23. This approach works more reliably now than using Ansible and systemd on Fedora 23.


cc


Use $HOME consistently instead of ~?


I'd record the sha1 just for giggles and verify it before running the code.


test+++


-f shouldn't be needed.


Thanks for reporting this. I'll take care of it


Thanks, I just updated this for you.

I think I was using if type(var) is str:  elsewhere  I’ll have to clean that up too at some point.

> On May 3, 2016, at 10:39 PM, Anton Burnashev <notifications@github.com> wrote:
>
> In Python 2 isinstance(i, str) may return False for unicode strings. I think the easiest way to make neurolearn compatible is to use six <https://pypi.python.org/pypi/six> as in http://stackoverflow.com/a/23692403/318359 <http://stackoverflow.com/a/23692403/318359>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub <https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346651>


@juan2ramos no me di cuenta a que horas pasó, ya lo recupero


what about a table that blender game checks against, and only allows you to
modify bind codes it created somehow?

On Tue, May 3, 2016 at 5:26 PM, Diego Lopes <notifications@github.com>
wrote:

> I'm sorry but this is really, really dangerous. I would not allow the user
> change the bindCode, imagine if a wrong code is given? The texture gets
> messed up outside the game too! [image: :fearful:]
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/UPBGE/blender/commit/f566d1e43933fe95f0842ec791ef4d551a831864#commitcomment-17345909>
>


ok


si, ya quedo!


DO NOT CREATE ANY NEW ACCOUNTS UNTIL YOU PULL AND USE THIS UPDATE ON YOUR DEVICE. We were setting height and age as strings in initial setups instead of Longs causing crashes, I already corrected all of the accounts that were made like this so don't make any more until you update your device to this version!


Thank you, that's a timely fix, as I just got "ValueError: Y does not match
the correct size of data", because coincidentally file paths were unicode.

On Wed, May 4, 2016 at 5:46 AM, Luke Chang <notifications@github.com> wrote:

> Thanks, I just updated this for you.
>
> I think I was using if type(var) is str: elsewhere I’ll have to clean that
> up too at some point.
>
>
> > On May 3, 2016, at 10:39 PM, Anton Burnashev <notifications@github.com>
> wrote:
> >
> > In Python 2 isinstance(i, str) may return False for unicode strings. I
> think the easiest way to make neurolearn compatible is to use six <
> https://pypi.python.org/pypi/six> as in
> http://stackoverflow.com/a/23692403/318359 <
> http://stackoverflow.com/a/23692403/318359>
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly or view it on GitHub <
> https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346651
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346684>
>


So the six.string_types works?  Or I should fix something else too?

> On May 3, 2016, at 10:51 PM, Anton Burnashev <notifications@github.com> wrote:
>
> Thank you, that's a timely fix, as I just got "ValueError: Y does not match
> the correct size of data", because coincidentally file paths were unicode.
>
> On Wed, May 4, 2016 at 5:46 AM, Luke Chang <notifications@github.com> wrote:
>
> > Thanks, I just updated this for you.
> >
> > I think I was using if type(var) is str: elsewhere I’ll have to clean that
> > up too at some point.
> >
> >
> > > On May 3, 2016, at 10:39 PM, Anton Burnashev <notifications@github.com>
> > wrote:
> > >
> > > In Python 2 isinstance(i, str) may return False for unicode strings. I
> > think the easiest way to make neurolearn compatible is to use six <
> > https://pypi.python.org/pypi/six> as in
> > http://stackoverflow.com/a/23692403/318359 <
> > http://stackoverflow.com/a/23692403/318359>
> > > —
> > > You are receiving this because you authored the thread.
> > > Reply to this email directly or view it on GitHub <
> > https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346651
> > >
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly or view it on GitHub
> > <https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346684>
> >
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub <https://github.com/ljchang/neurolearn/commit/78af2bd5123d7f300a77bc87fb04ed63b81e660e#commitcomment-17346734>


지금 소스 코드를 보면 패킷 구성이 제대로 안된 것으로 판단됨. 버퍼 구성했으면 실제로 패킷을 전송(pcap_sendpacket)해 보고 wireshark에서 패킷을 잡아서 제대로 패킷이 나가는지 확인해 볼 것. 이상하게 된 필드를 알아 내어서 왜 필드 값이 이상한지 알아 낼 것. 디버깅할 때 절대로 다른 사람의 도움을 받지 말것. 디버깅도 능력임. 지금까지 수업한 것에 그 답이 다 있음.


42는 뭐야?


Can we declare this as ErrNotSubdomain?


프로그램 인자값 주어 지지 않으면 어떻게 돼? argv[1], argv[2]에 쓰레기 값이 들어 가잖아. argc range 확인해.


What is the rationale for this complicated mechanism to extract `default_branches`?


Haha


This could be in a util class


To document, should we name the return values lang, err?


new change for wechat


OK


@jgillich Nope, just ignorance. Thanks for letting me know about this—I've already begun using it.


don't remember, I think I may have copied this from the protobuf library?


k it worked


A <link> tag can occur only in the head element.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/link


Don't leave _stuff that do nothing_, eg. empty class attributes in your commits.


great work. just one request could you open the issue tracker? :)


Although fine with HTML, do not close void elements.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Document_Type#Document_Type


@michaelklishin: any suggestions of how to address the following use-case without pipelining non-nowait requests?

A processor that reads/writes to many queues, creates all of the queues and bindings it needs at startup. The list of queues changes dynamically, but not very fast, so on startup most of the declares are effectively noops.

Our current implementation with pika 0.9.14 and rabbitmq sends out queue declare and then binding declare statements pipelined (in batches with a limit on how many are outstanding in parallel, to work around the high CPU load issue @awelzel mentioned). The main thing however is that these are pipelined/non-nowait. Pipelined to avoid too much delay, and non-nowait because we need to know that queue was declared before we can create the bindings, and we need to know that the bindings have been created before we can start processing (or messages we publish might be lost).

Given that (following this fix) we can't do pipelined requests unless we make them nowait, any idea on how we could solve this? If we do all the declares one after the other startup could take minutes if latency is not very low. But if we fire them off nowait, how would we know when they are done?


## Remove `EntityComponent` in favor of plain `React.Component`

`EntityComponent` tried to be a shortcut to creating `React.Component` for entities with some benefits:

* simpler to declare
* `entity` property.
* render function implements `ViewOverrides`

Unfortunately it was a dead-end when you needed state or additional properties, and increases the concept count.

I've managed to remove `EntityComponent`, `EntityComponentWithState` and `EntityComponentProps`, move `EntityFrame` inside `StyleOptions`, and implement `ViewOverrides` by monkey-patching `render` function.

The new API is longer to write, but I hope is also easier to understand and modify.

BEFORE:
````C#
export default class Exception extends EntityComponent<ExceptionEntity>> {
    renderEntity() {
    //..
    }
}
```

AFTER:
````C#
export default class Exception extends React.Component<{ ctx: TypeContext<ExceptionEntity> }, void> {
    render() {
    //..
    }
}
```

## How to migrate:

```
FIND: EntityComponent<(.*)>
REPLACE: React.Component<{ ctx: TypeContext<$1> }, void>

FIND: EntityComponentWithState<(.*),
REPLACE: React.Component<{ ctx: TypeContext<$1> },

FIND: renderEntity(
REPLACE: render(
```

And in order to fix the import declarations you may need something like:

```
FIND: EntityComponent,
REPLACE: TypeContext,
```


Use a space between the last selector and the declaration block.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Declaration_Block_Separation#Declaration_Block_Separation


Nevermind.  it is needed for potential future changes.


wg mnie jest to czytelniejsze i krótsze przede wszystkim. Jest wiele innych miejsc (choćby te wielkie pętle), gdzie należałoby o czytelności pomyśleć :)


Use a space after a property name’s colon.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Property_Name_Stops#Property_Name_Stops


wait so jira will only appear if he's offline?


The `Sentence` class does not have a method called `ner`. I believe `nerTags` is the correct function to put here.


I meant, made password hash iterative, not salt.


The issue here is that ```containers/cluster.tsx``` is not referencing ```typings/browser/main.d.ts``` so it doesn't know about the ```react``` module.

The simplest solution would be to include the typings files using tsconfig.json instead of ```/// <references``:

```
{
  "compilerOptions": {
    "target": "ES5",
    "module": "commonjs",
    "jsx": "react",
    "noImplicitAny": true,
    "noEmit": true
  },
  "files": [
    "../typings/main.d.ts",
    "./interfaces/protos.d.ts",
    "../lib/protos/protos.d.ts",
    "./app.tsx"
  ]
}
```

It seems that plugin-typescript is not following the same algorithm as tsc when resolving dependencies which is described in this issue https://github.com/Microsoft/TypeScript/issues/1066. I think I can probably improve this, so I will investigate.

I will also look into how it was working in 4.0.5 as this seems like a bug although it may have been due to the files being loaded in a different order.


ps w linii 11 też jest


Hi,

I think that this line causes
`Warning: Illegal offset type in isset or empty in /vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/Compiler/ReplaceAliasByActualDefinitionPass.php on line 137`

Am I right?


For any CSS dimension, there is no space between the unit literal and the number.
https://developer.mozilla.org/en/docs/Web/CSS/length


Does it make sense to add suppress support to closure_js_binary() for symmetry in the API?


You could, I think the real limitation is the fact that we have no idea
what plugin attempted the call without trying to print a stack trace.
That's not a clean approach, I think for now a try-catch is more pythonic.

We should take Ben's approach with a grain of salt since some of his
choices reflect the way wrapper was built and what limitations were self
imposed

On Wed, May 4, 2016, 12:43 PM Surest Texas <notifications@github.com> wrote:

> should I just go back to:
>
>     try:
>         self.client.message(string)
>     except:
>         self.log.debug .....
>
> ?
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/suresttexas00/minecraft-wrapper/commit/210ef24e54e7e284d6eeac36ddb9ef725d9fc62c#commitcomment-17355396>
>


Avoid leaving _stuff that do nothing_, eg. empty declaration blocks in your commits.


sorry alex this is only my test... i write this time with Anders and i have
a proplem in my rom with selinix i forgot enable a line in r7plus... i love
the DU rom.. and i love a header picture and all devs fuck me up for
this... i hate the original launcher and love the Nova so i insert this...
my rom have no Du support so i delete updater and other DU supports...

thanks for your hard work

2016-05-04 21:59 GMT+02:00 Alex Cruz <notifications@github.com>:

> How about you go back to just cherry picking and maintain authorship?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/pixelfreak2005/android_device_oppo_msm8939-common/commit/7d328a116866901aee9dcbc3abcfd56d98908d03#commitcomment-17358204>
>


Actually on a second look, I don't think your producer yet supports applying the SF only to certain gen_matches (in our case we decided to apply it only to genuine electrons faking taus, i.e. those with gen_match 1 or 3), or am I missing something here?


You can send a PR. :P


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/23/violations
```
plone/recipe/codeanalysis/__init__.py:176:13: T001 print function found.
plone/recipe/codeanalysis/__init__.py:191:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:200:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:248:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:59:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T001 print function found.
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Too much/little indent on this line?


Welcome to IT Experts

We understand that the world of IT moves forward at a blistering pace, and we’re committed to staying ahead of the latest developments to ensure you get the best Computer Support and Computer Repairs available.

No matter how tough your challenge, with IT Experts you’ll realise the potential of your technology and see a greater return on your investment at all levels.

When your network grinds to a halt, or a server comes crashing down, you need a fast response every time.

Have your ever worked out how much it costs you per hour to have your network down?

IT Experts react swiftly whatever your problem and work tirelessly to find solutions that get you back on your feet with the least downtime.

We can help you no matter where you are based in South Africa, from Polokwane in Limpopo, to Cape Town and Durban or any other area in South Africa.

Call us at +27 15 296 4666 and you’ll be put through to the person you need quickly and efficiently, no hanging around on hold or waiting for an email from some distant department – you get the Computer Support you need – from real IT Experts, really fast.

Whether you’re starting with a poorly installed or optimised system, need replacements and Computer Repairs in Polokwane or Limpopo, or need recommendations for a new or upgraded network – call IT Experts today for the best computer support in the area.

Our services include:

    Remote Network Support and Service
    Networking
    Software installations
    Networking solutions
    Server maintenance including emergency response
    Computer sales and repairs
    Cost-effective upgrades
    Diagnostic services
    Affordable on-site support contracts

Our Computer Support and Computer Repairs in Polokwane and Limpopo extends to almost every aspect of IT installations – from a single desktop PC to a fully networked office building, and we will always provide the best advice and the most cost-efficient solutions for your Computer Support needs.

For more information on our Computer Repairs in South Africa, or any of our Computer Support services, visit our Contact page and get in touch – one of our team will be happy to help.

Keywords : Computers Polokwane  |  Computer Repairs Polokwane |  Computer Support Polokwane
( http://www.mritexpert.com/ )


Some test


This is great idea, go!


( http://www.mritexpert.com/ )


When quoting attributes values, use double quotation marks.
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks

Omit the protocol from embedded resources.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Protocol#Protocol

Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


@gfilipiak see #18671


lol......


When `$referenceId` is instance of `Symfony\Component\DependencyInjection\Reference` and is used to index an array it somehow does not use `__toString()` magic method.


Take a look at using setup_provider, as psav mentioned that it is a special fixture that will setup the provider if it is not already there (aka: will add MW provider if there is not one).

Ex: test_name(provider, setup_provider)


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/24/violations
```
plone/recipe/codeanalysis/__init__.py:176:13: T001 print function found.
plone/recipe/codeanalysis/__init__.py:191:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:200:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:248:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:59:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T001 print function found.
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@gfilipiak `setFactoryService()` expects a string, not a `Reference` instance


It is recommended to start the head with the charset declaration. See https://developer.mozilla.org/en/docs/Web/HTML/Element/meta


Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


OMG NO INLINE STYLES ⚠


@xabbuh thx


Good catch, thanks.


An _img_ tag should not go inside the _head_, but inside _body_ instead.


Hey -- it's closed after the display.  But yeah, I'll deparenthesize.


⚠ Don't use representational attributes in the HTML


fyi @k-nut


A PR of mine had feedback not to extend the Provider class. May not be relevant for this, but something to look into (or see if the PR feedback flags this or not).


System-Fan-Model (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2173 of 2193 tests passed, 536 test warnings)

 * 553 tests had: AUD diffs.
 * 408 tests had: RDD diffs.
 * 20 tests had: EIO diffs.
 * 4 tests had: ESO small diffs.
 * 4 tests had: MTR small diffs.
 * 19 tests had: BND diffs.
 * 8 tests had: ESO big diffs.
 * 2 tests had: MDD diffs.
 * 16 tests had: MTD diffs.
 * 7 tests had: MTR big diffs.
 * 17 tests had: Table big diffs.
 * 7 tests had: ERR diffs.

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


⚠ Don't use representational attributes in the HTML
Separate structure from presentation from behavior.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Separation_of_Concerns#Separation_of_Concerns


Angel, probando ésta configuración. me percaté que al generar la OP, con varios renglones. Le añade a cada OP la configuración del último renglón del pedido de Venta; lo ideal es que en cada OP generada, se muestre la configuración establecida en el renglón. En la La cantidad total (mp_qty) si asocia la cantidad de cada renglón. Te voy a enviar una cotización y las OP que se generan, para que veas que toma solo 1 de las configuraciones.


Hi Rene, no problem!
Yes, indeed in "my" producer I compute the scale factor for all the events (ideally, it should be run only in the Drell-Yan-specific pipeline) and then I take care of applying the scale factor only to the ZLL template at plotting level. Since I think that the recommendation is common, we can introduce the gen-matching there as well.
The other obvious difference is that in my case the SF are loaded from a rootfile, which is a solution that I tend to prefer because has the advantage of being quite flexible, since in future one would have to update the rootfile and not the C++ code. The disadvantage is the usual need of committing binaries to github, which unfortunately is a problem common to many weight producers in our framework.


Nice work using a pseudo selector here!


Yes, I've also notice the generation is broken since this commit


Yes


Avoid using the universal selector. In this case, setting font-family on _body_ instead and letting inheritance do the rest would get the same results.


Updated style guide to include new class for 50% width responsive images.


yeah, its a strange thing to set out to anyway right?


why the not minified versions? just for debuggging ?


That's exactly the code I was looking for. RW and FF now work properly. Thanks!


One thing to think about for next time would be to use more descriptive names for your classes and IDs, as well as to hyphenate multi-word ones, like this:

`<div class="header-top"> and <div class="header-main">`


Try being the least specific you can in your selectors. Do these styles really depend on the styled element being a _div_, or could we say that anything with a _main_ class should look like this?


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/25/violations
```
plone/recipe/codeanalysis/__init__.py:176:13: T001 print function found.
plone/recipe/codeanalysis/__init__.py:191:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:200:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:248:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:59:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T001 print function found.
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


MakeDegreeDaysConsistWithASH901 (JasonGlazer) - x86_64-MacOS-10.9-clang: Tests Failed (1678 of 2188 tests passed, 5 test warnings)

 * 510 tests had: Table big diffs.
 * 5 tests had: AUD diffs.

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


Is it possible to use `'protobuf>=2.6.1'` here?
This would allow using [protobuf 3.0.0](https://pypi.python.org/pypi/protobuf/3.0.0b2.post2), in order to add Python 3 support for the TREZOR ([issue](https://github.com/romanz/trezor-agent/issues/11#issuecomment-216883265)).


@rlorenzo I may have answered this. It is permanent until the user revokes it.


Myoldmopar-patch-1 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2191 of 2191 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)


I think the logic for this was to also support resumability with http downloads. I guess if an http transfer is `paused` then it can be resumed.


Try being the least specific you can in your selectors. Do these styles really depend on the styled heading being in a _div_ called _article-title_, or could we say that all _h1_s should look like this?


i think I will not - that would require patching the very basis of VCL and consequently all the 3rd party components...


Use a space after a property name’s colon.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Property_Name_Stops#Property_Name_Stops


Awesome work, Justen!  Your code is very easy to read, and we like that your CSS naturally follows the flow of the elements on the page.  Also, great to see you using `<article>` and css selectors that you picked up outside of class!


Your welcome! :)


@TimWolla So I think we discovered this issue at our organization... But wow, this repo is completely open to the world.


It was only used for torrents and it is really a remanent of vuze code


Font-weight values can be either a numeric value _or_ a keyword, not both.
https://developer.mozilla.org/en-US/docs/Web/CSS/font-weight


//TO-DO: if (seeding_on && can_seed) { seed() }


Yeah; thanks for the suggestion. I'll change it in a future commit. I think the way you suggest is way more clean.


You can't do that since every possible updates that occured on the server in the meantime will be lost


OK sounds good. Thanks!


VER 550 is defined in loader.h,add it manually.


I'm assuming this is WIP. Can i request that you leave font options in the list of warnings themselves, because this current system appears to only set the warning all in one color. But I was planing on having individual words highlighted depending on context.


I'm confused, isn't that the purpose of claiming we're now the new source of truth?


Look like you may have forgotten to include the image when you pushed to the repo.  Reviewing the console in developer tools for any errors can help prevent this going forward.


Hey @JeetKunDoug why this fix not into 2.1? thanks


DOCTYPE should be all-caps.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Document_Type#Document_Type


When quoting attributes values, use double quotation marks.
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks

Omit the protocol from embedded resources.
https://google.github.io/styleguide/htmlcssguide.xml?showone=Protocol#Protocol

Omit type attributes for style sheets and scripts.
https://google.github.io/styleguide/htmlcssguide.xml?showone=type_Attributes#type_Attributes


Pay attention to indentation. _img_ is inside _body_, so it should be indented one level deeper.


Thanks!


Use meaningful or generic class names instead of presentational names.
https://google.github.io/styleguide/htmlcssguide.xml?showone=ID_and_Class_Naming#ID_and_Class_Naming


好像是教昴功课的时候出现过这个发型?


`y` seems unneeded here.


这里还是改了一下


@gregw can you double check these javadoc changes?  (the nightly / release builds of 10.x were broken due to bad javadoc)


Yes left in by mistake


These 2 methods no longer exist.


jajajaja
se suponía que traería la imagen no?
como quito un commit por cierto


Don't CSS in your HTML


@evilaliv3 Yes sir. Sorry this snuck into the commit.


The `startWebapp()` has been removed.
But replaced with what?


Pay attention to indentation, unnecessary whitespace, general formatting.


Desconsidere estas alterações


Desconsidere essas alterações


Só alterei esta linha, porém não está redirecionando para a página


Hi Fabio, yes agreed, the root file is definitely the "nicer" and more future proof way to go (also in case we ever might need different factors for different analyses). I am also perfectly fine with doing the "selection" according to gen_match at the plotting level.
As I recall the major effect was on ZLL, a very minor one on diboson and (almost) none on all other backgrounds; On a site note: the decision to apply it consistently to all samples using a gen-match of 1 or 3 was made on a rather short timescale, so if there are reasons to limit it to only apply it for the DY ones I think we can also make the point that this change might be good to have. However we need to keep in mind that we are very close to unblinding the MSSM analysis (at which point people naturally become reluctant to change anything not absolutely necessary about the analysis), so for now it might be necessary for the MSSM analysis to stick with applying it to all samples based on the gen-match (which I think this can also be done fairly easily at plotting level when using the EleTauFakeRateWeightProducer).
I will give this a try in the next days and switch to it (including cleanup of the other producer) once I compared it to the current results.


util.js:3014 Uncaught TypeError: Cannot read property 'modal' of undefined


Wouldn't it be better to use error codes provided in the protocol? E.g. "resource locked" or "precondition failed" instead of "queue name collision"? While they may not be as obvious, they are also well understood in the community and are easy to google.


A _hr_ is a self closing/empty/void element. It must not have an end tag.
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/hr


Quick reply.  Hopefully this one actually worked. It seemed to in my testing.


C'mon, we are well past "0.0.1". If I had to pick a version for now, I'd go with "0.5.0" and slowly climb to "1.0.0" (over how many minor releases we feel comfortable with).


Pay attention to indentation. Closing brackets should be on the same level as their opening selector.


If you replace the `(.)` after `nfIO` with a `($)`, somehow `gen` gets reapplied to `n` at every run, which is bad. `bg'` should do that but not `bg`.


Try to be the least specific in your selectors. Should all elements that has the class 'center' be aligned center, or just the images that have said class?


Hey Boris ! I've just realised that with this check you will be unable to set startBitrate to 0. Maybe it should be changed to !== undefined ? WDYT ? or we will never want 0 ?


Aybssssssss :laughing:


https://plus.google.com/102341784722576412208/posts/FLTXakYJ4EE
https://plus.google.com/+LarsFosdal/posts/Tdg66UtBRMc
https://plus.google.com/104660942073125276831/posts/YGb9Mr8QdgN


Go to /bone101 so that I can update that forward page as needed.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 123](https://teamcity.labkey.org/viewLog.html?buildId=321879&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7 (2 new), passed: 48 Build time: 01:40:14
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cds.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

```


Commit the changes


Great! Good luck for LTC!


Do you want to hardcore the IP address or protocol here? I think no.


@vlachoudis with defaults value (0) the message is show at every startup. Quite annoying. What about change the default value to something really small (-1)? Or show it only before tool change cycle begin?


I like this. Should we merge to master?


I submit the issue in version 6.0.0 vs 0.5.8 greater zoom stops per frame image


Oh, right. I don't think we ever want to use 0, though.


Backwards compatibility and closing #14


awesome work using so many semantic elements!


In cases like this one it might be beneficial to use the **`yarp::os::LockGuard`** mechanism.


Some sites have problems with this:

For example, example.com:

![badexample com](https://cloud.githubusercontent.com/assets/8844455/15028657/ea531532-1217-11e6-9972-356c47c3deff.png)

Do you know what is causing this?


Hi Rene,
ah this is new to me. From Christian slides of a couple of weeks ago the recommendation was to apply the SF only to the ZLL template. If this needs to be applied consistently to all samples, then it makes more sense to use a gen-matching inside the Artus producer, because I think that it would be more tricky to do this at plotting level (it is straightforward for the ZLL templates because we have already these weights:
https://github.com/cms-analysis/HiggsAnalysis-KITHiggsToTauTau/blob/master/python/plotting/configs/samples_run2.py#L32-L34
https://github.com/cms-analysis/HiggsAnalysis-KITHiggsToTauTau/blob/master/python/plotting/configs/samples_run2.py#L51-L52 and the e->tau SF could have been appended to those)


It seems there's a `lock` here which is wrongly placed.


Great to see you making multiple commits for this assignment !


Other than that, I like it. I think it's unconventional to put content outside the `<body>` element (like [Hunter said](https://github.com/BarlowProgramming/redirect/pull/22)) but it does work well, except for the few cases like example.com.


удаляет пустой массив


greater what zoom ?
I'm sorry but I don't fully understand you.


@Reflejo When I run `build.py` with this change I get a failure at the very end:

>🚀  Copying Info.plist ...
Traceback (most recent call last):
  File "./build.py", line 131, in <module>
    main()
  File "./build.py", line 115, in main
    shutil.copy("./Info.plist", framework)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/shutil.py", line 119, in copy
    copyfile(src, dst)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/shutil.py", line 82, in copyfile
    with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: './Info.plist'

Any ideas how can I fix it? If I use the old version I get another problem when trying to run the app that complains about the lack of Info.plist in GoogleMaps.framework.


Awesome to see you using relative units here with the "em" and "%" with your widths


greater what zoom ? - stop cage image


last update


@NeighborhoodInfoDC/review I was wrong, upon further investigation the Premise ID is used in matching properties so we need to find a work around for this for the April data. It may only be missing from this month or it could be a permanent update.


Right. I added that check before I added the `foreach`. Fixed.


stop cage image? If you could post a screenshot, we could understand you better.


Is the content type case sensitive?


Hi there! Any plan to release that commit on npm? Seems npm only knows about beta.3 while there is beta.7 in github.
Thanks!


max zoom = freeze image


@cgchoffman fwiw you can configure multiple emails on github.


With the `a` tag inside of the `button`, only the text will be clickable.  Next time, try wrapping the entire `button` in an `a` tag, like this, so the whole button is clickable

`<a href="https://www.facebook.com/"><button>Facebook</button></a>`


Myoldmopar-patch-1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1635 of 1635 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.92%25-green.svg)


That's a good idea, I've filed a story: https://www.pivotaltracker.com/story/show/118964183

QueueNameCollision is the case when the client's random generation produces a dupe. Doesn't quite match the `<doc type="scenario">` in the spec for resource-locked, but it's close enough.

There might well be internal errors that don't align with the protocol error codes, as they generally describe what goes on between client and server. If they're hard to pigeon-hole, I guess we should make new errors?

Also, thanks for drawing my attention to this duplicated enum!


@SuperLamic those denies are still present until using permissive .... also lags has nothing to do with SELinux


What's the reason for changing the test from html to rich text?


Myoldmopar-patch-1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Você tem que parar de editar arquivos sem necessidade


Faz um commit adicionando sua página de faq


Without casting to bool, AddValue(code, short val) overload will be used.


So when you fully zoom out, it freezes ?


Why are you using `cover?` here instead of checking that `follow_up_sent_at` is `nil`?


I am running into this issue on RMySQL_0.10.8 - is there a fix?


You're right. I can jump it up a few, as we should've started on 0.1.0 anyway.


Thanks! I didn't understand why it failed when it passed in the PR.


Yes.


No notes.


En realidad la incidencia es la #36


misleading var name?


Follow the coding guide for the comments. Here we won't need any. The comments are in the Header files only, to be documented as part of Doxygen automatic doc generation.


This is great. :)


Either there is a tab in the comment, or extra spaces. Fix


Bitte nicht immer dieses auskommentierten kram in den master pushen


This is perfect too. 👍


Fixed.


committing


Please mame an issue if this is the case. Refer to the wiki for making an issue correctly.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#EH2sZz5JTFWgjJ2l5WwoWg


Please also post a screenshot. It helps *a lot* to see it visually.


b could be more descriptive


It is also natural to have a higher CPU load - we are avoiding enemies here ;)


Use of Frame outside of the event handlers are prohibited. I am not sure why are you using the frame here. But if this action is necessary, it should be part of the event handler of the prefetcher and not a stand-alone function. You can clarify the use via responding to this comment :)


If all you do is getting the module, get it via prefetcher->cache or prefetcher directly.


you can use self closing tag for this.


Same here.


Just so you know, I dont think this actually does anything we would use as debug info, it just adds the `ng-scope` and `ng-isolated-scope` on the every angular element so you can easily hook into the scope of the object using console or use third-party angular tools (like batarang) however these don't even work with DuckieTV because it's a chrome extension and other apps can't do things with other chrome apps so, I think it would be better to leave it disabled as it drastically increases the speed of rendering.


OK. about 5 minutes wants to help get the error


On line 147 (and elsewhere in this file), changing the content type in the request to use string interpolation might make it more readable?


:heart:


Remove commented out code. No commented out code.


this is very bad

the file is require every time we need to call a function in the API
Consequences : so much latency...


there should be a space between the attribute names and values


:+1: :+1: :+1:


**important**


This is bad practice to have `/node_modules` in a require


No FIXME either. Take into account the type of prefetcher now. Via a switch for example. For non-implemented types you can add a warning/error that indicate the type is not supported yet


remove


same as above


We're currently only using the reset and clearfix functionality of compass. That's not enough for me to stick with some broken software. I'll add normalize.css for the 'reset' part. (I'm not sure it's actually necessary, since fava only works with modern browsers anyway)


this file is added, he provides informations to know is the auth is required or not


not sure it matters much, but could we create the readblock after the initialization of isManaged, res and matchedres?


Since this warning is here, you won't encounter other cases (in the comment above). So use a switch with default throwing an error (an error in this case shows that the type was wrongly set, and is fail-safe)


add comment what this does. Specially in the event handler we need consistent comment or otherwise it can become very confusing


return


Also, do we need to hold the readlock after line 561?


yes


Fine


cache != NULL ?? How that can be a thing? Use assert (which is active only with --enable-debug). There should be a module to load from, and each module has to have a cache


Congrats!


same here


I am working on my homework. Currently I have the version number and I can use a zip file. I only need to add the functionality to read out the downloadlink.
Maybe you can upload the new simple Layout, because my buildroot system wont work.


if no auth go to default profile


It could be caused by `window.loop` getting called twice.


while loop ...


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#cqPMJIhNTiOOi5CqwX71JA


one line comment. // Debug and trace information
The rest looks great


comment+code+space. here comment is missing


do we need to hold the readlock past line 308?


This won't stop already submitted tasks. It might need to be `shutdownNow()`.


create file and put content in it should be in one commit :)


http://s32.postimg.org/tcmicrsrp/scinka.jpg

http://s32.postimg.org/yp6rd5rad/scinka2.jpg


should we shorten this to num_useless_prefetches


please use 2 spaces for indentation:
https://google.github.io/styleguide/htmlcssguide.xml?showone=Indentation#Indentation


Should the content type be non-null in this test?


It should be on the same level like:
```
<title>How She Got There — Galleys — Medium</title>
```
or nested:
```
<title>
  How She Got There — Galleys — Medium
</title>
```


`[version]` needs to be removed from the URL


same thing


Contant --> Content?


Sorry, Katya. I've been in meetings and am just seeing this. There's still an issue?

On May 4, 2016, at 4:54 PM, katyaabaz <notifications@github.com<mailto:notifications@github.com>> wrote:

@NeighborhoodInfoDC/review<https://github.com/orgs/NeighborhoodInfoDC/teams/review> I was wrong, upon further investigation the Premise ID is used in matching properties so we need to find a work around for this for the April data. It may only be missing from this month or it could be a permanent update.

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly or view it on GitHub<https://github.com/NeighborhoodInfoDC/HUD/commit/3f7fb5958bf17353bbc00ebb8e765617492fba01#commitcomment-17359056>


you should use double quotes:
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks


Same thing


Forgot, this is part of #141.


use indentation


this `:param:` can be removed. The input description is good though, I'd suggest moving that down to the form parameter below.


I like the rest. It follows the current format perfectly. It might end-up changing after the tests but for now they look very good 👍


unnecessary closing braket


I need to hold the lock because the "user" pointer should not be at risk of being invalidated by another thread deleting that user


Wrong indentation for the whole article block


Yes, the Premise ID wasn’t included in the raw data. There’s a note on the website saying so, but I don’t know how the Premise ID is used because I haven’t dug into the macros yet, so you may know off hand whether that is an easy fix or not.

From: Peter Tatian [mailto:notifications@github.com]
Sent: Wednesday, May 04, 2016 5:23 PM
To: NeighborhoodInfoDC/HUD
Cc: katyaabaz; Comment
Subject: Re: [NeighborhoodInfoDC/HUD] Updating MFIS data for 04-30-2016 (3f7fb59)

Sorry, Katya. I've been in meetings and am just seeing this. There's still an issue?

On May 4, 2016, at 4:54 PM, katyaabaz <notifications@github.com<mailto:notifications@github.com>> wrote:

@NeighborhoodInfoDC/review<https://github.com/orgs/NeighborhoodInfoDC/teams/review> I was wrong, upon further investigation the Premise ID is used in matching properties so we need to find a work around for this for the April data. It may only be missing from this month or it could be a permanent update.

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly or view it on GitHub<https://github.com/NeighborhoodInfoDC/HUD/commit/3f7fb5958bf17353bbc00ebb8e765617492fba01#commitcomment-17359056>

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub<https://github.com/NeighborhoodInfoDC/HUD/commit/3f7fb5958bf17353bbc00ebb8e765617492fba01#commitcomment-17359440>


@towardthesea
Actually, the check should be something like this:
```cpp
if (isPresent=reply.check("table"))
    tableHeight=reply.find("table").asDouble();
```


Remove or add code corresponding to the comment. If this is a comment for the following piece of code, connect them together:
// We don't ....
//
//
//
//Prefetcher statistics.


Hit on what?


please install the latest version. Does it still occur?


...or even something like this:
```cpp
isPresent=reply.check("table",Value(0.0)).asDouble();
```


same as before. More descriptive. How this corresponds to be a miss. and miss on what?


great to see you using css properties we haven't covered in class yet!


Unnecessary `p`, you don't have other `.divider`. Always use the simplest selector as possible!


`strncpy` does not assure that the result is NULL terminated (it is not if the source string is longer than the destination size). This makes this look unsafe. That's why in general we try to use `Common::strlcpy` (which is basically OpenBSD's `strlcpy`) intead of the usually poorly understood `strncpy`. This function gives these guarantees.


missing space


ERROR: /mnt/data/frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java:771: The method updateBatterySBHTextColor() in the type StatusBarHeaderView is not applicable for the arguments (boolean)
 same error in this file for
updateSBHIconColor
updateBatterySBHIconColor and updateCarrierLabelColor


Leave shorter //TODO. Comments should not be in the format /* */ . Never. Won't allow doxygen to document it.


I need to hold the lock in 561 because subsequent code modifies "matchedRes" and "managedScope"


Remove commented code. // TODO should be indicative of this.


@nataliapanferova @hishnash a replacement suggestion to the *evaluate* method:

```js
this.evaluate = function() {
	var regexFlags = 'g';

	if (this.ignoreCase) {
		regexFlags += 'i';
	}

	if (this.multiline) {
		regexFlags += 'm';
	}

	var regex = new RegExp(this.re, regexFlags);
	var m = regex.exec(this.input);

	if (m != null) {
		if (m.length === 1) {
			return m[0];
		}

		return m[1];
	}

	return '';
}
```


Awesome to see the responsive nav and h1 adjustments!


same as above


you should use double quotes:
https://google.github.io/styleguide/htmlcssguide.xml?showone=HTML_Quotation_Marks#HTML_Quotation_Marks


Sweet! make sure you call the final method my_mergesort to be consistent and so driver.c still works


unecessary newlines


OK answer here

Thank you for fast help :)


If statments should follow the coding guide-line:
if (condition)
{
      code
}
else
{
}

brackets shouldn't be in the same line as if ans else.


Breaks tests...


same here. Completly fix of format here.


I clone your Fork, rotating npm install, error bursts everywhere, I can not make it work.


no fixme. Why fixme?


unnecessary newline


fix the statment brackets


you shouldn't use characters like this in your HTML:
https://google.github.io/styleguide/htmlcssguide.xml?showone=Entity_References#Entity_References


Should these tests remain to ensure the "sensible default" is not changed?


Yea. Im working on database hw with friends. Just threw the code I found
online and will tinker with it tonight.
On May 4, 2016 5:26 PM, "Evan Shone" <notifications@github.com> wrote:

> Sweet! make sure you call the final method my_mergesort to be consistent
> and so driver.c still works
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/evanshone7/cisc360/commit/93a8d58a5d18e29d99497e2cdc9855cdd7395c44#commitcomment-17359515>
>


unnecessary newlines


@FernetMenta when I remove `m_renderManager.WaitForBuffer(bAbortOutput)`, some games work but some just show a black screen. Is blocking mandatory? Could this be related to restricting RenderManager to a single buffer?


Just installing `libstdc++6` should get you the c++11 capable standard c++ library. Installing `g++-5` will pull in a lot more that you likely don't need.


same here.


Shouldn't we leave a chance to submitted tasks to run before shutting down ?


unnecessary empty line


Hmm, why is it not used - the first {} printed the message, the second {} the entire ex?


and again :)


For next time, keep in mind that IDs have to be unique among elements..so only one element could have `id="socials"` on a single page.  This would be a great place to use classes, like this:

```
<li><a class="socials" href="https://twitter.com/girlshawn35">Twitter</a></li>
<li><a class="socials" href="https://www.instagram.com/girlshawn35/">Instagram</a></li>
```


I tried this which seems to work: https://github.com/thumbtack/GoogleMapsFramework/commit/9c54b3a3155c481d38c9e14364d7f5e183d522a6


This PR broke SciGraph, because two CURIEs resolve to the same prefix. SciGraph is not able to do the inverse resolution (IRI to CURIE).

In general I don't think it makes much sense to have duplicates in either the keys or values, we have to be able to convert in both directions.