I can suggest a better implementation tomorrow
Cheers

On 19 janv. 2016, at 21:36, Sam Yates <notifications@github.com<mailto:notifications@github.com>> wrote:

I think we need to rethink the get_propensity() method: passing all the values back via a std::vector is fairly heavyweight, and it's not necessarily easy to provide all the propensities for any SSA selector implementation.

I'd rather modify the API to support a propensity() method that takes a key_type patameter and returns the corresponding propensity, or an operation_not_supported exception (at the discretion of the implementation.)

I'll merge the pull request and submit some proposed amendments as a separate request.

—
Reply to this email directly or view it on GitHub<https://github.com/delalond/rdmini/commit/5f4b60569ad5e705ab45269712c15a414f9242ca#commitcomment-15540608>.


Nevermind, I saw the following commit.


Updated docs:
https://cordova.apache.org/docs/en/latest/cordova/events/events.html


I have just added resolving IP addresses in host names ...


lol, sometimes I wonder why we even have it enabled


hell yeah


jaja, si. lo agregaste en el master y yo hice un pull por eso estan en este commit


Okay, the xcf files have finally been pushed. I can try to take a look at the domelights later, but not today unfortunately. Is that all right with you?

> So no happily finally closing issues!

That's the best feeling, right? :smile: got to love these green GitHub buttons!


Cf. 6bec9de618f30a06e7bce576412876afd16ce550


Should we have a step to test the browsers in our supported browsers matrix?


Good point; do you have any other ideas on how to avoid this spurious redirect?


It looked different 2 commits ago: https://github.com/prolificinteractive/swift-style-guide/commit/d2bc7cd9f7dec965d5e2f37d5f357b684ebb5135

 I'm confused on what happened. Let's review together


The notification happen just when a new song or list of songs are added to Queue.. should not be notifying when the view changes e.g switch between playlist/search results/likes.

If that happen.. it's a bug.


I think its better than gregorycu's implementation, and would +1 it for inclusion into master. We can also take his, and then commit this extension.


sera un refactoring para mi en el futuro


Please also remove this statement


haha...tja, ist halt java ;) Wer hat das nicht schonmal gemacht :D


the return value? You mean the returned object?


Its general question to understand your intentions (best practice and match with file names) :)


Lot to review, I'm on it tomorrow first thing.

Could you add some comments?


U projektu ti ne trebaju datasetovi i table adapteri. Jednostavno ćeš preko onog promjer za Cars, direktno popuniti objekt preko SqlDataReadera. Znači tu funkciju spremi u AutomobiliDAL i neka ti vraća listu objekata AutomobiliVM.
U klasi AutomobiliVM neka budu property kao u tablici automobili...


Hmm. If someone has been offended, we need to "do our best to right the wrong". I think that goes too far.


+1 to the `class Broker` description from [above](https://github.com/Ichimonji10/pulp-smash/commit/a056d1ffd7f8a4c5af996021940d128804ef5467#commitcomment-15540702).

+1 to using the `which`. Very nice.

I didn't realize the `Service` and `ServiceManager` were either-or examples, but I see that now. +1 to modeling `Service` the way you've written it.

@Ichimonji10 Thanks so much for this great work. I like the discussion before we dig into the implementation. Good planning.


@denysdovhan I don’t know what you want to do, could you explain that?


Sure, I was using 1.6


A glorious return!


Is Matlab not mature yet?


Can you add a comment explaining this ? :D


There is still a "from numpy import pi, inf" in casadi.i, so you can do casadi.inf, casadi.pi (as before).


The call to `GetRecentlyAddedMoviesNav` is missing the `getDetails` bool (arg #4) so `hideWatched` is being interpreted as `getDetails` and no items are being hidden. Same for `GetRecentlyAddedEpisodesNav` below.


This is a bit unclear to me. Maybe like `show_student_recent_results`. I don't really like that either... what you have could be confused with not showing the results after they finish taking a quiz, which is another potential use-case.
ummm.... any ideas?


Oh, I suppose I can just switch `Editable` in line 368 to the correct value... That will solve half the problem...

However it will not solve the issue of the obliterated `viewingCell` state; but I think I can solve that in a similar way.


Thanks for posting this, this helped me get started :+1:


Arguably, feel free to update.


Ovo je Odlično...


And this one. Thank you


The node.js variant of chakra engine that powers Windows IoT relies on `chakra.dll` that is shipped with Windows 10   OS. The `process.jsEngine` for this variant is `chakra`. The PR relies on ChakraCore (which is open source and whose source code is embedded under `deps\chakrashim\core` ) The `process.jsEngine` for node.js built on chakracore is called `chakracore`.
Since PR is solely based on ChakraCore, I think I should remove `chakra` check from here.


This is probably the reason, indeed. Though I don't really understand why this was necessary before that. This happened long ago (570060732bd9acd004de01125d287a9ca664f770).


Oh actually it does say it in the final file. I was looking at a specific commit. I guess I'm confused by Github


Can I put some pointer to Yacoda in the paragraph down?


That looks wrong. list is already a list, you don't need to create a new range from it


This huge comment can be removed and integrated in the commit message if you want


Why is this necessary?


newer versions will check dependencies automatically, you should upgrade devtools as well ;)


This is an older commit of the document you're looking at, but good idea, I've added it to both contributing and release docs.


Yes, traditionally, the plaintext links are mailto links (for people that access email via a terminal, lots of them in CSAIL). The link address should be something like group_name + '+' + str(tid) + UPVOTE_SUFFIX + '@' + HOST

and there should be a place in smtp_handler/main.py that looks for that upvote/un-upvote email - take a look at the commented out code with "def handle_upvote" and handle_downvote. and test to make sure the mailto works properly on the dev server.


I believe you are correct, sir!

On Tue, Jan 19, 2016 at 11:08 AM, Benjamin W Stookey <
notifications@github.com> wrote:

> I suspect it still pulled down the zip, but when it found that the version
> was the same it didn't update the cache.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AASHE/hub/commit/9e2891f9aa8ff570fb2330141ab8a3fa63c544cc#commitcomment-15533918>
> .
>

--
Robert Erb
Web Application Developer
Association for the Advancement of Sustainability in Higher Education
www.aashe.org | bob.erb@aashe.org | (888) 347-9997 ext 35

--
Join us in Baltimore, Maryland from Oct. 9 - 12 for the AASHE 2016
Conference & Expo, themed Beyond the Campus. Call for proposals and
reviewers now open <https://ww4.aievolution.com/ash1601>!

Connect with us:   Website <http://www.aashe.org/> | Facebook
<https://www.facebook.com/aasheorg> | Twitter
<https://twitter.com/AASHENews> | LinkedIn
<http://www.linkedin.com/company/aashe>

------------------------------
Email Confidentiality Notice
<http://www.aashe.org/email_confidentiality_notice>


2245154-NewVRFHRModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: Tests Failed (1667 of 1986 tests passed, 0 test warnings)

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


Are you  looking at latest fix up commit?


happy to :)


I'll submit a PR shortly with the proper fix, and then I'll remove this change from this PR.


Ok cool, will do. Thanks again.


Good point.


SQLConnection objekt ćeš instancirati unutar funkcije u kalsi DAL. Znači poanta je da otvoriš konekciju na bazi, odradiš što trebaš i odmah zatvoriš...


It doesn't hurt to not have it around either ;)


Oh, this is memory error! Try running the code trough valgrind.

You are accessing 'tmp', after memory it points to was freed (in line 66)


This is fixing delete button to remove right row, i added '#name' => 'remove' . $uuid, and now $form_state->getTriggeringElement() gets element is clicked and before that is get just last element. Also i remove some unused variables.


kompletno obriši autoDataSet.xsd To ti ne treba. Nakon onoga primjera što sam ti poslao, to ti ne treba.


`@student_view_result = @assessment_settings.student_view_result && @user_assessment.assessment_results.by_last_final.any?`


Aha no - looks like I've just spent the last hour getting to the root of this while you had already fixed it. :)


Windows 10 IoT Core is supposed to be ARM only, right? I think `process.platform = 'ARM'` and `process.engine = 'chakra` should suffice in that case?


For the moment!  I'm editing it now and when I'm done, I'll put it in the
internal repository.

Best,
Cezar.

On Tue, 19 Jan 2016, Adam Sandberg Eriksson wrote:

>
> It seems the other file is missing!
>
> —
> Reply to this email directly or view it on GitHub.[ACm-uvdWR3PfKqFjdbbYDngr1SrTuyrfks5pbpw3gaJpZM4HIFQ6.gif]
>
>
>


Sometimes I wonder why I don’t already have `rake rubocop` as a pre-commit hook in my local copy…


This is a bad solution in terms of complexity, triple loop `O(n3)`. I don't think Google will consider this. Because this is a last resort.
There should be better `O(n)`, or `O(n.logn)` solutions


Namespaces should not change


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


No. You could have x86 and x64 arch binary for node.js. However as I said, since this PR would be targeting ChakraCore and not chakra.dll it would be better to remove condition for chakra.


emad


$ npm install

Install for development

$ git clone git@github.com/ksidedev/react-fundermentals.git
$ cd spark-products-lib
$ npm install
$ npm start


I did mention it on forum...


I don't really like all the `Eliom_lib.poly` I introduced in this commit. But I don't see how to do any better.


Can we revert this? There is no 1.5.0 and people are showing up in IRC confused.


@glyn Fundamental issue here in that you're using the "un-validated" version of the request.  Once a value has entered the flow, all subsequent references to it should be from the flow itself.  I'll push a branch with an example of how you might do that.

/cc @twoseat


> You aren't going to remove the clickgui, aren't you?

Not in v2.13, that's just a myth. Maybe in v3.0 or v4.0.

> Because it is still very useful in enabling mods quickly.

Nope, also a myth. Navigator is actually much faster, as you don't have to manually search the mod you want to toggle in a giant mess of buttons that just gets bigger and messier with every update. Instead, Navigator gives you a nicely sorted list of buttons that will learn your preferences and put your most used mods right at the top so that you won't have to look around that much in order to find them.

And if you want to toggle something you don't use very often, then typing the first few letters of its name into a search bar is still much faster than looking trough the entire list until you find it (searching it manually takes [up to 46s](https://twitter.com/Wurst_Imperium/status/684366755993616384), typing 3 characters takes [only 2s](https://twitter.com/Wurst_Imperium/status/684377212204748800)).

After you found it, it's only one click to toggle it, no matter if you use ClickGUI or Navigator. Other actions, such as managing keybinds, moving sliders and changing modes, are faster and easier with Navigator than they are with ClickGUI.


Here i added '#name' => remove . $uuid parameter to remove button and now $form_state->getTriggeringElement() gets clicked  widget button, before this is always get last added widget . Also i remove some unused variables.


Sure!


So you did (haven't been on there much today).


> Whats missing that is preventing it from being 2.13?

Small things, I might even get it done this week.


I've gone ahead and done this in the most recent iteration of my "broker" branch. See: https://github.com/PulpQE/pulp-smash/compare/master...Ichimonji10:broker


x1==x2 И y1==y2 ну в таком случае по смыслу фигура не сдвинулась с места


Implementing Serializable was the main point of this PR, but it seems it got undone in the second commit..?


Is it worth it to have a Csc overload that takes a `BuildPaths`?


:+1: as always


This one should also use StringUtils, for consistency.


https://xkcd.com/859/ Please ;-)


I only used Arrays.copyOf(T[], int) because http://www.gwtproject.org/doc/latest/RefJreEmulation.html#Package_java_util says it is implemented in the current version. In vanilla Java you can't do `new T [60]` either if I remember right, since T might not have a public constructor.


This will use extra data (DNS roundtrip) and battery, needs to be stick'd behind a toggle.


Oh, alright. I support reducing it to a single name. The intricacy of `chakracore` vs `chakra` might not be obvious to an outsider, so I'd prefer it just to be `chakra`, but no strong opinion.


digits ))


Great job!


fast-forwarding master to auto = fd69662c


While this technically works, it's not fully parameterized.  Read the "Better yet, use parameterized messages" subsection of the following link:

http://www.slf4j.org/faq.html#logging_performance


Hi @hawkowl my current view on this is: the "API" of Crossbar.io, that is the stuff that users may rely on and where guaranteed stability is a goal is:

1. command line arguments to the `crossbar` invocation
2. static node configuration format
3. WAMP meta API

Everything else is private and can change anytime. In particular, log **messages** (but see below).

Putting "tools" on top of log output that parse that log output is wrong IMO, and I don't want to _encourage_ that practice by trying to keep log messages constant.

Obviously, we can't technically stop people from doing that, as much as one can e.g. directly import CB internal Python classes - but I won't spend time on that.

That being said, I think what @meejah hinted at might be something that could reconcile our views.

Let's take a concrete example.

The REST bridge bails out on a HTTP request that isn't of content type `application/json` with a HTTP status code 400, and a response body that is a string which describes the error.

https://github.com/crossbario/crossbar/blob/master/crossbar/adapter/rest/common.py#L101

A unit test should check for that HTTP status code, and the specific response body. As that HTTP response body is "API", it definitely should not change (between CB minor versions at least).

However, the _formatted_ **log message** produced by the REST bridge for this case for one is only logged at level "debug" (which makes sense), and is not "API": https://github.com/crossbario/crossbar/blob/master/crossbar/adapter/rest/common.py#L99

Assume that code would log like this:

```python
self.log.debug("[request denied] - {code} - {reason}", code=code, reason=reason)
```

Then all the relevant info (but see below) that does indeed constitute "API" would be forwarded to structured log.

However, even when doing so, that _should still succeed_ if I changed the line to

```python
self.log.debug("HTTP request denied with status code {code} ({reason})", code=code, reason=reason)
```

In other words, nothing should rely on the formatted message.

But the structured log information (code + reason) isn't enough.

How about this?

```python
self.log.debug("Error {error_id}: HTTP request denied with status code {code} ({reason})", error_id, code=code, reason=reason)
```

where `error_id` would be an integer ID that enumerates _all_ the different errors/warning etc that can happen in Crossbar.io.

So given a log observer that only looks at the structured log information (error_id, code, reason), but *not* the formatted message, a unit test could then check for specific log output without relying on the formatting of the message (which is only for human consumption).

What do you think?


It is not that you are looking to the log view all day long. As you know I want to keep the number of options limited, but maybe it is useful, so please create an issue for this, so that I don't forget about it.


is `utf8output+` legal?


Maybe mirror the ordering with the parameters?


Ok figured it out, the PRs order is chronological so I was commenting on the oldest one :/ Well now I know


There is an other problem with actual firmware (v1.0.0-rc6) kernel driver it does not manage new commands
see https://github.com/airspy/firmware/blob/master/common/airspy_commands.h
actual kernel driver manage only up to CMD_SET_PACKING = 0x13, which is wrong as in new firmware
0x13 corresponds to AIRSPY_MS_VENDOR_CMD
missing commands are:
AIRSPY_SET_RF_BIAS_CMD            = 20,
AIRSPY_GPIO_WRITE                 = 21,
AIRSPY_GPIO_READ                  = 22,
AIRSPY_GPIODIR_WRITE              = 23,
AIRSPY_GPIODIR_READ               = 24,
AIRSPY_GET_SAMPLERATES            = 25,
AIRSPY_SET_PACKING = 26


Will do.


опять деления )


emad


Pretty sure.  Good catch.  Will add a check for that.


demote


I think when things get this large you should replace with a method

ng-if="ubicacion == 'main.views.certificates_error' || ubicacion == 'main.views.certificates_labor' || ubicacion == 'main.views.certificates_vacations' || ubicacion == 'main.views.certificates_payroll' || ubicacion == 'main.views.certificates_income'"

should be something like...

ng-if="certificatesRoute(ubicacion)"


Actually that is why I was asking if "it worths PR" as I didn't look that much capabilities of dynamic... My bad, sorry :).
Anyway, that's great, I didn't know we could that!


modlist


Is Console.Out guaranteed to be reference equal with itself?


help


And what about performances?


setowner


Hmm, I think we can probably improve the public interface here.

What do you think of:

``` js
exports.app = app
```

As the only export for now.


Yes, the return value is the returned object.


Second commit, added project files.


@bmbouter Thank *you* for the feedback.


commands list


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


creategroup


kick


Based on the implementation it's guaranteed to be so long as there are no intervening calls to `SetOut`.  I couldn't find anything in the documentation stating if this is guaranteed to be true by design though.


id


I think this broke the build.


Please consider validating the message.  This could pass even if you just have a test issue.


ok pour le pull


Not debug code: the method IDs are part of the protocol. While protobuf can describe RPC calls, it doesn't define the serialization for making the actual call. So Blizzard defined an extension for specifying which method is called. That extension must be defined for `protoc` to be handle to handle the method ID fields.


idmember


Typo here


help


@sum2012 the "ABI" line under CPU in system info should show x86 or x86-64 accordingly


Hey Brother,

I kept the three files in a folder..So XGB gets imported but does not run..
I keep getting the Attribute error: module object has no DMatrix attribute.

Is it because __init__.py is not there with proper initializations?

Thanks


Usualy in gulp plugins we do something like this:

```js
exports default function myPlugin() {
  return through.obj(function(file, encoding, callback) {
    file.contents = new Buffer(doSomethingWithTheFile(file.contetns));
    callback(null, file);
    // or
    // this.push(file);
  });
};
```

We do this to pass along transformed/cloned files. You may read more about this in [gulp API docs](https://github.com/gulpjs/gulp/blob/master/docs/writing-a-plugin/README.md#modifying-file-content).

So now I ask how can I do this? How can I get transformed content from `cli.files` and convert it back to `Vinyl`?

BTW, where can I get log message  which will be produced by linter, for example? Previously, we do this using `log`, but now, as I understood, I should get it from `VFile.messages`, right?


Lo veo bien!!


This means we would stop using the container infra in Travis. Have you measured the build times? I think we could keep installing the packages the way we currently are.


emad


This the second and the last one, @domenic?


Great f2fs update. Thanks master


looks like the logfile comes from [the backup script](https://github.com/blueboxgroup/ursula/pull/1519/files#diff-d88754be6840d3b61804137a9c7688adR24)


Does it make sense to just set the topics using the ROS parameter server? That way any launch file can run this node and choose the topic names.


Everything here needs to be changed too?


Do we want a message here?


+8 - я догадываюсь, зачем это, но сомневаюсь, что оно работает


No. Everywhere where you have changed http://www.w3.org/1999/xhtml, needs to be un-changed. There are also other namespaces, e.g. around line 8055, 9363, and more.


Glede na to, da je datoteki ime `uvoz_zemljevidov.R` (velika črka za končnico), to uporabi tudi tukaj, da bo program deloval na vseh sistemih.


Similar to what @oberstet is suggesting, what I usually do when I "have" to check log messages (e.g. I do recall adding some tests where I assert on flushed log messages to confirm the error-handling "stuff" will actually cause an error to be logged) is to check for some minimal string -- that is, not the whole exact error message but with the above example may something like ``assert "request denied" in errmsg`` so, it's far less likely to fail on formatting changes etc while still capturing the essence of the test ("some sort of error was logged when I did the Bad Thing").

Nothing's really perfect here -- error messages can change, etc. and I basically agree with both of you ;) namely that mucking about in unit-tests looking for exact phrases sucks but also that (often) the existence of some informative log message *is* very important.

The error-id thing sounds pretty decent to me. If we called the kwarg ``cb_error_id`` or similar, perhaps ultimately we could update tests like this to simply look in the logger's event dict for the right arg + code...? As I understand it this would allow other log observers to reliably look for these, too (and even people reduced to ``grep``-ing text logs could look for the codes if need-be...)


Oh, that is fantastic to hear! I will look into that tonight.


este crane es el ejecutable que no debería subirse al github, por eso lo tenía en el .gitignore para no verlo cuando hago git status


Gemaakt met animateturret en animatehull 0 (geforceerde armor plates dus). Oranje eventuele prop, maar zijtekst word gespiegeld.


Why did we remove stderr?


saw approval from macintux
at https://github.com/basho/lager/commit/0fb72d7744d6c45f1ab3af31a866c587b71d81be


merging basho/lager/gh267-fix = 0fb72d77 into borshop-integration-306-gh267-fix


basho/lager/gh267-fix = 0fb72d77 merged ok, testing candidate = e8348964


saw approval from macintux
at https://github.com/basho/lager/commit/9ee64cd717a4191172060cfe4b538a51669f0259


It looks lIke Arrays.copyOf(T[], int) was added in GWT 2.7.0 and was not in 2.6.1 at all. 2.7.0 is over 12 months old, I think we can be safe using it if it's compatible with LibGDX.


merging basho/lager/gh267-2.x = 9ee64cd7 into borshop-integration-308-gh267-2.x


basho/lager/gh267-2.x = 9ee64cd7 merged ok, testing candidate = 17835c79


saw approval from macintux
at https://github.com/basho/lager/commit/cf4e4e504958d6bc3d5d9be891023dbff0cad2b0


merging basho/lager/goldrush-018 = cf4e4e50 into borshop-integration-313-goldrush-018


basho/lager/goldrush-018 = cf4e4e50 merged ok, testing candidate = 10fe4ffb


I didn't know the clean part of it, trying with that now :)


в-первых, молодец с массивом
во-вторых, нули лишние, можно без них
в-тертьих, всё равно не понятно, зачем последний нуль?


This commit breaks accordions if ng-class is used on the same element as panel-class....

A workaround could be to remove the ng-class and put it wrapped on a parent instead, but this seems wrong. I want to add an ng-class to the same element as a uib-accordion-group with a panel-class on it...

I will work on a fix for this breaking change.


Glad to see this button debugged :)


Looks good.

I would still add a sentence
```
B<pkeyutl> does not invoke specified digest function.
```

Better safe than sorry, and IMHO it's far better to repeat or rephrase something when it adds to the likelihood of being understood properly, than being brief and terse, crediting the reader with more intelligence than one ought to.


Hi,

As an aside "https://github.com/dmlc/xgboost/tree/master/windows" is not available. I needed to install xgb on windows but cant find those files..

Can I know where to find those files?


а на трёхзначность кто проверять будет?


you should probably just rename the args newEngineHost, isInvalidEngineHostname as opposed to commenting


see new instruction here https://xgboost.readthedocs.org/en/latest/build.html


this is my fork of the repo. please go to https://github.com/dmlc/xgboost for latest version


this was an upstream change.. this commit needs to have the previous line in it.


should be from 1 to 8, а ты проверяешь от 0 до 8


On my system (64bit Linux AVX), your implementation is more than twice as fast as dynamic_1023.


this was an upstream change.. this commit needs to have the previous lines in it.


Hmm.. hard for me to speculate without being able to reproduce.. Is there any way I can get more information about where the exception comes from? Haven't used travis before, and don't know how to reproduce the same environment as it has..


static


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


Yeah i removed that plugin. Because it did not do what I expected, Did not remove entities at all and i dont want all entities to be removed :) Just fun to have a villager standing next to a farm or other enitiies inside a building :)


would probably rather leave this alone for this commit.


I'm OK with merging this, however the applicationDataSpec no longer exists. Can you merge from master?


What changed here?


these two lines need to be removed (part of an upstream change)


No reason to be sorry about that... It is my fault that I haven't have tested it


It was good when my server was griefed with zombies :)


- maybe rename commit to "implemented network-watchdog"


@jfoug dynamic_1029 is SHA256... this format is SHA1. I guess you mean 1023.

Also:
```
$ ../run/john -test -form:raw-sha1-axcrypt
Benchmarking: Raw-SHA1-AxCrypt [SHA1 128/128 AVX 4x]... DONE
Raw:	17765K c/s real, 17943K c/s virtual

$ ../run/john -test -form:dynamic_1023
Benchmarking: dynamic_1023 [sha1($p) (hash truncated to length 32) 128/128 AVX 4x1]... DONE
Raw:	7706K c/s real, 7784K c/s virtual
```

BTW if we keep this format, it should be renamed from "raw-sha1-axcrypt" to "axcrypt-sha1". I missed that on my quick review.


haven't looked at this in a while. It seems as though signed_url might be what we want, based on
http://docs.aws.amazon.com/AmazonS3/latest/dev/ShareObjectPreSignedURL.html
and
https://github.com/sorentwo/carrierwave-aws/blob/master/lib/carrierwave/storage/aws_file.rb#L49


Ah, wow. Can't believe I didn't catch that.


Will add one.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1759 of 2088 tests passed, 0 test warnings)

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


The compiler never writes to standard error.  Even error messages are written to standard out. In the build protocol the error messages entry is always empty.


directly


Ok


have a look at the "version" crate (and its issues).


This seem like a pretty global change -- doesn't this mean we will never run tests in any time zone other than UTC?


hey :) these are pretty great, in case you haven't stumbled upon them already: https://egghead.io/series/getting-started-with-redux


Random blank line?


Should return `Mono<Void>`.


We normally write "bytecode" as one word (I don't know why, but we do).


Superb!
Excelent work!


Well, I guess there is a trade-off code modularity/performances which is inevitable... And this particular case will probably happen again.

May be this worth removing "axcrypt-sha1" as I don't know if this software is that used as the developper claims.

Up to you guys :)


it was as if thousands of voices cried out in unison "sync me! sync me!" and where then silenced.

FALSE

the death star is faster than a human can respond, there were no voices.

old obi won Kenobi is craaaaaazy.

btw Kenobi auto-capped. GG Microsoft.


woopsie


I would suggest
1) we change the implementation to return a single value
2) we test for the boundary values of the vector only

Best,

Fabien

On 19 janv. 2016, at 21:36, Sam Yates <notifications@github.com<mailto:notifications@github.com>> wrote:

I think we need to rethink the get_propensity() method: passing all the values back via a std::vector is fairly heavyweight, and it's not necessarily easy to provide all the propensities for any SSA selector implementation.

I'd rather modify the API to support a propensity() method that takes a key_type patameter and returns the corresponding propensity, or an operation_not_supported exception (at the discretion of the implementation.)

I'll merge the pull request and submit some proposed amendments as a separate request.

—
Reply to this email directly or view it on GitHub<https://github.com/delalond/rdmini/commit/5f4b60569ad5e705ab45269712c15a414f9242ca#commitcomment-15540608>.


@xavierlu , instead of returning false, try returning isTimedOut().


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/317


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/318


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/319


saw approval from macintux
at https://github.com/rabbitmq/lager/commit/31a5fc5b133ced8b17ef5caa0d3fe2dc7d0df431


merging rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b into borshop-integration-312-close-file-after-timebased-rotation


rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b merged ok, testing candidate = cbea55bf


saw approval from macintux
at https://github.com/basho/lager/commit/93d401a2c335dafa50ce223f1556348b5742c5d2


merging basho/lager/goldrush-018-2.x = 93d401a2 into borshop-integration-314-goldrush-018-2.x


basho/lager/goldrush-018-2.x = 93d401a2 merged ok, testing candidate = 0aabe8df


Does it work with `()` here instead of `None` (the empty tuple)?  That's the standard way to implement an immutable collection.


Rather than making someone potentially just delete all these and re-add them to enable-disable. Should a true/false to enable/disable. In my opinion at least... As well as listing the array obviously to allow selection tinker..


is this already working? it really interests me


Todo bien!!


THIS WAS THE ISSUE


This looks unintentional.


#for adding remote server
git remote add origin git@github.com:qatrainingca/PyAssignments.git


I see nothing unreasonable here!


I guess I'm wondering if there are existing spring boot apps, out in the wild, with more than one `HttpMessageConverter` which are working correctly without this change.

This doesn't quite make sense. We call `fireObjectMapperConfiguredEvent(each.getObjectMapper());` for each one. Should we really just find the "last" `HttpMessageConverter `, when there is more than one and fire that event once?


I'm not sure why, but this is barfing all over when you try to compile it for Black Mesa Source on Linux.  In the C++ standard library, no less.


проверка - это хорошо, да
но если Please enter, то всё остальное уже делать, наверное, не надо? :)


Just like the *accessors* for `bypassLinkingErrors` in the Config objects, we can't deprecate them now, because we need them ourselves. We can only deprecate them when we entirely remove support, at which point, arguably, we might as well just remove them, because we'll break compatibility again (depends on whether we have become more strict about Tools compatibility by then).


solved in baa3213


%s


No need for this (and I don't want these interfaces to start pretending to be API objects).


Thanks, I think the change we implemented in #517 enables this extra functionality optionally based on a configuration specified in the aws.Config. This was the only way we could enable the verbose output error listing without creating a breaking change.


Same as I commented on SpongeCommon's sister PR, I don't want these to be pretend API objects.


It's not like I'm the one starting this: https://github.com/SpongePowered/SpongeCommon/blob/master/src/main/java/org/spongepowered/common/interfaces/entity/IMixinEntity.java#L37
I use it somewhere in SpongeVanilla for something, let me check.


+1

Same fix already in NDIS 6.x code.


Ah well, not a very important one, I can revert that


BTW, attachment string needs to be full file path.
I.E: "file:///storage/emulated/0/Pictures/63884caa-167a-477a-9a65-f5beb99345a6.jpg"


Alyssa, would you prefer to present the first or second half of the key points?


Hi!

I had a similar thought! I've just created a pull request with changes to
the API document, and corresponding implementation. If you're happy with
the changes, we can merge it.

Cheers,
Sam

On 19 January 2016 at 23:00, fabien delalondre <notifications@github.com>
wrote:

> I would suggest
> 1) we change the implementation to return a single value
> 2) we test for the boundary values of the vector only
>
> Best,
>
> Fabien
>
>
>
>
> On 19 janv. 2016, at 21:36, Sam Yates <notifications@github.com<mailto:
> notifications@github.com>> wrote:
>
>
> I think we need to rethink the get_propensity() method: passing all the
> values back via a std::vector is fairly heavyweight, and it's not
> necessarily easy to provide all the propensities for any SSA selector
> implementation.
>
> I'd rather modify the API to support a propensity() method that takes a
> key_type patameter and returns the corresponding propensity, or an
> operation_not_supported exception (at the discretion of the implementation.)
>
> I'll merge the pull request and submit some proposed amendments as a
> separate request.
>
> —
> Reply to this email directly or view it on GitHub<
> https://github.com/delalond/rdmini/commit/5f4b60569ad5e705ab45269712c15a414f9242ca#commitcomment-15540608
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/delalond/rdmini/commit/5f4b60569ad5e705ab45269712c15a414f9242ca#commitcomment-15542539>
> .
>


I believe this was named `getSpongeData` to reduce any chance of method interference with other transformations from mods/forge.


Nit: trailing whitespace here and throughout.

You might want to enable both "Automatically trim trailing whitespace" and "Include whitespace-only lines" in the Xcode > Text Editing settings.


будет время - перепиши на switch-case, посмотри, будет ли меньше кода, удобнее писать


The exact error is coming from string.h:

```In file included from /home/tf2/dev/sourcemod-extensions/sourcemod-sendproxy-manager/extension/extension.cpp:34:
In file included from /home/tf2/dev/sourcemod-extensions/sourcemod-sendproxy-manager/extension/extension.h:44:
In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/string:40:
In file included from /usr/bin/../lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/bits/char_traits.h:39:
/usr/bin/../lib/gcc/x86_64-linux-gnu/4.8/../../../../include/c++/4.8/bits/stl_algobase.h:193:9: error: expected unqualified-id
    min(const _Tp& __a, const _Tp& __b)
        ^
/home/tf2/dev/hl2sdk-sdk2013/public/minmax.h:12:22: note: expanded from macro 'min'
#define min(a,b)  (((a) < (b)) ? (a) : (b))
                     ^
```


Credit: @heistrouble


Remove this change


The only reason why `IMixinEntity` had to extend `Entity` was to properly handle the similar methods being override.


@VeraZab Awesome yeah! Thanks for those, they look awesome!

This repo is just a playground to mess around with a few things and just have a boilerplate scaffold out to quickly iterate!

I'll take a more in depth look at those tuts fo sho!


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


https://github.com/alexander-daniel/redux-learning/tree/complete

This branch just followed the official redux documentation (which I found really clear and concise!) and just built out the basics.


What is the rationale for removing the `<p>` here? Thanks!


see @tmf, you *can* animate with `height: auto;` ;) Just set a `max-height` that's over-the-top. I didn't remember I was doing this...


аналогично ))


I added this additionally (as `getEntityData()` exists identically on SpongeVanilla and SpongeForge), `getSpongeData()` is implemented below as a default method


Thanks,

Will give it a try now and see if things work out..


Could you remove the references to `EmbeddedObject` in other comments in this file?


Ye, well reverted it now - no problem


What is the rationale for removing the <p> here? Thanks!


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


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


You got me wrong, not directly after view change - after playing song that changes whole queue... But still that's imho annoying for me, solution might be adding toggle in settings for that.


да ))


This import is not used (and missing on Android)


Fixed!


urllib2 is already imported- no need to extract the module


lmao much love 😁


@r0i ,

Agradeceriamos tanto @cmorlok como yo que se respetase la historia y autoría del repo mediante un fork.


Addresses and closes #5.


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


As long as this format is 2x faster than dynamic I think we should keep it.


ок )


Yes, I can see that, but in short, if any mod/Forge adds `getEntityData(): NBTTagCompoun`, we're kinda conflicting methods there.


AIUI, the string passed here is used to generate the URL, so we can't change the string without affecting the URL (as written, anyway).

I'd recommend making `linkForTopic` a private method in `SupportUtils`, then defining all topic URLs as constants on `SupportUtils`; e.g.:
```
public static var AdjustURL = linkForTopic("adjust")
```

That way, the magic strings like "adjust" are kept internal to the utils class, all in one place.


Please, can you use a fork of the origin project?

To keep the history & authorship of the repo.


да, ок


I'm going to remove this commit when I squash. We determined it wasn't really relevant to the RPC calls that these changes are making. You might want to leave this comment on @seelabs original commit: https://github.com/seelabs/rippled/commit/2a275053a727a3e7c1691fb50eb0aff42554d29d


Sooner or later we need to accept the modern times :) Ok for me!


Forge has `getEntityData()` already, that's the whole point of it (it is automatically implemented in Forge)


Can you also set the timestamp field on each envelope? Here and elsewhere.


Added expose-loader to be able to expose single modules via require/import


@hrydgard my phone cpu is Intel Z3560 , I checked the net that it is supported 64 bit.
My phone is running under Android 5.0
but ppsspp show 32 bit


OK the thing is the dynamic format support up to length 110 (2 digests) as opposed to 5 (1 digest). But 55 is enough. I presume we could write a slightly different dynamic that only does 55 and is on par with this speed.


Valid point, I had thought about that too, there are existing boot apps that use springfox that either
- are living with this bug (prior to this fix)
- or have figured out how to parry the number of ObjectMapper in play to just one.
- or are using spring-hateoas (which comes with its own object mapper) and have figured out how to use that instead.

All of those cases will benefit from this fix. Of course, these are based on just the questions I've been fielding so thats totally not representative.

About just using the "last" one, this class is just a bean post processor and fires events out when it finds object mappers. Its not making assumptions of who is consuming the events. Net effect of this change is to provide ordering (priority).

Subsequently provide extensibility hook that will, in effect, perform the role of only publishing out the "last" rather "best" ObjectMapper.  My initial thought is that we'd provide a hook in the docket to provide an intelligent ObjectMapper selector, some kind of predicate based selector.

thots? _FYI_ this is not released so we can still stop the press!


It looks like the log message is setting the timestamp on the wrong field (line [87](https://github.com/jberkhahn/loggregator_emitter/commit/7aaa7dfced62f2bdcd067281c0bb1c548f8006d8#diff-03f7ecc30fc413fc7cb1666277fc7626R87)). It should be `lm.timestamp` not `lm.time`. Do you mind fixing that also in this PR?


como que te regreso el comentario


`link` -> `URL`


Small typo on line 18 of this file (for some reason I can't comment there). Should be "available" instead of "avaialble"


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


Consider expanding test coverage to include the logic that changes the state of this new property.


fast-forwarding master to borshop-integration-306-gh267-fix = e8348964


fast-forwarding 2.x to borshop-integration-308-gh267-2.x = 17835c79


Merge sha 10fe4ffb is stale.


merging basho/lager/goldrush-018 = cf4e4e50 into borshop-integration-313-goldrush-018


basho/lager/goldrush-018 = cf4e4e50 merged ok, testing candidate = 7d2b04c1


Merge sha cbea55bf is stale.


merging rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b into borshop-integration-312-close-file-after-timebased-rotation


rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b merged ok, testing candidate = e485a4e9


Merge sha 0aabe8df is stale.


merging basho/lager/goldrush-018-2.x = 93d401a2 into borshop-integration-314-goldrush-018-2.x


basho/lager/goldrush-018-2.x = 93d401a2 merged ok, testing candidate = d9bed3e8


saw approval from macintux
at https://github.com/basho/lager/commit/66869e2d637bc51b4382f820f682f0453ce29dec


merging basho/lager/gh296-fix = 66869e2d into borshop-integration-307-gh296-fix


basho/lager/gh296-fix = 66869e2d merged ok, testing candidate = 3dad2b59


Хитрое решение, не сразу понял ))
В любом случае - палиндром, а не полином, но это придирки ))


I feel like we should just return an error to avoid an unexpected write


Is moving this interface into the `interfaces.entity` package out-of-scope? :princess:


This would potentially allow a by-pass of the ACLs too:
* Allow write to secret/foo/*
* Write to secret/foo/ results in write to secret/foo which is not permitted.


I don't think it would hurt to force the duration into an integer by default, as "200". For example, `isNaN("200")` is actually `false`, so a string "200" would not be converted in the code as it is. The `time || 0` check is awesome, and should remove any chance of a `null` value which is also considered a number in JavaScript, so this should work:

```javascript
var dur = parseInt(time || 0, 10);
this.delay = isNaN(dur) ? this.defaultDelay : dur;
```


ок


I will just remove then.. thanks for the feedback.


Possibly


Same concern here, we should just prevent the write


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


Maybe:
is2.write(reinterpret_cast<const char*>(data_), bytes_recvd)


We usually have a nice description in case of reflection failing with NMS.


Thank you for unicode caring :)


тут надо ||


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


Could you potentially eliminate this intermediate state if both `doneButtonTapped:` and `didPanWithGestureRecognizer` called through to a common function, hypothetically something like `dismissViewControllerAnimated:isUserInitiated:completion:`, passing `YES` for `isUserInitiated`, while `dismissViewControllerAnimated:completion:` calls through to the common method with `NO`?


I followed the instructions on the Troubleshooting page and filed Issue #48264 .


This section is redundant. Externalize `frequency`:
```js
function frequency // float -> middleware
module.exports = frequency(.6)
module.exports.frequency = frequency;
```


@nurse Nice, I always wondered what was the failure here!
I am not sure to fully understand though, why is the FOO variable not preserved?


If I want to enable this overlay, would it work before I recompile mesa and libdri?


Kan gewoon :joy:


D'après le père même de la typographie mathématique et de TeX, il faut :p http://tex.loria.fr/typographie/mathwriting.pdf


Fixed


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


Ship it!


add rubocop and it will check this for you :stuck_out_tongue:


Here are my benchmarks 110 vs. 55 using dynamic and axcrypt-sha1:

    $ john -test -format=dynamic_1023
    Benchmarking: dynamic_1023 [sha1($p) (hash truncated to length 32) 128/128 SSE2 4x1]... DONE
    Raw:	    5253K c/s real, 5253K c/s virtual

    $ john -test -format=dynamic_1023
    Benchmarking: dynamic_1023 [sha1($p) (hash truncated to length 32) 128/128 SSE2 4x1]... DONE
    Raw:	    9525K c/s real, 9525K c/s virtual

    $ john -test -format=raw-sha1-axcrypt
    Benchmarking: Raw-SHA1-AxCrypt [SHA1 128/128 SSE2 4x]... DONE
    Raw:	    12795K c/s real, 12795K c/s virtual


Yeah just noticed that and making alterations later when I'm not editing in Github :)


I see you done that.


@gabrielrussell I reported TOOLS-1039. Are there tests for these? This was a regression from 2.4 through 2.6, 3.0 and 3.2.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.97](https://pixelated.design/build/viewLog.html?buildId=334&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:21


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.94](https://pixelated.design/build/viewLog.html?buildId=335&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 267 Build time: 00:00:18


Why not private?
Also "inner" isn't really a good name. In Type inner is the innermost logic, while this is technically also that, it is confusing.
It shouldn't be used, so "fallback" or "default" would be a better name. I prefer "fallback".


@imzyxwvu  If you have fbturbo enabled, your pi will lock up. Compiled mesa and libdri will be in apt soon (already are in staging). I would recommend waiting for a blog post or something of the sort before playing around with this.


ok


Do not forget to adjust the comment above the change.


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


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

```


@scruffian This slipped by me when reviewing your PR (sorry). This change adds a lot of delay to opening a purchase. What do you think about either adding a destination type for when the card details are loaded, or adding an action to set `selectedPurchase.hasLoadedFromServer` to false after they are updated?


we need to treat replication controllers and deployment configs the same.. so DC needs to be cased in here as well.


RC and DC should be treated similarly.


Oh, merged at this point a few hours ago, also.


fast-forwarding master to borshop-integration-313-goldrush-018 = 7d2b04c1


Merge sha 3dad2b59 is stale.


merging basho/lager/gh296-fix = 66869e2d into borshop-integration-307-gh296-fix


basho/lager/gh296-fix = 66869e2d merged ok, testing candidate = 0076f317


Merge sha e485a4e9 is stale.


merging rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b into borshop-integration-312-close-file-after-timebased-rotation


rabbitmq/lager/close-file-after-timebased-rotation = 31a5fc5b merged ok, testing candidate = ccc0a5c3


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/326


@keszybz I do not understand why you set the offset to MHD_VERSION instead of 0


* Moves list of everyone's most-used alias to `/clientlist`
* Fixes bug of `/whois` not working on people who are still connecting.


Perhaps a comment explaining these fields. NMS is not always straight forward. What is CB and what is NMS?
Also what does "dot" mean, do we use that elsewhere?


Yes! Now I checked the whole history of this tween methods hahaha great it seems like they are going from hand to hand haha


Instead of recording the entire history, declare here the attributes of that history that you feel are important enough to test, and test only those? This seems similar to a golden-file test in that it may inhibit progress on unrelated code that would change this history in a benign way.


It took me a long time to understand why this has any effect at all; I didn't realize that Insist(focusPlan) has side-effects, it looked like an Assert() to me. It's not this patch's fault, but while we're at it, I would suggest replacing those Insist() calls with plain:

if (!focusPlan(plan, true, false))
   elog(ERROR, "could not mark plan as stable");

or something along those lines.


These seem to be the same for the if and the else clauses. Was that intentional?


The NMS methods does a null check. The MUtil ones does not. Symmetry please.


Ah, I see! Thanks for the explanation. :)


This could be simplified to `if capabilities.get("marionette"):`


Yes, Redux docs are really good! Say, I don't see any Redux in the Plotly/atelier repo. I guess it's because you guys started experimenting with Redux just more recently? Are you still just testing it out or pretty certain to widely implement it at Plotly? :)


Nice pointer. Thank you.


I've never understood why the testnet params have remained in chainparams.cpp for so long.

Perhaps partitioning these off into their own testnet_chainparams.cpp would be a worthwhile job after the hard fork.


indentation issue


@gorenc I would suggest to wait for the official patch. If you want to test it now - you can download the whole dev package (don't use it for production or live site) - https://github.com/boonex/dolphin.pro/archive/master.zip


This last note is redundant. If it was added in 1.8 it was obviously not there in 1.7.


Change `Javascript` to `JavaScript`.


Thank you, @girving .

It might be, if `collection` is used. But, in the overall context used in `tensorflows` (`word2vec.forward()`,  `fixed_unigram_candidate_sampler()`, and `_fixed_unigram_candidate_sampler()`), `list` is needed consistently. As you know, `tuple` is different from `list`. If we use `()` there, that breaks the consistency and might confuse users sometimes.

By the way, I have thought the above style is the standard way since it's in https://docs.python.org/2/tutorial/controlflow.html . Could you give me some pointers that you referenced? Thank you in advance.


What's wrong with this commented tests ?


Will do tomorrow. I'm done for today, worked for this almost 7 hrs

On Wed, Jan 20, 2016, 00:22 Antoine Bluchet <notifications@github.com>
wrote:

> Lot to review, I'm on it tomorrow first thing.
>
> Could you add some comments?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/estliberitas/PM2/commit/d394bddbbe9622be3f72605d398bf89bcb93655e#commitcomment-15541664>
> .
>
--
Sincerely,
Alexander Makarenko


This could be simplified to `if not value:` since we expect either a string or None.


haha, maybe consider an error message more explicit :)


:scream:


@swergas Fair enough, mais il y a encore un:
 isCollapsed = this.model.get('isOpen') === false ? true : false;
à la ligne 521, et personne n'appelle set('isOpen') où que ce soit maintenant.


@Kra1o5 esto es solo de prueba, lo añadi para ver los avances. En llegar al finde ya realizare el fork eliminando este repo.


We are looking at ways to remove this weird incrementally populated segment list all together.  Since we load one at a time we can reduce a ton of complexity in this class. Maybe we can just pull in for now but would be nice to add line notes in code of why are do this then loop.  Or look at making it a bit cleaner.  Have some ideas if looking.


Ignore that! I didn't notice the case change!


thanks, but unfortunately this typo can't be fixed without some hackery or re-installing libCEC on Windows, so this one can't be merged


Perhaps the "click behind" bug is caused because we never call session_start()

http://php.net/manual/en/function.session-start.php


spurious change


This line broke the Keras build. `ImportError: No module named 'pool'`.


error message more explicit :)
Also maybe de-duplicate this code in its own function to avoid copy/pasting errors?


Context won't load - invalid properties.


Ok, gracias @r0i .

Un saludo


Perfecto! :+1:


in acitivity.xml im pretty sure this causes docs to crash. Keeps crashing on me not sure if its the same for you. look at the system report and within the logcat it spit something out about the background color and toolbar

-            android:background="?android:attr/colorPrimary"
 +        android:background="@color/toolbar_background_color"


动态注册域名空间在init.php里做就可以了吧？


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


Estás seguro de que quieres conectarlo a test.mosquito.org (`ip: 69.172.201.107`) en lugar de a [ws://localhost:8000](ws://localhost:8000) donde está corriendo tu broker? ;)


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


En dan haal je bijv. behaviours research op bij de betreffende 'goal' die onder de competentie research valt. Hopelijk is het een beetje duidelijk..


Why are these lines changed?

```js

                    remove: strings['remove'],
                    titleName: strings['name'],
                    titleValue: strings['value'],
                    valueName: name,
                    valueValue: value
```


This isn't really an acceptable solution. By putting this here you are forcing a traversal of the entire input configuration where none existed before, which has a very serious performance cost. I am working with configurations in size on the order of hundreds of megabytes and even a single traversal of the entire term is too much cost to pay if it's avoidable. Indeed much of my recent work on the K infrastructure has been spent making sure that the entire configuration is traversed *fewer* times. I am pretty sure the right place to deal with this is during parsing anyway... you want a way to add KVariables to the syntax of programs, really, is what is at issue.

We can address this after the deadline if need be, but it does need to be addressed.


all tests pass:
success: http://buildbot.bos1/builders/test-lager/builds/329


some tests failed:
failure: http://buildbot.bos1/builders/test-lager/builds/330


This change is not reflected in Django 1.8.8, I had to make the change manually


Alright I'm not entirely sure about this one, I actually liked the option to through slides very fast, especially when doing it only on the Presenter window with 'G'. Do you really accidentally scroll easily? Plus we don't actually use any `ScrollWindow` in pympress, do we?


@aleph7 can you check this? If it looks good I'll add the rotation method to C4ShapeLayer, C4ImageLayer, etc.


What about fields? This isn't exactly a cache.


Justo eso es lo que tengo puesto ahora mismo, pero se ve que no hice el commit después de cambiarlo de donde lo saqué


Some formatting =)


"Get" or "get entity" could also work.


I'm curious what this actually changes?


Didn't  we agree to not use the hour thing on Massive Core?


nailed it


We should probably not bump the `view` event for this case.


Added to review 21

On Tue, Jan 19, 2016 at 5:13 PM, Chao Xu <notifications@github.com> wrote:

> Could you remove the references to EmbeddedObject in other comments in
> this file?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smarterclayton/kubernetes/commit/cb5a27604011b6afd170119295a540efb7b6a98d#commitcomment-15542790>
> .
>


#TODO yakışıklı olmadı


I wonder how this works


I like the simplicity.


ultimately here we need to decide whether to tailor cffi towards ffi's written by hand or generated ffi's. if it's the former then this may not be desirable.


This smooths out the arrow a fair bit more on my device.


"through", not "though" :)


LGTM


LGTM. Committed as 76776e46a2ccd3ca94d43525b746a8cec641cf99.


testing


Can you just kill off the modernize target, while you are here?


Pull requests welcome if you guys want to do more than what was being done here. I don't have much time to actively work on it


that was the normal GIT generator

guess it is fine the way github generates this stuff


And then you'd have to make another one for regtest_chainparams, and you'd have to make sure that each file that included the old chainparams.h would include all three chainparams, etc. It's better as one file.


Why is the default scale 3.14?  Can you add a doc string explaining?


This seems like a not-too-graceful way to exclude files. I don't think we need to fix it now, but long term I'd like something better.


Minor: update the comment in line 176 since we are not passing in a ref now as input.


Yeah, scapy is kinda weird. Super-versatile, though.


...and also using a webpack plugin for ng annotations instead of gulp plugin.


This should  be JavascriptConversion::ToNumber. ToFloat returns float (single precision) instead of double (double precision).


The usual Rails approach for this would look like:

```ruby
def create
  @topic = Topic.find(params[:topic_id])
  @sponsored_post = @topic.sponsored_posts.build(sponsored_post_params)
  if @sponsored_post.save
    ...
  end
end

private

def sponsored_post_params
  params[:sponsored_post].permit(:body, :price, :title)
end
```


If the file in .workers just contains a string, not a yaml dict mapping, then config is a string (truthy), not a dict.


Could you comment on this change GoogleFrog?
I am curious.


I think we are going to have to abandon ujson. https://github.com/esnme/ultrajson/issues/69 suggests this issue is fixed by master but based on my tests on various machines and python versions, this fix does not work majority of the time.


Thanks.


Why are measure and fail declared as virtual?  My understanding is that all algorithms should inherit from SimpleAlgorithm.  Do you expect subclasses of ScaledApertureFluxAlgorithm?


@Muleling cool dude!


This is wierd

![image](https://cloud.githubusercontent.com/assets/532272/12434876/87fa055a-bebe-11e5-81ca-1c82dafbdc26.png)


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 125](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=693&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


1023 is slower, mostly due to the flat buffers, vs the tuned loading of the mixed buffer.   But even a 'mixed buffer' SHA1 in dynamic is slower than 'raw-sha1'.  Not as bad as 2x slower, but still slower (20% or more likely).


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


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


I think we should always return something.  In the case of multiple sources, why not return an array of the sources?  For failure, we can leave it undefined, I suppose.


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


The ignore array should be passed to the Reporter no? I don't see this argument used in applyEmbeddedRuleset. If I pass it to Reporter (below on line 197) then this works.


Just go with `1.0`


Seems kind of hack-y. I can't say what happens if for instance you disable animations. Or you enable autorepeat. Why didn't the `actionFotKey` approach work? What I would do is in the animation generated in `actionForKey` if the rotation goes over π or -π then break it down into separate animations. We may need a custom CAAnimation subclass for this.


var expected = ['main.views.certificates_error','main.views.certificates_labor','main.views.certificates_vacations','main.views.certificates_payroll','main.views.certificates_income'];

 certificatesRouteNew = function (ubication){
     var tryout = expected.indexOf(ubication);
     return tryout > -1 ? true : false;
 }

 console.log(certificatesRouteNew("main.views.certificates_payrol"));


Oh no!  What happened??  I was pretty excited about this update!  :)


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


I understand the concern and was a little hesitating about `dmenu`, but I thought that if a user enable `i3` it is more user friendly to provide the default expected functionality.
The [i3 manual](https://i3wm.org/docs/userguide.html#_opening_other_applications) states that "Aside from opening applications from a terminal, you can also use the handy dmenu which is opened by pressing $mod+d by default.".
Shall we remove `dmenu` from the list or find another way like a dedicated option to allow user to not install it?


We've found security issues that are very hard to fix. It is being discussed on IRC right now.


Cool, if we are confident it won't break those apps with this edge case it should be fine.

Regarding the events being published via `ApplicationEventPublisher` I think I'd prefer if this class `ObjectMapperConfigurer` only ever published one event per initialisation - the `ObjectMapper`. If you look at how it's being used:
https://github.com/springfox/springfox/blob/709f77e64a214d3a632a66b863c73b5fca9a8dbb/springfox-schema/src/main/java/springfox/documentation/schema/property/ObjectMapperBeanPropertyNamingStrategy.java#L83-L83 we are relying on the last event re-assigning `this.objectMapper`. Really `ObjectMapperConfigurer` is about getting or creating the single ObjectMapper being used in the application.


So, we should be expecting something similar to the following for the output:

Module Name: `foo`
```json
{
  "fooTemplate": {}
}
```


Legal :+1:


autocomplet?


Definitely. Hard to describe why the `actionForKey` wasn't working... It just wasn't... It might have to do with having a property called "rotation" and then setting the key value to "transform.rotation.z"... In between, the animation doesn't know either where to get the current rotation value (i.e. i can't access "rotation" from the layer, and vice-versa).

Also the +/- π won't for animations that use curves other than .Linear

I can try and recreate the non-functioning actionForKey example I created and push that as a separate branch for you to look at.


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

```


Done


This is probably the right fix. Mumble likes to send packets with some information (and all other fields null, indicating no information in that field). So this is *probably* caused by the server sending information about parent but leaving name unspecified (i.e. null). It's probably worth digging into packet decoding to check though.


PlantCompBaseClassAgain (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed (1727 of 2056 tests passed, 0 test warnings)

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


I'm able to compile v7.4.1142, but v7.4.1143 fails to compile complaining about:

```
gcc -c -I. -Iproto -DHAVE_CONFIG_H     -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1       -o objects/pathdef.o auto/pathdef.c
ex_cmds.c: In function ‘ex_sort’:
ex_cmds.c:560:48: error: ‘DBL_MAX’ undeclared (first use in this function)
       nrs[lnum - eap->line1].st_u.value_flt = -DBL_MAX;
                                                ^
ex_cmds.c:560:48: note: each undeclared identifier is reported only once for each function it appears in
Makefile:2652: recipe for target 'objects/ex_cmds.o' failed
```


Gobo images with transparency aren't really a thing... this is boring


True.


Why is this comment here if it's not needed?


Also, I think it breaks down because the rotation property is custom (i.e. not defined in Core Animation, like `backgroundColor`, `strokeColor`, etc.)


What's the policy on documentation for common algorithm components?  It's not helpful to replicate documentation in all the algorithms, but it would also be nice to easily get to a description of what the various classes and methods do.


in acitivity.xml im pretty sure this causes docs to crash. Keeps crashing on me not sure if its the same for you. look at the system report and within the logcat it spit something out about the background color and toolbar


@littledan Check e7a3ca3d for reference of why `subarray()` was used.

@LinusU The Buffer API is going under some revision ATM, so passing the parameters in this fashion would be possible. Though now I wonder how the v8 change affects the native `node::Buffer::New()` API since it uses `Object::SetPrototypeOf()`.


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2089 tests passed, 0 test warnings)

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


These concepts are grouped in one file at the beginning, to give to contributors an idea of the scope of the project.


I'm fairly concerned with this naming, because of https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIViewController_Class/#//apple_ref/occ/instp/UIViewController/presentingViewController

can you rename this method to something more detailed, or less similar than what I've linked?


Valentyn was try to be funny


Not sure if I like testing winston directly. But you are stating in the test that it should use winston, so I feel better about it


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


indent your functions closing brackets.


@garbas Is this okay? I haven't proceeded with an update because of https://github.com/dialelo/turses/commit/dde099be534d891a79133e1ae15e84353ce99ea4


I was trying to be funny


nested: 'Dialogs can be nested. This example opens a Date Picker from within a Dialog.',


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


I get the following when building www:

/home/www/src/datavyu-docs/source/api/reference/class/CTable.txt:1: WARNING: Title overline too short.

===========
CTable Class
===========
looking for now-outdated files... none found
pickling environment... done
checking consistency... /home/www/src/datavyu-docs/source/api/reference/class/CTable.txt:: WARNING: document isn't included in any toctree
/home/www/src/datavyu-docs/source/api/reference/computeKappa.txt:: WARNING: document isn't included in any toctree
/home/www/src/datavyu-docs/source/api/reference/deleteVariable.txt:: WARNING: document isn't included in any toctree


Open a Date Picker dialog from within a dialog.


Let me know what you guys think of this, but it seems to me what we're getting at is that maps as a "diagram of forces" compared to trees and graphs shows how the form of a novel is more strongly determined by the relationships within the novel.   While Cerretti has a point that Moretti's "maps" here seem to be more geometric diagrams, I see no reason why his diagrams cannot be placed on top of the geographic maps.  Accordingly, while graphs may show broad patterns and trees are used well within the intimacy of technique, maps are the only one of the three modes, at least as illustrated by Moretti, that shows the internal construction of the novel in itself.  In so doing, it provides an image of the movement and dynamism that is integral for the novel's life, a fact we may tend to lose when we are too close to the text.  The problem is then like the one Moretti begins with at the beginning of his chapter, a problem of perspective.  Like a villager experiencing life of his or her village from the center, or the gentry seeing it from a far, only linear relationships of narrative emerge to us when we are too much on the ground.  It is only through the use of maps that the geometric and geographic form of the text emerges dramatically and clearly and that we may reinstate the internal movement of the novel in full.


hintText="Date Picker" />


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1508 of 1508 tests passed, 0 test warnings)

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


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


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


title="Nested dialogs"


```::Rails.application.config.paths['config/database'].any?``` already checks File.exist? https://github.com/rails/rails/blob/56dc95bc92301dae89e5d6b5ca306b043b095661/railties/lib/rails/paths.rb#L200-L202


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


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


oops, typo, already fixed in last commit


This problem has 5 subproblems all of which are needed to get full score.
1. Your public key has to be in pgp.mit.edu and searchable.
2. Your public key should verify the signed data you pasted.
3. Your public key should be registered with your student email you used to register to the grader.
4. Your public key should be signed (trusted) by at least one person (not yourself).
5. Your public key should have an expiration date, any date would do.
6. Your public key should be 4096 bits long (not 2048 which is the GPG default).


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


After more testing, I had to bump my <TargetFrameworkVersion> to v5.0 and not v6.0... Strange but looks like it is working for now. v4.4 and v6.0 thrown the TypeLoadException, where as v5.0 is appearing to be fine.
But there is still some inconsistencies between Android projects in mvvmcross repo, not sure it was intentional :)


Honestly, this was supposed to fall into the end of body of our main piece but I'm still struggling to figure out how to physically do that....


혹시나 deleteAll 문제는 이쪽 아닐지 보면 될듯~


Stephen, your comparison is great. It compares maps to graphs and trees really well, while also reinforcing the key points.


Coming along nicely :-)
Looking forward to a pull request!


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


Could prb get away without statics here.  Also, does this need to be a pointer?  Creating an instance of mraa::I2c would minimize the chance of pointer mishandling later.


        I would like to be in contact and collaborate with anyone using the Ecobee3 Thermostat and Keen Smart Vents to achieve energy savings better than either one alone,   And I would also like to interact with parties who have experienced "some failures", so I can have as few as possible "been-there-done that" learning experiences.
        Finally, I am going to throw in a unique set-up that Keen SmartVents has not yet tried.  I have three ducts 24 x 6+".  I will be trying to use two vents side-by-side in the wider ducts..  Making these vents "manual control" with the Ecobee moderating temperature in select rooms.  Ideally I hope we can find an algorithm that utilizes the Ecobee and Smart Vents to focus only on rooms in use indicated by three or four remote monitors.  And if tying to Weatherbug or external monitors actually can be integrated, that would exceed expectations.
        I look forward to working with you.


Well, you're both right. If it turns out that the atom is :nul but the fourth l of "null" is in protected memory, it could segfault even though strncmp would still try to read it. I'll add the length checks.


Missing a comma.


На самом желе правильно не "sTaskID" а "snID_Task" (если это всеже стринга, несущая в себе намбер ИД-шника таски), но а идеале nID_Task и при этом сделать ее лонг-ом


@mention @dias


Try make a commit on the cm gerrit @ review.cyanogenmod.org .. The devs don't usually merge pull requests


Needed to add various cpp files from WDL to make this compile in XCode. Visual Studio project not updated yet.


Awesome!


Fala mestre, blz?
Desculpe qlqr coisa é o único lugar q achei pra me comunicar.

Primeiramente parabéns, muito bom esse projeto, instalei funcionou bem. Mas, queria ver se vc consegue me ajudar numa parte aqui.

É o seguinte... tenho um projeto aqui que coloquei essa parte de auditoria, como faço para puxar os relacionamentos que ficam nos campos "old_value" e "new_value", por exemplo, fiz a auditoria do model "User" e nesse model tem uma chave estrangeira 'group_id', então como faço para exibir na listagem dos logs o nome do relacionamento, ou seja, nome = Administrador (referente ao group_id).

Não sei se vai dar pra entender.

Obrigado!


That’s not quite the same as what’s there currently.  `if capabilities.get("marionette"):` tests that the value, if it’s there, evaluates to something truthy.

`if "marionette" in capabilities and capabilities["marionette"] is True` tests for boolean equivalence.


Parameter-less constructor declared but not defined.


This defeats the whole purpose of autoboot: Booting without decrypting and patching the firm. :T


@trevnorris It looks like the old code called out to the three-argument constructor from C++ code. I wonder if calling the TypedArray constructor from JavaScript might be faster. Did anyone try that?


Won't this always be true?


Thanks!  Addressed in 9e8d764ef9984e1f6092f4110417b71ffb9ed992.


I need this mock to create RECORD on line 42. If I used @BeforeClass instead of static initialization, I could not do that. If I moved RECORD to the @BeforeClass method too, I could not make it final.


I agree @dwightguth, but combining *custom* variables with program syntax right now may open a whole can of parsing warms. Let's leave it for after the deadline.


Done.


As a matter of strictness, your code would make sense except that you should use `== True` rather than `is True`.
I think if the specification is that marionette is a bool, a simple if test should suffice; otherwise if other values are allowed, then your code would make sense because now you're implying other values are acceptable.


SMF uses spaces around concatenation: ' . $some_variable . ' rather than '.$member_id.'

The code looks good, though.


@blangel If not using spying, verifying AbstractLambdaFileHandler will fail. So how to deal with this? Using PowerMockito?


Removed.


Afraid it looks like we are unable to reach consensus on a solution.


No. I verified with the service that this is not mandatory and no initialization is needed. Removed.


Good point. Refactored.


ASIHPdev (shenb) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1121 of 1986 tests passed, 0 test warnings)

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


Gone. See above.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 125](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=693&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:36:47


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Here are the 4 pertenant format speeds on my system (XOP laptop).

```
Benchmarking: dynamic_26 [sha1($p) raw-sha1 128/128 XOP 4x1]... DONE
Raw:    8619K c/s real, 8676K c/s virtual

$ ../run/john -test=3 -form=dynamic_1023
Benchmarking: dynamic_1023 [sha1($p) (hash truncated to length 32) 128/128 XOP 4x1]... DONE
Raw:    4834K c/s real, 4825K c/s virtual

$ ../run/john -test=3 -form=raw-sha1
Benchmarking: Raw-SHA1 [SHA1 128/128 XOP 4x]... DONE
Raw:    12013K c/s real, 12030K c/s virtual

$ ../run/john -test=3 -form=raw-sha1-axcrypt
Benchmarking: Raw-SHA1-AxCrypt [SHA1 128/128 XOP 4x]... DONE
Raw:    11275K c/s real, 11301K c/s virtual
```

I was not able to get dyna_26 working for a 32 byte hash (like I did with the flat buffers).  I am getting a cmp(1) failure, so likely there is some buffer cleaning that is failing or something.  BUT, it should work at same speed as dyna_26.   Like I said, it still is noticeably slower than the hand coded raw-sha1 format.  A large part is in the optimized buffer loading, AND in the amount of overhead in general of dynamic. For slower formats, that overhead is not nearly as noticeable as in the fast formats. Also,, for SHA1, we do not crypt into our final buffer, so there is an 'extra' buffer copy for each crypt.  We copy the data into the final 16 byte buffer for each crypt when complete.  For iterative crypts, it is not that bad.  For a 'raw' crypt, you get the max hit. The only 2 hash that dump right into the final buffer are md4 and md5.


the indentation is still going to 4.. 😤


For now yes, the code is written so I can comment out the jvm+= line.


Agreed. Refactored.


Dead code?


Done.


This method hangs. The thread pool thread status is WAIT.


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


True. Refactored.


I don’t know the reason why it was like that before.


Why is this tied to fts_diskio_check guc? And why is this check performed as part of backend initialization? I would assume it to be periodic, like a cron job.

(I have to wonder why you wouldn't just use a cron job for this whole thing, anyway..)


unused?


Missing $ in the variable


Instead of point checking for negative or infinity here, can you do the following:

double numberLength = JavascriptConversion::ToNumber(firstArgument, scriptContext);
int64 byteLength = JavascriptConversion::ToLength(firstArgument, scriptContext);

if (byteLength != numberLength)
{

}

Also, add a test for NaN, -0, 0 and make sure it does what is intended.

This follows the algorithm outlined here: http://tc39.github.io/ecma262/#sec-arraybuffer-length

Extra kudos if you could get the Firefox, Chrome compatibility here. What do they do for this? In some cases, we prioritize the compatibility over other browsers to spec compliance.


At least use `== True` so that it compares values and not object identities.


i'll fix the rest tomorrow


@johncrossman , shouldn't params['uid] be uid_param in this case?


Leave these in the Travis config


I thought about doing that when I first put these tests in.  Then these tests caught a mistake that caused many extraneous SQL queries to come through which would have caused a serious performance regression -- and would have been hard to catch without a clear idea of exactly what queries are actually coming through.

I also feel that it is important for someone who substantially changes the set of queries that are executed by basic BQL operations to understand the consequences, and these traces do exactly that.  Maintaining the traces is not hard: py.test will tell you what changed when something changes, and that serves as a sanity check to make sure your change really is benign.  So I'm reluctant to change this now, having already derived considerable value from it.

That said, I would be happy to condense some of the repetition by writing

    assert foo = [
        ...
    ] + [...]*4 + [
        ...
    ]

since one section of this trace is executed once per row in the input table.


oooh! I like :)


ASIHPdev (shenb) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


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


dead


I know my print functions suck, but come on. This should just be a `print(array[nandcount]);`


Add a brief note to the commit message explaining the reason for this change? (I'm not sure why this is here, is why I raise the issue.)


下载及安装 改为 下载与安装；基本用法 改为 基本使用


TeamCity pull requests :: netty [Build 1061](https://garage.netty.io/teamcity/viewLog.html?buildId=4882&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Compilation error: Compiler (new) Build time: 00:02:15


111768812_VRFNoLoadOAFlow (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


Is this the only way? :-)


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


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


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


Open an issue against me to fix this as we should directly test this with unit tests.


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


True. I've fixed it and will push that soon!