@gabrielmansour It can be used up to version 2.0.1 of the driver (where you will get warnings all over the place). Higher versions of the driver are not supported, as the splat style arguments have been removed completely from the driver.


I mean replace the while loop by this:

```
view_i = destroy_handler.view_i
while _r is not None:
                r = _r
                _r = view_i.get(r)
```

This is faster as the destroy_handler.view_i look up is done only once.


> I'd say that it's quite beneficial and allows to gradually extract views and be fully functional :)

Yep I'll try to say it orally


これ自分の環境だとまだずれます。

いろいろ変えてみたんですがインデント治らないですね


similar to 9c12f002821f9567d5454e2ce3b78076dabffb54 : why did this not show up in tests before???


bbbb


I agree.. I just couldn't find the better place to locate this script. Would it be better to have independent repository to collect all converters?


I'll talk about that orally too


ok


@sigmavirus24 this looks like a different bug, https://github.com/pyca/cryptography/pull/2062


if enable then enable? seems strange no?


1679-DualDuct (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2157 of 2216 tests passed)

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


@timclassic: This branch and fork are out of date with respect to the main webmachine repo. It'll take some effort to bring it up to date. Part of the reason this work has taken so long is that those of us who worked on it moved on from Basho in 2014, but another reason is that one fundamental issue between Yaws and webmachine is that there are at least two good ways to integrate them. The way this branch works is via a Yaws `dispatchmod`, which intercepts incoming requests very early in the Yaws reception chain and allows webmachine to dispatch the requests and serve their responses instead. This approach requires the least amount of integration because it's similar to how webmachine works with mochiweb. But another good way would be to have a webmachine-like Yaws appmod that could serve requests within any Yaws virtual server, which would allow for the easy use of webmachine for certain parts of a server and regular Yaws facilities, like WebSocket or Server-Sent Events, for other parts. But this would require abstracting different parts of webmachine than those modified in this branch.


why don't simply send the manager to this function, it's enough no?


It was withspaces when I removed it and used tab instead.. but you are probably right. Tab-indentation are configurable and people might use different length.


Done.


no need to host on github, I can show on my computer :)


Can you write a test for this?


I will admit, I've never played POTCO before, but I can probably help with trying to get accounts working, fixing GUI, crashes, and bugs. I'm still learning but I think I could help restore the game. I also probably could help with the AI if someone could elaborate what is needed in them, as I never completely understood them. I find the project very fascinating and would like to help in any way possible :)


heh the french word is "responsabilité" :)


Hi, @AntunS

`_selectedDate` is set in delegate methods `[collectionView:didSelectItemAtIndexPath:]`


Look into the rust library to see if it has a space-padding feature built in already


Why ".idea" matters?


Any updates on this, out of curiosity? Was really looking forward to this system!


Can we please replace `site:current-user` by `node:author`? The former breaks the multitenancy migration...


We can cache the max_width value and pass it in to the print_justified_rank function instead of computing it every time


y u do this xD


for so many fwb


@varund7726  I'll make one build for it after that :+1:


We are doing rank.to_string() twice. Once here, and once for the actual printing. Put the to_string() result in a local variable, then we only have to do it once.


oh! i opened an issue on this repo for the *actual* problem (which might
actually be a 'swift' problem instead of a v-saio problem).

On Tue, Jun 23, 2015 at 6:53 PM, Leah Klearman <lklrmn@gmail.com> wrote:

> upgrading pip was the first thing i tried. it had the most recent version.
> it also had the most recent version of setuptools. it was upgrading
> testtools that fixed it.
>
> On Tue, Jun 23, 2015 at 9:53 AM, clayg <notifications@github.com> wrote:
>
>> I'm sure it's all related, but that gist is indicating a blow up trying
>> to install swift - and the explosion looks exactly like you don't have the
>> newest pip? This only just got merged yesterday - if you pulled before you
>> may not have theses fixes - can you pull again?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/swiftstack/vagrant-swift-all-in-one/commit/a4c5dbc4f6ca0c32a1ed35e262255a9dd81f3e10#commitcomment-11821037>
>> .
>>
>
>


why is this added? this is causing my `mvn package` to fail when `-DskipTests=true`


I guess if `writeln!(...)` calls `rank.to_string()` then `to_string()` is called twice.


1679-DualDuct (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


65 seems like a very arbitrary number. This looks better: char::from_u32(('A' as u32) + (c as u32)). Also, this line should probably be split into 2 lines.


@Puggyblue999 Do you have a skype or are you on GameSurge IRC?


We could but I think that it would make the signature kind of ugly.

``` rust
fn print_justified_rank<...>(writer : &mut TWrite, rank : usize, max_width : usize)
```

and the call to `print_justified_rank(...)` would have to call `board.get_number_rows().to_string().len()` which is weird.


My Skype is resolute2015


This is good.

But if I try to run "make format" on a branch with no changed files, it hangs.


add spacing after ":"


Caio I need `current-user` who did the action not `node:author` [Ticket #3855]`


duplicate ":".


if it's null, just show blank instead of "0".


90664130_AlternativeModelPAT (macumber) - i386-Windows-7-VisualStudio-12: OK (2185 of 2213 tests passed)

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


i think we should just create a dataframe on the fly with one row and one value, and then apply the sha function to it


65 is *obviously* the decimal representation of a Latin capital letter `A`


Oh sorry I thought it was story creation, but it's update... I see... :/


btw i think it is ok to not have sha function and just have sha1 in python/scala. we only need "sha" in sql itself.


That sig isn't any worse than the existing one, and the variable names would better indicate its purpose. `board.get_number_rows().to_string().len()` isn't all that bad.


We aren't all as smrt as you.


This one is slightly less of a case of premature optimization because the value is needlessly recomputed inside of a loop.


@camilonova ha, I agree with your sentiment on IE8.

I'll adjust the jQuery version requirement to ~2.1.x in ```bower.json``` and make a note in the docs for users who want to support IE7 and IE8 that they'll need to use jQuery 1.x.


I shouldn't say ugly. I'm trying to unclear/obfuscated.


Does it make sense to add CONN_KIND_AS_STRING(conn) instead of every `"s` though?


Apparently I can't have kids anymore, what a way to find out.


What's the reason for this commit?


@tesshead deployed! ;)


hmm...this is a wrong commit from my end. ignore this. apologies


Has there been a change for the GameOver call parameters bug (#867)? I'm glad isDead has been addressed, but that was always a workaround to the fact that GameOver's winningAllyTeams parameter is always nil, despite game_over.lua calling Spring.GameOver.


good comments, I integrated these in other slides


Generally I like keeping all the imports at the top of the file, so there's only one place to look for imports, even though scala basically let's you put import statements anywhere. There are some times when an import would bring in a lot of names and is only needed in one small region of code, and then I think a local import can be ok, but it should be used sparingly. That's sort of what's goin on here; I think I did it this way when I first made the change to mimic what java does, which is to bring the members of an enum into scope when you do a switch on the enum. But I'll change it because I think it's not a very good reason.


fixed the unused import.


To me it's more clear than passing in "rows."

Here was my though process reading it. Why on earth would the rank printer need the number of rows? Oh, it needs them to compute the max width. And it would obviously need that so it can justify things. *Now* it makes sense.


This should remain python since its standard


This needs a try: except block


I suppose since I picked such a good function name it will work out.


There's no good reason. I removed the `deviceName` method and just changed to `name`. I have a later change queued up that cleans up the `Resource` infrastructure once all references to it from java are gone and removes `getName` as well.


removed


Can you just remove the project from `nosql/pom.xml` instead of deleting all the files?


ok, thanks guys for quick response... will use the email in the future!


I have a later change that converts all the collection classes to scala. Importing java `Map` like this is generally a bad idea because it shadows `scala.Map` which is in scope by default. However, since this is just a temporary state of affairs, I'm not too worried.


应该这样就可以了
```
#define UNUSED(x)	((void)(x))	/* to avoid warnings */
```


@drbassett says "Good job @dboone for remembering to call `unwrap()`


I chose the verbose way because the log message prints the line number which helps in deciphering the condition that was triggered


Aww, thanks @drbassett. I'm learning from the best Rust teacher I know! :smile:


Can you elaborate more about the usecases where this happen?


Didn't realize I didn't need to edit the settings file.
It's set to true by default which I think people might like more since most non pet owners don't care about seeing specific stats for each pet.


coming soon...


absolutely; queued up for a later change.


Done, made a new commit


why is the field name being added to the assert?


Hey ewrogers(SiLo)

I wanted to get into writing programs for Dark Ages in my free time. However, I have no idea how to get started. I am a computer science major albeit a crappy one, and would really like to get started. The quickest way for me to learn is to see a good example. I'm a huge fan of your sleephunter program, and was wondering if there was ever a way I could look at the source code?

Even if you were not willing to share that, maybe an older version like sleephunter v3? Feel free to contact me at mirumojay@gmail.com. I look forward to hearing from you! Also feel free to delete this comment upon reading as it's kind of off topic.


this kids a LOSER!


L


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 80](https://teamcity.labkey.org/viewLog.html?buildId=256857&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:15:36
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


removed


get_ancestor sounds fine , as it's singular people expect it to be the nearest


That did what


Why exactly is this needed?


nameExpr could be a expression tree whose leaf nodes are all literals.


for linux_i686


an expression whose leaf nodes are all literals is considered to be valid as well.


damn tabstops :>


Thank you for the comment.

Addresses have already been changed to their pre-base58 hex format since long ago. We just forgot to update the BIP document. :stuck_out_tongue_winking_eye:


You're right, calling `build` produces new mutable objects each time, so it should have parens. Generally, I try to use or not use parens at the call site to mimic what happens at the definition site, and since build is (rightly) defined with parens, I'll add them to the call here as well.


sql/hive/src/test/resources/golden/constant object inspector for generic udf-0-cc120a2331158f570a073599985d3f55

The query is:
```scala
createQueryTest("constant object inspector for generic udf",
    """SELECT named_struct(
      lower("AA"), "10",
      repeat(lower("AA"), 3), "11",
      lower(repeat("AA", 3)), "12",
      printf("Bb%d", 12), "13",
      repeat(printf("s%d", 14), 2), "14") FROM src LIMIT 1""")
```
Since `printf` in Hive didn't change word case in `Bb%d`, therefore, `Bb12` is the right answer


Looks like the wrong option was replaced within `rabbitmq-service.bat`, `-boot start_sasl ^` rather than `-s rabbit boot ^` ?


> I've been reported ...

I'm sorry, but can you provide a reference to the issue. I don't remember such report. Moreover I have no problems to perform "service anything restart" via run_command.


是这样，朋友说它的docwithcn的域名到期了，所以才用apidoc.yii2.cn的域名代替了，请问这个域名在国内无法访问么？


there is a lot :)


While possible, I don't think any others are expected (even in the long term). Some new thing will have to be invented for that to happen.


Here is the bulk of the url which triggered the problem.

/NumberField/?natural=x%5E3+%2B+x%5E2+%E2%88%92+3%5Ex+%E2%88%92+1&search=Go

I used a web converter to get x^3 + x^2 − 3^x − 1 for the polynomial.  I am giving both versions in case the converted one did not past properly.

At first I thought the problem was the 3^x but we handle that ok.  The problem is the minus signs are not the kind you get from your keyboard.

I just added another line.  We replace those unicode dashes with ascii -, and then strip any other unicode from the string.  So, the function above won't be found (it is not a polynomial), but it won't give a crash either.  If you change 3^x to 3x, then it will find what you want.


"pimpl" is a common term.


Why wouldn't it be?


very good


Please let me know if you have any comments. @nancy-lin @dhorton06 @liciasf @akorkm


done.


ACK


sorry .. I've done that edit by mistake :)


This:
```
    val scalazTest      = "org.typelevel"  %% "scalaz-scalatest"  % "0.2.3"
```
@jedesah


You should also remove the StopException class itself. It's no longer used.


Changed to be more readable.


Yes, please ... I think that I will have finished refactoring the element view tomorrow.


Thanks for exploration, im using phpstorm .


ACK. I will also review the whole coding style since this is messy.


Kryo is required since this adds a specific serializer that Kryo will use when it sees a generic record.


@adidahiya, can you review these?


It would also be useful to explain why ]0 in an indexColumn can cause false positives. For example by giving an example of a=b\0c looks the same as a\0b=c. It is not obvious from the code that this would be a false positive...


This will push to original repo not to one's fork so will most probably fail unless the person has read-write access.


Ok. I will make the changes.


Quibble - paren on previous line (as you do everywhere else in this file!)


This section still relates to Google Code workflow rather than BitBucket where one would open a PR.


PHP Application developed on Zend Framework 2, MVC framework


That does sound like a possible problem. We can always produce a modified
source package if it becomes necessary.

On 24 June 2015 at 17:34, Johannes Mockenhaupt <notifications@github.com>
wrote:

> @benbc <https://github.com/benbc> I see, thanks. Just wanted to make sure
> this wasn't accidentally released.
> Hm, I wonder if this can cause issues for (Linux) distributions building
> from source. The source zip/tar at https://github.com/neo4j/neo4j/releases
> has the SNAPSHOT suffix (didn't find a source package on neo4j.com) ... I
> might be worrying too much tough ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neo4j/neo4j/commit/0c578a5f49ba9e0c3a45ba8ed2931208644dfcdd#commitcomment-11842811>
> .
>


Some issue


The `<hr/>` can be replaced by 3 hyphens (`---`) in Markdown.


I'm not clear what removing this has to do with the PR goal


That would certainly help. :)


The github page states this: *As of November 24, 2014, The project should be synced to Maven central* so I do not believe you need a special resolver for that.


mmm...good points.  I'll work on this let you know how it goes.

management....damn that e.  My brain doesn't want it to be spelled that way.


...unless it's already at a lower index


Is this getting the user's loans from the currently logged in auth token? That's good, but we might also want a route for getting all outstanding loan data for privileged users (librarians).


> Kryo is required since this adds a specific serializer that Kryo will use when it sees a generic record.

This method doesn't add that serializer, and you can use the technique to register Avro schemas in other serializers, too. Why add it here?


What you want to do with this?
And will this 'cp' command work?


Sorry for the mistake but I think this commit should not be taken into account!


This looks about right, but as I mentioned in my comments, we might also want to add privileged/admin routes for getting all outstanding loans and for modifying the loans on an arbitrary user. Furthermore, we should probably add a route on /book/ somewhere to determine if a given book is checked out or not (which is not privileged).


Pleas add strings to translate to DISK SPACE and my suggested is to add tooltip or string to name free and total.


Fixed and rebased to [eb4c0387265510549f42f5ffb18777f7655ed350].


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


```bash
$python2.7  ./src/test_import_export_channels.py
test_01_delete_channels_data (__main__.Import_Export_Test) ...

20:08:10 deleting existing sample_data...
----------------------------------------------------------------------
ok
test_02_export_channels (__main__.Import_Export_Test) ...

20:08:10 exporting channels to json file = ./src/../data/channels/sample_data
{
    "monitor": {
        "splitter_addr": "localhost",
        "splitter_port": "4552",
        "thumbnail_url": "http:/xyz.com",
        "name": "monitor",
        "description": "monitor"
    },
    "monitor1": {
        "splitter_addr": "localhost",
        "splitter_port": "4552",
        "thumbnail_url": "http:/xyz.com",
        "name": "monitor",
        "description": "monitor"
    }
}
----------------------------------------------------------------------
ok
test_03_import_channels (__main__.Import_Export_Test) ...

20:08:10 importing channels from json file = ./src/../data/channels/sample_data
{
    "monitor": {
        "splitter_addr": "localhost",
        "splitter_port": "4552",
        "thumbnail_url": "http:/xyz.com",
        "name": "monitor",
        "description": "monitor"
    },
    "monitor1": {
        "splitter_addr": "localhost",
        "splitter_port": "4552",
        "thumbnail_url": "http:/xyz.com",
        "name": "monitor",
        "description": "monitor"
    }
}
----------------------------------------------------------------------
ok

----------------------------------------------------------------------
Ran 3 tests in 0.001s

OK
```


Test before you submit the code.


Please revert this commit, we translate only through transifiex website. Update only English.xlf


OK.


Explain?

Was this part of some PR you just merged without comment, or some extra commit?
Doesn't this qualify as a balance change?

What do you mean by "in the way"?


using doublequotes breaks my usemin-concat build. can we just use singlequote instead?


Geraint... this is hardcoded english, shouldn't we use a language string ;-)


Thinking about it, I presume the RoF buff just reduces the time between firings, but each fire has this constant charge-up time associated with it. That's what you mean by "in the way"?
The buffs can only reduce the other component of the RoF, this part isn't scaled?


Thanks.
Will do some more local testing, and will resubmit the test with your above recommendations.


Why are this functions empty?


Did you add this on purpose or was that done by Android Studio?


Thank you for pointing that out. I believe this has been fixed in a subsequent commit and in the latest release.


I don't come across it often, is it gnome style?

it's not currently used in the weston code base, where we have other modular bits.  gl-renderer, all the back-ends.
All the other "objects implemented with function pointers" instances in weston are done differently than this.

To have similar style I think we'd put all the function pointers right in struct weston_launcher instead...
The specific classes would all be like...
struct weston_launcher_direct {
        struct weston_launcher base;
        ....

And then we'd typecast to go between.

My preference would be to see a similar style in this regard, but I don't want to throw huge amounts of paint on this bike shed...


Guillermo:  A couple of comments are worth making.

The "generate-ADMS.sh" script that you have commented out in this script is designed to be run from the top-level of the Xyce source tree, not from the utils directory in which it lives  (hence the comment at its top "Be sure to run this from the project root").   So if instead of doing a "cd Xyce-6.2/utils" you did a "cd Xyce-6.2" it'd work.

But it's mostly a script that processes all of our supported verilog models then diffs the output against the already-processed versions in the src tree, and is used when a developer changes the API of the device package, hand-edits the processed .C stuff to make it work, and then finally goes back and fixes the back-end to work properly.  We make sure the ADMS process reproduces the manual changes.  Once we're convinced it is right, we manually perform the "cp" operations that appear after the "exit" (and therefore never happen in the actual run of the script).

I (the Xyce developer who winds up doing most of the ADMS-related work) no longer use our locally-modified ADMS to do this work, and have been using the ADMS from Qucs 0.0.18 for some time, since you have already ported our local modifications into the main line.

The Xyce team has locally modified versions of bsimcmg and fbhhbt-2.1 that we actually use rather than the originals, and some of the modifications are things making sure the "'attr" macro is defined, or fixing up weird things that trip up our ADMS back-end.  Those .va files are all under the utils/ADMS/examples directory (a misnomer, since they're the actual verilog we use to process these models).  The exact files we process and the exact way we use ADMS is codified in that "generate-ADMS.sh" script and the "make_X_usable.sh" scripts in the associated directories.

The "absdelay" thing you note in fbhhbt is dealt with by our wholesale ifdeffing out of all the noise-related code in FBH.  That's because until Xyce 6.3 we didn't have noise analysis, and even in 6.3 we only support noise models in a limited set of devices (none of them Verilog-derived).

I really appreciate that you're making sure that ADMS development continues to work with our ADMST XML.  ADMS continues to be an important part of the expansion of Xyce's device set.


where this binding will take place ?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.299](http://teamcity.monogame.net/viewLog.html?buildId=8377&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 499, ignored: 6 Build time: 00:24:47


90664130_AlternativeModelPAT (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


@tobiasmaltenberger warum?


these days, I prefer `latency_ = std::max (minLatency, latency_);` as a tiny bit clearer but really, that's to taste...


:+1 I like xmlunit


Uncontroversial, :+1:


@tobiasmaltenberger wieso revert? Arbeitet Armin nicht noch an der REPL?


Not sure, that's why I asked.

I think there are times when we'd rather use direct instead of weston-launch, but I think maybe that's a problem completely unrelated to this patch series.

If this is the preferred order, I'm fine with it.  Was just asking for clarification.


agree


Can you add a
break;
here? Otherwise you may fall through to the next case label when another one is added.


It's pretty questionable if this actually helps clarity, but what it does do is make an awkward rebase for anyone with outstanding dev branches touching this code, and pollutes blame :/


Have you tested apache2 ?
Here apache is listed under 'not-working-yet'
https://github.com/guardianproject/lildebi/wiki#not-working-yet


The latter, I'll amend this with a future commit. I think it depends on the missing translations.


I don't think that the `duration_cast` is needed. `auto minLatency = clock_type::now () - lastPingTime_;` should do. @HowardHinnant?


ideally the url's should point to upstream locs where possible | some staging location.


Great post! Could you perhaps add an example of a "user story" that you're describing?


Minor nit re: necessity of `duration_cast` but it's fine as is :+1:


+2 @jimmyli97 you can merge this... however. I did not want to merge it in the wrong order


I guess inline math can never include escaped dollar signs?


@aculquicondor claro, el propio `grunt` podría "compilarlos" inclusive. @rasdan ¿qué opinas?


I was under the impression that since these methods are overriding virtual methods from their parent, they should still be marked as virtual for the sake of clarity. Is this not the case?


`io.druid.server.namespace.NamespacedExtractionModule#getJacksonModules`


rax retry tests


If that's the case then you are correct. Its hard for me to tell hierarchy from within github :)


I'm still scared of the changes to `_get_group_tuples` and `_get_nodes_from_db` because I don't know what they imply. Perhaps at least updating the comment would be helpful.

Otherwise, if you're confident about this, please feel free to merge!


done.


tbh I think I have enought little functions and they are easy enough. And it's only two uses, let's leave it as is


You already print errors here, don't you? https://github.com/yDelouis/androidannotations/commit/e1931d382901e427d65b606b451f418e280de931#diff-07efede047e94782ce1cedf4acb77c61R69


@chriscool Just curious, how could this probably cause memory leaks?


The total number of results is susceptible to change on a daily basis.


Not really; I'd have to dig through the sigma docs, I haven't really used it. I just wanted to flag the weak typing of `Function`.


You should also remove the related tests.


This was a test to see if we get some notifications early when setting user info, but it appears this is not the case. I'll revert this.


line 134 - 140 should go immediately after line 128


reack


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


i think linking to Wikipedia here is unnecessary.


1699_MeterTimestep (macumber) - Win64-Windows-7-VisualStudio-12: OK (2194 of 2213 tests passed)

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


When I install docker-compose 1.3.1 with this formula ```docker-compose --version```prints the following:
```bash
docker-compose version: 1.3.1
CPython version: 2.7.6
OpenSSL version: OpenSSL 0.9.8zd 8 Jan 2015
````
If I install it using the cURL command found [here](https://docs.docker.com/compose/install/) however, this is printed:
```bash
docker-compose version: 1.3.1
CPython version: 2.7.9
OpenSSL version: OpenSSL 1.0.1j 15 Oct 2014
```
The homebrew version results in this warning when run:
```
/usr/local/Cellar/fig/1.3.1/libexec/vendor/lib/python2.7/site-packages/requests/packages/urllib3/util/ssl_.py:90:
InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail.
For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
```


Try closing issues with commit messages in the future (`closes #6`), that way you can easily tell what commit an issue was fixed by since it creates a reference.


Yes, it works. kmalloc(0, flags) returns ZERO_SIZE_PTR. From include/linux/slab.h:
/*
 * ZERO_SIZE_PTR will be returned for zero sized kmalloc requests.
 *
 * Dereferencing ZERO_SIZE_PTR will lead to a distinct access fault.
 *
 * ZERO_SIZE_PTR can be passed to kfree though in the same way that NULL can.
 * Both make kfree a no-op.
 */
#define ZERO_SIZE_PTR ((void *)16)


Hmm, do we have to #include compositor.h before including launcher-util.h now?
Maybe we need a forward decl of struct weston_compositor for safety?  (likewise, stdbool.h needs to be included before this header...)


@leoniestevens Great discussion on how to work with programmers!


In theory you should bea ble to set a password while starting the basex server. But the basex and restd server are complety differnt things.
+2


Well they follow some kind of libtool scheme. Many change SOVERSION when they change it's major version number, but they are binary incompatible within the major release number.
No problem in ditching this.


So adding dimensions is not a necessity, so don't want to raise if dimension is not there. So doing the below.

        if datapoint.get('timestamp'):
            pbuf_dp.timestamp = int(datapoint['timestamp'])
        if datapoint.get('metric_type'):
            pbuf_dp.metricType = getattr(
                sf_pbuf, datapoint['metric_type'].upper())
        if datapoint.get('dimensions'):
            self._set_datapoint_dimensions(pbuf_dp, datapoint['dimensions'])
        self._queue.put(pbuf_dp)

    def _set_datapoint_dimensions(self, pbuf_dp, dimensions):
        if not isinstance(dimensions, dict):
            raise ValueError('Invalid dimensions {0}; must be a dict!'
                             .format(dimensions))
        for key, value in dimensions.items():
            dim = pbuf_dp.dimensions.add()
            dim.key = key
            dim.value = value


A parsing error used to call exit(), and that seemed reasonable than let the caller handle the error.


Under what circumstances can this happen? Do you have an example?


whitespace ? why not static import assertEquals


AFAIU, you use the index for sharing alarms between activities, right? I'd prefer if you could rather make GBAlarm Parcelable  and pass the entire alarm object. Caution though: you also need to implement hashCode() and equals() and possibly replace or "merge/sync" instances when already have them locally in e.g. a List.

I don't like this either, but we're doing this with GBDevice instances right now and until we have a better pattern, I'd like to keep that. I guess I have to read a little bit about this topic.


I absolutely prefer the long name,  much clearer.


what is the meaning?


ok despite whitspace and import


space


I can see that engine pulls open orders from brokerages. I would also like,
at least on live trading, to be able to query broker for executions after a
certain time, IB is able to do that. Tradier as well, after a fashion. I
believe it's algorithm developers responsibility to maintain a consistent
state, even after interruption. We just need the means to know what
happened during interruption. Every night, some stock exchange orders could
get filled, Over the weekend, or from 17:00 - 17:15, Interactive Brokers
could do something. Your data center could have power outage. Interruptions
can be normal or unpredictable. Querying for executions is a basic
requirement. One way is to have it available on demand, another is to have
it done automatically once interruption is detected. I'm not familiar
enough with your source to say what scenarios are covered already. I won't
need this for a while, not before I develop and test my system.

On Wed, Jun 24, 2015 at 3:26 PM, Jared <notifications@github.com> wrote:

> Ok understood; would you prefer if the algorithm just "restarted"
> automatically or are you seeking reconciliation for manual book
> keeping/accounting?
>
> i.e. Maybe we could do some more generic state management code -- e.g.
> serializing & storing binary value of the QCAlgorithm internal object every
> minute so it could be "restored" to its previous minute state.
>
> We do save the live orders data offsite every minute, so you should be
> able to reconcile it from the order logs up to 1 minute ago.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpavlenkov/Lean/commit/d772016df95710eee1b0f3b80572c43f6e7cd153#commitcomment-11846175>
> .
>


done.


Chargedelay is there to delay firing such that the visuals fx have time to complete,  but it gets in the way if you buff ROF. Lowering it outright is a fine change


There's a problem here. There used to be permalink for c_all and it's in tweets and other places: http://dropthepilot.no2bio.org/?party=_vaada_meshutefet#mk-status
At the moment, we don't have that functionality (tried: http://dropthepilot.no2bio.org/?party=c_all#mk-status ), but once we do, maybe also support _vaada_meshutefet as a hardwired patch for legacy's sake :wink:


What's 'dow'? :-)


@skeptical yes it works :+1:


Would like to see more descriptive variable names.


The reason may be `{:error, :not_found}` here. We should check the return value and flush the `:gen_event_EXIT` message (otherwise its possible to to receive it later and store it forever never to replace the console).


Nu cred ca atunci cand afisam in metri trebuie sa mai punem si virgula. Este suficient numarul intreg.


I'm on it. I implemented support for that, not sure what wine wrong.
Next version should include committees and support status

> On Jun 24, 2015, at 23:36, The Dod <notifications@github.com> wrote:
>
> There's a problem here. There used to be permalink for c_all and it's in tweets and other places: http://dropthepilot.no2bio.org/?party=_vaada_meshutefet#mk-status <http://dropthepilot.no2bio.org/?party=_vaada_meshutefet#mk-status>
> At the moment, we don't have that functionality (tried: http://dropthepilot.no2bio.org/?party=c_all#mk-status <http://dropthepilot.no2bio.org/?party=c_all#mk-status> ), but once we do, maybe also support _vaada_meshutefet as a hardwired patch for legacy's sake
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/no2bio/mk-personal/commit/c2262f5d42a4ed74a63e5caf77e1e85cb01ed3a0#commitcomment-11847355>.
>


It depends on the type of `clock_type::duration`.  If that type is `seconds`, which I think it probably is, the `duration_cast` is not needed.
You can safely remove the `duration_cast`.  If it compiles, it will do the right thing.  And if it doesn't do the right thing, it won't compile. :-)


You're right. I had a commit which added this and I didn't know why...


Nici la km si nici la metri nu ai lasat spatiu dupa valoarea. Mereu o sa scrie "543.2m" sau "3.2km"


Oh sorry I forgot something:

```python
self._set_datapoint_dimensions(pbuf_dp, datapoint.get('dimensions', {}))
```


We should also fix the comments and the README.md instructions


@Haacked is still alive!


Поддержка SSLv3 уже полностью прекращена в 37-38, емнип, за исключением белого списка сайтов, которые без SSLv3 не работают совсем.


This is very informative but as a reader I am confused about what this has to do with Survival Analysis.
I think the paper would be improved by material in the introduction discussing the organization of the paper.
If, for example, the purpose of the paper is to provide a tutorial of a variety of statistical techniques (survival analysis, bootstrapping on stratified sample) using the NSFG data, it should say so. That would establish clear expectations.


whoops, looks like there's still some NBSP's. Please remove.


Do we need to check systemd version to ensure this capability is available?

Do we have to keep the old code around in case it isn't? :(


Actually, I'll do it. Simple enough ;)


@romjerome Are you suggesting that we change the Gramps XML?


@JordanCheney FYI: `./face_cluster_viz.py -h` to explain the options


Do we need to wait for this or ensure it succeeded in some way?


elsewhere you have import statements for packages you use. Should you have that for thinkstats2?


It occurs to me that we may want to store the date published as an NSDate. I will change the Book class accordingly.

I stored length, width, and height separately in the Book class, since I think it will make it easier to parse. What do you think?


Are we going to be logging the exopackage install correctly if this is here?


I'm curious why you don't use a Series here or some other lighter weight data structure


credits go to savoca!!!!


Alternatively, every value could get its own preference entry. And you would directly write every single entry when it is changed. I.e. no need for store() in every setter. But maybe the UI works differently in your case, because it's not a PreferenceActivity? (Why?)


I think the best experience for external Sky developers is a single tool that covers the end to end workflow for a Sky application. It sounds like pub might not be sufficient. In that case I would advocate removing 'sky:init' so that there is a single entry point (sky_tool).


You don't need git, since you can download a zip from github.  nor do you need rvm or homebrew.  I think it is important to separate the real pre-requisites from the others.


@antirez This reverses the x and y arguments - was that intentional? Perhaps it calls for a test?


Each function will sure to call this function, do an operation send then read. while the read should allways follow its send. So I think the lock can be here to pend other functions trying to use the mailbox. While when I runed it I always get blocked at the lock...


If you tried setting the association with an object of the wrong class it would show you a strange error because it would try to use a class such as LineItem, instead of Spree::LineItem.

> On 25 Jun 2015, at 05:40, Jeff Dutil <notifications@github.com> wrote:
>
> @jordan-brough not 100% positive but I believe certain associations and/or scopes had problems determining proper sql queries or would try initializing the model without the Spree namespace causing uninitialized constant errors.
>
> —
> Reply to this email directly or view it on GitHub.
>


This doesn't seem easy at all... except in the unusal case that someone has a brand-new computer (!)  I would skip this reference.


97647908-DocIssues4615 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1671 of 1671 tests passed)

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


Does this work on version 2.0 and older?


you're too late.


I've done the experiment:  It doesn't compile.  The `duration_cast` is necessary.  The `duration` of `clock_type` is finer than milliseconds, (like microseconds or nanoseconds).  So without the `duration_cast` `minLatency` becomes that finer duration.  And then the assignment from `minLatency` to `latency_` fails on line 549.  One could do the `duration_cast` there on line 549 if desired.  It doesn't really make much difference.


Looks good! Good description of a user story.
Dave


I've tended to write the docs as if people don't have special 18F privileges.. I figure people inside 18F either know they can clone or can use the fork workflow -- the later works fine for 18F-ers too and it makes the docs shorter :)


If GBAlarm was Parcelable, we could pass a GBAlarm[] to this method, so the receiver would not need to read them from the preferences. This could make sense when the storage of alarms is not device specific.


Let's not do this: looks like it wasn't required.


second commit


nice!

very nice!
ok!


And i suggested to move Disk Space before Login as... It is more clear and looks better.


ok!


ok


These are for testing, I suppose?


Raise an error of theres missing zeta data as well. I think the current
code does
On Wed, Jun 24, 2015 at 20:13 aoifeboyle <notifications@github.com> wrote:

> Oh, I forgot that part but I was going to just make it a warning, and then
> insert all the extra 1s? I don't want the code to break just because the
> zeta data of a random ion is missing. I'm raising an error if there's
> missing ionization data though.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aoifeboyle/tardis/commit/0ff678f5d905f44327b163cd72118cf94e8ba560#commitcomment-11845903>
> .
>


Nice addition to the source. Thanks. :+1:


I like keeping this reminder around... If you re-started your computer since running the bootstrap script, then you'll need to make sure elasticsearch is running.


Apparantly git did not track my changes. Most of these changes belong in the previous commit.


You are right!!! Fixed!


This was the line err'ing on windows. file://c:\mypath wasn't working.


Thanks for contributing the bootstrap script.  Nice addition!  See inline comments on instructions


Camera class: Added to functions for video logging purposes.\n The __init__ constructor I add a parameter to differentiate between fwd or base camera.The api_test.py sample run of program


Added bonus here is that this would also keep URI's from err'ing out... however the underlying objects may not support non local URIs anyways.


Here we probably need to finally add some real error handling instead of just logging. We should somewhere display an error message notifying the user that the alarms could not be set. Otherwise she could be angry when the alarms wouldn't fire the next day :-)

At the minimum we should display a Toast, or maybe even an error dialog. Everything else would probably be too much and worth a different ticket.


Coveralls doesn't add any value over Scrutinizer, thus duplication.


Day Of Week :)
--


Is it useful to send disabled alarms to the Mi? Or should we just skip them?


I just put it in during my previous PR, but since the last viewed source language is now being ordered to the beginning of the tab list, we no longer need to scroll it into view.


90664130_AlternativeModelPAT (macumber) - i386-Windows-7-VisualStudio-12: OK (2186 of 2213 tests passed)

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


why did u do this? u should reverse this changes


true. Do I need the comment to say that?


lgtm


No, they're default values: they're all disabled.


Was this intentional? Why?


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

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


No, and no.


changed.


I would too, but the existing code looks like this. Look above and below this. It's how libdbus code is usually written.


Love it, implemented and pull request submitted


I don't know, I don't use it very much. If you find it can, we can edit the regex to fix that too.


We need to send disabled alarms to overwrite the one on the band :)


I've done this now.


`infrastructures`, maybe we could use `platforms` instead?


Coveralls provide an history of the coverage evolution file by file.
Scrutinizer use the coverage information but does not show it.


Thanks for this. I was debugging an issue related to this earlier and it took me a few minutes to realize that this was probably a bug, and sure enough, you had fixed it. :)

By the way, perhaps this could be improved by using a simple macro to replace the similar sections of q_pop and q_serve? The code is nearly identical, so it would cut down on maintenance without adding the overhead of a function call.


@rubyyot Here are the changes. I used os.path.join and I changed game to self.room and used self.display in lieu of print.


@sdwilsh I think we are, instead of picking installSuccess up directly in the InstallCommand, we just return it here.  The events should also be sent to the EventBus correctly, without any additional events being sent, as we return directly after installing the exopackage, when we have one.  Or am I missing something here?


cool, thanks. worst case we just leave in a hack like this, but having
things in the code that pivot off of NODE_ENV is definitely smelly.

On Wed, Jun 24, 2015 at 12:12 PM, Robbie Carlton <notifications@github.com>
wrote:

> will let you know
>
> On Wed, Jun 24, 2015 at 12:12 PM, Robbie Carlton <robbie.carlton@gmail.com
> >
> wrote:
>
>
> > Initial research says no. I contacted them to see what their solution is.
> >
> > On Tue, Jun 23, 2015 at 8:43 PM, Lawrence Wang <notifications@github.com
> >
> > wrote:
> >
> >> don't really send push notifications in test env
> >>
> >> @robbiecarlton: i just wrote a test that calls the push notification
> >> code, and i’m putting this in here as a temporary fix. is there a test
> >> key we can use so that we can test connectivity to the API without
> >> attempting to send out real messages?
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/Hylozoic/hylo-node/commit/e3d1e3750e1e240c88ff2049da13d51e549ff6aa
> >
> >
> >
> >
> > --
> > Robbie Carlton
> >
> > 719.358.3482
> > <robbie@authenticmanprogram.com>
> >
>
>
>
> --
> Robbie Carlton
>
> 719.358.3482
> <robbie@authenticmanprogram.com>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Hylozoic/hylo-node/commit/e3d1e3750e1e240c88ff2049da13d51e549ff6aa#commitcomment-11845882>
> .
>


@ahjohannessen Cool, thx. For some reason I was under the impression that the most recent version was `"0.2.2"`. It just worked with `0.2.3`. Did you have any specific matcher in mind?


The progress does not update because there is no entry in the database after selecting 'No' I did not go to lunch.


I thought this was productive, so i merged.


Either IE9 has createEventObject or it doesn't, why do we need both here - this branch is inside a `if(self.dispatchEvent)` anyway


Hi Tom. I will fix the script accordingly. I just changed to my development machine I have to download again the Xyce sources to test it out again. I probably ended up running `generate-ADMS.sh` as documented, and forgot to update the script. I will take a closer look at the `make_X_usable.sh`.

The purpose of the new test scripts it to run ADMS (on every pull request or merge) against downstream packages. It will just test the Verilog-A parser and ADMST XML execution. I am doing this to give me some peace of mind to work more freely with ADMS without breaking other people systems. I could not automate the download of the Xyce tarball. Hence this is not yet activated on the Travis continuous integration. I guess I will continue to test for ADMS/Xyce regressions locally for now.

Thank you very much for the remarks and comments.


Hello,
I've seen you worked with realpaths. I've myself made a commit to fix those, but you forked earlier.
Maybe you can see if my solution fits your needs ?


better, less mathy ;-)


A few lines on why this is necessary would be nice - perhaps a comment. Also, why do we prefer createEventObject to createEvent anyway?

Isn't the standard way `new document.defaultView.CustomEvent("CustomEvent")`?


But Math is good ;-) hahahaha


Oh, so you use it for something specific?


AFAIK you should use onSaveInstancedata() instead of onDestroy() because the latter is not guaranteed to be called. (Only when the system is low on memory).


I think this is a great simple example to show how easy it is to use this mixin, but I think there's more that should be in this README:
* An introduction paragraph explaining what this mixin does
* An example with subscriptions
* A link to the guide: http://react-in-meteor.readthedocs.org/en/latest/meteor-data/


Feel free to use my stub generator https://github.com/Xenofied/DCStubGenerator


Changes to sqlCoverage tests (in advanced-inexists.sql) look good. (I have not reviewed the other changes.)


Honestly function composition was the first thing that came to my mind. There is no real reason I did it this way.


When can we expect this to be released?

We have some other fields that are not dates, but they're passing the check in parseQuotedPrimitive when we use DeserializeDictionary<string, object>.
```c#
if (JsConfig.DateHandler == DateHandler.ISO8601)
{
	// check that we have UTC ISO8601 date:
	// YYYY-MM-DDThh:mm:ssZ
	// YYYY-MM-DDThh:mm:ss+02:00
	// YYYY-MM-DDThh:mm:ss-02:00
	if (value.Length > 14 && value[10] == 'T' &&
		(value.EndsWithInvariant("Z")
		|| value[value.Length - 6] == '+'
		|| value[value.Length - 6] == '-'))
	{
		return DateTimeSerializer.ParseShortestXsdDateTime(value);
	}
}
```

Then they throw an error when the parse fails. The fields are usually alpha-numeric part numbers with dashes, like "1234567890T-12345". We are occasionally seeing issues with other fields that could pass this check too.

If we could override this function we could handle when the check is passed but the parse fails, and return the string that should be returned. We could clone the repo, but we would prefer to use the nuget package if possible.


Same here, onSaveInstanceState() or something else is probably better. Otherwise the alarms might never reach the device.


let's the two of us talk when you are back in the office, and we can figure it out


Cool bro, just the the `value` matcher :)


I can't understand, you declare `static` but why not move this to the top of the file, rather than giving it functional scope?


Can you add a link to the github issue where this is discussed?


I don't think weighing should happen at validation / testing time. Loss weighting is a training-specific strategy...


I can confirm, that it is irrelevant in this case, your investigation about *html* property are correct. So you must not correct this.
Anyhow, *MS Shell Dlg 2* font-family is nowhere else used in LibreCAD. The font-family attribute itself is only used in 4 ui files within *ToolTip* properties and with one exception set to *Helvetica*, a more common font-family available on all platforms.
I assume that *MS Shell Dlg 2* is default on Windows, on my Ubuntu box the default font-family is *Ubuntu*, not very open-minded too.
We have to keep this in mind and change the default to a platform spanning value when html formatting is used.


#1463.  This is a 'working' compiler for dynamic.

However, it ONLY understands these expressions (at this time)

md5($p)
sha1($p)
sha1($p.$p)
sha1(md5($p))

The reason we handle ONLY these expressions, is because they are hard coded ATM.


ok by me


Could you vertically-align the smart-alarm checkbox with the center of the time control?
Otherwise: very nice!


I think, there is no need for "." in the end of messages.


You are not mentioned in the log, was your fix offered as an issue and included from another dev?

```sh
$ git log | grep luisalvarado
-

$ git log | grep CYREX
-
```


Cool, that looks good


@Igor1234321 я не знаю, какую проблему ты этим коммитом решил, но что именно ты поломал - я знаю.
solve(log(x**2)) теперь равен 0 !!!


You can access pre-release packages of ServiceStack on [MyGet](https://github.com/ServiceStack/ServiceStack/wiki/MyGet).


Can we add a comment as to why we use `Tracker.nonreactive` here?


That's what Etherpad requested. :) I think technically the browser is not allowed to collapse double-slashes at its end, so it's important to exactly match.


That all? I must try harder to break it more the next time! :-P how'd you like a performance loop?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.394.c742fbf6ab48267c047e3b836fc6257b2509f6bc](http://build.palaso.org/viewLog.html?buildId=58875&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:54


Yep, this is because Etherpad double-fetches these and I copied the list blindly...


Seems to be a major update. Grats!


This works better because its a table rather than a list and reads better IMHO


Io ho iniziato a sistemare il codice, le modifiche sono sul branch refactory..
Finora ho:
- aggiunto l'utilizzo di [setuptools](https://pythonhosted.org/setuptools/setuptools.html) per l'installazione
- implementato il templating tramite la libreria [mako](http://www.makotemplates.org/), mi pareva un buon compromesso tra funzionalità e velocità

Adesso stavo cercando di migliorare l'embed dei link.
Comunque non ho testato benissimo il codice, mi servirebbe un dump o almeno qualche decina di articoli con parecchi commenti.

Per la ricerca da google prova così https://www.google.it/search?q=site:www.hookii.it/archived+mambombuti, a me vengono un migliaio di risultati.

Con i commenti in solr si potrebbe anche effettuare auto-tagging dei commenti e ricerca semantica (che comunque fa anche google, solo che abbiamo meno controllo).
Alcuni link che ho trovato ma devo ancora leggere bene:
http://www.opensemanticsearch.org
http://opensourceconnections.com/blog/2013/08/25/semantic-search-with-solr-and-python-numpy


Can we add a comment explaining why we stop this computation and create a new one? (And help a person reading this code understand that the new computation is created through `forceUpdate` => `componentWillUpdate` => `calculateData`)


NSDate makes a lot more sense, great thinking. I'll change my bookshelf
class when I get the chance.

On Wed, Jun 24, 2015 at 4:43 PM, Amitai Blickstein <notifications@github.com
> wrote:

> It occurs to me that we may want to store the date published as an NSDate.
> I will change the Book class accordingly.
>
> I stored length, width, and height separately in the Book class, since I
> think it will make it easier to parse. What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Librarius/Librarium/commit/29ba211540b1858f45e423eb79b42f6ca204db4c#commitcomment-11847493>
> .
>


This should be final with an all caps var name


OnBackPressedApiDependenciesTest, OnBackgroundApiDependenciesTest should be removed.


hello


I don't want to push you in some direction, it doesn't matter how this gets
accomplished, I just to be certain that no matter what, my algorithm will
have a full picture of orders and executions, sooner or later, even after
some unforseen events. If nothing is missed, then that'll be perfect.

On Wed, Jun 24, 2015 at 4:35 PM, Dmitri Pavlenkov <dmitripavlenkov@gmail.com
> wrote:

> I can see that engine pulls open orders from brokerages. I would also
> like, at least on live trading, to be able to query broker for executions
> after a certain time, IB is able to do that. Tradier as well, after a
> fashion. I believe it's algorithm developers responsibility to maintain a
> consistent state, even after interruption. We just need the means to know
> what happened during interruption. Every night, some stock exchange orders
> could get filled, Over the weekend, or from 17:00 - 17:15, Interactive
> Brokers could do something. Your data center could have power outage.
> Interruptions can be normal or unpredictable. Querying for executions is a
> basic requirement. One way is to have it available on demand, another is to
> have it done automatically once interruption is detected. I'm not familiar
> enough with your source to say what scenarios are covered already. I won't
> need this for a while, not before I develop and test my system.
>
> On Wed, Jun 24, 2015 at 3:26 PM, Jared <notifications@github.com> wrote:
>
>> Ok understood; would you prefer if the algorithm just "restarted"
>> automatically or are you seeking reconciliation for manual book
>> keeping/accounting?
>>
>> i.e. Maybe we could do some more generic state management code -- e.g.
>> serializing & storing binary value of the QCAlgorithm internal object every
>> minute so it could be "restored" to its previous minute state.
>>
>> We do save the live orders data offsite every minute, so you should be
>> able to reconcile it from the order logs up to 1 minute ago.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dpavlenkov/Lean/commit/d772016df95710eee1b0f3b80572c43f6e7cd153#commitcomment-11846175>
>> .
>>
>
>


If a not NULL format is passed to the function check_format will be 0.
Then if filter->lines is not NULL the format is set to a strdup'ed string which is not freed at the end of the function because check_format is 0.

Also shouldn't the strdup() be xstrdup() instead?

Now maybe a not NULL format can only be passed to the function if filter->lines is NULL, but in this case relying on that without a comment is fragile.


Thank you!


@timothysc Oh yes, I completely agree.


I can't find any reference to this command being used in the agar.io source.


This solves #95


This takes any action defined by the user, such as `ListInboundShipments`, and runs the related "ByNextToken" version of that action using the `next_token` argument.

All of these types of requests are identical: call the API by its action and give the token, and you get your response. This is true of any ByNextToken request in any of the APIs.


Marvellous!


You probably need to implement onResume() to refresh the list of alarms.


The difference is that instead of "manually" constructing a `SimpleConfigObject` (which just takes the keys as is), this is now parsing the keys, i.e., looking for dots (`.`) through `Parseable.newProperties()`


Should we throw an error if you're overwriting a key already present on data?


missing comments.
(I know the original code is not perfect on this point, but there is no reason not to do better on addition).


We use Swift to generate idl files in Twitter Finagle. Twitter Future is used in Finagle.


* TODO: rewrite this as it is doing way too much and is very messy now.


> You missed:
>
>     <link rel="prefetch" href="static/font/fontawesome-etherpad.woff">

Nope, it's line 51.

> you prefetch these files in pad.html, I'm curious how this actually speeds up page loading?

Remember that on Sandstorm we never display `index.html` -- we go directly to the pad, because each pad is hosted in a separate container anyway.

The reason this speeds things up is because it allows the browser to download several resources in parallel that were otherwise being downloaded in series. In particular there are a series of synchronous `require()` calls each of which forced a round trip to the server to resolve. If you look at the Chrome network trace, you can see this as a series of fetches that look like a staircase, with each one starting only after the previous one ends, using most of its time waiting for the round trip to the server rather than actually downloading content.


What is that version value? a "version" for current implementation ?


Noted. +1


Released!

> On Jun 24, 2015, at 10:48 AM, Juanito Fatas <notifications@github.com> wrote:
>
> @indirect <https://github.com/indirect> This post is already out <http://bundler.io/blog/2015/06/24/version-1-10-released.html>, but I did not find bundler v1.10.5:
>
> $ gem i bundler -v 1.10.5
> ERROR:  Could not find a valid gem 'bundler' (= 1.10.5) in any repository
> ERROR:  Possible alternatives: bundler
> Did you forget to push the release to rubygems? Thank you.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bundler/bundler-site/commit/464634b55e5474b91eaecb55e1ba01515f39a0b8#commitcomment-11844140>.
>


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.395.c742fbf6ab48267c047e3b836fc6257b2509f6bc](http://build.palaso.org/viewLog.html?buildId=58876&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:17


Oh interesting.  I hadn't realized that the context manager was being called within the worker thread/process.

Hrm, might this cause issues for serialization?  Are we going to inadvertently repeatedly pass the entire `Profiler` object back and forth between worker processes?


Perfect! Thanks.


I am sorry if I am being slow on this one. Are you suggesting that I rewrite the following line `result.run.attempt.run.getLeft shouldBe a [NoSuchElementException]` to something different?

What do you suggest I replace it with?


here, maybe only
s.append (http_end_of_header_line)
is needed, since previous Host header line already append CRNL   (end of header line)


Good Job
Maybe add space betwen size and GB also to disk space and qtip.

Move disk space to set place where is LOgin as , but login as move to right page.


Yes, exactly. The version number is sent in the User-Agent header.


Ah, good then.


in retrospect, chatting with @vtjnash, this may not have been right, since the cache and install dirs will need to be writable and don't absolutely have to be collocated in the same place as the winrpm package source


My original thoughts for this were something like the following:

```python
results = []

def f(results, key, dsk, state):
     results.append((key, dsk[key], datetime.now()))

x.mean().compute(scheduler_callback=partial(f, results))
```

This only gets us start times, not end times though.


Or -> OR, similar to LIKE.


You can leave
--| User agent header
--| User-Agent: eiffelhttpclient/0.1 .. and so on
but yes, avoid German words, and prefer English for this project.


If you you have `aurelia-cli` installed globally, you could do:

`exec('aurelia bundle)` as well


You should probably pass `{_throwFirstError: true}` here so that the test fails on this step if flushing fails (rather than printing an error to the console and failing on a subsequent step


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Good point! :-)


Yeah you may be right. I changed it.

I guess for me it was a question of not easily knowing what `hitArea` is.  You know its `HitArea` but you do not know what implements HitArea without digging. Here is the same thing but you get some intellisense suggestions.

Pandering to Intellisense though is probably not a great thing.


As i show on screen
![view](https://cloud.githubusercontent.com/assets/4134018/8341846/19c66e6a-1ac8-11e5-8ad2-b91356743c88.jpg)


Thanks. Fixed in v3 of the branch. https://github.com/chadversary/piglit/commit/6f5220fd214de50e06df0e006b832038f3990d4c


Some purists will point out that 1 << 31 is undefined in C. Use 1U << f_bits() to avoid such concerns.


Can we add a comment here clarifying the purpose of this test? (I'm not sure about it myself)


Sorry for the response delay, I didn't noticed this request earlier.
I just commited a better version, without any side effect on unmodified lines: see commit #a2a3ada


Coole Sache!!! :dancer:   :D


Are there some functions called by this function that need to modify the string pointed to by "format"?

I don't understand why you cannot just do:

```
    if (filter->lines)
        format = "%(refname:lalign16)";
    else if (!format)
        format = "%(refname:short)";
```

and not free format.


ditto re `{_throwFirstError: true}`


yep.


You'll need Git to be able to contribute to the repo. RVM (or rbenv) is needed to be able to install multiple versions of Ruby. Homebrew is the easiest way to install Elasticsearch. Yes, they are not absolutely required, but they are standard tools that make development a whole lot easier.


I'm not suggesting this as an alternative (it doesn't do half of what your solution does) but perhaps another way of thinking about the problem.  I find myself a bit allergic when things wrap others.

I like your callback solution because it composes really nicely with simple functions.  I like that it's easy to build things like this in just a few lines.


done.


good stuff


ooo fancy!


Yes,  this is so it works anywhere without an additional global install
required.  Developers just need to do npm install and optionally have gulp
global.


You still need an init.recovery.rc and a recovery.fstab


Now, line 22 will always be true and so we'll download gtest fresh every time. Since we're grabbing a specific version, this seems like just a waste of bandwidth. Add it to .gitignore instead?


you can replace 2. with 1. here, like above


it still produces incorrect results for graph bips98_606.mtx in simple_example folder.


Hrm, if we had a callback on the task starting and a callback on it finishing then we could accomplish the full profile bit with two functions

```python

results = dict()

def start(results, key, dsk, state):
    results[key] = [dsk[key], datetime.now()]

def end(results, key, dsk, state):
    results[key].append(datetime.now())

x.mean().compute(start_callback=partial(start, results), end_callback=partial(end, results))
```

This still doesn't get us thread-id


These names stink. How about defining macros for like

is_tesla, is_fermi, is_kepler, is_maxwell

and then you can define macros like

is_tesla_plus, is_fermi_plus, is_kepler_plus, is_maxwell_plus

as well as things like

is_tesla_or_fermi

or whatever. Sometimes you need to go intra-family, so you can do like

is_tesla_g94 or whatever. This also avoids re-listing the same gpu lists 100 times.


Why not just ``Arrays.stream(...)`` :)


Feel free to ask if you want to know anything special.


there's no gf100b defined above...


Compilation of projects broke with this commit, something's off


I would request the inclusion of "nationality or nation of origin" after "race, ethnicity". I've seen people harassed simply because they live in a nation whose government has fallen into disfavour, or whose nation was once at war with someone else's nation.


Nit: space between type and param.


The laptop script is meant to be run on both brand new and used machines. It only installs what you don't already have. The script is safe to run multiple times, and I run it regularly to make sure all my tools are up to date on my computer. What don't you find easy about this script? All it takes is copying and pasting two lines.

Is there an easier way you have in mind for someone to install Homebrew, RVM, and Ruby on their machine?


I'm not convinced by this change.  The toolchain/implicitProducts are a dependency, and simply dumping them doesn't seem right to me.  Can you tell me why this is the correct fix?


this image goes with the previous paragraph, so the empty line above (line 120) should move to under this line. not sure if it matters that much on the image layout tho.


This is incomplete. Either you should simply pass the command and its arguments as an array or you should also escape less-than and greater-than characters (<>), ampersand and pound (#), with a preference on the former as being more secure than the latter. If you do the latter (character escaping), you may still be vulnerable to command injection vulnerabilities.


I wrote some of this in e-mail, but you're missing:

aa, ac, af, ce, cf, f1, 106, 118, 126

this might not be so important for headers, but if this aims to be a complete list, i think it's nice to actually be complete.


Sounds good.


generally looks good to me :+1:


rax retry tests


Nit: Space between type and param.


Sure.

This is the test that if you call `React.render` from an autorun, invalidating or stopping the autorun doesn't affect the mixin's autorun.  It's the test that fails if `Tracker.nonreactive` is removed from the mixin.


Good point @petergaultney, I'll see to it.


Ha! I am! But I no longer maintain this repository. I forgot to turn off notifications. I just did this one because it was small. Still looking for more people to take over maintenance.


This looks like a bad copy/paste from the previous changelog entry.


What are you trying to make the graph look like? i.e. where do you want edges?


* TODO: rewrite this as it is doing way too much and is very messy now.


yeah i would be into that! any idea how much you're gonna need to work?


@radez I think instead of using "opendaylight_host" we can actually just index into the array controller_node_ips[0]

It is set for us earlier in the file.  Going to test it out tomorrow and see if it works.


@0xced thanks for the catch!


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@skeptical  I think that FoldersContainer could be always shown like for the video or mixed content and only when image is zoomed out it should be shown in the full window.


Good point. I think what I'll do is remove the import from the bootstrap script for now (until it becomes idempotent), and add a note to run the script after rebooting, or just every time before running the app. This will ensure elasticsearch is up to date, that it is running, and that all gems are up to date before running the app. How does that sound?


Because that first `df` parameter to `Forward` is ignored when the passed `EthernetDecoder` is not `nil`, as here (see the change to `forwarder.go` in this commit)

The point of this commit is to eliminate the `df` parameters wherever that is straightforwardly possible, i.e. wherever the DF status can be derived from the `EthernetDecoder`.  But that does not extend to `Forward`, which is called from various places without a `EthernetDecoder`.  So it keeps its `df` parameter for the sake of those cases.

Yes, it's ugly.   It's intended as a temporary state of affairs, pending fccd6095bf, which takes care of those remaining uses of the `df` flag, so that it is not present in the `Forward` method of the `InterHostForwarder` interface.  And even that commit remains a stopgap in this general area, because the ICMP frag-needed logic reaches its final state in 3bcabcb9c.

This division into commits reflects the order in which I implemented things, and I didn't want to squash the three commits because fccd6095b is already a lot to digest.  But perhaps the commit here is too ugly, even as an intermediate step.  In that case, I could rework the commits to keep the `df` flag everywhere, past the point where `InterHost` is introduced, and remove it later as part of the frag-needed changes.  The end result would be the same, but I can see how the presentation would be less disconcerting.


Hey why is this commit called "Worm BBQ"?  Please don't invite me to dinner!  :)


saw approval from fadushin
at https://github.com/basho/riak_pb/commit/831e6263fec6a0d71059842cb0d3bfeec97db917


merging basho/riak_pb/backport-revert/zl/revert-RpbYokozunaIndexPutReq-timeout-commit = 831e6263 into borshop-integration-122-backport-revert/zl/revert-RpbYokozunaIndexPutReq-timeout-commit


basho/riak_pb/backport-revert/zl/revert-RpbYokozunaIndexPutReq-timeout-commit = 831e6263 merged ok, testing candidate = b96d7aa2


I do not agree with this renaming. Although Xcode says *Analyze* in its logs, it’s actually building **and** analyzing as a side effect. Even when setting `RUN_CLANG_STATIC_ANALYZER=YES` the build artefact is still built as part of the process.


tested


```jl
 julia> module Foo
           "This is the `Foo` module."
       end
ERROR: syntax: unexpected end
```


What is this for


yup


Thanks! :)


```jl
julia> "This is the `Foo` module."
       module Foo
           bar = 1
       end
ERROR: cannot define function eval; it already has a value
```


It's normal to do so, as long as the key came from us.  We could throw an error if we go to overwrite a key that wasn't in `oldData`.


What is this check for?  It doesn't seem correct since service deployment is different from whitelisting.  Kilkar isn't automatically deployed in every state, but this check looks like it would have that effect.


Also, the bootstrap script uses Homebrew to install elasticsearch, which is why I added it as a prerequisite for Mac users only.


Nice, that's even better than mine.


The intention is to return "error" here, right?  If so, this should return ```true``` rather than ```false```.


You realise your routine is most likely slower, right?

Both approaches have the same expected number of EntityCategoryContains calls, but yours additionally has a hash table lookup for each one, as well as the loop overheads. It's not really sufficiently clearer to be worth the expense, methinks (this is fairly hot code, after all).

It is, however, probably sensible to pull the XP values out as named constants for the sake of readability. We could also perhaps rearrange the branches to trip the most common ones soonest to maybe save a teensy bit of time.


Thanks a lot for this! :)


if you did `checkout -b` before that branch already exists, so you don't need `-b` argument, do you?


According to [docs](http://www.postgresql.org/docs/9.4/static/indexes-partial.html) `WHERE` can be normal condition, so to be totally proper it should be written as:

    my $index = $table->add_index(name => 'myindex', options => [{using => 'hash'}, {where => {'upper(foo)' => 'bar', bar => 'foo'}}], fields => ['bar', 'lower(foo)']);

But this would require code similar to `SQL::Abstract->where`.


You're welcome. ;)


Nice.


@sils1297 as far as I know, we need this `-b` in order to tell Git to create the branch


OK!


no ID 49292
correct ID 5476

Id: 5476
	AegisName: "Hairband_Of_Grandpeco"
	Name: "Grand Peco Hairband"
	Type: 5
	Buy: 20
	Weight: 800
	Def: 2
	Loc: 768
	EquipLv: 30
	View: 473
	Trade: {
		nodrop: true
		noselltonpc: true
		nogstorage: true
	}
	Script: <"
		bonus bStr,2;
		bonus bMaxHPrate,-10;
		bonus2 bAddRace2,3,15;
	">


What's the purpose of the `awesome` property?


@yegor256 but you already did that in line 26


although not sure why but i also have this in Account class
```

  def self.api
    BookingSync::API::Client.new(BookingSync::Engine.application_token.token)
  end
```
as if it didn't work with sync out of the box


Id: 5501


Dan: is there a reason to avoid the simple recursive approach? I doubt the recursion limit will ever be a problem, and based on some experiments, it is faster than the previous approach. Thanks!


Never mind. Franco and I talked about this and the commands that are being run are trusted commands. So long as that remains the case (not running commands inputted from the user in the UI, for example), this particular use case is okay.


don't remember exactly atm, but the warning doesn't look too optimistic https://github.com/BookingSync/bsa-booking/blob/staging/app/models/account.rb


Like


Great! we will have proper api for cli in the next release. That will make things simpler. Until then, this is the best way to go.


Yeah man! Very cool!

We should push the cookie assignment into that 'else' block as well.


Mec c'est un peu tard non? :dancer:


saw approval from cuyler
at https://github.com/basho/riak_core/commit/ab0b7a3e24f7e063066e839711fe3d6afdb3f15c


merging basho/riak_core/nem-clique-0.3.1-version-bump = ab0b7a3e into borshop-integration-748-nem-clique-0.3.1-version-bump


basho/riak_core/nem-clique-0.3.1-version-bump = ab0b7a3e merged ok, testing candidate = 4b698a2c


This could be nice behavior if no chunk size is specified via e.g. ```MALLOC_CONF```, but as written it prevents setting the chunk size to something different.  You could change the ```opt_lg_chunk``` initializer to ```0``` rather than ```LG_CHUNK_DEFAULT``` and conditionally set ```opt_lg_chunk``` to either ```LG_CHUNK_DEFAULT``` or as above, based on platform.


Id: 4231


space between // and parseconfig, period at the end


all tests pass:
success: http://buildbot.bos1/builders/test-riak_pb/builds/140


Nice!


all your comments should not include the object at the beginning. e.g. this one should be "Implementation of Interface, TCPLoadBalancer,...".


should be "// Takes in a config file..."


Hmm -- why do we maybe see the old data at this point?


Oh! I can see you are the author of less. Awesome! I am great fan of less by the way.


space between // and If


... at the end


Where would `end` run? When it's popped off the queue? That's not the same time the task actually finishes running (although the difference may not be great). Worker id can only be gotten by the worker, unless I subclass both pools to change the handling code.

I really think getting per-worker stats is a good method, but it can be tricky. One possibility would be to have `execute_cm` tandem on the queue already passed to `execute_task`. Could either stage the result, and send it with the actual return, or tag it as a callback result somehow and send it separately. However, note that you're already serializing a queue for each task execution for multiprocessing, so doing a second queue might not be that much overhead.

Really, the IPC for multiprocessing should probably be reworked to not serialize a queue every call. This may diverge the scheduler from `get_async` anyway.


it's ok now! thanks...


No problem.

I see that all tests pass with no problem. I need to add tests for
GFlow.Node object and see.

For feature iterations, I'll try to open an issue before to remove
functionalities.

I'll try to add some GUI tests in order to find regressions when GFlow is
changed.

2015-06-24 12:29 GMT-05:00 grindhold <notifications@github.com>:

> I will incorporate the changes to the buildsystem but i will revert the
> changes to GFlow. After analyzing the changes i came to the conclusion that
> they remove features that I want to have in the library. Sorry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grindhold/libgtkflow/commit/5e26865775649cf0a27e729860296ee3e51a9c90#commitcomment-11843811>
> .
>

--
Trabajar, la mejor arma para tu superación
"de grano en grano, se hace la arena" (R) (en trámite, pero para los
cuates: LIBRE)


Doesn't apply.

The specific cases where a non-HS-authenticated connection might be used (such as file transfer data) are new features that wouldn't be used with older clients. If they somehow were, the channels would fail.


Hmm I think I got an idea why, I'll look at it closer tomorrow.


This makes sense, but an explanatory comment seems like a good idea.


Did you mimic this from somewhere else?


Ah, good point. We could, alternatively, also `test -e gtest-1.7.0`. Or we could add it to gitignore. I can go either way.


I want `Event.key` now :(


`execute_task` could also return its worker id along with the `key, result, None` or `key, error, traceback` tuple.

I think that this gets us the information we need without having to execute the user-provided context-manager within the worker.


I am seeking this modification but in my db does not exist


Ah. More important than "when the autorun is stopped" is "when the autorun is invalidated" since that's the whole point of autoruns I guess. So basically this test tests that `React.render()` inside `autorun` even remotely does the right thing?


Oh whoops, I forgot to do shadowing too.


"Ecce homo"


And yes, I think we would run `end` right after things are popped off of the queue.  I expect the slop here to be below the hundreds of microseconds range.


Oh, right -- yes.


@sils1297 oops, my bad, you're right


use gcfg


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 83](https://teamcity.labkey.org/viewLog.html?buildId=256986&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:55
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


They accept pull requests through their gerrit, not github.


As this config is optional: Sure you want to "echo" an error (STDERR) if it doesn't exist? Maybe better use `config_try_read()` for this as well?


I had a few questions and added some comments, but generally looks great!


@valib for me 'less is more' (and esp considering small screens in this case), but I can see your point too.


This is breaking the build on openshift:

[ERROR] Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.4.0:exec (exec-npm-install) on project claire-web: Command execution failed. Cannot run program "npm" (in directory "/var/lib/openshift/558b0dec4271862425000162/app-root/runtime/repo/claire-web/src/main/webapp"): error=2, No such file or directory -> [Help 1]


Oh I see. So the file padding has a prefix to it, so to speak.


This is for GNU make not CMake though.


I don't think `this` is necessary here since the usage is not in a lambda.

Same for some of the below cases.


0. records panel
 a. record strings should align colon regarless of changes
   - Implemented a new class to handle the label output (DisplayRecordLabels)
   - I implemented a new interface to store the Table Name Constants (ITableNameConstants)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1596


Not work without unlocked instance.


My kingdom for a static type system.


You are correct. Look up the `Result` class on the Rust docs. `unwrap()` just tells the current thread to `panic!` (i.e. print an error message and stop) if there is an error - probably not what we want here. `try!`, on the other hand, is an easier way to type:
```
if result.is_err() {
    return result;
}
```


InterfaceSupported


Couldn't this be expressed a little more idiomatically with `map`?

```scala
def freeze(): Unit = parent map {
    (oldParent) =>
        parent = None
        back ++= oldParent.iterator withFilter {
            case((key,_)) => !back.contains(key) && !whiteouts.contains(key)
        }
}
```

Or with `match`:
```scala
def freeze(): Unit = parent match {
    case Some(oldParent) =>
        parent = None
        back ++= oldParent.iterator withFilter {
            case((key,_)) => !back.contains(key) && !whiteouts.contains(key)
        }
   case None => { }
}
```

Looks good, though. :+1:


In Analyster.java I removed the Table Name Constants and the class not implements ITableNameConstants.java

The other code is just me debugging in Analyster. Don't worry about that.


You never use this parameter.


That's a good thought. I'll try that out. Just to clarify, my plan is:

1. `start_callback(key, dsk, state)`, replaces `scheduler_callback` (just a name change).
2. `end_callback(key, dsk, state, worker_id)`. I don't think state is necessary, but I'd include it for consistency. However, if we include state, then it should be run after `finish_task` has updated the state to mark the task as finished, which can add some more delay on task execution time. Probably not enough that we care though.


Comment on the change here?


Also, I think you should always use u32 in uapi headers, though I'm not 100% sure. I know I started a bad precedent above, this was before I knew better.


This won't prevent me from merging the pull request, but please remove this extra \


Is this code actually reachable?  @rxin points out the compiler is printing a warning.  Can you fix and add a test in a follow up PR?


From the linked page, it seems a better strategy would be to run 'oslevel -s', split it in 4 components (release, technology level, service level, service level release date), and then maps those components to the following fusioninventory model:
OPERATINGSYSTEM/VERSION           => "$release TL $technology_level"
OPERATINGSYSTEM/SERVICE_PACK => "$service_level"

This would avoid running oslevel multiple times, and performing multiple adhoc cleanups on output.


Actually, the rule is to only use `__u32`. Let's save this for a future cleanup.


LGTM


`unwrap()` unwraps an `Ok` `Result`, and `panic!`s on an error result. `try!` returns the result early if it is an `Err`, and unwraps and returns the value if it is an `Ok` result. Sounds like you have things backwards.


oh yes, I wanted to make a minor optimization but didn't get around to it, so I had ended up commenting it out. It's almost 100% unlikely to affect you, so just uncomment it out.


Are the first two checks really useful here?


as to match, I was worried about overuse of Match, because The Internet Told Me(tm) that match was slower than an if, especially when unpacking simple types like Option


That smells right to me.

I suspect that `get_async` would now need to take in a "get worker id" function, along with `apply_async` and `Queue` functions.


fixed.


The only two other producers that handle `->options` are MySQL and Oracle, and they are case-insensitive about the names (Oracle just passes them trough, because they're sufficiently regular and all go in the same place, apparently). We should stick to the same behaviour for PostgreSQL, but we can do that with a a simple `uc($key)` eq 'WHERE'`.

There should probably be a check that each option is only specified once, since `->options` is a list of hashrefs.


The entire schemabuilder has been removed.


> as to match, I was worried about overuse of Match, because The Internet Told Me(tm) that match was slower than an if, especially when unpacking simple types like Option

This is correct, `foreach` is the correct choice; the `match` example is just in there for completeness sake.


Maybe this should default to off?


fast-forwarding develop to borshop-integration-748-nem-clique-0.3.1-version-bump = 4b698a2c


Someone is going to have to explain me why foreach is the correct word then, because it sounds/seems super wrong in my head when there's not a collection being involved. there's either one or none, why are we foreach-ing?


fast-forwarding 2.0 to borshop-integration-122-backport-revert/zl/revert-RpbYokozunaIndexPutReq-timeout-commit = b96d7aa2


I'm guessing this is what you were after. Now the next step is figuring out how to determine whether a  graph is bipartite or not. And then, given a solution for this, how to animate the steps for colouring the graph. Sounds like a fun problem. I won't commit any more on the master branch - this is a no-no when collaborating. Not only does it step on your work, it could also potentially introduce bugs to the program. Thankfully, this program isn't a product with customers who could suffer from dodgy builds on master. So what I'll do next time I make any changes is do exactly what I would do on a real project: do my work on a separate branch and then make a pull request for you to review the changes.


There's inconsistency in the capitalisation of StructureMap (the official SM page has a capital M). In this commit the namespace has a lower case m, Structuremap, and the folder in the file structure likewise is lower case but in other places it's capitalised (e.g. StructureMapServiceScope). I suggest making these all capitalised to be consistent with the DI container name.


Why do we prefer the lodash bind method?


catastrophic pitfall would be a great band name


I believe this is also called in `Widget::get_str_attribute()` and friends so we should make sure it's not duplicated there.


90664130_AlternativeModelPAT (macumber) - Win64-Windows-7-VisualStudio-12: OK (2186 of 2213 tests passed)

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


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


It allows partials.


The $gcCmd dependency should go in Implicits, but the testPassed dep should go in OrderOnly.  That way the package won't build unless the test passes, but a change to a test file won't cause the package to be rebuilt.


Mostly it is one more thing to read, evaluate, consider.  Since it isn't required, I think it is better to leave it off.  (Sorry, I didn't read it too carefully in review -- I hadn't realized you can run it repeatedly, but generally, wouldn't want to test that it'll work with all my manually isntalled stuff, when it is easy enough to brew install whatever to get just one more dependency. )


So does [func.bind()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/bind)


> there's not a collection being involved

An `Option` _is_ a collection; it's a collection of zero or one elements. Most descriptions of the correct use of the `Option` type in Scala [suggest](http://danielwestheide.com/blog/2012/12/19/the-neophytes-guide-to-scala-part-5-the-option-type.html) that you think of it primarily as a collection, rather than a nullable reference (which would encourage `if (_.isDefined)`-style code).


The original code used "x" for longitude, "y" for latitude as arguments there, so I reversed them to fix the issue :-) But there was no *issue* actually, just wrong var names, the original arguments and the receiver function arguments match, positionally speaking, even in the old version.


ok -- I missed the line where it runs elasticsearch.  Good to note that it will make sure elasticsearch is running!


quoted.startsWith("\"") && quoted.endsWith("\"")


just some old hermit in a furry suit


Original upstream no longer exists unfortunately


nice commit :100:


I like :+1:


I don't know exactly, but I don't know of any rule that says it shouldn't be either.  In practice, it seems to be there 25% of the time.  I didn't dig into livedata to find out why.


Here you can use the variable $VARIANT


And also here you can use the variable $VARIANT


s/LInux/Linux/


While doing my port to Windows I've realized that this misses the use of pause in [detail/AtomicHashUtils.h](https://github.com/facebook/folly/blob/master/folly/detail/AtomicHashUtils.h#L33), which, although it does fall back to not using a pause, should be calling the functions added in this instead.

Also, is there an actual difference between `asm("pause")` and `asm volatile("pause")`?


@wkerzendorf In the current `plasma_array`, it says:

        try:
            zeta = interpolate.interp1d(zeta_data.columns.values, zeta_data.ix[phis.index].values)(self.t_rads)
        except ValueError:
            raise ValueError('t_rads outside of zeta factor interpolation'
                             ' zeta_min={0:.2f} zeta_max={1:.2f} '
                             '- requested {2}'.format(
                zeta_data.columns.values.min(), zeta_data.columns.values.max(),
                self.t_rads))
        else:
            # fixing missing nan data
            # issue created - fix with warning some other day
            zeta[np.isnan(zeta)] = 1.0

The main dataset has always been missing the zeta data for atomic numbers 29 and 30 and it doesn't raise an error.


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


@jorgenavas, please take a look. I've added support for unsigned operations. Perhaps it also fixes the example you had with division.


fixed here: 522d243d8d51ad15054914d6e063360d8adc0507 bro!


Need to add another one of these for the stout tests. In the autotools system currently, the libprocess project builds and runs the stout tests.

See: https://github.com/apache/mesos/blob/master/3rdparty/libprocess/3rdparty/Makefile.am#L143


Can you call ```e.getResponse()``` and check the status code instead?


Wooo!


I guess if I were to add a helpful comment, it would be something along the lines of that one subscription has gone away and another has been added, and the old subscription's data may not have been removed from the cache just yet.


`mkdir -p build` is nice because it doesn't return an error code if the dir exists which may be relevant if `set -e` is later added to the script (no need to hange here, just a tip)


Squash with the prior commit?


@Haacked thanks for getting to this, i was in process of reviewing it when you merged it in :)


Hi
I want the graph to Initially look like this
I have achieved it on my unpublished version but it's obviously not as elegant as yours and you would describe it as a hack.
"A bipartite graph is a graph in which the vertices fall into 2 disjoint subsets A and B such that no vertex in A is joined to a vertex in A, and no vertex in B is joined to a vertex in B."
I'm not sure if I will be able to do the manipulation required from my version which will require accessing an100 item array,there are 5 vertices per column in mine,to solve matching problems. I can see yours is the proper way but I don't understand a lot of it.
I managed by stepping through it to see where it was drawing the inter column edges and by changing one number prevent that happening in column A. Although from my comment I found a pattern for the column B error I didn't know how to correct it. I think you would say it was way above my pay grade.
 I've been watching some interesting tutorials by Douglas Crockford ,creator of Json and JSlint . He says unlike other languages most people never actually properly learn JavaScript but just pick it up by doing things and looking at references,he also referred to the abyss where programmers spend a lot of time trying to solve problems,programmers have to be optimistic. He also referred to a book called Thinking Fast and slow by a noble winner in psychology. He says nothing in our evolution has prepared us for programming so it's amazing that we can do it.But it explains why in general we find it difficult to do and make mistakes.

Sorry I'm a slow typer and your second email has just come in. I guess you've solved it thanks. Yes it is an interesting problem. I really enjoyed my last course and most of it was algorithms and I would like to have a go at writing some code to solve some problems. This one for instance starts with an initial "guess" at the maximal matching and then there is an algorithm which results in the answer. I'm interested in A.seeing if I can convert the algorithm into JS code and B,seeing it graphically solve the problem. I think I understand from your email is that you will work on a separate answer and then I can adapt bits I like.
Dad

Sent from my iPad

> On 24 Jun 2015, at 22:35, JamesBrill <notifications@github.com> wrote:
>
> What are you trying to make the graph look like? i.e. where do you want edges?
>
> —
> Reply to this email directly or view it on GitHub.
>


If you're doing another commit, I think it would be worth adding.


Looks like type.


Hmm, I think I wanted to publish some "data" besides `_id` and then verify that it was part of the query results, but that doesn't seem like a great reason.  Should I just get rid of it?


maybe `build.sh` should `cp -r html build/html` so that this script doesn't have to be relative to "build". in general, keeping `build/` dirs totally isolated and files inside it relative to `build/` simplifies a lot of things/expectations.


Hmm... Is this a bug in Meteor? Is this expected behavior? This surprises me (and I think it will surprise users)

@glasser, what do you think?


Just noticed this myself, bump.


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


I'd say yes, get rid of it.


This needs to be adapted, too.


Tres bien!


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 84](https://teamcity.labkey.org/viewLog.html?buildId=257009&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:56
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

```


To user an OrderOnly dep below you'll also have to change this to not have a dependency from the phony go package to the test source, otherwise it will re-bootstrap when the test changes, not rebuild the package, and loop.
Instead, you'll need to create a phony target that matches the test binary and add that to the re-bootstrap dependencies.


Works for me, on a large notebook with several of these : https://github.com/matthew-brett/perrin-academy/blob/master/fourier_no_ei.ipynb


I'd prefer to keep as little information as possible in the README, and almost all of it in the guide - what do you think?


Give it your all


This should be boolean. Original line was correct.


I forget how data is removed when a subscription handle is stopped.  Isn't the server involved?  If so, that would explain the lag.

We would have to enforce a property like, "We don't mark a new subscription as ready until all the data from stopped subscriptions is removed from minimongo."  I'm not sure what that would be like to implement; maybe you can write an app that is always in the process of stopping some subscription, I don't know.


@castaway

Hi! Can you please check if `...DEFAULT TIMESTAMP (... )` (line 36 in the related diff) is the actually correct way to do things? I can't seem to find this particular incantation in the ibm docs...


What if `getMeteorData` sometimes returns a key, and sometimes doesn't?

```js
getMeteorData() {
  if (x) {
   return {a, b};
  } else {
   return {b, c};
  }
}
```


Good catch with the redundancies.


changelog plis


I don't think this change makes a difference.

It makes a bit more sense to throw out everything that lacks a mountpoint attribute, like:

```
devices = [d for d in devices if hasattr(d.format, "mountpoint")]
devices.sort(key=lambda d: d.format.mountpoint)
```

That way, the code below that accesses the attribute directly, as:

```
if not device.format.mountable or not device.format.mountpoint:
```

won't look so suspicious.


A plan of what you want to accomplish by the end of the day.
Do the task assigned for you at a particular time. Don't procrastinate


@jwayman fyi.... might want to pull into NL docs. Adds proper sizing for headers with ===== and makes all hx larger than they are now. Looks better imho.


There is an issue with the libwebp repo


That should be fine.  The mixin basically diffs the old keys and the new keys, and deletes and adds keys as necessary.  We'll only throw an error if the key is present in the new keys and absent from the old keys, and yet is still somehow present on the data object.


Did you really intend to just delete the API documentation? Do we already have replacement docs?