I like it! I just realized Reincarnate and Samsara are not seperated, as well as DNA and RNA. On the album they will be individual tracks so I'm wondering if they should be on the page as well. Thoughts? I will add the sanskrit translations to the descriptions today.


yeah confirmed - sorry for the false alarm :)


Yeah I was kind of wondering about those being combined. They should all be separated with their own descriptions I think.


At the moment setting it to true results in:

java.io.IOException: Cannot run program "google_logo.png": CreateProcess
error=
93, %1 is not a valid Win32 application
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1047)
        at java.lang.Runtime.exec(Runtime.java:617)
        at java.lang.Runtime.exec(Runtime.java:450)
        at java.lang.Runtime.exec(Runtime.java:347)
        at com.bloc.threads.ImageGetter.run(ImageGetter.java:58)
Caused by: java.io.IOException: CreateProcess error=193, %1 is not a valid
Win3
 application
        at java.lang.ProcessImpl.create(Native Method)
        at java.lang.ProcessImpl.<init>(ProcessImpl.java:385)
        at java.lang.ProcessImpl.start(ProcessImpl.java:136)
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1028)
        ... 4 more

On Tue, Aug 25, 2015 at 7:38 AM, Michael Lustig <notifications@github.com>
wrote:

> Try second param as true here as it is the openWhenFinished boolean.
> Should do the trick.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12879774>
> .
>


D'oh, keep forgetting the README.  Updated with comments addressed.


xz


fix #25


I like it, a few thoughts - the button should be gray when disabled (the
texts don't match), and the CSS transition shouldn't happen - otherwise it
looks great

On Mon, Aug 24, 2015 at 6:30 PM, Jeff Sallans <notifications@github.com>
wrote:

> I was thinking we could do a default button style like this:
> http://jsfiddle.net/jsallans/o4pwenvd/3/
> It uses css transitions so those students think we are hip
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xby2/regular-expression-contest-website/commit/40eb3261c0216f4f2947e853878a6fcc84042e64#commitcomment-12868472>
> .
>


The word “ambiguous” is written correctly in the message itself, however,
the variable is called `ambigious`, obviously on purpose because the author
had to repeat it many times that would make the typo obvious unless the
author didn't know correct spelling of the word at all.

This is however not surprising given how many typos there are in Parsec.

Although it wouldn't that bad if `buildExpressionParser` were written in
more clear manner, that is, when every part of problem is addressed by
little function that is documented and clear, instead of being collection of
various functions put into one big `where` clause.


According to https://connect.microsoft.com/VisualStudio/feedback/details/988432/std-current-exception-returns-null-when-called-from-terminate-handler it returns nullptr. According to the docs, the return value is:
> An instance of std::exception_ptr holding a reference to the exception object, or a copy of the exception object, or to an instance of std::bad_alloc or to an instance of std::bad_exception.

I'm pretty sure it's the std::rethrow_exception that causes bad_exception to be thrown.


Hi,

I'ts causing a build error
http://prntscr.com/88p6b9


Ok I will work on that.


This is called `intercalate` in Prelude.


szkoda :-)


I left a question in Stacey's new wiki about coding Interpolated letters that may be relevant here. I am now wondering if we should look at tags for indirect discourse or for levels in narration: I want to look this up in TEI!


Why this hunk? Doesn't really change anything, except making the code longer...


I'll open an Issue on this, to help us keep track of our thinking as we try to figure this out!


Yeah, I'm always doing it the hard way. Like american folks say *"That's how real men do it"*. :wink:
Thank you very much for your help on this matter.


You're perfectly right! Thank you.


redundant


@azturner Should this have more in the down migration considering all the changes it's making?  I was troubleshooting an issue yesterday and tried to roll this back to rule it out.


Started failing for as numhosts does not exist accordint ot python, had to substitute with
```if network.num_addresses == 1:```


This is absolutely naïve. Did it ever occurred to author of the code that
translation to different language is not done by one-to-one substitution of
words? The whole function is thus over-complicated. And again one big
`where` clause making it difficult to understand.


Create your library, remove the library manufacturer.
Checked: New library work


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 405](http://192.168.30.225:8111/viewLog.html?buildId=2232&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Should be "getty-compatible", so missing a hyphen.


Tiny nits. Otherwise looks good.


Can't we just do `Grammar {start: None, rules: hm, tokens: t}`?


Ok! I had to edit my comment because I forgot the angle brackets, but I'll add it to the Issue section


Similar problem, had to substitute with
```value = bytes(network.network_address)```


Hello Carlos how are you
Iam working on Ant Colony for image segmentation and label the segmented part and I saw you Isula framework on GitHub this is a nice work and very useful for researcher

I have found your software in the Google search engine and an tsystem code  is a good and very useful.
Currently, I have need to turn to the programming Ant as thread  to allow ants work at the same time on environment Can you help me to do so.
With high respect and appreciation


yay my header organization OCD is spreading!


Might be an issue with where that image is being saved and where we're
attempting to open it at runtime. Works fine on my computer (iMac)

We'll consider you good to go on this one. No need to dig into cryptic
Windows errors.

On Tue, Aug 25, 2015 at 9:46 AM, Nathen Watters <notifications@github.com>
wrote:

> At the moment setting it to true results in:
>
> java.io.IOException: Cannot run program "google_logo.png": CreateProcess
> error=
> 93, %1 is not a valid Win32 application
> at java.lang.ProcessBuilder.start(ProcessBuilder.java:1047)
> at java.lang.Runtime.exec(Runtime.java:617)
> at java.lang.Runtime.exec(Runtime.java:450)
> at java.lang.Runtime.exec(Runtime.java:347)
> at com.bloc.threads.ImageGetter.run(ImageGetter.java:58)
> Caused by: java.io.IOException: CreateProcess error=193, %1 is not a valid
> Win3
> application
> at java.lang.ProcessImpl.create(Native Method)
> at java.lang.ProcessImpl.<init>(ProcessImpl.java:385)
> at java.lang.ProcessImpl.start(ProcessImpl.java:136)
> at java.lang.ProcessBuilder.start(ProcessBuilder.java:1028)
> ... 4 more
>
> On Tue, Aug 25, 2015 at 7:38 AM, Michael Lustig <notifications@github.com>
> wrote:
>
> > Try second param as true here as it is the openWhenFinished boolean.
> > Should do the trick.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12879774
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12879997>
> .
>


Meh, sorry for that, that was unintentional.


Fixed


Looks fine.


When you leave radix off, it defaults to 10.


how many more times are we going to write AFNetworking+PromiseKit? would love to see this DRY'd up in a category


Good catch, thank you! :+1:

There are indeed more in the database: right after the Grim Guzzler. That means that they could be aggroed if players have already made it to Shadowforge City (and this part of the map is still active) and that the Grim Guzzler door is already open but Argelmach is not dead yet. Unlikely, but needs to be fixed indeed.


hehe :)


@cesosag Seguro que esto redondea hacia arriba? si le pongo 1.2 me va a decir 2?


thx


what is supposed to be redundant? note that we use the same "store" utmp struct, and first set `.ut_type` to INIT_PROCESS, then to LOGIN_PROCESS and finally to USER_PROCESS, each time calling write_entry_both()... I don't see what is redundant here?


Instead of changing the commented ones, I would append a section

    # arch-installer: Allow passwordless root access.
    PermitRootLogin yes
    PermitEmptyPasswords yes

So that our change isn't hidden in the middle of the default configuration file. (I don't know whether later options override first ones, would be nice to check, but in this case since the originals were commented, there's no problem).


You want to make the string $window.innerHeight to an integer and then multiply by 0.8 (to get 80% the window inner height correct?)


Merci, au revoir.


Good idea! Requirement files can include comments and other inclusions though. Perhaps we should standardize how to handle these in our components.


Why the fuck aren't you drunk elsewhere?


also needs DRYing up


This does not work in polymer 1.0. See my comment in:
https://github.com/stampzilla/stampzilla-go/blob/master/nodes/stampzilla-server/public/app/elements/nodes-list/nodes-list.html#L56


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


d'oh, i misread. Ignore me.


Sounds good, that's why I put those comments in there since I didn't think
it would be a good use of time to dig into this.

On Tue, Aug 25, 2015 at 7:55 AM, Michael Lustig <notifications@github.com>
wrote:

> Might be an issue with where that image is being saved and where we're
> attempting to open it at runtime. Works fine on my computer (iMac)
>
> We'll consider you good to go on this one. No need to dig into cryptic
> Windows errors.
>
> On Tue, Aug 25, 2015 at 9:46 AM, Nathen Watters <notifications@github.com>
> wrote:
>
> > At the moment setting it to true results in:
> >
> > java.io.IOException: Cannot run program "google_logo.png": CreateProcess
> > error=
> > 93, %1 is not a valid Win32 application
> > at java.lang.ProcessBuilder.start(ProcessBuilder.java:1047)
> > at java.lang.Runtime.exec(Runtime.java:617)
> > at java.lang.Runtime.exec(Runtime.java:450)
> > at java.lang.Runtime.exec(Runtime.java:347)
> > at com.bloc.threads.ImageGetter.run(ImageGetter.java:58)
> > Caused by: java.io.IOException: CreateProcess error=193, %1 is not a
> valid
> > Win3
> > application
> > at java.lang.ProcessImpl.create(Native Method)
> > at java.lang.ProcessImpl.<init>(ProcessImpl.java:385)
> > at java.lang.ProcessImpl.start(ProcessImpl.java:136)
> > at java.lang.ProcessBuilder.start(ProcessBuilder.java:1028)
> > ... 4 more
> >
> > On Tue, Aug 25, 2015 at 7:38 AM, Michael Lustig <
> notifications@github.com>
> > wrote:
> >
> > > Try second param as true here as it is the openWhenFinished boolean.
> > > Should do the trick.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12879774
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12879997
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wattersnathen/github_checkpoint/commit/e75fe5753a4ee23025266b87e048e4e77f9de29c#commitcomment-12880233>
> .
>


This test name suggest that you are only reducing loop file size while you are testing both loop size shrink as well as growth. I think splitting it in two test cases would be better.


Changes Summary:

new class AbstractTripleRematch:
A new metamorphg stream class that delivers general functionality for rematching triples. As there are different types of data sources ("A", "B"...) with partially overlapping data, there might be overlapping ("same") data with different keys. Rematching means: define a priority key type (let's say: the key of data source "A") and apply that key to all "same" or overlapping data sets, whereever applicable. As the load of triples might exceed the capacity of the JVM memory, triples are stored into temporary files.

new class TripleRematch:
An implementation of the abstract class mentioned above, essentially defining, how the rematching mechanism works. (See identifyRekeyCases()).

new class Constants:
A central class that evolved during refactoring code, as there had been many hard coded Strings (in different classes). The idea of that class is to define every String once only and to make it accessible all over the package. Another approach of the refactorings was to parametrize methods more often, in order to prevent multiple access to same constants in the course of processing - this refactoring has to be carried on.

new class Helpers:
Centrally provide static code that is used in different classes (e. g. Dbs, Sigel, Enrich) in order to prevent cross-jumping between these classes.

Enrich:
Introduction of the use of the XmlEntitySplitter, defined in lodmill. The XmlEntitySplitter splits files containing mutliple ("isil") data sets into separate files. The reason for the doing so was to make sure that updates in fact overwrite old / existing data sets.
Enrich also makes use of the TripleRematch mechanism, as descibed above.

EnrichSample:
Introduction of the use of the XmlEntitySplitter, see Enrich.
EnrichSample does not make use of the TripleRematch mechanism, since it is designed for basic testing of small data sets. Nevertheless it can be redesigned, if that should be desireable.

Sigel:
setupSigelMorph() (formerly morphSigel()) makes use of the XmlEntitySplitter and therefore needs an XmlTee object. The tee object is inserted just in front of the XmlEntitySplitter (thinking in stream direction) and allows further chaining of another stream receiver (which is the PicaXmlHandler in this case), which would not be possible otherwise since the XmlEntitySplitter extends DefaultXmlPipe, which in turn only receives stream data but does not send any.


I copied it from SciLifeLab/TACA. True about the comments. I didn't consider that as you can see in the `requierments.txt` file. I'll remove the comments for now, and then I do think it should work.


nitpick, just pass `nil` if you're not going to do anything in completion


This will pass on a connection refused error too. So I'm not sure this test proves that timeouts work.

Myabe run a fake a registry using busybox `nc`? So it will accept connections but never reply? Or use more accurate assertion (IOError and *not* connection refused maybe)?


Yes we can -- good point!


This line is apparently useless. Also note that it probably doesn't mean exactly what you think it does; see http://www.htmlhelp.com/reference/wilbur/misc/comment.html for more.


I guess I'd also be happy if turning off timeout functionality in our code made the test fail. Does it?


Thanks for the PR.
I'm not sure this bit is correct as it's condition duplication. Are there any alternatives for PostgreSQL regarding ```LEFT JOIN``` replacement?


I don't think this was intentional…


Update


gem install rspec
I had this error when want install respec:
ERROR:  Could not find a valid gem 'rspec' (>= 0), here is why:
          Unable to download data from https://rubygems.org/ - SSL_connect retur
ned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed (
https://api.rubygems.org/latest_specs.4.8.gz)

Solve whit:
 gem install rspec --source http://rubygems.org


Looks fine.


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

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

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

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

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

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

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

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

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

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

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

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


That code was actually provided by bloc!

From my understanding it basically extracts the same values as a UIViewAnimationOptions enum

enum {
   UIViewAnimationOptionLayoutSubviews            = 1 <<  0,
   UIViewAnimationOptionAllowUserInteraction      = 1 <<  1,
   UIViewAnimationOptionBeginFromCurrentState     = 1 <<  2,
   UIViewAnimationOptionRepeat                    = 1 <<  3,
   UIViewAnimationOptionAutoreverse               = 1 <<  4,
   UIViewAnimationOptionOverrideInheritedDuration = 1 <<  5,
   UIViewAnimationOptionOverrideInheritedCurve    = 1 <<  6,
   UIViewAnimationOptionAllowAnimatedContent      = 1 <<  7,
....


Please remove the tabs from this page. We don't need "maintaining stations" and "created stations".


i'm guessing this is because we need to handle cases where the `recentSectionController` is `nil` due to an empty history?  can we make this behavior more explicit than a `nil` fallback? probably more elegant to simply "load" the section, but handle case where the section doesn't have any data to show by either showing an empty view, or if one isn't provided, omitting the section. then, we can add the section in response to one of the usual `didUpdateItems:` (or whatever) callbacks.


How are the tiny, small, large, and huge unordered collection node types added to the `nodeTypeNamesThatAreUnorderableCollections `?


added skeleton for block class


plz plz remove `console` calls.


🚨 missing `@weakify`/`@strongify`


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 405](http://192.168.30.225:8111/viewLog.html?buildId=2232&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:12:12


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.466.85ca016a0cfaba0058d099ee82df329adbc1873a](http://build.palaso.org/viewLog.html?buildId=61873&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 137 Build time: 00:07:57


I think just `Subtext.parse(request, tap, options, callback)` is ok. Looks a bit weird with arguments.


You should wrap the properties in backticks too. Like here: https://github.com/hapijs/shot/blob/master/README.md


see previous comments about handling empty sections w/o using `nil` fallback (which IMHO is too implicit/vague)


@makwanza Use the enum then... It is clear.


Please fix the indenting in this file. Apparently you are using a mixture of spaces and tabs. Use spaces only.


re: Automattic/mongoose#3272


Hi,

Thanks for writing, and I'm glad the code I posted here is useful to you.

It was on plans to enable multi-threading on the Isula code, but I haven't much time to dedicated it lately. Maybe if you share with me more details in what you're trying to achieve I can give you some guidance on how to implement it.

Hope to hear from you,

Carlos G. Gavidia


In our other repos we set `image_name`, but this seems cleaner. Any reason not to make the same change across everything?


Bugfix deployed


@posita did you find time to review my code quality? Also I can easily squish and patch this bug if you give me access to some temp repository that has the complete code.

Also I am anxious to discuss my skills and experience with you see if I can find some opportunity with you.

Thanks,
Talha


Can you able to share URI for OneDrive to be used on WP8


I can think of a few reasons :) but it seemed the least objectionable way to handle it:

* Separating the generated configuration into two parts: I don't see a way around this. You can't add node attributes to the CIB without knowing the node ID, and CTS doesn't know node IDs yet when it generates and deploys the CIB. So I added "post-up" configuration allowing commands to be run after all nodes are up.

* Where post_up() is called: I went with startall() because it's always called when a CIB is generated by CTS, and because it waits until *all* nodes are up, which is important here. Also, if someone uses a pre-existing CIB, then nothing will be added to node_attrs, and post_up() will be a no-op.

* Timing issues: I don't think any are significant. I ran into a few when trying post_up() in other locations, but here we can be sure that all nodes are known. It could possibly be worthwhile to wait for cluster_stable() again after doing the post_up(), but I don't think that will be an issue.

* Relying on crm_attribute: CTS does this elsewhere already.


Thanks!


would be nice if we could move this logic into the individual section controllers.  for example, what if the user's location changes and that section needs to reload as well? we can't implement conditional reloading for each sections inside the parent (SRP, LoD, etc.)


Hi I'm super interested in this project and more than willing to contribute, but I think there is this potentially unresolved issue https://github.com/jruby/jruby/issues/1740 for which I've been long seeking an answer. Also I thing it would be great to have maven integration (but for my projects [ruby-processing][] and JRubyArt, it's a bit tricky as the processing guys are resisting maven). Third parties have made processing-2.2.1, available in maven so I could potentially update [ruby-processing][], and thus provide examples of standalone ruby classes Vec2D and Vec3D as jruby-extensions, java integration ArcBall, and modules providing math function MathTool.
[ruby-processing]:https://github.com/jashkenas/ruby-processing/tree/master/ext/monkstone


Looks good.


it's linked to before. Do you want another reference?


I did make a mistake. I copied this from the line above and did the change in the browser since I didn't have a copy of the code on the machine I made this pull request on. I have updated the commit now and it should be good.


I can run it properly on my end. Any additional steps to reproduce the issue?


I've seen you do this elsewhere, why pass `NULL` instead of `nil`?


Yep, already have that locally :)


I'm doing `six.string_type(network.network_address)`, fix will come.


Typically we would rewrite untrusted_app.te to allow reuse by untrusted_app_noexecute without needing to generate it at build time.  Either by taking the core rules that we want to share into a macro that can be called by both, or by introducing a new attribute and writing the shared rules using that attribute.


리스트에 사이즈를 알려주는 size() 함수나 len() 이라는 내장함수가 없서서 만든건대
간단히 사용할 수 있는 함수가 있었군요
git이 아직 익숙하지 않아서 수정본을 새로 올렸습니다


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.443.85ca016a0cfaba0058d099ee82df329adbc1873a](http://build.palaso.org/viewLog.html?buildId=61874&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3346, ignored: 134 Build time: 00:06:15


fixme: Send options only to the current player


You can expose the async installed in Sails as a global variable : [cf ici](http://sailsjs.org/documentation/reference/configuration/sails-config-globals)


:heart:


Er det nødvendig med tom linje her?


@carl-mastrangelo Could you clarify what you mean by the order being messed up? Do you mean that the `SizeEnforcingInputStream` should wrap the raw stream rather than the decompressed stream?  We're already separately enforcing the `maxMessageSize` on the wire, so I don't think that would help.  The `maxMessageSize` is supposed to be the limit on the number of uncompressed bytes, so the byte counting would have to be after decompression.


Thanks, 谢谢


Don't we want to exclude untrusted_app_noexecute from this rule too?


Ja, det er det beste!


And by the way, you can reach me directly on carlos.gavidia@pucp.edu.pe

2015-08-25 14:54 GMT+01:00 ammarblue <notifications@github.com>:

> Hello Carlos how are you
> Iam working on Ant Colony for image segmentation and label the segmented
> part and I saw you Isula framework on GitHub this is a nice work and very
> useful for researcher
>
> I have found your software in the Google search engine and an tsystem code
> is a good and very useful.
> Currently, I have need to turn to the programming Ant as thread to allow
> ants work at the same time on environment Can you help me to do so.
> With high respect and appreciation
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cptanalatriste/aco-image-thresholding/commit/e6d7f230d52949d5f9476f25a339c1988ba4114e#commitcomment-12880222>
> .
>


i'm not sure if active is the correct lifecycle phase.  for example, if the user gets a notification and dismisses to go back to the app, we'll get inactive/active notifications even though they didn't leave.  better to observe resume notifications so we can update when they come back.  active/inactive is better suited for situations like pausing a game when the user gets a phone call, or in our case, handling audio/video playback after app becomes active again.*

\* audio session management is not fun...


Why this addition?


typo ... pepper


Isn't this check backwards. You want to not fail if its LAV Video.


@ggouaillardet No worries! I did as well.

I like the idea of having two libraries as well. Say, using `-lipm` for C code and `-lipm -lipmf` for fortran. Indeed you're right that it will be necessary to export the `IPM_` symbols.

This should be a lot easier to deal with than adding weak symbols to handle the case where the fortran MPI libraries are not included at the link line (e.g. when `mpicc` is used). I've experimented with this a bit in the `wrapper_improvements` branch, and:
* Testing for and supporting the equivalent of `__attribute__((weak))` in compilers that do not "speak GNU" can be not-so-fun (I've only added autoconf checks for the GNU style at the moment); and
* More of a problem: dealing with linking order properly on systems where static linking is expected... Here, regardless of whether the symbol is weak, the linker will stop scanning at the first definition of the `PMPI_` binding. Without adding some `--whole-archive`s to the MPI compiler wrapper, this breaks static linking.

Otherwise, this latter approach works beautifully for dynamic linking! (either directly or via `LD_PRELOAD`)


Seems a little confusing (seinfo=platform vs seinfo=platform_app_noexecute).  Why put app into the seinfo at all?  Why not just platform_nx?


:-1: move color into a category, which should be the same one used for link coloring in `NSAttributedString+WMFHTMLForSite`


Are you speaking to me in english? oO


Of course ^^'


Varstu búinn að klára HTML til að byrja á css?


side note: can't wait until we can get around to implementing URL handling & deep linking


This can be done as a single ifelse statement


This template sucks. Clearly we can't fix it all here, but we could start by doing a few things:

1. Delete the tabs.
2. Delete the apparently accidentally inserted line 11.
3. I don't like the `<br><br><br>...`, is there any better solution?


is this working? i still see two separate pages when testing this.


You are right as usual :)


Hm, another way to do minz/maxz? https://www.opengl.org/registry/specs/EXT/depth_bounds_test.txt


Fixed https://github.com/alexmarsev/mpc-hc/commit/7c3662d308b916cb8b35b80318be7d895e81a52c#diff-c7d921c873daa7f781aaa34a41410156R161


Agreed, too fragile and tight a coupling to the content of untrusted_app.te.
Better to explicitly distinguish rules in untrusted_app.te that we want to share or not share by wrapping the shared rules with an attribute or a macro.


Looks fine.


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


@thatoddmailbox did they give you early access to courses?


$Head -2.
N....press it after scf calculations to  see the various iterations
$grep -e ! scf.40.out  |  tail -1...........seeing at the top
$rm tar*.....removing files.....no spacing otherwise all will be deleted.
$grep -e ! scf.200.out.............for all the possible iterations
$tail -f si.scf.out
echo “runninf scf calculation”....prints what you are calculating.
$ ./run.sh &.....running background.....
$TOP....incase you wish to see the current position of your file.


> So if you constantly reference `foo.bar` you can get a performance boost by defining `var bar = foo.bar;`.
> (http://jonraasch.com/blog/10-javascript-performance-boosting-tips-from-nicholas-zakas)

Also, [this SO answer](http://stackoverflow.com/a/6974417/227176) explains more about this specific optimization.


The prototype is a little off. It probably should return void *?
The first two arguments need to be void * as char * gives signedness warnings on ARM.


kind of a nitpick, but would be nice to have separate protocol methods for `headerText` and `attributedHeaderText`, both of which would be `@optional` though one would need to be implemented (assuming we always need to show header text).  the homeVC could first check for `attributedHeaderText` and then fall back to `headerText`.


Now that we can preempt the trash removal when the task is shutting down, it is important that we perform a depth-first traversal and delete the subtree before deleting the parent node. The simple non-recursive implementation uses a breadth-first traversal. IMO it's not worth the complexity of designing a correct non-recursive version for a situation that has never come up.


Thank you for all your work, clowwindy.


Thank you 谢谢


I read somewhere that adding a rectangular path for your shadow makes it easier to render and can improve scrolling performance. Might want to try that out once we get to polishing


Correct, removed it.


Changing
```c#
Range range = this.CurrentEditor.GetRange(index + gotoName.Length + 2, index + gotoName.Length + 2);
```
to
```c#
Range range = this.CurrentEditor.GetRange(index, index + gotoName.Length + 2);
this.currentEditor.Selection = range;
```

would have had the same effect of selecting the whole line but without constructing a new Range onto the stack. This very likely won't matter though due to compiler optimizations done after compiling but will still reduce code redundancy and by that improve readability.


oh, never mind, you're doing that below. is `setBounds:` always called after `awakeFromNib`?  also, shouldn't this technically be done in `layoutSubviews` since `backgroundView` is a subview and its bounds might not be up to date after bounds are set (i.e. until the next layout pass is performed)?


is `setBounds:` always called after `awakeFromNib`?  also, shouldn't this technically be done in `layoutSubviews` since `backgroundView` is a subview and its bounds might not be up to date after bounds are set (i.e. until the next layout pass is performed)?


Sorry, I was erroneously assuming Locale and Timezone were somehow related, but they aren't.. So I'm guessing the timezone in that example has to be hard-coded.


why do we want to change the order?


I think GET/PUT/DELETE is the common order.


The general path below (default case) is unprotected?


can we abstract this into a category to make it easier to have consistent shadows across the app? will make styling sections easier as well


"is required to acquire" (also above)


Nei ég var í rauninni bara búinn að henda inn nav barnum. Ég læt þig vita þegar þetta er komið. Hef ekki ennþá komist almennilega í að klára þetta.


Please can you split these into separate commits?  It's a lot harder to review when you mix several changes into a single commit.  https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes


ok.


LGTM


就没有中文的么...


Looks fine.


@HelenaSabel @setriplette Are you seeing anything in angle brackets in these comments? Stacey mentioned using the "sic" element for the weird period here, but I can't see that unless I look at her comment in "edit" mode. I am seeing that [GitHub has a special "markdown" for code in these areas, and we should familiarize ourselves with it:]
(https://guides.github.com/features/mastering-markdown/)


Yep, I made a PR here: https://github.com/rcbops/rpc-openstack/pull/365


Comment it due to bugs


Need to include "<limits>" for g++.


The packed structs are making me nervous as I don't know whether or not this guarantees alignment safe accessors.


@masoodtalha, see my additional questions in my OP, specifically:

> ... What does `toggleSortableClickHandler` do? What *should* it do?

With respect to testing, how would you test this without access to the original repository?


Make this editable in the company view.


Это еще что за тег? Где все остальное?


Shouldn't we open KotlinClassFileEditor for Built-in files?


Можно указать ссылку на catalog-item.html


add 2.c


OK, I've updated the PR.


add


Note that


It was specifically written for a Node.js user, yes. It's not specifically a generalization of EnergyPlus, but can use EnergyPlus, EnergyPlus+Radiance, or the ISO Model to feed results into a linear estimation formula with error correction to provide instantaneous energy use estimations with progressively more accurate results as simulations run in the background.

So, you define a set of variables (user defined, they don't have to have meaning to EnergyPlus or OpenStudio) that you are manipulating. If it has enough data, it will return an immediate linear interpolation of the energy usage based on previous runs. If you feed in ISO Model results for the model, it can apply an error correction based on previous runs, comparing ISO Model against your most accurate EnergyPlus+Radiance results, for instance, and generating an error correction model based on the closest previous runs that were done by more accurate engines. I can give you more info if you'd like.

So, I'd prefer if you not remove it. It might need to be documented better though.


i think it should be const struct sol_http_client_pending, since does not make sense call the cancel() with this handle anymore.


Wasn't it 0, 5 before?

Minimum was set to 5 before for reason, to match https://github.com/ddnet/ddnet/blob/7aec49e6b1f57c4a509c0dfa7aec23c18969c082/src/engine/shared/config_variables.h#L127.


No, because the `self.preferred_host` here is the default one (see [defaults.py](https://github.com/ably/ably-python/blob/master/ably/transport/defaults.py#L28-L32)). This means default host will be the first to be tried, then fallback hosts after it. I just used existing methods of defaults.py, but if the name is confusing you, I can create another method called `get_default_host.` What do you think? @mattheworiordan


meirge


+1 as well


Works fine with M-x shell on GNU Emacs


Stack Overflow and GitHub both have APIs. We could create something that, when a GitHub issue is deemed to be not a bug report or feature request by a collaborator, posts the text from the issue (along with metadata) on Stack Overflow with the tags ```promise``` and ```bluebird``` then posts the link to that post in the issue and closes/locks it.


Looks fine.


Thank you! 谢谢！


Changed because of the removal of the regex matcher is used in the past to capture all commands and keywords from teaseAI code


+1 to ditch those. wallclock deserves better testing, though


I really don't know why this test failed, because if I ran it on my SGS4 with AS the click on the seekbar returned 127 as value.


An alternative to this would be explicitly testing for both IE and falling back to a secondary test for Safari. At present, this loosens up the check quite a bit, but could be okay. @jmesserly this change is proposed for both here and the HTMLImports boot. Would this lgty?


See this discussion here: https://github.com/kennethreitz/requests/issues/1236#issuecomment-133312626
It is safer to catch all. But if you want I can catch `requests.exceptions.RequestException`, but `requests` lib already had bugs where other exceptions were thrown up.


not sure, the idea of using pending is to follow the i2c api and to my mind pending is clearer , but i'm ok with this change if from the node's perspective it makes more sense


The content is correctly sanitized if this is removed.


Can we change the commit message to do not allow to put tombstone on an empty generation?


In the latest aospal release 20150821 the camera of my find7 does not work anymore. Is there a bugtracker anywhere? Previous release 20150704 was still good, but it suffered stagefright.


develop (DavidGoldwasser) - x86_64-MacOS-10.9-clang: Tests Failed

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


can we add a comment for this test?


I had the same problem. You need to wrap the code between `` ` `

as in ````<sic>something</sic>```


Angular will get loaded onto the page


Done. The key is the SHA-256 of password + salt. ;-)


Yes sorry, i've just copy/paste the "on" clause without read it ><

And for LEFT JOIN i don't know. I'm not an expert on postgres. I want to learn about it but i begin.


>Введение маски для одного донатера.
>mask/mintbandana


And by the way, I agree that using sic makes a lot of sense!


Hi Helena,

I understand the waxed letter to contain only

“Este es Amadís sin Tiempo, hijo de rey.”

The characters read this line out later, so I’m pretty certain. The letter is small enough to fit inside the ring, so that speaks to less rather than more being on it.

The book I’m writing is about epistolarity in chivalric fictions so the letters are a big deal for me! I think we should figure out an intuitive way to code them because I know I’ll be searching for them later.

S

—
Stacey Triplette
Assistant Professor of Spanish and French
Humanities Division
University of Pittsburgh at Greensburg
Faculty Office Building 200
150 Finoli Drive
Greensburg, PA 15601

On Aug 25, 2015, at 9:54 AM, Elisa Beshero-Bondar <notifications@github.com<mailto:notifications@github.com>> wrote:

Ok! I had to edit my comment because I forgot the angle brackets, but I'll add it to the Issue section

—
Reply to this email directly or view it on GitHub<https://github.com/ebeshero/Amadis-in-Translation/commit/92d7b23be8405e03dc749c466bf0a24b07e74cea#commitcomment-12880219>.


Looks fine.


can we move this out of for look?


also this


Looks good.


this error can be triggered by redefining  a definition as an application, and then the error message is misleading.


run pep8


You should except httplib.BadStatusLine exception here.


I am not sure why we want to do this for 8 times actually.


How can mb_list->mbuf ever be NULL ?  Can this if be collapsed?


Закрывающие теги рекомендуется не ставить


You've got pandect listed as a dev dependency, but it's actually used in production code - dev dependencies are for developers of the library (things like running tests) - so you should put `pandect` in the normal dependencies list instead.


Имя функции не по psr-1


Change to Excl.


//GCC测试用


No, Removed it.


When using `:compressed`, the ending `;` of the `@import` statement would be removed in the generated `css` file:
```css
@import url("//maxcdn.bootstrapcdn.com/font-awesome/4.4.0/css/font-awesome.min.css").fa-apple, .fa-android, .fa-twitter {font-size: 60px;}
```
This makes the styling following the `@import` statement not working.

Created a question in http://stackoverflow.com/questions/32203673/compass-removes-semicolon-when-compressing-scss-files-causing-issue-of-css-impo


I've got a new build.  Would you care to test it out first before I push it up to nuget?


Could you draft a bugfix release for this fix? This bug prevents compiling the `_bootstrap.scss` from https://github.com/twbs/bootstrap-sass.


round redondea al mas cercano.
Hay que usar http://php.net/manual/es/function.ceil.php


Параметры соединения с субд нужно уносить в конфиги, а в метод передавать.


:angry:


@ParthKolekar Are we really going with this? -_-


In the beginning when we fill the RX dma schedule it can.


В принципе, таскать по коду линк к базе не нужно (только если конечно у тебя не два (или более) подключений к разным  шардинг или программная репликация) . Открытый линк станет дефолтным для методов(функций) mysqli ext


blocked lines shouldn't be taken into consideration here


I was rather hoping that this complexity could go away, and the array could be collapsed down into a single buffer.  Can you ensure this by constraining the number of elements in the rx ring?


Suggest adding a comment here (similar to the one in the commit message) about hardcoded knowledge.


:cry:


good point


The method `lastdayofyear(dt::Date)` already existed.


yep


If possible, move this script commands in the Dockerfile, else, just be sure to delete the file at the end


I think I'd rather see this `after-all` up at the top - even though it happens last, it's sort of the inverse of the `before-all` and I like knowing that things will be cleaned up before going on to read the tests (sets my mind at ease :smile:)


¿Y no es eso lo que se necesita? ¿Redondear al más cercano hacia arriba como lo hace round con los valores por defecto?


Useriøst!


hmm... I'll take a look on i2c api.


Wow. Great find!


good :)


If xbps-install uses cache on disk, try to delete the cache in the same RUN command, may be deferred for #2


tut


Sure. I would love to.

--
Nick Berardi
(484) 302-0125
Sent on the go from my phone.

> On Aug 25, 2015, at 7:40 AM, Eric Sink <notifications@github.com> wrote:
>
> I've got a new build. Would you care to test it out first before I push it up to nuget?
>
> —
> Reply to this email directly or view it on GitHub.
>


Create separate method for input validation, the method should return true in case of success, and in case of failure it should return false & and error, and that error should be passed to the show alert method


`data.codec` shouldn't be a dev dependency since it's used in production code.


If possible, we should use another fallback ntp server


That actually sounds right then.  Carry on.


mono_compile_method () looks up/compiles the method in the current domain, while you want the method from 'domain'.


Perhaps outside of the scope of this review, but this should be converted to busdma as well


Change the code, drop the redundant usage of "[ ]".


@HelenaSabel Helena: Yes, I believe there IS a more permanent way to do this, and it's the way I used with Stacey. I'll write out how to do it over in Issues! (I'd never used the solution within the XSLT debugger that you posted, and now that I look, it *is* different from what I used with Stacey and what I've set in my own oXygen installation.)


Use the monetary widget each time you've got an amount.


round por defecto lo hace al más cercano.


Make this editable in the company view


Looks good.


@ParthKolekar What'd you do to the jQueryUI file?


Ha! .... yes sir! :)


This is my second commit


4872 of 12231 logged gpus have that.  Seems to only be found for GeForce / Quadro / NVS and Radeon / Firepro.

-[Unknown]


Blocked lines shouldn't be taken into account here


Shouldn't we use Request<Void> instead of the wildcard?


Change to Excl.


The monetary widget should be used each time you've got an amount.


Makes sense. I think I was being a bit too sugary. Would you like make the change, or shall I?


怎么没有东西了？ There  nothing???


嗯嗯，谢谢。 一直想 私信说声谢谢。 那天发了推，后来想想删除了。 真诚的感谢。

在 2015年8月25日 下午1:25，vitoland <notifications@github.com>写道：

> 速度非常快。
> 不过，也要注意安全唉，尽量不要开发大而全的客户端。
> 毕竟，C大刚被查水表。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shell909090/goproxy/commit/b1e59a436ac8de1f52c0340451d8d130afb5a5c5#commitcomment-12872382>
> .
>


If you can get me an email address, I will send you a download link.  I'm eric@sourcegear.com


Can we move this function to the bottom of the file?


By the way. This is what I sent to Xamarin last night. I took your source and built my own prepackaged library. And it still exhibited the problem.

--
Nick Berardi
(484) 302-0125
Sent on the go from my phone.

Begin forwarded message:

> From: "Berardi, Nick" <berardin@amazon.com>
> To: "mac+priority@xamarin.com" <mac+priority@xamarin.com>
> Cc: "clouddrive-desktop-dev" <clouddrive-desktop-dev@amazon.com>
> Subject: Need Help - SQLite3 Issue
>
>
>
>
> I have this very basic application, that throws an error with the SQLite lib version number.  It always comes back is 3005008 or 3.5.8.
>
> However if you run the strings command on it the bundled library is reported as 3.8.11.1.
>
> strings SQLitePCL.raw.dll | egrep “3.8"
>
>
> Can you help me decipher what is going on here?  The zip file attached contains a sample Xamarin App that shows the problem.
>
> The version 3.8.5 is the version of SQLite installed on my Mac.
>
>
> However, it shouldn’t be picking it up, because the library has an embedded sqlite3.a and we are referencing the internal library using __Internal, so the external library shouldn’t be getting picked up.
>
> Nick

--
Nick Berardi
(484) 302-0125
Sent on the go from my phone.

> On Aug 25, 2015, at 7:40 AM, Eric Sink <notifications@github.com> wrote:
>
> I've got a new build. Would you care to test it out first before I push it up to nuget?
>
> —
> Reply to this email directly or view it on GitHub.
>


hah nice! de93f5cd51cd07f2f333416ab8f5d4521366931a


@cylgom I think you actually need to check the existence of all 3. There's a chance that we might not have a `$loc` for a given user, and in the case that we do, there's a chance that we might not have the `latitude` and `longitude`.


reviewing this code, I'm concerned that we're evaluating (and scanning) the whole message string. if we have attachments, this can have a potential impact on memory usage.
this leaves room for future improvements.

also...
```
 str(gpg.encrypt(
                    json.dumps(data, ensure_ascii=False),
                    key["fingerprint"],
                    symmetric=False))
```

we're totally blocking the reactor in here, am I right?


This guy works as expected. I have changed to `.on` from `.once` for diagnostic reasons, but the overall behaviour is the same in both cases.


can we create the store in /tmp dir?


tut


- There should be a space after the colon and after the comma e.g. `fname: String, lName`
- The add contact should return the error from save in case of error
- And instead of
```
if !(newContact.save(nil)) {
            return false
        }
     return true
```
you can write `return newContact.save(nil)`


Could you add some JsDoc for the opts object and its parameters?


OK, this means i'm not understanding things.  I looked at where the ring gets filled initially.  Please explain what is happening with the reduce_rings label.  Is the ring really filled with null mbufs and unowned DMA descriptors, and you need to skip over them every time around the ring?


why do we need line 54?


check typo tobascob


Nice :)


I don't really care to debate this, but .. DNF means something to me.  DF doesn't.  I'm curious why you went out of your way to change this.


Thank you
我们是不会忘记你们对互联网所做的贡献的


These two `.on` calls don't do anything. The Modal appears and seems to function properly, but the two callbacks are not doing anything. No output to the console, either from the `console.logs` or error messages.


Typo... you mean *a* HTTP referrer ;)


Grammar schmammar


It looks like System.getProperty can throw for a number of different reasons. Can you wrap this in a try / catch?


And just for thoroughness, `this.createNewRdpPart` isn't called either.


iuui


Try swaping out the `<mach/semaphore.h>` include for `<mach/task.h>` and/or `<sys/semaphore.h>` and tell me the minimal configuration that works.

(I couldn't find any helpful documentation on exactly what these files contain online)


all those unused bits (ಠ_ಠ)
I'll change it to bool, but I don't understand why it would need to reparse the bff.
BFFVariables are serialized somewhere ?


Oh, wait, I do have an email address for you.


Yes, and that meant that everyone had mousesens 5 by default, which is unusable...


Good Luck!


Ah, i just saw now, that you already corrected it to 80.

For future reference a nice github feature hint:
adding ", which fixes #226" to the commit message of a commit only containing that one line that actually fixes that issue, will make github automatically reference that commit to the issue, and close the issue once the pullrequest was merged. :D

You can still add that, if you choose to rebase the pullrequest or be asked by  the comitters to do so  (it has become a bit clutterd now :D).


This is probably OK for now, but might be a problem if we ever want to use locking outside the Rails app.


No reason not to do it everywhere. I just hadn't discovered that env var until now


Looks good.  Feel free to merge after addressing the try / catch for System.getProperty


This used to be 2000 last year. David upped it to 2500. In the long run, I think sorting the features in the name index by static importance and selecting only the top 10 or so for each name at runtime makes the most sense. JeffK is on that, I believe.


@xtraeme why '.........................' ?

It make noise in `ps` (7 lines in a 80-cols terminal):

```console
$ ps auxwww | grep runsvdir
root      1295  0.0  0.0   1728   416 ?        Ss   14:43   0:00 runsvdir -P /run/runit/runsvdir/current log: .......................................................................................................................................................................................................................restrict 0.0.0.0: KOD does nothing without LIMITED. restrict ::: KOD does nothing without LIMITED. runsvdir /run/runit/runsvdir/current: warning: time warp: resetting time stamp. .
```


Yes, it can be a single buffer, but we will need two BUSDMA maps - right?

BUSDMA'ing the control path should be a separate job, like you say.


openssl_random_pseudo_bytes was introduced in 1.0. Although the 2.0 and master branch require PHP 5.3 minimally the 1.0 branch's PHP requirements are a lot lower so we'd need to have the version_compare for the 1.0 branch. Not having it in 2.0 / master makes sense but it needs to be in the 1.0 branch if we do this in the 1.0 branch (and I think we should).


But it could still be 100, 0, 10000 I guess.


继续开发呀，是不是被我党警告了？


I think currently you cannot test changes of syzygy code in fishtest. Without syzygy-bases it's a non-functional patch.


Two empty lines between blocks are fine I think, three are too much.


W3Schools ist keine gute Referenz. Für einen groben Überblick genügt das oft, aber mehr auch nicht (siehe auch http://www.w3fools.com).

Die HTML5-Spezifikation ist zwar etwas gewöhnungsbedürftig, aber durchaus hilfreich: http://www.w3.org/html/wg/drafts/html/master/semantics.html#the-link-element

Oder auch HTML5 Doctor: http://html5doctor.com/element-index/#link


OK, let me elaborate a bit: In line 113, yes, I agree with you. But in line 112 you probably mean skip, otherwise that sentence is exactly the same as the one in parentheses. "If you don't change X, you can skip this section (but please skim it anyway)" makes more sense than "If you don't change X, you can skim this section (but please skim it)" which sounds redundant the way it's worded right now.


Why is `protected` here?


This is great stuff! Thanks @0x6368


Hi @alexey-ernest!

You can get the public IP from the AWS metadata of the current instance:

```
curl http://169.254.169.254/latest/meta-data/public-ipv4
```

This way you can avoid the dependency on an external service.

More info here:
http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-metadata.html


Thanks


Has this been tested? I think this will still send the payment information to Stripe even if the `$values['email']` is set. If you move the `isset()` check first, it should short-circuit.


0, 0 is better, because 0 is a special value that uses the normal mousesens, which is what most people want. These Dyncam settings are hidden, not available in any menu, most people will never even see them.


The structures are allocated using M_ZERO. All mbuf pointers in there are NULL.

The reduce_rings can be removed, but is there to be able to get the system up and running in case there are not enough mbufs available to fill the complete rx schedule to the initially specified size. Remember the RX schedule must not contain holes, referring to the spare BUSDMA map and mbuf.


I think it runs only when you first time touch contentItem, but I'm not sure.
@sebastienros what do you think ?


Comments Added


I already made a comment in the source code (//Leona: Is this still valid or should this also be made on an asignSlots call?). The background is:
SCHMO(E) in BTN cards: I guess this also should lead to the linkMessage REZ_ADJUSTERS ?


Addressed the BC break in e9dbc67.


Should we do similar change to tap-pin


Thank you


can we add a few inline comments for the test cases?


Thank you for all your work, clowwindy.


oupsy, this is a mistake coming from another branch.
I'll remove it.


LGTM


Oh sorry, yeah.


vf


I think we should still print out what we expect and what we got.


same here.


@moy the URL for the other listed projects points to the path of the directory that contains the Sharness tests. So I think the URL for git-multimail should rather be:

https://github.com/git-multimail/git-multimail/tree/master/t


er.....

Nothing?


> Yes, and that meant that everyone had mousesens 5 by default, which is unusable...

Isn't it like 'deafult, min, max' (0 def, 5 min, 100000 max)?

So it was 0 by default because it's checked in camera.cpp for zero to detect if feature is used at all:
https://github.com/ddnet/ddnet/blob/7aec49e6b1f57c4a509c0dfa7aec23c18969c082/src/game/client/components/camera.cpp#L139

But 5 was "reasonable" minimum set across all mousesens as lowest value.
https://github.com/ddnet/ddnet/blob/7aec49e6b1f57c4a509c0dfa7aec23c18969c082/src/game/variables.h#L91
So to be consistent we either have to set minimal sens to 0 for inp_mousesens and ui_mousens or leave 5 here as it was done before.


Created new repository for CSCI 230 Class


Переменная $per_page используется для расчета limit offset-a но не используется в sql запросе.


This template sucks a bit. Let's not fix everything now. The `original_db` stuff does not exist any more, it was recently removed (search the git log for "dbsync"), and the `original_db` stuff in this template was overlooked, so it can go (however, it would be best for it to go in a separate commit, since it has nothing to do with bootstrap).


Thank you very much @fotos


You should use "Sensors/{}/value".format(id) or models.Sensor.get_by_id(id).value()


With the wildcard we can work with any kind of Request. I didn't know for what the generic type is used. So feel free to use void, but then I don't get why we actually have a generic type in case it's always Void.


Новости обычно сортируют по дате публикации.


I didn't think I'd need that many iterations for such patch ;-). Fixed.


ended up removing it from AnalysisFile.Analyze... we should just verify that the depth thing is functional elsewhere.


in this box model the root is not in the center of the box.
this makes many things much easier for the ray/cube collision test code.


done.


This `</sub>` should go before the `{% endif %}` of line 46.


@daroczig that's strange I really though I have fixed that. I am sorry, it seems like it was lost or overriden


Nice use of computed property macros!


`escape_markup(nDisks)` leads to some bugs in CentOS 7.1 (see https://bugzilla.redhat.com/show_bug.cgi?id=1256845 )


I'm actually not sure whether sorting is required. Just iteration over nodes should be enough, I'll check it.


OK, I finally understand.   Can you unload / sync it in mlx4_en_rx_mb() and remove this check from the critical path?


hello


Right, but it's strange that innerHeight is a string. Looking at the Mozilla Docs, it seems it's an integer: https://developer.mozilla.org/de/docs/Web/API/Window/innerHeight

Perhaps an Angular thing?

Also, the Radix message appears on the Travis build for this PR:
https://travis-ci.org/hawkular/hawkular/builds/77041097#L838 .

Are you sure this is the current cause on master? The lint message I get on the PoC branch is:
https://travis-ci.org/hawkular/hawkular/builds/76994583#L841

[ERROR] plugins/accessPortal/ts/accessPortalPlugin.ts(43,35): error TS2345: Argument of type 'number' is not assignable to parameter of type 'string'.


Необычный способ получения данных. Почему не
```php
while($row = mysqli_fetch_assoc($result))
```


Yes, it seems to use the same logic.


@dcs619 These two SQL statements I added this morning, don't really need a down, as their changes don't really depend on a matching code change, and the changes they're making should have been made a while ago, and the UP for them can run multiple times.

I'd say our philosophy on adding code to the DOWN migration is that you should be able to migrate up/down, but that it doesn't necessarily need to always restore data to exactly like it was prior to migration. For example, the SQL I added here to change the pledge analytics stored procedure would not get backed out on a down migration, but the proc would still work ( and bug would still be fixed ). And if the up migration is run again, it would not error.  That said... it does look like there may have been some code in this original migration that should have had a down (pledge list block being added). We'll try to be better about getting the down code also included in our migration roll up requests.


For the 1.0 branch checking to see if 5.2 is installed is good. But the 2.0 and master branches require 5.3 for namespaces so this particular version_compare in the 2.0 and master branches is unnecessary.


remove service name


I'm not sure. Should I remove it, as svg is the only format used?


@tfennelly Sorry, but it may happen that i was mistaken about changing serialVersions. Stephen described when they should be changed https://github.com/jenkinsci/ssh-agent-plugin/pull/4#discussion_r35648514 (one of collapsed replies, dunno why GH fails on links in such cases). Could you verify it again?


Линк таскать не нужно


comments should be commented out...


Why ?


Damn, this page is used whenever something is successfully deleted. It sucks greatly. We must remember to fix it some time.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1281 of 1281 tests passed)

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


Thanks! I've been converting them as and when I hit computed properties along the way.


Done in 6c3f8018.


este footer me parece feo


Параметр $id не преобразуется в нужный тип перед использованием в запросе. Потенциальная sql injection.
sql запросы должны быть полными (заканчиваться на ; )


Pretty sure you want the value of this key to be `@NO`


We shall not use Thread.sleep in new modules. Furthermore the functinaity of this class is duplicated with SyncPollingScheduler.java  If you need some synch polling then please use SyncPollingScheduler.java (change it if needed, but for OpenStack it worked fine as it is, so probably no changes needed)


Слишком радикальный способ обработки запроса несуществующих данных. Пользователь ожидает увидеть страницу 404 ошибки а не wsod.


@robdodson I kinda thought this was fixed?


@mrinaldhar

In the chat with @amoghbl1, I was asked to write a dam story.

Mind you, DAM, and not DAMN.


missing word "in*


Magic number detected! Что будет ,если вдруг придется поменять количество выводимых записей на страницу?


@sjmiles here is the code we were talking about last night


Other than `@logger` and `@path` this method doesn't share any code with the local locking code, which is what I [predicted](https://github.com/crowbar/barclamp-crowbar/pull/1346#discussion_r37129209).  It doesn't share the interface either (local locking uses `#acquire` and `#release`, whereas this uses a single method `#remote_lock` with an `action` option), and that inconsistency prevents reuse of `#with_lock` for both local and remote.  Furthermore, there's the fundamental difference I [already mentioned](https://github.com/crowbar/barclamp-crowbar/pull/1346#discussion_r37129209) that the local lock code uses an exclusive lock whereas the remote one needs to use a shared lock.

These are the reasons I previously suggested that it doesn't make sense to stick them in the same class.  So I'm a bit confused why you are still going this route.  Did you not notice any of my previous comments, or did you notice them but disagree and deliberately ignore them, or did I just fail to explain them in an understandable way?  If the latter, please don't hesitate to tell me my review comments are too cryptic and I'll try to explain more clearly over mumble.  If you disagreed then we should also have a discussion on mumble.  Thanks for your patience!


@addyosmani not sure who would know best about this...


To preallocate space for runsvdir output.

Don't use -w if this annoys you.


怎么没啦？


so I use this pattern a bunch... where I use items.* just so I can get the required change to recalc this value.

@sjmiles on a scale of 1 to :hankey: where does this fall :palm_tree:


Also, FYI, if you've already run this migration on a DB you plan to keep, you should run these two SQL statements I added this morning (sorry, just couldn't justify creating a new migration for two sql statements, but needed them in a migration prior to our weekly production update this morning)


psr - не более 80 символов на строку.


This is the message that cpython 2.7 and 3.4 gives.
```
$ python3.4 bom-latin.py
  File "bom-latin.py", line 1
SyntaxError: encoding problem: latin1 with BOM
```

pypy 2.6 says:
```
$ pypy bom-latin.py
Traceback (most recent call last):
  File "<builtin>/app_main.py", line 75, in run_toplevel
  File "bom-latin.py", line 0
SyntaxError: UTF-8 BOM with latin1 coding cookie
```

The pypy one is slightly more clear. Should we use that instead?


暂时先用着吧。。。。发布的时候发动态库再说


Mac动态问题出在哪里。。。是连接不能么？


I just update the variable locally as there isn't any need to update the SeriesListState


Тип возвращаемых значений: bool | void. Логично было бы возвращать id вновь добавленной записи.


Some style comments ahead - sorry :)

We usually use defines for constants. If it needs the type for the calculations we add an `L` to the literal.


sql injection


I would avoid using 'scope' and '$scope' to mean different things. You could try something like $directiveScope instead.


grabbing latest.


BTW I hate to be the annoying person who says "I told you so" but the complexity of this review is what I meant when I wrote [this](https://github.com/crowbar/barclamp-crowbar/pull/1346#discussion_r37129209):

> No, this is definitely not the right PR to do that work, which is outside the scope of this sprint. We need this remote locking urgently in lieu of a change we had to revert because it was bringing Jenkins to its knees, and we've already estimated that work. So let's avoid getting distracted, get a simple solution for the remote locking working now, and have a separate story for tidying up the local locking later.


Good find. I was frantically trashing the Valet project to get it building with carthage and didn't pay too much attention to the detail.


done.


This would be more natural as a dictionary keyed by the name.


На 90% напоминает функцию news_add. Можно было бы свести в одну, назвать save и выполнять запрос INSERT ON DUPLICATE KEY UPDATE.


Reason why I still monitor the SeriesListState mode is because when the seriesList is closed, it updates its own mode to 'favorites' which then resets the mode here, only simple way I knew of to reset the local variable here so when you reopen the SerieList, it isn't set to the wrong state from previous session.


Excellent! Congrats!


Glad you found `kSecUseAuthenticationUI`! I'm actually about to put out a PR with this change (that'll still allow building on Xcode 6.4+).


I think there was a reason for me using 'i' here.
Perhaps units could be inserted into the vector and invalidate it?


I don't see where `UNDEF` is used, so can be dropped?

Also, we don't use typedefs for structs/enums (besides for some inherited code or for descriptive purposes); instead we spell the whole name (e.g. `struct xkb_keymap`).


Got your oscilloscope to work. It's great! And of course it works nicely
with the new 'listen' feature. I just need to do one or two little tests
before I fold it into my master branch.

On Mon, Aug 24, 2015 at 1:04 PM Dan Piponi <dpiponi@gmail.com> wrote:

> Not hosc. It was the Haskell midi library that crashed. In fact, there are
> to Haskell midi libraries and both crash.
> On Mon, Aug 24, 2015 at 1:01 PM Dan Piponi <dpiponi@gmail.com> wrote:
>
>>
>> http://liblo.sourceforge.net
>>
>> hosc was segfaulting on me when I fed it input from 3rd party midi
>> software.
>>
>> How's the paralysis? Does it improve from day to day?
>>
>> Sent from my iPhone
>>
>> On Aug 24, 2015, at 12:23 PM, barnabyrobson <notifications@github.com>
>> wrote:
>>
>> What's this "lo" library ? Is that available on the web ?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dpiponi/Moodler/commit/5974fb0265cb0d4a7ee46127e7c18d72ea7ee9ce#commitcomment-12864919>
>> .
>>
>>


:dart:


Since this type is only used internally, there is no need for any encapsulation here, so it'd be better to just move the definition to the header file and use stack allocation.


OLD UNUSED


<<Typo>> fix. I have a typo in my spelling of the word, "typo" in my commit message. Embarrassing.


We tend to omit the braces for single statements, but if you prefer to keep them that's OK.


@bgerstle Currently section controllers only know about the title handed to them. They don't know where the title came from. So they can't handle changes in the model. I did not want to inject another dependency (and responsibility) into the section controller.

This logic being here is intentional. Currently, it is pretty light weight… I didn't split it out into another object now because we haven't locked down business logic of sections yet. So this responsibility may go to another object when that logic becomes more complex (i.e. 3 history sections). For now it seems acceptable until those needs become concrete.


This destroys handle, returned by apiCreateThread. Rejected.


newfiles:
class/data/checkdata.p
class/data/translit.js
manage/auto.p
manage/modules/update/502/info.info


ok


一定要加这个json？我试着考了exp-calc的package.json但是没管用。。


asdfasdfasdf


you used a loop here cause the old code used a loop ;)


Cool.


@bgerstle resume notifications? I'm not familiar - have a reference?


Shouldn't it be `TukeysFilter`


Going to move this to peerDependencies instead. Thanks for pointing it out!


`npm install -g gulp-apidoc`


@bgerstle NULL for C types, nil for object types.


To clarify, I mean that the BFF parsing code is not the hot path, because it only happens when the user modifies a .bff file (which they should avoid because it causes a rebuild).  In a normal build, the dependency graph and build rules are persisted in the database between builds.

Given that this code is not used in the majority of builds, I don't think bit-packing is warranted. It we're really worried about class size, there is some unused padding a little further in the file anyway.  For now I'd just go for readability/type safety by adding a new bool.


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


Should this say `self->stop`?

Maybe there can be a single function that takes the time, and then both `start()` and `stop()` call that?


`npm install -g beepbeep gulp-rename`


Should it not better throw an error? the warning will get lost...


Comentario de prueba


See issue #3.


Relates to #24


Why would you want to take a directed graph and treat it as undirected, or vice versa?


@bgerstle there is no color in that category - do you mean somewhere else?


Duplicate of d14d0d0


@Apostolique, alright i'll use the loader.user.js then. By the way, if you want to have the code when the program is ready, simply add me on Skype: shad0w.dev


TukeysFilter ???


Для обновления до текущей версии заменить файлы:
class/data/checkdata.p
class/data/translit.js
manage/auto.p

class/data/url_manage.p
lib/tinymce/ полностью папку обновить
manage/blocks/btext/index.html
manage/i/css/cms_style_min.css
manage/modules/structure/operation.html

manage/modules/update/502/info.info
manage/modules/update/index.html

и зайти на /manage/modules/update/


We would usual write this as
if (mode == GRAPH_ADJACENCY_MODE_INVERTED or
    mode == GRAPH_ADJACENCY_MODE_UNDIRECTED):

Likewise elsewhere.


Не нашел по коду места где происходит форматирование данных перед выводом в форму. Например, двойная ковычка, в тексте названия порвет input.


I was following the order of member functions of keyIndex. I think GET/PUT/DELETE is more readable. will change.


Awesome !  Yeah I'm hoping to add the synchronization triggers soon. (btw q. about the paralysis .. it doesn't change day to day; pretty consistent)


Logging a password is bad practice.


Please make this a docstring for the function instead.


sure


Sweet :) Recursive!!!


@E3V3A Sometimes for some strange reason in Java when returning things like this they don't always work so that's why I did the parsing before returning:
```Java
String timestamp = line.substring(0,line.length()-4)
.replace(":", "")
.replace(".", "")
.replace("-", "");
```
Its obviously ok because it works on my phone but I definitely came across a lot of issues like this before and something that needs to be tested.


extra space


no need to copy the context and update it. Just use with_context(initial_bal=True) as it will update the context and not replace it.


Added the jobsscript command, which allows scripting all SQL Agent Jobs:
jobsscript --server s1wppsqlmbi01 --database BIDBA --scriptDir c:\SqlScripts\


its convention that a package name would use all lower case letters


Just two small comments:

- Please also add this test set into the TestSuiteISAFW.py (so that it can be run inside a full test suite also):

suite = unittest.TestLoader().loadTestsFromTestCase(TestLAPlugin)
unittest.TextTestRunner(verbosity=2).run(suite)

- Please also correct the spacing on lines 59, 62, 63, 69-74. Otherwise it looks a bit jumping and unorganized.


Maybe multiply this by 1000, and then just assume `NANO` and drop the enum?

We can also go the other way; nanosecond granularity is really not statistically significant. (We should also add min/max/mean/median/stddev someday...).


Should use `insertEntity`: `get404` is unecessary.

```
       (requireJsonBody :: Handler Comment)
>>= runDB . insertEntity
>>= returnJson
```


Can you add another link?


For debuging... although for users.... You decide)


I wanted to make it more strict. When i revisit it, it doesn't seem to be useful. will delete


Code review here ... anyone? :pray:


Typo in inventoryInterceptULRS shouldn't it be inventoryInterceptURLS ?


Some of these cases need the trailing whitespace because that is how you add a line break in Markdown.


ok


Ooooops, forgotten debug log.

Thanks !


Высокая цикломатическая сложность. файла. Есть более подходящие подходы. В самом простом случае fsm.


develop (DavidGoldwasser) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2285 of 2343 tests passed)

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


Nicely set up.


scheduleCompaction `defer s.wg.Done()`. If we don't `s.wg.Add(1)` before, s.wg will panic for negative counter.


Looks good; some comments.


For the first one to work you also need to add this line to TestSuiteISAFW.py:

from LAPluginTestCase import *


Remove metadata


Thanks for the clarifications :)


This is reasonable and I think I need to change all.


When loading existing ContentItems, yes the `OnInitializing` code is run, but this happens early in the process, before any existing data is read out from storage.
So the fields set by `OnInitializing` are discarded moments later and overwritten by existing data.


Did we really need to add hello back in? :-1:


You might want to say "graph is {{p...}}" to be a little more explicit.


lgtm


what was this for?


How about we use `print` instead of `puts` here?


Não sei como fazer para ele guardar qual Operação ele vai fazer, =x se for pra fazer soma, da certo asdhusahudhusaduadhusa, posi a soma ta no sinal de igual =x teria que fazer algo nos sinais.. mas nao consegui ainda.


why did u forget


@gregwebs The code you're commenting on is fairly outdated btw; latest is here: https://github.com/yesodweb/yesod-scaffold/pull/76/files


Missing space before the `=`.


You might want to set this up in a similar way to regularity/acyclicality/... so that you have freedom to add options like "is a subgraph of" within the same rule.


Forgive me :3
@akhilnarang was lazy that's why
On Aug 25, 2015 9:14 PM, "srisurya95" <notifications@github.com> wrote:

> why did u forget
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team-Radium/frameworks_base/commit/d08057d3cebfb0c3e48b62402837e1e58bdf75ee#commitcomment-12883249>
> .
>


Yep, it's in the package.json. Just run `npm install` and you'll download the required dependencies.


please add a comment for this. thanks.


You can pass NULL to `g_free()`.


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1792 of 1793 tests passed)

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


http://www.gov.cn/flfg/2010-02/26/content_1544458.htm

《中华人民共和国著作权法》第四十八条第一款第（六）项指出，破坏权利人为其作品采取的保护其著作权的技术措施的行为，应当根据情况，承担停止侵害、消除影响、赔礼道歉、赔偿损失等民事责任；同时损害公共利益的，可以由著作权行政管理部门责令停止侵权行为。代码仓库中的作品没有相关法律、行政法规额外指出违法之处。

另：Github 还在遭受 DDoS 攻击。


Why not ```Arrays.indexof(a)```?


It would seem more natural to make a positive test that all the labels and edges match, and if they do then return True. Otherwise return False after the loop ends. That should avoid having to use 'found_isomorphism' variables.


Ended yellow:

LLR: -3.58 (-2.94,2.94) [0.00,5.00]
Total: 49041 W: 8267 L: 8219 D: 32555
sprt @ 15+0.05 th 3

I will wait till the dust of the other SMP tests has cleared and rebase on the new SMP master and test with 7 threads instead of 3. Local tests with 4 threads indicate that it combines well with http://tests.stockfishchess.org/tests/view/55db7bc20ebc595da9433914


@akhilnarang you are never forgived


@bgerstle It's here because the next line will blow up if it is nil. So thats a pretty practical reason to bail here.

It's also well established pattern to check for nil and bail at the beginning of the method - to be explicit about the requirements for the method. In this case, pass nil for a section… and we don't load it.

Also you seem to be conflating an "empty section" with a "nil" section when referring to empty view logic… they are not the same thing. We do not show an "empty section" when a section is nil - a section must be backed by a section controller… even an "empty section".


done


Hiding pointers behind is generally a bad idea IMO. Pointer and non-pointer have different semantics when assigned or passed as parameter and can be unexpected when using `const`.


I'm wondering if we should call this `style` instead, just to be consistent with the other rules.


Still seeing how this shakes out… don't quite no what we should do with this logic yet… we can discus and refactor.


A test for graphs with different numbers of vertices would be nice.


It was close to a good work around. It wouldn't full clear the state of the recognized anime and would stop Taiga from recognizing other anime on CR immediately. I made some more changes that seem to fix that issue.


Same thing here… starting to find the pattern… but not sure how we want to abstract it yet.


@TheFDGamer I don't think I'll be making a script for Bad Eggs Online 2.


What happens if one graph is directed and the other is not, or one is labelled and the other is not?


I had a few comments, but besides that this is much better than the current situation!


JSDoc ?


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


Instead of just returning `null`, we throw an `Error` telling the user something's wrong. Take a look at [borderZero](https://github.com/lesshint/lesshint/blob/master/lib/linters/border_zero.js#L50) for example.


Looks good, nice tests!


Add a comment / docstring to this file to explain its use


`npm install -g beepbeep gulp-rename`


Relates to #24


There are 2 tests which do not passed in ServerHandshakerTest since this commit (c7b90bb53fbe54468e5ce458d5eb7116c752c649) :
- testReceiveClientHelloIncludesUnknownCiphersInHandshakeHashGeneration  => NPE
- testReceiveClientHelloNegotiatesSupportedCertificateType => NPE


I think the original implementation was better. Because, when we dont find site settings, we create one with default values. Doing this check on each time it gets initialized does not sound right despite it gets cached.

I might actually consider this logic going into Orchard setup module than any of these.


Coconut WATER!


@jossmac @JedWatson be aware, changes in `1.0.0` of this package


add


Le pifie al crear la clase! tengo que Borrar


I see what you did here ;) But please add a comment


renamed to allDependencies


Oh yeah, sure. I wasn't aware that was a thing...


@jossmac @JedWatson be aware, this has changed to the above in `5.0.0` of this package.   Turns out `Timers()` was a bad pattern due to the storage being done per-class not per-instance (woops).


qdel?


Without knowing much more about how resources are handled here and such, I'm a little concerned about trying to use `time.sleep` to prevent the daemon from spinning up too much.  What do you think of starting a process in a thread and communicating to it through a blocking queue?


Sounds good by me


just a note for a future refact :+1:
a `static inline _pdb_example_free_values` could be factored out from here


colour macro's are deprecated, use span classes.


:mouse:


Can we have the same comment as above here too please?


This is how one add excerpt for each post!


На мой взгляд эти блоки фильтров не навигация, т.е. не совсем уместно использовать теги `<nav>` для блоков фильтров.


setting file_dir is generally a bad idea.


Ah so that's how you do it!


Note that the download URLs returned by booxtream are not secure.


Interesting, I didn't know that.

Why do we care about TCP though? I think the default for recent X now is not to listen on TCP, but if that's not the case on Mac OSX than we should pass `-nolisten tcp` to `Xvfb`; that's a good idea anyway I guess. Then we can use still use some high range where there's no "contention".


/cc @azamorano


Thank you


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1798 of 1799 tests passed)

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


Besides the possible simplification in the comment, this looks good too.


OK, but I thought there is no other place to place a user guide. If I was a user, I would care.


Подписи фильтров нужно обернуть в тег `<label>` и связать с инпутами по id.


This just breaks my heart.


Does this mean we now have a "standard" `manage.py`? That would be awesome


Do you have 85b79a33320b615c923a6a03b6d295d7543b3f55 from Aug 4?

with git master as of Aug 25, a real PIV and a Neo show:
card-piv.c:450:piv_general_io: 87 07 9a 266 : 255 256
but yours shows:
card-piv.c:450:piv_general_io: 87 07 9a 266 : 65536 65536


There's no auth component, we could rename this method.


OK


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


And I even wrote the unit tests. :broken_heart: :hammer:


Docstring here too


Seems like this is not quite the right syntax, as the travis build keeps barfing on it.


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This change actually broke the linter :grimacing:
I think the formatting was fine without this.


Although not fond of this change, since IMO this is a security feature, not allowing path to be modified by nasty input (such as hawkular/inventory/env/feed/resources/../../../../private/folder/passwords.txt or whatever may harm the server), proceed with applying.

I think in the future this should be handled server-side, decoding the %2F and verifying/validating the path.


another note for a future refactor: this function is HUGE :-)


why would you want to replace /turf/space with snow, instead of creating a valid new turf? This will cause a lot of issues and problems.


Gracias por añadir los comentarios!


Baby yisus craist doesn't approve this PR :)


Before Cocoapods


@bgerstle Yes - its called whenever a view is resized, regardless of how it is inited. This is the behavior I was looking for.

The bounds is **always** up to date after calling setBounds. (a layout pass may occur inside of the super implementation if its needed)

In general:
If I need to layout subviews (change the view hierarchy) in response to changes in the hierarchy, I override `layoutSubviews`. (https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIView_Class/#//apple_ref/occ/instm/UIView/layoutSubviews)
If I need to perform logic based on the bounds/frame that does not involve changing the view hierarchy, I override `setBounds`.


Interesting, I had no idea this was in there.  The context for my question is that I am playing around with removing the find tools functionality (at least for tools that we are shipping).


This seems odd. In the non EDITING block below, hitting enter sends \n, but here it sends \r\n. Which is right? I default Raw to \r\n, but this needs review.


ah I see :D


@pavolloffay can we do this temporarily, and fix this on the serverside in the long run?


/cc @azamorano


I wish you had provided an example vector or list to work with makeVector and cachemean as I was unable to get these to work. For example, with a=c(0,1,2,4,1,1), I received this error: "Error in x$getmean : $ operator is invalid for atomic vectors"


develop (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2309 of 2343 tests passed)

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


101615016CoilCoolingDXMultiSpeed-requires-Waste-Heat-Function (lgu1234) - x86_64-MacOS-10.9-clang: OK (1792 of 1792 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)


Насчёт блока товара есть упоминание в ТЗ, которое скачивается в пачке с макетами:

— Карточка товара: изображение товара — это ссылка на страницу товара.
— Карточка товара: название товара — это ссылка на страницу товара (название состоит из 2 строк, но по сути является одним блоком).
— Карточка товара: кнопка «Купить» — по клику товар добавляется в корзину (макета корзины нет, не делать).


Два разных блока с одинаковым классом `.price`, нужно исправить.


Bu dekoratör zaten django da var.


@arunk-s , would it look nicer like this?
def f(x):
    return {
        'a': 1,
        'b': 2,
    }[x]
I found this approach on stackoverflow, idk if something that looks nicer exists :)


develop (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: OK (2305 of 2343 tests passed)

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


Merged command and userCommand


Hi JaeHyuk,

Please review our hookup guide for this sensor [here](https://learn.sparkfun.com/tutorials/mpl3115a2-pressure-sensor-hookup-guide#mpl3115a2-overview). It sounds like you have an issue with your I2C timing out, and may need to double check your connections.


В принципе можно эти блоки сделать тегом `<article>`, но это некритично, можно оставить и просто дивом )


У одной из кнопок пейджинации должен быть выделенный класс.


Список тащемта избыточен здесь, можно просто ссылки подряд.


Kullanıcının authenticate olup olmadığını anlamak için böyle birşey yapmana gerek yok. Template seviyesinde is_authenticated bilgisine ulaşabilirsin. Bir de login_value zaten çok anlamlı bir isimlendirme olmamış. Bu tarz birşeyin olup olmadığını değerlendiren boolean ifadeler için is_... şeklinde isimlendirme yapabilirsin. Mesela is_logged_in gibi


*cheat-sheet


@844196 現在はリツイートされた呟きが別ツイート扱い(?)として表示されていますが、これって要りますかね？
こちらとしては、同じ内容なのに複数表示されているので見づらさを感じました。


Woops on me, my build folder must have had the stupid files in /img/ before the move a few days ago.


Will this interfere with the use case(s) of:
1.  I have a herbarium specimen, I create an Observation but forgot to check the specimen box?
2.  When I create the Observation, I have not yet decided whether to retain it as a herbarium specimen.  Later I decide "yes".
In both cases I want to add an herbarium specimen to an existing Observation which has no herbarium specimens.


Hmm, just occurred to me that I wasn't here all last week so you wouldn't have been able to ask me to explain any of my earlier review comments then ... that might explain the confusion :)


这个才是标准化的json。。。那个package.json是需要npm init建立的才比较标准。


Where is the `create` coming from?


Burada prefetch related yerine select_related kullanmalısın. Aksi halde çift sorgu çalşır. Prefetch_related ve select_related arasındaki farkı incele lütfen.


Can you let mbernasocchi please


@catong I've changed this to show the singular/plural text according to the number of team members.


Ana sayfada da burada da tüm blog postlarını listeliyorsun. Neden böyle duplicate bir view a ihtiyaç duydun ?


Секции на мой взгляд нужно применять только для блоков с заголовками. Об этом сказано в спеке ) Для простого разделения по блокам нужно применять безликие дивы.


Aynı optimizasyon sorunu burada da var. select_related, prefetch_related...


@mosra This should probably mean `Vector2::yScale(-1.0f)`?


Fixes #7


just revert back to an earlier version next time! not change something then pull!


@rwaldron can we uncomment this block?


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


It's not an error ;)))


ok


Category_Choices şeklinde bir isimlendirme yapamazsın. Pep8 e uygun değil. category_choices olabilir.
Bir de bunu sanırım dinamik yapman daha mantıklı. Category diye bir model oluşturup oraya bağlayabilirsin ama şart değil.


You should use `xkb_state_get_one_sym` here; multiple keysyms are an xkbcommon extension and are meant for things like unicode combining characters. Since X doesn't support them, you shouldn't bother here.


Will this take care of pages free'd some where within the chunk? The mm_pgsleft is only decremented, but how would you know which ones are free'd? Additionally, the memory is left unsed as well and cannot be allocated later. Is that intentional?


Ahh, right, yes, thank you! Wrote this from the top of my head and apparently I can't rember my own API anymore :)


@nunoguedelha main changes:
  * Added a Vector6 SpatialVector::asVector() method to SpatialVector, and used it where we were assuming that Twist/Wrench where descending from Vector6 .
  * Added swig warning supression for warnings related to CRTP pattern.

Not all test are passing for now, so please wait before merging!


Comment on a line that _will be_ modified in a subsequent commit.


Will try it for surely and discuss with Tao and other mentors and if possible changes in the next commit. :)


Comment on a line that _will **not** be_ modified in a subsequent commit.


Looks good to me. Happy to leave as is.


Buradaki isimlendirme sıkıntılı. Sen kategori değil kategoriye bağlı blog postlarını çekiyorsun. Buna uygun bir isimlendirme yapmalısın.


GNÉ ! J'AVAIS PAS CRÉE DE TYPE FILE !!!!!

PS : ma maman dit que je suis spécial ....


@aakpro  see #2901


Thanks, I'll fix that.


То же, что и в магазине. Тег `<section>` это про блоки с заголовками больше. http://html5doctor.com/the-section-element/


As you noted earlier, this is a bug (I wrote).  A `fee` of zero is valid.  Will you be fixing that and the associated error message?


reply


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1798 of 1799 tests passed)

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


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (751 of 751 tests passed)

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


I believe this mention of `version` and all following mentions of `version` should use `verifiedVersion`. That way, `latest` is converted to an actual version number and we've ensured a requested `version` exists.


(FYI, the latest attempt to publish automatically [failed because of a typo here](https://lists.w3.org/Archives/Public/public-tr-notifications/2015Aug/0021.html) :)


you


a code is compatible for raspberry pi2?
i have problems in gpio wireRelay


Sorry for being difficult here, but I am a bit confused on this. On each iteration, wouldn't it still just be setting antivirusName to the value of the last iteration (plus ;)?


whooo :beers:


Burada kullanıcının blog postlarını render ediyorsun sanırım. Bu isimlendirme de hatalı. View in ne yaptığını anlamak için diğer geliştiriciler her seferinde bütün kodu okumak zorunda kalmamalı.


You are right, now I understand what you mean. :worried: My bad.


We should provide .cmake files for plugins and clients to easily compile/link against the parameter framework. Opened #159.


Hooray code deletion!


Тут такой момент, что получается, что есть `<h2>`, но нет `<h1>`, что есть не совсем правильно. Как выход, но немного упоротый, сделать верхний большой логотип тегом `<h1>` и скрыть в нём текст. Или же попроще — сделать заголовки второго уровня незаголовками )


Blog postunun owner ının user ın profile modeli olması biraz mantıksız olmuş. Postun owner ı user olabilir.


Kullanıcının adı var zaten. Neden böyle birşeye ihtiyaç duydun ?


Only if you think it is fine to split the function in two parts. Because it is not correct to unmap the buffer in case there is no spare available, which happens if you unmap the mbuf at the top.


ik, thanks for remembering


It looks like we won't be able to search/filter based on resource type.


e e


test


yup, this LGTM. Like the simplicity. It looks like this code will now define CustomEvent if it was falsey (0, false, '', [], null, undefined?) realistically I think we'll only see the undefined case, in which case, it seems good to go ahead and define it.


Category_Choices gibi bir isimlendirme yapamazsın. Normal şartlarda herhangibir programlama dilinde bir değişken ya CamelCase, ya snake_case olabilir. Pep8 e göre snake_case olmalı.


Because the BanFetcher and BanCleaner threads are (as of this PR) daemon threads, they won't block the program from returning immediately. In the previous version they would frequently misbehave and ignore signals and so on. The ```time.sleep``` is simply to stop the main thread from returning immediately and also from spinning and using CPU. Does this explain my logic in making this change, or do you think it could/should be done better?


Yes, this is uncommented in latest commits


Fishtest does not support syzygy, I delete the test.


101682488_EvapCoolPrimaryAirSizing (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1789 of 1795 tests passed)

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


Hmm comment anyone?


Ah my bad!


Line notes go in here


Comment objesi blog postuna yapılmış bir comment veya comment e cevaben yazılmış bir comment olabilir. Dolayısıyla burada foreignkey bağı olamaz. Generic foreignkey kullanmalısın.


@johnbellone actually, if you leave that flag in, you can easily take advantage of [Travis’ bundle caching](http://docs.travis-ci.com/user/caching/#Bundler).


Will fix.


101064176-AddCO2DemandControlledVentilation (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1798 of 1799 tests passed)

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


Comment modelindeki alanlar zaten comment modeline ait olduğu için tekrar comment_ şeklinde bir prefix ile isimlendirilmesine gerek yok.


@keepoff07 リツイートに特定のクラスやIDが指定されていると`display: none`等出来ていいのですが、現行の仕様ではないようです。

![tumblr_nrx9v18trd1sp4ql3o1_1280](https://cloud.githubusercontent.com/assets/4990822/9472637/b2e3aa96-4b91-11e5-87b9-6ca1de2c9e03.jpg)


You should probably use `utf::foldcase` here, and probably should have a `ptb_normalizer` in the filter chain before invoking the crf.


Do we need to do anything for headless services other than just ignore them? That doc says "DNS is configured to return multiple A records (addresses) for the Service name, which point directly to the Pods backing the Service.". If they're pod IPs, then they're already isolated.


@pulkitsinghal, I've recently developed a simple Chrome Extension to view products from Vend API, check it out on [Chrome Web Store] (https://chrome.google.com/webstore/detail/vend-client/lgiljbkccbplmikkcenfbhokofjpffob). It's in nascent stage and it's something i'm doing in my spare time. Please let me know if that aligns in any way with what you are doing with this repo.


OK。


@geekyme is the promise required for react-router or from the redux side? I don't understand the need of a promise in a middleware. Shouldn't it goes through the whole middleware chain before rendering the page?


@lalinsky, don't we need to fix ```taglib-config.cmd.cmake``` too?


Typo: `After in *.drl:`


I ended up adding the gagesII csv despite its size


이 부분은 강의 내용 중 배운 문자열 메서드를 이용한다면
'Book Management System'.center(60, '*') 식도 괜찮겠네요.


Burada çok fazla gereksiz sorgu çekiyorsun. Kullanıcının blog postlarını zaten almışsın. Sonra tekrar kullanıcının profilini sorgulamışsın ve tekrar arkadaşlarını sorgulamışsın. Optimizasyon gerekli.


Is it just me, or is a funtion with a '-' in its name instead of say a '_' just looks plain wrong?


Inline comment from code review


Oops. I knew that but I forgot that I hadn't implemented it. So, we could either make Service have an array of Protocol/Port, or else we could just emit two Service objects with the same Name/Namespace/IP but different Protocol/Port.


View, kullanıcının blog postlarını alırken bir anda arkadaş ekleme formuna dönüşmüş. Hayırdır ?


In order to get the policy to compile, it was a choice between adding -untrusted_app_noexecute to the list of domains or adding -asec_public_file here (as I think untrusted_app_noexecute needs to be able to execute asec_public_file just as untrusted_app does).

We could potentially instead add -untrusted_app_noexecute above , then add a separate neverallow { untrusted_app_noexecute } { data_file_type  -system_data_file -apk_data_file}:file no_x_file_perms.


cambio!


İsimlendirmelere dikkat...


Sounds good to me. Can you submit a pull request with that change?