Hi Steve,

I've been using 807 for purely historical reasons but should swap to I33.
I'll start doing that for MAF.  Oh .. but the database positions were all
calculated using 807. Hmm.

I'm using de405 in openOrb/MAF.
I believe I am using all planets for perturbers, although it is just
labelled 'n-body' .. I'm thinking this is all planets but maybe not Pluto,
Ceres, Pallas, Vesta?

Could this be a time issue? I have been treating the "MJDs" reported in
opsim as TAI.

Lynne

On Mon, Sep 14, 2015 at 4:18 PM pveres <notifications@github.com> wrote:

> @rhiannonlynne <https://github.com/rhiannonlynne> @ivezic
> <https://github.com/ivezic> @mjuric <https://github.com/mjuric>
> @stevechesley <https://github.com/stevechesley>
>
>    1.
>
>    I found out that you use obscode=807 for LSST simulations, however, I
>    used I33 (which is Cerro Pachon). We should use the same obscode. Why do
>    you chose 807? I thought LSST is built atop of Cerro Pachon?
>    2.
>
>    Even when I use 807, I get very similar discrepancies in RA and DEC
>    between our results, see figures:
>
> [image: compare_lsst1]
> <https://cloud.githubusercontent.com/assets/13923689/9864145/513c6930-5afb-11e5-82db-565dfb71fe30.jpg>
> [image: compare_lsst2]
> <https://cloud.githubusercontent.com/assets/13923689/9864148/542ea540-5afb-11e5-94bd-2dc8717d7cc1.jpg>
>
> We think that ~0.5 arcsec differences are too much, and also there is a
> systematic offset (the peak is 0.56", 0.35" of the 0,0 in RA and DEC
>
>    1. In OpenOrb and MaF:
>    2. what do you use for planetary ephemerides? de405?
>    3. how many perturbers? We use all planets + Pluto, Ceres, Pallas,
>    Vesta.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/testMOPS/mops/commit/f995cb0253c75dcac003e68a62c813c8c5fb22b7#commitcomment-13225566>
> .
>


можно if не пистаь, сразу return условия возвращать


Hello @MarkRegalla27
cc:  @jonhanke  @julialintern

----
Really nice job in completing these advanced python exercises!
Q2 - there seems to be an unintentional typo in the original csv file. Professor Ross has title: 'Assistant Professor is Biostatistics'; there should be 3 unique titles.

Thanks for formatting explanations and results in an easily readable manner.  It is greatly appreciated and makes my life easier.

----
Some optional extras if you have the time:

a.  [Explore 'defaultdict' option in python](http://stackoverflow.com/questions/5900578/how-does-collections-defaultdict-work)

defaultdict means that if an key is not found in the dictionary that instead of a KeyError being thrown a new value is created. The type of this new pair is given by the argument of defaultdict.

b.  Explore pprint in python
Prints output in a more readable and structured format.


**func-names**: Missing function expression name.


**indent**: Expected indentation of 6 characters.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for topOfElement


**no-undef**: "componentHandler" is not defined.


**new-cap**: A constructor name should not start with a lowercase letter.


**no-unused-vars**: React is defined but never used


**no-multiple-empty-lines**: Multiple blank lines not allowed.


**no-undef**: "expect" is not defined.


**no-trailing-spaces**: Trailing spaces not allowed.


Maybe just a list of bibids going into a directory?  Also, I have a subsequent commit to this branch with a comment on it with the sed command to remove the wrapping zs elements from the xml.   Tho the command to remove the elements at the end is currently sensitive to the number of chars in the bibid.  I'm sure @jgreben can figure that out.  :smile:


The original <code>&mdash;</code> version of this post no longer causes an issue in the feed with my fix https://github.com/StackExchange/stack-blog/pull/174 (But @hairboat, please confirm the feed was the only thing which broke!)


static was removed (which jk re-added)

const static (current)
vs
const (your change)
vs
static (old change with compiler error)

https://github.com/12345swordy/spring/commit/b19c78e76c95bd9fd8ad54ef284eda131b0e43fb


I didn't notice this because it didn't affect gradle test
On Sep 14, 2015 16:18, "kryptodoge" <notifications@github.com> wrote:

> Yeah, I'll handle the jni updates
>
> On Mon, Sep 14, 2015 at 4:16 PM, Robin Zhang <notifications@github.com>
> wrote:
>
> > I can do this, although I think @makosblade
> > <https://github.com/makosblade> will handle this later.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/kryptnostic/krypto/commit/5a956395f5af89fee4a809864e731cd3e6227151#commitcomment-13225547
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kryptnostic/krypto/commit/5a956395f5af89fee4a809864e731cd3e6227151#commitcomment-13225562>
> .
>


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


Did you add the corresponding tests in test_views.py, test_cli.py and test_client.py?


The leading and trailing `.*` can be removed. Ruby's `match` behaves similarly to Java's `find` in that the entire string need not... "match".

Also, what do you think about a more descriptive name like:
```
THREE_NON_ALPHA = /[a-zA-Z_-]{3,}/
```


I can add those if you want


you missed some symbols like d\' check xml carefully pls . thanks


fast-forwarding master to auto = 072359f2


```
@property
def row_total(self):
    return self.qty * self.product_price
```

asi te evitas de agregar `row_total = property(_get_row_total())`


Be good (in the continuous integration style) to add the code that found the bug as a test?


Thank you ! I love what you guys are doing with React.


I was waiting for my other project 1000 tests to run. If you have time, don't hesitate to send other pull request.
I cheated, this one was easy, you coded it for me ;)


LGTM


Adding these files to .gitignore might be better as a separate commit


What's the reason for commenting this file out?


Excellent choice of package name, Zach.


I like this. I'm going to merge it in, but i'm going to refactor the code a bit to clean up the controller.


me parece que aca no es necesario agregar una relacion, quiza con un campo tipo `enum` con los `statuses` estaria bien no?

```
    STATUS_CHOICES = (
        ('STATUS1', _('Status #1')),
        ('STATUS2', _('Status #2')),
        ('STATUS3', _('Status #3')),
    )
    status = models.CharField(
        max_length=16,
        choices=STATUS_CHOICES,
        default='STATUS1'
    )
```


I definitely want to get around to implementing the other changes.

I have to finish up my actual project for work that uses this framework (which is working out VERY nicely, by the way). I should be finished within the next couple of days. Once I am good to go there, I will make the changes we've agreed on and issue pull requests.


`camelization` was a typo. If I remember right early versions of this package just checked for a truthy value so that might be why this slipped through


nice


In which case we need to make Nnedi3.Scaler's `Active()` method consistent - i.e. Status should be assumed to be Active if it's null. Otherwise NNEDI3's chroma status would be wrong when NNEDI3Chroma is selected.


94006714-MultipleTowers (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1869 of 1869 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)


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1336 of 1336 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-70.2%25-green.svg)


It may be better to check if the gcovr tool is installed and at the expected version in the configure script. Further, it may be useful to configure the coverage target as a configure option.


esto se puede hacer con un `aggregate` tipo `COUNT(items)`


Is there a way to perform this same check using gcov directly in a few lines, or is gcovr necessary?

Do you happen to know if gcovr is currently being distributed in a package for any linux distro?


Hi @GregMoney85
cc: @jonhanke  @julialintern
--
Great job on these python exercises!
Thanks for updating the format in the markdown file.

Adding:
Q2:  Sets - have unique elements


If things are going as they should the Status shouldn't ever be null, at least not for long.


@mscappini Thanks ! I really appreciate. I didn't want you to think I was not caring about those issues, that's why I fixed this one :)
I'm not bumping the version number just yet. I'll wait for other changes.


Test file


But nothing ever changes the `Status` in NNEDI3 chroma scaler?


awesome :). Does it play when the game is lost ?


@nikolai-b this one's for you!


Gitignore fixed, but I thought it's all about coding, new features development,  oo design, not gitignore skills.


True, fixed already.


Keep in mind that NNEDI3 Chroma Scaler's `Status` is also used in NNEDI3's `Active()` method, not just `MakeFilter`.


This could be written instead as:

```js
var communitiesContent = document.querySelector('.communities-content');
```


This file is not complete


same as above


Its better to handle entity-related stuff in dedicated classes(managers). It also helps keeping your controllers thin. Take a look at FOSUB/UserManager for example.


Please help


Instead of using `.onscroll`, let's use `.addEventListener`:

```js
window.addEventListener('scroll', function() {
```


Does this actually allow the special flowers to grow by using bonemeal on grass?


Could have just injected whole $comment here.


You are creating a new level of indentation, so the rest won't be executed if the path is set. Please give the if statement the proper level of indentation.


It's also used here:

```c#
            public override string Active()
            {
                var status = string.Format("{0} {1}/{2}", base.Active(), s_NeuronCount[(int) Neurons1],
                    s_NeuronCount[(int) Neurons2]);

                var chroma = ChromaScaler as RenderChain;
                if (chroma == null) return status;
                var statusFunc = chroma.Status ?? chroma.Active; // proposed changes
                var chromaStatus = statusFunc();
                chromaStatus = string.IsNullOrEmpty(chromaStatus)
                    ? Renderer.ChromaUpscaler.GetDescription() + " Chroma"
                    : chromaStatus;

                return status.AppendSubStatus(chromaStatus);
            }
```


For the record, and for what it's worth, this wasn't a typo but is the capitalization used in the IPP API Integration Guide v1.1. The remote test case used to pass with that capitalization. Presumably IPP have changed the API in the meantime without preserving backward compatibility.


so far so good, just need to:
1 ) make it so that you're not defining https://github.com/MHBauer/docker/commit/1b94bca136712e3fc745ddd74015554a0f3a9797#diff-3cff687e71c03682360fd6b422e7d5c0L45 on a Server object - it should be local to the API package, not Docker's
2) you register your impl with the API package.


I think he was imagining a feature where an instructor might change a user from being an assignee for an article to being a reviewer for that article, or some similar role-change operation that we never implemented. @ntdb: does that sound right?

Given that context of when it was introduced, I'm pretty sure it's dead code.


Is the question "why are you testing this condition?" or is it "why is is zero instead of null?".

The reason the assertion is there is to ensure a paginator isn't returning the count of something other than the models. (like number of pages)


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 622](https://teamcity.labkey.org/viewLog.html?buildId=272073&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 32 (2 new), passed: 9 Build time: 00:26:48
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 22 more failed tests, see build details
```


The constants "X" and "Y" are duplicated all over the code.


Note that this works well, except the profile field saving functions. So review the code and modify it to your needs.


Why not stick specific use shaders like this directly in the object that uses them or in the module specific shaders.cfg file instead of filling up the default anura ones?


I like this way of doing the win check, but don't like the name of the variable.  This is really a list of "winning" combinations.  It isn't a "win_check"


I think this belongs in feature/zammit


I have done this to show the data can be different. For example, there are the author's username as string, and to retrieve it from the comment entity the query to db is needed (lazy load).


Yep and I deleted that commit :-)


Formatting issue here.  It is imporant all the if and elsif are the same indentation.


@linuxbo I find I am a slower adapter of new technology. If I found something that works for me, I tend to stick with it until I am forced/motivated enough to move on.

I personally have no use for the iPencil, so I wouldn't probably use it. However, I can totally see some people I know using it to draw sketches on their iPad, and I'm sure we will be seeing a lot of apps for that soon. iPads are pretty popular, so I could see the iPencil taking off in certain markets, like with artists.


What's the point of multi-line `splitText` after this commit? It just draws each line on top of others.


The joy of force push, I added this to the proper place (feature/zammit obviously)


You should be able to collapse this in to 1 if statement with an or in it rather then doing an if and elsif.  It was weird to have 2 different cases in the conditional that both do the same thing.  In fact, if you do that, you would see that you can return the conditional itself.  An if statement is not even needed.


Ah right, the "CreateChromaFilter" method doesn't call MakeFilter...
Okay, then things might get weird.

I fear it might be necessary to choose a different method after all, things are getting a little confusing.

FWIW I've been experimenting with a way to add "Tags" to the filter chain. The idea is that every RenderChain adds a tag to the filterchain, and afterwards you gather all the tags and end up with a nice graph representing the entire chain.
Only I'm having some problems with tags getting lost when parts of the chain are optimized away...


Excellent! No zammit-changes should be ever lost. They must be persistent, even if the universe collapses.


Here can be not the only comment data.

Btw, i wrote some explanation about this part of work (notifications) to the boris' email. Read it tomorrow, please.


Instead of 5 lines of code this check has almost 100 which makes reviewing it anti-fun.
Also it's better to rely on HTTP method rather than `isXmlHttpRequest()`.
This is example of how it should've been done:
```
        if (!$request->isMethod('POST') || !$request->isMethod('PUT')) {
            return new JsonResponse(array(
                'message' => 'Access denied'
            ), 403);
        }
```


It is important to be contentious about whitespace.  This is a newline that makes the code read weird.


what is mandrilling? detailed notes!


mapとenumerate使うのが吉。
これだと、word_1とword_2の大きさが異なる場合は意図しない動作を起こす。


so many semicolons missing in here!! i know you love ruby, but this it's javascript world yo


リスト内包表記で書くと、一行で簡潔にかけると思う。
pi = [len(word) for word in words]


If I use it to upload a windows remote-assist invite file, generaled by commandline " msra.exe /saveasfile xxx..msrcincident  password", and the file include a USERNAME that is chinese, encode by UTF-8。
![xml](https://cloud.githubusercontent.com/assets/11346056/9841172/2a7c96de-5acf-11e5-8d61-e79196f83147.png)
The upload session will hang. Then I found the $iOptionalLength isn't correct because Double-byte Character Sets


okok.

I’ll check it through…sorry

> On Sep 14, 2015, at 11:01 AM, Fer Martin <notifications@github.com> wrote:
>
> so many semicolons missing in here!! i know you love ruby, but this it's javascript world yo
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/WDI-HK-8/VALA-app-valet/commit/f8498d77591a22bee7a7d8ab4e1338b52c789fef#commitcomment-13204381>.
>


Có 1 số vấn đề nhỏ mà anh nhìn lướt qua nên có thể sót:
1- Với promise khi nào có lỗi thì phải gọi hàm reject chứ không dùng resolve nhé. Ví dụ
 dòng 1042:  resolve2("coverageLayer.rtree_cachedTile undefined");
2- Nếu em dùng setTimeout với Promise phải chú ý tới khả năng là một promise bị resolve 2 lần (vì bản thân setTimeout là async). Anh thì không thích dùng setTimeout với Promise, thông thường nếu em muốn dùng retry như thế thì em nên nhớ là khi retry mà thành công thì tất cả các lần gọi tới retry đều phải stop

// Them 1 biến resolved
var resolved = false;

function retryLoadImage() {
    if (!resolved)
                                            setTimeout(function() {
                                                if (countTimes > maxTimes) {
                                                    // -- cannot load image.
                                                    console.log("cannot load image");
                                                    if (!resolved) reject("cannot load image");
                                                    resolved = true;
                                                    return;
                                                } else {
                                                    if (e.target.complete) {
                                                        console.log("retry load image");
                                                        ctx.drawImage(img, 0, 0);

                                                        var obj = createImageData(coords);
                                                        var imageData = obj.imageData;
                                                        var pos = obj.pointTileCanvas;
                                                        ctx.putImageData(imageData, pos[0], pos[1]);
                                                        // ctx.putImageData(imageData, 0, 0);
                                                        tile.img = new Image();
                                                        tile.img.src = canvas.toDataURL("image/png");
                                                        console.log("retryLoadImage");
                                                        if (!resolved) resolve(tile);
                                                        resolved = true;
                                                    } else {
                                                        if (!resolved) retryLoadImage();
                                                    }
                                                }
                                                countTimes++;
                                            }, 50);
                                        };

Thêm biến resolved để cho biết cái Promise đó đã bị resolve rồi thì không trả về nữa. Em cung nên check lại 1 số chỗ xem có cần thiết dùng setTimeout không


for num, word
カンマのあとにはスペースを入れる。
pep8整形ツールを使用するといいですよ。


If you don't have a make or license_plate required, this may help you have a more clear output:
```html
Car:{{currentPickUp.user.car.color}} {{" ~ " + currentPickUp.user.car.make}} {{" ~ " + currentPickUp.user.car.license_plate}}<br>
```


Now that you've got the basic form in there, let's revert back to the old style we had with some updates. I changed the opening `<form>` tag as well as the form group for email address. See if you can figure out what I carried over, and then apply that to the name, major, and reason fields.

```html
<form role="form" action="//iit.us7.list-manage.com/subscribe/post?u=8f0638c241465fc4dfa51a387&id=51d8e108dd" method="post" target="_blank" novalidate>
  <div class="form-group">
    <label for="InputName1">Your Name</label>
    <input type="text" class="form-control" id="name">
  </div>
  <div class="form-group">
    <label for="email">Email address</label>
    <input type="email" value="" name="EMAIL" class="required form-control" id="email">
  </div>
  <div class="form-group">
    <label for="InputSubject1">Major/Field</label>
    <input type="textl" class="form-control" id="major">
  </div>
  <div class="form-group">
    <label for="message1">Reason being interested in joining IEEE@IIT</label>
    <textarea class="form-control" id="message1" rows="3"></textarea>
  </div>
  <button type="submit" class="btn btn-theme">Submit</button>
</form>
```


+1
Great catch and fix!


@jrperritt So, I'm not sure how to unit test go functions that read files and arbitary URL's. Are there some guidelines on how one can do this?


意味のないコメントは削除


pythonっぽくていいと思います。
iterとして返すなら、メソッド名を考えるといいかもしれません。


please name it userAccountRemoved and take a look at https://github.com/TelescopeJS/Telescope/pull/1141/commits


Fixes #2


コメントの書き方もpep8に則って書いたほうがいいです。
pydoc作成など自動でできるので、オススメです。


Let's talk about the future of the United States of America?


I've heard there is upgrades on Marlin with auto leveling...where i can get this upgrades?


Let's talk about Donald Trump people.


Não seria melhor deixar "generator" no lugar de "restaurant", visto que a ideia é que o sistema cresça e pode vir a utilizar outros estabelecimentos além de restaurantes.


pythonらしい書き方でいいと思います。

メソッドの戻り値もこういう書き方ができることを頭に入れておくとなお良いと思います。


不必要なコメントは削除


return "%d時の%sは%s"%(x,y,z)としたほうが、明示的ではないでしょうか。


I wouldn't say that GTK UI behaviour is a good one.
For me it's not quite obvious that double-click should start playback. On the contrary, I got used to editing tabs with double click. And by the way, I get no context menu at all.


97668326-WholeBuildingLoad (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1853 of 1861 tests passed)

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


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/f6e516ff41e982e8297ae0e3ca92c1bc36a4cdd0


merging basho/riak_cs/bugfix/gc-batch-waiting-nothing-forever = f6e516ff into borshop-integration-1247-bugfix/gc-batch-waiting-nothing-forever


basho/riak_cs/bugfix/gc-batch-waiting-nothing-forever = f6e516ff merged ok, testing candidate = 792a2692


You don't want this to work on 0.5-?


この条件判断および戻り値を別の関数にすることで、プログラムがもっとスッキリすると思います。

def char2cipher(char):
  if ~:
    return
  return


In the `function reverse`, I define a `var o` that is equal to an empty string.
Entre parenteses in the `for` loop, I have three elements.
The first one `var i = s.length -1` is the starter of the loop. In this case, the loop stars at the position that is equal to the length of the parameter `s` minus 1 (the last character of the parameter `s` is at this position because we start counting the characters with the 0 position).
The second element tells the function to run until it reaches that point. In this case, until it's greater than o equal to 0, so until it reaches the first character of the parameter `s`.
The third element tells the function to do negative increment  after each loop.
With this part, the function works for all the character of the parameter `s`.
Then `o += s[i];` The variable `o` has been defined as an empty variable. So each time I run the `for` loop, the variable `o` becomes the character that is at the position of the variable `i` in the parameter `s`
And finally, the function returns the value of the variable `o`.


email thread fail

On Sun, Sep 13, 2015 at 10:59 PM, Justin Searls <notifications@github.com>
wrote:

> I thought we were talking about the global thing
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/testdouble/testdouble.js/commit/3e86bce75e8c0a52880dc7c30a811786e613d770#commitcomment-13204375>
> .
>


行頭にshebangを追加すると良い i.e.
```python
# /bin/env python2
```


- .gitignoreを利用し, swapファイル等の中間ファイルはコミットしない
- add . を 使わない


fixes https://github.com/ucldc/potto-loris/issues/2


Sorry that was kinda ambiguous, looks like the docbloc is out of step w/ the function definition (PouchW vs server)?


スライスの1つ飛ばしの仕方は
words[::2] のように書く


It was not indented, I made my own commit after the merge request created in order to get a dist working copy of mine while PR merged, will move that to another branch.
Not being a github expert didn't realize that PRs will get any commit made later than PR creation, amending that.


I'd say you should test both -1 and another negative value. The ```.insert``` bug in ```shape_padaxis``` illustrates well why only one negative value is insufficient. axis=-1 works now but who knows when will someone break it. :)


Why is this not the same as `pindex->GetBlockTime()`.


Add a method to unfollow your oldest friend every time you follow a new one on a First-In-First-Out basis.  Also changed the name of the DeleteOldPosts method to ClearQueue.


It's just a comment, the exercise is fine.
You should consider **avoiding the use** of the **`<div>`** element, only to use its default display property .**In this case** it's cleaner to **use  `<br>`** element to make a new line.
`<div>` should be use as container of a group of elements to apply common style or behavior.
:+1:


文字列の連結は なるべく join (ネストしまくるようなら for でもよい)
itertools の zip_longestを利用すると良い(fillvaluesにNone代わりに''を指定する)


pi = ''.join(map(len, sentence.split())
でいけるはず


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 607](https://teamcity.labkey.org/viewLog.html?buildId=271871&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:25:29
### Failed tests
```
org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 20 more failed tests, see build details
```


It might be a browserify compulsion, or a jshint/jscs compulsion.

Oh!! It just came to me while responding... I remember the *other* reason I'm explicit about global/window. `notAThing` will throw a `ReferenceError` if undefined but `window.notAThing` doesn't (it's just an undefined property reference).  When writing libs, i'd rather coerce to an evil, hard-to-trace `undefined` than throw exceptions in userland.

So I guess a combination of reference error avoidance, browserify behavior, and jshint/jscs lashings have gotten me in the habit of being explicit about globals. Ironically, the (only ?) time I still use implicit globals is in my tests. Which started this whole convo. woops


Acho que seria bom colocar a mensagem de confirmação que irá aparecer


I had the definition of string::substr() wrong,  I thought the two values were size_t indicies for starting and ending values but it's the starting valuing and the length of the text. Corrected in next commit


Thanks for looking this over. I'll have a look at integrating this approach into the master branch. What do you think of the additional commands I've added (specifically the option to do a partial clean/fetch and rebuild, and the --setup-cache for a full reinitialization)?


yes, i think this is the reason.  I used to   cp .vimrc to .xvimrc . i delete some lines in xvimrc which use vim plugin. then it works . thanks


I still think you should use the Qt versions.  They are pretty slick.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1615


sqrt grows way faster than log, so it'll be interested to see how this scales to longer TCs (and thus higher depths).


I'll make sure to do that next, I didn't see your last commen :) .  The reason why is used std is to use the find_first_of function, but now that the algorithm is written already and working I'll switch over to the qt versions


Good point, you're correct I didn't think about that :) as having the aggregator test send the posts in a transform.reduce will create 4 batches of 1 request instead of the expected batch of 4 requests.

IMO this is still better than not going through the batching code at all, but we can probably do even better...

I can try to change that transform.reduce to a transform.map to make the batching effective again, and handle the out of sequence cases in the test logic that checks for the aggregated usage values. That way the test can still exercise the batching without being sensitive to timing issues.

I'll push the updated code to my dev fork later this evening.


cool, yes that might make it a little easier for users to configure. Also I realised after I submitted the pull request that if you accepted my commit in, it would have changed the status line color for all users (to how I was using it personally). What I should have done was left the color as per default - but just add the *ability* for the status bar text to be changed - if they wanted it.


@mdorf I was trying to run tests and this breaks the create sample data for ontologies api


102941444_CorrectBSDFs (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


update


Do these even make sense to put in the readme for the *website*?


![screenshot_2015-09-14-08-53-32](https://cloud.githubusercontent.com/assets/8850770/9841557/d5b452aa-5abe-11e5-8675-c9a9f945ec17.png)
The icon is greenish instead of grey. And since its dimensions are small, it gets blurred while renderring.


Achei bem completo. Não identifiquei nenhum erro.


Do not need to commit this file, or files in xcuserdata folder


Just getting it off my computer #informationsilos


Je suis de retour de Québec suite à des funérailles, et il est passé 23h00. Pour le moment, je n'ai pas d'idées sur comment relever le chellenge alors pour ce soir je vais seulement me contenter de faire un dernier commit avec les corrections selons tes commentaires.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 606](https://teamcity.labkey.org/viewLog.html?buildId=271869&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 14 (4 new), passed: 1 Build time: 00:44:20
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

##### there are 4 more failed tests, see build details
```


Agreed with @ctbeiser, there's a better place for this than the readme for the website.


fast-forwarding develop to borshop-integration-1247-bugfix/gc-batch-waiting-nothing-forever = 792a2692


this commit


恩，我说是store功能删除吧


skrypt do budowania aplikacji


https://www.youtube.com/watch?v=o9pEzgHorH0 ;)


这种情况下，把feed作为一个property就更合适，这样initWithStyle:reuseIdentifier这个原生的方法你就直接用就好了，不需要自己写一个初始化方法了。


How is that different from the function that adds tile entities in a thread safe way? I don't remember the name off the top of my head.


Nice work, thanks.


@lmy441900 Please drop in your Inkscape translation if you intend to work on it.


nice


一般在初始化的时候，只create 和 add subviews，具体就是只用init，不用initWithFrame，然后在layoutsubview这个方法里去设置每个subview的frame。可以看看layoutsubviews这个原生的方法的使用。


I added the cacuration function as "calc". Could @kondo-takuto-mulodo review the code. Thanks!


maybe we should start to create Errors like https://github.com/tessel/t2-cli/pull/295/files#diff-464d0e2fb3c59e8716aa5cacbb54b6b8R248 so we can catch them with an instance vs. trying to rely on the message property.  more robust and not dependent on OS?


This should have been run as a parameter tweak test [0,4]


你查一下这个方法的注释+ (instancetype)dataWithContentsOfURL:(NSURL *)aURL
：Do not use this synchronous method to request network-based URLs.
这里有一点要注意，请求图片时要采用异步的方式，不然页面会卡死直到图片请求成功。
一般加载网络图片常用一些第三方的库，比如SDWebImage等。你可以查一下，用pod集成很方便


- At first, you have to put indent as 4 spaces.
- The function name is not camelcase.
- Constant names are difficult to understand.
- Variable names are difficult to understand.


Use match_parent


@topherfangio This is some great improvements, and together with recent improvements to getting it to stick the right place it works overall for us, but in Chrome it is still jumpy/flashy when scrolling, whereas Safari shows the md-subhead super clean and there's zero flashing. Not sure this the right place or that the flashy part is related to this particular commit - and if this is particular in combo with our code.


@takutokondo Thanks for your review. I'll update code and push again.


Mucha cháchara ;)


Thank you @jansegre!

In which browser(s) did this happen and what did it do?


Nice


Thanks for all the help!!!!! Definitely won't make that mistake again!!!!


Whats this for?


バグやん


ejercicio 4 1


hi


这里的格式是不是调整成类似这种形式代码可读性比较好：
	if _, err := fmt.Fprintf(w, "%s\t%s\t%s\t%s\t%s\t%s",
		name,
		internalIP,
		externalIP,
		makePortString(svc.Spec.Ports),
		labels.FormatLabels(svc.Spec.Selector),
		translateTimestamp(svc.CreationTimestamp),
	); err != nil {
		return err
	}


Don't write code like that.
Try to write like this:
<c:forEach var="person" items="${persons}" varStatus="status">

        <c:set var="personFormId" value="person${status.index}"/>

        <c:url var="editUrl" value="/person/form.html">
            <c:param name="id" value="${person.id}" />
        </c:url>
        <c:url var="deleteUrl" value="/person/delete.html"/>
        <form id="${personFormId}" action="${deleteUrl}" method="POST">
            <input id="id" name="id" type="hidden" value="${person.id}"/>
        </form>

    	${person.firstName}
        ${person.lastName}

            <a href='<c:out value="${editUrl}"/>'><fmt:message key="button.edit"/>
            <fmt:message key="button.delete"/>

</c:forEach>


If scala was so cool we wouldn't have to do this.


You can't pattern match this ?


@nikhilvallishayee It's just regex based parsing as of now. Here's how these notations are parsed into other abstractions built: https://github.com/ssrihari/ragavardhini/blob/master/src/movertone/core_old.clj#L68-L74

The current shape of this is slightly different, in that the swaras are prefixed with some notation for gamakas. https://github.com/ssrihari/ragavardhini/blob/master/src/movertone/core.clj#L70-L86

That said, this is not meant to be a long term solution. I think we should move towards a standard notation for machine related things. Something like @srikumarks's https://github.com/srikumarks/carnot.


This commit broke go get install of binary


(y)


forkしたリポジトリを編集して pull request を 出してくれると 嬉しい


Only tested on Firefox, but should happen on most browsers.

[Here](http://jsfiddle.net/jansegre/ywncgfv8/2/) is the issue including the exact version of the parent commit. Notice that the library is added on external resources which makes it included on the `<head>` part of the html, which I think is a perfectly reasonable place to put _some_ scripts.

And [here](http://jsfiddle.net/jansegre/ywncgfv8/3/) with this fix applied.


`# @@keys_array` it's not necessary.
`[:name,:age,:gender]` =>
`[:name, :age, :gender]`


空白文字のsplitならば sentence.split()でよい


Why all of them are `self-method`, just `instance method` is OK


Installation error in the package 0.90-beta2 by creating a new installation since your update.
You have remove a column in the glpi_users table, please update the insert into commands.

line 6771 - 6774:
INSERT INTO `glpi_users` (`id`,`name`,`password`) VALUES ('2','glpi','0915bd0a5c6e56d8f38ca2b390857d4949073f41');
INSERT INTO `glpi_users` (`id`,`name`,`password`) VALUES ('3','post-only','3177926a7314de24680a9938aaa97703');
INSERT INTO `glpi_users` (`id`,`name`,`password`) VALUES ('4','tech','d9f9133fb120cd6096870bc2b496805b');
INSERT INTO `glpi_users` (`id`,`name`,`password`) VALUES ('5','normal','fea087517c26fadd409bd4b9dc642555');


oops thanks for letting me know I'll remove the files

On Mon, Sep 14, 2015 at 9:39 AM, Thoai Nguyen <notifications@github.com>
wrote:

> I use JetBrain's tools too but IDE files should not be part of the repo. I
> know that these files cause issue when different IDEs are used.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xodigital/ArtVandelay/commit/a898fa344a33a6e52381a4bbdfcb0c7c9142097d#commitcomment-13203199>
> .
>


- [ ] 表現＋ニュアンスだけの話なのですが、「不要になったオブジェクトをセッション上から削除する場合は、sessionスコープのBeanのフィールドをリセットする。」の方が正確な表現かな～と思います（他との表現を合わせるという意味でも）。


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


How do you feel about checked exceptions?


- [ ] 「セッション中のBean」は「sessionスコープのBean」に統一してほしい。（意図があるなら教えてください）

- [ ] sessionスコープのBeanのライフサイクル管理の行は note の方がよさそう。


Why is this a macro? `write-ephemeral-file` is not.


why try?


There is not cleanup, so excepting forms into this method seems pointless.


It seems this would be a bit cleaner.

```clojure
(do
  (write-file ...)
  ~@forms)
```


Thank you for your correction!


you could also use the "join" method available from the Swift standard library


```clojure
(defn- persist-package-json? [project]
  (get-in project [:npm :persist] false))
```


Apa ini


itertools の roundrobin パターン を利用するとかなりスマート
```python
def roundrobin(*iterables):
    "roundrobin('ABC', 'D', 'EF') --> A D E B F C"
    # Recipe credited to George Sakkis
    pending = len(iterables)
    nexts = cycle(iter(it).__next__ for it in iterables)
    while pending:
        try:
            for next in nexts:
                yield next()
        except StopIteration:
            pending -= 1
            nexts = cycle(islice(nexts, pending))
```


Is this do redundant since this is a `when`? Feel free to fix this (even though it is unrelated to your change. Or don't.


This tell the what type of document it this one


Nice! Blev av med trollfacet :P


Thanks!


102941444_CorrectBSDFs (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2301 of 2337 tests passed)

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


It structured correctly, only missing the contextual tags and some commentaries, nothing major. OK


set型にする[] -> {}


why try without catch? What's the point?


```javascript
prev = prev.then(function(response) {
            console.log("in here", item);
            return p;
        }).then(function(response) {
            console.log("end------------", response);
        }).catch(function(err) {
            console.log("err-------", err);
        });
```

Hình như đoạn code trên trong hàm `removeMarker` không đúng (nó in ra không đúng thứ tự các hàm console.log ). Nó không in xen kẽ các hàm console.log trong promise p với các câu lệnh console.log("in here",item) và lệnh console.log("end-------",response); kể cả với trình duyệt chrome. Nó chỉ đúng thứ tự mỗi lần đầu tiên trong chuỗi promise chaning


Structured pretty nicely, only missing the contextual tags. OK


Why have this?  Shouldn't these settings live in the editor?  What if every different editor put a file in the repo like  this?  It doesn't scale well.

For Go I think "go fmt" takes care of this. YAML will probably not be used much in this repo


Maybe put those private variables at the top of the class, with the others?


Nice fix. Thank you!


This is awesome.  Does this get triggered on travis build or manually done?


Do we need two copies? :-)

I'd just make sbin->bin symlink


- [ ] `clearItems` より `clearCart` の方がよい。実装的には `Cart` 内の `CartItem` を空にする処理でよいが、クリア処理の具体的な内容は隠蔽するほうがよい。

- [ ] 冒頭の`SessionCart` のコード例にクリア処理を行うメソッドを追加して、簡単な説明も記載してほしい。


word[:2]でよい.
いっそのこと リスト内包表記で１行でかける
```python
{  word[0] if idx in locations else word[:2] for idx, word in enumerate(words)}
```


@c4milo Can you please check to see if this commit breaks the builder and update it if it does.


Thank you very much.it work for me.


書き方が助長(リストが一度作られる)
set(n_gram(sentence_1, 2)) でいけるはず


I'm still checking whether this works. Rebuilding the world, it seems...


opcode_alias array of const char pointers is redundant in this case because it's never used. I've left it there in case a future implementation decides to do something with that. Perhaps I could have a 'disassembly' command line flag which instead of running the bin file, outputs all of the instructions and operands it runs over? Right now it's doing no harm however, so I think i'll leave it there :)


format関数を使う
```python
 "{:d}時の{:s}は{:s}".format(x,y,z)
```


Consider a more descriptive name. I think `read-package-map-from-project` might be better.

This function name sounds like it is reading its given argument, which is a package. The given argument is a project map though.... blah blah blah


102941444_CorrectBSDFs (rpg777) - 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)


You read my mind--I was going to suggest implementing this :+1:


I believe `\w` includes `-`, which is probably not appropriate here.


typo: [...] at least _one_ of the three [...] ;-)


関数の命名規則はスネークケース -> typoglycemia


It does, would also match stuff like `1:\foo` but oh well :-)


Are you sure that you are getting the synchronization semantics you need here?

If two ```StatTrackingStore``` instances are getting constructed concurrently, with the same ```cachedStats``` instance passed in their parameters, they will each call their own private ```getOrCreateStoreStats``` function, which will get synchronized at the instance level. That would mean they could end up both creating a new ```StoreStats``` instance and the last one to put it in the ```cachedStats``` map would clobber the first.

I think a synchronized block using ```cachedStats``` itself as the mutex would achieve the semantics you want. Alternatively, making the ```getOrCreateStoreStats``` function static should do the trick as well, albeit with potentially coarser-grained synchronization.

Let me know if I misunderstood the intent of the synchronization here (:


nixpkgs seems to want bin instead of sbin everywhere, see `dontMoveSbin` in http://nixos.org/nixpkgs/manual/#ssec-fixup-phase


イイネ！


Sbin does not make sense in nixos :-)


hence https://github.com/zalora/microgram/commit/9227255ec2a067cb1e2263e06eac884cc6966a2a :)


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-MacOS-10.9-clang: OK (1793 of 1835 tests passed)

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


oops
hawkeye! ;-)


It works most of times. But sometimes bot takes items and cant set it to site due error.


This is the makefile for compiling the game


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (777 of 777 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-18.94%25-green.svg)


Ya it's the oddest thing.. It was booting once but for some reason the observer would fire twice.. Couldn't tell you why if my life depended on it.


could we have more comment at commit log?


@Mamoru1234 So that the duplicate `num[i]` won't be counted.


I meant issue https://github.com/giakki/uncss/issues/76

But many things have changed in my test case so it probably is something else. Better leave this for now. :)


do we need clear m_current here?


Is this useful since calling ```close()``` while ```referenceCount > 0``` does not do anything?

If the goal of ```close()``` is to make sure Mbeans are always unregistered, then perhaps you need a mechanism to force it? Maybe a boolean parameter you would pass to ```unregisterJmx()``` to force unregistration, regardless of ```referenceCount```? Otherwise, calling ```close()``` doesn't guarantee much, and it seems to me it should probably not do anything at all if it cannot guarantee its success.


Yeah, that's a good idea. Makes much more sense to be done in Ansible, and makes it heaps easy to extend the configurability. The script does need to be on the DTN, so I'll have to work out how to change it to handle multiple servers. Have you looked at that before by any chance?


@PeterHsieh @nirrek I changed this key to `{payment.date}` because `{payment.id}` always gives the same value (it isn't payment ID but it's property ID).
For example if I use `{payment.date}` for key and print `{payment.id}` :

Result


Em vừa sửa được rồi anh ạ, hình như em đặt sai đoạn code bên trên và nó thực hiện ko đúng thứ tự( em vẫn chưa hiểu tại sao ). Nhưng khi chuyển đoạn code trên từ hàm removeMarker vào hàm onMouseClick_removeMarker thì chạy ok. Như vậy là do code sai chứ không phải là lỗi promise chaining trên firefox hay chrome.


ㅁㅊㅋㅋㅋ


Fixed now in branch multilingual


change the name of the variable `i`


better change to this
SharedPtr<H265SliceHdr> currentSlice(new H265SliceHdr(), h265SliceHdrFree);
H265SliceHdr* slice = currentSlice.get();


#4402-VRF-outputs-and-multiplied-floor (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1799 of 1841 tests passed)

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


pSlice is a local variable, set it to null is useless


Should ```registerJmx()``` and ```unregisterJmx()``` be synchronized together on a common object?

The current implementation synchronizes each function independently of each other, which I think means you could end up with a situation where both functions get called concurrently and you end up with a ```referenceCount > 0``` and the Mbean unregistered, which seems like it is what you'd like to avoid.

You could use ```referenceCount``` or ```isRegistered``` as the mutex of a synchronized block in each function.


Should fix it server side so you are actually getting the id, as otherwise if you make 2 payments on the same date, they won't be able to be rendered.

This bug will have been introduced when the serverside was changed to perform a join of two tables:
```
sql: 'SELECT * FROM payments y, properties p ' +
           'WHERE y.propertyId = p.id AND y.tenantId = ?',
```
Both table will have an `id` column and so the second table attributes are clobbering the first, and so you are getting the id from the properties table rather than payments table. The query needs to be fixed to solve this.


Please follow convention and put the style.css file in your css directory with the other css files.


Still cleaning it up, but [here's](http://i.imgur.com/L1nMSTh.png) how it is currently.


I am just wondering - sprintf does normally NOT append to a previous string. There must be some other problem around.

As a further node - why do you use a \n in the format string, It will be stripped in the g_strchomp() below.


I left a few comments. Please address them if you think the concerns are legitimate. LGTM otherwise.


reload tableview only after all the data are ready


You HTML is invalid.  The `a` tag here for twitter is close with nothing in the content and at the end of the line you try to close an `a`.  This kind of mistake can easily be avoided by using a HTML Validator (https://validator.w3.org/)[https://validator.w3.org/]


你觉得此处应该用异步还是同步？


HTML is invalid two start `span` tags.  remember the structure for a HTML tag, again this kind of mistake can be eliminated by the validator [https://validator.w3.org/](https://validator.w3.org/)


I'd make this `p` a `small` tag to be more semantically correct.


@Passw, what OS/distro? What happens if you change the first line to `#!/bin/bash`?


Looks good!


I'd use a fixed width here.  Based on the `font-size` for body copy the correct `width` to get the quote text on the 4 lines is around 420px.  Remember percentages are relative so they will be different depending on screen size.  It would also be better to use either a `q` tag or `blockquote` for semantic purposes.


smart.  i think it makes a a lot of sense.  i did not consider this case.  i also like defaulting to `true`


Well, changing them into URLs isn't a bad idea. The regex isn't correct, so let's fix it... Anyway, the syntax is bad


What's the difference between the 2 setcolors? I wrote the original and it seems like the one you wrote would be better replaced by calling `setColors(startColor, startColor)`


To avoid confusion when we reach a hundred posts, we should have more descriptive names for images. Maybe a separate folder?


Are putting the image in the right drawable folders, use
http://bit.ly/1FH3Gcv to generate image assets from a png correctly
On 14-Sep-2015 8:59 am, "Darshan Mehta" <notifications@github.com> wrote:

> [image: screenshot_2015-09-14-08-53-32]
> <https://cloud.githubusercontent.com/assets/8850770/9841557/d5b452aa-5abe-11e5-8675-c9a9f945ec17.png>
> The icon is greenish instead of grey. And since its dimensions are small,
> it gets blurred while renderring.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saurabhsjoshi/VITacademics-for-Android/commit/846af53a6627208dd82116c0241a9a3be2d7773d#commitcomment-13204552>
> .
>


@wprater, that's a good point. I'm still learning about best practices for Error objects in JS but what you're suggesting makes a ton of sense. I'll make a separate PR.


Hi, I'm doing something similar. I'm glad you beat me to it and took care of the test.exe project
( https://github.com/zeromus/vgmstream )
But it looks like you had project references for fdk-aac, g719, and mp4v2. What's the plan here? I dont want to do anything that will stomp on work you intended to do.. can we commit them in ext_libs like the others? I prefer to do things with project references also, but.. I commit them. Anyway theres a precedent in vgmstream for using ext_libs.


You can use `padding` on the parent instead of the doing the transform here.  If you are going to do this the

    position: relative;

should be on the parent not the element itself.  e.g.

    #sign-up {
        position: relative;
    }


Yeah, it's becoming increasingly apparent the needs of each field type can vary dramatically, so allowing an easily extensible solution wins for code clarity and users...


@majianpeng `osd_op.outdata` is not set by then, see the failed test at http://qa-proxy.ceph.com/teuthology/kchai-2015-09-12_20:16:25-rados-wip-kefu-testing---basic-multi/1053246/teuthology.log .


hence we can dispense with this line.


@Thunderbots5604/owners I'll point out here that I was the one that wrote this documentation, and any questions should be directed at me.


Decided to go with standard es6 promises. let's always return a promise for all functions to make sure http response timeouts.
#6


Neden POST?


can i use it on c9.io ?!


Use what exactly?


This whole function seems a bit odd (`(into {} ...` and `distinct`?). What are the types of the `file-deps`and `lein-deps`? Is one symbol keys and the other keyword?

If we are refactoring this a bit we might be able to get rid of `transform-deps` too. If I am reading this correctly, the deps we get from project is a vector of two element vectors, the deps from project is a map of keywords to version strings.

```clojure
(apply assoc package-deps
             (for [[package-name version-str] lein-deps]
               [(keyword package-name) version-str]))
```


good.


Place the Google Fonts link in above your own styles as you are referencing the `font-family` 'Open Sans' in your style.css.  Remember the order is important.


Might be best to store this in a static variable or use `ItemStackHolder`


Not exactly related to the bug - but fixed code anyway.


I'd use a `blockquote` or a `q` tag here to make it more semantically correct.  A `p` will work but doesn't convey any specific meaning.


Nice, thanks :)


Hi Ensoftek,

After modifications, I committed this code to the codebase:
https://github.com/openemr/openemr/commit/7a26f614e9d8b2ad6bce117dc37d226b777d3149
(Thank you for the contribution!)

I have marked some places above with 'ASKENSOFTEK' with specific questions regarding your code. When your group has time in the future, please answer these questions.

Thanks,
-brady


Inline styles are bad, you should style these links within your CSS.  You could target them by

    footer a {
        /* Your styles here */
    }

Leave inline styling to JavaScript.


I'd use a `small` tag here for the copyright.


x is usually 0, and <body> usually has no particular offset. In this case, when it does, the tooltip should offset accordingly.

"screen: true" is to make sure the to render within screen.


`'com.android.support:design:23.0.1'` has `'com.android.support:appcompat-v7:23.0.1'` as a dependency, so you can remove `'com.android.support:appcompat-v7:23.0.1'`.


You are using a lot of `display: block;` everywhere.  Unless you have specified otherwise a normalise.css will have this set by default and the inheritance part of CSS will save you having to write it out again for each element.


arr 这个变量有用吗？


When the <body> has a height/width smaller than the window height/width, the smaller height/width should be used, just only the doc height case.


No, can you please make this read like this.
----
#### [Downloads](https://github.com/kvirc/KVIrc/wiki/Downloading-KVirc-source-or-binaries#source-downloads-master)
Try one of our fresh off the press nightly development builds for Windows (x86 or x64), Ubuntu and Debian.
We're also looking for volunteer packagers for various distros who can help us extend the availability to other OS's.

----
Instead of this wrap of both sentences, since they are not part of same paragraph.


.idea 目录也不需要提交，建议看一下哪些是必须的，然后把不需要的用 .gitignore管理。


Förbättring eller ej, det är mer transparent i alla fall ;)


Would be best to place within a `let `block


Este es el que vamos a presentar. https://github.com/nebulae-co/saber2014/blob/master/Documento_icfes.rmd


why is NSMutableArray instead of NSArray? and also, there's a better way to creare array:  NSArray *contentText = @[@"1",@"2"]


I'm not a big fan of prefixing the shell snippet with `$` or `>`. This is mainly because I'm really lazy, and I just want to copy the whole line and paste it in my terminal. I already see a couple of code highlighter that manage this case, but I don't know how they do that (CSS prefix would be copied right ?)


That's a good idea, however it need to be also written in the "blog" guide.


Have now idea about that :blush:


Close the streams!


摸摸大 这没有回复功能么= =


woops mis type will fix it soon.


I dont think that this is a good idea.
Because everyone that includes TreeMars.h will now include all those files as well, even though he doesn't need them. This slows down the compiler a lot.

Usually in c++ one tries to include as little as possible in the header file.


Right, thanks!


Something is not right here.


something is not right here.
2


I will then review and re-submit.


why not just return jsonToDictionary?


Just a thought ... does this plugin include Commons IO in its dependencies? If so we should use FileUtils.copyFile()


I will move them to the top, in my next commit, but I prefer them if it is used only in one function it is easier to keep them nearby.

But I don't have a preference and with editors you can track where they are declared, so I will move them to the top.


This is where #18 shows


Felix, AFAIK in Java synchronized method translates to
method() {
  synchronized(this) {
}
}

So they are synchronized on a common object. In C# there are many pitfalls of serialzing on this object as the interpreter has some dependency, but in Java, I did not hear of such a thing.


I will not use this symbol anymore with my commits :-)
Please understand that i am very careful when edit the code, but when i make a mistake it is hard to find out what i have done wrong when i commit a huge number of changes. When i make one tiny mistake there will be 3 sites down and i like to prevent this ;-) . I will try to make more changes in each commit.


you can directly extend from UITableviewController if the content of the VC is just tableview


Good and I'll then merge


Všetky headeri na widgetoch sú poposúvané. :(


For others,  this change was reverted in https://github.com/bitcoinjs/bitcoinjs-lib/commit/50a0d94303b72f542c9c2cb67e6b15ae75c8ca6c


good


matchin 拼写错误


Let me take a step back and explain the logic here. PipelineRoutedStats are created one per store client. This clutters with too many mbeans. Now all of them should share the mbeans. So the first creator will register it, the last to close will unregister it.

I see your point, that registerJmx and unregisterJmx does not correspond that information directly. Will think about how better to communicate this information.


UILabel is used mostly for some simple text, try to use UIView in this case


Mixes camel case and snake case.


A useful tool that you might consider is _objdump_ . You can run it to dump out all the headers (including sections) and disassembled code of an elf file.

objdump -x -d isodir/boot/main.bin

This will print out the headers and disassembled code. What caught my attention was that before my change the GNU identification appeared at the 4K(0x1000) mark and .text started at 8K(0x2000). Under "sections" at the top the file offset for each section is a useful thing to look at in this case.

GRUB has a requirement that the multiboot header has to be placed anywhere in the first 8K (and aligned on a 4 byte boundary). What happened in this case was the magic header was in the file but just beyond 8k, and that causes GRUB to think the kernel isn't multiboot capable.

There is also an alternate method of generating an image that isn't an elf format, but support multiboot (and non-multiboot) with a flat binary image. I recently helped someone on their [test OS](https://bitbucket.org/jlsinatra/sinatra/src/99200562ef7897b1bf4812be010af8e4f7b97789?at=master) by creating an [archive with changes](http://www.capp-sysware.com/misc/sinatra.zip). I simply took his code and modified it to support multiboot. The linker and compile process can be modified for a Linux environment. The original author was doing his kernel development on Windows, but it can be readily adapted for Linux development tool chain.


やっぱり怖い...？笑


You are right, this is a bug, they should be synchronized on the cachedStats and not on the synchronized block.


このファイル全体を新しい記法に直しましょう


Inconsistent naming. `get_body`, `assign_func_body` and this? Does this have a special meaning?


ok


Hey yorkie, I'm a follower of yours, I was wondering are you dumping this project soon, are gonna start rebuilding it?


Great job, pinksoda6!


You officially made your first pull request!


You don't need to "addSubview" to tableview, just set the "headerView" and "footerView" and call the delegate methods like cellForRowAtIndexPath to create the content of a tableview


do not need to create tableview if your controller is extended from UITableviewController


Dokuwiki would indeed be a better choice for new installations, so why do we not make it the default? But we've already got those existing installations, someone added ckeditor to the project, we didn't remove it and now have just to live with the consequences. At least the user is now forewarned and can decide what to do.


It has to, visit_<NodeType> is special for Visitor classes (I suppose, it is for ast.NodeVisitor for example).


@takutokondo I updated code. Could you review again?


I don't see the text either.
I think it's because the text isn't in en.coffee:
https://github.com/zooniverse/Galaxy-Zoo/issues/214


Use BufferedOutputStream and BufferedInputStream to improve performance!


Isn't it TShock 4.3.9, also SSC doesn't seem to work properly.


Does that mean that this code was obsolete?


Закинул в репозитори обновлённые файлы, commit в проге сделал, она вроде отправила, но pull request не даёт сделать, и на сайте не вижу этого последнего исправления...


I reviewed and it's good.


This was a new "feature" which wasn't present in the current profiler. But we can totally live without it. See https://github.com/symfony/symfony/blob/2.8/src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base.html.twig#L20


This problem still exists.


i think this has to be params
because its a `GET` right ?
data is for `POST`
at least in how **apiClient.js** is written
i tried this in postman and it worked with
url params
will double check tomorrow


delete


L.105-139: should be inside it's own private method


@mpetch thanks, I know a bit about `objdump`, just nothing about the multiboot spec :-)

I'll have a look at the flat binary, I didn't know that format. What is the advantage of it over an ELF multiboot?


@bnu


exactly! the whole point is that you discover everything after hitting the first URL.. I would actually quite fancy spending an afternoon (e.g. in a hack session) designing / building a 'truly' restful api


I agree that adding the hash is useful for the package name. What do you mean with -dirty flag?


Typically you will use either `display: inline-block;` or float the element.  I'm curious what was the reason you used both?


The removal of the padding wasn't strictly related to the list bullets, but the rule was a duplicate of https://github.com/CatalystCode/case-studies/blob/f23d6bb559beb4236e0bdc630d7fd39df9a3edf9/_sass/_base.scss#L10 so thought that it might be clearer if this section is only about the list style type.


Ah shoot Ill add the option to the module:install method

You can do it later like db:seed --addon=anomaly.module.store for example

--seed is respected from the migrate commands so I can add it to the install command and let it trickle down.

And the UI sucks for this at the moment - no --seed from there

Ryan Thompson
President / CEO
AnomalyLabs, Inc.
www.anomaly.is<http://www.anomaly.is>

t (877) 705-0806
m (309) 752-8581

On Sep 13, 2015, at 1:48 PM, Adnan Sagar, PhD. <notifications@github.com<mailto:notifications@github.com>> wrote:

--seed didn't work with module:install!

  [RuntimeException]
  The "--seed" option does not exist.

Also, I guess, no seeding when installing a module from admin?

Thanks @RyanThompson<https://github.com/RyanThompson>

—
Reply to this email directly or view it on GitHub<https://github.com/anomalylabs/streams-platform/commit/31613c7d29d928ee62712efe0f911b4a2e3d8e44#commitcomment-13202025>.


@bnu `_updateMenuItem` 함수에서 `$itemInfo->name`, `$itemInfo->desc` 에 대해 `removeHacktag` 함수를 실행하고 있는데 중복실행이 되지 않나요?


That's kinda interesting. Where is the difference to vanilla MW here? The running the script multiple times or the handling of multiple message boxes?

btw. you should set mInterMessageBoxe to 0 after deleting it. Otherwise the code is not exception-safe.


Best practice would suggest that you dash separate your names i.e.

    #main-nav {
        /* Your styles here */


Good grouping to avoid duplicating style blocks


#4402-VRF-outputs-and-multiplied-floor (rraustad) - i386-Windows-7-VisualStudio-12: OK (1799 of 1841 tests passed)

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


> Prawdopodobnie będziesz ich

je*


Bring your Google Fonts in before your style.css.  Remember you want use the `font-family` 'Open Sans' in your code so in order for the styles to work correctly the Browser needs to know in advance.

In other words place this

     <link href='https://fonts.googleapis.com/css?family=Open+Sans:400,300,300italic,600,800' rel='stylesheet' type='text/css'>

before this

    <link rel="stylesheet" href="css/style.css">


Very descriptive `alt` attribute, I like it!


ok


graj*


Ah yea, now this is duplicated and I didn't pay for the domain any more. Do you want to work on this project? Just curious, just feel free to share your thoughts anyway :)


Ah, I see now the 'display: inline-block;` really won't have any effect due to your structure.  If you did put another element after expect the unexpected ;-)


きちんと設定ファイルを編集していていいですね！
全て新しい記法に直してあげましょう


awesome


good


Consider it just another way to skin a chicken. One advantage is that you don't have all the ELF overhead (the overhead isn't excessive, but it does exist). If you need to place your kernel on a floppy in the MBR without a boot loader this method would support that environment too (you'd still need to change the linker origin point for the code).You can't place an ELF binary in the MBR. Basically using a non-ELF format works better where you need to support a multiboot or a non-multiboot environment.


You missed the pseudo-class `:hover` on the links (in your CSS of course) which you could have used to add simple behaviour such as underline the link or make the font bigger etc.


Good use of `small` tag here.  Mind those `alt` attributes above on the images, they'll raise an error in the HTML Validator ;-)


I think you should use block faces, then your intentions are more clear.


Thank you John; will fill out the template this week


no need for this line (ie var resp *http.Response)


@cornae I believe you asked for this? If yes, this should be in PI proto by now if you want to test. If not, any idea who wanted it and needs notification?


Could the potential block be null? Is such case we have an NPE right here.


how to open search page in new tab?


Hey Pat, I realised this after I submitted so I made another commit with the hovers included. I may have forgotten to publish them. Let me check when I get home.


Should this be the clicked block or the potential block? I think it should be the potential block.


If you are dumping it then no, but i wouldn't mind tinkering on this to rebuild it with you. I know your super busy. If not though, i will catch up with you on another one of your newer projects. Just let me know. I'm pretty busy nowadays, but it would be awesome to work with you on a project.


we don't want these <h1> headers, hence the pre-commit hook to check for them (and the failing build).

if you really want a header here, it should be <h2> (but I think the header at the top of the page probably makes that redundant?)


Ouch.


It seems damage isn't done when the player can painbuild.


nice bro


little endian*


This is a sample method on how to add a policy to system.
#10 is fixed with this method.


Without this change, Mix is broken. I am fine with placing this under bug fix although we need to fix the `File.stat` issue.


Note sure where to notify you of this, however I'm seeing a load error on windows for gem `1.3.14` on `ruby 2.1.6p336` no problems at all with `1.3.13` on the same system

```
C:/Ruby21/lib/ruby/gems/2.1.0/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `require': 126: The specified module could not be found.   - C:/Ruby21/lib/ruby/gems/2.1.0/gems/couchbase-1.3.14-x86-mingw32/lib/couchbase/2.1/couchbase_ext.so (LoadError)
        from C:/Ruby21/lib/ruby/gems/2.1.0/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `block in require'
        from C:/Ruby21/lib/ruby/gems/2.1.0/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:240:in `load_dependency'
        from C:/Ruby21/lib/ruby/gems/2.1.0/gems/activesupport-4.2.4/lib/active_support/dependencies.rb:274:in `require'
        from C:/Ruby21/lib/ruby/gems/2.1.0/gems/couchbase-1.3.14-x86-mingw32/lib/couchbase_ext.rb:1:in `<top (required)>'
```

I did check that the `couchbase_ext.so` file existed and uninstalled / re-installed the gem a couple of times


You've set a definite width and centered the content.  While not in the brief I'm happy you did so.


Why put a `width` and `height` of `auto` here?


The official Docker Best Practices guide [recommends](https://docs.docker.com/articles/dockerfile_best-practices#add-or-copy) the use of `COPY` over `ADD` for simple file operations as it is more transparent.


When you find that you've multiple padding or margin values best to write the shorthand property rather than the long one.


I agree, I would like to test with more threads, but it seems that this is
not allowed.

On Mon, Sep 14, 2015 at 5:25 AM, DU-jdto <notifications@github.com> wrote:

> sqrt grows way faster than log, so it'll be interested to see how this
> scales to longer TCs (and thus higher depths).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IIvec/Stockfish/commit/99b41ce4594b5dfde4595ae648562fe099da156a#commitcomment-13204524>
> .
>

--
IQ World <http://www.ultimaiq.net/>


comment test


I put [0,5] because of a new function, but I'll try [0,4] in the future.

On Mon, Sep 14, 2015 at 5:51 AM, Rocky640 <notifications@github.com> wrote:

> This should have been run as a parameter tweak test [0,4]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IIvec/Stockfish/commit/99b41ce4594b5dfde4595ae648562fe099da156a#commitcomment-13204652>
> .
>

--
IQ World <http://www.ultimaiq.net/>


How use by expo


Why did you use `display: inline-block;` here?  Remember it is really only used to allow you to set a `width` and `height` on the element which is not allow with `display: inline`.  The `p` would normally be `display: block` as set in the normalize.


I got your code working well on my pi with one button connected to GPIO, but I am really stuck on how to figure out how to add another button (button_B) connegted to pin 26. I copy all the lines and replace 18 with 26 and put below the current button code, but it does not work. I do not know how to program and just tryuing to hack something together. Would you be able to explain how to add more buttons.
Thanks in advance. V.


If you change the `p` to an `a` (changing the HTML accordingly) and do your `text-align: center` on the parent you will get a centered link.

    #sign-up {
        /* As above with the extra */
        text-align: center;
    }


Is this till the case? I refer to your commit comment: cfd99b6a71245663f4c31a8154b2588c22c0ecf1


Rewrote this to only add platform warning if running under plain windows.


Revert


If file logging is disabled the stack trace won't ever be printed.  I have my own glog fork but I think it applies to this version as well.


     <link href='https://fonts.googleapis.com/css?family=Open+Sans:400,600,800,300,300italic' rel='stylesheet' type='text/css'>

needs to go before

    <link href="css/style.css" rel="stylesheet">

because you use 'Open Sans' as the `font-family` in your CSS.  Order is important.


If delete_blob failed, there is no exception thrown.
Why use `begin...rescue...end` in a `begin...rescue...end`? Is the purpose to ignore the delete_blob failure?


\o/


What was wrong with the original fix?


Thanks for the pull request.

I'm a bit concerned if this is proper solution. From the logs, it says the problem is that it is trying to insert new row with key that already exists, and this is naturally very bad case. These IDs should always be unique since they are used for referring to files/folders.

Your addition surely prevents this error, but I'd like to understand why it happens. I'm afraid that now it can hide the real reason, because this is what happens now:
1) it first searches with the path, if it finds it then it uses the ID from there (as it's been created before)
2) if path was not found, then it creates new ID for the path and adds it

Now your addition goes in middle of this, checking if the created ID exists. And if it does, it uses it with no questions asked. The problem is that since phase 1 failed, it is guaranteed that the path is not there yet. So the ID found belongs to some other path, and in this case it should never be used.

I guess it should continue as long as it finds truly unique ID, but why does this happen? Does the function uniqid really return same value twice or what?


Thanks for the feedback. posixpath and platform warning for fusepy fixed, and rebased.

jotta-restore only supports restoring deleted files for now, but it's definitely possible to add restoring to earlier versions as well. Don't think I'll do that in this patch though, this is big enough as it is.


Hi @matthewberryman can you check if there's a line for PM2 package requirement in package.json? Noting Ben's use of the "^" syntaxt.

Also, see my comment above regarding README.md

I will then pull, and test again locally before merging.

Thanks,

Tom


Lol, that's really interesting... How old are you? Maybe you could join my business team if you want :)


Never mind, I see that disabling file logs is only a feature in my fork.


This HTML is invalid.  A `href` can't have a space in the name unless it is percent encoded.  i.e.

    Contact Us

Which is the link to the file or path you want to link too.  In this case you don't have a contact us page so I'd leave it as `#` which is a bookmark to the beginning of the document.


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


Thanks!


sure, i will add an option


Уже второй раз такое пишу imho. Больше так делать не надо.


@dgageot Are you sure of this system.out.println ?


Anh vừa xem qua thì có vẻ về ý tưởng thì ko sai nhưng implement như thế thì không scalable.
Hiện h theo anh hiểu thì em remove item nào (điểm sóng) thì em loại khỏi _rtree rồi sau đó vẽ lại các tile tương ứng. Làm xong thì lại tới item khác. Nếu có 100000 thì sẽ làm ngần đấy bước.

Theo anh giải thuật đúng là, em batching quá trình remove bằng cách. Em xóa toàn bộ các điểm cần xóa khỏi _rtree, sau đó vẽ lại tất cả các tile bị ảnh hưởng. Làm xong tất cả các bước này mới cần resolve 1 cái promise. Và việc chaining promise là chaining giữa các lệnh remove makers với nhau chứ không phải với từng item trong 1 câu lệnh xóa. Ví dụ, A yêu cầu xóa 10000 điểm, trong lúc A đang thực hiện thì B đòi xóa 1500 điểm, ngay sau đó C thêm vào 150000 điểm. Thì client sẽ thực hiện update theo đúng thứ tự A,B,C.

Nếu hiểu như thế thì làm dễ hơn nhiều và nhanh hơn nhiều, không cần phải sử dụng Promise.all đâu


By design. In case the previous error is ignored.


clase en java que permite separar las palabras en un aoracion
cuando encuentra una(,)(.)(;)(:)(-)(_)( )


One thing that could help understand, is to find the log for when the first ID was created.

In this case the error was with key '55e61039ee699', you could find the two occurrences with searching:

VALUES ('55e61039ee699'

Then if you can post the two log snippets with complete INSERT clause with timestamps, it would at least tell
a) are they executed simultaneously (is this why it generates two same IDs)
b) are they for same path, but with some error in path format


Why isn't this just a devDependency?


good idea, i will add a shift option.
i look if i can come up with something analogue for gamepad...


いじってるところに今日気が付いたけどいいね！


You mean, you prefer a log or you are asking if it's really overriding orchestrator's method?


Ah I see now why you put the `display: inline-block;` in your CSS for this one.  Your markup needed it because you have two paragraphs side-by-side.

What you could do here is to use a pseudo element first-letter which means you'd have one paragraph with no classes necessary.  In your CSS you'd then have

    article p:first-letter {
        float: left;
        font-size: 80px;
        line-height: 65px;
        padding-top: 4px;
        padding-right: 8px;
        padding-left: 3px;
        font-weight: 600;
    }


Not sure what you were attempting here


As I don't particularly like forced symbols and we use strings anyway when passing to the instrumenter, I completely agree with your solution!


vorbidden is undefined when SMCube isn't found


README


Typeo here on the making your HTML invalid too. `si` is not a tag it should be an `li`.  Please use the HMTL validator it's there to help everyone!


Shouldn't the version number actually be `2.0` because there is a breaking change in the filter by value syntax?


same error here on kubuntu 15.04.
changing to /bin/bash is working.


I would prefer a log. Or was it on purpose to have a sout ?


@petermaloney check out my wip branch.  It has some functions to split up the code, and I've added diamond helmet with light and megatorch.  It isn't fully tested yet, but is mostly working.  There's still some cleanup to do, and I still plan to add chat commands.  Leaving the game with megatorch activated will leave light nodes behind.

I've added an additional feature from dwarves_game that allows any of the regular light items (not megatorch) to trigger the light even if it's not wielded, but just in the main tool tray (8 slots at bottom of screen).  I decided not to do that with megatorch due to the performance hit.  All of the lighting decisions happen in the new function wielding_light.  Also added a craft to take a megatorch back to 9 torches.

This all feels a little laggy now, but I'll play with it some more when I get time.  Of course megatorch is still really laggy, but should be slightly faster because I do most of the math once at startup instead of during globalstep.  The big hit there is the sheer number of light points that have to be moved around.  The way I implemented the megalight broke the snap to grid code, it's only halfway implemented now.  It's all flickering right now because of the way Lua treats tables as a table index ([0, 0, 0] ~= [0, 0, 0]).

I'll probably break light changes out of the main globalstep function to a update_light_player similar to yours, for use in on_leaveplayer.  Your use of that function in on_joinplayer is unnecessary as globalstep will happen a small fraction of a second later.  I used your update_light_all function name for my globalstep.  It's just replacing an anonymous function with the named one and shouldn't create any performance penalty.  You could do the same with minimal changes.

I'm debating a function get_position(player).  Besides that (and update_light_player) I can't think of any code I'd like to split out to functions.  It's pretty maintainable right now and will be better when I finish cleaning it up.

Feel free to pull from this repo or I can commit to my branch in yours if you give me privs.


As mentioned in https://github.com/opf/openproject/pull/3421#issuecomment-139910923, this still needs proper investigation.


Hm running `*/` in my shell doesn't seem to give any useful results


There is a typo here: `self.burt` instead of `self.burst`


I wanted to be sure to see this. Will change to a log

On Mon, Sep 14, 2015 at 9:08 AM Julien Lancelot <notifications@github.com>
wrote:

> I would prefer a log. Or was it on purpose to have a sout ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SonarSource/sonarqube/commit/ea1a6ab2144921a880d3d347a95c43aa415a5d10#commitcomment-13206090>
> .
>


Oh, I use tabs for indent because that's just the type of person I am. I find tab indents to be neater and cleaner than using spaces for indents.


Thanks. Looks good.


@jaredsburrows I'm **fully** aware of that. The reason it is here is to make it obvious to people that it's used.


@oliverguenther see commit comment (as you mentioned you don't like forcing symbols in https://github.com/opf/openproject/pull/3421#commitcomment-13206066)


第三方


Believe it or not, I just turned 36, i just have a young face. via my @keltheceo twitter pic.


So whats going on with your business team?


Never mind this commit, I reverted it in the following commit.


Ну у нас русский язык, можем `lang="ru"` поставить


@OrKoN That approach doens't sound too bad. Right now, we are only accepting one single fallback image. In case that fallback fails, this emergency one is triggered.

In that case, I would probably allow for the developer to place a string or an array in the fallbackImage attribute, to allow for more flexibility. Then we could loop through the array, check them one by one, and in the end if none works, the emergency fallback kicks in.


@stakach does it work if you put this lib http://files.avsej.net/libwinpthread-1.dll to `C:/Ruby21/lib/ruby/gems/2.1.0/gems/couchbase-1.3.14-x86-mingw32/lib/libwinpthread-1.dll`?


I downloaded SMS Api Extension (v1.3) in civicrm version 4.5.0.
After that i am unable to give Provider name.(As there is no option to give provider name)
Then i went to https://github.com/CiviCooP/org.civicoop.smsapi .... There in commits "fixed provider name fixed section". It showing two changing files which was missing in last released SMS Api version 1.3. 4.So i created CRM/Smsapi/CivirulesAction.mgd.php and CRM/Smsapi/CivirulesAction.php inside org.civicoop.smsapi (which is the extension diretory name)
When i opened --> Settings - SMS Provider --> Still creating of provider name option is not coming.

Please suggest.

Please provide a complete updated zip file of sms api. I saw which file you changed CivirulesAction.php that is missing sms api version 3 downloaded file.


> So something we don't want to bottle would be keg_only, right?

:clap:


It should be restored.
Plz refer https://jira2.lgsvl.com/browse/ENYO-2516


There is already a anagram in strings folder. Why a duplicate ?


Fire pliers Lumen


Excellent! Thanks for the fix!


sdf


Nice one!


:cookie:


What I was wondering whether the intention of the analysis of clauses  in `addDisplayForm` was to determine those definitions that were added during `applySection`.  Because if that was the case, my restriction to `defCopy`s is correct. I added the `defCopy` attribute much later than you designed display forms, so the test for `defCopy` was not available then.


RefreshableViewの実装がListFragmentだとは限らないので，このように実装に依存したメソッドをインターフェイスに追加するのは設計がおかしいと思いますし，テストが書きづらくなります．


Gamepad vibrates also while shooting. Have you thought about that in this case? In Smooth Doom for ZDoom shooting with chaingun gave impression of screen shaking. Now, shaking for all weapons all the time could be too distracting, but maybe chaingun after a while (indicating lowering accuracy), or BFG 9000 on the moment of a plasm coming out of it?


                                                                                I don't see why this is needed. A fallback is anyways the last resort. I don't see what many would be good for. Also lazysizes would need to support that.                                                                                Gesendet von meinem BlackBerry 10-Smartphone.                                                                                Von: Nelson OliveiraGesendet: Montag, 14. September 2015 09:18An: ember-vcl/imgAntwort an: ember-vcl/imgCc: Thomas HoppeBetreff: Re: [img] Emergency fallback image added. (6cfcfff)@OrKoN That approach doens't sound too bad. Right now, we are only accepting one single fallback image. In case that fallback fails, this emergency one is triggered.

In that case, I would probably allow for the developer to place a string or an array in the fallbackImage attribute, to allow for more flexibility. Then we could loop through the array, check them one by one, and in the end if none works, the emergency fallback kicks in.

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


Don't Use + sign use '\'.


Not good.


BAD


ok.


Meeseeks boxes pull players immediately and at random from the set of ghosts if they have pAI turned on.

This is graceless and inelegant.  It may also be screwing with a subsystem or processing thing, as this has popped three or more times over two hours:
Warning. The Master Controller has not fired in the last 400 ticks. Automatic restart in 100 ticks.

Regardless, give a look to the code/controllers/subsystem/pai.dm code, make Meeseeks its own special role to have candidacy, or give it a summons like wizard or ERT (Do you want to be X?)


Can you show me link for this product :Ikea Koppla ?


Yes I believe so.


We could make a travis bot which automatically raises this bar whenever possible ;)


update to globalvariables.fakedate


This is similar to what was done for the core Login plugin as discussed in https://github.com/piwik/piwik/pull/8723 for Piwik 3.0 instead of 2.14.3, to not break compatibility with existing plugins.


popBackStackを呼び出した瞬間に，いまのviewがバックスタックの一番上にある保証はありますか？ そうでなければ，スタックの一番上にある他のフラグメントを終了させてしまうことになりそうです．


saw approval from shino
at https://github.com/basho/riak_cs/commit/652eaaa84f219f36d231932e83fce6c52158205d


basho/riak_cs/test/gh1247-eqc = 652eaaa8 merged ok, testing candidate = 7bcbcd3c


merging basho/riak_cs/test/gh1247-eqc = 652eaaa8 into borshop-integration-1248-test/gh1247-eqc


내용변경


Суров, ещё и расы разные предлагаешь? Ваяй бэк тогда хоть примерный.


Why not do this properly and use a MimeType parser?


Hi Chris,

I've created a PR (https://github.com/tlaukkan/zigbee4java/pull/61) for
found issues. Please review it.

Regards,
  Brhett

On Fri, Sep 11, 2015 at 3:42 AM, Chris Jackson <notifications@github.com>
wrote:

> Well, feel free to update and create a PR - I'd love to get the doc
> improved :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tlaukkan/zigbee4java/commit/a1474086f8224dd80251215a4259efdcbc270079#commitcomment-13168065>
> .
>


@pawelfus FYI - removed some old underscore variable names.


Sounds safer - who knows what weird C-T's browsers out there can send? ;)


> @ChristophWurst Sorry, just thought due to the minor change it was ok. Will never happen again.

thanks :)


@jsquyres @rolfv
this tests disables f08 support under OS X (10.10 aka yosemite) with gfortran 5.2.0
(gfortran-5 provided by brew)

```fortran
MODULE aaa
INTEGER :: aaa_unique
COMMON/common_to_both/COMMON_NAME_TO_BOTH
END MODULE aaa
MODULE bbb
INTEGER :: bbb_unique
INTEGER, BIND(C, name="common_to_both_") :: COMMON_NAME_TO_BOTH
END MODULE bbb
PROGRAM test_proc
  USE :: aaa, ONLY: aaa_unique
  USE :: bbb, ONLY: bbb_unique
END PROGRAM
```
the error is
```
/var/folders/my/nrglry8n6ddbz2vdtxzxz3rw0000gn/T//cc9N05q9.s:42:Ignoring attempt to re-define symbol
```

but the very same test runs just fine under Linux with gfortran 5.2.0 manually compiled.

under OSX, the test runs fine if modules are in different source files.

would that be good to have f08 support on OS X gfortran 5.2.0 but not PGI ?


this is necessary if qemu crashes... else your terminal is completely unusable afterwards and it will even be difficult to type in reset -I


Instead of encourage forking you should make a call to find maintainers and give them the key of the fort (add them on github, bower, npm, etc). This library should live long, with or without you.


Allowing UsersManager to be deactivated (without any replacement plugin) is actually less dangerous than doing it with the Login plugin - it won't require FTP access to the config file to re-enable it.


priorities


I even fetch binaries on master cb2a05b826.


take a look at : unfiltered.util.MIMEType


It logs and returns `None`.


When python starts, it initiates the integers between -5 and 256.
When you are in a console, each line you type becomes a different code instance so it is a little different than executing a python script with the same lines.
If different variables reference an integer in the "low range" then they both reference the same integer initialized by python.
On the other hand, for bigger integers which are not cached, since each assignment is executed in it's own code instance, each assignment creates its own integer 27. That is why in this case the operator 'is' will report False (BTW, the '==' will still report True).


Oh, that's a bug. It should be `MAX(MAX(...)...)`


*...you know I believe it.
But I still haven't found
What I'm looking for.*
(U2, Used without permission)


Please remove console.log here


del


please remove console.log here


Actually it should be
```
		int len = MAX(MIN(self->params.plaintext_length, 8),
		              self->params.plaintext_min_length);
```
I'm committing a fix


and here


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1627


This should be reverted -- this cask should only include ST3 beta versions. Dev builds go in sublime-text-dev.


The new "retry()" method provides retrying of sending of the new task message to SB.
We noticed, that in some rare cases Complete() of the current message (see place in code where retry() is invoked) and sending of the new message, which run in one transaction, might fail du transaction failure. This exceptional case can be mostly reproduced when host is running On-Prem and Azure Service Bus in remote hosting center is used.

This method is called inside of the transaction, when Complete succeeds and Sending of the new message fails. When that happen, without of sending of the new message, system would stay in inconsistent state:
- Looping orchestration would not be dispatched any more and
- Orchestration instance would be still tracked as running.


Sure, where have you committed the changes?

On 14 September 2015 at 01:37, Marco Vermeulen <notifications@github.com>
wrote:

> I've made the tweaks on my side. I just need you to look into a failing
> test before I can merge:
>
> SpawnProcessTaskSpec."should not write the pid lock file if the process exits abnormally"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gurrie/gradle-spawn-plugin/commit/90f2396f49e1c418a99c1c2acabfa4462800c3a0#commitcomment-13203509>
> .
>


Looks like C&P error


Doesn't this break backwards compatibility to older xcode versions?


with CM12.1 20150906 shamu image, I can not found this menu option in Call Setting,
Do you drop this Phone number setting feature for long?
also I found I can not install MyPhoneNumber App from Google Play, I have to push it with
adb root
adb push Myphonenumber.apk /system/priv-app
adb su -c "chmod 0644 /system/priv-app/Myphonenumber.apk"
adb reboot
can you at least make this installation easlier for public users?


Sure


async-await is indeed wonderful! HDT uses it all over the place.
I'm not really sure where I learned about it... though, like with most things, probably a combination of msdn and stackoverflow.


why is this commented out?


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


Do you want me to make the changes mentioned above and resubmit?


I made the function private to avoid it being used externally as-is.


Is this needed for compiling or did you do it only for beeing able to use the new APIs?


it runs fine on my computer, but Travis CI is failing.
Coding tests in shell is kind of clumsy too, I might rewrite all this with Python+[Lettuce](http://lettuce.it/)+[docker-py](https://github.com/docker/docker-py)


Not all of our events have handlers.


The timeout should handle it, but if someone sets the timeout to 30s instead of 5s, it would be nice to not to burn system resources for 30s before failing. I'm adding a check.


good catch, they shouldn't be here - corrected


This should be:
 cd nuttx
 tools/configure.sh haltian-tsone/retail
 make


yes, that was fixed in 57dda69


Thanks fot this


didn't you just add those some commits before?


You are right. The existence of the settings.xml should be checked.


Fixes should be in separate PR. You really should learn a routine for keeping your fork up to date and keeping PR up to date and that will be easier lol.


Wow! This looks awesome. :+1:


Actually just found out, this also allows for tables:
local stringToAny = function(str) return ((loadstring("return " .. str) or function() end)() or str)  end

stringToAny("{test1=1, test2=2}")
stringToAny("{1,2,3}")
stringToAny("{{1,2,3},{4,5,6}}")

>{
>  test1 = 1,
>  test2 = 2
>}
>{ 1, 2, 3 }
>{ { 1, 2, 3 }, { 4, 5, 6 } }


Lol, that's really amazing, you looks fucking young.....Sorry, I just thought you are a student, so wanna provide an intern from one of teams that I have joined.

If you are interested in hirecaesar project, I will wanna catch you at that time :)


:+1: adamffyl is totally right :)


maven calls are really slow. The JVM has to start. Therefore it will take about a second or so.
I think this will be too slow for command line completion.


Sample for settings.xml:

http://maven.apache.org/ref/3.3.3/maven-settings/settings.html


Yes, you are right. I will go ahead and implement your suggestions.


@tebanep don't worry, of course there will be a version 9 :-) this is the master branch, containing the code for the future version 10
https://github.com/odoo/odoo/blob/9.0/openerp/release.py is still a 9 alpha


fast-forwarding develop to borshop-integration-1248-test/gh1247-eqc = 7bcbcd3c


Mostly copy+paste and skeleton work, nothing fancy yet.


This is probably not the place for such questions.. I guess you should use the forum but anyway..
- Ikea Koppla is an ancient product and not sold new anymore.
- http://www.hprtechniek.com/product/infrared-pir-sensors
- http://www.banggood.com/433MHZ-Wireless-PIR-Motion-Detector-for-Home-Alarm-Home-Security-p-941723.html
- http://nl.aliexpress.com/w/wholesale-trc02-rgb-controller.html
No ideas where to order etc.


добавил новый метод


здесь меняем только цвет!


просто вырезал и вставил
это я случайно, пока разбирался что к чему)


добавление аналогичной формулы


i'm at git auth


@lindamartin if my understanding is correct, TraitMethodReferenceTree is a "reference", not a "statement".


@davidfestal notice what I did here—hope it's OK?


see pull request #273


すみません。
一応こちら廃止した意図または理由を教えていただけませんか？
クライアントから質問が来そうで・・・


... layout will **be** used ...


@pynicolas ok so it should etend Tree ?


What does "Acc" stands for in SensorFragment:263


- completed


1: M111 unrecognized component name: legacy