@johnclause
You've changed the section titles in all languages from "General Settings" to "General", "Advanced Settings" to "Advanced" and "Custom Integration" to "Integration". This is fine.
But as you changed this in all languages, the translations should be reviewed also. No one will probably notice the original english strings is differente unless they inspect commits.
I think you should always mark this changed strings *as fuzzy* so that translators know they have to pay atention to it, or else it goes down under all the strings that appear to be translated a long time ago.
I'm going to simplify the pt_PT correspondent strings, but it's a good idea to go back to all languages and mark those *as fuzzy*.


powinno


Fair enough :) I'd like to highlight the alternative `for … in …` syntax as well, which is what most people would use because it reads better and is usually more efficient with arrays and objects than a `for` loop:

``` js
var returnArray = [];
for (var item in array) {
  returnArray.push(item + " is " + array[item]);
}
return returnArray;
```


#build


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 11](https://teamcity.labkey.org/viewLog.html?buildId=250950&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 25 Build time: 00:43:50
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

```


Thanks for the working fork!  Is this variable currently being used?


There it is, the `for (… in …)` syntax :) Nice.

This time as well, another built-in level of abstraction is available:

``` js
return Object.keys(inObject);
```

[`Object`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object) is the kind of data structure most of the "objects" in JavaScript inherit from. It's kind of an abstract store, with keys and values, and a bunch of helper functions such as `keys`. That one expects one argument, the name of a variable containing an actual object you want to list the keys from.


Note: I'm assuming we'll write new steps for "trashed" rather than "removed", because Dark Ages introduces assorted cards that care about the contents of the trash (Rogue, Gravedigger, Forager) so we'll want the test framework to track its contents. Yes, this will need a bunch of edits to the tests for every existing trashing card, but hopefully pretty trivial edits.


I don't have a node_modules dir? After installing Karma, I dropped 'node_modules/.bin without success too :-(


looks lice COVERALLS_REPO_TOKEN should be set
please review docs of cpp-coveralls


Nice :)


Are we sure here? Mesa is a very critical component - did you test that on AMD / intel at least?


@TheSin- ,

there are line alignment issues with your demo :

http://jsfiddle.net/856bzzeL/264/

Scroll, resize and see.


In the future:

https://jonbellah.com/recursively-remove-ds-store/

If you need to delete files you should delete everything and then commit, not make a new commit for every deletion.


Yea, time to redefine the model for these bindings :P


Excellent use of `&&` here.


Looks good!


Now much more interactive vs having to figure out what you're looking for on each blog page


That's a much better alternative, thanks! I was trying to avoid unless and didn't know what to do.


Why don't we do the l.wg.Done() here after l.maintain()?

It seems more appropriate/symmetric since the l.wg.Add(1) occurs in AcquireLease().


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (351 of 351 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)


see comment above about doing this in the caller instead.


Use cmake to fix snappy's config.h


forward declare


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


Sure but in terms of size, the browser build is still smaller. However, I will mention the explicit listing as well.


:+1: !


This 3 should be moved to the .cpp


break


indenation


This drdump site is really nonintuitive ;/ I just realized that I had bookmarked wrong app (1.7.8), instead of  1.7.8.*. And I was really suprised that we had zero reports, hehe. I would have helped with reports.

I have a question tho. If the crash report is assigned to a bug and it's is marked as won't fix does user get info about it when similar crash happens? Do we need to set "Response Url"?


couldn't you just not set ADAPTOR_CLASS in the cmake file?


`go func() error {}()` doesn't make any sense :/


I think so


+1, we should either vendor or take a more distinct approach


This change creates trouble. I think it should be static_cast<uInt> to match the type of avail_in.


I'm glad you kept this since it really helped with some workloads under Linux.  But it got me thinking that it would be interesting to additionally include the illumos implementation of this function.  We could use a module option, say `arc_meta_strategy`, to make this configurable.  That would potentially let us more easily assess what strategy works best in the field.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 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-5.29%25-green.svg)


Oh ok thanks for the explanation! :+1:


There are 3 "QTracker" references below here. For completeness change it to "Tracker"


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/e43b1b8d0dce1277c1d519e2bd9b5f93613a395b/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/e43b1b8d0dce1277c1d519e2bd9b5f93613a395b/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/e43b1b8d0dce1277c1d519e2bd9b5f93613a395b/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/e43b1b8d0dce1277c1d519e2bd9b5f93613a395b/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/e43b1b8d0dce1277c1d519e2bd9b5f93613a395b/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


can you document the required argv, please?


This looks spurious, how about naming the state consts something more contextually descriptive like suffixing State to them?  Is there something idiomatic for this?

My brain wants this to look like an enum or define in C, which would typically be in caps in my C code e.g.:
> state := LEASE_STATE_BOUND

clearly communicating it as a global symbolic constant, can't we do something analogous in Go?


There are other references to "QTracker" inside qDebug() calls and code comments in the rest of the file. Change them to "Tracker".


@netj I think one way could be branch at a higher level (say groundFactorGraph), and don't branch in inner functions. I'll think more carefully about it.


Yes, section 4.4.5 of https://www.ietf.org/rfc/rfc2131.txt


"extracttarchroot error"


Even though I hate spaces passionately, just batch convert the whole repo and be done with it:

Ex, from repo root: find . -name '*.lua' ! -type d -exec bash -c 'expand -t 4 "$0" > /tmp/expandtmp && mv /tmp/expandtmp "$0"' {} \;

^Run for applicable file types but don't wildcard so you don't destroy the git hidden repo files.

For line endings do the same but recursive dos2unix


Also 4.4.5 of https://www.ietf.org/rfc/rfc2131.txt


I agree, since it writes to logfile, putting it into very fast called procs can be /very/ laggy. I once left it in air alarm status checking, it nearly killed server.


Good point. Need to check that renewal < rebinding < expire.


forward declare


This commit looks good, but there are more +/- (red and green) lines than necessary in this commit because many of the "changed" lines are just whitespace changes. Remember to use command-T (Autoformat) to reformat your source code files before saving them :-)


nit, this should just be `r io.Reader`


Add the Q_OBJECT() macro here too.


el archivo se llama Clase_Disparo.java per la clase implementada se llama Disparo.

por convención estos nombres deben coincidir


I want this back in


Blocking and not doing the eviction work here I suspect will be good for latency.  I'm sure we had the case before where many threads hit this concurrently and all contended with each other.  It probably better to just single the reclaim thread to take care of it and wait.


I want this back here.


throughout _its_ lifetime


Yep maybe later ;) (tomorrow). It's getting late in France :sleeping: :night_with_stars:


los comentarios de los atributos deben ser del la siguiente forma:

/**
*Este es un comentario de un metodo
*
**/

void metodo(){
...
}

mas informacion puede ser encontrada en

http://en.wikipedia.org/wiki/Javadoc
http://www.oracle.com/technetwork/articles/java/index-137868.html


Tried out using "unless" instead of "if !=" :)


I think it makes sense to have this always be on, there is no harm in it and it is useful to validate issues on these pages without having to figure out how to cause a 404 or 500


Incorporated awesome space saving/repetition reducing tip.


Decided to leave as is, but next time would use array and check for this above!


There is actually a google package called re2. I was thinking that there was a need to differentiate it. Is that not the case?


I am dumb, can you explain why this is necessary in the store tests?


This should a one liner.
If you remember when we were discussing the logger classes a few months ago, we agreed to break the braces when there is an initialization list otherwise they should be one liner(any empty method).


This assumes that `result` is an array...


Note that this line should be removed too.


This line can cause errors in the case of a new slave coming up into an existing stack:
```
+ cfn-signal -e 0 -r 'Stack setup complete' --resource ServerGroup --stack prod-mesos-MesosSlaveStack-XXXX --region us-east-1
ValidationError: Stack arn:aws:cloudformation:us-east-1:XXXX:stack/prod-mesos-MesosSlaveStack-XXXX/XXXX is in UPDATE_COMPLETE state and cannot be signaled
```


These constants should be set as const static


disparojugadorx deberia llamarse disparoJugadorX
por convencion en java los nombres de las variables poseen mayusculas en la primera letra de una palabra.
mas informacion puedeser encontrada aqui
http://www.oracle.com/technetwork/java/javase/documentation/codeconventions-135099.html#367


Q_OBJECT
(without parentheses)


We've already picked this up from 5056 so it should be automatically dropped when rebasing on master


I'm having trouble following what you are trying to do here.  `map(f, xf)` will return a transformer, `_stepCat` is essentially `identity`


*scott smiles*.  Counting characters?  Well, this name is unlikely to collide with any others.  I'm good with it.


아 생각해보니 웅철 말대로 이 상태에서 history.js 붙이면 될것같군?


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 14](https://teamcity.labkey.org/viewLog.html?buildId=251119&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **SUCCESS**
Summary: Tests passed: 26 Build time: 00:34:22


Did you mean to use your new `_flatCat` instead of `_stepCat`?


Do you mean it would be cleaner to change config.h.in instead?


Sure. Will do


Forgot the pubspec.yaml update?


Steve,
  Starting to get back into the swing of things, my intent for now is to keep the branches very small in size.  I am going to create new branch off of master for player sign up.


yes, this is more up to date: https://github.com/ramda/ramda/compare/master...buzzdecafe:stepchain
 but still not right


Why not $contents[] = $key?  Recording order of insertion might be nice.  Also, I see another argument for your plan to go to Redis!


`history.js` 지금 붙이지 말자. 내가 말을 꺼내기는 했지만, 브라우저 기본 동작을 방해하거나 조작하는 것들은 최대한 안 하는 게 좋음 (제대로 쓰기도 힘들고)


The only thing I'm worried about if we add Mapbox Satellite is people getting confused about which imagery is active. Thoughts?


These three should all be `const`.


LGTM


Buenisimo


Interesting fact, StopIteration can contain the return value of the generator in later Python versions.  Ref: https://github.com/jython/swingutils/blob/master/swingutils/threads/coroutine.py#L52-L57


`history.js` 안 붙일거면 개별 링크가 그냥 새 html 페이지가 되도록 하나? `stories_by.html.slim`에서 링크 누르면 `stories_of.html.slim`이 렌더링되는 식으로. 그렇게 하면 간단하긴 함.
트위터는 `twitter.com/user_name`과 `twitter.com/user_name/with_replies`의 관계가 그런것 같던데. 누르면 새로 페이지 로딩됨.


`const`.


Looks like it'll do the job.  What about going a step further and implementing IndexedFifoQueueStore, maybe even popMultiple if you're concerned about performance?  Would make the client code way less sleazy!


what is this "self" you are talking about :)


On line ~30, "validators" -> "validator's".  Things must be in pretty good shape if I'm getting that picky, eh?


This was wrong, copying the pointer was the intent of the code to work around const issues with PySys_SetPath. Did you actually experience an issue, or did it just look wrong to you?


ㅇㅇ https://github.com/fools-gold/swindler/commit/d1d4688#commitcomment-11265862 여기에서 얘기한게 그렇게 간단하게 하는 거였음.

> 탭바를 사용하더라도, 각 탭들이 페이지 이동하는 링크처럼 동작해도 됨.


Fix_sqlite_queries (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2123 of 2201 tests passed)

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


The point was to show how this works with test, class fixtures.
Add a second test to the above testcase.


Please document the function parameters.


This is a no-op here because of implementation of `_reduced` in Ramda.  `preservingReduced` should "double-wrap" the reduced object so that it will still return as reduced after early termination in the `_cat` implementation.


ㅇㅋ 알았음. ㅋㅋ난 ajax를 사용하자길래 이 방식인줄. 그런데 해놓고 보니 history.js를 안 붙이면 결국 탭이랑 똑같더라고.


Probably should be good.


I suspect that 'map' and 'cloak' are the only possible item types, but is it worth adding an 'else' clause that outputs a warning if an unrecognized item type is encountered?


I can add a class fixture to this test, but I do not want to add another test to the other InnerTestCase. If I did that, I wouldn't be able to make assertions that I'm making in the @teardown.


Hi @ddanderson, first up, I don't think this code is safe.  The value passed in here is not a copy: it may actually be a direct pointer to a value in a the table.  Modifying that data in place could cause a concurrent reader to see the modified value.

I would be inclined to make a copy of the bytes into a buffer, NUL-terminate it, then call `result_cursor->insert`.  It will be more code, but modifying the value in place doesn't seem like a good idea.

You could make the index have type 'u' instead, and create a `WT_ITEM` containing just the bytes of the value without NUL-terminating, but that's going to be harder to use than a string index.


Is use of boost::multi_index permitted, or does it require a petition?


@spilist 뷰의 세세한 로직을 컨트롤러에서 로컬 변수 넘겨서 조작하는 거 좋지 않은 패턴임. `stories`랑 `stories_of`가 같은 파셜을 쓰게 하지 말고, 공통된 부분만 다른 파셜로 빼내서 같이 쓰게 하는 식으로 해야 함


ㅇㅇ 그렇게 할 수도 있다–를 말한 거였는데 잘못 전달된 거 같네


move this to a template maybe?


Boost.MultIndex is on the whitelist, so it doesn't require permission.


My knee-jerk reaction was to encourage you to get rid of this "unnecessary" code, but on further thought I think it might be better to keep it, as it's the obvious thing to have, someone can activate it easily enough to try it, and it shows that you've thought about the issue (and commented!).


Shouldn't the `June 1st - August 10th` be in a separate `p` tag?


aqui ficou ainda SUPORTE e APOIO.

Acho que tinha que ficar só:

no projeto [PHP The Right Way][2], mantido por [Josh Lockhart][3] com o apoio da comunidade.


この箇所では user が Null になる可能性があるため、Null チェックを入れるようにお願いします。
合わせて Velocity 側でも $record.UserId が Null になる可能性があるので分岐を入れておいたほうがようさそうです。


Which skills? We should say we're teaching web development.


Could we change this message to the following for additional help?

'tf' was not found. Please install the Microsoft Team Explorer Everywhere cross-platform, command-line client and add 'tf' to the path.  See https://www.visualstudio.com/products/team-explorer-everywhere-vs.aspx.


The break between sentences has two spaces in it. Should only be one to be consistent with the rest of the website.


This is all deletions except a break.  In this case, I don't believe the code is copyrighted.


the handler variable will not be the same as 'this'? I mean, the current object is the form handler right? why getting it again?


Should have a space between the two words in HackCamp.


Fix_sqlite_queries (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I don't like seeing magic numbers in the code.  I realise you've just copied it, but perhaps you could add a comment explaining the choice of 5 (presumably minimum required for the fitting)?  Or have `polyfit` throw an exception if it doesn't have enough points (which moves the number closer to the code that sets it)?


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


ALActivity に userId の setter/gettter メソッドを用意したので、ActivityAllSelectData に追加した setter/getter メソッドについては差し戻すことができると思います。
外してみて動作確認を行ってみてください。


Remove the comma here. It's grammatically incorrect.


Comi bola. Já resolvo isso aí


comment says it's from Billiard, who we have permission to relicense from.


i'm not yet seeing the distinction. looking here:
https://github.com/transduce/transduce/blob/0df525451421baf0e626637203be8c54f7d4d212/src/transducers.js#L89


See 7459461280dc3945ceb67d44e1a711edefaa89ee


idk after which commit, but I am getting a metric shit ton of "Texture Importer Failure!" errors...
All I did was brighten up the room texture in PS to make it look better.
Any ideas?


Oh, to understand the comment, I think it's important to know that the matching is being done in order from brightest to faintest, which is why one wants to take the first match.


Corrections are done correctly, I approve.


`reserve`?


I never used the rlm_python, I found just looking the code. (btw, I love to inspect)


근데 이건 완전한 뷰가 아니라 js erb 렌더링인데 그래도 같은 원리인가?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 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)


Thanks! BitHub has sent payment of  $8.73USD for this commit.


just to note, currently cl-travis needs `sudo: true`


Non copyrightable


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 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 ok, no the code was fine so i've reverted your changes.

There's a discussion on memcpy and consting here http://stackoverflow.com/questions/30250779/how-to-copy-a-pointer-to-an-element-of-an-array-with-memcpy


Product price (and all other listing data for products) is handled using Feeds. While my library is capable of sending feeds and viewing feed processing responses, it does not handle the generation of feeds, as feeds are not exclusive to Amazon MWS and the documentation for them is a big mess.

As for logging in, it is not part of the MWS API, so I don't know anything about it, sorry.


i see now, you are forcing the reduced value. hmmmm


Don't use table, use bootstrap class


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


`transduce` has a `force` flag (questionable decision, may be changed) called in [PreserveReduced](https://github.com/transduce/transduce/blob/0df525451421baf0e626637203be8c54f7d4d212/src/transducers.js#L105) that forces the "double wrapping" in [`reduced`](https://github.com/transduce/transduce/blob/0df525451421baf0e626637203be8c54f7d4d212/src/util.js#L49).

Ramda could arguably just change the `_reduced` implementation to this:

```javascript
module.exports = function(x) {
  return {
      '@@transducer/value': x,
      '@@transducer/reduced': true
    };
};
```


Ah, of course... didn't notice the new release. Anyway, looks like they replaced 2.1.1 on May 1 then.

```console
$ gpg2 --verify electrum-2.1.1.dmg.asc
gpg: assuming signed data in 'electrum-2.1.1.dmg'
gpg: Signature made Fri May  1 11:11:50 2015 BST using RSA key ID 695506FD
gpg: Good signature from "Animazing <animazing@gmail.com>" [unknown]
```


There are two things going on in this function:
1. Optimise by using the squared-distance instead of distance (saving the sqrt).
2. Changed to support the duplicate reference object changes.

The second justly belongs in this commit, but I think it would be good to split out the first.


https://github.com/Asexual/agar.io-hacks  intergrate the esdf movement and click to move please.


yep, i saw it right before you posted. trying a solution now.


Okay, thank you so much.

On Wed, May 20, 2015 at 5:58 PM, Thomas Hernandez <notifications@github.com>
wrote:

> Product price (and all other listing data for products) is handled using
> Feeds. While my library is capable of sending feeds and viewing feed
> processing responses, it does not handle the generation of feeds, as feeds
> are not exclusive to Amazon MWS and the documentation for them is a big
> mess.
>
> As for logging in, it is not part of the MWS API, so I don't know anything
> about it, sorry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-11296586>
> .
>

--
God bless you!
Joseph


I think this should be specified in degrees rather than radians, as that's nicer for the user.


we did fix the conflicts.  hoorah!


I set vm.setHostId(host != null ? host.id : ""). I am worried about the null will cause null exception when i use hostId. Is my worry is unnecessary?


What about these other fields that don't have docs?  Could you update those while you're at it, please?


take a look at the actual commit: 23266f9718b29f522c4a2287c1c7e165c87aaf6f


뷰 로직이 컨트롤러에 있다 = BAD


test


hehe


i've made it worse :smile:


There's 28 of these errors now? I cant test anything.


haha, Ok.


fdsd


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


Why do you feel the need for this? Is it really better performance? Do you have data to back your statement? How is the performance different between JIT (V8 on Android) vs non-JIT (JSC on iOS)?


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/e608d14a4f3f127fb7ff8c707fdf9f91117d9c05/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/e608d14a4f3f127fb7ff8c707fdf9f91117d9c05/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/e608d14a4f3f127fb7ff8c707fdf9f91117d9c05/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/e608d14a4f3f127fb7ff8c707fdf9f91117d9c05/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/e608d14a4f3f127fb7ff8c707fdf9f91117d9c05/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1145 of 1145 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)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 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 is an orthogonal change.  Please do it on a separate commit (if you must do it).


Странно, `PREG_SPLIT_NO_EMPTY` решал бы эту проблему если в дампе было много переводов строк, в таком случае предпочтительно переделать на стандартную проверку типа перевода строки *никс/вин/мак - `\r\n|\r|\n` так:
```php
$aQuery = preg_split("#;\r\n|\r|\n#", $sFileQuery, null, PREG_SPLIT_NO_EMPTY);
```
либо
```php
$aQuery = preg_split("#;[\r\n]+#", $sFileQuery);
```
именно этот участок был обновлен в сравнении c 1.0.3 т.к. раньше дампы с сериализированными значениями приводили к ошибке


I think this is also an orthogonal change.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (351 of 351 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 should be `_xfBase.result`.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 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)


I think you need to leave the word "matching" in there, as it relates to the algorithm.


Well, except for `this.xf` is not defined...


This commit mixes doc updates with parameter changes with functional changes with bugfixes.  These should all be split into separate commits.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 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)


Is it necessary to drop support for version 0?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


I see now you just moved this from the modal to the form handler and used events to trigger it. Nice change, just needs that fix mentioned above, I think.


Ahh yes, I see it. Probably a bit overzealous, yes. Let me know if you want a new patch or you're happy to fix it as part of merging.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


sure :)  I try to avoid regexes when doing exact string matches, but `s///` is more perlish.


rax retry tests


well spotted!


I think this should be `map(f, _flatCat(xf))` as composition is reversed with transducers.


that gets me an unflattened list. so that's pretty close


Wow. Is it that simple??


I still don't get the difference. Why does it fix the issue and still produces correct outputs? I don't want to block progress, just trying to understand the fix.


@addyosmani I don't understand this change. Does `**/*` already mean recursively go through all the subfolders?


No es CINE HOYTS?


I thought "resolved" was a synonym for "galaxy" (or extended), but you're checking for multiple peaks.  How about `_isMultiple`?

The last line should be:

    return footprint is not None and len(footprint.getPeaks()) > 1


You could arguably just use this for the implementation of `_reduced` ... nobody should call `_reduced` twice.


By "the issue" you mean the subselection thing? It doesn't automatically fix it -- that got vetoed earlier in the thread because we can't guarantee x=y=0 is meaningful for all layouts, especially those that we copied from FieldTrip. This "fixes" the problem by allowing the user to specify what the center and scales should be manually if they want.


i think you've got it. thanks so much


You're not actually checking all of these.


Please don't leave in code that has been commented-out.


Recidente?


要不要这么拼呀，半夜两点了还在commit


Finally :)


i hear you. for the time being this may be closer to a kosher impl, but without the optional flag (eeewwww).


You probably want "interpolated" instead of "saturated" --- the centroid could be messed up by other means (e.g., cosmic ray removal).


@willwhitney You could use my [`fix-path`](https://github.com/sindresorhus/fix-path) module to work around this ;)


Go ahead and push up a new patch. Think that's preferred over fixing up later.


makeCacheMatrix can assign a value to an object in an environment and


Though this is something that would nice if Atom did for you, so you should probably open an Atom issue about it.


store this value in cache.cacheSolve checks cache for inversed matrix  generated by makeChacheMatrix,


if not, it inverses the matrix.


Why not inherit ``CallbackFilterIterator`?


Why not inherit from `CallbackFilterIterator` and `RecursiveCallbackFilterIterator`?


You might need a better game loop


No more error now thank Haru! I'll test if mapserver will also crash. Thanks


Ok. Very good start. Do you have a unit test for this?


Done


You guys and your "ideals" and your "correctness".  Why don't you just use Haskell ;)


Happy to help.


 m <- NULL
  set <- function(y) {
    x <<- y
    m <- NULL
  }
  get <- function() x
  setcache <- function(solve) m <<- solve
  getcache <- function() m
  list(set = set, get = get,
       setcache = setcache,
       getcache = getcache)


Seems like TestON params file is going to get larger with number of user configurable settings. Some time later we may need to standardize on these parameters names and list all of them in a sample PARAMS file for an end users convenience.


First try


## cacheSolve function first checks the matrix has already been inversed.
## If yes, it gets the inversed matrix from the cache.
## If not, it inverses the matrix and set the inversed matrix to cache.

## cacheSolve function first checks the matrix has already been inversed


m <- x$getcache()
  if(!is.null(m)){
    message("getting cached data")
    return(m)
  }
  data <- x$get()
  m <- solve(data, ...)
  x$setcache(m)
  m


set the value of matrix
get the value of matrix
setcache set inversed matrix
getcache get inversed matrix


makeCacheMatrix function creates a matrix and inverses the matrix, then caches the inversed matrix.


Hey @pei-jung,

The ordering is intentional. If the updateImage fails and throws an exception, we don't want to delete the old image.


Wondering if we should make this a map of regex to handler? I don't know that we'll have to intercept anything other than scope but it seems somewhat reasonable. (I have a sneaking suspicion we'll have to do something similar for subsystems if/when we decide to allow sftp or if we model http file transfer that way.)


Oh hey, fix-path + shell-path are pretty neat. Should add those to https://github.com/rgbkrk/atom-script


Ooh. I'll look into it!


Is there a reason we are going with 0.8.1.1 ? A lot of folks have been moving to 0.8.2.1 to help remove zookeeper offset dependencies and some more configurable ways for durability guarantees, new java producer too, etc. The kafka mesos framework can use both but think that we should promote the 0.8.2.1 release.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2118 of 2212 tests passed)

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


:sparkles: $$\lambda$$ :sparkles:


Yeah, just conditionalizing it out has worked for us so far.


This would also match on any command that starts with scope - like `scp0` - right? Don't we need to see if it's `scp` or `scp` followed by some whitespace or delimiter?


i think the remaining two test failures are because of bad assertions, so I will fix that up. Thanks again so much for the insights.


У меня стойкое ощущение, что нарисована только половина свиньи. Ну не может же быть так, что _вообще ничего_ с этой стороны не видно. Одна нога - ладно. Но две? И задняя меня беспокоит куда больше, чем передняя.


Хотя, может, у неё жопа просто шире, чем мне кажется? Короче, не знаю, что-то не так.


wrong doc


cool. I'll merge but I should restyle the readme to match my [new readme style](https://github.com/hybridables/hybridify-all) and update the tests.

You can continue to use same version range, cuz I follow `ferver` in which if not breaking change it is only needed to update the patch number.


Alright, I will make sure to auto-format them next time.
Thanks for the help.
Love, Lucas
> On May 20, 2015, at 8:14 PM, Nick Marden <notifications@github.com> wrote:
>
> This commit looks good, but there are more +/- (red and green) lines than necessary in this commit because many of the "changed" lines are just whitespace changes. Remember to use command-T (Autoformat) to reformat your source code files before saving them :-)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/lucas-marden/ScienceExpoProject/commit/d6a12b1422515ae13bfbe284f7ff19476f05c8dd#commitcomment-11296169>.
>


is this also handled if strict?


wrong doc I think


The 2.0 is a magic number.  It should be user-configurable.


Am I supposed to add descriptions to this file?


Why are you using clipped statistics on the match list? Shouldn't they all be good matches?


shouldn't this be 0.8.2.1 ?


I kind of feel like we should maybe have this outside in a directory, load the tables from directory /schema/*.cql (where each table gets another cql file)


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Note to self: breaking public API change


Ya that would probably look better too


I'm seeing errors like this:
```
The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/Wumpus.png' does not exist!
```

That probably means that you didn't commit the "80's shit" folder. My guess is that the issue that you are facing is something else, but I can't see it until you add those assets.


you're already passing submission id in the form action url... Why not pass the stage id there also?


Logging the scatter of every iteration at "info" level seems like overkill.  I don't think any other task logs at that level of detail.


wrong variable name


wrong id


While you're here, you can remove the `if (!bootstrapIps.empty ())` on line 463: the code immediately before it guarantees that you can never be in that situation.


So I actually ended up removing the urlBar/toolbar from here today because they were computationally expensive when running transition animation. I'm in the process of using the BrowserViewController's header/footer properties as the views that animate during the transition as it was originally but it's difficult to get the transforms right for it.


same values?


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (351 of 351 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)


Im pretty sure I did, but I'll check

Sent by Outlook for Android

On Wed, May 20, 2015 at 6:51 PM -0700, "Wasabi Fan" <notifications@github.com> wrote:
I'm seeing errors like this:
```
The source file 'd:/Users/kaeli_000/Source/Repos/Hunt-The-Wumpus/Final Game/Content/Images/80's shit/Wumpus.png' does not exist!
```

That probably means that you didn't commit the "80's shit" folder. My guess is that the issue that you are facing is something else, but I can't see it until you add those assets.

---
Reply to this email directly or view it on GitHub:
https://github.com/kyranstar/Hunt-The-Wumpus/commit/ec73fa7659f9c7f26de1edcad5760ea6c3b6a139#commitcomment-11297147


wait... Who are you?? You gotta be kidding me, right??


Well, looks like that was an online test that broke and wasn't marked properly as online. Nothing to do with this commit. Fix here:
https://github.com/Flexget/Flexget/commit/d177ee0e0f69df0b75af4e8f171b5769060ea7d2


为什么这么写？


So if i create a draft, type a lot, close my browser and come back later. I can delete the graph whitout warning ? I think it makes sense to check updated for closing the page, but not for deleting the draft.


I think you should use `array_filter` function in here


Interesting, Jetty seems to have some weird defaults.


Next step: a solution? ;)

> On 21.05.2015, at 11:48, Björn <notifications@github.com> wrote:
>
> the war file is missing in the repository so rebuilding the docker images failes
>
> —
> Reply to this email directly or view it on GitHub.
>


@bashor

DONE
* fileIndexes.kt: move val and fun to the beginning of the file
* fileIndexes.kt: use { parameter -> <newline>... for lambda-functions
* MockLibraryUtil: runCompiler -> runJvmCompiler
* SimpleTestMethodModel: move static final to the beginning of the file
* directives.txt: add links to issues
* directives.txt: enable test for annotations on types
* directives.txt: create issue for secondary constructors
* AbstractDecompiledTextFromJsMetadataTest: inline getJsMetaFile
* AbstractDecompiledTextBaseTest: make checkThatFileWasParsedCorrectly private
* AbstractDecompiledTextTest: use test.assertTrue instead of Assert.assertTrue
* AbstractDecompiledTextFromJsMetadataTest: use test.assertTrue instead of Assert.assertTrue
* AbstractDecompiledTextFromJsMetadataTest: remove tearDown
* testUtils: closeAndDeleteProject -> simplify
* testUtils: closeAndDeleteProject: remove unnecessary usages
* KotlinJavaScriptStubConsistencyTest: PACKAGES -> packages
* KotlinJavaScriptStubConsistencyTest: remove tearDown
* KotlinJavaScriptStubConsistencyTest: use abstract createStubBuilder
* KotlinJavaScriptStubConsistencyTest: move abstract members to the beginning of the class
* TextConsistencyBaseTest: remove tearDown
* TextConsistencyBaseTest: PACKAGES, TOP -> lower case
* TextConsistencyBaseTest: move abstract members to the beginning of the class
* KotlinJavaScriptDecompiledTextConsistencyTest: Pair -> <key> to <value>
* KotlinJavaScriptDecompiledTextConsistencyTest: extract common code for ResolverForDecompiler
* VirtualFileFinderFactory: use createForJvm, createForJs
* DecompiledNavigationUtil: reuse factory
* VirtualFileFinderFactory: use createForJs, createForJvm
* VirtualFileFinder: add JsVirtualFileFinder, JvmVirtualFileFinder


Fixed for using in China region


আইডিকে কেন পাস করা হচ্ছে? যেহেতু এটা স্টাফ কনট্রোলার সেহেতু স্বাভাবিকভাবে এটা staff.php মডেলের কাছে যাওয়ার কথা। আইডিকে পাস করার তো কোন দরকার দেখি না


nit: formatting is off


Write type signature as follows

```haskell

forceAtN :: ( DynGraph gr
            , PrimMonad m,
            )
         => gr y c                -- ^ The graph
         -> M.MVector ....   -- ^ The position vector
         -> m ()
forctAt N
```


Probably, yeah.


Submitted as https://bugzilla.gnome.org/show_bug.cgi?id=749668


Submitted to https://bugzilla.gnome.org/show_bug.cgi?id=739172


here too


Is this necessary? What does
`w@{ $FAI::current_dev_children{$d} }`
return?


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


I'm uncertain whether the `@return` definition is accurate, because I'm uncertain how `summarize` impacts what is returned. Hidden depths like this are sub-optimal. :smiley:


Use ST monad directly. This means that you can simplify the type
definitions. Further you can have type aliases for postion and force.

```haskell

type PositionVector s = M.MVector (ST s) (V2 Double)
type ForceVector s    = M.MVector (ST s) Double

```


This is great :+1:

Will localising each identifier individually, e.g. `:local(.className) :local(.subClass)`, continue to be supported, or are you planning to deprecate this? It makes for a simpler transformation in postcss-local-scope, so I'm hoping it's the former.


> exists in many classes e.g container_port_config.rb
> ?

yes, but it's leftovers and shouldn't be there


you mean something like that?


(system_is_livesystem sounded better to me than 'avoid_live_system')


We can enable a compiler warning for unused function arguments. It's currently disabled by:
https://github.com/simonjbeaumont/xen-api/blob/6c2b85bd99a63f4a512ed018fe88098791b7e772/OMakefile#L37

```
OCAMLFLAGS = -g -dtypes -bin-annot -thread $(WARN) +a-4-6-9-27-28-29
```

Warning 27 in the compiler docs:

> 27: Innocuous unused variable: unused variable that is not bound with let nor as, and doesn’t start with an underscore (_) character.


Trying to revert back


These cache value declarations are not generated. Compiling memoized derivatives produces errors like
```
[info] not found: value cache_13
[info] not found: value cache_14
```


All tests passed on Windows OS


@roji i confirm that now nuget works, thanks for the fast fix


![image](https://cloud.githubusercontent.com/assets/381660/7746417/6cdcded8-fff5-11e4-92cd-be3147237d03.png)

;)


yaml uses a small caps convention for key names: http://de.wikipedia.org/wiki/YAML


Rewrite the above in applicative notation.


These are specific for the gd environment


the omnlib build was somehow deleted from the repository. after a travis rebuild the file is uploaded correctly


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


What is the idea in having the GD prefix? The file is going to be called `.gitdeploy.yml` so the scope should be clear imho.


For it to work with LP ROM.
you need to do this.
https://github.com/dorimanx/lp_ramdisk_dorimanx/commit/151d5d82d50a47da2baaec4ad64031220538dece


I guess I overlooked this check. Nice catch there.

However, the test only checks for the globally installed `codeintel` command. The way the code is structured currently, we make use of files in the _third_party_ directory for the CI completion. What this test currently means is that if you have the globally installed `codeintel` command then the code won't be installed to the _third_party_ directory and CI completions won't work.

I imagine that we might have trouble putting a global install command into the `ycmd` code-base so I doubt I can change where we install things. However, I suppose with some time I can look into a way of taking advantage of the global `codeintel` if it is installed as a backup.


Please note RainTPL4 has no stable release yet. Please be aware of possible issues, it's not recommended on production yet.

Currently those plugins - Compress, ImageResize, PathReplace are old, deprecated plugins that are not compatible with RainTPL4 until the compatibility layer will not be finished.


Simplify these definitions using already available functions.


Shouldn't it be a peer dependency instead (or removed completely)? I'm having a duplicate ESLint install because of this.


Hi, sory for that, it's my first. How can i test the translation before ?


Do we need to worry about fstatbuf.st_size not fitting in an Int? I suppose that would cause file truncation at 2GB.

Might be worth at least putting an Error in in this case? (I haven't looked at the surrounding code, to see how much damage this might do).


Thanks for taking a look! I didn't get to implementing that part yet. But `cacheMap` is supposed to contain all the needed info for this generation. And the caches are supposed to be shared across (multiple executions of) the base program and the derivative, so they don't belong in the AST but somewhere else (a bit like import declarations — I expect them to be class members).


The prace_common package is not available as a debian package in ROS Hydro


> we need to consider scenario where multiple DB's are used in one deployment, to avoid any env var conflict e.g. GDUserPG:=PGUser. Here also stick to Go naming conventions.

The idea is, that the GDUserPG variable is going to be bound to the env variable that you are going to specify:

`user: MGO_USER`

The username is going to be bound to the env var called `$MGO_USER`


specify procedures for all turtles as well as for only power-sheep, remove education


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


This can't work per se.


FIY: released couple of minutes ago


bad json !


What I meant in https://github.com/inc-lc/ilc-scala/commit/f03fa1eeafbb641f830f4acea5753cc1f090f63a#commitcomment-11302451, more concretely, is that this can't work per se. Instead of `addImports(toScala(t))`, one would need some `addImports(addCaches(toScala(t)))`, where `addCaches` prepends declaration of the caches.


cc @ironsidevsquincy


I don't know how this can end up being NULL, but I imagine you've found a machine where it happens, and this seems like probably the right thing to do in that case.


And sorry if I anyhow suggested that the code was actually working. Even if I wrote "Memoization of base terms seems to produce reasonable source code", by "reasonable" I didn't mean "working" :-D


You don't have to get members VO. Next method in a code ``.isAuthorized()`` can consume member and check against vo/group admin/observer role.


同じく starter_kit の方も修正しておきました :+1:

https://github.com/yasslab/railstutorialjp_starter_kit/commit/7effabcf3366b1f4e88ee92e68fde9017ebf5822


@marksev1, did a flight test today. See http://www.youtube.com/watch?v=cF0JNc3qq1Y. I suggest we continue to discuss this here https://github.com/digitalentity/cleanflight/issues/1


Indentation.


This might help a bit in your tests, and you can see what is returned with the different parameters:
https://stats.wordpress.com/csv.php


starting online educations for needy person


Good spot! It looks as though it might be. Hopefully I can get rid of that too.


no other comment for me besides Youri suggestion. Though it might not be very easy to check in a test that a session is renewed


`--short-arrays` would be much better, imho.


Later reverted in 95a5477


There is no need to change this part of code


Indeed.. I overlooked this problem. And I did notice that the loading screen progress stays at 12% or something but I didn't  find the time to investigate :( .


А, ёптыть...


missing


hmm, it is better


missing method


Seems they managed to create some issues of their own, since using 't' and 'f' is actually contrary to the SQL standard and how every other DB implements it.


Line 410: need to substitute $EPMD in the second place as well: "epmd - kill >/dev/null"


Warum sollte start_rserve in der gitignore sein??


:+1: for thoroughness; I'd have probably forgotten the Readme!

However, departments weren't the only things we seed, so we should leave this in as just:

``You will need to run `seeds.rb` to seed some default data.``


As we query mongodb to retrieve 'NOT CRAWLED' urls, I wonder if we would increase performance by using integers instead of strings. I guess that it is faster to query by comparing integers than strings (although it is less verbose).


Fixing typos *and* removing all the department stuff! :star2:


Yes it would. As would removing anything before the currently updating component. And that's fine, because that's not really the point of this. The point is to avoid crashes when the list gets modified, which this does. And that's only necessary because the ArrayBuffer doesn't check for concurrent modifications. The only way to do this "properly" that I can think of would be to copy the list of updating components, iterate over that, and for each component check if it is actually still in the main list before calling its update method. Which is way too much overhead for something that'll rarely apply anyway. The only case I'm aware of where this is relevant currently is Computronics' boom card, which just clears the list, essentially, so nothing needs updating afterwards anyway.


Maybe `SetAsync` would fit better? https://github.com/danielwertheim/mycouch/blob/master/source/projects/MyCouch.Net45/IMyCouchStore.cs#L80


Nit: these options doesn't appear to documented anywhere.


Это, конечно, через ж.

Почему бы в методе `show` не подгружать объект notice для текщего пользователя и потом уже работать с ним?

```ruby
@notice = @question.notices.find_by(user: current_user)
```


Erik verkade ha problem att få det att funka trots att det fungerar på min dator. Raden musicSet = true; i datastorage måste även ändras till false eftersom false tydligen är standarden för ej mutning


MIT - Feel free to use it. Thanks.

Tom May


@NicolasTr why is it necessary to pad the key, can't it simply be encrypted without padding? I assume the answer is because that's what the java implementation does.


This is the API key to an unused account setup using `mail+cakephp-wakatime@chrisvogt.me`.


Получается, что в `can?(:create, Notice)` мы просто проверяем авторизацию, а в `can?(:destroy, @notice)` будет авторство.


Thanks!


Why is this not in master?


@eyakcn
My driver depends on the Aliyun SDK written by myself, and I pushed the driver up here before I had time to deal with the *godep*.

Check the *Installation* instruction on this project: [https://github.com/jiangshengwu/aliyun-sdk-for-go](https://github.com/jiangshengwu/aliyun-sdk-for-go).

Copy the *sdk* folders to *Godep*, then you can work around with it.


A better design would be to pass self to Tile.open() in https://github.com/PyThingy/roguelike/blob/44725c23013eb5d75dd6b15e558dd55125ef9d02/rogue.py#L68, then override that function in Teleport class

This way the whole engine stays the same and it's only the Teleport who decides how to react to "open" action.


unresolved conflict


What's the preferred fallback behavior here? We could spit a note on stderr and quit, show a messagebox and quit, or start the program as if the option wasn't passed. What makes sense here from user's perspective?


Just noticed it after compilation test, The name of the variable is `CFG_HOST_ARM_SHORT_ENUMS`, so I have used it as to when this is on short enums are used and when off long enums are used. But in other boards, the reverse logic is used. So I had updated it to ON, after the fix in the stack :smile:


trh


`authLDAPCookieMarker` is set in the options, but `ldapCookieMarker` is retrieved. So `$ldapCookieMarker` is false.
Rather than fixing the wrong reference, I set the variable to 'LDAP', which seems to have been the intend.


Was my fault anyway, I should not have enabled useless javascript all the time.


博君一笑1.1.0版


Good to include linting fixes for this as well.


Any details about the crash, @MStr3am ? (is it related to this patch?)


Okay, I got it.


@lynnetye: Here you're opening the Stripe Checkout form every time an element with the `go-to-stripe-checkout` class is clicked. Doing it this way is risky for a number of reasons:

1. You have to remember to add this class to every element on the page that leads to the payment step. It's easy to forget to do this. For example, what if we decide to add a link to the summary that goes to the payment step?
2. It only covers click events. What if there's some sort of programmatic non-click way to get to the payment step? For example, if we load the user's data and she's already entered a time, contact info, and pet info, we might want to take her straight to the payment step. In that case, we call `VetPronto.main.displaySectionOf('payment');`, and whoops there's a bug -- the Stripe Checkout form isn't shown.

Instead of thinking, "The Stripe Checkout form should be shown whenever certain elements are clicked", it's more accurate to think, "The Stripe Checkout form should be shown whenever the payment form is shown, no matter how you get there." If your code is DRY enough, this is easy to accomplish. You'd have a single function in your code responsible for displaying the correct step. Whenever it's called on to show the payment form, it will show the Stripe Checkout form as part of that process.


Denna raden i game kan nog tas bort i framtiden eftersom vi sätter ett startvärde i variabeln i datastorage istället:

if (DataStorage.getInstance().getRoundLimit().equals(NULL)) {
            DataStorage.getInstance().setRoundLimit(50);
        }


Hi,
I could not get the Target folder and  kstorm-1.0-SNAPSHOT-jar-with-dependencies.jar file in the kstorm Dir
Please Help me out....
Thanks in Advance


if you know also remove the extra () for improved readabilty and consistency, I'm all in


i would revert the two making html default, not everyone loves jsonapi


summary of off-line discussion: if sandbox knows about add-sourced packages, those need to be available, or cabal will complain.  if the sources are there, packages will be re-built even though that's not necessary (there may be a good reason involving timestamps on files or there may not be), but we can live with that.


dotenv-rails is a dependency


Nit: I'm finding that I actually have to read the code in this method to fully understand what parameters it will return. In [viewer.js](https://github.com/mozilla/pdf.js/blob/master/web/viewer.js#L1273), a very similar function is called `parseQueryString`, which feels a bit more descriptive to me.
How would you feel about changing `_getParameters` to e.g. `_getQueryStringParams`, or similar?


এখানে safe কেন আসছে?


@lynnetye: Your CSS class names aren't specific enough. Ideally, they should end in something that describes what they're modifying. For example, `.client-city-state` is vague, but `.client-city-state-field` is more obvious.

Also, this is the contact form, so these fields should be named "contact-" not "client-". That's more consistent and obvious as well.


This line and the one above it are indented too far.


good


The PR had already been updated with the extra () removed from the call... you're looking at old data :)


Respect :D


This is I think OK. Then the direct embedding should convert this into `Take(transform(lift(users)), Const(_))`. We can meet today at 3 if you would like.


Oops.. ok I meant for this to be a devDependency actually.


Sorry The message read SD> card read error.

On 20 May 2015 at 18:16, midopple <notifications@github.com> wrote:

> HI, what did you mean with "SD Card INIT Fail"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kliment/Sprinter/commit/04d0b538c4428f5f6b5c36d41b63e6429a0636ff#commitcomment-11289154>
> .
>


I think the signature of `lift` should be `T => T` and reification would transform lift into the deep embedding.


Uninitialized data variable passed as default value to getValue function call causing application crash if setting "cookies" is not set.


I like that. I'd like more.


lift should be introduced before reification.


This is a good idea, but I'm not sure about the implementation. The hardcoded values seem a bit too fragile.

I still stand by my approach of parsing URLs (either passed by the user or grabbed from the pages) as I implemented in https://github.com/rbrito/edx-dl/blob/master/edx_dl/edx_dl.py#L117-L157.

This "normalization" may be extra, unnecessary work, but it may save us from platforms that are constantly changing. And sites are generally constantly changing.

I guess that I can extract/refactor that code as a function and send it as a pull request to this project, so that we can use it for our own purposes.

Please, comment on my proposal.


It seems this commit adds a regression : https://bugs.launchpad.net/mixxx/+bug/1455211


My name is Leonard De Santana


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


Ha, was just tracking down why I was getting:

```
*** Running /etc/my_init.d/002-postgres-initialise.sh...
/etc/my_init.d/002-postgres-initialise.sh: line 30: unexpected EOF while looking for matching `"'
/etc/my_init.d/002-postgres-initialise.sh: line 32: syntax error: unexpected end of file
*** /etc/my_init.d/002-postgres-initialise.sh failed with status 2
```

And then noticed this commit that fixes it -- Thanks!

BTW: I think I tracked down the 'total size downloaded' bug, testing a fix for it now.


@lynnetye: `<input>` tags are self-closing, so you don't need a closing `</input>` tag. People will expect you to know this, so you should add a todo to go through your HTML and clean this up.


have hardboot patch ??? ! KEXEC_HB_PAGE_ADDR  := 0x88000000


`find` in cygwin does not recognize `-perm +111`


testcase DONE: d2106cf


beep


Dear  josemariasaldana，i hava met the same problem .but now when i run the project in eclipse ,the project stopped.the problem  is "ERROR n.f.odin.master.OdinMaster-Agent authentication list (config option poolFile) not supplied. Terminating",i thought the reason is no poolfile,i did not see the DEFAULT_POOL_FILE in this project ,and i do not know where i should write the "poolFile" by myself ,my system is linux of ubuntu12.04.Can you give me some hlep?Thinks in advance.


@lynnetye: I saw you removed this in a later commit, but regardless "customButton" is a very poor name. Keep practicing your naming -- you need to start thinking about it more deliberately and stop committing code with vague names like this!


Nit: Personally I actually find `cleanup` to be a better name (and it's quite commonly used in the codebase).


Looks good to me merged


@lynnetye: Hmmm, `stripe.js` isn't an external library -- it's a JS file written entirely by you. It should be in the "js" folder instead of the "js/external" folder.


Shot of courcse... not Shoot


hi nicolay, i'm in project to stitch images using censure method, i have read your post and installed it, but i dont know how to use your library, im using geany for the compiler and my operating system is linux 12.04. please tell me how to use it. thank you


first change made to the page


The thing is that "updated" should show only when post is actually updated and not all the time even when post was published with first take and haven't been updated since. Existing approach might return some errors in Webmaster Tools in case post have never been updated but it will offer much more accurate information.


Both is supported:

``` css
:local(.className) :local(.subClass)
:local(.className .subClass)
```


OK, from reading the code, the `subsections` of a `Section` is a list of `Subsections`, right?

I guess that we could use some documentation of this, so that the data structures that we use are a bit clearer to potential external contributors.


OK, good.


How to pull 908c46a into local?


Why add `tools/hisat/extract_splice_sites.py` instead of use the one installed by `package_hisat_0_1_5`?


@lynnetye: Why are you running validation to make this field required? This will be super annoying for anyone who doesn't have special parking instructions (i.e. most people).


Hi,

Have a look for the project called libcensure_demo (or was it libcensure_example?) or something like that on my github page. It contains a full demo how to use both censure and the included tracker.

Cheers,
Nicolai
--
This message was sent from a mobile device. Please excuse borked language.

Am 21. Mai 2015 13:01:09 MESZ, schrieb anyongaseow <notifications@github.com>:
>hi nicolay, i'm in project to stitch images using censure method, i
>have read your post and installed it, but i dont know how to use your
>library, im using geany for the compiler and my operating system is
>linux 12.04. please tell me how to use it. thank you
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/rochus/libcensure/commit/b21921c04861169bb1dc75acaa3441b748cb0290#commitcomment-11303241


I think this fix fixes https://github.com/logstash-plugins/logstash-codec-json/issues/9


To make `jt test fast` fast.


Good renaming of the function.

I only think that we should, perhaps, mention in the documentation that we are trying to make the code mimic edX's structure of sections, subsections, units etc.

Also, the docstring of `extract_units` mentions that it "parses a webpage and extracts ...". I guess that we need the input webpage to be a very specific kind of webpage. :)


Wouldn't `with ConcurrentBuffer[...]` help


hmm i'm thinking if it wasnt cleaner to just copy the functions tarxf and getfilesize to the beginning of the script so the rest could stay almost untouched


Programming in R - assignment 2.


It should tell the user if `sudo` can't be found.


@lynnetye: The overall design of the modal looks better after this commit, but why did you make the modal background orange? It doesn't match any of the other colors on the site.


You know this is technically not correct right?

The (c) in copyright statements is normally used to indicate the year of publication, to aid people in understanding the legal implications such as when it expires. Just updating it every year makes it essentially worthless for this purpose.

Anyway interesting project you guys have here and will definitely try it out!


@Rbn3D @wesnick

How about upstreaming this? In this shape it will not be accepted. But how about creating a special class for ODM?


Sorry for the long diff. It's due to inconsistent line-breaks within the file... not all line-breaks are UNIX format.

The actuall change in from line 144 to 153. (144 to 147 is new; 153 is changed -> added programClass to response String)


Here you zip two lists and transform the zipped list into a dictionary. OK?


точно, исправил. Но такой проверки на создание подписки мало can?(:create, Notice). Если уже есть подписка на вопрос, все равно cancan показывает ссылку. как это сделать правильно?


When building ``makeguids``, there are two options not recognized by the gcc in our CentOS 6 server.

    gcc  -Wall -Wsign-compare -std=gnu11 -fshort-wchar -fPIC -D_GNU_SOURCE -I/home/david_yang/onie/build/accton_as6712_32x-r0/efivar/efivar-0.15/src/  -Wmaybe-uninitialized -DEFIVAR_BUILD_ENVIRONMENT -c -o makeguids.o makeguids.c
    cc1: error: unrecognized command line option "-Wmaybe-uninitialized"
    cc1: error: unrecognized command line option "-std=gnu11"

The gcc version is:

    $ gcc --version
    gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-11)
    Copyright (C) 2010 Free Software Foundation, Inc.
    This is free software; see the source for copying conditions.  There is NO
    warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.


За границы выйдет


>But this is to do not get (at least here).

I still managed to get rid of the copy ```torrent_info``` here.
Sometimes it is useful to return to the previously written code...


@yfcai would you please refrain from reverting comments you do not understand?


TeamCity xBIM :: Xbim Essentials :: Release [Build 46](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1659&buildTypeId=XBIM_XbimEssentials_Release) is now running


Yeah but... But... It works fine for me...


sa


TeamCity xBIM :: Xbim Essentials :: Release [Build 46](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1659&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:01:06


I have revert this change


good


I'd rather add an import, the code is already unreaadble enough as it is.

But good catch on IntMap.


Here, if you look closely, the `_extract_download_urls` is, "in spirit", doing an unzip of the dictionary.

I guess that we can improve that "constructing"/"destructing" thing, if I am not mistaken.


change this to,

if not(item[key] or isinstance(item[key], bool)):


OK.


Nit: Since the `timeout` value is non-zero (and the same below), how about replacing it with a CONSTANT?


> How to pull 908c46a into local?

It depends on how you work with git (there are multiple wokrflows). For best compatibility with this project you'd do the following:
```
git remote add main_repo https://github.com/dbsrgits/dbix-class.git
git fetch main_repo
git checkout <the branch you are working on>
git rebase main_repo/current/blead
```
then follow whatever git tells you about conflict resolutions (`git status` is always handy if you are not sure what is the state of things)


I'm not sure if it'll ever be a problem, but we call `start` here before `PasswordDialog` connects to `QProcess::finished`.


Hi Laura,

Danke für Deine Bemühungen. Ich würde gerne ein paar Sätze ergänzen bzw. verändern, um den Code of Conduct auch für BPoC's zu öffnen. Darf ich? :)

Beispiele:
1. Herkunft = in der Black und People of Color Community gibt es seit langer Zeit einen Widerstand gegen diesen Begriff, weil viele z.B. in Deutschland geboren sind, sozusagen in einer Diaspora leben, aber immer wieder gefragt werden, wo sie herkommen. Durch diese immer wiederkehrenden Fragen gibt es ihnen das Gefühl, nicht dazu zu gehören, auch wenn sie noch nicht einmal die Sprache ihrer Vorgenerationen sprechen.
2. Befähigung = Befähigung hat auch viel mit Zugang zu institutionalisiertem Wissen (z.B. Schule, Uni, etc.) zu tun, denn nur mit Zugang zu Wissen über Privilegien, kannst Du Deine Skills/Kompetenzen herausfinden und erweitern.

Liebst,
Thuy


ew, never commiting again with VS.


i think this commit should come earlier in the history than "finish is deprecated" and "remove deprecated utils"


@opdenkamp are you OK with this Ill submit PR asap.

The setting show Off  > 59 or 59.94 In TV settings


Last summer we decided that

```
if (condition) {
    body
}  else  {
    body
}
```
Was the standard, so everything was reformatted to match.


The names were changed for newer versions.
See http://redis.io/topics/memory-optimization.


I didn't pay close attention to this code, but it looks OK. One thing that we may want to avoid is to make low level decisions about handling/formatting files/filenames separate from the rest of the program.


This pattern of "actions" does go against some of the principles of by-the-book, RESTful design, but we find that sometimes breaking the rules results in a clearer, more intuitive API. There's plenty of debate about how to implement this kind of functionality:
- some advocate DELETE (but that's not technically correct, as the resource would still be available after the operation),
- some favour PUT requests to replace the resource with a new, cancelled version,
- some prefer PATCH,
- moving all actions to top level resources, as you described, is also an option (but arguably clutters the API),

but each approach has tradeoffs. "actions" are at least easy for integrators, and keep the intent obvious.


you have autocreated remote called KeenSoftwareHouse, therefore you should use
git pull KeenSoftwareHouse master
git fetch KeenSoftwareHouse master

exact sequence is
git checkout master
git pull KeenSoftwareHouse master
git fetch KeenSoftwareHouse master
git rebase
git push origin

and yes, you are right, in case you screw up your master, using hard reset is option.


It never gets here, right?


Well fixed. Someone wanted to see Game of Thrones :smile:


Also implemented gamepad controls.


I am also hitting this problem.  I couldn't see a bug report on svn.boost.org or a fix anywhere- can you confirm it's still outstanding, if so I'd be happy to file a report?


first argument seems to be container ID.


Same here, of course.


"updates the chroot /src with the hosts /src" is wrong

better is maybe "updates the chroot /src with the contents of the current sabotage checkout"


Please add also .DS_Store for non mac users


Please also add .DS_store


Please add also .DS_strore


Nit: Since this is a constant, let's move it to the top of the file (and change it to UPPER_CAMEL_CASE).
I know that this is existing code, but while you're here, let's improve it! :-)


All three of `get_ipam_cidr`, `with_container_netns` and `with_full_container_id` call `docker inspect`; once for the Pid and twice for the container ID.

Then `if_has_fqdn` calls it again to get the FQDN.

This is broadly in the same territory as #654; I'm not certain you can call it fixed.


It's looking very good, but it needs improvements. It didn't work for me when I tried `lxqt-sudo pacman -Syu`. Pacman's process ended before it should.


Create the PR and we'll discuss it.


errors are exception which are not intended to be caught, assertionerror is the specific one which says that some assertion on a parameter failed.


Right else on newline was the most common.


I'd say:
```
if region_name.startswith('cn-'):
    s3_host = 's3.%s.amazonaws.com.cn' % region_name
```


@Rbn3D Are we still even using sortable?


Unfortunately, `file_grep` does not count the number of entries.


oookayyy :)


right. I don't know how this could happen and which effect will have on the started subprocess. By now I captured the error and reported it after cmd.CombinedOutput(). Perhaps a way to kill the subprocess (and also a timeout) could be useful to avoid a total hang.


Except the cryptic commit message ...
`Reviewed-by: Joakim Bech <joakim.bech@linaro.org>`


it is, but inside definition_find_cpp_library it looks soley for the c++-versions of the library (soley for *_debug/*_release versions)


Will change it :smile:


Marrant ces commandes de créations de schema par requete HTTP :)


Yea, I'm just sceptical of tests like that, since we have no idea where the `InvalidOperationException` is thrown from.


This is suitably mild :lemon:


hmm I think it shouldn't fail then.


"sh" seems distinctly Unix'y.
I'm no expert at ruby/rocco at all (these files were contributed by @Sharpie), but (if this worked on Windows before) does this work on Windows?
From what I read, `system` does return errors - see http://makandracards.com/makandra/1243-execution-of-shell-code-in-ruby-scripts - and exec replaces the current context, i.e., it ends the script when it finishes.


The cron script is either not running, or is running and it's something to do with this local::lib bootstrapping thing.

that gave me a headache when building this, lol. i could run all the commands to import the database and start the webserver from the exec line inside the docker, soon as i tried to script it, it wouldn't find the libraries no matter if i exported the paths exactly as they are in the bash .rc.

i'm hoping the cron is not something to do with that and simply my lack of experience with it.
just in case, i've rebuilt it with carton and dumped the make files in a separate folder on here.


I renamed it to multicall. By now it's diferrent for some implementation details from docker reexec. For this reason I haven't vendored it. But, if the reexec implementation looks valid we can just swap multicall with it.

Under the copyright you can see an "inspired by..." line, but I don't know which is the better way to give the right kudos...


SYSID_MYGCS and FDDroneControlManagerMavLinkDefaultSystemId defined as 255


Could you also explain _why_ we've done it? And add a link to the JIRA.


Seems good. Tried to do it in your proposed way.


I don't think it makes a huge difference in this case, but here you go: https://github.com/open-keychain/open-keychain/commit/1651f9fb610122f25838e2e763abfb61fe2404eb
If you Google for "assertionerror vs runtimeexception" you will find different opinions, but I see the point that an error is not intended to be caught. AssertionError was implemented in JDK 1.4 and thus BouncyCastle uses RuntimeException.


I tried this approach with a `@Component` class created by [angular2-now](https://github.com/pbastowski/angular2-now).  I like the idea of providing the actual constructor function, rather than a string which can be used to look up/create the object, but `$componentLoaderProvider` assumes that `component` is a string.


Nit: Ideally this line should have been in the previous commit, since I'm assuming that the code won't work correctly without it!?

(When a PR consists of a series of patches, it's always good to ensure that the code actually works at every stage.)


Why you apply rotation? Isn't that work done by TF? Also why you don't used vector accessors? r0 etc looks bad.


One alternative approach is to have multiple `lift` methods that lift `T` to some `Foo[T] <: Rep[T]`. This could be configured through the library. We would therefore lift int constants to `ConstColumn[Int]` and lift `users: direct.Query[T]` into `lifted.Query[_, _, T]`. The default `lift` will lift to the root `Rep[T]`.

The problem with having one `lift` is that we don't have fine grained control over the result type of what's lifted. `slick.lifted.Rep[T]` has quite a sophisticated structure.


Similar to https://github.com/timvandermeij/pdf.js/commit/2d71758a066c087c7f120ec1a4fedaafbcc4857b#commitcomment-11303735.


Plese change view to another name


I am puzzled by this commit. AFAIK, there is no poppler-data-* folder in the repository, so why exclude it?
OTOH, modules is part of the repo and includes CMake stuff. Right now, it's not actively used yet, but this should change in the not-too-distant future. So I'm inclined to keep it in...


Doesn't the declaration of caches depend on `t`? If so, then `addCaches` doesn't have enough information to generate cache declarations. Maybe `memoizedDerive` should have a larger domain?

```
def memoizedDerive(t: Term): (Term, Caches)
```


Thx


sad


Currently `dummyCanvas` has global scope. Shouldn't the refactored code use `this.dummyCanvas` instead, since otherwise this condition will always be true (and as far as I can tell `dummyCanvas` is never declared in the new code)?


Hey @tbosch could you leave some thoughts on why for the browser builds the code is transpiled from TS to ES6 and then from there to ES5 with traceur? Why not always transpile from TS directly to ES5?


I wanted to parse each of the different items as you suggest, but I discovered that this schema does not work for all the cases e.g.
https://courses.edx.org/courses/course-v1:Microsoft+DEV204x+2015_T2/info
So I kept it in a string like that. However I agree that the idea of moving or improving a function with a more robust parsing of the URL is a good one (less hacky than those indexes).


refactored API


right, yes, I agree some doc is needed for these structures, i will try to put more of this later on.


Quick question: can we be sure that rocco is properly registered as a gem? As I wrote before, I'm no expert in ruby/rocco at all, but could it be installed in other ways (e.g., from a package manager or a direct binary download)?


As `go test` creates an executable per package, chroottar.ExtractTar will clone and exec this executable. Si it's needed to call the multicall.Exec() in it.
Additionally: I added it to the tree and store tests as both were using the tar extract (using pkg/aci/render functions) but will be enough to add it to one init func per package.


@gfth4x07

PEP8 issue here.

```
pingo/detect/detect.py:100:1: W293 blank line contains whitespace
pingo/detect/detect.py:102:1: E101 indentation contains mixed spaces and tabs
pingo/detect/detect.py:102:1: W191 indentation contains tabs
pingo/detect/detect.py:103:1: W191 indentation contains tabs
pingo/detect/detect.py:104:1: W191 indentation contains tabs
pingo/detect/detect.py:104:1: W293 blank line contains whitespace
```


Quoting myself:

> But cacheMap is supposed to contain all the needed info for this generation. And the caches are supposed to be shared across (multiple executions of) the base program and the derivative, so they don't belong in the AST but somewhere else (a bit like import declarations — I expect them to be class members).


:+1:

Added to this one and test 86


Hm! is "yield self.fire()" valid? or are you missing a self.wait() around it?!


Yep!


Hmm, I can't find anything like that, did you mean `SynchronizedBuffer`? That seems to be deprecated and suggests the use of a `ConcurrentLinkedDeque`... which I suppose could work, but seems like a bit of overkill? Since this is not so much a threading issue, as a "self-modification" one - correct me if I'm wrong and that is more light-weight than it sounds.


@gfth4x07

```
pingo/rpi/rpi.py:142:1: E302 expected 2 blank lines, found 1
pingo/rpi/rpi.py:143:1: E101 indentation contains mixed spaces and tabs
pingo/rpi/rpi.py:143:1: W191 indentation contains tabs
```


Yes, actually I create this dict from URLs, to its extracted resources as a source of 'cache' for the further iteration of subsections where I look for the given units.


Yep! Dumb idea xD Already corrected


using of "dynamic" could reduce code and maybe improve performance a bit for successive calls


errno is set by malloc on Unix systems


This presumably resolves #365?


*gui 이상있음! 수정필요


why? is there a design decision behind that? I can conform to this, but I thought it was more natural to use APPLE rather than Darwin, because I have no idea if this runs on other Darwin based Systems (are there any). I tested this on OSX.


This header file is also missing in uClibc 0.9.32.1.  Is it suggested to upgrade to 0.9.33.2 if we would like to enable UEFI support?


Тогда нужно передавать в `can?` сам `@notice`.

Можно так:

 ```ruby
@notice = @question.notices.find_by(user: current_user) || Notice.new

can  :create, [Notice] do |object|
  object.new_record?
end
```


I would name this method `LocationIsIn` and simplify the implementation to use `locations.Contains()`.


kk


c


fff


No, sorry that was a Mistake.


@levicole DWMR. Looks good. Press on!


   Варианты появления Лизы:
1. Она приедет вместе с Сергеем.
2. Она будет в лагере до Сергея.
3. Странноватый. Через некоторое время после приезда Сергея, в лес рядом с "Совёнком" упадёт Союз с Лизай на борту. Скорее всего у неё будет амнезия.

    Набросок сцены в автобусе (вариант появления Лизы 2 или 3)
Сергей открывает глаза и не доумевает, что произошло. Ведь он только что заснул ( Зависит от принятого начала истории). Через некоторое время он понимает, что автобус ему кажеться знакомым. Встав и пройдясь немного по салону он замечает в окне ворота лагеря "Совёнок". Резко хватается за спинку кресла, чтобы не упасть от удивления. " Какого чёрта"- говорит Серёга не отрывая руки от кресла. Затем подходит к бардочку водителя и дастоёт от туда пачку сигарет с фразай "Кто знает, что может пригодиться в этом странном мире!?"  Потом он выходит из автобуса, обходит несколько раз вокруг него и направляется в сторону ворот.


Я СДЕЛАЛ ЭТО!!!!!!!!!!!!!!!!!!!!!


УРА!!!!!!!!!!!!!!!!!!


Встреча с лизой (день 2 )

Гг - Серёга, а не Семён.


Карты на развевание

Э- Эй, Сергей! Ты в карты играть умеешь?


Good


Как-то странно слились коммиты. Ну да ладно, главное чтобы ошибок не было.


Задание сценаристам (Ответ)

Надеюсь это имелось ввиду под заданием


Г - Годно


Können wir nicht sogar nur eine Methode machen:

```ruby
def add(article)
  # overwrite existing article - who cares if one was there before
  # set the empty to false
end
```


For cross-reference: https://github.com/chobie/protoc-gen-php/issues/5


@bmneely DWMR. :+1:


@kingstien it's already possible to do that with current API:
http://libmodbus.org/docs/v3.1.2/modbus_new_rtu.html

I agree to merge your patch to simplify the API but *it breaks the current API*.
The question must be asked on libmodbus mailing list to know the opinion of Win32 users and the documentation must be updated.


I like that name better too, and agreed that ```locations.Contains``` should be used. Thanks!


@bmneely DWMR. Press on! Speed is your ally.


Vorheriger Code umgeschrieben.
Das Ausgeklammerte ist der "alte" Code!


@livingincode DWMR (done with my review). Looks good. Go as fast as can.


We do not have additional buttons, so this value can be ignored. If you want, I will define it to zero


Have added these cucumber tests to make the outcome more explicit. What do you think?


Je suis au courant ^^" Mais SourceTree arrêtait pas de me les proposer dans
les commit, à chaque modif...
J'en avais marre de les ignorer à chaque fois, du coup j'ai commit et il me
laisse tranquille :x

Jon GARBAYO

Le 21 mai 2015 10:26, VincentEchavidre <notifications@github.com> a écrit :

> ça sert à rien de les commit, il se mettent à jour tout seul.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VincentEchavidre/webdocCCFD/commit/769a033ca567257632592afe646f95e79a003320#commitcomment-11300852>
> .
>


Pushed in commit c4a8acd.


Wouldn't it be safer to use
```
 (or (symbolp arg) (functionp arg))
```
?
It is conceivable that the argument could be a symbol of a function that is not yet defined, but will be defined when the source is used.


I had to revert this change to make it work on my emacs 24.3, which is the latest one on Ubuntu 14.04. Is this reason to open an issue? Or is my outdated emacs to be blamed?


Can you normalize this one too?


@ctapobep, not exactly understood what you mean.
The link to the JIRA issue is presented in commit message. Explanation for reasone is also presented both at the commit message and javadocs. Could you please clarify what _explanation_ do you mean and where is the link to JIRA expected?


@knownsoldier, when you have time, take a look at this.


Should be apply to 3.0.x too ? (we should apply first to 3.0.x and then merge back in develop)


Master doesn't necessarily mean stable. But, the tags must be always stable. For jQuery 1.12 UI, a tagged version will be based on master.


Probably :)

I'll do it when I check there is nothing else more.


Ah, yeah, if you set a default then that would probably be fine.


@alexblair1 DWMR. Two issues fro us to fix. 1. google.com should go to google.com not run a search. 2. the toolbar disappears if you leave the app and then return to it. We can fix these when we talk on Friday.


Just compiled it and it works great, thanks.


Nice, looking good :smile_cat:


Changed port numbers because Linux does not allow binding ports < 1024 without superuser priviliges


@claudiuvertistudio I would not add the /* TOP NAVIGATION MENU SELECTED ITEMS */ here, just the bind on click.

I think is way to expenses and useless to call jquery on each scroll action and calculate those things.


你试试[http://c.3g.163.com/CreditMarket/default.html](http://c.3g.163.com/CreditMarket/default.html)
如果这个页面没有问题，你要注意你的body宽度在和width的值要始终保持一致。里面不能要超出宽度的元素！横屏页面会变大，这是正常！不理解看看原理的Slider.


I ended up using `Process.spawn` instead of fork. My thinking was that the `fork` style ends up starting up two new processes. Is that even legit?


Don't think semi colon is needed, but I'm fine with this now that the string replace is gone.


Looks good now.


@janacuq DWMR. Nice. Press on!


What's the difference in using the "INT" and "TERM" signals? I had used "TERM", but no particular reason.


Du bekommst ja den article schon übergeben, daher brauchst du hier keinen neuen erstellen:

```ruby
snickers = Article.new("Snickers", 1.95, ...)
section_3.add(snickers)
```


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

VSO State: Done


Hier fehlt das @empty komplett. Am besten entfernen, zumal aktuell die Klasse auch kaputt ist,
da du sie gleich überschreibst.


ouai, on pourrait les lancer en batch dans la même requête, je sais pas encore exactement comment faire, next time


Yup.  Good stuff!


Forgot to explain: Now, if host disconnects, instead of broadcasting an empty MatchResult, we update the host and let him send the correct MatchResult request. Not sure if there will be something missing in the result, but at least it will be better than an empty one.


This fix did not show up in the issue tracker because it didn't say
Fixes #83
(I think it won't work without the #.)


Yes. It's why you should add it into your playbook like the following
```
- hosts: all
  remote_user: elao
  sudo: yes
  roles:
    - { role: elao.motd }
```


Well done!


I though I had deleted this line when I set "defaultImageUrl = require('../images/default-building.jpg');" in the image component.You can remove the background-image line and then I suppose you don't need the loader class styles.


move to `Unidoc` plugin? It's not Scaladoc


Better than repeating the adapter name in all of the plugin assignments.  Still a little more repetitive than I was thinking we were going for -- It strikes me as easy to forget to register a command subclass, or flubbing a rename somewhere.

Maybe next step is to update the registration so it sets this value instead of doing so in each class definition?


>(non-const libtorrent::torrent_info).

renameFile() spoils the fun :-1:


@fxprunayre I remember having done a similar modification for ifremer-sextant ; my mistake, I should have checked here if it required the same fix. From what I remember a getBean(LDAPUtils.class) is made somewhere in the code, making the call fail if there are several different instances of the same class.


Very good catch and high level sbt wizardry, whoa :-)
So now filters work right?


Adding Junit for testing and Logback for logging.


Entity description:
small tip error - "seperate"
check "with" vs "within"


small tipp error - repsonse


Se te ha pasado poner do_best_stories() de la línea 291 dentro del if justo anterior


Entity KnowledgeItem - there is an already existing one in OGIT:
https://graphit.co/docs/group__httpwwwpurlorgogitAutomationKnowledgeItem.html
Could you reuse this one? Since all entity names should be unique, if what you need is another entity, please explain why/how would it be different from the existing one.
(Adaptations to existing entities in OGIT could also be possible from our side for better integration at this point.)


This fixes the issue where links containing a // are interpreted as comments and removed


You sure have a loud mouth calling other devs patch monkeys when you can't even properly handle Git commits.. but hey higher numbers right?


>because it requires thread synchronization

I forgot about this aspect. You're probably right.

>`torrent_status::torrent_file`

I hadn't seen this. torrent_status usually doesn't need to sync threads. It is super fast. So `torrent_status::torrent_file` might be a cheap operation again. I'll ask arvid in the bug I linked.

>return a copy of the same object throughout the life of torrent_handle, we could cache it only once.

torrent_info can change if we rename/remap files/add trackers etc. So we can't cache it only once. But everytime we get a torrent_status update we should update the torrent_info cache too if my above sentence is true.


I'll still need the loader styles for displaying a gray background before image is shown. The school-image element is not added to DOM before data is loaded. But I'll remove the background-image line :+1:


Please use "TabTab" instead "We"


I don't think that this is necessary. Description is defined for top level pages while summary is defined is for books and pages if the author has provided one. Is there a particular reason why you think this is necessary? Perhaps I am missing something.


Better use `L'\0'` since it is the correct type.


could you please add to entitiy's description the explanation about ogit/function attribute in the context of this entity


hey @mthli , thanks for your note and your nice repo.

ok, i will thinking about add third-part download link later.
**BUT, why I think there must be a way to fix that problem.**

Why not check out how [Javelin Browser](https://play.google.com/store/apps/details?id=com.nubelacorp.javelin) and [Flynx](https://play.google.com/store/apps/details?id=com.flynx) did this.


cool!


+1 :)


Actually the most easiest solution is to inject a custom `PutEntityHttpRequestFactory` with following adaptions to the `GenerateRelativeUrl`-method:
```csharp
protected override string GenerateRelativeUrl(string entityId,
                                              string entityRev,
                                              bool batch)
{
    var parameters = new UrlParams();
    parameters.AddIfNotNullOrWhiteSpace("rev",
                                        entityRev);
    parameters.AddIfTrue("batch",
                            batch);
    parameters.AddRequired("new_entries",
                            bool.TrueString);
    var relativeUrl = string.Format("/{0}{1}",
                                    new UrlSegment(entityId),
                                    new QueryString(parameters));

    return relativeUrl;
}
```


The problem is not both 22 characters needed and extra trailing $?  One of CB's messages seemed to suggest that the $ was also an issue.


1. this has nothing to do with you

2. at least i have never onced claimed im a professional in this but i at least try to document what i do and give the proper credits where due..

3. you were somebody i really respected ..

4. Fell free to lay more on me if you want.. im used to be a punching bag.. this may be a "pseudo career" for you- for me its really a MEANINGLESS game... Check real life around you once in a while, you may get a grip..


IT WORKS!!! HOOAH!!!


Ja, das ist mir auch aufgefallen. Hast du schon eine Idee?


The way (by using the filename) in the proposal branch is more robust against this sort of error...


This did not work in IE9


Fixed in c33d9627121399649c284d987647b92d8fe6552d


@higs4281 I will fix it after the pull


No, setChecksumType() is more or less for unit testing only. The Validator checks the config file/theming option.


test


Muito legal essa linha


Set this to true, since the slave control port is not used.


<3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3 <3


Actually I decided to use modules instead of lambdas, as there was no need to access the caller's context in the parsers code


This is beautiful ^^


This is the only major cast I think we can't avoid.


Changed my mind; the specific points of #654 are fixed so I created #725 for this new complaint.


This was part of our SEO changes. I believe I checked some content and
the meta description tag was not there. I may have missed it. Is it
currently being populated for each page?

On 05/21/2015 07:30 AM, Amber Michelle Webb wrote:
>
> I don't think that this is necessary. Description is defined for top
> level pages while summary is defined is for books and pages if the
> author has provided one. Is there a particular reason why you think
> this is necessary? Perhaps I am missing something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Connexions/webview/commit/1f6617465a765bb22d9de915c5f521429944bb2f#commitcomment-11304594>.
>
> !DSPAM:1106,555dcff1271841667077270!


I don't understand. Why would you push a tag without any branch associated with it? If its not master, shouldn't it be somewhere else?


Amazing, really


Don't we need another `udevadm settle` here, to make sure `/dev/mapper/cryptroot` appears before running `mkfs.ext3`?


Configure the lifting method, it might be nice to have stronger type guarantees on the method names.


this should not be set here.. it's a global parameter set in nbt_Group


不建議在原始碼中放入中文喔

可能會有潛在的編碼問題


Of course it has directly to do with my because I'm the one with the original claim:
http://forum.xda-developers.com/showpost.php?p=60831167&postcount=581

So basically you insult me by insulting him in your little hidden forum. If you feel necessary to call out other devs patch monkeys like that without even knowing proper context, then I guess there's a good reason people treat you like a punching bag.

FYI: Minor Linux patches are 99%% meaningless for mobile devices. Just count the amount of files which touch any actually compiled objects.

Good luck in your endeavours.


Fehler im Wiki gefunden und hier behoben


Build and run


I don't know either... but surely we can change it. ... done (will push later)


Lift transformation still happens after embedding. I am still not sure if we need to change that.


Also for the cpu_bridge...


Next time you create a new branch for a new feature please create it from the master branch. This is because the master branch works, or someone does not get the model. It also is because all other features are merged with this branch and therefore you keep the overview of what is happening. It also is simply because of convention.


All is POIFICK but I think you wanted to make the 'input block' partial before the form block partial


:) Yes... this is already changed in next commit (it was late in the night..)


cool :+1:


ok.


Is there any reason you don't do it inside the role itself thant require the user to add it either on the host section, either on the role inclusion itself like :
`- { role: ansible-role-motd, sudo: True }`

Is it a parti pris ? :)


Honest to god i thought he was completely lying about you and that he used your name because noone will doubt andreilux

i never ever meant it towards you and i apologize for that!! i never mentioned your name or insinuated it was you!!! in my quote of him, i purposely left out the reference to your name!!!

i apologize and thank you for your wishes

to thank you for all you do, will be nothing and just words and it wont express the gratitude...

i apologize..

i will revert this and issue an apology towards you


Jeroen is referring to [this](https://mobiliza.org.br/channel/aleabdo/?f=&mid=31a64067afa20b8ab1996e4092ae2781f6c437fb68c0e8d610334f7f4d83af59@mobiliza.org.br this)

Not sure Jeroen, the leaked messages from that thread still show up in my zotfeed for a public observer after applying this patch.


This is necessary to comply with slick. `C` is the container type of the resulting value, like a `Seq` for a select * or single value if we do a `sum` on a column.


It should be populated for each page if it has a summary, else the description should be 'An OpenStax CNx book'. media.coffee line 64


好的，我想先放著當注解，跟大家討論說明後移除，還是都不用放好了?? 大家習慣直接看code


im not that good at git obviously... i wish i was it would make my life easier.. reverting it will be stupid.. ill completely "amend" it like it was never committed..


Run
Build


Still not sure how to optimize horizontal part, because 'outerHeight' returns undefined for all samples, so this.tooltip.style.top is allways set to 'NaNpx'


не, здесь надо TransactionsFragment.builder().build() https://github.com/excilys/androidannotations/wiki/Enhance-Fragments


der initializer den brauchen wir weiterhin, er muss ja open = false setzen


LGTM


das ist genau richtig


Let's just wait for Mike's reply. @redmatrix


die 3 methoden haben hier aber nichts zu suchen, das kommt später von aussen rein.
Niemals Anwendungsdaten direkt in die Klassendefinition.


und den require hier wieder entfernen


Usually, projects wouldn't do it I understand. But, for DB, what benefits we lose?


The idea is, when homing, to just move the Z-axis the same amount that has been babystepped previously (volatile or EEPROM), and still consider that point still on Z=0. This is necessary since one may have minimum software endstops enabled but also babystepping enabled, the user may be in a situation where they need to babystep up or downward closer or farther to the build plate during each print (the extruder is too high or low relative to where they would like the nozzle to be at the origin); this could not be accomplished with G0/G1 movements if minimum software endstops are enabled.

Also I changed it to use this newly defined BABYSTEP_OFFSET macro:
https://github.com/gralco/Marlin/commit/dddc071857f6e23ac5dd80d630f6a82898d531d5


А 4 зачёт


This is only set once inside adapter. You don't care about it as a user of an adapter.


I don't see a reason for the `if (node)` check (isn't `node` always defined?), and I don't see why you need to set `visibility`.   Isn't setting `display` sufficient?


I checked Concepts of Biology on beta and cnx.org. The description is correct in beta, but does not show in cnx.org.


If you had done that you could avoid some HTML, and this would be:

```erb
<%= partial :"groups/input-block", locals: {
  label: label,
  inputs: inputs
} %>
```

and then all the stuff you have from lines 6 -> 16 could go inside the `groups/_input-block.erb` partial (keeping this form-block group more abstract).

The heirarchy would then be:

```
form-block
--> input-block
   ---> input
# And you could add more input blocks and inputs inside those
--> input-block
   ---> input
   ---> input
```


Ok. My fault for not catching that.

On 05/21/2015 07:43 AM, Amber Michelle Webb wrote:
>
> It should be populated for each page if it has a summary, else the
> description should be 'An OpenStax CNx book'. media.coffee line 64
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Connexions/webview/commit/1f6617465a765bb22d9de915c5f521429944bb2f#commitcomment-11304836>.
>
> !DSPAM:1106,555dd308271844694712935!


The problem we had, is that most of our code is straight C99 and the C++
compiler can't compile it. When we needed to call a function that was
mocked from our C code, since the function was static C++ it didn't work.
This way the function is exported and the linkers does the job of finding
the mocked function.

On Thu, May 21, 2015 at 5:57 AM shlomimatichin <notifications@github.com>
wrote:

> hi sebastien,
> can you please explain why this was important to you? my original logic
> might not be very relevant, but i would like to understand this issue
> better.
>
> the original idea was to turn all global functions into "static inline"
> header implemented functions. yes, i realize that's not how it works right
> now.
> thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smsisko/Voodoo-Mock/commit/04ab662ce13a0c77680c726d90e1fc8d34f74b00#commitcomment-11302246>
> .
>


Я это не коммитил перед пушем...


@pSub @osener `error: attribute ‘cc.gcc’ missing, at "/home/travis/.nox/nixpkgs/pkgs/development/tools/thrust/default.nix":10:7`


ack


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3744/console to remedy the problem.


**title**


Is this part thread safe? (replenishing and id_type())
Cause this looks like a race condition to me


For some reason, FAE suggested to set this to false for 14.x designs even in the reference design it was not connected. I don't understand why though?


This interface doesn't make much sense considering ErrorOccurredDecorator. ErrorOccurredDecorator already has a method to signal if the postProcess should not be done.