The `unsupported` options doesn't do anything on its own, it only enables or disabled certain combinations of configuration options. "Enable" means that combining certain configuration options is either a configuration error, or is allows. Most likely, this change just breaks the build.

For example, combining the C++ interface and parallel (MPI) interface is disabled by default. The reason is mundane -- the C++ interface doesn't give access to the MPI functionality, there's nothing broken about it. Since one can use the C interface from C++ just fine, this is not a problem at all in practice.

What you need to do instead is to find out what breaks with Intel. I assume it's the HDF5 C++ interface. So you might want to disable C++ by default (I wouldn't do this). Alternatively, disable it if an Intel compiler is used. However, before doing this, we should check how common the problem is -- does it occur with all versions? With recent version? On all architectures? Or is this a one-off problem?

I wouldn't disable things without keeping note, in the source code of the package, why this was done, so that -- once the C++ Intel compiler works fine -- we can re-enable things.


a little inline comment in the code to generate a bit of API doc about the feature would be welcome if you don't mind


Ah, good catch. I thought there was probably a reason. I'll remove the TODO.


can you help me set mine up?


A sticky problem here and many other places is that browsers like to cache .js files.  So when they change the confusion can be monstrous.  I suggest that the "src" URL end with a parameter like "?v=nnn" (generated by PHP code) where nnn is a number taken from version.php.  That way when nnn changes, the browser will always load the new version.

Perhaps you have another idea to deal with it.  But with the number of .js files increasing, it becomes vital to deal with it in some way.


This can be done without needing Slice: `new String(response.getBytes(), UTF_8)`


stl-string is only 4 bytes on Linux, but it's 28 bytes on Windows - either this structure changed significantly since I added it, or that first field is actually a pointer to another struct.


modification a faire:
ligne dans la class StorUser DANS SERVERREQUEST.JAVA
dataToSend.add(new BasicNameValuePair("arrondissement", user.arrondissement));


It looks like this doesn't get unset when the webview location changes (by clicking a link in the page or by entering a url into the urlbar). Flagging this as a security issue.


Closes #1.


I like the suggestion although having the local offers some debugging convenience.


@aprokop The classes in xpetra/sup have not forward declarations like the classes in Xpetra/src (the fwd files are located in xpetra/src/Utils/ForwardDeclaration). To use the Xpetra shortnames one usually needs to include the corresponding fwd class in the MueLu..._decl.hpp file in order to guarantee that the #define XPETRA_XYZ_SHORT flags are defined. It could be that depending on the configuration the define flag for Xpetra::IteratorOps is set somewhere else by nested header inclusions.
I propose to generate and add forward declaration files for the classes in the xpetra sup library, too. They should not be in the same folder as the forward declaration files of xpetra to keep the two libraries distinct.
Anyway, your commit should fix the compilation problem.


頭が悪いのでtypoした


Agreed, will do!


Since this is based from comes from angular2's ListWrapper, I think we should note that before committing. Or just cut it in favour of the bluebird example, which is trickier in any case (`Promise<T>` has a type parameter but `ListWrapper` does not).


Each file should have its own test file.


To me it looks like the actual execution of the cfg script is not blocking and hence the make script will just start this generation and move on to actually trying to build the cpp file that actually depends on that script being done. But because the process is not blocking make will see the depdendency satisfied because the target looks like it is done.


test


WTF is this? @enricosada @kevinransom seems fsproj is adding this arguments instead of replacing them


Example when this might happen? (call destination?)


I've noticed that the other instance and its definition is under DEBUG. maybe you should make it under DEBUG, too.


@eschnett: if `--enable-unsupported` is just a check option and doesn't even affect the build, it probably doesn't need to be a variant.  Adding it as a variant means it is included as part of the version, but what is actually installed really only depends on the OTHER variants based on your description.  If I am understanding correctly, `enable-unsupported` just determines whether `configure` will let you try to build a certain way.  Is that accurate?  Could we remove this variant?


doh good eye, thanks for catching.


Will do.


esto es lo que diferenciaba uno de otro en el principal


Either that or add call guards.


hasta aquí era igual :P


非常感谢，我仔细看一下。

2016-03-08 12:34 GMT-05:00 Sam <notifications@github.com>:

> 98行有 一个小笔误
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bbtmtl/wds-section1/commit/d4524a7f545f3b8f21d988116d84e52112991fc2#commitcomment-16567779>
> .
>


probably doesn't matter, but it's be nice to avoid making vectors and things for debug instrumentation. I often do this with

```rust
debug!("{}", {
    // .. code that should run conditionally ..
    debug!("foo");
    "dump complete");
});
```

but I think there is a better way.


So what's the protocol for using this? Are you able to somehow call replace on a sequence of paragraphs or a document or something or do you need to narrow it down to a particular paragraph or run?

Also, would it locate something that started in the middle of one run and finished in another?


Wcześniej o tym nie pisałem ale w sumie w takich sytuacjach tego while można pominąć. Szansa na wystąpienie kolizji jest niewielka. Dla przykładu: gdyby przez 100 lat co sekundę generować miliard UUID to byłoby 50% szans na wystąpienie 1 kolizji. Możemy sobie pozwolić na takie ryzyko ;-)


@Shauren Nice, man! Glad to see you starting on this!


Nit: move this to the next line, or move the closing brace up to the previous line. Make up your mind man! ;)


remove the System.out


Why do you reset mocks all the time? You shouldn't be doing that.


do this


What about "Learn how to extend the core objects with your own methods"? I think this is clearer.


`Mockito.verify()` is equal to `Mockito.verify(..., Mockito.times(1))`


@tgamblin If you decide to remove the variant, please remove also `--enable-unsupported`. Building things that are `unsupported` by the developers is one thing. Giving the possibility to build ONLY them is another...


You can reset the mock here. (you're resetting it cause it's an integration test and you're reusing the mock, right? )


Should probably add a comment explaining why we're not using `locationsAreEqual` there; I recall wondering the same thing.


Why are you tracking statics? Its not like this can *work*.


I think that interface doesn't need to be static


I think this should be "page models" without the capitalization.


great commit message :joy:


Adding the test as well, since `SyntaxNode.Contains` did not have any direct test.


Surprised to see RefCell here -- is this just for convenience?


Couldn't you already share it now? I guess you didn't test it yet but that would be more convenient for discussing it ;-)

I suppose this also relates to #148 too, which discusses Programmer Dvorak.


This is a bug. More precisely, `Rvalue::Slice` is a bug,


"If you have one methods" -> "If you have methods"


Yeah that's why i propose we put the control of the table into the users hands. So maybe we'd implement some startup setup system where you can input whether or not you'd like players limited at 65k - or higher.

And thanks for the info about the auto-increment (bug?).


Added also and "id_" class.


"cannot override these default page methods with"?


Might avoid the need to compute from to the final `MovePath` types


you don't need `havingValue="true"`


Fixed


Thanks, will do.


I agree there – I just mean that for the purpose of tests at this level (as opposed to e.g. `LocationUtils-test.js`), it might be clearer as to the intent to do `toNotExist`. For the purposes of asserting that `push` works, the specific representation of "no state" is less relevant.


I tried doing the same thing (keeping a reference to the callback in a module-scope object) for `certificate-error` but it kept crashing the browser when I tried to call the callback later. luckily that doesn't seem to happen here.


put it to @Before or @After section of the test


This test fails....
The reason is because in your file you tell it to raise a IndexError. Which is awesome. Great idea..
The way to make this assert statement catch that is by doing this..
```    with pytest.raises(IndexError):
        linked_list_instance.pop()
```
Note, you will need to add ```import pytest``` at the top of your file. Which you should be doing anyways.


Could you check if we need a json reponse?


Can you verify that I wrote the docblock correctly?

Closed by a1493f2c85:

> Changes for pills.php


I don't think this proposal is so settled yet. Function.prototype.toString()'s specification is changing based on a proposal currently out for review, and that proposal is not yet at Stage 3. Maybe this test is at the intersection of all of those possibilities, however. @michaelficarra


I am not sure, if the escape is necessary here...


Closed by a1493f2c85:

> Changes for pills.php


You're right, i'll fix that. I'm testing the package before i'm publishing any versions.


Maybe we could let the user pass the value as an argument in this example.


I suggest you use http://joel-costigliola.github.io/assertj/ - once you use it you never want to use anything else :P


по Стрибогу замечаний в принципе нет


This comment isn't applicable here


'id' attribute is missing


@alalazo: `--enable-unsupported` doesn't actually enable anything.  It skips checks in configure, so it's not causing any problems per se.  It *allows* me to build `+cxx+mpi`.  Right now we have this situation:

* `+cxx+mpi+unsupported`: builds.
* `+cxx+mpi~unsupported`: fails to build at all.

The variant doesn't do anything but cause an error if you try to build with C++ and MPI.  I think adding `unsupported` as a variant adds an extra unneeded bit to the version


That change will cause a big memory leak. I’ll see what I can do.


The following doc(s) have been deployed to the [1434upstre](http://1434upstre-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/organizations/adding-a-custom-upstream](http://1434upstre-static-docs.pantheonsite.io/docs/articles/organizations/adding-a-custom-upstream)


Fixes #27, fixes #30


I've actually done a lot of testing and my findings have allowed me to make my solution much better than it originally was. It's kind of hard to explain. Can you wait?


Test added in 677dfc4.


Can you package all the classes together? I guess there's no need to have each class in a separate package isn't there?


You are doing this in the postgresql role rather than in the odoo role, is it much harder to do it in the odoo role?


Nit: indentation


@KathJohnson This change needs CSS in the theme specific stylesheets, originalTheme.css (light theme) and dark.css (dark theme). This allows for full-width layouts, like I did on the Klitsas site. For some odd reason the top nav was not put inside the header tag, which is not a good setup. I'll look at this soon and add styles to make this work on all the light theme sites.


Yeah I figured it out, seem like the issue was in URL. Mail Chimp needs email address to be concatinated at the end of url in md5() format. So URL needs to be  /list/$id/members/md5($email) the rest is fine


I don't know if it's a good idea to document methods that could also be in the core. On one hand, that makes the examples easier to understand, but on the other hand, readers might think that this behavior is already in the core.

What about something like `hasMoreSiblings($num)` that returns a boolean based on the passed number of siblings to check for?


which error ?


Found using IDA


Думаю, `.es5` писать в конце смысла нет, дабы не было путаницы.


Why use `Option`? This is only called with `Some`...?


I like the README addition, good work


ya I tried via a setTimeout w/ calling the callback first because of your experience there.


Welcome back!


shouldn't this be defined in CSS?


now the secret source of the copied code is revealed! :stuck_out_tongue:


Didnt remember.. Was a few weeks ago.. Somethin with audio i believe.. Did
u managed to compiler aosp?
Op 9 mrt. 2016 20:22 schreef "FayaX-PT" <notifications@github.com>:

> which error ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/quesoo/android_device_lge_g3-common/commit/ac9a9997ecbd2f1e334195b333f2b77e6ea45ec7#commitcomment-16598456>
> .
>


This button says "Save", which doesn't seem right since the login details are not saved anywhere. Maybe "Ok" or "Submit"?


Ezek mind létező ubék nálunk, build-depends fájl nélküliek, de táplálkoznak belőlük más csomagjaink, vagy csupán használják.
Az már más kérdés, hogy az a csomag, ami használja, az szükséges e valójában?


Now that sample_load [:s1,:s2,:s3,..] is working I've been trying to work out best way to allow time for sample loads. This is what I've currently got.
```
##| sample_free_all  #un-comment and run this first, then re-comment
##| stop
trigger=0
live_loop :t do
  sleep 0.1
  if trigger== 1
    cue :start
    stop
  end
end

load_samples sample_names(:loop)  #could use load_samples "loop" here instead
trigger=1
sync :start
sample :loop_amen #test these can run together
sample :loop_mika
sample :loop_breakbeat
```
This seems to work, allowing the minimum time required, whether samples are already loaded or not before using them. Playing 3 loops together is a pretty good test that they are there.
What would be nice would be some means of ascertaining when a load_sample command had actually completed, and the sample was available for use. Also nice would be a signal for when sample_free_all was completed.


No. I am trying


The caching issue is a problem during development too.

Along the same lines, but ultimately more flexible might be to write an "include_js" wrapper function. Would handle both the "$webroot" part of the path and a force cache refresh components.

The force cache refresh could have a development mode flag that uses "timestamp" to always force a reload, and the production mode could be tied to the version number.


It would probably be good to show the URL scheme here as well, so users know if they are submitting over http or https.


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 356](http://ci.infinispan.org/viewLog.html?buildId=37241&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 6396809633bf909a9a6f7540994af9804d44f89f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:34


Same here. While definitely useful, this example might make people think this is already implemented in the core.


nice


@tgamblin well I think my opinion is clear on that : if it was harmless the developer wouldn't have added that check. Anyhow, as you all agree,  I guess I'll just have to remember to build `~cxx+mpi` and `+cxx~mpi`.


This gives `NoSectionError: No section: 'env'` for me, see https://github.com/opengridcc/opengrid/issues/118.


Thanks for the heads up, @littledan. After I update the proposal (probably some time next week) to reflect the committee's wishes, I will also submit a PR to test262 which makes these tests much more strict.

For now, @jugglinmike can either leave this test here or remove it until my PR comes in. Is it inconveniencing you, @littledan?


Great example actually. I would name this `withGame` though, but that's just my opinion.


We've been doing `Expect(x).To(y)` everywhere instead; would you mind converting these over? (Admittedly I'm not sure if the rest of the resource is using `Expect`).

Here's a handy bit of bash to do it for you:

```sh
for pattern in \
  'Ω(a).Should(b) -> Expect(a).To(b)' \
  'Ω(a).Should(b, c) -> Expect(a).To(b, c)' \
  'Ω(a).ShouldNot(b) -> Expect(a).NotTo(b)' \
  'Ω(a).ShouldNot(b, c) -> Expect(a).NotTo(b, c)' \
  'Ω(a).To(b) -> Expect(a).To(b)' \
  'Ω(a).ToNot(b) -> Expect(a).NotTo(b)' \
  'Ω(a).NotTo(b) -> Expect(a).NotTo(b)' \
  ; do
  find . -name '*.go' -a -not -path '*/Godeps/*' | xargs gofmt -w -r "${pattern}";
done
```


Actually, just thinking about this a little more, I really like the explicitness of using named exports. It's very easy to scan a file and see public vs. private methods that way.


Are these comments necessary?


:cop: the local environment cop is on to you @romsson


Awesome work !  Can you explain why these needed to be re-implemented for user space?  It was my understanding that gcc already provided these in `cpuid.h`.  Are you trying to avoid adding a dependency on gcc?

```
/* Return highest supported input value for cpuid instruction.  ext can
   be either 0x0 or 0x8000000 to return highest supported value for
   basic or extended cpuid information.  Function returns 0 if cpuid
   is not supported or whatever cpuid returns in eax register.  If sig
   pointer is non-null, then first four bytes of the signature
   (as found in ebx register) are returned in location pointed by sig.  */
unsigned int __get_cpuid_max (unsigned int __ext, unsigned int *__sig)

/* Return cpuid data for requested cpuid level, as found in returned
   eax, ebx, ecx and edx registers.  The function checks if cpuid is
   supported and returns 1 for valid cpuid information or 0 for
   unsupported cpuid level.  All pointers are required to be non-null.  */
int __get_cpuid (unsigned int __level,
    unsigned int *__eax, unsigned int *__ebx,
    unsigned int *__ecx, unsigned int *__edx)
```


"Everthing" -> "Everything"?


Put your errors here, peolple will help you!
http://forum.xda-developers.com/chef-central/android/guide-android-rom-development-t2814763


It doesn't exist until the table record is there.  So exists should be false until then


Good example. Maybe we could use a similar one for the pages methods.


try:
```this.matchingGames = this.matchingGames.filter((game)=> game.noise === true);```


One of the configurations that failed was our warnings as errors build. I suspect that every other build was warning about this, but since it didn't fail it was missed. The other build is against gcc 5.2 which I suspect made this issue an error instead of a warning.


Phil, I'm not really understanding the purpose of this, but if the problem is anywhere it is in the hardware or firmware specific to the BRCM chipset used on Pi3B, not in the brcmfmac driver. The driver works fine with power management enabled, with other bcm43xx series hardware. (Is this an attempt to deal with those, brcmfmac: brcmf_sdio_hdparse: seq 106: sequence number error, expect 108, log messages I reported? Disabling power management didn't help.)


It's sort of annoying that the HDF5 developers have added weird "supported" settings for two orthogonal features.  At the moment I think the real issue is that if the user specifies one of these (+mpi or +cxx) then Spack can't express what the other should be set to when the user doesn't care. It seems that @alalazo would like to default to a supported configuration.

In this case, my general opinion is more in line with @eschnett because it seems that only Intel doesn't like the C++ interface.  So in most cases, `+cxx+mpi` will build and will be more useful and enable more sharing.


Whether You development and tech company is throwing a Christmas party or just a formal "hack athon", having a comic there can keep the mood light and keep the party going.   href="http://www.comedianagency">Comedian Agency</a>


That's true. I was not sure about expected outcome when asking for project level access. Probably filtering out component level ACLs is better approach. Feel free to send PR.


I think an `if` block with a status message like in your `$page->countSiblings()` example would be more useful here.


I think we can leave out the ifdef here.
exit is defined to be in stdlib.h everywhere


so: ```this.matchingGames``` is progressively filtered to have fewer items based on the filters.


Please fix the whitespace-only changes here.


Please make this return a `const Memory &`.


This should state {immediate: true} instead, right?


Yes, it will not work, since this needs more thought. Result does not have filter, but one time filter, so prep needs to be done


Debug code?


Can you please take a stab at having the Fish and Rat each just return their "state" (memory + extra stuff) in one object? Then those objects should know how to serialize themselves.


Looks great!


FYI:

`--  ^ List of node names to trigger the LBM on.
 +    --    Also non-registered nodes will work.
 +    --    Groups (as of group:groupname) will work as well.`

Whats the point of defining group or node when it gets overridden by any unknown node?
Sometimes i want to replace the unknown node with some different one but its not working because it gets removed anyway.

`Also non-registered nodes will work.`
^
There should be an option for this (Boolean variable) or am i doing something wrong ?


Same comments here as in fish.hh


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


See:
https://www.raspberrypi.org/forums/viewto
pic.php?f=28&t=138312
https://www.raspberrypi.org/forums/viewtopic.php?f=28&t=44044&start=425
https://discourse.osmc.tv/t/pi-3-wifi-unstable/14171


It is 28bytes long on windows, 4 bytes on linux, and it is a string.
![string](https://cloud.githubusercontent.com/assets/6759752/13647840/984f28c0-e63e-11e5-896c-dc98fecfa644.png)


This scares me. This code shouldn't have to know about Fishes and Rats, or at least shouldn't have to maintain an enumeration of the possible senders.


One thing you could consider here is to add minimum_players and maximum_players to Game -- this logic would be a lot simpler (ditto for time range below).


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


The state object should know how to serialize itself.


No, just the style we chose for these scripts over in VTK.


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1024 of 1024 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-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-30.61%25-green.svg)


Found using IDA


probably worth throwing an error if one of the keys isn't something we currently support - otherwise I could see someone accidentally typo'ing `nrpoc` and not understanding what was going wrong until too late.


Sounds good to me.  Thanks!


Perhaps `get_projectdir()` could take a second optional argument, which would be the default filename when nothing else matches?

So that:
```
if [ -n "$(get_projectdir busybox/busybox-target.conf)" ]; then
  BUSYBOX_CFG_FILE_TARGET=$(get_projectdir busybox/busybox-target.conf)
else
  BUSYBOX_CFG_FILE_TARGET=$PKG_DIR/config/busybox-target.conf
fi
```
becomes:
```
BUSYBOX_CFG_FILE_TARGET=$(get_projectdir busybox/busybox-target.conf $PKG_DIR/config/busybox-target.conf)
```

Also, since the function now returns the filename, perhaps the name of the function should reflect that... `get_project_file()`.


@a-ashbeck we'll probably use this a lot and it just occurred to me... it could be simpler:

`btn_text = @locale == 'english' ? 'Done' : 'Fijar'`


Issue5276 (wfbuhl) - 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)


Do we really need to be logging both the instantaneous values and the cumulative averages?


@tgamblin I'll retry tomorrow, but it was the C++ interface plus MPI (and Intel + IntelMPI). C++ without MPI went fine. That said I don't know the details of the build (I didn't inspect the logic inside configure and similar files)


Imo you just inline that function into the Lint class. I think doing
```python
check_prerequisites = check_linter_prerequisites_builder(executable, ("arg1", "arg2"), "program doesn't work")
```
is quite okay, though I dislike the name of that function, it really sounds pretty weird and complicated. But the code is quite readable imo, it looks like yours: Check on shell if program exists, `subprocess.call` it with necessary cli-args, check exit-code.

The only improvement I see is one can avoid passing the executable again to the test-call, but you need to again manually define `prerequisites_command` with the executable itself. As far as I see no redundancy is removed here^^


Yes, "ignoring the technical details on what properties $N$ must have" really was mainly referring to the requirement of N being prime, so that it wasn't necessary to introduce the totient function (although iirc there are other considerations about N; not just any N is appropriate I think). So if we introduce that, we should explain what the totient function is, also clarify that phi(N) = N-1 for N prime. Probably in that case, also discuss the problem of finding a generator g.

On balance I preferred not to go into the number theory stuff. Just "this construct can exist" kind of thing.

The clarification about *multiplicative* group is a good one, so I agree with that change.


It's just a nit either way. The alternative I picture looking more like:

```js
export default {
  getCurrentLocation,
  getUserConfirmation,
  startListener,
  pushLocation,
  replaceLocation,
  go
}
```

More consistent to how the history factories are set up.


This would also be cleaner if the senders expressed a state object (could be different types) that knew how to serialize itself. Not thrilled about the many positional arguments (and zeros).


Thanks for the reply, I think I found the problem.
If I create a debug statement like:
debug("something, something, something darkside\n");
I can add/remove the debug flag and everything compiles correctly

But if I add parameters:
debug("something, something, something %s\n", someside);

Compilation fails when I don't include the debug flag
error: macro "debug" passed 2 arguments, but takes just 1
error: 'debug' undeclared (first use in this function)

I get the same thing when I compile the demo.c

-----Original Message-----
From: Miles Richardson <notifications@github.com>
To: bitbutt/c-debug-header <c-debug-header@noreply.github.com>
Cc: gebuh <gebuh@aol.com>
Sent: Tue, Mar 8, 2016 4:55 pm
Subject: Re: [c-debug-header] Fix minor errors in README (74579b9)

Your code should still compile unless there's a bug (in my code or yours). See lines 48-52 in debug.h. If you don't include the debug flag, the preprocessor will replace the debug statements with ((void) 0) (a no-op).
—
Reply to this email directly or view it on GitHub.


Quality.


should be `return $val !== null && ! Utils::isFuture($val['valid_until']));` right?


I get on El Capitan: `/usr/include/c++/4.2.1/cstdarg:51:10:` fatal error: 'stdarg.h' file not foundIn file included from `/Users/rurban/Software/smhasher/Bitslice.cpp`


Outside of hashing, this is the first use of `friend` in Remy -- can we just do it the old-fashioned way with public accessors?


@alalazo: sounds good.  I'll think about how we could better represent constraints and preferences in directives.  I think that part of the problem is that `depends_on` doesn't differentiate between what is *required* and what is *preferred*.  I am thinking I could come up with places to put hints for stuff like this...


Gotta be careful since the model's start and end time is not the same as the Job's start and end time, for grid and cross-validation models.  Also, end_time() will blow an assertion if the job was cancelled.


Actually I don't think it's very appropriate because it uses ngrok and some other stuff. Too complicated to just respond with an index.html don't you think?


Fixed.


Is `flex-grow` still meaningful after this change? I looked up the commit that added the `flex-basis` (it was @scruffian), and both rules were added at the same time.


Doesn't this mean the handler is fired on all pages?

Also, if you ajaxify to dash more than once, does it get added multiple times? Maybe need to do a `.one()` instead.


@rhatdan
Added man page installation in Makefile.


keep the good work


No, nothing is inconveniencing me; I just wanted to make sure tests that are committed are accurate and reflect the language evolution. Happy to have you both working on tests in this area, and waiting for @michaelficarra 's tests when stage 3 is reached sounds good.

@jugglinmike for background, @michaelficarra and TC39 found the ES2015 language to be both more permissive than you might expect and impossible to implement correctly (correct me if I'm wrong), so I don't think it's worth testing that version.


Issue5276 (wfbuhl) - x86_64-MacOS-10.9-clang: OK (2124 of 2132 tests passed, 2 test warnings)

 * 10 tests had: EIO diffs.
 * 7 tests had: ESO big diffs.
 * 4 tests had: Table small diffs.
 * 3 tests had: ESO small diffs.
 * 2 tests had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: MTR small diffs.

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


The following doc(s) have been deployed to the [ttrowellco](http://ttrowellco-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/unsupported-modules-plugins](http://ttrowellco-static-docs.pantheonsite.io/docs/articles/sites/code/unsupported-modules-plugins)


2.1.0-alpha1 now please :)


Ok. I built the whole example in modular style so I added config.ru if you don't mind.
Btw. Is it a preference to use classic style for guides?


Hello. Plain command are procesed this way: https://github.com/trezorg/aioredis/commit/794f023a7f9bc6e0005516c1f2fb1232a6ad05f4#diff-5c4d2b2edb540110e522325fad3205beR385 and later self.execute. In case the command does not have arguments it is supposed to invoke on all nodes (on example flushall) -  self._execute_nodes, in case there are agruments is is supposed to be invoked on some certain node - self._execute_node. Also there is a mixin for custom commands - https://github.com/trezorg/aioredis/commit/794f023a7f9bc6e0005516c1f2fb1232a6ad05f4#diff-cb5be3b5c66dd6ad4c6c3163849c9078R1.  For the time there is support for only keys and scan commands.


nit: This variable name should probably be in all caps since. No need to fix it though. In the future we can update it if/when another constant is added.


yeap


\\ teste


we always prefer interface or class/enum to make clearly definition for any concrete class to follow.
In this case, we will remove firstRowIsHeader() function and let IFileFormat to be a marker interface.


this should be supported for operations, sources. Also test missing for parameter group


Ah, I see. Ya, that could be a bit clearer here. Not terrible to be too specific here though.


Thanks for Your work.
For now i've freezed my work on cm. Lenovo S650 (mt6582) have only KitKat, but seems latest mtk code doesn't like old ril blobs and i don't have enough of knowledge to make it work.


True. We should settle some conventions. In particular, I don't think we will be optimizing before we run borrowck -- but we still want to be clear on what implicit assumptions we are making about its structure (and minimize those wherever possible or at least dynamically assert them).


cool


It appears the `cpu_has_osxsave` and `cpu_has_avx` wrappers weren't added until 3.0 kernels, torvalds/linux@66be895158886a6cd816aa1eaa18965a5c522d8f however the X86_FEATURE_OSXSAVE flags go back to 2.6.27.  So it looks like we'll need to use the flags directly here as you've done.


Note that using `as u8` to convert to u8 is wrong since it rounds down. You have to explicitly round if you want that behavior. This is also potentially wrong because it doesn't clamp the output - if the channel is 255, and alpha is 1, the result doesn't fit in a byte.

BTW this too can be done as fixed point math. `((chunk[0] as u16) * 255 + 127) / (chunk[3] as u16)` maybe - I'm less sure about the math in this case. Also need to range limit the output.

It's all a bit tricky so this can be tackled separately if you want. Getting rid of the premultiply table should be a simpler matter.


:+1: been trying to get to the bottom of this for a while!


It looks like the kernel has provided macros for these calls all the way back to 2.6.27.  That appears to be the preferred interface so we should probably use it instead of explicitly calling `boot_cpu_has()`.  That way if they upstream kernel ever changes the macro names or renames `boot_cpu_has()` it won't break the build.

```
#define cpu_has_xmm             boot_cpu_has(X86_FEATURE_XMM)
#define cpu_has_xmm2            boot_cpu_has(X86_FEATURE_XMM2)
#define cpu_has_xmm3            boot_cpu_has(X86_FEATURE_XMM3)
#define X86_FEATURE_SSSE3       (4*32+ 9) /* Supplemental SSE-3 */
#define cpu_has_xmm4_1          boot_cpu_has(X86_FEATURE_XMM4_1)
#define cpu_has_xmm4_2          boot_cpu_has(X86_FEATURE_XMM4_2)
```

That said, I see below that the kernel was inconsistent about providing these wrappers.  So I'd be OK with not using the wrapper macros so at least our code is consistent.  Looking back at the kernel git history these appears to be an area of the kernel which has seen very little change historically.


@spikebrehm :stuck_out_tongue:


we need to guard against possibilities that port_instance, port_tenant being None, and port.get('network_id', '') returns ''.


Oh, I see. The ante has been upped, from "out of sequence" messages and crap throughput, to I cant access my Pi at all via on-board wifi with power management enabled on the BRCM wifi interface. I hadn't seen those forum threads..... ;)


Er moet meer shadow!


this proxy manager will serve as a good example in my slides about - how to create bigger problems to solve small problem, thanks. By the way, how much [schema LazyLoadingGhostFactory](https://github.com/doctrine/migrations/commit/6e0c1df125a12be77b3121e238cd7301d84d4308) improved the performance, for 20 or 200 migration files?


do you need to change the doc now to say dsk.xz and dsk.vdi.xz (using a "." instead of an "_" in the filenames?


JMP start
Write:
    DB "19.09"
    DB 0
start:
   MOV A, 19
  MOV B, 09
  MOV C, Write
  MOV D, 232
  PUSH B
  PUSH A
  MOV B,0
Function:   MOV A, [C]
  MOV [D], A
  INC C
  INC D
  CMP B, [C]
  JNZ Function
  POP A
  POP B
 ADD A,B


**note:** the duplicate line 220, is removed in later commit.


Also, this component is imported in approximately 48 other components (`spot client/ Notice = require | grep jsx | sort | uniq | wc -l`), not just global notices. For this reason, I think it's best if we make this change in a separate PR.


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2162 of 2170 tests passed, 2 test warnings)

 * 10 tests had: EIO diffs.
 * 7 tests had: ESO big diffs.
 * 4 tests had: Table small diffs.
 * 3 tests had: ESO small diffs.
 * 2 tests had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: MTR small diffs.

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


**SUCCESS** - TeamCity HotRod C# Client :: Pull requests monitor - WIN_S2012 [Build 99](http://ci.infinispan.org/viewLog.html?buildId=37245&buildTypeId=HotRodCSharp_PullRequestsMonitorWinS2012&guest=1) for 6396809633bf909a9a6f7540994af9804d44f89f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:12:33


@gtozzi - This looks like an interesting addition. We could definitely use something along these lines.

I think the main things missing here are the analysis document (roughly like a PDEP - "python-docx Enhancement Proposal" :) and the acceptance test(s).

Starting with the analysis page (docs/dev/analysis/features//*.rst, [this one](https://github.com/python-openxml/python-docx/blob/master/docs/dev/analysis/features/text/font-color.rst) for example) is the right sequence, because it gives us a chance to settle on the API. The API turns out to be the most important part, because it is the user interface of a library like this and because once committed to client code, it really cannot be changed. So it's important to get it right the first time and to make it consistent with the remainder of the API for usability sake.

Of course a different API has a different implementation, so it also good to get that right up front. In this case, I'm thinking an iterator rather than something that returns a list is probably the best approach. There was some prior work on this (it's a perennial on the issues list) and the iterator gives some solid advantages. One of the reasons I prefer it is that it doesn't imply that you can rearrange the document by rearranging the list, which of course wouldn't work. Also, it's easy to make into a list using a comprehension if that's the form most suitable for the client code, is lightweight, and is tolerant of in-loop modifications, such as deletes, if I'm not mistaken.

On the acceptance test(s) issue, the python-docx test structure is two-level, acceptance tests (under features/) serve double-duty as end-user level functional assurance as well as integration tests. The unit tests provide solid assurance of individual methods and properties but also generally integrate the oxml/xmlchemy layer. Between the two it is *very* rare to get a real bug report that locates a coding error. Design errors are actually more common :)

Let me know if you need more. If you were willing to pursue this one we should start by sketching out an analysis page. We can dig up the prior art of "iter_block_items()" I think it was called in one of the issues or something and see how we go.


Agreed.


My fork is depending on CircleCI, because the app binary is built on outside of the Docker Container:

https://github.com/ngs/Swifton-TodoApp/blob/master/circle.yml#L31-L33

So, I think this is not good to merge to the original repo.


I am not really sure why you have an else statement here. Typically this is only used when there you have a ```break``` in your loop. This else will only be triggered if the while statement is false.. Can you think of a better way to do this?


попробуй добавить строку
max_line_length = 120


OK, corretto come da suggerimento di Francesco.
Grazie a entrambi :-)
https://github.com/mozilla-l10n/appstores/commit/a447e2730b0478e502bcc0784575bf1222fab2a4


oo, la la :)

Good luck!


The following doc(s) have been deployed to the [1434upstre](http://1434upstre-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/organizations/adding-a-custom-upstream](http://1434upstre-static-docs.pantheonsite.io/docs/articles/organizations/adding-a-custom-upstream)


aici e de fapt `// TX | RX`


@ctn we wouldn't have so much cost in maintenance burden or switch to new interface because it's back-compatible already.
I will list here my reasons "why I choose to open it at ddf level"
First, DDF will be main point to use, so it is better to make it be fine-grained here.
Client don't need to take a look to any depth level of source code, they just need ddf [ dot ], that's it.
Second, it reduces a lot of time & cost in:
   + look up an api
   + don't need to call handler anytime ( ddf.getTransformHandler().transform)
 Third, it make others code look more clean
And last but not least, we could control this function at ddf level, for example: if 2 persons are using same ddf, one call getSummary already, another will get result not re-run old task.


Better way to handle this?


you can put it as a temporary directly in the call.


where is device_subnet_list defined?


2129_radiance_server_perf (rpg777) - i386-Windows-7-VisualStudio-12: OK (2361 of 2397 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.5%25-yellow.svg)


use the magic macro to avoid repetition


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2360 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.42%25-yellow.svg)


what is the need for this change? Doesn't debug default to false?


Not sure I like assuming doc/sqa


Development has basically halted on this project. I was mainly a bugfix. I'd suggest looking at https://github.com/drzoidberg33/plexpy


Thanks,
The speed improvement only depend on the size of your database (number of tables, number of columns and indexes).


@elchicovzl Tienes disponible, el hostname y el path. No coloques la direccion manualmente


Why is this here? Does it need to be printed and returned?


Here, console is telling me "Cannot read property 'display' of undefined"


Previously had some work passing `hide_me` to the function in the EventListener, and then redefining it.


more hard coded 'doc/sqa/'


More hard coded 'doc/sqa'


Instead of actually setting the setting, you could stub it out:

`allow(Rails.application.config).to receive(:ended) { false }`


btw I don't see how I created a problem with this solution. It's literally invisible and BC for all the users of the library.


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


Ditto for this spec.

If you stubbed it, you also wouldn't need to have an `after` block.


I was looking at making very similar changes to these today. Glad to see you've made a start already though. :)

Any reason to go with gradle 2.10 instead of 2.11?

Have you run into issues with the unit tests? https://discuss.gradle.org/t/gradle-thinks-2-10-is-less-than-2-2-when-resolving-plugins/13434/3


I initialy write documentation in the function header in javadoc, but somehow it's not compiled and I didn't have the time to investigate further, but it's there.


Not a fan of how the white edge directly touches the section edge, re-implement grey space without hr since i know you dont like hr...


Sorry to be nitpicky, but if a spec is in `spec/integrations`, it doesn't need to have `_integration` in its filename.


# Commit message

+ Added shape 'fence_gate'
+ Refactored NEBBlockFence
+ Refactored NEBBlockWall


Pls use a meaningful name for the couchapp, such as ftscp, instead of ftscpSplit.


Closed by 97d66a128:

> Remove pills from add_transaction. See 358080021c for thoughts on subviews.


Is this needed since you moved it up to the base `build.gradle` file?


see https://github.com/coala-analyzer/coala/issues/1859. If you need something like that now, look at the comment from sils1297 here: https://github.com/coala-analyzer/coala/issues/1855


Planning to override this only, or implement it here someday as well?


That's not something you could be doing if this was a multithreaded environement. It kind of assumes this so to speak.


Your linter should be complaining about this.


Yeah, when he got older!!!!!!!!
I AM 14!!!!! Not factorials, LOL


For next time: Need to code buttons for LabView for manual resets and such.  That will be a new experience :)


Dropping the pointer saves an unnecessary allocation, and makes the type easier to understand.


My bad, fixed with an amended commit (so as not to clutter the project history if you merge).


@mattcui are you planning on leaving this? BAD!


Good point. And relying on inheritance (which does work for Doxygen, I believe) is not really adequate in this case.


This seems like a really great idea.  I presume there are scenarios where this would have become an issue.


fast-forwarding 2.0 to borshop-integration-1361-feature/fd/hashtree-update-callback = 1ab5f5fd


Capitalize


where are we with respect to the deployment code for Apex? Would be it a bit hard to run smokes without it.


@ivrh  - This works with any software . The only thing is that it must identify multi site drupal uniquely with the presence of a file. like sites/default/settings.php for normal drupal install. Additionally multisite drupal must have a working template like the 5017 for normal drupal. I havent worked with Drupal much but I am sure multisite is supported by nginx ..so its just about identifying a file that uniquely identifies the drupal setup and a custom template that you added that support multisite drupal.


Good catch. I removed the duplication from `DatasetArgument` class doc string


That flag for installing the compiler must have failed to work.

https://msdn.microsoft.com/en-us/library/60k1461a.aspx

Look at the "warning" box and follow the instructions.

This is unfortunate. I'll double check the choco configuration tomorrow.

Yeah, I fixed those in the next commit, which now says it can't find
CMAKE_C_COMPILER or CMAKE_CXX_COMPILER...

—
Reply to this email directly or view it on GitHub
<https://github.com/energy-plus/EnergyPlus/commit/791d9a51c6a71db43e9e064a5e3bde7ce299d43c#commitcomment-16578444>
.


WHO THE F*** DID YOU EMAIL THIS TOO!!!!!!!!!!!!!!!!!


@mattcui why did you have to do this again? Makes no sense. This is the same commit as: b2ade4d7a418784654ceae07e2ab1f2657ce09f6


Use airlift's create temp dir.


Commit message looks good.  I like the specificity.


Converted to an array.


Minor: It might be worth updating the commit note to explain that you've had to update a couple of tests now that we require at least one `@permitted_next_node`.


The code was correct because I passed `help` to the parent class's constructor, which set the attribute. However, thank you for point this out because `help` and `default` are only relevant for `DatasetArgument`. I removed them from this class and the parent class.


What?  Sorry, no.  My mom just randomly sat down next to me to see what I was doing.


Thanks for pointing out the areas for improvement. I think the results are a big improvement.


I'm using f.lux by the way.  It's great.  It helps eye strain.


The args array is in the common OCI spec, so I don't think we can mess with that.


Added instance of HttpdConfig.


What is that?


len(in or out) == 0 の場合はnullだけはしたいかもしれないね。そうしないとなにをしてもリークしてしまう。全体的に対応しないといけないので別PRで対応しないと。

avutilに
type OptionalString *string

func NewOptionalString(str string) string {
  return &OptionalString{&str}
}

みたいに作らないと


`Use` is kind of..blah  lol


Nitpick...`Testing` seems a little misleading.  How about `Running the Tests`


It's a program that slowly removes blue light from a screen until a certain time.  Then it restores full brightness at a certain "waking up" time.  It helps relieve eye soreness from looking at a computer for a long time.  There's also a setting that maintains night vision.


Thanks Gale.  I told myself at the beginning to remember that, but I guess I should have don't it then.  :-)


viewed, nice using bit manipulation.


OK


So is the mom in response to the astered profanity, or the email?


Yes. Thanks!


I wonder whether it'd be clearer to use `@permitted_next_nodes` here, given that `@permitted_next_nodes = permitted` and we're using `@permitted_next_nodes` elsewhere.


I will test and merge


Hey, I have to go now.  I'll try to upload my tic-tac-toe sometime soon.  And my mom just randomly sits down next to me to make me feel uncomfortable because she wants me to go to bed.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 32](https://teamcity.labkey.org/viewLog.html?buildId=308990&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 34 (20 new), passed: 17 Build time: 00:58:15
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyAssaySummary: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyFilterPane: <no details avaliable>

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

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

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

##### there are 24 more failed tests, see build details
```


after this change the `down()` function (for migrate:rollback) no longer works because index 'users_status_address1_address2_index' doesn't exist. so line 60 should be changed:

```php
$table->dropIndex('users_status_address1_index');
```

... though I don't see why the original index -- status(6) + address1(255) + address2(255) -- would exceed the 1000 character limit.


Looks very nice.  I appreciate the links and the nice formatting.  Just a few minor comments...Would you mind tweaking those two headings and updating the PR?


O.K. Now I feel better!


BYE THEN!


GO TO BED!


`
goto :bed
`


Looks like this is in the wrong branch!


pls remove this file :)


@lzghuluwa 少了一个空格。你不看看效果的嘛~囧


Excellent!


`Orc type from file footer (%s) does not match expected type of table (%s)`


Pulled to update local repo, and the pushed the 2.2 assignment to GitHub.  Forgot to first create a new branch so all was done in the master.


I'm all in for C11, not sure if @lemire will agree.


Looking good :tada:


It looks like you missed one of my previous comments:

> `1, new ArrayType(DoubleType.DOUBLE), // should be map<varchar,date>`
> Same for the other bad ones.


Isn't that controlled with `CMAKE_BUILD_TYPE`?


Oh, weird.

Anyway, I think it's easier to just use `MINIMUM` and `MAXIMUM` than conditionally use the `sys/param.h` version or a local version. There's no need to use `#ifdef` when you don't have to worry about whether it's already defined.


I copied the existing CFLAGS: https://github.com/RoaringBitmap/CRoaring/blob/b454f3fff644c01571abcf960d666a063b9a5b93/makefile#L6


Super! This looks really useful. Thanks @adamretter!


Could be a great `enum` instead, no? Either way, I'm not there's any reason to extend `NSObject`.


Looks like you missed one of my previous comments:

> When you put arguments on multiple lines, always put the argument on its own line.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 31](https://teamcity.labkey.org/viewLog.html?buildId=308988&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 15 (15 new), passed: 36 Build time: 01:25:42
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cds.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 5 more failed tests, see build details
```


@barisusakli NodeBB v1.0.0 :trollface:


is this the current fragment that is being shown?


You don't need this. Uncaught exception from test method will automatically result in test failure.


some changes to deal with non X-window problems.


I added it in the past for `aligned_alloc` which we then reverted. I guess it wasn't removed from the makefile.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/114679803


If we can't change it to a string, then I don't think it's appropriate to require that it be pre-escaped.

Why are build and run different, again? Is it because build expects to pass something to a shell and run does not?


Could you send me the python fix?

On Tuesday, March 8, 2016, Raymond Lee <notifications@github.com> wrote:

> some changes to deal with non X-window problems.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dangeles/TissueEnrichmentAnalysis/commit/7dc5a82e6085e481be84903411facddf39279fca#commitcomment-16579168>
> .
>

--
David Angeles,
Ph.D. Candidate in Biochemistry and Molecular Biophysics,
Caltech


Does this avoid a Rubocop violation, or is it preferable for another reason?


I have some problems here.
guid  91749-91771 has data in my creature table.
Please see
('91771', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1980.72', '5364.88', '-12.3225', '0.226893', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0'
('91770', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1978.82', '5365.21', '-12.3448', '3.63029', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0')
('91769', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1747.9', '5492.11', '-12.3448', '1.3439', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91768', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1748.08', '5493.86', '-12.3448', '4.2586', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91767', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1821.12', '5303.35', '-12.3448', '0.767945', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0'
('91766', '20102', '530', '0', '0', '1', '1', '16431', '1', '-1818.99', '5305.67', '-12.3448', '4.03171', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0')
('91765', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7147.06', '-3830.22', '8.51396', '6.05629', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91764', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7144.12', '-3830.95', '8.50543', '3.26377', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91763', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7219.66', '-3731.91', '8.4635', '5.13127', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91762', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7217.57', '-3735.16', '8.45314', '2.28638', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91761', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7053', '-3779.9', '10.1478', '1.53589', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91760', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7053', '-3779.9', '10.1478', '1.53589', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91759', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7053.62', '-3777.21', '10.2047', '5.25344', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91758', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7185.2', '-3809.25', '9.01346', '5.51524', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91757', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7182.97', '-3811.81', '8.80395', '2.3911', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91756', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7219.66', '-3731.91', '8.4635', '5.13127', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91755', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7217.57', '-3735.16', '8.45314', '2.28638', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91754', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7147.06', '-3830.22', '8.51396', '6.05629', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91753', '20102', '1', '0', '0', '1', '1', '16431', '1', '-7144.12', '-3830.95', '8.50543', '3.26377', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91752', '20102', '1', '0', '0', '1', '1', '16431', '1', '6791.34', '-4747.01', '701.617', '2.21657', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91751', '20102', '1', '0', '0', '1', '1', '16431', '1', '-924.099', '-3718', '9.18514', '1.69297', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91750', '20102', '1', '0', '0', '1', '1', '16431', '1', '-924.656', '-3715.91', '9.13173', '5.06145', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');
('91749', '20102', '1', '0', '0', '1', '1', '16431', '1', '-897.124', '-3763.23', '11.9882', '4.32842', '300', '0', '0', '42', '0', '0', '0', '0', '0', '0');

How can I solve this problem?


You are right.  I don't know why I was thinking that we were talking about the CONSTRAINT account_contact_idx, hence the confusion :)
I pushed a fix for it.


Grattis! :D


This could just be a `StringRef`, since we won't modify this.  It's okay to form a `StringRef` for a string literal, because we know the storage for a literal won't go away on us.


Firebase ref = new Firebase(getString(R.strings.firebase_endpoint?));
params.put("userId", ref.getUIDorSomething());


Do I need to add spark.scope back to keep spark jars out of Livy's assembly after this change?


I will need to investigate that. The field is empty if dbtext is used.
I will investigate more and get back with more info.


My pleasure :-)


fast-forwarding master to auto = 006ced58


I would return `TypeSignature` given the argument is `TypeSignature` to make the code more coherent and less surprising. In this specific case, there are two additional benefits:

* This method can become `static` (Please make this method `static` as well)
* The code will be cleaner when you try to recursively call this method.


@isidorn I saw you fixed this configuration in your fork and your change is actually the correct one, it would be great if you send me a PR :smile:


Weird, I can't seem to repro this locally (with a fresh checkout of this github repo) or in fbsource:

```
[jeffmo@jeffmo-mbp2 ~L/react-native<master>]$ ./node_modules/.bin/flow version
Flow, a static type checker for JavaScript, version 0.22.0
[jeffmo@jeffmo-mbp2 ~L/react-native<master>]$ ./node_modules/.bin/flow
Launching Flow server for /Users/jeffmo/Projects/react-native
Spawned flow server (child pid=15419)
Logs will go to /private/tmp/flow/zSUserszSjeffmozSProjectszSreact-native.log
No errors!
```

I'm not super familiar with CircleCI -- but any clue why it would be any kind of different environment from that workflow?

(Note that I forked `react-native` and `git clone`-ed it immediately before that paste of my console)


is it deprecated?


I would call this method `toStorageType`.


I tried to copy the original Makefile which compiled with -O3 by default, unless DEBUG=1 was defined. It's probably best to stick with CMake conventions though. See 2b5d160a6ef6101e521f57418c5d159c706bcc1e.


@mmcco There are other places in htop that use MIN() and MAX(), and they are all surrounded by #ifndef to avoid repeated definition (at least for preprocessor). Using MINIMUM or MAXIMUM will break consistency with other code. They are not neat. They're cheap.


thx!!!


doh!


Yeah it is, I can rename it to mCurrentFragment or something to be clearer


Undo space.


Done in later commit -- I accidentally deleted your change when rebasing oops


One more tiny nit and we're good.

Can we not break before the `,` on lines 10-11?  In other words, try:

```
[procedural](https://en.wikipedia.org/wiki/Procedural_programming),
[object-oriented](https://en.wikipedia.org/wiki/Object-oriented_programming),
[functional](https://en.wikipedia.org/wiki/Functional_programming)
```

Thanks for the contribution!


Do we really need update api verison for every commit? I thinks bump API version is only needed for every release.


I would use commit message "Validate that type from ORC file footer matches table metadata"


@Explorer09 Dude, this is not a big deal. There are ~4 other definitions and ~5 uses - they can easily be changed as well.


`xCalloc` is supposed to die when memory cannot be allocated. Thus the check for NULL is meaningless.


No, monitor hosts only. In a large cluster we may have dedicated storage (OSD) hosts and we don't gain anything by running the agent on those.


approved


Finnally!


I don't know how to get rid of activities and make the transition!!! @avaidyam I'll make the transition, right now I'm just trying to consolidate what we have


Thank you for noticing this. Fixed.


The default value is ` false` actually. 0 can never be a valud value for these parameters. 0 denotes absence of these parameters


Nice catch, those should be `std::numeric_limits<ElemType>::max()`.


D-Link black, rev C1 works. The hub internally has two 5-port switches, switching hub 1, port 5 off disconnects the 4 right-most ports.

My mapping is the same as yours [A, B, C, D, E, F, G]:
A: hub 1, port 4
B: hub 1, port 3
C: hub 1, port 2
D: hub 0, port 5
E: hub 0, port 2
F: hub 0, port 3
G: hub 0, port 4

Can you give it another shot, making sure hub 1, port 5 is on?


This was me, you guys were looking at the screen when I did it
I will do it again


The commit note mentions upgrading Bundler but that doesn't seem to be reflected in the diff.


What.


It's not a good way to put much code in the commit log.


Right, I fixed that in the subsequent commit 4b780a3.

FWIW, there are probably many of those remaining elsewhere in htop. I don't think Hisham removed the NULL checks when he transitioned to x(M|Re|C)alloc. That isn't a problem, or course, but they should be combed out at some point.


@MartinBalerio não dou resposta mal educada para quem aponta algo que realmente precisa em um dos meus plugins. E muito menos fui mal educado aqui, ao contrário do que você esta esperando que eu seja.
Além que nenhum dos meus plugins ou eu depende de doações, é bem diferente, faço as coisas conforme tenho tempo livre, entretanto se esta pessoas estão me cobrando algo, acredito que eu tenho o direito de cobrar algo também.
A conversa poderia ter ido para um rumo totalmente diferente, ao contrário disso você preferiu isso, a troco do que? eu não sei.


Thanks!


micro for every commit, minor for every release. this make sure all interface change tracked. You can think this is contractor with user, every change you should notice the caller. else it will have mismatch between


Hello


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


Unless I'm missing something, it doesn't look as though the commit message includes the warning in the test output.


Thanks for these models for a base


Alias @MartinBalerio, para você ver como as pessoas me cobram o tempo inteiro coisas, veja o seu comentário acima me cobrando diretamente sobre algo.


it's overload function you only get information when you seen full function declaration. It's not in commmit  summary , it's in commit log, you can add as much thing as you can to clarify things


So task 3 only works if the infile is in your root directory, it will not work with a file path.


this should be decl-specifier-seq NOT decl-specifier-seqopt


@mnewsome :-D


redundant line


I would use commit message

```
Fix column types passed to ORC reader in Raptor

Raptor stores DATE and TIMESTAMP as integers in file. Therefore, it should
pass BIGINT instead of DATE/TIMESTAMP as the column types to OrcReader.
```


Thanks for the PR, btw this pod isn't released yet and I'm still working with FDWaveformView. Probably I'll release this library with my own customised version of FDWaveformView.


@davidmfoley I've moved my filtering logic from the separate pipe files to this file and now I'm stuck. As I have it now, this.matchingGames gets reassigned to whatever the last filter is, which is not what I want it to do. this.matchingGames is an array so I thought of doing something like: `this.matchingGames.push(this.games.filter((game)=> game.equipment === this._chosenEquip));` but I was getting an error: "argument of type Game[] is not assignable to parameter of type Game." Does this have anything to do with lines 21 and 22? Can you tell what's going on?


please use consistent indent format, indent level is 4 spaces.


Take a look at this article: http://www.adobe.com/devnet/air/articles/faster-byte-array-operations.html


hi,
I am a new gay.
I have something is not understand.
```
 +    if(args.no_ssl == 1):
 +            import ssl
 +            default_context = ssl._create_default_https_context
 +            ssl._create_default_https_context = ssl._create_unverified_context
```
I didn't see that the 'default_context' value is  not used.
When I run the 'getallvms.py', the result is follow:
```
Traceback (most recent call last):
  File "C:/git/pyvmomi_learning/src/pyvmomi_samples/samples/getallvms.py", line 102, in <module>
    main()
  File "C:/git/pyvmomi_learning/src/pyvmomi_samples/samples/getallvms.py", line 78, in main
    port=int(args.port))
  File "C:\Python27\lib\site-packages\pyVim\connect.py", line 663, in SmartConnect
    sslContext)
  File "C:\Python27\lib\site-packages\pyVim\connect.py", line 552, in __FindSupportedVersion
    sslContext)
  File "C:\Python27\lib\site-packages\pyVim\connect.py", line 472, in __GetServiceVersionDescription
    tree = __GetElementTreeFromUrl(url, sslContext)
  File "C:\Python27\lib\site-packages\pyVim\connect.py", line 440, in __GetElementTreeFromUrl
    sock = requests.get(url)
  File "C:\Python27\lib\site-packages\requests\api.py", line 67, in get
    return request('get', url, params=params, **kwargs)
  File "C:\Python27\lib\site-packages\requests\api.py", line 53, in request
    return session.request(method=method, url=url, **kwargs)
  File "C:\Python27\lib\site-packages\requests\sessions.py", line 468, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Python27\lib\site-packages\requests\sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "C:\Python27\lib\site-packages\requests\adapters.py", line 447, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)
```

my enviroment:
python 2.7.11
pyvmomi 6.0.0


Are we seeing this warning because we're now using the newer version of `Net::HTTP` that's bundled with Ruby 2.3?

Although the warning has been silenced in the test, presumably it might still be generated in our production code. Is that correct? And if so, do we need to be worried about it?


indent format inconsistency.


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


I would suggest you use for() instead of std::for_each.


![qq 20160121093207](https://cloud.githubusercontent.com/assets/17700992/13623787/fb5c9bc4-e5e3-11e5-911e-5937454839cd.png)


Looks good.  Now that you've added a commit to this PR, it would make since to have this as all one commit since it represents the same unit of work.  Will you squash these into one?


I'm not surprised hahaha

@Thecrazyskull once made a big deal about a fucking hex color.......yes, a hex color. Said that SlimROMs did it first hahahaa Who fucking claims a hex color?

A fucking hex color! http://gerrit.dirtyunicorns.com/#/c/23692/


indentation here.


Hm, that's interesting.
Then we probably want to use a union there itself ?
https://docs.python.org/3/library/typing.html#typing.Union

You can use
```
import typing
...
...
        maxstatements: typing.Union[int, bool]=0,
```

or something ?
@sils1297 is this supported by bears ?


I am trying to once again learn to use git and how to contribute to OpenEMR.

I believe I copied your branch into mine and with this made a new branch in my ophthal/OpenEMR_2 repository called “add_issue_and_code.
Then I made a small change to interface/patient_file/summary/add_edit_issue.php and committed it.
It should be on github as:

https://github.com/ophthal/openEMR_2/commit/d8b666bc96043e16fe6b2edb6e198fb04a35fa63

The change:

When the user adds a new issue by clicking on an option in the picklist, if the option had a code associated with it in the DB,
the code is also added to the form field “form_diagnosis” (why it is not called code or something similar I have no idea).

If the item was saved by the user without a code, the field is null and all proceeds silently.

How did I do?  Did that work?

Ray


redundant line


This seems wrong to me - according to (this)[https://github.com/aquynh/capstone/blob/3fcbd02eef5a63cc0001870ccfd15d13e2322cd5/include/capstone/capstone.h#L558], the function is defined as follows:

```
CAPSTONE_EXPORT
bool cs_disasm_iter(csh handle,
	const uint8_t **code, size_t *size,
	uint64_t *address, cs_insn *insn);
```

This only takes a pointer to an instruction, not a pointer to a pointer to an instruction, in contrast to `cs_disasm`.


Done. However, I've realized something. Once we get object size distributions, I think latency percentiles will be very deceptive, because they will be latencies for transferring objects of different sizes. There will be variation coming from the object size distribution, and not from the actual backend service.

Given that, should I even compute latencies, or should I just do either size / latency or latency / size? I've implemented latency / size, but I could easily change to any of the three options, or all of them.


Isso deveria ter ido para a issue #29


lololololol


dfsfsdfdsffsddf


Since `cs_disasm_iter` is specifically designed for a no-allocation case, I don't think that passing it null is valid.


> Seria ótimo se o @claudiosmweb conseguisse arranjar um tempo para adicionar isso.

Sinceramente, não considero isso uma cobrança, aliás, quem levou a conversa para esse lado foi você, como costuma fazer em todas suas interações com usuários.

Mas deixa pra lá... você está certo @claudiosmweb dá um tapinha nas costas, se parabenize e em vez de gastar seu precioso tempo livre argumentando, atualize o plugin. <- Isso sim poderia ser considerado cobrança :)


Perhaps move this to right before `if params.master_ip` below? This would allow us to eliminate the L149 `extend` call.


yes


I forgot to update the default here. I think this default is never used, because the PKB script always passes in a size. However, I updated to `"1024"` to be consistent. This does not accept the single size representation, and I've added a note in the help text.


Changed to 10 to match the benchmark script.


This has a diffuseFactor but not a diffuseTexture, so the comment above does not match.


Group all the roughness, etc.


Done. Thank you!


Delete the first sentence.


NOT "shall be the address", but : "is the address"


any chance we can remove this print?


Good job. I did see much less flickering at startup on Win7 32bit.


lgtm


Why can't you delete it and just check it in git later?


> As none of these new minor versions introduces backwards compatibility breaks

For example, zend-mvc breaks BC at 2.7.x.
Zend\Mvc\View\Http\ViewManager losts some methods.
ServiceLocatorAwareInterface was removed from the other classes.
Thus, old services raise E_USER_NOTICE.


Done.


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


107 = A12?


Just in case @Thecrazyskull has a bit of memory loss and wants to deny being a douchebag http://i.imgur.com/k1hrkjC.png


列表


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


@girving I put that statement in the setUp function for the test.  Double checked to make sure it is giving the same random numbers each time.  Is there any reason not to use random seed 0?  (I will force push later once we iron out the comments)


The replace now works on a single run not cross paragraphs or cross runs.
It's a method of run class.

Replace a single string on a sequence of paragraphs or on entire document is not so difficult, now I loop on all runs in my code (as I did with OpenXMLSDK).

Next and more difficult step is to replace a string that is "cross runs", this can be done at paragraph level (on my opinion). I'm thinking to use the paragraph.text. I let you know if you agree on my progresses.


This is made, but never used. The linter doesn't really like that...


By grouping the operators all together, an expression like `a + b * c` would be parsed (from left to right) as `(a + b) * c`, while it should be parsed as `a + (b * c)`.


Probably only remove this ` | K_IS | K_IS K_NOT | K_IN | K_LIKE | K_GLOB | K_MATCH | K_REGEXP` and leave the rest "as is" (see my next comment)


Yeah, thanks


**SUCCESS** - TeamCity HotRod C# Client :: Pull requests monitor - WIN_S2012 [Build 100](http://ci.infinispan.org/viewLog.html?buildId=37246&buildTypeId=HotRodCSharp_PullRequestsMonitorWinS2012&guest=1) for 6396809633bf909a9a6f7540994af9804d44f89f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:12:36


Can you rename `compute_bispectra` to `compute_bispectrum` to be consistent with the renaming of the class itself?


Where the difference between the test in (new) line 32?


to do: newest static piece should not go behind existing ones


Closed by a67e13efdb:

> Fix pills for Documents page


oh interesting, that makes sense. I'll revert this part.


Can we keep the link to our doc in there?  And 'Solution' is spelled wrong. (teehee)


Closed by 86bda5027

> Remove commented out code in adminacl


read notify would be better because it's the read that triggers the fact that the handshake is complete


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


Fixes #28


nope, missed that in the process of refactoring :)


String has only constructors that accept byte array. But if you don't want to add slices dependency here i can remove it.


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


Could you also document that using this parameter overrides all specs from above? So only the given config file is valid then.


Well, haproxy listening on the stats port means the health check is up and using nc and health check could  potentially complete against the previous binder/listener (not the newly launched server). Which is why I was checking that it was the newer "generation" of haproxy that was listening using lsof.

The newly launched haproxy server  sends a signal and waits for about 1 second (via retries) before it continues on
(https://github.com/haproxy/haproxy/blob/0ab8dd1b77c9358e51a74ca761cebca178d5a650/src/haproxy.c#L1684)

The SYN PR is basically holding packets coming into haproxy until the newer server is up, so that would reenable the packets after the check here.


Looks like we need an additional null check here. Try with
telephonyManager != null ? telephonyManager.isOffhook() : false


I'm going to guess you know what I am going to say about these....


done


This is a generic class, so it will be overridden only. for `TheKeySessionInterceptor` (not implemented quite yet) there will be some base logic to handle a standard way we indicate the need for a new session on CAS protected APIs


which magic macro ?


So I tried disabling linting for that one line, but it seems like the line actually *fails* the parse (since we're using a specifically ES6 linter).

It might actually be easiest to simply rewrite the function not to use the object spread parameter. I will push a commit that does so and if you hate it we can always rebase it out.

Stand by...


If `jshint_config` is `True`, everything above is useless. I would enclose everything inside the if for the `jshint_config == False` case. Also check then inside `config_file()` `self.config_options` for `None` and return `None` then. Because now you everytime generate a temporary file, which is not needed when providing `jshint_config`^^


The following doc(s) have been deployed to the [contactform](http://contactform-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/unsupported-modules-plugins](http://contactform-static-docs.pantheonsite.io/docs/articles/sites/code/unsupported-modules-plugins)


Love this separation


this would need to be overridden for the specific API.


Nit: I'd prefer to unnest this. If you really think encapsulation is needed here, move it to a module and its own file.


Utilizando el comando de Unix Grep se puede crear Expresiones Regulares útiles para extraer tag's especificas.
Juan Manuel Rosero Puente
0994117082


a483f180dd:

> Change css_header to style_bootstrap.css in globals.inc.php


[HZR] 2.4 Developer creates default display (right side article)


Projet annuel :)


Linting error


'Solution' is spelled wrong. (teehee) - [docs/articles/sites/code/unsupported-modules-plugins](http://contactform-static-docs.pantheonsite.io/docs/articles/sites/code/unsupported-modules-plugins)


CONSUME uses **instanceof** internally.
And because both Only and Not extend Modifier.

This means that the single line:
```Javascript
const mod = CONSUME(Modifier);
```

is (almost) equivalent to these 10+ lines

```Javascript
 this.only = $.RULE('only', () => {
         $.CONSUME(Only);
         return '=';
       });

       this.not = $.RULE('not', () => {
         $.CONSUME(Not);
         return '!';
      });

      this.modifier = $.RULE('modifier', () => $.OR([
                  { ALT: () => $.SUBRULE($.only) },
         { ALT: () => $.SUBRULE($.not) },
       ]));
```

The only difference is that you need choose which string to return
"=" or "!"

Which can be solved in many ways.
one would be:

``` javascript
function getModifierText(modifier) {
  if (modifier instanceof Not) {
     return "!";
  }
  else if (modifier instanceof Only) {
    return "="
  } else {
     throw ...
   }

  // in some rule
  const mod = CONSUME(Modifier);
  return getModifierText(mod)
}
```

This has the advantage of simplified(faster) grammar
and better error messages (Expecting **Modifier** but found "xxx")
But it may make the syntax diagrams less clear.

I'm sorry for being verbose :( but another alternative is to simply avoid the "single CONSUME rules"

```Javascript
this.modifier = $.RULE('modifier', () => {
              const mod = $.OR([
                  { ALT: () => $.CONSUME(Only) },
                  { ALT: () => $.CONSUME(Not) },
               }

              return getModifierText(mod);
       ]));
```

I think I need to collect these into a tips&tricks section in Chevrotain docs section. :)


there's an alternative using `locals()` that retrieves you all given arguments of the function as a dict. This could highly reduce the amount of code here, but having this list explicitly here is imo good. @AbdealiJK @sils1297 opinions?


No test for this extend?


2081-op-scheme-schedule (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)


LDLib
Курсовая по Программированию
Разработчик Криницин Антон, гр. 553502, БГУИР, ФКСиС, ИиТП, 2016 год

+ Тема:
 +    Создание библиотеки длинных чисел и операций с ними на языке С(Си).
+ Описание возможностей библиотеки
 +    Создание длинных чисел
 +    Простейшие операции над длинными числами(+, -, *, /)
 +    Некоторые алгоритмы, использующие простейшие операции над длинными числами.
+  Ход выполнения работы:
 +    Анализ и моделирование задачи(10% времени выполнения).
 +    Ознакомление с теорией по длинным числам и сложной арифметике(20% в.в.).
 +    Разработка и выбор алгоритмов для наполнения библиотеки(20% в.в).
 +    Проектирование основной структуры библиотеки(10% в.в.).
 +    Кодирование библиотеки(30%),
 +    Отладка и тестирование библиотеки(10% в.в.).
 +    Публикация.


Cool, was thinking that I could use the inheritance tree to simplify this.  Didn't get around to experimenting.


Just do the following:
```
# clean close
read [0x88 0x02 0x03 0xe8]
read notify CLOSE_RECEIVED
write [0x88 0x02 0x03 0xe8]
```
Then in the Gateway test you can check when the CLOSE_RECEIVED barrier is notified (by awaiting it) to make sure the timing is about correct.


I am watching you. Rox it!


BOOST_NOEXCEPT_DECLTYPE which does noexcept, decltype and stuff


Why are these `<a href` instead of `ReactRouter.Link`


well, it caught my attention when migration update started to download zendframework php code generator library, it was hardly understandable why migrations could possibly need it. But there I see a better diamond - a proxy manager framework for migrations. It was my rhetorical question, is it really necessary? Does it actually solve any problem?


Technically there could be a title and no other children, in which case `children[item].el_.focus()` would still throw an error.

And if we're going to check if `children[item].el_` exists, then we could use that as the test for a title too, right?


These should probably be indented four spaces.


Maybe "This script requires Python 2" would be more clear?


I dont see how running the bot into a camp of 5 gnolls at level 17 would be any good.
Search mob range?
Roam from Waypoint?

my bot got killed within the 2nd pull because he feels the need to take on 4-7 mobs at the same time


yall can use this in the GetHelp activity, I need to write a query for getting a random coping strategy and also I need to prepopulate it with some strategies first though


Commento per le, eventuali, notifiche.


Is this correct or it is a bug?
(100 & UART_RXFIFO_FULL_THRHD)
and
(110 & UART_RX_FLOW_THRHD)


I wouldnt do it like this


This was in prevision for OSX. But I guess we can skip it ^^


Why is firebase synced here ?


This line seems to override atts declared in 3221


Yea, now waiting confirmation from my client for Ruby on Rails usage :+1:


@girving I understand that ints do not have gradients, but the error message was slightly on the cryptic side.  I didn't remove the checking code, but moved it to compute_dx_and_dy (from _compute_dx_and_dy) to prevent the 1.0 here https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/kernel_tests/gradient_checker.py#L156 not calling the check.

The current test tests assertRaises "AssertionError", not getting a result.  But I'll remove it the tests and this commit if you still feel the same as before.  Or I can create a separate pull request for this.

Thanks!


Looks fine to me. Now we need to compare it with vanilla approach (`Kill_` method should be fixed in both cases) to see the real gain.


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


:laughing:


Aren't you executing before you check the view permission?


The related issue is https://github.com/robotology/icub-support/issues/195, not icub-main 195


Updated comments/instructions.


Loading I Frame into picturebox2


This is the perfect use of a class to create a general style you can apply in your markup. I notice in your html that you have applied this to divs which contain a tags. You're getting the effect that you want, but how might you implement this differently so that the div itself is centered instead of just it's contents? Notice in the inspector that the div takes up the full width.


Do you think period is good or should it be heartbeat_period?


Hi Mithendral. Yes this was a mistake. Fixed it now


This commit breaks the vert.x icon on the new project page.  Not sure really having an SVG icon here really provides benefit in this case?


Out of curiosity, is there no way to use apt-add-repository ? Hacking apt sources list seems a bit violent, no ?


You can just remove scheme: `//upload.wikimedia.org/wikipedia/en/6/6b/Yii-logo-transparent.png`


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


@kjvalencik Should we also test some absolute UTF8 string here? Such as '人人生而自由，在尊嚴和權利上一律平等。'?


knowing the ts video pid (physicalId) is a must for amcodec for ts playback. so should be re-added.


@opadron


them_ is wrong it must be theme_
just a small typo, that could case problems.


@opadron let me know what you think of this approach.


I just tested loophole (commit 9040e6) with a Polar A360 activity tracker, works fine, can connect to the device, "dump" it, "get" files selecively, "list" device info.
-> So you can add A360 to the "Tested with:" list.


This is an interesting solution to breaking the lines and it shows you're comfortable with the distinction between block and inline elements. That said, you'd run into trouble if your quote was 200 lines long. An even better solution would have been to set a width on the blockquote, center that within the #Bottom div and let the text flow naturally.


Something strange happened here.


68f8d8704f8:

> Remove inheritance on .small
Fixes the display issue with clinical reminders/appointments.


This change is resulting in .connector .fa-search being clipped.

<img width="858" alt="screen shot 2016-03-09 at 3 22 23 pm" src="https://cloud.githubusercontent.com/assets/895728/13649274/e4baa858-e60a-11e5-8955-c4449b452344.PNG">


Yes it's necessary and yes it solve a problem. You can look at the PR if you want more information #407.
Btw proxy manager is just a name, it's just a library to implement the proxy pattern. It's not a manager or a framework.


buf should be checked for NULL before get_task_comm call


can we provide a createTxSummary() which returns the new'd object?


You should also add

    mirror "https://bitbucket.org/eradman/ephemeralpg/get/ephemeralpg-1.8.tar.gz"


LGTM


why a surrounding div for the scripts?


Why such a big logic in `componentWillUpdate` ? This would block the javascript engine if `nextState.mails` is too large to be looped on.


you there?


Still too many ifs. You may assign `action = X.KeyPress` or `X.KeyRelease` and then check `ctrl`, `shift` and `alt` values. It should twice more compact.


This is terrific work! Your markup and CSS are immaculately formatted and the page is well executed compared to the comp. You're using block level elements and block styling with confidence.


Yeah @sg00dwin has a fix. See #7887.


Is this callback invoked when the setting are first loaded? If not, this will result in the ansi build setting being out of sync when sublime is restarted.


Please add more space above and below, it is not directly viewable to which block that belongs.


done and pushed in https://github.com/shtrb/marble-1/commit/5ec14881959c3518b224fd6e00785a426c56914a


I don't know. I'm not very familiar with Ubuntu and assumed there should be a way to do so, but those are PlayDeb's own instructions: http://www.playdeb.net/updates/ubuntu/14.04/#how_to_install


Probably a reminder email is good. I think a lot of it (this time) had to
do with door codes and student statuses.

On Wednesday, March 9, 2016, Ricky Mutschlechner <notifications@github.com>
wrote:

> Also +1 actually naming people (by name) who haven't done an event yet lol
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UW-UPL/Documentation/commit/4c113ec8134c0d4c5dc6a11d030eb41e120eaa59#commitcomment-16597888>
> .
>


Its unlikely that someone will have the time to dive in depth to a library he uses.
But at least when its open source software, it is possible to **directly** see how that library is used and draw conclusions (docs/api changes/runtime checks for common errors)

This is an **advantage** of open source I've not realized before. :)


This is appending to the file.


Nice! This is much better than processing the setting on every single build output change.


Before this change a new handler was getting added everytime you ajaxified into dashboard and then each time you navigated timers were reset multiple times.


INIT_LIST_HEAD should be called on &se->lh


Closed by 1b30c9c62:

> Remove demographicsOld and demographics_home


and what might that be? adding missing strings? That's not strange at all...


This should probably be set to "on_finish" since it is the default value.


ok. Then we can guess there is no easier way ^^


lol, nice spelling error


Oy that was blind traversal of the MIR type definition without thought about what moving out of statics would in principle actually mean.

(I'll have to go look at the pre-MIR code to make sure that whomever is in charge of rejecting attempts to move out of statics remains in charge of it in the post-MIR world, but you're right, this doesn't make sense.)


All issues should now be closed. Please review.


who is responsible for calling m_txSummary::log () ?


enlightenment 0.20.6


On it :+1:


So far we've said it's ok to retrieve the first record multiple times, why prohibit it unless the iterator has moved past it?


Added it, thanks :raised_hands:


Issue5276 (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1582 of 1582 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-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.87%25-green.svg)


No doc strings? Come on Phil.


console.log


use github page generator or learn html


I think this is wrong, as pointed by @mvitez. It should be `[s]`, if I'm right, this time.


Vet at @ehamberg  likte merge, mens @ljl og @iverds helte mer mot rebase?


Use ::


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


TaskCluster: @bhearsum does not have permission to trigger tasks.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#NqjbuObNQiSPS0GDTpatqg


Can we? Yes. Should we? Not sure.  I understand why this pattern is used for instantiating a concrete subclass of an interface, but we aren't doing that here. There are multiple examples of concrete classes, where the owner is referring to the concrete class explicitly, are instantiated by calling the constructor directly.


Use TODO or FIXME in such cases or this will be forgotten. Some IDEs display these automatically or you could just grep for them.


Add docstring


do we really need it? The next function is planned to be logged anyway.


Thank you @scanny for your answer and for the time spent reviewing the code.

I completely understand the need for planning carefully every move and deeply discussing and analysing every change, even the very small ones, like this one.
It would be an interesting experience indeed.

Unfortunately, it does not fit my current situation as it would require way more time than I actually have to dedicate on this project.
We have a deadline on next week, and almost all the code has been already written and is in its final testing stage. My plan was to contribute back by moving the already made code into the library, in time for the testers to verify it to ensure it is working, and to write standalone tests for it; then it could be refined and improved in future by somebody else... that's how many open source projects work.

I am sure there is still plenty of room for improvements and better design: there always is. Anyway, starting back from the analysis would require a completely different approach and, considering the amount of code to be analysed and, probably, redesigned, I think it could reasonably last many weeks: way longer after the project end date.

Once the project is done, the team will move over and I will have to follow.
I would love to keep contributing to this project on my spare time but sadly I already have some to follow and never enough spare time :)

I hope I'll get one more chance to contribute in future; when it will happen, I promise I'll reserve more time for this activity. ;)


This line (148) causes a System.MethodMissingException on mac, does windows need it?


I'm not sure about the name unless we're certain we want to turn this into proper streams in Java 8 (which I don't like as Java 8 Streams is really large API that will clutter ours, let's rather add a .stream() method for that)


Well, this stuff just happens :) It was an easy fix, no bad feelings.


you could also write a benchmark where `tenToNth` is calculated or cloned and not taken from a precomputed table. This way we can understand if Java optimizes loop where new objects are instantiated. Maybe we have such case (passing new object as extra parameter) in decimal scalars?


This url is wrong.  What is the right format for the url?  I can't get `https://api.mediacloud.org/api/v2/mediahealth?media_id=1&key=MY_KEY` nor `https://api.mediacloud.org/api/v2/mediahealth/1?key=MY_KEY` to work (both redirect me back to the core search page).


I am so proud of you


you're right.


Also: 1.9 was released this morning! Changes URL formatting for UNIX sockets which matches what SQLAlchemy expects.


I can see you're struggling to get this looking right. I think making the anchor tag a child of the h2 is causing some issues and recommend making them siblings.


isn't Values needed to build up parameters and hence should be in the root package? Or should we change that by auto-wrapping parameters into values?


true...


network-api-version is misspelled.


This file needs to be built out. From the assignment: "Be sure to include a section of the README to list resources or collaborations you used in completing your data structures assignments."


This fixes the thing that made my demo fall flat on Tuesday. I'd revoked the application's access to show the GitHub authorization UI, but BI hadn't refreshed the access token when I signed back in.


you mean version, right?


line1
line2


@bennett000 only change I did here was remove the `import reflect-metadata` (which would cause the issue in this setup also), but when removing it from ng2-redux and doing a build, the same error would keep happening. Maybe you can find something I'm missing