So it's pretty easy to swap a browser-specific module during build time. The easy solution is to create a browser-specific `replace` module that throws the desired error, and to configure browserify to swap replace.browser.js for replace.js. However, this feels a bit like scar tissue to me. Quibble is the true node-specific module, not `replace`. (Since we should also adjust `reset` as well.) I think it would be better to create a browser-specific version of quibble that exposes the same API and behaves accordingly in the browser. At present, we aren't going to create a version of quibble that actually *works* in the browser, so the browser version of quibble would instead expose an API that is full of noops and error-throwing functions. But in the future, if some form of quibble is made that exposes the same API and works in the browser, then swapping the quibble stub with a real browser implementation would be trivial.


Needs semicolon.


Nit: I don't think that the `{}` are needed? But they don't hurt, I guess.


In this and other commits you move away from the pattern of:
```c++
QByteArray qb = string.toXXX();
someFunction(qb.constData());
```
in favor of
```c++
someFunction(string.toXXX().constData())
```

"Back In the Day" we reports of user crashes that we tracked down to the QByteArray temporary being destroyed before someFunction was called, thereby producing a use-after-free of the return value of constData. That's why the pattern of QByteArray as a local named variable was so common.

Do you know whether something has changed w.r.t. Qt such that this is now safe?


![stuff](https://cloud.githubusercontent.com/assets/15854353/11244951/4df2dc5e-8e4b-11e5-9028-6cd398e2386d.jpg)
How to traverse these materials and use them?


I add stride_ and change elements_per_row_ type in this commit:
https://github.com/naxingyu/kaldi/commit/52e4b527b04798ce393e481a73fe71ed2cd3e64a
But that commit was #334 , before your commit #336 . There should be conflict when you pushed that commit. How did you handle that?

Xingyu


without this, links to these libraries are not resolving when creating the binary distribution javadocs.


Boolean arguments are difficult to understand at call site. How about passing in the same data object which is passed to `User.create`?

Also the node.js convention is to pass the callback as the last arg, please honour that.

```js
function expectCreateSuccess(data, done) {
  if (done === undefined && typeof data === 'function') {
    done = data;
    data = { name: OLD_NAME };
  }
  return function(err, instance) {
    // ...
    if (data.name) {
      instance.name.should.eql(data.name || OLD_NAME);
    } else {
      should.not.exist(instance.name);
    }
  };
}
```

Thoughts?


I'm also upgrading Joda and Jackson to more recent versions.


Using the header tag instead of -header parameter.


Nit: formatting is off


Adding a bottom tag with a JS script that opens external links in separate windows.


This looks much better now!

I am not sure if `OPTIONS_TRUE` is better than `{ validate: true }` as it hides the most important part of the test code. But if you feel it's worth it then I am ok with that.


Links to external javadocs. All must be declared here, including those useful for only a few submodules (e.g. extras).


Please add a space after double-slash: `// options ...`


you don't need this :-)


* Remove new line.


Right.  Thanks.  That was sloppy.  I'll replace it with something like this which gets the parallel construction right.

**BAD:**
```
(pm as Person).firstName = 'Seth';
```

**GOOD:**
```
Person person = pm;
person.firstName = 'Seth';
```

or

**GOOD:**
```
if (pm is Person)
  pm.firstName = 'Seth';
```

but certainly not

**BAD:**
```
try {
   (pm as Person).firstName = 'Seth';
} on CastError { }

```

I agree that the last one is really unlikely but it's pairing (`if (pm is Person) ...`) is common enough that I think it's worth including.

Again, thanks!


Made these internal again.


Instead of making the `:` optional, you should set `[: ]`. Now all word like `HACKER`, `REFACTORING`, etc... are partially highlighted.


It's the unique functional identifier of the developer. I don't think it's possible to change the key.


Done


@GawainLynch Too much of honor (I hope this doesn't lose in literal translation), that was Squiz Labs and they are located in … :tada: **Sydney**, which – we all know – is in lovely :tada: **Australia** … And now again to blame is … the :koala:

:stuck_out_tongue_winking_eye:


These changes are not merged in master?


This isn't very useful because the default page already shows a summary of classes.


Works for me (we actually want wcsnlen though since the null-character may
not be there). I was just reverting the bug.

On Wed, Nov 18, 2015 at 7:06 AM, Daniel Schürmann <notifications@github.com>
wrote:

> wcslen is a 5 liner:
>
> http://www.opensource.apple.com/source/Libc/Libc-320.1.3/string/FreeBSD/wcslen.c
> I would prefer inline that instead using memory allocation and a extra
> copy loop.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mixxxdj/mixxx/commit/a2e6864793dc366b49202fb0aa191d2b79f0e9e4#commitcomment-14474131>
> .
>


> On Nov 19, 2015, at 1:11 AM, Olivier Yiptong <notifications@github.com> wrote:
>
> Is it intentional for the index.html page to be in the langs directory?
>

Yes, as it's undergone localization (see l10n part 3, all the {{l10n-keys}} have been replaced).

> —
> Reply to this email directly or view it on GitHub.
>


Isn't this block a duplication of L118-L143?


ok


Ouch, I've forgotten to upstream this fix -- thank you very much!
BTW, you probably want to remove `echo` call.


It looks like this brings failures to functional selftests. I'm working on that and will push those in a separate PR.


These are now in LibTriboroughBridge/SharedConstants.cs and LibFLExBridge/Infrastructure/FlexBridgeConstants.cs


Is your intention to return an array or a value?  `sample(1)` will return `Array` but `sample` with implicit 1 returns val.


`SimpleCov.start 'rails'` should do this for you already.


Oh, I see, this is testing `automaticValidation: false`. Ignore my previous comment.


What about 100? (FWIW, I've actually tried to undo over 50 times before.)


I ask because as an end user of that API, I would implicitly expect sample to return a value, not an array.


What is the reason for this commit?
Im currently having an issue with this because it requires me to use a stable release of mailgun-php.


This change alters the tracking information format and makes it incompatible with the previous version of the tool. The old tracking information should be deleted by `'dirt reset'` command.


Done :)


o.o


Do we want to sync also https://github.com/windup/windup/blob/master/forks/frames/pom.xml#L94  to a guava version?


Looks good


Why does this need to always return null?


I would personally create a new `describe` block called `create` instead of adding a comment like that.

```js
describe('model setting: automaticValidation: false', function () {
  // ...
  describe('method create', function() {
    it('should throw with validate:true with invalid data'
      // ...
    });
    // ...
  });

  describe('method findOrCreate', function() {
    // ...
  });

  // ...
});
```

What do you think?


This is redundant, `doValidate` was already set to true.


一般都是对某个代码片段进行解析，从上到下的顺序。


Holy crap! This makes reload times much shorter!


except my previous guava it seems ok from my POV


Sorry I just coming back with another change to make on the formula because we had a user report that was hard to track because the bazel version was not included.

Can you set the EMBED_LABEL environment variable to the version '0.1.1-homebrew' so we can track how it was build? Thank you!


Validation is a potentially expensive operation. You should avoid calling `inst.isValid()` when `doValidate` is false.


Socket service works


@ktisha , since i can not message u neither can i comment on ur posts, ill just do it this way, in one of ur submitted anwser pages of the pycharm tutorial, yes i just started using pycharm and python, https://github.com/JetBrains/pycharm-courses/blob/master/introduction_course/lesson3/task7/len_function.answer
this one 2 be precise, it is not working as a legit anwser for me, and why cant u just do int(len(phrase)/2)


Yeah XP would still be possible but I have no interest in trying to support XP still.

AFAIK STLPort never resolved cross boundary memory allocator problems, those exist separately. I had introduced STLPort way back when MSVC's STL syntax was non standard, which made the code non portable (I was writing and testing primarily with gcc on GNU/Linux back then).

Looks like nowadays MSVC is fine with standardized STL, so we don't need it anymore.


This change will cause the problem described [here](https://github.com/webpack/loader-utils/pull/14) !

Please take this condition into account !


Missing end of sentence.


video_pkt should be cloned before adding it


These should return symfony redirect responses:
use Symfony\Component\HttpFoundation\RedirectResponse;
...
return new RedirectResponse('url.php?to_redirect_to=somewhere&error=failure%20error');

See here:
https://github.com/BitLucid/ninjawars/blob/dev/deploy/www/messages.php#L39


Done


We need to make the same tests as for 2.0.6 beta2 and 3 for the tree initialisation .
I'm working in a new Preset class for Settings , just the skeleton , perhaps you would like to fill it ?
in the next commit  ..


Oops, thank you for the reminder. :-)


404 error


Hi Mark

This helps. It is a challenge to get my head around the iteration.

Thanks.

On Wed, Nov 18, 2015 at 10:03 AM, Mark E Murray <notifications@github.com>
wrote:

> my solution is a little awkward and isn't the most clear step-by-step.
> things are kind of nested an awful lot, and it would be clearer if I had
> inverted the check on the key, since the first step is currently in the
> else statement which feels out-of-order. It would be more appropriate for
> it to read "if the dictionary does NOT have a key for the artist, add it
> with a new mutable array containing the first song title; otherwise just
> add the song to artist's array." So this is an accidental learning
> experience for how making your intent clear can be important for working on
> a team... ha.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/richardgmartin/ios-collections-001/commit/86f318b6951b368e0fb874a6aa985673fa0ac194#commitcomment-14474033>
> .
>

--
Thank you.

Richard Martin
613-697-8276
http://richardgmartin.me


de


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-MacOS-10.9-clang: OK (1953 of 1962 tests passed)

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


Hu that's a 2012 commit, wat.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5714](https://teamcity.mesosphere.io/viewLog.html?buildId=77505&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 197 Build time: 00:20:32
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


What a retard


fixes #18


You are correct. Mostly. This block of code should have never been removed as it's responsible for resolving  the relative path to the workspace. Rather, the code should have been changed by adding the .getRemote() method.


Unfortunately, these were idiomatically correct to begin with; this PR introduces English-as-a-second-language grammar errors.


使い出がないので c18eb270315fced376f80821826c488b5de000f2 でrevert済み


TeamCity pull requests :: netty [Build 677](https://garage.netty.io/teamcity/viewLog.html?buildId=3346&buildTypeId=pulls_netty&guest=1) is now running


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5715](https://teamcity.mesosphere.io/viewLog.html?buildId=77511&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:42


I don't? Tests fail without it


And what if the options is an url containing a variable to be substituted?


or with?


Having trouble getting docker-compose to run successfully on Mac OSX 10.11.1

Will try running as a container instead:
https://github.com/docker/compose/blob/master/docs/install.md#install-as-a-container


they are , but also in configurator !


nit: perhaps CreateInstance is more idiomatic name (as this is what Activator uses).

This is just internal though, so fine to leave current name.


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1959 of 1968 tests passed)

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


g


How about:

RW: Database field whose initial value is specified as a parameter in the object constructor. This field is writeable via the API.
StaticRO: Database field whose initial value is specified as a parameter in the object constructor. This field is not directly writeable via the API.
DynamicRO: Database field whose initial value is not specified as a parameter in the constructor, but is instead given a default initial value. This field is not directly writeable via the API.


I'm not sure "the entire runtime, or any arbitrary sub-system between a Mule instance" is accurate. If we don't support such use case I would remove this sentence.


oh, ugh. fair enough. we really ought to refactor it to read in the whole file before it starts.


@codels  Hello, your project is very good, do you have a SKYPE account?


That is probably a good idea.


Lets try this and see. Modifying styles for the styleguide is neccessary in some cases, but could potentially cause misunderstandings if developers use webinspector to read the styles while in the styleguide and consequently make wrong assumptions about how the element is supported.


Il reste un bug étrange. Lors de la première attaque, on fait deux lectures invalides lorsqu'on parcours la liste des cibles. Mais lors des autres attaques, tout se passe bien.
Ce bug ne cause pas de segfaults, mais je ne vois pas d'où il vient...


Please ignore this commit. This is the same commit 6b6bdad553d77c4313b4f2d13694050e2e5b449b but with a different SHA hash.


嗯，这样挺好的。
有想法show出来，这样大伙可以展开讨论了，哈哈。

棒棒哒。

如上的，我有什么想法，看到的想到的，也都会表达出来。


well, no ... singular form in format of "hasAnything" is commonly used in wrappers to refer either single or many sub elements.


this doesn't work for me. I don't have a `lang-tag` directory


When a function is public, I have to maintain backward compatibility. When a function is private, I have more flexibility to change the design. This means that unless a function has a common use-case and can be presented as part of the API, it will tend to be private.

Why do you want to use it on `:form-params`?


wah, kemarin ak jg mau menyarankan ambil datanya via API 😅

ak belum tau cara lain selain itu sih

Pada 17 November 2015 16.35, Cahyadi Triyansyah <notifications@github.com>
menulis:

> dari sini
>
> https://github.com/codeigniter-id/codeigniter-id.github.io/blob/master/app/teams/teams.controller.js#L16
> ambil dari dari parameter *url* tiap kontributir
> misalnya https://api.github.com/users/puterakahfi maka disana bisa ambil
> data *company*, *name* etc
> atau ada cara lain?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codeigniter-id/codeigniter-id.github.io/commit/b28476c7cc185b1ca0e6719c20b76fcec8889176#commitcomment-14437721>
> .
>


This change prevents migration errors on redmine 3.1

Migrating redmine_backlogs (Redmine Backlogs)...
== 36 TrustUniquePositions: migrating =========================================
-- remove_index(:issues, [:position, :position_lock])
   -> 0.0270s
-- remove_column(:issues, :position_lock)
   -> 0.3224s
== 36 TrustUniquePositions: migrated (0.3496s) ================================

== 37 AddRbProjectSettings: migrating =========================================
-- create_table(:rb_project_settings)
   -> 0.0028s
== 37 AddRbProjectSettings: migrated (0.0028s) ================================

== 38 RbAddHistory: migrating =================================================
-- create_table(:rb_issue_history)
   -> 0.0013s
-- add_index(:rb_issue_history, :issue_id, {:unique=>true})
   -> 0.0074s
-- create_table(:rb_sprint_burndown)
   -> 0.0021s
-- add_index(:rb_sprint_burndown, :version_id, {:unique=>true})
   -> 0.0063s
-- add_column(:issues, :release_id, :integer)
   -> 0.3194s
Rebuilding history...
    67 (     1/7431)...
rake aborted!
StandardError: An error has occurred, all later migrations canceled:

wrong number of arguments (1 for 0)/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/scoping/named.rb:24:in `all'
/var/www/redmine2/plugins/redmine_backlogs/app/models/rb_issue_history.rb:163:in `rebuild_issue'
/var/www/redmine2/plugins/redmine_backlogs/app/models/rb_issue_history.rb:329:in `block in rebuild'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/relation/delegation.rb:132:in `each'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/relation/delegation.rb:132:in `each_with_index'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/relation/delegation.rb:132:in `public_send'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/relation/delegation.rb:132:in `method_missing'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/relation/delegation.rb:99:in `method_missing'
/var/www/redmine2/plugins/redmine_backlogs/app/models/rb_issue_history.rb:327:in `rebuild'
/var/www/redmine2/plugins/redmine_backlogs/db/migrate/038_rb_add_history.rb:35:in `up'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:568:in `up'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:608:in `exec_migration'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:589:in `block (2 levels) in migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:588:in `block in migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/connection_adapters/abstract/connection_pool.rb:292:in `with_connection'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:587:in `migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:765:in `migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:995:in `block in execute_migration_in_transaction'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:1043:in `ddl_transaction'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:994:in `execute_migration_in_transaction'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:956:in `block in migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:952:in `each'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:952:in `migrate'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:820:in `up'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/gems/activerecord-4.2.4/lib/active_record/migration.rb:798:in `migrate'
/var/www/redmine2/lib/redmine/plugin.rb:481:in `migrate_plugin'
/var/www/redmine2/lib/redmine/plugin.rb:453:in `migrate'
/var/www/redmine2/lib/redmine/plugin.rb:467:in `block in migrate'
/var/www/redmine2/lib/redmine/plugin.rb:466:in `each'
/var/www/redmine2/lib/redmine/plugin.rb:466:in `migrate'
/var/www/redmine2/lib/tasks/redmine.rake:127:in `block (3 levels) in <top (required)>'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/bin/ruby_executable_hooks:15:in `eval'
/usr/local/rvm/gems/ruby-2.0.0-p643@redmine3.1/bin/ruby_executable_hooks:15:in `<main>'
Tasks: TOP => redmine:plugins:migrate
(See full trace by running task with --trace)


y a déjà un border-box avec bootstrap je crois, et les prefix sont fait automatiquement avec gulp pas besoin de les préciser.


Because
- :params combines GET and POST variables
- :params has no metadata if it has already been nested
- And: https://github.com/mpare-net/clj-trainwreck/blob/master/src/ring/rack.clj#L108


Done


OK


TeamCity pull requests :: netty [Build 678](https://garage.netty.io/teamcity/viewLog.html?buildId=3347&buildTypeId=pulls_netty&guest=1) is now running


**Correction : Settings create the file , folders are for configurator .**
I think this is not good. In my head handling files and folders are the same responsability, and the related code can be writed in the same class, not separated in two classes.

A ugle detail in current design is Settings::writeFile and Settings::readFile. The Settings class don't know "where" write or read, settings is asking configurator "where I can store data?" This coupling between Settings and Configurator is unecessary in my opinion. A solution to decoupling is pass the "paths" in the Settings construtor. Now the Settings class know just a path (QString or QDir), not a Configurator.

Decoupling is cool, but no solve the problem of "responsabilities". I think the gold question is "Who is responsible to load/save from json?" Configurator or Settings?

Another good design question:
Why we have a classe to create a folder (Jamtaba 2) and another class to create a file inside this folder? Why these 2 things are not handled in the same class?

My first idea when thinking about this last question is: We need just the configurator, the solution is merge que Settings stuff in configurator and remove Settings. But, always a but, I feel somethings in Settings don't make sense as Configurator members. So I decide test this "feel". I opened QtCreator, opened the Settings class and check all class members asking "this will make sense as a Configurator class member?"
The answer, in my opinion, is yes for all Settings members. We can merge the two classes and things will make more sense.

I don't a use for these two classes, we need just one, and Configurator is a good name :)


Yes sorry bash, it is not zsh.

OSX Bash doesn't understand that. I don't know why.

./build.sh: line 165: syntax error near unexpected token `&'
./build.sh: line 165: `        cppcheck |& tee $log'

Version information.

GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin14)
Copyright (C) 2007 Free Software Foundation, Inc.

So what does it even do? Can it be written in a different way?


typo: `support`


Registersleutel verwijderen 0 HKCU\Software\Microsoft\Windows\CurrentVersion\Explorer\UserAssist\{CEBFF5CD-ACE2-4F4F-9178-9926F41749EA}\Count
Registersleutel verwijderen 0 HKCU\Software\Microsoft\Windows\CurrentVersion\Explorer\UserAssist\{F4E57C4B-2036-45F0-A9AB-443BCFE33D9F}\Count
Registersleutel verwijderen 0 HKCU\Software\Microsoft\Windows\CurrentVersion\Explorer\RecentDocs
Verwijderen 2.6kB C:\Users\user\AppData\Roaming\Microsoft\Windows\Recent\AutomaticDestinations\5f7b5f1e01b83767.automaticDestinations-ms
Verwijderen 6.1kB C:\Users\user\AppData\Roaming\Microsoft\Windows\Recent\AutomaticDestinations\f01b4d95cf55d32a.automaticDestinations-ms
Verwijderen 6.2kB C:\Users\user\AppData\Roaming\Microsoft\Windows\Recent\CustomDestinations\590aee7bdd69b59b.customDestinations-ms
Verwijderen 5.5kB C:\Users\user\AppData\Roaming\Microsoft\Windows\Recent\CustomDestinations\969252ce11249fdd.customDestinations-ms
Verwijderen 9.1kB C:\Users\user\AppData\Roaming\Microsoft\Windows\Recent\CustomDestinations\c21386f4d2fb73ce.customDestinations-ms
Verwijderen 864B C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\0448BF2CA385066388F48E52BCD9B761D2876A67
Verwijderen 2kB C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\08F5545BBA5F4F270CAF780EC40DCA9168B8D435
Verwijderen 900B C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\29013309A7001508E025A9CEF743004743D1BC5E
Verwijderen 864B C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\57DFBBE26B60759B3E6D4EC6F73BB28D2FCE0033
Verwijderen 900B C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\C0AAE8C613FCCD4DC7B1DC8A12D43DC9D06DE19E
Verwijderen 2.2kB C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\D6E04376418D479E7DC26AAD5D24953D729D91E4
Verwijderen 864B C:\Users\user\AppData\Local\Mozilla\Firefox\Profiles\q2dqt6nl.default\cache2\entries\F0CD1D8AF7C3DBFD777DDECBF9EEA8BE69767042
Verwijderen 524.3kB C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\cookies.sqlite
Verwijderen 10B C:\Users\user\Application Data\Mozilla\Firefox\Crash Reports\InstallTime20151029151421
Verwijderen 98.3kB C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\webappsstore.sqlite
Verwijderen 196.6kB C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\formhistory.sqlite
Optimaliseren 0 C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\content-prefs.sqlite
Optimaliseren 0 C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\healthreport.sqlite
Optimaliseren 0 C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\permissions.sqlite
Optimaliseren 0 C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\places.sqlite
Verwijderen 229.4kB C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\content-prefs.sqlite
Verwijder de gebruikersgeschiedenis 0 C:\Users\user\Application Data\Mozilla\Firefox\Profiles\q2dqt6nl.default\places.sqlite
Verwijderen 0 C:\Users\user\AppData\Roaming\Adobe\Flash Player\AssetCache\NRB2M6ZY
Verwijderen 0 C:\Users\user\AppData\Roaming\Adobe\Flash Player\NativeCache\NativeCache.directory
Verwijderen 0 C:\Users\user\AppData\Roaming\Macromedia\Flash Player\#SharedObjects\C5YSKCZL
Verwijderen 492B C:\Users\user\AppData\Roaming\Macromedia\Flash Player\macromedia.com\support\flashplayer\sys\settings.sol
Klembord 0
Registersleutel verwijderen 0 HKCU\Software\Classes\Local Settings\Software\Microsoft\Windows\Shell\MuiCache
net stop wuauserv 0
net start wuauserv 0
Verwijderen 0 C:\Users\user\Local Settings\Temp\acro_rd_dir
Verwijderen 0 C:\Users\user\AppData\Local\Microsoft\Windows\History\History.IE5\container.dat
Verwijderen 20.1kB C:\Users\user\AppData\Local\NVIDIA\NvBackend\backend.log.bak
Verwijderen 93.9kB C:\Users\user\AppData\Local\NVIDIA\NvBackend\journalBS.jour.dat.bak
Verwijderen 23.5kB C:\Users\user\AppData\Local\NVIDIA\NvBackend\journalBS.main.xml.bak

verwijderen---- remove


ssss


change it to .forloop


fffffff


Ah, ok!


I will try it.


@agitator This commit now enforces use of the newer plone.protect on all Plone 4 installs that want to use Products.EasyNewsletter :-(

ZopeXMLConfigurationError: File "......../parts/instance/etc/site.zcml", line 16.2-16.23
    ZopeXMLConfigurationError: File "......./src/Products.EasyNewsletter/Products/EasyNewsletter/configure.zcml", line 27.2-27.32
    ZopeXMLConfigurationError: File "........./src/Products.EasyNewsletter/Products/EasyNewsletter/browser/configure.zcml", line 71.2-76.32
    ImportError: cannot import name IDisableCSRFProtection


I'm nowhere near finished, sorry.


w


type: length


revision comment on GH


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5716](https://teamcity.mesosphere.io/viewLog.html?buildId=77513&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 0, ignored: 197 Build time: 00:13:14
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


ok


A little note


I myself haven't met with such usage before, but it doesn't matter. If you say it's commonly used then we can go with your naming conventions, I just need to understand what conventions you want applied. So, to sum up once more:

```
<report>
    <hasPart>
        <learningOpportunitySpecification/>
        <learningOpportunitySpecification/>
    </hasPart>
</report>
<hasAttachment>
    <attachment/>
    <attachment/>
</hasAttachment>
```

Is this what you had in mind?


typo: `instance`


Is this mandatory ?


J'aurais mis le border-radius à 3px, les angles légèrement arrondis ça fait plus propre.


The fixes to the char broadcast tests look good.


pose wasn't correctly initialized. rviz didn't care. However, tf's pose calculations do.


@tillrohrmann @alexeyegorov Actually there are 1.0-SNAPSHOT binaries available. See http://flink.apache.org/contribute-code.html#snapshots-nightly-builds

Please keep in mind that these are targeted at developers and not officially released.


@elieserdejesus  my idea is to use the recordTab , do like with the blackbox , remember ?
User will choose in a list existing saved presets or open a dialogbox to create a new .
Instead of using  VstSettings we'll use PresetsSettings  for that job


This **requires** the project to be build. Is that intentional?


Would you be able to make a new release with this change? Thanks!


Good note that clarifies our behaviour. I would not use "backends" but "remote servers". And Kamailio doesn't really "transform" but more "because internally the Kamailio transaction layer generates a 408 in the case of no response from the remote server".

Thank you for working on documentation updates!


Claudia dosnt get it


OK. That's good. This can wait. Like anyway any other plugins is not compilable in OSX yet.


This is fine for now but we can make a AbsenceRecord service to abstract the logic of making these $http requests at some point.


Because that's a real world. :wink:


Create a text file called HelloWorld.md


yes


cool


fixed


maybe add test for
map<>
map()
array<>
array()


> On Nov 19, 2015, at 2:45 AM, Olivier Yiptong <notifications@github.com> wrote:
>
> this doesn't work for me. I don't have a lang-tag directory
>

You need part 3, which hasn't been merged yet ^_^ (see PRs waiting for review). After that, branches need to be created (was part 5).

> —
> Reply to this email directly or view it on GitHub.
>


looks good modulo comments.


Done


Cause method underpinning it cannot guarantee null-safety. And indeed there could be lack of such an element due to parser recovery.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5717](https://teamcity.mesosphere.io/viewLog.html?buildId=77514&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:22


@WestLangley @bhouston

While implementing this I mistakingly wrote `texture2D( envMap, reflectView.xy * 0.5 + 0.5, specularMIPLevel )` and... it kind of worked! The optional third parameter `bias` does something similar to Lod, but I fear the `bias` values are hard to map to what we're after.

However, I wonder if we should this when EXT_shader_texture_lod is not available. What's better in that case? Incorrectly blurred reflection or always sharp reflection?


Updating the last pose here, would mean to compute incremental changes only.
However, we want to know the final angle difference.


"Initial"


Use systemctl status auditd


Thank you for your approval

On Mon, Nov 16, 2015 at 10:02 AM, edgar-isla <notifications@github.com>
wrote:

> I approve.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gabed457/utah-k12/commit/c2be4718b8bf49158a017102660647651385a78f#commitcomment-14420665>
> .
>


Igree.
I see you create a STANDALONE_PRESET_FOLDER_NAME.
Are you thinking in separated presets for standalone and plugin?

Humm, I think we have no option here, the presets need be separated because in standalone a preset include the inserted plugins, in VST plugin we can't insert plugins in the input tracks. A simple solution is just load the same preset in standalone and VSt plugin and skip the inserted plugins if the preset is loaded in VST plugin. But separated presets is more clean.


Please, use [stackoverflow](http://stackoverflow.com/questions/tagged/three.js) for help.


It doesn't make much sense to have the "Types of Smart Answer" heading any more.


Explain what happens with the ongoing operations over the connection provided by the previous connection provider. If I'm running a flow with operationA and operationB of configX, suppose I execute operationA with current provider but before operationB I change the provider, which provider operationB will use?.


TeamCity pull requests :: netty [Build 677](https://garage.netty.io/teamcity/viewLog.html?buildId=3346&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5298, ignored: 21 Build time: 00:16:20


go fixer alors :3


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (894 of 894 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-25.13%25-green.svg)


Further:
1. Can probably remove glOrtho() call line 616; it is described as typically used with a GL_PROJECTION matrix. Or move outside the `if` statement (always run it, just in case)
2. Better understand and document GL_PROJECTION, which is used elsewhere but only in a limited way (e.g., SimpleImageStim)
3. Limitation: viewPos only accepts norm units at present


What happens with the provider? Is it disposed also?


INTEGRATION TESTING_2015-11-18 17:56:44


INTEGRATION TESTING_2015-11-18 17:56:58


thanks, they are now also available by triggering the `--rascal` or `--rascalc` flags on the `rascal-shell.jar` :)


 **Configurator is a good name :)**
pero Configurador es mejor !
lol


not not?


As stated in the commentary, using the $viewValue for display is the proper way to format stuff for angular. Also, going that route allows the use of angular formatters.  In my particular case, I want to display dates not locked to the user, but locked to a physical location regardless of the locale of the user. Being able to use formatters fixes that for me.


Here, projection onto the motion axis is required. Otherwise the code looks fine.


Check if the right logs are there in the file or not.


Ideally nested-params middleware short-circuits and updates both `:params` and `:form-params`. Or instead of updating `:form-params` add another key with just POST variables, or alternatively annotates :params with metadata which keys came from POST and which are GET variables.


thanks


Odd: The names for the test files seem wrong, wrong params given the images. These images were autogenerated by tests.utils.compareScreenshot


Interesting, I wonder if typeclass derivation is failing for this new form of SourceFileInfo


Not working.
The constructor for the getvenuephotorequest has not the correct parameters.


line 10 comment GH


GroundDomain-Output-Variable-Name-Changes (mmatts) - x86_64-MacOS-10.9-clang: OK (1961 of 1962 tests passed)

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


Is this reference still needed? It would be best to not have the TriboroughBridge assemblies reference either the FLEx or Lift assemblies, but to allow those to reference TriboroughBridge.


line 5 GH comment


WRTODO: "included the file" -> "included in the file"


Is this 'using still needed?


Ezee, please fix the code
```
if(appType == QtFatalMsg){
         abort();
}
```
When I rename the variable this "appType was renamed too. This is completely wrong. Fortunally this condition will never be true :)

Just change "appType" to "type".


Seems like the 1.0-SNAPSHOT binaries are missing. Looking into the problem right now.


I'll consider making it public for Ring 1.5.0. As you say, it's pure and at this point I think I'm unlikely to change the design of the nested parameters middleware, so it's probably safe to make public.


I'm assuming your final version will not actually have a size method?


@berserkingyadis is this useful?


Already fixed upstream.


resolves part of #66


Consider run-time implications.


Fuck you're right about the latter.

The former, my theory is that devtmpfs behaves differently when mounted _over_ a working `/dev/pts`. This is what my experiments have indicated, but I might have accidentally left some state in play which affected this :/

Investigating...


Check first, before allocating memory.


Simple answer:  yes, they are different, although thanks for asking because while looking at this I realize that I hadn't yet overridden onlyTrashed().  When I fix this then onlyTrashed() will actually be a thin layer around a call to whereSoftDeleted().

Long answer: I decided to document in some detail: https://github.com/tomb10516/NeoEloquent/wiki/Why-did-soft-deletes-need-fixing%3F


You already made a new ListNode, labeled x.


Unless you're working with coordinates, x is almost always a poor variable name.


This API should be renamed in "UpdateNumTxQueue" to make clearer its functionality, since the tx queue num is initialized at 1.
Then, it misses the update to upper layer (through a callback). If there is a Qdisc which has a number of queue synced with the number of hw queue, without callback you should ensure the order of function calls to make everything work:

Create NetDevice
SetNumTxQueue
CreateQdisc
SetQdiscQueueN(Device->GetNTxQueue)

works; other combinations dont.


It would be lovely if you could propose a PR for this :)

Thanks for forking!


I see the if(appType == QtFatalMsg), this is completely wrong :)


The nested params middleware can't update `:form-params`, as there's an implicit contract in Ring that only `:params` is ever changed, and that any `:*-params` keys are left raw.


Anytime you're using offsets (eg, +1 or -1) COMMENT so I know why!


empty start listener


This whole change is not necessary. We just run the test as fedora user. You can make it what it used to be :)


Wifi update should be in its own commit


@agitator Fixed with https://github.com/collective/Products.EasyNewsletter/commit/e4bc721b06be721eec59469eb6f9df496158bd37

Nicked from https://github.com/collective/collective.mailchimp/commit/ec61ac6a46c29ad4528ab9c17a35e573650e88e1 ;-)


Weird - I remember writing the rest of that sentence... basically I was concerned about the cost of the extra hop; we should benchmark it at some point. Will add this in before merging, along with other requested changes.


Not portable, should use cin.get()


Yay! Right, I have to call database.getChatHistory.
However with that everything runs ok, except the history in the db is not updated!! :(
Is the SQL update command correct?


I suggest something like "... were signed incorrectly, which can lead to installation problems. We strongly recommend that you use the newly signed versions and distribute them to your users."


I discussed this with @AndresTanasijczuk , we both agreed this is harmless and improves the code uniforming how the white and black lists are applied when ignorelocality is specified and when it is not specified.


As we discussed in class, this method should never return false.


I admit this is purely a stylistic change, mostly because the phrase "the missing package manager" needs a subject close by to tell you that it's a `library manager` **for** a certain domain.

e.g:

# The Missing Library Manager
> Is this for managing all of those 'missing' libraries that have been cluttering up my hard drive?
> Perhaps the library manager itself is missing... from where?  Where did the library manager run off to?

# The Missing Library Manager _For Embedded Systems Development_
> This phrase is much less ambiguous.

As an aside, the phrase "The missing package manager for OS X" is used by Homebrew.  I think the inclusion of "for OS X" is really essential there too, because the phrase doesn't work as well without it.


nice cleanup!


Move this test to normal cloudtests module.


I'd like to maintain this information inside the NetDevice subclass, to avoid calls to SetTxQueueN (or UpdateTxQueueN).


82-88 seem unnecessary, since all developers have signed their add-ons before.


CreateEmptyMessage might be clearer still... as you say, it's internal, so at least we can fix it later.


You should be consistent in how you use the this.fieldName convention.


yep. Do you want to make a PR to the webpack-1 branch?


First letter should be upper-case


saw approval from javajolt
at https://github.com/basho/riak_pb/commit/6e75b5ec69662724f577ce286663a8966877653a


merging basho/riak_pb/fixes/am/fix_encode_timestamp_cells = 6e75b5ec into borshop-integration-163-fixes/am/fix_encode_timestamp_cells


basho/riak_pb/fixes/am/fix_encode_timestamp_cells = 6e75b5ec merged ok, testing candidate = 327d6e7b


Instance variables should not be created within class methods like this one. It was fine where it was before. Now calls like this one https://github.com/fog/fog/blob/master/lib/fog/openstack/requests/compute/list_tenants.rb are not possible to make.


ooo didnt know that, thanks


WakeCallback has not the queueId as parameter; how the upper layer can know what queue has woken up ? By checking all queues?


#include <iostream>


Thanks! :wink:


int8/shpalloc bug exists upstream


Thanks, fixed both.


This doesn't seem like a good way to go about it, personally; it would probably be better to H.get_species() and just ensure that the pictures are properly named what get_species() would return.

There shouldn't really be an entire butt var just because of copy machines.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5718](https://teamcity.mesosphere.io/viewLog.html?buildId=77515&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:38


looks good


You should implement a fallback to the hard coded values; otherwise, all the current projects that use the CMS will break. `{{ action.options.localize.deleteButton || 'Delete' }}`


Is this reference still needed? It is best to not have the Lift assembly reference the Flex assembly.


As this is configured in the docker file, I don't know if this is worth it to make it optional.


I'm afraid this doesn't work as expected. With 1 thread and bench depth 17 I get 'Total 629 Mean 0.394277' on average for the additional factor. Seems too low. And with 7 threads things get even worse. 'Total 4947 Mean 0.157065'

I am no mathematician, but I think you need the log(nodes(n) - nodes(n-1)).


remove this methods. Same logic as Object.equals


Thank you!    :-) Only comment is that there are two official SPDX formats, "tag:value" and "RDF".   Can you add those labels explicitly to SPDX2?  SPDX2 on its own (which defaults to RDF) is ambiguous.


- [ ] Remove debugging in this func


k


Is this 'using' still needed? It is best to not have the Lift assembly reference the Flex assembly.


category and categories. not catagory


It is hard to tell without debugging into where this goes exactly (All the indirections!), but does Activate actually call SetForegroundWindow or does it just call Focus? At least in the code that I was referencing (where the foreground app is our sandbox host of the WebOC devenv exe wants to switch the foreground window) we call AllowSetForeground from the foreground process (render process in this case) and then call SetForegroundWindow from the other process (devenv in our case, main process in your case).


Also removed comment about Edge breaking as it seems to be fixed.


Oops.   Just saw the other commit.   Ignore last comment.


Ezee, inside Settings.cpp you will see
```
class Subchannel{
public:
    Subchannel(int firstInput, int channelsCount, int midiDevice, int midiChannel, float gain, int boost, float pan, bool muted, QList<Plugin> plugins);
    int firstInput;
    int channelsCount;
    int midiDevice;
    int midiChannel;
    float gain;
    int boost;//[-1, 0, +1]
    float pan;
    bool muted;
    QList<Plugin> plugins;
};
```
I think a simple approach is
```
class Subchannel{
public:
    Subchannel(int firstInput, int channelsCount, int midiDevice, int midiChannel, float gain, int boost, float pan, bool muted, QList<Plugin> plugins);
    int firstInput;
    int channelsCount;
    int midiDevice;
    int midiChannel;
    float gain;
    int boost;//[-1, 0, +1]
    float pan;
    bool muted;
    **Preset preset; //always using a preset**
};
```
About create new presets:
I think the easy to use approach is put another menu entry in the input channels. So user can create channels, insert plugins, adjust levels and pan, and click in the menu "Save as a Input Preset":
![image](https://cloud.githubusercontent.com/assets/1012741/11246183/bd903370-8dfd-11e5-842a-3105117a0eff.png)

I don't have a cool idea to remove presets. And I suggest, for simplicity, don't allow preset edition.


This is an async operation. We should use the callback for  the bulk insert to create the absence report. See this [StackOverflow QA](http://stackoverflow.com/questions/26592825/mongoose-insert-data-with-promise).


Dear @naxingyu, I left my response note at https://github.com/kaldi-asr/kaldi/pull/336.
Hossein


humm, I'm wrong in the last comment. Presets are not per subchannel, presets including all channels. You can have a preset using just one channel, and you can have another preset using 4 channels and a lot of inserted plugins.


There is no documentation to describe when subclasses should use StartTxQueue or WakeTxQueue. My opinion is that should be exposed only one between them, and the other keep private. E.g. if we keep StartTxQueue private, subclasses are forced to use only WakeTxQueue, which calls StartTxQueue internally and calls also the callback to wake upper layer


These are reduntant, I'd like to avoid increasing so much the API


Checking uint >= is useless


no. they're not the same logic. The ConnectionKey is used so that I don't depend on the arbitrary equals() and hashCode() implementation that the config might have. Instead, I guarantee implementations that depends on unicity.

Default equals and hashCode at the COnnectionKey level would apply over the instance instead of the key


This should be documented a bit more


Documentation on the use of a bitmap is required


@sokra shouldn't it set `NODE_ENV="production"` for `-p` flag, it currently sets it for `-d` flag.


If a NetDevice subclass calls StartTxQueue, and upper layer has all packet already enqueued, it results in a complete starvation of the system.


WakeCallback should have a uint8_t parameter


* We use `_.forEach` below. We should try to keep the functions consistent in case we need to refactor some point in the future. I prefer to use the full name (`_.forEach` over `_.each`) instead of the aliases in case they deprecate the alias.


vcfutils.pl is part of the BamTools package that I downloaded so I was just
able to use that.

On Tue, Nov 17, 2015 at 3:33 PM, Felix Grewe <notifications@github.com>
wrote:

> why that? you should only need bcftools (incl vcfutils.pl) and samtools
> (all here: http://samtools.github.io/bcftools/) not BamTools.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/felixgrewe/map_n_extract/commit/03c99c88e9c62df8a8f49a976c19cb4c0423c02c#commitcomment-14456175>
> .
>

--
Dawson White
PhD Candidate
Department of Biological Sciences
University of Illinois at Chicago
Department of Science and Education
Field Museum of Natural History
Email: dwhite31@uic.edu
Skype: dawsonwhite


Done


In the latest model Result class includes both the result label as "number" and the acquired credit value ...

But i guess you could also add same credit element to both LOS and LOI where LOS credit element is optional and value could be literal. Description element would also be nice in LOS. ... Without those why would there even be any information about LOS?


Surely this is overkill for what can be just written as  ~ `obj == this || (obj.getClass().equals(getClass()) && InterfacesTag.class.cast(obj).interfaces.equals(this.interfaces))`?


Was this on purpose? Looks like $string is used below.


@thomas-jung it looks temp/, log/, public/ and some images pushed.


@khanson679 In this case, it's the need for case **conservation**, specifically. I think everything is covered on page 17 of the [handout that I wrote as an intro to LaTeX](http://bit.ly/latex-workshop).


There is the need to check if the queue is full or if the drop is caused by AQM policies


For ENVMAP_TYPE_EQUIREC and ENVMAP_TYPE_SPHERE, setting bias to specularMIPLevel is okay-ish.  The issue is that bias does a rectangular filter which is incorrect in these two cases, especially near the top and bottom poles.

In a best case, I would try to unify on just the cubeToUv2 for all situations, it would work without the extension on all devices and support correct blurring.  I would advocate just take whatever format as user has and we convert it to the cubeToUv2 format -- then we could support the half dozen formats without getting a lot of glsl code, just more conversion options to cubeToUv2.  The other reason for this conversion workflow is because the filtering needs to be done with a lot of care, and if we only have a single unified format, we only have to get it right once.


:ok_hand: :ok_hand: :ok_hand:


@robsonpeixoto Why not to use it in this very scope, where it's defined? Otherwise, `FuturePool.apply` gives you a `Future[A]` that you can map/flatMap over.


Can you capitalize comments here and in the test?


TeamCity pull requests :: netty [Build 678](https://garage.netty.io/teamcity/viewLog.html?buildId=3347&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5293, ignored: 21 Build time: 00:26:45


I'd rather named this variable as 'adminNetworkId'.


Should I yell about a possibly unnecessary allocation here?


Pourquoi ? Et le test ?


great!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5719](https://teamcity.mesosphere.io/viewLog.html?buildId=77516&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 197 Build time: 00:04:43


Pourquoi pas fusionner avec l'autre ?


The callback to the insert will be called with `cb(err, docs)`. Instead of doing a find after the insert completes, you can just do something like:

```
Student.collection.insert(newStudents, {ordered: true}, function(err, createdStudents) {
  if (err) { return handleError(res, err); }
  _.forEach(createdStudents, function(student, index) {
    newEntries[index].student = student;
  });
  ...
});
```

This also saves us a trip to the db to look up the students we just created.


I installed the ggtd-rcmcarddav branch, created a contact in an existing RC carddav addressbook, then deleted it.  It was deleted on the RC side, but is not deleted on the backend.  The contact is redisplayed in RC after a carddav refresh.  This message is returned on the roundcube/errors log:
[18-Nov-2015 10:20:59 America/New_York] PHP Warning:  Missing argument 2 for carddav::card_delete() in /etc/roundcube/plugins/rcmcarddav/carddav.php on line 681

Allowing a return code of 200 from the back-end (in addition to the already allowed 204) in the delete_record_from_carddav function in the carddav_backend.php file seems to resolve the issue, negating the need for this commit.  That seems to be the difference between OC and radicale back-ends on a delete (OW returns a 204 on delete, radicale returns a 200).

I'm not sure how to contribute that change as I am now to github.  Do I have to fork the repo to submit a pull request?


This is not the current commit - it got changed (see https://github.com/ermshiperete/flexbridge/commit/80acb5127a5427fc5f1a27073f4834cdce45cac0). It might be beneficial to review this change on reviewable (https://reviewable.io/reviews/sillsdev/flexbridge/64#-)


LGTM


This is not the latest commit (see above)


yeah


I'm strongly against this; the static functions of the kernel, which are *not* associated to any data structure, should be inserted in proper container classes, like TrafficControlLayer. There's nothing like that in the Linux kernel; see for example http://lxr.free-electrons.com/source/net/sched/sch_fifo.c . Dequeue returns a packet to the caller; does not get a packet and call Send.


Il y a une mixin tu veux dire ?
Car j'ai essayé un `@include box-siziing(border-box);` ça n'avait pas fonctionné.


Yes, would be better if all of these flags were themselves command objects.


Same fix already applied upstream.


Was this on purpose? Looks like $string is used below.


@YaqiWang - You didn't think this through. This opens this file on every processor! I'm seeing sporadic crashes on our testing systems.


Normally, I would create the `permissible` variable and then do:
```go
ol := &OptionList {
	allOptions: permissibleOptions,
	permissible: permissible,
	typeName: "OptionList",
}
```


ok, thx !
fuck .... lol ah ah ah ...


Hi all @rmandelb and @rmjarvis. I have a quick question on this config. When I forked and hacked up the great3 public scripts, they did a lot of cuts to the input cosmos catalogs (the versions linked on the great3 website) that don't seem to appear in this config. (I don't understand most of the cuts, but am interested in figuring out what each does. The code comments helped definitely.) Do the input galaxy catalogs that galsim pulls by default using this script (https://github.com/GalSim-developers/GalSim/blob/releases/1.3/bin/galsim_download_cosmos.py) have these cuts applied? Looking at the script, I think the answer is no. Does the config above need to apply these cuts?


Unhandled error.


Please submit a PR to serialize this information. Better yet you really should be making an Outputer for this kind of stuff, even for a test to avoid these kinds of failures.


rudimentary


This class is very simple and does not provide basic functionalities:

- Set different queue type for the three bands queue
- Control what is happening inside queue (enqueue/dequeue/drop)
- Does not provide a consistent implementation of attributes (i.e. MaxPackets, what happen if tomorrow another queue disc uses instead "Packets" to store the maximum number of packets allowed?)


This looks good. Make sure you run `grunt` to build to see if there are any build errors before merging. It looks like there are some linting errors that would prevent a successful build.


I'm singing in the rain!


Is this field necessary? Seems to be used only in `func (ol *OptionList) Type() string`. I would just "inline" it.


A device cannot "wake" the traffic control layer, resulting in a permanent starvation


thought so


For things like these a ``` // TODO ``` would be nice so it doesn't get lost


After this change when isntalling with:
"brew install --HEAD homebrew/php/arcanist"
I get a non working version. I suspect the prepackaged 'conduit-5.tar.gz' is too old to work with HEAD.
Is it possible to trigger downloading libphputil from HEAD when using --HEAD?


Added test cases


Yeah, but I just wondered why you did remove the Doxygen files, which may help devs browse the code ^^'


How about testing duplicated options passed to NewOptionList?


Oh, sorry for not considering this. How about just limit this test for one single processor? Or I will protect the entire act with `processor_id()==0`.


It works fine. The most important thing we must remember to logout.
I found one drawback.
![4554](https://cloud.githubusercontent.com/assets/7686999/11246690/ba09387e-8e29-11e5-995f-78caf3765adf.jpg)


Are you performing swapping before adaptation or using some other method to do this transformation?


Yes, definetly. This is the kind of overview I was talking about in the other issue.


Que os parece?


This is not cuda-specific, so I would prefer to avoid cuda-specific terminology.


Yes, the 30 Hz works fine, it always did.

Whenever I change the Hz value in SwtichResX it says "Not Saved" when I do save with "cmd S", it saves then asks for a reboot. I do that too but it doesn't work again. SwitchResX shows "Not activated - invalid?" (please see above first picture)


>About create new presets:
I think the easy to use approach is put another menu entry in the input channels. So user can create channels, insert plugins, adjust levels and pan, and click in the menu "Save as a Input Preset":
Sounds good to me
And add a load preset too that opens a dialogbox , centered .


Duh.


题意十分繁杂，具体中文题意大概如下（引自别人的博客）
>
a)第一个玩家拥有全部的52张牌，而其他的玩家暂时没有牌在手上
b)每一个玩家各自报数，若报的数与牌堆顶上的牌数字相同即为match
c)每个玩家各自报数，各自匹配； 也即是说，A报一个数并记住，比较牌堆顶上的牌，接着是B报自己的数，比较自己牌堆顶上的牌，以此下去
d)报数均从1开始，若没有牌在手上则不报数，待至有牌在手上时将上次所报之数加一作为此次的报数
e)若某个玩家（非最后一个玩家）匹配了一次，也就是说报得数跟牌堆上的数字一样时，将此牌放入下一个玩家的牌堆的最底层； 如果是最后一个玩家，则丢弃该张牌。 若玩家不匹配，也就是说报得数跟牌堆上的数字不一样时，将该牌放入自己牌堆的最底层
f)记录由最后一个玩家丢弃了的牌数

本质来说是使用队列进行相关模拟的操作，不难。
**注意** 将`playerNum`数组初始化的时候，不能使用`memset(playerNum, 1, sizeof(playerNum))`，因为`memset`不能将数组的初始值赋为一…… 掉坑啊！ 还是老老实实循环赋值把。

注意只有两个数之间才有空格，防PE。


If the expectation is that this function is always called with a non-null pointer, I'd turn this into a `CHECK_NE(private_key, nullptr)`.


Ah, now I understand.

I have added the HTTP 200 code to the function as well as assigning a default value to parameter 2 in the new delet function. Can you please retry now?


Nit: lets make this private, and maybe call it `_observeHandle`?


cool!


Oof. Good catch Martin, you rock!


Will do a PR shortly.


aaaaaaaaaaaaaaa


So to be clear, I advocate that we should standardize on a single cubemap/environment map layout, probably cubeToUv2, rather than supporting a bunch of them.  It simplifies the code and also allows us to consistently filter these things via this: https://github.com/mrdoob/three.js/issues/7402


Yeah my fault I just fixed it (https://github.com/Homebrew/homebrew-php/commit/784eefc6838dceece37014a186bb6c8c49b10333). Run `brew update` and try again, it should work :smiley:


There is no try/catch here.


drxdreded


How about using `$translator = $this->container->get('translator')` to avoid the repeated calls?


I'm not exactly sure, but I think we are using a heuristic there, so there may be examples that it doesn't cover. @mglukhikh should know more about this


Code bizare !


Yes, remove this line


Start


You are correct, bcftools also has vcfutils.pl.

On Wed, Nov 18, 2015 at 10:12 AM, Dawson White <dawson.white@gmail.com>
wrote:

> vcfutils.pl is part of the BamTools package that I downloaded so I was
> just able to use that.
>
>
>
> On Tue, Nov 17, 2015 at 3:33 PM, Felix Grewe <notifications@github.com>
> wrote:
>
>> why that? you should only need bcftools (incl vcfutils.pl) and samtools
>> (all here: http://samtools.github.io/bcftools/) not BamTools.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/felixgrewe/map_n_extract/commit/03c99c88e9c62df8a8f49a976c19cb4c0423c02c#commitcomment-14456175>
>> .
>>
>
>
>
> --
> Dawson White
> PhD Candidate
> Department of Biological Sciences
> University of Illinois at Chicago
> Department of Science and Education
> Field Museum of Natural History
> Email: dwhite31@uic.edu
> Skype: dawsonwhite
>

--
Dawson White
PhD Candidate
Department of Biological Sciences
University of Illinois at Chicago
Department of Science and Education
Field Museum of Natural History
Email: dwhite31@uic.edu
Skype: dawsonwhite


Thank you! I will have a look ASAP. You mean for the highest thread?


Right, I'd just limit the output to rank 0.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5720](https://teamcity.mesosphere.io/viewLog.html?buildId=77517&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:43


the batteries are interchangeable, so if its your front derailleur you just shift the rear into the biggest gear and move the battery from back to front.

Or have a spare battery with you.

But if you are enough of a boy scout to carry an extra battery, you probably remembered to charge your battery before the ride anyways.  They've got about a 2-month charge time on them, it's not like just a single ride will do it.


Lol, I just saw this now. Sure, why not? :+1: :smile:


Yes, I've built TAs.


Is that needed, or just a typo?


Huh?


> Sorry if I'm wrong, but a weak reference won't work because it'll just be dropped right after it gets assigned, no? And with ARC, I don't think we have to clean up.

@kwonye `weak` indeed will not increase the retain count for the `addSiteAlertController`. In this case however, you don't want to retain the `addSiteAlertController` as it will be retained via the `presentViewController:` implementation. If retained via `strong`, once the `addSiteAlertController` dismisses, it would still be retained and not automatically cleaned up leading to unnecessary calls in `setEditing:` and `toggleRightBarButtonItems`.

Also, the auto cleanup of `strong` properties is only when the object retaining the other object deallocates.


can you change your commit message ? you have two commits with the same message ....


The comment states `(remove entire try/catch block)`


Forget it... :see_no_evil:


Wow that's a thorough explanation, wonderful. So now using `onlyTrashed()` will remain there except internally it will call `whereSoftDeleted()`, but will we be documenting the existence of such method? (`whereSoftDeleted()`) for it to be called if need be, and if so how would you define the difference?


The point was to get *some* data output into some location on the machine when a totally unexpected and previously un-encountered error occurs.


println "toto"


:+1: Looks great!  This is such a good start on a great unit test suite for this plugin.


that *is* funny.


Now you don't require this function anymore.  You can put the testing logic for each scenario here.

Additionally, add failures to your tests.  The plugin returns errors in certain conditions.  Test them.

Changing from an array to a map/table allows you to name your tests (so you know which one failed) and intermix failures/successes, like in the example I linked you to.


Thanks Miguel:  I was also getting the *ember-invoke-action* error which I
notices was  which I noticed was also fixed in the same commit.

This is amazing work: the docs pages look great and explain the library
clearly and concisely.

I really appreciate your efforts.

Thanks again

On Wed, Nov 18, 2015 at 4:04 AM, Miguel Andrade <notifications@github.com>
wrote:

> Hi.
> This was fixed upstream.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paddleman/ember-leaflet/commit/b98cf49a89834007435060ad05dd2a1fe2b89b43#commitcomment-14469849>
> .
>


Very cool! Thanks for doing this. But I don't think this works for my histogram, or is this some python3 thing I don't know about?

```
## HISTOGRAM	java.lang.Integer
insert_size	All_Reads.fr_count	All_Reads.rf_count	All_Reads.tandem_count
2	2102	677	394
3	2069	755	170
4	2304	654	126
```


Lol.

Otherwise please hold comments for now :smile:

This branch is very much unfinished.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5722](https://teamcity.mesosphere.io/viewLog.html?buildId=77519&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:12


This change breaks the use case from #846, since the controller argument is really a function (which isObject validates to true), and later gets constructed with ```new``` (https://github.com/chaplinjs/chaplin/blob/4a2eea5e9429d4b694cd29f3e625bc679ed1de21/src/chaplin/dispatcher.coffee#L100)


@paddleman Thanks! :)

Please report back any issue you might have.


Hallo zusammen,

im Quellcode auf Folie 13 kommt es beim Kompilieren zu einem Fehler. Ich habe in Zeile 21 den Eintrag

ifstream in(fname);

zu

ifstream in(fname.c_str());

geändert und danach ging es problemlos.

Grüße
Oliver


Ooh, your picard output has more columns than mine, that's exciting! Not a Python version thing no (Travis checks both versions so should pick that up). It's because it's trying to use '`2102    677 394`' as an `int` because I told it to limit the split to two columns. If I remove the second argument with `split` and also catch `IndexErrors` then it should work. Just a second..


I'm not an English expert like @glima but I would say both are valid, despite predefined is more used nowadays.

Anyway, this patch is pretty irrelevant...


change the parameter


Already sent, sorry...

2015-11-18 17:38 GMT+01:00 Pablo García Sánchez <notifications@github.com>:

> More typos fixed, setting the quotes to bold, because I think it is more
> readable @JJ. Also, URL options to break urls properly for issue #40
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/JJ/modeling-volunteer-computing/commit/231dcfc259d1de3594828e838114c4bfc96a8f01
>

--
JJ


also, property should not have open or custom getter


Gotcha, I was getting null with a valid group ID so I figured I'd share a fix


nit:I'd say `quality checks` instead of `tests` here.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 104](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=496&buildTypeId=XbimEssentials_XbimEssentials) is now running


spelling


non mais tout le site est en border-box par défaut.


you'll want to put `set -e` to the top of this file. That way if there is a failure in one of the steps, this script itself will also fail.


Same here, you want `set -e` up top (under the hash-bang).


Woo hoo.  Working staggered multigrid!  👍🏻


I am wondering why you are spying on localStorage. Isn't it better to let the call go through?


Not needed. Super class handles this.


`set -e` here too.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 104](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=496&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:17


今天在使用您的这个框架的时候,报错出现在这一行if (NSFoundationVersionNumber >= NSFoundationVersionNumber_iOS_8_0) , 提示信息为:Use of undeclared identifier 'NSFoundationVersionNumber_iOS_8_0'.  后来我打印了一下系统版本,在NSObjCRuntime.h 里加上了#define NSFoundationVersionNumber_iOS_8_0 1144.170000 然后就解决了报错.  然后我来github 准备提交这个错误,发现您已经更新 .  看来是我离线下载的版本比较老所出现的问题吧.


Can you please stop mentioning my handle (@tomas) in these commits? I have nothing to do with this project.

Thanks.


More found. there should be a 24 hour format.
![65767765](https://cloud.githubusercontent.com/assets/7686999/11247289/78fa6800-8e2c-11e5-8163-a0510641d6a6.jpg)


Unless you check before click that "form" doesn't exist, this test is not complete. The form element could have existed all along


f968ca2de17dc800fbf0a2386e6c545a714ae48a should fix it. I don't have any data like yours though, so please give it a try to make sure.

Do you know what the other columns mean?


Spelling (`extension`).


I think there's a stray ) at the end of it.


The reason this is not an enum map is that the keys are not valid java identifiers and we want to have them as keys in the returned JSON.


same goes for this. You have to check for it to disappear first


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5721](https://teamcity.mesosphere.io/viewLog.html?buildId=77518&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:12:02


Just noticed these files got committed.
Can we get rid of them ? @naveenoid


I don't understand, how including word 'colleague' can ensure misuse of invitation. I think invitation token is enough for that. And if you just want to mark that this particular invitation is 'user' or 'colleague', you don't need to send this value from client.


Yup. Just catch the rest of the list to an `_`.


:-1: Since this returns a fresh map instance every time, I don't think we need to make it unmodifiable.


> I would advocate just take whatever format as user has and we convert it to the cubeToUv2 format

How do you convert spherical mapping to cube mapping?

Lately, I've been doing some [dailies](http://mrdoob.neocities.org/) to experiment with the new material. I noticed that in order to have blurred reflections the envMap had to be a cubemap. The editor still doesn't support cubemaps so I'm just using simple spherical maps by now as they do the trick and they're cheap.

On yesterday dailies I experimented with blurred reflections, so I added a `CubeCamera`. With that working the lack of blurring in spherical environment maps became too obvious so I decided to implement it.

This scene uses spherical environment mapping in the torus.

Before: http://mrdoob.neocities.org/010_/ (notice how the torus reflection is always sharp)
After: http://mrdoob.neocities.org/010/


looks good! I guess it will solve the problem of being unable to delete kinto.ini file


Same fix already applied upstream.


Could have done that, didn't :wink: (yours would have been neater)


agree


Javadocs and `@since` will add.


Indeed. Good catch.


Ugh, yeah, this makes no sense. I'm just going to change it to use the six.u() function.

    u('\n'.join(self.lines()))


I just changed the generate point of new statements. we know when we start the adapting procedure, the rules had been matched already. Therefore, the first sentence to be replaced appears, we can start to change or replace with the new sentanses, which is correct for swing to SWT.
However, the other APIs accept this style or not remains to be verified.


Closes #33, closes #40


Assigning data to a struct sol_ptr_vector * here instead of using it directly may improve code readability and avoid further usage errors.


Well, "is not modified between the check and the usage" describes situation well enough. On entering a loop we lose data flow information for all variables which are changes in the loop. For example
var x: String? = ...
if (x != null) {
    while (true) {
        x.length // Error: type mismatch
        x = "abc" // because x is changed here
    }
}


103643052-FiniteDifferenceFluxOutput (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1432 of 1432 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-70.49%25-green.svg)


Hi Kate,

This is a huge milestone!

Congratulations!

Carleton

On 11/18/2015 9:42 AM, maddyscientist wrote:
>
> Woo hoo. Working staggered multigrid! 👍🏻
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lattice/quda/commit/26ee8021625c168aa9de5b4b9172e2865b0b28e0#commitcomment-14476910>.
>

--
Carleton DeTar
Department of Physics and Astronomy
University of Utah


Yep, indeed It is --no-linenumber...

The typo is fixed in 6.5.12.

Thanks,
N.

On Wed, Nov 18, 2015 at 3:33 PM, Alexey Radkov <notifications@github.com>
wrote:

> should it be --linenumber or --line-number?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/awgn/cgrep/commit/f8c4cbbe0801c1b5545eb2923e1a829d9def600a#commitcomment-14473101>
> .
>

--
The difference between theory and practice is bigger in practice than in
theory.


HDR textures always use linear color space. See comment from spec: https://www.opengl.org/registry/specs/EXT/texture_sRGB.txt

> preclude floating-point RGB formats, signed RGB formats, new HDR formats, and certain compressed RGB formats from being included because such formats don't really even make sense for sRGB.


@romansaul This is super great (check before reload), sth I always wanted to have to increase robustness +1. Is there any chance that we see the error in nginx log such that we can spot the issue via logentries?


Not sure either, but making these `LinkedHashMap`s wouldn't hurt, IMHO. Not sure what JSON spec says about order of entries in an JSON object, but at least Jackson retains the order, so we probably should, too.


Actually November 2015. It's not 2016 yet!


Looks good.


This patch looks good.


It does not introduce more complexity or performance penalties and provides ability to add Reservoirs that hold typed objects. It may help in the future if we decide to optimize instanceof in the sweep() implementation.


Also we've seen numerous occasions where there were mistakes in the manual "merge" and changes in master were accidentally lost.


Looks good. 10:45AM, 11/18


Oops, i forgot line 4 & 6. Sorry.


I have added two additional tests, one with delay/no delay and one with delay 1second/delay 2 seconds


I think you misspelling ""olvieve""


The changes to the following tests look incorrect:

test/openshmem/feature_tests/Fortran/alloc/test_shmem_shpalloc_01_int8.f90
test/openshmem/feature_tests/Fortran/alloc/test_shmem_shpalloc_01_real8.f90

The barrier tests were also dropped from the uh-tests suite.  We can leave them if they are passing.

All of the other changes look good.


Changed to only allow run offline calls to run when the DB is offline, all calls will fail when DB is in transitioning state.


I try not to extend or nest base selectors, make a separate class and associated mixin if it's a style reused elsewhere.


Sorry about that.


@PonteIneptique why changing 2.2 to 2.1 ?


I have midterm exam tomorrow morning . I will change my commit after      De : Amon Moce Rodolphe BAZONGO <notifications@github.com>
 À : delwende/OLDEXAM <OLDEXAM@noreply.github.com>
Cc : delwende <birbaeliane@yahoo.fr>
 Envoyé le : Mercredi 18 novembre 2015 8h32
 Objet : Re: [OLDEXAM] first commit (5facfa0)

can you change your commit message ? you have two commits with the same message .... —
Reply to this email directly or view it on GitHub.


So yeah. The columns are read orientations, and I'm guessing depending on the aligner these columns might be variable. Number of reads aligned in Forward-Reverse, RF and in Tandem orientations. The FR, RF and Tandem columns are also present in the summary table above this histogram.


GroundDomain-Output-Variable-Name-Changes (mmatts) - 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)


We still using py34 for regendoc on `tox.ini`, how about updating that it to use py35? We may leave that for `2.9` though, no rush.


Re: line 24 in _single.scss comment - you end up with long selectors like this


I think this should just be a link to the root path.  <a href="/" something like that.


this looks like you're multipling timestamp * value. maybe say  "Returns a list of timestamp value pairs [(timestamp, value), ...]`


This is may_cancel in the old framework. Any reason to change?


It doesn't have to be. Either of them can be null, but not both.


This is refered as 1 in the original edition and in Patrologia Latina. I have no idea why it was 2. We might not care but this was weird.


mhh ok.
I'll revert this.


@khanson679

> Also, it's an interesting question of what to do about the subscripts. Let me think about it for a bit.

So, the most straightforward thing to do in this case would be `[$_{\mbox{NP}}$ NP VP$_{\mbox{ing}}$]`, but this uses the wrong font size (see [here](http://tex.stackexchange.com/q/70632/32888)). The ideal thing to do would be `[$_{\text{NP}}$ NP VP$_{\text{ing}}$]`, but `\text` is a command that is provided by the package `amstext`, so it would mean that users would have to load `amstext` (or `amsmath`, which loads `amstext`), whenever they wanted to print the abstract. I'm not sure what the best thing to do is. What do you think?


Can we print the failure here too?


No wait. They are present as rows
```
## METRICS CLASS        net.sf.picard.analysis.InsertSizeMetrics
MEDIAN_INSERT_SIZE      MEDIAN_ABSOLUTE_DEVIATION       MIN_INSERT_SIZE MAX_INSERT_SIZE MEAN_INSERT_SIZE        STANDARD_DEVIATION      READ_PAIRS      PAIR_ORIENTATION        WIDTH_OF_10_PERCENT     WIDTH_OF_20_PERCENT     WIDTH_OF_30_PERCENT     WIDTH_OF_40_PERCENT     WIDTH_OF_50_PERCENT     WIDTH_OF_60_PERCENT     WIDTH_OF_70_PERCENT     WIDTH_OF_80_PERCENT     WIDTH_OF_90_PERCENT     WIDTH_OF_99_PERCENT     SAMPLE  LIBRARY READ_GROUP
456     83      2       343609  455.057504      153.190354      259023581       FR      31      61      93      127     167     215     279     377     555     875
950     890     2       748238  243.963049      241.014954      472132  RF      787     1199    1425    1583    1781    1845    4795    8121    14955   72821
1311    126     2       537477  267.159451      231.40836       131762  TANDEM  47      95      141     191     253     447     1999    2551    18291   16
```


singleton means something different in programming. https://en.wikipedia.org/wiki/Singleton_pattern

maybe say `points: a timestamp, value pair or list of timestamp value pair`


i dunno man :shell:


Use `$('.share-trigger').on('click', function(e) { ... });

`on()` is better than click for mobile perf.


按照这个添加的，可是总是乱吗，为什么啊？


In fact it is still listed as may_cancel in shared/operation.go in this tree, so there is some inconsistency here.


This will give an error in the next example code block (where `str_s` is used)


Spec in adcec1dac2655df59f4f1ab85001158f0a2d66e7.


Hold on -- I'm checking on this RIGHT NOW :D @Isabelle1512 , Nicole's latest work won't be at the version that Judy sent you to because that version only includes what we showed at the last demo :-/


Is it acceptable to ignore the others? Just take the first row / column? The counts for FR seem to be quite a lot higher. Alternatively I could sum the numbers I guess.

Just realised that these extra rows in the table will mess up the script as well. Hmm. Could you make a new issue with your output file please? Or e-mail it to me?


There's an interesting bug related to that. An endless loop if the ffmpeg libs are missing.

From here: https://github.com/xbmc/xbmc/blob/master/xbmc/Application.cpp#L598 we get to https://github.com/xbmc/xbmc/blob/master/xbmc/win32/Win32DelayedDllLoad.cpp#L107 which leads us to
https://github.com/xbmc/xbmc/blob/master/xbmc/Application.cpp#L3027 which ofc raises an exception again. Thus we get an endless loop.

I have a special solution in mind, but not a general, because there could be other exceptions leading to it.
Does you have an idea?


Please use all instead of bin. As and idea use that for code completion


You are correct: those catalogs do NOT include those cuts.  If this config is to make a truly GREAT3-like sample, it would need to do so.  I think this would require some additional config magic, and the question is whether we want to go to those lengths.  My take is:

- We want a way to make cuts on catalogs.
- But I am less worried about making these scripts reproduce the exact GREAT3 cuts.  Probably some basic cuts will be needed though, to avoid complete junk.

BTW if you have questions about those cuts, I'm happy to answer them.  You could post on great3-public since that's where the code is?


Can you fix your indentation please (I'm guessing you're using tabs instead of spaces).

I'm also not sure if this is the right fix.  I suspect there's a deeper issue that's causing both this and the other manifestation you found via pure cursor movement.  So can you add a comment like:

    // HACK: There's some kind of issue with CodeMirror cursor movement around line sentinel characters,
    // so for some reason deleting the next line can cause the cursor to move to the previous one
    // (see https://github.com/firebase/firepad/issues/209).  For now, explicitly re-position the cursor
    // in the right place.

Thanks!


OK – I’ll wait to do the others.

From: Michelle Hertzfeld [mailto:notifications@github.com]
Sent: Wednesday, November 18, 2015 11:53 AM
To: 18F/doi-extractives-data <doi-extractives-data@noreply.github.com>
Cc: Brantley, Isabelle (US - Arlington) <ibrantley@deloitte.com>
Subject: Re: [doi-extractives-data] Copyediting detailed steps (3e655b5)

Hold on -- I'm checking on this RIGHT NOW :D @Isabelle1512<https://github.com/Isabelle1512> , Nicole's latest work won't be at the version that Judy sent you to because that version only includes what we showed at the last demo :-/

—
Reply to this email directly or view it on GitHub<https://github.com/18F/doi-extractives-data/commit/3e655b56dbcaab4ebc395239d6e268b07400ae84#commitcomment-14477258>.

This message (including any attachments) contains confidential information intended for a specific individual and purpose, and is protected by law. If you are not the intended recipient, you should delete this message and any disclosure, copying, or distribution of this message, or the taking of any action based on it, by you is strictly prohibited.

v.E.1


@flexiti you said "2. Library <Adafruit_NeoPixel/Adafruit_NeoPixel.h> should also be checked / modified"
but where is that changes?


Is it not more clear to have file names that describe what the function does?


Please merge


**And add a load preset too that opens a dialogbox , centered .**
Ezee, I think is more easy use menu again. We can create e menu "Presets" or "Load Presets" with N entries based in saved presets:

Menu
    Create Channel
    Create Subchannel
    Save preset -> show a prompt to type the preset name?
    Load Preset -> show a submenu with saved presets.


And did you make sure the variables were indeed set in the TA build? In fact `$(call foo)` is simply ignored when foo is not defined, so in any case you won't get an immediate error. I'm guessing that you have used optee_test and still had the expected values, because the TAs include the private configuration file from optee_os (aka `export-user_ta/host_include/conf.mk`) which has all the `CFG_` settings.
If this is true, all makes sense and my recommendation of using `:=` still remains.


I would make a new class for this, something like ```.swatch-desc``` rather than nesting


should be sorted in 885cc8b


<i>I am no mathematician, but I think you need the log(nodes(n) - nodes(n-1)).</i>

Yes, that's what my experimental code did.


Keep everything to the ```$size-x``` variables, shouldn't have any hard coded rem units


See line 46


awesome!


> How do you convert spherical mapping to cube mapping?

It shouldn't be hard.  One just needs to resample from one to another.  One could do it on a per pixel basis -- write a fragment shader that runs over the whole output environment format and converts from the pixel location to its direction, and then you use that direction to query into the other environment format and then write that result to the output.  The converter could actually do any format to any other format, we just need pluggable functions to get direction from XY and XY to direction for each format to convert.

I do advocate a single standard format for ThreeJS -- it will make things so much easier to implement.  Either a 6-sided cubemap or the cubeToUv2 format -- I was thinking of a 6 sided cubemap but then @tschw convinced me that cubeToUv2 with support for multiple LOD levels is the best solution.

I am fairly busy for the next month so I do not have time to do this unfortunately, I was hoping that this was of interest to @tschw.


Oh, sorry, I see. You're not using this at all, and it gets axed in a later patch. Never mind then.


Why is this necessary?

Things created with this.Foo.create() should automatically pick up this.Y


that is what i said , no ?
You want to make it ?


Yup sure thing. RF alignments would be relevant for mate-pair data :wink:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5723](https://teamcity.mesosphere.io/viewLog.html?buildId=77521&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:05:42


Ok, will push a fix right away.


We may want to use differently styled teases for posts and pages, OR exclude pages from search. Otherwise you get this:
![google chrome - yld local - - screen shot nov 18 2015 11 54 27 am](https://cloud.githubusercontent.com/assets/445861/11247807/b5f9d01a-8deb-11e5-851e-b03953f248f2.png)


What is opt_X in situations where its defined but not a context?  That's curious.


ok thanks, just wanted a reference for the source of the change.


please help ... in the last 0.5 i can't escape from the snow zone


This leaves XFAIL_TESTS undefined.  I would prefer to replace this with just:
XFAIL_TESTS =


Good fix.  I'll apply upstream...


Fixes Issue #1


GroundDomain-Output-Variable-Name-Changes (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1967 of 1968 tests passed)

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


Change to a separate class instead of nesting - something like ```.subnav-item-img```


what?


Add a class instead of nesting ```.subnav-item-img``` or something


> http://mrdoob.neocities.org/010/

That is really beautiful.  It seems slow on my PC and I've got a high end PC with a GeForce 980.

> The editor still doesn't support cubemaps so I'm just using simple spherical maps by now as they do the trick and they're cheap.

I think the editor may never need to support cubemaps if we allow for one to specify the envMap in any format -- such as a equilateral or spherical image.  We would convert upon first load to our standard internal format, likely cubeToUv2, and then we could calculate the blurred LOD levels and the irradiance map, and use that.  I think it could be done all on the GPU and fast.  I think that is a really nice workflow.


I don't see this commit in the latest release (1.0.3) and also 0.50 is broken. When will it be released?

Cheers


:smile: Thanks!


yep - that was a mistake!


Does this have to be a member of operation, since you're only using it locally and returning it? Having it be a member makes the lifecycle unclear.


Make new lines for comma separated selectors, so:

```.tax-collection,
.single-project {
    ....
}```


LGTM


awsome


this check is really necessary, since it could be null even if no default was set.


103643052-FiniteDifferenceFluxOutput (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1959 of 1968 tests passed)

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


Thanks @vkostyukov


fixed


Why did you change that? I think it is better to have expected values in a single place rather that in several places.
Also referral doesn't belong to practice in our system, so I think we shouldn't compose fake objects just for ease of use. What do you think?


> It seems slow on my PC and I've got a high end PC with a GeForce 980

It isn't slow because of ThreeJS or JavaScript, Chrome is 95% idle.  So must be slow for another reason.


@garyb I was going to make a release for field puns and the new generics stuff. Sound ok?


btw `available subcommands` would sound better to me...


much better.
+1


I retried it and the result is the same, except that there is no longer the warning.  This commit seems to only delete locally, with no delete done on the back end.  That is what I am seeing.  The contact is deleted only in RC, and returns on a carddav refresh (as this change seems to cause the delete on the back-end not to be performed).

I found that ONLY the change to allow 200 as a return on delete was needed to correct usage with radicale as the backend.  After allowing the 200 return addition of the hook above and it's card_delete function is not necessary and actually seems to be detrimental.


'const' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).<br>'arrow function syntax (=>)' is only available in ES6 (use esnext option).


'const' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).
'arrow function syntax (=>)' is only available in ES6 (use esnext option).
'removedTeam' is defined but never used.


Missing 'new' prefix when invoking a constructor.<br>Unexpected '`'.


'arrow function syntax (=>)' is only available in ES6 (use esnext option).
Expected ')' and instead saw ';'.
Missing semicolon.


Missing semicolon.


'const' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).
'arrow function syntax (=>)' is only available in ES6 (use esnext option).
Missing 'new' prefix when invoking a constructor.
Unexpected '`'.


Identifier 'team_path' is not in camel case.


Mixed double and single quotes.


Expected ')' and instead saw ';'.<br>Missing semicolon.


Danke


'import' is only available in ES6 (use esnext option).


'class' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).


'export' is only available in ES6 (use esnext option).


'let' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).<br>'arrow function syntax (=>)' is only available in ES6 (use esnext option).


No need for this.  We don't have to create special constants for `VCAP_APPLICATION` in the production code.


'let' is available in ES6 (use esnext option) or Mozilla JS extensions (use moz).


Identifier 'team_ids' is not in camel case.


Identifier 'team_ids' is not in camel case.
Missing semicolon.
Unnecessary semicolon.


Expected an identifier and instead saw '}'.


Expected ')' and instead saw 'this'.


Fixes #31 !!!


Missing semicolon.


Identifier 'user_path' is not in camel case.


Identifier 'leader_team_id' is not in camel case.


Identifier 'team_id' is not in camel case.
Missing semicolon.
Empty block.


No special case for `VCAP_APPLICATION`.


103643052-FiniteDifferenceFluxOutput (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1959 of 1968 tests passed)

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


I guess same question here about chanRun and chanConnect.


Thank you


No need for this test at all since there's no special case for `VCAP_APPLICATION`.


Good idea.  More generic than a special `VcapApplicationBuilder`.


When I run npm install in /misirlou/misirlou/frontend, I get this error. Ideas?
```
Bundling failed: Error: ENOENT, no such file or directory 'js/bundle/main.css.map'
```


This is an infinite queue, pratically... to be consistent, the dimension of the queue should be managed in the main class; not in the subclasses.


I generally use the ```_filename.twig``` notation for anything that is extended - but now that I'm looking at other things I've been kinda inconsistent. Whomp.


Let's do a single `require()` call and store it in a variable.


We can just get rid of this function all together with this patch.


We should probably have these in ```common.glsl`` or something similar:

```
vec2 directionToSphericalMapUV( vec3 direction );
vec3 sphiercalMapUVToDirection( vec2 uv );
etc...
```

Then we only have to have them implemented in one place.


Use ```cell()``` mixin and ```$size-x``` variables for sizes, also ```color: white``` should use a variable (I believe would be ```$color-rotate-text```)


Use ```$size-x``` unless you had a specific reason for using percentages


I haven't had time to test this, but does it really handle editor with parameter correctly? For example, it should be possible to input the editor as `/usr/bin/vim -f`, and massren would then run `"/usr/bin/bin", "-f", filePath`. From the look of it, the update doesn't handle this case. Maybe it's not covered in the test cases, which would explain why the tests pass.


Add a class instead of nesting a general selector, and perfect opportunity for flex box centering! ```align-items: center``` is probably what you are looking for


We should file an issue with IndexedDBShim pointing out this missing interface so that this patch eventually becomes unnecessary.


Yeah!


Checking that printer-make-and-model matches the name of the driver doesn't work for all printers. For example, our Xerox WorkCentre 7220 uses a driver called "Xerox WorkCentre 7220.gz" but its printer-make-and-model comes back as "Xerox WorkCentre 7220, 3.5.1", causing munki to always re-install the printer as the values never match. Same goes for our Konica Minolta C280 where printer-make-and-model is 'KONICA MINOLTA C280 PS' but the driver name is 'KONICAMINOLTAC280'.


My nemoh.cal has '---' at the end of file. This new code above does not works in my case and I always delete manually  the 3 '-'.
nemoh.cal : ".....and y direction and dimensions of domain in x and y direction---"
Bemio : "...ValueError: could not convert string to float: ---"


Seemed like earlier in the review there was a push to remove the superfluous commas.


Which label classes did you want to use for LIBRARY or OTHER?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 5725](https://teamcity.mesosphere.io/viewLog.html?buildId=77525&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 201 Build time: 00:04:43


I think this whole thing should be a different module - like, ```.archive-quote``` - I can talk to her about this. I had added the flexible content blocks there but it looks like everything will be the same, and better off as a solid field. Will talk to her about this tomorrow.


Besides @lpereira's comment, this should check for `size`, not `PATH_MAX` to be more correct.


Use mixin


Good catch :)


@andrechrist  The output of nginx -t is surpressed by confd but the current branch raises hopes:
https://github.com/kelseyhightower/confd/blob/bec0b60ec4331f3a3ee1db104f46983a6f035798/src/github.com/kelseyhightower/confd/resource/template/resource.go#L225

Even nginx has a new option -T that is same as -t, but additionally dumps configuration files to standard output which then confd will log ...


+local function isBotAllowed (userId, chatId)
+  local hash = 'anti-bot:allowed:'..chatId..':'..userId
+  local banned = redis:get(hash)
+  return banned
+end
+
+local function allowBot (userId, chatId)
+  local hash = 'anti-bot:allowed:'..chatId..':'..userId
+  redis:set(hash, true)
+end
+
+local function disallowBot (userId, chatId)
+  local hash = 'anti-bot:allowed:'..chatId..':'..userId
+  redis:del(hash)
+end
+
+-- Is anti-bot enabled on chat
+local function isAntiBotEnabled (chatId)
+  local hash = 'anti-bot:enabled:'..chatId
+  local enabled = redis:get(hash)
+  return enabled
+end
+
+local function enableAntiBot (chatId)
+  local hash = 'anti-bot:enabled:'..chatId
+  redis:set(hash, true)
+end
+
+local function disableAntiBot (chatId)
+  local hash = 'anti-bot:enabled:'..chatId
+  redis:del(hash)
+end
+
+local function isABot (user)
+  -- Flag its a bot 0001000000000000
+  local binFlagIsBot = 4096
+  local result = bit32.band(user.flags, binFlagIsBot)
+  return result == binFlagIsBot
+end
+
+local function kickUser(userId, chatId)
+  local chat = 'chat#id'..chatId
+  local user = 'user#id'..userId
+  chat_del_user(chat, user, function (data, success, result)
+    if success ~= 1 then
+      print('I can\'t kick '..data.user..' but should be kicked')
+    end
+  end, {chat=chat, user=user})
+end
+
+local function run (msg, matches)
+  -- We wont return text if is a service msg
+  if matches[1] ~= 'chat_add_user' and matches[1] ~= 'chat_add_user_link' then
+    if msg.to.type ~= 'chat' then
+      return 'Anti-flood works only on channels'
+    end
+  end
+
+  local chatId = msg.to.id
+  if matches[1] == 'enable' then
+    enableAntiBot(chatId)
+    return 'Anti-bot enabled on this chat'
+  end
+  if matches[1] == 'disable' then
+    disableAntiBot(chatId)
+    return 'Anti-bot disabled on this chat'
+  end
+  if matches[1] == 'allow' then
+    local userId = matches[2]
+    allowBot(userId, chatId)
+    return 'Bot '..userId..' allowed'
+  end
+  if matches[1] == 'disallow' then
+    local userId = matches[2]
+    disallowBot(userId, chatId)
+    return 'Bot '..userId..' disallowed'
+  end
+  if matches[1] == 'chat_add_user' or matches[1] == 'chat_add_user_link' then
+    local user = msg.action.user or msg.from
+    if isABot(user) then
+      print('It\'s a bot!')
+      if isAntiBotEnabled(chatId) then
+        print('Anti bot is enabled')
+        local userId = user.id
+        if not isBotAllowed(userId, chatId) then
+          kickUser(userId, chatId)
+        else
+          print('This bot is allowed')
+        end
+      end
+    end
+  end
+end
+
+return {
+  description = 'When bot enters group kick it.',
+  usage = {
+    '!antibot enable: Enable Anti-bot on current chat',
+    '!antibot disable: Disable Anti-bot on current chat',
+    '!antibot allow <botId>: Allow <botId> on this chat',
+    '!antibot disallow <botId>: Disallow <botId> on this chat'
+  },
+  patterns = {
+    '^!antibot (allow) (%d+)$',
+    '^!antibot (disallow) (%d+)$',
+    '^!antibot (enable)$',
+    '^!antibot (disable)$',
+    '^!!tgservice (chat_add_user)$',
+    '^!!tgservice (chat_add_user_link)$'
+  },
+  run = run


The above 3 lines should be simplified and use CoreUtilities.split


I can't figure it out. Can you pass by when you have time?


Same here


On the search page, "Find Course" is an h1 element.  Does it make sense to have the breadcrumbs remain an h1 element, so that "Find Course" doesn't move when we go to the enrollments page?


developer : @ArashTFB


Ok, this is a bit of a mess, both points are correct, something else is going on here. Let me kill the PR and come back with something saner.


@phase Since `""` will be statically allocated, if no copy is made, O will try to free the statically-allocated string. Not sure if that's an error on Windows, but on Linux, it crashes with an invalid pointer error...


Should we really not clone transient properties?  I'm not sure that's right.


Create new class for this (can't now b/c of WYSIWYG - will do if/when moving it to it's own field)


saw approval from andytill
at https://github.com/basho/riak_kv/commit/90ffdcb05811a8a517ef77182eac0aa7152d61ca


merging basho/riak_kv/feature/az/ts-list_keys = 90ffdcb0 into borshop-integration-1268-feature/az/ts-list_keys


basho/riak_kv/feature/az/ts-list_keys = 90ffdcb0 merged ok, testing candidate = 78f83861


You may want to comment on the whitespace changes in include/asm-generic/global_data.h
in the patch description.  Otherwise this looks good.

Acked-by: Wade Farnsworth