Where would you put it then?


Every object that does not have a team assigned (in normal levels, or objects that never have teams, for example resources) has object.team = 0


剩下的明早再看。


Do you need any help


pr-downloader is/will be used for the engine. the python thing can't be used there, so please spent no time into it / don't use it.


"by factory()" - copy-paste error :(


@varund7726 don't remove this
It'll cause errors
If you want remove the commit in settings for your builds, but let it be in the manifest


In explanation of the "type" parameter, probably under the word "research"


Since there is nothing stored in the TGenActionChooseData struct, we don't need to define it. Just remove these lines.


Again, just delete the line where we create the struct, and then just leave action->data alone (it will be NULL and thats ok).


Sorry for this. Should be fixed now!


This breaks everything for me.


Do we want these in even without `ENABLE_MODULE_SCHNORR`? If you run kcov without --enable-module-schnorr you get an angry red 0.0% for schnorr_impl.h.

It is only included from `src/secp256k1.c`; maybe this inclusion should be conditional?


@anandology -- these questions have a couple of interesting characteristics:

1. The questions require pretty much a full python environment. For example, we need to simulate sys.argv. We would soon want to test `fileinput` as well, which may require local files. Eventually, we'll need to test Pandas with CSV files, database modules with databases, etc. Need to keep these in mind when building the test environment.
2. The questions are dependent. That is, the second question *requires* the first question to have been asked immediately before, and answered. So we can't quite plan for a random question bank and use it directly.

Thought it would be useful to try to implement these (I'll add more questions) in the test framework we hav in your Working with files notebook.


alert()


donné*


Sans la majuscule, faut bien le bon nom de la table.


Au lieu de "De User", mets plutôt "enregistrés".


Sorry,
I committed all of these changes accidentally - please do ignore them!
I've rewritten most of it in a later commit


Sorry,
I committed all of the changes in this file accidentally at this point - please do ignore them!
I've rewritten most of it in a later commit


Change le type :


add tidy.txt


for the third time: the maximum clock frequency is 146MHz.


"Isère ou met à jour l'utilisateur donné en paramètre."


Also, added to print content of file to the screen if it exists or print a not found message if it doesnt


Typo: --no-foce-no-debug


It does not appear that this code is thread-safe, or meant to be thread-safe. You should therefore be able to derive a local type from LARGE_INTEGER and initialize QuadPart in the constructor. Might as well initialize it with QueryPerformanceFrequency directly, in that case.


Soit tu mets des espaces après toutes les virgules, soit tu n'en mets aucun, pas un mélange des deux.


Is there an off-by-one error on this line? Does `g_queue_peek_nth` take the index or the item number as the parameter? In other words, if you give a 0 index, does that return the 1st element? If you give a 1 index, does that return the 1st element or the 2nd?

If `g_queue_peek_nth` expects an index to be passed in (so, 0 is the 1st element and 1 is the 2nd), then we do have an off-by-one error: `g_random_double` may return the valid value of 1.0, and then your code will compute an index that is one element beyond the end of the queue.

Can we test the `g_queue_peek_nth` function to determine its behavior, because I don't completely understand the [API docs](https://developer.gnome.org/glib/2.32/glib-Double-ended-Queues.html#g-queue-peek-nth):

> g_queue_peek_nth ()
> Returns the n'th element of queue.
> n : the position of the element
> Returns : The data for the n'th element of queue, or NULL if n is off the end of queue.


I should pick that one on my (unsupported) egl-isengard, right?


@aktech I have removed this `line` in the `latest commit`. Since this line is never reached. Currently we are dealing with the `rational-functions`, because of which criterion to reach this `line of code` is `not achieved`.


> "script" instead of "headjs"

Cake view [does use script](https://github.com/FriendsOfCake/crud-view/blob/master/src/Template/Layout/default.ctp#L40)


I'll have to look into this on win32 but I'm pretty sure that the usage of `git` won't work like this.


@aktech I have removed this line in the latest commit. Since this line is never reached. Currently we are dealing with the rational-functions, because of which criterion to reach this line of code is not achieved.


C'est tout bon pour le reste ! On testera dès qu'on aura les controllers.


A case which doesn't quite do the right thing just yet ;)


Similarly this line has been removed for the `same reason` as line [here](https://github.com/gxyd/sympy/commit/12c5f496815e58de8dd56ba9a8ef2518f401ecc4#diff-1299a2752b6796eed94f8cb6510761f2L127)


A.k.a a bug


salam ostade bozorgvar,,, code shoma ro sare ma ja darad


Dynamite! When optimizing, compilers usually decide that `this == nullptr` is always false!


Gotcha.

I didn't stop and double check how in sync the source repo was with your fork.  Sorry about that.

Both of repos could use a bit of a 'quick start' on their README.  Sounds like this quick start is only relevant to your fork.

You might consider adding an expanded repo to helps those that find your tool through a search instead of a blog post explaining how to use it.


If you have a better fix, please change it. Previous version used `return this` which raised exception when `NULL`.


Oops, yes :) Will merge this later, sorry


I'll have a look at it.


@lorenzo what was the reason for putting all the js in the head ...?

This will be [the cause for confusion here](https://github.com/FriendsOfCake/crud-view/commit/96dc2c2df979cf18cd9b4d6dde8fa13d3471c0f1#commitcomment-12358964), and undoes the good-intentions of the crud view always putting the js as the foot of the page, by default.


Line 45 breaks the build because this does not adhere to standard code style (extra semicolon)


We're taking inspiration from ActiveSupport here. But also, loudly failing and letting the developer know when it comes to helpers IMHO is better than silently failing/producing a wonky output.


Arf


I have used here `solveset_real` instead of `singularities` since. In singularities expressions like
```
>>> expr = (x**2 - 1)/(x**3 - 1)
```
are first simplified to `(x + 1)/(x**2 + x + 1)` which is `wrong` for finding the `range`. I have added the `test case` [here](https://github.com/gxyd/sympy/commit/12c5f496815e58de8dd56ba9a8ef2518f401ecc4#diff-169421b4b53ba8be36041f07899de1afR45) which was previously giving wrong result.


@pdion891 were these PRs merged on master?


So, your initial expectation was that the function would return the top-priority-tech / highest-priority-tech, or the-priority-of-the-current-top-priority-item or something like that?  If that had been the intended purpose of the function then I would object to the name "priority_one" as misleading.   Also, such functions simply wouldn't be compatible with how this function is used.  Sometimes we just have to accept we had a moment of confusion without blaming the function name :smiley:

So, I simply can't agree that "priority_one()" returning a priority value of 1.0 is misleading.  Though I do need to fix the docstring as you noted.


This causes all sorts of problems :(

Shouldn't it be:
`new (...args: any[]): any;` at least?

Everyone now ends up with compile errors/warnings like...
```
WARNING in ./someClass.ts
(29,24): Argument of type 'typeof SomeClass' is not assignable to parameter of type 'Type'.
  Type 'SomeClass' is not assignable to type 'Type'.
    Property 'apply' is missing in type 'SomeClass'. (2345)
```
When doing something like...
```
bind(SomeClass).toClass(SomeClass)
```

Please help here.


Maybe you a break statement should be added here, so the intention is more clear, and also that would add a little optimization.


Sure. That will be excellent


k


@nodiscc could you please keep the empty last line – in any file


only needed for bypass renderer and systems which have vsync disabled


Argh, forgot to change indentation...


@j4yk how about this?


@thelgevold Your error above is not related to router typings but rather changes to angular2 typings and the flact that `toClass` now expects of type `Type` which I don't quite understand.

@alexeagle Could you please help shed some light on this.
This commit seems to be the culprit as to why the latest 32 typings are causing these issues:
https://github.com/angular/angular/commit/19d8b221b4a4cae952a8120fd281364940bfaa36


Oh, yeah. The new save system, can also save arrays now, even though the whole system is disabled by default.


Лучше заменить на List<Color>, создать класс Color


aku bingung nag bagian insert e bang..help yahh


Dit werkt goed!


They should have been removed. They are now replaced by a robot test.


En concret, amb Ubuntu sempre és millor agafar versions Long Term Support (LTS) para treballar. Així estàs tranquil perquè tens cobert el manteniment del sistema operatiu i les actualitzacions de seguretat fins al 2019 (gratis!).

Després, dins d’Ubuntu mateix hi han diferents “sabors” segons el tipus d’ús que hagis de fer, o en aquest cas, para la màquina on anirà instal·lat. Xubuntu és caracteritza per fer que el sistema operatiu sigui el més lleuger possible i no consumeixi gaires recursos de la màquina. Això també ho fan para reduir l’impacte de les grans marques obstinades en fabricar productes nous cada 6 mesos… (cal?)

Sort!

------------------------------
David Romaní
636 963 301
http://www.flux.cat

> El 25/07/2015, a les 14:52, Anton Serra <notifications@github.com> va escriure:
>
> Sí, tinc la possibilitat de fer ho!
> Està bé que m'hagis recomanat la versió perquè mhavia descarregat la
> ubuntu15.04.
>
> A vore si me'n surto.
>
> Anton Serra
> El dia 24/07/2015 17.14, "David Romaní" <notifications@github.com> va
> escriure:
>
> > Si tens la possibilitat de matxacar una màquina antiga millor, et recomano
> > una distribució Linux tipus Xubuntu http://xubuntu.org/ <
> > http://xubuntu.org/> 14.04 LTS, és com una Ubuntu normal i corrent però
> > pensada para màquines amb pocs recursos. Instales des del CD que et pots
> > baixar, és lo més fàcil perquè allí mateix ja pots formatejar tot a sac.
> >
> >
> > Sort!
> >
> >
> > > El 24/07/2015, a les 16:18, Anton Serra <notifications@github.com> va
> > escriure:
> > >
> > > Tranquil David, fes la teua. Ja m'aniré aspavilant.
> > > De fet ara estava mirant això del Linux i la idea que em dones
> > > d'instal·lar-lo en un pc antic és molt bona perquè en tinc un amb windows
> > > xp.
> > > Ja aniré fent provatures.
> > >
> > > Així, si no ens veiem abans, ho fem el 8 d'agost.
> > >
> > > Gràcies!
> > >
> > > 2015-07-24 15:49 GMT+02:00 David Romaní <notifications@github.com>:
> > >
> > > > Ho sento però tinc molta feina endarrerida, el tema de les obres m’està
> > > > descolocant totalment. Em sap greu, però haurem de quedar més endavant
> > fins
> > > > que tingui més controlada l’obra del coworking. Ja t’avisaré.
> > > >
> > > > Si encara no has aconseguit configurar Apache amb Windows, continuo
> > dient
> > > > que amb Linux és molt més simple. Si no vols instalar el dualboot (hi
> > ha
> > > > risc de que acabes borrant el disc dur sencer si no ho has fet mai) a
> > > > l’ordinador que tens, l’altra opció és buscar una màquina antiga.
> > > >
> > > > Linux es pot instalar millor amb ordinadors antics amb poca RAM i disc
> > > > dur, per norma general, consumeix molt menys recursos.
> > > >
> > > >
> > > >
> > > > Sort!
> > > >
> > > >
> > > >
> > > >
> > > > ------------------------------
> > > > David Romaní
> > > > 636 963 301
> > > > http://www.flux.cat
> > > >
> > > > > El 24/07/2015, a les 11:24, Anton Serra <notifications@github.com>
> > va
> > > > escriure:
> > > > >
> > > > > Hola David,
> > > > >
> > > > > Aquesta tarde estic per Amposta, però fins les 18 h.
> > > > > Si t'interessa que quedem, em dius algo.
> > > > > Sinó ja et diré de quedar la setmana vinent a l'hora que normalment
> > > > quedem.
> > > > >
> > > > > Parlem.
> > > > >
> > > > > 2015-07-14 8:37 GMT+02:00 David Romaní <notifications@github.com>:
> > > > >
> > > > > > Yes, but try to config Apache web server before. You can find some
> > > > hints
> > > > > > here: http://symfony.es/documentacion/
> > > > > >
> > > > > > Especially here
> > > > > >
> > > >
> > http://symfony.es/documentacion/como-configurar-bien-apache-para-las-aplicaciones-symfony2/
> > > > > > and here
> > > > http://symfony.es/documentacion/guia-de-instalacion-de-composer/
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > >
> > https://github.com/aserratorta/Symfony-Training/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12148750
> > > > >
> > > > > > .
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Anton Serra Torta
> > > > > 616.50.13.32
> > > > > —
> > > > > Reply to this email directly or view it on GitHub <
> > > >
> > https://github.com/aserratorta/SymfonyTraining/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12339322
> > > > >.
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/aserratorta/SymfonyTraining/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12343561
> > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > Anton Serra Torta
> > > 616.50.13.32
> > > —
> > > Reply to this email directly or view it on GitHub <
> > https://github.com/aserratorta/SymfonyTraining/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12344149
> > >.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/aserratorta/SymfonyTraining/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12345351>
> > .
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/aserratorta/SymfonyTraining/commit/e549f641479e46be29f7dab198fa76c1596bb0bf#commitcomment-12357059>.
>


Register commands dynamicaly its better. Look at FactionsPE at http://github.com/randomaltthing/pocketmine-mp-plugins in the commands folder. Look at the files BaseCommand and FactionCommand or FCommand or whatever. NOT the FCreate file. You can copy and edit if you like. Then in the main class add


Лучше заменить на `List<Character>` создать абстрактный класс `Character` и по классу для характера


i'm an idiot


+1 Will indent it in my next commit


Ah, I think I just realized about perhaps what you "really" meant-- it could be that 'ambiguous' rather than 'misleading' is the word that would have conveyed your meaning better.  I could understand the view that the function name is (very) slightly ambiguous, though frankly that is possible for nearly any name.  In this case I think that the (corrected) docstring and the manner of actual usage should clear up any ambiguity just fine, but I guess we could consider changing the name to "const_1"


+1 Will add it in my next commit


Лучше заменить на класс `Message`


OK, so clicking "Add WHERE Clause" once would default to displaying the latest 24 hours of data, and clicking it again would still show latest 24 hours of data? I like the idea of having reliable defaults so I will probably implement that suggestion, thanks.


Okay thx.


Then that should probably be backported for those to v15?


Told you something was missing in 7b44fa9d916c875f1e093963dff4c815bb67a3c5 ^^


Lines 9 to 15
This object has been moved and renamed from SnackObject to Snack.  Including you so when you pull, you will be aware of the changes @jessnev4life


Yep yep, exactly. :+1:


`tcp_is_cwnd_limited()` needs the real `in_flight` variable in 3.10, otherwise it breaks congestion avoidance.

Note that the 3.10 prototype for cong_avoid was:
`cong_avoid(struct sock *sk, u32 ack, u32 in_flight)`


@Jalle19 I was thinking S01E00 for pre-pilots and/or S00E0x for 'specials'. S00E00 just means 'not applicable'
Certainly I can see this warrants an explanatory comment. If you don't think this is sufficient, I'm open to suggestions. (The problem also applies to the EPG_TAG as this is where I got the fields from)


@NathanWalker Thanks for the clarification.
I looked at the source and I don't see any unit tests or examples using the HashLocationStrategy. Instead it's currently using a MockLocationStrategy in unit tests.
@jeffbcross @alexeagle: Curious if this means there will be a change to the bootstrapping or simply a bug?


I'll look at the toClass(Type) issue first thing tomorrow.
For the router see https://github.com/angular/angular/issues/3282

On Sun, Jul 26, 2015, 9:57 AM Nathan Walker <notifications@github.com>
wrote:

> @thelgevold <https://github.com/thelgevold> Your error above is not
> related to router typings but rather changes to angular2 typings and the
> flact that toClass now expects of type Type which I don't quite
> understand.
>
> @alexeagle <https://github.com/alexeagle> Could you please help shed some
> light on this.
> This commit seems to be the culprit as to why the latest 32 typings are
> causing these issues:
> angular/angular@19d8b22
> <https://github.com/angular/angular/commit/19d8b221b4a4cae952a8120fd281364940bfaa36>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/borisyankov/DefinitelyTyped/commit/f6c8ca47193fb67947944a3170912672ac3e908e#commitcomment-12362028>
> .
>


+1 will add it in my next commit


+1 Will indent it


+1 Will rename them in my next commit


@sindresorhus just letting you know this reintroduced the scrolling bug  #27


yep:)


+1 Will add the comments in next commit & will update the code


It's not necessairly a bot. It could be a building.


+1 Will add the comments


I will change the if from index.php and put all on init.php. thanks


looks good, not as surprising as before :)


+1 The customer bean object is created inside this method when the values are set for it


Is this necessary? I would imagine you would only inherit from this class if this were true.


These hooks do not go there, these event are in /boot/rules/70.build_posts.php


@Divran The latest (1.3.4) isn't AFAIK. 1.3.0 (from this commit) seems to be.


this file can be placed into a current directory.  The syntax is this:

1. $JOHN/john.conf is loaded  (or -pot=john.conf)
2. at bottom of john.conf is .include '$john/john-local.conf'
3. it is alright if that file does not exist, the ' ' does not warn
4. finally .include './john-local.conf'
5. again, no problem if the one in the current directory fails.

So now a user can create a john-local.conf in the $john directory, and it will be used for all runs of john. Then create a john-local.conf in his working directory, and override everything (including the 'global' $JOHN/john-local.conf)

That is why there is no directory specified

I left the deprecated run/john.local.conf file in the .gitignore


Can you takeout the change on file en_US because is older than the new files uploaded.


Please add `--max-count=1`


I think we could make this more flexible: (not tested)

`sed --regexp-extended -e 's/^.*<!-- +HTMLTITLE +(.*) +-->.*$/\1/g'`


Yeah! Do you mind taking a screenshot with your great colors? @sindresorhus


This formula allows to compile Octave 4.0 on OS X Yosemite. However there still some minor issues (bugs in the gui and osmesa). I am glad for any help. @dpo @mcg1969.


Are you going to submit a pull request for this?


Tif-tof?


I don't hugely object to this as a selector, although it's a little fragile in my view (changes to the HTML could so easily make this over-broad). However I'd like to see a comment in the SCSS source explaining what it does, ie what `a[rel]` is selecting compared to just all `a` within `.original-transcript`.

(And then I'll probably want `rel="alternate"` on the non `next`/`prev` anchors, at which point we'll have to fix the fragility ;-)


Is reported here => https://github.com/Fred89/bludit/blob/master/languages/fr_FR.json ;)


Why the 500000 latch value? I dont see both start and end frames for APA102 LEDS (read https://cpldcpu.wordpress.com/2014/11/30/understanding-the-apa102-superled/). Where are them? With these, i always ged all LEDS working but the last: i think it's because of the latch value.


This is beginning to look like an ordered list. Given that at the moment it's just naked phrase elements inside a `div`, that would probably be nicer semantics. Not important, though.


@zml2008 What happened to `getEntity()`?


+1 Actually I started building this assignment using Spring MVC where I was using only one Controller class i.e Teller. But when we asked about whether to build this application using Spring then Vaishali said that dont use Spring here so I build a separate teller for each corresponding jsp. Will update the code.


Just a question: Shouldn't `checkFlip()` be called here, too?


+1 Will complete the assignment with updations by monday afternoon


@alexeagle This may help (not 100% certain as maybe other code needs to change too? If so, maybe pull that PR and update):
https://github.com/angular/angular/pull/3300


Hmm.... and what happend if `options[:time_zone] = 'abc1'` ?


I don't see a requirement in the code to order the entities... It works fine when they are un-ordered


Конфликт после добавления
<code>{$oPhoto->getDescription()}</code> in textarea
(страница не подгружается полностью)
Да и нужно ли это тут?


+1 Other source codes are present in the src directory. Created a One to many mapping using set. Sign Up & login are complete in this hibernate example


How would one check now whether these extensions are available with ContextCapabilities?


Is this where you're saying the name isn't showing up?

If my name is showing up when you go to the show page for my user, but yours isn't when you're on your show page, then the problem is in the data, not the code.  Have you checked to confirm that both users have a value for `@user.name`?


Also: stylistic updates to changelog output


Hello, this line add a compilation bug in test libraries:
```text
Scanning dependencies of target minimal
[ 63%] Building CXX object tests/minimalbinding/CMakeFiles/minimal.dir/minimal/minimal_module_wrapper.cpp.o
[ 63%] Building CXX object tests/minimalbinding/CMakeFiles/minimal.dir/minimal/obj_wrapper.cpp.o
.../Shiboken2/tests/minimalbinding/minimal/obj_wrapper.cpp:7:18: fatal error: QDebug: Aucun fichier ou dossier de ce type
 #include <QDebug>
                  ^
```
One possible fix is to add in "tests/CMakeLists.txt"
```cmake
find_package(Qt5 REQUIRED COMPONENTS Core)
include_directories(${Qt5Core_INCLUDE_DIRS})
```
But I am not sure we want to force Qt dependency here.


I have removed all NV program extensions (i.e. assembly shaders), under the assumption that no one will use them. I can easily add them back if someone asks.


* clearify = clarify ...


Thanks for it!


I don't know, ask Lwmte. That's exactly how it was before the refactoring. I just restored it to the previous working state.


Ah, I see. Probably to reduce jar size? Personally, I am fine with the removal. Just thought you planned another mechanisms than this Kotlin file to check extension availability.


Done.


Also, I would have liked to keep ScrollToActive to work even with SeriesList disabled, which is definitely possible however I couldn't get it to work, it wouldn't correctly scroll to its position so i just left it completely disabled for now if SeriesGrid is disabled.


Note that the commit message is wrong - the correct version number is "0.3.1".


Having different behavior for one field depending on the value for another field feels a bit "magic" to me. I'd prefer using `-1` for N/A, preferably indicated by a constant/define somewhere.


Spelled canola wrong


Chapter 03 - Sorting Algorithms/BubbleSort2.Tests/BubbleSortTests.cs

Yes it is - will do it. Still working.


See #1


Im gonna start importing the models/animations on karthus/veigar, il leave kassadin for you since you said you wanted to try it yourself but il get started on the others to keep progress going forward.


You have this weird sixth sense O.o. I'm about to commit kassadin model as
we speak.

On Sun, Jul 26, 2015 at 6:39 PM, Zarthbenn <notifications@github.com> wrote:

> Im gonna start importing the models/animations on karthus/veigar, il leave
> kassadin for you since you said you wanted to try it yourself but il get
> started on the others to keep progress going forward.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coldAmphibian/Defense-of-the-Legends/commit/bef43bf06c2b199aeae98de5b7846df19c21a687#commitcomment-12362172>
> .
>


Конечно же нет, исправил - 494127cfe46ac33bd7af49e3401b913c69040d76


El ambiente de trabajo se puede seleccionar ?


Introduction
This second programming assignment will require you to write an R function that is able to cache potentially time-consuming computations. For example, taking the mean of a numeric vector is typically a fast operation. However, for a very long vector, it may take too long to compute the mean, especially if it has to be computed repeatedly (e.g. in a loop). If the contents of a vector are not changing, it may make sense to cache the value of the mean so that when we need it again, it can be looked up in the cache rather than recomputed. In this Programming Assignment you will take advantage of the scoping rules of the R language and how they can be manipulated to preserve state inside of an R object.

Example: Caching the Mean of a Vector
In this example we introduce the  <<-  operator which can be used to assign a value to an object in an environment that is different from the current environment. Below are two functions that are used to create a special object that stores a numeric vector and caches its mean.

The first function,  makeVector  creates a special "vector", which is really a list containing a function to
1. set the value of the vector
2. get the value of the vector
3. set the value of the mean
4. get the value of the mean
makeVector <- function(x = numeric()) {
        m <- NULL
        set <- function(y) {
                x <<- y
                m <<- NULL
        }
        get <- function() x
        setmean <- function(mean) m <<- mean
        getmean <- function() m
        list(set = set, get = get,
             setmean = setmean,
             getmean = getmean)
}

The following function calculates the mean of the special "vector" created with the above function. However, it first checks to see if the mean has already been calculated. If so, it  get s the mean from the cache and skips the computation. Otherwise, it calculates the mean of the data and sets the value of the mean in the cache via the  setmean  function.
cachemean <- function(x, ...) {
        m <- x$getmean()
        if(!is.null(m)) {
                message("getting cached data")
                return(m)
        }
        data <- x$get()
        m <- mean(data, ...)
        x$setmean(m)
        m
}

Assignment: Caching the Inverse of a Matrix
Matrix inversion is usually a costly computation and there may be some benefit to caching the inverse of a matrix rather than computing it repeatedly (there are also alternatives to matrix inversion that we will not discuss here). Your assignment is to write a pair of functions that cache the inverse of a matrix.

Write the following functions:
1.  makeCacheMatrix : This function creates a special "matrix" object that can cache its inverse.
2.  cacheSolve : This function computes the inverse of the special "matrix" returned by  makeCacheMatrix  above. If the inverse has already been calculated (and the matrix has not changed), then  cacheSolve  should retrieve the inverse from the cache.

Computing the inverse of a square matrix can be done with the  solve  function in R. For example, if  X  is a square invertible matrix, then  solve(X)  returns its inverse.

For this assignment, assume that the matrix supplied is always invertible.

In order to complete this assignment, you must do the following:
1. Fork the GitHub repository containing the stub R files at https://github.com/rdpeng/ProgrammingAssignment2 to create a copy under your own account.
2. Clone your forked GitHub repository to your computer so that you can edit the files locally on your own machine.
3. Edit the R file contained in the git repository and place your solution in that file (please do not rename the file).
4. Commit your completed R file into YOUR git repository and push your git branch to the GitHub repository under your account.
5. Submit to Coursera the URL to your GitHub repository that contains the completed R code for the assignment.

Grading
This assignment will be graded via peer assessment.


Why not? Shiboken is already Qt dependent.


Lol, so I assume everything went by without any errors. Some others were having issues with importing, but its good that some got by without any problems.


I'm trying to reduce "noise" from stuff that very few (if any) people will use. For example, I have not included any non-ARB extensions that have been moved to the core up to OpenGL 2.0. That was an arbitrary threshold that I thought made sense. I also haven't included very specialized (NV_video_out/capture) or advanced (NV_x_program) functionality.

I have now made a tool that automatically generates kotlin templates from the OpenGL registry SVN repo. This allowed me to validate the current templates and also easily add any extension that was missing (see the recently added EGL support, OpenGL ES is coming soon).


However it is also possible to auto generate the loading functions directly from the schema which would clean up the ObjectMgr massively.


Extra space between `_cb` and `(`


I don't think `user_data` needs a cast here.


Missing space between `if` and `(`.


The formatting is weird here compared to how we usually indent long functions.


You can pass `-1` and get `gtk_clipboard_set_text` to call `strlen` for you.


ohh


Not sure if it's just how Github renders it, but it seems to be indented with spaces instead of tabs.


Actually I need you help with him before that. I can;t seem to get rid of Anti-Mage's weapons and clothes. They just sort of hover around him...

I've added the bit about wearables on his kv file but they stay around.


This did lose the cool number-pad entry method on mobile, but life will go on. Somehow.


That doesn't work:
```
OpenTomb: src/character_controller.cpp:278: static void Character::getHeightInfo(const btVector3&, HeightInfo*, btScalar): Assertion `rs->sector_above != nullptr' failed.
Aborted (core dumped)
```


Thanks Steve


simply use
cards.addAll(player.getLibrary().getTopCards(game, rippleNumber));


Fixed in 2c423bf


Better move thi smethod before apply so it's easier to check if all needed vars are set.


The commit message for this isn't very good, it should tell what the change does, not which file it touches. Other than that and the trivial cosmetic issues commented on inline, it looks ok to me (not tested for compiler warnings or if it works yet).


should be fixed in 553a5f4a31a0d2cf26bfb8b996f7f9f86b6a173c


@Jalle19 - the problem with the event ID in PVR_RECORDING it that it can only refer to one EPG event.

Given the excessive number of repeats on TV these days (and the number of +1 and HD simulcast channels) it is quite common for a show to appear 5 or more times within available EPG data, even on a single channel.
On my mythtv backend I have three copies of most channels with their own channel identifiers (DVB-S / DVB-T / DVB-T2 sources) plus there are usually +1 and HD channel duplicates. Each of these would be another EPG id, so at least 5*3*3 or 45 EPG entries for a typical completed recording without breaking a sweat.

strRecordingId is declared in PVR_RECORDING as a 'unique ID on the client' and as such seemed the best way of identifying an EPG entry as already recorded.

Possible alternatives:

1. Generate 'pseudo recordings' for each of these duplicate entries - this just seems wrong.
2. Make the eventID a list (say covering the first 100 EPG entries).

A similar issue exists for timers.
Here it seems slightly less wrong to use duplicate timer entries with a 'duplicate of upcoming recording' state. The scheduler for a mythtv backend already provides these along with the 'active' ones so it would be relatively simple to generate entries like this. I expect other backends will do similar.

Alternatively the PVR_TIMER Event ID could be made a list here as well.

I don't have strong feelings about which solution would be best (one of these or something else), but this would be a cool feature and the current API isn't adequate to support it properly imo.


Are the values LIB_PATH and BIN_PATH standards? Or set by autoconf/make? They are not defined on my system.


What's this? :grimacing:


I'm guessing nobody is monitoring this thread anymore.  It's been over a week and I haven't heard anything from anybody.  I guess this doesn't work?


Did not work!


This is slightly better, but this doc was awefully wrong, it's not a line at all :) (string was copied from the Line class i think), the point instruction will actually place one quad to each point in the list of points. the width/height of the quad will be pointwidth.


Just get an older guardian to do it. When i went to Boston, my brother did it for me. So im all good

Sent from my iPhone

> On Jul 26, 2015, at 12:57 PM, "KCDagger" <notifications@github.com> wrote:
>
> I'm guessing nobody is monitoring this thread anymore. It's been over a week and I haven't heard anything from anybody. I guess this doesn't work?
>
> —
> Reply to this email directly or view it on GitHub.
>


I changed my user ID since then, I'll update to yours if you merge the PR


just a ghost - probably leftover from early I don't remember anyhow


TargetPointer is never equals null if not explicit set to null. Only targetPointer.getFirst(game, source) can be null.

Ripple always works, also if the triggering spell was countered or exiled.
So you have to use the last know information in that case.
http://www.mtgsalvation.com/forums/magic-fundamentals/magic-rulings/magic-rulings-archives/309339-counter-versus-ripple


Probably yeah. Nothing is published on cocoa pods specs repository though, I'll let you handle that


This reads another line, must be removed now that you're reading in the `while` condition.


Jason, for what reason you did not put this function into  H. object?
what's the purpose to have a Framework Helper?


> Out of curiosity, why is --disable-idn only done when doing the default build?

That would have likely been carried over from when the two formula were merged into one. I essentially preserved the current builds for everything as they were before, as much as possible, to avoid surprising people with new dependencies. I'm fine with someone changing it, if they want to.

>  Looks like including it only for the --without-gui case is relying on the transitive dependencies in the GUI libraries to provide it.

This isn't unusual for GTK+ stuff. GTK+ has such a large dependency tree that it drags in enough that explicitly declaring them all can be overkill. It's declared with the non-gui build because there's no GTK+ dependency to call Pango which calls Glib.

Likewise, if someone wants to do some combination of dependencies here to simplify the formula, I've no objection.


So no event needed as long there are no card sthat need that event.


This was a request by scarecrow420 the WMC addon maintainer (not sure of the github login) and also something I asked @ksooo if we could have before the original series recordings PR went in (for mythtv).

In the end we squeezed it into Lifetime for mythtv, but apparently it doesn't work adequately for wmc, so there are at least 2 addons which would benefit.
(See http://forum.kodi.tv/showthread.php?tid=227026&pid=2061355#pid2061355)


nitpicking: i think i would have written this as a one-liner like this:
`f_open(&file_object, (char const *)file, (replace == REPLACE ? FA_OPEN_EXISTING : FA_CREATE_NEW) | FA_WRITE);`

But just a matter of taste.


do you have the hero id line and the enabled line aswell? Those are important


What's the purpose of this line? Wouldn't this be optimized/clear out by the compiler?


yes, if no kernel support or misconfiguration - nothing happens.


Yeah was the ID missing, had edited my previous comment.

What are we going to do when we hit 128 though...I think that's the cap.
There must be a way of completely removing the previous wearables right?


Be carefully with using the getCard method. Because it will return null for tokens.
But also tokens will trigger Warped Devotion, so I guess you don't need to check here if it's a card.


Because this is (namespaceless) C header, i personally would have taken something like `DBB_FILE_EMPTY` (or `FILE_EMPTY` `FILE_REPLACE`).


nice one!!


If you set the text of an ability you have to add the dot on the end.


Do not use this annotation style.


To fix Range Error I had to use the blocking method-- readdirSync. Is this necessary. I cannot have nested synchronous methods..? Is this what promises are for or ... no?


No, we are using plymouth.
There is a misconception that the splash setting needs to be there for
plymouth to run, splash refers to the grub splash image, that if your using
one would be stored under /boot/grub/splash.jpg. This is a splash screen
like windows 95 had. This has nothing to do with plymouth actually. Anyway,
I haven't uploaded my latest profile changes yet, I'll do that know.

PS. Do you know why on startup you can hit esc or home to get the boot txt,
but can't return to plymouth? the strange this is this works on shut down.
Personally, I think maybe plymouth is being loaded before the driver or
filesystem it needs to see it's own theme directory.

On Sun, Jul 26, 2015 at 12:17 PM, Philip Müller <notifications@github.com>
wrote:

> So you don't use plymouth anymore. If so, you can get rid of 'quiet' too,
> to get the classic text mode back.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/netrunner-rolling/manjaro-tools-netrunner-overlays/commit/31c0918f8ab8c69e3400e4018497cb0d9a46e0d6#commitcomment-12361892>
> .
>


ubuntu does not provide a package dcadec-dev. if this requires team-xmbc ppa it should be mentioned.


I'm already working on it submitting it myself :)


Will the more granular cluster levels help with https://bugzilla.gnome.org/show_bug.cgi?id=541608?


@psiroky why do we exclude descriptor that actually defines handlers and listeners used by workbench for jbpm runtime? Moreover why we leave it only in EAP distribution?


https://github.com/bharathns/opensource/blob/master/cppdays/QueueSyncAsync/QueueSyncAsync.cpp

This is the client code
and this is the actual class (bit involved if you want understand everything)
https://github.com/bharathns/opensource/blob/master/cppdays/QueueSyncAsync/QueueMgr.h


it should be "blue creatures and black creatures", your text can be read with the different meaning.


Putting splash in the grub line is only so there is no quick display of the
txt console between loading the kernel and plymouth starting, instead you
would get just a black screen.

On Sun, Jul 26, 2015 at 2:09 PM, James Kittsmiller <
jameskittsmiller@gmail.com> wrote:

> No, we are using plymouth.
> There is a misconception that the splash setting needs to be there for
> plymouth to run, splash refers to the grub splash image, that if your using
> one would be stored under /boot/grub/splash.jpg. This is a splash screen
> like windows 95 had. This has nothing to do with plymouth actually. Anyway,
> I haven't uploaded my latest profile changes yet, I'll do that know.
>
> PS. Do you know why on startup you can hit esc or home to get the boot
> txt, but can't return to plymouth? the strange this is this works on shut
> down.
> Personally, I think maybe plymouth is being loaded before the driver or
> filesystem it needs to see it's own theme directory.
>
> On Sun, Jul 26, 2015 at 12:17 PM, Philip Müller <notifications@github.com>
> wrote:
>
>> So you don't use plymouth anymore. If so, you can get rid of 'quiet' too,
>> to get the classic text mode back.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/netrunner-rolling/manjaro-tools-netrunner-overlays/commit/31c0918f8ab8c69e3400e4018497cb0d9a46e0d6#commitcomment-12361892>
>> .
>>
>
>


Why is the synonym more important than the symbol itself?


Why is the synonym more important than the symbol itself?

I think the line above is missing `symbolExact^20`.


Por enquanto eu acho que está ok. O ideal é termos esse texto em algum documento dentro de uma pasta `docs`. Mas como ainda estamos bem no começo, acho legal deixar essa msg. Depois passamos para algo mais técnico que pode ajudar quem queira colaborar.


Mets plutôt "protected", ça permettra d'accéder aux attributs si on fait de l'héritage.


Not all the fields you would want for a propper 'backend data' scraper are there at the moment.

The idea was to re-use the existing EPG structure to get at some of the missing data which a videodb scraper might want (actor/director etc...).
@ksooo 's most recent suggestion was actually to add extra fields to the PVR_RECORDING tag.
His reasoning was that as actual EPG data records don't usually hang around for long in a backend, providing it with the recording (how the backend would store it) makes sense.

The 'scraper support' change was to provide just enough info for ttvdb / imdb scrapers to work.
This change was intended to add the rest of the backend's data to 'fill in the gaps'.


https://github.com/TrinityCore/TrinityCore/commit/455df6e498791b27c789918f0cd92434b835fb7f

spawning done - note that there are some double spawns. since i'm not shure if they must be spawned because of phasing i let then there - feel free to remove them


spawning done - note that there are some double spawns. since i'm not shure if they must be spawned because of phasing i let then there - feel free to remove them


À rajouter :
```PHP
/**
 * @var TypeUser
 */
protected type;


Pour la phpdoc, c'est :
```PHP
/**
 *
 */
```
Avec bien deux astérisques après le premier slash, pas une seule :P


other related data, that is useful for you maybe: https://gist.github.com/Rushor/641873742fd8da3201e8


Shouldn't this be a responsibility of a controller-view? This makes the component not re-usable.


I will fix this.
I used the quick fix "implement methods" so the issue should come from the formatter used by Intellij.


Valeu!


I'm curious, what was the problem with extracting the version from `__init__.py`?


Remplacer
```PHP
$this->extTypeUtilisateur = TypeUtilisateurDAL::findById($extTypeUtilisateur);
```
Par :
```PHP
$this->type = TypeUtilisateurDAL::findById($extTypeUtilisateur);
```
Qui correspondra bien à l'attribut de User rajouté vis à vis de mes commentaires sur le commit bd93abc498b791d3948b9b7b2c13aabeb7be89c6


WoW!! YOU ARE ROCK!!!! :+1:


ok


touch HelloWorld.md ## This is a mardown file


Remplacer
```PHP
$this->extTypeUtilisateur
```
Par :
```PHP
$this->type
```
Qui correspondra bien à l'attribut de User rajouté vis à vis de mes commentaires sur le commit bd93abc498b791d3948b9b7b2c13aabeb7be89c6


Existing style would be to align the *name, *source etc and call the helper something like AsbPackageEopkgHelper or something.


plz add a sign for teleport player in arena !
like :
line 1 : 1v1
line 2 : $world
line 3 : PLAYERS: ,?/2
line 4 : ?


Are you using the Eclipse code formatter plugin?


gboolean on new line pls.


a


polease remove trailing _


No need for {} single line conditionals


so this is changed now? I think that's all I was looking for with that.


this was probably a mistake - we need roll bar.


space before bracket pls, and gchar not char


There are quite a few of these {} single line


Could you use GMarkupParser here to avoid another dep perhaps?


Remplacer
```PHP
$this->extFichier
```
Par :
```PHP
$this->avatar
```
Qui correspondra bien à l'attribut de User rajouté vis à vis de mes commentaires sur le commit bd93abc498b791d3948b9b7b2c13aabeb7be89c6


Good catch :) Done.


if calloc fails, we're sunk, so no need to handle. Also, g_new0 () please rather than the raw calloc call


I am not exactly sure. Sounds like a bug.

What I did with this commit was merely moving config around, the exclusion was there before the changes: See https://github.com/droolsjbpm/kie-wb-distributions/commit/bf96b054ccff6072374538fcfaa16a6e5c151e6b#diff-923f623a389cfca0d3aec0722ff71cc2L122


space bracket etc * lots


gboolean new line * lots


À revoir pour les attributs "clé étrangère".


g_warning (), and no \n is required * lots


This has a slightly different purpose. EPG data is requested in bulk from the client for a restricted range (say 3 days). The backend has much more data (between 7 and 14 depending on listings provider).

Timers will therefore identify recordings outside the EPG data range available to Kodi. If you want to 'view programme details' on a timer, you can go to the EPG database.

If it doesn't have the record you want, this API allows you to request EPG information just for that particular programme, which can then be displayed, or even added to the EPG database as a singleton for 'next time'.

I did wonder if the client ought to be able to 'push' these records to PVR core, or maybe PVR core just requests EPG ID details it doesn't have for timers with dates later than the bulk update period at the end of a timer update sequence.


You could use _cleanup_ptr_array_unref here, no?


_cleanup_free_ perhaps?


single line {} not


Do you need stdarg here?


Thanks for your comments. I have informed Niki. It seems to be challenging to find a solution working with multiple devices having multiple partitions each. Due to other improvements of this commit, we wouldn't like to just revert it.


this is changed by you on this branch. Though I think we can not merge this changes because png file here and commented config for rollbar. Did you get troubled with rollbar?


Aw yea I think I forgot to update that! Thank you for noticing I will do so soon :-)


@blueyed It should be like this, right? b943a9d76dc8a9295423f693b11d6fb174144fb6 seems to remove too many envs or was that the intended change?

Travis has for some reason not picked up pushes to master the latest couple of months, it should now be fixed.


Forgot to note in the commit message that this is for #253


Did you updat the nw.js wiki to reflect this? or would you like me to do that?


I don't think this code should be in the visit.


Yes, rollbar very useful, I know about every error in production


change the error message,  need to change it latter for msvc build


@silverwind I think we can remove the `tab-size-#` classes. It looks like GitHub changed it to use `tab-size` as the class name and included `data-tab-size="#"` instead.

I checked the [files](https://github.com/StylishThemes/GitHub-Dark/blob/master/github-dark.css), this diff & [gists](https://gist.github.com/Mottie/ab2f026e6624e2733c1f) but I didn't remove the extra classes ust in case they are used somewhere I haven't checked.


I would think that if the loop is **not** iterating over a collection, that is when you want to create the warning.


I've never experienced this on the only 2 duo's I have(There might be a third duo, can't remember)


Should the container be on the production component? It seems like more than the production component might need access to it?


This is still the latest release since the last 2 weeks.  Will there be a weekly release soon?


Again, I don't think that this code needs to be in the visitor. You already have the loop in the refactoring class. I think you need to understand what the purpose of a visitor is. A visitor is best used when you need to visit all the nodes within an AST subtree of a particular type. Here, you are only concerned with the beginning part of the for loop.


I had trouble with rollbar in development. I think i need to set an Enviromental vriable to fix?


In fact, this will probably give you embedded enhanced for loops, which is incorrect.


as far as I know, no. More custom champions has been a requested feature for a while though so im sure valve will implement it soon(Tm). I think defense of the shines has more than the cap though so we might be able to reverse engineer how they did that


Keeping 2 buttons.
* 1) Show always in all modes except mobile (`gt-sm`)
* 2) Show in `sm` only, condition: help is open

Welcome to provide better suggestion. Note that the repo is on an older version of angular-material due to a reason cited in one of the previous PR


This comment doesn't seem correct.


I know it's not the whole picture, but it probably works for the common real world scenarios.


This isn't correct. Use the ASTView utility in an example here to see what kind of information you can extract from the for loop expression.


@blueyed  I think it should not be there, I removed that section in 464f8b50945e5ee65a7010ccc65d75ee717a6ede and it did not affect the test envs. Feel free to correct me/revert this if I broke something that you intended to change!


change IIterator to DatasetIter, or other name close, maybe do not inheritate form ``::dmlc::DataIter``.


> Are the values LIB_PATH and BIN_PATH standards?

I don't remember if there are shell variables for the default path used to store libraries and binaries.

> Or set by autoconf/make?

Sometimes I read

~~~
Installation directories:
  --prefix=PREFIX         install architecture-independent files in PREFIX
                          [/usr/local]
  --exec-prefix=EPREFIX   install architecture-dependent files in EPREFIX
                          [PREFIX]

By default, `make install' will install all the files in
`/usr/local/bin', `/usr/local/lib' etc.  You can specify
an installation prefix other than `/usr/local' using `--prefix',
for instance `--prefix=$HOME'.

For better control, use the options below.

Fine tuning of the installation directories:
  --bindir=DIR            user executables [EPREFIX/bin]
  --sbindir=DIR           system admin executables [EPREFIX/sbin]
...
~~~

I checked autoconf documentation and this is the standard.
I will change the pull request to match this.


make this a public function that returns std::string. make a default implementation that returns ```{'label', 'data'}```


nit: whitespace


I am intersecting invisible objects, now it's not working anymore.. could it be an option?


what kind of trouble? I don't set Env var. in development because it's disabled. I only drop this config for heroku


index_t is typedefed already, remove mshadow


@jglick In this plugin, other developers are putting the annotation over the method declaration


Should be no need for argument checking in internal functions...


use init list for constructor of member


Prefer no boolean coercions, here and below; use valueSet.size === 0 or valueSet.size > 0


Hm, we can only have one featured image, so would maybe a radio button make more sense than a checkbox?


:worried: what did you actually change?


domSymbolTree should probably expose these directly.... reaching into a dependency's internals is not great practice.


-1 has the drawback that, when a new field got introduced by an API change, you always have to change the pvr addon code to initialize the newly field, even if you not even can support it. If the default for the field is 0, the addon dev has nothing to change in the addon code to "support" the new field (given the code is so clever that the struct gets generically initialized using memset(..., sizeof(struct-type)) - at least if all that is to do is to initialize it properly. Got the idea of the trick?


Seems weird that we require a float here. Can you maybe post a screenshot of how it looks like?


I don't understand how this test description matches the test?


There is no need to check the same test twice ( take a look at the previous line).


in 0d9a34a65bb2be0f5f6ea3dcd8e7ae973d0b866f


I'm having some trouble following where this is used... seems like a strange way of doing things.


i dont know what this is


31312e0a74d5a0779433e85d17baef15b94c1ee4

this is my SHA 1


The amazingly cool thing about Rust is that providing mutable iterators is a copy-and-paste job if you write the immutable version first. I found this out in my experiment with a tree. Unfortunately, I do not know of a clean way to refactor the copy-and-paste aspect (could do it with a macro but that is such a hack) https://github.com/FranklinChen/baker-samefringe-rust/blob/master/src/iterators.rs


Oh I see, you force it via js to behave like a radio button. Why not use a radio button directly?


then how could we implements the method, ThrownExceptionFinder.getThrownUncaughtExceptions(), this method return list of error,


You deleted the entire CSS file. Mistake?


Hmm, I like it?


clearfic > clearfix


WoW! Nice Job Man '-' :+1:


Slightly confused about the checkbox/radio button issue. Otherwise lgtm. Do you maybe have  screenshot from your environment to see how it looks like?

Also, maybe Lems mentioned that we might want to support selecting a author as a featured image without listing them as a author. But we can address this in a second iteration.


`entry` might be null here...


The problem with using -1 is that the addon would have to remember to set the value to -1 if it wasn't using it.
Is it possible to include an intialisation routine in xbmc_pvr_types.h which sets these values to -1 by default when a new PVR_RECORDING is created?

While I agree wholeheartedly that 'magic' is bad, I'm only using ttvdb scraper compatible 'magic' (http://kodi.wiki/view/Add-on:The_TVDB). :grinning:

You could make it a 'SeasonAndEpisode' class which takes care of the 'magic'. (mySE.SetSeason() / mySE.GetSeason() etc...). The problem is that sounds like a project in it's own right which would end up tied to ttvdb as a scraper. Maybe someone has already made one?


Potentially solved by atvcaptain:
https://github.com/oe-alliance/oe-alliance-core/commit/49510391b94935ae0f35b167841356376e4815d8


Found the way to let it works on OpenERP 7.0-20140421-231236:

use the 'limit' attribute and pass to the extend function a JSON object rather then an integer (otherwise it's just ignored)
```
+        try {
+             var list_options = {limit: JSON.parse(this.node.attrs.limit)};
+        } catch(e) {
+             var list_options = {};
+        }
```

**Question**: is it possible to pass a value to obtain "Unlimited" option?


The first step is to move this code to its correct location, in the refactoring class.


@deepfire: Thanks for tackling this one. We need to get to the root of this problem.

Note that `haskell-cabal-get-dir` is called once and only from `haskell-session-cabal-dir`. `haskell-session-cabal-dir` is named as if it were a simple accessor and is used as such everywhere except when session is created.

Lets refactor this in a simpler way:

- [ ] when session is created use `haskell-cabal-find-dir` and store it in session using `haskell-session-set-cabal-dir` in `haskell-session-make`
- [ ] move all `haskell-session-*` functions from `haskell.el` to `haskell-session.el`
- [ ] remove all logic from `haskell-session-cabal-dir`, make it a simple accessor that never ask questions
- [ ] remove `haskell-cabal-get-dir`, it will not be used at this point
- [ ] fail with error message if cabal dir cannot be found, it is required further down in the code

Then we are able to get away without the need for a defcustom.


Good, that's all I needed to know.


@opdenkamp what's your opinion on this? IIRC you're the one who removed `strRecordingId` from this struct (@metaron-uk we used to have exactly this).


makeCacheMatrix <- function(x = numeric()) {

  # holds the cached value or NULL if nothing is cached
  # initially nothing is cached so set it to NULL
  cache <- NULL

  # store a matrix
  setMatrix <- function(newValue) {
    x <<- newValue
    # since the matrix is assigned a new value, flush the cache
    cache <<- NULL
  }

  # returns the stored matrix
  getMatrix <- function() {
    x
  }

  # cache the given argument
  cacheInverse <- function(solve) {
    cache <<- solve
  }

  # get the cached value
  getInverse <- function() {
    cache
  }

  # return a list. Each named element of the list is a function
  list(setMatrix = setMatrix, getMatrix = getMatrix, cacheInverse = cacheInverse, getInverse = getInverse)
}

# The following function calculates the inverse of a "special" matrix created with
# makeCacheMatrix
cacheSolve <- function(y, ...) {
  # get the cached value
  inverse <- y$getInverse()
  # if a cached value exists return it
  if(!is.null(inverse)) {
    message("getting cached data")
    return(inverse)
  }
  # otherwise get the matrix, caclulate the inverse and store it in
  # the cache
  data <- y$getMatrix()
  inverse <- solve(data)
  y$cacheInverse(inverse)

  # return the inverse
  inverse
}


very good


@Shelob9 Per the WP Core PHP Documentation Standards `@return` descriptions should end in a period. Mind updating this?

https://make.wordpress.org/core/handbook/best-practices/inline-documentation-standards/php/#1-functions-and-class-methods


@emnh can you use `material.visible = false` instead?


@Walkman100 Pleasure bro :smile:


looks more readable than previous solution, but it is not convenient to create this method. In first case I should do just one click in Idea, but in HashCodeBuilder I should remember about ALL fields that I have, and enumerate them manually in this method. Lets discuss it in skype, qoz I don't think that we really need it.


Ich rede mir den Mund wohl fusselig, wenn ich sage: "Bitte den Workspace nicht mit comitten"
Naja ...


This line should be removed


@deepfire: This `kill-session` needs to be fixed as follows:

- [ ] it should not ask questions at all (no need for a defcustom)
- [ ] should kill only GHCi buffer (and `*haskell-process-log* if enabled)
- [ ] session should unlink itself from every buffer/file/whatnot

Note that the last item is pretty complicated but it can be done.


I have created a pull request adding these two lines. It is more to test github workflow and how to contribute so I can delete it if it is useless. Thanks.


And I have no idea what the jbpm runtime needs... I assumed whoever
configured kie-drools-wb following the introduction of Asset Management
knew what they were doing... but long and short; if something's not right
let's fix it!
On 26 Jul 2015 19:25, "Petr Široký" <notifications@github.com> wrote:

> I am not exactly sure. Sounds like a bug.
>
> What I did with this commit was merely moving config around, the exclusion
> was there before the changes: See bf96b05
> #diff-923f623a389cfca0d3aec0722ff71cc2L122
> <https://github.com/droolsjbpm/kie-wb-distributions/commit/bf96b054ccff6072374538fcfaa16a6e5c151e6b#diff-923f623a389cfca0d3aec0722ff71cc2L122>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/droolsjbpm/kie-wb-distributions/commit/bf96b054ccff6072374538fcfaa16a6e5c151e6b#commitcomment-12362364>
> .
>


ok


just noticed, there is a typo in this class) we have to rename it


IIUC this will not work because the bytestring object will not be rooted.

I don't think it's a good idea to do this in general, for one thing, whether the string can be mutated by the c code now depend on the value of the string. It is probably better to leave this decision to user code.


Internal engine / game mechanics reworking and features are not relevant to a features and fixes changelog for players such as this.


An incorrect implementation might assume `<img id="foo" name="foo">` should result in a `HTMLCollection` with two entries. I'll try to reword it


Yikes, I see what you mean! ;-) This better?

https://github.com/kivy/kivy/commit/0af6d8f1eb265b647ca0fb3a8110f90da2f103d7


@DDuarte Yes, I'm serious.

I see what you mean. There are certain columns that aren't used anywhere (comments, notes) etc. Then, why would we even load them? Sure, they might get selected, but they won't be used, so, it doesn't really matter.

Every line of code in selecting columns through Fields is 1 column (because the index is being incremented for each column). If someone adds a field in the middle and it doesn't match the database's structure, then I guess he doesn't really know what "increment" really is. He shouldn't modify Core if he doesn't know what he is doing. (Just saying).

About the "INSERT INTO `table_name` VALUES (xxx);

There are PreparedStatements that have this SQL code. But, they use column names (e.g: INSERT INTO `creature_template` (`entry`, `difficulty_entry0`, `difficulty_entry1`, `difficulty_entry2` ... etc) VALUES (xxx). What if a certain column gets changed? This would mean that the Core will fail and it might not save some stuff that might actually be important.


@mrdoob That works for me :)


Thanks! Closed that issue.


i have compile error , can u fix it plz ( this is last revision in trinitycore )

 3>C:\Src 3.3.5\TrinityCore 24.5.2015\src\server\scripts\Commands\cs_anticheat.cpp(130): error C2248: 'Position::m_orientation' : cannot access private member declared in class 'Position'
2>C:\Src 3.3.5\TrinityCore 24.5.2015\src\server\game\Anticheat\AnticheatData.cpp(42): error C2248: 'Position::m_orientation' : cannot access private member declared in class 'Position'
3>          c:\src 3.3.5\trinitycore 24.5.2015\src\server\game\entities\object\Position.h(55) : see declaration of 'Position::m_orientation'
2>          c:\src 3.3.5\trinitycore 24.5.2015\src\server\game\entities\object\Position.h(55) : see declaration of 'Position::m_orientation'
2>          c:\src 3.3.5\trinitycore 24.5.2015\src\server\game\entities\object\Position.h(26) : see declaration of 'Position'
3>          c:\src 3.3.5\trinitycore 24.5.2015\src\server\game\entities\object\Position.h(26) : see declaration of 'Position'


OK


test


i see it


My bad! wasn't issue #3, but #13


hey :P


Even if you use some trick (which is possible) in `cconvert` to root the possible full string, it is still very dangers because the pointer is invalid immediately after the ccall function exit.

This might be fine for many cases, but will make our somewhat error-prone interface for passing pointer to c functions even more so.


a *lot* better :) thanks


This was already fixed here: https://github.com/PhilJay/MPAndroidChart/commit/077b25446be05afa89c93a805e55f47c4b9e99d1
So these lines are redundant


## This is a markdown file


Lol, this was in July... [original](https://gist.github.com/Walkman100/d3da44062d7eebd18548/revisions) (3rd from the top)


Then it's not Relay that joins the channels, they just are in the channels list.


Ag, *June


I concede, makes sense.


I think the GitHub account thomsmits has been deleted


just to show the download speed ONLY on panel


Sweet!


Indeed. Looks they finally figured that a tab-size of 8 is excessive :laughing:

I wonder if there's a option somewhere for this now. I'll check more later if it's safe to remove our styles.


Right, it used to be a runtime dependency and hence pulled in transitively. However, there are use cases where one cannot or does not want to use EL. EL is really only used by the constraint violation error message interpolation. So we provide a Hibernate Validator specific interpolator which is not using EL in which case the dependency to EL is not required. This needs to be explicitly configured, though and won't comply with the full Bean Validation spec. In this context we changed the scope of the EL dependency.


// @ktrotter-emsi


This should probably go directly upstream ping @fernetmenta


Vezzra, did you get most / all of MatGB's other suggestions in the release branch changelog fix commit?


I'm seeing this too. Reverted for now: https://github.com/sindresorhus/editorconfig-sublime/commit/07b889699b744ec4b134ac74d5e4878cd3a4f585


I see. I don't think that the `'cam1'` piece of the PV string needs to be hard-coded. At SRX, the area detector PV does not happen to include `'cam1'`.


This needs packaging in OpenELEC, right?


@FranklinChen I've personally started to favour just generating the code from string formatting.

You can also decompose the problem into a smaller unit: make node iterators, and build the tree iterator generically on top of the kinds of node iterator. Btree does this:

https://doc.rust-lang.org/src/collections/btree/map.rs.html#80-102

https://doc.rust-lang.org/src/collections/btree/map.rs.html#977-1043


Instead of `user { Fabricate(:user) }` - just `user`. no block.


There is no easy way to *deselect* a radio button, that's why I used checkboxes.


Done.


Точно. Done.


И правда.


That's a very good point. Since it'd be just as clear, and a bit less verbose, I'll just make the selector more specific instead of adding the comment.


Just a note: you can use `#[doc(hidden)]` to keep public methods meant for internal use out of the docs.


Take this as parameter? To soon be fetched from nin.json?


__ y _<UperCase> son palabras reservadas del C, usar el template.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 212](https://teamcity.labkey.org/viewLog.html?buildId=262697&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5 (1 new), passed: 20 Build time: 00:35:11
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySavedGroupPlot: <no details avaliable>

```


Good catch. I'll just get rid of the Cstring aspects of the PR.


por favor basar todos los nuevos archivos en https://github.com/ciaa/Firmware/tree/master/modules/template


mejor incluir ciaaPOSIX_stdint.h


Eventually yes. It's not required - madVR 3dlut files can be used without it.

I have support for loading ICC profiles using lcms2, but it needs to be reworked to keep video in limited RGB.


tenemos ya definido el formato? no me qeuda claro por que se incluye usb.h, pero tal vez es mi culpa que tengo que revistar un poco como era la estructura de archivos :)


I think this should be need_emulation ${ARCH%.*}, otherwise it's checking 'need_emulation foo.bar', rather than 'need_emulation foo'


tendriamos que ver el tema de namming, fijate como esta posix y otros modulos. y no seria mejor comenzar todo con ciaaUsb o ciaaUSB? y luego lo que venga? digo para no ensuciar el namespace. en este caso el _hid_protocol_t ni veo necesario que este.


We can't do that by itself right now because I'm still using coalesce() internally, but maybe it can all be simplified now. Also, I need to think through what's possible with event loops sending an event round the loop more than once, which is possible if you use filter.


`pristinePrice`

OMG :heart_eyes:


Right, I did start looking at the btree implementation for tips on what people do.


no seria mejor
ciaaUSB_Hub_isActive?
<Modulo>_<SubModulo>_<Function>?


usamos 3 spaces y no tabs en el firmware...


This `noSuchMethod` and the `noSuchMethod` from 113 are almost the same. Please factor out the same code and reuse it.


можно просто `complete {...} `, без обращения к контексту


por un tema de allignment es mejor poner las uint32 primero luego las uint16 y por ultimo las variables uint8.


es necesario mezclar macros y type defines? no se podria primero macros y luego los typedefs?


usamos extern para que quede claro... :) igual funciona así


CodeBook.md describes the variables, the data, and any transformations or work that was performed to clean up the data.
run_analysis.R contains the script to perform the analyses
tidy_averages_data.txt is the output file created out of the script


лучше логировать саму ошибку тоже


Besides, Calamares adds that line back in on the installed system anyway.


cada parametro de un macro debe estar con parentesis. sino puede que se interprete mal..


no tenes que definir el null vos.


el fixme es correcto


тут переменная, которая лежит в payload не используется, т.е. можно просто `case msg: Fetcher.Payload`


Use row only in frontend.


se deben usar los headers del firmware.


Transform this =>
![origin](https://cloud.githubusercontent.com/assets/2198839/8895576/609b8b60-33df-11e5-9e9c-b4005a51b866.png)
in =>
![me](https://cloud.githubusercontent.com/assets/2198839/8895577/6f083c48-33df-11e5-8dae-97c5df3e43b6.png)


es una buena practica usar siempre { }


yo prefiero poner
if (USB_CLASS_HID != piface_desc->bInterfaceClass)
en vez de
if (piface_desc->bInterfaceClass != USB_CLASS_HID)

de esta forma se evitan posibles errores.


esta funcion tiene demaciados returns, es mejor que una funcion tenga solo un return. muchos returns es como usar goto... :( no es una buena practica.


queremos usar realmente int o mejor int32_t o algo que tenga largo definido?


mejor seria suar una variable de return y tener un unico return


en todas las declaraciones deberia haber un commentario de doxygen como en el template


Are you sure? I think the default is 1.0


if it is expected that the user will edit the template before running the instance for the first time (which I did) then it wold could sense to change this to

```if mapping['auth']['shared_secret'] == 'change-me': ```

which would suppress the " Your shared secret has not been changed from its default. Your system will be insecure until you change auth.shared_secret in config.toml." warning.

Another option would be to template the secret to be empty and check that it's empty here.


no es bueno comentar codigo, mejor seria un #if 0 y un comentario. asi queda claro que no es un error.


@pelme
IIRC the intention was to remove redundant combinations, as per the methods doc:
> Returns a list of testenvs that include all different Python versions, all Django versions and all database backends.

The aim was to be more friendly to Travis.


Yeah, I considered that, but I wasn't sure if I wanted to completely hide it so I chose to add a comment for now. I have to decide how private it should be.


So far so good.
But you left out to implement the individual card ability/effect.


http://web.engr.oregonstate.edu/~mcdoncam/multtable.php?max-multiplier=10&min-multiplier=5&max-multiplicand=10&min-multiplicand=2
example url


Thank you @maser


si es unsigned deberia ser 0xFFFFU, todo los valores que sean mayor o igual a 0x8000 y son unsigned debe tener una U


the commit was generated :p


se podria agregar una variable y evitar el doble return


You have to implement this ability. Look at other cards that trigger at the beginning of your upkeep (BeginningOfUpkeepTriggeredAbility), and how to target an opponent to get an idea. This card is incomplete as it stands.


Btw, there's one too many spaces between the progress bar and percentage. It looks weird. I removed it in the screenshot, but can you remove it from the code?


no hay break? es bueno ponerlo siempre.


se deberian usar las librerias de la ciaa ciaaPOSIX_strotol seria en este caso, no se si existe. con gusto lo podemos discutir en el grupo de firmware. no es una muy buena idea, pero es asi como se resolvio hasta hoy... :) :(


TOOO many returns...


вообще хорошо! пару комментов оставил только стилистических, но это так, мелочь.


esta funcion puede no retornar nada?????


penwidth() will never fail.
I believe this is also true for other pen functions (althrough all the others have functions that throw task error code as exception - only for Scribbler in CTaskPen, unused)
pencolor() SHOULD actually fail for incorrect values, but that's not implemented right now, it just clamps the values to range White(0)-BlackArrow(17)


I got it booting now with the 3.10 branch


Thank you Sindre!

Yeah, I agree, I'll remove it. Brightness-cli ha this as well right?


List various EXE installers?

```
if ($installerType -ne 'MSI') {
  # Uncomment matching EXE type (sorted by most to least common)
  #$silentArgs = '/S'               # NSIS
  #$silentArgs = '/VERYSILENT /SUPPRESSMSGBOXES /NORESTART /SP-' # Inno Setup
  #$silentArgs = '/s'               # InstallShield
  #$silentArgs = '/s /v"/qn"'       # InstallShield with MSI
  #$silentArgs = '/s /f1"$issPath"' # InstallShield, requires uninstall.iss file
  #$silentArgs = '/s /a /s /sms /f1"$issPath"' # InstallShield's PackagefortheWeb (PFTW), requires uninstall.iss file
  #$issPath = $true # if using $issPath in silentArgs
  #$silentArgs = '/s'               # Wise InstallMaster
  #$silentArgs = '-q'               # Install4j
  #$silentArgs = '-s -u'            # Ghost
  # Note that some installers, in addition to the silentArgs above, may also need assistance of AHK to achieve silence.
  #$silentArgs = ''                 # none; make silent with input macro script like AutoHotKey (AHK)
                                    #       https://chocolatey.org/packages/autohotkey.portable
  $validExitCodes = @(0)
}

if ($issPath) {
  $toolsDir   = ""$(Split-Path -parent $MyInvocation.MyCommand.Definition)""
  $issFileName = "uninstall.iss"
  $issPath = Join-Path $toolsDir $issFileName
}
```


What is nullCnt expected to be, when running several search threads ?


I think maybe this should be handled differently; if you have more than 1 object to select from, that's probably not a good sign....maybe:

```
$key = Get-ItemProperty -Path @($machine_key6432,$machine_key, $local_key) `
                        -ErrorAction SilentlyContinue `
         | ? { $_.DisplayName -like ""$softwareName"" }

if ($key.Count -eq 1) {
  $key | % {
    $file = ""$($_.UninstallString)""

    if ($installerType -eq 'MSI') {
      # The Product Code GUID is all that should be passed for MSI, and very
      # FIRST, because it comes directly after /x, which is already set in the
      # Uninstall-ChocolateyPackage msiargs (facepalm).
      $silentArgs = ""$($_.PSChildName) $silentArgs""

      # Don't pass anything for file, it is ignored for msi (facepalm number 2)
      # Alternatively if you need to pass a path to an msi, determine that and
      # use it instead of the above in silentArgs, still very first
      $file = ''
    }
    Uninstall-ChocolateyPackage -PackageName $packageName `
                                -FileType $installerType `
                                -SilentArgs ""$silentArgs"" `
                                -ValidExitCodes $validExitCodes `
                                -File ""$file""
  }
} elseif ($key.Count -eq 0) {
  Write-Warning ""$packageName has already been uninstalled by other means.""
} elseif ($key.Count -gt 1) {
  Write-Warning ""$key.Count matches found!""
  Write-Warning ""To prevent accidental data loss, no programs will be uninstalled.""
  Write-Warning ""Please alert package maintainer the following keys were matched:""
  $key | % {Write-Warning ""$_.DisplayName""}
}
```


In file keyhandler.html, this commit replaces embedded tabs with spaces.

In file removeformatting.js, this commit rolls back changes not needed for the identified issue #267.


Damn. Nice work!


Consider using the options format on lines 37/39 in your main `number_to_phone/2` function.


Yeah I think it might of been before (pretty sure I just copied this code over from master, modulo a couple tweaks).


眠りの巻物 #551


混乱の巻物 #552


    python glm_parser.py -i 5 -b 2 -e 6 -t 0 -p ~/data/glm-parser-data/penn-wsj-deps/ -d 07-25-2015 --learner=average_perceptron --fgen=english_1st_fgen --parser=ceisner
    07/25/2015 05:48:04 PM INFO: Feature count: 4301085
    07/25/2015 05:50:40 PM DEBUG: Correct_num: 16, Gold set size: 16, Unlabeled correct: 40317, Unlabeled gold set size: 46451
    07/25/2015 05:50:40 PM INFO: Unlabeled accuracy: 0.867946868743 (40317, 46451)
    07/25/2015 05:50:40 PM INFO: Unlabeled attachment accuracy: 0.873191102291 (42238, 48372)


@sheagcraig - I don't think I understand well enough what's going on here. The next line, a `for` loop, generates this traceback:
```
Processing /Applications/AutoPkgr.app ...
{}
Traceback (most recent call last):
  File "./recipe-robot.py", line 953, in <module>
    main()
  File "./recipe-robot.py", line 919, in main
    handle_app_input(input_path)
  File "./recipe-robot.py", line 398, in handle_app_input
    create_buildable_recipe_list(app_name, prefs)
  File "./recipe-robot.py", line 333, in create_buildable_recipe_list
    for recipe_format, is_included in prefs["RecipeTypes"].iteritems():
KeyError: 'RecipeTypes'
```
But since `prefs` is a global dictionary, I would think that the `init_prefs()` function reading it from the plist on disk would be sufficient to populate it with keys and values that are accessible by other functions. Maybe that's not true?


Package pyOpenSSL need not be installed. The most important is whether it is or not. Frankly, I don't know an important version of it

If you have new ...unless you've installed PyOpenSSL. In this case it's recommended...
(in which case? the first or the latter. This is really important and it must be clear and easy to remember.


Here, i compiled a version using HikariCP instead of our pooler. See if you notice a difference: https://www.dropbox.com/s/5oyxrsauzylcken/HawkEye.jar?dl=0

I can't really tell. It seems to insert faster?


What is the purpose of this? It also seems to break the code highlighting http://networkx-metis.readthedocs.org/en/latest/install.html#github.


before 你應該要操作 command 的 model 把資料塞到資料庫囉

然後

```
before(async (done) => {
...
}
it("index all comments", async (done) => {
...
}

```

改成

```
describe.only('comment', () => {
  describe.only('index comment', () => {
    before(async (done) => {
    ...
    }
    it("index all comments", async (done) => {
    ...
    }

...
```

因為你的測試資料是為了 index comment 而建，所以包在一起囉


looks good. thanks!


Typo. I meant to delete a trailing space.


rax retry tests


i do not know why this did not work the other way.  But this does fix the problem.


>I thought that's the new line

Yes. It was earlier throwing an error because he wasn't using `std::move`


this is my first github repository


`eqnuote`?


LGTM :+1:


Nice, didn't know about that!


Yup, I tried many different values and compared them with each other. I'm not sure if this can change depending on the bot's width or ground. Anyway, any value grater than 1.0 didn't change anything for me, and negative values looked like 0.


@jleclanche Nice, didn't know about it!


I'm going to do that (have it in another branch), but I wanted to get the beatbean stuff refactored out of the looper first


Just to give you some code review feedback: I feel like the label/slider code is very similar across the different rubric bits, seems like it could be wrapped into a function and re-used.


For 1v1 against a player we need this player have killed us ?


That sort of makes sense. The `.template` ending is hardcoded in some places though (think code compliance checker).


FYI, jQueryUI tends to be a large library to bring on -- I don't recommend bringing it in unless you intend to use a lot of things from it. If it's just one thing, I'd find a dedicated library.


Probably, but in source code they theoretically can fail (if I understood the code correctly...). I wasn't sure if I should just describe the result types as void or leave it as it is now.


@knneth thanks for the feedback, I totally missed that one. I will fix it up.


YOU ARE SUCHHH A FAGGGG OTTTT!


I love the idea of using the slugified directory name for the index -- maybe the bucketname on s3?


FYI.
I maintain carbon-c-relay in Fedora+EPEL, see:
https://pkgs.fedoraproject.org/cgit/carbon-c-relay.git
https://admin.fedoraproject.org/pkgdb/package/carbon-c-relay/


... with RSpec 3.


Yes.

// @kevva


Not really nessecary as npm takes care of this, but why not.


That is to control which packages are allowed in each repo on the build server. Its only partially implemented. Actually, I don't think I will keep it. I am planning to restructure the way the build server stores its data soon. But for now, any new packages you add (that previously did not exist in the repo) need to have the  `_allowed_in` variable as shown above :grimacing:


nit: this might better be called list_files, since it will skip nested modules (although I guess it also skips non-module files too).


I think it would be a good idea to be explicit with the warning:
"Could not find data.yml or data.yaml; using default options"


Pull requests are welcome :smiley:


fixes #7


Note comments around original components of VibratorHW.java.  Since confirmed working a separate commit to clean up will occur soon.


https://gist.github.com/Rushor/e6069c800b5b070b965c - See you on 6.x :)
![Bild](http://i.imgur.com/PoI2fNo.jpg) - Rndmpic of that beautiful zone


No I did not update the wiki, it's my fault for not reading it properly in
the first place.

Heading up the EVENTS section is already this clause:

"Following events can be listened by using Window.on() function, for more
information on how to receive events, you can visit EventEmitter."

I rushed in and started using window.addEventListener cause that was already
in use in app.standalone.js

My bad ;-)

  _____

From: SchizoDuckie [mailto:notifications@github.com]
Sent: Monday, 27 July 2015 06:32
To: SchizoDuckie/DuckieTV
Cc: garfield69
Subject: Re: [DuckieTV] fix #515 (6e695aa)

Did you updat the nw.js wiki to reflect this? or would you like me to do
that?

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/6e695aa61b5f424d0c83e39a5e3
cde582dbd8e2a#commitcomment-12362391>  it on GitHub.
<https://github.com/notifications/beacon/AGnK-IPm6rYbay7Ace-iRQyLOE45uEL8ks5
ohR8ogaJpZM4Ff5kA.gif>


The line was included in order to avoid the compilation warning `-Wunused-parameter`, as all warnings cause a failed compilation.


I returned an empty string here instead of None to prevent a subsequent type error, when None is passed in to `get_expect_params()`

Whenever `get_called_src_line()` returns `""`, specter seems to print out values instead of variables for failed expects: e.g.  `'a' to equal 'b'` instead of `var to equal other_var`. So I thought an empty string would be okay.


Always use async methods? Are there cases when not to use async?
What are best conventions for naming callbacks? What about names for modules and main?
What is best way to handle error? I used a try/catch/finally here to customize error handling a bit..

NOTES:
I like the official solution's use of the array method 'filter'
console.error --> use it!
for/each instead of long notation


I accept your good taste ; )


Should the access statements not be below the function description?  Correct me if I'm wrong! Just looks a little funny to me.


Allright awesome :+1:


## This is a markdown file


Is capitalisation of types the convention?


@codenirvana Can you add some for non-integers too?


Result of this potential fix:
On ET8k, with internal HDD and 2-partition USB-stick, flashe a freshly brewed ViX-image that holds this fix; better, but not perfect:

1- The good news is that on the reboot following the flash all mounts are there, allowing for a restore during FIW. The HDD is mounted as HDD.

2- Also good news is that there are no more 'double' mounts.

2- The less good news is that the USB-partitions are SDA5&6, while normaly at least one of them (the FAT32 partition) was USB.

3- The next good news again is that the mountmanager allows for changing the mounts to USB/USB1 and that they survive the reboot.

So: not perfect, but workable.


I think removing this to a plugin is not so good; I really like that s9y blogs behave "canonically" by manipulating the URL so that a user can always have this date routing, without the owner installing something.

An archive/date view is something that is IMO a core functionality to a blog, and is really the distinction of a blog from a CMS system. Removing that would really hurt me. :)


I am happy for suggested code improvements. I assume this comment is to avoid possible naming conflicts in other files? Then also add a prefix to the other enum names?

(Oh, EMPTY disappeared from the rest of the code - will delete.)


Same here, I don't like removing this feature.


need to restore this path from absolute on wozniak's computer to eclipse-relative path:
${PROJECT_DIR_NAME}/src


Allow percent as a legal value for the width and the height.

Automattic#2445


Why was `System.out.println("mibbzz is gay");` removed?
This software does not function correctly without it.


Yes, indeed.  I knew you reported it but couldn't find.  Now I see it was against Pango...


Modified version


This removes warnings from players, is this intentional?


cluster_level 1 will do that.


@CPA-Poke here it is


Yeah. Somehow i think namespacing by prefixing makes sense for C enumerations. Main goals: avoid conflicts and improve code readability.

Examples:
**cURL** (i really like cURLs code)
* https://github.com/bagder/curl/blob/master/lib/ftp.h#L42
* https://github.com/bagder/curl/blob/master/lib/transfer.h#L29

**GIT** (also good C code IMO)
* https://github.com/git/git/blob/master/diff.h#L182
* https://github.com/git/git/blob/master/sequencer.h#L16


change original JSONResponse to accomodate id and delete this


> Sure, they might get selected, but they won't be used, so, it doesn't really matter.

Asking the database to retrieve data that is not going to be used is a waste of resources and can lead to serious performance issues, especially if one of the unused columns is a blob with lot of bytes.

If you are concerned about SQL queries written wrong and not persisting the data correctly in the database, I have to remind you that both 6.x and 3.3.5 branches are a work in progress and they should be tested before applying the updates to your production environment. You can also contribute to the project by peer reviewing all commits to ensure new fields are read/written correctly.


/me idly wonders whether this is worth adding in to libsyntax...


ourse Project

You should create one R script called run_analysis.R that does the following.

Merges the training and the test sets to create one data set.
Extracts only the measurements on the mean and standard deviation for each measurement.
Uses descriptive activity names to name the activities in the data set
Appropriately labels the data set with descriptive activity names.
Creates a second, independent tidy data set with the average of each variable for each activity and each subject.
Steps to work on this course project

Download the data source and put into a folder on your local drive. You'll have a UCI HAR Dataset folder.
Put run_analysis.R in the parent folder of UCI HAR Dataset, then set it as your working directory using setwd() function in RStudio.
Run source("run_analysis.R"), then it will generate a new file tiny_data.txt in your working directory.


Hi there! Thanks for ur fix!
Unfortunately that didnt help me. In case it might be useful for somebody i will post here what i did in my case:

#41  if (!this.options.keepOpen) {
            $('body').click(function (e) {
                if ($(e.target)[0] === that.$choice[0] ||
                    $(e.target).parents('.ms-choice')[0] === that.$choice[0]) {
                    return;
                }
                if (//e.target.tagName.toUpperCase() === "INPUT" &&   ****only this line to be comment***
                    ($(e.target)[0] === that.$drop[0] ||
                    $(e.target).parents('.ms-drop')[0] !== that.$drop[0]) &&
                    that.options.isOpen) {
                    that.close();
                }
            });
        }

Regards!


OMG :smiley:


Not thrilled with this. The underlying implementation details of argparse is being leaked and the function is not really doing exactly what the name suggests.

The function does not just validate the type, it also does casting (because of argparse)


This breaks display of some log entries such as EKF3.IPD, which rounds to zero


You can't have an explicit `<` in html.  You need to use `<`


Great, but what about other uses of cluster values in Pango? A quick look at the code suggests it expects combining marks to have the same cluster as the base mark, would it be an issue to use level 1 by default?


so, if you know that it's wrong, you should know how is correct. i'm listening your ideas.


We should review those indeed.  Where did you spot such a case?  Lets followup on the pango bug.


I like to switch pango to level 1 by default, after resolving these issues.


Why the formatting change? (Doesn't this break the bootstrap test?)


The fact that the core does not start after a database structure change is a "feature" and not an issue, I'd say. If this wasn't the case, it would be easy to change the core and forget about the db, or the other way around, when changes happen.

The way it is now, every wrong query results in a clear error, including the data that is saved to the DB. If you are concerned about your saved data, just check the db error logs every so often. It's better than having possibly wrong data being saved due to queries without checks for column names.


We should probably do this differently than firing an event every 100ms hoping to catch the page at the bottom.  We could do clearTimeout/setTimeout in the scrollInterval function to run a similar function to the scrollInterval.

The order of operations in scrollInterval -
1) test/remove event handlers
2) test/assign state

seems reversed too - it would make better sense to reverse these around?


I would almost go as far as to do `getPlayer().transform(Functions.indentity());` to avoid this ugly cast.


![Uploading atm.gif…]()


I wonder if we should dump ChangeSet completely and just have `type FileMap = HashMap<String, StringBuffer>;` plus some functions?

I think the comment is out of date in any case.


See mercurial pull request at https://bitbucket.org/zurmo/zurmo/pull-request/952/allow-command-line-installer-to-use-non


OK. Thanks. I will do so in another pull request since it will affect lots of files. Feel free to let me know if you have a prefix preference.


See updated code.


Oups....

Note to myself :  Never write commit message when I'm tired


Hi Brady. I did not update the Copyright because I had the impression that Richard Heyes want to keep the same copyright.. Let me know if I need to changed.
tested on Summary -> Documents


I just wonder whether we should delete the related code, since I still see some places calling methods in ```score_cache.rb```.


@sgrif Looking at this commit remembered me of #18674. You were positive but then @al2o3cr raised some concerns and it got forgotten.


josuebrunel and pacoSam


use LoggerInterface wherever you are using echo...


myArray is a bad variable name.


it's a count, so $userCount == 0 is more readable.


1v1 plugin still crash my server. The console displays this message : Parse error: syntax error, unexpected '$game' (T_VARIABLE), expecting function (T_FUNCTION) in phar:///home/gsp_20/151.80.76.69-19147/plugins/phar_1v1_Fombnlakxk31dP1.phar/src/onevone/MainClass.php on line 14


huh? what are you doing with userArray? and 0?


why? are you checking for null?


is using sprintf the best option?


UserDataFactoryInterface. Let's stick to Interface, not I. I looked at some prs libraries and they were using Interface.


space.


good


Author          : josuebrunel-pacoSam


remove Hippo from our code base.


## This is a markdown file


rework on your spacing.. we are doing two spaces.


return type for function?


I'm talking about the default value if you don't specify it


write a constructor User.


remove these var_dump please...


it logs with level 'ERROR'


@hynekcer Does this seem to make it clear enough? It's definitely a complicated situation.


Aahaha mdr.
Fais comme ceci
1. Ouvre le fichier en question avec un editeur
2. Modifie ce que tu veux y modifier
3. Enregistre
4.
```shell
$ git add mon_fichier_modifie.py
$ git commit -m "J'ai ajoute mon nom"
$ git push origin master
```
5. Verifie sur github :wink:


After an offline conversation with @tacaswell, it appears he has some ideas regarding how to deal with this issue. Tldr, make every piece of the PV configurable in `area_detector.py`


why are you checking in these commented lines?


And I was talking about it too. "I tried many different values and compared them with each other".


Remove these Get functions...


Write contructors for these entities..


indentation.


check the indentation for classes and follow the same rule..


The user is **not** expected to change the template config. @kofalt and I already had a discussion about this on Friday.
Could it make sense that, if no user-defined ```config.toml``` exists, ```live.sh``` does not actually launch the application, but rather prompts the user to edit the newly created ```config.toml``` as desired and then relaunch ```live.sh```?
I think that we already agreed that such a newly-created ```config.toml``` will contain a new, random secret.


check indentatino.


Good. Yeah, all of this seems like it should be inspected or configured rather than hard-coded. There's a lot of boilerplate in there.


**[phpUnit_trunk_ExternalIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunk_ExternalIntegration/1653/testReport/) for this commit**


Whitespace nit.

-[Unknown]


0 means what/


remove Hippo...


But I am not sure of anything now. Someone else should probably check it too :/


Sollte nicht nötig sein, da es der default constructor ist: http://stackoverflow.com/questions/9907722/why-is-default-constructor-called-in-virtual-inheritance
Ich nehms aber im nächsten commit mit rein.


</div> still doesn't seem to be aligned.


@fievelk, @ewanklein: this commit has caused this error: https://github.com/nltk/nltk/issues/1056

To add a corpus we need to agree its name, make sure we have permission to redistribute it, and add it to nltk_data.


Garvin, I'm pushing a bit ahead here but I'm okay to revert this, and I certainly expected you are not totally okay with this (at least at first ;-) ), that is also why this is in a feature branch. Let's discuss it in the forum, or even better, in mumble very soon.I have some ideas how we both can get what we want. Next weekend?


next line:
<x:frag>
    { }
</x:frag>


Or during the week?


hey, didn't realize that this was what you intended to do when passing the dataValues, my bad. i think you should just be able to do gigInfo.toJSON() and that should give you and object with the data properties. i should have read this more carefully earlier


Good. But I hope we'll eventually support at least OpenCL 1.1?


{
on the next line. (this comment doesn't show exactly what I want, so let's sync if you dont get it)


is XHPRoot the right type?? you are retuning :x:form here no?


i'm not so sure to add this, i'll think about it.
You will expose the key.


whoops -- nothing to do with #1.  This fixes #2 and #3


Also, printing the secret at the time of creation would be great. The user typically needs this to launch the reaper on another machine.


This is not needed, you are guaranteed an undefined macro will resolve to 0 (zero) in `#if` without a warning.


@ytokas Enterprise library and database added. Please review


Is the ecommerce functionality completely gone or is it going to be replaced?
I started just today, to use functionality of it to automate my invoicing... the fact that I could let WSP handle the products and renewals would make it easy to generate invoices in my own invoicing system... But if it is completely gone, i'll have to make something completely new on the side... (which will be a lot of work without WSP providing the details)


Note 0c91c169c0489bae5aed18992458a7f8767686ea, which removes the docs (I don't think anybody needs them).


Yep, you're right. The ApplicationPolicy class it inherits from already has these methods though, so I just left comment_policy.rb alone. Not sure what I was doing for summary_policy.rb


Even if I don't want to discuss here, you could prepare our discussion: I was seriously wondering if there is one theme or one plugin that is using this. Is there something? I'd like to have a look at it to question my own judgement before we talk.


If you spent only 100ms at the end of the page, the metric might not be useful ;-) This currently works well and is a lot more performant than to run a random tracking function as often as possible.

Without a test case that shows this to not work, I'm hestitant to go back to the old version (which did have measurable and visible negative impact on mobile).


Feel free to improve if you have a way performant way - but make sure to limit scroll event handlers.


@kares Activating "--1.8" on master uncovered bugs there as well.  I'll take a look tomorrow.


Eh possibly but libxml2 performs better and cleans up properly, plus this is a stripped copy of what I used internally :]


Apologies. Came from the port which doesn't use GLib


Yeah I use { single lines} in all my projects, so didn't really check hard enough here. I'll fix them in v2


No this is a fallback if we hit failure, in successful cases we need to retain the `GPtrArray`


Erm. Derp. :]


Besides, this discussion would probably be better in the right repository? This isn't even the right one.


To allow it to work you will have to go to [Encrypt configuration data](https://ci.appveyor.com/tools/encrypt) and create the value for the nugget api key and replace the current value.


haha well spotted @grudelsud, completely forgot to update the title!


pro 10/10


I'm no longer seeing the symptoms that originally inspired these changes; therefore, I will leave this the way it is for now.


That's what I've done.


Well... In my repo at least ;)


По принцип трябва да е start_hour / star_minute. Защото ако е time, трябва да пазиш цялата дата. Kогато реално ти трябва само часът.


you broke ImmMap...


invx <- NULL
  set <- function(y) {
    x <<- y
    invx <<- NULL
  }
  get <- function() x
  setinverse <- function(inverse) invx <<- inverse
  getinverse <- function() invx
  list(set=set, get=get, setinverse=setinverse, getinverse=getinverse)


Quoting myself :) :

This change will make rsnapshot
execute the preexec / postexec sripts always before /
after touching the backup directory. This is done
independently of the operation (sync / backup / rotate).


m_ for member. and group public and private together.


space after if. same for all other occurrences.


space after //. applies to all other occurrences.


oh man.. what are theses.... you need to wrap this around some kind of Config class..I explicitly said in the comment: "Be very cautious adding memebers to this class as it can become unmanagable easily"


Fixes #499.


no... why are these function here??