I should have read the documentation on that. Now noted and updated.

On Tue, Mar 17, 2015 at 1:21 PM Dmitri Shuralyov <notifications@github.com>
wrote:

> Sorry, you misunderstood what I meant. The entire if statement can be
> replaced:
>
> -if strings.HasSuffix(execpath, deletedSuffix) {-   execpath = execpath[:len(execpath)-len(deletedSuffix)]-}+execpath = strings.TrimSuffix(execpath, deletedSuffix)
>
> If you look at the source of strings.TrimSuffix, it already performs the
> HasSuffix if check:
>
> // TrimSuffix returns s without the provided trailing suffix string.// If s doesn't end with suffix, s is returned unchanged.func TrimSuffix(s, suffix string) string {
>     if HasSuffix(s, suffix) {
>         return s[:len(s)-len(suffix)]
>     }
>     return s
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kardianos/osext/commit/2a9b5b41e03fe71cd0bc557665aa301a4ac97e95#commitcomment-10245784>
> .
>


saw approval from jdm
at https://github.com/nnethercote/servo/commit/f306febd13bf407a2282b89cc129bc611054b88a


merging nnethercote/servo/system-reporter = f306febd into auto


nnethercote/servo/system-reporter = f306febd merged ok, testing candidate = 29b158e9


I think you mean // For AMD64 using RBP here


Anyway, I fixed it


Ah yes that's right. Can you verify that this message gets displayed
properly when logging in with an inactive user? A test would be good too.

On Tuesday, March 17, 2015, Tejas Patel <notifications@github.com> wrote:

> Sorry for misunderstanding the Issue. So basically what you want is to
> redirect the users who account are deactivated to a landing page letting
> them know that their account is disabled? If so I think we already show a
> message on the login screen when the deactivated users try to login. You
> can see that here
> <https://github.com/virtualcommons/vcweb/blob/develop/vcweb/core/forms.py#L109>.
> Let me know if I'm still misunderstanding the issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/virtualcommons/vcweb/commit/96764b9e1c0d150d63b4c17dbc99223ea8f5d51d#commitcomment-10241978>
> .
>

--
Sent from Gmail Mobile


good catch!


Edit: Issue was actually that I needed to sync GitHub for Windows and GitHub webpage


Cool. Did you have the chance to review the whole `thread.rb`? I think we might have to move some code to here.


why is this include necessary? winbase.h should never be included directly.


`Prerequisites: ~naradr, ~structures.soviets`


The setup.sh script of the test-subsystem plugin also need to be launch. It set up isAlive and needResync files which are needed by the plugin.


Hey @JustArchi Good work bringing all these optimizations back to L, I'm pretty sure they won't break anything cause I've tested most of them (I mean not so optimized as yours but in the same way) on my [SaberSunset](https://github.com/SaberSunset) builds when based on PA-legacy and it ran pretty good without issues. For 5.1 I've rebased to use pure Google AOSP sources on my SS with CyanogenMod patches for other devices, well I'm still working on it, but I would like to ask you for permission to use your optimization levels on it always giving the right credits. Thank you ;)


OK, there are 2 ways of writing this - 1) either you don't write it at all, or you do `def new; end`  :)


Sure, that's fine


But why?


Мы же договаривались, что в репо поместим только то что нужно :)
Папки tests и runtime тут явно не к месту) как и большинство файликов в корне) прошу исправить)


*are __passed to__ `receiveRecover` before*

But I think it'd be better to explain the general mechanism first and then just point out what's the function of each of these within it.


It's possible.  About 10% of moves r fast moves so hypothetically if you could get the verification time to 0(w/o making any bad moves) it would be 1% of total time so at most worth about 1 elo like a 1% speedup.  Conversely going to 20% would cost you about 1 elo but maybe some fast moves would improve.  It may be difficult for the autotuner to make progress based on such small changes.


I know Kevin uses where().first in videos but it is Rails 3 syntax more likely and nowadays `find_by` is always preferred when searching for 1 record.


I like this, but it also makes me lol :laughing:


De rien m'sieur. Il REST pas mal de bugs à corriger dans ce fichier...


Такое ощущение что вместе пушили) Мы же договаривались, что в репо поместим только то что нужно :) Папки tests и runtime тут явно не к месту) как и большинство файликов в корне) прошу исправить)


Genius at work


all tests pass:
success: http://buildbot.bos1/builders/test-riak_pb/builds/114


You're right. It should have been done that way. I'll change it.


Looks good!


Hey, cm-12.0-optimized is pretty DIRTY right now as I didn't finish having fun yet. I'd suggest you to wait a few days for V4 release, which will be clean in one squashed commit instead of such dirty tests like this one.


Genius at work :)


new commit


static import `BIGINT`


it looks like `ordinalityChannel` is unused. Remove it


Would be nice, if we aren't going to use astropy units throughout (and we should still think about and test that) if we could at least use degrees() and radians() instead of 180 / pi.  And similarly, it would be good to have SEC_PER_JUL_YEAR for SECS_PER_DAY * 365.25....


Good job on flash.now :). However I am not sure if you want to tell user what you have in database. Usually it is a prevention against malicious users - to say "We have sent your reset password to the request email" (even if the email was wrong) - of course your won't send it.


I did a bad, 4 songs*


LGTM.

(I still want to reduce the duplication, but this is an improvement.)


add a `checkState` to assert that the type, if present, is BIGINT


I am a nice comment


Don't use `.find_by_xxx` this will lead to `undefined method` under the hood and will be picked up by dynamic `find_by` which is very ineffective. Instead use `.find_by(reset_token: params[:id])`


@janodvarko  use `npm link` to link in the module instead of installing it?


I don't think you need the actual type here, since it is always `BIGINT` instead I would make this a `boolean`


Additional comments


*`uncommitedEvents. We remove them once they're persisted since they aren't uncommited anymore.*

Also would it break anything if we removed all the uncommited events at once after persisting them? If so, it might be worth explainin. If not, I think that'd be clearer.


if you render, its always flash.now.


Several users complained and I have to agree, having red icons for outgoing transactions is more consistent to the red color of the outgoing amount of a transaction.


It could be.

I just decided to do it that way.


Empty line - delete this.


Good job!


We only want to pull stuff in as we need it - rather than pull in the whole file - for classes were some of it is implement in Java. Otherwise you risk overwriting Java methods which is a pain to debug.


Pretty straight forward.


Изменение юнит-тестов для метода createFileName в связи с изменением работы самого метода.
Добавление юнит-тестов для метода getBytesFromFile.


You may want `Bounds: 44, 88, -2, -33` here.


small independent *what* studio
could be dance for all some people know ;)


"useful" seems odd, would "uncommitted" better describe the data requiring the cache flush?


    mono-proclib.c: In function ‘mono_cpu_usage’:
    mono-proclib.c:709:16: error: storage size of ‘resource_usage’ isn’t known
      struct rusage resource_usage;
                    ^


Oh, I missed that.  Never mind, then.


overall this seems fine to me


Это, кстати, относится и к вашему курсачу)) Выкладывать в гит всякие манифесты, скомпиленные бинарники, настройки среды бесчеловечно и противоречит Женевской конвенции))


    mono-proclib.c:714:2: error: implicit declaration of function ‘getrusage’ [-Werror=implicit-function-declaration]
      if (getrusage (RUSAGE_SELF, &resource_usage) == -1) {
      ^
    mono-proclib.c:714:2: warning: nested extern declaration of ‘getrusage’ [-Wnested-externs]
    mono-proclib.c:714:17: error: ‘RUSAGE_SELF’ undeclared (first use in this function)
      if (getrusage (RUSAGE_SELF, &resource_usage) == -1) {
                     ^
    mono-proclib.c:714:17: note: each undeclared identifier is reported only once for each function it appears in
    In file included from ../../mono/utils/mono-proclib.h:7:0,
                     from mono-proclib.c:7:
    mono-proclib.c:715:54: error: ‘errno’ undeclared (first use in this function)
       g_error ("getrusage() failed, errno is %d (%s)\n", errno, strerror (errno));
                                                          ^
    ../../eglib/src/glib.h:599:70: note: in definition of macro ‘g_error’
     #define g_error(...)    do { g_log (G_LOG_DOMAIN, G_LOG_LEVEL_ERROR, __VA_ARGS__); for (;;); } while (0)
                                                                          ^


I would suggest to assert on reset link path instead of just "Click" as  the link is important part of whole process.


Yep, exactly what I was looking for.
Honza


typo: lenght


Oops sorry Andy I forgot to PR this from my fork.


    ddevault@makase /t/t/mono-git> gcc -v
    Using built-in specs.
    COLLECT_GCC=gcc
    COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-unknown-linux-gnu/4.9.2/lto-wrapper
    Target: x86_64-unknown-linux-gnu
    Configured with: /build/gcc/src/gcc-4.9-20150304/configure --prefix=/usr --libdir=/usr/lib --libexecdir=/usr/lib --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=https://bugs.archlinux.org/ --enable-languages=c,c++,ada,fortran,go,lto,objc,obj-c++ --enable-shared --enable-threads=posix --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-clocale=gnu --disable-libstdcxx-pch --disable-libssp --enable-gnu-unique-object --enable-linker-build-id --enable-cloog-backend=isl --enable-lto --enable-plugin --enable-install-libiberty --with-linker-hash-style=gnu --disable-multilib --disable-werror --enable-checking=release
    Thread model: posix
    gcc version 4.9.2 20150304 (prerelease) (GCC)

On Arch Linux over here, compiling with [this script](https://aur.archlinux.org/packages/mo/mono-git/PKGBUILD).


Per my comment in the ticket, we will need to add exclusions here for the dependencies that this module includes that are already a part of core.


I don't like it!
Change font size


What's with these changes? We should terminate with error if we can not allocate memory. Normal mem_alloc() does.

BTW as seen in http://www.openwall.com/lists/john-dev/2015/03/17/8 it might be a good idea to print how what size was requested too...


Not to put too fine a point on it


Say I'm the only bee in your bonnet


Please rewrite


@bdvholmes This is an excellent addition. I didn't even think about this in my implementation, but it just saved my bacon.


Make a little birdhouse in your soul


I pushed the code https://github.com/JordyMoos/JMItemCode/commit/7e9988c8c839c6772c4b05877a22da595fea0135


What is this column variable for?


@celiala Looks like u got it - awesome! :dancers:


Do these need to be 64 bits? I.e., does the comment in fc4418e4a52d142d9e3b5ea449a3f425ddb90ad9 apply?


`LocalOffset: 0,0,2200` should work.


Hi Oliver-

Thanks for your email. I have mpsse.py installed--not sure exactly what you mean. I was thinking today that I might need to make sure that the ftdi kernel driver is not loading--at least I think the kernel driver should e u loaded.

Thanks,
Richard

Sent from my iPhone

> On Mar 17, 2015, at 4:00 PM, Oliver Kleinecke <notifications@github.com> wrote:
>
> do you have pympsse installed correctly?
>
> —
> Reply to this email directly or view it on GitHub.
>


https://github.com/firebug/firebug.sdk/issues/4

Honza


please fix this :)


Can you write a test if you think there's a bug ?
Le mar. 17 mars 2015 à 21:27, Lilians AUVIGNE <notifications@github.com> a
écrit :

> Ok, thanks for your answered.
> Perhaps I don't override correctly the errorPage process.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodeStory/fluent-http/commit/a862caf60fc4d4dd4baeda6316ea0d915f092265#commitcomment-10245915>
> .
>


... document


When you are sending out emails, make sure you clear them out before and/or after. `after { ActionMailer::Base.deliveries.clear }`


Would be useful to be explicit about what's needed for SMP.


Successfully changed permissions.


I have a secret to tell


Why?


From my electrical well


I also have a `createAndFundAccount` (singular) function available. For me, though, creating them all separately got tedious and I started making copy/paste errors. This is another pattern that I stole from the JS tests, specifically `create_accounts` in `test/testutils.js`.


Ar mai trebui si link catre site


danderson.work


It's a simple message and I'm leaving out the whistles and bells


So the room must listen to me


Filibuster vigilantly


My name is blue canary one note spelled l-i-t-e


My name is blue canary


One note, spelled L-I-T-E


My story's infinite


Like the Longines Symphonette


It doesn't rest


Da na na na na


Blue canary in the outlet by the light switch


Who watches over you


@raehik will be angry cause of the emails he's gonna get! :laughing:


Yeah :)


It'll revive memories of Y8


habs nicht getestet, sieht aber richtig aus :)


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1651 of 1655 tests passed)

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


ack


There's a picture opposite me


Fixed. Formatting cleanup will be in my next push.


Of my primitive ancestry


嗨,

hjwdzh@gmail.com 想關注您

****** hjwdzh@gmail.com 是您的朋友嗎？ ******
好/是: http://invites.flipmailer.com/signup_e.html?fullname=Wchargin/car2014&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=e7acfdae-3b1a-4f47-b9b6-8dcab36b61c4&from=hjwdzh@gmail.com&template=invite_reminder_3_b&src=txt_yes

不/否: http://invites.flipmailer.com/signup_e.html?fullname=Wchargin/car2014&email=reply@reply.github.com&invitername=hjwdzh&inviterid=37132591&userid=0&token=0&emailmasterid=e7acfdae-3b1a-4f47-b9b6-8dcab36b61c4&from=hjwdzh@gmail.com&template=invite_reminder_3_b&src=txt_no

按一下這裡，將您自己從所有這類電子郵件移除
http://invites.flipmailer.com/uns_inviter.jsp?email=reply@reply.github.com&iid=e7acfdae-3b1a-4f47-b9b6-8dcab36b61c4&from=hjwdzh@gmail.com


Which stood on rocky shores and kept the beaches shipwreck free


It's just an edge case.
If you have the parent folder with a different directive path, you will inherit a wrong path to load your bower components.
e.g
```
my-code
 |-.bowerrc
 |- angular/
 |
 |-protractor/
    |
    |-website/
```

In my parent folder the `.bowerrc` was looking like this
```
{
  "directory" : "static/components"
}
```
and that was causing an error during the `npm install` of protractor `website` folder.


Though I respect that a lot


I'd be fired if that were my job


Stop


For I


Can Take


I think you missed a curly bracket. Since that came from the else portion of the code.


No More


Ok


Aici as spune doar Spor la munca!


Why this change?

Perhaps reasonable for testing in the short term, but we will want to make it the same as amd64 (i.e., 256) as soon as feasible.


After killing Jason off and countless screaming Argonauts


@ntdb What's up with this '500'?


You should catch OSError ;)


Bluebird of friendliness


Like guardian angels its always near


This is made and optimized for MSM8226.
Your board might need some extra flags definitions (BSP,...)


YUS


当时的代码不忍直视啊


awesome job, Octino!


one tab too much


Ok I'll actually stop


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1147 of 1148 tests passed)

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


This is a submodule I added... Sorry, I'm gonna fix that.


I'm absolutely in favor of keyword arguments.


Let's buy one for Mr Meehan http://www.thinkgeek.com/product/e791/


I think if anything to be inlined it would be the it block, I always prefer to have the blocks as follows, though:
```ruby
describe "#method_name" do
  before do
    the_thing.method_name
  end

  it "works" do
    expect(the_thing).to eq(the_right_thing)
  end

  # I don't mind this if you have a bunch of simple things to check
  it "sets a" { expect(the_thing.a).to eq("a") }
  it "sets b" { expect(the_thing.b).to eq("b") }
end
```

If it's just one test as in these examples, I think I'd still rather see:
```ruby
describe "#reason" do
  it { expect(request.reason).to eq(reason) }
end
```


Why do we need to compile? It's done by maven-compiler-plugin since we compile only Java classes.


Thanks guys, I'll try to remember that next time.


An egregious case of copy-pasting from my rails console. Patched!


Ok. Thank you.


:laughing:


This causes GitHub's renderer to add a horizontal scrollbar, let's make it shorter or move the comment to the previous line


Yeah


Since component tests load spring context anyway, shouldn't we remove our single TestNG test?


The `margin-top` property was causing the vertical alignment of the slider to be too high...

![screen shot 2015-03-17 at 3 57 38 pm](https://cloud.githubusercontent.com/assets/1365941/6697481/c4a9ab10-ccbe-11e4-8cc3-38d3df95ec4b.png)


Looks good!  Use public interfaces instead of instance variables where possible.  In this case, you have public `left`, `right`, and `value` methods.  Use those and not the instance variables.  Think of it as [eating your own dog food](http://en.wikipedia.org/wiki/Eating_your_own_dog_food).


These dependencies must be test scoped.


API's -> APIs


@smarterclayton I've added this but left it commented out for now while we discuss where the v2 registry image should land.


If a list is returned other function in search.py won't work. Did you do this because render.py get a list in argument ?


@smarterclayton PTAL


@marmbrus Should we make the DataFrame the "recommended" way to do it with the AvroSaver/implicit one being the alternative?


space needs to be between the the if declaration and the parameters :p
Twiz will get made ;p


Why?


79453838-AutomateSomeDocumentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


r=jgraham


r=Manishearth


Yeah, prefer the unified `load`/`save` API for all examples.


Yea render.py takes a list of dictionaries. Sorry if I wasn't clear about this.


fixes #69


needs some rework?
https://github.com/kholk/kernel/commit/2fb18674bb8947177eae7051d61e80ef0db6c566


I can't see the PR?


Moved this into a script, that can be modified for testing purposes.


Next up, make it so that we can actually read and use the data.


86121534_1304_WarnOnPATUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Closes #4


shoudln't they include/functions/simd/??


Can you please ellaborate? @CaptainElliot


Interesting, will remake this correctly asap.


As you make fixes and get this working, please do contribute them back! I'd be happy to merge your fixes when you're happy with them.


Seems legit.


nice fix, bruh!


There's an extra 's' in here David...


that's quite a strange coding style you got there. why so much unnecessary whitespace?


I have a problem with this line.
Current implementation related that links is LINKLIST or LINKSET and if Property is null i can't create edge between to vertex. Asked question on orientdb group about this issue and luca tell me use LINKBAG instead.
https://groups.google.com/forum/#!topic/orient-database/6_oP4ikTVd8
I think this will be a braking change on the driver.
WDYT ?


Is there a need for singleton? We use Spring IoC in these tests, so we can simply create those instances there and inject them right into tests.


RabbitMQ: http://www.rabbitmq.com/changelog.html
AngularJS: https://github.com/angular/angular.js/blob/master/CHANGELOG.md

Perhaps what I'm getting at is that there should be something like "Cleaned up and made spacing calculations more consistent" which covers line 16, 22, 23, and 26.  Line 17 means that you actually have to change your projects if you changed spaces in them (so maybe the wording is too vague?).  Line 19 means you should update your configuration file if you use TeXworks.

In any case, this is just my opinion among many. As long as the guidelines are clear, I will follow them.


I won't be able to make it either but my excuse is not as legit as
Javier's...I'm just going to a St. Patrick's day bar crawl.

Javier - I hope she is doing ok!

See you guys next time!
Deirdre

On Mon, Mar 16, 2015 at 8:13 PM, Scott Windsor <notifications@github.com>
wrote:

> Oh no - I hope your mother-in-law is okay.
>
> No problem, we'll be here when you have time again.
>
>
> > On Mar 16, 2015, at 8:02 PM, Javier Soto <notifications@github.com>
> wrote:
> >
> > I won't be able to make it to the meeting this time. I have my parents
> > visiting from Spain and my mother in-law just a broke a hip. Are these
> the
> > infamous Ides of March? [image: 🙀]
> >
> > On Monday, March 16, 2015, Scott Windsor <notifications@github.com>
> wrote:
> >
> > > Yup, we're on Ch 16 this week [image: :thumbsup:]
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/SeaRbSg/little-schemer/commit/156bb1b5d2208c6a6a4d7339c77a265d5169c290#commitcomment-10227807
> >
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SeaRbSg/little-schemer/commit/156bb1b5d2208c6a6a4d7339c77a265d5169c290#commitcomment-10227974>
> .
>


Load time for "api/app/views/spree/api/products/show.v1.rabl":
10 products: 7996ms (Views: 7755.6ms | ActiveRecord: 232.3ms)
30 products: 21802ms (Views: 21198.6ms | ActiveRecord: 595.4ms)


{this} instead of the card name


`perform(post())` can be encapsulated inside `Users`.


This is epic, don't know why there was an if and else in the first place.


87727128_GridViewInheritance (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


ack from me too


looks like the paragraph reflow ate a space after exist,


because it was a clean merge ;p

But yea it likely needs to be reset

On Tue, Mar 17, 2015 at 2:52 PM, Eric McCann <notifications@github.com>
wrote:

> this smells a bit funny... why the backwards merge with such a huge delta?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VanirAOSP/packages_apps_Bluetooth/commit/e27a9ba01e1f5e7da384601a25297fc820f3f95a#commitcomment-10244115>
> .
>


Can be simplified to `User has permissions to create topics`. As an alternative - you can use `and:` label if you have multiple `given:`.


Report


Temporary workaround until #2661 is fixed. When the dataService cannot find an entity with the referenced ID, something is wrong and throwing an exception is probably the right thing to do. For now, we filter them out.


Can you use social-feed in general? yes - both technically and legally.
Can you use bower.json? I am not much familiar with phonegap, but I believe you can.


DAMN SUN


the output looks good and the code pretty neat


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1036
success: http://build.servo.org/builders/linux2/builds/1036
success: http://build.servo.org/builders/android/builds/1070
success: http://build.servo.org/builders/mac1/builds/1252
success: http://build.servo.org/builders/mac2/builds/1252
success: http://build.servo.org/builders/gonk/builds/223


`init` can be invoked by Spring IoC if you convert it to instance method.


html too hard


Closes #1


30414/Palko A. Lorand Arpad/assignment 2.1/main.c

you're not printing the right thing here..


I'm not really interested in working on a plotting library right now. This is just about slightly touching up the other David's old code to make it work with current GtkD and Dub.


should be `if (` not `if(`
its oracles convention that twiz gets picky on :p


Remember, there are four cases to deal with.  I'm going to call the value we're looking for `needle`, as in "needle in a haystack"

1.  `needle` doesn't exist
2.  `needle` exists and has no children
3.  `needle` exists and has exactly one child (either left or right, but not both)
4.  `needle` exists and has exactly two children (both left and right)

To handle each case:

1.  If `needle` doesn't exist, there's nothing to do.
2.  If `needle` has no children, remove `needle` from the tree
3.  If `needle` has exactly one child...
   - If `needle` has a left child but no right child, replace `needle` with the root of `needle`'s left sub-tree
   - If `needle` has a right child but no left child, replace `needle` with the root of `needle`'s right sub-tree
4.  If `needle` has exactly two children...
    - Find the largest value in `needle`'s left sub-tree (i.e., `left_max = needle.left.max`)
    - Replace the value of `needle`'s node with `left_max`
    - Remove `left_max` from `needle`'s left sub-tree

This works because, by definition, the maximum value in any subtree has at most one child, i.e., it might have left (smaller) children but because it's the maximum value it can't have any right (large) children by definition.  This reduces the two-children case to either the 1-child or 0-child case, either of which we know how to solve directly.


for future one could write `last_pos = 0 if item is None else item.end()` but its only a matter of style so let's leave it with what it is now we have so many acks ;)


fast-forwarding master to auto = 29b158e9


Great commit !


Awesome, really like this ! :smile:


Vote up for this! Spent hours debugging until lost hope and went to github project PR's page.


90298376_ResultsViewerUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


30414/Nistor A. Rares Constantin/assignment2.1/main.c

considering you have a ```prev``` field, this function could be made a bit simpler


this is the most extensive testing I have ever done, this is great. You can drop bombs and this function will very very probably hold. :)

ack


good work overall


Don't use blank lines here (//comment example)


one can use find ./existing-path instead, That would fail, if existing-path does not exist. Subtle difference, but still relative paths.

Thanks for caring!
VP


@chrisseaton I know, but you moved `randomizer` from `api/kernel/common/thread.rb`
 to `rubinius/common/thread.rb`.

However, `randomizer` was extracted from https://github.com/rubinius/rubinius/blob/v2.4.1/kernel/common/thread.rb#L123 `kernel/common/thread.rb`.

There are more methods who were extracted from https://github.com/rubinius/rubinius/blob/v2.4.1/kernel/common/thread.rb but they are still on `api/kernel/common/thread.rb`.

So if we move `randomizer` than we should move all the others methods as well (like `recursion_guard`).


Looks good


saw approval from jgraham, Manishearth
at https://github.com/Ms2ger/servo/commit/a8553c36064f336fb3e2b0ab4e62a19ced8561f6


merging Ms2ger/servo/wpt_20150304 = a8553c36 into auto


Ms2ger/servo/wpt_20150304 = a8553c36 merged ok, testing candidate = ff39ae6c


there is an issue with the confirmation code , the tool says that the code was sent but it never arrive
what is the reason for this?


...ending... : theres a STAR MAN!!! WAITING IN THE STARS!


fast-forwarding develop to borshop-integration-112-zl/add-optional-timeout-to-RpbYokozunaIndex = 78c50efa


Oki doki, thanks for uploading all the files.


I definitely went the `a_sym_call` to begin vith, but that left me with a type mismatch. I guess I needed to find `new_call_1` to fix that?

Thanks for the code review. Glad we're working "together" again after so many years. :)


HE'D LIKE TO COME AND MEET US BUT HE THINKS HE'LL BLOW OUR MINDS


Nit: prefer '///' for doc comments.


HE TOLD ME


30414/Popescu P. Diana Gabriela/Assignment2.1/doubly linked list/main.c

what a privileged girl ```carina``` must be to have her name as a variable


LET THE CHILDREN USE IT


LET THE CHILDREN LOSE IT


The PR was for some reason merged regardless of the fact that 3 people said it's better to use this code. And then we end up with posts like this: http://community.mybb.com/thread-167116-post-1140267.html#pid1140267 because of using codes that are not officially supported by external sites..


LET ALL THE CHILDREN BOOGIE


The code looks nice and clean


:smile:


this wasn't breaking because we are not passing `-DTORRENT_DEBUG` to the swig options right?


30414/Florea I. Razvan/assignment 2.1/main.c

I think this function can be trimmed down a bit..


Would it be cleaner or easier to to change the form to have ```completed_at``` instead of deleting ```is_completed```?


SPI initial test and debug complete. SPI state system able to send data in multiples.


but now you have a ```prev``` pointer! this function can be made much shorter


HE'D LIKE TO COME AND MEET US... BUT HE'S CLINICALLY DEPRESSED


Correct


30414/Florea I. Razvan/assignment 2.2/main.c

you can create any file with any extension and edit it just as a text file in notepad


Overall, pretty good work


sure!


Ce nu mergea cu santinelă?


Yep, pretty much what was needed.


Weirdly enough, this is in fact the misreferenced version. I'll overwrite this change ASAP. Will work more tonight.


LOL! C'est ça ton problème? Moi j'ai aucun problème.


HE'S SPENT SO LONG WITHOUT COMPANY, HE'S BECOME A BIT SAD.
WHY ARE WE shouting?


The comment

```
## Locate the official packages, we intentionally skip
## version 3.5 as it does not work on Linux when LLDB is loaded (causes crash)
```

(lines 3-4 of `FindLibClang.cmake`) is now inconsistent with respect to the addition of 3.5.

Also, `FindLibClang.cmake` looks for clang versions in reverse order of `FindLibLLDB.cmake`.


Don't forget about specific names for operations when dealing with queues and stacks..


Cand am înlocuit cu santinela, imi compilează dar se blochează instanța

On Tuesday, March 17, 2015, Cosma Alex <notifications@github.com> wrote:

> Ce nu mergea cu santinelă?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sergiucurea/DSA-lab/commit/465aa5282b57f5f08921bc577a8ebccb5a5d2f2f#commitcomment-10247164>
> .
>


I do, too, but some of the dependency unwinding is a little less straightforward and will have to wait.


30414/Spatacean V. Daiana Maria/Second/main.c

very nice, using such a ```find``` function


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

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


init commit


LGTM.

Thanks!


Probabil ai uitat o inițializare pe undeva


Mono.Addins and MEF don't require you to do this work to discover assemblies and or create instances. In fact they create instances automatically when needed. and we have control over what instances are going to be allowed multiple, and what are going to be shared.


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1150 of 1151 tests passed)

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


:+1: :four_leaf_clover:


На своем сервере. Картинка не загружается, а когда кликаешь открыть обьявление оно на белом экране. Остальные открывает нормально.


ありがとうございます！
教科書的に写経してる部分が多いのですが、そういったところ大事ですね！
もっと意識しながら進めます！


Great catch! That flag should definitely be on by default!


:100: pkt.


I DUNNO


GUYS I NEED HELP!


This is what we are looking to avoid, and this is where Mono.Addins and MEF can help us.

We don't want to have to add References to inject modules. We just want to copy them into the folder. MEF has Directory Catalog
https://msdn.microsoft.com/en-us/library/system.componentmodel.composition.hosting.directorycatalog%28v=vs.110%29.aspx

Similarly Mono.Addins have directory discovery.

Does Ninject have anything in that manner?


@ntdb from near the end of the batch:update_constantly on staging:

[2015-03-17 18:30:29.774 INFO ] Updating Course cache
[2015-03-17 18:30:29.102 ERROR] Mysql2::Error: Unknown column 'users.role' in 'where clause': SELECT DISTINCT COUNT(DISTINCT `users`.`id`) FROM `users` INNER JOIN `courses_users` ON `users`.`id` = `courses_users`.`user_id` WHERE `courses_users`.`c
ourse_id` = 349 AND `users`.`role` = 0
rake aborted!
ActiveRecord::StatementInvalid: Mysql2::Error: Unknown column 'users.role' in 'where clause': SELECT DISTINCT COUNT(DISTINCT `users`.`id`) FROM `users` INNER JOIN `courses_users` ON `users`.`id` = `courses_users`.`user_id` WHERE `courses_users`.`c
ourse_id` = 349 AND `users`.`role` = 0
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `query'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `block in execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_adapter.rb:373:in `block in log'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activesupport-4.1.8/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_adapter.rb:367:in `log'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:228:in `execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:232:in `exec_query'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:240:in `select'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:24:in `select_all'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/query_cache.rb:70:in `select_all'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:265:in `execute_simple_calculation'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:227:in `perform_calculation'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:119:in `calculate'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:34:in `count'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation.rb:248:in `size'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:85:in `update_cache'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/delegation.rb:46:in `each'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/delegation.rb:46:in `each'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:202:in `block in update_all_caches'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:201:in `block in transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:209:in `within_new_transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:201:in `transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/transactions.rb:208:in `transaction'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:201:in `update_all_caches'
/var/www/dashboard-testing/releases/20150312181952/lib/tasks/cache.rake:13:in `block (2 levels) in <top (required)>'
/var/www/dashboard-testing/releases/20150312181952/lib/tasks/batch.rake:13:in `block (2 levels) in <top (required)>'
Mysql2::Error: Unknown column 'users.role' in 'where clause'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `query'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `block in execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_adapter.rb:373:in `block in log'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activesupport-4.1.8/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_adapter.rb:367:in `log'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract_mysql_adapter.rb:303:in `execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:228:in `execute'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:232:in `exec_query'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/mysql2_adapter.rb:240:in `select'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:24:in `select_all'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/query_cache.rb:70:in `select_all'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:265:in `execute_simple_calculation'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:227:in `perform_calculation'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:119:in `calculate'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/calculations.rb:34:in `count'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation.rb:248:in `size'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:85:in `update_cache'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/delegation.rb:46:in `each'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/relation/delegation.rb:46:in `each'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:202:in `block in update_all_caches'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:201:in `block in transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:209:in `within_new_transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/connection_adapters/abstract/database_statements.rb:201:in `transaction'
/var/www/dashboard-testing/shared/bundle/ruby/2.1.0/gems/activerecord-4.1.8/lib/active_record/transactions.rb:208:in `transaction'
/var/www/dashboard-testing/releases/20150312181952/app/models/course.rb:201:in `update_all_caches'
/var/www/dashboard-testing/releases/20150312181952/lib/tasks/cache.rake:13:in `block (2 levels) in <top (required)>'
/var/www/dashboard-testing/releases/20150312181952/lib/tasks/batch.rake:13:in `block (2 levels) in <top (required)>'
Tasks: TOP => cache:update_caches
(See full trace by running task with --trace)


@joeferner if this is testing varargs, isn't creating the String[] array here incorrect? Likewise in the next test.


Maybe wrap this in a variable so that future readers aren't confused by the `+ 2`?


cool thanks


With what?


With what


Anyway, use gitter instead


Yeah


@ppbdrinker Sorry I'm so slow to merge PRs! School is sapping up almost all of my time. At one point, I had some collaborators that were able to help curate it, but folks have moved on to other projects.

Just curious, what are you using Novocaine for currently?


ONCE I CREATED A NEW BRANCH WHAT ARE THE LINUX TERMINALS FOR COMMITTING TO THE CODE


Ciao


Use gitter


same as normal


I know, I'm just **THAT** good :wink:


`git checkout -b Branch`


This still doesn't work, but at least it looks like it should. Small victories. Last commit for right now.


This was already present.... oh well :P


Is there a delay before this is available for upgrade? I tried `bundle update sidekiq-pro`, but this version was not found. We just purchased Pro last week, so this will be our first upgrade. Thanks!


O sa ma mai uit dimineata, acuma oricum e prea târziu.

On Tuesday, March 17, 2015, Cosma Alex <notifications@github.com> wrote:

> Probabil ai uitat o inițializare pe undeva
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sergiucurea/DSA-lab/commit/465aa5282b57f5f08921bc577a8ebccb5a5d2f2f#commitcomment-10247209>
> .
>


Damn! Was that all I was missing.. Thanks for picking that up


I'd prefer dump to go before console.log, because the console will be available in workers as well, and that will break things showing up there (ie when running as part of desktop firefox).


If there's a better (saner) way to do this, please let me know :|


Erreur, vitesse > 1 = NaN sur l'angle


2.0.1 has not been released yet.  Expect it by the end of week.


Tą '5' bym wyrzucił do stałej - potem zmienimy w polu a w `'min_length': u'Hasło musi posiadać co najmniej 5 znaków.'}
` zostanie.


it was by intention that a user can join a battle without having the engine... but hmm, maybe disallowing is better for usability.


maybe better: don't start spring if version doesn't match


Comment to restore the branch and revert the code? Those are GitHub features.

You just trowed the PR to the trash.

Seems like you couldn't reference the issue in this commit either.

#1836


Title change


@hxim Thanks for the suggestions! I have incorporated them and checked the bench sig is unchanged.


Awesome. I was just looking for a way to do this...and it was in the midst of being added. Sidekiq Pro is sweeeeeet!


ouch - you are completely right!
Will merge this


great!
will merge it


Very messy, but it gets the job done.


stay outta checkout ya bums!


looking goooooood to me!


my oh my how did you work that magic?!?!


What do you mean by the standard routes?  I added the route here so we don't include the stub-login-form route if stub is not used.


slick button!


If you don't have `BinarySearchTree#remove(value)` return a reference to the root node of a tree with `value` removed then you'll have to write your code from `needle`'s parent's context vs. `needle`'s context.  This would add an extra layer of conditions to everything, but it would still work.

That is, if you're on `self` and see that `self.left` is the node you want to remove, you'd then go about checking whether `self.left` had 0,1,2 children and setting `self.left = ...` to the appropriate thing.  You'd then want to do the same check for `self.right`.  This essentially duplicates the logic.


"magic is an illusion."  Doug Henning


no need for extend here, there isn't anything being extended


Wydaje mi się że
`$passwordInput.data('original-title', errors['password'])` by zadziałał identycznie, a jest preferowany przez jQuery.


@Tsunamical can you send me your Hangouts/G+ account link? I need to be in touch with you, also when we need to chat with all SS devs (U, ME, Render and Erivaldo for now) we need hangouts for better taking decisions and so on. I'm patching CM repos by now, soon we'll be able to start compiling it and fixing what need to get fixed.


Design é urgente


maly komentarzyk


You will want to define an `EmptyBinarySearchTree` that is a subclass of `BinarySearchTree`.  I don't know a clean way around this, because you'd want it to also be a subclass of `EmptyTree`.

The only way I can think of is to make some kind of `Empty` module that you can include in a class to add the appropriate "empty behavior" to instances of that class.  Or a `Tree` module that could add tree-like behavior to a class.

Both are ugly in their own way.  One is likely to be slightly more "natural" than the other, but I haven't thought it through so I can't say offhand.

I have my code structured w/ the weird hierarchy, i.e.,

```
BinaryTree
  EmptyTree
  BinarySearchTree
    EmptyBinarySearchTree
```


kolejny komentarz


- srcNoData should be renamed to pdfSrcNoData to match naming convention
- I will be tempted that USE_NODATA=1 to be the default behaviour, and people specify USE_NODATA=0 explicitely if they want input nodata to be ignored.
- maybe USE_INPUT_NODATA would be less confusing with the existing NODATA option which is about the output ?
- updating gdal_proximity.dox would be needed
- a new test case in autotest/alg/proximity.dox would be needed


to go back to master branch I type git push right?


For the two lines above, Do you want to strip the slash (/) in the replace on 144 and 145? I was running in to bad paths and changed this in my branch.


This is a workaround to this issue until we have hockeyapp uploading builds. Then we'll fix this.

https://travis-ci.org/ivanarellano/hockeyapp/builds/54779832
```
Support library repository outdated. Downloading update...

FAILURE: Build failed with an exception.

* What went wrong:

A problem occurred configuring project ':app'.

> Could not resolve all dependencies for configuration ':app:_debugCompile'.

> Could not find com.android.support:appcompat-v7:22.0.0.

Searched in the following locations:

file:/usr/local/android-sdk/extras/google/m2repository/com/android/support/appcompat-v7/22.0.0/appcompat-v7-22.0.0.pom

file:/usr/local/android-sdk/extras/google/m2repository/com/android/support/appcompat-v7/22.0.0/appcompat-v7-22.0.0.jar

Required by:

hockeyapp:app:unspecified
```


Seems simpler than other options


Hiba nie do końca, bo przez attr dorzucam atrybuty, których można używać standardowo, a przez datę można wstawiać cosiekkolwiek. Ale to moja pokrętna logika, spójrz [tu] (http://api.jquery.com/attr/).


r=jdm


milestone


The first commit - just playing


Non portable functions


It is in the Update Reg endpoint commit: https://github.com/edx/edx-platform/commit/4b83f97f862601c809b2fedb7d57b59eb1b2c6eb#diff-9c15f7dd12df13f452e1c32f8b68a166R1533


@intale - I think the name primary_urls and default_urls is a little ambiguous.

Maybe think of it like a TrackingLink has one primary_url and has many secondary_urls

or a TrackingLInk has one default_url and has many alternative_urls


Are you [persisting](http://en.wikipedia.org/wiki/Persistence_%28computer_science%29) searches at all?  If not, there's no reason to make `Search` an ActiveRecord model — it could be a plain ol' Ruby class.


How does this work?


Tak, ponieważ funkcja `document.getElementsByTagName` zwraca tablicę wszystkich elementów `<form>`. Żeby dostć się do pierwszego musismy odwołać się do elementu o indeksie 0, czyli: `var form = document.getElementsByTagName('form')[0];`. Ale najlepiej w sumie dodać id formularza w kodzie HTML i potem odwoływać się po id.


Just used git-rev in an internal project; worked like a charm! Thanks @acfoltzer


WOW gordon you deleted my work good thing I saved it on my laptop...


plz help, my inquiry number is: 11773 26519
                        and the date is: 3-17-15


It is in the Automatically link accounts commit: https://github.com/edx/edx-platform/commit/081695181e42e5ab99eab25289c55fe332e995e7#diff-ab6a1d1a08588c2a45d57ed7762e9651R82


can can can can


Awesome! I've only tested it on my dev laptop and our Jenkins machines, so please let me know if I missed a weird corner case, particularly with how often it triggers a rebuild of that module.


Given `ZFS_OBJ_MTX_SZ` is used as a mask, wouldn't you be better off with a power of two?

```
include/sys/zfs_znode.h: #define ZFS_OBJ_HASH(obj_num)   ((obj_num) & (ZFS_OBJ_MTX_SZ - 1))
```


Hmmm okay


To allow output result to show in normal wiindows notepad properly instead of showing in 1 line.


As explained on the forum this makes the normal dimlet actually generate normal worlds. If you want an amplified world then use the amplified terrain dimlet.


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


W taki wypadku najlepiej nasłuchiwać bezpośrednio na inpucie, a nie na formularzu, Formularz może mieć parę inputów, a my chcemy wykonać funkcję gdy "podniesiemy" klawisz z konktetnego.


This commit broke collectd. There is also another cause for collectd breakage in master, having to do with rabbitmq (that's what the current hydra build error shows).

I have little time to dig into this, just thought I should notify you (all).


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1037
success: http://build.servo.org/builders/linux2/builds/1037
success: http://build.servo.org/builders/android/builds/1071
success: http://build.servo.org/builders/mac1/builds/1253
success: http://build.servo.org/builders/mac2/builds/1253
success: http://build.servo.org/builders/gonk/builds/224


I love it. Feel free to tweet that! :-)


This repository seems to be not found


Ah, well, yes, looks odd. I think I needed to do it like this for some reason but don't remember why. I will do some tests with IE <= 9 and see if I can confirm...


It's probably fine to squash this commit into `e98dcad2059f2929e2c2bdf06f8fa58edea7dd2f`.
It'd make the history a bit nicer ++ This diff isn't too informative


added DSN number


I think I tried that kind of init … but was is too really in the process for the ADE reset.


can you make the hex all lowercase, like the rest of the hex codes?


I actually liked keeping them as separate Mixins so there's flexibility in the future to add tests to the base one if we wanted.  And it's clear when looking at the subclass what test class is inherited and what 3rd party mixin is used.


fast-forwarding master to auto = ff39ae6c


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


I think this file should be named `jquery.libLink.js` (capital `L`), otherwise this fails on a case-sensitive OS: https://github.com/leongersen/noUiSlider/blob/9d05e27c645a/Gruntfile.js#L52


Why the clone here?


Thanks. Sorry about that! .. I'm usually the one telling everyone else to fix their spaces & tabs, indentation, etc. ;-)


83782874_LBNL_ThermostatOffsetFault (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1651 of 1655 tests passed)

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


Well, the comment got old after I've added the inverter code at lines 91 to 94. i should update this in the next commit :)


Also filtering the post by thread (e.g. /s/dota2) is broken for now since the post array is not attached to the render process anymore.


teste lucas


We could just use squared distances, saving some computation time. I also wonder if direct multiplcation is faster than `Math.pow()`; Can we test that?


My first reaction about "is data sent" is that it doesn't properly describe push messages with no body, which are more like tickles. However, the sheer fact that *something* was sent by the application server is data in itself, so we're not talking about included data, but about the concept of data.


Just out of curiosity, why? They seem unused to me?


This will almost certainly return to haunt us...


I agree.  I have moved it to common/djangoapps/third_party_auth/tests/utils.py.


Eu que fiz


At the time I was wondering why we didn't use ``/login``. But now that the views are in this package it doesn't make sense anyhow. I'll remove this comment.


Soft! Men vi borde väl också lägga mysqli_real_escape_string på allt det här för att undvika sql-attacker med? Det funkar väl bra om vi lägger in den här strip_tags-funktionen i safeString() i DBQuery coh kör den överallt istället?


stalker :D


I'm just saying as a general rule. These silly jokes have a way of slipping
into production :)

On Tuesday, March 17, 2015, Vladan Mitevski <notifications@github.com>
wrote:

> Yea. That was a joke. I don't see anyone are offended. BTW, we should
> create app and not to worry about names. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reciprocity/ggrc-core/commit/611c34cdd7b9c014035932c84f8f4fb9ca50d726#commitcomment-10232443>
> .
>


lol.. i forgot about that, looks like it's a typo because `config` is passed in so should be the second param to that extend


I suspect you want to do lea TMP6, OBJ:TMP6->data;


Need to put the test for result back in here.  Go ahead and do that and then I'll merge the pull request.


To prevent this name from getting too generic with time, I suggest we get back to "YarnHelper" or "YarnUtils".


why flags when you can return?


OK, LGTM, waiting for the rebase.


This utilizes the fix you previously used in the travis config. See https://github.com/Connexions/openstax-accounts/commit/a48b8188819c799c07119d8eb3a5517b5df268b3#diff-ea33c78d19af6bf60483d3b777ec200bR32

What I've found is that it is also useful for working with an accounts instance that is running within a VM.


Forwards and resolved >>>>> backwards and dirty
On Mar 17, 2015 5:16 PM, "Derek" <notifications@github.com> wrote:

> because it was a clean merge ;p
>
> But yea it likely needs to be reset
>
> On Tue, Mar 17, 2015 at 2:52 PM, Eric McCann <notifications@github.com>
> wrote:
>
> > this smells a bit funny... why the backwards merge with such a huge
> delta?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/VanirAOSP/packages_apps_Bluetooth/commit/e27a9ba01e1f5e7da384601a25297fc820f3f95a#commitcomment-10244115
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VanirAOSP/packages_apps_Bluetooth/commit/e27a9ba01e1f5e7da384601a25297fc820f3f95a#commitcomment-10246896>
> .
>


Yeah, I know.... had no time for that.


Can you explain the bug and please report it? Thank you ^^


updated.


@ncri no problem...just thought I'd leave the comment in case it comes up for you or anyone...I was seeing odd results coming in to one of our apps where the / was missing and finally tracked it back to the JS. I'm still looking in case there are other root issues. I've changed it in my fork so we'll see.


why?


Warning: insecure (can be spoofed), but everything is behind reverse proxy for now, so it's not a problem.


Is there a reason to do the split here?


Might want to do a null check here as well (I've seen some crazy things in my day lol).


Quite a bit of config changes, not just a software update.


Tbh I'm not sure what it is and didn't had much motivation to reduce yet :)

For some reason `parameters.to_a` started to return a `Array(NoReturn)`


Upon reflection, we will adhere to the original request from admin to have links to the student-health-plan.pdf and student-health-emergency-response-protocol.pdf in both student-services/registration and and student-services/forms. These are important forms, and placing them in only one would cause confusion with our end users.

Although this technically violates the "one document one location" rule of thumb, it is a minor risk in that a) the pdf is only in one location, and b) the registration category is dynamic and will change over time anyway.


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/5f45bf1032963d96f4e4a2e9a103634bdf06344c/app.js)


Is there any chance you could remove all of trest runner and nuget project file changes before the final commit? If not, I can do that :).


Good catch. I missed that in class.


What is this intended to do?
Here is the default value for that attribute: `default['apache']['mod_ssl']['strict_sni_vhost_check'] = 'Off'`

`unless` seems like the wrong choice to control that setting from being exposed.


Cara,faz um pull request este arquivo.


The `if` buys us nothing here.


:+1: I've merged this.


r+


not quite getting that to work:

    src/jit/emit_x64.dasc:1375: error: bad operand mode in `lea rq,i?':
            | lea TMP6, MVMObjectStooge:TMP6->data;

I decided to write MVMObjectStooge instead of MVMObject because MVMObject doesn't come with the data part ...


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


I've never seen this done like this before. I would have thought that iterating over the string via indexer would be faster, but I'm not sure as you are getting the unicode information once (wonder what this code does with non unicode values). Is this what [Char.IsUppercase](https://msdn.microsoft.com/en-us/library/system.char.isupper%28v=vs.110%29.aspx) does?


i'll refactor that, aye


87727128_GridViewInheritance (macumber) - i386-Windows-7-VisualStudio-12: OK (2182 of 2195 tests passed)

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


Did all the proposed changes, if there would be a more elegant way, to share the code for the requests, please describe and will adopt.


does this also need something other than add? because this one is the one that actually seems to end up wrong on the callee side.


> Tock to v8.3.0

So it ticks for small increments and tocks for larger version changes?


r=jdm


Stylé


saw approval from jdm
at https://github.com/glennw/servo/commit/3fb3e66adaa1351684f363389b017323e504287c


merging glennw/servo/iframe-navigate = 3fb3e66a into auto


glennw/servo/iframe-navigate = 3fb3e66a merged ok, testing candidate = 755581ff


Line 27 prevents the build from completing and exists with an `error code of 1 in package : template`.  It effects all template builds (Fedora and Debian).

I am not exactly sure what you were attempting to do here or I would have fixed it myself.  For now I just commented out the line so I can do template builds.


use the gnuplot scripts to generate frames and
`convert -delay 10 -loop 0 plot*.png animation.gif `
to generate the video


I have the tests for width and height finished - let me know if you find anything else "crazy" - i had just installed my git repository when you got the trashed line ends. Got it corrected now.


Ale dorzucasz atrybuty data - nie do końca wiem na czym ma polegać różnica.


Haha... The BnP code is so ugly it's almost a joke xD ...


I think this one is a duplicate. This information is now located in both zfs send AND zfs receive.


what is `features`?
don't see it defined anywhere

Also, you need a method that can evaluate a trip or a set of trips using the model and return the utility function for the trips. Since each model is the only thing that knows about the features that it uses, this function really can't be put elsewhere.

Also, since we may have many potential models, we should have a module with a utility factory, and this should be the `SimpleCostTimeModel` or something like that.


can this key pls be funnier?


Indeed I should use a proper "if" here...


How about `init` blocks, should they not be separated with newlines?


Grimm Any chance you will release your full set of files? i would have no problem leaving your/others names  on all the files would obviously just want to put our billboards and things


Non portable!? pexit() is a JtR function.


or i could add a comment


;-) I know that feeling


:+1:

Edge case if user clicks on refresh button while the options to show and hide apps is present, the overlays and the app is hidden. Making a note to address later.


thanks

On Tue, Mar 17, 2015 at 6:15 PM, shmuelr <notifications@github.com> wrote:

> Good catch. I missed that in class.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shmuelr/touro-mco152/commit/47b717706d40ad9054136af6822251589d58851a#commitcomment-10247945>
> .
>


random comment


This is not the recommended manner to deal with this. Instead add an else {} condition for when there are no errors


I like the lambda!
Very elegant!


```sql
ALTER TABLE `judge_commentmptt` ADD COLUMN `hidden` bool NOT NULL DEFAULT '0' AFTER `body`;
```


do we call for this file anywhere from a .dts?


Background: breaks the MPTT.


I bet you got them as non portable because you copied it to memdbg.c without including misc.h. I will put them back.


rax retry tests


HIIIIIIIII


:Looks good.
Allen


I tried using the `abstractmethod` decorator from ABC but then ran into metaclass inheritance issues.
We use this pattern of having subclasses override class variables in `XBlocks` and the `Modulestore` layer as it is a pythonic way of doing this.
However, I don't feel strongly about this so I went ahead and updated it to use a NotImplemented method design - to be consistent with how it's done in oauth_exchange.


Uppercase/lowercase might break mingw. Safely presume that mingw is using lowercase. (if that's even a target for mono).


I think we should prefer to find another way to achieve this if possible, since if I interpret the intention of `SDL_SetHint` correctly, those are supposed to be specified by the user, and not internally by the system. Reading here: http://wiki.libsdl.org/SDL_HINT_VIDEO_MINIMIZE_ON_FOCUS_LOSS#Default , it looks like the default for this should be `1`. @Daft-Freak / @urkle, what do you guys think?


I'm not sure this is the correct behavior


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png)


Shouldn't it be multiplication?  The quantity we need to find here is the mass flux of air.  If the fuel fraction is ff, then the air fraction is (1-ff).  The total mass flux is mdot, so the mass flux of air should be mdot*(1-ff)


Well done! :+1: Keep rocking... for free...


saw approval from aberghage
at https://github.com/basho/riak_api/commit/14f28bab4b2cfa79bd15741768ef544a19ba4e6d


merging basho/riak_api/rebar/zl/update-riak-pb-in-rebar-config = 14f28bab into borshop-integration-85-rebar/zl/update-riak-pb-in-rebar-config


basho/riak_api/rebar/zl/update-riak-pb-in-rebar-config = 14f28bab merged ok, testing candidate = 2781e667


Oppps three is an error in this commit, this should be applied on line 61. I'll fix it tomorrow


Thanks for that!


yes just ready for this damned merge to be done,,,

1 more repo.


The keyboard shortcut was not set by default but I could simply change that in the settings and it works very fine, nice work :heart:

![screen shot 2015-03-17 at 23 37 56](https://cloud.githubusercontent.com/assets/1086961/6699059/b2b3e084-ccfe-11e4-9bc1-017c54c4eab2.png)


super-staff?


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png)


``do-while`` looks nicer since you need to perform this at least once.


Trzymaj się języka angielskiego przy definiowaniu zmiennych i funkcji. Ta linijka fajnie pokazuje jak dziwnie może wyglądać mieszanie.


Actually I want to use lower case attribute names, but lower case names are already reserved by PythonLayer.


I do not see FastStack implementation and sign of iterating from newly added to older elements , it just iterates over ArrayList from begging to end, so usage "descendingIterator" might NOT be the same.


Tutaj nie musisz definiować zmiennej `bool`. Wystarczy, że zwrócisz wartość od razu.


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/cb845d34cc4d2ebb679aedae335643b6714649c9/app.js)


Cool. This is good. I like it. We practically have our REST interface documentation without having to write volumes. In any case, once we start with this version, it is fairly easy to write a self-generator to compile this into HTML pages to be made part of MP documentation. E.g., something along the Jekyll kind of thing, though we probably don't need to go to the full blogging capabilities like Jekyll does.


Cool.

So just as a note: If you use the numbers you'd have to check the container count manually. I think we should leave that job to haproxy.


Oops, this bit wasn't supposed to make it into the commit. It's probably not necessary.


It supposed to be `save_private` not `save_priv`, as well as `upload_public`, not `upload_pub`, as you mentioned with your final decision? I don't think three characters makes a very big difference in terms of typing YAML, but makes things less cryptic.

Could you please expand the parameters?


test test


Definitely forgot to put a better message for this and cdd6e05eb5b3752f98c99941d1f5c66dc0cb843d before merging the pull request. My bad.

This commit implements a different implementation of `BaseController.parse_params` and fixes the tests accordingly. cdd6e05eb5b3752f98c99941d1f5c66dc0cb843d just fixes some minor test errors.


See commit comments at e5cdb8e938be20b9b67d5e47b44331036bcec29a


this needs to be `from elasticsearch_dsl.connections import connections` (I am not happy about it so it might change in the future, but for now that's what the import path is)


This hunk wasn't supposed to make it in either! Sorry for the mess.


looking good, but changes to max_7219 example are missing


I am sure line 1142 is a typo!


This is an additional refactoring for `BaseController.parse_params`, pushed too early into master. See comments at e5cdb8e938be20b9b67d5e47b44331036bcec29a


OH DAMN


(for the future)

things like this can be configured on `OpenBazaarContext`, and when clients do an initial handshake they can exchange networking settings, wether or not they want to talk using zlib and what not. Say there's a store that has lots of requests, a fat networking pipe, but it's getting bogged down processing too many requests, it could drop the zlib compression and free up some cpu.


Hi,

why those modifications ? I don't understand, could you explain me.


Oka :)  perdón :P
El mar 17, 2015 4:35 p.m., "Hugo Ruscitti" <notifications@github.com>
escribió:

> si, aunque suene raro si, es super común. por ejemplo jquery [1] o
> bootstrap [2] lo hacen
> así, tienen un directorio build o dist con la última versión compilada, y
> lo actualizan a cada rato, es lo mas práctico para acceder a la última
> versión.
>
> Lo del ver muchas lineas en el git diff es un bajón, pero se soluciona
> usando un cliente de git o haciendo diff solo de los archivos puntuales.
>
> 1 - https://github.com/jquery/jquery/tree/master/build
> 2 - https://github.com/twbs/bootstrap/tree/master/dist
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Program-AR/pilasweb/commit/1be8f322ab08d0844ed9ab8a6d3356a26ce52061#commitcomment-10244894>
> .
>


I've yet to test this but won't this break in project setups that situate vlad_custom/ *beside* Vlad itself? For example, this is my setup:

```
nosferatu/
├── vlad/
│   ├── vlad/
│   ├── Vagrantfile
│   ├── .git/
│   └── [vlad stuff...]
├── vlad_aux/
├── vlad_custom/
│   ├── .git/
│   └── [tasks/ etc...]
├── settings/
│   ├── nosferatu.aliases.drushrc.php
│   ├── vlad_settings.yml
│   └── vlad_custom_settings.yml
└── docroot/
    ├── .git/
    └── [drupal codebase...]
```


Did you test my suggestion? Did it work?


better example here I think is:

    # creating a connection named 'production'
    connections.create_connection('production', hosts=['localhost'], timeout=20)
    # now we can use the alias
    s = Search(using='production')
    # override the search to explicitly use a given connection
    s = s.using(Elasticsearch(hosts=['localhost']))

Also please use python's comments (starting with `#`, not `//`) for better rendering


Assuming I need to register?


TeamCity Helios :: Helios PR Build [Build 18](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=272&buildTypeId=Helios_HeliosPrBuild&guest=1) is now running


all tests pass:
success: http://buildbot.bos1/builders/test-riak_api/builds/96


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png)


This is how it's done :+1:
"***Compress and then encrypt***"

Easier for compression algorithm to find patterns, the result of the compression adds entropy which is good for encryption.


This is causing the build to be very slow (and also refreshing the context when loading the app). We need to set this DAO via the moduleApplicationContext.xml.


No problems.  Work perfect now.

By the way the applauncher is really cool.
Thanks
On 17 Mar 2015 17:16, "Ken Moore" <notifications@github.com> wrote:

> Sorry about that!
> Apparently I forgot to add the 2 new files to GIT. I just added/pushed
> those in now. Just re-pull the sources and you should have them now.
>
> Thanks for pointing that out!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pcbsd/lumina/commit/75202336d3e110c46ed1c7f916c4946deeaf8326#commitcomment-10240647>
> .
>


1) unconditionally setting the root variable will probably cause issues with the encrypt hook
2) assigning a variable recursively can lead to timing issues.


Why move group comment to a line comment?


TeamCity Helios :: Helios PR Build [Build 18](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=272&buildTypeId=Helios_HeliosPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:57


Hi,

why you have removed my PR: https://github.com/david63/david63-cookiepolicy/pull/3 ?


wonder if there's `data.encode('lzmash')`

LZMA > BZIP2 > GZIP (zlib)


Wow, this helps a ton! What was the trick? It doesn't seem obvious from the diff...


test 4


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


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/59a68654-f7d0-483d-a503-59f80e90f2d9/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2815ce43-4055-4bd9-af03-52355eeea82e/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/27d847f5-9d16-49aa-9410-dca636ffc96c/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/5fc4f09e-b81d-4aea-be1f-3a8c32c14479/Chrome/elements.button.700px.baseline.png)


nvm, lzma, would be too slow


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/b2a6cbf1-51a7-4b28-818e-a6f20b471406/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/f44b0db8-209c-486f-b833-d92dca2852ad/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/abdcdeb5-09da-41aa-abfa-9694a1466016/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/216fddf5-8491-4c95-942d-99c20a5eab54/Chrome/elements.button.700px.baseline.png)


The README should say what is the default variable name and where it can be set. Can we add the issue tab to this repo or a link to JIRA, I'd like to propose a new feature.


@UlysseM we should probably come up with a way to track version numbers.


Theoretically I totally agree. In this case Syncano will always use APIClient and this class is treated as sealed, so there was no risk of subclassing it.


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/2b61af1f-8f39-45c1-b742-7a17475341f1/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/84f7f0f7-6616-44be-b61b-557309b0617b/Chrome/elements.button.700px.baseline.png)


Diffs found in 1 browser(s): Chrome

Chrome

![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.700px.png)
![http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png](http://visualdiff.ngrok.com/api/diff/a5726eb9-0d32-44cc-8118-079ca584c348/Chrome/elements.button.700px.baseline.png)


Typo at `Encryptes` -> `Encrypts`


Sure


I am going to change it to SyncanoNullSafe


Alas not for the defines as you now reference c twice, so isascii(x++) does terrible things.

SDCC supports "inline" but cc65 does not so you can probably inline them fine in SDCC and gcc but not cc65 - which is fine I think. For 68000 we are going to want the table driven approach, for SDCC inline and for cc65 out of line so we can't win on a single implementation

Also your isalpha looks very broken.

That aside  I think it needs a new clean patch to follow wtf is going on.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1038
success: http://build.servo.org/builders/linux2/builds/1038
success: http://build.servo.org/builders/android/builds/1072
success: http://build.servo.org/builders/mac1/builds/1254
success: http://build.servo.org/builders/mac2/builds/1254
success: http://build.servo.org/builders/gonk/builds/225


Fully agree. Enums, consts and dedicated SyncanoException (as well as SyncanoError) will be added


Isn't refactoring `gulp copy` another PR ;)


Great eye. Just overlooked it.


2/2

Straight ballin~


It's more a matter of taste but I agree that this approach may suit more users in the future than getter methods. Will be changed in the next commit.


Please fork next time.

2/2


fast-forwarding develop to borshop-integration-85-rebar/zl/update-riak-pb-in-rebar-config = 2781e667


fast-forwarding master to auto = 755581ff


Of course, Mantle is going to replace this temporary code.


Version `0` not allowed?


Please add some usage documentation. Thanks :)


This should be deleted, there is no view for the create action.


Nice!

2/2


At huge word sizes, the new `sbvTestBit'` is much faster than `sbvTestBit` from the SBV library, which does a mask and compare all at the huge bit size.


See FS#1174


This file never gets rendered so it should be deleted.


Yeah I'm flying blind. I'll have to read up more on it. Can you point me to some literature?


Tab characters T______T


Great!

2/2


Tab characters T_____T


Multiline search really necessary?


The i18n propagation script is going to turn it into an end-of-line comment in every other file (and I was a bit surprised that it kept it at all), so I wanted to mirror what the Diplomats will see in en.coffee as well.


@shankari `features` is defined here


Should not be use "using namespace" in example


Awesome possum

2/2


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.15](http://teamcity.monogame.net/viewLog.html?buildId=5722&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 452, ignored: 6 Build time: 00:14:17


To be consistent with the `ActiveTab` property, I guess this could use `ActiveNotebook` instead of `DockNotebook.ActiveNotebook`?


help please my number is~ 10128 81758


Ok, looks good to me :)


Is there a reason to limit this to a setting?


tout vas etre centré par default ;) ,plus que besoin des mettres les elements


fin les classes avec les elements


Ah, right. I remember running into something similar where the `Integer` created by a mask operation wound up running into the memory limit for gmp, crashing the whole program. I think that was from one of @weaversa's diabolical examples


LGTM!


Defaults from resharper, actually.
Honestly after you get used to the extra space it's easier to read stuff. I find it hard to look at "unnecessary" compressed code.
Since I'm the one maintaining it, I'm not going to undo auto-formatting just to make others happy. That's extra work on my part for what makes no difference, in the end.
A JavaScript coder would say c# in general has too much white space.
Styles are just that - styles. Nothing that is legal is "unnecessary," so long as the people maintaining code are happy with it. :)

-----Original Message-----
From: "Michael Zanca" <notifications@github.com>
Sent: 3/17/2015 5:12 PM
To: "dodexahedron/EssentialsPlugin" <EssentialsPlugin@noreply.github.com>
Cc: "dodexahedron" <brandon@dodecatec.com>
Subject: Re: [EssentialsPlugin] Whitespace only. (7936050)

that's quite a strange coding style you got there. why so much unnecessary whitespace?
—
Reply to this email directly or view it on GitHub.


Yeah, 1080p isn't working for stock camera. However, it works on several other camera apps like A Better Camera or if you use the old camera/gallery app from jellybean.


maybe use variable here to avoid repeating the same code?
```
QString windowTitle = f->getFriendWidget()->getName();
if(!f->getFriendWidget()->getStatusString().isNull())
windowTitle += "(" + f->getFriendWidget()->getStatusString() + ")";
setWindowTitle(windowTitle);
```


I think the comment should say right, not left?


Yeah, I think allocating the masks is the big time/memory waster here. Unpacking a million-bit word into a list of bits means allocating a million million-bit masks, so it adds up.


imo, a better version of this is:

```
onInsert: function () {

}.on('didInsertElement')
```


I would recommend to use specs2 library. You could rewrite assert as:
result must be empty // executable Scala code
Which is more readable from my point of view.


Gang Hideout 3, not 2...


saw approval from aberghage
at https://github.com/basho/riak-erlang-client/commit/bfc56235ef0f43326b8a6f7f418f5a059d4cb102


merging basho/riak-erlang-client/feature/zl/timeout-for-search-index-create = bfc56235 into borshop-integration-207-feature/zl/timeout-for-search-index-create


basho/riak-erlang-client/feature/zl/timeout-for-search-index-create = bfc56235 merged ok, testing candidate = ec12b2d6


flexibility, not necessary at the moment.
Things like that are always optional in networking protocols.

Famiilar with the HTTP header `Accept-Encoding: gzip, deflate`, not all browsers support it, not all requests need to be gzipped.


I guess that "object method value" style is good for some purposes like English-like DSLs, but in this case it makes the code more unreadable. I would avoid it.


not all messages need to be compressed and zipped, for instance, pings, byes


Alrighty just checking.


@puchi- mysql_real_escape_string wird hier nicht benötigt. Das Prepared Statement escaped automatisch. Außerdem hat mysqli eine eigene escape Funktion da mysql ohne i veraltet ist die funktioniert dann so: $mysqli->real_escape_string($username) (aber auch die wird hier nicht benötigt)


You define a class with a single method that looks like... a function. I wonder can you just use a function instead?


Psp emulator in javascript. I never though it was possible. Keep up awesome job !


Why are you making one CPU work harder than all the others? You could just use all cores and lower the frequency as much as possible to share the load which would be a lot better...


TeamCity Helios :: Helios Unstable Latest [Build 6](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=273&buildTypeId=Helios_HeliosUnstableLatest) is now running


it'd be nice to do this all in one go like this:

```
module.exports = {
  solveSudoku: solveSudoku,
  parseBoard: parseBoard,
  ...
}
```
http://stackoverflow.com/questions/16383795/difference-between-module-exports-and-exports-in-the-commonjs-module-system


Updated!


It's more idiomatic to use pattern matching here:
experiments match {
  case Nil => result
  case _ => ...
}


lol javadoc. idk anybody that writes js comments like this, but it's fine, and readable


:smile:  updated!


TeamCity Helios :: Helios Unstable Latest [Build 6](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=273&buildTypeId=Helios_HeliosUnstableLatest) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (1).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies) in D:\code\fake\src\app\FakeLib\UnitTest\NUnit\Sequential.fs:line 45   at FSI_0001.Build.clo@92-7.Invoke(Unit _arg5) in D:\Buil... Build time: 00:01:11
### Failed tests
```
Helios.MultiNodeTests.TcpClientServerReplyTests.ClientServerReplyTests.Should_receive_reply_from_server_MAX_200b_messages: <no details avaliable>

```


TeamCity Helios :: Helios PR Build [Build 19](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=274&buildTypeId=Helios_HeliosPrBuild&guest=1) is now running


@ddmler Danke für die Info!


TeamCity Helios :: Helios PR Build [Build 19](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=274&buildTypeId=Helios_HeliosPrBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 107, ignored: 19 Build time: 00:01:04


maybe rename something like `checkRowForValue` would be a little clearer


`checkColForValue`?


[Deployment available online](https://s.brightspace.com/apps/ipa/dev/df72b0a147ad261ad334b9a9092798112e7095aa/app.js)


Weired, can't see that on my local :confused:


updated!


This should be:

```
    if(f->getEventFlag() == 1)
        return tr("New message");
    else if (status == Status::Online)
        return tr("Online");
    else if (status == Status::Away)
        return tr("Away");
    else if (status == Status::Busy)
        return tr("Busy");
    else if (status == Status::Offline)
        return tr("Offline");
```


The result value looks very complex and you need to carry knowledge about what this complex data structure means through your code. Would it make sense to create a separate class for result and move code for accessing various parts of the data structure to accessor methods? To make it functional style you could add separate builder class/data-structure for it.


The whole logic was flawed, I've updatd the way we get docker_id's and removing them from the cached list.


I have changed it to shift so that you can see the errors when you run the .rb file (pasted below for you).  Any thoughts?  Again, the file runs fine when using 'drop(1)' instead of shift.  How odd/strange.

refresher_spec.rb:51: syntax error, unexpected keyword_end, expecting ')'
refresher_spec.rb:84: syntax error, unexpected end-of-input, expecting keyword_end


rename `checkValid` maybe?


I would avoid this "object method" because it  impairs readability in this case.


this whole thing could just be:

```
return checkRow(...) && checkColumn(...) && checkZone(...)
```

no need to "return true" and "return false" in a conditional


Nit: Add a newline at the end of the file.


some tests failed:
failure: http://buildbot.bos1/builders/erlang-client-test/builds/109


I'd be into that. You have to make some assumptions since level-name can only take relative paths, but I don't see that as a problem.
I'd also like to develop a pattern that doesn't involve a full download of the jar on every run. Running docker stop and start are fine unless you want to pass multiple different arguments in succession.
My eventual goal, by the way, is to have something that resembles a CI build pipeline. I'm brand new to minecraft editing, but I assume there are ways to programmatically generate structures and terrains, and this would be building whatever that script is and storing the result in a data container that could then be pushed to S3, or deployed onto a new server.
I'm also very interested in what you're trying to accomplish with your minecraft go client stuff...

On Tue, Mar 17, 2015 at 1:15 PM -0700, "Geoff Bourne" <notifications@github.com> wrote:

Ah, nice technique to separate the world save data from the jar, properties, and other bits. Perhaps we should make /minecraft-data a standard part of the minecraft-server image itself.

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


Nit: Remove these blank lines.


Ok so `features` is a matrix that includes rows from the trips and the alternatives? But the parameters to regression.fit are the feature matrix and the result vector, right? So how can you pass parameters of `features` and `trip_features`? Don't you need a result vector instead?

http://scikit-learn.org/stable/modules/generated/sklearn.linear_model.LogisticRegression.html#sklearn.linear_model.LogisticRegression.fit


Nit: Remove one of these blank lines.


why do we need the return model line here ?


It is a wrong place, it should be defined start routine, that check existens of a table


It's more idiomatic to use pattern matching here.


If you add "[skip ci]" to the commit message, travis will ignore the commit


Add newline at EOF.


Remove this?


LGTM. I mentioned some small formatting crud to clean up -- I figure you're okay with that since you removed all of the trailing whitespace :)