drop the original stacktrace? why? you should inform the developer about the original error, not just funny(?) message. for example:
    err = new Error('the ' + type(err) + ' ' + stringify(err) + ' (' + err.stack.substring(0, err.stack.indexOf("\n", err.stack.indexOf("\n") + 1)) + ') ' + ' was thrown, throw an Error :)');
a little part of original stacktrace instead of "the error {} was thrown, throw an Error :)"  meaningless funny message.
under nodejs


Perhaps worth noting that this accesses 5th item in the list?


Thanks, I plan to implement the device + user support. Haven't though about exactly the best way to tackle that yet but now that I have my own table it should be quite easy.

The import function is cool, but for now the users can change it themselves.

This is the start of my hostile takeover of PlexWatch :smile:


When there's just a single return value, no need for a list:

Returns (boolean) did the  test succeed?

Same elsewhere


Same here.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17070/testReport/) for this commit**


Do not forget the library. Good at all.
Library that have to add:
#include <cstdio>
#include <cstdlib>


Sure,  fixed #0f0e0d144c2de4459032f803ecad1c2bb74b4908


Hi Leon,

I have now formally submitted this patch to the qemu-devel mailing list, so I will drop this pull request.

Many thanks,

Andrew


Suggest giving dictionaries some love.


The original short description is preferable as it's more succinct and easier to parse.

Same elsewhere


That's ugly. Couldn't we patch uploads.py at deployment time instead?

The only differences I see are:

+import sys
+
+PY3 = sys.version_info[0] == 3
+
+if PY3:
+    string_types = str,
+else:
+    string_types = basestring,

and

-        if isinstance(dst, basestring):
+        if isinstance(dst, string_types):

Did I miss something?


Not needed.


Can we have addresses here?


This will silently corrupt valgrind memory when flush_misses_reg will exceed MAX_FLUSH_ERROR_EVENTS.


space after if


Unnecessary init


`a` and `b`


@mbernat Cool :-)


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 11](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=22&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


I don't see how you do the rest of the logic in that case


Note that else if will raise an error!


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 11](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=22&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:03


Fixes an indentation issues.
Before:
```js
if (true) {
    } else {
    }
```
After:
```js
if (true) {
} else {
}
```


Hello Phil,

In what sense is this (single response pattern as an input ) a bad behavior?

Consider the scenario where one has calibrated whole items based on manyy response patterns (given by many examinees). However when estimating the ability if he/she is interested in knowing intermediate and final ability of only one student; in this case also it is a bad behavior? If yes why not in the case of one dimensional its not considered to be bad?


@kimuramanabu リポジトリを落とす人向けです！おっしゃる通り階層が足りないので、config, templates, logsの階層を作ります！


Maybe we should use green (or blue) here?


Thank you! That was doing my head in.


I believe it is somewhat explained in this talk: https://www.youtube.com/watch?v=GsLZz8cZCzc


But it's using new_val (doing the replacement) only if bool(pattern_matches) != bool(complement)


Great. Your changes work beautifully.

I deleted the axopatch branch on my side.


`lazy var` sounds great!


There's an itervalues changed to values.
(Which is actually bad for python2 support)


Hard core! I suggest introducing list comprehensions in more detail, importing standard libs with something like CSV as an example. Namespaces. Then pip install something. Then clarify difference between 2 and 3.

This will be plenty. Scons is way too advanced.


@martipello did you create rootView?
See line 104:
`View rootView = inflater.inflate(R.layout.fragment_main, container, false);`

(and then return it in line 110).


Well, mentioning pandas and built-in would be good.


Then, you can have `FakeConditionsMet.preg_month` always return `EMPTY_FIELD`, since that's what you are doing when it raises Exception.


If you have a clean php install and just echo anything you will get a warning... (at least from my experience)
This happens in bootstrap code then, but not by calling `Base->get('TZ')`


One other minor tweak is to swap method and jac, so that the latter can be used as a positional arg.
(this is minor, debatable, up to you.)


The problem with this approach is that if you run this script in cron *adding* data to an existing CSV file, you will get the header data again for each run.


TODO alert with module for importation


would it be possible to change from "Depends" on OPM and RT to "Suggests" OPM and RT?
There are some people that don't use RT but want the antennas anyway (because they are compatible with AntennaRange)

out of curiosity, could I made the changes if I wanted to? I don't want to bother people (like @dbent) if it's something I can do myself. Not that I don't apreciate the help of course :) I just don't want to be a problem for anyone


OK, I was a bit fast updating that. But please note that this code is repeated a zillion times in the code and my regex skills are not as good as yours... We should really try to avoid code duplication like this for maintainability.


Error: need to remove the `


is text/   for text/plain and text/html ?


I think you're right. I considered the `www` subdomain as a redirect to the root domain, but in practice it's usually the other way around and indeed, you do not want to end up with `x.www.domain.com`.


good catch. apparantly not used that often (and no test for it....)


Этого как раз можно назвать
```ruby
account_owner
```


@theute it works, but it's very unreliable. I re-run the travis build 3 times during the night and those "curls" weren't working (http://git.io/vtVdd). I did the same 10 minutes ago and they passed and the real values are there.

This is really bad, I'd leave it for couple of builds to see if it was some kind of glitch yesterday, or if it's normal behavior on travis.

(https://stackoverflow.com/questions/31113316/running-curl-rest-calls-from-travis-before-install-script-is-very-unreliable)


test2-shi pirvel assert-ze ragac shecdomas agdebda, amitom komentarshi chavsvi


However, couldn`t we just check if the current subdomain is `www` and remove it *only* then? IMO that would cover most real life cases instead of having to edit the suggestion each time when your main wiki us hosted on a subdomain and not on the root domain.

WDYT?


Doc


@atombaby , is there an ipython notebook server available? This might be a nice way to intro folks.


A great start, @metasoarous ! Thanks for getting this going ahead of time.


encourages to use?


on a small scale


object-oriented languages


Please move that to the cc file.


drop the only


Hej @ejabach, cool idea to move these statistics configuration into the my_enable_stats section :+1:
Now you need to add spec tests for that. Maybe add a hint to the readme too.


programming context?


Is n't ng-doc important?
Those can be useful for new comers.


I think we want to run this only when adb mode is needed? If that is the case then we should drop [Install] section and "install -m 644 -D systemd/adbd-prepare.service %{buildroot}/lib/systemd/system/graphical.target.wants/adbd-prepare.service" from spec file.


s/That/This/ ?


first commit


2 -> two


@albertosouza blz Alberto, não reparei que estava enviando o .DS_Store.
@gpedro Valeu, vou add aqui!


There is NO point in removing files --- the WHOLE repository, including its history, must be deleted


doc


Consider downloading visual studio and compile the project for yourself.
Alternatively you can take a look at http://jenkins.x64dbg.com there is a
directory with a snapshot in there.

On Mon, 29 Jun 2015 12:11 cxj98 <notifications@github.com> wrote:

> if some file not changed, will you daily build new updated files, thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/x64dbg/x64dbg/commit/c753461a3592c4692ef76ef9bf9450394b39c037#commitcomment-11907007>
> .
>


ну это вообще безпалевней некуда


Maybe make the `what` an actual question and `I would argue..` a separate sentence?


Ah, it must have snuck in at some point.


Renaming `bp` to break is still on our radar, right?


Behind poolboy how would we call the worker?


I did not want to change anything from the script instructions. https://netguru.atlassian.net/wiki/display/DT2015/Dashguru+MongoDB+Data+Migration+to+PostgreSQL

I would suggest running the script first and then introducing any changes to the script code. What do you think?


need to reset here if `:raw`


Harumph. You do know that there are other video formats. My `.avi` videos [stopped](https://code.google.com/p/ankidroid/issues/detail?id=2669) since this commit.


in-code -> in-source (buffer) perhaps? Sounds a bit clearer to me.


"make install prefix=$HOME/lmdb" ?


Btw, maybe should should check for word end in the regexp.


Yes. Thanks for catching that.


that this approach is..?


Looks like you misspelled it


Shouldn't it be `cider-debug-mode`? It's not private after all.


Any specific reason of dropping `react-a11y`?


this seems off from our usual code style


double like in filtering sentence


Are the single quotes in `author` and `link` already properly escaped?


`line[2] == 'CDS'` не работает?


thx


@Simn The thing is, you can't just update `isSpace`, because nbsp is at codepoint 160, so you need to be encoding-aware. That would render a lot of implementations (e.g. neko) at least 100% slower. I'm not sure marginal gains on JS warrant for this.

@ncannasse No, they don't. In fact [Java just blindly considers everything smaller than `\u0020` a whitespace](http://docs.oracle.com/javase/7/docs/api/java/lang/String.html#trim()) which is cruder still than our implementation. If speed matters that much, then sure, let's do it (although it pains me infinitely, since my avatar is `0x02` on MS-DOS :D). For .NET, the behavior actually changes between versions (see "Note to callers" in https://msdn.microsoft.com/en-us/library/t97s7bs3(v=vs.110).aspx ). And there is an [exhaustive list of all space characters according to the ES spec](https://github.com/paulmillr/es6-shim/issues/253), which indicates that `U+0085` is not included in the ES6 spec, as opposed to .NET's [Char.IsWhiteSpace](https://msdn.microsoft.com/en-us/library/system.char.iswhitespace(v=vs.110).aspx) which does.

@waneck I feel that it's important for such core classes to behave as consistently as possible. Nothing stops you from exposing the native behavior for those who are aware of the differences and deliberately decide to use the native variant. But the truth is that Java and .NET are taking very different approaches to the subject.

@explorigin Yes, of course. I'm eager to see the data resulting from benchmark you are working on and learn what's wrong with mine. But until such time, I maintain there is no conclusive data (yet) to suggest that making this change will increase speed so much that it's worth going through all the issues it entails.


Sort of. It's definitely not meant to be activated by the user, but `define-minor-mode` sadly doesn't have a non-interactive option.
Tough I'm not particular on that.


Is it? This is auto-format of Android Studio. I don't see this defined in our code styles.


Can't the init logic be embedded in the mode itself? It sounds odd to be able to activate a mode manually when it won't be initialized properly.


Thank you @AveVlad


can we scrable the contents of `raw` when we're done?


Should handle this return gracefully if self.video.name is None.
See: Error
AttributeError at /getinvolved
'NoneType' object has no attribute 'rfind'
Request Method:	GET
Request URL:	http://mapstory.beta.boundlessgeo.com/getinvolved
Django Version:	1.6.10
Exception Type:	AttributeError
Exception Value:
'NoneType' object has no attribute 'rfind'
Exception Location:	/home/mapstory/.virtualenvs/mapstory/lib/python2.7/genericpath.py in _splitext, line 91


Well I will lead you in the ionic part and you lead me in the backend part.
As of now come up with a design for the backend.

Regards
Giritheja


OK, got it.


yes, all text/ mimetypes match

On 29 June 2015 at 12:40, Ernest Micklei <notifications@github.com> wrote:

> is text/ for text/plain and text/html ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salandur/critter/commit/9b3b0297481663664a5b9ca020aeed7937af11a7#commitcomment-11907404>
> .
>


I get an error due to the changes to `acq4/drivers/nidaq/SuperTask.py`. The import of NiDAQError `from .nidaq import NIDAQError` fails which subsequently prevents the script from loading all the daq devices.

Here is the error:
-------------------------------------------------------------------------------------
2015.06.29 12:59:36 Error configuring device DAQ:
ImportError: cannot import name NIDAQError
ImportError: cannot import name NIDAQError
  File "C:\Anaconda\lib\runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "C:\Anaconda\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Users\2-photonScan\experiments\acq4\acq4\__main__.py", line 44, in <module>
    man = Manager(argv=sys.argv[1:])
  File "acq4\Manager.py", line 218, in __init__
    self.readConfig(configFile)
  File "acq4\Manager.py", line 304, in readConfig
    self.configure(cfg)
  File "acq4\Manager.py", line 331, in configure
    printExc("Error configuring device %s:" % k)
  File "acq4\util\debug.py", line 19, in printExc
    acq4.Manager.logExc(msg=msg, msgType=msgType)
------- exception caught ---------->
  File "acq4\Manager.py", line 329, in configure
    self.loadDevice(driverName, conf, k)
  File "acq4\Manager.py", line 461, in loadDevice
    dev = devclass(self, conf, name)
  File "acq4\devices\NiDAQ\nidaq.py", line 28, in __init__
    from acq4.drivers.nidaq.nidaq import NIDAQ
  File "acq4\drivers\nidaq\nidaq.py", line 11, in <module>
    import SuperTask
  File "acq4\drivers\nidaq\SuperTask.py", line 8, in <module>
    from .nidaq import NIDAQError

-------------------------------------------------------------------------

I tried but could not get the import to work properly.

Thanks for help.

Cheers,
Michael


By the way, it's no big deal but if you comment on the PR diffs instead of on my fork's commits then the comments show up on the PR's conversation (which is a little easier to keep track of).


Great commit... Thank you very much :P :+1:


Starting here (I think), a lot of entries are duplicated (cf. this with [line 203](https://github.com/JuliaLang/julia/commit/4f4e2f9c8ba8d06f498c846162d7e2437289b7ad#diff-badd7d54b8cfdba53f7e1064089334adR203), then [237](https://github.com/JuliaLang/julia/commit/4f4e2f9c8ba8d06f498c846162d7e2437289b7ad#diff-badd7d54b8cfdba53f7e1064089334adR237)-[248](https://github.com/JuliaLang/julia/commit/4f4e2f9c8ba8d06f498c846162d7e2437289b7ad#diff-badd7d54b8cfdba53f7e1064089334adR248), [254](https://github.com/JuliaLang/julia/commit/4f4e2f9c8ba8d06f498c846162d7e2437289b7ad#diff-badd7d54b8cfdba53f7e1064089334adR254)-[265](https://github.com/JuliaLang/julia/commit/4f4e2f9c8ba8d06f498c846162d7e2437289b7ad#diff-badd7d54b8cfdba53f7e1064089334adR265), etc.). Wut?


Where's migration for announce column ?


ping @hamorphis


we could, but if you're worried about someone doing direct memory access, there are much simpler ways. After all, we're reading all this from stdin. This is also done by an admin user, which means the http transport will have read the client certificate key, and is still holding the certificate in memory.


Ah ok, avi is not listed as on officially supported format... Does it work
on all your devices?

http://developer.android.com/guide/appendix/media-formats.html

I could set it to only block certain formats known as audio files instead
of only allowing known video formats, it just would have taken more work.

On Mon, Jun 29, 2015 at 7:56 PM, ospalh <notifications@github.com> wrote:

> Harumph. You do know that there are other video formats. My .avi videos
> stopped <https://code.google.com/p/ankidroid/issues/detail?id=2669> since
> this commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hssm/Anki-Android/commit/5e84b1e23e537f8374402e3b1ee60ae4d6ce5dff#commitcomment-11907604>
> .
>


Лишнее действие. Лучше два контекста: user is logged in и user is not logged in


is working now with: composer install and composer install --no-devs
thanks a lot, i'm testing with marcxml records and is very fast and confortable.


Thanks you very much for the PR, would you be kind and fix the formatting? :-)


본적이 없는 컨벤션인데, 이런 형태로 포맷팅을 해야 할 필요가 있을까요?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17075/testReport/) for this commit**


Default 접근 제어자를 사용해야 하는 이유가 있는 것이 아니라면 은닉화하는 것이 좋겠습니다


The idiomatic way is to use `Option`.


Этот ```user_ count``` избыточен


Good idea.

Date: Mon, 29 Jun 2015 03:36:07 -0700
From: notifications@github.com
To: bounded-lsq@noreply.github.com
CC: n59_ru@hotmail.com
Subject: Re: [bounded-lsq] Implemented the first version of generic interface (133395a)

One other minor tweak is to swap method and jac, so that the latter can be used as a positional arg.

(this is minor, debatable, up to you.)

—
Reply to this email directly or view it on GitHub.


Hi,
it make sence, and I'm not against. But I would like to see also option to do that like --dir SOMETHING what copies content of directory, to remote test directory
    Regards


Typo (Simulaeous => Simultaneous) in the commit message.


이런 종류의 주석이 꼭 필요한 것일까요?

가급적 변수의 이름을 잘 짓는 것으로, 대부분 해결할 수 있다고 생각합니다.

꼭 필요한 것이 아니라면 없애는 것이 좋을 것 같습니다.


> npm ERR! 404 404 Not Found: webcamjs
> npm ERR! 404
> npm ERR! 404 'webcamjs' is not in the npm registry.
> npm ERR! 404 You should bug the author to publish it
> npm ERR! 404
> npm ERR! 404 Note that you can also install from a
> npm ERR! 404 tarball, folder, or http url, or git url.


singleton으로 사용하고 싶다면 Spring Container를 사용하고 있으므로, Bean으로 만드는 것이 좋을 것 같습니다.


Maybe the first run should be excluded from the statistics, because it would include the compilation times. For the subsequent runs everything should be already compiled.


有翻译错的地方


这里错了


The function name does not seem to match the explanation on line 100.


Wont `#define IS_RECEIVE_FIFO_FULL(x) ( ( (x >> 16) & 0x7) <= 8)` always return true?


дак и есть два таких контекста...просто на  top уровне юзер залогинен по умолчанию


이 class는 반드시 필요한 것인가요?

아래의 SheetFile의 특정 필드만 가져와서 getter만 제공하는 형태의 class를 필요로 하는 이유가 있나요?


r u gonna comment every commit, princess :D


중간의 연결이 추가되거나 제거되는 형태로 사용되는 것이 아니라
sequential하게 접근하는 종류의 데이터라고 생각됩니다.
ArrayList가 낫지 않을까요?


In my honest opinion, upgrade_url method should return a string with the url or an empty string, what do you think @juanignaciosl ?


Is that right?
To check: Is time being maintained correctly for each point


Just wondering if https://github.com/Tux/speedtest/commit/0064412de550932fb50a074451a0ea1a53cac0e6 fits your needs in the perl version

``` text
$ speedtest -P
<?xml version="1.0" encoding="UTF-8" ?>
<prtg>
  <text>Testing from My ISP (10.20.30.40)</text>
  <result>
    <channel>Ping</channel>
    <customUnit>ms</customUnit>
    <float>1</float>
    <value>57.23</value>
    </result>
  <result>
    <channel>Download</channel>
    <customUnit>Mbit/s</customUnit>
    <float>1</float>
    <value>39.37</value>
    </result>
  <result>
    <channel>Upload</channel>
    <customUnit>Mbit/s</customUnit>
    <float>1</float>
    <value>36.63</value>
    </result>
  </prtg>
```


Won't this always return true? If you bitwise-and something with 7, won't it always be 7 or less?


enum을 사용하고 있는데 String으로 된 key로 접근하는 것은 비효율적으로 보입니다.
animataionSheetMap의 key를 TextureEnum으로 하는 것이 더 좋을 것 같습니다.


blah test blah


I only have Android 4.2 and 4.4 devices. There it works. I guess positively matching sound file extensions might work better, but wouldn’t be foolproof either.

Maybe some other test based on the file (content) can be used?
Maybe use a list of only-video, one list of only-audio, and use the (maybe slow) content-based test only for ambiguous extensions?


Строка 56: имя поля мне кажется не очень понятным


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17076/testReport/) for this commit**


변수의 이름이 tmp이고, 하는 동작도 이 부분만을 보고는 알기 어렵다고 생각합니다.
private method로 분리하고, 적절한 이름을 붙이면 가독성이 좋아질 것 같습니다.


Are you supposed to unlock, do something, and then lock? Instead of the other way around?


Строки 19-22: некоторая неаккуратность с использованием кавычек. Можно сделать одинаковые.


Строки 160-163: вероятно, имелось в виду передача констант, а не строковых литералов


We should perhaps write a `whatsapp_to_msisdn` function so that people (likely future us) reading the code doesn't wonder what the `[0:-15:1]` is about. Might also be polite to replace `15` with `len('@s.whatsapp.net')` and I think we can leave out the final `:1` since that's the default.


1행만의 조건문이라도 중괄호를 사용하는 것이 여러가지 컨벤션에서 추천되고 있습니다.
굳이 독자적인 컨벤션을 사용해야 하는 이유가 있을까요?


I'd propose using something more common here, like compatible with sum or cksum tools available in devkitPRO/msys and in most Linux distributions for sure. The first one is BSD checksum, simple and fast yet 16-bit only. The second one is Ethernet-compatible CRC-32 - slow but much more reliable. IMO 16-bit should be enough for file integrity check and I can make a pull request tonight. If you insist on using 32-bit checksum, it may take a bit longer to implement the second one.


We don't know if the D-Bus call is failing due to D-Bus being unavailable, we just know that the call failed. Avoid messages being overly precise without data to base it on, that can mislead when debugging stuff.


Please explain why you are using "return" here. This file is included, and "return" will stop execution if called from the global scope.


why it would be better to check for existance instead for a value?
In my opinion it's important to check if the value has been set exactly as expected (especially we should do this for privacy settings)


일반적으로 3이상의 indent를 사용하는 것은 지양되고 있습니다.
또한 개념을 잘 표현하지 못하고 있다고 생각됩니다.

private method로 추출하고 이름을 붙이는 것이 어떨까요?


I messed up in the rules.py file with the indentation.


@tchap oh, maybe I'm finally getting it. So slices are just references to arrays (or their subarrays) and `Labels` thingie is an array so needs pointers. I'm not used to this C shit anymore :D


startPlay라는 이름은 의미가 중복되어 조금 이상하다고 생각됩니다,
그냥 play면 안될까요?


Попробуй изменить константу `SCALE` и проверь, что всё работает правильно


perfect!!!!


remov the unesued


`if not os.path.isfile(os.path.expanduser('~/.telegram.conf')):` would have been much more readable/pythonic.


let's make the filter window


Do you have the latest version of the develop branch?


현재 재생상태인지를 확인하는 method에서, play를 시키는 것은 이상하다고 생각됩니다.
playWhenAnimationIsEmpty 등으로, 하는 행동을 잘 표현하는 이름을 짓는 것이 좋을 것 같습니다.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17077/testReport/) for this commit**


Если константы `SCALE` и `SHIFT` взаимосвязаны, сделай так, чтобы одна вычислялась через другую


Well, ok, but you could e.g. have `if [ "$make_zip" = "yes" -a "$i" = "$GTK" ]` in the loop, which would make sure it's done for GTK only (and so avoid doing it uselessly for the others BTW)


mélange de controller dans la vue


@mbernat No :D `Labels` is a pointer to slice. Really, it's just about encoding. You almost never see raw arrays in Go...


#bios
@@ -0,0 +1,2 @@
legacy update enable


Because that's an unnecessary update to core files - Gary pointed me to this way in the addons best practices thread: http://forums.oscommerce.com/topic/407530-addon-development-best-practices/?p=1727270


Typo


Could it be that `instance_parameter` belongs to the `Function` struct?
```rust
struct Function {
    //...
    instance_parameter: Option<Parameter>,
    parameters: Vec<Parameter>,
    //...
}
```


Typo in the filename.


I can change the flow from

    (cider--turn-on-debug-mode some-response)

to

    (setq cider--debug-mode-response response)
    (cider-debug-mode 1)

If that's preferred. Then `cider-debug-mode` can refuse to activate if `cider--debug-mode-response` isn't set.


@tchap But you just said that slice is a referential type, so doesn't need pointers o_O


I don't know where the syntax like @code{.cpp} come from. Tell me if you know under what syntax it is used.
Otherwise, let's use standard markdown syntax for the tutorial to make it look better on Github:) i.e. use 4 spaces or "``" to format the code.

I also suggest to add the following contents after you finish the main coding work, but anyway it is up to you to decide whether or which to add.
* A sample set of chessboard images and a sample code of calibration using them
* One or two sentences explaining that catadioptric model can also be used for fish-eye camera.
* Possibly some comparison with `fisheye` module on calibrating some fish-eye photos.


children


It's lost.


You mentioned a risk of getting too much log files (one per step) in one directory, so maybe we should delete the file after sending it to the write_to?


Ещё нашёл баг: запустил змейку с большим количеством яблок (конкретно с 200, но это не влияет) и некоторое время успешно ел яблоки. В какой-то момент яблоко не появилось (оно оказалось под змеёй). Когда змея отползла, яблоко стало видно.


First commit.


The filter windows ?

Sorry need more info (still new to this)

Sent from Windows Mail

From: Fastcolors
Sent: Monday, June 29, 2015 7:29 AM
To: fastcolors/skin.lcars
Cc: metaloid

let's make the filter window

—
Reply to this email directly or view it on GitHub.


PHP7 will be built with VC14 (VS 2015), not the VC11 you seem to use in your build script.


Okay, I reverted that change for now. We should decide what to do with C# version tho.


Чтобы не дублировать тело цикла (хотя оно здесь и состоит из одной строки), можно просто указать `apple_on = True`


why you did so?? O_o


@firejun Thank you for sharing the report. I need the CPU and memory monitoring result(chart). It would be nice if you consider that.


good catch, didn't know this specific syntax issue for less.
Thanks :)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17078/testReport/) for this commit**


@brianmario you could open issue on Electron. Also can you attach `log-net-log` dump with and without the `ignore-connections-limit`. If it actually works , you should see more connect_jobs per groupname and there would be no log about `SOCKET_POOL_STALLED_MAX_SOCKETS_PER_GROUP` or `SOCKET_POOL_STALLED_MAX_SOCKETS`


I'm writting this code at main.py and it gives me a syntax error
reply(img=urllib2.urlopen(http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read())

Which is the problem? I'm a noob at this, sorry.


Строка 83: при желании можно сократить код до такого:
```python
position = self._direction_action[direction](*self._snake.head())
```


Smart! :-)


added a TO DO on the page.

Let's get this skin up to speed and we'll try to make it into official repo!
what do you say?


@mbernat I meant that you don't need to care about whether you are passing a pointer or a value, because copying is basically for free since it's just a reference under the hood. But for the encoder it matters of course. It's all explained somewhere above in this long thread :-)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17079/testReport/) for this commit**


Note that (it may not apply to `Widget`s) sometimes functions that are constructors for this module purposes are just "function"s. `gtk_css_provider_get_default` is an example.


You mean moving `cartodb_com_hosted?` check inside `upgrade_url`, isn't it? Totally agree. I'm doing it right now.


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


reply(img=urllib2.urlopen('http://
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
www.arenalsound.com
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
/
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
wp
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
-
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
content
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
/
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
uploads
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
/2015/06/
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
as_cartel5
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
.
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
jpg
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
').
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
read
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>
(
<http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(>))

Am 29.06.2015 13:43 schrieb "Shurkevin" <notifications@github.com>:

> I'm writting this code at main.py and it gives me a syntax error
> reply(img=urllib2.urlopen(
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> ))
>
> Which is the problem? I'm a noob at this, sorry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yukuku/telebot/commit/a702777b2bec2517cb4fcc6f94d9638cff734a5f#commitcomment-11908282>
> .
>


Yep, thanks! :)


Seems good to me. Actually, I don't think it is very risky.


Let’s go for it

Sent from Windows Mail

From: Fastcolors
Sent: Monday, June 29, 2015 7:45 AM
To: fastcolors/skin.lcars
Cc: metaloid

added a TO DO on the page.

Let's get this skin up to speed and we'll try to make it into official repo!
what do you say?

—
Reply to this email directly or view it on GitHub.


I didn't know that we could put the tests class directly in the setup.py. Interesting.


// this is the question


Very tiny bikeshed: we can leave out the final `:` too.


Je n'ai jamais eu de problème avec le Frustum :-/


R.utils package name fixed


I was originally going to add an extra block, something like:
```java
        if (isOnPath("perl"))
        {
           return  getCommand('makeglossaries', options, getBasename(file));
        }
        else if (isOnPath("makeglossaries.lua"))
        {
           return  getCommand('makeglossaries.lua', options, getBasename(file));
        }
        else
        {
           // throw an exception? Make an educated guess and call makeindex?
        }
```
The scenarios are:

1. Perl is installed and on the path so just run `makeglossaries`.
2. Perl isn't on the path so run `makeglossaries.lua` if it's on the path. (I'm going to rename this `makeglossaries-lite.lua` following Paulo's recommendation.) This Lua script will only be available once I release the next version of `glossaries`, so the user may not have it installed.
3. Perl isn't installed and the Lua script isn't on the path.

I'm not sure what would be best here for 3. The only things I can think of are:
* Throw an exception but give the user some help as to what to do.
* Test the existence of `.gls` or `.acn` and run `makeindex` with a warning that the rule is just guessing. This would only work if the document uses the default settings.

With the current code, if the Lua script isn't available the user will just be informed that the rule couldn't execute the command. Should we just leave it as it is (in which case I'll make the change you suggest) or should we provide a fallback for case 3?


Btw did u check the new look for the osd ?

On the branch section

Sent from Windows Mail

From: Fastcolors
Sent: Monday, June 29, 2015 7:45 AM
To: fastcolors/skin.lcars
Cc: metaloid

added a TO DO on the page.

Let's get this skin up to speed and we'll try to make it into official repo!
what do you say?

—
Reply to this email directly or view it on GitHub.


Так как я изначально предлагал сделать не это, думаю, в этой функции почти нет смысла, можно её перенести в `put_apple`. А там уже сделать так, чтобы если больше нет свободных позиций на поле, возвращалось значение `False`. Если установить яблоко удалось, вернуть значение `True`. Кстати, заметь, что чем меньше свободных позиций на поле, тем меньше вероятность попасть на свободную точку яблоком, а значит, цикл будет всё дольше выполняться. Можно попробовать проверить, является ли это в действительности проблемой. Хотя даже если и является, то довольно редкой.


Hi
During my attempt for installing gr-foo at make command i had the following error:

:~/gr-foo/build$ make
[  5%] Building CXX object lib/CMakeFiles/gnuradio-foo.dir/wireshark_connector_impl.cc.o
/home/aaaaaa/gr-foo/lib/wireshark_connector_impl.cc: In member function ‘virtual int gr::foo::wireshark_connector_impl::general_work(int, gr_vector_int&, gr_vector_const_void_star&, gr_vector_void_star&)’:
/home/aaaaaa/gr-foo/lib/wireshark_connector_impl.cc:165:57: error: no matching function for call to ‘gr::foo::wireshark_connector_impl::delete_head_blocking(pmt::pmt_t, int)’
   pmt::pmt_t msg(delete_head_blocking(pmt::mp("in"), 100));
                                                         ^
/home/aaaaaa/gr-foo/lib/wireshark_connector_impl.cc:165:57: note: candidate is:
In file included from /usr/include/gnuradio/block.h:27:0,
                 from /home/aaaaaa/gr-foo/include/foo/wireshark_connector.h:21,
                 from /home/aaaaaa/gr-foo/lib/wireshark_connector_impl.h:20,
                 from /home/aaaaaa/gr-foo/lib/wireshark_connector_impl.cc:17:
/usr/include/gnuradio/basic_block.h:259:16: note: pmt::pmt_t gr::basic_block::delete_head_blocking(pmt::pmt_t)
     pmt::pmt_t delete_head_blocking( pmt::pmt_t which_port);
                ^
/usr/include/gnuradio/basic_block.h:259:16: note:   candidate expects 1 argument, 2 provided
make[2]: *** [lib/CMakeFiles/gnuradio-foo.dir/wireshark_connector_impl.cc.o] Error 1
make[1]: *** [lib/CMakeFiles/gnuradio-foo.dir/all] Error 2
make: *** [all] Error 2

for solving this error i used your previous revision (instead of last revision)
starting from scratch, I changed the revision during the compilation of the latest revision.

After cloning the repo, just enter the following command:

git reset --hard 00beec517d096e4f2f6a45250750369db90dc104

Best Regards


Строка 138: с какой целью подобная итерация?


I didn't know that either
I wonder whether it's a bootstrap2 issue, mah


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


Written by Phuong H. Hoang and Ha M. Doan, April, 2013


@TreyE  @sagarlekar

dob,  are we using datepicker_control or text_field?


This is a set of presentation slides. The comments are added between them while I discuss what happened. If you'll notice there are subtle changes between each slide with the same heading.


Не думаю, что это решение проблемы. Это, скорее, её избегание. Но если пользователь сможет выбирать количество яблок и размер поля, всё свалится.


**Misleading commit message**

> New .htaccess file to allow file revving for cache busting - includes modification to Gulp task

Modification to `moveotherfiles` task to including moving files not in a directory


bootstrap should be under assets so the link should be <link src="static/assets/bootstrap/css/bootstrap.min.css"/> which does not load unless the test server is running. It still displays incorrectly on my computer - not 100% wrong but there are bugs.
Since you copied this you have to change the header because you removed main.js from its script tag, so now the search isn't working.


@prclibo I find out @code{.cpp} in tutorials of other modules in opencv_contrib. I guess it is something in doxygen so they use this rather than "```".

This tutorial is a draft version, I will update it during development. :)


What's the reasoning behind changing the number of images per row to 4? I chose 3 so that they remain somewhat visible on mobile devices.


Here you say "sending"


this doesn't seem right, usually there are only a set number of files in this directory. what have you got here that's so big?


Строка 88: когда змейка имеет излом под прямым углом, разворот выглядит не очень интуитивным действием. Возможно, некоторые ходы всё же стоит запрещать. Есть ощущение, что разворот будет удобным, если нажата стрелка, соответствующая положению хвоста, а не предыдущей клетки змеи


Can the "to set" be removed?
If yes please do so.


@tchap Yeah, I thought it's because of the encoder, I even mentioned it couple comments above, but then you started talking about slices, so you confused me.

So again, as I mentioned above, there should be two separate types, one for the queries (handled automatically by the encoder, can contain pointers and nils) and one for the actual data that should only contain plain arrays and no nils. I think you never responded whether it's a good idea or not. I mean, it is better code, but maybe it's just not worth it.


Remember to free ```config``` with ```libusb_free_config_descriptor```, either in the exit paths or just before the loop if you store ```config->bNumInterfaces``` in a variable.


@jscotka right! I see that is already tracked on issue #350. @ldoktor since we have another issue for that already, I believe we can treat the two things independently.


Any specific reason why you call ```libusb_close()``` in this function? having the open function call it seems more symmetric.


Added labels for DFormat


Why detaching kernel drivers again?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17081/testReport/) for this commit**


Yeah that's what I meant... there's already a content based test for
ambiguous extensions.

On Mon, Jun 29, 2015 at 8:22 PM, ospalh <notifications@github.com> wrote:

> I only have Android 4.2 and 4.4 devices. There it works. I guess
> positively matching sound file extensions might work better, but wouldn’t
> be foolproof either.
>
> Maybe some other test based on the file (content) can be used?
> Maybe use a list of only-video, one list of only-audio, and use the (maybe
> slow) content-based test only for ambiguous extensions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hssm/Anki-Android/commit/5e84b1e23e537f8374402e3b1ee60ae4d6ce5dff#commitcomment-11907990>
> .
>


Yes. They are similar problems. However the cause of the two issues are not exactly the same. For this issue, it is caused by the improper constructor while for #156, it mainly caused by not having a proper setter. I'll send a PR for fixing #156 then.


Would be great to actually check for the correct size of separator as we know what it should be.


+ Ensure `filemtime` is a whole number


Трябва да е POST.


This linebreak, might be good for Tooltips, but what about other causes.
I have envisioned better error messages when not enough (or too many) args is passed to a command.


Трябва отделно да се обработи ReservationException, за да се върне съответния статус, който е CONFLICT, а не INTERNAL_SERVER_ERROR.


@mbernat I did respond. I said that it an extra effort that might not be worth it, but ok, I will look into it...


Строка 138. После завершения игры без неё бросается ещё одно яблоко на поле. Получалось как-то некрасиво, говорить, что все яблоки собраны, но показывать, что появилось ещё одно


Here you are still setting the configuration at **each write**, ideally the configuration should be set once at open-time.

I the comments of the previous version I proposed to add a configuration parameter to the open function and have some wrappers to keep compatibility for when the standard config is required:
https://github.com/dominicgs/hackrf/commit/0d330cbb657ee5427c44acd79cdd1f7c837f0b3f#commitcomment-10803965


Not needed, ```set_hackrf_configuration``` has just been called.


Трябва да го могат само администратори. Трябва да се върне съответния статус UNAUTHORIZED, когато някой BASIC се опитва да плаща.


setlocale is a C function and does not throw exceptions.


reservations.


GitHub


@1aurabrown Let me know if that describes it well enough.


Because changing the configuration could cause kernel drivers to be automatically attached to interfaces on the new configuration.


@Path("/{username}"), Гетовете не трябва да имат paylaod.


Като горното.


Not sure it will be possible given that Task (and its subclasses) are part of scheduler-api and Containers/InternalTasks are part of scheduler-server.


LOL, i can't understand how it works :) .

Why `(?=[^-]|$)` needed after `(?:-*)` ? If you eat next `-`, gready then next will NOT be `-` without lookahead check. Do i miss something?


Спасибо!


btw, they're still referenced in the `index.html`.


I'll be able to make more comments tonight when I'm at a keyboard, but I definitely lean toward the block version now that you raise those concerns. From a user-experience point of view, it would be very strange indeed to ask for `makeglossaries` and get a complaint that `makeglossaries-lite.lua` is not available.

Perhaps the cleanest solution is another Java 'primitive' that takes a variadic array of programs (`String... programs`) and tries each one in succession, returning the first program on the path. That way a message could say 'None of ... were found on your PATH. Are you sure you have this tool installed?`.

It would be even better to have a version (or same-name generalization) of `makeCommand` that does this.


There is a bigger plan for this: removing the second configuration.  It causes problems with CPLD updates on Windows and is inconsistent, for example we don't use a separate configuration for the firmware update.


tki


This is unrelated to this pr. I'd recommend creating a separate issue about this.


sieht sehr gut aus!


Pardon; not `makeCommand`. A signature such as `String getCommand(String options, String arguments, String... programCandidates)` would be cool, but would certainly break many existing rules.


not sure if "path" should be conceptually  included into enum;
flag should be a char


Could always introduce a new class in the Task hierarchy that is not exposed to the public API but that we know that it exists so that we can perform a cast and retrieve what is needed.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17083/testReport/) for this commit**


luaL_checkboolean throws an error when there is no bool given

you want luaL_optboolean


Bad idea. The problem comes from HUI_Scrollbar and not HUI_Button.

HUI_Scrollbar::Update is wrong: you should:

Check if the button is active. If yes, update the position of the button.
Update the button.

With this, it should be OK.


Строка 138. Какое сообщение он показывает?


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


If worth to add a real example from bloomberg, as explanation why this "strange" rule was added.


Not to mention there's a ")" missing


@bw2 , @chapmanb this appears to have broken use of `--tempdir` that isn't NFS'ed. I can look into it later, but let me know if any ideas for quick fix.


Ah disregard. I see the change now.

I didn't look for changes outside the code. (You know, I was looking for thing that made the snippet safer. Apparently, you didn't include that in the presentation :))


Это просто PyCharm пишет, что need more value to unpack


There is a translation system in GeneWeb. So you should write :
```ocaml
Wserver.wprint "%s" (capitale (transl conf "add the previous name as a surname alias"));
```


Действительно, я запустил так, сработало


Но вопрос от PyCharm остался, строка подчёркнута


Which I had, but it was removed and I did not know if it was removed because it caused an issue somewhere or not. I'm just going to assume it is not and push it back in when I have time.


Crazy it snuck by for so long.


Строки 118 и 129: плохо, что методы проверки состояния игры решают за всю программу о её завершении. Лучше, чтобы они возвращали `bool`, а дальше уже завершать игру с соответствующим сообщением. С подобным поведением проще добавить возможность начать новую игру без выхода из программы.


LGTM.


reack


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.311](http://teamcity.monogame.net/viewLog.html?buildId=8485&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:26:41


Is this equal to use Latching option?
http://wiki.ros.org/roscpp/Overview/Publishers%20and%20Subscribers#Publisher_Options


Строка 138. Вероятно, это происходит из-за инициализации пустым кортежем. Динамическая типизация Python не позволяет точно знать содержимое переменных в каждый момент времени. Скорее всего, PyCharm считает, что если ты используешь кортеж, его размер всегда будет одинаковым (что довольно логично). Думаю, есть смысл заполнить его какой-нибудь координатой по умолчанию (или получать её от пользователя)


Ah. That is what I get for not looking at the code. I assumed `FindByID` was just `FindBy<element>` instead of its own unique function!
:blush:


Here you should add the code that update the index.


The two snippets are identical. I'm pointing out the error in the code. The code is CLEARLY wrong, but only clang has an issue with it. This is why you must support and check multiple compilers across multiple platforms to get the highest quality C++ possible. Which is the entire point of the talk.


```ocaml
let sp = UpdateInd.string_person_of base (poi base np.key_index)
in
let p' = UpdateIndOk.effective_mod conf base sp
in
patch_person base np.key_index p';
```

This code is not necessary


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17086/testReport/) for this commit**


There's a little typo in `exapmle`


noeue


Here you need to do :
Update.update_misc_names_of_family base (get_sex p) {family = get_family p}
```ocaml

```


Строка 138. Занятно, если я в начале задаю кортеж не через (), а через `tuple()`, то всё хорошо


You can click the [pencil icon to edit a file and make a pull request](https://help.github.com/articles/editing-files-in-another-user-s-repository/). `suggests` is a weak relationship, users won't get those mods unless they explicitly ask for them. Dependency on `Kopernicus` is implicit by the dependency on `OuterPlanetsMod`, if you're removing that dependency then yeah, you can add a `depends` on `Kopernicus`.


@k-okadaさん
hrpsys-baseのサンプルディレクトリはsampleですが、インストール先ディレクトリ名がsamplesになってるのはなんででしたっけ？
install先のファイル（環境モデルのwrlなどもろもろ）を参照しようとして、時々タイポ的なかんじで間違えてしまうので、
どちらか一方に統一されてるとうれしいと思いました。


This kind of makes sense, considering the flow of the application, it is locked at `bcm2835_uart_interrupt` before it is unlocked.


Well, I think we might just want to use the fallback implementation. Given that .NET's apparently just checks against `Char.IsWhiteSpace`, I don't think we'll be much slower. And there can still be a `<platform>.Lib.trim` to expose the raw native behavior.


:+1: Pero movelo a otra rama y armá el PR (Pull request) a unstable.

Desde consola situado en el directorio del repo:
git fetch (para bajar la info del repo)
git checkout -b custom-npc unstable (creás la rama en base a unstable)

Hacés los cambios que tengas que hacer.

git gui -> Stage changes -> Agregás un mensaje que represente el commit -> Commit -> cerrás la GUI
De nuevo en consola, git push origin custom-npc (se sube a remoto, o sea, Github).

Listo.


This is a good point


@mbernat Fixed in 9a11607


I knew there would be a way to do this, I just didn't have it.


Потому что `()` -- это что-то вроде числа, т.е. объект с известными свойствами, а `tuple()` -- это вызов функции, а значит её значение определится во время исполнения, поэтому о свойствах поля, в котором сохраняется результат, никаких предположений не сделать


Just found some code duplication...


Yes, I merged `acq4/develop` into my `develop` branch before reporting this error.


Осталось исправить эти замечания к коду и задание будет полностью зачтено.


did you also update os-autoinst?


lgtm


you need GNU Radio 3.7.6 for this. I will adapt the CMakeLists.txt file to produce a better error message. Thanks


It took me a pretty long while to figure out what the old regex did too =)

But yeah, seems to be redundant now that the negative lookahead has it. I came to this form through quite a lot of iteration so that seems to be not needed anymore.


You'd want to `remove_const` here too


I suspect Jacques would be just as unhappy (i.e., just as happy) to have to choose between "fromProb pi" and "pi" - right Jacques?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.693.e1e6c6a7b8b05bdfc45320f4c6c0493825924d37](http://build.palaso.org/viewLog.html?buildId=58986&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 355, ignored: 3 Build time: 00:03:35


Probably better to add this to `BufferAttribute` constructor directly?

```javascript
THREE.BufferAttribute = function ( array, itemSize ) {

	this.uuid = THREE.Math.generateUUID();

	this.array = array;
	this.itemSize = itemSize;

	this.needsUpdate = false;

};
```


Before the update, you need to update different indexes.

```ocaml
delete_key base (sou base (get_surname p)) (sou base (get_first_name p)) (get_occ p);
            patch_key base (get_key_index p) (sou base np.first_name) (sou base np.surname) np.occ;
            let key = sou base (np.first_name) ^ " " ^ (sou base np.surname) in
            patch_name base key (get_key_index p);
```

But this is not enough ! Several individuals may share the same first_name and surname. So to make the difference, GeneWeb uses an occurrence (occ). So before you patch an individual, you need to be sure that
this first_name + surname + occ is not already taken. After that you can patch the person.


Yeah. I get it now. "safer" didn't refer to the code. I should have been there :)

Thanks!


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17088/testReport/) for this commit**


s/You're/You've


1 "F" in professional


thanks @myfreeweb . I was looking to generate cnonce as described here: http://stackoverflow.com/a/12795014/2623771 but couldn't figure out quickly what could be the value of 'Bytes' in the given code. Do you have any idea of what possible values could 'Bytes' have?


Hi!
I'm very interested in this patch and I wonder why the new added method `bySshWithEveryRetryWaitFor` is present in v0.9.3 but not used yet in `DockerCompulterLauncher` ?
Something like:
```
-            PortUtils.canConnect(host, port).withEveryRetryWaitFor(2, SECONDS);
+            PortUtils.canConnect(host, port).withRetries(3).bySshWithEveryRetryWaitFor(2, SECONDS);
```

Thanks in advance!


That's a good insight.

I was thinking about exporting the service vocabulary as a JSON file, just like I do it with the XML Schema, which you can access through /service_schema.xsd in the broker.

It would contain the properties, their types, documentation and enumerations (e.g. "firefox", "chrome" for the browsers).

I don't know, if I can finish it shortly. For now, just display some important properties, not more than ten.


it's a pretty safe assumption that nearly all calls to `calculate` will update the routes. So this is a premature optimisation.


sync with upstream


Excellent changes, @kaseybon.


lol, thx


Именно.


Do you mind renaming the methods to `get`, `delete` and `exist`?


I think it would be better to move the `if getattr(conn, 'auto_open', 1) == 0:` block inside the `try:` section, like this:

~~~
        # If this is a persistent connection, check if it got disconnected
        if conn and is_connection_dropped(conn):
            log.info("Resetting dropped connection: %s" % self.host)
            try:
                conn.close()
                if getattr(conn, 'auto_open', 1) == 0:
                    # This is a proxied connection that has been mutated by
                    # httplib._tunnel() and cannot be reused (since it would
                    # attempt to bypass the proxy)
                    conn = None
            except Exception as e:
                log.info("Closing connection failed: %s, force new connection" % str(e))
                conn = None
~~~

Otherwise there is a code path where conn = None is set before getattr is called. It does actually work (because of the default value), and will set it to None a second time, but I don't think it's very clear.


sorry, me derping trying to update my local fork using the gui interface. I've given up and gone command line like you real men.


Yeah, I really like of those merge fuckups :(


狗腿...


It's missing `[--term-(old|good|bad|new)]`


`Routes` should know nothing about `InterHost`. I suggest adding a `onChange` function to the constructor which we then invoke here.


太狗腿了，退回！


Here too.


Should the minions also have these ports opened?


But wouldn't a name change always go via the (ui) master? So in that reference the master name is always the current one?


Right, it does really make a huge difference.  Although if there are no special cases (i.e. just fromProb), that would likely simplify things.


I'm willing to bet your custom decoder does just as good a job or better than the `utf8Decode` combinator. I think the extra code here could be dropped. Or we can leave it and test/benchmark it later. Your call; they are both correct.


indentation?


I would propose putting the functionality "set selectedServices scope variable based on URL parameters" into the UI Router configuration, using the controller field of the state configuration, see: https://github.com/angular-ui/ui-router/wiki#controllers


32 . What is it that dogs have 13 , cats have 3 and humans have 4?
Ans.Blood types


Not sure if we should exit here. What if we reset a tooltip to null (then it should show no tooltip). I think we should just continue here.


No need to create a separate tooltip. From what I remember, you can just set the tooltip to a string object.


put this question


Et dieu créa le libre arbitre


Not according to the Salt docs http://docs.saltstack.com/en/latest/topics/tutorials/firewall.html


Haven't the --term-XXX options already been remove before by parse_name_args?


除了狗腿之外，其他沒有問題，好好反省...

接著把 show 文章完成囉


I don't understand lines 11 and 13. Why are you copying $stateParams.type to BOTH "serviceType" and "type"? Isn't one of them redundant?


Shoudn't this function be called parse_term_args()?


Could have sworn I did that ...
Pushed again.

On Mon, Jun 29, 2015 at 6:01 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> indentation?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/50ae99d4f40a78156bc9fe1cd28f1b63bfd100ee#commitcomment-11908987>
> .
>


Really?  In a non-fully connected network, e.g. a multi-datacenter case consisting of two fully-meshed clusters with limited connectivity between the two, isn't it perfectly possible that a gossipped topology change might not change the routes used by a particular peer?


@rovo89
Just FYI. There's an issue with getActiveXposedVersion replacement not working as it's a static method returning primitive type (int). Probably some optimizations in effect are preventing hook from working thus -1 is always returned. I think we had this in the past. Changing return type to Integer fixes this issue.


`scope.showAbilities = !scope.showAbilities;`


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17089/testReport/) for this commit**


32.What is it that dogs have 13 , cats have 3 and humaanns have 4?
-Blood types


Не воспроизводится. Поменял на
```python
if line[2] == 'CDS':
    ...
```
сравнил результаты исходного скрипта и его модификации, разницы нет.

Попробуй вывести перед сравнением `line[2]` и `line[2] == 'CDS'` и скажи, что выводится.


ack


Maybe it would be great to rescue only `MigrationError` error?


It looks like the synopsis of `git bisect start` in the doc and its usage string in the code are not changed yet.


`return assembledAvengers.hasOwnProperty(avenger.name);`


Ahhhh that makes a lot of sense. It doesn't help that I had been reading
"occ" as occupation.

I did try a very similar approach yesterday (copying and pasting the
delete_key code out of updateIndOk.ml) but didn't get very far.

On 29/06/15 13:23, ipfix wrote:
>
> Before the update, you need to update different indexes.
>
> delete_key base (sou base (get_surname p)) (sou base (get_first_name p)) (get_occ p);
>              patch_key base (get_key_index p) (sou base np.first_name) (sou base np.surname) np.occ;
>              let  key=  sou base (np.first_name)^  "  "  ^  (sou base np.surname)in
>              patch_name base key (get_key_index p);
>
> But this is not enough ! Several individuals may share the same
> first_name and surname. So to make the difference, GeneWeb uses an
> occurrence (occ). So before you patch an individual, you need to be
> sure that
> this first_name + surname + occ is not already taken. After that you
> can patch the person.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/patrickangusjohngrant/geneweb/commit/dbdf86f24c145a45434849c72a77232baac7c14e#commitcomment-11908874>.
>


This has already been done at line 42 (6 lines above).  It points out one of the objections that I have to handling configurations, and the editing of them as is done in the present methodology.

All test configurations need to be independent of the other tests.


Otherwise this looks good to me, thanks!


Thanks for implementing this!


LGTM


First: you're missing the semicolon after the entity reference. It should be " " and not "&nbsp".

Second: Using non-blocking spaces to put HTML elements apart is not the best way to do it. You could use CSS margins as an alternative.


@manisandro I've been trying to track down a regression on OSX which is causing attribute changes made via QgsOgrProvider::changeAttributeValues to get lost. It's only visible on OSX, and I've bisected it back to this commit. You can see the failure in the results of the zonalstatistics test, eg http://dash.orfeo-toolbox.org/testDetails.php?test=30531850&build=184378 (that's the first failure following the ogr pool commit).

I'll keep digging - but do you have any ideas what could be causing this?


Fixed by 98016e97143b8280e0b060b979f491239cd863cd, thanks.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17090/testReport/) for this commit**


Thanks!


oh no this is really ugly, you print the stuff and then capture stdout in order to reprint it again instead of just submitting the metainformation you want to print and print it in the right order in the first place!


I guess so.


Thank you @TeckniX I will update it accordingly with your comment.


This is great news : )


It describes that yaml should have attribute. So it should check for attribute existance.
maybe description should be better.


TeamCity xBIM :: XbimExchange :: Build [Build 361](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1956&buildTypeId=XBIM_XbimExchange_Build) is now running


Eran, you're right.  So having just done s/ASN.1Cert/TBSCertificate/, we're gonna have to change it again.  :-)
However, this commit is just for TRAC ticket 4.  There's a separate TRAC ticket (80) for re-introducing the Issuer Key Hash.


Key는 enum을 사뇽하는 것이 좋겠습니다.

초기화하는 부분은 다른 곳에 있을 것 같지만...

이 경우 map에서 get한 결과가 null이라면 null pointer가 발생합니다.

예외처리를 하여 기동 전에 알아차릴 수 있도록 하는 것이 깔끔하지 않을까요?


this is the kind of commit yuou don't want, you just introduce a feature and in the next commit you fix something in it. Why don't you squash those and make it right in the first place?


TeamCity xBIM :: XbimExchange :: Build [Build 361](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1956&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:16


Can you explain the purpose of `hawkular.scheduler.time-units`? I understand it determines the unit for tasks schedules, but I wonder why we couldn't simply use seconds, to avoid profusion of config props.


I'm still debugging it - The next commit doesn't fix it - circleci still
fails.
Still needs work :)

On Mon, Jun 29, 2015 at 6:13 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> this is the kind of commit yuou don't want, you just introduce a feature
> and in the next commit you fix something in it. Why don't you squash those
> and make it right in the first place?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/eb63c69a521f68c317e953cba7a05c6536f76cee#commitcomment-11909172>
> .
>


No: parse_name_args works on a copy of "$@".


Thank you, I've fixed this in commit f3a9e66


Simple typo in the comment, "the the"


Good point, I've fixed this in commit f3a9e66


Thank you, fixed in 9cfa404


Another little nit typo in the comment.  Not from you, but could you correct it?


Should this be:

```php
class OAuthAccessTokenException extends InstagramException { }
```


Bit 또는 animation에 대한 null check가 필요해 보입니다


Ajout de "satisfait ou remboursé" en dessous du dernier CTA
Passage de la deuxième bannière oblique de full img à img + texte
suppression des 20 px de padding right sur l'ensemble de la page
application du bon comportement hover sur le premier CTA


consider not adding the extra blank lines


These test work ONLY if, initially, the MOTHERBOARD is BOARD_RAMPS_13_EFB.

However, they also CHANGE all future tests to use the BOARD_RUMBA.

This points out a fundamental design flaw in using this methodology to change configurations.

An appropriate methodology would have each test run its configuration independently and NOT rely on the existence of any other tests.


@liamjbennett ,Thanks to reply me so quickly.

Why do you think to cover Windows, who care? In your forge module, compatibility operation systems are linux, no windows. I recommend to focus on linux system only.

Second, the forge module's `Dependencies` is missed this new archive module, you can update it.

Why I hate to include archive module is, I can use `archive` separately if I want. But now, you merge it into puppet-rundeck, I have to use it, and do the useless setting on puppet master, and update my `Puppetfile` as well.

I nearly spend 2 months on this puppet-rundeck module and try to make it run in our system, and still not sure if I understand it fully or not. If this module goes too complex, I may give up and replace by puppet-docker container solution.

This modules is already good enough, slow down for adding new features, make it simple and stabilised, write documents (especially on how to prepare the hiera file) will be helpful for everyone.

I do appreciate what you did.


SpriteBatch가 구체적으로 어떤 동작을 하는 것인지는 잘 모르겠지만...
전/후로 반드시 처리해야하는 작업이 있다면. 그것을 포함하여 abstract 클래스로 만들어서 처리하면 코드의 반복을 줄일 수 있을 것 같습니다

이곳 이외에 다시 사용될 가능성이 있다면. 관심사를 분리하는 것도 좋을 것 같습니다


Can you get my pull request for the upgrade to hasPermission() for group inheritance?


Suppose I sent counter data for a metric I haven't created upfront. What should I do to inform Metrics that I want rates to be computed for this metric?


I'm new at 3d printing, I have a WANHAO i3 and I can't seem to get the part to print in the center of the build plate. also cannot get the printer to print from my computer using usb cord. can anyone help?


Well, while it already has other stuff in it, it's a valid point.  And anyway if we could I really would rather have this in win32.c… but ok, maybe it's out of scope here.


# Configuration example
Now you can put multiple pools in the same config. When multiple pools for the same algo are specified, only the first one will be used. High frequency pool switching is not supported yet and I'm unsure on how to implement it so there's not much of a point.

    {
        "pools": [
            {
                "url": "stratum-eu1.hashlink.eu:13031",
                "user": "USER.WORKER",
                "pass": "abcd",
                "diffMultipliers": {
                    "stratum": 1,
                    "one": 256,
                    "share": 256
                },
                "algo": "fresh",
                "protocol": "stratum",
                "name": "FRESH-hashlink"
            },
            {
                "url": "http://mine4free.noip.me:3333",
                "user": "MYR_WALLET",
                "pass": "unused",
                "diffMultipliers": {
                    "stratum": 1,
                    "one": 1,
                    "share": 1
                },
                "algo": "grsmyr",
                "protocol": "stratum",
                "name": "MYR-grsmyr-m4f"
            },
            {
                "url": "mine4free.noip.me:5567",
                "user": "MYR_WALLET+0.0075",
                "pass": "unused",
                "diffMultipliers": {
                    "stratum": 256,
                    "one": 256,
                    "share": 256
                },
                "algo": "qubit",
                "protocol": "stratum",
                "name": "MYR-qubit-m4f"
            },
            {
                "url": "stratum.ftc.theblocksfactory.com:3333",
                "user": "USER.WORKER",
                "pass": "abcd",
                "diffMultipliers": {
                    "stratum": 1,
                    "one": 65536,
                    "share": 65536
                },
                "algo": "neoScrypt",
                "protocol": "stratum",
                "diffMode": "neoScrypt",
                "name": "FTC-TheBlocksFactory"
            }
        ],
        "algo": "neoScrypt",
        "driver": "OCL",
        "implParams": {
            "qubit": [
                {
                    "impl": "fiveSteps",
                    "linearIntensity": 768
                }
            ],
            "fresh": [
                {
                    "impl": "warm",
                    "linearIntensity": 512
                }
            ],
            "grsmyr": [
                {
                    "impl": "monolithic",
                    "linearIntensity": 768
                }
            ],
            "neoScrypt": [
                {
                    "impl": "smooth",
                    "linearIntensity": 56
                }
            ]
        }
    }

More interesting: you can now specify settings for all the algos in a single file. In theory you can specify multiple settings for each algo but I'm still thinking on how to match them so not much of point here either but convenience.


Optimization of package install process by `--no-install-recommends` parameter - to keep image small.


งง จังเบย


커밋 내용과 관련없는 변경은 별개의 이슈로 분리합시다.


@ken datepicker_control
I see my changes in 3b4616c were done with this commit


thanks!


I swear I don’t typically follow along this closely. :innocent:


Could you add Javadoc for this one?


Now where is that secret demo... :smile:


True, i was just focusing on getting the bug fixed. Forgot to look at surrounding code. Yours is neater.


> […]However, l18n doesn't work by default […]

Well, when I cross build on Linux, it definitely does work -- and `ENABLE_NLS` is definitely defined in *config.h* without anything specific.

Please take a look at config.log to see why the real check defining `ENABLE_NLS` would have failed.  IMO it'd be at most a missing build dependency, but if there's something more it'd be nice to know.


@juanignaciosl, remove this :dancer:


Você declarou uma variável key e passou uma string. Era isso mesmo?


맵 데이터를 비트 연산으로 다루는 것은 좋은 아이디어라고 생각합니다

하지만 이 부분은 단순히 던전에 종속되어 사용하기에는 관심사가 별개인 것 같습니다

MapTable등의 이름을 붙여서 별개의 클래스로 분리하면 재사용도 가능하고, 개념을 분리할 수 있을 것 같습니다


Read only `VOLUME` is used to sync container's `/etc/localtime` with host to avoid timezone difference.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17091/testReport/) for this commit**


Link supplied does not show fix?


Damit wird an die Stelle gleich die Postnote geschrieben, falls die nicht belegt ist, wird das Feld "pages" dorthin gesetzt.


@miguno Thank you for replying! v2.0.1 release was cut and not available when I tried it. Thank you for updating.


Спасибо за ответ.


>`gentrifying`

![](http://new1.fjcdn.com/thumbnails/comments/4404258+_7fb8735ce2cbb7f557e52783be8f236f.gif)


Update가 반드시 수행되어야 하는 것이라면 이벤트 리스너를 상속하여 해당 동작을 포함하는 abstract class를 만들면 코드의 중복을 줄일 수 있을 것 같습니다


ok, also fine ;)


YW :)


Seriously?


this looks good.

everything working?


@sagarlekar Thanks for the input :smile:

I just point out this, as I saw the commit


Looks good to me.


是被墙了，但是还是有漏网之鱼噢，secure子域名是可以正常访问的~


make (someone or their way of life) more genteel. "a gentrified Irish American"


함수의 의미가 불명확하다고 생각합니다

파라미터의 이름도 flg라는 것이 무엇을 의미하는지 여기만 봐서는 이해가 힘듭니다.

이름의 변경이 필요해 보입니다


Shouldn't variable names be unified between proto_l2tp_setup() and proto_l2tp_teardown() ($config vs $interface)?


see the comment I just posted. We may not always want to wait.


ack, thanks


indentation's hard huh? ack :D


ack


At this point it seems to be the only sane thing to do. We can do better by plumbing error channels through in that we can wait for just the fastest failure, but ultimately, we need to wait for everything in the success case.


Oh snap.


Yep seems to work fine!

What in the world are you doing up so early!? :P

Sent from my iPhone

> On Jun 29, 2015, at 8:58 AM, Bobbilee <notifications@github.com> wrote:
>
> this looks good.
>
> everything working?
>
> —
> Reply to this email directly or view it on GitHub.
>


Ah, looks like I just got gentrified


The handshake mechanism really shouldn't have to know about `Interhost`. I suggest we generalise this to just `Features` and `conn.Router.Features()`.


I think it's better if you just put a button there with the caption "Compare selected services". I'd put it on the right side. Also, I would put a hint somewhere, such as "Select services for comparison".


상수는 전부 대문자로 표기하는 것이 자바의 컨벤션입니다


Fixes #5


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17092/testReport/) for this commit**


>_< On it, thanks!


Actually, this already only waits for the single fastest failure. Plumbing channels would just be cleanup.


C'est normal car on peut considérer un plan de deux manières: soit ax + by + cz + d = 0 ou  ax + by + cz = d. Personnellement, la deuxième écriture me semble meilleure parce que, lorsqu'on demande le plan qui a pour normale (0, 1, 0) et à distance 1, on demande bien le plan Y = 1. Donc, comme j'exprime les plans au travers de cette convention, je dois ajouter un signe moins pour la composante d. J'ai l'impression d'être le seul à utiliser cette convention bizarrement.

Le facteur 2 devrait se retrouver dans la matrice de projection (cf. https://github.com/g-truc/glm/blob/master/glm/gtc/matrix_transform.inl l. 289) mais comme je suis un gland, je ne sais pas quelle autre matrice je dois changer en conséquence, donc c'est purement palliatif :)

Sinon, cela n'a rien à voir mais j'ai été étonné que les angles soient exprimés "main gauche" (cf. https://github.com/Gawaboumga/NazaraEngine/blob/UnitTest-Update/tests/Nazara/Math/Matrix4.cpp l. 94). Pourquoi pas ?


변수의 이름이 이상합니다.
왜 대문자여야 하고, N이 의미하는 바가 무엇인지 알기 어렵습니다


genterific


I'd much rather you override the whole method in API shims than modify core files directly.  Upgrading Rubinius is really hard with all of our one-off fixes.


nice one gents


This line should read:
```
return False
```


If we limited an identity to one credential per type (which we already do more or less, just not formally) we could have an ``addCredential(Object)`` and a ``removeCredential(Class<?> type)``, I guess.


ack


Please hide the service list in the detail and comparison view, e.g., using ng-show.


Yeah if we went to pure attributes this would look different I think.  I don't know how fine-grained we could get but at the least we could have a ``setAttributes(xxx)`` method.


@dbent thank you, this is exactly what I want. Most of the people come to know my mod because they already use OPM and / or RT. but sometime I get people asking to make my mods work even without OPM and/or RT. So I rather having it as a Suggestion rather than a dependency, otherwise all the work I do to make them Stock-Compatible would have been wasted time.
Thanks for the tips! I'm still new to Github, I hope I won't mess stuff up


@tchap So after all this talk about why `labels` are a pointer they're now not anymore? Are you fucking kidding me? :D :D :D


Opaaaaa acontece :+1:


Is that with the official version that I released yesterday or some custom build?


좀 더 객체지향적인 해결책이 있지 않을까요?

그리고 이것이 정말 인라인으로 처리해야 할 만큼 복잡도가 낮은 연산인가요?


my point was that we probably don't **want** to wait. The only part broken in the code was in the case when we waited (and that when we didn't wait no error was logged), and you fixed that, but now we always wait and I don't see why that's "sane".


Right, I just copied the technique 5 lines below :smile:
I'll fix it.


read much better
```represent the of the widest range```
some extra word


Hello. picodrive 1.91 version from Thunder Force 4 down screen Pending frequent symptoms ... Thunder Force 2 and Stage 3 are broken screen

Would version 1.91 and porting request if the gp2x f-300


위의 방식보다는 이쪽이 훨씬 좋다고 생각합니다


Or rather I'll let you fix since I do not know well the mentioned bootstrapping issue.


Ohh sorry, didn't notice that, too much JS coding lately :P I'll fix it.


ipython directive


1이라는 magic number는 상수로 분리하는 것이 좋다고 생각합니다


Diagram is good


Use a generic name for bootstrap, not a version. If we need to update the version, let's update it everywhere. Also, is this something that is only a bootcard dependency? If so, won't it slow down loading of other pages?


I don't see a reason to break the existing scripts. A 'scripts-after' seems fine, but no need to replace 'scripts' with 'scripts-before'.


why is caching off?


Good suggestion, would you be able to cook a patch?


Oh, damn. Just to make sure, is this master? In particular, is 100de728e2e3c5c7ba9531bf45599dda742d1f87 included?


Cool.  I probably missed the one 5 lines below then :-)


What happens if you run e.g. the PDQ controller without -d?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17093/testReport/) for this commit**


@tchap I'm probably blind but where is this `StoryRequest` type defined?


Sounds reasonable to me. In my mind anything `turn-on/off-mode` is just a legacy of the past.


Why are the tests commented out?


I don't know why, but if I click on the button I get an URL with two slashes in it, e.g., "/services**//**justcloud/compare_with/Microsoft ..."


@mbernat Well, I changed the client library so that there are two separate structs now. You should be happy now ;-)


I had the issue that search was no longer working for the shops. No results returned. So I've tried this fix and the search function is working again. Unfortunately the products are indexed for all shops. Meaning that product A from shop A is also returned in the results of shop B. I have added products for all stores, otherwise I cannot share customers and shopping carts in a multistore environment. While this has been working properly before. I've tried to restore the index per shop and per all shops. Same result. The ps_search_word table contains words that are indexed for all shops. E.g. the reference number is indexed for shop A and B. While it only should be indexed for the categories assigned to it. This is how it used to work. Somewhere in my setup/config something has changed in the database. Any suggestions?


It's the official, running on Nexus 5 stock LP 5.1


@pentaho-nbaker where do you think this is needed. I added this to the kettle's assembly. Are you trying to add it for report designer ?


@mbernat It's in the PT client library. Which is being developed by me, but it's not really a part of this project... But the change is here: https://github.com/salsita/go-pivotaltracker/commit/12a86b99a8db8abeff64ed3da6ed18f2228e786c


Removed psr-4 section because it is already handled via classmap. Having both sections results in composer to create warnings regarding ambiguous classes.
psr-4 will make sense once all classes have been renamed to use namespaces.
In that case it might also be a good idea to move all classes in the Classes folder...


@tchap Ah, okay. To be honest, I've got a bit lost in this thread. Let's just ditch it and pick up later, if necessary. At least I now have a bit better understanding how it all works, so it wasn't a complete waste of time :)


Almost done, but you have some merge junk left behind here.


:+1: for the timeout. It'd still be nice to identify the broken code that results in the ready file being missing.


This addition is dead code. select_fallback is true for all audio streams, and this whole condition is essentially for subtitle streams only.


Only streams of the same type are compared here.


Maybe it is a need because you're using c# travis. You should be able to go around this with:
```bash
alias pip='pip --user'
```


There's another one in vendor/lib/


Ah, I see. It seems fine to omit this commit.


It used in new-style provision logic in #234


My problem was that if the automation didn't finish, I had to leave the ui to get a progress report.  I think we should file a defect on the fact that the file stays open by the app.  The file writes in append mode should not collide.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17094/testReport/) for this commit**


@kristophM Consider having git ignore the `.idea/` directory, its cluttering up the commits a bit.


Hi,

https://github.com/PrestaShop/PrestaShop/commit/4c87e57e08ba543bbe3743e53b521ea8a8335ba4

Regards


@hydrangea8604
Cần xem xét lại những chỗ tính toán ngày tháng, tốt nhất là nên dùng thư viện xử lý ngày tháng (java mặc định có hỗ trợ class Calendar hay sao đó).
Nếu tính bằng tay này thì chắc chắn là không chính xác được vì chưa tính các trường hợp năm nhuận.


Yeah, it's what I did. Now it's all in the mode and it also tells the user what's the right thing to do if they try to activate the mode.


> but with custom build of xposed framework.

I cannot give support for that. But if it's your own build, check if something like https://github.com/rovo89/android_art/commit/d586da3e8940e6433acb86d6526a6ffbfc0b2c0d is included. Not sure what else has changed in 5.1, maybe the optimisation has to be disabled differently there.

In any case, XposedBridge and XposedInstaller shouldn't need to be changed. Please don't claim that they're buggy until proven otherwise.


Will there be anything that needs to be removed?


+1

On Sun, Jun 28, 2015, 20:12 Eric Niebler <notifications@github.com> wrote:

> "The value of expressions required to be equality preserving must persist
> absent any direct modification of the input objects"
>
> I say run with it. The committee can reword it, and that's ok as long as
> we have a paper that makes all the issues clear.
>
> Per your comment here
> <https://github.com/CaseyCarter/stl2/commit/189a6e8d342358f89530f05c6e3c430aa2779f2d#commitcomment-11874632>,
> we'll have Destructible and Constructible with no real semantics, the
> latter subsuming the former, since that's still a worthwhile change IMO. I
> personally think we can do without the rest, since MoveConstructible is
> just Constructible<T, uncvref_t<T>&&>, although that may be unwieldy
> enough to justify enshrining it as its own concept. No strong feelings.
>
> Movable is where basic "object-like" semantics (like address-ability) go,
> Copyable adds that copies are independent. Maybe the allocation
> requirements get moved to Movable and Copyable (breaking with N3351)?
> Some blanket wording about value of equality-preserving expressions being
> stable unless the algorithm explicitly modifies them (like suggested above)
> covers the rest.
>
> Since we have blanket wording that all expressions in the concepts chapter
> are equality preserving unless otherwise stated, we'll need to make an
> exception for Constructible so as not to over-constrain it.
>
> Sounding good?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CaseyCarter/stl2/commit/2d54e6b6275c9d1f97b3c5d899171ef53ef5dc04#commitcomment-11901431>
> .
>


You can also make them a `recommends` relationship, which installs them by default but allows the user to explicitly choose not to.


Yeah, it'll be slow, and I don't care enough to make a mass movement about it.

That said, this whitespace isn't really trailing anything.


IMHO not, it shows that this parameter is `self`, so when generate code, just write `&self`. Maybe with assert! to same type as Class.


hah, very cool, thanks


Yes, my fork is in sync with yours, so that commit is already there.


Good luck with this one commit. You may finish it up next year! There are about a million headers that require being added to Kbuild files, I added every file in include/linux to the Kbuild file, then it said went to saying I had to add asm headers next... so I said screw this commit!


:D awesome thanks for the contribution


This possible 'function'. Don't know what to do with it now.


I use this to create a temp file in system TEMP directory.


@dbent that sounds good, I'm still trying to figure out how to do it. The "edit" (pencil) icon says "you need to be on a branch to edit". I've forked this to my account (idk if that was the right move) but I can't figure out how to create a new branch


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17095/testReport/) for this commit**


do we still need to seek even if we truncate the file the successive line? I think that when you truncate the file then you are done and do not need to seek at the beginning of the file.

```
>>> import tempfile
>>> fp = tempfile.TemporaryFile()
>>> fp.write('ciaociao')
>>> print fp.read()

>>> fp.seek(0)
>>> print fp.read()
ciaociao
>>> fp.truncate(0)
>>> print fp.read()

>>> fp.seek(0)
>>> fp.seek(0)
>>> print fp.read()

>>> fp.write('ciaociao')
>>> print fp.read()

>>> fp.seek(0)
>>> print fp.read()
ciaociao
```


Custom styles should be allowed.


You can use "Checklist-model" to add items to an array based on the checked state of checkboxes: http://vitalets.github.io/checklist-model/


@tchap may I ask why the latest staging branch instead of, say, the latest release branch or latest stable branch?


Можно использовать `angularjs-rails-resource`


are we not about to delete the whole vhost?


probably warrants a docstring


Can you put that code in pastebin or something?
I don't know how to indent it

El lun., 29 jun. 2015 a las 13:47, Telegrambeans (<notifications@github.com>)
escribió:

> reply(img=urllib2.urlopen('http://
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> www.arenalsound.com
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> /
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> wp
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> -
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> content
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> /
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> uploads
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> /2015/06/
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> as_cartel5
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> .
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> jpg
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> ').
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> read
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >
> (
> <
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> >))
>
> Am 29.06.2015 13:43 schrieb "Shurkevin" <notifications@github.com>:
>
> > I'm writting this code at main.py and it gives me a syntax error
> > reply(img=urllib2.urlopen(
> >
> http://www.arenalsound.com/wp-content/uploads/2015/06/as_cartel5.jpg).read(
> > ))
> >
> > Which is the problem? I'm a noob at this, sorry.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/yukuku/telebot/commit/a702777b2bec2517cb4fcc6f94d9638cff734a5f#commitcomment-11908282
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yukuku/telebot/commit/a702777b2bec2517cb4fcc6f94d9638cff734a5f#commitcomment-11908323>
> .
>


@yummyZhou @yihui Git有点赞吗？亟需添加点赞功能~


I fear this produces the same linker error on `-O0` as we had with the hw kernel.


Лучше `bower_components ` добавлять в gitignore


I wonder if the file should be truncated here. It could be that you seek to 0 and then write something that is smaller than what you wrote before.


Let's say that you write an endless loop in the automation script. In that case no logging will ever be shown in the UI, not even with your solution imo. It will just end up in the BRPM log file so you need to ssh into the box. With the BRPM Content logging solution you can always see the logging real-time from the UI anyway when using the "log.html?request=123" url. The step-based log file is only used to have the logs show up in the Notes tab of the step.

Even if the append mode file write doesn't collide today, we can never know when and how the BRPM team may change this behaviour. Admittedly, the chances of this causing trouble later on are quite small.

> On 29 Jun 2015, at 15:20, Brady Byrd <notifications@github.com> wrote:
>
> My problem was that if the automation didn't finish, I had to leave the ui to get a progress report. I think we should file a defect on the fact that the file stays open by the app. The file writes in append mode should not collide.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/BMC-RLM/brpm_content/commit/f837b576d313f932662cc71dcfbc8244fdc90c89#commitcomment-11909795>.
>


TeamCity xBIM :: XbimScripting :: Build [Build 20](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1959&buildTypeId=XBIM_XbimScripting_Build) is now running


No wait, my mistake, it's status==200, I read ti wrong!


TeamCity xBIM :: XbimScripting :: Build [Build 20](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1959&buildTypeId=XBIM_XbimScripting_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:31


а как тогда будет работать это:   let (:account_owner) { create(:account_owner_user) }  ?


I thought I was going to have to do an inefficient channel until I remembered the trick for mutable Process1.

I suspect `utf8Decode` is a little faster, but only a benchmark would say.


As it's currently written, `getCommand` performs some complex transformations under the hood to ensure the proper execution, so that's definitely a method I don't want to touch any time soon. :smile:

We could have something along these lines as well (untested, typed on top of my mind):

```java
programs = [ 'makeglossaries', makeglossaries-lite' ];
foreach (program : programs) {
    if (isOnPath(program)) {
        return getCommand(program, <stuff here>);
    }
}
throwError('Nothing was found, help!');
```

Here we go. :tada:


ack


This view are the files as they exist at a particular point in time. You want to look at the latest versions on the `master` branch here:

- [SigmaModExpansions](https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/SigmaModExpansions.netkan)
- [SigmaOPMExpansion](https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/SigmaOPMExpansion.netkan)
- [SigmaRTExpansion](https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/SigmaRTExpansion.netkan)


I think this file is missing.


Indent this line twice, and the indent the next two lines once.


On 26/06/15 05:47 PM, Richard Eckart de Castilho wrote:
> You have to add a
>
> |<relativePath>../de.tudarmstadt.ukp.dkpro.wsd-asl</relativePath>
> |
>
> here, otherwise the module won't find its parent POM.

Thanks; that did the trick.


What is an AType?


sûr ??? Il n'a pas de controller dans le dossier edition


Yes, is this lib going to be deprecated? If so is there any replacement?


There is a type on the commit message (unused)


typo ;-)


Ok, I see. It looks like there is some duplication of code though.

Also maybe it should be:

```
 --term-good|--term-old|--term-bad|--term-new)
    shift 2;;
--term-good=*|--term-old=*|--term-bad=*|--term-new=*)
    shift;;
```


The space after the cast should be removed again :)


See 7c6fe9e05f153860afc6d32b895ae60bba56a208


I tried simply using `as SortedMap`, but `as` applied to maps is different from [the list case](http://docs.groovy-lang.org/latest/html/gapi/org/codehaus/groovy/runtime/DefaultGroovyMethods.html#asType(java.util.Collection, Class<T>)). It tries to use the keys as field names and values as field values, [see here](http://docs.groovy-lang.org/latest/html/gapi/org/codehaus/groovy/runtime/DefaultGroovyMethods.html#asType(java.util.Map, Class<T>)).

But yeah, seeing that we are going to do a lexicographic order on the keys anyway, maybe `DefaultModuleBundle` can just offer a constructor that accepts any old map and does the conversion.


public final class System
	extends java.lang.Object IT Technical Consultant
     // Constants
    public static final PrintStream err;
    public static final PrintStream out;

   // Static methods
  public static void array copy(Object src, int src_position,
	    Object dst, int dst_position, int length);
  public static long currentTimeMillis();
  public static void exit(int status);
  public status void gc();
  public status String getProperty(String key);
  public static int identityHashCode(Object x);
}


Beau gosse. :eyeglasses:


Nice!


What about using an environmental variable here?

```js
"clientSecret": process.env.CLIENT_SECRET
```


I'm thinking this didn't actually exist yet for this commit.


inliner simple_name dans cette méthode.


@mbernat Actually now I see a problem. It should try the release branch, and if that one does exist, it should try the staging branch. The point is to get the commits that are on trunk but not incorporated in the previous release. By previous release I mean any release, like being tested, staged or released. Since the released scenario is basically the same as the staging one, I should check for the release branch and then for the staging one. Will fix that.


@mbernat Actually this is ok as it is since this function is being used when starting a release, so the release branch never exists at that moment, but we should not count on that fact, so I will make it check for the release branch as well anyway...


atoi is specified as "not required to do error checking", which is bad for user input.


@dbent is this the right way to do it? #1725


I believe you can leave out unassigned variables. If not, then the Jekyll code should be fixed-up.


What Is This?


Thanx for merging this..was gonna submit PR...but no need.
http://forum.xda-developers.com/showpost.php?p=61555840&postcount=227


Yep!


thanks!


Looks weird, AbortException designed for not throwing stacktrace. If you want it, then just throw e and it will be printed to listener.


Looks good to me


I see two bugs in my screen but I cannot see yhe correct number, cause my vision is not too good.


Nope, It's a problem with the code generator cc: @silvolu / @ianbarber


It would be smoother, I agree.


Not sure how to better explain it, but the eclipse ctf parser when you would typedef struct would register the first field of the struct instead of the struct. This test IMO tests 1 thing, the typedef struct, but it validates it by forcing a lookup so it generates less false passes.


Specify the unit (byte, I guess...)


doesn't need to be hex, will fix, this was a copy paste issue.


Request for review from @sgjesse , @ricowind


I'm basically refactor things to look more like https://dev.openwrt.org/browser/trunk/package/network/services/ppp/files/ppp.sh#L144 . @nbd168 wrote both lines defining `config=$1` vs. `interface=$1` in `ppp.sh` and he usually knows what he is doing ;)


Changed it, and updated it to 1000 (I had set 100 for debugging purposes).


Dev purposes. It doesn't seem to be good at recognizing when a template has changed.

David

> On Jun 29, 2015, at 09:12, Drew Westphal <notifications@github.com> wrote:
>
> why is caching off?
>
> —
> Reply to this email directly or view it on GitHub.
>


Fixed das fehlende Leerzeichen zwischen Vorname und Namenspräfix


that was a bit confusing since absolute to me would have been ONLY trace.$STREAMNAME.,, blabla but stream.event is wrt the event itself. I have no issue changing this to absolute. Just wanted to explain the confusion.


Can be shortened / cleaned up:
```
names = name_output.map { |name| name.strip.split[1]}
```

Also if there are only two elements of the array you could use `name.strip.split.last`


Anpassen des Namenformates 'editor' an 'author'


in function name stay `childs` or better change to some `has_children`s synonim?


Lines 95-96 seem to be a leftover and not needed anymore
```{r}
env <- new.env(parent = args[[1]]$env)
env$. <- .data
```


Generally explicit returns are not added for single-line methods (although it's up to personal preference)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17098/testReport/) for this commit**


Thanks for merging master into your branch! This makes it easier for us to merge it back. Which I'm sure you're well aware of. In fact, that's probably the very reason you did it. But thanks anyway!


Ah. Cool.

On Mon, Jun 29, 2015 at 9:42 AM, divmazie <notifications@github.com> wrote:

> Dev purposes. It doesn't seem to be good at recognizing when a template
> has changed.
>
> David
>
> > On Jun 29, 2015, at 09:12, Drew Westphal <notifications@github.com>
> wrote:
> >
> > why is caching off?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drewwestphal/jococruise-theme/commit/f28c9f1ea80edb571ac52f6e3ad7ab63f0cc6c2e#commitcomment-11910284>
> .
>


Let's change 40px to reference the constant. It'll be harder for it to fall out of sync with the code (like here :D )


should be casper-nightmare-phantom


Similar to the above loop:

```ruby
emulators = output.drop(1).map { |line| line.split("\t").first }
```


what tab settings are you using in your editor?


high-level API does not expose it. We are working on it. Cray-style  vectorization does not cut it, everything must be via intrinsics (fixed aligned vectors, etc.) + need Rys for high AM.

Boys engine is already vectorized. It's memory bandwidth limited, so vectorization does not help much.


Can use the `upto` method to avoid manually tracking the index:

```ruby
`1.upto(20) do |attempts|
...
end
```


lgtm


'andothers' fügt 'u.a.' hinzu


Это более-менее ясное название, потому что команда `after` оставляет после себя `after_id` (Alarm identifier), принимаемый на вход `after_cancel()`


Hello,

@lunika It would be nice to add the tag [@since](http://manual.phpdoc.org/HTMLSmartyConverter/HandS/phpDocumentor/tutorial_tags.since.pkg.html) ?


Could you move the cast up here so that we don't have to force unwrap below?
Something like `guard let repositoryBody = body as? NSDictionary where response.statusCode == 204 else {`


did you create the custom social link and CSS for this checkpoint?


What does count do?


I'll check again


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.425.319ba4b496a43c3fa4108f3206b8461e5fdfd859](http://build.palaso.org/viewLog.html?buildId=58993&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3229, ignored: 119 Build time: 00:12:18


Vergiss diesen Merge. Sourcetree hat nur deinen Commit bei mir noch einmal mitgepushed.


Если делать такой разворот, то змея, идущая под 90 градусов сломается, если пытаться её развернуть
Такой ход запрещён, а в предыдущем случае всё было хорошо


Hmm nice job trying out the different scenarios. Don't you think it'd make sense to create an enum for it, like [here](https://github.com/czechboy0/XcodeServerSDK/blob/swift-2/XcodeServerSDK/XcodeServer.swift#L201).


@Wackerbarth
If we loose the track, we can just copy in the original Configuration.h.
Se line 59.

We already know this is not the perfect way to do the tests, or the configuration. But at least it is working for now.

Lets make our new RC, and than we can have a look on principal changes like your proposal to do it.


OK. As I said, I can't say anything about custom 5.1 builds. Maybe more changes are necessary there. Obviously, the issue doesn't occur for me on 5.0, otherwise I wouldn't have released it.

It's a bit strange though. Why would this check be successful: https://github.com/rovo89/XposedInstaller/blob/art/src/de/robv/android/xposed/installer/InstallerFragment.java#L173
But not the similar check in the welcome and module pages? If you can reproduce the issue, could you check the values at that place? I think the active version must be 65 there, else a different message would be shown...


Well what I'm saying is, are you gonna care that my IDE does it or should I turn it off? :P


Submit another pull request to Homebrew updating the version.


I'm in minnesota

Sent from my iPhone

> On Jun 29, 2015, at 8:03 AM, mslee017 <notifications@github.com> wrote:
>
> Yep seems to work fine!
>
> What in the world are you doing up so early!? :P
>
> Sent from my iPhone
>
> > On Jun 29, 2015, at 8:58 AM, Bobbilee <notifications@github.com> wrote:
> >
> > this looks good.
> >
> > everything working?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


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


@tchap ah, cool.