i am use in robot it worked well but dont work
tihis errore:
This application is temporarily over its serving quota
what happend???????????


Hello Michal,
thanks for your PR. It's a good idea to use the backports.ssl_match_hostname module. To keep the same the ldap3 package requirements between Python3 and Python2 I'd prefer to not add the requirements for the module in setup.ppre-y (even if only for python lower than 2.7.9), so I think a simpler solution could be to add:

```python
try:
    from backports.ssl_match_hostname import match_hostname, CertificateError
    return
except ImportError:
    pass
```

at the beginning of the utils.tls module, keeping the backported code (updated to 3.5) in the modules itself. This could not break any compatibiility in pre-existent ldap3 installations and would use the backports.ssl_match_hostname module if already installed. I could explain this in the docs.

What do you think of this solution?

Bye,
Giovanni


@ExSport I think that I made wrong description. Of course for transcoding the final output muxing is used by all encoders internally and for all output formats but I mentioned that possible remuxing by TsMuxeR is only used for the MPEG-TS output format :)


Ok. Ja, Samuel und ich haben beide  > 2.2 ... da gab es auch einen triftigen Grund dafür ... weiß aber nicht mehr welchen


Con esta modificacion evitamos que el directorio "user_guide" entre dentro del control de versiones ya que no tiene caso incluirlo.


Prefer `fetchFromGitHub`, or at least `fetchzip` over `fetchurl` when downloading archives on github. `fetchurl` is not deterministic.


I am looking forward to running this and studying the code.  Great work!! Congratulations!!


I just commented out where it was called, can be uncommented when there is no fatal error


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

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


Please refer to the constants that are defined in `stdenv.lib.platforms`.


This does not do what you want. `.getText()` returns the characters that were parsed to create this node--in this case it would return something like "`</HW>`".  Elements in the ise DOM do not contain their children in the usual sense; the DOM is actually more like a stream.  To get the child elements, you'll need to start at the `StartTag` and walk through the DOM until you get to the corresponding `EndTag`. You'll probably want to use a node stack to keep track (eg. like `NestingValidator` does).  Also keep in mind that only newlines in `TextNode`s matter; whitespace inside of tags etc is irrelevant.


This methods are for database access and donor account checking


Sounds good


@dlamblin I've found the relevant issue in our support channel (that's actually my team). I'll keep an eye on it.


#4 fully addressed now


Closes #67.


Closes #68.


It's actually possible to have a non-empty L node, in which case this would throw an Exception.


I probably should not have tagged this commit 1.3.1.3. since it has changes that belong in 1.4.
Should I release the whole thing as 1.4 or cherry pick these changes onto 1.3.1.2 or not release anything to hackage. The changes to lucid-svg,-0.5 will solve issue #32 about small floats not rendering.
Since this is basically a bug, I would like to get this change on hackage.
thoughts?


Thank you very much!!! :)!!!!


This is because in the original checkin contains tabs.


I mentioned this in the PR text, franlky I was a bit over-paranoid in maintaining previous functionality.  I have been tempted to file a PR which mainly includes cleanup of this script.  There is a dozen or so other places that things can be tidied, and a bit of cruft cut.

If you start an issue on their JIRA, I can contribute my line-notes and a cleanup PR.


For what i see that commit affects the axis, not what I changed. The old one would still not work for what I see after a quick reading. I dont have the very latest retroarch, so I cant test it either. It is still working for me.


For studying the code I will be soon updating the README file and uploading FULL documentation on it!! Stay tuned!!!


closes #32


I'm hoping this commit won't introduce problems or change much of normal runs at all. But it speeds up things like `../run/john -list=format-details` by a couple orders of magnitude (and that means things like TS and Johnny will be a lot faster). Now kernels are not compiled until just before auto-tune, in reset(). A few formats are unchanged, namely @Sayantan2048's and @claudioandre's ones that are "different". But I'd like them changed too if possible.


More notes about this here:
https://github.com/apache/spark/pull/6336


Great! I added those as well.


:bee:  / :+1:  -- were this not form-encoded data, US-ASCII (per RFC) would be spec, since it's form-encoded, UTF-8: [source](http://docs.oracle.com/javase/6/docs/api/java/net/URLDecoder.html#decode(java.lang.String)


u mad?


No worries :-) Great work so far!

Using bare parens to represent an empty list is deprecated; in a future
version of LFE attempting to do that will actually result in an error.

On Thu, Jul 9, 2015 at 1:56 AM, Eric Bailey <notifications@github.com>
wrote:

> Thanks for all these. I'll follow the m:f convention going forward.
> What's the preferred syntax for the empty list, '()?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lfex/xlfe/commit/2c58947d39924372a477e2c7059c03884b5775da#commitcomment-12077836>
> .
>


Nitpick over phrasing: "Injects the test entity's customised lifecycle tasks."


:hamburger:


I was reading the docs, but I think that connect() and disconnect() doesn't throws exceptions.


Was this on purpose? The formula above uses just $h$.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.743.2a77a8d45ee400877c99e4b548c922bdacae6807](http://build.palaso.org/viewLog.html?buildId=59536&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:06:07


Not sure, but isn't this a unnecessary duplicate?


I meant $h_f$.


Ah, but here it is $h_F$.


Hi Andrew, thanks for the review again.
I applied all your new comments. I'll merge with the latest, squash commits and open a new PR, so you have a clean slate.
Minor thing on PN_RETURN_IF_ERROR: This hides the return statement, making it hard to unwind state on the branch when the failure occurred. When we are going to have state unwind (I really hope for it), we'll have to either switch back to not using the macro or change the macro to - for example - goto a cleanup label of some sort. For now I'm OK with PN_RETURN_IF_ERROR.


There are two guides for LFE -- one a general programming best practices
guide and the other a style guide (inherited from the CL style guide). I
think they could both use a refresh, at this point (I don't think there's
any mention of how to type empty lists, e.g.):
 * http://docs.lfe.io/prog-rules/1.html
 * http://docs.lfe.io/style-guide/1.html

On Thu, Jul 9, 2015 at 2:02 AM, Eric Bailey <notifications@github.com>
wrote:

> It might be a good idea to fork the Clojure style guide
> <https://github.com/bbatsov/clojure-style-guide> or start one from
> scratch. Or maybe there's already one floating around.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lfex/xlfe/commit/2c58947d39924372a477e2c7059c03884b5775da#commitcomment-12077899>
> .
>


Will do!! :)


LGTM.


Good points, I'll do that.


If the first word can't be found using the standard dictionary, we need to switch to using an alternate dictionary instead (so the value of "useStandardDictionary" could currently change in the "if").


This assumes that the split-lines are encoded correctly (ie. that they have both 'i' and 'f' parts and no intervening lines that aren't 'm' parts).  If we don't check the structure first, this error message could be misleading.


The patch doesn't address the impurities in `gen-posix-names.sh`, but it does fix a build error on 10.10.4


Here you should either require the name to be known for both the numerator and the denominator, or accept unnamed records for both.


This is a markdown file. It's just a text file. Google markdown for more info.


second @rong4188


Psst. You typoed "need" in the commit message.


"Transaktions" => "Transactions"

Rather than "Share", you might consider the verbs "Export" or "Save" and the user is unlikely to want to share their transactions with others (more likely just making a backup for themselves).


I think the reason is that most mixin classes in projects using mixin is named `Mixinxxx`. And using "xx_01" like name is not suggested by the google code style.


Based on the problem from U of Washington http://practiceit.cs.washington.edu/list.jsp


`value_counts` on a series gives a slightly cleaner way of writing this.

`killings.race.value_counts()`


@MCGallaspy Is this still true, now that `import_all_child_modules` is pushing up the daisies?


Nice - good to remember `sharex=True`!


No need to name columns here, but it still works.
`majors[['Major', 'Unemployment_rate']].sort('Unemployment_rate', ascending = False).head(10)`


Really interesting optimization where you consider the calculated weights to decide how to move the candidateIndex. :shipit:


50 31337 M473!


#84 ウィンドウ名でハイフンの使用を止めることで、IE8,9で動作するようにしました。


You can also order the months, like this:

```
killings.insert(5, 'numerical_month',  killings['month'])
killings['numerical_month'].replace({'January': 1, 'February': 2, 'March': 3, 'April': 4, 'May': 5, 'June': 6}, inplace = True)
counts = killings.groupby('numerical_month').count().name
counts.index = ['January', 'February', 'March', 'April', 'May', 'June']
counts.plot(kind = 'bar', title = 'Deaths per Month')
```


特定のレコードを全て取得する際は、fetchAll()に、$selectを渡さなくてもOKです。
```PHP
$t_thread_tag = Bd_Orm_Main_ThreadTag::createTable();
$list = $t_thread_tag->fetchAll();
```

可読性を考えると微妙だけど、へたしたらこれでもOK
```PHP
$list = Bd_Orm_Main_ThreadTag::createTable()->fetchAll();
```

まぁ、上の方の書き方で良いと思います。


Nice Work!  Looks Great!


It runs property tests for all the laws for Traverse and its ancestors, Foldable and Functor. Here is the output when you run the test in sbt:
```
[info] ExpSpec
[info]
[info] traverse must satisfy
[info] + traverse.functor.invariantFunctor.identity
[info] + traverse.functor.invariantFunctor.composite
[info] + traverse.functor.identity
[info] + traverse.functor.composite
[info] + traverse.foldable.consistent left fold
[info] + traverse.foldable.consistent right fold
[info] + traverse.identity traverse
[info] + traverse.purity.option
[info] + traverse.purity.stream
[info] + traverse.sequential fusion
```
It’s part of Scalaz.


@EloiZ ,
I get this error after this clean up:
```python
Generation:
*** ValueError: ('total size of new array must be unchanged', (60, 2800), array([ 60,   1, 700]))
Apply node that caused the error: GpuReshape{3}(GpuAdvancedSubtensor1.0, MakeVector.0)
Toposort index: 36
Inputs types: [CudaNdarrayType(float32, matrix), TensorType(int64, vector)]
Inputs shapes: [(60, 2800), (3,)]
Inputs strides: [(2800, 1), (8,)]
Inputs values: ['not shown', array([ 60,   1, 700])]
Outputs clients: [[Shape_i{0}(GpuReshape{3}.0), GpuElemwise{Add}[(0, 0)](GpuReshape{3}.0, GpuDimShuffle{x,x,0}.0)]]
```
any idea?


@SubJunk yep, release away


Yeah, I realized after this that Arbitrary instances shouldn’t be defined in the tests, so I’ve been writing them and looking for them in the wrong place.


I've installed Linux a few times over the years, mostly on older equipment.  Finding drivers was always a problem.  On many occasions, I had to use a driver wrapper and get the windows file .. just as  though I would have had to for windows.. and even that did not work.  You can poke all the "windoze" fun you want, but all my spigot/craftbukkti servers work great under windows and were quick and easy to set up.  I think in the end it is just a preference.. and we all learn the tips tricks and hacks for our particular environments. I personally found that newer hardware (at least that __I__ use is rarely 100% supported by Linux and you have to do all the firewall configuring, etc, etc... all automatic on Windows, for better or for worse..

For all the complaining, My "windows bugs" have not be _that_ numerous.  Besides, I'm not going to be the only one that runs windows... and not everyone is Linux literate.. many servers are just hobby horses (like mine!)  Keeping your code windows friendly can only help you out.


I think you meant this:

```
majors.sort(columns='Median')[['Major', 'Median']].tail(10) # highest median
majors.sort(columns='P25th')[['Major', 'P25th']].tail(10) # highest 25th percentile
majors.sort(columns='P75th')[['Major', 'P75th']].tail(10) # highest 75th percentile
```


What exactly are you running?
Also can you try with the latest version of the code?
Tell me =)


> Harmless

But still annoying and presumably should have an issue for it?


You could add:
`plt.subplots_adjust(bottom=0.4)`
adjusting the bottom in order to show the full x-axis if you'd like.
http://matplotlib.org/api/pyplot_api.html#matplotlib.pyplot.subplots_adjust


no, it can be merged
all the needed changes already were done


この辺りの、WHERE句やGROUP BYをセットする記述をする時、下記の様に書くと綺麗にまとまります。
```PHP
$sb_thread->thread_tag->tag->innerJoinChain();
$sb_thread->thread_tag->tag
  ->addWhere('id',$this->param('tag'))
  ->thread_tag->tag->group('id');
```
このコードは、addWhere と group の２つだけだけど、
下記のように何度も条件を付ける場合だと、今の書き方だとさらにごちゃっとするので、覚えておくと良いです。
```PHP
$sb_thread->thread_tag->tag->innerJoinChain();
$sb_thread->thread_tag->tag->addWhere('id',$this->param('tag'));
$sb_thread->thread_tag->tag->addWhere('stat',$this->param('stat'));
$sb_thread->thread_tag->tag->addWhere('create_at',$this->param('date'));
$sb_thread->thread_tag->tag->addWhere('flag',$this->param('flag'));
$sb_thread->thread_tag->tag->group('id');
$sb_thread->thread_tag->tag->order('id DESC');
$sb_thread->thread_tag->tag->limit(10);
```
この方が綺麗。
```PHP
$sb_thread->thread_tag->tag->innerJoinChain();
$sb_thread->thread_tag->tag
  ->addWhere('id',$this->param('tag'));
  ->addWhere('stat',$this->param('stat'));
  ->addWhere('create_at',$this->param('date'));
  ->addWhere('flag',$this->param('flag'));
  ->group('id');
  ->order('id DESC');
  ->limit(10);
```


this line is so wrong


comment on a line


<scratches head>


nother comment on a line


97647908-DocIssues4615 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1163 of 1163 tests passed)

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


containerの階層の下にpage-containerを持ってくる


new commit added by me.


Yeah, that's the idea.


No, you’re right, this just uses depth. But are you sure the default is 100? At the end of this file, I do
`def expGen = Gen.resize(100, arbTerm(arbExp).arbitrary)` and that works, but if I don’t resize, then it explodes.

I don’t know how I could achieve that with `Term[F]`, since `F` is what knows how many branches there are, but `F` doesn’t know it’s recursive.

What’s happening here is that a node of F is given an Arbitrary of `size-1`, and when we get to `size <= 0` the generator always fails, so its parent can only generate non-recursive cases. It’s a bit hacky (using `Gen.resize` recursively and the failing generator, because we need it to work on an … arbitrary Arbitrary).


Oh, and this kind of test is what pointed out that `Ring[IdHandling]` was broken, so they already found a bug (but, granted, one in code we weren’t using anymore).


I don't follow Google code style, and the _01/_02 naming is so I can keep patches separate for better compatibility down the road.


Ok now New-VolumeShadowCopy returns the VSS service to its initial state after execution (apparently the Win32_ShadowCopy class' Create method will start the VSS service, but not stop it).

I added the -WhatIf and -Confirm parameters to Remove-VolumeShadowCopy, and made the DevicePath parameter mandatory.


@EloiZ , Actually in recent version, I cannot load parameters :)
```python
ERROR:root:missing values for parameters: set(['/recurrentstack/simplerecurrent_3.W', '/recurrentstack/simplerecurrent_1.W', '/recurrentstack/simplerecurrent_2.W', '/recurrentstack/simplerecurrent_0.W'])
```


Yeah, that works. I was mainly interested in feedback about the commented sections. Those are the areas where I'd like to put in data based on the current user, but I'm not sure how to do so.


Okay, I finally figured it out - this is due to your change where it now gives a default name to the `.tex` file, so I was just compiling the same old file again and again.  I would apologize for the noise, except I was silly and thinking that sample-article.xml would then have its name be sample-article.tex and sample-article.html - a nice naming convention.


Yeah, you can wrap all the logic that is related (which in this case seems to be the whole contents of your callback) in a well-named method to make it easier to read. It's also easier to spot bugs when you put non-trivial logic in a method, because your code reviewer can check your intention (i.e. the method name) against the actual implementation.


--> 주어진 [css 파일에 설정된 transition](https://github.com/helloheesu/2015-02-HTML5/blob/d69b778723c99dbf3533ced761f95e4efb79e8b8/asset/base.css#L197) 때문.
setInterval / requestAnimationFrame / jQuery fadeOut (아마 내부적으로 같은 메커니즘을 쓸 테니까) 어느 쪽이든, 일정한 시간 간격마다 opacity 를 바꿀텐데,
바꿀때마다 매번 새로 1초라는 긴 시간에 걸쳐서 opacity 가 바뀌므로 우리 눈에는 안 보이는 것.
css 에서 transition을 지우고 했으면 li 해도 됐겠지.


But name `Mixin...` is still used. olee just chose a not so good name without `Mixin` prefix. oops.


In general, it seems easy to randomly partition the remaining size among the recursive parameters, but again, can’t see how I could achieve that with `Term`.


nice :octocat:


Is Virtualbox essential ? Cannot we have kvm+ Qemu +libvirt ?


In the definition of transitive, is the reference to `AnyRef` a typo? The transitive requirement should be valid for instances of type `Any` as well, right?


See issue #5 . Support for libvirt is being added.


This is throwing warnings about the CLT being updated sometimes, which is strange. `700.0.57.2` is what `/usr/bin/clang --version` reports on 10.11 with the latest Xcode and CLT installed.


javascriptの扱いになれたら、``$.ajax({});``でリクエストを投げる部分は、submitボタンもresetボタンも、１つの処理を呼ぶようにすると、更に綺麗になるかも。
今は練習中だから、特に問題無いんで、直さなくても良いですよ。


jaja gracias gonza!! :+1:


forgot that I needed an index.html to start


Now EmailAuth Problem
- [ ] Some EconomyAPI does not sync problems.
- [ ] Inventory Hot Bar Doesn't match problems.


But will this now fail style checks? it's not guarded. I think it's better to still to save off the message along with other state in `ThreadingSuiteState` and log it outside.


Hey, why did you make this change? I called it appendBytes on prupose, since I felt appendData didn't capture the operation very well.


it's fine if the names of the variable in initializer list are the same as the member variable. see http://stackoverflow.com/questions/6185020/initializing-member-variables-using-the-same-name-for-constructor-arguments-as-f


Commit 9aa0915fc2077cce2d1efe127181d87dbc569ea7 的修改又提交了一遍= =
可以看一看是不是 revert 了，但是编辑器没有关上，之后又保存了之类的~


hm, it's weird. Seems to be working on my computer.
Are you sure that:
- you build a model with architecture layers=x and units=y and rnn_type=blabla
- you load the parameters from a dumped model where layers=x and units=y and rnn_type=blabla ...
I just mean that the arguments of the model should be the same of the one of your dumped model.


I think you're right. The original text was introduced here: https://github.com/scala/scala/commit/9638b5c79adb9aed20a84eab9a21730e9f697281#diff-5c0df4a652df400c788c25667290814aR89


sqlite_timeseries (mbadams5) - x86_64-MacOS-10.9-clang: OK (1744 of 1744 tests passed)

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


Soooo... Should we merge this? Does anybody want to fill out the timestamp (1478124553) and the total number of transactions?


Actually right now, elisa2 is so slow. I will try it again yesterday in the
lab.

On Wed, Jul 8, 2015 at 10:47 PM, EloiZ <notifications@github.com> wrote:

> hm, it's weird. Seems to be working on my computer.
> Are you sure that:
>
>    - you build a model with architecture layers=x and units=y and
>    rnn_type=blabla
>    - you load the parameters from a dumped model where layers=x and
>    units=y and rnn_type=blabla ... I just mean that the arguments of the model
>    should be the same of the one of your dumped model.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mohammadpz/RNN_Experiments/commit/384158723c678b0b0fa2f16ab2b7ce7c4edf5212#commitcomment-12075804>
> .
>


这个是出于什么考虑取消了？
或者说原来是出于什么考虑加上的 `colorlinks`?


Atualizacao


Someone could use your Yammer OAuth credentials to post on your yammer directly!


Ok, do you mind if I raise a pull request for this?


Ok, let's try tomorrow


@nnsnodnb
お願いします。
意図したカラム12超えならOKだと思います :+1:


说实话没什么道理加上。
或许是为了一致？不同 driver 的默认设置不全一样。


This seems to cause issues on 10.11 Beta 3. Compiling, opening Xamarin Studio or executing "mcs" from the terminal throws the "could not discover the mach TLS offset" error.


@KamranMackey this requires a separate PR.


@KamranMackey it would have been caught if reviewed.

I think previous merges potentially broke Vundle for a lot of people.


The code I just checked in is busted :(   It is not doing a DynamicFunc__append_keys


I see. I missed the part discussion.


rax retry tests


Instead of this method, we could add:
```java
/**
  * Ensures the truth of an expression involving one or more parameters to the calling method.
  */
public static void checkArgument(boolean expression, String message) {
    if (!expression) {
      throw new IllegalArgumentException(message);
    }
}
```

It's more generic and we can perfectly use:
```java!
Preconditions.checkArgument(!name.isEmpty(), "Table name can not be empty.")
```


I don't think so, but just in case I'll revert the #616 merge.


#31 & #32


NumberBox


> Would first cut with just strupr() strlwr() be adequate for now?

Sure, but I think we have good functions somewhere. I can tweak that if needed.

> Note, it is all or nothing in dynamic. md5($p.uc($p).$s) is not not legal in dynamic

As long as it bails out and don't chug along with false negatives I think that is totally OK.


You could use [`w-json`](https://github.com/IonicaBizau/node-w-json) here. :smile: You're very welcome! :sparkles:


what is Ember.A for?


Good job snuzz :)


Thanks @swatijadhav What happens if the job continually fails... does it perpetually retry? i.e. should we keep a count and kill it after 10 retries?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1167 of 1167 tests passed)

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


This breaks a normal build for me
```
...
Configure finished.  Now 'make clean && make -s' to compile.
DES_std.c: In function 'DES_std_set_key':
DES_std.c:639:17: warning: array subscript is above array bounds [-Warray-bounds]
   while (DES_key[i++]) k += 2;
                 ^
ar: creating archive aes.a
ptxas info    : 0 bytes gmem
Undefined symbols for architecture x86_64:
  "_sph_whirlpool", referenced from:
      _dynamic_fwrlpr in ccOlO9UB.o
      _dynamic_fwrlpc in ccOlO9UB.o
      _dynamic_fwrlp6 in ccOlO9UB.o
      _whirlpool_hex in ccOlO9UB.o
  "_sph_whirlpool_close", referenced from:
      _dynamic_fwrlpr in ccOlO9UB.o
      _dynamic_fwrlpc in ccOlO9UB.o
      _dynamic_fwrlp6 in ccOlO9UB.o
      _whirlpool_hex in ccOlO9UB.o
  "_sph_whirlpool_init", referenced from:
      _dynamic_fwrlpr in ccOlO9UB.o
      _dynamic_fwrlpc in ccOlO9UB.o
      _dynamic_fwrlp6 in ccOlO9UB.o
      _whirlpool_hex in ccOlO9UB.o
ld: symbol(s) not found for architecture x86_64
collect2: error: ld returned 1 exit status
make[1]: *** [../run/dynacomptest] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [default] Error 2
```


Nice Job!


This doesn't seem to agree with the above comment.


Thanks, I am 101% sure I tested it - but I'm not sure how it could ever work before aa17542.


The patch doesn't address the impurities in `gen-posix-names.sh`, but it does fix a build error on 10.10.4


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.744.98174a7a0b90b20a704201fa057a981b087952fb](http://build.palaso.org/viewLog.html?buildId=59537&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:03:52


I think that at least the base icon/splash resource must remain on the ```resources``` folder. Just like @felquis said, once committed these files rarely change and the base app a least have a better default icon and splash.


cc @fkling – this is a nice transition state in which we can use both parsers. This is based on the other commit that adds jscodeshift – do you have some time to add Parse support so we could merge this? :)


@rcefala I think they will be reached when you override those defaults in ```terraform.tfvars``` with empty string values, eg:

```
run_subdomain = ""
apps_subdomain = ""
```

Will test it tomorrow, since that is the way it should work, which I also put in the comments:

```
# If RUN_SUBDOMAIN is set, then use it's value to replace the default subdomain. Otherwise (if empty), don't use a subdomain
```


Great! Now let's hope my recent commit doesn't break it again :scream:


This should probably just go into the readme.


Awesome! :+1:


Neat! I need to bookmark this page!


Agreed. Although as you said though a unit test would be very useful.

I've started to look at how to do that in Openfire, but there don't really seem to be any good examples. Openfire seems to have a surprisingly small number of Unit tests (unless I'm looking in the wrong place!)

I'm playing around at the moment and using PowerMock to mock statics (XMPPServer.getInstance() ) for example so I can build full end to end Unit tests. Have you done anything like that or have any recommendation on that front - I know you use TestNG on Babbler. Do you have any experience or suggestions on which way to go for writing Unit tests for Openfire?


There are [security restrictions](http://docs.travis-ci.com/user/pull-requests/#Security-Restrictions-when-testing-Pull-Requests) for external pull requests in Travis so the Sauce Labs tests can only be run on pull requests from within the repo.

[Savage](https://github.com/twbs/savage) can work around this (video.js is using it currently) so we can't necessarily put this logic in videojs-automation.

Also this PR should fix all the build problems.


I agree @jjallaire on the use of global options. Not a big fan of it either. But this seemed to be the least intrusive way to do it. I will merge it in later today if there are no other comments.


Hi Nathan,

This is Jim Hester from Bioconductor.  I am going to do the review on GitHub to make it easy to have inline comments and link to lines.  It looks like 705d4d2042b60c1e18ed79c2312056f3805af522 roughly corresponds to 0.99.5 (which you uploaded to the issue tracker).

## Code formatting ##
1. We recommend using 4 spaces for indentation rather than tabs.  This ensures that your code is formatted as you intend regardless of other's tab width settings.  Most common editors have a setting to automatically use spaces for tabs, here is documentation for ([Rstudio](https://support.rstudio.com/hc/en-us/articles/200549016-Customizing-RStudio#editing), [emacs](http://www.emacswiki.org/emacs/NoTabs), [vim](http://vim.wikia.com/wiki/Converting_tabs_to_spaces), [Sublime](https://www.sublimetext.com/docs/2/indentation.html)).
2. I would also trim extra whitespace from the ends of your lines.  All of the above editors have an option to show visible whitespace which will make this issue readily apparent.
3. Extraneous semi-colons - R only requires semicolons when you have multiple statements on the same line, lines with only a single statement do not need a semicolon.  I would recommend you remove them in this case.
4. You have a number of lines which are very wide, e.g. [R/calcLocEnrichment.R#L108](https://github.com/sheffien/LOLA/blob/08607ed06e735cea7c6ea4c70ff366dbc61aebca/R/calcLocEnrichment.R#L108).  We recommend keeping lines to 80 characters or fewer.  Remember a single statement can span multiple lines if needed. i.e.
```r
  orderedCols = c("userSet", "dbSet", "collection", "pValueLog",
                  "logOddsRatio", "support", "rnkPV", "rnkLO", "rnkSup",
                  "maxRnk", "meanRnk", "b", "c", "d", "description",
                  "cellType", "tissue", "antibody", "treatment", "dataSource",
                  "filename")
```
5. You have commented-out code in a number of places, I would recommend either removing this entirely if it is now unneeded or moving it to the examples of the function if it would be useful for a user.

## Code Usage ##
1. [R/data.R#L24-30](https://github.com/sheffien/LOLA/blob/705d4d2042b60c1e18ed79c2312056f3805af522/R/data.R#L24-L30) Small thing, but I would put this code as an example in the data description.  Just add enclose them in a `\dontrun{}` block so they won't be run when using `example()` or `R CMD check`.
2.  Use of the range operator (`:`) should be done with caution.  For example [`1:length(testSetsGRL);`](https://github.com/sheffien/LOLA/blob/master/R/calcLocEnrichment.R#L36) would return `c(1, 0)` if the length of `testSetsGRL` is 0.  Instead use `seq_along(testSetsGRL)` which does the expected thing in this case.  [`1:nrow(annotationDT)`](https://github.com/sheffien/LOLA/blob/08607ed06e735cea7c6ea4c70ff366dbc61aebca/R/calcLocEnrichment.R#L27) likewise will have the same issue and is best replaced with `seq_len(NROW(annotationDT))` which will be robust to most inputs.
3. It is recommended to use only `message()`, `warning()` and `stop()` for status and error reporting.  These functions produce messages which can be easily suppressed by the user.  Please convert all `print()` and `cat()` calls with the appropriate function call.
4.  In [R/postEnrichment.R#L68-L86](https://github.com/sheffien/LOLA/blob/08607ed06e735cea7c6ea4c70ff366dbc61aebca/R/postEnrichment.R#L68-L86) you override the users setting of the `scipen` option.  We would prefer you to respect the users setting, but if you feel it is appropriate to override it temporarily a more robust solution is to use `on.exit()` to reset it, which will also reset the value properly in case of an error during function execution.  Also note that calls to options return the original values.  i.e.
```r
oldScipen = options(scipen = 4) #use scientific notation for pvalues.
on.exit(options(oldScipen), add = TRUE)
```
5. Consider using `rtracklayer::import.bed` to read a BED file into a GRangesList rather than your custom solution ([R/utility.R#L95-L147](https://github.com/sheffien/LOLA/blob/08607ed06e735cea7c6ea4c70ff366dbc61aebca/R/utility.R#L95-L147)),  this function is well tested and widely used throughout Bioconductor.

## Description ##
1. If possible I would suggest having as few packages attached in `Depends` as possible.  It is best to put the packages in `Imports` and import the needed functions into your package namespace rather than polluting the global namespace using `Depends`.

## Tests ##
1. It is great you are testing your code, great job!  However it looks like only (some of) the functions in `R/utility.R`, `R/loadRegionDB.R` and `R/calcLocEnrichment.R` are being supersized by your test code.  I would suggest trying to write tests for the functions in `R/postEnrichment.R`, `R/LOLA.R` and `R/splitFileIntoCollection.R` as well.

Overall I think this is a well written package and if the above comments are addressed I would be happy to accept it into Bioconductor.

I will be posting this same correspondence to the Bioconductor issue tracker so we have a record.

Thank you for your submission and patience with the review process!


What is the significance of 31.25ms and the 32000.0 magic number?


OprahSSL https://twitter.com/thegrugq/status/619151830975541248 haha


This file is in our .gitignore, but it keeps showing up when comparing branch commits. Honghao, it's better to use
```
git rm --cached database.yml
```
to eliminate this. Or in fact, use the defaults of username and password (your current unix username and password, I believe)


Np :-)


> Using bare parens to represent an empty list is deprecated; in a future
version of LFE attempting to do that will actually result in an error.

Ah, great to know!

> There are two guides for LFE

Cool, I'll check them out and maybe fork locally.


A new squashed PR will be great.

I agree the macro needs to change to handle cleanup actions, if the cleanup is simple (and it should be) it might be possible to just put it in the macro directly before the return happens - but we'll have to see how that turns out.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.745.57b194e0aa630fb6d8451fbbedbb5b443bade713](http://build.palaso.org/viewLog.html?buildId=59538&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:04:15


While tests are working, now this thing needs only some minor fixes... Ill look into this tomorrow.


@affo this is a really clever way to comment in the middle of a command! Kudos!


@Blumfield Why :hamburger: ?


@mottosso:

> But overall, this collector shouldn't be doing this. The published version shouldn't be determined from the working file, but rather from other published files.

This is what I was aiming for. It's collecting highest version from version folders in the published directory. So basically just figuring out what the previouslu highest version of a publish was for this asset.

I have this as a Collector so it's easy to replace the versioning implementation without having to recreate the full integrator. This also means other Plug-ins can acces this version information, if they need to.

> Actually, where is os.walk walking? How are you controlling the current working directory here?

Good catch. Consider this version branch experimental / pseudo-code. I had little time this morning and did a quick draft. Still wanted to share te concept I was going for to start discussion. ;) Anyway, it should walk the publish directory so it retrieves the highest previous version from previous publishes!

Also the regex pattern was out of the top of my head. Will have to double check that one as well. ;)


This is questionable.


You can just use `#load "../../packages/FSharp.Formatting/FSharp.Formatting.fsx"` which removes the need to explicitly reference all dependencies..


is this 31.25 microseconds?


:)

2015-07-09 22:57 GMT+03:00 gpyl <notifications@github.com>:

> Will do!! :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/npyl/paTriangle/commit/a5cf68eaa3dc5fc896b3b0d668461264ad2948bc#commitcomment-12092477>
> .
>


Indeed, it probably could be ported to non MFC classes easily, not much depends on MFC there.

But anyway, I think it would be better to center to work area instead of screen size.


So its ordering-sensitive?


Here's another comment??? Is it commenty?


See my comment on the similar code for HungWordValidator in issue_8.


:+1: (I like it, versus say scala's inference of "Any" in such cases)


Hey @Slava - is there some tool you are using to do this translation? Or is it all elbow grease?

Just curious because I'm trying to do some of my own as well.


Ok thanks. Does GitHub notify you every time I update this repository?

> On Jul 9, 2015, at 12:53 PM, Colin Clark <notifications@github.com> wrote:
>
> I've deprecated the name "simpleASR" in Flocking, and it will be removed in a future release. That unit generator has been renamed to "flock.ugen.asr" instead. Both names work, but better to use the new one if you can so your code won't break in the future.
>
> —
> Reply to this email directly or view it on GitHub.
>


Merci de observatie !


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


@RafaelC This really made me smile! So, you're not to edit the SimpleCalculatorTests.m file to make the tests pass, but rather the SimpleCalculator.m. Read this file to see what the functions need to do in order to pass, and then make those changes in the other file.

Should you need help, there are answers in the wiki - https://github.com/Bloc/ios-exercises/wiki


Also, the naming of this class would suggest that the HW code should be here too...


woot!!


Fixed in latest commit.


For my use case, ordering-sensitive is okay. The data structure itself is order sensitive, if someone wants a canonicalized representation they'd have to use a dictionary from variables to coefficients.


Or call ``merge_duplicates`` first before comparing.


sorry, I must hvae mistaken this yesterday. I meant to refer you to [welcome/index.html.erb](https://github.com/tlulu/ConNextor/blob/f35193c1f2dcb90e4f263d412398614537b0e718/app/views/welcome/index.html.erb#L11)
The line there adds an event listener to a custom turbolinks event. See [Turbolinks](https://github.com/rails/turbolinks#events)

Also the general styling we use is to name the anonymous function (for readability), and put it in the corresponding javascript file (for modularity). The welcome/index file should explain.

I also like to put the scripts on top, for better "warning" when reading. But I guess it doesn't matter. Each of these events aren't run when interpreted anyways, instead they are triggered by events.

Last thing: `type="text/javascript" was deprecated in HTML5 :)


Be careful of the order of the & operator precedence. This line is equivalent to GetBits () & (BIT_MARKED == BIT_MARKED), because == has more precedence compared to &


is there a reason not to do the same approach for both windows and all other platforms?


ok, thanks ;)


@postmodern Shouldn't this be `rm -rf` instead of `rm -f`?


@Optimusprimeums Oh okay, that's weird it would be inconsistent between releases. Thanks for the builds :) When you build do you checkout the tags? There's a report on the forum about one of the OS X builds not being a release and I'm not sure if that's a mistake I made or what


I'll merge this fix (and any others) into the previous patch before uploading to master.


When you remove the return from the Sass task in Fabricator it does fix the sass watch error, but it presents a new bug which is almost just as annoying if not more. The sass is now compiling synchronously as the watch task instead of being a dependent of the watch task. Therefore sometimes you have to save your sass file twice just to see the changes in the browser appear.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.433.f43da47aadfea29862134258ff4562a184ac7ad1](http://build.palaso.org/viewLog.html?buildId=59539&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3249, ignored: 119 Build time: 00:13:26


Oops, fixed.


I am pretty confident it works, but I only tested it with the detail maps I have, which aren't the best for checking it out... Let me know if it works!


93469234-DemandResponse2 (vidanovic) - x86_64-MacOS-10.9-clang: OK (1687 of 1687 tests passed)

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


Nice.


We might look to set `5000` as a constant in config/initializers/constants.rb, and call it using erb variable (which is also available in .js.erb files). But this is just about practice


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.408.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59532&buildTypeId=bt323) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 858, ignored: 54 Build time: 00:36:02
### Failed tests
```
LibChorus.Tests.dll: LibChorus.Tests.sync.LargeFileFilterTests.GetFilteredStatusForFilesHasExpectedResults: <no details avaliable>

```


This needs to check the _full_ stack, not just the parent.


93469234-DemandResponse2 (vidanovic) - Win64-Windows-7-VisualStudio-12: OK (1690 of 1693 tests passed)

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


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1170 of 1173 tests passed)

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


I would just omit the cve for now.


We seem to be using `.remove` as well. Do we need to log that here somehow as well?


Тестове завдання тут.


Any advantage to making it a static?


These configurePhase and buildPhase look very similar to the defaults. What is the purpose of adding them explicitely?


@Raesu DWMR. `return @"Press on!";`


If I generate chat messages between two users too quickly the monitoring plugin stores them out of order.

Agree, I hate doing this, but it's a test plugin (poor excuse I know!). I also am using a thread pool to help mitigate these kludgy pauses. Any suggestions though appreciated if there's a way to do this better :)


I changed the lexi, so that there is a parse error. Both $p and $u must either all be UC, all LC, or all left alone.  $p and $u are independent however

```
Dyna expression syntax error around this part of expression
SKEIN224(uc($p).$p)
                ^
Invalid token following the . character

Dyna expression syntax error around this part of expression
SKEIN224($u.$p.uc($p).$p)
               ^
Invalid token following the . character
```

I find this better than format ST failure.


I think `sexplib` should be propagated.


really????
undo last commit, delete, and push force


Uh, why the revert?


Great work here! Liked the way you worked on this.


That is interesting. @titzer could you file this as a separate bug? We could explore it, and I'd be OK for it to be in MVP if we can show it's useful, or leave for FutureFeatures if we don't get to it soon enough.


Is `-qr` here a typo?


Two problems:
 * this is only checking the parent. `<FONT size="1"><S><FONT size="1">...` is also redundant.
 * the `size` attribute only matters if it's on another `FONT` tag. Theoretically, this attribute could be used on other tags in the future.

Keep in mind that only the closest `FONT` ancestor matters. Eg.  `<FONT size="1">a<FONT size="2">b<FONT size="1">c` is okay.


This has nothing to do with your change.

I think this regex should have `[H,L]` changed to just `[HL]` since the former also matches a comma, no?


Same comments above for FONT apply here to FOREIGN.


Should this be for /validators or /validations?


Yo también llegué más o menos a la misma conclusión que vos, justamente por eso todavía no armé el pull request (no está terminado esto).

Sí, se podría pero el getType debería ser estático (actualmente es de instancia).


On windows, we can get the exact id using the process handle. On Unix, we don't the have native handle for the process, so i am verifying by getting the IDs of all processes with corerun name, and verifying if this process id is actually on the machine and is in the set of those process IDs.


Awesome, thanks:)


This doesn't conflict with the comparison operator overloading, right?


We've been using [`meteor-babel`](https://www.npmjs.com/package/meteor-babel) to transpile [tools code](https://github.com/meteor/meteor/tree/devel/tools), in order to evaluate different ES6+ features internally before we release them externally.

The interesting logic is [here](https://github.com/meteor/meteor/blob/devel/tools/main-transpile-wrapper.js). Note that this approach works only for server-side Node code, because it depends on the `require.extensions` API, so it probably won't work for client-side code. We have a different implementation (based on Meteor compiler plugins) planned for package and app code.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.746.57b194e0aa630fb6d8451fbbedbb5b443bade713](http://build.palaso.org/viewLog.html?buildId=59545&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:04:29


Yes, I'll update the library to support that too. thanks for the catch.

On Thu, Jul 9, 2015 at 1:19 PM, micwehrle <notifications@github.com> wrote:

> We seem to be using .remove as well. Do we need to log that here somehow
> as well?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Vungle/mongoose/commit/d0ecf2d63fe9120275cf50e7a280eecdc4cd0b61#commitcomment-12092897>
> .
>


Because you're never in Slack anymore.


fixed: 4fa9c90


This was style editing for readability and mostly bug fixes due to typographical errors or untested cases. I also added lines to try catch blocks if you want to run matlab in `-nojvm` mode where progress bars are not supported.


This whole method takes each user and iterates over all their roster entries. For each roster entry it generates 10 separate conversations, each with a new <thread> element.

I found out that though that <thread> is optional in the XMPP spec (grrr) and the monitoring plugin ignores it. Instead monitoring uses a MAX_TIME value to determine when an existing conversation between two users is expired and to create a new conversation . What this does method does is set the monitoring plugin's max time value to a really low value of 30ms and then pause for that long so monitoring creates a new conversation for the next iteration of this loop.

At the end of this method the max time value in monitoring is reset.


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Not unless someone was using ``addConstraint(m, isequal(2x, 1))``.


Thanks. Anyway this error was fixed on successive commit.


Per discussion on the phone, it's probably fine to use the copy routines for sendv/sendmsg since there's a number of descriptors  vs. host memcpy speed tradeoff to make.  But for prefix_send we should only use copy for FI_INJECT mode.

Non-prefix send can still do a copy, since again we're back to a 2 descriptors vs. host memcpy tradeoff.


Agreed. I just went with the convention used in the userCreation plugin. It uses System.out.println everywhere and tells the dev to check System.out in the readme.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.747.039eb197b665e4def1ac2e54faecc133f45a8fff](http://build.palaso.org/viewLog.html?buildId=59547&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:03:47


Thank's for tip. Am incercat sa fac si fara IF - uri.


should be: float | Decimal


You can refactor function to avoid code duplication


don't need float_to_decimal here because round_money always returns decimal


testGetLogsWithTimestamp ?


any special reason you removed the #define and let the 4 fall from heaven? :-)


Solved !


I will work on it.


who is cleaning up now? we only manage the external given ptrs in that class and someone else deletes them for us?


Why was this file in the working copy, and why was it added?


95574472_ip_measure (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


@mizzao hey, if you are asking about es5->es6 translation, I've done it manually here.

In the future, I think, we will think of running @benjamn's https://github.com/benjamn/recast tool.


Seems good to me. We only need to be worried about using it too much, gulp is incredible as it is because it's fast.


can't read /edx/app/edxapp/edx-platform/requirements/edx/post.txt


Fixed in a8627758958851313ccdc4cc45a29133cd966182


hunk not needed - introduced while debugging?


I think a better approach might be to keep the 16 levels of brightness, but create a percentage meter across the front of the glasses so the user can get some feedback on how high it is set.


@clintongormley Is this an error? Does the delete operation actually **create** an index if it doesn't exist? Doesn't seem to make sense.


Got the color palette idea from here: http://erskinedesign.com/blog/friendlier-colour-names-sass-maps/


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.748.c3141b1ace25b2b78e293d8079ad10f0b406ebaa](http://build.palaso.org/viewLog.html?buildId=59550&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:04:01


@nicatronTg I just was lol


@Daniel-Medeiros these direct links of AFH expire in a while, just give the normal link to page


Ok, I was going to fix this problem but you're already on it. Anyway, if I can give my opinion, I would do something like that:

```javascript
if (this._parentForm && typeof this._parentForm.fire === 'function') {
    this._parentForm.fire('iron-form-element-unregister', {target: this});
}
```

Because if the parent is not a Polymer element and it has a property called _fire_, the script will going to failed for the same reason.


Ah I see, this was probably somehow caused by a bug in some git implementation which failed to distinguish the `functors` and `functors ` directories. That's easy to guard against: Don't use directories with trailing spaces.


Hi,

I think the original line
```javascript
return fs.rmdir(dir);
```
was intended to pass promises to the `promise.all()` through the `sortedDirs.map()`. Your change (using the `extfs.isEmpty()` callback variant) would break this by passing an array of `undefined` to `promise.all()` (returning the promise from the callback has no effect here). Using `extfs.isEmptySync()`would fix that.

Daniel


if you leave out the call to float_to_decimal, your pullrequest should be okay. I did not have the time to test it in the real application, but this unittest works. (How do we add it to the code?)
```
for i in range(1000):
    # round up 0.005  -> 0.01
     assert AbstractShoppingBackend.round_money(Decimal("0.005") + Decimal("0.01") * i) == (i + 1) * Decimal("0.01")
```


Oof, yeah, though this isn't *so* bad. Maybe generic_works should belong to batches instead, and this code would change to `work = batch.generic_works.first`?


Ah, I didn't see that, that explains it.


93469234-DemandResponse2 (vidanovic) - Win64-Windows-7-VisualStudio-12: OK (1693 of 1693 tests passed)

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


Ah, I should have looked at the PR discussion! I've filed an issue for doing some general tidying up of the script: https://issues.apache.org/jira/browse/SPARK-8960

Perhaps the EC2 support would be better served by living in a separate project. Someone recently mentioned this idea to me. What do you think? Do you know if there is an issue in JIRA for that?


My bad - the "View" links took me to the files in "disconnected head" mode (instead of the testlink_1_9 branch), so I couldn't see the subsequent commits.


Change it to English.


Adding OpenLayers/Control/Button.js in dlf/plugins/pageview/class.tx_dlf_pageview.php helps.
It is required by ZoomIn.js, ZoomOut.js. This has to be fixed in git master.


hey not sure what correct assertion is b/c this is using chai assertion library. current repl's not working right even though its in production. i'm in touch with spangler and we're trying to get everything straigtened out.


I can confirm that this commit fixes 10.11 Beta 3 compatibility.


This line means -- this is Ember data property. You could do this with a plain ember attribute, let's discuss in slack.


And I wanted to say thanks for your changes! This little script needs a bit more love than it has been getting.


@jstedfast People using your magic -funroll-loops!


Yeah, it makes sense, that's what they provide the callbacks for.


Jeff Copperthite


Thank's for tip . Am incercat sa identez codul scris, si sa folosesc acoladele.


I need you to add your name to make sure that you have access


Lowercase "K"


The removal of _CRT_NON_CONFORMING_WCSTOK; causes the x86 build to fail.
Should this be added back?
:smile:


`node_id` is not a URL, so this is inaccurate.


That is true, we can avoid a few non useful procedure calls. It will be changed. Thank you for noting. I just did it that way out of sheer habit of using collections.


ok gotcha, this makes way more sense now.  changed some things and it seems
to be working on localhost.

On Thu, Jul 9, 2015 at 2:30 PM, Bobbilee <notifications@github.com> wrote:

> Ok so rethink this assignment.
>
> u are connecting the posts and summaries, so migrations are needed. (ie
> foreign key etc.) and ur summaries needs to be nested inside posts.
>
> all actions but the index action need to be filled out in summaries
> controller.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agoldfarb/bloccit/commit/e2a1f3e81d210082104f5ea64ed6341fc5a629da#commitcomment-12092054>
> .
>


no problems
thanks for your help

On Thu, Jul 9, 2015 at 10:41 PM, Quietust <notifications@github.com> wrote:

> My bad - the "View" links took me to the files in "disconnected head" mode
> (instead of the testlink_1_9 branch), so I couldn't see the subsequent
> commits.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TestLinkOpenSourceTRMS/testlink-code/commit/29dbd5162da392f2651003542baa3c5947c5b56b#commitcomment-12093273>
> .
>


:ship:


Right, I missed that.


```
$ ../run/john -test -form:'dynamic=md5(md5_raw($p))'
Benchmarking: dynamic=md5(md5_raw($p)) [128/128 AVX 4x3]... DONE
Many salts:	9928K c/s real, 10027K c/s virtual
Only one salt:	9954K c/s real, 9855K c/s virtual

$ ../run/john -test -form:'dynamic=md5(md5($p))'
Benchmarking: dynamic=md5(md5($p)) [128/128 AVX 4x3]... DONE
Many salts:	9885K c/s real, 9984K c/s virtual
Only one salt:	9984K c/s real, 9885K c/s virtual

$ ../run/john -test -form:'dynamic=md5(uc(md5($p)))'
Dyna expression syntax error around this part of expression
md5(uc(md5($p)))
    ^
Invalid token following a ( character
```
I assume a non-raw H() will output lower-case hex (this should be stated in docs btw) so I just tried if I could change that to upper-case hex. I'm *not* suggesting you should implement the above until we see need for it (if ever). It's good it failed gracefully.


Use the singular `NodeLog`.


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

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


Oh, sorry, I think I missed the point of the question! And also missed a chance to plug my tool… thanks @Slava!


This doesn't belong; a `NodeLog` does not have a `fullname`. Perhaps add a either a link back to the log's user (if applicable) or a nested representation of the user. @brianjgeiger might have thoughts on this.


As mentioned in a previous comment, it might make sense to either link or nest the log's user.


Moved public members first, as those are most important for readers of a class. Makes sense?


#define is used only here. I don see any reason to keep it.


I made all tests of [gold-email-input passed](https://github.com/PolymerElements/gold-email-input) with this change.


The friend function doesn't need to be public; the operator<< doesn't even need to be a member function -> reduces interface of the class.


Hello,
this line should be removed. It's killing a lot of stuff in every theme.
The typoscript condition is useless.

Have a nice day.


I wasn't sure about this change.... eslint was warning...


These includes seemed weird to me: this would force any client code to compile all expression/statement types even if that code wouldn't need them, so I removed them.


Another slightly bigger change here


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Checked !


I'm not sure if the return statement is necessary.


"_this" seems like a subpar name for an argument; replaced with what it actually is -> statement


So you would prefer I use `GetMonitorInfo` on current monitor?


I found a problem with ```np.clip``` in certain situations. We want [low, high), and ```np.clip``` returns [low, high]. I fixed it (in my version) by adding a tolerance:
```
np.clip(exp_val + self.low, self.low, self.high - 1e-5)
```


yep. the Effects11 manages these ptrs. any AddRef and Release calls on do nothing.


commit message reference the wrong Issue. This spec is related to issue #594


If there was data for prior years, you would have counted those too.
`len(killings[killings.year == 2015])` returns an int for just 2015, not a tuple for the whole df.


Que monito!!! Flux 2.0 va quedando a punto!


The removal of **_CRT_NON_CONFORMING_WCSTOK;** causes the x86 build to fail.
Should this be added back?
:smile:


Yep, something like that. It also make sense to show window on the same as mpc-hc was before crash instead of primary.


Why isn't this file named with a `.py` extension?


Even though not strictly necessary, these comments on closing braces ease reading of code living in namespaces. This is taken from the Google style guide.


Use: `killings.gender.value_counts()`


If I omit the CVE the CLI test don't pass. I could just have it blank if you think it'd be better


Typo


I don't like this bit at all... but I can't think of a nice way to work around it.  Basically `createView`, `DOM`, and `Router` are excluded from webpack.

If we introduce a require statement here, then browserify bundles them in.  Thinking out loud, perhaps build.js could be configured to have React and React-Router as globals.


@smbarbour Please try the newest build of forge. This should be fixed now. I didn't see the derp until today..


Thank you

On Thursday, July 9, 2015, Liam Foley <notifications@github.com> wrote:

> If there was data for prior years, you would have counted those too.
> len(killings[killings.year == 2015]) returns an int for just 2015, not a
> tuple for the whole df.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Samrdreyfuss/SF_DAT_15_WORK/commit/415270b0f7dfe4949f466a76dd5708e926c756be#commitcomment-12093519>
> .
>


it's not used at all and cannot be accessed. this is cosmetics


See @mkrueger? This diff demonstrates why I filed https://bugzilla.xamarin.com/show_bug.cgi?id=3001 ;)


95574472_ip_measure (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I can't see a rendered difference with or without this PR.

The rendering changes depending on whether or not I click the Compatibility Mode Button (see screenshots) but the output is the same before the PR is applied as after it is applied, I don't see any difference.

Are you seeing differences?  Can you provide screenshots?

For reference, I'm using IE8 on Windows 7.

## Compatibility Mode Off (both
![image](https://cloud.githubusercontent.com/assets/5194588/8587266/2311bf24-25c7-11e5-858c-eb125d71804a.png)

## Compatibility Mode On
![image](https://cloud.githubusercontent.com/assets/5194588/8587276/33fae0f4-25c7-11e5-9dd5-3f9df6de614b.png)


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Moved further down beyond the class definition so the reader doesn't have to parse this typedef, especially since it's not used in this file/class anywhere.


Other than 9-12 on Majors, Nice Work!  Let me know if you want to go over those on Monday.


It's not just unrolling - the if is reduced to 1/4 :)


Well I'm not sure what DrDump does but I will make our dialog show on the same monitor than the progress dialog (which is probably on the same monitor than MPC-HC).

Speaking of dialog, any better with the top-most thing?


do it!


iOS mesajlasma ekrani css i (denenecek)


uh oh. nice!


Has nothing to do with that - I formatted the whole file therefroe wrong bug for the diff.


iOS mesajlasma ekrani cssi( denenip, duzenlenecek)


Nope, I just had your repo open in a tab and noticed that you'd added awesome new changes.


Fixed. Use IWaila to add


93469234-DemandResponse2 (vidanovic) - i386-Windows-7-VisualStudio-12: OK (1693 of 1693 tests passed)

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


iOS mesajlasma ekrani ( denenip, duzenlenecek)


Are you sure that running `cmd` won't just spawn another wine process?


95574472_ip_measure (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Kisi listesi duzenli olarak cagirildi


Do you prefer an order of includes?
In general, a defined order makes parsing for the reader a lot easier when there are a lot of includes.
For instance:
1) C system includes
2) C++ system includes
3) 3rd party includes
4) own includes
Each of them sorted alphabetically, layed out block-wise.


Check out the `revision` numbers on these bottles.

` update 3.4.3_1 bottle. `

` revision 13 `

CC @Homebrew/owners


 Asterisk.class.php
 Classe para gera eventos para o asterisk.
 Segue o legado do phpagi e asmanage.php, foi adapatado para usar apena uma classe.
 O metodo construtor gera conexão com banco mysql e ami via socket.


Modal kapatma da android de sorun oldugu icin preventDefault eklenerek denendi. Hala sorun var


I suggested to check the type of _fire_ (like in the PR #10)  because if the parent have a property called like this the script will continue to failed.


Even though "statement" might be rather intuitive, each class should have a comment that eases understanding of what you're currently looking at.


close #61


Sounds good - thank you

On Thursday, July 9, 2015, Liam Foley <notifications@github.com> wrote:

> Other than 9-12 on Majors, Nice Work! Let me know if you want to go over
> those on Monday.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Samrdreyfuss/SF_DAT_15_WORK/commit/415270b0f7dfe4949f466a76dd5708e926c756be#commitcomment-12093593>
> .
>


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1693 of 1693 tests passed)

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


It's meant to be launched as a Linux executable, so I thought it's better to name it without the .py, since it isn't a module.


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


What is the meaning of EXPORT here? I encounter the error
> plugin.cc:17:1: error: ‘EXPORT’ does not name a type

building this. If I remove EXPORT everything goes fine. Replacing it with lowercase "export" doesn't work either.

Is it supposed to be a #define somewhere that I'm missing?


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


These will, among billions of other failures, change `--dev=0` to `v=0` which is not quite what the customer ordered. I'll just revert it locally but this needs to be totally re-worked.


A free standing, non-member method to_string() taking a single argument is as self-explanatory as it gets, thus, no commenting required.

operator<< on the other hand is way less intuitive. What does it mean to stream a statement into an output stream? Thus, this requires a short comment.


Both:
http://stackoverflow.com/questions/6004070/execute-shell-commands-from-program-running-in-wine
and what I tested show that it launches a Linux process. I ran this command in wine 1.6 and it properly launches whatever it's given. If anything could be worring here, it's whether the FAFpath is correct.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.749.1834420bf03edb8544dff509e35ffa5d037bdab7](http://build.palaso.org/viewLog.html?buildId=59555&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:07:07


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Hm okay. If you think it's not worth considering any more, do tell. I think the path separator patch would be suitable for mksh itself, too, unless you do end finding a different way that's not too complex/backwards/bad/ugly. The directory separator part though… I prefer to rewrite all \ to / (yes, I saw you already did that) and not have commit 759b6ec merged. (We'll see about drive letters depending on the : vs ; issue, too.)


I had another guess, but I have no elements to pick.
Rationale for the extra directory: Cai added the redirect from ` functors` for people pasting a URL from a certain PDF :wink: . But I think we can drop that, because the "certain PDF" :wink :stuck_out_tongue_winking_eye:  is not so critical any more.


OK, I'll take care of that.


I work with Javascript more than Python, but this appears to be cycling through the different layer sets and then kicking out the 4c layer. Would the deprocess be able to be put in the deepdream loop to kick out all the subversions of the layering?


Hey @NeilFraser,
Just in case you haven't noticed, this commit as reverted some updates from the translations. Still experiencing issues with translatewiki?


use `ATOMIC_INIT(0)` instead of `{0}`


For #2332.  Change this to happen on the MyData page init().
e.g. total counts are calculated a single time


Heh, had no idea this was an existing issue of sorts. Funky. Thanks for the link!


and: https://github.com/Homebrew/homebrew/pull/38628 -- looks like it wasn't reverted?


It's easier to ask "why would I want to see `$metadata$`"? I don't know if `constructor.$metadata$` is just internally accessible to this lib, or if it is noticed on objects in the application. I assumed the latter case and that's why I hid `$metadata$`.


Any idea where this code was taken from? Hoping to use this as a library. :) Is it licensed part of the go-qml project?


I did some testing
1. MPC-HC binary is relinked even though only crash reporter dialog files were modified. Minor issue, though would be nice to have to separated.
2. `SetWindowPos(m_hWnd, HWND_TOPMOST...` doesn't work. Window is probably activated before this and it only work when activating. I did some testing and this works. It will create topmost dialog.
``` diff
diff --git a/src/mpc-hc/mpc-hc.rc b/src/mpc-hc/mpc-hc.rc
index 1e537c6..6e26a52 100644
--- a/src/mpc-hc/mpc-hc.rc
+++ b/src/mpc-hc/mpc-hc.rc
@@ -1106,6 +1106,7 @@ END

 IDD_CRASH_REPORTER DIALOGEX 0, 0, 300, 240
 STYLE DS_SETFONT | DS_MODALFRAME | DS_FIXEDSYS | WS_POPUP | WS_CAPTION | WS_SYSMENU
+EXSTYLE WS_EX_TOPMOST
 CAPTION "Crash reporter"
 FONT 8, "MS Shell Dlg", 400, 0, 0x1
 BEGIN
```
still DrDump progress window and various messages like .pdb one are not ontop obviously, but our dialog is enough I think.

3. (not related to DrDump) We have a bug when ontop is set to always. Going to fix this right away.
4. Crash dialog looks fine :+1: One suggestion, maybe add an option to restart mpc-hc same as firefox has in thier crash dialog?


what are you going to do with the previous incarnation of the thread pool?


@txdv ,

It so happens that all commits are automatically unit tested. You are right about the bug when `Concat` is called with a list of comma separated strings/objects, for example:

`string.Concat("1", "2", "3", "4", "5", "6", "7", "8");`

Nevertheless, your commit fails the unit tests: https://ci.appveyor.com/project/Leonid/bridge/build/1.5.29-master. Actually, here are the tests and their results:
```
s = string.Concat(new string[] { "1", "2", "3", "4", "5" });
assert.DeepEqual(s, "12345", "string.Concat()");

// Expected: "12345"
// Result: 	"1,2,3,4,5"

s = string.Concat(new object[] { 1, null, 2, null, 3 });
assert.DeepEqual(s, "123", "string.Concat()");

// Expected: "123"
// Result: 	"1,,2,,3"
```

Please note, that the 2nd case has one more bug since in C#, `Concat` "ignores" `null` values.

Also, I noticed that you were too quick to close your pull request so I'll just post my thoughts here:

The bug is here:
```
[Template("Bridge.String.concatArray({strings})")]
public static string Concat(params string[] strings)
```

and here:
```
[Template("Bridge.String.concatArray({objects})")]
public static string Concat(params object[] objects)
```

How Saltarelle does it? Here's the answer:
```
[InlineCode("{strings}.join('')")]
public static string Concat(params string[] strings)
```

The elegant Saltarelle solution can be applied to all other cases just by changing the attribute name from `InlineCode` to `Template`. It cannot be used in these two cases because `Template` does not handle parameters the same way `InlineCode` does. For example, the following:
```
[Template("{strings}.join('')")]
public static string Concat(params string[] strings)
```

produces: `"1","2","3","4","5","6","7","8".join('');`

when called with the list of comma separated parameters of your example.


no exception stack trace?


new about GUI


Don't you want to complete this._db here?


Aye, still there. Looks like Mike pushed a change in https://github.com/Homebrew/homebrew/commit/fc3a30dbbfe209d1aed493723092273683ca3598, and then Jack added and reverted a change, but the block change itself was never undone.


xD


No, but personally I prefer to make methods static, if they can be made static. There's a myth about better performance, but I don't know if it's true.


We should add a lint test for this :)


Look into doing this as a .map() or other Observable operation on this._db, instead of creating a new Observable:

return this._db.map(...).


Maybe the out of order comes from the 8 concurrent threads?


This is needed, because this is caused every boot. I mean it won't happen in the future because you'll have to authorize for it with the new Accounts GUI, whereas the current build automatically authorizes it.


Exellent


Please use db names like specified:
gps_lat
gps_lon


are you sure there is always going to be at least one element in `m_pendingOffsets`? first with throw is if there is no element in the set. If you are not sure you may want to consider using `pollFirst()` instead


I think this should be stored in parsed format, i.e. as sd_id128_t, rather than as string.

Also, spurious newline


this should really take an sd_id128_t i think or NULL.


What types of errors can happen here? Should all of them map to null?


We don't place {} for single-line if blocks.


Use a constant for 'readwrite'


spelling : DBi_bts


Huh? Why did you change this?
Many devices doesn't properly support Neighboring cells from API (Samsungs)


All right, granted :)


The store name should probably be configurable.


Yes, Openfire has very few tests. It surprised me as well. And it's a bad situation, because you never if you break something.

I've never really worked with PowerMock and/or Mockito. Simple unit tests with Junit or TestNG were always sufficient for me (but you can probably write more powerful tests when mocking).


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.404.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59558&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:21:49


Only available on LTE on laterst APIs like ~19


since this is for the agent should this be installed in the agent? also if we leave it in here we should have a default for the virtualenv_dir


This is not bearing it is: Bit Error Rate


Is there a way to write this in a non-nested form?


seems like this var declaration can be added as another line to the group above


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.434.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59562&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:06:00


@geemus would love a release that contains this! We're hitting it consistently in [Bookbinder](/pivotal-cf/bookbinder).


We're not ...yet.


It looks like I accidentally rebased an earlier version of `previousRun` into a previous commit. This is the first version of `previousRun` which works correctly.


Aha, found it here: https://github.com/peterhellberg/wavefront


forgot to remove the memory and cachesize variables from ApplicationModule. Not needed.


Should we have a separate error class for the full AST parser?


Removed dynamically alter styling, instead, use css class.


ftr, we use 2-spaces indentation pattern.


I know that eva thats why I put todo there should I put 0 for default


The RefCount object is only something to use if there are multiple threads involved when handling the objects. In this case this doesn't apply, hence please use a simple "unsigned n_ref".


More robust way to find DOM element. prev() was fragile.


Earlier I had removed the Injector.values cache to allow the Provider to handle this concern, but I removed it when I thought it was causing an injector stats test to fail.  Anyway, it might be a good idea to let Providers handle caching.


Haha your hardcore eva


This was added here because we're now recursively walking pipelines, and we need better debug output.


burda galiba bir hata var?


Will do eva no problem


Nitpicking: I'd recommend to check against > 0 and exit early, so that we don't need to indent such a long code block.


- Création du projet Zawaj
- Epuration des répertoires
- Création du Bundle : FichesCandidatsBundle (permet la gestion des candidats)


These come form using the Injector APIs in `_handle_provider`.  Is there a reason not to use them?


@aullman I think there is a bug somewhere... I ignore the effect of setMaxFrameRate(null), but I guess the effect should be the same as setMaxFrameRate(30). Anyhow, with the null thing I don't see in the logs any request asking for a higher framerate, and once we hit 15 because the window is small, if we go back to a big widget size, we still receive 15fps


SHA_BUF_SIZ is not defined in a non SIMD build.


Scheme of the process model for Data Mining Based Decision Support System


I just wanted to put it on your radar so it gets attention later when
people try to actually use sufia-core.  For now, I'm focusing on the 130+
failing tests, so refactoring this isn't on my radar.


I need to put a check there to check if nc_list has a value and not null. What is this holding eva a string list of nc?


develop (rraustad) - x86_64-MacOS-10.9-clang: OK (1166 of 1166 tests passed)

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


The test runs quite a bit longer now.  Could you reduce this to 300000?


Great Vent for the Ormerod 2, makes large difference.


I figure when we start to export this we should switch the data and length parameters around. Usually we first pass the ptr and then the size, so we should do the same here, too.


Please merge, with the reduced count.


Looks fine.


ALREADY


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

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


> .。
> .！
>  "

Do not do that -___-


hmm, i think the max parameter should be "unsigned", and the special value should be (unsigned) -1 to turn off the max logic. After all values like -5 or so make no sense, hence we should not allow them via typesafety.


Awesome, thx!


@sambowler - interesting. Working perfectly for me on Android 5.1. Are you running this exact branch and having issues on your device or using this code in your own app?


Same issue as discussed in the mssql12 format.  SHA_BUF_SIZ is only defined in SIMD builds.


the word `anonymous` is misspelled here,  as well as in the user model


_handle_provider could be even shorter if Injector.apply() worked with classes.


Would not using `Math.log1p` here make the accuracy of the sample questionable? `p` values are typically small.


Any non SIMD builds will NOT be able to build AT ALL.  This must be re-done.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


First SURF doesn't work any more in non free version.
In my computer works, but in the raspberry not.

i'am using raspberry pi model B, with raspbian 7.8, Python 2.7.3, bcm2835-v4l2


Is there a reason why you wish to return option. Wouldn't returning NULL be more appropriate?


This is not going to work because you don't do: `newItems.add(child);`
Secondly, what if you have `var X:Id;`. In this case you want to add the list terminator.


LGIMO


ack :)


ack


Why is this commit in the PR? Could it be gotten rid of with a rebase or cherry-pick or something?


I'd really prefer if we could merge the two if branches, and only have a single invocation of link_set_hostname(). i.e. do "hostname = link->network->hostname", in the second branch and then just invoke link_set_hostname() on "hostname".


Also, if `U` is a uniform random variable, 1 - `U` is still uniformly random, so you can just say `log(u)` instead of `log(u)`; no need for the subtraction.


95574472_ip_measure (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


Please do not clobber *hostname on failure. Please read the hostname into a temporary string ptr, then verify it, and only then modify *hostname.


ack
Interestingly my environment does not have `PYTHON_ARCH`, maybe this is just coincidence. It works anyway when setting directly the x86 libs^^


Actually we may completly remove this : since the function was rewritten, no major regression were spotted so we can assume the extra code won't be needed.


Look at this for non SIMD build.  There is no binary.  Easy fix.  Just a
```
#else
    return 1;
```

Same bug shows for rawmd4 and nt2


i want to keep the ?* and ?+ syntax here


Same here


I changed this so that vim-enhanced stopped breaking it's bash syntax highlighting.


93469234-DemandResponse2 (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (651 of 651 tests passed)

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


no ack :P


Bare parens will keep working, it seems too idiomatic to drop it. Quoting
it of course works. What will never work is using nil as the empty list,
nil is the atom nil and never the empty list.

Robert

On 9 July 2015 at 22:04, Eric Bailey <notifications@github.com> wrote:

> Using bare parens to represent an empty list is deprecated; in a future
> version of LFE attempting to do that will actually result in an error.
>
> Ah, great to know!
>
> There are two guides for LFE
>
> Cool, I'll check them out and maybe fork locally.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lfex/xlfe/commit/2c58947d39924372a477e2c7059c03884b5775da#commitcomment-12092595>
> .
>


95574472_ip_measure (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Hmm, not sure I follow? isn't this the code that is supposed to reset the hostname when we loose the lease?


Can we drop this commit from the PR? It should be fixed with PR #20


Please assign "hn = hostname" here, so that afterwards you can just reference "hn" directly, instead of needing a ternary operator twice.


The error persists, but with a different error message
```bash
.........terminate called after throwing an instance of 'Teuchos::NullReferenceError'
  what():  /home/abinashmeher999/gsoc/symengine/symengine/teuchos/Teuchos_RCPNode.cpp:720:

Throw number = 1

Throw test that evaluated to true: true

Teuchos::RCP<SymEngine::Basic const> : You can not call operator->() or operator*() if getRawPtr()==0!
Aborted (core dumped)
```


Thanks for clarifying. I tired that first but for some reason it didn't work. I have a suspicion that since echo outputs text with line breaks in it, the whole command fails. I'd be happy to see if you have a better solution.


Doesn't ?factoid [show|set] make more sense though? ?* and ?+ don't make any sense at all to me and are harder to understand.


It goes both ways, yes maybe easier to make typo, but easier to cross check for typos...


bugger, forgot to remove that :p


Just fyi, this breaks Linux compile

Working on it atm


i like the shorthand
id say keep both here


Qui comunque ricordiamoci che non invia una READDATA ma era per testing:


ok cool
whatever make you happy! :D


So if we were to use this, would we also create the widgets that belong to the campaign page in the persist method in the campaign page builder? Similar to the example at [CodeClimate](http://blog.codeclimate.com/blog/2012/10/17/7-ways-to-decompose-fat-activerecord-models/):
```ruby
def persist!
    @company = Company.create!(name: company_name)
    @user = @company.users.create!(name: name, email: email)
end
```
Where does the validation happen for the widgets, and could we split widget creation logic into several files - e.g. widget_builder with a service for each object that requires it?


95574472_ip_measure (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


:bowtie:


Agregue esto porque uso el equals para Literals. Tarde o temprano lo vamos a necesitar, capaz que no hay que implementarlo, pero me parece que si.


if no arg is given, should print a generic help text and all commands


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

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


Is no terminator needed?


Lol thats what happens when you keep looking at code


No. An additional semicolon might be added by subsequent data in the stop reply. e.g.: line 208 in the new version of the file.


File an issue?


In that case, what type would you suggest? It should be something representative of the most amount of player objects a list can hold, right?


make sure not to overwrite the existing tag


It made some another problem. Cant make initial flipped plastel reinfirced tabble.


Makes sense, will do.


fix this.


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Last thing I wanted to add is that if you are rounding up your samples with `ceil` then your samples are from the set `{1, 2, 3, ... }`, and the mean for that interpretation of geometric distribution is `1 / p`, not `(1 - p)/p`. https://en.wikipedia.org/wiki/Geometric_distribution has a nice explanation.


This is good for now.  Thanks, @flyingzumwalt.


What is "this.location" and where does ist come from?


Code-police here: Can we plz have an empty newline at the end of files?


we probably should at some point revisit the error classes we have for the whole L20n :) And maybe have ParserError for both, entries and ast parser?


Cheers  believe me it saves time if you look how many times DBTableColumnIds is accessed you know what I mean. I will comment original columns name in.


I doubt this is a good idea. Maybe we should firs try it with pulling the branches before executing the split and see if this doesn't require us to push. If somebody checked out the subtree before it might come to conflicts if we force push.


O, @alicetragedy, we were working late tonight (on our FIRST pull request!!) and I forgot to pull the team picture from Roos' phone. It would be awesome if you could please wait for the second picture, before moving to live.


line 772 is not needed but the value will be set in a few lines later


This is how it is 'supposed' to work.  uc/lc can ONLY be used around $p or $u

```
$ ../run/john -test -form:'dynamic=md5(MD5($p))'
Benchmarking: dynamic=md5(MD5($p)) [128/128 AVX 4x3]... FAILED (cmp_all(1))

$ ../run/john -test -form:'dynamic=md5(MD5($p)),debug'
push
push
app_p
f5H
f5h
Expression=dynamic=md5(MD5($p))
Flag=MGF_FLAT_BUFFERS
Test=@dynamic=md5(MD5($p))@1ceb3e00bfa610e1c55453734a5c6b7e:abc
Test=@dynamic=md5(MD5($p))@5a0d320dc62bebcd6a831a149236b4bf:john
Test=@dynamic=md5(MD5($p))@d5411e2f4bf9f8c60334801d369364e3:passweird
Func=DynamicFunc__clean_input_kwik
Func=DynamicFunc__append_keys
Func=DynamicFunc__MD5_crypt_input1_overwrite_input2
Func=DynamicFunc__MD5_crypt_input2_to_output1_FINAL
MaxInputLenX86=110
MaxInputLen=110
```

It looks like the upper cased MD5 needs to set the upper case flag (I thought I had that being done??)


Could you add a test that doesn't have a trivial solution like this?


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


it's a fux for firefix! or a fire for fixfux :-D


First of all, on_cancelButton_clicked() - is a not standard slot, but one that will be connected automatically (using auto connection mechanism in Qt). Thats great, but why we need it? Please, check documentation for QDialog::accept() and QDialog::reject() and as well as QDialogButtonBox class. Itis not only provide native layout for buttons (on different OSs) but also allows to assign button role, so we do not need to create any custom slots.


?help prints all commands. What would the generic help text be?


@F3PiX that's okay – you're excused ;)
just ping me when you comment with the second photo, and I'll add it soon after that. lovely! I think you and @roosdebildt need some well-deserved sleep now :crescent_moon:


@tianon Thank you!
I have to admit that my skill, in this case, was to search the right thing on Google: http://stackoverflow.com/questions/9522631/how-to-put-line-comment-for-a-multi-line-command

Thank you again!


Nice, thanks!


configurable description from config
On Jul 9, 2015 17:51, "Jan Dalheimer" <notifications@github.com> wrote:

> ?help prints all commands. What would the generic help text be?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robotbrain/rubybot/commit/42a7fe4adb26f5d50d7a46d0b741ab71a46da357#commitcomment-12094513>
> .
>


Yes, that was the idea, but since I don't have a phone whch provides this, I don't know how ti should look...


line 617 to 621 is not needed
the line 616 set the selected option value
What browser do you use?


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


Fixed in 5233c97.

Something's weird with SHA512_ALGORITHM_NAME for non-SIMD but it's not caused by this commit


Just an FYI: the same thing needs to be added for x86 support; monolite till fails, which is why @gotnull was seeing issues compiling on 10.11.


@agentsmith988 Your's seems to be missing com.android.server.NotificationManagerService which I think is part of stock Android.

@kain83ita Your constructor seems to not match 5.1 on the Moto X 2nd gen.  Can you point me to your ROM's stock image?  I'll need to look at SystemUI.odex.


Thanks, that will help


created two sample Activity Layouts.


Zzzzzzzzzame to you  :-)


BoySka:
> why should nested virtualization be explicitly disabled?

it is not a requirement.

I only put in there explicit configuration in order to remember the
default value and the parameter that needs to be change to enable it.

--
vinc3nt

+-------------------------------------------------------------+
|GPG Id key: 4096R/DAA26AC4 2012-12-07                        |
|GPG Fingerprint: 19D8011C 81891778 7A1986DC 28E8567F DAA26AC4|
|OTR Fingerprint: 86F3EE43 8A9340D2 FF92A828 81C84DC2 FEDD839E|
+-------------------------------------------------------------+


Damn, nice. Have a random dent on me @changetip


This is dumb...how dare you SOP the String "Test"?!?!?


I switched out postgres for mysql.  It should be consistent now.


lol what a boss


Translatewiki only updates the json language files, not the js files.  The language changes in this commit are the propagation of the json changes to js as a result of a recompile.  I can't speak to the changes in Indonesian or Russian, but the Japanese changes are clearly improvements rather than regressions.


Could you also push this to the 1.8 branch?


Should be some "red" in here somewhere.


That was because on a non-gpu build --dupe-suppression can be abbreviated as -d, but on a gpu build -d collides with --devices.


@olivergierke It appears there is no way to support both Spring Data Commons 1.10.x and 1.11.x with the same version of code since 1.10.x does not have `DefaultRepositoryBaseClass` and 1.11.x appears to require it. Is there a way for Spring LDAP Data integration to support both versions?


Yesss :)  We started talking about this with @Pike today.  We were talking about `ResourceError` and  `EntityError` (difference in scope) vs. `SetupError`, `ParseError` and `RuntimeError` (difference in time of event).  No concrete decisions yet, but let's talk about this more.


I don't like that an array construction is forced here.  Maybe keep both old & new methods?


Ok, I have it fixed:

```
$ ../run/john -test -form:'dynamic=md5(MD5($p))'
Benchmarking: dynamic=md5(MD5($p)) [128/128 AVX 4x3]... DONE
Many salts:     9155K c/s real, 9192K c/s virtual
Only one salt:  9173K c/s real, 9089K c/s virtual

$ ../run/john -test -form:'dynamic=md5(MD5($p)),debug'
push
push
app_p
f5H
f5h
Expression=dynamic=md5(MD5($p))
Flag=MGF_FLAT_BUFFERS
Test=@dynamic=md5(MD5($p))@1ceb3e00bfa610e1c55453734a5c6b7e:abc
Test=@dynamic=md5(MD5($p))@5a0d320dc62bebcd6a831a149236b4bf:john
Test=@dynamic=md5(MD5($p))@d5411e2f4bf9f8c60334801d369364e3:passweird
Func=DynamicFunc__clean_input_kwik
Func=DynamicFunc__LargeHash_OUTMode_base16u
Func=DynamicFunc__append_keys
Func=DynamicFunc__MD5_crypt_input1_overwrite_input2
Func=DynamicFunc__MD5_crypt_input2_to_output1_FINAL
MaxInputLenX86=110
MaxInputLen=110
```


Can you add to the commit that this is based on https://github.com/docker/machine/blob/master/libmachine/provision/boot2docker.go#L165?

Also, shouldn't we use more of the boot2docker `/etc/os-release` file?

```
$ boot2docker ssh cat /etc/os-release
NAME=Boot2Docker
VERSION=1.7.0
ID=boot2docker
ID_LIKE=tcl
VERSION_ID=1.7.0
PRETTY_NAME="Boot2Docker 1.7.0 (TCL 6.3); master : 7960f90 - Thu Jun 18 18:31:45 UTC 2015"
ANSI_COLOR="1;34"
HOME_URL="http://boot2docker.io"
SUPPORT_URL="https://github.com/boot2docker/boot2docker"
BUG_REPORT_URL="https://github.com/boot2docker/boot2docker/issues"
```

Final point: I do think we should propose a fix to upstream buildroot to make it possible to customize `/etc/os-release` based on the name and version info entered in the config. Not something to do right now, but we should once our basic work on orbital is done.


Actually never mind, that' isn't practical.  It probably doesn't matter in practice.


develop (rraustad) - i386-Windows-7-VisualStudio-12: OK (1169 of 1169 tests passed)

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


We should also ignore any IDE specific files


BoySka:
> what's this for?
>

I don't recall at moment the specific issue I had with volume_cache, but
I fixed it in that way

--
vinc3nt

+-------------------------------------------------------------+
|GPG Id key: 4096R/DAA26AC4 2012-12-07                        |
|GPG Fingerprint: 19D8011C 81891778 7A1986DC 28E8567F DAA26AC4|
|OTR Fingerprint: 86F3EE43 8A9340D2 FF92A828 81C84DC2 FEDD839E|
+-------------------------------------------------------------+


@andycastro seguindo a ordem alfabética, o logotipo e as informações na tabela precisam ficar depois do Rio de Janeiro :cherries:


My only worry is amount of shapes and initializations in runtime. For tooling etc. I'm happy to have every error imaginable.

I'm also ok giving us a bit of time to see how the various errors needs settle.


I see, thanks for the clarification!


Dang, I did test non-SIMD but then I had these bugfixes and forgot to re-test.


Closes #73.


@mlucy Could you verify that these are the intended exception types when r.http fails?


I tested :
- the normal startup (PWR pressed between 2s and 5s)
- the reset from the BL - OK (no need to press PWR for 2s)
- the WDT reset - OK (no need to press PWR for 2s)

But I don't like too much this change!


@paulolieuthier I'd like to do a release of qtermwidget + qterminal. Can you take a look at this bug?


@KBrizzle have a random dent on me @changetip


Hi @kbrizzle, I've delivered a Bitcoin tip worth a random dent (360 bits/$0.10) from @MaxFangX to your ChangeTip wallet.

**[Learn about ChangeTip](https://www.changetip.com/tip-online/github)**


I'd rather have a comment saying what alignement is assumed.


Any reason not to merge this?


All is well.  I simply copied new files over to my VC development directory and things would not build any more.


Also have changes for the Sniper team


I see, so just putting the u left of the bracket should fix it. I still think these are dangerous regexp's but maybe they aren't that bad.


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Can you please include a CHANGELOG in this release?  If possible, can you also include details of changes made in previous releases as well?


This should be called only once per warningLedTimer , but as implemented, led will toggle on each `warningLedRefresh()` call (and this function is called outside of warningLedUpdate..


Shouldn't this say query? I don't see the search action.


The autotimer plugin on the box does not allow set a custom location except from the recording locations.
Do you have a modified autotimer plugin?


Without keytool the Export Signed Package functionality in the android mode doesn't work.


BoySka:
> why are you using your "regular" ssh key for vagrant, too?

while I was working on this, there were some ssh authentication issues
related with the ssh key embedded in the available vagrant version at
that time:

http://dev.freepto.mx/vagrant/utils/vagrant_1.6.3_x86_64.deb

this was a workaround

--
vinc3nt

+-------------------------------------------------------------+
|GPG Id key: 4096R/DAA26AC4 2012-12-07                        |
|GPG Fingerprint: 19D8011C 81891778 7A1986DC 28E8567F DAA26AC4|
|OTR Fingerprint: 86F3EE43 8A9340D2 FF92A828 81C84DC2 FEDD839E|
+-------------------------------------------------------------+


Why did you comment this out? Imho, it should remain asis within current issue.


I suspect the problem is that the root directory returns a path of "".. so maybe insetad we want a line a few lines up that does

if (path.length()) path = path.substr(1);  // drop leading /


It should extend FormComponentPanel


i think this return true should be removed in this case


Is it possible to move this code in another commit ? It's not related to alignment...


IMHO we should simply symlink `/var/lib/boot2docker/etc/hostname` to `/etc/hostname`.


No need for extra form: page themself already wrapped to form.


Just because compilation would be broken. I'll fix it surely in final commit.


No need for markup registration if you will use FormComponentPanel


add this code


So, what is this project gonna be? :smile: New homepage?


I tested :

- the normal startup (PWR pressed between 2s and 5s)
- the reset from the BL - OK (no need to press PWR for 2s)
- the WDT reset - OK (no need to press PWR for 2s)

But I don't like too much this change!


Cool, so it's just MD5() instead of md5() and was implemented already?


Do you really need downgrade to Collection? Model do ordering, so List should be much more reasonable.


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

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


Where did you cherry-pick this ?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.750.1834420bf03edb8544dff509e35ffa5d037bdab7](http://build.palaso.org/viewLog.html?buildId=59574&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 369, ignored: 3 Build time: 00:04:48


awesome, thanks!


We should probably just add this as a symlink to the image instead of adding it during our docker's init.


No need to store in a list. Just print each of them directly in the output.warn() method call


Is there away to call of the batch file that started Chrome in the first place?  I have an issue with an environmental that causes problems with certain extensions using Native Mode.  The batch file set the environment variable comspec back to cmd.exe and I only want that done on the Chrome windows.   I do a lot of command line things and I prefer TCC over CMD.

Thanks

George Worley


Comentario de la Pruebas del commit