May want to try this in the editor of the readme - because sizes seem to change based on the width of the page


Also, we don't necessarily need to mention "Scrutinizer code quality" - as the badge already says "code quality".

Also, is "branch" in "branch coverage" important ?


It's already a high-order function (since it takes a function as a argument), but I agree that the currying style in the interface is cleaner. I'll fix that.


first commit
such a big commitment
haiku
but really what is this doing now


We have almost the same kernel. I do not believe in lt03wifi no this problem.


Although I agree with @tombentley that a file system based on `LocalStorage` might not be a perfect fit I do think it's a pity that those interfaces are sealed and we won't be able to provide our own implementations.
Now I understand that this is done so you can go creating your own `File`s and `Path`s willy-nilly, that they should be created by the proper `System` implementation, but `sealed` seems too heavy-handed in this case.


Use camel case.


Fix Coding Style.


sorry - but one more thing to say. The heading for this section seems inappropriate now - maybe "Project status" or "Code status" seems more appropriate than "Build status" ?


I'm not convinced by the table, spacing issues apply obviously dependent on where it's shown and all the URLs need to be in one single line which is hard to edit.

[![Linux Build Status](https://img.shields.io/circleci/project/coala-analyzer/coala/master.svg?label=Linux Build)](https://circleci.com/gh/coala-analyzer/coala)
[![Windows Build status](https://img.shields.io/appveyor/ci/sils1297/coala/master.svg?label=Windows Build)](https://ci.appveyor.com/project/sils1297/coala/branch/master)
[![Codacy Code Quality](https://img.shields.io/codacy/f0ac979fa93f49509cba9086754a50d4.svg?label=Quality (Codacy))](https://www.codacy.com/app/lasse/coala)
[![Scrutinizer Code Quality](https://img.shields.io/scrutinizer/g/coala-analyzer/coala.svg?label=Quality (Scrutinizer))](https://scrutinizer-ci.com/g/coala-analyzer/coala/?branch=master)
[![codecov.io](https://img.shields.io/codecov/c/github/coala-analyzer/coala/master.svg?label=Branch Coverage)](https://codecov.io/github/coala-analyzer/coala?branch=master)
[![Documentation Status](https://readthedocs.org/projects/coala/badge/?version=latest)](https://readthedocs.org/projects/coala/?badge=latest)


It's too long, it should be less than 120 in Java.


Warn isn't a PSR level so shouldn't be allowed.


TAG is empty?


alpha sort.


Ahh sim Roberto, vou dar uma olhada nesse schema, ajudo sim, até porque eu fiquei quebrando a cabeça nesse simples diretório errado porque não estava validando meu NFe. Pode aguardar pelos meus pull requests.


File name is not correct


@Souldrinker They don't actually return any data, so I'm not sure what the type parameters should be. Should it be `Promise<void>`? Is that valid?


It is preferable to have an empty line at the end of the file.


Commenting is not present


Well, I didn't say this doesn't happen on lt03wifi.
Why don't you just fix it and link the commit here?
I'm bothered with more important issues right now, mobicore isn't useful to me at all.


Valeu !!!!


Maybe we could just use json_encode on text values ? It could be easier than loading a new smarty modifier.


same as above


add comments it applies globally throughout your file


how will this actually change the edit?


Add `typedef` to all these enums as well as the one in group_moderation.h. These are now global variable definitions, which is most likely not what you want.


I'm not against.
I'm not sure which way to go.
Thank you. I'll try to find a solution


97647908-DocIssues4615 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1240 of 1240 tests passed)

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


bad naming conventions


This causes a BUG!
as when I run it again in a notebook it silently does not perform all add_x_quicklookup !!


alpha sort.


Commands:
git rm -r --cached FolderName
git commit -m "Removed folder from repository"
git push origin master


same as above
try to use logical table naming convention


@jean-philippe-bousselin Please post a patch on d.o


extra spaces.


The reason I didn't bother with a try-with-resources in this case was that by spec Statements are required to close open ResultSets.  What do you think?


Error: command rm didn't work


What does `[<Measure>]` buy you?


This is redone in #4122


usage of comments
/* */  for multi lines
and
-- for single line comments

follow them from next time


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 577](http://build.prxm.it:8811/viewLog.html?buildId=3497&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 290 Build time: 00:14:14


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 578](http://build.prxm.it:8811/viewLog.html?buildId=3498&buildTypeId=Jarvis_DocumentStore_CI) is now running


aa


Thank you for the excellent response. I'll watch the projects carefully for a release. These, and KnpGuard will make Oauth handling (at least, the logging in with, and collecting initial data) far easier.


Good catch, thanks.


The var level is not initialized anymore


np :wink: glad to help!


First fix to do, is to replace add_qty by float(add_qty) in my previous
commit
    add_qty is a string, so when it is multiply, that return 0 and set
your qty_uom to 0

What you are doing will not work !
What you want to update in the checkout is your field "aantal"... but
_cart_update will update uom_qty...

I'm in holidays until augustus

Good luck for the end !

JKE


Should this be `Backward`? Nothing huge, `Back` for some reason is implying "to where I was before"


I'll try to answer any questions that I can :-)

On Thu, Jul 16, 2015 at 1:58 AM, prebbz <notifications@github.com> wrote:

> Hi Steven,
> This is a very interesting project, currently I am working on designing a
> Bluetooth board which will speak CBus to my Clarion head unit, and wonder
> if you are still active to answer some questions re: the design
>
> Kind regards,
> -Marcus
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/srwalter/cbus-emulator/commit/cc46c5dac43213ac6b56ee9a8943f4d759519623#commitcomment-12195679>
> .
>

--
-Steven Walter <stevenrwalter@gmail.com>


Wow, I never thought of it like this. That's so awesome!


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


uhm yeah.


can this be wired into the `tc`'s stopper?


Please commit in smaller chunks, for easier rollback/review!


test comment


test comment2


Dude, awesome!


good initiative to move this to its own module!


Haha, why an abomination?


I'm not sure what I was thinking here, now it seems this should simply be `Pointer => "c_void",`.


use commenting standards as suggested in previous file


@JKE-be thanks for your comment I will play around with this. Thanks a lot for all the feedback and help man!
Enjoy your vacation! :+1: :smiley:


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 578](http://build.prxm.it:8811/viewLog.html?buildId=3498&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 240 Build time: 00:08:49


spelling mistake


This is the core idea.


te rog modifica aici


comma separated selectors: each in a separate line ...


Fixed in b64abb91928b5bf1205b9018efe8cddf361ea802.


Do the changes in _this_ file also affect other projects?


Localization?


why not <f:link ...?


same as above


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


localization


fluid ..


Fluid?


I cannot beleive i never saw the issue on my side .... I really do not understand! Maybe it was browser specific, as I only tested on OSX/Safari ?


у грида есть кажется опция, которая задаёт текст, когда нечего рендерить


It's ok for fps, but for time information in general, double precision could be needed.


add one empty line.


Remove please


alpha sort.


**If** this is meant to be in line with the `crm` provider and the definition given in the type, by the way, then the order of this needs to be swapped. The type seems to want to redefine the order in which primitives are specified.


Too long.


This change breaks the control for me in Chrome.
I'm clicking to one of the suggestions, but the change will be reverted by this line, so writeToModelIfNeeded at the end will not do anything, thinking the value stayed the same...


айдишники кстати не выводят обычно нигде. если только этот айдишник не в каком-то особенном формате. типа VIN для машин. тогда он и смысл несёт, и как ключ в базе может использоваться


space after the comma "hidden", "shown"


This may confuse some, as it sounds like it's creating two separate indexes, when it's really a shorthand form of

   compoundIndex(ascending("quantity"), ascending("totalAmount")


```if (!encoded_chain_ref || !private_key_ref)```


удаляй таблицу эту, назад пути не будет)


should also add a note about the implementation lives inside core\lua\MixinUtility.lua and core\lua\MixinDispatcherBuilder.lua


Similar to above.  It's a compound index, not one index followed by another.


Types of Comments:

Single Line: //
Multi Line: /* */

Use single line comments if the comment can be accomadated in single line


Add newline!


isn't it equivalent to PR #4908 ?


hi dear i have 5 class of animals images how can i classify them with HOG EXTRACT FEATURE AND SVM


indent.


You're right, I didn't remember it! I'll reread both :)


у тебя открывается редактирование детали по клику на модели оО? элементы редактирования должны быть собраны в последней колонке. и здорово не словами писать "Редактировать" или "Удалить", а раздобыть иконок)


Comment during code commit should be meaningful. it should explain what is being committed.


Could you add a positive test for

      private $x = null; /* x is null by default */

And negative test for

     private $y = null; /** invalid doc block y is null by default */


@hjwsm1989 why you define another `name` hiding the one at https://github.com/hjwsm1989/ceph/commit/852b08fa0492b1208aba2c7ff4b283b7e61311df#diff-74b5f48f5ff89ca1eaa06f2cce3e9477R2110 ? they are identical, i believe.


+1

Will add a folder to hold all the images


Here you can flatten the list with `.toList()` and have your subscription receive only one element. Also, you can call `notifyDataSetChanged` in the `subscribe`, and do that only if indeed the list is non-empty.


Ohne jetzt *der* Algebra-Spezialist zu sein hätte ich entweder die alte Variante

> *Relation* von Objekten auf Firmen

oder die Variante

> *Funktion* von Objekten auf Potenzmenge aller Firmen

gewählt. Die hier eingeführte Variante scheint mir nicht stimmig. So wäre doch auch eine Zuordnung von mehreren Teilmengen von *C* zu einem *o* aus *O* denkbar, oder?

Ich lasse mich gern belehren, falls ich da falsch liege. :smiley:


@samccone are we going to keep the old commits and not squash after review? This is removed on last commit.


I'd rather not. We call this only from those tests that leave transactions running. If we did this on every `Stop`, we could wind up obscuring heartbeat goroutines which are never shut down.


You change is good but considering multiline match arms style, may look better as
```rust
Pointer => {
    match inner  {
        "void" => "c_void",
        _ => return Ok(inner),
    }
}
```
Generally, I should factor out the "fast track plain type" path and use here. Some day.


I think the tests should be fixed in another way :)
We definetely want a unique id on the page!


тело if - в фигурные скобки


Yes, but it doesn't make a difference for them.


чего temp-то?) sparePart


You should set the scheduler onto the result of the `Func1` inside `flatMap`. That would allow every flatmapping call to be done in parallel on network scheduler threads, rather than sequentially on a single network scheduler thread.


I was wondering what kind of bug it was and why it was solved by lowering the road class from 4 to 3? Do you remember?


four spaces is not identical with others.


OK didn't notice that. Will revert and fix the test otherwise.


Also, it would mean that you no longer need to act on stream termination, and you can just use `subscribe(Func1)` instead of `doOnNext/doOnTerminate/subscribe`.


remove this line.


this will actually pose a problem to us... it means that these resolvers have to be added to the deployment of `ensime-server` itself since we use `sbt` for that (excluding many users behind corporate firewalls... myself included). Is this stuff available in maven central?


ah so! thanks. Google has a stun server as well, do you thing this one could be more reliable?
Or may be you could download a list a try the first, the second on fail, etc?

Here is a list: https://gist.github.com/zziuni/3741933

Thank you.


ack


it could be better readable if its
`rake populate:references_and_external_flag_to_attachments`


This can be enough:
```diff
diff --git a/src/ploneintranet/microblog/tests/test_tiles.py b/src/ploneintranet/microblog/tests/test_tiles.py
index 57b6d56..3368002 100644
--- a/src/ploneintranet/microblog/tests/test_tiles.py
+++ b/src/ploneintranet/microblog/tests/test_tiles.py
@@ -55,7 +55,7 @@ class TestSetup(unittest.TestCase):
         tile.update()
         self.assertEqual(tile.post, None)
         # check if we render correctly
-        self.assertIn('div id="microblog"', tile())
+        self.assertIn('div id="microblog', tile())

     def test_newpostbox_tile_submission_on_portal(self):
         ''' This will test the existence of the newpostbox.tile
```


alpha sort.


Ok, #4908 commented upon, feel free to merge, it looks just fine!


Ok, yes, I don't get any complaints from the compiler when I changed to Promise<void>, so that seems to work.


Too long.


reack


Please remove the unused import.


extra spaces


extras spaces.


ditto.


extra spaces.


Good github, jak to hezky zobrazuje


Give me a few minutes and I'll get it out :)


Кстати, про категорию - надо её сделать числом, точнее enum-ом. т.к. строковый поиск очень медленный. элементы enum-а можно атрибутами мометить с нормальным названием, чтобы потом на UI выводить. А вообще, даже отдельной таблицей категория быть должна, т.к. естественно, что с ходом времени категории могут добавляться или убираться


winrm.Basic not "basic"


Same as above


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.352](http://teamcity.monogame.net/viewLog.html?buildId=8779&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


По какой причине мы используем разнае подходы в написании похожего кода?
Здесь
self.class_variable_set :@@comments_counter, self.class_variable_get(:@@comments_counter) + increment
а в 9й строке
base.send :class_variable_set, :@@comments_counter, 0

В этом есть какая-то необходимость? Подобный подход очень сильно сбивает неокрепшие умы, вроде моего.


@katydalton The sample looks good.
Could you clarify the variable names? (same comment would apply to the original 2D version)
e.g. gdb => geodatabase
e.g. flayer => featureLayer


@samueltardieu just to be sure I understand well. You're ok to merge *after* I've updated code regarding your comments? Or you want me to merge now and fix things later?


Peux etre du coup , faudra t'il mettre a jour la bdd


This not work with annotations mapping!


Add one empty line after copyright.


There were problems with the 2x rewrite so it's a [rollback](https://github.com/AtomLinter/linter-eslint/issues/160#issuecomment-121681913) to 1x but as 2x was already published it needs a major version bump.


remove override.


Remove the break if you really want to run `aar` from `aaa` ;) Btw, `aar` can be slow on big binaries.


What if _dialog  gets deleted (because the apps exists).    probably worth putting it in a QPointer<> on the stack.
(because even this could be deleted possible)


remove the extra spaces in this file.


So FileName.cues.txt and FileName.cues.autosave.txt


Can we name it the *pirate command* please?
AAAAAARRRRR!


I think plain `io:format` would not work either.


unused


saw approval from bkerley
at https://github.com/basho/riak_ql/commit/5cf6b4895de45557003b49c69ecca28f49daa8c9


merging basho/riak_ql/gg/integrate/timeseries = 5cf6b489 into borshop-integration-6-gg/integrate/timeseries


basho/riak_ql/gg/integrate/timeseries = 5cf6b489 merged ok, testing candidate = c01e5d88


Why removing this?


Yeah you'd think JSON would handle this natively :(


Actually I would hope that Scalar::Vector<T> also works in kernel code. This would also be necessary in order to support any `SimdArray<T, N>` where `N` is not a multiple of 32.


Is set this way so you can only select authors with published posts, otherwise you'll end up showing authors with no post


`self->path = g_object_ref (path)`, no?


tab instead of whitespace here?


Why are we creating a custom user model? Why don't we just extend the default one provided by Django?


I suggest you integrate them then merge. If you prefer the other way around, this is fine with me too :)


closes #53


I don't think nvcc will ever break this. So define it only as `const Vector<T>`. Also, I prefer if you don't use `AsArg` anywhere in the CUDA implementation. This is really just a bad hack in the x86-based implementations to support the crazy quirks of MSVC.


this will go to puppet, leave it here for now but add a TODO: puppetize


What has changed here?


@mamchenkov yum is dead.


I'm happy I collapsed those headers. ;) It'd go way too wide.


ping @JanTvrdik


Will explain later. Continue with this for now.

Varun Jain
LinkedInProfile <https://www.linkedin.com/profile/view?id=69628206>
#: +91-8376846196

On Thu, Jul 16, 2015 at 5:41 AM, Utkarsh Bansal <notifications@github.com>
wrote:

> Why are we creating a custom user model? Why don't we just extend the
> default one provided by Django?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/trolltartar/Newsybackend/commit/c4c748d0242b6557c1534083a1fa49b5106447d0#commitcomment-12202214>
> .
>


same here: TODO: puppetize


This is a layering violation - I'm OK with that if we can't think of anything else (creating an `OstreeSystemRepo` subclass just for this would seem quite heavyweight).

What if we had `ostree_repo_new_system_with_sysconfdir()`, and it took `sysroot + '/etc'`?  There's nothing in the repo that needs to access data outside of sysconfdir, right?


@jturcotte I now had the case were even waitForFinished took several seconds... i hope this has no deadlock potential somehow


See https://github.com/dangerfarms/fuqu-app/commit/eaa9b65 for a bit of refactoring here and in the test.
Also moved the module to Common, as it is not related to the Profile.


toBeFalsy() is true for `""` or `0` as well. Should be `.toBe(false)`.
https://github.com/dangerfarms/fuqu-app/commit/eaa9b65


If the property does not contain special characters, such as `-`, the apostrophes are it are not needed.


It's one of the killer features of F#: [Units of Measure](http://fsharpforfunandprofit.com/posts/units-of-measure/).


Yeah, probably... if for no other reason than to have a sense of evenness with `Forward`. Good thought, thanks.


As I'm not an expert with RX stuff. I propose to merge now as is. If you have time, you may then commit what you think is better. So I could learn from your diff :)


@ningosi according to the ticket comments, shouldn't this be greater than or equal to 1000?


И еще. Не был бы код гораздо читабельнее если бы мы использовали скобки?
Вместо
`base.send :class_variable_set, :@@comments_counter, 0`
`self.class_variable_set :@@comments_counter, self.class_variable_get(:@@comments_counter) + increment`
написать
`base.send(:class_variable_set, :@@comments_counter, 0)`
`self.class_variable_set(:@@comments_counter, self.class_variable_get(:@@comments_counter) + increment)`


Could you please use parenthesized version of return here and all of the other places?


Why would we want `chockidar` to be added to [`optionalDependecies`][1] and not just `dependencies`?

[1]: https://docs.npmjs.com/files/package.json#optionaldependencies


Ok.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ql/builds/16


For this commit to be useful, we need to add the requirements.txt to readthedocs.

In `Settings > Advanced Settings` we need to set the following:
 - Set "Requirments file" to `docs/requirements.txt`
 - Allow "Use system packages"

This was tests here - https://readthedocs.org/builds/abdealijk-c-test/3092914/


Hahaha. I threw it in there so I could test getting an error message at the bottom of the screen. In reality, the command here is `gq`, and the following `q` translates to a text object corresponding to `1<Line>`, and really is part of operator-pending mode, not command mode. Of course, I could have explained that in the comments... but I'm a looser at 11:30 at night. :)


Thanks Andrew. Much appreciated. I am still investigating why search is not working. Will update you once it is sorted at my end.


Maybe there is something we can do about the performance penalties:

https://bugzilla.libsdl.org/show_bug.cgi?id=3064


Hum...why are we now creating the directory?  Was this for the test suite?

For packaged installations of OSTree I would expect that the config directory comes with the package.

OTOH dynamically creating dirs in `/etc` will make the systemd guys happy at getting closer to empty `/etc` so I'm fine with keeping this too.


According to ST, of these only 0x8D should be considered retryable. All of the others are fatal (e.g. 0x8B and 0x8E are retried internally and 0x8C is a 'bad arguments' type of error). So I've just been bitten by this (some months later)...  I'll be reverting this line soon.

Since it was added to support outstanding packets to non 0, 0 chips on SC&MP 1.4 (which isn't going to see the light of day), this shouldn't be required.


Okay, ich finde es auch nicht dramatisch :D ist mir nur aufgefallen als ich den code gelesen habe.
Hier mal ein Bild von unserer Lobby ;) ich finde, dass Weizen gut dekoriert. :)

http://files.enjin.com/819077/modules/Jan/2015-07-16_14.44.59.png


Could you verify that the API output is working?
I want to incorproate the rating into the Glitter skin, but the ratings don't seem to show up in the API while it is working fine in Plush:
![capture](https://cloud.githubusercontent.com/assets/5703454/8722976/5a34a850-2bca-11e5-8966-edbff012759e.PNG)


вот тут `final` полезно.


Yup, I completely agree. Making a list that doesn't allow its readers to recommend something is not really something I would consider a good fit for this list.


Лучше так
```java
if (MEDIUM_COLOR_THRESHOLD < percentage)
```


I don't know why the commit isn't picking up my sub projects


tryは、createとcreate_snapshotは別にした方が良いかと思います


Cool. I'll make these changes.


Oh, that's a nice addition! Great idea.


Instead of deleting this test, can you skip it instead?
t.Skip("this needs to be added back in CC 1.2 when it is smarter")
Something like that, so we don't have to rewrite all of this from scratch.


Это автоматический стиль последнего эклипса ?


nice!
I can see only one problem, we have the right dpi only when we change monitor.

WM_DPICHANGED documentation says:
>Sent when a window's position changes such that most of its area intersects a monitor with a dots per inch (dpi) that is different from the DPI before the position change.

When testing (with 3 monitors): I receive WM_DPICHANGED message when I change monitor even if the dpi is the same.
We can optimize by testing if the dpi has really changed.

>To get the system dpi setting, use the CDC::GetDeviceCaps function with the LOGPIXELSX flag. If you do not cancel dpi scaling, this call returns the default value of 96 dpi.

We always receive 96 dpi. When application starts, the dpi should be the dpi of the monitor which intersects the application's window.

We can use the following code in `MainFrame::OnCreate` before calling `ConsoleView::RecreateFont`

```
  UINT dpiX;
  UINT dpiY;
  RECT rect;
  GetWindowRect(&rect);
  HMONITOR hmonitor = MonitorFromRect(&rect, MONITOR_DEFAULTTOPRIMARY);
  GetDpiForMonitor(hmonitor, MDT_DEFAULT, &dpiX, &dpiY);

  ConsoleView::m_dwFontDpi = dpiX;
```

>The following actions cause DPI changes:
>- user moves the application window to a display with a different DPI.
>- user changes the resolution.
>- user changes the desktop scaling value by using the slider control in Control Panel.

Documentation says how to handle the first case (with WM_DPICHANGED). But I have no idea on how to handle the two others.


Looks like it was trailing whitespace that got removed.


@mano3493 You must be testing the `site` object here. Not the options.


ok, but you should improve commit message (50 char for title, rest on different line)


The point of that commit was to get rid of these crazy conditionals.  I'd like to find a way to do as much of this testing as possible by using simpler methods on classes than having these huge multi-part conditionals no one can follow.


98812420-SwimmingPoolHeatingSystem-4805 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1684 of 1685 tests passed)

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


@mano3493 By doing this `site` will not be available to the outer scope. Declare `var site` at the top-level of the function and in the `beforeEach` use: `site = new Site(options);`


@borshop merge


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.352](http://teamcity.monogame.net/viewLog.html?buildId=8779&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:26:36


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.353](http://teamcity.monogame.net/viewLog.html?buildId=8783&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Что за магия ? Переименуй `position` во что-то понятное. У меня позиция с точкой ассоциируется.


This definitely isn't the right place for this one.  That said it probably needs a bit more thinking to figure out what the right place would be...


@matz
There is noting function description for d:  (d!)


I use this type 0x02 for cycle routes. Road_class=4 is meant for automotive routing (=motorways) on some (all?) Garmin devices and I think it gave routing errors when you set your device in bicycle mode.


Hello, I like the feature, but I am not really happy about having an option to store the localconfdir. Also, for completion we could have a single loop `for col in ${kak_runtime}/colors/*.kak ${localconfdir}/colors/*.kak;`. And a last thing is that I think this will fail if localconfdir does not exist (you'll get '${localconfdir}/colors/*.kak' as a completion candidate).


Why it should be called 'fh' instead of 'file'?


Да, по вызову форматтера ничего не меняется.


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


yep i think this is the one
i will stick with this one


мвмивпи


It seems that I've managed to merge master into my branch with resolving the conflicts manually, so I think I'll be able to do it again but from my changes into master if I get the write privileges. However, some things still don't work even after the merge so I'll resolve these first.


Could you please use PSR-2?


98812420-SwimmingPoolHeatingSystem-4805 (rglazer2) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


One more space


めも
他の場所直したらなおったみたい


test


All the translate tests must be placed here: https://github.com/phalcon/cphalcon/tree/master/tests/unit/Phalcon/Translate/Adapter


ack seems to work


Looking forward to new versions coming out. Would be lovely to see the npm package updated too! Keep up the great work.


Sorry and thanks!


Hi,

I encountered several APPCrashes under Windows 7 Embedded before I used this fix.
The crash was on the call to "memmove" (msvcrt.dll) with an acces violation exception.
Can you confirm that it was the cause?

Thank you.


copy-paste :) space


There is no reason to hide the login form initially. As a user, you will want the minimum number of clicks.
Also no reason to disable to login button (this causes issues when for example chrome autofills the login fields currently).
See https://github.com/dangerfarms/fuqu-app/commit/4d30b45 for changes


Even if you *don't* select "avoid highways"? And I thought that bicycle mode was not really supported by your map, that automotive mode should always be used (too many issues otherwise).


Sure thing! For now the sources are still in review, will send a pull
request once they land.
On Jul 16, 2015 4:18 AM, "Peter V. Saveliev" <notifications@github.com>
wrote:

> Do I miss something needed? I'll check through the kernel sources, —
> certainly there should be — but if you need here something particular not
> implemented, pls, ping me so I do it. Thanks in advance!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/drzaeus77/pyroute2/commit/4389237fc274a0e39b61edf853c0d8b9241883df#commitcomment-12200687>
> .
>


It is working properly,
You must have an account though.


Nice. I didn't know that's possible.


missing docstring


Bad practice to make query directly in controller, use Repository class of certain entity.


shouldn't "libcoreclr.do" be "libcorelclr.so" ?


Выглядит лучше, чем раньше imho. Но на гитхабе уже приходится скролить.


according to https://api.jquery.com/event.which/, `e.which` is the preferred way to access the keycode


@annevk this is wrong, Object.keys only returns the keys.

This works:
```
Object.keys(params).forEach(key => { url.searchParams.append(key, params[key]) })
```


that class is never removed


For this commit to be useful, we need to add the requirements.txt to readthedocs.

In `Settings > Advanced Settings` we need to set the following:
 - Set "Requirments file" to `docs/requirements.txt`
 - Allow "Use system packages"

This was tested here - https://readthedocs.org/builds/abdealijk-c-test/3092914/


reack


I have an account, a VIP one.
Still it only shows up in Plush and not in the API output?


i usually like being explicit and add `== '1'`


I just checked and the ConfigurationObjectBuilder registers the instance, not the provider. I could do this on the manager side but it is required.


`if previewer is None` ...


Shouldn't this use root shell too? Technically it's possible to fetch images as a normal user if user is in `rkt` (or something) group. But I don't think we should assume that.


stdlib comes before thirdparty


字符串匹配kmp算法


Ah, in the comment. Right. Thanks for spotting it.


docstrings please :)


I tested it with being "" and collectd handles it fine.


Why not `for (Sample s : samples)`?


`ImagePreviewer`


That's a relief. Yes,please have the manager do it on the factory 's output
On Jul 16, 2015 10:07 AM, "Ana Felisatti" <notifications@github.com> wrote:

> I just checked and the ConfigurationObjectBuilder registers the instance,
> not the provider. I could do this on the manager side but it is required.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mulesoft/mule/commit/04eb996134233acf129cd929be71c4a14e3bf310#commitcomment-12202727>
> .
>


This could be the default implementation of `generate_content` with a `template = None` in the base template (which you then override in the subclasses)


You mean, it doesn't deliver to API-calls in Glitter?
I will check again tonight.


My bad, shall fix.


Yeah, we should be able to fetch as a normal user so that's why I didn't change it but I guess it's better to use the root shell here to not complicate things.


you can use babel with utility.inlineEnvironmentVariables


`for previewer in get_file_previewers()` - no need to assign it  first


This still will not going to work as the new avatar driver is in a extension and getclass returns the namespace ass well. Therefor they replace the part with preg_replace('#^phpbb\\\\avatar\\\\driver\\\\#', '', get_class($driver));

This will work:
	public function get_driver_config_name($driver)
	{
		$classname = get_class($driver);
    	if (preg_match('@\\\\([\w]+)$@', $classname, $matches))
		{
			$classname = $matches[1];
    	}

		return $classname;
	}

Left over is the part acp_users

Around line 1851 changed to
						$template->set_filenames(array(
							'avatar' => str_replace('ucp_', 'acp_', $driver->get_template_name()),
						));

It's working in this way assuming the ucp template is named exactly the same as acp templates so they also can be one at the end


do we ever not have preview content?


@tchaikov they are in different for cycle, do you mean there are two 'name' variable define in get_health function?


yeah, there are few comments with ".do"


you want `<pre class="none">` here and no ` | safe` - it's text so html etc should be autoescaped


Bad practice to make query directly in controller, use Repository class of certain entity.


Confirm : https://bugs.limesurvey.org/view.php?id=9778


This is some forgotten debug garbage:) One must remove it to get the test working in current state.


missing semicolon


I haven't clear the users' contacts yet when they logout.


The intent is to have the new driver to define the get_config_name() method, and not inherit the generic one. There you will be able to put whatever makes sense for you.


This is already done in the very first line within the loop over all the elements in the XML. It will now always add all the namespaces for each element and let the namespace manager deal with collisions and what not. This has a tiny performance impact but I don't think it will ever become an issue.

I don't think this exception can trigger in any case; it is just there as a safety measure.

The PR also includes a test case where a namespace is defined in an element so I think everything works correctly.


Thanks, I guess I did not test this properly after all.


fast-forwarding develop to borshop-integration-6-gg/integrate/timeseries = c01e5d88


sorry, my mistake. you are right! they belong to different loops.


See the screenshot above on the left, it doesn't output the current ratings for each slot.
The rating_avg_video/rating_avg_audio fields are not there while they do show in Plush.


@kurtraschke Ok, thanks!


for streak


True...this is currently not possible - I'll fix it. But I really hope nobody generates an XML where this is an issue...


aaa is suposed to be slow :)

> On 16 Jul 2015, at 14:35, deffi420 <notifications@github.com> wrote:
>
> Remove the break if you really want to run aar from aaa ;) Btw, aar can be slow on big binaries.
>
> —
> Reply to this email directly or view it on GitHub.
>


@equinusocio neither gulpfile.js or gulpfile.coffee is showing the gulp icon in v1.5.1?


Is this assertion to catch programmer errors or is there a possibility this happens in the wild? I'm hesitant on having code hard crash due to assertions.


The code says:
```java
it (true) {
  return true;
}
return false;
```
Wouldn't it be better to just return the condition?


스토리섹션인포 대신 스토리섹션을 그냥 사용하면 안되나요?


@cyanogen Memory leak here: dsi_cmds.buf must be freed.


Where do you use this?


Oh ok.


Nit: I wonder if we can use an enum here instead of a class?


Entering `^D` twice in the middle of a line to input an end-of-file on a UNIX shell.  It's the same for all commands (try with `cat`, for instance).

The command I gave was meant to have `"blah blah blah"` as an argument to each `echo -n`.  You should see that `diff` gives a non-zero return code for your example (even when you take out the extraneous `cout`s).


Hi @romanbsd

Yup, that is does. Our test quite does not currently include VCR and relies heavily on integration testing.

It's something we're aware of, but the test suite isn't terribly long to run and this was a higher priority issue.

Happy to accept PRs!


c'est ok de mon coté


@SortaCore I know, I just never incorporated it because properties are usually pretty dynamic even in simple extensions.


Entendendo


Creating


@jamiecobbett looks like that redirect issue is fixed in master but not released yet https://github.com/phatworx/devise_security_extension/commit/00b4584466ace1a13a4ff4158d52386b21ebeff0


late dinner, and had a couple of beers, but what an amazing job you did here, the code is iike 10 thousands time cleaner and simple :)


My original intention here was to (temporarily) hide the sitrep, objects list, pedia etc. panels when the production/research/design windows are open and restore *all* of them when closing those windows instead of them staying hidden (see [L5076](https://github.com/mel-odious/freeorion/commit/f23541c75cd51f139a739fc530cb90dce5c1fbe5#diff-fd5dfd184bfde5b71d9194b6d2ab0cd7R5076) and [L5095](https://github.com/mel-odious/freeorion/commit/f23541c75cd51f139a739fc530cb90dce5c1fbe5#diff-fd5dfd184bfde5b71d9194b6d2ab0cd7R5095)).  However, the order of the function calls there means that those panels are immedately re-registered and stay visible.  I could fix that which would make the sitrep panel disappear while the production window is open but now I'm not sure which behaviour is preferable...


You can use `calloc` to combine the `malloc` and `memset` in one operation.


I‘ve added this line. I have changed this comment


...sony_ric and selinux functionality


This file apparently has not been committed.


Do not use getText() on PsiElement instances unless you 100% sure you need it.

In this particular case you're missing out the following scenario:
```erlang
handled() -> erlang:is_atom(a).
missed_out() -> 'erlang':is_atom(a).
```


What do you think?  Does it add readability or take away readability?  When you reach the end of reading a program, what state do you expect it to be in?


Merged in 70e26c7.


Yeah, it should mention Safari 8 explicitely. :+1:


Nit: Can reduce this to self._favicons.sort { $0.width > $1.width }


`not` was supposed to be pseudo-code.  Don't use the alternate spellings, because they're too cute, people don't know them and iirc MSVC doesn't support them by default because of that.


Love the deferred :+1:


Careful with this. It's a very inefficient method of doing it. Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) and its first answer. Can you extract a way to calculate this in `O(1)` instead of `O(n)`?


This does a first look a little weird, the reason behind this is other filters (`liveblog_before_insert_entry`, `liveblog_before_update_entry`, `liveblog_before_preview_entry`) that call each extensions filter function ([here](https://github.com/bigbitecreative/liveblog/blob/extending-entries/classes/class-wpcom-liveblog-entry-extend-feature-authors.php#L82)) passes an array. Instead of adding a check if array into the 4 different filter functions we felt it was better to wrap it here.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.353](http://teamcity.monogame.net/viewLog.html?buildId=8783&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:25:30


This does a first look a little weird, the reason behind this is other filters (`liveblog_before_insert_entry`, `liveblog_before_update_entry`, `liveblog_before_preview_entry`) that call each extensions filter function ([here](https://github.com/bigbitecreative/liveblog/blob/extending-entries/classes/class-wpcom-liveblog-entry-extend-feature-authors.php#L82) passes an array. Instead of adding a check if array into the 4 different filter functions we felt it was better to wrap it here.


dodadeno zaradi redovite koi ne se click-nati da im se prikazat redovite.


sdf


The location of the form tag and buttons can be improved. I was having a hard time because of the way the partials and conditionals rendered. Could employ the new HTML5 form attribute for input elements.


Why not fix these in the jsp pages instead of here?


I noticed when using `get_liveblog_state` it would throw an error on for example the 404 page instead of returning false. So I made the change here so it would return false instead. I later added the admin check as it’s needed in the admin panel to allow you to archive or enable a liveblog.


Should 'compromised' be 'comprised'?


tuka se **krijat** site naslovi na kolonite


should `0` not be returned if `a` or/and `b` do not have a position vector ?

````
If compareFunction(a, b) returns 0, leave a and b unchanged with respect to each other, but sorted with respect to all different elements. Note: the ECMAscript standard does not guarantee this behaviour, and thus not all browsers (e.g. Mozilla versions dating back to at least 2003) respect this.
````


needed to be changed in order to get min element height: 45px.


I think this needlessly copies `Strong<ObjCBlockCallback`. You can use move semantics here, or pass a `VM` and `ObjCBlockCallback*` directly and initialize the handle inside.


Well, you might as well make the type of the `globalObject` parameter be `NativeScript::GlobalObject&`.


@cdwort we can't use VCR because what actually talks to the network is wkhtmltopdf


site koi go zadovoluvaat uslovot ( expandirani i imaat nekakov podatok) ke bidat povtorno prikazani posle blur eventot ili ako e prazen search inputot


Please, try writing less code which doesn't do anything.

In this case you don't need these ```op() ->``` function clauses except for the first one. You can also call functions like this: ```erlang:'<'(1, 2)```.


There are three notification types that a user can subscribe too during the liveblog event:

1. **A key event**
This is triggered when the author uses the key event command: `/key`, so the user can be notified when a key event happens.

2. **An author alert**
This is triggered when the author uses the alert command: `/alert`. This is useful if the author wants to notify users, but is not significant enough to be classified as a key event.

3. **Tagged alert**
A user can subscribe to tags that the author posts. For example, if the author is liveblogging an Apple event, a user could subscribe to `iPhone`. They will then get notified when the author uses the `#iPhone` hashtag, so they don't miss that part of the event. The author could add a note to the liveblog to let users know what tags they may use throughout.

Notifications will only fire if the user has navigated away from the page, so they don't get unnecessary notifications whilst they are watching the feed.


za da se **prazni prebaruvanja** koi ocigledno ke gi prikazat site redovi


Is this raw-fmt-opencl-test branch stale? Should we delete it?


Yes, thanks.


This section renders strangely when the markdown is converted to HTML.  View the file from github to see what I mean.


98812420-SwimmingPoolHeatingSystem-4805 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (653 of 653 tests passed)

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


Ah, yeah, that makes sense. Haven't thought about how VCR works in a while.


OK, thanks. I presume we can't also get these hard-coded locations from the API?


people will simple continue -> people will simply continue


Same problem as with the alternate glyph section, above.


关注医院


Oke found it!
The statements are in the *if web_dir* in the api.py, so they never get called for the API.


OK


At first this looked like calling a static member, I typically use `this->name()`, but this will work too...


The parenthetical expression does not belong to the second sentence but to this entire change log entry, so it feels more appropriate as a separate sentence.  Either that, or it should be make parenthetical to the first sentence (*the dot...below the line*) where it is more appropriate.


Is this really the name of the `AuxKernel`?


if no action selected, return the best action


This seems like a rather extreme value -- what is the reasoning?


Here's an example of why `continue;` gives the incorrect behavior:

```
$ echo "1 + 2 3 - print" | ./stack_lang | ./use-result
--> Ignored invalid input: There must be at least two values on the stack to use an OP.
# ./use-result gets 1 -1, which is incorrect, and return value is `EXIT_SUCCESS`, so we're saying the computation was good.
```

Once we determine that we get a syntax error, it doesn't make sense to continue, because there can be no semantics at all.  Your program is not doing nothing, it's continuing on as if nothing happened, but leaving the stack in a state you can't reason about.  Thus, all subsequent operations will necessarily be incorrect.  Why do them at all?  You're making a false dichotomy between silently not doing anything and continuing on emitting an error message.


Awesome, thanks! :)


These turn rates are in degrees per second -- so turnrates above 360 means it will be able to spin a full circle per second -- are you sure this is intended?


Again, the parenthetical expression belongs to the change log entry, so I think it should either remain a separate sentence or be made parenthetical to the first sentence (*The final line...including the score*).


The program will not exit with the name of the exception.  It's undefined what happens.

Don't use exceptions.  We haven't talked about their behavior, and this thinking above is dangerous.


Do you want to add the doxy comment for the deprecated version?
1. they reference `name` but the parameter name is `deprecated_name`
2. the are in front of the new c-tor, not the old one...


What about `1 2 3 +`?


These logging statements shouldn't be here


but it's for markdown too.. markdown has an html output.


[I have](/sigmavirus24/betamax) =P

It only works for ruby applications doing HTTP through something that WebMock or other libraries can replace. wkhtmltopdf being a binary (and further, one written in C++) cannot be mocked out by VCR. We'd need a moderately complicated transparent proxy to capture the data with a way to play it back and store it as data.

I'm 90% sure one exists, but I have almost no desire to learn how to use it, install it on Travis, or deal with it on people's development machines (because I know *we* will get bugs asking how to set it up instead of people looking at the docs/FAQ/StackOverflow for that project).


Copy and paste: this is not `AuxKernel`...


progreeessss!!!

A. after Replace == 'Kerbin' with isHomeWorld when game is open in Eve Gui all config data it show correct aka is identify the planet!!! \o/ finally !!!! i was search in rest of code & noo more Kerbin name present or at list my eye it was not see anymore :)
aka HighLogic.LoadedScene == GameScenes.MAINMENU it can use a custom name planet bye bye hardcoded Kerbin name > hyp hyp huraaa <

B. what i describe about loading / query Eve mechanism & all manually procedure what is need to be done each time when game is open are same, something is messing "LoadedScene" "GameScenes" procedure setting in a temporar loop dimension what IF Eve gui it use & hit apply button it correct partial problem
- i try to trace all key moment about this issue but you have spread in all file pice code what are use & i think you are only one who know to set correctly all puzzle together, partial i succeed to undestend but my lack of real knowledge in coding set me in hard position to see were is problem :)

C. here it is GameLog & Screenshot done with this new test release, some screenshot are mest up i know coz i not have correct proportion or is not proper adapt at your new change, for that only Pingo & Proot it will make them look nice me i only try to figure & help make Eve compatible with Kopernicus / RSS system :)
- test was done using  -force-opengl -popupwindow %command%
GameLog
https://drive.google.com/file/d/0B9RK2PFOUhe3QlNYN25feGx5ZGs/view?usp=sharing
Screenshots
https://drive.google.com/file/d/0B9RK2PFOUhe3Zng4SmM5aDEzb0E/view?usp=sharing

hope it help & if you need more details don't hesitate to yell :)


-> two different templates


no need to underscore all those template names btw


:+1: done


remove this


We're suggesting that users run commands as follows:

SPARK_DIST_CLASSPATH=$(hbase classpath) spark-submit --jars $((hbase classpath) | tr ":" ",") word-count_2.10-0.0.jar word_count/romeo_juliet.txt test_0238432 7212

The SPARK_DIST_CLASSPATH has to be overridden by the user (which I think will be a problem for everyone in the current configuration done in PR #41) to use the hbase classpath for any bigtable operations that happen in the job's main method.  The --jars is for the distributed jobs (AFAIK).

We're only proposing to setup ALPN in the install_spark.sh file.


Hey!

Just wanted to let you know that you can run just

`./gnome-shell-extension-installer 6 8 15 55 112 120 495 517 602 613`

and then `gnome-shell --replace &`.

The script already supports being called with more than one `EXTENSION_ID` and already grabs the `GNOME_VERSION` from the installed gnome-shell, if no version is specified.

Maybe I should add a `--replace` parameter to automatically restart GNOME Shell session after all installations are done.

Cheers!


Remove


ok, i thought that cert_created meant that the cert where just created in this session.


We're getting there, but what's the point of separating it into four different loops?  If our goal is to separate I/O from computation, we choose a useful data interchange format: a `vector<char>` is not very useful as a representation of a `string`.  On the other hand, if we structure our loops correctly, I/O is still separate from computation, we avoid the need for `vector`s, and the code becomes easier to follow, because we have fewer things to keep in mind and only two loops total.


OK.  For my clarification, how come you are separating out online and offline install and separate update-online and update-offline commands?  Why not do a single install command that will attempt to match GUID and then perform appropriate installation (try online if HW supports it first but fallback to offline)?


No, the original one was correct...


I found the problematic path:

Rizzoli & Isles

If I add this path with EMC to emc-permsort.xml EMC ignores most of the file -maybe all entrys after the entry for 'Rizzoli & Isles'?- after a Reset/Deep Standby or GUI-Restart. All other paths including German Umlauts added before are okay (if i don't add the above mentioned path). Even a path with a '+' in it works. But a '&' in a path and EMC forgets all already written entrys in emc-permsort.xml.

In the next minutes i upload a tar file with my actual emc-permsort.xml and a emc-permsort with the above mentioned path to the link i mentioned before. I only have to switch on my STB and copy/tar the files.


@Souldrinker I just published v0.13.7. See if that works for you.


Yep, good. Just don't hesitate to ping me if some basic NLA functionality is needed to help you to write extensions :) NLA code pieces in the kernel are so heterogeneous, so it's hard to cover all the functionality at once, and I just implement some things on demand.


Does this remove functionality that people may be using? (will it break any existing config?)


shoud 'jQuery Software Foundation' be 'jQuery Foundation' ?


greetings and stuff


스토리섹션은 모델클래스이고 세이브시에 현재 사용중인 StorySection에 대한 정보가 필요합니다. 따라서 StorySectionInfo라는 클래스를 만들었으며 사용은 StorySectionManager를 이용하시면 됩니다.


Je to tam dvakrat, tak jsem to prvni odstranil
unset() zrusi promenou


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.766.939a0e33edd86adbbdd2820bbe4cf8784641eb9c](http://build.palaso.org/viewLog.html?buildId=59789&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 379, ignored: 3 Build time: 00:03:49


Yeah, there's no way to get around some network calls in our test suite if we want to run any "Is it a PDF" tests like:

```
expect(pdf[0...4]).to eq("%PDF")
```

and not be testing our own stubbing.

I think it's a legitimate question whether we're doing the right kind of stubbing and testing in our suite and a review is needed. But, the suite doesn't actually take that long to run, and I'm happier having fewer unit tests than fewer integration-y tests, so it's not my highest priority right now.


Are we not adding a space between `()` and `{`?


These assignments have no effect.


same as above, shoud 'jQuery Software Foundation' be 'jQuery Foundation' ?


PHPCS: Usage of ELSE IF is discouraged; use ELSEIF instead


Do we really pass this by value? Just making sure...


##This is a markdown file


This commit relates to Jira ticket : https://opensourceehr.atlassian.net/browse/PR-2498


Yes. Like @gnarf said, this was just a find/replace Django for jQuery so probably a lot of those


Well `ostree_repo_is_system()` needs to know the actual system root path, not for access but for comparison with its own path.  One could argue that function is also a layering violation, but I don't really see a way around it for the special case of using `sysconfdir` for certain repos.

Subclassing would be technically cleaner, but I agree it's a bit heavyweight now.  I'd follow your API rule and hold out for a 2nd use case.


Why not `const std::string &`?


Aha, I see that I left NLA_FLAG not implemented [1]. Ok, will look what to do with it.

https://github.com/svinota/pyroute2/blob/master/pyroute2/netlink/__init__.py#L1366


Heh, indeed.


Okay I will change that. Thanks for pointing that out.

On Jul 15, 2015, 2:52 PM -0700, Nan Davenport<notifications@github.com>, wrote:
>
> Don't know if missing jquery is causing problems, but if so I'm pretty sure that this link syntax:
> <script src="//code.jquery.com/jquery-1.11.3.min.js"></script>doesn't work on a project that is not on a server -- e.g. just viewing files locally.
>
> It's necessary to use the whole url including http:
> so<script src="http://code.jquery.com/jquery-1.11.3.min.js"></script>will work from your local file structure
>
>
> —
> Reply to this email directly orview it on GitHub(https://github.com/code-newbies/js-tuesday-hangman/commit/3e6c3737863a2ed0c9f5cea8069c277d5f294d9f#commitcomment-12190332).
>
>
>


I blame EVE.


Needs to be updated after CApplicationMessenger update.
See: https://github.com/popcornmix/xbmc/commit/ca991de5b60427ba5a1027eb5dabd930b05e01e0#diff-5438535e55ccf7e8181128c8cf3a38e2R1232


IBA Opt-out (by Google)

Save your interest-based advertising opt-out preference permanently.


Yup, looks good, thanks :)


Indentation


This should let me use it like a service instead of a page. Useful when I create my refresh ajax functions.


This comimt refers to Jira ticket: https://opensourceehr.atlassian.net/browse/PR-2527


@dukesrg No, the outcommented patch below could make trouble, this patch lets you update just fine.


I think I've read all in detail. It looks good. I have a few questions. Should I do that by email or comment here?


These seemingly friendly `return`s are actually probably quite harmful. Love that you're implementing some argument validation, but highly recommend `ArgumentNullException` and similar.


Yup, thought I caught most of the 'jquey software foundation' ones


Why?


Similar to the Argument validation comment above - just want to make sure you don't miss this one because it's not at the top of the method. :)


@dukesrg Do you mean something like the struct Syphurith wrote here: https://github.com/roxas75/rxTools/issues/80


I'd remove the spaces after/before `(`/`)`


s/Sbt/sbt/g


We don't want to change our code to give the incorrect answer, even if that answer is easier to to test.  We want to test the correct answer.  If doing so is too hard, we want to change the test, not the program.  That way, the program gives the correct result, but the test may be more lenient.

```
./ternary | xargs printf "%.4f"
```


Not sure if this is proper workaround, but for SM result of `Error.prepareStackTrace()` was `undefined` in my tests, so I better not use it in such case. For example `require('express')` was failing with :

```bash
TypeError: obj.stack is undefined (/[...]express/node_modules/depd/lib/compat/index.js 31:6)
```

Maybe it's better to check why a stack is `undefined` in first place.


I say we keep the discussion open, here for history and clarification later


a space between `(` and `{}` and add space after `:`


Is it enough to move the rating section outside the "if web_dir".
Or do you need the rest of the verbose info as well?


啊


hi


Is there a public email to reach the board? Foundation@ ?


@Oxicode See https://github.com/cakephp/cakephp/pull/7032#issuecomment-121861542


Should `mocha-phantomjs` also be locked down?  Otherwise could they update their peer dependency of `phantomjs` and break us again, right?


I thought when the template is not used directly from a controller, we "underscore it".


Yeah, it was for the test suite.  I guess I could have the test harness create it, but this seemed more robust.  Didn't know about the packaging expectation.


no we usually do it for library-like templates that only contain macros


Yea! Time for Mishra Possibility Storm!


ok good to know.


EchoPingHttp, the HTTP/HTTPS probe for SmokePing, also leaves the port number in the "Host" HTTP header.  (This form of the Host: header is quite legal and should be supported anyway. :-)


Hmm. We cannot fix/improve them and we cannot remove them.

I wonder if a 30s/15thr simplification test would succeed but then we loose the STC value of history.


@psychon
Ok.  But `lua_isinteger` has been added only recently (found it in the source, but not in my installed Lua):

    LUA_API int lua_isinteger (lua_State *L, int idx) {
      StkId o = index2addr(L, idx);
      return ttisinteger(o);
    }

Lua 5.2 seems to implicitly convert numbers to integers, while this behavior is opt-in (or at least different) for Lua 5.3.

I'll follow up the discussion on the PR itself.


@hartmannaf exactly!


@pwalsh since that commit it should validate datapackage with just ```name``` property and will not validate datapackage if there is non-empty item in ```.resources``` with empty ```.path```.


We don't ever want to commit unfinished code to the final repository.  When a patch is reviewed, we don't patch our patch, because that first patch is useless, pollutes the repository, and prevents us from doing useful testing using our history.  We make a new patch or patchset.


@andresgutierrez, leave this commits in history, or reset?


No.  `drawin_moveresize` gets called for `set_x` etc.. :/


Think I have located the original piece of offending code (in the sklearn version): it was failing to keep the variance to a minimum value when there were no points in one of the clusters (so the self._covars_[self._covars_ == 0] = 1e-5 in the _init mehod) Thanks


That's fair enough, I'll change this.


Floor46〜50の作成 #469


You mean the template will be an argument?
``` Python
@classmethod
    def generate_content(cls, attachment, template=None):
        return render_template(template, attachment=attachment)
```


These can be combined in to one line like this:

attr_accessor :title, :priority, :complete


no, a class attribute like ALLOWED_CONTENT_TYPE is


Generally, models and views of wizard are placed in a directory other than views/ and models/, so in the wizard/ directory.


yes. google does not allow unicode in batch requests.
as soon as i sanitized the emails, the batch went fine.


I do see why you wouldn't bother with it, although personally I would say that not allocating them in the first place is better.


Aha ok. But not in upper case this time, right?


@talaj nice catch. can you ticket+testcase. I missed that usage when fixing the original bug!


well it's kind of a constant, so i'd say uppercase is fine ;)


It's enough to move it outside the if.


remove ()


You forgot `gantry-theme`.


today is the two year anniversary of this commit, @timcowlishaw :fireworks:


I looked around on the ApiServer and ApiDiscoveryServiceImpl but could not find a call to "setAccessible()" methods on the fields that are annotated with @Parameter.

What do I mean? If a field is final, it won't have a new value assigned unless the "setAccessible(true)" is called. If we try to update the value of the field via reflection, it will throw an exception.

Do you know if the "setAccessible(true)" is called somewhere else?


Could provide a better description for this? What are the different ways of treating the textures in shaders and which boolean value corresponds to them? Also, keep in mind that many mod developers may not know what displacement mapping is.


In the base class (`Previewer`), should I initialize the constant with `None` or not?


This seems not to belong in this commit.


yes


You're right, I forgot `git add -p`.


I believe we can reduce the clutter by using `.toFixed(0)`, since the scores are unlikely to have that much precision (and the users probably do not care to distinguish between an edit with say 75.01% and other with 75.02%).


Should be #config/property :)


remove file này


No, I don't. Honestly, I mainly copied the existing code to add these parameters.

While doing so I found various things I want to fix in the S3 Secondary Storage code, but that's for a later PR.


These changes seem logically separate, right?


`StringUtils.isNotBlank(canonicalMetadataName)` instead?


commit


Interesting.  I'll take a look at that when I get a chance later today.  You're welcome to work it up and submit a PR, too.


Hello,
is it on purpose that the for loop is ended by this return (or the other return statements underneath) ?
Is the data dict always only one entry?


Ca ne va pas marcher ça. `toker -> token`


Yeah - I'm hit or miss on the trailing closure syntax.  I tend to use it when I'm on a roll.  Autocomplete gives the more verbose format, and so when I'm in more of an explorative mode I tend to let those bleed through.


Hm… now I actually do have time to do it =)


Should this use `self.engine_id`


@andreastage perché non metti i nomi dei campi in inglese? Poi aggiungi la traduzione in italiano per la simpaticissima Elisa/Erica/Non Mi Ricordo Più di PG... così rimane in linea con il resto del progetto e non si sa mai che non possa finire upstream..

/cc @giaespo @xrmx


As with exercise 1, this isn't strictly necessary.


The reference to #idl-array points to nothing, so this specific change looks suspicious.


same here simplify by getting the value and return the url `if (StringUtils.isBlank(valueCanonical))`


oh, forgot to add docstring here


Yes, but unfortunately that version broke other tests in Signon. Similarly the later versions I tried broke other tests.

We are planning to upgrade various dependencies in Signon shortly, so this should get the time spent on it required to untangle the problems.


Is a max width enough?  Well, it is for my current issue but what if a page contains just one fine vertical line? When that line is scaled to 4800 pixel width, it'll be gazillions of pixels high, no?


Hey @ivanhny,

My wife worked for a publishing company, would you mind sharing which one it is?


Hello @wsabina

do you work with any other languages?


Glad you are having fun.  What city do you live in now?


Hi @stevebull. How long did you live in Honolulu?


What city are you in right now?


@MadOgre but...could you put in where you've lived while you've done this Coding? ...LOL, just for practice...


Hey @dwyeth how long did you live in NY? What made you get into working on the web? Would you mind adding that to your bio?


en fait ca marche quand même, mais je corrige
comme quoi l'init des attr dans __init__ ca se discute...


Hi @gecko25, what did you do in Costa Rica?


I just checked about 10 command implementations and none of them have attributes initialised and with the final modifier.

Just to make sure we don't get into trouble, could you please change the initialised ones into something like:

private Integer connectionTtl;

By default they will be "null" anyway and we won't have problems with reflections throwing exceptions. Just to be safe. :)

Cheers,
Wilder


Ahh, you might get a deadlock in that case actually, since a message might trigger the extension to send more message, and since they are sync message, it might wait for the gui main thread to check the event loop.

Since this is the legacy extension and that we only use the result for a qDebug, could we start it with QProcess::startDetached instead maybe?


Because you are extending the Schema prototype, I think this declaration will cause the schema records to be shared across all Schema objects.  I think you want it to be a separate instance for each instance of Schema, right?


ack


Hey musicalevolution, I don''t think you are Peter Bell. Please correct.


How long have you been living in Brooklyn? And formerly?


Do you mean to expose a new public API on Schema?  If you can get rid of this, it would be better.


Same comment as above


update


Could this `result *= 3;` be moved outside the `if` blocks?  Will this change the behavior demonstrably?  What would doing so do for maintainability?


@jackieneon would you please add how long you lived in the NYC to your bio


No problem! MSVC is keeping us dumb :)
I should check what MSVC version we are using and try to update it, maybe it supports inline declarations.


Quoc -- do you live in Newark now?


This error message doesn't seem accurate anymore.  The allowed types are no longer just primitives


By the way, rimraf is closer to `rm` than trash – and it performs way better on my machine. It also has a CLI: <https://github.com/isaacs/rimraf/tree/9b97ac62e1b459d84dbe18f20757bfe4374e65ab#cli>.


Same as above


this doesn't cause a non-0 exit code anymore.


thanks for adding this. Could you use try/finally to make sure this gets logged regardless of the control flow?


Can you add a second test where the order of Document and Fax declarations are reversed?


No. It's just setting the value of `value`, a locally-declared variable.


@musicalevolution NYC


commit message needs adjustment ;)


Correction:
After adding a path with '&' in it EMC forgets ALL sort entrys after a Reset/Deep Standby or GUI-Restart.

If I add an new path in this state with EMC EMC writes a complete new emc-permsort.xml with only this one path.
If I instead use an editor and delete the line with the '&'-path or use a copy of emc-permsort.xml without a '&'-path all sort entrys are okay after a GUI-Restart.


Merci Riku!! C'est apprécié.


reack


The issue here will be scalaz-stream, which is not published to central. So even if I did somehow get doobie over there we would still need that resolver. Why are these repos blocked? Sonatype isn't any more trustworthy or reliable than anything else.


LevelX, is there an easy way to see what cards are not implemented for the cubes? I think this one finishes the holiday cube, but wouldn't be surprised if the peasant cube was missing some.


Wollen wir uns mal auf eine Minecraft Version einigen ich schlage vor das wir die 1.8.7 benutzen.


Thanks @OCVA !


Hi @stevebull
I added the info you requested.
ds


ok,thank, i would check the code once again


This error handling code is far away from the condition it's checking.  Could it be better to say

```cpp
if (!read into str) {
  cerr << "No digits read.\n"s;
  return EXIT_FAILURE;
}
```

From that point on we know we read in the input.  It could even be clearer to write:

```cpp
read in input with cin;
if (!cin) {
  // as above
}
```

which clearly shows what we're doing.


@alicetragedy could you double-check if the callout thing is working? it's only being used in the applications and I don't don't have any data for imitating this locally yet...


Something went funky here, causing a syntax error after merging. Gonna look into it tonight.


I updated my trunk and enabled the VIP system in the core.h file. However my map server crashes always I try to log in. This is what I got.

(gdb) bt full
#0  0x080bf70f in clif_display_pinfo (sd=0xbc302e8, cmdtype=4) at clif.c:17552
        len = <value optimized out>
        i = 0
        details_bexp = {0, 0, 399900, 0}
        details_drop = {0, 0, 899900, 0}
        penalty_const = 0
        cmd = 0
        tot_baseexp = 100
        tot_drop = 100
        info = <value optimized out>
        szdetails = 13
        fd = <value optimized out>
        tot_penalty = 100
---Type <return> to continue, or q <return> to quit---
        maxinfotype = 4
        factor = 1000
        details_penalty = {0, 0, 24576, 8}
#1  0x080ecf25 in clif_parse_LoadEndAck (fd=8, sd=0xbc302e8) at clif.c:9988
        i = <value optimized out>
        guild_notice = <value optimized out>
#2  0x080d766d in clif_parse (fd=8) at clif.c:18151
        cmd = <value optimized out>
        packet_ver = 30
        packet_len = 2
        err = 7
        sd = 0xbc302e8
        pnum = <value optimized out>
---Type <return> to continue, or q <return> to quit---
#3  0x08209d66 in do_sockets (next=100) at socket.c:849
        rfd = {__fds_bits = {256, 0 <repeats 31 times>}}
        timeout = {tv_sec = 0, tv_usec = 21000}
        ret = 0
        i = <value optimized out>
#4  0x08207fbd in main (argc=1, argv=0xbff73b94) at core.c:364
        next = <value optimized out>


why's that needed?


please squash this commit into the previous one


현재 StorySectionInfo는 단순히 StorySection에 대한 getter / setter만 제공해주는 역할을 수행합니다. 다른 추가될 내용이 있는지, 혹은 다른 ~Info 클래스와의 일관성을 유지하기 위해서 만들었는지 등 추가적인 설명이 필요해 보입니다.


Schick mir mal nen Link zu der 1.8.7 per Unterhaltung


Chicago is an amazing town.


oops, seems to be a relic from a pip command i ran


This chain makes me a bit uneasy. Would it make sense to rewrite it in three lines to add a bit of context of what is each value? Also, if the first value is null it would raise an exception.


you are welcome, I made some changes please have a look. I will close this issue.


handlers are made in 'Write Everything Thrice' style :sweat_drops:, would appreciate  suggestions on how to DRY this code


Do sphinx references like this work? I would normally do
```python
:class:`MDAnalysis.core.AtomGroup.AtomGroup`
```
or something similar.


While a cube tournament selects the cards by random, missing cards are written as WARN to the server log. So you could start some cube tournaments and look if some warnings are written to the log.

Another possibility is taking the card names from the cube to excel second column. Add 1 to all lines of the first column. Save as text file and import it as deck.
So you get all missing cards as import error.


The result tuple (Int, Int) is not a Point2D since represents only the start and end coordinates of the Y axis. The X coordinates are not taken into consideration to calculate the boundaries.


welcome to NYC. 5 minutes or 5 days or 50 years, it is a great place to be.thanks.


Good article


yes good call


インデントは徹底しよう


1: M111 unrecognized component name: workflow
1: M190 line is too long (70 > 50)


Replace "Nothing" with ``None``, since that is actually what is returned.


Tassilo Horn <notifications@github.com> writes:

> Is a max width enough?

Ich skaliere die Höhe entsprechend.

Andreas


You can't read the articles before they are launched in Goodies! HOW COULD YOU ? :open_mouth:


nice and elegant.


:grin:
I read it yesterday myself.. was gonna add it but saw your addition


Looks like some line spillovers. Can you re-wrap this text to avoid?


Good idea. Thanks!


グローバルで宣言する必要がないものはローカル（今回で言うとkisetuHantei()の中）で宣言しましょう


/ban @dkhamsing


reack


We're not using API for things like this currently, but I have an issue on a very similar topic https://github.com/alphagov/smart-answers/issues/1708.


Ha, this is part of the CartoCSS parser magic. @javisantana and I discussed this some time ago and ended up deciding that there isn't much of a better approach. If you take a look at Carto's source these kind of ultra-chained accesses are quite abundant.


dodaden "prevrten triagolnik" za da se poklokupva so modot (prikazi/sokrij)


#14 Topics based on [ECMAScript compatibility table](http://kangax.github.io/compat-table/es6/)


:cry:


comma


where are you hiding?


Btw we should have a slack channel!!


Thanks a lot. :+1:


Comment that it is volontary that we don't put sparse_grad in the props.


Oh, ok! :checkered_flag: then


No idea, but there are enough corporates that block bintray that it would cause serious problems for ensime deployment. Can we ask upstream to publish to central? BTW looks like it'll happen https://github.com/scalaz/scalaz-stream/issues/258


Sample manual test paths:

/calculate-statutory-sick-pay/y/additional_statutory_paternity_pay/yes/no/2015-05-21/2015-05-29/yes/2015-03-20/2015-03-27

/calculate-statutory-sick-pay/y/additional_statutory_paternity_pay/yes/no/2015-05-21/2015-05-29/yes/2015-03-20/2015-03-26


orthogonal or orthonormal?


the old code had a bug, good fix all those change.


Having both a *link* method and a controller quickly becomes confusing for others working on the project as you have two places affecting the main logic of the component. Have a look at the version without the link function - e810d312c69e89a4fd3331a25334fbb09a032196
We have updated the file names to better match the meaning of each component -  5b79935

Let us know if you have any questions about the changes made.


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: requirements


1: M111 unrecognized component name: mixer


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: mixer


1: M100 needs more reviewers
1: M101 signature is missing


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: demosite


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: elasticsearch


In hindsight, I'm not sure that was a good feature though. It was to solve a problem that people didn't really have (that is, allowing large integer values to be exactly stored)


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: gitignore


수고염


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
22:1: I100 Imports statements are in the wrong order. from invenio.ext.mixer should be before from six
23:1: F401 'CollectionFormat' imported but unused
29: D101 Missing docstring in public class
33: D101 Missing docstring in public class
37: D101 Missing docstring in public class
42: D101 Missing docstring in public class
46: D101 Missing docstring in public class
50: D101 Missing docstring in public class
54: D101 Missing docstring in public class
58: D101 Missing docstring in public class


those true/false comments are outdated, no? should all be `false`


Thanks Charles!


Good input tests!


remove sparse_grad and comment that this is what we want. See the comment in \__eq__ that you removed.


    func kisetuHantei(){
        if number == 1 || number >= 11 {
            myImageView.image = huyu
        } else if number <= 4 {
            myImageView.image = haru
        }else if number <= 7 {
            myImageView.image = natu
        }else {
            myImageView.image = aki
        }
ちゃんと検証してないけど分岐はこれぐらい短くできるはず


How do you mean? Everything is under 80 chars, but not optimally packed if that's what you mean?


good fix, the old method where buggy.


Students/Lecu C. Alexandru Calin/Assignments/src/Assignment_Multiples.java

Careful, this is a very inefficient way of doing this. Check out [this post](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) and see how you can change this from `O(n)` to constant time!


See comment above on "orthonormal"


98812420-SwimmingPoolHeatingSystem-4805 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1170 of 1170 tests passed)

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


remove that method too.


New file create


The problem is that what you really want to test is that the wkhtmltopdf binary receives the correct and proper arguments, rather that it does what it's supposed to do...


I was trying this out last night as well, were you getting the following warning?

Cannot read property 'mtime' of undefined.


Usually, class names follow the CamelCasing convention, not needing underscore separators.
Good work otherwise


Aha. Great minds and all that :-)


remove \__str__ bellow


LGTM :+1:


But why is cheat-sheet a dependency here?


That sounds great! :+1:


Bah, they show what used to happen…


Habe ich gemacht :D


grep for \__str__ in this file and remove all those that do nothing. They aren't always at the same place in the op.


Ok, if you can open a doobie issue with some pointers I'll see what I can do. I have never used maven so all of this stuff is mysterious and weird.


You can change this in your own clone, of course :)


The file is online.


update


Not sure I understand this. Many molecules can have bond length smaller than the box length and still be straddling the periodic boundary. Perhaps I'm misunderstanding the use of this function?


Looking forward for GitHub for Dummies


add the props and remove methods


Try my new patch too
https://github.com/Anonim47/android_device_samsung_degaswifi/commit/50b3e7a7bce4b2508e05eeb8f2e5811be18304db


rm


카메라 설정을 위해 super.makeStage()를 호출해주셔야 합니다.


fixed


LGTM :+1:


`mb_strlen` would return the number of characters, but in this case the number of bytes is required.

If I were to use `mb_strlen`, the following situation may occur: given the following string: `????+`, where `?` represents a multi-byte character (lets assume that every `?` is a 2-byte character) and `+` is a newline, the first value of `$i` is `0` and the last one is `4` (because there are 5 characters). Bytes `$str[0]` and `$str[1]` are the first character, `$str[2]` and `$str[3]` are the second one and `$str[4]` is going to be the first byte of the third character. The fourth and the last one (which is actually a new line) aren't going to be processed at all.

I could have used the `SqlParser\UtfString` class, but it causes unnecessary overhead.


The initialize method looks good, however the spacing is wonky and the completed method has the following issues:

1. It's named wrong (see below for more information)
1. It takes a task as a parameter to the method. Since completed is an instance method we already have a reference to a task, in this context that task would be called `self`
1. Task.new.completed? would create a new task and ask it if it was completed by calling the `completed?` method, however, `completed` has not been defined and even if it was it would be named wrong.

If you look at task_spec.rb you'll see that the tests for completed look like this:
```ruby
	describe "#completed" do
		context "uncompleted" do
			it "is not complete" do
				expect(Task.new('Pay bills').complete?).to be false
			end
		end
		context "completed" do
			let(:task){ Task.new('Pay bills')}
			it "is complete" do
				task.complete
				expect(task.complete?).to be true
			end
		end
	end
```

In this code you see the tests are calling two methods on a task:

`task.complete` and `task.complete?`

`task.complete` is telling the task that it has been completed so the task should mark itself as completed.
`task.complete?` is asking the task if it's complete.

One implementation of this is to create an instance variable, `@completed` and set it to false when the task is initialized and then set it to true when the `complete` method is called.

```ruby
def initialize(title, priority = 10)
  self.title = title
  self.priority = priority
  @completed = false
end

def complete
  @completed = true
end

def complete?
  @completed
end
```


this 대신 tableStack에 추가했으면 더 좋겠군요. 물론 정상적으로 작동할지는 모르겠습니다


마찬가지로 super.makeStage()를 호출해주세요


:task is not needed here, other than that this TodoList class looks good.


this is simply bad. There was much other stuff in \__eq__ and \__hash__. What are you triing to do? Add format and kmap in it.


This trailing comma is invalid in JSON, removed in [next commit](3b492de8e33cd7e5dccccb6538a80264df4dd950)


Canols is wrong!


config.ini

[Credentials]
Username = foo
....


Try to do this in the application configuration.


http://lammps.sandia.gov/doc/change_box.html

Lammps calls it orthogonal, and I think I've referred to it everywhere else as that, but a quick poll of my office got me 3 different names for cuboid boxes :)


keep those methods and they don't do the same as the default one.


Beautiful


idem, don't do as the default, so keep them.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 152](https://teamcity.labkey.org/viewLog.html?buildId=261309&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:06:31
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

```


It's not to pack them inside the box, but to make sure the molecule DOES straddle the box.. so the opposite of what you're thinking


There is an error here, see #34


Sorry, en des de "por asignatura" es "por carrera", es decir un solo plan activo por carrera


there's two other occurrences of lower case blackberries nearby, if you re-edit you're PR will automatically update then I can accept


Got dragged into a meeting half way through scanning this commit, but turns out the rest is all cool. Glad suggestions are making an impact. :)


@flangofas this project is currently setup to work on CentOS 7 only (due to some significant differences in package names, tools, etc).  CentOS 7 uses yum.  And even later, when yum will be replaced with dnf, like in the current Fedora, there still will be an alias and the command will work.  You'll just get a warning that the command was redirected to dnf.


I don't think this is really much of an improvement, also I thought of an instance where normal modules can have multiple actions (but we can fix that once this PR is merged).

But I think revert this change for now. (sorry)


pasted from https://github.com/ceph/ceph/commit/121bf976cff1e7f1aa518e5f7e78e66969cd7665#commitcomment-12117291

@tchaikov wrote

> why not just test `op.extent.length == oi.size` here instead of `op.extent.length >= oi.size`? easier to understand: if we read the whole object, we pass CRC to the context for verifying. and in `FillInVerifyExtent`, we set the EIO if the calculated CRC does not match if we have an expected one.


@SomethingSexy Sounds like https://github.com/karma-runner/karma/issues/1494, any info you have on this, I much appreciate it


here


and here, after key-only


pasted from https://github.com/ceph/ceph/commit/121bf976cff1e7f1aa518e5f7e78e66969cd7665#commitcomment-12117291

@dzafman wrote

The only reason we check against the oi.size again is later we have the actual bytes read.


Sorry about this commit. I screwed up by not pushing my commit for 0.13.3 to Github. Fixing.


@jonenst I think you're right! Fixed in e280bafabeffc32eee147ce8d15b02303c966228. Thanks!


Ah...now I understand. It's early morning here. :D


почему get?


This answer is correct, though the additional `puts` statements are unnecessary. You also could have written this as :
```
def is_of_age? age, minimum_age
    age > minimum_age
end
```


не используется


Might it be clearer now to drop the `SingleObjectMixin`? I guess  you'd then remove this line and restore the `get` method with the first line being:

```python
self.object = self.model.objects.get(slug=self.kwargs['slug'])
```


LGTM :+1 and I have discovered the need to add a space before the :  for example :1234:


It's just GitHub then. No worries.


My bad.  c4ea4e36d7669cede31842da4ab5f999c4045dc9

-[Unknown]


I did this because the sendRedirect() method doesn't work; it doesn't redirect to the specified redirect path. Because of this, after editing and saving a user/org, it redirects to the add new user/org page instead of the edited one. When I track it down, it's because it calls the sendRedirect() in LiferayPortlet https://github.com/liferay/liferay-portal/blob/master/portal-service/src/com/liferay/portal/kernel/portlet/LiferayPortlet.java#L107 and it always uses the redirect param in the request instead of using the given redirect path https://github.com/liferay/liferay-portal/blob/master/portal-service/src/com/liferay/portal/kernel/portlet/LiferayPortlet.java#L380-L390.


Ha! In that case no big deal. :D


Looks good, but I'm not sure if I wouldn't put some 128 Bit aes ciphers first to increase performance.


Ya it looks like that issue.  I am still digging in.  This are the karma specific dependencies I am running now

    "grunt-karma": "~0.12.0",
    "karma-coverage": "~0.4.2",
    "karma-requirejs": "~0.2.2",
    "karma-phantomjs-launcher": "~0.2.0",
    "karma-chrome-launcher": "~0.2.0",
    "karma-qunit": "~0.1.4"
    "karma-jasmine": "~0.3.6",


thanks for taking the time to add these, however generating the Project is _not_ the same as building it (in fact the build gets kicked off later in some cases) so could you please remove these events?


Thanks for the suggestion. Done in 865766d9fc6c1911a570d0de9c0e911815c63bf5


I'm going to go ahead and default the txindex to on across the network, otherwise some of the peers are going to see budget items disappear / finalized budgets that report there's an issue (because they can't find some of the proposals in them). Also, I think this was an issue for DS/IX from time to time. This should improve things on the network significantly :+1:


Values of width and height attributes are written as plain integers. Could you please remove the `px` notations? Thank you!


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1238 of 1238 tests passed)

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


Unused variable.


Maybe we should remove the Line "Sonne"


actionResponse.sendRedirect() doesn't work either because it also ends up in LiferayPortlet's sendRedirect() and tries to get the redirect path from the request.


I think it's most technically correct to look at the bonds as vectors and compare each component against 1/2 box length for that component... but assuming that box lengths are large compared to bond lengths.. then this is safe!


Looks great! Thanks!


опечатка -> `config`


I prefer configure this variable to '/var/opt/influxdb/meta' for default.


Думаю, можно не писать темплэйт в одну строчку, а отформатировать для читаемости.


@alexdeleon review my comments please :)


Thanks :)


To a certain degree, yes, but we actually get some benefit from calling through to wkhtmltopdf - it reveals when the command line arguments have changed. I would not be comfortable with a test suite the only tested that it received the correct and proper arguments - maybe if we had more development power, but not right now.

@romanbsd We can currently do that through a regex over `PDFKit.new('blah').command`. But while where we use regexes and where we call through does need looking at, I don't feel like can safely release to our users with only testing that way.


I'll add a timeout


very nice. in real life, ternary operators are mainly used when trying to show off to friends. :wink:


The "action" already has the `req`. Does this really gain that much?


Yep, thanks for the input. I axed this in favor of just defaulting the txindex to enabled.


why? There are defaults and if you don't supply parms, it'll use the defaults and they get printed out.  If you goof with the conf entries in the file, the log statement will print them out.


あと演算子の優先順位についても、&&は<=などよりあとに比較されるので、正しく動作しますが、意識的になったほうがいいかもです


Looks good. When everything is green i'll merge it.


Is there any evidence, that Python requires an absolute path here?


hmm. this doesn't seem like it would compile, to me.


I think you might have a compile error - on line 55 you have an `else`, but I don't see a preceding `if`!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.354](http://teamcity.monogame.net/viewLog.html?buildId=8787&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


It uploaded with the if/else loop file. I just kind of gave up here.


I thought this simply dropped memory off the end of the vector, and free just unused information?


maybe add a default with a warning/error just in case ?


teste Comentario


# [Managing the Activity Lifecycle](https://developer.android.com/training/basics/activity-lifecycle/index.html)

![A simplified illustration of the Activity lifecycle, expressed as a step pyramid. This shows how, for every callback used to take the activity a step toward the Resumed state at the top, there's a callback method that takes the activity a step down. The activity can also return to the resumed state from the Paused and Stopped state.](https://developer.android.com/images/training/basics/basic-lifecycle.png)

Implementing your activity lifecycle methods properly ensures your app behaves well in several ways, including that it:

Does not crash if the user receives a phone call or switches to another app while using your app.
Does not consume valuable system resources when the user is not actively using it.
Does not lose the user's progress if they leave your app and return to it at a later time.
Does not crash or lose the user's progress when the screen rotates between landscape and portrait orientation.

the activity can exist in one of only three states for an extended period of time:

Resumed
In this state, the activity is in the foreground and the user can interact with it. (Also sometimes referred to as the "running" state.)
Paused
In this state, the activity is partially obscured by another activity—the other activity that's in the foreground is semi-transparent or doesn't cover the entire screen. The paused activity does not receive user input and cannot execute any code.
Stopped
In this state, the activity is completely hidden and not visible to the user; it is considered to be in the background. While stopped, the activity instance and all its state information such as member variables is retained, but it cannot execute any code.


Yes, in Actions and in MooseApp they are pass by value.


ack


aaaa melhorar


sd


asdasd


That's correct...


This fails when running the executable on Linux (de_DE.UTF-8) in a path with non-ascii-characters, e.g.umlauts. Why are you converting argv[0] without considering locale, while for the other args, you are considering locale?


ccc


There you are: https://gitter.im/iOS-Goodies?utm_source=share-link&utm_medium=link&utm_campaign=share-link


cc @jaeilepp FYI


Why is this needed? Right now, uniqueId is temporary, debug only. We'd need to make a decision to keep it for good if we make other uses for it.


following


Add in the commit description that the domain for rendering the payment block, used in portal_sale (sale order emails), has its domain updated about the environment, that makes more sense (!= website or ecommerce).


@nitesh1989 You're volunteering, right? =D


Can't you return on prroduct.template.ids ? Batching the method ?


As commit message says, I'm using this for debug purpose: I can make custom breakpoints in IDE that trigger only on some condition. The problem is that sometimes it's buggy and I cannot access `uniqueId` field, while I can reliably access `hashCode`.


corner case


Hm never seen this before, thanks


`You've tried to access iOS-Goodies's private organization room and you do not belong to this organization.`


`str` is redundant you are anyway doing it in the next line


No problem.


Ops, that is for the organisation. This is for the repository https://gitter.im/iOS-Goodies/iOS-Goodies


sum is uninitialized before its first use in this program


#1483 also fixed with this too?


ok


@ptrv Here's [a gist showing the problem I'm seeing](https://gist.github.com/abingham/5fc5b2bc34645e45551c).


ㅣ


From what I can judge on: :+1:

Please add testing instrcutions to https://github.com/owncloud/client/wiki/Testing-Scenarios-2.0


An easier way to do this is:

`@tasks = []`

which is usually what you see in ruby, but your way is totally correct.


Another way to do this is:

`tasks << task`


Est-ce vraiment nécessaire de faire le test sur jQuery ?
Si je ne me trompe pas, quand jQuery est inclut il défini à la fois $ et jQuery, donc si $ n'est pas présent jQuery ne devrait pas l'être.
Je pense que l'ajout du require devrait suffire non ?


Good Job!


find . -name "*.fastq" -print0 | xargs -0 -I {} mv {} fastqfiles/


You can define which activity to use as the main activity in the Android manifest file, AndroidManifest.xml, which is at the root of your project directory.

```xml
<activity android:name=".MainActivity" android:label="@string/app_name">
    <intent-filter>
        <action android:name="android.intent.action.MAIN" />
        <category android:name="android.intent.category.LAUNCHER" />
    </intent-filter>
</activity>
```


Fixed on https://github.com/Samsung/GearVRf/pull/163


Isn't better if the `data.probability.true >= 0.8` is also replaced by a test on the rounded value? Otherwise users might be a little confused when they find an edit with 80% of probability (from e.g. `Math.round(0.795 * 100)`) not being styled by `ores-alert`.


AFAICT, this won't work for an atomgroup that contains more than one molecule, as only the molecule with the ``reference_atom`` will get traversed by the loop since it traverses through bonds. This is a problem if, say, you want to make all waters in the system whole.


Rocco, I'm trying to understand your changes but it would be faster if you could just give me some hints on how to see the problem and what it solves for each changes.
Thanks :)


哎呀竟然忘了…


Looks like duplicated code


Why did you strip the "number" part of the variable? I can still merge this but the original variable name makes for a more readable code.


It looks like this test can't fail because hllB isn't used in bitVectorB. It was already the case before anyway.


Please mention in the commit message that UCL is disabled (to save space in the SPL), and that the resultant full binary is less than 0.5mb.

Can you rename fix-gpt-checksum to amlogic-fix-spl-checksum and install it with that name too.

Please add another commit that enables CONFIG_EFI_PARTITION


I was thinking that we should do this as part of the gem itself, not just the tests. (Anyone using this gem on oraclejdk7 is going to make this same change to their config, so we may as well build it in.)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1241 of 1241 tests passed)

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


This is a bad commit.   The canonicalAudioStreamBasicDescription is the internal output/destination format and is always 2 channels.  asbd in this proc is the *input* number of channels.


It's CLASS:onSpawn(client)


This is just a link so it doesn't need inside a div or have any special formatting


@brianr I've been looking at the way to detect the Oracle java.vendor and the version. And `java.vendor` property sometimes is not what we'd expect.


I thought you'd want to quote everything **but** keywords. Isn't this doing the opposite?


Now that you know that the value of `if` statement can be directly used.
is `any()`  from python built-in functions any usable.


This is just a paragraph, so you should replace the div with a p tag


Sturm commit, not action, my mistake


When a record has no url to the website (payment acquirers or delivery method), its default url is still "#", as it has not been overrided. In that case, we want the button to be a simple toggle, to be able to publish the record.


@gevou It's better to save `// XXX` comments for missing functionality. We should create github issues for missing documentation.


You must implement the `onCreate()` method to perform basic application startup logic that should happen only once for the entire life of the activity. For example, your implementation of `onCreate()` should define the user interface and possibly instantiate some class-scope variables.

```android
TextView mTextView; // Member variable for text view in the layout

@Override
public void onCreate(Bundle savedInstanceState)
{
    super.onCreate(savedInstanceState);

    // Set the user interface layout for this Activity
    // The layout file is defined in the project res/layout/main_activity.xml file
    setContentView(R.layout.main_activity);

    // Initialize member TextView so we can manipulate it later
    mTextView = (TextView) findViewById(R.id.text_message);

    // Make sure we're running on Honeycomb or higher to use ActionBar APIs
    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB)
    {
        // For the main activity, make sure the app icon in the action bar
        // does not behave as a button
        ActionBar actionBar = getActionBar();
        actionBar.setHomeButtonEnabled(false);
    }
}
```


No.  This is just a coarse heuristic for how much to pre-compute.  This will probably eventually become a lot smarter, but doing yearly chunks seems like an okay first guess.


use strong tag here


Hi @ianbrunelli
Thank you for the tip.
I just gave it a quick try. This is easier yes.

One caveat however is that when the extension does not exist for the actual gnome shell version, it just fails to install. In contrast, when you provide a version on the command line, when the extension does not exist to it, it displays a list of alternative version to choose from.

It would be nice to have the script automatically select the nearest previous version available.
For instance, when running gnome-shell 3.16.3, but highest version available for an extension if 3.14, then select 3.14 without an error (maybe with a warning?)

By the way, thank you for the good work on this script. :+1: