@someara Could you please explain - why you've replaced `unless` with `if` ? Seems like it reverses the behavior of `node['yum'][repo]['managed']` attribute.


Question: I understand Linux client will be allowed to upload a file containing these characters in the filename. What happens on the Windows client trying to download such files?


:green_heart: Using minified versions is an excellent way to improve the performance of the application.


I'm not sure it was a very conscious choice to use either `if/elsif` or a `case` statement. Having looked at the code again I can't see an obvious pattern as to when I've used one over the other. I agree that it would be better to be consistent but I think I'll leave that for when we hopefully get to improve this Smart Answer.


CConfigurableElementAggregator has a quite interesting (ie not obvious) semantic. but you are right that it could be asserted.


me lukas*cz ok thx man

---------- Původní zpráva ----------

Od: HungHaoHiep <notifications@github.com>

Komu: Levak/warfacebot <warfacebot@noreply.github.com>

Datum: 9. 10. 2015 10:03:25

Předmět: Re: [warfacebot] Update game version for EU/NA/TR to 1.1.1.3718
(481939d)

"

HI Levak !

Update game version for EU/NA/TR to 1.11500.3725.17500

https://www.warface.com/en/news/recent/warface-hotfix-8th-october-2015
(https://www.warface.com/en/news/recent/warface-hotfix-8th-october-2015)

And Update game version for VN to 1.1.1.328

http://warface.goplay.vn/news/details/2982/gioi-thieu-big-update-3-4
(http://warface.goplay.vn/news/details/2982/gioi-thieu-big-update-3-4)

—

Reply to this email directly or view it on GitHub
(https://github.com/Levak/warfacebot/commit/481939d83c4adf05c737abb4d73d380d9e4816e8#commitcomment-13680680)
.

"=


... :cry:


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


xx


@thomasjo we still need an .eslintrc to tell eslint to use this parser though, no?


> Minor: The `should`'s in this test read a bit strangely, I wonder whether it might be better to use `test` instead.

I went with `should` because that's what the other integration tests use but I agree they look odd. I'm inclined to leave them as they are in this branch and consider changing them in future.


:yellow_heart: The tasks defined (except the `default` task) are executed in parallel. You'll need to make them run sequentially so that changes are reflected on server refresh.


Don't we still end up logging twice?


:green_heart: This is excellent work.


the windows client will not try to download such file because for windows it will still be ignored


@moscicki This algorithm applies to both ends, so files with these characters in the file name will be ignored on Windows. No transliteration is being attempted as this is a recipe for disaster if the mapping gets lost, amost other issues that would arise from this.


I committed a slightly modified version of this; I moved the lines for the subdirectories into .gitingore files in those subdirectories. Some of the directories already had a .gitignore file. For others, I added one.

Thanks for spotting this!


Should the second snippet say (``&ampl``) here?


CBECC (kbenne) - i386-Windows-7-VisualStudio-12: OK (2325 of 2349 tests passed)

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


> Idea: I don't think it's important to do in here, but I wonder whether there would be merit in modifying add_response to take an expected node name which it would also assert - thereby converting the comments into code...? We could even have an add_responses method which took a Hash of question name versus response...?

That sounds good. I've added a [Trello card to triage](https://trello.com/c/tJ1lXFyL/105-modify-add-response-so-that-it-takes-the-name-of-the-node-it-s-expecting-to-be-on).


Added another line in hub wanting to see it in local


This is a hub commit


It would be great if the text said if this was case sensitive or insensitive, e.g. " ... or if the case sensitive phrases ..."


I need a variable name meaning "the new breadcrumb wrapped by jQuery". I could call it `jQueryWrappedNewBreadcrumb` but it's too much IMO.
Since I've seen, on the web, some examples using `$` as a prefix for DOM elements wrapped with jQuery, I thought it was a good idea, since `$`, in this context, is bound to jQuery.
I simply do not override the variable because it comes from a function parameter and it could be seen as a bad practice. It's part of the checkstyle: http://checkstyle.sourceforge.net/config_coding.html#ParameterAssignment


:yellow_heart: It is recommended that directives be placed in a separate directory to improve the modularity of the application.


> Did you consciously not move the relevant tests over from SmartdownPlugins::PayLeaveForParentsTest in this and subsequent similar commits...?

Nope. It was a complete oversight that I didn't copy them. I'll add the relevant tests to this branch.


In reading through these changes, I noticed that in OracleDbSupport.java, there is this code:

    @Override
    protected String doGetCurrentSchemaName() throws SQLException {
        return jdbcTemplate.queryForString("SELECT USER FROM dual");
    }

This SQL won't reflect any changes made by ALTER SESSION SET CURRENT_SCHEMA = ...;
An alternative might be:

    select sys_context('USERENV','CURRENT_SCHEMA') from dual;

Docs: http://docs.oracle.com/cd/B19306_01/server.102/b14200/functions165.htm

Does this affect the issue this commit is aimed at?


- `px` is not a very good variable name (what does it stands for?)
- so, you convert a position into time. While this may be correct (at very first sight, I'm however surprised that the computation does not involve any physical constant like `G`), I believe you should add a detailed comment explaining what's going on...


Why are you pushing these?


@thong-dang-mulodo please remove .idea folder because it is generated by your editor.


That's kinda bad. Could you somehow run the seeds from the feature spec? `bundle exec rspec spec/features` should work out of the box.


增加对小些字母的支持


We do, but this way the cd command and anything else the user puts in the
script is logged, and if krun exits with an error message that will be
stored in /tmp.
On 28 Oct 2015 10:55, "Edd Barrett" <notifications@github.com> wrote:

> Don't we still end up logging twice?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/softdevteam/krun/commit/a269306b113cced3aff6f47a96b5437126b945b0#commitcomment-14035198>
> .
>


Yeah, reported this with matplotlib/matplotlib#4262 but It obviously only is fixed starting with 1.5.


汪正波新的波形数据只含有数据，没有其他的控制信息，而且每个深度点对应一个数据文件，每个文件包含了一个大包的数据，


This solves it, but I find it annoying :frowning:. See #755.


现在的文件格式fopen操作叫方便，以后也可能换成freopen


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


Steve,
I might be starting to figure this out.  I am back on the rails console, no need to respond until I do some more work on the console.


Hello There


The templates in ```torchbox/templates/``` should be moved to ```tbx/core/templates/```


Hiiiiiii


Do we prefer ```&&``` or ```;``` to separate bash and shell commands?


I guess it's down to personal preference.

I feel that any given project should use either one or two error types (one for the library's API functions and the other for internal functions to use) since it makes working with the project easier.

I see your point about `get` being able to return some unrelated error, but for me this would be like objecting to a function which doubles a number returning an integer, since integers can be odd and this function has no business returning an odd number.  I wouldn't expect to see a new `even` type created for this function to return, and that's how I view the errors for all Vault functions.


Hello,
 this is a great addition, but shouldn't content type default to 'application/octet-stream' if it is not text?
 Also please add 'image/svg+xml' for the '.svg' extension


Thanks for the logs.
So everything is ok when pressing home button + restoring.
However I don't understand why the core still flip buffers after stopping emulation,
it's not happening on my device, I checked on PC and it's not happening either.

If this is happening with all video plugins this looks like a core issue.


I could be reading this wrong, but I think you may not have account for the panels. I read it as `for each die in dice` `for each die in dice` `roll` but I could be wrong.


Since you are creating a special case for the calendar button eliminate the calendar specific code from this line as it will never be executed


hey There


&& is preferred!


use getTranslatedString here: always translated strings visible on screen


Casting to int (or int32_t) before negate should be fine here. Tempting to just change the type of uiBin to avoid the cast altogether.

The negate is optimized away by GCC in the hot path (< 0 comparison ends up as an sra rather than an srl to get the resulting bitmask).


Shouldn't this include be at the start of the script?


Do not translate any strings, use the ones that already exist as they are.


Hey Bud


getTranslatedString


Sometimes you use `TESTINGBOT_{KEY,SECRET}`, sometimes (karma launcher) `TB_{KEY,SECRET}`. Maybe you could agree on one of them.


Nit: `the required` not `required the`


This is wrong, you can't eliminate the modules, they are required inside the calendar. You have to eliminate them only in the create button but not on the calendar.


@thong-dang-mulodo please edit function name follow standard format: Method names and function names must be declared in camelCase.


Checkmarx scan results 28 Oct 2015 04:12 UTC -07:00

High
Medium
Low
Info

New
26
17
30
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

41032b39802e86ae32b64af866e630eaf4940700	@amitchen22


no need to change any translations for this fix


Checkmarx scan results 28 Oct 2015 04:12 UTC -07:00

High
Medium
Low
Info

New
6
33
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

64950ba3148081eaf42ccc64e6f9f6009c3b5c5b	@amitchen22


What about adding a static_assert ensuring that `T` is a descendant of `K` using `is_base_of`? I know it will anyway be checked whether `mWrapped.*method` exists when trying to compile, but adding the assert will lead to much clearer error messages.


The same problem with latin2 characters. eg.: łóźć


I tested the change. In general it seems to be working correctly, the only issue I found is that now the calendar create functionality doesn't work: if you click in the calendar the create menu must appear with all the options, that is broken now.


This could have been passed as an argument from the router to the function to save this call and simplify the code.


True. Lemme try: https://github.com/CodePadawans/ataru#setup


I don't think I understand the  blackTile() changes. Replacing the static data with non static makes sense, otherwise you'd only initialise the black tile on first use, which might be the wrong size for subsequent uses. But then I don't get why there's also a mutex? Surely each FlatTileWriter instance is used for only one execute(), and parallelGather() ensures that the calls to FlatTileWriter are serialised anyway?


Note that using "new" is never a good idea since in the future new is not new anymore ;)


Attribution is to the Foundation rather than John in the 2.1.1
```
Copyright 2005, 2014 jQuery Foundation, Inc. and other contributors
```
- https://code.jquery.com/jquery-2.1.1.js


No, but if it catches more corner cases it could be a welcome addition.

Want submit a PR? (Make sure to run all Oracle tests on your machine first)


The reason being how the shell will handle errors, right? Using ``;`` will hide errors:

```
$ echo "hello" ; false ; echo "world"
hello
world
$ echo $?
0
```

Using ``&&`` will abort on an error:

```
$ echo "hello" && false && echo "world"
hello
$ echo $?
1
```


> Have you confirmed that this kind of whitespace change (near a markdown table?) has no significant effect on the generated HTML.

Yup, I can confirm that the generated HTML is the same:

```
<thead>
    <tr>
      <th>Dates</th>
      <th> </th>
    </tr>
  </thead>
```

## Before

![a-before-conversion-to-ruby](https://cloud.githubusercontent.com/assets/6556/10786971/79e58f1a-7d65-11e5-9a6a-3453332c208b.png)

## After

![b-after-conversion-to-ruby](https://cloud.githubusercontent.com/assets/6556/10786973/7dcf26b8-7d65-11e5-9b54-32387dd44556.png)


formatting


To do: categorizing data (by author, series, date), uploading data/stories, log-in system, sign-up system, searching system, main landing page (dashboard), user profiles (pictures, about sections)


Still wrong :P


@Gianluca-Addis when you write a list you should add a `space key` or `blank ` between the `.` and the first word.

1. Good example `2. Put ....`
2. Bad example `2.Put ....`


"Pasing" -> "Passing"; "strResult" -> "error" (btw, you already explain this above).


> Given that this change seems to appear quite a lot in this commit, I think it might be good to mention something about it in the commit note.

Done.


ffs stop using github editor


local.py是个什么鬼


TeamCity Jarvis Framework :: CI-GitFlow [Build 49](http://build.prxm.it:8811/viewLog.html?buildId=4366&buildTypeId=CqrsFramework_CiGitFlow) is now running


https://github.com/DarkstarProject/darkstar/issues/2153 even!


This particular comment should go the the method's documentation instead.


为毛要写两次。。


Isn't this test data file already in the platform?


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


我记得这里的result应该是个列表吧。。。。而不是字典。。


You're completely right. This is a mistake to use MID to do that.
I would like to make difference between the first response and the notification, but this is not really needed.
I will revert this commit.
Thx a lot @boaks.


tes commit


N_first : starting node index
N_user : how many adjacent node indexes you want to have

(img.data).T has originally 59412 nodes, and time-series raws of length 1200. First 29696 nodes of the data matrix refer to left hemispherical nodes. The remaining 29716 nodes correspond to right hemispherical nodes.

For the lefth hemsp. nodes:
N_first = 0
N_user = 29696

For the right hemsp. nodes:
N_first = 29696
N_user = 29716


file is called `superfish.js` rather than `jquery.superfish.js`


Hi,

This commit is generating an error on our end, after our latest composer update.
We have tested against the different versions/commits since our previous update on c3933ad (1.0.2) August 31, 2015

This is the error we get
` [2015-10-28 11:28:48] request.CRITICAL: Uncaught PHP Exception Symfony\Component\Debug\Exception\ContextErrorException: "Warning: spl_object_hash() expects parameter 1 to be object, null given" at /Applications/MAMP/htdocs/extranet/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/UnitOfWork.php line 2492 {"exception":"[object] (Symfony\\Component\\Debug\\Exception\\ContextErrorException(code: 0): Warning: spl_object_hash() expects parameter 1 to be object, null given at /Applications/MAMP/htdocs/extranet/vendor/doctrine/mongodb-odm/lib/Doctrine/ODM/MongoDB/UnitOfWork.php:2492)"} []>`

I don't know why, but it happens on a Form where we have a collection of embedded documents.

            ->add('slots', 'collection', array(
                'type'          =>  new SlotType($this->currency),
                'allow_add'     =>  true,
                'allow_delete'  =>  true,
                'by_reference'  =>  false,
                'cascade_validation' => true,
                'error_bubbling' => false
            ))


`ENoSuchParameter`


I was talking about the sporadic EBADF.

let's see if ```system``` behaves better then backticks - probably since it does not catches STDOUT


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.3](http://build.prxm.it:8811/viewLog.html?buildId=4366&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 128 Build time: 00:04:05


TeamCity Jarvis Framework :: CI-GitFlow [Build 50](http://build.prxm.it:8811/viewLog.html?buildId=4367&buildTypeId=CqrsFramework_CiGitFlow) is now running


Exactly, otherwise we would need to add magic like "set -e".


one letter variable in a large scope is meh.


I have used the embeded wall but the wall display 3 rows ...
I have used the parameter numRows = 2 for displaying 2 rows but it display always 3 rows...
How I can display 2 rows?
![screendemo1](https://cloud.githubusercontent.com/assets/7901301/10787125/df50a91c-7d6e-11e5-9cc0-93a0cca1c716.JPG)
thanks
Vincenzo M.  Italy


for consistency: please use ".h" as suffix.


I have the processor count variable named differently: `ProcessorCount_cmd_getconf`.
Additionally there are some variables I don't know the meaning of.
`CMAKE_LIBRARY_SEARCH_PATH` and
`OGS_ADDITIONAL_SUBMODULES_TO_CHECKOUT`.

Nice touch :nail_care:
:fast_forward:


@rgbkrk please use `max-width` instead so this still works on mobile devices.


this should probably be `:0`, no?


afais toolchain-funcs is unused


It is.
I have only recently become aware that tests in a module, use test data in core.
I will remove and run the tests and see if they still work.


Could expand this test with explicit use of a gap character other than the minus sign, e.g.

```
seq = Seq.Seq("ATG~~~AAACTG", Gapped(IUPAC.unambiguous_dna, "~"))
self.assertEqual("M~KL", seq.translate())
```


Idon't know really why, but the maker tends to generate the tint texture bigger than required when refreshing:
In rpg_core@3439 the method:
   Sprite.prototype._refresh = function()


Be careful with `assert`, since it bubbles up all the way to the actor system's `uncaughtExceptionHandler` and exits actor system and jvm. Ok to use when you are 100% sure that it will never happen, but otherwise `require` is better.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.2.0-unstable.3+3](http://build.prxm.it:8811/viewLog.html?buildId=4367&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 128 Build time: 00:03:25


I don't think exception messages should have line breaks, since they typically end up as log messages and log messages with line breaks are a pain.


Theres also `jquery.wiggle.js` and  `jquery.slideto.js` which don't appear to have been picked up under brooklyn-jsgui


Test cases were failing earlier, because the emit_event method assumes a serialized object for both exam and attempt. This change fixes the tests.


LOGGER is never used in the file.


You're right Martjin, I'll add this check


are we sure we want to use dynamic casts? Consider android.


Thanks for the PR! Could you change the exception to be a NPE, more like `throw new NullPointerException("A non-null field name is required");`? Cheers


`Startseite Protokoll` finde ich nicht so klar.

Title: `Beginn Seitennummerierung Protokoll`
Description: `Bitte geben Sie die Seitennummer an, mit der Seitennummerierung im Protokoll beginnen soll.`


Theres also jquery.wiggle.js and jquery.slideto.js which don't appear to have been picked up under brooklyn-jsgui


ps: please squash commits (or use `git commit --amend`), an see brett19's comment about migrating to gerrit


ElementHandle.h


Yes, ``XX`` means two letters. The code is checking for any two characters after ``global``. Later on the code tests that these given two letters are correct values.


merge


if we end up with a lot of these we should consider using the boilerplate plugin, we use it in same places in Akka, e.g. https://github.com/akka/akka/blob/master/akka-actor/src/main/boilerplate/akka/japi/tuple/Tuples.scala.template


no need for the "else" (since there is a return in the if)


Hurra! Ser bra ut dette! :)


Hat das einen Grund warum nicht devdependency?


> I think it might be worth including the paragraph (or at least the first sentence of that paragraph) from the previous commit note explaining where the new calculator methods have come from in the note for this and other similar commits:

Done.


Giiiiidameg, for et vakkert prosjekt!


- px stands for previous x.  p<stuff> is used all over the code of Morse to denote previous<stuff> (see imu, velocity sensor and so one).
- we have dx = vx * dt, where vx == 1.0 by construction (assuming setLinearVelocity is working properly). So dx = dt. We have vz  which is influenced by G (but it comes later). I can put a plane under the cube to make sure it does not fall, but it complexify a bit the logic.


Since you've defined setSettingsAsXML in the same PR, you could make it take a reference instead of a pointer.


the 2 dots in the example were meant as spaces not literal ..


Would I be right in assuming that the IP address here will be in the "private" IP range for docker containers (172.16.x.x or whatever)?


Better to be safe than sorry. :)  Cheers!


Dammit. I'll try commenting on the right thing this time...

I'm confused as to why IE8 won't honour the output of `margin-bottom:1em;` — is there something I'm misunderstanding here? (100% possible today.)


I'll clean the newlines. If you send me an include for gtk apps, I'll put it in.


> In the commit note you say "salaries include a decimal place __where necessary__", but it looks as if Ruby Smart Answers __always__ includes a decimal place.

Commit note updated.


@Nothing-Dev, not to be an ass, but we were all noobs at one point. Comments on github commits isn't the best source of knowledge. Before android's more widespread use of vectors in lollipop, I had zero experience with vectors. I learned how most people do; Google search is an important asset to anyone looking to tinker. I'm not going to walk you through it, but I'll get you started. Use inkscape to create vectors. It's not as simple as just "transforming" into a vector. Hi res images can usually be traced in inkscape using "trace bitmap" tool. You'll find that you usually get better results just creating the vectors by hand using inkscape's various tools to create and edit paths. One you have your svg vector, convert to Android VectorDrawable xml using http://0xd34d.github.io/svg2android/


Please add some test (if there is not one already).


請問, 所以如果我是一次新增一個test case的情況, 這時候做重構就是沒有問題的, 對嗎?


Since you had also edited the file, I got a MERGE CONFLICT when I tried to do a pull. I had to resolve it manually, but left your other changes intact - fixing the a tag code syntax and adding your avatar. then I committed the result and tried to push again. We'll talk about Merge Conflicts Thursday


Are you implying we should check out the branch instead @lvca ?


Should this be `$(host)` instead of `$(url)`?


@rampant1018 I meant the later.


good api


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


very good


Just a query on the plugin - I can see this ties up with the entry in the overrides.yaml, its not clear how/where this file gets picked up from the [license-audit-maven-plugin](https://github.com/ahgittin/license-audit-maven-plugin) readme?


There is an entry for `jquery.wiggle.min.js`.

`slideto` (and `dataTables.extensions`) we have rewritten ourselves as the provenance of the original code is unclear.  I've updated the messages in these files to make them clearer.


@thedaniel Nope, check the link to my package.json file. But to save you some time :wink:

```js
  // ...
  "devDependencies": {
    "babel-eslint": "^4.1.3",
    "standard": "^5.3.1"
  },
  "standard": {
    "parser": "babel-eslint",
    "globals": [
      "atom",
      "latex",
      "afterEach",
      "beforeEach",
      "describe",
      "expect",
      "it",
      "jasmine",
      "runs",
      "spyOn",
      "waitsFor",
      "waitsForPromise"
    ]
  },
  // ...
```

Obviously we don't need the `latex` global though.


what's this about?


We can make this a strict check for consistency


A mistake, most likely, fixing.


yep :)


i got  error like so i removed it

libsepol.sepol_context_to_sid: could not convert u:object_r:mmc_block_device:s0 to sid
/home/aapav/cm13/out/target/product/ms013g/obj/ETC/file_contexts_intermediates/file_contexts:  line 779 has invalid context u:object_r:mmc_block_device:s0
libsepol.context_from_record: type mmc_block_device is not defined
libsepol.context_from_record: could not create context structure
libsepol.context_from_string: could not create context structure
libsepol.sepol_context_to_sid: could not convert u:object_r:mmc_block_device:s0 to sid
/home/aapav/cm13/out/target/product/ms013g/obj/ETC/file_contexts_intermediates/file_contexts:  line 780 has invalid context u:object_r:mmc_block_device:s0
Invalid context file found in /home/aapav/cm13/out/target/product/ms013g/obj/ETC/file_contexts_intermediates/file_contexts
external/sepolicy/Android.mk:157: recipe for target '/home/aapav/cm13/out/target/product/ms013g/obj/ETC/file_contexts_intermediates/file_contexts' failed


@thomasjo ah, i added the `devDependencies` but not the `standard` key!


For newer systems (starting with Nehalem), most BIOS implementations lock the MSR registers that contain the control bits for the features. I havn't tested it on the most recent systems but I assume the BIOS vendors havn't changed the BIOS behavior.
The logic of likwid-features is still included in the source code, so there should be a big problem to integrate it again. At least to see which features are enabled/disabled in the BIOS.


Thanks @ddemidov - I knew `set` was the key to the solution once I saw the bug :-)


NullPointerException too can be used here. I added IllegalArgumentException since this exception is already used in put() for checking other parameters. Primary purpose for this exception is to alert the developer that he is not checking the 'key' name while calling put().


I don't recall whether or not the initStream and createStream are covered in the [RTMP specification](http://www.adobe.com/devnet/rtmp.html) from Adobe or not, what we started with was reverse engineered many years before it was released. The "play" method is the intriguing operation since the values decode don't appear to make sense.
The stream id and channel code needed updating and clean up anyway, so this bug with AMS gave us the perfect opportunity to take it on. The Number type allows us to be compatible with whatever Adobe throws at us in the future, probably...


oh, binary data in version control ... why? How to modify this data? Why not store the sql-dump in version control?

How to proceed?


Jævla bra paragraph!


`checkGetValidity` duplicates the `isParameter` check.


`checkSetValidity` duplicates the `isParameter` check.


Is this actually causing execution problems?

Or is there just a warning?  If so, I think can probably remove the warning.

If the simulator is not stable after 30 seconds, then I can imagine you might have a problem - if the simulator is in the new iOS 9 "booting" state, trying to install/uninstall/launch apps will probably be unstable.

Please let me know.  I want to solve this issue.

https://github.com/calabash/run_loop/issues/307


Perhaps we should have (a representation of) the expression itself here. And maybe a comma after 'expected' above.


coding style: `} else {`


Hey, [JsonResponse](http://api.symfony.com/2.3/Symfony/Component/HttpFoundation/JsonResponse.html) is available and we can get from:
```php
$response = new Response(json_encode(array('success' => true, 'branches' => $repoBranches['headRefs'])));
$response->headers->set('Content-Type', 'application/json');
return $response;
```
down to
```php
return new JsonResponse(array('success' => true, 'branches' => $repoBranches['headRefs']));
```

Isn't that :cherry_blossom:?

*PS. This will need to get changed in many places in the code*


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


Thanks @tchak!


@alecive It is performing is using row or column major ordering for the vectorization?


Looks good to go


repoman gives a warning before you commit that:
```
RepoMan scours the neighborhood...
  ebuild.badheader              1
   dev-ros/robot_state_publisher/robot_state_publisher-1.12.0-r1.ebuild: Invalid Gentoo Copyright on line: 1
```
Please pay attention to repoman warnings.


"//" should work too. There shouldn't be a special case for the root element.


Something like ``pspace(x > 0)`` (a Relational with an ordinary symbol) could be added.


No because we need the scheme as well.


https://github.com/gentoo/gentoo/pull/160 I did, and it was a huge waste of time, so I'm not going to write another huge PR that you will then not merge either


We can use elastic/packetbeat instead of packetbeat. This should give us a valid link


version updated to 4.1.8


copy-paste detected :) there is no conversion done here.


plz understand my problem may be i am unable to operate the program
correctly so i request to keep calm and guide me through the process of
operating from beginner to expert , because i was not totally understood
the process describes by you website and also let me know the pre-requaires
  for example .net framework C++ visual basics dll Directx Silver light or
and power-shell upgrade..
Regards :  Your well wisher

On Mon, Oct 26, 2015 at 11:45 AM, Nishad TR <notifications@github.com>
wrote:

> It seems like this tool is not working for you, please stop using this
> tool and try to find any other alternatives.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nishad/udemy-dl-windows/commit/6e7df719cb19ad9bb23799ed7204d6e44c620281#commitcomment-13976734>
> .
>


Hey @wngravette, making my way to your request on #1, but trying not to code like I did 3 years ago, first. ;)


nullptr


@Shauren may you cherry pick this for 3.3.5 please?


cherry-picked to master

It looks like this method is executed even when infill layers aren't combined.

Shouldn't you just do `if (amount < 2) return;` ?

Isn't it true that the normal infill (1 layer thick infill) is already generated before this method is called?


@lucabrivio: I did, as explained in comments and on the forum.


this note doesn't seem useful to me.


element


I think it's better to put this line of code before line "if isLogged"


thanks


yeah @ShreyasAdPushup this is a matter of preference. however I tend to use NPE for these specific case.
see [row 119](https://github.com/ShreyasAdPushup/couchbase-java-client/commit/043dcf09a270e42e1ec32ec0432f20c261d2f16a#diff-0bb738d1c99c932c47884db50f52e07bR119) for instance

>**side note on why:** the NPE is more specific to the exact error, it is used that way as well in the jdk (along with `IndexOutOfBoundsException` for other specific cases for example) and in Guava (they decided to fold and follow the jdk de facto convention and the advice of Joshua Blosh in his `Effective Java` book).


it's good style to set pointers to zero after deleting the object pointed to


strings won't need initing.


nm, this wasn't the constructor.


why i use it .it says that the channel token is  missing


:+1:
nichts gefunden


You could/should remove all of this code and simply call `GridGenerator::hyper_L` :-)


Also this commit is only on 15.10, while the corresponding change to the settings json file is on master and on 15.10. (https://github.com/Ultimaker/Cura/commit/d0b5fe84b8fb6e039652ca0ffabfabfcaf853c79)


Reverted to start


@hamalMarino @carlosjoserg Notice how (my mistake) in this commit there is also code changing friction parameters and update rate for bumpers. Rather than reverting, I prefer leave this commit here as a reminder to tweak these parameters, referring to #10 the main motivator


Great job :+1:


Isn't `size_t`unsigned?


Würde das auch gern mal mit dem OTC System testen aber kann ich unsere aktuelle Deutsche Übersetzung schon mit einpflegen sonst fang ich ja wieder von vorne an?


Yes it's better so.


If not difficult could you decompose next logic to multiple methods.

Thanks :)


Woop!


^ For those new to this, it's a [submodule](https://git-scm.com/book/en/v2/Git-Tools-Submodules).


oh yea :clap:


thanks


Why the last assertion is "#" ?


I believe what you have is good enough, I was just commenting on your TODO comment.


I think you forgot to include SQLite.Interop.dll in the project - the build is currently failing.


Any time.
Any thoughts about renaming repository?
If I remember correctly old name would link to new one, too.


This can, of course, be simplified to
```javascript
if (!types.includes("array") || types.length > 1)
```
But this version has the advantage of being safe against duplicates e.g. `["array", "array"]`. Don't know if it's necessary here.


I'm triggered that the slider is inversed thus less is more.


Please use camelCase like in the rest of the code.


as I said, fix repoman


nitpick: we try to sort #include lines now...


Some issues I found in `contrib/ndislib/ndisintermediate/ndis_imProtocol.c` (interestingly I cannot view the file in github!?, therefore I post my comments here):
- line 843: `Status` is assigned a value but this value is not returned
- line 1125: `bReturnNbl` is assigned a value but unused (please don't use the "b" prefix!)
- line 1475: Setting `pVethInstance_p` will have no effect to the outside, for this you would require to have a double pointer.


why? It tells you that something is wrong. Don't ignore repoman warnings.


Imports inside the namespace declaration like in the rest of the project, please.


if extract_first_word() fails, then word will never be initialized, but _cleanup_free_ will try to free it. This will explode...


When this plugin will be ready ?


解决了对分区目录进行设置时，该分区内的子目录无法完全从ftp配置文件/erc/config/proftpd中删除的bug。


I tried to it that way but I had several errors that I do not know how to fix so I finally did like in the first option:
def date_sent_ok(self):
timestamp_in_milliseconds = self.marionette.find_element(*self._time_sent_locator).get_attribute('data-l10n-date')
return (int)(timestamp_in_milliseconds // 1000)


Ja, nee. Ich schätze Statement wäre richtiger. :-D

On Wed, 28 Oct 2015 at 11:43 Johannes Hoppe <notifications@github.com>
wrote:

> .... jetzt bin ich auch verwirrt! [image: :sweat_smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Angular2Buch/code/commit/2712171895d5f9209ba7c1180948bed7c71b6d56#commitcomment-14034891>
> .
>
--
Viele Grüße
Gregor Woiwode
http://woiwode.info


Also, I'd just propagate the error you receieved, i.e. write this:

```C
if (r < 0)
       return log_error_errno(r, "Failed to parse --gnutls-log= argument: %m");
```


Also I found a problem with this: time.ctime(timestamp_in_milliseconds // 1000)

You retrieve the data in this format: Wed Oct 28 12:12:03 2015, and when doing the assertion, the time sent, although converted to the same format could be different in the seconds shown. So, I did not know how to do that assertion and did it comparing the date as it is stored, i.e. 1445603423.74


:smiley:

On Wed, 28 Oct 2015 at 13:09 Gregor Woiwode <gregor.woiwode@gmail.com>
wrote:

> Ja, nee. Ich schätze Statement wäre richtiger. :-D
>
> On Wed, 28 Oct 2015 at 11:43 Johannes Hoppe <notifications@github.com>
> wrote:
>
>> .... jetzt bin ich auch verwirrt! [image: :sweat_smile:]
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Angular2Buch/code/commit/2712171895d5f9209ba7c1180948bed7c71b6d56#commitcomment-14034891>
>> .
>>
> --
> Viele Grüße
> Gregor Woiwode
> http://woiwode.info
>
--
Viele Grüße
Gregor Woiwode
http://woiwode.info


The license and copyright notice should be moved into a region right at the top of the file, outside the namespace declaration.


If the geometry of that function doesn't quite match the one you need, you can always rotate and stretch it using the functions in `GridTools`, see step-49.


Is there maybe some reason for this special wait time which can be noted here?


@ophian You cannot comment this in serendipity_getImageFields, or the "ORDER BY i.name" will no longer work...


You may want to [view this diff ignoring whitespace](https://github.com/hiroaki-yamamoto/node-validator-nu/commit/fa4548576ecbdbf990a837609155449ca2787ec3?w=1) to ignore the changes due to the change in indentation.


Hi Chris,

I just wanted to check that you got my project OK. Github had me editing
the index file as a separate document, so I'm not sure mine got submitted
in the same way as everyone else.

Thanks,
Cindy

On Tue, Oct 27, 2015 at 12:13 PM, Chris Lindgren <notifications@github.com>
wrote:

> Remember to do this work on the "gh-pages" branch. Right now, you are
> working on the "gh-pages.io" branch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CLC-4662W/historicizing/commit/8a784122ec64bcc76c9b10c25331835a05f37e47#commitcomment-14017524>
> .
>


You used `strv_consume()` on a string you used `_cleanup_free_` on. This means, the string will be made part of the strv, and then hoewver freed as we exit the scope... This will explode. Make sure to assign NULL to `word`, after successfully adding it to strv.


@Kimmax Just update the migrate... branch its 3 commits behind


`ret` is unused


Thanks a


cherry-picked to master, since the corresponding change to the settings json file was also already merged to master.

Why do you take the max over layer height?
Shouldn't `storage.getSettingInMicrons("infill_sparse_thickness") / std::max(storage.getSettingInMicrons("layer_height"),1)`
be `std::max(storage.getSettingInMicrons("infill_sparse_thickness") / storage.getSettingInMicrons("layer_height"),1)`
?


needs NULL initializaation, see other commit.


In other words, if you don't need it any more, just remove it.


please pass "r" instead of "0" as error.


Please create a new issue at https://github.com/nishad/udemy-dl-windows/issues
This option is to comment and discuss on commits.


This set-frame-orientation could be outside the #if because it does exist for helis even if it's not used.  This is not a bug of course.


Ooh shiny!


assign NULL, please.


Would it be sufficient to include `common/oplkinc.h` instead?


use log_error_errno() and pass r to it...


return r here, instead of converting things to EINVAL.


Ouch, sorry :(


I'm also more used to PY3, but there is a small advantage of checking for PY2: when Python 4 is released the code won't be using Python 2 code path again.


:embarrased:


You are still in function `setup_system`, but you are now assembling the matrix and rhs, i.e., the function name is no longer indicative of what you really do. It might be worth breaking this function apart into two, where this part would be in a function `assemble_system`. The function is too long at 150 lines anyway :-)


It's great to see this change to set the ap.using_interlock to the one-hz loop.


After jumping in with a "huge PR" without discussing it beforehand, don't complain it was not merged.

Oh, and yeah, thanks for wasting my time explaining you how to do it properly while you were just trying to justify how what you initially wrote was so perfect that I had to use your scripts and do it the way you want it to be...


Why not keep it simple and name this just `Lookup`? There is no conflict with LINQ and cost of it preserving order is practically negligible so we don't need a constant reminder. That fact should be moved into a (prologue) comment or `<remarks>` section.


minor: please rework this (two options):

````
    if ((pInstance = OPLK_MALLOC(sizeof(tCircBufInstance) +
                                 sizeof(tCircBufArchInstance))) == NULL)
    {
        return NULL;
    }
````

OR

````
pInstance = OPLK_MALLOC(sizeof(tCircBufInstance) + sizeof(tCircBufArchInstance));
if (pInstance == NULL)
    return pInstance; // or return NULL
````


isn't there a function `scalar_product` that you could use instead of `double_contract` with its awkward template arguments?


Do you mean `runtime._source_encoding`? I think it is not necessary because there are several fallbacks for `runtime._source_encoding` [here](https://github.com/scoder/lupa/blob/master/lupa/_lupa.pyx#L200), it shouldn't be None.


好无聊啊


Why include selinux? This module now has the nasty side-effect of setting the selinux-policy?


Ok, i will revert it and take care of this in the cassandra implementation.


No, porque el lifecycle del datastream se encarga de eso. Trabaja en cascada.


There have been several suggestions on how to fix it, you didn't accept any of them, despite QA telling you. I'd even say that is disrespectful towards QA.

My PR was just an attempt to help. If you don't like it, do it yourself and don't expect other people to do all the work for you. _You_ can fix your packages, _you_ can fix repoman.


@ignacionf che vuelvo a agregar indextank al requirements por que levante una máquina de cero y no anda, aún hay dependencias en el código con indextank que deberiamos limpiar


Looks like obsolete and not used anymore.
Honza


I figured out an issue that I am having with the code. Right now the loop is set to run and subtract .001 from the prior teams total, essentially it creates only one solution for a given point total. So if there are multiple teams with say 150 total points there is only 1 listener then it decreases. This creates a limit on teams that can be created. Is there anyway to change the loop structure so multiple teams with the same point total can be created?  I'm stuck on trying to solve that problem but I am not sure how to do it.

> On Oct 22, 2015, at 8:39 PM, geekman1 <notifications@github.com> wrote:
>
> That's interesting. I think you will have to have different models for GPP and 50/50. In fantasy sports, people talk about high ceilings and low floors. Basically, a high ceiling team has a lot of upside potential, but high risk (a lot of downside potential as well). This would be useful for GPPs. Then there is low floor, lower risk teams, usually chosen by consistency. These may be useful for 50/50 games, where you only need to be above average.
>
> I am curious as to how you will simulate games and get individual player scores from these simulations. I assume you would need some type of multivariate model to account for all players on the two teams playing eachother?
>
> —
> Reply to this email directly or view it on GitHub.
>


Can you please copy&paste that into the proper pl/po/opensuse-landing-page.pl.po in SVN?


The containment check can be avoided as `Add` later does this already. [`HashSet<T>.Add`](https://msdn.microsoft.com/en-us/library/bb353005(v=vs.110).aspx) returns `false` if the element wasn't added because it is already present. The code could instead just read:

```c#
if (!seenKeys.Add(key))
    continue;
```


OK


Sure, no worries :)


There is an existing function `VectorTools::compute_mean_value()` but I think you can't use it because it doesn't exist for `hp::DoFHandler`.

In future cases such as this, I think you'd reduce your own burden of maintenance if you simply extended the function you have and put it into deal.II itself. This function is a candidate for one that could move to the library.


No, as i want the Session in the cassandra implementation.


Is it on purpose to not return the value of `retReleaseRxBuffer`?


Создание контакта

пгп


also, what does `Vect_` refer to? The fact that the variable is a vector? You don't need to repeat the type in the name of the variable.


I suspect we should just pass the _dt into the constructor instead of using a set_dt.  I know we did this set_dt thing in other libraries but it never really changes so passing into the constructor should be fine.


@karthilxg I followed your commands and it is still not working. My guess is that your `bower.json` is referring the wrong file:

    "main": "dist/spring-security-csrf-token-interceptor-extended.js",

should be (note the `.min`)

    "main": "dist/spring-security-csrf-token-interceptor-extended.min.js",


afais these are just headers, so should be DEPEND only, no?


first, there's a (useless) test dependencies just above
second, the driver is pulled by cassandrar backend dependency


@miltontony: Please review. Added notes for creating a release, not urgent, but please double check that I haven't left anything out.


You can use `has_item = graph.containsItems<envire::core::Item<smurf::Sensor*>::Ptr>(a)` here.
It's cleaner than the iterator approach :-)


Yes, this is what I had in mind, great.

Honza


проолл


Isn't there a function in deal.II's `GridTools` namespace for this already? If yes, please throw away your function and use the one from deal.II. If not, please move this function into deal.II.


I hope you love this one


Попробовал сделать так

    {{ form.username.label_tag }}
    {{ form.username }}
    {{ form.username.errors }}

    {{ form.password.label_tag }}
    {{ form.password }}
    {{ form.password.errors }}

    {% if not form.username.errors and not form.password.errors %}

        {{ form.errors }}

    {% endif %}


same here -- use what's in deal.II or move it there


close #1


This is spelling error!
Right will be canola!


and here: move to deal.II


Explore adding file path
- [ ] @BardyshBorys


/giphy puto amo macho

:100:


Yo fui sacando todas ellas. Fijate que me encontré con vistas que llamaban a Searchify, etc. El tema es que sin indextank vemos los errores, con indextank no vemos que tal vista, vaya a saber por qué, llama a searchify/indextank. En algunos casos, ni siquiera lo importa para usar la lib, simplemente la llama.


It already is. Please check this release https://github.com/HotFireyDeath/HalloweenPM/releases
You will need to .phar it yourself, but the plugin is ready and works.
It has been submitted for review in the PocketMine plugin repo.


This function has multiple non-const reference parameters. Are you modifying all of them and want the values propagated back to the calling site? If so, this function really could use documentation that explains what it does and returns.


Создание контакта

ололол


Yikes, "discussioned"...


Just ran the tests and they have a problem with `e.message`.  Went away if you just pass `e`.


Should this be in all upper case? This way it looks like a constant, but in fact it is not.


BUeno entonces no lo agrego pero ojo que como está no anda nada si creas una máquina de cero


OK, pero mi duda esta más orientada a la linea 156, solo consulto a los DS que estan en estado draft o pendientes. Incluyo a las VZ o por esta cadena de publicaciones es suficiente con ver que los DS no esten aprobados para detener el publish_childs?


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

{{ form.username.label_tag }}
{{ form.username }}
{{ form.username.errors }}

{{ form.password.label_tag }}
{{ form.password }}
{{ form.password.errors }}

{% if not form.username.errors and not form.password.errors %}

    {{ form.errors }}

{% endif %}


So it looks like the input curve is a set of connected straight lines.  I wonder if some other opinions from team (Paul, Jon, Leonard) should be gathered re the curve so see if it could be made completely smooth instead.
I also worry that the 400 ~ 600 part implies a dead zone which seems unrelated to the THR_DZ parameters that we have currently.

By the way, we actually have an AP_Curve library that might do what you want.  If you're not going to use it we should probalby remove AP_Curve (I can do that of course).
https://github.com/diydrones/ardupilot/blob/master/libraries/AP_Curve/AP_Curve.h


CC must be quoted, it can contain whitespaces, better do the same for the other linker variables too


Patrick P. hat geantwortet. Es heißt "Statement". Siehe https://github.com/Angular2Buch/code/commit/b742f41948ab25b822f6fde33d1eaf0bfa501230


Why this commit is not in master version ?


whaat the :D can we not do this with a for-loop and a bash array?


I think the style here is fine -- it's just a regular `for` loop where the counting variable is incremented in each iteration.


thanks :)


Updated, thanks


The code is somewhat beyond my python knowledge, and I found no useful documentation.
-> questions:
* I assume I can't .add xmls to bibupload and later decide to stop it.
* is there a way to write at least the task-ids of the upload jobs to the log file? Otherwise how can I track that the upload worked OK??? And check what was really uploaded? The info in the log file is only what I wanted to do which is not necessarily the same as what I did.


ya lo vi, por eso hay varios commits más mios:

https://github.com/datal-org/datal/commit/ce03faba85ea01193a656bb2ca700068f5be78c8
https://github.com/datal-org/datal/commit/69d07849ce3c512fc15da54883bba34bce15135d

Yo tambien tire un destroy y luego miré los errores. Los que fui encontrando son estos de arriba.


Looks like a regression after https://github.com/zalora/microgram/commit/e2ecc0fa1bc2d60a59a70ef316d468fe9086632f ? I suppose the "prime" api has to land in microgram too.


Graciass


Oaah und hasts gleich für mich korrigiert, danke! :heart:

On Wed, 28 Oct 2015 at 13:29 Johannes Hoppe <notifications@github.com>
wrote:

> Patrick P. hat geantwortet. Es heißt "Statement". Siehe b742f41
> <https://github.com/Angular2Buch/code/commit/b742f41948ab25b822f6fde33d1eaf0bfa501230>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Angular2Buch/code/commit/2712171895d5f9209ba7c1180948bed7c71b6d56#commitcomment-14037212>
> .
>
--
Viele Grüße
Gregor Woiwode
http://woiwode.info


mark input arguments as `const`.


the only two ways I know to find my writing mistakes is to ask someone else to read what I've written or to slowly read it out loud to myself.


исправил в шаблонах, где нужен только просмотр, убрал тег label


이상하게 올라와 있어서 다시 올렷습니다.


The function does not conform to the `verb_noun` style -- the name does not actually explain what it does.


No prime, please. Curl wrapper seems much better.


I don't think this is the right place where to call this.props.togglePause
It should rather happen in MainToolbar.onTogglePause.

Honza


That just looks unreasonably amazing! :).  I feel like a spoiled only child right now. This is going to make developing new roads so easy.  I didn't do this for all the roads because it is a POC but if you look at 5 and 6 Lane highways you will see that I grouped all the textures by type (ground/elevatedbridge/slope/tunnel).  Supposedly, this would allow for a more streamlined settexture sequence.  Not sure though...


I mean `sdk.curl = pkgs.curl + ssl`


Maybe this check should use "mode_has_manual_throttle"?


It's not worth abbreviating `solution` to `solu` -- and if you used a good editor, you could even avoid typing the full variable name by using auto-completion :-)


function name is not `verb_noun`


Maybe we should consider the use of `<anchor ana="start">` with a `@corresp` attribute to link it to a particular `<anchor ana="end">` for cases like this.
I think the `<anchor ana="start" synch="#M1_p1_c43"/>` should "end" after "centered" (I word we are missing in our output because is outside of the `<anchor>` relationships).


Typo :)


:+1: for readable variable names. I'm not keen on inline comments either.


OK


too many parameters on one line


En teoria ya no se llama más a las clases Searchify si fallan. Todas estan encerradas en un try except ImportError. Pero... puede fallar :)


should be params


Ooops, a stupid bug here. The coordinate should be divided by texture size. Will fix that by next push.


FYI `Teaser.defaultProps.dateFormat` could effectively be done with:

```
(date) => date.toLocaleString('en-GB', { month: 'short', day: '2-digit', year: 'numeric', hour12: true, hour: '2-digit', minute: '2-digit' })
```


hello


Cool! Seems Awesome!


Why is there a need for:
n = min(N_cnt, n)

Is it to prevent such cases that user types an unrealistic N_count value? For instance, if original matrix has only n=59412 points, and if N_count is given as 60000... Then N_count will simply be ignored, right?


I've already reverted it, no worries, it works again!


There ist a typo:

> You need to import **wht** .sql files in the order


Because we ship with a prepopulated sqlite database for testing purposes.


Why include ::selinux?


Only createStream is covered in the rtmp specification... I believe we should not use in internal code double type for stream ids until it will become necessary.


Changing the type of uiBin seems to be the simplest solution here. I'll submit a PR for that, and adding some simple casts elsewhere to silence other MSVC warnings - I checked with both gcc and clang that they still emit the same code.

FWIW, MSVC didn't manage to optimize the comparison and negation here, at least not on the optimization settings currently used.


You are right.


Your file should not have any further information in it aside from the school name, I'd recommend removing line's 2-5


That's fair.


I am not sure... In general it is done-ish, but it needs some more tests. And there is no gtk version, yet.


its implied that it is OSX, as developing for iphones can be done only on OSX


CBECC (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2321 of 2349 tests passed)

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


It's just more convenient to have global variables specifying the size of main texture. I guess it's gonna be a nightmare to replace all occurrence of them by `pass_tex[0]`


If you want to display some traces on the console, i suggest you look into the npm '[debug](https://www.npmjs.com/package/debug)' module.


how about one of:
- Alooma-ios
- Alooma-iOS
- Alooma-iphone (mixpanel-iphone style) ?
- (just) Alooma

?


yes


That was only a POC, feel free to give me more feedbacks if it doesn't work well... =)


It took me a really long time to convince myself that this is actually correct. Your control flow here is
```
  for (q=...)
    {
       Vector<double> local_rhs1 = 0;
       for (i=...)
       {
          local_rhs[i] += a;
          local_rhs[i] *= b;
       }
  }
```
This is only correct because the variable was empty before the addition; otherwise you would also multiply *whatever was in the variable before* by b, possibly multiple times if you had declared the variable before the loop over `q`. In other words, what @chu575 points out is not just a minor style issue, but it actively makes the code very difficult to read/makes it very difficult to convince myself that what you add into this variable is indeed a*b, and that you're not computing `(something + a)*b` instead.


Anti-spam 4.1


It's not forgotten, `deband_fbo` is newly added. I can't reuse `chroma_deband_fbo` since the texture size is different, and we gonna recreate both fbo for each frame.


function name


C'est quoi la note pour plagiat déjà?

```sql
--
-- Structure de la table `hands`
--

CREATE TABLE IF NOT EXISTS `hands` (
  `tu_played_id` int(11) NOT NULL,
  `tu_id` int(11) NOT NULL,
  `ca_id` int(11) NOT NULL,
  `us_id` int(11) NOT NULL,
  PRIMARY KEY (`tu_played_id`,`tu_id`,`ca_id`,`us_id`),
  KEY `fk_hands2` (`tu_id`),
  KEY `fk_hands3` (`ca_id`),
  KEY `fk_hands4` (`us_id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1;
```
https://github.com/PCreations/Dixit-Online/blob/master/sqlMPD3bis.sql#L127-L140

**NB:** utilisez des migrations pour créer vos tables.


too many arguments on each line; do all of them have to be non-const references?


Acknowledged, will remove these assert()s.


Also, I considered doing e.g. "~uiBin + 1" as well, which is a well-defined operation with no extra conversion between signed and unsigned (I don't remember all the cornercases in the standard about which of them are defined and which aren't) - gcc and clang managed to produce the same code with that as well, but just changing the parameter type seems much less convoluted.


Acknowledged, will fix that by next push


Thanks Brady! - all comments implemented and pushed.


Sorry for this one, tried to push before pulling and looks like rebase didn't work in this case :/


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


does it not work for iPad apps?
I like Alooma-iOS


the static check says b is unused if no TIMER_x_EN are defined, I decided to rewrite this commit in a simpler way in https://github.com/gebart/RIOT/commit/fd93a3ffd827d36060960829844809fe8031bb7c, currently waiting for Travis.


是什么？


I believe `pass_tex[4]` will be cleared during the chroma processing. That's why we need to backup `luma` and `alpha`  a few lines below


break missing????


But isn't it quoted? It can either be "mpc++" or "$(tc-getCXX)".


I've changed this manually because my code was expecting that a note related to a particular string would be located before the `<anchor ana="end">` of that string. @ebeshero and @setriplette,  view my comment on Southey's chapter 2 so you can tell me if it would be a more accurate mark-up to allow "nested" anchors.


All those requests :) But yes, that's not a bad idea...


There are already tests with singleton record instance arguments (for instance, test/succeed/Issue674.agda). This patch doesn't change which instance arguments get resolved, just how quickly, so I don't know what I would put in a new test.


We just temporarily upscale chroma align with prescaled size. See my comments on the main thread.


That's also a really confusing name because it might suggest that you actively want the variable to mean something *different* than the variable name `dofs_per_cell` (with two 'l').


не закрыл кавычки))))))))))


This can only happen if you assigned an FE_Nothing to this cell. Might be better to either query the active_fe_index of the cell, or at the very least leave a comment that explains the circumstances in which the `if` succeeds.


Deberia ser:

DataSet.objects.filter(pk=self.dataset, last_published_revision__isnull=False).exists()

Si todos los hijos tienen last_published_revision en NULL(o sea ninguna de sus revisiones fue publicada), podemos saltear la publicacion en cascada de los hijos. Esto es equivalente a decir que si hay al menos un hijo que no tenga NULL en last_published_revision(alguna de sus revisiones alguna vez fue publicada y quedo en ese estado) debemos intentar publicar los hijos en cascada para que se hagan las validaciones correspondientes antes de publicar el padre.


Steve,
I think I figured it out.  I am going to make some multiple comments so you can read them more easily.


They process different planes, and under different condition. But the general idea is the same, use cheap bilinear filter to handle non-essential planes to match prescaled size. Also see my comments on the main thread.


终于找到了


minor: Is it somehow possible to break that line?


Acknowledged.


Fixes. Thanks


might be worth adding a comment what this class represents


Embedded notes have now a `<string ana="start">`,  `<string ana="note">` and if needed `<string ana="end">` but I could change this to a nested `<fs>`


firstRepos
first repository in crossover assignment
this is my second commit;


second update


@jdannberg -- that's actually how astyle produces it, because it doesn't indent the `case` blocks relative to the braces for the `switch`.


`if (input == "h")` is a lot easier to read :-)


it says
```
CC=$(...)
```
so no, the outer result is not quoted
should be
```
CC="$(...)"
```


"spring-context" is not wrong but I think "spring-core" is sufficiant. We want to have as less dependencies as possible. As far as I know spring-context uses spring-core. Can you please try it out?
mvn dependency:tree | less
this command shows you the dependencies.


@MichaelNedzelsky add test for internal method override in test module


Remove one ";"


Not understanding this comment.
The first parameter to CreateInputStream is a pointer to the player (or NULL if we aren't the player). That hasn't hanged with this commit.


code review.


objects of this type don't actually represent a quadrature, but what *kind* of quadrature. I'd rename the enum to `QuadratureKind` (and in the process get rid of the all-upper-case letters as well; same for the other variables).


I wonder if we could come up with a more specific name.  For example, maybe it's only used for an engine (which uses gas as opposed to a motor which uses electricity) so it could be "k_engine" or "k_engine_run" or "k_engine_start_stop".  "motor_run_enable" seems very general to me.


>Ну и небольшие косметические изменения.

![title](https://cloud.githubusercontent.com/assets/1483359/10788832/168db4da-7d8b-11e5-82ca-b9f39745b215.png)
Ну и стало криво же. По вертикали не по центру, крестик без правого поля. Сделай нормально плиз, алсо замени там белый #FFFFFF на #F5F5F5, чтоб не так ярко было и с кнопками панели совпадало.


Yes, thats correct.


Where is the `aAdapterGuid` used?


Hehe, I see you've noticed github's comment function...

For the right hemsp. nodes its enough to specify:
```python
N_first = 29696
N_user = None   # this reads to the end
```

So the only magic number you need to know is "29696" as you see for example in eigen_decomp/corr_top10.py:
```python
# apply --hem argument
if args.hem == 'full':
    N_first = 0
    N_cnt = None
elif args.hem == 'LH':
    N_first = 0
    N_cnt = 29696
elif args.hem == 'RH':
    N_first = 29696
    N_cnt = None
```


FYI, I'm looking into enhancing this to work for Python 3 as well as 2.


We use `bilinear` here since it guarantees to produce bit-exact results. But this is no longer true once prescaling is introduced and sub-pixel offset is introduced, bilinear filter won't be bit-exact in that case.

Also note that it's not critical to correct these offset (by main scaler) in the first placed. So I think it's acceptable to remove any half pixel offsets.


Is this ok?


Correct! We just want to use the interface. The client code doesn't know about the car service implementation.


I was thinking that if the name is the same, a person reading the code using this function would wonder why is there a Lookup implementation, and would have to fish out details from the comment. But if I turn the comment into a xmldoc then it would be just a mouse-over away. Will change.


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


Här deklarer du en global variabel eftersom du inte har ```var``` framför


Where is `pMiniportInst` used?


This pr was merged too long ago to back out at this point; personally I have two projects based on the mods. If the stream id is causing issues internally with your code, simply use Number.intValue() to get the int instead of the double.


Correct! But only a small thing. the id of the car service should start with lowercase -> carService


Thank you! :dancer:


This import was removed after this commit as it wasn't needed.


Nice!!


Generellt tycker jag att man ska undvika att hårdkoda pixlar i JS. Bättre att korrigera i CSS eller leta upp ett mer passande element att scrolla till.


sorry


Curious what the rationale was for this? 200 seems like it should be fine


Agreed.

Conversion from signed to unsigned is always defined to behave as 2s-complement conversion. Other way around is implementation-defined when the unsigned value does not fit in the target type as a non-negative integer (although int32_t is defined to be 2s-complement). IIRC at least.

It's mainly important that we avoid a conditional branch here. Dropping the mask and just multiplying uiRange by uiBin instead should not be much worse, and might generate better code with MSVC. I still need a convenient MSVC setup to test with.


Correct! The wiring of the object web is now done by the spring container. So we don't need manual wiring anymore.


Wrong commit link, this refers to https://github.com/s9y/Serendipity/commit/73e15645d693da527df2e9483fdb81b5baee4fc6 and https://github.com/s9y/Serendipity/pull/367


If it were me, I'd put these into a structure in the .h file.  That way they are kept grouped together but their existence is still visible from the .h file.  Either way is correct.. in that it's not a bug of course, it's more of a style thing that we don't hide blocks of RAM eating variables perhaps so they can be more easily found as we hunt for RAM when things get tight.


why no ImmutableList.of ?


I went back to the parent, child, and family relationship to begin with because that was working.  I realized that a parent and a child each belong to one family.  However, a family can have many parents and a family can have many children.  So if I am not mistaken a **parent** has a **many to one** relationship with the **family** and a **child** has a **many to one** relationship with the **family**.  However, a **family** has a **one to many** relationship with the **parent** and the **family** has a **one to many** relationship to the **child**.


这个地方的两个break会导致使用mycat注解时，sqlType解析不对的情况，会解析为ServerParse.OTHER，导致SELECT语句不能落实到readHost上执行


Använd ```updateHistory('#' + anchor)``` så blir det mer konsekvent och skapar inte ett nytt history som man kan backa till.


Checkmarx scan results 28 Oct 2015 05:59 UTC -07:00

High
Medium
Low
Info

New
6
33
399
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

64950ba3148081eaf42ccc64e6f9f6009c3b5c5b	@amitchen22


Corrected to core:
[INFO] \- org.springframework:spring-context:jar:4.2.2.RELEASE:compile
[INFO]    +- org.springframework:spring-aop:jar:4.2.2.RELEASE:compile
[INFO]    |  \- aopalliance:aopalliance:jar:1.0:compile
[INFO]    +- org.springframework:spring-beans:jar:4.2.2.RELEASE:compile
[INFO]    +- org.springframework:spring-core:jar:4.2.2.RELEASE:compile
[INFO]    |  \- commons-logging:commons-logging:jar:1.2:compile
[INFO]    \- org.springframework:spring-expression:jar:4.2.2.RELEASE:compile


on htdocs/core/class/commondocgenerator.class.php
the add on construct have made an error on /htdocs/exports/index.php
please see
https://github.com/Dolibarr/dolibarr/pull/3816

we need it on 3.8 branch


Maybe use "[" and "]" instead of "<" and ">":
`cd <openPOWERLINK_directory>\drivers\windows\drv_ndis_[pcie;intermediate]\build`

The "<" and ">" are used for hinting to the oplk root dir...


also here


Corrected


Det finns ju redan en ```scrollToTip($tip)```använd den istället. Tycker också att det är problematiskt att kolla bredden på fönsret i JS. Men har ingen bättre lösning heller :P


I didn't see that. I'm just wondering what "non-essential planes" are, and why it uses bilinear for them.


- and here also

( "<ARCH>" is okay in my view )


Nossa.


Wouldn't this fail since you're adding string inside of a string array percent literal? I don't think `'libasound2'` etc. would resolve as packages. Remove the quotes or use a regular array literal.


A **parent** has a **one to many** relationship to the **children** through the singular **family**.  Likewise a **child** has a **one to many** relationship to the **parents** through the singular **family**.


move the constructor at the start of class


@gabizou `supports(EntityType)` shouldn't have been removed.


@hellosylvee DWMR (done with my review). This is good. Press ahead.


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/bacf220ee9034adc9b5b1b8ab9e8c231148d7c14


merging basho/yokozuna/bugfix/zl/sort-without-score-bug = bacf220e into borshop-integration-570-bugfix/zl/sort-without-score-bug


basho/yokozuna/bugfix/zl/sort-without-score-bug = bacf220e merged ok, testing candidate = e64f9af4


return null ? really ?


Also, if we default to omitting GHC, two out of four tests will continue not to be exercised by CI. Consider that test 4 has almost certainly **never passed before**, even though it’s been added to the formula eight months ago: https://github.com/mietek/homebrew/commit/ad94d39ef30744dc98081a747b598bf9581ba414


first commit;


Please, read CONTRIBUTING.md.


@hellosylvee DWMR. More good work. Keep it up!


Actually, that's not quite true. With the eta expansion in place you don't actually need any instances around. Added test (4bac556f).


Generellt är det bättre att lyssna på propagerade events istället för att binda direkt på ett urval av element. Skulle rekommendera att köra ```$('body').on('click', 'a[href*=#]', onAnchorClick)``` istället för att spara undan elementen i document ready.


=$SRCDIR


And whats the output just with "sprin-core"?


As this is the preferred way to reuse directives, the code looks pretty good to me.
Also, the field behaves as expected.


If it is really redundant (I don't know), maybe it could be at least set by the thing that sets pass_tex? Anyway, it's not entirely clear to me how the code works (or rather, all the implications). I'm just asking to minimize redundant state.


Ohh. Right! Thanks.


I definitely don't want any merge commits. Did you not rebase against master?


Yasss :+1: :D


second;


`rebase` next time, like `git pull --rebase liushuyu staging`.


Thanks, I'll fix them


Yep exactly!
n = N_cnt would not work if N_cnt > n. In that case  we could also return an error but I think it's ok simply to use the largest possible n.

BTW there is another broken case if N_cnt < 0 but IMO we don't need to handle all obviously stupid user input.


this looks like a very bad idea : you do a n+1 queries pattern on the same table !


first;


ok


no throw ?


junit should be used just with scope test, as it's not part of the application itself. its just needed in test phase.
<scope>test</test>


@yoojinkimhong DWMR. You're shipping code. :truck: :checkered_flag:


you should use optional + or


hey look ma, my first commit in about 10 years.  :-)


@rvonstange DWMR. Ship code. :truck: :checkered_flag:


What about ...
```
DOCKER_TAG_1: itk-v4.8.0_use_system_libraries-off
DOCKER_TAG_2: itk-v4.8.0_use_system_libraries-on
```


>Also note that it's not critical to correct these offset (by main scaler) in the first placed. So I think it's acceptable to remove any half pixel offsets.

Why?


@rvonstange DWMR :hammer: :bug:


[It is required](https://developer.github.com/v3/#user-agent-required) so please update the note to reflect this.

Also (unless I'm missing something) we're doing unauthenticated requests which limits us to [60 requests per hour per ip](https://developer.github.com/v3/#rate-limiting). It is probably ok, but it is also quite easy to [fix this](https://developer.github.com/v3/#authentication).

Finally, for compatibility in the future we should explicitly request the `v3` version with [correct accept headers](https://developer.github.com/v3/media/#request-specific-version).


Perhaps change "non-transient" to "direct" since that's a less obscure way of stating which dependencies are listed.


There was a little bit of merge conflict copy that you left in here:
`=======`
but it's mostly okay since this file got deleted afterwards :D


I still need to push my local-git-commits + upload the archive for that(includes support for payload 2.5).


@rvonstange DWMR. iOS app shipping requirements are not trivial. Press on.


The problem is, `pass_tex` itself is not persistent. See `finish_pass_direct()` from `video.c`, `pass_tex` get reset for each `finish_pass_fbo()` call.


There seems to have some small issues with the vanilla roads LODs when I activate the 4 and 5 lanes HW...
(The node textures are getting slightly deformed, and a small 1px gap appears between the segments)

I was trying to figure out what was it, but I was quickly overwelmed by the amount of code there was =)

I honestly prefer splitting the code by functionalities rather than by road versions, preventing the functionnalitirs being coded multiple time, but that's only my humble opinion =)


>  You can fix your packages, you can fix repoman.

Yes. I just happen to have limited time and have to prioritize. If you happen to have different priorities than me, then maybe it would be more productive to fix the underlying problem instead of wasting everyone's time with useless comments here, wouldn't it?


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


不喜欢拆开的~


Я их все равно в svg переделать хотел. Заодно починю.


@max-kammerer Please take a look at https://github.com/JetBrains/kotlin/commit/3be5d28c14dcba510e1b4a928a0f7c9da66cede1


Чтобы мы не делали двойную работу. Ты переделываешь кнопки панели в векторе, или сделать это мне?

Просто я занят, поэтому до выходных заниматься скриптом не смогу совсем.


可能那种是更常见，但对于这种二维矩阵的题目，岂不是二维增加了(i, j) 到 i * cols + j 的 encode & decode, 是不是应该因题而异？


更正


This looks good! I think I'd prefer this fix then, probably with a short comment mentioning that `tmp` is used because it might happen that `p == o`. Thanks!


To create a child and associate it with a parent, I first need a parent.  Assuming I have a parent and the parent has a family_id (which he should based on ```#create``` in the parents_controller), I can say ```child = parent.children.new(child_params)``` but I can perform the same action with more verbosity by ```child = parent.family.children.new(child_params)```.  If I am not mistaken, the ```#children``` is a short cut for going through the family to create a new child.


I think it might be a bit misleading showing the html separately like that. The current layout should be clear enough.


OK, will fix.

On Tue, Oct 27, 2015 at 9:38 AM, Mulia Arifandi Nasution <
notifications@github.com> wrote:

> IPv6 maximum length is 45 characters.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/framework/commit/a9f00d587b4b15fcb94a70e8d5fb70b44b247e01#commitcomment-14012350>
> .
>


You only initialize measurements here. The Test object is long lived. This should be done either in RoboTest.setup_job or RoboTest.run_job. Probably run_job is better.


But I still don't get why it wasn't working before.


따봉 배달왔습니다.

https://github.com/meoooh/rubyworld/commit/34aea03fc97668e0c5655d27c04378bbdbccc558#commitcomment-14037045


I don't like the quote the newline approach to line continuation. Let's just wrap this in parens and drop the \


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1429


@VisualKei алсо, наверное некропроблемы, у меня все ок, если что.

@Y0ba, ты и так много работы сделал, отдыхай. Я хотел пока только эти две, может переведу и другие кнопки.


I thought graph use the same set than statistics but graph are done for a subset of these domain names. https://github.com/WeftWiki/phetools/blob/master/statistics/graph.py#L224 I'll pull this change soon


Some interesting black magic to turn errors into exceptions.

```
set_error_handler(function($errno, $errstr, $errfile, $errline, array $errcontext) {
    // error was suppressed with the @-operator
    if (0 === error_reporting()) {
        return false;
    }

    throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
});
```


The user might have their own log4j that doesn't specify a console appender.  Why don't we do something simpler and just change the level to debug if its analysis exception?


Только проверяй внимательней flex+svg в некроопере. :smiling_imp:


Do we need this call? We're using the method's result or storing it.


Would have been funnier if you had written "Josh rox".  Missed opportunity.


Close #31


I would prefer it if the `Subscriber` interface was simpler.

```golang
type Subcriber interface {
    Run(file string) error
}
```

The various events (pre-rotate, post-rotate, etc) would be handled using a `map[Event][]Subscriber` data structure instead of the `[]Subcriber` one we have now.

Subscribing would be like this:

```golang
func (l *Logrotate) Subscribe(e Event, s Subcriber) {
    l.subscribers[e] = append(l.subscribers[e], s)
}
```

Dispatching events would also be like this:

```golang
// inside rotate function
for _, subscriber := range l.subscribers[PreRotate] {
	if err := subcriber.Run(file); err != nil {
		return err
	}
}
```


With the children and sports, a **child** can **have many sports** and a **sport** can **have many children**.  This is a **many to many** relationship between **children** and **sports**.  Based on some more reading of the Rails documentation and StackOverflow I added ```has_many :children_sports``` to both the child and sport models.


If not difficult - could you make decomposition that will seems something like this.

Idea of this code is to make all alternatives cases more readable.

```javascript
if (poolConfig.id) {
    return modifyPool(poolConfig.id, _.omit(poolConfig, 'id'));
} else {
    return loadPool(poolConfig).then(function (pool) {
        return pool && modifyPool(pool.id, poolConfig) || createPool(poolConfig);
    });
}
```

Thanks!


Typo: `subcriber` -> `subscriber`.


1. The channels are aligned correctly before yuv conversion, so there won't be any visual artifacts
2. The user explicitly specified "scaler-resizes-only", and expect performance gain. There is also chance for huge quality gain, since it's bit-exact
3. Half pixel offset are really difficult to be noticed during playback
4. Using "bilinear" with non-integer offset is probably the worst idea, we probably want to disable "scaler-resizes-only" entirely rather than use "bilinear" for pescaled video.


Nit: Could we replace this is with min(self.layout.thumbnailCount, numOfCellsFromData)?


@joecaintic DWMR. Shipping code. :truck: :checkered_flag:


> fix the underlying problem

I cannot fix your workflow, I am afraid. The best I could come up with was a script that aids your workflow. You only wanted to use it if all developers use it before you. I cannot provide further help on that problem, except pointing out mistakes.

The sed-line to fix copyright years is here: https://github.com/gentoo/gentoo/pull/187#issuecomment-147482062 you can apply it yourself on your packages/categories. I don't see why I have to run that for you, especially since even that PR was denied.


I have no issues with my code and double stream ids. I just worry about possible rounding errors, but tests show me that I am wrong.


Change log wrong month

0.5.5 2015-11-27
- Make mass action admin controller route compatible with SUPEE-6788 patch


I think this change isn't needed here.


@joecaintic DWMR. Callbacks are used pervasively. :telephone_receiver:


Okey, so windows completely ignores these files, while linux/macosx
completely supports them. Sounds good enough as an improvement of current
scheme. Thanks!

On Wed, Oct 28, 2015 at 11:56 AM, Daniel Molkentin <notifications@github.com
> wrote:

> @moscicki <https://github.com/moscicki> This algorithm applies to both
> ends, so files with these characters in the file name will be ignored on
> Windows. No transliteration is being attempted as this is a recipe for
> disaster if the mapping gets lost, amost other issues that would arise from
> this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/owncloud/client/commit/91525a7d33c531c04772376e833ed0238a8ee510#commitcomment-14035217>
> .
>

--
---
Best regards,
Kuba


why the semicolon?


i want a laptop.


Yeah, it seems that in `FffPolygonGenerator::slices2polygons()`, it first calls `processSkins` on all layers, which generates infill. Then directly afterwards, it calls `combineInfillLayers` on every mesh. It'd be neater if it didn't call `combineInfillLayers` at all or it terminates early if `infill_sparse_thickness` is less than twice the thickness of a layer.

The latter is because my original commit was on 15.10, but was merged by @awhiemstra in commit Ultimaker/Cura@936c2f936535343ad7e1d608906a631f507091e2 to master.


QQ这个 ios最后提交appstore会有问题。得删除QQ的plist，不知道什么原因，麻烦看能更新下不。


Same typo.

Perhaps we can add a method to dispatch events so we can reuse some code.

```
func (l *Logrotate) dispatchEvent(e Event, file string) error {
	for _, subscriber := range l.subscribers[e] {
		if err := subscriber.Run(); err != nil {
			return err
		}
	}
	return nil
}
```


Не знаю какие там могут быть некропроблемы в простейшем стиле. Всё просто же: ты сделал флекс-контейнер, но не задал ему `align-items: center;`, зато задал заголовку 100% ширину и не задал кнопкам `flex:none` - естественно они ужались до упора вправо.


shouldn't this file be part of resources


@lyndonbartlett88 DWMR. You're shipping code. Well done. :truck: :checkered_flag:


Still I think it would be good to replace this `before`/`after` with

```ruby
before do
  allow_any_instance_of(ActiveRecord::ConnectionAdapters::IndexDefinition).to receive(:case_sensitive?).and_return(false)
end
```


Если ты хочешь оставить только цифры. То лучше `/\D/g`


Yes it is, it only adds extra features for IOS like the `aboveStatusBar`property.
Everything should work on Android.


You need this activated for UID_CPUTIME compilation (depends) and then you will have stats on battery graph.


You are eating up the error here.  Missing a throw statement


tier 1 point 3


Here's the error

<img width="975" alt="screen shot 2015-10-28 at 11 21 34 am" src="https://cloud.githubusercontent.com/assets/2964631/10789695/1c7f6f02-7d66-11e5-8029-3ff20de3067e.png">

We don't have billing activated, just invites


The comment no longer matches the conditional.  if len(self.measurements) < self._iterations just means we didn't get a measurement for each iteration not that we didn't get any measurements.

The case where len(self.measurements)  == 0 is separate from len(self.measurements)  <  self._iterations

 len(self.measurements)  == 0 is busted and should get the special treatment, email etc.
 len(self.measurements)  <  self._iterations is just an error and should be treated like a regular error.


> I cannot fix your workflow, I am afraid. The best I could come up with was a script that aids your workflow. You only wanted to use it if all developers use it before you. I cannot provide further help on that problem, except pointing out mistakes.

Relevant, and only worthy, discussion happened in the bugreport. Please read it and cut the BS.

> The sed-line to fix copyright years is here: #187 (comment) you can apply it yourself on your packages/categories. I don't see why I have to run that for you, especially since even that PR was denied.

yeah, thanks for your help, and please continue to comment on every single commit where this hasn't been updated...


I assume EOL style differs due to plattform differences (CRLF vs. LF).
Best practice is not to depend on platform settings e.g. manually write explicit LF newlines on every platform and also always specify file encoding such as UTF-8 in the code rather than depending on system properties and plattfrom settings.


У меня все красиво. Проблема у вас))0


linande põletamise mäng


tier 1 point 1


I disabled it because it is for 4.2. Not necessary, i suposs, on M.

:)


>Half pixel offset are really difficult to be noticed during playback

But that whole point of chroma positioning is to apply this offset anyway?

Also, I don't think the luma plane ever has subpixel offsets.


break the line. this should work:

                self.test_failure(self.name, 'TEST_UNEXPECTED_FAIL',
                                  'No measurements detected. %s != %s' % (
                                      len(self.measurements), self._iterations),
                                  PhoneTestResult.BUSTED)


The server?


Since this is empty you should delete it.
Also screenCenter isn't called anywhere here. Does it auto-center to parent?


Кстати, починил градиент в хроме. Тот оказывается не понимал аттрибут `gradientUnits="userSpaceOnUse"`.


Why did you disable the comSubExp module for these tests?


constructor


@cehallen DWMR of checkpoint 42. Well done here. This was not a simple assignment. Shipping code! :truck: :checkered_flag:


prototype


I don't think i store the whole "const char *" stuf in CName


@EthanGold60 Thunder renders for other players on the server side now


test case


@Kimmax unable to build the project. missing symm


To wrap this up: We merge the commit as is and @m-stein removes the zynq-related part when merging #1667.


comments in JSDoc format


how about return self.analyze_logcat() and just get rid of pageload_metric all together.


В этом модуле всё плохо с названием методов:
а) глагол get_ использовать в названиях не принято: во-первых, он путается с понятием getter (доступ к свойств, то что в руби называется attr_reader), общепринятым в ООП; во-вторых слишком общий — для данного случая лучше подошёл бы просто recommend, как короткий понятный глагол («порекомендуй!»)
б) несколько методов с похожей функциональностью, но с разными названиями — это совсем плохо (get_recomend и get_watched_rec — второй хотя бы стоило назвать get_recomend_watched); но в данном случае, мне кажется, лучше вообще обойтись одним методом, в который зритель бы передавал, хочет ли он уже посмотренный фильм, или ещё несмотренный; такой подход более расширяем (например, в будущем мы можем захотеть вариант «выбери из всех, и смотренных и несмотренных»)
в) глагол show_ обычно используют для методов, которые что-то выводят (в браузер, или в консоль или ещё куда); а если метод просто возвращает что-то, что объект просто хранит — лучше вообще обходиться без глагола (мы же пишем, допустим, `movie.title` а не `movie.get_title` или `movie.show_title`, правда?)


I think that might not work ? If I remember correctly Rspec mocks only
works on those defined functions and this function is not defined yet.

Anyway I can give it a try later

On Wednesday, 28 October 2015, ronen barzel <notifications@github.com>
wrote:

> Still I think it would be good to replace this before/after with
>
> before do
>   allow_any_instance_of(ActiveRecord::ConnectionAdapters::IndexDefinition).to receive(:case_sensitive?).and_return(false)end
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allenwq/schema_validations/commit/a3ee26729f8364f1a04aee63b4dc37c9b8ea38cf#commitcomment-14038560>
> .
>


Eftersom det är ett jQuery-objekt borde den heta ```$target```


yup, i can use those methods it when they'll got merged


Här finns ingen event-variabel


To make sure the package is also included


Whoops?


Ну так флексы же не пишутся, в других местах всё по уму сделано и всё норм. Там вполне можно и `float` оставить, зачем там флекс наворачивать?


They also have great wines, may be add a wine glass emoji? :wine_glass: :heart:


We aren't building the dict any more, right? We can drop this line in the docstring I think.


Read up on Java exception Handling


You forgot to mention the cat :cat2:


QQ这个 ios最后提交appstore会有问题。得删除QQ的plist，不知道什么原因，麻烦看能更新下不。


Nej förlåt jag som såg fel... never mind denna


I dont think thats correct
you need to introduce at least an enum case


sorry i commit many times so test case may not hightlight in this commitment but i have done it


ﾐﾀﾖｰ　バグ対応ね


> float оставить
Нельзя. Он много где съезжает.


remember results could be 0.

if results is not None:


Great stuff! :smile: :+1: :sparkles:


I then went to the rails console.  I had one child, bruce and I created 2 more children emma and jim.  I then created three sports: swimming, basketball, and baseball.  I then created three sports for bruce:

```bruce.children_sports.create(sport: swimming)```
```bruce.children_sports.create(sport: basketball)```
```bruce.children_sports.create(sport: baseball)```

I created a sport for emma:

```emma.children_sports.create(sport: swimming)```

Finally, I created 2 sports for jim:

```jim.children_sports.create(sport: swimming)```
```jim.children_sports.create(sport: basketball)```

Now, I can find all of the sports for bruce:

```bruce.sports.all```

I can find all of the children involved in swimming:

```swimming.children.all```

This seems to feel correct.  As noted above I am creating a *child -> sport* relationship through the *children_sports* table.  Am I on the right track?  Adding ```has_many :children_sports``` to both the child and sport model seemed to fix things.  Why did that addition make things work?


Profe un readme que no tenía el repositorio, pensé que era algo que debía
tener por si algunos ojos lo ven, dado que el sistema está en línea y los
repositorios son públicos es probable que alguien lo ubique ... dado que
vamos a concentrar allí los parsers incluyendo el que desarrollo para el
rtl pues quizá si alguien llega allí puede ubicarse en el sistema y quizá
tiene alguna colaboración para otro tipo de medición (caso supuesto)

Es solo una descripción de lo que hay allí. Discúlpeme si está mal haberlo
hecho

2015-10-28 3:55 GMT-04:30 Andrés Arcia-Moret <notifications@github.com>:

> Cuenteme .. que es esto?
>
> Saludos
>
> --
> A/A/
> Il 28/Ott/2015 00:06, "Freddy Rondon" <notifications@github.com> ha
> scritto:
>
> > @Maureen-Hernandez <https://github.com/Maureen-Hernandez> Thanks for the
> > contribution.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/spantons/zebra-parsers/commit/17b1d17ca706176b078b90c50dce82c6cef23cc2#commitcomment-14027171
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/spantons/zebra-parsers/commit/17b1d17ca706176b078b90c50dce82c6cef23cc2#commitcomment-14032259>
> .
>

--

Maureen Patricia Hernández González

hernandezgmaureenp@gmail.com

skype: maureenhg9

+58-424-783-6757


Wouldn't it be better to call [GammaController enableOrangeness] here? I'm implementing a widget and this is where widget and main app get out of sync when both can enable/disable.


Ну тогда сделай всё грамотно, я написал в чём проблема.


This fits better in `pkg_pretend` since pkg_pretend is the only phase that can get carried out _before_ e.g. a world update. Then the user can make an informed decision before anything is merged. He won't see pkg_setup warning messages once the world update has started.


u had to comment -_-


The aim of that was to prevent getting a division by zero if `layer_height` is 0 and an underflow if `layer_height` is less than 0. The check for combined_infill_layers to be at least 1 is performed in the input check of `combineInfillLayers`.


Yep


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


But no phoenix ROM now =(


congrats :+1:


ok the whole next section is too hacky


making a data copy so we do not have to drop the dummy column.


Eller varför inte ```scrollToAnchor('#tips');```?


beech


PerfherderArtifact(suites=[phsuite])


further, i have made a similar change -> which looks a whole lot nicer and has all correspondences defined


Kbye


That's a duplicate of `-Wextra`, c.f. https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-W-302


Смотри, в переменные объекта мы обычно кладём то, что является его **состоянием**. Состояние фильма — его название, год выхода, режиссёр и т.д. А вот вес состоянием объекта не является (= он одинаковый для всех фильмов этого класса), и описание¹ тоже не является — оно вычисляется на основании других полей (который, теоретически, могут поменяться, и тогда описание, вычисленное при создании объекта фильм, станет неактуальным)

¹ кстати, слово review не очень подходит — оно намекает, что это «отзыв» (пользователя или критика); длинную строчку, описывающую объект, обычно так и называют — напр., описание=description


Лучше сделать и без флексов, и без float'ов, а добавить что-то вроде
```css
position: absolute;
right: 0;
```
классу `de-win-buttons`. Заодно можно будет убрать этот `margin-left: 32px;`.


If luma and chroma plane has half pixel relative offset, it's obviously noticeable.

But at this point it's (correctly aligned) RGBA plane that we are talking about. It should not be noticed since there are no reference object to confirm the offset actually exists.

After all, it's really up to user whether he/she wants to correct these half pixel offset. If he wants, he can specify "scaler-resizes-only" explicitly.


空行


what am i suppose to be looking at?


Probably should check that this is a symbol and throw an error if not


ert.util.cmem.free :)


Ta bort ```= anchor```. Och egentligen behöver inte det här ligga i en callback alls längre.


same here.


Now I can find all of the sports bruce is involved in:

```bruce.sports.all```


Хм. Как вариант. Щас попроюбую.


Example from plugins.py:

```
    @assertConfigLoaded
    def complete_run(self, text, line, begidx, endidx):
        arguments = self.splitArguments(line)

        if len(arguments) > 2 or len(arguments) == 2 and not text:
            return []
        return autoCompleteList(text, self.getPluginNames())
```


@lexmag thoughts on this?


Не забудь дать `de-win-head` CSS-правило `position: relative;`.


It should be #if defined(__MINGW32__) && !defined(DXGI_ERROR_INVALID_CALL) in order to compile on latest Mingw (5.2) since DXGI_ERROR_INVALID_CALL is already defined there