Yes, maybe I'm being a little bit too self-pedantic. And maybe my comment is not clear enough. What I meant here is that the `onMemberUpdated()` function would have typically be reserved for the `updated` event listener, but I'm also making use of it here because it is necessary and convenient.


it is being done in `setCurrentUnit()`.
p.s. there isn't any `setUnit()` function.


Yep, I'll do a fresh build off of /develop and let you know!


ok


I thought it was clearer to the reader. 'Updates' aren't applied blindly. New values are assigned only if they're different.


Looks good. :shipit:


Same above.


Canola is misspelled.


a


Actually that's a great question, I'll just do some tests... I assume a view would inherit the access control of the inputs, but what if they are mixed? and a matview, where the data is actually *there*...? I'll just run a couple experiments and report back.


I'm taking a bit of a closer look at this one, so no need to dive in yourself if you're busy with other stuff.


yes i do here


Looks nice, but minor changes in TaintDetector and CommandInjection (testcode) are causing merge conflicts with my recent pull request.

I think that *getInjectableParameters* should use HashMap for better efficiency and readability.


恩，那个反斜杠是不需要的，你直接删掉就好了


Very much deserves some documentation and a comment/reference to the SWIM doc (since these are the "override" rules defined in there). They're also slightly different because SWIM has no notion of a "leave" nor does it keep "faulty" members around.


ok i see it


TAMAH!😁

On 26 August 2015 at 18:14, Rushty Baysantos <notifications@github.com>
wrote:

> Nag push na po kami kanina ng bagong update sir. Gumawa kami ng new branch
> then dun po namin nilagay :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TUPM-BSCS/iConfess/commit/bd6c80a1c0a208a51e28d4d4e4ff8d90534657bc#commitcomment-12899535>
> .
>


Hmm - let me have a play with this and see what's not working.

Ah - I think I figured it out. I think the code is okay, you're just passing invalid boxes in...

If you change the last bit to:

    if b.hasVolume() :
        bDown = f.formatToYDownSpace( b )
        self.assertEqual( f.yDownToFormatSpace( bDown ), b )

then it passes. It's because it is moving the same edge twice.

I'll take care of that case in the actual conversion.

I quite like the idea of doing both bounding box change and Y-direction at the same time - maybe call it formatToCortexSpace or gafferToCortexSpace, because it seems to be more used for ImagePrimitive conversions than necessarily EXR. If the methods are going to change name, maybe "fromCortexSpace", "toCortexSpace"?


The method lineColor() and nextColor() call eachother - is that nice?


Could you make a seperate Vector2 for movement instead of making a new variable on the AddForce line?


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/27932f8ab42d1e275a59a80b390454ab656e39a3


merging basho/yokozuna/hotfix/zl/just-do-aae-tiemout = 27932f8a into borshop-integration-542-hotfix/zl/just-do-aae-tiemout


yes it is right


basho/yokozuna/hotfix/zl/just-do-aae-tiemout = 27932f8a merged ok, testing candidate = 704b3c9c


I'm assuming this code effectively documents the format -- but this is totally ad-hoc/heuristic.


:sunglasses:


Why take away the ternary operator avoiding "(null)" on output?


Ditto.


Is that really necessary?


Now I see :D


Also, this code appears to mistake days for tenths of a week. @dpopcmh can you confirm what you're trying to do here?


Alright then.


I will rewrite the player controller with raycasting tomorrow/ or you can do it, I think we're going to need that instead of this, especially with the mechanics we're planning on using.


But, it is not related to function input arguments which is what 2XX category is for.


Also this code doesn't work.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.530](http://teamcity.monogame.net/viewLog.html?buildId=9837&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:24:48


Interesting code for subset fitting...


both are used in the test case: https://github.com/hchiorean/modeshape/blob/MODE-2109/modeshape-jcr/src/test/java/org/modeshape/jcr/JcrUnorderedCollectionsTest.java#L156

I made them `orderable` on purpose, to test the validations in the code - there's nothing stopping users from doing the same. Since they are used only in the tests, I'm not convinced it's worth adding separate "invalid" node types for them.


That's because Archi is not made for all the devices in the same way... You need to adapt them for each one .


What's the rationale on these cases? I've been returning -error when sending error packets too, I think it's fair.


Would it be better to use `gcloud docker push` instead of `gcloud -a; docker push`? Not sure there's much benefit to the "wrap everything" pre-amble.


Too many things in the initialization list, it's more readable like this i think.


HAHA. Nang-aano ka eh no bigs? **pabebe wave**

Joke lang po sir 😹

On Wed, Aug 26, 2015 at 9:50 PM, Neil Aldrin Bigay <notifications@github.com
> wrote:

> TAMAH!😁
>
> On 26 August 2015 at 18:14, Rushty Baysantos <notifications@github.com>
> wrote:
>
> > Nag push na po kami kanina ng bagong update sir. Gumawa kami ng new
> branch
> > then dun po namin nilagay :)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/TUPM-BSCS/iConfess/commit/bd6c80a1c0a208a51e28d4d4e4ff8d90534657bc#commitcomment-12899535
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TUPM-BSCS/iConfess/commit/bd6c80a1c0a208a51e28d4d4e4ff8d90534657bc#commitcomment-12903712>
> .
>


For this we need a 1.5 release tag on the repository, otherwise the package system will issue a error.


Mind if I make this `REGISTRY_ROOT` next time I'm in here? Conventionally, environment variables are capitalized.


again, there is nothing to stop user from defining node types like this, but there are validations in place to safeguard against this.
Again, I can change the node type but it's only used in tests.


There was a reason for `gcloud docker -a` worked better than `docker push` but I don't remember it


Which file are you referring to here for lines 13, 14, 16?

On Wed, Aug 26, 2015 at 9:02 AM, Eliot Sykes <notifications@github.com>
wrote:

> What do you think about lines 13 and 14 given the flash handling code on
> lines 16+?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kristophM/Blocitoff/commit/33b24b9e816bd7ac02394e3a78c16614dacd3121#commitcomment-12902598>
> .
>

--
Kristoph Matthews
Founder, Boxbee <http://boxbeeinc.com> - We're hiring
<https://boxbee.com/jobs>!
A CNN Top Startup
<http://www.cnn.com/interactive/2014/10/tech/cnn10-startups/index.html?hpt=te_r2>
Watch Boxbee Online <https://www.youtube.com/watch?v=lCt2QcinIlQ>
415.754.0699


Sure


lgtm


`selectedUnit` is being used for multiple purposes in the class, if you find its usages. I don't think its a good idea removing it.

Also, the `restore()` method itself is called by the `CourseBaseActivity's` `onCreate` before any initializations. By overriding it and calling `super`, we are actually ensuring that `courseData` & `courseComponentId` are populated before we do anything.


do it! thx @nlzet :+1:


ok


OK


oh .. i forgot to remove that line, while I was testing.


here


@hoholee12

Ugly is __________________ really lol


You are correct. But again i haven't cleaned up the code after making the suggestion Pavan gave yesterday - instead of returning std::function just the call the function. Hence you are seeing this obsolete code with exceptions. I will do the clean up today.


here?


yes


Fixed with https://github.com/alephobjects/taz-docs/commit/f797ce248a9d85af6d0283b9ba6e4c1398cddc6a


Yep, I just got that figured out this end as well. I don't think the code is entirely okay though because we're not taking care of inclusive/exclusive in X yet - here's another test we should include :

```
def testDisplayWindowCoordinateSystemTransforms( self ) :

	f = GafferImage.Format( 10, 10, 1.0 )

	self.assertEqual( f.formatToYDownSpace( 0 ), 9 )
	self.assertEqual( f.formatToYDownSpace( 9 ), 0 )

	self.assertEqual( f.yDownToFormatSpace( 9 ), 0 )
	self.assertEqual( f.yDownToFormatSpace( 0 ), 9 )

	self.assertEqual(
		f.formatToYDownSpace( f.getDisplayWindow() ),
		IECore.Box2i( IECore.V2i( 0 ), IECore.V2i( 9 ) )
	)

	self.assertEqual(
		f.yDownToFormatSpace( IECore.Box2i( IECore.V2i( 0 ), IECore.V2i( 9 ) ) ),
		f.getDisplayWindow()
	)
```

I think I'd rather refer to it as the EXR convention as it'll be more widely known, and Cortex is going to contain less and less image stuff over time. But I do prefer your shorter naming suggestion, so how about  `fromEXRSpace()` and `toEXRSpace()`?


ok you pass


Return limit:0 please, i got 20 items in all my exports without it)
In base 600+ forms)


i wanted to avoid comparison's every time they are required. It is not gonna cost anything too much, just three booleans.


This looks like a not completed refactoring?


So since we have the if-else block containing the flash notifications starting on line 15, I assume you suggest taking out 13 and 14? (`<p class="notice">`...


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


@pavanky @umar456 Exceptions are residual code of my earlier prototype, i will clean this up today while working on Windows version of dynamic loading.


I don't think this change is required. Please correct me if I am wrong.


Integrating message pass-through into the next release. Thanks for contributing.


I can write a feature spec. But the way to restrict this I imagine, is to create a policy using Devise?


You can remove this conditional now.


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1242


Yeah, I really need it, along with normalizing across the 3-month chunks in a longer series. I'll be working on it, but probably stop-and-go.

I had no idea Google Adwords could be used for that... Sounds amazing! If that could be incorporated, we'd have a really powerful tool. Glad I started on the refactoring though — this could get complicated. :)


That seems a bit confusing. That `Fail` will be called if the things *are* equal, but it would print `Not Equal:`.


Can you please submit this change as a separate pull request?


Rather than use exception for flow control, the method can determine which method to call using BsonReader#getCurrentBsonType


That last test won't work...

Creating a GafferImage.Format(10, 10) will actually create a display window of (0,0)-(11,11)

I think what you're doing that test based on is:

    f = GafferImage.Format( IECore.Box2i( IECore.V2i( 0, 0 ), IECore.V2i( 10, 10 ) ), 1.0 )


So do the tests successfully create nodes using these types? If so, I'd consider that a bug because being orderable and allowing SNS violate our self-imposed limitations on unordered collections.

I also would be concerned about a user finding these examples and patterning their own node types after them, and then struggling to figure out what went wrong. If you don't want to change the node type names, then please add a comment describing how these are invalid and used to check ModeShape verifications.


Yes, the maximal manual and section were intended to use all fields (not just the required ones).


This is incorrect. The current README.md in master is correct.


Oh sorry, my mistake. It looked like it was being done in `getUrlForWebView()`.


@Zomis that is a good point, that is the value that I had it set to for testing purposes and didn't change it back, I should have set it to at the least 60 seconds


warum ein neues "replace-pattern"? hätte man vllt besser auch mit `###LABEL###` arbeiten sollen?


Could we please consider a third parameter '$expire' so that I could determin/set it for the opt-out-cookie from the statistic plugin?


Shouldn't GetAuthSessionTicket wait for GetAuthSessionTicketResponse_t before returning a result?

From https://partner.steamgames.com/documentation/auth
> To guarantee a valid ticket, the client must wait for the GetAuthSessionTicketResponse_t callback.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/76) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


mulitple -> multiple


Actually, brain fart. Please ignore that last message!


Done.


nice fix m8


You had me convinced for a minute, but after thinking a bit more I was just beginning to smell a faint odour!


> `selectedUnit` is being used for multiple purposes in the class, if you find its usages. I don't think its a good idea removing it.

OK.

> Also, the `restore()` method itself is called by the `CourseBaseActivity's` `onCreate` before any initializations. By overriding it and calling `super`, we are actually ensuring that `courseData` & `courseComponentId` are populated before we do anything.

If it's called from the super classes `onCreate()` method, then it is guaranteed to be initialized after calling `super.onCreate()`. We don't need to also override the `restore()` class, unless we actually want to access the saved state bundle.


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - 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)


Eliminate in line 1 the comma in parentheses (or --head) (or --location)
Place period after "location" and capitalize "This" in line 2 (...moved to a different location. This option...)
Replace in line 3 "on the new location." with "at the new location."
Clarify in line 3 by adding "either option is": "If either option is used together with -i/--include or -I/--head, headers from..." (UNLESS it is the headers from all requested pages that are being used together; I don't think that is what's meant)


Hey! Can you send me a PR to dgilperez/schedule_attributes, please? I plan to maintain this as long as I'm using it in my projects.


typo -> The main


So I think the issue is still in the test.

These lines, I think, should be:

    self.assertEqual(
        f.formatToYDownSpace( f.getDisplayWindow() ),
        IECore.Box2i( IECore.V2i( 0 ), IECore.V2i( 10 ) )
    )

    self.assertEqual(
        f.yDownToFormatSpace( IECore.Box2i( IECore.V2i( 0 ), IECore.V2i( 10 ) ) ),
        f.getDisplayWindow()
    )

(This is based on currently only changing the y values, not changing from exclusive to inclusive)


:tada:


In Meteor 1.2, you'll be able to just return the `promise` object from the method, and the result will be delayed appropriately.


~okeanos authentication token. If you have an account you may find the authentication token at (Dashboad-> API Access) https://accounts.okeanos.grnet.gr/ui/api_access.


This just returns a size tuple without actually changing the shapes of ``set`` or ``val``, so I don't think this line does anything.


There we go - pushed changes to fix the empty height issue, and adding your test (with my tweak!)


It's done for all the vendors, please take a look on the following commit id

2528e33 IB/core: Pass hardware specific data in query_device


What am saying is do the page the same way you have been doing the rest of the other rest+angular pages but in order to have the validators, use a fragment to load them into the JS on the page , makes sense?


Return limit:0 please, i got 20 items in all my exports without it)
In base 600+ forms)
I made tests on export processor, but only returning this helps.


hmm .. now that I see it, we aren't using the `bundle` in our override, so yeah, it makes sense to move this to `onCreate`.


a more descriptive example.. how to send the file


What a weird typo


@manucorporat  Im also wondering why?


плюс фикс багодрожи


hey


Confirmed: reverting this fixes `pypyPackages.setuptools` build for me.


Done


See https://github.com/solettaproject/soletta/pull/564#commitcomment-12830155.


I was writing the test based on the assumption we should be changing inclusive/exclusive properly at the same time. The test asserts that the display window can be converted between the two conventions and yield the right result in both - with your changes that is no longer the case, and I don't think it really has any meaning...


thanks for the explanation :)


Please revert this commit. These files are automatically generated/modified by the build process and changes should not be checked in.


NA/EU updated
[2015-08-26T19:26:03.652+07] FileVersion: 1.11300.3654.17500
[2015-08-26T19:26:03.652+07] ProductVersion: 1.11300.3654.17500


Ack


There seems to have been a massive uprising of spaces amongst the peaceful community of tabs we've established throughout GafferVille.


Good catch


Since this does break the app I made a quick fix but I don't have permission to push it here. The code is:

```js
if (gestAge && gestAge.length > 0) {
        var weeks = 0;

        var qty = gestAge[0].valueQuantity ?
          gestAge[0].valueQuantity.value || 0 :
          gestAge[0].valueString ?
            gestAge[0].valueString.value || '0W 0D' :
            0;

        if (typeof qty == 'string') {
          qty.replace(/(\d+)([WD])\s*/gi, function(token, num, code) {
            num = parseFloat(num);
            if (code.toUpperCase() == 'D') {
              num /= 7;
            }
            weeks += num;
          });
        } else {
          weeks = qty;
        }

        p.demographics.gestationalAge = weeks;
        p.demographics.weeker = weeks;
      }
```


150 тук-туков имеется в виду


I had similar issues with the bullet colors too. I think it's because I forgot to save my scene before I committed. These are nice changes, and excellent documenting too by the way. I made changes to my project before checking to see if the repo was updated, so now we have a conflict. I pushed my changes to a new branch so you can see what I've done. But I'm thinking I'll probably just revert back, and go with your changes, then I can add in my SelfDestruct script after. I'll have to make sure it's habbit to always check for changes to the master before adding things on my local repo, hopefully avoid conflicts down the road.

I think we should go with your naming convention moving forward in this project. I hacked this code out fairly quickly, but I prefer to use full names like targetRotation vs targRot.


Use KASSERT() here.


Worth mentioning that nowadays the return value from process is being ignored. It's not really clear what the framework could do about it besides logging.


Can we just remove this premature optimisation as well?


This too already exists in uicommons module, you don't need it


And this one?


Keep me posted on your r10 results.


совершенно не нужно тут это поле.


It checks the shape and returns an error if they don't match.
In 0.4 it throws a DimensionMismatch, in 0.3 an ErrorException.


Since we've already done a memset(), do we need this?


Or this?


It probably wouldn't work when it's accelerating but I couldn't figure out a way to download images during acceleration without eating a ton of memory lol


All of these comments have been addressed. I still have a few left over from the last review that I need to address. I will let you know when that is complete.


I agree, Done.


I cannot remember what, but I do remember that there was a problem with that. I'm fairly sure it used to be `PriorityQueue` at some point but then I changed it back into that heappush stuff there.

Naturally I didn't document that anywhere and can only guess now :(


Ah thats nicer  - done.


Do we need to branch here? Can we not just use one general purpose branch?


The orientation detector stopped when we pressed the `fullscreen button` on the `PlayerController`, causing, orientation changes to go unnoticed.
So, it had to be started here again.


https://github.com/tnataly/url-shortener/commit/ebc24e765ee32bff6023398a7d4eab34abf51e0e , line: 9. Find optimal length


Ahh good point, thanks.


@mikepetersonccv , after last rollout to production, the contribution tab is extremely slow. Looks like one of the reasons is due to this change, specifically the inner join to person that was added here. This is the qry profile with that join...

![image](https://cloud.githubusercontent.com/assets/548180/9493123/5130d1f4-4bb2-11e5-9cce-e0a059e21fb3.png)

and this is without the join...

![image](https://cloud.githubusercontent.com/assets/548180/9493131/631a8108-4bb2-11e5-9f13-5967af2d02d8.png)

Notice there's a lot less reads/cpu for the two sql statements that are executed when the grid's SetLinqSource method is called. It also seems to generate some wierd sql...

![image](https://cloud.githubusercontent.com/assets/548180/9493466/8d282836-4bb4-11e5-9201-f5592e9106b7.png)

I also found that having the "includes" on the transaction and pledge queries slowed things down considerably (timeouts). Even though without them more queries are run (for each row), it still was much faster.


Oh, I forgot, we don't use `origin` term anymore, use `dest` or `destination` instead.


https://github.com/tnataly/url-shortener/commit/ebc24e765ee32bff6023398a7d4eab34abf51e0e , line: 21. Add 5.times...


Fair, thanks.


Sounds about right. That stuff was under heavy development back last winter, then I kept running into firmware issues and put it on ice, and only backported what I needed. The join stuff in current devel only came later, so I guess I just never ran into that. Always good to have some more pairs of eyes on critical code like that.


wow - static_cast<> - good one! :+1:


Ahh - I see what you meant.

Why don't I change the Format methods over to fromEXRSpace/toEXRSpace, after which this test will be a valid one.


And we don't need this `toString()` call in pipeline: https://github.com/churkin/testcafe-hammerhead/blob/3bf8deb8cc22f8e5c502889d8b6a12074a5f5b4e/src/pipeline/index.js#L119

Since `err` is always string now.


利用window.getTheme吧 =。= 设个字体颜色


'lib' is already in the LOAD_PATH, just `require 'shorter'`


Don't duplicate them like that, just ask for a hand to make a regex to match both.


Можно заменить на:
```haml
= link_to 'Главная', '#'
```


Cool, maybe I'll try it out. I use `gcloud docker push` when I do this operation locally and it seems fine.


cool !  -- my head is in the other branch at the moment so sorry if I am not following this so well :)

I will take a look at packaging later, for now I am totally ok if this is mac osx specific since we are testing on that platform and I imagine for 0.9 we will do a bit by hand to get this out.


Sounds good - do the box methods have much of a knock on effect throughout the rest of the code?


Лентяй :D


Can you please change that to a 'const string' instead of a hardcoded one?


the fastest way to do this is:
if (!obj->typinf->schema || !obj->typinf->schema[0])
or
if (!obj->typinf->schema || !*obj->typinf->schema)

(no function call)


looks good - great to have better error handling in NTY errors


`rescue all` is bad, catch the specific exception


There'll be a bit of a knock-on affect, but only in as much as I can remove a load of box.max -= Imath::V2i( 1 ) lines!


Hey,
I am a bit curious. Is the flush for linux, using syscall, actually working? What is the TCFLSH const doing? As far as I understand, the second argument for IOCTL should be the request code number, which then should be syscall.TCIOFLUSH. The third argument would be a pointer to data, which could be nil in this case.

I am still learning so if I am wrong, please explain the code to me.

Thank you!


@mrfelton - I'm looking at this today and I can't understand why I said:
> some of this code is duplicated

Maybe you cleaned it up since then or maybe I am just getting old :)


issue #24


Here validation logic is mixed with the URL normalization, it should be a separate step


ok, I'll add another function for it.


Cool. Sorry for the bother on this - in retrospect we should have done a full review of the low level stuff before pushing it through the rest of the code.


Integrated suggested changes. Should wait for API update which will be done tomorrow.


Is this _really_ a thing?


I did some cleanup @pulkitsinghal - forgot to note it on here


missing java doc parameter


testing a feedback comment


That's not very efficient, you try each `length` only once. I would add a few retries


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.928.a0a5c2102ece6b4e287f31913e07de194723fba0](http://build.palaso.org/viewLog.html?buildId=61918&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 464, ignored: 6 Build time: 00:06:23


Should put java doc comments to all the methods as they should be consistent.


OK


Woah woah woah!


профиль форми


This changes are causing FCs for com.android.phone
http://forum.xda-developers.com/showpost.php?p=62516921&postcount=1100


Let's make this const.


@nfeske my share of the release notes.


It's very weird this is happening out of the blue. That AMI+Type was a valid combo until today, I guess. Did they deprecate this base image, or is it one of ours.


I'm going to claim that it's entirely accidental. Not a hostile take-over at all. <ahem>


@abnero There is a wiki that contains a set of solutions for many of these checkpoints - https://github.com/Bloc/ios-exercises/wiki  Use that when needed.


Whoops, nice catch! 8bdfbfffe02c4e80ba8a6b2708a019c865bc6598


No, no and no to all 3.


Let's assign directly to h here to make a clearer match with the compute implementation.


Some params missing in doc comments


So good


форма регистрации.md


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1795 of 1795 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)


Are we sure that srcml really uses using_stmt in all cases? C# actually contains two distinct forms of using:
1. The "using statement" like "using System.Xml;"
2. The "using block" like:
```
using(var reader = new XmlReader(...)) {
    <statements>
}
```
Does srcml mark both of these with using_stmt? Depending on what srcml does maybe we need another entry in this enumeration.


License year is not consistent. Use 2015 as others


Good


You allow a user to manually set `id` and `clicks_count`, I don't think that's correct behavior


@chelmuth, @nfeske: please add this to staging - fixes a regression - thanks @cproc for debuging and helping


Use unlikely() here.


That's weird indeed - I just assumed the change hasn't actually been deployed.


there is also a camel case utility in the proj. update to that.


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1801 of 1801 tests passed)

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


Why did you use DatabaseCleaner instead of transactional fixtures? I guess, default settings would be ok


![image 6](https://cloud.githubusercontent.com/assets/13983096/9496496/e3688d06-4c10-11e5-891e-c748221a7625.jpg)


OK


谢谢，祝好


Thanks.


@evomellor , now that registrations have to become public eventually and are pulled into SHARE, the 2 middle points start to make less sense as 'registration' points (and I've seen a general inclination for people to not want to register unless they absolutely have to because of the public requirement and since everything on the project is timestamped anyway). I think, for the intro materials, it's probably better to keep it to the two obvious places where registration has a very useful function above and beyond the regular project (i.e. pre-reg and a the point when you need a DOI/the project is in a final state). So, I would take out the list.


LOL I didn't even notice that! LOL


You didn't realize that the way that GitHub account is protected is at the same security level as the GPG key. If the GitHub account is lost, the private key and the email account might be lost as well.


Samba upstream doesn't like six; prefer `from io open StringIO` here


Use KASSERT() here.


Cool, looks solid to me... @brianz should probably take a look at this one too.


Hi @satanasov,

the path it should not be: ```$phpbb_root_path/ext/anavaro/postlove/themes```, like line 44 ?


It's better to have the code to remove old URLs in the controller, where you're adding the new ones.
Here you just need to get them


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


Access modifiers are not provided.(private)


Está certo, se eu utilizar a sintaxe que vc colocou recebo um erro de "AttributeError: 'super' object has no attribute 'fechamento'"
http://stackoverflow.com/questions/1021464/python-how-to-call-a-property-of-the-base-class-if-this-property-is-being-overw
https://mail.python.org/pipermail/python-dev/2010-April/099672.html


Use constant() ??


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (753 of 753 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-8.87%25-green.svg)


@ctrueden @imagejan So, in my local branch,  i did a git rebase -HEAD~largeNumber and deleted the offending commit. Now i see in gitk that the rebase just stacks all the commits i didn't delete from the patch list ontop of where I starred the rebase: This loses the previous structure of the branching and merging. That kind of freaked me out.... but perhaps its no problem? Did I do the right thing?

Question)
 If i (force?) push my local branch to github, will it also wipe this comment thread off the face of the earth? If this commit is wiped out, then where does this nice conversation go?


a) this is not the place to ask about wordcloud, this is: https://github.com/amueller/word_cloud/issues
b) your question is about installing a python package in general. Better go to stackoverflow. You need a python installation with numpy, then you can just ``pip install wordcloud``.


Remove extra space at beginning of line.


It's a shame we can't force it to use a full year representation, but never mind.


still not the right place to ask this question, but closer.


There's Rails routes and route helpers for that


comment here. grant


Are these really necessary?
Any extra change is painful to push upstream.


hahah


I think it's useful in this context to distinguish between what we've called 'tagging' and 'linking' because we're doing both, but perhaps Finding Things already have conventions for these names which we should follow - any thoughts @rboulton?


add comments


We can pick a locale that defaults to full year representation, however it seems that the rhel6 version has this format hardcoded (haven't checked the source).


I might keep this one as-is just to match the style in the rest of the default heroku buildpack. If we end up modifying the buildpack further, we can adjust the style then.

Sound good?


Add unlikely() here.


This allows a race condition. If 2 people click the link at the same time, counter will be incorrect


FROMAGE OUI OUI OUI HON HON HON.


This change is not neeeded – it works regardless of whether map returns a list or generator


Correct, Fixed.


Doesn't match the class name.


The path was preceded by a forward slash, breaking the route on redirect.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.445.cadd6d27e41d806dbb480110f455190e9a0e3b6c](http://build.palaso.org/viewLog.html?buildId=61925&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 262 (260 new), passed: 2985, ignored: 235 Build time: 00:03:34
### Failed tests
```
Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_CreateItemAfterFirstCall_EntryIsReturnedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAllItemsAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteByIdAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_ModifyAndSaveAfterFirstCall_EntryIsModifiedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_ModifyAndSaveMultipleAfterFirstCall_EntriesModifiedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_CreateItemAfterFirstCall_EntryIsReturnedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteAllItemsAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteByIdAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_ModifyAndSaveAfterFirstCall_EntryIsModifiedAndSortedInResultSet: <no details avaliable>

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


I tried that, and it didn't build. I forgot to include `st.h`!


Why twice?


Replaced the unit test default image.


develop (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1278 of 1278 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)


Yes, pretty much on all variable evaluations.  Basically, we want to avoid any spaces in the variable causing it to expand to multiple arguments.  Even if we don't know a reason why it's unsafe, I always put variable evaluations in quotes if I don't specifically want the parse-by-IFS behavior.

```
Michaels-MBP:~ mlyle$ HELLO="hi there"
Michaels-MBP:~ mlyle$ mkdir test
Michaels-MBP:~ mlyle$ cd test
Michaels-MBP:test mlyle$ touch $HELLO
Michaels-MBP:test mlyle$ ls -al
total 0
drwxr-xr-x    4 mlyle  staff   136 Aug 26 07:47 .
drwxr-xr-x@ 244 mlyle  staff  8296 Aug 26 07:47 ..
-rw-r--r--    1 mlyle  staff     0 Aug 26 07:47 hi
-rw-r--r--    1 mlyle  staff     0 Aug 26 07:47 there
Michaels-MBP:test mlyle$ rm hi there
Michaels-MBP:test mlyle$ touch "$HELLO"
Michaels-MBP:test mlyle$ ls -al
total 0
drwxr-xr-x    3 mlyle  staff   102 Aug 26 07:48 .
drwxr-xr-x@ 244 mlyle  staff  8296 Aug 26 07:47 ..
-rw-r--r--    1 mlyle  staff     0 Aug 26 07:48 hi there
```


ping @abhinandanprateek

I'm trying to understand the purpose of this variable and the next one. I understand they control when some of the tests are skipped.
What I don't get is: Why would we want to skip tests that we included in the build?


use e here as well?


Check if blueflame area is write combined ...


interesting :-) Thanks!
I'll merge the pull request and change some cosmetics in the script later.

btw: gmt.exe is usally included in the archives from openmtbmap. So it's really an issue of not depending on wine or linux-gmt. I'm happy to get rid of the dependency though.


In order to show a user his shorturl below the form; Without reload it doesn't appear.


I don't think you can do it that way. List comprehensions are written in the same order as you would see `for` loops.


The size of this diff should be enough reason in itself to stop the space madness.


develop (axelstudios) - 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)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.446.38e44070d8cb6dbb2ebe853917778268cf19f3c5](http://build.palaso.org/viewLog.html?buildId=61927&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 269 (7 new), passed: 2985, ignored: 235 Build time: 00:03:16
### Failed tests
```
PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.UnusualKeyboardId: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.RegexLayoutAtEndOfKeyboardId: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.RegexLayoutInMiddleOfKeyboardId: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.RegexLayoutAtBeginningOfKeyboardId: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.FallbackToFirstKbd: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.NoVariant: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.DefaultKeyboard_WithVariant: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_CreateItemAfterFirstCall_EntryIsReturnedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAllItemsAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteByIdAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

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


I was intending to use standard partials (introduced after this commit) and to help me detect duplicated content I've split all "suspect" paragraphs to have one sentence per line.

This has now been done in the last commit in this PR.


Use `if sys.version_info > (3, 0): unicode = str` in the befinning of the file, rather than `six`. Samba doesn't like dependencies.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.468.cadd6d27e41d806dbb480110f455190e9a0e3b6c](http://build.palaso.org/viewLog.html?buildId=61924&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 137 Build time: 00:09:44


@ccx-testerbot runtests feature-cea-708


Thank you for the request. It has been added to the queue (id: 77). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/77) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


ok with the changes save as is


Great work - Thanks a lot


oops, "to 10" and not "to 5"


Great, thanks! Can you submit a PR?

  -J

On Wed, Aug 26, 2015 at 10:13 AM, vlad-ignatov <notifications@github.com>
wrote:

> Since this does break the app I made a quick fix but I don't have
> permission to push it here. The code is:
>
> if (gestAge && gestAge.length > 0) {
>         var weeks = 0;
>
>         var qty = gestAge[0].valueQuantity ?
>           gestAge[0].valueQuantity.value || 0 :
>           gestAge[0].valueString ?
>             gestAge[0].valueString.value || '0W 0D' :
>             0;
>
>         if (typeof qty == 'string') {
>           qty.replace(/(\d+)([WD])\s*/gi, function(token, num, code) {
>             num = parseFloat(num);
>             if (code.toUpperCase() == 'D') {
>               num /= 7;
>             }
>             weeks += num;
>           });
>         } else {
>           weeks = qty;
>         }
>
>         p.demographics.gestationalAge = weeks;
>         p.demographics.weeker = weeks;
>       }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpopcmh/growth-chart-app/commit/ffacc3fa58819a139afa12976c31bacf0d81da70#commitcomment-12904371>
> .
>


OK


Use `from io import StringIO` here as well


I wanted to keep import behavior consistent across files, but if you think upstream is more likely to accept fewer changes I'll remove the unnecessary ones. :)


Thanks @dnwick, this has been corrected by @ayash


Сервер не будет знать о юзерской температуре, потому что это ни на что не влияет: если залочено - то не меняем, если не залочено, то меняем на режимную.


Good point.


Ого. How about just take the length of the newest link?


Shoot! I was specifically invoking modernize with --no-six, this got here by an accident. Will fix.


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


But you have handlers for html, json, and js, do you really use all of them?


@patrickfrancis Yea, sorry. I was afraid I'd accidentally push one of those files.


Read more about this hint here: https://www.igvita.com/2015/08/17/eliminating-roundtrips-with-preconnect/


Hm how would you improve the name? The filter checks if the contests match the ontology terms in the specified focus area, I guess we could call it ONTOLOGY_TERM? But then the focus area part gets lost, maybe ONTOLOGY_TERMS_IN_FOCUS_AREA or something?


В основном репозитории эта ошибка уже исправлена.


Goes away with a gain <= 0.2554

On Tue, Aug 25, 2015 at 10:38 PM barnabyrobson <notifications@github.com>
wrote:

> Does the problem go away if you put a gain on the output ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpiponi/Moodler/commit/5974fb0265cb0d4a7ee46127e7c18d72ea7ee9ce#commitcomment-12895415>
> .
>


Кроме HTML ещё что-нибудь получилось сделать?


* alignment is different to other function prototypes
* why is there an empty line?
* Adding a prototype to the main header file means, we have to implement the function (at least as stub) in all implementations of ctrlucal-*.c


@solael it seems that the commit are done by zhao ? is that normal ?


shouldn't there be a generic account doing these operation like feelpp-robot on the server


I believe these files are only updated when you build with msbuild. If you only build through Visual Studio then they won't be modified and you won't have to worry about it.


on other error continue???
I think we should add:

else
{
    return kErrorGeneralError;
}


有时候第一次wait，程序如果还没有退出，就会失败。实际上再重试一次就好了。


This is only true if you are logging in.

If you are logging out a fingerprint is not enough since you are testing whether the idp cert is set here:
https://github.com/onelogin/php-saml/blob/master/lib/Saml2/LogoutResponse.php#L167

I guess someone forgot about the LogoutResponse when creating the fingerprint feature.

This ofcourse invalidates this documentation since the statement is false. No, you can not use a fingerprint instead of the the whole x509cert since the processSLO uses the LogoutResponse that doesn't support it.

This really threw me of. Just had a customer that had to pay me 2 days extra for debugging why their saml logout wan't working as intended.

Should be very easy to fix.


My mistake


@hoholee12 I hate fucking '[['... Is supposed to know var without "" & other stuff, but I didn't get work it, without "" & with "", nothing... I will use it never, go to the hell!! lol


"that'll be only 1,000 silver then." works better when using numbers IMO.
I think that you wanted to translate so it read "a thousand silver", but in number form "1,000" is read "one thousand" so it doesn't fit that well.


-amase ui
+ama*z*e ui
连框架名字都打错了


SIGKILL程序必须会退出，所以得死等。


I mean the commit message title has a typo!!! :D :D


There's now some nil class issues when calling merge_runners. But I'm kicked out of the Momento right now and have to look at it later... O.o


Yes .. I think this may be the overflow problem I was seeing. Right now the portaudio stream has a flag which is set to "paClipOff" and the docs say "Disable default clipping of out of range samples" so .. that may be the difference between the two styles. In the old style it may be handling values that are over max_short better ... I think in general we should solve the overflow problem such that we can use that no clip flag but I'll play with the setting to see if it does better in the case of overflow.  I noticed if you plug a audio_sin to the output directly you get a signal over 1 and it overflows so I'd like to fix that too.


`//` operator


two fold question, keep in mind i do not have any prior experience with any programing methods, I am running Freecad and have tried to instal curaengine plugin by using the terminal and copy/past the initial comads cd~/. Freecad MOD .... frome the "easest way to integrate Curaengine is to clone the document at the right side of the screen." then restart freecad and it will pop up in your workbench.  Im not doing something right and am at a loss to integrate the code, i can see all the code files. and the terminal has ran a long list of code withought an error message but again i dont know what im looking at


Don't you translate any more?


typo nrd_record_ipv4 =>nrd_record_ipv6 ?


Maybe you can pick some rules of this config and apply it to build? Its easy to follow rules of checkstyle as of it has fast check on `mvn validate` (i.e. before tests)
https://github.com/jenkinsci/github-plugin/blob/master/src/test/resources/checkstyle/checkstyle-config.xml


This does not work for me (Firefox 40 on Linux), since `console.log.toString()` returns
```js
function () {
    [native code]
}
```
(including the newlines). So maybe replace some of the spaces by `\s+`, and wrap the expression in `^…$`? Otherwise, the regex content might appear as part of a literal string within the function, leading to a false positive.


second part how do i set up cura to be Delta compatable i have the external probram on my Mac


Unfortunately, we don't always know exactly which domain a method comes from, hence why we scan each domain. Here's one such place: https://github.com/mono/mono/blob/master/mono/profiler/proflog.c#L1035


Да, согласен. Добавил.
Предлагаю расценивать соединение с тарантулом, как составной процесс:
    1. TCP соединение
    2. Аутентификация
Реализовал, исходя из этого утверждения.

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

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


Do you have search on archive, there is already this question... (but no remenber the answer !)


Close #1


@Phil-Friderici Does this work? Spec-tests not running on my vdx :(


See my comments


This could just be an `else` instead of `elsif arr1[0] < arr2[0]`


Agree!


sadad


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 421](https://teamcity.labkey.org/viewLog.html?buildId=268572&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:26
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.469.38e44070d8cb6dbb2ebe853917778268cf19f3c5](http://build.palaso.org/viewLog.html?buildId=61937&buildTypeId=bt223) outcome was **FAILURE**
Summary: Compilation error: PalasoUIWindowsForms.Tests\PalasoUIWindowsForms.Tests.csproj (new) Build time: 00:01:32


The ctrlucal_getFd() is only implemented in the ioctl CALs since in e.g.

hostif stack libraries it is not needed, is this also valid for
ctrlucal_getMappedMem() ?

Von:    Wolfgang Seiss <notifications@github.com>
An:     zelenkaj/openPOWERLINK_V2 <openPOWERLINK_V2@noreply.github.com>,
Kopie:  zelenkaj <joerg.zelenka@br-automation.com>
Datum:  26.08.2015 17:01
Betreff:        Re: [openPOWERLINK_V2] [FEATURE] Add user CAL modules for
Windows PCIe interface (5f4f951)

alignment is different to other function prototypes
why is there an empty line?
Adding a prototype to the main header file means, we have to implement the

function (at least as stub) in all implementations of ctrlucal-*.c
?
Reply to this email directly or view it on GitHub.


祝好，感谢！


The ansible-debian-mediagoblin submodule isn't properly structured as a role, so, yeah it won't work. I'm fine with getting rid of it for now.


@katinatez you don't have problems when building for bacon?


oops...


No we add the translation framework for the gui later. As it would be a separate one


@ThomasBurleson Why are you switching it to classes? Is the plan to support both attributes and classes now?

The layout attributes work very well and help you keep general styling classes separate from layout attributes.


this could be nice, but unit tests don't comply "coordinate is ndarray"


Is optimized code 1000 times faster  than debug code?


"that'll be only 1,000 silver then." fits better.

I think you wanted to translate it so it read "only a thousand silver", but when using 1,000 in numeric form it is read "one thousand", so in the end is read like "only a one thousand silver"


Oh my..


Glad you figured out my hellishly poor comment on the previous commit! Must have been on the wrong meds. ;-)
Now to figure out how to merge you changes into "my" branch. :-s


You've accessed the array element an unnecessary second time. Should just be `arr1[0]`, not `arr1[0][0]`


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


does it make sense to migrate dbetabinom to TMB ... ?


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **82/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/77) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


i.next() is called in the line below (github automatically collapsed that line):
`Proposal p = i.next().getLeft();`
`i.next()` needs to be called in the first iteration, that's why it has to be in the body of the for loop.


@jmrenouard why do you inline Text::Template instead of listing it as an additional requirement if you want HTML reports?


This function has a typo in it's name that should be fixed before merged into development.


Whats the reason for this?


Shouldn't we do the same for arrays as well to support symtable serialization?

```php
$a = 1;
unset($a);
unserialize(serialize($GLOBALS));
```

> Notice: unserialize(): Unexpected end of serialized data in /home/nikic/php-src/t157.php on line 5
> Notice: unserialize(): Error at offset 5139 of 5140 bytes in /home/nikic/php-src/t157.php on line 5


Recht haste
https://github.com/redaxo/redaxo/commit/fa5ade04aa55a9930817b0d21901b841673c0c9a


IMO an argument for the flags is missing here, something like 'private', 'public', 'MSDN'. That has to be adjusted in the subsequent code also.


If this test is purged from cw-7cores results then any advantage is gone...


oh, didnt see :+1 :)
On Aug 26, 2015 5:15 PM, "Johannes Bachhuber" <notifications@github.com>
wrote:

> i.next() is called in the line below (github automatically collapsed that
> line):
> Proposal p = i.next().getLeft();
> i.next() needs to be called in the first iteration, that's why it has to
> be in the body of the for loop.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CCI-MIT/XCoLab/commit/4336898fd22de46e0b5ff0af10a02afde75e4ece#commitcomment-12906038>
> .
>


重复


@bzwei -  I'm curious why the "deployments" method changed to "list"?


This does not work for me (Firefox 40 on Linux), since `console.log.toString()` returns
```js
function () {
    [native code]
}
```
(including the newlines). So maybe replace some of the spaces by `\s+`, and wrap the expression in `^…$`? Otherwise, the regex content might appear as part of a literal string within the function, leading to a false positive.

(Sorry, meant to comment here – you can leave out `gulp dist` if you want, I’ll do that before publishing anyway).


try


Here, you handle the special case where there is only 1 element left in each array. It is possible to write the `while` loop on line 25 such that it will handle this case nicely. May be easier to explain in person, happy to do so!


yes, will fix this


统一合并


`Hastbl.iter (fun sr _ -> if not (* entry in srs *) then (* remove from Hashtbl sr_cache *)) sr_cache`


already have obj_ctx here, no need to do the casting again


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.470.cfc24843cf1c495a85e8908a290d3add7cf8f24a](http://build.palaso.org/viewLog.html?buildId=61938&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 137 Build time: 00:05:17


Yes, and it is expected now that we make sure we clean both batch groups


do we need this? I noticed it's for c# files. @Colbitron


develop (EnergyArchmage) - 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)


This is a fantastically elegant implementation of the merge sort! Super easy to read and understand. I ALMOST THOUGHT YOU WROTE IT YOURSELF!! lol


shouldn't s->bucket_info already hold the bucket info?


Currently it is not possible to pass an `ExecutionEnvironment` defined with the scala api to a method written in java that works with the `org.apache.flink.streaming.api.environment.StreamExecutionEnvironment`.
Added a getter to to get the scala StreamExecutionEnvironment for java scala interoperability.


none...  remnant of some debug prints.  Sorry for that...


OK


What about banners, (demo) slideshow, flags and icons?


ah, that's for the source where we read the data from...


I think this callback is useless


same here


nice


This is just the default gitattributes generated by either GitHub for Windows or Visual Studio 2015. We can remove it if you want as we probably aren't going to be dealing with C# and ASP.NET applications for your app anyway, but leaving it there doesn't hurt anything either.


Absolutely correct :+1:


You're right. Thx for spotting!


I googled for "string site:www.winpcap.org/pipermail/pcap-ng-format/" and none of the results seemed to show an answer.  One from Guy mentioned it in passing:
http://www.winpcap.org/pipermail/pcap-ng-format/2012-July/000051.html

...in there he talks about using a zero-length string to indicate "an entry with no value".


TeamCity Azmq :: Build (Windows) [Build 162](https://78.46.71.115/viewLog.html?buildId=595&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 132](https://78.46.71.115/viewLog.html?buildId=594&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 115](https://78.46.71.115/viewLog.html?buildId=596&buildTypeId=Azmq_BuildOsx&guest=1) is now running


I don't think we have to check for overlapping RunDays in our merge candidates. I've never seen that as an issue.
If you still think about implementing this, I'd rather see this in `find_runners_only_differing_in`, since then all merge methods will gain that improvement.

Further, I'd like to be a bit more specific about which one of the runners is the correct one. why not use the criteria from below, using the the one with most accents (I argue that it's easier to miss an accent than to add a wrong one). I'll comment in a method you could use.


TeamCity Azmq :: Build (Linux) [Build 132](https://78.46.71.115/viewLog.html?buildId=594&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:07


maybe better to return fullCwd in the 'else' case?


@ctrueden, @hinerm, @dietzc

Here is the projection example.  It is a little bit obtuse, but maybe that is OK for the tutorial, as we can show how to use ops that do not have ideal interfaces (yet).

Let me know if any of you have suggestions to make this script cleaner.

Projections currently derive from  AbstractComputerOp, so we have to create memory for the output and pass it to the op.  @dietzc any reason for that??  In the future should the projections derive from AbstractHybridOp??


Thanks! I think `JSONStream` sounds hella cool.


Is it worth noting in the doc comment for this method and the next one that batches (in this case) or records (in the next case) will be skipped over if they result in errors?


TeamCity Azmq :: Build (OSX) [Build 115](https://78.46.71.115/viewLog.html?buildId=596&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:02:20


Yeah, is this something your system added?


TeamCity Azmq :: Build (Windows) [Build 162](https://78.46.71.115/viewLog.html?buildId=595&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:02:45


Nice implementation. One place you could improve this is around the 2nd and 3rd while loops. Because you're merging two HALVES of the initial array, you will never have more than 1 element left over after the first while loop finishes (when the starting array has an odd number of elements).


So, I tested it with oracle jdk 1.8.0_05 and that works.
I did some debug session.
This is the NPE stack trace:
```java
java.lang.NullPointerException
	at org.eclipse.californium.scandium.dtls.ECDHServerKeyExchange.<init>(ECDHServerKeyExchange.java:131)
	at org.eclipse.californium.scandium.dtls.ServerHandshaker.receivedClientHello(ServerHandshaker.java:557)
	at org.eclipse.californium.scandium.dtls.ServerHandshaker.doProcessMessage(ServerHandshaker.java:249)
	at org.eclipse.californium.scandium.dtls.Handshaker.processMessage(Handshaker.java:295)
	at org.eclipse.californium.scandium.dtls.ServerHandshakerTest.processClientHello(ServerHandshakerTest.java:212)
	at org.eclipse.californium.scandium.dtls.ServerHandshakerTest.processClientHello(ServerHandshakerTest.java:198)
	at org.eclipse.californium.scandium.dtls.ServerHandshakerTest.testReceiveClientHelloNegotiatesSupportedCertificateType(ServerHandshakerTest.java:162)
```
This happen because in `ServerHandshaker.receivedClientHello(ClientHello)`
`ECDHECryptography.fromNamedCurveId(negotiatedCurveId);` returns `null`.
This happen because in `ECDHECryptography.createKeys(ECParameterSpec)`, an exception is raised `InvalidAlgorithmParameterException: Unsupported curve`.
This exception was catched in `ECDHECryptography.fromNamedCurveId` which logged :
`"WARNING: Cannot use domain parameters of supported group [secp256k1] to create ephemeral keys: secp256k1" then return null.`

After some investigations, it seems that openJDK support only secp256r1, secp384r1 and secp521r1 (See sun.security.ec.NamedCurve class).

So I see 2 problems :
1) the tests do not passed on openjdk anymore, because of the addition of
`extensions.add(DtlsTestTools.newSupportedEllipticCurvesExtension(0x0016));`
we could fix that by using another curves supported by openjdk (e.g. secp 256r1)
2) Scandium `ServerHandshaker.negotiateNamedCurve()` should not accept curves which is not supported by the JRE which run scandium. A solution could be to call `ECDHECryptography.fromNamedCurveId()` in `negotiateNamedCurve()` and refuse the group if we are not able to create `ECDHECryptography` with it (I mean if `ECDHECryptography.fromNamedCurveId()`  returns null) and so send alert Handshake failure. Maybe this will be even better if negotiateNamedCurve returns directly the ECDHECryptography object ?


section_def_focus_area_filter
On Aug 26, 2015 5:00 PM, "Johannes Bachhuber" <notifications@github.com>
wrote:

> Hm how would you improve the name? The filter checks if the contests match
> the ontology terms in the specified focus area, I guess we could call it
> ONTOLOGY_TERM? But then the focus area part gets lost, maybe
> ONTOLOGY_TERMS_IN_FOCUS_AREA or something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CCI-MIT/XCoLab/commit/4336898fd22de46e0b5ff0af10a02afde75e4ece#commitcomment-12905658>
> .
>


Yes. Can add.


Its because I use rbenv and rbenv-gemsets so I dont pollute my global ruby installs with lots


The comment for this method still mentions the filtering – needs to be updated.


ahh yes thank you!


Just realized that this is going to mutate the SparkContext's Hadoop configuration; I think that we should work with a copy of the configuration instead.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.447.cfc24843cf1c495a85e8908a290d3add7cf8f24a](http://build.palaso.org/viewLog.html?buildId=61939&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3353, ignored: 134 Build time: 00:10:07


You're saying the issue should present on Linux in the Quark benchmark scenario?  I at least see light and dark grey output.  If not, would you mind posting some screenshots of observed and expected behavior?


:yellow_heart: The summary for this commit message is a little misleading. It is recommended that the summary be about the changes that occurred in the commit.


Good thinking, a Pundit Policy is one layer of protection, that'd be a good thing to do.

Another layer of protection to consider adding would be to do as many of the operations as possible scoped to `current_user`, for example:

```rb
def create
  @item = current_user.items.new(item_params)
  authorize @item # Add this line when you've got the policy written

  @items = current_user.items

  if @item.save
    ...
```

Note how we're no longer relying on the vulnerable `params[:user_id]` to tell us which user to work on.

Instead we're using devise's `current_user` to ensure we only ever work on the logged-in user.

Even if a user tampered with the user id in the URL (which would change `params[:user_id]`), they would not be able to modify another user's items.

Also note that this controller does not require a user to be logged-in currently. Consider adding a `before_action :authenticate_user` to the parent `ApplicationController` so authentication is on for all controllers by default. This is the more secure approach than relying on remembering to add the `before_action :authenticate_user` to each new controller class.


Thanks @liewegas , this looks great to me.
I've been unable to test it today. I will build it tonight and give it a good run through tomorrow.


Зроби 2 різні класси зі значимими назвами, а не геометричні фігури)


> Here is the projection example. It is a little bit obtuse, but maybe that is OK for the tutorial, as we can show how to use ops that do not have ideal interfaces (yet). Let me know if any of you have suggestions to make this script cleaner.

Actually we could have a `project()` namespace which implements the basic projections functions + one more general one which takes a `Class<O> extends Function<I,K>>` as input. However, we can't make sure, that `<I,K>` matches the provided input / output. But maybe thats good enough. Anyway, this pattern (using/passing ops in/to ops) will appear over and over again in the future if we try t really reuse existing ops in our code while maintaining extensibility and flexibility.  We should discuss if we can generalize this pattern a bit and simplify method calls (e.g. as described above)) at the hackathon.

> Projections currently derive from AbstractComputerOp, so we have to create memory for the output and pass it to the op. @dietzc any reason for that?? In the future should the projections derive from AbstractHybridOp??

Yes, we just have to make sure that the input is `n-1` dimensional and the dimensions fit.


fixed in pull #30


ahh so jekyll is installed on a project basis using the .rbenv-gemsets


Rule of thumb: when you update `#include` lines, update the Makefile.


Compártelo


for me yeah it is. But I think unless you use rbenv-gemsets or the rvm version it juts installs it to that ruby version globally


Conocéis [overcommit](https://github.com/brigade/overcommit)? En muy bueno para prevenir que se cuelen este tipo de cosas. ;)


Specimen class or type cannot be changed once specimens are collected.


I'm not sure what you are asking :-)


I think we should just remove theese three last lines as we talked about on the pull request discussion.


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1278 of 1278 tests passed)

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


The whole idea is that if all the three (or more) runners have runs in only different years (e.g. Michel: 2009, 2010; Michél: 2011, 2012; Michèl: 2013), then it's likely that they are the same person...

And who has then more accents, Michèl or Michél? ;-(


I have this error: in Getting the Novint Falcon working with Linux. Can you help me please...

BuildSysCMakeLib.cmake:493 (MESSAGE):
   LIBNIFALCON requires an out of source build (make a build dir and call
   cmake from that.)

   A script (Makefile or python) should've been included in your build to
   generate this, check your project root directory.

   If you get this error from a sub-directory, make sure there is not a
   CMakeCache.txt in your project root directory.
 Call Stack (most recent call first):
   BuildSysCMakeLib.cmake:21 (MACRO_ENSURE_OUT_OF_SOURCE_BUILD)
   CMakeLists.txt:23 (INITIALIZE_BUILD)


I believe you forgot got `pod trunk push` to update the version available via cocoapods. See issue #129. If you search Cocoapods for youtube-ios you'll also see it's still at version 0.1.3. https://cocoapods.org/?q=youtube-io


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.28.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=61932&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:35:29


:+1: sounds solid to me


develop (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1278 of 1278 tests passed)

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


ah nice, that's a better name. Thanks!

I'll drop the filter at the end to fit the naming of the other enums if that's ok (and the full name will be `ProposalPickerFilterUtil.SECTION_DEF_FOCUS_AREA`, so the filter is already in the enum name)


:+1: Thanks a lot


You can just do ```var request = require("request");``` instead of wiring it in.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 9](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=123&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


wow, that is strange.  I tested this against ccvdev and it made it 10x faster.  I'll go thru it again and see what is up


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-MacOS-10.9-clang: OK (1794 of 1794 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)


Oh, I see.


Going to that URL gives me a 404. Are you sure that's right?


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 9](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=123&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:15


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1801 of 1801 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)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.448.cadd6d27e41d806dbb480110f455190e9a0e3b6c](http://build.palaso.org/viewLog.html?buildId=61941&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3346, ignored: 134 Build time: 00:09:46


@jmunhoz there are a few needed cleanups here that I can see, but before that, my question is whether you tried putting the functionality at RGWCopyObj, rather than at RGWPutObj? Seems somewhat more appropriate


Forgot to mention @itmike, he made the swedish translation, I added into my branch, sorry.


Why are you ignore xml files?


Hi Jürgen,

@Transactional(propagation = Propagation.SUPPORTS, isolation = Isolation.REPEATABLE_READ, readOnly = true)

if you have this transactional configuration, you would also get the warning, although you do not need a transaction. Is this correct?

Hanh


Could you please add the license headers


```ssh app1.hoodq.com "docker stop platform && docker rm platform && docker run --name=\"platform\" --restart=\"always\" -d --net=host registry.hoodq.com:8080/platform"
platform
platform
b3e3316ce0ce096ab0a6abb3023c83467703ff04aae6ea0a2dd9071bd6debf9c
ssh app1.hoodq.com "until curl 172.16.0.10:8080 >/dev/null 2>&1; do echo -n \".\" && sleep 1; done"
...............................ssh app1.hoodq.com "sudo /opt/hoodq/haproxyctl/bin/haproxyctl \"enable server staging/hqp1\" "

ssh app1.hoodq.com "sudo /opt/hoodq/haproxyctl/bin/haproxyctl \"disable server staging/hqp2\" "

ssh app2.hoodq.com "docker stop platform && docker rm platform && docker run --name=\"platform\" --restart=\"always\" -d --net=host registry.hoodq.com:8080/platform"
platform
platform
05d9423e0ed4568a6c8dd3b0cd01ad0f7af71da77c53126c0c6c607792c014e0
ssh app2.hoodq.com "until curl 172.16.0.5:8080 >/dev/null 2>&1; do echo -n \".\" && sleep 1; done"
...................ssh app1.hoodq.com "sudo /opt/hoodq/haproxyctl/bin/haproxyctl \"enable server staging/hqp2\" "```


이 부분을 문자열 format() 내장 함수로 처리했다면 str()함수와 문자열 + 연산을 반복적으로 하는 것을 방지할 수 있겠지요.


Wow, I also seem to have forgotten to remove my new textures.....eh, do with them as you wish I suppose


awesome :D


@davidkhess thanks, updated in current working branch.


А name обязательное поле? Оно может быть такое же как id?


Is this send being used to block the HTTP request in ServeHTTP. This is relying on the fact that the channel has a buffer length of 1. If we increase the buffer size wouldn't this test fail? It would be nicer if we could have this request be served by a HTTP handler which parameterizes the sleep timeout and we can see that the write succeeds when the sleep is less than the timeout and fails when it's more.


typo


This should probably be `model = models.StudentGrade`.


This is actually a feature. XS opens .exe files as a solution, so you can run and debug them. So exe files should not be filtered.


Actually I was suggesting that for the first branch of the if.

For the second branch I think it should be:

    (weighted * filtered_mask).sum() / (mask * (weights != 0)).sum()

pending architectural changes


Do you have search on Wireshark mailing ?


cmeiklejohn/riak_ensemble/features/csm/namespaced-types = af137c76 merged ok, testing candidate = bb49f2ab


merging cmeiklejohn/riak_ensemble/features/csm/namespaced-types = af137c76 into borshop-integration-80-features/csm/namespaced-types


saw approval from javajolt
at https://github.com/cmeiklejohn/riak_ensemble/commit/af137c76ccc65c357faee9c5fd4b66622e3a7911


while 문과 open문에 list 문을 이용하는것은 좋지 못할 수도 있습니다.
오히려 while 이 아니라 for 문을 사용하면 시퀀스 자료형인 리스트 사용할때도 좋겠군요.


101789044_gridviewFilters (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


@mdo we use an older version of this style guide over at Khan Academy, and I noticed that the `ul` has been removed from in front of `.category-list`. The comment to the side "element + class namespace" seems a little out of place.

Is this intentional? Should `ul` be there, or is it better to leave it out?


This needs to have a `Startup.OnThread` otherwise thread_zero will become fully devoted to updating the physical utilistaion and Xapi will not finish starting up.


I suppose we can do that but I tried to be homogeneous with `ServerHandshaker.receivedClientHello(ClientHello)`.
Maybe we can move this in constructor in ServerHanshaker too, if we decide to do that I prefer create an another pull request/commit as this is not directly link to session resumption. (If you're agree)


Consistent or not.  This is not good enough.
We have  M-x R    and so I want to keep  M-x julia


@scottwainstock The API URL is really `/api/v1/regions/closest_by_lat_lon.json` :smiley:


[Source](http://honnunarstadall.hi.is/ritgerdir_0) from the University of Iceland


Take a look at the other modules that use `Xapi_startup.OnThread` because I don't think either the `Server_helpers.exec_with_new_task` nor the `Debug.with_thread_named` are necessary. Neither start a new thread and (for reference) I think the former uses the latter anyway.


А у разных тегов не может быть одинаковых классов?


Also adds a bunch of images


4590-VRF-defrost-power-calculation-problems (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1801 of 1801 tests passed)

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


The checksum should be `54bfe4db98f4e34b883121ad9a3c92ec22cd10d7a0a90c40815d6ecce178876b`.


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


Oh, it was because it didn't like the different blocks of changes or something


I'm ok with that. I didn't look deeper into that but I will probably need a ResumingClientHandshaker operational. So I should probably focus on this first ?


suggest to put the css in separate file, using the fs / insert-css combo.


Oh, do you mean to update the patches to run first?


No idea ... added back in 2007 with the comment
! Attempt to stop loops on BBC parsing. (Subs.php)


mono_compile_method () returns an ip belonging to the _current_ domain, while the mono_jit_info_table_find (domain, ip) looks it up in 'domain', so for domain!=current domain, the lookup will fail.


suggest to remove console.log


While you could change it to a sleep, you introduce more issues.
By causing the next request to block, we are expecting the timeout to trigger. We don't want to race against a sleep which will appear as flakes when it doesn't work.

We could make an explicit channel of 1 but either way if some one changes the assumptions above the test will fail which is a good thing.


Whoops. Thanks for the heads up. bbca0af91f5e75901d7ec66a58c2fdc4dd0faf41


would it be an idea to use class-style selectors (e.g. active, selected) and define styles in external css? javascript would add/remove the appropriate tags on mouse-over / mouse-out.


Can you add a comment here describing why this is negative? We're guessing that it's used to simulate a timeout without actually having to sleep. But it's confusing at first.


would it be an idea to use "class" tags for this and define the styles external css?


Oubli d'enregistrer le fichier ! Le commit suivant inclut les modifications ici décrites.


Why was this file added?


Hello,
when will you update this on atom.io ?


Overall looks good, a couple of questions and one addition request.
Juda


Self doesn't need anymore


Thanks for the quick work @jbrantly


@roflo1 yes, the postfix increment operator is slower for non-primitive types.


Thanks for making these changes. In my browser, the names show up nicely and the lines don't seem to change their appearance (see attached image). Suggest to make the selected/ deselected different in the lines more visible.

![screen shot 2015-08-26 at 9 01 33 am](https://cloud.githubusercontent.com/assets/1084872/9498906/16630bc2-4bd1-11e5-8878-f153eded0ae1.png)


Are these guys meant to be the singular `component` instead of the plural `components`?


Can you change https://github.com/dotnet/corefx/blob/master/src/System.Diagnostics.Tracing.Telemetry/ref/System.Diagnostics.Tracing.Telemetry.csproj#L5 as well?


Do you have some PCampReview testdata which has more than one MV in order to be able to test properly?


Что такое спек? )

Мы просто проходили это на вебинаре и в скрин-касте так https://htmlacademy.ru/intensive-participant/15/resource/45#step38


@Angelfentanez
Tienes conflictos inyectados.


17:51:06 ∙ Ironii: Stanzilla: ill just report it here, the new moving condition is triggering multiple times during spells like mind flays etc
17:51:18 ∙ Ironii: might want to add a spam check
17:52:32 ∙ Ironii: i think spells like that trigger PLAYER_STARTED/STOPPED_MOVING once for each frame


Do we need to include the / there?


Limnoria runs best with CPython 3.3 and newer, but also supports 3.3, 3.4, 3.5.

Something bothers me in how these two lines are expressed.


This commit would be better name "CP-13861: Add the ability to send guest agent features to xenopsd".


How else does one set DependsUpon?


cool! :)
On Aug 26, 2015 5:38 PM, "Johannes Bachhuber" <notifications@github.com>
wrote:

> ah nice, that's a better name. Thanks!
>
> I'll drop the filter at the end to fit the naming of the other enums if
> that's ok (and the full name will be
> ProposalPickerFilterUtil.SECTION_DEF_FOCUS_AREA, so the filter is already
> in the enum name)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CCI-MIT/XCoLab/commit/4336898fd22de46e0b5ff0af10a02afde75e4ece#commitcomment-12906731>
> .
>


Getting  formObj.isValid() is not a function


Should use `Features.name_of_feature` here.


As soon as I get access to my dev PC, which should be somewhere near the 1st of Sept. Sorry for the wait.


This commit allows the following to function properly:
```python
from commands.client import ClientCommand
from commands.say import SayCommand
from commands.server import ServerCommand

@ServerCommand('test')
@ClientCommand('test')
@SayCommand('test')
def test_command(*args):
    print(len(args), args)
```

Previously, only the last one would work properly, so the first two would do nothing.


ok no problem !


Sorry, it was caught in https://github.com/docker-library/mysql/pull/93 and just needs to be synced back over here.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.448.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=61944&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:17:21


Здесь должны использоваться часы virtual clock, потому что таймер влияет на состояние машины.


[070] and [090]

I can rebase my changes on top of the latest `master` before we merge the PR. Taking care of any conflicts in the process.


Worth adding a comment that explains why `bool_of_string` will not raise an exception here.


"canola", not "canols!


haha you're absolutely right! ;)


@tomsmeding Why not? Magister returns nicely escaped HTML content, while we want text content.

Now that I'm thinking about it, we could add an option to get the raw HTML content from Magister.


Use YouTube's embed iframe with autoplay enabled, and then use CSS to set display: none; on the iframe itself.


What do you think of this @djazayeri? The reason for this is to be able to use these field widgets and still be able to add angular directives, i was thinking of adding another option named showErrors so that one can opt out of displaying field errors since with angular UIs they become useless.


Thanks for this.

However, I think the correct fix is to disambiguate regex patterns from comments (e.g. change the pattern to "/[^/]"). See the git log for line 209 for why regex patterns are needed.

If you can provide a PR that does this, I'll happily merge it!


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/223


Same ? re: `component` vs. `components`. Doesn't seem that any particular type is being enforced for either one in `propTypes`, so I'm not sure yet if it matters or if they're usable interchangeably. Will look for the code that handles the values...


I've both google'd and used my email client's archive of emails for
wireshark-dev (5 years worth), and can't find anything about that. Although
figuring out the right terms to search for is hard...

-hadriel

On Wed, Aug 26, 2015 at 11:49 AM, Alexis La Goutte <notifications@github.com
> wrote:

> Do you have search on Wireshark mailing ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hadrielk/pcapng/commit/d4a4032e07322cf436823789c3ef1c2394623e17#commitcomment-12906948>
> .
>


LGTM


with this patch on dual sim phone SystemUI crashes on boot and stop on "starting applications".
in logcat it reads:

D/AndroidRuntime( 1391): Shutting down VM
--------- beginning of crash
E/AndroidRuntime( 1391): FATAL EXCEPTION: main
E/AndroidRuntime( 1391): Process: com.android.systemui, PID: 1391
E/AndroidRuntime( 1391): java.lang.NullPointerException: Attempt to invoke virtual method 'android.view.View android.widget.FrameLayout.findViewById(int)' on a null object reference
etc

I fixed it with this patch:

--- SystemUI/res/layout/msim_super_status_bar.xml
+++ SystemUI/res/layout/msim_super_status_bar.xml
@@ -43,20 +43,25 @@
                    android:layout_height="match_parent"
                    android:scaleType="centerCrop"
                    android:visibility="invisible" />
-        <com.pheelicks.visualizer.VisualizerView
-                   android:id="@+id/visualizerView"
-                   android:gravity="bottom"
-                   android:layout_gravity="bottom"
-                   android:layout_width="match_parent"
-                   android:layout_height="match_parent"
-                   android:clipChildren="false"
-                   android:clipToPadding="false"
-                   android:visibility="visible"/>
     </com.android.systemui.statusbar.BackDropView>

-    <com.android.systemui.statusbar.ScrimView android:id="@+id/scrim_behind"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent" />
+    <FrameLayout android:id="@+id/scrimview"
+                 android:layout_width="match_parent"
+                 android:layout_height="match_parent"
+                 android:visibility="visible">
+        <com.android.systemui.statusbar.ScrimView
+                android:id="@+id/scrim_behind"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:importantForAccessibility="no"/>
+        <com.android.systemui.statusbar.VisualizerView
+                android:id="@+id/visualizerview"
+                android:gravity="bottom"
+                android:layout_gravity="bottom"
+                android:layout_width="match_parent"
+                android:layout_height="match_parent"
+                android:visibility="visible"/>
+    </FrameLayout>

     <include layout="@layout/msim_status_bar"
         android:layout_width="match_parent"


Точно! А ну тогда лучше оставить ul? Потому что как сделать выделенный класс, если ссылки не должно быть?

1

2

3

4


ok, please merge manually


Why did you remove the old mapping?
IMO they are valid methods to access the `EntityType` as well.


Perhaps this should be up to the user to choose: `epoll` = Linux only; may want BSD `kqueue` or the `select` default.


Fell free to push the fix to our Gerrit (review,carbonrom.org)


Now you'd have to manually edit the project file :(

There are a couple of cases where the IDE will set it automatically (when you add CodeBehind etc).


Space missing :anguished: ?


PGM has it like that, i can remove it if you want


А это тоже было на вебинаре так https://htmlacademy.ru/intensive-participant/15/resource/45#step11

Вообще главную мы разбирали на вебинаре, я просто её тоже сверстала, а потом по скринкасту проверила, просто для себя )


I wasn't sure if we want to add a retry here too. Thoughts?


You don't need the `@compat` macro since `include_dependency` is an ordinary function call that is defined (as a no-op) in Compat, and `Libdl` is defined and exported as an ordinary module.

You only need `@compat` to handle syntactical changes (like `UInt(7)`) that cannot be implemented as ordinary function calls in 0.3.


:+1: :+1:


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2310 of 2343 tests passed)

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


Por supuesto! Un error tipográfico. Gracias por avisar. Corregido


console.log - prawie zawsze usuwamy na koniec pracy z kodem


I liked pushing the `merge` button. It felt as if I am contributing with something;)


Makes perfect sense. Would be nice to know it works before I move it though... Any existing test examples from glmmadmb?


rób wcięcia - to zwiększa czytelność kodu


Missing a return statement in the latter branch ;)


skrypt PRZED zamykającym znacznikiem "body"


ugh


thx dad


972124-2NewDetailedIceStorage (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1794 of 1794 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)


random.sample() 은 참 편리한 함수죠. 간단하게 잘 짜셨습니다.


Our discussion today covered/ answered your question, right?


Why memcmp() rather than strcmp() here?


还是原来的名字好。


podobne regułki staraj się grupować ten kolor "#ccc", zamiast nadawać oddzielnie, mógłbyś dodać w ten sposób:
`a:link,
a:visited,
a:active
{
color: #ccc;
}


nice


This definition, and the corresponding one in Rect, means that a Vector will compare as equal to a tuple (and any other iterable type with the same value) - I see you're making use of this fact in the unit tests. This seems a bit dubious and un-pythonic - Rect in particular has lots of behaviours that don't exist on an ordinary tuple, so it shouldn't compare as equal to one. I'm concerned that this could lead to subtle bugs where one bit of code passes a tuple where another bit of code expects a Vector / Rect.

Would you be happy to add an `isinstance(other, Vector)` check here, and update the unit tests to do things like `self.assertEqual(tuple(rect), (50, 50, 250, 450))` instead?


staraj się unikać wywoływania funkcji w HTMLu


Tests!? I can't believe my eyes :wink:


usage :
type a_start in console to load alarms and start alarm thread
type a_count in console to see number of loaded alarms


"Friday" nie "Friaday" :D


r+


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/92d9a347954f026b537928927686a609f23b8805


merging basho/yokozuna/hotfix/zl/just-do-aae-tiemout = 92d9a347 into borshop-integration-542-hotfix/zl/just-do-aae-tiemout


basho/yokozuna/hotfix/zl/just-do-aae-tiemout = 92d9a347 merged ok, testing candidate = 4b59473d


remove path from signature


This isn't getting the userid from JSON; it's getting it from the route.  This is the right thing to do, it's just misnamed.


make this internal


.Contains rather than IndexOf


Make this just ```\\.staging\\```, and make ".staging" a constant.


I'm new to github and gerrit :)
If I send you in email (to support@carbonrom.org) the patch it's ok ?


Didn't see the discussion here and raised https://github.com/jgm/pandoc-templates/issues/116. It seems like `\PassOptionsToPackage` can avoid the clash.


remove comment


refactor ".staging" out


skrypt przed zamykającym znacznikiem "body"


Verify that the RootPackage exists (it won't if it failed to load for some reason.) Also consider making GetDepth static - it can take in RootPackage.


add a space before this method


define this closer to its actual use. Also add ```.staging``` to this.


memcmp() vs strcmp().


Then you need to remove the `ngettext` functions^^


I'm pretty sure you don't need this interface. The `auth` package is the only part that cares whether something is signed, and it already defines its own `Signed` interface.


Is there a particular reason for this clause in the last paragraph:  "provided that supported peers are known to possess any omitted certificates they may require." ?
It seems that only self-signed certs may be omitted, in which case the peer needs to have prior knowledge of them anyway, or am I missing a use case.   It seems this clause insinuates other certs besides the self-signed root may omitted.


Working to fix this


实体对映的DaoParams实现（Demo）。具体可以生成。


@prudhomm Fixed.


감사합니다... 네트워크 부분을 수업을 못들어서 인터넷을 레퍼런스해서 채팅 만들어봤습니다.


You're probably right, but if I reuse the Connection what should I do with the previous establishedSession attribute. I suppose we can keep the last etablished session waiting the new one was etablished but in this case, the `getSession()` will probably not do what it was expected.

Not directly linked, but I just thought that create 2 connections in the store with sessionEstablished which have the same sessionID will probably imply unexcepted behavoir for`ConnectionStore.find(SessionId)`. :confused:


Set the VersionRange to nothing because it won't work with duplicated packages.
// TODO make it compatible with version ranges.


Refactor _allModules.Add into AddModule call, and verify that we are properly checking for depths.


moduleDir should always end with ```node_modules```, so optimize for that.


Add more comments.


Do we really need the sign check here?


You never ended up using `IsSigned()`, so let's get rid of it.


3 razy powtórzyłeś "webDays[i]". Ogólnie zasada jest taka, że jak coś się powtarza kilka razy w funkcji - robimy z tego zmienną. W tej sytuacji powinno pomóc np. coś takiego `var element = webDays[i]` i później używasz tylko zmiennej `element` np. `element.style.height`


how about this: extern void node_render_test();


괜찮습니다.
과제 검사 하다보면 이게 어디서 나왔는지는 다 알지만 보고 만들었어도 그것을 자신의 것으로 만든다면 가치있는 일입니다.
남이 짜둔 훌륭한 코드를 계속 보고 다음에는 응용 할 수 있도록 합시다.


De uma lida nisso aqui:
https://en.wikipedia.org/wiki/Null_Object_pattern


zamiast "0px" używaj po prostu "0"


grupuj - `header, footer { height: 60px; }`


Tem certeza que está certo isso?


Should be LoginDatabase


Thanks for making an effort, @chalkie666! It sounds like you got really close.

Unfortunately, by choosing "largeNumber" for your rebase, you indeed flattened out all the old merges, which is no good. What you typically want to do instead is `git rebase -i HEAD~4` where `4` is the number of commits back you need to edit. You can use `git log --oneline` to quickly see your recent list of commits.

Another way is to do `git log --oneline`, find the hash `123abcdef` of the last problematic commit, and then do: `git rebase -i 123abcdef^`, which will let you edit everything up to and including that commit.

One alias I really like is:
```shell
git config --global alias.graph "log --date-order --graph --decorate --oneline"
```
After running that command, you can write `git graph` to see a nice tree view of your history, including branches and tags labeled by name. This is nice to verify that after a rebase, commits sit on top of one another properly in the expected places, with branch structures preserved as expected. You can pass multiple arguments to it, too—e.g., `git graph HEAD master origin/master` is nice to check where your branch, local master and remote master all sit relative to each other.

> If i (force?) push my local branch to github, will it also wipe this comment thread off the face of the earth? If this commit is wiped out, then where does this nice conversation go?

The conversation stays, but gets adjusted to show a message like "So-and-so commented on an obsolete commit". PR threads are pretty resilient in general.


it says 16 here in the denominator. should be 24.


Then 32 bits are definitely enough.


넵ㅎㅎ 그래도 안돌던 소스로 회생을... 늦은시간까지 고생이많으십니다.
내일 뵙겠습니다.


Can we get by with `map[string]interface{}`?


upgrade username to muarine


Models from database cannot be added to canvas.  Error ---'module' object has no attribute 'Datavalues'


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1245


What's wrong with changing the user?


Why? Mmm.. "this" here is a D3 selection context


Кажется, что это не последняя версия. Есть удобная команда `mvn versions:display-plugin-updates` чтобы проверить все ли модули последних версий. И аналогичная для зависимостей: `mvn versions:display-dependency-updates`

Собственно, версии можешь не обновлять, если лень.


Don't scare me!


What is [FromServices] doing and how does it work? Why are you no longer defining a controller constructor with context? I'm confused as to how the new code is dependency injection. Can you please explain?


See issue #1655


I'm looking into betabinom examples.


Я всегда убирал `language="java"` т.к. не вижу в нем смысла (но все кто копипастят из интернета его обычно добавляют :) ). Без него всегда все работает и писать нечто лишнее и без понимания зачем оно нужно, мне не нравится.


sorry it's more of me trying to check "is null", but... oh, i see that it probably would return false for 'ok'.  you're correct, not necessary


Nitpick: These two lines can be replaced by `Set... = new HashSet<>(Arrays.asList(new TopicPartition(..)))`


there may be some in the bbmle vignette


Many of the other responses document the possible values for `errorCode`, see `JoinGroupResponse` for example.


lgtm


develop (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1278 of 1278 tests passed)

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


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1797 of 1798 tests passed)

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


I just found an example (written by bbolker) using `emdbook`. I'll reimplement it in glmmTMB.
http://stackoverflow.com/questions/11705062/generating-beta-binomial-responses-with-covariates


ideally, we should comment when an error will be returned.


LGTM


Can you change this to something more like `expect('this was called').toBe(false)` or something like that? That way if it's ever called and the test fails, the output will be a little more telling.


why do you put 12 case after 13 case?


nm, i see why


I know this is the field name in `StopReplicaRequest`, but I wonder if the name makes it obvious that it's actually a boolean.


:fearful: The files property was deleted, now the gem has no files when I get it from bundler!


@KenthMobiTV Guys I understand that this works but this is not the best solution and it does not fix the first part of the if statement just that one item.

data: "data:image/"+images[j].imagetype.toLowerCase()+";base64, " + images[j].__text,

We are wasting cycles by parsing items in the xml2Json parser and then not using them.  Why can we not just apply the properties in the cue to the _children object.

:-1:


Thanks for pointing that out :)


Is this supposed to be position === 'static' ?  IE if the node is statically positioned then the offsetParentIs not the body?