No idea, one guy tested it with a 9GB pcap and said it worked great. That's the largest I've heard it used with; I'd imagine it can go a lot higher than that.


does this work with all that whitespace? (template noob question i know)


I was taken a bit back by this; what was the difference between `.work-state` and `.work-status`

Could you rename the `.work-state` to `.work-processing-state` and also make change in the two HTML files?

- [ ] https://github.com/ndlib/sipity/blob/master/app/views/sipity/controllers/comments/index.html.erb
- [ ] https://github.com/ndlib/sipity/blob/master/app/views/sipity/controllers/works/show.html.erb


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


yeah, works like a charm.


(b750dd5) The title of the commit have a non authorized type.
 The allowed types are :
 * feat
 * fix
 * docs
 * style
 * refactor
 * test
 * chore
 * version


Has this been considered for inclusion in master yet? It's been a while...


(361a9e6) The title of the commit have a non authorized type.
 The allowed types are :
 * feat
 * fix
 * docs
 * style
 * refactor
 * test
 * chore
 * version


Noice. :+1:


s/exists/exits  :)


LGTM


Hi @ChanJoe11, please contact Hongsheng Li (lihongsheng AT gmail DOT com) directly if you have questions concerning this fork. Thank you.

There are some workarounds for segmentation data preparation.
- Write your own data layer that provides both images and segmentation ground truth
- Provide two data layers (one for images and one for segmentation ground truth) and ignore the "label" tops

Some built-in loss layers (SigmoidCrossEntropyLossLayer, SoftmaxWithLossLayer, etc. ) directly support pixelwise operations. So the network could be trained end-to-end without problems.


We can use QLocalSocket, if there is a pyqt binding for it. Failure to open the socket should lead to auto-renamed session.


update


We could but a dead code eliminator like ProGuard should erase all of it, if you never call PeerGroup.newWithTor.


ok


We need the campus name. Otherwise they all just default to Main. Is this data not available?


The UI has been looked over by one of our UI/UX people.  It looks good, however there is an issue with the button as far as our accessibility standards go.  It's using only color to indicate completion which doesn't work well for users with vision disorders.  We should probably add an icon or something as well to make it more clear that it has been marked as complete.


so why is this if statement here?  I don't think we need it


we should be able to just use this line


There is somehing wrong


.work-status class seems to wrap .work-state and .work-type. I will go ahead and rename work-state to work-processing-state


permit the topic here


missing newline


inconsistent indentation


I prefer use [depd](https://github.com/dougwilson/nodejs-depd) to deprecate the API.


think about this for a minute. It means local development isn't possible anymore :(


`0, 16` would lool better


ProGuard is not a scalable solution. Adding deps adds unnecessary build time.


Только не пойму, курсор (даже если не светится) стоит по умолчанию на нулевой позиции. И если пользователь не выберет строку - перемещается самая верхняя...


I've tested it in Kali Linux with a 70Mb pcap file, the result didn't come out as I exceped, missed a lot of such as POP IMAP. And how can I know the programe analyse the whole pcap file? Can set a endup word or flag?


Don't know. So far, I have to review the changes before I can commit them.


Subprocess ought to accept a list of arguments, so using ``" ".join(params)`` should not be needed - and worse might break if any of the filenames happened to have spaces in them.

Also you probably should be using ``.wait()`` or ``.communicate()`` as in the other subprocess calls in this file.


If you want to send a pull request, this commit should be in a separate branch, not in master.

Also, the first line of a commit message should be no longer than 5 characters. For example, it could be simply:

    Add `base` argument to `int` filter


Good points - rebased.


Admire,
I see there is still no link between the campuses and courses. Is there a way to get this from the data?
It looks like the entire data dump in fet_sample_data is a row for each campus course combination. We should be able to unbundle this and get the link?
Also the courses are added one for each line of the sample data. If we unbundle these we should find many duplicates...


My Kali Linux has a memory of 2Gb and hardrive of 40Gb, which is a VM.


* The javascript code style introduced in here is totally inconsistent with the ones used in other places
* We need to setup phantom and at least some kind of CI (at least for PRs)
* missing semicolons all over the place
* The `jasmine/` folder is totally needless nesting.
* I personally feel like jasmine isn't that trendy anymore. Happy to hear other thoughts but IMHO strange you start a new suite with it these days.


According to the SonarQube 4.3 documentation, this endpoint doesn't accept a parameter called 'resource'. For this use case I think it should use the 'componentRoots' parameter, which can be retrieved by a previous call to the RESOURCES endpoint (line 28), in the <key> element of the XML. See doc at: http://docs.sonarqube.org/pages/viewpage.action?pageId=2392181


valueOf(true) is a function which returns Boolean.TRUE.

By using Boolean.TRUE directly, you avoid the intermediate function.


Thanks @nikgraf


Hello,
I did a cherry-pick of your commit, and the test fails. Is there requirements I am missing? Could you check it is ok for you?
Thanks!


Did we considered well why we're using a makefile here? Seems like a pretty easy thing to do with gulp/grunt or even just npm scripts and this isn't cross-platform.


Minor comment: Since this might eventually be parcels, or MAZs...you might want to consider naming this something more ambiguous like Demand or other.


Here's my official request to bring the version number back! :-)

I can't think of any mobile more restrictive than an iPhone, and even there it seems there is plenty of space for the version number. I'm sure you had a good reason to implement this, but I just can't think of any platform where the TGD, Vault, Titan, Warlock, Hunter and hamburger icon would be a tight fit.


@MadsBuus don't know if this is a good approach or if we should do this using nokogiri or something


@emiapwil The resource-id should not be assigned by user in config. Instead, it should be parsed from network map json. So here it should be "my-default-network-map". What do you think?


Minor, but this is some weird alignment. Also, there's no newline at the end of this file (and a handful of others).


Why is it not scalable? You only care about dependency sizes when actually shrinking for production devices, right? So it shouldn't need to be run all that often. And I don't think it makes much difference to the build time of bitcoinj, at least I never noticed it doing so.

If we don't include this dependency, I'm not sure that PeerGroup will compile any more. It's probably possible to rearrange things to fix that, but over time bitcoinj will get more dependencies as it gets more features. It just seems inevitable.


Wonky alignment here, too.


Great pull request, could you also add this to tests/all_parameters_enabeled.

And what are the default values that gitlab provides?


how i can use the whole Mail Stuff NOW?? I need something like SMIMESignedGenerator what i have to to on Android?? Thank you...


Why does the minified code live in the repo? Bad practise, confusing and unmaintainable IMHO.


What is the reasoning to replace tap with tape? Just curious.


Hi! I found an error in bntRadio directive. It's need to change for this line:
```javascript
if (!isActive || !angular.isDefined(attrs.uncheckable)) {
```
This allow the radio to be uncheckable.


Should this be "timelineIds" and not "userIds"? Also "userId" below.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.445](http://teamcity.monogame.net/viewLog.html?buildId=6508&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:20


> Why function inlining gives such a huge boost?

Inlining is more likely for small functions (there are other heuristics across engines but that's the main one). JITed code is built on the profiling and assumptions of an earlier interpreter and/or base JIT. A function invocation is a blind spot for a JIT, lots of arbitrary code could be executed to invalidate assumptions the JITed code was based on. Because of that some optimizations are avoided in the presence of a function invocation. Inlining improves performance here so much because it removes the overhead associated with the function invocation and can then apply other optimizations (various hoisting and type specializations) similar to if the callback was written directly into your iteration loop. Because the operation you're doing is trivial (the overhead of a function call shows up more).

 The way lodash handles [lazy evaluation](https://github.com/lodash/lodash/blob/master/lodash.src.js#L1306-L1364) nixes inlining because the callback varies from iteration of its inner loop. I may kick that code around a bit to see if I can get inlining to work but it really only benefits tiny functions (those that show up in microbenchmarks). Lodash does still benefit from inlining in the non-chaining API but won't get lazy-eval/shortcut fusion (the big win).

> I have been playing with the last example you showed me and I noticed something interesting if I place a second parameter to the `filter(name)` function lodash again becomes slow. Do you know why?

Yes, this is by design. lodash 3 tries to have it all. By that I mean the same chaining syntax devs are familiar with "just works" and gives them perf wins. If a dev provides other arguments to a callback then shortcut fusion (cutting out the iterations and intermediate array creation) is observable and users have reported issues: "Why does my code work when not chaining but breaks when chaining?". Because of this lodash only applies the optimization if the callback has one argument (which is common for those taking advantage of it). lodash's chaining is deferred until an explicit `.value()` or implicit `.value()`, like with `reduce`, is called. Every chained wrapper has its non-lazy and lazy metadata so if lodash has to jump out of lazy eval because of the arguments provided or the result is a non-array then it's doable.

> I am thinking of implementing a built-in mechanism for inlining function in the generated code. What do you think about that? Have you thought on that topic? It's impossible in all cases but it may be beneficial in some less complex. Do you think it can give a performance boost?

It may give a boost here or there but I prefer relying on the engine to handle things because they can perform nested inlining (func "a" is inlined into "b" which is inlined into "c") and specialize for them. My approach is to keep my external exposed functions small, handling any optimization disqualifications there, then hand it off to internal functions which can be inlined or further optimized without disqualifications (works pretty well and doesn't require dynamic function constructor or manual inlining).


If we need to do this more often, we can do that. I'm not going to pull in another dependency because of this very simple one-time need, which we will remove pretty soon.


@pazdera There is still code here with an interval of 1 day, is that still invoked? If it is it would conflict with the above.
```
/usr/share/kano-desktop/kdesk/icon-hooks.sh:            sudo /usr/bin/kano-updater check --gui --interval 24 &
```


Yep, it's should have been timelineIds. (this is copy-paste code, so there is one more place to fix variable name).


I think this name is confusing. "getSubscriptions" implies "everyone who subscribes to this post" or "everyone who will see this post in their default timeline". This is very much not the same as "the groups to which the post has been published".


Do you have a special reason to do the refactoring (deleting name parameter) and the addition of the new username-parameter in the same commit ?


Thanks for the heads up. I've been busy. Will look at commits this weekend.
I can say I'm a bit confused. I thought that I saw a commit written by you
that accomplishes string primitive keys, so I'm not sure which functor of
mine is needed. I did write a functor for the same purpose but I thought it
was needlessly duplicative of too many lines. Anyway, as I said, I'll look
closely this weekend, figure it all out.
-Paul
On Apr 28, 2015 11:23 PM, "Carmelo Piccione" <notifications@github.com>
wrote:

> @pat227 <https://github.com/pat227> I jumped the bullet and in merged in
> a functorized version of your primitive key strategy. Check out this merge
> and see if it suits your needs. I took out your logging for now (want to
> discuss as separate PR), but feel free to leave it in your fork.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/struktured/riakc_ppx/commit/3ad55dc3ea2d8a828f9997b205cfeac9c5d84061#commitcomment-10955837>
> .
>


You could use our media mixin:
```@include media('<=phone') { }```
```@include media('<tablet') { }```


`spec=Application`


The code works fine, but this loop is unnessesary. Array.prototype.sort() handles the iteration for you


Monkey-patching `callback_url` in a `test` environment breaks `OmniAuth.test_mode`, thus making all the integration tests fail. Hence, our test code can now ensure OmniAuth has defined `callback_url` without resorting to any shenanigan.

/cc: @exalted


Any suggestions? In this case "subscriptions" means "there are timelines that are subscribed to that post"


yes...


Didn't need to create a new branch, even though I should've, but this was the only thing I needed to fix in the css.


The variable declarations are at the head of a code block, so it's OK (even in C99)


We are one


Now that you have tagged 1.0.1, should F-Droid release this APK as the default instead of 0.9.30f ?


But that's not what we need to send to the client, isn't it? A single user has no reason to care about all other users who will see this post in their timelines. They only care about the authoring user and the groups.


subscripted nodes?


Hah try getting it in master within a workable timeframe! ;)

At least it's out of the i10n repos and in one shared place now, I figure
we can bugfix it etc and patiently wait for the day when  that it can
finally go in master. Also someone will need to change the demo au data for
dev builds and whatever else and I seriously didn't have time or
inclination to finish that whole package, so this was an achievable middle
step.

On Wed, Apr 29, 2015 at 6:40 PM, Paul Mackay <notifications@github.com>
wrote:

> Isnt seeds.rb something that needs moving back into OFN project?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openfoodfoundation/ofn_deployment/commit/2f606f14b0b980364e152995ecf05fc8ed67fe91#commitcomment-10958992>
> .
>


isn't this really part of an unrelated ticket?


If you add the following condition at the begining of the index function then you wouldn't need the else part.
if(!$this->session->userdata("user_id")){
  redirect(site_url('login'));
}


Ja, das ist der letzte Stand der constraint api


@jeffbrown I fixed the broken test and the build is passing, but what I really want to do is simply add the `AutoTimestampEventListener` to the tests.  I can't seem to figure out how to add that event listener to `grails-datastore-simple`.  Please let me know if you have any suggestions.


yes we can ...


Apparently, I was not clear enough. It sends to a client only timelines that include this post, e.g. My RiverOfNews timeline, My Posts timeline, GroupA Posts timeline, etc.


In my opinion the new password option should be a sepperate commit ?


University of Washington


William Blake


Das liegt daran, dass ich die Klassen geschrieben habe, bevor der Rest portiert wurde. Wird demnächst aufgeräumt.


armées


armée


to not listen for vertical scroll for.


Version 2 Created


yes we can ...
but don't change ...


Seriously? Should my team be putting "SLAC" next to copyright or this is part of UW contract?


Yeah, I never meant to update the version string, that snuck in from some
other work. Separately, it appears that Conda's SAT solver ranks a
'nightly' build strings over numeric build strings, which is interesting.

On Tue, Apr 28, 2015 at 8:05 PM, Anthony Scopatz <notifications@github.com>
wrote:

> I am 95% sure that this is a problem. --use-local only applies if the
> version requested is present locally. What this commit does is change the
> locally built package from being named cyclus-0.0-nightly.tar.bz2 to
> being named cyclus-1.3.0.tar.bz2 (or similar). So conda install
> --use-local cyclus=0.0 will go out to binstar rather than using the
> local, just-built package. Also, Travis won't necessarily show an error if
> the v0.0 on binstar happens to be compatible with the tests in the PR.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cyclus/cyclus/commit/a9084f97f070723ab8900283e6d3cf7df98993d2#commitcomment-10954749>
> .
>

--
Matthew Gidden, Ph.D.
Postdoctoral Associate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph. 225.892.3192


So, my original implementation still wrapped the string in a protobuf encoding (not a big deal but thought it was undesirable for you because of visibility or performance reasons).

This new one captures the spirit of your implementation by providing more a more abstract form of serialization. See the type ```Protobuf_capable.Raw_S``` :

https://github.com/struktured/riakc_ppx/blob/master/src/lib/protobuf_capable.mli#L13-L18

which allows ```Core.Std.String``` to directly implement the key signature without protobuf encoding.

The api didn't break however, as I made a functor called ```Protobuf_capable.Conversion``` which converts a ```Protobuf_capable.S``` to a ```Protobuf_capable.Raw_S``` automatically.

Your desired version with a key as a primitive string is now done with ```Cache.Make_with_strring_key```:

https://github.com/struktured/riakc_ppx/blob/master/src/lib/protobuf_capable.mli#L13-L18

Hope this clears things up.


hi how are you


The OS is Ubuntu 14.04 on an older 32-bit Intel CPU.

In your CharLMMinibatchLoader.create()

A work-around I added after the code for cutting off the end so that it divides evenly fixed the contiguous problem.

     -- FIX: added check if contiguous
    if not data:isContiguous() then
       data = data:contiguous()
    end

An easy one for convenience sake. Add a semicolon at the end of this line:

ydata:sub(1,-2):copy(data:sub(2,-1));

I copy/paste code into the console while in th> to quickly check things and got memory overflow.

That's it.

No more problems so far.

Thanks. Mark.

-----Original Message-----
From: Brendan Shillingford <notifications@github.com>
To: oxford-cs-ml-2015/practical6 <practical6@noreply.github.com>
Cc: mszlazak <mszlazak@aol.com>
Sent: Tue, Apr 28, 2015 6:04 pm
Subject: Re: [practical6] minor typo in PDF (96749c8)

What is your OS and CPU architecture? Make sure you also update torch, in case you have an old installation. There might also be issues from having too new of an installation, but that doesn't seem to be the case.

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


1: M111 unrecognized component name: formatter


http://jsonformatter.curiousconcept.com/ ou whatever Vim magic next time


Why is this needed?
It might be worth fixing, since it can provide a decent speed improvement.


Looks very good, thanks! I am still building packages and will hopefully get around to test these changes especially on XP tonight.


@gautric @geertschuring Guys, I've removed the stub of the documentation of some components. I'd like this page to be always neat and clean, so we can send links like this one [1] to the public. Please try to avoid keeping unfinished documentation here, so our reference page always looks cool :)

[1] https://github.com/camel-labs/camel-labs/tree/master/iot#camel-kura-wifi-component


No whitespace before `:`


I think we could use "mocha: true" instead of most of these globals now :)


Should I have introduced this new variable only for HAVE_LIBGMP? See commit https://github.com/magnumripper/JohnTheRipper/commit/972c88a943db3df8160d26f454f810f2493f526a for Clipperz format.


`{ alias: { 'bootstrap' } }` -> `{ alias: 'bootstrap' }` maybe?


For theme colors follow the pattern in [framework_and_overrides.css.scss](https://github.com/ndlib/sipity/blob/master/app/assets/stylesheets/framework_and_overrides.css.scss#L10)

So in this file line 128 would be:
`color: $work-status-color`

And add the following to `framework_and_overrides.css.scss` on line 11:
`$work-status-color: $gray-light !default;

This allows the color selection to be overridden in the default theme.


> Please feel free to change me to Contributor status for this package and add yourselves as Maintainer(s).

Thanks for the offer! There are still two bugs that are bugging me which I want to address first, then get another dev preview out, and then we'll see!


See RFC-45 for details -- https://jira.lsstcorp.org/browse/RFC-45

But note that it's still "proposed" for now, so there is no "should" about it (yet).


Yup, but this is not approved yet :)


Yes that's a typo.


Why we have pro sniper here. Is not in the non-gaming DB


Should I have defined done() only for HAVE_LIBGMP, and used either done or fmt_default_done depending on HAVE_GMP?


why don't you say anthing


The contract situation looks pretty clear (irrevocable license from contractors to LSST but no copyright assignment) but @jdswinbank is correct that the RFC has not been Adopted yet. Either way, previous instructions were for "LSST/AURA" and not "LSST Corp".


Missplaced backtick


Yes, and I find this to be incorrect: we only need the Posts timelines.


No. It gets overridden in a few places...but usually should do nothing.


I did not expect such a detailed explanation, @zerebubuth thanks a lot for your comment!
The reasons and the decisions behind this code have become way more clear


War das echt so geplant? Also vorallem weil du auch noch beginLatencyConstraint in beginLatencyConstraintBefore umbenannt hast, was ja jetzt eigentlich schon für die Vertex spricht. Naja, können wir morgen noch mal drüber reden.


i would suggest keep this default as firefox as this is our main target.


fixed, anything else?


except require, requireApp and loadBodyHTML


@dalgaaf to render the ceph doc does not involve nosetests, so probably it's not a must for a document writer ?


it seems that two files have been renamed.
next time us "git mv oldname newname", as it helps keep track of changes while here an other time is impossible to visualize immediatly what is changed without doing a manual diff.


Also JobGraphSequence != JobGraphSequence wird chaos^2... Wenn da nicht spezial Änderungen dran sind, wegschmeißen und die andere nehmen.


:thumbsup:


It is now 3.6


@mikand : I revised the example. Please make a squash commit for this example and then open a PR with a single commit adding this example to the repo.


Eigentlich habe ich es genau deswegen unbenannt, weil es für die Edge gelten sollte.


I have so much fun reading CodingStyle changes


**Sorry** ....

![Alt text](http://www.gogaminggiant.com/wp-content/uploads/2010/07/facepalm.jpg)


TBH I don't understand the need to have this here :) I think test/unit/.jshintrc is good enough, we don't need a separate one in each directory?


Still not as funny as add_constribution()

Sent from my iPhone
401-523-5780

On Apr 27, 2015, at 5:04 PM, David Thibaudeau <notifications@github.com<mailto:notifications@github.com>> wrote:

misspelled, conpute_intensity should be compute_intensity.

—
Reply to this email directly or view it on GitHub<https://github.com/campreilly/UnderSeaModelingLibrary/commit/af1c7a870374a3dfd60ec33636490a6fec1094f6#commitcomment-10926440>.

The information contained in this message, and any attachments, may contain privileged and/or proprietary information that is intended solely for the person or entity to which it is addressed. Moreover, it may contain export restricted technical data controlled by Export Administration Regulations (EAR) or the International Traffic in Arms Regulations (ITAR). Any review, retransmission, dissemination, or re-export to foreign or domestic entities by anyone other than the intended recipient in accordance with EAR and/or ITAR regulations is prohibited.


Postgres support isn't well tested since it was added entirely by a 3rd party.

Before the fix, the logic was
```ruby
p = false
m = false
foo = ( true ) and ( (p) or  !(m) )
```
postgresql_enable => false
mysql_enable => false
#true

postgresql_enable => true
mysql_enable => false
#true

postgresql_enable => false
mysql_enable => true
#false

postgresql_enable => true
mysql_enable => true
#true

After the fix, the logic is
```ruby
p = false
m = false
foo = ( true ) and ( (p) and (m) )
```

postgresql_enable => false
mysql_enable => false
#false

postgresql_enable => true
mysql_enable => false
#false

postgresql_enable => false
mysql_enable => true
#false

postgresql_enable => true
mysql_enable => true
#true

So if any of the ci values are set, you can set either postgres or mysql but not both. That sounds right.


/cc @akre54


This is just preserving prior semantics...so I'll stick with it for now. also fixed.


@mikepetersonccv we have 287k check transactions that I had imported on our beta server and this is taking a really long time to update.  If I stop the migration while it's processing everything is that going to corrupt the data?


I'd respectfully disagree with this. Posts timelines is how FrF works by default, it explicitly shows only Posts feeds, however if user subscribes to user Likes or Comments FrF will show them as well. (though it introduces one of our differences with FrF, i.e. in Pepyatka when user subscribes to another feed we automatically subscribe her to Posts, Likes and Comments timelines, however FrF subscribes her only to Posts).


Will do, thanks. =) I wanted to leave the possibility for a few more changes open before I actually upload the 0.1.0 version to pypi, so I held off on the tag for now. I'll add the tag to whatever version gets packaged up to pypi.


gone.


Yeah, it's not very trustworthy when different doc pages give different
information, which again is different from how it works in real life when
being tested. Well, that's PayPal for you I guess. (And actually, I see it
happening quite ofter for different payment solution providers as well.)

The bottom line is - don't blindly believe the documentation, test
everything yourself to avoid nasty surprises. ;)

M

On 29 April 2015 at 14:44, Cédric Albrecht <notifications@github.com> wrote:

> Weird... From now on I'll be wary of the PayPal's documentation !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maciejzgadzaj/commerce_marketplace_paypal/commit/9a38d8fd3585e10bf023f41bd8f17c9ef83d45a8#commitcomment-10962642>
> .
>


You rock man. Thanks


The three queries are `begin;` `insert into ...`, `rollback;`


반환형 선언하세요 컴파일러 필요없는 아트블론이여....!


this is my commit you know!


1: M111 unrecognized component name: fixtures


copy&paste - error: line 376 is obsolete :)

apart from that the driver works fine - tested it locally.


You can remove all of this if the metadata is still maintained in Git


shouldn't it be changed in devices-8960.c ?


To enable all costume item's stats and bonuses to effect players remove or don't commit "src/map/status.c"

Remove these: "i == EQI_COSTUME_TOP || i == EQI_COSTUME_MID || i == EQI_COSTUME_LOW" on line 3098


No, it won't corrupt anything. If you want, you could manually run some SQL to set the TransactionCode to some non-empty value, which would effectively skip all those when migrating


Why not just use dict type for request and reponse here? I think it's over-complicated to define a customized class and its attributes dynamically.


Do we want to change also plugin and XML namespaces?

Plugin namespace change can cause compatibility issues with 3rd party plugins or plugins from old versions. I don't think any of these scenarios will/should happen.

XML namespaces are just formal thing, they are not used anywhere in the code, so only drawback is that it's a lot of files to change. They are only used during XML validation against XML schema.


I found a bug in my code by running the edge test case you pointed out :) . I was forgetting that when nRemainingNulls == 0 there is no byte written to the array. I will fix it with the next commit.


Would it help if induction was on sizes of derivations instead of derivations themselves?


out of curiosity, why "-j3" instead of -j2 or -j4?


Should I set TransactionCode to the check number?


It doesn't compose and it forces a threading model that you don't always want (subscriber on the UI thread).


My main problem with this commit is the commit message.  It should be more like "3 stylistic changes and some code simplifications.".  AFAICT the variables were initialised and the optimizations make zero discernable difference to performance.

I would also like a comment with the change to ModuleManager.cpp to say what the new code does.

If this were a patch, I would reject it as an optimisation and a fix for uninitialised variables.  If it were a patch and presented as stylistic change and simplification to some code, I would comment on it on that basis.  Dan, I suggest some discussion on audacity-devel, if the plan is to add more and build this up into a pull request.


Say whaat?


Where's this CloudKit.momd thing?


my compiler did not recognize an IF block as a proper block. It would have been fine within a WHILE, DO, CASE or anonymous block, but not in an IF.


Part of my intention was to create an identifier that could be tied to other systems, like DNS records or Namecoin identities, or anything else.


1: M111 unrecognized component name: formatter


mind you, it could have been C90 rather than C99. Maybe a --pedantic switch would enable that warning


An interesting ticket on this : https://caremad.io/2013/07/setup-vs-requirement/


oh for heaven's sake, what are you using visual studio? :)
or are you compiling -pedantic :)
I'll move them to the front, mumblegrumble


This is the commit to inject the checksum into the source stream for encryption and removing it from the data on decryption.  @ddanderson please review the changes specific for this.  @keithbostic just FYI for your general code review.


1: M111 unrecognized component name: fixtures


@jlucerond done with my review. checkpoint complete. if you have anything in this commit you want to discuss with me please make a note of it. press on!


is this right fix?


Looks like there is confusion between two things: "where was this posted?" and "why do I see this in my River of News?". For the first question, likes/comments are irrelevant: only the posting user and the groups are. For the second, even in the original FrF, I can see a post which I'm not subscribed to but which my friend liked. I tend to think that it's best to send those two pieces of information independently.


Could you please fix the indentation to match the rest of the file


I'm actually not getting the warning, just spotted the problem by heart :>
I guess I used to build with -pedantic


@ajoslin - thx


This commit is very old (Nov 16, 2013). UrlPath class (and the whole plugin) was totally refactored in the last version (1.4), making it compatible with SonarQube 4.x. You should checkout the master branch.


In the docs the city passed in the near filter uses double quotes (https://developers.urx.com/api/search-operators/).

Just want to confirm that this works with single quotes.


@WouterJ is this really needed to be committed? or was that a mistake?


Hopefully this is clearer.
```
+    # No bytes specification is described in the man pages. A number without
+    # any suffix is interpreted as indicating the number of filesystem blocks.
+    # A suffix of "s" specifies a 512 byte sector. It is omitted here because
+    # the lookup is only by standard binary units.

```


done!


Done.


Both https://github.com/tibbe/haskell-style-guide/blob/master/haskell-style.md#indentation and http://snapframework.com/docs/style-guide advise using 4-space indent and 2-space ``where``. @taphu doesn't like 4-space indent because his VT100 runs out of columns pretty quickly, so he's also proposed 2-space indent and 1-space ``where``.


it was required at first (as we were caching `/vendor/`). Now we are caching `~/.composer/cache`, it isn't required anymore. It shouldn't harm anyone though :)


doh! I could have left it...


LGTM


[Pedantic] Parens not needed here.


shouldn't this go in devdocs or something? this seems like it's very much wip still and is pretty misleading to have as a top-level file


Thanks, that solved the problem.


Is it not easier to just do

```C
setShouldReloadScript(should_reload_script);
```


This is a very good point that "where" and "why" are two different things. I'll take a pause on these matters for a time. How do you think "why" could be separated?


=> Reset password


If you're going to use PHP_SELF then I suggest you use something like
<?php echo htmlentities($_SERVER['PHP_SELF']);?>
Although I still wouldn't recommend using it at all.


Is is possible to end up with a `//` sequence?


Yes, but does it give any errors during compilation?
I don't see anything obviously wrong with that function, maybe we are using something that needs a workaround on MSVC.


Fixed #4


okay


the reality is, the original code setShouldReloadScript() shouldn't even need a parameter, since if you call it with 0 it won't do anything.


Oooh! I'll see what i can come up with. I do use twitter. what's your
twitter name? i can add you! Mine is mienaikoe i think?

On Wed, Apr 29, 2015 at 1:18 AM, uniaquinas <notifications@github.com>
wrote:

> I think I'll go. Also this Thursday there's this thing called Pay It
> Forward Day, and you do random act of kindness and then tag 3 friends with
> #tagyourgive so they can do random acts of kindness and so forth. it's
> going to be done on instagram and twitter if you use those. But definitely
> tell others about it, to do good stuff.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mienaikoe/lupur/commit/ea610e76109f40b440a3c287a0136ecde357d1a7#commitcomment-10956564>
> .
>


@doudou I did not find rock_gazebo/resolve_worldfiles_and_models_arguments. Can you see this for me?


Uff, yes... Time for Feierabend...


@kalashnikovisme @Khelek  https://robots.thoughtbot.com/rendering-collections-in-rails
мозговзрыв про то, как можно рендерить коллекции)


The style checker is pretty simplistic and will miss this particular case.  You'll see other instances like this in the code which were overlooked.  Anyway, I can fix this in the merge.


Right.  That would make sense given the name of the method and the parameter.  If you changed the name to something like `lua_reload_script()` then it would make sense without the parameter.


I just supposed I was badass and capable of editing valid JSON withoud verification but it appears I'm not --'


why ? let's be pedantic ! I'd actually add -pedantic to the bots build (\cc @robe2).
it's a long time I don't build on old *bsd machines and even longer on solaris, it's useful to stick to ANSI C until we decide to switch to something more recent...


> Applications built with portability in mind don't rely on absolute paths.

I'm not really sure of this assertion, where I work (a french newspaper FYI) we cannot do a reverse without the domain, because of the architecture of the project with multiples instances on different hosts.

After a bit looking, this is the contenttype app who add the domain in the url.
django/contrib/contenttypes/views.py def shortcut

Note that get_absolute_url() who does not return the real absolute url, but the relative, is an old debate, and when reading the pro/cons, this problematic is not that simple to solve.

> Is it a realistic scenario that with a multi-sites setup we enter a comment on one site and receive the answer on another one? If yes, is this setup also possible with Zinnia? (Don't Zinnia and its commenting have to live together on one site?)

Yes if you have a farm of sites and a centralized comment system. And yes this setup is possible with a subdomain.

> Would you accept such a pull request?

Nope, this behavior is certainly intended, and I don't want to patch/modify results coming from another generic app.

The best solution is to open a ticket on the django_comments repository for having more informations on this behavior and why not proposing this change.

Regards


Maybe leave this "getSubscriptions" as "why" (and leave the client to sort out which of these to show) and add another method for "where"? Note that FrF showed you that a post was cross-posted to a group even if you weren't subscribed to that group (and even if the group was private and you couldn't see its contents at all).


Well, one option is for `make dist` to just ship with the PDF.


sure, if you know what it is :)


oops


done.


fixed in the commit


LGTM modulo comments


@kalashnikovisme @Khelek я бы не коммитила пустые файлы тестов... вот так заходишь в проект, видишь кучу файлов в папке test и думаешь: "ну отлично! тесты есть!" а их на самом деле нет =/


It was my understanding, that @LeaVerou prefers simple things, and didn't wanted to use npm/grunt/gulp just yet. If I recall, I saw a bash script with call to closure compiler service in some other @LeaVerou repo, so I created a Makefile as very simple tool for contributor (no npm/grunt/gulp installation or knowledge required).


I've rechecked, the `->` syntax comes from 1.9.3 so feel free to use it.


I actually get the following error when I try to compile. :(

```bash
submodules/closure-library/closure/goog/base.js:236: ERROR - illegal initialization of @define variable goog.DISALLOW_TEST_ONLY_CODE
goog.define('goog.DISALLOW_TEST_ONLY_CODE', (COMPILED && !goog.DEBUG));
```


I looked around more and the luaTask uses the parameters in both ways, so I'm inclined not to change the signature. Can streamline this call, then.  So it's a question of overhead of the if statement or the call stack, in addition to the visual comprehension in the code.. :)


I have a feeling that we might end up having "where" pretty much the same as "why"?


Can you `from constants import DEFAULT_DOCKER_API_VERSION`?  This broke my use of the client since it was using this constant to determine which version of the API was supported.  I have to support both pre-1.2 and post-1.2 versions of docker-py, so updating my code isn't simple.


> Anyway, I can fix this in the merge.
>
Ok, thanx.


@astorije, I wasn't suggesting particular versions of Node.js or npm. Those versions I mentioned in the files are just the versions currently used in production. So, if we decide to upgrade the production servers to different versions, OK with that too — let's just make sure we mention that in the documents, and that we at least try to reproduce bugs and test releases against *those* versions. Simply to rule out issues related to us using different versions of Node.js and npm.

I don't think this is a problem for contributors. Most of the time, people will use whatever versions they happen to have, and things will work fine. My idea is only that the core team (those releasing and deploying) would agree on those versions (at least on the *major* part of the version!).

That difference in the behaviour of `npm dedupe` is an example of why it'll be good to agree on particular versions. That one isn't critical, since it's an experimental feature we don't rely on. But we have seen npm apparently doing things differently in different machines (and it might because they were different versions).


Version 2.8.0 (511) keep crashing while selecting profile or group photo from gallary


Well you could create a wrapper call to fix it and use it here if you want?  Or you could use it as is.  This is a very minor thing and I think we are starting the enter the realm of bike shedding at this point.  Do as you see rationally fit.


Perhaps more concise, you could build up the frame values array with a `for` loop and `-[NSString stringWithFormat:@"astronaut_%@.png", [[i stringValue] someMethodWhichLeftPadsZerosToFiveDigits]]`.


Can you just catch the ImportError?
On Apr 29, 2015 9:05 AM, "Kevin Stone" <notifications@github.com> wrote:

> Can you from constants import DEFAULT_DOCKER_API_VERSION? This broke my
> use of the client since it was using this constant to determine which
> version of the API was supported. I have to support both pre-1.2 and
> post-1.2 versions of docker-py, so updating my code isn't simple.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/docker/docker-py/commit/ae329cb3acb3ff38bd986215c99b1136a6183f78#commitcomment-10966994>
> .
>


il manque le fichier realloc_securisé.h et realloc_securisé.c


`-[NSString stringByPaddingToLength:withString:startingAtIndex]`


Oh right!


Has been streamlined.


Ok cool :)


This looks like a typo.


this not an equivalent of !("metadata_json" in data) because this.model does not have such a field: it is in this.model.attributes


It worked in my tests.  I can change it to double quotes though if that's preferred.  I'm not sure why I used single quotes--probably to avoid escaping.


That's reasonable.


No worries at all!

I think that is indeed my github username (though I haven’t used it in a couple years)!

-Jen

From: Jennifer Sun [mailto:notifications@github.com]
Sent: Tuesday, April 28, 2015 11:43 PM
To: sunnifer/learningtogether
Cc: jennifersun
Subject: Re: [learningtogether] hello world additions (bacb43f)

Hey there - weird, thanks for letting us know! By the way, is your Github
username "jennifersun" by any chance? If so, I think I might know what's
going on. Apologize for the random messages!

On Tue, Apr 28, 2015 at 1:55 PM, jennifersun <notifications@github.com>
wrote:

> Hi there,
>
> I’m not supposed to be on this thread, but you’ve cc’ed
> jennifer.t.sun@gmail.com, which is my email address. I imagine we have
> the same name and very similar gmail addresses, so just wanted to let you
> know.
>
> Thanks,
>
> Jen
>
>
>
> From: Jennifer Sun [mailto:notifications@github.com]
> Sent: Tuesday, April 28, 2015 10:59 AM
> To: sunnifer/learningtogether
> Cc: jennifersun
> Subject: Re: [learningtogether] hello world additions (bacb43f)
>
>
>
> i thought i'd git add'ed sandbox, but it looks like i've also git init'ed
> my entire computer… so,
>
> 1 - how do i remove .git on my home directory? i tried to use rm according
> to a bunch of guides, but it kept giving me errors.
> 2 - also, does .git work recursively? (and i'm assuming recursively means -
> if i manage to remove the .git on my home directory, it then goes and
> removes all the .gits in all the folders within my home directory, like for
> my 'dev' folder/everything tha'ts currently linked to github?)
>
> have time tonight, so gonna keep digging… i know it doesn't really matter
> because i haven't actually committed anything / everything is .git'ed
> locally, but it just bothers me and i *want to know how to remove it! i
> have to know!* !!!!!!!!!
>
> may the world know my rage
>
> On Tue, Apr 28, 2015 at 12:29 AM, jayzach99 <notifications@github.com>
> wrote:
>
> > remember,
> >
> > stage to commit --> commit to local repo --> push to remote repo
> >
> > corresponds to
> >
> > git add --> git commit --> git push
> >
> > On Tue, Apr 28, 2015 at 12:28 AM, Jaison Zachariah <jayzach99@gmail.com>
> > wrote:
> >
> >
> > > did you > git add sandbox ?
> > >
> > > On Sat, Apr 25, 2015 at 10:43 PM, Jennifer Sun <
> notifications@github.com <mailto:notifications@github.com%0b>
> > >
> > > wrote:
> > >
> > >> OK, so I keep forgetting to read your latest instructions and get
> myself
> > >> into sticky situations. I managed to sync *parts* of my local commits
> to
> > >> Github BUT… Github doesn't match my local files right now -- there's
> no
> > >> 'sandbox' folder
> > >> <
> > >>
> >
> https://www.dropbox.com/s/xhx9pb6592tpgrm/Screenshot%202015-04-25%2022.42.04.png?dl=0
> > >> >…
> > >> :( :(
> > >>
> > >> (I also realized we don't actually need the 'sandbox' folder, because
> > all
> > >> the files should just go into 'fundamentals' but… I'm trying so hard
> to
> > >> use
> > >> the command line + not just fix everything on the web browser+the
> github
> > >> mac app --- help?)
> > >>
> > >> On Thu, Apr 23, 2015 at 2:12 AM, jayzach99 <notifications@github.com>
> > >> wrote:
> > >>
> > >> > @sunnifer <https://github.com/sunnifer> ... so github is awesome
> cuz
> > >> you
> > >> > can converse about code as well!!!
> > >> >
> > >> > I just added a new directory + file., you should run 1) git fetch 2)
> > git
> > >> > pull each time you start up so you have all the latest changes that
> > are
> > >> on
> > >> > the remote repository (aka github)
> > >> >
> > >> > —
> > >> > Reply to this email directly or view it on GitHub
> > >> > <
> > >>
> >
> https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10861862
> > >> >
> > >> > .
> > >> >
> > >>
> > >>
> > >>
> > >> --
> > >> Jennifer Sun
> > >> 732-763-0556
> > >> @jennifersun <http://twitter.com/jennifersun>
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10903820
> > >
> > >> .
> > >>
> > >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10932390
> >
> > .
> >
>
>
>
> --
> Jennifer Sun
> 732-763-0556
> @jennifersun <http://twitter.com/jennifersun>
>
>
>
> —
> Reply to this email directly or view it on GitHub <
> https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10943195>
> . <
> https://github.com/notifications/beacon/ABvsmz4nVzSRwIbxC3E0VJqu861q0CgVks5oD5fNgaJpZM4EGrsC.gif>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10947560>
> .
>

--
Jennifer Sun
732-763-0556
@jennifersun <http://twitter.com/jennifersun>

—
Reply to this email directly or view it on GitHub <https://github.com/sunnifer/learningtogether/commit/bacb43f882570d6ddf1c7408d1473ff3c2218fd8#commitcomment-10955986> .  <https://github.com/notifications/beacon/ABvsm_zbK7vbZumEAimQYU7HqOLRSxEaks5oEEragaJpZM4EGrsC.gif>


@kslong this means python now prints out a header like this if you compiled with changes which were uncommitted

```
!!Python Version 79c_dev
!!Git commit hash 7c69391d1d38fcb660dbb392db48fddfb24c6045
!!Git: This version was compiled with 2 files with uncommitted changes.
```


Don't use it for now. ZetLab samples in development phase.


Thanks


I added a workaround in next commit... Needs testing on other compilers, but should be pretty clean...


Yes of course the API needs to keep modularity in mind. It's not difficult to do.


I'm still learning git xd.
well I will see how can this be solved :+1:


blargh


How about renaming this method to ```fireAsync``` for simplification purposes?


Hi @berquist,

I'm the creator of this package , I hope it may be usefull to you. : )  Sorry, this code is far to be perfect. So don't hesitate to report any issue / commentary. I would like to improve her (and my self...)

Have a nice day !

PS : You're right for the `os.path.join` is a better practice. I have make the change in my version too. Thx !


1번 줄에는 6개 출력, (합은 7)
2번 줄에는 5개 출력, (합은 7)
...
6번 줄에는 1개 출력, (합은 7)


No, not at all. "Where" is "this was posted to yole's feed and FinalAMA". "Why" is "piggymouse liked this".


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.


fixes #3


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


Is it working with the updated dependencies?


@mechiris Thank you for the nice words. :)


Note, this is particularly obvious with the steamos xpad driver on Ubuntu 14.04 where something (Xorg?) always immediately opens and closes an input device whenever a new one comes on-line - the first time each gamepad is turned on it will mostly fail the race and be stuck with the blinking LEDs until power-cycled again.


@babycaseny I don't have much experience on that part. Do you have any pointers that I can use to do so?


I think we should avoid such monstrous first lines in commit messages.


I just glazed over syntax, and you can remove all your references to the instance variables here since these are attr_readers


Maybe we should note that you might have to go through the following setup
* Install Node (this also gives npm) and gem
* Install bower (`npm install -g bower`) and grunt (`npm install -g grunt`)
* `cd` into the repository and fetch the dependencies using `npm install` and `bower install`
* Ensure compass is installed `sudo gem install compass`


Yes, thanks for reminding. Put it in todo list.


> It was my understanding, that @LeaVerou prefers simple things, and didn't wanted to use npm/grunt/gulp just yet.

This isn't keeping things simple. An npm script would be cross-platform and reasonably more simple. You don't need any knowledge to run `npm run` and we have a `package.json` anyways. I would also argue that makefiles do make it complicated as well for beginners.


Strange that it worked in your tests, it looks like the grammar is expecting a `phrase`, lat/lng, or geohash following `near:`, and `phrase` is defined as being wrapped in (explicitly) double quotes.

https://github.com/URXtech/search-lib/blob/master/query_language.peg


Also, Make requires to know a new 'programming language' for many of those contributors you're talking about. They all know bash (and thus, npm scripts) or javascript (gulp/grunt).


It may be a good idea to use `#freeze` on the methods (`methods.map(&:to_s).map(&:freeze).freeze`) to prevent modification of the available components after the initial cache; modifications to it would then be done through overwriting the method and adding the arrays.  This way, the "cache" as it were would never have an invalid state (i.e. modified by an external method) - it would either hit, and be correct at a certain point, or miss, causing a reload.


whoops, good catch.


:persevere:


:beers:


> It would be useful if all the internal logging output could be put behind a macro/feature that defaulted to no output.

@hugoduncan is this commit what you had in mind?

I didn't get features to work properly, since this "feature" has no dependencies.


Fixed #1


@djs55 is this patch still needed? (I can't find where we discuss about the same question few months ago)


Was i not supposed to see this? :P


Now that this is all one line, there's only the `@` before `filemtime=`stat...` and this will crash and burn:

    /bin/sh: line 3: @echo: command not found


What is interesting list of actions (in redis it's going to be a sorted set, I presume) is a materialised list (is it a right term, can you remind me?) and we can restore it based on likes and comments sets.


- slightly more efficient
- doesn't set 'class="foo "' with a bogus space at the end


See what? I see nothing!


Shouldn't "api" be tested for existence before being used?

```
// Sanity check
if(!api) {
    <HANDLE_ERROR>
} else {
    var options = {
        uri:api
    };
}
```


"where" can't be restored later; it's an essential decision made by the posting user. "why" can be restored indeed.


This is a :bug:. `prime_iter(10, 20)` will yield 2, 11, 13...


Doesn't this mean that the user has to also import ``staticSnap``?


OMG I almost peed my pants


WTF?!? I just added the following 8 lines to schedule.js at line 9823:

```javascript
        if (this.cfg.extender) {
            if ($.type(this.cfg.extender) == "string") {
                window[this.cfg.extender].call(this);
            } else if ($.type(this.cfg.extender) == "function") {
                this.cfg.extender.call(this);
            }
        }

        this.renderDeferred();
    },
```


变为 projectName/repoName
增加 branchName


first commit


projectName/repoName
增加 branchName


Fatal error: Call to a member function enablePlugins() on a non-object in /var/www/*****/mysite/_config.php on line 21


That is indeed strange, and I think we should try to understand why it works.  I don't want to depend on something that's undocumented and not understood so I'm going to change to double quotes.

I did the following and got back a bunch of results near LA:
```
$ urx-api hilton near:'los angeles'
```

I did something similar in Emacs and got what looked to be very similar results.


Good job.


David,

This works great for cross references when pointing to top level chapter. However, it doesn't work if I make a cross reference to a table. Any idea how to fix that?

Example:
[[tuned_table]]
.Tuned Profile for RHEL OSP 6 Environment
|====
^|Tuned Profile ^|System Role
| throughput-performance | Controller node(s)
| throughput-performance | Provisioning node
| virtual-host | Compute node(s)
|====

. Activate the appropriate tuned profile for the corresponding nodes. Information on what tuned profile to set can be found within the <<tuned_table>> table.


Why ~ instead of ! ?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.450](http://teamcity.monogame.net/viewLog.html?buildId=6511&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:47


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


check on tc_gallery_fancybox isn't really needed since the gallery fancybox works just when you have the tc_fancybox enabled -> https://github.com/Nikeo/customizr/blob/master/inc/parts/class-content-gallery.php#L55

Maybe you want to change the behavior of the gallery fancybox ? (also the notice in the Customize then must have changed)


JFTR. This is connected to two stories: https://github.com/pepyatka/pepyatka-server/issues/73 and https://github.com/pepyatka/pepyatka-server/issues/67. Once we limit number of comments and likes we send to a client we'll have to have this materialized list.


git ate tuned_table in the cross reference example and the extra < > signs


I'm confused, @astorije. I thought I understood what you meant by “anonymous variable” [in that other case](https://github.com/w3c/echidna/pull/161#issuecomment-97158519), but now I think I didn't.

Here, `_` is an argument you have to have because it's in the signature of the function that `exec` expects as its second parameter. And you give it a funny name simply because (and to indicate that) you know you won't need it in the body of the function. I like to use [metasyntactic variables](http://catb.org/jargon/html/M/metasyntactic-variable.html) for that, thus: `foo`, `bar`, etc. But apart from that, all fine.

The other case we were discussing (`var DocumentDownloader = {}; var _ = DocumentDownloader;`) is completely different, isn't it? There, `_` exists just to make expressions shorter. So it could as well be called `DocDown`, or `dd`. And the variable is definitely used. You call it “anonymous” simply because you decided not to give it a meaningful, descriptive name. Yes?

Trying to learn here, since I know anonymous functions in JS, but never heard of anonymous vars :¬)


Or `contribthanks.sh`? A typo (missing `b) or trying to keep 8+3 file uniqueness ?


please add this to index.html instead.


Any reason the contentType definition was removed here?


@doronzzz - wasn't part of the problem that on aws /widget.html doesn't exist?


this sort of sloppy documentation isn't like you, phil. do you think that just because you're leaving you can slack off?


Created new stroy https://github.com/pepyatka/pepyatka-html/issues/304 to implement "why" block.


We would like to pull down all the data, because:
 * the overhead is not terrible (1.8G)
 * this future-proofs the build so we can set it and forget it and this should also help limit the number of CRs we need from ops


this looks **so** much cleaner.


You are right, of course. Fixed it.


mmm no. it cant open url's yet. that  note was for me to remind me of a feature i planned to add.


Hello,

Thank you for the report, I have reverted and applied another patch at 6e25d6e589e4aa79768c483bca5c5669aaf52376.
This should not affect the planning process while still fixing the initial bug.


I changed it but I am not sure


You cannot do this @SirEdBr, you need to read what the license actually says before you change it


I tried making them a consistent 1 space only, but it really looks horrible.  I'm not sure what we should do here, honestly.  This seemed like the least bad thing to do.


@ddanderson and @keithbostic I saw this and already committed a change for this typo.


`PeerImp::send` takes a `TMGetObjectByHash const`, so why remove the `const` restruction here?


I was trying to avoid including that header for a single swap, but I ended up having to do so for ARRLEN(), so I guess I can change it.


Hi Thomas,

Thank you so much for writing this; I've been wanting it for a year or two now. There's a lot here, so it's going to take me some time to understand it.

Ultimately, I'd like to be able to read and write all of the formats present in the BSE web interface, and some others that aren't present.

I'll submit some pull requests in the future, so you don't have to copy and paste diffs from the website. Cheers!


I can't tell if @kgary thinks it's funny or scary.


@prakash @dirvine: this is an example of how to construct a RoutingMessage


我这边试下来， 在 `_node_changed.erb` 文件里的
```ruby
<%
   topic = notification.topic
   node = notification.node
%>
```
topic 为空。


The cast should be unnecessary.


Must use av_free().


pls make it Unix-friendly (forward slashes, I think Windows understands them fine)


This is the easiest to start a website. Oh ya don't forget when you make a website on a file on a computer put Name.html so you can see it as a website.


I'd rather keep the try...except.
If this test fails it should be marked as a failure and not an error.


Ditto.


It seems weird that it still has a header, and keeps part of the verification in the hashed filename, and another in the header.


not related to this PR, but s/podInfraContainer/podInfraContainerChanged


It's not the same. This would automatically redirect the option.


A reverse cursor is never SinglePass


@Abhinav1997 You're welcome :)


yes, i think they were conflicting, and I suspect if the display attribute appeared after the style attribute it might have worked but even that would still be confusing. This removes all the display attributes for the layer like your original algorithm, I simply extended it to include another place where the display attribute may appear


if you use "my" then you don't need to use "this". The point is to be able to know if it is a local, member or static variable by its name, without IDE assistance. "this." accomplishes the same goal, but combining them is a little bit too much


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.


Le require à la première ligne fait placarder la prod.
Je l'ai enlevé pour le moment, il faudra trouver une autre solution.


Python 2.6 includes KeyboardInterrupt into Exception, that's why StandardError was used: KeyboardError shouldn't ever get swallowed silently, or you won't be able to break this loop with Ctrl+C:
for u in url:
    try:
        save(parse(download(url)))
    except StandardError:
        pass

So you need to set
If python3:
    StandardError = Exception
and leave StandardError here.


ok we need two sets of dependencies: concatenated and not concatenated.
For example I use tc-scripts (concatenated) so fancybox isn't loaded because of missing dependencies ...


`myConfiguration = Preconditions.checlNotNull(configuration);`

make `myConfiguration` final?


really?
because android have un MAME core update from few days ago (2015/04/22)

iOS no update since 2015/03/31


s/Healthy/Health?


This conversion needs to be constrained. Also, there may be a better way: a conversion operator from `cursor<T>` to `cursor<U>` when `T` is convertible to `U`. I don't remember if `basic_iterator` currently handles cursor conversions. I think so.


git add archivo
git commit -m "myapp"
git push


any sense to use `try` with resources here? Otherwise you don't close myWriter if exception occurs


Are these last four really needed?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.453](http://teamcity.monogame.net/viewLog.html?buildId=6514&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:16:59


TeamCity MonoGame :: Develop (Win) [Build ???](http://teamcity.monogame.net/viewLog.html?buildId=6517&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Win10) Build time: 00:00:03


А как сделать, чтобы без выбора строки не переносилось, так и не понял.
Ставлю курсор в -1 позицию, как пишут на msdn.
cur_sel = SendMessage(list2, LB_SETCURSEL, -1, 0);
А нижеследующая функция для чтения позиции курсора возвращает 0...
cur_sel = SendMessage(list2, LB_GETCURSEL, 0, 0);
И получается, что пользователь может переносить строки начиная с верхней (нулевой) не выбрав при этом что-либо мышей.


I think that's backwards. You should be testing this.


You may want to use safe_substitute incase there are other ``$`` in the URL - is that valid in a URL?


do you really want a singleton? You can just create a new instance every time you need it, no?


Errm, this won't work. You won't have a window at this point. You could use the main screen instead.


@sergeyklay :+1:


Oh, you're right. I got a warning in a previous version of the code, but it seems that was due to a different error.


Well. It's valid in the path component. So, if someone is running with `https://fqdn/$galaxy/` for some, insane, reason, this would fail. Will switch it out. Thanks for the comment; I've only used substitute a couple of times.


Why is one better than the other?


Documentation say elements might be a list or iterable, that's why "if isinstance(elements, basestring):" was there.
In python3 you could just set
basestring=str


forward slashes


Do you think it would be better to hash the metadata / header stuff into the filename as well, then?


something is wrong with the indents here


bla bla


Yes, I had thought of that. But I think this won't be the last check to be made in here. When there are new language features, there might be additional checks in here for them to see if we can introduce a variable for them. One could also combine all the conditions in here into a return statement, for that expression is what truly determines `CanIntroduceVariableFor` and that'd quickly be one huge expression. considering all that I just kept the additional check in the same style as existing code :smile:


comentariu


comentariu la cod


On my analysis test suite I get very severe correctness AND performance regression  from one of the latest commits (haven't binary searched for the exact one though yet).
I'll binary search to the offending revision, but maybe there's something popping up on your own integration test suite?


Never mind, just realized you are "getting" the network map. So my suggestion here is that rather than use rid as a param to init the StaticFileNetworkMapBackend class, we should set it as a param of request, which is more natural for me... So I think we have some different opinions in the design of alto server. Look forward to discuss some details with you in the next sync up : )


make them final?


Generally, "u" is needed here for python2 or non-english characters could be processed incorrectly.
only re.compile(u'', re.U) form uses unicode locale -- less known re library wart.
We use escape here so might be we shouldn't worry, but more safe solution is just to add "u".


Hi @kuraga

I would make also the `_notify` method part of the Freezer prototype. Since it is needed by get and set and it will be publicly accessible, moving it to the prototype saves memory and makes the code cleaner. You can take it from line 13 and from line 53:

```js
// I think that freezerMethods is a better name
var freezerMethods = {
	get: function(){
		return this._frozen;
	},
	set: function( node ){
		var newNode = this._notify( 'reset', this._frozen, node );
		newNode.__.listener.trigger( 'immediate', this._frozen, newNode );
	},
	_notify: function( eventName, node, options ){
		...
	}
}
```

If you make a PR I will merge it :)


მნიშვნელოვანია სემანტიკური tag-ების გამოყენება: ეს არის ინფორმაცია screen reader-ებისთვის და web crawler-ებისთვის (საძიებო სისტემების რობოტები, რომლებიც აინდექსირებენ ინფრომაციას შენი ვებგვერდის შესახებ).
ამიტომ აუცილებლად სადაც კი კონტექსტურად html5 tag ერგება უნდა გამოიყენო. ამ შემთხვევაში nav, როელსაც div-ის ანალოგიურად შეუძია ქონდეს ატრიბუტი.
ანუ აქ იქნებოდა არა <div id="menu"> არამედ <nav id="menu">


1640-AddHVACPart1 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


You have to consider all platforms not just ios (even though iPhone 4 would probably have space issues too) there is a lot of low tier devices in the Android and Windows Phone world that might not fit everything I'm trying to do there so removing the version # is only natural. Thanks for your feedback. in the future I might make the TGD word clickable to display the version number in a popup


Great idea, maybe factor out the code in the search results and include it here too?


This is awesome. I love talking on these topics. I believe you understand what I am doing in jsblocks so lets ask you one last thing. I will just clarify something that you can't see. I build the dynamic function using an API on top of it which is clean JavaScript which compiles to the weird looking code you are seeing. Do you think that part of the framework is worth exploring and further improving. I am asking because I am still wondering if I should invest more time into it.

Your answer was so interesting. From what I know you are using benchmark tests in order to ensure that you don't break the code optimization which is probably quite tricky not to break - is that right? And is there any elegant and efficient way for making the optimization available when there is more than one argument or this is not possible or you won't do it?

Thanks again for the complete answers.


But, man, It's TUNING form, we use it locally all names stored in xml files and that's all...
what we could break with this? nothing than styling things.. all tuning now uses tuning strings so name issue no longer exist, relax, it's safe :P


ეს აღწერ, html-ს რომელი ვერსიაა შენი დოკუმენტი <!DOCTYPE html> მიანიშნებს, რომ ეს html5 ვერსია არის და ზედმეტი  <!DOCTYPE გიწერია


One or two regression tests would be appreciated so that I can also test against your scenarios. It is likely due to shared variables that my own test suite has limited tests for.


Why are you reverting this? It doesn't leave the ability to run a specific test(s)? That seems like a good thing?


Done.


yep I know, just fixed this. I'll push in a minute


should be "we lose" not "we loose"
we could avoid syntax error with conditional import -- but does anyone need python 3.3 support?


Bad spelling!


Works fine, I also have a automated tests for this, all seems good, except you can write one that should pass, but it's don't :)
I'm staring to like unit testing, It's much quicker than test all the  things for real every time, I hope I can cover most of important code with tests asap.


That's correct. In my opinion `std::ios_base::sync_with_stdio = false;` isn't a normal "best practice" but would be a fine-tuning optimization for someone who cares about very fast iostream performance.


Yes, but there was no package.json when this Makefile was added. To minify code contributor just runs make. That's all. No need for npm, nothing to learn.


Yes, it is the opposite as for non-reversed iterators. This is tested in https://github.com/gnzlbg/range-v3/commit/ba81492f6447608e2048d5aa8924993bccd02a72#diff-84245291d619fe0a11e741dd386a4056R143.


@glena I'd go with:
> If you need to get redirected to a `callbackURL` you must remove the callback function on the `.login()` call like this:


FYI: Looks like the source map isn't checked in.


> In case you want the token on the query string in order to handle this from your own backend, just go with `_callbackOnLocationHash` in `false`, like this:


not sure why it's needed. unnecessary but minor slowdown introduced for python 2:
```>>> a = [1,2,3]
>>> id(list(a)) == id(a)
False```
So it looks like array copy occurs for these entries.


It should be safe to return a dictionary with values set to `None`, doesn't it work?


> No need for npm, nothing to learn.

A front-end guy is more likely to be familiar to be familiar with npm. There's no need to learn npm to run `npm run`.


No, they are not needed. Fixed.


The value `sys.version` should not be used as there are no guarantees across Python implementations that the version is at the beginning. Instead `sys.version_info` should be used.


Oops... typo in title:  Modified/cleaned* join actions.


How can i creat a superuser?


Sweet. I originally had it this way but switched it back before I committed as I wasn't sure which you'd prefer.


This causes an error `E121: Undefined variable: context` on the first `for` loop.


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

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


This is great!

Could you perhaps make this a markdown file?


I check for ftp:// in order to prevent handling an archive on an ftp
resource - the GIO file instance returned is not seekable and python's
zipfile module fails because of it.

Christian

On Tue, Apr 28, 2015 at 1:09 PM, Mladen Mijatov <notifications@github.com>
wrote:

> Why are you checking for ftp:// then?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cmallwitz/Sunflower/commit/2b41c91e13aa254eb26e6e7688a00b003742e6b4#commitcomment-10939258>
> .
>


There actually is if you want it to work on 1.0 versions aswell.
In 1.1 they started to push userdata insted of a uid value in the arguments.


Ah, nice, I wasn't sure you were aware that Guice 4 was just released :)


I think I can do this. Let me do this and config. @1156407698 You just work on GUI.


fala tu


Thanks for the quick answer and solution!


da pull ai


Indeed, right now it requires a bidirectional iterator.

I thought that it might just need to require an input iterator, so that one could advance it with --it instead of ++it, but such a "BackwardIterator" seemed like a pretty useless abstraction.

Fixed.


space after `#`.


Showing people something and telling them "don't do that" won't work.  If anyone uses this example, they will cut and paste that code to start, and then, maybe later, fix it so it doesn't export an environment variable to the world.


The funcion I added is named FHEMduino_PT2262_Off_For_Timer, the message attribute I added is off-for-timer.


@arqex What do you think about this commit?


@arthurvr I'm the wrong person here to be convinced. You should try to convert the author :) I'm all for npm, if something. But still, the thing which requires no installation is simpler :D


Nick I figured out what the problem was. You made these changes on your fork, not Neil's.


GAMEOBJECT_TYPE_MAP_OBJ_TRANSPORT& taxiPathID undeclared, compile fails.


Yes, I've seen it happen writing my own conflict checkers.  I've used try/catch to handle the unexpected, which is what [we're currently doing](https://github.com/netz98/n98-magerun/blob/master/src/N98/Magento/Command/Developer/Module/Rewrite/ConflictsCommand.php#L148).

I'm not a pro with reflection.  Can you give me a pointer of how to use it in this case?

As far as I can tell we're going to [get an exception anyway](http://3v4l.org/K3QLl).


* Using negation-first approach for function definitions is a preferred development pattern from my experience. (Example below)
* I would remove the empty return after the callback has been made (see note in example)
* Also, you should be able to remove the additional variable definition for 'options' within the function block by passing in an object directly (unless you have it mapped elsewhere...but I don't see it if you do). Example of this is provided in below comment.

```
// Example (expects 'options' to have been pre-defined);
function find(params) {
    // Handle missing domainName required param
    if(!domainName) {
        callback(new Error('domainName is required'));
        /* In javascript a function either returns a value or executes the callback.
           You're handling that in the negation first approach above and by executing the callback.
           No need for empty return statement.
       */
    }

    // Handle missing callback required param
    if(!callback && 'function' !== typeof callback) { // #if !domainName
        callback(new Error('Callback must be supplied and must be a function'));
    } else {
        client.get({uri: domainName}, callback);
    }
}
```


This was inttroduced. by the 2to3 tool. Basically it boils down to the fact that in [Python 2 `dict.values()`](https://docs.python.org/2.7/library/stdtypes.html#dict.values) returned a copy of the dicts values, whereas in [Python 3 `dict.values()`](https://docs.python.org/3/library/stdtypes.html#dict.values) returns a view object which changes when the underlying dict changes.

However in this case the dict isn't modified while iterating so I will remove that.


Can't you squash these commits?


If webpage_url is None, [YoutubeDL.add_default_extra_info()](https://github.com/rg3/youtube-dl/blob/master/youtube_dl/YoutubeDL.py#L677-683) does not add the ```webpage_url``` field, which causes an assertion failure in [this line](https://github.com/rg3/youtube-dl/blob/master/test/helper.py#L158).


I'll have this reviewed by end of the day sir, thanks. :)


Wow, I didn't even consider that super simple solution. Yes, simply clicking TGD to show version number would be perfect!

And yes, I'm surely not considering all platforms... :-)


$ sudo munin-run mysql_slow
DBD::mysql::st execute failed: Access denied; you need (at least one of) the PROCESS privilege(s) for this operation at /etc/munin/plugins/mysql_slow line 944.


Did you test this? I'm not sure where the http:// and https:// will come from


I logged into the `cache` server, and saw lots of stuff like this in the log:
```
Traceback (most recent call last):
  File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock
    self.process_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
    self.finish_request(request, client_address)
  File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "/usr/lib/python2.7/SocketServer.py", line 651, in __init__
    self.finish()
  File "/usr/lib/python2.7/SocketServer.py", line 710, in finish
    self.wfile.close()
  File "/usr/lib/python2.7/socket.py", line 279, in close
    self.flush()
  File "/usr/lib/python2.7/socket.py", line 303, in flush
    self._sock.sendall(view[write_offset:write_offset+buffer_size])
```

I'm not sure why that was happening, so I just killed the process and restarted it.  It's possible there's some kind of leak going on inside one of the libraries on the server, but things are working a little more smoothly now.  I'm creating something to ping the index of the server and hopefully notify me when it's not working, so I can catch this faster.


This is another case of something 2to3 inttroduced. I think I can remove this as well since `entities` won't change while iterating.


Yes,

All ASCS change to USDA
FAIRCHILD change to Fairchild
OTHER change to Other
TOBIN change to Tobin
TX DOT change to TxDOT
TXDOT change to TxDOT
txdot change to TxDOT


I like this question


What the hell? I merged changes made to Convert.hs into develop with Johan WS yesterday, but it says I made all these changes today. I'm worried.


I think that {include: } could be very useful to condense material and keep it functional. How would that be implemented for our purposes?


`offsetBase` is not really the base of `offset`, so the name seems awkward.  Similarly for `highestOffsetBaseSeen`.  I think of these values as "window numbers".

We never seem to actually use `di.highestSeqNoSeen` except to calculate its window number here.  So I'd suggest reworking it to store that window number as `currentWindow` in the `NaClDecryptorInstance`.

The casts to `int` for `offsetBase` and `highestOffsetBaseSeen` should be casts to `int64`.  golang allows `int` to be 32-bit, and the prospect off wrap-around errors here made me go to the effort of estimating how long it might take to encounter such an error with a 32-bit `int` (I got 150 years).

The relationship between the the case conditions isn't as immediately obvious as it could be, and they are not in what I see as the logical order (the "default" should be between the second and third cases).

So putting these all together, I think `advanceState` would be more readable along the lines of:

```golang
	var (
		offset = int(seqNo & ((1 << WindowSize) - 1))
		window = seqNo >> WindowSize
	)
	switch delta := int64(window - di.currentWindow); {
	case delta < -1:
		...
	case delta == -1:
		...
	case delta == 0:
		...
	case delta == 1:
		...
	default: // delta > 1
		...
	}
```

I hope this makes it clearer that we are enumerating the cases where `seqNo` is before the previous window; in the previous window; in the current window; in the next window; after the next window.


I don't think it will show until it's merged into master. See the `/master/` part of the URL


I think it's because you merged remote into local and not the other way around.


So, I misunderstood what was going on here. ``VarE`` takes a [Name](https://hackage.haskell.org/package/template-haskell-2.8.0.0/docs/Language-Haskell-TH-Syntax.html#t:Name), which needn't be a local name. You can construct a name referring to a thing in the scope of the TH function using a single quote. So this is actually really slick.


I know, but this macro precisely handles the case where the parameter is `nil`, and uses the window from the main screen then. So I could just pass `nil` explicitly, if you prefer.


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


Oh, now I see:
Line
return FHEMduino_PT2262_On_For_Timer($hash, @a) if($a[1] eq "off-for-timer");
has to be
return FHEMduino_PT2262_Off_For_Timer($hash, @a) if($a[1] eq "off-for-timer");

I do another pull request


I see, maybe it would be better to use `'webpage_url': webpage_url or url,` (assuming that the input url is not overwritten) or a comment should be added.


needs fixing.


Probably not. Everyone who uses Python 3.3 should be able to update to 3.4 but I thought I might leave a hint for people from the future. And If someone really want's 3.3 a conditional import with a `raise_with_traceback` utitly function is easy to add.


Messaging creds to Coby


This code is after a return statement, which means that it can never be reached when executed. If this code was before the return statement, it would delete the vector before it was returned, so the returned value would be nonsense.


Actually, coming to think of it we can add the configuration to our buildbot which would try the latest Z3 nightly and then complain.
It is of course possible to use our regression testing suite, but that would require compiling CPAchecker and using benchexec.


You are right. This was again introduced by 2to3 (since it assumes that you only target 3) and I missed it.


I also need the `:email` and `:hide_last_name` fields.


Lowercase is intentional, I decided to write the tool "wireshell" from now on (like the console command)


I don't think we need the links to MSDN for these APIs. It's easy to find.


What is `format.js`? Did you mean `format.html`?


This commit was somewhat rushed during my lunch hour. I will be working on it tonight and it is my objective to have this revision completed in 48 hours max. This will include a complete install from scratch, following the guide.


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1133 of 1133 tests passed)

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


great, so we just need the shi(m|v) now, right?


Oh, awesome, yes you're right!


Seems like the conditional might be unnecessary here?


did I miss something ?:/


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


This change is because  `echo -e` is not compatible for `sh`.


:fu:


Oh right. My bad.


"api" is set at the beginning of the file and is behaving like a constant. If it's not being passed in, then why should we have to check it?


Linking this to debug seems sub-optimal to me.  I thought that declaring a feature in cargo would let you write something like:

```rust
#[cfg(not(feature = "debugxml"))]
macro_rules! ndebug {
    ($($e:expr),*) => {
    }
}

#[cfg(feature = "debugxml")]
macro_rules! debug {
    ($($e:expr),*) => {
        println!($($e),*);
    }
}
```


Star Twins too


Thanks!  Does the start command need to come after you add the configuration or does it pick those up on the fly?


I did not try but I wonder: does that work on python 3? Indeed, python 3 has OSError split into several exceptions. I do not know if the original behavior still works. cc @HubLot


Am I allowed to give :+1:? :)


ayy lmao


Well, nope, I was talking about having "full" support for old ie versions, the shi(v|m) [mozilla?] for html5 elements.
For example ie8 doesn't recognize the footer, article, figure, etc. etc.


You may not all of that. I just passed a bunch of things by reference and used maps and stuff. Use whatever you want to communicate.


ok


@rbvermaa Does nixops copy this out of tree? If not, is there any reason to not use a relative path, rather than depending on `$NIX_PATH`?


https://github.com/aFarkas/html5shiv
https://css-tricks.com/snippets/javascript/make-html5-elements-work-in-old-ie/


Which version of Sync Gateway are you running?
Can you try running the latest from the master branch? You can find the info to build it from source in the readme https://github.com/couchbase/sync_gateway#user-content-building-from-source
Here's the guide for configuring CORS http://developer.couchbase.com/mobile/develop/guides/sync-gateway/administering-sync-gateway/command-line-tool/index.html#cors-configuration


May not need*


yea i tried that...
but I couldn't figure out how to set that feature


That's for the Ajax request that's used to RSVP/unRSVP for lessons. `format.js` allows the `app/views/attendances/rsvp.js.coffee` script to be executed when the RSVP or unRSVP button is clicked.


[](std::string::element_type c) { std::tolower(c); }


Just deleted a "useless" Settings button on Mobile Action Bar. Little detail :)


You're my new best friend. This hadn't penetrated my consciousness yet, so thanks for the reminder about its existence!

For what I'm doing now, the old way isn't too bad, because almost all of my changes are to the C code (which, paradoxically, is in a few ways easier because building is so much faster). But for the future, I'm totally sold on this.

I'll add a version of this to the docs.


Very interesting. Good luck!


I think it should be kept for Windows users who prefer sticking to Command Prompt or PowerShell instead of bash.


Let's just early return here. Otherwise this change looks good.


From the pull request message:

> Hurley wraps its headers in a custom hash-like class that doesn't work well with rspec's include matcher. I've tweaked the problematic test to a form that works with the existing HTTP Adapters, as well as Hurley, and hopefully still captures the spirit of the check.

I think rspec is being [too strict about `Hash`-ness](https://github.com/rspec/rspec-expectations/blob/master/lib/rspec/matchers/built_in/include.rb#L90-92), but is easy enough to work around on our end.


That happened because I lazily selected the whole code block and added a tab on each line, will update that asap with the updated docs ;)


@rmcmx could you review this, please? If you have any comment or suggestion please let me know


done


std::size_t rs = snprintf....
string_t(buf.data(), rs)


we knows xD


same think


Is is most likely to much to ask, but it would be amazing to have one more argument (`cmd_help` maybe) that we could use to document how to get and install the missing program. Thus, we could have something like `cmd_help="Use 'pip install weblogo'"` to install.


you'll be screwed if they check the source code :P


I wouldn't mention the section number here, because it might change after adding/removing other sections


hmm, I think it is better to drop this commit for now. Process holds the lock as well.
The problem is that the seek handler is driven by the mainthread and a call to seek may block for 100ms. This needs a rework for J***


@erimerturk let's add a dot at the end of the comment line.


Nada de pull pra andre
Em 29/04/2015 14:04, "andrepaes" <notifications@github.com> escreveu:

> da pull ai
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eliasjnior/prog3rejected/commit/4781735760a9e357f5cd1d789285efbdb38af06f#commitcomment-10968100>
> .
>


`true`, not `TRUE` :D


@erimerturk remove this line


I tried improving this method a thousand times, but I always failed miserably trying to make it nicer. Anybody accepts the challenge, please? :D


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.456](http://teamcity.monogame.net/viewLog.html?buildId=6521&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:10


hadle --> handle


Woooo! Tak to je parada :)
Hele, zkousel jsem tam dat ty assets a trochu sjem narazil. Rikal jsem si, jaka je k tomu paradni dokumentace http://jtwig.org/documentation/features/asset/ ale podle ni to uplne nefunguje. Podle me tam chybi to pridani toho asset resolveru, ale nevim jak to udelat, nenasel jsem pro to zadnou metodu toho jtwigViewResolver. Muzete se na to nekdo zkusit podivat? Diks


these are unfortunate but hopefully won't matter much ... the question is whether a previous configuration using `name:` and `path:` will work as before or if there will be any issues (migration required).
... esp. if it's counter-intuitive (e.g. not being able to use '/' sounds like it) we better "hack-in" the previous behaviour.


This has been addressed now.


`KeyboardInterrupt` and `SystemExit` will still work fine. They were always subclasses to `BaseException` and thus siblings to `Exception`. The only real difference here would be that with this change this now also catches `StopIteration` (which will never bubble up this far) and `Warning` (which is not the worst thing to catch). See the exception hirachies for [3.4](https://docs.python.org/3/library/exceptions.html#exception-hierarchy), [2.7](https://docs.python.org/2.7/library/exceptions.html#exception-hierarchy) and [2.6](https://docs.python.org/2.6/library/exceptions.html#exception-hierarchy) .


I wrote this so the values that were allowed before will be used if no `options` is passed to `as_json`. You can setup the options hash however you want it and when you call `as_json` with your `options` hash, the `as_json` method will use your hash.


Good idea


ah, this is actually what I meant at https://github.com/substars/trinidad/commit/879d9f02657c6e4af6e16a7371c061184cc05ba3#commitcomment-10968566 ...


LGTM.


這個造成所有 .event 點下去會導向到 undefined url


Hey i worked out the issue you need to generate a application json not a web one


可以打印出值，再return


You should use the IP address of the machine here please.

e.g.

    ip=$(ip addr show eth1| grep -o 'inet [0-9]\+\.[0-9]\+\.[0-9]\+\.[0-9]\+' | grep -o [0-9].*)

you may also want/ need to set the hostname to this value also to curtail any host naming issues for slaves and master communication


what would you do if you don't know the name of the parameters?


in the future create branches for changes rather than messing up your master branch, which should always reflect the latest upstream release code.


[Sem](https://github.com/fit-fms/fms/blob/master/src/main/java/fms/Application.java#L63) pridas tu metodu. Pouzij pro assety slozku ``/WEB-INF/public``.  (vedle views).


Closes issue #17


I know. Just perfecting.


@gmr, just wanted to ping you about my question above. Thx


remember the open / close principle. If I added another servlet or jsp then I would need to change something in the filter. And we don't want that. filter should be written in a generic way. what would you do?


@dalaro Please remove that assertion entirely. That's a stupid way to check that .system() in VertexQuery really does what it's supposed to. We have tests elsewhere to do that. This assertion will likely bite us again in the future.


Is this really the only change in this file, that we want to use jep/commands.msvc9compiler instead of distutils?  Might be possible to force this in without copying the whole file.  Since this is mostly a copy/paste, I'd prefer it if we could override with less code or make it super obvious with comments about which piece is different (preferably the former).


In a consuming project, I think you can just add `features = ["debugxml"]` to the serde-xml dependency.

I'm not sure if you can specify features for the test profile though.


> I build the dynamic function using an API on top of it which is clean JavaScript which compiles to the weird looking code you are seeing.

Yap, I did something similar, but much more basic, in earlier versions of lodash to compile in older environment fixes on the fly. I have a soft spot for it.

>  Do you think that part of the framework is worth exploring and further improving. I am asking because I am still wondering if I should invest more time into it.

For me it ended up being a negative. The competition pointed to it as a reason to avoid the lib, "too magic", "too complicated", "not easily understood". It also prevented contribution because other devs didn't know how to easily augment or improve it. The approach I mentioned earlier, handing off to internal functions, is a good alternative.

> From what I know you are using benchmark tests in order to ensure that you don't break the code optimization which is probably quite tricky not to break - is that right?

No benchmark needed. The affects of shortcut fusion are still detectable, unit-testable, with something like a counter variable. The hint of the additional params or no params is a strong indicator that a user is relying on the traditional behavior. The optimization hits on a common case, operating on arrays & referencing the value param.

> And is there any elegant and efficient way for making the optimization available when there is more than one argument or this is not possible or you won't do it?

I'm not aware of one without trying to interpret user code.


reasoning?


Can you document this a bit further, how it helps us avoid PyMODINIT_FUNC for windows which we don't need since we're running from Java?


Added in List of Ciphers and Deciphers.


why does eslint have trouble with this?


Unnecessary
http://docs.oracle.com/javase/7/docs/api/java/util/Map.html#get(java.lang.Object)


Put the annotation on the same line


Deleted the test.cpp file.


Yes it does, I've fixed this but I still need to fix another coversion issue in the `reverse_iterator.cpp` tests.


Can you document this further, mentioning that the distutil version strips off the manifest at the end of .dll files, but we need the manifest there?


Added in the ListCiphers and ListDeciphers methods


+1 on this if we are fixing some abnormal styles


Initial creation.


This won't actually end up in the build as it's now a variable


Added test file.


Thanks for the heads up @Vincent-Michael xD


Updated the READme file.


the 'p' missing here breaks the compile, corrected before merging


What does this method do?


Oh, I think I see what you were saying, about having no dependencies for the feature. umm


I think LD_PRELOAD is not necessary except for Linux.


First round of main.cpp. Place holder.


Looks good, :fish:


First instance of class cpp added to repo.


Something in here broke test/test-server.js


iirc it's actually faster to have these? cc @petkaantonov?


First instance of class header file added to repo.


Further updated the Readme to reflect new project direction.


Do you have the source of where these numbers came from? Probably worth documenting that here.


Further updated Readme


`self.assertTrue(mock_participate.call_args[1]['prefetch'])`


Further Updated Readme


This is a dictionary so order shouldn't matter, but right now it's kind of ordered in the way it executes (except for clean).  So maybe put build_ext done near build?


Final update to readme to include all necessary sections.


If it's a list and you don't care about it being an array (which I don't think you do here), don't use numpy.sort. Just use sorted(). In a quick benchmark I just ran it's about 5x faster.


removed unnecessary code from main.cpp


My last run was 5.442 seconds with 2 nodes for the whole thing.


nit: What about having a CKBuildInitialComponent it would keep the scope frame creation an opaque process the consumer doesn't have to worry about.


Debugged and made changes to Message.cpp, my class cpp file.


The value should be "'$h'".


Final update to Readme file.


Yes but doesn't make any difference for just 3 calls.


Estaría bien que se marcara como comentario esto, mientras no tengamos un API KEY de DISQUS.


Does `reverse_cursor::mixin` still need a ctor that takes `reverse_iterator`?


Great, passing a listener was definitely weird and misleading


Ops another patch needed:
http://stackoverflow.com/questions/7153470/why-wont-filter-work-in-interent-explorer-8


Habría que cambiar aquíu también a pymxweb.


За что ты его так обозвал o__o


Science (2)?Why is that?


@petrblaho Shouldn't this be using FactoryGirl?

cc @blomquisg @Fryguy


Edwin,

I'm not sure where these diffs are coming from, but it looks to me like
RowMajorWithVectorization is being compared to an 8.2 release as opposed to
an 8.3. Or maybe the system thinks that RMWV is an 8.2?

-Amir

On Tue, Apr 28, 2015 at 5:24 PM, Tik-Tok <notifications@github.com> wrote:

> RowMajorWithVectorization (amirroth) - x86_64-MacOS-10.9-clang: Tests
> Failed
>
> [image: Build Badge]
> <https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-c64e4f293ae88f03376feea3e3e926eb65d67438-x86_64-MacOS-10.9-clang.html> [image:
> Test Badge]
> <https://nrel.github.io/EnergyPlusBuildResults/EnergyPlus-c64e4f293ae88f03376feea3e3e926eb65d67438-x86_64-MacOS-10.9-clang.html>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NREL/EnergyPlus/commit/c64e4f293ae88f03376feea3e3e926eb65d67438#commitcomment-10951727>
> .
>