Rozwiązanie oczywiste, to przenieść image do klas, które dziedziczą i tam już na sztywno wpisywać ścieżki.

Być może jak będziesz wpisywał dużymi literami, to można nadpisywać te atrybuty choć wydaje mi się to wątpliwe. Tak więc w tym wypadku raczej po prostu przeniósłbym chyba te pola do klas niżej.

Rozwiązanie z nadpisywaniem czegoś w konstruktorze jest hackiem.


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

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


Removed


Updated


Does this work? Usually the `#password` method on a Devise model is not a readable field. It stores an encrypted version of the password and not the actual password.


Nie trzymaj na raz i pliku views i modułu o tej samej nazwie. Być może to działa, ale może kiedyś się zdziwisz, że nagle coś się nie będzie chciało zaimportować.


Jag är full, jag gör det imorgon istället!


> Alternatively, you could re-define B's constructor to invoke super from an immediately-invoked arrow function (similar to what you're doing in B#incrementer already).

I think that's a different test.


Such a little thing makes such a big difference. :sob:


Jeżeli już korzystasz z templat djangowych, to raczej częściej spotykane jest trzymanie ich wszystkich raze w jednym katalogu, a nie rozsiewanie po aplikacjach.


Thank you for this information kilbith,
This mod isn't merged to the server. Indeed we merged to the server when we update news.txt, for the most of time
For the moment we try your mod in a local server, we are certainly going to tweak it a bit (we think the "in-game" tetris is too slow, but it's just our opinion)

We will report to you our bugs/fixs found if we encounter bugs :)
(And from all of our devs team, thank you for following us ! :D)


Thanks! BitHub has sent payment of  $10.41USD for this commit.


New test added. This test updated.


Not sure about the name, includesTestsFrom maybe?


Seems like `s` could be moved from as well.


Zaraz się naprawi:)
15 kwi 2015 21:35 "Jacek Migacz" <notifications@github.com> napisał(a):

> Tak w ogóle, to piszę się "physics" a nie "phisics"... chyba że to
> francuski, to pardon!
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/PawelPolawski/crocodile-carl/commit/cd5d95e23c46f6907a305194e7078035460fe79c#commitcomment-10743593>
> .
>


Dziwnie korzystasz z tego grida, tak jakbyś nie dokońca wiedział jak on działa.


IINM, pointer arithmetic on `void *` is a GCC extension, which defines its size as 1, so it could be replaced as a standard `char *`. It's opinionated, but personally I am for to avoid casting to `void *` where possible, because it could hide some relevant compiler warnings on improper type conversion.

This particular case seems to be proper though. :cactus:


Nice find.
Have not seen the error actually happen in a very long time. It doesn't occur at all in 3.0.


Split into two tests: strict, non-strict


J


You are right, I was confused about some JSON concepts. I must have made a typo, because now (retrying it) it loads fine.

I will refinde the validator and tests.


Great use of `.each`! Your code is clean, concise and easy to read! Just a couple minor points (I'll add them to the relevant lines)

Here, calling your parameter `element` in your `.each` callback isn't very specific. It's hard to know what `element` stands for without reading the code or looking at the markup you're scraping. Naming it something like `row` would be clearer, a name that represents the actual contents or data of the row would be even better!


this is not correct. the `/** should be on the next line like everywhere else.


I've never seen this done. Can you expand on how this is a better practice?


as I understand the docs, the tags here will apply a tag to all tasks, not filter the tasks run through the include, which means this cannot work.  Not sure if I have that right.


If you use this fix, the Intelli-plug is no longer. That indicates to me Faux Clock.


I'm not suggesting you change the code now but, you could also use the `.tap` method as an alternative way to achieve the same result.

```ruby
def find_wiki
  @wiki = Wiki.find(params[:id]).tap do |wiki|
    authorize wiki
  end
end
```

`.tap` takes a block, passes the object that you called it on into the block as the argument. You can then do whatever you want in the block and when you're done, the `.tap` method returns the object that you called it on originally.


Thanks!! :heart:


My personal habit is to try and keep implementations in the .cpp file in the same order as declarations in the .h file.  `configValidatorKey` comes first in the .h file but is last here.  Not a big deal, but I thought I'd mention it.


The position of all of these backslashes is messed up.  Fix them or don't bother aligning them and just use one space.


@fabienwang
![](http://i.memeful.com/media/post/BRkm0vw_700wa_0.gif)


Here, `this` is the same as your `element` parameter. It's quite clear what `this` refers to, but `element` may be even clearer. But more importantly, every time you call `$`, in your case as `$(this)`, it creates a new jQuery/cheerio object. To make the code a little bit more computationally efficient, it's not a bad idea to save `$(this)` or `$(element)` to a variable you can reuse:

```
var row = $(this);
var permission = row.children().first().text();
var absoluteUrl = row.find('a').attr('href');
...
```


Weird but funny.

It seems to work.
The arrows are 3-bytes characters. No problem for me, but does it work on all clients ?


Ok. I looked through the utils quickly and didn't realize that jointTransform did this. I will change my code and make a pull request deleting the util I added. Thanks Twan.


This message is incorrect.


merged at the requests of keith D:
no pull requesterino made


You have a method that does this, `set_wikis_to_public`, which you could call here instead. This would DRY up the code and give you a single place to make changes to the behaviour.


Last sentence added.


Change this one for the one in useraction, and this for the other one.


Pointed explicitly to the logo design documents, with separate link to generic landing page.


:beetle: From [here](http://pubs.opengroup.org/onlinepubs/9699919799/functions/snprintf.html) (emphasize is mine):
> Upon successful completion, the `snprintf()` function shall return the number of bytes that **would be** written to `s` had `n` been sufficiently large excluding the terminating null byte.

That said, the return value`printed` can be greater than `env_sz`.


Try to use a shared library first.  Also, use `find_package_handle_standard_args` like the other finders do.


This method shouldn't be tagged with the @ override annotation.


Live and learn. Agree with void* arithmetic.
However, all casts breaks type checking, I didn't get the point here.
And yes, this is a bit crazy line. It could be more safe, and more verbose

On Wed, Apr 15, 2015 at 11:41 PM, Eldar Abusalimov <notifications@github.com
> wrote:

> IINM, pointer arithmetic on void * is a GCC extension, which defines its
> size as 1, so it could be replaced as a standard char *. It's
> opinionated, but personally I am for to avoid casting to void * where
> possible, because it could hide some relevant compiler warnings on improper
> type conversion.
>
> This particular case seems to be proper though. [image: :cactus:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/embox/embox/commit/3dce1d40beb7e6d7921ee00f91624568a4bda919#commitcomment-10744904>
> .
>

--
С уважением,
Антон Козлов


Ha! Took me a while to find this. I was getting weird exceptions about datetimes coming from couch that looked like `'2014-08-26T15:20:20.062732'`—notably with no `Z`, which I've made stricter in my work with tightening up `DateTimeProperty` for microseconds and timezones.


There should be an space between two methods.


This should be passed through `cmake_config.h.in` like everything else does.


This method shouldn't be here.


I know, I just do it for my own "readability".  I made a lot of these changes, but I reinstalled the OS on my laptop and lost the changes before I could push them.  :(


Thanks for removing the console.


This method does not follow the code guidelines, you should format this method.


It's something I've been doing in my gems.

  1. Define the root module in the root file, so you can store root level constants, root level cvars, etc, within the root namespace.
  2. Require the subfile inside of the root module so that there's never ever a "we don't know what `Clearwater` is" issue.
  3. Require relative just because it's easier in subfiles to use require relative and there's no real reason to use "require" for non-thirdparty pieces.


It works both ways, however I feel this is cleaner.


is it possible to write without argument System.exit ?


oops. This change went in accidentally. I'll fix this.


this wasn't a typo, its *Brit.*


I prefer the former, as they get a specialized styling.


I think this test is incorrect. The part of the spec you cited states:

> An ArrowFunction does not define local bindings for arguments, super, this,
> or new.target. Any reference to arguments, super, or this within an
> ArrowFunction must resolve to a binding in a lexically enclosing environment.

This is misleading (at best) or just incorrect (at worst). According to [9.2.12 FunctionDeclarationInstantiation(func, argumentsList)](https://people.mozilla.org/~jorendorff/es6-draft.html#sec-functiondeclarationinstantiation):

> (18. If the value of the [[ThisMode]] internal slot of *func* is lexical, then
>    a. NOTE Arrow functions never have an arguments objects.
>    b. Let *argumentsObjectNeeded* be **false**.

"argumentsObjectNeeded" modifies the behavior in exactly the way you would expect from its name (see step 22)--no arguments object is created for Arrow functions. The algorithm takes care of other concerns (irrelevant to this behavior) until step 27:

> (27. If *hasParameterExpressions* is **false**, then
> a. NOTE Only a single lexical environment is needed for the parameters and
>    top-level vars.
> b. Let *instantiatedVarNames* be a copy of the List *parameterNames*.
> c. For each *n* in *varNames*, do
>   i. If *n* is not an element of *instantiatedVarNames*, then
>      1. Append *n* to *instantiatedVarNames*.
>      2. Let *status* be *envRec*.CreateMutableBinding(*n*).
>      3. Assert: *status* is never an abrupt completion.
>      4. Call *envRec*.InitializeBinding(n, **undefined**).
> d. Let *varEnv* be *env*.
> e. Let *varEnvRec* be *envRec*.

And for completness, *hasParameterExpressions* is defined according to ContainsExpression, which operates as follows:

> ArrowParameters : BindingIdentifier
> 1. Return false.
> ArrowParameters : CoverParenthesizedExpressionAndArrowParameterList
> 1. Let formals be CoveredFormalsList of CoverParenthesizedExpressionAndArrowParameterList.
> 2. Return the ContainsExpression of formals.

Which means that in the case of `(arguments) => arguments`, *hasParameterExpression* is **false**.

If you agree with my interpretation, I will create a ticket on the ECMAScript bug tracker regarding the confusing non-normative note.


Bad commit name! Added Logs folder to gitignore


Hello Thomas,

I already updated the file, but it still shows the error message "There was
a problem with the Amazon library. Error: Secret Key is missing!"

Please check the attachment.

How to debug and find the issue?

Thanks.

On Wed, Apr 15, 2015 at 1:04 PM, Thomas Hernandez <notifications@github.com>
wrote:

> No, I mean that you replace "YourAmazonStore" with the name of your store.
> You should end up with something like this:
>
> $store['Faith-Hope-Love']['merchantId'] = '____________';$store['Faith-Hope-Love']['marketplaceId'] = '____________';$store['Faith-Hope-Love']['keyId'] = '____________';$store['Faith-Hope-Love']['secretKey'] = '____________';
>
> Also, I forgot to mention that, until the next version, you must give the
> store name to each Amazon object if you are using the code in the stable
> branch. You can omit the store name when initializing an object if you are
> using the master branch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-10744159>
> .
>

--
God bless you!
Joseph


what would that prevent?


I'm neither a big fan of hardcoded values... I'll try to rework this part.


I'm not following the comments about positive/negative layer range, but the code makes sense to me :)

Seems like before the layer calculation, the sort_val range is between 5 and 34. And if there is a valid nonzero layer set, that ignores the above and either sets it to on the positive side between 35 and 39, or on the negative side, 0 and 5. That match your intent?


Oops, thanks

On Wed, Apr 15, 2015 at 11:48 PM, Eldar Abusalimov <notifications@github.com
> wrote:

> [image: :beetle:] From here
> <http://pubs.opengroup.org/onlinepubs/9699919799/functions/snprintf.html>
> (emphasize is mine):
>
> Upon successful completion, the snprintf() function shall return the
> number of bytes that *would be* written to s had n been sufficiently
> large excluding the terminating null byte.
>
> That said, the return valueprinted can be greater than env_sz.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/embox/embox/commit/3dce1d40beb7e6d7921ee00f91624568a4bda919#commitcomment-10745049>
> .
>

--
С уважением,
Антон Козлов


Here, `element` is an OK parameter name, since each item in the result set is indeed an element, but a more specific name would be even clearer. Something like `row` or better yet, something that describes well what the data or content of the row actually is.


I hope this was the correct way to do this, I'm completely new to Git so my apologies if I got it wrong.


create a separate class and just every time to contact him?


Thanks and sorry for the inconvenience. ;)


Yes, that's right - the OSM `layer` is a special tag used to force-override the other settings. So it just puts forces those to the upper or lower ends of the range. We can try to get that intent into the comments :)


And here `this` refers to the same `element` that is passed as a parameter to the `.each` callback, and it could be argued that using `$(element)` would be a bit clearer, especially if it was called something like `row`, making the final statement `$(row)`. That's much clearer what we're working with: a table row as a cheerio/jQuery object.


Потрібно використовувати  ng-switch


Great work! Your code is clear and concise, I like it!

I left a couple nitpicky comments above, let me know if you have any questions!


This seems very brittle to me. Can you count on wget always returning an FTP directory listing in the same format? Or, are those generated on the server (in which case, can you count on the server always returning them in the same format)?
When scripts get this complicated, I always reach for Perl, and I'd use their FTP module. I know a lot of people are allergic to perl, but python's another good choice.
A scripting language might not be *quite* as portable, but I think it would be a lot easier to read and maintain.


Comments? Help on editing >_<?


for exampe


The `Scope` constructor and `new` invocation isn't necessary here:

```diff
 var calls = 0;

-function Scope(usurper) {
+var userper = {};
  [1].forEach(value => {
  calls++;
  assert.notSameValue(this, usurper);
  }, usurper);
-}
-
-new Scope({});
```


Are you computing the training accuracy with the Dropout activated?


Wow such excite


OpenSSL enables compression by default, even after [CRIME](https://en.wikipedia.org/wiki/CRIME)?


Okay, I'm done reviewing this patch! Very nice work :)


let's assume it's System.exit(0) :)


good. should reduce support effort on IRC.


Thank you so much for finding and fixing this!   My JIT'd code was not working with exceptions at all and now that I applied this patch, it works fine.


Try `scope.none`


that's what guys on big projects do :) the more of duplication you have the more places you have to remember about to fix a problem in :)


What's the usage model for `insertTrustedKey`?  Does one call `would_accept` and then call `insertTrustedKey`?  If that's the model, then there's a timing hole, since the lock is released between the two calls.  I suppose that's why the `throw` is here.

Assuming I've guessed the use model correctly, I think you're be better off with a `maybeInsertTrustedKey` that returns a `bool`.  Then the caller doesn't have to worry about an exception and the lock can be held for the entire operation.


  * Why such a high value?  Also, the `= 0x*` shouldn't be used except where needed (see the holes in the command lists because someone skipped from `0x09` to `0x10` or similar.
  * Use `// 0x*` instead if the list is long enough that it would be usefull.
  * Comments always document the following line(s), not the preceding line(s).
  * Also, don't use three-line `/**/` comments when a one-line `//` comment will do.


I don't understand the purpose for this patch. feedparser doesn't yet use the autodoc extension for Sphinx, for instance. Also, this really only adds to a total of four functions. What does this offer?


Oops, forgot a `->lock()`!


Why skip all these values?


hey!

Something seems a little off, the submit click handler only gets attached when !('createDomElement' in config) || config.createDomElement) is true, but what if they are both false? If you want to attach a handler to an element created elsewhere?

I added an else as a quick fix but not sure it's the way you'd want to go about it:

	if (!('createDomElement' in config) || config.createDomElement){
		var submitButton = $("<input/>").attr({
			id : config.id,
			'type' : 'button',
			'class' : config.cssClass || submitButtonClass,
			name : config.name || 'submit',
			value : config.label || 'Submit'
		}).appendTo(documentStatusPanel);

		if (!('responseHandler' in config) && config.url) {
			config.responseHandler = this.options.submitResponseHandler
				|| this.swordSubmitResponseHandler;
		}
		submitButton.click(function() {
			self.editor.submitXML(config);
		});
	} else {
		$("#"+config.id).click(function(){
			self.editor.submitXML(config);
		});
	}

Is the idea that people always should create a dom element?


Why skip all these?


One day I will investigate those Rook Contact Checks more closely.
a) Rook supported by Rook is not a case which is considered a "Rook Contact Check"
b) And rook safe check are only computed "at a distance" from the King.

Good luck with your test.


IMO the INT pin is not connected on rev5 ..


No documentation?


Thanks for noticing. Rescheduled.


This env is specific to DGRAM. May be SOCK_DGRAM_DROP_RATE is more appropriate.


Dooone in
8461092e4cc47108d22ec8aa7cb95e46619dbabb


Yeah, I think you could just use `u, err := reqURL.Parse(id); u.String()` actually


This was the buggy code!


Also, this could be moved to the init() code.


PS: And maybe using `NotImplementedError` in `_close()`?


This functions correctly, and the AI appears happy with it - However is quite spammy on the log from MagicTuckCardEvent https://github.com/magarena/magarena/commit/fd02ed68bf0b5420364fdc9ca4a688c660f67c88


Check this article as to why this is happening.  http://hackaday.com/2015/03/12/arduino-v-arduino-part-ii/


These are generic functions so they shouldn't imply that they're supposed to be used with players.
Also, use `const std::string &` where possible to avoid copies, and avoid wide strings if possible (use multi-byte strings instead).


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This is a common pattern in Rails (rendering a partial with some local variables) and so there's a shortcut for it.

```ruby
<%= render 'form', topic: @topic %>
```

does the same thing, just with less typing. :)


Great!


will the cleanup i think


I think it is better to protect under debug flag


Indeed, I changed the type and forgot to change the constant.


Same comment as above for all the changes.


I will see if I have time to finish it tonight but I kinda doubt it.


I wanted auth handling opcodes to be clustered at 0x50.


>
> However, all casts breaks type checking, I didn't get the point here.

Oops, I meant that no explicit cast is needed between `void *` and any
other pointer. That is, `void *` is literally *any object* type and should
be used when the exact type is unknown. So, if one would declare some
object as `void *` and then accidentally assign it to an object of some
other type (even an explicitly typed one), compiler would swallow it
silently.

In, other words, avoiding `void *` is just preventive measures. This is
hardly related to these case :)


Aaand another one :)
Using `__del__` for the runtime assert that checks whether you closed your object correctly?
So just add:
```python
def __del__(self)
    # Maybe wrap this piece into an if to close the stream anyway.
    assert self.__closed, "ConfWriter needs to be closed!"
```


I think this could be more appropriate in either fabric or inside TX context


Not sure now ... maybe I just overlooked that ...


See previous commit^^


hi! the tradevalue/trade and fill/value variables naming is inconsistent between buy/sell (still true on the head)


Yes, I think this is ready for a final review.


this thing works


Daniel, não. O cenário tem que descrever um comportamento executável, para que sirva de base para criar um teste.


sounds good...


No worries. Ping me if you manage to push, if not we will push tomorrow.


Updated.


Follow "The Easy Way™" on http://stackoverflow.com/questions/359424/detach-subdirectory-into-separate-git-repository


It might be better to avoid `std::string` here because it makes copies of everything.  Also, `bytes_v` and `bytes_salt` are leaked.


yes the training accuracy is only approximate. test accuracy is computed with :evaluate() enabled on the model.


it works


as someone who first look at the code today, I find that these 2 blocks (type ==1 and type == 2 further down) look ripe for refactoring into their own methods. Otherwise I find the rest code to be quite readable overall!


What is the sleep for?


vai funcionar assim mesmo que seja com um endereço existente?


thanks m8


We'd like to keep the |eta| cuts; only the PT cuts should be removed.


exato


Tutaj można listować wszystkie dostępne listy użytkownika z linkami do ich wnętrza (/{name}).


Using `std::numeric_limits` would be important in a template where the type might not be unsigned.  Here, it adds verboseness but not correctness.  I addressed this issue by giving the constant a terse name and a brief explanatory comment.


Well we can always push tonight if you want, I've finished up the rest so only the video driver work-around is pending.


`$1$` indicates an MD5 hash.  Use something like `$srp$` instead.


Tutaj wyrzuca java.lang.NullPointerException :D


It's so the message is more visible, but maybe that'll slow builds too much. Appcfg is already pushing a message, so this may be unnecessary.


Ah, simple copy-paste error. Fixed in 2226a6d52e9c2556367a42ab0499d5d0c22cc8cf.


@jordonias What version of mocha are you using and applying these changes to? 2.2.4?


Thanks for reply!


This is fine as we already expose it in lwt.


rather than nesting all of these under "main" can we establish them as their own patterns?


we aren't `crypt(3)` compatible.


One other thing to consider here is that `_to_float` will always return a floating point value or None. If layer_float is not none, we may want to coerce to an integer so that sort_val always ends up being an integer. Probably makes sense to call the variable something else then too :) Maybe `layer_val` or `layer_int`? Or alternatively, you can just coerce `sort_val` into an int afterwards.


Yeah, if you still want the visibility maybe knock it down to half a second or something, 1 second is a pretty long time


Yea absolutely, must have been a formatting error. I'll recommit.


As you prefer. If you think video driver work-around can wait for next time we can push tonight.


This one is deprecated now.


> we think the "in-game" tetris is too slow
Normal, it's server-dependant and strangled by jitter.


I like this pipe system, it makes it nicely readable and compact! :+1:


This matches my original read. I allowed the non-normative note to cloud my interpretation.


Maybe.... Maybe.... We need to make this private, and have a get function handle this.


Another bug is we do not initialize the child of a mapping.
We need to add:
m->c = NULL;
I will patch it after those pending request got merged.
This kills me one day.


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

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


I guess it can wait since nobody cared to report the issue for weeks apparently...


Neither jsoncpp nor lua do. What are reasons to prefer that method?


warum ist der raum außerhalb des spans?


GitHub comments do not allow attachments, so I can't see anything.

What is the exact line you use to construct the Amazon object, and what is the `$store` array in `amazon-config.php` like? With the actual IDs censored out, of course.


lgtm


Ok, tonight then.


Założenie prawdziwe, ale tylko w kontekście użytkownika. Dwóch różnych użytkowników może mieć listy o tych samych nazwach, a przynajmniej nie widzę powodu, dla którego nie mogliby.


Star imports are not nice.


@danielstjules `2.2.4` correct


@pkellner this won't work -- it was correct in 995842f5017a0273b40a1654e9848e71e4169dbc and this commit breaks it


Ok.


Hey @chrisaljoudi and @gorhill.  I see in platform that the uBlock product supports safari and opera.  I was wondering how big of a market that is with installs compared to firefox and chrome.  Those seem to be more of a hidden feature for those two browsers.  But I don't use them much.  I was just wondering what percentage of installs you have on those browsers.

Great job on the vAPI abstraction.  I think I see why you are doing that now reading some of your source code.


No, sorry, that wasn't my intention, I just didn't handle that case correctly.  Here is another pull request which I believe should behave properly.  I haven't gotten to test it yet, but wanted to share it to make sure it looked in line with the existing button situation you have:

https://github.com/UNC-Libraries/jquery.xmleditor/pull/35


Very clumsy attempt to edit this file in firefox on my S3.  This workaround is necessary since in-app file edits are not available


give


Let me know if this is the right place to handle this


*splooch* ;)


Misspelled canola


looks like this function is broken now...

my weather addon uses xbmc.getRegion('meridiem') to detect if the users uses a 12-hour clock or a 24-hour clock.
it now always returns 'AM/PM', even if a 24-hour clock is used.


Oh, this is a nice observation.
It would explain a few oddities regarding previous caching attempts..


Where is the random gone now?


It's more straight forward to just write:
```c
pins[0] = &pin_B13;
pins[1] = &pin_B12;
pins[2] = &pin_B15;
pins[3] = &pin_B14;
```


Didn't optimize launch bounds


@binarycleric this is what I got. You likey?


You could use just an `mp_uint_t` here instead of a bufinfo, since it seems that you only need the len attribute.


Bra där! :beers:


I replaced the set with a map, and `node["comment"]` is now populated properly.


Voila pour mon Site, il est vraiment pas fini, j'ai voulu faire un truc
beaucoup trop long par rapport au temps qu'il nous restait. Je vais
surement l'améliorer donc même je serai pas noté j'aimerai quand même avoir
des conseil en ce qui concerne la mise en place d'une plate forme qui me
permette de mettre des images scans quand je regarde le code d'autres sites
ils mettent d'interminable balise table et div je comprend rien. Merci et à
la prochaine.

https://github.com/KenhyKounkou/Scan-MOF

Le 6 février 2015 12:08, Clément Oriol <notifications@github.com> a écrit :

> Bon, dans l'ensemble, c'est pas mal. Il y a de bonnes bases, c'est très
> correct.
> Je regrette juste : un vrai manque au niveau du rendu visuel. C'est très
> simple, assez terne, c'est dommage. Ca manque un peu de modernité. C'aurait
> été intéressant de t'inspirer d'autre site pour essayer d'avoir un rendu un
> peu plus sympa (jolie palette de couleurs, boutons, choix typo...).
>
> Et il manque le footer que j'avais demandé, dommage !
>
> En résumé :
> Utilisation de git : 3,5/5 Essaye de donner des noms plus parlants a tes
> commits. "ajout du lien google" -> OK / "manque que le css" -> Pas OK ; )
> Qualité du code : 3,5/5 -> Très correct. Manque un peu de prise de risque
> (rien qu'on a pas vu en cours, peu de recherche perso), mais c'est bien.
> Professionnalisme, Rendu Visuel :2,5/5 trop simple
> Investissement perso :3/5
>
> 12,5/20
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KenhyKounkou/repotest/commit/af3113f0e4a29557d084832a6dd0d0fa3ca77705#commitcomment-9621139>
> .
>


`ClosableObject` is what it name says, indeed a comment may be appropriate here: it *can* be closed. E.g. a `Printer` is a `ClosableObject` and can be closed while a `ClosablePrinter` (you know a better name? I see how this name confuses here) then enforces the closing!


yeah right, I confused myself a bit. Good that you reviewed that before @smfn3321  ;)


Could you elaborate on what is the advantage of having `BoolOn` over just using the following?
```scala
w[TextView] <~ On.longClick {
  (textView <~ show) ~ Ui(true)
}
```


You don't need the `/>` to close an `<input>` tag.   That's something that was trendy five years ago when XHTML was in style, but with HTML5 it's fallen out of fashion.


Zamowiona :)


This is minetest, not a generic library for srp authentication. About the const thing, will do that.


Jedna uwaga ogólna. W przypadku akcji z metodą POST można się zastanowić nad następującym sposobem obsługi wysyłania formularzy:
wchodzimy na stronę (GET) -> wysyłamy formularz -> obsługa formularza w odpowiedniej akcji (POST) -> przekierowanie na stronę (GET).
Obecnie przy wejściu na listę GET'em nie dostajemy listy już dodanych słówek. Jeżeli chcielibyśmy je tam dodać to powtarzamy kod z akcji dla POST, czyli średnio. Przekierowanie zagwarantuje nam, że akcje dla POST zawsze obsługują dane (np. dodanie do bazy danych), a następnie przekierowują na stronę dla metod z GET.

Proponuję używać bardziej opisowych nazw (np. zamiast wl to wordList), ponieważ w przyszłości nikt nie będzie musiał się zastanawiać co to jest. Fajnie byłoby wymyślić bardziej opisowe nazwy dla langA oraz langB.

Generalnie elegancko zrobione (poza tymi kilkoma uwagami) ;)


I will let you push first I guess.


I'd make this just admin=False


def requires_admin(self)


Rewrited the whole class.


Really? I thought I will have easier to rebase translations.


Niente avevo levato quello per sbaglio! ;) Tranquillo!

Il giorno 15 aprile 2015 20:44, Marco <notifications@github.com> ha scritto:

> Cosa hai pulito? (Oltre il readme riuppato)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mulasmarco/Panizzi-DePinto2015/commit/a6c93f2aa81c413c2ba4534881ec94ee263e9467#commitcomment-10742602>
> .
>


Hmm.. I revised the Intelli-plug in couple of kernel apks and Intelli-plug is there and works well.


What is the expected behaviour / result for the two cases (12-hour and 24 hour clock)?


Hmm yeah true, I will push now then.


you're missing lrucache.hpp (and related license information if needed) and the makefile.am changes


I am not sure this parses correctly :) By the way, you can make the HTML output more readable by delimiting code with triple backticks and specifying the language:
```
```scala
...
```


`longClick`?


What is this? :confused:


Done, normally the submodule is updated too.


I think this does not parse.


:question:


When I added this feature to Compositor I had misinterpreted how Stylistic Set names are arranged in the <name> table — Stylistic Set names are not sequential starting from ID 256, the names are instead arranged to match the order of the features (i.e. Stylistic Set 4 could be the first Stylistic Set in the font’s features and would be named at ID 256, Stylistic Set 10 could be the second feature at ID 257, etc.)

The correct Name ID for each Stylistic Set is compiled into the <GSUB> table as a parameter of the Stylistic Set’s FeatureRecord:

    font = TTFont(fontPath)
    gsub = font[“GSUB"]
    for featureRecord in gsub.table.FeatureList.FeatureRecord:
        params = featureRecord.Feature.FeatureParams
            if hasattr(params, "UINameID"):
            print featureRecord.FeatureTag, params.UINameID

There is support for calling a feature’s UINameID parameter in [Behdad Esfahbod’s fork of FontTools](http://github.com/behdad/fonttools/), but not in [the version on SourceForge](http://sourceforge.net/projects/fonttools/) (and this patch is built to not cause an error with the SourceForge version).

The table loading order in the Compositor font had to change to make this work, loadCMAP() and loadFeatures() had to come before loadInfo() so that the GSUB table would already be loaded by the time the names were unpacked. I don’t believe it looks like this will cause any problem, however.


Weird, their website is working just fine.. :(


12-hour:  AM/PM
24-HOUR:  /


maybe self.requires_admin?


Script is ready. Next stop: some Heroku instance that runs it once in a while and pushes the changes to this repo.


@valerie-roske

What actions do we take on the Hound comments? Are we expected to resolve them before the request is merged?


How can I free a stack object? I see the malloc, but in this method, bytes_v is a stack object, isn't it?


Let's not warn about this.


Common git ...
rly?
das war die revert note -.-*


I'd be inclined to move the construction of `validation_manifest` down to where it's first used (line 443).  Then I'd limit the scope of both of these variables to just where they are being used by adding some extra braces.  The names are similar enough that, by limiting the variable scopes, we can reduce the likelihood of copy-and-paste errors in the future.  Just my take.  Your take may be different.


Right.  When you think about it, the current behavior makes more sense for reading e.g. from a physical DVD.

Anyway, reading multiple CSO frames from the file (less fread calls) is an improvement but a minor one, so in the case of CSO it's not very important.  I do think it'd matter more for e.g. the index .gz support, and as @avih mentioned the small size of reads is likely hurting the cache lookup there too.

-[Unknown]


rax retry tests


It's working here


1 one line


This feels much less idiomatic than just
```scala
w[TextView] <~ On.longClick {
  val base = 1
  val res = base * base
  Ui { res; true }
}
```
In a way, it is logical for event handlers to accept UI actions, even if it’s just `Ui(true)`, because the handler code will be run on the UI thread, and the input type acts as a reminder. I think that trying to circumvent that might bring some confusion.


2 on line


:+1: I hate the current sound, I'll work to integrate this.


Eclipse trolled me.


Although the type parameter X of ... is specialized, miniboxing and specialization communicate among themselves by boxing (thus, inefficiently) on all classes other than as FunctionX and TupleX. If you want to maximize performance, consider switching from specialization to miniboxing: "@miniboxed T":


Great warning!


Pushed. (Submodule works fine)


Ok, I will update Transifex.


sss


Maybe we could start the listener name with a lowercase letter for consistency with `On` and `FuncOn`?


I will line them up now using spaces, but for the long term we should think about a way of treating these characters. Its a bad idea to ident at the left with tabs, and to align at the right with spaces. Either we do space idents in this file, or we drop the align idea.


Hello Thomas,

It's working now.

Thank you so so much  ^^

On Wed, Apr 15, 2015 at 2:24 PM, Thomas Hernandez <notifications@github.com>
wrote:

> GitHub comments do not allow attachments, so I can't see anything.
>
> What is the exact line you use to construct the Amazon object, and what is
> the $store array in amazon-config.php like? With the actual IDs censored
> out, of course.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boostsup/phpAmazonMWS/commit/2d7627ab25cabcd5c7ae5a6ddd2a2b684ff2c42b#commitcomment-10745755>
> .
>

--
God bless you!
Joseph


Did you intend for all group boxes created to be named scalerBox?


Right, but then there are also models that take only `clean` and `noise` signals, such as the SII, or that take only the `mixture` and the `noise` such as the sEPSM, so I see only two options (although I'm probably not seeing a 3rd and 4th one…):

1. All arguments are mandatory, but nothing stops your from passing `None`. You get something like `srmr_obj.predict(None, noisy, None)`.
2. All arguments are named parameters, so the signature is `predict(clean=None, mixture=None, noise=None)`. It could also be `mix` instead of `mixture`.

Option 1 is the simplest, but 2 is more general. It also allows one to just call `predict(clean, mix, noise)` without worrying about the parameter names and the API is still consistant with the other models. The call to the `SRMR` would be `srmr_obj.predict(mix=noisy)`.


I should make it more general, shouldnt I?  I'll make it groupBox or  something similar. Thank you


Whoops! Overlooked this one. Just fixed in commit d5db15dd00f11960a68f4e7a6c36120880dc4906.


Nah it's fine. The webstore still has the extension at version 5 and the old screenshot though


What happened to this functionality?


Looks great @dleacock!


hmm, I thought that <?= was not technically a short tag? I could have been wrong. I have not been using <? vs <?php for this reason, but for consistency I have no problem with using <?php echo regardless.


just do it again:
git pull https://github.com/vnaydionov/yb-orm.git test_join


Not sure what's up with this. Probably Sublime Text being really annoying and not defaulting to spaces. Will fix in my IDE


Some formatting changes accidentally got lumped into this commit.


what specific template like data is being used here?  Is there something passing through to a lodashed document?  If so what is the parameter... Could just this.fs.copy be used?  The reason I am making this point is that there should be some clear indication of what template data is being passed through.


Are the tests passing or failing with these configurations?  Apparently the bug with yeoman gen .19 has not fixed this as of yet.


you should probably add some tests to prove this works


I don't see that we put anything for `src/lib/util` header files in Makefile.am. I did forget to `git add` the file (fixed now)

I pasted lrucache.hpp's LICENSE file at the top of the file.


I think it is worth discussion on if we should use this moving forward.  I know that they were planning on removing support for `<?p=` in php6, but php6 never happened.  I'd like to wait to use them until we see what php7 plans to do.


Huh.  I'm surprised you can stream to a const reference to a journal.  That looks fishy...


Alla fine non useremo degli `alert` ma delle funzioni definite da Java, anzi per quel che ne so gli `alert` vengono ignorati completamente dalla WebView JavaFX e probabilmente anche da Android.

Shame on you, you make me write things in Italian, what a dead language...


Hi, may I ask why did you revert these changes? I was thinking about adding setuptools back again to be able to use `python setup.py develop` and some other goodies.


All of the tests are passing


Ping?


Since models will have different inputs, I think 2 makes more sense.


should be dcterms:spatial


should be dcterms: provenance


Cool! Re the plane direction!


Merged!


Reverted https://github.com/milessabin/shapeless/commit/78dcfa809deb35306291b42e5f77d566cf3b72a3#diff-e4c28242d8096e41607d2a93b56609dfR164


Давно пора


I think it's much better option to force USE_SDL2 on Linux as it's forced on Mac


I thought this was covered in the Lifecycle document rather than the dev policy?


Glad to hear it.


I messed up the branching.  Resubmitted here: https://github.com/syl20bnr/spacemacs/pull/1189


It would be nice to note that those types come from `macroid.MultiEventTweakMacros`. Speaking of which, should not they be in `macroid.MultiEventTweaks` instead? I would prefer not to expose the `***Macros` objects to the end user.

Would it make sense to have a more general way of identifying the <s>event</s> event handler type? Like `Handler[Unit]`, `FuncHandler[Unit]`, `Handler[Boolean]`.


I looked but could not find it. I may have missed it.
On Apr 15, 2015 3:00 PM, "Mikeal Rogers" <notifications@github.com> wrote:

> I thought this was covered in the Lifecycle document rather than the dev
> policy?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jasnell/dev-policy/commit/625c63dabfce7fa483cff9edbae40035f857dc5e#commitcomment-10746429>
> .
>


“must”


“...isn’t necessary *to* implement...”


Thanks. Could you give https://github.com/Montellese/xbmc/commit/2c266ec9a331a4f96865cf8fd34e6dd2d2efc58b a try?


Following my comments on the documentation, I have to say this feels like magic to me :) I’d be grateful if you could elaborate on your use-case and why the stock handler does not work well.


No one will ever be confused by making scala == and eq behave exactly the opposite of java == / equals they must have thought to themselves...


Nice. Dont know if you saw the questions above bit is copytpl necessary or will copy do?

Sent from my Windows Phone
________________________________
From: Shayne Boyer<mailto:notifications@github.com>
Sent: 4/15/2015 5:58 PM
To: OmniSharp/generator-aspnet<mailto:generator-aspnet@noreply.github.com>
Cc: xtianus79<mailto:xtianus@live.com>
Subject: Re: [generator-aspnet] Merge pull request #108 from peterblazejewicz/refactor/fs (fbe3482)

All of the tests are passing

---
Reply to this email directly or view it on GitHub:
https://github.com/OmniSharp/generator-aspnet/commit/fbe34821d6148a9ea0bd9dd042a42b0d7722238e#commitcomment-10746392


Forgot to save when I added the the necessary echos, etc to make it play nice with PHP.


`_root_.macroid.Ui` (as recommended by @xeno-by)


Ah, because it's still an open PR :) https://github.com/joyent/nodejs-advisory-board/pull/33

I even address how they move over and get merged in to the new ladder https://github.com/mikeal/nodejs-advisory-board/blob/lifecycle/governance-proposal/WG-Merger.md


I get where the name `mOut` comes from.  But I got confused further down the file when I saw `mOut` when I figured the variable should be local.  It *was* local, but I misled myself based on the name.  An `m` prefix is one of the conventions used in this code base to indicate member variables.  Perhaps `manifestsOut`?


@MacDada
https://github.com/dudekm/blog/issues/17


let's do this


Does not compile against latest ctrulib. main.c does not reference new shader file names, and it also contains old functions, etc.


This is awesome!


consume_callback() returning is not the same thing as partition EOF, it just means there are no more messages in the local fetchq (more might be on the way from the broker).
So I think these lines should remain (in msg_consume)


Do you have a specific reason to use several strings rather than multiline strings? I think they look a bit nicer:
```scala
c.abort(
  c.enclosingPosition,
  s"""The ${on.name.toTermName} handler must returns a boolean.
     |You must end it with true or false values, or doing something like this if you want return `true`:
     |    val yourUiCode = dummyAnim(circle, color) ~ Ui { true }
     |    yourUiCode.get""".stripMargin
)
```
Regardless of the string style, the code example does not very idiomatic, as I already pointed out in another comment.


Why do you call `gc()` three times in a row? :)


Note: this wasn't supposed to be here


Hmm... since `ClosableObject` is like it says closable, I would force the closing already here. If it's not mandatory to close an object, you don't need to make an `ClosableObject` out of it^^


I hadn't removed #top initially. But thanks to your comment, I realized I now can.


@dudekm Pododawaj sobie PHPDOCi, żeby NetBeans podpowiadał.

Nad zmienną wpisz `/**`, kliknij `enter`, uzupełnij poprawny typ.


And fixed the /box profile page POST route


@dudekm zaimportuj sobie klasy u góry, żeby uprościć użycia


Nothing was done in this commit, despite the title


Avoid unused private fields such as '$project'.


Avoid unused private fields such as '$deployment'.


Avoid unused private fields such as '$optional'.


PHPCS: Line exceeds 120 characters; contains 161 characters


PHPCS: Expected 1 space after closing parenthesis; found "\n        "


'deployments'   => $deploymentRepository->getLatest($project), // FIXME: Make project injected in the constructor so we don't have to keep passing it


Hi @vrabaud, I found the problem, rather simple. I prepared a patch, but I didn't create a PR because the problem looks like something very particular of my configuration:
```
diff --git a/src/detection/DescriptorMatcher.cpp b/src/detection/DescriptorMatcher.cpp
index 4a5d070..cebd369 100644
--- a/src/detection/DescriptorMatcher.cpp
+++ b/src/detection/DescriptorMatcher.cpp
@@ -211,7 +211,7 @@
         matcher_->knnMatch(descriptors, matches, 5);
         for (size_t i = 0; i < matches.size(); ++i)
         {
-          for(int j = 0; j < 5; ++j)
+          for(size_t j = 0; j < std::min(matches[i].size(), (size_t)5); ++j)
             if (matches[i][j].distance > radius_)
             {
               matches[i].resize(j);
```
In many cases, matches[i] contains less than 5 elements (varies seemingly random between 0 and 4)


This wrapping style is a bit confusing, I would suggest this:
```scala
w[Button] <~ text("Hi") <~ id(2) <~ text("Hey") <~
  MultiOn.click[View](
    (onClick: UnitEvent) => action
  ) <~
  MultiOn.click[View](
    (onClick: UnitFuncEvent) => {
      (view: View) => action
    }
  )


Scott Meyers "Item 31: Avoid default capture modes" suggest thats you capture explicitly.  I think you're capturing only `from` and `msg`, so it shouldn't be too bad...


And with the documentation I mean generally for all member functions, so you write something like how you should derive and what this method is intended to. For example:
```python
def _close(self):
    """
    Closes the stream. Override this method to implement own closing behaviour.
    """
```


Leave note here @jgoldfinch


I don't understand what this endif is for.  It looks like a mistake.


@MacDada
https://github.com/dudekm/blog/issues/18


No, we have our own Linux code and thus we don't need all `sdl_` sources on Linux.


@Optimusprimeums thanks a lot for another successful release!


Nice!  Thanks.


@dudekm Takie poprawki w oddzielnym commicie – nie ma to powiązania z „wyświetlaniem postów”


PHPCS: Line exceeds 120 characters; contains 128 characters


// TODO: Clean this up so we only get the list of servers which we should display do


@dudekm Zgodnie z `PSR` powinno być `getId`, a nie `getID`


@MacDada wiem, to przez przypadek poszło.


This test has a really long name, I would suggest something shorter, such as “work with multi-listeners with default implementations”. The remaining details are not crucial and can be reconstructed from the code itself.


Yes, forgot to push this change.


@dudekm Brakuje typehinta


if ($project->servers->count() > 0) { // FIXME: This should filter to deployable servers


@dudekm Zgodnie z `PSR` powinno być `findById`, a nie `findByID`


@dudekm Plik jest w przestrzeni `\Dudekm\Blog\Entity`, więc wystarczy samo `new Post`


(y)


Mucha carga sobre el thread de la UI la bloquea... ahora anda fluído. O al menos eso me parecía.


All sentences need to be in quotes


Good catch.  Fixed.


No 5.1 kernel update?


@dudekm Ma zwracać napis


What does this do exactly? Can't wrap my head around it. ¯\_(ツ)_/¯


@dudekm wg konwencji `renderUrlInfo`


PHPCS: Line exceeds 120 characters; contains 161 characters


PHPCS: Expected 1 space after closing parenthesis; found "\n        "


'deployments'   => $deploymentRepository->getLatest($project), // FIXME: Make project injected in the constructor so we don't have to keep passing it


well, fuck. too much git add


We had problems with staging quasiquotation in libsyntax in the pass: I believe this module was using `quote_*` like, a year ago, but we needed to switch to the manual scheme for some reason.

Maybe the new implementation resolves these problems?


indeed. it's originally because it's filling in from the outside world json in Application.scala@createSession.
Normally, the path should always be there, but this is safer here how the option acts as a guardian.

You can test that further if the path might not be there when creating session


"plugin needed to handle lto object" warnings on built-in.o linking and
"internal error ldlang.c" error on vmlinux.

And this is bfd linker, FYI.


tested and it does the trick. thx!


Hm, how would I tell? BTW, there was the comment
```rust
    // FIXME #15962: should be using quote_expr, but that stringifies
    // __test_reexports, causing it to be reinterned, losing the
    // gensym information.
```


@dudekm Przekaż sobie zmienne do `execute()`, żeby wiedzieć jak się to robi.


@pjanik Do you think it's ok to postpone initializing globalIframeSave until document.ready? If not, I can just remove the SaveIndicator support, since that's what was needing this change.


It has no well-defined type actually, it's just a wrapper around the list of websockets that delagates the send function.
```
new {
  //[...]
}
```
Creates an object respecting an anonymously defined contract. It's like
```scala
def toThat(a: { def toThat:That}:That
```
A kinda duck typing thus.

However, this guy is hidden that's why I didn't bother creating the class for it :-D


What about using `ENV['DATABASE_URL']` here? In this case we could run it against any ORM.


@marksb We haven't been really great at enforcing this as a team, but the idea is to resolve them before the request is merged. If the team finds Hound to be unhelpful, we can remove it, but it would be good to be consistent with the line lengths and ruby hash syntax.


@westonruter Hey sorry I just realized I pushed https://github.com/xwp/wp-dev-lib/commit/069599442a6232c3d2da5cf59c46eccb850ea80d directly to `master` instead of issuing a PR from a bug branch.

This code did fix the problem for me, however I also had to ensure there was a `.jscsrc` config file in the project root, otherwise another build error would occur.

So the `-e .jscsrc` file exists check may not be working as anticipated here...


Thanks for the heads up, yes I must've forgotten to declare the variables. I'll fix it asap. Hopefully I don't end up breaking everything lol.


@dudekm a właściwie to wymyśl lepszą nazwę – bo co to jest „url info”?


this can still freeze, though less often perhaps, but the state could be changed between these calls


The `pre-commit` hook always runs with the project root as the working directory, so it should be ok.

Thanks!


@helvalius that is a very useful link and a good point. thanks, will switch to one of those planet files :)


@dudekm Zastanów się czy bez zaglądania do środka metody, czytając tylko nazwę, wiedziałbyś do czego ona służy?


Cool.


@olee you might want to look into why this commit fails the Jenkins server and Travis.


@dudekm co jeśli wpis o podanym id nie istnieje?


no need to do this for non-objects, but it won't hurt either


I believe the original problem was that quotation would effectively print literals of the various AST types, e.g. a literal `Expr { ... }`. The compiler's `quote_*!` macros had to be hard coded to whatever definition of the AST they were compiled against, but this meant that changing the AST caused problems in stage0 when bootstrapping: the stage0 compiler's hard coded literals didn't match the definitions in `syntax::ast`.


LGTM.


please no manual locking but always use `CSingleLock`


PHPCS: Line exceeds 120 characters; contains 161 characters


PHPCS: Expected 1 space after closing parenthesis; found "\n        "


PHPCS: Line exceeds 120 characters; contains 128 characters


'deployments'   => $deploymentRepository->getLatest($project), // FIXME: Make project injected in the constructor so we don't have to keep passing it


if ($project->servers->count() > 0) { // FIXME: This should filter to deployable servers


// TODO: Clean this up so we only get the list of servers which we should display do


@offlinehacker Can you try to add completion https://github.com/sferik/t/tree/master/etc?


Hm, I don't believe the current implementation produces any AST structures. However, it does create literal tokens, so I suppose there could be a problem if token definitions were changed...


Are you using the development version of this or the latest released version on NPM?


@djazayeri and @wluyima what do you think of this? https://github.com/openmrs/openmrs-module-uicommons/commit/c0c4455910b69a66fd0fe43c81148deb9d290c0a

BTW, thank you so much @wluyima for giving me a walk through that code after today's design call. :)


typo is preventing me from testing this on mapping


Erlends way of cracking the whip on the devs ;)


Yes, I noticed your work. Currently I like to fix some issues/experiment with the latest release version. But I will switch to the dev version when it gets somewhat stable.


Config Update


@hungrymedia: I thought you were adding a gem here? All I see is a duplicate of the previous commit.


> I don't know why they bother storing 'user' and a user.name.

Because places like Campfire and friends *do* have other data.


Hi Nick,

If you have FastMail set to "Sent Items on SMTP" how do you avoid having messages duplicated in your Sent Items folder when you send via iOS?

Andy


@chulkilee I seem to be unable to view the page with your most recent comments after rebasing everything and `git push -f`.  Luckily, I still have it in an email:

>Actually manifest_path method name is wrong since it is not returning manifest path.

>I'd move the logic to detect actual manifest path to manifest_path method and drop filename arg from it. Then the method is responsible for detecting manifest path.

The point of having a separate method is to reduce the repetition.  Otherwise `release_path.join('public', fetch(:assets_prefix)` will show up 3 times => More work to maintain.  With that in mind, I feel that most of your comments below are not still applicable, but please share your newest thoughts in light of this clarification.

> And then rename the method to detect_manifest_path and store the result to manifest_path variable to memorize in tasks calling it.

> With this refactoring, using_manifest_and_backup_paths method will be just calling manifest_path, not including any logic around manifest path detection. Therefore I'd extract asset_manifest_backup_path method and let tasks to call the method and manifest_path directly, not via using_manifest_and_backup_paths.


At least it works for MBR now with latest code.


yeah this should do the trick


I'm not sure I get all of this code, but don't we want to return `false` here?  Otherwise we'll move `m` into `old` on line 167.  Is that what you want?  If it is what you want, you might want a comment that explains why we're saving `m` when we say we're ignoring it.


Let's use the servo branch here so we don't have two different glutins in use in Servo.


@davidrenne I don't have stats for Opera, but Safari has about 10% of the total user base.


i think this is not a bug, the only thing that we should prevent here is that integer-overflow


I placed them within a pair of braces following a new function prototype. :-)  I can split it into two functions, if you prefer.


This won't work if expr is a numeric literal or a new expression without arguments. Possibly there are other cases where it doesn't work.


Weird, when I was looking I didn't see that. Switching to using that.


I'm not sure I like the `maybe_` functions, to be honest with you. Instead, I'd opt for:

    if (accept_manifest (...))
        insert_manifest (...)


Typo removed.


Got it.


I like the change, but I found the func name a little confusing specially because it's with `MergeConfig` in the same file. Rename `MergeConfigParts` to make it clear about their different purposes and/or add godoc.


I thought we said this should only happen for the confirmation section/question. And this should only happen when the focus gets to the last element


^ awesome


This is used by some code-bases in order to have pass mock clients for testing.


Simplify the return of search results


I'd add a line to separate classes to extend from child components


I'm not sure that `primeTrustedKey` is better than `insertTrustedKey` here. Is this "priming" a trusted key? Or is the trusted key "prime" in some sense?

`addTrustedKey` is concise and descriptive: we're adding a trusted key.


`export default class`


on new line 228: should that reference to linkType not be changed to linkTypeWithKey as well?  Just a question as I'm not sure what the purpose of the url is.


The if condition will evaluate to true only when focus gets to the last element of the very last section.


What the dinkus have I done. I shall fix it

On Wed, Apr 15, 2015, 6:43 PM Sean C. Davis <notifications@github.com>
wrote:

> @hungrymedia <https://github.com/hungrymedia>: I thought you were adding
> a gem here? All I see is a duplicate of the previous commit.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seancdavis/sapwood/commit/5c6b79fe22bda5121c533c649947533f7c3f447b#commitcomment-10747002>
> .
>


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


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


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


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


Is it possible to pass non-String values here?


Bernardo eu não entendi o "continue" aqui, se a condição for false, ele sai fora do "for" ou do "if"?


No rush. Just wanted to get your gem added to `master` whenever it's ready.


you can remove these lines, nothing is coming from above


I might be wrong but i would also think that for the fields in the confirmation question the statement below is true:

field.parentQuestion.parentSection.questions[0] == field.parentQuestion

Makes sense?


A regra do exercicio era que cada Elfo vai atira-se flecha em todos os Orcs?


The new test is wrong again. Its setup as SPRT-Test instead as SPSA-tuning.


I was talking about cmake files.

    if (USE_SDL2 OR (${CMAKE_SYSTEM_NAME} MATCHES "Linux"))

is ugly. You should force `USE_SDL2`


can be set as static as it doesn't require `this` at all


@djazayeri we basically agreed with you to keep the focus on the fields in the confirmation section if the user continues to tab


Oh, but that way we get all sdl_ sources into compilation.


For users of TurboDNN: the scripts will replace this by an appropriate solution


The log message was wrong.  A maximum `seq` means the master key is revoked, so we don't insert a new ephemeral key for signing validations, but we do update the manifest cache with this new knowledge of the master key.


My god.. look how many lines you have coded, GREAT work. :+1:


we should convert all this to yml, json is not good at human interaction


Probably we should remove SDL1 completely then. It's not used on Windows. We use SDL2 on Linux and OS X


A comment would be nice here, to explain why we are storing this.

The short answer is that although at this point we've deleted the ephemeral key, we want to track the fact that the master key is revoked. Imagine, for example, that the master key was compromised: someone could generate a new "valid" ephemeral key, which the server would happily accept. By keeping track of the fact that we've seen a revocation (which has the largest possible sequence number) we ensure that any such ephemeral keys will be rejected.


really nice :+1:


You are very correct. In fact that is what i first did before committing. But i wanted to protect the code from any future changes, just in case we end up with a confirmation section with more than one question. If at all that could ever happen. :)


Thanks Josh.  One function should be plenty.  There's a balance between granularity and comprehensibility.  I'm sure you made a good choice.


I just was reading this code. I like to ask, why uses an addition instead of using bitwise or, to build an word from bytes.
There is some hidden optimization trick here or I'm very ignorant and there is something more fundamental that I don't see.


Sim, só que os comandos para instalação são diferentes

2015-04-15 14:50 GMT-03:00 Joaovictor144 <notifications@github.com>:

> Isso funciona em qualquer servidor linux?
> ex: Ubuntu server 14.04.02
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ramonccj/Tutorial-Ouncloud/commit/5309b880ea42c246f904dd94547fffd7e8d5f472#commitcomment-10741489>
> .
>


Okay, `addTrustedKey()` it is.


@dkayiwa, you are right that the condition only evaluates to true only for the last element but you also assuming that the last element is always in the confirmation section, i wouldn't guarantee that and this is why i would rather explicitly check the type of the parent question model and ensure it is of type ConfirmationQuestionModel


You would rather like to use the lokal codo, instead of the global one. Furthermore, you want that everyone generates the doku with the same tool.


yes the zabbix.agent.servers and .servers_active is legacy stuff that hasn't been removed yet.  the zabbix.agent.conf.Server and .ServerActive are what get written to the Conf file.  If you want to put together a pull request I'd be happy to review and accept.  Otherwise it may be several days before I can look at this.


a direction **after**


why did you remove my coffee task? I really liked it..


J'adore lire tes messages de commit, c'est comme de la poésie ^^

PS : la $factory de la ligne 194 n'est jamais utilisée ;)


Subaddress Identifier and Subaddress type are a little flipped. Subaddress Type would be something like "Tower" SubaddressIdentifier would be something like "North"

(North Tower)


Uhhhhh what?


field.parentQuestion is not assuming one question in the confirm section, actually it sounds to me like field.parentQuestion.parentSection.questions[0] is what assumes there is always only one question which might change as you said.


I don't know why its done that way - we inherited the code.

There is no particular performance advantage I can see from it. In fact because of carries it's probably a good deal worse on some compilers (eg 6502).


Interesting, I didn't do this...


In general, the `getent` command is the preferred way of getting information out of the various databases (passwd,group, etc).   Can you update the conditional to `if getent group fuse` [...]


Fixing


The doc generation is a bit weird, it works if it's hosted, but doesn't work if browsing in github


Ahh yes, that is a good idea.  Either that ore disable remote commands so by default it installs as per the zabbix documentation...


Coding wise I found no errors what so ever. No missing semi colons, code ran efficiently on my computer. I did not have to make any corrections. Flipping between the pages was also efficient because it was done on tabs. The borders and colors were clearly laid out and there was no complications in trying to read the information as the colors contrasted quite well. Now a couple of aesthetic things; When the application was run on a web browser, the text on the second tab stretched into the boarders and was unreadable. As well the picture did not pop up on the first page when ran on web browser. These are understandable seeing how it appears you built this on an android emulator which you should have. Finally, the second page looked a little plain with just a couple of base colors and a block of text. I would have tried to do something a little bit fancier.. however personal opinion not required as the project did more than serve its purpose as a business card. All in all effective coding and good job on the project.

PS: not a fan of the colors chosen ;)


This is too late but I just noticed that the rewrite does more than "violent language", and I am really annoyed.

Where did I write that we experienced any regression in v2.3 in the original? I just told people to find regressions compared to v2.3; if something was working in v2.3 and is broken in v2.4 release candidate, that is something we need to find and fix (or revert).


Hey @DeborahZ could you try to remove this, and commit this again. I think you should be good to merge once this is gone.


`Microsoft.Framework.Runtime.DependencyManagement.LockFileLibrary` = alias this? `DnxLockFileLibrary`


Addresses issue #35


Can we do this manually with Xml for now?


Merge


apprently its been patched & yes it had the exploit,
to use it u go near enemy champion left click him and just "E"


@devvteam i created a ticket for this error - it's #66


more great catches! Thanks!


Or not...


Hi @nathansobo, after this change, is there a way of suppressing autoscroll in operations such as cursor.moveRight? or editor.moveRight? Would it be the intent here to add (options) everywhere where a cursor can move, in case someone needs to suppress autoscroll?
I'm asking because in vim-mode, motions get combined with operations, and the *yank* operation which copies text into clipboard should not always scroll around. However, there currently seems to be no way of preventing generic cursor movements from autoscrolling.


By the way, is there a better way to "hide" the drafts from getting published too early than keeping drafts in draft/ and then moving at publication? GitHub "blame" does not seem to follow renames so it is rather hard to track down where a breakage like this was introduced.


poor TCall :-)


I think we don't use term `metatag`, but `metadata` instead.


OK, I didn't mean for you to copy all the old tests over, but I guess it's OK.  All your assert.equal() calls are backwards though.   Also, it's weird to label this section as ".regression", because everything in the tests/ and testsDOH/ directory is a regression test.


Let us assume the last section has two questions. Would these be the same?
field.parentQuestion.parentSection.questions[0] == field.parentQuestion
field.parentQuestion.parentSection.questions[1] == field.parentQuestion


different*


I'm not fond of `maybe_` functions myself, but they serve a purpose when dealing with thread races.  It looks like there might be a thread race here, but I'm uncertain.

If we're trying to minimize the number of throws we have (which has sometimes been a goal) then my model has been to use compare-and-swap style interfaces (like the `maybe_` dudes).  If we're not trying to avoid throws then just `try` the operation and `catch` if it throws.

What I want to avoid is a usage model that looks like this:
```
if (can_I_do_this()) {
    // Some sly son of a gun could slip in while the lock is released...
    try {
        do_this();
    }
    catch (std::exception) {
        std::cerr << "Oops, couldn't do it after all...";
    }
}


Of course, I think this is not critical issue, so just wanted to keep the PR minimal (although it's already a giant one). I'll add this back with manually XML parsing


"in macro context" sounds misleading. `TypedExpr` actually represents an expression typed in normal compilation context. Or rather, in any context, but since we don't support macro-in-macro yet, `TypedExpr` represents expression typed for specified target.


@Simn correct me if I'm wrong


I'm not sure the exact warning - I used -Wall - but that would be my warning name of choice. :)


Thanks! BitHub has sent payment of  $10.65USD for this commit.


yes.  thanks for catching this.  I've recreated the migration.


Use find_or_initialize


Try to use ruby idioms? Concat instead of +=


This is interesting.  I hope you do try to tune it as you mention in your pull request.  Also, if you expand the array from [7][6] to [7][8] leaving the extra entries simply unused it "may" run even faster because the compiler is better at accessing arrays whose lowest dimension is a power of two.   Good luck!


`$headers[]` is not an associative array, so `isset($headers['Content-Type'])` will always be false.  The fix is to change back to use `$headerName`.

This appears to have been a typo, and was preventing `application/json` payloads from being serialized.


@marcelinkaaa It wasn't a simple icon replacement.


We found and fixed a bug in the PHP template which was causing PHP arrays to never serialize down to JSON.


For future reference, an expectation is not needed here since it would fail on the next line if it couldn't find the Name field.


This is looking great, Brent.  Is 8 space identation the Go standard!?


this is just a comment---you're free to use this on previous releases.


This is intentional. Later after we refactor dnu publish, we will delete this method. We don't need to delete the alias in this way.


points to your home dir instead of M2_REPO ?


no test? :)


Committed at: https://github.com/openmrs/openmrs-module-uicommons/commit/9dcc55e4af1b06c7d1b4cb10296fa095c1a32305


Ah! With that then you are right with that situation. I personally didn't envision the same situation where we will add a new question to the confirmation section because it is NOT meant to have 'questions', i believe we can only add another field to the confirmation question e.g a 'Save and Continue' so that the user is brought back to the form after submitting and in this case field.parentQuestion would always be the same.


Will you explain how this conditional is working? The way I'm reading it:

* Business owner clicks on "Update business"
* The matching controller action will be `business#update`
* It will hit the method listed here
* If the business is sending a PATCH request? (`@business.update`), redirect back to the show page
* Otherwise, the business owner is sending a GET request to the `/:slug/edit` route so that he can update the form?

Please explain if I'm wrong.


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2177 of 2197 tests passed)

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


OK.


This needs to be a singular path name. We got rid of all instances of `businesses`.


perhaps use the `unpause` method?


Is it really needed to check that because it will show down (a bit) the process I think? :-)

According to the doc, it seems we don't need to remove the file before:

> On Unix, if dst exists and is a file, it will be replaced silently if the user has permission. The operation may fail on some Unix flavors if src and dst are on different filesystems. If successful, the renaming will be an atomic operation (this is a POSIX requirement).


See "Restore verify() call in unpackManifest()", which addresses this.


can this be private?


This will break again if more than 1 '=' is present.

Change to ``split(';', 1)``.


I prefer `_err` (or `e` if pressed) - I have to think what `serr` is.


Wrap third party services into our own service


Please review this, @casey-collab


Ok. What was the prob though.  Want to learn from it. :)


Ah, thanks for the clarification, Josh.  If the `VersionEncoding` enums, or whatever end up being called, are used in multiple files then they need to be in a header.  I don't have a sense for any file better than `RippleAddress.h`.   If the enum is at `ripple` namespace scope I would prefer it to be a scoped enum, however.  Better one name running loose than nine.


Form requires it. I see....kind of...


Thanks for the heads up!
Good coding.


s/err/_err/g
s/ret/err/g


@seancdavis here it is.


Okay, done.


Why not just return DirectoryInfo[] directly now so can get rid of the ToArray on line 129.


Blz. Pode Créu.


@iksaku Boy, it sure has been a while. Sorry for taking so long to answer, but thanks for the advice!


I added some documentation to explain things.


Hmm, interesting -- I couldn't duplicate this locally, but it was also reported here:
http://forum.pkp.sfu.ca/t/error-on-uploading-image-on-journal-step-5/232/4
Could you dig a little further?


``if rating is not None and rating.strip():`` does the same thing and is the preferred.style.


This expression breaks the plugin as `self.config` is not accessible from this scope.
See the following error when trying to use the plugin:
>   File "/usr/lib/python2.7/site-packages/beetsplug/fuzzy.py", line 33, in string_match
>     threshold = self.config['threshold'].as_number()
> AttributeError: type object 'FuzzyQuery' has no attribute 'config'


Change to `update_attributes`


Could you expand on what you mean here? Why not?


Not sure what happened. Everything but the "main.c" got committed and uploaded. The newest commit should include this now.


This is really a hack. It has a side-effect, because rotating an Animation has a side-effect.
But I don't want that.
So I had to clone first the current Animation, then rotate it, then execute the normal rendering.
After that set back the original Animation.

All of this of course synchronised.

Also I had to change Chocolate so that Animation.clone() will also clone all of its sprites (in a vector)
I had to change Animation -> val sprites : Sprite*   to   var sprites : Seq[Sprite]


Looks good.


$headers[] is not an associative array, so isset($headers['Content-Type']) will always be false.


"What an exceptional person he is." (?) "they are." (What is "the Ironbaum" referring to? Is it a name or a  people/culture?)


Nice explanation, Nik.  The exact text you wrote should be captured as a comment here.


"Do you not support the commander of the Orc troops in Beruto-ui intersection."

"Do not you" is an improper format. More context would clear this up.


I think I'd prefer that the coordinates print asDegrees.  It's easier to understand.


You can change the file extension to .js


I feel like this would read better as:
```
if ( batch.paused? )
  qless_job.move(qless_job.queue_name, delay: batch.retry_delay)
end
```
Did you do it the other way to save the call to redis?


0. 加了icon挺牛逼的 哈哈

1. 所在城市选其他之后，要不加个框框给填一下？
2. 有时间可以加个检查邮箱格式；
3. 选择学生之后，是不是要加个框框填一下专业？
4. 是不是需要加个社交网络的item，比如微博、穷游？

以上我都可以做哈


[str.endswith](https://docs.python.org/2/library/stdtypes.html#str.endswith) can accept a tuple too:

`if key.endswith(('__commit__url', '__html_url', 'target_url')):`


我觉得先给大家看看吧，看看他们会不会还有什么建议？


Me likey. I'd love to shove everything into a single .sqlite file for the bot, but that could be annoying. I *like* being able to edit crap that I shouldn't touch.


:dromedary_camel: :+1:


This is just temporary.  Feel free to fix it @jacobsims


I see. I think.

Even though this is the IRC adapter, I guess your saying campfire needs user to be set?


If known, could this be explained better?  Is this a bug in the qemu side?


Consider reverting this style change for the comment so it doesn't look like you changed this stuff


Not to spent much time on this, i think when the focus gets to the last element in the entire form, it should remain there. Putting the confirm button before the cancel one in the HTML and floating to the right is actually the root cause of this bug and this is why am thinking that changing the HTML so that actually the confirm button is after cancel should be the most straight forward fix


this.m_strokeStartPosition seems to be only useful for slope tool.
if so, it can be replaced by this.m_endPosition


@rodricios  Merge?


on that note, the hash symbol should have been canonicalized too


1. trivial improvement:
msgPrefix = this.config._strictMode ?  '[ERROR] ContextParserHandlebars:' : '[WARNING] ContextParserHandlebars';
2. suggestion:
will you consider having the package name configurable? say, those using express-secure-handlebars or secure-handlebars may find such an error/warning message confusing


done.


I would if it was made in SVG. It's just a png image.

I'll try to remake it on inkscape


samplePosition is essentially the same as this.m_startPosition


We should combine the maxGroupingSubset with constants that overlap the partitionBy to determine the pre-partitioned symbols.


yeah, understood.
thanks, I will learning more about rails plan, then I can contributing codes good and make sense. :smile:


Yes. Sorry, I was busy setting up 2way auth


rax retry tests


I apologize. Perhaps 'Application.Exit();' does not respect loops (trapping some of the application when it thinks it has exited...)


crap.  all that and i misspell grammar.


how you mean? adding defaults isn't making it hard coded


In my scheme, brush strength is incremented by 0.1 and within the range [0.1, 0.5]
This is arbitrary. <b>We should discuss on this.</b>


s/Missing/Incoming/


it can be a local variable since I've changed the ditch tool behavior.


Not sure why but something in this change caused the entries in my iPad to have images of varying sizes. I readded the constraint for the image view to be width = 41 and that makes it look good on my iPad again, but I am concerned this could reintroduce the issue this commit was meant to solve. Should I go ahead and check it in?


maybe call keys => groupKeys


Here we need to come up with a better name for `sendReceiveMissingFds`. Just renaming it to `sendReceiveIncomingFds` isn't as readable. May be `sendReceiveSharedFds`?


@gc00: I have added some line comments where you missed the "Missing" :-).


去掉告警


I think we can safely say that a "private" fd is one that is shared by exactly one process. That way the logic/explanation would become more general and will follow the code a bit more closely.


I wonder if this check can be fancier, since it doesn't need to check each `if`
````
	switch (class_idx) {
		case JOB_NINJA: if (level >= 10) base_sp -= 20; break;
		case JOB_GUNSLINGER: if (level >= 10) base_sp -= 17; break;
	}
````


No podemos asumir que la gente usa _brew_


trying new merge


This is done by looking up the shared-area to see which connections have been declared as "incoming" by other processes.


ruling out this.m_undoRequest; it only lives for a short while upon request.

ruling out this.m_mouseLeftDown and this.m_mouseRightDown to make it possible to change brush settings on the fly.


This should probably be: rand(100, 100) to be equivalent to the `R` code


@gc00: Let's rephrase the private/shared wording here as per my comment above.


ditch tool uses brush strength to determine height difference.
this.m_trenchDepth = 3 meters.
height difference = 3 at 0.1 brush strength, 12 at 0.4, and 15 at 0.5 (maximum)


@aborkowski The things you have pointed out are features, not bugs. One of the flaws of the Enigma machine was that it would never return the entered letter. This is a 20th century problem that is solved by my 21st century code. The use of 'h' for 'yes' gives users an extra layer of protection against keylogging malware. Regarding the change in ASCII, we plan to fork the ASCII standard should such an event occur, and recommend that our clients use our legacy version rather than any new version which will no doubt expose millions of users to threats.

Thank you for the suggestions, and I recommend you read the FAQ at /dev/null.


I would vote for using https://github.com/typhoeus/typhoeus or https://github.com/excon/excon over RestClient, seems like a lot of gems use one of these. But I may be bike shedding at this point.


changed this.m_trenchDepth to correspond to the unit (meters) used in game.
the multiplier c = 64 is called locally in ApplyBrush()


rax retry tests


Technically speaking adding hardcoded defaults (different for each of 36 types) - is hardcode. And making use of hardcoded "tftdmode" is making it even more. Unhardcoding means getting rid of specific defaults (single default for everything), and getting rid of special "gametype" flag. What's the point? If there's tftdmode, you can make alternative default values for it. Why to bother filling rulesets?


Haha, yeah, it is kind of annoying :) I recently wanted different records in different variants of my ADT and it was just too weird in Haskell.


I was already wading through the code for ideas, seemed as good a place as any. Can remove it if you wish.


if you use "git commit --amend --no-edit" after staging everything you need, you can tack stuff on to your latest commit


tried to pull this directly from a template but no dice - still would prefer that over adding a computed for this in the controller


I'm not sure but this is not enough.
In japanese, we use japanese Kanji.

http://www.rikai.com/library/kanjitables/kanji_codes.unicode.shtml

How about to use https://github.com/mattn/go-runewidth ?


the unsorted pic group doesnt have a number - its undefined


Although if this gets developed further to a proper fix, it will be useful for the following cards:

Karn's Touch
Karn, Silver Golem
Sydri, Galvanic Genius
Titania's Song
Toymaker
Xenic Poltergeist


Added comment explaining what it does


> I would if it was made in SVG

It's up to you, I don't want to bother. The SVG is necessary because I have to resize to so many different sizes, some much larger than the ones seen above (the badges for the Chrome and Opera stores).

I am not an artist, so I got really tired of trying to figure what to do with the icon, and now I sit at the computer and I don't really like that new blue thing I did earlier. All this and and the Chrome store won't even let me know if this is really the issue or not in the first place.


Sorry for asking here but is there any preferred means of communication for bangular?


per 2, the code will continue to refactor to address this, right now, the name of Context Parser Handlers is not correct actually.


Aww, I can't bet more money than I have now :(


unsorted isn't an array of arrays like sorted is


Great!


I'm sure you know this, but this will eventually want to be factored out to a config parameter.


--this file is for the database for CBR
CREATE TABLE recordhead(
                        rcode int PRIMARY KEY,--record code
                        lcode int, --locate code
                        city varchar(40), --city name
                        spot varchar(60), --spot name
                        fdate date, --first date
                        ldate date, --last date
                        ucode int, --user code
                        uname varchar(40), --user name
                        num int --bird species number
                        );

CREATE TABLE birdname(
                      bcode int PRIMARY KEY,
                      family varchar(30),
                      bname varchar(60)
                      );

CREATE TABLE birdlist(
                      rcode int REFERENCES recordhead(rcode),
                      bcode int REFERENCES birdname(bcode), --bird code
                      amount int, --bird amount
                      PRIMARY KEY(rcode,bcode)
                     );

CREATE TABLE birdinfo(
                      code int,
                      bcode char(5),
                      latin varchar(40),
                      chinese varchar(40),
                      english varchar(40),
                      iucn2013 char(2),
                      cites2014 char(2),
                      national1989 char(2),
                      fauna char(6),
                      corder varchar(8),
                      lorder varchar(30),
                      cfamily varchar(10),
                      lfamily varchar(30),
                      PRIMARY KEY (code, bcode)
                      );

COPY birdname FROM './birdname.csv' WITH CSV;

COPY recordhead FROM './rh1utf.csv' WITH CSV;

COPY recordhead FROM './rh2utf.csv' WITH CSV;

COPY recordhead FROM './rh3utf.csv' WITH CSV;

COPY recordhead FROM './rh4utf.csv' WITH CSV;

COPY recordhead FROM './rh5utf.csv' WITH CSV;

COPY birdlist FROM './birdlist1utf.csv' WITH CSV;

COPY birdlist FROM './birdlist2utf.csv' WITH CSV;

COPY birdlist FROM './birdlist3utf.csv' WITH CSV;

COPY birdlist FROM './birdlist4utf.csv' WITH CSV;

COPY birdlist FROM './birdlist5utf.csv' WITH CSV;

SELECT count(*) FROM recordhead; --find the total number of records

SELECT count(*) FROM birdlist; --find the total number of item

COPY (SELECT DISTINCT lcode, city, spot FROM recordhead ORDER BY city) TO './locate.csv' WITH CSV;

CREATE TABLE location(
                      lcode int PRIMARY KEY,
                      city varchar(40),
                      spot varchar(60)
                     );

INSERT INTO location (SELECT DISTINCT lcode.recordhead, city.recordhead, spot.recordhead FROM recordhead);

--choose the spots with most records

COPY (SELECT recordhead.city, spot, count(rcode) FROM recordhead GROUP BY city, spot ORDER BY count(rcode) DESC LIMIT 100) TO './topspot.csv' WITH CSV;

--choose the birds with most items
COPY (SELECT birdlist.bcode, birdname.bname, count(birdlist.bcode) FROM birdlist, birdname WHERE birdlist.bcode=birdname.bcode GROUP BY birdlist.bcode, birdname.bname ORDER BY count(birdlist.bcode) DESC LIMIT 100) TO './topbird.csv';

COPY (SELECT birdlist.bcode, birdname.bname, avg(birdlist.amount) FROM birdlist, birdname WHERE birdlist.bcode=birdname.bcode GROUP BY birdlist.bcode, birdname.bname ORDER BY count(birdlist.bcode) DESC LIMIT 100) TO './topbirdper.csv';

COPY (SELECT DISTINCT birdlist.bcode, birdname.bname, birdname.family, recordhead.city, recordhead.spot FROM birdname, recordhead, birdlist WHERE birdlist.bcode=birdname.bcode AND birdlist.rcode=recordhead.rcode AND recordhead.spot='弄岗保护区' ORDER BY birdlist.bcode) TO './弄岗保护区.csv' WITH CSV;

COPY (SELECT DISTINCT birdlist.bcode, birdname.bname, birdname.family, recordhead.city, recordhead.spot FROM birdname, recordhead, birdlist WHERE birdlist.bcode=birdname.bcode AND birdlist.rcode=recordhead.rcode AND recordhead.spot='弄岗保护区陇瑞站' ORDER BY birdlist.bcode) TO './弄岗保护区陇瑞站.csv' WITH CSV;


역시역시Clan 모바일 웹
4월 14일 완료내역

1.일반회원 기능 (전체완료)
 - 길드정보 조회
 - 공성전 점수 조회
 - 길드전 참여 현황
 - 가입신청페이지

2.관라지 등급 기능
 - 길드원 조회 및 탈퇴
 - 길드원 등록
 - 공성전 점수 등록
 - 길드전 참여 등록

3.개발 환경
-Framework : Spring 3.1, Mybatis
-DB : MySQL, MariaDB
-Server : apache tomcat
-tool : sts(eclipse)
-Language : HTML5/CSS, javascript, jdk 1.7


Hi @Aluxian , you should use desktop-file-install and shouldn't modify file permissions manually.
Like this: `sudo desktop-file-install something.desktop`


Should mention 1.99 as a way to transition?


Teste de comentário


Added My Resume


need to remove Print button from indexpatient in view/history


sweeeet thanks kev


Why dont you just use storageURL declared as static string in deviceRepo instead of the link. It will be easier to change later.


`px` is not a good unit, use `em` or `%` instead.


Nice catch, thanks!


Or, if it needs to be bigger, you can add `fa-lg` or [another size class](http://fortawesome.github.io/Font-Awesome/examples/#larger).


I dont think we need the try and catch block here.


Helium is an app, not a framework


Add Bomber class with a parachute and some sort of launcher.


so nice ;)


Not clear to me why notebookPath has to be an Option[String] here. AFAICT, this should be a required parameter in the new code flow.


Not clear to me why notebookPath needs to continue to be an Option[String] here. AFAICT, this should be a required parameter in the new code flow.


"px" values are more uniform across browsers, but I guess in this case it does not matter. Browser scaling increases both px-specified and em-specified fonts it seems, so there is no practical difference?


ลองดู


Fixes #11


Suggest using an explicit type here for readability.


Shouldn't this be tagged as `xml:"creator"`?


Cool, I will change this. Wanted to make sure.


Looks good overall. Only a couple of nits here and there.


Yes, good point. Especially since the translation will drop any column that don't have an assignment


Can you also share testing done with this patch?


Bangs enforce strictness. So when the `Boid` constructor is called, the arguments are evaluated to normal form (what haskellers like to call "forcing a thunk"). You can do this to function arguments too (in fact, a data constructor is really just a function):
```Haskell
foo :: a -> b
foo !bar = baz
```
Here, `bar` will be forced as soon as it is passed to `foo`.
This is generally a good practice since it can improve a program's memory usage. Anytime we construct a `Boid`, we'll force the evaluation of its arguments.

Of course you don't want that if you're passing in an infinite list or something, but in our case its safe.


Updated


So I just went to compile it and noticed: 1) I have already compiled it on my computer because my Ubuntu uses libc2 19 too. and 2) You never need to compile it. Just symlink to the old one and that works perfectly.


改完了   老师


Kev, port 22 is duplicated here. Just thought I should let you know. Thanks for creating this repo! It's so useful for me! :)


`typeof(toGroupNumber) === 'undefined'` then


The ones you cited do not have this problem as the list of targets is fixed when applying the effect. In this case, the problem is the set of permanents affected by the first static and second static is not the same.


Good deal, that was already taken out a commit or two after. Think I like this solution for now...


don't make them in one line, it's a too long line.
and BTW, what's does these specific number come from, they're not looking like random values. if this is come from gamma curve, please add comments of which gamma.


Does this commit prevent the creation of mortar from using more limewater than available, leaving a barrel with negative amount of water ? I made a stack of mortar and now I have -998 millibuckets of limewater in my barrel.


There was a bug that it was draining limewater based on how much mortar was created, instead of how much sand has been used, which resulted in the negative values. This commit fixes that bug.


@gorhill I tried using the `.patch`-formatted version of your commit, but that wasn't merge-able due to other slight differences in the files.

Did some more research, and I could't figure out how to merge in the changes without having a pull request already there.


@gorhill any idea how to simply import the commit?


same as above.
you can make every 8 digital number in each line. even we don't limit the max alphabets in a line, normally C++ limits to 120.


Sorry, I didn't get a proper place to ask this question. Can we get a release notes for `mongodb 3.0.3`? I didn't find anything either in Jira or in plugin doc.


text correction


Except for Titania's song - Remove it.


i think its fine the way it is


I don't think this does what I want, but maybe I am not reading the README right. It seems to be for bar, line, sparkline charts, etc. Not for browsing, management, commenting, etc. on tickets.


oh, I didn't think the gamma handler was not initialized before this set. It's my problem. you can remove set_gamma_table here when you resubmit with other changes. we'll consider all similar settings in framework change.


Thx...


这个程序是不是有什么错误，我怎么找不到


@aiesteves can you add names of people who have contributed so far (testing your memory! :-)


Thanks Derek! Now is clear for me


The permanents affected by Titania's Song is also fixed, cause the way it would be implemented is first get the list of "noncreature artifact" then for each one apply two effects to it.


So just something like `ln -s tools/sbs/linux-x86_64-libc2_15 tools/sbs/linux-x86_64-libc2_19`?

-[Unknown]


The crucial difference for MagicStatic is that the list of permanents is determine once for each MagicStatic. Whereas in the other cases mentioned, the list of permanents is determines once and the effects are applied to them.


My bad, thought it was a spell. It is an enchantment and would have the same problem as March of the Machines.


コメントありがとうございます。連続性が見えないのと強い意図は無かったので、画像切り替えるアクティビティとコンパスを使うアクティビティをプロジェクトで分けていきたいと思います。
当初はなるべく新規プロジェクト立ちあげの時間を削減するために、既存のプロジェクトにActivityを追加していく予定でしたが、確かに連続性や完成形態が見えないと「なぜここでこの機能？」という疑問を持ってしまう人が出てきそうだなと改めて感じました。さらに毎回プロジェクトを立ちあげたほうが、新規プロジェクトのやり方を覚えられるので、連続性が見えないプロジェクトに関しては別プロジェクトにした方がいいと感じました。


Exactly.


yup. i was thinking of termbox and termbox-go https://github.com/nsf/termbox-go on which termui is based on. recency (and bling UI) bias in effect.

Does termbox https://code.google.com/p/termbox/ make more sense for the application in mind?


heh - I  would never have found this since I just copied it from . . . .


This should probably be one line higher, so we break before the color is added a second time, not after.


da, prav si tova moje da se popravi


Sorry about the delay.  Merging this now but am also going to change the environment variable to C_TAP_VERBOSE (I should have done this with SOURCE and BUILD) to avoid conflicts with other uses, since VERBOSE doesn't have a standardized meaning.


Should've been on master


implementation under process


can we be consistent with tool names, and either describe them by their "action" or their "representative object"?

i think "action" would be easier for everything. draw, delete, move, select, test (for "weight tool")...


Correct, we had to be whitelisted by travis.  So it has to be pushed to a branch on this repo, not a fork.

-[Unknown]


I agree with Merry on consistency


Sounds good to me!


Should these files be in `js/tools/draw.js`, do you think?


Yes, this issue prevented firefoxdriver from loading into browser.


I have the SVG ready, how should I send it to you?


It's unfortunate that we're coupled so tightly here. It would be better if we could say `^1.11.0`. But then of course the lack of anything like bower.lock makes people's apps more fragile...


Wouldn't it be conceptually simpler to make `assets` an atom, and `reset!` it in the watch-dir function?


 if(x() > 800)
        setPos( x() - 800, 600 - y());
    else if(x() < 0)
        setPos( x() + 800, 600 - y());

    if(y() > 600)
        setPos( 800 - x(), y() - 600);
    else if(y() < 0)
        setPos( 800 - x(), y() + 600);


No, they should not be per game. Whether it is a third person game or a first person game is irrelevant. Locking camera angles works the same in first and third person.
It's very much a per-person preference based on personal taste, and a per-session preference based on whether you are sitting at a desk, sitting in a swivel chair, or standing up.
You don't seem to like locking camera angles for third person games, but I do like locking camera angles for third person games. So there is no way we can specify settings per game that everyone will be happy with.
There needs to be a per-game setting for whether it works or not, because many games simply can't read the camera angles correctly.


most excellent! thanks!


数字前后有空格也是合法的，不过我想一般人不会这么干，就没加，我再加上吧。


Accidental regression of 30e21e2.


That logo is much better


Yes it does. The test verify the case of teaser view mode.


Aye, a bunch of homebrew crap rely on the affine transform crap being set to the identity matrix before they even run their basic framebuffer draw crap.


css


Wish this was documented somewhere... not knowing about this rename drove me crazy


Yeah, it was set in the renderer's backend, but not in the I/O register memory.


the manager


Reminds me that I need to recheck all the other registers for the state they're left in when the bios intro ends.


Sonic Advance is picky on that stuff if you skip the bios intro... I think


The audio FIFOs are the tricky ones. Magical Vacation was broken before 900dc3c13f1f062126d5dae14d4f4d4ce5ef97b4 if the BIOS weren't used.


XML


I added this feature to ofxAppUtils after I saw your fork: https://github.com/danomatika/ofxAppUtils/commit/255fc79654dc1ff100b8fbda6afcd8b9eb12e653


I didn't write that code; don't know why it's listed as my addition. But yeah that's a change we can make


Yeah it's not being used so I'll remove that. Any suggestions on where control should return false then?


Alright, see: 0e1a521


Hello.
Before this commit I was able to put any image(**png or jpg**) under ".gif" extension and OBS still was able to show it on screen (on 0.64b). But now I can't (on 0.65). OBS is showing red restangle.

I'm glad if that commit makes OBS more stable and I hope it doesn't breaks OBS's designed functionality.
But secretly I wish to turn things back. =)


@adrian-wang  Why we need a Row with a value of 0 at here?


This should be:
```java
tasks.withType(JavaCompile) {
    // Supresses internal API (e.g. Unsafe) usage warnings
    options.compilerArgs << "-XDignore.symbol.file=true"
}
```


Material 暗色状态栏在 5.0 下可以加一个 Black 30% 的叠层


Or this works too :D


Why on earth would you change the extension of a PNG or JPG to GIF?


lgtm


Leave a line note.kc. Dxtbz,


##This is a markdown file


@ganeza レビューしてね


added some functions to breakout.cpp to make window drawing / refresh cleaner simpler call.

1. created refreshWindow() to allow a single call from WindowProcedure().  When WM_PAINT message is received all will repaint.
2. created paintText() to paint text to the window maybe ultimately to simplistic, for now a functioning placeholder.


@ganeza ここなおしてね


# Problem 8

#Use a for loop and interpolation
for x in post_insts:

#Output a .csv file to make three columns (emo_score, comment_counts, and like_counts)
#Then use excel to make scatterplots.
#One scatterplot is of comment_counts, another is of like_counts.


Clarification on Problem 8 that Excel is to be used for scatterplots, whereas .csv file is to organize the data.


Hive will return a `0` on successful adding jar


You still need to remove this block.


Hmm, right.  Well, tell me what you think we should do: we could force each model to have a `MODELNAME_model` method, or maybe we could use the pyspeckit Registry to test them?


Roger that. 'twas too hasty of a commit.


whoops this one is missing a README... I shouldn't have merged it! can you please add the README to get this to a valid state?


:clap:


Yup. That why we can't have nice things :(