oops. k, switching. also made a stale comment - totally wrong =)

On Thu, Apr 23, 2015 at 2:30 PM, Jordan S Read <notifications@github.com>
wrote:

> I think you are looking at a stale commit
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jread-usgs/streamMetabolizer/commit/42c9ac967dcef6b129b7a4fa1124f99a3cba9e07#commitcomment-10875676>
> .
>

--
Alison Appling
Postdoctoral Fellow, USGS Powell Center
USGS Center for Integrated Data Analytics and University of Wisconsin -
Madison
Phone: (608) 821-3927


No way to add extra column to allow movement into unique multi-column key at some point?


It's better to use `base::ASCIIToUTF16` instead of constructing `base::string16` from `L""`.


delete CNAME


@func0der yea you're right. I've reverted it back.


deleting everything


probably better to not count the compilation time here by using [compile_jit()](http://halide-lang.org/docs/class_halide_1_1_func.html#a4bb020bfc9d7e83325dc59b271513dfc) before


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/320


It looks like this is assuming that `partitionFutures` is a power of 2. Add an assertion if so


good point - how do you call the code after? Another call to realize() or by using the function pointer that compile_jit() returns?


well. what you did: you clicked on the commit link in the PR. that brought you here (into the commit in my fork)

what is better: in the PR click on the "Files changed" Tab and comment on the changes there. those comments will become a part of the PR and can be seen there even after i update the commit like i just did. the comments here, are only visible for those who know where to find this commit.


What's the error message that you're getting? Do you have an example that I can try to compile on my machine?


rghj,


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


Current functionality of the puppet provisioner generating keys/certs and then cleaning up after doesn't really work in our environment. Added CLI argument that changes nothing by default, but allows user to turn off the certificate creation/cleanup.


Pass the vector by reference please.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/59772335


`realize()` basically memoizes `compile_jit()`, so running it once (for small enough pipelines) before collecting timings should be sufficient; or use the function pointer that `compile_jit()` returns.


I don't understand the difference sorry hehe, anyway glad you did the tight thing i hope.

and sorry!


@thomasgohard It's tricky to debug or understand by simply looking at the source code because essentially, part of the CSS only applies to browsers with native `details` `summary` implementations and part of the CSS only applies to browsers which require the polyfill, so whichever inspector you're looking at the code through, there's always some part of the code which is inexplicably pointless, unless you know that it's for another browser.

Too bad CSS doesn't support things like C's preprocessor macros. In a way though, this is better. It's actual “portable” code, not just a bunch of browser-specific implementations.


You can also just do:

```
bal.dialers = make([]*dialer, 0, 0)
bal.trusted = make([]*dialer, 0, 0)
```

To avoid special handling of a slice of length 1 with nothing in it.


Why is this patch needed at all? Is there an upstream bug reported for it?


Actually it looks like `compile_jit()` then `realize()` works: https://github.com/halide/Halide/wiki/Minimal-GPU-example


I didn't see harm in marking `init` and `new` as `NS_UNAVAILABLE` because I figured that it would help other collaborators in case they wanted to subclass `ORKObserver`. But as per you request I just removed those: this internal class is simple enough. I also added *nullability* notation.

Regarding `new NS_UNAVAILABLE`, I contend that every time that a class has `init NS_UNAVAILABLE`, it should have `new` annotated the same. If you only annotate `init`, the compiler will forbid using that initializer directly but will allow using `new`. That was the case for `ORKRecorder` while I was adding the *identifier*, and some tests were creating some wrong recorders using `new`. These errors became immediately spottable after adding `new NS_UNAVAILABLE`.


But should we really count clay as an ore? Sure it spawns in veins, but it's not something I would think of when talking about an ore.


Yepp, roi->x and roi->y are set to zero there. Of course we could do the same in finalscale which would avoid the specific wrapper functions. OTOH the reason for that fiddling is not obvious at all when reading the code. Better we keep the wrappers and add some reasonable comment there.


No tak tylko, że to co częściej robi się w popupie niekoniecznie musi być wygodne.


could have used
empty($entity->name) ? variable_get('anonymous', t('Anonymous')) : $entity->name;


I know it's not introduced by this PR, but converting the image format twice is pretty dumb.


fast-forwarding master to auto = 19b96146


hi how to add new topic plz help me


Can someone do this too, pretty please? :)


its all about that you won't be able to find your comment on the pull request anymore

but no problem. ;)


This `if` seems unnecessary


rax retry tests


I LOVE THIS


yes.


Well, it worked for me (IWFM) on the latest Yosemite with the latest Xcode.
I'll try it on another Yosemite machine, then maybe on Mavericks.
Does the old version work on your machine?


Did you need to update this as well?


you broke it ;(


First, querySelectorAll might not be well supported by IE8.
Could you please send me  a test case to allow me to study how to fix this issue for complex situations (embedded repeats,...)?


Is this the thing you were waiting for? :)

I'm a fan of expressing an idea and getting it working and then breaking it up into more maintainable chunks.


You can use shorter class name


Needs change to div, will still need id for totalDonations.js. Will this break document.getElementById("total").innerHTML = ...; ?


Hehe, just in a good mood.


No love for windows apparently.


88737258-EPW-Ruby-Access (jasondegraw) - 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)


Bit hacky. Would be better to adjust SS_* defines.


Sure thing, I would be greatful. Let me know when you have something for me to try out.


return is missing?


consistent spacing in each of these blocks - first two have extra space, the third doesn't :] not a big deal but just pointing it out


Wonder if this should be `instanceRepo` instead of `domainRepo`.


@chrisseaton Good catch, I updated the help.


The indentation is wrong here now.


Passing a boolean for that doesn't sound like an improvement to me. It is much more logic to call getGameMap()->isServerGameMap() than having to pass a boolean. And it would allow newcomers to call this function with the wrong parameter. Moreover, as a newcomer, I wouldn't understand why it is done like that for isClaimed and not for everything else. I understand why you did that but IMHO, unit tests are not important enough to justify such a hack.


loop is still continued on a failure?


Interested in this function, so have been trying to read through it. I think this line is equivalent to just using `scale`
```r
sx=scale(x, center = T, scale = F)
# scale leaves an attribute to indicate the scale factor for each column
all.equal(xc, sx, check.attributes=F)
```


I think any block that converts to different items when broken qualifies as an ore. At least good enough for me.


Your <body> tag should be completely empty according to the requirement.


The build team has reviewed. Please let us know if this needs to be back-ported.


Lol


Memory leak, member_ids is not getting cleaned up.


Thanks for cleaning my mess


 I get something like that:
! Package inputenc Error: Unicode char \u8:ðºð¾ not set up for use with LaTeX.

And then the result is that the line with non-manuals is there (also in Russian), but not the glosses. Attached is the .tex file and the resulting .pdf

Четверг, 23 апреля 2015, 12:38 -07:00 от Jonathan Keane <notifications@github.com>:
>What's the error message that you're getting? Do you have an example that I can try to compile on my machine?
>—
>Reply to this email directly or  view it on GitHub .


@ntdb This is 'update_all_revisions', but it does 'next' if there are no revisions.

Should this instead be:
       next if c.students.empty? && c.revisions.empty?


наибольший → наименьший


50% dos issues #2 e #4


ada ji sedikit sedikit ku tau kak. Cuma ku lupa mi kalau yang saya upload belum pi diperbaiki selection na. Karena kalau saya pake yang ini, cuma "if" yang pertama ji yang tampil, selebih na malah tidak mau tampil sama sekali. Yang saya edit terakhir harusna ada "if", "else if", sama "else" na kak, cuma entah saya lupa atau apa, malah ter upload yang ini kak.


Hello World!


Given an array and a value, remove all instances of that > value in place and return the new length.

The order of elements can be changed. It doesn't matter what you leave beyond the new length.


cool


See 2.2 here:
http://www.php-fig.org/psr/psr-2/
I assume the idea is to make sure no code accidentally ends up outside a php tag.

Also, the full <?php should be used (just a [compatibility thing](http://php.net/manual/en/ini.core.php#ini.short-open-tag)); see 2.1 here:
http://www.php-fig.org/psr/psr-1/


Might be worth adding a comment to that effect.  I'm happy to do it unless you want to.  If you take it out, I'm pretty sure a test will fail.


Last time I tried to push to Travis CI with 1.8, `nvm` failed to install it.


This method seems to be accomplishing what it sets out to do clearly enough, but this a good example of the kind of code that quickly clutters up the controllers and is best to be moved elsewhere. You might instead write a method on the Post model that does this work, that way the controller can just pass the data through. Another good option, especially if the model already had lots of code, would be do make a new file/class that had the responsibility of parsing the tag string.

Another thing to consider is what will happen if some of the tags save but then one throws an exception? Would you want to have the post created with some of the tags or would you want to have the whole process aborted? Rails gives you a nice way to handle this called a transaction. You call it like this:
```
ActiveRecord::Base.transaction do
   #call a method inside here that runs your code
end
```

If there is an error thrown, all database actions that happened inside the transaction will be rolled back. It's a good practice to use when saving multiple objects in one request.


yes, it is not a good idea, pls close this pr


My next endeavor  will have to be learning css programming huh


change


Hilarious.


Basically this is similar to the sample function provided by instructur


Doing small commits like this one will make it much easier to use git to make changes. Make lots of commits. You can always squash them later.


cool ;-)


We should not be leaking JCR exception outside of this interface.


Done


We should not be using jcr.Nodes outside of fcrepo-kernel.


버그 계속 찾고 있습니다! ㅋㅋ


REVIEWのTEST
REVIEWしました。


I double check this, it is need for android only

compile in src folder

success, opensuse 11.4

```
gcc -DHAVE_CONFIG_H -I. -I..  -I../libhttpd/ -DSYSCONFDIR='"/usr/local/etc"' -Wall -Wextra -Wno-unused-parameter  -MT commandline.o -MD -MP -MF .deps/commandline.Tpo -c -o commandline.o commandline.c
```

fail, android ndk

```
arm-linux-androideabi-gcc -DHAVE_CONFIG_H -I. -I..  -I../libhttpd/ -DSYSCONFDIR='"/usr/local/etc"' -Wall -Wextra -Wno-unused-parameter -MT commandline.o -MD -MP -MF .deps/commandline.Tpo -c -o commandline.o commandline.c
```

```
In file included from commandline.c:35:0:
conf.h:84:1: error: unknown type name 'pthread_mutex_t'
```

btw, ```HAVE_LIBPTHREAD``` never used.


LGTM.


@davertmik, @burzum: what comes to my mind (also because of https://github.com/pscheit/Robo/commit/49656ff0d9b9e0686a0bbd3382d314207e638669, #133, #137):

Shouldn't we always use ProcessUtils::escapeArgument() directly in the `option` and `args` methods?
Is there a case where this breaks some shell call?

All the other manual escaping in all tasks need to be removed then of course.


Design: ideally create a small api with db (mongo), accept POST  GET  JSON - > {id: name,  data: htmlContent }


Hmmmm, when was that? we have an 1.8.1, but not 1.8.0.


Changes were done to group each examination. The grouping was based on collapse view.


It was like 2 days ago when 1.8.0 was out. I did just try 1.8 on a different module and it looks like Travis CI is working fine for it now.


What's actual guid in Win8 SDK?
I need to put this back to test all reflection GUIDs because some of MinGW implementations don't have this defined.


check out this method http://apidock.com/rails/v4.0.2/ActiveRecord/Relation/find_or_initialize_by

See the comment is Diederik's repo about a similar method https://github.com/dkroondijk/Blog/compare/peer_review#commitcomment-10860041


optimized and -flto'ed


@okamayana


> It was like 2 days ago when 1.8.0 was out

1.8.0 never existed, it'd be 1.8.1 :)

Weird though. ¯\_(ツ)_/¯


ドリアン太郎！
修正お願い致します


Here you go, @normanwn


Would you be willing to share a .jar build of this with me? I've been looking for a 1.8.3 version of this plugin.


Is this comment about this particular struct, or the whole FileInfoLoader thing?

Seems like walking the tree, hashing files, etc, is part of "isolate", which then generates the "isolated" file. Then isolatedclient should be responsible for uploading the "isolated" file and its contents to the server?


Carro baseado no ![Enduro](http://imagem.band.com.br/FOT_FT1_25908.jpg)


Gotcha. I just follow https://github.com/iojs/io.js/blob/master/CHANGELOG.md and 1.8.0 was in there like 5 days ago and so tried to use it on Travis CI and wasn't successful and gave up on 1.8.


ドリアンだ！


@lrytz This is an ongoing issue that affects a lot more projects than just this, and is currently being sorted out.  I have a lot of sbt plugins to migrate to bintray, the process is tedious and time consuming, and I have a very short deadline to do so, and no time budgeted to actually do this work.  So I'm currently waiting for the sbt bintray plugin to be fixed, so that I can fix this once, rather than having to implement a work around or back out the changes in all the projects I've updated so far, and then fix yet again.  Please be patient while this happens.


A bit of code formatting can make a big class like your BlogsController more readable. Readability counts for a lot when you need to go back later and change something. Try adding a line between the methods.


I wouldn't bother with the link; people can look it up themselves if they want to (and it also means that if someone is using a sufficiently old version of Haddock then this won't parse properly).


thanks :+1:


Please leave the parentheses in around `Graph gr`!


Is this `ignore:` definition made redundant by the negated statement `'!tests/specs/libs/*.js'` above?


very nice


Can you hold off on a new release, there a bigger issues with this change. I need time to sort them all out.


Please indent the `-- ^` so that it lines up with the one for the first `CFun` (because I think it looks prettier ;-)


@matthewd oh wow, nice find :), I can't even


Please indent the subsequent lines of documentation; i.e.

```.haskell
-- | Many functions take a list of nodes to visit as an explicit argument.
--   'fixNodes' is a convenience function that adds all the nodes present in a
--   graph as that list.
```
(and do this for the other multi-line comments)


Here's what's on my machine (Win8.1 SDK):
````
interface DECLSPEC_UUID("8d536ca1-0cca-4956-a837-786963755584") ID3D11ShaderReflection;
DEFINE_GUID(IID_ID3D11ShaderReflection,0x8d536ca1, 0x0cca, 0x4956, 0xa8, 0x37, 0x78, 0x69, 0x63, 0x75, 0x55, 0x84);
````


I wanted to point out that it might not make sense to have ship and hit as parameters for this object. The reason being that whatever parameters you put in an object definition should really only be constructure values, or things that you absolutely need when creating a new object.  Well, in the code you are setting shipobject as a ship through a prototype function (addShip) thus making the constructor parameter un-used. Also when creating a new ship it probably won't be hit - so you can omit that parameter as well - and then set it later via myShip.hit = true


竜は強すぎるので、もっと弱いのに変更してくださいー


What does the 2nd this.ship; statement do?


@JoeVogel AEeeouAeeeouu! LOL!


android-blog/android-developers/Android Support Library 22.1/readme.md

去掉这句吧


@koikeiko レビューしました！ご確認くださいー


GroundDomainUpdate (sushobhit) - i386-Windows-7-VisualStudio-12: OK (1632 of 1639 tests passed)

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


will fix that. thx.


最新版的Android Support库在Support V4、AppCompat、Leanback、RecyclerView、Palette和 Renderscript库之中添加了很多有用的组件，当然内部也有一些变化。


它包含了很多向下兼容的类，使得向下兼容变得更容易。


Yeah the Listener interface is defined by this class though.  So you can check for specific push events.  I'd add a Listener method that includes the event and deprecate the old one...and see what kohsuke thinks.


rax retry tests


Is this feature to track history of appointment or encounter?  The application (and real life) allows encounters without appointments. This feature is of great interest because the application is not capturing data that will actually make the practice more productive.  From that perspective, appointment statuses are just the bookends - start and finish for the encounter.  By normalizing the database table and not relying on appointment status, you can make this feature very customizable without need for any software changes.  E.g. you can still get basic information as :
SELECT pid, encounter, MIN(date) AS time_in, MAX(date) AS time_out FROM patient_tracker WHERE encounter=?
GROUP BY pid, encounter
Your contributions are greatly appreciated.


Interesting way to solve this problem, I think we need to do something more like registering your parser to be added to the list of middleware that the pixi loader adds by default.

Otherwise multiple plugins wont be able to all inherit from the pixi loader and do this.


We should expose resource from `PIXI.loaders`, otherwise your package here will depend on resource-loader and duplicate that dependency.


tested and working


While you're here, please defgroup a customize group and make this part of the group.


Опечатка, вместо 'е' должно быть 'ё'. Теперь комнаты с маленькой "ё" не пускает >_<


This issue was due to me being in the process of figuring out type classes in Scala.  I probably should have left a comment saying that I was still unsure of some things.


Other than the nit, lgtm.


Your <body> tag should be completely empty according to the requirement.


Nice catch


dunno, haz 2 test


This breaks our plugin in our Windows 8.1 project. We just updated cordova internally and these changes were included in that. We have a plugin that has references added via the <framework> tag. These references are winmd files and require a dll to be in the same directory they reside in but this dll is not referenced in the jsproj. We were relying on the <asset> tag to put the dll in the location where it is needed like this <asset src="src/windows8/RBA_SDK.dll" target="../plugins/com.yourwaresoftware.ismp/RBA_SDK.dll"/> now that this change has been implemented we get the error "not located within project!" from common.js. Although this might not be the actual intended use of the <asset> tag we have not been able to find another way to get this file into the correct target folder. Can this "ERROR" be changed to a "WARNING" instead.


It seems that the game can not be played on mobile device.


Dynamic XPath evaluation should not be processed at this layer but just with an XPath evaluate() function.
BTW, Javascript part of XSLTForms is not yet integrating its own XPath expression compiler: this is currently performed with XSLT only.


You removed the deep copy of the original molecule. Is that intentional?


good call


This PanicAlertT is inconsistent with the PanicAlert below.  (Probably not much point translating, since the shader error message will be in English.)


nice find!


png->PNG please


Is there any introduction of this game? I’m not sure how to play it.


remove TestLogging


Today, i can only say, that happens in situation when we have repeated section with select1 in it.
For example if radio has name="foobar" during cloning, new radio also has name="foobar" so it will be selected and orginal radio will be unchecked. If name of new radio will be another when appendChild will be called, there will be everything ok.
Here is simplest example of this: http://jsfiddle.net/zp6haegu/1/


I disagree. Passing it as a boolean makes it much clearer that the function has different behaviour in server or client mode. Doing it the old way also introduces a dependency on the gamemap class where it's not needed, something that impacts compile times. I think it should be done like this in every function that has client/server differences, but changing this everywhere would be a bit much for one PR.


I guess it's for the sake of unit testing TileSet and simplifying TileContainer. Seems good to me.


We don't need to handle SS as a module. Just compile it as regular C file, like it's done for sock_srv.c for example.


There’s no way to start a new round. (Either a button or just start a new game will do.)

In mobile device, the boundary of the game area is out of sight/overflow… I can get the snake go out of the green area and come back.


Also for compile time.


@arfon @bkeepers Is that sufficient?


Normally, you can avoid using `!important` with specificity


Seems like this would make more sense as an internship method. Something like:

```
def students_by_interest_level(level)
  ratings.where(interest: level).map(&:student)
end
```

Or, the more single database query method:

```
def students_by_interest_level(level)
  students.joins(:ratings).where(ratings: { interest: level })
end
```


why the generic?


In the future please try to make the existing settings files backward- compatible with such changes. Otherwise I have to revise all of my work if I want to use an more recent version of PEcAn


:+1:

Do the unit tests run now?


There’re some problems with the replay button.
Mobile device: The touch event for replay is actually positioned right at the top of the replay button instead within the button.
Desktop device: It does not work at all. (Mobile comes first, so this one is less matter.)


It seems that I can not attack the enemy and the game can only be played on desktop device. I can not start a new game in mobile device.


needs isset instead of empty to let you set these to false or 0


corrigé a455601


argh, shouldn not search/replace version numbers but setup an automated task to do releases really.


I don't think that's a requirement for the homework.
https://docs.google.com/document/d/1IYopKyMarGSsJMB_i9nzyZYgGKi_vfYqltu96GshT60/edit#heading=h.2dpxkywqhfb2
And for this game we are going to use realTimeSimpleService.js, so we don't actally need to make body empty.


Yeah, growing from 0 (which I guess is the default if you just say `make([]*dialer)`?) is better for `bal.trusted`.  I understand the optimization in `bal.dialers` because it is known to have the same length as `dialers`, but `bal.trusted` will eventually have way less than that (with any luck recruiting give mode users).


// testing


The game right now is like a hurdled-version snake. Player needs to bypass
those red walls on map to reach the green 'reward'.

There are several different maps, each map has different walls.

The 'reward' will change position randomly after several seconds.

Regards,

2015-04-23 16:01 GMT-04:00 Youlong Li <notifications@github.com>:

> Is there any introduction of this game? I’m not sure how to play it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jz1371/PROJECT_Tank/commit/aecb43a345477bda9b8c1282cc433c8c651e587b#commitcomment-10876211>
> .
>


Actually, models that have unsaved changes can still be found as long as they're **persisted**. Change unsaved to unpersisted.


Sorry, I know about that, I thought the password login would override it.
On Apr 23, 2015 2:21 PM, "Patrick Steadman" <notifications@github.com>
wrote:

> Because my username is set as the git user on that computer...look up how
> to change the git user.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ptsteadman/baxter-teleoperation/commit/f835ec6b93fe76a0af693da1e515a5c74fc4c9d6#commitcomment-10874484>
> .
>


you should hide this :100:


Everything looks good to me.


Thank you.You are right, I found that out some months ago, but only have changed it when I came across it (like [here](https://github.com/zarquon42b/Morpho/blob/master/R/rot.proc.r#L6-L7)). As this package dates back to the dawn of my programming skills, this is probably not the only quirk you will find  I had mimicked that way of centering a matrix from another package.) ;)


How'd you get the links in the commit message?


Disable block build https://github.com/OptiPop/build/commit/5e9e7667e90532cc319e16e39234cb8ed2593db8 error is caused by it attempting to make an F2FS block which is not currently supported.


Should we mention using `after_commit` instead of `after_save` here, @rafaelfranca?


The code in read.settings takes care of this and will convert your existing pecan.xml file to this new code.


Долго думал, как назвать — aib.tNum, или aib.isThr. Так как это номер треда, и индикатор режима треда одновременно. Причем в качестве индикатора используется гораздо чаще. Назвал `aib.t` по аналогии с `aib.b`.


Why was this done?  The documentation still says that this should work:  https://github.com/AutoMapper/AutoMapper/wiki/Configuration#configuring-visibility

Without BindingFlags, there's no way to map non-public members.


Got it :)


This function seems to just replicate the behavior of uploadAssignment(), with the exception of being able to specify the name of the temporary intermediate file. I understand that this is used by the meteor UI, however, it's unclear why it is needed.

- The submit.submit() function will copy the temporary file to the vmchecker repo anyway, so the temporary file is not really useful.
- If the intended use of this call was to allow the arbitrary uploading of files, then why call submit.submit() on each call?


Thanks! :D
Probably it will be my next choice.
But now I need the project to be done ASAP, so JavaScript is just the job :)


Holy moly this change really sucks.

I see what you were trying to do here, but it comes down very harshly on the users. I like sidebar ads, often they are subreddit ads, or at the very least they are higher quality. Sponsored links really suck, I don't really like to see them.

This isn't "for simplicity", you are flat out taking features away.

If reddit doesn't want my views/clicks on sidebar ads, then so be it.


Did you just cowboy a `GetValue` into core?


hum, don't push this type of branch in master, the second cancel the first.

It's a little bit dirty


Canola is misspelled.


Why do we need the function? It just calls sock->ops->release() which is just inet_release(), a prototype for ss_do_close().


This commit fixed the situation where draft items were being exported into the main course export. All draft items should only be included in the `/drafts` folder. The course items weren't being retrieved within the branch_setting context manager - now they are.


I will rebuild that after the raid. Which means I will force master to clean it.


poulet_a aime ce repo


If it has its own pre-suite, can probably drop the `passenger` flag? No harm in keeping it for passenger-specific tests though.


You're welcome. I'm sure everyone will love gravel.


The requirement resides in page 10.

> All based on HTML5 canvas element, see intro1, intro2. Your <body> tag should be completely empty, and the platform will create 1-8 canvas elements.

No point will be deducted for this one for homework 9 though.


It's a hack.
I suggest it be removed, as there are smarter ways to achieve this.

On 23 April 2015 at 23:12, Călin Iorgulescu <notifications@github.com>
wrote:

> This function seems to just replicate the behavior of uploadAssignment(),
> with the exception of being able to specify the name of the temporary
> intermediate file. I understand that this is used by the meteor UI,
> however, it's unclear why it is needed.
>
>    - The submit.submit() function will copy the temporary file to the
>    vmchecker repo anyway, so the temporary file is not really useful.
>    - If the intended use of this call was to allow the arbitrary
>    uploading of files, then why call submit.submit() on each call?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rosedu/vmchecker/commit/192dde4df5e075801bb8ea97fa310d850be11815#commitcomment-10876362>
> .
>


Tu te fais chier toi non ?


Above I've attempted to capture the reported situations in unittests.


88737258-EPW-Ruby-Access (jasondegraw) - i386-Windows-7-VisualStudio-12: Build Failed

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


Stop using github GUI

Get this: https://git.wiki.kernel.org/index.php/MSysGit:InstallMSysGit

and learn to use it: http://git-scm.com/book/en/v2


I have updated the css file in the develop branch, it can be found here: https://github.com/aferrero2707/PhotoFlow/blob/develop/themes/photoflow-dark.css

The attached image shows how the photoflow window looks like on my linux box. As you can see, buttons have some light outlines and some widgets get an unwanted blue color... The idea would be to have a very simple appearance of the widgets so that they do not distract the image editing.
![screen shot 2015-04-23 at 9 10 16 pm](https://cloud.githubusercontent.com/assets/6229237/7306516/0d548146-ea06-11e4-8361-afd0a840d1f4.png)

Thanks a lot for the help!


Sure, I don't think it really matters personally for a project like this.

But I will make a new branch next time and merge it from there so there are less commit messages.


Should be 'bootstrap warning yellow ...'


Thanks for sparing me a lot of work :D Btw, how did you manage the successful merge? I was too fighting with it for a while and just cherry-picked your work because I had no fun in continuing it ^^


I removed the function.


I don't think the text is entirely correct after the changes. It should say somehow that it tries to obtain the instanceId from the portlet ID, but it may not be there either. The current wording seems to imply that it will always derive an instance ID from the portlet ID.


I don't think you need to build a `Block` here, `addPage` just looks at the values, so you can just use an `int[]`


Made them consistent.


Rather than just setting a fixed capacity, the intention of this is to prevent any costly reallocation of slices to happen.

[As far as I know](https://golang.org/doc/effective_go.html#slices) the problem with not assigning a capacity and just letting the slice grow would be `append()` taking care of it capacity for us. `append()` will start by setting the capacity to a conservative number, if this capacity is ever reached, `append()` will just reallocate that slice with a bigger capacity in another part of the memory, I doubt the actual `append()` implementation just doubles the current slice's capacity as illustrated the example, but it surely grows large enough to avoid more reallocations to happen in a short amount of time, reallocating is more expensive than allocating.

In the other hand, this is a slice of **pointers** to dialer, so it requires significantly less space than a slice of dialers, so I think we won't be wasting a lot of memory by this allocation and we'll surely prevent reallocations.


In my opinion hiring a consultant "programmer" on an hourly vs a fixed rate will be determined based on the scope of work defined/agreed by the client. For example: If the project at hand has a clear defined business rules, and functionality specification, it would be beneficial to hire a programmer on a fixed fee and contingent upon execution of milestone deliverables. This approach following the waterfall methodology and the reasoning is based on the ability to mitigate any change orders requested by the client. This way, any new change orders will incur additional cost/time and will prompt a work order to the client to approve.


c'est pas qu'une impression


Ok. The problem is that I have code like

```r
met <- settings$run$site$met
```

That now needs to be converted to

```r
settings$run$inputs$met$path
```

Straightforward enough to fix, but in general, is there a reason not to duplicate this in order to maintain compatibility of old settings files?


@tisto This has been released to PyPI, but there is no tag on github. Can you check if you have unpushed changed locally?

Otherwise I can make the tag and do the postrelease.


joł


No. Just a little bit longer than before.


Hey. Do you think it would be possible/feasible to split the normal version from the enhanced version, as in two separate files? Part of it is aesthetics, but it may be easier modifying the code down the line.


Hopefully RE-4470 will be fixed first.


Subclassing from nothing is certainly incorrect, but what should we subclass? `str` and `unicode` are immutable, which lead to various unpleasantries (and I'd much rather have the `JSONString` immutable as well).


I don't see any attachments here, could you email them to me directly (jkeane@gmail.com)?


Add a space between `.button` and `{`


Did we agree to put formatting changes in separate commits? (cc: @glasserc)


This `publish()` was causing the later draft import to be ignored - but only when `recursive_build()` and `_import_course_drafts()` were wrapped in the same bulk_operation. Removing the `publish()` yields the same intended result - the published block is still created via the fall-through code. Somehow `publish()` is creating a published block differently than the fall-through code - and in a way that is not compatible with a bulk_operation.


I have a question on the key - can it contain spaces? For the last project, I thought that the key couldn't have any spaces, or else MapReduce will use only the first element as the key in the next map.


We are not supposed to have a replay button in the game. The mega game app that will host our game has the New Match button that our game is supposed to use. That's why we did not try to improve the replay button. It is just supposed to be removed, so I don't think the grade should be deducted because of this.


(d->mwilight & ~(1 << 0)) und status ist nicht das selbe.
Auch  htolel(SKINNY_LAMP_ON) != htolel(d->mwilamp)


This would be an excellent time to practice vendor prefixes! Here's a place where you can generate all of the "fallback" gradient styles: http://www.colorzilla.com/gradient-editor/

Let's aim to get that into one of your next few commits. I'll create an issue and milestone for progress.


first


first write


yay github


This should not be in this commit.


The contract in `BlockKey.from_usage_key()` indicated that sometimes a BlockKey was being passed in to `reference_block_id()` instead of a `BlockUsageLocator`. A BlockKey is what's needed anyway - so just return it.


merge maćka


Got it. I'll describe that for the formModelBean param. Thanks!


In docview buffers we already use `s` for this. We should be consistent.


Same thing here, add a space between the class selector and the opening bracket.


Truth. Any preference for "unpersisted models" or "unpersisted records"?


Ok... Maybe it's because Yoav didn't change the requirement after we all discussed with him for some reason. But thats fine.


so wird ein neuer status gesetzt, auch wenn das nicht zum device gesendet wurde


It really was a missing piece of the framework. I think it's more complete now than ever. :-)


Good use of HEX values! :+1:


This change doesn't actually fix the issues associated with PLAT-452. However, this code more accurately represents the versioning happening before I converted `BlockInfo` to a class from a dict. The code before was checking for the existence of keys and branching differently as opposed to a key being None.


"Mark private functions" or something. They were already "namespaced".


Ok, I was just thinking that creating a temporary file somewhere and leaking its name to the UI (e.g., like uploadAssignment() does via its 'file' return parameter) is redundant if we were to remove temporary files after upload. That's why I wanted to make sure there wasn't some further underlying logic here.


OK, I just pushed up the `mutex-lock` branch which adds a semaphore around the code that loads the definition body. Give that branch a try and let me know if it resolves your issue.


Thanks for the hard work @rschmukler ! My issue seems totally fixed : http://codepen.io/jgx/pen/QbWjNE :+1:


Do we need these to also carry platform information like other projects?


We should zero the count and null the array pointer, otherwise if we throw the exception on line 626 we could have a count != 0 and try to free memory that we didn't allocate.


Got it :)

I might be a little behind on this one since Yoav didn't tell me that.

On Thu, Apr 23, 2015 at 4:26 PM, Dongbo Xiao <notifications@github.com>
wrote:

> Ok... Maybe it's because Yoav didn't change the requirement after we all
> discussed with him for some reason. But thats fine.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DependingOnMood/Pappu-Pakia/commit/d0c840972981511fa3e57961f12d26dc8b880aa5#commitcomment-10876585>
> .
>

--
Youlong Li
M.S., Computer Science
Graduate School of Arts and Science
New York University
chongzizil@gmail.com


Ant help on this I can't seem to get it worked out.


yep :)


yes i should definitely be in BB


I mean that we could theorically use `.danger` in other places than in a thread, so it's not a specific style


This goes in the MissionDirector and its readme

https://github.com/UCSD-AUVSI/MissionDirector


We should probably be sure this and the release lines land for 1.1.1


maybe


we may lose some precision here? where do we use this?

 http://railsware.com/blog/2014/04/01/time-comparison-in-ruby/


line length?


here too


and here


I see that you moved this warning to the ##Usage section, but it looks downgraded. While I don't think "warnings are thrown" needed emphasis in the earlier version, short keys are (as I understand it) a serious security issue and the warning should emphatically say so.


yeah but I think it's a regression from when rishav implemented the new delete/mark as read UI.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2095 of 2201 tests passed)

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


The correct property name to style a page's font is `font-family: Arial` if you're only adding the font's family and not other styles like color, size, etc. Remember specificity over brevity!

If you *are* adding those other styles along with the font family you can use the solo `font` property as a shorthand, like so:

```
body {
  font: font-style font-variant font-weight font-size/line-height font-family;
}
```

More on this shorthand [here](https://css-tricks.com/snippets/css/font-shorthand/). You can use as many or as few of these as you want, but you need at least font size and family...If we were writing it with the styles you're using right now it would look like:

```
font: 12px Arial, sans-serif;
```

Adding the "fallback" `sans-serif` will matter less when we start using variables, but more on that later!


I am not super familiar with front-end JavaScript and I wonder, might it be a better idea to `throw` rather than to simply `console.log`?


Yaaasss! HEX colors!


Actually, probably the opposite; rather than adding platform information here, I should be removing it from the .sln, since this assembly should be platform-netural.


l'email octo n'existe plus


in the future we might want to redo this using flex :) (but the recipient panel is tricky: we need to slide it down using transform...)


@pwidry15 In your note, you wrote about a scalable technology (back-end) to support millions of users. Essentially, having a group of developers focused on sustainability utilizing Amazon S3 web service should be sufficient enough to support growth of users. However, I agree that such team would incur a large sum of compensation.


I added tags:true instead of branch:master and delete the platform entry so it can be used in windows and linux platform as well.  On the code side, I have deleted all the sys.exit() and replaced them with exceptions.


ah. I see what went wrong here. Nice fix.


remove the string import


The csproj included a different PathInternal for each platform?


This isn't really descriptive; could we maybe say what IOV stands for here instead?


You're right I'll make sure to get rid of the spaces. Thanks for pointing
that out

On Thursday, April 23, 2015, David <notifications@github.com> wrote:

> I have a question on the key - can it contain spaces? For the last
> project, I thought that the key couldn't have any spaces, or else MapReduce
> will use only the first element as the key in the next map.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/d1vanloon/cs5621-project-2/commit/22e653b951c38b12a3e22ec06404f8faa048f264#commitcomment-10876540>
> .
>


oops


A useless cast


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/f21f7b1be130f148614c68ebeefb51f13a33036f


merging basho/yokozuna/perf/zl/cache-nval-for-coverage-plan = f21f7b1b into borshop-integration-478-perf/zl/cache-nval-for-coverage-plan


basho/yokozuna/perf/zl/cache-nval-for-coverage-plan = f21f7b1b merged ok, testing candidate = cb41f6ad


You did not close file.
```python
with open(outfile) as f:
    if build_no in f.read():
        continue
```


Same as IOV


nice method. :+1:


Why don't you allocate this buffer on stack?


:+1:  Thanks!


Does the reducer look okay otherwise? I wanted to get together after
security tomorrow and go over job 1

On Thursday, April 23, 2015, Jordan Ross <rossx624@d.umn.edu> wrote:

> You're right I'll make sure to get rid of the spaces. Thanks for pointing
> that out
>
> On Thursday, April 23, 2015, David <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> I have a question on the key - can it contain spaces? For the last
>> project, I thought that the key couldn't have any spaces, or else MapReduce
>> will use only the first element as the key in the next map.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/d1vanloon/cs5621-project-2/commit/22e653b951c38b12a3e22ec06404f8faa048f264#commitcomment-10876540>
>> .
>>
>


Your closing bracket should have one less space in front of it!


Let's get some vendor prefixes in this gradient as well!


#4833-Documentation-for-HX-controlling-to-SP (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1136 of 1136 tests passed)

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


You probably don't want to allocate the buffer each time, consider allocating it on stack.


88737258-EPW-Ruby-Access (jasondegraw) - Win64-Windows-7-VisualStudio-12: Build Failed

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


As above.


You should extract common function to a separate file.


do not comment out these lines. you can try a differnt way using extern to avoid static in namespaces

see the commit below which also fixes the unused variable warning

https://github.com/libkml/libkml/commit/73563c403870139d622fe24ae05f8300ecb704a1

If you could update your pull request, then i can merge them


Both of these elements are nested inside of the `<a>` tag so should be indented the same amount of spaces.


`shell=False` is default behavior [check_output](https://docs.python.org/2/library/subprocess.html#subprocess.check_output)


Perfect indentation! :zap:


I'm not sure this section should be removed. We're removing this section in modules if the section provides no useful information. It's pretty self-evident that Apt package/service/configuration files will be affected, but the rest seems worth mentioning.


Nailed the indentation here, too! :100:


dieser vergleich ist falsch, hier fehlt der vergleich mit der instance (d->mwilight & ~(1 << instance))


(d->mwilight & ~(1 << instance)) müsste das sein


yay back to main repo!


I don't think your suppose to actually throw errors if you want to silently handle them, you just log since the error is really only for developers and be hidden from the user.  It should appear under the console when your debugging an app.  I usually see people do it that way and then if the message is fatal they send an alert box to the user (usually something about contacting the admin), but you might want to double check with someone.


This `<p>` tag should be indented/aligned the same as the anchor tag above it because they are nested the same number of levels as it.

:dancers:


i don't understand this change-- it's still available through cocoapods, right?


this is less-formatted for me now


Whoops. Sorry about that I forgot to make the push that fixed this. I've removed the license completely.


but at what price: https://github.com/django-pylibmc/django-pylibmc/commit/097f1b4c6b86cbd1fb7bbfa461a90aec997999a5


I forgot that it is not possible to merge any changes to google/libkml which you forked. I had created a repo libkml/libkml. So that external contributions can be welcomed. Unfortunaltey google shelved libkml for some unknown 'good' reasons.

see the related issue
https://github.com/google/libkml/issues/4


this just appears as a link for me, not the image


Indent this `<span>` one more time because it's nested inside of the `<a>` tag.


Perf! :sparkles:


@westonruter This should do the trick. I just tested it and it worked for me.


Something here is off by 1 space! Spend a second combing through this when you get a chance.


It seems that OCL_MomentsFixture_Moments test data should be regenerated too.


OK. Sound. I am coding something at the moment, but I will merge this soon. Many thanks, @GarysCorner.


Ugh, you're right.  Ok, I'll add those here.


https://github.com/VividROM/android_kernel_lge_hammerhead/commit/6b6d6fcaf473e32d6be2f41cecad9eb02fc53ea2 :D


@jgoux thanks! Glad to know it's working!


Ah, I see


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yay less custom classes!


I LOVE how detailed your commit message is!


No problem thanks for letting my participate, it's a good way to get the code daemon out when I'm having a mental block about what I want to code.

:)


Not only images should be allowed


Hi, could you please take a look again on this change, because if user-data is not in json it fails with exception.

```
2015-04-23 19:43:34,621 [salt.loader      ][CRITICAL] Failed to load grains defined in grain file ec2_info.ec2_info in function <function ec2_info at 0x7fdec89c8410>, error:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/salt/loader.py", line 1249, in gen_grains
    ret = fun()
  File "/var/cache/salt/minion/extmods/grains/ec2_info.py", line 116, in ec2_info
    grains.update({'user-data': _get_ec2_user_data()})
  File "/var/cache/salt/minion/extmods/grains/ec2_info.py", line 104, in _get_ec2_user_data
    data = json.loads(response)
  File "/usr/lib/python2.7/json/__init__.py", line 338, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded
```


Без палева


No sir, I don't like it.


Fix this


Yup. Please check and get agreement of @shalako and @dieu and @gregoehmen using Tracker.

Thanks for following up on this. Best.


There's some solutions for this here - http://stackoverflow.com/questions/2446156/is-there-a-pluralize-function-in-ruby-not-rails

Not sure if it would be worth implementing them though


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1095


well, it's a good behavior to have I think so. Like that it's easier to check the stable branch which is master.


Thanks Cedric. As far as I remember that hasn't been finished, and it might not work 100%, but hopefully should at least point you in the right direction. Keep me in the loop on your further developments please!


The current comment seems a bit terse. May be elaborate a bit : "This variable is used to figure out the file extension for index and data files. When the server supports compression, this variable's value is typically ".gz" or else it holds and empty string".


Slight extra ` ` in the translation, please use:

```
"Impossible de supprimer l'instance de service, clés de service et fixations doivent d'abord être supprimées"
```


This sounds pretty broad... do you mean to say **all** classes, types, and defines require `apt`, or all the classes, types, and defines in this module?


Nice !


The psm part looks good to me.


There's a different way of doing it now, just with a flag instead of BindingFlags, which is a type that isn't available on all platforms.


Fix this. Should be D3D11_MAP_WRITE if usage == D3D11_USAGE_STAGING


Moving all of these examples to the Usage section was a good move. It's better organized and more useful.


Btw I think the new transitions are decent and physically realizable (no
weird twists). Baxter turns it's head a little which serves as a cue for
being noticed. Let me know what you think.
On Apr 23, 2015 2:21 PM, "Patrick Steadman" <notifications@github.com>
wrote:

> Because my username is set as the git user on that computer...look up how
> to change the git user.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ptsteadman/baxter-teleoperation/commit/f835ec6b93fe76a0af693da1e515a5c74fc4c9d6#commitcomment-10874484>
> .
>


look´s like a Kevin-Code (-:


Typo: "givenby" -> "given by"


Didn't we just turn this into a sha512sum in bd98bd62ae6426621228845d11f48c9dc473826d?  I think it was broken before this change, but since you're here, might as well fix it.


It seems to be better to use focus (which checks if the window is already focused or waits for it to focus) rather than waiting for the activate event.


indentation ?


Woo! CSS commenting!

CSS commenting is so excellent.


Later this method can be extended sounds better than "needs to be improved".


You can also add me in here as an expert.


Makes sense! Re-upgrading.


Restoring. Just FYI, I did run this section past Morgan and she thought it was pretty self-explanatory. Better safe than sorry, though!


10/10 would lol again


Asi se vel footer luego de este ultimo commit
![footer](https://cloud.githubusercontent.com/assets/2413746/7307281/fab980a0-e9c8-11e4-8ba4-606be02d10ca.png)


Revert this. Set m_maxTextureSize in RenderSystemDX::CreateDevice as before.


test https://gist.github.com/KostyaSha/345dd69cf14621183206


Restore this with DX11 reality


You also need to set this mtu on the veth which is going into the container for every endpoint created. While you do that can you also add a endpoint test which verifies that this is indeed being dong?


I approve of these edits.  The feel I was kind of going for originally was "We've already done this.  This tells you what verb to do, now go find the steps to do it!"  With that in mind, do you still think this is too sparse (post edits)?  Should we include more detail/example code/etc?  To me it seems fairly clear, but I also went to college for this for five years, have been working in the industry on it for 5.5, and don't even remember a time when I didn't know how to use a computer.  Not having a lot of experience with pedantic theory, I gratefully accept any and all perspective on what will or won't make sense to somebody who doesn't already do this for a living...


Previously, there was a warning that this module would destroy existing content that was not declared with Puppet. You've completely removed that warning, but it seems like this has serious ramifications and needs to be mentioned.


written by real ronny


Let's practice with vendor prefixes here! Go ahead and pick out the vendor prefixes from [this article](https://css-tricks.com/box-sizing/) and get them into your next commit.


Thank you for the comments. I believe I have addressed them in the new commit. Let's see!


Holy indentation Batman! Make sure you're using spaces and only 2 for each time you tab!


Same indentation inconsistency. Just spend some time combing through this file and cleaning up the indentation.


fast-forwarding develop to borshop-integration-478-perf/zl/cache-nval-for-coverage-plan = cb41f6ad


Good use of HEX value here!


86046458_CondenserToZoneDoc (Nigusse) - x86_64-MacOS-10.9-clang: OK (1641 of 1641 tests passed)

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


Since all 6 HEX values are identical you can shorten this to just be `color: #333;`


I'm not clear why there are two "Defaults" here.


Again, I'm not clear why there are two "Defaults" here.


ADded formatting back.


@archonic this is not a change, this is a fix!
This is the way the WooCommerce always worked.
In just removed the option in the latest versions, however someone did not properly checking.
We'll not make any more changes here, just because it has the filter and also to prevent incompatibility!


Apt seems to be all caps through the rest of this doc.


Yeah, I'm updating this to be clearer


I'm 99% sure that line 72 and 73 don't ever end up being used.  There was original intent that they would be.  Try removing them or commenting them out and see if anything breaks.


You can actually include all three of these border colors in the `border` line above this.

Like so:

```
border: 1px solid #adb1b8 #a2a6ac #8d9096;
```


It won't appear as an image until merged because it refers to `master` and it's not merged there yet. Hopefully it will just work :)


LGTM


@lots0logs : Can I delete that .SRCINFO file or is it necessary?


Oops, just noticed all the noise in this commit, sorry about that, my editor does that stuff automatically on save.


Right... Will take care of it, thanks.


Not sure whether my issue is also resolved by this commit but when I delete a row from list view then also a file data is removed from the corrtesponding row of joined table (not whole joined data, it's set to be kept when deleting row from main table).


Yup, that's like it.
It was actually somewhat hard to verify that all details were properly transferred. Verifying that with other changes mixed in would have taken more time and concentration than I'd have had available for this.


It is theoretically possible to "poke" objectB using a static (not dynamic) objectA.  That is, simulatorA could (via script) slam the position of static objectA into a state of penetration with dynamic objectB, thereby causing objectB to activate and move.  The logic above would, I think, prevent all simulators from taking ownership of objectB for this case.


Hey can you integrate sizing (rescaling)? This script is rad by the way, i really need to say that, since this thing is great.


@greenscar you can always run from source if you need the feature. but it should be merged for others


good thought. cleaner code.


Yep, I meant "in this module". Thanks!


Let's practice vendor prefixes here, too! Here's a great generator for gradient fallbacks: http://www.colorzilla.com/gradient-editor/

Let's get this into your next commit, also. :boom:


Looks good, thanks! I always thought trigger would include the event anyway.


Yes because the rest of the method has GetValues so I left it consistent. :p


Perfect use of `border-color`! Yee!


I haven't looked at it much, so I can't speak to the rest of the reducer.

I'll be eating lunch after security, but I can meet in the lounge.

From: JordanRoss1992
Sent: Thursday, April 23, 2015 3:38 PM
To: d1vanloon/cs5621-project-2
Cc: David Van Loon

Does the reducer look okay otherwise? I wanted to get together after
security tomorrow and go over job 1

On Thursday, April 23, 2015, Jordan Ross <rossx624@d.umn.edu> wrote:

> You're right I'll make sure to get rid of the spaces. Thanks for pointing
> that out
>
> On Thursday, April 23, 2015, David <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> I have a question on the key - can it contain spaces? For the last
>> project, I thought that the key couldn't have any spaces, or else MapReduce
>> will use only the first element as the key in the next map.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/d1vanloon/cs5621-project-2/commit/22e653b951c38b12a3e22ec06404f8faa048f264#commitcomment-10876540>
>> .
>>
>

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


Remember arial is a default font on most computers, so it doesn't need quotes.


Consider limiting your lines to 120, or something. If you don't have your IDE in fullscreen and the editor as the only windows it gets a bit annoying scrolling sideways and then back between each row to read it. Also some of the code lines are too long. Would be great if you could look over this in all the changes. It may seem like a silly thing, but it actually gets less delightful to deal with code having such long lines.


Починка http://iichan.hk/b/res/3396422.html#3497059

Почему-то `value instanceof File` не работает, потому просто проверяю наличие поля `name` в блобе.


Same HEX color note...Because the HEX values are all the same you can use a shortened version. Like so:

```
color: #111;
```


Wouldn't it make more sense for the default to be null? I don't know why they did it that way in AFPChain. Lets just remove this line.


Nice catch! :fire:


Same 6 values, use `#fff`


Add the border colors to the `border` property above.


The previous example of two code samples were not equivalent.


@ni  do you think


Woo box shadowing! :ballot_box_with_check:


Probably a bug. We should do a deep copy.


Grammar: add a comma at the end of the line.


Interesting implementation.  I like it!


Commit #2 is okay except for the grammar nitpick.


I'd set them to null. You would have to set all these values subsequently, so there's no point allocating space for them.


i dont know why i started the repository using the eclipse plugin (i have been using git for a couple of years), but since some features didn't work i jump to the console


LOL okay, serves me right to comment on an outdated diff.
Still, English requires a comma after `hyperbolic`.


Drop the '' around arial.


Use `#111` to shorten.


It's generally frowned upon to expose member variables as part of the public API.  If we wanted to use this variable in the JSP page, we would normally add a "public boolean wasDeleted() {}" method.

In this specific case, we can get similar information by looking at the View action, so we don't need it.


bryvi's Correct Code for Programming Assignment #2


Loading the movie once up here cleans up the code below a little.


Perfect indentation in this block! Woo!

On elements that have things nested inside of them, make sure you wrap the element by hitting the return key so that the elements opening and closing tags are aligned.

Ping me if you need further clarification.


Ow lol wut, must have been autocomplete, so sorry


what do i want to write


Checking the which button the user clicked let's us know what they intended to do, so having a wasDeleted variable get set is probably more trouble than its worth.


resolves #2


Same with nested elements here. If a containing element is a block element, the element that's nested inside of it should be on its own line. We'll practice!


It is better to add && at the end of the export lines even if exports are very unlikely to fail as it gives a good example to follow.


Drop the closing span tag onto a new line to be aligned with its opening tag.


I think you committed this by mistake


Also, in case docker0 is already present, this change is not enough to make the mtu change happen.

But I am not sure if we want to apply bridge changes if a bridge is already there running and other containers are already using it with the current MTU.

@mrjana WDYT ?


Re-pulled from my master branch when rebasing...


Since you now have a connection passed in execute(Connection conn), how about refactoring this to call executeSQLWithConnection(conn) instead of duplicating lots of code here?


?? pourquoi pas Values tout simplement ??


rax retry tests


so ruuude :-1:  :(


Имхо лучше делать дифференцированные коммиты: одна фича - один коммит. Версии можно делать тегами.


v1.4.6 was tagged but https://github.com/kylecoburn/PokeBot/blob/647ab341e1ce352e5367ef18bc57b54f995b0200/main.lua#L11 still says v1.4.5.

To remove the existing v1.4.6 tag:

```bash
git tag -d v1.4.6
git push origin :refs/tags/v1.4.6
```

Then `main.lua` can be updated and a new v1.4.6 tag can be created and pushed.


Album 1 can easily be linked to the Front page. Here we just provide some basic PHP structure. More to follow...


+ sign shows the changes that was done


Yeah I guess that's totally reasonable to stay consistent with the rest of the method huh https://github.com/vanilla/vanillainfrastructure/pull/6


figura.add(linha);

todas as clases ja tem um metodo que separa e pega cada valor daquele formato de salvar

acho q precisa soh colocar os IF`s para separar circulo, elipse, ponto, reta.


What... how could I have made such a typo?!?


If the user tries to enter an invalid article and then tries to click `Lend article` again, the error message still appears.


at this point is better to add some information


j'ai également fais quelque chose comme ça au début. tu peux te contenter d'envoyer le tableau, et faire le length dans la jsp grâce à ${fn:length(tableu)} :100:


How do I get it


take it off!


There was a mistake I made here. We backup the idarray iff we get a higher log_posterior. So it should be something like:
if (log_post_rs > max_log_posterior) {
	max_log_posterior = log_post_rs;
	if (ruleset_backup(rs, &rs_idarray) != 0)
		return;
}
And max_log_posterior should be initialized -1e9 instead of 1e-9...


Forgot it in commit message.
References issue #1432


Uhm... I don't like this. The function claims to return a `std::runtime_error` but returns nothing - instead it just throws the error. Further down, we call `throw badSourceError (nVersion)`. Can't we just (a) ditch the Hungarian and (b) change the function thusly:

static
void badSourceError (int version)
{
    throw std::runtime_error ("bad source: " + std::to_string (version));
}


This line should have last name before first, right?


Please leave the lint target in,  I use it. Or change all the lint targets across the strong-* to be pretest. Adding the pretest is fine.


Hi, in the laravel4 support it says 'laravel5 support'.
Regards,
Damian


Do you need to cast obj to BaseOpenmrsObject in order to get a uuid?


Do you need to cast the above in order to set the uuid?


Same questions here about casting, as in the two places above.


That escalated quickly :)


Aww, that would have been hard to track down. Glad you caught it.


Слушай, а у тебя краутчан на некроопере открывается?


en que navegador es? porque lo testee en diferente tamaños de pantalla xD
![cap](https://cloud.githubusercontent.com/assets/6947272/7308029/0d446cf8-e9ce-11e4-9578-501519492f59.png)


Неправильно спросил. Работает ли скрипт на краутчане с некрооперой?


Normally we name tests like:

    Test<Primary Struct><Test Name>

in order to avoid collisions in the testing namespace for a package and provide a bit more info. So you'd rename this as:

    func TestScannerPaceInterval()...


What is the use of overriding this method if you are doing exactly as in the base class?


86046458_CondenserToZoneDoc (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1647 of 1647 tests passed)

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


Con estos cambios mi combobox para seleccionar la calidad de café ya no funciona.


Should just be `len(list(nonisomorphic_trees(order)))`, though I question why such a simple function is not left up to the end user.


This is a really tight window. The circle CI continuous integration testing we have is quite slow sometimes, making even ridiculously simple tests which rely on clock measurements flaky.


Bugs:

Monster data not getting posted

Setting dropdown in armor aside of skills pages causes armor aside to not load; setPiece is getting stored in session, since clicking on a skill name from the main section again shows the armor aside, with the proper slot selected in the dropdown


86046458_CondenserToZoneDoc (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1647 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (346 of 346 tests passed)

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


Там у меня сам сайт не совсем работает:
> Unhandled Error: Cannot convert 'parent' to object

По всей видимоста, сами разработчики краута забили на оперу.


All these comments are not needed and should be dropped. One may simply pull them from git history.


Firefox v35.x

![firefox](https://cloud.githubusercontent.com/assets/2413746/7308061/580fa5e0-e9ce-11e4-9679-2e8fc0c9ee19.png)


Ah, scratch that...you're just ignoring integer division truncation errors.


Corrected the function to return a value... ooops!

I originally had it the way you suggested it - but that doesn't work, I got a dozen errors.

The reason is simple - these functions return a value.

If you call a function that always throws an exception, and then return nothing, you get an error.  So you'd have to add all these spurious return values to quiet the compiler.

So the throw has to be in the function that uses it to terminate execution.


@differentmatt we should probably just start using i18next's interpolation for this, like https://github.com/codecombat/codecombat/blob/master/app/locale/en.coffee#L430


Fairly confident this cast is safe, but...


ist die sicher nötig? die messung dauert doch auch ohne pause schon lange genug, oder?


What is the user of overriding this if you are doing exactly as in the base class?


А что не так с куклой? Вроде на первый взгляд всё работает. Сейчас буду смотреть что там с `ihash`.


This "reset" makes logo image be ugly stretched. We need to adjust correct styling for it.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (345 of 346 tests passed)

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


Schön gelöst, eine maximalanzahl an durchläufen wäre aber wünschenswert (Sonst läuft das programm ewig...)


Links in pdf wrapper documents to pdfs need to be renamed also


Should be either "non-isomorphic" or "nonisomorphic", but certainly not "non isomorphic". Same objection applies to the section header in ``generators.rst``.


This seems to be linking to an apk for something called 'Smart School'?


Ya, https://github.com/codecombat/codecombat/issues/2702


bitte noch ein paar leerstellen.. :stuck_out_tongue_winking_eye:


Is there any license to use JOANA's code, such as Eclipse Public License?


Isn't this uuid null? Can you assign an explicit uuid value?


// Konsolen-Befehl ausführen


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

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


I like that quill. :)


@jeremymasters don't understand what you're asking. the problem is that if there are non-unique elements in there now, creating a unique index would fail.


please ignore me :tongue:


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/0e41e373c670a8f363fa342d943a7827997c91f9/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/0e41e373c670a8f363fa342d943a7827997c91f9/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/0e41e373c670a8f363fa342d943a7827997c91f9/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/0e41e373c670a8f363fa342d943a7827997c91f9/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/0e41e373c670a8f363fa342d943a7827997c91f9/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Ist das bereits asynchron oder wäre es möglich eine weitere Funktion zu erstellen, die den Befehl asynchron ausführt? Evtl. einen `bool async = false`-parameter?


86046458_CondenserToZoneDoc (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1647 of 1647 tests passed)

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


Das ist ja temporär, richtig? Falls nicht bitte brav nen kommentar drüberklatschen.. :smile:


GCC warns about unused local typedef `coordinate_type` above.
Should it be removed or e.g. passed into the `value_writer`?

Currently `value_writer` takes `double` so the coordinates are converted implicitly to `double` before they're passed into `write()` function. This may be a problem since Boost.Geometry is type-agnostic.
Could arbitrary types be supported or is this a limitation of the format?


leerzeile


leerzeile.... * nerv *


В некроопере и на http://410chan.org/b/ тоже не работает скрипт. В консоли `Generator throw:  [object Error]`. Подробнее не пишет.
Та же хрень и на http://chuck.dfwk.ru/df/

Возможно, дело в какой-то настройке, которая там была включена.


Gefällt mir! :+1:


Consider adding "return this" for both onPress and onRelease functions so that is a little more javascript-y.


Kommentar?


Sind 10ms, aber wird auch ohne funktionieren. Muss man testen


Typo. This should be "polarity".


Schön übersichtlich und leicht verständlich gestaltet, gefällt mir!


This should be a parameter. In my case I want "inline-block". To workaround this I have to use SVG.


Ммм,да. Дело в настройке. Кукла с дефолтными настройками загружается нормально.


I liebe Denglisch!


Oh God! Just, how did I manage to do that? Anyways, fixed. Thanks for letting me know.


Hi,
You do not need write permissions to my project. Just fork the project, do your changes on your fork and then send me a push request via GitHub. I can then integrate your changes to my repo.


Кстати, какой там адрес у борды в i2p?


86046458_CondenserToZoneDoc (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Das `else` vom `else if` kannst du dir sparen, da steht ein return drüber... ;)


RunConsoleCommand oder GetRealDistance? GetRealDistance geht nicht async. RunConsoleCommand geht async, nur meine Faulheit war stärker :P


Fail! :stuck_out_tongue:


:joy:


fixes buildersbrewery/linden-scripting-language#1


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (346 of 346 tests passed)

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


pour moi tu n'as pas besoin de defered.promise d'angular, reste avec les promesses natives wrap l'appel au scope d'angular
```javascript
db.get()
  .then(function pouchResolved(doc) {
    return new Promise(function promiseToExpose(resolve) {
      $scopeRoot.$apply(function () {
        resolve(doc)
      });
  }).then(function scopeRootApplied(doc) {
    $scope.myvar = doc;
  });
```


Danke dass du meine falsche Reihenfolge eingehalten hast, machts einfacher weils konsistent ist! ;)


Got an example?  I'm on the .NET platform with C#.  The wiki page should probably be updated with this sample as well.


Fehlt oder muss eine hin? Sehe nicht wi


die ... was ...?


Кстати, у них нумерация постов вроде нормальной стала. Может уже и можно поддерживать без переписывания половины скрипты.


I'd rather have:

    if test -e /data/ipfs/config
    then
        echo "Found …”
    else
        …
    fi

because it matches Git's [coding][1] [guidelines][2].  We don't have to follow those coding guidelines here though, that's just my preference.

I *do* think we should drop the `!` and flip the blocks around, since double negatives like ‘if not not …’ trip me up when reading the `else` block ;).

[1]: http://git.kernel.org/cgit/git/git.git/tree/Documentation/CodingGuidelines?id=v2.3.6#n106
[2]: http://git.kernel.org/cgit/git/git.git/tree/Documentation/CodingGuidelines?id=v2.3.6#n121


Das ist ein Eyecatcha das muss so


oh, danke, ist mir wohl beim mergen passiert...


Muss das noch angepasst werden? Wenn ja, mach mal :P


ich glaube die war schon richtig, oder?


done.


Das wird man nie erfahren :P


die auch?


I would reverse this logic so that it assumes NORMALLY_HIGH is the default for all but DIGITAL_IN_PULLDOWN or DIGITAL_IN_WAKEUP. I think that the most common button types are NORMALLY_HIGH. The one exception to this is buttons on the EVB since we set them up to wakeup the imp which requires the normally_low polarity.


muss da ein kommentar hin?


ähm...?


kommentar


leerzeilen??


Loving them extra node related commands. Haha.


Need to add DIGITAL_IN_WAKEUP (for Pin1 only).


weg damit


Fix per suggestions above.


Jip


naja, wenn du meinst.. :)


Wurd im neuen Commit bereits gefixed


You are right. I fixed the javadoc, thanks


Done :)


ok


No need to make this array a member since it is only used locally in tryOpen().


I'm auch


Die Leerzeile?


Neuen Commit bereits gefixed


ich meine RunConsoleCommand. Du musst deine Faulheit bekämpfen! :stuck_out_tongue_winking_eye:


Done commenting! Great changes.


done.  (actually done, not TODO'd)


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (346 of 346 tests passed)

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


unsupported python object --> Python object does not match any provided type tag

(Note that earlier in ft_flatten the Python object is converted into a C object and this function will report whether the object is supported or not)


fehlt oder muss eine hin? = ja oder ja? :laughing:
die antwort aber ist: nein => da ist eine zu viel.. ;)


:D


initial commit


I'm more familiar with MySQL so forgive me if this is not do-able in this DB.  I've seen cases where columns were non-unique and then another column was added and populated during the creation with a timestamp such that each row would have a different timestamp.  Thus creating a multi-row unique key that includes the ID.  Sorry if I'm off on a dead end.  I don't mean to distract from the work.


Ja das kann ich machen :P aber wenn Thomas noch 3 Sekunden gegen die Wand fährt passt die Melodie besser


ne, ist schon gut so. Solange wir es überall falsch machen ists ok.. :D


to be continued.. :D


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


This says that the parameters are all "required, except on Debian and Ubuntu and where otherwise specified." But this module says above that it's for Debian and Ubuntu, so wouldn't they all be optional unless specified. This also implies that this is for other platforms too, but that isn't really clear throughout the doc.


The default description note seems a little confusing to me, although this may be more about what systems this module supports; it reads like Debian and Ubuntu are the exception rather than the main use case. I'm also not exactly sure what action I should take here about the default, given the parenthetical.


Let's pin down these systems in clearer way, maybe more like vcsrepo. Also, I was a little confused throughout the readme about whether this module is just for Debian and Ubuntu, or whether it can be used on other systems. In some places, specific Debian and Ubuntu defaults or parameters are called out, implying that it can be used on other systems, and Morgan tells me that Apt isn't strictly Debian and Ubuntu.

So, clarify the Limitations, and then go back through and make sure the discrepancie


soweit bin ich noch nicht...


on the `aria-haspopup` attribute, does it needs to be the string `"true"` or could it be just `true`?


:joy:


jus one space after the `)`


ja


You can eliminate some of the switch cases by defining a local/anonymuous function int getFrameRateIndex(mad_units madUnits)


gut


I realised they should be the same size. The counters should be 2 cache lines apart to avoid false sharing. So we we have labels the same size we can have 4 byte length plus 124 bytes for the value. Labels should not be longer than this.


Nice!, it close https://github.com/TrinityCore/TrinityCore/issues/10046


Is here the right place to mention that t.flatten([1.0,3000000000]) works but t.flatten([3000000000,1.0]) does not? fasttypes fixes this, so it probably doesn't need to be fixed in types.


I think this must be also in the maintenance branchs... (3.7 - 3.6...)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (346 of 346 tests passed)

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


Thanks @DheerajKhajuria , this is better approach than previous version


Should this still be commented out?


@codenameone Please review for merge into 3.0


It should be possible to reduce the redundancy introduced by those many switch/case statements. Just a feeling ;) Would be nice to reduce the code bloat a little bit.


Could also be hidden in the .cpp file


I don't think that will render nicely in godoc. You can verify locally with:
godoc -http=:6060
then look at http://localhost:6060/pkg/github.com/shishkander/interrupt


Close() error
so it matches io.Closer.
Do not panic, just return an error on second call.


Not needed., only channel needs to be initialized here.


That is a race condition.


Should this be `var kind = require(...`?   Also, I don't think you would require the module you are actually declaring, would you?


Are you sure that's a needed commit?
Because that was added to let you exit the chat.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (346 of 346 tests passed)

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


This is not thread safe.


considering adding a `String name` to the class, which would require edits of `Hero` and `Dragon`.
thoughts?


Want to re-open the bug to specify what should be done, then?


LGTM, just some cosmetics.


Why in a defer? It makes the whole code harder to read.


ublock->unblock


:kissing:


or I can. Does 1MB work for the size of both?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (346 of 346 tests passed)

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


Thanks again, @obiltschnig! These were really pain in the ass


ever should be every


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (346 of 346 tests passed)

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


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


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


`'chidNodes'` looks lika a typo.


Well, ye.. We required SDL2 for joystick input anyway.


test [`lodash.take`](https://lodash.com/docs#take) test


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


#$@ag


please explain why `indexOf` was used.


@samaasallam


Also in progress is a way to externalize elements in the FLA file and add features to the controls therein.


good.


No, I think this needs to be `domainRepo`


There seems to be a love/hate thing going on with changeOrigin... no one lists it as an option in their docs (including node-http-proxy), and grunt-connect-proxy has removed it entirely (I suppose favoring the headers object instead).

Any idea why that is?
Would you be in favor of adding a "headers" option like that?


Let's go with records. Also ditch the try again part and add the job class to aid debugging.


Yea sure go for it.


Just to confirm, is this resolved per our conversation offline? Can't recall if it was before or after the time of this comment.


The second one is a note from Morgan that I should have deleted. Thanks for the catch!


gives error with your correction, changing back to "time.time()" corrects it


86046458_CondenserToZoneDoc (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


I don't necessarily agree with export networkType. It should remain unexported.


This seems like a weird way to write this, is it a py2/py3 thing?


@phase GitHub will auto-link GH-xxx or #xxx (see https://help.github.com/articles/closing-issues-via-commit-messages/)


We should not include bridge driver in the callers of libnetwork. bridge should only be known to libnetwork.


Done


There should be four space indentation here, AFAIK.


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


It is in two...
One's filename is MouseHunt AutoBot.user.js
The other is MouseHunt AutoBot blended.user.js


s/exaxt/exact/


The blended one is the one with enhance


This is just incase of code overload where the enhancements will screw up with the rest of the script


Вот такая вот ошибка на краутчане:


პლუს ნიშნებით ვხედავ რა შეიცვალა ამ ფაილში.


Thats simpler. Thanks


@wjordan , is there a reason you dropped "if @game == Game.applab" in this refactor?


Btw the core of enhancement is written by CNN, whilst core of the bot is written by Ooi

My builds combines and fixes everything and tries to bring more needed features in.


I am forking this project and adding a comment.


Might want to remove these print statements


Это тамперманки?


@Ocramius what changed in DoctrineORMModule that justified the bump to `zendframework-2.3+`?