Please check how frequently you called the serial port I/O. At the Control center, you should only call the serial I/O once (one write operation and one readline operation) in every period of the loop. The period should be around (128 ms to 256 ms). Both write and readline should be one string that contains the way points and the status info). If there is no emergency or mission change, the write operation should be either null or not called.
My impression is that your current codes used serial I/O too frequently. It intervenes the two levels of processes too frequently and beats the purpose of make two level architecture.


Yes noticed that when porting to Python. We use mediator servers to negotiate the connection.


You can actually do:

var testList = [
  [`\n--- Math tests ---`],
  ["Unary operations test", "pytests..."],
...etc...
];


¿No hay que agregar el tag acá?


Is this really correct? As I read [the source code for Android](https://github.com/NativeScript/NativeScript/blob/master/location/location.android.ts#L144), only the value `3` is honored. All other values will set the accuracy to `android.location.Criteria.ACCURACY_COARSE`.

On iOS however, it looks like it works a little different: https://github.com/NativeScript/NativeScript/blob/master/location/location.ios.ts#L146


harvest?!


`attachments.html` since it's not user-facing?


same condition twice


LGTM, with the caveat that I've only read it, not tested it.


missing trailing linebreak


`StringUtil::endsWith` or even `rtrim(..., '=')`?


this should go to stderr


Is this used in interpreter?

Also, you can import this if you add a reference to the DefinitelyTyped definitions for async:

```{typescript}
/// <reference path="../bower_components/DefinitelyTyped/async/async.d.ts" />
import async = require('async');
```

With the typings, the TypeScript compiler will appropriately typecheck your usage of the async library. (And if you're using an editor with autocomplete, you'll get autocompletions.)


:fu:


those () are not absolutely necessary


See comment in interpreter about importing.


also, s/meter/miter/


Fixed here: https://github.com/openpeer/cryptopp/commit/19bd6fd464c54d4dbbfc1f837adcdc2bc9ae2ffd


indentation


Since we use browserify in the browser, you can install async to `package.json` using npm and it should "just work":

```
npm install --save-dev async
```


r+


I've accepted and merge the pull request.


@SVitarelli indenta isso da forma como está no guide:
 ``
TREASURES =  [
  Treasure.new(:pie, 5),
  Treasure.new(:bottle, 25),
  Treasure.new(:hammer, 50),
  Treasure.new(:skillet, 100),
  Treasure.new(:broomstick, 200),
  Treasure.new(:crowbar, 400)
]
``


`SpecificFolderMixin` already  does that


Oh, ich seh es direkt vor mir, wie du fluchend vor dem Rechner gesessen bist 😜


t1-4 or v1-4?


?

This callback will fire after the first `setImmediate` happens in the interpreter, and will prematurely start the next test.


no, ya lo carga en el EventUtil :P


the proyect has   the following error cannot resolve FIELD_TYPE AttachmentMapper as I can resolve this error.


the hell is corn candy?


done; thanks!


This line is implying he was betrayed not doing the betraying. Harugals, betrayed? or even Harugals was betrayed? would be better


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/91f9c6406038ea1ea7e6f4ab3d3f0bdb23b461ea


merging MonsieurNicolas/stellar-core/cryptoAgility = 91f9c640 into auto


MonsieurNicolas/stellar-core/cryptoAgility = 91f9c640 merged ok, testing candidate = 5ea631f6


What is this library and where is it supposed to come from. It doesn't appear to be a lua rock and I looked in my distro's wxlua package and it doesn't have seem to have what this needs. What am I looking for to satisfy this?


In most instance of translation the Goddess is just referred to as Goddess not with a "the" in-front.


Open an issue describing exactly what you are doing or ask on discuss.elastic.co


many `weave ...` commands actually work w/o weave running *as long as ipam isn't needed*. So I wouldn't just drop the `when_dns_running`. Replace it with `when_weave_running` instead.


I think this is ok, we might get some comments regarding use of raw JS rather than backbone. But I'm not sure if it's a requirement.


Line up "The mysterious" with the rest of the indentation.


Dan was working off rage that we had nothing to do (hardware woes :disappointed: )


Line up "The mysterious" with the rest of the indentation spacing.


Hey @willbreitkreutz, I think this has been committed to master, but there hasn't been a new release yet so it's not available at the moment.


Add description
issue:
MUM-1907 search bar - all pages
MUM-1908 search button - all pages


@kjlaw89 Ok, great, thanks.  I was wondering if that was the case.


hmm okay that would make sense


dsff


Do we want to keep the backend terminology consistent with the UI?  ie  history vs metadata?  I think we probably do.  (This change would apply to places where SnapshotMetadata is used).


Is this a bug?


The duracloud version should be 3.3.0.


Why not use a single line with `fprintf`?


You might want to ensure here that the GSourceFunc always returns G_SOURCE_REMOVE and thus is never called again


FYI @nickpell, help text has been added to the new report landing page. You'll want to incorporate it into the redesign I think.


Good. Should do this in the next 24 hours.


It's actually the main binary that wxlua produces, so I'd expect wxlua package to include it (maybe as libwx.so). I compile this library myself for all supported platforms, so you can grab the binary for Linux from [ZeroBrane Studio repository](https://github.com/pkulchenko/ZeroBraneStudio/tree/master/bin/linux).


:sob:


My 2 cents: in France, you usually write Lastname Firstname, so we might have other people complaining now ;-)


if so, readme should be updated, doesn't?


@fernandog im clueless but thing is if i remove the sess and sess=self.session  it works again . Is it really needed?


I believe it is more conventional to use "alternate" rather than "isAlternate" as the member name.  Similarly isAlternate() rather than getIsAlternate() and setAlternate() instead of setIsAlternate().  Perhaps there is some reason I'm not considering why you used this naming convention?


Hi Oleksii,

I just ran into this problem with keepalive check. I used a older version with this bug still present. Question is, why is the keepalive check not reporting any subscriptions, shouldn't it inherit the host's subscriptions? If not, is that check somehow configurable / how would I route email if I didn't have a default email address setup ?

Thanks!
Alex


Tests now run on Travis but they fail :(


@trhoden thank you!


Yep getGroupHierarchyNames() is all I needed really. Disregard the getParent() stuff. The sooo much for building all this and being so helpful!


if i made the function public you can already do:

```
ofParameterGroup parent = p.getParent();
if(parent){
   parent.something()
   ...
}
```

but since it's not a pointer and it's not a common pattern in OF, it would be really strange to explain


ok perfect!


:+1:
It'd be nice if the generator (the code that prints out the actual diagnostics when the test fails) was able to look up the strings in resource dictionary and print out code that already does this.


Mir reicht rutschen bis zu einem Punkt.

2015-07-02 17:18 GMT+02:00 allgaeu <notifications@github.com>:

> Das Bild /Street_slide_17_mitZielfkt_mitvr_w=0.01_tf=20.jpg passt am besten
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Fallstudien/Optimalsteuerung/commit/25007815bd454f057e9c672b9e45edf45f53349d#commitcomment-11977661>
> .
>


Clonk-Karl suggested adding 'C4DP_Last' so that one can always iterate over all drawing passes. I did not use it in the loop later, I'll explain it there.


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/6ac0cd93a111cc6b836ede1e70f3793e7f49c377


merging basho/yokozuna/2.0to2.1/zl/forward-merge-2.0-into-2.1 = 6ac0cd93 into borshop-integration-524-2.0to2.1/zl/forward-merge-2.0-into-2.1


basho/yokozuna/2.0to2.1/zl/forward-merge-2.0-into-2.1 = 6ac0cd93 merged ok, testing candidate = b4ecf7db


Ok, so what should I do about it? Remove it entirely?


Instead of a string, you probably want to convert this to an association with the User model.  That way, we can take advantage of all the user information already stored there and connect leverage the authentication system that already exists for that.


Perhaps we should be waiting indefinitely for the signal?


The constants for alpha? Yes, they are chosen arbitrarily in the sense that it looked good with these values. I fiddled around with the values a lot on one of my other branches.


This is the bugfix


XMPP did not exist before 1999, so I don't think that the "CCYY" construction in XEP-0082 needs to support prefixing with a minus sign for time prior to the year 1 BCE.

As to years with more than 4 digits, those would be needed starting in the year 10000. Although I see no harm in mentioning the possibility of that, I doubt that XMPP will still be in use 7985 years from now. ;-)


Would just using ${CMAKE_INSTALL_PREFIX} not work with CPACK here? Because it would really make the most sense, given that you have to specify it in order to get setup_runtime.sh to work properly.


Já estava com esse termo, na verdade na página checkout eu não alterei nada. mas estou fazendo as alterações


I'm concerned here that if this.snapshotAlternateIds has a size > 0  then this result of this method call (assuming the snapshotAlternateIds parameter is has items)  will result in adding the new list to the old rather than what we would expect - namely all the values.


Fast drei Stunden 😓

--- gesendet von unterwegs ---

> Am 02.07.2015 um 18:47 schrieb Johannes <notifications@github.com>:
>
> Oh, ich seh es direkt vor mir, wie du fluchend vor dem Rechner gesessen bist 😜
>
> —
> Reply to this email directly or view it on GitHub.
>


@bsonntag This works, but I am just adding a note that the pair is not necessarily single-typed. The wildcards allow different types, as long as they have a common ancestor `A`, the input type of the function.


hola @carloshs92 creo que "module.exports" dará problemas en internet explorer, creo que se debe realizar un "polyfill" para evitar tener problemas con internet explorer.


:runner:


Initially I wanted to use that after Clonk-Karls suggestion, but I did not. The reason is: we only use 3 drawing passes. We have cases for the first three passes, so If someone adds another drawing pass then the switch will execute the default case.


Não estou conseguindo achar essa atividade #97164546 no pivotal tracker.


Hmm, it works though. Any suggestions?


Not sure


I copied that from another function. Maybe it is obsolete there, too?


its working now... please include it back in


Não entendi essa


Thank you for the review. I did not expect any more comments, so I pushed it after two days. How should I handle the comments? Just push a new commit with the corrections or rebase and squash?


Oh you're right! Also, add the following test case if you haven't already (it's the dual of yours and would fail if you took my suggestion, but with the intersection check first).
```ts
var x: (A & B) | (C & D);
var y: A & B;
x = y;
```


@bsonntag _It contains fome_. :grin:


@digitalextremist here we have to take `Celluloid::Group` & check `@group`??
All the actors are in the `@group` array right? correct me if I'm wrong.


Hola Erwin

Por favor tené en cuenta que cuando hicimos el TP, no nos habían enseñado ni functores ni listas. Cuando Franco lo mandó, aclaró que existían las tuplas y cómo se usan, así que pensamos que había que resolverlo en base a eso.

Si querés lo modificamos arreglando los errores y agregando lo nuevo que sabemos.


@tkrotoff - we do watch `disabled` in the [Select component](https://github.com/angular/material/blob/master/src/components/select/select.js#L269-L287).


Good catch, thanks! Wouldn't we avoid this problem if we made the primary key for `stored_relations` `(user, program, relation)` instead of an autoincrement `int`? Is it actually possible for rows in `stored_relations` to have duplicate `(user, program, relation)` values (if not, I guess it means the catalog schema is denormalized for some reason)?


what are you doing !!!???? why are allocating memory to private ??


Just nitpicking here, but should the method names be in `{@code}`?


Factory firmware ?


So CCSD(2)_F12 density works now?!!


And I would add a comment saying that it's necessary to try unwrapping existentials on both sides, because you don't know which side might have nested universals that need to be prioritized.


About using the global definition for include dirs, I talked with @cmarcelo and @barbieri and we saw it's not possible since we've optional builds, we may choose not install some headers if it was disabled on build time.


looks like this:
![newlos](https://cloud.githubusercontent.com/assets/4228461/8483070/a1cbb50c-20f6-11e5-8871-55172133bb5e.png)


@dcarty01 I'm not sure how that `2012`-prefixed migration got created. That number at the start of the migration file name is the year 2012 (note the others are this year 2015), so perhaps there was time travel involved somehow?


Spelling Mistake, fix it.


Why cannot you use gf_calloc() ? why are using GF_CALLOC ?


what are you doing !!!???? why are allocating memory to private ?? its a memory leak!! Think and code please


Should this be 2015?


Yeah. I was worried about this. To my understanding JS is mostly synchronous, so could we fix this problem by always loading settings.spec.js? Or does the browser load each JS file asynchronously?


please use gf_free() instead of GF_FREE ()


I would like not to change EXTRA_HEADERS here since it's logically an internal variable known only by tools/build/
I'm coping a patch adding obj-*-headers += my-header.h to describe the headers a given module/lib/component wants to install. With that we handle the conditional install inside the module's makefile just like we do with *.o.


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1173


This change is addressed in https://github.com/ceph/ceph/pull/5122, so if that PR goes in before #4936, this commit can be removed from #4936.


this is called called `hclust_generated_examples.jl`, fixed in #47.


Copy pasted comment! Not acceptable


As noted in another comment, you probably want to associate the comment with a user too.

Also something to think about:  As written, comments could only be added to `ideas`.  Is that the only thing we want to associate comments with?  Right now, yes.  But will that change in the future?  How would you make this model more flexible so you're not duplicating it if we decide to add comments on a profile, for example?  Not saying that we will do that and you 100% shouldn't refactor anything.  We'll cross that bridge when we get there, just throwing out things you should be thinking about as you start so you can save yourself work down the road.


Interesting


nao entendi


Vou pedir para a galera do design montar 2 banners simples para mim, assim evitamos este problema :)


What do you think about changing `.dashicons-admin-filter` to `.dashicons-filter` and `.dashicons-lock-unlock` to `.dashicons-unlock` in this PR as well?


muito grande


do not see anything in class


Integer division?


import statement should be added


Eu dei uma procurada no google por template de ecommerce em bootstrap a galera geralmente usa como botão só para poder agrupar os elementos, eu achei que seria legal pois já consigo explicar o btn-group


Sounds good, thanks!


Add a comment


Great job!


Just push a new commit with the changes. Rebase probably won't work because git.openclonk.org doesn't allow non-ff pushes to master...


How tall?


cool


To delete a file in the terminal:

Step 1) Use the `rm` command with a path to the file you want to delete:
```bash
rm db/migrate/20120702142349_create_posts.rb
```
Step 2) Tell git not to track the deleted file any longer using the `git rm` command with the file path:
```bash
git rm db/migrate/20120702142349_create_posts.rb
```
Step 3) Commit the file deletion to the git repo using `git commit -m ...` and the file path:
```bash
  git commit -m "Denise put a commit message here" db/migrate/20120702142349_create_posts.rb
```

NB. When doing the above commands, its essential that the file paths are correct. Ensure they are spelt correctly and have all of the slashes in them.


Salut Adrian!


Sorry

I misunderstood your comment.

I thought you were saying that the @entries= [] wasn’t a good solution so I went looking for a better one.

Your comment:

The MenuController#nuke_entries method should call another method on @address_book (which contains an AddressBook instance), like @address_book.delete_all for instance. The business code (@entries = []) only makes sense in the context of an AddressBook and belongs to that class.

Could you try to fix that?

I made the change so now the delete_all method uses the @entries = [] to assign an empty array to the @address_book instance variable which is referencing an AddressBook object.

The code works as expected and I cleaned up the extraneous reference to @addressbook I had added.

Question though,

I can see that the Entry class has the initialization method which would add the state of the instance variables when a new Entry object was instantiated.

But, I don’t see that object being instantiated anywhere explicitly.

We instantiate an AddressBook object and we reference the entry.rb file in the addressbook.rb file so we can access the methods in that file but how do we actually use the setup of the Entry class?

Thanks

Dan Williams, PMP, CSP

Cell: 425-877-6333

From: Jean-Denis Vauguet [mailto:notifications@github.com]
Sent: Wednesday, July 01, 2015 10:45 PM
To: danielwilliams12/address_bloc
Subject: Re: [address_bloc] Added delete_all method to address_book.rb (c0e10f4)

Why didn't you simply reuse @entries = []? There's no need for the address_book accessor, which doesn't make sense. Could you explain it to me?

—
Reply to this email directly or view it on GitHub <https://github.com/danielwilliams12/address_bloc/commit/c0e10f49a0fc75500c8b7cab4a90eaa4662b71be#commitcomment-11967846> .  <https://github.com/notifications/beacon/AGEhg4T0fZIODqAbN28OoegBvtUHqC_4ks5oZMdAgaJpZM4FQaFP.gif>


I hope you are having a great summer!


baixo nivel


comment


where in virginia?


hello


To be really fancy you can assert that `getSurface()->Hgt % 2 == 0` before, to make sure integer division makes sense.


canols to canola


<?php
//Memasukan File header
include_once'index_header.php';
?>

 Simple Messanging System [SMS]

                    Description

 Sign-In

                    Form

 Sign-Up

                    Form

<?php
//memasukan file footer
include_once'index_footer.php';
?>


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


left one comment in file (I think)


bla bla


If you love snow in the summer, you should buy a house in the mountains! :)


Yes, we should.


@twhayes  You did!


Salut 2


Further details on this "Inside the ~/code/bloccit directory, run rake db:reset":

Step 1) Open Terminal

Step 2) `cd ~/code/bloccit` - this will put you in the bloccit directory.

Step 3) `rake db:reset` - this will run a rake command that rebuilds your database (db) from scratch. This will re-run all of the migrations. It helps check that the migrations are now working correctly and to check you don't see the error mentioned in the original message.


@twhayes The beautiful city of Charlottesville.


nao amarrando o comportamento adequadamente


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.720.0408eafccf83e3eb200a35cad8982242e8e7e973](http://build.palaso.org/viewLog.html?buildId=59237&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:06:10


cool


nao implementa o teste


@matigonkas where did you get a apikey? I use the contact form some days ago but they haven't responded yet


Agrege nombredel ramo


Synced.


Yeah but I put a few things in anyway for some cleanup.  Looking back on it
I'm like wow there's so much we could have done to make it shorter
On Jul 2, 2015 1:14 PM, "Hima" <notifications@github.com> wrote:

> Interesting
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mattschueler/nisticaTKExcelCompiler/commit/3f061dc6bf2ac0a26448c78ae40e2262199a10f3#commitcomment-11980134>
> .
>


This commit changed the returned data, and breaks my application.
It doesn't take long to debug and fix my application, but I hope "changing default behavior" won't happen frequently. :-)


+1

If not primary key, we ought to at least put the unique constraint on (u,
p, r) if it isn't there already.

On Thursday, July 2, 2015, senderista <notifications@github.com> wrote:

> Good catch, thanks! Wouldn't we avoid this problem if we made the primary
> key for stored_relations (user, program, relation) instead of an
> autoincrement int? Is it actually possible for rows in stored_relations
> to have duplicate (user, program, relation) values (if not, I guess it
> means the catalog schema is denormalized for some reason)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwescience/myria/commit/b9ebaba873a2c41074f1ea8e7f1da68843d86217#commitcomment-11980058>
> .
>


Unown is also worse than regular Pikachu. If it's fully-evolved and not Combee, it belongs here. :|


I don't see the `executes` part.


hello


can you update the code like this:
```
} else {
		return $value;
```


test 123


adding new line


The bug was here. The `cb->currentIndex()` was 8 if last item in the combo box was selected. The size of `sticks[]` is only 8.


Other are just cosmetic fixes.


fast-forwarding master to auto = 5ea631f6


I already fixed it in 'next', no?


@kfeudner @freefoodchannel  Free Pineapple Pizza in Zelda, ask for Kayla


I will check..


Need a Pull Request for the revert or just push to master?


Just wondering since you can have 0-9 in your variables.. does the implied multiplication still work?  Since you removed the lazy operand, I think it would require a defined operator for the separation now?

Side note, lots of things have been added since you forked, if you are interested in looking through it and adding some of the functionality to yours.  Gamma calculations can now be done (meaning all positive real numbers can be used for factorials, and unlimited nesting of parenthesis are two from this week.


Pull request


Unown was removed recently too due to heavily biasing level balance.


Initial Commiut


No, `next` need this fix also, I will port it...


:100: looks great


No pasa nada, es para que aprendan.
Más tarde les envio otra solución

On Thu, 2 Jul 2015 14:11 AVinitzca <notifications@github.com> wrote:

> Hola Erwin
>
> Por favor tené en cuenta que cuando hicimos el TP, no nos habían enseñado
> ni functores ni listas. Cuando Franco lo mandó, aclaró que existían las
> tuplas y cómo se usan, así que pensamos que había que resolverlo en base a
> eso.
>
> Si querés lo modificamos arreglando los errores y agregando lo nuevo que
> sabemos.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/losNoRecursantes/TP-Grupal/commit/03bdd8e95afe60dc679152b0c616be85bd99008c#commitcomment-11980049>
> .
>


inappropriate variable name!


Thats the version i have

FontForge-2015-04-30-Mac.app

> On Jul 1, 2015, at 4:56 PM, Lasse Fister <notifications@github.com> wrote:
>
> Whatever, if it's "2015-04-30" something between then and now must have changed. You should update to a more recent version using the developer daily builts.
>
> got to http://fontforge.github.io/en-US/downloads/mac/ <http://fontforge.github.io/en-US/downloads/mac/>
> and there use the "Daily Dev" button. It reveals a link to FontForge_latest-HEAD.app.zip <http://fuuko.libferris.com/osx/packages/FontForge_latest-HEAD.app.dmg> which is apparently not a zip but a .dmg
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Tarobish/Jomhuria/commit/7e7555336b254fb95d05117c42b6467fc2869647#commitcomment-11965239>.
>


This makes total sense


I'm not sure I understand the problem with React.  I have all of my strings in a seperate `en-us` file which I then pass through pseudo-localization in order to produce a `pseudo` file.  All the strings are then pulled from that file so that I can verify the localization.  React virtual diffing works fine since the strings in the file don't change.  Are you pseudo-localizing on the fly instead?


Me late


also changed the output format to simple since it look like it's easier to process it


Do we need this file any more?


Done: https://github.com/HowardHinnant/rippled/commit/562d72dc92e8bbeb5d369f522e6d87026f655546


Commit message summary typo: s/inhert/inherit/


Presumably we'll need to get more familiar with the contents of this file when we start adding features? I'm guessing this is the "hello world" example.


Do we need this file in source control?


@Hellowlol  The api key isnt mine ,it is the same from the original fork from where i ported this https://github.com/junalmeida/Sick-Beard


missing spaces around != looks really odd to me.


Well, if it is in fact unique, then I think it should be a primary key, because that would allow us to simplify the code by doing upsert instead of delete+insert. (Actually, it looks like SQLite `INSERT OR REPLACE` requires only a unique constraint, but I think we should have the foreign key in `shards` be `(user, program, relation)`, which would require the primary key in `stored_relations` to also be  `(user, program, relation)`). Does anyone know of a good reason for the primary key in `stored_relations` to be an autoincrement `int`?


s/the/any would make it "right" even for one error. (pluralization nitpick)


Additions by rfriedel look good. That'll make the code that much better.


Thanks!


fast-forwarding 2.1 to borshop-integration-524-2.0to2.1/zl/forward-merge-2.0-into-2.1 = b4ecf7db


nao deve ser parte do commit


There's a problem with the code here.


Ok, it's not maptools, it's mapfix, but you get it (I assume).


baixo nivel


Go do that. :)


do we need all this providers in SR?
If are country providers that's ok (files with subs and dual audio)
but global provider we have a lot, don't you think?


monskey


Oops... Well this isn't implemented yet.


You are welcome :smiley:


Thanks Sanley!


doing (K) cast twice. Perhaps you could use "as" and a null check instead.


Closes #30


Agreed, surrogate keys are evil and should never be used unless it is to
solve a measurable performance problem.   But if it's too painful to
reorganize the schema, we can at least make sure things that are supposed
to be unique are constrained to be unique (to afford upsert but also to
catch dupe bugs.)

On Thursday, July 2, 2015, senderista <notifications@github.com> wrote:

> Well, if it is in fact unique, then I think it should be a primary key,
> because that would allow us to simplify the code by doing upsert instead of
> delete+insert. (Actually, it looks like SQLite INSERT OR REPLACE requires
> only a unique constraint, but I think we should have the foreign key in
> shards be (user, program, relation), which would require the primary key
> in stored_relations to also be (user, program, relation)). Does anyone
> know of a good reason for the primary key in stored_relations to be an
> autoincrement int?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwescience/myria/commit/b9ebaba873a2c41074f1ea8e7f1da68843d86217#commitcomment-11980582>
> .
>


stack could be borrowed fro the pool.


Jinmei says, yes!!!


@schrepfler In that case Packer should probably run their own tap.


Thanks, @tak1n - it looks like the fix I proposed will also fix the regression in `SimpleDelegator` caused by this commit (even though it's the opposite of the case I was talking about - a `BasicObject` that *does* implement `respond_to_missing?`.  I'll work on this tonight and add a spec that should keep this both cases safe.


Yep I just seen all the proof I need to now tell this is Definitely fake Gaben's name changed to  IskU0os
and the game ID Leads to some game with tanks. Time to sue.


This will change the behavior of docker and I wanted to make sure that all the interested parties understood it. So those who are using static binary and so far if docker was defaulting to devicemapper, that will not happen any more and instead docker will use overlay or vfs by default. As priority list looks as follows.

        priority = []string{
                "aufs",
                "btrfs",
                "zfs",
                "devicemapper",
                "overlay",
                "vfs",
        }

So whereever we were choosing devicemapper as default, now overlay will become the practical default for docker shipped static binary.


Overlay has his own problems, so I am not sure if choosing overlay is now a better default or not (for static binary).


ndis_initDriver() is the first call for driver initialization and no other calls are possible without the completion NdisMRegisterMiniportDriver().


Btdigg is a dht trackers so they have loads of results. (Similar to strike)


@lAnubisl do you have any unit tests or any other tests for this new code? what about comments/documentation? does this work for you?


Sure. The class constraint would limit this helper's use, but it's kind of awkward anyway - the MultiDictionary should have the same K and V (just one generic parameter), but the current only caller of this is of type MultiDictionary<MethodSymbol, Symbol>, and a value that's not a MethodSymbol should be treated as not in the multidict.


![Uploading Screen Shot 2015-07-02 at 1.30.49 PM.png…]()


I think docker folks need to take this decision whether this is right change for them or not as they will default to overlay now after this change.


I don't think the implementation in STL uses for loop here due to the efficiency issue. Please use memcpy instead.


I should really rename it to "addSnapshotAlternateIds" instead as that is its intended function.


Do we really want to add a config at this point? and here in the soletta repo?


We're still unsure of how we'll proceed with the bson library; for now, we'll be using this repo as the dependency since we need the mobility of working on it directly and using the changes.


It will actually be 3.4.0-SNAPSHOT as that's what in develop branch... it's a correction I was going to do today.


What he said, Unown has been removed because it is a liability to any team. Also, a Pokemon should at least make PU by usage. That's a fair minimum requirement for Random Battle.


yeah makes sense.


@vanzin I think it will be nice to allow user to choose use hadoop configuration on cluster-side or on client-side. In my company's hadoop cluster, when I use this feature is spark 1.4, it will trigger some bugs to use hadoop configuration on client-side (it's our hadoop bugs, not spark). So if I can choose whether use this feature or not would be nice to me.


color: #7bcc73;


Many of the mandatory routines are not implemented as there are no special functionality to be implemented for the openPOWERLINK PCIe device.

thx I will include the doxygen comment.


Yeah, Agreed. I'll make this change and commit it to this branch.


yes that looks like an accurate description of what was discussed (even with a few improvements here and there :)


No, this was causing issues with fields in the grid because it was overriding a 0 margin later, and causing the last radio/checkbox to drop down to the next row.


COPIRAIT


nitpick: typically the convention is to always have a trailing comma so that if another thing gets added, this line doesn't get touched again.


auto-complete getter and setters in Eclipse. I will rename the methods so they make more sense.


@litao-buptsse you can point `HADOOP_CONF_DIR` at an empty directory, and add the real hadoop configuration files to the client process's classpath to achieve that.


Actually I think handle creation should still be allowed. But only with staticHandles: true.

So the PR LGTM


It means that in current master branch ,ini files placed to ini sub-folder of the project. .ini files still need to be copied into Plugin folder along with .dll


Would it be helpful in a case of reference.Kind == Method, to check if it already has entry in capturedVariablesByLambdaNew ?


Thanks for the Pull Request, this was much needed clean up! :+1:


But does it workt


Yes what hellowlol sayd, this specific provider will have (in teory) torrents that dont have the private tag set on the torrent and have either dht and pex exabled (meaning almost every torrent created) same for strike. While torrentz searches the websites itselfs for torrents. Bitsnoop i belive its similiar to torrentz but has way more torrents
Those are the ones that i believe whould garantee  results even if you would only have one enavled


Need to do the conversion from shift-jis to utf8.
[github.com/qiniu/iconv](https://github.com/qiniu/iconv) is good!


Just to confirm, is there another click handler which actually does the publishing to LinkedIn?


Looks good. :+1:


Line 61 of part III Groundwork seems to be giving me an error. I've added the appropriate imports to the MenuState.hx file, import flixel.util.FlxSpriteUtil. Yet FlashDevelop keeps haggling about flixel.ui.FlxButton having no screenCenter field.


@vanzin your suggestion works for me! But I don't think it's a elegant solution. It will be a puzzle to users, especially when their spark configuration works fine in old spark version, but not well in new spark version.


Yeah okay. I updated the doc based on the android api not the nativescript
implementation of it.


Those changes were working for the default HTML, but any HTML customizations were causing div issues. The accordion instructions weren't working, and if there was any whitespace after the closing div, that was causing problems too. People were manually removing closing divs in order to fix their forms.


BTW, The pgmlink cmake scripts have already been updated to include dlib in a [pull request](https://bitbucket.org/bitbucketdev/pgmlink-topsecret/pull-request/2/minor-cmake-updates-for-ann-dlib/diff) to the "topsecret" fork.  So once it is merged, and then pgmlink-topsecret is merged to the master branch here on github, there shouldn't be any extra work for you to do to support dlib...


I think I did it.  How can I tell if it is correct now?


It would be curious to see actual IL here. Just in case we are not doing something goofy.
I suppose more tests for the optimized cases are coming. - recursive, mutually recursive cases and the like...


WHATTT


Again, it looks good to me


> By default, the client will try to read the `REDIS_URL` environment variable and use that as URL to connect to.

https://github.com/redis/redis-rb/tree/v3.2.1#getting-started

See also https://github.com/resque/resque/blob/v1.25.2/lib/resque.rb#L101.

Maybe this whole block (lines 6-14) could just be removed?


Weird. What browser? I'm on chrome.


@litao-buptsse users in general should not need this workaround, since in the vast majority of cases configs are pretty standardized across clusters.


Cider 0.9 is only available in the *dev* packs at the moment. All work Sam adds to the project go into the *dev* packs and the *stable* packs are only updated when he does a release.

To use the *dev* packs, put something like this in your *emacs-live.el` (changing as necessary if you don't use all the packs):

    (live-use-packs '(dev/foundation-pack
                      dev/colour-pack
                      dev/lang-pack
                      dev/power-pack
                      dev/git-pack
                      dev/org-pack
                      dev/clojure-pack
                      dev/bindings-pack))

Many of the *dev* packs depend on git submodules. After updating your git repo to the branch/commit you want to use, be sure to run `git submodule update --init --recursive` to update all the of the submodules in the *dev* packs.


Found this `ack` bool on the parsed frame. Why is it there and why always true?


Why remove this try/catch?
> It's possible a cleverly malformed transaction or corrupt back end database could cause an exception during transaction processing.

Is this still true? If it is, where will that exception propagate to now? Can it bring down the entire application? Does that opens us up to remote exploit attacks?


test


The regression should be fixed in commit b74edc0937


Yeah I will change that today, Thanks!


why would *anyone* name their container this? :dragon_face:


apparently mine was broken for him too, though it worked for me


You should use `using flixel.util.FlxSpriteUtil;` instead of `import`.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


Perhaps `s/clientDB/db/g`. We generally name the `client.DB` structure `db` elsewhere in the codebase. Consistency.


omg, way to go


ikinci el eşya

bizi ara


was there a reason to delete this file?  we have it in `.Rbuildignore` so check shouldn't complain about this


It's tricky to see: commands_queue_.front()```()```;


The miniport only exposes a set of standard routines MINIPORT_XXXX which are only invoked by the operating system.
As these routines are similar to callbacks, I have considered them as private functions like other callbacks in stack.


That sort of sucks. We should probably figure out a better story for storing datums.


(Though for this PR it is fine).


Is there a reason you removed "&& ... 'Professional Faculty'" rather than adding "|| ... 'Ranked Faculty'"?


OK, how about I revert to my original fix (before I changed to upsert) and create a separate issue to revise schema for `stored_relations`/`shards` and switch to upsert queries?


So what's that got to do with Firefox?


fuck


Hellow World !


@WonderCsabo

I'm trying to mock a static class in my project in same way  as explained in the
[wiki](https://github.com/robolectric/robolectric/wiki/Using-PowerMock) for sdk 21, but I got
stackTrace caused by java.lang.NoClassDefFoundError: org/fusesource/jansi/Ansi. Even if I try to add the
jansi dependency into build.gradle, the exception remains. Do you have any idea?


@TSC21 looks cool, isn't it? :)


@eduardocl i do not know what is this janis project i have to be honest. It is needed by PowerMock? Post the whole stacktrace.


Yes. Please refer the above comment for explanation.


@ximinez Because `ripple::apply` handles the exceptions now.


`filter: invert(100%)` doesn't work on Chrome, you need to prefix it with -webkit, so you can add another `-webkit-filter: invert(100%);`


Ah yes, I see where I missed it. I interpreted it not as a codeword but as a description to use that library. Thanks!


This looks pretty weird now..


no its not a bug cause i is limited to 0 and 1 but it will be fixed because it looks clumsy


Ha. Commit message.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.431.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59246&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:05:54


Got it.
```c++
auto& command_to_execute = commands_queue_.front();
command_to_execute();
commands_queue_.pop_front();
```

Or even better:
```c++
for (auto& command : commands_queue_) {
  command();
}
commands_queue_.clear();
```


Mauvais nom de commit, woops! Mais ça c'est le bon commit pour la pagination!


This scrolling doesn't really work in Chrome.  In Firefox it breaks after about 30-40 lines


Try using different ping id ;)


Could you please include this patch into a release? Thanks!


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.401.94ce61c40440c1a08e42d56dc316d48ca0bc174f](http://build.palaso.org/viewLog.html?buildId=59247&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3319, ignored: 130 Build time: 00:18:27


Hi @TooTallNate,

I'm using your lib in a project of mine and I still get the old restrictions values (AD & SUBSCRIPTION). Playback of tracks therefore fails since they all end up being seen as unavailable.

Do you have any idea why I'm still seeing the old restrictions format? Is there something else I need to update on my end? Perhaps a US-specific account issue (I have a CA account)? Any info would be greatly appreciated!

I reverted this commit in my fork and everything started working again but I'm not comfortable living with this kind of issue hanging over my head! ;)

Thanks Nathan and keep up the cool work!


No problem at all :)


I think I would be inclined to also add a MICROPY_HW_ENABLE_I2S which is like a master switch and would cause this entire file to take 0 bytes if the option is disabled.

I think we should also do something similar for other peripherals as well. This is especially useful on the smaller chips (like the '401).


WAT


Updated


HOLY WAT


OH MY WAT


Patches added includes at least:

- https://github.com/mustardamus/wysihtml5/commit/96380c2b813d47b0e5220ac01cc81be2a0a493c2
- https://github.com/Maluen/wysihtml5/commit/1aea4638e8347bd28ea18c06332e1185dab5d2bf


yes, we can update the implementation later.


@arminc Should we add `staff` to the additional groups? The `docker` under in boot2docker is part of that group as well
```
docker@boot2docker:~$ id -Gn
staff docker
```


I think that you should create an enumeration of pin names, and use those as indicies rather than using raw numbers.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.721.9bc123ff04b60bd01b325874ab7c74d2d0e9ca5f](http://build.palaso.org/viewLog.html?buildId=59251&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:04:35


I would always prefer to not write 'return' since it's implicit, but up to you


I need to squash the commits for this PR before merging, and will add the trailing comma, too


It would be easier to evaluate these changes if each bullet point in the commit message was a separate commit.


yes


this can be done with .select instead of .map


it work but i only removed the line:

<?js= title ?>

it's working !!

why you did not merge it to the repo ?


If this controller wouldn't be used without inheriting it would be better to call it AbstractCallViewController


@WonderCsabo I'm not sure if it is needed by PowerMock because I was using it normally with roboelectric, the issue shows up when I start to use:

@Rule  public PowerMockRule rule = new PowerMockRule()

in order to mock a static class I wrote.

If I delete the line, the tests run perfectly, but I can mock the static class. I think gradle not put [jansi]jansi (https://github.com/fusesource/jansi) on classpath when running the tests, although I can import its classes.


"gladius" esta repetido


@simonvanderveldt we can do that, I left it out because I was not convinced it is necessary


Shouldn't this be a .Router()?


HUH

On Thu, Jul 2, 2015 at 2:21 PM, Joseph Alves <notifications@github.com>
wrote:

> OH MY WAT
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smlee/pathfinder-ui/commit/89f1881e9eb6084682ddb43b9373060c106ee34a#commitcomment-11981428>
> .
>


Could also just be a Vector.


Good point, I'll think about a clean way to do it. It is a pretty substantial chunk of code to support hardware that most people probably won't use.


This line is out of place now. I would put it back in filter_loop


yes, the VETH implementation will require parsing the NET_BUFFER_LISTS and passing the data buffers to the DLLK using veth queue.

This is in the TODO.


We need something that recreates this functionality. AppActions.loadPage is still referenced in app.js:54


This should be _after_ the while loop!


The RTL/RFL behaves in a wrong way. When you pickup RTL for the first time, it must use the alternating L/R fire pattern and no ammo count. In TV/F3 you cannot run out of TRL once it's collected but in TRCL it switches back to PAC/SKL. The dual laser pattern can only be activated when picking up TRL at least once more. The quad laser pattern is activated correctly but the fire interval must not increase!


Klopt, dit ging goed omdat er voorheen de code was die automatisch een eerste laag selecteerde. Als er dan geen lagen waren klopte deze tekst. Nu die code weg is moet dit inderdaad terug naar 'Maak uw keuze'


Hoist/initialize your variables.


ok test


nit: Should this be moved farther up to be alphabetical?


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1682 of 1682 tests passed)

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


Not sure what this method is for, I think if you fix the issue with testing for a number within the while loop then you'll fix the problem.
Also, since a number can be two digits, for the second call you should be doing 'gets' and not 'get_char', otherwise you'll be limiting yourself to 1-9.


Updating of language should be after restarting of application. Now it causes crashes, because during language switching ComboBox widgets looses items which are dynamically added.


My pull request creates both heat and cool setpoint devices for a thermostat. Please review and approve.

Thanks


No need to use bracket notation here. Just .name. Are you using a linter?


why does this look like it's all in the default clause? With another nested default clause later? i.e. the indenting looks wrong.


Sorry, but i am pretty sure that is not needed by default for PowerMock. I just run `gradlew dependencies` and that dep is not listed. Also i can run the tests in the `PowerMock` branch.


Both of these macros can be unified like so:

```
#define af_print(exp, ...) af::print(#exp, __VA_ARGS__)
```


I think it's not right to use total here, you should use a MAX_CONTACT_DISPLAY value (like 15). Or maybe you don't have a test for that yet?


This can be done as a .map instead.


You are doing a check on @contacts.length twice, so it's not the most minimal amount of code. I would pull this code back into the main method, now that you've extracted `filter_loop` it makes more sense to have it back there.


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`I try really hard to remain quiet.`

STOP LYING ON THE INTERNET!


I don't know, what has it got to do with Firefox, the PR doesn't mention Firefox and neither does this commit.  The reason I closed the Firefox issue is because it was due to a CORS issue with the services we were using, so https://github.com/ably/ably-js/commit/97ef57970da73b8cf06a93e2bcf355d7c90f8924 fixes that.


Updated.


Something like this?
```C
#define CLK (0)
#define WS  (1)
#define TX  (2)
#define RX  (3)
#define MCK (4)
```


I ran into the same issue you were having with Circle CI and godeps.

(/home/ubuntu/.go_workspace/src/github.com/tools/godep is from git@github.com:tools/godep, should be from https://github.com/tools/godep)

Google and your project saved my bacon.

I am really curious if you have any idea *why* -u caused this issue or why removing -u fixed it.

Because it makes me uncomfortable not to know why it worked.

It seems like circle should be creating a new instance with every build so -u shouldn't matter.

Thanks.


This is odd. CreateModuleBuilder doesn't report semantic errors otherwise, right?


sqlite_timeseries (mbadams5) - x86_64-MacOS-10.9-clang: OK (1717 of 1717 tests passed)

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


Add a TODO to remove our metrics now that prometheus is the default.


I think we should rather move the whole if statement up to the caller.


what a wonderful test!


I noticed that stmhal/board/make-pins.py currently doesn't have the I2S pins. I think that we should enable them by adding a line into the support_fn array that looks like:

'I2S' : ['WS', 'CK', 'MCK', 'SD'] and I'll have to look at ext_SD.


This way the BAR count starts from 1 instead of 0.


It's not mentioned in http://legacy.python.org/dev/peps/pep-0008/#imports but it's definitely quicker to read.


@WonderCsabo
Thanks for the help.


backslash not required.


@Saurutobi Would it be alright and just create a base page that all other pages would inherit that would have methods for making sure your properly logged into the session. I cant seem to get the masterpage check login function to work still? Got the idea from here http://stackoverflow.com/questions/2679362/create-global-asp-net-function


Evilness intensifies, but cool stuff!


@eduardocl sure, but i do not know how i exactly helped you. ;)


That sounds good to me!


Open a follow up issue that this needs to come from config, and that we should be using the kube-proxy extended arguments to set it.


You can just add defined(__DragonFly__) to the

#elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__)

line here, the FreeBSD version of this function also works on DragonFly.
The KERN_PROC_PATHNAME sysctl used in the FreeBSD versino is the usual way to implement this functionality both on FreeBSD and DragonFly.


The third argument to assert.deepEqual is the message when the assert fails (https://nodejs.org/api/assert.html#assert_assert_deepequal_actual_expected_message). So I guess this should probably be "Does not use key".


rostber, такая запись не допускается.
count(false) == true
Дело не в красоте кода, а в том, что если $this->visitor_info будет  равняться false, то условие  выполнится.


Since `swap_picture` is already a function, you can set the `success` handler directly to it: `success: swap_picture(query)` instead of wrapping it in another function.


@felquis bacana! Podemos fazer algumas melhorias no texto, mas só posso olhar com mais calma a noite.
To the :moon: /o/


```'Dude, can you stop disturbing the whole class in the middle of lectures?'```

BULLSHIT


I don't really like all of the hard-coded pin numbers. I think that they should be replaced with constants which come from the mpconfigboard.h This comment applies to the entire file.


The "sort by key" drop-down is gone from table Browse.


Don't forget the `;` :wink:


i heard there's a new coreos tool that only works on container's whose names are a palindrome


I think that this could be simplified by using the af information from the pins.c file (especially after adding I2S support into make-pins.py)


Since this isn't returning a `Ledger`, rename to `getView()`? Or because the callers treat it as if it's a ledger, keep the name?


I will check and let you know


$result here will have different properties depending on the
method being invoked. if we support using one file for both Set and Test
we will have to decide what value to return


@SolomonShorser-OICR all of these commands have the wildcard stripped. However, the 3.0.8 launcher doesn't match so all these commands are broken. Please revert this change (the version info is useful, don't rename it to pancancer.jar only).


I'm about to push two fixes, but I still don't know what the "page" variable is on the home page, so I may also push a hidden place to show it so that last bit can be fixed.


@wnavarre there should be no space around ``=`` symbols.


I originally had it there to catch multiple callbacks, but I'll change it so we don't get another flaky test - thanks.


Note: When you add callback support, I think that rx_dma will need to come from the heap or data space and not be on the stack (since rx_dma will need to remain valid until the DMA is finished)


The structure is three nested case statements.  For this particular statement, everything but 'Fedora' is falling through the default.  This is required because Fedora is part of $::osfamily == RedHat but its version numbers are in a separate namespace from EL releases.


## This is a markdown file


Use a `switch` statement here.


Nice ternary for looping the index! You could shorten this a bit too, by getting rid of the `picID =` in `picID = 0`, or getting rid of the `picID = ` at the start of the expression. So, it could be:

`picID = picID < num_pictures - 1? picID += 1 : 0;`
or
`picID < num_pictures - 1 ? picID += 1 : picID = 0;`


What is the purpose of the `+ 1` here? It looks like we're trying to make sure the hash is invalid and different from the previous, but if that's the case, why not just use `zero`?


Updated.


It should be ``Recharge(datetime=…, amount=…, balance=…, retailer_id=…)``


Yeah, we don't need to add the comma change here, because you already merged that to master.


Holy formatting hell.


We should define it as a ``@property``.


test


This mostly answers my question above. It might be worth copying or rephrasing the comment to the `Ledger` ctor above.


Tested on IE 11 with no noticeable difference in behavior when viewed in other browsers.


Can we define this function in another file or class? Call it StartupOperations or something. Would decouple the permissions from the urls and simply have one function call to init all of it.


Yes, the `@group` array holds the actors. But be advised, it is protected by `@mutex` and accessed by `Celluloid::Group#each` and `to_a` which use the Mutex.


You shouldn't generally respond from the server with just a string. Make the response an object with a "message" property.


" Proper fix for assholes in weird countries that use commas where they…
… should be using decimal points."

WHAT ABOUT INTERNATIONALIZATION FOR openDCIM? CAN'T YOU WRITE CODE TO COPE WITH IT? ARE WE EUROPEANS ASSHOLES FOR YOU?
Poor spezialist1! Not very uplifting from you!


wygląda dobrze


That's a good question. David would know more than me but my guess is that clients want to use the hash as a key in their associative maps so we have to give them something unique. I'm in favor of never returning a hash for open ledgers but that requires RPC versioning and is a client breaking change.


I **think** it's better (more descriptive) to nest `describes` instead of doing several successive tests in the same one. Like so:

```
describe("Backbone Despotism", function() {
  describe("Uses foreign keys", function() {
    var MyStrictModel = Backbone.StrictModel.extend({
      props: {
        firstName: Backbone.StrictModel.type.STRING,
        lastName: {
          type: Backbone.StrictModel.type.STRING,
          foreignKey: "surname"
        }
      }
    });
    it("Uses foreign keys", function() {
      var model = new MyStrictModel();
      model.set({
        firstName: "James",
        lastName: "Cauty"
      });
      assert.deepEqual(model.attributes, {
        firstName: "James",
        lastName: "Cauty"
      }, "Attributes does not match");
    }
    it("Prefers key to foreign key", function() {
       ...
    }
  }
}
```


My vision is that the input should be a docker_state object instead of fetching the file each time.  Thoughts?


I didn't bother to rename it. I figure that the code is already so ugly that it can't get worse.


Of course!


What is localStorage for exactly?


I think ``recharge.amount`` is better than ``recharge.recharge_amount``. Can you confirm @yvesalexandre?


Using a hardcode offset doesn't seem to jive with the array declaration below:

struct _pyb_i2s_obj_t *pyb_i2s_obj_all[MICROPY_HW_ENABLE_I2S2 + MICROPY_HW_ENABLE_I2S3];

If only 2 or 3 is enabled then there will only be one array element. Using i2s_id - 2 seems to give the wrong index in some configurations.


@nyalldawson: Fixed in PR https://github.com/qgis/QGIS/pull/2190, sorry @jef-n I had to revert you setenv/putenv commit which was the reason for some ugly random failures and some repeatable test failures which you "fixed" patching the test (the ampersand regex).
Some other test failures were due to a couple of real fixes introduced by @mhugent , I've updated the reference getcapabilities e and getprojectsettings output documents.


Sry, nope no specific reason. I just though the repo is quite cluttered and removed some file from the top folder.
Can rewind...


First off, I wrote that.  Second, I put the fix in to support your bullshit.


You can use `FileSystem.getLocal(new Configuration())`, then you don't need to throw `URISyntaxException`.


Describing the effect of longevity as unimodal fits for some areas of the parameter space (defined by `delta` and `nu`), but not all of it.  The are also combinations of `delta` and `nu` that just catch either the first (increasing) or second (decreasing) parts of your description.   I was trying to note that over the whole parameter space, increasing longevity tends to shrink the range of conditions where low quality is beneficial, as it begins to overwhelm the other parameters, allowing widespread contamination and spread for even very virulent pathogens.


In 6.0.8 I was using JsonConvert.Serialize and getting a string of the rows/columns, deserialized just fine. With 7.0.1 when I run in the webservice, same result, but when I call the web service from an app, I get a serialized string that contains all the properties of the Datatable, did something change with the datatable serialization between 6.0.8 and 7.0.1?

Thanks
Alan


``len(list(recharges))`` will be faster.


`there is no question that people ask me frequently`

ARE YOU SURE?


Please add `catch (Throwable t) {throw closer.rethrow(t);} finally {...}`.


LGTM other than the comments.


Same as the mean value of ``recharges_amount``.


hmm, so FreeBSD is not so different from DragonFly after all.
From Valgrind i know that /proc version works, I just wasn't sure about KERN_PROC_PATHNAME.
Could this enter in current form? If it builds will redo later.
I fear poudriere doesn't allow direct patch patching (lines should match exactly?)
I'm still learning basics.


I added the Ovaskainen and Hanski paper (on line 187 below), but could probably do more here.  Just haven't had enough time to find the right citations.


Haha, pensando en la implementación del método de resolución de la ecuación, caí en cuenta que mi clase no tiene por qué contener (y de hecho no lo hace) información de un sistema con fuerzas específicas. Entonces, no debe ser muy complicado procurar seguir la construcción para que la clase simplemente sea para un sistema de masas sujetas a fuerzas que se entregan (parámetro de función derivada, con las constantes y quizá el tiempo); también debo agregar forma de determinar la forma de cada masa en la animación, por ahora sólo agregaré array con radios. Quizá después haga funciones para momento lineal, angular. Creo que esto me servirá para como proyecto final.


ok


Well! My excuses for spezilaist1! And for you  wilpig stick your code-shit up your ass! ASSHOLE!


Perfect!


Get rid of this change


Thanks for your constructive criticism.  Please submit pull requests for code changes.


@mvemkr Why not make these protocol relative? http://www.paulirish.com/2010/the-protocol-relative-url/


:+1: I like it


Miming your commenting: Shouldn't be an else here? :wink:


K, but where it is now (in custom_theme.css), it isn't being used because it's overridden by the margin set in style.css. This is the dilemma I always run into with the CSS. To me, it definitely looks nicer to have that 10px right margin when the label is set to inline. What would you recommend?


## This is a markdown file


One thing to keep in mind when writing jQuery selectors, is that every time you make a selector, for example `$("#picture")`, jQuery makes a new lookup in the DOM. If you're going to be working with an element that you don't need to look up each time, like you're doing here, you can save it in a variable:

```
var pictureContainer = $("#picture");
pictureContainer.fadeOut(3000, function () {
    pictureElement.find("img").remove();
    // etc.
});
```


Why ``NotImplementedError``?


755 is fine


Yuck, okay. So basically it was just preventing any customization. Good to know. Thanks for adjusting it and sorry for the trouble! I'm going to test it out now and see if it all works well for me.


We don't need another format for ``read_orange`` I think. @yvesalexandre, can you confirm?


Now I'm sure that PowerMock is not the problem :-)


It was the one that applies to the smaller subset. Since this is all gonna burn soon, I didn't care much. But since I'm `isinstance(...)`ing below, we could change that.


Pin numbers aren't actually hard-coded at all. This line was just a convenience feature for initialization - you can call  `i2s = pyb.I2S()` without any arguments and it will give a default configuration matching the SPI(2) pins on the pyboard. Otherwise you can pass in a list of pin names which gets checked for I2S usability.
Each of the I2S pin functions (CLK, WS, SD and SDext) are available on two hardware pins for each I2S port, and on top of that SD and SDext can each be TX or RX respectively. That is why the pin validation logic is so complex. Although I am happy to try to make it less complex ;-)


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 650 tests passed)

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


Merged manually in #4a394a01ccbc4e3f3e33be85a78eba8f8881629c.


We'll get around to that :D


In case it's helpful, this figure shows the same information as Figure 2, but with the three panels stacked on top of each other.  The vertical color ramp in each cell is the same as the color ramp in Figure 2 and the x-axis of each small panel is longevity.  So the black line shows the effect of increasing longevity for the given `delta` and `nu.`  Possibly just confusing, but helps to show the range of different effects of changing longevity with the other parameters fixed.


Added raw hash functions.
Added utf16().
Removed md5u() type functions.
Added whirlpool, tiger, gost hash types.
Removed base64-e (appended equal signs to make even hash)
Added base64-crypt alphabet (md5_64c)
Fixed bug where test builder was not using constants properly.


I thought I had removed these before, but they should actually be gone now.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 108](https://teamcity.labkey.org/viewLog.html?buildId=259009&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:04:29
### 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>

```


Authorization e de 2 ori.


sqlite_timeseries (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1722 of 1723 tests passed)

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


@indutny, this is a HEADERS frame sent by the server with `.sendHeaders({})` that when passed to the parser, throws:
` { [Error: incorrect header check] errno: -3, code: 'Z_DATA_ERROR' } `

This part `001700e8ff` is weird, since I only passed one header, it should go streamId | Number of KV. '0000000100000001'.
Ideas?

```
    stream.sendHeaders({
      more: 'headers'
    })
```


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 107](https://teamcity.labkey.org/viewLog.html?buildId=259006&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4 (3 new), passed: 23 Build time: 00:34:00
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

org.labkey.test.Runner.testSessionTimeoutIntercept: <no details avaliable>

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

```


@sojanpr (cc @aaustin ) all of the session state's are now subject to race conditions.

I would advice to delay adding anything to the queue until the queue is empty. What I mean by that is this:

Right now your logic is:
- Branch#foo is called
- Process logic for foo
- Add bar or baz to the queue depending on logic
- Execute queue (shift and process)
- Execute bar or baz

I propose
- Branch#foo is called
- Store "foo call" in queue
- Execute queue (shift and process)
- Process logic for foo
- Execute bar or baz

This way if Branch#foo or Branch#bar is called whilst the queue is processed, or a queue processed item changes a state which changes the flow of the logic in the foo call, it will correctly have this flow when it reaches the logic.


#html
Alicia Arcos Méndez


`handleError` isn't defined anywhere. Consider next-ing all errors.


@jfoug why didn't you put that additional info into the commit message?


This looks neat... What are you using Irmin for in this context?


I heavily disagree, Random Battle should be for every pokemon.


This should probably be something along the lines of: client_root = http://your-site.com/chat/


`console.log` won't break out of the function. Consider returning or an else statement.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.406.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59265&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:20:21


I have been told in the past that too long of commit message was not supposed to be done.


Thank you very much for your time! Tomorrow I will fix both.
El dia 02/07/2015 20:18, "Jose Martinez" <notifications@github.com> va
escriure:

> This scrolling doesn't really work in Chrome. In Firefox it breaks after
> about 30-40 lines
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alfonsovng/UglyChat/commit/dabcb32e748700179cff95639e2dfd208c252422#commitcomment-11981379>
> .
>


You also need to add the new constraint to the content view (line 137). Please always test your code (make sure it compiles AND it does what it is supposed to be doing) before submitting it.
If you submit before it works, please make that in the submission message. Thanks!


Not sure if this commit, but recent one in the last week, we have thousands of emails going in circles. Was working fin until 6/30


I pushed a sample to this branch and will amend it until all questions are sorted out :)


Maybe a `.then(null, next)` to next errors.


:speak_no_evil: :exclamation:


The KERN_PROC_PATHNAME sysctl is the more idiomatic way to do this, procfs in its current form is more or less condemned to die at some point in the future, being replaced by the KERN_PROC sysctl subtree.
I guess it could enter in this form (if it works) (though it's rather jrmarino's decision), but the change will even be smaller if you  make it use the FreeBSD variant :)

You can just adjust that branch on your github clone (e.g. by squashing the fixup commit using something like "git rebase -fi HEAD~3" into this commit), force push the branch into your github repository, and the update should automatically show up in this pull-request.

patches from the dragonfly subfolder are always applied after the freebsd-ports' patches in files. So if that file is already patched by freebsd-ports, the dragonfly patch needs to be relative to the version after the freebsd-ports' patch is applied.


更新的資訊要寫大概寫一下啦...


Jajajajaja si es vdd es q en prolog lo ponemos asi entonces ya es la costumbre jajaja gracias por el apoyo profesor es el mejor


Right - but are the default pins that you're assigning above valid for the 401, 411, 405, 407, 427, 429, and 439 chips (which would all use this file?

I used to have a function which would take a pin and a function from pindefs_stmhal.h and would search through the alternate functions for a given pin, and return the af index. You can see the tables which are generated build-PYBV10/pins_PYBV10.c file. Currently the I2S functions only exist as comments since they're not listed as supported in make-pins.py. I'd be happy to add the I2S functionality in make-pins.py if you like.


This is working well for me in my basic testing. I didn't test super thoroughly since Steve and you have looked at it already too.


Build and test


I 'm not sure of the strength of this :s


I have fixed the login functionality with reference to #95968924 and as per our discussion on July 1st. And tested the behavior of the login page as per our discussion on July 01 st.

Please test


1. Я бы учитывал. Хотя можно же оба варианта обсчитать...
Как раз такими случаями можно оправдать последующий переход от наиболее длинной подстроки к каким-то более сложным моделям.
2. Надо разбивать на 2 парадигмы. Правда, если список слов для двух парадигм одинаков, то их потом можно объединить. Например, в случае с -ой/ою для тв,ед в 1 склонении.
3. Я бы оставил неполную парадигму.


```I would put up a FAQ here```

WHEN!! (It has been 9 days.)


crypto is not being used, maybe remove it(?)


You can do that, or create an enumeration:
typedef enum { CLK, WS, TX, RX, MCK, NUM_PINS } i2s_pin_t;
and then CLK, WS etc get automatically assigned and NUM_PINS gets assigned 5.


does `val.GetValue()` work? the return is of type `interface{}`


Sorry @dhylands, I think I misunderstood your comment - I agree that all of the `&pin_xxx` constants could be replaced with constant names that make it more clear what the code is doing.


I just posted a little blurb about what I'm hoping to do in #7. Basically it should provide a way to synchronize the contexts in various ways. Maybe it can also be hooked into a general hash consing mechanism too but I'm not sure yet.


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

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


Good!


Yes, we're pseudo-localizing on the fly in our translation map function. Our translation function takes English strings, so when rendering in pseudo mode we just pseudo-localize and pass it back.

Does having a consistent instead of random mapping create any problem?


Almost, but `GetValue()` returns pointers, not values.


@TeckniX what do you want me to do here ?


On line 22 of `AddressBook`, you have:

``` ruby
@entries.insert(index, Entry.new(name, phone, email))
```


sqlite_timeseries (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1722 of 1723 tests passed)

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


Can I use such badges for my plugin?
https://github.com/King2500/PhpAdvancedAutoComplete


HBIRE_vector_take2 (amirroth) - x86_64-MacOS-10.9-clang: OK (1675 of 1677 tests passed)

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


Pretty neat indeed. I just have to do the same now to covariance propagations :)


I 'm not sure of the strength of this :s


Functions like this one were removed (the md4_hexu).  We no longer had any code using it.


So for when TF2 then? :P


Heh, brilliant. :)


Detect a spelling error, do you mean to use canola oil?


Shouldn't it say `/* io.js v3.0.0 */`?


Oh, man, we're going to :gun: the competition with Emoji-fied IRIs :)


This is a few days of work coming up with basic User / Groups structure


Nightly passed, except for the above comment.
Weekly has two spurious failures (only error messages differences in fastOpt of two different Scala versions -- who knows what happens there?).

Tagging.


I'm not really in favor of this change.  It is far more code, and far less explicit.  It is idiomatic in go to wrap handlers with closures.


```python
queryset = self.__class__._default_manager
```
I 'm not sure of the strength of this :s


This seems like a bunch of unnecessary abstraction.  We use closures for a reason - because they're simple and easy to reason about.  I'd like to hear a justification why this is necessary.


@mcimbora I would prefer that the CacheStatus object just retrieved the current CacheAvailability for each cache, and then the comparison to the previous version was performed in the processCacheStatus method instead.


improve comment


Bem-Vindo ao UHM!

 O UHM é sistema sistema voltado ao apoio da gestão das unidades basicas de saúde, auxiliando no seu gerenciamento

            e dando apoio as tomadas de decisões para que elas possam ser efetivas e tragam mais benefícios a população. As unidades

            de saude tem suas atenções voltadas para a população específica do território em que se encontra. Assumindo, portanto, a responsabilidade

            sanitária e o cuidado dessas pessoas e trabalham considerando a dinamicidade existente no território em que vivem essas populações.

Unidades de Saúde do Estado

          Nome
          Territorio
          Bairro

        <% @unidade_de_saudes.each do |unidade_de_saude| %>

          <%= unidade_de_saude.nome %>
          <%= unidade_de_saude.territorio %>
          <%= unidade_de_saude.bairro %>
          <%= link_to 'Mostrar', unidade_de_saude %>
          <%= link_to 'Editar', edit_unidade_de_saude_path(unidade_de_saude) %>
          <%= link_to 'Deletar', unidade_de_saude, method: :delete, data: { confirm: 'Você tem certeza que deseja deletar esta unidade?' } %>

        <% end %>

		  <button type="button"><%= link_to '>>Cadastre Aqui!', new_unidade_de_saude_path %></button>

Conheça o Banco de Medicamentos

O Banco de Medicamentos é onde estão armazenados todos o medicamentos disponíveis em uma determinada unidade de saude,

    sendo possível posteriormente analisar todos os gastos e manter sempre o estóque razoalvelmente cheios para que possam

    atender as necessidades da população.

		  <button type="button"><%= link_to '>>Acessar BM!', new_unidade_de_saude_path %></button>


Expected failure but the test passed, should the comment be removed?


:clap: nice!


Uhoh typo


not necessary call back FileValidator


The change in ripemd256 was a bug fix. The test strings generated if someone used ripemd256_raw or timemd256_64c would have been wrong.


Even though it's on different, test branch? I can do this, don't see a problem.


orn?


I would say "candidate" rather than "contestant" here.


Equipe Profissional

          Nome
          Cpf
          Sexo
          Codigo us

        <% @funcionarios.each do |funcionario| %>

          <%= funcionario.nome %>
          <%= funcionario.cpf %>
          <%= funcionario.sexo %>
          <%= funcionario.codigo_us %>
          <%= link_to 'Mostrar', funcionario %>
          <%= link_to 'Editar', edit_funcionario_path(funcionario) %>
          <%= link_to 'Deletar', funcionario, method: :delete, data: { confirm: 'Você tem certeza que deseja deletar esse funcionario?' } %>

        <% end %>

    <button type="button"><%= link_to '>>Cadastrar Funcionario', new_funcionario_path %></button>


While translating I thought, you may as well say "I've read". Saves space and is clearer (?)


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.4.456.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=59254&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 871, ignored: 50 Build time: 00:36:44


The reason these unnecessary abstractions were added was to facilitate the test - with closures we'd have no way to identify the `HandlerFunc` coming out of the `AuthenticationFilter` closure from any of the others


I honestly cannot remember. Sorry! I think it may be something to do with the circle ci dir cache stuff that it does but maybe not. Glad it helped though! Wish they would improve their golang support...


just, waited for this questions :). there is a internal plugin whitelist for now. i will extract the quick silex implementation into a symfony bundle with some more caching to allow more traffic.


not necessary call back OrgValidator


Passed for me as well.


Lista pacientes

      Nome
      Data nascimento
      Rua
      Numero
      Bairro
      Contato

    <% @pacientes.each do |paciente| %>

        <%= paciente.nome %>
        <%= paciente.data_nascimento %>
        <%= paciente.rua %>
        <%= paciente.numero %>
        <%= paciente.bairro %>
        <%= paciente.contato %>
        <%= link_to 'Mostrar', paciente %>
        <%= link_to 'Editar', edit_paciente_path(paciente) %>
        <%= link_to 'Deletar', paciente, method: :delete, data: { confirm: 'Você tem certeza que deseja deletar esse paciente de sua lista?' } %>

    <% end %>

<button type="button"><%= link_to '>>Cadastrar paciente', new_paciente_path %></button>


90664130_AlternativeModelPAT (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


I would say "vote" here instead of "ballot" (and similarly below, e.g. `ballot_list` to `vote_list`).


maybe it was a copy pasta error?


`one of my closest friends is a lazy bum`

I SAID STOP LYING.


Lista de atendimentos

      Data
      Hora
      Status
      Codigo funcionario
      Codigo paciente
      Codigo diagnostico

    <% @atendimentos.each do |atendimento| %>

        <%= atendimento.data %>
        <%= atendimento.hora %>
        <%= atendimento.status %>
        <%= atendimento.codigo_funcionario %>
        <%= atendimento.codigo_paciente %>
        <%= atendimento.codigo_diagnostico %>
        <%= link_to 'Mostrar', atendimento %>
        <%= link_to 'Editar', edit_atendimento_path(atendimento) %>
        <%= link_to 'Deletar', atendimento, method: :delete, data: { confirm: 'Você tem certeza que deseja deletar este atendimento?' } %>

    <% end %>

<Button type="button"><%= link_to 'New Atendimento', new_atendimento_path %></Button>


Which line is overriding what other line?


So beautiful. You're a genius. :+1:


Radio button for debug info


I suggest making this atomic and not the functions itself updating the keys.


Choose analysis area based on MTP contour


I think you are correct - I haven't tested much with I2S3 enabled, other than to confirm that it compiles.


"".format()


Do we create the state machine if this is true but there are no awaits?


This key creation is ultimately insecure.


Color issue, should be several colors in the contour


Other code uses ``cls`` instead of ``klass``.


`Long Story`

We've got all the time in the world...


Unnecessary change.


bad name. what happens when we add another constructor?


Tagged source map tests pass.

Publishing.


Unnecessary change?


Keep the original as an overload.


Duplicated code from another test


This apiKey is ultimately insecure.


No, I think that should be fine.  Thanks!


I don't want to remove fixltx2e yet, because many people may be using releases from before 2015.


Todas las cosas que vi que cambiaste son mejoras para la legibilidad del programa, pero no deberían haber cambiado en nada la performance, porque muchas de estas cosas las hacer el compilador con common subexpression y linear invariants.
¿Es cierto, es decir **no** mejoró la performance?


Keep non-generic Run overload.


The previous appointment type check was very specific. All the shortAppraisals that are iterated over in this jsp file either come from the database or were inserted here to display that the employee needs an evaluation created. The only time that this has caused an issue was during testing due to bad testing data.

I will add a comment to the jsp file, after you're done with the code review.


Do you just open the sfdir font from the repo, or open a UFO or another format and then export it?


Ah, ok, **si** mejoró. Cuac. Bueno me callo.
Acodate de poner el o los commits que te llevan a ese cambio en el mismo README.md


You don't need to test this against all files in the resources. This should take a lot of time to run. Just create a string with a basic html and the if the values are being assigned it better. Avoid reading files, it takes to long when we have a lot of tests.


PHP Fatal error:  Class 'Core_Business_ContainerBuilder'  not found in /xxxx//config/bootstrap.php on line 27


All methods that return task should be -Async and take an optional cancalation token.


No need to commit commented lines.


You mean user.password


Oh, I'm sorry, I didn't intend to commit this line..


Hmm, I thought about breaking down the commits further, but the commits would have ended up being either too small or putting the codebase in a state where it wouldn't necessarily compile after each commit. I ended up spending 30 minutes grouping the commits. I could've spent more time breaking down commits or committing earlier, but then you'd have to do git rebase to fix commits and history.


you need to delete the class_index.php file in the cache directory


I love '' state


yayuh resolve that shit broski


Sudeep's algorithms for generating queries


You should not edit scope properties in a factory.


Show grids on google map


So it looks like you removed all the references to `mTxnAccount`, but didn't remove `mTxnAccount` itself.


dawhhhh shiet, look who it is! =]


Especially if you're about to go to another state (also especially never).


Awesome, looking forward to it! Thanks!


Hi Oliver. Obviously you have much deeper knowledge of spring internas than I do. To allow me to understand and learn whats going on, could you please add a few more notes when you optimize code without open issue? Thank you!


You should instantiate this with an actual value (like null or something).


Minor nitpick: These indents are created by a tab, not spaces. I recommend using spaces.


Hi,

Thank you for your help.

Could you update it in F-Droid and Play Store?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.722.9bc123ff04b60bd01b325874ab7c74d2d0e9ca5f](http://build.palaso.org/viewLog.html?buildId=59276&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:04:02


Wow, since when it's allowed to extend not abstract module?


@jocel1 TY. Easily forgotten procedure. ;-)


Hi @nlalonde ,

I am correcting the grammatical errors. However, these users make mistakes again.

![screenshot_27](https://cloud.githubusercontent.com/assets/7838711/8486024/83490490-210a-11e5-9698-d4fe53a0c84d.jpg)

I'm tired again correct these errors. Do you examine the below screenshots?

Thanks.

![screenshot_28](https://cloud.githubusercontent.com/assets/7838711/8486073/e2830b72-210a-11e5-810c-7bd44fe4d40e.jpg)

![screenshot_25](https://cloud.githubusercontent.com/assets/7838711/8486076/ec6a5866-210a-11e5-80e1-9b77c541c4c8.jpg)

![screenshot_26](https://cloud.githubusercontent.com/assets/7838711/8486085/f159fd54-210a-11e5-9d71-17a3c90ad35b.jpg)

![screenshot_24](https://cloud.githubusercontent.com/assets/7838711/8486092/00ec4efc-210b-11e5-9e6e-6f1fc3c788d3.jpg)

![screenshot_23](https://cloud.githubusercontent.com/assets/7838711/8486100/055cc4e4-210b-11e5-8f61-b32387d84ec0.jpg)

![screenshot_22](https://cloud.githubusercontent.com/assets/7838711/8486109/16223b92-210b-11e5-84cc-8048de6de516.jpg)

Report = Rapor et  (in Turkish)
same
Flag = Rapor et

Flag / Bayrak (in Turkish)

These and other errors too.. Please only give me the approval authority.

My references for this:
http://www.destekforumlari.com/c/discourse
http://www.discourse-tr.com
http://www.discoursetr.com
http://www.discourseturkiye.com


:smile:


Looks good to me!

One thing to think about here is performance.  Instead of scanning the file 26 times (once for each letter), what if you wanted to speed it up by only scanning the file once - how would you do that?   You don't need to rewrite the code, just explain roughly how you'd set up the code to do that.


Shouldn't need to specify type here.


LGTM.


I tried to inject the `SignInManager` instead of the `UserManager`, because it seemed more appropriate. The problem was that I wasn't able to resolve the `SignInManager` via `serviceProvider.GetRequiredService()`. The `contextAccessor` was always `null`. See also... http://stackoverflow.com/questions/31192425/contextaccessor-cannot-be-null


LOL ..


Coexistence, my friend. Instead of deleting the Java source code, could have created a pull request that adds Scala code for the game as well.

Your code looks good. Both of ours need some proper formatting though. And with "proper" I mean in accordance with the new code formatting policy that I have come to accept now and that which I use in my working projects.


I have done this by using flags in c plus plus, it passed all the test in hacker rank, although is [mine a correct solution](https://github.com/shadow-stranger/utopian-tree/blob/master/utopian-tree.cpp) ?


That is annoying, I wonder if we can fix it?


I'm not sure whether it's appropriate or not to use an `async` method override here. It seemed so to prevent using `.Result` for each `Async` method call into `UserManager`. Also, given that I'm using `async`, I needed to return nothing instead of using the `Task.FromResult` pattern that was there previously. Clearly, I need to learn more about using `async`. Thoughts?


If this was meant to fix Fifa Street shadows I want to confirm that Shadows were working ok before this revision, This version seems to have broken them now.


This looks like a change of behavior. Previously we wouldn't throw on null. Throwing is correct but may not be backward compatible.


Also added a fix for the SQL in addgroup


Thanks! This was helpful.


I used the out of the box `IdentityUser` and `IdentityRole` for now to keep things really simple. Going forward, I might extend one or both. Extending `IdentityUser` seems like a good idea to store profile information that the demo project can return as one of the claims in the access_token. Thoughts?


Space after comma.


Include options for determine the length for each square.


A lot of commits did that, sorry  :)  Not something I plan on doing too often in the future!

Also I should probably limit the feed to not be everything ever.


I don't know what `AddDefaultTokenProviders()` does nor whether it's necessary. I put it in strictly because all the Identity demos I've reviewed appear to have it. Ah, frameworks - there's so much magic.


## Signum Roadmap: C# 6,  VS2015 and Angular|Aurelia

Visual Studio 2015 has been [finally announced for 20th July](http://blogs.windows.com/buildingapps/2015/06/29/release-dates-and-compatibility-visual-studio-2015-and-windows-10-sdk/).

It's a huge release, including C# 6.0, Roslyn, .Net 4.5 / .Net Core, ASP.Net 6 and Grunt/Gulp support.

Out intention is to keep Signum Framework up-to-date, making use of the new features and removing code that doesn't make sense anymore.

I've been doing some initial work on Single Page Application, with the idea of using Aurelia, but after the VS announcement we've decided to re-order priorities.

### Step 1: C# 6 (August 2015)
Adapt the code to the new awesome features of C#6 ([video](https://www.youtube.com/watch?v=-FkNRQ7ubH0) and [wiki](https://github.com/dotnet/roslyn/wiki/Languages-features-in-C%23-6-and-VB-14)). Inclding:

* Progressive elimination of all the extension methods that work on null values. Now with `?.` we can be more succinct and explicit. This includes `Try`, `TryDo`, `TryToString` and `TryGet`. Regular expressions to convert the code will be provided.

* Simplify entity properties declaration using a combination of the new *Auto-property initializers* and adapting the code in [KindOfMagic.Net](https://kindofmagic.codeplex.com/). The same technique could also be used to auto-instantiate Symbol and Operation fields.

* Code analyzers: Progressive addition of [code analyzers](https://channel9.msdn.com/Events/Build/2015/3-725) to make Signum-aware suggestions / warnings / refactorings.

### Step 2: .Net 4.6 and ASP.Net 5 (September 2015)
If case you don't know, ASP.Net has been rebuild from scratch and is only 'concept-compatible'. Razor views and controllers should be more or less similar but all the dependencies to System.Web have disappear (no more HttpContext, global.asax, app.config...) and now uses OWIN pipepline system.

I don't expect huge code changes but they will be complicated ones.

### Step 3: Aurelia or Angular 2.0 (November 2015)
At this point will be in a much better position to move to Single Page application because:

* Grunt / Gulp will be built-in in Visual Studio 2015.
* MVC and WebAPI will be using the same programming model (necessary to send Json entities).
* Hopefully jspm and Typescript will work better together.

Additionally, we will have a more clear idea of what the evolution of Aurelia / Angular 2.0 is going to look like.

### Step 4:  .Net Core. (around 2016?)
In theory, .Net 4.6 is probably the last monolithic .Net Framework. The next version will be called .Net Core 5 and will be mainly distributed via Nuget and will have many breaking changes.

I expect the transition to be rather slow, since libraries have to be updated to the new Framework manually. When the ecosystem of libraries is ready (including WPF) we will be able to move.

## How to be prepared:

* Learn the new C#/.Net features.
* Install VS2015 as soon as possible, `CS6` branch  will be merge into `master` the 20th of July and there will be a new 'master2013' legacy branch only for bug fixes.
* Update the framework frequently in the following months.
* Explain to your boss/client/girlfriend(or boyfriend) that by doing this changes the application will be 'evergreen', meaning that you don't need to rewrite it from scratch every few years in the next cool thing.

Don't be scared! Everything is gonna be 200 Ok.


Here it triggers an generic `change` event.


I didn't know who to do constructor injection in any other way with a class that we're instantiating like this. Suggestions? Is this fine the way I did it?


15 - Another day 27 Hours, 39 Minutes, 20 Secondes
А сколько это - 27 часов? :)
Мы скайпе обсуждали - надо посчитать с момента начала последних суток


Here it does not trigger a generic `add` event. We must listen to the specific `add:fruits`. Is that correct? See above.


Good work tinchoz49, you beat me to it!


You removed NFE/LC -_-


Same here,


ping @snurra


If you see anything that doesn't seem quite right in the way I've scoped variables, please let me know. I'm not sure whether the scope of them is appropriate or not.


Should be Task<object> not object.


Pick one, every Pokemon, or every viable Pokemon.


I posted the last comment before seeing your reply - I admit I hadn't considered anything beyond the 405 and 407 chips.
Having the I2S functionality in make-pins.py would be helpful - that would probably provide a more elegant way to do the pin validation which you commented on below (line 535).


I can't find how to send messages fromgithub, so I'm sending this straight
from gmail.

Would ye ever take a look at what I've been at there with that silab af
yours. I sent you an aul 'pull request', as the man says.

I've been workin online - on Github in fact as Android Studio was giving me
a massive pain in the hole - and, from what I can tell I've been workin on
a new branch of *my *fork (these lads'd need to calm with all their *slightly
*different terms for things) called patch-1.

Anyway, to see me work go here:
https://github.com/yipidee/silab/compare/master...surrealmccoy:patch-1

On 30 June 2015 at 08:50, yipidee <notifications@github.com> wrote:

> Working hard (or hardly working, hoho) I see McCoy (@surrealmccoy
> <https://github.com/surrealmccoy>). Trying to make fancy smancy
> exclamation marks are ye? Walking round thinking you're deadly at the same
> time no doubt.
>
> Actually, "fonts" could be a cool feature down the line.
>
> By the way, you did this on the addMaps branch of your fork. You can work
> away there and send me a pull request (probably not as dirty as it sounds)
> and I can still incorporate it into the real master. But if you want the
> latest app, you'll need to rebase, which might be difficult (I say might
> because I have no idea, I think you have to specify my repository as
> upstream and the rebase against it)
>
> Git is quite interesting when you think about it. Must be very powerful if
> you know what you're at. Linux is opensource, they use Git for version
> control.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/surrealmccoy/silab/commit/3bbaa084d841331ba6a6b5bcc1a4dfc309eee5c1#commitcomment-11925929>
> .
>

--
Cathal.


Yes. I noticed this problem when I was changing the preference tests, and this needed to be changed (since we shouldn't actually be testing this in CSS files). This was the first thing it occurred to me to change it to, and it didn't work.


Shouldn't it be RunAsync and take an optional cancellation token?


does it build on windows now?


Taken from `man gcc`:

```
-D name
    Predefine name as a macro, with definition 1.

-D name=definition
    The contents of definition are tokenized and processed as if they appeared during translation phase three in a #define directive.  In particular, the definition will be truncated by embedded
    newline characters.

    If you are invoking the preprocessor from a shell or shell-like program you may need to use the shell's quoting syntax to protect characters such as spaces that have a meaning in the shell syntax.
    If you wish to define a function-like macro on the command line, write its argument list with surrounding parentheses before the equals sign (if any).  Parentheses are meaningful to most shells,
    so you will need to quote the option.  With sh and csh, -D'name(args...)=definition' works.

    -D and -U options are processed in the order they are given on the command line.  All -imacros file and -include file options are processed after all -D and -U options.
```


So, what if the actual cloudprovider after some time wants to support another interface which they didn't do before. They would have to change the config file and reboot their whole system with kubernetes. But still I see your point that there are too many requests I send which would not be needed if this was in the config file.


The compilation can then be also asynchronous and cancallable.


I'd say that for "fred(10:12:14~20, 18:10:23~40)" leaves the app to decide which pixel axis is closest to RA and which closest to Dec. For "fred( 100:200, 13:00:00~20 )" this is an error due to ambiguity; you can not specify one pixel range and one WCS range for a SKY domain. It all breaks down at a pole as well as for 45 degrees.


I actually have a PR in progress which looks like it will fix this by accident, let me get back to you...


Blame @asiekierka.


need comma!


o certo seria: And eu opto por adicionar o atleta


I don't know if that's PSR. It's more something I'm used to and that I applied here.
Anyway, not a big deal. You can leave it as is, I won't cry I promise :P


Esse cenário já foi corrigido... está no commit novo, que por sua vez não foi dado o pull


@winsock agree so much


We need this line to start the next activity?


Starting the project


Esse cenário já foi corrigido.. e por sinal, ele não faz mais parte da feature.


sqlite_timeseries (mbadams5) - 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-92.55%25-yellow.svg)


O que difere do anterior é que esse cenário é de controle e o outro é de GUI.


esse preenchimento de login feito aqui pode ferir o princípio de dry(don't repeat yourself), pois se se faz necessário aqui se faria necessário na maioria dos Given dos testes, alé disso apenas preencher os campos e visitar outra página sem concluir o login não teria nenhum efeito


Heinous fuckery there. That should call a view images method directly.


No it wasn't.


It's a very good job!
But when I was trying to run the project, there were serveral exceptions! And I want to know how can I contribute for this project? could you give me your email address or contact me with junjie_liao@qq.com? Thank you!


visit acessa diretamente a página, todos passos feitos antes não surtiram nenhum efeito.


Não seria "I want to have access to all the other features", e o "So that I can <fazer algo, ver o repositorio, acessar index, etc>"?


O Given já direciona para a página, sem ser necessário um novo visit


[cool](http://uxmovement.com/buttons/why-ok-buttons-in-dialog-boxes-work-best-on-the-right/)
LGTM


@aldenml - merge if you agree
[UX explanation of why this works better](http://uxmovement.com/buttons/why-ok-buttons-in-dialog-boxes-work-best-on-the-right/)


o then deve chegar se algo ocorre, aqui estão sendo feitas algumas ações e nenhuma checagem, acredito que deveria ser parte de um when e não um then


Since it's not used to reinsert the key every time, remove should probably just return the value.


I love San Francisco; been here 30 years; play tennis; I play guitar in classic rock bands and classic jazz combos play Miles Davis tunes.


This file is well-written, Rob.  Thanks.  One high-level comment re: the commit.  We should probably have a way of clearly distinguishing between comments that (1) represent literal translations of the Charter language, and (2) other commentary on the voting method.  For example, the characterization of a "properly cast" vote is an example of the latter.  I don't have a suggestion yet on the best way to notate that distinction, but just wanted to flag this as a suggestion for the future.  I'll also make some edits to this commit to fine-tune some of what is written.


But won't this throw `KeyError` instead of returning None if it doesn't exist?


Não entendi o motivo de criar um atleta nessa verificação, acredito que não seria necessário


wait why are these all includes now? if it should execute, it should have .sh. we probably had a miscommunication


it's here then.


A Bit of Info about me


不好意思，可能是我vim编辑的时候不小心删掉了，没注意，已添加


It's switched. Correct this


LOL


thanks


não é necessária uma comparação com boolean, um jeito mais intuitivo de fazer a verificação aqui proposta seria
@atleta = Atleta.find_by_cpf(arg1)
@atleta.should eq(nil)


Precisa especificar o valor que vai ser inserido?


É recomendado que cada instrução só tenha um "Assert", uma boa prática seria substituir essas linhas apenas por uma como: assert (testDeleteBook1 == null && testDeleteBook2 == null)