Currently the TRUE and FALSE would be in lowercase... that is what was generated in the pass. shall it be uppercase instead?


Wait a sec, I misread: this fixes the bug I hit. Let me check what verison I'm working with.


Je ne parvien pas à intégrer le formulaire dans la section alors qu'il est present dans le template ,¿


Errazagoa agian:

```javascript
// nonbait exekuzio-fluxu nagusitik kanpo
const autocompleteLangMapping = {
  eu: 'eu-ES',
  en: 'en-US',
  es: 'es-ES',
  fr: 'fr-FR',
  de: 'de-DE',
  jp: 'jp-JP',
};

// Hemen
var autocompleteLang = autocompleteLangMapping[source];
```


@migurski What kind of help are you looking for?


Note that you didn't bump the timestamp here, so this didn't go out to clients, which meant that the GBV translator was being used (and producing worse data in at least [some cases](https://travis-ci.org/zotero/zotero/jobs/58499872#L214)) for ISBN lookup. I've pushed out the new version.


Patch updated with ``encoding`` and ``other``, reformatted, checked into develop branch. Thanks for reporting this!


I see what you're saying


Oh, leave it how it was then.  It accepts any capitalization of true/false.


can you plz fix this so that i can ship


I am fixing don't worry. Nearly there. There are a couple of other failures too.


Not exactly the same...seems that there won't be a '\n' between pre/suffix and the rest of the text if I do that.


Zertarako metodo hutsa?


With this one would work though.


You're right, it's cleaner for now. I'll change it. And thanks for adding those flags. I looked around to install locally and this looked like the prettiest way, though not very conventional.


mudança de titulo.


Kalterik ez bere horretan uzteak; sekula ezin da jakin kodea nola *hazi*ko den.


It is probably safest in these cases where there are a large number of changes to create a PR first and check that Travis is happy before merging.


New EnemyManager


I technically shouldn't have patched the official MarkdownTest files so maybe I'll remove this change later but it doesn't really matter


lgtm


Bad news...this fixes only "push" event..the other event are also messed up..and not only because of git.io...but plain not working as they should.

For example: "$repository__owner__name/rb (in $ref__branch): $__comment__user__login commented on commit $__commit__message__firstline http://git.io/vvBNC"
Only gets the repo name right and the git.io(with extended changes to the if in this commit)..
More or less i fixed them except "commit_comment" where i'm missing info...


Variables should be in camelCase


Oh, true.  You could add a "\n" on to them when you pass them in.  Or leave it how it is if you think it's cleaner.


This would break rhel7/centos7 (maybe 6 as well) as there isn't a /bin/ip link or otherwise, only /sbin/ip. Can you instead add a check for /sbin/ip, and if it fails, check for /bin/ip instead.


yo team add


surely the `iso_checksum` has changed?


Needs some "should not ignore" tests... http://www.smogon.com/forums/threads/3521891/page-4#post-6164689


Probably, but I'm taking this one step at a time :)


Confirmed; that's already fixed in OJS 2.4.6: https://github.com/pkp/pkp-lib/issues/337


(the correct, available iso is still downloading)


Fixed Issue #1
Enhanced app design. Although more enhancement is needed! Issue #3


Certains me donne déjà des idées et seront utilisés ! :)

En ce qui concerne interlude, de la transition de 1m30 à la finalité de la musique, quelque chose me chiffonne, c'est comme si j'attendais inespérément sans l'avoir, des accords à la guitare dans le meme thème, mais peut-être plus rapide, plus "mouvementé", toujours dans le même genre mais "changeant" enfaite :)
Pourquoi ne pas rajouter un instrument de 1m30-1m40 à la fin ? On est dans le registre d'une musique à la fois mystérieuse et qui nous amplie de soupçons autour de nous, qui nous invite à nous méfier, on ce laisse emporter... Mais n'étant pas grand mélomane, mon avis s'arrête là, je ne sais pas quel instrument ferait un bon compagnon ^^


It's a bad copy paste!
I have tested on this simple projet https://github.com/lauvigne/simpleRelease
I have done again the PR.


You should really be rebasing instead of doing merges since it makes the history harder to follow


I haven't seen that, I just tried pulling the latest master and `./gradlew assembleRelease` , it completed without issue. I'd guess gradle/sdk/build-tools version mismatch but don't see how it'd happen. Are you using the gradlewrapper or using your system's gradle? Try the wrapper.


This needs to be "0.6.0" for the templating to work. You probably need to add your newerst Cargo.lock.


Do they give you a cut for plugging them or you're just in love ;)


Thanks for checking!


Back in the day, we didn't have no stinking rebase on RCS.

Ok, I'll do that next time.


In Kuena 4 theres comes a popup with 2 input fields, one for the link itself and one for the text. I really like this solution and i case you leave the text field, it should show "«link»" or something like that?


I would even suggest truncating the iplog_history since 99% of our users don't use it


Thanks @Nowheresly


this looks ok to me (also almost everything passed on travis ;) )


ack


`permit` is meant to raise an error if there's other parameters, so this was blowing up as there were. It really doesn't work well with our flat params as Rails adds top-level params that aren't related to the model we're working with.

Usually this would be nested within an `alerts` object, so we'd have a `params.require(:alert).permit(...)`, and that subhash wouldn't have anything in it we didn't expect, and if it did that would be when `permit` should raise.


I wonder if we should/could recast this in terms of seconds since the last peering event?  That would avoid DONTNEED on a freshly peered PG where we may not have a warm obc cache and may get false positives.


When running "cargo run -- snapshot" i get:
"One or more required arguments were not supplied
USAGE:
    hat snapshot [FLAGS] <NAME> <ATH>> "

Looks like PATH isn't parsed right.


easier for us if youland your commits to master and not a branch.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 tests passed)

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


I am not in favor of renaming artifacts in general, and in this particular case as well.

In general, because renaming artifacts create incredible operational and legal headaches on the scale you can't even begin to imagine, in certain places :)

in particular, because If "samsara" is to refer to computing environment, (let's say algebra environment in particular), then its code is not contained in any single module. instead, it's dispersed around.


For some reason, running the program with no arguments prints both usage and license (cargo run).


hooray for java8 streams! happy to see so much code could be deleted thanks to them


Reinsert user and try again


excellent work! :+1: rebase and merge at will


Is this the list of all objects in the system?


I saw this in Kunena 4.0, however having thought about this a little, I don't want to start adding lots of functionality just for 1 button in the BBCode section. This popover that gets displayed when you hover on the "link" button looks as shown in the below screenshot.

![screeny](https://cloud.githubusercontent.com/assets/2019801/7147631/2b8e3512-e2f4-11e4-9195-3b4cd07e5919.png)

In my opinion, this is enough guidance for a user :)


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

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


Merge pull request


Can you add a longer comment explaining _why_ you would want to pass this (I don't think we even know)


deviceID and userID should have the same names as the attributes in rails


maybe make this true


What about logging the IOException and then rethrowing it?


Yes, it was easier to unit test this way.


It's the version shipped with latest Ubuntu LTS and also the lowest 5.5 we can test on Travis


cmake does support transitive dependencies, so there is no need add ethash-cl in 'eth' target. It's only in 'libethcore' and it's optional right now.


Javadoc and license header please.


Empty method ?!


Always pin to the target element and not the `$rootElement`.


:+1:  -- works


Better to wrap the `SchemaNotFoundException` in an `IOException`.


MarkdownDocs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


`proxy` -> `host`.


See [mode](http://hapijs.com/tutorials/auth#mode) for supported params


Makes sense. Others have suggested this as well, and it would certainly be symmetrical given the alternative attribute-proposal https://github.com/w3c/webrtc-pc/pull/196.


Yes that's already really helpful ;)


Why the check?


Ooops, was looking at the wrong thing


Look good, and we still need to fix one more error


Some declarations don't have names.  For example, we don't have names for declartions with complex computed property names.  If we don't have a name, then we don't put it into the list.


btw rearranging folder structure is a breaking change for bower users


This commit seems to be enough to fix the build on windows. All your other PRs shouldn't be needed unless you checked-out the source with the wrong git settings that turns linux line breaks into windows line breaks


Good good :)

I'm just waiting to here back from George and then the update will hopefully be released tonight, if not, tomorrow morning


Can't this just be a predicate?


I don't like getters that will only return one thing, or nothing


The usage and license is from the catch all final match statement. I think
in the original version I thought that was the intended effect. Removing
this line will fix that if it wasn't desired:

https://github.com/kbknapp/hat-backup/blob/master/src/hat/main.rs#L170

I'll look into the path here shortly (I'm on mobile right now)!

For some reason, running the program with no arguments prints both usage
and license (cargo run).

—
Reply to this email directly or view it on GitHub
<https://github.com/kbknapp/hat-backup/commit/a8518f321ee9b65536b6d1bde0dcbf6931ccd85b#commitcomment-10724346>
.


@sslavic  Revert the change and lets mark this Jira as 'Won't Fix'.


MarkdownDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


So something like `[Symbol.iterator]` has the name "iterator"?


retest this please


Так действительно гораздо понятнее :)


Correct.  (This is just preserving behavior btw).


También se creó la funcionalidad de one time login para el envío del mail a un mail ingresado, sobretodo se trabaja en auth en la periferia de eso.


This looks like it might be a simpler way to achieve this:

https://github.com/axyjo/leaflet-rails/pull/41/files#diff-54286332bc068201b74395aefc59ff24R29

```ruby
output << "var map = L.map('#{container_id}', #{options[:map_options].to_json})"
```


this fix #2  Issue


I tried this already without success. Did it again and got `cabal: unsatisified constraints` regardless of texmath min version, but now reinstall succeeded. Really strange...

Thanks!


Weird, I'll check tonight when I get home from work.

--
:::::::::::::::::::::::::
Diego F. Aguirre
973-885-0724

http://www.linkedin.com/in/diegoaguirre3d
http://diegoa3d.com/

On Tue, Apr 14, 2015 at 3:58 PM, dothework <notifications@github.com>
wrote:

> @Diego-F-Aguirre not much here to look at. did all the files get committed?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Diego-F-Aguirre/Blocstagram_V2/commit/47afd8ff26a35cfff85aa62e33433ecde1c8b076#commitcomment-10722273


I should have commented that to make clear why it was needed :)  I made the same mistake initially.


Great. Thx.


Should be in backend test not backend source.


np, thank you for the heads-up :)


Ah, the path I believe is because clap uses the unix `--` alone to specify
"only positional argument follow"  but `snapshot` isn't a positional, its a
subcommand.

It's perfectly possible to make `snapshot` and the other sub commands
positional arguments instead, but you'd "lose" the ability to easily expand
those later (as well as things like `hat-backup snapshot --help` ). It
would also be harder because not all of the commands require both PATH and
NAME.

If you want I can post up a branch showing the commands being positional
instead of subcommands.
On Apr 14, 2015 5:16 PM, "Johan Brinch" <notifications@github.com> wrote:

> For some reason, running the program with no arguments prints both usage
> and license (cargo run).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kbknapp/hat-backup/commit/a8518f321ee9b65536b6d1bde0dcbf6931ccd85b#commitcomment-10724346>
> .
>


broken and your +10% is your -i 20 intensity change


But it's a protected method...
For comparison: ```src/Illuminate/Foundation/Auth/ResetsPasswords::getEmailSubject()``` is just the same.


Хотя вот эту проверку с эксепшеном отсюда наверно можно убрать. Да?


Use junit instead of the assert keyword.


Ah yeah, good catch! I wasn't thinking about the Cargo.lock. I'll include
it tonight once I'm at a computer. 0.6.1 should be the min version, since
it fixes a bug with whitespace from 0.6.0.

This needs to be "0.6.0" for the templating to work. You probably need to
add your newerst Cargo.lock.

—
Reply to this email directly or view it on GitHub
<https://github.com/kbknapp/hat-backup/commit/52bd2de8051ff318c5683ef8599211028145e740#commitcomment-10724249>
.


Visibility of tunnel entrances and the little map on the top-right corner.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2109 of 2196 tests passed)

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


apologies, should have caught that on review.


`and more`?


Ok.


I think FE will like to have the popup stats returned in the plans shown in the `/api/courses/:id/events` endpoint.  Can we still do that?  If so, I'm ok with taking them out of `/api/plans/:id` and moving them to `/api/plans/:id/stats`.  Seems like `detailed_stats` is maybe better as `/api/plans/:id/review`, which would contain the `stats` information in addition to all of the exercise details.


This conditional will always catch. `emitEnvelope` is used to emit this node's own messages only.


remove


Why did program move to the front? I would think it replaces typeInfoResolver


What do you mean, "this should not happen"? If it's really the case, then it's worth printing a warning message.


This syntax in PhabricatorRepositoryCommitMessageParserWorker.php fails with PHP 5.2:
```php
$autoclose_reason = $repository::BECAUSE_AUTOCLOSE_FORCED;
```


I would suggest to use an already existing package for time, for instance joda time http://www.joda.org/joda-time/


If you are interested in this project please read the Report.pdf to get a full overview.


Why do you have to get this here?


uh? What leaves which possibility opened?


Please follow our bug reporting guide here: https://secure.phabricator.com/book/phabcontrib/article/bug_reports/


require_once (dirname(__FILE__).'/helper.php');


$items = (new modNinjaRssSyndicatorHelper)->getNinjaRssSyndicatorFeeds($params);


$doc = JFactory::getDocument();


I thought adding trywait_for_content was a good idea at the time, but it should probably be avoided.


typeInfoResolver


$db = JFactory::getDBO();


just clean up


just clean up, there is an outer @internal so this one is not needed.


@jaynus, I think this should have the ^0.33 too


> During the cleanup, pod's stage1 gc entry
the pod's?

> This gives a stage1 to cleanup anything related
huh?


Please check with UX / @kathi-fletcher about whether this "big movers" section is to be included in the review page.  I have some recollection that it maybe isn't because teachers know who is doing well or not.


FacilityTabGridView (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)


I think we should add a check to see if shallow is selected and then use that if needed. Also, we may want to checkout the git deploy command and see if it can be done that way too...


sure i will move it.


Please do the same fixes in sample solution readme files as well.

Thanks.


foreach($doc->_links as $link=>$array)


Not exactly what we want but it's a start. We might need to reuse this code for the place we want it to this. What we want is when the user clicks on a category it will open another grid view of items and not to an item activity which is doing now.


Would this be okay to be put on a live site?


With some minimal testing I did it looked good with Kibana 4.

Do you have anything else in mind?
On 14 Apr 2015 22:32, "Julien Vehent" <notifications@github.com> wrote:

> Is this going to work with ciphersuite being a
> map[string]ConnectionCiphersuite ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla/TLS-Observer/commit/3f5c36303743c65b4521538b2f9c3f90bf50cb7d#commitcomment-10721798>
> .
>


Is it needed to have both `getCurrentLedgerSeq` and `getLastClosedLedgerNum` `getLastClosedLedgerNum` is only used by `CatchupStateMachine::applyHistoryFromLastClosedLedger` in which case we know `mTrackingSCP` is false


Is this a weird name when used as the value of `course` above?


$item = new stdClass();


@hodeyp I was able to recreate this issue in Safari and added a fix to master.  Should be good now. =)


if (file_exists(JPATH_BASE."/components/com_ninjarsssyndicator/assets/images/buttons/".$item->feed_button))


The issue of tests running twice is gone.


r+


No, it started with a notification on a commit, not by me. That is why I asked what n9iels did.
Almost all notifications are on issues and PR's, but a few are on commits. This PR was linked to the commit, but not all are. In github in FF it is very complex finding the PR that contains the commit.


Is 10 whitespaces at the end considered many? If so, them yes I have tested.

2015-04-14 23:37 GMT+03:00 Šimon Lukašík <notifications@github.com>:

> You have tested with datastream with many whitespaces at the end?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shlomizadok/foreman_scap_client/commit/489e8051dd51776505920aa1b4404d251c1dcb1d#commitcomment-10723637>
> .
>


You should add more comments.


Really? I don't think so.


It all compiles fine but then I get when trying the demo file:
  Uncaught TypeError: Cannot read property 'RegisterVariable' of undefined
  Uncaught TypeError: undefined is not a function
  Uncaught TypeError: Cannot read property 'initialize' of undefined


I'd place normal above other flavors


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


What is the difference between BPP 1 and BPP 2?


NOOB


Is that better?


saw approval from gmarceau
at https://github.com/MonsieurNicolas/stellar-core/commit/f5ca9d204e16d6a58cf16eb8273faeda6a584508


merging MonsieurNicolas/stellar-core/herderUpdates = f5ca9d20 into auto


MonsieurNicolas/stellar-core/herderUpdates = f5ca9d20 merged ok, testing candidate = 1b202711


All links in here still seem to be broken. All produce an Error 404 online at http://organize.djangogirls.org/step_by_step/index.html -- Where should they go to?
(I've tested that page only, it's my first time on this great tutorial. Awesome, by the way. Really awesome! Congratulations!)


Were doing stupid things in code back then. Probably should remove this repo in favour of https://github.com/ethereum/go-ethereum/tree/develop/crypto/secp256k1


We do not have dependencies, do you still think we need a virtualenv?


you got it!!! you win!!!


`muster` with Travis-CI? :smile:


Oh me oh me, I didn't know there was a built in caching module! That was on my todo list! #happydays


OK. I Added a line.


What does this "4" represent? Should it be a CONST?


Sorry for forgetting about the time-stamps.

(BTW the data quality is not worse see the GBV entry but I guess the MARC translator might need a better handling of titles and subtitles.)


hola


I'd keep the function name. "bad FP values" is pretty vague as it is.


@dojadop Correct. We're assuming Java 6/7 users aren't on the most recent versions, but the same workaround for Java 8 applies to those as well.


odd -- let's look at that. LoC also uses MARC under the hood, so this should definitely be doable.


thx for your reply.

im assuming i can confirm all the mentioned steps of your guide. including fsuipc (registered) installed as administrator

i just opened the serial monitor to check if arduino sends some commands.

the LUA log shows:
********* LUA: "OAC_CMDGW" Log [from FSUIPC version 4.939k] *********
    53415 System time = 14/04/2015 23:34:03, Simulator time = 23:33:23 (21:33Z)
    53415 LUA: beginning "C:\FSX\Modules\OAC_CMDGW.lua"
but no incomming commands.

do i need some other plugin? FSX SDK? AXE SDK?

i owne FSX PRO (boxed) + ACC pack + AES + Airbus X Extended + FSUIPC4 + your OAC_CMDGW.LUA

your idea is amazing and i hope so very much, that i can build my own Arduino A320 Homecockpit ,-)

Take care | kind regards


What am I missing? :)


can you please try with the trunk of TOD ? thx


Add a comment then about what it will do in ZK with regard to the path.


Oh yes, `h2` implies a heirarchy.  These can be wrapped in the html5 `<header>` tag and grouped if need-be.


\o\ \o/ /o/


wow you are quick 👍


ok


doh, whitespace isn't consistent _again_ :^(  Fixing...


Looks fine to me, although I thought typicalvalues.m2 would need entries for pairs (pairs HashTable and pairs BasicList). Are they generated from / replaced by the comments you added to actors2.d? Anyway, never mind my comment elsewhere, I'll let you fix this up, I won't try to push the changes I made.


Hello Pieter!

I was also using ReceiveBytes in the [ZGuideExamples Espresso](https://github.com/metadings/zguide/blob/master/examples/C%23/espresso.cs#L91)...

Please can you update [ZeroMQ the Guide](http://zguide.zeromq.org/page:all)?


is there a recommended place to get these files if we still want them? or is support being dropped?


It is required because of accessing private property, `scrollDiv_`.


haha let me deploy this :)


Thanks. updated.


fixed in 96bf89065aa756a3e5412a4d97dc9cdb270d6ccb


Yes, the comments in the *.d and *.dd files get added to the file tvalues.m2, which is loaded by
the file typicalvalues.m2, and contributes items.  I prefer having the typical value descriptions near
the code, in those comments, so I took the opportunity to move them just now, since I had to change
one, anyway.

Thanks!


I put a download link to the Hello World Python in the readme. Try that
one. I think it may be the same as the one Nick just sent.

    ____       __
   / __ \___  / /____  _____
  / /_/ / _ \/ __/ _ \/ ___/
 / ____/  __/ /_/  __/ /
/_/    \___/\__/\___/_/

Not sent from my iPhone because I don't have one

On Tue, Apr 14, 2015 at 3:27 PM, ploquel <notifications@github.com> wrote:

> Wow awesome, Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PieCrafter314/python-browser-bar/commit/205dba1a7a3e7e8d2a89ce40a028690a68e0cadd#commitcomment-10723465>
> .
>


Ah... I see.  My Ruby fu is weak apparently.  Thank you.  :)


Ups, sorry for this.


I wonder how that's broken on master


If `$tag` isn't set, then the URL route never matches and this method never gets invoked, so I'm not sure this check is necessary.


Hey -- before you go any further, you should look at -dev -- there is *major* refactoring between -dev and master, so if I were you I would not work on master right now.


Haha, yeah. My bad dude


I allready installed the one Nick sent me. It seems to have alot of tools. I opened your program with pythonw and it ran. Nice job.


Merges from any work done on master back to dev would potentially be a lot of work.


Why was this not disabled for the d803? Was it a mistake or is it required for d803?


Ce n'est pas très utile, je ne crois pas que les super-utilisateur ait un accès aux echecs de compilation de carnets qui ne sont pas les leurs (à vérifier)


Mmm, is this right? It blocks my exploit, but it HTML-escapes instead of CSS-escapes. Might break valid stuff later once embeds are opened up more.


I will look at it later. I guess that they use the subfields differently...


sure, that would be better.


Il faudrait rajouter un lien pour contacter les administrateurs, sauf si on utilise le formulaire de contact pour ça.


Yeah, you might be right. Will take a look into this, thanks.


`$lang['Roll eye']` should be `$lang['Eye roll']`


Ça pourrait être cool d'avoir des layouts par défaut sous forme de `fixtures`. Mais c'est hors-scope ici je pense.


`parts[:2] != ["AI","TAG"]` -> `not tag.startswith('AI_TAG')`

`[x.upper() for x in parts[3:4]] != [tag_type.upper()] ` -> `parts[3].upper()  !=  tag_type.upper()`


It seems like this change was a mistake?


You can reverse condition and return instead of continue.


Yes, protected methods can be overwritten by definition.


May be return `AVERAGE`?


Un peu de doc sur ce que fait cette fonction ? Je ne suis pas sur de tout comprendre =)


This is actually a minor distinction about which I'm not sure which way is right. Functionally, it'll work either way. But it seems more correct to do it this way, and I'm not sure I'm right on that.


Ici aussi une docstring ne ferais pas de mal, même si on se doute bien de l'utilité de cette fonction.


Is there an option to disable this in order to keep using tabs?


That is MARC from GBV looks like this:
```
<datafield tag="245" ind1="1" ind2="0">
  <subfield code="a">Zotero</subfield>
  <subfield code="b">a guide for librarians, researchers and educators</subfield>
  <subfield code="c">by Jason Puckett</subfield>
</datafield>
```

and this is how LoC does it:
```
<datafield tag="245" ind1="1" ind2="0">
  <subfield code="a">Zotero :</subfield>
  <subfield code="b">a guide for librarians, researchers, and educators /</subfield>
  <subfield code="c">by Jason Puckett.</subfield>
</datafield>
```
Uff, I guess we need more examples for these to see how to deal with it...


The following block of changes is the result of inverting the `if` condition as suggested by Resharper.


I have no idea what the backslash actually does. I assume this function is for some kind of URL extraction using regexps?

It's interesting, though, that I could fix the bug without understanding what is going on. Maybe an algorithm that detects simple programming patterns (and small violations thereof) could also fix some bugs ;)


I found a better fix in febb5ee6e6114b7808f46321b9b7832c4d3b553d btw


Similarly, in routes.php, we say that this endpoint should only route to this method for POST requests. So we don't need this check either.


What is this for?


We have good logic for this in place for the 200 field:
https://github.com/zotero/translators/blob/master/MARC.js#L383
we should just extend this to 245 -- probably just an oversight.


I noticed this will introduce issues with other languages that are dependent on `method.method`. I'll make a change that makes it safer;

The problem is that Java doesn't have specific selectors for things like variables, etc. and only has `meta.method.body` — which is everything in a method block.

If you wan't more fine grained selectors, I'd suggest creating a copy of `Java.tmLanguage` and add your changes there, it's the only solution I can think of.

For now I can make a specific block for Java Methods and change the color to either what methods usually have in other languages, or to the default white text color :-)


Consider me red-faced. I guess I didn't a mention of a minimum Xcode version, so got confused.

Thanks for the quick response, @OliverLetterer !


a pydoc for a nested method is superfluous -- if you still want to document this method, use a comment (and none of the :param stuff.)


@jaynus, you can remove that setVariable propagation


We definitely don't want to be retrieving every `CheckIn` record in the system.


Because this will be a common usage pattern I have added an `org.specs2.specification.ExecutionEnvironment` trait

```scala
import org.specs2.specification.mutable.ExecutionEnvironment

class MySpec extends Specification with ExecutionEnvironment { def is(implicit ee: ExecutionEnv) = {
  "Let's check this scala future" >> {
    Future(1) must be_==(1).await
  }
}}

```


Hi Paris, this looks good to me. Did you have any questions about the gitflow steps in particular?


The difference in both cases is the "ids" parameter.
For the api you have to send the ids of the collections that you want (as it's received from the header), but for the web that parameter doesn't exists.
I could define "ids" as an optional parameter but in the case of the API if you don't send anything in the header you would get all the Releases(with the restriction of the permission), but I thought that wasn't the behivour that we wanted.

I will user different names for each case, if that's OK.


OK, no problem.


(though I don't understand the code there? Looks wrong?)


I feel the need to do

data[key] = data[key][config.embeddedNewKey] ? data[key][config.embeddedNewKey] : data[key];

in the fetchFuntion to unwrap the embedded items. Should this be a fix in the coming versions? Otherwise is there a better way to do what I do? Please also check this:

http://stackoverflow.com/questions/28732273/angular-spring-data-rest-exchange-the-underlying-fetch-function


LGTM


I heard you guys say on the conference call we had one person who hadn't got theirs committed yet and wanted to make sure I had correctly done it.

Thanks you!

Parris

Sent from my iPhone

> On Apr 14, 2015, at 3:33 PM, Eric Johnson <notifications@github.com> wrote:
>
> Hi Paris, this looks good to me. Did you have any questions about the gitflow steps in particular?
>
> —
> Reply to this email directly or view it on GitHub.
>


can you set the precision to be always 2 or 3? This is not the case currently.


Great catch.  Thanks!


@bittner good catch! +1


I figured the round would do that, but I guess I need `'%s' % round` instead, didn't consider that MPL might change it on me


As it is right now, every time you click the yelp icon, it will log to the console 15 places in San Francisco. The searchTerm that it posts to welcome#search is not actually an address, since the api call is hard coded to search San Francisco address. Rather, the searchTerm is "food" or "restaurant" or "business". It's a sort of filter for the search api call.

Steps to make it work:
In the address bar, rather than type in an address, type in "food", then click on the marker, and click on the yelp icon. Check the console to see list of business names.


There are some serious, fundamental issues with this code.

The general idea is much simpler than what you're doing here:
-   Use the item tag (from the URL) to try to retrieve the item record in a `try` block. Catch `ModelNotFoundException` and return a 404 result if it can't find the record.
- Look at the request body for `room_id`. If it's not there, return a 400 response.
- If `room_id` *is* there, attempt to retrieve the room record with that ID in a `try` block. Catch `ModelNotFoundException` and return a 400 result if it can't find a record. Make sure you understand why we return 400 in this case but 404 when we can't find the item.
- At this point, we have what we need to create a new `CheckIn` record. Don't generate a random time for the `created_at` field. The ORM will handle that automatically.


It's also hard coded to print 15 names. I will change this to 5. Need to figure out how it gets these names. Is it by rank or what not?


Much better Naming convention


Yessir


this should be intersect


lgtm


@Eonasdan Can you describe what you did to these files to fix the permission issue? We may need to update any of the font files later and want to avoid reintroducing the issue.


notr


params get out of alignment.


Awesome.


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2172 of 2196 tests passed)

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


>Tools.excapeHTML
kek


@hotice Fair enough.


Additionally:
- ['ROUND PUSHPIN' (U+1F4CD)](http://www.fileformat.info/info/unicode/char/1f4cd/index.htm) (:round_pushpin:)
- ['CITYSCAPE' (U+1F3D9)](http://www.fileformat.info/info/unicode/char/1F3d9/index.htm) (no emoji yet, apparently)


I'll add some of my verification code from PicturIt. We need to make sure the uploaded images are valid :P
we should also definitely allow sprite2 files, however the assets should be processed locally with jszip.


up to b747b0dd1


Thanks! I'll follow https://secure.phabricator.com/book/phabcontrib/article/bug_reports/ next time


@czue https://github.com/select2/select2/tree/stable/3.5 has the 3.5.2 code.


Slept by in 41b4c3cbc12cf0ad3a7a506a3c0a604e58554ce4 + 29ef012c16c0a933ce3448751668929c9972ba50


fixed this typo in later commit


Hi @gpoore

If we shouldn't just enclose the "#1" instead of "escapeinside}{#1"} could you
please add a comment to explain why?

Thx+Peace, R.


Moved this definition slide to first slide after initial exercise


Hmmm I don't think this would do a deep copy. Can we at least make a note about that here?


Nourhan Hassan </br>


@jvasileff It's probably a reasonable option for unassigned code points in the range `0..#10FFFF`, yes, especially since it's going to be really expensive to impl `isDefined()` in JS. However, I think it's probably still better to throw for `int` values outside that range, don't you think?


Best commit ever! :P

"Seriously, what was... oh."

I have no idea why that's like that myself (though my guess is it's something to do with there being two i18n folders??), but if you figured it out, perhaps adding a comment would be good.


11-Nourhan Hassan </br>


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


Nourhan Hassan <br/>


workshopped.


std::cout << "Moo says Hello World...";


[![The Fighter of the Spirit](http://api.artsmia.org/images/1366/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1366)


[![Tahkt-I-Sulayman Variation II](http://api.artsmia.org/images/1704/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1704)


[![Chinese Rock Garden](http://api.artsmia.org/images/31068/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31068)


[![Black Moonlight](http://api.artsmia.org/images/65021/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=65021)


lglglglg


That should say "User authentication groundwork laid".  Oops.


I think this should rather be `Have you uncommented Dotenv::load in bootstrap/app.php? [yes|no]`


Thunder Wave should be separate because it doesn't fit under a description like "should ignore type immunities" :-\


Thanks!


fast-forwarding master to auto = 1b202711


There's a missing newline here.


I know, didn't think of it well on the rush. I'm updating the tests now.


Building name and description don't have to be unique.

Also, if you're just going to count use `get()`, that actually retrieves all the records and creates PHP objects for them. Very inefficient.


I couldn't get this endpoint to work. I get the following error:

```
FatalErrorException in ProviderRepository.php line 150:
Class 'Illuminate\Html\HtmlServiceProvider' not found
```

Do you guys see that?


Yes, that's true. It doesn't. But then, neigher does the "bad" example :)

I'll add a note.


And, corrected :)


What is this redirect for?


When you successfully create a record, you should return 201. Also, instead of returning a string saying the operation was successful, return the object that just got created.


Sure, makes sense to me


They discovered this exploit...
http://www.spigotmc.org/threads/psa-json-in-books-can-lead-to-clickable-commands.59950/
And
Idk if that helps you.
http://paste.ubuntu.com/10791544/


I'm not sure that routes.php will route to here without a building ID, so I don't think this check is needed.


It seems very fast at first although it got laggier as it went on. Also RIL and WiFi are bugged. Went back to MIUI ROM and it works wonderfully again.


Yeah, I had a blast of a time with myself :) It was not supposed to be "i8n" but rather "kalite.i18n" :) The revert is because I was too excited about the whole __package__ hack to actually test the fix before pushing :)


Suggest making this "packagename" for consistency across the replaces all being lowercase.


This is a confusing response message. 404 means "not found",which is appropriate here, while "bad request" is 400, which isn't.


Resolves https://issues.jenkins-ci.org/browse/JENKINS-23262


1. Modified recipe GET route to include id params and not query.
2. The URL I was using for the one recipe display was returning html and I needed it to return json


Class assignment A3


Damn... you've been busy. This commit is gonna totally save my bacon. Thank you so much!


This endpoint didn't work either.

```
FatalErrorException in ProviderRepository.php line 150:
Class 'Illuminate\Html\HtmlServiceProvider' not found
```


Should be part of the SDK it downloads for compilation. I should be able to make one on my computer today.


The logic for the 200 field in the Unimarc case should be able to adapt for the 245 field in the Marc21 case. What do you mean with "the code there [...l]ooks wrong"?


Any plans to release to pypi? I just ran into this bug today.


Hi, the error is gone! Thanks.
I still cannot detect any object, but is a different problem.

I continuously get:
```
No descriptors loaded
********************* found 0 poses
No descriptors loaded
********************* found 0 poses
No descriptors loaded
********************* found 0 poses
.....
```


I was using Java 8. For some reason Intellij is not behaving the same way as mvn. The fix for now is a cast, we can clean things up later.


After thinking about this a little bit, I need to do more here. Instead of just throwing an error if the strategy is defined, I should check to see if the defined strategy also uses the same keyword. If it does then there's no reason to throw an error because it's just a re-register (which can be safely ignored).

The place where an error occurs is when attempting to register a locator name that already exists but is tied to a different keyword. That relays that there are two different intents and ultimately is an error.


I don't think people will run into this issue once we switch to Mysql. I suggest we punt on this issue for now.


Your commit https://github.com/square/keywhiz/commit/a2f01c8c17d86ff5ed9b83053c552b3f53a5968c should address this.


I think this is totally wrong and will confuse people a lot (if they aren't confused yet by the branching of uBlock to 2 versions), as uBlock0 should stick to the root icon, and the fork (chris's version) should change their icon instead.


But brainstem results under anything other than `results` are not ordered, so this won't work in practice.


initialize should do everything until before the initBuffers function call, since initBuffers is the only thing that needs the vertices/vertexnumber. so after initializing the renderer, the user could give it the scenegraph and the renderer can traverse the scenegraph and figure out the amount of vertices for initializing the buffer


bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla bla
bla bla bla bla bla bla bla bla bla bla bla bla bla


Would it be possible to view the slides "online"?


a


aDSADASDA
SDASASDASD
22@@@@@@@@


Finally got around to trying to revert `reducer` to `reduceLeft` and nix `recur` (for curiosity and intrigue) but managed to get the type checker stuck in an endless loop. I envy your Swift fu sir :persevere:


Aaaaaaaaaaaaaaaaaaaa


Ну я бы типа false возвращал.


Old comment.


FacilityTabGridView (evanweaver) - Win64-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)


aaaaaaaaaaaaaaa
![logo 2x](https://cloud.githubusercontent.com/assets/163234/7149391/8f61d6a4-e315-11e4-8af5-1b5f66e8d3a6.png)


ВЗЯЛ И ИСПОРТИЛ ВСЁ


I think none, @alexsoble — looks like were all misunderstanding Ben above.


a  little more love on your try/catch just in case @aldenml


Там смысл в том, что у активного персонажа disabled и стрелка, а у остальных ссылка на переключение. И ещё надо, чтобы у активного без ссылки, но я не помню, как это правильно делать, надо доки читать.


Git探索中，还是玩不转啊！


might not have an assignment for all predicate symbols


LGTM modulo the change of camelCase vs. lowercase.


No semi-colon needed when closing your loop statement. However, on line 21, you do need to end your method with a semi-colon. =)


Does that glob pattern work recursively? Because there are no awk or sh files in `runtime/` itself.


Thank you for review Rob, switching to camel case... will do force push after that.


I assume so as it's called GLOB_RECURSE, but I haven't tested.


Basically 7cec8b68313d8535fcce9e0058e96636f9dfed1b but better.


This is wrong for a couple of reasons:
* Blocks are always created and written by a type. `BlockAssertions` needs separate methods that create and write the block using the appropriate type.
* Date actually uses a different type of block than bigint (it uses integers instead of longs).


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1129 of 1129 tests passed)

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


Sad about the missing test on this on the `#publish` method.


Move this refactor to a separate commit


Move refactor


For more details see
https://help.github.com/articles/dealing-with-line-endings/


Oh, wow. It's getting late here.. please go on. :)


my guess is that you wanted to make
```java
properties.getConstants()::containsKey
```
to be
```java
node.getGroupBy()::contains
```
?


also, you probably need to do the same operation in the unpartitioned case as well


:+1: Glad to help


Static import `nCopies `


:+1:  thanks


@drlube Is there a better way to create a "slug" out of Hebrew?


esta no me gusta


this should be combined with the input constant bindings


> How the payed hacked client works...
> http://pastebin.com/a0e9fU4U

That client does some ChatComponent magic in addition to the JSON stuff. I'll see if that fixes it.

> They discovered this exploit...
> http://www.spigotmc.org/threads/psa-json-in-books-can-lead-to-clickable-commands.59950/
> http://www.spigotmc.org/resources/bookexploitfix.5897/

That plugin has only 79 downloads - nothing to worry about.


This is missing a semicolon.


lol, yeah, that's so broken


I don't think the name of the PR and commit is right since it's not clear to myself who am I and wouldn't like that to be in the commit history of the compiler. In any case I appreciate that you still have the energy for that kind of thing but please put your own name there for `flip` and not mine. Best, D.


No, for the unpartitioned case no constant can be propagated


Do we need gccgo?


I don't think this bottle was intended to be added. See the note below the bottle block.


good point, we missed that when removing it.  will fix now.


@ioggstream The goal is to pass something of the form `-P "escapeinside=<open_delim><close_delim>"` to the command line ([Pygments command-line reference](http://pygments.org/docs/cmdline/)). Because the delims will often be unusual characters, I want to quote the whole option to try to minimize the chance that the shell on some system will interpret some character in an undesirable way. Currently, `\minted@def@optcl@e` passes `#3=#4` to the command line. So the easiest way to get the correct part quoted is to put the opening quote in `#3` and the closing quote in `#4`. There are ways it could be made more elegant, but this is currently the only case where I want to make sure the option is quoted.

Looking back at this now, I'm thinking that single quotes might be better under non-Windows systems. Using `{-P escapeinside}{"#1"}` works under Windows and provides correct escaping. I'll have to try some tests under Linux, but if `"#1"` or `'#1'` works there, then there's no reason not to change to the more straightforward form.


                              dispatch_async(dispatch_get_main_queue(), ^{
                                   // Your code here!
                              });


the path should be /home/anaonda/envs/py3k/bin


If this happen with MPI_BARRIER, it is suggesting something wrong is gong on more than just the timer runs ahead.


@Alexander01998 You don't know how this is being spread out. I find this source code in hack forums xD.


besides since it seems I still do care, I would simply say `flip f x y`  is `f y x`.


Thx!


Not a huge deal, just a style thing -- remember that JavaScript users prefer camelCase over under_scores.


you don't need this if-clause check here, it is already filtered


Looks great overall! Thanks for putting your files in weekly folders


> Is that not something we should fix in the IDE instead?

We should. However, either way the shorter syntax is much easier to read.


Needs a `defer tick.Stop()`


Let us know after reviewing Robert's code if you have any questions on how this code works! Remember that you can either come in / hipchat in on Saturdays for extra help if you need.


The compiler validates all references when converting them to the fully-qualified form that appears in the documentation XML file. Since there is no `StackOverflowException` in the PCL, this reference would give a warning during this translation unless it is fully-qualified in the source. Note that tools like Sandcastle are not limited in the same way; generated documentation for this file will correctly link to the MSDN documentation for [`StackOverflowException`](https://msdn.microsoft.com/en-us/library/system.stackoverflowexception.aspx) as long as it appears with the form you see here.


92380320_RadianceGTest (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2197 tests passed)

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


Just a style thing -- JavaScript users prefer single quotes (') over double quotes (") for strings


Needs a `select` on `done` channel avoid leaking a goroutine when closing the broker.


Nice use of `typeof` and strict equality (`===`)


This is a reference to a non-public type in a different assembly.


Here's what I have in mind in terms of work flow, but just so you know why I'm emphasizing the previous convention:

```python
from requests import get
from libextract import extract
from libextract.strategies import TABULAR
from libextract.coretools import node_processor
from libextract.formatters import table_json

@pruner_or_filter_or_processor
def if_table(node):
    if node.tag == "table":
        return table_json(node)
    else:
        return node

strat = TABULAR + (if_table,)
tabs = list(extract(height_data.content, strategy=strat))
```


Found BROKEN by full.


Comment isn't quite right


In that case, what's the benefit of using the fully=qualified name?  Will the compiler still verify it in some way?  If the type is a different assembly, this wouldn't seem to help refactorings, and if the type is internal, it wouldn't seem to help with documentation linking.


@evliu session bonanza: a session within a callback within a iron router route method


I don't have a problem with these being shortened for readability reasons; I was questioning the cited reason.  We should ensure the issue is raised to be addressed for the IDE as well.  Thanks.


Also, this never returns an `error`.


I can tell you come from a formal programming background :smile:

Another style thing -- JS users prefer:
```
function calc() {
     // stuff
}
```

over:
```
function calc()
{
     // stuff
}
```


Makes sense.


D'oh -- you're right. I added `done` after adding this, but forgot to use it.


r+


Nice comments! JavaDoc!


Nice work David!


saw approval from MonsieurNicolas
at https://github.com/graydon/stellar-core/commit/5d8f63301adf2f752bceb2833479e91f8d939907


merging graydon/stellar-core/fuzzer = 5d8f6330 into auto


graydon/stellar-core/fuzzer = 5d8f6330 merged ok, testing candidate = b9965ef4


Yeah, I was wondering about that. I thought that this function *might* return an error in the future (this code is still WIP) so didn't want to change the signature if it did. I can remove it, I don't think it'll ever return an error, not now anyway.


Agree ok to remove generic stuff for new installs, but to not touch for upgrades.


Yes and probably it should be merged into previous line.
On Apr 14, 2015 03:57, "Vladimir Sapronov" <notifications@github.com> wrote:

> @cyberb <https://github.com/cyberb> Is syncloud-resize-sd.service added
> twice by mistake?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/syncloud/platform/commit/3d05bc96ba10547af182fcccc4161d45d9b2c2be#commitcomment-10705182>
> .
>


Thanks for the style tip, Parker!

     On Monday, April 13, 2015 8:04 PM, Parker Lusk <notifications@github.com> wrote:

 Nice use of typeof and strict equality (===)—
Reply to this email directly or view it on GitHub.


Good use of RegEx!


Not a big deal, but I always like to use [strict equality](http://stackoverflow.com/questions/359494/does-it-matter-which-equals-operator-vs-i-use-in-javascript-comparisons) (`===` and `!==`).

For example, `"5"` is a string and shouldn't really equal the number `5`, but if you do `"5" == 5` you'll get back `true`.
However, if you do `"5" === 5` you'll get what you expect, which is `false`.

There are some cases for `==` -- like when you don't want to be strict on user input, but most of the time I use `===` just out of good practice.


Great job Wing!


These changes are for `calendar`. Unfortunately, there's still this weird issue of the calendar URL forcing itself to be `calendar/` (with a trailing slash) whereas other pages on the site do not forcibly redirect the same way:

(they accept the path with or without the slash at the end -- but do not forcibly change themselves, AKA if you type in the URL without the slash, the page you land on won't have a slash at the end.)


Ah, so much nicer. Only 7234 more instances to go :smile:


This doesn't look right. The predicate needs to be intersected with the node.getCurrentConstraint() to be correct. You should basically copy what is being done in visitFilter and copy it so that it is run here every time.


:muscle:


@frankrolf I had to add `reload(outputTTHints)` after this line because the list of hinted glyphs would not change after the first run (even after selecting other glyphs), and because the output file just kept growing and growing with repeated lines.
I tried to make some changes to `outputTTHints.py` to prevent these problems, but couldn't really understand where they're coming from, so I went with the quick solution of using `reload`.


Good job for using strict equality! (`===`)


The users don't have to write their own decorators, they could just write:

```python
@prunes(selector)
def pruner(node):
    return node, metric(node)
```

But if they want to make their code configurable then they can write their own decorator. I think the `@node_processor` decorator should a class that can compile into a function once at runtime, i.e.

```python
class NodeProcessor(object):
    def __init__(self):
        self.tag_map = {}

    def on(self, *tags):
        def decorator(fn):
            for item in tags:
                self.tag_map[item] = fn
            return fn
```


testing


这两个ini文件不需要检查, 这个是固定好的配置文件


Great style! Good job!


DISTSTYLEs EVEN and KEY are optional because they can be inferred from the distkey specified in the columns, maybe we do not need them


Make sure to space this out correctly!

```
function getLength(number) {
    return number.toString().length
}
```


rax retry tests


null check


Great code!

Try adding more spaces between lines and adding more comments for yourself/others when you look back on it.


only introduce filter if not equal to TRUE


Just a style thing, but JS users prefer single quotes (') over double quotes (") for strings


Created dotnet/roslyn#1968.


Hello,

Break out of the POAP process on the switch in question and login to it. Once you are in the switch, issue a “dir” command. Locate the relevant “poap.log” file in the bootflash:, and then execute the “show file bootflash:poap.log.xxxxx” command, and this should tell you exactly why the script is failing to execute. See the example below.

[cid:AC4224A9-2562-475B-AEC1-E2D4AA2F25A1]

Regards,

Jody

From: ashfaqch99 <notifications@github.com<mailto:notifications@github.com>>
Reply-To: joddavis/nexus9000 <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Monday, April 13, 2015 at 4:46 PM
To: joddavis/nexus9000 <nexus9000@noreply.github.com<mailto:nexus9000@noreply.github.com>>
Cc: Jody Davis <joddavis@cisco.com<mailto:joddavis@cisco.com>>
Subject: Re: [nexus9000] Update poap.py (32b7bf9)

I am using above mentioned code, but due to some reason it poap script download successfully but not executing giving me error "POAP Script execution failed".
Any idea how i can resolve this issue???
downloaded POAP script file
2015 Apr 13 21:38:17 switch %$ VDC-1 %$ %POAP-2-POAP_DHCP_DISCOVER_START: POAP DHCP Discover phase started
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Using DHCP, information received over mgmt0 from 192.168.175.120
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Assigned IP address: 192.168.175.180
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Netmask: 255.255.255.0
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: DNS Server: 192.168.175.1
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Default Gateway: 192.168.175.1
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Server: 192.168.175.120
2015 Apr 13 21:38:39 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script Name: poap.py
2015 Apr 13 21:38:49 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script download has started
2015 Apr 13 21:38:49 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: The POAP Script is being downloaded from [copy tftp://192.168.175.120/poap.py bootflash:scripts/script.sh vrf management ]
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_DOWNLOADED: Successfully downloaded POAP script file
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_INFO: Script file size 20404, MD5 checksum f4427f42bc28b0e6d1bdb28d286e78d7
2015 Apr 13 21:38:50 switch %$ VDC-1 %$ %POAP-2-POAP_SCRIPT_STARTED_MD5_NOT_VALIDATED: POAP script execution started(MD5 not validated)
2015 Apr 13 21:38:51 switch %$ VDC-1 %$ %POAP-2-POAP_FAILURE: POAP Script execution failed

—
Reply to this email directly or view it on GitHub<https://github.com/joddavis/nexus9000/commit/32b7bf997654e35e4d7a45c5e566ad0bae2ac70c#commitcomment-10701810>.


Good job using strict equality (`===`)


I'm not seeing this fail locally... Why do you think that is?
`CoreMatchers`?  Where is that from?


Good job! Thanks for being super classy ;)

I like the spaces between logical chunks of code and the comments. Keep it up!


Could do with a space after the ellipses.


Do we want to use any test frameworks for our stuff?


Yeah, it works
```
>>> from rest_framework.reverse import reverse
>>> reverse('v1:taps-list')
u'/api/v1/taps/'
```
It looks like [request](https://github.com/tomchristie/django-rest-framework/blob/master/rest_framework/reverse.py#L29) can be passed as an optional arg to build a fully qualified URL


scroll イベントは高頻度で発生するので、 https://github.com/cowboy/jquery-throttle-debounce などを使って適当に throttle したほうが良いです。

```js
$(document).scroll($.throttle(250, function() {
    $(this).scrollTop() > 50 ? header.addClass('scroll') : header.removeClass('scroll');
}));

```


``prototypes`` branch is better :D


eetetet


Good thinking. I'll test and likely accept ASAP.

Thanks!


Obviously this is "questionable" content for a comment. In my mind, the fact that this change raised the questions it did is one benefit. I would also argue the following:

1. If we are referencing a specific type, we should make an effort to use a *proper* reference, even if the IDE doesn't provide additional help today. There are a multitude of potential benefits we could get automatically if a future release does support these references (e.g. Visual Studio could go so far as to open the [corresponding file](https://github.com/dotnet/coreclr/blob/master/src/mscorlib/src/System/Collections/Generic/Comparer.cs#L89) someday, even across repositories).
2. We should not use a "plain" `cref` attribute, because that would introduce a compiler warning today.

Note that I would certainly be in favor of a rewrite of this comment that provides the same overall information without referencing the internal type directly. For example, it could refer to the "default comparer for this type", etc. I didn't spend too much time on this particular aspect for the first pass.


scroll イベントのたびに DOM が取得されてしまっているような気がするので、$(document).scroll() の外で header は宣言したほうが良い気がします。


Firefox login and upload popups are working just fine.
Firefox has never made an effort to stop me from downloading an already
recapped document,.

On Mon, Apr 13, 2015 at 6:34 PM, The Free Law Bot <notifications@github.com>
wrote:

> Sounds good. If you get it pinned down, let's get this convo moved over to
> an issue so it can be tracked properly. Seems like it could be as good a
> starting point as any.
>
> On Mon, Apr 13, 2015 at 3:04 PM simality <notifications@github.com> wrote:
>
> > Again, working in Chrome, will test in Firefox in a few hours:
> > As a test, I also went to this docket:
> > https://ecf.gand.uscourts.gov/cgi-bin/iqquerymenu.pl?188990
> > And pulled document 147 which is available on recap. I did NOT receive a
> > notice/warning saying that the document was already available for free.
> >
> > At a guess I'd say that notifier.js is broken.
> >
> > I need to get ready for work, will post more in a few hours. If not
> > possible will post more after I get off work.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/freelawproject/recap/commit/3e3261ca4af56c056d7f059acb9091e03c3ba55d#commitcomment-10702065
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/freelawproject/recap/commit/3e3261ca4af56c056d7f059acb9091e03c3ba55d#commitcomment-10702486>
> .
>


:scissors: if you know what I mean :^)


Why do we need to throw current buffer here? throw/cactch logic is unneeded.


I don't think you need this field anymore


Hi Jie,

I dont think your mutation has the same interface as needed for integration. Please refer to @p8peng mutation classes. I tried to do it myself but I realized that its better that it's corrected by you on your branch.

Akshat


前来点赞！


Sorry, I was mistaken. Fixed in 4f84f5e


:warning::rotating_light: TABS?!?! :rotating_light::warning:


It wouldn't be thread safe, though you could build a list of multipliers in a thread safe manner and then make the final product.


I can still trigger an infinite loop without it.


I like this idea. Let's have this be a milestone of sorts, maybe after an official release, and see if people like the decorator idea in the first place? I'm planning on drawing up publicity for this soon, maybe this coming weekend, got school projects to finish too :(


I don't think there is a need now to drop the currentConstraint from the TableScanNode after JSON serialization/deserialization. That would probably make this class more uniform and easy to understand.


It's all good :smile:


annotate JsonProperty?


I am not sure what I will be doing with this, but it looks interesting.


How to clone your http://sundaykofax.github.io/baby-legs/ to my git hub?


Whoops, I meant in ClientComponent under ApplyArmor


oops, i mean put the lint part in a separated lib. I have created the `lint.js`. so you can run `lint` as a separated task. And have you write a test for the mismatch?


Ok, I think we should draw up publicity first and get more feedback before we start implementing anything gigantic.


The user info is added into the file


groupingColumns => localProperties


I guess we can start defining building-specific soon, it would remove the need to specify health here.


* mean we import all the controller from the controller package


04-09-2015-Lin Xu-Assignment3.html


all variable might need to be private from now on


Good point, it could be a parameter on the health component instead


piggy vivo back but need recoding to fit with our logic


Thought about this more, and your original change makes more sense:

```js
window.location.href.replace(/\/[^\/]*$/, '/');
```

Reasoning:
1.  If `is.node` is `true`, we don't use any sort of `location` object.
2.  If `is.node` is `false`, `window.location` should exist.
3.  If you are somehow causing `is.node` to be `false` when you're running on the server, it's your responsibility to provide `global.window` and any necessary properties thereof.

* * *

But after writing that, I'm realizing if Mocha needs to make fewer assumptions about its global context (perhaps Browserify solves some of these issues for you, but that's another story).  If a global property is not present in *both* the browser and a server context, it should *not* be referenced as a global!

```js
// this is ok in Mocha code
setTimeout(function() {
  // ..
});

// this is not
var cwd = process.cwd();

// likewise neither is this
console.log(location.href);

// but this is OK
console.log(window.location.href);

// and so is this
var cwd = global.process.cwd();
```

Why?  Because users need to be able to mock a browser context on the server, and (as it stands), Mocha wants to run its server code in the browser.  If Mocha doesn't assume anything is global except the variables *which are global everywhere*, the maintainers win because maintenance is easier, and the users win because writing tests is easier.

I mentioned Browserify earlier, because I'm thinking it'd significantly reduce the size of Mocha's library codebase, while solving these problems for us... someone get on that.

cc @dasilvacontin


Cancel is *supposed* to be the last button in the tab order.

E.g. it should generally be:

    <button type="submit" class="confirm right">Save</button>
    <button type="button" class="cancel">Cancel</button>


Format comes from galloc module

https://github.com/suyouxin/android_vendor_marvell_generic/blob/android-5.0.0-xo4/marvell-gralloc/gc_gralloc_fb.cpp#L62


This seem unused method anymore while we modify server msg and pass to the controller signal through updateServerMsg() methos


Not sure if this is the proper way to fix given my lack of experience with Swift, but it did resolve the crashes.


you can return here well known properties like:
ORDER_KEY >= 0 etc


@idf 蛋总关注了好紧张……


piggy vivo's back


yup no longer global variable anymore guys!!!


Added Descent Maximum warning
Handle multiple soundtracks
Move retro tracker command from .ini to shortcuts.


I think it should always have a source culture, but I wasn't 100% sure on it


This is the boolean that  decide the thread run method would be activate or deactivate if  there is lost connection or connection is closed


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


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1128 of 1129 tests passed)

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


The controller interface was changed to abstract


rax retry tests


we extends the abstract class instead of using implements for interface.


`help`?  Or are you looking for more (usage instructions)?

```
> help
Help for command root.

Sub Commands for root:
exit       Exit from the console.
cls        Clears the display
exec       Command that manages exec.
mv         Moves the player to the specified room.
rinfo      Gets or sets information about the specified room, or the current one if no room is specified.
li         Lists all rooms.
```


String Id is the identify - name of the controller
ex: LoginCont


we set the identity for every controller from the main class


fast-forwarding master to auto = b9965ef4


This is the Netbeans system files to be able to working on Netbeans


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

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


yodaテイクありがとおおおおおおおおおおお。
:ox:


维修费用 是不是 维持费用 比较好？


Needs to pass true.


Needs to pass false.


so one good thing about using promise is that you can avoid the stairs-like indentation.
You can write this as:
```
  upRating: function(req, res, next) {
    Curricula.find({where: { id: req.params.curriculumId }})
    .then(function(curriculum){
      return curriculum.increment('rating');
    })
    .then(function(newcurriculum) {
      return newcurriculum.reload();
    })
    .then(function(newcurriculum) {
      console.log('Successfully increase rating for curriculum', curriculum.id);
      res.json(newcurriculum.rating);
    })
    .error(function(err){
      console.error('Error in finding curriculum to increase rating',err);
    });
  },```


appraoch -> approach


If we allow them to not include a year, then we should default this value above, and if during parsing the year's not present, use the current year.


review, res, desc size increased


Please could you make it so that the API mimics the one here: https://nodejs.org/api/http.html#http_http_request_options_callback

I'm happy to have it take either an object or a string (URL).


任何人，任何机构都可以创建建模和部署相关的商业项目。
P2俱乐部组织相关的总结和交流。


WTF, ok. LCTM...


超过20分钟，会每周调整内容。


how are the models in rest_api/models.py related to users?


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


Put the `(eq (current-buffer) start-buffer)` to the while loop:

```elisp
(and (spacemacs/useless-buffer-p (buffer-name (current-buffer)))
           (not (eq (current-buffer) start-buffer)))
```
That is, while still finds useless buffer and not go back to starting buffer, keep looping. When you go back to starting buffer, it is already there so you don't need to switch again.


People who don't speak English may object - but I cannot recommend a good alternative other than a slug


jki


seems to me that this is pretty much the same as the one for curricula. It is good practice to avoid code duplication as much as possible because if there is a bug or you want to make a change, you'd need to change code in both places. This is more work plus higher chance of forgetting to do one.

In this case, I'd recommend creating something like a `BaseController` that all the other resources inherit from


Yep. They were failing the build.


hpwh_residential (nmerket) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Looks good!


Ios apps using swift


@imdreamrunner starred. 求链接下载。


Should this not be sent in with the logical plan instead of being a field?


We may also want to consider creating some sort of a "global state" object that carries public references to the `Map`, `Cave`, `Player`, `MapRenderer`, etc. that we can pass to any class that needs it. Then we can steer ourselves away from being entangled by static classes and properties.


Update slick with verticalSwiping


Well my first build booted up and shut down on Paul. If you wanna check it out or get with him, here is the link https://www.androidfilehost.com/?fid=95916177934554593 Maybe you can give a little helpful criticism on the build. I did pull the camera from the build until I'm able to bring it up to 5.1.


NP, just send you a few changes from the last section I was missing. We
should be good to go afterwards.

On (Apr-13-15|20:46), J W Debelius wrote:
> Updates the notebook with Yoshiki's comments
>
> Thank you @ElDeveloper!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/JWDebelius/American-Gut/commit/74c278fc281225c8f86431d1bb6ef53ec8896eeb


维护费用 会不会比较好


Is there a reason why main thread is explicitly invoked here?


Awesome, I'm still working on putting everything into a database.  You seem to still be experiencing the same issue that loading the list takes forever.  I think once that database is implemented, it will speed most things up.  Thanks Mike


@xsacha so Travis suggested this to fix our memory usage problems (sometimes we were/are getting half as much free RAM), but now Symbian is dying.  It says sbs doesn't support libc2_19... maybe we just need to install some compat libs?

-[Unknown]


No, we don't support hash-partitioned semijoins


 Nope.  Leave it even for new installs, it's useful and used frequently for
things unrelated to billing (which is why your change is a good one).

Tony McCormick, CTO
www.mi-squared.com
Support: 866-735-0897, Direct: 713-574-6709
My Calendar:  http://bit.ly/XznvDo
 ... we have to slow down a bit to be able to speed up a lot... (me)

On Mon, Apr 13, 2015 at 10:10 PM, Brady Miller <notifications@github.com>
wrote:

> Agree ok to remove generic stuff for new installs, but to not touch for
> upgrades.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aethelwulffe/SunCoast_OpenEMR/commit/045bdf3f4b53b5a4196412da2d629caa89cf4395#commitcomment-10705282>
> .
>

--
Please be aware that e-mail communication can be intercepted in
transmission or misdirected. Please consider communicating any sensitive
information by telephone.  The information contained in this message may be
privileged and confidential. If you are NOT the intended recipient, please
notify the sender immediately with a copy to hipaa-security@mrsb-ltd.com and
destroy this message.


No. The table writer produces a stream with some opaque info needed by the table commiter. None of the written columns is propagated through it, so the only properties that can be derived are general partitioning and placement (coordinator-only)


Use Naming Conventions such as SM_(NAME) for Static Meshes


I'm definitely okay with using this as a template since we know it will get us full credit!


Adding the 'url' property doesn't break the interface for mimicking `http.request()`; it simply allows a full URL to be provided (as if for cURL) without needing to `url.parse()` it first.

Example of usage:
```javascript
    var request = httpSync.request({
        url : config.url,
        headers : config.headers,
        rejectUnauthorized : !config.ignoreSSLErrors
    });
    var response = request.end();
    result = response.body.toString();
```

as opposed to
```javascript
    var params = url.parse(config.url);
    params.headers = config.headers;
    params.rejectUnauthorized = !config.ignoreSSLErrors;
    var request = httpSync.request(params);
    var response = request.end();
    result = response.body.toString();
```

Otherwise, can you clarify what you are objecting to?
Thank you


Hi Andy,

I think this log line should be moved out scope since we can have multi logs since while ... loop.
Regards,
Hoang


Sure - you got a point.
Please could you
[1] Add yourself to the list of contributors in package.json, and
[2] Bump up the last component of the version number so that I can release to npm after merge.

Thx for the PR!


div.text-center でいけますよ


This is better packaging than having this directly in the html


I need to read up on this more to understand what this should be used for. It seems an overall better way to manage importing and shuffling static resources.


Apparently Chrome still requires it on createAnswer :(


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1129 of 1129 tests passed)

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


dijajd


filename should rename too


It has not been implemented yet.


Hey Kev, according to this blog post (http://goo.gl/BVS4Q3), there is a way to stop automatic reboot on update. :)


oooooooooooohhhhhhhhhhhhhhh.
angularjs, you'r a warrior !


Hi Derek! Can you please explain me with the second comparation of the string with itself as lowercase? Thanks!
I completely understand the first one, it was my first attempt to solve it, but I can not understand the second comparation.


You could avoid allocating the new buf and the copy at the end with:

_, err := rand(hw[2:])

at line 133.


hpwh_residential (nmerket) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Yes! You're right!
However it is not a showstopper for 1.58.0: it will be fixed in next release (i.e. 1.59.0).


ljkl


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2107 of 2197 tests passed)

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


I am fine either way, just get ready for the new layout_options argument that lets you specify more than one table...


What about the char_pet table? This will not conflict and is not redundant?


それはtextでは


The style.css doesn't load this way. What about
`wp_enqueue_style( 'penguin-style', get_template_directory_uri() . '/style' . $suffix . '.css', false, $penguin['Version'] );`


ok


can please add some logic explanation ?


char_pet is more permanent data (pet names, attachments, etc) than what
this is dealing with

On Mon, Apr 13, 2015 at 10:37 PM, helixhamin <notifications@github.com>
wrote:

> What about the char_pet table? This will not conflict and is not redundant?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkstarProject/darkstar/commit/7969159a201db61e3ba80cdd598bc498f7f467eb#commitcomment-10705953>
> .
>


@memmett, please keep the two trailing whitespaces here. It's the Markdown way of saying "start a new line but not a new paragraph"


checkout


Hi @clebergnu, please always use `except Exception`, otherwise you're catching `SyntaxError` and other base `BaseException` exceptions.


I ended up removing the last check for min range being greater than ground clearance - 10, as blades of grass were able to cause it to fail


GHX_Performance_Enhancements (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Do you mean something like this?

```java
Expression predicate = node.getCurrentConstraint().orElse(BooleanLiteral.TRUE_LITERAL);

if (node.getPredicate().isPresent()) {
    predicate = combineConjuncts(predicate,
                    DomainTranslator.toPredicate(
                            spanTupleDomain(node.getPredicate().get()),
                            ImmutableBiMap.copyOf(node.getAssignments()).inverse(),
                            symbolTypes));
}

return pullExpressionThroughSymbols(predicate, ...);
```


oh it's merged, let me send a patch for that...


LGTM


ah, yeah.. a left-over from some refactoring


Det er nu en af de bedste årgange.


ugh, merge conflict? just noticed.


This wasn't ideal but we were saving QubitJob twice before anyway. Once after the initial runJob, and again after the QubitJob saved previously so we could use its 'id' as object_id in the job note. I figure it's just better to save the QubitNote only on the fly here.


yup


Hey sir, it seems you didn't handle the case the close reason is sent from server side?


This works because the intersection is being done by visitFilter, and the only place where currentConstraint is set is in AddExchanges. But, I agree: it's weird and confusing. I'll fix it.


\o/


I told you not to write query in views :)


i meant this comment primarily in the case where there isn't a filter directly above the tablescan (e.g. the filter was entirely absorbed by the tablescan)


GHX_Performance_Enhancements (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 tests passed)

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


see latest


Doctest is failing in Travis build.


Could you change the port variables to ingressPort and egressPort?


asda


I had to release the version again (just a common glitch with the old mvn-release plugin version). Everything should be fine now


テキストを修正したのですね


I split most patches  https://github.com/jshint/jshint/compare/master...XhmikosR:various?w=1

Now, if someone could merge https://github.com/jshint/jshint/pull/2087 beforehand, that would be the best since there will be conflicts.


@mmatts these changes look good. I'm assuming the GHLEBase issues will be addressed separately. A few notes:
* Having isEven as a 1-line inline-able function call would also have been fine and just as fast with inlining enabled. By "inline" I didn't mean that you had to manually inline the operation.
* The reducing in trig calls is great. I would move to declare-and-initialize usage as in the line comment I added to both reduce code lines and allow semantically const values to be declared const and as close to their first use as possible.


I think the intent was as an example but I definitely see the potential for confusion. I will update it to `$HOME`.


Possibly, but I'd probably do that as a separate change later. There might be side-effects due to implied ordering from the outputsymbols list vs the non-deterministic ordering of the assignments map keys.


恩，感觉维持、维护费用比较好。维修的话，有歧义。

维持费用，感觉更符合这类游戏一贯的翻译。
维护费用，感觉比较符合一般人的想法。

这个也是一个基础的词，要不要有个全文统一的翻译方法？


A repository for files for sharing for a collaborative development project


This change breaks the color of the track head for the current track and selected state of the track head. Track heads can be selected in order to attach filters to the track. It should only be blue (or green for audio track) when the track head is selected. It should be yellow when it is the current track.


Typo? (-ran)


This version is wrong. We need to delete it from the local maven repo on bamboo...


This kind of usage `Real64 const cosTheta( std::cos( theta ) );` has the advantages of reducing lines of code and marking them const.

While we are at it using `Real64 const coilRadius( 0.5 * coilDiameter );` in these computations could be faster depending on how smart a compiler's optimizer is (or isn't).


Ahh, I see what's going on. This is half commands/half example output. The `/home/todd` is intended to come from the `make install`.


TBH the name TEST_STATIC sounds like it should be static if testing. how about just 'STATIC' ?


A lot of unnecessary metadata in this google url
Also I would either remove the commented lines or put a comment at top describing their purpose


I do not like the pulsating animation because it is too distracting. I agree that we need some clear way to show selected blanks.


What's the reason to set `LANG` to `null` here if it's overridden during the nosetests anyway?


where exactely is urllib2 used in this file?


what does the leading _ mean?


GHX_Performance_Enhancements (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


The FolderPath property is already static and have a setter, which means as long as I call the 'AppDataContractSerializer.FolderPath = "asa";' before Start(), then I can set the path. The FolderPath property creates a default path if it isnt set, but uses one if its not null.


if DCM.c.z < 0.9 dont do check


nice


@robertmesserle there seems to still be an issue when using Safari. Now when resizing it resizes correctly and then a few milliseconds later the entire md-tabs shrinks to 0 height and then back to theocracy size again.


Based on conventions of desktop software, I expect the shift modifier to expand the selection, not move it. Wait, I recall seeing other commits handling multi-selection, but currently I do not see how to make a multi-selection in this branch. How about Ctrl modifier for moving selection and shift for expanding selection (when implemented)?


Looking good :+1:


`var i = 0; results.length; i++` would be better?


Probably would be a better idea to wrap this data into
`$(document).ready(function() {...});`

Reference : http://learn.jquery.com/using-jquery-core/document-ready/


能讲讲有什么区别么


It supports it, but does not expose a way to make a multiselection in the UI yet. I have had the same thoughts, but is expanding the selection a common action in a video editor? Regardless, I'll move it over to the Ctrl modifier since it's available anyways


yes ,it's just a helloworld


Hmm, I do not particularly like overloading the Esc key like this. I like that it toggles the player. What if I want to do that (by keyboard) and not clear selection? Still, we need something for this. Maybe Shift or Alt modifier?


Yeah, it's an option - I was mostly just inflicting more pain on myself ;)
Systems should be able to recover from machine crashes, having the reboot
in there made me aware that the zk peering had issues. I've dumped that
approach for now because of that.

On Tuesday, 14 April 2015, Victor Wibisono <notifications@github.com> wrote:

> Hey Kev, according to this blog post (http://goo.gl/BVS4Q3), there is a
> way to stop automatic reboot on update. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/silarsis/paas/commit/6be73426c9c00f7355f11d6abb6b5f0e251c08f8#commitcomment-10705887>
> .
>

--
Sent from Gmail Mobile


I'll add a TODO


```c++
foreach (const QString &key, torrents.keys())
```


Good work! I will merge it when the API server is up.


We could for a Ctrl-XXX shortcut, something like Ctrl-D for deselect. What do other editors do?


感谢你所做的一切。


I can't help but comment that my process_printing_options(function) was meant to deal with printing options.

It is also beneficial to manipulate LetterBlocks rather than raw strings as LetterBlocks could be made aware of individual letters (LetterBlock class could be modified to keep an array widths for each letter, just add some code to the __add__ method): then you won't see half-letters on the screen (if that's desirable) or you could substitue letters with dots.

Also padding on the right is redundant functionality as it can be achieved by "padding: {left: 0}" (perhaps I should've made it explicit).


@MarioE tells you that your style is wrong and to fix it, I tell you to do it and apparently I am Hitler.  Get over yourself, you are killing the project.


Understood :)


If we change the Shift+Cursor for moving selection to instead use Ctrl, we should also change this to use Ctrl modifier.


Will try to do that in the future. Though, it is your job to check if your commit passes jenkins. :)


這版 commit 的主要目的是留一份原始版的 box2dweb-arrow.f 下來。因為接下來要亂改它，改壞了可以隨時 checkout 回來。 嘿嘿！


Hi @paulpilone,

I like your idea to make animation settings configurable, let's merge this additional feature to initial repo.
I'm only worrying about these parameters normalization (just not to get too high value or zero, ofc it related to user of this control, but better to normalize it ourselves instead of getting some unexpected behaviour).

I also think we can go on with it without init methods redefinition (I avoided it because it will be less lines of code to check default value rather then initialize it in all init methods), what do you think about it?

Also I had an idea to extend setProgress:animated: method to setProgress:animated:animationSpeed: or something like that (it will be additional method) - just to give user ability to play animation with speed they want, and do not change animation properties to smthg and back again each time. However, this method have it own pros and cons, so don't know yet is it good idea or not.

What do you think?

Kind Regards,
Andrew


Tested with P = 2, I = 0.5.


I definitely like that we have something to show selection outside the view. This pulsating animation is actually more subtle than pulsating the potentially large block of a selected clip. However, there is a vertical alignment bug when the tracks area is taller than the scroll area and is not at top (scrolled down some or to the bottom).


Hi,
In fact the `GeminiLocale:NewLocale` line is copied from the enEN.lua file, I did not set the true/false values on purpose :) I did not really know what was this arguments, I'll make the modification.


I would think so, even though it likely won't be feasible to automate testing for all services at this point. Have you used `mocha`? Any suggestions for another framework that might be better suited to our needs?


why 1? as long as player is none, play speed is undefined.


I'm sorry you are all caught in this situation.
Perhaps it would be more constructive to leave comments on the breaking change though:
https://github.com/MinecraftForge/MinecraftForge/commit/3fc40b955acb088d28354d348b283bce054ddef1

Forestry is dependent on Forge, and for the most part they do not make these kinds of breaking changes. When they do happen, everyone must update and we all can move on. Greg is in a different position, he can orchestrate things to fit on his terms. I want to maximize compatibility with other mods and right now that means keeping up with Forge.


Reverted in 0276f3f3fb609aabf8d9ba7aea10c80ec979989d. Sorry about that!


Done: https://github.com/olbat/RaidCore/commit/e02e2963bc1cdbfdba98b2fa084a27babfd2bbbc :)


RestartServers and updateConfigurAndRestart methods have been declared as class methods. Since they belongs to this class and will be using only in this class , you can declare them as instance methods.
If you agree with me please make the changes and resubmit the patch.


I'm not particularly happy with this commit, to be honest. I would like to retain the ability to store an entire CSV file in memory, rather than requiring it to be mapped manually in some fashion. The calling convention you came up with for the header and body separation also seems quite awkward to me.
I'll think of a solution of my own.


And you are more than welcome to use Convert.ToInt32, you just need to read the docs more carefully and take into account that it throws exceptions if the string is not an int.

https://msdn.microsoft.com/en-us/library/sf1aw27b%28v=vs.110%29.aspx


update


update readme


I don't know what other editors do. Often, I have to research that. Ctrl+D is fine with me.


git test git test git test git test git test git test git test git test


@djazayeri this happens only on the confirmation page because it is when the end of the tab order is reached.
@wluyima are you suggesting that i should just change the button locations such that confirm is on the left while cancel is on the right?


You just need to recompile SBS for each libc update


lgtm


My mistake.

# My original meaning is likely to follow this guide. https://github.com/golang/go/wiki/CodeReviewComments#variable-names
# I think your points is make senses. I changed them back in https://github.com/xs-nanjing/go-xenserver-client/commit/9f8a20c98c10225b56f3cf929eca75804dec8a9d


Done


Interesting choice to go with Parse and catching an exception, instead of TryParse that does the hard work for you, resulting in less lines.


I would evaluate only checklistValue, value I will use it as it is. This way you have the opportunity to pass an actual value, without enclosing it in string delimiters.


Are there crash issues? Should the 1s/3s be delayed more? I think it could just be called on opening the Settings page. Not sure how much work that is.


je ne sais pas pourquoi il me dit que j'ai modifié la première ligne ...


@mogoodrich I don't know where this is coming from. I tried deleting this version from /root/.m2/repository/org/openmrs/module/allergyapi* on bamboo (and also from mavenrepo.openmrs.org) but it seems like it got recreated somehow in our bamboo...


apparently the windows and centos 5 builders aren't too happy with this


With labels I mean the text that is next to each field. For example I can not use a JSON string like this:
{"ticket":"1","Date":"","Status":""}
Because I get the same GUI if I use the default object, but with less fields.


`jekyll server` to `jekyll serve -b ''`


bon ce patch n'est pas utile puisque c'est le même que mon 6, j'ai juste voulais refaire pour qu'il n'y ai pas la modification de la première ligne ... apparemment il veut quand même la modifier.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Living Quarters 生活区会不会更一点？舍区比较简洁，但是似乎平时不太用这个词。


does this work?


This might not work as expected - as getNextCoord skips the center - but resetCoord doesn't.
A better approach would be to just save and restore the scanPos in your harvest loop.


Failed.


remove sysout


Why is ed25519 different from ecdsa in not requiring an `#include "publickey/ed25519module.hpp" ` or similar?


change readFile name


Hi, I fixed the XSS vulnerability problem.
I was surprised that jQuery has no escapleHTML function.
I added a escapleHTML function.
http://nrbm647.blog.fc2.com/blog-entry-40.html


... and an `init_ed25519(module)`?


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


change second value to 48000


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2176 of 2197 tests passed)

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


### If we highlight Cortex-A57

* Used in premium devices
* Superior performance
* Each category will have one or two devices

### If we keep as Cortex-A53

* It's not Cortex-A53 I want to highlight, it's simply 'Cortex'
* Qcom's next chips will have Kyro cores, which gives better differentiation in the future (like we have Cortex-A7 and Krait-400)
* Unlikely to be many more Cortex-A57 devices (Z4, maybe a compact?)
* There may be a couple more Cortex-A53 devices, but Sony seems to prefer MTK these days
* we don't need to keep premium devices separate, see Krait-200 devices

The category names probably suck to most other people anyway, but I chose them according to how *we* group them and develop them. Essentially CPU core is the *only* constant in each family that gives a logical name. It's the only way to keep the number of categories low and with more than one in each category, plus not to have to worry about having some devices supported but not others within a category.

'Snapdragon' doesn't work as it's constantly redefined, year of release mixes up all the chip types, MSMXXXX is too narrow and leads to too many categories. [*]

However my logic is holding well so far. All Krait-200 devices were moved to legacy at the same time, even though there were many different chips used in this family (four I believe). Whenever it happens all Krait-400 devices will move at the same time, and all Cortex-A7 devices will move at the same time.

I think it's highly unlikely we will move Cortex-A53 devices to legacy at a different time than Cortex-A57 devices.

The Cortex-A53/A57 chips are similar enough to keep them grouped, and it's sure that Kyro devices will have enough differences from our dev PoV to keep them separate from these Cortex devices.

[*] Possibly Sony's family would work for us, but we would have to retroactively (re)define *all* of the legacy devices.

The biggest issue with that is splitting up the Krait-200 categories. We used fusion3 instead of lagan, and some of the Gen2 Krait-200 devices will either be grouped with Gen1 Krait-200 devices (under blue) or made into their own categories under vishkan (huashan clearly had this but shares a kernel with blue; taoshan and nicki were independent).

What's more we (read: Alin) invented the yukon category ourselves, only tianchi clearly had this, the others were independent.

Moving to the obsolete devices, it becomes even worse.


install ncurse and use system("clear");


method should name-> convertStringToFloatArray(String textLine);


Taken from before ApplicationPlayer split https://github.com/xbmc/xbmc/commit/e4fbb96b3cdf63dcecb56cb48cbc3af1c264214c#diff-63246197ac0f0ac89358c1155ad21d5bL392
IsPlaying() will return true once a player exists but playspeed is only set once the stream is open

The endless loop could happen at https://github.com/xbmc/xbmc/blob/master/xbmc/cores/dvdplayer/DVDPlayer.cpp#L661 if the stream was not opened in time
I could repeatedly reproduce the endless loop on a Raspberry Pi B+ playing an internet stream with bRestart=true as first media (once a media had been played with bRestart=false it would not get stuck in an endless loop since the playspeed would have been initialized to 1 before opening the player)


Do not put the confirm button on the left. It should be semantically before
the cancel button within the HTML, and thus before it in the tab order, but
it should be floated right.

On Tue, Apr 14, 2015 at 1:45 AM, dkayiwa <notifications@github.com> wrote:

> @djazayeri <https://github.com/djazayeri> this happens only on the
> confirmation page because it is when the end of the tab order is reached.
> @wluyima <https://github.com/wluyima> are you suggesting that i should
> just change the button locations such that confirm is on the left while
> cancel is on the right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-uicommons/commit/97618792e7adc5a90ef1a3471e403c451148d1fe#commitcomment-10706415>
> .
>


good point. I'll have to think about the reset for a while. I was planing to change the "main code" structure a bit anyway, maybe I'll do that without waiting for Agri's API...


I couldnt make it work when I tried that. Heres my code:

```elisp
(defun spacemacs/next-useful-buffer ()
  "Switch to the next buffer and avoid special buffers."
  (interactive)
  (next-buffer)
  (let ((start-buffer (current-buffer)))
    (while (and (spacemacs/useless-buffer-p (buffer-name (current-buffer)))
                (not (eq (current-buffer) start-buffer)))
      (next-buffer))))
```

It just cycles through normally.


pointsList.get(numberOfLine)  it's one string -> should be pass ass parameter in method


Yes, that definitely reads better.


Since `seqnum_cmp()` cannot return anything else than -1, 0 or +1, I think `else` is enough.


method should has name e.g -> convertFloatArrayToPoint


Why `insecurerandstr`?

I know this line isn't touched by the patch but it jumped out at me. Certainly ECDSA nonces do need to be secure, but I was under the impression that Ed25519 signatures were deterministic, so what is the seed used for?


This method shoudl be in PointFactory


Bad name of class


Most recent commit is installing successfully with original eclipse settings, no issue with gc/memory


remove initzializaion


Otherwise +1.


Since you've already compared for "less than", this could be a simple `==` I suppose.


... but ideally split this out into a separate pull request if that isn't going to slow things down.


Sorry its taking me a little longer to get that list done. I had a few personal things come up and will get back to finishing the list shortly.


return listOfLines;


remove return section


+1, but I did not smoke-test that any of the 3 build options actually work.


Nice additions.


+1. I did not smoke-test that benchmarking still works.


i have a doubt.

is there any chance to make any linux compatible mpc version or branch?.

there are good multimedia software for linux, but anything compare to MPC. :/


saw approval from shino
at https://github.com/basho/riak_cs/commit/bd6f4a212edc94587446da4c8e801639cb185fc4


merging basho/riak_cs/feature/quota-hook = bd6f4a21 into borshop-integration-1118-feature/quota-hook


basho/riak_cs/feature/quota-hook = bd6f4a21 merged ok, testing candidate = bf67549f


第一次提交