@AkamaiDASH, I noticed that this line causes TypeError if I click "CC" menu button for the mpds that do not have captions. There is another issue I see - if I play an mpd with CC first and then select a track w/o CC and click the CC menu, instead of TypeError (as in the first case) I still see the list of captions of the previous mpd. I do remember you said this is still work in progress, so you are probably aware of these issues, so I am sharing what I see just in case, so that we don't miss anything.


are these " parts of the file format?


@mitour ，用 slim 改寫 courses/index 之後，Button 預設的間隔會消失，暫時加了這三行 CSS 來確保 Button 看起來跟原本相同，如果有更好的解法或是 CSS 寫法，再麻煩依照妳的習慣整理。


Thanks for the PR.. you're completely right! :) Can you change your commit message to use "mqtt" instead of "mutt" ?


thanks a lot .


Missing a word. The number of AngularJS "components"


This reverts 0451f19. It looks you missed submodule update?


Why is this now conditional on non-Windows?

Windows absolutely needs to use libusb_get_next_timeout(), because libusb_get_pollfds() isn't available on Windows. See bug #343 - the fix was to check libusb_get_next_timeout(), which we hadn't been doing.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Why do we have the error message twice ? We should have an error message only for the extra components I think ?


This seems like a good idea, but I'm curious about the bonus value (seems awfully high to me)?


Just use:
`if (oldRowTextChild)`
It covers the case if a value is `undefined` as well. The only exception is when the value is a number, then `0` would resolve to `false`, which you often don't want


Why not. But "apply" don't exist in F#. The domain return events, not apply events.
But we can apply commands and project events on projection


提交简单的redis链接校验,实现重连


I don’t think that this is the expected output. Maybe it would be better to deactivate the test and create a ticket about it.


Hello Dimitri,
thanks for your prompt response, but I'm afraid that with your fix you reintroduced a bug for which 123 & 123[4-5] = 123 that was fixed with https://github.com/dimitri/prefix/commit/cc62e51246309f88785d99f62fa0bb454463975d


This is json, isn't it?


@wkennington can be fixed by using extraSrc in gcloud-golang. I have a commit lying around somewhere but can't find it atm as I'm on my phone.


Should we not be AST-rewriting one form to the other so we don't have to look for both cases when planning?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.544](http://teamcity.monogame.net/viewLog.html?buildId=9913&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:27:50


:+1:, snygg catch med ```present?```, visste inte att den metoden fanns.


We don't store references to DOM elements, but rather references to their IDs. We sometimes have utility methods that give you back a DOM element, but it is never stored. The reason we do this is because we think of controls as existing in memory rather than in the DOM. The DOM is just the rendered representation of the state of the control. We often re-render controls from scratch which would invalidate the reference to the DOM element.


Remember that you can grant "Review" permission to users without "Approve" permission. That way, they can view/update unapproved records without being able to approve them.

I think you want to enable users to see unapproved records during "update" if they also have review permission (rather than having to use the review-method) - which can be solved without breaking the overall approval-concept.


Same as above: I don’t think that this is the expected output. Maybe it would be better to deactivate the test and create a ticket about it.


probably a bad conflict resolution when merging branches


You can use the JQuery shortcut to get the DOM element : $("#buttonNext")


W,V-==*/>
I'm so tired and hungry and tired of this month in which the two countries in Asia and Africa is the most recent quarter and the first place in my room


Replace one string


:heart:


First, currently the generic type isn't used. Secondly, it's in identity aggregate. It's not repository but Repository Change.
Use generic obviates type union, because implementation will not prove support all types.
But we can move in infrastructure.


You need g++ to compile, not gcc, as this is C++.
It could be an OS thing, though. I'm using a Mac and this is untested on Linux-based systems.


:skull:


כמובן שאל תתיחס להדפסות כי זה רק בשביל צרכי דיבוג, הכל יעבור ללוגר


:beers:


Поигрался я с конфигами: сделал отдельный пустой класс конфигурации для рутового контекста - все работает. Перенес бин туда из конфига сервлета - тоже все работает даже без `@Autowired`
Да, получается я действительно инициализировал оба контекста одинаковым конфигом и одним и тем же бином. Буду знать.


אתה גם יכול לשחק במחלקת טסט עם מלצרים וכמות האנשים שהם יכולים לשרת, למשל תיתן למלצר 1 ו 3 כמות אנשים של 1 ותראה איך 2 הלקוחות מתפזרים ביניהם


Ah, good idea.


No there are too many trinity guys :D


You can kill `BOOTLINT_COV`. It only exists for checking code coverage of Bootlint itself.


:rocket:


@HendrikPetertje what? how is this logical?


And now I'm derp for having read the commits in chronological order


typo when making the first version will append that later
sorry


Исправил в новой версии.


Fantastic. Hopefully things also work with brew and fink. It seems I can't confirm that myself without removing macports first, as the compilation picks the ports version no matter what I do.


puts →　put


85723528_Issue1815 (macumber) - i386-Windows-7-VisualStudio-12: OK (2291 of 2343 tests passed)

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


Since TypeScript 1.5 we started using "let" instead of "var". We should write all new code using let.


This is applied with https://github.com/OpenSC/OpenSC/commit/8da31d271e2f29e6ce67fcf4886301f2b5a7f1d2


This is a great change since anyway we care STObject, not BEntry that is just a R-tree entry.


@anodelman I think it should be:
```ruby
cmd = "#{String(useshell).shellescape} -c \"#{String(cmd.shellescape)}\""
```


1: M111 unrecognized component name: base


Are you sure this isn't a consequence of https://github.com/Zarel/Pokemon-Showdown/commit/bcdb9b2926905cfb66db553c97ee2842f133dadf ?


Is there any reason why `read-char` cannot be used? It can respect enabled input method.


No problem :)


Do you have testcases associated with these?


Are these domains of the Product Branch Circle? Is this the right place for these? :"?":


I'll have a look


I agree and would prefer not to complicate it.  Why do we need
GribV1Record if it knows nothing of the files format?

On 9/1/15, didier <notifications@github.com> wrote:
> Sure, but IMO it's clearer that way,
> GribRecord class doesn't know anything about files format, moreover it
> allows a copy and paste of GribRecord.cpp and GribRecord.h in
> weather_routing plugin.
>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/did-g/OpenCPN/commit/09da9997a6ba42f0a4c6e031ae14a6c56616fc85#commitcomment-12984825


You could simplify this `if` statement to:
`$("#buttonNext").prop("disabled", bottomRecord < (page + 1) * maxLines);`

Same goes for `if` statement below


update


en


pls update the name of Web_Test_Tools_Dev_Guide_v1.0 to align with the comments of https://github.com/testkit/testkit-lite/pull/114


Storing the current page instead of the `from` and `to` record IDs causes the grid to loose track of where it is when you resize the browser window.


Much better :)


fgg


PR created: https://github.com/puppetlabs/beaker-rspec/pull/74


Hi all : after fixed it. I think it can be removed .... Then this fix a feature request : Max number of answers for a survey
* If a quota don't have any answers, test it quit if "completed response" == "quota number" (must be put clearly in manual, and maybe/surely add a string in "quota view")

In your opinion: good or bad Idea?


Instead of `emptyString`, just send through `""`


hmmm true that! ops :P


You broke my builds with it :(


We use `===` almost exclusively. [Here is why](http://stackoverflow.com/questions/359494/does-it-matter-which-equals-operator-vs-i-use-in-javascript-comparisons)


Mark Karpov <notifications@github.com> writes:

> Is there any reason why read-char cannot be used? It can respect enabled input method.

It doesn't read events.

--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997


No need to define the type here, it would be inferred from the expression


TypeScript has support for [template strings](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings). They are awesome!


Why would you want this?
I don't see why pia-tools should interfere with docker/libvirt/etc network devices


My longer term memory is awful, so in this case I can't remember the specific reasoning behind why. I think it's because it just seemed wrong seeing `response.response_headers` in the code.  Or maybe one of the tests/code was using it on another object which only returned `headers`.

Can you give me the specific error, maybe I can wrap that class in a conditional for older versions of Faraday?  Thanks.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1294 of 1294 tests passed)

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


Why this change?  The grib plugin still draws the arrow head.


This prevents a crash of the server when multiple malicious clients are frequently connecting.


Thanks, @davidanthoff, yes, I am aware of FastAnonymous and I am planning to use it indeed.

To answer your question directly, there will certainly be an interface that will allow you to pass a log-target with your desired signature. There may be other minor changes (for example what is now a likelihood model will become simply a parameter), so if there is any small adaptation you need to do in your code (without altering your log-target), I would be more than happy to interact with you and get it sorted.

To give you a couple more design reasons that led me to define the log-target differently at a lower level, it is important to have a state type, because it gathers together fields such as state.value, state.logtarget etc. Then there are two options, either to call f(value) and return logtartget or to call f!(state) and internally (inside f!) do state.logtarget = state.value. I opted for the latter, but will certainly make sure that the interface will not affect the user.

About your constructors, I definitely agree that this is statistically the right definition. I was under the impression that iso-normal and diag-normal in Distributions achieve the same outcome as your code, but I could be wrong about this, so will change it to the code you provided above and will commit the changes to metadata.


Да, просто возвращай `ViewResolver`.


So the pull request is failing because you used // to add the comments. Comments in po files are done with #: tale a look for example at the comments where the string is, for example:

 #: ../qml/components/WizardCategoriesPage.qml:148

Can you update the pull request?


Replace the Korean text with English


testing #1


testing #2


this should be fixed now.

On Wed, Aug 26, 2015 at 8:15 PM, Dmitry Stogov <dmitry@zend.com> wrote:

> I'll think.
>
> On Wed, Aug 26, 2015 at 6:17 PM, Nikita Popov <notifications@github.com>
> wrote:
>
>> Shouldn't we do the same for arrays as well to support symtable
>> serialization?
>>
>> $a = 1;unset($a);unserialize(serialize($GLOBALS));
>>
>> Notice: unserialize(): Unexpected end of serialized data in
>> /home/nikic/php-src/t157.php on line 5
>> Notice: unserialize(): Error at offset 5139 of 5140 bytes in
>> /home/nikic/php-src/t157.php on line 5
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/php/php-src/commit/25f9e255264d8196f51a22707e797977e4cd3995#commitcomment-12906068>
>> .
>>
>
>


explicit запрещает неявное преобразование из класса который принимает конструктор как параметр в констурируемый класс.
Набросал небольшой примерчик с комментариями где ошибки.

```c++
#include <string>

class A
{
  public:
  A(const char* c) {}
};

class B
{
  public:
  explicit B(const char* c) {}
};

class C
{
  public:
  C(const A &a) {}
};

void TestA(const A &a)
{
}

int main()
{
  A a = "test";
  B b = "test"; // MSVC error C2440: 'initializing' : cannot convert from 'const char [5]' to 'B' explicit запрещает неявное преобразование из какого-либо типа в наш объект.
                // MINGW ошибка: conversion from 'const char [5]' to non-scalar type 'B' requested

  C c("test"); //Вызовет A("test")

  C d = "test"; // MSVC делает двойное преобразование (по стандарту допускается одно неявное преобразование)
                // MINGW здесь тоже говорит conversion from 'const char [5]' to non-scalar type 'C' requested

  TestA("test");

  return 0;
}
```


I don't agree with this change because later on (different commit) you pass a -VC value which I don't understand.

Is the current really backward for grib files?  How about climatology?  It's easy to fix one and break the other.  Or is the current correct but the calculation to find the wind speed and direction (over water) incorrect somehow?


I see, due to an unscrupulous cop.
Anyway, the commit log looks incorrect, doesn't it?


谢谢！


The hardcoded "dl" user should be remplaced by something else, like a config options (eg: `TRANSMISSION_USER`). Since in most cases, pia-tools is run as root the config files would land in `/root/.config`. I don't like this.
And please make use of the XDG variables (`XDG_CONFIG_DIR` in this case)


Yes :)


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 tests passed)

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


you really want to build the entire UBO in memory, then do a single glBufferSubData call (or even better, look at Dolphin's buffer streaming, but that's for later).  It's faster to just upload the whole block than keeping track of which ones are dirty.


żeby używać JodaDate potrzebana jest dodatkowa zależność i odpowiednia admotacja na polu


if you use `oview()` instead you don't need the `M == H` check, possibly also the invisibility check.


`index(..)` returns `null`, not `false`.


These variables and the functions `loadDataRows`, `onClickNext` and `onClickPrev` will end up in the global scope. With shuffling around a few bits you could remove all of them.

This is one of JavaScript's weaknesses; quoted from "JavaScript: The Good Parts" :
"The worst of all of JavaScript's bad features is its dependence on global variables. A global variable is a variable that is visible in every scope. Global variables can be a convenience in very small programs, but they quickly become unwieldy as programs get larger. Because a global variable can be changed by any part of the program at any time, they can significantly complicate the behavior of the program. Use of global variables degrades the reliability of the programs that use them.

Global variables make it harder to run independent subprograms in the same program. If the subprograms happen to have global variables that share the same names, then they will interfere with each other and likely fail, usually in difficult to diagnose ways.

Lots of languages have global variables. For example, Java's public static members are global variables. The problem with JavaScript isn't just that it allows them, it requires them. JavaScript does not have a linker. All compilation units are loaded into a common global object.

There are three ways to define global variables. The first is to place a var statement outside of any function:

`var foo = value;`
The second is to add a property directly to the global object. The global object is the container of all global variables. In web browsers, the global object goes by the name window:

`window.foo = value;`
The third is to use a variable without declaring it. This is called implied global:

`foo = value;`
This was intended as a convenience to beginners by making it unnecessary to declare variables before using them. Unfortunately, forgetting to declare a variable is a very common mistake. JavaScript's policy of making forgotten variables global creates bugs that can be very difficult to find."


A description is mandatory in SassDoc.


Zgadza się, jak nie będziesz wiedział, to daj znać, podpowiem.


Very nice. I like zebras


убирай зависимость от домена!


ok


Leave the office clean every day so it's a welcoming place for us and our guests.


fix redmine 3.x.x higher


All users with update access correct (by role & realm, where appropriate).
This seems correct to me.
They are filtered out of lists so wouldn't be stumbled on accidentally, but if the user is given them to edit then they can do so.
I don't see this being an issue in any usecase...not just the current one


Just like the hover color of a negative point gets the negative color, I would expect a point within a zone to get the zone color on hover. Can you have a look at that?


TeamCity ReOpenLDAP :: master [Build 18](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=265&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


:+1:  :heart:


@llasram You're right, interpolation is backwards here. Would you like to open a pull request to fix it?


amended


Жуковец Татьяна, 4 группа, специальность ПОИТ


why are these two commented?


Nu kan vi skicka tillbaka all data i en metod


Its not alias description, its actually firewall WAN/NAT or any other rule description that has been used in the error message.


итьб
смаиптрь


oh, thanks :D


empty if? `unless` ?


`@param {*} $value - Value to test`


both stair pieces go on the lower level, no? or was that changed?


`@return {Bool}`


`@param {Arglist} $values`


This wasn't working originally - the update form during review only got shown due to some of the other changes.
We /could/ look to revert this part  back & not having a separate update button/method...however then we need to create a new custom form for this context & critically we cannot support components of components.
Having the option to go to the normal update environment is much more RAD and  I don't see this as breaking security in any way.


In case you want to complete it, this function is from me I believe. :)


`@param {String} $unit [rem]`


`@param {Number} $base [config-get('i-base-em')]`


Obsolete comment.


Missing `$at` description.


For all with update access...yes at first I saw that as a hole, but on reflection I thought it wasn't - we can still limit to role/realm & those with Update access should generally (probably always) be able to update Unapproved records even if they cannot Approve/Reject them.
There can be many cases where a record needs to be worked on by several hands & so restricting unapproved records to original editor or reviewer doesn't seem right to me.


`@param {String | Null} $trait [null]`


Will think about that. Also about the Cacheable trait. Can discuss that later.


setMethod -> addMethod


Wrong order of parameters.


`@param {Color} $color [config-get('i-border-color')]`


`@param {Length} $width [config-get('i-border-width')]


yes, u'r right


`@param {String} $style [config-get('i-border-style')]`


`$border-style` doesn't exist.


`@param {Color} $background-color`


Bevor i continue to work on all this i need some tests, after this i will finish it and add you as author.


`@param {Color} $light-color [config-get('contrasting-color-light')]`


`@param {Color} $dark-color [config-get('contrasting-color-dark')]`


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 tests passed)

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


Missing description.


This is a bit spammy, You are firing of a request for every row instead of fetching all the records at once. You can simply:
`$.get("records?from=" + count + "&to=" + (bottomRecord - 1), ...`


`@param {String} $unit [rem]`


add the separate `path` parameter so we can still check if `host` passed in is empty or not at L241.


I think this should be done only if the user set that he/she want to see the linter errors only from the current file. If he/she set to display all linter errors from the project then I would expect us to iterate all of the errors in all of the files


`@param {String} $image`


`@param {String} $type`


`@param {Number} $size [100%]`


It still does not work even after i put them on columns. I think the css in uicommons is not seen. Because it all works if i put the css back to adminui.css or the jsp itself.


@q-src why this change? i get an error after this, the param should be an array(?)


If the variable name describes its purpose well (as in this case), no comment is needed


I constructed a toy example to convey my point, which shows the software engineering (design and performance) of my point. Try:

```
type State
  value::Float64
  logtarget::Float64
end

f(x::Float64) = x^2
g!(s::State) = (s.logtarget = (s.value)^2)

n = 1000000

state = State(1.5, 2.3)

@time for i = 1:n
  state.logtarget = f(state.value)
end

@time for i = 1:n
  g!(state)
end
```

Anyhow, it will not affect you as the end user, just giving you some insights of the lower level design :)


The DD4T templates allows you to publish empty fields, If your metadata schema has a field viewModelKey, but the value is empty. in that case this method will return string.Empty. returning string.Empty is causing issues in the Equals method.


Ups sorry. Will have a look at that.


Ah, right. Done in https://github.com/owncloud/vm/commit/293f1cb5e0802c465e1ffc04b44cc04b9e8778c8


http://stackoverflow.com/questions/32326473/how-to-add-class-dynamically-to-list-tagli-in-imagepicker-js
kindly go through this link and help me


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hi


Nice!


Although I don't see it as necessary (I don't see a security hole here), I am OK with giving the users in the SAMBRO context the Review permission.
However I still think we need this to be applicable to the update method...which gets trickier.
The s3rest part is easy - can just change to (pseudocode):
if method == "update"and has_permission(review, table):
    ...
The s3aaa part is much harder though.
The solution I put in is, to me, cleaner & not breaking any security.
(I am aware that the Unit Tests need updating to match this definition)


Fant ved en tilfeldighet ut at armenske språkkoden ikke er `arm`, men `hye`:
http://www.lexvo.org/page/iso639-3/hye

@asgeirr: Hvordan er språkkodene importert til data.deichman.no i utganspunktet? Utelukkende basert på data fra MARC-postene?


Т.е. может защитить и уберечь в некоторых ситуациях? =)


should this be group_one rather than "group one"?


`./sandboxes/code-instrumentation/code-instrumentation` -> `./sandboxes/code-instrumentation/index.js`
So, import will turn into:
```js
import { getAttributesProperty } from './sandboxes/code-instrumentation';
```


step_definitions rather than "step definitions"?


@xdqi
Please fix the travis.
I have upgrade the compileSdkVersion and the build-tools.

Or we kan use the **denpendencies status** to avoid this annoying issue


#SECOND-2 FIxed


group_one rather than "group one"?


Yeah, I just copied them from the TLC role definition so it's more obvious here what domains are available to define policies upon.

As ProductDev Secretary (and a bit as OrgDev) I'm experimenting on how to make our governance easier to use. I also plan to work on these usability issues together as part of Holacracy Learning.


#SECOND-2 Fixed


Does the `version 5.5` stuff makes any difference for you?


DD4T.Core doesn't have any dependencies on MVC, I have moved XPM stuff and some of the Attributes that are depended on MVC, to the DD4T.MVC. If you have the time please review the code, You can find the changes here: https://github.com/dd4t/dd4t.mvc/tree/feature/viewModels;


remve.
instead, in case of exception log it and re-raise


wanna know how it can be edited.


can't we use template type deduction for this?


Yes thanks


really needed?


rewrite


So, to put this requirement together:
- users with "review" permission should be able to open unapproved records with the normal "update" method rather than having to use the "review" method (in fact, with any single-record method including component views)
- however, unapproved records should not appear in list views (or any multi-record methods)

I can do this if you can back out the current changes. The unit tests won't need updating then.


It should navigate to the new group...


It should navigate to the updated group.


this should be in the `it`


1: M110 missing component name


If the fix was to remove a return, that was actually intended.


expect do
   cache.set 42, SecureRandom.uuid, 'not a hash'
end.to raise_error ArgumentError


Exactly, use a register. Probably you need extra 32 bit register (13,14,15,16). Code changes needed.
Shouldn't we close this branch anyway? This is not mio3 specific anymore.


Will probably introduce Glyphicons back as separated files only loaded if an option is enabled.


Not really, as it could trigger conversion to defined push, such as the bool one

> On 1 sept. 2015, at 10:09, Pieter Vantorre <notifications@github.com> wrote:
>
> can't we use template type deduction for this?
>
> —
> Reply to this email directly or view it on GitHub.
>


this bug is fixed in 9b333cc. It should be 0770 for a folder,.


Я нашел хороший пример но к сожалению в жизни пока других придумать не могу. В общем если мы предположим имеем собственный класс реализующий например массив. И в конструкторе хотим передавать размер этого массива. То потом MyArray A = 2 будет как-то не совсем правильно толковаться при чтении кода.


Why is it not yet in https://github.com/ncjones/editorconfig-eclipse/blob/master/pom.xml ?


Some comments on the 6 points we mentioned in the instructions:
1. Brief followed: Volpunte. Only fetched rows that are displayed. Navigation work as expected.
2. Code Style: Feedback in other comments. Code organization could be better by containing variables and functions inside the class instead of polluting the global namespace.
3. User centered thinking: Interaction with grid is intuitive. There is no way to navigate to a specific page in the grid, say if I wanted to see the last page.
4. Comments: Great. Comments throughout.
5. Performance consideration: Only rows are fetched that are displayed and record count and columns are cached. Unfortunately a request is fired off for every row. This would quickly cause the browser to hit its limit for number of connections per hostname. [See limits here](http://www.browserscope.org/?category=network)
6. Aesthetics: Great job. Zebra striping on rows makes it easy for the eye to follow the row from start to end. Headings stand out. Some bugs when re-sizing the browser window, but would be quick to fix.


W pierwszej wersji zapytanie zrobiłem przez adnotacje @Query ale bezpieczniejsze wydawało mi się skorzystanie z wbudowanego w Spring Data Repositories mechanizmu budowania zapytań (http://docs.spring.io/spring-data/jpa/docs/1.9.0.RC1/reference/html/#jpa.query-methods.query-creation). Możesz odesłać mnie do jakiejś dokumentacji? Proszę o odesłanie do dokumentacji, bo nie wiem czym poza konstrukcją będzie różniło się powyższe zapytanie od tego stworzonego z zastosowaniem @Query.


:+1: Thanks for the quick turnaround.  Will look out for 1.6.0.


set expectation that cache.gc is called, then sleep for 0.1


Create ja_JP.inc by @tatsuyaueda


What about something like this:
```c++
void function(int); // this will be selected for int only

template <class T>
void function(T) = delete; // C++11
```


rather rephrase to "...purely software-based openPOWERLINK solutions running on Windows..."


Skorzystałem z @JsonIgnore w konsekwencji zastosowania ResponseEntity<?>, żeby nie wyciągało mi danych z tabelą produktu tabel.


Dynamic hooks are really slower than real hooks. Avoid using this trick would be more MVC. Having the view at almost last moment in the runtime, calling the model and the whole database stuff etc is not the best thing. A controller is preparing a hook, dynamic hooks were made for customising themes quickly but are inefficient I would say( I did not test I just presume)

Regards


Maybe add a sentence about the possibly cycle times with the PCIe card.


Good stuff! :)


This code doesn't even compile!
You cannot seriously commit code that doesn't compile. It's obvious that you haven't tested it!

Also, you must either run "gofmt" manually on any source file that you edit, or use an editor that runs it whenever you save a file. I recommend SublimeText with GoSublime.


I don't see any differences between $smarty->assign('hook', Hook:exec('something')); and {hook h='something'} which do the same thing but maybe it needs more tests


ok


@xiaojun1604 Are you profiling the editor or the device player? The memory footprint of objects as displayed by the editor profiler are oftentimes greater than what they'd be if profiling the dedicated player.


uninstaller


@mystor, can you please mention in the comments _which_ nightly build rust-cpp is known to work with?  It's broken on the latest one :(


Probably, what's annoys me is the time this is processed, in the middle of smarty layer.

Regards


This breaks Basic Auth on some servers. When checking with Wireshark, the request does not contain any Authorization header, so the server replies with error 403.


Intended to PR.


Let's just call it "ping", instead of "pinging"


Fixed in https://github.com/jitsi/jitsi-videobridge/commit/e2495c7336c41c8ff981b6d62bfa5cf896aaa464


There're two dots at the end of the sentence. Think there should be one.


I'm probably missing something, but isn't this just `event.altKey`, `event.ctrlKey`, and `event.metaKey` [on the click event](https://developer.mozilla.org/en-US/docs/Web/Events/click#Properties)?


output staging is where stdout and stderr are processed, too, which are the other two things in the unit sandbox which we look into.  That motivated the placement, and also the desire to keep the CU execution part as small as possible, as the time spent in the execution component directly correlates to the core utilization.


Already addressed in https://github.com/gallardo/opencms-core/commit/b244656ed3a0337dca6f73e7d9824e80b57284f4.


Poszukam czegoś predefiniowanego w Joda.
Muszę przyznać, że brzydkie ale działa.


This is not according to the Connect Holland composer package naming specs. I'd suggest using the name connectholland/project-connectholland-website.


array of layouts


@Buri is right, @Zales0123 could you fix these, please?


To be 100% sure if someone edits this you could add the `,` after nothing as well.


The former assumes the PROF file is actually in the unit sandbox. The latter is a fair point.


This is perfectly fine :)


NB It is actually required to not imply access permission to unapproved records just because the user has update permission. The point here is multi-tenancy: users may be allowed to update records in all realms, but still have access to unapproved records only in certain realms.

If we imply permission to access unapproved records from the mere "update" permission, we may take away the option for org to reserve that access to unapproved records to themselves while sharing approved records with others (which is, in a way, the whole point of approval in multi-tenancy).


because maintenanceCost and dangerPerc are final fields, therefore I did not create any setters for them. actually, I did not really get this part: should i call them? and if so, what are these two methods supposed to do? (as 'final fields cannot be assigned')


What is the `=1.3.0` file for?


Yea, just had this issue during installation. But I find
```php
$gmt = $ds == 1 ? $gmt-1 : $gmt;
```
way less confusing


@aspdye Thanks! One thing though: Please change the second point to:

> :inbox_tray: **Multiple mail accounts!** Personal and company account? No problem, and in a nice unified inbox.

And then remove the unified inbox point. Thanks! :)


Could you add informations about the affected systems? Does this happens with mono or native on Windows? Which version of mono/.Net do you use?


Логотип обычно делают ссылкой на главную страницу.


The former then also dictates that the OutputStagingComponent runs on the same resource as the ExecWorker.


Shall the possible cycle times added as part of the note?


How come you've used two asterisks in `**kwargs`? Splatting only requires one, like `*args`.


%s/defailts/details/


Nice work on refactoring these defaults into constants.


@Richard-Yunchao


На мой взгляд не стоит вводить дополнительные сущности, основанные на содержимом фильтра: infrastructure, houseroom. Содержимое может измениться, и эти классы станут неактуальными. Более правильно будет описать оба блока как «блок фильтра» без привязки к содержимому.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 tests passed)

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


It might be nice to tidy up this and the same expression on line 20 into a callable block?

```ruby
condition = proc { current_time - initial_time < expire_after }

while proc.call
```


Is it possible for `last_sample.get` to return anything other than a `Gst.Sample` or an `Exception`-like object? I realise you're probably just being paranoid (although if it's neither, it's still still returned...).


Note: The correct bench is 8349057 and not what the commit says.


LGTM! Thanks @torfinnberset .


А этот блок фильтра особенный, его тое не совсем корректно привязвать к price, луше охарактеризовать его как «диапазонный блок фильтра».


the PROFs are explicitly places in the sandbox, so that can be assumed true.  See for example https://github.com/radical-cybertools/radical.pilot/blob/devel/src/radical/pilot/agent/radical-pilot-agent-multicore.py#L4186.

I am not sure I understand the point about co-placement.  We do assume a shared FS for the sandbox, don't we?  The same argument would hold for other dependencies which already exist (STDOUT, STDERR, files to be staged in general unless an absolute path is used).  Am I missing something which makes PROF special?


hehe... always one :sweat_smile:


Кнопка «Показать» вероятно нужно будет внести в контейнер с диапазонным фильтром.


hi thanks for making this feature, i really need this feature, hope this PR will merge soon :smile:


You couldn't do it without breaking the API for `save_frame` unfortunately, but it would be nice if you could do `image=None` and if `None` then use `last_got_sample`. I suppose you could move the `stbt screenshot` API to the core code to do the same purpose.


No they are not, as we are moving as much as possible to TMPDIR. Which also invalidates the shared FS assumption.


Нужен класс текущего выделенного пункта.


support rather than source?


Нужны уточняющие классы для стилизации стрелочек.


@andybak, looks good, can you merge this into master ?


As I said before, this test should be under WebGL2 rendering context. Please add code for this. You can find example code like this: if (contextVersion >1) { };


saw approval from gordonguthrie
at https://github.com/basho/riak_ql/commit/b55c11a5f2e760ef38f201f9ef815765027cf16c


merging basho/riak_ql/rts252_binary_field_names = b55c11a5 into borshop-integration-15-rts252_binary_field_names


basho/riak_ql/rts252_binary_field_names = b55c11a5 merged ok, testing candidate = 5dcf4676


So, I think the concept should be to enable users who have review-permission on a record to open that record in any single-record method (READ, UPDATE, DELETE - if they also have permission for that method, of course) in regular CRUD, i.e. including component views.

That way, unapproved records can still be limited to realms while sharing approved records across realms, which is why this was separate. I still see this as required.

And with REVIEW and APPROVE being separate permissions, we can allow many hands to edit a record without implicitly allowing everybody to approve it. But that was always the design - no concept change here.


The bounty is already closed. You could point people towards http://marijnhaverbeke.nl/fund instead


Не очень хорошо, что на странице нет h1, но зато есть h2 (это против спецификации). Кстати, это сильно влияет на SEO?


re: the ability for people with the Review permission to see unapproved records in normal list views.
Intuitively they will expect to see them there...in which case we should add a list_field for approved_by & a filter for Approved/Unapproved (Radio button probably is best UX).
If we don't put them there then we need to add a separate menu.
I think I prefer the intuitive option with less menus.
I think this this the current status, so I see no need to change core but we can add the list_field/OptionsFilter to cap_alert.
(It is then easy for a customise_xx to filter these out of normal views if-desired by a deployment)

The multi-tenancy scenario you give is theoretically possible although very unlikely in practise I think...almost inevitably the update access would be realm-restricted here.
However, I am prepared to allow this possibility & hence require explcitly giving the Review permission to allow Update of Unapproved records.
So this is the new requirement.

To my understanding this requires the s3rest change I highlighted above:
if method == "update"and has_permission(review, table):

The s3crud change stays as-is.

The s3aaa change I will backout & allow you to modify accordingly.

OK?


landscape.io will warn about such style. `Exception` class should be used explicitly.


cc @lethalman


Having a feature to close a survey using quotas if X completed responses were collected is definitely very useful. Your approach may be a first start but it needs to be documented and we should also add a note at the quota GUI how this works.
For future versions having a special "Total completed" item at the quota GUI instead of taking any question without answers would be a huge benefit and make it far easier to use for the general user.


Isn't `go install` going to build that tool again? Seems pointless to me. But ok.


Чтобы в одном случае поставить 2 звезды, а в другом — 4, к классу stars нужен дополнительный подкласс с количеством звёзд.


Make sure that the rest change does check review permission on the record and not just the table ;)

In CRUD, you should put back the "or" - so that it is still possible to update a record when using the "review" method (otherwise you must use "update" which is not available from the list of unapproved records). This is a workflow thing, not about permission.

S3AAA changes may not be required if we require review-permission for unapproved records (because "review" is automatically added to the methods if unapproved=True, so this should just work). But I can test it and make any necessary modifications.


> I think the concept should be to enable users who have review-permission on a record to open that       > record in any single-record method (READ, UPDATE, DELETE - if they also have permission for that     > method, of course) in regular CRUD, i.e. including component views.

Whilist not currently required this is consistent yes, although I don't see this as limited to single-record methods.

No concept change and allowing for rare edge case conditions...at the cost, of course, of complexity for deployers...I'm OK with this but it is a cost.


Hello!
Ah yes sorry: It happens only with Windows, never with Mono.
The server is NemakiWare but I suspect SharePoint has the same problem.
Here is a NemakiWare demo URL: http://trial.nemakiware.com:8080/core/atom/bedroom (admin/admin)
Reverting the paragraph above fixes the problem (but it might have other consequences I haven't tested much).
Thanks a lot!
Nicolas


No, if caches it inside of pkg, technically all it does is a `cp`

On Tue, Sep 1, 2015, 01:43 lethalman <notifications@github.com> wrote:

> Isn't go install going to build that tool again? Seems pointless to me.
> But ok.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/cf92e411eeb98b495ec5fc9c589f945aa3ac9ba1#commitcomment-12993474>
> .
>


Добавлена функция удаления английских слов. Функция предотвращает возможность заменять аббревиатуры полными названиями, имена переменных и т.д.
Нужна ли эта функция.


ok, typos fixed. thanks.


You are right, they are final fields -> you won't be able to create setters for them and will only be able to set them in the constructors. Well, you can simply leave it like this then or remove the final modifier


Yes, i think :
* Update manual
* Adding a "warning line" if there are no "answers/response" set in Quota GUI with something like this : "This quota don't have response : this set a max response to whole survey".

Something like that.

Denis


Execuse me. Does this project support mesos 0.23.0?


Could we make this a deployment setting whether to include unapproved records in multi-record views?

In principle, they could be filtered out per-view where people don't want this - but having an option for a global default behavior might be more useful.


OK, that is answer to the [question "what is afterwards"](https://github.com/marijnh/tern/issues/116#issuecomment-136155834)


This is the line that was causing a warning.  The `allowTriggeringUnreviewedPatches` on the right hand side of the assignment is not a constructor argument like the others, so it resolves to the class member and is just assigning the same member variable to itself.


Maybe even:
```
elif detected_encoding['encoding'] == 'ascii':
     detected_encoding['encoding'] = self.encoding
```


Ah ok.  Not sure where I got that from then :s


Mind to write this in [heredoc](https://de.wikipedia.org/wiki/Heredoc)?


When outputting to the console, it makes sense to only output it once for the entire file.

When highlighting in an editor, it makes sense to highlight every line.

I don't think we can determine which components would be the 'extra' ones. I think only showing an error message for some of the components is a bad idea.


This is just a very minor issue for sure, but I think the affiliations of the PC members are partly out of date. For instance, [Matthias Weidlich is at HU Berlin](http://matthiasweidlich.com/). Andreas Schönberger is at Siemens for four years already. [Matthias Kunze is at Zalando](https://bpt.hpi.uni-potsdam.de/Public/FormerMembers).


@dustinmoris Quick turnaround :+1:


Hi @stof is this fail OK?

https://travis-ci.org/jcalderonzumba/MinkPhantomJSDriver/jobs/78185879


Function shouldBe add if(contextVersion >1){}@Richard-Yunchao


Sounds good. A proper warning message may be something like "You didn't
add any questions to this quota. Thus, the system will just compare all
completed responses with the max value you have set."

Question is what to do if there are > 1 such quotas? Check for the
lowest max value?

Best regards
Marcel

Am 01.09.2015 um 10:46 schrieb Denis Chenu:
>
> Yes, i think :
>
>   * Update manual
>   * Adding a "warning line" if there are no "answers/response" set in
>     Quota GUI with something like this : "This quota don't have
>     response : this set a max response to whole survey".
>
> Something like that.
>
> Denis
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LimeSurvey/LimeSurvey/commit/bccdf8e70e6d788bd64dfa803be37c6b78d9c36b#commitcomment-12993507>.
>


This "seeing of unapproved records" was considered a separate workflow in the original approval concept (i.e. the spec was: records should not enter regular workflows while unapproved), which I can fully understand.

Retaining that option at least as a deployment setting seems necessary to me.


Ja, ist beim Refactoring verloren gegangen. Aber du hast es gefunden :)


Of course - SAMBRO may be different, and require unapproved records to pass through regular workflows. In fact, that /is/ the new requirement.


Not sure, worth trying, but with function there's no partial
implementation. With solution, I'm afraid that derived class that are not
bound will trigger the deleted function, not the base class one. Even if in
this case, it's a valid push

On Tue, Sep 1, 2015 at 10:18 AM, Pieter Vantorre <notifications@github.com>
wrote:

> What about something like this:
>
> void function(int); // this will be selected for int only
> template <class T>void function(T) = delete; // C++11
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/crazyjul/dukbind/commit/46a667dc2678a9da7ce2afa47f8548c112a3b55e#commitcomment-12993056>
> .
>


Hmm, I am confused.  I mean, it would be nice to be able to move the unit sandbox to TMPDIR, but then the input stager and output stager (which populate the sandbox) would also need to run on the same node **as the CU**.   Also, this would make symlinks to the pilot's `staging_area/` impossible.

The only thing we moved to TEMPDIR so far is the shell spawner's work dir, which is not where the PROFs etc end up.  Are we talking about the same thing?


in which version of Angular UI Bootstrap will be available the option popover-is-open ?


可以说中文吗


Makes sense, I will change the commit.


Here might be an interesting lead to follow:

```diff
diff --git a/t/email-content.d/gerrit b/t/email-content.d/gerrit
index 6bbb9f6..49a8114 100644
--- a/t/email-content.d/gerrit
+++ b/t/email-content.d/gerrit
@@ -1,4 +1,4 @@
-$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter Sûb Mitter (sub.mitter@example.com)
+$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter Sûb Mitter (sub.mitter@example.com)
 Sending notification emails to: Refchange List <refchangelist@example.com>
 ===========================================================================
 Date: ...
@@ -11,7 +11,7 @@ Message-ID: <...>
 From: =?utf-8?q?S=C3=BBb_Mitter?= <sub.mitter@example.com>
 Reply-To: =?utf-8?q?S=C3=BBb_Mitter?= <sub.mitter@example.com>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: démo-project
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Oldrev: d245c99162aff6fff4879e5d5c17d454766b45db
@@ -23,7 +23,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Sûb Mitter pushed a change to branch master
-in repository demo-project.
+in repository démo-project.

       from  d245c99   m1
        new  902dfe1   a5
@@ -54,7 +54,7 @@ Reply-To: Joe User <user@example.com>
 In-Reply-To: <...>
 References: <...>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: démo-project
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Rev: 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
@@ -65,7 +65,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Sûb Mitter pushed a commit to branch master
-in repository demo-project.
+in repository démo-project.

 commit 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
 Author: Joe User <user@example.com>
@@ -88,7 +88,7 @@ index 63a911f..7ed6ff8 100644
 To stop receiving notification emails like this one, please contact
 Administrator <administrator@example.com>.
 ===========================================================================
-$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter Submitter without Email
+$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter Submitter without Email
 Sending notification emails to: Refchange List <refchangelist@example.com>
 ===========================================================================
 Date: ...
@@ -101,7 +101,7 @@ Message-ID: <...>
 From: From <from@example.com>
 Reply-To: Submitter
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: démo-project
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Oldrev: d245c99162aff6fff4879e5d5c17d454766b45db
@@ -113,7 +113,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Submitter without Email pushed a change to branch master
-in repository demo-project.
+in repository démo-project.

       from  d245c99   m1
        new  902dfe1   a5
@@ -144,7 +144,7 @@ Reply-To: Joe User <user@example.com>
 In-Reply-To: <...>
 References: <...>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: démo-project
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Rev: 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
@@ -155,7 +155,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Submitter without Email pushed a commit to branch master
-in repository demo-project.
+in repository démo-project.

 commit 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
 Author: Joe User <user@example.com>
diff --git a/t/email-content.t b/t/email-content.t
index ebf8dde..51fc82e 100755
--- a/t/email-content.t
+++ b/t/email-content.t
@@ -144,13 +144,13 @@ test_email_content '' 'test_when_finished "git checkout master"' \

 test_email_content 'Gerrit environment' gerrit '
 	# (no verbose_do since "$MULTIMAIL" changes from a machine to another)
-	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Sûb Mitter (sub.mitter@example.com)" &&
-	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Sûb Mitter (sub.mitter@example.com)" >out &&
+	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Sûb Mitter (sub.mitter@example.com)" &&
+	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Sûb Mitter (sub.mitter@example.com)" >out &&
 	RETCODE=$? &&
 	cat out &&
 	test $RETCODE = 0 &&
-	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Submitter without Email" &&
-	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Submitter without Email" >out &&
+	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Submitter without Email" &&
+	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Submitter without Email" >out &&
 	RETCODE=$? &&
 	cat out &&
 	test $RETCODE = 0
```
Are such changes to the tests reasonable, or is it better to create new tests for Unicode --project command line options?  This uses the French https://fr.wikipedia.org/wiki/D%C3%A9mo.  This kills test 11 `not ok 11 - Gerrit environment` in both Python 2 and 3 (when changing the interpreter on the first line of `git-multimail/git_multimail.py`).


chinese  sofa


@eliben I suppose that this is unwanted change.


Actually, this should not depend on the server side at all (As you already have proven in that the *Request* misses the Authorization header). However this raises the question if MS.Net behaves erroneous or if that is a "known feature". After all Preauth just means "Always present an Authorization header". Perhaps the Credentials are null in some code path?!


#include <iostream>

using namespace std;
class student
{
private:
    int id;
    string name;
    float grade;
    string  level;
public:
    student()
    {

    }
    void set_Student_Data()
    {
        cout << "\nEnter student id name grade\t";
        cin >> id >> name >> grade;

    }

    void calcLevel()
    {
        if(grade > 100 || grade < 0)
            level = "Error"; // Error
        else
        {
                int l = grade /10;

                switch(l)
                {
                case 10:
                    level = "A";
                    break;
                case 9:
                    level = "A";
                    break;
                case 8:
                    level = "B";
                    break;
                case 7:
                    level = "C";
                    break;
                 case 6:
                    level = "D";
                    break;
                 default:
                    level = "F";

                }
        }

    }

    void diplay_Student_Data()
    {
         cout << "\nid\tname\tgrade\tLevel\n";
         cout << id << "\t" << name << "\t"<< grade << "\t" <<level<< endl;
    }
};

int main()
{
    student  st;
    st.set_Student_Data();
    st.calcLevel();
    st.diplay_Student_Data();
    return 0;
}


nit: not sure if it's accurate to say it's MessagingService client, since conversation service will also need this client. Maybe we can say `Initializes client for MessagingService  and conversation service`?


Instead of doing the above, couldn't you just use the sysfs interface and remove fd?


Beautiful :)


I removed it afterwards, created it by mistake u.u


Im not talking about moving the unit sandbox (which is arguably in a contract with the user). Im talking about implementation details like the run script, PROF file, whatever, which don't need to be exposed to the user, and therefore don't need to reside on a shared FS.


一年前的一个练手项目，基本功能有通过GPS以及基站定位的LBS闹钟，与室内的WIFI SSID设定的LBS闹钟，可作为基于地点上下文提醒，过站提醒，手机情景模式自动更改等功能


Maybe the default batchName could be a timestamp, or some other unique identifier?


And running on the command line:
```
$ LC_ALL=C python3 /data/software/git-multimail/git-multimail/git_multimail.py --submitter='Sébastien (seb@example.com)' --stdout --oldrev=HEAD^ --newrev=HEAD --refname=refs/heads/master --project='Démo' -c multimailhook.mailingList=foo@example.com -c multimailhook.from=pusher
Sending notification emails to: foo@example.com
===========================================================================
Date: Tue, 01 Sep 2015 10:58:18 +0200
To: foo@example.com
Traceback (most recent call last):
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 3683, in <module>
    main(sys.argv[1:])
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 3675, in main
    run_as_update_hook(environment, mailer, refname, oldrev, newrev, options.force_send)
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 3361, in run_as_update_hook
    push.send_emails(mailer, body_filter=environment.filter_body)
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 3291, in send_emails
    change.recipients,
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 1949, in send
    for line in lines:
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 865, in generate_email
    for line in self.generate_email_header(**extra_header_values):
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 1181, in generate_email_header
    self.header_template, **extra_values
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 800, in expand_header_lines
    value = header_encode(value, name)
  File "/data/software/git-multimail/git-multimail/git_multimail.py", line 479, in header_encode
    return Header(text, header_name=header_name, charset=Charset(charset)).encode()
  File "/usr/lib64/python3.4/email/header.py", line 217, in __init__
    self.append(s, charset, errors)
  File "/usr/lib64/python3.4/email/header.py", line 304, in append
    s.encode(output_charset, errors)
UnicodeEncodeError: 'utf-8' codec can't encode character '\udcc3' in position 2: surrogates not allowed
```


Oops, just noticed I didn't push the removal of the file


I implemented it based on this documentation:
https://msdn.microsoft.com/de-de/library/system.net.webrequest.preauthenticate%28v=vs.110%29.aspx
Do you use a custom authentication provider?


> Make sure that the rest change does check review permission on the record and not just the table ;)

True ;)

> In CRUD, you should put back the "or" - so that it is still possible to update a record when using the        > "review" method (otherwise you must use "update" which is not available from the list of unapproved      > records). This is a workflow thing, not about permission.

A 3rd button for 'Edit' is in Biplov's commit to be merged.
I can see the utility of an embedded update form for simple records although it's not that intuitive.
I assume that the 'Approve' button would act as a 'Save & Approve'?
It's confusing having so many buttons with unclear implications to users.

>S3AAA changes may not be required if we require review-permission for unapproved records            > (because "review" is automatically added to the methods if unapproved=True, so this should just       > work). But I can test it and make any necessary modifications.

It didn't just work, which is why the modifications were made.
The SAMBRO template's 'Alert Approver' role 1st saw 'record not found' (due to the s3rest change being needed) and then 'not permitted' because they weren't the created_by of the record.
I think I can fix this myself...just testing.


When you have few backgrounds to swatch then when you scroll down and blokcs that activates background switch are visited 1st time then backgrounds are changed but when you visit those blocks 2nd time, e.g. by scrolling up then backgrounds are not changed anymore.


I think the code should use the public constant for `spark.sql.parquet.mergeSchema` instead (to remove possibility of typo or another inconsistency).


How about putting a nice panorama picture of Vienna into header? :)


NB There is a certain risk if you allow unapproved records to enter regular workflows that they become undeletable (due to RESTRICT references) and can no longer be rejected.

Unfortunately, preventing actions which make unapproved records undeletable is not possible at the global level - so that needs to be taken care of in the individual workflows.


Wow nice man


> Could we make this a deployment setting whether to include unapproved records in multi-record views?

I'm OK with that, but note that I am not making any changes to that currently...I am OK with the current setting (include)


10/10 could not replicate


Hey @tcard. No worries. Ok thanks, I'll give that a try when I get a spare minute. Thanks for the history too :+1:


:+1: thanks to add a way to keep sourcemap in concat workflow. You save my day!


@andrewhickey @daniel-abbey


YEs


85723528_Issue1815 (macumber) - Win64-Windows-7-VisualStudio-12: OK (2304 of 2343 tests passed)

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


nice , add bin in .gitignore


I meant it ```Initializes client that will manage connection to MessagingService```, will make it clearer :)


sd


c**o**mes


I get

     syntax error, unexpected tCONSTANT, expecting keyword_then or ';' or '\n' (SyntaxError)

on this line. Adding parenthesis fixes it:

    value.is_a?(String)


chec**k**


We have a class to store username/password, but it is just for storage convenience, and in the end we just send everything to DotCMIS. This happens in the "Add folder" wizard, just after entering the CMIS URL/username/password, here is the code:
```
            Dictionary<string, string> parameters = GetParameters();
            parameters[SessionParameter.AtomPubUrl] = url.ToString();
            parameters[SessionParameter.User] = user;
            parameters[SessionParameter.Password] = Crypto.Deobfuscate(obfuscatedPassword);

            // Create session factory.
            SessionFactory factory = SessionFactory.NewInstance();

            // Return repositories.
            return factory.GetRepositories(parameters);
```
(in the CmisSync.Auth.GetCmisRepositories method)


You can specify security constraints directly from your routing by providing a role or an array of roles with the `roles` parameter.


learned not trained? Are they interchangeable?


I think the name Subscribe is better but meh what do I know...


You **can** provide


you **can** specify many rules


Hey @swdevbali,

thanks for you kind words :D

I still think that such simulation approach is not the best for testing kivy apps. As I created this example, i thought that selenium like is the best: selecting with xpath was working good for browsers. But in case of kivy we have access to direct data model of the application, so we don't need such a bridge for selection, we can access data model directly. There couple of problems with this way also, but still it shows much better results.

I think i will publish "yet one more" tool for testing in kivy one day ;)

Cheers,
Sergey


Since this is hyped atm, why not add microservices to the topics? I am thinking of something like "Microservices (architecture, lifecycle, deployment)"


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.485.3da15e10b314e2cbc911263482e52cf228c07bd1](http://build.palaso.org/viewLog.html?buildId=62408&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:16:55


Is intersting but imho clearly not réserve to be in on the index


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.464.3da15e10b314e2cbc911263482e52cf228c07bd1](http://build.palaso.org/viewLog.html?buildId=62407&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3356, ignored: 134 Build time: 00:17:36


Yes I revert some of it later, but I'm not sure, it could be a double mistake elsewhere, but in your master branch with current gribs I have , which are displayed correctly, it doesn't work the effect of current is reverted, with a motor polar the boat is moving in the wrong direction. Easy to see here, we have currents up to 10 knots.
I will double check for wind speed over water.

I haven't look at climatology yet.


tisdagen den 1 september 2015 01.59.28 skrev  Fran Boon:
> A 3rd button for 'Edit' is in Biplov's commit to be merged.
> I can see the utility of an embedded update form for simple records although
> it's not that intuitive. I assume that the 'Approve' button would act as a
> 'Save & Approve'? It's confusing having so many buttons with unclear
> implications to users.
The problem is that an "update" button will not return to the list of
unapproved records ;) therefore the idea to have "Review" act like the normal
"Open" (i.e. implicit update form if the user has permission). It could in
fact just be renamed into "Open".

The idea behind this update option was that the reviewer could add e.g.
comments to the record, i.e. simple changes related to the review-approve
workflow - not major update processes.

The key thing is that it should return to the list of unapproved records.

-1 to an "Update" button there - doesn't quite belong into that workflow.


Sounds good.  Thanks.


Maybe we need a sysfs.read_int() so that we could skip .fd and use that instead.


I think the code could benefit from using constants for `0.0` and `1.0` a lot. It appears in so many places.


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 tests passed)

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


Sounds good to me! Nice investigation. :+1:


Looking good now, not tested yet (will do that later)
thanks!


lgtm


Seconded. I have a PR on the way.


Is this rake task needed since we have this one? https://github.com/FundingCircle/shipment_tracker/blob/master/lib/tasks/db.rake#L14-L17


No


dd


You can't do that. the `currentVersion` variable is not used but you still need to set the current version.


This has to be reviewed by our designer, @abolishme.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.486.0a9383e4154958702e922729a58d801f8a0a09ec](http://build.palaso.org/viewLog.html?buildId=62412&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:06:17


This broke iso_minimal_new_kernel: http://hydra.nixos.org/build/25314303


> The key thing is that it should return to the list of unapproved records.

A nice touch which we can do by adding a _next to the update URL...it may get lost when opening tabs, but worth putting in


Ok


yeah so I can't use % in my git messages. Just imagine they say %.


ack


是不是需要更新服务端和客户端才能用这个图文混排啊？

能接受的图片内容是不是类似以下这种格式的？
[IMG WIDTH=123]测试图片[/IMG]
[IMG WIDTH=456][/IMG]

里面的widht单位是像素吧？不带width参数是不行的吧？


...however, such an "Edit" button would break out of the workflow - which seems unclean.

Being able to add comments to a record while reviewing (e.g. as a note to the original editor, or to other reviewers/approvers) is useful, but breaking out into major edit-workflows doesn't seem to belong here.

Do you still see a need for that if users can open unapproved records from regular workflows?


@sindresorhus is right, and I was giving it some thought earlier on. Now not only does pageres (or *insert library here* library) have to up-keep their own documentation, someone will have to take the time to update it here, too. While in theory a "lot of traction" would keep it in check, practicality states that it **definitely will never happen like that**.

Instead, why not use the projects' existing docs (creating new `wat-xyz` repos for things like built-in Javascript functionality, perhaps) and create an index that lives in the root of wat that has a key/value dictionary of names/repositories. In the event you want something from that name, you can `git clone` the repository down and just use the docs from it.

Just a thought. It depends entirely on your ability to parse data, though. As well, it depends on whether or not the project is `wat` compatible. I could foresee more projects making their docs `wat` compatible being a more feasible approach than to keep two copies of documentation up to date between two separate, unrelated repositories.


Technically no "solution" for that is required Mazi.

Basically if you specify multiple quota, each is evaluated in order, and
the first one to exceeded will / should stop the process.

On Tue, Sep 1, 2015 at 10:51 AM Marcel Minke <notifications@github.com>
wrote:

> Sounds good. A proper warning message may be something like "You didn't
> add any questions to this quota. Thus, the system will just compare all
> completed responses with the max value you have set."
>
> Question is what to do if there are > 1 such quotas? Check for the
> lowest max value?
>
> Best regards
> Marcel
>
> Am 01.09.2015 um 10:46 schrieb Denis Chenu:
> >
> > Yes, i think :
> >
> > * Update manual
> > * Adding a "warning line" if there are no "answers/response" set in
> > Quota GUI with something like this : "This quota don't have
> > response : this set a max response to whole survey".
> >
> > Something like that.
> >
> > Denis
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/LimeSurvey/LimeSurvey/commit/bccdf8e70e6d788bd64dfa803be37c6b78d9c36b#commitcomment-12993507
> >.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LimeSurvey/LimeSurvey/commit/bccdf8e70e6d788bd64dfa803be37c6b78d9c36b#commitcomment-12993591>
> .
>


> I think this this the current status

Actually it was but only for the original author, which is even more inconsistent/confusing!
I've left the deployment_setting as an @ToDo for now.
The original usecase was DRRPP which I don't see getting an update in the near future.


akc


Ah, so you talk about what will be, not what is, got it.  So lets move it around when that becomes reality then?


`filterNot` perhaps to improve readability?


@BenjaminKim regex가 익숙하지 않아서 실수했네요. 많이 부족한 코드입니다 ㅠㅠ


Well, what "is" here locally. What "is" in the repo doesn't really work.


Run this to sync static files before:

    cd /var/www/my_geonode
    sudo python manage.py collectstatic


Hi,

I apologize for the question, but what is the underlying problem you're trying to fix?

Some parts of this patch may be good or required (setting the `SCS`, clearing the `COV` bit - not sure), but the "avoid an overflow" login does not make much sense to me.

`(TBCCTL6 & ~COV)` - what is the point of this condition? It will be truthy whenever some other other bits are set in `TBCCTL6`. Which will obviously happen, because the code itself sets some bits to nonzero. Is the intended meaning  `!(TBCCTL6 & COV)`? Even so, why have this check at all?

Also, if a double capture happens, then is probably just means that the DCO speed is too slow to be calibrated with the current method. Have you actually observed double capture events?

If this patch is good, `msp430/f2xx/msp430.c` should also be fixed.


I would generally prefer that changes for different issues will be handled separately.
How do you think about to [reuse topic branches](https://git-scm.com/book/en/v2/Git-Branching-Branching-Workflows "Branching workflows") eventually?


TeamCity ReOpenLDAP :: master [Build 18](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=265&buildTypeId=ReOpenLDAP_Master&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:32:50


NYCP is the other use-case with record approval, and there we don't want unapproved records to enter regular workflows (and there is also that risk of unapproved records becoming undeletable).

Haven't quite been thinking of DRRPP anymore ;)


Yep, Sam have right. Same for other Quota :first one by order is the default. Updatebale via plugin event.


心凉了


i.e. your changes break NYCP, this is where I'm coming from ;)


@aithal-i2 :+1:

> Just try to make most of markdown format. :)


@sunsidew 네, 괜찮습니다. 그 부분만 제가 정리해서 다시 올려드릴께요.^^


удалить везде префик group_, слово "группа" уже есть в названии класса


удалить лишние импорты


Папки bin, .settings, test-output не заливаем в репозиторий


Even worse, the documentation here will get outdated and I as a project maintainer will have to handle support questions about `pageres` not working according to the outdated `wat` docs. Outdated documentation is worse than none, IMHO.

-

Some resources to existing solutions to this:

- Create parsers for various kinds of docs: https://kapeli.com/docsets (You can actually use these docsets in `wat`)
- Golang has a standardized way to do docs: https://golang.org/pkg/archive/tar/ We sorely need something like that for Node.js.

-

This discussion should be moved into an issue.


Without emoticons I tend to answer as is ;)


服务端不用，不过因为你刚才的需求所以还是更新下。客户端等我Release一个版本
是的
是的，不带width参数我判断不出来大小，所以干脆强制了


You are right. I mostly copied it from a path on SIGA ( https://github.com/iCarto/siga/commit/c487c02b1759637c3b734622518fa19ba3f1905a ) without thinking so much to avoid have a forked version of ELLE on SIGA.

Constats feature need a full rewrite. To not break SIGA and keep the old behaviour i will vote to write again load(IProjection), loadViewLayers(IProjection) and loadOverviewLayers(IProjection) as they were previously. It introduces duplication but avoids change anything in client code.

What do you think?


ack


Thanks! https://github.com/yiisoft/yii2/commit/2085dda183642bde361595f92ed36d2baecadf4e


ak


Vulnerability is another use-case of record approval, but that has its own workflows and isn't currently used, is it?


But your comment here is visible, so maybe simply comment what a better commit comment would have been?


Normally, the server should answer with a http 401 response, but if I visit the server URL with a normal browser like firefox the server returns http response 500. Am I right? If that is the normal behaviour of your server, the authentication could fail.


commit message shortlog is too long


Sorry, the diff should have been:

```diff
diff --git a/t/email-content.d/gerrit b/t/email-content.d/gerrit
index 6bbb9f6..0480933 100644
--- a/t/email-content.d/gerrit
+++ b/t/email-content.d/gerrit
@@ -1,4 +1,4 @@
-$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter Sûb Mitter (sub.mitter@example.com)
+$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter Sûb Mitter (sub.mitter@example.com)
 Sending notification emails to: Refchange List <refchangelist@example.com>
 ===========================================================================
 Date: ...
@@ -11,7 +11,7 @@ Message-ID: <...>
 From: =?utf-8?q?S=C3=BBb_Mitter?= <sub.mitter@example.com>
 Reply-To: =?utf-8?q?S=C3=BBb_Mitter?= <sub.mitter@example.com>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: =?utf-8?q?d=C3=A9mo-project?=
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Oldrev: d245c99162aff6fff4879e5d5c17d454766b45db
@@ -23,7 +23,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Sûb Mitter pushed a change to branch master
-in repository demo-project.
+in repository démo-project.

       from  d245c99   m1
        new  902dfe1   a5
@@ -54,7 +54,7 @@ Reply-To: Joe User <user@example.com>
 In-Reply-To: <...>
 References: <...>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: =?utf-8?q?d=C3=A9mo-project?=
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Rev: 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
@@ -65,7 +65,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Sûb Mitter pushed a commit to branch master
-in repository demo-project.
+in repository démo-project.

 commit 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
 Author: Joe User <user@example.com>
@@ -88,7 +88,7 @@ index 63a911f..7ed6ff8 100644
 To stop receiving notification emails like this one, please contact
 Administrator <administrator@example.com>.
 ===========================================================================
-$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter Submitter without Email
+$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter Submitter without Email
 Sending notification emails to: Refchange List <refchangelist@example.com>
 ===========================================================================
 Date: ...
@@ -101,7 +101,7 @@ Message-ID: <...>
 From: From <from@example.com>
 Reply-To: Submitter
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: =?utf-8?q?d=C3=A9mo-project?=
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Oldrev: d245c99162aff6fff4879e5d5c17d454766b45db
@@ -113,7 +113,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Submitter without Email pushed a change to branch master
-in repository demo-project.
+in repository démo-project.

       from  d245c99   m1
        new  902dfe1   a5
@@ -144,7 +144,7 @@ Reply-To: Joe User <user@example.com>
 In-Reply-To: <...>
 References: <...>
 X-Git-Host: fqdn.example.org
-X-Git-Repo: demo-project
+X-Git-Repo: =?utf-8?q?d=C3=A9mo-project?=
 X-Git-Refname: refs/heads/master
 X-Git-Reftype: branch
 X-Git-Rev: 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
@@ -155,7 +155,7 @@ Auto-Submitted: auto-generated
 This is an automated email from the git hooks/post-receive script.

 Submitter without Email pushed a commit to branch master
-in repository demo-project.
+in repository démo-project.

 commit 902dfe1c4025851d6b175c8f1efeee9ee1a0b74d
 Author: Joe User <user@example.com>
diff --git a/t/email-content.t b/t/email-content.t
index ebf8dde..51fc82e 100755
--- a/t/email-content.t
+++ b/t/email-content.t
@@ -144,13 +144,13 @@ test_email_content '' 'test_when_finished "git checkout master"' \

 test_email_content 'Gerrit environment' gerrit '
 	# (no verbose_do since "$MULTIMAIL" changes from a machine to another)
-	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Sûb Mitter (sub.mitter@example.com)" &&
-	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Sûb Mitter (sub.mitter@example.com)" >out &&
+	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Sûb Mitter (sub.mitter@example.com)" &&
+	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Sûb Mitter (sub.mitter@example.com)" >out &&
 	RETCODE=$? &&
 	cat out &&
 	test $RETCODE = 0 &&
-	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Submitter without Email" &&
-	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project demo-project --submitter "Submitter without Email" >out &&
+	echo \$ git_multimail.py --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Submitter without Email" &&
+	  "$PYTHON" "$MULTIMAIL" --stdout --oldrev refs/heads/master^ --newrev refs/heads/master --refname master --project démo-project --submitter "Submitter without Email" >out &&
 	RETCODE=$? &&
 	cat out &&
 	test $RETCODE = 0
```


why sleep?


это не используется, можно удалить


commit message: why do we need this?


If you are testing the application on chrome without a server please run this command in command line
FOR MAC
$ open -a Google\ Chrome --args --disable-web-security
FOR PC
$ chrome.exe --disable-web-security


default values are part of a good functionality, the thing should work well with defaults. Not sure if we want this...


Папки test-output, bin, .settings не заливаем в репозиторий. Заливаем тлько папку src


ah next commit... ack


I didn't see this PR before I submitted https://github.com/raspberrypi/noobs/pull/276 which contains a complete translation for Italian, not derived from this one.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.465.0a9383e4154958702e922729a58d801f8a0a09ec](http://build.palaso.org/viewLog.html?buildId=62411&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 3354, ignored: 134 Build time: 00:21:14
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.CommandLineProcessing.CommandLineRunnerTests.CommandWith10Line_NoCallbackOption_Get10LinesSynchronously: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.FallbackToFirstKbd: <no details avaliable>

```


I do not say we should only have one error message.
We shouold have one message per extra rules. So for example, if I have 10 components in a file, and we have configured the rule to accept only 5, we can display the error messages five times, for the last 5 items defined in the file.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 tests passed)

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


Oh, awesome. Didn't realise the source was on Github, could've opened a PR! :+1:


Following @tombye comments


cái này là em update mà ghi nhầm create new


+1 Thanks!


ddd


add a `key`


add `idx`


I would keep the compatibility with as much LTS lines as possible. Even if the most of Jenkins installations use 1.609.x and above, the most of big-scale installations still use old cores and even custom forks due to stability requirements. If there is no particular reason to upgrade, I would recommend to change the dependency to 1.565.x line.

Additional comments:
* You've added the mandatory dependency on matrix project. It is not recommended, because the plugin may be decoupled from Jenkins core at some point. If there is no need in such dependency, it's better to remove it
* 2.0 version presumes that you are going to break the binary compatibility or to change the data formats on the disk. Do you really have such plans?


1: M110 missing component name


35:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from wtforms.validators


37:1: I100 Imports statements are in the wrong order. from urlparse should be before from sqlalchemy.event
42:1: I100 Imports statements are in the wrong order. from .noteutils should be before from .receivers


48:1: F999 redefinition of unused 'WtgTAG' from line 46
48:1: F999 redefinition of unused 'WtgTAGRecord' from line 46
48:1: F999 redefinition of unused 'wash_tag' from line 46


I recommend to reuse [the macro "AC_CONFIG_MACRO_DIR"](https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Input.html#index-AC_005fCONFIG_005fMACRO_005fDIR-61 "Finding configure input") instead of a direct inclusion call here.


Hi,

Please find my answers inline.

> Hi,

>I apologize for the question, but what is the underlying problem you're
trying to fix?

When I am using rtimer peiodically in Cooja (the same problem was not
observed in hardware). The simulated mote sometimes gets stuck in "while(
!(TBCCTL6 & CCIFG))" forever indicating a double capture and overflow. I
think this is due to periodic rtimer use (as far as I can tell, I am using
rtimer correctly).

>Some parts of this patch may be good or required (setting the SCS,
clearing the COV bit - not sure), but the "avoid an overflow" login does
not make much sense to me.

>(TBCCTL6 & ~COV) - what is the point of this condition? It will be truthy
whenever some other other bits are set in TBCCTL6. Which will obviously
happen, because the code itself sets some bits to nonzero. Is the intended
meaning !(TBCCTL6 & COV)? Even so, why have this check at all?

And (TBCCTL6 & ~COV) is the same as !(TBCCTL6 & COV). This condition breaks
the while loop when a double capture happens (Again I dont know why this
happen in Cooja).

>Also, if a double capture happens, then is probably just means that the
DCO speed is too slow to be calibrated with the current method. Have you
actually observed double capture events?

>If this patch is good, msp430/f2xx/msp430.c should also be fixed.

I think so, I send a patch for only f1xx since I only tested the patch for
sky platform.

Furthermore, I highly encourage people familiar with mspsim to have a look
at the way it handles rtimer events.

Regards,

Sedat

On Tue, Sep 1, 2015 at 12:19 PM, Atis Elsts <notifications@github.com>
wrote:

> Hi,
>
> I apologize for the question, but what is the underlying problem you're
> trying to fix?
>
> Some parts of this patch may be good or required (setting the SCS,
> clearing the COV bit - not sure), but the "avoid an overflow" login does
> not make much sense to me.
>
> (TBCCTL6 & ~COV) - what is the point of this condition? It will be truthy
> whenever some other other bits are set in TBCCTL6. Which will obviously
> happen, because the code itself sets some bits to nonzero. Is the intended
> meaning !(TBCCTL6 & COV)? Even so, why have this check at all?
>
> Also, if a double capture happens, then is probably just means that the
> DCO speed is too slow to be calibrated with the current method. Have you
> actually observed double capture events?
>
> If this patch is good, msp430/f2xx/msp430.c should also be fixed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sgormus/contiki/commit/48aacbad9769049d4acc7dc62f718cb7fe9fbb5d#commitcomment-12994074>
> .
>


rename in `value` in order to be able to work with `fieldFor`


Does this now work for you so that I can adapt the unit tests?


Yes that's a known bug in NemakiWare. It should return 401 indeed.
Could it be the actual cause of the problem?


document `value`


Why?


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Comment Testing


Hi @warden,

  You don't have to convince me about the interest for everyone (users and developers) to try to keep stuffs up-to-date and thus requiring Java 6 (and I hope soon J7) is fine like it is to require a "recent" version of Jenkins core. The problem is to define what we are calling recent for us and users. The current LTS is based on 1.609 and as there is no API change in 1.609.x versions for me it is better to start the support to the fist one (1.609.1). We are agree to advice to users to upgrade to 1.609.2 or 1.609.3 versions ASAP but we have to be pragmatic and we know that in many IT services upgrades are done on few times per year and thus it can takes months to see it done.
  In parallel the fact that Atlassian recently upgraded its cloud instances on atlassian.net to a version which doesn't support SOAP anymore makes the usage of the 2.0 version of the plugin critical. That's why I think we should do our best to deliver it as soon as possible and for the largest part of the community because your plugin is widely used.

Cheers


Would you mind specifying *what* is not working?


Why? What are the 8 bytes you think need to be skipped? The specification doesn't have any data before first_mb_in_slice, as far as I can see?


Totally agree. This dependecy is pulled by tempto transitively by mistake. Internally tempto uses slf4j for logging.
I will fix that in tempto.


因為在第二行所 render 的 `comments/form` 當中 new 了一個物件，因此在印出 comments 的時候會多一個沒有內容物件，暫時透過 `unless comment.new_record?` 來避免。


(mein öffentliche ipv6 addresse - damit die Breminale Seite auch  nach der Breminale noch per IPv6 erreichbar ist
wollte kein extra Klone der Webseite auch noch auf meinem ffhb server erstellen, daher eine IPv6-addresse aus "deinem"/"meinem" Bereich)


@cviral30 And those are?


Yes, that is the problem! The server doesn't act like Microsoft expects it. Mono always sends the Authorization header if "conn.PreAuthenticate = true;" is set. .Net starts with the Authorization header only if the server asked for it once with the 401 http status. After one successful authorization the Authorization header is added to all following requests.
https://weblog.west-wind.com/posts/2010/Feb/18/NET-WebRequestPreAuthenticate-not-quite-what-it-sounds-like


Use a variable.  `rgba($black, 0.2)`


Vulnerability isn't used in Production yet, no.
NYC isn't really broken by this I would argue, although it certainly could have the deployment_setting set differently to SAMBRO


@cviral30 Only readme can be mardown?


You don't need the parenthesis around the require_once argument.


conversation?


There is no need to multiply by the COP as `capacity_in kW`already is the heat output capacity. Related to https://github.com/quintel/etmoses/issues/217


Good spot.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.466.0a9383e4154958702e922729a58d801f8a0a09ec](http://build.palaso.org/viewLog.html?buildId=62413&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 3355, ignored: 134 Build time: 00:11:38
### Failed tests
```
PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.CombinedIbusKeyboardSwitchingAdaptorTests.FallbackToFirstKbd: <no details avaliable>

```


這一行因為如果 form_for 丟進去的 record 是現在的 `[course, course.comments.build]`，依照慣例會跑去找 `course_comments_path`，在找到更好的寫法前，暫時透過指定 `:url` 來解決。

同時，為了讓之後更新畫面可以比較方便、避免 `id` 重複造成的問題，手動給定 `id` 來解決。


Is this right? Creating a `BuildableItem` as well as a `FlyWeightItem`? If it is, maybe add a comment explaining what's happening and why both are needed etc. Seems a bit strange.


:+1: for rc


Can we delete these lines?


This volume (100 liter) conflicts with the volume that is set in `heat_pump_classes.py` (1000 liter). Either is fine, but they need to be consistent. Part of https://github.com/quintel/etmoses/issues/217


:boom:


There is no need to multiply by the COP as `capacity_in kW` already is the heat output capacity. Part of https://github.com/quintel/etmoses/issues/217


I've fixed, but it might still raise invalid_arg, since the underneath functions will raise if the dhcp packet is malformed.


@prudhomm (just for information)
That will modify the timeout on every plateforme. You can use the `feelpp.site.*.cmake` file for a specific plateforme with
```bash
set(DEFAULT_TEST_TIMEOUT "60")
```
(yes: the variable name is `DEFAULT_TEST_TIMEOUT` with no `FEELPP_` prefix in that case).


This will need to be adopted to latest HTML reporting refactoring.


Can be moved there. I think that original intention was to keep tests as separate from product code as possible.


NIT: one line if statements without braces around the code block.


3 stages are:
1. Working directory
2. Staging area
3. .git repository


No, we can have any text file which can be wrriten in markdown language.


We should print a warning here (mentioning that we ignore the assignment, and then exit with 0)


I was tempted to split the commit ... but in the end things were related and I hacked this thing in pretty quickly .. it anyway requires a bit of review and it's going to change.
I'd like to move all autoconf extra files somewhere else than the source root tree but I'm still a bit fighting with autoconf &co


Oh I was not aware of this... sorry for the trouble!


Also needs a log message here, but this one should probably be failing.


刚刚提交了问题到服务端的这边：
https://github.com/zsxsoft/danmu-server/issues/1

那就是说，我目前的老版服务端和老版客户端先不用安装，直接等你把新版客户端发布以后我再安装新版服务端和新版客户端是最合适的了，对吧？2者配合起来一起更新才最好。

谢谢


The 2 sets of nested if's in here are identical other than the filename that is being checked. Could that be separated out into a separate proc so that if we want to change the ordering in the future we only have to change it in one place?


Hmm, this was lacking an OOM check before... We should really take the chance and add one.

just `if (!l) return log_oom()`


tfSize instead of sizeOfTF
and for sizes use size_t


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1814 of 1814 tests passed)

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


this is not good change something


This should be applied to cm-12.1 as well.
Thank you the MMS in my build is work now.


This is a really nice change Dan, excited to try RC2 soon! Thank you!


Isn't think going to make the unit testing less deterministic. I.e. it might pass one run, fail another ?


Better: Your browser does not support file uploads for security reasons.


Does your comment apply also to QueryStats class? Currently it is defined in the same package as BasicQueryInfo.
If we want to have a QueryStatsClient in presto-client module we should probably move QueryStats class there too (like you do with QueryResults class). Right?


Hmm, there seems to be some inconsistency, here. You don't seem to use cmd, here but you use it in the trigger_cmd function. I'm not sure about the details of systemd init scripts implementation but this seemed like worth pointing out.


How far does it go back? Is IE8 still supported?


1: M110 missing component name


I thought of the interfaces as sweetspot.


NYCP, as per previous spec, does not want unapproved records to appear in regular workflows, no - but have them confined to the explicit "Review" workflow.

Allowing reviewers to open unapproved records in any single-record method would not harm, I don't think - it just that including of unapproved records in lists and reports which isn't desirable.


I'd suggest to use a `either` return value... similar to what we do in tls: [interface](https://github.com/mirleft/ocaml-tls/blob/master/lib/reader.mli#L24), the [actual implementation](https://github.com/mirleft/ocaml-tls/blob/master/lib/reader.ml#L431) might throw/raise, but is [protected by a catch](https://github.com/mirleft/ocaml-tls/blob/master/lib/reader.ml#L27).

thus, a caller of the parser does not need to check for potential exceptions, but has to check and handle the different result types explicitly (instead of using a monad, certainly [rresult](http://erratique.ch/software/rresult) could be used).


It's a matter of taste, IMO it reduces clutter, there's a lot more wind arrows displayed than with the grib plugin, could be an option both here and in grib plugin though.
Moreover :
- I'd like to add current arrows (It's there but it reuses wind arrows ), ie more clutter.
- with opengl they are ugly, not as ugly as current arrow > 3 knots but ugly nonetheless.


thanks for all your contributions! Hope you every best!


Nice work, sir.


참고로 new delete 사용하면 밀리초단위까지 걸림;;...


if (


@brucepreston I've finally got round to looking at this. I've added a few minor comments inline. Also, would it be possible to add a couple of unit tests for the _sort_by_source_priority proc? (somewhere in lib/cucumber/tcl/test)?

I'll need to get Matt to look at the ruby stuff as I'm no expert on that.


I may be wrong, but as I understand it, mmix is the correct command. mmix and smix replace cmix in 1.10


Looks good


Considering that XELB will (hopefully) be used in projects other an EXWM, I think we should not make any assumption on how it will be called.


There looks to be a function called "stringFromCFString" in osquery/core/darwin/conversions.cpp that can take care of converting this to a std::string "safely" - where you wouldn't have to worry about guessing sizes of cpu_id.


thanks Brady for your time. Do you want me to fix it ?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 483](https://teamcity.labkey.org/viewLog.html?buildId=269571&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:24
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


gctronic changes for bug fixes


nit: why is tabIndex=-1 needed?


Здесь по-хорошему нужно подменить полную ссылку вида `[some text](http://some-address/path/)` на один лишь тайтл вида `some text` и уже его спанить.

Я очень плохо знаком с Java. Насколько я понимаю, тут у нас text иммутабельный. Собственно, вопрос: насколько правильно будет тут пересоздать текст с нужной строкой и спанить уже его? Мне почему-то кажется, что это не особо хорошо.

И, если так не стоит делать, то как лучше сделать? Не выльется ли необходимость подменять некоторые куски текста в рефакторинг всего отображения текста?


What is this for?  This can be optimized a bit.  What does `current_user` do?


Plugins can load, but no events or commands yet, So unless writing a world
generator, provider or a Protocol addition, plugins can't do much (yet).

On Tuesday, September 1, 2015, legoboy0215 <notifications@github.com> wrote:

> YAY!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RedstoneLamp/RedstoneLamp/commit/e7ce759ae72e40e37ce0c9da52e5803ccfb1a26a#commitcomment-13011878>
> .
>

--


read lastSuccessfulRequest.get() before ticker.read()


@tlwalla1 Please review.


Still getting an exception when RenderChainUi == Identity


SingleFeat rollout can be very time consuming, allow cancel between PID


Thanks - I've actually been thinking about making this guide into a script, to use on a clean install. If I get anywhere I'll send you a PR.


To prevent the tab from being selected.


What is the reason for adding the string interpolation?


thanks!


same like above


use sizeof function
strncpy(setsec->micAlg, str, sizeof(setsec->micAlg));


Technically incorrect, politically correctness is not my concern.
There were no string interpolations, `#$gvar`, `#@ivar`, `#@@cvar`, nor `#{expr}`.
Double-quoted string does not always have a string interpolation.
Double-quoted and single-quoted strings only differ syntactically, the former allow all escape sequences and interpolations, but the latter allow only `\'` and `\\`.
When you change from

~~~ruby
foo = "bar=#{bar}"
~~~

to
~~~ruby
foo = "bar=" + bar # or
foo = sprintf("bar=%s", bar)
~~~

then you can say "removed a string interpolation", but this commit is not.


please use stncpy instead of strcpy, this may lead to buffer overflow.
strcpy(setsec->username, str);
strncpy(setsec->username, str, sizeof(setsec->username));


I think using `unwind-protect` (hidden in nice portable macros) would serve those projects best, don't you?


Fixes #1


I am backing out this commit.  This is not an acceptable picture


Fixes #2


Hmm, I thought I'd removed the need for that one. Apparently not.


Forgot to mention, but also added support for Thermal Expansion power storage in this commit.


LGTM.  As a note, saying:

    Fixes: I#31191

...will make issues.bestpractical.com track the branch, and update the bug when it's released.


don't need a 'use strict' when using es6 import