Hi.
On Mar 7, 2016 3:00 PM, "Florian Weimer" <notifications@github.com> wrote:

> Remove macros extend_alloca, extend_alloca_account [BZ #18023]
>
> And also the helper macro stackinfo_alloca_round.
>
> extend_alloca simply does not work on x86_64 and current i386 because
> its peculiar stack alignment rules.
>
> Here's an analysis of the _dl_fini situation (before the removal of
> extend_alloca).
>
> Dump of assembler code for function _dl_fini:
> <+0>:   push   %rbp
> <+1>:   mov    %rsp,%rbp
> <+4>:   push   %r15
> <+6>:   push   %r14
> <+8>:   push   %r13
> <+10>:  push   %r12
> <+12>:  push   %rbx
> <+13>:  sub    $0x38,%rsp
>
> The function pushes 6 registers on the stack and allocates 0x38 bytes,
> which means that %rsp is a multiple of 16 after function prologue.
>
> The initial alloca allocation does not change %rsp alignment:
>
> <+210>: shr    $0x4,%rcx
> <+214>: shl    $0x4,%rcx
> <+218>: sub    %rcx,%rsp
>
> %r15 is the address of the previous stack allocation, it is used below.
>
> This is the extend_alloca reallocation branch:
>
> <+734>: add    $0xf,%rdx
> <+738>: and    $0xfffffffffffffff0,%rdx
> <+742>: lea    0x1e(%rdx),%rcx
> <+746>: shr    $0x4,%rcx
> <+750>: shl    $0x4,%rcx
> <+754>: sub    %rcx,%rsp
> <+757>: lea    0xf(%rsp),%rcx
> <+762>: and    $0xfffffffffffffff0,%rcx
> <+766>: lea    (%rcx,%rdx,1),%rsi
> <+770>: cmp    %rsi,%r15
> <+773>: je     0x7f963940b673 <_dl_fini+787>
> <+775>: mov    %rdx,-0x58(%rbp)
> <+787>: add    %rdx,-0x58(%rbp)
>
> (a) %rdx, the new requested size, is rounded up to a multiple of 16
> (+734, %+738), and the result is stored in %rdx@738.
>
> (b) %rdx@738 + 31 is rounded down to a multiple of 16, the result is
> stored in rcx@750 (+742, +746, +750).  So %rcx@750 == %rdx@738 + 16.
>
> (c) %rcx@750 bytes are allocated on the stack (+754).  %rsp is rounded
> upwards to a multiple of 16, result is stored in %rcx@762 (+757, +762).
> This does not change the value of %rsp because it already was a multiple
> of 16.
>
> (d) %rsi@766 == %rcx@762 + %rdx@738 is compared against %r15.  But this
> comparison is always false because we allocated 16 extra bytes on the
> stack in (b), which were reserved for the alignment in (c), but in fact
> unused.  We are left with a gap in stack usage, and the comparison is
> always false.
>
> (@XXX refers to register values after executing the instruction at
> offset +XXX.)
>
> If the alignment gap was actually used because of different alignment
> for %rsp, then the comparison failure would still occur because the gap
> would not have been added after this reallocation, but before the
> previous allocation.
>
> As a result, extend_alloca is never able to merge allocations.  It also
> turns out that the interface is difficult to use, especially in
> cojunction with alloca account (which is rarely optional).
>
>         [BZ #18023]
>         * include/alloca.h (stackinfo_alloca_round, extend_alloca,
>         extend_alloca_account): Remove.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/bminor/glibc/commit/683543bbb3e2c1b17554c4096d00c2980f39a802
>


@rhcastilhos Você não acha que faz mais sentido ao invés de páginas colocar perfis? Acho que Perfis faz mais sentido. Então teria perfil de comprador e vendedor. Não consigo ver sentido na palavra páginas.


Although I agree, we already have a galleries.css file in the repo...maybe
we should move all of the styles in that file over to the instructions and
then get rid of the galleries.css file.  As it stands now we have CSS in
multiple areas.

On Mon, Mar 7, 2016 at 4:32 PM, Chris Ullyott <notifications@github.com>
wrote:

> @skylerkatz <https://github.com/skylerkatz> I think this CSS could be
> moved to an actual .css file but I'd wish developers choose inserting the
> CSS into an existing stylesheet over making an extra HTTP request to
> gallery.css.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MonkDev/monkcms-snippets/commit/4491a17ac8349163c55de575be2088f340a1010e#commitcomment-16550606>
> .
>

--

- Skyler Katz
  skyler@monkdevelopment.com


I want to merge #557 into master, but it's failing tests -- it that expected?

This is concerning (from the console output):
```
/usr/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'install_requires'
  warnings.warn(msg)
Could not load settings file.
#     Could not find __init__.py
Started building gpkit...
```


Good luck on LTC!


It has to manage its own scrolling because pages can style the scrollbar; in WebKit1 we wound up with the web page scrollbar side-by-side with the GTK+ scrollbar when that happened, which was really terrible.

I'm not sure why it no longer implements GtkScrollable though. I think perhaps it should. You could ask KaL in #webkitgtk+ about that if you want.

Maybe it'd be easiest to implement this using `webkit_web_view_run_javascript` to execute `Window.scroll`, `Window.scrollTo`, or `Window.scrollBy`? Not sure, but it might work.


Yeah


Asie, I thought you quit working on this?


Fixed!


Oops, sorry, I missed this -- I'll do a quick PR


update and :shipit:


Committed to wrong branch by mistake, whoops


Excellent! Glad to see this on the master branch, too, since that makes it easier to ship features based on it out to users.


This appears to have broken the website: https://buckbuild.com/rule/apple_library.html

A test plan of CI isn't really sufficient for changes with documentation.


@rhcastilhos Minha opinião é que "Perfis" faz mais sentido do que "Páginas"


I think the hack in the first bug would work, but the only way to fix this properly is to modify WebKit. The SoupSession lives in the WebKitNetworkProcess, so it's not possible to directly access from the UI process; this would have to be done via internal message passing.


I'll clean this up after the merge, and put codenarc in for the next person.


Actually you should just merge 1.9, I guess.


I just thought that having an alternative of this function with `Option<Iter>` instead of `Option<u32>` could be very useful.


Sounds good!


still failing gpkit_Push...


Cool!


See earlier response - as far as I can tell there are no angular externs in lib/closure-library. We don't currently require closure *compiler* (as opposed to closure *library*) which has them since it seems rather heavyweight just for a few externs, but let me know if you think that would be cleaner.


There seems to be something wrong with the setup.py in master:
```
python setup.py build
...
error: package directory 'gpkit/models' does not exist
```


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1018 of 1021 tests passed, 0 test warnings)

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


I'm looking into why it said the tests passed on #554


I'm still confused here whether this is the right way to `send`. According to the docs, there is no `rg4js('send')` method, so this is still synchronous? Does this make sense for an ansynchrous version? Do we need to do additional work here to make sure `Raygun` is available?


For #1, and #2


Reworked the names in 48c6633.


Oh I know what this is.


@sgrif FYI: https://github.com/rails/rails/pull/23197


I'd suggest to extract the list out of the loop (this value is recomputed every time, but it doesn't need to)


Done.


I added the extra assertion in 48c6633.


I see this in here https://acdl.mit.edu/csi/job/gpkit_Push/buildnode=reynolds,optimizer=mosek/186/console

```
package init file 'gpkit/models/__init__.py' not found (or not a regular file)
```

but I don't know why that didn't fail the build


I removed the  `not nil` assertions in place of assertions with explicit id values in 48c6633.


ur casts are terrible. u casted variables to uint64, then cast back to uint32, it just simply can overflow again.
fixed overflow with overflow.


Sorry, não devia ter dado este commit aqui. vou voltar para o anterior


:notes: For the times they are a-changin' :notes:


This is the `SourceDeblendTask` configuration.  Having a `doFoo` Field and a `foo` ConfigurableField is a very common pattern in our code.


@borshop: retry


I think I misunderstood something here: if I want to modify columns 2 and 3, I will have the following colums and values:

```Rust
columns = &[2, 3];
values = &[&1, &2];
```

Then when get to this assert, it can only fails since values has a length of 2. I'll provide a patch for it tomorrow.


Could `gpkit/models` still exist on the testing machine somehow? I had to remove it with `rm -rf` on my machine


^ &#128077


Yeah, they were useless, fixing the code


not 100% why there are two initializers. I noticed that the raygun was being instantiated twice in the original code, so I'm not sure both types of initializers are needed. I don't fully understand the difference b/t the two types in addons, so could use some direction here.


might be time for a new minor release?


@JohnCover Hey i havent had the chance to test this yet, but i was looking at the diffs and i thought id shed some light on what im expecting, eventually this whole activity will disappear and will be implemented into the login activity

But what i want is...
1  Set default location to csulb(just hardcode the coords)
2  attempt to get **Last known location** via the network provider
3  start the timeout and start attempting to get the gps location
4  if we cant dial in on a location in 10 seconds the (what appears to be) timeoutexception will fire and set the usersLocation to what we were able to get. Either the default(worst case), last known(medium case), or gps loc(best case)

So basically, we dont update ```userLoc``` until the timeout event has fired OR we get the gps location before the timout has occured

Let me know if you have any questions or if that doesnt make sense


👍 @MaxKroshka


Did you know Donald Trump wants to ban pre-shredded cheese in the US?

....he wants to make America grate again


I argued my position on this 3 years ago; it's the same as @TheImmortal's. I didn't really understand @Zarel's position back then, but now that it's clearly outlined here I see that it's even more unreasonable than I thought.

The point is a base HP of 1 gives a minimum of 11 max HP at level 1. So does a base HP of 0. In fact, every base HP below 50 has the same minimum max HP with 0 IVs and 0 EVs at level 1.

If we want to support an easy way for mod developers to have a max HP of 1, then using an impossible base HP would be the way to go -- i.e. ``|| this.template.baseStats['hp'] === -1`` on this same line. If a mod developer wants to support giving Shedinja more HP, then they can add ``&& this.template.baseStats['hp'] === 1`` to the Shedinja hardcoding. There are a number of ways to accomplish all of this without breaking the HP stat formula for everything with a base HP of 1.


%{version} and %{name} mess up with yum-builddeps


No I completely blow away the testing directory and do a fresh clone every
time we run the tests

On Mon, 7 Mar 2016 at 19:51 Warren Hoburg <notifications@github.com> wrote:

> Could gpkit/models still exist on the testing machine somehow? I had to
> remove it with rm -rf on my machine
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hoburg/gpkit/commit/cf3083e223e67aa8c2705a620967d5808fa64fb9#commitcomment-16550872>
> .
>


This seems wrong: summing logs is not equivalent to logging sums.  Related problem: these commits do not have tests.

Also, please refactor this code so that it is nearly all shared with SoftmaxEigenImpl.  There's a rather large amount of index manipulation which should not be duplicated code.


2097-EP-8-5 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


@ThisTestUser im not sure, the arrows fly sideways and stick sideways, but if you use ".enchant" on the bow, they fly better


I want eve3ryone to lok at this pull request and see if you like it


@signalpoint I found that a similar approach has to be applied in other procedures as well in order to make an entity editable without errors in case of different language.
I'll send you a pull shortly, of you like.


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2150 of 2167 tests passed, 1 test warnings)

 * 15 tests had: EIO diffs.
 * 7 tests had: ERR diffs.
 * 14 tests had: ESO big diffs.
 * 2 tests had: Table small diffs.
 * 11 tests had: Table big diffs.
 * 10 tests had: MTR big diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: ESO small diffs.

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


rename the directory


@liach

Uh no, we run things according to a plan. At this time, 1.9 is not going to be officially versioned.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18609](https://teamcity.mesosphere.io/viewLog.html?buildId=156196&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:17


He intentado sacar relaciones entre los offsets de los cubos y la semántica ha quedado mucho más confusa, por lo que al final lo he dejado como offset y le he puesto los valores directamente. Quizá haya abusado de este recurso.
Por otra parte, la constante SHIP_NOSE_ROTATION_CORRECTION_FOR_HELPER_CYLINDER tengo mis dudas acerca de su aporte a la semántica.


`color: white;` should work here. Should definitely avoid using it in your html. If you find yourself adding the css property and it's not working, You should first see if the code is even showing up by using the chrome developer tools. If there isn't something wrong with the code (a missing semi-colon or bracket) then you might have accidentally set the color somewhere else and that could be overriding your changes here.


i think here is something wrong . qs text added in void update icons .i think this is the wrong call . and  detail view lag because of the detail view text buttons at the bottom right . it stucks when text buttons on header . having text view error on log


@TrinityDevelopers I know, @byteandahalf told me


What does a prefetcherType empty implies? Precisely, what's the difference between Empty and Invalid?
If they are the same, the empty should be removed.


Why this change?


when you set the font-family you are only setting the font that the page should be using. Since you don't specify the font-weight in your css file it will resort to the default weights. In general this is appropriate as you should have some default weight for your webpage and then you can adjust it for the specific sections you need to change.


리뷰 완료했습니다.


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Why depth is set to 1? If there is no prefetcher, all the variables should be set to 0. During construction (via constructor), you can set the depth to 1, if user doesn't pass the depth as a variable in the configuration file.


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


archiving

On Tue, 8 Mar 2016 2:48 am Connor <notifications@github.com> wrote:

> everyone knows how to make an internal radar??????
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Snorflake/csgoradar/commit/99aa36624b6075fa5ab6cff308685bf7bacb0057#commitcomment-16543183>
> .
>


Could we use a ConfigurableImageDataStrategy here? I'd like a "do the right thing by default", which would appear to be in memory if its small, and file if its big.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18610](https://teamcity.mesosphere.io/viewLog.html?buildId=156228&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:59


Every Enumerated variable should start with invalid, and set to one of the two options during construction. This makes sure that no unforeseen bug occurs (for example when we don't have a prefetcher, but a valid type)


This is the error I get with the default value:
<pre>
No distributions matching the version for sword2 (from -r /rpmbuild/SOURCES/archivematica-storage-service/requirements/base.txt (line 22))
</pre>


If both setter and getters are used (setAddress and getAddress), then the variable address should be declared as a public member, and setter and getter removed.


Same here with next. remove the getter and setter, and declare it public. Although this is a inline function call, it still degrades performance, in comparison to using a public variable. In a lot of cases (such as when --enable-debug is used during multi2sim configuration) the multi2sim is compiled without optimization (-O0) which means all of these are considered as a call to an inline function


same as above


I worry about the speed of this line: three string comparisons for each of the `paramNames`, getting executed for every log message.  Maybe it doesn't matter because it's only for the verbose setting, but maybe there's a better way of doing this?


noticed the force push here... i was interested in some of the other items in yesterdays version of this page.. :)

still got that around?


Use C++11 range-based `for`?


I still wondering that how to define for() loop in qt5


I kept on getting the error that `pot.get()` returns a value above Integer.maxValue, it's much too high for our program to handle.


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-MacOS-10.9-clang: OK (2112 of 2129 tests passed, 1 test warnings)

 * 15 tests had: EIO diffs.
 * 7 tests had: ERR diffs.
 * 14 tests had: ESO big diffs.
 * 2 tests had: Table small diffs.
 * 11 tests had: Table big diffs.
 * 10 tests had: MTR big diffs.
 * 2 tests had: MTR small diffs.
 * 1 test had: ESO small diffs.

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


anyway good work bro ......appreciate it very much.


`private`


Awesome, thx @PeterDaveHello!


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18611](https://teamcity.mesosphere.io/viewLog.html?buildId=156234&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:08:18


Cmabia tal


poor que ??


por que


I think we need to handle DES blobs in the same way we handle image segment data. If the DES is just TREs, we're OK because they are fairly small. However there is a controlled DES (LIDARA) that is a .las file in the DES. So that can get huge pretty quickly too, and we're most likely to want to save that kind of thing off as an external file rather than to keep it in memory. If that is too much for this round, I'd be OK with kicking that off to a new ticket for a future release.


Yeah that seems likely, the i18n javascript that is copied to `public/assets/javascripts/i18n` depend on the `extend` method which no longer seems to be available in the `i18n-js` node module.

@thenickcox any ideas on how we can simplify the i18n situation?


Hey, please use 4 spaces for indentation and don't reformat the schema in models/tweets.js


correction to ref: https://openuserjs.org/discuss/Import_went_wrong


we are in class prefetcher so the use of prefetcher at the beginning of the function's name is redundant, and makes the code ugly. For example a function call from outside to this function looks like this:
mem::Prefetcher::prefetcher_do_prefetch(...)

Also based on the coding guidelines the functions should be named using capital letters at their beginning like this:
DoPrefetch(...),

Lastly, can you do a better naming for this function? :confused: DoPrefetch is kinda weird. We have nowhere in the code a function that starts with Do :confused:


AccessMiss(...)


this should be just `Tweet`. And the mode is exported, not the Schema ;)


AccessHit(). Also why not using Hit and Miss? instead of AccessHit, and AccessMiss.
From outside it is apparent that we are talking about the prefetcher hit and miss:

prefetcher->Hit(...)
prefetcher->Miss(..)


Work on photodrop

On Mon, Mar 7, 2016 at 4:54 PM, Franklin Shieh <notifications@github.com>
wrote:

> 👍 @MaxKroshka <https://github.com/MaxKroshka>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ProfoundMongoose/GithubBounties/commit/8e63e3ff9af5fbe1aeb50fc5d20c3afc867188e2#commitcomment-16550906>
> .
>

--
*360 223 4002*


GetType()


you don't have to copy all the comments again. They're already in the Schema


@tamorim este foi o único ponto que não consegui revisar. Pesquisei aqui e parece sr alguma coisa com as novas versões. Antes funcionava, o que eu fiz foi colocar um exemplo na explicação do campo. Por enquanto vamos deixar assim.


Can you give a case why this is needed? I am conditioned to shudder and worry at the mention of "duplicate" and "kerning". Curious why you want this.


@hudakdidit Not without digging into to it a good bit.


Often giving classes names that reflect what they look like can be a bad idea. The issue comes if we at some point decide to change how this h1 element might look. For example lets say our logo is changed so that it uses the smallest font-weight. Well all we need to do to change that is simply go into our extrabold class and change the font-weight to something different. However now we have this class name that doesn't mesh with what we are seeing on our page and anyone looking at the html/css might be confused. So instead of using a class like extrabold, we can instead use a class like "logo" or something more descriptive of the type of content that is being represented so we don't run into strange mismatches.


```ruby
   def iterative_search(name)
    entries.each do |entry|
      if entry.name == name
        return entry
        break
      else
        nil
      end
    end
    nil
  end
```


SetType(...) Also, again, no setter and getter together. Technically, you are not allowed to use setPrefetcherType. The prefetcher type is set during the initialization, and shouldn't ever change during simulation.

So only use getter


I would remove this change. The help text belongs in documentation, rather than configuration script.


Just use getter. Remove setter


Remove setter


Thanks for contribution.. It is quite handy for doing measurements.


This breaks automatic brightness, at least on honami. If automatic brightness is enabled in weak, but still enviroments where theres still plenty of light to for example, read a book the display almost goes off, it's impossible to see anything at all.


Please make a pull RQ on this (minus the change noted above) and I will merge.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (398 of 398 tests passed, 0 test warnings)

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


I love what you did with this part of the commit. It seems very clean. But at the end I left you a commit that is very very important. Please read that and follow if you find it reasonable :smiley:


@argelius handleChange is a little bit general, maybe selectedTabChanged, or can this also happen for other reasons?


slip this commit in after


But we keep the change that GNU gets selected with 'y'?

On Tuesday, 8 March 2016, Philip Lamb <notifications@github.com> wrote:

> Please make a pull RQ on this (minus the change noted above) and I will
> merge.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ThorstenBux/artoolkit5/commit/1573ee3ef1dbe20a1b6179037f86b525a85e4cf6#commitcomment-16551305>
> .
>

--

Thorsten Bux | Developer Advocate
thorsten.bux_c@daqri.com  <thorsten.bux_c@daqri.com>

www.artoolkit.org


Document format issue is introduced at this point:

There should be "```" to close the code block.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (390 of 390 tests passed, 0 test warnings)

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


The items I ended up removing in the cleanup were:

+ https://github.com/oh-my-fish/theme-agnoster
+ https://github.com/oh-my-fish/theme-cmorrell.com
+ https://github.com/tuvistavie/fundle
+ https://github.com/fishery/shark


One thing to remember is that when you really want to change an elements style you want to be accessing the element itself. So if you want to change the color of h1 or your h2, you want to add those stylings to those elements specifically, as opposed to relying on the parent element itself.


@geraldb could you check this format for rounds and game? Is it correct?

I think it's okay but I could not validate by sported gem because the command 'sportdb new br' doesn't work, should I add some configuration to another repository to fix this?

thanks


www.baidu.co
public static void main()


I will try to fix this issue in new commit.


great


@tamorim todos os pontos foram corrigidos.


When you run into some of these problems, sometimes the simplest way to address is to remove some of your styling elements and return to a default state. This can be done through the chrome development tools or by commenting out your css code that doesn't seem to be working.


Doh! I updated the app, but forgot to restart the process. Try again please.


Very nice! Thanks


If you mean building the binary SDK with the gcc/libstdc++ toolchain, that is a choice made by the build engineer, rather than anything that needs to be addressed in the current configure script.


is this working, don't you have have an object {content: 'A', tab: 'B', tab: 'C'} that converts to {content: 'A', tab: 'B'} ?


Out of curiosity, why move the sqlite download to amazon aws?


Hey! Cool. You're welcome. I'm making those changes we discussed. Let's sync up again before I commit them.


@lzghuluwa thx!!


This `.babelrc` setup is not optimal. It's probably better to set it up as something like

```json
{
  "passPerPreset": true,
  "presets": [
    { "plugins": ["./build/babelRelayPlugin"] },
    { "presets": ["es2015", "stage-0", "react"] }
  ]
}
```

This lets you follow the more common execution path where the latter 3 presets are applied in a single pass, and avoids potential edge cases (maybe bugs) like https://phabricator.babeljs.io/T7191.


Julian,

I am very pleased with this commit. It seems you have it all figured out, but I have a request:

Please fix the header files based on the commits that I provided. I tried my best to just comment the header files.
In your next commit which I assume will not take that long (since you did a great job this time), you should update the header files but **Remove all other .CC files**.

I promise you that I drop everything, and comment on the new .h files, right away after you send me an email and say you commited them. If you comply with my comments ( I swear it is not me, it is standard programing + the coding guideline) I will give you the green light to commit the CC codes. I assume it will take you to recommit the CC files no more than 5 minutes.

With recommitting the CC files alone, I can fully focus on them and review them right away. But if I review them right now, you eventually have to change them to comply with the new header files. What happens is, if I review them now, then when you recommit the CC files, github does not show all of your code, and shows the partial changes. It would be harder for me to review the code (since I would require to read the whole code and understand them).

You did a great job here and I hope this extra work does not discourage you. Believe me when I say it would make the review process much easier and better, and therefore you will get the best version of code for pushing to the master branch.

Also this has happened to all of us at some point (Rafa did the same to me in multiple occasions) so I understand if you get angry at me, and I promise I make it up to you :tongue:


Now it works! :-)


Still not generating osd hosts config in ceph.conf. Checking out why.


remove tab


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 22](https://teamcity.labkey.org/viewLog.html?buildId=308680&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8, passed: 39, muted: 4 Build time: 01:17:19
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

```


Just proposing to change the question from 'would you like to use clang' to
'would you like to use GNU' as for now GNU causes less trouble then clang.

On Tuesday, 8 March 2016, Philip Lamb <notifications@github.com> wrote:

> If you mean building the binary SDK with the gcc/libstdc++ toolchain, that
> is a choice made by the build engineer, rather than anything that needs to
> be addressed in the current configure script.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ThorstenBux/artoolkit5/commit/1573ee3ef1dbe20a1b6179037f86b525a85e4cf6#commitcomment-16551395>
> .
>

--

Thorsten Bux | Developer Advocate
thorsten.bux_c@daqri.com  <thorsten.bux_c@daqri.com>

www.artoolkit.org


okay ... I will remove them .


Isn't renaming to prefetched better?
Also isn't Boolean better?


bool isPrefetched() const {return prefetched;}


There has to be something in there


Hello World!!


Were you unable to use the 'Open Sans' google font for this page?


Do you need the setter? In this case I am not sure. Make sure if you can just use the getter, otherwise make it a public variable _bool_ for the block


Is this more or just comments?


Hello, what is the reason for this PHP code to be commented out, making the default payload non functional?


what is value? can it be other than true or false?


Using this branch as a starting point, I was able to get sbt incremental compilation to work with Dotty (I'll push my work in a few days once I've got it cleaned up), thanks! I'd like to share a couple of comments:
- This won't work as-is because of classloading issues, we need to pass objects of classes in `xsbti.api.*` between the compiler and sbt, so these classes must be loaded from the same jar and also from the same classloader. Loading from the same jar is easy, just depend on `"org.scala-sbt" % "interface" % sbtVersion.value`, but loading from the same classloader is much more tricky: the classloader that sbt uses to load the compiler bridge does not work for that, I had to pull it apart and make my own using reflection (when I push my code I'll make sure to properly document how and why this is done).
- I'm now skeptical on whether having this phase be part of the compiler is a good idea: unless the whole incremental compilation logic of sbt is moved into the compiler, this effectively ties the compiler to a specific version of sbt. Changes to the incremental compiler api are not uncommon, one happened last month for example: https://github.com/sbt/sbt/commit/65768446986d4e521cdb3f82f807d4886474c216#diff-01234e8de7c750474f13a8328eb91125, and improvements like https://github.com/sbt/sbt/issues/1104 will require more changes. If this phase becomes part of the compiler then we must ever freeze this interface forever or live with the fact that a new version of sbt might suddenly break compatibility with every released version of scala. Do you see some alternative to this?


-> 'if they exist.'


#5531-HVACTemplate-5ZoneUnitarySystem-shows-max-iteration-warnings-on-design-days (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1576 of 1579 tests passed, 0 test warnings)

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


remove extra tab


@rlpereira

About the format - The idea is to use the proper team names and NOT the lowercase keys e.g.

    santossp    0 - 0 flamengo
    gremio      2 - 0 nautico
    pontepreta  0 - 2 saopaulo

Should be something like:

    Santos SP    0 - 0 Flamengo
    Grêmio        2 - 0 Náutico
    Ponte Preta  0 - 2 São Paulo

Hope that makes sense. Let me know.


I try.


Excellent structure to your HTML and great naming of your classes/id's


Err, never mind, that doesn't work. Need to dig into this more.


Nope :)

Look at the edd_deprecated function


this `-va` argument is not necessary but I would like to keep it, to show code in verbose in console doesn't hurt:

1. when graphic system works nobody will read this; when graphic system doesn't work, this verbose code will hint user this could be part of the problem

2. also below code, could you keep indented ? otherwise when code becoming long, it's going out of maintainable


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (3048 of 3048 tests passed, 0 test warnings)

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


从来测试的  可以上传了  github花太多时间了


Can't these variables be part of the prefetcher itself:
Why it should be part of Cache?
I assume the cache only has to have an object called Prefetcher prefetcher, and these are variables associated with the object Prefetcher. They are zero, when there are no Prefetcher, and the PrefetcherType is invalid, and the object prefetcher doesn't exist (a unique_ptr).


Don't forget to note this in the change log.


Thanks @chrismattmann


This is great :+1:


@anarchivist I'm struggling with a relative link from the FAQ to the new blog. According to things I read, like [1], it looks like we need a baseurl variable in the config file. I don't see one there. Is this something you can help with?

[1] http://ilovesymposia.com/2015/01/04/some-things-i-learned-while-building-a-site-on-github-pages/


i love doze ... its good lol


Wah, this changes the error behaviour significantly... Judging by the test suite (and the changelog) though there is no mention of the this significant change at all. Our app relies on the fact that the `error` key is there upon receiving an array based error, but now this is no longer true... Is this intentional?


I'm very confused


It's ok to pass these variables to the constructor of the cache, but then they should be passed to the constructor of the prefetcher as well (The cache constructor acts as an intermediary between the configuration file, and the prefetcher constructor for these values)


okay. It got reformat when I clicked beautify ... so I thought I should keep it ... I will get it back to it's previous format!


ok, github only showed part of it, its fine!


@kim-yannick @mcccs @ThisTestUser i can indeed confirm the latest changes have the arrows working smoothly again


For font-weight you just use the number and not a pixel designation. So it should be `font-weight: 600;`


missing semi-colon


I think the name for this design pattern is [https://en.wikipedia.org/wiki/Facade_pattern](facade).


read it


It might be clearer to call it `self.session`


So there is no longer any problem with the Trajectories? Also, is it accurate at least within a block?


There is no specification for the formatting of JavaScript. Maybe your beautifier is a little more strict than mine :stuck_out_tongue_closed_eyes:
But we should keep it consistent.


Hi @ngokevin,

I think this change breaks ```removestate```. ```removestate``` expects ```is``` to return the index of the state, or false if it doesn't exist. After this change, ```removestate``` does is not able to remove the proper state because it doesn't know its index. I opened #1107 to track this issue.


It doesn't make it non functional. The comments are specifically crafted to make it so that the `<?php` tag is included when there isn't already one, but commented out when it's already part of a PHP code block.


@rlpereira   About:

>  I could not validate by sported gem because the command 'sportdb new br' doesn't work, should I add
>  some configuration to another repository to fix this?

I try to add a "in-situ" Datafile to the repo later this week than you should be able to just use: `$ sportdb build`. If you don't see the Datafile in the repo by Friday, please send a reminder. Cheers.


It would be better to keep the raw_api passed into the constructor.  That way, if it's a stub it won't use the real one.


Done, thanks @jlaako


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (390 of 390 tests passed, 0 test warnings)

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


button is a somewhat strange html element because it often comes with so much default styling. But you've done a pretty good job getting rid of that styling, though I'd go with `border: none;` Also sometimes instead of setting a width to elements that already don't take up the full width of the page, an easier solution is to just adjust the padding to make it look the way you want. That's where the chrome developer tools can make finding the right size a little easier.


Why are left_class_kerning and right_class_kerning dicts?  I don't see where you use the values.


@drryanjames good catch!


Good overall html layout, there are a couple simple syntax errors in your CSS that once you correct should help you fix some of the missing design requirements.


@argelius if i look at the example it first shows the second page and then switches at the beginning, that needs to be fixed, i will look at it, other than that it works great!


Any reason not to use https://github.com/kylewm/mf2j/blob/master/src/main/java/com/kylewm/mf2j/JsonDict.java here?


- 更高研发效率的益处
- 按这一思路，较有想象力的方法包括
- 药物一旦， 将提供给
- 激励科学家自愿研发新药物
需要把形容词转化为主动


làm tốt lắm, nhưng số hoàn hảo dùng for là chưa được nhé


@pierpa68 That would be great, thank you.


The SHA of this commit is really nice 😝


OK


sure :)


very cool! You probably want to also include builders that accept cache loaders as well (e.g. for bulk loads).


ráng làm tiếp đi anh


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-NoThreads: OK (394 of 397 tests passed, 0 test warnings)

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


init


我想回退到这个版本怎么？ @ @DougLee @SuperZDev


FYI: In the situation you want to export the sensor data for some other softerware (matlab for example), convert the 'quality' into uint16. The original format of the quality is uint 8 and it will easily cause read failure since the encoder will take it as a char.


bài làm rất tốt và chính xác :dancers:


Is there a reason for the `~` in the bower.json? Would you be open to using the `^` to align with the package.json. QUnit v1.21.0 fixes an issue we're seeing running Ember unit tests (it has to do with retrying test fails).


      代理使用weak的原因：
          (1) 防止对象间互相持有（互相或间接互相设置为代理）导致的循环引用，而出现无法释放的情况。
          (2) 防止直接持有对象，造成代理对象被释放掉的时候仍然错误的发送消息。
          (3) weak 可以将被释放的对象自动设置为nil，保证内存安全。


@ajjl We can not accept WIP's into the master branch.

With all the separate patches, it difficult to read what the final solution really is.

Please provide fewer patch sets, and better commit documentaiton about how and why this solution was selected.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (398 of 398 tests passed, 0 test warnings)

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


@joegc This commit also reverted #201. Please be more careful next time.


Please remove this line in this file and other files.


Please insert license information into header.


@SamVerschueren awesome idea, feel free. I think the code for GitHub API access is already in https://github.com/Microsoft/vscode-extension-vscode/blob/master/bin/install#L61 and maybe could be shared across the two. The downside is that you will need to also support GITHUB_TOKEN to prevent throttling, so it might break existing travis integration tests.


so why is this commit here


Looks the issue here is that this kind of change is hard to maintain correct in a PR that could be debated for ages.

Sometimes you just need to commit the change and work from there.

**it was a mistake to force 95%+ of users to `s/getInstance/getActiveInstance/` plain and simple**.

The correct thing should have been to make it like this.

Now if having `getInstance()` blow up now is too much of a stretch, then I would be ok with

```
@CLIResolver
@Nullable
public static Jenkins getInstance() {
  return  HOLDER.getInstance();
}
```

**for 1.x** and switch to the `@Nonnull` blow up variant for 2.x

Now as to what code actually needs to check for `null` (or what code *should* be using `getInstanceOrNull()` after this change)

* Code that runs before [here](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L782)
* Code that runs after [here](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2906)

So consider the plugins... they are not even on the classloader before [here](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L830) so anything in a plugin that is run during start-up can assert with impunity that `Jenkins.getInstanceOrNull() != null`

So we are left to deal with plugin code that might execute during tear-down...

* All the `@Terminator` annotated methods can assume `Jenkins.getInstanceOrNull() != null` as they will be invoked [before theInstance is set to `null`](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2840)
* All the Node shutdown stuff can `Jenkins.getInstanceOrNull() != null` as they will be invoked [before theInstance is set to `null`](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2860)
* All the Queue persistence logic again can assume `Jenkins.getInstanceOrNull() != null` [still before](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2889)
* Custom loggers, [safe too](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2904)

So can anyone show me a case where a correctly written plugin actually could see `Jenkins.getInstanceOrNull() == null`?

The only place I can see where it might matter is in [HudsonIsRestarting/index.jelly](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/resources/hudson/util/HudsonIsRestarting/index.jelly) so any tag on the rendering path for that which includes plugins might just possibly have a valid case for checking against a `null` Jenkins... so all that means is that we should change `Functions.isExtensionsAvailable()` to also check against termination... that seems like a sensible change in any case, and one that should be made irrespective of this change. So I have files [JENKINS-33377](https://issues.jenkins-ci.org/browse/JENKINS-33377) for that issue.

So again I ask:

**Outside of core, is there any legitimate case where any plugin can have a code path where `Jenkins.getInstanceOrNull() == null`?**

I say no. I also say that of the 65 uses in core of `Jenkins.getInstanceOrNull()` at least 30 of them should safely be replaced by `Jenkins.getInstance()` as they would also only ever be invoked between [`theInstance=this`](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L782) and [`theInstance=null`](https://github.com/jenkinsci/jenkins/blob/bb7c8fcedbcc9b51c5b1bb5b32810af5ac6b1ffb/core/src/main/java/jenkins/model/Jenkins.java#L2906). The only reason I didn't fix them in this commit is making a change like this requires touching everything at once in a short time to ensure any legitimate usages do not fall through the gaps


Ok.


I've just tested yesterday and it works great. Great job! <3


The former is a lot less invasive than the latter, which could break external modules too.


This looks like a hard-coded literal. Consider passing the name of this file as an argument to the function.


Hey, as I'm currently writing a simulation. What's the benefit of having to call `byzcoin.NewSimulationByzCoin`over simply having `byzcoin.NewSimulation`? The latter doesn't contains redundant information... (which would be true for every simulation)


`.build/venv/bin/pip`?


This for `datetime` needs to be `DateTime` but the code above with give lowercase, this would fix the issue

            {%- if fields_mapping[field] is defined and fields_mapping[field]['type'] in ['date', 'time'] %}

            ->add('{{ field }}', \Symfony\Component\Form\Extension\Core\Type\{{ fields_mapping[field]['type'] | capitalize }}Type::class)

            {%- elseif fields_mapping[field] is defined and fields_mapping[field]['type'] in ['datetime'] %}

            ->add('{{ field }}', \Symfony\Component\Form\Extension\Core\Type\DateTimeType::class)


Is this commit already pulled? Or shall I make one?


I want to avoid having `NewSimulation` in multiple places in the code. What do the others think?


why call `f.close`? It should be close by the `with` construct.


Downloaded and copied DPDK in SPDK and verified the above changes in the following files i.e
spdk.common.mk
spdk.subdirs.mk
source path :/root/uma/nvme-spdk/spdk-master/mk/.

To build SPDK used
cd dpdk-2.2.0 && make install T=x86_64-native-linuxapp-gcc DESTDIR=.

Once the prerequisites are installed, I run 'make' within the SPDK directory to build the SPDK libraries and examples.
Here, i'm getting the following PCI related errors on my set-up (Linux :2.6.32-573.el6.x86_64)
== spdk-master/lib/util (all)
  CC file.o
  CC string.o
  CC pci.o
pci.c:47:23: error: pciaccess.h: No such file or directory
cc1: warnings being treated as errors
pci.c: In function ‘spdk_pci_init’:
pci.c:91: error: implicit declaration of function ‘pci_system_init’
pci.c: In function ‘spdk_pci_device_get_domain’:
pci.c:107: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_bus’:
pci.c:113: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_dev’:
pci.c:120: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_func’:
pci.c:126: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_vendor_id’:
pci.c:132: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_device_id’:
pci.c:138: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_subvendor_id’:
pci.c:144: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_subdevice_id’:
pci.c:150: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_class’:
pci.c:156: error: dereferencing pointer to incomplete type
pci.c: In function ‘spdk_pci_device_get_device_name’:
pci.c:162: error: implicit declaration of function ‘pci_device_get_device_name’
pci.c:162: error: return makes pointer from integer without a cast
pci.c: In function ‘spdk_pci_device_cfg_read8’:
pci.c:168: error: implicit declaration of function ‘pci_device_cfg_read_u8’
pci.c: In function ‘spdk_pci_device_cfg_write8’:
pci.c:174: error: implicit declaration of function ‘pci_device_cfg_write_u8’
pci.c: In function ‘spdk_pci_device_cfg_read16’:
pci.c:180: error: implicit declaration of function ‘pci_device_cfg_read_u16’
pci.c: In function ‘spdk_pci_device_cfg_write16’:
pci.c:186: error: implicit declaration of function ‘pci_device_cfg_write_u16’
pci.c: In function ‘spdk_pci_device_cfg_read32’:
pci.c:192: error: implicit declaration of function ‘pci_device_cfg_read_u32’
pci.c: In function ‘spdk_pci_device_cfg_write32’:
pci.c:198: error: implicit declaration of function ‘pci_device_cfg_write_u32’
pci.c: In function ‘spdk_pci_enumerate’:
pci.c:206: error: storage size of ‘match’ isn’t known
pci.c:214: error: ‘PCI_MATCH_ANY’ undeclared (first use in this function)
pci.c:214: error: (Each undeclared identifier is reported only once
pci.c:214: error: for each function it appears in.)
pci.c:219: error: implicit declaration of function ‘pci_slot_match_iterator_create’
pci.c:222: error: implicit declaration of function ‘pci_device_next’
pci.c:222: error: assignment makes pointer from integer without a cast
pci.c:223: error: implicit declaration of function ‘pci_device_probe’
pci.c:229: error: implicit declaration of function ‘pci_iterator_destroy’
pci.c:206: error: unused variable ‘match’


I understand $route.reload() is required?


This is sparta!


So this rounds down from stimtime + 2 ms to the nearest grid, correct? Shouldn't this round up to the nearest grid, or is the 2ms supposed to be large enough to always go to the next biggest grid time for all test cases, which is then sufficient for all spikes to arrive at spike detector? If it's the latter, it would probably be good to add a comment by the tested cases in the assert or die regarding what is the maximum testable case value (0.1? 0.2?)


Is it guaranteed that `vp.dir` will be empty at this point?


Example of gitignore


No problem, thanks for having a look :)


Спасибо, опечатку исправил. После закрывающих скобок в конце абзаца точки не стоят по чисто эстетическим причинам.


@knagato おお...ありがとうございます。翻訳を追加しました。
https://github.com/mozilla-japan/gecko-l10n/commit/2c9e03aca4a91e7c33ec5601ffb66d12eee2ce43


@matze I don't know if this is the safest but it is the fastest and I tested it with `g_strsplit` and it works.


Nossa @fabiozaffani você não faz ideia dos DIAS que passei procurando por algo assim!

O problema era que ao criar o cliente via API, ele não recebia esses dados.
Alias, podemos acrescentar CPF no array e todos os dados "extras" do plugin são importados para pessoa física.

Só faltaria que esses dados também sejam importados ao criar um pedido, já que mesmo enviando os campos pra conta do cliente, woocommerce não reconhece eles o que faz com que não apareçam no ressumo do pedido, nem no painel do cliente.

Seria ótimo se o @claudiosmweb conseguisse arranjar um tempo para adicionar isso. :+1:


`read -r`


strange... i will try again.


no empty line


Hmm, I see. But I think `packagename.NewSimulation` instead of `packagename.NewPackagenameSimulation` describes the same. Anyway, I will continue with the simulation and adopt to whatever we agree on.


`minijail0` already has everything you need.


why we need let f->fourcc = fourcc twice?


Why did you remove new line? It will raise errors even for warnings which follow in the middle of the text


lol, thanks :)


It will always be the last one :)

On 8 March 2016 09:59:14 GMT+00:00, "Zoltán Bodor " <notifications@github.com> wrote:
>err msg: ... from pool
>I'm nitpicking : )
>
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/Financial-Times/coco-system-healthcheck/commit/b7386ec2976e935ad2c44a7d506fb30ca463ef39#commitcomment-16557431

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Not saved in test mode?


Please add the description for the new added subroutine.


BTW I have created https://github.com/jenkinsci/jenkins/pull/2091 to fix the only code path I could find where a plugin could *accidentally* see `Jenkins.getInstanceOrNull() == null`


ソースコードみた感じだと、ffmpegはどんなふうにallocateされてもOKみたいです。ようは、freeしていないからallocateした方がfreeすべき。唯一のfreeはffmpeg.cの

```go
    /* close each encoder */
    for (i = 0; i < nb_output_streams; i++) {
        ost = output_streams[i];
        if (ost->encoding_needed) {
            av_freep(&ost->enc_ctx->stats_in);
        }
    }
```

というと、ライブラリーが自動的にstats_in/outを解放してくれないので、それがリークとなってしまう可能性があるのでtranscoderでcloseする前に解放しないといけない。

一応、ルール的にできるだけC.free/C.mallocよりもC.av_malloc/C.av_free系に統一したいです。

```go
C.av_freep(unsafe.Pointer(&ctx.CAVCodecContext.stats_in))
```

まずはこれで、ctx.CAVCodecContext.stats_inを解放してfreepだからctx.CAVCodecContext.stats_inをnil (NULL)にしてくれる。


来一些评论


このコミット文やと何がしたかったのかがいまいち伝わらないかな
何をしようとした？


Oh, right.


code style requires to have the opening bracket in the new line.


`members` can't be `null` because a few lines up they're defined with `let members = [];`.


`return new HashSet<>(hueFacade.getAvailableLamps().values());` :see_no_evil:


Uhm you are right, I meant it does not need to be passed in as `Time` at `StrictPeriod` object creation, but `@anchor` is indeed converted  to `Time` in both `StrictPeriod` and `WeekPicker` and several `.strftime`s are perfomed around when needed, yet preserving the original `@anchor` "type"

I'm probably missing the point of your original question then :dart:


We have fixed the utilization of the AP as ARP cache. This allows communications:
- between odin STAs
- between the odin AP and the sTA


would better add some debug info if open renderD128 failed.


これは危ないです。
inはgoのmanagedメモリなのでgoのGCによって解放されたらC側でそれが無効なポインターになってしまう。

libavutil/mem.c にあるav_memdupを使う (av_malloc + memcopy):

```go
if len(in) != 0 {
  ctx.CAVCodecContext.stats_in = C.av_memdup(unsafe.Pointer(&in[0]), len(in))
  if ctx.CAVCodecContext.stats_in == nil {
    return ErrAllocationError
  }
} // else {else自体要らない。freepによるnil/NULLのままでいい}
```


Good point -- that's the reason I wrote the age_in_days method, but I overlooked that when I re-wrote the query. I updated claims_to_remind with the code you recommended (see the next commit: https://github.com/rachelcolby11/Wrapt/commit/d46c883b84ce507ebfd3770d4b6c764f89812e5d).

I think switching the database to Prostgres will take more than changing the config/database.yml file because I had been letting Heroku handle the production database -- my understanding is that Heroku automatically overwrites the database.yml file which currently indicates a SQLite database for every environment, and runs Postgres instead. Since I'd been doing it this way, I didn't have Postgres installed locally.

I downloaded Postgres from postgresapp.com and tried to install it but have run into some issues trying to configure it -- specifically configuring the command line, which is needed in order to install the pg gem..... However, now that I think about it, perhaps I can bypass that step, since I think the pg gem is already installed -- at least it appears that way from looking at the gemfile.lock. I'll work some more on it tomorrow and see if I can get further. Many thanks for taking the time to look at my code and help me improve my app!


You forgot to add the image itself:

```bash
$ git add views/asset/img/petitOctocat-125x125.png
```


Let this class be implemented by TrayController


@dfeyer Can you please explain the reason for this?


ok! Fixed now with a new commit


[R/get_average_nltt_matrix.R:13:3:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt_matrix.R#L13) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:51:28:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt_matrix.R#L51) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
    nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                           ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:57:23:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt_matrix.R#L57) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
    stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                      ^~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:79:9:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt_matrix.R#L79) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  xy <- get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:58:11:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L58) *style:* **Put spaces around all infix operators.**
```r
      xlim=c(0, 1),
         ~^~
```
[R/get_average_nltt.R:59:11:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L59) *style:* **Put spaces around all infix operators.**
```r
      ylim=c(0, 1),
         ~^~
```
[R/get_average_nltt.R:70:30:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L70) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
      nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                             ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:76:25:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L76) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
      stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                        ^~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:88:12:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L88) *style:* **Put spaces around all infix operators.**
```r
        col="grey",
          ~^~
```
[R/get_average_nltt.R:89:13:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L89) *style:* **Put spaces around all infix operators.**
```r
        xlim=c(0, 1),
           ~^~
```
[R/get_average_nltt.R:90:13:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L90) *style:* **Put spaces around all infix operators.**
```r
        ylim=c(0, 1)
           ~^~
```
[R/get_average_nltt.R:101:9:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L101) *style:* **Put spaces around all infix operators.**
```r
    xlim=c(0, 1),
       ~^~
```
[R/get_average_nltt.R:102:9:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/R/get_average_nltt.R#L102) *style:* **Put spaces around all infix operators.**
```r
    ylim=c(0, 1),
       ~^~
```
[tests/testthat/test_get_average_nltt_matrix.R:73:45:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/tests/testthat/test_get_average_nltt_matrix.R#L73) *style:* **Put spaces around all infix operators.**
```r
    "((AE:1,BE:1):1,(WD:1,YD:1):1):5);", sep="")
                                           ~^~
```
[tests/testthat/test_get_average_nltt.R:5:45:](https://github.com/richelbilderbeek/ribir/blob/cff1581b7387302b34fa7642fea664f2bb2d9974/tests/testthat/test_get_average_nltt.R#L5) *style:* **Put spaces around all infix operators.**
```r
    "((AE:1,BE:1):1,(WD:1,YD:1):1):5);", sep="")
                                           ~^~
```


Don't provide users the ability to set the title or the message type.
Title should always be the script - Message type should be differed by the method being called (notifyUser, warnUser etc.)


Test comment on a commit


<在标题栏仅显示文件名>
嗯，这个是对的，现在6.9的翻译<只在标题栏显示文件名>
其实还是有歧义的，那标签栏不显示了？
我提这个主要是title bar 和 tab bar差别比较大
是的，试一下功能比较好，我提这个我看了英文的还试了几下才敢提的~~~
感谢你维护这个翻译！


I tried $scope.$apply() instead of it, but it doesn't always work properly. Sole update of $scope.classes doesn't update page view


Please add this attribute to `doc/report_event.conf.txt`


you apparently need a `# pragma: no cover` comment, just after the colon, so coverage isn't measured for that line


We need pip to install dev-requirements the line below.


I think you forgot to commit a file: fatal error: lwip/ip_addr.h: No such file or directory


Commit message "dva manjša popvravka" je precen neuporaben (a priznam, da tudi sam kdaj pa kdaj napišem kaj takega).  Če prav vidim, je popravljeno zamikanje, torej bi lahko pisalo "dokumentiraj funkcije in popravi formatiranje kode". Potem vsak takoj ve, da se ni spremenila vsebina.


:P not that great and not used tbh


if `output` is always expected to have a length of one, why not just log an error and return in line 44? That'd be way more obvious to us and the user what happened.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18629](https://teamcity.mesosphere.io/viewLog.html?buildId=156647&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:11


configuration of kexec hardboot


Glad that you solved it. Thanks for awesome piece of software!


is this going to be ok with mym launcher encase some trys to install via the options?


and what about this?
https://github.com/elibyy/tcpdf-laravel/blob/master/src/ServiceProvider.php#L29
https://github.com/elibyy/tcpdf-laravel/blob/master/src/ServiceProvider.php#L41


this is so much simpler isn't it!?


unneeded statement


IMO `fix_structure` -> `fix` would simplify readability


Can be final, right?


of course it's a lint result, why not simply name it `result`? The lint doesn't really give any info on top


@dmmiller Thinking long term about this, is there any kind of integration test that could potentially pick up on these kinds of bugs in future? It kind of freaks me out that the Native --> JS interop could just blow up like that. :disappointed:


Codestyle: Space after `for`.


Right, but it should be `.build/venv/bin/pip` and not `.build/venv/pip`. See the line below. ;)


Please add comment stating that the 'reproducible' file is used by functions from `problem_data.h`.


fine for me. I've just brought them back


What if the script crashes or something else crashes? Detect what items have been added, and add that arraylist into a hashmap, so you can simply remove them once something happens.


This is a hilarious bodge :+1:


I'd remove this empty line since it separates the else from the if where it belongs to


From the Slack chat:

> Query parameters should not be part of the cache. `$uriPath` refers just to the `/path/without/parameters`
> if `storeResolvedUriPath()` is called with query parameters that would be a bug


https://github.com/Nukkit/Nukkit/blob/e6df291f518b06e3e59b78c950d03edec41f54a2/src/main/java/cn/nukkit/Player.java#L2478-L2484

@MagicDroidX please revert this


Isn't this method also already in the TrayController?


空行いれよう


Dgdgdsgdsg


`otherClass.getInstance().actionPerformed(e);` will work here


actually I'd find it easier to read if you do:

```python
if 'fix' not in result:
    diffs = None
else:
    ...
```

this way the else is closer to the if and can be associated with it easier making the code equivalent but easier to understand :)


Codestyle: Whitespace at the end of the line.


For some reason these files haven't been uploaded here. So strange.


what a commit !


kdsfasdfds


A new line is required before Credits.


use sizeof (tempKey)-1 instead of 128


Maybe we should consider allowing `entry` and `account` options here as well. That way we stay backward compatible. Let's also update the [inline documentation](https://github.com/codevise/pageflow/blob/master/lib/pageflow/seeds.rb#L138) for this method.


use more constants :3 like RC2_KEY_SIZE here


actually I'd use `"{class} ({rule})".format(self.__class__.__name__, result['ruleId'])`, this way the rule is shown to the user natively and he can ignore the bear with the actual rule :)


aタグをlink_toに変えれてていい感じ！
あと=とlink_toの間に半角スペース入れてあげると完璧！


can you remove this line? If not something's wrong with eslint IMO.


Looks good.


Did you evaluated the Apple RC2 version? Seems shorter than this one http://www.opensource.apple.com/source/CommonCrypto/CommonCrypto-55010/Source/RC2/rc2.c


this has come a long way and its awesome! For now it needs work


Nope, not with the context of getFoo


Let's also match "KB/s" here


It seems we are seeing this bug again on PHP 7.0.3.  The following passes on 5.5.21 but fails on 7.0.3:
`    public function testSimplexmlLoadString()
    {
        $callResult = simplexml_load_string('<c><x/></c>')->c;
        $actual = json_encode($callResult);
        $expected = '{"x":{}}';
        $this->assertEquals($expected, $actual);
    }
`
I'd appreciate any insight on this.  Should the original bug be re-opened?


We also need to update the other tests in this file. I think they might be failing for the wrong reason now (unknown `entry_id` param instead of authorization?). Let's also add a positive case. That way it's a bit more clear that the others really do not add memberships because the signed in user is not allowed to and not just because the controller actions do not work. ;)


Wouldn't it be better to directly use an XML parser here?


Should be fixed now.


yes


Hello.
Just once more time checked descriptions and added some changes in https://github.com/hermitblue/lp_ramdisk_dorimanx/commit/6601d38693a322856397593c2aadced42eb2458c if you like to take a look.
I have a question: Why did you changed min to Min in some places? All document have that logic: Maximum short Max
and minimum short min
Now I think that XML file is completed. Final fixes are now up to you.


i want to commit


@kanerogers we do have tests around this.  The problem in this case was that we had so few places using promises and on iOS they had made one assumption and I hadn't realized it until I dug further.  I think we actually have a lot of test infrastructure in place that tests the Native -> JS interop.  This was just a new piece and I made a mistake when getting it working initially.


@Zangetsu38 could you post the callstack from visual studio? Just run a simple demo like gs_gcm_triangle from within visual studio and share the callstack


When not using ssh-agent and specifying an ssh key on the commandline when creating a machine, this works.
Afterwards, this breaks all commands that need to ssh to the host (like `ssh`,`upgrade`,..) since the copied `id_rsa` key is no longer loaded.


@apeyser You are right, I hope the improvement I added right now addresses your concerns.


Isn't `super` less confusing since it immediately tells you it's from the superclass?


Between "ns" and "ms" for milliseconds here, let's match microseconds with "µs" too


Alternate system set fixed/absolute in js too :). (or use js/no-js)


Ja, hast schon recht, ist bequem aber ineffizient. Ich mach mal ein Issue dazu, dass zu ändern. Würde aber zuerst alles in den Master mergen, sofern es sonst einigermaßen ok ist. Sprich, ich würde solche Sachen eher als separate improvements betrachten. Aktuell gibt es irgendwie zu viele Abhängigkeiten zwischen den branches...

> Am 07.03.2016 um 01:44 schrieb bjoerntm <notifications@github.com>:
>
> Ok, das als vorläufige Lösung für den Test zu haben ist für mich völlig in Ordnung. Aber am Ende sollte man da denke ich ein striktes Paketformat haben, im Sinne von "die ersten 128 bit sind die Session ID" und so weiter. Wir wollen schliesslich Bits sparen ;-)
>
> —
> Reply to this email directly or view it on GitHub.
>


With the latest esp-open-sdk, this file is found in esp-open-sdk/esp-open-lwip/include


how does on specify the location of the os?


upppps :imp:


For consistency - though we wouldn't expect such low frequencies, at least for the GPU itself - let's match "Hz" here


"Hombre" (H) and "Mujer" (M) are more apropiarte and will be consistent with other occurrences.
I suggest accepting the change here.


It's in the README: https://github.com/coolcrowd/control-ui#configuration


Upps, weil du das Emoticon in der Commit Message vergessen hast? :hammer:


Why use underscores for variable names?  Isn't this only used in class context where you want to make these variables hidden/private?  So please don't use in this context, cause they're local variables


Not really. We test for py 3.3 - so this line is covered fine


but in the container? I don't build the container


It's better not to use ActiveRecord models in migrations. The shape of `Membership` (i.e. declared associations, validations) might get out of sync with the db schema at this point of the migration history. Trying to run this migration might then fail, because certain attributes are only added by later migrations etc.

Let's `execute` raw SQL here  like in [this example](https://github.com/codevise/pageflow/blob/master/db/migrate/20150826125745_insert_main_storylines.rb)


`datetime.now` is non-monotonic and is affected by clock updates - use `time.monotonic` instead?

Unexpected things are going to happen here when the clocks change :P


I suggested this as it gives a traceback which is useful for debugging if it ever happens.


Also it's more efficient to update all memberships in one `UPDATE` statement instead of each individually.


Just realized this isn't idempotent because of the requirement on a single item in the channel. Might be a better idea to just have a close-edge-poll channel that we use to notify all instances of the function, and then check the `v.Ready` value to verify if we're connected or not.


git clone git://git.freedesktop.org/dbus/dbus/neo20067*/


That isn't a style object prop, so the original description is better.


Needs to be removed as eslint should never give a decode error


Eventually it would be nice to store better state about the connection (Connecting, waiting for session, connected, etc) vs just a couple bools on the struct. For now this seems to work OK.


Removed the super.


Good call, I'll change that (and any others)...


please more xxx.


I was shy to say to avoid cache flooding … because it’s not a correct protection the problem for cache flooding still exist. In our case we need this because we have a search engine with tons of facets … so we create 100 cache entry every seconds … and more or less soon redis will use the full memory of the server

The current log of our server for a few milliseconds
```
1457432561.818940 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:6ef3dda8f18511196b300057ee8071e1" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Energies+renouvelables+-+solaire+-+photovolta%C3%AFque"
1457432561.823296 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:1d82428e617f3b5669c797121f4cb95c" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Domotique+-+Automation"
1457432561.828205 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:dbdcb143eed84dee8f9627c869474f20" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Sanitaire"
1457432561.834072 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:2a13e715babed34bc0f97567583f5c1b" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Cuisine"
1457432561.839482 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:dd831500d2e58bac978fbe55cc837a58" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=D%C3%A9tection+et+protection+incendie+-+S%C3%A9curit%C3%A9+-+Surveillance"
1457432561.844850 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:0c0a98f080c513837476bf0c4d5ae750" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Ing%C3%A9nieur+%C3%A9lectricit%C3%A9"
1457432561.849297 [8 127.0.0.1:35694] "SET" "Flow_Mvc_Routing_Resolve:entry:76a83826e9bdffd365f21b44f3998086" "fr/rechercher?filter%5BbyActivity%5D%5B0%5D=Chauffage-Ventilation-Climatisation-R%C3%A9frig%C3%A9ration&filter%5BbyActivity%5D%5B1%5D=Cuisines&filter%5BbyActivity%5D%5B2%5D=Electricit%C3%A9+-+Tableaux+-+T%C3%A9l%C3%A9communications-+Lustrerie&filter%5BbyActivity%5D%5B3%5D=Luminaires »
```


ditto


`len(lru.cache)` and `PushFront` should be inside lock as well


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2363 of 2398 tests passed, 0 test warnings)

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


Ah, thank you!


If we end up with a suffix for each 10^0, ^3, ^6 and ^9 order magnitude, for each unit, we can simplify this special case by scaling from MHz into Hz and then no need to modify the giga/mega scale factors.


And i presume you want the value to be in MB ==> you can execute the command 'du' to return in MegaBytes
du -B 1M   --> this will return the size in MB
Read the manpage if you need anything else


I don't understand. You have to build the container, otherwise you won't be able to run it.


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


This breaks a case where the id_param is missing - previously it'd return `nil`, now it returns `""`, making https://github.com/ryanb/cancan/blob/master/lib/cancan/controller_resource.rb#L67 true.


Отлично поработал.


The `sleep` is no longer needed since we've switched to Diego.


but the unused bits are really 0!


Let's also handle kilo/milli here


where download jsonapi1.9 .jar ?

thanks


Awesome to hear - just wanted to make sure! Glad to hear you guys are on
top of things. := )

On Tue, Mar 8, 2016 at 9:32 PM dmmiller <notifications@github.com> wrote:

> @kanerogers <https://github.com/kanerogers> we do have tests around this.
> The problem in this case was that we had so few places using promises and
> on iOS they had made one assumption and I hadn't realized it until I dug
> further. I think we actually have a lot of test infrastructure in place
> that tests the Native -> JS interop. This was just a new piece and I made a
> mistake when getting it working initially.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/facebook/react-native/commit/c9a1956c4fd9cfb6bf883c835b501d097181e1a4#commitcomment-16558073>
> .
>


you can remove innermost `Header.`


Good use of partials and layouts.

I also like the commit message here, as you listed a succinct summary of the changes ("refactor layouts to use partials") and you gave a reason for those changes ("so timerecords_validations/show can have different navigation links").


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18630](https://teamcity.mesosphere.io/viewLog.html?buildId=156648&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:07


> Hmm, I tend to prefer to leave it fail (that is, no < dep) and have an open bug blocking the tracker: that way, ffmpeg-3 wont be unmasked until it is fixed

There's a [bug](https://bugs.launchpad.net/mixxx/+bug/1530484) on the mixxx bugtracker. Let's wait until they fix it.

> and what about libav then ?

Can we add libav support later if somebody really needs it?


done


If you are going to try to say 'its easier" remember, it should be "it's easier". The apostrophe in it's indicates there is a letter missing - the i in is. So either use it's easier or it is easier but NOT its easier. Sorry to be a pedant but grammar mistakes can make material look sloppy ;-(


use NumericalPoint::normSquare to compute the square of the l2 norm


idem


`inject_float` is the name of the function in the existing module intended to be replaced


As per our email discussion CSS classes should use a dash instead of an undescore. The inconsistency here ("form-control" next to "last_field") is the most jarring here though.


that is a mistake - fixed


tableNames - будет создаватся при каждом использовании метода


FieldNode implements Variable, which defines
    boolean isClosureSharedVariable();
    void setClosureSharedVariable(boolean inClosure);
So... yes... it is unused.. Not sure Depretaed makes sense for a method from an interface, without making that interface method deprecated... only that the interface method is used


Oh, and don't feel the need to fix that right away - create a ticket for that project-wide refactoring.


you could write pValue += 1.0 to explicit the fact that's a float


What is the purpose of moving this change?


- En fait, dans `coefficient_proratisation`, je ne comprends pas la distinction entre `heures_remunerees_volume` et `forfait_heures_remunerees_volume`...

- Dans `plafond_securite_sociale`, que veut dire `heures_non_remunerees_volume` ? Congé sans solde, maladie ? Dans ce cas le plafond ne devrait pas être proratisé ([source](http://www.legisocial.fr/actualites-sociales/755-proratisation-du-plafond-de-securite-sociale-les-regles-connaitre.html)).


Shouldn't the titles be different?


why use all these temporary variables ?


Indeed! Fixed it and a few other errors. It should be good now.


For some reason I'm still seeing `EntityReference`s in `CallExpression`s.


instanceof - достаточно ресумкоемкая операция


как на счет switch ?


done


Had to remove the double quotes.
Otherwise this condition  `if %2=="" ( ... )` is never true, since setting a variable as without value just reset it, as: `>set RABBITMQ_NODENAME=`


作者用了async ，.net4.0 vs2010根本没法使用，怎么办？直接使用你生成的CommLib.BdPush.dll吗？


Where is the github issue for the 'signParams' change? I'd like to understand it better.


How do we make use of this metadata once it's in Atlas? Can I use it in my Packer build (eg within the AMI description?)


There is no point in catching this exception, or in telling its name to the user. That means that `raise F…` is better implemented as `assert false`. Only use exceptions when there is a point in catching it.


I think pid in amcodec is used for TS playback, so it shouldn't affect, because in Kodi it plays only video.


Thanks, test extended.


Rather than `cf-push`ing on every run, we could use something like `cf apps | grep graphite-nozzle` to only push the app if it's not already deployed? It's really not good to be pushing on every deploy as it'll restart the app and we'll have metric gaps.


It is declared in validationinterface.h


Good spot :)

The title is hardcoded to "Try MiniMesos.org" so I'll change it and update branch


dfgf


asdgfg


[R/get_average_nltt.R:88:12:](https://github.com/richelbilderbeek/ribir/blob/c4ba357b90723fcd9f18d00382c91a3973bc3bd5/R/get_average_nltt.R#L88) *style:* **Put spaces around all infix operators.**
```r
        col="grey",
          ~^~
```


ahh, so it never ran. I thought that was the case, but did not see it. Good catch.


Below you use `abort_path(@file)`. Why in this line `validate_path(@file)` is not used?


Brady I am able to solve Calendar issue here are details

interface/main/calendar/modules/PostCalendar/pnuserapi.php line 478 and 502 template variable "interval" being assign two times and this interval variable is used to properly position events on calendar views

I think the second assignment on line 502 should be "intervals" if it is being used

Second issue related to user icon not showing, I changed "$TPL_IMAGE_PATH" to "{$this->_tpl_vars[TPL_IMAGE_PATH]}" and now user icon is properly displayed

https://github.com/practiceprovider/openemr/commit/3301613f0b98f05e24a20dea7306fd68193f19a4


I do not see how these cases do the right job


太屌了


To download latest episode of a series 1080p HD


I assume this script has been grabbed from somewhere online, would be nice to have the link in the commit message for future reference.


наименование констант принято делать в верхнем регистре


Why did you remove all the capital letters? “NaN” is written “NaN”, seriously. And the beginning of sentences is easier to recognize with one, too.


I think the `spec.triggers.strategy.rollingParams` [here](https://github.com/danbev/mbaas-monitoring/blob/44ea352fb87184797923609965d9680b0841412d/nagios.yml#L61..L64) should be removed, since it's using a recreate strategy rather than a rolling one.


好吧原来这样不行


This fix works for my julia 0.5-dev. After changing the source it no longer produces the error of locale utf-8!!

Thank you!


Made a PR to rename the cura_app.py on Linux: https://github.com/Ultimaker/Cura/pull/654


Thanks for clarifying!
For the record: So `Route::getResolvedUriPath()` indeed returns the path *including* query parameters. But I think we could/should skip them in the cache entries altogether.

I just realized that `RouterCachingService::buildRouteCacheIdentifier()` seems to ignore query parameters already, so this might be changed as bugfix even!


ccc


ddd


I removed other occurences from codebase with:
https://github.com/facebook/presto/pull/4736


These lines, not needed?


@danbev @david-martin What's the purpose of the additionel provider label?


Blev någon merge konflikt, för jag råkade skriva in i player.js. men är rätt säker på att jag löste det utan problem och tog bort det jag skrev! Hoppas iaf, se gärna över så att jag inte har pajat något.

Request:
model.getPlayers();
model.getMap(); //path to src
player.image;
där image:
    image.alt = "text to be shown if the picture fails to load";
    image.width = "width in em";
    image.height = "height in em";
   image.src = "path to image";


Yeah - there is an annotation and model object with same name.
I will name annotation just `@LongVariable`


@willdean I just remembered. This string should be in the Resources file. You should be able to use that here without problems.


@dcam0050 Can you please add write permissions?


`dev_releases` is only mentioned here - should it be hardcoded?


Is this function only called on the master? If so, this is probably OK. If not then we probably want to avoid this explosion of Db calls from a slave since the RPC overhead will be high.


[R/get_average_nltt_matrix.R:13:3:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt_matrix.R#L13) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:51:28:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt_matrix.R#L51) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
    nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                           ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:57:23:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt_matrix.R#L57) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
    stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                      ^~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt_matrix.R:79:9:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt_matrix.R#L79) *warning:* **no visible global function definition for ‘get_average_nltt_matrix_impl_1’**
```r
  xy <- get_average_nltt_matrix_impl_1(phylogenies = phylogenies, dt = dt)
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:70:30:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt.R#L70) *warning:* **no visible global function definition for ‘get_phylogeny_nltt_matrix’**
```r
      nltts <- c(nltts, list(get_phylogeny_nltt_matrix(phylogeny)))
                             ^~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/get_average_nltt.R:76:25:](https://github.com/richelbilderbeek/ribir/blob/4779a7aeb4c91634c1ad17079670524fe4ab86d8/R/get_average_nltt.R#L76) *warning:* **no visible global function definition for ‘stretch_nltt_matrix’**
```r
      stretch_matrix <- stretch_nltt_matrix(nltt, dt = dt, step_type = "upper")
                        ^~~~~~~~~~~~~~~~~~~
```


This looks like a good improvement. Why are we making `op` an optional argument? Looks like we'd always want to call with it.


We didn't discuss this enough during the meeting, so let me comment on this further here.

Adding filters to the application controller is potentially dangerous, as they will trigger for every controller action you have right now, as well as the ones you write in the future. Putting authentication there (i.e. `before_action :authenticate_user!` line) is probably safe, at least as long as most of the application requires user to be logged in.

In a way the application controller code sets up a "norm" which most controllers follow, while some exceptions can bend it. That is the case for your `make_user_complete_uploade_file_validation` filter - you set it up in the application controller and then skip it in `TimerecordsValidationsController` and devise controllers. This is OK, but only if completing file upload is really so important that it should interrupt anything else the user may wish to do.

What I'm saying is not that this code is bad, but you have to be aware of the implications. Every new controller you create will abide this "interrupt user if file validation wasn't completed" rule. Alternative is to put the filter only in controllers that deal with file uploads. This would make the rule specific to that part of the application and no "skip_filter" calls would be necessary anywhere else.


Thank you both. I've sent https://github.com/emberjs/ember.js/pull/13064 to address this.


A typo: "uploade" instead of "uploaded".


I think these are quite good but it would be better to have each exercise as a separate commit in case the maintainers only want to implement/include one of them?


Perg replace reverted where 'e' modifier not used
https://github.com/practiceprovider/openemr/commit/3a58d4a544461e9b73af10e630830e0c306a4359


whats this?


Now looking at TCPDF stuff


@AnthonySteele the version number of the latest is 5.2 not 5.1, I believe


If there isn't an upper limit of 10 characters for the nagios password, I'd suggets a longer one.


We have class hooks depending on the state of the upload and this broke it. Calling the progress callback *after* complete seemed odd anyway? Complete should be the *last* callback, shouldn't it? Is there an objection to this?


OK. Please see update here: https://github.com/jembi/bsis/commit/7d4d3ac72ed661debe7ddea5d4eec6d4c796a389


I don't think you are using this function. But I think it's a dangerous function. If the programmer wants an `abstract_float` that contains f, the programmer can request that from the start with `inject_float`, not make an uninitialized `abstract_float` and then change it. And `insert_float` is dangerous to use on any `abstract_float` that is not currently uninitialized.


Note that table defaults are overwritten by the defaults in the Entity. They are included here for completeness.


## This is a markdown file


#push without using alias


Please update your puppet module on forge.puppetlabs.com!! This error is still in that module (0.3.0)


It's listed under here:
https://github.com/TTLabs/EvaporateJS/issues/115#issuecomment-193656957

Basically we create a token for security (as mention in the 1.0.0 PR) which needs to be sent back to all sign requests. When trying to resume an upload, the token differs from the original request and we don't sign the request as a result. By sending back the *original* token, the request can be signed.


I don't understand the question. ``timeout`` is indeed an optional property of ``appx.AlertMessage``.


```
jenkins:
  skip: '.*skip.*'
```


the whole point is that you don't build the container yourself, for the usage look at the compose file [dev-pybossa](https://github.com/coolcrowd/object-service/tree/docker/image/compose/dev-pybossa) or [dev](https://github.com/coolcrowd/object-service/tree/docker/image/compose/dev).


TeamCity jonas-requests :: WindowsBuild [Build 18](https://build.neohq.net/viewLog.html?buildId=568573&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Then I guess I will implement it in the same way...

Thanks


updated with latest data


`Int64.bits_of_float f` must be masked with 0x800fffffffffffff here, or instead, `reconstruct_nan` should have been `Int64.logor 0x7ff0000000000000` in place of `Int64.logand…`


I really wish that I could have been included at the design stage of this module, because there's a number of potential issues here that could have been avoided if we had talked first:

1. Using environment variables (or `process.env`) to store secrets is a backwards-compatibility hack. We do this for legacy apps via `vault_to_env_js` and a `Secretfile` because that allows us to avoid converting lots of legacy code during the migration. But strictly speaking, it's a very bad way to use vault, because it means we can't honor the "time-to-live" values for secrets. Every secret in vault has a TTL value, and once that TTL has expired, we're supposed to refetch the secret. For use with `vault_to_env_js`, we're forced to set all TTLs across vault to 30 days, which is a terrible idea for things like dynamic AWS credentials. A correct in-process API would handle TTLs correctly (I can talk about designs for this).
2. This library appears to do raw HTTP instead of using the [node-vault](https://github.com/kr1sp1n/node-vault) package, which means that if the Vault authentication scheme changes again (which has happened once), we can't just update to the latest client, but we'll instead need to maintain our own HTTP bindings.
3. This library does not honor `VAULT_ENV` correctly, which is necessary for handling development vs. staging vs. production.
4. Given two related secrets `service/foo:username` and `service/foo:password`, this library appears to make two separate HTTP calls to the vault server. This will fail badly for dynamic secrets, which we want to migrate to as soon as possible.

My recommendation: If you're going to access vault from inside your node process (which I think is a great idea), then please consider using `node-vault`, handling TTLs and dynamic credentials properly, honoring `VAULT_ENV`, and setting aside an hour sometime so that we can thrash out any further design issues in detail. Doing so will make our vault migration much smoother, and it will help us avoid becoming permanently tied to hacks which I only introduced to ease the transition, and which I was hoping to remove in the future.


`&*` has no effect. It just gets a pointer of the dereferenced pointer, which is the original pointer again.


updated with latest data2


I see this was changed anyway in a more recent version...


Giant massive NACK.

What else is `utilities.is_executable()` for if not this?


@aishmita-aggarwal we should always use methods with `!` for omitting any errors with validation and so on.


and what happens when we get here with no methods?


-integrazioni effettuate da Emanuele Gugliandolo


Yes, of course. It's done now :smile:


Yes, for instance when you log in/out, when saving a document fails, etc.
As for now a grey shade appears behind the error message box but vanishes before the message automatically closes (the timeout are not the same).

To be honest I am not very found of the grey shade but at least this change makes sure it is synchronised.


Nice one. Thanks for mentioning in the commit message, it will be useful for me as well.


Bug fix was in Step5 class.


ack


Can I do something more? Install de bundle and setting the configuration,
for example?

2016-03-08 9:33 GMT+01:00 David Romaní <notifications@github.com>:

> if the tests are working, for now it's enough
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Flexible-User-Experience/hidraulica-disseny/commit/e5a94315ee20e80c885147e937eb38e10b0f18b3#commitcomment-16555904>
> .
>

--
Anton Serra Torta
616.50.13.32


@s7design FYI.


These last 4 lines should've been taken out of the `if` and `else`, since they are the same in either case. It just needs the `category` set.


Capybara's #find keeps looking for the selector in a loop until it appears on the page or the timeout occurs. So in theory this shouldn't help. Unfortunately I don't know a better way to solve this, though.


[SI-H-_Adjie Moekti Noer Rahman_155150400111126.docx](https://github.com/AdjieMN/Tugas-Bab1/files/163043/SI-H-_Adjie.Moekti.Noer.Rahman_155150400111126.docx)


Can you explain what's going on here?


Does this mean there won't be any new releases from puppetlabs for geppetto?


What's the benefit of having multiple param lists for the defined methods?


I guess this change should be mentioned in the readme


Do you mean through a `rescue` statement?


@chris-allan Does that address the issue you mentioned with settings.PAGE in the function def? Not sure if I understood you quite right?


this is new project


hmmm?


take care of the indent


render する際に、render 先のテンプレートがなくて以下のようにテストが落ちていて、
```
Failures:

  1) EventsController GET #new ログインユーザがアクセスしたとき ステータスコードとして200が返ること
     Failure/Error: get :new

     ActionView::MissingTemplate:
       Missing template events/new, application/new with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :coffee, :jbuilder]}. Searched in:
         * "/Users/task/projects/perfect-ruby-on-rails/app/views"
     # ./spec/controllers/events_controller_spec.rb:17:in `block (4 levels) in <top (required)>'

  2) EventsController GET #new ログインユーザがアクセスしたとき @eventに、新規Eventオブジェクトが格納されていること
     Failure/Error: get :new

     ActionView::MissingTemplate:
       Missing template events/new, application/new with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :coffee, :jbuilder]}. Searched in:
         * "/Users/task/projects/perfect-ruby-on-rails/app/views"
     # ./spec/controllers/events_controller_spec.rb:17:in `block (4 levels) in <top (required)>'

  3) EventsController GET #new ログインユーザがアクセスしたとき newテンプレートをrenderしていること
     Failure/Error: get :new

     ActionView::MissingTemplate:
       Missing template events/new, application/new with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :coffee, :jbuilder]}. Searched in:
         * "/Users/task/projects/perfect-ruby-on-rails/app/views"
     # ./spec/controllers/events_controller_spec.rb:17:in `block (4 levels) in <top (required)>'

Finished in 0.61889 seconds (files took 2.93 seconds to load)
19 examples, 3 failures, 1 pending
```

`app/views/events/new.html.erb` という空のファイルを作ったのですが、本には作るようなことは書かれていないです。
とはいえ、今後は必要になると思うのでコミットに含めました


You might also want to update the "expect(waitingThread wait(0.05), is false)" on line 54 and add a Thread Yield() above


TeamCity jonas-requests :: WindowsBuild [Build 17](https://build.neohq.net/viewLog.html?buildId=568479&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 31857, ignored: 117 Build time: 2:2:31
### Failed tests
```
org.neo4j.test.ha.ClusterTest.testClusterWithWildcardIP: <no details avaliable>

```


Haha. How do you know about this?


I changed the extension to locate the user in AD in different location of CN = 'user' + base_auth_search .

I modified the search attribute givenName for Uid , and did the search in AD by sAMAccountName field, and starting this , I took your real address with the function getNameInNamespace

I believe that getuid and getNameInNamespace properties that I used , are present in old Zimbra verções , I am using the summer 8.6.0_GA_1182 (build 20,150,907,121,026 ) which worked perfectly .

In AD I'm using Samba4 con Zentyal 4.2.1.3 .


The reservation is built just before – why don't you add the labels there? I understand that setting the labels is nicely encapsulated here, but reservation labels by definition are a part of the reservation, so I wouldn't split creating the reservation and adding the labels.


Musicians


same as above – create reservation with labels instead of 2 steps?


unack (testing)


As much as the chances of 2 processes having the same pid.


ready


If we had swiftlint enabled it'd moan about no new line at the end of a file.