@komagata
ありがとうございます！
`$(this).on 'submit', ->` を `$(@).on 'submit', ->` に変更して、問題なく動いていることを確認できましたので、 https://github.com/fjordllc/interns/commit/172f00154d60c5812116501fa69c38d47ebcf562 で push させて頂きました。


fixed


This is a good spot for the new build deck function. I should be able to make a slight change in the controller to call the appropriate deck (based on alex's user choice page).

In case you decide to reference the issue tracker with your commit messages, this is issue #16


Alright! That should be everything for Assignment2!
I think with everything, you can see that I'm still figuring out the best way to organize, (and code), but it's there..


av_expr_parseに任せたい。
クラッシュする？


jghljgllh


同様


Да. Вполне.


@zd2921

届くか


fixed in git, thanks !


そうすね、lengthが0だとクラッシュしてしまいます。
Evaluateも同様


Hi. Is LED blinking too for colors outside the basic RGB?


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.16](http://build.clear-measure.com/viewLog.html?buildId=4885&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


Actually, I missed lines 40 and 41 that you previously added to commands.h, and I couldn't figure out why you changed line 9 of main.c. Now I see that you changed that back and it makes more sense. Thanks!


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.16](http://build.clear-measure.com/viewLog.html?buildId=4885&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:54


I think the clean solution would be to factor the autoscaling functionality into a totally separate module. It would be a pure function, possibly configurable, that would take a dataset, perform some logic, then return a hopefully useful dimensions array to pass into parcoords. The `csvstat` function from Python's `csvkit` library is a good inspiration for what that might look like. This function could also be used for other chart types, like line charts, scatterplots or data tables with sparkcharts.


Нет. Мы не поможем этого пока сделать, поскольку программа должна работать поверх двухкомпонентных mwm-к. Т.е. поверх старых mwm-к. MapOptions используется и замены ей нет. Позже надо будет выпилить под корень.


Oops. Should e a defer here.


Ага, специально ломал. смотри GH-3


fixed


everything looks good. This should close issues #16 and #21


I don't think this have a top ten classification.


Remove - CSRF (only one category)


The other statuses say ....

Поправил


negative ~~offset~~ length


Думаю, что и в iOS тоже. Сам сделаешь (после мержа) или лучше мне?


:angry:


It is printed in run-build.ps1too. I needed some kind of logging to see why we were not producing release bits in VSO CI.


hello


Great❗❗  Thanks for adding EPShapes 💟 . awesome-swift is really awesome 😎


comment


Why not print it in the C# instead of in 2 places?


add a line note.


Sieht schon gut aus!!! Schön wäre, wenn wir das auch noch in den Ansichten teampersons uns matches bekommen würden ;-)


note to whoever, fedora needs something else to get omp to work


ah, right. Thanks for catching it. I'll make the change.


별다른 이유는 없습니다. 고쳐야 할 것 같습니다 ^^


Ah. Thanks.


This contributed to #22


Makes sense.


@antoviaque Is there not some way to just make any @opencraft.com addresses that don't match forwarding rules in this postfix layer so that they are passed on to the Google Apps transport? It would be nice to not have to maintain this list separately.


Done. The patch submitted.

Best Regards,
Xiaoxia Zhong

From: lizhong1008 [mailto:notifications@github.com]
Sent: Thursday, February 18, 2016 12:03 PM
To: ZhongXiaoxia/libyami
Cc: Zhong, Xiaoxia
Subject: Re: [libyami] h264enc: Fix the encoding failure issue for IBBP mode. (7a65af9)

m_reorderFrameList.size() is unsigned value. Would better change to if (m_reorderFrameList.size())

—
Reply to this email directly or view it on GitHub<https://github.com/ZhongXiaoxia/libyami/commit/7a65af9b6885963999bc56a446985460b754d0fb#commitcomment-16166563>.


Could you change assert statements in `flickEnd`? Because `500m` is not accurate timing


I think I will make my master same as openemr/master

git fetch openemr
git reset --hard openemr/master


Yup, it's required because the superclass FragmentPagerAdapter only has one public constructor that takes in a FragmentManager: https://developer.android.com/reference/android/support/v13/app/FragmentPagerAdapter.html#FragmentPagerAdapter(android.app.FragmentManager)


Test comment


can you attach the output sample?


Looks like it does. Tested it on my o3DS.


test comments


Need to change this to get_mnemonic not get_acronym.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/settings/ConfigurationGathering.py`, line 123.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/settings/ConfigurationGathering.py
+++ b/coalib/settings/ConfigurationGathering.py
@@ -120,7 +120,6 @@
         cli_sections["default"].add_or_create_setting(
             Setting("config", find_user_config(os.getcwd())))
         print("Tried to find config.", find_user_config(os.getcwd()))
-

     targets = []
     # We don't want to store targets argument back to file, thus remove it
```


Shouldn't we rename `MessageIds` to something meaningful given that we changed the behavior?


Thanks, I will update to ashwin's when I have my PC


This will let you immediately clone the repository to your computer. Skip this step if you’re importing an existing repository.


Did Syncplicity write this source code specially for Liferay Nativity?
Or was part of this source code copied from an existing Syncplicity-sponsored project?
If the latter, would you mind telling us where the original can be found?
Thanks a lot!


Thanks for your suggestion, I have added it into cmakefile.txt.


line 18 as well in RunNetflix.in


Like you mentioned when we met, please revert this to `www.tutorons.com`


in git, thanks !


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (912 of 912 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-25.83%25-green.svg)


i have to think a bit, here.


CSS: Centre align and min width #3


First draft as I explore architecture. Will look at more efficient gulp flow soon 🤔


This shouldn't be necessary---`dataType` describes the type of the return data from the query, and it doesn't look like you're doing anything with the return data.


This is customary for csharp-mode development. We bump the version number and update the change-log as we go. 0.8.13 is not ready yet, but you can see that there's already some changes compared to 0.8.12:

https://github.com/josteink/csharp-mode/blob/master/csharp-mode.el#L290-L293

This commit here was just boring "book-keeping".


This function return by value, *i.e.*, returns a copy of "std::set<RS_Undoable*>" object.


Can we make this a `$.post` instead of `$.ajax` to (1) reduce the lines of code (2) maintain consistency with how posts are made elsewhere in this code?


more love


You can check my init-company.el, it's based on latest stable version of company.  https://github.com/redguardtoo/emacs.d/blob/master/lisp/init-company.el


뭔가 압박하는 느낌이 나는데, 그런의도는 전혀 없어요.;;;

코드리뷰 잘하는 방법에 대해서도 우리 모임에서 한번이야기해보면 좋을 것 같아요.
저도 다른사람이 제 코드를 본다고 하면 숙제검사 받는듯한 부끄러움 나와서 좋아하지 않거든요. 서로 마음 상하지 않고 부담 안되면서 코드리뷰 할 수 있는 방법이 필요할 것 같아요.


Can we add a `dataType: 'json'` argument to this `$.post`?  That way we don't have to parse the json returned as a response in `addExplanation`---we can just use it right away like an object.


> but for me this is better than nothing

I don't agree. this entire code is crap. either something fits into a design or it doesn't. this doesn't, hence I call it crap :)


RIP `eval`

The method that will not be missed. :+1:


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/settings/ConfigurationGathering.py`, line 123.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/settings/ConfigurationGathering.py
+++ b/coalib/settings/ConfigurationGathering.py
@@ -120,7 +120,6 @@
         cli_sections["default"].add_or_create_setting(
             Setting("config", find_user_config(os.getcwd())))
         print("Tried to find config.", find_user_config(os.getcwd()))
-

     targets = []
     # We don't want to store targets argument back to file, thus remove it
```


Most worthless commit in the history of SpongeAPI.


For consistency, let's access properties on objects using dot notation instead of brackets.


Here's the meaning:
DC0-Shard0 = Datacenter 0 - shard 0
DC1-Shard2 = Datacenter 1 - shard 2
...
There are 9 shards :)
The IPs are random here cause there's nothing set up anywhere. It's just an example


* at const to correct constness
* considering to return a const&, to avoid copying of a set object, unless we have solid reason to use return-by-value

```C++
std::set<RS_Undoable*> const& RS_UndoCycle::getUndoables() const;
```


@gabriele-v @omalleypat @vomikan please have a review for this change to upgrade budget table from v1 to v2 to include NOTES


I'm noticing some black square appear on the floor instead of an obstacle. Is that on purpose? I see theres a black square randomly on a floor tile in the spritesheet so maybe that's it? idk. Everything else looks fine.


Let's stick to a consistent scheme of defining keys of objects---either consistently surround them in double quotes, single quotes, or leave them quote-less.


的确不是源文件，我后天就把这迁移到源文件上去。（后天有deadline，现在正在发疯的赶）


@riyapenn there is a typo here. It should be +376[0-9]{6} for Andorra. Thanks!


Yep. How else would we do it? It's a private member.
All I know is it stopped the crashing; and all undo testing I tried works correctly.


btw, which version of TDM do you have ?
Here, I have no problem with mingw-w64 provided with win-builds


System.out.println("sky")


maybe i can indded anyway adding those values in a examine_pe_common.h file


Shouldn't this be `log2(0) == -1`?


:clap:


Yes, we should match the effect as well, just ignore the value.


f-knorr <notifications@github.com> wrote:

> Before committing, I checked the English language. For this language no action is needed.
>
> Today, I have checked some French and Italian examples. In these languages the
> suggested fix would apply. However, I can hardly think of a scenario in which
> sentences can start with a newly added word... By the way, right now, these
> languages do not have any additional words in spelling.txt.

I don't see a need to spelling.txt in French. The French dictionary is very
well maintained at http://dicollecte.org/home.php?prj=fr
Word list and tagging is nearly perfect for French. The updates once,
in a while, are nowadays mostly for rare scientific words being added.
Those new words can wait for the next update dictionary.

Regards
Dominique


@DFurnes This is the magic sauce to get the `NativeAppEventEmitter` working within the React Components. https://github.com/facebook/react-native/issues/1049


땍!


Make sure to keep your concerns separate by handling them in different blocks of code---`tutoronsConn.addRegions` happens smack dab in the middle of preparing a response to the server.  What you should do is one and then the other, so that it's easier to keep track of what lines of code belong to one conceptual unit :)


I think this chance.js call sometimes throws an exception. Min somehow is larger than Max sometimes


If more `onresize` events are attached to the document these would also be triggered here. Do we really want this? Why not just call `@overflowTabs()` here?


Returning a reference to a private member doesn't make sense to me.


I expect that if you are posting with a contentType of `application/json`, you don't need to stringify, you can just pass the object as data to the `$.post` method.  Can you double-check this, and avoid stringifying if it's not necessary?


mobilde adresler şöyle oluyor: 'm/www/inferences


Totally, that seems obvious now. Will do.


m/www/inference/edit


Хотя можно вначале функции объявить
$config = cmsConfig::getInstance();
И заменить
cmsConfig::getInstance()->ctype_default
на
$config->ctype_default


please add kerker 2


howdy, is this making it into a release?


Your CR comments, plus whittled it down to a single failure channel for counting failures, and then one more separate channel to indicate that all requests failed. Thanks for your help on this one, @whyrusleeping!


Can we name this something like `showTime` to be more indicative of the event that this time records?


and `hideTime` (see above)


bu ayet notu ekleme için olan değişken. inference için ayrı değişken olmazsa hata oluşur


The URL here should also point to `www.tutorons.com`


大哥，我这个改对了吗？我是菜逼啊。。。

 @PinkaLulan

Great God, I changed this right? I was a novice


pink to blue,not good


Come to think of it, we probably don't need to `PATCH` the region record at all if we are storing all of the show and hide actions in their own table.  Could we make things simpler and remove this patch and the accompanying columns from the database table?


aman deyim, bu problem olabilir.
modalı kapat yeter.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/43//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (13)
src/plone/api/content.py:425:1: C901 'transition' is too complex (13)
src/plone/api/tests/test_doctests.py:21:1: F401 'transaction' imported but unused
src/plone/api/tests/test_doctests.py:119:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


bunun ayrı değişkeni olmalı. annotation=ayet notu


Can we log when a tooltip is shown as well as when it is hidden?


There is no need for this.  TWRP should automatically handle twrp.fstab. Infact theres no need to have seperate branch for TWRP unless we want to build CWM too.


bunun için de ayrı inference değişkeni olmalı.
Tanımlandığı yere bakıp yenisini ekleyebilirsin.


I've just noticed that this update is causing an javascript error (with multipoints in a map anyway):

TypeError: geojson.geometry is undefined
L.FieldStore<._serialize()
 leaflet.forms.js:38
L.FieldStore<.save()
 leaflet.forms.js:13
L.GeometryField<.onCreated()
 leaflet.forms.js:201
L.GeometryField<.addTo/<()
 leaflet.forms.js:139
o.Mixin.Events.fireEvent()
 leaflet.js:6
L.Draw.Feature<._fireCreatedEvent()
 leaflet.draw.js:9
L.Draw.Marker<._fireCreatedEvent()
 leaflet.draw.js:9
L.Draw.Marker<._onClick()
 leaflet.draw.js:9
o.Mixin.Events.fireEvent()
 leaflet.js:6
o.Marker<._onMouseClick()
 leaflet.js:7
o.DomEvent.addListener/s()
 leaflet.js:8
 leaflet.forms.js:38:13

Updating it to
        var is_geometrycollection = (geojson.geometry && geojson.geometry.type == 'GeometryCollection'); fixes it. I will submit a pull request.


@navis Thanks!!!


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/44//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (13)
src/plone/api/content.py:425:1: C901 'transition' is too complex (13)
src/plone/api/tests/test_doctests.py:118:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/45//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (13)
src/plone/api/content.py:425:1: C901 'transition' is too complex (13)
src/plone/api/tests/test_doctests.py:118:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Hi Matthew,

All of the functionality is here, which is great!  This commit includes substantial JavaScript additions, so let's clean it up to get it to a consistent style with the rest of the document.

In addition to the stylistic mentions throughout the file, I'll also mention that it would be great if variable and function names could be camelCased instead of underscore_notation.

Thanks for your awesome, hard work on logging!  Let's keep it up and get this code looking great.

Andrew


right now it is the way it is - expect lots of changes (just like in asp.net core as well)


Probably we can, but we also need to keep it for backwards compatible.


That's good! I'm looking it over right now.


Thank you for the explanation!

> When the header is at level 2, it shows up in the sidenav.

That's the reason, why @GeeH proposed level 3: #17 ("Headings in blockquotes should use ###")

> From a reader's perspective, a short sentence summarizing the note is far more likely to draw attention than the generic "Note".

I like this! :+1:


Is this feature being removed indefinitely?


Best practices: functions should return once. Better to use if/else then return $version


I'm still working through some of the probably self-imposed issues...but some updates from code might help especially OAuth entry:

https://github.com/geoffjentry/twitteR

"At this point you should have twitteR installed and can proceed:
library(twitteR)
**setup_twitter_oauth("API key", "API secret")**

The API key and API secret are from the Twitter app page above. This will lead you through httr's OAuth authentication process. I recommend you look at the man page for Token in httr for an explanation of how it handles caching."

Also from

http://docs.ggplot2.org/current/geom_bar.html

 "binwidth geom_bar no longer has a binwidth argument - if you use it you'll get an warning telling to you use geom_histogram instead."

I hope that helps anyone else...


TeamCity Chorus :: chorus-precise64-lfmerge Continuous [Build 2.5.6.8fbd94e0f82ee63be757ed3cf4afca23fb1584c9](http://build.palaso.org/viewLog.html?buildId=69300&buildTypeId=ChorusPrecise64lfmergeCont) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:28


Gradle outputs the current task :compileJava and :compileTestJava which ends up running right up against the first error's file path, which ruins the quickfix unless it's handled separately. I'm not certain if there's a better way to handle this, but this works.


This is the errorformat for all other java errors, it's lifted directly from the builtin javac errorformat.


OK, I will rebase and start new branch fresh.

Sorry I didn't respond earlier, but I did manage to build Qt5.5.1 natively (including the webkit components, which basically doubled the build time) on RPi 2 running Wheezy, and installed via sudo make install which put the qt libraries in /usr/local/qt5/lib.  I set $LD_LIBRARY_PATH to that, and then was able to build make release of MuseScore and that runs fine (side note: audio works via ALSA although I notice a faint constant static and quiet echo each note after is played, which might be an issue with the synthesizer...intesteringly if I export WAV and open in audacity I don't see any background static...I don't know if I should bother looking into, since I remember Raspbian Jessie did not have this audio playback issue).

BUT I'm running into problems make portable, since none of the qt libraries are being copied in copy-libs.  I even forced set qt_prefix="/usr/local/qt5/lib".  But says "cp: cannot stat `': No such file or directory" and the result portable AppDir/lib/qt5/lib is empty.  I can execute the mscore-portable binary, but has error "Unable to query physical screen size, defaulting to 100dpi. To override, set QT_QPA_EGLFS_PHYSICAL_WIDTH and QT_QPA_EGLFS_PHYSICAL_HEIGHT (in millimeters)." and hangs at start screen.  Now I am able to set those to something and use mscore, but the mouse cursor is duplicated at an offset position, and top menus except File and View are usuable (every other menu is a black rectangle).  I suspect this has something to do with not being able to copy the qt5 libs...for reference, I've attached the output of ldconfig -p, which does not show any of my qt5 libs:
[ldconfig_p_wheezy.txt](https://github.com/ericfont/MuseScore/files/135716/ldconfig_p_wheezy.txt)

Note to myself: I couldn't figure out why the qttools libs (which include things like QtDeclarative) were not copied to /usr/local/qt5/lib.  Eventually I gave up and just did a force copy via sudo cp * from the qt build lib directory to /usr/lib/arm-linux-gnueabihf but that didn't seem to help copy-libs find them.

I'm considering going back to *wheezy* which had qt5 binaries, since compiling my own qt5 on *jessie* is giving me such headaches.  I would very much like to know why ldconfig and hence copy-libs aren't noticing my self-compiled qt5 (especially considering that for testing purposes I'd like to be able to run the portable with different versions of qt5, since there always seem to be version-specific qt5 bugs).

On last thing, when building make portable on my armv7 RPi2 running Raspbian, I noticed the AppDir directory had a name with "armv7" in it, even though I'm fairly certain that Raspbian is entirely armv6 (since the same image runs on RPi 1 and RPi 2).  I think somewhere in the make portable code it is querying the CPU, when I think it should be querying the compiler instead.  For instance, if you want to make a 32-bit x86 AppImage using a 32-bit cross-compiler running on an x86-64 machine, then you would want the name of the AppDir to be named a 32-bit x86 after the cross compiler.  In fact I might even try to build the ARM binary on an x86-64 machine using cross compiler (e.g. with https://github.com/raspberrypi/tools/tree/master/arm-bcm2708) so the recipe would work on x86-64 machine, and so would want the folder to be named with armv6.


@marwahaha : No issue, just wanted to chime in to say that this looks really good

:+1:


I posted some diagrams and some use cases for just as a teaser for us to ignite discussion and gather ideas!  See you tomorrow


i like this grouping format. but i normally prefer to have my constant look different than a normal variable


:+1:

I need to do this in the app too


モデルではなくテーブルの情報を書いていきましょう


Productsテーブルではなく、Prototypesテーブルの方が良いかと思われます。


This rings a bell: @chumer you should probably change toString to be unusable, otherwise inventive users like @smarr are likely to parse the result of toString() to obtain information that is otherwise not accessible.

In case you guys believe parsing toString() is good idea (I don't), we have to cover the format of toString() by tests as it becomes part of the API.


TestCustomCommand (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (2078 of 2079 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)


I found my long-awaited dream spinner on polythene. I've been sticking it everywhere on lichess since.


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


Note that I'm not using polythene, I just scrapped the HTML/CSS:

https://github.com/ornicar/lila/blob/master/public/javascripts/big.js#L338
https://github.com/ornicar/lila/blob/master/public/stylesheets/common.css#L258-L284


Did you mean jarjarJar.output**s**.files[0] ?


Do not set `arguments` on top and then `+=` it here.
Cuz, when you do this the class variable is changed and that class when used next time will have this argument in it already


You need to add `{filename}` in this now


1. A pod has nil/empty tolerations , but a node has some taints, **should return false**
2. A pod has some tolerations, but a node has nil/empty taints, **should return true**
3. A pod has nil/empty tolerations , and a node has nil/empty taints, **should return true**


code review:
- use let instead of var. There is no need for the extended scope of var here
- declaration and reassignment of newState before '!state' -check is unnecessary
- long code blocks in conditional statements (like here in 'else') are hard to read

proposition:

`if (!state) return initialState;
let newState;
switch (action.type) {
...
`


Hi @taxido ! Could you please check this branch and let me know if it works well with Djrill now?
https://github.com/artemrizhov/django-mail-templated/tree/feature/cleanup


So fast o_o I'll have to wait till friday when I get my desktop back !


If you use `_process_corrected()` you can simply do

```
def _process_corrected(self, output, filename, file):
    try:
        output = json.loads("".join(output))
        newoutput = "".join(file)
        for lintLines in output[0]['messages']:
            # etc etc etc
```
which is much simpler


The settings for license should live in Sell Media -> Settings -> License tab. The tab should be displayed/registered IF there is an active extension installed. So you can get rid of all the customers settings page and options below.


WHY do you need a bare except ? Catch only specific exceptions


I still disagree that this is needed ... why don't you use `prepare_file` like hou you do with `config` above ?


You're actually testing the configs in all the tests - Does the linter **require** a config ? doesn't it have default values ?


* since *l* is a reference, *undoList.pop_back()* must be after all usage of l
* since *l* is still on *undoList*, *for(auto& cycle: undoList) cycle->removeUndoable(u);* will remove u from *l* as well, and this is a bug, causing *u* to become null
* a quick fix is to remove *u* from *cycle* only when * &l != &cycle*


Do we really need the `.tab` class on the `li` in order to make this work. Wouldn't `.tabs > li` also work?

Then shops with customizations won't have to update all their overrides just to add a `tab` class.

This would "just work"™ for them.

WDYT?


Could you minify if please ? :)


Nice... I think we will have a lot less crashes after this. :sunglasses:


importing a file to import a file... redundant much?


Really? That's sweet because polythene is now a dependency in lichobile as well, for tournaments tabs!


matter of taste: I generally move the NOT operator right in front of the term it negates


You should give direct links


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.588.8fbd94e0f82ee63be757ed3cf4afca23fb1584c9](http://build.palaso.org/viewLog.html?buildId=69301&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 932, ignored: 56 Build time: 00:19:06


new


I'm a little sad, semantically, that the `nav` tag is going away. Maybe do something like with @tvdeyen suggested: `<nav class="tab">`?


We do do something similar (in 1.3!).

But the thing about node modules in browserify is that they are intended to be used *the same* on client and server (thing utilities like underscore). They aren't intended to have different behavior on client/server (think things like livedata or a router)


那就是通過 boost 間接調用的吧，然後我在 mac 上編譯時 link 到 libiconv.dylib，而且 boost 實際上調用了 icu 組件（我用 MacPorts 上的 boost，應該是同時編譯了 iconv 和 icu 兩個依賴，但默認只開啓了 icu）。

Homebrew 的我沒試，好像是用的 icu，不用 iconv。同時編譯指南也沒指出潜在依賴。


You're right, of course. Sorry!


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1026.8fbd94e0f82ee63be757ed3cf4afca23fb1584c9](http://build.palaso.org/viewLog.html?buildId=69304&buildTypeId=ChorusPrecise64v25Cont) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:20


Correct the paths : `$DICTS_DIRECTORY/fuzzdb/fuzzdb-master/discovery/predictable-filepaths/filename-dirname-bruteforce`


hi


Hey Alissa, I found the solution to the integration test fails while googling. You can see my solution [here] (https://github.com/arolling/java-CD-organizer/commit/70167860dcee2c747724b6c1a1baff798f130cc6) - it turns out the HTMLUnit webdriver has javascript turned off by default and you can add "true" when creating it to turn it on.


I don't know how good polythene does as a JS lib, for I haven't used it. But stylistically, it's the bomb.


g


also fixed google play badge


Haha, that's hilarious (by that I mean it's not)...


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

```


On connection a conversation list is created, you can query this conversation list for the conversation ID's
some example code:
self._user_list, self._conv_list = (
            yield from hangups.build_user_conversation_list(self._hangups)
        )
convs = sorted(self._conv_list.get_all(), reverse=True, key=lambda c: c.last_modified)
for convss in convs:
            print(convss.id_) #conversation ID


fixed


cc @zmillman


editing the comment for integer condition


Nice


You are welcome Gil, in fact I am learning as I contribute, so feel free to yell at me if you see something you don't like :)


Explanation about the problem can be found in issue #17


Testing github comments


一个方便，通用的后台管理界面


a


Here


We could instead regenerate the CLI output, but then it'll be inconsistent across different versions (and what if it changes again in the future?)

Set everything without a defined ordering to `nil`!


嗯，似乎icu和iconv都可以。具體看boost是怎麼編譯的。所以這個應該是boost的依賴。


[這裏](http://www.boost.org/doc/libs/1_48_0/libs/locale/doc/html/building_boost_locale.html#bb_build_opts) 只說 `boost.locale.iconv` is off by default on Windows and Solaris。言下之意其他系統默認會開？


This is good. But this can be made shorter and simpler. Consider the prompt you were given, to **return** the match if you find it.


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1450 of 1450 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.39%25-green.svg)


Nice !


Would be cool if could somehow keep werr, maybe wed create a follow up issue to investigate further


說句閒話，把 librime 拆了吧？可以打發掉 Boost 和 glog 以外的所有依賴。


Great that the different options can now be controlled by Menu!
Really great work Ikari!


dfgdfgg


кастомная функция здесь избыточна, есть нативный метод isNaN


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.180.8fbd94e0f82ee63be757ed3cf4afca23fb1584c9](http://build.palaso.org/viewLog.html?buildId=69303&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 944, ignored: 50 Build time: 00:39:26


permutation with repetition (by recursion)


Oops, I accendately pushed this hack. This will disappear next time the wrapper generator is launched


Дим, страну нужно задавать в настроках виджета, мало ли из какой страны заходят посетители на сайт провайдера.


可以啊，類似於linux上面的情況。


Be careful the input case like below
 ex) -1 -2 -3 -4 -5


沒啥大問題，MacPorts 上除了 opencc 其它庫都有。


 b ,mnb b,h#

> [`_****_`](

-

- [ ] # @1. url

)


You probably have to cover the case of multilingual metadata too ?


What about development?


thank you for the amazing code.it helped me a lot to incorporate infusion soft in to my e-commerce cart.


Maybe not, it will probably work even if only using PT_FreeText.


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


@julienmalik fyi


Resolves issue #3


fix build : https://github.com/healthstone/RE/commit/70b14b9da46e2d7e17d9647f96099962292afc14


Yes, I think I agree, this was unnecessary.


Indeed, urlopen returns a file-like object that's *not* a context manager in 2.7. Haven't seen that one because I tested it with a file mock...
We should also close it after the import.


Thanks for the detailed description!


Is this required?


Oh and it seems like you can just use `with closing(urllib2.urlopen(...)) as stream:` in 2.7


delete a picture


Complex ternary opeartor:
Could be:
```
const isValid = validate !== true || this.validate(this.props);
```


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?


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


Missing line break at end of file.


No check for `validate` prop ?


If the target is already present, then why to add it again?


Create HelloWorld.md


ok


:heart:


I think we ought to keep this out of `src/leveldb` if this is a patch to bitcoin, could interfere with future subtree imports?


this commit was done by me


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


@johnsimons Well, I had a license issue on my local machine yesterday that I resolved by using the debugger and that resulted in these changes ;-)


@mathiasbynens, `copy` supposed to copy contents only


Did you test this? It looks okay.


I'm not sure the validation logic is only checking if it satisfies `required` constraint.


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


why not change ioctlCommandMap.command to int


I'll make an issue so that we can decide to add this or not.


Should this 'track by ....' also be applied on the disabled and ignored pages?


`zip_stat()` also supports `ZIP_FL_ENC_RAW`.
Yes, if you don't know the encoding, then you'll have to try twice.
Btw, do you know what tools creates these files with GB18030 file names inside?


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1027.3f7a9632ef07a76a63bbc4a7d74c953db82134be](http://build.palaso.org/viewLog.html?buildId=69306&buildTypeId=ChorusPrecise64v25Cont) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:36


Nice, didn't even know that this is possible.


@ferhung-mtk  , sir any success in fixing Wifi and Bluetooth ?


idも一応カラムなので足しておきましょう


Thanks for making all these formatting fixes and enforcing them with this new target!
One item I noticed in the previous commit is "}else{" without spaces. Can this be added to the style?


I guess you can't write-out the `http_code` here as long as it is not defined yet


@akbertram it's just build fix. Can anybody help with real translation ?


Do legacy clients cause any issues with this?


@yuriyz did you change the number of parameters? In that case it might be better to change a new key and then do ./gradlew i18n:push. That will update PoEditor with the new key and then when a translation is ready, we can update with ./gradlew i18n:pull.


一度本番環境のprotospaceのユーザー登録のフォームを確認しましょう。まだ足すべきカラムがあるはず。。


Thanks, I'll take a look. Still not really ready to switch my config over to `company` yet... I have trouble getting used to the popup appearing automatically. :-)


jade files house-cleaning


@ramonsmits Before you raise an issue take a look at https://github.com/Particular/ServiceControl/issues/629.  I thiink it'll be covered by that.


Do you have a test case for this? I'm not quite sure how a `@PolyAll` will interact with a type variable.
Why is the additional `@NonNull` necessary? The upper bound for T already is `@NonNull Object`.


テーブル名にアンダーバーは使わないようにしましょう。


> btw, which version of TDM do you have ?

TDM-gcc 5.1 from 28 June 2015.


typo気をつけましょ〜


This should be reverted, and the "@addtoapi" should removed from the backup_db in webserve.py. The reason i added this way was to override any messages.

They way is added atm the api will say its success even if it fails. And the format will be off.

```{"response":
               {"msg": "",
               "result": "success",
               "data": [
                    'message': 'Database backup failed.'

                    }
                ]
            }
```


belongs_to :userですよ！（わかってると思うけど。。。）


Variable declarations needs to be at the top of the scope


r == -1


実はtagは特殊なアソシエーションの組み方をするんですが、今はわからないと思うのでおいおい学んでいきましょう。とりあえず今はこれでOKです！


Hi,

You can also do that with the following steps(this will basically delete your current master branch and then replace it with the openemr/master branch):
git fetch openemr
git checkout openemr/master
git branch -d master
git checkout -b master

-brady


should be `if (pool == NULL)`?


You should still call consume_stop() for each consumed partition, and then queue_destroy()


Is this worth reporting to anaconda-issues?


Rephrase as "Multiple partitions are allowed in -C consumer mode"


olen ikuinen


mist saa hesekubonkei?


We use these classes in different modules/extensions so I would keep the platform specific error handling in the DPD_Core_SOAP.api.class.inc file.
eg: https://github.com/mvgucht/php-DPD_Core_Class/blob/master/DPD_Core_SOAP.api.class.inc#L46

```php
try {
    $this->Login = new DpdLogin($this->DelisID, $this->Password, $this->Url, $this->TimeLogging);
}
catch (Exception $e) {
    if(is_callable('watchdog')) {
        watchdog('DPD Exception', $e->getMessage(), WATCHDOG_EMERGENCY);
    } else {
        error_log($e->getMessage(), 'DPD Exception');
    }
}
```


int_v float test missing


Oops :)


Well for some reason, twrp.fstab does not get copied :/
Hence this hack


Sorry, happened by accident, should be moved to another place ... will update the patch


tagとのアソシエーションも忘れないで下さい〜


Looks like you misspelled "Canola"


I have a doubt about 039. This is not a registered marc field. Why not 375?


There is also a ForumCode.Variables which already defines some variables and which is included in some pages. Since the isNestedSpacesSupported could be handled just like other variables in XWiki (e.g: $hasEdit), maybe we don't need a macro but a variable. WDYT?


If returned name is a token, free should not be called. If returned name is not a token, then `name` should also be freed.


might be same as above.


I forgot to delete that one. It might be omitted in my latest push, not sure.


Much better, tests seem pretty powerful too 👍


yes I did with reset --hard and now my local master is identical to openemr/master :)

Next thing to do is upgrade Html2PDF correct?


empd (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2128 of 2132 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 4 tests had: EIO diffs.
 * 4 tests had: ESO big diffs.
 * 2 tests had: RDD diffs.
 * 4 tests had: Table big diffs.
 * 3 tests had: MTR big diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.
 * 2 tests had: ZSZ small diffs.

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


@Richard-Yunchao & @qkmiao please help to review it, thank you.
vec4Alignment should be 16, but deMath.INT32_SIZE is 4, so should multiply 4. The dEQP native code also multiplies 4.
This patch is related with deqp/functionals/gles3/uniformbuffers.html


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


This implementation detail affects [coding style issues](https://en.wikipedia.org/wiki/Programming_style "Selection of programming styles").
Is it required for the shown use case that [tail call elimination](https://en.wikipedia.org/wiki/Tail_call "Code transformation for tail calls?") will be applied?


That's perfect, thank you so much! I know, I felt so bad. :scream: by the way, in the future if you see unmerged PRs (that are just minor changes or picks) feel free to merge them. (you should be able to).


According to this document  https://wiki.codenvycorp.com/display/PSR/License+Key+Management   we have  another email for users who have a valid license.  I gues we should use an one e-mail for all places in UD(in che-footer too). WDYT?


TestCustomCommand (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2078 of 2079 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)


I did a "diff -ur" on the src/glibc(-2.21) folder after a "oe bake --rebuild -t patch" on this and the previous HEAD and there is no difference, if the .git and .pc folders (containing our patches) are ignored.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.589.3f7a9632ef07a76a63bbc4a7d74c953db82134be](http://build.palaso.org/viewLog.html?buildId=69309&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 932, ignored: 56 Build time: 00:18:32


empd (Myoldmopar) - 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 is about Uniform block members' layout, the set of rules is at OpenGL ES Spec 3.0.4 P70.


Form1
Private Sub Command1_Click()
Set db = New ADODB.Connection
db.CursorLocation = adUseClient
db.Open "PROVIDER=Microsoft.Jet.OLEDB.4.0; Data Source=" &App.Path& "\register.mdb"
Set rs = New ADODB.Recordset
rs.Open "Select*from register", db, adOpenStatic, adLockOptimistic
If rs.RecordCount> 0 Then
Do Until rs.EOF
If Text1.Text = rs!UserName And Text2.Text = rs!Password Then
MsgBox "COngratulations... Welcome To Our System!!!",vbInformation, "Prj Inventory System"
Set rs = Nothing
Unload Me
Form4.Show
ctr = 0
Exit Do
Else
rs.MoveNext
ctr = ctr + 1
End If
Loop
If ctr> 0 Then
Set rs = Nothing
MsgBox "Access Denied... You Cannot Access The Program..", vbExclamation, "Prj Inventory System"
With Text1
.Text = ""
.SetFocus
End With
Text2.Text = ""
ctr = 0
End If
End If
End Sub
End Sub
Private Sub Command1_Click()
If rs.RecordCount = 0 Then
MsgBox "Record Is Empty...",vbExclamation, "Prj Inventory System"
Else
rs.MoveFirst
Txt1.Text = rs!ProductID
Txt2.Text = rs!ProductName
Txt3.Text = rs!ProductQty
Txt4.Text = rs!ProductPrice
Text25.Text = rs!Supplier
Text35.Text = rs!Category
Text37.Text = rs!RetailPrice
Text38.Text = rs!DateEntered
Txt1.Enabled = True
Txt2.Enabled = True
Txt3.Enabled = True
Txt4.Enabled = True
Text25.Enabled = True
Text35.Enabled = True
Text38.Enabled = True
Command7.Enabled = True
Command6.Enabled = True
Command8.Enabled = True
Command4.Enabled = True
Command3.Enabled = True
End If
End Sub
End With
Text3.Text = rs!Firstname
Text6.Text = rs!Lastname
Text4.Text = rs!Middlename
Text11.Text = rs!Age
Text12.Text = rs!Gender
Text13.Text = rs!Status
Text14.Text = rs!Birthdate
Text15.Text = rs!Address
End Sub
Private Sub Command18_Click()
Set rs = New ADODB.Recordset
If Text3.Text = "" Then
MsgBox" The Search field is empty Pls... Fill In Thank You Very Much...",vbExclamation, "Prj Inventory System"
Text3.Enabled = True
Text3.SetFocus
Text3.Text = ""
Text6.Text = ""
Text4.Text = ""
Text11.Text = ""
Text12.Text = ""
Text13.Text = ""
Text14.Text = ""
Text15.Text = ""
Else
rs.Open " SELECT * From employee " & _
" WHERE (((employee.Firstname )='" & Me.Text3.Text & "')) ", db, adOpenStatic, adLockOptimistic
If rs.EOF = True Then
MsgBox "The User Number you entered does not exist!",vbExclamation, "Prj Inventory System"
Text3.Text = ""
Text3.SetFocus
Text3.Text = ""
Text6.Text = ""
Text4.Text = ""
Text11.Text = ""
Text12.Text = ""
Text13.Text = ""
Text14.Text = ""
Text15.Text = ""
Else
Text3.Text = rs!Firstname
Text6.Text = rs!Lastname
Text4.Text = rs!Middlename
Text11.Text = rs!Age
Text12.Text = rs!Gender
Text13.Text = rs!Status
Text14.Text = rs!Birthdate
Text15.Text = rs!Address
End If
End If
End Sub
Private Sub Command8_Click()
If Txt1.Text = "" Or Txt2.Text = "" Or Txt3.Text = "" Or Txt4.Text = "" Or Text25.Text = "" Or Text35.Text = "" Or Text38.Text = "" Then
MsgBox "Some Field Is Empty...",vbExclamation, "Prj Inventory System"
Else
rs!ProductID = Txt1.Text
rs!ProductName = Txt2.Text
rs!ProductQty = Txt3.Text
rs!ProductPrice = Txt4.Text
rs!Supplier = Text25.Text
rs!Category = Text35.Text
rs!RetailPrice = Text37.Text
rs!DateEntered = Text38.Text
MsgBox" Product Deleted!! Salamat...",vbInformation, "Prj Inventory System"
With rs
.Delete
.MoveNext
If .EOF Then
.MovePrevious
If .BOF Then
End If
End If
End With
Text1.Enabled = False
Text2.Enabled = False
Text3.Enabled = False
Text4.Enabled = False
Text25.Text = False
Text35.Text = False
Text38.Text = False
Command3.Enabled = False
Command4.Enabled = False
End If
End Sub
Private Sub EmployeeProfile_Click()
Frame1.Visible = False
Frame7.Visible = False
Frame4.Visible = True
Frame10.Visible = False
Frame13.Visible = False
Form5.Height = 9199
Set db = New ADODB.Connection
db.CursorLocation = adUseClient
db.Open "PROVIDER = Microsoft.Jet.OLEDB.4.0;Data Source=" &App.Path& "\register.mdb;"
Set rs = New ADODB.Recordset
rs.Open "Select*from employee", db, adOpenStatic, adLockOptimistic
Text3.SetFocus
Text3.Text = ""
Text6.Text = ""
Text4.Text = ""
Text11.Text = ""
Text12.Text = ""
Text13.Text = ""
Text14.Text = ""
Text15.Text = ""
Text3.Enabled = True
Text6.Enabled = True
Text4.Enabled = True
Text11.Enabled = True
Text12.Enabled = True
Text13.Enabled = True
Text14.Enabled = True
Text15.Enabled = True
End Sub
Private Sub UserAuthentication_Click()
Frame7.Visible = True
Frame1.Visible = False
Frame4.Visible = False
Frame10.Visible = False
Frame13.Visible = False
Form5.Height = 6500
Set db = New ADODB.Connection
db.CursorLocation = adUseClient
db.Open "PROVIDER = Microsoft.Jet.OLEDB.4.0;Data Source=" &App.Path& "\register.mdb;"
Set rs = New ADODB.Recordset
rs.Open "Select*from register", db, adOpenStatic, adLockOptimistic
Text10.Enabled = True
Text9.Enabled = True
Text8.Enabled = True
Text10.SetFocus
Text10.Text = ""
Text9.Text = ""
Text8.Text = ""
End Sub


Another option could be to collect ids during havesting and index in batch at the end like LFS harvester https://github.com/geonetwork/core-geonetwork/blob/develop/harvesters/src/main/java/org/fao/geonet/kernel/harvest/harvester/localfilesystem/LocalFilesystemHarvester.java#L158


You drop the flush here and not in updateMetadata, not sure if that could have side effects ?


I'll refactor that after tests.


Solved.


You are right, the flush should be still there.


@akbertram yes, I've changed number of parameters. Ok, I will change key and invoke that command.


Hi Ter,

I responded last night (UK Time) but it seems to be missing!

I had a look at that log file, large as it was, and it seems to be failing to parse line one of the checker test harness that I've been using since (almost) day 1 to test this grammar. Line one is a comment:

`// Import ANTLR's runtime libraries`

And it is showing that it doesn't like the two forward slashes at positions 0 and 1 in the line and subsequently, the text `import ANTLR`. Strange, Java compiles it ok.

I used the process outlined in the `README.md` file in the `examplesdoc` folder to test the new grammars in a sandbox, everything worked fine.

```
export ANTLR4="/home/norman/bin/antlr-4.4-complete.jar"
export CLASSPATH=".:${CLASSPATH}:${ANTLR4}"
alias antlr4='java org.antlr.v4.Tool'
alias grun='java org.antlr.v4.runtime.misc.TestRig'
alias jar='${JAVA_HOME}/bin/jar'
alias java='${JAVA_HOME}/bin/java'
alias javac='${JAVA_HOME}/bin/javac'

cd ~/SourceCode/git/grammars-g4/tnsnames
mkdir ~/sandbox
cp examples/* ~/sandbox/
cp *.g4 ~/sandbox
cd ~/sandbox

antlr4 *.g4
javac *.java
jar - cf tnsnames.jar *.class
rm *.class
java -cp "./tnsnames.jar:$CLASSPATH" tnsnames_checker < tnsnames.test.ora

Tnsnames Checker.
Using grammar defined in tnsnamesParser.g4.
Parsing ....
Done.
```

The only change to that file in this latest commit was to move it from `examplesdoc` to examples. I think `examplesdoc` came from the October change by Tom Everett to make automatic testing easier - it wasn't one of my original folders. I moved the test harness from `examplesdoc` to examples as Tom left it in the wrong place and it sort of broke the readme's instructions on testing the grammar.

HTH - if it doesn't, I don't know what else to suggest - we might need an ANTLR guru! ;-)

Cheers,
Norm.


I know it's not mandatory, but while looking at the code, found that inconsistent use of curly braces.

All other methods use the convention:
```
method_name()
{
    # Method body
}
```


That's why I added 'is_callable'.
When you throw an exception it's possible the entire website fails on this, resulting in internal server errors. This just because the API can't be reached or no shops are found.
When returning FALSE, and logging to the error log of other platform specific logs, the website can continue to work. In the case of Drupal it will just show the user 'No shops were found'.


Could this potentially be 0 on platforms with low resolution clocks and thus cause a div by zero below?


Please be careful with such dependencies.
A script to search for missing dependencies in setup.py will say that persistent is missing but not understand that ZODB3 is providing this package already.
Newer versions of zodb pulled persistent out as a dependency of its own, but the first version extracted from ZODB is incompatible with the ZODB version we use in plone.
Unfortunately, the only hint one gets is this:
```
ValueError: _p_serial must be an 8-character bytes array
```
That is quite hard to debug


Well it's not yet implemented anywhere other than CliGit, (JGit doesn't
seem to have an equivalent). I can investigate addition of a test for
force, it should be relatively simple.

On Wed, Feb 17, 2016 at 6:46 PM, Mark Waite <notifications@github.com>
wrote:

> I'd really like a test added which confirms the "--force" argument has the
> desired affect, especially on the CliGit implementation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jacob-keller/git-client-plugin/commit/b9bdafd926df8aa6af28b94d78097cc7273beab7#commitcomment-16165828>
> .
>


A warning is not breaking anything, it shows a warning (by default only on first occurrence, but configurable) on stdout. It is usual to show first the warning and then remove the function in the next major version.


Requires mopidy 2.0.0 now


Requires pyspotify-2.0.5


я так понимаю эти эксепшины пришлось добавить из-за ГуглАПИ.
как правило, не хорошо менять интерфейсы.
и в этом случае следует перехватить их внутри класса (походу это в методе translate()) и вызвать уже эксепшены именно нашего ТранслейтСервиса (они определены в папке exception).


En écrivant ce programme, j'ai vu que je devrai écrire 2 fois une partie de la formule mathématique pour calculer le prix à cause du if{}else{}.
Heureusement les mathématiques sont là.
Je ne sais pas si on y gagne en temps de calcul mais je pense qu'on y gagne en lisibilité. Et puis je trouve ça plus joli que ce gros camion de if{}else{}.
Tu te souviens que j'avais déjà fait la même chose en php et que cette méthode avait été qualifiée d'effet de bord par le professeur. Là, un deuxième professeur n'a pas utilisé ces termes. Il a dit que c'était astucieux et propre. Du coup je ne sais pas si c'est une pratique répandue ou pas. Si tu tombes sur des formules du même type utilisant des formules mathématiques plutôt que du vocabulaire algorithmique ça m'intéresse. À mon avis on doit pouvoir faire des choses assez amusantes comme ça.

Par contre je ne sais pas du tout à quoi ça peut servir ^_^ Peut-être économiser du temps, des calculs, aérer du code, éviter des formules lourdes.


check gemfile.


If i don't make error we can false before line 1348. Then we get here only if user are OK, but DB don't exist. Hard to find the way to have :
* test mysql version first : send the exception
* if yes : test user for this DB
* if not test user without DB


accept


Hi Ter,

ok, I have no idea what the "stuff" in the pom.xml file is for, other than automated testing by the look of it, but line 21 will probably need updating:

`<grammars>tnsnames.g4</grammars>`

That was the situation previously, now we have two separate grammars, `tnsnamesLexer.g4` and `tnsnamesParser.g4`.

Might that be what broke the build?

HTH

Cheers,
Norm.


Nice, I didn't know about this possibility.


The goal is not to load all the modules but only the one that are related to the flight plan you are using, so the user don't need to figure out which one he needs to include in its airframe.

Regarding the other point, not sure you gain a lot. You probably need to flash something at a point, and if it is only a part of it (some modules, a new flight plan) you probably loose the global optimization made by the compiler.


How should that be done ?


Please contact the mailing list in case of any questions.


:+1: Neat


Most of the commented out code came from rails @andela-bmakinwa


I think one could use the `None` value type to indicate that there is a missing value.


Would you like to construct such [a version identifier from other variables](https://cmake.org/cmake/help/v3.5/command/project.html "General project settings by CMake")?


Looks like you misspelled "Canola"


1) насколько я понимаю еще остались вопросы по 2-му пункту замечаний/пожеланий. или подумай: может быть вообще сделать как-то по-другому. но чтоб это другой решение удовлетворяло другие реализации транслейт-сервисов. без решения 2-го пункта у нас не примут задачу.
2) закомментированный код надо удалить - он уже всеравно сохранился в комитах, и если надо - он будет доступен.
3) кроме того, что сказал мне все нравится. Молодец быстро справился :)
4) подзадачами 2-го круга (итерации) будет: вынести значение ключа и имя EESN во внешний файл property


Most of the unnecessary new lines are from previous commits and they've been removed in more recent commits.


I saw that, but how can you check for version ?
Should the code be duplicated and specialized in a per-version read_battery function ?


The description you gave here makes it like the taints is filter/selector while we actually want the tolerations as filter/selector. To avoid the ambiguity, I suggest to change the comments like this:
```go
			// We reached here which indicates that no tolerations on the pod tolerates this taint.
			// So we need to return a false value here, indicating that this node isn't a fit for this pod.
```


just a test


Why ? if a switch case better ?


Thanks for the suggestion.  I have implemented it.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.696.b845273b61821245bf14957cf6abcd171d54114a](http://build.palaso.org/viewLog.html?buildId=69312&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:09:39


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.181.3f7a9632ef07a76a63bbc4a7d74c953db82134be](http://build.palaso.org/viewLog.html?buildId=69308&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 944, ignored: 50 Build time: 00:37:23


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.687.b845273b61821245bf14957cf6abcd171d54114a](http://build.palaso.org/viewLog.html?buildId=69311&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3427, ignored: 126 Build time: 00:10:42


May be need to add **android** here?


@nguyen-viet-thanh-trung Does it mean that there may be more than stbl->nb_sgpd_in_stbl sampleGroupsDescriptions? Same below for other_boxes?


没太想明白，相对于以前直接定时然后更改状态的做法，这样的优势是什么？


How do you think about to [support any more software package formats](https://cmake.org/Wiki/CMake:CPackPackageGenerators#Overall_usage_.28common_to_all_generators.29 "Selection of software package generation")?


@nabinhait, may I ask why is it required to have Sales Invoice items to have a warehouse specified?

Reason I ask is because I can no longer create a Sales Invoice Credit Note, it merely prompts "Warehouse is mandatory" and I can't seem to find a way around it.


The rename from Download Manager to Queue is my favorite change in this.


Now failing 40 / 76:

FAIL tests/filesys/base/syn-write
run: ((null)) open "stuff": FAILED
pass tests/userprog/args-none
pass tests/userprog/args-single
pass tests/userprog/args-multiple
pass tests/userprog/args-many
pass tests/userprog/args-dbl-space
pass tests/userprog/sc-bad-sp
pass tests/userprog/sc-bad-arg
pass tests/userprog/sc-boundary
pass tests/userprog/sc-boundary-2
pass tests/userprog/halt
pass tests/userprog/exit
pass tests/userprog/create-normal
FAIL tests/userprog/create-empty
FAIL tests/userprog/create-null
FAIL tests/userprog/create-bad-ptr
FAIL tests/userprog/create-long
FAIL tests/userprog/create-exists
pass tests/userprog/create-bound
pass tests/userprog/open-normal
pass tests/userprog/open-missing
pass tests/userprog/open-boundary
pass tests/userprog/open-empty
pass tests/userprog/open-null
pass tests/userprog/open-bad-ptr
pass tests/userprog/open-twice
pass tests/userprog/close-normal
pass tests/userprog/close-twice
pass tests/userprog/close-stdin
pass tests/userprog/close-stdout
pass tests/userprog/close-bad-fd
pass tests/userprog/read-normal
FAIL tests/userprog/read-bad-ptr
pass tests/userprog/read-boundary
pass tests/userprog/read-zero
FAIL tests/userprog/read-stdout
FAIL tests/userprog/read-bad-fd
FAIL tests/userprog/write-normal
FAIL tests/userprog/write-bad-ptr
pass tests/userprog/write-boundary
pass tests/userprog/write-zero
FAIL tests/userprog/write-stdin
FAIL tests/userprog/write-bad-fd
pass tests/userprog/exec-once
pass tests/userprog/exec-arg
pass tests/userprog/exec-multiple
pass tests/userprog/exec-missing
FAIL tests/userprog/exec-bad-ptr
FAIL tests/userprog/wait-simple
FAIL tests/userprog/wait-twice
FAIL tests/userprog/wait-killed
pass tests/userprog/wait-bad-pid
FAIL tests/userprog/multi-recurse
pass tests/userprog/multi-child-fd
FAIL tests/userprog/rox-simple
FAIL tests/userprog/rox-child
FAIL tests/userprog/rox-multichild
FAIL tests/userprog/bad-read
FAIL tests/userprog/bad-write
FAIL tests/userprog/bad-read2
FAIL tests/userprog/bad-write2
FAIL tests/userprog/bad-jump
FAIL tests/userprog/bad-jump2
FAIL tests/userprog/no-vm/multi-oom
FAIL tests/filesys/base/lg-create
FAIL tests/filesys/base/lg-full
FAIL tests/filesys/base/lg-random
FAIL tests/filesys/base/lg-seq-block
FAIL tests/filesys/base/lg-seq-random
FAIL tests/filesys/base/sm-create
FAIL tests/filesys/base/sm-full
FAIL tests/filesys/base/sm-random
FAIL tests/filesys/base/sm-seq-block
FAIL tests/filesys/base/sm-seq-random
FAIL tests/filesys/base/syn-read
FAIL tests/filesys/base/syn-remove
FAIL tests/filesys/base/syn-write
40 of 76 tests failed.
make: *** [check] Error 1


What I propose I'll do for a next update is to create a new error handling class so that we can add known platform specific error handling in that file. And change all the Excpetions to use that class/function. + return FALSE as you propose to keep things running.

Thanks for the feedback! It's really appreciated.


Test cases are forthcoming.

I think that `@PolyAll` should behave on a type variable like it does elsewhere:  conceptually there is one instantiation for each type in the type system.  Maybe that should be documented in the manual, but it didn't feel like a special case to me.

The additional `@NonNull` prevents `@PolyAll` from being interpreted as `@PolyNull`; it excludes the nullness type system from the scope of `@PolyAll`.


Thanks, good catch.

Other characters we might want to strip/replace:
* `?`
* `/`, `\`
* `:`

Anything else you can think about?
e.g. Unicode Symbols like German umlauts: `äÜöß` ?


Do you happen to have a Macbook? Other reported the same.
It needs some testing there because I don't have one..


This line would be better if it was wrapped as a comment within `<!-- -->`

Also as the template will displayed in edit mode (and many people will never preview) I think all links should be raw to make them easily copy-pasteable. Also preferably on their own line.


Need to add instructions for setting up database.


Thrilled to see this ;)
Let me see if GLSL shader will work now...


might be better to initialize them in the order of declaration?


np. then dont pick it ;)


hi @otisg, please review and merge


LGTM


Have you checked "Update Stock" option? In that case system asks for
warehouse, for a credit note, it should be unchecked.
On 18 Feb 2016 13:54, "Javier Wong" <notifications@github.com> wrote:

> @nabinhait <https://github.com/nabinhait>, may I ask why is it required
> to have Sales Invoice items to have a warehouse specified?
>
> Reason I ask is because I can no longer create a Sales Invoice Credit
> Note, it merely prompts "Warehouse is mandatory" and I can't seem to find a
> way around it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frappe/erpnext/commit/82cef59fb38f1567171cacc0cb315976864cd71a#commitcomment-16169298>
> .
>


mysql 数据字典


reverted


nn


Можно заменить DbSet на IDbSet. хуже не будет


Signed Off


저건 안해도 되는데... ㅡ.ㅡ;;


Is there anywhere to see an example use with promises? The docs seems outdated.


Can you modify the function definition to
```go
func TolerationsMatchTaints(tolerations []api.Toleration, taints []api.Taint) bool {
	// ...
}
```
I'm trying to make it more general so it would need less extra work when moving from here to some where else like `pkg/taints`.
Besides, I think better to put this function in `api/helper` if currently not putting it in `pkg/taints`.


Please remove the trailing comma which is causing errors, then I can merge into master.
Thanks!


This never made it into a 3.x build, did it?


Space.


Я никогда так не выделял память. new не вызывал.заботало.спросим потом это


`PodToleratesNodeTaints` would be a better function name.


This should be equivalent to `usbd.epBank0EnableTransferComplete(CDC_ENDPOINT_OUT)` but, I guess, `usbd` is not directly available in `Serial_`. This may means that we should handle this signalling outside `Serial_` (maybe inside `accept()`)? Don't know just a thought...


Why replace the protocol? Isn't this less flexible?


Fix bug sync database to entities jpa


Instead of just logging the error, I could throw an exception and handle it where this method is called. Let me know if it sounds better and I'll have a look.


ditto. IIRC,  gcc complains about wrong order in the initializer list.


empd (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (998 of 998 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.73%25-green.svg)


But as you may see, i do not write and do not read chinese.

> Am 18.02.2016 um 01:52 schrieb jasonczc <notifications@github.com>:
>
> @thebigsmileXD
> A Chinese phrase:别给脸不要脸
>
> —
> Reply to this email directly or view it on GitHub.
>


I think template list would be nice here, e.g.
```
### Steps to reproduce
- add your
- steps here
- as a list
- using hyphens
```


Yes good if we already have a place! And yes it's a variable and not a macro (my bad).


I'd make this non-configurable (cause we want it always there). Also, use the full path to the sock file here (and remove `/var`


For type variables there is a difference whether an annotation is present or not. Does `PolyAll` on a type variable mean that a qualifier for all hierarchies will be present?

`PolyXXX` qualifiers only make sense if they appear at least in a parameter and a return position. If it is used only in a parameter position, it is more clear to use the top qualifier.

In this signature, the type variable is already ranging over all other type systems and the NonNull on the upper bound only limits the Nullness type system.
A test case would help clarify what issue this is solving.


两个优势
1. web上能看到还有多久离开安全模式，不是盲等
2. 等不耐烦了，还可手工操作退出


`caclulate`? :D


Thanks @mezz


Why replace the function with a macro?


Should the installation destination become configurable?
Will [the software "CPack"](https://cmake.org/Wiki/CMake:CPackPackageGenerators#DEB_.28UNIX_only.29 "Where should this software be installed?") take care for this detail already?


First of all, If you wanna blame sth. Plz open an issue, make a list and point out the plagiarisms you said. You should show and compare the code. Or use other way to show some evidences instead of balabalabalabala in several commit. It looks like you type a lot of words ,including some stupid word like “Fxxx Yxx”, to show ITX team stealing. In fact your words are meaningless without any logic. And your behavior just like a kid.
Second. Do you know the word "From" from the title of this commit? Form your comment i think you should buy a dictionary to learn sth. Some developers from IM team are also contributed to Genisys (Thanks a lot). Do you mean they are also stealing?
Third. Do you read "Genisys' README.md"? Do you know there is a part called "Acknowledgements". We get the permission to get the code's copy and show our acknowledgements. You say Genisys copy "everything!!!" and still "keep stealing". Do you mean that although we have permission, some Genisys' works are still "Stealing"? Then i suggest you to ues dictionary again.
Fourth, You say "Genisys Stealing Everything From ClearSky". Emmmm......Genisys has Reastone and ClearSky also has. So Genisys steal? Genisys add colored sheeps and ClearSky add colored sheeps in another way at the same time. So Genisys steal? wow what a rude guy.
Fifth. Because of you, i go to homepage of your term and read the introduction in README.md to avoid make some mistake or say some silly word. No acknowledgements,emmmmm... a really good team with all original code. Oh. You will rewrite from copy. wow! "About 20 times faster than offical PM repo" Does this mean that if my server can support 10 player using offical PM, then if I use ClearSky, about 200 players can join one time. I think i should have a test sometime, hope it will not let me down.
BTW, I wanna say that sometime your words on behave of your team. I hope that other members in your team do not like to say "Fxxx Yxx" or say sth really irrational.

Mainly like this.@dog194


Perhaps a tip here would be helpful:
`<!-- You can drag and drop images here -->`


I noticed that Shiv has fixed this issue.


Я имел в виду, что "The" не нужен. Ну ладно, совсем не критично


Actually i suggest we kill this branch and fix #629  - the extra logging is planned for that fix


Hey Chema,

There is something I'm missing here. This is the line which fails in Firefox, but what I don't understand is why did it work in the other browsers :)
We create the first editor with empty add toolbar, and we never simulate an event, which would trigger rendering of the Toolbar Add. Why then we expect button add (ae-button-add) to be rendered?


IME, Midje is much nicer to use, especially when things get complicated.

take a look https://github.com/marick/Midje


不过现在想想，好像没必要，略傻。。。


should return false.
(Shiv has fixed this)


please fix tests that put files here instead of blindly ignoring them


@flavour 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.

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


can we move `j` into the `for` loop? as it is not used elsewhere?


This is an interesting idea, but far more than what we expect from "microprojects" IMHO. Microprojects are done before applying, by each applicant, at a time when their school keeps them busy. Most past microprojects were just ~1-3 patches of ~10 lines each.

I think this, plus "whitelist/blacklist" could give a real project idea.


@thebigsmileXD I would be grateful if you would summarize everything and compare the code to show that plagiarism is present.

And no, don't think me as for any side for prejudice.

From @PEMapModder


I'm just working on st2contrib stuff and I noticed this broke pack tests :/

```bash
ERROR: test_get_build_num (test_get_build_number_action.GetBuildNumberActionTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "<string>", line 3, in wrapper
  File "/home/travis/build/StackStorm/st2contrib/packs/circle_ci/tests/test_get_build_number_action.py", line 50, in test_get_build_num
    vcs_revision='dhjhvjVv635r6735', project='area51')
  File "/home/travis/build/StackStorm/st2contrib/packs/circle_ci/actions/get_build_number.py", line 16, in run
    extra_headers={'limit': int(search_limit)}
  File "/home/travis/build/StackStorm/st2contrib/packs/circle_ci/actions/lib/action.py", line 34, in _perform_request
    self.logger.debug('URL: %s', url)
AttributeError: 'NoneType' object has no attribute 'debug
```


这里将“文字”去掉直接说“[Admitted]和[admit]……”感觉会好些


I mean do not make your face shame.
Do not make yourself look like a kid.


that's true.


Feel free to put a note on your desk :-P


Yes, sounds reasonable. I will switch this and adjust the export (https://github.com/rism-t3/marcxml-tools/issues/2).


I am using it strictly for debugging.


I don't think this commit should be in here at all.   I feel that the Update steps each represent a point in history of the development of the database format, and should **never be changed once they've been released**.   In order to safely allow the upgrade to run on any installation which has **any** previous version of the software installed, you can only ever add to the end of the list of updates.

The new add-id-column code should be run after all the existing upgrade steps have completed.


It's used in AbstractPool#flush to shutdown and remove mcp when all listeners are removed after flush


Not sure why you introduced a test list here - I think there is still some gap in understanding OOAD.


I think `config.fetch(:delimiter, ",")` would do the same thing?


I wont start typing issues on the Phone 😃


This PR actually includes the changes submitted in PR#1


See https://github.com/Plexxi/st2/pull/6 for details


@samueltardieu
Because it's new: https://github.com/blog/2111-issue-and-pull-request-templates


You forgot to add the test into the test_order.


Take a look [here](https://github.com/owtf/owtf/blob/develop/framework/config/config.py#L125). while doing -f `target_urls` will be empty hence `load_works` function will not add the work.


looks great


Hey Iliyan, that's a good question :)

https://github.com/liferay/alloy-editor/blob/master/src/ui/react/test/util/utils.js#L65

We do focus the `nativeEditor` after creation, that's why I assume the `ae-button-add` should be rendered... makes sense? Maybe that is not enough for Firefox to trigger the rendering?


Declare variable closet to use - other wise it leads to bad code - you need read "Clean Code" - http://amzn.to/1VpepRz . Strongly recommend reading this book.


Could we use these ones above individually as micro projects?


I like this.


Bad - code here - listNode != null, the order of check is critical in an if condition evaluation. If listNode is null this will throw an Null pointer exception


Reporters should implement both of these methods. If you make this change, README.rst has to be rewritten.


Why not rename this to get_sql_session_factory? Pyramid already has a session factory concept for request.session.

Note that I explicitly tend to use **sql** in names like that, since there are a lot of non-SQL databases as well that are commonly used (MongoDB for example).


looks like a bug fix!


Can you make the struct for `podTolerateTaintsTests` to
```go
	podTolerateTaintsTests := []struct {
		node	*api.Node
		pod       *api.Pod
		scheduled bool
		test      string
	}{
	// ...
	}
```
so that we can have tests to verify the permutation and combination of taints on node, tolerations on pod, length of taints/tolerations, operator, value and effects.


Great. I would also suggest to change "m" and "w" to "male" and "female". Also there are still a few non registered field (e.g., 549?), at least in the tag config. They should be dropped, I think


Got some more ideas from the lastest failed KML upload (https://github.com/mapbox/unpacker/issues/900):

```
NZ Post Postcode Boundaries (June 2011; licensed only for Mix & Mash 2011)
```


If you only need the basetestcase and silentreporter, please only move these to project dir. Tests should not be in project.  In this scenario, all tests would be installed to site-packages if you install the package.


Can't we use the "latest" lintr ?
I don't like hardcoding `0.3.3`


Yes! I have tested this, with the previous code Deleteall() function is actually not deleting plugin output from the database hence showing the `Integrity Error: Key (plugin_key, target_id)=(semi_passive@OWTF-IG-002, 10) already exists.`


This should be called R commands


Ah, yes, that would look much nicer. Nice catch. I'll change it!


After writing your code you need to dry run your code - to find bugs. Can you find the bug in your code.


Why are you asking me?


Could you add r-base to `$deps` as we won't be having any more R deps to install through apt-get anyway
We'll always be installing it using `Rscript`. Hence, a new variable is unnecessary


what re you doing??


Unnecessary line


You should know that I have not read the code about this and sheep in CS. I catch packet, analysis and write the code. I have TODO first, you don't.


Yes, changing from "m" to male etc. would be better, I will prepare this in
the export.

Concerning non registered fields I will have a look, thanks a lot.

2016-02-18 9:50 GMT+01:00 Laurent Pugin <notifications@github.com>:

> Great. I would also suggest to change "m" and "w" to "male" and "female".
> Also there are still a few non registered field (e.g., 549?), at least in
> the tag config. They should be dropped, I think
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rism-ch/muscat/commit/1993466687501a9a0cb05c5afac63d51cd13581f#commitcomment-16169719>
> .
>

--

Répertoire International des Sources Musicales
Stephan Hirsch
IT-Manager
Zentralredaktion
Sophienstraße 26
D-60487 Frankfurt am Main
Tel: +49 (0) 69 70 62 31
Fax: +49 (0) 69 70 60 26
http://www.rism.info


@sils1297 wants better names for these.
Anyway, why do we need `1` when there is only 1 testcase ? can you rename to `RLintBearTest`


@mleroi According to this function (https://github.com/WordPress/WordPress/blob/master/wp-admin/includes/network.php#L334), get_home_path() could not return the "real" htaccess path, I guess with some weird windows installs. We may have to c/p this piece of code here. What do you think?


if you remove the `enumerable_type2` template argument, you get a compiler error because `typename enumerable_type::compare_type` does not exist in most cases. However, if you make this a template argument, the SFINAE principle applies.


The function signature is add(int value ) -


我无聊会偶尔打打服务器(斜眼)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1307.789112345247b71a0872ea65a0ac7a7406d229e0](http://build.palaso.org/viewLog.html?buildId=69327&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1003, ignored: 12 Build time: 00:08:56


Em làm phần admin thêm, sửa, xóa course, subject


I considered, and am still open to, `get_dbsession_factory` as the final apis injected are `registry['dbsession_factory']` and `request.dbsession`. Thoughts?


can we simply remove this?


Node addition is always at the beginning not at the end . this is very important  as it make add o(1) complex.


You need to override the `check_prerequisite()` function here to ensure that the user has `lintr` installed.

Currently, `Lint` just does `which ("Rscript")` and if it returns an error we know R is not installed.
But, even if I have R installed, i may not have `lintr` installed. So, you need to also check for that.

Check `CheckstyleBear` i wrote a very similar thing there yesterday


zhy,帮我看看这个饥饿游戏怎么样呗
Link:https://github.com/keithkfng/HungerGames


this could lead to a null pointer exception


For summoning spells you can just use the base game units if you want(because in some cases they have their own base class so its hard to recreate them)


Will give a wrong answer if head == null.


你最好问问作者


i think that you think youself so important that you think something is a bit similar is stealing yours. what you said is really ridiculous.at the end,i want we all should think more before do one thing.


I've been using the Pico platform to develop a fun in-house CTF for my workplace for a while now. I wrote this simple little script to take some of the hassle and formatting out of writing basic (non-autogen) questions. I (maybe optimistically) that if anyone else comes along to use this platform they may find it useful.


I would use request.sql_session as well. I tend to already have something like request.mongodb for Mongo-using sites, and the db-naming would be confusing if you use both.

FWIW I am thinking of a set of changing to pyramid to add requset.sql_session and have a way to support multiple engines and metadata instances. I'll probably get to that next week.


这是个fork版- -算了，我还是开issue问吧...仿制的真Tx像Lifeboat


画像だけコミットしたつもりが…。画像の項で、EPUBとブラウザの話をちょっと切り分けました。


我爽朗的笑了起来


Note that I'm not making hard promises on that pyramid_sqlalchemy thing - since multiple engines or metadata instances are only used by <1% of apps I would rather not support that at all than complicate the API for normal users.


@Efforto 笑什么笑，严肃一点。


@appleguy Awesome! Have had to use a workaround for this bug (up until now 😄), didn't know how to fix it, glad you got a fix in.


@CrayonDing 噫，那让我滑稽的拍桌子吧#Funny


You have some typos there "Port", "Proxy"


:+1: so far looks so good. This afternoon we'll try to work out those 422 errors.

Also we prefer to use `superagent` as a class like in their docs http://visionmedia.github.io/superagent/ so for example a test would look like:

```javascript
it("should remove 'myapp' from repository", function (done) {
     agent.del(URL + '/apps/myapp').end(function (err, resp, body) {
        should.not.exist(err)
        expect(resp.statusCode).to.equal(204)
        done()
    }
}
```

To be honest I did not know that you could use it like that! Let's try to use the library as on the snippet above.


Why is this hardcoded?


@CrayonDing 我曾经以为他是个正常人，没想到还真是个小傻逼...


Wouldn't it be nicer if you used something like buzz for handling http calls? This also makes it easier to mock. (no more curl_init and curl_setopt stuff in your code)


why do you put this in an array? I'd just return the Payload object.


Just a question : do we need really utf8mb4 on all DB ? Think only
* Reponse table
* Question text table (language string)
* ANswre language table
* Survey language table

All other can be leave at utf8 ?


Hi Jean,

Thanks for all the PRs. I think this change would be better as two methods, since they are two different calls.

Could you refactor the idLabels call to add_labels_by_id and then just take the label id instead?

Cheers,
Plish


prototypesとtagsの間には中間テーブルがあるはずだからhas many:tagsじゃないですよー


I'm also interested in reasoning behind this change. I tried to find it by checking git log, but I didn't have much success.

The comment only tells "where" the variable is initialized, but not "why" which is what I'm interested in.

In any case (if there is no specific reason for this change), I would prefer to move initialization of those two classes back to the constructor to make the tests work again.

In general, I'm not a fan of this late initialization since it basically makes it really hard to understand the code and see when the variable is or should be assigned. Either of those two approaches is better:

1. Pass those arguments to the `Action` class constructor.
2. Introduce "explicit" initialization interface (e.g. add `initialize()` method to `Action` class)

Both of those approaches would require to to re-think and change current Action unit testing framework so moving initialization back to the constructor is my preferred approach.


编码时空格要转换成 `%20`

所以此处代码应该为 `ps[i].Value = strings.Replace(url.QueryEscape(str), "+", "%20", -1)`

ref: https://docs.qingcloud.com/api/common/signature.html


qwq我只是来打酱油的...


@willdean in a sense you are right but i checked all the other scripts to make sure they still work with this new format. Further this script only creates these tables if they do not exist, which for existing repositories makes no difference at all.

I felt it was silly to create the old format just to have it updated a few steps further down...


Ok, I'll test it once more and then if it doesn't break anything, I'll merge it. :)


Impede bardos lancem magias e meditem ao mesmo, nao gastando mana.


各位，切记不可骂人


Pretty darn tempting to rename it to `request.sql` and the like... `request.sql.query(Page)` kinda rolls off the tongue.


No code changes in this commit; just ran Source->Format from Netbeans. Done because my actual changes were getting buried in a massive diff of reformatting anyway - at least now that noise is in a separate commit.


@davekellam  :thumbsup:


This can be duck-typed ("hasattr(meshgrid)"), I'll fix that.


+1 looks fine to me


There is a bug there, missing a \ to change the line of the string


Certainly very intuitive. :+1:


Closes #20.


I really don't like having lots of initializer code (especially for third party libs) in the engine class file. Does putting this into `api/config/initializers/versioncake.rb` work?


我的天，瞬间清理...这种时候还是表现素质吧...


Reeds fixed in https://github.com/bitsoflove/User/blob/f89441f3d257e5624497671395c8dca0852480cc/Database/Migrations/2015_12_16_194249_add_user_role_relations.php


哈？瞬间清理了什么？


prototypesテーブルのアソシエーションも考えましょう〜


The get_checklists_json doesn't currently have an option for appending the id to the end, where as the fetch json using the base uri does. Granted this is not ideal, especially if endpoints change.


like++


dedicated is just one use case of the taints-tolerations, here, you should use words related to taints/tolerations instead of dedicated


Thanks @MckinstryJ , sure will work on it.
Not much aware with site..just started ,if something goes wrong please guide me :)


I actually like this, because it is always a misery for most users why actionless views in rails work.
Now it's obvious that this action view exists.


I think this will fix the date issue that I mentioned here.

https://github.com/gintsgints/sequelize/pull/6


Corrigindo bug das flags e características nao atualizarem pro caster qnd sai do polymorph, e sim pra outro player que esteja perto no momento.


I don't understand the purpose of this commit.  The code modified in this commit has been already been disabled by the ifdef 0 from the previous commit.


Try not to commit files like myapp.tar.gz, LICENSE.txt and so :stuck_out_tongue_winking_eye:

Looks interesting I was not thinking of doing things this way, but rather to do it like the front-end does.

If you make an HTTP POST to <dashboard ip><port>/api/<app-name> the dashboard will do anything else for you.

Using the superagent library as @cayrodmed1 is doing with mocha tests...

```
request.del('/api/apps/' + appName).end((err, res) => {
   // app is uninstalled
})

request.del('/api/activities/' + name).end((err, res) => {
   // app is removed
})
```

Take a look on file: https://github.com/netbeast/dashboard/blob/master/public/components/apps/app.jsx


@rbouqueau when we add a segment, we merge traf->sgpd (respectively traf->other_boxes) to stbl->sgpd (respectively stbl->other_boxes). They should be removed from stbl when we release this segment.


This one can be either a list of microprojects (each item can be one) or a full project. If you keep it on the microproject page, I think you should make this clear. If you move this to the real project page, then you should try to sell the project better. As-is, reading the list may feel like "here's a list of painful things to do, do as many of them as possible" and it may not attract students a lot. OTOH, continuous integration did a lot of progresses the last few years, Travis CI is a hell of a good tool and working with all this can be really exciting.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.182.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=69321&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:49:11


for Bras d'Or Lakes Scenic Drive


Yeah, I prefer this to the 3 buttons. I think we should use the word "Insert" rather than "Import" for the button and text before the dropdown though. Also, probably better to make the previous behaviour (replacing), the default for the dropdown.

One other thing to consider. Which I came across when working with translations for the row creation dialog. Some languages have very different word ordering. So translating something like "Import before current layout", where we're forcing a specific order (Import || before || current layout), might be impossible.

One solution might be to only have the dropdown and let users figure it out from there. So the 3 options could be:

* Replace current
* Insert before
* Insert after

That way we'd get around any potential formatting issues with translation. Plus only having the dropdown and the button might look a little neater.


Fixed! Remove comment


Hi @carljm . Everything looks good to me and this commit is awesome :+1: !


all of a sudden started getting error for these 2 module . `module not found` . So added it . If you don't face any error without these 2 plugins then we can remove it for now , and Later I can investigate why I was getting this error


Since it is an example I chose the (slightly) more verbose version :)


ok thanks!


third version bbb


Removing the ` sorted it. They were only there as they are definitely showing on your link
https://github.com/samaaron/supercollider/blob/ed1956adc73f5596c821f50c16b88c544e804fd9/README_OS_X.md <https://github.com/samaaron/supercollider/blob/ed1956adc73f5596c821f50c16b88c544e804fd9/README_OS_X.md>
Ok on the pull request. I’ll take a look at that, and comment on its desirability

Now moved on to my next problem see issue #1001 Never ending fun :-)

> On 17 Feb 2016, at 22:37, Rainer Schütz <notifications@github.com> wrote:
>
> These little backticks -DSNDFILE_INCLUDE_DIR=/usr/local/include -DSNDFILE_LIBRARY=/usr/local/lib/libsndfile.dylib look a bit weird, did you really use them? With cmake path arguments you only need to use string delimiters if you have a space in the path. For me installing libsndfile used to work without problems, although I admit those -D..... arguments are far from fun.
>
> There is a pull request for sc3.7 atm that should allow to detect a standard homebrew install of libsndfile automatically. Worked for me. Would be great if you took the time to try it. More positive feedback makes more probable that it will be committed for the 3.7 release.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/samaaron/sonic-pi/commit/bf87e14d1c8cff87d379b759d7e784f221d27f3f#commitcomment-16161763>.
>


Why options and imports are at the bottom?


Maybe we could reconsider the usefulness of the GET_NULL vs. user-friendliness. We could store the number of null results in report (we would lose separate stats, but maybe these are not that necessary - they would peak out in histogram).


Our messages crossed. Yes, I think it can.

I'm not merging the PR for now to give you an opportunity to take my comments into account, but just say if you want me to merge as-is.


Cool, I'll add a note.


Por qué no pruebas a poner aquí el timeout en vez de mandar eventos vacios? Y en el AppControllerView, lees la store y la pones a false acto seguido. Puede que al no quitar la class aquí al repintar se repita la animación, aunque no consigo hacerlo funcionar


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.184.5ecbecf5b29f079a6d9f1b1d45ff12635e0726a2](http://build.palaso.org/viewLog.html?buildId=69339&buildTypeId=bt2) outcome was **FAILURE**
Summary: Compilation error: src\LibChorus\LibChorus.csproj (new) Build time: 00:02:14


Ok Fine good to go


pylint actually detected a breaking change in the latest version (0.0.6).


zxcvgxsdf


Yes - it's either something like that or send some kind of message to the current top view. Not sure what's best approach. Probably something that isn't view specific - there are two main situations I can see - failed to load db at startup and errors retrieving or storing new data.


At the end, what we should do is to stop arguing and start coding.


`get_tm_sqlsession`, `get_sqlsession_factory`, `request.sql`, `registry['sqlsession_factory']`

Yay? Nay?


This reason seems in favor for an other extension than `.src.whl` since pip (and certainly other tools) currently tests for `os.path.splitext('something.src.whl')[1] == '.whl'` to detect a wheel so maybe `.swhl` would be better suited ?

Even if pip would not end up using them since `any.src` is unlikely to be a supported platform tag for any system.


Can this now be removed? You seem to have removed the need for this to be assigned here?


Either write javadoc completely and document params or don't add javadoc at all to private methods.


@PEMapModder Yes. And I am going to do my homework.


Looks like the indentation changed from spaces to tabs, making it very hard to see the real differences. Could you change back to spaces and force-push the amended commit?


Endre e-post signatur


IMO it is easier to read and understand an if/else than a one-liner. I like the original version better.


Yes, indeed, missed that.


In this function should be compared with twosum, threesum, foursum algorithms.


ack


Ace


Please add a check for `nmlvcd.IIconPhase == 0` in this if-clause to avoid a "funny" value for nmlvcd.Nmcd.DwItemSpec.


testing my first commit


I know what you mean about the inefficiency, but it's only once on the first run of the site.

The reason I have raised this is that my new tests build a database (in a sqlite memory context) before every test.  So they run all the upgrades from scratch before each test.  As things are in this branch at the moment (i.e. including this commit), they fail with some strange problem in the Repository table (it reports 0xffffffff rows, which causes an OutOfMemory during the table copy in AddGuidColumn)

If I put the update steps back to how they were before this branch, so that the updates run as the would have done, as a complete sequence, then the tests run OK.

I vote for purity rather than worrying about the very slight inefficiency when first building a database.


@championswimmer <https://github.com/championswimmer> I've invited you to
our org so when you accept the invitation we can assign you to issues,
otherwise it might happen that somebody else grabs your issue unknowing
that you're working on it. The assigned person always gets to do the issue
in case of conflict.

2016-02-18 6:32 GMT+01:00 AbdealiJK <notifications@github.com>:

> @sils1297 <https://github.com/sils1297> also, he needs to be added to the
> org.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/a40ec58ccb1a00d9ce8a6a30863ed1fdd8f4d779#commitcomment-16167384>
> .
>


Gosh thank you so much for cleaning this up. Hardly can believe I wrote such a shitty error message.
:cake:


`request.sql` has my vote.


https://github.com/biolab/orange3/blob/master/Orange/data/filter.py#L181

Not sure if it is used anywhere yet though.


This version of the 'power' function shows that you can safely use the the value passed to it, as it's going to be changed only internally. It is a copy of the original value.
Can be used in the powerArray.c


Indeed, that is disabled code, changed to be in pair with the new memory management interface defines that print module name.

The whole code has to be removed actually, probably I left if there when I did the new dynamic mem manager interface just in case it needs to be reverted, but it is clear now is not going to happen.


@samuelkraft It's the same API. Just that it returns a promise rather than taking success and error callbacks. A PR will be awesome to update the docs.


merging test2 into MASTER just for renaming


I have no idea why are you making the priority calculation so complex, what I got from the design is:

  1. We need only take `TaintEffectPreferNoSchedule` into consideration in priority, while others are processed in predicates.
  2. The variants of `NoSchedule` implies no preference in priority.

Or, am I missing any thing?


This'll render an empty `<ul>...</ul>` if the user has no custom queries, or set the number to display to `0`


retirado o antigo corretor do bug de dupe (que nao funcionava anyway)


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


Retirado o antigo fix do bug de dupe


0777? do I want this? surely at a max 0775


Because input like `last_name,first_name` doesn't get split with preg_split and same pattern.


Ok, then I am wrong, actually, I misread it to be the shutdown time.

My idea was to return all entries which take place during or after the given timestamp, then subtract an offset if the first entry has started before (timestamp - start time of first entry).

Should it be 'where btime + uptime > %i' ?


Don't ask me why, but order of this two lines matter in jRuby (they cause some tests to fail). In MRI it doesn't matter the order.


With Java 8, we could create OperationStats.IDENTITY that would merge with all other OperationStats (producing the other) and the ugly for-cycles with branching like that could be rewritten into
`entry.getValue().stream().reduce(OperationStats.IDENTITY, (s1, s2) -> s1.with(s2))`
where `with()` is a default interface method
```
OperationStats os = s1.copy();
os.merge(s2);
return os;
```


I'm not suer if that's the right way to do that. There might be better was to convert a string to a strem. Maybe @d00rman can comment on that.


extract PlanNodeId


:+1:  for Chronix


that's mkdir's default already, and it's affected by the current umask()