Hi @microvolution,
- GTWVG examples found in 3.4 would only build if you use Harbour 3.4 and the contribs that come with it (`gtwvg` and `gtwin` for the most part). They _won't_ build with the 3.2 version of these contribs. You can find fresh 3.4 binaries under **Releases**: https://github.com/vszakats/harbour-core/releases
- As for the issues, please make it clear if these were experienced using (or suggested for) 3.4 or 3.2.
- Depending on the version involved I suggest reporting these problems - along with minimal examples that demonstrate them - inside the respective repositories' Issue page:
   - in [3.2 (upstream)](https://github.com/harbour/core/issues), if it's reproducible there. For these cases it's probably the best pinging Pritpal (@bedipritpal), author of GTWVG, and settle on any required fixes/patches together (I'm not a GTWVG, or Windows GUI user myself). The [Harbour development list](https://groups.google.com/forum/#!forum/harbour-devel) is a good place to reach him.
   - I can help with any 3.4 specific regressions, if you [report](https://github.com/harbour/core/issues) them with said minimal example.
- Thanks for the Bible reference, I'll try looking it up.


lock and commit readme delete


`ctx.fireUserEventTriggered` will fire it to downstream handlers - this handler will not receive that event.  We want to handle the event ourselves in this handler.  User events is only for communicating custom event types with other handlers, not for communicating with ourselves.


fast-forwarding 2.1 to borshop-integration-1282-ku/switch-rt-dvv = 59bf19bb


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1450 of 1450 tests passed)

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


reenable this at the end of test case, or does it automatically reset for each test case?


Joe, this looks like great work, I'm doing a private import and test now.


For the record, removing `dev` from the component list was a conscious decision.  It was a no-op component in this build configuration anyway.


OOC: isn't all `app` folder loaded on rails starting process?


Yep - but for any sub folders rails is expecting a namespace.

So `Params::UserPolicy` rather than `UserPolicyParams`


Also fixed bug: reading opcode in wrong order; fixed instruction to find correct font location


By adding it here rails will include it without the need for a namespace


Should be spelled "CANDIDATES", not "CANDICATES". And these days, OMZ prefers the double-bracket `[[ ... ]]` form, which you can use with `==` and `>`.


OK, one last thing: maybe we should check for null on the value first? Otherwise we might get a NPE in the `else`.


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


We're running some pipelines where the configuration options have to be supplied during setup-time. To prevent super-long commandlines, I added the module "Getopt::ArgvFile"  - this allows to pass all commandline options via a file.

Usage: perl init_pipeline.pl Bio::EnsEMBL::Hive::PipeConfig::Whatever_conf    @command_line_option_file.conf

Format of command_line_option_file.conf :

--option_1 value1
--option2 value_2

See CPAN for more information


felicidades con todo lo que están haciendo... enserio, se lo merecen


you are doubi


It's work with my case when I meet the error message.


@ask ok, got it


maybe Packages-\<type>-\<brand> as in: **Packages-Netrunner**, **Packages-Manjaro**, **Packages-Live-Sonar**


Fixed in 5a522855e8fd99f24708877adf3b024c7d1e8651


Yes, I'll fix it.


You are the best, maaan!


Caused https://issues.jboss.org/browse/WFLY-4554


Yes, we will delete this paragraph.


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


In the future:

```
<p>Note that this step is shorthand for the generic SCM step:</p>
<pre>checkout([$class: 'SubversionSCM', remote: 'http://sv-server/repository/trunk']]])</pre>
```


This repository is meant to provide an example for *forking* a repository on GitHub.


Should've called this commit derpy mcderpistan.


the forward reference is enough.


I would prefer using 'default_only' on a YAML attribute such as 'mt_full_support' or 'mt_lite_support' instead of compare product id in the ruby api file


yeah, we agree.


https://github.com/scottschurr/rippled/blob/18723a4137ae78c04a62e21812b5104ff48b58d7/src/ripple/rpc/handlers/AccountObjects.cpp#L141

Without a SignerList you return false from getSignerList, then false from getAccountObjects, then return RPC::invalid_field_error (jss::marker) from doAccountObjects.

Your branch differs from develop (https://github.com/ripple/rippled/blob/develop/src/ripple/rpc/handlers/AccountObjects.cpp#L140) and the bug would go away with a rebase.


Why sizeof vs strlen?


Congrats on these incredible results!  (Perhaps there is opportunity to tune this with SPSA and some of Marco's genetic switches?  MC had a post on earlier in the year about this.)


Might be an idea to actually name the class.


Not sure how much difficulty it would be to replace underscore with lodash...  But we have been replacing it as we can.

I think this could be handled nicely by https://lodash.com/docs#difference

Generally, try to use Array.map or other as opposed to for loops.


yeah!


@SunilMohanAdapa
when I deactivate the WAN connection plinth always throws an exception when I click on any connection.

Exception Value:
'NoneType' object has no attribute 'get_uuid'
Exception Location:
/usr/lib/python3/dist-packages/plinth/network.py in get_status_from_connection, line 105

adding:

```
if primary_connection:
```
to line 105 of plinth/network.py solves this issue


This should remain 0.2.2 until this sprint is over.


rest is fine to me. I didn't see the interface.yaml/rb change where 'no shutdown' is applied to service instance applied to interface.


hahahah
https://github.com/VSEphpbb/lightbox/pull/23
That is my PR plz

I'm sorry
I made it ready once you sent us but it refused to send PR due to we have the most bad internet in the world.

Thanks


@gerkey PEP8


nitpick: In Go, error messages usually start with lowercase. http://blog.golang.org/error-handling-and-go


Is it really important to indicate the seconds here ?


:heavy_plus_sign: :one:


I hadn't considered that approach. Good idea.


Superr :)


`%a%d` is replacable with `%w`


Nice! Sorry for not merging this sooner, I have no idea how GitHub works


Ilya, I am not sure I agree with your claims here with JS disabled.

What the server can observe is the *next hop* RTT and the *next hop* download speed - which may be the client, or may be any number of intermediates (such as proxy servers). So the sweeping claim that the server can observe the client RTT isn't backed up.

I think it's important to consider and capture that nuance here, since in those situations, new information is being made available to the server that was previously mitigable by user action. Similarly, when considering the "JS enabled" scenario, we need to fully consider existing user mitigations (if any exist) and compare them with the new information being made available, and determine whether any additional mitigations need to be explored - either by glomming onto additional mitigations or introducing new mitigations. It may be that with JS enabled the issue is unmitigable, but we shouldn't be making broad statements - that's sort of my fundamental objection :)


Edited so it's correct


directive parameters are typically namespaced with brFoo as here: https://github.com/digitalbazaar/bedrock-angular-identity-composer/blob/99dbc76bc74bf5b096418352bcb8a099d9a3637b/identity-composer-directive.js#L16-L22

but, in the event that the parameter matches the variable name in the scope, you can abbreviate to '='


@c4milo Thanks!


Again, see above remarks about considering how users can and do mitigate this.


Thanks


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 120](https://teamcity.labkey.org/viewLog.html?buildId=292326&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:03:15


yes!


Finishes #24


Yes, or tags maybe? >sonar >netrunner >manjaro ?
Do we get all in one file with tags? I think so, but not sure.
Building sonar takes very long, 45 mins download alone.


I suspect that ProcessCancelled could be thrown from somewhere as well, although this is a bit overcautious, I agree.
Making it serializable doesn't help, one need to make sure that serialization id is not changed, and this is simply easier to maintain inside daemon code.


> What the server can observe is the next hop RTT and the next hop download speed - which may be the client, or may be any number of intermediates (such as proxy servers). So the sweeping claim that the server can observe the client RTT isn't backed up.

Ah, fair point. Let me take a run at this..


Not sure if it was here it happened, but at the moment this is pretty much a copy of text from above, under "Incorrect URIs". (Should that entire section be removed perhaps?)


Oops, yes, sorry. I'm not sure how I got to that. One thing for sure is that this is needed for ctrl+left but I guess it might break other things. Will need to revisit this.


Nice solution. Thanks for jumping on it so quickly.


This should be after the `org.` imports.


I wasn't aware of this, thanks.


I have Question, What is the BuildProcessor.jar and what that jar doing?? helping compilng? And Why push @NeedsNoArgConstructor?

Sorry for my Bad english and no Java knowledge ;.;


parens are not required for typeof.  typeof foo === 'object' is the preferred syntax.


I see the problem. How about {(X, Y)} ? What do you think?


Probably not.


Can you put an empty line before and after the `<p></p>`?


This is awesome :+1: Good work, Charlie!


A few comments:
* Split `{amount}` in `{amount} {currency}`, you can't be completely sure how locales write currency (symbol before or after the number? Space or not?). Also add a comment explaining what these placeholders are, and an example for English.
* `tip our hat`: if that's associated to the dapper fox, please explain it in the comment so that locales can try to use the same image.
* Explain how the `<i>` will be used and why it should be maintained.


I dont know why my 'Git for Windows' dont catch this (common/countries/Refuge Point.txt → common/countries/Lordaeron Remnants.txt) in my file changes...happy to see that your eyes are open :+1: !


yes, I totally agree. I think this is out of place and we don't need it. the real reason, as you say, is FTEUSODE. Can you make the effort to move this whole thing to that section?


This will cause issues with the dependency resolution. We can't use a fixed `parser` version - it's not like we don't know the consequences. :-)


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


yes, do them both ways


While you're at this - mismatches -> doesn't match the number of


seems ok to us.


Let's get this merged in! I didn't know about the slack community until browsing through the PRs here. I would love to see that on the community page :)


this commit breaks R CMD INSTALL ... not sure why so am removing the txt file for the moment.


`sizeof-1` and `strlen` can be mostly used interchageably. I was originally going to do `sizeof-1` (compile time vs runtime) but seem to have slipped into `strlen`. Line 283 has to be `strlen`, because the string is dynamically chosen, but I have no preference one way or the other for the rest.

I will change them to be consistent, but the choice of which way I should make them consistent is up to you. If you have no preference, then I will make everything strlen.


we don't insist on it. seemed like a good candidate for a yellow box.


yes, we think this is the first time for that


example of output

pairFastqFiles(path=".",extension="_*",readPrefix="R")
remember, the light is an easy burden... not heavy or cumbersome. remain in the light...
 [1] "./041814-1-N_CGATGT_L006_R1_001.fastq.gz"
 [2] "./041814-1-N_CGATGT_L006_R1_002.fastq.gz"
 [3] "./041814-1-N_CGATGT_L006_R1_003.fastq.gz"
 [4] "./041814-1-N_CGATGT_L006_R1_004.fastq.gz"
 [5] "./041814-1-N_CGATGT_L006_R1_005.fastq.gz"
 [6] "./041814-1-N_CGATGT_L006_R1_006.fastq.gz"
 [7] "./041814-1-N_CGATGT_L006_R1_007.fastq.gz"
 [8] "./041814-1-N_CGATGT_L006_R1_008.fastq.gz"
 [9] "./041814-1-N_CGATGT_L006_R1_009.fastq.gz"
[10] "./041814-1-N_CGATGT_L006_R1_010.fastq.gz"
[11] "./041814-1-N_CGATGT_L006_R2_001.fastq.gz"
[12] "./041814-1-N_CGATGT_L006_R2_002.fastq.gz"
[13] "./041814-1-N_CGATGT_L006_R2_003.fastq.gz"
[14] "./041814-1-N_CGATGT_L006_R2_004.fastq.gz"
[15] "./041814-1-N_CGATGT_L006_R2_005.fastq.gz"
[16] "./041814-1-N_CGATGT_L006_R2_006.fastq.gz"
[17] "./041814-1-N_CGATGT_L006_R2_007.fastq.gz"
[18] "./041814-1-N_CGATGT_L006_R2_008.fastq.gz"
[19] "./041814-1-N_CGATGT_L006_R2_009.fastq.gz"
[20] "./041814-1-N_CGATGT_L006_R2_010.fastq.gz"
>


LGTM


thanks for that infos! Sounds really intresting for me.


Same rule should probably be applied to method arguments.


to be_empty


I didn't fix it, @oschaaf did all the work. I'm only the person who did the pull request.


Could change this to `var xpathmodels = makeXPathModels()` to make it clear that that's the recommended way to do it for anyone reading the source. Not a blocker though.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 859](https://teamcity.labkey.org/viewLog.html?buildId=292328&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 14 (1 new), passed: 215 Build time: 00:17:17
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.DataFinderTest: <no details avaliable>

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

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

org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

org.labkey.test.tests.RedCapTest.testLongitudinal: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.tests.RedCapTest.testSeparateDemographicProject: <no details avaliable>

org.labkey.test.tests.RedCapTest.testVisitBased: <no details avaliable>

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


Some weirdness in the formatting here.


By the way, I would make the sender name and the message of different colors for distinguish them better.


When is it possible to have this exception thrown? What would I have to do to end up with exception?


I would rebase the fix to the indentation with the first commit.


Wait, Phil. Are you 'grsmIRT only possible for unidimensional models' want to say instead 'gpcmIRT'?


> When it does, since $_mpu_irq vector has been changed from default vector

OK, I see, but I have to reject this.
Btw, dosbox also seem to have some hacks in
this area. For example it seems like it looks up
the BLASTER variable and fires the IRQ accordingly
(not for mpu perhaps, I've seen this for DSP).


the sub command ruined the basespace app because if the sample name includes SRR1[anychar]  or SRR2[anychar]  the sub command will change the sampleName!  this must be changed!


I would use `subject` here instead of defining a helper method. The RSpec DSL is quite good and you should be able to achieve most things without having to define helper methods.


fix indent


@seonghobae this is not what you are looking for. It's a different model related to `grsm` for rating scales, not partial credit.


@games647 Awesome! Where do those numbers come from walk & fly speed?


@mikejoseph check out index-base.html as a starting file for projects.


Thank you. Maybe in a second moment, to make fine tuning.
For the moment the direction to go is pretty much clear, so SPSA would take to long and waste to much ressources.


Same as above I would prefer if you used `subject` instead of defining this as a helper.


if the user doesn't like  to search for "R1_*"  , this is flexible to allow for "R1.fastq.gz"  or any of the sort.  one just can't use the sub command.


Ah, that's actually what I wanted, I just didn't know the syntax.


What should the default be? I just wanted it be overwritable via environment variable.


I am not sure why this change happened, but if it is something you intended to do, it should probably be in a separate change from adding the extra specs.


@philchalmers Yes I know this is grsmIRT, but you wrote
```R
stop('gpcmIRT only possible for unidimensional models').
```

This is correct I think.
```R
stop('grsmIRT only possible for unidimensional models').
```


You can pull this out as the `subject`.


How about `>=`, then?


This repo is just a collection of assignments and snippets done in C/C++


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 121](https://teamcity.labkey.org/viewLog.html?buildId=292347&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:30


Yep, good idea.


Same as above, you should be able to declare this and reference as the `subject` for both examples in this spec file.


@seonghobae ugh, I need more coffee today. Thanks


My preference is for consistency based on a given rationale. So, using `sizeof -1` for all the static arrays would be ok. Using strlen for convenience on all types would be ok too. My comment was meant to address treating some static arrays differently from the other static arrays.


A lot of this is because code isn't implemented yet. I'll just remove this from the pull request.


I would consider tweaking your generator settings to exclude helpers, css and coffee and manually add them as needed. You will end up with a lot of empty files in this repo otherwise.


I would prefer if you left a line between `describe` blocks.


OK, good idea.


That'd be fine.


I would pull the `get :show, id: quote.id` into a `subject` so it's clear what we are testing here.


Same as above, would prefer this to be pulled into a `subject`.


You have an extra space between the `)` and the `{`


Extra new line here.


```
Scalar subqueries are rewritten into a join in a following manner.
From:
 `SELECT * FROM nation WHERE regionkey = (
 SELECT regionkey FROM region WHERE name = 'AFRICA'
 )`
into:
 `SELECT * FROM nation, (
 SELECT regionkey FROM region WHERE name = 'AFRICA'
 ) temp
 WHERE nation.regionkey = temp.regionkey`
```

That example is a little misleading, and it doesn't match the comment above it. There's no JOIN in either of those queries. It might be easier to explain in terms of what the plan looks like.


Where the section name in the cross-referenced file is the same as the section in which the reference is made, it can be omitted.  For example, in this case, it is enough to write `@Sancti/02-22`.


IO.binread is not 1.8 compatible, but we do have [File.read_binary](https://github.com/MSP-Greg/yard/blob/master/lib/yard/core_ext/file.rb#L62) for this.


This commit should also update the reference to `01-18t` in `web/www/horas/Latin/Tabulae/K1570.txt`.


reuse of variable name `key` here is hazardous and confusing.


CFRelease doesn't handle NULL - mind wrapping it with a check? (feel free to refactor slightly to avoid calling `SecCertificateCreateWithData` if rawCert is NULL as well. thanks!


This could be pulled into a separate `context`, something like, "with more than one quote`, that way you can create the second quote - `quote_1` in a `let`.

Remember that in times when you need a `let` to be executed right away or without explicitly calling it, you can use `let!`.


Created issue based on this commit.


one man pleased to help. i did here http://vk.com/wall-947906_295792


Does this merge need to be pulled upstream?  That would only be the case if there were some conflicts that were resolved manually.  If it merges cleanly, then the merging will happen when the pull request is merged.


whoa how are you commenting via github....?! such new things. this went
into my spam folder for some reason.

yeah, i agree - I'm going to change the scoreboard to make the colors more
apparent, and i'll consider changing the placement of the result but I
don't know where else it would go since the side or the top would look
awkward as well. also going to add a reset button.

thanks for the feedback!

On Thu, Dec 17, 2015 at 5:48 PM, Faraaz Ismail <notifications@github.com>
wrote:

> What do you think about adding a bit more padding to the bottom? It feels
> like the result text is too close to the bottom of the window? Or maybe
> moving that text somewhere easier to see. Seems a bit counter-intuitive to
> look down to see the result of the game. Maybe you could also change the
> background of the text to the color of the winner?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nreddy216/rockpaperscissors/commit/3ca3d47daf32aa9d4f2f2e09bac63b46a902e43e#commitcomment-15056792>
> .
>


isn't

```python
if os.path.exists(self._rosdep_sources_path):
    shutil.rmtree(self._rosdep_sources_path)
```

clearer and less context heavy?


It looks like the MTVU thread is always active. It can be always cancelled in that case. The exception can disappear for all we care :)


Also, we should make it where you can "Drag" like with left click the size of the orbiting object.


homepage


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9893](https://teamcity.mesosphere.io/viewLog.html?buildId=95697&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 207 Build time: 00:08:13
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Hello,
It's been a while since I worked on this library, so please forgive me if I don't quite understand the issue. From reading #341 and looking at the release history, the library has not had a release since I wrote this code, which would explain the crash. Perhaps @pyricau could give a bit more information here?


pas la peine de préciser la conf ici, il suffit de créer un fichier 'browserlist' à la racine de ton projet ;)


i will asap.


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 860](https://teamcity.labkey.org/viewLog.html?buildId=292350&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 215 Build time: 00:17:08
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cas.SSOwithCASTest: <no details avaliable>

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

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

org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

org.labkey.test.tests.RedCapTest.testLongitudinal: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.tests.RedCapTest.testSeparateDemographicProject: <no details avaliable>

org.labkey.test.tests.RedCapTest.testVisitBased: <no details avaliable>

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


> For consistency with other projects I manage. Nothing critical.

@jdalton Thanks for the answer!

--

Note (in case you didn't know): **in general**, it is better to [have the trailing `newlines`](http://stackoverflow.com/a/729795).


I would prefer not to conflate the rerr field in the logs to contain what would be valid regional geo blocking results. Could you please introduce a new field to store additional data such as radd, such that it's generic and can be used for additional String data by other implementations?

I believe that we'll need to add some code to the HTTPAccessRecordBuilder, HTTPAccessRecord, HTTPAccessEvent and HTTPRouteResult classes in order to introduce the "radd" log field and a property to store the data that's accessible from TrafficRouter.route().


Spatial has an uppercase first letter, while relative hasn't any. This should be fixed, both in the code and in the files.


Default walk/fly speed


It looks like this file was removed, but not added back to the 7.x directory.


I'd change this method to:

* If the input page has 0 positions, throw it away.
* If we're already holding on to a page (with > 0 rows), fail with a SUBQUERY_MULTIPLE_ROWS exception
* Otherwise, hold on to the page


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


The `Category` property is not always defined.  To get a color for each record, the `AQI` property is used to lookup the category using the category range specified from [AirNow](https://docs.airnowapi.org/aq101)


HomePage


Jag tänkte faktiskt på det att det borde hänga kvar på övriga sidor men det har aldrig hänt mig så jag antog att det redan var fixat nånstans i koden. Vad har session med header() o göra?


I'm reverting this in branch 1.6 for now because I'm seeing `IllegalStateException` related to shutdown hooks in tests:
```
[info] - Log overwriting *** FAILED *** (10 milliseconds)
[info]   Expected exception java.io.IOException to be thrown, but java.lang.IllegalStateException was thrown. (EventLoggingListenerSuite.scala:103)
...
[info]   Cause: java.lang.IllegalStateException: Shutdown in progress
[info]   at java.lang.ApplicationShutdownHooks.add(ApplicationShutdownHooks.java:66)
[info]   at java.lang.Runtime.addShutdownHook(Runtime.java:211)
[info]   at org.apache.hadoop.fs.FileSystem$Cache.get(FileSystem.java:1474)
[info]   at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:263)
[info]   at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:124)
[info]   at org.apache.hadoop.fs.FileSystem.get(FileSystem.java:247)
[info]   at org.apache.spark.util.Utils$.getHadoopFileSystem(Utils.scala:1663)
[info]   at org.apache.spark.scheduler.EventLoggingListener.<init>(EventLoggingListener.scala:66)
[info]   at org.apache.spark.scheduler.EventLoggingListener.<init>(EventLoggingListener.scala:59)
```
Don't want to cause any regressions in the release.


ta fonction a un callback, ça ne m'étonnerait pas que ce soit asynchrone, du coup c'est trop tard pour tes headers...
genre regarde là : http://stackoverflow.com/questions/4482686/check-synchronously-if-file-directory-exists-in-node-js


`$PATCH=` must be replace by `PATCH=`


NOT NULL?


Hey Bo, would you mind to send me a patch of the changes you made, so that I can integrate them to FGAddon?
Thanks
Jonathan


why aren't we doing this in `WMFSearchViewController` itself?  maybe out scope, but should we be logging all view did appear/disappear events (e.g. via an extension to ARAnalytics)?


Looks good apart from that.


extract the URI format and using String.format.


Ok, thanks for the catch! :)


similar to my comment above about logging VC views in their `DidAppear` method, won't this omit cases where the user pops back to Explore/Saved/etc.? this also applies to search above


lol, ok.


Por que você mudou o nome das colunas geradas? Em geral, quando a modelagem começa a partir das classes e a empresa não impõe nenhum padrão, o melhor é deixar o default


Why do we need an instance-level `rowsCount`? Either we have a single page with one row (see comment above), or no pages.


I really don't think this is needed. The lines are not longer than the generally used line length, and are perfectly readable (I think more so, considering there's only 2 parameters involved).


Excellent!


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.421.0](http://build01:8888/viewLog.html?buildId=13631&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.423.0](http://build01:8888/viewLog.html?buildId=13632&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.422.0](http://build01:8888/viewLog.html?buildId=13633&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.423.0](http://build01:8888/viewLog.html?buildId=13632&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:07


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.422.0](http://build01:8888/viewLog.html?buildId=13633&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Compilation error: src\Sample.IntegrationTestsWithSettingsThatDoNotHaveConfiguration\Sample.IntegrationTestsWithSettingsThatDoNotHaveConfiguration.csproj Build time: 00:00:13


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.424.0](http://build01:8888/viewLog.html?buildId=13634&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.424.0](http://build01:8888/viewLog.html?buildId=13634&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:04


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.425.0](http://build01:8888/viewLog.html?buildId=13635&buildTypeId=ConfigInjector_CI) is now running


as mensagens você poderia ter deixado no arquivo externo, em um .properties. Inclusive você poderia manter as mensagens padrões e só posicionar do lado dos campos.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.425.0](http://build01:8888/viewLog.html?buildId=13635&buildTypeId=ConfigInjector_CI) outcome was **FAILURE**
Summary: Cannot start build runner; exit code -42 (new) Build time: 00:00:05


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.421.0](http://build01:8888/viewLog.html?buildId=13631&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:42


Closes #38


I think it's a lot better than before


This pattern of doing something after return in a finally block can lead to subtle bugs.


Why do you care for contiguous IDs?  Oh, you're not talking `_ID`, you're talking `position` (or `idx` in this codebase).


Consider future proofing this name.


So, the todo is fixed, right?


oh so glad they did that..it was annoying...and easily could be found from SHMEM


This breaks a few tests because `BoardState`s not created as a result of following actions from the root incorrectly have their turn number set to zero on creation.


I am still doing a lot of errors with EF, you are probably right.
However, I was expecting the error for an `Add` not an `Attach`. From what I remember if you do not change the state of an entity the updated properties will not be saved on database.
See this [SO thread](http://stackoverflow.com/questions/15336248/entity-framework-5-updating-a-record) on the subject.
I will test...


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


hey Robson!


Bravo, great use of names to reveal meaning!


I don't see `rowid` in the schema at this commit.  Perhaps it's a rebase/ordering issue?


ao invés do <select você poderia ter usado a tag do Spring, <form:select. Ele já gera o select certinho para você.


Does this code work for you?  I am also running on Visual Studio 2015 trying to get this to work.  I have the following test program:

```
#include "../../7zip-cpp/7zpp/7zpp.h"

int main()
{
	SevenZip::SevenZipLibrary lib;
	lib.Load();

        // This opens with normal 7-zip
	SevenZip::TString myArchive(L"test.zip");
        // This directory exists
	SevenZip::TString myDest(L"temp");

	SevenZip::SevenZipExtractor extractor(lib, myArchive);
        // Fails below!
	extractor.ExtractArchive(myDest, nullptr);

    return 0;
}
```

I've even tried making my own callbacks, but everything I've tried fails on:

```
bool SevenZipExtractor::ExtractArchive( const CComPtr< IStream >& archiveStream, const TString& destDirectory, ProgressCallback* callback)
{
	CComPtr< IInArchive > archive = GetArchiveReader( m_library, m_format );
	CComPtr< InStreamWrapper > inFile = new InStreamWrapper( archiveStream );
	CComPtr< ArchiveOpenCallback > openCallback = new ArchiveOpenCallback();

        // Fails on the function below!
	HRESULT hr = archive->Open( inFile, 0, openCallback );
```

Am I doing something wrong or is it not working for you too?


avoid using concrete types.


If you do this, decouple it, so that we don't do too much at migration time.


-  I mainly focussed on assigning myself to the issues(#23,#52,#80) that related to the area(course discussion forum).
-  I have created some of the issues #85,#101,#102,#113 and assigned myself.
-  Issues #85,#101 can be considered as a new feature  to the course discussion forum.
   -  Issue #85 mainly focussed on alerting the participators of the system so they can know what they are deleting in the course discussion forum.
   -  Issue #101 focussed on viewing the deleted messages within some period which helps the participators of course discussion forum to know what they have deleted.
-  Issues #102,#113 are releated to other area which mainly concentrated on the activities done by a participator.
-  I took the responsibilities for these two issues that are not relavent to my area because i thought that the owner of the issue should take the responsibility.
-  Finally,I got a clarity that it will be better to focus on one area. I will stick to the course discussion forum.


Esse controller ficou com muitas responsabilidades. O melhor seria você quebrar o controller para deixar só os comportamentos específicos...


Wait, did you add a `while (true)` loop in the Rubberduck code base?! :fearful:

lol, whatever works at that point! :+1:


@bgerstle yep - you got it - the peeking really messes that up. There are radars to add some sort of property to UIViewController so you know when you are being peeked. But until then, yeah it sucks a bit.

Have some thoughts on adding some of this functionality using categories on UIViewController + Associated objects, but didn't wasn't to go down that hole yet.


I really need this so I can sell my 3ds!
Inquiry code is the same, but it's the 18th now.
Please help?


orz，之前都没注意这个问题，学习了


Interesting that this seems to be the one line in the entire function that uses 4 spaces instead of a tab to indent.


Consider making the `if let` bind the `IconType` for safety.


@UniversalMediaServer/developers is this a good time for a release?


Having problems opening Project M on Dolphin since the recent update. When I try starting Project M, I get this error below.
![untitled](https://cloud.githubusercontent.com/assets/16161703/11907405/30797ae2-a599-11e5-9811-f19a68a8391f.png)
The main source is the Kirby Roy and Mewtwo metal texture changes. I would know because I have attempted to add metal textures to the Roy and Mewtwo hats via brawlbox. Please revert the Kirby changes until this issue can be solved.


Not sure how these changes got in here; reverted in 5c2623249132c967a4d2e1f23cb834ea6ae0bb1c.


@SubJunk I have a couple of things I'd like to get in, but I've been slow lately - there's been so many other things.

I'm about 98% finished with the crowdin branch, and I'm working with finishing up the "RendererSearch" branch right now.


Is there an implicit limit here, like `.count < 1000 - 3`?


Getting the server version seems like a waste of time. It's not used for anything.

If the purpose of this is just to check that the connection is OK, I don't really see the point of that either. If the connection details are wrong, surely we'll find out shortly anyway, when we try to connect "for real".


@EisenbergEffect looking simply at the code, and reviewing the knockout changelog, this looks correct. I'll be able to review it early next week.


Soft transition to 4 spaces indentation everywhere... :smile:


The consistency part was that we are using a static import of `assertEquals` in one place, and `Assert.equals` in another place. It'd be better to just choose one. Not a big issue though, and we don't have code standards that mandate this (or anything else).


I think the schema name needs to be quoted here. Or is it quoted already in the mr_query.relations list?


This seems awkward.  What's the longer term vision here?


This doesn't agree with "<Root>" earlier.  Is this intentional?  Is this ever displayed?  How does this fit into the l10n story?


Quoting, escaping


:+1: Death to tabs! :grinning:


Yep, I read this wrong. Will pull this branch instead of getting from gradle since the release is a tad stale.


This surprises me, I expect you to set `is_deleted = 1`.  Do you intend to witness this in the synchronizer?


Good idea! :)


Awesome!  Could you change the line 21 to

@implementation NSDateComponents

and eliminate the : Object?  Looks great otherwise.


Add tests for SimulatorWrapper.


I'd like to keep indentation consistent at 4-spaces, if you don't mind. Thanks for working on this!


Do a \ here. I'd rather we pass duplicate \\ to the Nuget pack command (which won't care) than to chase down bugs the moment OutDir doesn't have a trailing slash.


`DataHandler#getContentType` now returns values like `application/octet-stream; name*=UTF-8''logo2%C3%9C` and `application/octet-stream; name*=Cp1252''logo2%DC`

`Cp1252`  is not equal to Charset.defaltCharset().name() or Charset.defaultCharset().displayname() on a german windows pc, therefore I considered it best to make the change to the test.


Grrrrr.  This was added after I organized them.  We really need this: https://github.com/palantir/gradle-baseline


I don't think E'foo' syntax should be used with pgdb.escape_string() function. Plain 'foo' should do.


This block is really opaque. What does it do?


Renamed directories


After `finish()` is called, this method should return either:
* The currently recorded page with one row
* A page with a single row and a null value

I would restructure the operator in the following way:
* There should be three logical states: accepting input, flushing, done
* `finish()` causes a transition between accepting input -> flushing
* `getOutput()` should return null when in the accepting input state
* `getOutput()` should return a single page when in the flushing state. In this case, it also transitions to the done state
* `getOutput()` should fail if in the done state (to protect against bugs in the caller)

This could be achieved by:

```
boolean finishing;
Page page;

states:
   accepting input ::= finishing == false && page == null
   flushing        ::= finishing == true && page != null
   done            ::= finishing == true && page == null

finish()
    if (page == null) => page = new single-row page with nulls
    finishing = true

getOutput()
    verify not in done state
    if finishing => return page and set page = null  (transition to "done")
    otherwise => return null
```


You produced the machinery for doing this in a transaction, but don't use it.  To come?


Just pointing out that comparing IPv4/6 addresses in their "a.b.c.d" and "1234::abcd" string formats is probably not the best idea anyway. Better to parse them to e.g. a (family, ip_integer) tuples (or some other form that doesn't depend on case and such) and compare those.


Название игры, тест комментария на гите


By me, hat's better to be implemented as plugin. (IMHO)


maybe call this "recordSubqueries"


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


What is the right thing to do from a language point of view. Is there one clear answer (for example "yes, we can ignore")?


The error handling is muddy here, with `err` out of the closure scope and also a closure parameter.  It gets worse, too, with `err = error` later.  Can this be simplified?


Could the suggested sites be inserted around here?


lgtm.


lgtm


Should not rely on error text, those change from release to release. Check SQLSTATE instead.


yup


Throw an exception instead


@pb00068 Hey man, congratulations. Can you confirm that this is good news for cores greater than 24?! Here you used 10-ply long giant skips?


Move this class to the core


Awesome!


My different frame you mean iframes?

On 18 December 2015 at 16:08, ven <notifications@github.com> wrote:

> that's gonna break if it goes through different frames. is that an issue?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/perl6/nqp/commit/14b8a63bf5aebdb1e5d8a4c691821d6dfef510fd#commitcomment-15067544>
> .
>


Change classname to com.ringcentral.android.sdk...


Oops, meant to leave @borisfom as the author for this commit.


Looks good to me. I haven't touched this code base in quite some time so if this is working for you and fixes the issue I'm happy to accept it.


@pb00068 Does this patch help Stockfish to reach deeper depths, or is it more about trying to get all of the helper threads to have variance by means of searching into different parts of the tree? Mind my ignorance... :+1: )


I am working on translate plug-in but I can hardly call image search a plugin. This functionality is built-in in most browsers.


This name is too generic. I'd change it to EnforceSingleRowOperator


why in plugin management section ?


@jhawkzz Here's the wiki on the topic of commit messages: https://github.com/SparkDevNetwork/Rock/wiki/Committing-Your-Code


Is this legal Swift?  `String)(conn` looks odd...


lgtm


Likewise, I'd call this `EnforceSingleRowNode`


we have to have python2 now per https://pulp.plan.io/issues/1383


What version of RSpec are you using?


pep8 specifies import groups and ordering different. Go read that and adjust.


The jstree.dnd_outside.js plugin is nearly identical to the original jstree plugin - because of the options naming scheme and the existence of a built-in dnd plugin, I search-replaced every 'dnd' with 'dnd_outside', except for where it identifies class names for css (we want to use the same ones as the builtin dnd plugin.)  The event handler for 'dnd_outside_stop.vakata.jstree' was modified so that it does not actually manipulate the jstree directly when the move is finished, instead firing a new event 'finish' with all the information required for such a manipulation.  In our program we'll use that information to operate on a remote queue, and when that is successful jstree node-moving events will propagate back to keep the tree in sync with the remote queue.


Didn't realize it wasn't. Will check and change after lunch

> On Dec 18, 2015, at 1:20 PM, Matt Sawyer <notifications@github.com> wrote:
>
> I'd like to keep indentation consistent at 4-spaces, if you don't mind. Thanks for working on this!
>
> ―
> Reply to this email directly or view it on GitHub.
>


What happens in the `INSERT` if no icon is found for the URL?


Parsing the list of relations from the text format dump seems a bit hacky. Since we're adding server code to support this feature anyway, I would suggest adding a server function that directly returns the base relation OIDs. In the server-side, you can do it a lot more robustly.

Come to think of it, perhaps we should just add it directly into the backend, instead of as a separate library in gp_toolkit. I don't see any harm in adding a little backend function, and it would avoid the extra makefile, .so file etc. infrastructure

Hmm. I think you could even do it without any server-side support with a little hack: create a temporary view for the query, and check what tables the view depends on, in pg_depend. It's a bit hacky, so probably better to just add the backend function for that, though.


Not important for this, but we internationalize almost all strings using gettext. There are some good articles that talk about this. Also when using gettext it's best to use dict style string formatting instead of positional formatting.


This is a clean way to do this, nice


why comment instead of remove?


This patch has nothing to do with that test, we should probably disable it, its very fragile.


Reading this code, it seems you should be able to just fill.  Are you being cautious?  Am I missing something?


These could become '\n' values in the string above, but are also fine as-is


```
brew install cmake automake autoconf libtool
```
Exactly the list I came up with.  Just the Command Line Tools from Xcode will suffice, and can be [installed](http://osxdaily.com/2014/02/12/install-command-line-tools-mac-os-x/) with `xcode-select --install`.


Why not use includes http://eclemma.org/jacoco/trunk/doc/prepare-agent-mojo.html#includes ?


Interesting. This looks like it may be what I was missing. Might we run into trouble due to the overloaded meaning of `-1`?


You used a different pattern with a cursor and checking for failure or emptiness earlier.  Is this just style, or is this substance?


@Alymosul  please check the App\Post and workspaces.show as the function comments() in App\Post returns null whenever it is used


This did not appear to fix it for me. Seems to require an actual version like "needs": "^0.8.0"


                                   Section       Old       New  Percent
libswiftCore.dylib                  __text:  2412869   2249541    6.8%

libswiftAVFoundation.dylib          __text:     1144      1130    1.2%
libswiftAppKit.dylib                __text:     7916      7642    3.5%
libswiftCloudKit.dylib              __text:     1144      1130    1.2%
libswiftContacts.dylib              __text:     1144      1130    1.2%
libswiftCoreAudio.dylib             __text:    15853     15576    1.7%
libswiftCoreBluetooth.dylib          __text:     2184      2170    0.6%
libswiftCoreData.dylib              __text:      671       671    0.0%
libswiftCoreGraphics.dylib          __text:    22216     20730    6.7%
libswiftCoreLocation.dylib          __text:     1144      1130    1.2%
libswiftCoreMedia.dylib             __text:     8590      8170    4.9%
libswiftDarwin.dylib                __text:    10039     10055   -0.2%
libswiftDispatch.dylib              __text:      872       872    0.0%
libswiftEventKit.dylib              __text:     1144      1130    1.2%
libswiftFoundation.dylib            __text:   228442    219930    3.7%
libswiftGLKit.dylib                 __text:     1644      1624    1.2%
libswiftGameKit.dylib               __text:     1144      1130    1.2%
libswiftGameplayKit.dylib           __text:      351       351    0.0%
libswiftLocalAuthentication.dylib          __text:     1400      1338    4.4%
libswiftMultipeerConnectivity.dylib          __text:     1144      1130    1.2%
libswiftObjectiveC.dylib            __text:    10941     10826    1.1%
libswiftOpenCL.dylib                __text:      764       764    0.0%
libswiftSceneKit.dylib              __text:     5713      5489    3.9%
libswiftSpriteKit.dylib             __text:    10027      8874   11.5%
libswiftWebKit.dylib                __text:     1144      1130    1.2%
libswiftXCTest.dylib                __text:   101851     97131    4.6%
libswiftsimd.dylib                  __text:   127421    123066    3.4%


For API compatability against Django ORM it's best to use `collection.objects.all()`. Not important here, but a general best practice.


(After a cursory look the answer seems to be no)


The thing being passed in is a model not a collection. s/collection/model/


nice, it's like unittest :+1:


It is an annotation processor which ensures at compile time that marked class subclasses has no-arguments constructor which is needed by registry.


Achei q seria útil,

mas tem problema nao, vou tirar e botar no gitignore.


Also if plugin, it would bi nice if different url can be set _Yandex Images, Yhoo Images_ etc. (just proposing)


ImmutableList is preferred.


Didn't cross verify against jshint docs .. but, travis seems unhappy with this pull request .. so, maybe you've missed some cases or typoed a jshint directive?


an explicit close of this later would be good.


I'm a bit confused what you mean by running "Action Cable servers in their own processes" in production. If you are using Action Cable in production, it will exist as a separate app?


This should have gotten rid of the final quicksim dependence.


At the moment https://dl.google.com/dl/page-speed/psol/1.10.33.2.tar.gz is still returning 404.


Done in 7bd164bc58725f02e6147ab6beb4d966aca4ffcb. Thanks!


question something seems weird

 if (mIncomingInvitationId.equals(invitationId)&&mIncomingInvitationId!=null)

should not be the null check first?

 if (mIncomingInvitationId!=null&&mIncomingInvitationId.equals(invitationId))


Please avoid snapshot dependencies. They are unstable.


I'm not sure I understood your question


`initialize()` looks this up automatically so you don't have to


I'll think about that.


Same app, different process. For Basecamp 3, we run the main app with
Unicorn. Then have a separate config.ru that we run with Puma for the ACa
processes.

On Fri, Dec 18, 2015 at 10:43 PM, Jeremy Cohen Hoffing <
notifications@github.com> wrote:

> I'm a bit confused what you mean by running "Action Cable servers in their
> own processes" in production. If you are using Action Cable in production,
> it will exist as a separate app?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rails/weblog/commit/3a964a06dfd4a5f2a6c19e43952cc457cdd177cd#commitcomment-15075687>
> .
>


You don't need to pass any params here, `initialize()` looks it up on its own.


After rebasing you'll find that this no longer works.  Please see [here](https://github.com/cloudfoundry/cf-java-client/blob/master/cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/DefaultSpaces.java#L38-L58) for an example.


Improve searching of buildables and testables in a scheme


Если вы уже пошли таким путём, то getDataList() в фрагменте категорий будет излишним. Ведь, по-сути, это будет дублированием кода.
Аналогично можно поступить и для трат - метод getAllExpenses() (статический как и тут) и с фрагмента трат убрать getDataList(). Ну и в лоадерах, в обеих случаях, возвращать НазваниеКласса.getAllSomething().


Fixes #29 also


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


It depends. As this is only an optimization we are fine to catch the most common cases.
But as it's mostly @swiftix 's code, I'm redirecting the question to him.


Do not add binary files to KCP, keep the link as was to a needed .zip file, this commit will be reverted.
KCP is for text files to build apps, not a place to store compressed packages.


I have no idea so far, how far we could go with the size of skipt. No idea how they skale at the moment.
I will make some test anyway, when your machine is on fishtest.
For the moment I try to optimize for 20-24 threads.

About depth, I don't know I never measured it.
Jorg Oster telled my that my patch with 3 threads does search deeper in avg 2 plies than current master does.
Anyhow I think, we are widening the search and since bestmoves seem to be found earlier through this strategy, it has in the end also a positive effect on depth. Having bestmove changes near the leaves, means concluding iterative deepening cycles faster, which allow you to go deeper.


@vargaz Jenkins says it's either this or https://github.com/mono/mono/commit/27302ec4515314f55ee960b5575f6041d0ebe9cf (sorry I commented just on one)


Yes, I haven't uploaded that yet, sorry!

1.10.33.2 will be live on monday, and at that point https://developers.google.com/speed/pagespeed/module/build_ngx_pagespeed_from_source will start saying 1.10.33.2 instead of 1.10.33.1.


`@Singular`


Ulf also did a bunch of work to clean up all of the build warnings.  The ISx builds are very shiny now.


`default` visibility


I don't know how closely you're watching CI, but one test run just failed because of `libc++abi.dylib: terminating with uncaught exception of type std::__1::system_error: mutex lock failed: Invalid argument` from this commit: https://ci.realm.io/job/objc_pr/2636/configuration=Release,swift_version=1.2,target=osx/console


Ok thanks !


У вас есть get и set методы. Почему тогда все поля класса public?


To make it robust, I'm going to sort the torsions by their index after the
files of read it so that the order doesn't depend on how the files are
sorted or named.
On Dec 18, 2015 2:09 PM, "John Chodera" <notifications@github.com> wrote:

> Also, how do the rmse and sigma compare now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChayaSt/Torsions/commit/b061dde320f151f51cede443a2d8f732adf8b712#commitcomment-15072835>
> .
>


Good indicator that you've forgotten `@Singular`.  If you had added it, you wouldn't have needed to create a `Collection`.


Added read me


Hah sorry I cross-posted.

Hmmmm that's a tough question - I think it's ok because whenever we talk to the cluster we use `last_offset_consumed + 1` (that is, the cluster always thinks in terms of next offsets, I think?). But it certainly does feel very bug prone, somehow :(


When wrapping method declarations, place each argument in a separate line:

```java
public ScalarNode(
        @JsonProperty("id") PlanNodeId id,
        @JsonProperty("source") PlanNode source)
{
```


s/once/one


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1017](http://teamcity.monogame.net/viewLog.html?buildId=12703&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Nit ']'


SMSG_REALM_SPLIT till used, but it's called by bnet packet, no ?


Excellent numbers Arnold!

Thanks.

> On Dec 18, 2015, at 1:41 PM, Arnold Schwaighofer <notifications@github.com> wrote:
>
>                                Section       Old       New  Percent
> libswiftCore.dylib __text: 2412869 2249541 6.8%
>
> libswiftAVFoundation.dylib __text: 1144 1130 1.2%
> libswiftAppKit.dylib __text: 7916 7642 3.5%
> libswiftCloudKit.dylib __text: 1144 1130 1.2%
> libswiftContacts.dylib __text: 1144 1130 1.2%
> libswiftCoreAudio.dylib __text: 15853 15576 1.7%
> libswiftCoreBluetooth.dylib __text: 2184 2170 0.6%
> libswiftCoreData.dylib __text: 671 671 0.0%
> libswiftCoreGraphics.dylib __text: 22216 20730 6.7%
> libswiftCoreLocation.dylib __text: 1144 1130 1.2%
> libswiftCoreMedia.dylib __text: 8590 8170 4.9%
> libswiftDarwin.dylib __text: 10039 10055 -0.2%
> libswiftDispatch.dylib __text: 872 872 0.0%
> libswiftEventKit.dylib __text: 1144 1130 1.2%
> libswiftFoundation.dylib __text: 228442 219930 3.7%
> libswiftGLKit.dylib __text: 1644 1624 1.2%
> libswiftGameKit.dylib __text: 1144 1130 1.2%
> libswiftGameplayKit.dylib __text: 351 351 0.0%
> libswiftLocalAuthentication.dylib __text: 1400 1338 4.4%
> libswiftMultipeerConnectivity.dylib __text: 1144 1130 1.2%
> libswiftObjectiveC.dylib __text: 10941 10826 1.1%
> libswiftOpenCL.dylib __text: 764 764 0.0%
> libswiftSceneKit.dylib __text: 5713 5489 3.9%
> libswiftSpriteKit.dylib __text: 10027 8874 11.5%
> libswiftWebKit.dylib __text: 1144 1130 1.2%
> libswiftXCTest.dylib __text: 101851 97131 4.6%
> libswiftsimd.dylib __text: 127421 123066 3.4%
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/059bb7c6598adea81234a912805bd0b2d43a34a7#commitcomment-15075638>.
>


vse horosho


Fundamentally this pattern of synchronously `get`ting results is both non-idiomatic and poor performing.  The correct design is to leave everything as `Stream`s and allow the reactive framework decide when to dispatch, in parallel and non-blocking if possible.  Please see [usages of the `zip` operator](https://github.com/cloudfoundry/cf-java-client/blob/master/cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/DefaultRoutes.java#L61).


Nit (here and below).

You could use the array's length to decide if empty and then you can use the Joiner
with [appendTo]( http://guava-libraries.googlecode.com/svn-history/r14/trunk/javadoc/com/google/common/base/Joiner.html#appendTo(java.lang.StringBuilder, java.lang.Object[])) that you can pass to the `usage` function.


So these additions are a bit problematic.  It now means that [`list()` returns a partial set of data](https://github.com/cloudfoundry/cf-java-client/blob/master/cloudfoundry-operations/src/main/java/org/cloudfoundry/operations/DefaultSpaces.java#L63-L66).  Is this what we want?  Should anything that returns a `Space` return the same amount of data populating that `Space`?


See my comment above (nit) regarding passing the StrinbBuilder to usage.


Ah, okay then.


I believe that by convention, we've called this `get()`.  Also, this should be a `*Request` object rather than arguments.


Nit, StringBuilder


Just saw your comment later /and/ I remembered why I did this (I wanted to have the analyses verify each other using -sil-verify-all /before/ the loop canonicalized ran). That way we know that the analyses were correct before canonicalizing and are correct afterwards as well.

> On Dec 18, 2015, at 11:18 AM, Nadav Rotem <notifications@github.com> wrote:
>
> Michael, why do you need these passes? Where are you using them? It looks like these passes simply call the analysis.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/810181c6d1ee2ad9ac0804f1032c644b1351e20a#commitcomment-15070560>.
>


Ok. So I've started implementing a user list feature here.


oops


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1017](http://teamcity.monogame.net/viewLog.html?buildId=12703&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:40


"I am altering the deal. Pray I don't alter it any further."


Okay. Could you please insert a comment here explaining that this is to circumvent a bug in tvheadend? Addidionally, a bug fix in tvheadend would be great.


There should be a scenario for each character - I think I only see Brad?

I.e. The *SCENARIO* tag determines the "starting scenarios" by interacting with a particular entity - and shouldn't have to jump directly to a question scene, or jump to a scene that doesn't involve (in this case Brad) character.

SeiStory and BradStory files in util/StoryParser are some pretty good examples.


Leave a Comment


That's funny. I asked the eyeglass developers, and they told me it would work. :p Feel free to re-open the original issue/pr.


This is what a comment on a line of code looks like.  Nothing else useful to say here - beautiful sentence structure.  :)


I think `0007` is a reasonable compromise.  That should prevent us from breaking lots of systems when this is merged.  I'm also okay with the zfsdev_umask output, there's not much we can do about it and frankly most people shouldn't be touching this in the first place.


I think this is good, but I looked at the code, and we never actually set that flag for windows. Probably another bug we need to fix.


`true` and `1` don't need to be surrounded by `()`


I don't know where to comment this but there's a glitch with knuckles where I side B with knuckles on the right and I get stuck on the left side of the stage with no hitbox and I can't escape. I can't replicate this glitch. I don't know why it happened


sounds good. thank you.

> On Dec 18, 2015, at 1:59 PM, Michael Gottesman <notifications@github.com> wrote:
>
> Just saw your comment later /and/ I remembered why I did this (I wanted to have the analyses verify each other using -sil-verify-all /before/ the loop canonicalized ran). That way we know that the analyses were correct before canonicalizing and are correct afterwards as well.
>
>
> > On Dec 18, 2015, at 11:18 AM, Nadav Rotem <notifications@github.com> wrote:
> >
> > Michael, why do you need these passes? Where are you using them? It looks like these passes simply call the analysis.
> >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/810181c6d1ee2ad9ac0804f1032c644b1351e20a#commitcomment-15070560>.
> >
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/810181c6d1ee2ad9ac0804f1032c644b1351e20a#commitcomment-15075932>.
>


When you refresh this please also make sure you add a section to the `zfs-module-parameters.5` man page.


In this case I would status instead of stat (that also translate to 'statistics') and remove the extra space between 'of' and '#' (if github isn't fooling me).


Hey @wsargent , I've opened an [issue](https://github.com/playframework/play-ebean/issues/67) regarding the ebean situation, and I saw this commit. Any chance of having a release that contains this update( or a new one that updates to Ebean 6.13?).

Thanks


Are you sure ? callerid can be different. If you don't use firstname and lastname please remove it from the list, because they coming from dao and i don't see the merge in dao.


kkk


The `val <= nmsg_res_errno` part doesn't work (for me), because we'd always have to update the implementation of `nmsg_res_lookup()` whenever we add a new `nmsg_res_*` value.


This patch LGTM.


lol gdb protocol.


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


Wow :D I have difficulties with formspecs...
Thanks for this very fast good work !!


@see will break this text block in a odd way. if you want just a link to 'sol_network_link_addr', try @ref


A release for Christmas :-)


This patch makes me cringe, but LGTM still :D


Please do not use `default_scope`.  It is too likely to have unexpected effects.  Also, please put any changes in `VersionConcern`, not `Version`.


Leave a comment


Alternatives:
* Use empty/null value for last label if not provided.
* Change usage to label:value (missing ":value" suffix would be considered as a null value).


@rdvincent  I re-added niftilib to be able to build conversion tools, when niftilib is disabled in libminc. Maybe another solution should be possible?


Comment on 0095e27


Yeah, was also thinking about implementing some touch UI features. Will fix when I get a chance


Do you want a `LIMIT 1` here?  Or `SELECT DISTINCT 1`?


Bad indent


This seems like a broken toolchain. Are you sure C++11 threading support is even enabled?


We need @brief in the group's text as well. So a 'brief' is shown in the list of modules. Applies for both groups


Sorry, maybe we should prefix "Going to delete project \"" + X + "\". Are you sure [y/N]: "


FWIW you could use `Array.wrap` from Activesupport (we already require pieces of ActiveSupport; it's just one more minor piece)

```ruby
require 'active_support/core_ext/array'
Array.wrap({a: 'b'}) # => [{:a=>"b"}]
```

@jonhkr


i'll make the change on https://github.com/RipTheJacker/survey-gizmo-ruby/pull/45


@Tolo0 @PEMapModder is already working over it ;) You can checkout the progress at #3765


Rename to `copyTranslations`


This also seems to have added my empty scheduling activity. Idk how that happened lol


Hey I think you'll need this
A much better solution

https://github.com/ResurrectionRemix/android_frameworks_base/commit/2078be24eb76af22877b0cefe4d1ffa3f1f6d83d


Nit s/args[0]/projectId/


Closes #43.


Actually, only 'Network' will make use of a @brief


oops, the schema should be quoted here.


@Awsoman62 that would go into an issue thread (a new one you'd make in this case) and not here.


No spaces === ugly CSS :p


Your assumption is correct.  We're testing InnerObjectFormats using a SigningAccount.  So we need to make a good environment for multisigning.  Line 97 looks pointless.  Sorry about that.


This fixes the bug...
![bad_idea_machine_lolcat](https://cloud.githubusercontent.com/assets/156575/11908552/1060818a-a5ab-11e5-8cbb-494665e11bc3.jpg)


There is obviously something wrong with all those `isinstance` checks and if you have any idea how to get rid of them or at least improve this code, I'm listening ;-)


Put all arguments in a single line.


s/@see/@ref/


We should find a way to get access to the exception here and just return `exc.detail` instead of doing all this.

I thought about putting the exception directly in `self._errors`, I don't know how you feel about that ?


Can you add a comment explaining why this is being treated differently?


I had to do this to make the browse api tests pass


You are right. I changed the comment to ```# get server version, which is dumped to minirepro output file```. In fact, the get_version function was used to retrieve version info, instead of simply checking db connection.


They don't do anything yet though...


Ooh, dangerous territory.

I'd be concerned about a client storing records from the deep past and far future next to a reasonable timestamp; seems dangerous to introduce a non-local clock to local changes.


```javascript
 describe('A', function() {
   console.log('A');
   it('B', function() {
     console.log('B');
     describe('C', function() {
       console.log('C', this);
       it('D', function() {
         console.log('D');
       });
     });
   })
 });
```

Results in
```bash
> jasmine

A
Started
B
C
.

1 spec, 0 failures
```

So, where's D?


But how to override something like `PaperTrail::Version.where(:item_id => @dog.id)` otherwise?

Initial problem was in failing test

```ruby
      dog_versions = PaperTrail::Version.where(:item_id => @dog.id)
      assert_equal 4, dog_versions.count
      assert_nil dog_versions.first.reify
      dog_versions[1..-1].each { |v| assert_equal 'Dog', v.reify.class.name }
```


The ```E'foo'``` systax was not quoted. see code ``` val = "E'%s'" % E(val) ```. The plain 'foo' was actually quoted as you mentioned.


I know you're not submitting a PR, but we don't want to check in this user-activity.js file -- this likely means your .gitignore is out of date and needs to be rebased.


This wont trigger a brief to be displayed, since the enum is documented, doxygen already creates a link to it next to flags (and addrs). Its better to have a short text (even a silly one) than this reference. Something like "Link Flags" and "List of Network Addresses".


sorry m8, accidentally moved your changes here when trying to combine some of my small commits


Can you elaborate on `preserveDeletions`?  This is "preserve deletions that have not been synced upstream yet", right?


I don't know that service_name and default_provider arguments are necessary here. You already check for options[:provider]. Why not use the options hash to check for options[:service_name]. So something like this:

```
@service_name = options.fetch(:service_name) # Mandatory
@provider = options[:provider] || 'Microsoft.Compute'
```

It does mean a bit more typing in the super calls for the subclasses, but I think it will be clearer, too.


Add a note indicating this is a cross join. Unfortunately, the way we encode cross joins is not very obvious.


An 8-ELO patch? Incredible! Good job @pb00068 !


A transaction begins at https://github.com/robconery/moebius/commit/6cad0557fa5add085ab3e54d4c4c758588a94e7a#diff-77df777038e3084880cf45989e676b86R217 (and still does on master). If a failure occurs inside `execute/2` the transaction is rolled back and the error is raised as a runtime error. The rollback makes sense because otherwise you are in a failed transaction and any queries will error. The simplest fix for this case is to begin a fresh transaction before running create_document_table so that create table/index is run atomically and use the commit at https://github.com/robconery/moebius/commit/6cad0557fa5add085ab3e54d4c4c758588a94e7a#diff-77df777038e3084880cf45989e676b86R219 to end the second transaction (if a second one occurs).

Perhaps to explain better here is the output of one of the tests when tracing `Postgrex.Connection.query/3` calls:
```
$ mix test test/moebius/document_test.exs:24
Including tags: [line: "24"]
Excluding tags: [:test]

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.202.0>, "drop table if exists artists;", []]
    (moebius) Moebius.Runner.execute/1

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "BEGIN;", []]
    (moebius) Moebius.Runner.open_transaction/0

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into artists(body)\nVALUES('{\"name\":\"Spiff\"}')\nRETURNING id, body::text;\n", ["{\"name\":\"Spiff\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "ROLLBACK", []]
    (moebius) Moebius.Runner.execute/2

<<<<Should BEGIN here?>>>>

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create table artists(\n  id serial primary key not null,\n  body jsonb not null,\n  search tsvector,\n  created_at timestamptz not null default now(),\n  updated_at timestamptz\n);\n", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create index idx_artists_search on artists using GIN(search);", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create index idx_artists on artists using GIN(body jsonb_path_ops);", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into artists(body)\nVALUES('{\"name\":\"Spiff\"}')\nRETURNING id, body::text;\n", ["{\"name\":\"Spiff\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "COMMIT;", []]
    (moebius) Moebius.Runner.commit_and_close_transaction/1
```

However that is not always enough because if we run the test "executes a transaction" with the following line added at the start of the test
```elixir
"drop table if exists user_docs" |> Moebius.Query.run
```

```
$ mix test test/moebius/document_test.exs:185
Including tags: [line: "185"]
Excluding tags: [:test]

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.202.0>, "drop table if exists user_docs", []]
    (moebius) Moebius.Runner.execute/1

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "BEGIN;", []]
    (moebius) Moebius.Runner.open_transaction/0

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into monkies(body)\nVALUES('{\"company\":\"Microsoft\",\"name\":\"Peaches\"}')\nRETURNING id, body::text;\n", ["{\"company\":\"Microsoft\",\"name\":\"Peaches\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into cars(body)\nVALUES('{\"name\":\"Toyota\"}')\nRETURNING id, body::text;\n", ["{\"name\":\"Toyota\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into user_docs(body)\nVALUES('{\"name\":\"bubbles\"}')\nRETURNING id, body::text;\n", ["{\"name\":\"bubbles\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "ROLLBACK", []]
    (moebius) Moebius.Runner.execute/2

<<<Just rolled back the inserts! Probably want to roll back to a save point set immediately before the failed insert>>>

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create table user_docs(\n  id serial primary key not null,\n  body jsonb not null,\n  search tsvector,\n  created_at timestamptz not null default now(),\n  updated_at timestamptz\n);\n", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create index idx_user_docs_search on user_docs using GIN(search);", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "create index idx_user_docs on user_docs using GIN(body jsonb_path_ops);", []]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "insert into user_docs(body)\nVALUES('{\"name\":\"bubbles\"}')\nRETURNING id, body::text;\n", ["{\"name\":\"bubbles\"}"]]
    (moebius) Moebius.Runner.execute/2

** (Dbg) #PID<0.195.0> calls Postgrex.Connection.query/3 with arguments:
    [#PID<0.203.0>, "COMMIT;", []]
    (moebius) Moebius.Runner.commit_and_close_transaction/1
```


Only a suggestion: my standalone tool https://github.com/igneus/pslm offers the same functionality as psalmpreprocessor.rb and is (or is only believed to be? :) ) more mature.

pslm is already used to point psalms for my late booklets built using hiram (like Exaltation of the Cross and Officium parvum).


echo ("mu change");


This could be a `UNION ALL`, right?


In this case, since a link for sol_vector will be created, its better to have a full doxygen block above addrs with something link
```
/**
 * @brief <something>
 * @see <struct>
*/
```


Ah, I see why you're separating.  Ta.


is the `at_least(1).times` redundant? isn't that default behavior for expect?


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


@c buf


@c true and @c false


can haz assertion?


thanks @andkov  -

I've added Raven and Codingtask to your .xls file highlighted in yellow.

I don't have enough information about "global" and "fluid" to categorize them because I don't know what measures these are. My guesses, though, are that "Global" belongs in its own category and "fluid" under Visuospatial reasoning.


Consider asserting this.  Might be nice to see our failing assumptions in the wild.


this comment is a bit out of date now -- it's either a transaction or truncation


aa


test-line


I'd maybe move the Java class to a different file


Closes issue #35


File not found - it's not in this location.


echo("my change");


Naw, concating isn't likely for html.


This is embarassing since I'm the reason maxCount is here (its in the original) - but maybe add a comment about what its for since at first glance I can't recall


s/@c/@ref/ to create a link


@c 1 and @c 0


going to need to wipe contents of web/uploads/files and web/uploads/images to ensure changes take effect


Maybe something like SQLContextProvider or something instead for the name of this trait?


s/@c/@ref/


This function has become really large and seems like it could be implemented by a visitor that composes in an escape analysis. I.e.:

class InstructionAnalysis : public SILInstructionVisitor {
  EscapeAnalysis &EA.
}


Why add the initialized variable if we are always just setting it to _sc != null in runBefore()?


s/@see/@ref/


nope.


what are you doing @nortikin  ?? I made a duplicate node for you (viewer_curves_2d.py you can make all changes you want to that), but please leav viewer_curves.py as it is.


ditto


@pb00068 Interesting stuff; thank you for your quick response. I will be installing Fishtest tonight on my rig. It will be interesting to see if you can get ELO even with 28 cores. You should be able to gain something since 28 cores is a lot. There should be advantages with even more threads available.

Take care,


i like tabs, what about you ?


The incantation is ``--enable-threads=posix``. https://projects.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/mingw-w64-gcc#n67


I'm quite confused here.  `getSQLToOverrideParent` accepts a folder GUID; but I see no guarantee that `guid` is a folder at all.

I was led to this by trying to understand how the siblings of a child removed from the mirror get re-indexed, and I still can't answer that question.


should we force this as part of the `setSaveableTitle:` method? i.e. so you can't set the saveable title w/o a source?


Because, I'm checking its value at the overriden runBefore method at JavaDataFrameSuiteBase


e pq não deveria ser versionado?


Why do we support this?  A user can bookmark the same URL multiple times.  Is there ever a UX that we want to support that removes all those bookmarks in a single shot?


Does this work in firefox? Should it be`...: 0 0`; (Also for consistency)


Did some progress on #189, not closing yet as I still have to test it thoroughly.


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


Explanations are good, you can merge without a v2 after fix the formatting issues. Thank you very much!


Hmm.  Interesting, this answers my previous question about multi-delete.


Can't we have a property type that be configured to be anything you want? any function you want?

For example

{type:path}

path = function(){
   return __dir+__file
}


I set the priotity to -100 because even w/o the non functional change/bug pointed out above it was still otherwise a functional patch due to the other changes.  Since the fixed version passed though I have now stopped it.


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


fixed (https://github.com/ajacksified/reddit-mobile/commit/40e6d908f21a6f0aeae0c04ad41add0ed44ffbf0)


Have you tried to put `-scheme Taylor` instead?
There are 2 testing targets: unit tests and integration tests.
`TaylorFramework ` triggers just the unit tests, while `Taylor` triggers them both.


Let me try on my fork to confirm


Thumbs up. We're ready to release.


ah makes sense.


Well, I don't know if this is actually needed. The thing here is that you have to have literal constant as type provider constructor type parameter and the path should work on both .Net and Mono:

    type MonoProvider = TypeProvider<"some/path/here">
    type WinProvider = TypeProvider<@"some\path\here">

...so will this provider to be able to parse just Unix style path in both environments?
Now this seems to work, but if this is really needed, then maybe line 185 should have same kind of parsing?


is there an edge case here where the list starts on the 2nd, and so we miss a log of the "first time the section appears"? or, if the list starts somewhere in the middle, and the user scrolls up, resulting in them seeing the last item in the section first.  this is a bit hard to describe in a comment, hit me up if it's unclear.


Consider getting a stamp on this UI stuff from someone else.


What's this for? the connection object isn't saved in an instance variable and then we connect in the `connect` method. I'm not sure whether it's better to have one connection per transaction or to keep a persistent connection. It's unclear which you've chosen though.


so you're thinking it's only disabled (or unspecified) in the "Release" config? i guess we don't need to override it on the command line


Hey,

Stop showing off!


Actually I wrote another branch exactly the same with your idea that let
the server function return the relation OIDs. But considering that dumping
the query object is a well defined functionality and by preserving the
intact query it enables a lot of possibilities that can be handled by
external tools.

I agree that adding it as a backend function would avoid the extra .so and
such. The reason we took that approach is that we want to back port it to
4.3 branch to support enterprise customers. By putting it in an so file,
enterprise customer can start benefiting from this tool without a catalog
upgrade. They only need to load the UDF after they do a minor version
upgrade (i.e. binary swap).

Thanks~
Haisheng Yuan

On Fri, Dec 18, 2015 at 1:36 PM, Heikki Linnakangas <
notifications@github.com> wrote:

> Parsing the list of relations from the text format dump seems a bit hacky.
> Since we're adding server code to support this feature anyway, I would
> suggest adding a server function that directly returns the base relation
> OIDs. In the server-side, you can do it a lot more robustly.
>
> Come to think of it, perhaps we should just add it directly into the
> backend, instead of as a separate library in gp_toolkit. I don't see any
> harm in adding a little backend function, and it would avoid the extra
> makefile, .so file etc. infrastructure
>
> Hmm. I think you could even do it without any server-side support with a
> little hack: create a temporary view for the query, and check what tables
> the view depends on, in pg_depend. It's a bit hacky, so probably better to
> just add the backend function for that, though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hsyuan/gpdb/commit/89fb4fe8e77d801d6620b6f1faa1e9f73e943556#commitcomment-15075573>
> .
>


We should remove this FEATURE_APPX define I don't think it is needed, people will only include this file if they want this PInvoke.


I commented instead of removed where it seemed to me that the code was part of an incomplete feature (or incomplete removal of a feature) that might want to be completed at some point in the future.  A lot of domino was originally from other projects and then hacked into shape, so it has some old hooks into other functionality.  I didn't want to remove them outright, just in case we wanted to put them back some day.

(Here, the script fetching part of domino was never completed, and probably will never be completed.  But I wanted to leave in place the documentation saying where the script to execute would be found.  Below, there is reference to a StringReader which was originally exported from the CSS parser, but it currently unused -- but maybe we'd want to export it again someday.  The `nonascii` regexp looks like it was originally part of a `isNonAscii` predicate, but I'd have to look at the CSS spec to figure out where it was originally supposed to be used; I didn't want to complete remove it yet.)


what's your reasoning behind using `#ifdef` over `#if`?  I usually use `#if` and let it implicitly fail if it's not defined. Otherwise, it can always be defined, or be extra explicit with `#if defined(PIWIK_ENABLED) && PIWIK_ENABLED == 1`


yay for nullability annotations!


No - in fact we don't want a source if its an articleVC. (see comment on the analytics category)


Use the $(CommonPath) and add the link. See https://github.com/dotnet/corefx/blob/master/src/System.IO.FileSystem/src/System.IO.FileSystem.csproj#L78 as an example.


do we want "committed" to always log "open"?


Resolves 02b256d4ee2f26d492ad46ed9dc53aa8d8d3d617


Looks like you already remove this in the this commit so you can ignore my comment from the earlier commit.


this is undone in a future commit…


Ok.


you could use a context manager ti handle the connecting and closing. Besides being fancy this will ensure that your connection is closed even if there's an uncaught exception before you explicitly close the connection.


lol


Update to use CommonPath and links as well.


The cell variable depends on neutron.get_position_vector() and neutron.get_direction_vector(), so if I try to initialize the neutron after getting the cell I get and error: "local variable 'neutron' referenced before assignment".

Should I calculate the neutron's direction vector in monte_carlo so I can find the cell before initializing the neutron?


IMO that means the article container should pass a special object as the source, not that the source should be omitted.


this phpdoc is invalid too


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


explicit > implicit, better to say exactly what the source is than infer it when missing


the type is required, and is not optional, even if it's just filled with `mixed` as a placeholder


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


Yeah we want logs for everything - we can segment out different versions/configs/OSs/devices/etc in the dashboard.

I can't think of a reason we would need command line control of this.


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


include the parameter name here too, `and_close=True` to improve readability. Someone skimming your code won't know what this parameter refers to. "query" and "data" are self explanatory (or can be understood by looking at the previous two lines), a literal boolean is pretty opaque though.


I made this fix in the develop branch.


Hello, have you seen [this](https://github.com/fperrad/tvmjit)? You may like it. :)


I believe this is a typo. In its current state it evals to:
return (x == -0.5f) ? 1.0f : 0.0f;

I think it should be:
return (x >= -0.5f && x <= 0.5f) ? 1.0f : 0.0f;


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


?? really need it?


Oops, that was supposed to have a close function there too. It's only there to ensure the database exists and can be connected to before we create a class instance to use it.


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


we need revisit it after move it to Linux/Host side.
for v4l2codec, va_drm backend should be enough.
the remaining question is: could the driver recognize ANativeWindowBuffer structure without va_android backend.


repush pls if still nessecary


This doesn't seem to quite match https://dom.spec.whatwg.org/#validate-and-extract (in the XML case) which is the controlled spec I think?


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


Actually the article VC no longer needs to conform to WMFAnalyticsLogging (I'm passing the title for logging page views)

So it should be good now. I'm still leaving the comment as to make sure no one ever tries to re-add the article VC as a source.


Might be worth citing https://dom.spec.whatwg.org/#concept-element-attributes-get-by-name here, since the controlling spec is not immediately obvious. (Or wasn't to me, at least.)


ref #393


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


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


variant?


The implementation below seems to match the spec better, fwiw.


isSaddled?


when we move to Linux/Host, expect it works without android run-time env, but minimal libs.


The name of this variable is not accurate. The expression is equivalent to `type != INNER or symbols is empty` and it's the condition we check to see if this is a cross join. We used to have an explicit check for type == CROSS, but it seems to be gone. We'll have to clean that up at some point.

So, rename the variable to `isCrossJoin`


Why can't I find one by apt search？

- 发送自我的Sony Xperia™智能手机

---- Mingcong Bai编写 ----

I am removing your package, you should check if there is a duplicate before you do anything.

---
Reply to this email directly or view it on GitHub:
https://github.com/AOSC-Dev/aosc-os-abbs/commit/a1221235607801ad8d92012eed7af78244bff0bf#commitcomment-15070651


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


Not much - I just do what I see Apple/other devs do for NDEBUG or DEBUG - devs are just expected to check existence just check existence, not the value. apple never sets `NDEBUG=0` or `DEBUG=0`


result:
https://paiza.jp/poh/ando/share/a8edf2ce


@DanielZendejas @adambard


no longer needed articleVC no longer conforms.


when we talk about v4l2codec, these functionality should be wrapped inside lib, not in player.


Cool, thank you for your response.


probably will just be `saddled()`


This looks weird D:


Maybe make an explicit statement about how this is deliberately more general than https://dom.spec.whatwg.org/#dom-document-createelement (which references http://www.w3.org/TR/xml/#NT-Name).
Also:
1. I think attribute parsing rules are actually more strict than tag parsing rules.  You can't have "=" in an attribute name, for instance.  You might need two different validity checks here if you following the parsing spec.
2. OTOH, the form of the NameStart/etc productions expressed in http://www.w3.org/TR/xml/#NT-Name are not too evil -- perhaps we can actually just desugar that to a slightly evil regexp and implement the spec exactly.  (I just wanted to avoid having to load a unicode database and query character properties, etc, but it looks like the spec just hard codes that actual valid ranges, it doesn't make us track the unicode categories as new unicode versions are released.)
3. `setAttribute` actually uses the same `Name` production as `createElement`, so that's a plus to standardizing on the actual spec (https://dom.spec.whatwg.org/#dom-element-setattribute).


These changes are probably not necessarily if we follow the actual DOM spec?


Would that also work when comparing, for example, a `StringValue` with a `DoubleValue`? This code ended up this way because the data is heterogeneous. But if its possible to use more standard mechanisms to do such comparisons, then that would be preferable.


I think that to a casual reader the comment here/name of canPointToSameMemory is misleading.

Specifically, what EscapeAnalysis:: canPointToSameMemory() does is just compare the two values and see if one is local and the other escapes. In such a case, they can not point to the same memory. That is exactly what the ripped out code here does, except that the new implementation is more powerful and general.

On the other hand, the general name canPointToSameMemory and the comment do not make it clear to the reader what is actually being computed here. It could be something completely different potentially for all the reader nows.

@eeckstein Can you either make the name of the function more specific to what it is trying to do or add in a more specific comment here about what canPointToSameMemory does.


Sorry but is not possible to use "command" in transactions because SQL is not supported in  transactions on the binary protocol. So, only CRUD  methods are allowed in a transaction. All other methods will be simply ignored.


@nadavrot This looks ok to me. My only comment is that the way EscapeAnalysis::canPointToSameMemory is being used does not make it superficially clear what it is actually doing. One has to read the actual source of the function which creates burden on the reader to go from the use-site to the definition-site.

I asked Erik to put in a fat comment or change the name to make it clear what is actually going on here.


We love tiger tail


@nairdo Ignore my last email! I didn't see this context so I wasn't sure at all what you were referring to, lol! Thanks man.


won't this need to still reload the systemd units?


Sorry, I don't get this. Newly introduced m_seeking is set here and there but the value is never checked or used somewhere else?


the source code of libyami_codecparser.so is from external at the moment.
do we really want to build it into libyami.so?


ready for today


More important: Fixes function expression vars not being registered correctly.

Actually meant: fixes the fact that hovering over a variable in the output area didn't highlight the declaration of that variable inside the ace editor/


i changed


If that's possible that would be awesome. I wasn't able to parameterise `Value` because in one `Matrix` there will be different types of `T` (string, double, etc.). But if you know of way to make that work, I'd love to see it. If you've the time, could you maybe experiment and see what kind of changes would be required?


don't mess the ios stuff. It can conflict with xcode. Focus on web


Is the regression test scheduled w/ the wrong book?


Good catch! Corrected...

On Fri, Dec 18, 2015 at 11:13 PM, mstembera <notifications@github.com>
wrote:

> Is the regression test scheduled w/ the wrong book?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zamar/Stockfish/commit/38adb487ca37f276674d04e92dd3213f4d47cf72#commitcomment-15076989>
> .
>


Could be, but could also be different execution contexts (i.e. Plugins etc)


Also after reading the code in the context of the larger file, this breaks the flow of the code. The next comment is:

// Ok, either O1, O2 are the same or we could not prove anything based off of
// their inequality. Now we climb up use-def chains and attempt to do tricks
// based off of GEPs.

Checking the escape analysis needs to go after this comment and the comment needs to be reformulated. Specifically something like:

// Ok, either O1, O2 are the same or we could not prove anything based off of their inequality.
// Now attempt to prove that one of the original values are non-local and the other is local using
// EscapeAnalysis. If one is local and the other is not, they can not alias.
//
// *NOTE* The reason why EscapeAnalysis takes in the original values and not the underlying values is that...
if (!EA->canPointToSameMemory(V1, V2)) {
   ...
}

// Escape analysis has failed, now climb up use-def chains and attempt to use GEPs to disambiguate the values.


@ebeshero could you please take a look @ this Javascript to see if this makes sense? Also, how do you add a second attribute task? For example, this highlights the @who=nellNelson/workingGirl attributes, but what if I also wanted to highlight the @ana =male/female attributes as well?


Sorry, I wasn't very clear, let me try again.

You're basically constructing a string like this:

```
"E'%s'" % conn.escape_string(val)
```

I believe that's slightly wrong. escape_string() is designed to escape the input so that it can be put inside a normal, single-quoted string literal. So you should do this instead:

```
"'%s'" % conn.escape_string(val)
```

The difference is significant if the input contains a backslash, and you have turned standards_conforming_strings on ('on' is the default on more recent PostgreSQL versions). With the E'<value>' syntax, any backslashes in the value should be escaped, but in the plain '<value>' syntax, they should not be escaped. escape_strings assumes that the resulting string will be put inside '', not E''.


Hi Bumxu,

I'm currently using Chromium, could that be the issue?

Thanks,

lowend


In that case, it would be more straightforward to fetch the server version after establishing the connection, and doing it through the same connection we use for other stuff, instead of invoking psql.


No, the deferred line still should be there, but it has a bug. I still want to cleanup all the images in case something fails. So it should be `defer func() {osArchTestRemoveImages(tests)}()` (modulo `gofmt`).

http://play.golang.org/p/WsQ_Gl0OeZ


Fixed!


use `exit` everywhere instead of `die`. `die` is only a alias of `exit`


This http://tests.stockfishchess.org/tests/view/56748d5c0ebc592d552a4254
shows no difference compared to master.


What about when i==idx, it seems like you don't actually check it.  So this func only tells you if a previous dnode consumed this slot.


need this for oppo keyhandler


could you pass it (dmu_buf_impl_t*, int idx)?


well this should pass quite quickly... :)


When you go back and fix this, can you also rename isLetPointer to not use the term "pointer" maybe isLetPointer to something more swifty. Maybe isLetBinding?


this breaks stuff, it says some lang strings are missing and they are not...


Awesome! Merging :)


\o/


Help example doesn't match the command


looks good!


Does it really?


Remove emtpy file services/qaDashboardService.js