> I'm a fan of fail early

then maybe you want the create() method to be the one to throw. that avoids even having the BrokenCurl class altogether.


I don't mean to be a REST-fundamentalist here, but shouldn't the ACCEPT-header be used by the client to request different format by setting desired media-type? Media-type for csv is text/csv, json is application/json etc. This would be straightforward and standardized.
https://www.iana.org/assignments/media-types/media-types.xhtml


I assume I should release 1.18 with this fix no ?


Ok the rename to 'MinnowBoard' bugs me, there's already a minnowboard 1 so you just can't call it that. I vote keeping it called Minnowboard MAX and in the docs mention that all compatible mmax boards are treated as if they where mmax.


clear all;

n = 100; %number of samples

for k = 1:50 %50 iterations/runs

x = rand(n, 1); %x is uniformly distributed from 0,1
y = rand(n, 1); %y is uniformly distributed from 0,1
z = zeros(n, 1); %initialization of z

for i=1:n
    z(i) = x(i)^2 + y(i)^2 %calculation of z
end

for j=1:n %indicator function: if (x,y) are within the square, accept, if not reject
    if z(j) <= 1
        z(j) = z(j);
    else
        z(j) = 0;
    end
end

z( z == 0) = []; %removing values not in unit square
I(k) = length(z); %finding number of samples within the unit square
Area_of_circle(k) = I(k) / n; %calculation of area of circle
Pi_estimation(k) = 4*Area_of_circle(k); %calculation of pi estimate
end

Variance_pi_estimation = Var(Pi_estimation);


why?


yes it is current QDataStream implementation default. Qt doesn't claim that it will never change in future and it can be also explicitly set. So it seems in the end that it id highly probable that QDataStream can be used for SI binary encoded integers. These all assumptions I had to read in the doc in result of your comments. Write one for(..) cycle is much more easier for me, even if I can save one line of code with QDataStream. So philosophically yes, you might be right, but what is a real gain from proposed change? Readability? Efficiency? Portability? Performance? Code size? Give me one strong reason to make the proposed change. FYI, QDataStream read int implementation is at http://code.woboq.org/qt5/qtbase/src/corelib/io/qiodevice.cpp.html#_ZN9QIODevice4readEPcx, it doesn't seem to be much more simpler to me :)


clear all;

n = 1000; %number of samples

for k = 1:50 %number of iterations

x = (rand(n,1)*2.2)+0.8; %creation of uniform distribution from x in [0.8, 3]
y = zeros(n, 1); %initialization of function y

for i=1:n
    y(i) = (1+sinh(2*x(i))*log(x(i)))^(-1); %getting values for our function, y(x)
end

sample_mean(k) = sum(y) / n; %calculation of sample mean

area(k) = (3-0.8)*sample_mean(k); %multipling the sample mean by the volume to get an approximation of the area

sample_variance=var(area);
end


Well the changing of PLATFORM_NAME seems pointless now...


This is what's avoided by calloc :)


Rendering problems....
The class com.gc.materialdesign.views.ButtonFlat
On Sep 9, 2015 6:38 AM, "Sandra Konotey" <buelersandra@gmail.com> wrote:

> Thanks
> On Sep 8, 2015 10:33 PM, "Bartosz Kosarzycki" <notifications@github.com>
> wrote:
>
>> You cannot import the library via .gradle file anymore.
>>
>> Just use File -> New module -> Import JAR or AAR Package
>> from Android studio to import the *.aar file as a module
>>
>> Point to the direction of the file I'd uploaded to github:
>> compiled-versions/MaterialDesign-md_ready-release.aar
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kosiara/MaterialDesignLibrary/commit/e3a94ec606062eeb3ae2421a119cd729c43fee76#commitcomment-13123798>
>> .
>>
>


Ok this looks like a bad merge, you need this pointer.


I would just use void for parameters here since you don't use them.


But where are they defined? This will cause a segfault


@xiexingguo could you update the title of the commit message? like
```
msg/simple: start over after fails to bind a port in specified range
```

coz by reading `Update` a certain file does not help the reader, before he/she dives into the code.


No need for a display in this example.


Disagree, those boards all have a uart on /dev/ttyS0 which is generally used by getty but doesn't have to be.


Is that really what the boards declare themselves?


Thanks, Done


clear all;
P = [0.75 0.25; 0.6 0.4];
pi_vector = [1 0];
n=500;

new_state_vector = pi_vector;
temp_vector = zeros(1,2);
g = zeros(1,500);
final_state_vector = [1 zeros(1,n-1)];

    for i=2:n
    temp_vector = new_state_vector;

    new_state_vector = temp_vector*P;

    g(i) = rand;

        if new_state_vector(1) >= g(i)
            final_state_vector(i) = 1;
        else
            final_state_vector(i) = 2;
        end
    end

    for j=1:60
        final_state_vector_last_60_points(j) = final_state_vector(j+440);
    end

        figure;
        stem(final_state_vector);
        xlabel('kth run, k from [1,500]');
        ylabel('Current State, where a given state for a kth index = 1 or 2');
        title('DTFS MCs plot for 500 samples for a=0.25, b = 0.6 for probability transition matrix P');

        figure(2);
        stem(final_state_vector_last_60_points);
        xlabel('kth run, k from [441,500]');
        ylabel('Current State, where a given state for a kth index = 1 or 2');
        title('DTFS MCs plot of the last 60 samples for a = 0.25, b = 0.6 for probability transition matrix P');

        number_of_ones = find(final_state_vector_last_60_points == 1);
        sum_num_of_ones = length(number_of_ones);
        percentage_of_ones = sum_num_of_ones/60;
        percentage_of_twos = 1 - percentage_of_ones;


No, that's not how the MSON was described in the fixture above.

```
- e4 (enum[number]) - typed enum -> samples && default should be numbers
    - sample
        - 3
        - 4
```

1. `enum[number]` still allows other MSON types like `string` and stuff to be included.
2. The sample in the MSON above is an array according to the MSON syntax. So, it should be treated as an array


Not sure why we need a difference in the platform type from mmax, but in any case needs a matching types.hpp entry


fair enough, lgtm.


I think it would be prudent to check which event source the first event came from as proof of which timer ended first. You can use ev.timer.source and compare it to the value returned from al_get_timer_event_source which you can cache when you need them above.


done


should this be >= ?


My concern was that the switching from the `ResumingClientHandshaker` implementation to the `ClientHandshaker` implementation during runtime by means of setting the `fullHandshake` property and delegating to the superclass seems a little *awkward*. But that might only be due to personal preferences ...
Regarding the `doProcessMessage()`, you are probably right. I think we could pull up the switch statement into this method and delegate to abstract `doReceiveClientHello(), doReceiveClientKeyExchange()` etc methods. This way we might be reduce code duplication even further within the `Handshaker` hierarchy. What do you think?


@3nids been thinking some more about this. Now I'm wondering if click click mode for node tool isnt the right approach. Would a better solution be:

- click selects a point, ready for deletion via delete key or navigation via < >. Corresponding row is selected in the node table, so coordinates can be manually entered.
- click and drag moves a node

The current ux is quite weird. You have to click to select a point, and then it automatically starts moving. To type in the node table you then have to move the mouse to the table, and all this time you have the rubberband following the cursor and going all the way across to the node table.

What do you think?


d


Should we add Muscle as an alternative?


> if user in enum will write array, he probably means members of enum.

We should not be making that assumption, because this is a valid MSON and if we make that assumption, then the MSON syntax will be much more ambiguous and will lead to more problems further down the road.


Cool, thanks @hpehl


Or another suggestion is to have an orb_topic called "boot_complete" that is generated by the entity/module that can determine/deduce the information( i am assuming that commandar would be an ideal one ). Then the mavlink_receiver can block until this message/topic is published.  This will not need any statemachines/QGC changes.


@wooorm Caught by Alex ;)


The deciding factor is reaching the end of the startup script, and this is how its done on all platforms.


Change requirements.txt to fix AzureWebSites git deploy.


Fixes #2622


Hi,
thanks fo this plugin, it works but i found an issue using it:
by default, all the conditionnal fields are show.
For example in the registration form, based on a country, a choice of city will be displayed...but it display all the fields of city related to each country
Do you experience the same issue?
Regards,
YaRi


@boboldehampsink I see this commit in reference to https://github.com/phonegap/phonegap-plugin-push/issues/31.

Any reason why this is not in `master` yet? Also, registration at init will also need to be removed to make most of programatically registering, right?

I don't know why this is not considered in first place. Without this, as per my understanding, app developer cannot give an ability to user to disable push notifications. Please let me know if I'm wrong. Thanks!

cc: @macdonst


@VPusher
les fichiers `.ts` on ne peut pas les recuperer avec un npm ? plutot que de les mettre dans le code du projet ?


prompt("What is your name?");


What about making the ctrl modifier do the click click move style.

On Wed, 9 Sep 2015 5:09 pm Nyall Dawson <notifications@github.com> wrote:

> @3nids <https://github.com/3nids> been thinking some more about this. Now
> I'm wondering if click click mode for node tool isnt the right approach.
> Would a better solution be:
>
>    - click selects a point, ready for deletion via delete key or
>    navigation via < >. Corresponding row is selected in the node table, so
>    coordinates can be manually entered.
>    - click and drag moves a node
>
> The current ux is quite weird. You have to click to select a point, and
> then it automatically starts moving. To type in the node table you then
> have to move the mouse to the table, and all this time you have the
> rubberband following the cursor and going all the way across to the node
> table.
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/a2f0763bab551da366b315fe562a571ec4dd1edd#commitcomment-13128924>
> .
>


In a shell script, you can use `pushd` and `popd` to temporarily change the current working directory for the duration of the script (e.g. to the project root). I wonder if that works in Windows?


1: M111 unrecognized component name: legacy


1:1: F403 'from bibexport import *' used; unable to detect undefined names
1: D100 Missing docstring in public module
1: L101 copyright is missing
2:1: F403 'from bibknowledge import *' used; unable to detect undefined names
3:1: F403 'from websearch import *' used; unable to detect undefined names
4:1: F403 'from websession import *' used; unable to detect undefined names


1: D100 Missing docstring in public module
21:1: I100 Imports statements are in the wrong order. from fixture should be before from invenio.config
21:1: I201 Missing newline before sections or imports.
22:1: I101 Imported names are in the wrong order. Should be CollectionData, FieldData
22:1: I201 Missing newline before sections or imports.
23:1: I100 Imports statements are in the wrong order. from invenio_search should be before from invenio_search.fixtures
26: D101 Missing docstring in public class
109:1: F999 redefinition of unused 'CollectionData' from line 22
109: D101 Missing docstring in public class
867:18: E231 missing whitespace after ','
868:18: E231 missing whitespace after ','
869:18: E231 missing whitespace after ','
877:18: E231 missing whitespace after ','
878:18: E231 missing whitespace after ','
879:18: E231 missing whitespace after ','
887:18: E231 missing whitespace after ','
888:18: E231 missing whitespace after ','
889:18: E231 missing whitespace after ','
897:18: E231 missing whitespace after ','
898:18: E231 missing whitespace after ','
899:18: E231 missing whitespace after ','
907:18: E231 missing whitespace after ','
908:18: E231 missing whitespace after ','
909:18: E231 missing whitespace after ','
914: D101 Missing docstring in public class
1043: D101 Missing docstring in public class
1066: D101 Missing docstring in public class
1544: D101 Missing docstring in public class
1622: D101 Missing docstring in public class
1970: D101 Missing docstring in public class
2133: D101 Missing docstring in public class
2808: D101 Missing docstring in public class
3026: D101 Missing docstring in public class
3876: D101 Missing docstring in public class


这样修改的话，安装说明里也要做更新的，不然服务端机器如果没有在env里指定`MONGODB_URL`，会造成无法连接mongodb的问题


Do we really need to set last=true here? I mean, if downstream behaves, we will not ever be invoked again, right?


풍악을 울려라!!! 야호~ 감사합니다 ㅠㅠ


@sindresorhus awesome! :)


In general I'm open to find a better approach if you feel requiring the command on the end of the startup script is too limiting.


Ignore the commit message; actually this does not contain the fix for #1408.


Its a typo, will rectify and submit by the end of the day


the magic fix is from @lpshane :)


'link' => url('taxonomy/term/' . $wrapper->id()),


link to the upstream issue on JIRA : https://issues.apache.org/jira/browse/CB-9240


Yes I'll remove those commented codes


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1838 of 1839 tests passed)

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


ok


So, I guess this is for robustness, to ensure that from now on we always just cb(true) and return, no matter what?


Pushed for Tango at least 409f8515.


https://github.com/apiaryio/mson/blob/master/MSON%20Specification.md#44-sample

```
- list: 3, 4 (enum, sample)
```
Is equivalent to:
```
- list: *3, 4* (enum)
```
Which, is equivalent to:
```
- list (enum)
    - Sample
        - 3
        - 4
```


Merci, c'est résolu dans edc75b2 :)


`Suppress` :)


We've had some feedback saying that the Java 8 examples are hard to understand if you are not an experienced programmer. We plan to have special Java 8 content.


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hi,
This feature was just committed to the OpenEMR codebase:
https://github.com/openemr/openemr/commit/f64de84a1b9285cb4e70086de2d6345386e9f83a
Note it was extensively modified, but the same overall functionality was kept intact.
-brady


https://github.com/apiaryio/mson/blob/master/MSON%20Specification.md#43-nested-member-types

```
- colors (array)
    - red (string)
    - 5 (number)
```
Implies an array structure whose individual items MAY be strings or numbers with sample values "red" and "5", respectively.

```
- colors (enum)
    - red (string)
    - 5 (number)
```
Implies a colors Property Member Type that MUST only have a value of the string "red" or the number 5.


Remove printStackTrace and handle it properly


Why do you think this is better than replacing the reference?


dfd


cc @chrisseaton


yeah, this is a little bit awkward as osd_lock is not a recursive lock.

could have a `unique_ptr<Locker>` for this:
```c++
unique_ptr<Locker> lock;
if (!osd_lock.is_locked_by_me()) {
  lock.reset(new Mutex::Locker(osd_lock));
}
```


Ok so it does not really make sense to keep this unit tests. Ok!


dkm


Yes I'll move CSS files to the top, had thought of putting only the JavaScript files to the bottom


Yes, you're right.


Going back to views ? Why ?


Bootstrap-min should require Jquery


@bgiesing This indeed an outdated path. I will update and squash this. But please update your main opengapps repo (and try to keep track of it when things change ;-) )


Yes! Perfect. This is an occupational hazard when using exceptions as part of the normal control flow - similar problems show up in the speculative plumbing for unions and intersections in `flow_js`.

Oops, not quite perfect... I think you need to pop back to the topmost `VarScope`, right?


机器人？


这版本好奇怪。。


removed my comment which does not make sense, we can not hold the lock in the whole method.


Yes, I saw you fixed already :) thanks


Ugh, no it's worse than that. You need to remember the `LexScope` you pushed on 1350, and pop back to (and including) that. Because a) the try/catch might of course be in a lex scope, and more importantly b) an arbitrary stack of lex and var scopes might have been pushed at the point where the throw happens.


km


m


Issue?


ok


imho, could have a better name for `need_lock`, e.g. `is_osd_locked`, as we have other locks in `OSD`.


I don't think so. We catch the exception right outside the sole call to `catch_clause`, so we should unwrap to the right place. We just need to be careful to stay balanced against the `push_lex` ~8 lines up.

If you're right, it should be simple to come up with a failing test, but I'm not able to. Can you?


**Eslint**: 'return' outside of function


If you don't trim the url string; it will crash.


@rkintada I'm not a socket expert but why is the use of memcpy insufficient? We are copying the data from the client who sent the message into our struct for the target client. Why should the port not be specified?


fdgdvgsdg


Избавься от статических методов


I found that poor performance when low memory killer executed


This will crash on a ipod with a url that works fine on a iphone.


Good idea, will do that.


Still working on that project?


@mattab I had removed that one, and the translator who added it asked me to readd it. As he has done quite a lot translations for the language I couldn't give him a proper reason why we would remove his site, but other are allowed to stay.
Btw. I would vote for completely removing all URLs from here. There are already some translators signing up to add a link only. Another good option would be to disallow anyone to edit that string and add a comment instead, that anyone who would like to get mentioned should get in touch with us. But we should set up some rules which pages will be added and which not.


Thanks.


This field should probably be transient, we don't want to save it along with the publisher.


@opengapps/packagers that had pulled Nick's commit (so for sure @NicholasBuse himself) please reset your HEAD to the one of the remote repository. I removed the erronous play store commit of bgiesing and squashed all those changes into 1 commit.
@bgiesing is still lucky for source_competition.txt that it will be counted as his... :-P


(What you're saying in your second comment does make sense, now that I see your referring to possibly pushed lex scopes inside of `b.Block.body`... still can't make Flow blow up on it though.)


You need to add: intent.addFlags(android.content.Intent.FLAG_ACTIVITY_NEW_TASK); -- if you are using the application context, you must pass the intent with this flag.


> the caller shall take control of it.

is sort of confusing, how about

> it was acquired by the caller, pretend that nothing happened.


saw approval from shino
at https://github.com/basho/riak_cs/commit/1c23c6fda53a73f58b9000828804ae014f9f34ba


merging basho/riak_cs/bugfix/gc-manager-eqc = 1c23c6fd into borshop-integration-1240-bugfix/gc-manager-eqc


basho/riak_cs/bugfix/gc-manager-eqc = 1c23c6fd merged ok, testing candidate = f57bf74b


could use `BOOST_SCOPE_EXIT_END` to do the cleanup.


Curly braces can now be omitted.


Hi @AdrianAlan what is the rational behind setting LumiBased in case of LumiList being passed ? By setting the splitting this way, you have a high chance at skewing the total job job and make request fail assignment

https://hypernews.cern.ch/HyperNews/CMS/get/prep-ops/1939.html


Please note in my implementation; I returned if it succeeded or failed.    And I wrap both items in try catch so that bad url's don't crash the openUrl function...


Would an other commit message be more useful for your log readers?


thank you @ztmurphy21 ! :smile:


Better to pass only tenantId internally and get tenant domain when it wants using util method.


Totally forgot to merge this to the "stable" version...

That won't help much, though ;)

There are lots of undefined biomes in the list.
However, that shouldn't affect anything but less log output errors.
I guess it would be easier to fix the following biomes here than wait til the next update of the "stable" MGC: 'snowy' 'junglee' 'jumble' 'grassytwo' 'junglee' 'grassytwo' 'apline'.

Note, that the "alpine" biome IS defined, but still logs as an error.


thank you @modus75


isNotEmpty not covered white spaces, better to use isNotBlank


sdfasdf


video
image


Ok, the last patch in this series fixes them all up again. Missed that.


@GrahamCampbell thanks for pointing this out, I pushed a fix


Drag and drop prevents from using advanced digitizing. I am not in favor of having the two modes as it adds more configuration (softwares  should behave the same everywhere) and it would make the code much more complex.
I am quite sure that the selection model on the table view would ease deletion and selection of nodes.


@GrahamCampbell thanks for pointing this out, it's fixed now


嗯 好神奇 我开房点确定后 他自己PR


嗯。。连minor都不想升级的时候用这招。。不过我见过 x.x.x-alpha 没见过这种 嘿嘿


Kind of off topic, but is there a reason we hard code the ID's in the mentioned code when we have them here: https://github.com/spark/particle-cli/blob/master/lib/deviceSpecs/specifications.js

Also why do we bother with the MFG name at all?


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


That is incorrect syntax for Java Andrius the {0} will not be substituted with the value of url you'll get "Request sent to {0}" as output.


Fixed.


![screen shot 2015-09-09 at 12 22 15](https://cloud.githubusercontent.com/assets/5224923/9756306/69aa1622-56ee-11e5-8a47-b2e12fbc5ad1.png)
![screen shot 2015-09-09 at 12 28 48](https://cloud.githubusercontent.com/assets/5224923/9756307/6a259522-56ee-11e5-811d-4674d8a2aade.png)


@JuliusSenkus add `<add key="LuceneExcludedPages" value="/500/"/>` configuration to nuget pkg, too.


@jodal Thanks, fixed it.


Check null cacheEntry.getIdentityProvider()


Omformuler så det er klart at man skal spørge igen.


won't work with multi user installs -- need workaround


nice work !!!


I didn't mean to be insulting, but I think that

  - PhD student: before go/nogo
  - PhD candidate: when go/nogo passed

no?


See #34 :bowtie:


"when we enable `ckeditor` view, this also mean that admin will
be able to upload images accordingly, which this will cause a routing
errors if the admin try to upload one."

=> I don't understand this line. What is 'routing errors' mean? When I config ckeditor to rails admin's field and try to upload image to server from that field, never seen any errors.


here i forgot to add the module base. so this code had not been able to work  since this commit.


I hadn't received anything.
Also, it reads "I will no..."  which can't be interpreted as "I'm certainly willing to honour the terms of the deal".

Anyway, it's too late now. I don't want to conduct any business with you anymore.
You are to delete 4hunger and adbs and anything that came with them as well as revert the license to 2.1+ (the easiest way).


fast-forwarding develop to borshop-integration-1240-bugfix/gc-manager-eqc = f57bf74b


Dear author,

We tested, but we can't install the extension into Design Studio 1.5.
Is there anything wrong from our side?

Thanks, and best regards,
Zhan


+1
The first versions of action plugins differentiated alerts vs actions domains, but the missing alerts was left in some properties.
I'll update them on all plugins.


We don't run all reminders at once. Only one task per day is executed so users are fetched one time.


Seems like we might not actually need this include in the header?


update userlist.html


http://www.letv.com/ptv/vplay/1334633.html#vid=1334633  time is wrrong.


Фух,
 :)) к счастью удалось выдернуть из файлов class все правки,
осталось только JSP написать.

9 сентября 2015 г., 2:36 пользователь Александр Пархоменко <
paraleksi@gmail.com> написал:

> Что-то я с гитом перемудрил. Сегодняшняя работа коту под хвост.
> Удалось только кое-что вытащить из target.
> Но как это обратно выцепить из class файлов я не знаю.
> Завтра(сегодня) сделаю заново.
>
> 8 сентября 2015 г., 14:44 пользователь Slava Semushin <
> notifications@github.com> написал:
>
>> Если тег уже стоит то не важно когда пушить, если его еще нет, то
>> исправьте и потом поставьте/запуште.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/AleksSPb/Spring-demo-App1/commit/91f8bd84000a3c2457bdf6316654d14d75f52c81#commitcomment-13110048>
>> .
>>
>
>
>
> --
> Всего наилучшего, Пархоменко Александр
>

--
Всего наилучшего, Пархоменко Александр


Throughout the Gaffer API we pair get/set methods so that the existence of one implies the existence of the other. So let's just call this `currentFileFormat()` to make it clear that it's not something that can be set. Perhaps you could also add a comment stating that it returns the file format that would be used if execute() was called in the current context?


I never tought about it that way. Just seemed inconsistent to me.

On 09 Sep 2015, at 10:08 , Hugo Ledoux <notifications@github.com<mailto:notifications@github.com>> wrote:

I didn't mean to be insulting, but I think that

  *   PhD student: before go/nogo
  *   PhD candidate: when go/nogo passed

no?

—
Reply to this email directly or view it on GitHub<https://github.com/tudelft3d/website/commit/32611e06d561274da5a7bfed3365aa5c8e8f8e26#commitcomment-13129769>.


Are you sure? It might be that I just spent too much time with python. I'll
fix it when I get to the office.
On 9 Sep 2015 10:02, "Mads Nielsen" <notifications@github.com> wrote:

> That is incorrect syntax for Java Andrius the {0} will not be substituted
> with the value of url you'll get "Request sent to {0}" as output.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Praqma/codesonar-plugin/commit/77c4c8608a3693d4097eac8c7b78ef7a98762ee7#commitcomment-13129668>
> .
>


These are correct fixes imo. Since the condition_variable flags are now guarded by mutexes, it's no longer necessary to use atomic variables for them.


nice


Is it worth calling this `fileFormatSettingsPlug()`, to differentiate from GafferImage::Format?


Indeed, I think I'm going to refactor the location of this method.
The original location was on the constructor (and there synchronization is not needed), I moved into the process() to support unit tests without mail session connectivity, just to validate the email creation.
I think that this method should go to the constructor again, with a specific flag for unit tests mode.


I agree in principle, but we should probably have a consensus decision on this . . .


close #6


I have updated the license to 3.0+ and no one can make me change it to 2.1+


Yeah, would help if I didn't fat finger the c++11 directive to the wrong place, when I added it.


Not required for macro; it uses $crate


We use "" rather than <> when fetching anything other than system-installed headers. I'm not 100% sure that's right, but let's at least be consistent across the board and fix things all at once if it's not.


+1
I tend to use the old school instanceof but yes, getType() improves the semantic here.
Thx.


@jplitza @mortzu @genofire Dieser Workaround sollte dafür sorgen, dass fastd zuverlässig neustartet.


Seems like this one might not be needed?


Caps? Also; file does not exist. If you mean `18_builder_macro.rs`, then the sample is more extended and saturates the capabilities of the macro.


wow :+1:  :+1: :+1: :+1:  :goberserk: :godmode: :goat: :key:


Right! I thought it was used only for display, fixed.


I'm not familiar with Design Studio specifically, but there isn't too much to this project. It's really just including the .js and .css file in your webpage and calling the `wJSDrawGraph` method with your parameters?


Does this thing really work in practice?


Why?


    <nav class="main-nav clearfix">
    	<div class="inner">
            {{navigation}}
            {{> topbar}}
        </div>
    </nav>

感觉这个 nav 更应该写成 nav.hbs


I have a general complaint about such a fix for this warning.

I don't think renaming structs so they don't have the best name is a good solution to these warnings. I would prefer another solution. Was there any success with putting this/these in anonymous namespaces?


One small thing I should have spotted in a previous review. We pass smart pointers (like boost::shared_ptr) to functions only when that function needs to share ownership of the object in some way. If the function just needs access to an object and the lifetime is already guaranteed by the caller, then we just pass raw pointers to make it clear that no sharing of ownership is happening. So out should just be `const ImageOutput *`.


Hehe, copy-paste from @kbknapp :)


+1
I like the idea.
I think is good to add them to this PluginMessageDescription class, as is where the properties are consumed.


It's a very politicised issue these days (see https://www.timeshighereducation.co.uk/news/dutch-phd-candidates-fear-a-downgrade-in-status/2018838.article), but I strongly think the go/no-go doesn't make a difference. It should be PhD candidate always.


& to and


Added test method that takes 6 parameters.


I will revert it *again* and squash it. So afterwards everyone should update their HEAD again from remote.


Not sure why `@PendingFeature` isn't working here, as it's failing currently (it seems to behave differently in case of exceptions) :/


ok


In fact, i like this mode not you because i don't familiar with c++. I


It's a small thing, but moving the declaration of optionsPlug inside this conditional ( `if( const ValuePlug *optionsPlug = node->formatSettingsPlug() )` ) would scope the variable inside the conditional and make it a bit easier to read this large function (knowing that you can forget about that variable lower down). Alternatively, we could push the "do the format settings" stuff into a separate function.


the actual parameterized test method signature is composed from the where block + extra parameters, defined originally


@bgiesing (and other @opengapps/packagers ) please reset your 'arm' HEAD to remote origin now


How does it come that you have deleted so many linux-headers, while some of them are still available as gentoo-sources (e.g. 3.10, 3.12, ...)?


@NicolaeNMV Hi, the [Playframework ML](https://groups.google.com/forum/#!forum/play-framework) is the right place for asking questions about Play or play-slick :-) Using an in-memory DB in Slick is actually really simple, just change the Slick driver you have configured in your application.conf to be H2. If you have more question, please ask them in the ML I linked. Thanks!

P.S. You definitely shouldn't add play-jdbc module, as it won't work (you'll get the following error https://www.playframework.com/documentation/2.4.x/PlaySlickFAQ#A-binding-to-play.api.db.DBApi-was-already-configured)


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (775 of 775 tests passed)

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


could you format this in several lines?


Unfortunately I couldn't use default parameter values, as it complicated method calls and that info wasn't available by the time we are was constructing the method call to call the test


Hello, i update this two files and sync finish without errors right now, but the clan members list on site doesn't open.

jinja2.exceptions.UndefinedError
UndefinedError: 'dict object' has no attribute u'executive_officer'

Traceback (most recent call last)
File "C:\Python27\lib\site-packages\flask\app.py", line 1836, in __call__
return self.wsgi_app(environ, start_response)
File "C:\Python27\lib\site-packages\flask\app.py", line 1820, in wsgi_app
response = self.make_response(self.handle_exception(e))
File "C:\Python27\lib\site-packages\flask\app.py", line 1403, in handle_exception
reraise(exc_type, exc_value, tb)
File "C:\Python27\lib\site-packages\flask\app.py", line 1817, in wsgi_app
response = self.full_dispatch_request()
File "C:\Python27\lib\site-packages\flask\app.py", line 1477, in full_dispatch_request
rv = self.handle_user_exception(e)
File "C:\Python27\lib\site-packages\flask\app.py", line 1381, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "C:\Python27\lib\site-packages\flask\app.py", line 1475, in full_dispatch_request
rv = self.dispatch_request()
File "C:\Python27\lib\site-packages\flask\app.py", line 1461, in dispatch_request
return self.view_functions[rule.endpoint](**req.view_args)
File "C:\Users\user\Desktop\whyattend\whyattend\webapp.py", line 152, in decorated_function
return f(*args, **kwargs)
File "C:\Users\user\Desktop\whyattend\whyattend\webapp.py", line 1187, in clan_players
last_battle_by_player=last_battle_by_player)
File "C:\Python27\lib\site-packages\flask\templating.py", line 128, in render_template
context, ctx.app)
File "C:\Python27\lib\site-packages\flask\templating.py", line 110, in _render
rv = template.render(context)
File "C:\Python27\lib\site-packages\jinja2\environment.py", line 969, in render
return self.environment.handle_exception(exc_info, True)
File "C:\Python27\lib\site-packages\jinja2\environment.py", line 742, in handle_exception
reraise(exc_type, exc_value, tb)
File "C:\Users\user\Desktop\whyattend\whyattend\templates\players\players.html", line 1, in top-level template code
{% extends "layout.html" %}
File "C:\Users\user\Desktop\whyattend\whyattend\templates\layout.html", line 163, in top-level template code
{% block content %}{% endblock %}
File "C:\Users\user\Desktop\whyattend\whyattend\templates\players\players.html", line 70, in block "content"
{{g.roles[player.role]}}
File "C:\Python27\lib\site-packages\markupsafe\_native.py", line 22, in escape
return Markup(text_type(s)
UndefinedError: 'dict object' has no attribute u'executive_officer'

And another thing is that the "payment" doesn't show in template.
Do you have any clues?


Yes, makes sense, will change.


Nitpick: can you sort them?


nitpick: can you sort them?


the key/value pair aren't actual `<BigInteger, BigDecimal>`s, but I guess that's a Groovy thing


Does JPEG2000 also have a compression quality setting?


the preset page rendering and menu fixes could also go out with a 0.6.3 type release...


This breaks #86


seen u


It's about mess with adding and removing endpoints dynamically. I believe it will no longer be necessary after rework, at least not in this form.


혹시나 해서 코멘트 달지만 2개조로 개편되었습니다 notification 확인해주세요


is it reachable as iUserVale was tested checkValueAgainstSpace() ?


@3nids I can see how that would work for geometries with a small number of vertices, but what about polygons with thousands of nodes? (That's what I usually work with). In this case you can't use the table to select as its impossible to identify which a node from thousands of coordinate pairs, so the only way to select is by clicking with the mouse. At the moment is impossible to do this without triggering the move...


Без этого у меня вечером падало с ошибкой.
Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/threading.py", line 920, in _bootstrap_inner
    self.run()
  File "/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/site-packages/telebot/__init__.py", line 42, in run
    task(*args, **kwargs)
  File "group_bot.py", line 81, in tomorrow
    subject, tm = timetable.get_next_subject()
  File "/Users/artem/Programming/MiniProjects/CS_MSU_203_bot/timetable.py", line 28, in get_next_subject
    if is_now(now, subjects[x]):
  File "/Users/artem/Programming/MiniProjects/CS_MSU_203_bot/timetable.py", line 41, in is_now
    return (get_time_delta(tm, sbj["start"]) <= 0 and
  File "/Users/artem/Programming/MiniProjects/CS_MSU_203_bot/timetable.py", line 21, in get_time_delta
    return t(tm2) - t(tm1)
  File "/Users/artem/Programming/MiniProjects/CS_MSU_203_bot/timetable.py", line 20, in <lambda>
    t = lambda x: time.mktime(time.strptime(x, "%H:%M"))
OverflowError: mktime argument out of range


I'm not sure I agree with the two modes thing really. Sure it adds more
complexity but we have to be careful not to make the default behaviour to
jarring to existing users.

On Wed, 9 Sep 2015 5:59 pm Denis Rouzaud <notifications@github.com> wrote:

> Drag and drop prevents from using advanced digitizing. I am not in favor
> of having the two modes as it adds more configuration (softwares should
> behave the same everywhere) and it would make the code much more complex.
> I am quite sure that the selection model on the table view would ease
> deletion and selection of nodes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/a2f0763bab551da366b315fe562a571ec4dd1edd#commitcomment-13129617>
> .
>


I'm curious about the need for this branch - I would have guessed that if there is no format, then execute will fail anyway, so we don't really need to differentiate all the failure cases. Perhaps if there is no format we should actually be returning an empty hash as we do when the filename is empty? It'd be nice to ditch this branch if possible since it seems quite prone to not being kept up to date as we add new formats.


Oh god why :trollface:


Why do not use checkValueAgainstRange() ?


Seems like this removal could be accidental?


odbc 있는데 jdbc 또 필요한가요?


It worked! thanks to @frozenfung


maybe this is not compatible with parallel for. Do we need to check for main thread here as e.g. in FeatureFinderCentroided


Excellent.


@mauritsvanrees I've added the user 'plone' as owner.


이 어노테이션 정의는 왜 필요한건가요?


Should we remove `puts_output_on_success` option completely as no one is using it now.


We need a loading icon while the class in being opened/closed.
Font awesome provides some nice animated icons: https://fortawesome.github.io/Font-Awesome/examples/


done!


Oh, I accidentally added a basic reincarnator @ the spawn :)


We can revert back to previous logic *and* try to achieve performance close to `respond_to?`. There's a trick with `singleton_class.method_defined?(key)` so it would match instance methods of ancestors and self-methods of an object. Calling `methods.include?(...)` is anyway bad.

However I still think that current implementation is more adequate and correct. If object responds to some method (whether it's truly defined as instance method or it's just `respond_to_missing?`'s claim) then it's unsafe to overwrite its behavior. I cannot say more until I see a concrete example of something counterintuitive. Maybe in your setting you don't need `SafeAssignment` at all?

I would also like to /cc @michaelherold as he's an author of original implementation. It's interesting what he thinks.


This patch affects the drawing of report maps and so was followed up with https://github.com/Indicia-Team/client_helpers/commit/5477eec96b765e143edc7dd9b1cf462a96e4adf0


typo: last_n_samples_seen


LGTM :+1:


cmp() functions have to return consistent values. i.e. cmp(a, b) == -cmp(b, a).

Also cmp(a, b) == cmp(copy(a), copy(b)).

This suggests that equal actually is the best answer. :-)


The cache gets generated and the warning in `xfce4-appearance-settings` disappears.


For information. When I put this code in 1.c:

    void f() {
        if (!Token::Match(tok, "%name% ( !!)"))
        {}
    }

and run the match compiler on it then it generates this source code:

    // pattern: %name% ( !!)
    static bool match1(const Token* tok) {
        if (!tok || !tok->isName())
            return false;
        tok = tok->next();
        if (!tok || !(tok->str()==MatchCompiler::makeConstString("(")))
            return false;
        tok = tok->next();
        if (tok && tok->str() == MatchCompiler::makeConstString(")"))
            return false;
        return true;
    }

    void f() {
        if (!match1(tok))
        {}
    }

Now your optimisation will just change:

        if (!match1(tok))

into:

        if (!tok->isName() || !match1(tok))

and this should have a very small impact on performance imho. Especially if match1 is inlined.


Here you are trying to get the topic from its association with this `@post`.  But how can you do that since you have defined `@post` before this line?


Thanks! Drupal is now on `6.1.0` :)

https://www.drupal.org/node/2563799


단순히 파일목록을 나열한 코드 Block이라서 언어문법 강조가 필요하지 않습니다.


setSupportProgressBarIndeterminateVisibility(mSyncInProgress /*|| mRefreshSharesInProgress*/);

won't work anymore with the actual action bar and must be implemented as ssen in FileDisplayActivity, e.g. in the onCreate method line 214


1) .project ve .pydevproject dosyalarını ignore edin, Eclipse IDE geçince kendi kendine meta data oluşturdu.

2) python da parantez yerine tab sistemi var. tab length o yüzden önemli. best practice 2 kullanmak. ilk yazdığım kodda tab length 8 idi. Onu düzelttim.

3) IDE de tabs yerine whitespace yerleştir diye bir opsiyon var, onu eclipse de aktive ettim. ben kendi editörümde TAB tuşuna basınca \t değil de, iki tane whitespace yerleştirmesi lazım koda. Eğer kodu \t ile yazarsan, tab lenght 2 değil de başka bir şey olarak ayarlanmış bir editörde saçmalar. o yüzden bu ortak çalışırken önemli bir ayar.


I'm open to ideas and certainly want to be standardised. Bear in mind I want the API to be easy to call from matlab, R and similar. So anything that needs to fiddle with headers is not that appealling.

The goal is not to write a REST service, but if we can be compatible that would be great. I worry that by adopting this for one endpoint will mean I then have to honour it for all the others.


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


@gforcada That is a lot nicer! https://pypi.python.org/pypi/plone.api


Successfully tested in [daily-aliphysics build #12](https://alijenkins.cern.ch/job/daily-aliphysics/12).


test


Я не знаю, что вы там перемудрили, но история про вытаскивание изменений из class-файлов впечатлила. Не проще ли было написать все сначала, чем декомпилировать class-файлы?

:-)


Could these new images go in GafferImageTest rather than GafferTest? Or, even better, do you think we could use a testing methodology that doesn't require adding more binary data to the repo? Do you think we could just test against the pixel data for the original image (with some tolerance for any lossy compression) and then assert that the metadata implies we got the datatype and compression etc right?


I agree, let’s leave phd candidate then.

On Wed, Sep 9, 2015 at 10:21 AM, Ken Arroyo Ohori <notifications@github.com>
wrote:

> It's a very politicised issue these days (see
> https://www.timeshighereducation.co.uk/news/dutch-phd-candidates-fear-a-downgrade-in-status/2018838.article),
> but I strongly think the go/no-go doesn't make a difference. It should be
> PhD candidate always.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tudelft3d/website/commit/32611e06d561274da5a7bfed3365aa5c8e8f8e26#commitcomment-13130007>
> .
>


Published as 2.2.0


Issue with ladspa and fast lookahead limiter should be fixed in latest ffmpeg


저 spring-jdbc는 jdbc로 db 연결할때 스프링에서 제공해주는 driverManagerDataSource를 이용해서 추가해주었습니다. 처음 프로젝트를 구성해보는거라 제대로 하고 있는건지 잘 모르겠네요 ..ㅠㅠ


I don't like these line-breaks between the object and the method being called. I'd put the line-break before the `store` instead of after.


don't keep delete in the URL.

Verbs in a URL are usually a warning sign in a rest api


Alex, porfa, cuida un poco mejor la generación e inclusión de imágenes. La regla es que tienen que ser añadidos en .svg pero también en cada uno de los formatos (16, 22, 32, 48, 64, 128).
En este commit no veo skill-badges.png en 16 ni en 22.
No veo cas.png en .svg ni en 16, 22, 48, 64 o 128.
No veo crons.svg en ningún formato .png
No veo lock-closed.png en 48, 64, 128
No veo lock-open.png en 16, 48, 64, 128.
lock.svg solo existe en svg pero no se si es archivo de trabajo solo o si se usa de algun lado...


@Repository는 transaction 관리를 하기 위한 의도로 추가하였습니다..


谢谢你们~


next.svg y previous.svg ya están?
No veo tamaños 16, 22, 48, 64, 128 tampoco. Esto genera casos raros donde no aparecen los iconos.


Sorry I think my comment was to really clear. I can read Python, I don't need you to explain what does this code. I want to read about the meaning of the "heading '!'".

Maybe something like "Returns a tuple (neg, param), where neg is True iff the param was negative (i.e., starts with a '!') and param is the argument, without the potential starting '!'"


told ya, same problem with the css path


danke


첫줄이군요.


there is no sense to catch exception and re-throw it without re-creating another type of exception or at least adding some extra specific information.


Good to know, i'll try asap!


@Repository 말구 FirstRepository라는 annotation을 새로 만드셨길래요... FirstRepository 어노테이션이 왜 필요한건지 잘 모르겠어서...


dfrgdgdgf


one-line method:
return myQueue.Values.Any(x => x.Contains(item.Item2));


cvsdaf


Falta lp_calendar_event.png en tamaño 16.


cvxbdgd


Hi Chacha

Thanks for the comment

As attach in the screenshot below is the 'routing errors' that i mention, this happens when the users click on the image icon in the ckeditor, either they browser server or go to upload -> choose file, this is the 'routing errors' that will appear.

Do let me know if you can't replicate the issue, maybe I can have a look to learn something from it.

![upload1](https://cloud.githubusercontent.com/assets/7512904/9757463/feca8c70-570a-11e5-89f9-dd6404c7e944.png)
![screen shot 2015-09-09 at 3 52 58 pm](https://cloud.githubusercontent.com/assets/7512904/9757464/fecbfbbe-570a-11e5-804e-3bd07f228bf1.png)


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


I was thinking about that but I think it still might come in useful.


I quite like the idea of pushing it into a separate function, to keep it all a little more readable.

Personally, I've never liked doing assignments and tests inside a single if() - I find that it makes things a little less readable, as two things are happening at once...


@reylibutan mana ni imoa tanan?


4: M120 missing empty line before bullet
4: M121 indentation of two spaces expected


we should probably check status of the response


Extra empty line detected at class body end.


Surrounding space missing in default value assignment.


Line is too long. [112/100]


Line is too long. [101/100]


hahaha


Moved to the top so that `check` and `_checkRecord` are together, and all private methods too.


Thx, I learnt about [octothorps](http://www.thefreedictionary.com/octothorps) *and* [avoirdupois](http://www.thefreedictionary.com/avoirdupois), which, from a French perspective, is both hilarious and mistyped :capital_abcd:


Having another look through the OIIO code, there's a "jpeg2000:CompressionMode" that I missed, and should support. No idea what the options are right now, but I'll do some digging.


**Eslint**: 'return' outside of function


**valid-jsdoc**: Missing JSDoc @returns for function.


Obviously. But periods are not propagated by the current client implementations, so let it go, let it go…


close #85


I remember you said that it was possible that the file format would, if it was expression-based, sometimes not be able to give the full name until you're in execute(). Maybe I misunderstood, but that was what this was meant to cover.


Which commit are you on and wat did you try to build? I'll see if I can reproduce as well.


* which status do you want in case of error, "FAIL" ?
* we should have a placeholder view in case of error but for now we can use an error message


I'm pretty sure that I meant to say "File sequences with as a placeholder for the frame numbers." That's pretty clear, right?

(In other words, yes, accidental deletion)


아아..git을 처음써봐서 어떤 라인인지 제대로 파악을 못했네요 ..ㅠㅠ
매퍼 스캔을 위해서 추가하였습니다.


The MANIFEST.MF also contains the execution env?


Ok bin durch, bis auf das mit der Datenbank.


![malayladu - discuss](https://cloud.githubusercontent.com/assets/708174/9757743/ff7b7d1a-56ff-11e5-89f2-81f9ef6bceeb.png)


Yup, I'll look at doing that. Some file formats may not pass the information of what compression they were created with through as metadata. I'll check what I'm able to tell, though. It may be tricky to keep all formats running through the same larger method, but I'll see what I can do.


Excellent.


I am not sure I understand what the diff does.  del should also remove the entry?  Also, e is guaranteed to exist AFAICS?


Исправил ошибки при сохранении файлов во время создания заявки.


Just an idea, why not store them in an array and do a  ```flash.now[:error] = errors.join("\n")``` in the end?


Using System.out for logging is a bad practice. Use Logger instead


@majianpeng sorry for bringing this up again, iirc, i put the my concerns that this is not likely happen. say concrete file plus checksum. but you explained that when times go by, the holes in the image are filled. so there are good chance that this object is transformed into a concrete file (as opposed to sparse file) after a while. and deep scrub will help set the checksum if we checksum is invalidated due to random write.

could you put some comments/notes here to elaborate the rationale a little bit? as verifying the checksum of a sparse file can incur some doubts.

thanks.


這邊用大寫主要是為了當載入時用 sequlize 查出來時，將會是大寫的 Images 這樣就不需要再進行轉換


I won't hold up the merge but I prefer the variable 'atable' here
(The fact that it's a component table here isn't a big deal, it's a Table object)


I removed this method as it was only used here and I found it made the code clearer to do so.


Better try out in master directly. Otherwise it will be harder to run the demo.


okey...I will follow up that in the next PR


On accepte des chaînes de chiffres arbitrairement longues ? Elles ne doivent pas commencer par 0 ? Est-ce que `\d+` ne serait pas plus simple que `[1-9]\d*` ?


What about non-php files. They could still be views in custom system. For example, I have a markdown engine.


No, e is not guaranteed to be in environ (anymore).


You could usefully have:
field = atable.info_id
field.default =
field.readable = field.readable =


generally prefer the dotted notation where possible (only use alternate where it's a variable)
rtable.is_template.default


Nice little enhancement, but some useful cleanups to do pre-merge :)


What's the purpose of all-items-count?


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


merging basho/riak_cs/feature/improve-access-stats-test = b9729016 into borshop-integration-1239-feature/improve-access-stats-test


basho/riak_cs/feature/improve-access-stats-test = b9729016 merged ok, testing candidate = 38d65276


like any Eclipse plugin project.


Should use PRId64 instead of %d


Now file holes are supported:
* it should be stated in the commit message. I guess it is valuable to indicate that "holes" are not optimized with respect to the used space disk as the blocks made of "0" are really stored.
* test 6010 should be back


May be we could optimize by first filling the last block before adding new blocks. But it can be done in another PR.


In case of error, we may have update few blocks, but not all of them. Should we revert back the file in the original state?


恩 努力


This should point to the commonjs file


Set toolchain options to have dynamic linking. Seems required. Fails on my end because you set in $CRAYPE_LINK_TYPE=dynamic  out side of EasyBuild in your environment and on my end this is not the case.


Perhaps we could reorder the plugs so that EXR is first, DPX second and TIFF third? That'd make the UI a bit more useable by putting the common options at the top until I get my side of the activators sorted so we can hide the invalid options.


You can actually use `->make()`, then attach everything as a relation, avoiding the need for a manual ID.

As it stands right now, there is a slim chance this could cause a key collision.

Redo the entire section to this:

```php
factory(App\Models\User::class, 5)
    ->create()
    ->each(function (App\Models\User $user) {
        $set = factory(App\Models\BeatmapSet::class)->make();
        $maps = factory(App\Models\Beatmap::class, 4)->make();
        $set->beatmaps()->add($maps);
        $user->sets()->save($set);
    });
```


interesting -- what can remove it between `for e in env.keys()` and `del env(e)`?  Is there another thread active?  system-wise I would have considered the env to be immutable.  I don't doubt that you saw that happening, just trying to understand...


I feel we should log something (for example if getDocument() fails because of a DB problem, it should be reported).


never mind... - multiple patterns may match the same e...


An error log might be better here since it's really not the kind of thing that is supposed to happen and in that case we would love to get details on the cause.


Because this is a public facing Scott Logic repo, there is a set format in which the README must appear. @ColinEberhardt can confirm.


Its even simpler. The entries in the two launch methods are generally duplicates.


There seems to be a bit of inconsistent formatting in here :

- Some acronyms are all caps (DPX) and some are not (Tiff). All caps would seem most in keeping with the rest of the UI and codebase.
- Some presets have initial capitalisation (Float) whereas others don't (float). Initial capitalisation would be most in keeping with the rest of our presets.


My comment was meant as an answer to  the TODO.


Invalid file path.


that would assume what we generate always has the same execution environment as what we use to generate it.


:))
IDEA их открыла на ура!

9 сентября 2015 г., 11:51 пользователь Slava Semushin <
notifications@github.com> написал:

> Я не знаю, что вы там перемудрили, но история про вытаскивание изменений
> из class-файлов впечатлила. Не проще ли было написать все сначала, чем
> декомпилировать class-файлы?
>
> :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/91f8bd84000a3c2457bdf6316654d14d75f52c81#commitcomment-13130516>
> .
>

--
Всего наилучшего, Пархоменко Александр


Need to add more clicks


Das sollte wohl eher ins Unit-File von fastd, dass er beim Runterfahren seinen Socket wegräumt. Und was der Pfad zum Socket ist müsste aus der Config ausgelesen werden.


Yup - Tiff should be TIFF. Others are attempting to be what they are named as.


break? :)


Sure - sounds good to me.


@clinta nevermind my other question.  Just saw this.


Hi @mbj is this the intended version for drop of ruby < 2.1? If so, would you please consider bumping the minor version for this? That way, there won't be any problems with Reek's dependencies.


Check ``XWiki#getPropertyClassFromName``. It doesn't log anything. I just took the code from there.


Maybe it would be good to have these in the order that they are called?


Add a proper prefix to the commit message.


The other solution people suggest is re(installing) requests and livestreamer

`pip install --upgrade --force-reinstall requests livestreamer`

Or if it wouldn't work as well, try easy_install

`easy_install --upgrade requests`


A quick note about that change.

`redirect_to @resource` will infer where you want to redirect to, by inspecting the `@resource` and devising what would be the best path to go to. It will fire an HTTP request to that path (GET, POST etc.), so one will actually reach another controller's action. It could be in the same controller or in another, depending on what `@resource` is, but the main idea is that a controller's action will be reached first, and a view will be rendered *as a side-effect* of that.

`render` basically bypasses that process and directly tries to render the specified view. It *may* be dangerous, for a view may have "dependencies" (instance variables set in its matching controller's action). When using `render` like that, the view will have access to the context as set by the action calling `render`, which is most likely not the action matching the rendered view. That opens the possibility for the view to not be fulfilled with the fully-fledged context (in terms of variables) it expects.

Makes sense?


other random word suffixed with question mark


Excellent.


This is great! :fist:


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


@jihoonl
* global_marker_list의 type이 ar_track_alvar_msgs/AlvarMarkers맞나요?
* - name: viz_marker_pose_list(type: visualization_msgs/MarkerArray)는 없어도 되나요?


True that, my bad


mprotect and posix_memalign are independent from getaddrinfo_a


why wrap line ?


Might be good to change this wording because native web components won't set up descendants by this time, which will eventually be the majority of cases.


why not just DynamicLibrary (otherwise should be DynamicallyLoadedLibrary which is really verbose)


Since PRId64 did not work correctly on 32bit system (e.g QEMU), so I give up.
In here, we assume the offset would be much smaller than 2^31


why wrap?


↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑↑
!!!!! !!!!! !!!!! --- Edit HTML Here --- !!!!! !!!!! !!!!!


Why the extra line??? :angel:


include last


Marco, I see that the FF tests are getting auto-approved. If so, I hope you intend to fix it. Else it is an easy target for a troll.


Sure but XWiki class code is usually not the best inspiration :) It's not like it was a complex thing to do so the question is just if it's relevant or not.


I won't hold-up the merge for this but generally we do:
rows.first().id


would suggest a template so that the cast is done in the symbol access


also could you export `make_summary_fun()`?


您会用 shared_ptr 和 enable_shared_from_this 么喵？


`Math.random()` is potentially repetitive, I'd use `(new Date).getTime()` instead.


auto


Do you mean `compressed_idx < 0` here?


Thanks @joelwilliamsmith - yes this is correct. We do not have specification as such for README file, however, as this is a public project we should describe what it is and who it is for. The current README does that just fine in my opinion.


See [this comment](https://github.com/Animatron/player/pull/327#discussion_r39022743).


Los estilos de academica y blue lagoon son los mismos. Es normal?
Debido a la sirena, sería mejor que academica tenga otro banner y blue lagoon mantenga el actual (aunque un "lagoon" es un lago - agua dulce - y las cirenas viven en agua del mar...)


Waaaaaaaaaaaaaaaaaaaaaait, global entity ids? You shouldn't use global entity ids, isn't? It's a bad stuff which cause conflicts with other mods. Check issue 1880 on Forge issue tracker.


for (auto dynLibrary: _subsystemLibraryHandleList) {
delete dynLibrary;
}

but kevin would suggest uniqptr


why using new?


I added the log. I used warning and not error because the problem may be related to the input class name (e.g. it includes the wiki part and that wiki doesn't exist).


sirenas :tropical_fish:


P.S. I know it was used before :)


In general, looks ok!


Deleted the commenting line


:+1:  my bad


Shouldn't this just be:
````
if (r <= 0)
        return (wanted = false);
````
Cause otherwise, when neither "naked" nor "...=" keys are provided the wanted result check is not cached at all.


This seems to beak `dev_package_deps`:

```
# devtools::install_github("gaborcsardi/disposables")
pkg <- disposables::make_packages(aa = {}, imports = c("rpart", "dplyr"))
devtools::dev_package_deps(file.path(pkg$lib_dir, "aa"))
```

The `rpart` and `dplyr` packages are explicit dependencies of the dummy package `aa`, but not shown.


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


А смысл ее добавлять, если она не используется? Добавил бы вместе с контроллером.


validator -> validators


#4672-AirLoopHVACUnitarySystem-with-heating-coil-but-blank-heating-supply-flow-fields-does-not-heat (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1307 of 1308 tests passed)

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


Before commit_meta_file() is executed, the file size would not be updated.
This is an atomic operation, but if something wrong with do_block_operation(), the new created blocks would become garbage and leave in file system.
We might need to design a garbage collection mechanism to deal with it.


idiot


Wouldn't be better to take args instead of tuple?


Add the username, etc in here. Remember that anything you specify manually in `->create([ ... ])` will override these values.

```php
$name = $faker->userName;
    return [
        'username' => $name,
        'username_clean' => $name,
        'email' => $faker->email,
        'user_password' => bcrypt(str_random(25)),
        'user_ip' => '127.0.0.1',
        'user_twitter' => $faker->userName,
        'user_website' => $faker->url,
        'user_lastfm' => $faker->userName,
        'user_msnm' => $faker->userName,
        'user_from' => $faker->city,
        'country_acronym' => $faker->countryCode,
        'remember_token' => str_random(10),
        'user_sig' => $faker->text(300),
    ];
```

You can also give types of users, and so on (which we'll need):

```php
$factory->define(App\Models\User::class, 'subscriber', function (Faker\Generator $faker) use ($factory) {
    $raw = $factory->raw(App\Models\User::class);
    return array_merge($raw, ["osu_subscriber" => 1]);
});
```


Я продолжаю считать, что коммит этот бесполезен, т.к. эти сообщения никто не использует.


Ok thanks, was just wondering why this isn't working :)


Вообще чаще используют find, а не select, при именовании таких методов..


Map would be nicer:
```python
map(lambda validator: validator.validate(self.request), validators)
```


The library requires Python2 and does not work in python3. I tried using it with python3 by following the instructions and got error: No module named Tables. Then i went on to search through the source code, and found out that code is Python2 and not python3. Kindly update that


where the fuck is the code


Since this commit I always have the message "Database user does not have enough privileges." in the webinstaller. In previous beta/alpha versions of Magento 2, we use something like GRANT ALL PRIVILEGES ON synbshop\_%.* TO 'synbshop'@'localhost' WITH GRANT OPTION, and now the % is not interpreted by this code. So to bypass this we need to put each database in the GRANT (for exemple GRANT ALL PRIVILEGES ON synbshop_faz_magento2merchant1.* TO 'synbshop'@'localhost' WITH GRANT OPTION) but it's not a good solution.

Can you fix it ? or find a workaround ?


> BTW, WinPcap is moving to or has moved to MSBuild,

I saw that `.sln` file. The terribly outdated `Win32\Prj\libpcap.dsp` should be rewritten similarly (no one is using `.dsp` files nowadays). So building libpcap on Windows could as easy as:
```
msbuild -p:Configuration=Release Win32\libpcap.sln
```

But I use my home-brew GNU-make build system here. Mainly to add features to windump as
shown here:
  http://watt-32.net/pcap/#windump


fragile (if this directory's name changes, it will break).

If you really want to add this directory to the includes (which is already more or less implicit even though some people may argue it is not), you can write instead:

    include_directories("${CMAKE_CURRENT_SOURCE_DIR}")

or, after `add_library`:

    target_include_directories(pfw_utility "${CMAKE_CURRENT_SOURCE_DIR}")


I understand. OTOH I think from R this could be achieved with f.ex. getURL("http://www.omegahat.org", httpheader = c(Accept = "text/csv"). And GoldenCheetah as a server should honor such a request.

In that way any http-client can use the API, and that is a Good Thing :-)

(Same thing goes for status-code, GC should use the standard http status codes.)


So it turns out that the jpeg2000 compression mode is a bitmask with 6 options in. It looks very low-level, so I'm very tempted to suggest that we don't support it, at least not initially.


Hm, I didn't actually mean to push this commit to master, w00ps. Let's update the README.

From what I've seen, this inversion must be enabled (as is done by this revert commit) to talk to a gateway, but disabled to talk to another node. Not sure about the RFM95 though. AFAIU it should expose an identical interface as the SX1272 (in fact contains the same chip, but rebranded I've heard). No time to look at the datasheet now, though.


this assertion currently fails, it seems the phpunit bridge takes over too aggressively here, even though I registered the handler as @nicolas-grekas suggested...


What are those file for?


Не, это не URL это какой-то недоюрл.

Используй тут нотацию как в RequestMapping-е. (И см. ниже комментарии).


Can you ignore them? Otherwise we will end up with all editors preferences...


Вместо этого используй `UriComponentsBuilder` из Spring-а.


Здесь используй просто константу.


rather use a logger (see other classes) instead of a print stack trace


Ok. I will do it. ;)


Will update the commit message to mention we have file hole support in this patch.
By the way, I also send a PR https://github.com/OP-TEE/optee_test/pull/27 to test this patch including extending file size and file hole.
How to get back 6010? I didn't see the test item in Github optee_test.


Здесь, нужно использовать вложенные `spring:param` или что-то типа такого для формирования URL-а.


That's one useless line of code here!


Thanks.. this fixed my problem. I used to get 'MOVED' exception for every SET/GET operation...


Should maybe leave this out and rely on a `Post` relation.


how about 3.7 ? ;)


Здесь уместо вынести метод `ParameterSource buildParameterSource(Person person)`, который бы и мапу строил и `MapSqlParameterSource` создавал/возвращал.


`str_replace(" ", ",", $faker->words(7))` could be slightly more elegant


Во-первых, не забывай ставить `{` и `}`


`$faker->boolean()`


Во-вторых, не надо возвращать `null`. Если уж и возвращать, то `Optional<Person>` (новая штука в восьмерке).

Но. Тебе надо сделать совсем иначе. Там есть метод, который тебе находит ровно одну запись -- используй его. Он сам бросает исключение, если а) не найдет ничего или б) найдет больше одного


This should also use $faker, so that the same seed gets the same result regardless. It's a requirement for testing.


Better is to return `null` in such cases.


The `GraphDatabaseService` type and the `graphDatabaseService` identifier should be on the same line.


The ```toluabind.c``` is now being treated as a "generated" source file, although it is being checked in into git to solve the chicken-and-egg issue to build ```tolua++``` tool. In this commit you have checked in a new version of the ```toluabind.c``` but without the accompanying changes in the ```lua/compat-5.1.lua```. It is either you have included this file accidentally or you need to check in your changes in ```lua/compat-5.1.lua``` as well, or otherwise another round of generation of this source file would revert back your changes.


Good call. Thanks.


It's needed either here or in ImageWriterBinding.cpp. It seemed more appropriate to put it here, as ImageWriterBinding never directly references a StringPlug, but won't compile without it.


+1 to question


[look yourself](https://github.com/monitoring-plugins/monitoring-plugins/blob/v2.1.1/plugins-scripts/utils.pm.in#L28), there is no such exit code defined as `$ERRORS{'FAILURE'}`


@wkennington This attribute name ("syncthing") does not match the package name ("go1.4-syncthing-v0.11.24"), so "nix-env -i syncthing" won't work as expected.

Also note that "v0.11.24" is not a valid version string because it doesn't start with a digit. So it breaks "nix-env -u".

Also, what are these `outputs = ...` hacks for? Looks rather ugly to me.


@JeroenSormani Running latest WP? All the CSS pulled in correctly? I'm not seeing it.


What does these lines mean?
``` C++
std::string whiteLine {};
getline(weaponFile, whiteLine); //white line
```
By the way, you forgot to add `std::` to `getline(weaponFile, whiteLine); //white line`.


Workaround for: FreeRDP/FreeRDP#2876


必ずしも画像は三枚とは限らないので、3.timesはいらないですね


@jfoug
> So after this change, the -test-full=0 should not issue a PASS for formats which should or should not have the FMT_SPLIT bit, with no 'white-list?'

When we find a FMT_SPLIT error, I did not change the result to "ERROR". **I will do this**, thanks for reminding.


Алсо, в `KeyEditListener.getEditMarkup` добавь кнопкам `class="de-button"`, чтоб тоже были в общем стиле.
А текст-инпутам сделай `size="19"` - тогда они по ширине станут равны кнопкам и вместе с ними будут выглядеть единым блоком. `size="26"` это просто пц какой-то, куда им столько.


library(plyr)
Medcount<-count(medsOut[1:486941,],vars="encounter_id")
head(Medcount)
tail(Medcount)
plot(Medcount)
library(sqldf)
join_stringmedcount<-"select charges_all.*, Medcount.freq from charges_all left join Medcount on charges_all.encounter_id=Medcount.encounter_id"
charges_all_join_Medcount<-sqldf(join_stringmedcount,stringsAsFactors=FALSE)
head(charges_all_join_Medcount)
tail(charges_all_join_Medcount)
charges_all_join_Medcount$freq[is.na(charges_all_join_Medcount$freq)] <- 0 #replace na's with 0
medfreq<-ddply(charges_all_join_Medcount,.(patient_id),summarise,medfreq=sum(freq))
head(medfreq)
length(unique(medfreq$patient_id))
patient_matrix<-merge(medfreq, patient_matrix,by.x=c("patient_id"),by.y=c("patient_id"), byall.x=TRUE)
head(patient_matrix)
#lab count
labcount<-count(labs[1:10153073,],vars="encounter_id")
head(labcount)
tail(labcount)
join_stringlabcount<-"select charges_all.*, labcount.freq from charges_all left join labcount on charges_all.encounter_id=labcount.encounter_id"
charges_all_join_labcount<-sqldf(join_stringlabcount,stringsAsFactors=FALSE)
head(charges_all_join_labcount)
tail(charges_all_join_labcount)
charges_all_join_labcount$freq[is.na(charges_all_join_labcount$freq)] <- 0 #replace na's with 0

labfreq<-ddply(charges_all_join_labcount,.(patient_id),summarise,labfreq=sum(freq))
head(labfreq)
length(unique(labfreq$patient_id))
patient_matrix<-merge(labfreq, patient_matrix,by.x=c("patient_id"),by.y=c("patient_id"), byall.x=TRUE)
head(patient_matrix)

#icd9 count
#GrahamOut <- read.csv("/project/msca/data/uchicagomed/Capstone_\ New\ Data/Graham_out_ALL_deID.csv", sep="|",header=T, fill=T)

#head(GrahamOut)
#GrahamOut$ICD_9_CM_Diagnosis_Principal <- 1
#ICD9freq<-ddply(GrahamOut,.(PATIENT_ID),summarise,ICD9freq=sum(ICD_9_CM_Diagnosis_Principal))
#head(ICD9freq)
#ICD9freq$patient_id<-ICD9freq$PATIENT_ID

#ICD9COUNTMATRIX<-merge(icd9Dx, GrahamOut,by.x=c("patient_id", "encounter_id"),by.y=c("patient_id", "encounter_id"), byall.x=TRUE)
#head(ICD9COUNTMATRIX)
save(patient_matrix, file="/project/msca/capstone3/patient_matrixrevise.RData")

str(patient_matrix)
lmdata<-patient_matrix

lm<-lm(y2_charges~y1_charges+labfreq+medfreq+sumlos+encountfreq+hospitalizations+num_ER_visits, data=lmdata)
summary(lm)
lm2<-lm(y2_charges~num_ER_visits, data=lmdata)

#Visual relationship of medfreq to charges
#str(charges_all_join_Medcount)
#charges_all_join_Medcount$Charges<-as.numeric(charges_all_join_Medcount$Charges)
#reg1<-lm(Charges~freq,data=charges_all_join_Medcount, na.rm=T)
#plot(charges_all_join_Medcount$freq, charges_all_join_Medcount$Charges)
#abline(reg1)
#plot(charges_all_join_Medcount$Charges,charges_all_join_Medcount$freq)


This PR still uses a project-local ESLint, it just stops using an old one pulled in by grunt-eslint. So we should be fine there.


HANDSOME


Issue4530UpdateIDDandIORefDOAToZonalUnits (wfbuhl) - Win64-Windows-7-VisualStudio-12: OK (1839 of 1839 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)


Yeah, I can see that.  I figured it'd be a special flag, but we should probably just always preserve the actual `jldoctest`.  Simplifies things and makes it consistent.


This will become Wallet, and the old Wallet will become Android Pay


1、这种情况下，使用
```
cat << EOF
现在，我们可以开始安装浏览器了：我们当前提供有firefox、opera和chromium

请注意，chromium浏览器使用的pepper-flash插件需要yaourt才能安装

EOF
```
代替echo。

2、你直接这么改，不想想用户如果不安装yaourt的话（我就不喜欢用）该怎么办呢？另外你说是chromium需要 flashplugin、还是fx需要chromium-pepper-flash？你这样做的话是不是甚至用户选择不安装浏览器都会装上两个flash？


VS-equipment-issues (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1310 of 1310 tests passed)

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


Yeah, that definitely makes me feel better.


Why it's enabled by default for `opengl-hq`? While it can improve visual quality for some content, it still changed the image significantly (the lost details, added noises). Not mention that it can slow down the playback of down scaled 4k movies (it's rare though). Essentially, it's a video filter implemented as source shader, which might not be necessary in all cases (etc. film grains)


You do know that if someone managed to insert a harmful ModuleScript, the module would return unsandboxed data, right?


Thanks for pointing it out! I've become a bit complacent w/ the CSP to catch fails. :)


另一个问题，即使用户为了一个flash去装yaourt、可是yaourt的安装选择在前面……


This commit is "breaking" the plugin GlobalTabList. Even if configured to be global the list shows players from current server only. Maybe @CodeCrafter47 could fix it somehow?


Scala is [licensed under the BSD-3 license](https://github.com/scala/scala/blob/2.11.x/doc/LICENSE.md) -- did you mean to use that one instead of Apache? IANAL, but I'd be surprised if you could just relicense like this.


"/tmp" -> "" since "/tmp" is the default.


"NewSoftTao(\"\", nil) = %v; want no error"


thumbs up


@sanmiguel: This change is a hack to temporarily set some `inet` options. Would it make sense to pass options to `websocket_client` generally?


I would structure this code as follows:

1) take "probePage" and a page from the pagesIndex
2) make the larger page the actualBuildPage and the smaller one the actualProbePage (choose better names) and remember which is which (something like probeOnLeft)
3) outputProbePage = createProbePage(actualProbePage, probePosition);
4) return buildOutputPage(outputProbePage, actualBuildPage) (creates new page based on probeOnLeft)

This way you will only have one copy of this complex code.


Сюда имхо стоит добавить `border-radius: 2px;`, а то слишком остро это всё.


maybe use an Iterator for this?  (try it out and see if it is simpler)


The following three variables are in the global scope and are mutable. Since they have a special purpose, I would suggest adding comments and briefly justify their scope.


No dramas mate :) Easy fix.


This was the source of many non-convergence issues. By running the problem arouse when eventually `sk` and `yk` converged to 0. For the case of `theta = -1` this usually implies that the algorithm converged towards the boundary. These estimated close to the boundary could be used but they are certainly not a stationary point so one must proceed with caution.

For the general case, of `theta`, we must be more careful because we might have either of the two issues:
1. We could converge very close to the boundary similar to the case of `theta = -1`.
2. The function may be discontinuous, for instance when `theta = -1/3` the Cressie Ried function takes the following shape:
![image](https://cloud.githubusercontent.com/assets/6549178/9777161/73062608-5715-11e5-9ef3-b1a372c910e4.png)

In general care must be taken to ensure the function is concave, for instance the function may converge for `theta = 1/2` however the CR function (and consequently the objective function is not, the critical is an inflection point).


Please add a comment.


Is this another spot that'll need to be updated when there are new releases? Might be good to document or automate (by pulling the info from the top level `package.json`)


I feel like this comment might make the code more confusing. The important part of the code isn't that we're getting a numerical value, but that we're getting a value that is at least 1. I think it's also confusing in that the value will always be a string rather than a number (albeit a numerical string).

I think this would be less confusing if we just said something like "make sure value is at least 1".


The function now returns the estimated Inverse Hessian which can be used for de-bugging issues with convergence.


I think this corresponds to `ReflectionInfo`? If so, would a better name be `InjectableInfoModel`?


This is unlikely to work on the TV.  We haven't added any support to the driver for fetching DOM nodes from the TV.  If you can find a good workaround for the TV, please proceed.  Otherwise, you should probably test this in the browser.


Looking throughout this PR, it seems like there are a bunch of places where we'd need to update the version number and URL of angular.


Would be useful to have an INFO level message, so it is easier to debug (e.g. Travis failures).


develop (rpg777) - i386-Windows-7-VisualStudio-12: OK (2319 of 2337 tests passed)

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


mind your indentation


Added quotes around the '50' as postgres will throw an error message otherwise ("HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts.")


class names must be PascalCased


def


Would be good to have 2-space tabs in this file to be consistent with the rest of the code.


Note that this reverts the changes to audio_policy.conf and system_prop.mk that were done in d14e3a4cd91c3d1955d5c92ed0bd5fccf91b063c

I should really have called that Fix Camera and initial work on audio...


2-space tabs


Very visually appealing code.


VS-equipment-issues (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1830 of 1837 tests passed)

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


I might be missing something. The CacheStorage container that holds CacheEntry instances is all for managing to on-disk cache, so GLib's GCache and GHashTable don't fit here.


If this has a code block, should we also be indenting it this deeply? Should the fragments be normalized so there isn't extra indentation on each line? (maybe this belongs in a follow up PR, this is just something I noticed)


That field makes a ton on sense in the serialized GVariants, but I realized it didn't make sense in the in-memory struct.

For example, this change introduced a new version because of the change in how hash arguments are stored. When an old cache entry is read, it's automatically upgraded from version 1 to 2. The in-memory struct is always the same regardless of version. It just affects the CacheEntryUnserialize step which needs to understand previous cache entry versions.


Are we using JSPM now? Just curious, I think it's fine to have this in here regardless.


formatting: change tab char to 2-space tab


That would work fine if the args value wasn't a union. What type self->args actually represents depends on the algorithm. In this specific case, it would work if salt is always the first item, but it would stop working if other algorithms with other args were added.


tarea 1 ejercicio 1 3


Yeah I wrestled with that. For now, I'd like to keep the old pattern. I'm just switching up the ordering of a lot of statements and adding some new similar ones.


gotcha! ;)


Clean and simple like it should be.  Nice program! However, I feel like keyboard.NextInt() is a safer option in this situation.


It seems invalid headers have increased. Does repoman not check them anymore?


Novice error. Happened when I renamed 2 directories to each other (api/api2). Obviously, the password's been changed again...


Changing this to `eachSeries` might help with throttling


@gravyboat can i merge my own PR?


@miigotu
Not 100% sure if it is related to this commit.
Gave develop branch a go.
Freshon works fine.
But IPT says that the URL is not found even though its definitely correct
https://gist.github.com/flightlevel/65cda84a58f88c141b40


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


I just added the lint property to npm script in my package.json, ran `npm run-script lint` and got

```
0 info it worked if it ends with ok
1 verbose cli [ 'C:\\Program Files\\nodejs\\\\node.exe',
1 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'run-script',
1 verbose cli   'lint' ]
2 info using npm@2.11.3
3 info using node@v0.12.7
4 verbose run-script [ 'prelint', 'lint', 'postlint' ]
5 info prelint @blinkmobile/bic-jqm@4.1.2
6 info lint @blinkmobile/bic-jqm@4.1.2
7 verbose unsafe-perm in lifecycle true
8 info @blinkmobile/bic-jqm@4.1.2 Failed to exec lint script
9 verbose stack Error: @blinkmobile/bic-jqm@4.1.2 lint: `eslint .`
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:213:16)
9 verbose stack     at EventEmitter.emit (events.js:110:17)
9 verbose stack     at ChildProcess.<anonymous> (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
9 verbose stack     at ChildProcess.emit (events.js:110:17)
9 verbose stack     at maybeClose (child_process.js:1015:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
10 verbose pkgid @blinkmobile/bic-jqm@4.1.2
11 verbose cwd C:\src\bic-jqm
12 error Windows_NT 6.3.9600
13 error argv "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run-script" "lint"
14 error node v0.12.7
15 error npm  v2.11.3
16 error code ELIFECYCLE
17 error @blinkmobile/bic-jqm@4.1.2 lint: `eslint .`
17 error Exit status 1
18 error Failed at the @blinkmobile/bic-jqm@4.1.2 lint script 'eslint .'.
18 error This is most likely a problem with the @blinkmobile/bic-jqm package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     eslint .
18 error You can get their info via:
18 error     npm owner ls @blinkmobile/bic-jqm
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]
```

I assume its working for you, probably just a windows thing.

:(


Good point. I think I'll leave it because I do want to make the algorithm configurable which makes the hash size dynamic. I decided to ignore that feature for now, but I'd like to get to it soon.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


15% faster

https://gist.github.com/anonymous/6b4acae4159a49a34fa0


Note that this was done on a different machine (my home desktop with a ssd) from the original benchmarking (my work laptop with a ssd)


r+


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


merging MonsieurNicolas/stellar-core/assetIsValid = 6f20f8a5 into auto


MonsieurNicolas/stellar-core/assetIsValid = 6f20f8a5 merged ok, testing candidate = b97bbdc4


Could do, but it's in the `spring-boot-parent` project and there's not a lot in that folder.


I was going to do that....


congrats :clap:  :clap:  :clap:  :clap:  :clap:  :clap:  :clap:  :clap:  :clap:  :clap:  :clap:


Haha oh yeah! Completely untested though I am going to try it this weekend.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1309 of 1309 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)


띄어 쓰기를 일부 수정합니다. 그리고 '컴파일 타임'을 '컴파일 시점'으로 수정합니다.


Done.


중복되는 '때문에'를 삭제합니다.


일부 단어의 띄어쓰기를 수정합니다.


Atualizado.


newline at end of file.


띄어쓰기와 표현을 다듬었습니다.


Some file may have no update or change in this year, but its ok we can update for all as we are doing open source


That sounds better, having diagnostics rather than too much background magic. I will work on that.


Ok, sounds good.


there are a few of us watching, we can all merge your requests


No, this is just me thinking "Cache... I'm sure GLib has one of those!". Sounds like the GLib one and GHashTable don't apply, so sounds good.


띄어쓰기를 수정했습니다.'최적화를 할 수 있다'는 '최적화를 구현할 수 있다'로 목적어에 맞는 서술어를 추가해 수정했습니다.


Watch out for smart quotes sneaking in.


the input tag for the url does not render properly unless 'http://' is included on the url.
here is a solution for user experience. instead of leaving value=" ", include the http://.


approved


It does work for me here.

So, at least on OS X / Linux, project-local modules with CLI entry points (like `eslint`) can be accessed as I have via package.json scripts.

I wonder if we need to swap it for `$(npm bin)/eslint .` instead? This also works on OS / Linux, and maybe it works in Windows, too?


Arrgh. I will have to check both specs for those.


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


what is get_user here?


wlanpub包含云平台公共属性和方法, 如MQ的连接、收发操作,mimeType,serviceName等常量定义

## 使用方法
首先加载wlanpub库：
	var wlanpub = require('wlanpub');
	然后若要使用MQ相关方法属性可以使用wlanpub.mqoper子模块，系统调用接口使用wlanpub.wsys子模块

### mqoper.sendMsg(pkey, msg, pubOption)
    `sendMsg`方法用于webserver端发送消息到MQ。
    `pkey`为云平台业务模块名。`msg`为http request解析出的头体内容, 消息格式为{ 'head':'', 'url':'', 'method':'', 'body':'' }。
    `pubOption`为publish发布参数，目前只用到{ 'replyTo' : '', 'correlationId'  : '1' }。

### mqoper.replyMsg(pkey, msg, pubOption)
    `replyMsg`方法用于云平台业务端处理收到的http request后需回应消息到MQ。
    `pkey`为收到消息中的deliveryInfo.replyTo。`msg`为业务端需要回应的消息内容, 消息格式自定义。
    `pubOption`使用收到消息中的deliveryInfo.correlationId，即格式为{'correlationId' : value},其中value=deliveryInfo.correlationId。
    `deliverInfo`来源具体参考方法`connectReadyForService`中的介绍。

### mqoper.connectReadyForService(serviceName, recvCallback)
    `connectReadyForService`方法用于云平台业务连接MQServer并向业务队列订阅消息, `recvCallback`为订阅消息的回调函数，
    业务端需要在该回调函数中处理订阅到的消息，并通过`replyMsg`方法应答消息。
    `serviceName`为云平台业务模块名，需确保是约定好的。
    `recvCallback`函数原型约定如：
    function(message, header, deliveryInfo) {
        if (message.data) {
            var msg = message.data.toString();

            /* process received msg... */

            /* assemble your reply msg... */
            /* reply msg through MQ: 其中value=deliveryInfo.correlationId */
            replyMsg(deliveryInfo.replyTo, yourMsg, {'correlationId' : value});
        }
    };

### mqoper.publishOption
    `publishOption`属性仅webserver端会用到，业务端参考。用于定义发布消息时要采用的选项，目前只用到replyTo和correlationId。
    后续变更参考该属性值。

### mqoper.mimeType
    `mimeType`属性业务端维护，webserver端使用。用于定义MIME类型，具体见`util/wconf.js`内容。
    若有新增类型，需要更新该属性即wconf.js文件。

### wsys.getLocalIP(iftype, family)
    `getLocalIP`方法用于获取指定网卡类型的ipv4或ipv6地址，若指定类型有多个接口默认取第一个的地址。
    `iftype`为网卡类型，如eth, lo, tunl。`family`为地址协议类型，IPv4 or IPv6。
    举例：wsys.getLocalIP(eth, IPv4)。

### wsys.mkdirsSync(dirpath, mode)
    `mkdirsSync`方法用于同步创建多级目录，其异步版见`mkdirs(dirpath, mode)`。
    `dirpath`为待创建的多级目录名。`mode`为目录权限设置如0666、0777。

### wsys.mkdirs(dirpath, mode, callback)
    `mkdirs`方法用于异步创建多级目录，其同步版见`mkdirs(dirpath, mode)`。
    `dirpath`为待创建的多级目录名。`mode`为目录权限设置如0666、0777。
    `callback`为创建完后的回调函数。

### wsys.trimSpaces(s)
    `trimSpaces`方法用于将字符串中多个空格转换成一个空格，主要用于格式化行输出获取指定栏目的值。
    举例：tmp = trim(stdout).split(' '), 然后tmp是个数组对象。

### 主要事项
    `util/mqpara.js`中`hostNames`需要本更新成与本地系统MQServer的hostname一致。


Thank you


To be clearer in all of these, I think the final sentence should be revised to say "...of the *client's* request." This is because a) prior mentions of what the client does say "access attempt" and not "request", and b) some of the notes now talk about the resource server's interim request in order to give context for why the resource server suddenly cares about any of this.


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


I do not completely understand the TThreadedSelectorServer. Could you tell me how many helper threads are being used by the selector ? Or did I misunderstand this facility.


Sim, eu já tinha feito isso @Schiffers, só esqueci de subir para a versão atual.

Feito!
https://github.com/otsolutions/Vanaheim/commit/b8c68a872874b3892df271529aa459d90dcad9ab

Obrigado :)


Agreed, it's the client's request.


fast-forwarding master to auto = b97bbdc4


I need your name in the form used for other names. The git specification requires a valid email address.


Why not just say "the issuer URI" here? That's defined in §1.4 and it's unambiguous.


FWIW, SET STATISTICS IIRC only actually accepts a 32bit value.


Ah, I see. I didn't notice args was a union. The style guide doesn't actually mention unions specifically, but from what I can find they seems to be strongly discouraged. This is a case where it could pretty easily be avoided at the cost of using just a little more memory. There's also the option of turning the "basic" and "salt" members into pointers, but that seems like overkill.


Done. Updated configure.ac and Makefile.am to store those macros in autotools/m4.


In the interest of cleanliness, ditch these commented-out var_dumps.


Fixing in my next commit. I found a small issue.


I've addressed this in 5e3c2e02a0457deb96c2f3eb10a844029273512e, using `iconv` by default, but providing a simple `iconv.py` which can be used in case `iconv` isn't available.


102687702-Schedule-Unit-Tests (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2294 of 2346 tests passed)

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


No user rr-cirkits contributed to the JMRI SVN code. Please update only the username you used at SourceForge (if you had not made an SVN commit to the JMRI SVN code, you can ignore this file entirely--it's only being used for the SourceForge->GitHub migration).


I, however, am not github, yet I too roar.

On Sat, Aug 29, 2015 at 10:11 PM 1425 <notifications@github.com> wrote:

> I am github. Hear me roar.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/errorcodexero/fiddle/commit/0aafdf2d897c74812966df53f479c59277c39fff#commitcomment-12964028>
> .
>


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


please don't uncomment this.


#4402-VRF-outputs-and-multiplied-floor (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1840 of 1840 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)


instead of needing a subshell, could this be `command which xz 2>&1 > /dev/null && nvm_is_merged_node_version "$1"`


i'd prefer to explicitly cover gz and xz here rather than using `*`


Done


This is my version to fix issue #7


baw que legal isso


Add original notices (author, copyright, license)


←_←


102687702-Schedule-Unit-Tests (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Some people might complain that the **500** argument in the **singleShot()** call should be a wordy thing like **DELAY_TO_UPDATE_DISPLAY_MODE**, however in this case it is trivially obvious to any reasonable programmer exactly what that 500 is for... so I like it the way it is.  However this whole thing looks like a hack to that naive new developer --> it deserves a comment about how it is for preventing a hang when changing display modes.


Initializing file for logging errors on heroku.


VS-equipment-issues (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1830 of 1837 tests passed)

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


I think the idea is that you also check in the m4 macros that autotools puts there. Like dbus does in their git tree: <http://cgit.freedesktop.org/dbus/dbus/tree/m4>


I agree. the pull request must have been some leftovers from the previous commit df13822 by the same author from the Sep 1st. The original commit has been merged by josh a few days ago. This merge should be reverted.


Understood. It is an acquired taste. Does look better after staringgg at it
for a while thanks!
On Wed, Sep 9, 2015 at 6:56 PM Chris Guthrie <notifications@github.com>
wrote:

> Yeah did that on purpose. Thought it looked better and less cramped. Could
> argue against it though
>
> Sent from my iPhone
>
> > On Sep 9, 2015, at 11:56 AM, erfangc <notifications@github.com> wrote:
> >
> > this was intentional?
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/erfangc/DecoratedStockChart/commit/75b581ee32e57f11c2dc54c80ddbb36076b74bc5#commitcomment-13148266>
> .
>


Fixed : https://github.com/Mei152/open_todo/commit/858d37a5ba1690da46916ae8e2db41350548d0d0


It's enabled by default because debanding virtually always improves the result with practically no noticeable distortion or degradation of the image.

Perhaps the default settings could be a bit less aggressive, though. (eg. reverting to the values used in the “old” deband shader, the one that people have been using and testing for the longest amount of time)


ahh that makes sense, thanks for your detailed response.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (777 of 777 tests passed)

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


Yeah, as I noted in a comment on the issue, it seems to suffice, but I was sticking pretty close to what you suggested since we hadn't had much opportunity to discuss it. I'm okay with that since the meanings do seem to align perfectly.


@simonmarklar so, assuming the git-hook work is separate from this PR, is this fine to merged? Or does it break things for Windows users?


Why is this look like it is removed. This does not look correct


本地的仓库已经建立


how come 4 for blocks instead of putting all this in a single one?


ha, submitted this as root from polestar, maybe after having a few from left hand...


On 09/09/2015 09:06 PM, Randall Wood wrote:
>
> No user rr-cirkits contributed to the JMRI SVN code. Please update
> only the username you used at SourceForge (if you had not made an SVN
> commit to the JMRI SVN code, you can ignore this file entirely--it's
> only being used for the SourceForge->GitHub migration).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RR-CirKits/contributors/commit/e55981e6bff72678249130e46f4096331eed5947#commitcomment-13149592>.
>
Randall,
rr-cirkits is my SF account. I did not have write access, so Bob or Ken
always applied my code changes. They were mostly icons and decoder files.

Dick :)


valeu


Because the view of button is more suitable than a TextView,
Of course I can change it to CardView,too,if you need.


- Should remove funtion showhasdkey, B/c this function only need on the fist time start app, after get key we no need to use this function again!
-  sharePhotoToFacebook(). text share should be change -_-

- isCheckTimerStop should be change ( flow convention )


You now require('movement') its based off what the plugin announces as not whats in the default_plugins list, the 'move' there is for settings, and should probably be changed to 'movement'.


May need to do some `GLIB_PRIVATE` style magic where the command line gets secret access to hidden shared library bits.


You need to change QuantumFields too.


eh? too many cookies???


The jspm line is prophylactic


your description doesn't sound good about the rt2x00 dual mode. shouldn't we use it more a more stable?
do we really require this patch? who use this dual mode?


why not defer twice?


btnForgot, edtEmail should be change ( please use follow convention )


Go fmt wont align types when they are separated by line break.  This commit fixes this for most of the elements.  We should probably wait for Go to add better handling of line breaks in templates for the rest of the types.

See:

https://github.com/golang/go/issues/9969


i like to explicitly declare the variable so that the IDE doesn't bug me


figuirng out how this works still..


Hey, take a look here: https://github.com/arco/android_device_samsung_lt01-common/commit/c1971937557a836184c9bce1c43e2413023b914a
Both netflix and youtube should work fine with that.

Good luck! :)


I am still getting the same errors.
This is the resource on Stack Overflow I relied on: http://stackoverflow.com/questions/6032134/testing-the-user-model-with-rspec-devise-and-factory-girl.

I don't think the users I'm trying to create are being added to the users database.


Yes


Change


Reboot