@tvsonar seems that case ``if ((f < ...) || (f > ...))`` isn't covered nor by test, nor by implementation.


inconsistent use of braces


`One thing to notice here is that in this post we won't resort to any magic here.` ->

`Important thing to notice is that we won't resort to any kind of magic here.`

`We won't do macros, bytecode manipulation and any other "magic techniques".` ->

`We won't do macros, bytecode manipulation or any other "magic technique".`


It is missing indeed, fixing both tests


why double equals here instead of triple?


Should I have not pushed this? I'm confused as to how I can keep my .gitignore separate from the upstream mater one.


I think it would be much clearer if you returned from there... because it is one specific use case... then some newlines before handling data


I apologize for not just making a PR, but found another one here:
https://github.com/GoodBoyDigital/pixi.js/blob/dev/src/core/renderers/webgl/managers/WebGLManager.js#L36


@englercj


but return is not needed in an if else


Probalby a mistake indeed.


Where is progressbar2 used? I only see blessings below.


Merging my branches


States are specific to the checker as of current implementation.


Please also consider the following cases:

* user assigns an ElementTree
* user assigns an etobj.Element


We may also want a __delattr__().


This change is proposed


Why are we using two different specs?


why not

if (data instanceof Blob)
  processBlob(data)
else if (data instanceof URL)
  console.log("....)
else if (typeof data == 'string')
  console.log("...")
else if (typeof data == 'object')
  processJson(data)


The new design is live in most of the popular Stack Exchange sites, I believe. Do we need this check? On Super User & Web Apps, the icon is not aligned, and I think removing this check should correct them


yes


@ghedamat No worries! :)


onAdapterAvailable(NfcAdapter adapter) {
   adapter.onmessage =


too complicated, retry on popen is hacky.
maybe calc w/ shell the core count and pass it as env
or open a /proc/ directly from C code


Zou ik in git doen, niet in installer.
Zit er dus al in: https://github.com/sumocoders/fork-slideshows/tree/master/src/Frontend/Files/Slideshows


@tvsonar seems that case (f <= ...) && (f >= ...) (parentheses) isn't covered nor by test, nor by implementation.


indenting


@tvsonar and what about ``f <= ... && b && f >= ...`` ? ;)


Okay ich überlass Dir die Bezeichnung, what works better for you.


die zit goed, lijn 27 ontbreekt gewoon in de diff waardoor het zo lijkt


Hmm push failed yesterday...

![Giphy](http://media0.giphy.com/media/10VJ2YDMoNEBl6/giphy.gif)


adapter.onmessage


Waarom is dit weg? Is handig als er dimensions moeten zijn voor images...


Those lines are not meant to test CanICA. They're simply making sure the test data are what they should be (more blue than red). They're testing the test data for sanity. They're meta-tests if you like. At worst, those lines don't do any harm.


ue event or ev instead of evt (quite unusual abbriviation)


Wie bereits im IRC besprochen, bitte mal schauen ob das LQFP-48 package aus lib/ref-packages.lbr oder ref-packages-longpad.lbr von Eagle selbst brauchbar ist.


Ack


@pablomoreno61 Here https://github.com/sergeyklay/phalcon-component-example/blob/mail-example/tests/MailTest.php


乱码


Yes, and will be checked at line 203.


ik weet dat het daar zit, maar als je de zip van github download en die via de backend upload dan werkt dat niet, die files kwamen er niet te staan, op deze manier kan je de module makkelijker toevoegen want fork handelt het allemaal af en zo moet je niets kopiëren


Few points:
1. This is going to be used in View only then it should be part of respective helper.
2. Each time this method called, it will perform complex logic and prepare country list. It will take cpu time and memory. Use [memoization] (http://www.justinweiss.com/blog/2014/07/28/4-simple-memoization-patterns-in-ruby-and-one-gem) to cache the value


in which case country would be `nil`?


ik zou dat er eerder aan toevoegen op het moment dat het nodig is, want nu zit het er in zonder dat het iets doet, want daar word in de frontend niets mee gedaan, kan dus voor verwarring zorgen


User Rails 4 style validators


We -> I


@asergi Can you pull the ASCII char replacement ?


Use first person here, you are doing those things.


Is in backend dat er iets wordt mee gedaan, als het ingevuld is... dimension restrictions


an matrix -> a matrix

This clearly -> It clearly


source http://stackoverflow.com/a/26889118


Я бы заменил на

```
Procedure.s rtip(txt$)
  Dim words.s(CountString(txt$," "))
  For i = 0 To ArraySize(words())
    words(i) = LCase(StringField(txt$,i+1," "))
  Next
  RandomizeArray(words())
  For i = 0 To ArraySize(words())
    If i = 0
      words(i) = UCase(Mid(words(i),0,1))+Mid(words(i),2)
    EndIf
    output.s + words(i) + " "
  Next
  AddGadgetItem(3,-1,output)
EndProcedure
```

потому что LinkedList тут использовать излишне
но это не критично


on a spreadsheet -> at a spreadsheet


Thanks for the hint. Makes perfect sense to not have them in the project .gitignore


is there any chance we could share this dispatcher with the DB pool? Because ultimately they'll all be blocked on the same connections.


Related to #10


Fix comment, since now it is a ref not a tag.


wieder alles ganz?


There are elegant ways to show alert messages
Check http://stackoverflow.com/a/17181766/517483 and https://gist.github.com/suryart/7418454#comment-1327611


added changelog see https://github.com/sulu-io/sulu/commit/45cfd745db57137dbec4f1dfb5052cbb1115a3e6


Hi Boris,

Thanks for the patch. Just a question. This patch will speed-up transfers when deploying a VM in the front-end, right? i.e. the image and system datastore living in the same host, right?


an class -> a class
and instance -> an instance
pass his value and the argument into it -> pass its value and the argument into the array
all othe -> all other
in chain -> in the chain


ah ok :) my bad, ik had in de frontend gezocht, gefixed in 26a1e9b


Related to issue # Pogo refactoring


You have added migrations but why you haven't committed `schema.rb`?


Wow, a 0 line fix!

![desmond-choo-not-bad](https://cloud.githubusercontent.com/assets/7300760/7469298/18b157a2-f312-11e4-90c8-18e964630cfd.jpg)


@plymouthmarina in firefox the following error is given: "EncodingError: The given encoding is not supported." aka audio format given is not supported by Firefox. According to the specs MP3 should be supported: http://www.w3schools.com/htmL/html5_audio.asp. So either we need to specify the format somewhere in the code or the mp3 uses an unsupported encoding (mp3 can be a bit of a funny format, so totally plausible)

Don't think we care about IE support so we can try to give it a .wav file instead and see if that works. Just need to find a converter somewhere...


It looks like this was a commit added mid last year - I assume that @pilz got a notification during the repository move. My conclusion therefore is that we've depended on compass for quite some time now.


I would propose to have a polymorphic method on Universe .isMate() normal Universe returns false, you return true.


No need to `as Vector3` because the class short name is already `Vector3`.


No lo sé, algo como naranja o así. Estaba mirando colores [aquí](http://en.wikibooks.org/wiki/LaTeX/Colors#Predefined_colors) y me pareció muy curioso que eso fuese un color, así que lo puse xD


I would advice not to mix the syntax -> I would advise against mixing the syntax
That both an requirements -> Those are a requirements
one twist you probably noticed is -> one twist you probably noticed are


Thanks mate!


Wat


1. Undefined `$blockID`.
2. A block ID would never be larger than 255.
3. It is a good practice to use `===` instead of `==` unless you have special reasons.


in ogni caso se fosse lo commento e lo faccio sparire, nel dubbio lasciamolo, non costa niente


Argument for `getTile()` missing.


What is the meaning of doing this if you don't use it at all?


Hey yourself there is a typo here


@corinnekrych would it be possible to include this code in the actual SDK ? Because if every time this needs to be done for an app, it means a lot of messy code, and likely nobody will be using that


I don't think you need the inheritance here. This is all static stuff anyway. And then you can still make MateClasses and Classes final.


Samma två gånger?


we already use jdom for this kind of stuff


Wow, nice catch. This got me good confused :)


no hard coded versions please


OOM check


What? `->p->`?


Tää ei toimi jos oot alemma sivulla, pitää muistaa korjata myöhemmin.


Ты опять засовываешь функционал в For. Мы же убедились, что это не комильфо. Лучше один раз пройтись по строке LCase'ом, чем утюжить им каждое i, при этом сэкономив строку.


should be written to ./target


And it seems like it should be `>=` not `===`


Sisi lascialo, guarda su fb.

Ora non committare più,  sistemo una cosa coi charset e poi lo lasciamo
così se no magari salta qualche bug all'ultimo
Il 05/Mag/2015 10:35, "ghia16" <notifications@github.com> ha scritto:

> in ogni caso se fosse lo commento e lo faccio sparire, nel dubbio
> lasciamolo, non costa niente
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/igorravasi/Memo/commit/33b71cee22aef9b338938102e5bd8c4c53adbef5#commitcomment-11040249>
> .
>


It should be `broadcastMessage()`. For the `broadcast` function you need to provide a permission node, which shouldn't be what you wanto.


Hi,

Can you change your pull request to something like this?:

if (g_machine_type == C16) {
  buffer_magic[0] = TAP_MAGIC_C16;
  g_fat_buffer[13] = C16;
} else {
  buffer_magic[0] = TAP_MAGIC_C64;
  g_fat_buffer[13] = C64;
}

Then we don't set the buffer twice and also use the enum instead of a magic value. We should probably change this to a switch statement and also take into consideration the video timing (NTSC / PAL), but that can wait.


`elseif()`? No, just `else`.


Do you know what `isset()` means?


There is only one home?


No, I don't want that. The idea is to use the submodule for version control and not to be always deploying HEAD.


Again, what is the meaning of getting `$text`?


`?>` is not needed.


1. Learn PHP.
1. Indent your code properly.


oups github folding had me, my bad :)


As discussed on the list, I'd do that differently. Implementing the NodeVisitor visit method by calling a custom polymorphic method on SOMNode. There you can then do what every you feel like, but, normally, you'll just need to implement the wrapping in the leaf nodes.

But thinking about it, there might already be something for that in Truffle. You could check the NodeUtil and related classes in the frame work. I think, the instrumentation API might also need to wrap each node, so, I would guess, in their examples, you might find something already.


I can't see it


True. Let's upgrade the cooldown once we trigger spells only if castable.


Also, as you can see, travis fails because you forgot to add the Apache Licence header to your new files. Have a look at the other examples and include that initial commment :). We've all been there, don't worry!

To make sure you have the correct coding style and that you did not  break any existing functionalities, in your flink project:
- do a cd flink-staging/flink-gelly/
- mvn verify

When you get BUILD SUCCESS, you're good to go :)


Should return int (negative error code). Return new object via extra param json_variant **ret.


Ho fatto l'ultimo commit e controllato che funzioni tutto senza bug, tutte le stringhe a posto e i layout pure, siamo tranquilli


hypot(x,y) == sqrt(x*x + y*y); // not x*x + y*y


Nice one. Thanks!

On 5 May 2015 at 08:48, Theo Armour <notifications@github.com> wrote:

> 20 pounds sent to coderdojo.
>
>
>
> On Mon, May 4, 2015 at 4:32 AM, Oliver Moran <notifications@github.com>
> wrote:
>
> > Theo, thanks for the offer. I'm OK but if you feel like a donation, you
> > can give to one of these:
> >
> > - https://coderdojo.org/donate/
> > - http://www.chernobyl-international.com/donate
> >
> > Or a cause of choice in your own area. All the best.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/theo-armour/jimp/commit/8569b3176b479cbd2b780f239434c60c9a24b8a5#commitcomment-11023804
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theo-armour/jimp/commit/8569b3176b479cbd2b780f239434c60c9a24b8a5#commitcomment-11039668>
> .
>


It's false up until the first AUD unit (i.e. it's use is to locate the first sample). After that you're right, setting it to false and immediately back to true again doesn't achieve anything. We should probably rename it to findingFirstSample or something, and remove the line of code that sets it back to false. I'll do that.


I think we both agree that parsing Ruby code ourselves is not the best of ideas. Especially because I sometimes make pre-releases that do not follow that naming convention (e.g. 0.5.8.pre1intenthook1)


I like this change


Remove comment.


Same here


I don't think we have to change this, if we just require a relative file instead.


verloaded -> overloaded


I like this commit.


@leafo thank you. I removed `recalc_every ` from source code and hope to find  time to update `sticky kit`


C'est vrai que c'est la seule chose que j'aime pas dans PHP-cs-fixer. Mais la prochaine fois que tu vas le lancer il va remplacer par ça. On fera ça à la fin ;)


Remove trailing whitespace


Ok, will reword


I'm missing something... why wouldn't a specialization of `get_pointer` allow support for `offset_ptr`?


It seems you didn't really resolve the conflict here (as well as in `seeds.rb` below).

You need to read messages printed to the terminal carefully. And when there is a conflict never ever do `git add .` Instead, fix each file manually and add them one by one.

Please read the resource on merging conflicts: https://www.bloc.io/resources/git-continued#conflicts and get back to me with any questions.


?
 On May 5, 2015 11:19 AM, "firasnajjar" <notifications@github.com> wrote:

> Why are we using two different specs?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SAP/chevrotain/commit/28dddf32520583c7c5ed60c8f82cf285d857faa8#commitcomment-11040024>
> .
>


Ok fine, I will remove this then and update the Ruby lib, so implicit attach then on `get`, `enter`, `update` yes?


Snazzy comment!


There is a Tuple2ToVertexMap that does this in /graph/utils


ArraySize() очень быстрый. Как раз из-за того, что это массив фиксированного размера.
Проверяется просто:
```
Dim someArray.b(1000000)

start.i = ElapsedMilliseconds()
For i = 0 To ArraySize(someArray())
  someArray(i) = 5
Next
Debug ElapsedMilliseconds() - start

FreeArray(someArray())
Dim someArray.b(1000000)

start.i = ElapsedMilliseconds()
For i = 0 To 1000000
  someArray(i) = 5
Next
Debug ElapsedMilliseconds() - start
```

Строку не сэкономить, потому что все равно придется добавлять слова в массив. Ты наоборот предлагаешь еще одну строку добавить.


@ThibaudDauce Il manque la méthode `getCasesControleesAttribute` de la classe `Joueur` aussi. Peux-tu voir ça ?


And a Tuple3ToEdgeMap at the same location :)


Seems weird. It is string, so use strlen() instead.


I think we have converged to the sane solution:
```
    def _get(self, available, installed, pattern):
        result = []
        if self.status & self.AVAILABLE:
            result.extend(_by_pattern(pattern, False, available))
        if self.status & self.INSTALLED:
            filtered = _by_pattern(pattern, False, installed)
            found_ids = [item.id for item in result]
            result.extend([item for item in filtered
                           if item.id not in found_ids])
        return result

    def get(self, *patterns):
        res = dnf.util.Bunch()
        res.environments = []
        res.groups = []
        for pat in patterns:
            envs = grps = None
            if self.kinds & self.ENVIRONMENTS:
                available = self.comps.environments
                installed = [CompsAdapter(id, self.prst.environment(id))
                             for id in self.prst.environments]
                envs = self._get(available, installed, pat)
                res.environments.extend(envs)
            if self.kinds & self.GROUPS:
                available = self.comps.groups
                installed = [CompsAdapter(id, self.prst.group(id))
                             for id in self.prst.groups]
                grps = self._get(available, installed, pat)
                res.groups.extend(grps)

            if not envs and not grps:
                msg = _("Group '%s' does not exist.") % ucd(pat)
                raise CompsError(msg)
        return res
```


Is it necessary to suppress unchecked warnings now?


Make sure you delete all the auto-generated comments.


Type and cleanup attribute suffix do not match. This is confusing, people will be grepping for _cleanup_json_variant.


Any recommended settings for falcon?


Be mindful that this will leave the old wrappers laying around if someone is upgrading.


No extra empty line here.


Could you also instantiate this in the gx constructor?
Then one could check for `gx.progressbar is None and not gx.silent` and have an optional dependency on `progressbar` (The currently shipped package on Fedora doesn't work over here, so I need to wait for that bug to get fixed...)
Furthermore `gx.silent` would get honored then.


Just a cosmetic bug :) Maybe we could make this 66.2 factor a private static constant. It would add more expersivity to the formula.


you'd still deploy kilo, just head of that branch, not master, which I guess is what you were talking about.


documentation?


no image alias here?


I have in fact not followed that pattern, see https://github.com/ably/ably-ruby/blob/master/lib/ably/realtime/presence/presence_state_machine.rb#L12-L21.  Instead like `Connection` and `Channel`, there are states for presence that reflect the current client's presence state. I personally prefer my approach as it's more consistent for developers i.e. I am not sure they should think that the current presence state is `UPDATE` simply because the last event sent was an update, in terms of their presence state, they are in fact `ENTERED` on the channel. Can we discuss?


Snazzy comment!


typo in newRespons, I guess you wanted to write newResponse.


Is preHandler and postHandler a better name?


Use `etc_dir_for`


Is this useful?


Use `log_dir_for`


Création du projet


Specify owner


@JKE-be Thank You!


Why a string?


where is global defined? shouldn't you pass it in, using `this` outside of the use strict?


`self->z_rcomp_port` should be `self->z_halrcmd_port`


Granskad


All lines with this meaning = This checks should be changed to this checking or this check.

Line 85 = Get info if the course is public.
Should be = Gets the infor if the course are in public mode.
Line 87 = content i the code example.
Sholud be =  content in the code example.
Line 88 = True means yes the user have the right.
Should be = True means yes, the user have the rights.
Line 94 = If course exists
Should be = If the course exists
Line 99 = Allowed to edit this course (full editor).
Should be = Allow to edit this course (full editor)
Line 118 = This will show a error message if the course id doesnt exist.
Should be = This will show an error message if the course id doesnt exist.
Line 126 = This will show a error
Should be = This will show an error


Could you rename Binlog to something else? It make me feels wired.


@yDelouis the `ItemClickHandler` is not prepared for extending types, and also for the View param. That will be implemented in #1076 and #1285.


`owner` and `group`


This fix the saturation of this value but the fix proposed is very tied to the architecture.
Maybe the real issue is that the TangoType should be changed to match the uint64?
Related to pull request #6


fixed, moved it to the end.


Can you add a `group` on those ? (I do realize you're copying them over, this should probably have been done some time ago)


This method should be removed. Was used for debug.


Hi Ruben,

not just on the front-end. And also the transfer doesn't necessarily have to be faster, it will just decrease the CPU load on the machine(s). Let's look at an example:

(src/tm_mad/ssh/mv)
```bash
$ ssh node1.example.org 'tar -C /var/lib/one/datastores/0 --sparse -cf - 1554' | ssh node2.example.org 'tar -C /var/lib/one/datastores/11 --sparse -xf -'
```
As you can see in the example above, `ssh` is used twice although one of the nodes is definitely locally available. Piping `STDOUT` from one `ssh` connection to another `ssh` connection is CPU-intensive because all the traffic has to go through `ssh` twice (encrypted & decrypted twice, prepared for network transmission twice, etc.).

My modification is doing the following:
```bash
$ tar -C /var/lib/one/datastores/0 --sparse -cf - 1554 | ssh node2.example.org 'tar -C /var/lib/one/datastores/11 --sparse -xf -'
```
or
```bash
$ ssh node1.example.org 'tar -C /var/lib/one/datastores/0 --sparse -cf - 1554' | tar -C /var/lib/one/datastores/11 --sparse -xf -
```
As you can see, `ssh` is used only once because I can identify which machine is local and access the files directly there. Which means there is only one CPU-intensive process running and I can transfer either:

1.) more data __or__
2.) the same amount of data faster __or__
3.) the same amount of data at the same speed with less unnecessary load on the server

1.) and 2.), of course, depend on your current network saturation. If you were limited by CPU performance before then now you can transfer more or faster (or both :-)).

I hope this helps. I'm open to suggestions on how to make this even better.


nearly finish!!!! @JamesXNelson


You're going to want to use `copytruncate` here. We cannot tell the Scalr daemons to reopen their logfiles (and you're not telling the other daemons to reopen their log files).


Remove debug.


We for sure want to expose the State for a symbol to the checker, but:
- There is no good reason to also inspect unreachableValues (this in fact raises false negatives in CloseResource)
- The merge operation currently guarantee a single state for a symbol
So the contract should be changed to:
public State getStateOf(Symbol symbol)


Use `etc_dir_for`


I don't think this is a question of common use cases, rather than offering flexibility and following expected behaviours. Setting 'defaultTouchEventsEnabled' on the ScrollView is the other way round, we need to enable/disable touch events on the chart, not the ScrollView.

How would you disable the recognizers on the charts then? I haven't found a way to do so without changing your source code.
I mean the chart does have a 'defaultTouchEventsEnabled' flag, why not just use this flag to determine whether the chart should add its default pan/pinch/tap/... gestures? Enabled by default of course, but don't add them when the user explicitly sets 'defaultTouchEventsEnabled' to false (which would be the expected behaviour).


It looks good, although I can't run this code. This branch (as well as "More-CRUD-Assignment") has no migration for the questions table.

The rule to follow from now on is to use a single branch for each assignment and make sure all the code required is there. If the new branch depends on another, start it from its dependency. Let's take a look at an example.

```
$ git branch
  create-questions
* master
```

You are on master and you have `create-questions` branch which has the migration for questions table. Now you want to make some views for the questions table. Here are your options:

1. Continue your work on `create-questions` branch.
2. Merge `create-questions` branch into master first.
3. Create a new branch basing its code on the `create-questions` branch.

Let's say you want to do point 3 (since you already know how to do 1 and 2). First, switch to `create-questions`:

```
$ git checkout create-questions
Switched to branch 'create-questions'
```

And now create your new branch:

```
$ git checkout -b question-views
Switched to a new branch 'question-views'
```

You will notice that your new branch will have all commits that your `create-questions` branch had. One depends on another, so it makes sense to do so.


Oh! I'm so sorry I didn't see that.


Don't include this in the commit(s)


As discussed in mailing list, one option will be to provide a new method sendMetrics(metricsId) that will be called twice:
- application:didFinishLaunchingWithOptions: (for app not running)
- application:didReceiveRemoteNotification: (for app in background)
this method will call a separate end point to send the aerogear-push-id. If we decouple it from registration endpoint i don't need to store the id i can just send it to the proper endpoint.


16435e5c already adds an explicit migration for the HiddenResults column. Running another should result in the same structure, but will reset any values that exist.


You should use JIRA RESTful API for this.


defaulTouches are meant for the "default" system behaviour, which is
passing the touches to the parent view. Think about the event bubbling and
preventDefault() in javascript.

For disabling the gestures there are specific properties to disable them!

בתאריך יום שלישי, 5 במאי 2015, AlBirdie <notifications@github.com> כתב:

> I don't think this is a question of common use cases, rather than offering
> flexibility and following expected behaviours. Setting
> 'defaultTouchEventsEnabled' on the ScrollView is the other way round, we
> need to enable/disable touch events on the chart, not the ScrollView.
>
> How would you disable the recognizers on the charts then? I haven't found
> a way to do so without changing your source code.
> I mean the chart does have a 'defaultTouchEventsEnabled' flag, why not
> just use this flag to determine whether the chart should add its default
> pan/pinch/tap/... gestures? Enabled by default of course, but don't add
> them when the user explicitly sets 'defaultTouchEventsEnabled' to false
> (which would be the expected behaviour).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielgindi/ios-charts/commit/382eb1f18d176ab1dd0f393b72480e2736b3b248#commitcomment-11040647>
> .
>


Salut à tous,

Depuis l'hôte qui héberge sickbeard que donne un wget ou curl de l'URL de
récupération du torrent ?
Ca donnera surement plus d'indication sur le problème réel... après je me
souviens plus ce que sickbeard utilise pour récupérer les torrents mais je
vois que ça de possible et un petit coup d'oeil sur les sources avant (ou
indication de sarakha ou autres contributeurs) t'aidera je pense à pousser
le bon diagnostic...
pour ma part j'ai migré mon sickbeard ce week end de mon syno vers une VM
debian tout roule comme sur des roulettes... (sur mon syno tout roulait je
voulais juste libérer un peu de CPU/RAM)
j'utilise pas Download station mais transmission sur une autre VM

Le 5 mai 2015 10:05, Grincheux41 <notifications@github.com> a écrit :

> Bonjour tout le monde!
>
> j'me permets une petite relance sur ce sujet... j'ai pas relancé avant par
> fainéantise, mais j'en ai marre de devoir retélécharger tous les fichiers
> torrent sur le site t411...
>
> là ce matin j'en ai encore 6 qui sont considérés comme "FIchier torrent
> non valide".
>
> Une idée ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sarakha63/Sick-Beard/commit/2a7b9f710d2a8720b17c6364916ab5852dea040b#commitcomment-11039838>
> .
>


It looks like a bug in line 1161. There is unwanted symbol "`"


Great :) Then do it, please :)
(Maybe the Persistor can directly return the Adapters but no need to do it now.)


yes, this is a bug


Why here? There is already a module for this.


Pas mal


Ok.

I personally like use PSR-4 in projects. I can add new classes, rename, ... without regenerating autoloader.

But in the resulting external package is PSR-4 slower without --optimize-autoloader


Cool :D


`'cache_reddit'` should be `$ENV{reddit_subreddit}`


Sure, I will change it later.


Just a suggestion, perhaps would be cool to make it like this: if above x altitude than faster descent if below x slower descent and in between in some zone exponential slowing to the slower descent speed.


The * should close to the property name rather than the class name


WAV doesn't work either...


@corinnekrych @sebastienblanc +1


I'm still having the same problem but I'm trying to compare with your code.
My PHP versions is this:

PHP 5.6.8 (cli) (built: Apr 16 2015 14:31:05)
Copyright (c) 1997-2015 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies
    with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2015, by Zend
Technologies
    with Xdebug v2.3.2, Copyright (c) 2002-2015, by Derick Rethans

What's yours?

2015-05-05 10:24 GMT+02:00 Serghei Iakovlev <notifications@github.com>:

> @pablomoreno61 <https://github.com/pablomoreno61> Here
> https://github.com/sergeyklay/phalcon-component-example/blob/mail-example/tests/MailTest.php
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sergeyklay/phalcon-component-example/commit/9ad7c50375ef44ad59ec16759f50ac3c7de3ec73#commitcomment-11040111>
> .
>


Var names should be lowercase


I see. Kinda misleading that property. ;)

I must be completely blind, but I fail to see where I can disable the panGestureRecognizer added to the BarLineChartViewBase class. From what I'm seeing, this recognizer is always added.
Sorry for being a little demanding here! ;)


run: `openssl req -x509 -newkey rsa:2048 -nodes -keyout key.pem -out cert.pem -days 365 -subj '/CN=<IP>'` (add `-nodes` command line option, then there won't be any encryption)


We don't normally use github pull requests because the repository on github is just a mirror.

Under what circumstances does the out-of-bound read occur?


I'll look at it later and let you know... (As hard it is to believe, I'm
not always in front of a computer :)

בתאריך יום שלישי, 5 במאי 2015, AlBirdie <notifications@github.com> כתב:

> I see. Kinda misleading that property. ;)
>
> I must be completely blind, but I fail to see where I can disable the
> panGestureRecognizer added to the BarLineChartViewBase class. From what I'm
> seeing, this recognizer is always added.
> Sorry for being a little demanding here! ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielgindi/ios-charts/commit/382eb1f18d176ab1dd0f393b72480e2736b3b248#commitcomment-11040778>
> .
>


Yes, this could be done, but one can't foresee every possible pointer-like type that a user-defined allocator can provide as its `allocator::pointer` (for instance, tests use a homemade [`non_raw_pointer`](https://github.com/boostorg/multi_index/blob/master/test/non_std_allocator.hpp)). `raw_ptr`, on the other hand, it's as generic as it gets.


thanks!


Thanks would this justify tagging a new version? 1.2.1 For example?

*Motivation*
we use https://github.com/francoispluchino/composer-asset-plugin to install Bower assets using Composer.

This dependency `"bower-asset/jquery-stickytabs": "@stable"`, results in fetching the 1.2.0 branch, which still required Bootstrap >= 3.0.0, while our application is stuck on Bootstrap 2.3.2.


@Kirvx :)


Non il faut que t'utilise `php-cs-fixer fix app --fixers=-empty_return` ça ne prendra pas en compte la règle


It is possible that we don't run into it but Coverity thinks it can happen. Should I send a mail to the mailing list instead?


确实有些逻辑没有实现呀，但是UI真的漂亮:)


why did you move it to an `error-pattern` comment?
It should work fine with just adding the underscore to the varname


Which shipped package on Fedora? If you do a pip install progressbar2, it should just work.

I've made it honor silence (though gx.silent has been removed along with gx.debug_level as they both are replaced with logging levels).


If, like me, you find that ampache stops working after this update, check that your apache conf file allows overriding AuthConfig within your ampache folder.

e.g.

<Directory /var/www/>
        Options Indexes FollowSymLinks
        AllowOverride FileInfo AuthConfig
        Require all granted
</Directory>


lgtm


Thanks @pmichna for pointing it out. I am actually planning to keep these two lines (29-30) and remove 22-23, next time I update README.md.


Thanks for the immediate response.

If it works, it sounds great!

Now I am forced to explicitly set the TARGET before running npm, I wish you can improve it!

P.S. On the other story, I would prefer to keep it as "config/index.js" so that newcomers (like me) are not confused if  the parameter should be a folder or not.


it's a node thing i think


There is no good reason to also inspect unreachableValues (this in fact raises false negatives in CloseResource)


OK, I see the difference. Although you could document that multi_index uses the `get_pointer` customization point and it was the client's job to specialize it for their `pointer` type, with `raw_ptr` the client doesn't need to do much, just support explicit construction from `0`.


LGTM :+1:


Hello Eliot,
why did we remove the following line ?
> @user.update_attribute(:email_favorites, false)


I'm here now so don't worry :)

As far as the patch goes, I don't think this is the right solution because it silently stops filling the buffer on overrun. It should probably wrap instead, but I'm not sure what is correct. I'll ask Ted (the original author) to take a look.


This is not the Tango way to start a device server. This is usually the role of the starter.
Related to #6 #12


The line 67 from my MailService:

$content = $view->getRender('mail', $name, $personalizationParams);

throws the error:

Violación de segmento (`core' generado)

2015-05-05 11:11 GMT+02:00 pablo moreno <pablomoreno61@gmail.com>:

> I'm still having the same problem but I'm trying to compare with your
> code. My PHP versions is this:
>
> PHP 5.6.8 (cli) (built: Apr 16 2015 14:31:05)
> Copyright (c) 1997-2015 The PHP Group
> Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies
>     with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2015, by Zend
> Technologies
>     with Xdebug v2.3.2, Copyright (c) 2002-2015, by Derick Rethans
>
> What's yours?
>
> 2015-05-05 10:24 GMT+02:00 Serghei Iakovlev <notifications@github.com>:
>
>> @pablomoreno61 <https://github.com/pablomoreno61> Here
>> https://github.com/sergeyklay/phalcon-component-example/blob/mail-example/tests/MailTest.php
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/sergeyklay/phalcon-component-example/commit/9ad7c50375ef44ad59ec16759f50ac3c7de3ec73#commitcomment-11040111>
>> .
>>
>
>


@eliotsykes I have updated it.


What is the need of Ant? Qmake is not enough?
Related to #6 #11


@kalehrishi


> P.S. On the other story, I would prefer to keep it as "config/index.js" so that newcomers (like me) are not confused if the parameter should be a folder or not.

We can skip whole `--config config/index.js` if we move configuration to project root and name it as `webpack.config.js`. Then webpack will find it automatically and things stay simpler.

I'll do these changes to the book in a bit.


I just wanted to start a discussion with that patch. The current fix should make it clear what the issue is. Thanks for taking care ...


Is this specific to the Solaris deployment?


1: D400 First line should end with a period (not 's')
22:1: E402 module level import not at top of file
23:1: I100 Imports statements are in the wrong order. import os should be before from six.moves
23:1: I201 Missing newline before sections or imports.
23:1: E402 module level import not at top of file
24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: I100 Imports statements are in the wrong order. import tempfile should be before import time
26:1: E402 module level import not at top of file
27:1: I100 Imports statements are in the wrong order. import sys should be before import tempfile
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. import datetime should be before import sys
28:1: E402 module level import not at top of file
33:1: I101 Imported names are in the wrong order. Should be abort, url_for
33:1: E402 module level import not at top of file
34:1: I201 Missing newline before sections or imports.
34:1: E402 module level import not at top of file
35:1: I201 Missing newline before sections or imports.
35:1: E402 module level import not at top of file
36:1: I201 Missing newline before sections or imports.
36:1: E402 module level import not at top of file
38:1: I100 Imports statements are in the wrong order. from invenio.config should be before from six
38:1: E402 module level import not at top of file
39:6: E126 continuation line over-indented for hanging indent
64:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.config
64:1: E402 module level import not at top of file
65:1: E402 module level import not at top of file
66:1: E402 module level import not at top of file
67:1: E402 module level import not at top of file
68:1: E402 module level import not at top of file
69:1: I101 Imported names are in the wrong order. Should be get_all_restricted_recids, get_record, record_exists, search_unit_in_bibxxx
69:1: E402 module level import not at top of file
71:1: E402 module level import not at top of file
72:1: E402 module level import not at top of file
73:1: E402 module level import not at top of file
74:1: I101 Imported names are in the wrong order. Should be EscapedXMLString, X
74:1: E402 module level import not at top of file
77:25: E203 whitespace before ':'
78:25: E203 whitespace before ':'
79:25: E203 whitespace before ':'
84:25: E203 whitespace before ':'
88:25: E203 whitespace before ':'
106: D205 1 blank line required between summary line and description (found 0)
118: D400 First line should end with a period (not 'r')
118: D200 One-line docstring should fit on one line with quotes (found 3)
134:9: E266 too many leading '#' for block comment
135:9: E266 too many leading '#' for block comment
146:1: E302 expected 2 blank lines, found 1
146: D202 No blank lines allowed after function docstring (found 1)
146: D400 First line should end with a period (not 'r')
146: D200 One-line docstring should fit on one line with quotes (found 3)
168:1: E302 expected 2 blank lines, found 1
168: D200 One-line docstring should fit on one line with quotes (found 3)
178:1: E302 expected 2 blank lines, found 1
178: D401 First line should be in imperative mood ('Get', not 'Gets')
178: D202 No blank lines allowed after function docstring (found 1)
178: D200 One-line docstring should fit on one line with quotes (found 3)
186:9: E221 multiple spaces before operator
191:1: E302 expected 2 blank lines, found 1
191: D401 First line should be in imperative mood ('Return', not 'Returns')
191: D205 1 blank line required between summary line and description (found 0)
201:1: E302 expected 2 blank lines, found 1
201: D400 First line should end with a period (not 'e')
201: D205 1 blank line required between summary line and description (found 0)
211:1: E302 expected 2 blank lines, found 1
211: D400 First line should end with a period (not 'e')
211: D205 1 blank line required between summary line and description (found 0)
221:1: E302 expected 2 blank lines, found 1
221: D209 Multi-line docstring closing quotes should be on a separate line
221: D202 No blank lines allowed after function docstring (found 1)
221: D400 First line should end with a period (not 'r')
221: D205 1 blank line required between summary line and description (found 0)
230:1: E302 expected 2 blank lines, found 1
230: D400 First line should end with a period (not 'e')
230: D205 1 blank line required between summary line and description (found 0)
246:1: E302 expected 2 blank lines, found 1
246: D400 First line should end with a period (not 't')
246: D205 1 blank line required between summary line and description (found 0)
278:85: E261 at least two spaces before inline comment
278:86: E262 inline comment should start with '# '
283:1: E302 expected 2 blank lines, found 1
283: D200 One-line docstring should fit on one line with quotes (found 3)
288:5: E266 too many leading '#' for block comment
289:5: E266 too many leading '#' for block comment
296:9: E116 unexpected indentation (comment)
297:13: E116 unexpected indentation (comment)
298:17: E116 unexpected indentation (comment)
299:13: E116 unexpected indentation (comment)
300:17: E116 unexpected indentation (comment)
301:13: E116 unexpected indentation (comment)
302:17: E116 unexpected indentation (comment)
303:13: E116 unexpected indentation (comment)
304:17: E116 unexpected indentation (comment)
305:13: E116 unexpected indentation (comment)
306:17: E116 unexpected indentation (comment)
308:9: E116 unexpected indentation (comment)
309:13: E116 unexpected indentation (comment)
310:17: E116 unexpected indentation (comment)
311:13: E116 unexpected indentation (comment)
312:17: E116 unexpected indentation (comment)
313:13: E116 unexpected indentation (comment)
314:17: E116 unexpected indentation (comment)
316:1: E302 expected 2 blank lines, found 1
316: D401 First line should be in imperative mood ('Print', not 'Prints')
316: D202 No blank lines allowed after function docstring (found 1)
333:37: E713 test for membership should be 'not in'
334:13: E266 too many leading '#' for block comment
335:13: E266 too many leading '#' for block comment
349:5: E266 too many leading '#' for block comment
396:1: E302 expected 2 blank lines, found 1
396: D401 First line should be in imperative mood ('Generate', not 'Generates')
396: D202 No blank lines allowed after function docstring (found 1)
415:1: E302 expected 2 blank lines, found 1
415: D401 First line should be in imperative mood ('Generate', not 'Generates')
415: D202 No blank lines allowed after function docstring (found 1)
440:30: E261 at least two spaces before inline comment
446:9: E266 too many leading '#' for block comment
447:9: E266 too many leading '#' for block comment
470:9: E266 too many leading '#' for block comment
475:1: E302 expected 2 blank lines, found 1
475: D401 First line should be in imperative mood ('List', not 'Lists')
475: D202 No blank lines allowed after function docstring (found 1)
475: D200 One-line docstring should fit on one line with quotes (found 3)
496: D401 First line should be in imperative mood ('Return', not 'Returns')
496: D202 No blank lines allowed after function docstring (found 1)
512:12: E127 continuation line over-indented for visual indent
521:1: E303 too many blank lines (3)
521: D401 First line should be in imperative mood ('Generate', not 'Generates')
521: D202 No blank lines allowed after function docstring (found 1)
521: D208 Docstring is over-indented
545:17: E128 continuation line under-indented for visual indent
546:17: E128 continuation line under-indented for visual indent
547:17: E128 continuation line under-indented for visual indent
548:17: E128 continuation line under-indented for visual indent
549:17: E128 continuation line under-indented for visual indent
565:1: E302 expected 2 blank lines, found 1
565: D200 One-line docstring should fit on one line with quotes (found 3)
571:1: E302 expected 2 blank lines, found 1
571: D200 One-line docstring should fit on one line with quotes (found 3)
579:1: E302 expected 2 blank lines, found 1
579: D401 First line should be in imperative mood ('Generate', not 'Generates')
584:1: E302 expected 2 blank lines, found 1
584: D401 First line should be in imperative mood ('Generate', not 'Generates')
588:1: E302 expected 2 blank lines, found 1
588: D401 First line should be in imperative mood ('Return', not 'Returns')
588: D209 Multi-line docstring closing quotes should be on a separate line
588: D400 First line should end with a period (not 'd')
588: D205 1 blank line required between summary line and description (found 0)
603: D401 First line should be in imperative mood ('Return', not 'Returns')
603: D400 First line should end with a period (not 'C')
603: D200 One-line docstring should fit on one line with quotes (found 3)
617: D210 No whitespaces allowed surrounding docstring text
631:29: E261 at least two spaces before inline comment
631:30: E262 inline comment should start with '# '
638:31: E261 at least two spaces before inline comment
638:32: E262 inline comment should start with '# '
652:1: E302 expected 2 blank lines, found 1
652: D401 First line should be in imperative mood ('Return', not 'Returns')
652: D200 One-line docstring should fit on one line with quotes (found 3)
673:1: E302 expected 2 blank lines, found 1
673: D401 First line should be in imperative mood ('Generate', not 'Generates')
679:1: E302 expected 2 blank lines, found 1
679: D400 First line should end with a period (not 'e')
679: D205 1 blank line required between summary line and description (found 0)
692:1: E302 expected 2 blank lines, found 1
692: D200 One-line docstring should fit on one line with quotes (found 3)
712: D200 One-line docstring should fit on one line with quotes (found 3)
727:31: F841 local variable 'e' is assigned to but never used
731:1: E302 expected 2 blank lines, found 1
731: D200 One-line docstring should fit on one line with quotes (found 3)
740:5: E266 too many leading '#' for block comment
745:5: E266 too many leading '#' for block comment
756:9: E266 too many leading '#' for block comment
760:9: E266 too many leading '#' for block comment
765:1: E302 expected 2 blank lines, found 1
765: D400 First line should end with a period (not 's')
765: D205 1 blank line required between summary line and description (found 0)
772:5: E266 too many leading '#' for block comment
777:39: E261 at least two spaces before inline comment
777:40: E262 inline comment should start with '# '
783:5: E266 too many leading '#' for block comment
787:5: E266 too many leading '#' for block comment
789:12: E713 test for membership should be 'not in'
790:16: E126 continuation line over-indented for hanging indent
793:18: E261 at least two spaces before inline comment
795:5: E266 too many leading '#' for block comment
797:12: E127 continuation line over-indented for visual indent
804:5: E266 too many leading '#' for block comment
806:12: E127 continuation line over-indented for visual indent
814:12: E127 continuation line over-indented for visual indent
827:12: E127 continuation line over-indented for visual indent
828:12: E127 continuation line over-indented for visual indent
832:5: E266 too many leading '#' for block comment
834:12: E127 continuation line over-indented for visual indent
835:36: E261 at least two spaces before inline comment
839:5: E266 too many leading '#' for block comment
841:12: E127 continuation line over-indented for visual indent
845:12: E127 continuation line over-indented for visual indent
848:5: E266 too many leading '#' for block comment
850:12: E127 continuation line over-indented for visual indent
853:5: E266 too many leading '#' for block comment
855:12: E127 continuation line over-indented for visual indent
860:1: E302 expected 2 blank lines, found 1
860: D401 First line should be in imperative mood ('Run', not 'Runs')
860: D400 First line should end with a period (not 'k')
860: D200 One-line docstring should fit on one line with quotes (found 3)
866:90: E203 whitespace before ':'
866:91: E231 missing whitespace after ':'


24:1: I100 Imports statements are in the wrong order. from flask_login should be before import pypeg2
25:1: I201 Missing newline before sections or imports.


25:1: I201 Missing newline before sections or imports.
27:1: I100 Imports statements are in the wrong order. from flask should be before import six
28:1: I201 Missing newline before sections or imports.
29:1: I201 Missing newline before sections or imports.
30:1: I201 Missing newline before sections or imports.
32:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils


53:1: I201 Missing newline before sections or imports.
54:1: I201 Missing newline before sections or imports.
56:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from werkzeug.local
186:5: B901 blind except: statement
254:9: E731 do not assign a lambda expression, use a def


why you remove the step on ivi ?


@ThibaudDauce J'ai vu ton message dans l'issue après ;) J'avais oublié qu'on pouvait donner des bouts de règles comme ça en paramètre.


and ***are*** located


this seems the wrong way round. Shouldn't the `SearchService` fixture take an `ActorSystem` which is created by the `TestKit` fixture?


Changed Strings


Hi


Why is return type JsonNode?Why not ObjectNode??? If use ObjectNode, allow chain with this set method.


Yeah, the idea is that this repo would define versions of the RPC product and that means it must reference a specific SHA. With each new version the SHA could be updated depending on the purpose of that version.


rweerwer


I agree, it was a little verbose to have so many files (at the beginning there were 4 files there) and took me some time to figure it how it works (again, as a newcomer to webpack, etc.). The simpler the better, always!


Added title item to Complex alert according to https://developer.apple.com/library/ios/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/Chapters/ApplePushService.html#//apple_ref/doc/uid/TP40008194-CH100-SW1


I think I was trying to allow this to be kept separate from the global EC so that we didn't get deadlock... are we only using the EC below?


I realized that as I was typing :P, but sleep now, because I have to be up for work in 3.5 hours


Yep - it was only for the Futures - so I moved it to an akka managed EC.


а как по другому? Дело в том что, делал без объявления функции, но проблема в том что при рекурсивном вызове строка var array = []; обнуляет массив в котором уже есть елементы, в результате - 0 на вложеных элементах.


missing /sbin ubuntu installations fails


не возвращает


i would suggest to directly define it as:  post '/users/:user_id/address'...aliases are not worth it in padrino controllers. I think we should move this to users_controller.rb. There you will need to define an action: post  ':user_id/address'


alooooooooooo


Don't you think we should have just rethrow the error here and let the users catch and handle the error themselves?


Yeah, you are probably right - I've spent far to much of my weekend dealing with cake pattern test fixtures to tell anymore.


You dont need to return @input. It is an instance variable and it can directly be accessed from the controller


@ace2nutzer @ChronoMonochrome One or more of my last 11 commits (dated 5th may) causes freeze on "samsung galaxy s advance" and I can't neither access to recovery in fact I had to restore system using odin. Do you know which one could be?


проверка надо т.к. нам надо искать только в типе "элемент". else упустил согласен, но по сути он нам и не нужен.


I was thinking that the operations involve SRC and DST and those are from/to the "front-end" (image datastore) to/from the host... So I guess that checking hostname would only evaluate to true when the host is the same as the front-end...


More tests:

$content = $view->render('mail', $name, $personalizationParams);

throws the same error, but pick gives blank content:

$content = $view->pick("mail/$name")->getContent();

2015-05-05 11:21 GMT+02:00 pablo moreno <pablomoreno61@gmail.com>:

> The line 67 from my MailService:
>
> $content = $view->getRender('mail', $name, $personalizationParams);
>
> throws the error:
>
> Violación de segmento (`core' generado)
>
>
> 2015-05-05 11:11 GMT+02:00 pablo moreno <pablomoreno61@gmail.com>:
>
>> I'm still having the same problem but I'm trying to compare with your
>> code. My PHP versions is this:
>>
>> PHP 5.6.8 (cli) (built: Apr 16 2015 14:31:05)
>> Copyright (c) 1997-2015 The PHP Group
>> Zend Engine v2.6.0, Copyright (c) 1998-2015 Zend Technologies
>>     with Zend OPcache v7.0.4-dev, Copyright (c) 1999-2015, by Zend
>> Technologies
>>     with Xdebug v2.3.2, Copyright (c) 2002-2015, by Derick Rethans
>>
>> What's yours?
>>
>> 2015-05-05 10:24 GMT+02:00 Serghei Iakovlev <notifications@github.com>:
>>
>>> @pablomoreno61 <https://github.com/pablomoreno61> Here
>>> https://github.com/sergeyklay/phalcon-component-example/blob/mail-example/tests/MailTest.php
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/sergeyklay/phalcon-component-example/commit/9ad7c50375ef44ad59ec16759f50ac3c7de3ec73#commitcomment-11040111>
>>> .
>>>
>>
>>
>


eeeeeeee


线性回归，主要利用最小二乘法，为解决欠拟合的问题，引入局部加权（加入核函数），为解决矩阵为非奇异阵，引入缩减系数的方法（岭回归，前向逐步回归等方法）

具体见：http://blog.csdn.net/moodytong/article/details/10041547


you should have a constant file that  save all the constant like url, notification name ...


Why are you removing is_primary? Also, please add new migrations for addition and removal of columns. Migrations are supposed to be immutable.


Same problem with long email addresses and websites! We should find a way to space this dynamically. Actually this as been on my todo-list for more than a year. ;)


как лучше сделать? :) помойму всё красиво...


Changed to develop since the master is pretty empty and not a good test subject.


Just another copy of the NSA Android apk.


will change later, see https://github.com/reborndb/qdb/issues/8


other than this, the PR looks good. If you could swap them round so `SearchServiceFixture` doesn't create its own actor system then this looks good to go, thanks!


Tests are missing. Please add them and act on the comments. Rest of it looks cool.


Im good with this. Awesome job Eddie. We shall wait for Nate to look at this.


var aaa = node.childNodes;

aaa[i]   ???


This CSV has already been added in https://github.com/alphagov/whitehall/pull/2129

Would you be able to consolidate these?


A faire sauter!


Ca marche pas.


Done :)


I think that  you should have a function that  use the project mode debug or distribute to determine whether it need output


``` if (isTextExists(node.childNodes[i], text)) { ```

возвращает булеан и так

вы написали

``` if (true) return true ```


This fixes an error because All Groups is only one of three boxes on the page found via the Groups menu item, so the page title should also be Groups.


``` var element = node.childNodes[i]; ```


https://learn.javascript.ru/function-declaration-expression
https://learn.javascript.ru/named-function-expression


@pablomoreno61 Please use full example. You can just clone it and test.


@pablomoreno61 branch `mail-example`


Thanks for posting the work.  I made some changes to the `h5particles.txt` which I will commit now.

@roigcarlo, @tuopuu  ,  maybe a PR would be the best place to continue this discussion.  I have some general questions but will ask them there.


Weird. The bug didn't show it's ugly face until I did this. Will investigate.


Thanks for pointing that out: https://github.com/jenkinsci/acceptance-test-harness/commit/7aaa05cd31d6afbd45e5b77ef1067c98767a1e3f


*attachment

May not be worth fixing since this is class is doomed to fire.


@michaellnorth @embersherpa r? kthx.


`options || (options = {});`


Hmm, so perhaps we have a different bug somewhere.


@DennisMoosmann du hast schon wieder vergessen das hinzugefügte modul zur bower.json hinzuzufügen. Dh das die app nur bei dir funktioniert, bei allen anderen kracht es ;)


Hello @dionysiosAthanasopoulos I just realized that youre not commenting the changes that you're doing after each review step. It will be nice to have a feedback after each review step as it eases the review process.

Don't be afraid about the commit style, a rebase will be done before merging your PR  in order to merge all the changes in a single commit, but from now I would love to have something like:

- Monitor V1
- Tests added
- Code refactored

So please for your next commits could you try to be more descriptive?

Thank you very much :D


This is already handled by osc:

`osc delete rcs --all`
`osc delete pods --all`


I was actually using `show` on purpose, but I don't insist on keeping it.  However, you want to do this instead:

```haskell
    , FrontendMsgSuccess $ "Service key: " <> fromServiceKey key
```


`osc delete services --all`


They probably equal the ones from GTAIV (as the rest of the native didn't really change), could somebody confirm that?
```c#
public enum class ExplosionType
{
		Default,
		Molotov,
		Rocket,
};
```


if urls.startswith('https')


Isn't there a message that you can match again?  That would be more interesting; everything is dashboard nowadays.


@sprankhub oh, I just saw your *widthForStringUsingFontSize*. That should make this easy!


I was thinking of something more like this:

{@code live:Bloomberg}

{@code latesthistorical, DEFAULT_HTS_RATING}

{@code fixedhistorical:2011-08-03}

{@code fixedhistorical:2011-08-03, DEFAULT_HTS_RATING}

{@code snapshot:DbSnp~1234}


*redirects


I would have said that this already works... But I may be wrong. If you are sure that this is still a problem, please open an issue.


@Denwid @hubifant @sphingomyelin
Just made a small change to allow to resize the property window (test here : http://dhlabpc3.epfl.ch/relational/dev/client ).
To use, you can either cherry pick it, or rebase your branch on dev, or wait until merge.


![pusher_03](https://cloud.githubusercontent.com/assets/3203117/7470234/0e04ff22-f356-11e4-89dc-0485ac96bb4c.png)

クライアント側にcommentが追加されたときに再読み込みしなくても更新されるような仕組みを書く。クライアント側のコードなのでJavaScriptで書く必要がある。RailsではJavaScriptをCoffeeScriptというサブセット言語（AltJSと言う）で書くのがデフォルトになっている。

app/assets/javascripts/comments.coffeeをJavaScriptで書き直すと以下のようになる。

```
$(function(){ // おまじない
  // チャンネルの購読
  var pusher = new Pusher('982dd54e0a971180f612')
  var channel = pusher.subscribe('my-channel')

  // チャンネルのイベントを監視、発行されたときの処理を書く。
  channel.bind('my-channel', function(data){
    // comments idを持つHTML要素の中にcommentを一行追加
    $('#comments').append('<tr><td>' + data.content + '</td><td>' + data.name + '</td></tr>');
  });
});
```

やっていることは簡単でmy-channelのmy-eventで新しい動作があった場合、その内容をcomments idを持つHTML要素の中に追加しているだけ。

next: 44eaf82527835934faa8c3e20cdac41c3cffd5f6


First commit in 4 weeks, woop! :star2:


Done in latest commit on the develop branch: 9798fb4


as you can see [here](https://github.com/maxogden/multiplex/blob/master/index.js#L134) multiplex uses its own internal id pool and what you pass to `createStream()` is pure meta. yeah the documentation is a little wonky there.

i created a gist to demonstrate: https://gist.github.com/juliangruber/866f388f0b9e2e591462


Still, "message" variable is initialized outside of the debug check condition, remove this variable altogether and put the message inline ..


I think it should be -v $(PWD):/host here


Vielen Dank. Das war mir gar nicht aufgefallen.


This application can be configured with a set of images and a wav file. It will bind them together, to create an Executable JAR File, that displays a slideshow.


This is quite a lot of code with no tests.  How comfortable are you with that?


please merge them all together into one request.


You could also use [raw string literals](http://doc.rust-lang.org/reference.html#raw-string-literals) (`r#"Some "fancy" description"#`) instead.


Okay the properties are named:
`dragEnabled`
`highlightPerDragEnabled`
`pinchZoomEnabled`
`scaleXEnabled`
`scaleYEnabled`

Same goes for the Android version :-)


Could be merged into the previous commit.


Right. Or the line above could be: branch = opts.get('environment') or 'base'
That would also catch an empty string environment name.


I could but they are not serializable (bad for record/replay) and require a polyfill.


Without this, the error is similar to https://github.com/tpaviot/oce/issues/461


@dimagi/team-commcare-hq FYI to use touchforms locally you now need to build the JS files with grunt


Yeah I don't like that syntax. It's un-expressive, and I my jshint/jscs warns on it.


Maybe these should be a relative path or a path set as a variable


Yes I agree.


I want to specifically use Promise polyfill without all the other polyfills provided by corejs through Babel.


For example:
magnet:?xt=urn:btih:51ab56b3b74edd0eda7ddf5cc266ae36bc37b199&dn=Salem%20S02E05%20HDTV%20XviD-FUM%5Bettv%5D&tr=udp%3A%2F%2Ftracker.openbittorrent.com&tr=udp%3A%2F%2Ftracker.publicbt.com

http://rarbg.to/download.php?id=tzxngj5&f=The+Last+Man+on+Earth+S01E13+1080p+WEB-DL+DD5+1+H+264-BTN+%5Beztv%5D-%5Brarbg.to%5D.torrent

Extratorrent seems much less frequent now, but these links are possible:
http://extratorrent.cc/download/4175287/Naked+and+Afraid+S04E03+Mayan+Sacrifice+HDTV+x264-W4F%5Bettv%5D.torrent


Shouldn't there be a corresponding pragma pop?


@af7 The problem with the solution I proposed that env variables are per process in Node. This means the Webpack configuration couldn't see it even if I set it (separate process). As a workaround you can try `set TARGET=build && webpack` kind of script. I added a warning about this at the book.

By the way, I simplified the configuration as promised. There's no need for `--config` anymore.


updates


Yes, the Q matrix is set in the ctmarkov class, but it can be changed when
creating an instance of ctmarkov. It is currently set as whatever is in
ctmarkov in the markovObjects.py file. I didn't think about the equilibrium
frequencies needing to match up with the Q matrix, although it obviously
makes sense. Thanks for mentioning that. Thankfully, I set the equilibrium
frequencies as arguments in the function, although it would be ideal to
automate them from the Q matrix.

On Tue, May 5, 2015 at 4:20 PM, Jeremy Brown <notifications@github.com>
wrote:

> The likelihood function seems to work well. Here's what I got:
>
> t.treeLike(t.root)
>
> ('the log likelihood:', -0.9280676731846361, 'the likelihood:',
> 0.11801367285640654)
>
> I might not have mentioned this, but the convention is to use the natural
> log of the likelihood (rather than the base 10 log) as the
> "log-likelihood". I'm not entirely sure why this is true, but it's
> something to be aware of. It's a little difficult for me to verify that
> these calculations are correct without being able to read in an outside
> tree and dataset, and to be able to set the equilibrium frequencies and
> relative rates (R-matrix). Are you setting the Q-matrix in your ctmarkov
> class? Can other values be used? Bear in mind that the equilibrium
> frequencies that you pass as an argument to your treeLike() function need
> to match the equilibrium frequencies used to define your Q-matrix in order
> for the calculations to be valid.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/henicorhina/CompPhyloSpr2015/commit/d07079dc136861a6f48733bada92c4d364d39413#commitcomment-11054191>
> .
>

--
Oscar Johnson

Ph.D. Student
Museum of Natural Science
Department of Biological Sciences
119 Foster Hall
Louisiana State University
Baton Rouge, LA 70803


又一个被翻译坑。。。。


Got it! Thanks for the concise explanation.


that seems to make more sense...


Need a space here between client_ip and statement


You could write a method that determines if the game is over. Ex

```ruby
def game_over?(p1_lives, p2_lives)
  # your code here
end
```


Ok, yeah -- the installers should render the check unnecessary. :shipit:


Also space here between client_ip, and "Client..."


add comment from the PR, why do we catch this


You can replace this with `if current_turn`


I would rename the function to be more explicit, like "test_can_trace_client_ip_in_query"


This commit was supposed to be:

"Improve description of `derfinder_analysis.R`".

I guess that the backticks broke the commit message.


You could break this up into it's own method

```ruby
def guess_correct?(guess, answer)
  # your code here
end
```


Drop " * Note"


Tested in my environment with the test-case that was having an error, worked fine.


Weird that this can only be called after calling createOrderedTask, perhaps factor slightly differently?


Good job! The code looks good overall. I would break up some of the logic into methods.


So actually in local tests this is 127.0.0.2, I talked to Adam about this as I thought it was returning the coordinator ip rather than the client ip. I'm just not sure how to go about fetching the actual client ip from the cluster.


This breaks things for me.  Please see issue https://github.com/rg3/youtube-dl/issues/5624


We can probably remove this try/except block here, since we can directly verify that it's '127.0.0.1'


Seems like a bug Dev12 csc


Mounting this inside ``gre/modules`` constrains future us.  You can always make ``resource://jetpacks/`` point to ``resource://gre/modules/jetpacks``, but as soon as you refer to ``resource://gre/modules`` you make it almost impossible to move the jetpacks around as a unit.  This is because you can't update out-of-tree consumers (addons!).

This bit me: I would have loved for Android-specific modules to be rooted at ``resource://not-gre/modules`` so that I could do interesting Android-specific-things for those modules, but I concluded it could not be done.


Something's fucky.


*fruits of our labor


Interesting. I ran the test myself multiple times and I always got back '127.0.0.1'.


These probably shouldn't be an enum, but NS_OPTIONS instead.


May I suggest using args4J for processing command line arguments? :)

Makes it easy to handle many variables in arguments. ^^


I know you're doing this is a tech demo but I think this approach is not what we will want.  First, we almost certainly don't want all in-tree features in ``/jetpacks/`` -- they should live closer to their features, in ``browser/`` or ``mobile/android/`` or whatever.  When the time comes, we can find a declarative build syntax for this stuff.


I'd consider a bit of refactoring here, so it's easier to add more cases, but in general I'm happy with what's being tested!


I always get back 127.0.0.2 it's probably because in osx you need to add the extra local loop back addresses. Using a fixed IP is probably not the way to here.


adding 16 counters with threshold 5


Hola, ¿instalaste camptix y configuraste todas las claves de MercadoPago?

El plugin actualmente está funcionando para la web de WordCamp Buenos Aires. http://buenosaires.wordcamp.org/2015/

¿Qué versión del plugin estás utilizando?

¿Lo descargaste desde GitHub o el directorio de plugins de WordPress.org?


I think @Mossop suggested this, but I hope we can generate this metadata in the future.


I don't have time to fix this right now, but just to flag that if we're going to do this we really ought to make everything in Builtins and IO either part of a namespace, or assert that they can't be used anywhere else.

I've just encountered a problem in some code which uses 'fork' ambiguously. It's probably correct that having an unqualified name fork at the top is bad so putting namespaces in the appropriate places seems like the right thing to do.

Actually I don't think there is any value in anything being unqualified. It's just a bit of a tedious update to do.


Ok, now I see the refactoring, but I'd still break out each of these test groups into a separate test case so that mistakes in one test don't accidentally invalidate the results of other tests. False positives are ok, but false negatives are also possible and best avoided.


Fixed in e0f1c2f


Sorry, no, that's not what I meant to say.

I don't think the FD check can do any good here.  As far as I could tell when reading through it with you yesterday, `closeCachedConnections` properly waits to fire the `Deferred` it returns until all the connections have had their `connectionLost` called and are properly removed from the pool.  So all the polling-for-FDs can do is to cause a hang rather than an exit with an error, if there are unrelated clients or another connection pool spinning up new connections.

So there's definitely no need to put it back in.

That said, what the test is trying to do is to tell the pool "immediately kill any outstanding connections, you are done, don't make any new connections past this point, just error if you try" and there's no way to tell the connection pool to do that; this is a missing API in Twisted and should probably be added.


This needs to be in a library folder like we talked about on the phone. `library/images/library/wordmark-header.svg`


Classes that start with an underscore are part of a single layout. They belong in that layout's partial.


Thanks man!  Good catch :)


Absolute path to images. `/images/library/homepagebg.png`. I also don't see this image committed anywhere.


This belongs to the Library layout, not the Footer component


Why is this fixed position?


Wut?


This belongs in the Library layout, not the Heading component. Heading component classes start with `heading`


Wordmark is its own component


Just to make a reference, this is issue #17


I'm not sure I understand what this class is suppose to be for


Library layout


Why title-1?


Why are we setting font sizes? Why can't we use our Pattern Library patterns? Why not extend our H1?


the question mark means it's an optional method on this internace. the formatting is curtesy of clang-format. Martin has a fix pending :)


What is this?


After looking at the Appveyor and mingw web sites, I believe that mingw uses the Windows DLL library, so perhaps Appveyor's Windows library is different from yours and buggy.  I couldn't really find what precise environment Appveyor uses.

I don't get much information from gcc -v on that platform.

I believe the new check might pass because while
(exact->inexact (expt 3 1000)) => +inf.0
(exact->inexact (expt 3 100)) => 5.153775207320113e47
is finite, so I expect the Windows library to get
(flexpt 1. 5.153775207320113e47) => 1
(which is an intermediate result _num.scm computes) correctly.


what an amazing commit (test purposes)


hmm.. what should it be?


I'd describe this function as "converts bytestrings to native strings, but leaves text untouched", given that you've defined those terms in the module header.


This is not part of the main content component


Not part of the main content component


I think you meant 'ShiftKey'
LOL


Instead of this you could put view="plone.app.layout.globals.interfaces.IViewView" in the ZCML for the viewlet, and then it'd be handled as part of the viewlet adapter lookup. This is probably fine, though.


That should be
 (flexpt 1. 5.153775207320113e47) => 1.
of course.


Part of the Library layout


Not part of the sub-nav component (or needs to be turned into one


Same goes for most of the below


Remove. From Pattern Library


Thanks rasky, I will look into incorporating it when I get some time.


Library is a layout, not part of global. Should be in a `layout` folder


Yaaay! :+1:


Funciona hasta la linea 245 :cry:


Many of these variables don't pertain to the pull request. Please remove the ones that don't.

Also, split into multiple partials, not one large one


Isn't this a tad risky? Maybe there's some reliable condition to check instead to guard against an infinite loop come about due to bugs?


wut?


$env:chocolateyPackageFolderに値が入らないことがあったので、$versionに固定値をそのまま入れるようにしました。


Instead of multiple imports, create an `_index.scss` file in each folder, import the partials into there, then import the folder. For instance, you'd have a `global/_index.scss` file and you'd `@import "global";`


Where are the rest of the imports?


What would work here would be just use a regex, were we look for 127.0.0.x that should be tolerant of everyones env, and get rid of the socket validation stuff, and provide at least some level of sanity checking.


why?


Where is the wrapping `_library` layout class?


Write-ChocolateySuccess, Write-ChocolateyFailureがdeprecatedになっていたので削除しました。


ahh "Ouput" :)


This is not a [heading role](http://www.w3.org/TR/wai-aria/roles#heading)


I'm retarded


Actual code


This link will go nowhere. If you want it to go to the home page, it should be to `/`


If there is styling attached to this, it needs a class


Instead of header nav, why not `main-nav`?


Súper


Seems to be an issue similar to this one (contains a nice answer):
http://stackoverflow.com/questions/16429331/multiple-hotkeys-not-working-with-alt-number-keys


bet, winOrLoss, wallet, totalPlayer and totalDealer are all going to be variables in the main function, not part of the class, so that'll simply things here. if we don't have the two integers for totalPlayer and totalDealer, how will we keep track of their current points value, in order to check it in the boolean expressions?


Supongo que jsoncargar usa TDAJSON porque xmlguardar recibe TDAXML?


i'm not sure how you want to do void playGame, but I think it would be a function in main, not part of the class?
double Wager is also part of main


Ahhh we should really be int64_t everywhere :)


@rubahness https://github.com/myano/jenni/pull/172


fml. I'm making issues from now on. :cactus:


Hey man, you might want to remove this file and re-add it to git with this values coming from env vars or something.
- just my 2 cents (:


Mentioned wrong issue. Should be issue #19.


const int SIZE is still set to 13 up at the top. if we reset it to 52, then the size of the arrays when they are declared would be that variable, instead of the number 52.


52 would be SIZE here, also


There should be a new line separating mixins from css.

body {
  @include clearfix();

  padding: 0;
}

body {
  background-color: red;

  @include clearfix();
  @include transform(translate(-50px));

  padding: 0;
}


should use background-color instead of background. Using background will overwrite previously declared styles like background-image, background-position, background-size, etc.


Use border-width: 0; instead of border: 0;. Border will overwrite previously declared styles such as border-color, border-radius, etc.


Css properties should be alphabetized.


Cool. I didn't know about doc frags. Thanks for the clarification.


test


nit: is how dart style format this line currently? seems ugly.


Can you explain the reasoning to move away from decorators to static injection?


Thanks for finding that! :)


Hi there,
The changes made to Collection constructor doesn't make sense to me.
 `new Collection(new Model)` should give a collection that contains one Model.

 But this change convert the Model to an array, and returns a collection that contains one Array:


I no longer have play game in a class. I used the structure you put up but
am making changes for all the posibilities. So i will just leave it in
main. Also in terms of wager. winOrLoss could possibly equal to P which is
Push which means there is a tie. If there is  tie then player returns his
bets and nothing happens
On May 5, 2015 5:29 PM, "goldfish" <notifications@github.com> wrote:

> i'm not sure how you want to do void playGame, but I think it would be a
> function in main, not part of the class?
> double Wager is also part of main
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/b36e1ba2e47a911bd052977125fbdf50784cc3a4#commitcomment-11057074>
> .
>


adding final is good
adding the type in the [] isn't needed if you want to bet on getting some decent type inference for generics.


It's purely a preference. You can use whichever one you prefer. In this case, by using the static inject property, it lets me bring in one less import. Also, for training purposes, it lets me delay teaching about decorators until further into the app-building process. There's no technical reason though and the framework still supports both. If you are using TypeScript, it's nicer since you can use `@autoinject`. There's also a Babel plugin in the works that would generate the inject decorator based on Flow types. So, you have some options based on what you like. Ultimately, all of these options just create the static property under the hood.


don


done


用新的 proxylib.py 替换旧版本的，运行goagent提示：syntax error。


style discussion point: given this is private perhaps just assume inference and write
final _PADDING_SIZE = 25;
alternately
const _PADDING_SIZE = 25;
although I don't really see the point of const vs final for int. doesn't really matter either way


Why is TraitSet.getValue(String taxonName) deleted? It is a much safer way of referencing a trait than by an integer.

> On 6/05/2015, at 10:55 am, Tim Vaughan <notifications@github.com> wrote:
>
> This commit seems to cause a build failure - Tree won't compile anymore due to its use of TraitSet.getValue(String taxonName), which is deleted here. The alternative, TraitSet.getValue(int taxonNr) was previously marked as deprecated.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/CompEvol/beast2/commit/051ec4092035167694ca4d019f7bb1d4d888d1aa#commitcomment-11055724>.
>


in  void card::shuffleDeck (int deck[i], int amount)
if we seed srand in main(), will we be able to use rand() in a class member function?
if we seed srand within the function, and have multiple calls to the function, it'll be re-seeding with the same time, because the cpu cycles so quickly. the professor mentioned something in class about 'sleep'? i didnt catch all of it.


Is this still the intended color?  It no longer matches the comment -- [muddy yellow](http://www.colorhexa.com/c7af42)


I think I miss something. Why do we not allow generator in ambient context?


lgtm overall modulo minor tweaks.
as an aside: I like how you didn't squash the commits for this CL. it made it easier to review than if they had been squashed.


if you set winOrLoss to either w or l, and have totalPlayer and totalDealer set to their final points value, i pass all of those into my Wager function, and it determines whether it was a tie or not, so you don't need to code anything for a tie, just update the dealer and player points


yeah. but then it is hard to read. I'll give it a shot and we'll see.


Light blue -- or intentionally [bright green](http://www.colorhexa.com/00c116)?


这个无视好了。。。


if the compiler is configured to not skip on errors then we will not get here. so you control this from the tree declaration file.


So... What happens if it isn't of `len(val) == 1` ?


Can you PR your time unit fixes when you get a chance?


这个也无视好了。。。


wager also checks if the player got exactly 21 points, since that pays out 1.5 times their bet


这个也无视好了。。


yeah, something to be revisited in all of our plugins. broccoli seems to generally prefer sync I/O. supposedly async doesn't buy you much in this scenario. I'm not sure if that's really the case, we should verify.

anyway, most of the time is currently spent in tsc. looks like 200-300ms when a single file is changed.


I think our code is fine.


I think `$e->getMessage()` is not enough to identify problems. `(string) $e` required to get error message file, line no with full stack trace.


How to add Catagories search in sale register?


Sorry about that - should be fixed by cbb95491f6237c2ba98ac55e18ee3dee58a56595


Thanks btw! super cool


This is just horrible


When did we add this? I have been asking for something like this for a long time now :)


line 39, do we still need to call `assertEventDispatchThread()`?


I usually put two blank lines between class/function/method declaration it makes the code easier to read when there are horizontal breaks between significant parts of the file. clang-format doesn't seem to mind


yeah


Right now you have it seeded in main but i dont think thats necessary.  I
thinks its better if we use it in the shuffle part of the member function.
Also we need
#include <windows.h> to use the sleep function. And the syntax will be

Sleep (1000);

The number in the sleep is the number of miliseconds. This will slow down
the for loop waiting a second to continue so it can be resseded every time
with the clock.

However 1000 miliseconds is really long because then the entire shuffle
will take about a minute to complete. So instead of a 1000 we might try
like 100.

He talked about this the time you werent there so no worries.
On May 5, 2015 5:41 PM, "goldfish" <notifications@github.com> wrote:

> in void card::shuffleDeck (int deck[i], int amount)
> if we seed srand in main(), will we be able to use rand() in a class
> member function?
> if we seed srand within the function, and have multiple calls to the
> function, it'll be re-seeding with the same time, because the cpu cycles so
> quickly. the professor mentioned something in class about 'sleep'? i didnt
> catch all of it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057203>
> .
>


why you do this?


@pranavkm just added it today in place of the the DNX packages. We probably should have just started with this :)


The error handler adds the file and line number where the message was generated. If you ask the exception object where the exception was raised, it will tell you it is from the error handling function itself.

This is what a generated error actually looks like in the report:
```
An error occurred during processing; checking has been aborted. The error message was: Undefined offset: 14 in .../PHP_CodeSniffer/src/Standards/Squiz/Sniffs/ControlStructures/ControlSignatureSniff.php on line 88
```


Good point. Maybe we can keep the future resolve in bay by setting up a promise for previous resolve that will be resolved when the templates are done added, this way we can then enforce ordering.


add script to MajorOperatingSystemVersion and MajorSubsystemVersion in PE header from 6 to 5


/posts.json show all posts why not?


Yay!! @imathis, how long would you expect for a porting guide from 2.0 to 3.0?


Everything seems fine. The comment I made applies to all updated sources. It's more a question really, I guess I would be okay with keeping the assert. Either way, just thought I'd point it out.


1000 x10^(-3) is 1 second, so sleep(1000) will work.
Sorry I missed the class. I studied that entire day for my physics test last thursday.


Parens here are unnecessary


It's too bad there's no native `chop` function in JS.


I guess there is no real use case for showing global streams AFAIK. Doesn't bother me, you seem to have a use for it.


Hmmm :)


Reordered the flow of the first 9 slides and some formatting.


done


now that i think about it, i need totalDealer and totalPlayer for the wager function, to check for blackjack and ties


Added "Setting up your projects" to walk learners through opening project files.


Why did you do this?


There is not dev or production env for plugings. These will run at the bundling time. So, this won't run inside production.

But, you can check whether what's the command uses this plugin by looking at `process.argv`


What files will be cleaned?
Will extra files (created by user) be deleted?


@dmilliken Could you double check the Typecasting slides? The second one refers to converting numbers to strings but the example after it shows the "1/2-0" example.


May be `bundle.once 'end', cb`


@yjun2 you understand the value and use of `+ (instancetype)` method declarations... if not let's cover it.


No worries. Yes it will work but it will run really slowly. He showed us in
class.

Also for the play i have removed my code for draw. However, in the wager,
you dont have a scenario if neither player reaches 21 or stays under 21 for
example:

Player : 19
Dealer: 20

The way its set up is if this scenario happens then the char value passed
is an 'S'. So if this happens it doesnt return who wins or the payout
 On May 5, 2015 5:56 PM, "goldfish" <notifications@github.com> wrote:

> 1000 x10^(-3) is 1 second, so sleep(1000) will work.
> Sorry I missed the class. I studied that entire day for my physics test
> last thursday.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057330>
> .
>


From my point of vision, call stack is important when you are dealing with unknown exceptions/errors/warnings/notices


I just repeated the previous variable assignment code block so learners could see the old value of x without going back to the previous slide. Let me know if that makes sense.


@yjun2 add `+ 1` to the end of all the calls that use `arc4random_uniform()` like this: `[self randomStringofLength:arc4random_uniform(10) + 1]`  This results in not getting a 0 as a possible value. We don't want the UI populated with zero length strings - ie. no strings.


So, I haven't researched setuptools-scm, but I'm curious what benefit it gives us over the way @jpadilla suggested.


Expanded step 1 to explain assigning a function as a value to a variable.


@yjun2 excellent solution.


这里想说的是dislike吧，unlike是不像


This is rather ugly, and adds a small amount of bloat to every directed object which currently has no `swig_override` array.

Why not just encapsulate this array in a helper class, with a constructor which does the `memset()` call?  Then the `memset()` is automatically used precisely when this class is, rather than having to match up the logic to emit the `memset()` with the logic which emits the `swig_override` array.


@yjun2 done with my review. press on.


Because it doesn't change the meaning of the signature from a consumer's perspective. It's basically an internal detail, similar to default parameters.


Only updated to old repos


jus like stick with more features and then we can pare back as we need


you are going to want to add a LICENCE and README.md to thes prohect :+1:


so on results 2 I do not display the majors anymore but you are correct in that the order of advisors stays the same. Should I include the majors on the second page? I'm not sure if the student will forget which advisors belong to which majors or if that matters anyway


hola como estas necesito que me ayudes con este codigo por favor en que parte del otro codigo lo ingreso gracias


I'm not printing a call stack in the error report and I'm not leaving the exception uncaught for the whole run to die. So this is the best I can do. Plus, I cant remember a time when the stack trace from a sniff warning was actually useful in diagnosing an issue.

If there is an error, you know what file and sniff was involved and you can debug it from there. Or I can if you choose to add a bug report, I'll do it for you.


It started to feel hacky that the way to get the type was to call checkIteratedType, and pass nothing for the error node, instead of just calling getIteratedType. Also this is for iterables. But for iterators there is never a checkIterator. So it felt asymmetric as well.


ASSERT(c->getIndexCatalog()->haveIdIndex(&_txn));


I think `\ErrorException` is exactly what we need for that context:
```php
public function handleErrors($severity, $message, $file, $line) {
    throw new \ErrorException($message, 0, $severity, $file, $line);
}
```


if Player : 19
Dealer: 20, doesn't dealer win, because he is closer to 21? that should change winOrLoss to 'W'... unless i am confused about how to play. i admit, i've never played blackjack before. i just googled it to make the wager function. it said if neither goes over 21, then whoever is closer to 21 wins


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1649 of 1649 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)


@3nids looks much better, thanks! A couple more notes:
- the default size is really tall, with a lot of empty space. Even when resized the title bar still takes up a lot of room and means there's a lot of wasted space above the widgets. I wonder if the title bar can be hidden to avoid this?
- Can the panel be hidden from the panels menu? Otherwise there's an action which has no effect there.
- A background colour would be great as well to make the widget stand out (like how the messagebar is coloured)


some thoughts, after thinking about it for a bit.

i primarily stopped using nix because it bought me very little over "daily backups of full system and /etc in particular" + "binary package history going back a few weeks" + gentoo, and mostly made normal maintenance quite a bit harder. i don't have enough fucks, and nix requires more than gentoo for me.

in gentoo, i have these problems with package management:

- in principle, only /etc (and /boot and some minor other stuff) has state. in practice, a bit of state creeps in and makes cloning a config, rolling back or just plain updating suck.

- dynamic linking bullshit silently breaks shit. (this is many times worse in no-type-system land. in my experience, at least half the python tools i use are silently broken at all times, in a way that a braindead type system would've caught.)

(on that note, why are we dynamically linking system packages why. this is particularly bad because of nix' extra indirection that slows stuff down (git-annex used to have *3 seconds* of delay on a ssd before they switched it pack to static linking) and how badly it interacts with different environments. they already rebuild everything on updates. dynamic linking is sin.)

- compiling everything takes time and is mildly annoying, especially on weaker hardware. long, messy builds because someone can't into stable are even worse. (glaring at all of haskell in particular).

i hoped nix would help with some of that. in principle, something like it should. however:

- no way to sanely pin versions besides "fork nixpkgs and maintain it forever because there's so little standardization" just like arch, or sadly anything that isn't source-based like gentoo. this shouldn't *inherently* be a limitation of nix' approach, but yeah. (this is the main reason i literally *can't* switch away from gentoo (or "build everything myself", i guess, which becomes more attractive every day). software breaks on me all the time and i regularly need to stop upstream from shitting down on me, at least temporarily. like, recent grep versions have been broken for me for months now. *grep*. this is not just "yet another bloat nightmare", this is *grep*. (inb4 "implying gnu isn't bloat") so a package manager that doesn't allow me to pin versions without pushing major maintenance burdens on me or locking me out of all updates, even mostly unrelated ones, is fundamentally unacceptable. rollback is nice, but mostly uninteresting, and restore from backup is fast enough for the one day a year something actually explodes.

-  nix in principle should help with long builds due to hydra, but in practice didn't. i need to maintain my own version of core things a lot (see above), so i mostly build my own packages. nix rebuilds everything a lot on updates, so i compile *even more* than with gentoo, nullifying hydra. at least sharing packages between machines worked nicely. (i don't really deploy anything, just share updates and rarely clone a setup, so i don't care much about easy deploy either way.)

- general linux problem of really terrible foundations that you can only cruft over and which makes everything suck. (yes, 600mb memory use for any operation at all is perfectly reasonable for my package manager, that will make me want to run it on a lot of stuff.) (yes, no attempt to standardize or sanely decompose your package tree will make future maintenance easy.) (yes, bash is an *excellent* foundation for anything.)

- different environments turned out useless for me. i've kinda come around to go's "one global set of libs" style whenever possible, and just locally vendoring a few libs when necessary. i don't find separate environments actually useful or worth the hassle (extra setup, needs dedicated shells, harder to coordinate different projects on the system).

i might revisit nix in the future, especially if someone brings in plan9-style fundamentalism. i really like fully deterministic anything, and found nix overall the second-least terrible package manager to use (minus "still young" problems i'm fine with at this stage). at least you *can* fork the package tree and rollback easily.

many of these problems aren't about nix' ideals, just the current iteration. it's mostly not revolutionary *enough* to be actually better on net than gentoo. that would probably require chucking way more backwards compatibility than nix is willing to do. that sucks, but linux is mostly shit, so not having access to most of it is not really a problem for me.


Arrays of multiple length shoudln't be rendered with a {{x}} mustache pattern, but {{#x}}{{.}}{{/x}}.

My understanding was this was just to handle the single item array edge case as any kind of contatenation would likely be wrong.   IOW, change the template.


Good point. Didn't consider that. I'll change it.

I get what you mean with the testing now. Takes more work, but it'll be there forever and provides consistent results. @radialglo


Isso claramente não funciona.


cleaned up some extra concatenation and added default value


Yup, sure was. Not sure how I missed that, but thanks for the catch!


and I can't spell -> commit message should read:
"Forgot to include 'quality' argument in BlueStar instantiation in Award class


i'm not sure, but i think the play function arguments will at least need (char& winOrLoss, int& totalPlayer, int& totalDealer) for arguments, because i need them for the wager function. if you have another way you'd prefer, i can adjust wager. right now i tried to calculate different win/loss scenarios in Wager , so that you wouldnt need to have a bunch of loops checking that stuff in your play function.


You can do that with any exception, including the custom RuntimeException. But then the error report contains a stack trace, which I don't want.


Would you be able to clear this up @alexpopinga ?


Synchronization Problem Solved


Got it. Thank you for the detailed explanation!


When i wrote 'production', I meant the build process for bundling the app for production. When that is happening I want to change the `debug` value to false.

I didn't think of checking `process.argv`. That's good thinking. I'll look into using that. Thank you.


This makes me happy. This was a pain.


Four builds.  The first build looks for `d_` options, then `e_` options.  The second looks for bare options, then `e_` options.  The third looks for `d_` options, then bare options.  The fourth looks for bare options, then bare options (but this is special-cased to be skipped, since it would not find anything on the second pass that was not found on the first).

You also need to set builddir for each of them or use at least one of `builddir_prefix`/`builddir_suffix` so that it is auto-computed.  I recommend never building in-tree.  My `site-local.py` has a default `builddir_prefix`, so I forgot to mention that you need one.  See [`SConstruct:DXXCommon.UserBuildSettings.default_builddir`](https://github.com/dxx-rebirth/dxx-rebirth/blob/9dc22b1ee6d7c372f340e3233cd410a807e69e0e/SConstruct#L1105) for how `builddir_prefix`/`builddir_suffix` generate a directory name.  Try:
```
scons d_debug=True e_editor=True dxx=d+,e+, builddir_prefix=build/
```


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


What do you think is best. Leaving it in main thw way it is or putting it
into a function. I think if we use a function we will have to return too
many things
On May 5, 2015 6:14 PM, "goldfish" <notifications@github.com> wrote:

> i'm not sure, but i think the play function arguments will at least need
> (char& winOrLoss, int& totalPlayer, int& totalDealer) for arguments,
> because i need them for the wager function. if you have another way you'd
> prefer, i can adjust wager. right now i tried to calculate different
> win/loss scenarios in Wager , so that you wouldnt need to have a bunch of
> loops checking that stuff in your play function.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057510>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.77](http://teamcity.monogame.net/viewLog.html?buildId=6981&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:13


UB Android SDK API Enpoint Progress


Careful -- for the OAuth 2 "implicit grant", the redirection should be putting these values (`access_token`, `token_type`, etc) after the `#` of a URL -- not in query parameters. See https://tools.ietf.org/html/rfc6749#section-4.2.2 for details.


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


Why do we have the condition here? Why some blocks are different than other?

Consider:
```C#
using System;

class B
{
}

class C : B
{
    public C()
        : base() // breakpoint here, drag IP to '{'
    {
        Action a = null;
        a = (Action)delegate { a = null; };
        a();
    }

    public static void Main()
    {
        new C();
    }
}
```


I could take care of that scenario but it would be downgrading the task for
your wager. After taking care of the victor i will determine your
winorloss. So you will only have to pass in the winorloss and the bet
instead. Which makes the wager a whole lot easier.

This would be more work for me but i feel like it would look cleaner
On May 5, 2015 6:28 PM, "Zubeir Said" <said.zubeir@gmail.com> wrote:

> What do you think is best. Leaving it in main thw way it is or putting it
> into a function. I think if we use a function we will have to return too
> many things
> On May 5, 2015 6:14 PM, "goldfish" <notifications@github.com> wrote:
>
>> i'm not sure, but i think the play function arguments will at least need
>> (char& winOrLoss, int& totalPlayer, int& totalDealer) for arguments,
>> because i need them for the wager function. if you have another way you'd
>> prefer, i can adjust wager. right now i tried to calculate different
>> win/loss scenarios in Wager , so that you wouldnt need to have a bunch of
>> loops checking that stuff in your play function.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/fishtez/blackjack/commit/69c60a6d16d9e18b81c303f14e7d04faf4239a1a#commitcomment-11057510>
>> .
>>
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.78](http://teamcity.monogame.net/viewLog.html?buildId=6985&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:55


cool, where can I find this course? from coursera?


@borromeotlhs, thanks for your contributions! Hey, do you think that you could do a pull request to fix this issue, as it may help potential users that would could miss this comment? I would, but I'm a total Github nublet.


I think that "that are" is not needed. So, "one of more files that are committed" -> "one or more files committed"


ASSERT(c->getIndexCatalog()->haveIdIndex(&_txn));