I have to investigate, the kernel doesn't boot with the old implementation.
On 25 Jun 2015 21:35, "RenderBroken" <notifications@github.com> wrote:

> Also I noticed users complaining of freezing up like SoDs with your new
> version. Maybe this is the reason why.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/anarkia1976/AK-OnePone-Reborn/commit/cead63987414bba31ac7a504877903ddfa214008#commitcomment-11868778>
> .
>


This needs a better name


In this commit? That sounds like it should be separated out or at least noted in the commit message. Sorry if this has been done previously and this code is merely outdated. (Although maybe it deserves a mention in the message either way.)

Wasn’t this also beneficial to speed? I’m fully trusting you guys in this aspect though.


Blank line


Probably you're right, unless the integer is a multiple of a large power of p, which happens r rarely.


Check for null


Is this the best package?


Fuckyeah!!!


Done: https://github.com/nicolewhite/slackbot-meme/pull/2


This can use Guava


Can use Guava


1. K
2. This actually aimed to prevent errors at shutdown. Take a look a shutdown hook is added on method addShutdownhook.  So calling NettyServer.stop will close the channel and then the shutdown hook will call stop again and an exception will be thrown......


It was actually being used right up until I added the update function to all entities ^^


Add null checks for all the `Optional`


@dlebedynskyi
I just want to train green cryomancers to purple and notice, that I coudn't define custom profile for Black Ice.
As I said in another thread, we have redundant information in definedTask object. You used in addProfile() key value and he used in custom profile definition taskListName property.


Move to dev dependency


Remove stale tests


Looks good.


ack, i though this is "coordinate_sold" and took a while to think 'sold' to whom. :D


hmm. weird. I will setup the same to see if it is related or if it may be something else causing the no-boot.


closes #39


Maybe we can drop this patch for now. I will work on creating another pull request to cleanup this. I think it will be good idea to query the playbin's uri and suburi property instead of maintaining those in GstPlayer.


What about `PermissionDenied`? This should be in declared in the same place as the rest of the `file` module exceptions.


[vsphere driver] not working with Esxi free license


Hard coded KeyLength


No I meant waiting for the periodic position reporting notification to go backwards instead of waiting 5 seconds and then checking the position


For consistency these should be declared in alphabetical order. And exported in the same order.


Per la VM che si occuperebbe dell'indicizzazione ho aperto un account su ovh
Non appena la feature é pronta potrei far partire una VM KS-1 a 3 €/mese,
che mi sembrano molto ragionevoli
http://www.ovh.com/fr/cloud/instances/

Per le specifiche della ricerca Erni potresti metterle nella sezione wiki o
issues?
Così non ci perdiamo la discussione

In teoria non sarei favorevole sulla ricerca raffinata, direi che con
un'istanza dedicata possiamo cercare in tutti i campi.
E aggiungerei l'autocomplete, ma non oso sbilanciarmi dato che non ho mai
lavorato con solr

/s1m0

2015-06-25 16:04 GMT+02:00 ernitron <notifications@github.com>:

> Jacopo ho visto il tuo codice e mi sembra decisamente migliore del mio...
> direi che dobbiamo fare un merge dimmi quando sei pronto ;)
>
> Mandami una mail all'indirizzo ernitron AT gmail proseguiamo là se non ti
> dispiace per i dettagli operativi.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ernitron/hookii/commit/20d7101bfbd5e8eb754eab3f3d1ecef66d365bf7#commitcomment-11861949>
> .
>


Thanks a lot you are always a gentleman.
On 25 Jun 2015 21:47, "RenderBroken" <notifications@github.com> wrote:

> hmm. weird. I will setup the same to see if it is related or if it may be
> something else causing the no-boot.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/anarkia1976/AK-OnePone-Reborn/commit/cead63987414bba31ac7a504877903ddfa214008#commitcomment-11868984>
> .
>


I agree, the less we store the better :)


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2190 of 2213 tests passed)

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


Per la VM che si occuperebbe dell'indicizzazione ho aperto un account su ovh
Non appena la feature é pronta potrei far partire una VM KS-1 a 3 €/mese,
che mi sembrano molto ragionevoli
http://www.ovh.com/fr/cloud/instances/

Per le specifiche della ricerca Erni potresti metterle nella sezione wiki o
issues?
Così non ci perdiamo la discussione

In teoria non sarei favorevole sulla ricerca raffinata, direi che con
un'istanza dedicata possiamo cercare in tutti i campi.
E aggiungerei l'autocomplete, ma non oso sbilanciarmi dato che non ho mai
lavorato con solr

/s1m0

2015-06-25 21:49 GMT+02:00 Simona Hookii <s1m0n4.hookii@gmail.com>:

> Per la VM che si occuperebbe dell'indicizzazione ho aperto un account su
> ovh
> Non appena la feature é pronta potrei far partire una VM KS-1 a 3 €/mese,
> che mi sembrano molto ragionevoli
> http://www.ovh.com/fr/cloud/instances/
>
> Per le specifiche della ricerca Erni potresti metterle nella sezione wiki
> o issues?
> Così non ci perdiamo la discussione
>
> In teoria non sarei favorevole sulla ricerca raffinata, direi che con
> un'istanza dedicata possiamo cercare in tutti i campi.
> E aggiungerei l'autocomplete, ma non oso sbilanciarmi dato che non ho mai
> lavorato con solr
>
> /s1m0
>
> 2015-06-25 16:04 GMT+02:00 ernitron <notifications@github.com>:
>
>> Jacopo ho visto il tuo codice e mi sembra decisamente migliore del mio...
>> direi che dobbiamo fare un merge dimmi quando sei pronto ;)
>>
>> Mandami una mail all'indirizzo ernitron AT gmail proseguiamo là se non ti
>> dispiace per i dettagli operativi.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ernitron/hookii/commit/20d7101bfbd5e8eb754eab3f3d1ecef66d365bf7#commitcomment-11861949>
>> .
>>
>
>


Noticed that after starting to look at the tests. The plugin for an argument text object that I have been using for vim has ```a``` as the shortcut, which doesn't conflict as angles only have ```<``` and ```>``` as shortcuts as far as I can tell, hence the choice. Not entirely sure what to do here. Whichever is chosen I should be able to make the change in this pull request.


If the OwnerId logic is removed, what is the expected behavior if a user tried to update a record they are not the owner of?


The listener is attached to `['foo', 'value']` is using `swap` semantics. So it shouldn't be triggered twice.

In fact, it shouldn't even be triggered by this change:

```js
      structure.cursor().update(function() {
        return Immutable.fromJS({ foo: 'hello world' });
      });
```

Since value at `['foo', 'value']` is the same before and after that update operation:

```js

const NOT_SET = {};
const before = structure.cursor(['foo', 'value']).deref(NOT_SET);

structure.cursor().update(function() {
    return Immutable.fromJS({ foo: 'hello world' });
});

const after = structure.cursor(['foo', 'value']).deref(NOT_SET);

assert(before === after);
```


Since plurals is never going to be observed and is private:

1. Rename it to `_plurals`
2. This can be simplified to `return this._plurals[thing] || (thing + 's');


Err... radius* not the rotation of a cylinder


You could also make it really private by declaring it outside the class, like `const _plurals = {}`, then you don't need `this`, just `_plurals[whatever]`


Ah got it :)


Note globals.php defines an IS_WINDOWS constant, so you can use that.


::Compare method need to be simplified and changed for better sorting of spectators...


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


Shouldn't these clone to build/host and build/target?


Richtig, normalerweise sollten die beiden Skripte zum Einbinden reichen, da sie die weiteren Referenzen von selbst her holen.
Ich schaue dann gleich nochmal, spätestens morgen kann ich genaueres einstellen ;)


"g" get away in regex that causes error!


They should receive an error from the API indicating the proper reason why
they can't change that value.

On Thu, Jun 25, 2015 at 12:50 PM, msimons1 <notifications@github.com> wrote:

> If the OwnerId logic is removed, what is the expected behavior if a user
> tried to update a record they are not the owner of?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/msimons1/visual-studio-tools/commit/7ffe6a6adaca0041219e1452ec1427ac42a780c0#commitcomment-11869060>
> .
>

--
Thanks,
Dave


Wrap first


1679-DualDuct (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Wrap all these also


Can you use List<GBAlarm> instead of ArrayList? In Java you normally use the interface types (List,Set,Map) isntead of the concrete types (ArrayList,LinkedList,HashSet,HashMap,TreeMap, ...).


The idea was to have the non-inner be what you would want to delete if you want to remove the argument, while the inner is what you want to change if you want to replace it. Assuming a style that allows newlines to break argument lists, for example as below in haskell, you'd want the inner to not select any surrounding whitespace, which is what the implementation does.

```haskell
[ an_item
, another_item
, one_last_item
]
```


Yes, that's what clone_build_tree does.  See its usage message.


Static importy


You can't set as bellow to obtain same thermal effect without change the mass ?
mass = 0.00001
thermalMassModifier = 1000
skinThermalMassModifier = 1000

Main problem is to not disturb craft in mission... Why not create new part AR2000 which apply new mass and with thermal effect, and keep AR202 as is (old mass and no thermal effect).

I submit design a new most beautifull and realistic part ; less colors (almost grey box), tiny led, and with a size related to its mass (100g = cup of coffee). Real computer in space have 5 to 10 kg.


Let’s do an Option that either outputs json or html to stdout.


Check if that's also possible for the other things where we currently just "wait 5 seconds" :) Doing that is not really scalable, with every new test everything will just take much much longer then


Can you please also implement a matching hashCode()? This is necessary for lookup in HashMaps for example.


@azturner oops, thanks for letting me know.  I've added the code back


`this.sanitizeAnswers` doesn't exist


Graphic test


is there a trick to figuring out which files are related to which?
i've been manually crawling through them and looking at the webserver
output trying to find which pages are called when

or is it just famililarity with MVC frameworks?

                  Christopher Chan

On Wed, Jun 24, 2015 at 5:42 PM, jonyen <notifications@github.com> wrote:

> I made some other changes myself, but you'll want to edit these files as
> well:
>
> app/views/devise/registrations/edit.html.erb
> app/views/devise/registrations/new.html.erb
> app/views/home/index.html.erb
> app/views/layouts/_user_widget.html.erb
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/basketbaseb/bible-fitbit/commit/2e1f6b0b9696368ad8dc66037b2271353995a9e9#commitcomment-11851123>
> .
>


Initial commit


Very nice!


Forums in a root (website.com) were unusable. Having a forum in a subfolder seemed to work better (ie website.com/forum), but I still hit 404 pages when I went to /discussions or categories. I guess maybe others might run into this issue as they test...


Thanks @ryanoasis for the tip. Totally makes sense. Cheers!


Well, it depends on your setup. I like to just do everything in the
terminal and use vim, so I typically go to ./app/views and then do "grep -r
'username' *" to find out which files would be relevant. But if you use an
IDE, you would just search for that. To be safe, you can go up one level
and search under "./app".

You would want to have a good understanding of how MVC works, and most of
the changes would have to be in ./app/views, since nothing really needs to
be modified in the controller or the model.

On Wed, Jun 24, 2015 at 7:00 PM, Chris Chan <notifications@github.com>
wrote:

> is there a trick to figuring out which files are related to which?
> i've been manually crawling through them and looking at the webserver
> output trying to find which pages are called when
>
> or is it just famililarity with MVC frameworks?
>
> Christopher Chan
>
> On Wed, Jun 24, 2015 at 5:42 PM, jonyen <notifications@github.com> wrote:
>
> > I made some other changes myself, but you'll want to edit these files as
> > well:
> >
> > app/views/devise/registrations/edit.html.erb
> > app/views/devise/registrations/new.html.erb
> > app/views/home/index.html.erb
> > app/views/layouts/_user_widget.html.erb
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/basketbaseb/bible-fitbit/commit/2e1f6b0b9696368ad8dc66037b2271353995a9e9#commitcomment-11851123
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/basketbaseb/bible-fitbit/commit/2e1f6b0b9696368ad8dc66037b2271353995a9e9#commitcomment-11851818>
> .
>


I'd prefer this in a separate commit (though it can be in the same pull request).  Not a big deal tho.


commit


Hello,

This is  my first approach, I'm working now in clean the lintian warning, also in the build package with pbuilder.
Please check my future commit.

Thanks paisano,
Best regards from Venezuela


OK


Does your mother know that you are so niubi like this?


@eamonoreilly @alstab did some pretty major refactoring, make sure to pull before you work on the ise add-on more.

I got pretty confused when reading through the C# code, so I refactored things to make it more understandable. Should help encourage other folks to help contribute in the future, since it should be easier to pick up.

Only refactored runbooks and assets so far, there's probably more to do.


So   this  is a Stock initiated transfer  and it checks  if DeepFreezer partis the TO part,  and if  that part  is   Full do Nothing... DeepFreeze will revertthe Stock  Xfer


メソッド名を修正


Oh you're right, it should be: https://github.com/TelescopeJS/Telescope/tree/master/packages/custom


So  this is  a Stock  Xfer  or  SM Xfer?or both?
If full, abort the transfer.


Hi GLolol. I made a type. I did use the period in terminal. Regardless, I just downloaded it from the browser and bypassed the clone. All good now. :+1:


It's technically not perfectly redundant, but it works in our case, so it's fine.


This is a nice feature. It will take me a few days before I can test it because I'm currently running a new workstation and GLUT isn't behaving (so text doesn't display in the ND window).

It would be great if there was a way to display feature labels for the "independent" octant mode but I can't think of an easy way to do that without making the text cluttered.


* Make sure additive noise works
* Compare IC vs Q plot with no countermeasure
* Add comments to methods if necessary
* See if it's possible to extract api for simulation, it too much repetition of code.


Ok. I thought I need the migration for the model Meta class to update the permissions, but I guess you just use `syncdb --all`


Thanks I'll scope this out. My first goal was to convert the old lib into something a little more modern and maintainable, so it's not unreasonable that I probably missed a step that your PR fixes.


You could also set up a separate binary repo I guess, and then people could checkout with --depth 0 and get just the most recent binaries.


@thibaultCha can you push the analytics plugin in a separate branch?


delaySecond는 생성자에서도 받을 수 있도록 하는게 좋을듯!


@tyang82 Do you know how to make it work?


There is already a placeholder "Chat" in the xml


test


Hah yeah I was intending on submitting this to crystal, I just put it up on
GitHub to get it off my laptop

Also I'm not sure if it's pushed right now but I ran into a weird crash, so
I have a commit with that. Haven't yet narrowed it down

On Wednesday, June 24, 2015, Ary Borenszweig <notifications@github.com>
wrote:

> @will <https://github.com/will> Awesome!! Maybe eventually we could
> replace std's Benchmark with this. I think Benchmark.ips is much more used
> in Ruby and gives more useful information, and having a good,
> full-batteries benchmarking module right in the std would make it available
> painlessly to everyone. What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/will/benchmark-ips-crystal/commit/cafe206f4ba2d45e04591768d6dd5069b7b9875c#commitcomment-11851665>
> .
>


develop (asparke2) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


OK, I'll change that as well. Thanks.


Canola oil is misspelled.


Hi, Crosswalk project change TextureView to SurfaceView by default in https://github.com/crosswalk-project/crosswalk/pull/3089#issuecomment-111337953.
So there is a issue in  https://crosswalk-project.org/jira/browse/XWALK-4470, Could you please  take a look?


Not sure if that is possible, as for create&tutorial pages there location have been changed.


nice


Fixed typo


Some typo here.


Typo here.


unnecessary fully qualified name


I got these files from stock. Anyway, the naming doesn't really matter.


Please provide your command line/test machine/CUDA driver version so that I can reproduce this. It runs correctly on mario with CUDA 7.0.


This is the first line of the readme file


Right -- I think this was sent close to where it sends up the name - can't check right now, but more or less it tells the server to start the game.


This is my first repository


:+1: thank you :D


LGTM +1


Could you review this? I think this is right based on the php reference you gave me.


ok. I will add time complexity for online LDA. It is proportional to (n_iters * n_samples)


Looks like you already created it  glad it could help. I created this a while ago when all of the escomplex/esprima stuff was up in the air and I needed ES6 support for toying with generators. Feel free to grab whatever pieces you need as I haven't had time to touch any of it in a while.


@Xusine1131  Room::getCurrentDyingPlayer要判断是否返回空


holy shit dude


there is a big chance of our check-style complains about this "...118 * a". The workaround Glima is using is: "118 * (a)"

:(


You shouldn't need to remove the final just for the tests. Use reflection to set the field.


Resizing the images to save bandwidth. Pretty cleaver!


@snatchev I'm usually maintaining the change-log in the Releases section of GitHub.


NP, like u said I PR'ed it with your changes still there. :+1:


yeah apache2 works
![screenshot from 2015-06-25 08 16 46](https://cloud.githubusercontent.com/assets/11634765/8345873/a0e570b8-1b12-11e5-837c-b6e9bf048a4c.png)
![screenshot from 2015-06-25 08 17 17](https://cloud.githubusercontent.com/assets/11634765/8345875/a176b3a2-1b12-11e5-84c5-b66b902f21c9.png)


Não compila.


I'm not sure but what has changed ? I just tried on my computer and it seems it's like before.

Also I want to point out I think for the upcoming stuff (because we will have to focus on particular pages, like main page, download etc). If I do modification on my local server you won't be able to test. Ideally we should have an access to a server somewhere in a "semi public" environment so we can test and both do progress.


add html


Thanks !
Minor nit: "nework" => "network"


Ah, I totally missed the change to `Forward`. How about we introduce a `Send` here, going somewhat in the direction you are heading eventually. So we'd end up with
````
Forward(dec *EthernetDecoder, frame *ForwardedFrame) -> forward(dec, dec.DF, frame)
Send(df bool, frame []byte) -> forward(nil, df, &ForwardedFrame{srcPeer: conn.local, dstPeer: conn.remote, frame: frame})
forward(dec *EthernetDecoder, df bool, frame *ForwardedFrame)
````
That way we raise the abstraction level and confine the df-is-passed-in-two-ways ugliness to one (internal) function.


Why not just 'ShowLV', 'tier'?
It also has LV-Name, LV-ID for pool['Name'], pool['ID']


Exactly, affected are users doing this kind of operation:

    ~]# cp A.service /etc/systemd/system
    ~]# systemctl enable A.service
    ~]# cp B.service /etc/systemd/system
    ~]# systemctl enable B.service

where I assume A.service has After=B.service in [Unit] section.

Because A.service has After=B.service in [Unit] section, a unit object
for B.service is created when parsing A.service. However, there's no
B.service in /etc/systemd/system yet; the unit object results in
UNIT_NOT_FOUND in load_state. The unit object still remains in
m->units hash table until systemctl enable B.service.

As you explain, we can avoid this by systemctl daemon-reload each time
we add or remove unit files in unit file's search path.

I think the problem is that there's no documentation that explains
that (sorry, as far as I know).

Although I never do such kind of operation because I have a common
sense that putting configuration files first and then turning on the
functionality or turning off the functionality first and then removing
configuration files, sadly, there's users with no such common sense in
real world.

So, could you add step-by-step explanation (as a manual page?) about
how to deploy unit files and what we should and must be careful about
there?


Great find!  Thanks for explaining this issue earlier.


I made a rebase&edit for the earlier commit.


int can be 2 bytes, so put the thermistor_resistence above of reference_temperature to avoid holes in the struct


I see. I'll fix this and re-create pull request.


See it from local, that message just means github don't show big changes.


minor, you don't need initialize the variables here (you are already doing in the for())


this is not final commit, it was added for your review,
please guide me regarding how to get it working ?
as there in no cronfile i.e rc\ /2 exists, am I supposed create one and add all these scripts into that file, to be processed after starting ?


List<GBAlarm> was good :-)


@thibaultCha, from the nginx doc:

> $request_time
request processing time in seconds with a milliseconds resolution; time elapsed between the first bytes were read from the client and the log write after the last bytes were sent to the client

http://nginx.org/en/docs/http/ngx_http_log_module.html


> if level == 666 this code will not be evalueted because it's bigger than array size (that is static const defined in this function).
>
> This removed check is redundant. Have i missed something ?

Note the array is initialized with `[SOL_LOG_LEVEL_DEBUG] = "DBG"`. `SOL_LOG_LEVEL_DEBUG` here can be a big number, like 10.  In this case if `level == 6`, level_names[level] can be NULL if there's no name for that log level, it will crash with this change since it's a "sparse array".

I think it was made like this because before at some point it was giving a "LOG-xxxx" names for log levels that don't have a well known name. I'm not sure if it makes sense anymore, though.


Ideally we would have a constant for "alarm" somewhere.


should we check if `node_array` is nil?


so in it's current form this only allows 1 scheduled recycle time?


@kkennedy1  same questions/comments as other commit.


@mpc-hc/developers: Massive failure. This is for next batch. Whenever you want to push.


GetPixelColor gives not color of pixel but data from colorBuffer (used for SSAO effect)
Actually cImage class has no method to access 8-bit image buffer pixel by pixel. You need to add it.
Before access to image8 bufer you need to call ConvertTo8bit function.


Simple and elegant.

I take it supervisor ended up being an overkill? 😊


Presumably mismatched braces means that it isn't obvious what should be selected, which I guess means that object should be aborted and the previous selection should be returned instead. I'll think more on it tomorrow.


would be nice to detete it somewhere


The callback annotation was incorrectly copy-pasted again.


This might provide the general idea how to include the git commit id, but most likely needs to be adjusted:
https://stackoverflow.com/questions/8215785/how-can-i-use-ac-revision-with-git

I think `git rev-parse --verify --short HEAD` is all we need (unless `git describe --exact-match` indicates this is a tagged commit), may be add '-dirty` if ` git diff-index HEAD --quiet` indicates local changes.
(Some changes like additional plugin formats will be ignored by `git diff-index`, but that probably doesn't matter.)


Hi @anasmithiam im using 1.4 since its was released and I didnt experieced any issues (multiple apps).
I just read the posts from the link to the ionic forum and cannot find anything that really bothers me. Did I miss something?
The ES6 and new router stuff is on a completely different page.

Is there anything in particular you are worried about?
Thanks for your time :)


What if there's the decl for IFLA_INET6_ADDR_GEN_MODE but there isn't any decl for IFLA_INET6_FLAGS?


Impliziert dann sicherlich eine neue ntswf-Version (2.3 o.ä.)


Hello @jpellerin how are you?

I was trying to use your plugin, but this method disabled a specific test execution (with selection). What would be the reason to overwrite `get_test_selections`?

Thanks for the plugin, it's simple and effective.

[]'s


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 87](https://teamcity.labkey.org/viewLog.html?buildId=257269&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:21
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


I'm afraid one slash should not be here :)


This controls what gets put in the pypi release, so we need to update it just before pushing out each release.


Mira que no tengan las imágenes copyright.


I'm not sure which commit did it, but it looks like I the configure activities don't show the changed alarm at all anymore. I think the details activity did show the correct alarm, but now, both show fixed alarm times, whatever I change.

May be related to passing alarms around -- you probably need to replace the old alarm with the updated one when returning to the ConfigureAlarm activity.


Thanks for flagging that up Nick. I'll take a look, I thought I was setup
as described but if the spaces aren't appearing something must be wrong!
On 25 Jun 2015 19:20, "Nick Airdo" <notifications@github.com> wrote:

> @arranf <https://github.com/arranf> Thanks for contributing fixes and
> code. We really appreciate this sort of involvement. Would you take a look
> at your VS settings and see if the "Insert Space..." options are checked
> (as mentioned here:
> https://github.com/SparkDevNetwork/Rock/wiki/Coding-standards)? I've
> noticed that the spaces are missing from your recent code.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arranf/Rock/commit/8760d2559048c9282636f2d475ca31bc26aa2a43#commitcomment-11867310>
> .
>


Changed and another commit sent. I will post on the forum . Thanks


Added clickable text to top banner to access the "about page"


Bin mehr für `key?`, weil `include?` auch auf Strings und Arrays geht und man da erstmal wieder eine paar Zeilen drüber schauen muss, was `result` eigentlich ist. Und semantisch unterscheiden sich alle diese `include?`-Dinger ja auch noch.


The filesystem is smart, so it works, but you're right, it shouldn't be there.


Oh sorry didn't see this before i pushed my commit. But i looked around and it seems that these files are user-specific and generated by Visual Studio / MonoDevelop every time you open Unity.
So it seems there are no reason for having these in git


What if that zip code was invalid and therefore the record was unique? I'm not seeing where the invalid zip code gets removed under this scheme.


Oooh, but the extra slash has a Twin apparently :D


mount先は/apiとしてください


Added a check box to enable artifact rejection and a spinbox for the rejection value


It could went to core. We would have hacks package with version ranges.


Kein neu eingeführtes Verhalten - aber ist es dem `@task_callback` erlaubt, ein `nil` zu liefern? Oder ist das eher als Fehler anzusehen, und `nil` ist nur ein valider Zustand, wenn es keinen `@task_callback` gibt?

Neu eingeführt ist hingegen, dass der Task-Callback gezwungen wird, einen Hash zu liefern. Aber das ist ja mit dem Commit Absicht.


Does the memory not get freed by deleting row_pointers later on?


bummer... this requires pretty decent boost version


It turns out that tempo2 has two names for ecliptic coordinates:

RAJ: ELONG or LAMBDA
DECJ: ELAT or BETA

I did not see a way to properly obtain the parameter name that was in the parfile. So now libstempo defaults to ELONG/ELAT, even when LAMBDA or BETA was in the parfile. Is that desired behavior? It sort of violates the principle of least surprise, but I do think it is acceptable.


Yeah, I don't like moving the declarations to `include/linux/kexec.h` either.


{} x 3


Example Usage decorator method is incorrect. Decorator should be `@chassis.util.parameters.parse_dict(...)`.


I will add buffer for image8. Maybe we can rename the other Parameter to prevent ambiguity.


How does Python's super() work with multiple inheritance?
http://stackoverflow.com/questions/3277367/how-does-pythons-super-work-with-multiple-inheritance


'installing'?


No, because row_pointers is array with pointers to first pixels of image lines. Releasing memory for this array you just delete array of pointers, but not image buffer.


Thanks for the reply, Dan. I think I will write a "floorlog" function following you earlier suggestion, because floor(log_p( )) is used a few times in the package.


I would suggest ``is_editable`` as an alternate name for this property. "locked" has not been part of our vocabulary to date.


Are we saving their access token in a file? Any worry on the security of this? Someone steals the file they have access to the subscription


It would be kind of awesome if all these images (Dropbox, Github, Dataverse, etc.) linked back to the homepage of each project.


`gather` hat eher die Bedeutung von "einsammeln". Gemeint ist aber eher "herausfinden".


@jodoglevy Could you encrypt it using DPAPI, using Lee Holmes' method?

Cheers,
Trevor Sullivan


Even better (just a single git command:

```
$ git describe --tags --dirty=+
1.8.0-jumbo-1-2731-g49e5ba9+
```

I guess that says 2731 commits ahead of 1.8.0-jumbo-1, commit id 49e5ba9, plus some uncommitted local changes.
We need to replace (part of) the current version string with that git output (unless git fails)

```
 $ ./john --list=build-info |head -n 1
Version: 1.8.0.6-jumbo-1-bleeding
```

Apparently, 1.8.0-jumbo-1 was the last tagged version.


Hat so eine `WorkflowExecutionCollection` direkt ein `any?`


What? Why are we using a non-standard time format in our API interfaces? This should be a must fix defect for v2 of the API.


This should probably be upstreamed (whichever is the current upstream, sorry but I don't remember)


@gibiansky This was the culprit.


I wouldn't put this in the actual MVP, it's just here for PoC testing to prevent constant logging in. I wasn't *too* worried about it since the token will expire before long, but we could remove this even for testing if you want.


I did not have a `msg_type` here before.


The .3ds build results in a 'card has been removed error'. This is expected though as GW artificially limits service access. No big deal as I doubt most GW owners will need the feature.


-1, the code is required to be proof to holes in the enum and array. As Lucas said


Yup, fixed in 75d73fa0ebd6ca84d174822f935701f12f1a90dc


Die Spec ist hinreichend bedeutungsfrei. Sie ist äquivalent zu

`File.read("lib/ntswf/client.rb").lines[60..62].map(&:strip) == ["def active?(workflow_id)", "domain.workflow_executions.with_workflow_id(workflow_id).with_status(:open).first != nil", "end"]`

Lies: Sie testet kein Verhalten, sondern eine ganz konkrete Implementation desselben.


You can short it by declaring the constants as float/double. Just add a ".0" in them


Awesome, thanks for the quick acceptance! This will come in handy soon.


Not for now, but shouldn't we have a way that all failures are automatically logged if there's no errback for it? Another related question: what happens to consistency of email data if one of these is not created correctly, the failure is logged, and the app continues to work as if nothing had happened?


Why so? I've updated them to make sure it works with the sbt build for this 3rd-party.

They will disappear when we make it a proper dependency (it's now ready to move to a Maven/Nexus).


Related to #12


@skeptical thanks, fixed in 861b8b2


The API documentation says the dates have to be in this format:
Format is yyyy-MM-dd’T’HH:mm:ss.SSSZ

Before the change, I was getting this error:
/Users/sorayasherwin/.rvm/gems/ruby-2.2.0@linkApiRuby/gems/json-1.8.3/lib/json/common.rb:155:in `parse': 757: unexpected token at 'java.text.ParseException: Unparseable date: "2015-06-23T19:11:13-07:00"' (JSON::ParserError)


Weil?


What might be a big deal, however, is that every .3ds built with ctrcommon/citrus will likely get the same error.


could you update snap to the current version? Since there is a bug in old version and cause termite for using samtools. Snap group fixed that bug in 1.0 beta0.16.
https://groups.google.com/forum/#!topic/snap-user/0TSEvC0NP7w
Thanks


No need for an ifdef around them. As long as you don't use the declarations, it'll compile and link just fine without problems even if they're not actually defined anywhere.


Field is never used, please remove


We should discuss this at next weeks sync.


* Add comments on methods
* Extract api for simulation
* Fix Matlab plotting problem


@oamike I didn't find any existing format to match the API's so I created my own.


Shouldn't it be "ifdef" instead of "if !"?

Is the HAVE_... Always defined?


Maybe we want to use something like this, so we avoid this problem wherever it might appear?

```python
class caseless_dict (dict):
    def __setitem__ (self, key, value):
        dict.__setitem__(self, key.lower(), value)

    def __getitem__ (self, key):
        return dict.__getitem__(self, key.lower())

    def get (self, key, default=None):
        return dict.get(self, key.lower())

    def has_key (self, key):
        return dict.has_key(self, key.lower())
```

From: https://docs.python.org/2.3/lib/optparse-extending-examples.html


Konkreter: Uns reicht auch eine wesentlich ältere Fassung ...

Wir gehen natürlich davon aus, dass keines der einbindenen Projekte etwas dagegen hat, sein aws-sdk-v1 auf einen recht aktuellen Stand hochzuziehen. Aber sollen wir das auf diese Art "erzwingen"?


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 7](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=16&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Exit code 1 Build time: 03:08:59


Wouldn't simply surrounding the macro implementation with { } be enough without the extra loop?


бля нихуя не понятно))


Doesn't DECORATE have a C definition, or is this an exception for the 32 bit x86 case?


Thanks! :) I was able to disable it and use the backlight. I was also able to boot up like normal. So with that, I am not sure if that is the cause. Are you running with Transparent Compression Disabled on your current release? I know I had a no-boot situation awhile back because I had it set to disabled. But CM made changes so I dont think it matters anymore but its an idea. Your code looks fine but thats how Kernel work goes. . . lol. Let me know if you need anything.


it might be a way, but I'm not sure if it will make the code more obscure.
app will continue to work, yes. sometimes an experienced "hang" comes from an unhandled failure, and not having a log is quite frustrating. I should start a crusade to add errbacks everywhere.


if i modify in foreign files I normally try to follow the convention in this file.
So I use no m_ and also rename methods xys_something similar to the style existing.

To be honest: the file could be converted completely into kodi style, but must not.
To be respectfull against the original author, we could leave it as it is....????


PHPCS: Expected 1 newline at end of file; 0 found


Class ServerController does not have a Docblock comment.


Method ServerController::index does not have a Docblock comment.


just forgot to close first section


Add the `org.testng.internal` package to the [illegal imports rule](https://github.com/jclouds/jclouds/blob/master/resources/checkstyle.xml#L60)?


Okay, the check might have made sense in the past, but now those if()s will always be true.


This safe accessor looks like it could be a generally useful addition to the ``MapFeature`` superclass.


change .gitignore.txt to .gitignore


Is bitbucket down, or have you removed your settings and framework/base repos? https://bitbucket.org/DipDro420? I am trying to sync your repos to build Cyanide for S3 LTE but get sync errors with this 2 repos. And syncing them from Cyanide repo gives me build errors? Do you have any info or help for me?


Very nice.
Try PHP 7 also.
WP core emits on `E_DECRECATED` warnings.


I'd change this to:

    to indicate whether we're interested in being notified right after
    the mdoc has been written (as it's the first doc to be written, this
    is faster, though potentially unsafe).


今回はhtmlをroutes.rbを元に取得したいのでrake routesである程度存在するURLを確認したいのです
なので
```mount API => '/api'``` にできないでしょうか？
試しに確認してみたら
https://github.com/systemlink/e-sche/commit/e403d950a0fde51b596bd1eda9127c912aaaa2c8#diff-8812cec49240789009b71b1ff58b4451R2
を削除できそうに見えました


In general, allocating more memory here is not a good idea. In SavePNG function I have used setting of row_pointers to existing memory objects (pointing to first pixels of every line of image)
```
	png_bytep *row_pointers = new png_bytep[height];
	for (int y = 0; y < height; y++)
	{
		row_pointers[y] = &image[y * width * 3];
	}
```

In this way I have saved a lot of memory, especially when image is very BIG (line 16000x12000). Now you are creating another copy of whole image.
It's only reasonable to create new image buffer when you are saving image with alpha channel, where you need to combine pixel colors with alpha.


wat... Maybe just replace it with ``static int16_t zero_line[STRIPE_WIDTH];``, since all of it is default-initialized to 0 anyway.


actually there is a caseinsensitive dict already in mail.utils, I just used this one-shot because of the short scope


Is this really better as lookup table instead of as expression?


Great job finding and fixing this.
My question remains... why are we using this non-standard time format? do we really need accuracy to one-thousandth of a second? My preference would be to fix this on the API side - use and iso format.


roger, thanks.


Missing licence


It's incomplete assembly without 32-bit version.


I don't understand this build fix. It's compiling arch/x86/kernel/crash.o, which should only be included in the build if CONFIG_KEXEC_CORE is enabled. But then this branch of the #ifdef should not be compiled at all.


How could you?!


:+1: Thank's for fixing!


I think the only way it could happen is if

PAL_InitializeCoreClr(exepath1);

Create a thread that does GetModuleFileName(NULL), GetModuleFIleName(exe module handle) or the code in debug.cpp (DebugBreakCommand()) gets called at the same time another:

PAL_InitializeCoreClr(exepath2);

Is called. I don’t think that is typical.  Usually it goes from null to a string (PAL_InitializeDLL and then PAL_InitializeCoreClr(exepath)).

What would reduce the race conditions is to grab the module lock around this code that frees/sets the exe_module.lib_name, but the lock doesn’t get initialized until after that code in Initialize.

mikem

From: Jan Vorlicek [mailto:notifications@github.com]
Sent: Thursday, June 25, 2015 12:21 PM
To: mikem8361/coreclr
Cc: Mike McLaughlin
Subject: Re: [coreclr] Fixed the PAL_Initialize* order problem. (5562242)

@mikem8361<https://github.com/mikem8361> There is a lock around the initialization, but not around the usage of the exe_module.lib_name. What I meant is the case when the PAL_InitializeDLL call already returned and the thread is trying to do something with the exe_module.lib_name while another thread is in the middle of the call to the PAL_InitializeCoreCLR.

—
Reply to this email directly or view it on GitHub<https://github.com/mikem8361/coreclr/commit/556224275d2f7701b3ea25e35f6873afd5c462ca#commitcomment-11868530>.


@oamike since I am new in this API world, what do we need to do to request this change?


nit: Should this be added in a different file, because when AnalyzerTestBase is added, this file will be too long to read, and better code organization.. If it's only this one method that differs, then it's ok i guess.


@kdandola I also thought this was a very important topic in class and I think your use of Uber as an example is extremely relevant. Prior to starting class, my natural inclination would have been to build a website first for any web-based service needed. But you’re right, in the case of Uber, there really is only a mobile application and so building the mobile app must be the first priority. I am now much more cognizant when it comes to thinking about how a web-based service might be used and then considering that context when deciding which digital elements to develop first.


teste


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


I like it, changing it...


Please document this property. It's not available in the doc.


I'm not really a big fan of commits that introduce "orphaned" functionality like this. I'm OK with removing the old stuff in another change, but at least implement a common interface or wire a portion of existing functionality through this to demonstrate that it's headed in the right direction.


FYI, this made the keyboard avoiding tableview quit working at all our apps in 8.3. Not sure why 8.3 would have fixed keyboard avoiding for you, but it did not fix it for our apps. Figured you may care to know. If not then feel free to ignore my comment :)


My haval fails also.  I simply force install it.


LGTM


I think you are mixing the parsing with the execution of actions.

I suggest something like:

```public IncomingMessage(String message) throws IOException {
         JsonReader reader = new JsonReader(new StringReader(message));
         reader.beginObject();
        while (reader.hasNext()) {
             String name = reader.nextName();
             Command command = Command.forName(name).parse(reader).execute();
       }
}
```

Then you create an enum Command with an instance for each kind of message.

In the parse method you put the parsing and assign to instance members of the command. In the execute method you put the logic to be executed (send sms,  dismiss,...)


This is the project ID (It is similar to an UUID).


formatsolutionwithanalyzersasync is not asynchronous here.. Why not await..


Command acknowledged


I've tried to write C versions in such a way that it closely corresponds to SSE/AVX version. And compiler likes that and generates vectorized versions of C functions (sadly not for all).


I tested the method and it seems to save the post ok.  In the rails console, I can find the Post and the associated Vote.


Thanks! :)


This revision is clean.


Hi Vivek

is it possible to talk to you over webex?


@LPTK
Can you investigate why `as` is producing a compile error here?


Yes, equals() and hashCode() are "pairs", you always add them together.


This currently doesn't work on mac as bit-crusher not supported.
Error message in Error log.
exception in GrafDef_Load: UGen 'Decimator' not installed.
while reading file '/Users/rbn/sonic-pi/etc/synthdefs/compiled/sonic-pi-fx_bitcrusher.scsyndef'


This makes me not like that rule.


```env['api.tilt.root'] = Rails.root.join 'app', 'views', 'api'```
の誤りですかね？
あとここを直してjsonを試しに取得してみたら

[{"event":{}},{"event":{}},{"event":{}}]

とjsonデータが返ってきました。
実際は

{"event":{"title": "タイトルだよ", "note": "内容だよ"}}

みたいな感じで取得できるようにお願いします


If you don't use a HashMap, Hashtable or anything else using hashCode(), it all works as desired. But as soon as anyone does, he'll run into problems. That's why you always supply both of them so that they are symmetric.


do we absolutely need to deal with the empty string case?


That was an oversight, thanks. In an earlier commit, when the method didn't have a body, I couldn't mark it async because it gave a compiler warning, and I couldn't await it since it wasn't async. Then when I added the body I forgot to await it. Good catch!


Do you want to move line 739-743 (and line 727-730, 734-737?) inside this '{'?


I believe that as of https://github.com/babel/babel-loader/commit/51336323b9bd938826d2ff99fdbdb4c9cc33b2e5 this comment is now incorrect.


Yeah I don't know if I like this rule, the one about typing everything even where the type is inferred.  It's like, if someone disallowed the "auto" keyword in C++, which is extremely great.  Let's turn it off.


I agree, I'll move it to a separate file.


Wahoo :D hey les mecs, on a atteint les 300 commits :D finalement c'est presqu'un gros projet hein !


You should also change the types in the include/ directory, which define the libdaala* API.


Command acknowledged


Hi dbongs,

Sure, Please let me know your preferred time tomorrow and I will send you the meeting invitation. Thanks.

Thanks & Regards
Vivekjyoti Pramanik


xrange()


Ok

Dave Carroll

On Jun 25, 2015, at 1:26 PM, msimons1 <notifications@github.com> wrote:

We should discuss this at next weeks sync.

—
Reply to this email directly or view it on GitHub
<https://github.com/msimons1/visual-studio-tools/commit/7ffe6a6adaca0041219e1452ec1427ac42a780c0#commitcomment-11869694>
.


Please add a comment as to what interface you're implementing.


curious why not just: prefix ++ "." ++ toStrict . decodeUtf8 $ instanceId   ?


It is indeed. This is what happens when I merge things in the train!


Interesting idea: we could change the interface to let people decide what they want to take and what they want to ignore?


comment on this line goes here


Hmm, no, it's part of the regular Activity lifecycle. But maybe onPause() and onResume() would be more sensible.


when do we get this with brew install package?


preferably using code: `var _ interfaceType = myType{}`


Hi Vivek

we can schedule for 5pm if the time is convenient for you


Great :-)


Absolutely!


I like what you've done here!


non-private members before private members?  And before the constructor?  Is the order configurable?


I would keep the part of the comment that tells you that the `NodeID` is by now initialized. It's a bit subtle.


Ive committed it now, just trying to work out how to add a new commit to that pull request.  Or should I make a new one?

> On 25 Jun 2015, at 11:45 pm, Nicolas Dorier <notifications@github.com> wrote:
>
> you forgot to include TokenizeTask.cs in the commit.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bitcoinbrisbane/NBitcoin/commit/416df5842ff605850ee77f67e7b551e996131452#commitcomment-11861460>.
>


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 88](https://teamcity.labkey.org/viewLog.html?buildId=257302&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:30
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


asdfasdf


Lists are banned! Will remove. Thanks!


downloaded the whole file "class.Repository.php" now and replaced the existing one by the one of commit 894fdae. Again, no errors (neither within GUI nor within logfile) and Sync works.


fwiw the convention is such in ruby, too


Tuples to be jettisoned, sir.


I can't seem to find any callers of this function. Maybe it should be removed?


`this._modal` is very generic, it would be good if the name of the variable gave a hint about what this modal is used for.


What?  You had to pull this out for the linter, or was this a decision you made?  What rule was violated here?


empty line here?


Question of opinion... Do you think this is worth a new patch, or can it wait until the next feature/hotfix?


Yes. You are right. Thanks!


Can you explain that it's overriding the attribute from the A.LayoutBuilder instance? It feels like it's an attr on this instance when you just read this.


Better(?)
```python
from ..core import units, flags, util
```


1679-DualDuct (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Fork CM Device Tree and Do A BringUp....


This should be "Removes" instead of "Remove", since that's the pattern for method docs.


@Kubuxu Haven't solved the problem with Core not depending on anything, which means we can't access LaunchClassLoader class...


Done in 7787b50067f3a7e2c4b0a15cdfa8bf9a4cbc2b07.


Aha, that's good to know, I thought it was some OSX or clang issue. But this commit was still correct.


Does this fix the black screen issue?
On Jun 25, 2015 11:03 PM, "Kacper Michajłow" <notifications@github.com>
wrote:

> @mpc-hc/developers <https://github.com/orgs/mpc-hc/teams/developers>:
> Massive failure. This is for next batch. Whenever you want to push.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpc-hc/mpc-hc/commit/6abc16717c1cb8f9a4ab1440307054485da1e1a4#commitcomment-11869281>
> .
>


I think it'd make sense to cut a 5.2.2 with the properly removed dependency, but not that big a deal either way.


Nice. Hopefully with repetition, this will become more natural.


Oh. Forgot about that.


@sunaku Thanks for making up for my laziness.


I do like enforcing the return type on functions, that seems like it increases readability.


Perhaps elaborate on what it does.  Not just when it's called, which is already implied by it's name.


Hi dbongs,

Sure, I will setup the meeting and send you the invitation. Thanks.

Thanks & Regards
Vivekjyoti Pramanik


Oops. That's right. On it.


Gahh. Yes, we will refactor that beast.


Shouldn't this also affect the high-jump?


Yes, that's it.


Do we also want fancy indexing and boolean indexing, i.e. return iterator for specific frames? Might be neat.


I think this is not correct because prev_seq_point will point to it.seq_point, which is overwritten every time seq_pointr_iterator_next () is called.


Using a `do...while` loop allows you to use the macro like a function. Just using braces doesn't work if you use it in an `if` statement with an `else` clause:
```c
if (condition)
    alCheck(something);
else
    something_else;
```
expands to
```c
if (condition)
    { something; sf::priv::alCheckError(__FILE__, __LINE__); }
    ;   // << Note this semicolon
else
    something_else;
```
which is a syntax error.


This is by far, the ugliest instance of this.  But if your function is that complex, shouldn't we have a named type for it?


I've noticed that you have configured the bleeding edge of the build tools. Perhaps use the latest stable version.


Yup!


We might want to discuss this, "Variables before functions" is running afoul of our "bound" functions which are apparently considered variables. I also personally prefer private before public, although I don't think I feel strongly enough about it one way or another to even enforce that rule.


Here's the link you sent me that describes the problem for future reference:
http://comments.gmane.org/gmane.linux.network/95427

Let's hope this never comes up again.


There wasn't a duplicate entry, it's the only one. I'll put it back.


It did cross my mind, but I don't understand why people would want irregularly spaced frames.  Then again, it's harmless to add, so why not.


Oh man... I can't get one right today... I had removed that already, but I totally messed with the tags . Thanks :)


I thought the field instance is now used.


This fixes an issue when removing a Volume. The Docker API has changed but Kitematic wasn't adapted.


Document that `self._f` is set as a side effect. It might become pretty difficult for people reading the code later to figure out where this crucial event happens.


hi @rbnpi - this is only an issue if you've compiled Sonic Pi manually and haven't pulled in the extra SuperCollider ugens: http://sourceforge.net/projects/sc3-plugins/


And in C# too.


could just
```python
return func(self, *args, **kwargs)
```


Oops


Do we really need to tests all these headers?

By the way I would go wit the following approach:

#ifndef IFLA_INET6_UNSPEC
#define IFLA_INET6_UNSPEC 0
.....

So we don't need to test anything on build system and have to maintain these checks for ever (not that maintaining it should be a great effort, but it would be more straightforward IMHO)..


We will want to do something like what libvpx does for platforms that don't have stdint.h. In fact, you can just reuse the code that libvpx has for this purpose:

https://github.com/webmproject/libvpx/blob/master/vpx/vpx_integer.h


Ack, my bad. Also thrills me how did it compile fine all the time.


@githubruno and nope, it did *not* take me just 10 minutes to write this code.


Also our_id...


this interface should be defined by mithril.


```= {}``` can go there


please ignore commit...


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


Sadly, the linter doesn't give you many options yet.  It's those three settings only.  We could write our own rule or turn any off.  I'd be down with turning "variables-before-functions" off.


Maybe log the err also.


Unfortunately, this broke tests initialization on systems without GIS libraries:
```
  File "/home/tim/django/tests/gis_tests/models.py", line 11, in <module>
    from django.contrib.gis.db import models
  File "/home/tim/django/django/contrib/gis/db/models/__init__.py", line 5, in <module>
    from django.contrib.gis.db.models.aggregates import *  # NOQA
  File "/home/tim/django/django/contrib/gis/db/models/aggregates.py", line 1, in <module>
    from django.contrib.gis.db.models.fields import ExtentField
  File "/home/tim/django/django/contrib/gis/db/models/fields.py", line 1, in <module>
    from django.contrib.gis import forms
  File "/home/tim/django/django/contrib/gis/forms/__init__.py", line 2, in <module>
    from .fields import (GeometryField, GeometryCollectionField, PointField,  # NOQA
  File "/home/tim/django/django/contrib/gis/forms/fields.py", line 4, in <module>
    from django.contrib.gis.geos import GEOSException, GEOSGeometry
  File "/home/tim/django/django/contrib/gis/geos/__init__.py", line 5, in <module>
    from .collections import GeometryCollection, MultiPoint, MultiLineString, MultiPolygon  # NOQA
  File "/home/tim/django/django/contrib/gis/geos/collections.py", line 8, in <module>
    from django.contrib.gis.geos.geometry import (
  File "/home/tim/django/django/contrib/gis/geos/geometry.py", line 18, in <module>
    from django.contrib.gis.geos.prototypes.io import (
  File "/home/tim/django/django/contrib/gis/geos/prototypes/io.py", line 38, in <module>
    wkt_reader_destroy = GEOSFunc('GEOSWKTReader_destroy')
  File "/home/tim/django/django/contrib/gis/geos/prototypes/threadsafe.py", line 39, in __init__
    self.cfunc = getattr(lgeos, func_name + '_r')
  File "/home/tim/django/django/utils/functional.py", line 205, in inner
    self._setup()
  File "/home/tim/django/django/utils/functional.py", line 338, in _setup
    self._wrapped = self._setupfunc()
  File "/home/tim/django/django/contrib/gis/geos/libgeos.py", line 57, in load_geos
    '", "'.join(lib_names)
ImportError: Could not find the GEOS library (tried "geos_c", "GEOS"). Try setting GEOS_LIBRARY_PATH in your settings.
```


Cool we definitely need it. If you can push my lite builds compilation
patch we can do have it tonight (I'm not at home)
On Jun 25, 2015 11:57 PM, "Kacper Michajłow" <notifications@github.com>
wrote:

> Yes, that's it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpc-hc/mpc-hc/commit/6abc16717c1cb8f9a4ab1440307054485da1e1a4#commitcomment-11870312>
> .
>


nops... it's an sparse array...


I'd be down for adding it.  What should I call it?


Looks like you don't need this import.


Perhaps add an `assert self._f.is_closed()` or something similar?


yo, I've that change locally... plan to apply once nocrypto gets another release..


I seem to be missing `LegoBaseQuasiExp`. Where is it defined?


The commit is fine, things would not run if it was not. I was just informing you that perl's Haval mod sux.


Ok I will push it.

@Underground78 @alexmarsev Also this if you agree https://github.com/kasper93/mpc-hc/commit/7a2737ade9111e88091569db0e475606daa0b103


I've used the f101_short as a template. Could you review this and tell me if the conditions (regn in a subset and after 2012) should be changed? The long view has many rows, but the short one does not (no regn is in the given subset).


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


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


This was already added in d7f14ae3757a7dd9ae79497359529f4775a6dcb2, but it probably does make more sense under this heading.


Oops, sorry, I didn't see this remark. I'll change it so that your text is used in the mouse operations section. The feature itself is very convenient, by the way!


Why many changes? It is strange...


`LoweringLegoBase` already extends `BaseQuasiExp`.


Our intention was to use a utility function under `search' called ` _find` which, in addition to returning the searched-node, also returns the the previous node. This allowed to reuse the same logic for both the `search` and `remove` methods. `search` does not require the `left` object for its return, so it is dropped.


`protect_from_forgery` is for CSRF, not CORS. Are we good with turning off CORS and CSRF? That seems... dangerous.


> ```
$ git describe --tags --dirty=+
1.8.0-jumbo-1-2731-g49e5ba9+
```
>
> I guess that says 2731 commits ahead of 1.8.0-jumbo-1, commit id 49e5ba9, plus some uncommitted local changes.

Awesome, this string is really good. It solves some of my concerns with using a commit hash, as it tells we have local changes. Also, the "commits ahead" and "last tag" are good things.

> Apparently, 1.8.0-jumbo-1 was the last tagged version.

I usually only tag releases.

The remaining question is where to put this. We can't put it in configure.ac since the dependecies are so good these days, I usually just pull and `make -sj4` without re-configuring. So it has to be in Makefile. However, it would need to

1. Always be updated (so always re-building some .o file). Perhaps simply update some version.h that is sourced by listconf.c - listconf.o is probably very quick to rebuild.
2. Not clobber with git. The file created must be in .gitignore
3. Silently handle the situation we don't have git.
4. Silently handle the situation we have git but our tree isn't a git clone.

3 & 4 need to fallback to something and still produce that version.h (or whatever we chose)


Can you also update the comment?


:D sorry!


not sure when environment would ever be set to localhost, default config is set to dev


hm... why we do not need to hold the lock in this case?


This isn't "Orphaned", this is my active development area.  It doesn't hurt anything by being in the code, even if it's pushed to master.

Perhaps if the transitional code was more complex during transition you might want to use a development branch, but this has been structured in such a way as to have no appreciable side effects.  It's not worth that overhead.

The goal of these transitional PRs is to cut down on the length of Code Review; I feel that this is very effective.


woah :D


why not just term?


Based on this note, should I explain the previous function `_clickColMoveTarget` either?


Wait, wat? What's the issue here?


That does not seem right! If I remember well the batch with modules that did not have functioning SWR measurement circuitry was before the Plus came out!


On 25 Jun, 2015, at 13:59, Richard Gowers wrote:

> It did cross my mind, but I don't understand why people would want irregularly spaced frames. Then again, it's harmless to add, so why not.

Because @dotsdl is thinking about how to access frames by specific times (1) or when a certain criterion is true (2) and fancy indexing might make it possible to write this in a very natural way.

(1)

   frames = convert_times_to_frames(timepoints)
   for ts in u.trajectory[frames]
      ....

Actually, for the previous case we might even want something like

  for ts in u.trajectory.bytime[timepoints]
      ...

(2)

   for ts in u.trajectory[something_is_true_for_this_frame]:
      ....


I disagree - I can't effectively review this code without seeing how it integrates into existing facilities. When reviewing the eventual integration I'll need to page this commit into my brain in addition to the integration commit itself.


Is necessary to call first the fr_syserror() to be kept the errno state.


Add mee


why the reader side will update the term in this test case? Can't we control that? Then we do not need to introduce the new api, right?


`bewest@hither:~/src/veneer$ RLWRAP_EDITOR=vim rlwrap  -pBlue -m -q'"' node repl.js  examples/sample_scheme.mkrn examples/scheme_use.mkrn `  Used to produce errors about undefined not being a function.  When I re-order these statements so that `e1 = eval0(...)` occurs here instead of earlier, everything seems to work, no errors are produced, but neither are any answers.

The other examples still seem to work the same way, not sure what happening.

```bash
bewest@hither:~/src/veneer$ RLWRAP_EDITOR=vim rlwrap  -pBlue -m -q'"' node repl.js  examples/sample_scheme.mkrn examples/scheme_use.mkrn
#t#t#t#t#tfinished everything null []
bewest@hither:~/src/veneer$

```


Oh haha, cool then. :+1:


Not my job
On Jun 25, 2015 4:55 PM, "Sanyam Jain" <notifications@github.com> wrote:

> Fork CM Device Tree and Do A BringUp....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/device_motorola_titan/commit/02219f6a12c396e4603fa3304c0dbd6d913321ce#commitcomment-11870259>
> .
>


I'll put it back inline.  Pulled it out to sort out why the linter was complaining about spacing.


xfree (key);
xfree (val);


Yes, why not. That would be the first and only usage I have for G+. The only other being occasional TRCL related topics that you bring up. Expect a junk/test post or two because I never used G+ so far (yes, unbelieveable!)…


Other than the licence, it runs great.


@Zidane If you're happy with these changes it can go straight into master. Otherwise I'll open a PR to discuss further.


Deliverables, maar verder heemaal mooi!


s/to use//


s/project is just/repo only contains/


s/it's/its/


so lets say i rename field1 to field2, and my new code starts writing new data to field2. But the data for field2 (which was field1) is still in the old field1 for earlier records.


s/so that you will know what will need to be defined/to document what must be defined/


Please move this as constant out of the loop.


How do I re-enable this?


reasoning behind sending to global sink rather than player's sink?


s/As well as containing/Besides the inventory,/


LOL

On Thu, Jun 25, 2015 at 2:43 PM, Michael Shoemate <notifications@github.com>
wrote:

> Command line bullshit. Moved central node calculations out from beneath
> heavy loop. Another optimization.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Shoeboxam/Texture_Synthesis/commit/e0122ac5438cdd202894eed5e1bf9fb89bec65fa#commitcomment-11867813>
> .
>


s/into/in/

clarify "these" from line 22 vs "this" from line 23: say "secrets repo" or "main repo" or something


I strongly agree with Matt - we need more small commits with well-defined interfaces and fewer mega-PRs. If you can't foresee how the new functionality will be used then that may be a problem with the docs or tests, but the answer is definitely not to require that more and more stuff must be included in the same branch before merging.


Thanks Hykem!
Really exited right now, gonna try it right away! :)


s/basic//  s/looks like this/is/


Because I get the error:

```
src/Canteven/Metrics.hs:57:23:
    Couldn't match expected type ‘Data.Text.Internal.Text’
                with actual type ‘[a0]’
    In the ‘prefix’ field of a record
    In the expression: options {prefix = newPrefix}

src/Canteven/Metrics.hs:59:17:
    Couldn't match expected type ‘[a]’
                with actual type ‘Data.Text.Internal.Text’
    Relevant bindings include
      newPrefix :: [a] (bound at src/Canteven/Metrics.hs:59:5)
    In the first argument of ‘(++)’, namely ‘prefix’
    In the expression:
      prefix ++ "." ++ (toStrict . decodeUtf8 $ instanceId)

src/Canteven/Metrics.hs:59:35:
    Couldn't match expected type ‘[a]’
                with actual type ‘Data.Text.Internal.Text’
    Relevant bindings include
      newPrefix :: [a] (bound at src/Canteven/Metrics.hs:59:5)
    In the second argument of ‘(++)’, namely
      ‘(toStrict . decodeUtf8 $ instanceId)’
    In the second argument of ‘(++)’, namely
      ‘"." ++ (toStrict . decodeUtf8 $ instanceId)’
    In the expression:
      prefix ++ "." ++ (toStrict . decodeUtf8 $ instanceId)
```


paste this in your syntax sttings:

{
"draw_centered": true, // Centers the column in the window
"draw_indent_guides": false,
"trim_trailing_white_space_on_save": false,
"word_wrap": true,
"wrap_width": 80  // Sets the # of characters per line
}


maybe add a quick reminder of how group_vars/*.yml map to the main repo plays/inventory items


Well, I didn't want to add a `.` if `instanceId` was empty.


There are web browsers plugins like Postman that address this on the client side.


I've removed the ugliness for now.


If set up this way, -i is not necessary for ansible-playbook to find the repo.  However, you can choose your own setup and point to the secrets repo with -i or <name of env var here> if you prefer.


I think you should consider write compat code for PY3 too.

`map` is an iterator in PY3. 'range' = 'xrange'.


Why did we hard-code a `rem` measurement here? What was the intended purpose?


nitpick: why not replace all of these calls with `_memupdatemap()`?


I think we should leave it on for now.  Overall, it's a good practice to do this.  It might be worth adding 'treat-bounded-fn-as-fn' to linter itself.


You can use the existing MultipliedValue and effect.setText().


Yea, I should make a custom function :)

On Thu, 25 Jun 2015 23:24 Nukem <notifications@github.com> wrote:

> nitpick: why not replace all of these calls with _memupdatemap()?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/x64dbg/x64dbg/commit/dc1ad6b608ad3a44076a5cf3f6e3195e82d85de7#commitcomment-11870796>
> .
>


s/SHA256_DIGEST_SIZE/SHA256_DIGEST_SIZE * 2 + 1/


Man... I just started tweaking the new world gen system to see all the stuff we can do with it.
And man are the results awesome. All these changes were made simply by changing 3 numbers.

![2015-06-25_16 05 48](https://cloud.githubusercontent.com/assets/11893049/8365969/8b0db4f0-1b56-11e5-8dc7-4da51a5f00e4.png)
![2015-06-25_16 06 28](https://cloud.githubusercontent.com/assets/11893049/8365976/9b48c5da-1b56-11e5-9ab7-d0a1810f47b4.png)
![2015-06-25_16 08 03](https://cloud.githubusercontent.com/assets/11893049/8365977/9b4c900c-1b56-11e5-8c28-4be2fbade3d2.png)
![2015-06-25_16 10 23](https://cloud.githubusercontent.com/assets/11893049/8365979/9b4e8f56-1b56-11e5-95ff-8313ce7d3677.png)
![2015-06-25_16 11 57](https://cloud.githubusercontent.com/assets/11893049/8365978/9b4d294a-1b56-11e5-8d92-4a3bdca0483e.png)


I think this is no longer used @uaalto no?


I second @gordontucker.  This change has caused keyboard avoiding to break for me too.


In vanilla, the 'player has joined the game' message is sent to all players.
Specifically, in `ServerConfigurationManager.initializeConnectionToPlayer`
it calls `this.sendChatMsg(chatcomponenttranslation);` which sends to all players.


This is related to the tab names in anal-window.xml so should /add/ a new one not change the existing


se reproduce atunci cand deschizi notificarea


BTLE is not supported at this point. but there is nothing wrong with the change :)


Opened issue about this at https://bugs.eclipse.org/471055


I still don't know well Haskell style best practices...


What about the comma operator?


This allocates `sdim * 4` (or `* 8`, depending on architecture) bytes — `sizeof (char*)` is the size of a pointer, not of a `char`. You probably don’t want this.

And in fact, `sizeof(char)` is guaranteed to be 1 (that’s how it’s defined), so this is entirely redundant anyway. Just do `malloc(sdim)`.

The cast is also redundant.


can this hold values rather than pointers?


It's the size of the buffer used in Multicast.go. Now that serialization is done by JSON the buffer just needs to be "big enough", but still depends on the implementation of message.go, so that's why I keep it here.


Meh, I keep forgetting that the issues are in another repository. I actually wanted to include a reference to cmangos/issues#74 and cmangos/issues#274.


About 15 minutes before the robot was set to be sequestered I frantically tried to implement some type of mission architecture. It is pretty much garbage so this commit reverted back to a point where it was commented out.


You know that there are no browsers that support deflate-frame anymore, right? (it was never a standard)
The world is moving towards permessage-deflate instead (which actually *is a standard*)


:bomb:


Because it could only be modified in this loop. It seems to be tricky, and I will lock here too.


This is not module.exports.logger ?


Command acknowledged


@jhawthorn I did some testing and `CollectionProxy#reset` is indeed broken in Rails 4.0.13 which is why it was breaking for me in the old spree_store_credit_payment_method repo.  I'll update to `reset` here in solidus.


@sne11ius  Could you please help me write these queries.
https://github.com/sne11ius/play-silhouette-slick-seed/blob/master/app/models/daos/slick/UserDAOSlick.scala#L26-L46

I am using play 2.4 and slick 3.0


Just for interest: Is there a specific reason why the earlier Bison version was used?


Yeah ! Nice, thank you very much. :+1:


Please show us how to enable the Django debug toolbar with Postman.


#skillesTisLissas


Of course, you are completely right, shame on me !


Please move this to Bucket.setTimeSlotDateTime


Looks like we're dropping this log?


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


I've just make the change :)


Hi JV, I made a [best practices](https://github.com/Tacoma-JS/BestPractices) page referencing your suggestion, thanks for the tip.


thanks... i'll take that in the next version.


@alstab this is the refactoring i just mentioned. all done for now on runbooks refactoring so you can work on runbooks witohut worry of merge conflict


Eeeeeeeeeeehr. How did that happen. Will reinstate.


So are chat messages -- the point is to allow redirecting all globally broadcast messages from the player, so use the player's chosen sink.


fixed locally.  it and the others you brought up will be in.  except, i haven't quite got the right headers for the mcp products yet.


I get the following error when I try to load the data in VoltageTrace.m
![errmessage](https://cloud.githubusercontent.com/assets/12864873/8366229/27710ce8-1b61-11e5-84f1-fb1c383ab6b1.png)


How about adding testFilterPercentFilterAll?


I cleaned up the types in `include/daala/codec.h`, however `include/daala/daaladec.h` relies on ogg_packet.


Could we make this a runtime option rather than compile-time?  You could set their default values to -1 in the class declaration and allow to set them in the constructor, adding the parameter to detect as well.  Later we will want to read these values from the configuration file.


i already had support to do family grouping in my generator but hadn't quite tested it fully yet.  just did that locally and the results will show up in the next update.


We haven't been updating the config.html since the Pro docs are the ones we post. We should be updating it, though.

Removed undocumented from application so the docs could generate correctly (as it is in Pro).


So If i tag has errors, you don't bother testing the children?  I guess this makes sense, but was wondering if there was a particular reasoning behing this.


wrong indent


I don't think a swap size of `availableRamB / 2` is useful at all. If you aren't going to create a swap partition for hibernating, then with that amount of RAM, you may as well not create swap at all.


> why the reader side will update the term in this test case?

When it finds out that it should receive MsgApp with term 1, it updates its term and builds stream again.

> Can't we control that?

1. Because update happens on reader side, it is hard to know whether the writer side is ready for this update.
2. The other way is to use term 1 when initing reader, but that needs change too.

> Then we do not need to introduce the new api, right?

Yes. And I hope so.


I am thinking of implementing a GUI using Tkinter or WXPython. Preferably the latter for ease of use and feature wise. A more object oriented approach will suffice. But program is still in early stages


@SubJunk looks like it applies to [x265](http://x265.readthedocs.org/en/default/presets.html#tuning) also :).


Nice pic! Looks good to me

Dave


Flameshadow Conjuring and Alhammarret's Archive are using similar code to Felhide Spiritbind and Notion Thief, so are experiencing similar bugs as the ones I posted for those cards in the main forums.


[heavywater] New rails_hello_world created (version: 0.1.2)

- https://fission.d2o.io/pipeline/job/344315ec-820e-4c06-83cc-0000048d0000


try T.concat [prefix, ".", toStrict . decodeUtf8 $ instanceId]
IMO it's still better than folding in this case


Here, I'll open an issue to track this.


i don't either but i feel it's good to separate individual functions with an empty line


Sorry, that was a typo! Fixing now.


See on line 45 below, `styleF` calls `implStyleF` which first works out the style name and then it calls `__macroStyleF` with the name, which returns a `MStyleF`. I specify `__macroStyleF` here to make the macro replace the `styleF` call with a call to `__macroStyleF`.

For example. `val name = styleF.blah` is converted to `val name = __macroStyleF("name").blah`.


@mollieru You linked https://github.com/18F/pages for USDS?


Looks great!


no, 0.00001 will break the FlightIntegrator math


I already changed the C++ library to mraa if you want to take a look
Regards!


Sorted. Is it worth adding that you should install these extensions to the INSTALL.md?


Although almost all tags extend AbstractTagHandler, not all do, nor it is a requirement that they do.  So this should changed so that it fetches the handler but only continue casting it to TagValidator if handler != null and handler instanceof TagValidator.

Also, I noticed that the in the main code when parsing a form it has a simple tag handler cache. Might want to add one here as well if the performance seems slow.


Right! Noticed that too and was going to change it in my revisions but forgot. I don't know of any reason for that.


@killpack @pdougall1 any recommendations for refactoring this and https://github.com/frolfr/frolfr-server/commit/385a0d694a960ae6d5a25c50eb934a8da957cdbc#diff-6663c5115504b98a04ea93e2421825feR51 ?


per discussions elsewhere: this list will be updated to add
#define __nv_gf116__ 0xcf
#define __nv_gf114__ 0xce
#define __nv_gk110b__ 0xf1
#define __nv_gk208b__ 0x106
#define __nv_gm206__ 0x126

and, adding family macros similar to the folowing:

#define __nv_in_fermi_family(x) ( (x) != (x) ) \
		|| ((x)==__nv_gf100__)\
		|| ((x)==__nv_gf104__)\
		|| ((x)==__nv_gf106__)\
		|| ((x)==__nv_gf108__)\
		|| ((x)==__nv_gf110__)\
		|| ((x)==__nv_gf114__)\
		|| ((x)==__nv_gf116__)\
		|| ((x)==__nv_gf117__)\
		|| ((x)==__nv_gf119__)\
	)
#define __nv_in_g8x_family(x) ( (x) != (x) ) \
		|| ((x)==__nv_g80__)\
		|| ((x)==__nv_g84__)\
		|| ((x)==__nv_g86__)\
		|| ((x)==__nv_g92__)\
		|| ((x)==__nv_g94__)\
		|| ((x)==__nv_g96__)\
		|| ((x)==__nv_g98__)\
	)
#define __nv_in_kepler_family(x) ( (x) != (x) ) \
		|| ((x)==__nv_gk104__)\
		|| ((x)==__nv_gk106__)\
		|| ((x)==__nv_gk107__)\
		|| ((x)==__nv_gk110__)\
		|| ((x)==__nv_gk110b__)\
		|| ((x)==__nv_gk208__)\
		|| ((x)==__nv_gk208b__)\
		|| ((x)==__nv_gk20a__)\
	)
#define __nv_in_maxwell_family(x) ( (x) != (x) ) \
		|| ((x)==__nv_gm107__)\
		|| ((x)==__nv_gm204__)\
		|| ((x)==__nv_gm206__)\
		|| ((x)==__nv_gm20b__)\
	)
#define __nv_in_tesla_family(x) ( (x) != (x) ) \
		|| ((x)==__nv_g200__)\
		|| ((x)==__nv_gt215__)\
		|| ((x)==__nv_gt216__)\
		|| ((x)==__nv_gt218__)\
	)


the codebase is not uniform, but for new code we should follow what's in uncrustify_cpp.cfg. In this case, align the continued lines in the open parenthesis:
```
# False: align parameters under the open paren
indent_func_call_param                   = false    # false/true
```


i feel we should change the reader side instead of writer
On Jun 25, 2015 2:41 PM, "Yicheng Qin" <notifications@github.com> wrote:

> why the reader side will update the term in this test case?
>
> When it finds out that it should receive MsgApp with term 1, it updates
> its term and builds stream again.
>
> Can't we control that?
>
>
>    1. Because update happens on reader side, it is hard to know whether
>    the writer side is ready for this update.
>    2. The other way is to use term 1 when initing reader, but that needs
>    change too.
>
>  Then we do not need to introduce the new api, right?
>
> Yes. And I hope so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yichengq/etcd/commit/3d1a9596b9291fbc266ce0b42091b1c604219141#commitcomment-11871103>
> .
>


Remember that gapps-config.txt should contain all the GApps and not only the one included in that package. The mini package should delete all GApps found in the stock package before installing the selected ones.


note: the following can be moved to docs later.  i just wanted to be able to explain the drf scheme somewhere easy to point out.  we can amend/edit it in whatever way folks feel looks/works best.


This closes it out for #12


btw not sure why this doesn't work with http protocol


Thanks! :)


As said in the previous commit, we should aim to make these runtime options rather than compile time.  It would also allow us to deduplicate the code in these functions.


@kevva Can you release v1.2.0?


user can't register if they don't pay?  Doesn't seem right for Business owner.


Maybe consider putting a space before the `]]` if this breaks again...


Ohhhh :-( Thanks for noticing.
Could you please test if #4922 is better in this regard? (I'll be afk until tomorrow, feel free to commit it if necessary)


indptrは累積人数みたいなものだと考えるとよいと思います。
例えば大学0が3人、大学1が2人、大学2が4人受け入れるならindptr = [0,3,5,9]となります
（resp_matchedは長さが大学側の受入人数の合計である1次元配列ですから、この長さはindptrの右端の値と一致します）
resp_matched[indptr[j]:indptr[j+1]]を考えると
j=0のときresp_matchedの0番目から2番目までを取り出した配列、ということになります
同じくj=1なら3,4番目ですし、j=2なら5番目から8番目です。
たとえばresp_matchedが[8,7,6,5,4,3,2,1,0]となっていたとしたら、resp_matched[indptr[0]:indptr[1]]は[8,7,6]になっているので、大学0が受け入れる学生は学生6,7,8だ、ということがわかります
1対1のマッチングでは女性側のマッチング結果を1次元配列で表しましたが、その際の1次元配列の形式を崩さずに（多重の配列とかにせず）結果を表すための方法となっています

あと、学生側の出力は上の例なら[2,2,2,2,1,1,0,0,0]です。こちらも重複を（大学ごとにある程度の数まで）許しているだけで、1対1と特に変わるところはありません。


@jonandr I understand. Thanks anyway for the patch, it will be useful. From my point of view the idea is good, but the implementation should not rely on an additionnal program. Additionally copying files does not seem the best way for doing that. Directly writing the content maybe via communication pipes or something. I don't know actually and in any cases, this is not a trivial matter.


No need for another declaration here, it's in the header.


Yup, but on x86 those packages are not (easily) available, so I am willing to compromise on this for the moment, until there are any x86 devices around that actually offer support for the missing apps (mostly a restriction on the lack of NFC on x86 devices).


This is always returning ```NULL```. The auto_ptr was already released (thus set to NULL) on the line above.


Evening James,
Thanks that's what I'm after. I went for a long walk today and I was thinking about your code and I believe what it does is create the array that I have in mine but whereas mine is fixed I guess yours is dynamic or able to be manipulated.I was going to try and understand how it works this evening but got absorbed in trying to overdraw an edge with a different colour on my version. It's now on GitHub for amusement.
Anyway I will study yours tomorrow. The next job I suppose is coding up the algorithm,that will keep me busy for a while.
Got an interesting book called Code for my birthday.
Hope you had a good day.
Dad

Sent from my iPad

> On 24 Jun 2015, at 23:13, JamesBrill <notifications@github.com> wrote:
>
> I'm guessing this is what you were after. Now the next step is figuring out how to determine whether a graph is bipartite or not. And then, given a solution for this, how to animate the steps for colouring the graph. Sounds like a fun problem. I won't commit any more on the master branch - this is a no-no when collaborating. Not only does it step on your work, it could also potentially introduce bugs to the program. Thankfully, this program isn't a product with customers who could suffer from dodgy builds on master. So what I'll do next time I make any changes is do exactly what I would do on a real project: do my work on a separate branch and then make a pull request for you to review the changes.
>
> —
> Reply to this email directly or view it on GitHub.
>


thanks for the thanks :)

in resman we use a complete set of headers for every new gpu.  they have gotten ridiculously large.  some folks internal suggested trying to drag all the gpus into the same fileset.  for the moment that produces a much smaller set of text than the alternative.  but, i wouldn't be averse to making a hwref dir for each and providing full headers for all of them.

i noticed that the differences, while enumerated, are hard to spot given how they're grouped (by equiv class).  i was thinking about some way to make it more obvious to see/spot the diffs visually between any two gpus, say.  we could do that in these headers, but also i have been working on some visualization stuff which really might be better suited in that role (ala the envtools/rnndb stuff).

i'll work on making things a little less jarring :)


this wont do.. write something else.. see cartridge


The payment process is too coupled to the sign up process.  Instead, a user should be able to signup, and then if they are paying, the payment should be processed.  The UI/UX can still be the same, but you should break up the actual data sending.  Perhaps send the stripe token as an optional 3rd parameter to the action `users.create(router, user, stripeToken)` and having the action make a separate API request to charge the user if the `stripeToken` is present.


? extends Application


Inside the RCInput implemention you have access to the scheduler.  Maybe it would be better not to add the ifdef here and instead make RCInput_Navio call a setter to change the period?


Can I safely assume that this is just all code-motion, or should I go line-by-line?


Don't make stripeToken part of the user object.  Send them as separate parameters: `chargeCustomer (user, stripeToken)`.


@XhmikosR: I won't push after all, I found one issue with DVB which I want to fix first.


PHPCS: Expected 1 newline after opening brace; 2 found


very cool!  How long do you think it would take to do the python version?  I can help test that.


Good job, this is probably better.


This commit causes #1192


[heavywater] New rails_hello_world created (version: 0.1.4)

- https://fission.d2o.io/pipeline/job/344315ec-820e-4c06-83cc-000005170000


Why randomize the limit?  You always need a large enough number of events to ensure that the resulting sample is close enough to ```filterPercent*limit```.  Too small of a sample and the test would likely fail.  I'd expect ```getWrittenEvents().size()``` to be +/- a percent or so of ```filterPercent*limit```


writedocs.go? The current version doesn't build for me.


This is a nice addition :+1: Looking forward to not having to constantly use moo-icons to generate new material-ui icons


this should be O_RDONLY | O_CLOEXEC.  Why are you using O_SYNC?


We'd probably want to configure /write and /distribute separately.  Else internally fired events would get sampled too.


Whoops, leftover logline from debugging.  Will remove.


This comparison is a bit dangerous.  Unless you've defined "equality" for a custom class like the `Node` class here, you aren't 100% certain what is being compared.  I would prefer to see `if self.head is node:` because the source of that comparison (the identity of the value being compared) is known and reliable.

This same complaint holds for line 44 below.


I think a comma would work in this case, but I see `do...while(0)` used a lot more so I think it's best to stay with that.


wow ;-)


Fixed


This thoroughly breaks `make test` for me, and also breaks `make test P=2` (but less so)...


Great, I'll merge it tomorrow


closes #135


The info is moved to thelanguage file


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


You need {} - this is JS, it doesn't parse like a real language.


Where does the standard library require only destructibility? I'm wondering if maybe we can drop this concept and merge it with Movable, as in N3351.


Ditto re: {}


Yup, that's a good call :-)


Please rewrite your commit message to remove the tongue-in-cheek joke.


Missing space


Oh damn I forgot to fix a few tests of the check framework. Shoot. There
are several tests that are wrong now, but this should be fine for tests
that aren't directly testing the checker framework. I'll clean up ASAP, my
bad.

On Thu, Jun 25, 2015, 6:05 PM Ben Lerner <notifications@github.com> wrote:

> This thoroughly breaks make test for me, and also breaks make test P=2
> (but less so)...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brownplt/pyret-lang/commit/9cfb7db9ffc866df39a6afb9e3f3d51fd8e5936f#commitcomment-11871521>
> .
>


I think this is backward - if we have lvm_vg_name defined and a btrfs rootfs, you will create as btrfs subvolume before doing lvm.  I think we want to prioritize lvm higher than btrfs.


This seems to be left out from a merge conflict


Public API needs to be drm_meson, not meson_drm.


Just realized this is ```-O0```, I really wish it was 2 or 3.


When I tested transcoding with zerolatency (x264) and without it, I didn't spotted any performance degradation as mentioned below for --frame-threads 1:
http://x265.readthedocs.org/en/default/cli.html#cmdoption--frame-threads
Maybe also tests from other systems will be helpful to state if pros are more beneficial then possible cons:-)


Joke removed!


Why was benchmark logging removed?  Is there an alternative?


should this be a `PrestoException`?


Why "-v"? is Pytest to quiet without verbose mode? I can't test it right now.


Is this sabotaging people who might use this library outside of AWS?


Alcuni hint: Questo poteva diventare più corto
```js
var album = [];
```
Idem per gli altri.


I miss the checks for the return value of mmap (!= MAP_FAILED) and lseek (!= -1). But maybe I'm being overly cautious?


I think you meant for this to be conal?


(I'm on it, don't bother)


I'd guess because the logger always print the timestamps and you can get this information form the timestamp


Har lagt til dette manuelt pga commit-konflikt.


Not really. This is not a direct sql-invokable function. Layers above should be doing validation and throwing PrestoException if necessary.


I thought it was a temporary solution, I just wanted to make sure and leave a reminder that this behavior should be changed when we'll have a stock package for x86.


OKay. It is really a port from nose to pytests. It is rather similar. A bit simplified perhaps (no class for example).


@phillipross thanks! I'll test this out soon!


We find the names TypeInfoProvider and CustomJsonConverter somewhat confusing as they both don't really describe what's happening. How about JsonFormatEnhancer (or something implying that this add additional metadata to the format)?


Avresti potuto usare un oggetto contenente gli array:
```js
album = {
  Album1: {
    title: "Album 1",
    artist: "Artist 1",
    country: "Country 1",
    label: "Label1",
    year: "Year 1",
    price: 0
  },
  Album2: { ....},
  Album3: { .... }
};
```
Poi ti iteravi l'oggetto e le varie cose


Any reason for changing this from relative? It is now ignoring any padding that the nav bar has
![screen shot 2015-06-25 at 5 19 39 pm](https://cloud.githubusercontent.com/assets/1663615/8366859/62c091f4-1b5e-11e5-95b1-6e7f0cc12aa9.png)


You didn't seem to use our source formatting. I case you use Eclipse we have a formatting template in the root dir. We are using 2-space indentation.


I think it would be nicer to read if these methods returned the obj as well.


does it work with `nix-shell --arg biegunka "./nix/biegunka.nix"`?


1679-DualDuct (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Throwing an error here feels a bit heavy (maybe a warning instead?), but I'm not completely sure because I'm not sure when this case is likely to come up.  Can we wait on this for now?


this was a really weird merge conflict ... all that happened was some files needed to be moved. git, your better than that!


These two lines do not conform with python indentation convention.


Thanks, this looks really good! The PR currently contains 2 commits with the same message. Can you squash the two commits into one? I can do it for you but then the PR won't show up as "merged". Don't want to steal any credit ;). I've also made a couple of inline comments for minor issues.


You're welcome! :-)

Sebastian


WIP implementation for #2


If `self.text` is also nil then this still doesn't completely help.

I would suggest adding `self.html = (self.html) ?:@"";` following that block (on line 94).

This would also remove the `Dictionary value cannot be nil` warning that the Analyzer currently drops when using Xcode 6.3.2+.


The indentation should be fixed, everything else looks good.


Yes, because start time will be randomly longer.
No, because `HttpException`s thrown by `Aws.instanceId` are caught.


Oops, I was thinking of just the problem of formatting JSON output. Still, a better solution might be to create a simple [Tastypie HTML serializer](http://django-tastypie.readthedocs.org/en/latest/serialization.html) to avoid overhead of middleware which may be relevant for performance profiling.


Turned out to be hardware issue. Anyway https://github.com/mpc-hc/mpc-hc/compare/develop...kasper93:c4c919f7b6824daa3f969c59ad8824f2d2adc77 for today.


@jrubin16 Great point, Jessica. I too initially thought that hiring a full-time developer would be least cost-effective and I was surprised to learn otherwise. I also hadn’t given much thought previously to time zone being such a crucial factor when it comes to measuring the productivity of virtual teams. I thought the class gave great insight on this matter and I hadn’t thought of South America as a potential pool of developer talent for projects primarily based in North America.


You can reuse this function for update paid. Give another name and reuse the function.


There are some `send_message` references inside [this](https://github.com/hoffmabc/OpenBazaar/blob/master/html/controllers/market.js) controller. I suppose they are not related, but I am pointing them out, just in case.


spacing


reviewed


ACK


@aradhnataneja I also thought this was a great insight from class. Thinking about users of your digital products in terms of “user stories” grouped into buckets of “epics” is a great way to think thoroughly about who is visiting your site/mobile app and what each person’s unique needs are. I also feel that this method forces you to prioritize the digital features you should build first. I also found it useful to categorize these groups into epics, because I felt that helped me generate ideas about additional digital features that might add value for the group.


nope belongs_to always singular


has_many always plural


@leggewie Sorry for the delay. I'm a bit swamped these days by other stuff. Should calm down in a few weeks.

I'd be ready to merge that but can you please surround the command lines with `<code>` entities instead of the quotes ? Also, some quotes are not symmetric (some use `"` others `'`).


not sure what ur doing here. u never have a view for destroy action?


@mattlistro Great point, Matt. I too hadn’t given much consideration to the cultural challenges that might result from working with developers around the world. Usually I think of culture gaps occurring when people meet face to face. I’ve always thought that body language and tone of voice could potentially pose problems when doing business with someone from a different culture where I’m not familiar with the cultural norms. However, I hadn’t given much thought to how those same challenges might translate digitally. It was great to stop and think about how those differences might manifest and how I can prepare in advance for working with virtual teams abroad.


again for any sort of destroy action u never have views for this. just like update and create actions u have an edit and a new view but not for for update and create...same story with destroy


The naming convention seems to be Stop as opposed to done.


"Scalar" is a term of art in the standard. You don't mean scalar.


is this working for u or no?


This only removes the documentation. Do we intend to remove the [actual arguments](https://github.com/hoffmabc/OpenBazaar/blob/541ef3591023a1d983407dabe84202c3f19e1ac9/node/openbazaar.py#L44) at a later version?


What is this used for? Nothing appears to access it


I know about cherry-picking and rebasing (I'm actually giving Git trainings at work so that's really not a problem) but I think making a pull request is a nicer way of reviewing. Also, Github doesn't force you to rebase to merge outdated pull requests : as long as they would merge without conflicts, you can merge them.

Do no hesitate to make bigger Pull Request with several things in them even if they are not completely related: changes to the website are easier to review than code changes, so it is much less a problem.


@fabriciovaladao Excellent point, Fabricio. I too got a lot out of learning this process. I think this procedure is a really great method for communicating the digital vision you have in your head to a developer. It’s also a great way to keep track of the different types stakeholders that you expect will use your software. However, I think the most valuable aspect of this method is that it helps you uncover potentially unforeseen problems, simply by forcing you to walk through digital features from a variety of user perspectives.


Should this be wrapped in a migration?


typo


cc @Wilfred


*Empty*, *Web API*, and *Web Site*


Yep, but IIRC you get prompted about it the first time if you don't configure it explicitly.


I have updated and corrected the strings


@jdegoes how does this look for an implementation of `query`? It took some head scratching, but I think it's about right.


This works, but I noticed the FAB reappears when you rotate the device during a search.


The comma is also a nice idea! I might change to that for simplicity... Are there no caveats?

Here's a discussion on StackOverflow: http://stackoverflow.com/q/154136
It seems that at least the points mentioned there (locally declared variables) don't apply here.

By the way, I renamed this branch to [**bugfix/robust_check_macros**](https://github.com/SFML/SFML/tree/bugfix/robust_check_macros) and added further commits, which make also `glCheck` less of a time-bomb and improve the diagnostic output of both macros.


This seems to be a reasonable patch. Indeed, while programming this, I wasn't aware, that people might directly define a macro name with the `xacro:` prefix.


Rahmat! Lekin man repository'ni o'zi uchun so'rganidim :) ...


Nice! I'll probably use this to clean up the grossness in the mesos package.


It seems the previous commit got built and pushed live and my build is using it. The previous commit being the broken aae6b03bc7a8fe0f3ecee0b07cc17b77bec80eab. Can you fix this?


Is this correct? You shouldn't need to escape the 's now that you are using "


`AzkabanJobLauncher` is not available in `gobblin-runtime`, so this won't work. BTW: the build failed on Travis.


See changes here: https://github.com/meteor/react-packages/commit/a45c3dd20c457058db97b1272d89a2973e89998b

I made all of them except throwing an error when overwriting a foreign key on `this.data` -- see comment about that.


So technically del is an npm package but not a Gulp plugin. Which probably doesn't matter to anyone reading the doc. Maybe: "using the gulp-jshint plugin to run JSHint", or better: "using Gulp to run JSHint"? (accurate without getting too much into the weeds...)