И согласно стайлгайду лучше эту штуку вынести в `activate()`

```
activate() {
  this.giftBox.getList()
    .then(giftBoxes => {
      this.giftBoxes = giftBoxes;
    })
}
```


You're doing doing some manipulation of the ``gf`` dictionary values but I don't understand what from this comment.


Thanks! :+1:


Agreed. problem is that I cannot test as it is always passing on my local. I tried exactly same set-up with Travis (python version and etc) still all passing on local and failing on Travis. :(


It will be clearer to declare error, but after es5 transpiler it will replace `let` for `var` and will complain that *error* was defined in the next if statement.


Import order.


Please send this to the mailing list; we don't communicate through Github for discussions on this project.


I know, that's the main reason why I wanted to double check... we should avoid situation that we look at code fragments in the future and ask ourselfs "why the hell was it done that way?" if the answer is "becaus the programmer was unaware of a better solution"


why don't you use  ̀collect` here ? There's `Collectors.toMap()` that could fill your needs


According to the manual page this should be `extras`:

```
       dnf [options] list extras [<package-name-specs>...]
              Lists extras, that is packages installed on the system that are not available in any known repository.
```

The two behave differently on my system so we need to ensure we are using the correct one - without the 's' I still see packages that are available in configured repos (there should be no 'extras' packages on this system):

```
# dnf list extra *dnf* | wc -l
52

# dnf list extras *dnf* | wc -l
Error: No matching Packages to list
1
```


`<` and `>` are visible as is in the generated docs.


Celui si ne marche plus ?


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2131 of 2131 tests passed, 550 test warnings)

 * 550 tests had: AUD diffs.

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


why `Each`?


I think you should document what this class does and why you should use it. Preferrably also in README.rst.


I am sorry I meant the other patch unprotected_wifi_tether. This patch is in upstream.


See comment above.


You are right. Missed this.


@pauloslund: I believe use can use `wfSharedMemcKey` here as there's nothing wiki specific in the cached data, right? This will reduce the number of HTTP requests we perform to Parsely (currently ~215k / 24h).


Please use the project list style:

```python
    files = [
        '/etc/dnf/dnf.conf',
        '/var/log/dnf.log'
    ]
```


All message will be put into outbox right ? Why don't you get it at `process` start ?


Candidate for v0.1 branch, I'd say?


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


Eh.. You might want to trim the FATFS first before uploading. For example when you made it Short File Name only or disabled Codepage (932=Shift-JIS Japan, 936=GB2312 Chinese)..
Also you might want to use `.gitignore` to remove the produced files, including `.elf` or other from repo.


It does indeed, I've just verified it. And it turns out that ECS handles this scenario very well.
So yes, you can use this feature to build your docker images on top of docker (by using the ecs controlled docker instance).


Changing to this, maybe:
"GF features value are lists. In case the list consists of one element only (e.g. ID/ACC in Pfam) they can be converted to a string. This makes it easier to make some string comparison"


Also for 0.1 branch?


you should use `MessageId` constructor to avoid the string construction


This name is a bit confusing. In fact, it changes whether article to which the save is attempted contains more then 1 infobox (which is an unsupported scenario). Maybe article-unsupported message and checkSaveEligibility() would be more explicit


Gitter room created https://gitter.im/opentk/opentk, any further discussion on this subject should probably be done there.


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


This is also where I edited the build.gradle file to add dependencies for DatabaseManager. This is an important commit!


wefaqerfvgregf


You should explain a bit more what we expect, in particular:
  - a minimal builder file exhibiting the issue (preferably using only socket interface)
  - if necessary, a simplified version of .blend file used


command *handling* behavior?


event *handling* behavior?


I have used this plugin to share on social media. Only help I need is to fix in image from my website instead of default image from ayo plugin to share, same issue for description.


you should not give `Optional<Mailbox>` if you want to call get on it


EJB client and Stateful changes


snapshot *handling* behavior?


My mistake. Fixed.


Several locations that need addition of *handling*


I'm not thrilled with passing a clean status object here...


@pynicolas fixed


But now the point is that a sdist isn't a packaged up source tree. A sdist (current terminology) follows PEP 314, and the rest (including the existence of setup.py) is only defined by de facto standards. You should either remove the term sdist from your text (which will be hard, as pip gets most of its "source trees" by downloading sdists from PyPI and unpacking them) or you need to modify your definition of sdist somehow.


CreationMessage or MessageCreation ?


As I've stated a few times, I really don't think you should be making DB updates inside a postp
This should be done onaccept


why protected ? and why not in the builder ?


Awesome work @cstub. Please squash and push!


The values were deliberately all lists, just like the ``SeqFeature`` object's ``.annotations`` when parsing GenBank files. Having a mixture of object types (some strings, some lists of strings) makes comparison harder.


kjjj


Don't remove this constructor, or we will get a java.lang.NoSuchMethodException: com.docdoku.core.exceptions.NotAllowedException.<init>(java.lang.String)" on soap clients


cf. the “Collection Classes” part of https://www.objc.io/issues/2-concurrency/thread-safe-class-design/
I came to the same conclusion because I discovered objects (articles) being held indefinitely.


> Should we remove all UserString* usage and remove fo.userString* from python interface interface?

Hm, good question. Personally, for the time being, I'd prefer to keep them, just in case we *might* need the ability to translate a stringtable key in the AI scripts.

But in theory you're right, with the ability now to send stringtable keys in a chat message and get them translated on the receivers side, there is no more need to do that on the AI clients side, other than debug messages, and those usually are in plain English anyway.

The only case where translation on the AI client side might be required is when we want to send a parameterized stringtable key. If I understood Geoff correctly, this currently still doesn't work with chat messages, so in this case we need to translate the key on the senders side, substitute the parameters and send the resulting string.

Once sending parameterized stringtable keys via chat messages also works, we probably should remove the `userString*` functions from the Python interface. This will force the scripters to send stringtable keys instead of translated strings, which IMO certainly is a good thing.

Is there general consensus on that, or has anyone any objections?


At lot of logic here, it needs a lot of unit tests IMO.
Coming in further commits ?


nit: remove


So, I should just pass the dictionaries and take the comments out?


@anosach-luxoft, this seems correct to me. `The dependencies for SDL Core vary based on the configuration` -> `dependencies are different, depends on configuration`.


@flavour This is not for recently created record but for others which takes value from recently modified record(arow).

Besides the cap_area_tag has multiple onaccept for same id if we input multiple tag:value pair and sometimes the master table is not visible to component table and recently created id of the component table

I tried doing this in onaccept but was not successful so I implemented it here


we don't do single line constructors (and methods)


`path.join` here works fine, for example:

`path.join("C:\\Cordova\\mobilespec", "/platforms/android/build/outputs/apk/android-debug.apk")`
on windows returns:
`'C:\\Cordova\\mobilespec\\platforms\\android\\build\\outputs\\apk\\android-debug.apk'`


why do you need this constructor at all ?


Looks fine to me.


Would ruin the joke :)
I guess I should change it to something more serious?


why not type="date" ?
https://www.w3.org/TR/html-markup/input.date.html


Definitely @jonatas !
Error handling is very poor at this stage. My focus was to get the happy path going so we could trigger deploys.


> Just for your information @teichsta :

thanks for noticing @lumachan. Has been fixed and build is running …


Um, I don't understand... what got fixed in en.txt in commit 544f9c2cd1e2c52158f10a3a6a1513e0ffa0cfd1 and what has that to do with this commit here...?


#84
Done


Merged.


You must not create a new form, rather add this checkbox to the existing form. You will have to reorganize the page a bit, because it declares 2 forms in lines 31 and 38. We need to declare just one form, which contains all the fields.


В целом подход ок! Но есть замечания.

(Я все замечания пишу просто к этому коммиту, они относятся ко всему коду)

* Именование: все файлы и папки с маленькой буквы! (Можно уже не исправлять, много суеты, но _обязательно_ запомни на будущее) Т.е. movie_catalog и т.д.
* Должно быть достаточно `require 'MovieCatalog'` чтобы использовать весь твой код. Сейчас это не так.
* Ты правильно сделал модуль MovieCatalog в главном файле — но тогда надо в него и сложить все свои классы. Иначе в какой-нибудь программе, которая использует твою библиотеку И какую-нибудь ещё, занимающуюся фильмами, и сама хранит фильмы в базе, окажется 5 разных классов Movie и они передерутся :)
* Хорошо бы примеры использования, которые у тебя и так уже накопились, положить в папку examples/
* Хорошо бы сделать исполняемый файл, такой что:
```
bin/movies --genre Comedy
# выводит список комедий
bin/movies --country-not USA
# выводит список фильмов не в США
bin/movies --directors
# выводит список режиссёров по алфавиту
# и так далее, вся базовая функциональность, которую мы ещё в третьем занятии писали
```
* Добавь гем YARD и сделай немножко доков (можно не прям ко всему-всему, но чтобы видно было, что ты понял как документировать классы и методы)
* Добавь гем Rubocop и постарайся исправить самые частые из проблем, которые он тебе покажет


@vilchik-elena Possible improvement: consider cases where the '{' is followed by white space characters.


log.debug. Este tipo de mensajes, como administrador de JBoss no sirve para nada y resulta bastante molesto. Es información que sólo sirve para desarrollador y por tanto debería estar en debug.


途中で、

```
Rails.application.routes.draw do
  root to: 'welcome#index'
end
```

とするところがあるけども、その前にRSpec入れちゃってここの部分をテストした方がいい気がする


DEFAULT_CHARSET looks like a bad idea. We use UTF-8 everywhere I guess


The part of the message with "reportedGrowth > 700" should be edited too


Should `new` be included here?


Sure, but then these URLs are wrong. Fixed in bfa641c4f46738ccd833df08020eb92c17e3bc94.


TeamCity Civic Assembly :: Build all [Build 61](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1217&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


@pynicolas I don't think it will bring a lot of value (as most of IDE remove trailing whitespaces) but it's easy to do


@vilchik-elena Unit tests don't cover that case


you should use @VisibleForTesting


You're missing _MouseEvent.MOUSE_MOVE_ support for an **mRightMouseDown** boolean, but it's a simple fix.

I do agree that this is something missing from Starling, but proper implementation and class design should be considered here, as it would be confusing to include support for "right-click" functionality in the Touch class.


@SeanHayes Yeah there was a resource which wasn't attached to API instance. Latest version did solved that issue. Thanks!


s/indicitive/indicative/g


Have you checked if sed magic  '1!G;h;$!d' works on OSX too?


@pynicolas indeed! fixed in fdeef24b7852b90dca65c2d0a839bd4c8615e078


Ack


Wie setzt man das am bsten um??
habe im Attr noch Signal Repeats hinzugefügt


@petergoes in retrospect I find formatting the `### Why?` and `### How?` as headings a bit of an overkill. Should we lower them to `**Why?**` and `**How?**` (more similar to AngularJS Style Guide)?


Yah, so this change is relevant and shall be considered some kind of cleanup


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


c


Although, you don't have the "Fixes"


Is it better to put the type signatures on separate lines for clarity? I.e.:

    usGallon :: (Fractional a) => Unit 'NonMetric DVolume a
    usGallon = mkUnitQ (ucum "[gal_us]" "gal" "gallon") 231 (cubic inch)

    usQuart :: (Fractional a) => Unit 'NonMetric DVolume a
    usQuart  = mkUnitQ (ucum "[qt_us]"  "qt"  "quart")  (1 Prelude./ 4) usGallon

Perhaps having two units for the example isn't necessary?


I dislike the new loading screen. I think the old one was more suited (and higher res) where as the new one is more of a workshop image.

Thoughts?
@WindStrike72  @Karawasa @elklopso


Делай лучше проверку нашелся ли журнал


because there's no bug concerning this


@tixxdz
Hi Djalal,
While I was reading this code recently, I found these initializations a bit strange. Both shift_uids and shift_gids are initialized to 1, and when mount options are given, they are turned on again. There is no way to turn them off.
Were they probably meant to be turned off by default, and afterwards turned on only when mount options are given?


I'm not that intimate with the whole script, but the line I changed works in the same way:
diff <(git tag --sort=version:refname  | sed '1!G;h;$!d') <(git tag --sort=version:refname  | tail -r)


TeamCity Civic Assembly :: Build all [Build 61](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1217&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 166 (1 new), passed: 6, ignored: 4 Build time: 00:07:32
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455794165874: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

##### there are 156 more failed tests, see build details
```


I'm not a fan of the alignment of the units. The `Prelude./` throws in a big gap that just looks strange and difficult to parse to me. Especially with the parenthesis. If you really want the full alignment I think I would suggest aligning `usGallon` with `cubic` (and not with the `(`).


And besides, if we have log_threshold set to 0, no slow requests will be counted and tracked.


I would move the else part to a def in order to keep the if-expression simple.


Amount takes only one “m” :D


What did SieveToMultiMailbox became ?


Za zvezdicami postavi presledke, pred seznamom in za njim pa prazno vrstico, da se bo ta lepo izpisal.


I suggest you to update this to `"github.com/Financial-Times/http-handlers-go/httphandlers"` according to Dan's refactoring.
To update the dependency version you should to run `go get -a github.com/Financial-Times/http-handlers-go`


Can you revert this commit, it is wrong


This is a problem, because we already have a column named Value and column names must be unique.


Same problem as with Value.


"While we do not hope to reach exhaustiveness in units we do add units on an as-needed basis, and users are more than welcome to submit pull requests to \thispackage{} for any units that would benefit their work."?


Nitpick: :wink:

```sh
curl -4 http://wttr.in/${1:=Paris}
```

References:
[1] http://unix.stackexchange.com/questions/25425/what-does-param-value-mean


:+1: :+1: :clap:


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


generator が読み込めないよって言われた。
```
$ ./bin/rails g rspec:install
Running via Spring preloader in process 79458
[WARNING] Could not load generator "generators/rspec/install/install_generator". Error: uninitialized constant RSpec.
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/rspec-rails-3.0.0/lib/generators/rspec/install/install_generator.rb:1:in `<top (required)>'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:281:in `block (2 levels) in lookup'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:277:in `each'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:277:in `block in lookup'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:276:in `each'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:276:in `lookup'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:138:in `find_by_namespace'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/generators.rb:155:in `invoke'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/commands/generate.rb:11:in `<top (required)>'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/commands/commands_tasks.rb:135:in `generate_or_destroy'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/commands/commands_tasks.rb:51:in `generate'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/commands/commands_tasks.rb:40:in `run_command!'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/gems/railties-4.1.1/lib/rails/commands.rb:17:in `<top (required)>'
/Users/nakano/projects/awesome_events/bin/rails:9:in `<top (required)>'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
/Users/nakano/.rbenv/versions/2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
-e:1:in `<main>'
Could not find generator rspec:install.
```


http://qiita.com/yuriken27@github/items/30bc7dde013cc0812d4c
この記事を見ると、 `3.0.2` に上げればよい？


As far as I can tell from vim's docs, I don't think closing braces need to be escaped.


Could you add a line break after `enabled;`?


From where is `Scientific`?


Is ALLOWED_HOSTS actually necessary? I've never had to do that..


Thanks for the tips, good to know. However I've removed this for https://github.com/kud/my/commit/e9ad9f909fff282af0a91c7bae4b06980c96c9cd because by default, it's geolocated.


> I'm not sure if it's related, but I noticed that a functional list must not be commented out (or a value that contains only a key), otherwise it doesn't fetch the actual translation in the current selected stringtables.

The problem is, when you copy the functional list over to other stringtables than en.txt, you need to maintain them in all stringtables. Otherwise, if someone only changes a functional list in en.txt (in our case, move a greeting to the list for another aggression level because they feel it fits better there), that change will not work for those who changed the language.

The example you've given isn't really a functional list, apparently what happens there is that macro substitution is used. *That* of course must not be commented out, as the macro directive [[STRINGTABLE_KEY]] is replaced with the corresponding plain text, *which is language specific*. If that key is commented out in the French stringtable, it will be fetched from the English one, which of course contains English plain text. That means, such entries (that employ macros) are *different* in each stringtable.

*Functional* lists however must be *the same* in all stringtables. Which is why they should only appear in en.txt and be commented out everywhere else.

So I think you need to fix that in the French stringtable (from what I've seen, you copied the greetings functional lists over).


It was missing the end of the block anyway!


Ok. Got it. LGTM


This is the solution to the bug you sent me didn't you o.o
The one about the garbled characters ?


Looks good to me. I'll then accept the pull request.


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (997 of 997 tests passed, 0 test warnings)

![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-28.63%25-green.svg)


Please align this 2 line comments to the right as on line 328


Only if I modify `BibDatabaseWriter.writeMetaData()` and add an extra block of code specifically for writing save actions (That's how it works for groups). Do we really want a separate block in `BibDatabaseWriter` for every specific meta tag? Right now, the serialization specifics are all contained in `SaveActions`, which is rather nice. If I do the change, the serialization logic is distributed over multiple classes.


align these comments to the same column as above, on line 194


No need to `dup` here if we're applying a non destructive method?


I'd find it more easily understood (with less code paths below) to set a "mapping_type" name to either OrderedDict or dict, and then *always* use that as the hook parameter. Which e.g. would also allow to use "bunch" as the dict container (which I often do, and currently needs a bunchify() call on the return value).


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


Shouldn't `oauth2client>=1.4.6,<=1.5.2` be required as `oauth2client>=2` breaks the code in the examples?


The issue is actually not with potential name clash .. but the subsequent read of matrix `foo/` will fail since the folder will not have data + output from `transform()`


8155d8b でできた


Closes #131


Mars


I propose to reorder these two sentences to introduce `DynQuantity` first.


I tend to share @gabriele-v 's view and find that adding duplicate tables introduces tremendous confusion and possibility for bugs.
If a user opens a database in Android app what would my assumptions be? Should I trust the database version in Pragma? I'd say yes. However, the code is not up to date and it will continue using table_v1 for any work. Later, when I update the code, it will start using table v2. However, all the splits for the transactions created in the meantime will be in table_v1. Sorry, users, get your data in order yourself?

The same goes for older versions of desktop app. Even though it will *appear* to work correctly, in that the application will not crash, using an older version of the application with a newer database will **not** result in sharing data but the old application will keep using table_v1 and the new application will use table_v2. Well, this certainly *will* be wrong because the split category results will be missing. If someone does not update all their instances at the same time, they will not figure out the problem until they actually need the data. In that case, the original data migration is useless since the new data is now spread across two (or more, if we continue with this principle) tables.

The history for the schema is contained in the source code and I believe that is where it should stay. The version number should not appear in the table name. That is an implementation detail. But, since it is already there, let it be. We could remove this in a future migration, if required.

So, while you guys can have a relatively uniform application version, I certainly can not achieve this with Android's 1,000,000 different versions. Even with the desktop application I've noticed different versions across platforms and people using very old desktop version on Linux, for example.
If they upgrade their Android app, again, they would be writing to table_v2 and when they synchronize the database back to their older desktop application - voila - all the data in table_v2 is gone and the data from table_v1 is shown. Fabulous.
Just seeing this would make me, as a user, dump the software faster that you can say "don't".


"Canges" should be "Changes"


Fuck off


`!t.pos.isRange || t.pos.isRange` is always true


i2cModeは別のi2cデバイスとの衝突を避けるために別でやらせる


Actually all of the html files were transformed into blade templates.


It has broken a lot of the examples:
```
➜  vedeu git:(master) bundle exec ./examples/dsl_alignment.rb
/home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/geometries/dsl/dsl.rb:98:in `align': wrong number of arguments (given 4, expected 0) (ArgumentError)
        from ./examples/dsl_alignment.rb:26:in `block (2 levels) in <class:AlignmentApp>'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/repositories/model.rb:41:in `instance_eval'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/repositories/model.rb:41:in `build'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/dsl/geometry.rb:63:in `geometry'
        from ./examples/dsl_alignment.rb:25:in `block in <class:AlignmentApp>'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/repositories/model.rb:41:in `instance_eval'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/repositories/model.rb:41:in `build'
        from /home/home/ch1c0t/sources/ruby/aliens/vedeu/lib/vedeu/interfaces/dsl.rb:48:in `interface'
        from ./examples/dsl_alignment.rb:21:in `<class:AlignmentApp>'
        from ./examples/dsl_alignment.rb:8:in `<main>'
```


The `:1` is not an initialization to 1 but just means the size of the `int` is just one bit rather than the usual `sizeof(int)`.


I think the other logical branch of this statement needs to return `text` - if `recompressed_text` is blank at this point (which can presumably happen if `censored_uncompressed_text` is blank.


気づかなかったけど、Restartのほうがスマート


Gi


No


Rubber stealing this commit


I would add, before this line:

```python
try:
    check_call(["git", "diff-index", "--quiet", "HEAD"])
except subprocess.CalledProcessError:
    log.error("Component %s has uncommitted changes, cannot create release" % component)
    sys.exit(0)
```

To stop people from attempting to create releases from dirty trees.


That's why I said that it's depends on google/google-api-python-client#182 which fixes the code to support 2.0


Hey, just wanted to know if this is supposed to fully replace the work done in #1963. I understand this is much simpler to maintain and more correct than what was done there. (not taking the removal of #1963's code personally, but I would have liked to have a more "lodash docs'-like version of this, as it was my original intention). Is there anything I can do to help get back on that track?


@thedae enjoy!


Aside from the issues noted, this looks good. :+1: Thanks!


What did you even do?


Great job, but docker have different network modes. In this case you are assuming that docker is using iptables, which is not aways true.

```
Digest: sha256:84318f5a742c58a8570b4f229b4d3b75a4a9afbb4f657d8c4a9beb502f975239
Status: Downloaded newer image for ehlers320/elasticsearch-head:latest
25fc1642b534f055f3af1246133e5267c65e85e825b77032121c64821383de07
Error response from daemon: Cannot start container 25fc1642b534f055f3af1246133e5267c65e85e825b77032121c64821383de07: failed to create endpoint elasticsearch-head on network bridge: iptables failed: iptables --wait -t nat -A DOCKER -p tcp -d 0/0 --dport 9100 -j DNAT --to-destination 172.17.0.1:9100 ! -i docker0: iptables: No chain/target/match by that name.
 (exit status 1)

```


oooo! I like!


Fuck

Sent from my Cyanogen phone

On Feb 18, 2016 12:37 PM, Henrik Sorensen <notifications@github.com> wrote:

What did you even do?

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


This hash is clearly wrong, since it is exactly the same as the hash for alsa-utils below.


Hmm is there a reason they aren't in the map?  Maybe it would be better overall to get them into the map

Nonetheless commit lgtm


- Displaying order by id? is this valid? I think it should be `order by first_name`
- You can write directly like `UserProfile.order(first_name: :asc)`. Please change.


&& has precedence over ||


Result quality code


Probando


indentation issues


Please name correctly your commits via our commit naming convention


do_server is declared like:
int do_server(int port, int type, int *ret,
                int (*cb) (char *hostname, int s, int stype,
                           unsigned char *context), unsigned char *context,
              int naccept, int family);

So, it looks like the order of the family and naccept params of this call site should be reversed.


indentation issue


It think is worth highlighting more clearly what is a big benefit in my mind: that it allows the use of other "Num a" (dimensional-oblivious) libraries, e.g., for linear algebra, while automatically checking of sanity of the computations in said libraries at the boundaries when promoting back.


do you want to update this?


> Hmm is there a reason they aren't in the map? Maybe it would be better overall to get them into the map

Yeah, I have had doubts that I am maybe being too lazy here. AFAIK there is no good reason NOT to have them in the map. I could probably fix the other weirdness too (line 297).


I modified this one to work for PDF Expert--both highlighting and underlining. Any one interested can [grab](https://www.dropbox.com/sh/srmwqyqi4vfwlkm/AACMm1yWdBoROwcrGLukM8uXa?dl=0) it.

`https://www.dropbox.com/sh/srmwqyqi4vfwlkm/AACMm1yWdBoROwcrGLukM8uXa?dl=0`


same as you receive:
```
// for example
// but you can change the arguments before passing them to `next`
singular: {
   pre: (next, root, args, context) => next(root, args, context)
},
```


> However I've removed this for e9ad9f9 because by default, it's geolocated.

Yeah, indeed. I didn't see your update.


Why didn't you increase the replication factor for system_auth keyspace?


Indentation iseus. Please fix


Hi, just to let you know that this change breaks your plugin in all browsers.


Indentation iseus through out. Please fix


Que tiempos...


`npc_koltira_deathweaver` assigned to creature with entry 28912 but this creature dont have any equipments in creature_equip_template. I think is database error.


is this needed?


The removal of the ruby 1.8 branch here probably belongs in another commit.


But still a good explanation. I will use your trick for other scripts. :)


I like request.sql as well.


Why is the response removed from extended logging?


my issue with just throwing an exception is that because this is in a utility, it would change the behavior of the method significantly, which is why I suggested deprecation


I just used JSLint here and it requires all local variables to be declared in one statement at the top of the function.
Is there a style guide somewhere that I should follow instead of using JSLint?


You mean for consistency? Because the feature is indeed `cmake_configure`.


The "future proof ness " alluded to in this comment is presumably the hypothesized fine grained switching of the current task discussed in the README, right?

I ask because I interpret the README's hypothesized task switching as just a throwaway idea,  *not* the plan of record

But the comments here make it sound somewhat like there *is* a plan of record to do something different in the future that will benefit from these calls


コメントテスト


ここはなぜ空行になっているんですか？


I agree with Matthias, I don't think it improves readability to split the implementation across different source files. If you want to move something out, I would move the `Participant` classes out of the trait. They are already separate classes and as such deserve their own source file.


> I ask because I interpret the README's hypothesized task switching as just a throwaway idea, not the plan of record

Correct.

> But the comments here make it sound somewhat like there is a plan of record to do something different in the future that will benefit from these calls

There is not. Not sure if there's a better way to phrase the comment -- or maybe I should just not bother trying to be "future proof" against a scenario I don't actually foresee happening.


Nice version lol


(But it just kind of seems like the edges were being added "by accident" otherwise.)


I think you're suspicion is correct. We might as well pass a string since we don't get a stream and [librsvg converts it on its own](https://github.com/2gis/node-rsvg/blob/master/index.js#L31-L33).


But this one is more optimized. The old one called the `$request->header()` three times where this one calls it one time.

Also it's one line instead of 5 which is also an optimization. And imo is a one line if-else more readable than one over 5 lines.


「マスターサーバは」→「マスターサーバプロセスは」

"come and go"が「通信する」と訳されていますが、これは正しいのでしょうか？
"always running"と対比させ、「移り変わる」という感じの意味で使っているのではないでしょうか？


@dragosh by default it will omit raml version for path


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16257](https://teamcity.mesosphere.io/viewLog.html?buildId=138251&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:20


So what's the folder for love.filesystem and things like io set up to?


Should the logExtended method also be reverted? It lost it's response object at one point as well.


@nursix I am trying to implement this:
quoted from Nuwan - <i>In Area tab, we decided that we do not need to ask the user to
select the Info block because regardless of the language or update or
any other aspect, the alerting area is common to all. Even if we
wanted to indicate two separate alert areas, still it they would
belong to all relevant Info blocks.</i>

I implemented this for single cap_area or cap_resource (including their info_id). But I am struggling to implement this for multiple cap_area or cap_resource under single alert.
Background: cap_area_location and cap_area_tag(multiple KV pairs) are component of area. When I add one area, I can see the info_id for the alert and create the area for both the info segments (hence say both en-US and local language). When I add other area for same alert, I can create area for both info_segments. Same goes on with cap_area_location and/or cap_area_tag.

But with two or more alert_areas, when I update the cap_area I cannot figure out which pair (having different language) of cap_area or cap_area_location or cap_area_tag or cap_resource should be updated because all the info are same like alert_id, info_id etc. If I try to read the location of the newly updated alert, then I am not able to figure out which next record needs to be updated because of the same alert_id, info_id etc.

This is not for recently created record but for others which takes value from recently modified record(arow).

Besides the cap_area_tag has multiple onaccept for same id if we input multiple tag:value pair and sometimes the master table is not visible to component table and recently created id of the component table

I tried doing this in onaccept but was not successful so I implemented it here

Please review and suggest if there is a simple way of doing this out.


:+1: :rocket:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16256](https://teamcity.mesosphere.io/viewLog.html?buildId=138250&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 Build time: 00:02:20


Added with out comment, pls see!!


Yeah its fine,  no need to revise comment.  If some one cares enough they will track down this PR and see our discussion


ale po co? może zwrócić np `2`?


(Or do these commit level comments get GC'ed when/if you delete this branch? )


##Example Comment
_This is a try to **MarkDows**!!_


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


no that one is when coala autofixes something, this one would happen if you edit the file in an editor with unicode chars


"more formal knowledge"の訳は悩むところですが、「より体系的な知識」というのはどうでしょう？
「本格的」でも全然良いので、指摘ではなく単なる提案です。


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


Would be better to be consistent about either calling this method `inject` or `reduce`.


uhm, does this render correctly in rst anyway? I actually don't think so...


it's a mess, the file needs changing, you can get help from a tool like pandoc that cann autoconvert it for you


Noted. https://github.com/frappe/erpnext/commit/67f40775ff588ea997522ebb18028e2a3b09e6cf
I'm using self.conversion_rate and self.company_currency instead of self.get() in drop_ship_invoice.py should I change it?


Yes, recursively (so even in sub directories, etc)


} catch (ClassNotFoundException e) {


what was decision process to change severity of this message?

in my opinion this line even shouldn't not be here. it is method 'is_connective' and does the job ... returns True/False according status of Host. without context of usecase you can not decide whether it is error or not ...

I would rather do not see it here.


npc_koltira_deathweaver assigned to creature with entry 28912 but this creature dont have any equipments in creature_equip_template. I think is database error.

I would be glad if someone create issue.


The commit message here doesn't seem right.


Nice. But I do not know where to how to pass the string argument.
The documentation is ... let's call it shallow.


the one you're looking for is https://github.com/coala-analyzer/coala/commit/b54ec7ff5b2799e63ed5ef26f70961ea94f7e395 which you didn't review


not required, because you know those fields exist in the drop ship invoice and it's mandatory.


`#660066` is no longer fred style, i should remove that comment


nit: Add a period at the end of the sentence


確認しました。問題ないのでマージします。


This comment is related to code readability, which I hope other people agree.
In my opinion, you should define a function that implements the application logic, and pass it as parameter to `app.Action`
For instance:
```
func main() {
    ...
    app.Action = contentPreviewLogic
    ...
}

func contentPreviewLogic {
...
}
```


OK I see your point, I'll look to add a new method then.


이걸써야할지


요걸써야할지...


w efekcie selectory nie zwrócą żadnych elementów drzewa dom (bo pusty content)

w efekcie `_get_parsed_search_results` nie będzie miał po czym iterować, czyli zwróci pustą listę

w efekcie => będzie to wyglądać jakby na yt nie było żadnych filmów o zadanym haśle => a przecież wybuchło api, a nie, że nie ma wyników


Nice. Yes. Is there also a way to prevent them specifying a sha which does not exist on origin?


nie zmniejszasz licznika to nadal masz pętlę nieskończoną ;)


-  [ ] SizeOfライブラリの話はnoteに寄せたほうが良いかと。またSizeOfライブラリはそこまでデファクトではないので、「～をするためのツール(例えばSizeOfなど)を用いて計測する必要がある。本チュートリアルの計算式はSizeOfでの実測値の傾向を参考にしているが、実際のサイジングの際にはどのように算出するかを個別に検討すること。」のような書きかたのほうが良いかと。


Fixed, thanks!


Because it's currently disabled @jasmas


Youch, I'll look into pushing out the fix ASAP then.  Thanks for the
feedback, and apologies for the inconvenience!

On Thu, Feb 18, 2016 at 2:48 AM, Denis Dzyubenko <notifications@github.com>
wrote:

> This patch breaks Q in a major way. We have just hit this in our
> production server - after autoscaling and getting new EC2 instances (which
> automatically does npm install) we got the latest version of newrelic
> module 1.25.2, all hell broke loose after this.
>
>    [ ESC[32m'TypeError: Q.nextTick.runAfter is not a function'ESC[39m,
>      ESC[32m'    at trackRejection (/var/app/current/node_modules/q/q.js:1057:20)'ESC[39m,
>      ESC[32m'    at reject (/var/app/current/node_modules/q/q.js:1131:5)'ESC[39m,
>      ESC[32m'    at _fulfilled (/var/app/current/node_modules/q/q.js:836:20)'ESC[39m,
>      ESC[32m'    at self.promiseDispatch.done (/var/app/current/node_modules/q/q.js:863:30)'ESC[39m,
>      ESC[32m'    at Promise.promise.promiseDispatch (/var/app/current/node_modules/q/q.js:796:13)'ESC[39m,
>      ESC[32m'    at /var/app/current/node_modules/q/q.js:604:44'ESC[39m,
>      ESC[32m'    at runSingle (/var/app/current/node_modules/q/q.js:137:13)'ESC[39m,
>      ESC[32m'    at flush (/var/app/current/node_modules/q/q.js:125:13)'ESC[39m,
>      ESC[32m'    at doNTCallback0 (node.js:419:9)'ESC[39m,
>      ESC[32m'    at process._tickDomainCallback [as _tickCallback] (node.js:389:13)'ESC[39m ] }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/newrelic/node-newrelic/commit/efea1d589ffd8bbd4b1338d888d6befb6ef6789d#commitcomment-16172094>
> .
>


are you fucking stupid?????????
why do you even program lmao


I also removed the last 's' from "taskgens" :-)


I have written a lengthy grump post in the forums read and weep as the saying goes :). I'm not gonna spend any time discussing and I'm not gonna maintain movieinfo. It wasn't mine from start. You can remove Groveshark from the list for sure it's dead.


I like this idea -- perhaps we only report inaccessible paths if no accessible paths were found


For @flavour to review and comment - I don't understand the question, nor the goal. Fran can explain.


Thank you so much! :tada: I was going to hack around this for building React but now it just works. Perfect!


I agree. Besides, I'd hate to throw away good artwork.


Audio problem: I agree, don't bother fixing this for now - let's get something building first.

Qt prefix: shouldn't end in "lib" (it should be one directory up from "lib", i.e. "/usr/local/qt5"). To be clear, what you want is something like:

export PATH="/usr/local/qt5/bin:$PATH"
export LD_LIBRARY_PATH="/usr/local/qt5/lib:$LD_LIBRARY_PATH"
./copy-libs /path/to/AppDir

If that doesn't work then we need to fix copy libs, but for the time being inside copy-libs you can set:

qt_prefix="/usr/local/qt5"

but if you are using your own Qt build you still need to set LD_LIBRARY_PATH before calling copy-libs. (If you are using a distribution's Qt package then then Qt would already be in your library path.)


First part of my feedback done, more to come :)
http://realm667.com/index.php/en/kunena/wolfendoom-blade-of-agony/602-c3m2-untitled-dragonfly


If mu4e-compose-complete-ignore-address-regexp is nil, it will ignore all addresses, which is probably not what you'd want.


このコミットは出来れば一個前のに入れて欲しかったです
(内容的に前のコミットで出たコンフリクト解消なので)


Lmao, is this meant to be a public repo?...


Removing this prevents the preference file (and therefore user hot-keys) from being loaded. This becomes a problem when using the hot-keys preferences menu at the title screen from a fresh start, as only default hot-keys will be available.

Is there any way to work around this without breaking the singleton issue this commit was supposed to fix?


これインスタンス変数にする意味は？


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


ここらへんshowとかでも共通になると思うので、
メソッド定義してbefore_actionで呼び出してあげましょう


Ahhhhh I meant Add "git-rev"!


no. keep it.


if movie.user_id == current_user.id　以外の場合はユーザー更新されたと思っちゃうよね？
なんで、if elseで状況分けてあげて、
当てはまらない場合の場合分けを書いてあげて


@AustinXII It is really cool but he took it from fender (owner of Origins).


Are those TDIL scripts ready to release?
I had finished Metalphosis cards so you can use them.
https://github.com/Fluorohydride/ygopro-pre-script/tree/master/scripts/TDIL-JP


At first I had this working, then I encountered an issue disrupting my progress, but now I figured out the cause for the issue and share details here in case anyone else encounters a similar issue.

I'm using a linux environment and have found that this issue affects:

gummi: xdvipdfmx:fatal: Cannot proceed without the font: /home/sager/.local/share/fonts/fontawesomebackup/fontawesome-webfont.woff

texmaker: "Error: Couldn't open file '/path/to/document.pdf': No such file or directory."

Primarily the issue occurs if in your system fonts there is a woff file for FontAwesome. I had one in my ~/.local/share/fonts/ directory inside its own fontawesome directory.  Removing the fontawesome-webfont.woff file resolved the issue.

The issue was: inability to render document due to fontawesome not being properly detected.


I really tried guys :smile:

We could use the old one after with few edits I guess


Lua's io lib should be setup to read from the romFS as well, but it's read only so won't be able to write anything to it. Should be the same with love.filesystem, but I am gonna work out a solution for writing stuff to the SD card somewhere.


Any specific reason, why this is an interface? If this was a (final) class, we could provide a method to retrieve all supported payment types for validation.


既存を変更っていう場合ならうまくいきそうだけど、
・edit画面で新規登録する場合
・消す場合
とかこの書き方で対応出来る？


can you please tell me where is the option for creating blog? how i can create it?
second thing you extends base_blog in many templates but i can't even find your base_blog file?


Hier fehlt das Prefix `yrewrite_`


Not yet: they are complicated (especially the Xyz Magician), and given how
many bugs came up with the easier "Performapal" cards, I'd rather test them
before I upload them. Worst case scenario, I'll upload the "Magician" cards
this Saturday. I haven't started the "Metalphosis" yet, so thank you,
that'll make things go faster! :)

2016-02-18 13:08 GMT+01:00 Mercury233 <notifications@github.com>:

> Are those TDIL scripts ready to release?
> I had finished Metalphosis cards so you can use them.
>
> https://github.com/Fluorohydride/ygopro-pre-script/tree/master/scripts/TDIL-JP
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EerieCode/EC3/commit/c4d8c7c8646323540e7e6914251b9e48994cfaef#commitcomment-16173659>
> .
>


I got confused with the line below....

```
kw['cmake_config'] = self.name
```


If you say that "you are unable to", and ask me for a simple way around this, then there is only one obvious answer ;) and you don't want to hear that, do you?

So please ask Fran to help clarifying your question, no way I can find out what your problem is from this draft code and your text above in any reasonable time. Sorry, really.


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


これ、意味ないですね。戻します。


Because errors here were (almost?) never useful - usually I get back something related to strings (because when error occurs, our app returns a string with exception instead of expected json), e.g. `TypeError: string indices must be integers, not str`, we never get back what actually happened with the app or, this being integration tests, to be able to drop directly into the functions/methods that caused the issues.

This way at the very least when test fails you get back somewhat descriptive error, and if you still want to drop to the location where this happened, you can pass in the `--pdb` option to `nosetest` and see more for yourself (of course you can do this anyway).

I am also not the happiest with this but it was more useful to me than the before-mentioned error or them alike, though I know why it's problematic - do you think I should remove it altogether?


close and cleanup


move this up


I'd probably inherit this from base product.


image_tagも()いらないね
文字列はシングルだね
↑
ここらへんの指摘繰り返すのやだから、そろそろ癖付けて！


Good work mate.

On 18/02/2016 8:58 PM, Mark Cooke wrote:
>
> @mr-sneezy <https://github.com/mr-sneezy> - you might find these
> changes handy.
> Also, nick has pulled in my updates to SoftwareSerialWithHalfDuplex
> (updated using SoftwareSerial for Arduino 1.6.5r5 as a base) - should
> be smaller and more robust to timing issues as 1.6.5r5 doesnt use
> static tables.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/micooke/SoftwareUart/commit/e6e59a1c73194ae65a01d444be4cb09646f13e3f#commitcomment-16171717>.
>

---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


Fixed, thanks!


By the way, I found out the correct syntax for this is `@:optional` (with a colon).


Shouldn't default version be hardcoded and changed every time there is a version bump for this library? Otherwise there is the possibility that the library breaks over time if it doesn't get upgraded by the user.


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


How does AD handle roles if these always return false?


Can you also change versioning documentation?
http://release-engineering.github.io/productmd/terminology.html#versioning


@mr-tweaker keyhandler is highly WIP atm, i'd suggest not using it would be better


yes because the show updater runs at startup now.


ここを今、しかもajax　笑


link_toは、
link_to 'hogehoge', piyopiyo_path, class: 'class1' ってかけるよ


oxygen-icons needs to stay in there. legacy-icons **only** consists of icons broken out of systemsettings, that went into >=oxygen-icons-5.19.0. Depending on <oxygen-icons-4.14.50, the set already made sure that conflict wouldn't come up anyway.


In my opinion, you should clarify where the `pingHandler` declaration is.
I assumed that it was in `handlers.go`, but it was not totally straightforward.
A possible solution in my mind, which I hope it is not too Java-oriented, is the following:

1. Rename `handlers.go` as `contentPreviewHandlers.go`
2. Create the a Struct `contentPreviewHandlers` with `pingHandler` as attribute
3. Rewrite the line as: `r.HandleFunc("/__ping",  contentPreviewHandlers.pingHandler)`

Another option is to create a `contentPreviewHandlers` package.


Also, the only reason it's 'higher def' its because I resized it to 1920x1080 to save a bit of space.


This closes #27.


same here, legacy-icons only is an addition, not a replacement.


この書き方だとクラス効かなくない？


コミット長いっす


Tá melhor que antes, acho que é o caminho a ser seguido.


this is epic! =)


how big can it be?


@AbdealiJK this one still needs reveiw


you are correct!


これp効いてる？


I understand the requirement from Nuwan, although I think this is actually cap_area_location which is common to all Info segments & not cap_area?
cap_area's Description field is one which should be localised if I am not wrong?
- hence should be different per Info segment (if the multiple Info segments are for multi-linugual support and not just an update)
- in which case we most definitely DO need to specify which Info segment this belongs to...unless there is only one in which case we can indeed hide/automate that
- if we could assume a single Area Location per Alert then we can automatically prepopulate the alert_area_location when creating new Area segments and copy to all if one is changed....however it seems that we can't assume that, which is where the problems start to show up
- of course a nicer UI would seem to be to include the Area Description(s) within the Info segment instead of in a separate form...although with the need to handle multiple Area Locations, this would need to be an Inline form and I'm not sure we can handle an Inline LocationSelector inside an Inline Form

If the Area Description does NOT need to be localised then we have 2 options for this:
(A) Copy the Area to each Info segment
- ugly in terms of UI & also in terms of code/DB storage
(B) Have the Area linked only to the Alert and not to the Info segment
- this seems to make the most sense to me.
- however do we need to support importing of CAP messages which have a different area per info segment (which is perfectly legal CAP standard, even if apparently nonsensical)

We already create Areas as components of the Alert.
Although currently we could ask for the Info segment, that is hidden. This seems to match the UI requirement currently.

Currently there is an onaccept called 'update_alert_id' (I have already suggesting just calling this 'cap_area_onaccept') which is supposed to ensure that the alert_id is populated if we only have the info_id...am not sure what usecase that covers, but doesn't seem to be how the UI works. If we needed to support option (A) then here would be where we could put in the info_id (if singular) or copy to each info_id (if multiple). That doesn't seem hard, although would be ugly. We /could/ then filter out the duplicates in the UI and auto-update them all if the visible one is changed, but this seems all unnecessarily complex...option (B) looks nicer to me.


idまで拾わなくても、movie.userでインスタンスが取得できるはずだから、それでおっけ


The title should be proper English.
For example, change it to "Use UTC time zone for BUILD_DATE_FORMATTED" or something similar.


Minor type.
"desriable" --> "desirable"


h3も聞いてないと思う


think we should keep this as it, just ```application.lang``` with default locale.


что это???


Ack


hmm, I already have an @flavour asking me exactly the same question!
Biplov: It is NOT efficient to ask us bioth the exact same question in different GitHub commits...this is asking us to do a lot more work in parallel..please refrain from doing this!


@alban geez, thanks! ;-)


this is already fixed in master.  thanks!  in fact, i will close this PR.


will add a ```locales``` that will give us the locales from ```Accept-Language``` header or a list of 1 elem with global locale.


なんかpaginationとはいえロケットとか残ってるの気持ち悪いな...
いっっっっっったん全駆逐しない？
後からがいいならそっちでもいいけど...


I would much rather see this working in onaccept
"others which takes value from recently modified record(arow)." still sounds like an onaccept usecase


the wrong indent


this var is never used


This patch doesn't work in python 2.4

 ```
> rhn_check -vv
An error has occurred:
exceptions.AttributeError
See /var/log/up2date for more information
```

```
> tail /var/log/up2date
  File "/usr/lib/python2.4/site-packages/rhn/rpclib.py", line 182, in __init__
    password, timeout)
  File "/usr/lib/python2.4/site-packages/rhn/rpclib.py", line 219, in default_transport
    transport = self._transport_class_https(timeout=timeout)
  File "/usr/lib/python2.4/site-packages/rhn/transports.py", line 257, in __init__
    timeout=timeout)
  File "/usr/lib/python2.4/site-packages/rhn/transports.py", line 44, in __init__
    xmlrpclib.Transport.__init__(self)
exceptions.AttributeError: class Transport has no attribute '__init__'

```


never used


подумай и задай вопрос правильно


this is nice, we will use ```lang tags``` for locales and get rid of this class (it is useless now)


And what if a user doesnt restart/receive an update for a week, will it still update on a schedule?
And me and othrrs that run dev and receive more than one update per day SR will never update the shows? As it will wait the 24h..


「ファイル末尾の空行削除」
なぜこれした　笑
改行しましょう


this stuff should move into its own file for the Diet benchmarks


locale will be the first element of the new ```locales``` method.


cc @AlbertoFuente @makotot @Rokt33r @Layzie @charbelrami @sohje for new translations. Thanks


I can check if they don't exist on any remote, but I don't want to do the parsing to check if those remotes are actually upstream.  That would look something like:

```python
contains = check_output(["git", "for-each-ref", "--contains", "HEAD", "refs/remotes", "--format=%(objectname)"]).strip()
if len(contains) == 0:
    log.error("Component %s has changes that are not upstream, cannot create release" % component)
    sys.exit(0)
```


I think so would be better
```
include LICENSE
include README.md
graft webpack_loader
global-exclude __pycache__
global-exclude *.py[co]
```


It looks nice, thanks.

I will take it from here, just let me know if what I comment makes sense or if missing something.


torsdagen den 18 februari 2016 03.50.31 skrev  Biplov Bhandari:
> we do not need to ask the user to
> select the Info block because regardless of the language or update or
> any other aspect, the alerting area is common to all
To me this sounds as if it requires a model change rather than form-
postprocessing: if you need to replicate the same link with no exception, then
better to move the link one level up in the entity hierarchy.

Easier, cleaner, better performance, less prone to errors or inconsistencies.

Dominic


Kronenbourg est-il le sponsor officiel du gouvernement ?


I don't think, we should add the actually unsupported payment types `CASH` and `BTC` here. Will post a follow-up to make the tests pass without them.


indexed vector values must maintain the order of the values.


Esta é a pagina de entrada do avanildo.github.io

- 222
2
2
2
2
2


Maybe, this file can be moved to a `logging` package?


先頭で、 `require 'spec_helper'` とするのをしてなかった


Что это за хрень, бля!? )


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1549 of 1549 tests passed, 0 test warnings)

![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.65%25-green.svg)


Yes, its really confusing

# dnf list extra | wc -l
Error: No matching Packages to list
1

# dnf list extras | wc -l
94

Any ways extra(s) also lists 'Available packages' which is not of interest. Hence I'm removing this command till the confusing is clear.


これを追加しても変わらず


farman review


gg


comment on this commit


The legacy (empty) notification message is handled first in InageWatcher::handle_notify.


Ты не знаешь, как переводится слово epic или тебя удивляет монитор перед глазами? А может, ты держишь руку в кармане и загадываешь мне загадку, как хобот Федя?


salam


если ты спрашиваешь про содержимое коммита, то это дабл-эпик. Потому что только ты можешь знать, *что это за хрень*.


Hello nice work there,
i want to ask if u can upload the database and thanks.


посмотри скриншот с моим скриптом - выслала тебе на почту


Not really - `list extras` is a command but `list extra` is the `list` command with an argument of `extra`. If there are no packages on your system matching `extra` then it prints nothing.

I see the utility of this it's just the original patch was doing `list extra` (list packages with names matching 'extra') instead of `list extras` (list extra packages not available in any repo).


確かに…今更で申し訳ないです


(and yeah, the command design is pretty dumb but the documentation seems clear)


This is how it looks : do you really want the name of test as well ?
there is an element called testcase which describes the case ....

<testsuite errors="0" failures="0" name="pytest" skips="1" tests="0" time="84.426">
  <testcase classname="git.rhevm-art.art.tests.rhevmtests.system.dummy_test.test_dummy.TestDummy_REST_NFS" file="git/rhevm-art/art/tests/rhevmtests/system/dummy_test/test_dummy.py" line="109" name="test_case_003" time="83.4092900753">
    <skipped message="Status: NEW URL: https://bugzilla.redhat.com/show_bug.cgi?id=1163718" type="pytest.skip">/home/talayan/git/pytest_marker_bugzilla/pytest_marker_bugzilla.py:223: <py._xmlgen.raw object at 0x7f244fc74210></skipped>
  </testcase>
</testsuite>


This commit didn't change which server variables were read. Can you tell me a bit more about your setup?


Hi Team,
We need this functionality to work. Please merge this pull request as soon as possible.


Portuguese of Brazil isn't the same in Portugal, these are different countries. But some words or phrases are the same. I recommend create another file with name 'pt', with the same values.


[portfolio .zip](https://github.com/Nittinshankar/Nittin-creations/files/136182/portfolio.zip)


lol .....hmmm I clearly mentioned .java file! and where is your branch?


Did not like the 2nd run at all.

So one last try. Using default values for A, etc
and params are:
SideAdjustment[][],8,-16,16,4,0.002


"Sure it's broken, but at least it's consistently broken!"


Terimakasih Bro.


i need to check, but i think array fields would not fully work with this component


fgdfgdsfgsdfg


That does not fix the problem. The cached files are indeed no longer removed upon updating a post. Only the file for that post is removed from cache.

However, I expect related files to be cleared from cache too. For instance, the blog list or the home page if it contains latest posts. And this is no longer the case. Unless I've missed something ?


That is the parameter (not the feature) the user should pass. That means "this cmake build depends on the cmake configuration taskgen named `kw['cmake_config']`.


Got it, will be fixed before final version


done


ну и зачем ты удалила содержимое? пздц


tried this script as follows: python downloader.py --youtubeid YcZkCnPs45s --output

but didn't seem to work?


This is very useful! Can you make a PR to upstream?


See issue @https://github.com/logstash-plugins/logstash-output-webhdfs/issues/15


好多beta2的插件都没升级到beta4,坐等正式版


or request.sqla to be a bit more precise? I tend to use sqla to name sqlalchemy stuff.


:+1: the link should probably read `AskTheEU.org` as that's how the site refers to itself.


it wasn't clear for me that the plugin discovers all bz marks and genberate bz object in a list ... and through the test it doesn't connect with bugzilla at all.

this is more decriptive for me ...


ok.  Did that, but copyLibs still can't find Qt5 libraries.  Still looking at new output of ldconfig -p, and I don't see any Qt5 libraries that I've made (although I do see eight Qt4 libraries that must have come preinstalled with Raspbian Wheezy).  Maybe do I need to use some Qt switcher for letting ldconfig know to look for Qt5 libs instead of Qt4?

On another subject: considering that lasconic mentioned would want to build inside Travis, which is an x86-64 virtual machine, there won't be the ability to chroot into arm image, and would have to build mscore-portable with cross compiler.  In that case, would it make sense to run the copy-libs stuff one time on my Arm machine, compress & upload all the needed libraries somewhere for Travis to grab whenever compiling a new mscore-portable nightly and simply extract the uploaded libs into the AppDir_/libs folder?


That is much less intuitive though. FWIW SQLAlchemy's (or perhaps just zzzeek's) standard abbreviation for SQLAlchemy is `sa`, not `sqla`.


We can refactor this later, but LGTM :+1:


Nice catch -- thanks.


I think we do not need a dot here,


Do we want to log or in general do anything with the exception, or are we always sure that it's because of privacy modes?


Commit message should say:

Removed `from .[etc]` import syntax


i will do the changes ...

i prefer to have one styling among all tests and make too much effort to accept any style

it make more confusion for me ...

we see a lot of places in art where you send string or list or none ..  this is really annoys me so i try not to do it ..


Any news on when plexus-interpolation-1.23 will be deployed?


http://blog.willnet.in/entry/2014/06/10/142341
この記事を見ると、 `spec_helper` ではなく、 `rails_helper` を require しないといけないようです。

* `spec/spec_helper.rb` : RSpecの全体的な設定を書く
  * 例えば、ランダムでテストを実行したいなど
* `spec/rails_helper.rb` : Railsの全体的な設定を書く

[ソースコードのコメント](https://github.com/TasukuNakano/perfect-ruby-on-rails/blob/make_welcome_controller_spec/spec/rails_helper.rb#L29-L31
)を読んでみると、
`config.infer_spec_type_from_file_location!` を設定すると、 `GET` や `POST` といったふるまいのメソッドが使えるようになるっぽい
で、それが設定されているのが `spec/rails_helper.rb` に設定されているので、 `requre 'rails_helper'` とすれば良いということかな


could you please add some comment here, what are you going to do with this function, what was your intention?
It's not clear.


@vrv  Thank you for the note. For the label_image the correct flag is

`--output_layer=final_result`

not --output_layer=final_output


Actually thinking about it I guess it's normal behaviour :p

Any idea how I could force homepage and blog list page to be cleared from cache upon post update ?

I guess your fix is working.


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


Nice work :+1:


I am not sure why it happends and how we shall avoid it yet.  Its very
anoying thne its happends

2016-02-18 10:36 GMT+01:00 Michael Werle <notifications@github.com>:

> This commit changed the line-endings on a pile of files again.
>
> Could we please get guidance as to what the line-endings and file formats
> should be on all files in this project so that we're not constantly
> spamming the commit log with whitespace changes?
>
> Sorry to be pedantic, but it's (a) hiding the actual changes, and (b)
> stuffing up merges into my own working branches.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EDDiscovery/EDDiscovery/commit/617f6d9ba191a78587911e82004bb5a84dd7d52f#commitcomment-16170646>
> .
>


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 264](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=794&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:45


There are appropriate channels to discuss whether or not it is fundamentally broken or simply non-standard. You don't break userland code without those discussions.


Коммент на вторую строчку


Kannst du auch noch einen Verweis auf die OpenSenseMap geben? http://www.opensensemap.org/


Коммент на весь коммит


When extracting Exif data from a RAW file shot with a Sony camera the lens name/type was not extracted and displayed even though it was set.

Original:
![bug](https://cloud.githubusercontent.com/assets/9962967/13143742/508f937e-d646-11e5-8df6-f10ae954f639.png)

Fixed:
![fixed](https://cloud.githubusercontent.com/assets/9962967/13143746/528ee026-d646-11e5-9a84-e146b15855fb.png)


Thanks. Doc for this now working properly too.


yes, we can ;-)


Remove this line before merging.


well... `sqla` is in fact the same uppercase letters than in **SQLA**lchemy so to me it feels more intuitive than `sql` alone which looks too generic to my taste, but `sa` would be effectively more aligned with what's used in all SQLAlchemy examples, good point :)


hey


Needed for minimal tests. Probably will be changed in future.


Revert before merging


eu acabei comentando sobre isso em outro lugar e por algum motivo não é o mesmo que esse.... o bug foi corrigido no sphere?


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


ok. vc sabe o que significa o TDATA3 e 4 no container?


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 157](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=795&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


LGTM


I will do ... strange that this wasn't detected


"(" too many. #fail


look


"formal"は形がちゃんとしている、ということなので、体系的、は確かに良いですね。いただきます。


Not need anymore.


{Number}


Hah - I think this project needs an .editorconfig :stuck_out_tongue:


Closes #4


this is going to happen all the time until the sed replace in release.sh is updated.


make a PR.


i will remove this ... we are not version dependent anymore
Yay


@RedX2501 I have come to the conclusion you're right, and how you've done it is probably the best compromise.

I ran with it reverted for a bit, but then the lowercase script doesn't like a new version database, etc, etc.

Let's stick with your approach :+1:


Hi Björn,

What was the reason to add these quotes here? We're having issues loading datasets using the "load" tool since the update to 0.18.1.0.

error message:
<pre>
job command line:
ln -s "/Shared/galaxy-data/database/files2/000/dataset_601.dat" input.vcf && bgzip -c input.vcf > input.vcf.gz && tabix -p vcf input.vcf.gz && gemini --annotation-dir "/home/galaxy/galaxy-dist/tool-data/gemini/data/hg19/2016-02-11/gemini/data" load -v input.vcf.gz -t "snpEff"         "" --cores ${GALAXY_SLOTS:-4}  "/Shared/galaxy-data/database/files2/000/dataset_662.dat"
-----------------------------------------------------------------------------
job stderr:
Fatal error: Exit code 2 ()
usage: gemini [-h] [-v] [--annotation-dir ANNOTATION_DIR]
             {examples,load,amend,load_chunk,merge_chunks,query,dump,region,stats,annotate,windower,db_info,comp_hets,pathways,lof_sieve,burden,interactions,lof_interactions,autosomal_recessive,autosomal_dominant,de_novo,mendel_errors,browser,set_somatic,actionable_mutations,update,roh,bcolz_index,fusions,gene_wise,qc}
             ...
gemini: error: unrecognized arguments: /Shared/galaxy-data/database/files2/000/dataset_662.dat
WARNING:galaxy.datatypes.binary:<galaxy.datatypes.binary.GeminiSQLite object at 0x7f588a453450>, set_meta Exception: no such table: version
</pre>

If you check the job command line you'll notice the $infostring argument is empty, but still quoted. I think the tool interprets this empty string as an argument, cancelling out the $outfile param.

Matthias


Yould we make it `pre-drupal-scallfold-cmd` and `post-drupal-scaffold-cmd`, for the unlikely event of another plugin doing the same.


This will never get triggered since `TARGET_OS_IPHONE` is also `1` for simulator targets.


This one seems to be in the correct order


será que o memory do summon não vai fazer falta para algum outro script?


Fix this one


So, if I understand correctly, you want to the `Request` class to have two methods:

 - `locales` returning the list of acceptable locales.
 - `locale` returning the first element of `locales` or default lang if empty.

The thing is, that `Accept-Language` does not contain a list of language tags, with witch we can create `Locale` instances, but a list of language ranges (e.g. `Accept-Language: "fr,en;q=0.7,de-*-DE;q=0.3"`). Here, `de-*-DE` matches all of the following tags:

- `de-DE` (German, as used in Germany)
- `de-Latn-DE` (Latin script)
- `de-Latn-DE-1996` (orthography of 1996)

but not `de-AT` (German, as used in Austria).

Thus it is not possible to simply extract a list of locales from the  `Accept-Language` header. But it is possible to negotiate the content of a list we provide. The hole algorithm is already implement in `Locale.LanguageRange.parse`, `Locale.filter` and `Locale.lookup`, so we only need to pass a list of acceptable locales. Since, for a defined application, one usually wants to support only a defined list of languages, they should be configure somewhere, hence my try to have two entries in the config file:

- The list of languages one wants to support (e.g. `["fr-CA", "fr-FR", "en", "de"]`)
- The default language, when `Accept-Language` is not provided or no match can be found

Of course, there could also be only the list, the first element being the default language...


xBlocks are installed as python site packages. need to include this directory in setup.py.
we do not need to think about `/venvs/edxapp/src `


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


这里利用到顺序栈是一种特殊的线性表，直接用前面的seqlist顺序表来实现，采用尾插法就可以


in general, we don't comment out code unless we know for sure that we want to keep it.  The commented out temp credentials test from before was only commented out because the expected failure decorator didn't work with one of the unit test harness.


This enhancement proposal is a bit lost here as commit comment, so, creating an issue here #5


é bom colocar o nome correspondente no lugar do numero para sabermos +- do que se trata


Looks good to me.


Fixed in 46562dc.


Ok, thanks for the explanation.


Approved


please delete instead of commenting


Just safe side, make the applying `str` to `string` object. I think, one of field in DnDv2 is sending boolean `True` here.


You should try to use `iconClassName`, e.g.:

```
    <IconButton
      iconClassName="muidocs-icon-custom-github" tooltip="bottom-right"
      tooltipPosition="bottom-right"
    />
```

We are generating font from the all SVG icons. You may need run some gulp/npm scripts first to update font file, also don't remember current prefix (Syncano is using `synicon-`, in this case is `icon-`  as far as I know).


@mattdrayer fyi


{Array<cdf.components.BaseComponent>} or {cdf.components.BaseComponent[]}


If we use tmp only in the if, tmp should be inside the if.


{cdf.components.BaseComponent}


o flag A | B está correto. deve ser o script que aplica o dano que está ignorando


I can't recall the details, but there was something about the key ordering that was important.  I believe checking for the key ordering was one of the design choices for this test, I just can't remember the details now.

Is this not doing the keys in a specific order?


we can use a simple queue_reader.join here as both threads and greenlets support it in the same way.


renamed


line 476: {Array\<cdf.components.BaseComponent\>} or {cdf.components.BaseComponent[]}


Hi, could be possible to merge this PR? We have the same problem....


console.log(player);


Thanks.


`moment-timezone` is already in `dependencies` – is this on purpose?


now im a little bit lost...
compile.sh is already on 7.0.2 in php-build-scripts
but the other can be changed to IM


`console.log(go)`


Nice. mp4box.flush is meant for when you know no more data will arrive. Haven't tested multiple flush.
To get the moov data you can do box.inputIsoFile.moov and so on, if you want to inspect the boxes yourself. Otherwise, box.getInfo() (or info in the onReady callback) will give you preprocessed information.


three methods for Request locale:
1) the two we already mentioned
2) and the one you added: ```locales(Collection<Locale>)```

but I won't add a new property: ```application.langs```, it will be just one: ```application.lang``` with supports for multiple locales using the tag format.

cool?


I just had a look and it currently only seems to work when downloading the comments ordered by time. So, I just removed the ordering option altogether (https://github.com/egbertbouman/youtube-comment-downloader/commit/9ea43c1fad22748d520fc3b9df5a98f8e1f9d569). Could you retry using the newer version?


Hmm, I don't know why it's not finding the libraries. Just be aware that you need to set LD_LIBRARY_PATH before you do any command involving libraries (ld, ldd, ldconfig, lddtree, ldd-recursive, etc.). Also:
- Did you try manually setting "qt_prefix="/usr/local/qt5"?
- What directories are inside "qt_prefix="/usr/local/qt5" (important ones are "bin", "lib", "plugins" and "qml").
- Do these (or similar) files exist?
  - /usr/local/qt5/lib/libQt5Core.so.5
  - /usr/local/qt5/plugins/platforms/libqxcb.so

I don't know much about cross-compiling, but I think you are correct about needing to copy the libraries for the Travis VM. Don't do it just yet though, best thing to do is to create a simple "Hello World" style C++ program and try to cross-compile it on Travis, just so we know how, then add a single library dependency and try again. Travis is free to enable for open source projects if you wanted to do this.


This was just my failure ... sorry fixed it and will update the TTS. Can you retest?


### Answers

> we have BoardMeta, BoardBase and Board. And the last one is actually the real base. Can we simplify the inheritance here?

There are actually two different inheritances here: `BoardBase <- Board` and `type <- BoardMeta`. `BoardMeta` is actually a metaclass - it means that the **object** `BoardBase` (a class is an object in Python) is of type `BoardMeta`.

In other words, every `BoardBase` subclass is an instance of `BoardMeta`, which means that each created subclass is initialized with `BoardMeta.__init__()`. That allows us to have automatic registration of the class in `_board_classes` when a board class is declared.

Using `BoardBase = BoardMeta('BoardBase', (object,), {})` is a workaround so that it works with both python2 and python3 (unfortunately). Version-isolated solutions would require only the class `Board`, without the need of `BoardBase`. The python2 solution would be setting the class property `__metaclass__ = BoardMeta` and the python3 one would be `class Board(metaclass=BoardMeta)`. None of them are compatible though.

Sorry if you know all that. Just wanted to make things clear.

> keeping a static _board_classes would do it (or class property, or classmethod called during init)

So, the point of using metaclass is automatic registration of the class when *the class is created*. If I understand it correctly, your suggestion is actually to register the class when it's instantiated, which would be pointless and make `get_board()` fail.


:ship:


Ah! Why do I think the hard way.


I don't want any ore or gem are generated. Why you change it?


It is not necessarily the URL, that determines the current page. Moving the check to PageObject makes it more flexible.


line 586: @ param {@ link cdf.dashboard.Dashboard#updateComponent|updateComponent}


This patch seems mostly ok for me,
but the problem is what the test don't test pg_nic_new_by_id anymore,
which may still be useful.
So I guess we should add a test to test the pg_nic_new_by_id before pushing there patches.


add command line parsing to trigger certain "jobs"


うまく意訳しているような気がして、読み飛ばしていたのですが、全然違いました。
client processesが接続を求めてやって来て(come)、それに対応するserver processesが(forkされて)出て行く(go)、みたいなイメージで、そういうことが起きている間も、マスターサーバプロセスは接続を待ち続ける、ってことだと思います。


No problem, this is why we're testing it ;)


e cade a correção? oauehaouehao


It seems perfect to me.


yeah, you got a number, convert it to string and then run a regex over it :P


line 618: {cdf.components.BaseComponent}


Yes those calls are thread safe. All work is deferred to the sessions
io_service.

On Thu, Feb 18, 2016 at 2:35 AM, yupuquiz <notifications@github.com> wrote:

> Hello,
>
> Just a question about integrating Autobahncpp into a multithreaded app.
> The session member functions (publish/call..) are thread-safe? How could
> be the library use when it is integrated in a multi-thread app?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/davidchappelle/AutobahnCpp/commit/eb3e2615c53218039c5f37a559d5eb334fa81b2e#commitcomment-16168633>
> .
>


@delpouite Attention pour la requete prospect le count n'est pas à -1, il faut donc gérer différemment, là la stack prospect ne marche plus du coup :/


@delapouite Attention pour la requete prospect le count n'est pas à -1, il faut donc gérer différemment, là la stack prospect ne marche plus du coup :/


Async ChunkRequest . Copy and Paste without copyright


line 645: @ param {cdf.components.BaseComponent}


Teszt


Awesome! Let me know what I screwed up as well .... is the datamanager working for you?


That automatism does not cater for templates and versions, that's why I'm not using that feature.


TestCustomCommand (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2110 of 2111 tests passed, 549 test warnings)

 * 550 tests had: AUD diffs.
 * 3 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.
 * 1 test had: ERR diffs.

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


for consistency it should be vertically aligned with if


TTS is updated. ...


This is what I suggested too, although I think it's not that easy:
https://github.com/biplovbhandari/eden/commit/8495b49034da2a2938988118ad48337e9dadbdce#commitcomment-16169566


yup


@wojtek-fliposports double return on line 107 :+1:


The config had old "world generation" list and the "general", "enchantments" lists were listed twice.
I regenerated the config using new Projectred version and changed the new "World Gen" list as suggested by the mod author:
https://github.com/MrTJP/ProjectRed/issues/989 since volcanoes and copper spawned with this old config.


Haven't had any issues so far, as a matter of fact, it's downloading the latests annotations right now.
Is it normal we have to redownload the annotation with every version? I can't image a lot of changes between both versions.


This also changes the color of all instance variables and references to `self` to white in ruby. Pretty jarring after you have gotten used to the old color.


`.jst_node .jst_node__string`
vs
`.jsontree_node .jsontree_node__string`


@do3cc shouldn't that have been a 5.0 release instead? The breakage that can result from updating 4.1.0 to 4.2.0 is not what one would expect.


setup tabs in ur msvs2015!


For instance `explain analyze select count(*) from tpch.sf1.lineitem group by returnflag;`


Not sure about this one ... but between 0.10 and 0.18 I would assume a few new annotation files.


line 27: If `dataAccessId` is optional, the dashboard will be the second parameter and in line 51 an exception will be raised when trying to access `getQuery` of undefined


@do3cc excellent work though!


pep8

https://www.python.org/dev/peps/pep-0008/#id41

`Always use self for the first argument to instance methods.`


TestCustomCommand (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


It runs at startup now. That is exactly what this PR is fixing.


Ok, done.


Thanks for keeping copying by Genisys -.-


Please let me know if everything is working, than I will update the TS.


Oops. I meant that this closes #7.


I tend to repeat dependencies used in tests to make sure tests aren't affected if for some reason I don't use `moment-timezone` anymore as normal dependency. So yes, this was on purpose.


I meant between 18.0.0 and 18.1.0, we downloaded the annotations last week when we installed the new tool from the TTS. Or could this be some .loc file I needed to update?


Cool!

Honestly? I did know about .forLanguageTag (Java 7) or LanguageRange (Java 8) :)

I will make a new release in the next 2 days.

Don't forget to star/follow the project at github/twitter and of course anything else you want to share will be appreciate


Meaning i will never get my shows updated since i update in a time frame smaller than 24h?


This is an abstract class, and other implementations out there (not part of jetty) have decided to follow the various updates to the NCSA log file format.

Jetty supports the original NCSA log file format, the one that most tooling around access logs also use.

We will log what is considered extended per the original format. (which just happens to not include any response details)

Standard NCSA format (non-extended), will still include response status code, and response length.

If you want Jetty to support the updated extended log definitions, consider submitting a pull request for Issue #113


Try to push this to upstream?


this fixed issue #10979


Ok, this is a technical issue should not happen in wild - it's just testing on TTS. This version was never on TS and so it will not happen.


fixed.. sorry :)

> On 18 Feb 2016, at 13:11, Juan Ignacio Fiorentino <notifications@github.com> wrote:
>
> @wojtek-fliposports <https://github.com/wojtek-fliposports> double return on line 107
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/wojtek-fliposports/django-oidc-provider/commit/d69828b8db29c7d6aeb32045c3b2f5ecf5c21f87#commitcomment-16174800>.
>


Det ser ok ut för mig!!!!!


Yep, pretty much... Have yet to achieve mastery.

-Anna

> On Feb 17, 2016, at 21:36, Brekmister <notifications@github.com> wrote:
>
> Congrats! First time?
>
> —
> Reply to this email directly or view it on GitHub.
>


I am yet to try this, but at a first glance, you're never committing any offsets when disabling autocommit - can't see any call to neither  `commitSync` nor `commitAsync`, so I think that the current behaviour is expected (as I said, I am yet to try this).


Good one
Feel free to add more context to add.
Also a Pull request template would be great as well


Can you make the test for .bash_sessions_disable be ```[[ ! -f "${HOME}/.bash_sessions_disable ]]``` ?

That better fits with the house style of the rest of RVM, even though your test is correct.


This change did not lose the response object, in fact it gained a more complete Response object for people that extend our RequestLog mechanism. (there are a number of libraries and groups that extend our RequestLog format)


I believe it should have been a 4.1.1 because it doesn't add new features. Before this, we got silent errors, after this, the errors are more explizit. I made a 4.2.0 only for making a 4.1.x branch, because I don't want to screw up buildout_coredev.
Looking at semver, maybe you are right. The exception handling can be seen as a breaking change of the public API.
I am undecided :-(


Yes, I tried manually setting qt_prefix to that, and the echo responds with qt_prefix: /usr/local/qt5

inside /usr/local/qt5: bin, doc, imports, include, lib, libexec, mkspecs, plugins, qml, translations.  And inside each of those dirs are what seem to be valid.

/usr/local/qt5/lib/libQt5Core.so.5 exists (as well as .so, .so.5.5, .so.5.5.1)
/usr/local/qt5/plugins/platforms/libqxcb.so exists.

I'm 100% sure I compiled Qt5, because I was able to build and execute MuseScore release.

One thing I'll try know is to do a make clean, and then redo make portable, with $LD_LIBRARY_PATH set first.


No problem, I'm restarting everything. I'll let you know if anything comes up!

M


@alecpm @vangheem you seem the two working on these packages. There's still a jenkins failure, a robot framework test, can we revert back and only merge once everything works?

I put enough effort on the pull request jenkins job so that we don't need to blindly put things on master or on checkouts.cfg without knowing if it will work or not.

I have at least 2 pull requests that I can not merged because the pull requests jobs are broken due to your commits on master.


"Play this sound as positioned" means that the only outcomes that make sense are either playing the sound as positioned, or not playing the sound at all and logging an error about why.


**HEAD commit has no message.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


Why did you add days to the expiry date?  Have a look at your Visa. It's exp is MM/YY, right?  That is what I used. No need to rewrite my code.


I do ask myself if this is the right way to go. If we overload all the methods of the channelmanager in the reporter what is the point of having a separate channelmanager and reporter?


How are these broken on OS X?  Is it something in the base library or our implementation?


In what way is the a better structure then the one you deleted?


I agree with the premise of DELAYED_MANUAL - In fact I think that both AUTO and MANUAL should have a batching strategy (much as what we did with the WindowingOffsetManager - by the way, we can reuse the notion of OffsetManagers somehow, since the client allows for pluggability). Probably we can call them BATCH_MANUAL  and we can provide a pluggable BatchingStrategy (although I cannot imagine anything beyond `every X messages` and/or `every X seconds`)


yes, 5.0 you are right. It doesn't matter if it is just error handling, it is a change in the public api, it is not backward compatible.
Some people had tests that there passing and now don't have it any more.


I know, the changes will we rewritten after I get it working. Also I will include the ini parser as submodule. This branch is a simple quick and dirty development branch, which I uploaded to allow people to help me finding problems.
But thanks for your feedback, I will do it.


Şuanda sadece nexus 5'te düzgün çalışıyor.


explicit


Done. Fixed by using random user/group names in every testcase


That being said, we should consider using the `Consumer` interface directly. Better opportunity for testing, and if we provide our own creation logic - I can see a few opportunities to decorate for thread-safety, provide additional behaviour, etc.


use imageSwapped to determine framerate. The texture upload should not be more frequent than imageSwap


It's kind of a weird solution to export the exception as placeholder for the (non-existent) backend class object and then test if they are callable. There's another issue though. In order to test for the theano backend, we're already trying to import theano at the top of the module (which of course would fail in that case). Also, handling the autograd backend as soft dependency needs some more attention. Say a user supplies both cost function and gradient explicitly. When using a first-order method, there's no need for automatic differentiation at all, but we'll raise an exception anyway cause the (in this case unnecessary) autograd backend can't be loaded.

Maybe we can abstract the backends even further by adding an `is_available` and an `is_compatible` method to the backends. Then we wouldn't have to import theano in problem.py at all. In autodiff/_theano.py, we handle ImportErrors gracefully, and just return `False` from `is_available` if we notice that the theano module couldn't be imported (by doing
```python
try:
    import theano
except ImportError:
    theano = None
```
The `is_compatible` method would in turn return a boolean if the backend understands the cost function and argument passed to the method.

Finally, we should add unit tests for this. It should be possible to fake a non-existent module by doing
```python
import sys
import autograd
sys.modules["autograd"] = None
```
The next `import autograd` line should then raise an ImportError.


`console.log(on va y arriver)`


aaa


Why did you delete the inventory sheep icon ?
The new stonemonster texture isn't added to the _stonemonster.lua_ file

I see the summerfields sheep texture is now the only texture used, that's great

i don't see pathfinding enabled for all mobs, is it normal ? All mobs need it, aggressive and passive, because it's a great improve for the follow feature for passive monsters, and aggressive melee monsters need it.
For monsters with throwing attacks, we need to try if it's correctly done...


Hi Frederico, are you missing a file? This isn't passing on my machine either :(


effects.remove(effect); => it.remove()


Any objection to adding an ESLint rule to prevent this from happening? This is silly. I'm sorry.


A slightly better solution is not creating `BoardBase`. We can do the following:
```python
Board = BoardMeta('Board', (object,), dict(Board.__dict__))
```

I'll do that here and send a new version of this PR.


Ok fixed


> Lets see what Travis has to say about it :).

Works in Python 3 too (phew :))


`triggeredBy`


`updateHTML`


> 772 additions and 6 deletions

 "Nothing important"


pop?


@jcheng5 I had to put in the following two ad-hoc resize handlers for dygraphs to get them to show up in ioslides and reveal.js slideshows:

https://github.com/rstudio/dygraphs/blob/master/inst/htmlwidgets/dygraphs.js#L131-L144

This is because for dygraphs when a graph is display:none it's width and height are 0 and these are not updated when the display goes visible. This might be a problem for other widgets worth solving at the framework level. What do you think?


Not sure where there's the best place to put the logic that's currently in **AccessibilityNodeInfoGetter**. Review and suggestions are very welcome.


Bad exception definition. Need to follow standard exception format


```javascript
if (!(key in data)) {
    console.error('Invalid key: ' + key);
}
```


I know this is not original with you, but passing `Mono<String>` to these static methods (for `organizationId`) is probably an anti-pattern; we should be getting the `organizationId` in the mainline stream and passing it in as a `String`—like we do `spaceId`. I notice this, and `getRoutes` are the only two static methods that do this.


Ok, I will add this


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 157](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=795&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:37:02


what happens when you are talking to an older client that doesn't send this?


Vamos lá, problemas:
* Em `menuManager.lua`, em diversas funções você usou `menu.curr_state` ao invés de `menuManager.curr_state`
* Em menu não seria "bom" você chamar a função do game para transitar estado, mas sim chamar uma função do menuManager que está logo acima na hierarquia, aí ele sim faria o que fosse necessário (nesse caso chamar a transição do game), mas ok, isso não interfere no funcionamento, logo não alterarei.
* no start do menuManager você dá start em **TODOS** os estados, o que está errado. Você só dá start quando ele for pra respectiva tela. Além disse, a função de start **NÃO** deve receber nPlayers. nPlayers é passado para gameManager a partir da playerSelection, não tem nada a ver com o start do menuManager.
* Para passar o nPlayers como parâmetro a partir do menu (ao chamar a função de transição), temos que adicionar o ... nas funções setState e goToGameManager.
* Na main, você retirou o game.load() e botou o menu.load(). Mas o load do game tem que ser feito! Se não os dados de player, imagem, etc, não vão ser carregados, e vai dar crash assim que um jogador for instanciado.


keep if update


I think it does make sense to seperate the Reporting and the Managing. It would make the amount of modules smaller if we rip out the Reporters completely. But that would mean we end up with a huge ChannelManager, which might make it harder to maintain. Also concerns would not really be seperated than. The current situation makes it very easy to implement a Reporter that is usable in most use-cases (easier than making an adapted channelmanager).


If in doubt ... hacks!


why not directly -march=silvermont ??
for refrence :
‘i686’
    When used with -march, the Pentium Pro instruction set is used, so the code runs on all i686 family chips. When used with -mtune, it has the same meaning as ‘generic’.

‘silvermont’
    Intel Silvermont CPU with 64-bit extensions, MOVBE, MMX, SSE, SSE2, SSE3, SSSE3, SSE4.1, SSE4.2, POPCNT, AES, PCLMUL and RDRND instruction set support.


AsyncTask 的来源是PocketMine项目下Anvil世界处理器的源码，加入SyncTask声明不是什么难事
我也没有copyright这是CS项目的，完全遵守GPL协议规范


done


Consertando erros do commit: https://github.com/LexLoki/RPG-Codinome-Brocolis/commit/681a04148ea28283ddbce18c10123a27b1da8f91


Using this code, the insert into my Google Spreadsheet is duplicated in each field. The submit URL doesn't show repeat fields, so not sure why this happening.


Need to check that after this failed, we actually indexed this properly


@theod Mhh, could the try... catch block stay in`threadCallback` ? Else, if a process throws an exception for some reason during the execution the software will crash (because the function is ran from a thread) instead of just stopping the execution.


I've checked and it doesn't change the output... the weird thing is that the tests passes if I debug in Firefox, but not on a normal run :cry:


Humm, there is alerady the brick name writed,
We can add a pg_side_to_string() function, but I would rated to that in another commit.


This comment is unneeded.  You don't use an intermediary calculation of `dimen@temp@one` in the following if statement.


I think this is needed for https://github.com/SeleniumHQ/selenium/issues/1627#issuecomment-183611709


Looks good


Why was this file removed here?


a


But the extended log did include the response object up until this commit: 396562 ?

I'm asking because we have an implementation of this abstract class that that broke when the response object was removed (api changed). The non-extended format changed to log(request, statuscode, responselength) and back to log(request, response), but the logExtended has permanently dropped the response.


Findwithtagをできる限り使おう


Thanks.

Wouldn't it be better to use "xytext"  to keep this working in older mpl. We can be certain that if xyann is not present then xytext will be present (unless in the future MPL removes xyann and xytext).  Add a coment in the line "box.xytext" to remind devs that this is for compatability.

```
if hasattr(box, 'xyann'):
   box.xyann = ...
else:
   box.xytext = ...  # Old MPL no exception here
```


Тимчасовий, злиється з update keywords


probably not, but future 440 students might appreciate it.


Hi!
Wouldn't it be nicer to write it like this: `import Video, {Controls, ...} from 'react-html5video'`?
I can provide PR if you ask


@vsjulu why you need the pad_after? per https://software.intel.com/en-us/articles/avoiding-and-identifying-false-sharing-among-threads, `align(64)` would suffice. and facebook guys are using `align(128)`[1], so i guess `align(128)` is a safe bet?

---
[1] https://github.com/facebook/folly/blob/master/folly/detail/CacheLocality.h#L132