（あんまり詳しく見てないけど）下の方にあるn=+1 というのが気になります
これだとnに+1, つまり1を代入していることになります……が、たぶんやりたいのはnを1増やすことだと思います
そうしたいのであれば n+=1 が正しい表記になります


Please move this above the no-host-prefix option.


@Alexander1998 Is this actually gonna be compiled into a version?


Of course.


I just commit my implementation by using mutex of Yuni:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/9ad71107e98149dc3e3e628e839d79f042437eca
Thank you for your commit and sorry for not accepting it.


If you could make a few minor changes here and do a new merge, that would be great. Thanks for the new option!


Ah, good to know :) I was wondering if I was just doing something stupid…


Yes, indeed.


I already removed all traces of this promise with the last commit :) I can return it back if you want (promise scoped by ``renderThreads``` function only), but there will be no places in code who will use it. Tell me :)


on define will also notify those tests. defines are also "aligned" to late joining observers.


wae


if we don't use it, we don't need it :)


Sanket, use more descriptive variable names.
imageViewWidth
imageViewHeight
imageWidth
imageHeight


Yeah, the ```<br/>``` is pretty strict... it even gets rid of it's occurrence in commits, that's why just ```our.<br>```


Nothing which C or C# don't require as well :-)


Oh! Quite helpful in getting things done though, this doc. Thanks :)

On Sat, Jun 13, 2015 at 1:26 AM Ryan Dale <notifications@github.com> wrote:

> Good point, not as critical though.
>
> Disclaimer: I haven't touched this for like 5 yrs, there are probably
> better ways of doing this by now!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NightFury13/sphinxdoc-test/commit/8d1473d4934ce332a61ed28c1fdfd99226ed40c7#commitcomment-11660925>
> .
>


Thanks to both then, I' ve found this thanks to you! ;) Really a great idea, works fine even on our aosp!


This part could be an issue. It save twice to the database while there is no need.

```ruby
bookmark = Bookmark.create uri: uri # This does an INSERT
bookmark.title = 'Processing URL...'
bookmark.save # This does an UPDATE
```

You could do

```ruby
bookmark = Bookmark.create(uri: uri, title: 'Processing URL...')
```

or omit the title for now and handle that in the models

```ruby
class Bookmark < ActiveRecord::Base
  DEFAULT_TITLE = 'Processing URL...'
   # overwrite `title`
   # `super` call the original implementation of `title`
   # which is autogenerated from the table column names
   # by `ActiveRecord::Base`.
   #
   # If it return nil (or false), fallback to `DEFAULT_TITLE`
   def title
     super || DEFAULT_TITLE
   end
end
```


I'm thinking about this... if `responseObject` is nil, or anything but a `UIImage`, the unboxing should fail, in which case completion is never called. Since we can't trust what's on the other end of the URL to be a usable image, should we add an else block here as a safety net and call the completion block from within it?


"check_c_source_runs" doesn't make sense when WITH_NATIVE_INSTRUCTIONS is turned off as then we rely on run-time test which can actually be faster than when disabling whole optimization. We could further possibly optimize it by short-circuiting the run-time test to always return false.


I also modified the code to use the latest entry in each 24 hour period for the probe readings (excluding temperature).


So it looks like removing this entirely sets it to the defaults, which is totally open.
![screen shot 2015-06-12 at 4 25 36 pm](https://cloud.githubusercontent.com/assets/903219/8139333/c053ed86-1120-11e5-8955-3b9661ae8070.png)


Published to http://ui.edev.d2l.com/70_c9fcee7b33f0d33707614cd9924e4383e15de082


Cambien el void "nuevo"  por "limpiar", para usarlo en otras diferentes acciones.


Aren't these instances (other than the 10xl) ebs optimised by default (like the c4 instances) ?


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

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

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

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

```


This is a test reme file


`var now = new Date();`


Same thing happened here.


@StefanKarpinski What do you mean "What kinds of object should they be?"  They are simply types, used for parameterization, no values.
You seem to have little faith in the Julians' ability to get a design right at first... considering the breakage that happened with the "Tupocalpyse", and the major breakage planned for "Arraymeggedon", maybe I can understand that attitude.
If it is an approach that I approved of, such as Jacob's, I'll take your bet...
$10? $20? A round of drinks at the Muddy Charles?  I've already done this, twice (not twice because I got it wrong at all the first time, but once for single-byte encodings, back in '92, and then extended it for multi-byte encodings and Unicode in '96, but nothing at all broke from the first design...)  What I handled then was a lot more complex than the simple cases here, which are simply subsets and encodings of Unicode.

@quinnj Should they all be marked as `abstract` instead of `immutable`?  Does it make any difference, since they are only used for parameterization?
Also, I disagree that Encodings should be introduced along with larger changes (I've had too much bad experience with trying to get a sweeping change in, even just to fix outright *bugs*).
Better to get it in, have it used in a limited case (i.e. `check_string`), let people "kick the tires" a bit,
before making it exported and documented, and trying to use it (and `Strings.jl`) to replace the current string handling mess...


CSS Masks not supported by: IE (8,9,10,11), Firefox (37,38), Chrome (42,43), Safari (8), iOS Safari (7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), UC Browser for Android (9.9)


CSS Filter Effects not supported by: IE (8,9,10,11), Opera Mini (5.0-8.0), Android Browser (4.2-4.3)


CSS outline not supported by: Opera Mini (5.0-8.0)


`var firstDate = new Date().setHours(dateHours[row][0][0], dateHours[row][0][1]);` ?


Imported RouteConfig from angular2/router and decorated my App component:

@RouteConfig([
  { path: '/', component: CustomersComponent, as: 'customers'}
]);
export class App {}

ES6 module loader is throwing an unrecognized annotation though (doing in-browser compilation right now to keep it all simple): Unsupported annotated expression (WARNING: non-Error used)

I'll keep playing with it.


prefixes* – you can just edit this file in GH too! https://github.com/oomphinc/origins/blob/72504e469113ed5a5257e8dae287f91be53e9289/README.md

perfect for tiny little changes like this


`redirect_to` is pretty smart

`redirect_to action: 'index'` will work because if not specify, it will assume that you are talking about the current controller.

`redirect_to ':index'` will also work because if you only get a symbol as arguments, Rails will assume that you are talking about the action.

Actually, you can also pass any object that response to to_params

```ruby
class TrollObject
  def to_param
    '/trolololo'
  end
end

redirect_to TrollObject.new # will redirect to /trolololo
```


Using `@RouterConfig` helps since you can use the DirectiveMetaData reader to read all metadata for a component.
oops sorry that was a private repo here's the correct link.
https://github.com/angular-class/angular2-webpack-starter/blob/a3ad6384235a29953f2c4705ac837ab2b9b54bf4/src/app/components/app.ts#L49


I think this variable can be removed, unless you intended to use it for something else. It doesn't appear to be referenced anywhere else.


> You probably want rm -f there, not rm -rf.

The change to rm -rf is deliberate. As per the commit message, some systems (MSYS2) implement ln as cp -r so using rm -rf is necessary for removal (and harmless when ln creates proper symlinks). Yeah you can argue for removal of that part but it's so minor that I don't see it as a big deal. If Bryan wants me to split that off into a MSYS2/Windows-as-build pull request then that's also fine.

Regarding the symlinks, I'm strongly under the impression that they were workarounds for legacy bugs and are not needed in any modern scenario, therefore I don't do any of them for multilib where they muddy the waters even more than non-multilib. Once it's bedded in under this highly experimental feature, I'll be looking to remove all of that sysmlink stuff.


This gets oauth working, but styling needs to be fixed.


this a comment on a commit


No they are not.


fddf


This is quite inefficient - there's no need to query the modulestore at this stage. As far as I can tell, you can replace this with:

```python
chapter_key = course_key.make_usage_key('chapter', chapter))
section_key = course_key.make_usage_key('sequential', section))
vertical_key = course_key.make_usage_key('vertical', vertical))
```


OK-ish. I have some other comments, but since this is a moderately urgent issue, that's for later.


Since you're changing this you should use the new syntax `unicode(items[0].location)` instead of `items[0].location.to_deprecated_string()`


:+1: :+1:


Hurray! Doctests! :+1:


On Friday June 12 2015 13:32:26 Mika Lindqvist wrote:
> "check_c_source_runs" doesn't make sense when WITH_NATIVE_INSTRUCTIONS is turned off as then we rely on run-time test which can actually be faster than when disabling whole optimization.

No, I don't agree, since even if we confine the use of -msse4 and/or -mpclmul to a single compilation unit, nothing guarantees that the compiler will not use those instruction sets where it sees fit outside of the runtime tests.

Besides, why make the cmake file more complicated then necessary by having check_c_source_compiles in addition to the _run version?

That is not to say that there could not be a way to circumvent the tests altogether, for those who want to do a cross build.

R.


Looks OK. I have not yet tested the code, but this is the beginning of supporting different layouts, like MongoDB university, stanford, whatever. I like it.


I like what you did there!


You're indeed right, i absolutely missed that!


1440-AWS (axelstudios) - 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)


Even better!


OK, this will grow. :)


Good.


These cass_data_type*_custom* decls appear to be undefined elsewhere. Leftover from hacking?


Fix added, thanks Eric!


I wonder if this should be main.js, and we should rename the other one as main_legacy.js. I'd like to make it clear that all new tests should go in here.


ok cool will update PR


Dan,
when testing this commit, with the sakila sample database, I open the db, click on Routines and get an error 500. Error log says:
````
Class 'SqlParser\\Fragments\\GroupFragment' not found in libraries/sql-parser/src/Statement.php on line 147
````


Кроме такого варианта:
```html
<div class="tt-event-time">
    <span>9:20</span>
    <span>10:55</span>
</div>
```

Могут быть ещё два ([yandex.atlassian.net/browse/XD-34](https://yandex.atlassian.net/browse/XD-34)):
*У события может быть время начала, но отсутствовать время окончания. В этом случае событие остаётся выделенным до тех пор, пока не наступит другое событие или не наступит новый день:*
```html
<div class="tt-event-time">
    <span>9:20</span>
</div>
```

*У события может отсутствовать время начала и время окончания, в этом случае событие никогда не выделяется:*
```html
<div class="tt-event-time"></div>
```


Could you possibly write a single comment without being a dick?


Published to http://ui.edev.d2l.com/71_bb9adfc23d3d5526260964719aca7642a57a78bc


The commit message seems correct. `--tree-view` != `--tree view`


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1163 of 1163 tests passed)

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


1440-AWS (axelstudios) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Oh, the title....


I might be worth discussing here, there, or somewhere what the distinction is between cass_data_type_new_udt and cass_user_type_new_from_schema.


the kernel seems to just randomly cut off...


@davixd I do not understand what you are trying to achieve here? Could you explain it to me?


Curious if keyspace and type names are necessary for this route. I don't think there is any checking, and maybe we don't care, as long as the types line up?


For API convenience, it might look like this:

```js
get broccoliFeatures() {
  let features = this.__broccoli_features__;
  if (features === void 0) {
    Object.defineProperty(this, '__broccoli_features__', {
      writable: false,
      enumerable: false,
      configurable: false,
      value: features = {}
    });
    return features;
  }
  set broccoliFeatures(features) {
    if (typeof features === 'string') {
      features.split(/\s*,\s*/).forEach((feature) => {
        let flag = feature.charAt(0);
        if (flag === '+' || flag === '-') feature = feature.slice(1);
        this.broccoliFeatures[feature.trim()] = flag !== '-';
      });
    } else if (typeof features === 'object' && features) {
      Object.assign(this.broccoliFeatures, features);
    } else {
      throw new TypeError('features must be a string or Object');
    }
  }
```

This would make the API pretty elegant IMO, something like:

```js
class MyPlugin extends BroccoliPlugin {
  constructor(inputTree, ...args) {
    // Pretty readable way to do it, right?
    this.broccoliFeatures = '-read, +rebuild';
  }
}
```


Wouldn't the following work?

```c++
hana::for_each(hana::range_c<std::size_t, 0, Rows * Columns>, [&](auto i) {
    using T = std::remove_reference_t<decltype(data_[i])>;
    data_[i] = static_cast<T>(rhs.data_[i]);
});
```

Also, don't use the detail namespace, you sneak! :-)


Why?

Le ven. 12 juin 2015 18:30, Loïc Payol <notifications@github.com> a écrit :

> You said it could be used for any topic, but ... Not actually the case
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/palra/univ-mmorpg/commit/2569a2123a7996cd295a279062ca20862998490f#commitcomment-11657546>
> .
>


Great! I was going to do this, but you were faster!


@wkennington With this patch I can build ncurses shared lib on darwin, and use it to build mariadb (the commandline client apparently uses ncurses). Thoughts? I've not tested on Linux, and am unable to test on Cygwin.

It's perhaps a little sloppy, but what do you think about the general direction?

(I can't seem to just set `shotp -s extglob` in postInstall, because `eval` or something complains about syntax errors in the extended globs used, even if they wouldn't actually be used until `extglob` is turned on.)


Since we export this file as syncthing-versioning(7) manpage, a proper description (=Introduction) section is needed.


Good!


Great.


1440-AWS (axelstudios) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Строк в таблице может быть любое число, не обязательно 4.


commit comments


Humm, I missed this. I thought that the former version would work...


There's a small issue with SpawnEmitter, as it returns the emitter and therefore causing a TypeError (TypeError: this.m_rgActiveParticles[7][0] is undefined) when updating the particles in running.js:
```
this.m_rgActiveParticles[ 7 ] = [
                    this.SpawnEmitter( g_rgEmitterCache.healing_wave_crosses, 1280/2, 720/2, this.m_containerUI),
                    this.SpawnEmitter( g_rgEmitterCache.healing_wave_mist, 1280/2, 920, this.m_containerUI)
                ];
```
and so on...

Suggesting to update the code like so:
```
window.g_Minigame.CurrentScene().SpawnEmitter = function(emitter) {
        emitter.emit = false;
        return emitter;
    };
```


Excuse me???  Just *what* is your problem?


Well.. For me the difference in execution speed is about 48% in favor of using _compiles version, so I see it as an regression... I only could see one place where the code differs when the specific optimization is turned on but the processor doesn't support the instruction... I've been running the code with all optimizations short-circuited without any crashes since they were added.


BTW, your comments do not conform to the Julia community standards...


This function writes a FAB to an hdf5 file.


Ugh. json encoded in escaped HTML...


Nice and simple, I like it.


You can create an `api.Health` client directly with a different address and inject it. I documented how to do that [here|https://github.com/Wikia/go-commons/tree/master/consul].


Can we merge this apt-get line into the one we have in `install_postgres` in [util/install/install.Ubuntu.sh](https://github.com/HazyResearch/deepdive/blob/develop/util/install/install.Ubuntu.sh)?  The scripts there are our new installers, and we don't want Travis to diverge from that.  The `make depends` below doesn't install postgres, so you may need to add a line after that:
```bash
   - make depends
   - util/install.sh postgres
```


"who" is for persons. :) You meant "videos that...". ;)


Great work!


que me haces followeando xD


It's not in the spec, what does namespace do on code generation side? Is it much different from the module?

Guessing from the wording, I'd expect it merge the scopes contrary to modules. Or is it one of those cases where a new keyword is preferable to the old one?


Функции `checkDate()` и `chekHours()` можно объединить в одну `checkDate()`. А так как код этих функций больше нигде не используется, кроме как в `findMarked()`, то все проверки можно перенести внутрь `findMarked()`.


Types.ID(m.To)


This is equivalent to `T::size == columns`, isn't it?


ovo sam dodao da bi sidebar mogao da se koristi dinamicno. Ne naravno sa ovakvom primitivnom funkijom, ali nekom nadogradnjom ove f-je.


@spinda unfortunately that doesn't work because `Cabal` (the library) knows nothing about sandboxes, so your expression just evaluates to my hard-coded path. The sandboxing logic happens entirely inside `cabal-install` (the binary) which doesn't export an API.

The sandbox hash appears to be calculated based on the sandbox directory (https://github.com/haskell/cabal/blob/master/cabal-install/Distribution/Client/Sandbox.hs#L137-L158), which we can heuristically assume will be `cabaldir </> ".cabal-sandbox"` and catch most cases...

Really though, I think this is a sign that we should try to avoid putting any cabal-related logic into LH.


OK, good.


This already happens. The only caveat is the @version parameter needs to be incremented every commit.


I tested the batch edit and it works
5. table editor. Add notes as a field that can be batch edited ** DONE


this should be .dll.a with mingw


Good!


This is still not right.

You want to build the string here with the "cheese" you get:

    return "My favorite cheese is \(cheese)"


Nice - great info. For whatever reason it's not liking @RouteConfig() (see error above) so I'm back to using it in the component. I'll circle back once I get some other issues resolved. Thanks again - really appreciate all of the info.


clang fails on this though :(.


Just curious, what was the reasoning behind the revert? Was it discovered that this was the correct formula after all?


My problem is that you are continually passive aggressive, argumentative, and dismissive of other people's hard work and disrespectful of other people's time and energy. Now you are trash talking the design decisions that have gone into Julia and the process by which we decide on and handle breaking changes. This is beyond galling and is incredibly insulting.


Why is this `!=` rather than the more classical `<`? I am intrigued.


Thanks for the fix!
When would this be available in the pypi package?


Shouldn't there be a different message for Creative-Mode Error and First Inventory-Slot Error?


I think it's already pretty inconsistent. I'd rather just use it where it adds clarity rather than make a rule about it.


As soon as I get confirmation that it fixes the problem. Does it?


OK. This should be good, but I think it is more correct to return NULL when er == would_block, since we did not actually receive a connection.


Kaphotics didn't factor weather into his calculations, so the extra damage from the low roll wasn't because the order was reversed. It was sandstorm.


Можно заменить на:
```js
var crtUserDate = new Date(now.getFullYear(), now.getMonth(), now.getDate());
```


> Shouldn't there be a different message for Creative-Mode Error and First Inventory-Slot Error?

Yep...


Здесь к месяцу прибавляется 1, а дальше в `checkDate()` из переданного `month` вычитается 1. Это как-то странно. Но в любом случае эти переменные лишние. Ниже написал, как можно обойтись без них.


RIP


@magcius I am looking through all the branches of the metrics repos at Cosimo's request. Curious if we are still interested in recording OS info or if that's no longer a priority.


indeed, I'll do it on Monday ;)


Does this do the right thing?  I thought there had to be a python directory in the top level.  Maybe that is an LSST specific thing.


Thanks for setting up the testing framework. It's been on the todo backburner for a while


You have deleted this whole file, but these settings are not duplicated in the tests and the commit message does not say why.


I think you misunderstood, MD5 was already done with this commit and works just fine. It's just that I did it without actually understanding mask mode - like you said MD4 and MD5 are so very similar I could just copy everything but the actual hash function.


Is that FIXME true, btw?


Habit from C++ iterators?
I think `<` is better for integers though.


Please doc why you do that, thanks. :)


@jmurphyau did all the work.


`createRedux()` already binds, so we don't need this. I changed this here rather than a separate PR because it made testing easier.


kek


Please push this updated Pod to repository. Latest uploaded version is Using TOWebViewController (2.0.14)


Sure, Done


FYI: The ChromeOS firewall bug linked above has been marked as fixed.


fix to SignalFx (my miss)


This code is an exact duplicate of code in control.py.  I would much prefer to keep the constant overrides in one place.  That provides an obvious place for documentation and there is no chance of the two getting out of sync with each other.  If you moved constants.py.small to constants_test_override.py you could do something like the following:
```
import great3sims
import constants_test_override as constants

great3sims.constants = constants
```
I think something like that should work.


I think we had an issue for this... but I can't seem to find it


ggyyhjjj


Fix this.


oi knp diganti itu


change to * 1000 (with spaces)


I think this should be reverted, for reasons already expressed.


Yes it did fix the issue for my application.


Is the template being used already, or is it there for when we introduce a more complicated test? I don't know much about JS testing, so bear with me: wouldn't it be better that tests that observe contents of pages be acceptance tests in Rails, and the JS unit tests have self-contained, synthetic cases instead?


CSS Masks not supported by: IE (8,9,10,11), Firefox (37,38), Chrome (42,43), Safari (8), iOS Safari (7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), UC Browser for Android (9.9)


CSS Filter Effects not supported by: IE (8,9,10,11), Opera Mini (5.0-8.0), Android Browser (4.2-4.3)


CSS outline not supported by: Opera Mini (5.0-8.0)


This is a reply to the comment


Can you increment version number?


Yeah. Also, nobody is really updating these.


We should get rid of (all instances of) this.


Use `.toString()` instead of `.toRegExp().toString().slice(1, -1)`


Done.


* TODO: Don't allow this to run if there is already a pending deploy or no servers


* TODO: rewrite this as it is doing way too much and is very messy now.


* FIXME: This does not appear to work.


// FIXME: Change this so it is link the other 2 notifications


If render is called multiple times, wouldn't this be a problem? I think you need to set the HTML.


Heya Andy,

Nope but why not? :)


Does this mean order of flags no longer matters?


I'm not sure I understand this one.  Why not complete replay and then clear the temp collections?  Don't we need to do that anyway in case we replay the full create->write->write->clone case?


@claudioandre problem seen on nvidia. Besides, the python-style indentation block only fooled our eyes, not the compiler ;-)


See here: http://agar.gcommer.com/index.php?title=Protocol#View_Update_.28opcode_17.29


Oh, a later patch does cause temp clearing to happen after replay.  Is this still necessary?  It seems like having write behave differently between replay and not replay is brittle.


I would add a comment to that effect.


I want to put a transaction here for these two database queries so that if one fails the entire thing will be rolled back, but not 100% sure how to do it. Thought we could discuss it at our next meeting.


Did this cause a bug?  For a primary, while active, they are the same (once the initial activation exchange completes).


How do you mean? For example, the order of flags has never mattered in the sense that they could be anywhere after the last subcommand. However, in the positional sense, order has always mattered. For example, calling `rack --json servers instance list` prior to this PR would have given you an error since the `json` flag wasn't global. As of this PR, the format must be:
`rack [global flags (currently only --json, --csv, --table)] <service> <subservice> <action> [command flags (like --fields, since they're specific to the subcommand)`.


1440-AWS (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2126 of 2202 tests passed)

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


Actually, that's not true.  While peered, but not active, you want the current interval start, not last_epoch_started.  So, you want same_interval_since, I think.


I am, but I never got a chance to respond to all of these nits.


Needs blobs to be uploaded. :cry:


No, I've copied/pasted the original minimap files and didn't see that ^^


ok. I will add something. Moreover, I will also add the override tag


Good catch, it's a leftover from some previous version I think.


Thanks!


The `auto-mut` terminology is confusing here since `mut` has not yet been introduced. Maybe a one or two sentence description of `mut` is needed before the term `auto-mut` is used.


at least please comment to say what it's doing :)


If this happens, then it will return to Pharo without restarting the timer. I don't know what the effect will be.


* TODO: Don't allow this to run if there is already a pending deploy or no servers


* TODO: rewrite this as it is doing way too much and is very messy now.


* FIXME: This does not appear to work.


// FIXME: Change this so it is link the other 2 notifications


yep, that's what this does


Looking at this makes me wonder if "EventDispatcher.once" shouldn't return a promise :)


The problem is `regular` still contains those 10 entries that contain more than one code point, i.e. these:

```js
	"\\ud83c\\udde8\\ud83c\\uddf3",
	"\\ud83c\\uddfa\\ud83c\\uddf8",
	"\\ud83c\\uddf7\\ud83c\\uddfa",
	"\\ud83c\\uddf0\\ud83c\\uddf7",
	"\\ud83c\\uddef\\ud83c\\uddf5",
	"\\ud83c\\uddee\\ud83c\\uddf9",
	"\\ud83c\\uddec\\ud83c\\udde7",
	"\\ud83c\\uddeb\\ud83c\\uddf7",
	"\\ud83c\\uddea\\ud83c\\uddf8",
	"\\ud83c\\udde9\\ud83c\\uddea",
```


@doutriaux1 is the rename from `dimname` to `dname` going to cause problems here?


done, didn't fix the OSX non standard sequence problem


mmm I don't like this so much, but now it's too late to think properly. I'll see if I can think of something tomorrow.


Que carajo es esto?


I reverted this commit in https://github.com/WikiWatershed/model-my-watershed/commit/1dfdd42bd15d6eb5ab731697fdba6a046f00e6a2 and created an [issue](https://github.com/WikiWatershed/model-my-watershed/issues/386) to look into a better solution.


`unitless(..)` returns a boolean, not a number without unit.


I am trying to re-work it. Unfortunately there is no stand-alone usage of `mut` yet, it makes it difficult to introduce without usage.


Looks Ok to me.


derp this should be `unit`
http://sass-lang.com/documentation/Sass/Script/Functions.html#unit-instance_method


In my phpinfo.php, nginx version is 1.5.11, how can I update nginx core to 1.7.10? Do I have to compile and move it into dir usr/versions/1.7.10.1/nginx/sbin ?


What workaround would you recommend?


just pushed the fix


uhm ... manual inclusion/exclusion is probably out of the equation. will try to see if I can automate this. if not, I think I'll stick with the curent safe way to match expected sequences.


Great link thanks! So I guess there's no reason to handle any of these info in here, I'll update the comment


Rate can be negative, and can also be much higher than 4.0


Could we rename this to just `edd_pie_graph()`?


I think @gridaphobe is right, but in the short term some shady
hack to handle this should be ok. (Perhaps easier to just depend
on the GHC version rather than this macro.)


You can move this into the lock


You could detect the multi-code point entries programmatically, and exclude them from the array that gets passed into `regenerate`.


:+1: good catch


> How do you mean? For example, the order of flags has never mattered in the sense that they could be anywhere after the last subcommand. However, in the positional sense, order has always mattered. For example, calling rack --json servers instance list prior to this PR would have given you an error since the json flag wasn't global. As of this PR, the format must be:
> rack [global flags (currently only --json, --csv, --table)] <service> <subservice> <action> [command flags (like --fields, since they're specific to the subcommand).
>

+1


`unit(..)` returns the unit of the value, so I am not sure what you are trying to achieve here.


ah yeah, wasn't immediately clear since it's up here separate from the rules that are using it


Now seeking will never work if rate != 1.0 :) You only can use SEEK_TYPE_NONE if you only change the rate. If seeking to a position happens you need to provide it


Also when using SEEK_TYPE_NONE you can just use GST_CLOCK_TIME_NONE as position.

For a first version, just keep it at SEEK_TYPE_SET all the time and seek to the current position. This can be optimized later.


Keep this, and just additionally add the rate here


do you realize how many appveyor builds have failed this week because of broken stuff pushed to master? please go through a PR if there's any chance of breaking the build


You chmodded this file now, too.


nope sorry this was not the correct fix either. basically the root of the issue is

> On windows, the scss compiler fails due to it being unable to handle converting `px/px` units to `em`.

on OSX this is not a problem, it seems like something is off in the windows bin


Better use [Vector3](http://docs.ros.org/api/geometry_msgs/html/msg/Vector3.html) here.


Thanks @HugoGiraudel we are trying to work around the bug in the sass compiler that sam linked to above.
Would appreciate your input on a better workaround, or better yet, a fix for the root cause.


you are right, if they would be defined in an external stylesheet styling svgs that are embedded in html via <svg><use xlink:href=""/></svg> is only possible in firefox.
But it is possible to have style blocks inside the svg. If you have lots of svgs exported from illustrator, they will all have the same css-class-names so if you combine them in a single spritesheet, they will influence each other which is not desirable at all... therefor I preprocess the individual files (with the awesome extract-svg-styles node_module) and prefix their css-classes inside their inline style definitions with a classname created from their filename and put this class on the root svg element in the single svg file. upon spritesheet generation, one could just copy this class or take the name like in the approach above, since it is the same anyway, but i just did find it easier, than getting the class attribute of the shape. i hope it is more or less understandable..


This change appears to fix an otherwise broken `--enable-builtin-atomics` option, which doesn't exactly line up with the commit message.


Maybe use a spin button for this? Or a slider?


Why should this ever be NULL?


You cannot return an error as a text


(See https://github.com/sass/sass/issues/1743)


really?


If you used a separate build subdirectory and then ````cmake ..````, you wouldn't need these ;-)


You cannot solve it on your end, unless you strip the unit entirely, which I do not recommend. Just wait for Sass to fix it I believe.


1440-AWS (axelstudios) - 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)


The fix is easy: strip the unit entirely, then convert it to `em`. It sucks, but it should do the trick.


You could use `promote_type(T, FloatingPoint)` here. Or you could take inspiration from what `middle` does, as the goal is very similar here: `typeof(zero(T)/1)`. Not sure which one is the best -- I think the second is more general, though in practice it shouldn't make much of a difference.


:+1: thanks @j2sol!


How about stopping more?
key 3 = if drop exit then 93 emit then ;


Just an FYI, this breaks the Sales Report email plugin:

http://www.woothemes.com/products/woocommerce-sales-report-email/


ignore this comment, I'll add a comment about the overall issue on the PR itself


Good catch(es) @frank-dittrich, it's so easy to miss these. We should try to make more use of shared code but there's so much to fix, it never ends.


Removing umfpack here explicitly assumes that trilinos is compiled with umfpack, if people are okay with that assumption it should be fine, but goma does need umfpack even if trilinos is not compiled with umfpack.


npm-debug.log hay que sacarlo del git, no deberia estar


Could you please try `0em + $line-height-base` maybe? Provided you're okay with the value being expressed in `em`.


Nooooo


I'd just call this publisher.


What about the StyleCI badge? :P


There's already a `publish` method on `Observable` so will that conflict with the class extension stuff?


1. I just pushed a heuristic to use the `dist-sandbox-HASH` directory if we find a `.cabal-sandbox` directory next to the cabal file. It's very shady though, and will break horribly if cabal changes the layout of sandboxes.
2. It turns out we can avoid the weird `dist-sandbox-HASH` nonsense entirely if everyone stops using `cabal install` to build LH and instead uses the [standard sandbox workflow](https://www.haskell.org/cabal/users-guide/installing-packages.html#developing-with-sandboxes):
```
cabal install --dependencies-only
cabal build
```


A helper/wrapper that takes care of the entire

"NativeMethods.SetForegroundWindow(Process.GetCurrentProcess().MainWindowHandle);"

item?


nickname would be *Dealloc* but no problem :+1:


Will this revision be pushed to Rubygems? I'm only seeing `0.9.3` at the moment


Thank you!


yea, let me do a pre-release now.  I've got a few other things I want to go out with 0.9.4, but I think we can do that real soon.  (maybe tomorrow)


Pour les bombes, il faudra prendre en compte le fait qu'on utilise des références en jeu, et que si une référence est mise à null, ça n'est pas le cas des autres références qui pointent vers cet objet qui devrait être détruit


Should return `TransformablePublisher`.


Ok, did a pre-release for 0.9.4.pre1.


Perhaps you could write a comment in the source code here that "the successfulness of both `worksWhenRemainderIsNegativeUsingMOD` and `wordsWhenDivisoerAndRemainderAreNegativeUsingMOD` prove that we're using truncated division in the mod operator (as opposed to floored or euclidean)."


Doesn't need the `throws`


Question: do we want to pass `dispatch()` prop to the decorated component?


hmm ... Here's what I get

```
[root@host-015 hadoop]# cat /etc/issue
CentOS release 6.6 (Final)
Kernel \r on an \m

[root@host-015 hadoop]# ls /sys/kernel/mm/
hugepages  ksm  transparent_hugepage
[root@host-015 hadoop]#
```

Maybe it would be better to simply list the directory and use whichever is actually present?


Ok thanks!!!


yep, commander!


Yeah I figured it would be a nice way to approach normal call dispatch nodes at a smaller scale.
This is a bit different than Get/LookupConstantNode in that the composition is simply done with two calls in each node, like the RubyCall example in Graal.


Perhaps it would be ebst to only append this character if there is at least 1 QueryParam


чувствую это не будет работать в IE, ¯\_(ツ)_/¯


Reference to a temporary, here. (delete the &)


I'd recommend that when you're making a second commit which just corrects mistakes in the first commit, that you use `git commit --amend` and then do a push -f. This effectively squashes the commits together.

For what's here now, I'd recommend that you squash the 2 commits together and push -f to the same branch (github PRs are branch based). Then just make a comment that you did it (github doesn't send out notifications for pushes)


может комментарии удалить?


и пустые правила


You can do this with a local tarball, I don't think there is anything github specific


This file doesn't seem to exist..


Hi Aruzhu. You have a lot of cool projects! You have published two different gmail-passwords in this commit. I would suggest changing your gmail password if you haven't done it already.


Glad I could help! Cheers!


я плох с юникодом. скажи, что это за символ?


The goal was you could use it with your own http server as well.  The auth code might need to be updated to make that possible.  But it seems more useful long term, since github-http is just http fetch + auth


Yeah, that suck in.


Responding to ajgarlag


Coach should be lowercase `belongs_to :coach`


I guess I should have been more specific. I was not aware that flags before this commit could only go after the last argument.

I suppose there are no less than four ways you can have flags:

1. Only after the last argument
2. Specific to a particular argument
3. Specific only to the binary itself (the first argument)
4. Arbitrarily located anywhere

From what I gather, before this commit it was 1. And now it's 1 and 3, with the _global_ flags being 3 and the command flags being 1.


Why is artist here!?


Thanks, I normally do but this commit is intentionally separated so @pfalcon can review it


Должно быть `dateHours.length`, раз в цикле `dateHours[i]`


Это уже лишнее, надо убить. А вообще это были _Glyphicons_.


Хорошая идея.


Восьмом?


+1 absolutely busted my chops.


<!DOCTYPE html>

	<link rel="stylesheet" type="text/css"
	href="style.css">

My App

image

In ac ipsum quis turpis adipiscing commodo. Mauris fermentum quam in risus eleifend fringilla. Aliquam placerat, ipsum sit amet cursus rhoncus, augue mi tincidunt erat, et consequat velit massa a velit. Cras eleifend lectus nec diam ornare, eu placerat mi ultricies. Etiam tempor iaculis feugiat. Ut tincidunt tempus libero non convallis. Morbi consequat eros eget elit porta porta. Curabitur placerat eros dui, vitae lobortis sapien laoreet ac. Maecenas a lacus diam. Morbi ante lectus, ornare sed suscipit at, tempus elementum purus. Suspendisse sodales hendrerit pharetra. Integer odio risus, convallis eu egestas ac, venenatis nec leo.


This is a test


fix #358


@rmg  https://github.com/strongloop/strong-mesh-models/blob/feature/model-updates/server/auth.js ?


  guixii <notifications@github.com> a écrit :

> Quentin j'ai un doute !
> Est-ce qu'il faut que je demande au bateau la position à chaque fois
> avant de demander ce qui m'entoure ou est-ce que les membres de
> l'equipage le savent avec un param dans le constructeur ? Parce que j'ai
> pas vu de messages implementé dans bateau pour avoir la position
>
>   —
> Reply to this email directly or view it on GitHub[1].
>

On peut mettre un paramètre dans le constructeur. Ca me parait bien.

Liens:
------
[1]
https://github.com/Shyzuna/IA04ShipProject/commit/c1dbc5e799ab266739e4ab55dc03348e1a48f684#commitcomment-11657525


это не важно


Exactly.


Pasting over Max's comments.

I don't think you need any of this though, it's a code smell to have to pass the logging level to a library, while the application can control that on its own by calling logging.getLogger(...).setLevel(...).

I also think you should give a more distinct logger name than signalfx.__BaseSignalFx. Just signalfx would be good, which mean you can actually define you logger at the module level:

logger = logging.getLogger(__name__)

class __BaseSignalFx(object):
    def some_method(self):
        logger.debug('hello')
I also don't think you should set the formatter. This is also for the application to control; don't bother with it.


Hey! You're conflicting


Wow. That's messed up. I wonder if there's some option for that enables the symlink. Anyway, I am filing an issue for this to get it fixed for you. At any rate, this doesn't enable THP, just disables compaction on the already-enabled THP.

https://issues.cask.co/browse/COOK-57


> You seem to have little faith in the Julians' ability to get a design right at first... considering the breakage that happened with the "Tupocalpyse", and the major breakage planned for "Arraymeggedon", maybe I can understand that attitude.

@ScottPJones, this comment is disrespectful and attacks @StefanKarpinski and his hard work directly. Stefan was sharing his thoughts on a design decision and you turned it into a personal questioning of his abilities and ideas. Not ok. You may disagree with another's opinion, but an appropriate response is focused on providing evidence or reasoning around the idea at hand, not making a personal, even if somewhat offhand or joking, comment.

On abstract vs. immutable, I've never really heard a good argument one way or the other, but there is somewhat of [precedent](https://github.com/JuliaLang/julia/blob/c73da0c7312cbbc1a90fa228d44e95eae6f31930/base/sort.jl#L247) for making them immutable. The corollary is with `Encoding` being a true abstract types, while `UTF8` is a "concrete" subtype, i.e. you wouldn't need/want to subtype `UTF8`.

On waiting to put `Encodings` into Base, I think the counter-argument goes back to Stefan's original comment here. `Encodings` will end up touching several corners/places in Base and string handling, so it's hard to know exactly how they should be structured until a larger integration is carried out. Sure, they seem to work fine for `check_string` now, but maybe when they're incorporated into some other code, we realize we actually need them to be structured a little differently (I've already been making tweaks and changes here and there as I've continued to piece things together). This also isn't quite as drastic as a "complete sweeping change" since the user-facing API will stay relatively the same, and it's mainly a major reworking of the internals. In these cases, I think it's much better to incorporate all the appropriate changes at once (which can be thoroughly tested by CI and by pulling changes locally) to make sure the new internals are solid, and then releasing, hopefully only needing deprecations.


You seem to be living on another planet.  Go read my comments - I have heaped loads of praise on both Julia, and the people who have come up with it, and have many many times thanked all of the time and energy that people have spent on *constructive* criticism.  I have NOT ONCE made a derogatory comment about another person here.
Trash talking the design decisions?  I don't think I've said anything that hasn't been said before, by any number of people, *including* yourself... (I dared criticize string interpolation... turns out Jeff is against it also... I criticized the choice of `*` for string concatenation... do I need to quote your very own words back at you???).  Many many people have complained about the fallout from the "tupocalypse"... maybe that could have been better handled?  I agree that the new approach for tuples is better, but that just points out that the *original* design had a flaw so severe that it was decided that a very painful *breaking* change was necessary.  Why do you feel commenting on that is so insulting?  Were all of the design decisions I've criticized your personal decisions?  Or do you feel threatened if I bring up obvious bugs in the code?


이미 84라인에서 FileNotFoundException을 catch 하고 있기 때문에 throws를 할 필요는 없습니다.


Cool.


Another reason why IDE support is so important... is if Mojang completely goes crazy with internal code changes... Like, adding/deleting classes/methods... IDE code-completion (ctr+spacebar on Eclipse/Netbeans) will help find replacement methods. Also, the IDE will be more helpful for finding an import location if it's changed.


@alexmarsev: With the checks back where they used to be.


Doesn't show when a mover is created... have to look deeper on this one


Looks good.


You can't inherit from `NotEnoughActionPointsEvent` and redefine TOPIC. Anyway, it should not be the case as we have to follow the topic -> name event class hierarchy. In fact, `action.move_reject` (event thrown when a character cannot move because he does not have enough AP), and `action.cure_reject` (the same, but for cure (not yet implemented)) have in commmon the fact that they are both linked to the lack of AP, but sementically, we have to focus on the event hierarchy than on the functional meaning of the events.


@quinnj I didn't think that comment was an attack on Stefan... he was the one that seemed disrespectful of *your* work, i.e.:
>I would bet cold hard cash on us not getting it 100% right and causing future breakage for people who used them.


Sorry, totally missed that one.


TeamCity Azmq :: Build (Linux) [Build 122](https://78.46.71.115/viewLog.html?buildId=566&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 122](https://78.46.71.115/viewLog.html?buildId=566&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:06


Just test.


しゃれ乙なコードだなぁ、ω、
日本語対応と演出いれてるのね、ω、
今度あった時解説おなしゃす、ω、


Not sure why this is diff. Should it not be enabled from before?


"Wenn man sich überlegt, dass Männer wie Harond mit ihren Verbrechen davonkommen..."

Vor den Auslassungspunkten fehlt ein Leerzeichen, da da nicht ein Wort abgebrochen wird. Bekommst du sowas auch automatisch behoben?


These are two different plugins. Confusing, I know.


Use File licenceFile instead of String


Can you rename this method to showLicenceString because Python won't be able to make the difference between a file and a string


Please throw ScriptFailureException instead


Can you make this variable Final?


True but was this never enabled and exported into the feature before? Should it not be enabled?


Same for this one


The script give only the path no ? So it's a string


You tell me to use these, but I can change it ;)


Eventually we'll probably delete OEQINQ altogether, but I put this in the proc here so that this program would run instead whenever a menu item calls OEQINQ.


Rather than deleting this, could you comment it out this call and add some clarifying text with a FIXME above it? If the code's not there, we'll forget to add it back later.


@quinnj Putting Encodings into Base would not mean that they are cast in stone, right from the beginning.
What changes (except for removing Binary, which I think should go back, and moving it out of a module, for convenience) have you had to make to Encodings?  I hadn't yet seen any.


Hello there,

I think the wording is not very clear, let's try this one:

Security Fix (High Risk) - Upgrade as soon as possible - Switch to JSON to avoid unserializing untrusted data when handling responses from PayPal.

It's very important to tell people that this is a security fix with high risk, please, do it :)
Have a nice day.


This looks good.


Good idea. Done.


A file path should always be represented as a File() object. Python will do the transformation for you


SFE is a kind of CE


Not sure what you don't like here are exactly, but I'd be happy to use better and not risky fix! :) Can't think of any today as well :)

What I personally don't like here is that I use ```defaultPanel``` directly since we don't have ```Navigation.isDefaultPanel(panel)```


yhghggh


Wouldn't this change be functionally equivalent to original?


good riddance


Deliberate policy to parallel usage for iterators.


A new release would be very helpful!


@senderista @jortiz16 Merge artifact?


well, probably this link should rather go into the commit description instead of the comments


The "mediaoembed" is not yet ready for testing, so it should not be enabled.


Eval.hpp deleted ?


maybe also remove randomMT() but otherwise +1


> I've been hoping for Taps to not need to be git repos for a long time now.

It's actually been the case for a long time and just not advertised.


We must talk about that... See you on IRC tomorrow


Eval.h deleted, there is still a Eval.hpp

Le sam. 13 juin 2015 à 00:55, Navid <notifications@github.com> a écrit :

> Eval.hpp deleted ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RomainGoussault/Deepov/commit/1600524d70ff3313cdbb82299747f8fdb0edc2ea#commitcomment-11663214>
> .
>


We could add a constructor for the capture/special moves


noch einen gesehen


Функции `makeDateColumn()` и `makeArrHours()` больше нигде не используются. Создавать функцию имеет смысл, если она будет (или может) вызываться в разных местах. Но похоже, это не тот случай. Поэтому можно просто весь код функций было бы оставить тут, а для сохранения читаемости достаточно добавить однострочный комментарий, например:
```js
// Формируем массив дат
var dateArr = [];
for (var i = 1; i < headerInfo.length; i++) {
    var dateColumn = headerInfo[i].innerHTML.split('.');
    dateColumn[0] = parseInt(dateColumn[0]);
    dateColumn[1] = parseInt(dateColumn[1]);
    dateArr.push(dateColumn);
}

// Формируем массив с временем начала и окончания события
var dateHours = [];
for (var i = 0; i < tableHours.length; i++) {
    var first_time = tableHours[i].children[0].innerHTML.split(':');
    var second_time = tableHours[i].children[1].innerHTML.split(':');
    dateHours.push([first_time, second_time]);
}
```


Аналогично нужно проверить, есть ли смысл в других функциях.


nice


:tiger: :raised_hands: :frog:


This is weird... are you using an outdated version of composer?
Or why are that lines removed again?

If I'm running
```
composer update
./build.sh
```
locally, exactly those lines are added back.


[![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)


[![Iris Awakening Hypnos (Ending Night)](http://api.artsmia.org/images/120122/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120122)


[![Aurora (Morning)](http://api.artsmia.org/images/120123/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120123)


[![Apollo at the Zenith (Noon)](http://api.artsmia.org/images/120124/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120124)


[![Apollo Descending (Evening)](http://api.artsmia.org/images/120125/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120125)


[![Endymion and Diana (Early Night)](http://api.artsmia.org/images/120126/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120126)


[![Sacrifice to Diana (Midnight)](http://api.artsmia.org/images/120127/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120127)


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


[!["Elephant and Three Clowns on Tub" mechanical bank](http://api.artsmia.org/images/80418/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80418)


[!["Chief Big Moon" mechanical bank](http://api.artsmia.org/images/80420/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80420)


[!["Bismark [sic] Bank" mechanical bank](http://api.artsmia.org/images/80441/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80441)


[!["Hoop-La Bank" mechanical bank](http://api.artsmia.org/images/80763/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80763)


[!["Trick Dog, Six-Part Base" mechanical bank](http://api.artsmia.org/images/80765/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80765)


[!["Hen and Chick" mechanical bank](http://api.artsmia.org/images/80916/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80916)


[!["Acrobat" mechanical bank](http://api.artsmia.org/images/81089/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81089)


[!["Uncle Sam" mechanical bank](http://api.artsmia.org/images/81095/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81095)


[!["Eagle and Eaglets" or "American Eagle" mechanical bank](http://api.artsmia.org/images/81102/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81102)


[!["Creedmoor" mechanical bank](http://api.artsmia.org/images/81104/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81104)


[!["Monkey with Tray" or "Monkey, Automatic" mechanical bank](http://api.artsmia.org/images/81107/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81107)


[!["Lion and Two Monkeys" mechanical bank](http://api.artsmia.org/images/81111/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81111)


[!["Teddy and the Bear" mechanical bank](http://api.artsmia.org/images/81115/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81115)


[!["Professor Pug Frog's Great Bicycle Feat" mechanical bank](http://api.artsmia.org/images/81134/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81134)


[!["Girl Skipping Rope" mechanical bank](http://api.artsmia.org/images/81138/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81138)


[!["Clown, Harlequin and Columbine" mechanical bank](http://api.artsmia.org/images/81144/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81144)


[!["Bull Dog Bank" mechanical bank](http://api.artsmia.org/images/81328/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81328)


[!["Flatiron Building Bank, Large" still bank](http://api.artsmia.org/images/81400/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81400)


[!["Springing Cat" mechanical bank](http://api.artsmia.org/images/81604/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81604)


[!["Commonwealth Three-Coin Registering and Savings Bank" still bank](http://api.artsmia.org/images/81626/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81626)


[!["Owl, Turns Head" mechanical bank](http://api.artsmia.org/images/81634/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81634)


[!["U.S. Tank" still bank](http://api.artsmia.org/images/81743/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81743)


[!["Fidelity Trust Vault with Little Lord Fauntleroy" still bank](http://api.artsmia.org/images/81757/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81757)


[!["Frog Bank" (or "Two Frogs") mechanical bank](http://api.artsmia.org/images/81863/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81863)


[!["Frog on Round Base"](http://api.artsmia.org/images/81866/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81866)


[!["Tammany Bank" or "Little Fat Man" mechanical bank](http://api.artsmia.org/images/81890/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81890)


[!["Humpty Dumpty" mechanical bank](http://api.artsmia.org/images/81960/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81960)


[!["Independence Hall Tower" semi-mechanical bank](http://api.artsmia.org/images/81983/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81983)


[!["Lindy Bank" still bank](http://api.artsmia.org/images/82108/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=82108)


[!["Sputnik Bank" still bank](http://api.artsmia.org/images/82718/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=82718)


[!["Columbia" still bank](http://api.artsmia.org/images/83684/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83684)


[!["Tower Bank" safe still bank](http://api.artsmia.org/images/83685/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83685)


[!["Hall's Excelsior" or "Cashier Bank" mechanical bank](http://api.artsmia.org/images/83706/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83706)


[!["Bank of America Cash Drawer (registering)" still bank](http://api.artsmia.org/images/90098/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90098)


[!["Japanese Safe" still bank](http://api.artsmia.org/images/90129/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90129)


[!["Security Safe Deposit" still bank](http://api.artsmia.org/images/90193/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90193)


[![12-room dollhouse with dolls and furniture](http://api.artsmia.org/images/118536/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118536)


[![Bleached Bone Reef](http://api.artsmia.org/images/120152/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120152)


[![Head of the Buddha](http://api.artsmia.org/images/560/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=560)


[![Bodhisattva, one of a pair](http://api.artsmia.org/images/646/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=646)


[![Presentation Hanging for an Eightieth Birthday](http://api.artsmia.org/images/661/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=661)


[![Maitreya (The Buddha of the Future)](http://api.artsmia.org/images/724/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=724)


[![The Bodhisattva Guanyin and Flying Apsaras](http://api.artsmia.org/images/807/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=807)


[![Udhyana Buddha](http://api.artsmia.org/images/1373/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1373)


[![Figure of a Monk](http://api.artsmia.org/images/1473/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1473)


[![Eighteen Arhats (åå«ç¾æ¼¢åå·)](http://api.artsmia.org/images/4565/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4565)


[![Chakrasamvara Embracing Vajravarahi](http://api.artsmia.org/images/5240/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5240)


[![The Bodhisattva Kuan-yin](http://api.artsmia.org/images/5424/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5424)


[![The Bodhisattva Guanyin](http://api.artsmia.org/images/5788/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5788)


[![The Bodhisattva Avalokitesvara](http://api.artsmia.org/images/5808/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5808)


[![Exalted Gathering in the Green Woods](http://api.artsmia.org/images/9332/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9332)


[![Ritual Buddhist Bottle Vase and Cover](http://api.artsmia.org/images/9719/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9719)


[![Ritual Buddhist Basin](http://api.artsmia.org/images/9720/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9720)


[![Bodhisattva, one of a pair](http://api.artsmia.org/images/11983/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=11983)


[![Standing Buddha](http://api.artsmia.org/images/12948/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=12948)


[![Head of a Crowned Bodhisattva](http://api.artsmia.org/images/13718/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=13718)


[![Flying Apsaras](http://api.artsmia.org/images/17458/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=17458)


[![Flying Apsaras](http://api.artsmia.org/images/17462/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=17462)


[![Figure of Bodhisattvas, one of a pair](http://api.artsmia.org/images/29581/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=29581)


[![Figure of Bodhisattvas, one of a pair](http://api.artsmia.org/images/29582/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=29582)


[![The Monk Ananda](http://api.artsmia.org/images/30633/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30633)


[![Imperial Candlestick](http://api.artsmia.org/images/31166/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31166)


[![Imperial Candlestick](http://api.artsmia.org/images/31167/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31167)


[![Benediction Scene](http://api.artsmia.org/images/31223/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31223)


[![Farewell to Hua Yun](http://api.artsmia.org/images/31225/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31225)


[![Buddhist Reliquary](http://api.artsmia.org/images/31226/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31226)


[![Seated Guanyin](http://api.artsmia.org/images/32122/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=32122)


[![Stone Rubbing of the Five Hundred Arhats (Wu-pai Lo-han ying ch'en tu)](http://api.artsmia.org/images/33630/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=33630)


[![The Eighteen Lohans](http://api.artsmia.org/images/45524/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=45524)


[![Stele of Maitreya Buddha](http://api.artsmia.org/images/61570/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=61570)


[![Buddhist Ritual Water Bottle (Kundika)](http://api.artsmia.org/images/61724/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=61724)


[![The Pleasure of Fishes](http://api.artsmia.org/images/62754/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62754)


[![Spherical Censer](http://api.artsmia.org/images/63346/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=63346)


[![Buddhist trinity of Vairocana, Maitreya and Samantabhadra](http://api.artsmia.org/images/83514/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83514)


[![Covered Incense Burner](http://api.artsmia.org/images/90694/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90694)


[![Buddhist Long-handled Censer](http://api.artsmia.org/images/92368/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=92368)


[![Votive stele](http://api.artsmia.org/images/99734/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=99734)


[![Dvarapala (Guardian Figure)](http://api.artsmia.org/images/100842/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=100842)


[![Leaves from the Album "Three Friends of Winter"](http://api.artsmia.org/images/106533/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=106533)


[![Leaf from album "Three Friends of Winter"](http://api.artsmia.org/images/106534/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=106534)


[![Leaf from album "Three Friends of Winter"](http://api.artsmia.org/images/106535/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=106535)


[![Leaf from album "Three Friends of Winter"](http://api.artsmia.org/images/106536/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=106536)


[![Leaf from album "Three Friends of Winter"](http://api.artsmia.org/images/106537/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=106537)


[![Album of miniature landscape paintings, from a double album set](http://api.artsmia.org/images/108703/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=108703)


[![Album leaf](http://api.artsmia.org/images/116140/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116140)


[![Album leaf](http://api.artsmia.org/images/116141/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116141)


[![Album leaf](http://api.artsmia.org/images/116142/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116142)


[![Album leaf](http://api.artsmia.org/images/116143/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116143)


[![Vase with the Eight Taoist Immortals](http://api.artsmia.org/images/157/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=157)


[![Ink tablet in the form of a turtle](http://api.artsmia.org/images/513/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=513)


[![Fan Painting of a Taoist Magician](http://api.artsmia.org/images/5048/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5048)


[![Kâuei-hsing (God of Literature)](http://api.artsmia.org/images/5798/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5798)


[![Official Taoist Seal](http://api.artsmia.org/images/11862/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=11862)


[![Generals Who Died for their Country and Officials of Former Times](http://api.artsmia.org/images/31143/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31143)


[![Recessed-leg Altar Table](http://api.artsmia.org/images/31168/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31168)


[![Nymph of the Luo River](http://api.artsmia.org/images/33126/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=33126)


[![Money Tree](http://api.artsmia.org/images/46241/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=46241)


[![Immortals Beneath the Pine Tree](http://api.artsmia.org/images/46877/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=46877)


[![Two-piece Candlestick, one of a pair](http://api.artsmia.org/images/64422/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=64422)


[![Two-piece Candlestick, one of a pair](http://api.artsmia.org/images/64423/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=64423)


[![Taoist Stele of Five Deities](http://api.artsmia.org/images/64878/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=64878)


[![Celestial Worthy of Primordial Beginning](http://api.artsmia.org/images/68565/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=68565)


[![Chen-Wu, The Taoist Deity of the North](http://api.artsmia.org/images/82959/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=82959)


[![Numinous Treasure Emissaries](http://api.artsmia.org/images/91031/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=91031)


[![The Exalted Emperors and The Gods of the Five Directions](http://api.artsmia.org/images/91032/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=91032)


[![Image of a Celestial Worthy](http://api.artsmia.org/images/99452/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=99452)


[![Portrait of Wang Shimin](http://api.artsmia.org/images/5122/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5122)


[![Scholars Playing "Go" Under Pine Trees](http://api.artsmia.org/images/5197/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5197)


[![Portrait of Confucius](http://api.artsmia.org/images/5790/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5790)


[![Guanyin on Mt. Potalaka with Sudhana](http://api.artsmia.org/images/5819/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=5819)


[![Album Leaf of a Scholar in Landscape](http://api.artsmia.org/images/30356/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30356)


[![Scholar in his Creek Side Retreat](http://api.artsmia.org/images/30981/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30981)


[![The Immortal Hanshan](http://api.artsmia.org/images/31144/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31144)


[![Two versions of Bodhisattva Guanyin](http://api.artsmia.org/images/31224/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31224)


[![The Gathering at Orchid Pavilion](http://api.artsmia.org/images/60018/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60018)


[![Lady at Dressing Table](http://api.artsmia.org/images/60019/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=60019)


[![The Three Purities, one of three](http://api.artsmia.org/images/62138/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62138)


[![The Three Purities, one of three](http://api.artsmia.org/images/62139/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62139)


[![The Three Purities, one of three](http://api.artsmia.org/images/62140/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=62140)


[![The Buddhist Guardian Mahabala](http://api.artsmia.org/images/91030/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=91030)


[![Beauty in Winter](http://api.artsmia.org/images/91473/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=91473)


[![The Singer Su Xiaoxiao](http://api.artsmia.org/images/99549/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=99549)


[![Crippled Zhang Meets Hunchback Li](http://api.artsmia.org/images/101493/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=101493)


[![The Seven Sages of the Bamboo Grove](http://api.artsmia.org/images/105241/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=105241)


[![The Gathering at the Orchid Pavilion](http://api.artsmia.org/images/109255/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=109255)


[![Court Ladies at Play](http://api.artsmia.org/images/114428/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114428)


[![Covered Box](http://api.artsmia.org/images/186/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=186)


[![Ovoid Bottle](http://api.artsmia.org/images/205/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=205)


[![Tiger Pillow](http://api.artsmia.org/images/210/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=210)


[![Covered Bowl](http://api.artsmia.org/images/212/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=212)


[![Beehive-shaped Vase](http://api.artsmia.org/images/213/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=213)


[![Bottle Vase](http://api.artsmia.org/images/216/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=216)


[![Tea Bowl](http://api.artsmia.org/images/462/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=462)


[![Storage Jar](http://api.artsmia.org/images/489/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=489)


[![Li tripod](http://api.artsmia.org/images/511/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=511)


[![Lien (cosmetic box)](http://api.artsmia.org/images/512/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=512)


[![Beaker-shaped Vessel](http://api.artsmia.org/images/526/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=526)


[![Beaker (tsun)](http://api.artsmia.org/images/527/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=527)


[![Lidded Jar (yu)](http://api.artsmia.org/images/528/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=528)


[![Ewer with Pheasant-Head Stopper](http://api.artsmia.org/images/804/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=804)


[![Shallow Bowl with Double Fish Motif](http://api.artsmia.org/images/666/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=666)


[![Hu (Ritual Wine Jar)](http://api.artsmia.org/images/1985/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1985)


[![Tea Bowl, Honan ware](http://api.artsmia.org/images/2369/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2369)


[![Bottle Vase (chui-hu)](http://api.artsmia.org/images/2383/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2383)


[![Incense Tripod](http://api.artsmia.org/images/2732/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2732)


[![Double Gourd-shaped Ewer](http://api.artsmia.org/images/3237/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3237)


[![Tea Bowl](http://api.artsmia.org/images/3239/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3239)


[![Funerary Storage Jar](http://api.artsmia.org/images/3512/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3512)


[![Globular Jar](http://api.artsmia.org/images/3634/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3634)


[![Ovoid Bottle](http://api.artsmia.org/images/3592/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3592)


[![Mei-ping Vase](http://api.artsmia.org/images/3635/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3635)


[![Mei-ping Vase with Cover](http://api.artsmia.org/images/3711/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=3711)


You need to check that name and ns have the same name, if ns is not nil


It looks like `&` is missing for multiple query parameters


Ähm? English? Wus? Ist das eins der Dinger die keinem auffällt weil die Ingame Äußererungen zu selten sind oder gar nicht funktionieren? Oder weil keiner die „Teufelstimme“ wählt?


dfghjtliiiiiiiiiiiiiiiii


teste


@markstory I have two confirms in line:92.

# Confirm
(1) If $types has same keys with $this->_defaults, $this->_defaults will not be overwritten by $types.
Because left operand has a priority than right operand in "array + array".
If $types can overwrite same keys, must be as follow.
```
$this->_defaults = $types + $this->_defaults;
```

(2) When called addDefaults(), $this->_defaults is possible to be null.
Because it is not initialized in line:31.
Next code makes error, therefore line:92 has possibility to throw error.
```
$a = null + ['id' => 'integer'];
$a = ['id' => 'integer'] + null;

Error: Unsupported operand types
```
It is required to cast as array variables for avoiding error.
If can not happen that $this->_defaults and $types are null, it is not required to cast.

# Suggestion
If (1) (2) is right, should be as follow
```
    public function addDefaults(array $types)
    {
---     $this->_defaults = $this->_defaults + $types;
+++     $this->_defaults = (array)$types + (array)$this->_defaults;
    }
```

If use to initialize on line:31, to cast array for $this->_defaults is not required in my code.
However it makes very wide side effect.
```
--- protected $_defaults;
+++ protected $_defaults = [];
```


See https://github.com/heiseonline/shariff-backend-php/pull/33


это уже не пишем


Этот ассерт не нужен


kjlllpñopoò


Yes, I noticed that earlier. Already fixed it.


replaced self.request.user by request.user, has nothing to do in this commit


lhuyuteduitgyuklbjh


@nmonterroso, @ArturKlajnerok I added the `Respose` after trying to test the users controller. I noticed that the controller method I was writing was coupled with `osin` so I wanted to unwind that. Please take a look and let me know if you have any issue.


`hola bbmkbbgnnn


holaçhuaraca


@Wackerbarth The only problem is that some languages might want to put "Move" before "X" so it's better if they are just left unified for the time being.


KJHIUIYIYIYYI


Compiled fine for Moto E. I'll test asap.


@bradenmacdonald I've tried this, and it causes my test to fail. This is probably because the test scenario is slightly contrived-- but the reason why the test ends up working is that it gets the block type information canonically, whereas this code snippet you are writing here forces the use of a few magic constants, 'chapter', 'sequential', and 'vertical'. If the blocks, for some reason, don't conform to that, this will fail. I don't know if this is realistic, but I do know they're planning to turn verticals into XBlocks at some point, so I don't really know what that's going to look like.

Thoughts?


Updated to @RouteConfig per your suggestion (thanks again). Unfortunately, it appears that none of the routes are getting loaded though. No errors, router-outlet gets added to the DOM but a route is never loaded. I tried using router.navigate('/') but when I dig into the router registry I'm seeing any routes included. Any ideas? This router is kicking my butt right now and I need to move on to some "real" parts of this little demo app. :-)


Назовите тестовые классы со смыслом. Это ContactTests


Interesting choice to check this in.


методы пишем с маленькой буквы


Hier müsste dann doch halbautomatisch ein „…“ und kein –


also please make the c++ SP on par.


* FIXME: Need to include the server IDs?


Use a `switch` statement


Убираем


Ок


@Kelketek  They've [already turned vertical into an XBlock](https://github.com/edx/edx-platform/blob/master/common/lib/xmodule/xmodule/vertical_block.py#L23). What kind of test is failing? As far as I know, in edX platform a chapter is always an XBlock/XModule (they are treated the same) of type 'chapter', a section is always of type 'sequential', and a unit is always of type 'vertical'. The Studio enforces that structure. And since the code here is supposedly returning the IDs of a chapter, a section, and a vertical, then they should be of those types, right? I'm curious how the test is setup if this is failing.


Should we change timeTools.timestampNow() to be more useful, instead of accessing epoch_diff directly?


Na toll. Alle werden substantiviert, nur der Plünderer darf nicht. Der muss aktiv handeln ;(


This is also a `switch` statement in disguise.  WIthin a radio group, only one radio button can be checked, so I think you can use a CSS/jquery style selector to select the element that is checked and then `switch` on it's id.


По-хорошему надо импортировать с `.`. А как ты запускаешь тул после установки?


@who pushed the fix.


I think you misunderstood.  constants.small.py was a temporary solution
which I have since removed.

Perhaps I wasn't clear.  The problem was that you could not change certain
parameters because of a bug in great3sims.  But I have since found a
workaround and removed constants.small.py.  The workaround is demonstrated
on the current checkin.

constants.small.py was part of a previous checkin.

On Fri, Jun 12, 2015 at 2:20 PM, SimonKrughoff <notifications@github.com>
wrote:

> This code is an exact duplicate of code in control.py. I would much prefer
> to keep the constant overrides in one place. That provides an obvious place
> for documentation and there is no chance of the two getting out of sync
> with each other. If you moved constants.py.small to
> constants_test_override.py you could do something like the following:
>
> import great3sims
> import constants_test_override as constants
>
> great3sims.constants = constants
>
> I think something like that should work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lsst-dm/great3-public/commit/757724952cacd4da88d68eb52fd2a0ef103cad5f#commitcomment-11662165>
> .
>


This is another thread on the same commit


public class ProyectoFinal{
public static void (String [] args){
System.out.println("Wilber Pariona Huaraca");
    }
}


Muito bom!


Der Erste ist dir entwischt.


System.out.println("Esto sale en pantalla\n");
System.out.println("caracteres");
String valor = "Este es el mensaje: ";
System.out.println(valor+ " Hola");


Might consider pushing Opal .8rc1 into this. Any fiddling with that yet?


with it also comes a brand new Docker tooling.


We wanted to have a way to easily start/stop and deploy Docker containers directly from Eclipse.


three major platforms:


Hi Már,

thanks for the input. I'll have a look at it on monday!

Have a nice weekend
/Jonas

> On 12.06.2015, at 15:39, Már Örlygsson <notifications@github.com> wrote:
>
> Cherry-pick changed missing-key output from @musterknabe's repo
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/maranomynet/translate.js/commit/6d301827003eb6c8b78cd2b14aeccb3d3d63e0ab


Removing the "linker emulating GNU ld" bit causes a regression in autoconf-based projects compiling on Linux: libtool's configure script add-in no longer recognizes emcc as being GNU ld, and because it doesn't know anything specific about emscripten it disables shared libraries.


Care to elaborate?


* TODO: Define a better validator for the repository, should ensure it is a valid URL and has .git at the end.


* TODO: Check for input, make sure it is a valid gitlab hook, check repo and branch are correct


* TODO: Use a decorator pattern here


* TODO: Don't allow this to run if there is already a pending deploy or no servers


$server->first = (count($output) === 0); // FIXME: Let backbone.js take care of this


* TODO: rewrite this as it is doing way too much and is very messy now.


// FIXME: Handle this situation as it is then unclear what went wrong


* TODO: refactor these 2 functions


* TODO: Should this be an attribute?


* TODO: Translate


'cipher' => MCRYPT_RIJNDAEL_128, // FIXME: See if we can update this to the recommended 'AES-256-CBC'


Με το req.connection.remoteAddress παίρνουμε την IP του χρήστη και όχι του proxy server σωστά;


It's super useful though


Really want to do something like:

```
<input id="csvSelection" type="radio" value="1">CSV with spaces<br/>
<input id="csvSelection" type="Radio" value="2">CSV without spaces<br/>
```

and then in the JS code `$("#csvSelection").val()` to get the value 1, 2, ...


`Debugger.Launch()`?


In other words: ugh, whatever, I don't know how to use Git anyway.


but also to provide a better overview and easier/faster access to common operations, from a visual perspective.


Eu sei o nestor disse que dá para o 20.


With Eclipse Mars released, you can get it


holCSVnmño{´


latest greatest builds, you can use the `Linux Tools` project nightly builds update site at


Então vão ser dois 20 na pauta


hkdfcvgbjtnh


To use the plugins, it is assumed


Docker's Installation guide


[the?] Docker tooling, you will


sfshdffdliuserfiuwernwemfc


fgvsdfvsvxcvxcvddfgfrgvdfgg


yfcguhmn  nm nm nm


instances, with image


dgbsnhmnjdtmtmt


for the selected Docker instance/process


Docker instance/process


qknsdv ndq


In the screen above, the


dwfegrhkyuliñoñ{´+
}


yufktdujt


Docker daemon


Oh, sorry, missed a `.` in `../auth`, so I was looking in the wrong spot.


try to detect


in Linux, it will


I think I misunderstand why we use concat in the first place (I think we use it in stores)


if on Windows or OSX, it will


Quem te viu o trabalho foi o rui couto, não foi?


detectable, you can


@Shwinn Awesome!


Created #1960


image, you use


Here, I'm simply


System.out.println("eso es todo men ");


Os eventos era dificel de fazer?


ok

On Fri, Jun 12, 2015 at 12:51 PM, Ricky C. <notifications@github.com> wrote:

> You probably need to update the views too!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CoderDojoNYC/nano-music-station/commit/31e9c77abee36f6a47da4e5180888607f8cd5ec9#commitcomment-11657931>
> .
>


the image, the easiest


if(!(O.status & ORGAN_ROBOT)) doesn't this mean person needs to have an mechanical heart to get spider eggs inside him?


is to right-click on the image in the Docker Explorer


Ok!

On Fri, Jun 12, 2015 at 7:29 PM, Ricky C. <notifications@github.com> wrote:

> [image: :ok_hand:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CoderDojoNYC/nano-music-station/commit/31e9c77abee36f6a47da4e5180888607f8cd5ec9#commitcomment-11663835>
> .
>


`namespace` === `internal module` there is no new functionality introduced. this is just to avoid confusion with ES6 modules, which are just referred to as `modules` hence forth.


Here, I've initially


Foi sim, senhor RmsCouto.
Não, mas era preciso ter imaginação (e tempo) para fazer uma simulação que seja relativamente coerente com os dados do sistema.


right-click


In this example, I'm only


By default, we


click `Finish`, the container


Can you please give me a copy of all debugged programs?


public class ProyectoFinal{
public static void main (String [ ] args){
System.out.println("jhumary soto lozano");
}
}


In here, the port 8080 (the web server) is mapped to port 32768


public class proyectofinal{
pubic static void main(String [ ] args){
System.out.println("yalina quinteros");
}
}


a minute ago I was about to fix and create a pull request on this after I got a bunch of errors in the console :)


Eu estive a ver o vosso trabalho agora, fogo vocês pensaram em tudo.  A vossa cena da meteorologia esta... meu Deus.


Thus, to show this, I just need to go to


Once built, the image will show up and it will be available to


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-MacOS-10.9-clang: OK (1665 of 1668 tests passed)

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


Ah, you beat me too it!! I was playing around with a new syntax and forgot to revert it and then pushed it.... gotta love tests (if i had them!).

This commit should fix it as well:
https://github.com/AdamBrodzinski/react-ive-meteor/commit/8982d7f5e5fada1ab2520672ff47d31b1a3de29a

Thanks!!


including getting a tree view


For Eclipse Mars,


this should only be checked if openssl is loaded


oops; wrong commit message for this one. Just removing an unnecessarily negative mention of a feature that is not available in Uchiwa.


For Eclipse Mars SR1, we


edges fixed, like


Nice! You can map from http://ipinfo.io country codes to names using the JSON country code to name mapping file at http://country.io/data. Would also be great it you updated the README to mention ipinfo


Work also started in Eclipse CDT to support using Docker images to build binaries for an OS other than the one you are running on.


It's the future :)


or Mac, but target native deployment on various Linux architectures.


Furthermore, in http://tools.jboss.org[JBoss Tools], we are working on better integrating Docker with Eclipse server adapters, to ease deployment of your web applications to a Docker container.


public class ProyectoFinal{
pùblic static void main(String [] args){
System.out.println("Aydee pacco");
}
}


I would rather separate cases module from tree transformers/ast modules. Even to the point of having 3 separate modules (even if I usually hate such splitting). But then it's mostly because of more granular dependencies. Ast module require Parboiled and compile/runtime support classes (`Extractions`), making those dependencies optional actually increases complexity and number of dependencies for the user. Additional benefit is that we could release it sooner, without much advertisement until it will be ready for larger audience (if ever).


You can see how server deployment works with the current Docker tooling, by leveraging  http://tools.jboss.org/blog/2015-03-02-getting-started-with-docker-and-wildfly.html[docker volumes] and http://tools.jboss.org/blog/2015-03-03-docker-and-wildfly-2.html[remote deployments] support.


Não gostei muito da expressão "setagens", seria melhor "ajustes" ou "configurações".


bugs, please


This appears to be a blank commit...?


can you double check that this one was merged correctly. It has a random comment here.


Hehehehe


HOLAA


public class proyectofinal{
pubic static void main(String [ ] args){
System.out.println("aydee  pacco");
}
}


As it turns out, there were indeed some name bugs, but there was also an error in the definitions in this module. I'll push the fixes to the name bugs as soon as I have got them cleaned up.

The definiens of `Fiber` has a free variable `B` which was not introduced in the definiendum; unlike Agda, only variables which are computationally irrelevant may be omitted. (In Agda, relevant variables may be left out and elaborated in via unification.)


This fixed my problem with Polymer Starter Kit running on local machine. I keep getting an error when trying to gulp


Yes, something failed when merging the commit. I am looking into it. Sorry about that.


93469234-DemandResponse (vidanovic) - 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-69.4%25-red.svg)


93469234-DemandResponse (vidanovic) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Nice. Thank you.


looks good to me.


Love it


the psm changes look fine to me.


93469234-DemandResponse (vidanovic) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Can you open a Jira for that?


why not just use reject instead of reject!?


This looks useful. We've been using spec flow a fair amount (maybe 100 specs) and I started out using tables a lot and comparing things really meticulously. But then looking back at older specs it is more difficult to determine what the heck the data in the table even represents. So a sort of pattern that I've been trying is something like:

Given I have an order with these specs
SpecName
---------------------
IsLateAdd
BookedInPeakSeason

And then in the step I would create an object to map those features to and then have a custom object builder that would create the domain object to your specifications. It just made things a lot easier to deal with than being like "clearly this was a late add booking because notice the two dates in these random tables and how one is close to the other one..."

So all that said, I have been trying to distance myself from tables for explicit object creation. However, there are certainly enough times that I still want to just make simple objects and this feature seems like it would take one of the annoyances out of specflow


Just checked out a book from the public library titled Sams Teach Yourself SQL ... working


I don't think that's equivalent... that array expression should translate to (e.g.) :
```lua
e1[e2]
```
In your change it is :
```lua
e1;
e2;
```

It needs to be given as array access logic, because there may be special getter/ setter behavior defined for that (in fact, the way that I track the length variable depends on a special setter for the table).


Thank you so much for your considerable and instance response.It is really a great encouragement for me to go on with project alike.


Got it, others part is LGTM.


Fix #1


The commit message a bit confusing, maybe could consider: After submitting  the write transaction of promoted object to tier pool during promoting, osd would kick blocked read ops in flight. But those requeue reads could not read objects until the promoted object is persisted in tier pool. The latency maybe not less than do_proxy_read() in some case.

Others looks good to me :+1:


Acked-by: @spcui


@burkhartt Exactly Tim, what you said here:

>  But then looking back at older specs it is more difficult to determine what the heck the data in the table even represents.

A long time ago, I remember using a 10-property table to represent the state of an object.  Then a test came along later where just one of those ten properties would change, but the meaning was entirely lost.

Using a table to represent more generic details of an order with a table sounds like a great idea, I had never thought of that.  Hmm....


this commit means we _do_ keep the newline, correct?


Shouldn't you use some env variable which points to DSE bin directory?


wait, i dont understand why this is here


the more we go this route, the more i think we need something that's universal + self-describing asap. let me give this some thought before we merge this PR.


I also added a note here that TCP has to be open, as golang seems to do that more often.


https://github.com/Theo1016/ProjectSDK.git

常用工具类 整合中,长期作业

Application 抽象类 已整合

Activity 基类 已整合

异常捕获  已整合

日志   已整合

图片缓存 已整合

内存管理 已整合

数据缓存 已整合

数据请求前后封装 已整合

网络请求 已整合

json解析

下拉刷新

横向滑动

侧边栏

常用动画

常用适配器

数据统计

APP更新

百度地图

广播(数据同步)

下载管理

URL管理 (APPSearchURL,BillboardContentRequestor)

APP通信


I can't remember why I did it like this, but I did have a reason at the time. I don't think it matters at this point though.


> Disable this ***is*** you experience crashes


84691154_AutomateBuildingOfDocs (lefticus) - Win64-Windows-7-VisualStudio-12: OK (1674 of 1674 tests passed)

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


this one is wrong actually. Angular dependency injection is used by setting the dependency in quote ('$scope') but then you can use whatever variable name you want instead of ``scope`` you could use ``foo``, it would still work
```Javascript
app.controller('myController', ['$scope',function(foo){
   // foo is the scope
}]
```

I don't know why it does not work for you


Exactly bc you know what it means from a high level and don't have to memorize all of the business rules that define what it means specifically


nên để hàm index lên trên cùng


nit, squash into one line


nit, declare tmp_dir as close as first time use


其實、check element 那邊... 可以少掉那個 else....
像這樣... ㄜ 、我覺得比較乾淨~ (這單純是個人的毛病...|||)
```
if (!peekElm) {
   return false;
}
// listener class
```


Our company - Xiamen Finehope Polyurethane Co., Ltd devoted in promoting widely usage of polyurethane since 2002. During the past 12 years, we had worked with many prestigious companies. http://www.polyurethanesupplierschina.com/jp/products.html
http://www.polyurethanesupplierschina.com/


actually my rebase messed up, sorry...fixing


I'm little confused, did not `base::CreateTemporaryDirInDir()` take case of OS differences?


thanks forgot about the switch to std


Movin' on up in the world.


Hi @TelanJ the config_opts.h describes the default config.
So I think here you could use ceph default config not "cephx,none", because the config is "cephx" or "none", not "cephx,none"


I think maybe in this test case toposort should not be used, so that there's no need to be worried about fgraph changing. local_dimshuffle_lift.transform(node) can be used instead.
```
    def test_recursive_lift(self):
        v = T.vector(dtype="float64")
        m = T.matrix(dtype="float64")
        out = ((v + 42) * (m + 84)).T
        g = FunctionGraph([v, m], [out])
        init_str_g = ("[DimShuffle{1,0}(Elemwise{mul,no_inplace}"
                      "(DimShuffle{x,0}(Elemwise{add,no_inplace}"
                      "(<TensorType(float64, vector)>, "
                      "DimShuffle{x}(TensorConstant{42}))), "
                      "Elemwise{add,no_inplace}"
                      "(<TensorType(float64, matrix)>, "
                      "DimShuffle{x,x}(TensorConstant{84}))))]")
        self.assertTrue(str(g) == init_str_g)

        # call transform function on the output node, get the new output node
        new_out = local_dimshuffle_lift.transform(out.owner)[0]
        opt_str_g = ("[Elemwise{mul,no_inplace}(Elemwise{add,no_inplace}"
                     "(DimShuffle{0,x}(<TensorType(float64, vector)>), "
                     "DimShuffle{x,x}(TensorConstant{42})), "
                     "Elemwise{add,no_inplace}(DimShuffle{1,0}"
                     "(<TensorType(float64, matrix)>), "
                     "DimShuffle{x,x}(TensorConstant{84})))]")

        # compare new output node's graph with the original one
        self.assertTrue( str(fgraph_of(new_out)) == opt_str_g )
```
It may be a stupid question, but is there a "fgraph_of(node)" function in theano? I tried to use gof.FunctionGraph([v, m], [new_out]), but some errors happened.

Thanks


Is this measurably better than a ConcurrentHashMap?  Thinking about other threads seeing the result of calls to `addParser`, even if the additions are restricted to one thread, is giving me flashbacks to half-forgotten FAQs and makes me think of a concurrent map until a profiler says no.

Back in my Java days, I used this basic technique in static initalizers.  I think it's fine through `gatherBuiltIn()`.  I'm less confident that there aren't subtle bugs once handed off to the app.


typo


Ah thanks - I'll take a look at doing that :)


If we're going global, does this carry any weight as a trait?


`setAccessTime` coming to the `directory` package near you in next release!


este es mi proyecto de java


@nikivazou I don't understand the description here. Can you add a concrete example please? Thanks!


merging changes from jasonsimpson/master


Esta bien q te mande a orion, técnicamente es imposible q te lleve a tu PC, porq por protocolos de NAT, tu computador no es visible en internet (tu pc no tiene IP pública), entonces, U-cursos no podría mandar información a tu PC.
Por construcción, U-Pasaporte envia la respuesta y redirecciona a orion. (No tiene sentido hacerlo de otra forma). Es por esto q para usar la app localmente, no se debería usar el ingreso por u-pasaporte, porq no testearias el codigo local, sino que el q está en orion (y por tanto, no tendría sentido abrir el local).


I feel like we should just take a node here - it's straightforward enough in the examples to pass in a `span` or whatever.


oh, I forgot renaming instances name...
I'll fix it