Why is this not `MAIN_THREAD = that();`?


Uh... these files are pretty confidential and the repo is public.


![bola](https://cloud.githubusercontent.com/assets/13932890/9456368/fd8d79cc-4aac-11e5-98c2-94637619ae7f.png) Imagem do player


Ok.  Can you add a comment saying that here, as it's completely nonobvious to me.


VS doesn't like to add ending newlines.  I don't think we should try to fight it on csproj files.


I changed it again in f8800cf. Now we say "recommended" for 14.9 and 14.12, "not recommended" for 15.5 beta and we say nothing for others (including 15.7 for now). I think this avoids the "implied not buggy" I was worried over.


Asd! ;)


something


Some setups have 1k+ DB, we can do 1k calls to HS2, so we probably need to do a lazy loading instead


BTW one problem with this is that any future changes might not get tested at all for older drivers (eg. including 14.9) so that information might become misleading sooner or later. But this should be fine for now.


4590-VRF-defrost-power-calculation-problems (rraustad) - i386-Windows-7-VisualStudio-12: OK (1799 of 1799 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)


Hi m，
I made one update now.

I am curious the 4.2 patch could work well in my env, why it fail in your env


@yujuhong Sure, just leave a TODO will work for me :)


We can, but I haven't found a convincing use case for this, other than logging an error...


HOW DID I NOT DO THAT


WHAT IS LIFE


INTELLIJ WHY


Right before a release, we'll set whatever versions we have tested to "recommended" and drop it for other versions.


Here I am worried of displaying the page with 5k tables, from past experiences GetSchemas calls can take minutes while 'show tables' goes faster, we might need to show comments on hover or actually implement https://issues.cloudera.org/browse/HUE-2939 (latest probably the cleanest?)


I cannot say it for sure. But, I remember someone complained about 14.12 I tried all bug's failings formats, and everything just worked.

BTW: your last change is perfect.
BTW2: indeed. If there is a volunteer reading this that likes to have old software, please, keep it at 14.9 for AMD.


TeamCity ReOpenLDAP :: devel-2.4 [Build 180](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=62&buildTypeId=ReOpenLDAP_Devel24&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:37:03


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


It's so crystal clear that this is problematic. I do hope you understand
that this cause problems with additionally installed frameworks.

Compile the same source code preopt'ed and not, and use oatdump with
boot.oat if you want to see the differences with your own eyes.
Google has made dalvik and ART to recompile everything when frameworks has
been modified for a reason.

If you still want this, let your users know of the potential problem or fix
this yourself by integrating gapps to the build, which I failed to do.
On Aug 25, 2015 8:25 AM, "Łukasz Domeradzki" <notifications@github.com>
wrote:

> I even turned own my own "archi pedantic" mode to find ANY clue of
> something wrong happening and nope, everything is crystal clear, no bugs,
> no FCs, clean logcat, and blazing fast.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ArchiDroid/android_device_samsung_i9300/commit/627616675ce51662916efa53282d5e4a35401366#commitcomment-12869150>
> .
>


Hi Jesus,

There's a lot of tabs from oneelse's previous code.
I just maintained them so that the intention of the commit don't get
obscured.
But I can fix it. Thanks for noting. Almost home.

Em Seg, 24 de ago de 2015 21:16, Jesus Castello <notifications@github.com>
escreveu:

> The indentation here seem to a bit out of whack :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abinoam/ruby-monday-blog/commit/107de3ce6c030e41f18b90677118677ae1a7117c#commitcomment-12869758>
> .
>


> Right before a release, we'll set whatever versions we have tested to "recommended" and drop it for other versions.

Perfect


What are you talking about exactly?


LGTM :+1:


调节系统设置的权限


Why? `cpw.mods.fml.common.Optional.Method` does the exact same thing. And it's built into FML


If you cannot use mod_rewrite, please set index_page in application/config/config.php line 73.
ex:
```
$config['index_page'] = '';
```
to
```
$config['index_page'] = 'index.php';
```


I really really wish one could use eg. Debian/Ubuntu's "update-alternatives" command for just switching between drivers (OK if a reboot would be needed). I'm pretty sure it could be done but it would take some time setting up.


hmm, mainly is just because the apiserver will reject an update if the pod doesn't exist, e.g. `Error from server: error when replacing "examples/pod": pods "nginxa" not found`, so I think we'd better keep the same semantics here.


@yujuhong ^


Por que é um StepBuilder?


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

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

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

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

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

```


@Keno

`_Static_assert` is a c11 feature and we are currently only asking for c99 (or gnu99). According to @kshyatt, this breaks the compilation on latest `icc`. Should we just disable it there or should we explicitly ask for c11 / gnu11?


np. done.


101615016CoilCoolingDXMultiSpeed-requires-Waste-Heat-Function (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1275 of 1275 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)


if `run()` is called off the main thread... it's run directly on whatever thread called it. **WHY**?


I am on `icc (ICC) 15.0.2 20150121`


hmm...we are not actually the authority of pod existence, and we "decides" that this is a pod update ourselves only. It'd be really surprising if we contradict ourselves...


I told you already - I can't see any problems while DO using additionally installed frameworks such as google maps or camera. If there is no issue in using Google Maps which does utilize google maps .jar, and there's also no issue when using Google Camera, which does utilizies google camera .jar, then I have no idea what problems you're talking about.

Unless you provide any reproducable use case for your theory, sorry but you're talking nonsense that I can't confirm in my build, even if I deeply want.

I released an experimental release to my users already. If they find such reproducible use case, I'll revert it. Until then, I consider your problems as a myths


101615016CoilCoolingDXMultiSpeed-requires-Waste-Heat-Function (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


木村さんから教えてもらった https://imageoptim.com/ で画像サイズを小さくしました。
`$ brew cask install imageoptim` でもインストールできます。


Also, according to [cppreference.com](http://en.cppreference.com/w/c/language/_Static_assert) we can just use `static_assert` for all c11/c++11 compilers since we already include `<assert.h>` instead of conditioning on MSVC as in https://github.com/JuliaLang/julia/commit/c63e3cc657d291d948d0b281533afdb80dca4a6b


thx!!


This is the line that was forgotten after all this time. This is really all that was needed to fix the issue.


More context: kubelet/config keeps an internal pod cache and use it to decide whether a pod spec change is ADD/UPDATE/REMOVE. There is no reason to not trust the operation type, unless the two caches divert (which would be a bug). We thought about merging the two caches, but it seems like too much work for now.


@LionNatsu Personallly I suggest submitting a bug to their BuildTools. It's always possible to save one and set it back.


#sriderp


@tractorcow Weird indentation indeed. Yuck! Fixed now ;)


Is this line valid? On EAOs OSX nightly build, I get a  java-doc build error at this line, with:

error: unexpected end tag: </code>
15:11:12   [javadoc]      * @example   condition([true, false, true], 1, 0) = [1, 0, 1]</code>

Is there a missing <code> at the start of this line?


lol its for rohit and adam. + something is broke so dont worry about it. ill make a new fixture when i have time.


@abnero DWMR. :+1:


yes mistake, sorry, I thought closing it would close the request.
Still getting my head around github.


I believe this is fixed by 33245bbff0750de9a7a5767cc38a4b33af535df0


ok on est bon pour un bon sommeil


Eli faut avancer la gestion des dossiers hein on a plus le temps


develop (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


@robin850 Fair enough :smile:
I thought Rails would keep cutting old versions out until version 5 release.
Thanks!


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

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

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

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

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

```


great! It's a big project for the dynamic enhancements! Thanks a lot!


So what exactly did you add in this update and why is it called the oop udate?
Also I have a suggestion. you should add:
- commands:

in your configue so that when we do /kit {name} that command also runs. That will allow us to add effects with our kits. Please use that in your next update or if I can already do that please tell me how. Besides that this plugin is amazing! Really good job.


I find a weird and fixed via https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/db46ea7cd2db7a21a60d86e34187cd288982649e. We can discuss this further tomorrow.


4590-VRF-defrost-power-calculation-problems (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1799 of 1799 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)


You implemented the right method now and it probably also displays that log message in your console when you switch tabs. But the assignment says to:

    When the user switches tabs, your implementation should print the title of the newly-selected view controller into the console with a message:

    New view controller selected: [title]


whoa... diffuse lighting


Yep, looks fixed to me.


I like the idea, good suggestion!


运行时的话，这里不用for 真的好么


lgtm


oh sheeeeeeee


I'm afraid one need a multi boot setting (my experience says on LTS you have to select version X and on e.g. 15.10 you will have to do something quite different).


@andersonbrandon typo in the first line
````require 'bcrypt'````


I agree that this would be proper, but the current way is too convenient to remove. See 2fa6674ba185f4dad7e8f8043cb4faf6920f8fab.


Maybe it's already specific enough now; I don't think there will be many non-Arastta powered websites (if any) that are matched by the current pattern. But if Arastta has always been using Arastta.org, it could be turned into something like this:

    Powered by <a [^>]*href="http://arastta\\.org"[^>]*>Arastta

or, less specific (to be a bit more tolerant for variations in the web address):

    Powered by <a [^>]*href="https?://(?:www\\.)?arastta\\.org[^>]+>Arastta
But the current regular expression may already be specific enough.


dis.


No need to initialize to zero as ret is first assigned the return value of rxe_xmit_packet().


Looks good!
Could you turn the changes in commands/fs.py to do a while loop to work with recursive inks as well?


Some minor wishes:
– Use ː instead of : as length marker
– Use non-syllabic diacritic as in these [Examples](https://en.wikipedia.org/wiki/Diphthong#Examples), such as ɑʊ̯ instead of ɑu
– In ɑ:̃, the creaky marker should be on ɑ, not on the length marker


add [] and {}


remove


> 1 - **I like the `--verify-base-url`** that you can call whether you override `base-url` or not - I agree it's often a nice convenience to fail fast, and I think having that available if you've overridden base-url is handy.

Great, I'll implement this today.

> 2 - One of my frustrations in the past with `verify-url` is that my selenium tests are usually mixed into a suite of other tests. Sometimes I'm running py.test -m 'not selenium' which doesn't call any of my selenium tests and as a result there's no need to spin up my test server. My experience (also with pytest-mozwebqa) is that verify-url happens seemingly at test collection and as a result is noisy when it shouldn't be. **Is there anyway to make `--verify-base-url` only happen when you're actually running a selenium test(s)?** (I have a feeling that the answer is no, but thought I'd ask)

In my opinion, this should run whenever a base URL is used. It might not be just Selenium tests that use a base URL, you could have tests that check HTTP headers, etc. I will likely implement this as a fixture that depends on the `base_url` fixture. If there's no base URL then the verify would be skipped.

> 3 - I think I've been misunderstanding `skip-sensitive` - I'm now understanding that sensitive_url is true or false once for the whole test run. The skip_sensitive is per function, because a function may be marked explicitly as nondestructive. (I previously thought that it was somehow checking the url that was being accessed for each different test - like there might be sensitive parts of a website or not.). I think I'm right in thinking that the use case for this is that you run the same test suite against different versions of the same website - `dev/local` | `staging` | `production` and you want to know what tests not to run against a production environment. And you're taking the cautious approach so all tests are dangerous by default. Assuming all of this is right, --sensitive-url=None seems good & clear. It might be worth adding a little more to the docs to explain these concepts more. e.g. "By default, all URLs are considered to be sensitive." I saw this in the context of "all urls within your test suite" but what you mean, I think, is "By default, any base url is considered to be sensitive."

You're certainly understanding it correctly, and it helps to know your previous understanding. I'll revisit the documentation in an attempt to make this clearer. Thanks!


He, github ate the xml tags i used. edited it now, look again, it should make more sense now...


Can't wait to start using this. Is there anything holding this back from being accepted?


Could this WQE be neither atomic nor RDMA read/write?
iova might not be accessed at all in that case, but still I would prefer not to access the union fields if they are not valid.


Use GetLocalizedString() and add string to Resoruce


Note that "--force" is something we consider an "option", but the major verbs like "reboot", "halt", "poweroff" are "commands"... i.e. it matters whether the arguments are prefixed with "--" or no...

Basically it boils down to: The "command" encodes what to do, but the "option"s encode how to do it...


Use LocalizationConverter instead of hardcoded strings


Thanks!


The default encoding of the Erlang source files was Latin-1 in R16 and UTF-8 in later versions. To fix this, either write the non-ascii letters as a numeric list or add an encoding declaration to the file.


依然要改


I don't understand this condition.
If a non-RDMA read is retransmitted, you always start it from the top? Why not retransmit sends or RDMA writes from the middle (from the last acked psn?)


ping @imurchie make sure you pull those, the prefs logic was messed up.


It basically allows you to say that a client which has one privilege should also implicitly get some others. We use it in some places already, grep for it in the sources. Basically, we put it wherever another closely related action exists that is safe to also grant if the user authenticated once for a higher privileged one...


nice


Well, but if people use --message= multiple times, then they still can add muple items to the wall message. And I think that's actually OK, we should just concatenate them with spaces and be happy.


@cviral30 What does this mean? Please elaborate.


This should be an `else` branch, right? There could never be a WR that is both WRITE or SEND, and also READ.


@cviral30 So, you cannot code a page in HTML without columns or only 6 columns?


@cviral30 What about `{ display: inline; }` elements?


c'est quoi "<%%="????


$authMethod = getAuthMethod(controller);
if (null === authMethod)


I'd say.


@cviral30 What it means to have valid class names? What is the difference between class & id?


Is comp.psn guarenteed to always be in the range of the first WQE?


@cviral30 Which tool are we talking about?


@cviral30 What do you mean colour of an image?


@cviral30 Why so?


pull member variable auth up here


@cviral30 What is bootstrap?


pull up to base class


Auch. Typo fixed in https://github.com/WordPress/WordPress/commit/25c5d372d3c2ba519ae31de21af726bd727eb2a2


Looks good!
Could you please also do the small change (in a separate
commit) that will use getenv() instead of misc_e6_envvar() here?


move behind if check


A possible issue with using two buffers is that one might be in cache and the other one might not be (probably the pt buffer, since it was written to at a later stage in the program, but a sufficiently capable attacker might even have more influence over the cache contents). The difference in a cache hit or miss is then still a timing sidechannel.

An idea might be to set a mask depending on the value of err and always compute the and over the plaintext and the mask before returning. Let me try and implement a fix to show you what I mean.


This is great. We'll need to get the documentation file created so we can add the examples and other details.

If you wanted to get started early you can copy one of the files in the \Documentation folder like *Get-Layout.ps1*. We convert this file into the maml help file for PowerShell and the markdown file for the gitbook.


I think `impossible = error` to indicate situations that are impossible.


Done


LOG.debug('Ignoring return code %s: %s' ,  (return_code, msg))
Also, better to use key value in this case like
https://github.com/openstack/cinder/blob/a9d071ce70af754698b7a87a1ffbf7812118e034/cinder/volume/targets/cxt.py#L100


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

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

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

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

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

```


I think this isn't needed, as you don't require any extra packages (and so, don't need any repo) in the framework definition


This commit deleted spring-social-showcase-boot/src/main/resources/application.properties witch rendered spring-social-showcase-boot unusable.


findIdentityByToken -> findUserByToken


ditto


@Kasaku We thought about creating separate libraries that get deployed as add-on modules. Our first example of this might be for commands interacting with xDB such as Get-Contact, etc.


This code doesn't take input from Employee object


Part of issue #62.


move inside the if clause


IB spec define ordering rules (Table 79 Work Request Operation Ordering) that require fences not only when IB_SEND_FENCE is set, but also for certain operations. Do you handle these rules anywhere else?


做了一些改动


Can you make this a constant, and preferably a bigger constant.


h2. Projects and technologies we love and specialize in
h3. But not limited to!


The whole drain completion seems not related to this function. Also parts of it can be shared with the completer's draining code (at least the event callback code).


@jackpoz: intended code is timer 3 second or if false then random timer between 8-11s
with your changes you split it into two arguments, so the result is always 3-11s or 8-11s


thats a change of the meaning..
files are not supposed to have a closing ?>


Think we're already root at this point so don't need the `sudo`?


@garyo these lines were probably removed by mistake (these are deprecated but should not yet be removed)


bullet points don't close with a full stop

http://www.oxforddictionaries.com/words/bullet-points


Do we still need the platform specifier here?


ops, denne blokka skulle ikke vært med i denne comitten


Ok.. done it.. new branch is RR-5.1.1
https://github.com/RR-msm7x30/android_device_samsung_ancora/tree/RR-5.1.1


please s/CC15/cc15/g at least for the udev rules
c.f.: https://github.com/rad1o/f1rmware/pull/29
I've created a trivial PR (212) to make your life easier


Don't know if it is worth the effort. Would have to create a new file system with nested symbolic links to test. Furthermore, it is not immediately clear to me how to pass an updated argument (newpath) to call() in the command_cd class of commands/fs.py


yeah you are right.. ı forgot that.. will be reverted asap


Can change this comment to:

```diff
- # TODO: Just use CSV.generate when Ruby 1.8.7 support is dropped
+ # TODO: Just use CSV.generate
```


#4858-VRF-IO-Ref-documentation-shows-incorrect-values-for-schedule-priority-control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1277 of 1277 tests passed)

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


replacing english with american english? why?

http://www.oxforddictionaries.com/definition/english/lower-case
http://www.oxforddictionaries.com/definition/american_english/lowercase


Won't this ```if``` statement always evaluate to ```true```, given that ```options_.first_obs = NaN;``` in ```global_initialization.m```


jepp, skal prøve å få gjort det snart!


I'll adjust the text so it fits the screen width better.


I do not like that update, I am ok there only for "private" usage instead of protected.


@ccx-testerbot runtests master


why uppercase B?


If we have more than one variable in the log string, then we have to use key value.
But since, we are always going to use return_code = '-3501', no need of using two variables.
I think below would be enough
LOG.debug('Ignoring return code -3501: %s', msg)


Will test in the next day or so.


where?


why do you need this? just insert string.


I'm not sure I understand how the requestor and the completer tasklet coordinate access to their shared variables. I don't see any locks around them. How does it work?


I'm not sure I understand how the requestor and the completer tasklets coordinate access to their shared variables. I don't see any locks around them. How does it work?


is it not good to use message=msg in line 92


i would rather call it something more uncommon like augeasprovider_grub_version to workaround possible conflicts with variables defined by others.


1.gün yapılanlar


>Ah, ok, so that's not even for the technic radiation resistance calculation?

as far as l know radiation_resistance is used for resistance calculation
>Why not manage that on side of the armor mod?

l already try to do that.
>Perhaps someone in the future will come along and change that value, because it's "too high for reactor shielding"

it's no problem, the armor mod uses technic's radiation data
and before you wrote that 80 is not the wrong value
> use any existing technic shielding resistance as base and apply a factor 6 or so.

l still don't know exactly how to use the radiation resistance value for armor


line 94 not needed


`<link>` Tags sind anscheinend nur für externe Ressource gedacht. Man könnte self auch weglassen, die aktuelle URL kennt man ja, oder?
> The <link> tag defines a link between a document and an external resource.

http://www.w3schools.com/tags/tag_link.asp


Nit: "An TypeError" should be "A TypeError"


LOG.debug("DotHill Request URL: ", url)


s/skim/skip?


:beers:


if paylen is already initialized, why do you initialize it again here?


Definitely skim! Everyone should at least be aware that pills exist.
Yours,
r


# Enrichment Score(GSEA) compute Tools for Lincs data

## Tools

A brief description of the tools is given below. The tools implement the function of parallel compute ES in multi-core environment. The Matlab implementation of the tools is currently the most mature. I used the 1ktools(https://github.com/cmap/l1ktools) tools to parse the .gctx file which stored gene profile data defined by Lincs(CMap) based on HDF5 file format. However,the performance is not very satisfactory. The C implementation to achieve at least 30 times faster than Matlab. But 1ktools can not parse the .gctx file by C. So I used Matlab to parse the .gctx file、extract the gene profile sets and write to .txt file. C will read the file 、complete parallel computing ES and write out the result in binary file . There also is a Matlab Script to read the binary files and can be later analysis. I will update the tools as they become available.

### Matlab Tools: matlab/

#### Requirements:

1. Matlab R2009a and above

#### Setting the MATLAB path:
Enter the "pathtool" command, click "Add with Subfolders...", and select the directory ParalES/matlab.

#### Tools:
* [**ESquick.m**]: Compute single Enrichment Scores.
* [**ESScore.m**]: parallel compute Multiple pairs of ES.
* [**getSampleforMat.m**]: parse the .gctx file and extract the gene profile sets we need.
* [**ParalES.m**]：main entrance . Setting Parameters and complete parallel Computing tasks.
* [**getSample.m**]: parse the .gctx file and extract the gene profile sets we need for Writting.
* [**PreESforC.m**] : Setting Parameters 、 extract the gene profile sets and write to .txt file.
* [**importES.m**] : Read the Enrichment Scores Result Matrix computed by C.

### C Tools: java/

#### Tools:

* [*ParalES.c**] read the .txt file 、complete parallel computing ES and write out the result in binary file

#### Demo:
* [**compileParalES.sh**]: compile the C source code to Executable files.
* [**runParalESLinux.sh**]: run Executable files in Linux.

## The LINCS Dataset

The CMAP Cloud API offers programmatic access to annotations and perturbational signatures in [the LINCS L1000 dataset](http://lincscloud.org/) via a collection of HTTP-based RESTful web services. You can get the .gctx file stored gene profile data by the Website.


Any reason not to implement this as
```C++
bool SimulationConfig::hasThresholdPressure() const {
    return ! m_ThresholdPressure.empty()
}
```
?


@ngeiswei Yes, actually this is just for example, so I should write as:
`PYTHONPATH="$PYTHONPATH:<OPENCOG_ROOT>/opencog/python"`.

Anyway the thing necessary for python path is still two thing(cython lib, python module), isn't it?


> In my opinion, this should run whenever a base URL is used. It might not be just Selenium tests that use a base URL, you could have tests that check HTTP headers, etc. I will likely implement this as a fixture that depends on the base_url fixture. If there's no base URL then the verify would be skipped.

I completely agree. To be completely sure I'm understanding -  `base_url` is not called by your test then verify should be skipped. So if you have tests:

````python
@pytest.mark.mark_a
def test_with_base(base_url):
    pass

@pytest.mark.mark_b
def test_with_no_base():
    pass
````
* `py.test -m mark_a` - `verify_url` will run
* `py.test -m mark_b` - `verify_url` will not run


# Enrichment Score(GSEA) compute Tools for Lincs data

## Tools

A brief description of the tools is given below. The tools implement the function of parallel compute ES in multi-core environment. The Matlab implementation of the tools is currently the most mature. I used the 1ktools(https://github.com/cmap/l1ktools) tools to parse the .gctx file which stored gene profile data defined by Lincs(CMap) based on HDF5 file format. However,the performance is not very satisfactory. The C implementation to achieve at least 30 times faster than Matlab. But 1ktools can not parse the .gctx file by C. So I used Matlab to parse the .gctx file、extract the gene profile sets and write to .txt file. C will read the file 、complete parallel computing ES and write out the result in binary file . There also is a Matlab Script to read the binary files and can be later analysis. I will update the tools as they become available.

### Matlab Tools: matlab/

#### Requirements:

1. Matlab R2009a and above

#### Setting the MATLAB path:
Enter the "pathtool" command, click "Add with Subfolders...", and select the directory ParalES/matlab.

#### Tools:
* [**ESquick.m**]: Compute single Enrichment Scores.
* [**ESScore.m**]: parallel compute Multiple pairs of ES.
* [**getSampleforMat.m**]: parse the .gctx file and extract the gene profile sets we need.
* [**ParalES.m**]：main entrance . Setting Parameters and complete parallel Computing tasks.
* [**getSample.m**]: parse the .gctx file and extract the gene profile sets we need for Writting.
* [**PreESforC.m**] : Setting Parameters 、 extract the gene profile sets and write to .txt file.
* [**importES.m**] : Read the Enrichment Scores Result Matrix computed by C.

### C Tools: c/

#### Tools:

* [**ParalES.c**] read the .txt file 、complete parallel computing ES and write out the result in binary file

#### Demo:
* [**compileParalES.sh**]: compile the C source code to Executable files.
* [**runParalESLinux.sh**]: run Executable files in Linux.

## The LINCS Dataset

The CMAP Cloud API offers programmatic access to annotations and perturbational signatures in [the LINCS L1000 dataset](http://lincscloud.org/) via a collection of HTTP-based RESTful web services. You can get the .gctx file stored gene profile data by the Website.


`items|length` gives the length of the current page, not the full count of results.


Added because Mage_Widget_Model_Template_Filter checks interface of the block


this does not look right below the shutdown case


Spurious space between function name and opening bracket... See CODING_STYLE...


IIRC, we need to do the same in FixedSizeTrees as well


same here


The `rxe_pkt_info` is stored in the control block only for receive packets, right? If that's the case please note that in the comment, and also near the macros `SKB_TO_PKT` and `PKT_TO_SKB`.


That's my understanding too, but I will test it.


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


Can it be SD_BUS_PROPERTY then? Without the WRITABLE.


Should you revert noack_pkts when performing a retransmission?


I figure the `DIR` should be enclosed in `<replaceable>`.


if this switch statement had a default case with an error log, we would have noticed erlier


Also the `<replaceable>DIR</replaceable>` should be mentioned on the `-D` line, too.


Sarah,

On Mon, 24 Aug 2015, Sarah Graves wrote:

> Is this line valid? On EAOs OSX nightly build, I get a  java-doc build error at this line, with:
>
> error: unexpected end tag: </code>
> 15:11:12   [javadoc]      * @example   condition([true, false, true], 1, 0) = [1, 0, 1]</code>
>
> Is there a missing <code> at the start of this line?

yes, thank you, you're right.  This was causing a warning but not
a build failure in my (J2SE1.5) build environment, which is why
I didn't spot it.

Fixed now (0a0bdac).

Mark

--
Mark Taylor   Astronomical Programmer   Physics, Bristol University, UK
m.b.taylor@bris.ac.uk +44-117-9288776  http://www.star.bris.ac.uk/~mbt/


Students/Ghiurau A. Alexandra Cristina/Assignment 3/src/javasmmr/zoowsome/controllers/Main.java

I know, if I had the solution, I would have left a comment as well. Thing is, I still got a duplicate case error for both reptiles and birds and haven't figured out why (unfortunately, it's not because of the `String` associated to them).
I know why, usually, you can get a NPE, and I also know this is not a solution. When I'll have it, I'll let you know. :mailbox_with_mail:


For easier merging of core CVS I'd like it as
```diff
 	keys->ptr = keys->buffer;
 	keys->lock++;

+	if (cont)
	if (guessed_keys->count) {
 		keys = mem_alloc(size = sizeof(struct db_keys) - 1 +
 			length * guessed_keys->count);
```
Sod the coding style - things like this never need manual merging.


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


Funktioniert das so? Oder soll ich statt dem div etwas anderes verwenden?


Yes, I tried it without and it blew up :(


These little things are clever! :) Sorry for the noise but when I see something neat from you I need to say thank you for sharing that. :)


This is good.  I think the we should put a comment here that this should be reverted once we get our migrations condensed, thoughts?  Ideally we'd stay inline with the gem so that we have to support it as little as possible :)


Well, why drop the WRITABLE? I mean, it's clearly not as useful as your new method call, since it doesn't do PK, but then again if you are root anyway it doesn't matter...


proto number
What return type is needed? if it's float|double


this will be merged into the `deallocate` commit. I would have done it locally but my git is freaking out right now.


OK, I'll remove that line.


Would prefer `must throwA[...].await(...)`, without `Await` or `success`.


If I understand correctly, this leaves the comp field in ATMETH uninitialized, and can leak information from the kernel.

Please make sure you zero out anything uninitialized in outgoing packets, either explicitly or by zeroing out the skb beforehand.


The scan failed.
Included commits

Commit ID	Owner

9872989bd39d3dd5979d62e853474ea3c26087cf	@BearSpace


Done


I think the Q_Key needs to be taken from the QP, if the high bit in the WR is set. See IB spec: C10-15.


I guess he meant:
```c++
if (!CheckGuardAlive())
    task.Repeat(Seconds(3));
else
    task.Repeat(Seconds(8), Seconds(11));
```


That is ok as well.. tnx.


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


johnh.conf should also be adjusted.
The new config option should be mentioned in a comment, together with default value and other possible values.


There's a typo in the commit message.


No worries, I was just errrring because you mentioned of CISO :p


Good spotting noticing the duplication, though here reconsider the usage of `entries_2`.

The foundation talks about DRYing up specs, with perhaps a little too much emphasis. DRYing up application code (i.e. non-spec code) is often good. In specs, DRY is less important.

DRY in specs is sometimes a good idea, but sometimes it makes the tests hard to follow.

One purpose of tests is to document behaviour for other developers and your future self. Having code in **tests** be easy-to-follow is more important than removing all duplication from tests. Duplication in tests is fine if it helps make the test more understandable.

DRYing up the specs here by introducing `entries_2` is something I'd consider avoiding. The reason being is that it makes each individual `it` block a little harder to follow, and the method-under-test (`import_from_csv`) is no longer visible in each test.

My preference would be to call `book.import_from_csv("entries_2.csv")` in each `it` block.

If a developer were *determined* to DRY up this however (not my recommendation), a method with an intention-revealing name would be a little more appropriate than the `let` on line 95. For example, `def import_entries_from_csv`


too soon? :p


Please be safe!


I often use this: `{Vector2{defaultFramebuffer.viewport().size()}.aspectRatio(), 1.0f}`


Also, continue_single doesn't match the john.conf CamelCase name style. I re-wrote this already. If this f*cking DoS attack stops I'll push it too :angry:


@flavour I added lookupField:priority to retrieve the priority value?
After saving the form, and then when I try to change the priority value it shows invalid value error in the priority field. However if I remove lookupField:priority the priority value is not retrieved and need to set it again.
What am I missing here?


description added


This package path needs to remain as `github.com/...`.


@santosh1994 dont commit unwanted changes., also please add the README as well.


I think it should be 'review_user_name' now ;)


we had an endif before this? Syntax error?


Thanks for the PR! Looks good, I will merge this into version 1.0.6.


@houtanb yes, it will. This perfectly mirrors the case for nobs, which is also set to NaN and then sorted. However, this does not alter anything. But if you prefer, I can simplify the code for both nobs and first_obs. I think this is still left from the time when they were initialized to [] instead of NaN


There is no possibility of `self.target_content_type` being None at this point (and if there were, the first line of `render_html` would fail) so this test can be removed.


Done


Ok I did wonder.. I wasn't able to build locally without it as make didn't use the ./cmd/deploy.go I was modifying.. I'll revert this though


Ok, a while loop will do the work indeed. I will create a new pull request.


Hi. I am using your code. first above all, thank you for sharing your code. I have some question about your code.

I'm using intel edison and spark-fun's MPL3115A2 : https://www.sparkfun.com/products/11084

And I connect this sensor to sparkfun's I2C breakout block(https://www.sparkfun.com/products/13034) for connecting intel edison.

now when I upload SparkFunAltimeter.ino which you uploaded, I can get some values but it is a little weird like this :

Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00

Should I check some part of your code?
I've never changed your code.

actually I am really beginner for arduino code.. Please give me any help.

Thank you in advance.

Regard.

JaeHyuk


Your GOPATH will need to be set up properly so this repo should cloned to `$GOPATH/src/github.com/fabric8io/gofabric8`


Note to myself: no need, just redirect to Inbox.


Please rename payload to something that indicates it is the payload length.


Good suggestion


@EugeneButusov, I do not think such behaviour is always desirable. And I think it can break autoLogin.

I expected you'll clean up sessions in log out. Is it possible?


remove the option, `displaytopProducts` and call topProducts only if count is greater than 0. so by default, set `count` to 0


change `TOP_PRODUCTS` to `currentTopResults` like the above one.


It's ok I've done it all - just about to push


I think this should be part of the library, I need this feature


The scan failed.
Included commits

Commit ID	Owner

9872989bd39d3dd5979d62e853474ea3c26087cf	@BearSpace


Hi. I am using your code. first above all, thank you for sharing your code. I have some question about your code.

I'm using intel edison and spark-fun's MPL3115A2 : https://www.sparkfun.com/products/11084

And I connect this sensor to sparkfun's I2C breakout block(https://www.sparkfun.com/products/13034) for connecting intel edison.

and now I get test code from here : https://software.intel.com/en-us/iot/hardware/sensors/mpl3115a2-atmospheric-pressure-sensor

when I comfile upper code, I can get some message and values but it is a little weird like this :
-------------------------------------------
Incorrect device id - read: 0x00
Executing Sensor Test.
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
Write to trigger measurement failed
Write to trigger measurement failed
Error sampling pressure
  Warning - sensor values not changing.
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
pressure value = 0, altitude value = 44330, sealevel value = 0, temperature = -1
Write to trigger measurement failed
Error sampling temperature
-------------------------------------------

Should I check some part of your code?

actually I am really beginner for C code.. Please give me any help.

Thank you in advance.

Regard.

JaeHyuk


use naming convention.
change `processTopproducts` to `processTopProducts`


It is better to set all cmake "-D" flags in one go. If you don't, cmake could make decisions based on the previously specified flags and make them persist in its cache.


@santosh1994 what is this code change for?


Also better to set all modules and env variables before running cmake the first time.


@santosh1994 why commit this?


any reason to move away from the stack?  that protects against stack overflow in corner case of very deep trees.


@santosh1994 why do you need to commit this?


Building the schema from the definition fails in latest graphql-js:
```
"Node" expects field "friends" but "FakeNode" does not provide it.
```
I think this is due to invariants added in 4.2 that enforce types adhere to interfaces they the claim to implement: https://github.com/graphql/graphql-js/commit/73401ed28c6755f8f4c234f8389aa1ede9edef92

I added these fields to the type to fix the test schema.


Hang on: I've changed the first code file. Will that screw your stuff up?


"shall have little difficult" -> "shall have little difficulty"


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (752 of 752 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-7.91%25-green.svg)


Awesome :+1:


Hi Joe,

Presumably you don�t have a Rev 2 Model B and have had to use different GPIO assignments before?

http://www.raspberrypi-spy.co.uk/2012/09/checking-your-raspberry-pi-board-version/

If you do have Rev 1 Model B then it should be possible to override the default tingbot and ads7846 pinouts with overrides to their dtoverlay defaults in your boot/config.txt.

We need to get you a Pi 2!

Cheers,

Duncan

PS My latest attempt at a build failed (maybe a network outage my end so I�ll try again) with:
Successfully installed pillow requests
Cleaning up...
+ git clone https://github.com/tingbot/tbprocessd.git
Cloning into 'tbprocessd'...
error: gnutls_handshake() failed: A TLS packet with unexpected length was received. while accessing https://github.com/tingbot/tbprocessd.git/info/refs
fatal: HTTP request failed
child process exited abnormally
    while executing
"system "./retry.sh 5 sh -c 'cat build.sh | ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -i tingbot.key -p 2222 pi@localhost bash'""
    (file "vm-build.expect" line 46)
make: *** [build] Error 1

On 25 Aug 2015, at 09:41, Joe Rickerby <notifications@github.com> wrote:

> This looks good! I suppose this isn't going to work on my RPi model B because of the different pin assignment?
>
> �
> Reply to this email directly or view it on GitHub.
>


rxe_requester already checked this bit. Is it really needed?


Hm, looking at this source code I noticed that here we have double checking of the instance type of $user

```
            if(!$this->getUser()->isAdmin() && $user instanceof User) {
                 if($user instanceof User) {
                     $user->setAdmin(false);
............
```
It seems safe to remove the inner condition.


"we would have modify the" -> "he would have to modify"


This commit is being reviewed as a part of review issue #49.


Bump this to 22 to get the range [1, ..., 21].


Unhandled Fundamental Failure


I agree that `update_every_n_cycles` is less technical and easier to grasp. My only concern is that if I read `update_every_n_cycles` without knowing anything else, I'm tempted to answer 'yes, please', not '2'. But maybe documentation is enough to fix it. We are already saying that `update_freq_divider` also needs some doc to make its meaning clear...

I like the prescaler suggestion, but I find it technical as well. I'll propose `update_skip_factor` based on @ipa-fxm ideas, so the list of favorites is:

`update_every_n_cycles`
`update_freq_divider`
`update_freq_prescaler`
`update_skip_factor`

I'll ask you to bike shed with me one more time, and tell me which you prefer. It's now clear to me that the parameter needs to be well documented :-)


1st checkin


Rahat uyu Jim! Çok isterdim senin tanışmayı, sohbet etmeyi. :'(
Rest in peace!


Yes, we will create modules for new functionality spaces, but this looks pretty core to me :) As I said, I thought about us missing those (especially the first commandlet), at least a few times, so it's great that you did those!


Why doesn't this code use the completer tasklet?


Hey cturt, please contact me: tb3@gmx.fr
It is not about ps4 but nds !
Thanks.


I installed 11.6 and the TnT controls and it is faster but there is a
problem. The attached html only displays a short vertical bar and a few
words of text at the top. 11.4 does a better job but the images still
aren't shown even though they've been downloaded and are available.

On Sun, Aug 16, 2015 at 2:47 PM, BerndGabriel <notifications@github.com>
wrote:

> 11.4 uses Unicode as well as 11.6. So, Unicode shouldn't prevent you from
> updating to current version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BerndGabriel/HtmlViewer/commit/5ae61acf6811ad78b90085bd3a591606d1a3db88#commitcomment-12727767>
> .
>


Since this case can be ended only by aborting the process instance, it might be good to consider to add a "milestone" to terminate the case like we have here https://github.com/ibek/jbpm/blob/BPMSPL-12/jbpm-test-coverage/src/test/resources/org/jbpm/test/functional/casemgmt/TerminateMilestone.bpmn2 .... WDYT?


I just merged this.


IMO this should be left abstract for the implementations as they both have different code.


To remove empties from infields.
Need to add this in different comment.


one


刚才就是所说的那种GITHUB只能在网络好的时候用么？？竟然传了多次无法同步。。。。真是醉了。。。坚持打卡！！！


**[phpUnit_trunkIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkIntegration/8050/testReport/) for this commit**


:tada: 1000


Currently none, but will be changed to FTPS


two


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


That URL doesn't seem right. ;) Guess this is just a dummy though anyway.


its okay, no need of sending it as a different PR., send a different commit for it.
so for this PR send 2 commits. and also add a comment on this code.


If qp->mtu is always a power of two, you can use ALIGN().


Now login web with Email and pass = bnetuser +pass ?


Hi. I am using your code. first above all, thank you for sharing your code. I have some question about your code.

I'm using intel edison and spark-fun's MPL3115A2 : https://www.sparkfun.com/products/11084

And I connect this sensor to sparkfun's I2C breakout block(https://www.sparkfun.com/products/13034) for connecting to intel edison.

now when I upload SparkFunAltimeter.ino which you uploaded, I can get some values but it is a little weird like this :

Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00
Altitude(m):-999.00Pressure(Pa):-999.00 Temp(c):-999.00

Should I check some part of your code?
I've never changed your code.

actually I am really beginner for arduino code.. Please give me any help.

Thank you in advance.

Regard.

JaeHyuk


Thats on track. Ill add a method to determine which service (deo, ftps, https) will be used.
This might also go into ceph.conf: i.e.
key_store_mode = deo
key_store_mode = ftps


Fixed with 61188dbdc3b4f495891bf415e51c4f9f79f18217.
It was a problem with the thread dependent string builder.


This function supports all numbers
[-inf; +inf]


This branch was created from source of config_option branch, not master, just forgot to switch to a master


Thats a good point. I'll take care of this. Thanks for mentioning!


@JohannesPfeifer yes, please simplify this commit (using ```git ---amend```)and the ```nobs``` code (in a separate commit). Even if they were initialized to ```[]```, the ```if``` statement serves no purpose. There is no point in replicating bad code.


@MitchesD: I wouldnt classify changing code not doing what you wanted into code not doing what you wanted as change in meaning - all this change does is get developer attention so it can be fixed correctly.


hgysvsvdvsud


If the timer is already pending, shouldn't the code delay it?


@chalkie666 I once got very helpful advice by @dscho on that matter, as a [comment on one of my pull requests](https://github.com/fiji/Bug_Submitter/pull/2#issuecomment-47783312).
Maybe this should find its way to the [ImageJ wiki](http://imagej.net/How_to_contribute_to_an_existing_plugin_or_library) at some point.


new doc changes


No - it's fine


I think this is a new punctuation mark, the middle dot. However, I didn't add it to the regex file just yet, because I wanted to check it with you first. Maybe it's just a normal dot and I'm seeing things?


Is a 'continue' an ExitForStatement? 'ExitFor' stops looping but 'continue' starts the next loop


This commit is being reviewed as a part of review issue #4.


I bet these opcode functions can be made simpler with helpers that separate the operation from the first/middle/last/only part.


We might be able to do that schema less as well?

So src="//maps.google.com/maps/api/js?sensor=false"  which should work for both secure and normal ?


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


ah makes sense ok thanks.  I'll put some getting started docs together at the end of this.


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


I suppose that whole Background should be named like 'I am on Registration page'


Phew! I was really worried. Does this need to be done on any of the other resources?


Only indent fix in this file


Chapter 1 ready to be proofread by Stacey


English is a little "backwards" ;-)

message: 'Ajax request is not succeed!' --> 'Ajax request did not succeed!'
details: 'Error detail is not sent by server.'  --> 'Error detail not sent by server.'

Also should this not be in the localization consts somewhere?


Thanks! Updated with correct URL.


I can't find --save-dev in npm documentation. What is the purpose of this switch? Is this switch necessary?


Here there is a quoted text, followed by an interruption, and then what it comes afterwards is an indirect quotation of the letter.


I don't think putting this section here works...
But everything else looks like it's right.

I think this section needs to move to a different place... I'll comment where I think it should go....


ermm.. I am not familiar with locales. Is there any way at the moment to get any time zone information (even if it is just a standard offset) out of a locale?

It is even worse with DST's


@tomason Logged here https://issues.jboss.org/browse/JBIDE-20504.  We need to probably handle projects as a special case.


expect(@current_page.text).to include 'reg'


I think the other section should go somewhere here.
Model it on the "BaseConfig" parameters above.
But then make something like... "BaseCompileConfig"
and make the preBuildDependencies add there.

Then make all the latter steps use "Using(.BaseCompileConfig_debug)" etc instead of BaseConfig that they use now...

What do you think?


Specifically... `highlights.py` tries to import `lrrbot.twitch` for some utils. Which means importing `lrrbot`. Which would create a global `LRRBot` instance. Which would create a `TwitchWhisper` instance. Which would create and schedule an `asyncio` task to pump the throttle buffer. But it would never run the event loop, so the task never got processed. So `asyncio` would complain. So `cron` would send me an email every day with its complaints.

So what we've learned: if you really want me to fix something, best way is to have an automated system send me an email about it every day, and then I'll get right on that like a week later. (please don't do this)


> Maybe this should find its way to the ImageJ wiki at some point.

[It already did](http://imagej.net/Git_topic_branches#Advanced_topic_branch_editing_.28AKA_rebase_on_drugs.29).

Johannes also wrote a [nice guide for the NAR project](https://github.com/maven-nar/nar-maven-plugin/wiki/How-to-contribute#incorporating-fixes-into-the-pull-request-ie-rewriting-the-topic-branch).

And there is a [great guide in the Git book](https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History).

And a [great guide from Atlassian](https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase-i/) (the Bitbucket people).

And two (!) nice guides from GitHub: [About Git Rebase](https://help.github.com/articles/about-git-rebase/) and [Using Git Rebase](https://help.github.com/articles/using-git-rebase/).

If things are still not clear enough, please, feel free to edit/rework/reorganize the ImageJ wiki to make things better.


На обеих бордах (подозреваю, на всех с тинибордом последней версии) существует мультипостинг, но со скриптом он несовместим.
По факту, отправка картинок не работает.
Использование костыля
```js
			fixFileInputs: { value(el) {
				var str = '';
				for(var i = 0, len = 5; i < len; ++i) {
					str += '<div' + (i === 0 ? '' : ' style="display: none;"') +
						'><input type="file" name="file' + (i === 0 ? '' : i + 1) + '"></div>';
				}
				$id('upload').lastChild.innerHTML = str;
			} },
```
как на `8ch` не помогает - ошибка `you like a bot`.
Чертов тиниборд.


- Why don't use lastfm? http://www.last.fm/api/show/album.getInfo
- How is the album art exposed? aka which API call invokes it?


haha, les verbes irréguliers réguliers.
Merci.


Does Idea do this reparsing internally in normal mode?


what's up


Yes, but below, where the tags are explained, the closing tag is mentioned, so I thought they'd be mentioned in the overview too.


**[phpUnit_trunkIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkIntegration/8051/testReport/) for this commit**


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


What about making another AJAX request for all your tasks here? Now, in a second `.then` block, you can use the information you received from the `/lists` and `/tasks` AJAX requests to render your lists and tasks.

It's hard to visualize these things so happy to talk about it in person this week. Let me know. The solution makes use of this approach as well.

Awesome job so far though!


Really?!  Is that the best you can do?  Just kidding....I have no idea what this means...good work.... ;-)


А mlpg.co вообще дохлая параша, хотел совсем выпилить ее упоминание.


I'm afraid there is still a chance of a ConcurrentModificationException here.


Please move the description above the homepage.


Because by this point if it's not on the main thread, its on its own thread
At least, that's the idea


На тиниборде последних версий появился встроенный апдейтер.


谢谢你的贡献


On Mon, Aug 24, 2015 at 8:44 AM, dscho <notifications@github.com> wrote:

> Thanks!
>
> I would like to ask for an amended commit message:
>
>    -
>
>    in the commit message, let's describe the why rather than the how (the
>    latter is redundant with the patch, the former is more interesting to
>    developers studying the history):
>
>    Mingw: verify both ends of the pipe () call
>
>    The code to open and test the second end of the pipe clearly imitates
>    the code for the first end. A little too closely, though... Let's fix the
>    obvious copy-edit bug.
>    -
>
>    We also ask you to sign off your patch (see
>    http://elinux.org/Developer_Certificate_Of_Origin for details)
>
> Could you amend the commit message with git commit --amend -s and then
> force-push to update the Pull Request?
>
I'm sorry for the late answer. I will do the suggested changes in the
commit message and update the pull request as soon as possible (in the next
minutes).

--
Jose


If by "unwanted attributes" you mean some default ones such as @default="false" or @N="part" that are generated after an XSLT transformation because they are part of the TEI Schema, you can get rid of them:
- In the XSLT debugger view of oXygen, click on the blue screw next to the Saxon version menu to as to open the "Advance options" window.
- Under "Saxon -HE/PE/EE options", uncheck "Expand attributes default ('-expand')"


This must be `"websocket"`, otherwise Chrome bails.


I didn't think so. It is a poor man approach to fire VFilePropertyChangeEvent for changed file.


@birdsarah take a look at https://github.com/davehunt/pytest-selenium/commit/4e2f58756a699ebabd244a5632e2376c7684c4a1 and let me know if you have any feedback.


> do u means I should just add new changes for feature, don't change any format in whitespace?

Either that, or you make one commit that fixes white-space (and I can easily verify that `git show -s <commit>` comes up with an empty diff, i.e. there are really only white-space changes), and then *another* commit that introduces the functional changes.

> I have test my patches on the Git for windows 2.5.0, it work. and, I will port this pull request to git for windows 2.x

Perfect. I look forward to seeing that Pull Request.


I'm fine with this.


Because in this document "keyword" word appears more than "key word".


Thanks! We are close to kickoff and have a few spots left. Could you send me a few lines about yourself to ahk@e-conomic.com?


I am not sure if we should use `childOption` here. According to my test it didn't not really have an effect unless I use option.


Yeah, at least when setting run_async to true for a queue and running start() on it, it'll always be async by this point.
I just added the check just to guarantee that it's never on the main thread, it shouldn't really decide anything else about threading


Pedantically, no. This is a list of things. Tuple could have been "faster", but this is more semantically correct. Tuple is a struct, e.g. field one is name, field two is email etc, a shortcut for an object, each index has a unique meaning. List is a list of homogenous values.


If there's an issue transmitting the read response, how can you send an RNR NAK?


@Toxaris  the above datatype is what we talked about yesterday. Only with one bit of additional information: `Stream[R x (1 + Parser[Any])]`. The implementation is just like the idea rather ad hoc and needs reconsideration.


TeamCity pull requests :: netty [Build 108](https://garage.netty.io/teamcity/viewLog.html?buildId=711&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5183, ignored: 21; resource leak detected Build time: 00:31:04


Now this is novel for our framework since `decl` can be written without any knowledge of what will follow.


To be strictly correct, if an object is exported, and its meant to be private, then it must start with `_` or `__`. In this case the function is inside a function, not visible from outside, so by definition it is private, and thus no need of `_` etc.


i was able to get bower


Most tests don't work right now since `flatMap`, `delegate` and left recursion is not implemented, yet.


It looks like if there's an incoming packet, but there was an already outstanding read reply, the code will ignore the packet until it completes the read. Is that true?

This is wasteful, and perhaps not according to the specs (not sure). The new packet may be a read request retransmission, requiring the responder to start sending the responses from an earlier psn.


is it app-tutorial-php?


Hm, the commit message is wrong, right? You want to say that you skip 1 and 3 and only do 2?


Thanks a lot, I fixed it :)

You can localize it by,

abp.ng.http.defaultError.message = '...';

you can use abp's client side localization here.


This code could also be made simpler (like the requestor part) by some helper functions that extract the first/middle/last/only info out of the opcode.


Good catch! Thanks :)


I would move this to the ConcurrentDistinctMultiValueMap. The code is anyway bound to this special map implementation.


Hi @frankmorgner, this commit is still causing problems for me.

I get an error performing (RAW RSA) signature operations using a Yubikey NEO (card-piv.c driver). Git bisect narrowed it down to this commit.

     pkcs15-sec.c:481:sc_pkcs15_compute_signature: Private key path ''
     sec.c:68:sc_set_security_env: called
     card-piv.c:2209:piv_set_security_env: called
     card-piv.c:2213:piv_set_security_env: flags=00000014 op=2 alg=0 algf=00000000 algr=00000000 kr0=9a, krfl=1
     card-piv.c:2240:piv_set_security_env: returning with: 0 (Success)
     sec.c:72:sc_set_security_env: returning with: 0 (Success)
     sec.c:54:sc_compute_signature: called
     card-piv.c:2340:piv_compute_signature: called
     card-piv.c:2269:piv_validate_general_authentication: called
     card-piv.c:447:piv_general_io: called
     card-piv.c:450:piv_general_io: 87 07 9a 266 : 65536 65536
     card.c:353:sc_lock: called
     card-piv.c:490:piv_general_io: calling sc_transmit_apdu flags=1 le=256, resplen=4096, resp=0x7ffc813adc10
     apdu.c:563:sc_transmit_apdu: called
     card.c:353:sc_lock: called
     apdu.c:352:sc_check_apdu: Invalid Case 4 short APDU:
        cse=04 cla=00 ins=87 p1=07 p2=9a lc=266 le=256
        resp=0x7ffc813adc10 resplen=4096 data=0x7ffc813aed90 datalen=266
     apdu.c:617:sc_transmit_apdu: inconsistent APDU while chaining
     card.c:393:sc_unlock: called
     card-piv.c:496:piv_general_io: DEE r=-1300 apdu.resplen=4096 sw1=00 sw2=00
     card-piv.c:498:piv_general_io: Transmit failed
     card.c:393:sc_unlock: called
     card-piv.c:558:piv_general_io: returning with: -1300 (Invalid arguments)
     card-piv.c:2322:piv_validate_general_authentication: returning with: -1300 (Invalid arguments)
     card-piv.c:2398:piv_compute_signature: returning with: -1300 (Invalid arguments)
     sec.c:58:sc_compute_signature: returning with: -1300 (Invalid arguments)

It's not included in the OpenSC log, but with a couple of extra printfs I found that before this commit, I had `card->reader->driver->max_send_size = 0` and `card->reader->driver->max_recv_size = 0`, after this commit, `card->reader->max_send_size = 65536`, `card->reader->max_recv_size = 65536`.


is this debug message relevant in case of create a volume from snapshot?


you can remove the other TS drivers since they are not present on device


Hi, thanks.
I will check on whis week.


:+1: good catch, thanks for the update!


O commit se refere à estilização, onde estão as configurações de estilo?


There have been a couple more changes to the Paho API and I didn't keep this updated, I have made some changes now so if you pull and build it again it should work.


in review they can ask why 5 seconds.


String hardcoded no código? :(


Como a Toolbar pode ser utilizada em várias Actvities seria interessante criar o layout em um arquivo separado e fazer um include do layout aqui.


This is not needed, since you flatten the results anyway in line 177. Works without.


done


we were checking in dothill_common.py that there is sufficient space for this operation. then do we need to check it again?


Can this really happen?


Acredito que este commit de merge era desnecessário.


Thanks, need some adjusts, but OK.


This should work properly cross-app. However, the slide-in transition was useless here. If we don't know how high the ad is gonna it won't animate. Checked in Community [here](https://github.com/lonelyplanet/community/pull/2389).


(noot)


LMAO, type -> typo


I still think you should look into a proper time library. But this is a fine placeholder.


kkkk


Please add a space after `,`


Please add space after `//`. It should be also capitalized, and finish with a dot.


run-and-gather-output returns a list of Pairs now that you List-assign to a list of Pairs.
Previously it didn't matter in which order run-and-gather-output returned the pairs, because we bound a Capture to the list of Pairs and it did the right thing. We lost this nice ability. I'd say we should not pretend that it still works that way and use a simple list assignment instead:
my ($output, $stdout, $stderr, $passed) = run-and-gather-output(...);
and a plain return $output, $stdtout, $stderr, $passed in run-and-gather-output.
That way it's clear that no magic is happening and we don't need the .value in line 33.


Add space after `if` or `for`.


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


a


Same comment format applies to here. Start with a capital letter since the comment should be a sentence.


Loud and clear. Thanks.


事世要变了


Comment rule.


@swfrench my bad, I completly missed that point.
imho, the cleanest option is to have two libraries
 - libipm.a for c
 - libipmf.a for fortran
if libipmf needs libipm, then the IPM_* symbols cannot be made static
(I made a proof of concept, and I can make a pr when it is complete)
a "less worst" option is to add to libipm.a pmi_***_ weak symbols to make the c linker happy
(note I did not test this part yet)


Add a space after if


do `} else {` in a single line.


I think so (not being a {c,p}ython here). Thanks.


Completed NormalizeFastaIntegrationTest successfully (with help from @lbergelson)


Thanks!!!!


No line with only white spaces.


No white spaces before the end of the line.


Whoops, that shows how observant I am.  The very next commit.

@bgamari I would admittedly feel a bit intimidated trying to talk to them; the extent of my knowledge on this matter is pretty much the man pages. Are you interested in bringing it up with them again?


TeamCity OsmSharp :: GitHub - OsmSharp [Build 1054](http://build.osmsharp.com:8080/viewLog.html?buildId=5184&buildTypeId=bt16) is now running


I guess `unsafeLogWith` confused me, everything is probably totally safe ;)

Now that I think about it, what I don't get is why you created your own interface, instead of just providing an implementation for the `error` interface. It all feels very hacky and unnecessary. But anyway, this just a philosophical debate, you're not going to change it. But I'm glad I understand it a bit more (or rather, that I understand that it's okay that I don't understand it).


Thank you


。。感谢，愿安好。


You totally rocks, well done !
With the concat :+1:


What's the point of this function?
RDMA length checks cannot be done without checking the rkey first. Is it for the WQE length? Isn't that being done as part of the copying?


感谢，祝好


Can we lock these in to a specific version? For the other Polymer elements we've been using the caret. https://github.com/PolymerElements/paper-elements/blob/master/bower.json#L27


This will return `true` when the label is empty or unset, won't it?


I'd like to see either result moved into the enclosing if here, so we don't have another variable at top level, or move the function call into the if and avoid the result variable completely.


At one point I needed a custom error outside of `errs` that would implement that interface and was thus usable with `errs.Log` and stuff. But I have a feeling I didn't need it in the end, so it's more like a historical leftover I guess now. But it's not really a pain, so I haven't changed it, there was no need.


For private/element-only methods, can we prefix them with an underscore? e.g `_valueChanged`?


please add a message with D(()) here so a user with debug turned on can see what's happening. that should probably happen in both cases when authgroup is configured.


If it fails, you should return the read/change callbacks to the original ones.
To make it easier I would replace the callbacks after this check.


1) fair enough.
2) no it isn't. You don't need a factory if you only produce one object. So you can remove one layer of useless bureaucracy ;)


thanx man


Hey!

I assume that the logging is too chatty for you, is that correct?

I would certainly accept a PR changing the log level, if you have the time to put one together :)

Thanks,
Cory


And so what are constructors good for? :-)


yes.. should be removed...


Maybe you could use the same style as on line 443 above?
This line would just be:
```
    reddit,
```


I am amused.


Unless these includes are needed by anything else than util.c I think it's better to have them in util.c instead (might be true of other includes here as well).


В общем, я был прав.
Если имиджборд не содержится в ibDomains, а инициализируется через ibEngines, то в случае, если в init используется Cfg, возникает ошибка `TypeError: Cfg is undefined`.
Пример: http://latchan.lv/b
Тут два варианта. Либо прописывать в ibDomains

```js
get 'latchan.lv'() { return [ibEngines['form[name*="postcontrols"]']]; },
```

то есть вписывать все и звестные нам тиниборд-доски, либо избегать использования Cfg в init.


:+1: cool added to - sass/partials/_type.scss


Lol I remember when all you did was ModPE [here](http://github.com/trek it/eathook)


add a : after "cti-users mailing list"


This is a working but temporary version I didn't implement a proper timeout option nor do I want a "final" release to rely on an external module like websocket-client.

Writing my own websocket implementation would probably go beyond the scope of this, but maybe it can be included in the project like flashmedia?


Oh, great! @ctrueden thanks for the links!


I don't think anyone cares. Even I don't :D


Thank you


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.4.0.1054](http://build.osmsharp.com:8080/viewLog.html?buildId=5184&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 548 Build time: 0:0:0


Thanks.大家都来访问会增加github负担吧。毕竟正在被ddos.


Yeah, good catch. Thanks Jan. I did test it, by creating two parameters: one with a label and one without. Then I verified that at runtime, one of them had a label and one didn't. But apparently I was tired and didn't notice it was the wrong way 'round. Doh!

Fixed in dec79224e0347c3a4562a3ab2424e097f33e41eb.


:tada: congratulations :tada:


ありがとうございます、take care of yourself.


Why the change? Should `error` is be initialised too?


Done


Note to muself: Allowing to execute code from url -> limit to public methods ? + unnecessary redirection, just go to Inbox


Loved to read all those comments! :smile:


Seems louche.


:fireworks:


Ok, thx I'll change that!


This doesn't make any sense. It's the same as `opts.key?(:fanout_persist)` apart from it returns `nil` when `:fanout_persist` doesn't exist.


commiting


Okay, thx!


It is great to know that, thanks!


Planed -> Planned


Hmm, well ideally that `chunks` array should be turned into something like a `LinkedList<Chunk>`, but I'm not really too excited about making larger changes to an internal class that will eventually go away anyways.


same here


No worries! Thank you very much!


LGTM


Good idea! Done!


@mrdejong not sure. Probably just a stupid copy-paste mistake. :/


@differentreality: When I log as info-desk, I still see this button. He is not admin. How can I fix this?


What about just preserving the inS3 property inline at this point?

	if lastCompressedUpdate.inS3 and not _.some(compressedUpdates, (update) -> update.inS3)
		compressedUpdates[0].inS3 = lastCompressedUpdate.inS3

with an _ = require 'underscore' at the top


@AdenFraser almost certainly


Thank you


\o/


Great, having to implement the fix as a temporary measure through an extension at the moment... so would be nice to get that removed when a 3.1.14 release is rolled out :+1:


vaddr can be NULL for reasons other than misalignment. Then the reported error will be wrong.


Yes, that is what I meant. Thanks.


This should be done via filesystems supplements, e.g. btrfsprogs has "Supplements: filesystem(btrfs)" in the spec file.


True, true


Will be removed as soon as **glr** becomes master or gets merged into master.


It's better to invert condition in such cases.


You can still write 'Please' with a capital letter at the start of the sentence. :smiley:


You missed this, Davis


Why put this check (and None assignment) after passing the `options.cache_dir` to `WheelCache` ?


keep this value as a constant


word-break: keep-all;

Co sądzisz o: https://developer.mozilla.org/en-US/docs/Web/CSS/hyphens ?


Instead of `var instance = this` couldn't you have also used Template.instance ?


remove this space


Yes, you're seeing what you think you're seeing, but it's not necessarily meant to be a new or different mark. I believe that actually is a period with the type block put in upside down by mistake. I think it's a <sic> kind of situation unless we see a lot more of them.


STATSのSEをつける #740


This is overriden by the code below, right? What do you actually need from the incoming packet?


This is good code to have because we're going to have around twenty interpolated letters in Amadís. Elisa--is this the way you would code the letters?


Probably code could have been simpler if you had split this class to two configuration for different editors.


:heartpulse:


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


:+1: IMO that should improve things a bit. thanks.


Testing


What logic is behind 4 fields and one function? :)


I agree with @matjames007 on this one. Having 777 on a web application isn't really secure (even if we are using a cloud service). Generally 755 are safer permissions, or at worst 775. That aside, file permissions shouldn't be how you try to fix a coding issue. Its probably indicative of an "overreach" of some other application function.


Where does the code calculate ICRC of the payload?


Unrelated to the patch but since you touched for reformatting: this `7` is probably a copy-paste error, I think you could even omit the size here (and in others) since there's a literal definition.


And where do you check incoming icrc by the way?


Yup, exactly on this one. Of course always be careful with this fix since its not going to just test if the class is "list" but if the class or base class is list. Of course, if we wrote code where it mattered, that would be bad.


Just looked really funky on Github where tabs = 8 spaces


could you also add a test case please?


```
<script
   class='stripe-button' src="https://checkout.stripe.com/checkout.js"
   data-key="<%= @stripe_btn_data[:key] %>"
   data-amount=<%= @stripe_btn_data[:amount] %>
   data-description="<%= @stripe_btn_data[:description] %>" >
</script>
```


Would moving this to be private instance variable of `Mobile` have a reasonable impact on performance by allocations?


I don't recommend this approach. The server side doesn't guarantee the order of the output. It may be that the `root` disk is displayed last now, but that may change in the future, even with a minor release of the server.


I suggest going with WSGI, since in less than 3 months puppet-keystone will drop eventlet configuration and set WSGI as default.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 404](http://192.168.30.225:8111/viewLog.html?buildId=2227&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Sorry that was not clear, is it required to create a new instance every time the priority is changed?

Cause you seem to be reusing the manager1HumanTaskClientApiClient, and resetting it with the "newPrioirty*" and assigning it to TTaskAbstract, so i was wondering why cant we reuse the same instance?


This commit destroys the life-time objects. Rejected.


@HelenaSabel About those expanded attributes, yes: We determined that on Stacey's installation of oXygen, the "Expand attributes default" was checked, so when she ran the XSLT it was generating  these additional attributes. We fixed it, but thanks for recording the solution here.

The reason I wrote the portion of the XSLT that is now commented out is because the XML file was saved in Dropbox with the expanded attributes as well as possibly some new markup that we actually do want. I wasn't sure I could find an earlier version of the file to restore that wasn't missing the new markup, so this code just "scrubbed off" the extra stuff. Now that we are on GitHub, we may not need to use this code anyway!

Note: our line comments will disappear after we push a new version of this file, so for something like this that we might want to save for long term, we can try writing it in one of the Wikis or as an Issue. I'll do that in a mi ute to illustrate how these work.


Thx~


(y)


In Python 3.3 they [reintroduced the unicode-literals syntax](http://legacy.python.org/dev/peps/pep-0414/). So if you want to make it work with both versions, I would choose in favor of it.


This was actually for bug#9460.


I know we want this, but adding it to the readme or contribution guidelines doesn't seem to help at all. New users come and ask questions without reading these docs, and I don't blame them - there is no way to make contributor guidelines obvious on github, except for the tiny yellow box on the "New issue page" which only contains a link to the guidelines

How do other popular open source projects deal with this? Closing issues is not really an option, as its very draining for both the person and the maintainer (the person is slightly put off, and the maintainer [may feel guilty](http://byfat.xxx/what-is-opensource-and-why-do-i-feel-so-guilty))...


Yeah @developit when I 'sass' I mean the 'scss' version. 👍


use pillow, not pil


This is breaking simple_query in our app - did you forget to add it somewhere?


Please squash this extra commit


cool, I didn't know there are such things available :smile:


we're basically getting now this wherever we do simple_query

```Erlang
::{{nodedown,disconnected},
   {gen_server,call,
               [{error,disconnected},
                {do_query,{simple_query,"truncate users",[],infinity}},
                infinity]}}
Stacktrace:
[{gen_server,call,3,[{file,"gen_server.erl"},{line,212}]},
 {pgsql_connection,call_and_retry,4,
                   [{file,"src/pgsql_connection.erl"},{line,1432}]},
 {fc_api_pgsql,'-truncate_all_tables/0-lc$^1/1-1-',1,
               [{file,"src/fc_api_pgsql.erl"},{line,14}]}]
```


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


@spion I asked about this a while back, no good answers https://opensource.stackexchange.com/questions/18/how-do-i-stop-people-from-opening-questions-in-the-issue-tracker


File...


Плохо ставить 1 в названи переменных


Have we ever considered increasing/decreasing this?


Aha, but you didn't sign your commit in the past, did you? So how could we validate the key and know that the key truly belongs to who you claimed to be.


Bonjour,

merci de corriger les erreurs de frappe => tempalte par template dans les deux fichiers.

Cordialement


LGTM


ок


That must be "nfs-client".


Instead of providing ```src_snap_name```, how about the snap id?  That way you can retrieve the snap record directly via ```key_from_snap_id```


What are the cases when this method returns null?


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


If you remove '=' all'll be ok


etggm,


You will need to scan all the records to ensure the ```dst_snap_name``` doesn't already exist.


remove this
all will work anyway


got it!


need to test what length causes the server to fall over and/or optimize the
search loop

On Tue, Aug 25, 2015 at 9:15 AM, Scott Snyder <notifications@github.com>
wrote:

> Have we ever considered increasing/decreasing this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/foursquare/twofishes/commit/f1980f39432574b5087a1223afad2739c37453e2#commitcomment-12879286>
> .
>


That would be additional improvement to make in another pull request. I first try to minimize visibility of fields as much as possible.


`it's` -> `its`


Null is returned when the editor is opened without file, i.e. for built-in source.


missing ']'


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 404](http://192.168.30.225:8111/viewLog.html?buildId=2227&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:12:12


'target'


Elisa, do you know a way to make that option the default behaviour? I have
to uncheck it everytime I open oXygen (which it's very annoying because I
forget most of the time forcing me to run my transformations twice). I
couldn't find something similar in the "General Options" menu, so I don't
know if you're actually supposed to uncheck it everytime or if it is a
quirk of my own oXygen installation...

2015-08-25 15:09 GMT+02:00 Elisa Beshero-Bondar <notifications@github.com>:

> @HelenaSabel <https://github.com/HelenaSabel> About those expanded
> attributes, yes: We determined that on Stacey's installation of oXygen, the
> "Expand attributes default" was checked, so when she ran the XSLT it was
> generating these additional attributes. We fixed it, but thanks for
> recording the solution here.
>
> The reason I wrote the portion of the XSLT that is now commented out is
> because the XML file was saved in Dropbox with the expanded attributes as
> well as possibly some new markup that we actually do want. I wasn't sure I
> could find an earlier version of the file to restore that wasn't missing
> the new markup, so this code just "scrubbed off" the extra stuff. Now that
> we are on GitHub, we may not need to use this code anyway!
>
> Note: our line comments will disappear after we push a new version of this
> file, so for something like this that we might want to save for long term,
> we can try writing it in one of the Wikis or as an Issue. I'll do that in a
> mi ute to illustrate how these work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ebeshero/Amadis-in-Translation/commit/afe627d3c28d558db83e9b72a5549df50e3cbdb4#commitcomment-12879166>
> .
>


#4952-VRF-hard-crash-if-terminal-unit-list-error (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1277 of 1277 tests passed)

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


Dunno what's wrong. It works fine for me. From the log, it looks like there are missing files or it can't load dlls. Dunno try recompiling the source code as a release.


I think we should add a pointer to the [fault detection](https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-discovery-zen.html#fault-detection) section of the documentation at `timely fashion`.


Wenn wir schon dabei sind hier... @heisig @sjmielke wollt ihr den Alias studienberatung-minf@ifsr.de auch auf Philipp's Adresse weiterleiten und nicht mehr auf meine? Das wär toll :)


please correct: SHORT(patch->width)


`loose` -> `lose`


アンロックのSE作成 #739


Projects for the SoftUni course


Maybe getJavaProject() should be exectued here?


Sorry, I thought about Append method :)


`time until detects it and master` -> `time until it detects this and the master`


This code is not thread-safe.


deluge should be added to these as well as it shows a cmd window as well but should not.


`ack` -> `acknowledge`


Understood! Let me change it to a normal dot since we can always change it back if we find more.


Comments for the inputs / outputs -- plus same suggestion for swapping snap name w/ snap id.


Additional dot (.) after sickle.

- Jacono feat. Jiri


Szkoda, że hyphens prawie nigdzie nie działają...
25 sie 2015 15:01 "Bartek ." <notifications@github.com> napisał(a):

> word-break: keep-all;
>
> Co sądzisz o: https://developer.mozilla.org/en-US/docs/Web/CSS/hyphens ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CodersCommunity/czatrepajr/commit/fabfb639ad2e576345b083c91de29dc39228393c#commitcomment-12878981>
> .
>


Prefix `minimum_master_nodes` with `discovery.zen.`?


Duplicate `See`?


Hmm sorry - i guess its another dependency that happened to have changed at the same time


everything looks good.

1 suggestion:
Can you include a note:  'We allow http callback urls for only developer apps however you will need ssl/https callback url once you are ready to move your app to production.'


`acknowledges it but do` -> `acknowledges it, but does`


谢谢


can we optimize some code like below?
if backend_type == 'linear':
    self._request(Here api regarding linear)
else:
    self._request(Here api regarding virtual)

then can we use a common check_volume_progress for both by adding one more parameter say backend_type in
 check_volume_progress?
 In titanium,  self._request("/show/volumecopy-status") takes some time around five seconds to show volume copy
correct status, so in the logic we added 5 seconds.
Also, in titanium, once volume copy is complete, its take around 5 seconds to update MC database so we are waiting 5 secs before returning.


Since this code is non-trivial and reading/writing is the same between all the ```old_snapshot_XYZ``` functions, it might be nice to create helper methods to decode the snapshots from the header and to re-write the header with the new snapshot collections.


Ah, thanks Jay


The spec name is still incorrect. No big deal but it's misleading.


Probably no need to hold the md_lock and flush the IO


So simple, what a shame of me...


This addition was for the special case of Mail Channel where follower can decide to choose to either receive One mail per thread(new thread on particular channel) or All the mails including replies to that thread.


Unfortunately, this does not solve the problem. Even trying to execute line 87 will result in throwing std::bad_exception on MSVC. You have to move this #ifdef earlier.


Not sure what this loop is doing


And what behavior is expected if null (for interview tasks)?


Why isn't this dynamic from the configuration in conf - currentLanguage?


Hi @alg5,

should be: ```/versions```


I recommend using descriptive function names. `nwhack` and `nmsify` are useful shorthands, sure. However it's not obvious from their somewhat cryptic abbreviations what they might do.

Note that this applies for others as well and not just these particular function definitions.


You weren't holding the snap_lock -- so the it could have been invalidated at this point.


I'll add a test for this..


Addresses #106 .


As this is the first line of the background it should begin with 'Given' ideally and the second step should be 'And'.


`a fait`


Probably :) This commit only changed a type.


Formatting is ugly in this file. Please check that you have "Insert spaces for tabs" enabled and fix it.


`héberger`


```allow a snapshot to be renamed``` --> ```rename a snapshot```


Addresses #28.


I know. Curently however the code is too hackish. Either we clean the code, either we drop the feature. Will see.


`de la branche v3.x existante`


`de répertoire similaire`


```destinationsnap``` --> ```destination snap```


Done


These are undefined by default, but there should be a test that when you pass a non-undefined value, this value is used.


self.lotto.__len__() 을 직접 호출하시지 말고 len(self.lotto)을 사용하시면 len이 self.lotto의 __len__을 호출합니다.


`Documentation en cours d'une nouvelle`


r- illisible


I agree. Dealing with these kind of misdirected requests makes the project team look mean and feel guilty, as @spion says. As far as visibility of other support channels, it doesn't help. Obviously in the case of jQuery there are lots of places to get help; both StackOverflow and our API site are at the top of every search. However, the more popular a project becomes the larger the 10% of confused/misdirected people becomes.

I've asked Github for a markdown file that would be shown above the box with instructions for correct use, and a template that could be put into the box to encourage reporters to give the right info. The link to the CONTRIBUTING file isn't working, and confused people are the last to read the README.


R- illisible.


`les contributions occasionnelles, disant que les ...`


Hi Aecker. Nice work!. One remark. In the Toy example the input parameter is number of trials (N) not number of neurons (q) which is 16, Am I right?, best


`réaliser leur pull request et`


Private/public is handled through metadata. So you could do something like:

```clojure
(def ^:private LOG (atom []))
;; or, equivalently:
(def ^{:private true} LOG (atom []))
```
`defn-` is just shorthand for using metadata in that way:
```clojure
(defmacro defn-
  "same as defn, yielding non-public def"
  {:added "1.0"}
  [name & decls]
    (list* `defn (with-meta name (assoc (meta name) :private true)) decls))
```

Usually a good goal is to remove all the top-level state you can, preferring instead to inject it into the functions. It's a little more cumbersome, but it's a way to separate stateful stuff from non-stateful stuff. So in the case of `LOG`, you *could* add an argument to both `handle` and `update-log` here for the log (maybe partially applying the `handle` function when you want to pass it around). Just a thought, not something that's urgent to fix. [Stuart Sierra's component](https://github.com/stuartsierra/component) library is a nice practical example of the pattern of passing around all the state together, but don't incorporate that here since you're limited to Clojure+Java core stuff - just something to keep in mind for real Clojure projects.


what do you mean with this? it is not an action it is a pref summary.


`conversation`


@MattiSG ,
This modification introduced a bug. The `code` property is defined as required in [`DroitSchema`](https://github.com/sgmap/ludwig-api/blob/master/lib/models/schemas/droit.js).
Right now ludwig crashes when you try to add another expected result to a test.


Ah. I should fix it. :-D


`les 6 et 7 août`


These were deleted due to changes in the API


some comment


extra`=0` that was removed in lint patch


`a été créé`


You (and I) need to get into the habit of making clean commits. The intention of squashing into one does not matter; if you think "given that I will squash these commits into one it's OK if I do some quick-and-dirty committing in the meantime", then you will have trouble maintaining good committing habits. In addition, such large work could be many commits, it need not necessarily be a single commit.

On the specific issue at hand, i.e. the margins and paddings, I am testing with the tip of the branch, not with this specific commit. In the tip of the branch `custom.css` still specifies some margins and paddings; and if I comment them out, I see no difference. Could you tell me exactly at what screen width these margins and paddings make a difference?


`et plus de 30 groupes`


在线客服系统的测试代码/golang


Try second param as true here as it is the openWhenFinished boolean. Should do the trick.


`écrit`


Thanks @moy, but I think we try to keep the list sorted...


`répertoire`


`les notes dans le répertoire`


removed empty comment from previous commit


Immediately after line 17, `{% if object_list %}`, you open a `<div class="table-responsive">`. You should close that div before line 45 (`{% else %}`), not after line 47 (`{% endif %}`).


The closure has to capture the ```self```, ```ignoreActor``` and ```check``` parameters so you couldn't easily move it around. Luckily, ```GetAvailableSubCell``` doesn't get called much, so it's not an issue.


following openWhenCompleted...


I'm glad there's a sane solution!


Eventually gets us to here...

If !mOpenWhenCompleted, then the file should just download, and not open.


il manque le lien pour [la team d'Évangélisation](https://github.com/nodejs/evangelism)


`"Weekly Updates"`


Nevermind, this needs to be handled differently.


I'm not sure what the profiler would do with the exception even if it got one. Just assert that it's not set and print the message if it is?

Can you expand on the domains issue?


Arigatou, and sayonara.


Yes.  The original reason for appending "Invoke" was b/c this command is specialized to asserteq of an invoke while in theory there could be other asserteq forms allowed in the future.  But perhaps that won't be the case and anyways we just rename then.


Crap, ja som precis fixade det också :P

Du var 1 min före mig


Just looking at this part where you are clicking the button confirm completion on the case list. This might cause issues perhaps where there are multiple rows in the case list with this button. Might be worth replacing with what you have done below where you are finding the specific element of the table and clicking it.


Good point.


I find this example particularly tricky because it goes from a direct quotation to being reported. If in the letter the only written text is: "Este es Amadís Sin Tiempo hijo de rey"
Then it should be coded with something similar to this:
    <milestone unit="writtenText" resp="#Darioleta" ana="start"/>Este es <persName>Amadís Sin Tiempo</persName> hijo de rey: <milestone unit="writtenText" ana="end"/></cl>
    <cl xml:id="M1_c215">y sin tiempo dezía ella porque creía que luego sería muerto:</cl>

However, if the explanation of the name is written in the letter, I guess it should be more accurate to do something like this:
    <milestone unit="writtenText" resp="#Darioleta" ana="start"/>Este es <persName>Amadís Sin Tiempo</persName> hijo de rey :</cl>
  <cl xml:id="M1_c215">y sin tiempo <milestone unit="writtenText" ana="intStart" type="reported"/> dezía ella porque creía que luego sería muerto:</cl>
<cl xml:id="M1_c216">y este nombre era allí muy preciado porque assí se llamava un sancto a quien la donzella le encomendó.<milestone unit="writtenText" ana="end"/></cl>


Bare revert denne hvis det feiler igjen