As of TODAY, `en-US`
```
JSON.stringify(Services.search.getDefaultEngines().map((e)=>{return e.getSubmission("FAKE").uri.spec}), null, 2)
"[
  "https://www.google.com/search?q=FAKE&ie=utf-8&oe=utf-8",
  "https://search.yahoo.com/yhs/search?p=FAKE&ei=UTF-8&hspart=mozilla",
  "https://www.bing.com/search?q=FAKE&pc=MOZI",
  "http://www.amazon.com/exec/obidos/external-search/?field-keywords=FAKE&mode=blended&tag=mozilla-20&sourceid=Mozilla-search",
  "https://duckduckgo.com/?q=FAKE",
  "http://rover.ebay.com/rover/1/711-47294-18009-3/4?mfe=search&mpre=http://www.ebay.com/sch/i.html?_nkw=FAKE",
  "https://twitter.com/search?q=FAKE&partner=Firefox&source=desktop-search",
  "https://en.wikipedia.org/wiki/Special:Search?search=FAKE&sourceid=Mozilla-search"
]"
```

Issues:
- varies by locale
- unclear for some how they differ as url searchs vs urls from search pages.
- things with 'page 2' and such... is that a new search in your system?  (that's a "follow-on").  They will be counted here.

In general, it's not totally clear how closely these queries track the number of searches.  My claim is that these are an undercount, but I could be wrong.


SPOILER GARNET IS COOL :two_women_holding_hands:


probably the travis example they gave is either for php or rake, I googled to find you an example that matches what your project is using: `travis example angular`

https://www.google.ca/search?espv=2&q=travis.yml+example+angular&oq=travis.yml+example+angular

Try this result:

http://willmendesneto.github.io/2015/02/16/using-travis-ci-for-run-angularjs-unit-tests/


Meep


@oritwas we shouldn't assume that name and id couldn't collide. We should prefix them with different prefix to ensure that.


Ping @kevva wdyt?


do you know by now what this was for?


s/Build/build


@surdupetru  P8 is 64bits kirin 930


add line break after method header


add linke break


s/repo/repo:


no, I have no clue


That makes sense.


Claim / idea:

- this marks the uploaded packet as "dirty".   I would try to only use 'clean' packets in analysis.


Node directions for LH Valkyrie (Viking)


Based on input from Mr. Phillips, we will be going with a different solution. Clearing the float in a css file will achieve the same result.


I still have a white frame around the workspace and the doc list.


maybe github messes up, but last line should have one more tab than `^ super ...`


why is `toogle note` a string instead of a symbol?


There are ways of Doing It Right, but they are really hard, and require parsing all the urls.   These also have changed over time.


you have to leave this part as we still support oEmbed.


If that's the case, then I think the best solution is to move `prop_type` up to the `jtx` namespace and get rid of the `detail` namespace entirely. The second best solution would be to move both `basic_prop` and `prop_type` back into `JTx.h`. I don't see any reason to put them into `Env.h`, though.


any define somewhere we could use for this 37?


same thing, Kodi either strarts to act crazy or insta-crashes back to command line.


s/kubectl/kubectl:


add line break


Extra blank line removed.


hah: remove line break? have comment directly below the method header, then add a line break?


LGTM, will test tomorrow


Had same issues in chrome/win8 but seemed to be related to angular 1.4.1 ... The error disappeared with 1.4.0
Anyway Thanks!


can we do this in a deployhook instead of the run script?  talk to @mfojtik, he knows how to do that now.


@tuchida I have been looking at this very old PR as well to try and get it in.

I had to make a few merges, which I made in the branch "tuchida-for-of_loop". The test passes except for the test on "for (var n of null) {}".

Can you take a look at the branch and see if the merge looks OK to you? I'd also like to see if we can find an external test for this, although I can't find one easily in the V8 test suite.


i know this works. but wouldn't it be nice to test isCollapsed directly with a boolean?


Maybe update a timestamp instead, to avoid building up a huge test page in production?


I would do the counts not as percentages for these reasons:

- being able to debug specific kinds of errors.
- no loss of precision possible.
- being able to throw out "huge histories" or other (yet to be discovered) cases.


k


You're more than welcome to incorporate dynareOBC too: https://github.com/tholden/dynareOBC
Seems easier to use, and is both more flexible, and should be more accurate.


because `toggle note` is the tooltip text


Idea:  add a few 'canary' domains.  (I would pick netflix.com, facebook.com, youtube?) for reference.   This allows saying "search is more common than going to facebook" or the reverse.


tmp for sessions is fine for now, we have client affinity routing.  we can add DB storage later.


I think you're right. I needed this for a module to make sure a value isn't `<0` but I guess that isn't the expected behavior.


Static import


knowing nothing about places, is this right?


@who added Android template changes.


Why you don't use "bindValueIfExists" or similar?


thanks for your instant response i'm updating the bundle to the 0.5 version. I will then discover the bundle.


+1, app needs to work with arbitrary database service name, so it should key off the DATABASE_SERVICE_NAME env variable and determine the rest from there.


why new test on instance?


Why did you create a global function?


space after `during:`


Cool!  :)


Should this have been made private? The prose in the docs above seem to indicate that the application developer is intended to be able to use this, and we've been using it to great success for a while.


todo?


Does this work for both Mongo 2.4 and Mongo 2.6?


I'd leave this out so we get a prompt AttributeError if a subclass forgets to fill it out.


Thanks for the note! Yes this was to fix an OS X bug. Will continue thinking of ways to remove it now its only on Windows.


Do the files happen to have .JPG as the extension?


coding standards: add linebreak after method header


@nodiscc I've set the milestone for #456 to beta11. Before merging it gets merged, there should be an "alert" message (described in https://github.com/RequestPolicyContinued/requestpolicy/issues/456#issuecomment-103533732). And, I'll create a unit test. Then I'll gladly merge this feature.


I was to fast with committing my stuff here. This message was describing the next change.


utyutyuytu


choose: spaces in brackets or no spaces


Can you do a clean log that I can post to possibly get this fixed? (yeah I know there is no data on the crash, but they will probably ask)


Options:
1.  try to shoe horn this into telemetry.  + exists in Fx,   - hard to get data back out (for me)
2.   stand up a service.  - has to be built, tested, debugged.  + you control it.

I don't know which tradeoffs you like more.   IF you go telmetry... I would do a 'keyed histogram' and the key of historgram is the jsonified   'counts' object.


Good point on the empty access parameter. I added the handling in this caller function but it's probably better doing it within DB::getDBConfig(). In that I can also leave getDBConfigDSN alone.


I don't like the parameter there, it should only be able to schedule when the input changed in the first place.


It is a strange function.


I don't think this example works as-is. Either we need to change the exported function in `minim` or change the example.


libdirs+=( "${libdir}" )


The first paragraph of your answer is important additional information which should be merged into the answer I think.

The current concept of the questions and answers section is easy describe and to follow. We wrote down the questions we received during our presentations and we tried to answer them (the length of the answer could be proportional to the frequency of the questions).


I suspect the maxunit fix you have here is addressing what caused the issue I raised earlier, seeing how your solution was more or less what I ended up trying, although my semi-synced state meant it didn't actually work when I did it.


ImageStreamTag


this all looks fine to me.  A bit repetitive.  `Promise.all()` exists.


cakephp-example:latest


I hate UUID ids for addons :)


Haven't tested on 2.4. I'll try tomorrow, GMT+1 over here.


this will need something :)  Own your work! and face the yelling just like the rest of us.


With the control structure from your example I get this output:
```
origResidNorm: 0
origResidNorm: 1
  Starting FGMRES iteration 0
    original rel error: 1.11022e-16
    1.11022e-16 <= 1.05367e-08
  finished iteration 0 with relResidNorm=1.05367e-08
    original rel error: 0
    0 <= 1.05367e-08
Uncaught exception
Uncaught exception
Process 0 caught error message:
Lucky breakdowns not yet handled
 (with castExcept.what()=std::bad_cast)
```
Any ideas?


Thanks a lot ! I was actually trying this exact solution right now :)
I'll check this out tomorrow and merge it ;-)
Thank you very much, you'll be credited.


remove point after [check]
remove space after do: [


OK


You should call this file "contrail-vrouter.sh" so that rjil::test will find it.


> "Not at all, the more info you can provide the better"

considering the fact that is exactly what you didn't do when you were the secretary, that kind of invalidates your point a bit. I used your previous posts as examples for how to make the posts, so you basically set a bad example, and now that is how I want all new posts to be because it follows the previous style of note taking and the fact that the notes don't really get looked at, and if someone does look at them they can read simplified posts and not posts that have a large amounts of unnecessary information in it. The vote was informal to being with and its only goal was for you to get the opinion about what the website should look like, which you got. If you want to remember this sort of information, you can write it down yourself somewhere, but it does not need to be apart of the ACM minutes. The important part is how many votes each one got because that is the primary deciding factor.

> "Any hand written notes are entirely unofficial and as far as the organization is concerned, do not exist - that is just an extra step you decided to take to make things easier on yourself."

they could become official, they just aren't because the meetings are always all over the place and we move from topic to topic at random times, and sometimes come back to topic later, and they are just a mess. so I try and make it look like our meetings actually make sense and organize what we talked about into a logical fashion.

> "Either way the additional info hurts nothing."

The information you added just makes it look bad and is completely unnecessary, and it doesn't have any sort of logical pattern to it, it just has a massive amount of indents.

on a side note, you were doing my job for me, which I don't really appreciate because I was present at the meeting, yet you still took minutes, so we could have avoided this whole argument if you just let me do my job.


Fair enough


I think thats a good idea


Golint [does not seem happy](https://travis-ci.org/weaveworks/weave/builds/67054131) with this...


OK, so according to the discussion during the previous Sprint Review, the priorities are being removed. Good.


Thanks, I updated the README and lit metadata.


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

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


fix #7294


unsupported as any !linux platform


this should be classroom.example.com


Is that ever set? It looks like a cache variable that never gets populated :).


hi


Good catch. I think it should be `registry.toElement`


Can you fix all such occurrences?


Yep, with the change to a FloatValue, the unittests change similarly. Nice.


OK


`local libdir libdirs=()` -- in one line ;).


As mentioned in the PR description: I did do this by hand but the diff is so large that I couldn't focus on each and every scenario as much was deserved.

Specifically see this:

> Please note: This PR makes an initial attempt based on my best understanding. These @public and @private declarations can absolutely be reviewed and changed as needed.

In light of that, please open a PR updating to mark as `@public` and we can decide each on a case by case basis.


That's not right either. I think it should be `minim.convertToElement`


You meant `local libdirs=( $(get_libdirs) )`. Otherwise you won't get an array, so it will always be -eq 1…


I appreciate it's a stumbling block - do you think this could be a documentation issue?

I guess the idea is that these polyfills aren't needed in many environments, and can also be polyfilled using other means too, so I don't mean to impose the Promise or URL polyfill implementation, or the extra 5KB on everyone else.


In bash you usually use `0` for truth.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 68](https://teamcity.labkey.org/viewLog.html?buildId=255323&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:08
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


It's to pass already evaluated arguments to child nodes.
A sort of shortcut to compose the nodes yourself in specializations.


Don't use `[` in bash, it's dangerous. Use `[[` which doesn't expand variables prior to processing conditionals.


Skip this commit!


Der er vist en integrations spec i url dictionary som tester at nøglerne er tilstede for de forskellige site keys. Skal vi ikke opdatere den (eller fjerne den) ?


Looks good!


Fixed


Uuuuuh sorry but this is definitely not the right way to do it.
We have to change the way the commands are handled to allow them being used by command blocks, but certainly not change the pattern that executes those commands. This is complacting the plugin a lot. If you don't get what i mean, i'll implement it myself


why? in this way the note can be collapsed or not by default


Creepy and wrong. Don't try to convert libdir suffix into something it ain't. Remember that the new multilib profiles will have `lib` for 32-bit libs and `lib64` for 64-bit.


here you go: http://xbmclogs.com/p3gznonzj


so do we want to have notes collapsed by default or not? we can test the default behavior with explicit booleans


Relying on variables being declared locally is confusing. Rather pass that as a positional parameter.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


In general I wouldn't replace "Spendable" with "Sendable." This code is commented out anyway.


Overall notes:

1.  Sample size.  Sample size is kind of strange to describe without talking about size of effect, and what the competing hypothesis is.  These are mostly percentages.  For the total (especially), you will have precision of within a point even with "not very many" O(1000).

2.  The biggest weak points here are
  a. urls aren't searches (except when they are
   b. regexes are terrible for this stuff.  (but so, so cheap :) )
   c.  this study doesn't let you say enough.  (see below)

3.  Code itself is good, safe and efficient.
 a/  Queries are async.    GOOD
 b/ The queries themselves have to do full table scan because it's an UNanchored "Like".   You could speed it up by using the 'full left side" of the queries strings.  (like "https://www.google.com/search?q=").  I don't know if this is a practical problem.
  c/ switching to full left side would reduce the 'weirdness'  a bit, at the cost of turning your estimator into an undercount.  I also don't know how practical this is.
  d.  how do you wanto handle searches within google / yahoo?

4.  The things *I* would want to know are
 a.  how do searches break down?   navigational?  Sites?  Multiterm?  Repeated?
 b.  how often are searches repeated?

I would want to know those things, b/c the seem like great places to build UI.

This study doesn't address those points at all.  It could do so by parsing the 'q=' arg for queries.   This may or may not be worthwhile.

5.  Predictions:
- total searches will be 5-20% of history.  If it's 50% I would be shocked.


Hello, currently I am working on a module for recognition of facial expressions in real time, I would like to know more about your work, could you provide me the paper pertaining to your project? , my email is m14240372@itleon.edu.mx
Thank you!!


Ein Teil vom Übungsblatt war es, dass man das Zeug, was in der getUrlUsernameTables() Methode ausgegeben wird, halt ausgegeben wird.
Damit das autark vom Rest funktioniert, habe ich halt ne zweite Main-Methode hinzugefügt ;)


Hmmm the principle of the clock is to be set for each receiver. Here you set a timer for all those clock receivers. I don't think it's working as expected in real situation. Besides, many many imports to remove.


I might put a 'belt and suspender' check for locale here.  If it's not en-us, throw and break.  just in case.

OR, upload locale too, so you can do it at the server.


Selbe Begründung wie oben ;) Da es eine autarke Main-Methode ist, muss ich das Objekt neu erzeugen ;)


Just a little update. Both issues have been fixed (RSA key pair and useless port exposed). The key pair is generated at runtime on the host machine and mounted into the concerned containers.


The receiver shall not use a method that changes the state of the channel containing this receiver. This will create a loop in game i think.


And anyway, I'm not sure if it's a good idea to allow combining different wxwidgets versions for different ABIs. The module would certainly be simpler if you just set the same thing for all libdirs (this is e.g. how eselect-opengl does it).


Fun with breaking things!  I have one file locked and cant pull or push. I have to fix it up in the work tree and then use git add/rm <file>  I tried to skip the tree and just removed the file, no tango! :dancer:
git keeps it referenced even with the file removed and I'm sure it can be restored as well.


r+


This is not the medium for this kind of discussion and I hope that you realize that your comments are somewhat immortalized, due to the fact that


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


merging jedmccaleb/stellar-core/master = 197eb094 into auto


jedmccaleb/stellar-core/master = 197eb094 merged ok, testing candidate = bb1fd73f


Method Project::scopeTemplates does not have a Docblock comment.


Method Project::scopeNotTemplates does not have a Docblock comment.


Marge


Because in order to do that you would need your dev box to be in a hosted environment, accessible to the universe? Which is not a thing that most people have the capacity for, and which is inadvisable for a dev box. See also: problems getting our personal Jenkins to use Web hooks all those aeons ago


This will simply install the script.

You need to add a rjil::jiocloud::consul::service resource, too, to add it to consul.

I think this will do the trick:

  rjil::jiocloud::consul::service { 'contrail-vrouter-check':
    interval      => '10s',
    check_command => '/usr/lib/jiocloud/tests/contrail-vrouter-check.sh',
  }


Por que isso foi deletado?


Should that also apply to other python versions?


Good job messing up the javadoc in everything.


@dylanwenzlau
I noticed: https://github.com/FindTheBest/DB/blob/master/src/DB.php#L153-156
So we did take care of empty $access parameter. We should be good


@dylanwenzlau i think this looks good.  Only question I have is whether or not this is a SQL standard or just a MySQL standard.


@dylanwenzlau Fixed.


Do we need this?


Updated to Version 2.


* TODO: rewrite this as it is doing way too much and is very messy now.


No it will set it for each individual receiver


looks good


> I appreciate it's a stumbling block - do you think this could be a documentation issue?

It could be.


I'd probably yank this and just use AppConstants directly, but happy with this if you like it :)


If you're doing this, you may consider to use the `functools.wraps` decorator. It helps to keep the correct "metadata" on the function.

Also, but this is personal taste, you could use a better variable name than `thing` ;-)


Ah, JSHint will tell you about stylistic stuff like having 1 space instead of 2, or missing a semi-colon. The point is that you shouldn't need to worry about it, jshint allows that. But the significance is that it's become an expectation that there will be no /easy/ syntax flaws because you should be using jshint to fix those.


// FIXME: Should this be an event rather than a command?


looks good to me lets do it


:smile: :hamster:


Uh, profiling? Also, I am not sure -m64 is in any way portable across more than x86_64? Or is this not a general-use CFLAGS? Sorry, I only looked at the commit.


Okey. I don't have much experience with commands yet


Whoa...you can just call `.ids`? That's awesome!  Are only ids supported, or can you pluck anything you want, like `.names`?


PHPCS: Expected 1 newline after opening brace; 2 found


They're here: https://github.com/Mikeronomicon, about halfway down. https://www.evernote.com/l/ADEnm80RgD9Lpb1ZGy8q-vpDZ5frGFIRYws


// FIXME: Also copy persistent files


@rochacbruno poderia falar mais sobre isso ?


Is this debug code, or does this just make it consistent? (I can't remember)


Also, it does look like you got it sorted out. :)


Great :+1:
After completion merge slot 1 and 2 :smile:


Jo danke ;) das habe ich jetzt mal so übernommen :D sieht sinnig aus.

> Am 16.06.2015 um 23:07 schrieb UntilSunrise <notifications@github.com>:
>
> Ein Teil vom Übungsblatt war es, dass man das Zeug, was in der getUrlUsernameTables() Methode ausgegeben wird, halt ausgegeben wird.
> Damit das autark vom Rest funktioniert, habe ich halt ne zweite Main-Methode hinzugefügt ;)
>
> —
> Reply to this email directly or view it on GitHub.
>

_________________________________________________________________
Firemail.de Freemail - 10 GB f�r Mails und Daten
POP3 und IMAP kostenfrei
http://www.firemail.de/


Only ids; it's a shortcut for `pluck`, just because it's particularly useful.


I missed this one. I'm not sure why we need to play this `setattr`/`getattr` dance here.

Wouldn't

```python
self._position = position
```

be enough?


printf("(Unsupported)\nThis version is no longer supported! Use GTALua 1.0.0!\n"); ??

=> printf("(Unsupported)\nThis version is no longer supported! Please update your game!\n");

I think it's better right?


*argh* `untrack` was my mistake.

Regarding `reponame="outside"`: Should we adjust the test [“track outside git repo”](https://github.com/github/git-lfs/blob/b756d5611613b6eb55ae359dda3e8e86f7dac3d5/test/test-track.sh#L87) accordingly?


You were thinking you had many problems to solve :)


I did see that getDBConfig returns early if $access === ''. My point is that the callbacks will not be called, and thus the return value might not have any hostnames at all, so you wouldn't be able to use the information from getDBConfig('some_db_name', '') to connect to the database.


Created https://github.com/systemjs/systemjs/issues/525.


przepraszam,  niechcący to napisałem


I think I finally got a deadlock free code for this. Now, by default
it updates the report every 60 seconds if there is any task
waiting (I'll try to make this time configurable later).
Give it a spin and let me know how it works.


Michał Puzik to zjeb


a chuj kogo to obchodzi


I had to add an updateConstraints here to adjust things back?


wrong name


@trystant, is this a merge commit?


Minor. Since you're assigning a keyword argument, you don't need spaces around the `=`

```python
parameters.sort(key=lambda x: x.name)
```


@bcclaywell I'm baking these into the next release.  Any more packages that might be useful?


Hey jacklekk, thanks for noticing and flagging the issue, I've just added

``` javascript
window.console = window.console || {
        log: function () {
        }
    };
```

inside the file footer IE8 Shims so they are all at the same place.

Thanks again


where does the http://horizon. bit come from?


Thanks for the PR. I've added a couple of "random" comments in the code. ;)


You could probably set this in the animation and have these animate nicely. Seems like a good follow up though. No need to hold this on it :)


Remove.


develop (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


I understand that you want to conditionally avoid the fetch; I'm just suggesting changing how it's implemented for clarity and flexibility


Does `find_by...` return a single item? Should this just switch to .where(:provider => @provider).first?


GTFock is not really meant to be used by anyone right now.  All of these
changes are being checked into the public branch so one of my
collaborators can get them easily.  The profiling and -m64 flag you are
seeing are only being activated when you build GTFock (and I forgot that
I hard coded the profiling flag in there for it. Thanks for reminding
me, that needs fixed...).

Ryan

On Tue, 2015-06-16 at 14:21 -0700, Michael Banck wrote:
> Uh, profiling? Also, I am not sure -m64 is in any way portable across
> more than x86_64? Or is this not a general-use CFLAGS? Sorry, I only
> looked at the commit.
>
> —
> Reply to this email directly or view it on GitHub.
>
>


@moritzdannhauer @jessdtate Not sure how this line got moved here, inside a tight loop, from up above (line 76). This is the cause of the massive slowdown described in issue #1029. I will commit the fix.


We should add more information to the crash. Probably include the channel name and possibly the arguments. Let's try to make it as similar to the GenServer report as possible?


Thanks.


yep, didn't see that. I will fix in my next commit (in this PR or not)


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

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


rax retry tests


@Johnnykahhaleh, the page title should not be hard coded in any of the pages.
I already fixed this for you.


Ah, probably. Good call.


This technique won't work if you have a server that starts with '0' to '9'. Those are valid names (consider `114dns.com`).

A better way of doing this is using the `net.ParseIP()` function:
```go
if net.ParseIP(ServerInput) ==  nil {
```


@Johnnykahhaleh, body tag should include a class that identifies which page we are.
I already fixed this for you.


:shipit:


@Johnnykahhaleh, includes should not be commented out.
I already fixed this for you.


Прогони код на pep8, есть небольшие недочёты


саботаж!


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


@Johnnykahhaleh, individual pages should not have a footer - these are provided by the global includes.
I already fixed this for you.


PHPCS: Line exceeds 120 characters; contains 124 characters


PHPCS: Whitespace found at end of line


@Johnnykahhaleh, added a few comments above.
You don't need to do anything, just FYI.


Looks good to me :). How am I supposed to do now? You'll apply those changes to my PR, or do you want to submit a PR to my branch?


Thanks. Note that with this commit, the health overlay values have decreased. Max is reserved (as it used to be) and min is also reserved for dead creatures (it didn't use to be). So, now, we have 6 steps between max and death ^^


Well, `tComponentView` is being used so that `redo` knows which view component to use to trigger the teardown. It gets reset every time `undo` gets called. If we can look things up by id, then yes, that reference can go away. I'm not sure what the right answer to this is...


I added also on win environment. Rigth click on My computer -> Properties -> Avanced configuration

It's working with Arduino 1.6.5


Mmmm, point. I was just thinking "what if I wanted to run DevPost in production" but maybe you don't want to encourage ~that sort of behaviour~.


We should use an error page when this happens.


Да, видимо, не обновил человек.

Стандартным роботом считаю ваш, №24?

--

Yours sincerely,        mailto:dl@bugtraq.ru <dl@bugtraq.ru>

Dmitry Leonov          http://www.bugtraq.ru

*From:* Walrusteak [mailto:notifications@github.com]
*Sent:* Wednesday, June 17, 2015 12:35 AM
*To:* asugubkin/AS.1204.SysProg.2
*Subject:* Re: [AS.1204.SysProg.2] Revert "Update" (b98bc99)

саботаж!

—
Reply to this email directly or view it on GitHub
<https://github.com/asugubkin/AS.1204.SysProg.2/commit/b98bc994d8c29866157488af872e4abe5f321a2b#commitcomment-11712953>
.[image: Image removed by sender.]


@arthurbarros não sei qual foi teu nível de acompanhamento nas transações do pagseguro mas eu com projeto pessoal tive que colocar vários remendos por conta de envio de status inválidos vindos da api deles e além disto é bem "burra" e confusa o uso dos transaction_code e outros deles.

Eu usei no meu projeto por que na época possuia propaganda na TV, traz confiança mas em nível de API e etc é patético.

Sandbox p Dev: Piada, nunca vi um Sandbox ser *diferente* do Oficial nas chamadas.


Remember that it `has_many :repos, through: :contributions`. Since this is a many-to-many we have to use contributions to connect users to repos. This is true of the Repo model as well. The `has_many :users` there should be a has many through.


And I do suppose it's possible that hooks could be coming from a git server on a box on a trusted local network, possibly?


If I'm understanding this correctly the integrity error here is that the stream already exists in the DB?  If that is the case should this be something like a 409 return code instead of a 500?


Какие планы развития?

Планируешь ли ты улучшать оценку, или это просто для развлечения?

Для пятёрки стоит добавить появление яблок на поле после съедания, также стоит разобраться с автоматическим перемещением змейки. Если это сделаешь, сомневаться в оценке точно будет некому =)


This isn't the correct syntax for string interpolation. You meant: `"token #{ACCESS_TOKEN}"`.


It is possible to get a reply that has no entries - for example a server with only AAAA records.

We should also do an AAAA lookup.


I don't think we care whether it's stdout or stderr. And let's use some nicer formatting. What about this:

```js
proc.on('output', (stdout, stderr) => {
  log.info(`[AVD OUTPUT] ${stdout || stderr}`);
});
```


That probably makes more sense.


r+


We should go ahead and build an array with all A and AAAA records, and return that. The caller can pick one to use.


I will show it tomorrow but the first test was working fine


A more important consideration is that if we perform this check here, then we will have to do the DNS lookup before we do each HTTP query. Such lookups should be performed in advance. (I'll note where, below...)


we'll need to make this camelCase, since the public API of DB is camelCase.

I'm sorry if bind_values or to_string confused you, I still need to change those to camelCase.

Because this is a mysql extension, we'll probably want to make low_priority just return $this here, and then override in MySQLQuery.php to actually set the $low_priority variable. Otherwise, code that runs against postgres or other databases will fail when this method is used. Another option would be to make this method throw a not-implemented Exception by default, and override that in MySQLQuery.php.


fast-forwarding master to auto = bb1fd73f


Пока да. Когда будет турнир проходить? Хотелось бы обновить его


Understood, but, it's generic, and in misc, and even with only one consumer, IMO a different commit is clearer.  Not a big deal.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2152 of 2213 tests passed)

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


PHPCS: Expected 1 newline at end of file; 0 found


PHPCS: Expected 1 newline after opening brace; 2 found


Class TemplateController does not have a Docblock comment.


В принципе, я готов хоть сейчас.

--

Yours sincerely,        mailto:dl@bugtraq.ru <dl@bugtraq.ru>

Dmitry Leonov          http://www.bugtraq.ru

*From:* Walrusteak [mailto:notifications@github.com]
*Sent:* Wednesday, June 17, 2015 12:40 AM
*To:* asugubkin/AS.1204.SysProg.2
*Cc:* Dmitry Leonov
*Subject:* Re: [AS.1204.SysProg.2] Revert "Update" (b98bc99)

Пока да. Когда будет турнир проходить? Хотелось бы обновить его

—
Reply to this email directly or view it on GitHub
<https://github.com/asugubkin/AS.1204.SysProg.2/commit/b98bc994d8c29866157488af872e4abe5f321a2b#commitcomment-11713042>
.[image: Image removed by sender.]


There's a minus sign here and `repo.find_or_create_by` instead of `Repo.find_or_create_by`.
I'm confident you understand the right thing here, it just seems like you might be going too fast.

Slow down and give your brain time to check what your fingers are doing, or better yet, test things out in pry as you go!
We can talk about better ways to try out code as you go. Things look good otherwise.


still having this issue.....why?


I would move all this into a getter for browser.sessionData. (and have the setter cache it in a hidden private _sessionData variable if you want).


Hi, could you help me how to run your app with http://www.sp13pabianice.pl/plan/lista.html url?  I modified url in MasterlistDownloadRunnable.java to "http://gagla.h2g.pl/masterlist" where I put url to my schedule list. when I run the app and try to synchronize and download the html plan from my web the errors appear in this VulcanHandler.java line:
shortName = new String (values[1].toCharArray (), 0, values[1].length () - 1);
errors:
...
java.lang.ArrayIndexOutOfBoundsException: length=1; index=1
at
 at com.github.LiquidPL.kochanowski.parse.handler.VulcanHandler.characters(VulcanHandler.java:192)
...
 at com.github.LiquidPL.kochanowski.parse.TimeTableDownloadRunnable.run(TimeTableDownloadRunnable.java:87)

I would like to use your app just for learning purposes. What else should I modify in code to be able to download schdule from my web?


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


merging MonsieurNicolas/stellar-core/scpUpdates = 3d768119 into auto


MonsieurNicolas/stellar-core/scpUpdates = 3d768119 merged ok, testing candidate = d36249b0


Думаю, не стоит, еще не все внесли последние правки и перебороли гитхаб, лучше провести днем


also i removed the keys adding balls


This looks a bit cleaner to me as the IF-structures are duplicating less code for the HTML output. The idea to "fix" the path in the upgrader seems okay to me, and will prevent strange replacements in the future I believe.

To keep this harmonized and in case of trouble easier to patch, I would vote for porting this approach to 2.0 branch as well?


> Does `find_by...` return a single item?

[Yes](https://github.com/rails/rails/blob/3e36db4406beea32772b1db1e9a16cc1e8aea14c/activerecord/lib/active_record/relation/finder_methods.rb#L81). (Well, that's `find_by`, but this will currently still be complaining that it's not spelled that way.)


It should use the default db connection defined in settings.php shouldn't it?


Sorry, I think the MERGE_HEAD was out of sync because I had PHPStorm and a terminal window open to this directory at the same time.


Did you mean for this to be a `.cljc` file instead of `.clj`?


Where does this limit come from? I don't see it in 6936


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

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


Oops... a bit of both.

I was having an issue, so I added a number of these to places that were not yet so decorated. But I ended up deciding I didn't want to properly consider which ones truly deserved it, and thus meant to drop them again. I'll kill it.


swal({title:'Hello', customClass:'error'});

swal classes = sweet-alert error showSweetAlert visible

swal({title:'Hello', customClass:'success'});

swal classes = sweet-alert error showSweetAlert visible success


That's messed up, lol


We can go ahead and check each DNS server to see if it is an IP address or a host name here, something like this:
```go
var dns_servers []string
for dns_server_name := strings.Split(S_DNS_SERVERS, ",")
    if net.ParseIP(dns_server_name) != nil {
        dns_servers = append(dns_servers, dns_server_name)
    } else {
        ip_addrs = getServerIPs(dns_server_name)
        dns_servers = append(dns_servers, ip_addrs...)
    }
}
```


Do you want to mention changing advertiser 0? Are you going to block it?


_in_ range :tangerine:


Good catch.


С предыдущим тестом разобрался, но после добавления проверки на  пользователя в аттачмент контроллер, стал падать этот тест
```
/home/user/.rvm/rubies/ruby-2.2.2/bin/ruby -e $stdout.sync=true;$stderr.sync=true;load($0=ARGV.shift) /home/user/.rvm/gems/ruby-2.2.2@xvecess/bin/rspec /home/user/xvecess/spec/controllers/attachments_controller_spec.rb --require teamcity/spec/runner/formatter/teamcity/formatter --format Spec::Runner::Formatter::TeamcityFormatter --example AttachmentsController
Testing started at 0:40 ...

NameError: wrong constant name #<Question:0x00000003ea86b8>
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/inflector/methods.rb:261:in `const_get'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/inflector/methods.rb:261:in `block in constantize'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/inflector/methods.rb:259:in `each'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/inflector/methods.rb:259:in `inject'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/inflector/methods.rb:259:in `constantize'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/core_ext/string/inflections.rb:66:in `constantize'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/belongs_to_polymorphic_association.rb:7:in `klass'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/belongs_to_association.rb:45:in `find_target?'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/association.rb:138:in `load_target'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/association.rb:53:in `reload'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/singular_association.rb:9:in `reader'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/associations/builder/association.rb:115:in `attachable'
./app/controllers/attachments_controller.rb:6:in `destroy'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/implicit_render.rb:4:in `send_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/abstract_controller/base.rb:198:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/rendering.rb:10:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/abstract_controller/callbacks.rb:20:in `block in process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:117:in `call'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:117:in `call'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:555:in `block (2 levels) in compile'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:505:in `call'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:505:in `call'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:92:in `_run_callbacks'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:776:in `_run_process_action_callbacks'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/callbacks.rb:81:in `run_callbacks'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/abstract_controller/callbacks.rb:19:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/rescue.rb:29:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/instrumentation.rb:32:in `block in process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/notifications.rb:164:in `block in instrument'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activesupport-4.2.1/lib/active_support/notifications.rb:164:in `instrument'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/instrumentation.rb:30:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/metal/params_wrapper.rb:250:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/activerecord-4.2.1/lib/active_record/railties/controller_runtime.rb:18:in `process_action'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/abstract_controller/base.rb:137:in `process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionview-4.2.1/lib/action_view/rendering.rb:30:in `process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/test_case.rb:632:in `process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/test_case.rb:65:in `process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/devise-3.5.1/lib/devise/test_helpers.rb:19:in `block in process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/devise-3.5.1/lib/devise/test_helpers.rb:72:in `catch'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/devise-3.5.1/lib/devise/test_helpers.rb:72:in `_catch_warden'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/devise-3.5.1/lib/devise/test_helpers.rb:19:in `process'
/home/user/.rvm/gems/ruby-2.2.2@xvecess/gems/actionpack-4.2.1/lib/action_controller/test_case.rb:532:in `delete'
./spec/controllers/attachments_controller_spec.rb:13:in `block (4 levels) in <top (required)>'
./spec/controllers/attachments_controller_spec.rb:13:in `block (3 levels) in <top (required)>'
-e:1:in `load'
-e:1:in `<main>'

1 example, 1 failure, 0 passed

Finished in 0.106708974 seconds

Process finished with exit code 1
```


Для остальных будет вторая попытка (первого же тура), в зачет пойдет
максимальный результат из двух.

--

Yours sincerely,        mailto:dl@bugtraq.ru <dl@bugtraq.ru>

Dmitry Leonov          http://www.bugtraq.ru

*From:* Walrusteak [mailto:notifications@github.com]
*Sent:* Wednesday, June 17, 2015 12:45 AM
*To:* asugubkin/AS.1204.SysProg.2
*Cc:* Dmitry Leonov
*Subject:* Re: [AS.1204.SysProg.2] Revert "Update" (b98bc99)

Думаю, не стоит, еще не все внесли последние правки и перебороли гитхаб,
лучше провести днем

—
Reply to this email directly or view it on GitHub
<https://github.com/asugubkin/AS.1204.SysProg.2/commit/b98bc994d8c29866157488af872e4abe5f321a2b#commitcomment-11713110>
.


@fccoelho I made this small change to your worker, is it ok or were you planning on using the `pypln_id` for some special reason? The `file_id` is an id generated by GridFS when it stores the file and is also unique to each file. If that's enough to be an id in the elastic index, then I think this change should be fine.


There is none in 6936, though I think it prudent to apply one in the interface, otherwise the size of the POST data would become huge (in size and the number of POST parameters), and amount of processing performed during the poking would increase. In practice, we don't get that many categories from zvelo, so is probably a mute point.


6th rank ones are useful but mostly on the kingside (or near the king).  So maybe some adjustment in king safety along with the reduced scope would work.

A rook getting to the 8th rank defended by a pawn is often very good, but I don't really think of that as an outpost...just a promotion tool.


Perhaps a better approach then is a CSV-style load like uploadPlans? e.g. uploadBrandUnsafeConfiguration?


This will be blocked by TG.


Would deserve a note that in case ppl override it, the basic invalidation wouldnt work. Or change that as well.


Not 100% sure about this one. If we're running startup debugging, we may still want syslog-ng to fork into background at some point. That's what I'd expect at least.


I see that it was trashed in 267d6f0f0108a85167b6574dcf97f6c3e171cde1 and fixed in 2349d9cc1229605c3cc5ce9f9141453e266a1fe6. Something similar happened before 0202011bc0cd9312da4a5fbe83feb1df0ffd9b31. Could it be a lines delimiter issue in the editor?


I don't think we should validate transaction codes. Simply validating that they're two digits should be sufficient. The reasoning for this is not all banks use the same transaction codes. For instance, NAB has additional codes such as: `[51, 52, 55]`


Then I think it's ok to return nil - security context has no meaning.

----- Original Message -----
> unsupported as any !linux platform
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pmorie/origin/commit/6a805c85e09a5920c57d615583ac8ddaea4b0f94#commitcomment-11712342


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

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


Story checks out. @tnm would never build anything on Redis.


This second '' parameter is technically a "Title". Does it do anything? (Seems like it would be useful state to have if we decide to show a back-forward list eventually).


Might be good to add a prefix to this result since the name might be generic and ids should be unique across the DOM.


@brandondunne Let's make this change now that we are on Rails 4.


256 seems too low: I'd like whitelisting to be done here when we get things categorized wrong..


Все-таки лучше провести днем, к тому времени наша команда тоже, надеюсь, представит роботов. Будет, как минимум, интереснее


Not sure about defining a collection in a view, this should probably get moved to `models.js`


I don't know if this cast is safe. I know that char, unsigned char and signed char are three types in C.


Nope, I'm removing this commit.
A string is not enough believe me (people filed issues before about this).

Force pushing to master - sorry - you may have to do `git reset HEAD^ --hard && git pull --rebase`


OMG TYLER YOU'RE ALIVE

On Tue, Jun 16, 2015 at 12:14 PM Adil Virani <notifications@github.com>
wrote:

> WHY NOT I'll change when done pls chill
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11710222>
> .
>


I don't think it's necessary to make this configurable, but it is quite likely that we'll change this to a metric unit.  Looks like it shouldn't be a problem, just planting the seed.


"не все внесли последние правки "


как раз внесут под тактику уже залитых


Why isn't this the script you defined in the other commit?


Here's another Rails 4 cleanup.


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


OK, it is safe, or at least widely used throughout the codebase already :).


I guess this is fine but this really needs to be moved out post 3.0.  Create a follow up issue for that.


Why these sinle-line functions are needed while ss_dropdata() uses raw write lock calls?


Давайте проведем уже, сколько времени прошло. Неужели кто-то думал, что опять продлевать будем?


put this in a context manager; otherwise you're messing permanently with user's filter settings
```
with warnings.catch_warnings():
   warnings.simplefilter('always', DeprecationWarning)
   warnings.warn("KDTree has moved to MDAnalysis.lib.KDTree", DeprecationWarning)
```

Use this approach for all the other stubs, too.


Agreed, with 2 upload files, one with the categories and the other with the whitelist domains. I still think a limit of some kind makes sense, though with CSV style, we could say it is 65535 whitelist domains.


Why such a hurry to backport this? We can't make a windows release right now anyway because opensuse broke libstdc++ ABI which broke ZMQ which breaks IJulia.


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

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


test


Я уже обещал проведение тура тем, кто не хотел открывать заранее свой
окончательный код.

А ваша команда все равно уже идет вне конкурса.

--

Yours sincerely,        mailto:dl@bugtraq.ru <dl@bugtraq.ru>

Dmitry Leonov          http://www.bugtraq.ru

*From:* Walrusteak [mailto:notifications@github.com]
*Sent:* Wednesday, June 17, 2015 12:56 AM
*To:* asugubkin/AS.1204.SysProg.2
*Cc:* Dmitry Leonov
*Subject:* Re: [AS.1204.SysProg.2] Revert "Update" (b98bc99)

Все-таки лучше провести днем, к тому времени наша команда тоже, надеюсь,
представит роботов. Будет, как минимум, интереснее

—
Reply to this email directly or view it on GitHub
<https://github.com/asugubkin/AS.1204.SysProg.2/commit/b98bc994d8c29866157488af872e4abe5f321a2b#commitcomment-11713278>
.[image: Image removed by sender.]


hm, out hashcode is not consitent with equals - i guess we could be fine with just computing name in hash code and using it to compare, instead of listing all fields. i might be wrong, but anyways we should find minimal set of properties that uniquely identifies Topic, not use all of them


Agreed, that would make a good prefix as per my comment above


My problem with this separation into two versions of this class is that it's lead to a lot of mostly duplicated but subtly different code. It would be nice if this were factored in such a way that opening up one of the two specializations showed me the things that are different rather than a lot of things that are largely the same. Not that this necessarily has to change here.


If possible yes, but I don't know if it will apply to 3.x versions. There
is a rewritten version of the patch for 3.4.

On Tue, Jun 16, 2015, 14:16 Domen Kožar <notifications@github.com> wrote:

> Should that also apply to other python versions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/4c8f84e8afe4b782909981e23f245b7c310ff2a5#commitcomment-11712613>
> .
>


Cool, latest patch is `0.4.3` pulling into b-shim and publishing


This seems like an odd parameter to call "truncate" given what truncate means in the context of the PBD. It's not the wrong word, it's just slightly confusing in context.


Please use `&&`


does it always point to schema error? if so - why not reuse schema mismatch exception defined in Json schema checks?


i guess some inline comment would be helpful here


Also add a logger message
```python
import logging
logger = logging.getLogger("MDAnalysis.lib.KDTree")
logger.warn("The MDAnalysis.KDTree module is now at MDAnalysis.lib.KDTree; the old module location "
                   "is deprecated and will be removed in release 1.0")
```


I removed it and have it in another directory (deployer image) but haven't added it back yet.  I am going to follow your guidance above by including it in that image.    Alternatively, I can follow the pattern of deployer and create a custom image for us.  The former might be more expedient given the time constraints but a custom image would fit the pattern in place.


The last release, 0.7.1, was before this, so if you're installing from PyPI that's the reason.. I'll try to get a new releas up soon!


тем не менее, пара-тройка лишних роботов добавила бы интереса
кстати, я изменил стандартные значения Rmax и K - с меньшим Rmax атаковать можно было только при высоких значениях V, а меньшее К не давало достаточное кол-во очков тем, кто не только сидел в зарядке


The association defined by `belongs_to` should always be singular (e.g. `belongs_to :todo_list`).

Also, it's not clear to me why their are multiple classes with the same name in different files. That's generally not a good way to experiment. If you want to talk more about how to test code and try things out, let's find some time to discuss it later this week.


For the record, while the testrunner succeeds now, there are some gcc compile-warnings now

```
lib/checkio.cpp: In constructor ‘CheckIO::ArgumentInfo::ArgumentInfo(const Token*, const Settings*, bool)’:
lib/checkio.cpp:1378:92: warning: declaration of ‘isCPP’ shadows a member of ‘CheckIO::ArgumentInfo’ [-Wshadow]
 CheckIO::ArgumentInfo::ArgumentInfo(const Token * tok, const Settings *settings, bool isCPP)
                                                                                ^
In file included from lib/checkio.cpp:20:0:
lib/checkio.h:89:8: note: shadowed declaration is here
   bool isCPP;
        ^
lib/checkio.h:90:16: warning: ‘CheckIO::ArgumentInfo::tempToken’ will be initialized after [-Wreorder]
         Token *tempToken;
                ^
lib/checkio.h:89:8: warning:   ‘bool CheckIO::ArgumentInfo::isCPP’ [-Wreorder]
   bool isCPP;
        ^
lib/checkio.cpp:1378:1: warning:   when initialized here [-Wreorder]
 CheckIO::ArgumentInfo::ArgumentInfo(const Token * tok, const Settings *settings, bool isCPP)
 ^

```

clang 3.6.1 build fails entirely (travis didn't catch this at all, for some reason, maybe its clang is too old?)

```
[...]
clang++ -Ilib -Iexternals/tinyxml   -include lib/cxx11emu.h -pedantic -Wall -Wextra -Wabi -Wcast-qual -Wfloat-equal -Winline -Wmissing-declarations -Wmissing-format-attribute -Wno-long-long -Woverloaded-virtual -Wpacked -Wredundant-decls -Wshadow -Wsign-promo -Wno-missing-field-initializers -Wno-missing-braces -Wno-sign-compare -D_GLIBCXX_DEBUG -g  -std=c++0x -c -o lib/checkboost.o lib/checkboost.cpp
clang++ -Ilib -Iexternals/tinyxml   -include lib/cxx11emu.h -pedantic -Wall -Wextra -Wabi -Wcast-qual -Wfloat-equal -Winline -Wmissing-declarations -Wmissing-format-attribute -Wno-long-long -Woverloaded-virtual -Wpacked -Wredundant-decls -Wshadow -Wsign-promo -Wno-missing-field-initializers -Wno-missing-braces -Wno-sign-compare -D_GLIBCXX_DEBUG -g  -std=c++0x -c -o lib/checkbufferoverrun.o lib/checkbufferoverrun.cpp
lib/checkbufferoverrun.cpp:707:48: error: default initialization of an object of const type 'const std::list<const Token *>' without a user-provided default constructor
                const std::list<const Token *> callstack;
                                               ^
lib/checkbufferoverrun.cpp:707:57: note: add an explicit initializer to initialize 'callstack'
                const std::list<const Token *> callstack;
                                                        ^
                                                        {}
1 error generated.
Makefile:305: recipe for target 'lib/checkbufferoverrun.o' failed
````


remove code that is not tested ;-)


This is actually a pretty good way to approach it given that I didn't give a lot of advice on how things were supposed to be structured in the app class.

Most importantly, your queries `find_or_create_by` etc and associations are coherent and good. :+1:


These are so similar they could be squished to a single helper function data


add a
```reST
.. versionchanged:: 0.11.0
   The MDAnalysis.KDTree module was moved to MDAnalysis.lib.KDTree
```


this declaration gives us nothing, remember that HahsMap will grow when load factor is exceeded, meaning after we add second element it will be rehashed. why not use some Guava stuff for immutable map of two entities in constructor?


64k sounds like a reasonable start. Maybe you also want a way to query something about the current lists (so it's convenient for import-zo to find out the last-modified timestamp or a token?)


Want to do another pull request?


I highly doubt this.


oh, now i see it's map of Enums - use `EnumMap` for this


You want a `belongs_to` here, not a `has_one` because the foreign key `user_id` should be stored in the `todo_lists` table, not the `users` table.


Went out with b-shim v3.8.9.
Thanks!


Here `position` is a string with the name of the parameter that will be assigned to be the `_position` parameter. So the `getattr` part is really required. The `setattr` isn't, you are right. And it reduces the code readability, so.... away it goes


do we need a copy here? maybe we should sacrifice it for the sake of performance


robotbase.04.dll
 robotbase.06.dll
 robotbase.07.dll
+robotbase.08.dll
 robotbase.09.dll
+robotbase.11.dll
 robotbase.17.dll
 robotbase.19.dll
 robotbase.20.dll
 robotbase.22.dll
-robotbase.24.dll
+robotbase.24.dll 5


моего робота нет


If you wind up not using this class, rip this file out in a later commit!


so am i


I would use current_user.name and then switch to just username & email if no name.


Shouldn't this be associated with the `-d` flag?


at last we're getting rid of comatibility mode :D


Там вообще-то целых 5 24-х будет, согласно условиям.

Некоторые значения будут меняться в процессе.

--

Yours sincerely,        mailto:dl@bugtraq.ru <dl@bugtraq.ru>

Dmitry Leonov          http://www.bugtraq.ru

*From:* Walrusteak [mailto:notifications@github.com]
*Sent:* Wednesday, June 17, 2015 1:06 AM
*To:* asugubkin/AS.1204.SysProg.2
*Cc:* Dmitry Leonov
*Subject:* Re: [AS.1204.SysProg.2] Revert "Update" (b98bc99)

тем не менее, пара-тройка лишних роботов добавила бы интереса
кстати, я изменил стандартные значения Rmax и K - с меньшим Rmax атаковать
можно было только при высоких значениях V, а меньшее К не давало
достаточное кол-во очков тем, кто не только сидел в зарядке

—
Reply to this email directly or view it on GitHub
<https://github.com/asugubkin/AS.1204.SysProg.2/commit/b98bc994d8c29866157488af872e4abe5f321a2b#commitcomment-11713441>
.


Will do, maybe later this evening, or first thing tomorrow.


make result final


У меня не воспроизводится


It's fine to reuse the existing image - it's going to be on every node anyway.  It's fine to put your script in the openshift/origin image (via a direct add)

> On Jun 16, 2015, at 6:05 PM, Mark Turansky <notifications@github.com> wrote:
>
> I removed it and have it in another directory (deployer image) but haven't added it back yet. I am going to follow your guidance above by including it in that image. Alternatively, I can follow the pattern of deployer and create a custom image for us.  The former might be more expedient given the time constraints but a custom image would fit the pattern in place.
>
> —
> Reply to this email directly or view it on GitHub.
>


Не использую подобную конструкцию импорта. Используй сокращения: `import tkinter as tk`


Is this printStackTrace intended? In any case I would log it with the logger.


given when then


do we need jackson annotations here?


Final take: looks fine, but the code duplication really does hurt. The difference between the two versions of this method lie principally in how we calculate offsets, with the underlying logic being identical.


The trouble is the mess that tends to be created if later cookbooks assume something is mounted somewhere when it isn't. It's difficult to clean up once the data is written.


As usual, I can't sneak anything past you :) A customer needs it. Let me know if it causes any problems.


I would frankly rather have logic that doesn't use the storage cookbook instead of having the storage cookbook not set things up when it doesn't know how to.


since we added catchExceptioon to this test class, how about changing this one as well?


Please revert this commit. A string is not enough!
On the other hand I just noticed the npm deprecation.
Sorry, wasn't aware.

How are you supposed to point to a custom license then?
Maybe this is more correct?

```
{ "license" : "LicenseRef-LICENSE" }
```

My license is somewhat custom (mainly MIT), but mentions my name and reserves all rights first before granting some. See [here](https://github.com/thlorenz/proxyquire/blob/master/LICENSE#L1-L2)
I'm not an expert, but I don't wanna change license implications without proper thought.


I am wondering if iRODS has some file locking mechanism. Otherwise, how it deals with multiple microservices trying to act on the same file at the same time.


given when then sections (and also in test below - boy scout rule)


Done.


I would move this one also into `createExecutorInfo()`.


Hi Evan,

Please could you explain where the getPoints() method from line 141 is defined?
Attempting to build this project results in errors from the absence of this method.


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


Why hide and not remove?


Pretty sure we have to leave `edd_error` here so that we don't break styling of existing themes.


Only if don't agree with what I proposed here https://github.com/mpc-hc/mpc-hc/commit/b61ee85909b826082d2182303e4f6d19da0ec03f#commitcomment-11671425 and won't hold me responsible for any possible fireworks. Peace.


Same for `edd_errors` above


PHPCS: Expected 1 newline at end of file; 0 found


PHPCS: Whitespace found at end of line


Class TemplateController does not have a Docblock comment.


Class Template does not have a Docblock comment.


maybe we shoudl wrap this up in some helper class? now it's not too reusable and creates noise in test (all private methods here)


Can't remove it here. We have to keep all existing classes for people that have styled off of these.


Can you extract this try catch in some method to make the code more readable. Something like:

```
private void safeOnShutdown(final boolean logRatherThanThrow)  {
    try {
      ...
      onShutdown();
      ....
    } catch (final RuntimeException e ) {
      ...
      if (!logRatherThanThrow)
           throw e;
      ...
   }

}
```


I'll test the `fuctools.wraps` tomorrow (apparently you cannot run / test hyperspy without X server running... or at least I can't) to see if that does what's needed.

The `thing` was just a leftover from trying not to mix namespaces... And once @francisco-dlp showed the way (i.e. the `get_f_wrapped` function outside the scope of the class itself), I just left it there.

Thanks anyway!


make sure you're using Shattered's verision of PD-classes:

https://github.com/00-Evan/PD-classes


@pippinsplugins we CANNOT keep those....100%. if we want to standardize..we have to remove.


I think that we should consider here what should be done with Errors (extending java.lang.Error) ... In a different pull-request I was asked to make my code permeable to them by @akiezun.

When doWork causes an Error, should even attempt to shutdown?
If doWork threw a RuntimeException well but shutdown throws an Error should it be log the first and re throw the second?


fast-forwarding master to auto = d36249b0


Nicely spotted, thanks!


We can't keep them, we cannot move forward if we keep these...and it'll break our standards.


Moving conversation to issue


Two contexts in one context ;)?


@akiezun please comment.


Good job. This sort of optimisation has wide-ranging benefits.


This needs a try{} to catch the exception thrown by the last action and turn it into a Failure().


This is a public repo after all. Hopefully it'll be fine, but it's unclear what bug this is fixing on release-0.3 - everything referred to from #11644 looks like post-tuple changes.


For things like that I usually rely on auto-cleanup, such as a0933e8


On 16/06/2015 22:04, John MacFarlane wrote:
> I've pushed a change to master replacing the gperf hash
> lookup with a binary search. If you have a chance, could
> you have a look at what I've done and let me know if it
> looks sane? It shaves off > 100K from the binary, and
> there's not a noticeable performance hit.

This is not quite what I had in mind but it's an interesting approach.

I can see to failure cases:

* A random invalid entity string can hash to the same value as a
   valid one. For example, `&foobarbaz;` could hash to the same
   value as `>`.
* There might be hash collisions between valid entity strings.
   I didn't check, but it's entirely possible that there are.

What I had in mind is a simple struct like

     struct cmark_entity_node {
         char *entity_string;
         unsigned char replacement_bytes[8];
     };

Then do a binary search on a sorted array of these nodes in C with
`entity_string` as key. This involves a lot of string comparisons which makes
things slower than comparing hash values. But it reduces the size of the
binary even further.

If you think this is too slow, another approach is to precompute a
traditional, non-perfect hash table. Preferably using open addressing to
reduce the overall table size.

Nick


this change is breaking for the following urls : `mailto:`, `tel:`


@leomendezg accessing the cursor after get close is going to explode. Fix that!


TBH, I would kill off this SavedTab class and just move these encoders/decoders into Browser.swift itself. Maybe there's some reason for this though....


Why only identifier? `Event` should be `Equatable` so two objects can be properly compared


You really don't need to specify types here. The compiler can infer that request is of type `MockHttpServletRequest` because it's a val and that's what you're sticking into it.


The syntax here seems pretty gross.

I'd prefer:

```js
exports["appendChild: no siblings"] = {
  setup() {
    ..
  },
  benchmark() {
    ...
  }
});
```

It seems like all the other boilerplate should be eliminate-able.


I don't think result.toString is what we want here. The return type should be a list of attribute values (inc refs, although it might include entities in the future).


It's actually fixing a bug if you do the following
1) Enter an invalid discount code, hit apply, error shows
2) Go down and empty the 'First Name' field, and Process checkout, it'll remove the errors wrapper in teh discount field so you cannot generate another discount error. Hiding prevents that.


Go ahead and squash.


OK. Nothing to complain about.
Well done!
Those were your thoughts to express on the paper and since they are related, I can't say that we cut them out.


Needs review/adding to by @hworld


+1

BTW I was looking at the sources of _AutoDetectParser_ to verify it to be threadsafe, it looks like the Parser instance is cheap but the expensive point is its initial configuration.
Turns out the default configuration you're implicitly using here does perform some suspicious resource loading from withing the TCCL.. we might want to use a _Service_ for this so that we can get both:
 - a single instance per SearchFactory
 - the Tika configuration initialized with the right classloaders


Can we use `Exception as e` in all these cases please?


I am going to fix it! :)


{{__future__ absolute_import}} here?
```python
from .ds9 import *
```


do we need to prettyprint this?  seems like it just results in larger messages that will be processed by some server (during 'working' conditions) anyway


I think the coding standard prefers one item imported per line.


You should probably make some thorough notes of parts like this... maybe it was useless, but maybe it's an awesome feature which was just waiting to be finished/polished.  I can't really tell what it does looking at it for a second, but I wouldn't go around deleting commented code just like this.

People in the older generation didn't use/know VCS and just commented potential bits in and out as they seen fit.


This should not have the `.keys()` in it.
```python
for key in maskPlanes:
```
works just fine.


I don't think there's a problem. I think I only wanted an integer
Em 16/06/2015 18:50, "Flávio Amieiro" <notifications@github.com> escreveu:

> @fccoelho <https://github.com/fccoelho> I made this small change to your
> worker, is it ok or were you planning on using the pypln_id for some
> special reason? The file_id is an id generated by GridFS when it stores
> the file and is also unique to each file. If that's enough to be an id in
> the elastic index, then I think this change should be fine.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/flavioamieiro/pypln.backend/commit/2aaa10febc0c492e95c2f6a86d7e249f5694eb4d#commitcomment-11713187>
> .
>


hah.  got sidetracked midway through smashing `a` and `a:hover` together.


This is a neat patch, @salsh. You're already aware of the coding style issues, so I'll mention each syntactic issue only once inline and concentrate on the interesting bits: the semantic aspects.

First, some high-level feedback:

- Respect, you did all the hard work yourself: explicit loops, if-then-else, etc. I wonder if it would be easier to express the grammar in a more declarative fashion. After all, many of the high-order parsers can be easily chained with the existing DSL. At the same time, it's a bit of a pain to parse into structs, so I do understand your choice here.
- You often use literal constants to denote the end of an iterator, e.g., `f + 4`. It is not guaranteed that `f + 4 <= l`, so the parser may run into trouble (e.g., segfault) for ill-defined input. I have to admit, error handling is currently not very well integrated, you'd have to check the return value of each call to `parse()`, which clutters the parsing logic. Maybe in a later step, once you've established that the code works for well-defined input, it would be a good idea to make it safer.
- It seems you're slurping in the entire file at once (via the new `byte_based_source`). You can do that if you know that the data you get is always a single file and *it fits in memory*. In high-performance scenarios, this is rarely the case though, so a more natural design would be to identify records in your binary protocol which are a natural sequencing point. For example, many ASCII files are line-based where `\n` is a natural sequencing point. At the point, you could run the parser on the current data chunk.
- Finally, does it work? How do you test it? It's a significant piece of code that you're adding, so some unit test along with it would really help. Also, if someone wants to verify your actual implementation, it would be a good idea to add a comment with a link to the grammar that you've used as blueprint.


Конечно планирую, это же интересно!
Но, скорее всего, в конце недели.


Yes I am
On Jun 16, 2015 4:58 PM, "Ian Macalinao" <notifications@github.com> wrote:

> OMG TYLER YOU'RE ALIVE
>
> On Tue, Jun 16, 2015 at 12:14 PM Adil Virani <notifications@github.com>
> wrote:
>
> > WHY NOT I'll change when done pls chill
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11710222
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11713306>
> .
>


Хорошо.
А в чём её плоховизна?


Adding an `_` between `bgpdump` and `binary` would help readability, i.e., going with `bgpdump_binary.cc` (and also the corresponding class names).


Yep, that's exactly how to integrate the source. :+1:


Please use the modern form of `raise`:
```python
raise Ds9Error("some string"...)
```


Thanks! Fixed in faa92ba6ca6f98af80e947461585569e3d796be8


This is confusing. I thought that there's only one sut and with that assumption `sut.count` doesn't make much sense. Instead of for loop you could use functional stuff like `flatMap`, `filter` etc. Anyway, I think you don't need to create `it` for each populated object. If you use filter properly, you will get one `Bool` that will indicate if the behavior meets our expectations.


When do we call SS_CALL() with sk->sk_user_data as NULL?
SS_CALL is a Tempesta call, which can be done only on Tempesta owned sockets. It seems it should be BUG_ON() statement if we do SS_CALL on socket with zero sk->sk_user_data.


В том, что не понятно, какие имена в итоге находятся теперь в области видимости


I'll try tonight as soon as I get home - it's a bit hard to judge from here :^)


Ок, тогда я посмотрю на следующую итерацию, в этой прокомментирую по минимуму


develop (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


This affects a lot more than nukes, if that wasn't your intention...


@bparees when I was testing scaling in the Django example a few weeks ago, each request was being served by a different pod, round-robin. Even to render a single page, multiple resources were fetched from different pods. Did this change recently?


In general, [constructors should only do minimal work](https://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Doing_Work_in_Constructors) an get an instance into a valid state. Everything non-trivial and expensive should take place in other member functions. In this case, you're even doing I/O! A better way would be to do this in the first call of `extract`.


These should not be public so `gceNetwork` and `gceInstance`.


Maybe I am not understanding sth properly but I think that 'if' is needed. Look at this: https://github.com/openworm/org.geppetto.simulation/blob/datamanager/src/main/java/org/geppetto/simulation/RuntimeExperiment.java#L282 (anyway I realise it is not a big deal :)


BTW @brmeyer great job in finding the performance problem, but fixing the TCCL issue is not a requirement to fix this.

I've opened https://hibernate.atlassian.net/browse/HSEARCH-1899 as a follow-up issue.


Coding style: all members have a `_` suffix to make it easier to distinguish them from local variables. Especially in long functions like your parser, this is very helpful.


Xcode messed up indentation here :/


I have a few comments.
Functional testing is good but does not cover everything. Having unit testing with might be very useful to validate different contexts and also the syntax. Rspec tests are usually not an option when evaluating the quality of a module.
I think we should try to keep these tests and try to fix them, and finally use rspec to gate the modules beside functional testing.


I think @bparees means APPLICATION_DOMAIN :wink:


```python
import os

entries = (
    os.path.join(dirname, filename)
        for dirname, _, filenames in os.walk("/")
            for filename in filenames
)

print "\n".join(sorted(entries))
```


It's fine to leave a function with an empty body. Even better would be to add a TODO comment so that it's easier to spot for someone that you've done this intentionally.


You could just pass `self.constructChainedRequestWithQuery` instead of wrapping it in closure :)


DBManager.getInstance().deleteObject(Constants.Prefs.PREF_PARAM_SHOP_OFFERS, ShopOfferListLoader.class);

к базе не нужно обращаться в основном потоке - это опасно.
у меня так было сделано исключительно в тестовых целях.


That's a good point. We might be able to save a bit of space by switching to a compact writer. Let me try that out.


Oh cool, I too have little experience with emulation, I started with a Chip-8 because it was simple. Now I'm trying the NES and wow it sure is more complicated, lol. I'm dreading working on the PPU, it'll be a challenge to tackle.


Heh true :] Added initial global settings object earlier so I can do global icon size changes very soon (Working on some kind of pref mechanism that doesn't suck, and works fine with limited buttons.. ^^)


Coding style: one space between keywords, `if(...)` -> `if (..)`.


Can you give just a bit more detail. Say that it's the GCE backend running or something.


Ideally, `extract()` can be called incrementally to parse out one entry at a time. See also the comment on the sequencing points at the bottom.


Coding style: local variables are lower-case and underscore-separated.


Can you pass `GCEBackend` by pointer here (and all methods here) since the struct is pretty large.


Any magic constants should have a comment for what they mean, even it's just a few characters or words.


Whoops, typo. "Added boost project"


Değişiklikleri aldım abi :) Drawline fonksiyonu güzel çalışıyor :+1: ClickIsInRange fonksiyonunu implemente etmeye çalışırım. Bir de elindeki versiyonun her zaman localde kopyası olsun. En azından ikimiz de githubı iyice öğrenene kadar :)


Falcon is responding with a 500 instead of a 409. i totally agree that it should be a 409 but I am not sure why it does not return a 409. Any thoughts?


It's possible that after updating to beta 2 i still have 42fb669 in system info ?

See:
http://s3.postimg.org/72ihwglwz/20150617_005031_1.jpg


Let's store the _afterData at the start of undo()


oh so maybe you can help me learn more about Chip-8, I looked for a Chip-8 code to study, but did not find one that works as expected. Can you sir link me one?


Like them tests!


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

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


How does the function enable sotirq? I the comment line actually valid?..


Coding style: no space between template parameters, `static_cast< int >` -> `static_cast<int>`.

In this case, the `static_cast` is actually unnecessary: `x.nag` is of type `uint16_t` and the RHS is an integer literal. The compiler promotes the expression to type `unsigned int`.


Ok, you fixed it later! :)


I got a exception (ServiceManager:96) if I stop the client where the ServiceManager was not started. I removed the ServerSerializerRegistrationsService from the server because I didn't want this feature due to problems with obfuscated serializable classes.


@robdockins might be relevant to test0018:  http://worldbuilding.stackexchange.com/questions/10054


Sorry Mattie but you will need to remove the `-$RVER` in there as well as that will cause the strings to be different. We don't care a whole lot about the resin version anyways.


I don't understand these "#undef foo" and "#define foo(i) foo((int) i)" stuff.  Why is it necessary?  In fact, functions declared in ctypes.h may be macros and you should not undef it if that's the case:

http://pubs.opengroup.org/onlinepubs/009695399/basedefs/ctype.h.html

`The following shall be declared as functions and may also be defined as macros.`


Does this need to be quoted?


Comments don't match, but that's okay for now. :stuck_out_tongue_winking_eye:


Paul Chiusano asked the following question in an interesting discussion about Elm's strengths and limitations vs Purescript (https://groups.google.com/forum/#!topic/elm-discuss/oyrODCgYmQI).
Maybe this question reflects the best the motivations behind our discussion so I insert it here:

"This is sort of a philosophical question, but is the primary goal a) to develop the best technology possible, or is the primary goal b) to make [a particular DSL] popular and widely adopted? These things aren't always in conflict, but if they are perceived to be, I'd kind of like to know which goal wins out."


We had to do that because the original macros (such as ACPI_STRLEN) we casting things such that we didn’t want to go through the entire code base just to re-cast things.

From: Jung-uk Kim [mailto:notifications@github.com]
Sent: Tuesday, June 16, 2015 4:01 PM
To: acpica/acpica
Cc: Moore, Robert
Subject: Re: [acpica] De-macroize calls to standard C library functions. (3b1026e)

I don't understand these "#undef foo" and "#define foo(i) foo((int) i)" stuff. Why is it necessary? In fact, functions declared in ctypes.h may be macros and you should not undef it if that's the case:

http://pubs.opengroup.org/onlinepubs/009695399/basedefs/ctype.h.html

The following shall be declared as functions and may also be defined as macros.

—
Reply to this email directly or view it on GitHub<https://github.com/acpica/acpica/commit/3b1026e0bdd3c32eb6d5d313f3ba0b1fee7597b4#commitcomment-11714223>.


and I failed at typing by begin I mean beginning. lol


definitely not loving this... as you might expect.  There was no way to have this node specify all its own prefixes and not worry about those it inherits?  Also, have we considered upgrading xstream?  do we know the bug fixes between what we are using and the latest (1.4.8 it seems?)  At least this is a living project, even if the documentation is hard to find.  It seems like universal thread-safe prefix management is something we should be able to achieve.  Seems weird to worry about efficiency and then end up doing operations like this


This solved my issue! I found getPoints() in an updated version of watabou.rect.

The readme link to PD master classes led me astray. Thanks for taking the time to answer, Evan.

<3 <3 <3


@S0N-0F-A-S0N Omg, we are not in the same timezone. Please just send me messages here, I will reply to you ASAP. XD Sorry for not being on skype.


Is `toJSON()` still in `ModelObject` and `ModelJSONProtocol`?


The function is never called w/o ss_do_close(), so it's better to introduce a function like ss_close_failover() which does the same and calls ss_do_close().


Аналогично предыдущему.
DBManager.getInstance().deleteObject(Constants.Prefs.PREF_PARAM_SHOPPING_LIST, ShoppingListLoader.class);
Неправильно работать с базой в основном потоке.


VERSION is still 42fb669 in http://sources.openelec.tv/devel/kodi-15.0-beta2-daedd5f.tar.xz - looks like the Kodi package wasn't updated after this fix went in, and before beta2 (5.95.2) was built.


BTW: https://docs.python.org/3.5/whatsnew/3.5.html#pep-471-os-scandir-function-a-better-and-faster-directory-iterator

Really looking forward to 3.5 now (async, await...!)


This is still broken, should link to https://github.com/Carthage/Carthage/issues/340 not the README?


I don't think it will really matter but wouldn't this change the active zone from center back to an empty string because it is not in an else statement? Shouldn't affect the final outcome at this point in time but it might mean that it is not worth returning center at all?


Why do you return a copy of the entire file? `bvector` is not a local variable, so neither RVO won't kick nor will the compiler have a chance to return a xvalue.

Maybe you wanted to just fill `bvector`? By the way, a more name would be `byte_buffer` or something along the lines. (It's fine to use short names as long as their meaning can be inferred easily.)


Ouch :laughing:


This might give some inspiration for a format: https://github.com/elixir-lang/elixir/blob/02f9d8a724d6438c1e7a3efc49a0938d5204d6c0/lib/logger/lib/logger/translator.ex#L85-L201


Not sure what you have in mind for uses of resourcePath_, but remember that the resource path is only a concept in release mode (only relative paths are resolved against the resource directory). That is why we have an option for each path (so that the development-time path can be an arbitrary other absolute path somewhere in the tree).


Consider adding download.com
http://insecure.org/news/download-com-fiasco.html


Why did you change back to old-style formatting?

```python
if i % 10 == 0:
    v = '{date}-{version}'.format(date=time.strftime(...), version=i)
    client.send_event(...,
        properties={'version': v})
```


Instead of creating a local variable `r`, a better idea would be to change the signature to
```cpp
template <typename Iterator>
bool body(Iterator& f, Iterator const& l, format& f) const;
```
In fact, it's necessary for the function signature to not change and always return a `bool`, otherwise it wouldn't work with the rest of the Parseable concept.


my least favorite exec is here eh?

On Tuesday, June 16, 2015, Acidity <notifications@github.com> wrote:

> Yes I am
> On Jun 16, 2015 4:58 PM, "Ian Macalinao" <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
> > OMG TYLER YOU'RE ALIVE
> >
> > On Tue, Jun 16, 2015 at 12:14 PM Adil Virani <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> > > WHY NOT I'll change when done pls chill
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11710222
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11713306
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11713924>
> .
>


I was afraid of that.  Is the casting really necessary?  Actually, "#define isdigit(i) isdigit((int) i)" kind is really redundant as far as I know.  The only reason I see may be ACPI_SIZE <-> size_t stuff but it is also not necessary as far as I can see.


What's this uniqueness constraint for?


Consider adding download.com
http://insecure.org/news/download-com-fiasco.html
http://www.howtogeek.com/198622/heres-what-happens-when-you-install-the-top-10-download.com-apps/
https://www.eff.org/deeplinks/2011/12/downloadcom-debacle-what-cnet-needs-do-make-it-right
http://boingboing.net/2011/12/10/cnets-download-com-secretly.html


You can save yourself some typing by hoisting the `parsers` namespace:
```cpp
using namespace parsers;
```
This is fine inside a function and in implementation files.


Why not include the space in the other string and save a string concatenation?

```python
session.headers[...] += \
    ' {name}/{version}'.format(...)
```


It’s not to say that this might take a couple of passes, but we wanted to minimize the disturbance for now. The actual definitions have not changed from the macros that were in use.

From: Jung-uk Kim [mailto:notifications@github.com]
Sent: Tuesday, June 16, 2015 4:07 PM
To: acpica/acpica
Cc: Moore, Robert
Subject: Re: [acpica] De-macroize calls to standard C library functions. (3b1026e)

I was afraid of that. Is the casting really necessary? Actually, "#define isdigit(i) isdigit((int) i)" kind is really redundant as far as I know. The only reason I see may be ACPI_SIZE <-> size_t stuff but it is also not necessary as far as I can see.

—
Reply to this email directly or view it on GitHub<https://github.com/acpica/acpica/commit/3b1026e0bdd3c32eb6d5d313f3ba0b1fee7597b4#commitcomment-11714331>.


Coding style: 2 spaces indentation.


Preferred formatting would be:

```python
    def _format_send_data(self, cumulative_counters=None,
                          gauges=None, counters=None):
```


What do you think of this idea?

The `WHENCE` file is the database of the firmwares included in the kernel.org git firmware repository. What if we cross referenced each firmware with what drivers are included in the `DOTconfig`?

Obviously, this doesn't contain ALL firmwares, but the ones that are there and the drivers that are included will be covered. This can be supplemented by  `kernel-skeleton/lib/modules/all-firmware` , which, if we use this method, should also be cross referenced with *something* (maybe `DISTRO_PKS_SPECS-*-*`?) to see if that driver is included in the build and delete redundant entries. Of course there will be exceptions to this rule too.


failed to round percent.


This wp_cache_get() isn't doing anything here?
Move to just before filter.


Fix, base class doesn't care about serialization


The better way would be to set the template render mode by default for this image.


Cool, you registered the `format` struct in the registry! Now it's possible to use it easily to find the parser via `make_parser<bgpbinary_parser::format>`.


With the full response of ~2200 metacards, there's only about a 1MB difference in file size (from 65725090 down to 64855327). To get more out of it, I think I'd have to go in and explicitly whack whitespace. I just don't think it's worth the change unless y'all want it.


I understand it not correctly, sorry...


Is it possible to avoid using globals?


Why do we need the GH version?


he does. :)


Yeah, these shared variables are bizarre. Store them somewhere instead, e.g. `this`.


Testing:

```
$ PANTS_DEV=1 ./pants clean-all compile ./examples/src/antlr/org/pantsbuild/example/exp::
$ head /Users/qma/workspace/pants/.pants.d/gen/antlr/antlr3/gen-java/examples/src/antlr/org/pantsbuild/example/exp/ExpAntlr3Lexer.java
// $ANTLR 3.4 examples/src/antlr/org/pantsbuild/example/exp/ExpAntlr3.g 2015-06-16 15:45:46

  package org.pantsbuild.example.exp;

import org.antlr.runtime.*;
import java.util.Stack;
import java.util.List;
import java.util.ArrayList;

$ PANTS_DEV=1 ./pants clean-all compile ./examples/src/antlr/org/pantsbuild/example/exp::
$ head /Users/qma/workspace/pants/.pants.d/gen/antlr/antlr3/gen-java/examples/src/antlr/org/pantsbuild/example/exp/ExpAntlr3Lexer.java

  package org.pantsbuild.example.exp;

import org.antlr.runtime.*;
import java.util.Stack;
import java.util.List;
import java.util.ArrayList;
```


path.resolve instead of string concatenation, please


Actually it'll be v0.6.0. The v0.5.0 release that we cut was based on v0.14.x


s/^/export/


Yes, I understand your intention but it breaks FreeBSD, for example.

If some platform has ctypes.h, which contains this:
```
#define isdigit(c)      ((c) >= '0' && (c) <= '9')
```
then acenv.h cannot do:

```
#undef isdigit
#define isdigit(i)     isdigit((int) i)
```
because it may not have a symbol `isdigit`.


:sparkles:


Hey @nicksnyder, copyright notices should be from the year the package was first published: https://en.wikipedia.org/wiki/Copyright#Copyright_notices_in_the_United_States

>  consisting of the copyright symbol (©, the letter C inside a circle), the abbreviation "Copr.", or the word "Copyright", followed by the year of the first publication of the work and the name of the copyright holder


I see that given https://github.com/Joris-van-der-Wel/jsdom/commit/8ff647939282512c5d700f2be22832b5b7d99f7a maybe it is not all eliminatable. Still, maybe

```js
exports["appendChild: no siblings"] = documentBenchmark({
  setup() {
    ..
  },
  benchmark() {
    ...
  }
});
```


@thejameskyle
The build for this looks a little funny now:
original:
```js
  var CONTAINS_SUPER = /xyz/.test(new Function("xyz")) ? /\b_super\b/ : /.*/;
```
vs boilerplate:
```js
  var CONTAINS_SUPER = /xyz/.test(function () {
    xyz;
  }) ? /\b_super\b/ : /.*/; // eslint-disable-line
```


現在メッセージ作成などでイベントログには記載しないようにしていますので、
削除についても行わないようなかたちでOKです。


Once this happens, it will still pass back the reformatted code. Maybe it should pass back `None, encoding` instead? Also, since we're adding this flag maybe you should modify `__init__.py` to use it too.


You want this?

#ifdef isdigit

#undef isdigit

#endif

From: Jung-uk Kim [mailto:notifications@github.com]
Sent: Tuesday, June 16, 2015 4:17 PM
To: acpica/acpica
Cc: Moore, Robert
Subject: Re: [acpica] De-macroize calls to standard C library functions. (3b1026e)

Yes, I understand your intention but it breaks FreeBSD, for example.

If some platform has ctypes.h, which contains this:

#define isdigit(c)      ((c) >= '0' && (c) <= '9')

then acenv.h cannot do:

#undef isdigit

#define isdigit(i)     isdigit((int) i)

because it may not have a symbol isdigit.

—
Reply to this email directly or view it on GitHub<https://github.com/acpica/acpica/commit/3b1026e0bdd3c32eb6d5d313f3ba0b1fee7597b4#commitcomment-11714446>.


done


aipo.message.reloadRoomList
の中身は変更される可能性もありますので、他と同じようにメッセージ削除専用のコールバックを指定するようにしましょう。


Pretty sure that's an upstream typo, but ok.


タイムラインのファイル更新は必要なさそうなので、元のコードに戻しておきます。


This seems like it's inverted… Shouldn't it inject the CSS `unless` `disable_css` is true?

```coffee
@inject_css(instance) unless @config.ui.disable_css
```


Found.


Is it useful to disable whole auto-click feature? We still need money for upgrades though to survive the 500lvl gap faster. Make it like 5 clicks only as example.


should be
level >= CONTROL.goldholeThreshold
if we want it to start at 2000, not 2500.


I was thinking about making it 1 click


The global value should get undefined here, too...


@Arthur2e5 a split from gcc-runtime.


The options are pretty limited. I believe, but am not sure, that I could edit the annotations on the Geometry object itself to have it recommend its own namespace prefix, but I'm concerned about the cascading ramifications of that. I could attach a `NamespacePrefixMapper` to the Marshaller, but in their infinite wisdom, the Sun engineers back in the day put that in a `com.sun` package and stuck it in rt.jar. Trying to get that to build with maven is...tough. (And, because I *couldn't* get it to build, I couldn't even guarantee that it works; what I read about it was that the `NamespacePrefixMapper` could *recommend* a prefix.)

Some JAXB implementations might wrap and re-expose that, but I'm not sure about that.

We could skip using JAXB for marshaling the geometry and drop to XStream for it as well. But that's an object defined by someone else and if we managed the marshaling manually, we'd risk getting out of sync if it changed.

Honestly, I only rarely saw a different namespace prefix and in order to test this final solution I had to rerun the unit tests about a dozen times until I got an `ns2` instead of `ns1`. How? Why? I've no idea. The upside of that, at least, is that other than the check against `startsWith`, we'd rarely fall into the regex.

(And now I'll go hang my head in shame.)


HAML is used for templating.

%li{data: {invitee: '{{<variable>}}'}} {{}}

Does the trick.


getTemp の中にある entitiyId ではなく、リクエストパラメータの中にある entityId を参照した方がトラブルは少ないと思います。


We are using Makeflags thought, don't you remember?


Well whats funny is that it works (as in not crashs) as long as your pin is not correct. That pretty much rules the getText out.


I wonder if we should have an audit check for formulas in core that are also in migrations.


メッセージが自分のものなのかを必ずチェックするようにしましょう。


@Arthur2e5 I will sed them right off, after moving the bricks today.


Does this cause us to waste Raining Golds and lose money?


Are all the instance vars needed? If not, they should be removed. The lighter the objects, the better.


This should be moved to eventsSetup


I had that originally. I wasn't sure what to do here... in the docstring this function says it'll return (None, encoding) if the file doesn't exist. Really `in_place` should do something different... !

`__init__.py` uses `WriteReformattedCode`, a lightweight wrapper around `open_with_encoding`, to print this or print to stdout (if not in_place). I don't think it's any cleaner to use this there tbh.

(IMO using `WriteReformattedCode` here would be indirect for the sake of being indirect!)


Phoney like u..


Yup, but we are skipping levels in bigger steps.


That should already be taken care of in line 53


No reason to have a different function, when the handler only calls. This can be moved to events.removeInviteeClicked.


We should probably move this somewhere that looks a tad more reputable :smile:. Having said that, I wouldn't necessarily put it on the place the linux stdenv keeps its bootstrap tools, because we're going to be doing quite a bit of work to them still and need to be able to update them.


Nice one.


I'm pretty sure it's the getText() call. I spent hours debugging this a year or so ago.


No, it won't help because it is still undefined.  If you do:

```
#ifndef isdigit
#define isdigit(i)     isdigit((int) i)
#endif
```
It makes little sense, though.


Why did you backport to afterlifelochie/fontbox@61bbb57b1e21f39fc39acdfa05030049d0afb225? Commit afterlifelochie/fontbox@b5b4fee2249637d30f4cd2c7e9fc95c8226a78b9 resolves the issue preventing compiling on headless systems and supports the changes made in PR's #764 and #765.

You will not be able to compile Minechem + afterlifelochie/fontbox@61bbb57b1e21f39fc39acdfa05030049d0afb225 without undoing the changes in PR #764 and #765, as these PR's add changes to the Minechem implementation which are not present in the older commit referenced.


I basically stopped any gold income once I ran this version.  Reverted this change and started gaining significant gold again.  Gonna have to find a way to coordinate rain of golds so that we dont use them and stop clicking at the same time.


Make sure to remove every call to `console.log()` before merging. Also, please rebase this branch and reopen the merge req.

    $ git checkout dev
    $ git pull origin dev
    $ git checkout event_new_refactor
    $ git rebase dev
    $ git checkout dev
    $ git merge event_new_refactor
    $ git push origin dev


"I got a exception" is the least useful message ever.  Without a stack trace, don't even bother mentioning exceptions... seriously.

As to the other, how else will you handle serialization?  You have to have the exact same classes registered on both ends... or are you using custom serializers or something?  Somehow you will have to make the classes readable and writable or there is no point in registering them anyway.


PS/FYI github does an annoying thing if you comment on lines on the commit rather than the files tab (https://github.com/google/yapf/pull/158/files):

If I force push and lose this commit the comment is no longer associated to the PR (this is *surely* a bug)...

If you comment on the files, upon a change to those lines, it becomes an "outdated diff" (hidden by default, but you can expand it to see the old comment/diff).


@akaariai I keep working on the python3 version here:
https://github.com/pitbulk/python3-saml

Soon I will publish it at the OneLogin repo.


As long as the hash matches up, it should be OK.