TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7398](https://teamcity.mesosphere.io/viewLog.html?buildId=83281&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:11:55


We may want to figure out a way to customize this, but for now it should probably stay a local change.


Let's improve grammar in commit description.
its vmas so it would be natural to declare them as const.


thanks! this was needed.

however, we could/should just remove the test/requirement to console log a particular string and then we won't need these stubs.

@sgnl and @JoeKarlsson1 you can decide what you need.


Ok


I think this was trying to fix the same thing as #1371, and is probably no longer necessary.


干得漂亮~


dd


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7397](https://teamcity.mesosphere.io/viewLog.html?buildId=83264&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:08


This can't be static. Info should only be given via callback if player is allowed to know it. This is general cheat-proof design and encapsulation.


一个简单的WCF Restful服务的例子，包括服务的配置以及方法的实现


I don't understand the purpose of these two template changes.


cool


Oh, that's interesting! I never observed `libvirtd` crashing with enabled `MPROTECT`. (`qemu` needs disabled `MPROTECT` and `RANDEXEC`, though). How did you observe this?


Turns out this is a bug of my own again.  NetBSD uses asprintf(), and thereby avoids
this problem, but asprintf() isn't sufficiently standardized to use here, I suspect.
Additional suggested fix:
https://github.com/ntp-project/ntp/commit/1ee4072f732b8d72cc063de78c242641e03521f9


So I have a commit already that has the location, version and expected version, and I'm thinking through the documentation side.

Do you think details on how to fix should simply be a multi-line exception message? Let the exception then propagate and be logged or reported to whatever naturally catches the exception? Perhaps the exception can extend a base class that will support outputting the error to CLI, if the exception is caught at the CLI level? I'm thinking it could then highlight the error as red and output additional details, but hide the stack trace.

Maybe something like:
```ruby
class Gemstash::ReportableError < StandardError
  def puts(cli)
    cli.say cli.set_color(message, :red)
  end
end
```

Then the base CLI class could catch this type of error and call puts... subclasses could override and output different things (like a bold + red header and red message... I'm thinking that might be good for the output of this error in particular).


Thanks I saw the link is broken now. Unfortunately, there's no copy around the web of the original source where I got the inspiration for this exercise. I will remove the link and leave the one on the dataset description.


I was here using a dictionary where the key is the 3-mer and the value is the number of how often it appears. I was using a `defaultdict` which defaults to 0.


This should be a template override if they care.


Hi Benedikt! No, `map` is not lazy (on Arrays) - you can see it if you add a logging output in the map closure.
However, `reverse()` is very fast and `insert(_, atIndex:0)` is very slow (because it has to shift all existing elements to the right).


@majianpeng i mean in both cases, `ReplicatedPG::on_change()` already takes care of the cleanup. so we don't need to do repeat it here.


Is the value of `YCMDPATH` correct? If I'm reading things correctly, it seems like it ought to be "../ycmd/ycmd" since ycmd and emacs-ycmd are siblings.


Yeah I removed this line in a later commit haha... The CM repo script was just expecting a different folder for the dts files.


Grammar improvement.

To track anon shared VMA pages dirtiness we create a bitmap for it. Each bit in this bitmap corresponds to particular page in VMA. Bit with value of 1 in this map states that page is dirty. Bit 0 states that the page is clean.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7399](https://teamcity.mesosphere.io/viewLog.html?buildId=83284&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:08:07


BTW, I'm really happy to see this kind of work. I'm a big fan of test automation, and this is a big step in the right direction for emacs-ycmd.


Not quite sure what your comment means, but nice catch on the typo


ok.


You have two wait loops in this function.
The intention is that drivers shouldn't block the operation. Uclass driver (serial-uclass.c) expects that device driver returns -EAGAIN, when busy for getc()/putc() methods.

Please check this:
https://github.com/hallor/u-boot/blob/dragonboard/drivers/serial/serial_s5p.c#L141


/n


The name of this method should explain what the method does i.e. count post donation counselling that is not flagged for a donor. It also needs tests.


Fields in this struct are formatted so please add spaces between new fileds types and  names.


Definitely need to move scrolling side-effect to middleware. It will cause troubles on server-rendering and isn't 'idiomatic redux': main property on reducers is their purity.


А в чем проблема в грамматике сделать real?


@questbeat this bugfix would help on 2.x too


I never got this to boot, so I can't say. These are just my own guesses. I gave up on CM and started working on pure AOSP instead.


上海数天信息科技公司版权所有,归所有内部员工使用。


Instead of this four lines, doesn't `angular.mock.module(deployModule.name)` work? It should. It it does not work, this means that deployModule has incorrect dependencies declared, which you can easily fix :)


As far as I know the qualification is necessary for Q_SLOTS; it might work now without because you'll use the not-string based version.


Awesome, thanks for investigation!


@fairlight1337 Thanks for the patch. I tested both with a dummy example, and my pick-and-place application with live-logging and live-querying. Both seem to work fine. :)


Oh. As a good guy you need to write this macro likes this.
#define BLOCKS_CNT(size, block_size) (((size) + (block_size) - 1) / (block_size))

This is classic. suppose you call  BLOCKS_CNT(1024, a + b);


I was thinking about raising an issue about this to always set it to false because it caused a problem with my cpanel hosting.
When sending in the clear using port 26 phpmailer tried to switch to tls, but the mail server doesn't allow that on port 26, only on port 465, so the connection attempts repeatedly failed.

The phpmailer default for SMTPAutoTLS is true so there is no need to set it here, but I don't really see why it needs to be enabled. I have now changed my settings to explicitly use SSL with port 465.


OK, maybe not. :see_no_evil:


Yes, Actually, I mentioned that user needs to ssh to EMR master on line 22
But, some people might prefer to build tar.gz not on EMR master but on dedicated build box of their laptop.
So, yes, in this case they need to ssh to master before that step (line 120).
I'll fix it


Does this mean Signal iOS now uploads the numbers of all my contacts to OWS servers?


Looks like tool does not exist in 6.4 branch (see Jenkins build failure).


FYI none of these f460 repos of mine are functional. Just playing around, you can see I didn't even bother to write commit descriptions. You can find my email on my profile.


This commit caused a compiling time regression: http://caml.inria.fr/mantis/view.php?id=7067


:+1: as I got an error with a full traceback today.


On 11/27/2015 02:22 PM, Lennart Poettering wrote:
> This defeats the usage of using automount units to parallelize bootups, no?
>
> Consider you have a slow fs, and want to continue booting, without
> waiting for it. So all you care about is that the automount unit is
> established, but it doesn't have to be backed by anything yet, because
> you know it will be backed as soon as somebody accesses it.
>
> Or am I missing something?

I must admit I may have overlooked the automount case but I though that
a mount unit with the automount option set shouldn't be 'auto-started'
at all during the boot process but only when the automount path is accessed.

So adding an ordering dep shouldn't have an impact on them...

... and after looking at the fstab-generator one more time, I can see
that a mount unit with an automount option is pulled by the
local-fs.target which seems wrong since it will be activated
unconditionally at boot.

Giving a quick test without this patch applied seems to confirm this
behaviour and applying this patch seems to help:

http://susepaste.org/14893435

>
> Also, we so far told people to use "nofail" for devices they don't want
> boot delayed for. With this change that is removed too, no?
>

Well, that's supposed to be re-added by the next patch of the series I
sent: "fstab-gen: use the smallest device timeout value when 'nofail' is
set". This  patch uses a device timeout set to its smallest value in
order to not delay the boot process. It's kind of hack but at least we
don't try to unmount fs too early, see previous comments and commit message.

Thanks.

PS: can I send patch to the systemd mailing list instead of using this
webui which seems to bring more confusion than any goods ?


this looks really strange from interface perspective - we have `done` method but we never start anything with start, begin, etc


Agreed


please check return code of resize_shmem_pdirty_map here


Why it is needed?


True but gitdown cannot handle them.


Don't commit this ;) But you can disable the chrome extension temporarily from Chrome.


sure, this PR is not done, do not merge ~~~


Why just not itterate items?


Martine, Frotnendisti.cz jsem dal pryc. Slo spise o subjektivni hodnoceni, nez informace. Pro me bylo sice zajimave si precist, ale nebylo by pro nezucastnene. Dava ti smysl?


Awesome commits. Thanks! Really looking forward to using Emacs with Swift now :)


Move the 'default' option to the bottom.


If checkpatch.pl script doesn't scream about this, then this is okay,
otherwise maybe it should go to arch/arm/include/asm/arch-msm/serial.h?

The same for other files from this patch set, which uses such convention.


How about check here type of formatter?


This call is repeated for several times and the only things that's changing is the first argument.
Collect the argument value into a separate value and move the call to views_date_sql_format() outside the switch statement.
This would make it little bit cleaner.


Fixes issue #78.


How are these benchmarks to be used?


Why DataCenter:datastore1 rather than using an empty version? are those names guaranteed to exist?

If this is supposed to be a hint, then the hint should be rather used in the ui.


Is it really necessary to add `os: osx` if you already have the `osx_image` key?


Martine, tohle je určitě důležité, pojďme zkusit rozpracovat:

- Navrhuji odkazovat spíše na blogpost: http://zurb.com/article/1416/foundation-6-is-here a vyjít s informací o novinkách tam.
- Tvoje sdělení "Přináší nový způsob práce" navrhuji upřesnit. Myslíš ten Yeti Launch? http://foundation.zurb.com/develop/yeti-launch.html


Multiple declarations of my $e here, triggering warnings:
```
"my" variable $e masks earlier declaration in same scope at t/110_bignum.t line 38.
t/110_bignum.t .............. ok
```
(line number since changed)


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (916 of 916 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-25.6%25-green.svg)


imho this could also be under .vsphere.datastores_hint


Mezery mezi UL/LI jsem dal pryč, ať to nemáme roztahané. Je OK?


###References:
1. [Paperclip thumbnail generation](https://github.com/thoughtbot/paperclip/wiki/Thumbnail-Generation)
2. [Paperclip class methods](http://www.rubydoc.info/gems/paperclip/Paperclip/ClassMethods)


@traversaro , the semantics is supposed to be a transparent processing to anyone using the class Transform. `setPositionSemantics()` and `setRotationSemantics()` are to be called only by `Transform::setPosition()` / `Transform::setRotation()`. And in that case, the function call is already encapsulated by `iDynTreeAssert`.
I guess you did this because you're using semantics directly in `DynamicsComputations::getRelativeTransform()` for instance. In that case, I would use `iDynTreeSemanticsOp()` right there.


rename to getNumberOfIndexes


Typo etarchive


Good catch. Thank you.


부분 일치 테이블의 정의
table[i] = STR[0...i]의 접두사이며 접미사인 최대 문자열의 길이 (자신 제외)


Tahle sekce je perfektní! To jsou přesně ty věci, co by mi chyběly. Díky. ;)


as resize_shmem_pdirty_map function states about resizing not expantion then we need to handle cases
when mem_size == si->pdirty_map_size and < then it.
If == then explicitly do nothing and return. If less then realloc.
Otherwise you can call this func expand_ and not write additional not needed code suggested above. (i would just rename - that's better we can discuss why).


dghhgfhx


formatting ?


Can we use it? I didn't find this in docs.


I would assign Inst Indexes to local variable


Should somehow express that this is the path on the datastore, not on the glance node..


이소스빌드되는거맞는건가요?
Gproj가웬지 cm13부터드랍당할거같아서말입니다


formatting


@fpodoo The disadvantage of committing without testing is that it doesn't always work; in this case the group is `analytic.group_analytic_accounting`, not part of the `sale` module. Therefore the field was never visible.

Fixed as of 8469a4dea0d62df0cca35f0e69eadfabe70e219d


@fpodoo Please update the .pot file when doing this kind of change (but we try to avoid changing translatable strings in 9)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7402](https://teamcity.mesosphere.io/viewLog.html?buildId=83291&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:58


copy comments from original code


might want prefix your commit message with "Objecter: " instead. could also squash these two commits into a single one.


you didn't copy:
```
if (id == 0)
  continue;
```

why?


Great! If any side effects come up, you're welcome to file more issues that I will then look at :).


Looks like pdirty_map_size can be computed using size if size is always multiple of PAGE_SIZE.
This should be true. Then it is btter to remove pdirty_map_size.


good


probably we need to remove ALT_MATCH at all for now since it is not generated


TeamCity Habitat :: Cruise [Build 301](http://build8ua1:8080/viewLog.html?buildId=758&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:25


It seemed to work in my testing, but I'm willing to believe you regardless.

I'll update just now.


So, I missed this below as well, but in JS there's no %s. Just remove the %s bit, so that the line becomes

    'string, got ', subjectInterests[i]);

Same on line 63.


Thanks. I was hoping for some feedback. I left it like this, to be aware of the setting, and possibly allow a specific config in phpList. I should be safe to use, as the fallback is as before. It may add some overhead in delivery.


1) We don't need to unlink non-exsiting one.

2) On  a sane service, uploading a file should be implemented with 2 phase - creating a temporal file on 'garbage-cleanable zone' and  moving it to destitiation path. This logic shows why. Current 'writeFile' is poorly implemented, and we might lost existing file when writing is failed or stopped by some reason (process crash, for example) for we will unlink the file before writing a new one.

Anyway, I'll merge this code. We need another batch job to handle 'uploading' properly.


This was a direct cause of the typos.


Sorry, I don`t know you don`t speak Chinese. :p

I just hope any original android work on F460.

good luck~


Any reason why we can't use project scope installation by adding it to our composer.json?


First off, just note that the AddressBook functionality is SUPER old.  It was [deprecated and turned off by default in MT 4.2](https://movabletype.org/documentation/appendices/config-directives/enableaddressbook.html).  So, continuing to support this is questionable and the code is likely going to be a little less than awesome which means confusing...

To your questions:

> What is the purpose of the $params{from_address} = $address; line in AddressBook.pm and why is it placed where it is?

`$params{from_address}` is used on [line 115](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L115) as part of the argument hash to [MT::build_email](https://github.com/movabletype/movabletype/blob/master/lib/MT.pm#L2714).  `$address` is used further down in forming the email header (note the lexical declaration on [line 103](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L103), outside the block?).  They could have obviously done:

    $params{from_address} = $address = $app->config('EmailAddressMain');

...or even simply used `$params{from_address}` later instead of `$address`.  I'm not sure why they chose to be so verbose about it... (my guess: old code).

> I can see that line only gets triggered if there is an author, as moving [this line](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L112) down two causes no author notification to be sent.

Moving that line down two changes it to this:

    my $address;
    if ($author) {
        $address
            = defined $author->nickname
            ? $author->nickname . ' <' . $author->email . '>'
            : $author->email;
    }
    else {
        $address = $app->config('EmailAddressMain');
    }
    $params{from_address} = $address;

That sets `$params{from_address}` regardless whether there's an `$author` or not.  Maybe that's how it should be and maybe that has weird effects.  In any case, we're talking about old deprecated code and, without researching it a bit more, I'm not sure what their intention was.  In any case, none of this should matter one iota.

> How can I prevent the duplicate e-mail sent to the author without modifying core MT code? It seems if the only notification to be sent is for the author, I need to provide the author address, but doing so results in two e-mails

Well, regardless of what the core code does in preparation for sending:

1. The email body, email headers and other related parameters always [get passed](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L135) to [`MT::Mail::send`](https://github.com/movabletype/movabletype/blob/master/lib/MT/Mail.pm#L23) which does the heavy lifting of the sending.
2. But, before it sends the email, it [executes the `mail_filter` callback](https://github.com/movabletype/movabletype/blob/master/lib/MT/Mail.pm#L141) with references to the headers, body and other parameters
3. This puts them fully in NotifyWho's control since it implements a `mail_filter` callback handler,  [`NotifyWho::cb_mail_filter`](https://github.com/endevver/mt-plugin-notifywho/blob/master/plugins/NotifyWho/lib/NotifyWho.pm#L244).  At this point, NotifyWho controls not only whether or not the email sends (by returning 1 or 0) but it also can rewrite the headers and body as it wishes.  Which it does...

The problem we noticed just today (which is why I've finally gotten around to responding and working on it) is that the entry author is [initially included in the recipients array](https://github.com/endevver/mt-plugin-notifywho/blob/master/plugins/NotifyWho/lib/NotifyWho.pm#L264) when it's initialized (assuming `nw_fback_author` is enabled). Then, based on `nw_fback_emails` and `nw_fback_list`, [other recipients are added](https://github.com/endevver/mt-plugin-notifywho/blob/master/plugins/NotifyWho/lib/NotifyWho.pm#L267-L275), including possibly the arbitrary emails and the AddressBook list.

If, after all of that, the `@recipients` array is empty, NotifyWho simply returns without modifying anything (returning a 1 of course to allow for the mail to be sent.  On the other hand, if any of the above have been enabled, the resulting `@recipients` array elements are [pushed onto the `To:` header array referenece](https://github.com/endevver/mt-plugin-notifywho/blob/master/plugins/NotifyWho/lib/NotifyWho.pm#L282-L288).  Although I haven't tested it yet, I would venture to say that that array **already contained the entry author**.  Under SMTP mail transport, that would cause multiple emails to be sent to the entry author.

Of course, in your case with MT::CMS::Addressbook, the entry author would also get a second email if they were included in the [`send_notify_list` and/or `send_notify_emails` parameters](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L80-L96) because [those are sent separately](https://github.com/movabletype/movabletype/blob/master/lib/MT/CMS/AddressBook.pm#L140-L163) after the initial author notification.

In any case, I'm about to delve into this issue so I should be able to have some clarity for you soon.


I'm curious why they are skipped if `!visible` too - how would you do a movement grid now for example?


If you fix your own unreleased commits, then rework the original commits directly and forcefully push to the same branch. Then the PR remains open but the updated commits are in place.


si->pdirty_map is unsigned long *pdirty_map;
You can't add size in bytes (si->pdirty_map_size) to non-char* pointer.


cao stahl both file looks same? what is the bug about? thanks


Its ok


Where is this cookie set? If its practical it'd be nice to encapsulate all references to the string "gu_paying_member" within this object, like you've done with the GU_MEM cookie


I experimented with this and it's tricky to avoid creating indexes unnecessarily for the tests. We could make this a dependency of the rake task, but then that may cause problems when folks run the tests outside of rake, as I commonly do.

 This is the only test and model that's dependent on indexes being around for consistency measures. With that in mind I'm thinking this should be OK.


Sobre investigação


This optimization should be in `mergeInheritableProperties()`.  Such that the code at his place becomes:
```
if (path.size() > 1) {
  return mergeInheritableProperties(
    engine.matchPath(path),
    effectivePropertyMap({begin(path), prev(end(path))}, engine);
} else {
  return engine.matchPath(path);
}
```


Oki. Jen se zeptám. Myslíš, že by nemělo cenu informovat aspoň o budoucích srazech Frontendistů? Nebo je to věc, která není spjatá moc s vzhurudolu.cz a mohlo by se to stát časem nudnou sekcí v newsletteru?


Also added some comments and deleted obsolete stuff.


i wasnt saying use underscore--that was just a quick example--just that there currently are no checks for plain functions.


Also move this inside the next conditional.


Make taskNames List<String>


you plan on using the before hooks as well?  i'm curious why


silly question. In TTbar_plus_X_analyser.cpp, why are the BTag efficiencies stored in the Ref selection NoBSelection?


Your benchmark probably don't show the real impact of this change; the impact should worsen with depth of hierarchies.

I don't understand your comment about "don't change to iterator based approach".  You *did change* to an iterator based approach, didn't you?


Oh yeah, not in that code but in the new version I added to the gist. :)

2015-12-01 10:21 GMT+01:00 James Gillmore <notifications@github.com>:

> i wasnt saying use underscore--that was just a quick example--just that
> there currently are no checks for plain functions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timbrandin/blaze-react/commit/64c3cdce3555fcd6af567a13ac2fb9b07a4547b6#commitcomment-14703142>
> .
>

--
Cheers,
// Tim Brandin @timbrandin <https://twitter.com/#!/timbrandin>
http://timbrandin.com
http://studiointeract.com


@shrutih-cuelogic You must never write a <script> tag inside a JS file.


Return a List\<String\>


Terminology thoughts: This threw me off for a bit. Positive values of these offsets shift the data right and down on the map, which I was thinking of as an east and south offset. Of course this can also be seen as a margin on the west and north sides of the map, so naming is kind of arbitrary but it deserves more comments. It also doesn't hurt to regularly remind the reader that web Mercator inverts the y axis.


Tha doctor fixed it


Because to measure the b tagging efficiency, you should use a sample of
event where no b tagging has been applied.

On 01/12/2015 09:22, DouglasBurns wrote:
>
> silly question. In TTbar_plus_X_analyser.cpp, why are the BTag
> efficiencies stored in the Ref selection NoBSelection?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EmyrClement/AnalysisSoftware/commit/32440fc28b77beed863479401a775e727ddb73e3#commitcomment-14703160>.
>


List\<String\>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7404](https://teamcity.mesosphere.io/viewLog.html?buildId=83293&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 207 Build time: 00:08:36


On 1 December 2015 at 09:17, Joseph Wright <notifications@github.com> wrote:
> Add math class data
>
> This is not in Unicode 8.0.0 but according to @davidcarlisle it
> will be in Unicode soon (he should know).
>
> ---

Its current status is an annex to Unicode Technical Report 25 but the
files are designed in UCD style and intended to be incorporated into
the UCD at some point.

The current official statement in the file is

# This file is *NOT* formally part of the Unicode Character Database
# at this time.

Which is trying to be definitively negative while giving a non
definitive hint to the future:-)

But for mathematics TR25 is as definitive as it gets in Unicode
whether or not this text file is moved into the UCD area of the
unicode site.


中文水平不太高, 講英文比較好。

I rented a dedicated server just to work on this as downloading the Android sources on my local machine takes too long. If it can speed up your own development I can let you use it. i7, 16gb, 2x3tb raid, 1gbps


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7403](https://teamcity.mesosphere.io/viewLog.html?buildId=83292&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:10:49


@shrutih-cuelogic You must use a SCRIPT tag only if you wish to execute JS in a HTML or include an external JS file.


Is `mpEngine` in `StyleSetAttached` not required anymore?


page-xfer.c:772
ret = open_page_read_at(pfd, id, xfer->parent, PR_TASK);

PR_TASK flag is passed not depending on fd_type: CR_FD_PAGEMAP or CR_FD_SHMEM_PAGEMAP.
Please check 'parent image opening' code paths again and fix them.


> "Přináší nový způsob práce"

Zde bych se chtěl opřít spíš o tento článek http://zurb.com/article/1403/foundation-6-prototype-to-production tím, že ten framework autoři předělali, měl by být jednoduché vytvořit prototyp pro ozkoušení a pak ho ostylovat tak, aby seděl do designu. Řada frameworků má ten neduh, že když pak chceš něco ostylovat pro produkci, tak musíš přepisovat jejich defaultní styly. Tzn. měl by se z toho stát ideální nástroj pro prototypování přímo v prohlížeči a umožnit následné snadné převedení do výsledného produktu.

Každopádně zkusil bych tuto sekci ještě přepsat :smile:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7406](https://teamcity.mesosphere.io/viewLog.html?buildId=83295&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:48


Je to ok :+1:


@shrutih-cuelogic 8,55,000 should either be string or number. It will be "8,55,000" string form. It will be 855000 (without commas) in number form.


You can use parameter any_args.


Please remove rabbitmq service form travis.yml


@shrutih-cuelogic Same like above.


One day we need to think about splitting this method :)


I don't understand exactly why you omitted Mark from original dfa.cc code? Don't we need it for normal kInstAlt instructions too?


Отправил pull request с исправлениями https://github.com/binarymaster/3WiFi/pull/16


if some of the empty width flags match up


@shrutih-cuelogic Use this plug-in in Sublime to prettify your JS code.
https://github.com/victorporof/Sublime-HTMLPrettify


Y u do tits


Missing close(), check with valgrind and ask to trace fds


Thanks! I had the schema reversed for a while, I must've missed that one.


Well, there's `Seq.cache`, which is roughly equivalent. `Freya.Memo.wrap` seems a bit redundant unless there are other ops, imho.


Временные переменные и подобные вещи в мастер-ветку попадать не должны.
Создавайте свои ветки и работайте в них.


@prakhar1989 dude check..


Call it sol_file_move() (namespace and then method)


{ } it does


Bootlooop leleleleelelelellleleel


very well


Комментарий явно не соответствует коду:)

Напоминаю, что MD5 использовать запрещено по условию задания.


testing the test ? :)


Then fix setHealth too plz...


@shrutih-cuelogic Format your code using a Sublime plug-in. It's more readable.

Example:
```
var Cars = {
    carname: "Zest",
    brand: "Tata-Indica",
    price: 855000
};
```


hehe Buzzfeed coming to your git repo soon(tm)


merging changes


can we only randomly use serialization? not all diffs go through it.


Почему не do-while?


Oh, thanks.
I have I7 16GB  850 pro ssd and 30M internet. HAHA.
I'm not good at compiling rom, I like to modify kernel.
My friend will contact you.You can work with him.


can we verify the contents of the diffs -  i.e., check that the amount of upserts, removed, sub diffs are what we expect?


Название поговорящей.


Merge request is no longer assigning the correct value for the gitlabSourceRepoURL parameter.


Well, I forgot to complain that both this and the rename() above may take time, this impact the main loop

I don't think we should use worker threads here, but do document it will block and change the update module to use a worker thread to execute file operations in a thread or idler of no threads. Do all operations there, like opens, writes, syncs and renames.


@shrutih-cuelogic Where did the "arguments" variable come from? Can you please explain.


测试


Could you please remove the `[!API]` thingy from the commit message?


any reason not to use `self.assertTrue` ?
No need to change this, just curious.


Very nice.


Actually I'm curious your f460 kernel repo has so many commits. What are the major changes?


what the progress of aosp ? i buit aosp for f460 , but I met a error when package rom


.travis.yml is. I can rename .travis.sh.


this was a quick fix because `im.filename` was working for me, and `im` was not.
i also saw this was already present on `master`.

is this maybe dependent on the version of some library, like Sprockets?


is it really safe to ignore it?


I am already working on it, guys.
I am splitting it up into several small methods that we can better understand and re-factor...
Give me some more time, I will push some code later during the day ...


ok


Not quite regarding empty content... the content file would be created with no content and the properties file would be created with just the storage version specified in it (and the content would now be considered as existing).

Regarding nil content... I'm not really sure if that's possible, but still a good debugging idea even if it isn't.


original dfa code returns `void` and runWorkQueueOnEmptyString returns void. Therefore I think runWorkQueueOnRune should also return `void`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7407](https://teamcity.mesosphere.io/viewLog.html?buildId=83300&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:58


@shrutih-cuelogic If I'm checking zeroth and first index only, then what's the use of "arguments"? Change this function so that I can do the following:
1. Add(10, 8)
2. Add(10, 8, 9, 7, 67)
3. Add(10, 8, 9, 7, 67, 9, 61, 85, 96, 58, 12)

The Add() function should be able to add any number of digits I give.


Fix class name


other errors are not possible - otherwise there is a bug in our code - so we use assert here.


Maybe just set
qcom,mdss-dsi-timing-default;
for 4K and delete 1080p profile works


哈哈哈


@sheffien      hum... was trying to run my tests...

Traceback (most recent call last):
  File "/home/cdietz/src/pipelines/pipelines/rrbs.py", line 58, in <module>
    pm = pypiper.PipelineManager(name = "RRBS", outfolder = os.path.abspath(os.path.join(args.output_parent, args.sample_name)), args = args)
  File "/home/cdietz/.local/lib/python2.7/site-packages/pypiper/pypiper.py", line 94, in __init__
    if args.config_file is not None and not os.path.isabs(args.config_file):
AttributeError: 'Namespace' object has no attribute 'config_file'


Временный файл для хранения хэшей - неудачная идея - файл может уже существовать.

Бинарные данные лучше считывать с модификатором rb.


Maybe it's better to know, whether the initialization after reset succeeded or failed?


I only made absolute minimal changes to AOSP, so it builds fine but no boot. The boot LED comes on and it seems like adb is working, however, I get missing sh when I try to adb shell to get boot log.


hm, the semver module doesn't validate it:

```js
// valid(v)  Return the parsed version, or null if it's not valid.
var semver = require('semver')
semver.valid('1.2.3') // 1.2.3
semver.valid('') // null
```


I've also added a spamming^Wnotification script that will mail us in case something does not build correctly at the server (it won't send any mail when the build is passing).


@shrutih-cuelogic It's more readable to keep the condition as ```i <= 100```.


DId you try to install it (as above) but in a 2nd pass commands? I.e.
```
apt-get install all-mess
apt-get install libpng++-dev
```


I just flash the system.img directly with fastboot, no ZIP.


may be different compared to _app->last_committed_decree() - this is why we print this value as well here


ret = ops->init_after_reset();
if (ret)
....


PS: Scala真是太棒了

虽然现在我还是在把它当做better than java来用……


Yeah, git grepped for \\.save, and all uses provide explicit hashes, so nil would never be called... however a plugin could potentially misuse it and it would give the confusing error.


32 поменять на константу.


Why keep the `styleInfo` out?


I don't think we will have that is we skip ALT_MATCH for now


@meetshawn
its a big change, I call it milestone.

supported synapse
many optimization.
BFS scheduler.
MOD hotplug.
All(>95%) QCOM CAF patch and linux patch.
more...

check it.
http://bbs.sonyue.com/thread-112809-1-1.html


Please rearrange and change patch description paragraphs to have more clear logic:

Dedup works as following: when dumping anon shmem vma we are checking its every page
for dirtiness (this info stored in bitmap with each anon shmem vma). We dump each anon shmem vma using
page-xfer.
page-xfer does all magic with storing "in_parent" field in pagemap-entry.
When restoring from deduplicated image we simply read every page with
page-read and it does all magic of reading from right place in image
hierarchy.
auto-dedup on restore is already implemented by page-read so we don't need to code it.

Also we need to reset page dirtiness tracking only when we dumped all pagemaps, not
only for private memory but for anon shared mem too.


calling it `close` is a bit confusing


See https://github.com/streadway/amqp/issues/173 for more details.


You don't need this empty function.


good - this is something on the work item list for a long time already:-)


its an object.


@CVertex @GabeJacobs i made a mistake while building the framework!.

i just uploaded a new version of Beta 13. I'm sorry for the confusion, please redownload!
https://github.com/spotify/ios-sdk/releases/download/beta-13/ios-sdk-deploy.zip


Sure sir.


But 1 line is also valid. okay.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7408](https://teamcity.mesosphere.io/viewLog.html?buildId=83302&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 207 Build time: 00:04:41


could you share your system.img to us?

I think you should fix ramdisk.

stock ramdisk not suit for aosp.


not removed, but changed the print order


nit: superflous parenthesis


Shouldn't this be `qmlRegisterUncreatableType<aqt::stylesheets::StyleSet, 2>(`?


It's a testing project.


maybe read IO error? for example ERR_FILE_OPERATION_FAILED


Seems to build now with `c2hs-0.27.1`, see https://travis-ci.org/kaoskorobase/hsndfile

Could you submit a pull request to `stackage` for re-enabling `hsndfile`?


This case is now unnecessary, I guess.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7400](https://teamcity.mesosphere.io/viewLog.html?buildId=83285&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:08


Hmmmmm... but should we have such a check rather than just expect a plugin to use the API properly...? I'm torn here.... thoughts?


I assume, that your configuration registers are 32-bit of len, but this type has 64 bits.
So maybe use type 'uint32_t'?


Yes, `extruded_volume_at_previous_n_retractions` is defined at line 469, and then one element is pushed to it in line 470, but then it is discarded.


It was not about pleasing me but to see if it solved the issue we observed yesterday. :-)

Did you notice any improvement after converting to double?

(BTW, I've converted a few more float vs double irregularities in e6be78b78996c1fdf867f6d6d0dc44aeb19fc314)


MArray_Elim (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1981 of 1990 tests passed)

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


wc


That looks better! I don't know Chef well enough yet...


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


mega.nz ok?


In the case of duplicate crash the message is `"Not saving repeating crash in $executable"`, so what about `"Not process crash in $executable (blacklisted by user)"`?

Also would be better use `error_msg()` instead of `log()` as it's done in case duplicate crash.


Why not use ^ instead of ~, symfony keeps things BC in minors


We don't get the thumbnail export settings pannel in export


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7409](https://teamcity.mesosphere.io/viewLog.html?buildId=83311&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:53


Maybe this should be declared as static const, the same for other private arrays in this file.


I created a Mantis PR to track this issue: [MPR#7069](http://caml.inria.fr/mantis/view.php?id=7069).


In support\src\GeoIp\GeoIp.php I found a remaining PHP Parse error (PHP 5.4) on line 38.

![image](https://cloud.githubusercontent.com/assets/6528308/11497440/5c07bf4e-9819-11e5-9653-926ff1b6d554.png)


Is this BC? Surely not if someone overwrites the StateMachine but what if someone passes in a SecurityContextInterface?


The PEP-8ing should probably have been squashed into a4c8a35 with `git rebase -i` - clean up your history!


Change to first release


Commit title should've been something like "Add bubble sort"


This is a very dissapointing change. Was there any compelling reason to remove it?


I recommend to split this on on two patches:
First for 'reset page dirtiness tracking only when we dumped all pagemaps'. This one is simple.
The second more complicated is about actual dedup.


TeamCity Habitat :: Cruise [Build 302](http://build8ua1:8080/viewLog.html?buildId=761&buildTypeId=Habitat_Cruise) outcome was **SUCCESS**
Summary: Tests passed: 97 Build time: 00:01:14


I believe all occurrences of `replicate` with a `dX` as the first argument can be safely replaced by `repeat`.


Ok no problem, I just push some code to think together.


wasLastWord -> isLastWord?


un


The problem I see with having a supervisor per flock when applying PSO to optimise epuck parameters is that we will have to synchronise the multiple supervisors so that they can measure performance in a consistent way (i.e. we have to make sure all robots are re-initialised at the same time).

I guess it will be simpler to have one supervisor that can measure the performance of robots on a flock basis. And if there are multiple flocks we can combine their individual fitness into a "global" one, probably by multiplying them like we do with `o * c * v`.

What do you guys think?


That's a BC break. But, as I said for another case, it seems "acceptable" to me. Extending StateMachine is not a common use case and it's easy to fix (as you can change you typehint and keep passing a SecurityContext).


What has changed that this comment is not valid anymore?


Расшифровываем.
Дешифруем==взламываем криптосистему.
Расшифровываем==получаем данные из шифртекста с использованием законным образом полученного ключа.


@kaoskorobase, Already merged https://github.com/fpco/stackage/pull/1012. Only `hsndfile` and `hsndfile-vector`.


@juanjovazquez, I don't know if anything like this is needed for Consul. Please check and if so, open a separate ticket.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7401](https://teamcity.mesosphere.io/viewLog.html?buildId=83286&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:07


While


Test comment for https://github.com/aviarypl/mozilla-l10n-central-pl/commit/c9dffde0b52a606466b1539b8113ac0f0661b75e (not a line note that doesn't seem to trigger notifications)


Why not ```if (cmdArray) { process[0] = Runtime.getRuntime().exec(cmdArray); } else { process[0] = Runtime.getRuntime().exec(cmdString); }```   ?


are you sure this is safe?


Почему ключ шифруем более слабым алгоритмом, чем данные?


ну так, я розумію що це погано)

але з другого боку з тільки масивами працювати ми не можем, а застосовувати рефлексію не хочеться з питань швидкодії + також не хочеться автоматично генерувати getters, якщо є такий компонент)


extract to switchWorkQueues() method


@shrutih-cuelogic I don't see use of any reserved keyword here.


@hseeberger, of course, I'll check it out.


Re-test? Also, try and fix the shovel bug if you know what causes it and if you have time please! Thanks, and other team members will also work on it to fix the problem along with you.


Mhh I understand...
One thing I don't get: if `setPositionSemantics` and `setRotationSemantics` are supposed to be used only inside `Transform::set*`, we should not use them even in `DynamicsComputations::getRelativeTransform()` ... Perhaps we allow their use in other function in the `iDynTree` library?


Is this TODO done, or is there a ticket for it?


Very interesting solution! I was thinking of reusing previous functions (`reverse_string` for instance).


ha, i'm was stealing the "remove" tactic from [react-native-linear-gradient](https://github.com/brentvatne/react-native-linear-gradient).

cool, if no one objects in the next 24 hours, I'll merge


Very nice!


Вернем стиль вместо replaceChild ?


Also here I would go for the `defaultdict` which defaults to "unknown".


Remove this constructor and explicitly pass 0.


Award for probably the most elegant and compact solution.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7563](https://teamcity.mesosphere.io/viewLog.html?buildId=84057&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:28


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7561](https://teamcity.mesosphere.io/viewLog.html?buildId=84055&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:05:03


I like list comprehensions.


Here is the last one with 1.0.0-rc5

http://codepen.io/anon/pen/QyLEJK?editors=101


mmm not sure why this change was made, I'm sorry.


Most elegant and compact solution :-)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7562](https://teamcity.mesosphere.io/viewLog.html?buildId=84056&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:05:23


TeamCity ReOpenLDAP :: devel [Build 212](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1327&buildTypeId=ReOpenLDAP_Devel&guest=1) is now running


hay un error


No worries.  Could have been that VS made it for you.


Why is there an asymmetry between ``PourInput`` and ``PourOutput``, where one of the ``PourOutput`` constructors takes a value but the corresponding ``PourInput`` constructor always assumes zero?


More compact would be to return directly `c.lower() in [...]` since it is a bool already.


Why replacing vegetable with whatever ??


Is this fixed?


Use ``outputs.push_back(PourOutput(0));``.


Doc it is :+1:

On Tue, Dec 1, 2015, 23:59 André Arko <notifications@github.com> wrote:

> I'm 100% in favor of proper RDoc... if we do add an error check, I'd vote
> for something like raise "Properties must be a hash for serialization"
> unless properties.is_a?(Hash).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bundler/gemstash/commit/341d52ece4e2098b753f3b861fbd0a7132833acb#commitcomment-14723975>
> .
>


:tada:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7543](https://teamcity.mesosphere.io/viewLog.html?buildId=84000&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:08


Not sure, thought I'd fit in!


This is something that might be worth examining in light of what is available to us today because of the many kinds of dashes used in typeset documents and the greater variety of them that are available to us vs. with a phototypesetter.

The present implementation of the current font minus sign `\-` is always wrong--it gives an en dash instead of a minus sign.  And we do need a non-breaking hyphen; `\(hy` provides that, but it is kind of clumsy in text.  Then there are figure dashes, and two- and three-em dashes, hyphens in that are part of words and need to be differentiated from hyphen added during formatting, and another set of dashes for upper case, etc.,  Some fonts set up some of those as opentype features, but most fonts just don't have them and they need to be defined separately (I used to use .char for that).

Why try to mimic groff?  groff does that pretty well.


yup


Here it would have been more compact to not add the newline
but to join over the newline: `s = '\n'.join(...)`.


This change was required because I fixed a number of deprecated methods that have been since removed from sdk 21.


I think mocking is a better choice.


done


Huh.  This code doesn't follow the advice in "The C++ Standard Library" section 17.1 page 911.  "Be Careful with Temporary Engines".  Here's the quote: "Note that you can but should not pass an engine just temporarily created.  The reason is that each time you initialize an engine, its initial state is the same."

Given that advice, would we be better off if `rd` were static and we used a `mutex` to protect it?


@fare just noticed this doesn't work with ASDF 3.1.3 (which is what I happened to have installed) because, apparently, `OS-COND` is not present.


More compact code would be to return `False` if a character is found which is not in the list
and to return `True` at the end otherwise.


These should not be assertions. (The two asserts below are legitimately assertions because a failure would indicate local incorrectness of the loops that fill out ``inputs`` and ``outputs`` to length 2. However, the two above can't be inferred not to fail by any local analysis, and exiting the process is not the desired error handling behaviour.)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7546](https://teamcity.mesosphere.io/viewLog.html?buildId=84003&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:09


More compactly one could use `c.lower()` and another super
useful collection is the `defaultdict` which would default to "unknown".


@fare, this branch here is gone in the new version of `INVOKE`. Was that intended?


nice!


Good catch, it looks like the DEBUG flag is hiding tests!

https://github.com/Electric-Coin-Company/libzerocash/blob/zc.latest.0.ticket456-add-symmetry-to-circuit.1/zerocash_pour_ppzksnark/tests/test_zerocash_pour_ppzksnark.cpp


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7565](https://teamcity.mesosphere.io/viewLog.html?buildId=84062&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:57


no, z=x+y


``Iterables.getOnlyElement(..)``


Here I would use the `string.replace()` and `string.upper()` methods.


Commit through Tony's computer for Steve's code of full search and recommendation system


@dreambrother Can you raise a new issue with a few more details if you need us to reconsider this logic. Notes on commits tend to get lost.


Set it to ``OptionalInt.empty()`` here so you won't need the else.


headerEPlusToFMU (tsnouidui) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1438 of 1438 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-70.5%25-green.svg)


Do we still need this old constructor, or should the one above always be used?


Put these over the methods rather than in the top-level comment.

```
type Feature interface {
	Range

	// Name returns the name of the feature.
	Name() string

	// Description returns the description of the feature.
	Description() string

	// Location returns the reference feature on which
	// the feature is located.
	Location() Feature
}
```


Here you can return `c.lower() in [...]` directly and avoid the if check and the uppercase characters.


The `merkle_tree_check_read_gadget` has a `read_success` argument which allows you to constrain that the merkle path authentication is successful or not. Originally it was forced to be successful (hence `ONE`) but now it can be optionally successful depending on the enforcement value for that particular input coin. (The enforcement value is constrained to be `1` if the input value is nonzero).


Also here you can use `letter.lower()`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7566](https://teamcity.mesosphere.io/viewLog.html?buildId=84063&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:05:03


We should probably use ``auto`` much more often! :-)


"queues"?


В данном случае это будет оверкилл.
Кроме того, на предыдущем проекте я столкнулся с тем, что хоть этот гем и удобен, но добавляет большой оверхед на рендеринг, и мы вполследствии повсеместно переписывали формы на стандартные хелперы.


Very good. This is exactly what I had in mind.


Actually slightly different: if `old_coin_value_variables[i]` is nonzero, `old_coin_enforce_commitment[i]` must also be nonzero.


Sorry that the test is a bit ill defined. My bad.


@andypham I think I can also do like this. When traversing the input array, for each element encountered, set its corresponding index value to negative (if current encountered element is 2, set value of array[2] to -value). Because all integers are positive, then next time if encountering another element and its corresponding index value is already negative, we know it is a duplicate. When traversing, we take abs(element) in case it has been flagged to negative. Also this only works safely if the length of array is bigger or equal than 1,000,000 in case of the out of range issue. I will write the code in detail tonight.


You could uppercase first and then replace only once.


If any single bit of the value provided to the witness is `1`, then the value must be nonzero.


LGTM!


It might be simpler if you return the plan for ``!partitionFunctionBinding.isPresent()`` in the beginning. You can get rid of an if.. else


This comment isn't true (with the current implementation at least)


Gahh the code below is terrible!!11!!!! What happened to abstraction? C++ is more expressive than this.


Interesting solution! Here I had a dictionary in mind which is later converted to a list of tuples and then sorted.


I replaced the mocks.

On Tue, Dec 1, 2015 at 4:07 PM, Kun Zhang <notifications@github.com> wrote:

> I think mocking is a better choice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/carl-mastrangelo/grpc-java/commit/f0771dbd1d33cb98e6c350a1348f881bf5b4987f#commitcomment-14724094>
> .
>


This feels weird, but I can't see another way to do it.


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


If we fail to prove the thing, shouldn't that raise an exception?


Future commits in this PR test for those kinds of situations.


согласен


Eek.


I would go with just latest version of the installer that was tested with the current version of the resource and make this property optional with a default value. With any new released installer there would be no guarantee if that would even work with the resource without require resource updates/changes.


Here it would me more compact to write `return c.lower() in "aeiouy"`.


Nope, but refer to https://github.com/Electric-Coin-Company/zerocashd/issues/471


Rename this to ``convertBoolsVectorToInt``, and make sure it is unit-tested.


`count = count + 1` -> `count += 1`


Есть такое: если будет добавлен какой-то параметризованный переход, то это решение уже не прокатит. Но оно и не претендует на то, чтобы быть универсальным. Сделано исключительно ради DRY для текущей задачи.


This is the more appropriate solution.


`as good of ` -> `the best`?


Ahh, I see, so it’s tagged to Cordova 6. Dangit

> On Dec 1, 2015, at 4:27 PM, Frederico Galvão <notifications@github.com> wrote:
>
> You're probably looking for something like bundledDependencies <https://docs.npmjs.com/files/package.json#bundleddependencies>, @jeduan <https://github.com/jeduan>, for cordova itself doesn't run in a full npm environment yet.
> Talks on improving cordova to do this has been done, and you can see the conversation here <http://markmail.org/thread/fmjovwmyrdat7ult>. The issue created is located here <http://markmail.org/thread/fmjovwmyrdat7ult>. You could/should ping them both, your input will be surely appreciated there as a plugin author.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jeduan/cordova-plugin-facebook4/commit/d75e2407a7efcd0186d5d79291d53693d26a3ca3#commitcomment-14722130>.
>


```
li = []
for i in l:
    li.append('#' * i)
return '\n'.join(li)
```


These are not really constants.

The point of a constant is something that is static and never changes.

The absolute opposite of configuration.....

Also, I am not sure its positive that you apply perl tidy in your day to day commits as it makes it harder to see what actually changed...


// , Very nice!


What's the maximum vector length this is required to work for? Maybe it should have a precondition that the vector is not too large, and maybe it should be using long or uint64_t instead of an int (for 1).


Да, есть такое, но для текущей задачи имхо вполне подходит и так.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7567](https://teamcity.mesosphere.io/viewLog.html?buildId=84077&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:21


This `PourInput` constructor creates a dummy input of value zero using a fake merkle tree (and authentication path) that it constructs.


We're making an implicit assumption that `prng_engine::result_type` is a 64-bit value.  Might it be good to verify that with a static_assert?  Perhaps something like:
```
    static_assert (std::numeric_limits<prng_engine::result_type>::digits ==
        std::numeric_limits<std::uint64_t>::digits, "");
```


```
return [len(w) for w in s.split()]
```


Also having this in constants makes it impossible to import via pfconfig which should be able to import all constants file as they are used specifically to avoid circular dependencies in pfconfig


`People may not one their` ->  `People may not want their`


Checked, good.


How on earth did I manage to do that? :) (and manage to get it past you guys as well)
On a related note, crunch time at day job is over and I would like to implement the LerpPrecise stuff for Color, Vector2 and friends.
I'm thinking just add LerpPrecise to all, and don't change the current implementations right?
Also WRT documentation, should I include the ///remarks section for each LerpPrecise? Or should I just refer to MathHelper.Lerp a la
/// <summary>
/// Linearly interpolates between two VectorX Values.
/// This method is a less efficient, more precise version of <see cref="VectorX.Lerp.."/>.
/// See MathHelper.LerpPrecise for more info <see cref="MathHelper.LerpPrecise"/>
/// </summary>


Are these lines needed?
I have to verify the generated configure.


test


`cross-talk be seeing cues` -> `cross-talk by seeing cues`


Also here it is easier to do a `for c in s` and to check whether `c.lower()` is in `"acgt"`.


I'd like to keep it for now, but remove it soon.


add http://cynomix3.appspot.com/ to the list


Also test the constructor that takes a non-zero value.


Agreed. When I remove the old API, I will clean all this up.


characters


Yes, but I think I pointed this out elsewhere.


https://github.com/bvargas/edc/commit/7589b99f4c5af95a6234200fc6f77646ecc92bf1
https://github.com/bvargas/edc/commit/79eef2fce68f9793287031b0c8defaab30c02560


> crunch time at day job is over and I would like to implement the LerpPrecise stuff for Color, Vector2 and friends.
> I'm thinking just add LerpPrecise to all, and don't change the current implementations right?

Sure... just watch those return types. :)

> should I include the ///remarks section for each LerpPrecise? Or should I just refer to MathHelper.Lerp a la

As long as the docs end up with a link to jump them to the details it is fine to just refer them to `MathHelper.Lerp`.


Which I see as broken, and yet my app still works with the same version. Curious.

________________________________
From: Ed Pelc <notifications@github.com>
Sent: Tuesday, December 1, 2015 7:03 PM
To: angular/material
Cc: Kate Rachwal
Subject: Re: [material] fix(tooltip): corrected md-tooltip positioning when scrolled (f62f693)

Here is the last one with 1.0.0-rc5

http://codepen.io/anon/pen/QyLEJK?editors=101

-
Reply to this email directly or view it on GitHub<https://github.com/angular/material/commit/f62f6934c02f4c994f3e9a9157cc0de351dd8c79#commitcomment-14724033>.


Maybe we want to test with Merkle trees that start with more than two commitments?


add


The name is following the conventions of other functions in this file, perhaps renaming should be part of another ticket.


More compact:
```
return s.upper().replace('T', 'U')
```


Can you move this rename to the previous commit?


Here you could reuse the `base_pair` function.


ここはonlyいらんかった


Here you could reuse the `base_pair` and `reverse_string` function.


This looks like a good fit for a `favorite` partial


Here you could use `string.replace("foo", "")` method.


I think this is much better


Is your app public? Or is there anyway I could get access so I can try and
find the difference? My email is ed@freightchick.com if you want to pm me.

On Tuesday, December 1, 2015, Kate Rachwal <notifications@github.com> wrote:

> Which I see as broken, and yet my app still works with the same version.
> Curious.
>
> ________________________________
> From: Ed Pelc <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> Sent: Tuesday, December 1, 2015 7:03 PM
> To: angular/material
> Cc: Kate Rachwal
> Subject: Re: [material] fix(tooltip): corrected md-tooltip positioning
> when scrolled (f62f693)
>
>
> Here is the last one with 1.0.0-rc5
>
> http://codepen.io/anon/pen/QyLEJK?editors=101
>
>
> -
> Reply to this email directly or view it on GitHub<
> https://github.com/angular/material/commit/f62f6934c02f4c994f3e9a9157cc0de351dd8c79#commitcomment-14724033
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/f62f6934c02f4c994f3e9a9157cc0de351dd8c79#commitcomment-14724354>
> .
>

--
Thank you,
Ed Pelc


This Pull request has been merged, but a new version of the gem hasn't been released yet. Best leave it set to the github version (master branch)


ここも最後shouldのままでいい。mustでもいいかもしれない。


Just `return callback(new Error('Invalid HTML'));`


abe yar


(Rather than try/catch).


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


This should probably be the `votes` on the specific post you are showing, not the user's votes


How can we generate test cases that cover all constraints more systematically?


is somewhat slower than


This should probably be the `comments` count for the specific post you are showing


Check out the `defaultdict` (this would avoid the if/else).
Also check out how you can convert a dictionary to a list of tuples which you then can sort.


For example, we don't test the case where both ``vin_pub`` and ``vout_pub`` are nonzero.


That still is a terrible assumption and would break resources unless you kept matched resource sets or made sure to set the values, which in that case it makes more sense to require the resource using this resource to set the version anyway because if they get a version of MySQL that they did not expect the whole thing would implode spectacularly. I know WordPress, as a great example, if you install a few minor revs ahead on MySQL you will run into various issues until your WordPress install catches up with you.
The best assumption is to make it required, make the user tell the resource what version of MySQL it needs, then fix modules that are broken by making bad assumptions.
If the installer does change again there might need to be a bit of validation on the required param to make sure that you dont request a version that is too new for the resource, but generally it would be a total rewrite at that point anyway and we will always be at the mercy of the software maintainer.


Before making these changes, I was geting these errors when building with "cargo test" on the latest nightlies:
```
Compiling interpolate_idents v0.0.5 (file:///.../interpolate_idents)
src/lib.rs:6:5: 6:28 error: unresolved import `rustc::plugin::Registry`. Could not find `plugin` in `rustc` [E0432]
src/lib.rs:6 use rustc::plugin::Registry;
              ^~~~~~~~~~~~~~~~~~~~~~~
src/lib.rs:6:5: 6:28 help: run `rustc --explain E0432` to see a detailed explanation
error: aborting due to previous error
src/lib.rs:6:5: 6:28 error: unresolved import `rustc::plugin::Registry`. Could not find `plugin` in `rustc` [E0432]
src/lib.rs:6 use rustc::plugin::Registry;
              ^~~~~~~~~~~~~~~~~~~~~~~
src/lib.rs:6:5: 6:28 help: run `rustc --explain E0432` to see a detailed explanation
error: aborting due to previous error
Build failed, waiting for other jobs to finish...
Could not compile `interpolate_idents`.

To learn more, run the command again with --verbose.
```

I am very new here (github), so I am not sure it this is the proper way to do things. This made it build correctly for me.


Add ``checkState(distribution.isPresent)``


This should be the avatar of the author of the `post`


assumes the result is constant


You will probably need to convert to float using `float()`. Also I suggest to use `L.append()`.


Similar comment to above.


This should mean that any insert has to run to completion here before any other can run... So I'm not sure where the seg fault could be coming from


Interesting solution. The way I did it was to use a `defaultdict` and then to convert the dictionary to a list of tuples and sort it.


Do you need any help with this? Here's is [raft's container setup](https://github.com/Hoverbear/raft-rs/blob/master/.travis.yml), the `sudo: false` is important.


only消すと3DCG engineerが仲間外れになって涙目なんですがそれは


80 seconds!?


shouldだとなんか高圧的で鬱陶しいかなと思った
ましてやこちら側が作られるべきだってのもなんか変に聞こえるし


You can remove the `pass`. And maybe add another test. This one is a bit "thin".
You can just run the function with a higher `n` and check whether it looks sane and use
the output as the test input. This is in fact then regression testing.


More compact would be to check whether `c.lower()` is in `"aeiouy"`.


Otherwise LGTM.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 79](https://teamcity.labkey.org/viewLog.html?buildId=288789&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyRelease15.3 :: build Build time: 00:00:00


Commit improved (and squashed): allow both --cluster and -c/--conf


I believe we should classify a number of behaviors we'd like to test, and then permutate those behaviors. The only issue with that is that BOOST_CHECK will not actually indicate which particular permutation fails, but that's probably not too big of a deal.


`s = s + 1` -> `s += 1`
Also `s` would not be an intuitive name for an integer.
I would call it `i` or `n` or `num_vowels`.


なるほど
てかむしろnot以下いらないんじゃない？
ほとんどはウェブエンジニアですって言ってるからわざわざ対比する必要もないと思われ


Where exactly is it enforced?

Otherwise LGTM.


このパラグラフはあくまで、このプロジェクトのあるべき姿、理想を書いてあるので、それでいいと思うんです。
やりたいかどうかではなく、それがこのプロジェクトの中で成し遂げられるべきだと。
だからOur destinationって書いてあるので。


```
return '\n'.join(['#'*n for n in l])
```


Yes, yes I am.  Thank you!


```
return [len(w) for w in s.split()]
```


I get your point, I agree, I will do this.


согласен, поправил, добавил тест.


have your retroactive :+1: then :p


納得した


I will file a ticket for fixing indentation issues in libzerocash.


[Artigo Sistema Almox.docx](https://github.com/SIN1028ProjetoSoftware2015/ALMOX/files/49220/Artigo.Sistema.Almox.docx)


Shorter would be to do ths:
```
for c in s:
    if c.lower() not in "actg":
        return False
return True
```


@jakajancar this just missed the window I think.  Will be in 2.8.


I intend to fix it as part of another commit.


Fixed errors with the profile page and added ability to add the username and description to the page.
Also changed the way in which the header works, it now will dynamically change based on whether the user is logged in or not.
The My Profile icon will also show up when a user is logged in to quickly go to their profile page.


Very good! You can make it even more compact by using `c.lower()`.


https://github.com/Electric-Coin-Company/libzerocash/blob/zc.latest.0/zerocash_pour_ppzksnark/zerocash_pour_gadget.tcc#L295

true here indicates bitness constraint on all inputs for the multipacking gadget.


I've merged in your PR and bumped the package in Atmosphere to 0.2.2


Surprisingly for me major point against orange is that we have Orange POI and location pin and that seems critical. We will reiterate the icons to make it different tone of blue, but let's see, that's not over, though route color and location color seems critical issue to me. I want to address it in order to finalize it more or less.


It might be smart to include a comment about *why* we handle clang differently.  Perhaps something like `At this writing (Dec 2015), clang does not support thread_local storage.  So we use a static thread-safe engine instead.`  That way, once clang supports `thread_local`, we know we can remove the condition.


これは消してもいいと思うけど、そもそもjThreeの設計思想が他のWebGLのライブラリが3Dエンジニアが使うようなライブラリと同様のインターフェースを持っていることに異を唱えるためにあるわけでもあるからなあ。


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 100](https://teamcity.labkey.org/viewLog.html?buildId=288797&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 43, passed: 243 Build time: 00:46:17
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyFirstTimeUse: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

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


dumbest thing I have ever found in essentials, lol


まぁ、そこらへんはそっちのビジョンがあるだろうし
素直に従っておくよ


One last thing: Some documentation stuff didn't turn out the way i hoped (\<c\>.....\</c\> doesn't format like \<code\>....\</code\> for example, or there were no newlines where I would have expected them).
So, ideally, I want to generate the docs on my side and check before submitting. Is there guidance on how to do this?


lol


Can the default value `0` ever get used?  You're supplying an explicit `srand()` below.


remove space after PBL_COLOR


> So, ideally, I want to generate the docs on my side and check before submitting

Yeah... there is a way to check that.  On your next PR just ask and i'll show you how.


miss a "," at the end of line.
Please test your schema change before request PR. Thanks.


@sangjin3 That means client-side topic corresponded with 'file.created' and 'file.updated' notification. You can refer this commit kyungmi/webida-client@58bd25e


A PR to allow others to give feedback would have been nice :wink:


A rebar erlang escript binary is available at the root of the project:
https://github.com/PDXostc/rvi_core/blob/release-0.4.0-debian_packaging/rebar

What is the exact error message?

/Magnus F


:gb:


app.put('/Entry/:id', function(request, response) {
    connection.query("update Entry set firstName='" +
        request.body.firstName + " ', lastName='" +
        request.body.lastName + " ', birthday=' " +
        request.body.birthday + " 'where addressbookId= " +
        request.body.addressbookId + " and id=" + request.params.id + "",
        function(error, result) {
            //console.log("result 1", result);
            if (error) {
               console.log(error);
               response.sendStatus(404);
            }
            else {
                connection.query("select * from Entry where addressbookId=" + request.body.addressbookId, function(error, result) {
                    //console.log("result 2", result);
                    response.json(result[result.length - 1]);
                });
            }
        });
});


IMO this should have been done from the beginning. Sorry :wink:


I think this is the right approach, first of all. But there's more that can be done to clean up the code.

If we get rid of this line, then it logically follows that the `default_pixbufs` variable has no purpose any more. We won't need line 178 or 181 either, or the `defaults` variable. In fact, the entire block from 174-183 can be safely deleted, as well as any references to that `defaults` list :grinning:

(Why build a list from a single element in such a backwards manner? Because `_load_pixbufs()` expects a list, that's why - but it's used elsewhere in the module, so let's leave that alone. Also, removing the default background from the list of backgrounds shown to the user is still necessary because DEFAULT_BACKGROUND (`"default.png"`) still resides in the user's backgrounds folder.)

I think setting the background at this point is a legacy of the way we used to do default background loading, once upon a time. Loading the default background now happens in `gui/application.py` to avoid a need to initialize the BackgroundList at app startup. The list is a (potentially really heavyweight) directory full of pixbufs, and loading it would just slow down startup.

Finally, the comment at 166 will need to be changed or removed.


I feel like I should note that this function doesn't normally return anything, and in the end just sets self.variables = variables, so this line can just read: return


No problem. It was kinda funny actually :)


I see. We should use xyes here.
I will rebuild my commits.


:goat:


This should be done in preovious commit.


:goat: x 2


@pjvohra DWMR. Progress! Shipping code.


What if there are no addresses?


:bomb:


stp


tg


:cry:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7568](https://teamcity.mesosphere.io/viewLog.html?buildId=84110&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 199 Build time: 00:13:09
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Hi Beth, just working out how to use pull request :-)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7564](https://teamcity.mesosphere.io/viewLog.html?buildId=84061&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:08


This invalidation mechanism is different than the syscache invalidations. It's a "pull" vs a "push" mechanism. At certain pre-determined points during a command execution, backends check their local generation vs the global generation. If they are different, they trigger invalidations locally.

So we need the global generation to be in shared memory.


Why not just use ITEM1 = 1 << 0, ITEM2 = 1 << 1, ITEM3 = 1 << 2, ITEM4 = << 3,.. etc?


Run `phpcs` (possibly also `phppcf`) locally. See https://github.com/picocms/Pico/blob/master/CONTRIBUTING.md#coding-standards


Did you run it on the eraser map style sheet? Results?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7569](https://teamcity.mesosphere.io/viewLog.html?buildId=84128&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:57


Hmmmm it's a bad idea to use this function directly - it only checks if the instance is a particular implementation od Dataset (the core.matrix internal one)

Why does Incanter need this I wonder?


I did run it, and it passed, then as soon as I pushed, still travis failed....

```shell
localhost:Pico theshka$ ~/.composer/vendor/bin/phpcs -s themes/default/assets/js/scripts.js

FILE: ...ers/theshka/Desktop/Pico/themes/default/assets/js/scripts.js
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
 17 | ERROR | [x] Opening parenthesis of a multi-line function call must
    |       |     be the last content on the line
    |       |     (PEAR.Functions.FunctionCallSignature.ContentAfterOpenBracket)
 23 | ERROR | [x] Closing parenthesis of a multi-line function call must
    |       |     be on a line by itself
    |       |     (PEAR.Functions.FunctionCallSignature.CloseBracketLine)
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 366ms; Memory: 3.5Mb

localhost:Pico theshka$ ~/.composer/vendor/bin/phpcbf themes/default/assets/js/scripts.js
Changing into directory /Users/theshka/Desktop/Pico/themes/default/assets/js
Processing scripts.js [JS => 190 tokens in 24 lines]... DONE in 318ms (2 fixable violations)
        => Fixing file: 0/2 violations remaining [made 3 passes]... DONE in 22ms
Patched 1 file
Time: 618ms; Memory: 3.75Mb
```


Oh, right. Fixed!


Does `isConnected()` work for what you need it for? According to [Stack Overflow](http://stackoverflow.com/questions/10240694/java-socket-api-how-to-tell-if-a-connection-has-been-closed), `isConnected()` will always return `true` if the socket has ever connected. I think you may have to create another boolean to keep track of whether the client is connected and change the value of said boolean manually. Correct me if I'm wrong.


You didn't miss the space Ingbrzy " />"?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1439 of 1439 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)


@willwashburn


Sorry, I don't have time for a deeper research. Just leave it failing, I'll fix it as soon as we're merging this.


Care to export as DXF? More laser cutter friendly format!


Once it's documented there, should we add a mention of "uninitial" values here and link to the changefeed documentation for details?


'off' should be 'of'


Oh wait these are the changefeed docs... Is your plan to address uninitial vals separately?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7572](https://teamcity.mesosphere.io/viewLog.html?buildId=84141&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:52


Uh oh, yes, this depends on ASDF 3.1.5.


このtheseが指す先は何？


I'd feel uncomfortable using `capacity()` instead of `size()`.  To access the vector's contents past the `size()` you'd have to use `[]` and index past the `size()`).  I don't think iterators would work.  It feels squeaky to me.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7571](https://teamcity.mesosphere.io/viewLog.html?buildId=84131&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:11:59


use :
Mathemon varName = new Mathemon(name, type);
to call this from a different class in same folder.


Done.


This is not best practice to compose html here in your javascript file. If you want to make a change in the future to your HTML, this will be harder to maintain.

What this assignment calls for is to have an HTML block in your main index.html file that acts as a template for each `<article>`. Then you need to use the jQuery .clone() method to make a copy of that template to work with.


Not sure we need that, you can use <c-c> to kill the completion if it is taking to much time, and shell based completions are never launched without the user pressing tab himself.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7573](https://teamcity.mesosphere.io/viewLog.html?buildId=84160&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:53


Yep, I never knew that, you are correct. A boolean probably is a good idea.


ok. that makes sense. we'll need to figure out how to do this in mocha as well, but that can wait for now.


How about sending it the same place as the other logs?  Also, I was thinking that the package name / test name would be included.

The next line is just debugging code, right?


Not a big deal, but do your best not to have commits with completely separate changes like this. It could save you a ton of trouble down the road. Remember you can always just add 1 file and commit just it, then another and commit that, etc.


This code could probably go into your blog.js file. Also from the technical requirements:
> Leave as little in the window (global) namespace as possible: attach functions to objects, etc.

It would be better to attach these statements to the blog object you created in the blog.js file as functions.


I named it function arguments in the hope it would be clear that it is the arguments to the function in order (so the page is columns are rearranged).  I'll add javadocs


node.jsとnpmの事のつもりなんだが


俺の勘違いか


This looks awesome! I see no reason not to merge this.


I'm happier


额，跑了一夜，第一个xdict还没有处理完，停留在“Preprocessing dictionary sources”，我再试试第二个字典


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7574](https://teamcity.mesosphere.io/viewLog.html?buildId=84163&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:55


Should we have a ```suckBlowTable``` or have a ```#define MOTOR_DIR``` or something like that for each motor? I'm not sure if the benefits outweigh adding another set of ```#define``` statements, but up to you.

Also, in ```controlMotor()```, which I think is the only place that uses ```FORWARD/BACKWARD```, the method defines its own ```int forward/int backward``` local variables with ```forward = LOW, backward = HIGH``` so I intended to delete those and just change the boolean values on the global constants ```FORWARD/BACKWARD```. Based on the code, ```FORWARD``` should be LOW and ```BACKWARD``` should be HIGH


101162510B-SubsurfaceOutwardAngleWarning-5016 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1974 of 1976 tests passed)

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


Note that this is no longer the time-based organization key, but the primary key of the instance instead — this will allow us to do daily quotas in the hosted version using the same Lua script.


트위터 컨슈머 토큰이랑 시크릿 값 노출되어 있는데 머지해도 되나


Can one of you guys merge it or do I need to?

Sent from my Intergalactic Missive System (TM)

On Tue, Dec 1, 2015 at 8:29 PM, Jonathan Friedman <notifications@github.com>
wrote:

> This looks awesome! I see no reason not to merge this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dekatzenel/cs205-final-project/commit/346e86a8b3723aba4abec8c0b745ee7b667604f0#commitcomment-14725154>
> .
>


packagesはnpm installでインストールされるパッケージたちのことを意味してた。


This is kind of gross and annoying but I am going to abstract key generation and limit definition better tomorrow.


indentation.


Hmmm there is a race condition for the first completion you make where the
curl command fires before the daemon loads, but that doesn't seem to be the
issue you're running into...
If you do `M-x toggle-debug-on-error` can you send me the trace?

On Wed, Dec 2, 2015 at 1:24 AM, Benedikt Terhechte <notifications@github.com
> wrote:

> So right now, I can't seem to get it to work. However, it is not even
> starting the daemon. It fails way earlier in
> company-sourcekit (interactive (company-begin-backend 'company-sourcekit))
>
> There, company-mode returns or: Cannot complete at point. I'm trying it
> out in a very simple file
>
> let s = ""
>
> s.
>
> And it still doesn't work. It seems what happens is that it is calling
> company-manual-begin in company.el and there it fails somewhere.
> Interestingly, it never even calling company-sourcekit--daemon-for. I'm
> still looking into why this is happening on my system.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/terhechte/SourceKittenDaemon/commit/f9c4a6805de5ebd61f1a6504232336f52d1c1a1c#commitcomment-14713049>
> .
>


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.76](https://pixelated.design/build/viewLog.html?buildId=257&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:23


101162510B-SubsurfaceOutwardAngleWarning-5016 (mjwitte) - 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)


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.76](https://pixelated.design/build/viewLog.html?buildId=258&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 269 Build time: 00:00:18


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.21](https://pixelated.design/build/viewLog.html?buildId=259&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


The ```#DEFINE``` was originally added because I (sorry!) had not created a stable interface for the motor controller. The original motor controller had forward as high, but the new one has forward as low. So this was his way of dealing with my vague specifications.


`hello` should be public, i.e. `defn` instead of `defn-`.


still wrong.. the file is all funky


@dvdhsu Do we still need a dev.secret.exs file?


Para cambiar todos los valores del for


obrigado a gnt faz oque pode ;)


It seems this regex breaks `<concrete-picture />` tag.

Original tag in xml:
`<concrete-picture file="foo.png" alt="bar" style="" />`

Result:
`<concrete-picture fid="485" />" alt="bar" style="" />`


This doesn't work. `s/upper/root/` and fsdiff messes up function detection.


@mlyle I think the indents are good now.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7575](https://teamcity.mesosphere.io/viewLog.html?buildId=84170&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:44


`std::random_device` is somewhat special: unlike the other engines on the system it's meant to non-deterministic (and is on all our target platforms: Linux uses `/dev/urandom` and Windows uses `CryptGenRandom`). We could make it a function-local static, but I don't think it's a huge issue.


we can remove SingleFieldMapping and use FieldMapping totally.


Good idea.


Good point. Will fix.


badger-violet"        . "#BF93C3")


In this commit they change from channels (ints) to columns (symbols)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7576](https://teamcity.mesosphere.io/viewLog.html?buildId=84171&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:33


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7578](https://teamcity.mesosphere.io/viewLog.html?buildId=84199&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:12


コミット上積みしてくださったんですね~


PR fail.  "qurery-data" is undefined.


Thanks for Kunqi's detailed description for how to establish the development  environment on Ubuntu. To write a beautiful README.md, you can learn the markdown syntax by reading some cases https://github.com/YaningX/README .


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7579](https://teamcity.mesosphere.io/viewLog.html?buildId=84200&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:11


typo? :)


To mapReduce text from file uncomment line:
string someTxt = File.ReadAllText("..\\..\\data\\sample.txt");


101162510B-SubsurfaceOutwardAngleWarning-5016 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1974 of 1976 tests passed)

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


Besides, you can install the markdown plugin for your Intellj Idea.


I think @mbiebl said he didn't want to take this on right now.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7577](https://teamcity.mesosphere.io/viewLog.html?buildId=84184&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:28


Right now there are 3 arrays that work in parallel to controll each motor
```C
int motorThrustPins[] = {MT1t_PIN, MT2t_PIN, MT3t_PIN, MT4t_PIN, MT5t_PIN, MT6t_PIN};
int motorDirPins[]    = {MT1d_PIN, MT2d_PIN, MT3d_PIN, MT4d_PIN, MT5d_PIN, MT6d_PIN};
bool suckBlowTable[]  = {true,     false,    false,    false,    false,    false};
```
We should leave it in the array so we can easily access all three parts.

@ddworken why is only one of them true, what significance does that one motor have?


TeamCity ReOpenLDAP :: devel [Build 212](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1327&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:43:15


TeamCity ReOpenLDAP :: master [Build 37](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1328&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


I think it should be fine.


The package descriptions feel a bit like copy&paste. Should we try to build these from a single source? Or is this a candidate for a follow-up? I'd be happy to have this split out into a separate issue.


I added totalCores to info to get the max activeTasks for determining the activeTask color strength
I considered over 10% failure terrible, so the red color strength is maxed out at 10% failure
Given how large completedTasks is compared to activeTasks and failedTasks it's color strength ignores 90% of completedTasks to totalTasks ratio


Lol.  I think it lost a few points of elegance, but I like that it matches the default theme now.  It'll probably also make a good transition for new users.  When I first installed Pico, I remember being a little thrown off by that teal color and the fact that Pico's theme looked nothing like its website. :wink:


@yehudasa ping


OK.


Excellent, much better @xrmx.


Why do you explicitly blur the tapped element? Buttons usually maintain focus after being clicked. I am writing a pull request that would focus on the tapped element before firing the click event because it is needed in my usecase. I want to know if there will affect some other usecases inadvertently.


yes


OK. get it.


Yes.
On Wed, Dec 2, 2015 at 1:36 AM Anthony Guo <notifications@github.com> wrote:

> @dvdhsu <https://github.com/dvdhsu> Do we still need a dev.secret.exs
> file?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dvdhsu/fiberboard/commit/c689bb856c72fef30fe2841ff4afb1df87dbf994#commitcomment-14725255>
> .
>


Is this better than older version


`x86_86`?  Where did you get this exotic CPU?


# TODO: Update the formatting docs


These would be very confusing for other people to understand. I completely understand why mockito requires them, just not expected in production code.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7580](https://teamcity.mesosphere.io/viewLog.html?buildId=84221&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:01:07


It seems like this might not be well behaved if `InputIterator` doesn't iterate 32-bit types.  If it iterates a 64-bit type it throws away half of the passed in state.  If it iterates a less than 32-bit type, then the upper bits of each word in `seed_` get filled with zeros.  Doesn't seem like what you'd want...


That one motor had the power pins reversed so when the motor controller tries to go forward it actually goes backwards (my fault, but easy to fix in software).


The problem of werid qsnow profiles seems to be solved...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7582](https://teamcity.mesosphere.io/viewLog.html?buildId=84234&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:01:12


//irrelevent plot options, adds a story where there really isn't one
import java.util.Scanner;
import java.lang.Math;

public class PlotStuff{
   public static void main(String[] args){

      PlotDecision();

   }

   public static void PlotDecision(){

      Scanner a = new Scanner(System.in);

      //randomizes the decision to be made
      double d = (Math.ceil(Math.random() * 4));
      int rndm = (int) d;

      //calls a method to make the player decide something
      if (rndm == 1){
         leftRight();
      }
      else if (rndm == 2){
         upDown();
      } else if (rndm == 3){
         downHole();
      }else {
         System.out.println("Nothing interesting happens...");
      }
   }

   public static void leftRight(){

      Scanner a = new Scanner(System.in);
      String choice = "";

      System.out.println("The hallway has ended, do you want to turn right or left");
      choice = a.next();
      //makes sure the player made a valid choice
      if (choice.equals("right")||choice.equals("left")){
         System.out.println("You turn "+choice);
      }else {
      //if their choice is invalid, they are told and the function is recurred
         System.out.println("That is not an option, sorry");
         leftRight();
      }
   }

   //these should all be binary decisions, and therefore use the same format
   public static void upDown(){
      Scanner a = new Scanner(System.in);
      String choice = "";

      System.out.println("You come across a staircase, do you want to go up or down?");
      choice = a.next();
      if (choice.equals("up")||choice.equals("down")){
         System.out.println("You go "+choice);
      }else {
         System.out.println("That is not an option, sorry");
         upDown();
      }
   }
   public static void downHole(){
      Scanner a = new Scanner(System.in);
      String choice = "";

      System.out.println("There is a hole with a ladder in front of you, would you like to climb down?");
      choice = a.next();
      if (choice.equals("yes")||choice.equals("no")){
         System.out.println("Good Choice");
      }else {
         System.out.println("That is not an option, sorry");
         downHole();
      }
   }
}


comentario inicial


What's the purpose of this map? During normal operation, only the Site thread will access the execution engine.


prueba de inicio de git


386: `BridgeFree(threadList.list);`
389: `threadList.list[currentThread].BasicInfo.ThreadId`

lolidonteven


We probably need to update the `updated_at` attribute here too - although a longer expiry time (> 20 minutes) may be sufficient


Yes. When you look at a function, think about what assumptions it makes about its usage and environment. These are the things that restrict it to only working in its current environment, rather than being generalized to other environments. These other environments can be other usages within the program, but, more importantly, they can also be future versions of the program. Right now the title "Script Error" is the only thing restricting it to its current usage. There are, of course, other assumptions not currently relevant that you will want to keep in mind -- like how the input is a narrow-char string, or that the Windows GUI would not look too out of place.

When someone is going through the codebase, they will skim through this file to understand what it does, what's involved -- shrink it down in their head so they can understand its purpose without needing to know the details. They will look through the list of function names. Every extra function in the file is weight -- something more to process. dialog.cpp on the other hand is very small; someone can quickly digest it down to "things that open windows" and pay it no more heed.

Here's a question: How can you get both a general function and not have to duplicate code?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7581](https://teamcity.mesosphere.io/viewLog.html?buildId=84225&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 204 Build time: 00:04:53


Code sai hết rồi


Is that "means" a typo that should be dropped?


@dkayiwa Yeah. You are correct. But, the search index property is only used as a trigger to start the whole process. So, it has done its job well. :)


Good catch! It's obvious when reading `StyleEngine::styleSetProps`: The only time when we're inserting something into `mStyleSetPropsByPath` is directly after emplacing it into `mStyleSetPropsInstances`.
```c++
StyleSetProps* StyleEngine::styleSetProps(const UiItemPath& path)
{
  const auto iElement = mStyleSetPropsByPath.find(path);
  if (iElement != mStyleSetPropsByPath.end()) {
    return iElement->second;
  }

  mStyleSetPropsInstances.emplace_back(estd::make_unique<StyleSetProps>(path, this));

  auto* pStyleSetProps = mStyleSetPropsInstances.back().get();
  mStyleSetPropsByPath.emplace(path, pStyleSetProps);

  return pStyleSetProps;
}
```


Yes, I make compilable version. And now I working on returning only needed ifedfs. So I already return it, but didn't comit yes


Comment


That is what I get for fiddling with vim configs midway through a task. :-P


ok, updated on Issue55-try-3


tét


1) jupp, thats better ...
2) thats why i'm checking zk.exists first ...

anyway, i will merge your version of this here ...


Sorry if that was my fault. Is there any documentation on how to change my code to get it to work?


Fixed, thanks for pointing that out