This was very lazy and inexcusable of me: it crashes when line === 0 :-(
This caused https://github.com/cben/mathdown/issues/85, fixed in efe3a4ff29c612bef7d359f051dfc041ff496e3c (not yet deployed in mathdown).


always


Model class names should be singular - STKScore.  It is understood that there might be several instances of STKScore objects.


Even in the prototype development (e.g. scientific experiment), sometimes we need high performance, while the global variables are unavoidable. So the best solution is to tell people **not to involve a global variable in any loop**, as I mentioned in #8870.


You want me to implement painting transparent arcs using AWTDemoDonuts.java. Right?


I would like to be sure caret support is part of composer. AFAIR, the last time I have tried to remove it, it was a so old version that it is not supported.


intercept error mess... Seems about right :p


for color picker


Ugh, no that was not intentional... I'm fixing it now


The testsuite requires this patch (guzzle/guzzle3#54) to be installed


c.Assert ?


The version of composer running on the travis container architecture is definitely new enough for the caret operator. I use it. :)


What is the reasoning of using initPrivate here?


Sure, I can do a conditional using `os`.


Looks like this privateInit method was copied out of BNR exercise solutions.


Then, I will remove it. Thanks!


@indutny My understanding is that `now < first._monotonicStartTime` should evaluate to `true` only if `first` was added while processing the list of timers. So I would think the two conditions are equivalent in terms of behavior, or am I missing something?

Do you have an example where the second approach would lead to `Timer.now()` being called more often?

Thank you!


That is not the best solution. Stefan already pointed out what's wrong with your interpretation of the problem with globals. Please reread what he wrote.


@lifthrasiir ah, sorry, I should have checked about whether some of the follow-up commits addressed the casing of `NAN` et al...


I suppose a pull request that would combine all movement fixes from racesow would be perfect.


also, if we add the 'QB_XS' character internally for the qb_perror function, then it looks like the qb_do_extend macro is going to strip off everything after this character. For people who aren't aware of this feature, wouldn't that mean a change in previous behavior?


why are we not using clang standard library?


In the future, large chunks of code that get moved around like this are probably candidates for extracting as a method.


Yes. But, please wait an hour (or so) - hopefully the developer will fix
the example.

Well, I don't expect you to do it using the class.  I hope you will copy
the code/logic, and implement drawing in
DynamicTrackCoverageWrapper.CoverageShape (or something like that).

Ian Mayo
Deep Blue C Technology Ltd
- New technologies, implemented well

On 30 April 2015 at 16:43, Haris Peco <notifications@github.com> wrote:

> You want me to implement painting transparent arcs using
> AWTDemoDonuts.java. Right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pecko/debrief/commit/d8dcf7a4b48097cc5dcb422d7aaed378db2e15a6#commitcomment-10987622>
> .
>


Updated HolderComponent for edit holder details method


Added logic. Could not test yet


lgtm.  :)


This last test would be important to make sure any fix for this issue (which would probably require inspecting previous stackframes) still works when there are two different none-slices on the same line.


Eu acredito que alguns arquivos utils estavam lá...
Mas ok...


Good housekeeping. This could have been done in the previous commit, and would have made it easier to review.


I haven't started to treat the packages for the tupacolypse ;)
I'm waiting for this benchmark to be fast:
https://github.com/SimonDanisch/FixedSizeArrays.jl/commit/21a2c2136169a6b4a512488e0e0ed703de59fa35

You can look at the examples in GLVisulize/test to get a feel for the "new" api. The files in GLVisualize/visualize/ are not documented yet, but probably relatively easy to read. They all boil down to the same principle, of creating a dictionary that can feed the shader with the right data.

I'm not sure yet about the edit area, but I guess it will boil down to using the boundingboxes of some visualization. So for a button, you would make an interactive area for the boundingbox of that button.
But maybe, you will just register a "callback", for the current object under the mouse cursor.
Right now, every pixel on the screen has a group id, which could be used to find the object under the mouse cursor.


good job! 10/10


keren gw cek di $HOME/ForensikBBM


The only change by default would be that it will be printed as "| errno=N" rather than as "(N)". Printing extended info is the default, so it will be stripped only if someone explicitly calls qb_log_ctl() to do so.

Going back to my idea of qb_perror() scanning for QB_XS, it's not as trivial as I thought. Currently qb_perror() uses literal concatentation, i.e. fmt ": %s (%d)". If we want to split the info, it would need to allocate a new format string and copy the pieces. I don't know if we want to add that overhead to it -- what do you think?


Unused variable.


rax retry tests


nice job 10/10


It's just based on what it currently is.  That was based on tests with SMSSync and the old API.


:+1:  Might add something about git ignoring `node_modules`.  Those *should not* be checking into source control right?


@oschreib  @abonas @simon3z  Please review spec related files:
 vmdb/spec/models/ems_refresh/refreshers/k8s_refresher_spec.rb
 vmdb/spec/vcr_cassettes/ems_refresh/refreshers/kubernetes_refresher.yml


> because all shinken-objects attributes are mostly read ..

and for the attributes that are inherited by template thus that would unnecessarily raise/higher the getattr(object, attr) call time.

that's my point ;)

I'll try to do a POC of that in a new branch .. hmm.. sooner or later ;)


Replacing `bool` with `auto` is going _slightly_ overboard, imo.


Awesome.

By the way, what rounding modes does the formatter support?  I've seen plenty of arguments that when doing "round to nearest", one should resolve ties by rounding towards the even value.  It seems here that you have adopted "round ties away from zero", is that right?


I didn't check the caller yet, but if you're hoping to return a list of unmet capabilities to the client, the protocol doesn't support that afaik.


I didn't change anything, it was generated lowercase for me


Still have the shader, yes. As for the camera and other stuff, that's why I didn't tie them to the project. Because I don't/didn't want to step over anything made. Only thing actually "added" to the project is glut since Horia and I use it. Anywho, I'll re add everything as soon as I finish setting up something for IOS


lgtm


Que legal, isso :)

Os caras pensaram em tudo ;)


Thanks a lot for this.

I have moved to Zabbix si Will not have time to develop this Cacti image in the future.


I don't think you need to explicitly construct  a list here.


, if defined,


great job, 10/10 and will account for healing extra credit and half of battling extra credit


fine here. CCACHE_DIR is $HOME/.ccache-openelec
it's likely something in your build env clering / setting CCACHE_DIR to something else, behind your back.


modification to edit holder details method


if you use sockRequestRaw, you don't need bytes.NewReader


Yeah, it's for terminal logging.  It's not actually used in the code as written, but invariable I try to add logging and then wonder why it doesn't work.


Rename $pwd1 to password in value?


 Like I said, this strategy will partially be hampered by trying to come up with a way of properly detect the result of coefficient container type promotion. Keep in mind that other array types don't necessarily define this behavior well either, which could make life difficult...

```
julia> promote_type(SparseMatrixCSC{Float64, Int}, Array{Int,2})
AbstractArray{T,2}

julia> speye(3) + eye(3)
3x3 Array{Float64,2}:
 2.0  0.0  0.0
 0.0  2.0  0.0
 0.0  0.0  2.0
```

So we can't fall back on `promote_type` to return the "correct" coefficient type to define something like:

```
promote_type{B,T,N,A,B2,T2,N,A2}(::Type{QuArray{B,T,N,A}}, ::Type{CTranspose{B2,T2,N,A2}})
```

...if `A` and `A2` are different container types (e.g. `A` is a sparse matrix, `A2` is a normal `Array`). We could ignore promotion between the same types (since `+(::QuArray, ::QuArray)` can fall back to `+` for the coefficients), and just focus on promoting between different subtypes of `AbstractQuArray`. Then, we could fudge it and just do something like:

```
promote_rule{Q<:QuArray, C<:CTranspose}(::Type{Q}, ::Type{C}) = QuArray
convert(::Type{QuArray}, ctarr::CTranspose) = QuArray(coeffs(ctarr), bases(ctarr))
```

I'm not sure whether the vagueness would be an abuse of the promotion system or not. I don't see any problems with the above, but it feels kinda hacky. It would, though, solve the problem of what to do when `CTranspose` is genericize  - `CTranspose`'s type parameters will probably end up changing, since `AbstractQuArray` doesn't have a type parameter for the coefficient container.


i thought we made this file with uppercase?


@GordonSmith I suppose with these changes, we'll have to change the `require` shim?


I know but I've used sockRequest because I just need the status code and the body and not the response struct here, also I don't need to specify the content-type


The simplest approach would be to leave the implementation as it was (not using QB_XS internally), and add a note in the doxygen block that if QB_XS is used inside fmt, both the error message and errno will become extended information.


I don't like "caps" as a name


Does this (awesome thing) need `beast::static_initializer` so we can accomodate VS2013?


IBOutlets shouldn't be strong... But there is an exception to that rule:

IBOutlets that are wired up, but not in the main view hierarchy should be strong. Otherwise they can be nil by the time you try to access them.


What's server capabilities?


Append a string to what?


nit: the UID uniquely identified the pod. Are the concerns with the length of the unit file? If so we should use the UID only. If not, having something more readable is nice.


`$data_array['$password'] = $password;`


So the intent here was to take the notifications URL and transform it into the API url. This is to support backwards compatibility and the dual purpose of having the API for the extension to access, while having the tab open to the actual webapp notifications URL.

I think this should probably be solved by asking for the root URL of github/github-enterprise rather than this hack. That way I can just append `/api/v3/notifications` and `/notifications` or point to `api.github.com/v3/notifications` and `github.com/notifications`.


Are you seeing this problem on a 32-bit system? `COUNTER` is 32-bits on a 32-bit system, and 64-bit on an 64-bit system, allowing us to do integer division. In practical terms we would only see overflows on 32-bit systems.

I worry that this would result in lower resolution. For example, if we send 999,998 bytes in 999,999 us, our bytes/sec would be 0.

First thought is to use 64-bit numbers, but division would fail. Possibly we need the original formula for low numbers, and the new formula for large numbers. A possible test could be...

    if (stats->bytes_sent > ((COUNTER)~1) / 2) ...


Actually `$password` as per my other comment on L271 in register.php


As I was working through this refactor, I kept thinking "If I squint hard enough, there's a pattern here."  But I still haven't quite figured it out.

There must be some other JSON datasets to play with, but can't think of one off the top of my head.


This looping and then checking equality to one entry in the loop is rather confusing (also, checking equality is strange because it's not clear how object equality works in the spec).


Understand.


In most languages, `_` acts as an anonymous/wildcard/ignored/placeholder descriptor.

- In languages providing import mechanisms, you often see `import java.util._` (wildcard).
- In languages where you cannot specify name arguments, you often see `myFunction(_, bar)` where other languages (like Python) would support `myFunction(foo=bar)` (ignoring the first argument).
- It is sometimes used as placeholder variable. For example, in Scala, `List(1, 2, 3).map(_ * 2)` gets rewritten as `List(1, 2, 3).map(x => x * 2)`. There might be a similar example in Python.

I am sure I am forgetting dozens of use cases and examples :-)

I agree with you that the use of `_` is different in both these cases, but the goal remains the same: reduce the room used by meaningless information :-)
Yes, I could have used `DD` or `DocDown` as you suggest, but there is little point in doing so. For a local variable, it doesn't matter. The point here is that you don't want to read that as a variable but as an alias. Make it anonymous and you know where to look :-)

The only drawback this can have here, is when using [Underscore.js](http://underscorejs.org/) or [lodash](https://lodash.com/), as `_` is not protected in JS (but pretty much nothing is...). I find these library names very controversial as they name something meaningful with something that is meaningless. But it's not a big deal because, first, we are not using any and, second, you can name your imports the way you want (`lo` or `lodash` is much more suited than `_` in my opinion...).

Sorry for the big read :-)


Maybe truncate UID to some fixed length? @vmarmol


Have also removed use of ActionBarActivity as it is deprecated and replaced it with AppCompatActivity


Does ths actually need to be a JSON object or is it enough to make it a set of local variables?


Sorry, I was advocating only using the UID :) it is short enough that length shouldn't ge an issue.


I can't make it work; for some reason QUnit just times out whenever I use any node.js specific stuff.


@vmarmol Need to change the `pod` to `containerID`. But since I need the rkt pod uuid, so I need to change the containerID format, to be something like `rktuuid_appname_imageid_podname_podnamespace_poduid`. Which is...


OK, have done that now. If it all looks OK now then let me know and I can submit the pull request.


Any thoughts about what to do when wrapping in a `<div>` doesn't do the right thing? Such as when rendering a series of `<tr>`s?


I think this RedirectResponse would be more useful if I could specify where I want to redirect to. At the moment you have it hard coded to "http://localhost:5000/"


Or you can use https://github.com/ocaml/ocaml-travisci-skeleton/blob/master/.travis.yml :-)


Yes, but that doesn't remove the need for cache busting. =)


sockRequestRaw, to avoid reading the body


Probably we also need to erase new Fuel snapshot here because it didn't pass basic pipeline


ok


This is where full backwards compatibility becomes really hard - we should probably throw a warning in that case and advise against templates with multiple `<tr>`s


Beat me to the punch! I added it here:

https://github.com/vigetlabs/colonel-kurtz/commit/dc8cec6107ace1112358ea3f051e399ef6c9d0ce


AMScanConfigurationViewHolder3::AMScanConfigurationViewHolder3(const QString &frameName, bool squeezeWidget, AMScanConfigurationView* view,  QWidget *parent) :
	AMActionRunnerAddActionBar3("Scan", parent)
{
	view_ = view;

	AMTopFrame *frame = new AMTopFrame(frameName);

	if(view_){

		addWidget(frame);

		if(squeezeWidget){

			layout_->addStretch();
			addWidget(view_);
	  }
	  else
		addWidget(view_);
	}
	disabledWarning_ = new QLabel("");
	QFont warningsFont;
	warningsFont.setPointSize(32);
	disabledWarning_->setFont(warningsFont);
	disabledWarning_->setStyleSheet( "QLabel{ color: red }" );
	addWidget(disabledWarning_);

}


lets leave this for another cleanup pr, it's really minor


https://github.com/samseaver/KBaseFBAModeling/blob/master/lib/Bio/KBase/ObjectAPI/KBaseBiochem/Biochemistry.pm#L1341-L1343

and

https://github.com/samseaver/KBaseFBAModeling/blob/master/lib/Bio/KBase/ObjectAPI/KBaseBiochem/Biochemistry.pm#L1376-L1378

The mergeBiochemistry function was called on individual databases to
consolidate them, and on pairs of databases to merge them, and
consequently, duplicate searchnames were eliminated, or should have been.

On Wed, Apr 29, 2015 at 11:42 AM, Mike Mundy <notifications@github.com>
wrote:

> How did you make sure the search names never matched? Can we use the same
> technique for names? I'd like every compound to have a name.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/03768bff842f2d0934b241943d42bb05975a075c#commitcomment-10967636>
> .
>

--
Postdoctoral Fellow
Mathematics and Computer Science Division
Argonne National Laboratory
9700 S. Cass Avenue
Argonne, IL 60439

http://www.linkedin.com/pub/sam-seaver/0/412/168
samseaver@gmail.com
(773) 796-7144

"We shall not cease from exploration
And the end of all our exploring
Will be to arrive where we started
And know the place for the first time."
   --T. S. Eliot


It's just the order you're doing the math.  I'm seeing overflow once bytes_sent > ~1845623851294

1845623851294 * 10 * 1000 * 1000 == 18456238512940000000

That number is a touch larger than a uint64.  All I did was change the order the operations are done so that the system doesn't have to deal with intermediate values greater than UINT64_MAX. This isn't really a good fix, but it doesn't fail until significantly larger numbers now.


We have been hitting a regular stream of Mono issues in our development on Mono. The Mono folks have been very responsive on getting these issues fixed, but based on our experience and the fact that we don't do full stress testing on Mono we don't recommend using Mono in production. Our long-term plan is to have full support for running cross-platform on .NET Core.


Maybe say "...`coefftype` which gives the type of the coefficient container". The type of the coefficients themselves would be given by `eltype` (which I don't believe we've overloaded yet, but will soon).


Nice, we definitely need this but unfortunately we can't rely on the directory test to get this right.  My stock CentOS 6 system does have an `/etc/default` directory with one mispackaged file which throws this off.  It would be more reliable to add a `case "$VENDOR" in ... esac` block similar two the other two in this block.


I'm not sure what a "string containing a difference" is.


The groovy-all upgrade to 2.3.6 isn't specific to getting this to work with Java 8. I just needed to do it to get it to run correctly after this [New nebula.netflixoss build](https://github.com/netflix/zuul/commit/a764853549facfbcbc8596029f8c23b7fd300586) commit.


I'll show you my stats output too, it took me a couple times of doing the math to figure out exactly what was happening but the math does work out if you account for integer overflow.

```
Actual: 2386036898 packets (1842359678522 bytes) sent in 1653.01 seconds.
Rated: 1114459179.7 Bps, 8915.67 Mbps, 1443334.19 pps
Actual: 2390265283 packets (1845623851294 bytes) sent in 1656.01 seconds.
Rated: 573286.4 Bps, 4.58 Mbps, 1443272.83 pps
Actual: 2394505452 packets (1848897864644 bytes) sent in 1659.01 seconds.
Rated: 2545565.7 Bps, 20.36 Mbps, 1443218.34 pps
Actual: 2398748223 packets (1852173884153 bytes) sent in 1662.01 seconds.
Rated: 4511932.6 Bps, 36.09 Mbps, 1443166.07 pps
Actual: 2403011139 packets (1855464928142 bytes) sent in 1665.01 seconds.
Rated: 6480237.1 Bps, 51.84 Mbps, 1443126.08 pps
Actual: 2407265344 packets (1858750311243 bytes) sent in 1668.01 seconds.
Rated: 8438064.7 Bps, 67.50 Mbps, 1443080.39 pps
Actual: 2411507188 packets (1862025478515 bytes) sent in 1671.01 seconds.
```


Consider having the `static` on a separate line.
Consider having the introducing `{` at the end of line 76 so the individual constructors all line up.

STRONGLY consider having one line per entry and having them alphabetized.


seams you are right... filesystem problem :(
sorry for the noise


You can scan through these issues to get a feel for the kinds of things we have been hitting: https://github.com/issues?utf8=%E2%9C%93&q=is%3Aissue+user%3Aaspnet+bugzilla


I still don't know why `'` is needed, I'll look into this.


Econômico nas linhas (não sabia que dava pra fazer isso) :stuck_out_tongue:


Since you're putting in `const` whenever you can, you should put it on `iter`, right?


Right. Kind of a bummer 3.9.3 was tagged just before that was merged.


The status identifies problems with the reaction, not necessarily changes.
In the case where proton balancing was actively done, the letters "HB" were
used to indicate so. But in this case, this status means that no changes
were made, and the reaction is not "OK".

I'll have to have a look at the instance you just pointed out. I can't
speak for the formulas in the older biochemistry but in the new
biochemistry, yes, the status should have indicated that a proton is
imbalanced, I'll check.

Would you be kind to double-check a few more reactions, so to see there's
no systematic error?
S

On Wed, Apr 29, 2015 at 11:11 AM, Mike Mundy <notifications@github.com>
wrote:

> I thought that the status identified changes that had been made to the
> reaction to bring it into balance. Did I misunderstand that?
>
> Also it looks like the reaction status might need to be recalculated for
> the latest biochemistry. Reaction rxn12814 has equation cpd00350 ->
> cpd00012 + cpd00178 + cpd15002. Using the formulas from
> compounds.default.tsv that corresponds to C15H26O7P2 -> H2O7P2 + C3H6O +
> C12H22O and that reaction has 7 O on the left hand side and 9 O on the
> right hand side. And there are 26 H on the left hand side and 30 H on the
> right hand side.
>
> If you use the formulas from compounds.plantdefault_obs.tsv, the formula
> for cpd00012 is HO7P2 which means there are 29 H on the right hand side.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/c2e4824b9ca05a6230cc68a101b2ce0a4b2395e2#commitcomment-10967099>
> .
>

--
Postdoctoral Fellow
Mathematics and Computer Science Division
Argonne National Laboratory
9700 S. Cass Avenue
Argonne, IL 60439

http://www.linkedin.com/pub/sam-seaver/0/412/168
samseaver@gmail.com
(773) 796-7144

"We shall not cease from exploration
And the end of all our exploring
Will be to arrive where we started
And know the place for the first time."
   --T. S. Eliot


If you're consistent, this should be:

    static auto const limit = //...


Let me check again then

On Thu, Apr 30, 2015 at 12:16 PM, Sergiu Dumitriu <notifications@github.com>
wrote:

> I still don't know why ' is needed, I'll look into this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/phenotips/phenotips/commit/466d75bce5e9cfd7603026c4fb22ad57cd8ddfae#commitcomment-10988262>
> .
>


Ohhh.  I suddenly realized that you're optimizing for precision with this by deliberately using huge integers.  Unfortunately it's pretty easy to run into trouble the other direction too.  I'm hitting this wall after only ~27 minutes at ~9Gbps.


> Why do you call it shear thinning? You just want to test if the output
> of the strain rate is correct, right? Shear thinning normally only
> occurs when the viscosity depends on the strain rate...

It was a copy-paste problem. The text still said it from where it copied
it from, but the code just used the 'simple' material model.


You don't check the return type here!


The analyzer thought `first_tabpage` was becoming NULL because of the previous `goto_tabpage`. I've also checked that this assert clears the warning.
The following loop should close all but current tab page, so `first_tabpage` should not be NULL after it.
I don't know if it's saying it's ok because analyzer can detect the loop should not make `first_tabpage` NULL or because it's not analyzing that case.
So, we can leave it as is or place the assert before the dereference. What do you like best?


@somini  Ok, I reverted those commands.


Doesn't matter I guess.


Good catch, but could you just copy the parameter documentation from the base class while you're at it? (Otherwise we'll fix that later.) This is the formulation I meant: https://github.com/Lasagne/Lasagne/blob/master/lasagne/layers/base.py#L30-L31
Seems I missed the docstrings in `dense.py` when renaming the parameter in 8a5a8f9, sorry!


memchache ist not really needed ;)...... memchache will not installed,...... Memchached is even better from Performance....... the Perfect Install do install things, which are not really needed... I removed memcache, because memchached is installed ;)  For ever php5lib you also need the Program... ;) memchache is not be installed during Perfect Server Setup :)


`clock_gettime` is implemented in libcompat using time(NULL).


As mentioned in another comment, I'd like to see an alias for the address integer type used consistently across all the IPAM code.  The relevant uses of uint32 in the ring code should be converted, and probably the net.IP uses too.


Maybe define the constant `"4.2.0"` at one place instead? Later it can be incremented easier.


You'd have to throw a warning even with a template that generates a single `<tr>`.

Is there a container element that would let you wrap `<tr>` correctly? Can we inspect `vdom` to figure out what the best container to use is? ...looks like `<tbody>` does the right thing, but only if we wrap /all/ `<tr>`s in a `<tbody>`

(There are more problems, like `<span>` vs `<div>` depending on if the elements inside the container are `display: block`, `display: inline-block` or `display: inline`)


No, it looks like he means to change `limit` later. Perhaps by decrementing it as results are added.


@dleacock it's possible to test on BioXAS this morning; come by if you have time.


In fact I think this whole loop should be restructured so it's clear that for most properties, at the furthest remote (we should define a term like that which basically means "browser"), desired capabilities is not used at all for things like platformName, and both are only checked for pageLoadStrategy and Proxy.


Estás chamando a API de fato aqui?
Se tiver acho que dá pra melhorar esse teste com doubles futuramente.


This seems like a terrible idea.


We can derive the grace period from the `MinAge` of the `ContainerGCPolicy` We should add a TODO for that.


nit: we're already in the `rkt` dir we can call the file `gc.go` :)


Does ekt_tag_utils.h include prototypes for the public API?  Are applications going to need to invoke srtp_ekt_tag_encrypt()?  I would have thought the EKT encryption was done internally within libsrtp.  I'm wondering if we need to install ekt_tag_utils.h.


@nikomatsakis
Why?


I think this will work, but I wonder if you would get better results by using `preg_match_all`.  then, you could match on something like `(\d)` and get all numbers.


I'm not really sure why this is a per-session variable that can only be set here, whereas timeouts are per session variables that can only be set by a specific api call, and never here.


it still cache busts, it just inlines it into the main file. @marjan-georgiev


There must be some way. Maybe someone else can take a stab at it.


Line number where it gets changed?


I have to worry about simplify(self) being none anyway.


yes, install(DIRECTORY) installs an entire folder,

install(DIRECTORY bar DESTINATION foo)

will make the folder

foo/bar with the structure of the bar install folder instact.


nit: "state the same as the pod"


Can we document these fields a bit?


I'm guessing this is the init pid?


You're basically returning the same error each time!

Why not this code snippet:

    uint256 dirIndex;
    uint256 entryIndex;
    auto hasMarker = params.isMember (jss::marker);
    if (hasMarker)
    {
        auto const& marker = params[jss::marker];
        if (! marker.isString ())
            return RPC::expected_field_error (jss::marker, "string");

        std::stringstream ss (marker.asString ());
        std::string s;
        if (! (std::getline(ss, s, ',') &&
               dirIndex.SetHex (s) &&
               std::getline (ss, s, ',') &&
               entryIndex.SetHex (s)))
            return RPC::invalid_field_error (jss::marker);
    }
    if (! context.netOps.getAccountObjects (
            ledger, raAccount.getAccountID (),
            type, dirIndex, entryIndex, limit, result) )
    {
        // report error for both paths, perhaps using hasMarker.
    }


Não, a requisição feita é fake ─ a mágica fica por conta do [`stub_request` da linha 71](https://github.com/Bouckaert/donamaria/commit/067ee378839af45a25135859d02c29cdedf6d905#diff-ac323e6f8a180df7516a96ef6a70bae8R71). A resposta à requisição é o [JSON definido em mcn_response_bodies.rb](https://github.com/Bouckaert/donamaria/commit/067ee378839af45a25135859d02c29cdedf6d905#diff-01bfa1dc7ec07d686f3b09e99fe783faR52).


that's a lot of red


You can understand how it sounds when Microsoft declares Mono unfit for production. That was my point, really.


I recommend using JSON fixtures for this sort of thing.


Cool, filed #1. Thanks @benbuckman.


You wrote this comment (at 6349a612), not me.


Pois é, peguei do [guia de estilo](https://github.com/bbatsov/ruby-style-guide#no-single-line-methods) ─ que diz ser aceitável esse tipo de sintaxe.

Adoro economizar linhas! Pra mim, quanto menos linhas, melhor ─ contanto que não prejudique a legibilidade do código.


Hola.   I believe this has been fixed in the most recent version of Jolt.

https://github.com/bazaarvoice/jolt/commit/dc2e4530e2f02c9a8ae26b5e918cd4ebe7e9116a


@ht12a092 http://oecu.pw/API/RouteList.php にパラメータ無しでアクセスすると404が返ってきます。仕様変更ですか？


Haha, love it.


Hola.  This has been incorporated into the mainline.

https://github.com/bazaarvoice/jolt/commit/bcb332c6482732ee73bbcef1f71b7d06e59d0f89

Cheers.


check it out @tomviner! in Python 2 start and stop slices have different opcodes (no longer in py3, alas), so (at least on python 2 when there's only one slice on the line) we can disambiguate None slices like so! Changed the `xfails` below to only be for python 3 since they're passing on python 2 now! \O/


@meodai Mouhaha, was looking forward to this remark :D


I also recommend using https://github.com/gruntjs/grunt-contrib-watch

it can help you debug your tests and fix them more rapidly by recompiling your coffeescript constantly.


if netRemderAckReceived is false, when last lines got rendered, shall the client wait for ACK, or just leave it to the server?
Maybe we can wrap the whole if (netRemderAckReceived) ...into a function, since the code is needed twice?


Show!


This block can be replaced with the `udev_trigger` helper.


Tahnk you for fix as usaul :)

I remove the fucked roundcube for now, no more support from debian, no more support from us :)

Now the install fork like a charm on debian Jessie too, i've noticed the now apache directory are changed from /etc/apache2/conf.d to /etc/apache2/conf-enabled/, similar way as site-availabe, site-enabled,

Got to pay attention :)


This seems to make Haxe generate type parameters even with `-D erase-generics`:

```haxe
abstract A<K:String,V>(Dynamic<V>) {
    public function keys():Array<K> {
        return [];
    }
}
```

```cs
public sealed class A_Impl_ {
	public static global::Array keys<K, V>(object this1) {
		return new global::Array(new object[]{});
	}
}
```


There is another printf with ??? a few lines above. It looked odd/asymmetric to me that this one is split into two strings with a space, but the one above does not. (Or change both places to be ?\?\? - it should fix the warnings as well.)


You seem to like Jade, why not use Jade for your app html creation?


i cloned the branch and it evaluates:
nix-env -f $NIXPKGS -qa --description

seems to be good work, will merge it

thanks very much for your effort!


We seem to have lost some of these variables with no explanation why.
Also why do the buckets have different names?


I'll fix that right now


@pecko  - this is an alternate implementation, though they're both quite similar.  You may choose to use this instead.


Morning Drew,

Do you recommend Effective Java?

Benito J. Gonzalez - Unicon
bgonzalez@unicon.net <mailto:bgonzalez@unicon.net>
480.558.2360

> On Apr 29, 2015, at 3:12 PM, Drew Wills <notifications@github.com> wrote:
>
> For my money, I'd make go with a final class and make all the members private (though you will find active community members who disagree). I subscribe to the inheritance philosophy outlined in in Effective Java: design specifically for inheritance (meaning your class is almost certainly abstract) or not at all (meaning your class is almost certainly final).
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bjagg/uPortal/commit/43c830d9e4174b0136c78fb82e02fb401dd06451#commitcomment-10974234>.
>


Thanks!


@spawn-guy that appears to be only for `models`, this is regarding `requests`.

I keep seeing the shimming `rules` with modifications like what @xpromx says [here](https://github.com/laravel/framework/commit/924a7fcf21bbba4f4efc8e367f456cea5e4d25c1#commitcomment-9969020) as a possible fix however I don't think that works. At least not if you are adding new or removing from your `request` object.

If you look at line [88/81](https://github.com/laravel/framework/commit/924a7fcf21bbba4f4efc8e367f456cea5e4d25c1#diff-93a08199642b328e7c151fef15179216R81) You see that the `$this->all()` parameter will be [evaluated first](https://php.net/manual/en/functions.arguments.php) rendering any later changes to the contents unseen and unused.

The use of a sanitize method, like the one I have been using from [this laracast post](https://laracasts.com/discuss/channels/general-discussion/laravel-5-modify-input-before-validation/replies/34366) is really the only effective way to handle such a case.

I might argue though that the naming may need adjusted though since, at least for my use case, I am using it to define sensible defaults or massage the data from the form prior to the controller or validation having access to it so it is not simply a checkpoint for sanitizing as much as it is for modifying or processing imho.

Sanitizing I feel is more for data returned to the user, stored in the db, or stored in logs to prevent information from being exposed.


gave asplib a looksie. it's rather borked ;)

- you have to set TARGET_WINDOWS or TARGET_LINUX
- you don't set SOVERSION
- you don't write the cmake config mode file. that's why you have fuglies on the add-on end.

when you call find_package(xxx) in cmake, it first looks for
lib/<foo>/foo-config.cmake

if this is found, it's executed. this file then adds the necessary include file and libs (or rather, sets the variables with the info). this is how a native cmake library should behave.
if not it falls back to probe stuff (which tries pkgconfig - the config mode of 'autotools land'). then finally searching for files using find_path and such.


React can handle a single `<tr>` correctly. A wrapper is only needed when a template has more than 1 top-level element. So the easiest thing is to recommend always writing templates with 1 top-level element (which is what React does).

On the other hand, we can get around the inner elements block/inline problem by always using `span` as the wrapper, although that would render semantically-incorrect HTML. (in fact Instagram's entire application is nested inside a `span`).


I removed it and tested that nothing is broken. The variables appear nowhere else, so I think its safe to commit.


(I think this is still too crappy to be worth merging in its present state, but if I can that last test to pass at least on python 2 (the `b[None:] + b[:None]` on the same line case), I'd seriously consider it.)


This seems like a lot of repeated code.  Why not just have a $scope.newUser and simply pass $http.post "/api/users", $scope.newUser ... ?
Copying all these variables back and forth seems like quite an annoyance.


Change this to @jsftools specific repository


eh? I don't see a cache bust there... https://github.com/Swimlane/angular-systemjs-seed/blob/master/index.html#L33

Also, what is the point of this now? https://github.com/Swimlane/angular-systemjs-seed/blob/master/gulpfile.js#L122


This is highly uneccessary as you already used ng-true-value and ng-false-value to do this correctly.


Looks like we lazily parse everything but the pid and exitCode. Should we just eagerly parse everything here?


nit: Why not allocate this in the constructor? Memory usage should be minimal no?


Lets handle malformed input with a check instead of panic()ing


Should we move these consts defs to this file?


Should leave, because is ACK is not received it means that the server is not ready to receive it (or network is too slow). Event if it send the data, probably it will be received during rendering next animation frame.

About wrapping into the function, it's good idea, because it's the same code in two places.


Yeah, it is. But this subdirectory seems to be designed specifically to confused `make` and `protoc`... :stuck_out_tongue:

This is one reason why I haven't sent this for review yet.


Nethack Config


they said they didn't care if it showed everytime, as it is just up from May1 - June6


Thanks. Will try today later.


I think this is a legacy issue. We used to have some sort of special container for the module links (at least I assume this for explaining why this style rule exists) which is not available anymore. Thus, if you add a min-width to your main content container (in the theme) (which is done in my example) this accidently applies to the module links, too. So as the mentioned extra container is not there anymore, the inherit rule should go, too.


Just comment out this line to not remember within react session.


Closed via 9da229e234c2f903ff2569f3d9422b60d28dbf4e


I don't think ``time()`` is granular enough, it's only in seconds. This would need at least 1/10 of a second, preferably even better.


I've almost removed this a couple of times, but I still have some other code that uses Hound as a library and creates a Searcher to do some automated searching of a repository. We've used it to try out some automated flagging of potential security issues. So yeah, I effectively disable the limiter for this use cases.


@zjrosen1 why the change?


Thank you for the feedback.


I had the same problem. It is because the COMODO RSA root CA is not in our bundle.
I have installed certifi on my system, which is called by the requests module.
after upgrading it, it worked: "pip install certifi --upgrade "


That fork looks curious due moving all files to an extra folder. How did you do that? The history to Flyspray master looks to be lost on github.com (https://github.com/xcdam/flyspray/network)


seems it's not used


should be fish-mode-syntax-table


@marcandre hey there, I don't think I should open an issue for this, since it's not a stable version, but I found a bug with the UI here; basically, it's a typo:
`'undefined' === typeof fieldInstance.options.multiple`
should become `!fieldInstance.options.multiple` (identical to line 267)


so for the next binary release, we need to make sure the ca_bundle is upgraded


the issue is that this version doesn't deal with one-line block, like if true; echo 1; end


Added a link within a picture.


I would


Haven't looked but since its not marked `const` I assume it will be changed


The previous code is a bug. The `false` must be used to avoid `multiple delete`. The bug did not be detected since in previous examples, parameters are `scalars`. The bug did appear when GaussianARDKernel is used since the `weights` parameter is `not` scalar.


It's generally better to not hardcode colors into your css.  it makes it hard to theme or change colors rapidly.  Best to move these into a variables.css file.


@JamesXNelson

I am looking forward to see a release of this version,big milestone!!!!
Moreover,Nelson,I want to discuss with you about the business and project of our company
what is the best way to catch you? phone,email?

Thank you very much.
Regards


This is weird.  You used camelCase styling for the booleans, but underscore_styling for the date and id ?


I love tpope and all of his plugins so it was a very sad day. However Vundle in short makes it easier to manage plugins


hmm, even if we ship binaries with an updated ca_bundle, users running electrum from source will still be affected. we should find a way to ensure our ca_bundle is always getting used, not the one installed on the system


 I think this is missing a period.
 +maple_mini.menu.bootloader_version.original.upload.ram.maximum_size=17408


Rafael noob esqueceu de identar.
:)


ivb-3-14/Хлебников.А.А/Лабораторная работа №8/Matrix/Matrix/main.cpp

Зачем это тут?


Yes, but I changed it to execute a native process (git), as it depends on less libraries


If `hash` is null or `''` bcrypt is unable to find a salt string and will throw an error.


Sorry professor. Will resolve conflicts when pushing.

--
Deh Jun Tzou
Emory University Class of 2015
Email: dtzou3@emory.edu<mailto:dtzou3@emory.edu>
U.S. Phone: 404-973-8682

On Apr 30, 2015, at 11:35 AM, cengique <notifications@github.com<mailto:notifications@github.com>> wrote:

You made a conflict above and didn't correct it!

—
Reply to this email directly or view it on GitHub<https://github.com/mzhai2/ClearViz/commit/bbcda10e3f8466907c9a5c8e76e3e4e5ef07cc76#commitcomment-10987431>.


@J-east Are you sure this is what you intend? `get_secret()` expects the `setting` parameter is a dictionary key of the desired field name.


Move this inside Auth::check()


thx


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Use of `{}, {},` is obscure. And it depends on types like `uint256` zero-filling in the default constructor, a practice that I disagree with for all the reasons that `int i;` should produce an uninitialized variable (pay for what you use). Can we pass `zero` here instead of `{}`?


Don't you need to use `createBackendLogger = require('../lib/logging_utils').createBackendLogger`?
With your code you're setting the createBackendLogger to the modult logging_utils


I hate pages where you have to enter the URL again after authenticating :+1:


Well, `setUp`/`tearDown` are supposed to be generic. They're called before/after every test method in an XCTestCase, so they can't be tailored to each individual test.

From your example, I don't think the issue is with `tearDown`, but rather that we're not explicitly testing for something. If we really want to check that the number of tabs added are the number expected, let's add it to the actual test method, like you said. Relying on the slim chance that tests running on the same app instance might catch an error isn't a very robust strategy.


\? Doesn't work, clang complains that it is an invalid escape. Ok, let me add the string split at the other place as well.


:+1: :tada:


Thanks @Wildpetals!! I invited you to our team now so you can add these links as a commit :) let me know if you have any questions though! I also added you to next Monday's meetup group.


I believe the getValue() method is missing from the class mage.game.turn.Turn.


OK, this is looking good. What's the testing story look like? It's been a while since I've dealt with WSGI directly, so if there's a small library that makes mocking the WSGI environment easier feel free to add it. Probably one positive test and one negative should suffice.


Also can you go ahead and open a proper PR so it's easier to track the comments and changes?


done


wtf? Such a typo must not pass the eye of a developer unnoticed...


Agreed, I wouldn't expect it to work there. But I was surprised that you
had to amend a fix because it didn't build.

On Thu, Apr 30, 2015 at 6:38 PM, Kristian Amlie <notifications@github.com>
wrote:

> I don't think time() is granular enough, it's only in seconds. This would
> need at least 1/10 of a second, preferably even better.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kacfengine/core/commit/2b69c73ed386b50391b05acce8baab9cec9ae031#commitcomment-10988674>
> .
>


Makes sense. I think it doesn't break what we need, so I was just making sure it was something you wanted. Thanks for the explanation!


Nein, c'est trop moche, y a une fonction dolibarr qui affiche les users


@robertmesserle - Does the current width affect browsers? should we make the width more realistic; e.g.  `width:4000px`?


:)

I would say something more along the lines of "tool to reload and test modifications to base without recompiling the full system image"


es reebia?


XD


+| Jonny Baho | [Github] | [YouTube](https://www.youtube.com/watch?v=0YfKMiXk68Y&spfreload=10)| [@jonnybaho](http://twitter.com/jonnybaho)| |


changed


Yes -- I recommend [Effective Java](http://www.amazon.com/Effective-Java-2nd-Joshua-Bloch/dp/0321356683/ref=sr_1_1?ie=UTF8&qid=1430412965&sr=8-1&keywords=effective+java) very highly.  If you can make time, I think it would be great for you to read it.  I should warn you, however, that the current edition is from 2008.  The best thinking inside the Java dev community does tend to shift over time... also, advances in new JVM versions can make some older advice irrelevant or (occasionally) flat-out wrong.

But most of the insights in this book are more-or-less timeless.


Ok. So which uses /etc/default and who uses /etc/sysconfig? lsb =\> /etc/default and /etc/sysconfig otherwise?


It's auto-generated py [pandoc](http://pandoc.org). I write Markdown and then run

```
pandoc --from=markdown --to=latex -V fullpage=true
    -V fontsize=10pt --latex-engine=xelatex --mathjax
    --template=./default.tex -o notes.pdf notes.md
```

to convert it to a LaTeX pdf. The variables in the document class are settable from cmd line (e.g. -V fontsize=10pt). It's pretty neat.


Might be because I'm inserting it into Javascript

On Thu, Apr 30, 2015 at 12:18 PM, Andrey Ploskov <andreyploskov@gmail.com>
wrote:

> Let me check again then
>
> On Thu, Apr 30, 2015 at 12:16 PM, Sergiu Dumitriu <
> notifications@github.com> wrote:
>
>> I still don't know why ' is needed, I'll look into this.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/phenotips/phenotips/commit/466d75bce5e9cfd7603026c4fb22ad57cd8ddfae#commitcomment-10988262>
>> .
>>
>
>


do a paralel work


I will remove


He respondido bien una pregunta y me ha dado el punto. Justo después he respondido a la misma pregutna mal y no me ha quitado el punto... Voy a buscar el fallo


LGTM


If the object previously had siz


haha


You should put .idea/ in your .gitignore file so that it excludes the WebStorm project files from the commits.


Please see the ticket in one of our repos mentioning the stacktrace: ros-infrastructure/ros_buildfarm#65


Awesome! Thanks.


Very cool. The fixed pixel dimensions work on devices with a different aspect ratio?


There is phablet changes on those (hardware) repos


Wait, it's already there. Okay.


I tried coming up with some mock/patch to capture the execvp/execvpe function calls, but ran into a ton of problems. Took a platform-specific approach, since it executes `/usr/bin/env` and looks at the stdout.

What are you thoughts? I feel like I should just drop these tests, since all I'm doing is passing a parameter through the function to pass to `Popen()`, and not adding any logic around it.


Nobody is using the `ensureAuthenticated` function in `server.js` so you can delete this


The database connection is not used in this file. You can delete this.


Also, note the commit message. :smiley:


j'aprouve cette modification et en meme temps je teste les commentaires


seems line-number-at-pos does the same thing


Pourquoi enlever cette condition?


Made changes for "Strings" assignment.


Would you be interested in making this into a Pull Request, so we can merge it into the main codebase for next release? :smile:


bon pour que ca marche bien il faut creer des fragment et tout et la je suis pas en etat! je m'en occupe a la fin de ce week end quand je reviens de l'indoesie. des bisous


Nah, just a request from someone using the cookbook


this is wrong, if the order of limb is larm,rarm,head, then we should put joint-angle of the in the end of this line, and head joint in the joint-angle is the top of the list, which is inconsistent.

```
git diff e96178b a42c85d baxter.l

-     (setq joint-list (list left_s0_jt left_s1_jt left_e0_jt left_e1_jt left_w0_jt left_w1_jt right_s0_jt right_s1_jt right_e0_jt right_e1_jt right_w0_jt right_w1_jt head_pan_jt))
+     (setq joint-list (list head_pan_jt left_s0_jt left_s1_jt left_e0_jt left_e1_jt left_w0_jt left_w1_jt left_w2_jt right_s0_jt right_s1_jt right_e0_jt right_e1_jt right_w0_jt right_w1_jt right_w2_jt))
```


You should delete all the imports that are not used anymore in `server.js`. e.g. LocalStrategy, passport, session etc.


Since we're now using logger for consistency you should replace console.log with logger


oh sorry, I got confused: updating certifi did not fix it...


Although not *directly* part of this PR, this type check is weird. I'd suggest in a followup issue we look at switching this use ```class.isInstance()```, ```isAssignableFrom()```, or ```instanceof```


session is not used. It should be deleted.


Yeah, that's fair. Updated.


As for obtaining a group based on its name, I think it's simply the static...

```
  GroupService.findGroup(String name);
```

At this point in the history of uPortal -- and for the last several years -- group names are guaranteed to be unique (which you should keep in mind when it comes to allowing users to name their ad-hoc groups... there needs to be some checking).

As long as we're on the topic:  when a user tries to name an ad-hoc group, remember to...
* Check that the name isn't in use; and
* Run the submitted name against a Regex to validate that all the characters it contains are allowable

You should not be defining that Regex in _your_ code (package org...pags.testers);  the function of name checking needs to be something provided by and maintained within the core groups code, with everyone using the same.


Pourquoi des fragments ?


`"/"` should be something like `Config.rootPath` that defaults to /. Config.rootPath is currently wrongly assigned to protocol and hostname but this can be fixed.


Fragment vendredi et samedi pour gérer les tabs
On May 1, 2015 01:10, "Remi Pichon" <notifications@github.com> wrote:

> Pourquoi des fragments ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/24HeuresINSA/24h-android-app/commit/f411922d90f1e46aff54d3b9a0ea30c682077be2#commitcomment-10989209>
> .
>


I think it looks nicer if you define all your imports at the beginning of the file.

Here's a nice pattern:

```javascript
var mongoose      = require('mongoose'
   , userSchema   = require('./schemas/user
  , postSchema    = require ('./schemas/post')
```


Tu n'as plus besoin de fragments, j'ai deja tout mis en place.
Tu as deja le NightResourceAdapter, le NightDetailFragment sur lequel tu dois adapter un peu le layout pour correspondre a tes mockups


Second sentence here needs revising. This is not the reason the behavior is this way but it does provide this guarantee.


Je t'ai fais les tabs et le fragment est toujours ArtistFragment avec le field Day day pour t'indiquer quel fragment est ou


Please use spaces, not tabs to indent.


Does the client handle url encoding? What happens if my surname were 'von Melton'?


Why are we talking about Polymer here?


Not a long term solution by any means.
_threading._local() has no attribute bulk_create, so I created some


Hi,
why do we have to switch to /admin, we need to adjust many links in the views?

Best, Stephan


Behind the scenes `mongoose.connect` creates a connection and stores it `mongoose.connection`. To make things clear for developers you should export the connection instead of the mongoose object.

`module.exports = mongoose.connection`

They both work, but it's more clear what you export this way.


Possibly we create a ```CREATE``` & ```DELETE``` const? ```enum```?


Since it is added as singleton we need to add a queue mechanism for thread handling.


hello


Be consistent always use wstring.


proba


Yeah, going to have to be something like...

```
  public AdHocGroupTester(Set<IEntityGroup> included, Set<IEntityGroup> excluded);
```

Obviously, you will need to alter the code that calls the constructor -- and perhaps/probably some Import/Export code -- to support something besides (String,String).

If -- when the rubber hits the road -- it makes more sense to make those arguments (Set<String> included, Set<String> excluded), we can roll with that.  I just wanted -- all else being equal -- to put the complexity of looking up groups into the calling code, rather than into the constructor on an IPersonTester class.


Hi Maciej, I'm testing... I see huge decimals instead of the initial order total in the log (see below), shouldn't we use commerce_paypal_price_amount() here and for the secondary's amounts ?

Example log for an order total of 94,10€ with use of `commerce_currency_amount_to_decimal()` :
```
Adaptive Payments: pay key request:
array (
  'actionType' => 'PAY',
  'clientDetails' =>
  array (
    'applicationId' => 'APP-80W284485P519543T',
  ),
  'currencyCode' => 'EUR',
  'receiverList' =>
  array (
    'receiver' =>
    array (
      0 =>
      array (
        'amount' => 94.099999999999994315658113919198513031005859375,
        'invoiceId' => 'FM-30-31-1430414105',
        'email' => 'marketplace@webla.fr',
        'primary' => 'true',
      ),
      1 =>
      array (
        'amount' => 34.25999999999999801048033987171947956085205078125,
        'invoiceId' => 'V-30-1430414105',
        'email' => 'marchand1@luli-shop.com',
        'primary' => 'false',
      ),
      2 =>
      array (
        'amount' => 45.72999999999999687361196265555918216705322265625,
        'invoiceId' => 'V-31-1430414105',
        'email' => 'marchand2@luli-shop.com',
        'primary' => 'false',
      ),
    ),
  ),
[...]
```

Example log for the same order total of 94,10€ with use of `commerce_paypal_price_amount()` :
```
Adaptive Payments: pay key request:
array (
  'actionType' => 'PAY',
  'clientDetails' =>
  array (
    'applicationId' => 'APP-80W284485P519543T',
  ),
  'currencyCode' => 'EUR',
  'receiverList' =>
  array (
    'receiver' =>
    array (
      0 =>
      array (
        'amount' => '94.10',
        'invoiceId' => 'FM-30-31-1430413824',
        'email' => 'marketplace@webla.fr',
        'primary' => 'true',
      ),
      1 =>
      array (
        'amount' => '34.26',
        'invoiceId' => 'V-30-1430413824',
        'email' => 'marchand1@luli-shop.com',
        'primary' => 'false',
      ),
      2 =>
      array (
        'amount' => '45.73',
        'invoiceId' => 'V-31-1430413824',
        'email' => 'marchand2@luli-shop.com',
        'primary' => 'false',
      ),
    ),
  ),
[...]
```


rax retry tests


We don't need this ```else``` here - maybe we consider reducing the number of ```returns``` in this method?


Ah ok Nice ! Je regarde ça
On May 1, 2015 01:13, "Remi Pichon" <notifications@github.com> wrote:

> Je t'ai fais les tabs et le fragment est toujours ArtistFragment avec le
> field Day day pour t'indiquer quel fragment est ou
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/24HeuresINSA/24h-android-app/commit/f411922d90f1e46aff54d3b9a0ea30c682077be2#commitcomment-10989253>
> .
>


After checking my comments from the `db/index.js` file I recommend naming this variable `conn` from connection.

```javascript
var conn  = require('../db')
   , User   =  conn.model('User')
```

So, we import a database connection and then get a reference to the User model from that connection.


Define a new exception class for the DB and throw the same here. If the db connection is not there then application will also not work :).


I think we want to look at the isDeepMemberOf() approach.


When, and, and? No Then step?


Its not thread safe.... major.


`_position` seems to be unused arg now, is that purposely?


use call_once of C++11 for initializing. That will make it thread safe and initialize only once.


Just checking since I'd like to tag 0.3.8 today, is there something wrong with the osx builders that needs a ~~swift kick~~ careful fixing?


added the capability of making the files for which the TA had asked and refactored other classes


Forgot to stage that line. Staged in commit 2c3d3d8105091c40b598d25b05930e75f4f438de.


When writing logs you need to always use logger. Moreover, your logs must contain useful meta-information so that when the app crashes you can follow the logs and actually be able to make sense out of them.


I'm not positive, but I think this won't work the way it's intended.

In uPortal groups speak, "Distinguished Group" means something special (and unexpected).  The "Distinguished Groups" are the (essentially) hard-coded groups you MUST have, like "All Categories," "Everyone," "Guests," and "Portal Administrators."  (I think that's all of them.)


The  parsedown lib is not loaded in the top of lib/pico.php. I've test Parsedown with pico but in some project i use markdowExtra (specially when it's necessary to use html and markdown mix in content), does anybody think it's better in Pico to let the choice of the markdown lib like in others flat file CMS (piecrust for example) ?


Ah ok. Just checking out things :)


Sounds good. Really appreciate your emails!

Benito J. Gonzalez - Unicon
bgonzalez@unicon.net <mailto:bgonzalez@unicon.net>
480.558.2360

> On Apr 30, 2015, at 10:09 AM, Drew Wills <notifications@github.com> wrote:
>
> As for obtaining a group based on its name, I think it's simply the static...
>
>   GroupService.findGroup(String name);
> At this point in the history of uPortal -- and for the last several years -- group names are guaranteed to be unique (which you should keep in mind when it comes to allowing users to name their ad-hoc groups... there needs to be some checking).
>
> As long as we're on the topic: when a user tries to name an ad-hoc group, remember to...
>
> Check that the name isn't in use; and
> Run the submitted name against a Regex to validate that all the characters it contains are allowable
> You should not be defining that Regex in your code (package org...pags.testers); the function of name checking needs to be something provided by and maintained within the core groups code, with everyone using the same.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bjagg/uPortal/commit/43c830d9e4174b0136c78fb82e02fb401dd06451#commitcomment-10989206>.
>


All good, appreciated!

On Thu, Apr 30, 2015 at 6:19 PM, Ingvar Stepanyan
<notifications@github.com> wrote:

> Ah ok. Just checking out things :)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/babel/babel/commit/ea03f67505d17b54811eb6c1ba2b976150b6fb4c#commitcomment-10989379


Yes, JS has function scope local variables.
But I'm thinking about the "logical scope" of locals, the one that helps developers to ensure what is intended scope of the variable (which for example in case of a loop is usually limited to the code of that loop).
There are JS dialects that extends the language by explicit addition of so called "block-scoped" variables (with ```let``` or ```const``` statements). But, I think, that even without extra language extensions we can safely emulate scoped variables in JS just by using ```var``` within a minimal scope we want to use the variable.
I know, there are various styles of the JS programming, but to me having all local variables declared at the beginning of function is very similar to declaring all variables as global. That's why I prefer declaring locals within smallest possible blocks of code.
But, well.. this is your project, and you are the master. :-)


Ah! This is just like we do in all our Python. I had just assumed gofmt would clobber such chunkings -- it's good to hear that it doesn't.


I thought there was already a class CFile for representing the file. Do we really need one more ?


Btw `var position;` seems to be excessive since you have it already declared in parameter list (but it could be just part of your code style, dunno).


This code is also not thread safe, assume if you get two write requests at the same time ?


This is for the Functions Chapter


Where's `var position;` used?


it wasn't working before, as I'm familiar with json dictionaries they like to have strings for keys. But if there's a way to do it as a generic that works too. I changed it because it was erroring out for me.


We will use a .sql file to store all the queries and run them in a method by reading the file.

It will be difficult to maintain the sql code this way.


SELECT
DB_NAME() AS DatabaseName,
t.name AS table_name,
updates.last_update,
updates.last_seek,
updates.last_scan
FROM sys.tables AS t
OUTER APPLY
(
SELECT
MAX(ddius.last_user_update) last_update,
MAX(ddius.last_user_seek) last_seek,
MAX(ddius.last_user_scan) last_scan
FROM sys.dm_db_index_usage_stats AS ddius
WHERE ddius.object_id = t.object_id
) updates


If you can avoid `else` branches then it's good to do it for readability.

```javascript
if (!req.isAuthenticated()) {
  return res.sendStatus(403);
}
next();
```


setting as a string input...


Ah wait it wasn't the latest commit (the one that you linked). All good then :)


Right. I will update that code shortly. That was just an early stab that was incorrect.

Benito J. Gonzalez - Unicon
bgonzalez@unicon.net <mailto:bgonzalez@unicon.net>
480.558.2360

> On Apr 30, 2015, at 10:19 AM, Drew Wills <notifications@github.com> wrote:
>
> I'm not positive, but I think this won't work the way it's intended.
>
> In uPortal groups speak, "Distinguished Group" means something special (and unexpected). The "Distinguished Groups" are the (essentially) hard-coded groups you MUST have, like "All Categories," "Everyone," "Guests," and "Portal Administrators." (I think that's all of them.)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bjagg/uPortal/commit/43c830d9e4174b0136c78fb82e02fb401dd06451#commitcomment-10989373>.
>


Rename the file to DbSqlLiteConnector, so that its clear from the name itself. And in future if we want to add any connectors we can easily extend the DbInterface and call it a new name.


SQL Query to return last_user_update, last_user_seek and last_user_scan for checking table usage in SQL Server 2005 DB's and above.


should be `.loader`


Are you sure your authentication works well? Here you need to use the `passport` object from `authentication/index.js`, the one that has all the configurations (e.g. local strategy, serialize, deserialize)


We shouldn't be extending a single property. The whole loader should be extended.


Is this a test code ? Why call save meta data in getAllFiles?


What if SVG is available?


@mugifly BDにレコードがない場合404を返すように変更しました。
上記ファイルでは、ステータスが200OKとなっていますが、こちらも404が変えるように仕様変更をしたいと思っています。


@ThomasBurleson This has already been addressed - 4000px would give us the same issues as 9999px, doesn't really matter HOW big we set it, we just want it to be bigger than the user's content.


extra space need to be deleted


Nice stuff man, Its lot of code :).

Follow are some of the major comments.

1. Thread safety issues.
2. Call Once for singleton.
3. Rename the file names and try to use the already existing code.


I think you can implement the logout operation in a similar manner to login and let the request go /api/users. This way we can get rid of this logout router. The logout operation is similar to login, signup so it should be in the same place.


nested if layout not organised


done


fixed it


No need to send the `true` value since probably no body is using it on the client. Just status code 200 should be enough.

`res.sendStatus(200)`


uno


gj, somehow I missed that :(


The unit here should probably be %


You can replace this line with:
```var chemicalReactions = require('../reactions/chemical');```


![screen shot 2015-04-30 at 10 28 28 am](https://cloud.githubusercontent.com/assets/64780/7418581/b31e65d4-ef23-11e4-8286-419ba425e26f.png)


I think this is the Sass you actually want to use
http://sassmeister.com/gist/ca61760f96d1f1b07d4a


Not the best commit message in the world,  considering how much this changes.


let's create a separate function named something like `handleDashboardPostsReq(req, res)` that will do the work from this if branch.

The same for the if branch with req.query.author


This commit seems to have no relation with `{not in Vi}`. Since I don't know anything about the state of Python bindings I'm not competent to review this commit :/


Let's create a function named `handleSignupReq(req, res` that implements this if branch and then call it here. Same for login.


A common question we've been getting asked is how to re-enable the behaviour experienced with 0.5's `unresolved`. I'm happy for us to drop the Polymer mention completely here and defer it to our own docs if that's preferable.


Thanks, I thought it might have been a hack to get around windows server
2008 throwing cannot allocate memory.

On Thu, Apr 30, 2015 at 10:06 AM, Nicholas Russell <notifications@github.com
> wrote:

> Nah, just a request from someone using the cookbook
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nfrussell/chef-selenium/commit/11c4650114b25ea8931659ef09787c94c85825a0#commitcomment-10989155>
> .
>


Mixing whitespace changes with code changes is not good practice. Makes commits much harder to review.


You missed an occurence.


...and if nothing else, it's probably better for us to be allocating the same size blocks with every request, rather than different random sizes ever time.

Thanks for the gist; I'll roll it in.


done


Don't include HTML or Sass in the documentation. HTML gets auto-generated, Sass is contained within the stylesheet.


Looks like an error.


Tab's too large


I think you don't need the `time-locale-compat` dependency anymore if you enforce `time >= 1.5`.


Folder should be `loading-indicator`

Look at the writing style of the documentation for the other patterns. Try and match that as much as possible.

Also, your tab size is off again


removed


OK, I see you fixed it.


The problem with default: return false: `var key = checkKeyCode(unknownCodeByTheFunction)`


extra space


yeah. lets go the simple way. we can always re-approach this if we need to later.


empty line


Replacing this with GroupService.findGroup(name);

Benito J. Gonzalez - Unicon
bgonzalez@unicon.net <mailto:bgonzalez@unicon.net>
480.558.2360

> On Apr 30, 2015, at 10:19 AM, Drew Wills <notifications@github.com> wrote:
>
> I'm not positive, but I think this won't work the way it's intended.
>
> In uPortal groups speak, "Distinguished Group" means something special (and unexpected). The "Distinguished Groups" are the (essentially) hard-coded groups you MUST have, like "All Categories," "Everyone," "Guests," and "Portal Administrators." (I think that's all of them.)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bjagg/uPortal/commit/43c830d9e4174b0136c78fb82e02fb401dd06451#commitcomment-10989373>.
>


@CrazyPants Khndrel Keght? How I should translate this? Like Dire Slime?


Did something in particular require specifying the type?


Okay, good to know.


works for me


aceptado


done, Is this the only field that is a special case?


Does this mean we would display "Warning: Staging" for dev and test environments as well?


http://editorconfig.org/ :blush:


Ok, the change is to replace grabbing all the person’s groups to check for the inclusive/exclusive groups in the test, instead using IGroupMember versions of the test groups and using IGroupMember.isDeepMemberOf(test_group) of the Person retrieved from GroupService.getEntity()

That sound right?

Benito J. Gonzalez - Unicon
bgonzalez@unicon.net <mailto:bgonzalez@unicon.net>
480.558.2360

> On Apr 30, 2015, at 10:16 AM, Drew Wills <notifications@github.com> wrote:
>
> I think we want to look at the isDeepMemberOf() approach.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bjagg/uPortal/commit/43c830d9e4174b0136c78fb82e02fb401dd06451#commitcomment-10989317>.
>


Please use a named constant.


You might review the screen capture above to see if you want to make any changes. Aside from that, I think the next step is to generate a set of rollouts. I recommend following this development path since it will help isolate any potential issues before we fully integrate the systems:

1. Use your current `return_val` as if it is the parameters requested by the web browser. This means you will ignore any parameters the web browser sends to the web server. We will worry about connecting the parameters after we generate and return rollouts.
1. Generate a set of 100 rollouts from the get_rolouts function.
1. Return the rollouts in the following format:

Abstract:

    [
      [{"variable name":"variable value", ...}, {"variable name":"variable value", ...}],
      [{"variable name":"variable value", ...}, {"variable name":"variable value", ...}],
      ...
    ]

This is a list containing lists. The inner list contains a series of dictionaries, each of which is a single event.

Example:

    [
      [{"timber harvested":100, "acres burnt":2},
       {"timber harvested":10, "acres burnt":4},
       {"timber harvested":10, "acres burnt":3},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":5, "acres burnt":3}],
      [{"timber harvested":100, "acres burnt":2},
       {"timber harvested":10, "acres burnt":3},
       {"timber harvested":10, "acres burnt":3},
       {"timber harvested":5, "acres burnt":8},
       {"timber harvested":6, "acres burnt":100}],
      [{"timber harvested":100, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":100, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":5, "acres burnt":2},
       {"timber harvested":1, "acres burnt":4},
       {"timber harvested":1, "acres burnt":3},
       {"timber harvested":200, "acres burnt":3},
       {"timber harvested":1, "acres burnt":3}],
      [{"timber harvested":100, "acres burnt":2},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":6, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":7, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":8, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":9, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}],
      [{"timber harvested":10, "acres burnt":10},
       {"timber harvested":4, "acres burnt":3},
       {"timber harvested":8, "acres burnt":3},
       {"timber harvested":1, "acres burnt":8},
       {"timber harvested":1, "acres burnt":100}]
    ]


This should now be fixed by 0755698455a1ee0f4441efbeece3a0739546e683


Other than the above small comment, LGTM.


RowMajorWithVectorization (amirroth) - x86_64-MacOS-10.9-clang: Tests Failed

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


"" = true seems weird.  Upstream already take this?


Ah, we only show the warning if it's in prod or stage..


@ahmad3m3 :+1:


For instance, why shouldn't {"", "false"} be false?


Thanks @learningNerd :)


it is giving me the status verbiage to be displayed and allowing the popup to change the status.


@shoghicp XD


You should be able to use chef_gem here. delivery_build is configured to install gems into the chefdk installation.


Did you move this somewhere? I can't find it.


I'll sneak in a fix in my next PR.


fixed


agraga un estilo y ademas dale formato


Where is this value getting ingested?


ah much better :D I should read the full diff instead of individual commits :)


Nice!
Doing a quick fix and upgrading all the dependencies on the fly I like the spirit :+1:


Did you get the Sync Gateway CORS config to work?


потому что на max_retries говорит ошибку, что неизвестный параметр


If you're worried, then add a permission check here.


this solves scsynth zombies for me


changed


By default the DSL will assume that a `@NodeChild` is the same type as the superclass of the class it's attached to - which here is `CoreMethodNode`, and we want it to be `RubyNode`.


Use the shorter "share" link (minus your personal SO ID) for linking more succinctly.


Yup already merged


I’d say

```md
* only write ***ONE LINE OF CODE***
* and still keep it readable
```


странно. а какая версия celery?  http://celery.readthedocs.org/en/latest/reference/celery.app.task.html#celery.app.task.Task.retry


I'm fairly certain this doesn't work on the ListAll endpoint (or any of the List endpoints from what I can see).  To wit:

```
% curl -H "Accept: application/vnd.github.drax-preview+json" "https://api.github.com/repos/google/go-github"
{
  "id": 10270722,
  "name": "go-github",
  ...
  "license": {
    "key": "other",
    "name": "Other",
    "url": null,
    "featured": false
  },
}
```

```
curl -H "Accept: application/vnd.github.drax-preview+json" "https://api.github.com/repositories?since=10270721

[
  {
    "id": 10270722,
    "name": "go-github",
    ...
    (no license block)
  }
]
```


Whoa this might destroy things


Now, there is no way to view full website in Mobile anymore :(


This code is meant to translate boolean query parameters to go objects. If you have a boolean flag in your URL... say "follow", you most certainly want it to be true even if you set it to blank: /pod/foo/log?follow=&container=bar


Moving the `assert()` down would still catch the `NULL` problem with `goto_tabpage()`, but it could also catch an error with `tabpage_close_other()`, so I would move the test to the dereference.


Ez egyszeruen nem ertem miert kellett ide rakni. Ha gondolod beszeljunk rola...


Given evidence below, maybe add a comment that this is only populated when fetching an individual repository with `Get`?


Use ThreadService.queue


ты не тот метод смотришь
http://celery.readthedocs.org/en/latest/reference/celery.app.task.html#celery.app.task.Task.apply_async


Thanks Seth. I guess I have never tested to 27 minutes at 10GigE.

So, in theory this should give us 27000 minutes before an overflow, and would be a good compromise for precision. I don't have time right now to test it, but maybe you can give it a try

    bytes_sec_X10 = ((stats->bytes_sent  * 10 * 1000) / diff_us) * 1000;

In this case I would be interested in seeing non-zero speeds for very short bursts, for example `--packets 1000`.


blank line :cookie:


* delay это короткий алиас к apply_async


nem gondolom


@mbebenita i squeezed out another few per cent performance
can you give me your fps now?

(demos are updated)


mkdir? also, wrong path?


Magistraal :+1:


локально с последними версиями селедки работет? надо имитировать падение таска и посмотреть повторится ли она со 2-3 попытки.


I'm not sure...

But anyway, the current upgrade-1.3.xsl cannot generate a validate cib, if there's any of the elements in the old cib like:

```
        <write ref="rsc1" attribute="type" id="admin-write-0"/>
        <write tag="primitive" ref="rsc2" attribute="type" id="admin-write-1"/>
```
They will be converted into:
```
        <acl_permission id="admin-write-0" kind="write" reference="rsc1" attribute="type"/>
        <acl_permission id="admin-write-1" kind="write" reference="rsc2" attribute="type"/
```
Which are invalid against acls-2.0.rng.

While with:
```
        <write tag="primitive" attribute="type" id="admin-write-2"/>
```
It'll result in:
```
        <acl_permission id="admin-write-2" kind="write" object-type="primitive"/>
```
The "attribute" is removed. But at least it validates against acls-2.0.rng.


Didn't need to iterate to 256, and switches that is in use for above modules is only from switches 0-3,
SOC-1 lowactive/verilog does MSB.


Haha thanks! Yeah those were quickly realized.


Afaik this will not work in sh, but is the way to do it in bash.
I do not really think we need to keep compatibility with sh/ash any more, but we probably should check that it works with other modern alternatives such as busybox.
I do not know of any other way of doing this that is more portable though, so this is just a comment on the fact.


@assaron How can we refactor `pull` and `commit`? They have lot of common things.


ну в доке ж сказано делать так, насколько оно рельно работает - уже аут оф текущий скоуп, это проблемы селеры )))


RowMajorWithVectorization (amirroth) - i386-Windows-7-VisualStudio-12: Tests Failed

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


That's why I posted it... So people can check if it breaks on their system... People who use alternative shells usually also know how to make workarounds.


julia.mit.edu went down yesterday, the buildbots needed a little coaxing to
wake up.  They're up now.
-E

On Thu, Apr 30, 2015 at 10:18 AM, Tony Kelman <notifications@github.com>
wrote:

> Just checking since I'd like to tag 0.3.8 today, is there something wrong
> with the osx builders that needs a swift kick careful fixing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/julia/commit/69e04da61f0137ae1294ba401ac4ba3a11b20451#commitcomment-10989344>
> .
>


Done, Fixed: 0521405ae04547b8db


Why is it 6 here, but 5 above for `ProgressBar::StepNitpick`?


I'm ok with merging this.


это легко проверить. сначала вызвать в коде экзепшен. после посмотреть что бы таск встал снова в очередь:)


I think this could be written as `{success_button || null}`?


Hopefully it will help :)


Both can be final.


Yeah that¹s one of the things I was talking about.  I removed the index
completely.  It was kind of an accident.  But not really.  I got frustrated
using the terminal and just deleted the index.html from github all together
when I was trying to commit the new version.

Jason

From:  Daniel Henriksen <notifications@github.com>
Reply-To:  lakeytom/showlog
<reply+00504b23cbcdc34f82a6d3ea298df1e55f42f9c43634784692cf00000001115a2bae9
2a163ce00a7b11d@reply.github.com>
Date:  Thursday, April 30, 2015 at 10:43 AM
To:  lakeytom/showlog <showlog@noreply.github.com>
Cc:  Jason Harris <jason.b.harris3@gmail.com>
Subject:  Re: [showlog] Delete index.html (ff71614)

Did you move this somewhere? I can't find it.

Reply to this email directly or view it on GitHub
<https://github.com/lakeytom/showlog/commit/ff716146d8befc5b44198fa5dca87188
213f07bb#commitcomment-10989853> .


(fixed some issues in this commit in 06e67dc)


`recordsInMeter`.


`recordsOutMeter`.


Ditto.


It's to create directory where the ARCH Makefile goes when building out of source tree. It is ignored when building inside source tree because of the -p parameter


http://mbebenita.github.io/Broadway/storyDemo.html runs at 228 avg now on my machine (chrome)


I think I can tolerate this bug if it requires sending traffic at 10G for 18 days. :smile:

With my patch...
```console
# ./src/tcpreplay -i enp132s0 --netmap -K --limit=1000 --stats=3 -t --no-flow-stats packets.pcap
Switching network driver for enp132s0 to netmap bypass mode... done!
File Cache is enabled
Actual: 1001 packets (685358 bytes) sent in 0.001237 seconds.
Rated: 554000000.0 Bps, 4432.00 Mbps, 809215.84 pps
Statistics for network device: enp132s0
	Attempted packets:         1001
	Successful packets:        1001
	Failed packets:            0
	Truncated packets:         0
	Retried packets (ENOBUFS): 0
	Retried packets (EAGAIN):  0
Switching network driver for enp132s0 to normal mode... done!
```

With the first change you just proposed...
```console
# ./src/tcpreplay -i enp132s0 --netmap -K --limit=1000 --stats=3 -t --no-flow-stats packets.pcap
Switching network driver for enp132s0 to netmap bypass mode... done!
File Cache is enabled
Actual: 1001 packets (685358 bytes) sent in 0.001239 seconds.
Rated: 553154100.0 Bps, 4425.23 Mbps, 807909.60 pps
Statistics for network device: enp132s0
	Attempted packets:         1001
	Successful packets:        1001
	Failed packets:            0
	Truncated packets:         0
	Retried packets (ENOBUFS): 0
	Retried packets (EAGAIN):  0
Switching network driver for enp132s0 to normal mode... done!
```

It looks like the results end up about the same with both options, but it does seem like there is an off-by-one bug in your --limit option.  Let me know if I ran this incorrectly so you could see the difference you were expecting to see.


Do you mean capitalization? Even though they are static constants?


line 1268 also need this change.


Heh. I wrote this sql in those history perf patches I never landed. It is not very efficient for cases where you know the ID of the site already.


We're already in sync with LA.AF branch. CAF doesn't start a brand new kernel when new Android versions are released, they just fork off a new branch. So as long as we're cherry-picking from the new branch, we stay up-to-date. Also, Christopher added a lot of memory related patches from LA.BF branch.

I'll take a look at that charging patch. Seems like it might fix it.


I mean renaming `recordsIn` to `recordsInMeter`.


minor nit... it's no longer just a map value


Well, if you're mixing velocity and js, make sure you escape things properly:

    alert("${escapetool.javascript($services.localization.render('phenotips.vcf.consent'))}")


... looks like someone is editing the files outside of Lively :-o


Sorry, pretty new in Rust, did what I know ;)


@tomviner Do you actually think this is worth merging, or too ugly / not worth it?


Nice.


Nice


see line 4


Gotcha. I thought it assumed `Node` if not specified.


thanks. fixed it in https://github.com/hayloft/material/commit/cb05f92f7882d94b23d78483bef16f118bea898c and also updated to v0.9.0-rc3


/!\ Seulement les structures sans les data (à venir si j'ai le temps)


permanent comments in English please. ... but I like the cleanup work.


:thumbsup:


yes, just like python! and fwiw check() is great, i like it - but yes it
was a bit hard to see the scoping at first ;)

On Thursday, April 30, 2015, Hunter Blanks <notifications@github.com> wrote:

> ...and if nothing else, it's probably better for us to be allocating the
> same size blocks with every request, rather than different random sizes
> ever time.
>
> Thanks for the gist; I'll roll it in.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hblanks/pixel/commit/2065e02e414dfe210dc6014918a5ed6d4793b449#commitcomment-10989602>
> .
>


This rel="alternate" works for me - however how does one change mod_language as it still adds the country code to the default value?
```
<ul class="lang-inline">
  <li class="" dir="ltr"> <a href="/htw3/index.php/tw/hernia-surgery/who"> 中文 </a> </li>
  <li class="lang-active" dir="ltr"> <a href="/htw3/index.php/en/hernia-surgery/who"> English </a> </li>
</ul>
```

the "tw" shouldn't be there.


Feel free to email me; `james _AT_ wetheinter.net`.  I will be busy this weekend with immigration and pushing this stuff to central, but will reply when I can.
The next effort will be a site to document the features and roadmap of the framework.  I also have a few places where I want to change how stuff is done, but that will be discussed in the documentation effort. :-)


Im hoping to get more usa canada and aussie channels.


Ischs denn besser Exceptions z'handle als sie z'verhindera? Maine in dem Fall chann z'Programm mit am Input ohni wProblem umgah.
Wers nid sowiso besser wenn dä Check scho bim REST Layer gmacht würdi?


Good call.


interesting trick


Ya están en producción los cambios!


@daliamaarek :+1: :cookie:


You are correct of course.


This bug has no sense, we should never enter in this part of the interrupt!


Што за такая цікавая прога?


Shoud be has_many :pokemons actually


If I replace the if / else if / else by a switch, the bug disappears, wtf?


Please submit this to the zsh-completions project.


Should be Pokemons controller


If I add a temp++ inside the buggy if paragraph, the bug also disappears!


This indentation should match the simple_form_for line


Extra new lines


New line at the EOF


10/10

Missed out on the inference that could have been gotten from using pokemons


Yes, `secrets` is a JSON dictionary keyed by strings. One such key string is 'SECRET_KEY', used in [our settings](https://github.com/grahamu/flyingcracker/blob/01af123c40ebcd3abd677875208fa801564dea25/fc3/settings/base.py#L7).

When you changed the code to ```return secrets['setting']``` you asked for the JSON dictionary value matching the  key 'setting'. That only works if the JSON dictionary has such an element:

```
{
    "setting": "jake had fun",
    "SECRET_KEY": "a;ldskfjg[0qrudjfvco[jq[difu[",
    "this_is_a_key": "this is a value"
}
```

Here is my secrets.json file:

```
{
    "SECRET_KEY": "fEk1OaxOhFrabjousDayCallooCallayM2Ict8y",
    "EMAIL_HOST": "",
    "EMAIL_HOST_USER": "",
    "EMAIL_HOST_PASSWORD": ""
}
```


You don't need the id to be an instance variable


You can use the hash syntax rather than the sprocket syntax


New line at the end of file


nodejs for beginners, deploy your blog to heroku, how to node http://j.mp/1OTO0qv

perfect css hexagons http://j.mp/1DiyQV5

anatomy of a javascript framework http://j.mp/1IOyRou

the absolute beginners guide to node.js http://j.mp/1GmtPiQ

best practices for building angular.js http://j.mp/1aSHUJt


New Line


2 space indentation


Yay!


Ok. We'll look at it tonight.


#53 has already been merged


You've done well, my young Padawan. :-P


@abyrd could you review this when you get a chance?


was this one obvious?


Oups, I indeed reversed the condition. Thanks for noticing. I pushed 31cc84cf4 which addresses this and makes the code a bit cleaner.


any way to get this back fixed into 0.8.3?  We're not on 0.9.0 yet because it has several breaking changes.
Thanks for the quick fix!

Date: Thu, 30 Apr 2015 11:16:30 -0700
From: notifications@github.com
To: material@noreply.github.com
CC: bob.manz@live.com
Subject: Re: [material] Add support for multiple dialogs for 0.8.3 (1650734)

thanks. fixed it in cb05f92 and also updated to v0.9.0-rc3

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


ok mate thats good


rax retry tests


Could these warning cleanups be part of a separate PR? They don't have anything to do with flow control and just make the flow control harder to review.


thanks!


root_url works too!


+1 Much more logical fix than I came up with. Thanks much.


I'm with @vinniefalco here, that this relies on an unfortunate side-effect of the `base_uint` default constructor, which is dangerous. I'd prefer using `uint256(0)` instead.


You better update R before submitting!


I have used it with Cordova 4.0.0 and Android 3.7.1, I don´t try it with Cordova 5.0 yet!


Thank you.


good catch!


I'll merge but please change this back to "description". we're fixing the bot for the error message


Still working through everything, thanks!

On Thu, Apr 30, 2015 at 1:34 PM, Karthik Ram <notifications@github.com>
wrote:

> good catch!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ropensci/neotoma/commit/8fe8fd634455a9de257efeb9e502b8d28a302f2b#commitcomment-10990837>
> .
>


2 space indentations


Difficult to get through because you commited the whole project but 10/10


I see that you thought about the nested class case, but do you test that explicitly?  I didn't see it in the tinkerpop/features.  You do test nested classes that are explicitly mentioned in package.json, but not via the package spec (as far as I can tell).


I think that we can use @copydoc right?


is this version stable enough to use? for some reason when running this it never seems to build anything even if I toggle it multiple times :s and another thing does the "require: False" for buildings and crafting mean it's not fully supported at the moment or something?


Removing these didn't cause any build or runtime errors anywhere in the code that I could see, so I think it's safe to say they can be removed


Wrong doc


I think this removes test coverage for the recursive package spec.


nit: `reload( { stream: true } )`


Maybe one of these packages contains a nested class, so you could add a scenario that looks for it.


Yeah.

There's an obvious and simple improvement here where we pull the ID out of recordVisitedSite and pass it through to here, changing `>>>` to `>>=` in `addLocalVisit`. But with an index this approach should be very very cheap anyway.

But note that this *is* more efficient than calling `getIDForURL`, which is what we did before in some cases! I suspect that the situations where we know the site ID are few or non-existent, because we're recording the visit by URL and type after a page load, right?


nit: don't align var values
nit: `require( 'browser-sync' );`


I see a big difference between the 2. I see a large error in the first example:

    685358 / 0.001237 = 554048504.446240905 Bps vs. reported 554000000.0 Bps

In the second example, the error is less:

    685358 / 0.001239 = 553154156.577885391 vs. reported 553154100.0 Bps


why don't you make automatic bit flush after writting any non-bit data?


`./scripts/git-version` doesn't exist in the dcadec tree (this seems to have been copied from `mkpkg_lockdev`, which does have these files).


nit: see `bundle.js` comments


Rename to `modernizr.svg-smil.min.js`


Add a unit test for this case.


same from above, I think this is already documented in the parent class


Tabs are still off


Currently this is not stable enough for a release. I have everything disabled by default so you may need to enable the "engine" to get it to start building. You may also need to do some cache clearing if you are using the javascript bookmarklet.


TeamCity Hyrise :: Unit Tests [Build 75](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=83&buildTypeId=Hyrise_UnitTests) is now running


Remove these comments plz


@copydoc?


Eating the error looks wrong


The cases happening here are a "possible no write after bit"


this endpoint is disabled


please move to utils


Log errors


Don't we have a setter for this?


We should just flip this to site.lastVisitDate or something (heck, we could even store it as a column and avoid the entire join here). I hate having it around at all.


@chadoh Think it's worth keeping this around? I know `force` wasn't doing this check, but that might be nice to have.


Surely you could have used `trigger` here? (that is what is stands for?) It's still shorter than `coincidence_trigger`...


then maybe also make flush of pending bits in sendpacket ?


We have config/cmd/bulk for this - I'd like to see this loop be representable using Bulk (even if it means changing Bulk slightly to accommodate not having a factory.


*applicable


Ah, nice.


It also makes `Scope#rename` really solid and can rename exports/imports so they don't lose their meaning. Previously it would change the specifiers/export names which would break programs :/ Which means there were weird scenarios that caused code to break previously.


No, Windows and Linux don't normalize at all.


@paulrouget This caused #346  You probably meant either `.set('updates', Updates)` or `.update('updates', Updates)`


why not $this->getUser() ?


Since we are (lazily) ending our board strings with a comma when calling to_s, the split will have 17 elements instead of 16. Not sure if that is causing other issues or not


Because you implemented `flash_attack` within the `ApplicationController`, and `PostsController` inherits from `ApplicationController`, you don't need to define `flash_attack` in `PostsController` anymore.


this reference to ./sabotage seems to be a leftover


The compiled version (which is on bower and in the repo at https://github.com/facebook/flux/blob/master/dist/Flux.js, and npm) is ES5 compatible.


do you have a timetable of when it might possibly be stable enough for a release by chance? :o and yea I've enabled the engine multiple times trying to get it to work when I was messing around with it before it never built anything even though I had stuff ticked to build but it crafted stuff some stuff. and not using bookmarklet ways of doing this, I have set up a userscript to run the javascript code on page load as I was getting tired of having to worry about bookmarks for this stuff (both the master version and this version disabling and re-enabling which ever version I wanna use)


Should these be Board.new.to_s()?


Oh no. We still need the visit count for frecency.


Ok, thanks

I will remove it and re push.

Thanks

Dan Williams, SIQ Agile Coach
CSM, CSP, PMP
Cell: 425-877-6333
________________________________
From: Jean-Denis Vauguet <notifications@github.com>
Sent: Thursday, April 30, 2015 11:46 AM
To: danielwilliams12/new_options
Cc: Dan Williams
Subject: Re: [new_options] Added filters to posts_controller and application controller (c8fb913)

Because you implemented flash_attack within the ApplicationController, and PostsController inherits from ApplicationController, you don't need to define flash_attack in PostsController anymore.

-
Reply to this email directly or view it on GitHub<https://github.com/danielwilliams12/new_options/commit/c8fb913bfcb9bf680d0cd76194ffc8c1fee39080#commitcomment-10991098>.


`splits = set(splits)`


Integrated with style fix as 95e88de0aab257020dfc33248b86331cbfac28b1.  Thanks!


error is not defined here and will cause an error instead of raising the proper one.


No, the parameter lists are different.


wrong doc


> This commit seems to have no relation with {not in Vi}

It's not, it's just that I didn't want to rebase it on top of a separate PR... but I should probably do that.


and so it begins ;-)


Fixed


I take it you took this spec file from Fedora?

I wouldn't bother supporting migration from old SysV init to systemd and drop the triggers.
IIRC we've been supporting this for a few months somewhere around F-15 when we were in the middle of moving to systemd. Later on we found out that it didn't work as it should and just told users that upgrades were not supported. I forgot to remove it when we fully switched to systemd.


I think it should list what is **not** working. People assume that evaluate expression allows everything that Scala can do. :)


All BaseProfileForm child classes so far duplicate this comment, even if they encapsulate defined aspects of the user profile...


don't forget this!


that strong length-16 bool though :)


Makes sense, sorry, I don't have a time table currently. I have the latest branch running for a bit in the background on an existing save. Once I test a bit more it should be ready for release.


Don't commit commented out code.


We're using this pretty consistently -- yes, a setter would be good, but I'll leave that rather than make this case inconsistent.


Sorry, I realize the list is a list of known limitations. I think it's good to have it here.

However, what is partial functions with more than one parameter? They are by definition `Function1`, so with only one parameter.


No really, don't commit commented out code.


Blank line


Is this a typo?


надо бы отдельную категорию сделать...


I don't know if this will do what you want (because I'm not sure if transactions are enabled like this for mysql). Also, it doesn't seem to be testable probably because custom error handlers aren't triggered from the test client, I think. What if we turned on per request transactions? https://docs.djangoproject.com/en/1.8/topics/db/transactions/#tying-transactions-to-http-requests


lack return doc