No, it's the reboot that we need to start.  The risk is that we check the status before it's started, and get a status of green, so think that the reboot is complete.


Agreed.


釣銭が切れる の ほうが言葉に統一があるかな？ 強盗も同様に 強盗する のほうがいいかな？


The transaction export array for PepperJam has a different structure now. The status in in column 11. I have updated the code so it follows the new structure.


The number of cards is not an issue for the AI. It is more of the signal to the user, 40 cards imply some kind of Limited format with less control over the cards, you play with what you got, there is some luck involve in what cards you get.

60 cards imply the Constructed format, where you are suppose to put in all the most powerful cards in can find that have great synergy or try to pull of 5 steps combos.


What does this `external` thing do?


@beghelli, could you tweak this to avoid the Undevined Index warning? Thanks, @bozana.


Kinda rude IMO, but of course perfectly legal....


Comentari simulat d'una altre persona. He fet el comentari en la línia de canola oil


I am thinking on trying an intermediate approach:

- If the duplicate directory has no children at all then it is safe to delete
- Otherwise (it has children)
  - Report the duplicate `pootle_path` and the duplicate directories `id` asking for manual intervention, and
  - Abort the migration.


Could you use this param2 when setting the new cactus node?
https://github.com/minetest/minetest_game/blob/master/mods/default/functions.lua#L146

lf someone changes one side texture of the cactus, the plant shouldn't be convoluted, but straight.


use: [isImage()](http://localhost/ckfinder-docs/template.html#!/api/CKFinder.Models.File-method-isImage)


Hi Tim, shouldn't we uncomment this try-except block again, so that users without GPU support can still use the TemplateMaker?


I think this is better.  We don't have a python style guide, so I assume we're using PEP8, which specifies that we should keep to 80 columns width.  Also, we should be using () around prints, to be closer to ready to move to python 3.


Will fix 1/2/3 this week and get rid of the comments.


ack


Loading File is not necessary here (file object has `isImage` method)


Prebuilt decks regardless of the number of cards, do evoke a sense of Constructed play. As mentioned, it is supported but not recommended for the best play experience against the AI.


TeamCity Jarvis Framework :: CI [Build 80](http://build.prxm.it:8811/viewLog.html?buildId=3057&buildTypeId=CqrsFramework_Ci) is now running


ステレオタイプ << hogehoge >> がおかしいです．
<<mitigate>> は 和らげるという意味で ミスユースケースが発生した時に，少しでも和らげるためのUseCaseです．
breakやempmonayは<<MissUseCase>>でいいです


Kewl, I'll go with that, using 0xff as in the neighbouring line.


Can you also make books drawable?


DesignDaysGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


Sounds good.  I have a the old LO database with this issue so we can test.

Would it not be more elegant to check that the duplicated actually has no leaves?  Anyway, your suggestion sounds good and we can at least test it against the real use case we hit with LO.


Yes, you are right. I am going to fix this.


Maybe add error checking. The `.done()` method is called for good response and for response with `error`.


I think it's better to add how to install and use traceur, though it is a dumb question. `npm install -g traceur` `traceur try-mie.js`


TeamCity Jarvis Framework :: CI [Build 80](http://build.prxm.it:8811/viewLog.html?buildId=3057&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 119 Build time: 00:01:06


A factory+polymorphism will be nicer


Maybe this will help: https://github.com/HaxeFoundation/haxe/blob/development/std/java/internal/StringExt.hx#L76 ?


I'll gather all the information. Funny fact, I don't use `key=` at all.


* Split **str** into a list of strings, using the **delims** delimiters and output the result in **str_list**.


Confusing comment. Command name is `ImageInfo`.


np


Chararacters ?


dito


And referencing command is `GetFileInfo` in comment.


River water is water not from the sea, so without salt, isn't it?


s/list/vector/


ack


1687-ThermalZoneClone (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)


yes it absolutely should


fixed.


fixed, thanks!


That's an incredible refactoring! Is there any feature change?


I think you meant canola oil instead of palm oil.


I'm sorry about pushing the subject again, but how do you guys feel about eventually adding Sealed mode? Draft looks like stretching it too far right now, but Sealed isn't that far from what Magarena already does. I realize it would probably require an interface for the player to see what cards he got in his boosters and build his deck from, which is placing even more work on Lodici's shoulders. But I think it's more fun for the player to have some control over the deck he ends up playing instead of using the random generator, and would lead to much more balanced gameplay - if we are talking about the human and the AI playing decks built from, say, 6x M15 boosters, I definitely sense that this will be a much more even matchup. I think it's easier to fine-tune the AI for that setting.

Sorry for barging in on the discussion, I think I'm interfering too much here sometimes.

About the preconstructed decks: would it be interesting to eventually organize them a little bit better? Right now you can't tell which sets the cards come from, what kind of deck it is - all you have to work with is the title (besides actually opening them in the editor and seeing what they are, of course). Maybe they could be organized by format - precons from Modern, Legacy, X Block, and separate them between agro, control, midrange and combo - I don't like these names too much myself but they've been commonly used for a while and could allow the player to pick precons with more information. I'd offer to do that work eventually, if the idea looks interesting to everyone.


TODO: need to confirm this choice for returning in-sample & out-of-sample predictions. If this one is validated, we need to extend it to the other predict types


![new error1](https://cloud.githubusercontent.com/assets/12948508/8234536/d9726ea8-15dc-11e5-9d11-e2c073f20a9e.png)


`comments/create.html.erb` can be safely deleted, its not used.


`comments/new.html.erb` can be safely deleted, its not used.


it allows the user to specify the name of the DB service that will be created.  This in turn allows users to avoid name conflicts between services.


`<p>` tag on line 26 might be outputting more paragraph tags than you intend.


Completed this portion on 06/18/2015.


No intention to be rude.

We don't use multibit-hd nor intend to, it was just a small change to demonstrate that it may be easy to stop paying the BRIT fee but in reality it would be quite an amount of work to remove properly and cleanly and furthermore that the multibit-hd guys have done quite a lot of work in packaging and supporting this software and any "forker" would need to commit to the same release quality to have it used by people.

See http://www.reddit.com/r/Bitcoin/comments/39694x/wtf_multibit_hd_why_i_have_to_pay_a_fee_to_you/cs0tt1g and our reply http://www.reddit.com/r/Bitcoin/comments/39694x/wtf_multibit_hd_why_i_have_to_pay_a_fee_to_you/cs131hj


Ok, I see. Thanks


Another possible memory leak. I would use std::string/basic_string instead, as memory doesn't need to be handled manually.


This is for PR #185


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/ec3e0a443c1eb5e4bfe25894c963a6cb57ae5f9f


merging basho/riak_kv/bugfix/kv1142 = ec3e0a44 into borshop-integration-1144-bugfix/kv1142


basho/riak_kv/bugfix/kv1142 = ec3e0a44 merged ok, testing candidate = 7b2cf6a9


Metroid Fusion and Metroid Zero Mission now have digital sound. DMA sound quality still has issues (so it's not just these games that still sound bad).


Reason that you want both new and originals is so that you can make decisions on whether or not you want to invalidate based on the docs values being different.


Published to http://ui.edev.d2l.com/80_d0549125a3b4174d8bc7614306d8d2acf10d3b1e


Well, that was intended as a handle of the iterations within ca(). Like: If an update function needs information about which timestep we are currently in. But I think this is not required at all.


Check if `CommentsController#show` is used in the app. If its not, it can be safely deleted.


Could we conditionally not compile this.  I would be nice to save the space on systems which don't need it.  Ditto for its uses in the non-prune function places.


The underlying method would bail out immediately if EnableCache was set... and the method would probably get inlined. So we probably aren't saving any work.


all looks good.
what i would say is rather than use comments in the tests, we should leverage (Describe,Context,It) to relay the behavior we are testing, rather than sprinkle comments around. Leaning more heavily on the BDD structure makes comments redundant and allows us to just not have them.


As `CommentsController#destroy` doesn't render a view template, consider using local variables instead of instance variables.


@srih4ri _a testing Commit comment


@elemoine any reason why you fixed version of typeahead to 0.10.5?


Thank you for taking the time to explain the context. My apologies for being rude to *you*.


I kinda assumed you didn't (but wanted it documented in the issue you open), because if you did the change in 1.13.1 to 1.13.2 wouldn't have affected you because it only changed the default (and removed the warning).  The default changed from `@index` to `@identity` in 1.13.1 -> 1.13.2, there must be something going on in your setup that triggers this error when using `@identity`.  That is the thing we need to track down.


Nice, I was just going to poke you about it, but you were faster apparently.


@simianhacker This is where the majority of the discussion @w33ble and I had centered. Starting with the change from `provides:` to `exports:`, down to the requirement that references to files follow the browser/requirejs module id spec, and the support for globs.

A note about how globs work:
While plugins are discovered, their exports will be mixed into a pool of module ids that the server know about. In that pool will be module ids that point to other module ids (like the visualizations export below) which we are causally referring to as "aliases".

All of the strings in the app's `uses:` array are considered to be globs which will be run against the pool of known exports. If any of the module ids matched, aliases are expanded after being matched, and any glob that doesn't match at least one module id is considered an error. In future versions these globs could support optional `?glob` and negative `!glob` formats, but that's not official for now.

This list of module ids is shipped in the config for inclusion by the runtime/app main file.


rax retry tests


This change is causing a directory called "test-data,https://github.com/galaxyproject/galaxy-test-data.git" to get created.  I'm not sure that's the intention of this fix.

default_galaxy_test_file_dir is assigned to test_dir, then never used again.  test_dir is used to create the testing directory.


LGTM


Also the parameter name is misspelled.


ここは、お兄さんではないでしょうか？


I don't really understand why this is happening. liboctave-dev [_doesn't_ explicitly require the liblapack* packages](http://packages.ubuntu.com/vivid/liboctave-dev), and all the dependencies should be met by the atlas packages. Regardless, you don't need to uninstall anything to use nmatrix, you can just use [update-alternatives command](https://wiki.debian.org/DebianScience/LinearAlgebraLibraries) to force the use of the ATLAS version of liblapack.so.


Landed, thank you!


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Is it a good idea to quietly convert non-gussian priors to uniform priors, rather than notifying the user that he/she is trying to do something illegal via an exception?


ох


Any particular reason we want to set the content as dirty when retrieving a document?

Is it done so that, in case the document does not exist and the `isNew` flag is set we also end up with a `contentDirty` flag? If so, shouldn`t this be done in the store instead?


I think this is an appropriate thread to talk about sealed and prebuild decks. Thanks for the feedback.

Sealed mode is a nice middle ground, not sure how hard/easy it would be to get the AI to generate a decent deck from the sealed card pool though. That has to at least work reasonable for sealed mode to be feasible. The other problem would be it would take a while to setup before you can start playing, which doesn't quite align with our quick to play style. However, I think there would be users that would appreciate such a feature so we would include it if it was developed.

Regarding prebuilt decks, it certainly could do with a better organization so that more information is provided to the player to make a choice.


:+1: this got us working again. thanks for the quick fix.


reack


Allocating an extra buffer is unnecessary, you could simply cast the EventType as short, like that:
InBuffer->ReadInt16((int16_t *)&eventType);


Well done on getting the vote spec together right, its looking good!

The vote variable names are a good start. A minor improvement would if their names could reveal their intent a little more clearly. For example `up_vote`, `down_vote`, and `invalid_vote`.


vec


Published to http://ui.edev.d2l.com/81_6bfe1caff4ac455677ca42e77a6c48496e6f3ac3


set


Si la _edad es mayor que 0 haga edad = _edad, si no, haga edad=0;


We plan on supporting media and service messages very soon, so these should be processed as well.


s/by/with/


Slow progress so far.  New to python, sqlite3, wxpython, and coding in general.


With blank: true, the default finalizers won't be executed. See https://github.com/DigitPaint/roger/blob/master/lib/roger/release.rb#L35 :)


moving navigation menu to right is not important but everywhere a text exist should be RTL because if there is a single english text between persian text , people can't understand what is written .

(eg . translation of host(B) wildcard(A)is ( A B) .
now imagine i put the english word of wildcard in there for better understanding then if LTR is applied it would be shown like this ( B wildcard A ) but it should be (A(wildcard) b) .)

also dot , comma , parenthesis and etc are tricky , they might be shown okay in a sentence and they might not .

a question : is there any other way to change a extension lang instead of changing chrome lang !
like a drop menu to select lang or changing attrib in unpacked extension ?


s/previous value of result/result/


:+1 Good job


kill it


No worries.  Thanks for all your helpful feedback.  I think the latest commit is a much better step forward, if nothing else.


Great


Hello from FULL_TEST_CASE :p


empty string


YAY!


s/on success//


@varund7726 why nightlies lol
:P


The idea was that package.json wouldn't be required and therefore all of our plugin needs to be listed here. Also, didn't really want to have multiple config/package/info files


@spencerfinnell The is_home() check in your thumbnail code was a problem for me. Also, there is code which looks at the parsed form_data rather than posted keyword etc vals. Not sure of best action there, but I'd like to pass less form_data through if possible.


1687-OrphanedConnection (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)


Published to http://ui.edev.d2l.com/82_c958d227c1bc6bd4ddce26cafd182ff065006dce


I'm surprised you liked how I did the previous icon??? XD


It's just a convention I've picked up that I like from another project.
I also consider it a NIT and strongly believe is following the maintainers prefs. :)


The response code doesn't matter. You should be closing it on any non-error.


Published to http://ui.edev.d2l.com/83_70c531051d29ef23fae795894f7d588ba939f924


When my JSON API 1.0 request is loaded it goes a bit wrong at this point because the `type` is actually located in `resourceHash.data.type`. Is this expected?


Автоматическое сжатие осуществлялось модулем [mod_deflate](http://httpd.apache.org/docs/2.4/mod/mod_deflate.html), который включен по умолчанию при установке, например, в Ubuntu 14.04.


Oh, alright :)


When you update xorg packages, also update the file where they're generated from, please.


I was able to upload the image, but it didn't go to that specific folder. Will it still appear when the site is updated?

Matt Applegate, Ph.D.
Assistant Professor of English
Molloy College
http://mapplega.com

> On Jun 16, 2015, at 10:29 AM, Graham Higgins <notifications@github.com> wrote:
>
> Django handles the management of static images for us (like uploading to S3), so there's a bit of extra syntax involved.
>
> Images can actually be added to GitHub, in the manifestos/static/manifestos/images folder.
>
> Then the image tag should look like this:
>
>
>
> Django will insert the appropriate src link automatically.
>
> —
> Reply to this email directly or view it on GitHub.
>


DesignDaysGridView (evanweaver) - 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)


ack


I like the `publicPath:` config, but by putting the public directory next to the `main:` file instead of the `kibana.config.js` file it adds automatic support for using putting the whole source for your plugin in a src directory during development if you want.


It's not anymore; old comment?


那请你仔细想想吧


The caller of this method is responsible for closing the hijacked connection if we return one. The defer is for when we error before returning it.


That sounds like a bug. resourceHash should be the Resource Object (in JSON-API terms) so there shouldn't be a data key in there. Would you mind opening an issue?


No particular reason.


fixes #12


I agree with you. I did this because I thought the Fisher could only
handle gaussian priors, and I'm not too sure how the logging/warnings
functionality works in PISA

On 18/06/2015 16:20, J. L. Lanfranchi wrote:
>
> Is it a good idea to quietly convert non-gussian priors to uniform
> priors, rather than notifying the user that he/she is trying to do
> something illegal via an exception?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/steven-j-wren/pisa/commit/6c389a170d75f71d1389a7d008c6a7cf6130733a#commitcomment-11749672>.
>


Consider wrapping these tests in a `describe('Views', function() { ... describe('DetailsView', function() {... ` block. See the modeling tests or the other tests for an example.


That was the idea of the `kibana.plugin.js` file: The `kibana.plugin.js` file would serve as the package.json file for kibana plugins.

Basically copying the idea of a `webpack.config.js` file


Thanks @westcripp just now seeing this haha


Removing the original Copyright notices is very community un-cool act and also just plain not allowed -- see http://opensource.org/faq#preserve-copyright-notices for the reason why.

Changing the license from MIT to Apache is also troublesome, that type of change is only done by the person/group who owns the copyright.


but you didn't though, did you?


It would be nice to have a more general title and description in these metadata records, the current ones are copied from another place.


Also I should point out that I didn't mean to commit these changes I have made to pisa/utils/params.py. They are not required for the Fisher fix and were just something I needed to do for my personal 1D parameter scan script. Unfortunately I don't know how to undo a commit.


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

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


Old comment? The json now contains the `record_id`


@teleivo , yes please do that


现在改好了

在 2015-06-18 21:51:57，"Zhehua Chang" <notifications@github.com> 写道：

很高兴看到你用面向对象的特性。
但是请问你为什么要放弃这样的设计去使用nested class呢？
那种形式在标签很多结构很复杂的时候会大大降低代码的可读性和可维护性。
不知你是否认同我的看法。

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


@cocafe, having whole git history makes a bit easy to port some changes. Unfortunately, even LK3.1 doesn't boot up, when our device specific changes are rebased onto it, not to mention about LK3.4.


It is worth noting that these "boilerplate" lines are necessary because of a bug in oboejs https://github.com/jimhigson/oboe.js/issues/64 where both .done() and .fail() are being fired. For the time being I would suggest to extract these "setup" lines instead of replicating them in all our Actions.


ack


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2129


Okay, apologies then. I saw it in Slack and clicked the link due to your e-mail earlier, I didn't realize it was a personal-only commit and I wanted to make sure the changes didn't propagate.


Are you confident on this?

When there are multiple VMs in the same Cloud Service, there are multiple `Instance` elements, and although there is anecdotal evidence that the current machine is the first `Instance` tag, I've seen no documentation supporting that.


I misread the type of the scoped lock at https://github.com/rec/rippled/blob/06f3a73197a1a5b2eb41d84c8b1af7dd4a55aa69/src/ripple/app/ledger/impl/LedgerMaster.cpp#L1014 as `ScopedLockType` (it is actually `ScopedUnlockType`).  So this lock is correct as written.


Ok. Some things to watch out for: .net 2.0 in the old, 4.5 current. Also, have you made sure Bill has given the server you're testing on permissions to do with AD?

> On Jun 18, 2015, at 07:26, hollywoodheinz <notifications@github.com> wrote:
>
> @Saurutobi So i am considering taking the old website defualt page and just making that the login page and just changing a few of the names to make it fit better because for the life of me I can not get the active directory to work on the live server with my login page
>
> —
> Reply to this email directly or view it on GitHub.
>


What was this for?


Hello from FULL_TEST_CASE :p


can it not simply be a `val _objectMapper = new ObjectMapper`?
Since we're not exposing configurability in it anyway currently


@Kontakter, поправила код. Посмотри еще раз


I needed this change to handle one single source. Do you want me to change also in other versions?


I'm not sure we should bother with this step. The blob post we're working from discussed some flipping here, as described below, but again, we have no concrete documentation to support that, and *within the scope of the tool*, either the original or the mucked-with version is unique, so maybe we just skip this for now?


If that is the case, then I'll leave it in for now unless we actually do think it has no practical use (we haven't documented it and I don't see the value in having it for the global config)


The changes to Fisher.py and FisherAnalysis.py were changes I meant to commit for making the Fisher work with the new priors. But if you say you've already done it then we can just ignore this branch.


Testing to see if we can make case insensitive regexs


Hey Mike,

On Thu, Jun 18, 2015 at 5:34 PM, Mike Taylor <notifications@github.com>
wrote:

> Removing the original Copyright notices is very community un-cool act and
> also just plain not allowed -- see
> http://opensource.org/faq#preserve-copyright-notices for the reason why.
>
I agree this is not great from a community perspective. Where did I do that?

>  Changing the license from MIT to Apache is also troublesome, that type
> of change is only done by the person/group who owns the copyright.
>
Which we do. Whatever code we haven't written has been contributed by
people who have signed the BlueJimp contributor agreement:
http://bluejimp.com/bca.pdf

Emil

--
https://jitsi.org


> Note that if the current deck size is less than 40, it should be increased to 40.

If using the deck editor from the duel setup screen but if running from the main menu I think it should be 60 since that is the minimum size of all formats in the legality tab. Otherwise, every time you generate a random deck the legality tab will be pointless until you have the minium number of cards (ie. 60) in your deck. Perhaps @ShawnieBoy can explain how he uses the deck editor since he was the one who originally requested the legality check feature.


really hacky.... use a blank str for top level...


@Stasik0 @jpfr: As per your request the MethodCallManager is gone, as are linked lists in NodeAttachedMethods. The later is now a part of the method node (no UA_malloc() or _new() when creating nodes). Attaching/Detaching methods to nodes now swaps them in the node store and also sets/clears the executable flag - userExecutable is left untouched. As discussed on IRC, calling a method now returns a StatusCode (identical to that included in the outputArguments struct).


android 4.4 a 3 arrays pour wifi_ap_security, donc ce commit empeche probablement un Force Close.


Would the level-one heading be the title for the document, and thus no more level-one headings will be used? Or will it be the title of the first section, and other top-level sections (within the document itself, which is on the level that the title is on) will also be able to use a level-one heading?


#4 render the parsed toc. (btw: in fact we can do the render when parsing .... so we don't need the intermediate object.


This does not work as written.


Oh ok, I hadn't noticed this was a commit to a comment. The Philipp's contributions are acknowledged on the front page of Jitsi Meet ( https://jitsi.org/meet ) and also on our team and contributor's page ( https://jitsi.org/team ). The new license file simply does not contain any mention of any company.


IMO this is ambiguous, as the `--instance-name` is derived from an `Instance` tag in the metadata, and what you're referring to here is the `incarnation` tag in the metadata.


On peut mettre 2 comme limite pour critical ? Ca laisse un peu plus de temps pour agir


@yuyuyu101 i am afraid that, we are enabling ceph to run as a normal user, see #5015 . is there anyway we can move this to into the init script or in the form as a note in the document ?


Could we have that as a configurable option, rather than hard-coding two streams ?
Ideally, we'd also want to create up to `k` streams, where `k` is the number of distinct slaves.


This doesn't look good..


Noted. Will apply to further patches. Thank you! :bow:


Well, Fisher is working for me in my new_prior_implementation branch, but that doesn't mean I didn't miss some use cases...


I'm not sure but I think that this installs Devhub itself to the local `~/.m2` folder. I think `mvn compile` should be enough to trigger maven to download the dependencies to the local repository. If not, we might want to skip the tests for this stage `mvn install -DskipTests`


wut? I don't get it... why do you *enable* a npc and *register* it (in npcs_by_name) in order to delete it?


Can you explain why this response had to change?


Other uses of `nodesForXPath` in the code will usually assert.  It also appears that `NSLog` is used during the parsing stage.
Since this parsing *always* happens, I wanted to make sure we always make forward progress.
I can certainly use and `NSError` and `NSLog` it at the callers below if that is what you wish.
Makes no difference to me.


Maybe add a pointer to https://projectlombok.org/download.html


Hello from FULL_TEST_CASE :p


pointless function


Radi, welcome to the glorious world of Git and Github! Regarding your last sentence I'm going to let you in on a little secret: you _can't_ mess things up here, because it is a trivial matter of reverting if things go wrong, or we can just patch things to fix them as time goes forward.

So dig in, put caution to the wind, and give it your best shot. (at least that's my suggestion). Good luck!


slm


I see it renders properly, but shouldn't the numbers increment in the file as well? Or is this to prevent future counting issues?


A couple of changes:
   i) Changed return type to Type.BOOL
   ii) Added method to TypeList.addNativeMethods()
   iii) Added test case.
   iv) Added entry to documentation (gh-pages branch)
   v) Changed loops by Java's List.contains(Object)

Thank you for your code


Why this class is named MutableQualityGateHolderRule and not QualityGateHolderRule ?


Thanks for your contribution! Can I expect patches to this PR soon or is this ready to merge?


ces chaines diffèrent d'android 4.4


No worries, it doesn't look nice at all without context.


The if statement can be avoided by using numpy's squeeze function:

``python
vis = ne.evaluate('sum(ebk,3)',
    {'ebk': self.compute_ebk_jones()}).squeeze() \
    .astype(slvr.ct)
```


toLowerCase here


toString not needed


ok thanks!


Would be good to add this to the readme regarding pushing ?


That's a good idea, and I made a run at it.  `advanceLedger` is implemented as a lambda.  I'm not sure about where the standard sits, but at least our version of Visual Studio doesn't permit default arguments on lambdas (although clang does).  So, rather than insist that every call to `advanceLedger` include the argument (typically a `1`), I left it the way it is.  There are only two places where I do multiple consecutive `ledgerAdvance`s anyway.


I think you don't need to do that. It's written in the doc SPI that there can be multiple calls to createMetrics with the same SocketAddress and the metrics impl is responsible for reconciliating.

So if you decide to keep it this way you need to modify the JavaDoc that says so and update the test in Dropwizard that tests that because it will likely fail as the behavior changes.


Montenegro only allows same sex marriages if both partners are British. For this test to work I need to use the SS outcome that is most likely to allow SS marriages, thus showing the appointment booking link I'm looking for.

I'm aware this test is not ideal, so I'm open to suggestions :)


Aren't there still some old packages with uppercase letters in their names? Not a HUGE deal, but yeah ;p.


I am on it :+1:


fixed #2


Mind getting rid of this line?


uhm it should probably go somewhere in the docs but I need to write a thesis, you do that minicommit?


I have added mako hotplug to my htc one m7. and I get reboots on screen off with mako hotplug. same  snapdragon 8064 though. any ideas as to why? last_kmsg is not really helpful on this one


I saw you actually changed, the test in Dropwizard. We still need to update the Javadoc about this behavior change.


Thanks for highlighting this. I'll make the appropriate change in the four-thread-pol branch, since I'm making all the v4 changes in there.


Hello from FULL_TEST_CASE :p


it wont show up in the docs, you have to keep the original and add the full script afterwards... (it's a drawback of duplicating the lines, I'm not sure whether it can be avoided)


Could you explain this a bit more? Are you talking about link `id`s, as described in the Markdown spec? E.g.:

```Markdown
This is [an example][id] reference-style link.

[id]: http:/example.com/ "Optional Title Here"
```


Here's an example that shows the context preserved with in `_x`:
```js
function __$b20(__$ctx, __$ref) {
    {
        var _$0vBlock__$126 = __$ctx.ctx.block, _$0vElem__$127 = __$ctx.ctx.elem, _$0block__$128 = __$ctx._currBlock || $$block;
        __$ctx.ctx || (__$ctx.ctx = {});
        var __$r__$130;
        var _x__$131 = __$ctx;
        var __$l0__$132 = $$mode;
        _x__$131._mode = __$l0__$132;
        $$mode = "default";
        var __$l1__$133 = __$ctx._links;
        _x__$131._links = __$l1__$133;
        __$ctx._links = __$ctx.ctx.links || __$ctx._links;
        var __$l2__$134 = $$block;
        _x__$131.block = __$l2__$134;
        $$block = _$0vBlock__$126 || (_$0vElem__$127 ? _$0block__$128 : undefined);
        var __$l3__$135 = __$ctx._currBlock;
        _x__$131._currBlock = __$l3__$135;
        __$ctx._currBlock = _$0vBlock__$126 || _$0vElem__$127 ? undefined : _$0block__$128;
        var __$l4__$136 = $$elem;
        _x__$131.elem = __$l4__$136;
        $$elem = __$ctx.ctx.elem;
        var __$l5__$137 = $$mods;
        _x__$131.mods = __$l5__$137;
        $$mods = _$0vBlock__$126 ? __$ctx.ctx.mods || (__$ctx.ctx.mods = {}) : $$mods;
        var __$l6__$138 = $$elemMods;
        _x__$131.elemMods = __$l6__$138;
        $$elemMods = __$ctx.ctx.elemMods || {};
        $$block || $$elem ? __$ctx.position = (__$ctx.position || 0) + 1 : __$ctx._listLength--;
        applyc(__$ctx, __$ref);
        undefined;
        __$r__$130 = undefined;
        $$mode = __$l0__$132;
        __$ctx._links = __$l1__$133;
        $$block = __$l2__$134;
        __$ctx._currBlock = __$l3__$135;
        $$elem = __$l4__$136;
        $$mods = __$l5__$137;
        $$elemMods = __$l6__$138;
    }
```

TODO
- applyc(_x, __$ref) instead of applyc(_x, __$ref)
- restore __$ctx from _x post applyc

then theoretically it should fix https://github.com/bem/bem-xjst/issues/63


I was testing on the live server so it does have active directory permissions already set up, which is why I am testing on it but when I was trying the simple validation nothing was happening, but yeah everything is still valid on the default page for the old website ill just go though it again and make sure


Don't you need to include `UV__POLLRDHUP` here as well?


I've also sent an RFC for an e-mail discussion.


I'll check this with external libraries. I already tested a code with external libraries and it compiled without problem, I think it can be problem with W7, I'll keep looking, I'm trying to understand how the plugin works


16 is probably a bad value, it's POLLHUP on the BSDs.


Not sure tbh. Mako is beautiful because its so simple. Maybe check your
drivers for powersuspend and stuff that its interacting with.
On Jun 18, 2015 10:48 AM, "cannondalev2000" <notifications@github.com>
wrote:

> I have added mako hotplug to my htc one m7. and I get reboots on screen
> off with mako hotplug. same snapdragon 8064 though. any ideas as to why?
> last_kmsg is not really helpful on this one
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Orbit-/Vindicator-flo-aosp/commit/7d5037a09a6a1ec1e8dd5d23a02684831423099a#commitcomment-11750427>
> .
>


Fixed by a subsequent commit.


Your change is ok.

However, looking at this piece of code, it doesn't seem to behave exactly how I asked Stephen it should.
I will propose a correction.


@cooldeeparmar It seems an issue with your yii installation advanced app installation.

See this question: http://stackoverflow.com/questions/25882271/cannot-update-yii2-via-composer-bower-asset-jquery-could-not-be-found

If you are using Composer to upgrade Yii, you should run the following command first (once for all) to install the composer-asset-plugin:

```bash
php composer.phar global require "fxp/composer-asset-plugin:1.0.0"
```

You also need to add the following code to your project's composer.json file:

```json
"extra": {
    "asset-installer-paths": {
        "npm-asset-library": "vendor/npm",
        "bower-asset-library": "vendor/bower"
    }
}
```


The mobile wallets still have this feature (but it appears to not work). What reason was there for removing this backup method? What is the best way to back-up the mobile wallet.


I guessed it was something like that. I think it's fine to leave it as it is, but perhaps add your comment to the commit note to make it clearer.

Another idea would be to extract the `partner_british` response into an explanatory local variable, giving it a name that explains that the exact value isn't directly relevant, but it might be hard to come up with a good name.


another common option is to not store fullstart, but to instead just do:

```createNode(SyntaxKind.JsxElement, opening.pos)```


@cbrenner04 Nice work :smile:


convenient for someone to know how to configure?


The diff may be weird.  But i don't see you using tagName.


Use @api.model and let the `id` argument.


You replace too much of the comments here...


Ouaip, mais c'est pas forcément évident qu'il ne restera plus que deux mouvements après.
On peut faire un état intermédiaire genre "almostCritical".
Enfin de toute façon vu que le bombes tremblent de plus en plus, on sent rapidement l'urgence de la situation si on spamme pas le clavier.


todo should obviously read:

TODO:
- `applyc(_x, _$ref)` instead of `applyc(__$ctx, _$ref)`
- restore __$ctx from _x post applyc


I'm talking more specifically about application permission to access AD

> On Jun 18, 2015, at 08:51, hollywoodheinz <notifications@github.com> wrote:
>
> I was testing on the live server so it does have active directory permissions already set up, which is why I am testing on it but when I was trying the simple validation nothing was happening, but yeah everything is still valid on the default page for the old website ill just go though it again and make sure
>
> —
> Reply to this email directly or view it on GitHub.
>


Nit.  We (and the formatter) put spaces around the |s


> Sealed mode is a nice middle ground, not sure how hard/easy it would be to get the AI to generate a decent deck from the sealed card pool though.

But is this not a completely separate AI from the that required to play a duel? Just because you are using MCTS as your opponent does not mean it has to use MCTS or whatever to build a deck. Perhaps it does not even need to be AI but some sort of ingenious algorithm. What is important is how it is perceived by the player. It seems to me that something like this could easily be developed by someone who maybe wants to experiment or learn AI programming without impacting on the existing code base or even the UI. Once you have a working algorithm it should be fairly straight forward to bolt the UI on top.


NodeArray<JsxChild> ?


@taboularasa Yeah that's a pro, but on the con side it adds more for us to maintain in `suspenders` as `rspec-sidekiq` versions change and this API may change...


Good idea


Hey, you can't just change our format tags. This change must be reverted.


Ah thanks!


Would be nice to maybe extend oboe with a .success() method that does that HTTP status check boilerplate until the bug is fixed in oboejs ?


thanks @michaelwehrley not terribly difficult


Will send some data over when I next get a chance to work on this
(hopefully over the next couple of days).  Might not be the best code as it
was very late at night when I hit the problem, so it was a pretty rushed
adjustment.  I'm trialling it as a storage solution for a music library so
the code that triggered it is pretty complex, I'll try to whittle it down
to a small example when I get a chance and send it.

On Thu, Jun 18, 2015 at 3:08 PM, Tim Savannah <notifications@github.com>
wrote:

> Thanks! Glad to hear you're using it!!
>
> I'll have to think this over, it seems contrary to what's going on in the
> python3 "else" statement, though I am pretty tired atm. Can you provide
> some example data that reproduces the issue? kata198@gmail.com -- put
> "Fuzzy Penguins" in the subject line.
>
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mtskelton/indexedredis/commit/60f5e20fb5f62947170073a370332f25e38d39a1#commitcomment-11747830>
> .
>


http://majuric.org/software/cudamd5/  majuric.org is currently unavailable. Sorry. Try again later.


otherwise its will sit on the map forever. and chew up the number of NPCs on a map.


@emistry
Changing ASCII to euc-kr?
Intended?


true, you decided


Absolutely, good thought.  I originally wanted to avoid adding this entirely by using `zfs_iput_async()` but I ran in to lock contention problems when pushing everything through the taskq.  But wrapping it with a conditional is almost as good.


Remove it!


Is this test name still correct? I'm not sure it should still mention "notary public", should it?


:fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire: :fire:


We should check whether this is also used by Pool.join and DR operations.


It would be nice if the comment documented the consequences of the argument choice.  What is the difference between `howMany` being `1` vs `2`?


Dude! Patch your sh*t! :facepunch:


Redundant `begin` block detected.


Ambiguous negative number operator. Parenthesize the method arguments if it's surely a negative number operator, or add a whitespace to the right of the `-` if it should be a subtraction.


What is the reasoning behind having explicit nested operations for the first 16 elements and then switching to a for loop? Why not provide inlined definitions up to 16 as well? Just wondering since it's not obvious from the code or the comment.


I'd rather use

```
> **Note**:
>
> To be able to see rultor's logs - you need to be added as a "Reader".
> Please follow the instructions at
> [rultor's docs](http://doc.rultor.com/reference.html) to find your URN and
> add it in the .rultor.yml file in the "readers" section.
```


thats the note style we use in other locations too


test


Its how many operations you want to trace


I think it might be worth introducing a unit test for this method...


This is great, but it doesn't exist as a pull request over on the [official spork repo](https://github.com/sporkrb/spork).

All the :+1:'s in the world aren't gonna help if the author doesn't make a PR.


:ok:


Philipp's contributions are not the issue here (tho it is nice to have him called out in the README)

The issue is that you have removed the legal Copyright notice from the project *completely* - if your removing it from the license file for consistency then it needs to appear someplace else.

The second issue is that, according to the PDF that you gave, everyone who contributed to the project maintains their individual Copyright, so a licence change would require that all contributors agree to that change.


@joranE I've made a first stab at actually documenting `layer()` here. It's pretty rough but should be better than the nothing that was there before! If you have an ideas, or would like to contribute more examples, please let me know.


Made some fixes in cd96b93, above all I reverted that bogus change of format tag.


Changed it up.


rax retry tests


"Sets TLS" sounds strange. Was it supposed to be "Sets up TLS"?


doesn't it leak here, imho you need to delete replacement


>Perhaps @ShawnieBoy can explain how he uses the deck editor since he was the one who originally requested the legality check feature.

The main times I've used the legality check is in relation to decks for Firemind. Firemind takes a solely constructed view of Magarena. There would be no need for legality for 40 card decks, as they would be generated randomly from a pool of legal cards.


This doesn't seem to be called from anywhere.   Is it just a comment?


I believe it's already working as described. There's an example in the spec
that demonstrates this: normal dependency 'dependency.js' and devDependency
'dependency3.js' both depend on 'depdendency1-1.js', so it appears as both
a 'dependency' and a 'devDependency'. I ran this on a couple of arbitrary
node projects from github and adding devDependencies as an ignored group
filtered out stuff like mocha and its dependencies.

2015-06-17 20:33 GMT-07:00 Jacob Maine <notifications@github.com>:

> Or maybe this works already? I'm a little unclear.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LukeWinikates/LicenseFinder/commit/a41589f329d2e2e3447f437496d6e4a26d302854#commitcomment-11738592>
> .
>


no test for `throw new TypeError('Expected a number');`. :cry:


I have tried as you wrote
Still taken only version 2.0
And I tried to production server - also downgrade to 2.0

any @ dev in composer not helped


Fair enough, will add :).


Oh, i thought it was all done by hand here, I'll do that if not already done

On Thu, Jun 18, 2015, 08:31 Vladimír Čunát <notifications@github.com> wrote:

> When you update xorg packages, also update the file where they're
> generated from, please.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/e4316b8c17121042fd258df6df2fdbc5618c2b72#commitcomment-11749971>
> .
>


This got fixed in later commits.


Initialize* (so we're not mixing imperative with descriptive).
(optional) If you're willing to change everything to descriptive, I'd actually prefer that but I'm not sure how established the convention is of imperative vs. descriptive comments.


@sunbiz done :+1:


you noticed the `> ` in my comment? Which adds the quoting style making the thing even noteyier.


This should probably use pkgutil.get_data to load the JSON data: https://docs.python.org/2/library/pkgutil.html


They should be at the front of the queue.  I don't want to depend on it, though, as weird stuff has been making its way into mainline NDT recently.


Doesn't your IDE tell you "Can be simplified to 'if (!classicLifeStone)'"?
Also, the variable you're querying is not in scope, this won't compile :p


so you're saying leave it in?


那个extract_string里面的ch初始化最好使用'\"',另外我实在不理解把key_name字符数组从50减到40会对内存泄露有什么帮助,反而如果键名太长会导致存不下键名的问题出现,所以我不建议修改掉;


Doesn't work on my Mac OS X 10.10.1:

ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [all] Error 1


> ~~layers of objects~~ *layers of concern**


BURN IT OUT


@calumhalcrow if you want the keys to be hard coded let me know. but for know i have a feeling that it was not intentional to expose them as we discussed [on the story](https://www.pivotaltracker.com/story/show/95859342) about this.


我重新更新,你试一下那个release里面的代码.如果可以的话,嘻嘻


fetchone() may return None and [0] will be invalid.


I can't figure out what `tt` stands for. Suggest `current_time`


Typo in commit note: "appointment booking likes" -> "appointment booking links".


Его, в общем-то, надо вообще выпилить. Без нужды я бы не обновлял


Changed.


So I never actually verified if `verbose` is a `"logical"` so `if(verbose == TRUE)` and `if(verbose)` are different (the later evaluating to `TRUE` if `verbose` is a numeric like 1). I've since updated the `validate_true_false()` function to make sure that `verbose` is a logical. So I can go through and change `if(verbose == TRUE)` to `if(verbose)` now.


Done in https://github.com/kevva/is-negative/commit/521047f5bec425027c348ead6c06a450df4b0458.


As much as I hate using $dynamic tags in non-dynamic formats, we have no choice until dynamic can handle tag aliases or something.


Is now
` * Sets things up so that the server can accept incoming TLS connections.`


@rodrigopavezi interesting. Will keep this in mind and modify when we get the build working. Thanks!


Is now
` * Set things up so that the server can accept incoming TLS connections.`


I did that, they chose not to go with it


Yes, you're right


:smoking::sunglasses:


Hi @defunctzombie, this change was made more than a year ago. At that moment Copay was a prototype and we switched to force a backup before start using a new wallet. The project have evolved a lot since then.

I'm sure @matiu, Copay's team lead, can help you out with this.


I'll use 0x200 that apparently is free


I think we should add "from __future__ import absolute_import" all over the place so that we never have relative imports. I've seen relative imports cause pain later in a project after refactoring efforts and incomplete code coverage testing.


We'd prefer VM not move during the VDI.pool_migrate (old behavior).


@alanfx Congratulations on 1000th commit!


Right that's fine, but I think we should add comments to explain that


Left over from the Heroku days.


Would the GPU formats also profin from implementin their own source() method, to reduce memory pressure for high number of hashes?
So far, only very few CPU formats implement source():

```
$ ./john --list=format-methods:source --format=cpu 2> /dev/null |grep "^M"|cut -d: -f 2-
   LM [DES 128/128 AVX-16]
   dynamic_0 [md5($p) (raw-md5) 128/128 AVX 4x3]
   dynamic_2000 [md5($p) (PW > 55 bytes) 128/128 AVX 4x3]
   NT [MD4 128/128 AVX 4x3]
   Raw-MD4 [MD4 128/128 AVX 4x3]
   Raw-MD5 [MD5 128/128 AVX 4x3]
   Raw-SHA1 [SHA1 128/128 AVX 4x]
   Raw-SHA1-Linkedin [SHA1 128/128 AVX 4x]
   NT-old [MD4 128/128 X2 SSE2-16]
```


Use the standard license header comments, see any other file.


Aligns these error messages with others


Included for readability


Either work your magic or remove the comment


thanks, yes you're right here too. :+1:


Fixes #2


Fixed. To the degree there is a standard, it's imperative.


Deus tá vendo essa linha de código!


Added arguments to choose between verifying a list of emails in a .txt file or a single email.
Also added exceptions to prevent errors.


@biergaizi 我倒是不大会用 `add -p` 技术，要指定行号的时候就去 GitHub for Windows 里面忍受着卡到爆的 WPF 点点点了……


pourquoi pas yield le thread????


Pls. add your name here also.


It's not fine at all, but I will add the comments.


@mcimbora BOOM! Yes!


PHPCS: Whitespace found at end of line


:ok:


Which is to say: I too feel the pain of too many globals.


ack


Pls. remove PA-specific text here & translate.


Indeed, my bad !


That should also work like this (no need to `cd`):
`ln -s dash rootfs/bin/sh`


Lo de que el vértice para los muones sea firstGoodVertex lo habíamos puesto en su día para que se coja el primer vértice que pasa unos mínimos de calidad. Creo que es mejor dejarlo como estaba.


the url to the styles.css it not correct, and it shouldn't be in the server folder


Isso aqui não funciona, na real não sei como deve ser feito. Fiz Ctrl-C + Ctrl-V do Livros. da uma olhada @felipeas


I cannot use this way because in case of one of the other dimensions (like nchan) is 1, it will be squeeze() as well


Yes, but I don't care. Very few and they'll be removed eventually. I'd rather have a convenience for the 99.999% case.


It looks as if you've added the `:turkey` key & values to this file twice. Was that intentional?


Please see #1742.


These are I²C-specific, so I'd make the messages, so I'd use that in the warning messages.


Was there no existing test for Peru?


You probably don't need to delete then drop the table. Just drop it.  Or at least replace the delete with table truncation.


Outra obra prima. O @felipeas  eu queria fazer IEnumerable<SelectListItem> com os itens adicionados no braço, mas não achei métdo Add, Push ou similiar.
Tive a brilhante ideia de criar uma List<SelectListItem> e transformar para IEnumerable<SelectListItem>
Em runtime isso não funciona. Me dá uma mão.


I don't have a strong opinion. I can resubmit with that change if you want


static_assert(max_count > 0, ...)


changed


Yes, I agree.


Stray semicolon


Refactoring is still on the go.... Not finished.


Since `i` is not used in the loop below except for the indices, it might be a better idea to increment it by `2` instead, and avoid multiplication in the array subscript expression.


A coding style tip, try to hit enter before IntelliJ draws that line on the right. For this line, an Enter would be nice before the .ignoreFirst...
Easier to read ;)


You can specify a subset of axes to squeeze on, in this case, we only want to squeeze on axis 3.


:ok:


On 2015-06-18 09:21, Sam Thursfield wrote:
> I don't have a strong opinion. I can resubmit with that change if you
> want

I'm happy to merge as is, but I'd probably end up moving it myself at
some point :) maybe better if you re-submit?


First we need to actually support attacking tens of millions of hashes on GPU. Maybe we can now, I haven't tried that after the recent changes.


Mike, the bca.pdf Contributor Agreement I provided explicitly states that distributing under *any license* is allowed for either party. That's the whole point of this document. It does indeed let the original author keep (joint) ownership of the code, but as far as BlueJimp is concerned, none of the restrictions of the original license apply, including the possibility to distribute that code under other licenses. Or as the contributor agreement states:

"each of us can do all things in relation to your contribution as if each of us were the sole owners"

Still to make sure everyone is happy: I have updated the README file. Does this work for you?


Hi @Rich86man,

I found workaround for problem with IB_DESIGNABLE in CocoaPods: https://github.com/CocoaPods/CocoaPods/issues/2792#issuecomment-69316099

Just wonder if it works for you too


s/accomodate/accommodate/


This is a bad pattern.  We should be exposing the session to methods and individual methods should be calling session.commit().  The risk here is that multiple threads will call into the DB class - all sharing the same session.  Commit and rollback on a different thread of control will cause unpredictable behavior.


just call opencv sift and then modify each descriptor as in
https://github.com/openMVG/openMVG/blob/master/src/nonFree/sift/SIFT_describer.hpp#L32
```c++
    for (int k=0;k<128;++k)
      descriptor[k] = static_cast<unsigned char>(512.f*sqrt(descr[k]/sum));
```


reack


@BillClinton you missed a couple other spots for the package rename, be sure to use a tool that finds all occurrences of the old package name in the entire package tree when renaming


Shouldn't you ensure that the data is filled in correctly before you prepend the item to the list?


Keep doing whatever you are doing. It'll get better. :wink:


You mean to keep the "i2c-smbus" string? I'm okay with that, if so.


Look at how we did in Maestro for requirements. They are read from the `requirements.txt` file so there is only one source of truth:

```python
with open('requirements.txt') as f:
    requirements = [line.strip() for line in f.readlines()]

setup(
    ...,
    install_requires=requirements,
)
```


@yemel thanks! I was just searching around for email-backup related issues and found this one and figured it was a reasonable place to start the conversation :)


@felipeas  Não pensar numa forma de adicionar novas músicas, mesmo album. Até imagino o fluxo, mas não manjo em ASP.NET. Tem alguma ideia?


domain name


You still haven't fixed that. If I call `send()` on the `SignalFxLoggingStub` I'm gonna get a `NotImplementedError` from `add_to_queue`


valgrind does not show a leak here; nodestore_replace calls nodeEntryFromNode, which "frees the original node but not its contents". Since methodNodes do not have contents, the replacement is freed by calling _replace.

I was lucky in this case, thx for your hint though.


`add_to_queue` should not be a public method, people are not going to call that directly.


Yes, I agree we should define and describe the test statuses but still I'd allow users to override certain behavior as not all users share our views. Yes we define the future, we define how test framework should work like and we're responsible for being consistent, but we target devels and qa from various fields and they should be allowed to adjust the behavior to their needs. That's why we have plugins and configs instead of hardcoded values and blob of data.


Same as `add_to_queue`, this shouldn't be part of the public API


why 2469m? It seems oddly specific...


domain


Hab mich verschreiben meinte nicht setAnswers sondern setQuestions


OK


My recommendation is to stick as closely to known packaging systems as possible; the people who develop these plugins are going to be very familiar with node/bower packaging. This will lower the barrier of entry and make it easier for people to get started.


Will fix


Nitpicking, but could you move this below the `import generated_protocol...`?


I prefer to leave counting to Markdown, this makes future changes easier too.


As is, this code tries to read a JSON file loaded on my personal desktop. Please change the path to the example preprocess JSON metadata provided.


DesignDaysGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2193 of 2213 tests passed)

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


This entire DB wrapper needs careful vetting for threads, and also statefulness. For instance, in the latter case, I don't know if a failed query put the session into a state that it has to be recreated (or closed/reopened, or the transaction rolled back).
I find it hard to believe there isn't a best practices sql alchemy database wrapper class I can use here. After a quick web search did not reveal anything, I just whipped this one up using my best guesses as to how the database reference should be shared.


I guess I don't understand what `main.js` is, is it a file used in the browser or on the server?


You may be right, but I've encountered lots of issues when creating symlinks and this one surely works.
Another reason is that it's consistent with how all the other symlinks are created and I think consistency is important.


This is the only place you use `signalfx.Error`, which makes me doubt its usefulness, especially since a `TypeError` is better suited here. Change to `TypeError` and remove `signalfx.Error` altogether.


Nice, we're getting there! This is the last of it, fix those and I can merge!


UnmarshalRepoRevSpec should pick out the Rev and CommitID vars because it is called after PostMatchFunc. It should no longer try to pick out and parse ResolvedRev. I made a stupid mistake in not updating this code.


I agree with everything but Draw for Islands default on. That really should be off. Extension makes sense for maps where the land continues to the edge, but for islands, the terrain is clearly contained entirely within the map, so the extension looks conspicuous and makes the whole scene overcomplicated.


Rather don't overload people with information that they don't need.  Also South does a fine job of telling people that the migration rolled back or not.  Drop the "All previous duplicate..." to the end from that.


yes, that is a great change - thanks for letting me voice this concern and talk about it!


Toll


I would prefer this fixup to be in the relevant PRs


@BenJanecke: Was this commit part of a pull request? It would be useful to record why we decided it was safe to remove this draft flow.


No ma


Oh okay :)  Should I update update pull request with this change?


Also this one, out-of-scope fix


Perhaps it would be nice to print a message to the user mentioning that it's exiting. Just “Exiting” would suffice. Even if it's the user's own action, explaining what's happening doesn't hurt. ☺


+1, but maybe try both "_" and "-" in the module name? Like doing an os.path.isfile() and if not try with the module.replace()?


As above


s/as list/as vec/


We actually don't need to know that, unless you want it for debugging.

Please only communicate with the user when the information is important, I don't think that is unless the user specifically wants more noise.


s/as vec/as a Vector/


1687-OrphanedConnection (kbenne) - i386-Windows-7-VisualStudio-12: OK (2189 of 2213 tests passed)

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


s/as set/as a Set/


Arreglado issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/38
Con esto debería quedar funcionando.


Let's try to make our commits ⚛ !


Okay, will make revisions.


Is it necessary to have this outside the `unless` block below?


I'd vote to leave it without a default.  What is a subjective message without a subject?


> There would be no need for legality for 40 card decks, as they would be generated randomly from a pool of legal cards.

From this, I suggest when using the deck editor from the duel screen the legality tab is hidden. There are definitely two distinct modes for using the deck editor. Based on @melvinzhang's comments above - _Magarena is designed for playing the Limited format with randomly generated decks._ - you could even argue that the standalone deck editor via the main menu is a bit of a square peg in a round hole in the context of implementing a _game_ focused on quick, streamlined game-play. I think we all agree we don't want to end up with a Forge II.


Was Twitch one of those?


Delete this file


Excellent. Will you close the ticket?


With this and the additional comment I made, yes :)


ok


see my amended comment


1687-OrphanedConnection (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2159 of 2213 tests passed)

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


This one part (the test framework) will need to be expanded I think.

For one, not all messages will have that ``self.msg['msg']['user']['username']`` field.

Think, too, about how to test the irc meeting messages.  Some messages will have more than just two different ways to be rendered, so we'll need a more flexible way to express the ``expected_subjective`` target than just a single string.  Something more flexible than all the other tests currently in there.


ping @lolautruche


Yeah, just like this :)


I had not signed the bluejimp CLA at the time of opensourcing the initial version of meet so I doubt this allows you to relicense based on that. I am somewhat disappointed by the lack of community discussion prior to this commit (which should have been a pull request)

I do think the apache license is more restrictive than MIT and a bad choice for a web project.


Good docstring.


Always "program to interfaces". So, replace HashSet variable with Set.
Pay attention to the access modifiers for instance variables. Should it be private?
Explain the rational behind using Set via JavaDoc


Doesn't work. It removes the line break between the effects, but the wording itself stays the same. Should I just .setText() it?
The other tooltip problems are fixed now, as well as another one in Power Matrix.


Handle edge cases before adding to set


I like this.  I've wondered but never looked into whether other shrinkers in the kernel do the same thing, which would lessen the need to call them with ever-higher ```nr_to_scan``` values.


Then how to do it correctly?


changes to text


DesignDaysGridView (evanweaver) - 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)


Set#contains already returns boolean so you should just return that.


"with Weave and Docker" flows a bit better, for me.


A few comments/questions:

1. Don't leave commented out code in a commit.. Git is pretty good at giving you the lines back from history if you need them later

2. Why did you rename it to "msgpack". It's called cmsgpack for a reason, why not recognize the original package by keeping it's name? That includes the original URL it came from


CSS3 Transitions not supported by: IE (6,7,8,9)


look good to me


What IDE?


what's the purpose of this statement here?


Also.. turn the branch into a proper PR once you think it's ready


No reason, I probably forgot about the other method.


Actually I have no idea why these line was edited.
When I edit this file, I only simply click on the "Edit" button and add the missing "return" line, and commit.

After commit only I saw these line was edited.


No, just `I2C`.


For this test I would set the 'range' option to let it clear the input constant is within range. The way it is you're using the default range, which is 0 to 1023. Then I would write another test using an out-of-range input and validating that the constrain node do the wrap up.


Never expose implementation details in your methods. If you return your instance variable Set then the client code can perhaps modify it, which you don't want. Wrap instance variable set in an unmodifiableSet See http://docs.oracle.com/javase/7/docs/api/java/util/Collections.html#unmodifiableSet(java.util.Set)


Looks good to me.


In the case of live streaming, this would not be enough.  If the readyState is HAVE_NOTHING and we skip setting currentTime completely, live streams will not start properly.  Their streamStartTimes are generally greater than 0.  If the live stream has been running longer than its timeShiftBufferDepth parameter, starting at 0 is not possible (the segments are not available).  Some live streams actually never had a segment at time 0 if they use offset since 1970 in the encoder.

So if the readyState is wrong, you will have to defer setting currentTime until the readyState is at least HAVE_METADATA, not skip it completely.


Add test cases for each Dictionary method


revert back to String as this was just to explain you the "Program to interfaces" concept


存在笔误，问题很可能出在这儿


I'm wondering how hotly-contended this lock might become.  Among other things, it's needed along the ```.destroy_inode``` callback path.


Ya, pero en Top acordamos no imponer calidad. No debería afectar mucho.
Los Trees q me has pedido los estoy haciendo con el primer vértice

El 18/06/2015 a las 18:18, Juanra Castiñeiras escribió:
> Lo de que el vértice para los muones sea firstGoodVertex lo habíamos
> puesto en su día para que se coja el primer vértice que pasa unos
> mínimos de calidad. Creo que es mejor dejarlo como estaba.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/piedraj/TopAnalysis/commit/05f883c518d40f58eea7e4de3726681eb26ddc74#commitcomment-11751128>.
>

--
-
+--------------------------------------------------------------+
  Javier Fernandez Menendez

  Grupo de Fisica de AAEE
  Universidad de Oviedo
  C/ Calvo Sotelo, s/n
  33005 Oviedo
  http://www.hep.uniovi.es

  Phone: +34 985106252
  mailto:Javier.Fernandez@cern.ch
+---------------------------------------------------------------+


I think this import must be moved back after all the app.config calls, just before `@app.teardown_request` in line 31. If not, you get an error like this:

```
Traceback (most recent call last):
  File "runserver.py", line 2, in <module>
    from Flasktest import app
  File "/home/application/current/Flasktest/__init__.py", line 7, in <module>
    import Flasktest.views
  File "/home/application/current/Flasktest/views.py", line 1, in <module>
    from Flasktest import app
ImportError: cannot import name app
```

I got to confess that I don't know why.


missing indent


hhh


You seem to be having some composer caching issue... not sure why. Try this on another machine.

See if you can try [instructions in this web tip to refresh packages via composer](http://webtips.krajee.com/composer-fails-update-vendor-packages/).


ghjkl


Ok! :+1:


Що з відступами?


If I understood correctly the idea here is to make timeout happens before the second value on sequence, right?
So if TIMEOUT is working fine it should calculate 1/1 == 1.
But if it fails, it would calculate (1 + 2) / 1 == 1 as well.
Shouldn't you use other values on this sequence? Or am I understand incorrectly the purpose of this sample?


Is there a reason to drop this? Its from Django, why do we care or more what are the risks that someone actually did want this here and now we've just gone and deleted it.  Can we please justify why we're dropping something.


:shipit:


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

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


Could not find anything about runtime 5.0 yet. It should probably stay at 4.0.


Yes, you need  set text here and uf needed ad a <br> tag.


In general it's used to specify that the implementation is given in C or somewhere else where the code will be linked. In this case, there are a few builtin/intrinsic functions provided by the runtime so I just reuse those.


Alright then. Indeed the initial 5371 lines of code came from a fork that was not subject. So while the remainder of today's 83151 are, it is not trivial to distinguish. Until we are able to do that in a cleaner way, we will rollback the license change for this specific repository.


is there a reason this isn't "~> 4.2.2" ?


Ok, de todas formas yo había comprabado que pedir la calidad mejora un poco la eficiencia al cortar en dz. Son criterios muy loose los que se imponen, realmente.


Looks like there was a merge problem, but I'll just fix it.


@kevva you're the best! :heart:   I can sleep well at night knowing this module is well tested whenever I `npm install` it.


Just my own 2 cents but I'm surprised this isn't one of the changes MELERIX commented on.  I may not know the Spanish language at all, but I find this odd.

I suppose that "motor" refers probably to "engine", so this probably translates to "Engine of 32 bits" or whatever.  It really doesn't sound right to say that the Project64 core HAS 32 bits; it's more accurate to say that the core in use IS a 32-bit engine.  The hyphen is safer, but again I don't know Spanish so don't know if that sort of syntax is something the Spanish language doesn't allow or whatever.


I think the behaviour of `app.config.from_envvar` is not the expected here.

[Here it mentions](http://flask.pocoo.org/docs/0.10/config/#configuring-from-files) that it will load the configuration from the file that this environment variable points to.

Because that, I get this error:

```
Traceback (most recent call last):
  File "runserver.py", line 2, in <module>
    from Flasktest import app
  File "/home/application/current/Flasktest/__init__.py", line 7, in <module>
    import Flasktest.views
  File "/home/application/current/Flasktest/views.py", line 1, in <module>
    from Flasktest import app
ImportError: cannot import name app
Traceback (most recent call last):
  File "runserver.py", line 2, in <module>
    from Flasktest import app
  File "/home/application/current/Flasktest/__init__.py", line 21, in <module>
    app.config.from_envvar('STATS_HOSTNAME')
  File "/home/application/.app_env/local/lib/python2.7/site-packages/flask/config.py", line 109, in from_envvar
    return self.from_pyfile(rv, silent=silent)
  File "/home/application/.app_env/local/lib/python2.7/site-packages/flask/config.py", line 129, in from_pyfile
    execfile(filename, d.__dict__)
IOError: [Errno 2] Unable to load configuration file (No such file or directory): '/home/application/current/Flasktest/statsd.hostedgraphite.com'
```

[The Flask documentation](https://pypi.python.org/pypi/flask-appconfig) refers to the usage of environment variables, and it seems that can load them automatically, so it won't be needed to use them. But I was not able to make that work.

Other option is write something like:

```
app.config['STATS_HOSTNAME'] = os.environ['STATS_HOSTNAME']
```


Hey Matt, I left your name inside our Settings, the thing is, running from the original source, About menu caused heavy crashes due to the original ad layout. So I removed the whole ad engine commit by commit to test. And it stopped force closing.

http://imgur.com/d0SshDH

Thanks a lot for the great application source! It's a great utility :)


Incorrectly. The timeout should calculate 1 + 2 / = 1 , the buffer uses the default parameters  ( wich means that the expected amount of int packets that it should receive before triggering an execution is 4, so if it calculated 1 it meant that it received the two packets, timeout expired, and it calculated using the packet that it had correctly.


I'm not quite sire what should be the 3rd (last) argument to the `pipeRegistry.get` method...


I believe it could be a bit more elaborated.
You're using a single value, that is inside the accepted range...
At least use 3 values, one that is smaller than limit, greater than limit, and one inside the range, so you can exercise it a bit better. The way you have done this test, even a node type that just forward the packet would work just fine. No constrain is tested.


Tested only on HSW and BDW. As you know the current requirement is to support these features only on BDW and HSW. But I will make sure that nothing has broken in SKL.


Can I ask how for both of those statements?


why are you leaving a lot of blank lines at the end of the file?


Done


Corresponds to Issue #18


I thought that the correct type that accommodates how many entries can be in a container is size_t?

As a separate question, people seem to suggest that it the getter/setter functions are doing nothing except for returning/setting their variables, they should be removed in favour of direct access. Should something similar be done in the future?


No problems :) The about just had donation links for me as original app developer, but you are free to do as you want I guess!

Yeah I caught those FC's. Apparently, privileged apps can't use WebView as of 5.1. I figured this a bit back and added some code to detect that (or I think @arter97 figured this and did that).

I remove them on the AOSP branch, just a few days ago, I went through and removed all the ad code from the AOSP branch entirely :)

If you guys at Bliss come up with anything interesting for the app, please consider sending a pull request :)


Thanks! BitHub has sent payment of  $8.51USD for this commit.


You need to iterate through th active modes.
        for (UUID modeId :abilityToModify.getModes().getSelectedModes()) {
            this.getModes().setActiveMode(modeId);
                 for (Target...


Esto está duplicado en varios sitios - estoy seguro de que podeis sacar factor común.


If it is *possible* for someone to exercise those API additions on SKL then we cannot merge until a) we prevent their use with SKL or b) we make sure it works with SKL.


DoS attack because if you send enough handshakes, you can cause the server to stop accepting clients. For compare exchange see http://en.cppreference.com/w/cpp/atomic/atomic/compare_exchange.


Kind-of want to keep this as it is as it makes the default option very clear.


Also, I think the variables STATS_HOSTNAME and so on should be optional. The application should work regardless they are defined or not.


thank you for allowing the discussion and considering our opinions - I will close the issue.


wouldn't hurt to check the return values of these 2 tests, would it?


It's a good question :) I don't remember at all why I did this and ti's likely I copied it from somewhere. I agree with you, It doesn't look correct to set the content dirty flag when getting a doc...


use /usr/share/mana-toolkit/sslstrip-hsts not /usr/src/mana/


(SNAFU on the LICENSE update. re-fixed now)


Ok. Going on holiday in these days. I'll create a PR after back.


I think we should move to functions rather than methods where possible. Less state to manage and potentially cache miss on.


rax retry tests


old editor had a 'strip blank lines from end of file', the update broke this. sorry - already fixed in the editor.


This td needs a top class to avoid the "Members" label to end up in the middle if you have a lot of cards.


@jdonaldson I have working split method, but we must find out an optimized version.


Why the default response is not assumed `initialized`?


This is super-nitpicky, but Unix (GNU) convention would be `--clean-catalog`.


BTW, I've modified this to use return a struct. It's cleaner overall.

And yes, totally agree on moving to functions.


`--start-master`


`--start-workers`


Apart from the members label, everything looks good as far as I can see.


Ah right. I might readd it back within Settings soon. ORS broke again on this build so I have to find out why,  despite it worked on previous builds of 5.1.1. I'll resync the project soon and see what I can do to make it easier for the user.

If you haven't seen our fork of BlissOTA, this is how it looks like

http://m.imgur.com/KJ6rHiT,WRnRs3s,ACWxqjd,fyy8lHO

I exposed most colors for Theme Engine so that isn't the default theme. For some issue name of rom suddenly stopped getting reported back.... So I have to look into what happened.


I think it's plausible to ask the user to call at least one of the `Body()` / `JSON()` / `Code()` / `ContentType()` / `Headers()` methods in order to consider a constructed `Response()` object to be a valid response.


I've just gotten in the habit of avoiding partial functions
even when I think the list will never be empty, because who
knows, maybe another change down the line will destroy that
invariant.  The 'safe' package on Hackage has some nice
replacements.

+++ Imuli [Jun 17 15 21:30 ]:
>   So this one we know has at least one element, because revPair includes
>   our revision. The proper fix would be for filestore to put the parents
>   of a revision in the Revision, or have a helper function that returned
>   a list of parents. But yes, the method in Network/Gitit/Handler.hs is
>   better. Will fix. Thanks.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/imuli/gitit/commit/ced4f1bb0a0ffbae94ff8aa9784ec9a1ed732816#commitcomment-11739068


The default response was sent, why 500?


Valid questions. Switching name back and removing commented out code


So, we don't need the FF workaround code any more?   When did it become unnecessary?


bla


An uninitialized `Response()` was attempted to be sent, which, to my structured eye, is a logic error. :-)


`Ints.toArray()`?


Like this
```ruby
rescue_from :all do |e|
  Rollbar.error(e)
```


@ahankinson does this look better?


So if I want to send a default response, I have to write `Response().Body("")` instead of just the default-constructed `Response()`. Looks fuzzy.


What's the reason for this change?


Buen aspecto, y complejidades, y consts. Teneis un 10 en el ejercicio.


Just `Response("")` would do.


My first intention was to change test `url` to `tagged_name` everywhere, because it seemed as right thing to do. Then I realized we store and make tagged_name publicly available and use it everywhere, where it make sense.

Current implementation works the way that `url` is unique "test" identifier, but it's shared across the variants. We also publish `tagged_name` which is unique across the job execution and is what you see in avocado UI and now in html report.

One might argue we can rename the `tagged_name` to `UID` or `test_id` to be more clear on that.

PS: Now I realized what you have in mind. The commit message was probably a bit odd, I should have used `url` instead of `uid`.


Ah, OK then.


Thinking further, do you think it's worth supporting `Response(HTTPResponseCode.OK)`?


FWIW, I think the publisher is "Ecma International" while you're at it.


I retract my earlier suggestion since I think these config keys seem clearly hierarchical and benefit from the dotted notation. `tcp.connection.timeout.milliseconds` is maybe a borderline case but not enough to be confusing. Sorry!


We could do that, though, I was thinking that I might end up with an additional mode to support my hub and spoke needs since I need to deploy to 8 amazon regions and I don't want to hit github directly from sydney for example, and I don't want to push out from a single server either.


For the initialization of the muonSelection tool, this tool (VeryLoose id) is already initialized (the 4 muon ID's are initialized in initMuonTools)


Yep! How does it work?


For dead-simple REST responses this could be useful.


You shouldn't do this. You can require `1.x@dev` in your root composer.json, but `dev-master` in your package will probably give errors when other packages are require 1.x


If we were going to stick with the dotted notation then I guess this should be `max_heap_size.gb`, or say `heap.size.max.gb` for more consistency but possibly less readability.


:bulb: The GitHub UI is better tuned for comments on the diff of a pull request than it is for line comments on a particular commit.

Setting max version to 4.0 can prevent the extension from loading in 4.5 environments. Setting it to 4.5 can cause problems in 4.5.1 environments. Maybe it's better just to remove this line?


Nice! Really dig the new `edit()...commit()` syntax. Was just dealing with callback hell to stage property writes; very glad future projects won't have to do the same! :stuck_out_tongue_closed_eyes:


See above.


Well, eventually we will add the include back then :-)


Delicious.


El destructor es utilizado para poder borrar la variable de la memoria dinámica sin tener que apagar el equipo.


These comments can be removed now.


__author__ = 'MR.SJ'
#coding = UTF-8
import threading
import time

arg = 0
thread = []
def music(musicname):
    for i in range(3):
        arg = i
        print "musicname: %s>>>%s" % (musicname, time.ctime())

        time.sleep(1)
def moive(moivename):
    for i in range(3):
        print arg
        print "my love moive:%s>>>%s>>>%d" % (moivename, time.ctime(), arg)
        time.sleep(1)
def getthread(t, name):
    w = threading.Thread(target=t, args=(name,))
    thread.append(w)

if __name__ == '__main__':
    getthread(music, "mylove")
    getthread(moive, "bigbag")
    for t in thread:
        t.setDaemon(True)
        t.start()
    t.join()
    print "<<<<<<<<<<<>>>>>>>>>>>>>>>>>>>>>"


1687-ThermalZoneClone (kbenne) - i386-Windows-7-VisualStudio-12: 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)


This class is pretty self-explanatory, but we can keep the description. I didn't think of doxygen when I removed this stuff. Same for the other comments below, but there's nothing wrong with leaving obvious comments out.


I didn't want the archive module to use a variable from http in case it was combined with something else in the future.  S3 downloads or some sort of bittorrent option.

Both these changes are really about being more efficient now at the expense of future proofing.  I could go either way, though my preference is to be more future proof.


Sorry, I didn't think of doxygen, but I don't think it's a huge loss to remove these anyway. I can revert these changes if you'd like.


PTAL at 5387960 :+1:


How that is related to the topic? Use bug report in forum if find any problem.


It reads to me that this is dual-licensed, under both GPLv2 and MIT license terms.

Please confirm that.

It would be nice to say explicitly that this software is dual-licensed. Really though it would be more clear to just license it with the MIT license, as long as the work is fully original, as that allows sublicensing work, including to GPLv2 or any other compatible license.


Not sure what the benefit of parsing the workerID to `int` then immediately converting it back to `String` is, unless you think it improves consistency.


`--clean-catalog`


Radi,

Sure. Please feel free to branch out and modify the script. The script was
written in haste to just make it work for a specific project, so I didn't
get to optimize it in terms of usability etc.

- Kumaran

On Thu, Jun 18, 2015 at 10:45 AM, Jason Knight <notifications@github.com>
wrote:

> Radi, welcome to the glorious world of Git and Github! Regarding your last
> sentence I'm going to let you in on a little secret: you *can't* mess
> things up here, because it is a trivial matter of reverting if things go
> wrong, or we can just patch things to fix them as time goes forward.
>
> So dig in, put caution to the wind, and give it your best shot. (at least
> that's my suggestion). Good luck!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chapkinlab/sequencing-pipeline/commit/c8fe9590eb27a0511a6a8fa7a09fb903cfc47a8b#commitcomment-11750326>
> .
>


`--start-master`


We still need descriptions here. This is part of the docstring standard (and rendering assumes a description is give).


`--start-workers`


Help


```rb
def create
```


I think `prove -v -e 'perl6 -Ilib' t/` is a better option. Panda is not installed by default (so im not sure this will work as is). It also shouldnt be installed by default (at least for now), as this blocks testing modules with no dependencies when panda breaks (I also use an alternative package manager, but that can be worked around).


In this case it's pretty obvious what it is, so I removed it without much thought. Nothing against keeping this kind of thing. Notice how I left the two others above intact, since it's not as clear what they are.


Testing comments...


Study folder:
\\cnrastore-bdo\CVM_Inventory\CalLite\ModelSequence\29_Baseline_SJR


More to the point, we should make this docstring as close as possible to the corresponding docstring of the `*_tree` function.


I'm assuming you looked at the generated JSON and saw the right thing, presumable a `Memos` array containing two `Memo` objects.


```rb
redirect_to sakayas_url
```


test


Du meinst dass unser System large scale ist? Oder meinst du scaling von der Objective Fun?
Als ich das gegoogelt habe kam als 1. Ergebnis objective function handling (hat was mit der Abbruchtoleranz von der Objective Function und dem Skalieren dieser zu tun).

Hab gerade mal alle options von fmincon durchgelesen und unsere Toleranzen auf die dort genannten default werte gesetzt. Du hast sie immer ein Stück kleiner gemacht. TolCon ist trotzdem noch 1e-5 (default wäre 1e-6)
Ich habe aber keine interessante Option gefunden die uns weiter helfen könnte


Study folder:
\\cnrastore-bdo\CVM_Inventory\CalLite\ModelSequence\30_Enlarged Shasta


Até dá, mas teria que usar ```loaded={ !this.props.loading }```. Não fica meio confuso?


btw, there was a stray `x ` typo here


i think you are missing two cases:

```shell
kazoo/tests/test_client.py:        self.assertTrue(type(client._conn_retry) is KazooRetry)
kazoo/tests/test_client.py:        self.assertTrue(type(client._retry) is KazooRetry)
```


Maybe your editor did it?
Revert the cutin part?


In home.ctrl.js, there's a typo "alphebetizeByState", should be "alphabetizeByState". Other than that everything is good


No, this fix was specific to the problem in the plugin library; the fix was to remove the autofocussing. It was causing problems on mobile devices.


@ssciancalepore, there was an error in v8 of minimal, this key contains only 15 bytes. v9 adds one space character so that the key is regular 16 bytes.


Looks great, only a few very pedantic changes.


I've rewritten the `rx` code to use several regexps instead of one that invokes the esoteric `compilation-error-regexp-alist` parameters.


SP


prefer  `sakaya_params`


Ohep


preder `set_sakaya`


LGTM


Don't recall if this is intended or not. It's not just x, it's `x `. Let's remove the x and add it back if something broke.


Pensei nisso, mas acho que o loader deveria iniciar antes, logo que o dropdown é alterado.


The HTML doesn't validate.  Can you please update and comment here when it's fixed?


It's worth (I think) describing the cases in which `None` might be returned.


Please rename the file to `README.md` so that github renders the text nicely


yes, it looks good


I thought THCDeviceTensor would be removed from this specific request?


Please push this commit out.


I'm pretty sure it's stray. It is stopping the final summary of a failed scalatest suite from being detected.


Alright. I'll try and do my best to fit your advices.


I wonder why @lgarron 's pull request was not merged yet it was applied directly to this file. @adimare Do you have trouble checking out a pull request locally and resolving conflicts? I just want to know, maybe we could help you with git just saying.


Works great! Consider adding .capitalize to your @name = gets.chomp, that way if someone accidentally enters their name without the capital letter, it will still work.


Commit message should have contained reference to 33c430d003b03d8a078da89a38cca637ac8ba768, which introduces inexperienced jurors support.


This was (an ill-fitting name) for the dropdown of layers.  It was constricted arbitrarily, I figure we can style it appropriately when we know how many layers will exist.


this is what happens when you accidentally turn off numlock


Well, the ordering of local functions matter, so if there's no table, I need to carefuly order the functions in the order they are used.


You don't use an ide? -.-"


erm, maybe you can help me with that ?

I am not using any editor, I just simply edited the file using the github editor that provided by github.


You have `commit.State` set to `common.StateSuccess`, but you need to be either `common.StatePending` or `common.StateRunning`.


Removed it in next commit.


That is for fullscreen mode. Will add a comment in another commit (there are other cases where this is done)


Aynen oyle :D :+1:


I used this feature :(


@barryvdh So I should require `1.x@dev` in the Manager package?


IRO = Costume Polar Bear Cap
KRO = Valentine Heart


Thank you.


No I don't think you can do anything to fix this, unless wait for the new tag.


ack


g3 use lm3631


cheers - I have merged this; please do send Pull Requests when you've made anything useful!


Actually, with the current code, you can already do it, since we have the templated constructor. The second call to Cross() below is equivalent to the first one, but the precision is set explicitly. With the two parameter solution in the template, Cross(a, b) ≠ –Cross(b, a), i.e., the precision is float in one, and double in the other. Or worse, it may convert a vector version to scalar depending on the order of the parameters, which may not be intended, and lead to errors. This is exactly the kind of problem I want to avoid by not letting user code mix different types. You can always call Cross<Backend::Real_v>(a, b) and use whatever vectors you want with my solution, and everything becomes more explicit.

```cpp
Vector3D<double> a(0.0, 1.0, 0.0), c;
Vector3D<float> b(1.0, 1.0, 1.0);

Cross(a, Vector3D<double>(b));
Cross<double>(a, b);
Cross(a, b);
```


Comment should read "Build Jenkins project, instead of DockerHub".


LoL. Don't worry about the translations, I'm a 20+ years experience "pro" in international (and multilingual) PM :)


There are 2 versions of pssm_mat.material; one with shadows and shaders loaded with the material file, and a black one who will just render the texture without anything else.

When VS compiles RoR, it create a folder called "Release" or "RelWithDebInfo" at the game's root. So.. yeah.


Oh, i thought you had merged this in #293
Sorry I'll b away from my computer until Tuesday, could you cherry pick and merge manually?
Otherwise I'll do when I'm back
Thanks


test


This first commit


not sure what was changed here, but there's unnecessary dot in "content.,"


Regarding the the namespace, from the docs for the `XML` functions: `namespaces: a named character vector giving the namespace prefix and URI pairs that are to be used in the XPath expression and matching of nodes`. I just use `ns` for the name it because it's the only XML namespace and makes more sense when building my XPath expressions.

As for the space around the `=`, that is made optional by the linter. I had to go reread Hadley's documentation after reading your commend, and honesty I don't like Hadley going against the Google R Style guide and the Python style guide for that one. When assigning inside parenthesis for function calls pretty much every style guide says to do `some_function(param='my param', another_param=TRUE)`, not `some_param(param = 'my_param', another_param = TRUE)` which just gets long and ugly and I'm guessing that's why it's not included in the linter. It's not failing the lint that runs on `devtools::test()` so I'd rather leave it than go through 1000 lines of code and change them all :no_mouth:.


@tanepiper What was the breaking change that moved this to 2.0.0?


@barryvdh Guess I can just hope that the next release gets tagged soon so I can remove this again. Anyway, thanks for the info, didn't knew that this could cause an issue.


This is a test.


Could you add a CHANGELOG.md file to the project with version details? Just the last 2 I think is enough.


okay


sheesh


É, eu vi que a prop desse componente usa o loading negado. Eu acho ```loading``` mais simples de entender, mas tudo bem.


Why are there two spaces?


you forgot the "!" dummy


Your tests should not rely on the order of returned results.


thats supposed to be 0.12


Mas logo que o dropdown é alterado, instantaneamente, o ```beforeSend``` do ```$.ajax``` é executado.


1687-ThermalZoneClone (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2153 of 2213 tests passed)

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


You have this already in tasks.py no?


garment not helm


note


You win some, you lose some.


yes....
part of original script i made here:
```sql
-- Terrifying Abomination -- UPDATE
-- Scourge Hook - shouldnt be casted with full range.. (this will make us flying all over the place) must be max 20 yards.
UPDATE creature_ai_scripts SET event_param2 = 20 WHERE id = 3109801;
```
should give a diff result...


kRO vs iRO case, again... ~_~'


There's a grunt module which does this all for you: https://www.npmjs.com/package/grunt-aws-lambda


Are these integration tests covering something that's not covered in `parent_strategy_spec.rb`? If so, can we find a better home for them?


:camel:


Why are you returning a string? If you want to log this you should use a dedicated logging call, and return either a value, void, or True/False to indicate success/failure. @lingxiaoyang, is this the case?

Also, the preferred method of string formatting is "wfRun PROCESSING {0}".format(workflow_run_id)


done


This doubles the fugitive status if you have defined a customized fugitive statusline in your vimrc as I did with this line:
autocmd User Flags call Hoist("buffer", 1, { 'hl':'Visual' }, "fugitive#statusline")


@ssciancalepore I would propose to name these two defines using ```K1``` and ```K2``` instead of ```BEACON``` and ```DATA```, such that it is immediately obvious to anyone familiar with minimal draft.


Which operations? Transactions?


What does `now` do?


a common pattern for resources which operate on source/destination is to add Ensure property (even if it's read only if we don't support removal) similarly to File resource.


Good workaround!


Concordo. E acho que ele deveria ser encerrado no ```complete```, pra não ficar lá pra sempre em caso de erro também.


Thanks for doing this. I had to make some small change to avoid the data race in tests, and had been wondering about a theoretical race in the way this was implemented. Did you actually see one happen?


How about like using
  trait RequestLogging { this: ScalatraServlet =>
instead of RequestLogging extends ScalatraServlet


For minor updates I typically do update *both* source list and these files by hand, because the generation takes lots of time, but once upon a time I run it to be sure (e.g. dependencies might change, etc.).


Other changes added by XMLSpy: change order of some attributes; delete space before closing tag; delete extra space between attributes; puts comments on their own line, remove extra tabs.  Nothing I can do about this.  I hope the tool does it consistently!


1687-OrphanedConnection (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


You are right.  It's done already.


It fails if the proof state contains unsolved goals.
The first proof state produced by `repeat (constructor | eassumption)` contains the unsolved goal `... |- q a`.
The `now` forces the backtrack.


reack


Así se declara la amistad entre los objetos de la iostream y nuestro objeto.


Correct Spanish has no hyphens. That's why I modified the expression.

On Thursday, June 18, 2015, cxd4 <notifications@github.com> wrote:

> Just my own 2 cents but I'm surprised this isn't one of the changes
> MELERIX commented on. I may not know the Spanish language at all, but I
> find this odd.
>
> I suppose that "motor" refers probably to "engine", so this probably
> translates to "Engine of 32 bits" or whatever. It really doesn't sound
> right to say that the Project64 core HAS 32 bits; it's more accurate to say
> that the core in use IS a 32-bit engine. The hyphen is safer, but again I
> don't know Spanish so don't know if that sort of syntax is something the
> Spanish language doesn't allow or whatever.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/project64/project64/commit/cb935720f0328459226d1c64efd9e20055b779d3#commitcomment-11751867>
> .
>

--
- Julio


Add empty line between functions throughout this class?


My guess is not that hot but I've never measured it.  It's briefly taken during znode creation, destruction, and fs teardown.  It's always been something of a sore point as well since it's technically entirely wasted overhead under Linux.  The super block is already maintaining a similar list.

This is one of those things it would be nice to remove someday when refactoring the VFS code.   It would let's us shrink the znode by 16 a `list_node_t`.  Similarly the the znode_t itself is highly redundant with a Linux inode.  Merging the two would saves us lots of space per-file, this has been on the cleanup list forever.  See #227.


Así se le dan las ordenes al objeto externo, permitiendo hacer cout<<var1<<var2<<var3 con nuestro objeto por ejemplo.


I think I figured it out: it is a tactic which does nothing but fails when there is still a goal left.


Así se implementan las variables de nuestro objeto con cin>> y cout<<


`With no arguments, trace all`


Doesn't this apply to any [`Delegator`](http://ruby-doc.org/stdlib-2.2.2/libdoc/delegate/rdoc/Delegator.html) subclass not just [`SimpleDelegator`](http://ruby-doc.org/stdlib-2.2.2/libdoc/delegate/rdoc/SimpleDelegator.html)?


It does and active inode may be rotated to the end up the per-superblock lists.   But the specifics vary significantly by kernel version.


Se declara el destrcutor


Seems to work! I'll look at the neumes this afternoon.


Unfortunately it does persist in our index… filter the commits or rebase them some day.


Nyjal, are you still seeing this issue? I wasn't able to repro it from your repo. Can you send me a gif file of what you are seeing so I can investigate further?