You should do like the [Google Chrome guys did](https://code.google.com/p/chromium/issues/detail?id=405894#c52), i.e., check for the string `.find-login-action.` in the extensions bundle. Although that's not exactly orthodox, that will make it possible for other password managers to work without any further development on Firefox for iOS.

One of such password managers is [pwSafe](http://pwsafe.info).


This will break if other tests start adding users.
Never rely on hard coded primary keys.
User `self.me.id` instead.
(fixed in https://github.com/dangerfarms/tf-api/commit/393ad72)


For current installations one must include `apps_path` under `[storage]` within `scitran/config.toml`, like so:
```
[storage]
mongo_path = "/root/sdm/scitran/persistent/mongo"
data_path = "/root/sdm/scitran/persistent/data"
apps_path = "/root/sdm/scitran/persistent/apps"
```


What found you xD


@jonboulle Complained that AuthoritiveRegistry seemed to Docker specific.  Could we use a little more generic name for this like yum uses AutoritativeRepository, AuthoritiveURL?

ALso why is the difference between DistributionScope and License?


improved readability: entity could be anything, but we know it is an invoice. so let's call it invoice.


lolz


Lol.


merge


self.me.id is dynamic, won't break if other tests add users, changing the the PK of the user.


As dori ca sa argument sa primeasca un obiect de tip Category caruia tu sa ii extragi id-ul!


I havent made any codebits :(


Actually wait, maybe that should still be handled differently here too...


Hi Cachhupin. They are fun, btw.


KK xD


This is me.


I found you using the URL thing anyway :D


Nu cred ca este corect. Tu spui "selecteaza toate evenimentele unde ID-ul lor (a evenimentelor) face parte din lista de ID-uri de categorii. Cred ca in loc de BaseColumns._ID trebuia sa folosesti BaseSQLIteHelper.COLUMN_CAT_ID sau ceva de genu!


IRsendDemo.ino: In function 'void loop()':
IRsendDemo:21: error: 'class IRsend' has no member named 'sendSony'
'class IRsend' has no member named 'sendSony'


This causes an issue (Variable "label_format" is not defined) with Symfony < 2.6.
I have made a PR to resolve this: https://github.com/sonata-project/SonataAdminBundle/pull/3042


TeamCity YaccConstructor :: YC.Core General Build [Build 440](http://teamcity.codebetter.com/viewLog.html?buildId=192949&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Why not just use /sneak and have arguments?


Idk how to make a new one xD


I think we need to be careful here as we are changing the existing behaviour. For example, if as a user I specify that `overwrite` is false, then I would not expect my files to be overwritten under any circumstance (including if the modification time is older).

This should perhaps be rewritten from:
```java
if(!dest.exists() || ( overwrite == true ) || ((EXistResource) res).getLastModificationTime().after( new Date( dest.lastModified() ) ) ) {
```

to -

```java
if(!dest.exists() || overwrite == true || (overwriteOlder && ((EXistResource)res).getLastModificationTime().after(new Date(dest.lastModified())))) {
```


..because we use new stateiconimginfo, we need this files to prevent error
main server don't have this yet... only on sakray
and i believe you dont want people posting a thread in herc just because they dont have updated KRO.


this commit lgtm


1679-DualDuct (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


NSLog?>


Turn your email settings off with this. (If you haven't already.


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 tests passed)

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


not ready for re-review yet.. just realized Tyson had the same exact PR basically from 5 months ago.. never got looked at i guess


Aquesta ubicació no és correcta. Falta el directori config/, cosa que provoca un error en actualitzar la MV


Thanks you I was stuck on that xD


@nell-byler You can embed a plot from the [figures page](http://www.pnas.org/content/109/41/16474.figures-only) like this:

![ alt text ] ( link )

![Fig 1](http://www.pnas.org/content/109/41/16474/F1.medium.gif)


Wait making a new codebit


Thanks for spotting this problem @paultcochrane !


Do we have to do CODEACADMEY programming can we just stick on scratch ?


this is confusing. we should support one.


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 tests passed)

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


I'am not that good to be honest with html


inline comment


I dont have an opinion- just saying


general comment


this and the two above looks suspicious. why do you want this behavior?


We can stick to Scratch, I guess.


You're welcome, Cachhupin.


teehee


Nice catch :)


no_raw_loops (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

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


Firefox for Kids: very good idea.


Sometimes Idk whats more complicated. Scratch, or Javascript :|


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


Lol, yeah.


you should really use a go template for this and not human readable output.


Then after a month we can go on codeacadmey cauz i need to know properly how to do stuuff on it ..... Can  you  makee Studiou brand new and me and you will test people and see how good they are ?


Eh eh. :)


Im doing a competition for members for ShadowBox Inc- Maybe try that (:?


Thank you for picking up my slack on this! Several things have intervened over the last couple months, and I feel like I haven't had near the time I'd like to write code for non-work stuff.

I really want to get the vagrant VM back up and running on my computer and see if I can get the tests running locally. I feel like that's a big hindrance for me; I don't want to write lots of code if I can't also make sure it does what it's supposed to do.


Regarding to Alatis
![5304508_60x60](https://cloud.githubusercontent.com/assets/12817263/8065595/53cb2410-0eda-11e5-8aaf-7f884af75463.png)


closes #3


turns out the representation issue is fixed with 6.29


This is freakin awesome !


Ik just a suggestion :D


Yeah lets give all a people a task and then invite them if its perfect :full_moon_with_face:


Btw, I will try to make it so the settings win can be a std member and not a pointer, which is error-prone.


I accidentally added the `phpunit.xml` in this commit. It should have been in a later commit. Anyway, now it's here.


On 06/09/2015 01:53 PM, Daniel J Walsh wrote:
>
> @jonboulle <https://github.com/jonboulle> Complained that
> AuthoritiveRegistry seemed to Docker specific. Could we use a little
> more generic name for this like yum uses AutoritativeRepository,
> AuthoritiveURL?
>
I suggest AuthoritativeSource - Repository is a Docker term that has a
different meaning and URL is already a field for a link to human
readable information. At the same time, I still think people should just
settle for the name "Registry" for the place to retrieve containers ;-).
>
> ALso why is the difference between DistributionScope and License?
>
DistributionScope is different from License as there are other factors
than the software license(s) of the included software or the aggregate
work governing the distribution: In a company you might be building
images from Free Software, but you still may treat them as private,
non-redistributable. For a vendor like Red Hat, the stream of content is
governed by the subscription agreement, and while redistribution would
not be a violation of the license terms, it break the subscription
agreement.

Another concern with the License field is, that we should define whether
it applies to the content or the aggregate work of putting together the
image. If it's the content, would you list all licenses or just
"Multiple"? Is there value in listing the license of the aggregate work?
Might be too thin to protect anyways.


It looks as though this line is throwing some warnings?
Warning: Don't set the props property of the React element. Instead, specify the correct value when initially creating the element.

Are you seeing this on your side?


Alatis  ; Then after a month we can go on codeacadmey cauz i need to know properly how to do stuuff on it ..... Can you makee Studiou brand new and me and you will test people and see how good they are ?
 give all a people a task and then invite them if its perfect :full_moon_with_face:


For once, I agree. This is not what git means when it says "you are about to rewrite history" when updating a commit. The copyright notice should stay on 2015 and not be changed back to 2014.


Notification icon still appears on any screen size.

Message disappears when on mobile view (i.e. up to 768px) on both normal and detached modes.

Normal screen view:
![sign on message appears on normal view](https://cloud.githubusercontent.com/assets/3407469/8065630/a1c37c82-0eb0-11e5-8d34-019ffb8d6a21.png)

Mobile screen view:
![sign on message gone on mobile view](https://cloud.githubusercontent.com/assets/3407469/8065627/9de0cb9c-0eb0-11e5-866b-9b3dedcf3261.png)


Im sharing a Updates and Announcement project soon so I can see who could be a possible addition to the company.


@thanhdut That is the most compact solution ever completed by one of my students. What made you think to use MIN and MAX? Was that the first idea or was the For loop first and then the MIN/MAX?


Not anymore after next commits. Nice catch, I'll remove it.


Wait 'Free Proffesional Logos 2' has hit 500 Comments :D


> you should really use a go template for this and not human readable output.

There's no base64 decode in the template and base64 is guaranteed to be installed when this script runs.


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


Unfortunately not. Otherwise, the config doesn't get created.


And I've done my 3rd request :D


Alatis can you make a new account u chhose the name then you make and intro thumbnail and studiou i will give out tasks and we will judge for the account pass email me ; ayaanshazeb1@gmail.com


I do not think we need Kubernetes in this help text.

We do not say "Create a new Kubernetes pod", so why say "Create a new Kubernetes secret"?


Looks good. One thing I don't see is a test to verify future dates


TeamCity YaccConstructor :: YC.Core General Build [Build 440](http://teamcity.codebetter.com/viewLog.html?buildId=192949&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 136 Build time: 00:12:43


I'm also wondering if we should do `urbn_humanReadableStringForTimeInRelationToCurrentDate`


no_raw_loops (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

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


Does this help text make any sense either?  I pick on it only because of its use of Kubernetes, but who knows what Secret.Data is as an end-user?  Can we update this to be "List of filenames or directories to populate the data elements in a secret"?


Alatis you on ?


Zerty did ask politely and professionally. Throughout his posts today he never made a single personal attack, nor did he attribute bad motivations to when you removed the license file, which would be what 90% of people would have done.


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 tests passed)

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


De-pineappled: 7fa1647887baea1272c1fa922c226f8cb0ced1bc


:+1:

I'm ready https://community.centminmod.com/threads/php-5-7-phpng-support-testing-for-centmin-mod-betas.892/page-5#post-14467 !


> I do not think we need Kubernetes in this help text.
>
> We do not say "Create a new Kubernetes pod", so why say "Create a new Kubernetes secret"?

already commented on and fixed in the later comments commits


I'lk remove the other save config call at app end.
I'm saving before sound because the ogre cfg file doesn't store sound options.(Maybe it can?)


Anyone ??


TeamCity YaccConstructor :: YC.Core General Build [Build 441](http://teamcity.codebetter.com/viewLog.html?buildId=192950&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


closes #3


Im on but Im bored so Im sitting looking at the ceiling xD


I tried it. Value = 0 assings some invalid value to r, g and b, but it has no visible effect on screen. Values greater than 1.0 should not be possible from a math perspective, but may be possible if rounding errors are present?


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 tests passed)

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


Time.deltaTime is scaled by Time.timeScale, you'll need to do the same with your deltaTime value for games that like to slow down time using Time.timeScale.


This version of devise doesn't exist
I think it should be v3.5.1


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 tests passed)

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


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

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


The else case here seems unnecessary, no?


Two suggestions: (1) do not use `c` as a variable name, since that is already defined in base R. (2) the `df <- rbind(df, more_rows)` idiom is slow compared to
```r
result.list <- list()
for(commit.i in seq_along(commit_list)){
  one.commit <- commit_list[[commit.i]]
  result.list[[commit.i]] <- time_commit(test_path, one.commit)
}
```


no_raw_loops (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


I seem to remember asking for the `got_ip` call to be inlined into the `assert_raises`. Turns out that wasn't just a cosmetic issue. The code as it stands is buggy. [Fixed](https://github.com/weaveworks/weave/commit/bd364f787e63e88604a94bb42153b37baf87b4ac).


Consider providing a more meaningful argument name. `e` does not tell me much.


Sorry I wasn't clear, but why should we treat SQLite any differently from MySQL? Neither implements `CASCADE`, so either both should log a warning or both should throw an exception (I think the latter is better for the reasons I stated).


Just pushed an updated readme with an Issues section


Copied this from Simon's API. Will bring this up with him.


Alatis ?????????/


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

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


I did not realize that these lines are repeated twice.
Fixed.


ok. I had missed previous save (line 172). Forget my comment


Ok.


I was planning to do that PR but didn't get to it yet :) Thanks @robdodson.


Hi. Just finished lunch. Sorry about that.


Just out of curiosity: what was this for?  It broke blme-package internals ...


Yes...!!    http://goo.gl/uQvL4u


Right. Will update.


return null pls


Alatis ; Then after a month we can go on codeacadmey cauz i need to know properly how to do stuuff on it ..... Can you makee Studiou brand new and me and you will test people and see how good they are ?
give all a people a task and then invite them if its perfect :full_moon_with_face:


Oh right differnet timezones


First, I find this style pretty awkward compared to just using "if (foo != 0)" instead of "if (!(foo == 0))".

Second, won't this always evaluate to true? (Note that getpwnam needs to fail and sulogin got -e on commandline to trigger this codepath.)

Third, shouldn't the check for "!" actually just check the first character (not against the whole string)? An account which previously had a password and then got locked will get ! as a prefix, so that it can be unlocked and password kept.


Should log the exception itself for debugging since we're swallowing it.


Is there still any reason for a no-op here?


Yes, this should be `!==`, good :eyes:

Refactor sounds good too


I will test this! *updating rpm* +1


Alatis can you make a new account u chhose the name then you make and intro thumbnail and studiou i will give out tasks and we will judge for the account pass email me ; ayaanshazeb1 @gmail.com Alatis ; Then after a month we can go on codeacadmey cauz i need to know properly how to do stuuff on it ..... Can you makee Studiou brand new and me and you will test people and see how good they are ? give all a people a task and then invite them if its perfect :full_moon_with_face:


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

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


Option/initialize approach sounds much tidier than calling the .toggleExpando from way down in the "call initializer" code.


`channel.addEventListener("pushnotificationreceived", onNotificationReceived)` is simpler - you don't need another function for it.


Alatis have you read it ?


:+1: Exciting.


Full agreement.


Maybe we do not need them, because script functions can be called from object context only? Calling `GetLightColor()` without an object returns `nil` already.


`1` რას ასახავს?


TeamCity YaccConstructor :: YC.Core General Build [Build 441](http://teamcity.codebetter.com/viewLog.html?buildId=192950&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Building ..\src\YC.SDK.sln failed with exitcode 1. Build time: 00:09:10


Hmm I normally do not use sendall as its not possible to determine what has already been sent if something fails for some reason. but in this case this makes sense.


Summarize why what is not used?


No worries. I saw someone filed a bug earlier and figured I'd just get it out of the way :)


Where is the prefix length?


Fixed.
But (don't remember if I said this already) gui/* files require comprehensive fixes for coding style, so I could have missed such in them.


The double normalization was stupid, by the way :D


Secrets have been changed.


Hey, I just came across this, `commtrack_reports.RequisitionReport` is still commented out, should we just delete it?


Secret key has been changed.


Not sure whether we should put the key here. Any better way?


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

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


I haven't checked specifically, but I believe it works the same way as add_torrent_params::resume_data. There is created a copy of the object in add_torrent() function.


@mkreisl

why? this is how it runs without.
```
-- Found SWIG: /usr/bin/swig2.0 (found version "2.0.7")
-- Configured features:
-- Pulse-Eight CEC Adapter:                yes
-- Pulse-Eight CEC Adapter detection:      yes
-- lockdev support:                        yes
-- xrandr support:                         no
-- Raspberry Pi support:                   yes
-- TDA995x support:                        no
-- Exynos support:                         no
-- i.MX6 support:                          no
-- Python support:                         version 2.7.3 (2.7)
-- lib info: compiled on Linux-4.1.0-rc5-custom01-00001-gdef76a7, features: P8_USB, P8_detect, 'RPi'
```


Third comment above applies here as well.


do we need a map of maps here, or can we not just concat class.fieldormethod to form the key?


Sorry, leaving to the dentists in 2 minutes. I will do what you said. :) Bye!


stadyling?


This path is used for multiple times. There should be a const or macro for it.


asıl değişiklik


Bye when will you come back ?


should this be two diffs?


I started to fix SIMD but stalled (been out of town for a couple of days). I will fix it. Actually I can't see why you disabled it? There's absolutely no reason to chicken out as soon as the acronym SIMD is seen, especially not in formats where all the hard stuff are hidden behind functions.


ISend me the pass when you done on my email ; ayaanshazeb1@gmail.com


@mhoeber


That would probably work


Sounds good.


@srpearce


I don't think this will pass checkstyle; let's stick to the Java naming conventions. `isDeletedRelation()` would be fine.


Does the finally block get executed if you die in the catch block?  I would guess that a die inside the try block causes the catch block to execute, but a die in the catch block just dies, and does not allow the catch block to return (and execute the finally block).  Perhaps try catch finally works differently than try/except/finally in python does.


This is also a hard coding. Basically, we can put all such paths together and define constant variables for them.


>should always be between 0.0f and 1.0f, no?
What about rounding errors?


ack


Weren't skolem types necessary for representing existentials in Scala?


I think it will cause issues under Windows...


just curious will you add support for mapping different hosts to different import categories?


I wanted to use it, but torrent_handle::prioritize_files() uses int, and TorrentContentModel returns int. So I still left. This requires deeper processing.


Great Thanks! I have reviewed the 5.4 branch.

I have a few comments / concerns about this branch. Where should I address?


Yes, sorry for the noise. RPI is indeed ok, only -DHAVE_IMX_API=1 is necesary.


Addressed.


Can you rename this to muiThemeManager please? This will help avoid collisions with other components that use context variables.

Everything else looks great. Thanks @SimonDegraeve


@mkreisl

always only by human mistake. :)) I added second + line manually and forgot to reindex.


I think the original name was better; "mark" doesn't really communicate anything specific. I can't think of a great name at the moment, but even `markRelationDeleted()` would be an improvement.


I'd argue that we could use


i dalje ne kontam...


@lampson0505 I forgot to delete this line, it is not in use.  Will fix it in the next comment.


Also, it seems that the parser needs to be generalized to allow a pattern match having no cases, which arises when the discriminee is a member of an uninhabited type, such as False.
I tried to remove the following line, but I got a parser error:
https://github.com/FStarLang/FStar/blob/master/lib/constr.fst#L36

The analog in Coq is:
    (* Print False_rect *)

    False_rect =
    fun (P : Type) (f : False) => match f return P with
                                                end
     : forall P : Type, False -> P


Sorry if this isn't the correct place to ask this, but where in the documentation can I find the on-iron-select attribute? I've been searching for it everywhere, and I see no mention of it.

I would have assumed that it was coming from the Polymer.IronSelectableBehavior but I can't find that attribute/method in the documents.


Tremolo Implementation


Please keep a single empty line at the end of the files.

![screen shot 2015-06-09 at 20 31 15](https://cloud.githubusercontent.com/assets/1410358/8066211/964594c2-0ee6-11e5-827c-b5721cec2de9.png)


after playing with it i'm having a hard time justifying the switch to Wrapped Static &str. Id say we address it later if a better solution emerges


Accepted.


Doesn't this check conflict with the idempotence of delete? How did adding this check not cause the tests to fail, if they test consecutive delete attempts?


o


@lampson0505 Right, I am thinking of adding a config file with all the default configurations/paths.  Would that work?


![screen shot 2015-06-09 at 11 31 41 am](https://cloud.githubusercontent.com/assets/1269969/8066250/731ae344-0e9b-11e5-80ec-14bf0ffd06ff.png)

Change to `no associated html or css.`.


@lampson0505 I agree, there should be some sort of data (any better name?) folder for these stuff.  Maybe we could put it in the benchmarks folder?


And whatever we choose in the end, it should be the same for `-t` and `-T`.


fine work, Eric!


@lampson0505 the address-range is directly passed to address range of vnet command in azure.  It follows the same convention.  Maybe we should change this to a more general case?


Perhaps this could be more concrete and say what 'it' is here, e.g. 'in other words, the precedence of `if` behaves like this':


This line is missing a `;` at the end.


Have you checked that this works? I would expect it's still broken (now there are literal double quotes embedded in the table name, which parameterizing the query won't fix).


You should also remove the even listener. This sample is a good reference: https://code.msdn.microsoft.com/windowsapps/Raw-notifications-sample-3bc28c5d/sourcecode?fileId=50987&pathId=342730458


I understand that this is a refactor moving functionality out of NetworkOPs.  From what I can see this part of the refactor looks great.

I'm puzzled about what's going on here, however.  The `recentPositions_` map appears to only be used in this method.  The method returns no values.  Is the point of this method to hold the three most recent `shared_ptr<SHAMap> position`s so they have to stay in memory?  If someone knows the motivation for this method it would be great to add that motivation as a comment either here or in the header file.  Thanks.


Please test this, I need to know if there is still things broken here.


synchronous rendering should work with eyeglass. If it doesn't I think that would be a bug in eyeglass.


The intended pattern for `Schema.ofFields()` (which can't be enforced because Java sucks) is `name_1, type_1, name_2, type_2, ...`. It's just there for readability.


Only need this one, should probably also rephrase to "list of backends to keep during tests"


First, I think "if (foo != 0)" is prettier and easier to read style then "if (!(foo == 0))".

Second, won't this always evaluate to true? (Note that this codepath is hard to trigger since both getpwnam needs to  fail and sulogin command line option -e used, so I haven't actually tested.)

Third, shouldn't the check for "!" only compare against first character? (If an account with a previously valid password gets locked, the string is prefixed with ! to allow unlocking and preserving the old password.)


rax retry tests


I think I understand what swap! does here, but what does .state refer to?


as i said in the header of the pull request, i'm converting quite a ton of stuff, and i'm putting it here to have ppl helping me finding the errors. Most of them will be typos like this one.


That is good advice. Now that I moved the enum to C4FoWDrawStrategy.cpp, I cannot access C4DP_Last in that method anymore.
For the current implementation I want only three passes. As long as we handle more passes with the `default` case it would be crazy to automatically adjust the number of passes in this drawing strategy if we add more passes.


LGTM


looks formuler have issue, you can use same kernel from skylake


The third comment above applies here as well.


google fonts?


ესეთი ფორმატი ხომ არ ჯობია `/gameHistory`-სთვის?

```json
{
  "moves": [
    {"i":1,"j":1},
    {"i":0,"j":2},
    {"i":1,"j":2},
    {"i":2,"j":1},
    {"i":1,"j":0}
  ],
  "first": "01f31t-13t45-fsgst-53ge-wety2356"
}
```

გვეუბნება ვინ დაიწყო თამაში პირველმა და რა მიმდევრობით გაკეთდა სვლები(პირველი ყოველთვის X არის)


+1 that was an oversight


Please avoid adding this kind of comments, git keeps track of who added what and when.


GHX_Slinky_IDD (mmatts) - x86_64-MacOS-10.9-clang: OK (1670 of 1670 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)


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Good catch - fixed.


This looks interesting. Planning a PR?


Have you tested if this works. I recall the 2.0 library having a very different API schema. Will just switching the version number work?


@jpvasquez  Does that mean we do not need a whitelist? Is checking for `.find-login-action.` enough?


The problem here is, that wer first create LogicalDevices with respect to the XML we receive from the RWE central during start-up. Afterwards, openHAB tells us which Items were configured within the .items file. With introducing the possibility of inverting an items value, we have to invert he value of the LogicalDevice once we receive this item definition.
However, I'm not happy with those to two-piece architecture (LogicalDevices vs. Items)...Makes things complicated.


@jaypatel512 can you please add javadoc as to what this method does?


Couldn't you test for existence without a loop by checking if `server.getDatasetStatus(relationKey)` returns `null`?


Can you give an example on how to use this option?


With wrap the line last time I meant to do:
```
PYTHON=`sudo which ${system_python}`
./configure --prefix=$python_tmpenv
```
But maybe this is intended^^


hpwh_residential (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1671 of 1678 tests passed)

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


I think it would be better to move this to [Constants](https://github.com/paypal/PayPal-Java-SDK/blob/master/rest-api-sdk/src/main/java/com/paypal/base/Constants.java) and make it configurable.


Hi,

makes sense. I added a similar version to master (with exception handling).
Next time please do not push commits for different features into the
same pull-request (make a branch/pull-request per feature)

g
André

On 09.06.2015 17:14, André Luiz dos Santos wrote:
>
> Why not use sendall instead of calling send for each character?
> It works with SSLSocket as well as with ordinary sockets.
>
> def write_sock(self, string):
>     self.sock.sendall(bytes(string, 'latin-1'))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andre-luiz-dos-santos/tikapy/commit/57f521188c164a09b9d3e1ea89a0a2a1bb039b73#commitcomment-11591209>.
>


Nice +1


I do not mean to be nit-picky, but it would be nice to keep tab-indentation consistent.


@danielmarbach Confirming that @SimonCropp confirmed that with me


I like this.


I think Boolean.getBoolean is not correct method to use.
http://docs.oracle.com/javase/7/docs/api/java/lang/Boolean.html#getBoolean%28java.lang.String%29
Boolean.valueOf is in my opinion suitable here. Usage of getBoolean forces user to set in spock.logEnabled name of the other property which will contain value of 'true'


No need to redo this, but consider breaking changes like this into two commits in the future - the commit message is "Moved console.log to errorService.addError.", but the bulk of the diff is jsdoc changes.


I am still trying to consider how to better deal with "gettext.h". Since there is a c99_*printf for MSVC, shouldn't we make a stdio_compat.h or a msvc_compat.h? I'd prefer to instead call up a stdio_compat.h everywhere *printf is needed to ensure that we are closer to c99 compliance in code.

Is there also something about c11 compilers that I'm missing?


I'm very impressed with this. I've been wanting to refactor this non-sense for quite a while!
This works amazingly in most cases, except on landscape/ios8/ipad/modal sheet presentation.

If you don't mind, I will use this snippet and add it to the code base as it's extremely simpler than the fragmentation logic we currently have (and can't handle anymore!). Thanks!


I kinda like this.


@wlbragg The problem is you're still storing the state (cabinairtempC, surfacetempC, surfacetempC) in a Nasal variables. So what you need to do is add at the top of the loop:

```javascript
var cabinairtempC = getprop("/environment/aircraft-effects/cabinairtempC");
var surfacetempC = getprop("/environment/aircraft-effects/surfacetempC");
var cabinairdewpointC = getprop("/environment/aircraft-effects/cabinairdewpointC");
```

At the bottom of the loop I think it's only necessary to do the `setprop()`'s if `/sim/freeze/replay-state` is false.


I don't like this.


I understand your motivation to try to make this look nice, but IMO this is not worth it. This implementation requires some specific font or configuration, because it doesn't work in my xterm and I anticipate all kinds of problem on all the different systems we have to support. Here, what I get is an idented tree that is not readable at all (and I'm being honest: I didn't change anything, I just tested it on my terminal I use daily. It works on gnome-terminal though).

Therefore, I strongly advise against using these fancy effects. Standard ascii characters can be used to make a nice and readable tree, just like the output of /usr/bin/tree. You could differentiate the mux/non-mux nodes by using '-' and '=', it would be a nice touch, but please stay with 8 bit ascii.


this used to return 'i', which is the position of the message_number and message_key in the vector. now it returns 'index', which is just the message number.

this change also makes merging some of the other branches in a bit gross, so i'm thinking it can be deferred until after those are dealt with. i'll make an issue for it.

but, these variables definitely need to be renamed, and there should be some test coverage that would pick that up (out-of-order message tests should do it). will make issues for those, too.


There is another sample here: https://code.msdn.microsoft.com/windowsapps/Push-and-periodic-de225603/sourcecode?fileId=50984&pathId=1537871129


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

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


I used a regular for loop first, but then I realized the numbers aren't always in ascending order when I failed the second test. I've used the min/max functions before for C++, so I just looked them up for Objective-C.


Did you ever know that you're my hero?

Seriously, thanks.  Was not looking forward to digging through that tangle of JS.


90664130_AlternativeModelPAT (macumber) - 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)


Yeah, I found images in the second article but that wasn't the article I
was ~presenting~ on (the science communication one)....I can only find
links to PDFs to that one. Better to have *a* graphic from a different
paper or no graphic at all?

Nell Byler

ebyler@astro.washington.edu
Astronomy Graduate Student
University of Washington

http://www.astro.washington.edu/users/ebyler/

On Tue, Jun 9, 2015 at 10:58 AM, Brett M. Morris <notifications@github.com>
wrote:

> @nell-byler <https://github.com/nell-byler> You can embed a plot from the figures
> page <http://www.pnas.org/content/109/41/16474.figures-only> like this:
>
> [image: alt text] <http://link>
>
> [image: Fig 1]
> <https://camo.githubusercontent.com/1c6f5b0dad91c74297fc46c212ce6e64801c79d7/687474703a2f2f7777772e706e61732e6f72672f636f6e74656e742f3130392f34312f31363437342f46312e6d656469756d2e676966>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/astroDJC/suggestedposts/commit/884fb31a0fea7a9a57cabeac61fef4b9cb7baa74#commitcomment-11594897>
> .
>


You might need to do `e.cancel = true` as the default behavior if you don't want a background task to get this notification.


bugzilla comments seem to indicate this patch only woks in some instances and not on all distros. Will keep an eye on the followup bug.


I could not simulate a failure so the best I could do in the tests is to access a worker directly and delete the relation it contained. That is why the tests passed, I didn't directly call the delete twice.

For this case, we wanted to throw an exception on any update operations to tombstoned relation metadata. Since this seems to conflict with idempotence, should we just log a warning for now?


Ok, I pushed the improvements.


Actually, about `e` - its a common enough pattern for even handers to have this name for the argument - so it might be OK.


For example, you're flying at 5000 ft and the cabinairtempC is 0 deg C. Then you land at an airport and lets say the cabinairtempC has become 20 deg C. Now, you enter replay mode and go back to the moment you were flying at 5000 ft. The FR will then make sure that `/environment/aircraft-effects/cabinairtempC` is 0 deg C again, and your Nasal code will then correctly compute the `frost-level` and `fog-level` based on the 3 temperatures (cabinairtempC, surfacetempC, surfacetempC).


No worries. Sorry it took me so long to merge it in.


Like [one of the above comments](https://github.com/paypal/PayPal-Java-SDK/commit/6154d41321a0f14af0e36378cc16b3dede5f9e4f#commitcomment-11595839), hard-coded strings (```X.509```) are probably better placed in Constants.


[DocBlockr](https://packagecontrol.io/packages/DocBlockr) is my choice №1 for this :+1:


If you don't read the temperatures from these three properties, then if you enter replay mode and go back to that moment in the air, the cabinairtempC Nasal variable would still be 20 deg C. Which is wrong, because it should be 0 deg C.


@mbernat Yeah, that is exactly what it does. There is no `break` necessary in `case`, by default only the first one that evaluates to `true` is executed.

Yeah, you can also write
```go
if name := label.Name; name != config.ReviewedLabel && name != config.TestingPassedLabel && name != config.TestingFailedLabel && name != config.ImplementedLabel {
    ls = append(labels, ...)
}
```

I personally prefer this one ;-)


Hi, not all unfortunately. As far as I know the calls are same. What is different is the XML response.
But I wonder, if there would be way to configure qualysapi what version of AM API to use. Either in config file or a new optional parameter in the request call?

Vladimir Jirasek
Time zone: London/UK
Sent from my mobile device

> On 9 Jun 2015, at 19:45, Parag Baxi <notifications@github.com> wrote:
>
> Have you tested if this works. I recall the 2.0 library having a very different API schema. Will just switching the version number work?
>
> —
> Reply to this email directly or view it on GitHub.
>


asd


What about the case where a dataset is shared with this project?
Then the projectName will not point us to the project where the dataset is actually stored?
 We don't necessarily need the full path to the DataSet. The NameNode can "parse" the patha correctly, if we use the format for imported datasets "projectName$datasetName". Btw, we shouldn't use "_" (underscore) as a separator for projectName$datasetName.


I don't think this is Rhys's point. In sync-android we incremented the major version to imply a breaking change. This was because an old version of the code won't be able to open attachments created with this new codebase because it won't know to look in the attachments_key_filenames table to find out what the on disk filename is.


The first comment (and many others on here) are the very definition of passive aggressive, and my previous comment goes back quite a ways dan.  We've been talking today about ethics and licensing but when I mentioned giving credit to people who have provided original ideas for becti that he's used, he gets all pissed.  If you can't read between the lines then I can't explain it to you, it just comes from experience.


Excellent question @mercul3s : since we have to do Java interop - gen-class is used here to extend an existing Java class perforce - "state" is the state of the Java instance created from it. "Pure" clojure is by default immutable, so in order to do the interop cleanly, 1.) an atom ref type is set up by the class initializer (init); 2.) static Java class variables are set up and initialized by the GenericUDF API initializer (initialize) using the previously intialized atom ref, using the set! mutator.


Zufällig ein paar Files vergessen zu commiten? ;)


Then why is it not shown here? What I mean is that you need to do execute `./append_output.sh specification.py`.


@mbernat Alright, I can try...


What's this for?


test commit


Version. I can remove it.


If we allow C++11, then it should be an unordered_set.

-Steven

On Tue, 2015-06-09 at 10:31 -0700, Alex T. wrote:
> If we allow C++11 now (do we? @marinier @mazina @bazald ) why not use
> std::unordered_set for this. You're just using it as a container to
> check against and not requiring any order for traversal (you're not
> even doing a traversal) so it's constant time insert/lookup vs log(n)
> now. For large, successive, queries, might that matter?
> —
> Reply to this email directly or view it on GitHub.


I mean, what's its purpose?


Ok got it, thanks for the explanation :+1:


hey


A good overview on need to do this, from a high level perspective, can be gotten from an understanding of how to write a UDF in general: http://mark.thegrovers.ca/blog/how-to-write-a-hive-udf and from a discussion of the issues surrounding Clojure/Java interop and mutability: http://thinkrelevance.com/blog/2013/01/14/taming-side-effects-with-mutability


En este bloque miramos si hay un mapa cargado (un bg) en el punto de la camara. Dado que la camara no se mueve, los puntos son fijos.


Oh awesome I am going to give those a read through.


@ansis Any idea why this commit would cause no apparent problems when viewed but generate blank "actual" images in all the line test cases?


Acá calculamos: si dos puntas en un mismo eje son False, entonces el eje completo (el lado) es False


you can use `memcmp` here, as in https://github.com/NebulousLabs/Sia-GPU-Miner/blob/master/gpu-miner.c#L120


Wouldn't it make more sense to define these outside `__init__()`?


This is only a relative path instead of sys path. I should provide a legit mock sys path here.


No, because each of the possible class data keys belong to the class itself, so it makes sense to have them stored there.


On a more general level, on "state" and "value" and "mutability," I highly recommend this reading as well: http://clojure.org/state


Would it make more sense as being defined inside `_check_class_data` as a local function variable?


Leave the bytes as well.  I can envision a world where we check content.


hpwh_residential (nmerket) - i386-Windows-7-VisualStudio-12: OK (1671 of 1678 tests passed)

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


adys son las claves para cargar los mapas que faltan. Si una variable (top, por ejemplo) es false, signifca que ahí no hay mapa, por lo tanto agregamos la clave para cargarlo en el siguiente bloque


Does this grep really match what you are looking for? `grep -q '$exp_ips_regex'` matches the string `exp_ips_regex` at the beginning of a line, right?


Acá nos fijamos si hay mapas. El problema, me parece, empieza acá, porque las variables cambian de signficado.


You are missing `get_or_create_document`^^


no memcmp in opencl


And interestingly you don't even test it :P


Could you put this line before foreach, there is no point to redefine it on every iteration.


If a program creates a file or directory with mkdtemp, it's supposed to clean up after itself, which get tricky here as the only reason the files are written to disk is so that they can be examined/compared later. For production code, I'd want to be able to configure where the images go. Either method seems like overkill for what this example is meant for. Unless you have strong opinions, I'd rather leave it as is for ease of use.


r+


method called 'create_issues' but you create only one as I see
but I suppose it's the place you've stopped before pull request, if yes then I have no comments about code - just continue
(I'll make merge after the homework will be finished)


include the failing key in the message and an example of the correct format.


I really need to do some research on C# and Kotlin type of generics. They seem even harder to use when it comes to classes that can't be declared co- or contravariant, which Option kind of is.

I tried making the class just use T rather than out T, and it worked with having the extensions moved back as regular methods (with a little tweaking), but I'm sure it's not what you want.


Nope, that's not how shell interpolation works.


I don't like mutation of args.


@jaypatel512 can you explain the difference between this method and [the one right above](https://github.com/paypal/PayPal-Java-SDK/blob/master/rest-api-sdk/src/main/java/com/paypal/base/HttpConnection.java#L57)? I would rather like to have the code refactored and reused than copied over.


I see examples for other platforms. It could be a good idea to also include an example that shows how to set this up for windows.


I'd also like to point out that the same quoting was present in the original that you wrote ;)


There is no shell interpolation inside singles quotes:
```
echo '$HOME'
$HOME
```


Why you don't use the constructor?
```python
doc = DbusDocument(path)
doc.id = self._next_doc_id()
# ...
```


@arthurvr YOU FAGGOT!!!

![Alt text](https://search.disconnect.me/image?l=aHR0cHM6Ly9lbmNyeXB0ZWQtdGJuMC5nc3RhdGljLmNvbS9pbWFnZXM/cT10Ym46QU5kOUdjU1M4OVZvbGhrZnprQzZvQTdsMWNuSXUwMTRXdUhBdURUTTJia0tiQlRoWDVZUHNiVEo5UQ== "Optional title")


It sounds like you should be using a separate, perhaps personal repository.


try `echo "'$HOME'"`


If you mean the code in `test/config.sh`, it was rather different:
```
assert_raises "echo \"$got_ip\" | grep -q \"$exp_ips_regex\""
```


Jaja ok, sorry, a lot of comments are auto added by the editor.


Did you ask this in an email to the mentors ?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/9d7c6b93412118f3f4feac0dfa9a92a855be6788


merging MonsieurNicolas/stellar-core/offerAtom = 9d7c6b93 into auto


MonsieurNicolas/stellar-core/offerAtom = 9d7c6b93 merged ok, testing candidate = bdd300de


I don't know if this approach would be the best for this scenario. I know that you adopted duplicate the attribute 'is_final' in both tables (state and sponsor) to avoid joins when you list sponsor's state in my tasks view. If each sponsor already load all relations (transitivity), well, this scenario will be redundant and will have lower performance. Otherwise, this scenario gain some advantage because avoid a lot of joins when list sponsor's state. However, this approach will always be a tradeoff between write and read, considering that you have 20w/80r law i believe that its the best approach if the condition about transitivity does not verify.


I guess it depends on the point you're trying to make. If you think the graphic helps to educate others about the topic, then it's relevant. Would you like to include any, or should we leave it as is?


This test is wrong.  You need to see if source has an `=`


I made you a collaborator on this project. Feel free to make changes


Where will it be disposed? In the `DbusServer`?


Ah yes, my apologies. I [simplified that](https://github.com/weaveworks/weave/commit/ccd7153b9d9df01609b8557c2131f2e8786527ed) some time ago.


Is it ok to turn on fatal warnings here?


nice work :sheep:


anyway, the point is that single quotes inside double quotes are not special.


Looks good to merge


If it will be disposed from `DbusServer` if registered inside, I would state this more explicitly, so i.e.
> If this app is registered inside a DbusServer and is running out of documents, it will be disposed from the server.


:neutral_face: ... great... time to move to a different smoke testing framework...


We should probably name this - passportStrategies


@gabetax Hi, quick question! How can I generate this ```REVISION``` file? I ask because I tried to create a samson docker image to run the webapp inside a container but the process failed when I ran ```docker build .``` due to the missing file. Thanks!


not clear method name, maybe 'create_project'?


Кодировки поправь и лицензию пропиши, пожалуйста....


Can this line use `monitor_state_dir` from the method parameters instead of gflags?


ack :D


again not clear
method name should describe what the method does


consider renaming the 'success' and 'fail' parameters to 'onSuccess' and 'onFail'. It's a bit confusing when following the code. the reader can easily think that these refer to the original callback functions supplied by the app developer, while they refer to wrapper functions supplied by cordova-js's exec.js file : https://github.com/apache/cordova-js/blob/b0463746dd842818c1f08560e998ec847460596c/src/windows/exec.js#L61


I had used "[default]" as the default section to begin with, but I got `ValueError: Invalid section name: default` due to:

> If a section by the given name already exists, DuplicateSectionError is raised. If the name DEFAULT (or any of it’s case-insensitive variants) is passed, ValueError is raised.

https://docs.python.org/2/library/configparser.html#ConfigParser.RawConfigParser.add_section


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1163 tests passed)

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


Eof


Hmm, there is some inconsistency in doc files about this. Most of docs using `cordova-lib` as command name (see [compile.txt](https://github.com/MSOpenTech/cordova-cli/blob/master/doc/compile.txt#L3) or [prepare.txt](https://github.com/MSOpenTech/cordova-cli/blob/master/doc/prepare.txt#L3)). However, docs for `build`/`run` using just `cordova`. Probably it makes sense to use `cordova` here as well.


change this implementation (and all further) to right Select box handling (I've sent link in skype group chat)


Nice documentation.


@mbernat Fixed.


wrong name
I suppose it should be like 'change_password_from_to(old_password, new_password)


Awesome thanks!


@sils1297 @AbdealiJK You think it's necessary to rename the test function? Since the variable is called now `on_disconnected`^^


Sure, a warning would be fine.


This might be a good place to use `pending` from Hspec


we should use indentation as two spaces here, right?


which roles?
and maybe you can pass a params about new role value


Is this part of the if really needed if you set it to True by default two lines before?


This broke UEF Engineering Stations. See #656


Ah and now I have a good idea:
You should put this inside the `test_callback()` function itself. It won't be used elsewhere and is just an effective bypass for a lambda because of the assert. On class level this callback won't bother any more.


we need assertions somewhere
otherwise we don't have a tests actually but script only


Though now it tells me "Invalid author email".


TankDeadband (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1668 of 1671 tests passed)

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


Das Original ist über eine Stunde lang (Beethovens neunte von YT)


Yes. I'll send a PR to [cordova-docs](https://github.com/apache/cordova-docs) tomorrow.


I am guessing that underscore.isEqual handles nested structures, was that the fix?  Now that I think about it, you might even have an issue with that as a set is not going to be called on  an update..

i.e.

model.field.someProp = 2

would only invoke the get on model.field, so you might almost have to implement a hack on he Get method to force a dirty flag on it?  Not sure..

-----------

I was curious on the second change.. it looks like there is a change there to detect the same field in the value array.. however, a property should only be in there once. I suspect there might be an issue with the set logic which is causing the property to be put on the dirty fields array twice.


It will be better to use cherry-pick mate :)


aproved


Make that URL configurable in `settings/base.py`. The default can be `http://localhost:9200/`


Nice will check it :) thanks


https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-synced-flush.html would be a more direct link


That's a pretty good shout, although it does suffer in that you have to have a github account to create issues.


for(n in 6:8){
   Differences[3,n] <- (Averages[3,n] - Averages[3, (n-3)])/2


Yeah, it's for the normal unwatched mode. It's set to true by default, but may change to false when it evaluates the playcount == 0 condition.

Have you found an error or just inquisitive?


This is silly, plz fix.


ack


The URL would be something like `location.pathname + 'related-content'`.


You can't put django templating into a static file.


There is a downside to setting e.cancel = true as it will discard the entire default processing of the notification, which includes the UI changes for the toast/tile notifications as well. Ideally we should be giving the developer this option, but this may increase the effort required too. What's your take?


rax retry tests


Actually just got what you meant! It's not, but does no harm :)


Arguably these should be on master, as they're useful beyond SDPs.


this ">>>>>>>" can cause compile error,do you using git programs for chery pick?
you can use gitg to take a patch by commits and use "patch -p1 < file.patch" to apply them


@janacuq DWMR. This is done well. :+1:


Since you have a handle, you have the option of using the `print` method:
```perl
$handle->print( $self->_color_heading("Workflow") . "\n" );
```

If you'd like to also get rid of the concatenation, you can use the `say` method instead:
```perl
$handle->say( $self->_color_heading("Workflow") );
```


Pushed a bunch of commits -- I think I've addressed all of your (ribasushi's) concerns.

Also, I'm doing a lightning talk on this in the "Shallow End" talks at 2:30 today.


Это вроде недавно в рассылке оберона спрашивали про лицензию.


Ohh, thanks, you are fast :) Nonetheless, there are some more. PR ready here.


bist du dir sicher?


* TODO: rewrite this as it is doing way too much and is very messy now


english.xml
<Resource tag="CONTENT_TYPE">Content Type</Resource>BUTTON_DELETE


BEURK_INSTALL_DIR is defined in our beurk.conf, you can't set it in the Makefile.
On another note, "CAREFUL, this command infects[...]" would be more appropriate.


No `offset`?


ja bin ich


@mpc-hc/developers: https://github.com/mpc-hc/mpc-hc/compare/develop...Underground78:34e25d4, mostly fixing regressions:
 - one in the stable caused by the shuffle mode changes
 - two in the latest nightly related to sample tracking in the renderers
and also one commit which could possibly fix the renderer crashes we see on DrDump (but I'm not so optimistic).


Fixes #5


AFAIK, all platforms now has `clean` script, so this is very unlikely to happen.

Regarding `require` vs `spawn`, yes, this will be reworked to use `PlatformApi.clean()` instead of spawning, once https://github.com/apache/cordova-lib/pull/235 will be merged.


PA becomes official for geehrc? :P


Is it definitely safe to call `clearApplying` (and then release the lock) before calling `setResult`? Do callers know they can't look at the result in the Transaction object?


I agree, it comes from the API, just submitted a PR to fix it: https://github.com/elastic/elasticsearch/pull/11561


should you also validate whether successCallback is null ?


How can we replicate the old functionality?


And suggested reading - only after reading Rich's "state" article, since it follows up on it - is http://blog.jayfields.com/2011/04/clojure-state-management.html


I'm assuming this should be:

    self.ignore += [x.strip() for x in f.readlines()]

So that we get new ignore patterns in addition to the base ignore.  At the end of the read gitignore function, we could turn the list into a set, then back to a list again, to eliminate repeated items... Let me see if I can get something up.


Not sure if it matters, but did you mean to remove the trailing space in `code)' +` ?


ng-disabled bug is fixed, but now the width parameter is ignored


Why everybody escapes EOL to continue code...
@sils1297 We didn't agree on #558, did we?


This fixes #37.


What line are you referring to? do you mean setApplying (instead of setResult)?


Check my later commit in this pull request :)

Commit ab9de61 does not contain changes, but dd5d67a.
https://github.com/washwash/python-patterns/commit/dd5d67a6ef45879f193a7ec4d74d3155883f0e12


why wouldn't you just invoke next() here and let the express handle the rest? Either next(err) and then user's error handler can decide what to do with it, or just next(), which will return 404 in the worst case scenario?


Consider making this an ordered list, it would make it clear that they need to be done in sequence, and our brains pick up lists as TODOs instead of informational.


My idea is to overwrite, but I`m open to discussion.


There's only one call to `clearApplying`.

I think the best fix is to call `clearApplying` and set the result in the Transaction object in the `apply` function.


aproved


Regardless, for clearApplying() it's better to do them all in 1 batch within 1 lock than lock/unlock for every item.


@janacuq DWMR. I looked at three commits that make up this checkpoint - which takes a bit of time. You clearly understand the work. You worked hard - thank you for that.


OK, I can move it to the apply function.


for production shoud be build with react-tools (https://facebook.github.io/react/docs/getting-started.html)


The "Before" of the previous paragraph and the "Now" of this paragraph don't make sense; it makes it sound like I missed a step once I get to the creation of the disks. Consider merging the paragraphs.


I amended the commit in the pull request the3dfxdude/7kaa#44


Thanks Andrew


I'd justify your size suggestion here.


@msgrasser Yes it is


@bors retry

:+1: e61e6b655669fb5cee0422348671c37ea12f78ab


There might be a cleaner way, but it's critical that all calls to `clearApplying` be made before the call to `notify`. (Or, to be more precise, before releasing the lock after calling `notify`.) And I think we violate this now.


Hi Nate,

Thanks for your comments.  Here are all of my Galaxy processes.  The Galaxy job running under PID 23041 and children is the one that hangs forever.  How do I tell if this process is in the "T" state?

dot1x-cb-59:~ gvk$ ps -ef | grep galaxy
  502 22960    81   0  3:27PM ??         0:00.37 postgres: postgres galaxy ::1(54562) idle
  502 22961    81   0  3:27PM ??         0:00.28 postgres: postgres galaxy ::1(54563) idle
  502 22965    81   0  3:27PM ??         0:00.09 postgres: postgres install_galaxy ::1(54566) idle
  502 22966    81   0  3:27PM ??         0:00.24 postgres: postgres galaxy ::1(54657) idle
  502 23013    81   0  3:29PM ??         0:00.15 postgres: postgres galaxy ::1(54992) idle
  502 23014    81   0  3:29PM ??         0:00.12 postgres: postgres galaxy ::1(54995) idle
  501 22949 22944   0  3:27PM ttys000    0:14.23 /Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python ./scripts/paster.py serve config/galaxy.ini
  501 23041 22949   0  3:30PM ttys000    0:00.01 /bin/sh /Users/gvk/work/git_workspace/galaxy/database/job_working_directory/000/3/galaxy_3.sh
  501 23043 23041   0  3:30PM ttys000    0:00.05 /Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python /Users/gvk/work/ts_install/shed_tools/localhost/repos/iuc/imagej2_bunwarpj_align/8793d5edcd72/imagej2_bunwarpj_align/imagej2_bunwarpj_align.py --target_image /Users/gvk/work/git_workspace/galaxy/database/files/000/dataset_2.dat --target_image_format jpg --source_image /Users/gvk/work/git_workspace/galaxy/database/files/000/dataset_1.dat --source_image_format gif --min_scale_def 0 --max_scale_def 2 --max_subsamp_fact 1 --divergence_weight 0.0 --curl_weight 0.0 --image_weight 1.0 --consistency_weight 10.0 --source_out /Users/gvk/work/git_workspace/galaxy/database/files/000/dataset_3.dat --source_out_datatype png --target_out /Users/gvk/work/git_workspace/galaxy/database/files/000/dataset_4.dat --target_out_datatype png --jython_script /Users/gvk/work/ts_install/shed_tools/localhost/repos/iuc/imagej2_bunwarpj_align/8793d5edcd72/imagej2_bunwarpj_align/jython_script.py --max_heap_size_type default --max_heap_size 0


Closes #997 in the curriculum repo :)


saw approval from lukebakken
at https://github.com/basho/riak_kv/commit/e61e6b655669fb5cee0422348671c37ea12f78ab


merging basho/riak_kv/jira/lrb/riak-1802 = e61e6b65 into borshop-integration-1116-jira/lrb/riak-1802


basho/riak_kv/jira/lrb/riak-1802 = e61e6b65 merged ok, testing candidate = 36ff0844


dope


Since #558 is not approved (*sigh*), ack :D
Btw defining `create_document` etc. inside the `DbusApp` is way better and much more object-oriented style imo^^


The do {} while loop protects us from spurious wake-ups.


The choice of yours to use `c:\dwasfiles\` vs `C:\DwasFiles\` means your configuration has to resolve the entire path from root to the end.

The use of `c:\dwasfiles\` is actually an alias to the real/canonical path `C:\DwasFiles\`

Even with `File.getCanonicalFile()` this is true.

Even with `File.getCanonicalFile()` there would be an access exception and resulting invalid jetty.home or jetty.base, however it doesn't resolve symlinks, which means you can get duplicate xml files in your valid configuration.

Note that this file system access issue isn't just in jetty's start.jar, this will be present on the running Jetty instance as well.    If we didn't do this, resources would fail to resolve in the running Jetty instance, as all resources would be an alias, requiring you to disable various alias checking routines and opening up your server to many vulnerabilities only present when running under Windows (these alias checks and vulnerabilities do not exist on any flavor of Unix, including OSX).

Be extra careful when you specify paths in the configuration of Jetty on Windows, as the case of the path is important for security in web applications.

Start by using the proper and non-alias paths for your command line -Djetty.home and -Djetty.base directories.  (Case is important for the web and for the security layers of the servlet spec, even on windows)

The choice to use java's new nio FileSystem utilities has simplified and fixed many of the alias handling bugs and vulnerabilities present in earlier versions of Jetty.   However, your choice of using symlinks in Windows (something that  has been present since the introduction of NTFS, but barely used, and only supported on Vista or newer with Java 8) is likely causing you most of the issues you are having.  There is only a Windows FileSystem implementation, not an NTFS specific FileSystem implementation.    If your link starts on non NTFS (such as exFAT) and then switches to a NTFS filesystem then you are violating one of the assumptions of Java, that the referenced file will always be on the same FileSystem type.


Do you mean /rootfs/etc/fstab?


You could, after you're done with the per-transaction stuff, acquire the lock, call `clearApplying` on all transactions and then calls `notify`.

The problem isn't spurious wakeups, it's missed wakeups.


done


(We wake up from the notify, see that the applying flag is still set, go back to sleep, the flag gets cleared, and we're still waiting for something that already happened.)


I see now.


These changes look good. Thanks, Tyler. :-)


I suggest being more descriptive with your commit messages. What you have there doesn't tell other developers much about whats going on with the commit.


keep it like it is now^^


t ia2 -ae set 0 3200 240 2.9 0


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/66100407


Allows for internet connectivity


The issue here is that when we allow users to name their keys we don't keep track of the file names in the `map[string][]byte` so we either check filenames or bytes, or mutliple `Matches()`-type functions - is this one specifically fulfilling some interface?


CBECC (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2043 of 2158 tests passed)

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


Isn't breaking backward compatibility ?


Looks great! Do it.


you should be very explicit when building a response payload for an endpoint. What if the schema changes in the future and you return all of it without knowing?


If you want to see `WindowsLinkSupport.getRealPath()` source, look at the openjdk source.

 http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/687fd7c7986d/src/windows/classes/sun/nio/fs/WindowsLinkSupport.java#l156


@OwnDroid


*Makman2 wrote:*
> I hope I understand it right: We only traverse down until home directory?
*AbdeliJK wrote:*
> We only traverse down till home directory or we reach root/drive folder
Only home directory condition is not sufficient for files in different partitions in windows

But you can also have different partitions in linux, also they mapped/mounted in the same "file-space".
Maybe we should define an optional parameter `bottom` (or similar), that is by default the home directory, but is changable if we want to search until user-defined limit^^


That could be a good way to go. This is meant more for reference than anything else and I wanted to put it where somebody could find it, but the odds of someone else needing it are pretty low.


The payload response will be clear. At this point I'm just trying to get a simple query to work first.


if you don't touch this, then the intent of the method is preserved.


Probably displayed wrong due to markdown... ;)


I added more javadocs in the next commit. You can see that here https://github.com/paypal/PayPal-Java-SDK/commit/c24086de20b6d90df1eb3c30e467a157c4abc3c3


I have 2 Masternodes with your patch running on main-net now. Let's see how they look in 8 weeks.

If I shouldn't get payments in the next 5 days I'll send some [friends](https://en.wikipedia.org/wiki/Klitschko) over to your location ;-)


```

if (key === 'width') {
               return options[snakeCase(key)] = scope.$eval(value) + 'px';
              } else {
                return options[snakeCase(key)] = scope.$eval(value);
```

If width parameter is set to '175', it works as expected. if it is set to '175px', it is ignored.


thank you @JaviKia  this is very good. Are you ready for me to merge all your recent changes or are you still working on this?


Yep, this small change does it nicely:

        diff --git a/inspektor/inspector.py b/inspektor/inspector.py
        index 2518622..768c7dc 100644
        --- a/inspektor/inspector.py
        +++ b/inspektor/inspector.py
        @@ -24,7 +24,7 @@ class PathInspector(object):

             def __init__(self, path):
                 self.path = path
        -        self.ignore = ('*~', '*#', '*.swp', '*.py?', '*.o')
        +        self.ignore = {'*~', '*#', '*.swp', '*.py?', '*.o'}
                 self._read_gitignore()

             def _read_gitignore(self):
        @@ -36,7 +36,7 @@ class PathInspector(object):
                 for ign in (dot_ignore, git_dir_ignore, config_ignore):
                     if os.path.isfile(ign):
                         with open(ign) as f:
        -                    self.ignore = [x.strip() for x in f.readlines()]
        +                    self.ignore.update({x.strip() for x in f.readlines()})
                         break

             def get_first_line(self):


Isso não pode ir para `ICache`?


I would do something more like:

```python
rec = fetch_farmer(id)

payload = {"something": rec["something"],
                  "something_else": rec["something_else"],
                 }

resp = flask.make_resp(flask.jsonify(payload), 200)
return resp
```


Use -o to specify a filepath where you want the files downloaded!


Now I don't remember whether this is py 2.6 compatible :|


A few other points:

 * Please ident the variables (when -c) is enabled inside the branch, not in front of it. It's confusing to see some items (the ones with just one variable) having it in the same line, while multiple variables get to different lines. Besides, the identation of multiple variables is currently off by one space.
 * Can you make the output of -c to be a result of the processed tree? That means the leaves would include the variables that were inherited from their parents and no variables would be in a non-leave branch.
 * What's the motivation for keeping both --tree and --tree-view? I think we should keep things simple: one output should be enough for everybody. I'm sure everybody will get used to the new format.
 * Some options are incompatible with --tree. Avocado should return an error if they're used together (for example, --debug).


fast-forwarding master to auto = bdd300de


Because in some circumstances it may fallback to non-secure connection?
BTW, this patch is reverted, I commited the new one proposed by Ingela: 45aaefe739c8ea6c33d140d056b94fcf53c3df30
How do you feel about this one?


The flag is already being cleared in the apply() function.

What do you think needs to happen?


would be interesting to test  :
- modify src/main/resources/static-content.txt
- fetch("http://localhost:8080/static-content.txt");
- assertThat(result).contains("This is updated static.");


What about the implementations in register and ping?


ie, ideally it would be something like

```js
// assume we have a lib with common errors
if (!Fetcher.hasFetcher(resource)) {
    var err = new Errors.BadRequest('Invalid Fetchr URL');
    err.source = 'fetchr';
    return next(err);
}

// router.js
server.use(...) // wiring fetcher
server.use(function (err, req, res, next) {
    if (err.statusCode === 400 && err.source === 'fetchr') {
         return next(); // invoke 404 later on if you wish so, or maybe enrich the error message, maybe render 404 page, etc
    }

    next(err); // invoke other error handlers
});

// common response middleware, common error middleware code
```


Pelo que vi, um `private readonly Stack<IRawCache> cacheBackends` seria mais adequado.


@janacuq DWMR. I can see that it's ben difficult to keep the checkpoint numbers/submissions in sync. Working with the new files has been ideal from a mechanics standpoint - but the content of the checkpoints and the learning is superior - worth it. Press on!


Hi, I can se the request method has api_version parameter.

def request(self, api_call, data=None, api_version=None, http_method=None, concurrent_scans_retries=0,
                concurrent_scans_retry_delay=0):

I will have a look how t add AMv2 there. That would be cleaner.

Vladimir Jirasek
+447538790302
Time zone: London/UK

> On 9 Jun 2015, at 19:56, Vladimir Jirasek <vladimir@jirasek.eu> wrote:
>
> Hi, not all unfortunately. As far as I know the calls are same. What is different is the XML response.
> But I wonder, if there would be way to configure qualysapi what version of AM API to use. Either in config file or a new optional parameter in the request call?
>
> Vladimir Jirasek
> Time zone: London/UK
> Sent from my mobile device
>
> On 9 Jun 2015, at 19:45, Parag Baxi <notifications@github.com <mailto:notifications@github.com>> wrote:
>
>> Have you tested if this works. I recall the 2.0 library having a very different API schema. Will just switching the version number work?
>>
>> —
>> Reply to this email directly or view it on GitHub <https://github.com/vjirasek/qualysapi/commit/cbadb8f56bfbd83fc077f47da8329ef99ae6be16#commitcomment-11595796>.
>>


I actually wondered what side effects the new patch might lead to. It just ignores "unknown" algorithms. According to the code from `ssl_handshake.erl`, if there are no algorithms left, the default algorithms are used.


They look good, but you may not want to dump out stacktraces to the user, it might contain sensitive information. That info should only be available to devs in server logs.


I'd consider appending, given that, worst case scenario you end with the original set, and best case scenario, extend the set of things to ignore with the new info from gitignore.


CBECC (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


BTW:

 * https://trello.com/c/c25t08Nq/404-rfc-inconsistent-usage-of-run-it-s-implicit-in-some-cases-explicit-in-others
 * https://trello.com/c/VXI5c5aN/405-bug-missing-mux-entry-and-env-in-the-multiplex-command


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2109


One more reason to make two partials..


No worries.


I want to see all stdout/stderr on the progress report for successfully finished jobs. Could you make an option for this in bds.config?


Nah, just made a commit change for cherry-picking after every build so sync doesn't break. :) (And unofficial nightlife's can start again)


The two examples above are identical.


@kholia you need to stop using assert() at random. We use pexit() and error() in JtR.


Wouldn't using https://github.com/dart-lang/cli_util the proper way of getting the SDK directory?


What happens if the execution is `errored` or `canceled`?


When `lhs`/`rhs` are (LLVM) vectors, `cond` will also end up a vector, which causes the `with_cond` to fail (since the `br` instruction only accepts `i1` types, not vectors of `i1`).


Pretty straight forward solution.


Change and simplify to "By inviting users to an Organization multiple authors can publish and manage content owned by the Organization."


s/`Subscribe`/`SubscribeAsync`


you should be setting http methods you allow on all your endpoints like

```python
@app.route("/herp/derp/", methods=["GET"])
```


Саня продакшин ветку не трогай :) трогай только девелоперскую :+1:


ik volg je, ik zoek bijkomend een mooie manier om die te visualiseren, weet jij ene handige methode hiervoor?


I think I'm leaning towards logging the warning for both SQLite and MySQL. If I have an exception for these cases, the delete call will fail because it calls ```dropTableIfExistsCascade``` from the ```DbDelete``` operator. I could just switch this call to ```dropTableIfExists``` instead, but if say, a postgres relation has dependent objects, these objects will not be deleted if I only call ```dropTableIfExists```


Because it irritated me when I ran ". compile_static.sh" and my entire terminal closed.


If we do this, aren't we deleting the actual file from the file system when doing the preview so it cannot be downloaded after?


I think that this file is the one that we may want to delete.


@janacuq The bloc.io team is working to switch your roadmap to the new curriculum today. That's what I just was told.


I'd make reference to the idea that "Often, your team will share access both to a Github repository with the source of the content, and to the Organization on Outlearn.  Note that if you configure a GitHub content import source yourself with auto-import settings, any GitHub user who pushes to the `master` branch will trigger a re-import to Outlearn, whether or not that user is a member of the Outlearn Organization"


This needs to be coming from the controller manager server flags


Oh, PyCharm IDE indicates unused variables and _ indicated that this var used only for iteration


В этом репозитории тоже  не трогай production, трогай developer скую


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3807/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3806/


I had tested with PrBoom+, which doesn't seems to care at all about the .deh file.
Chocolate-doom finds it without the `-deh` so I'm tempted to let the command-line unchanged
(`doom -iwad /usr/share/games/chex.wed` ) ... but is it ok for Doomsday ?


Corrected in #1.


Non-colliding ports, at least. I don't know how to do that from the command line rather than a config file, which is why I didn't. Feel free to add that.


Cf. https://github.com/jenkinsci/windows-slaves-plugin/pull/1.


WHAT IS THIS I SEE?


Yes it is easier to handle for now


tested and working


Updated: https://github.com/jeffmendoza/kubernetes/commit/3638c0cf20ac99ccac18153ac7a9d979deb49bfe


Simplify to "You can import **from** any repository you have access to on GitHub.  You can import **to** your own Outlearn account or any of your Outlearn organizations."


And move that paragraph up above the example.  I think it's clear.


This is the line I'd most like to remove since it's the only bashism, if you have bright ideas.


Sure, simple is better :) I didn't have time to dig deeper and test behavior on all devices. I was having issue with input bar frame when `slk` is nested in other view controller.

I think one of the problems with nesting come from `slk_appropriateBottomMarginToWindow` setting bounds origin to zero.


To be honest, I only kept that case because I kept seeing it in other methods throughout MasterCatalog in cases where SQLite transaction aborts hit an exception. I can get rid of it though


Am Dienstag, den 09.06.2015, 12:55 -0700 schrieb Alexandre Detiste:
> I had tested with PrBoom+, which doesn't seems to care at all about the .deh file.

prboom-plus has the chex.deh patch embedded into its prboom-plus.wad
file. It is in the CHEXDEH lump.

> Chocolate-doom finds it without the -deh so I'm tempted to let the command-line unchanged

Yes, Choco finds it automatically if it is in the same directory as the
chex.wad IWAD.

> (doom -iwad /usr/share/games/chex.wed ) ... but is it ok for Doomsday ?

If I read "doomsday" only once more...

;)


Ugh, I remembered OSX requires the template but not that it can't support the directory option either. You know in Linux this is just $(mktemp), so much nicer. Fine fine I fix it.


fast-forwarding master to auto = 275ab437


So I guess we should close #324 (although I merged that into my Frozen Flask branch, but I can deal with that).


In this case the call to `retryPromise` is pretty hairy to read (since I'm passing a function and its parameters to another function), so I separated it from its use in order to make inspection easier.


@thomasrogers03 Is this gem designed to work with Cassandra 1.2 only?


Thanks!


Test results are pushed too...


can you move this into `stabilizeTree` where we already perform this check for diffing plugins? since all diffing plugins are rebuild plugins, we might want to remove the diffing plugin check and just keep just the rebuild plugin check in there.


I'll pop over now

On 9 June 2015 at 09:39, Simon Farrell <notifications@github.com> wrote:

> Either is fine (just got in from dropping Sylvie off)
>
> S
>
> On 9 Jun 2015, at 8:31 am, Alice Mason <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
> Yes, I was aware of that but couldn't spot what might be going on.
> Would meeting today at 10 or 2 suit you?
>
> Cheers,
> Alice
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/am8302/groupingModel/commit/a3e2617a93fb96b1804f5afc3acd40c99108788a#commitcomment-11579550>
> .
>


合并


Don't add directories ;-).


I noticed after reviewing the code once again that I misspelled a comment.


this is already in `stabilizeTree`


Is jsdoc what we're using in other places to document functions?


a Page Blocked --> a Blocked Page


This `before` block looks to be nested under `describe "Visiting profiles"`, which means it will be applied to all examples in the block including `describe "not signed in"` and `describe "signed in"`.


Sure.


Ah this looks awesome.. :) Does the physics act as an api that can just be used by Magic or does it have to be turned on for all explosions?


This is something even recommended by the avocado logging guide. Good idea!


llala


What is the difference? or just simple?


第二次


OK


an Error Page --> a broken page


:D I've been working with Thinkofdeath on this one- we added a config option to BlockPhysics so that you can turn off the explosion behavior. Then he added an API for me to use.

So, currently just Blast and Collapse use it- we're still sort of tweaking and working out the kinks, but it's definitely very impressive! I'm not sure if a "real" server would want to hand this kind of extra CPU load, though.


Yep, I haven't thought of that. Makes sense indeed.


Hmmm, OK, I can get behind this.


Documented all of tonight's new stuff here:
http://laravel.com/docs/master/responses#attaching-cookies-to-responses

On Mon, Jun 8, 2015 at 8:58 PM, Franz Liedke <notifications@github.com>
wrote:

> Awesome! Where is cookie usage documented, by the way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/28a5f58790cd717ba36a8a920fc1a8844029bbc8#commitcomment-11579699>
> .
>


ㅇㅁㅇ


I went ahead and tweaked this more and made it more consistent with disabling CSRF verification. See new EncryptCookies override in laravel/laravel.


Well done.


Ok .. looks amazing. :)


Verifies XWalkView can block response
Verifies XWalkView can redirect to default page when load error page


lalala


Yep, I've punted pretty hard here.


Fair enough.


Link To An Error Page" (not available page or internet connectivity is not available) to redirect to the settled page(baidu home page).


Also, boost was added as a dependecy. This is made to make it easier to be ported.


`s/will/you`


That's a pretty good and concise explanation. Nice.


Another verb valency error of mine :) Good.


What's JFX?


이 수정은 필요없을 것 같네요


Any thoughts on that?


rax retry tests


이 주석은 필요없을 것 같아요


Not really, but just a note that on the PC side, support for baud rates is driver/OS dependent. My Mac won't allow 250,000 to be used in the configurator, but 230,400 works fine.


只支持这个就默认啊，需要让用户写么？


webhook이 null이면 404 Not Found로 응답해야 할 것 같네요


Menu 组件脱离了 Dropdown 的行为，是不用单独存在的吧。


Merged.  Thanks!  (though I probably need to make the corresponding change to the gh-pages branch)


Closed with 2e96fb2.


These look wrong. "1" I believe means that something is NOT supported by the hardware, which is wrong.


Just come across the word Tree - that will be questions for tutorial 2 - cheers


So can I fix this by nesting my ```before do``` block under ```describe "signed in"``` like below?
```
describe "signed in" do

    before do
      login_as(@user, scope: :user)
    end
.....
```


According to the log of nosetest, it seems all related to the opt  "remove_constants_and_unused_inputs_scan" or other opts in scan_module. I'm debugging into it, but do you have any resource about mechanism of this opt or scan_module? That'll be easier for me.


resource type을 지정하지 않으면 디폴트가 프로젝트가 되니 이건 프로젝트를 삭제할 권한이 있는지 검사하는 것이 될 것 같네요.


Spectral clusters are now only calculated once per template, and then inherited in every child of said template.


done.


@Shougo I've found something odd using deoplete: sometimes when typing a letter in Insert mode, it will insert the following word many times. It does not always happen, and I haven't yet been able to identify a pattern for when it will happen.

I also find that [my shortcuts to exit Insert mode](https://github.com/alxndr/dotfiles/blob/73a48d3f/.nvimrc#L91-93) don't always work. Hitting command-[ does still work.

I don't know anything about VimL or debugging vim... But if I can help debug in any way please let me know.


yup!


rax retry tests


`=!= │ trigger: unable to create hook for trigger "eep" (bad arguments)`


hook을 삭제하는 것을 "프로젝트에 대한 갱신"으로 보고 있다면 Operation.UPDATE로 바꿔야 할 것 같고, "webhook에 대한 삭제"라는 의도로 표현하려면 resource type을 webhook으로 바꿔야 할 것 같아요.


Some problems with this test?


Yeah, obviously this isn't valid and these symbols aren't actually used (or there would be problems).  Without this, I get the following compilation errors when doing a debug build:

```
$ "arm-none-eabi-gcc" "-mcpu=cortex-m3" "-mthumb" "-Tlayout.ld" "-L" "/usr/local/lib/rustlib/thumbv7m-none-eabi/lib" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/examples/rgb_pwm_lpc17xx.o" "-o" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/examples/rgb_pwm_lpc17xx" "-Wl,--gc-sections" "-nodefaultlibs" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/libzinc.rlib" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/deps/libvolatile_cell-aaa8227b8af98fc5.rlib" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/deps/librlibc-68a115e07ddb7f5c.rlib" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/deps/libcore-457ad95defb42ceb.rlib" "-L" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug" "-L" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/deps" "-L" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/build/zinc-456b0486afb7648a/out" "-L" "/home/posborne/Projects/rust-playground/zinc/target/thumbv7m-none-eabi/debug/build/zinc-456b0486afb7648a/out" "-L" "/usr/local/lib/rustlib/thumbv7m-none-eabi/lib" "-L" "/home/posborne/Projects/rust-playground/zinc/.rust/lib/thumbv7m-none-eabi" "-L" "/home/posborne/Projects/rust-playground/zinc/lib/thumbv7m-none-eabi" "-Wl,-Bstatic" "-Wl,-Bdynamic" "-lm" "-lgcc" "-lnosys" "-Wl,--verbose" "-Wl,-Map=output.map"
GNU ld (GNU Tools for ARM Embedded Processors) 2.24.0.20150304
<...>
attempt to open /usr/bin/../lib/gcc/arm-none-eabi/4.9.3/../../../../arm-none-eabi/lib/armv7-m/libnosys.a succeeded
attempt to open /usr/bin/../lib/gcc/arm-none-eabi/4.9.3/armv7-m/crtend.o succeeded
/usr/bin/../lib/gcc/arm-none-eabi/4.9.3/armv7-m/crtend.o
attempt to open /usr/bin/../lib/gcc/arm-none-eabi/4.9.3/armv7-m/crtn.o succeeded
/usr/bin/../lib/gcc/arm-none-eabi/4.9.3/armv7-m/crtn.o
/usr/bin/../lib/gcc/arm-none-eabi/4.9.3/armv7-m/libgcc.a(unwind-arm.o): In function `get_eit_entry':
unwind-arm.c:(.text+0x154): undefined reference to `__exidx_start'
unwind-arm.c:(.text+0x158): undefined reference to `__exidx_end'
collect2: error: ld returned 1 exit status
```

Defining the symbols (even to something invalid) fixes the problem in my case.  There are a few threads out there but the "right" solution seem to be elusive.  Can you reproduce the problem on this branch without these changes?

Here's the generated map file: https://gist.githubusercontent.com/posborne/a560598b9cc88324b22d/raw/gistfile1.txt


Error in spelling of canola


I thought that's you are calling PGraphicsFX2D. I added it for the future. Incorrect, or shall I remove it for now?


What's this?


comment test


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


I thought so, doesn't look like it is though. One of the Traceur tests is breaking the build on [Node 0.12](https://travis-ci.org/babel/babel/jobs/65988074), but [iojs](https://travis-ci.org/babel/babel/jobs/65988075) passes.


Ok.  I have committed the correction.Date: Mon, 8 Jun 2015 19:40:55 -0700
From: notifications@github.com
To: bloccit@noreply.github.com
CC: Bmcg0707@hotmail.com
Subject: Re: [bloccit] assignment 54 completed, public profiles (6a66e49)

yup!

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


@ejeschke can clarify, but I think the observation object only has information about the time and place, but you need to feed in a target. So in the above example you compute the altitude and azimuth of the target `sf`/`sm` for an observer at the location/environment specified in `obs` and time `t`. Below (see [line 264](https://github.com/bmorris3/astroplan/commit/ccae34f3a508a9db311e45dd70fe7d25221f704d#diff-d1475973061987d43ea4fea64513b64aR264)), I think you *should* be able to call `cr.alt`/`cr.az`.


I have added a pic of my settings as an example of how to set them. #3


It was 024ae670cbde3acb3f414c04f36cd1c6e116a5ea.


Can you change this to "HackCWRU will happen again in Spring 2016, but until then you can view the Spring 2015 event here"?


commit also includes filter always on


void glScaled(	GLdouble  	x,
 	GLdouble  	y,
 	GLdouble  	z);


you repeat this path a lot:

```
/Users/annaglander/Code/monthlyrep/tests/test_toplevel_folder/
```

Consider abstracting it out:

```python
BASE_REPORT_PATH_TEST = '/Users/annaglander/Code/monthlyrep/tests/test_toplevel_folder/'
```


ou deixa assim msm foda-se


Oh sorry, forgot to ignore the results

On Mon, Jun 8, 2015 at 10:08 PM, Ryan Newton <notifications@github.com>
wrote:

> Don't add directories ;-).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iu-parfunc/lvars/commit/3a55fa03786b281712750257b3f80334abde1f7d#commitcomment-11579772>
> .
>


WOW, this may be a copy-and-paste error Orz. I will remove it.


这招好暴力


oh you actually **do that** but don't use it. Consider writing a function:

```python
def get_report_path(name, test=False):
    if test:
        return BASEPATH_TEST + REPORT_FILES_TEST.get(name)
    else:
        return BASEPATH_REAL + REPORT_FILES_REAL.get(name)
```


I think the specific use-cases above aren't necessary, nor is the apology.  Unless this was really the thing that was tripping up our pilot users.


`cli_util` depends on several non-Dart-Team packages, which can cause problems for packages that need to be brought into the Dart repository.


I don't have a solid justification. A quick search found many people reporting sizes on the order of 100's of MB, so I thought a smallish disk would be good, though wanted to mention the performance implications.


Have you tested this? Does it indeed fix the issue?


Typo:
"If you will import" --> "If you import"


I've just included it in a PR: https://github.com/slackhq/SlackTextViewController/pull/200/files

I did test it on many devices, versions and orientations, and all looks great, expect for that tiny edge case I was telling you. I might fix it later. Thanks again!


@knownasilya Very true. I'll try to tackle that in time


fast-forwarding develop to borshop-integration-1116-jira/lrb/riak-1802 = 36ff0844


`ps ajxw` should show both the pgid and process state on a Mac.


I really couln't care less. It works :+1:


Ended up using `current_command += 2` as the first line inside the if block.


Thanks :)


fixes #34  and closes #34


wat bedoel je met visualiseren?


Looks reasonable


That's sad and defeats the purpose of the package. It's intention was to provide a unique way of detecting the SDK directory :-(


@gravyboat I understand the value of PRs in general but am not sure i see it as that useful here - the salt code isn't really code in a strict sense and reviews without testing have less a value than with full-blown programming languages (in my opinion). Just my two cents - I'll do PRs from now on.


Just wondering, how did you create this box? I am currently trying to build a Debian Jessie x86_64 Vagrant LXC box myself, but running into issues doing so.

Thanks for your contributed box by the way!


I recently changed build/run to use cordova. Let's make it 'cordova' for all.


een mooie uitgelijnde formule zoals hij ook in de haskell broncode staat


Thanks, i try to read it for thursday.


Are done ?


I'm sorry i don't know how to undo this help!


Shouldn't that be new bitcore.PrivateKey()?


This is only used at one place. Why did you move it up here?


It was a good stop-gap solution, but dart-lang/sdk#16994 is being actively worked on which will soon obsolete the package anyway.


Would that be something like "when 'tailLines' is negative, show all lines"?


check `.length` to also catch empty arrays


@s-u, I am relying on you to eventually use some of of these ridiculously powerful commands. This one: start renaming a folder and press cmd-enter to fork the entire folder under the new name.


Hello, Yes, I have already finished. You can merge it. I think to add other
funcionalities another day about other parts but I will tell you.

On Tue, Jun 9, 2015 at 9:44 PM, Inbar Gazit <notifications@github.com>
wrote:

> thank you @JaviKia <https://github.com/JaviKia> this is very good. Are
> you ready for me to merge all your recent changes or are you still working
> on this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JaviKia/DocuSign-.NET-Client/commit/12bd5f04eb875421e314844b7d8688f6c90c1e4f#commitcomment-11597020>
> .
>


I would move it before the `$locations = get_nav_menu_locations();` line since the Customizer doesn't register new nav menus. A comment should be added too.

The `fallback_cb` change is interesting. Shouldn't we move this to `wp_nav_menu()`?


`ConfigureAwait(false)`


Future plans. But OK, to remove.


I made a recent experiment on my side, which seems to indicate that using vectorized multiplication is faster. However my context is slightly different, so you'd have to do the experiment again before reaching a proper conclusion. See #1255


GHX_Slinky_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1676 of 1676 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)


Who is this?

Did you try git revert?

--
Esha Joshi
B.S. Software Engineering, 2016
California Polytechnic State University, San Luis Obispo
ejoshi@calpoly.edu | 408-722-7149

On Tue, Jun 9, 2015 at 1:05 PM, dc2000usa <notifications@github.com> wrote:

> I'm sorry i don't know how to undo this help!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSC366/Spring2015-Section-01/commit/944d6ccc1e41b4fc34db39cf4962ee4403aa2c64#commitcomment-11597382>
> .
>


* TODO: rewrite this as it is doing way too much and is very messy now


// FIXME: Handle this situation as it is then unclear what went wrong


* TODO: refactor these 2 functions


'cipher' => MCRYPT_RIJNDAEL_128, // FIXME: See if we can update this to the recommended 'AES-256-CBC'


Can commits like these **please** include a comment about what changes we are getting in the dependency?


Should be fine - see https://github.com/kenwheeler/slick/blob/master/bower.json.


Looks like a left-over, going to remove it. THanks. And yep, the current version is not configurable.


Форк вы конечно можете вести, но хочу отметить, что хоть мы и убрали поле `clear`, авторизация через нешифрованный пароль в оригинале также возможна: для этого можно просто использовать поле `crypt`, указав `$cryptscheme = "clear"` в файле конфигурации. По нашему мнению, хранить пароль как в шифрованном, так и в голом виде просто нету смысла, по этому мы и `$cryptscheme = "clear"` никому не советуем, разве что по другому никак.


Victor, so what is the proper way to kill OsmAnd app in Android 4+?


No, it's about finished jobs. Report doesn't show stdout/err of successfully finished jobs.

http://mitra.stanford.edu/kundaje/leepc12/bds_monitor/test/test.bds.20150609_093337_667.report.html


rax retry tests


Where is this being used? Did you mean to use caps?


> Doomsday

"It justs works" this time


s/`PropagateEventAsync`/`PropagateEvent`


@clebergnu  good point, will do it in the next PR. thanks.


The way I describe this is

"A remote outlearn module must be available at the time of import.  For example, if you reference `delightsome/grand-second-module`, be sure that it exists on Outlearn before importing a path that references it."

Does that seem clear?


Let's say "manually re-import" instead of "manually refresh"


Have you considered putting the Cell in a field and using it from there, instead of retrieving it from the action?


There was also a bug in taillines. There is a if statement what filters out tailLines < 1. So If I have negative number in bds.config, it's just ignored not infinity.


This error is most likely just a follow-up error. The boot image should be created by zygote, and if that fails, the subsequent compilations will fail as well. So look out for the first error. It could possibly result in this one, yes, as it might prevent compilation completely.


`else {` should be on its own line.


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3808/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3806/


GHX_Slinky_IDD (mmatts) - 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)


`Task.WhenAll(...)` instead of foreach.


Makes sense to refactor it to re-use common components. I will go ahead and do that.


Use `loadsh/object/keys` instead because `Object.keys` is not transpiled by babeljs


ha. good to confirm with @rowenaluk but I think we're dropping explicit support for requisitions until such a time as we understand them better. could be wrong though and that might only refer to SMS requisitions


Makes sense.


Start comment with an uppercase and end it with a point. (I know our comments don't always respect that but you sould do it)


LGTM


use `lodash/object/keys`