Better use "!=" instead of "≠" to avoid this in the future


why? it's only used in two places


and should not be customisable


IMO, we should not transfer the ownership like that. If you need a copy, then just make it. But reading this function, I don't think you need it nor you should actually call the `g_key_file_free` function.


Мне не понятно название media-wrapper, что оно делает?


тут внутри дива picture ещё будет, там ведь 3 разные картинки


Um, messing with the reset here and adding 1px of border will most likely screw up the layout. ;)


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


You could move it into the `while` loop.


```
return dateValue.isBefore(this.getToday());
```


If it is not customisable, than it is ok.


You could move it inside the `if` clause where it is used.


That was not expected.


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


Some programs require newline to properly process files, we should probably keep it.


Ah right how do you compile to .net standard using the build project?


It is nice to document the responsibilities of imported modules


Doctype for strict mode was missing.


I'm getting a bunch of ruby errors about name not being a class or something. Pinning ASDK to `1.9.91` fixes it. `PINRemoteImage` is pinned because it broke my build previously...

```
  pod 'AsyncDisplayKit', '~> 1.9.81'
  pod 'PINRemoteImage/iOS', '= 3.0.0-beta.3'
```


JavaScript strict mode was missing.


Named arguments for a more descriptive function call.


I think you meant @returns ?


As I hinted, @NathanW2, I am now not sure how to update the qgis2web dialog from changed QGIS settings. I guess another button, labelled I'm not sure what, should requery QGIS as when the dialog first appears.


Descriptive variable names for date objects.


Hi @drmas, its been  a while that you have made this commit. One thing I wanted to discuss was is there a need to enable this ?.. well actually I have just made a pull request in which only if the text is not empty will it be pushed to the database.

`_addItem() {
    AlertIOS.prompt(
      'Add New Item',
      null,
      [
        {text: 'Cancel', onPress: () => console.log('Cancel Pressed'), style: 'cancel'},
        {
          text: 'Add',
          onPress: (text) => {
            if (text) { this.itemsRef.push({ title: text }) }       /** Will push only if text is not empty */
            else { console.log("Text Input empty") }
          }
        },
      ],
      'plain-text'
    );
  }`

Let me know your views  :)


у тебя два абсолютно одинаковых блока, а классы ты им дал разные, это не логично.


I have merged your changes. Thanks.


What kind of settings do you need to watch?


дывоаж


ыждовфа


фыав


Complete test coverage for all combinations of price conditions.


wrong param name


When a broadcast marked selective, two methods should be added to the DBus xml.
For example for below in a fidl file
```
 broadcast XXX selective {
 ...
 }
```
we will need following in the DBus xml

    <method name="subscribeForXXXSelective">
      <arg direction="out" name="status" type="b">
      </arg>
    </method>
    <method name="unsubscribeFromXXXSelective">
    </method>


can \r sneak in also?


This is not how you use `GError`. You should make it a pointer, not a pointer to a pointer. `GError *error = NULL;` and then pass `&error` to functions that take the `GError **error` parameter. In general functions that take `GError **error` parameter can expect one of two scenarios (any other scenarios are programming errors):
- `error` is `NULL` - the caller doesn't care about the specific error, but only whether the function failed or not (as indicated by its return value being usually `NULL` or `FALSE` in case of the failure).
- `error` is not `NULL`, but `*error` is `NULL` - in this case `*error` is where we store the pointer to the error

Anyway, this function should have a `GError **error` parameter IMO, so you could use this.


Alt+Tab as a temporary workaround - works.
But it pops up another unnecessary window (e.g. firefox), and tab applies to spring before window-switch, thus switching camera modes: Overhead (TA) -> Overview style.
(Using linux and default KDE window manager).
Often it is more convenient to simply move mouse into other screen (without window pop-ups and camera style switching).

Anyway, thanks!


Make sure calculator doesn't return erratic values for invalid arguments.


whatever, i'd recommend to keep it as m because recovery don't use ANSI_CPRNG, mark it built in kernel just making zImage bigger


мне кажется, что здесь заканчивается header и начинается отдельный промо блок, как в седоне, помнишь?


hy


There should only be one return statement at the end of the function.


You should check if the `system_revision` is `NULL` instead. If we receive the `error` variable from the caller, then it may be `NULL`. Dereferencing it will crash.


A bunch:

- addition/removal of layers
- layer visibility
- layer stacking order
- style changes
- changes to fields (eg edit widgets, aliases)
- addition/removal of scale bar

Probably lots of others I can't think of off the top of my head.


Enable JavaScript strict mode.


два заголовка оберни в обёртку, это немного общее содержимое, которое, к тому же, тебе потом нужно будет двигать


@splix @rguiu just proposal - static classes and methods, i think, better to separate obviously on top from other class elements:

class AAA {

  // here all static variables, methods and classes

  // here other variables, class constructors and ...

}


Finish Homework 3


I'm not sure about it, really. I think you should just return from the function in some way instead of continuing.


Named arguments using destructuring for a more descriptive use of the function.


Make sure price is a valid number, otherwise the function could break.


Rather check if `home_revision` is NULL. See above for reasoning.


100 set var
       "Burned>8"
         Q= int> 8
         R= int<2

200 If Sum Q,A = 0xA
        Echo LatLong2
        Else Echo LatLong4
Set Echoloc@ 2bb 9705

$ example 1,alphanum,00100
Query1 peril is White Auto tt"
Query2 R.JENKxxxx
Query3 Irene,Eliene, Abigale,Yorkshire, Hopi


Probably forward the error to the caller instead of continuing.


Hey, sorry for breaking you. Could you tell me how you are using these functions? I didn’t even realise we have this sub-README lying around until now.

Packager will go through a lot of changes in the next months, and I need to be aware of what people are using.


Have you checked all over `.py` files in the template for quite consistency? Might as well hit them all in 1 go.

If you're feeling edgy you could do the external apps too.


All valid product types are clearly listed. Very easy to add in new ones.


@bra1n OK, see #29 for my implementation. :)


что ты видишь, когда смотришь на макет, а там куча одинаковых элементов "о нас"?


All valid user types are clearly listed. Very easy to add in new ones.


Yes, I think so.


мне кажется, это очень похоже на цитату пользователя?


You could move it to the scopes where it is actually used.


а если там цитата, то можно лучше разметить эти элементы


- labels (field and formatting)


а блок с кнопками где?


Forward the error to the caller?


Hier ist noch ein Fehler...


Did you forgot to add JmapJamesServer class ?
How is this interface defined ?


Product and user types are strings. Numeric values without an enumeration type is very unintuitive.


@tombentley Are you sure this material really belongs in the _very first chapter_ of the tour?!

It is _already_ documented [here](https://ceylon-lang.org/documentation/1.3/tour/language-module/#numeric_literals).

I don't personally think we need to cover the whole language in the first chapter.


Not exactly:
* The value is not sent to Elasticsearch
* This is only for parsed/typed value at the object root, the raw data is stored into the `csv` object without modification


Only one linear path through the function logic. No nested switch and if-conditions. No multiple return points.


@quozl, honestly, I don' remember.


мне адрес не очень напоминает таблицу


Endlich gefixed! Coole Lösung!
![bitmoji](https://render.bitstrips.com/v2/cpanel/7681590-202294460_1-s1-v1.png?transparent=1&width=246)


Descriptive variable name makes condition easier to read and understand.


Missed that one, explains it. I'll update my values.


Not possible to break the calculator by feeding it an invalid date.


TeamCity Mobile :: Android :: Android SDK [Build 704](http://teamcity.hq.judo/viewLog.html?buildId=72332&buildTypeId=JudoAndroid_AndroidGradle) is now running


и я у тебя не увидел контейнера, для содержимого, ты его собрался на структурные элементы вешать?


Also created popUp with animation and fixed the website's error management system.


TeamCity Mobile :: Android :: Android SDK [Build 704](http://teamcity.hq.judo/viewLog.html?buildId=72332&buildTypeId=JudoAndroid_AndroidGradle) outcome was **FAILURE**
Summary: Gradle exception (new); exit code 1 (new) Build time: 00:00:46


обёртку для формы, и её стилизовать


Enable strict mode.


текст к форме не имеет такого близкого отношения, чтобы находится внутри


You could move it to the scope where it's used.


если не будешь использовать флексы, то ок)


Ciao Fede,

the PR has been merged. Just touch the Producer for making electron
configurable and do another PR as soon as you can. Alice can work in
parallel since I think you just need to touch the LeptonProducer and the
cfg.

Cheers,
RAffaella

2016-11-16 10:34 GMT+01:00 Fedespring <notifications@github.com>:

> Ciao Raffaella
>
> Il 16/Nov/2016 10:04, "radogna.raffaella" <notifications@github.com> ha
> scritto:
> >
> > Hi Fede,
> >
> > I agree with you that not all my point are very critical. I agree also
> that
> > these can be implemented next, and don't need to change your PR.
> >
> > One sentence makes me worry, I have to admit: "I mean, the code works
> >
> > unless you are not using the
> > WW or TauTau filter (already broken), the best would be that you approve
> > the PR and who is doing the next PR, please implement them."
> > What do you mean? the filters were working even if one should submit with
> a
> > bit of care DY inclusive and WW only and separately. Do you mean that
> with
> > your changes this is not true any more?
>
> No no I mean that, before my pull request,histos.py was broken for un
> indentation error here, at the filter, just it.
>
> >
> > Your PR is merged. But PLEASE let us know if your changes broke the
> > filters.
> > Then, please could you implement the comments in the Lepton Producer I
> > mentioned?! This is quite urgent since I have to work on the trigger in
> the
> > same file and I don't want to touch the electron part. It is important to
> > have the electron collection configurable from the cfg.
>
> It is configurable from cfg as in the past, I will put a comment at my next
> commitment. Please go ahead, don't be worried, in case I can show you this
> afternoon where you can put a comment, if you prefer to comment this part
> asap.
>
> >
> > All other changes in hystos.py will be implemented by me or Alice in next
> > PR.
> >
> > Thank you for let the electron selection work again in our code and with
> > mini AOD!
> >
> > Raffa
> >
> >
> >
> >
> > 2016-11-16 1:20 GMT+01:00 Fedespring <notifications@github.com>:
> >
> > > Ciao Raffaella,
> > > the replies are in lines
> > >
> > > On Tue, Nov 15, 2016 at 7:59 PM, radogna.raffaella <
> > > notifications@github.com
> > > > wrote:
> > >
> > > > Ciao Fede,
> > > >
> > > > thank you for the update!
> > > > It’s good that we can proceed.
> > > >
> > > > simply checking the code I would like to report comment.
> > > > Please try to check/fix as much as these points. Some of them could
> even
> > > > been implemented in next PR by Alice I think.
> > > >
> > > > in histos.py
> > > >
> > > > I would like to have the FULL list of selections back! you never know
> > > what
> > > > you need. especially for prescale
> > > > Now I see just
> > > > cuts = {'Our2012' : OurSelectionDec2012,
> > > > 'EmuVeto' : OurSelectionDec2012, # this switches on the dRMuEl veto
> > > > 'Simple' : OurSelectionDec2012, # The selection cuts in the module
> listed
> > > > here are ignored below.
> > > > }
> > > >
> > > Yes right, I cleaned it because I don't like to see too much obsolete
> > > things. But please restore only what could be useful like the
> "prescale"
> > > and not the entire list that was before, useless in my opinion.
> > >
> > >
> > >
> > > > and I think these indentations are WRONG
> > > > https://github.com/Fedespring/SUSYBSMAnalysis-
> Zprime2muAnalysis/blob/
> > > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/test/
> DataMCSpectraComparison/
> > > > histos.py#L219 <https://github.com/Fedespring/SUSYBSMAnalysis-
> > > > Zprime2muAnalysis/blob/0a41fc03a379402aae886ce7442a2a
> 6d5eeba320/test/
> > > > DataMCSpectraComparison/histos.py#L219>
> > > >
> > > > https://github.com/Fedespring/SUSYBSMAnalysis-
> Zprime2muAnalysis/blob/
> > > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/test/
> DataMCSpectraComparison/
> > > > histos.py#L232 <https://github.com/Fedespring/SUSYBSMAnalysis-
> > > > Zprime2muAnalysis/blob/0a41fc03a379402aae886ce7442a2a
> 6d5eeba320/test/
> > > > DataMCSpectraComparison/histos.py#L232>
> > > >
> > > >
> > > yes yes I know, but I realise that only after the PR, and since there
> were
> > > further commitments to do I decided to implement it at the next round.
> > >
> > >
> > > > ntuplify default to False to let data working!
> > > > https://github.com/Fedespring/SUSYBSMAnalysis-
> Zprime2muAnalysis/blob/
> > > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/test/
> DataMCSpectraComparison/
> > > > histos.py#L255 <https://github.com/Fedespring/SUSYBSMAnalysis-
> > > > Zprime2muAnalysis/blob/0a41fc03a379402aae886ce7442a2a
> 6d5eeba320/test/
> > > > DataMCSpectraComparison/histos.py#L255>
> > > > as before I suggest to put it to true only for files we need! so as
> it
> > > was
> > > > in 74X branch (having a list of samples where I explicitly ask sen
> info.)
> > > > We don’t need it for all of them.
> > > > Actually if you put it to false for submit data and you forgot then
> you
> > > > save no sen info, that is a problem for madgraph samples for instance
> > > >
> > > >
> > > ok it can be done
> > >
> > >
> > >
> > > > Then, about the Lepton Producer:
> > > > why I don’t see electron variables declared here in the class any
> more?
> > > > https://github.com/Fedespring/SUSYBSMAnalysis-
> Zprime2muAnalysis/blob/
> > > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/
> > > Zprime2muLeptonProducer_
> > > > miniAOD.cc#L39 <https://github.com/Fedespring/SUSYBSMAnalysis-
> > > >
> Zprime2muAnalysis/blob/0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/
> > > > Zprime2muLeptonProducer_miniAOD.cc#L39>
> > > > And why do you prefer to not have them configurable here:
> > > > https://github.com/Fedespring/SUSYBSMAnalysis-
> Zprime2muAnalysis/blob/
> > > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/
> > > Zprime2muLeptonProducer_
> > > > miniAOD.cc#L484 <https://github.com/Fedespring/SUSYBSMAnalysis-
> > > >
> Zprime2muAnalysis/blob/0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/
> > > > Zprime2muLeptonProducer_miniAOD.cc#L484>
> > > >
> > > > not sure this is good for the future. Let’s suppose you have to
> update
> > > > something…. better to pass them via cfg. do you agree?
> > > >
> > > >
> > > Yes and this is what is still done, but simply the way to access the
> > > electron is changed
> > > https://github.com/Fedespring/SUSYBSMAnalysis-Zprime2muAnalysis/blob/
> > >
> 0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/Zprime2muLeptonProducer_
> > > miniAOD.cc#L85
> > > https://github.com/Fedespring/SUSYBSMAnalysis-Zprime2muAnalysis/blob/
> > >
> 0a41fc03a379402aae886ce7442a2a6d5eeba320/plugins/Zprime2muLeptonProducer_
> > > miniAOD.cc#L65
> > > https://github.com/Fedespring/SUSYBSMAnalysis-Zprime2muAnalysis/blob/
> > > 0a41fc03a379402aae886ce7442a2a6d5eeba320/python/
> > > Zprime2muAnalysis_cff.py#L39
> > >
> > >
> > >
> > > > Could you let me know if you want to update this? do you want me to
> > > > approve your PR and then you fix/check this?
> > > >
> > > What needs to be changed are the first two things that you reported.
> Since
> > > they are not critical, I mean, the code works unless you are not using
> the
> > > WW or TauTau filter (already broken), the best would be that you
> approve
> > > the PR and who is doing the next PR, please implement them.
> > >
> > > Thanks a lot for the checks,
> > > Federica
> > >
> > >
> > >
> > > >
> > > > Raffaella
> > > >
> > > >
> > > >
> > > > > Il giorno 14 nov 2016, alle ore 15:24, Fedespring <
> > > > notifications@github.com> ha scritto:
> > > > >
> > > > > Ciao guys,
> > > > > so analysing the RunG PromptReco data, and comparing to the
> Giovanna's
> > > > file
> > > > > (before my modifications), I can confirm that we get exactly the
> same
> > > > > numbers for both the Simple and the Our2012 muon selection, and I
> have
> > > > for
> > > > > mass > 1 TeV 38 events.
> > > > >
> > > > > So you can go ahead with the pr.
> > > > > thanks
> > > > > federica
> > > > >
> > > > >
> > > > > On Mon, Nov 14, 2016 at 12:43 PM, federica primavera <
> > > > fedespring@gmail.com>
> > > > > wrote:
> > > > >
> > > > > > I'm still waiting for one last job that is in post processing.
> > > > > >
> > > > > >
> > > > > > On Mon, Nov 14, 2016 at 11:29 AM, alfloren <
> notifications@github.com
> > > >
> > > > > > wrote:
> > > > > >
> > > > > >> Hi Fede,
> > > > > >>
> > > > > >> I agree with Raffaella, Friday it was not fully clear that the
> > > > matching
> > > > > >> is perfect. I would also not put mass cut but rather check once
> you
> > > > have
> > > > > >> all your job done the total number of entries in the histogram
> for
> > > > Simple
> > > > > >> and Ourselection. It is important modification and we need to be
> > > > careful.
> > > > > >> Once you see that everything looks perfect then sure we should
> go
> > > > ahead.
> > > > > >>
> > > > > >> Thanks for the checks!
> > > > > >>
> > > > > >> Alice
> > > > > >> > On 14 nov. 2016, at 10:51, radogna.raffaella <
> > > > notifications@github.com>
> > > > > >> wrote:
> > > > > >> >
> > > > > >> > Hi Fede,
> > > > > >> >
> > > > > >> > well, I have been busy with my shift the full weekend and
> > > honestly I
> > > > > >> thought that we should have waited your job finishing since you
> said
> > > > that
> > > > > >> you have 7 jobs and 1 missing events. So I thought to myself to
> > > > consider to
> > > > > >> wait up to monday. Then, after you is Alice that should put her
> work
> > > > first.
> > > > > >> >
> > > > > >> > Since, btw, the weekend is over…let us know if your jobs are
> > > > finished
> > > > > >> and the matching is complete now ;)
> > > > > >> >
> > > > > >> > Thank you,
> > > > > >> > Raffaella
> > > > > >> > > Il giorno 13 nov 2016, alle ore 16:29, Fedespring <
> > > > > >> notifications@github.com> ha scritto:
> > > > > >> > >
> > > > > >> > > Ciao guys,
> > > > > >> > > Why my pr is still pending? I understood that it was urgent
> to
> > > > check
> > > > > >> that
> > > > > >> > > everything was really ok, and I did.
> > > > > >> > > Of course I also fixed that issue.
> > > > > >> > > F.
> > > > > >> > >
> > > > > >> > > Il 13/Nov/2016 07:03, "radogna.raffaella" <
> > > > notifications@github.com>
> > > > > >> ha
> > > > > >> > > scritto:
> > > > > >> > >
> > > > > >> > > Hi Benjamin,
> > > > > >> > >
> > > > > >> > > this is a well known issue ;) discussed in the software
> meeting
> > > as
> > > > > >> well…
> > > > > >> > > Next merged PR this will be fixed.
> > > > > >> > >
> > > > > >> > > Raffa
> > > > > >> > >
> > > > > >> > > > Il giorno 12 nov 2016, alle ore 21:17, Benjamin
> Radburn-Smith
> > > <
> > > > > >> > > notifications@github.com> ha scritto:
> > > > > >> > > >
> > > > > >> > > > Hi - this causes a problem for me. I just downloaded
> > > everything
> > > > > >> into a
> > > > > >> > > new area and I am getting:
> > > > > >> > > > python encountered the error: <type
> > > > 'exceptions.IndentationError'>
> > > > > >> > > > unindent does not match any outer indentation level
> > > (histos.py,
> > > > > >> line 251)
> > > > > >> > > >
> > > > > >> > > > Removing the extra space from L251 allows me to run. Can
> you
> > > > please
> > > > > >> check.
> > > > > >> > > >
> > > > > >> > > > —
> > > > > >> > > > You are receiving this because you are subscribed to this
> > > > thread.
> > > > > >> > > > Reply to this email directly, view it on GitHub <
> > > > > >> https://github.com/cms-
> > > > > >> > > analysis/SUSYBSMAnalysis-Zprime2muAnalysis/commit/
> > > > > >> > > 61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#commitcomment-
> > > 19798478>,
> > > > or
> > > > > >> mute
> > > > > >> > > the thread <https://github.com/notifications/unsubscribe-
> > > > auth/AE5tK_
> > > > > >> > > k8nXst7AN0qJcNLiYfS1pykc5Qks5q9h7ugaJpZM4KwhCe>.
> > > > > >> > >
> > > > > >> > > >
> > > > > >> > >
> > > > > >> > > —
> > > > > >> > > You are receiving this because you are subscribed to this
> > > thread.
> > > > > >> > > Reply to this email directly, view it on GitHub
> > > > > >> > > <https://github.com/cms-analysis/SUSYBSMAnalysis-
> Zprime2muAn
> > > > > >> alysis/commit/61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#
> > > > > >> commitcomment-19800484>,
> > > > > >> > > or mute the thread
> > > > > >> > > <https://github.com/notifications/unsubscribe-auth/
> > > > > >> AJHFO3ey0rnsaRWoyfWFeE3-aNr01fzsks5q9qhDgaJpZM4KwhCe>
> > > > > >> > > .
> > > > > >> > > —
> > > > > >> > > You are receiving this because you commented.
> > > > > >> > > Reply to this email directly, view it on GitHub <
> > > > > >> https://github.com/cms-analysis/SUSYBSMAnalysis-Zprime2muAn
> > > > > >> alysis/commit/61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#
> > > > > >> commitcomment-19802280>, or mute the thread <
> > > > > >> https://github.com/notifications/unsubscribe-auth/AE5tK3IYh
> > > > > >> TCJ7GHObejzGfbPAZzjd_pbks5q9yzrgaJpZM4KwhCe>.
> > > > > >> > >
> > > > > >> >
> > > > > >> > —
> > > > > >> > You are receiving this because you authored the thread.
> > > > > >> > Reply to this email directly, view it on GitHub, or mute the
> > > thread.
> > > > > >> >
> > > > > >>
> > > > > >> —
> > > > > >> You are receiving this because you commented.
> > > > > >> Reply to this email directly, view it on GitHub
> > > > > >> <https://github.com/cms-analysis/SUSYBSMAnalysis-
> > > > Zprime2muAnalysis/commit/61f7eb7fe28f773bb1c4e3fa0b5185
> > > > e317b4e5c8#commitcomment-19808936>,
> > > > > >> or mute the thread
> > > > > >> <https://github.com/notifications/unsubscribe-auth/AJHFO_
> > > > 5OK4CuFUUKLPo2kTBIvjwctJhEks5q-Df9gaJpZM4KwhCe>
> >
> > > > > >> .
> > > > > >>
> > > > > >
> > > > > >
> > > > > —
> > > > > You are receiving this because you commented.
> > > > > Reply to this email directly, view it on GitHub <
> > > https://github.com/cms-
> > > > analysis/SUSYBSMAnalysis-Zprime2muAnalysis/commit/
> > > > 61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#commitcomment-19811778>, or
> >
> > > mute
> > > > the thread <https://github.com/notifications/unsubscribe-auth/
> > > > AE5tKz9pERUWLS9U13fJ9ivq2eM1ENqdks5q-G8ggaJpZM4KwhCe>.
> >
> > > > >
> > > >
> > > > —
> > > > You are receiving this because you commented.
> > > > Reply to this email directly, view it on GitHub
> > > > <https://github.com/cms-analysis/SUSYBSMAnalysis-
> > > Zprime2muAnalysis/commit/61f7eb7fe28f773bb1c4e3fa0b5185
> > > e317b4e5c8#commitcomment-19833980>,
> >
> > > > or mute the thread
> > > > <https://github.com/notifications/unsubscribe-auth/
> > > AJHFO5JrwPygHoxN0SbJ4FjwZkcn0794ks5q-gElgaJpZM4KwhCe>
> >
> > > > .
> > > >
> > >
> > > —
> > > You are receiving this because you commented.
> > > Reply to this email directly, view it on GitHub
> > > <
> https://github.com/cms-analysis/SUSYBSMAnalysis-Zprime2muAnalysis/commit/
> 61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#commitcomment-19838173
> >,
> > > or mute the thread
> > > <
> https://github.com/notifications/unsubscribe-auth/AE5tK0wU320Q0_
> ZFBiD09eJVgWj1d1snks5q-kw5gaJpZM4KwhCe
>
> >
> >
> > > .
> > >
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub, or mute the thread.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/cms-analysis/SUSYBSMAnalysis-Zprime2muAnalysis/commit/61f7eb7fe28f773bb1c4e3fa0b5185e317b4e5c8#commitcomment-19842248>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AE5tKxdXb6s0c8tSHoleHP5f-PnS8nBQks5q-s5AgaJpZM4KwhCe>
> .
>


класс прописать для заголовка


This is a sample from imooc for Python crawler, and it's used for baidu baike, to pull out baike url, title and descrption, include 6 components: html downloader, html output, html parser, url manager and main function.


кажду пару input + label обернуть в див обёртку, либо див обёрткой сделать label ( что мне кажется менее избыточным) и внутри для текста inputa сделать span.


This cast is not necessary.


Unused variable.


зачем прятать? у тебя вон в легенде твоей написано тоже самое.


And why do you push directly to the master?!


tailing space


Hmm,  i don't know what does cprng exactly do and where, i just enabled it because of crypto. I thought it's related to crypto and encryption. Still, things get fitted into with this enabled.
I thought, you commented because of "=m"


![](https://github.trello.services/images/mini-trello-icon.png) [Partie 7.a : Classement Heros selon V / F](https://trello.com/c/oMKlqGGO/19-partie-7-a-classement-heros-selon-v-f)


Instead of being hard coded into the `travis.yml` file, the version number can probably be inferred from the rockspec file, somehow.


Fix the invoking method of service

To handle the request ,the method service have to invoke doGet or doPost. Compared to doGet,  doPost is safer because that information of committing cannot be shown in the address field.


@bc-karel-blavka This is not release 1.0.0, is it?!


All text-files had mixed line endings and indentation. That is fixed for all files to use `\n` line endings and four spaces for each indentation step.


Nein, die letzte Info (weiß grad nicht mehr genau von wem, bryce glaube ich) war, dass wir die relese von 0.91 "fast-tracken" würde, aber da ich davon bis jetzt nichts gesehen habe und mit 0.92 eh niemand mehr an 0.91 interesse haben wird glaube ich nicht dran, dass da noch was kommt.

Habe das auch nicht manuell geändert sondern über search&replace ;-)


no, most of stuffs you've enabled have nothing to do with twrp


Hi, sachin . Your blog is too good and very attractive video man . Thank you so much for that ..

And now i have got some errors and doubt . When i ask question like "who are you?"or something it is able to process and give the answer. But if i ask what is your aim? it shows processing immediately the next line comes as ask question . I dont know what is the issue i face ..

The next thing is when i ask esp listen turn on the light , it says "it is not able to understand"and so on  . Then I took the weather api and embed in the code , when i ask "weather information of chennai" it doesnot respond anything .

Could you help me to sort it out .


We have a UI bug permissions validation. We're getting the error 'Expected to find a method called "vm.validateJson" in the Controller'. This commit fixes it but I have no idea why. Why would using $scope work, but the 'controller as' syntax not work?

It's been broken since [this commit](https://github.com/trainline/environment-manager/commit/b31808b78a0aca0425303212227c40b4cd6978ec).


as far as I know, it is 440 in stock


You could move it to the scopes where it is actually used.


ok, I understand. Thank you very much.

2016-11-16 11:16 GMT+01:00 Mikael Svahnberg <notifications@github.com>:

> It is good that you have answered these questions too, but it was not
> really necessary (these discussion points were intended to be used as a
> warmup during the seminar). In the future, focus on the "Homework" sections.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/Arron-dong/LSRE---Zhipeng-Dong-/commit/f795d8c9f8d0ff01fcfab9b01e80a2b57fb8aef4#commitcomment-19842771>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/APVEz4iSQrHU741BQZ9lZqhAqHENdmsiks5q-tgJgaJpZM4Kzpy6>
> .
>


Hallo Ronnie
Thank for you pull. But wee are not release this project yet and are here for you can see how the work goes
So wee close this but thanks again


========== Сборка: успешно: 4, с ошибками: 0, без изменений: 13, пропущено: 1 ==========

operation system?


You could iniine it in the `g_build_filename` function call I think and then remove the `home_path` variable.


Reopen https://bugs.limesurvey.org/view.php?id=8247


I would recommend splitting the code inside the `if` and `else` into two separate functions.


:church:


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


:+1: :heart:


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/35637/testReport/) for this commit**


What you could do here (but that's up to you) is:
```c
gchar *key;
MergeSourceData *system_key_file;
while (g_hash_table_iter_next (&iter, (gpointer **)&key, (gpointer **)system_key_file)) {
    …
}
```


When i differ recovery config and main config, i just added crypto related stuffs. I am only sure about "CONFIG_DM_CRYPT" , because this is needed to have decryption working. Without this, it doesn't work at all.


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


yes, only that's necessary


Hallo Jugolo
Thank for you pull. But wee are not release this project yet and are here for you can see how the work goes
So wee close this but thanks again


why do we need to have this statement?


I wouldn't say that a reference with a dead link is well defined. Also, it explicitly allows use of point compression.


Okay i will clean things when have sometime.


@cht8687 Thanks!


I'd recommend adding a separate variable for the processed value and freeing the `val` variable explicitly here. This is very uncommon and rather confusing to pass the ownership of a string to a function.


fabs


These are not the correct changes to the file
Line 367 should have the -i2 remove #this is the one that is clobbered with a mono input device
line 378 can remain as it was #still works with "-i", "2" present


what is this?


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


if not hmac.compare_digest(str(token),str(computed_token)):if not hmac.compare_digest(str(token),str(computed_token)):if not hmac.compare_digest(str(token),str(computed_token)):if not hmac.compare_digest(str(token),str(computed_token)):


or this?


uh


Maybe `g_assert(val != NULL)` here?


This is lcfi hardcoded atm


Put the `j` and `key_length` variables inside the `for` loop.


Ah, woops.


Does it work?


"security considerations" is twice, also a typo: s/acrtual/actual/


@timblair, I think he meant you.


it depends what this directive does with valid-json attribute, it could do simple $parse, but maybe it doesn't. Anyways, I wouldn't get too perfectionist about this one, with amount of tech debt we have still hanging around in front code. :-) $scope will do fine.


![](https://github.trello.services/images/mini-trello-icon.png) [Partie 7.b : Classement agents selon nombre_reperages effectués](https://trello.com/c/kTeE5oag/20-partie-7-b-classement-agents-selon-nombre-reperages-effectues)


Ah, thanks for spotting that!


Optimizing png files will reduce the image size but I'm not sure about LCD/VFD pngs, Maybe we should not optimize these kind of pngs (Please let me know)
Also we need to conisder two things about HD skins :
1- Do we want less image size ?
2- Do we want 32-bit pngs ?
The answer to the first question is optimizing
About other things like plugins ... we can optimize without consider anything


This causes Kodi to crash when hostManager.getHostInfo() is null.
It can be null as finish(), which is called earlier if hostManager.getHostInfo() is null, does not immediately destroys the activity.


Latlong BAOhex20DAI GOOGL  4e 61 76 69 67 61 74 69 6f 6e


doesn't that mean we will add not 10 but 20 key exchange methods?


Basics learning schema. keep on it until you feel confident starting this.


delete comparator;


I think this is a better way to handle the resource classes to CDI beans instantiation. What you think is broken?


elses not needed


I've reverted this.


d


RFC 4462 explicitly says `K = f^x mod p`.

Getting ECC right is hard enough without digging through two levels of redirections and dead links.


maybe exit(); instead of return NULL;
This way you won't get NullPtrEx.


shouldnt it be called only for not normal scoped beans? (we can know it in the constructor using beanManager.isNormalScope(bean.getScope()) to avoid to request it each time there. ALso for normal scoped bean we can cache the instance and makes the getInstance/release very fast: https://github.com/apache/tomee/blob/master/server/openejb-cxf-rs/src/main/java/org/apache/openejb/server/cxf/rs/CdiResourceProvider.java#L263 vs https://github.com/apache/tomee/blob/master/server/openejb-cxf-rs/src/main/java/org/apache/openejb/server/cxf/rs/CdiResourceProvider.java#L281 typically


Echo loc updated


looks fine


Looking good (I'm testing but I also mean it)


Well that's fine that you've reverted it, but I would observe that we *do* document escape sequences for Strings, and I don't really see how that's different to this.


You could also use the tempfile module to return a random filename for this: https://docs.python.org/3.5/library/tempfile.html
Will this file be removed after it gets uncompressed?

Another option would be to create the `Exercise` object in the view, and use the exercise id to save the filename. You could then pass the ex.id into the worker process and get the file path from that.


Needed. It points out that there is a misspelled comparator name in config file


#3 fixed


Needs a `.`


And no pre-commit hook... again. Sorry.


![](https://github.trello.services/images/mini-trello-icon.png) [Partie 7.c : Historique combat entre deux dates + liste participants, liste gagnants, liste perdants](https://trello.com/c/cuMk0Gzq/21-partie-7-c-historique-combat-entre-deux-dates-liste-participants-liste-gagnants-liste-perdants)


Нет, ты не понял, вообще if и end не нужны.  Само выражение `year % 4 == 0 && ( year % 400 == 0 || year % 100 != 0 )` и так вернет true или false, что и будет результатом работы метода


Graph created with yEd (https://www.yworks.com/products/yed) with Graffoo plugin (http://www.essepuntato.it/graffoo/).


It's here:
https://github.com/aduprat/james-project/commit/20e5a7782283d5017c636a82ae4fe528b8d87d65
You need the 4 following commits :
- Remove Generics on Cucumber stuffs 63dcb04
- Introduce JmapJamesServer interface 20e5a77
- Introduce GuiceJamesServer interface bac3c8b
- Rename GuiceJamesServer to GuiceJamesServerImpl 2de122a


Hallo Jugolo
Thank for you pull. But wee are not release this project yet and are here for you can see how the work goes
So wee close this but thanks again


Can you show me where this screws up the layout? It looked good on my side. I tested several screens.


:laughing: Do you clone a fresh copy every time?


In the Yesod examples, instead of breaking the loop in `makeFoundation` with a temp foundation with an undefined pool, you can use [MonadFix](https://wiki.haskell.org/MonadFix) (and its mdo notation for instance).


GREAT Little script. It is always cumbersome to make bootable items on the mac, and this made it easy. I made a bootable SD card with it. Thanks.


算法


Испольузй постфиксную форму if, если в теле одна строка


И зачем тут else?


k не нужно считать, т.к. к элементам массива можно обрщаться с отрицательной индексацией, а она не меняется при увеличении кол-ва элементов в массиве, как -1 и -2 обозначают последний и предпоследний элементы, так и будут обозначать, вне зависимости от того, сколько там элементов


Используй постфиксынй if


Зачем тут else?


Can you provide more information about the configuration of your build?


NogoodHexF4 0 or 1


Чтобы не считать index, используй each_with_index вместо each


This pull request is ready to be merged


Дублирование кода и дублирование данных. Массивы отличаются всего 1-м элементом. Так зачем нам 2 массива, когда мы можем определить, високосный ли год и если да, поменять всего 1 элемент в нем? Алгоритм останется тогда неизменным


Пустый хеши принято задавать так: `shopping = {}`


Что тут с отступом?


Comment can be removed


А тут куда отступ подевался? Отступы они же не для красоты, а для того, чтобы было легче читать и понимать структуру кода


Do we need the new variable `$otherSubmissionFiles` ?


PS @rxin did you update Jekyll? this is still going back and forth on small rendering details. Just wondering if it's a Jekyll issue or not.


Empty lines here not necessary?


Давай более толковые имена переменным. Вот что такое a и b? Все что угодно, а если их назвать, скажем product и params, то уже ясно становится, что там такое.


вывести хеш можно прозще: либо `puts alphabet`, либо `p alphabet`


This is somewhat not needed.
You could set allocator and free for json on janus startup:
json_set_alloc_funcs(g_malloc,g_free);


Are all these variables necessary? -- This we will probably know, when other parts of versioning come, right? That's why it is also easier to review it at once...


@payes ,
Will it be good idea to put iscsi specs as code level comments to places you are coding/fixing ?

This will help others to get a hand on iscsi & start contributing as well.


Наконец-то рекурсия! Будет еще круче, если if заменить тренарным оператором.


Take a look at this... ;-)

https://github.com/MRPT/mrpt/blob/7f4ee1117732015cc15ab10f3ad5533b0b6ca5ea/libs/nav/src/tpspace/CPTG_Holo_Blend_expr.cpp#L17
https://github.com/MRPT/mrpt/blob/7f4ee1117732015cc15ab10f3ad5533b0b6ca5ea/libs/nav/src/tpspace/CPTG_Holo_Blend_expr.cpp#L57
etc...

I'll test a bit more and merge soon.


path and file name, as well as copyright information missing


Hallo Jugolo
Thank for you pull. But wee are not release this project yet and are here for you can see how the work goes
So wee close this but thanks again


используй << вместо push для добавления в массив.


Also make internationalization for page 403.jsp


Тут лучше использовать цикл с услоивем


Fix for jcassee/angular-hypermedia#42


to_s не нужен, у тебя и так ключи строками заданы


We manage the Yesod templates on the
https://github.com/yesodweb/yesod-scaffold repo, do you want to send a PR
there?

On Wed, Nov 16, 2016 at 1:30 PM, Paul Brauner <notifications@github.com>
wrote:

> In the Yesod examples, instead of breaking the loop in makeFoundation
> with a temp foundation with an undefined pool, you can use MonadFix
> <https://wiki.haskell.org/MonadFix> (and its mdo notation for instance).
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/commercialhaskell/stack-templates/commit/2ac15efb62835ab2377b97348780ee46fc909de8#commitcomment-19843641>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AADBB4L3t7LRkwo2qfsll3wijmYDv1nSks5q-ulmgaJpZM4KzvMr>
> .
>


Oops! Sorry about that...


Most of the time, when I'm at work. 😬


the tabs could be corrected, for better reading, here and below...


Зачем тут хеш? Достаточно массива с кол-вом дней


так у тебя по-умолчанию там 28, просто ничего не делай в этом случае


For this work, do you think it is a good idea add an overload to allow specifying not just the number of bytes but the offset? This would help remove an additional buffer copy if you want to send segments of the payload.


masterで作業してmasterにプッシュしたらあかんで
それしたらブランチの意味ないやん


若干の修正ってコミットメッセージも気をつけや！
なにをしたのか内容をきちんと加工


данные лучше преобразовывать в момент получения от пользователя


For the segment work, do you think it is a good idea add an overload to allow specifying not just the number of bytes but the offset? This would help remove an additional buffer copy if you want to send segments of the payload.


Added background to forest also


path and name wrong


Very interesting


Can't we use the `jsonFormat2` helper here? We can use the `FiniteDuration` converter from [apso](https://github.com/ShiftForward/apso/blob/master/apso/src/main/scala/eu/shiftforward/apso/json/ExtraJsonProtocol.scala#L24).


Constructors changed to __contstructor... -- see the most recent version...


И опять не нужный цикл for. Все решается гораздо проще и легче, если использовать each.


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


constructor...


Теперь хорошо бы из репозитория удалить эти `.idea` файлы.


I'm replacing these four lines with a simple `continue;` with BagelOrb's permission.


Ok thanks. I'll merge it in.


you could explain the purpose of "DEPLOY_BRANCH" too


Hello,
thank you for your correction for Debian.
I've also found out that if an interface is inactive or not available, a click on change of IP settings will return error. There variable PageNetworkIpSettings.settings is null.
I tried to shape it in following rows:
2368: PageNetworkIpSettings.model = self.model;
2887: if (PageNetworkIpSettings.settings || PageNetworkIpSettings.connection){	this.settings = PageNetworkIpSettings.settings || PageNetworkIpSettings.connection.copy_settings();}
2901: if (self.settings){var params = self.settings[topic];	} else var params = PageNetworkIpSettings.params;


Хорошо, а еще лучше будет, если это в класс обернуть. Тогда метод start будет публичным, а все остальные - приватными. И общие данные можно хранить в инстанс-переменных


Yep.


Обычно используют именование в нижнем регистре (`snakecase`): `first_station`.


Поскольку мы уже в контексте метода, я бы поменял название параметра на `types`.


@mvollmer ^^


how can i add support of v2 in my bot in nodejs


Yeah :/, I forgot we're already depending on apso.


Run `cookiecutter --no-input https://github.com/weaveworks-plugins/scope-plugin-generator` to generate **a** plugin with the default values. After the generation process is finished, the directory tree will look as follows:


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1622](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18165&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


maybe to put some contexts into the key name, where the key is used, e.g. manager.versioning or manager.setup.versioning or x?


will build the docker image for the plugin.


При вызове это будет выглядеть так: `station.show_trains(true)`. Не совсем ясно что это за `true`. Будет здорово использовать [именованные аргументы](https://robots.thoughtbot.com/ruby-2-keyword-arguments): `station.show_trains(by_types: true)`.


It is only me there are allowed to close a pull request!


hello


@lluixhi if you think anything is ready for the main tree, open a bug on bugs.gentoo.org and i'll commit to the main tree.  the idea is to try to clear the overlay completely over time.


а кто будет удалять это всё? тут стоит использовать умные указатели


See #1131.


wrong path and file name


может что бы не писать лок везде вынести логирование в отдельную функцию?


you can use heroNames in dotaconstants to get heroes keyed by name


adding change on purpose


LGTM 👍


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 95](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18167&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) is now running


Здесь вместо флага лучше использовать непосредственно символ с типом поезда. Потому что если добавится 3-й тип будет флаг трудно поддерживать.


I think it's probably better to make a new property called player_slot rather than reassigning slot


tabs i.e. indentation...


better name might be getShortHeroName


Like


Если это проверка на то что `train_route` не `nil`, то в случае когда это `nil` следующая строка выдаст ошибку.


TeamCity OTA+ :: rvi_sota_server :: migrations container [Build 95](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18167&buildTypeId=ota_plus_RviSotaServer_MigrationsContainer) outcome was **SUCCESS**
Summary: Running Build time: 00:01:21


What if config setting is Off and journal setting is On? Do both have to have the same?


This is what I get btw trying to use `:tag my::tag::here`:
```
 1 *** This is the debug buffer, where debug info will be written ***
 2 shell stderr: <<<
 3 awk: syntax error at source line 10
 4  context is
 5     D { print length(out) >>>  == <<<  0 ? "echo -color Error no such tag " ENVIRON["tagname"] : "menu -markup -auto-single " out }
 6 awk: illegal statement at source line 10
 7 /bin/sh: line 7: printf: write error: Broken pipe
 8 /bin/sh: line 7: printf: write error: Broken pipe
 9 >>>
10
```


Мы уже в контексте класса поезда, поэтому лучше назвать параметр просто `route`.


Что такое `default`?


<!DOCTYPE html>

  <meta charset="utf-8">

  <title>Basic Form</title>
  <meta name="description" content="A basic form">

  <meta name="viewport" content="width=device-width, initial-scale=1">

  <link rel="stylesheet" href="main.css">
  <link href='http://fonts.googleapis.com/css?family=Courgette|Roboto:400,300' rel='stylesheet' type='text/css'>

Udacity
          Down Under

          Flights
          Hotels
          Rental Cars

        <form>

              <input type="radio" checked name="tType" value="0"> One Way

              <input type="radio" name="tType" value="1"> Round Trip

              <input type="radio" name="tType" value="2"> Multi City

Where ya headed, mate?

              Full Name
              <input type="text" name="fName" id="fName">

              Enter your address
              <textarea id="address" name="addressTxt"></textarea>

                Departure
                <input type="text" list="depCities" name="depCity" id="depCity">
                <datalist id="depCities">
                  <option value="Sydney">
                  <option value="Melbourne">
                  <option value="Brisbane">
                  <option value="Perth">
                  <option value="Adelaide">
                </datalist>

                Arrival
                <input type="text" list="arrCities" name="arrCity" id="arrCity">
                <datalist id="arrCities">
                  <option value="Sydney">
                  <option value="Melbourne">
                  <option value="Brisbane">
                  <option value="Perth">
                  <option value="Adelaide">
                </datalist>

                Depart Date
                <input type="date" name="dDate" id="dDate">

                Return Date
                <input type="date" name="rDate" id="rDate">

                Preferred seat type
                <select name="seatType" id="seatType">
                  <option value="0">No preference</option>
                  <option value="1">Aisle seat</option>
                  <option value="2">Window seat</option>
                </select>

                  <input type="checkbox" checked name="jLetter"> Receive promotional offers?

                <button type="submit" id="submitBtn">Search</button>

        </form>

          About us

          Join our newsletter

          Need Support?

          © 2015 Udacity


Empty space at the end of the line?


empty space


Focus form submitted


agora funciona a persistência na tabela intermediária :+1:


Why was this reverted?


стиль именования сделайте одинаковым


Используй `snake case`.


This line has some flaws wth ";" and """ positioning.


how can i send you a message?


do we need this variable i.e. could we then only check `$otherRevisions`?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1622](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18165&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 261, ignored: 4 Build time: 00:11:42
### Failed tests
```
org.genivi.sota.resolver.test.PackagesResourcePropSpec.COUNT devices with installed package version group by package name: <no details avaliable>

```


aktiviraj premissionse v settingsih!


@quozl oops, guess who forgot to finish the PR?

https://github.com/sugarlabs/terminal-activity/pull/10


BTW: it currently assumes that there is only one ordering


the translate on the line above could be indented...


@p0psicles is this supposed to be here?


вот такая ошибка вылезла после установке "Parse error: syntax error, unexpected '}', expecting variable (T_VARIABLE) or ' in /mnt/iscsi/www/oleg3k/data/www/oleg33.myarena.ru/7/includes/sb-callback.php on line 2955"


Имеет ли смысл проверять правую границу, ведь станция в `stations_list` будет всегда иметь индекс меньший либо равный величине `stations_list.size - 1`.


Это автоматически из шарпа привычка, я забываю периодически о том что нужно юзать snake case


I see it is removed in the future commit...


Было бы удобней: `train.route = train_route`.


I see this was corrected in the future commit...


Can you please refactor the code as per what you have described and write the unit tests for the same.


Как станция узнает что с неё отправился поезд?


А если передадут станцию которой нет в списке?


It was creating exceptions all the time when there was no activity on sockets. A connections can in theory stay infective for several hours


Thanks for spotting this. Should be fixed in 92773ea57c8eb76315f850aa2c36fb1c6e97363a


Now I understand.
And yes, this question for native English users.
As for me, if I have possibility to exactly determine string and translate it (and users can see right label), thats enough for me.


Anche google mette i parametri nell'url https://www.google.com/search?q=find%20this


Ok


Interface methods may only be public of abstract


runningProccesses maybe? They only exist while running


It should be `float tmpDamage = float(int32(pdamage) + DoneTotal);`


Java naming convention dictates that interface identifiers should not be prefixed with an I.


protected should work


You want to disguise them as PGP messages?
But you still have the rest of the header giving away that it is not a PGP message...


Perche' !important?


Упс, да, мой косяк. Сейчас исправлю.


Use the most generic class/interface as the static type of your members and local variables.
In this case, the left-hand-side type (the static type) should be `Map` for more flexibility in your code.

We will cover this practice and the "why" behind it this weekend.


Tutto questo pezzo di codice e' illeggibile, dividi in funzioni


@godiard, thanks, no worries, I'll do some heavy testing on multiple platforms.  Also found a few other things to fix and improved the test harness.


Non c'e' un modo piu' elegante di farlo?


This change makes the parse_addr_args of the same file to return the arguments in the wrong format.


Always enclose single-statement branches of `if` structures in curly braces to avoid confusion and erroneous execution flow.


Можно так: `print "#{m} "`.

Либо использовать итератор `inject`:

```ruby
result = months.inject([]) do |titles, (month, days)|
  titles.push(month) if days == 30
  titles
end

p result.join(' ')
```


Попробуй найти способ сделать это с помощью метода итератора.


Oh, nice. Good find.


I messaggi dei commit devono essere molto piu' corti, metti il resto nella descrizione.


Можно обращаться к элементам массива с конца: `array[-1]`, `array[-2]`. Это позволит не поддерживать дополнительную локальную переменную.


Можно снизить количество итераций если итерировать по гласным вместо алфавита.


Cerca di configurare il tuo editor per usare gli spazi invece dei tab


I think Dropbox is a bad example here, because the software is directly related to directory and file management. Users know what it does and what they are signing up for - they are likely going to manage the directory themselves. On the other hand we are more likely to discourage messing around with the data dir unless you know what you're doing - what with cryptic names and database files/backups. Most users ideally should not have to worry about it. Claiming "best available" directory avoids having them to think about this.

Either way, fail and prompt is still better than moving existing files.


Ah, I see, thanks.  I've been fixing some problems in the same code in Maze and Chat, so I'll keep my eyes open for any Terminal change.


Sure Amit, I will keep that in mind from next checkin and will update
previous checkins.

Regards,
Payes

On Wed, Nov 16, 2016 at 5:10 PM, Amit Kumar Das <notifications@github.com>
wrote:

> @payes <https://github.com/payes> ,
> Will it be good idea to put iscsi specs as code level comments to places
> you are coding/fixing ?
>
> This will help others to get a hand on iscsi & start contributing as well.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/openebs/gotgt/commit/46197f33ce82319275b45152c0b6dff775af2e67#commitcomment-19843757>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AJ4EiiDgDrnf-Z-9D37ap7cfKX6CekTFks5q-uuSgaJpZM4KzvzM>
> .
>


@Scouttp ?


Perche' decodifichi gli input qua?


Что будет если пользователь введёт товар который уже есть но с другим количеством и (или) ценой?


註解 on Website 20161116 20:26


Missing period on end


Need space after if


Missing space after if


I would but I need my employer's approval so I thought that for a simple change like this I would let you know instead of going through the approval process. Let me know if you'd rather have me do it still, it's kind of a lightweight process anyway.


Missing space after if and for in this function


The title of this commit is wrong.
It it supposed to be "jobs_generate: support float number growth rate for SchemeGeometricSets"


A gcc6 -al a qt5-buildfix.patch dobható lesz szerintem.


Can we move run down? Seems like a lower-level method than main.


Nei commenti cerca di spiegare cosa fa il codice, non riprodurre il codice in linguaggio naturale.
Ad esempio, per `(strlen($object) > 0)`:

`Checks if the value of hints retrieved is less than the value defined in the constant MAX_HINTS`

non aggiunge nessun'informazione. Sarebbe piu' chiaro (se ho capito bene):

`Don't send more hints than MAX_HINTS`


Sbaglio o non la stai usando?


@sakawatari インデントが深すぎます。
https://shugo.net/ruby-codeconv/codeconv.html


Perche' non usi `$key`?


@sakawatari 空行は不要です。
https://shugo.net/ruby-codeconv/codeconv.html


I never say no to having more contributors on board :)

On Wed, Nov 16, 2016 at 2:27 PM, Paul Brauner <notifications@github.com>
wrote:

> I would but I need my employer's approval so I thought that for a simple
> change like this I would let you know instead of going through the approval
> process. Let me know if you'd rather have me do it still, it's kind of a
> lightweight process anyway.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/commercialhaskell/stack-templates/commit/2ac15efb62835ab2377b97348780ee46fc909de8#commitcomment-19844299>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AADBB47QOUD7ecSsAw38qlDv8e3ePm7Rks5q-vangaJpZM4KzvMr>
> .
>


I like having a means to halt testing if an exception wasn't thrown.
There are the ```US_TEST_CONDITION``` and ```US_TEST_CONDITION_REQUIRED``` macros which will either log a failed test and move on or return without executing any further tests, respectively. I would suggest renaming the macro in a similar fashion to stay consistent; i.e. ```US_TEST_NO_EXCEPTION_REQUIRED```


Named function expression can break (specifically they cause grief in FireFox).


@sakawatari add-postとなっていますが、コミット単位はもっと細かくしましょう！
イメージは、「ToDoリストのToDo一つ分」です。


Sarebbe molto piu' comprensibile se usassi [array_push](http://www.w3schools.com/php/func_array_push.asp) invece di `$data[$hintIndex]` e [count](http://www.w3schools.com/php/func_array_count.asp) invece di `$hintIndex`


Again, named function expression. Might be wise to skip `doneLoad` name here.


Sorry, the original repo didn't use this style, and my formatter applied it automatically, and I committed without noticing it, so I removed it later with this commit.


Non chiudere il php per i file php-only


@sakawatari add-cssとなっていますが、作業内容とあっていませんでした。
コミットメッセージはチーム開発時にとても重要なお作法です。
以後、十分に気をつけるようにしてください。


Ok :) I'll send a PR when I figure out the process.


Come per il js: non c'e' un modo piu' elegante?


Please create your own branch for tests as you conflict with my tests


@sakawatari Gemfileに空行は不要です。


Non usare numeri per definire degli stati... al massimo definisci delle costanti e usa le costanti


@sakawatari メソッド内に空行は不要です。
https://shugo.net/ruby-codeconv/codeconv.html


Non chiudere php


Да, но смысл от этого не меняется. Лучше продумать архитектуру метода так чтобы условия не повторялись (даже в альтернативной ветви).


@sakawatari コミット単位がざっくりすぎて、作業内容がわかりません。もっと細かくこまめにコミットしましょう！


@sakawatari Gemfile内は空行不要です。


TeamCity ReOpenLDAP :: master [Build 68](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=2941&buildTypeId=ReOpenLDAP_Master&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 02:04:03


TeamCity ReOpenLDAP :: Cake4Vadim (ps-stable, RHEL6/7) [Build 5](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=2942&buildTypeId=ReOpenLDAP_Public_Cake4VadimPsStableRhel67&guest=1) is now running


qweqwe


Changed the font size


So it's unusual case when item damage of ItemBlock is used not as block metadata - GT developer words


asd


The instance is effectively cached, complying with CXF `ResourceProvider` lifecycle (see please `isSingleton` property here, probably should be chanced to use `isNormalScope` there). So for non-request scoped beans, we will return the cached instance all the time. Does it make sense, @rmannibucau ?


provides commands to build, run, and clean.


I think this catch block can be simplified down to one call to ```US_TEST_FAILED_MSG```


@sakawatari コミットメッセージと作業内容が？なところがあります。もっと単位を細かくしてみてください。


two, **lowercase**.


@sakawatari 良いですね！


Hope this can get merged. Thanks for this fix.


@sakawatari これは・・・なんの作業ですか？w


Eri Nye the science guy!


@sakawatari 「add-heroku」ではなく、メーラーの設定ですよね？


这是我第一次在github上创建仓库


Please review the code.


`player.hero_id` but there needs hero by id, not name


nit-pick on style: stay consistent with the codebase. the ```&``` should be adjacent to the type


I have changed the pattern to 64 bit for the instruction in file lj_target_s390x.h.. let me know if that show you were expecting it


@sakawatari 無用な空行と深すぎるインデント、コミットメッセージを修正すると、もっと良くなると思います！
https://shugo.net/ruby-codeconv/codeconv.html


wow, very cool func men


Oh I see.   In that case this is fine but you need to handle undefined hero (can happen if when a new hero ID shows up and we haven't updated constants yet)


Added WINVER=0x0A00 for windows10 builds (win7 sdk)
Maybe i need to downgrade to windows8 later ...


salut


On VC14 we can only use wxwidgets 3.1


Could you elaborate a bit on the issue that is addressed here? I'm assuming this is the fix to a `RegExp DoS issue` that [npm warns about](https://github.com/npm/npm/issues/13323).

It says reported by @nstarke, but I could not find a GitHub issue, and I would like to understand how serious this is before going on a PR spree of packages that use an old minimatch dependency.


This can be `edebug("%s %j", message, error)`


There seems to be a problem with this change.
When running the script:

**sudo ./build/MoonGen examples/netronome-packetgen/packetgen.lua -tx 0**

I get the following error:

**[FATAL] Lua error in task master**
examples/netronome-packetgen/packetgen.lua:272: attempt to call field 'startTask' (a nil value)
Stack Traceback
===============
(2) global C function 'xpcall'
(3) Lua upvalue 'master' at file '/home/user/MoonGen/build/../libmoon/lua/main.lua:96'
	Local variables:
	 _ = string: "./build/MoonGen"
	 file = string: "examples/netronome-packetgen/packetgen.lua"
	 args = table: 0x40acc378  {1:-tx, 2:0}
	 cfgFile = nil
	 ok = boolean: true
	 parsedArgs = table: 0x4097c6b8  {}
(4) Lua function 'main' at file '/home/user/MoonGen/build/../libmoon/lua/main.lua:141' (best guess)
	Local variables:
	 task = string: "master"
	**examples/netronome-packetgen/packetgen.lua:272: attempt to call field 'startTask' (a nil value)**


> we are more likely to discourage messing around with the data dir unless you know what you're doing - what with cryptic names and database files/backups.

That's just for the moment, though. The goal here is for this to eventually be something users can access directly, with human-friendly folder/file names for attachments and the database files off in a subdirectory.


Do these function calls need also [a bit more software development attention](https://cwe.mitre.org/data/definitions/252.html "More checking of return values")?


As of this commit:
- The player can choose their playing piece
- The player can play the game with an agent
- The agent's strategy is randomly selecting from a set of valid moves


deliverable versione 1 bozza


We can have a single message and get rid of most of the code


A warm thank you for this ;-)


Use the ```EXE_EXT``` macro in ```TestingConfig.h``` and the ```#ifdef``` becomes irrelevant.


Coursera Project Practical Machine Learning

- project_report.Rmd: R markdown document for course project report.

- project_report.html: Compiled html file for course project report.

- data: Data folder containing csv files for training and testing sets.


We can use ` registerError('Missing resource provider token')` if we remove debug message


looks good except query key is changed. Is it intentional? If so, ok to merge.


I was going to make a fork myself to fix this, good job!


ProjetGroup
 a mettre dans le body ^^


Remove this


Added graphic display.


@JulianIso you can find my email on donation (support project) button.


Enélkül egyes ati kártyák használhatatlanik, az isó nem indul el normális grafikus módban.
Ez fontosabb, mint például a tseng, vagy s3, melyek már kőkoriak.


Thanks!
Query key is changed intentionally.


Yes, sure. Add an entry to the docs.


Needs to be correctly resizable. @Camphul


Woo, slack!


Hm, OK, i haven't tested it with a completely quiet server. I've tried something much more simple in #364. Lets see how the tests work out.


nit-pick: I'd suggest more descriptive function names other than ```testrc1```, ```testrc1``` ... ```testrcX```. Something that accurately and succinctly describes the test would help others navigate this code quickly in the future.


No major changes just yet.


@xcatliu 👍本来想改了提pr的，但是怕自己想错了，就查了下记录


@psolaimani +1


Whoops, yes. Should be ``-c sh:hardlink``.

Regarding the disk space saving: I think I managed to confuse myself regarding this.
So, yes, you're probably right (although there might be corner cases with ``-k`` where more than one original is possible or even more esoteric stuff like ``-D``). As separate conclusion, the default sortcriteria might better be ``pOma``.

> Hm, the only benefit of H I can think of at the moment (under the assumption that O is already given) is that it might be a little bit faster for -c sh:hardlink if H is given, because there are less new hardlinks that have to be created.

I think this speedup is negligible, if existent. My thought was that it reduces "fragmentation" by linking to the highest
link count. But when assuming no outside links (which is handled by **O**) the link count should be 1 indeed.
It might be still useful for rather esoteric usecases where the user really wants to link to the highest/lowest link count
for some semantic reason we don't know of.

Thanks for the corrections. Hopefully fixed in f7fa570.


Sorry for posting this here, but here is where I found out oneOf is missing :)

Anyway my use case for this function is that I get two maybes, and I want to check if one of them is Just x.

Basically OR logic for two maybes. I will copy the function and use it until I figure out way to not use it :D


Maybe code changed but in previous release org.apache.cxf.service.invoker.AbstractInvoker#getServiceObject was called per invocation and therefore the instance was not cached for all requests. Otherwise yes, isSingleton() should test if it is a normalscoped instance and in this case the context.release() should be skipped too.


@kurkale6ka create an issue please.


@wjwithagen i am not sure if we need this comma.


This patch only addresses the result on `json_dumps`, which is something Jansson doesn't care about I guess: it should not be part of the internal reference mechanism (`json_unref`) and so neither of its memory management either, or am I missing something? As far as I've understood, `json_set_alloc_funcs` only specifies how `json_t` objects are allocated and freed, but I may be wrong.


This is the changelog for JdeRobot 5.4.2, not JdeRobot 5.4.1


That will actually conflict with existing html br tags, for example:

```
<p>Text line 1<br>
Text line 2</p>
```

will have two line breaks; one for br and one more for \n


@bingmann here. feel free to cherry-pick.


revision="n"


Whether the directory is human-friendly or not, I think Zotero file management should be transparent until customization/troubleshooting is required. The interface for Dropbox file management is the file manager, but the interface for (transparent) Zotero file management should pretty much always be Zotero.

Probably something that won't come up too much though, so maybe not worth spending too much energy on arguing. Just putting in my opinion, but any decision does the job.


The name is a bit mouthful. In the software model we agreed to name traits of an object with the `Has` convention. Here it would give `HasAttributes<SELF>`. WDYT?


Even if browsers are permissives, & should be replaced by &


Same here


ha futtatod, mitol kap erteket a ```name```?


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1623](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18182&buildTypeId=ota_plus_RviSotaServer_Ci) is now running


public static void main(String[] args){

String[] - pachi space ani args hoina ra ??


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/35666/testReport/) for this commit**


@rmannibucau thanks, much appreciated, I will make sure this is the case.


@cpojer It should be `watch: !args.nonPersistent,`


That means that the thread numbering starts at `2`, right?


It should be `options.watch = !options.nonPersistent`


Can new threads ever be daemons by default?


Merged. We also see
```
.../repos/base-nova/src/test/platform/main.cc: In member function ‘virtual void Greedy::entry()’:
.../repos/base-nova/src/test/platform/main.cc:603:54: error: no matching function for call to ‘Genode::Parent::upgrade(Genode::Pd_session_capability, const char* const&)’
      _env.parent().upgrade(_env.pd_session_cap(), buf);
                                                      ^
.../repos/base-nova/src/test/platform/main.cc:603:54: note: candidate is:
In file included from .../repos/base/include/base/env.h:17:0,
                 from .../repos/base/include/base/connection.h:17,
                 from .../repos/base/include/rm_session/connection.h:18,
                 from .../repos/base-nova/src/test/platform/main.cc:22:
.../repos/base/include/parent/parent.h:204:26: note: virtual Genode::Parent::Upgrade_result Genode::Parent::upgrade(Genode::Parent::Client::Id, const Upgrade_args&)
   virtual Upgrade_result upgrade(Client::Id to_session,
                          ^
.../repos/base/include/parent/parent.h:204:26: note:   no known conversion for argument 1 from ‘Genode::Pd_session_capability {aka Genode::Capability<Genode::Pd_session>}’ to ‘Genode::Parent::Client::Id {aka Genode::Id_space<Genode::Parent::Client>::Id}’
```


Can a new thread ever have a different priority be default? Is that inherited from the parent thread or thread group?


COol


You may not care about performance, but this whole way of doing business is way more work than necessary.  It doesn't make sense to fetch a packed point and then unpack it.  There are functions in PDAL that will fetch data in any way that you like.  You just need to access them from the JNI interface (I think).


//not my opinion


在谷歌浏览器(版本 54.0.2840.71 )，火狐（版本49.0.2），虚拟机中的360安全浏览器（版本8.1.1）中兼容模式IE7+中，极速模式（内核45.0.2454.101）测试是能获取到值的
但是，之前在招商平台中这样写，记得是取不到值的。
```
<!--html-->
<div class="add">
    <select class="ac" name="secondCat">
        <option value="">--请选择--</option>
        <option value="1">aa</option>
        <option value="2">bb</option>
        <option value="3">cc</option>
    </select>
  </div>
```

```
//E.G.1.
 $(document).click(function(){
   console.log($('.add select').val());
 });
```

```
//E.G.2.
$('select').change(function(){
  console.log($(this).val());
});
```
测试地址【需要配合看控制台中的信息】：http://output.jsbin.com/mawotup


also fixed mantis:
0019706: Failed test: Felder als Pflichtfeld oder einzigartig bestimmen


Looks good and it should not have the issues with quiet connections as before...


TeamCity OTA+ :: rvi_sota_server :: ci [Build 1623](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=18182&buildTypeId=ota_plus_RviSotaServer_Ci) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 262, ignored: 4 Build time: 00:08:13
### Failed tests
```
org.genivi.sota.resolver.test.PackagesResourcePropSpec.COUNT devices with installed package version group by package name: <no details avaliable>

```


I think my last comment can be achieved using `loadPragmas();` on this line.


Sounds good. I renamed it.

The idea behind this interface is to have a similar user facing API (=DSL) in all places where attributes can be configured. On top of AttributeContainer.

If we go with this, we can maybe also reuse the implementation. Currently I copied it to all classes implementing the interface. We have to see where this interface ends up to be implemented in the end.


Hey, I noticed it after merging, you shouldn't delete "index.html" ".fav" from .gitignore.
I will correct it with another commit in the main branch :+1:
I should be more careful I suppose.


Am I replace "\n" with "<br>" sign in checkout_process.php?


:D


Also, I'm not sure what the ByteOrder.LITTLE_ENDIAN thing is about, but Java VM is big endian and PDAL data is stored in the byte order of the host machine, which may not be little endian.


Hi!
Might I ask you what this <icon> tag is intended and why you have removed it?
I can't find much info on phonegap docs.
Thank you


The files were formatted with `clang-format-3.4`. Since you have to reformat them, I suppose that `.clang-format` file is not sufficient to unambiguously format the files. Do other `clang-format` versions also format the files differently?

I think we still have to restrict the supported versions, if they do not guarantee to produce the same formatting. It is probably better to drop support for old versions, and instead use the new one(s). @grimmerm what do you think?


Honestly I think it's fine to just merge these to master, if it compiles. We can update later if needed.


@dawidmalina @pskumar448 we're currently making an effort to address the issues we have with the new Jenkins parent pom and our CI server so that we can get a new release out as soon as possible.


This is sketchy.


tr


See #3


Thanks, it fixes the following error at Magento version 2.1.2:

Sebwite\Sidebar\Block\Sidebar
Incorrect dependency in class Sebwite\Sidebar\Block\Sidebar in /vendor/sebwite/magento2-category-sidebar/Block/Sidebar.php
\Magento\Framework\App\Config\ScopeConfigInterface already exists in context object

But, this commit is not reflected in the module, if we install via composer.


given the small numbers of rows in a trigger result-set, filtering upon receive is likely to be more efficient than giving the query planner more work to do


Yup, I've been working on that. Will continue somewhere today...


@abellgithub Do you think this overload is useless (as functions below provide direct access to necessary dims through JNI)? I thought that would be not bad to have an `Array[Byte]` parsing function (as we provide `getPacked(dims): Array[Byte]` function).


@abellgithub yes, that's why there is an overload to pick necessary endian in all functions.


Yes, I saw the copy too. It's unfortunate Java doesn't have traits like Groovy, it would have been handy here.


Pretty cool commit my friend.


It's not about file management (and people indeed should never manage Zotero files via the filesystem) — it's that people have always wanted to be able to browse files in the storage directory. It'll be tricky, but the goal is to eventually make that possible, which is part of the reason for ~/Zotero.


This error indicates that you are using an old version of moongen.
The script works on the current master branch.


Can you fixthe link pls


Er, indeed. Bad rebase, thanks for the spot @lembacon.


Ups you're right thank you!


Reverted in cb504bc93e5502d742ec4a9cda899906b1ac788f.


I don't know.  It's your code.  The array of points thing had a specific purpose (Python Numpy arrays, I think).  If you aren't going to use your data as an array, then it sure seems easier to me to provide direct access to fields in points.  Way less thinking and decoding necessary.  You don't need to know anything about the point layout in a packed point, don't need to worry about the storage type, size, etc.  You can fetch data as you like from the PDAL functions -- just need to wrap what you want up.  This is why I initially suggested a use case as the start of making an API.


Great comment 👍 :'D


how we use it for security reason


CryptoPP берет на себя ответсвенность, такое странное API. Ниже в коде есть об этом комментарий


enter


@catalinbozianu Looking into this right now... Why does this need to be public? :rel


I read election ^^ not electron


Откуда этот метод взялся? Не могу найти его ни в либе, ни в документации сенчи


wewlad


This looks like a typo.


In communications.bluecrossmn.com/anokacounty/index.html please change the alt text for the image from 'Promo Photo' to 'Anoka County' instead


@pandusonu2 Will be more careful :stuck_out_tongue:


Quite right, thanks.


remove type


@abellgithub O: sry for a possible misunderstanding, but what does it mean _ seems easier to me to provide direct access_? there are already direct access functions, which read only allowed dims O: https://github.com/PDAL/PDAL/blob/7e288432eeba922ffbfd9f5763ad77400b2b09f6/java/core/src/main/scala/io/pdal/PointView.scala#L42-L43


Did you just add the api key...........


Thanks for the update. The libmoon was updated right after my changes were
made. Will update packetgen.lua to accomodate nlatest changes to MoonGen.

On Wed, Nov 16, 2016 at 3:33 PM, Paul Emmerich <notifications@github.com>
wrote:

> This error indicates that you are using an old version of moongen.
> The script works on the current master branch.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/emmericp/MoonGen/commit/4c02d14075ad918c9131659d69a948626953ab53#commitcomment-19845103>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AS__NQ0KLvHsW5Cn5eKJkPLxwLofJtjRks5q-wY3gaJpZM4Kz0aO>
> .
>


Igen, lefordult anélkül.


Is having this in there twice intentional?


You shouldn't need to pick an endian function.  You're always converting from host byte order to big endian (htobe32, for instance).  You don't have to choose at run-time.  In C-land, the host knows its byte order and there are functions to do this, which will optimize away on the right machine.  Seems much cleaner to me to hide all that ugliness in the JNI stubs.  Probably faster as well.


Ezek mind az x.org-server függőségei voltak (a 3-ban még most is azok), az x.org-server depends fájljának tartalma lett bemásolva.
Viszont tényleg nem ártana szortírozni, az elavult drivereket ki kellene dobni, csomagszinten is.


Je pense que cette fonction est générique est ne devrait pas se trouver là au même titre que decrypt.
Le mieux serait encore de le mettre dans core dans un module nommé crypto par exemple.
```python
from botanick.core.crypto import encrypt, decrypt
```


Python 的注释最好采用注释字符串的方式，如下所示：

```python
def func():
    """
         your comment content
    """
```


Idem que pour encrypt


レイアウト用のHTML無いのでここ意味ないですね…


I wouldn't expect it to be much more work if there's already an index on `id` and indeed it's virtually identical.

```
spindle=# explain select id, flags, triggerid from triggers where triggerid='4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f';
                                   QUERY PLAN
---------------------------------------------------------------------------------
 Bitmap Heap Scan on triggers  (cost=4.28..12.74 rows=4 width=36)
   Recheck Cond: (triggerid = '4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f'::uuid)
   ->  Bitmap Index Scan on triggers_triggerid  (cost=0.00..4.28 rows=4 width=0)
         Index Cond: (triggerid = '4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f'::uuid)
(4 rows)

spindle=# explain select id, flags, triggerid from triggers where triggerid='4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f' and id <> triggerid;
                                   QUERY PLAN
---------------------------------------------------------------------------------
 Bitmap Heap Scan on triggers  (cost=4.28..12.75 rows=4 width=36)
   Recheck Cond: (triggerid = '4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f'::uuid)
   Filter: (id <> triggerid)
   ->  Bitmap Index Scan on triggers_triggerid  (cost=0.00..4.28 rows=4 width=0)
         Index Cond: (triggerid = '4e6efed6-5e9d-4b3f-b224-3a8aa2cab25f'::uuid)
(5 rows)
```


まあそうですね...
でもブログページを作る場合、共通になりそうな部分をdefaultにすることになると思うので残しておきましょう！


Update as discussed.


hey,

if you run this code without `--harmony_regexp_lookbehind`, it 100% will show an error, right ?


You have forgotten field 42? No change made for this with new notation!


Collaborator: Review for this commit was created. [Link to review](http://localhost:8080/ui#review:id=1427)


make second sentence on separate line


ok. looks good to me


Third time luck. Still a problem.
The lines
      `jack_connect SuperCollider:in_1 system_capture_1`
      `jack_connect SuperCollider:in_2 system_capture_2`
should be
      `jack_connect Supercollider:in_1 system:capture_1`
      `jack_connect Supercollider:in_2 system:capture_2’
ie with : not underscore in between system and capture


строка 23


private final?


Welcome 程序


there is no need for declaring an empty constructor (there are some exceptions ...)


final


May I ask whether the two changed parameters for GBit Ethernet TX/RX delay are the result of testing or just replacing one default (7/31) with another (0/0)?

Since I can't see improved Ethernet performance with new settings so far.


![](https://github.trello.services/images/mini-trello-icon.png) [DAO, работающее через JDBC](https://trello.com/c/1SrgAII9/5-dao-jdbc)


Fwiw these only work with `DateField` and `DateTimeField` as far as i'm aware


not sure what is your plan for this solveTask method ... but i have gut felling you want to schedule this task? or is something which includes scheduling and then something else?


in the file communications.bluecrossmn.com/chs/index.html line number 60:
Please change alt attribute from
img src="images/CHS_logo_300x60.gif" alt="**Company Logo**" to
img src="images/CHS_logo_300x60.gif" alt="**CHS**"


neat!


in which case, yes — the additional `AND` clause would indeed be cleaner; I'll amend and roll it into `develop`


Wrong bounds [0,5] instead of [0,4]set on [this test](http://tests.stockfishchess.org/tests/view/582c259a0ebc590c800c8c25) but now its better to let it finish.


Sorry I do not understand why you say "dead link" ?


What happens when that link expires? Will it have the same content in term years?


Ok, please propose language, my intent was to heavily lean on the other SSH Key Exchange RFCs so we wouldn't duplicate a lot of language where not necessary, but we can add language as long as it is fully consistent with those other RFCs.


Thanks Ethan. I noticed this yesterday playing with an external synth and thought something was amiss


yes, I do not see any other good approach.


You are absolutely correct, thanks


ok, I'll try to remove the duplicate


test fails since activities are assumed to be equal. but they are not equal. Equality is specified with the underlying job, thus if two activities belong to the same job and are of the same type, they are assumed to be equal (see [here](https://github.com/graphhopper/jsprit/blob/job-activity-refactoring/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/solution/route/activity/JobActivity.java#L70)).
@balage1551 Is this intended? If so, what is the intention?


TeamCity ReOpenLDAP :: Cake4Vadim (ps-stable, RHEL6/7) [Build 5](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=2942&buildTypeId=ReOpenLDAP_Public_Cake4VadimPsStableRhel67&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:20:35


Це все добро в `utils.js`


Oh yeah just realized, I'll recommit the folder.


@ed888 why is it when you pipe code inside controllers/services you line up function one under another
but when it comes to angular module definitions you use different indentation?

what's with overall indentation and code style?
didn't i ask you yesterday to follow my lead?


Any chance an english version is available for the Lux chart of account ?


In file communications.bluecrossmn.com/county_of_st_louis/index.html line number 45
Please change

to


migrated to `travis-test-lanwen`


yes it does.


yes, if the parent thread is a daemon thread.


it can have a different priority if the thread group to which it belongs has a different priority.


will fix it.


Чувак, почитай про особливості multy-line з  `\`\``

І прибери ці всі зайві пробіли, це не по кодестайлу. Те що я колись так рефакторнув, то була помилка, викликана переобтяженням хешів такими конструкціями.


I am learning different data mining algorithms that will help through my data science skills. I will be adding codes on the above to make it a refined and reusable code.


@arkadig what's with indentation again?
each tab is 2 spaces exactly
line up your code so that spacing is equal everywhere


done right !! well done !


Associating with #3 with this commit.


I am getting a domain expiration notice for json-schema.org. Marc, are you
able to renew this?

FINAL NOTICE!  The domain(s) listed below are expiring in 5 days.

If you wish to renew this domain, please log into

your easyDNS members section immediately to remit payment for the renewal
invoice.

If your domain is not renewed within 5 days, it will be de-activated and
service for your domain will cease.  At that point, you will have 40 days
to renew the domain (at which point it will be re-activated).  If it is not
renewed within the 40 days, the domain(s) will no longer belong to you and
will become eligible to be re-registered by another party.

If you have any questions, please email support@easydns.com.

Domain Name, Expiry Date

json-schema.org, 2016-11-20

On Tue, Aug 2, 2016 at 6:50 AM, Ben Hutton <notifications@github.com> wrote:

> Great! I'll reply to github saying this has been done! Thanks!
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/kriszyp/json-schema/commit/1c4923479501cdf78c6dab4e9af4bb40dc7992f0#commitcomment-18487011>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AACFBt95w11Rc_-NDwqr7Jh5YbcrTZy3ks5qbz0lgaJpZM4IWPi0>
> .
>


I didn't find a way to set mPDF variables if is protected

this is my code

```
$pdf = PDF::loadView($this->view, $this->viewData);

$pdf->mpdf->setAutoBottomMargin = true;
$pdf->mpdf->setAutoTopMargin = true;

$pdf->setDefaultFont($this->fontName);

$pdf->stream($this->fileName);
```


Egy Pcie Ati kártyás gépen a Mate isó veres szolarizált képet produkált, majd feltéve a lice rendszerre az xorgos at meghajtót máris jó lett. Nem tudom melyek az elavult driverek, gondolom azok, melyek csak PCI buszosakban léteznek.
Az ATI az nem elavult, mert Pcie buszos eszközök tömkelege van belőlük. Az S3 már szinte biztosan dobható.
Elsősorban azokat lehet kihajítani, amelyek laptottyokban nem léteznek.


firstB


This is almost the same file twice, wouldn't once be enough?


Hmm... thought I try it out myself to test through all values each:

    #!/bin/bash
    #
    # script intended to test through TX/RX parameters. Should be called from
    # eg. /etc/rc.local with a short delay to ensure network is already up, eg.
    # sleep 10 && /usr/local/bin/test-tx-rx.sh

    TestPartner=192.168.83.146			# here 'iperf3 -s' must be running
    TimeToTest=120						# how long should iperf3 run each
    TX_File=/var/log/tx-value			# file containing actual tx value
    RX_File=/var/log/rx-value			# file containing actual rx value
    LogFile=/var/log/tx-rx.log			# result log
    SourceDTS=/boot/orangepi/new.dts	# source, must contain rx/tx set to 0!
    TargetDTB=/boot/orangepi/OrangePiH5orangepi.dtb 	# target .dtb
    TimeToTest=10

    Main() {
    	CheckPrerequisits
    	read TX <"${TX_File}"
    	read RX <"${RX_File}"

    	TX_Result=$(iperf3 -c ${TestPartner} -t ${TimeToTest} | awk -F" " '/sender$/ {print $7"\t"$9}')
    	RX_Result=$(iperf3 -R -c ${TestPartner} -t ${TimeToTest} | awk -F" " '/sender$/ {print $7"\t"$9}')
    	echo -e "${TX}/${RX}:\t${TX_Result}\t${RX_Result}" >>"${LogFile}"
    	cat "${LogFile}" >/dev/tty1
    	IncrementAndReboot
    } # Main

    CheckPrerequisits() {
    	export PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
    	which iperf3 >/dev/null 2>&1 || apt-get -f -qq -y install iperf3
    	which ethtool >/dev/null 2>&1 || apt-get -f -qq -y install ethtool
    	which dtc >/dev/null 2>&1 || apt-get -f -qq -y install device-tree-compiler
    	[[ -f "${TX_File}" ]] || echo -n 0 >"${TX_File}"
    	[[ -f "${RX_File}" ]] || echo -n 0 >"${RX_File}"
    } # CheckPrerequisits

    IncrementAndReboot() {
    	let RX++
    	if [ ${RX} -eq 32 ]; then
    		RX=0
    		let TX++
    	fi
    	if [ ${RX} -lt 32 ]; then
    		# write stuff to files, patch .dtb, reboot
    		echo -n ${TX} >"${TX_File}"
    		echo -n ${RX} >"${RX_File}"
    		HexTX="$(printf '<0x%x>;' ${TX})"
    		HexRX="$(printf '<0x%x>;' ${RX})"
    		sed -e "s/tx-delay = <0x0>;/tx-delay = ${HexTX}/" \
    			-e "s/rx-delay = <0x0>;/rx-delay = ${HexRX}/" \
    			<"${SourceDTS}" | dtc -I dts -O dtb -o "${TargetDTB}"
    		sync
    		reboot
    	fi
    } # IncrementAndReboot

    Main $@

In `/var/log/tx-rx.log` average values of each `iperf3` run in each direction will then be collected together with count of retransmits.


Hi,
Why do you ask me this?


isn’t 3848404f656dd53543544d20fc5cb80de76fe9be sufficient for this?


Depending on how the GPG key is retrieved, it can work with HTTPS.

I don't know how the script is written (and it's feasible), but if the keys could be still downloaded using HTTPS, it could be enough to ensure that the downloaded packages are correct.


By default, a class variable with no access modifiers is `protected`, which means it's visible to everything in the same package. You almost always want to make class variables `private`


The scoring of documents was messed up because we were only looking at R instead of the entire document set. Now that it uses the entire document set to come up with term weights the scores have normalized.


you**'ve** installed
~~by simply typing in your terminal~~ -> using the following command


I don't think you want this here, because it's a local variable visible only within the `DBHandler` constructor. I think you just want the `super()` call and then in `AddWorkout` you can use `this.db.getWriteableDatabase()` to get a reference to the database.


Further to my last reply, I didn't get time to work on this for last 3 days
because in India, there is a currency note crisis and the visits to the
bank to withdraw cash are taking too long. I will get to it as soon as
possible. Thanks.

On Wed, Nov 16, 2016 at 10:44 AM, Sanjay Kanade <kanade.sanjay@gmail.com>
wrote:

> Yes, I did, on the same day. It will need a design time debug to see if
> the flags are actually returned correctly from the loaded graphics. My
> guess is that the flags supportsPartialTransparency, Transparent etc are
> not always returned correctly, hence the reason for doing class checks to
> force transparency for certain class types. The bitmap has a different
> complication where the transparent flag may not be set but the transparent
> color is taken from some pixel as per Delphi legacy. What I will do is only
> do one round of design time debug with all the graphic files supplied by
> Dietmar and then decide what to do. May be, as you said, it would be best
> to always paint by new routine when not bitmap and leave the bitmap code to
> as it is. After that, we can send it to Dietmar for his opinion.
>
> Thanks,
> Sanjay
>
>
> On Wed, Nov 16, 2016 at 2:32 AM, Joachim Marder <notifications@github.com>
> wrote:
>
>> Hi Sanjay.
>>
>> Just to be sure: Did you see my other code review comments too?
>>
>> Best regards,
>>
>> Joachim
>>
>>
>> 2016-11-14 3:21 GMT+01:00 sanjayssk <notifications@github.com>:
>>
>> > OK. I will try today along with effect of all the new flags ON/OFF in
>> > Dietmar's code.
>> >
>> > Thanks,
>> > Sanjay
>> >
>> > On Mon, Nov 14, 2016 at 12:14 AM, Joachim Marder <
>> notifications@github.com
>> > >
>> > wrote:
>> >
>> > > These newly included units cause dependencies on additional packages,
>> > > which is why the VT packages currently show build errors. However,
>> these
>> > > units are not actively used here , they are just used to test for
>> certain
>> > > classes. We should check if this is really necessary, maybe simply
>> using
>> > > TCanvas.Draw(TGraphics) is sufficient.
>> > >
>> > > —
>> > > You are receiving this because you are subscribed to this thread.
>> > > Reply to this email directly, view it on GitHub
>> > > <https://github.com/Virtual-TreeView/Virtual-TreeView/commit/
>> > 660f81d2f30821bf1a66b4812f4ac6782d454041#commitcomment-19803063>,
>> > > or mute the thread
>> > > <https://github.com/notifications/unsubscribe-
>> > auth/ATX-xj-Ng3Vse39ozEF4myHR-Sh_2Ti8ks5q91p5gaJpZM4Kwhqr>
>> > > .
>> > >
>> >
>> > —
>> > You are receiving this because you commented.
>> > Reply to this email directly, view it on GitHub
>> > <https://github.com/Virtual-TreeView/Virtual-TreeView/commit
>> /660f81d2f30821bf1a66b4812f4ac6782d454041#commitcomment-19805226>,
>> > or mute the thread
>> > <https://github.com/notifications/unsubscribe-auth/AKH5-QFUO
>> N1JH275C221e0wxfOVBXrvNks5q98WsgaJpZM4Kwhqr>
>> > .
>> >
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub
>> <https://github.com/Virtual-TreeView/Virtual-TreeView/commit/660f81d2f30821bf1a66b4812f4ac6782d454041#commitcomment-19835661>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/ATX-xhFv2yeINwPtJxLhsocXDRf3Wu5zks5q-h4AgaJpZM4Kwhqr>
>> .
>>
>
>


maintainer**'s**


This is only for pratical purposes. Working on /resources directory does not demand working on /rules directory and vice-versa.
None of these files is needed, but I emulated the good idea from other languages (that usually only put it on resources).


@gheinrich Thank you SO MUCH for jumping into this repo and giving me advices 👍


lose "your"


user**'s**


Missing space around `=` at `var i=0`


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


TeamCity Jarvis :: Jarvis.DocumentStore :: GitFlow Debug / Test / Release [Build 152](http://build.prxm.it:8811/viewLog.html?buildId=10820&buildTypeId=Jarvis_DocumentStore_GitVersion) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:09


In file communications.bluecrossmn.com/logis/index.html line 49
<img src="images/logis_logo.gif" alt="Company Logo"
to
<img src="images/logis_logo.gif" alt="Logis"


Small nit, but I see that you're attaching `PR-URL` and `Reviewed-By` metadata in the commit. If you're aiming to be consistent with Node core, each reviewer should get a separate `Reviewed-By` line (it's easier to automated tooling).


Software Ambient Client.


In communications.bluecrossmn.com/mada/index.html line number 50 please change
<img src="images/MADA_logo_300x60.jpg" alt="Company Logo"
to
<img src="images/MADA_logo_300x60.jpg" alt="Mada Insurance Incorporated"


This and previous commit for admit part.


Change the backgroundColor in ondblclick event of javascript


quite right, we shouldn't just enable everything by default. Sorry @jelledevleeschouwer, this'll be off by default.

Turns out that there's a catch though. The unit tests compile with only the things that are enabled by default - so if I disable this by default, the unit tests don't even compile. Workaround for that added, feedback is welcome. It does mean that running the unit tests now uses a new command : `make units UNITS=1`  - I'll update the documentation soon.


~~all the~~ is a boilerplate plugin to help you quickly create your own Weave Scope plugin.",


@popara, have a look at the `or`-functions and its variants in http://package.elm-lang.org/packages/elm-community/result-extra/2.0.1/Result-Extra. Isn't that exactly, and more directly than `oneOf`, what you need?


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


este es el bueno


In communications.bluecrossmn.com/marsden/index.html line number 37 please change

 <img src="images/marsden_logo.jpg" alt="Company Logo"
to

 <img src="images/marsden_logo.jpg" alt="Marsden Holding"


No. It's not intended. Previously, when all job "components" had their own activity classes (PickupShipment, DeliverShipment, PickupService) it was perfect. Now we need another way to distinguise activities. I propose to give the activity an `orderNumber`: an index within the JobActivityList. This could be automatically set by the `JobActivityList.addActivity()` method, so the user creating the new Job type have nothing to do with it.
Then the equals has to be extended, that not only the job, but its order is matched.
What do you think? I can do the correction in 30 minutes.


some work on #679


I mean, I could try removing it again and waiting until the next JDK comes out. I suspect the issue is with the ETAG the server is returning someone saying that the file is unchanged (I forget the exact process that is used in the underlying workings of autopkg to tell if a file is new or not).


Ya will do. Sorry for the delay, midterm season


In communications.bluecrossmn.com/onebeacon/index.html line number 47
Please change
<img src="images/OBIG_websp_logo_BCBS.png" alt="Company Logo"
to
<img src="images/OBIG_websp_logo_BCBS.png" alt="One Beacon Insurance Group"


Hi there,

Thanks for quick reply. We have a gulpfile that creates packages based off our config and it goes through some custom logic that I cannot disclose. We then bind several scripts together using commonJS techniques at runtime. Providing an API for building bundles therefore is essential for our project.

Thanks
Ashish


Remove the line


Hello, this is a awesome project,
just reccived all the parts for it and start testing it
can u add the .fzz file please?


缩进可对上了？一般这种问题是语法错误导致的。

如果还不行的话建议把整个文件下下来替换：https://github.com/cc941201/XX-Net/blob/master/code/default/launcher/mac_tray.py


That really only caught intentionally bubbled up errors (via reject).  If you have any unexpected errors in the code while developing they would be lost (e.g. a typo).  Bluebird is also apparently much faster than native promises.


It is, but for some reason now [the command `installGithub.r` is not found](https://hub.docker.com/r/hlapp/rpopgen/builds/bkxbxvchf3cxnbgh9qm7hm5/), although AFAICT it's in the same package `littler`.

Should I switch the container to be based off of `rocker/verse` instead of `rocker/hadleyverse`?


very currency


@rawlingsj is the deploy necessary? This seems to deploy into current namespace (the one where fabric8 and jenkins are installed).
Usually the Jenkinsfile calls later a next step which deploys the release to testing/staging.


It does! Keep in mind it's not deployed yet ...


I don't remember if I mentioned this (Github isn't showing my comments in some commits), but you can omit the comparison with `true`.


you might need the carriage return here


the MtkCamera.h-file is missing.. :(