This is required for the next step of adding a local cache of banks referenced in the peaks file.

Previously these two functions were just global in the file scope of LoadIsawPeaks.h.

readPeak(), in particular, was then creating a new instance of the LoadIsawPeaks algorithm every time it was called (once for every peak in the file!) in order to call findPixelID() on it. This would prevent a local cache from working.

Now that it is a member, it can call findPixelID() directly. findPixelID() has also been made private along with this change.


ack


Использовать парагарф для каждой строки - сомнительное решение, по смыслу это скорее один параграф.


Awesome, I merged it. I _think_ that addresses all my concerns!


And because of that, you don't need to change `tftpboot_repos12_dir` below


You don't need to inherently invoke isinstance.

You might enhance code clarity by instead having a:

try:
....
except requests.exceptions.ConnectionError as e:
.....
except Exception as e:
....

It's a better way of accomplishing the same thing, as invoking "isinstance," you also are factoring in whatever Base classes that requests.exceptions.ConnectionError inherits from. The way your current code handles it, checking for requests.exceptions.ConnectionError could also return True for IOError or any RequestException from the requests package.

See: http://pastebin.com/SKVaXJaw for proof.


Turns out that the shared connection pointer does not need to be bound to the functor via std::bind on line 114. The connection itself can provide the required shared_ptr argument when it calls the receive message handler.


Make the final curly bracket of each method line up on the left side correctly


Line 21 does not close as well as the render function


Is there a benefit to `/usr/local/bin/setuidgid $USER` over upstart's `setuid $USER` and `setguid $GROUP`?  http://upstart.ubuntu.com/cookbook/#setuid


we don't use it in coala


what I meant was: It would be easier to write functionality around jdk's concurrent class and less 'errors' or corner cases to worry about.


We do not need local variables here, just do: `_cached_residual_values[type].push_back(_tmp_Re(0));` below. Similarly for the rows.


I think we want `17` here, and not `16`. Same for line 618. If not, the final validator will never vote (see line 377). This test is difficult to follow (and I think I wrote the initial version, so it's my fault).


ack then


Ячейки th и td вмегда содержатся в рядах tr.


I made a note about that in the other config - `setuid` and `setgid` are not supported in the version of upstart used in Centos and RHEL 6, which is sadly still quite common.


You're right. I simply moved a previous version of the code that was specific to uint8_t into this generic version. Now that it is generic, this check is needed.


The dual message check was just an oversight. I will fix. As for the transport class being created, this is something that was introduced and used by the other servers (i.e. websocket/rawsocket). We can address this in a separate commit.


Up to you! It seems like `numeric` or `categorical and/or date` is a common enough type to warrant its own type. Maybe rename to `coordinate`?


I think it will be a bit difficult to write tests, because we must compile the code for the tests, because we should test different groups, I will see how we can test that effektivly, maybe not on C++ Level, but on a bash level


Oups. Thx


.com offends me because .communist. Please change to .cap for .capitalist to appease my Western influence.


Where is the test that shows the "before" and the "after" condition? Where is the assessment of the compatibility damage which will be caused by a refusal to run a build if the getAffectedPaths() method returns an empty set.


Hier war ich mir ebenfalls unsicher. Ich habe mich für "Linie" entschieden, da ich davon ausgehe, dass es [Oma](https://de.wikipedia.org/wiki/WP:OMA)-verständlicher ist... Im Sprachgebrauch sind Kante und Ecke doch oft das gleiche. Ganz unabhängig davon sieht man auf dem Bildschirm eine Linie.


I'm LongXuanying, call me 18883385590


Also noticed yards was there twice which caused issues.


@lhyhsx 更多的是之前使用 NSOperationQueue 的时候 maxConcurrentOperationCount 也是1，而且每次有新的operation都会新起一个thread，然后结束后就被销毁了。因此并没有真正用到operation queue的优点。
用Runloop的话可以有一个常驻的thread来处理处理这些任务，而且这些任务一个thread也足够了。


lgtm


Fixed in a new commit just now.


Agree on the point that the character assassination and ad hominems in this thread weakens the point. If you're in a professional environment and your tolerance is so low seeing .bro impacts your productivity or sensibilities in some way, you're going to have a long painful time in a professional environment.


So, on the topic of "a server you can use with your browser", that turns out to be A Pain In The Ass™. The security requirements for HTTP/2 are high, which means you need a quite recent OpenSSL. Both the Twisted example and the asyncio example can do it (the Twisted one requires a Twisted patch that I haven't landed yet), but in both cases you need OpenSSL 1.0.2 or newer, which is comparatively rare.

I think it's worth documenting these requirements and what you do once you've satisfied them though.

Got suggestions for design patterns worth seeing? I can think of a few:

- coroutines, using asyncio
- event-driven, using Twisted
- no-shared-memory threading
- shared-memory threading (one threading example could use actual threads, the other could use gevent/eventlet)

Suggestions for more would be great.


Should be static.  Style: `CheckParameters`


I think we again have the same problem with reaction Id included into the rate law =(


There are


Hello Marin,

I am completely new to this process.  I was "watching" this file.  I looked and do not see a way for me to open a pull request at this time.  I remember seeing that option when I created the changes.  So if your request is directed to me, I'm not sure how to do that.  Sorry...

Jon


@brev Please do not disable rules we intend to keep, just downgrade from `error` to `warning`


As per your question about min size. You can use the attribute [area-min-size] on the <img-crop> element.
example: <img-crop image="image" result-image="menuCroppedImage" area-min-size="80" area-type="square" result-image-format="image/png" result-image-size="imageWidth" area-ratio="imageRatio"></img-crop>


I think, it would be nice to actually print echo'ed output back on the client side somehow. Don't you think so?


Laul.


Хубаво е да пишете коментарите тук, а не в кода


Just one more complication here: we have multiple keys by know (cern.ch.pub + cern-itX.cern.ch.pub).  They'd all need to be dumped to the temporary directory and delivered to the pull command as one string separated by :


Sorry,There are


<a href="https://twitter.com/mCabrera_gt" target="_blank">


Sounds good.


:+1:
Most excellent


Cambio aceptado.


non mais c'est loic qui c'est pas ce servir de visual studio il est pas content


Cambio aceptado..


Note: I understand these changed because we now take the floor when computing mThreshold (we used to round). I missed why we changed the rounding mode tho.


Hey, I would love to contribute on this project, can we chat sometime?


What happens now when a we call page.unload() on a settled page?


On 13 Oct 2015, at 14:34, Jonathan A Rees <notifications@github.com> wrote:

> oh I see, that comment is in the commit log, not in the release notes.

Yes, sorry. As you say, not a showstopper

> That's too hard to change, so I will leave it

If I remove ncbi ids when followed by a silva ID (reinstating only when there is no equivalent NCBI used elsewhere), then I get no duplicates at all. So it seems that all the problems come from the ncbi_via_silva ids.

Yan


The design of all MOOSE systems is that `computeQpResidual` computes this system's residual __contribution__. The contribution is then taken and either added or replaced in the above layer (typically in `NonlinearSystem`). What I see you doing is to take the original residual do something to it and return the value. This does not seem right.
Why are you doing it?


Лишние пробелы перед компанией. Содержимое ссылки не отбивается отступом и не пишется с новой строки - это строчный элемент.


As far as the changes/updates go, i will not be able to get to them at this time. I have seen sum bugs with the square but nothing that should stop you from using this. I have not tested circle yet and there may need to be some updates to that code. I will try to look into these things this week.


Цитату ты убрал, но надо что-то поставить на ее место, либо параграф, либо div. Нехорошо бросать текст овт так вот на съедение мнимым бокам.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1909 of 1909 tests passed)

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


time - строчный элемент, содержимое пишется в одну строку с тэгом.


I like the sound of `coordinate`, a lot.

@chriddyp would you mind making the change:

- ack for all `valType: 'any'`, change them to `coordinate`
- change `any` [here](https://github.com/plotly/streambed/blob/master/shelly/plotlyjs/static/plotlyjs/src/lib/lib.js#L1586) for `coordinate` and modif the description
- and that should be it


Ah, I missed that.  Thanks!


@iracic The `shell.welcome` property comes from the properties file you'd customize.

This is 2.11.8:
```
$ skala -Dscala.repl.welcome="hi"
hi

```


@brev This was part of the recommended [style guide](https://github.com/felixge/node-style-guide/blob/master/.eslintrc) referenced from the [README](https://github.com/numenta/numenta-apps/tree/master/unicorn#guidelines-1). If we need to deviate from this style then we should write our own version of the style document to avoid confusion


saw approval from gordonguthrie
at https://github.com/basho/riak_kv/commit/c9f905d486b4e7676bb354ad2fd004571357d73c


merging basho/riak_kv/fix/at/range_filters = c9f905d4 into borshop-integration-1217-fix/at/range_filters


basho/riak_kv/fix/at/range_filters = c9f905d4 merged ok, testing candidate = a734b380


Interesting. What was the measured overhead of a GL.IssuePluginEvent() call? It seems like a big deal for this to be introducing noticeable overhead, since its meant to be called every frame and you'd only be using it to improve rendering performance.


Provide comments with details about the component


Can you elaborate on the Unity "`GL.IssuePluginEvent()` overhead" you observed?


are you sure about this? All new FE styles (in site-core and categories) have not been using 4 spaces - that is mostly only in the stylus and I think the JS? I think maybe sticking to the 2 spaces is not a bad idea?


@NicoleGrondin @cldougl  - looks like you need to escape the text in this example:
![image](https://cloud.githubusercontent.com/assets/1280389/10458858/f1ff00f2-719a-11e5-8069-89ed889b8945.png), so replace `<br>` with `<br>`. Alternatively, If this code block is a variable, you can escape it with `{{ code_block | escape }}` (which I'm using here:  https://github.com/plotly/documentation/blob/source/_includes/reference-block.html#L105)


See https://github.com/brev/numenta-apps/commit/2de81e205fb935dbf9291f2cb5e74462b69feaf1#commitcomment-13743621


@tracernz You can put comments in the XML files, and in fact we already have done this for exactly the reason you state! https://github.com/TauLabs/TauLabs/blob/next/shared/uavobjectdefinition/stabilizationdesired.xml#L8


See https://github.com/brev/numenta-apps/commit/2de81e205fb935dbf9291f2cb5e74462b69feaf1#commitcomment-13743379


Yeah, I put the treading in LandUnit now at least. The current tread units seem to be inheriting LandUnit so that's a good compromise I guess. Otherwise we would have to import all GPG unit script files currently not in repository and change inheritance on those.


Tenhle fajl bych taky trochu obohatil


perfect- I'll fix that now


@ChALkeR thanks for pointing that out! :+1:


I have only 1 concern regarding this:

> Actions delete = new Actions(WDLibrary.getWebDriver());
> delete.sendKeys(webelement, Keys.chord(Keys.CONTROL, "a", Keys.DELETE));
> delete.perform();

Aren't CTRL+A then DELETE supposed to be 2 separate keystrokes?  We don't normally hold down CTRL+A when we press DELETE, do we?


Updated the pull request, incorporated review comments.


fixed in the new version!


Fix spelling in the commit message, please (m**e**ssages instead of m**a**ssages). Make commit message shorter by moving the description into the body.


What looked strange to me was the use of the form _"merge master"_ that has a specific meaning within the _git_ system, which I did not see in the actual commit, that's it.
Mine was just a curiosity.


Does the G5 supports the feature 0x64 (Time Between Reports)?
If so, then there is a huge chances that the scheme used by the G5/G9/G500/G700 is the scheme2 (not implemented her) and the second parameter is not the frame rate but the "angle snapping":
0 -> no change
1 -> disabled
2 -> enabled
3..0xff -> reserved.

The problem is that the documentation does not say when we have a scheme1 or scheme2. But it might be that if we have 0x64, we are using scheme2, if not, then we are facing scheme1 with the framerate.


reack


Remove echo done


Probably better to use warn here than STDERR.puts.


Like discussed it on gitter we keep it for now and open an issue for removing it again if applicable^^


Make commit message shorter.


reackj


Using HTTPS it is very difficult to hijack a session, i.e. there is no jsessionid passed around that can be used to piggyback on credentials. If an attacker created a hidden form, unless they somehow modify the HTML in the browser they can't forge requests in the authenticated HTTPS session. Without HTTPS it is very easy: just snoop the jsessionid and send a request with it, gaining the credentials of the account authenticated in that session. Similarly HTTPS is important, along with forcing parameters into the body, so that data to and from the server is encrypted (in the body it is, in the URL it is not).


hahaha that is absolutely accurate


It makes sense for the G9 given that its maximum resolution is 3200 according to your file. And given that the G5 is the same generation, it also makes sense for it to rely on the custom dpi encoding.


Ok, kann ich nachvollziehen - selbst wenn das mit der Kante vielleicht auch regional unterschiedlich ist. Danke!


why removing virtual keyword?


That makes sense. It would be nice if rationale was included in the commit message. Thanks!


- DuplicateMbox should live as a constant inside the culturefeed class. See https://github.com/cultuurnet/culturefeed-php/blob/feature/CUL-5/CultureFeed/CultureFeed.php#L92
- logged in check can be done via https://github.com/cultuurnet/culturefeed/blob/master/culturefeed.helpers.inc#L19 no try catch needed
- Don't use ! for user input. Use @ so a check_plain is done on the entered email.


write that it is public.


Yep, I actually did that as part of 2768cf3c75f002543854a75464fa7d7a5b0265ec: https://github.com/gfontenot/dotfiles/commit/2768cf3c75f002543854a75464fa7d7a5b0265ec#diff-03382ffc1aefad37248f7c6f82ed9c78R7


Included in the 1.6.5 release, which is now live on wp.org.


I thought we were going to do something like this:
```
_fe_problem.reinitNodes(master_nodes);
_fe_problem.reinitNeighborNodes(slave_nodes);
```
Then the constraint should pick up those values in `_u_master`, `_u_slave`.
```
nc->computeResidual()
// add the residual to the global residual
```
All logic how to process the contributions from `computeQpResidual` should happen in `computeResidual`.


delete


It's still used elsewhere


get it back


Delete this


alreadyClicked = false;


EmptySpace


seems the `ByteList` instance is completely redundant here and can be avoided


In Russian


Nitpick: /dev/kmsg is not really a socket (and /dev/log is a symlink)...


So containerFrame is passed in this method and not used? I don't see it used in the method.


No need to create a variable here


why this one is not virtual?


BTW

Why `options.render.call(this, input)`? `this` is going to be `undefined`, so that `render` method cannot access `this.state`. Is this by design?

Why `if (!!children.length)`? What's the difference between a truthy/falsy value and true/false in a context of conditional?


wtf? Where is need to create a class where you can do it in one method


;) VS makes it a one-click operation to fix everywhere though, so wasn't bad!


(I haven't found a way to comment on the lines that are not in diff.)

Why `options.render.call(this, input)`? `this` is going to be `undefined`, so that `render` method cannot access `this.state`. Is this by design?

Why `if (!!children.length)`? What's the difference between a truthy/falsy value and true/false in a context of conditional?


You have 2 `%s` in the message, but 3 elements in the tuple.


`override` implies `virtual`.


Bracket is placed incorrectly


I don't like that this is written using the ternary operator... a normal if check would be good too, and more expressive. Also, a comment would be nice, that explains why we do this hackish thing here...


És vajon ez működik?


Is this needed? IOW: is there any feature in yargs 3.27.0 not present in 3.8.0 that we now rely on?


awh, lame. Glad you found it


We are using 1.8 Java


* In Safari on the Mac: Bookmarks menu: Nearby
* In Epiphany (GNOME Web): Bookmarks menu: Nearby
* `avahi-bookmarks && xdg-open http://localhost:8080`


It crashes without?


Try not to add things that you did not change


So, is it expected that this code can be imported from two different places, and we don't know which one will be called first?


Actually, thinking about this, maybe it would be nicer to actually look for "Limit" in the string rather than chopping off a "Default".... Sounds more generic...


Faizal kab seekhega tu Front_end re .. hmmm  ?

HTML , CSS  , JS ke sath sath Bootstrap , kickstart sab  seekhega  re tera Faizal .


I use it so I don’t have to remember my computers IP addresses. I preview pages in different browsers and OSes from time to time.


are you sure that ````@type```` is  not needed here. I m not quite sure what titleInfo is doing as child of subject, but if we use the type to signal translation, as in the example below I could see uses for it.

````xml
<titleInfo type="translated" transliteration="chinese/ala-lc">
        <title>Luhang celiang riji</title>
    </titleInfo>
````


Java coding style


I think that specifying template type is not required
This remark occurs many times in this source code.


Did you have issues with the button?
Setting BUTTON_PIN 0 was fine to me, even before this patch.


k


@furhanshabir, thanks for the submission. I have a couple questions:

* How would one easily see what changes you made to the original?
* Does it work now? How would one know?

Thanks, and we look forward to seeing your responses!


This commit is just to share across machines, it doesn't need review yet @fengb


ack


need to do prebuilts/misc/linux-x86/ccache/ccache -M 50G
after sources are synced too


because the base implementation suits every need. This version is the client saying: "I don't care about the type, here's the value as a string - convert it yourself"


Because we don't need a cache in our case. Actually it doesn't matter 'cause of plugin runs only once, it makes sense when the same nunjucks instance works many times for the same template.


Didn't I ack that?
ack


This is not a fix.. `random` is unsigned. If you want a negative number, just do `-random(1234)`.

This "fix" may break other includes.


@rattoma Maybe I am wrong... When I merged ```master``` branch in the ```slice``` branch I obtained a conflict around these lines. As I found weird to cut the sample before the diagnostics I reverted this part. I will check.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (835 of 835 tests passed)

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


Faizal kab seekhega tu Front_end re .. hmmm ?

HTML , CSS , JS ke sath sath Bootstrap , kickstart sab seekhega re tera Faizal .


Must be typos in the code and README: **`Infulx`**


Does not work with Movuino 0.99.6, since pin 0 is also RX.


Controll should be Control


Hi is they a section where I can post all bugs I find?


`#pragma library` is meaningless in this context


Can be factorized as follow:
if isinstance(sourcestamp, (int, long)):


@jwaterfield yes that's a good point, I'll modify the image.


It is inserted when needed.

There is a button for this.

On Tue, Oct 13, 2015 at 5:28 PM, Duncan Paterson <notifications@github.com>
wrote:

> are you sure that @type is not needed here. I m not quite sure what
> titleInfo is doing as child of subject, but if we use the type to signal
> translation, as in the example below I could see uses for it.
>
> <titleInfo type="translated" transliteration="chinese/ala-lc">
>         <title>Luhang celiang riji</title>
>     </titleInfo>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/claudius108/tamboti/commit/ed2f78c6735b4be144a9533a0cc05ed8355d34b6#commitcomment-13744022>
> .
>

--
http://kuberam.ro
http://kuberam.ro/art


Nit: title the commit message using the imperative mood, i.e. "Remove" rather than "Remove". Item #5: http://chris.beams.io/posts/git-commit/


PVars in OnPlayerUpdate?? For a server with high rates or many players, this is not something you'd want.


aah, ok, I see. I thought pin 0 didn't exist :)


Actually if you input negative numbers into random (ex. random(-15)), it won't work.


Clever lol. I was looking for an actual substitution here with something like "String.replace()"


Het fixte niet :(


The third argument is a name, not a message, so it should be just "tableHandle"


Thanks.


yep, you're right


Greeting here is actually set to the string assigned to it. "Hello, how are you?"


It works as it should. It's unsigned. Look here:
```
printf("%08x", -15); // FFFFFFF1
```
So if you do random on -15, you will get a value ranging 0-0xFFFFFFF1.


OTOH if the boundary is removed in future there will be transferToInterpreter missing, I think it's safer to keep it.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2500


Ok I understand, it didn't know


I apologize if this is a silly question, I was wondering why this class has the long constant names with GT...etc. while the GTPageStyle has k ones?


@adynata ; @kbryerman

- Took out CDN versions of D3 and TopoJson from index.html.erb and are now included from app/assets/javascripts/application.js
- In the assets/javascripts folder, d3.js was renamed to map.js and put code inside $(document).ready

Does this help with the errors we saw in class last night on the console? I do not see errors in the console anymore. Check it out when you get a chance.


Same holds true as above. When you initialize a value, it's assigned to whatever is behind the '='. In this case, it's 40.


Add a comment:
```java
// TODO: This needs to be updated when we support dropping columns.


Oh yeah, good catch!


Grapje het fixt wel.. was vergeten code erboven weg te halen.


Maybe we want to simply use `data_dir.get_tmp_dir()` here?


LGTM


Huh. I assume you were still in need of a ui framework. What did you decide to move to that caused you to abandon efforts here?


I think you missed a word here.


Here I'm not so sure we want to use `.get_tmp_dir()`, since we might want that directory to be persisted.


Keep in mind that stats will update different if you double update it and not use a factor of 2...since stats are being compounded.

i.e

cmh.updateCMH(pos.moved_piece(move), to_sq(move), bonus);
cmh.updateCMH(pos.moved_piece(move), to_sq(move), bonus);

!=

cmh.updateCMH(pos.moved_piece(move), to_sq(move), bonus*2);


Why is this empty ?


reack


I actually have a tiny DNS server on my home network. `http://cassandra.kw:8000/` is a thing.


This code will need to change when we support drops, so let's make it more clear what is going on:
```java
TableColumn lastColumn = existingColumns.get(existingColumns.size() - 1);
long columnId = lastColumn.getColumnId() + 1;
int ordinalPosition = existingColumns.size();


The reasoning why use `.get_tmp_dir` is that thse random data dirs are very much meant to be a cheap and easy way to create disposable dirs for tests to use.


what's wrong with parsed-literal?


I would say, focus on your pull requests and I will make a pull request fixing all the warnings after this has been merged.


I'm thinking that we will/should do the same for:
talk-21 (v5.0 has targetSdkVersion of 23)
youtube-23 (v10.40.56 has targetSdkVersion of 23)
webview-23 (v45.0.2454.95 targetSdkVersion of 23)

thoughts?


You probably need to do the same triaging here based on the number of plots given the number of `axes` input that is done for `times == 'auto'`. Please add a test where `axes=` is passed in with e.g. 2, it should probably fail with your code currently.


Well, Debian should be listed first, as it is the parent distribution. :-p

But I will just be quiet for now. If we do hear of anyone having trouble on a specific distro, I am always happy to knock up an install of the same versions and troubleshoot. I doubt this will be necessary though. Java seems to cut across platforms very easily.


I know that is not the topic of this patch, but you should add a comment containing your explanation, to help the future developers that discover this code...


`axes=` with `times='peaks'` that is


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 663](https://teamcity.mesosphere.io/viewLog.html?buildId=58061&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 186 Build time: 00:08:44


Thanks. [26bb161ecb](https://github.com/Abramovic/logrus_influxdb/commit/26bb161ecb5b87f02c3efdf6f79be98ee1a40dd1)


You are inverting the meaning for the clause. It would be <> instead of =


This didn't have much of an affect on performance in this issue because it turned out that reading the file was not the primary cause of the long loading time.

Added this change anyway since the work was already done and CacheGrind indicates that this is indeed faster than the old implementation (typically by roughly 30% to 40%, depending on usage). It's still not an ideal implementation since the aim was not to modify behaviour in any way, including quirks (such as only one line ending character being consumed per call when loading a file with Windows or Mac line endings on a *nix platform).


usually you don't need the `./node_modules/.bin/`, because `npm run` will automatically look inside the `./node_modules/.bin/` folder for you


*TL;DR: *

What Paul is doing today for IS 8 and 9.Alpha2 is fine.

We could change that for Alpha3 if we (with customer feedback data) really
feel two URLs in Avail Software Sites is a legit concern.

*Blathering discussion:*

Here's what we do for JBDS IS 8:

https://devstudio.redhat.com/updates/8.0/integration-stack/compositeContent.xml

and for JBT IS 4.2:

http://download.jboss.org/jbosstools/updates/stable/luna/integration-stack/compositeContent.xml

So... yeah, we don't strictly need the JBT / JBDS stuff to be composited
into the same URL, if we make two assumptions:

a) that it's OK to tell users to use a SECOND URL for the IS bits, rather
than compositing the IS bits into the main JBT / JBDS sites

b) that the user has already installed JBT or JBDS from some URL and
therefore can resolve the TP bits that might be needed

That said, I believe (a) is no longer true, since Max has been saying for a
while now that it'd be awesome if instead of a SECOND URL for the IS bits,
we simply *add* the IS bits into the JBT / JBDS main composite URLs, so
there's only one URL shown in Eclipse / JBDS from which all the magic
happens.

Doing so introduces the possibility of an accidental overwrite (our release
guide needs to make sure we take into account that two different release
schedules are affecting changes to the same composite*.xml files) but
provides the user with a slightly cleaner experience... assuming that "too
many URLs in Available Software Sites" is an *actual* user complaint, not
just a hypothetical/cosmetic concern.

On Tue, Oct 13, 2015 at 8:40 AM, Paul Leacu <notifications@github.com>
wrote:

> I guess I disagree (respectfully). :)
> First - all of the supported install mechanisms work - JBoss Central
> Discovery, Eclipse p2 update sites, offline zip files (I'll even get the
> standalone installer to work tomorrow). Users are instructed to install
> JBDS first so it's already there when they go to install the IS. The
> exposed URLs are released URLs (development) and the IS is clearly marked
> as a development milestone Alpha2.
>
> https://devstudio.redhat.com/9.0/development/updates/integration-stack/
>
> https://devstudio.redhat.com/9.0/development/updates/integration-stack/earlyaccess
>
> This is the way Luna is set up as well and I think we'll cause confusion
> to users who naturally use the Eclipse 'Help > Install New Software'
> mechanism to install Fuse Tooling or Teiid Designer, etc. Dropping the
> Eclipse p2 mechanism or obfuscating it by combining it with core should at
> least be passed by the PMs (in my opinion).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbosstools/jbosstools-download.jboss.org/commit/744f1c0dcf31acb6c724a072d3e4fb499900798b#commitcomment-13739300>
> .
>

--
Nick Boldt :: JBoss by Red Hat
Productization Lead :: JBoss Tools & Dev Studio
http://nick.divbyzero.com


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 664](https://teamcity.mesosphere.io/viewLog.html?buildId=58062&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:07:48


Why? Also, why not pull in `<$>`?


@vilchik-elena it just tests that `complexityWithoutNestedFunctions` returns the complexity in a simple case where there is no nested function.


Sá eina villu.... í textanum hér:
Once this step is done you should have initialised an empty project and created the file ```package.json``` which describes your project.

Þetta á að vera bara:
Once this step is done you should have created the file ```package.json``` which describes your project.


also Mancy is pretty awesome :)


No, that's probably  true  even on shared FSs, thinking about it.  I mean, what better time to flush and free resources than there and then...


@benogle: I spotted this [line of code on master](https://github.com/atom/find-and-replace/blob/master/lib/project/results-view.coffee#L98) which seemed a bit suspicious and found out it was added in this commit. I _think_ the result of the `@lastRenderedResultIndex - @lastRenderedResultIndex` expression will always be `0`, unless I am missing something very obvious. In such case, please disregard this comment. :pray:

Otherwise, do you think there could be a typo or something here?


You should contact them and see if they would be interested in having your fixes submitted upstream.


Darren -
  Thanks for working on this.  Could you also commit a matching configure ?


I am fine with the functional change here.

I always look for opportunities to improve the state of the local code.  The obvious
things that catch my eye here are

1) the second/third clause of the third conditional are not indented "correctly" / "consistently"
with the other cases

2) The wrapping of the arguments to codegenAddrOf in the second clause is "unfortunate".

More tentatively I might have introduced some temp-variables with well chosen names to
cut down on the repetitive nature of, e.g., get(1)->typeInfo()->symbol->   I found I had to
carefully parse each instance to be sure I was thinking about LHS vs RHS etc.


Bad news is that running with this change ended up in a deadlock after 9 iterations.  I'll unravel and put details in the ticket.


Nothing wrong with it per se, but to me it implies that inline markup is used in that block (Because that is exactly what distinguishes `parsed-literal` from an ordinary literal block)


@jj-lucas Very throughly commented. Well done*

* Sarcasm may occur.


@maxogden Thanks, learnt something from you :+1:


Master is currently 1.9.12, can you bump this to 1.9.13 please?


Thats right about random but you don't get a valid integer (%i). And there is a option for default random native (BAD_random).

The reason i use PVars is for cross compatibility, you notice `t_OnPlayerUpdate[playerid]` which is used to detect if the updating part is already being done in any other script.

I may add FIXES_SINGLE for normal arrays instead of PVars as an option.


@podviaznikov I think this commit should be reverted. Filenames typically include the extension and this change broke the default slug function and my custom permalink function :cry:


See https://github.com/brev/numenta-apps/commit/2de81e205fb935dbf9291f2cb5e74462b69feaf1#commitcomment-13743621


implement hashCode and equals, toString would be nice.
The first 2 are important to make sure service options is equals to serialize/serialize version of itself.
Another option is to provide [`readResolve`] (https://docs.oracle.com/javase/7/docs/platform/serialization/spec/input.html#5903) and return `DEFAULT_TIME_SOURCE`.


Could you return the offer_id and the `appId` here as well? This would save making an extra call for each offer returned here. wdys?
What is this `id` here actually (it's not used in the `objectiveUrl`)?


Is it for easier coding that these are not methods and instead public to the interpreter? Just wondering.


well here we *do* use markup, why do you remove this?


TIMEKEEPER!


I am no expert in git.
I guess, it could be in the same pull request with a new patch.

Thanks

Laurent Perron | Operations Research | lperron@google.com | (33) 1 42 68 53
00

2015-10-13 7:42 GMT-07:00 TheCoolRob <notifications@github.com>:

> Yup. That is indeed prettier. VS also finds this acceptable :-)
> Would you like me put together another pull request?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TheCoolRob/or-tools/commit/9d5d74e55872339dedbc0ad0b95cea01e422a7c9#commitcomment-13742716>
> .
>


@vilchik-elena yes, but it's not clear in the code, and it's based on the squid index.


TeamCity pull requests :: netty [Build 405](https://garage.netty.io/teamcity/viewLog.html?buildId=2264&buildTypeId=pulls_netty&guest=1) is now running


because it broke the html output (check this code block on RTD)...


PR: https://github.com/hashobject/perun/pull/67


this code blows


I think the `-1` got left behind here.


dsa


.... and so if that earlier `-1` is gone, this `+1` can go too


Looks like those tests are making real requests, we should stub them out using something from here perhaps https://github.com/alphagov/gds-api-adapters/blob/master/lib/gds_api/test_helpers/publishing_api.rb


LGTM


Note that the "root" URL points to the usual top-level folder that contains `./META-INF/container.xml`. This requires the `filesystem:http://` fix in URI.js, see: https://github.com/danielweck/URI.js/commit/7db74757ae803a63ba55694d455afa995e8591df


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 665](https://teamcity.mesosphere.io/viewLog.html?buildId=58063&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:40


git gud


Off course, but that's a different commit is'nt it ?

2015-10-13 15:04 GMT+02:00 Séverin Lemaignan <notifications@github.com>:

> This should not belong to the commit, I guess
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexis-jacq/attention-tracker/commit/1bd9daf0330f657da05da32ff4727b68d800a432#commitcomment-13739936>
> .
>

--
Alexis David JACQ, Doctoral Assistant
Robotics Group, CHILI laboratory, EPFL


So, I can set a parameter outside the nominal range, and read it back that way.  But it gets used as set when computing computedValue which is then clamped to the nominal range?

If there's a nominal range, I, as user, would like to get notified that I set the parameter outside the nominal range instead of having it magically clamped somewhere else.


+1 Good to know about this warning too


.... and then the only place that needs a `+1` is here?


Viewer detected a naked '&' in the html - will replace with & in pull.


@pynicolas why new issue?


I have tested with
java -Dscala.shell.prompt="HHH" -Dscala.repl.welcome="TEST ============================== " -cp sc-dist/'*' scala.tools.nsc.MainGenericRunner

Prompt is correctly changed.
Welcome is *not* changed:
>> Welcome to Scala version 2.11.7 (Java HotSpot(TM) 64-Bit Server VM, Java 1.7.0_67)
[...]
>> HHH

Is "skala" using the same approach ?
I do not to see code in 2.11.8 (vs 2.11.x branch). How to check ?


Ok.  Leave it for now in this PR.  I can file an issue about this lower bound if it bothers me enough.


No thanks!


See https://github.com/brev/numenta-apps/commit/2de81e205fb935dbf9291f2cb5e74462b69feaf1#commitcomment-13743621


feed='changes'

@kroman0, яке значення "по старому" видає, `dateModified`?


Yes good idea !

2015-10-13 15:11 GMT+02:00 Séverin Lemaignan <notifications@github.com>:

> It is a bit problematic to hard code these expected targets of attention.
> What about having them in a separate json file, that you would load inside
> the node (the path to the json file could be set in the launch file, for
> instance)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexis-jacq/attention-tracker/commit/cd385150ebdaa6b3ecef40deca14b6eaa23aaa30#commitcomment-13740100>
> .
>

--
Alexis David JACQ, Doctoral Assistant
Robotics Group, CHILI laboratory, EPFL


editions/clientserver/tiddlers/Using TiddlyWiki for GitHub Pages project documentation.tid

Hi @tobibeer the "ReadMe" tiddler is linked, I think, isn't it?

In line 11 it points out that all this is done in `bld.sh`.


I was thinking about it. I just wanted to wait for updates for a moment that is sure that no new updates for the higher SDK arrive.
E.g. for talk is too early to know for sure that sdk 21 will nomt be continued with its own 5.x branch.
The same applies for YouTube, I want to see how versions develop for a bit longer.
WebView also has sdk 23 only since we fetched it from Marshmallow images. When a new WebView will be released I want to know which exact version is offered to Marshmallow users.


а тут умови вже не потрібно, досить просто передати feed, з параметрів


But please remind me in due time again ;)


fast-forwarding end-to-end/timeseries to borshop-integration-1217-fix/at/range_filters = a734b380


LGTM


Nice catch re hmac memory leak!


I cribbed it from the neutron code and added the `with_lockmode()` flag so I think it should do the right thing.  It does _seem_ to have fixed the race :-S


Also, I tried to fix the issue and ran the sonar analysis. Looks like the plugin just fetches the issues from the sonar qube server and does not analyze the local code.? Is that correct.?


By the way the latest version of this tiddler is here:

http://tiddlywiki.com/prerelease/#Using%20TiddlyWiki%20for%20GitHub%20project%20documentation


How special is this page exactly?


Profiling showed that the primary cause of the extremely long loading times was CompAssembly::getComponentByName().

Since this algorithm only calls that function to get pointers to banks used in the peaks file, and there are relatively few banks (15 in the sample file that was taking minutes to load), I have added a local cache of bank pointers along with a utility function to simplify its use.

The cache has the same lifetime as the LoadIsawPeaks instance, which should be destroyed once exec() has finished (thus clearing the cache and reducing the reference count on the shared pointers).

This commit takes the loading time of a MANDI peaks file with 6000 peaks from on the order of minutes down to a couple of seconds and allows this issue to be resolved without core modifications to getComponentByName().

A separate issue will likely be raised against getComponentByName() to deal with the cause of its poor performance in some circumstances.


@posita I had commented the code wherever I made the changes to code, but I realize that its not easier to view changes this way, so I committed the original file first and then made the changes, generating a diff file on github to easily view the changes: https://github.com/furhanshabir/VeritaseumTest/commit/419628494f59c0a48e1a9c54bfce6b36070b9d74

It does. In real case, I would love to write a test case in MochaJS for this. For now, one can hack into a browser console e.g. Google Chrome (Then press F12 to bring it up) and paste the entire code to check its validity.

Please let me know in case a test file is required.

Thanks
Furhan Shabir


well, that looks more like a style issue... but i agree, it's kind of ugly.

seems like this should be reported in the sphinx_rtd_theme issue queue...


Haven't tried it with node 4.x. Did they changed the url scheme in the new version?


yg


This is rather complicated. How about
```java
String columnType = sqlColumnType(column.getType());
if (columnType == null) {
    return;
}

String sql = format("ALTER TABLE %s ADD (%s %, %s %s)",
        shardIndexTable(tableId),
        minColumn(column.getColumnId()), columnType,
        maxColumn(column.getColumnId()), columnType);


Rony,

Don't understand the words but understand the issue. I assume you mean replace with '&'

David

On 16:59, 13 Oct 2015, at 16:59, ronys <notifications@github.com> wrote:
>Viewer detected a naked '&' in the html - will replace with & in
>pull.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/c-273/pwsafe/commit/b4b5b043b3bfa68418e891f52fc12cae7e93f7cc#commitcomment-13744863


console.log works both ways. When using commas, a space will be inserted between the strings, so no need to add a space at the end of the first string (or the beginning of the second). When using the plus sign no space is inserted between the concatenated strings.


@ktlim @jmatt Can we confirm that when you do a `git clone`, just having the `lfs.url` set in the **repository's** `.git/config` will do the right thing? I thought we needed the global config or `lfs.url` in order to bootstrap the clone?


subprocess has a `cwd=` argument.


Thanks @imikushin !


Yeah, you would pass a map in there to bind variable names to the actual variables, and the parser could then turn a string like "x*2+y>=10" into an actual constraint. It handles the four basic operators to build expressions, and the comparison operators <=, >=, and = to make it a constraint. No grouping though. Although it would allow you to do something like "(x*2+y)*3>=10", I didn't think it would be that useful.


What is this crashes after inserting the column but before adding the index columns? :)


I was thinking along the lines of "if `commits` *include fixup commits*". But yeah, I can see how that can be misread. Will update to `includesFixupCommits`. I think it reads better as a plural and it definitely [isn't wrong](http://english.stackexchange.com/a/146186/131011).


I usually run `git diff` for this.


On second thought; should *libcompose* itself be responsible for handling the override through `COMPOSE_API_VERSION` or the software *making use* of libcompose? @ibuildthecloud any thoughts here?


**comma-spacing**: There should be no space before ','.


**no-undef**: "describe" is not defined.


@vilchik-elena done


OK!!


you don't need exec here


What if `revision` is *newer* than `git_revision(...)`? We want to well-support that so that people can use newer versions of typeshed.


That is correct, it doesn't do anything and should be nuked.


And Django's extended support for 1.4 just expired on the first. I can submit a PR cleaning this up a bit.


can you remove it in a PR please?


Already fixed?


OK, I'll document that. You understood well. Also I propose to name the env var flag simply: ENABLE_POSTGRES_USER that would avoid a double negation.

I wondered if the var had to be prefixed by something special to show it belongs to this recipe. But a counter argument is that without prefix it could also be usable in Odoo one day. So I don't know. I propose ENABLE_POSTGRES_USER. Any better idea?


minor thing but all the alts are Nexttick.


А какой правильный?


Good eye, thanks again! fixed in 5534b2ee9ee2c71eaa447a0d90eea887aa4b0b9b


@Tolo0 https://github.com/PocketMine/PocketMine-MP/issues
Make sure to follow the contribution guidelines before creating an issue.


**semi**: Missing semicolon.


Offer suggestions: `git add`, or add to `.gitignore` or `.git/info/exclude` (which is actually somewhere else usually, so it is helpful to compute it).


>> * Does it work now? How would one know?
>
> It does. In real case, I would love to write a test case in MochaJS for this.

Is this not a "real case"? :wink:

> For now, one can hack into a browser console e.g. Google Chrome (Then press F12 to bring it up) and paste the entire code to check its validity.

How would one know this is your preferred method to test your code?

> Please let me know in case a test file is required.

I leave that up to you. This is your opportunity to demonstrate your skills as a developer, including your familiarity with best practices.


`cb` isn't necessary anymore because you're doing everything synchronously now that you aren't using `exec`. Would require a `major` version bump because it changes the interface.


Thanks but 'delete pcmd1;' has me stumped!

BTW Visual Leak Detector really is good. VS just gives you a list of leaks (coretest had over 90) and a minidump of each. VLD gives you a complete trace.  I just add '#include <vld.h>' in the main program when VS says there are leaks otherwise it is not included (slight performance hit with the tracking). You should install it for just these reasons.

David

On 17:02, 13 Oct 2015, at 17:02, ronys <notifications@github.com> wrote:
>Nice catch re hmac memory leak!
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/c-273/pwsafe/commit/8496558b676536be2edb5a336cfa3f800d30696d#commitcomment-13744962


Class has no instance fields, do you think `hashCode`, `equals` and `toString` would make sense? I don't even know what I am supposed to put inside those methods:)
I would go with `readResolve`.


Build tools 22 and above fixed this by creating separate layout files in
v17 and removing paddingStart/End from the default ones. We should still be
including both for now, and it will split things up for us accordingly
during packaging.
On Tue, Oct 13, 2015 at 2:33 AM Lars Grefer <notifications@github.com>
wrote:

> It seems some device manufacturers have used paddingStart in their forks
> of android before API 17.
>
> Lint says:
>
> If you are deliberately setting android: attributes in style definitions,
> make sure you place this in a values-vNN folder in order to avoid running
> into runtime conflicts on certain devices where manufacturers have added
> custom attributes whose ids conflict with the new ones on later platforms.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pockethub/PocketHub/commit/9206195f18addfcc98061aa53755df32c7a085d2#commitcomment-13735589>
> .
>


@christianwach - Sorry, I guess my thought was if someone adds new paragraphs to an existing commented social paper, the current comment bubbles would be in the wrong place because of the offset.

So my thought was you would want to move the entire comment bubble (and all comments associated with it) to its new paragraph position.  Moving just one comment and its children from the comment bubble thread would ignore all the other comments in that comment bubble thread, which I do not really understand.

I'm probably not getting the "orphaned" comment concept.  Can you explain it to me?


Could this be moved into updateMetaData so that we won't miss this?


typo?


I have an artist called M|O|O|N in my library, so '|' as a separator doesn't work. If "||" is found not to work for others, then at least now it's easier to change.


Master 1


Sure.   Well.. maybe not.

To test it I had to make other changes to my configure.ac because of another 10.11 incompatibility..

The old IST OpenMotif build is no longer compatible because it installs to /usr/OpenMotif which is no longer allowed.

What I did.. was to use “homebrew” and install a universal version of open motif in /usr/local

I can add that change as well.. but it would certainly break your current build system...

So.. we should talk about what options for open motif.  The two easiest options I found are through “homebrew” and “macports”.   In the past, I’ve also brewed and packaged my own version, which was previously not kosher.. but I see now it has finally been LGPL’d.  If we did that, we could install it in /usr/local/openmotif, or /opt/OpenMotif or something.

And also.. I see that in general we are building universal builds of mdsplus.. but the current uil will only produce 64 bit UID files.. so.. it will probably break on a real 32 bit mac.. (are there any?).  I do have a fix for that in my old fork.

also… fortran universal building… (wandering down the rabbit hole)

> On Oct 13, 2015, at 11:49 AM, Josh Stillerman <notifications@github.com> wrote:
>
> Darren -
> Thanks for working on this. Could you also commit a matching configure ?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dgarnier/mdsplus/commit/5b5f3a0f12cf47217526ea2a5aa2c73803059ed7#commitcomment-13744562>.
>


IMO, It is the same that to use **return** in the end of void function.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1909 of 1909 tests passed)

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


> Does the G5 supports the feature 0x64 (Time Between Reports)?

Yes. I have the "second edition" of the G5 (2007/blue&black version), it supports up to 1000Hz polling rate (1ms interval). I think I have read somewhere that the first edition is limited to 500Hz.

> If so, then there is a huge chances that the scheme used by the G5/G9/G500/G700 is the scheme2 (not implemented her) and the second parameter is not the frame rate but the "angle snapping":
0 -> no change
1 -> disabled
2 -> enabled
3..0xff -> reserved.

I tried writing every value between 0 and 255, and I always read 0.


can u please check the design sent by me. we will discuss more on this. i
cpuldnt call u tday as i am stuck in back to back meetings

On Tue, Oct 13, 2015 at 5:05 PM, AshwiniADeshpande <notifications@github.com
> wrote:

> desp is the description/self information put by user while logging in.
> sec_qun & sec_ans are the fields used for renewing the password if user
> has forgotten the pwd.
> rating is something which seeker will give to a user depending upon how
> the connect happened. I do not see anyother place where the user rating
> should go. Can u suggest somewhere else?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AshwiniADeshpande/VConnect-ghci2015/commit/451143b45acb72fb5418000d6f86e67d40e1908e#commitcomment-13737887>
> .
>


No reason to use the async form of `it` for this spec.


Offer suggestions: `git add`, `git checkout`/`git reset`.

Output a note instead of a warning for changes in the index.


Ну да, как-то натянуто, разве что для совсем зеленых новичков удивительно.
```
>>> class A(object): pass
...
>>>
>>> A() == A()
False
```
одинаковые же объекты.


Nice job, alignment is awesome! :)


Интересно, каким примером будешь иллюстрировать.


Well at least we coul test  F_7 and F_11 with multiplication or somethink like that.


Ah this broke my setup again, it has to point to the executable  `export GRAPHVIZ_DOT=/app/graphviz/bin/dot`


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


Only abort if the submodule is behind. Warn even if `dirty_stubs` is `True`.


Better to `throw` here instead of return a string; you're returning a truthy value on error or success, which can be a bit confusing.


`hashCode` could have returned 1, `equals` could have return true if instance of `DefaultClock`
and `toString` could have just printed the the name of the class and current time (though I agree, default
`toString` probably sufficient here).

Just providing `readResolve` sounds fine with me.


The underline needs to go all the way under the title (otherwise it is a unstable in the compile)


We should remove the last two lines from the robots.txt as well to avoid redundancy.


It is common way. If you haven't a specific need to do otherwise, you should use it.


@mavenugo you should rename Labels in libnetwork too then


I added the list of icons for CFBundleIconFiles and CFBundleIconFiles~ipad to be the same list.
Technically, we could probably clean that up to only include iPad relevant icons for iPad and non-iPad icons for non-iPads. But I have not found a good reference to create those two lists. And I have tested it as-is and it works as expected on iPad, iPhone 6, iPhone 6 plus, iPhone 5, iPhone 4, and iPhone 3GS. I have not tested this on iPad Pro.


Okay, will do.


Needs a way to override - needed e.g. for tox, or for a proper fix for JukkaL/mypy#915

Perhaps an environment variable?


:+1: done in another commit


Тоже интересно посмотреть на пример.


We can't quite confirm that, no.

The {{.git/config}} directory isn't versioned with the repository; you won't get some LSST provided version by cloning from GitHub. You'd have to add any local settings to that _after_ the clone.

However, as @ktlim pointed out in RFC-104, {{git lfs}} special-cases looking for a {{.gitconfig}} file in the repository root: details [here](https://github.com/github/git-lfs/blob/master/docs/spec.md#the-server). That's the behaviour this commit is relying on. And I confirmed it works for me, at least!


ROFL


Only abort if behind; warn even if `options.dirty_stubs` is True.

`default_data_dir` is buggy in JukkaL's version.


And those groups just with some elements missing or some of them twice.
Since both groups are cyclic we won't obtain any subgroup, so it should tell us, either it is no group or there are some equal elements, but besides it is a group.


you da best <3

On Tue, Oct 13, 2015 at 2:54 AM, Howard Chung <notifications@github.com>
wrote:

> brought back teamfight participation for @nickhh
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/yasp-dota/yasp/commit/bdaa71270ed4ce15a283fba47777ac90e6ebebe2
>


Done


@luigidellaquila Wasn't better to extend OBasicCommandContext class adding resultsProcessed property there?


The link did not appear as expected.  Commit: https://github.com/boss-developers/oblivion/commit/65e8774a8cee64672f9acaa0d3eca14fe524e03d


Perhaps this should `.strip()` the output since nearly all the callers do this, and the others should be fine with it.


Hi, yes they changed it to the same as they have in io.js. It's an easy fix, just have to parse the version number and do it the io.js way if version is 4.0.0 or higher. But I did not have time to fix it properly today.


Actually also made a pacman display instead of the percentage for uploads / downloads


Yes, the issue has been resolved.


Yeah, I know that. I just talked to @terrajobst yesterday about proper way of generating the packages. We should fix it soon.


@czue this change appears to cause [`TestCommCareUserResource.test_get_list`](https://github.com/dimagi/commcare-hq/blob/12403571eb65c040411bd71056cbb629f8dec094/corehq/apps/api/tests.py#L337) to fail (the newly created user is not found) when run with a properly configured cache. Somehow it's passing on Travis; I haven't figured out why yet. Do you have a suggestion about how to change the test or possibly this code? The test could invalidate the cache, for example, if it is acceptable for the API to return stale data.


I think you missed a couple of words in two of the bullet points. My best guesses:

 * We cannot reasonably migrate to Django's new migration system as its not compatible with South and there's runtime **no** transition layer -- and we're not going to be the ones to build it.
* In the future Sentry's primary datastore **will be some NoSQL software** and while we may offer a simple SQL implementation it will be entirely different than what exists today either way (thus no compatibility needs to be maintained).


It's odd that you needed jasypt to be manually imported. I was able to get it to work when I had it as org.jasypt. We can figure this out later.


Whoops. I mean with having the config only in `.gitconfig` not `.git/config`. I'd missed the new discussion on the RFC. I'm trying a clone now without a global config.


could you give an example on how to use this for us mortals?

I've tried allsort. I'm trying to make a custom close slide button


wow! great!


Took a quick look at the CSS, and it looks different because it's not a highlighted block. So this might actually be intentional and be more of a problem with sphinx not highlighting the parsed-literal block...


It appears twice here, I don't think that's on purpose


ここは、以下を参考に設計していく
https://github.com/Hiroyuki-Nagata/fswiki/blob/master/lib/Wiki.pm


fantastic. do we need String_from_boolean too?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 666](https://teamcity.mesosphere.io/viewLog.html?buildId=58069&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:05:39


@vilchik-elena because we now also check constants.


This is pretty cool.
Why does lint report this as an error then?


Done


@rodrigocruz

This is not a bug, you can be sure about the `url()` helper by reading this:
 * http://laravel.com/docs/5.1/helpers#method-url
 * https://github.com/laravel/framework/blob/5.1/src/Illuminate/Foundation/helpers.php#L709

Thank you for your efforts :+1:


This isn't a bad way to handle overrides.


It's something I experimented with earlier that seemed to reduce test failures. It might not actually be necessary (thought it was odd for it to make a difference, since django 1.8 says app_label isn't needed for anything anymore), I'll try undoing this change.


Ah, good to know this came from pdfbox. I filed an issue there to track getting it fixed there: https://issues.apache.org/jira/browse/PDFBOX-3022


@mislav I agree, it's not a good commit message.


I think we may want to use "DPN" rather than "DuraCloud Vault" here, to avoid confusion, since DuraCloud Vault includes the DuraCloud UI.


How is that different from base method ?


remove space


这一行是一个repository的标题吗？


Сначала условие, потом варианты результата.


Thanks Mark


Basically, what ```FromAY``` does is that it creates a mess. It takes data as given in parameter and updates the list using converted values. So, its parameter contains a list as read from the profile and then newly created ```NetworkInterfaces``` like map.


Hmm, is this right? I mean, we support translation in both ways, no? And then the compiler shouldn't like the double mapping of the key ids...

Or in other words, we should probably only support the aliases for the translation from name to id, but not the other way?


```
>>> reversed([1, 2, 3]) == [3, 2, 1]
False
```
WAT?


Don’t you need to reset the lexer after this?


Без примера. Просто субъективные ощущения и набросы.


FIXME: remove lastEmptySlot in here


I guess some json rpc response has this field even when the response is OK. but it is null. that's why maybe a test of null value is needed also?


Будет.


This doesn't build. Type has since gone away. Do you need to do another rebase?


TeamCity pull requests :: netty [Build 405](https://garage.netty.io/teamcity/viewLog.html?buildId=2264&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5623, ignored: 21 Build time: 00:38:09


Apparently this doesn't fix it


Mathieu mentionned you had a better fix for this?


"uat", thnx, autocorrect.


@tuxoko even in the error case we must call `unlock_new_inode()` to clear the I_NEW flag.  Otherwise in `iput()` final we'll trigger the warning because I_NEW is set on a inode being destroyed.

```
static void iput_final(struct inode *inode)
{
        struct super_block *sb = inode->i_sb;
        const struct super_operations *op = inode->i_sb->s_op;
        int drop;

        WARN_ON(inode->i_state & I_NEW);
```


Why? I don't have a git pull request. I'm attaching my comment to the most recent commit of the file that I'm commenting on. My comment isn't about an issue of your script. I'm basically summarizing the steps I had to take to adapt bashrc.prompt for my linux box, which isn't Debian/Ubuntu. Your script is very well structured, so it was easy to find my way around and fetch the missing file.

About "Assuming curl is installed". Well, if it isn't and git-prompt.sh is missing nothing is gained but also nothing is lost with respect to how the script works now. As you commented in the script, if git-prompt.sh isn't found "otherwise let's hope". Cheers!


No it's the Ricin user name and status message, not the one friends. :)


Remove the @ so i don't get constant notifications.
Thanks


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1370 of 1370 tests passed)

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


updated patch set LGTM.


Providing life best ways possible


this commit undone commit aae2151, or is this intentionally? seeing from commit message, i dont think so.


doing great thanks


Yes, but currently the only way to create a new Friend object is to accept a friend request, we want to be able to add a friend and see it in friend_list, also having friends not connected that comes online to be added in friend_list. :x


Looks good.


Sorry. I'll respin it after work. I thought I did a build test, but apparently I
had some stale build files or something.

On Tue, Oct 13, 2015 at 09:33:59AM -0700, Caleb Anderson wrote:
> This doesn't build. Type has since gone away. Do you need to do another rebase?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bwidawsk/engineers-nightmare/commit/8dfd607d82744d04d2d8aae447bed145b61e7074#commitcomment-13745676


delete this pls


Just spotted this because I've encountered #416 locally, and I thought it'd be worth point out this isn't actually a regression test - it passes on 3.6.4 (which I presume doesn't include this change). This is because the `contain` here is actually the contain method on `TraversableBeHaveMatchers`. The issue only occurs if `contain` is actually being called as StringBaseMatchers.contain to generate a `Matcher[String]` (this leads to the slightly weird situation where both `Seq("foobar") must contain("foo")` and `Seq("foobar") must not contain("foo")` pass as things are). To get a proper regression test you have to invoke it as `Seq("foobar").must(contain("foo")).not`, to make sure the implicit is applied by the compiler.


Done in latest commit.


@sakibj we want sidebar to be a fixed width


This stuff should go in the mingw package.


This also brings the LOOT Oblivion masterlist in sync with the BOSS masterlist as of https://github.com/boss-developers/oblivion/commit/f14c3f0ad9a29397720a0dbed6cffad4969c68be in regards to Bash Tags


FYI I think this `--libdir` invocation *probably* accomplishes nothing because bugs, but I'm too terrified to change it now.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 667](https://teamcity.mesosphere.io/viewLog.html?buildId=58070&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:10


hmm, i am not sure how i feel about this one. I mean, I see the usefulness, but it sounds so specific to bind this to exactly "unsigned", and nothing else... Maybe this should be a macro instead, and then we use it once for unsigned, where we use "safe_atou()" as parser, but support this for other parsers too. Similar to how we do this for DEFINE_PARSER() in conf-parser.c for example...

But I figure this all doesn't matter, and we can still turn this into a macro when we actually have another user of this, that requires a different type....

Hence I figure it's OK the way it is now.


Personally, I'd have put it there from the beginning, instead
of OCommandExecutorSQLResultsetAbstract.resultCount, but we can talk about
this

Luigi

2015-10-13 18:21 GMT+02:00 Luca Garulli <notifications@github.com>:

> @luigidellaquila <https://github.com/luigidellaquila> Wasn't better to
> extend OBasicCommandContext class adding resultsProcessed property there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/orientechnologies/orientdb/commit/40a8a15d185172e495f6bab95f082e1fb14d4fda#commitcomment-13745414>
> .
>


Тута у меня скрипт в знаках больше-меньше


@pedro-mendonca, @grafcom: would your languages need to have quotes? Which languages do you think quotes are needed for?


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 tests passed)

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


Тут функция с именем и количеством ножек у животного (которое и есть функция). А имя и ножки это переменные


@whyi This commit is very old and this code has been changed. Even assuming there was an issue in the code and the issue still existed today it would be very time consuming to check and respond to comments made on commits this old.


Тут тип это имя приравнивается к имени


oh


Nice!


I'd like to suggest you undo this - I did an rm for a reason there, which was to stop this from happening:

1. `somefile` is added as an asset.
2. `somefile` is referenced in two files.
3. `somefile` is later renamed to a more descriptive name,
4. but the name `somefile` is only updated in one file, or not in either, and stays in regardless

Under this method a file may be accidentally not fully renamed.


Hmm, how is "-5" and "5-" parsed currently? afaics -5 would be a parse filure, but "5-" would be parsed like "5-5", but is that really desired?

I figure this should be added ot the test case at least...


How about "Privacidad al máximo nivel"?


lgtm.


I might change it later.


I will suggest to return the response in the format:
{"token": <token value>} instead of returning the token directly. Same applies for the login API also.


Why?

I mean ... it _works_.


shouldn#t throw this an EINVAL if l > u?


Est ce que ce sont bien les rapport jasper tant espéré ? Si oui merci beaucoup


How about naming this API as api/v1_0/registration ?


@ricardclau Which feature from ansible 1.8 is required to run ansistrano?


If this is over 2 on localhost, something is probably wrong. It should vary from 0-1, but allowing it to go to 2 would have the test pass even with a 1 second GC somewhere during the test.


This is not a translation of the original text. If you want to change the text used in the `WiringEditor` class, you have to edit it, then the translation can be updated.


If Integer type is not enough for you, then use Long instead. It will make your life easier


Suggest refactoring the 10.0 out to a constant (2.0) in case it becomes floppy (unlikely).


My language is German.


should be `cl-loop` btw :-)


We should be consistent with this one. "Quick Share" wasn't translated on other web pages: https://transvision.mozfr.org/?recherche=Quick+share&repo=mozilla_org&sourcelocale=en-US&locale=es-ES&search_type=strings


hmm, something should verify that lower <= upper I think. probably parse_range() itself, but if not, then definitely parse_cpu_set_and_warn()...


Either one will work for me. If nobody else says anything, I'll change it.


@den2042 did you comment @manikitos patch on purpose?


@aweisberg Thanks for completing the test for this. Just started a new job so it's been a challenge to get back to it.


you can use "new ArrayList<>()" instead of "new ArrayList<BigInteger>()".


Eek. Maybe just retry the command with a timeout? (Sort of what NetworkManager-openswan does.)


Unless I’m mistaken, the `new BufferedTokenStream(lexer)` you’re passing to the formatter below will just yield EOF repeatedly. The formatter still works, yes, but any comments the converter emitted will be lost (does the converter keep comments?), and its line breaks won’t be taken into account either. You might as well pass `null` for the token steram to the formatter.


sleam gardaş


You can use constructor without generic. Something like this "new ArrayList<>()"


yes, please hold on
rate law should be fixed soon
working on it
Denis
On 13.10.2015 18:47, Martin Scharm wrote:
>
> @den2042 <https://github.com/den2042> did you comment @manikitos
> <https://github.com/manikitos> patch on purpose?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/whole-cell-tutors/whole-cell-reduced/commit/ac12ba924aef4feefccd7bede19fbb7e52ea61a5#commitcomment-13745944>.
>


@KrzysztofCwalina For MSBuild, we're using @onovotny's [NuSpec Reference Generator](https://github.com/onovotny/ReferenceGenerator) to automatically keep the dependencies in the nuspecs up to date.  It's worked great for us.


Can you make these more clear, like `OldestFirst`?


This is due to bad design of qBittorrent code. See ```LabelFiltersList::handleNewTorrent()```.
Once I get to that.


Thanks, I think you're right - I'll reopen #276 which I had unhelpfully tied to #272.


Nit: FOSDEM


You must use positional placeholders in python 2.6. `{0} {1}`.


I am not sure of python constructs, but does the "e" object has the whole exception or only the message, in Java we have something like e.getMessage(). We should return only the friendly message back, not the whole stacktrace.


Hey @joepadmiraal  thanks for this, but you committed a changed version of plugin.pro !
So unfortunately this is not OK.
Can you please re-send without that file ? Thanks !


It's better to catch more concrete exception for example ArrayIndexOutOfBoundsException


Hmm, looks like someone forgot to complete this function :)


Err... And what was the reason not to translate it?

I've just checked that Quick share does not appear in product strings, so no problem in that side. Ii just puzzles me to keep the term untranslated.


OK, I think I am misreading the spec here. The "frame rate" must be the one of the sensor, not the reporting rate of the mouse. That would explain why the values are between 1000 and 3000.

> I tried writing every value between 0 and 255, and I always read 0.

For scheme 1, the only correct values are 0...5 and 0x80...0xff (included), and for scheme 2 you only have 0...3.
Though the mouse does not answer back anything so it's difficult to know which parameters are valid.

In the end, I think we should just ignore byte 1 and say that it should always be 0. I am not sure we want to export the frame rate of the sensor and or the angle snapping...


Right version of OpenSSL to compile successfully in Xcode 7


Sorry, it's #13586 - thanks Victor for pointing out the solution.


This looks great. I pulled and tested. Also, the unit tests seem to be all fixed up. If you're ready, create a pull request, and we'll get it into master soon after this week's build.


If we need to modify XML during form creation we should probably convert this to a node script and parse the XML or handle it in the API endpoint.


For such simple task better use simple array instead of List.


Yeah, or just go with the old syntax ;-). Figured that out pretty quick.


"last_name" and "phone" are also required fields. Also move the closing bracket in L33 to previous line.


If you need to read only numbers from console, then try to use Scanner::nextInt.


OK
Regards
Ravi
On 13-Oct-2015 10:06 PM, "engstk" <notifications@github.com> wrote:

> Remove the @ so i don't get constant notifications.
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rachanta/kernel_n5_racer/commit/cb8945690bc2f0b5e130063fab2aaad717e5cc95#commitcomment-13745745>
> .
>


hmm, right, we're using both cl.el and cl-lib (`cl-list*` on the next line) which is pretty silly. I'll do a pass later to change all cl.el stuff to cl-lib.


Let's rename "body" to "request_data" to be consistent with the login APIs and future APIs we write.


Todays tomorrow wishes


oh, this is because commit aaee2151 wasn't done properly. The www/store-android.js file shouldn't have been edited directly, as explained here: https://github.com/j3k0/cordova-plugin-purchase/blob/master/www/README.md

File `src/js/refresh.js` should have been edited instead. Then changes applied to store-android.js by running `make`.


What's the reason for this bump? Even in face of the current spam attack, 2000 sat./KB is enough to get txns confirmed within a couple of blocks. I tested this using bitcoinj / TestFeeLevel.java.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 668](https://teamcity.mesosphere.io/viewLog.html?buildId=58085&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:16


We should return HttpResponseBadRequest here.


Looks like here you are showing them what they selected, but not actually printing the entry.


Hey, can you upload a screenshot of the .dia file? I tried to open it with Dia but it's not working. I'd like to see the diagram.

Also, being able to text the lists to other people would be convenient if someone else needs to shop for you (plus Dr. West said that the projects would probably be turned into phone appls)


If you can create another branch just with this commit and send the pull request I will merge into my branch and publish a new version on npm.


yeah don't worry, but unrequiring `cl` would be nice imo


I liked what you did here, please send a pull request with these changes


> Why?

Because `Control.Applicative.many` conflicts with `Text.Parsec.many`. Maybe I should just do `hiding (many)`.

> Why not pull in `<$>`?

...ha


Will suggest to return HttpResponse(status = 201, "User Created") here.


This seems simple enough but is failing tests on auth in travis.


Are your tests passing here? If not, it's because add_entry automatically puts the entries in in alphabetical order. So in your tests for the assignment, you need to reorder those, expecting the first one to be Daphne.


Are the .html files necessary in the repository? Aren't these dynamically generated?


Use wire.MaxTxInSequenceNum


I think this is just for testing. And I don't think we need to respond with a proper error for server side validation as all the specs will also be validated on client end. Client has nothing to do with server side validation. We need to produce an error on client end saying "something went wrong" instead of showing him the trace, or even a short message of the trace.


unindent those two lines above


I think pip needs the `--upgrade` option or it will ignore the existing package.


This commit does not add much value to the code base, but I am ok with change the tests names functions


acepto


OK, thanks for the note. I'll drop this change.
...It's quite easy to mess things up in current code.


ok esta bacan. lo fusionare con mi repositorio.


Consider defining `maxArea` in meters so we don't need to convert units later.


Commentary worth adding before publication:
It is worth commenting that you are "rolling the dice" to determine actual transmission duration *before* actual transmission, while the real world would do this when actually transmitting.

The main place this distinction can matter in the real world is with link aggregation. Because this simulation only models the network when all links are fully working there is no harm in this simplification.


As this is the bridge/snapshot application, which may be used outside of the DuraCloud Vault context, I'd be inclined to use a more generic system property for configuration. Also, since it appears that the work directory is nested under this directory, we may not want to call it the content root dir (as that implies to me that this is only for storage of content files, which wouldn't include application processing files like may be placed in a work directory.)

Maybe bridge.root.dir or duracloud.bridge.root.dir would work better.

On a related note, we may also want to update the current "root.username" and "root.password" system properties to fit whatever naming scheme we select here, so there is consistency during deployment.


Thanks


I think this commit might be incorrect


Por enquanto ignore os erros de português \`-\`


Unclear what this field is.


Ok this is clearly a project for basic scripts.

Nevermind me, I just wanted to point out that you don't know how the random function works.


Will do that


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 669](https://teamcity.mesosphere.io/viewLog.html?buildId=58086&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:08


I'd thnk the max should be 2 frames - 1 byte


Perhaps "mismatched block height and transaction locktime arguments"


Do we want to add a getter for the counts? Looks like something we will want to expose at some point.


Something along the lines of `utils.drawPolygon(map).then(validateShape).then(...` might be nice, since it would let us reuse the shape validation logic in the other 2 locations.


Drive-by comment here:

Per U.S. Copyright Law (the MIT License therefore applied), a copyright notice should contain the first year of publication.

> The notice for visually perceptible copies should contain all three elements described below.
> ...
> 2. The year of first publication.
> ...

https://en.wikipedia.org/wiki/Copyright_notice#Form_of_notice_for_visually_perceptible_copies


fmt without any formatting is a little silly


Just curious, shouldn't that be `ToList()` rather than `ToArray()`? From my unterstanding, `ToArray()` creates a list first anyway before copying it to an array.


Neat


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 670](https://teamcity.mesosphere.io/viewLog.html?buildId=58087&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:07


I'd just say that the transaction input is finalized.


Does it help anything to defer this call? The lock has to get freed before the `.Wait()` can return anyway, so if we release the lock before signaling it's more likely that the waiting thread can run immediately?

I am asking this mostly because I am curious / don't know, not that I think this is actually important haha.


What about relative url with ..?


> Appends *other*


Also, use F1 to debug boundaries.


Same as above.


> first key-value pair in *the* hash.


why isn't this omitempty? just curious


If this Cond object only has one reader and one writer, can it just be a boolean channel?


LGTM


@mistercrunch:  Just finished the unit test.  Everything should work now.


Thanks for the change request, but this should already be addressed. The dendrogram will only show if row_nodes and col_nodes have the group key

config.show_dendrogram is set here in Config.js
```
config.show_dendrogram = Utils.has(args.network_data.row_nodes[0], 'group') ||
  Utils.has(args.network_data.col_nodes[0], 'group');
```
then config.show_dendrogram is passed to params.viz.show_dendrogram and the dendrogram is only made if this value is true - this is in the Viz.js file

```
// add group labels if necessary
//////////////////////////////////
if (params.viz.show_dendrogram) {

  // make row dendrogram
  row_dendrogram = Dendrogram('row', params, row_triangle_ini_group);
```

As an example of a visualization without a colorbar try changing the network json in load_clustergram.js from default_example.json to harmonogram_example.json.

Please let me know if it is still not working.

Also the javscript files under src are being combined using grunt into d3_clustergram.js, so you should make changes to the source files rather than d3_clustergram.js

Cheers
Nick


How about, "Returns the first key if it exists; otherwise, returns nil" or something like that? Sounds a bit like it randomly alternates between returning the first key or nil. ;)


**the** hash


Just for you mate :wink:


Similar to `#first_key?`**,** but returns its value.


You have two ```__le__``` methods.


+1 for the naming


It would be nice not to suggest the user to use 'const' where it's not applicable.


>A dumbass online gamer who also hails from Brazil but is ignorant of rules and disrespectful of fellow gamers. Uses his or her lack of knowledge of the English language to try and get away with breaking rules

What a great change!


Comma => Similar to `#dup`**,** but duplicates the value as well.


I'm not sure this is a good idea, since we reach out and listen to other globals. My thinking was that we would wait to interact with anything outside of the `AtomEnvironment` instance itself until start is called. Maybe we should rename that method though.


Ah wow! I don't know how I missed that add_entry would put them alphabetically.


Hey @nicks and @bolinfest I was reticent to make a pull request for this JSX support for two reasons:

* React-style js is not yet natively understood via latest shipping plovr
* The browser.js script currently causes ['Method too large'](https://bugs.openjdk.java.net/browse/JDK-8136834) error in currently shipping JDKs 8 and 9, so you'd need a custom JDK build until [8u72](https://bugs.openjdk.java.net/projects/JDK/versions/17669) becomes available (even in early access).  I am running this JSX support with my own custom-built JDK that incorporates this method too large fix.

At the same time, this commit its "mostly harmless™" in that the babel compiler would only be activated for .jsx files.

To fully leverage the JSX support I added, you'd also need to also add this [compiler pass](https://github.com/rgpower/react-closure-compiler) I forked from @mihaip (to teach the closure compiler about React js syntax).  It's forked because of [these reasons](https://github.com/rgpower/react-closure-compiler/commit/6b403).

That being said,  I think you'd enjoy a pretty decent JSX experience, if you were to add the following to your plovr.cfg

```#javascript
  "custom-warnings-guards": [
    "info.persistent.react.jscomp.ReactWarningsGuard"
  ],
  "custom-passes": [
    {
      "class-name": "info.persistent.react.jscomp.ReactCompilerPass",
      "when": "BEFORE_CHECKS"
    }
  ],
```

and then run plovr with that react-closure-compiler.jar custom-pass in your classpath:

```#bash
java -Xmx1024m -cp react-closure-compiler.jar:plovr.jar org.plovr.cli.Main build myplovr.cfg
```


Life is about technical preference


and thank you for the contribution, of course !


Is there a word other that "literally" for this? Sounds kind of odd/informal too me.

Also, comma: preserving all symbols**,** including


Thanks @bantic for contributing a pull request, they are always appreciated!  I will be taking a deeper look into this change over the next couple days since it has potential for a large impact. It does look like at least some other SDKs convert the Metadata to lowercase before sending the request, so I do think this is something we should support as well.


Iterates **over**?


This is ugly. Can't we have JetObjectDeclaration.getContainingClassOrObject()?


you added a bit more text, which is great

however would also like to retain a real world example (bridge sse to amqp) as it helps to understand the purpose

Perhaps the section you added could be named ##usage and the original snippet could be placed in an ##example section?


Non fatal so perhaps this would be better logged as a warning.


答楼上：不是，请注意这个文件是某个branch下的readme文件，第一行确实表示这个repository的名字，但对其修改并不会反过来影响repository的名字。


Yes that sounds right... I think I'd prefer to do it in parse_cpu_set_and_warn since in principle there's nothing wrong with an inversed range (in other possible usage cases...) Will send an updated patch.


Thanks, I'll revert that. I had assumed that due to type erasure it would be fine, but now I see there is some insertion time checks with TypeToken.

I will still have to make changes to this area, as the method of joining the env variables together does not work for more than one variable


species=speciesID[14:-1]+"_"+speciesID[-1]

Still not loading in COPASI


An abstraction like a leaky bucket should really be encoded as a class. All the elemeents should be in one place.


Definitely a good addition.  Not a fan of needing the re import, though.

Would also prefer not stripping out the trailing whitespace (common user preference or not!).

Also broke this with the recent addition of py3 support.


Not sure if this is of interest to the project, but I use it at least. It's fairly similar to the getSeriesAllById, but only retrieves episode info.


I like the new name :)


Do debugowania console.log jest OK, ale później lepiej go wyrzucić.


Well the converter doesn't emit code with whitespace of comments in it. So I'll pass `null` as you suggest.


Do czego służy ta flaga?


What does this phrase mean?


dkcclphrase.ldk


Alright.


shouldn't be, rather?  `mv src/cljs/shopping/modern_cljs/connect.cljs` (note the `shopping` file in between)


field names are not clear hear, the meaning of none of these statement is clearn.


I really like the explicit tracking of inconsistent states. It made the storage code much clearer (and possibly correct-er)


That gets really tricky due to the way `extract_first_word()` works... I don't think there's a way for me to use it to differentiate `5-` from `5`. In my mind, `5-` should be invalid.

I really think this would be simpler using `strchr(t, '-')` and maybe even just a straight `strtoul` checking for `-` as the first invalid character after a number...

Do you think `extract_first_word()` is really the way to go here?


There is a commit that follows this one - I should have squashed them together.  Essentially added back in the workdir and nested the content dir underneath it.  The property name is duracloud.vault.workdir.  Perhaps I should change that to duracloud.bridge.workdir?


change `app => $app` just to `$app`


If you name method as getSomething then it should return some result. In you case better name is initSomething.


Javascript


Why did you remove these headers? They sound as useful here as in Julia Base.


Looks like this addresses the case where the current blob wasn't sent fully, and so it keeps trying until it gets it through. Once that blob changes, it leaves the thread until the next iteration, where it'll try to send the next one.


sure - I'll fix that.


You can move code block that check "number" parameter to separate method.


@gunta can you please create a plunker that shows the error


I just thought it would be simpler in the current context. :)


Nice work, this looks really good.


meaing of "rateset" is not clear.
Why is creation time relevant? When the last charge was assessed is relevant. And its initial value is when the
Rateset command was first applied. But what relevance does it have later?

sS


Instead, how about setting it to BOARD_SAMPLE explicitly instead of using a magic number? That constant is defined in control_pb2.py (as part of the protobuf Python output)


[frf.xlsx](https://github.com/guerramarcelino/regressoesespurias/files/12626/frf.xlsx)

Dados utilizados na estimação da função de reação fiscal. OBS: todas as variáveis foram defasadas em 1 trimestre na regressão em questão.


AFAIK, `ToArray()` is based on a buffer using growing arrays when the source collection cannot be cast to `ICollection<T>` and has a fast path that directly copies the source collection to an array of the same size when the source is an `ICollection<T>`. Since MSFT's implementation returns a list, I'd expect `ToArray()` to be "slightly" more efficient than `ToList()` in this case. Since `FindAll(ClaimTypes.NameIdentifier)` is not supposed to return a large collection, `ToArray()` shouldn't need to enlarge the initial array (whose capacity is around 4 IIRC).

But in practice, I guess the difference is really insignificant for such small collections :smile:


ah, got it. So if I  us d3_clustergram.js in the pharos code I get a JS error, screen shot attached. Unless I add the check for the group field it doesn't work properly

![image](https://cloud.githubusercontent.com/assets/34361/10462437/b2941d32-71ac-11e5-906a-73ae0dca1a02.png)


hmm, extract_word with DONT_COALESCE treats "5-" and "5" differently -- as long as you use it to parse both parts. Which you really should.

I am pretty sure that use extract_first_word() for these things is the best way to go, to unify the parsing logic. And for the cases where it's not good enough we should improve it to be good enough --- though I think this case it aalready covers nicely!


[vot.desp.xlsx](https://github.com/guerramarcelino/regressoesespurias/files/12628/vot.desp.xlsx)

Dados utilizados no post sobre despesas das campanhas dos deputados federais do Ceará.


mv src/cljs/shopping/modern_cljs/connect.cljs


NAK (nak docs -- https://www.kernel.org/doc/Documentation/stable_kernel_rules.txt)

I know it's a pain in the ass to do things like add __unused annotations, but it does avoid bugs to be super clean about GCC warnings.


Poked at this a bit more...I definitely did it to fix tests, but looking at the actual test failures, there might be a smarter way to fix this. Will investigate further.


@posita

Hi, I should have forked the gist ... but I didn't. (shoulda, woulda, coulda - sorry about that).

> Are your additions browser-specific, or can they be run elsewhere (e.g., in `Node.js`)?

It is browser dependent since I'm using window and Promise (which is supported just in the latest versions of browsers http://caniuse.com/#search=promise). It could be made to work in `Node.js` if I don't use window and attach the namespace to `exports`.

I haven't used `node.js` before and I don't want to pretend like I did, but making it work in node.js should be simple enough (and I'll do that tomorrow .. just to see exactly what it takes).

>Are your tests easily automated? If so, how? Care to update your README?

Yes, they should be easily automated with a grunt/gulp task and running the tests in PhantomJS. Then maybe integrate with Travis CI (will update tomorrow).

In your last example, since you closed red1, red3_rsrc will not equal red1_rsrc since it would be a new instance of the expensive resource.

>  // What *should* it point to? (red3_rsrc)

It depends. That's a design decision: do we want to be able to 'revive' a resource? To have `===` work we would have to not delete the resource when calling close, just marking it as closed.

We could add an `equals` method that would compare 2 expensiveResources on specific properties to decide if they are equal. But again, once you call `close` I would treat it as a sort of `dispose` function and you should get a new instance of the expensiveResource.

You also mentioned the license :) . My bad on this .. I just picked a license that I thought made sense for the code I'm writing. You could say that I just took a look at an idea and rewritten it, so no license was broken. It's not like I forked your code or anything :P. But I see your point.

Thanks.


This commit appears to have entirely broken bluetooth on jfltespr in the 10/13 nightly.


This won't work with BSD `sed` (i.e. on OSX) as the `-i` option has different syntax.

Also, you can cut out the copy stage by writing directly to the temp file, e.g.
```sh
sed '/<h:title>/d' $XFORM_PATH > $XFORM_PATH.tmp
```


I'm glad to hear you find this library usefull!

if you want responsive, you sholud look into `Tree.prototype.handleOverflow` method and call this method when the viewport resizes.


I am assuming this failingemailer is just for testing? can we put that in the _test file?


Have you tried pulling the latest repo?


Don't leave debug code like this in stuff that gets submitted for review. (It's not good practice to add "added by $NAME" type comments -- that's what git log is for if people really need to know).


In terms of the git log, it would be more idiomatic to make the short log short:

raw_packets.h: add data_fields_bsmp structure

Then the long log can describe them. ("These contain [...]").


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1373 of 1373 tests passed)

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


I love @Lhyen's comment about audience, because that's really what matters. Comments are a form of written communication used in tandem with the written code. Part of your historicizing project can understand the exigence of commented code, which ultimately indicate how complex ideas needed to be conveyed to human audiences about the code written to both human and machine.

Overall, I can foresee you talking about the history of code commenting, it's contentious events throughout history, then how it can be taken up in HTML and CSS web projects. I would suggest digging into the spec more. Maybe even search the Charles Babbage Institute for any historical materials related to code-commenting.


If this is 2D, where do you keep track of the dimensions?


Same question here.


would be good to add a comment about why we modifying the hostname verifier


But the flag is never set, because we never call insert_inode_locked. That's why calling unlock_new_inode() would cause WARNING.
```
void unlock_new_inode(struct inode *inode)
{
        lockdep_annotate_inode_mutex_key(inode);
        spin_lock(&inode->i_lock);
        WARN_ON(!(inode->i_state & I_NEW));
        inode->i_state &= ~I_NEW;
```


... and it's probably better practice to make the temp file with `mktemp`.  This will handle the cleanup automatically.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 671](https://teamcity.mesosphere.io/viewLog.html?buildId=58101&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:07:01


all of these functions need to be passing ``self``` and ```other``` as parameters, so you need to have commas in between, not periods.


Everything in this header begins with raw_ for namespace reasons. If it doesn't belong in raw_packets.h please move the definition elsewhere.

What is the purpose of this structure? Please add some notes in the git log to say why it's good and useful to add it.


Exactly. Github mangled this.


If you simply make plot.json a dict, you will actually get recursive diffs. (Not sure if that will trigger change handlers though! We might need a [virtual-dom hook](https://github.com/Matt-Esch/virtual-dom/blob/master/docs/hooks.md) to handle cases like this!)


These seem like very specific messages to assert... I'm worried that these aren't particularly scalable, since the result of the message shouldn't depend on the output saying that specific phrase - as long as it spits out some error message it succeeds.

For instance, we may decide later that "[NUM-ERROR]: '0Xf' is not a valid number" is a better message (hypothetically, of course). We shouldn't need to update the unit test because of that.


Missing signoff.


to be clear the recursive diffs should be much smaller in majority of cases....


>      * ticket id and subject is maybe not necessary as this is already in the
>        browser title

They're not always readable when the user has more than a few tabs open.


Email in the signoff needs to be in angle brackets (<), not parentheses.

It should be "Signed-off-by:", not "Signed off by:".


@zjx20 The getline function return length was just off-by-one, same as my own implementation.  It is working well with `return len - 1;`
@sriemer I have looked at aScanmem.  The gradle build process was confusing to me, but I think it does a good job at providing the command line scanmem and jni to Android, without interfering with running apps.  The two limitations of it I know of are that it does not modify enforcing SELinux, and is linked with -fPIC, so no newer Androids.


Missing signoff.

I'll stop noting missing signoffs in this patch series here. Please fix all the places where they're missing.


In my opinion we should:
 - keep both implementation
 - decide which one we want to use at compile time
 - if it is possible both implementation should have same API (shared header)
 - in that way most of the files will stay untouched and merge will be easier do done

Second implementation name proposal:
 - ext4_extent_full.c
Primary implementation name:
 - ext4_extents_basic.c


+1, why would we remove this?


Did you backport a bit more than indented here? :O


This line doesn't need to be here.  You're going to run py.test in your command line and as long as you have it installed, it will run and test your code for you.  Also, get your linter working so it can tell you where your spacing errors are.


yes it uses the latest repo


use the trusty `@api` here? ;)


scikit-image --> mltsp?


style note: I find that making the assertation note something like "expected parseNum stderr to be..." helps with readability of the code, and test-failures. If I ran the test and got the current message, I'd probably think that parseNum DID print "[ERROR] Number error: '0Xf' is not a valid number"


Please don't add unnecessary whitespace like this. Doing that sort of thing adds clutter to the diffs and makes them take longer to review.


Yeah I fucked it up :)


Sounds like it's an issue of code size.

>This can be added back at a later date if required if enough code-size savings can be found, for instance, the removal of the CLI.


This header may stay untouched. Only struct ext4_extent_tail looks like new type


Merge to sCommandClient.lua