please rebase instead of merging, that gets rid of the unnecessary commit


LGTM


How does this secret + HMAC security mechanism fit with the OAuth security model we typically use everywhere else in Abacus and CF in general?


Great API. Thanks for sharing.


Hmm how is this implementation better than the previous one?


sharedInstance這方法的回傳值是instancetype, instancetype可以是任意型別,類似id,但是id可以當成是回傳的型別和參數使用, instancetype只能是回傳型別. (?)


cmd


Was wondering why aren't you using a third party api like ZBarScanner, Zbar or some other 3rd party library.


Found it ! I had been searching in a stale Emacs buffer :-(


What did you mean by integration here?


please rebase/fixup this into the previous commit (`git rebase -i HEAD~2`, set this one to `fixup`)


Attention here.


> Also you can add `import objectAssign from 'object-assign';`

I like Object.assign better for the tests. I think `Object.assign` will always be available in the tests and `object-assign` adds another (dev)dependency.


This whole family based decision tree sucks. RHEL 7 is systemd, Debian Jessie is systemd.


oh I appreciate it!

On Mon, Feb 29, 2016 at 10:10 AM, Les Orchard <notifications@github.com>
wrote:

> I was kind of jumping the gun commenting here on the commit before it's a
> PR, but I was curious how this was working out & it looks cool so far +1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/meandavejustice/testpilot/commit/8f75c2dcacbdd966efa50fd09d83a7c43535b70e#commitcomment-16396241>
> .
>


@dejakob Awesome, sorry about the mistake and thanks for the patience :)


Is passing a secret to a publisher really secure? Usually secrets are not passed around right?


The following doc(s) have been deployed to the [1404upgrad](http://1404upgrad-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/settings/upgrade-php-versions](http://1404upgrad-static-docs.pantheon.io/docs/articles/sites/settings/upgrade-php-versions)
- [docs/articles/sites/secure-phpinfo](http://1404upgrad-static-docs.pantheon.io/docs/articles/sites/secure-phpinfo)


`RemoveMarkup` isn't a great name if the method is actually just getting the non-generic name for the class


Consider reminding readers that `std::bind` knows how to do the right thing when `this` is a `shared_ptr`. Example:
```
acceptor.async_accept(...,
    std::bind(&my_class::accept_handler,
        shared_from_this(), std::placeholders::_1));
```


When the page doesn't register one, history, wanted, etc. Since the view and this logic are separated we could probably have separate implementations if we wanted to, one for series details (where we have the handler) and another where we have the episode file attached, though if we're dropping subtypes we would want a way to retrieve them as well.


Hmmm.  It looks like `quality()` only gets called twice: once with `qin` true, and once with `qin` false.  that suggests to me that the code factorization is not quite right, although I don't have any immediate suggestions.  Should you pass in the lambda that chooses `Field` instead of `qin`?  I'm unsure.  Think about it.


I'm not familiar with debug info in LTO binary. But traversing all CUs to find the abstract_origin seems very expensive. And also, is it the right thing to do? Is it possible that two CUs have the same key but different value?


Ok, now I get it. So, in theory, if the App would support more advanced/extensive scheduling, then it would be unnecessary to run multiple instances, right?

What do you think of adding another setting for those notifications? That way, people using multiple instances could simply not enable unlocked by unknown?

In any case, I will keep my fork then, as I really want to see the manual unlocked.


This is a post, so it feels a bit strange to call this function publish instead of post (the comment above the publish function actually says 'Post' though).


@mjkallen nope! because pdflatex with error out with various return codes and still pass! handy huh? :)

@elfring we had no feedback on why running with 3 or 5 passes was required. we could theoretically expose an option with our upcoming `readthedocs.yaml` changes, but running this by default adds significant overhead to our build load, to only address an edge case.

If there is anything worth bringing up to the core team, issues is the correct way to do it. We have little visibility into comments on commits.


This makes sense, but I'd prefer more descriptive field name.
Payload sounds like something related to image content (RPM payload = cpio archive with files).

The examples you provides are Fedora spins.
Don't know if 'spin' is a good name either.
Do you have an idea what could be a good name of a compose/release building block under $release / $variant ?


This does not seem right. We need the Cleanup otherwise the memory will bloat for large binaries. Could you suggest a less expensive solution?


I think we could simplify the User model's code for selecting pantry items and DRY out the controller logic a bit, by using an ActiveRecord Scope which takes a parameter for the public/private/personal distinction.


This is the core/html.py file depending (as in "having knowledge of") two other plugins.


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


Always wondered how the ``` (comment)``` block should be used.
I've seen it being used as a "sandpit" to try things out and do repl driven development,
and I've also seen it being used to show example code.

Wondered what your thoughts are.


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


There should be a way to pass options here, like with the 'request' or 'abacus-request' modules. Actually I'd go even further and say that this function should look exactly that 'request', except that it'll take a list of uris instead of a single uri.


Should not be better to add `SOL` prefix ?


Nice to hear :smile:


This commit broke OS X builds - DBL_MAX is defined in cfloat.h. The following patch to include cfloat.h fixed the compilation issue on OS X.

```
diff --git a/src/Mod/Inspection/Gui/VisualInspection.cpp b/src/Mod/Inspection/Gui/VisualInspection.cpp
index 66850e3..d0a0a61 100644
--- a/src/Mod/Inspection/Gui/VisualInspection.cpp
+++ b/src/Mod/Inspection/Gui/VisualInspection.cpp
@@ -35,6 +35,8 @@
 #include <Gui/MainWindow.h>
 #include <Gui/PrefWidgets.h>

+#include <cfloat>
+
 using namespace InspectionGui;

 namespace InspectionGui {
```


It is an interesting project. In this project, your goal is to quantify music mixdown and find the theme. How do you define whether a mixdown is successful or not?  Since each song is unique and combined with different instruments and voices. Do you want to construct a mixing standard or paradigm for different categories of songs, such as country music and pop music?


$this->options() is een onnodige function call, hier kan je een private class member van maken.


:+1:  @hlee Thanks!


@mattab, you did review before and said everything is fine except test. Please do not change you mind later. Tomorrow we need to finish sprint and we don't have a time for another change. Moreover, I don't see a place where plugin can break login for regular user (see [line](https://github.com/wronan/plugin-LoginLdap/commit/1a15400d60c84bee8cef3dcbe4aa3cd84506b1b9#diff-70e5942b723451ed07b4d3c42c9dcb1aR197) ) could you elaborate?


@wsdjeg It must not do this. `:edit term//` must behave just like any other `:edit` command and that does not change to any mode, including terminal one, unless you created some hack to do this. Insert mode is not available in terminal buffers.


Dit moet volgens de style van het framework needsRehash zijn


Can you just fix this up so it's right in the previous commit?


что температура забыла в функции для форматирования времени?


I don't entirely understand all of the plumbing for this yet, but this reads like a pretty big code smell to me... artificially constructing timestamps seems like it would open the door for more new race conditions...


How do you think about to [use another software build system like "Latexmk"](http://users.phys.psu.edu/~collins/software/latexmk/ "Running LaTeX the correct number of times") for a while?


Both syntax types for declaring an array variable are correct, therefore no warnings during compiling.
But, your style is more preferred. The "AreaMeasurable figures[]..." - works but not preferred way.

Changed


I don't think that this module should do any retries, throttling or use circuit breakers, as that'll prevent its users to actually do that (or not) the way they want. This is similar to the request module, which lets its users decide however they want to retry, throttle or handle errors and circuit breaking.


abma; I presume you refer to ownerExcpAccWeight (not defInterceptType) https://springrts.com/mantis/view.php?id=3310


'payload' was the best name I managed to come up with. in fedfind I actually had two types of 'payload': there was 'flavor' (which was Server, Workstation or Cloud - the official Fedora flavors...this was called 'product' until Red Hat decided Fedora should call them 'flavors' instead of 'products') and then there was 'loadout', for non-product-y images. 'payload' was the property which would give you the 'flavor' or the 'loadout' for the image, if you just wanted "what does this image deploy" and you didn't care whether it was a flavor or a loadout.

This is also how it is in the [Fedora naming policy I wrote](https://fedoraproject.org/wiki/User:Adamwill/Draft_fedora_image_naming_policy) - that talks about 'PRODUCT' (because I never bothered to update it to say 'FLAVOR' instead) and 'LOADOUT'.

I agree it's not a great name, but it's hard to come up with anything better, because it's a slightly vague property, but one we really need. The best way I've managed to conceptualize it is "what bits does this image deploy", but even that isn't quite universal, and doesn't obviously imply a snappy name.

So I don't mind if you want to change it, but I don't have any better ideas, unless you like 'loadout' more. :)


вообще эта функция лишняя.


how about prefixing with "firebase-", to be consistent with other events in the element set.


asd


I was thinking you could only make it fire if there was a new commit on a day there wasn't already an APK update triggering, like

if [ $(newapkcheck) == 1 -o $(newcommitcheck == 1 ];

Still would only fire once per day tops. But yeah, you'd have to figure out how to do that check. :+1:


должна быть функция которая возвращает DateFormat


I think I fixed it last night

On Monday, February 29, 2016, Colin Schmidt <notifications@github.com>
wrote:

> Yes this is an issue. I had named the top level script lower case and
> tested 32G and 32I. Fixing it soon as I get a moment.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/riscv/riscv-tests/commit/0a4d7624d35b3a42ae0945d447f7c7e0a012cf82#commitcomment-16394264>
> .
>


I'd suggest to use the abacus-transform module for this like we do everywhere else in the Abacus codebase


I don't know what your policy is on metadata versioning, btw. This *does* constitute an incompatible change, at least as I understand it - formerly-valid metadata would now not validate any more since there wouldn't be any 'payload' fields in it. So it's a disruptive change. I don't know how you want to handle those, if we just do them and not care about previous composes, or we re-generate the metadata for old composes, or you were planning to support validating multiple metadata versions, or what...

and I guess we should at least check with Dennis and see if there are any other similarly non-compatible changes we want to make and do them all at once.


important


Suggest rename: sighashbyteslimit --> sighashlimit

Otherwise, ACK.


I don't quite like this change, I play the game with sounds off and sometimes take a screenshot, that message provides me valuable information which is now gone, you haven't even provided any log in place for that.

Another quite common way of notifying user of a screenshot being taken is a brief "flash", how about that? We already have night/day cycle which could possibly be used for that.


Here I'm assuming there's no supported releases neither.


See my earlier comment on security


Absolutely, that could be something worth looking into. Please open ideas for enhancement as issues, I was only able to respond here because I have a stroke of availability right now, otherwise this conversation would be lost.


Loading ans saving model with Siamese layer is working now
Before: Mismatch between trainable layer and non-trainable layer


It's not that complicated at all, but I honestly had my doubts something like this would be suitable for the program, since it dramatically increases import times and can make Blender slow. It's especially this option that makes me anxious to look into a way to avoid importing the same bricks over and over again, since it adds so much to the import time. But anyway, if you like it enough, I'll surely put it in a PR. I imagine it'd be useful to have a warning with option that it makes importing slower (personally, I split my bigger models in two parts: one with all of the bricks with hidden studs, and one with all of the bricks with exposed studs, and import them separately). Also, we' have to figure out what we want to do with the extra unofficial LDraw file. Right now, I have it stored together with the other primitives, but I imagine you'll want to store it together with the rest of the script's files.


You may have changed this in a later commit, but the threading macro seems to make this less readable IMO


A safe way would be to use ``var_export(..., true)`` (which will handle the quotes for you btw)


The second test should be `$self->compara_dba` only because if it's defined, `->dbc` is defined too, and if it's not defined, `->dbc` will fail


Change let to const, as this variable is immutable.


DateTime то есть. другая функа должна по шаблону форматировать текст. а на вход принемать DateTime. а выколупывание дня, года вообще выкинуть. оно нам не надо. там где надо это можно руками вызвать.


I was looking at nunit3-console runner with `--explore` and found that `<` and `>` in test names do show up properly using the console runner, but not on XS.


no, I'll fix it


Headers should be initialized the same way as body, there's no reason to use two different initialization styles for body and headers.


:bath:


So then it will just load in the models that we need, yes? i.e. the ones in all.r . Sounds good.


If message is actually a body (like shown below) I'd suggest to name it body.


Alright. I had to make some other changes to tests to get an I only
toolchain to work so I'll rebase off your commit
On Feb 29, 2016 10:36 AM, "Andrew Waterman" <notifications@github.com>
wrote:

> I think I fixed it last night
>
> On Monday, February 29, 2016, Colin Schmidt <notifications@github.com>
> wrote:
>
> > Yes this is an issue. I had named the top level script lower case and
> > tested 32G and 32I. Fixing it soon as I get a moment.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/riscv/riscv-tests/commit/0a4d7624d35b3a42ae0945d447f7c7e0a012cf82#commitcomment-16394264
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/riscv/riscv-tests/commit/0a4d7624d35b3a42ae0945d447f7c7e0a012cf82#commitcomment-16397033>
> .
>


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - x86_64-MacOS-10.9-clang: OK (2120 of 2120 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)


Both good points :) Not sure that showing the type of the `TestCase` argument is of any use though.


Sorry it just he said he gonna make email.php i he maked me very happy and now i just hopped he made it '-'


No particular reason, I just thought Google's Mobile Vision API is newer and since it's officially maintained by them I just expected to be easier to integrate, and less buggy than other libs, although I didn't really do any research to prove these things.

In any case it didn't seem to matter much because barcode scanning is such a niche use-case I doubt I'll need to deal with it ever again, so I just wanted to get it done as quickly as possible.


@Ved-vampir instead of having multiple attributes for the different compression params, define a structure that can be encoded / decoded, and that includes all the needed properties, and set it on a single attr.


if martinos is the only center with one of these, I am +1 for deleting it all together. thoughts?

reasons:
It would be messy to do for every site
it encourages people to go to the wrong place (if you are looking for a managed tool at your site, you should check with people at your site first)
Can't think of the others

Just thought I would add while you are in here.


$multifurcations should be an arrayref, not a hashref, since the key is an integer starting from 0


`return -EIO;`


I don't see a reason to make that variable mutable


* Do you call the tool "pdflatex" twice still today?
* Are there any further constraints to consider?


These functions (`list_*` from list.h)  aren't threadsafe as far as I know --- I think there is a case where an LKL thread is executing `syscall_thread` and a host thread calls `lkl_syscall`, and the call to `list_for_each_entry_safe` is racy. It might also be possible that two concurrent calls to `syscall_thread` race against each other, since those threads aren't under control of LKL's scheduler (?)


`if (cr < 0) `


`return cr;`


Hey Josh, I'm seeing a ConcurrentModificationException that we weren't seeing earlier...it appears as though a thread from the model update is trying to modify image items in the view while the map controller also does the same...seen any of this before?


執行父類別的init這個方法, 所以這裡是執行NSObject的init方法, 然後會回傳instancetype給self...
但是if裡面是判斷布林值...self到底是...?
為什麼要這樣？


Well, feel free to implement that. Its important that when you take two screenshots, the second doesn't capture anything that shows you took a previous screenshot. This is what currently happens with the error box that pops up.


Thanks! I will remove it now


Oh and if you're wondering why I wrote so much code by myself, the answer is that I didn't. Almost all of this code was copied verbatim from the sample app. Google should really have bundled all this low-level code within the library itself, it would've been *so* much easier then.


If we just had some additional logic here that checked to see if `last-updated-before-tag-computed` was `nil`, isn't there a way we could solve this without having to do all of this synthetic cache population and fake timestamps and such?


Actually, it now only loads in the ones we list in all.r. So this is really just to keep it tidier, but isn't necessary, yes?


Might be worth checking Erin's XRP balance here.


:-1:
I do not see the point in setting the service to the same value as component
worse, any custom settings for the "service" dimension are now overridden. This does not make sense for any generic component (mysql, apache, rabbitmq, ...) since these usually belong to at least one (openstack-)service


exactly! :)


D'oh yeah not sure why I didn't think of that. I'll fix the fix ;)


Yes, twice still. Our primary constraint is build time, additional runs of pdflatex compound time and memory footprint greatly.


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1012 of 1012 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-29.44%25-green.svg)


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2126 of 2126 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)


\s+


pourquoi y a un "?" a la fin ?


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Might be worth checking Carol's USD and XRP balances.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fYnX1eniTCq2aT9grTAWfg


Are we talking human controlled as in player or human controlled as in species in-game? I think it's the former but am not 100% sure reading the code.


Take for example this scenario - http://screencast.com/t/Fpu22P3oK8Q3

whereas in nunit3-console it shows

    $ mono Manual/NunitTests/packages/NUnit.Console.3.0.1/tools/nunit3-console.exe Manual/NunitTests/Nunit3Tests/bin/Debug/Nunit3Tests.dll | grep -i "<>"
    => Nunit3Tests.TestNunit3TestCase.Dean_Road<>False


Je mettrais ces lignes directement dans l'initializer de ton handler


I don't think this was carried forward correctly, but I recall the rules here being a bit odd when I last ran into this with Hiro and Nahi so I could be wrong. The problem is, on Linux at least, `/tmp` is both writable and world_writable. So, it's skipped in this list and then `.` trivially passes in most cases. MRI handles that case by also checking the sticky bit, which is set for `/tmp`. I'd patch it, but I don't recall exactly how this would break. As it is now, however, this change generally doesn't work because it makes the tmpdir resolve to `.`, which will probably create problems for people elsewhere.


That's a good point. You can leave it as is, or change it to `newDecimalType` per @ilfrin's suggestion.


If we are about to deadlock and we have no way to recover, we should just panic, IMO


Code may be modified somewhere else (for an instance the check on the stream reading or the stream reading itself may be removed) and you will finish with uninitialized variable. Uninitialized variable are bad.


Continuous integration in just the second commit! And implemented as a one-liner! Ingenious stuff.


Might be worth checking her XRP balance as well.


saw approval from JeetKunDoug
at https://github.com/basho/riak_core/commit/6be14e578fcceafccc254259930d5982d8ca4ffd


merging basho/riak_core/basho/mra/bump_exometer_core_1.0.0-basho5 = 6be14e57 into borshop-integration-817-basho/mra/bump_exometer_core_1.0.0-basho5


basho/riak_core/basho/mra/bump_exometer_core_1.0.0-basho5 = 6be14e57 merged ok, testing candidate = 7787ee30


Version 0.1


Maybe `checkNear()`?  Might be nice to add a comment about how close the `IOUAmount`s need to be, although I guess that detail should be at the declaration.


Yeah, but how is that useful?


Ok, leave it as is. We can always optimize later if necessary.


Might be worth checking her XRP and USD balances.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1341.90caa560bfda1f44a6ffbaade3b16631aa4446d0](http://build.palaso.org/viewLog.html?buildId=69739&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1020, ignored: 12 Build time: 00:07:00


Oh I see now, yeah. That's a different problem. I'll look into fixing it.


Sounds good.


Does such a command sequence look like a questionable duplicate?


I'm happy to delete the Martinos specific doc. I'll just say that Martinos
people should consult their local people, unless there is someone specific
I should list.


#derpCity lol


Let me take a look at how it's used by tests. Don't make any changes, yet.


Done


Thank you Russ. I amended this commit with meaningful message so that it will meet these requirements.

And yes, this is exactly what this change is meant to do :)


Yeah, this needs some work... I don't think it's ever been functional. :confused: Now that Northstar doesn't hold reportbacks or signups anymore I might even say we remove this section until we can break it into its own microservice.


That's perfect, thanks.


Since 70d3c76758db1ebd092ab12757cd43030a845bc9 :wink:


Drop one of them, probably do not need both.


This change works fine with linux and the following test code.

#define VKCPP_ENHANCED_MODE 1
#include <xcb/xcb.h>
#include <vulkan/vk_cpp.h>

int main(int argc, char *argv[])
{
  vk::PhysicalDevice pd;
  xcb_connection_t *c;
  xcb_visualid_t vid;
  pd.getXcbPresentationSupportKHR(0, *c, vid);
}


Yup you're right, I misread it


Nikita, kas par huiņu?


the last argument `{}` seems to be unused


It turns out that podsSelector is `AND` of  all pod names, not `OR`: https://github.com/kubernetes/kubernetes/blob/master/pkg/fields/selector.go#L219


I really wish I knew this code a bit more. Would you be open to a 30 minutes hangout/skype to walk through it?


I noticed that you left the hardboot page at 0x1fd00000 like the Nexus 6P (without reserving it) - would that work? A different port to the Op2 correctly reserved it and the kernel didn't boot so I'm not sure...


else can be omitted.


I don't think so.  We wouldn't know at this point from just seeing `nil` for `last-updated-before-tag-computed` whether or not a `mark-all-environments-expired!` call had occurred between the `get-environment-class-info-tag-last-updated` call and the `set-environment-class-info-tag!` call that calls into this function.

For the case that `mark-all-environments-expired!` had not been called, we would want the cache to be updated here.  This would be the "normal" case for the first update made to the cache for each environment.  For the case that `mark-all-environments-expired!` had not been called, though, we would not want to update the cache because it would essentially cause the `mark-all-environments-expired!` call to be a no-op.

Does that make sense?


這裡看起來好像是初始化...
但是初始化不是應該是這樣嗎？`NSMutableArray * fighters = [[NSMutableArray alloc] init];`


why don't you just call view->requestActivate() if the screen is primary below when a new screen is added?
by doing so you would spare a for loop


Why is this `default_syscall_thread_data`?


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


It still looks pretty similar to this in the latest code.  I can change it back to something more like the original if you think that looks better.  I don't have any strong opinions on this.


yes, sum = 0. I forgot to change it after testing.

Changed


eh, I don't think we should even be that special. In general though, I think a note before the install instructions that users might want to ask at their site if someone manages an install for them (a lot of users will not mess around with downloading and using tools unless someone else has done that leg work).


With `mapAliased` delegating to `map`, this doesn't work anymore. But I think supporting aliases in user queries is a bad idea, because
* it's cumbersome, the user has to guess the exact order in which the fields will be aliased. In practice, this order is determined by [Class#getDeclaredFields](https://docs.oracle.com/javase/7/docs/api/java/lang/Class.html#getDeclaredFields()), which explicitly says it makes no guarantee about it.
* it implies that we commit to a specific format.


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1570 of 1570 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.8%25-green.svg)


@kohsuke Looks to me like this should have been a 2.54/2.55 for main line?


lgtm


Essentially $newNode is a copy of $child. Why not use the latter ?


oops


No it wasn't broken. I was just looking around after a few failed build attempts. I have now successfully built and can the game runs fine.

Thanks fabiensanglard


typ oznamu?


Laravel Collective's HTML service provider from the [`laravelcollective/html`](https://laravelcollective.com/docs/5.1/html) package (for `HTML::` and `Form::`). Probably something we could remove in a future refactoring, but didn't seem worth the time of pulling out at the moment.


Their async producer send function has some pretty hefty retry logic: https://github.com/dpkp/kafka-python/blob/master/kafka/producer/base.py#L145-L211


A verifier en terme de perf


Chao, it's all in master now. Can you please push up your JUnits ASAP?


yes im wrong i know here. I didnt get time to check that out :P Will do it tomorrow, have my exams
Did the different port boot though? This too didnt boot


I'm not seeing doors to new race conditions being opened here.

The only significant difference with this PR is that an entry would be added to the cache if one didn't already exist at the time the call to get the environment's "last-updated" date.  The update is done within the context of the `swap!` function for the atom and so should be done in a thread-safe way - as all of the other cache updates are.

I'm open to another approach if you can think of one which solves the original problem I wrote up in the ticket.


I'd like to have at least #433 merged before a new release. The rest depends upon whether I get write access to the npm package or if somebody else makes a release. @tmcw @springmeyer I planned to ask you anyway if you could grant me access to releasing new carto versions on npm?


Yeah, maybe best to remove it so doesn't cause confusion, cause I was defs confused as to what was going on...


I prefer those changes related to tinydtls in separated commit, what you think ?


still? because i haven't this issue and i can open containers. (furnace, chest)


Hmmm I definitely agree that we would want to yield as oppose to penalize a flowfile when it's an overall connection problem but the way its designed is that method is processing a single flowfile. Specifically that try block is for where it's sending that message (channel is already connected, I believe) so there is a decent chance that the IOException is due to the message (ie. too big or some other flowfile related reason)


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#DPGFlanNS1ml8Ynf7Y72hg


Should be "Run it for 1 second" if we want to get real anal :eyeglasses:


Oh yeah, duh! Ignore me :cold_sweat:


human controlled


Btw, events client package has some nice methods, like Search: https://github.com/kubernetes/kubernetes/blob/master/pkg/client/unversioned/events.go#L42

They may be a little bit faster that what we currently do.


saw approval from nickelization
at https://github.com/basho/bitcask/commit/04151e9f6cd6d449dc5b285801ab726113c7f116


merging basho/bitcask/dr/bump_cuttlefish_to_2.0.2p1 = 04151e9f into borshop-integration-235-dr/bump_cuttlefish_to_2.0.2p1


basho/bitcask/dr/bump_cuttlefish_to_2.0.2p1 = 04151e9f merged ok, testing candidate = 78eb4364


this line


C'est un peu obscur ça gagnerait à être extrait dans une methode externe


@jodiwei a couple questions:
1) I just noticed that the code around the data has changed. Are you 100% sure this format:
Trunc(From_tz(Cast(le.created_at AS TIMESTAMP), 'utc'))
is equivalent to this format?
To_char(Cast(( From_tz(Cast(le.created_at AS TIMESTAMP), '+00:00') at time zone 'US/Central' ) AS DATE), 'yyyy-mm-dd')
2) Why is it starting after 146 days post lead creation? It looks like the current noloan_T10 targets people who had leads created 126-133 days ago.


In the case where network_type is not TYPE_OPFLEX, is a DVS binding really possible? I'm not sure how this works with the normal OpFlex agent either. But I'm wondering if the checking of the segment should be done by the strategy function. I think this would simply mean each agent type would need its own _try_to_bind_segment_for_<type>_agent() function that would check the segment, and if its OK, set call set_binding().


Yeah, it breaks in npm 2, but not npm 3, which installs all dependencies flat in one big folder. See pull request #31 for a way to update your `package.json` to get around this.


DRY


won't that lint program want these to be more like `data.playlist.uri` ?


- _**мов**_ ( і коми не потрібно)

- > Місце створення ваших змінних залежить від того, як ви їх оголосили і ECMAScript 6 надає вам простіший спосіб контролювати область видимості.

- деякі рекомендації  _**для**_  їх використання.


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


@eldark88 Why do we need this method if DSAPIClient is no longer a singleton? If the user doesn't want to keep the old state around wouldn't they just create a new client? Seems cleaner to me than having a method that may or may not clear everything they want.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#UnvCtTw2R8SYPeN_AtDkgg


override


This line will call _binarize regardless of the number of children in $child, so it will also be called on nodes that are already binary. Overall, I think that this line + L1688 mean that the whole tree will be duplicated to a new copy. If this is the case, it is better to keep the original object instances.


Done -)


saw approval from nickelization
at https://github.com/basho/riak_kv/commit/85191bce0dd9937e90dacc72595f7e110a7e3a3c


merging basho/riak_kv/dr/prepare_for_riak_2.2.0_update_deps = 85191bce into borshop-integration-1358-dr/prepare_for_riak_2.2.0_update_deps


basho/riak_kv/dr/prepare_for_riak_2.2.0_update_deps = 85191bce merged ok, testing candidate = 4d2672ff


Essentially $newNode is a copy of $child. Why not use the latter ?


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/server_databases.js


This is not clear to me -- what is the environment variable I need to set as user? How about add a reference to the following sections that you specify the env variables?


remove this.


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


Работа принята.


The "newline in error msg" bug first appeared on a test that was testing intentional panic in a vtctl command. At first I suspected something wrong with our panic handling (like not cleaning up grpc connections), so I looked over that part and noticed this leak.


Ça vaudrait pas le coup de deporter le .value ici ?


Add a sub section header here about "Environment Variables" will help divide the topics a little bit, otherwise, it is a lot list of instructions.


Not, they are amsterdam and toronto now... but I will update the code later when issue https://github.com/Anoncoin/anoncoin/issues/89 will be resolved


Add a sub section here like "network configuration"


Oh right sorry- these are both "in-between" fluids in the middle of the refining chain, so they are both different and can be broken down into different things. (Heavy Oil can be broken down into Light Fuel and Dense Oil, while Dense oil can be broken down into Dense Fuel and Residue)


typo `SetAcceptEncoding"(` instead of ``SetAcceptEncoding("`


We need to decide whether NestedSet::random_binarize() works on the node given only, or also recursively on the whole sub-tree


To make it easily readable, make these steps a sequence, like:

  * step 1
  * step 2
...

Or:

   1. step 1
   2. step 2
...


Indeed, I need to run it again after merging the queue saving dialog stuff...


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#K1Q7qd67S1enXL4zOc0zFQ


UI - неплохо, но адаптивность немножко ведет себя некорректно. Некторые элементы прячуться или уплывают. А так неплохо, работа принята.


Seems that this is the real optimum at STC because tuning was at STC. LTC may have a different optimum, unfortunately.


Agreed! Done!


`else return key`?


`else return value`?


I think passing in null is not the right thing to do, you need to pass in the actual session object


My feeling is that we don't need $newNode but can use $child directly. I think it means that \_binarize() does not need 2 arguments but only one (the node to binarize) since it is modifying the tree _in-place_. Then, \_binarize() could become binarize()


@GretaCB mind adding an entry to the CHANGELOG.txt about this release?


The reason `mark-environment-expired!` isn't subject to the same problem is that it will specifically cause a new entry to be created for the environment if it doesn't already exist.  If the original call to `get` for the environment would return `nil`, the value after the `mark-environment-expired!` call would be set to "something different" - which would have caused the code to avoid updating the cache for this set.  We don't get that behavior with `mark-all-environments-expired!` because it only invalidates the entries for environments which already exist.


I was just pointed to this change by https://github.com/pyinstaller/pyinstaller/issues/1848 and saw this line. This looks ugly and may be the reason for this error.
IMHO This should read
```
from . import tzname_in_python2
```


Issue #17 solved:


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#dK-bFwW7RaSGg5RWo52dhg


```python
2 ** 31 - 1
```


I understand $newTree is meant to be the new copy of $orig_tree. It should acquire the latter's name, node_id and distance_to_parent
See below: we may not need to make a copy at all.


@AJCStriker What changes did you make the mission.sqm?


Comentário atualizado.


Issue #16 solved


Absolutely way better, thanks!


LGTM :+1:


Maybe upstream it ?


Issue #7 implemented


Ok


@cmorlok From official 3.2.7 release.


- (я думаю, що ті оголошення використовує таки людина).

> Якщо ми використовуємо `var` оголошення, змінна переміщається на початок функції (або в глобальну область видимості, якщо оголошення відбулось поза функцією) не залежно від того, де воно відбулось насправді - це називається *виринання*.

- _незалежно_

- Щоб продемонструвати цю поведінку, розглянемо наступну функцію: (я незна чи то точно помилка, але так звучить більш по-українськи))


all tests pass:
success: http://buildbot.bos1/builders/test-bitcask/builds/586


Работа принята.


Just had to choose a number...


you should move all `extern crate X` to the `lib.rs` file.


This change has nothing in common with the ctt_hook. I think a separate commit would be better for this (with some explanation).


Shouldn't this be m_partialNanoseconds = 0? to avoid errors in adding two TimeStamps?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#WYqB573iSbS-RdGmlWrDTg


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


Would you mind if we called it **ipfs-hyperlog** in the link text, just to be explicit? I don't want to pretend to be the author of hyperlog, which was lots of way harder work. :)


Is it 2**31 or 2 ** 31? Neither raises error in pep8.

On Mon, Feb 29, 2016 at 2:10 PM, Xavier Bouthillier <
notifications@github.com> wrote:

> 2 ** 31 - 1
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yaoli/Theano/commit/2d3bfb7c48eaa66b4f9d4b47e091af432d5996b0#commitcomment-16397871>
> .
>


Excellent. I should be able to get the actual address of the relative offset more easily now, so this should work. Thanks!


untested ACK


- >коли `condition` буде **_дорівнювати_**  true.
бо тут якось незвучить. Рід непонятний)

- **незалежно**

останнє речення незнаю як змінити.


bugfix


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


I checked on OSX and Ubuntu 14.04 and apparently CLI doesn't have the `CORE_ROOT` behavior there. I added a windows-only condition to this property and added a comment.


Why don't we pass the Weapon object?


ah I see :+1:  cool
yea I was wondering if you wrote that all by yourself.


Changed here too, thanks.


Rubik's cube resolvin problem.
in this project I'll try to implement different algorithm to resolve the rubik's cube problem.
// First Algorithm
breadth method searching solution


@zhuowei can you help me brother? I arnt able to get the boot menu too. Recovery works fine. I have tried to compile with multirom-nexus6p's sources, Tassadar's dont work ofc, My tree is here: https://github.com/regalstreak/android_device_oneplus_oneplus2/tree/android-6.0-mrom


Please open an issue to track getting us back to PowerShell conventions.


12345 should be a constant


Don't worry, i will fix it as i merge the code


It's pathological, but... if there were two legit cache updates that were 1 ms apart, and this stuff was somehow called between them, then what would happen to the second one?  Does it matter that its timestamp would be equal to the latest one in the cache?  Or does it just matter that the timestamp that it observed before it started doing its work is different than the one it observes after doing its work?

Even if this works as-is, it seems confusing to me that we're using a timestamp as a value here but then arbitrarily modifying it.  If equality comparisons are the only thing we're ever using this field for, and it doesn't actually matter what type of data we put in there, then maybe it would be less confusing if we just had an atomicinteger that got incremented any time anyone mucked with the cache?  And we did away with the timestamp altogether?


LGTM :+1:
Thanks bruno


spacing is funky.


spacing is funky all over.


:heart:


`type='string'` should be `type='str'`


You need keep a reference to to this call otherwise if one calls `sol_oic_client_del` while this pairing is happening the callback will probably access invalid or leak resource. In other words, we should have a pending call for pairing to be able cancel this when necessary.


I think this also needs to include an optional dtype parameter in case the user wants to create a :float64 or somesuch instead of an int matrix. Or, for that matter, an int64 instead of an int32.


The code should work now, but the PowerUp class isn't working, and I haven't made the images for the PowerUps as of yet, so they're just empty rectangles.


You check that *all* integer dtypes generate a matrix of type int32, but that's not as it should be.


:byte is also an integer dtype.


Should they be green?


and epoxy
configure.ac
>PKG_CHECK_MODULES(DEPS, [gtk+-3.0 >= 3.16 glib-2.0 >= 2.40 mpv >= 1.20 epoxy])


fast-forwarding patch-1.7.2 to borshop-integration-235-dr/bump_cuttlefish_to_2.0.2p1 = 78eb4364


I don't think this leg warrants a warning since they're already positioned for the desired behavior.


Using npm 3.8.0
Still getting "Failed loading external module babel-register" - but it builds.

#31 fixes unknown plugin error, do we still need babel-preset-es2015 then?


Just use editor.clear(NOTIFICATIONS);


@rzarzynski note that we should return the same etag for s3 requests


Я не ожидала, что Вы посмотрите раньше срока сдачи и не успела ещё подправить адаптивность
К тому же это даже не последняя версия :(
Clean Code можно смотреть, я сразу лабу старалась по Clean Code


Thanks, I will look at it


fast-forwarding develop-2.2 to borshop-integration-1358-dr/prepare_for_riak_2.2.0_update_deps = 4d2672ff


Deklarirati kao abstract klasu


These functions aren't used, so this can just be removed.


on Sat Feb 27 2016, Dmitri Gribenko <notifications-AT-github.com> wrote:

> AnySequence and AnyCollection are not like regular collections (Array
> or Dictionary) -- they don't store the data.

In what sense don't they store the data?

--
-Dave


Love those handy meat tags ;)


ola sou André moreira começando a aprender um pouco mais sobre esta ferramenta


@OmgImAlexis first I used the $.each(data.xem_numbering), function(season, sceneException)... But then grunt was giving me errors...
```
   ../gui/slick/js/core.js
   2447 |                $.each(Object.keys(data.season_exceptions), function(index, season) {
Warning: Invalid array length Use --force to continue.
```

But then i tought, hmm maybe it doesn't like to loop through a key/value (object), thus used the Objects.keys().. but then it still gave errors. Could you pleasssseee take a look at this? We can keep committing against topic branch until it's solid.


Fair! It should be called that.


I think this entire return can be replaced with
```
    return (
        '@_ = List::Util::pairmap { $a => $member_coercion->($b) } @_;',
    );
```
We already require a List::Util that provides pairmap.


Just have a single shared pref entry. It reduces the points of failure. And it gives you something easy to print out when debugging.

Set<Text> texts = gson.fromJson(prefs.getString(NOTIFICATIONS));
...
editor.setString(NOTIFICATIONS, gson.toJson(texts));


Maybe we can start updating the board based only on the owned snake.
Example:
Player 1 Owns snake
Only updates position of snoke
Player 1 sends back positions of both snakes
Server receives positions of snake and snoke from both clients
If snake in client2 different from client1, change client2 snake to be same as client 1
Do the same thing for snoke owned by client 2
AKA rollback
When dealing with collisions:
Check if the positions are the same on both ends
If not, maybe as a failsafe use the message that was sent first???
Or rollback to when they were both the same


What glitch occurs in IE11 when excerpts have the opacity transition?


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


LGTM


Try context.getApplicationContext().getSharedPreferences(...);
Something weird might be happening with Context.MODE_PRIVATE given the bug you're describing


https://github.com/dotnet/corefx/pull/6145 is good to go but I got distracted and never merged it. I'll rebase and merge now so we can turn this off just for jenkins.


FWIW I did copy and paste these just for independent confirmation, though this is a very straight forward pull request.


ok!


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1570 of 1570 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.8%25-green.svg)


Do we still need CircularImageView now?


Correct: https://github.com/exceptionless/Exceptionless/blob/b64de0aa19e95ee22e4360be7ae1ce6840c8fb8a/Source/Core/Extensions/EventExtensions.cs#L250


I do get an error for ```2**31``` with my pep8 version but seing that the next line's ```2**10``` doesn't raise any error mean it's OK for travis' pep8 version.


I don't like this.  Instead of doing it this way, in the setUp() of the testing code, or else in the testing code for this testing method, write a yaml file to the right place.  That way you are actually testing the code that will be run in production  and not testing a different form of the code.


whaat?


Why?


The file has been removed, not replaced


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


@prudhomm Sorry about that, emacs took an initiative
I still have tome idea about this issue, i look at that tomorrow


oh.. oops.. ehmm.. sorry for noise :(


:thumbsup: done.


looks like you have been busy ! call / message me ! Whatsapp, Text ,phone call elaaam vitu unna kadasila github la thaan pudika vendiyadha iruku.


no problem ;)


@arcseldon what are your thoughts on this?
It works, but as far as I can see the futures aren't executed in parallel.
I will have to rewrite this and not rely on `R.liftN`


This is bad commit, please ignore.


They present a view onto the underlying collection, but don't create storage of their own.


The simple answer is because we don't need to.  We really only need to know the classname of the weapon to know whether to allow/disallow picking the weapon up.

All of the other 3 methods that use _get_handler_result also only pass in the weapon classname.  Well, that's not entirely accurate.  It could also be the basename, projectile name, or special name, which is why we convert it in _get_handler_result to the basename of the weapon.  But there is really no point in passing the Weapon instance itself.  We only use that to get the classname.

Though, we could always just convert the pointer to the edict and get the classname that way instead of creating a Weapon instance.


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2126 of 2126 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)


is this edit correct? singleOutlierIndices is not a boolean array


The problem w/ stuff like that is Kodi will change it w/o notice and then it's broken and I never notice because I don't use it. I'd rather do something that's official supported so it's less likely to break.


yeah, right. removing.


mvn clean


@benbalter: can you check https://github.com/github/government.github.com/compare/gh-pages...XhmikosR:gh-pages too? I can make a PR with those later.


The code repo doesn't have a license file, this makes it legally tricky to work on or even use.
I assumed the MIT license, since it's a permissive "do-whatever-you-want-not-my-problem" kind of license, which is in the spirit of not having a license. :)


Cijeli ovaj dio bi malo prepisao, Korištenje točnih indexa ($partsCount-2) nikad nije dobro rješenje, čisto jer se inputi znaju mijenjati i znaju se pobrkati stvari.
Moj prijedolog bi bio nešto u stilu
$reverse = array_reverse($mxParts, true)
$output = '';
foreach ($reverse as $rev){
ovdje uzeti prva dva i izaći iz petlje
}

Naravno provjeriti tj validitirati nekako jel to output koji očekuješ, ako nije baciti exception


@degrigis
what did you want to print whit this line:
printf("This means that the chunk 0x%08x will be inserted in front of the SmallBin\n");
?


Could we attempt to login with db.clientIdentityRepo.Authenticate here with both the unescaped and non decoded values to see if they're using the outdated client?


just a copy of the site


That's really nice! :>


the name RTC_GetTime() is reserved form the  library ...

MD380_  is my solution


Thank you! I was in a hurry and had no time to look at the problem. I fixed it now in ce114b65369e5f673eddd57507183a96504b2b77


great


We only need uptime in FxA for computing clock drift, and IIRC we don't do that on iOS. We only use wall clock in Sync for last-ditch cross-clock conflict resolution.


@deepsweet короче пидр смотри: есть небольшая проблема, когда в приложении у тебя `exportFactory: false`, но при этом есть слой, где внутри используется фабрика, например `core-components`. Все ок, пока не натыкаешься на что-то типа `import Titles from '#tabs/titles';` внутри тех же [табов](https://github.com/rebem/core-components/blob/master/src/tabs/index.js). В табах то мы предполагаем, что экспортируется фабрика, а в приложении у нас обратное и в итоге ошибка что мы типа вызываем класс.

Вроде логично, что слой сам должен решать, что он внутри себя использует, поэтому эта проблема решается добавлением опции `exportFactory` в конфиг слоя. Я попробовал уже — все работает. Сам код можно подрефакторить, но суть думаю понятна. Что скажешь?


Nekako validirati da je email, ako nije baciti exception


why @pavangandhi ?


These changes weren't content changes, just sphinx syntax that I noticed - they weren't syntax errors or warnings, but sphinx was joining the lines together into one.


test msg


Thanks.  I'm going to go ahead and merge this.  Then we can change the attributes once build tools are updated, new ones are pulled in to corefx, etc.


во-первых скажу, что ты пидор.
во-вторых, идею понял, код гляну уже завтра, как-то сложновато всё выглядит.


Knowning if we are running the Bids-collaborative version or not is a good idea, but if we put this commit in the official (Roberto's version) then it would also show  'Development!'.  May be we should look for an environment variable ?


Implementacija providera bi trebala znati na koga se odnosi. Treba separirati i razdvojiti logiku da se nalazi gdje pripada. Ova klasa uopće ne bi trebala ništa znati o tome koji provider je za koju domenu. Po njenom nazivu ona bi trebala samo znati vratiti url.

Ovaj cijeli array bi mogla riješiti dodatna klasa koja bi služila kao manager tj njen jedini posao bi bio kada se proslijedi validan mail da vrati instancu provider objekta koji je za taj mail, i onda ova klasa zna iz tog objekta izvući url


Update-SZVAV-for-Fan-Coil-ASHRAE-Method (rraustad) - i386-Windows-7-VisualStudio-12: OK (2126 of 2126 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)


TODO: Improve the commit message (why View is wrong, why the changes are necessary, ...)


Ova metoda bi trebala pozivati managera i od outputa koji manager vrati pokupiti link


damn! good catch, shows i need to add more tests


Well, the mace is green and so is the life tower omni, so I figured it would be better to keep the modified color scheme :smile:


Also added an Icon for the mods menu.


I incorporated a Singleton to hold both the students and signedInStudents array. This means updating the First View Controller, the one within the admin page.

Also, I updated the admin branch of the storyboard to have a nav controller. We need to figure out how to dismiss the entire branch...


quisiera saber en donde puedo conseguir una lista actualizada de canales. la verdad he tratado de buscar en internet y la mayor parte de estas no funcionan y la verdad no se porque. le agradeceria si pudiera explicarme como mantener actualizada la lista con canales de mi region. Gracias.


"Splunk SDK Java" is used in GetSplunk. It is offered here[1] with an Apache v2 license. Although it has no NOTICE, shouldn't it at least be listed here?

[1] https://github.com/splunk/splunk-sdk-java/blob/master/LICENSE


If the two cache updates occurred within the same 1 ms, the cache update performed for the second one will be bumped to 1 ms later than the previous one.  I did that when I noticed that some of the unit tests would sometimes fail if multiple cache updates occurred within the same the millisecond.

Yeah, maybe a atomicinteger would be better than just using the timestamp since we'd otherwise be doing these corrections anyway for multiple events that happen in a short period of time.  I'd only held onto the timestamp thus far b/c I was thinking it might be interesting to have the "rough" time that an invalidate occurred for troubleshooting purposes - or maybe for metrics in the future?


```js
var R = require('ramda');

function parallelFuture (Future) {
  return function (futures) {
    var n = futures.length;

    return new Future(function(reject, resolve) {
      var resolved = 0;
      var results = [];
      function resolvedYet(result) {
        resolved = resolved + 1;
        results.push(result);
        if (resolved === n) return resolve(results);
      }
      for (var i = 0; i < n; i++) {
        futures[i].fork(reject, resolvedYet);
      }
    });
  }
}

module.exports = parallelFuture;

```
This would run the futures in truely in parallel. Looks not as nice, but it works.


@chiahaoliu ok, nearly there.  I just left one additional comment on the testing code that wiggled its way into the production code it would be great.


I think the last 3 lines should be indented more.


Same question from me :)


добавление


go style: ivbCmpBegin. But we can clean this up later...


@ForbesLindesay


I think I'd expect these to live in the buildapi group, but it's slightly confusing the constants are in the authorizationapi package


enums in `sol-oic-security.c` are with wrong identation


I specifically didn't want to have any code do "greater than" comparisons when deciding whether or not to update the cache.  The checks are purely equality based.  So if the "real" time something happened is actually a sub 1 ms time earlier than what "last-updated" currently holds, it doesn't really matter.  The code for checking if the value in the cache should be updated for a set is purely based on whether or not the "last-updated" time when the "get" call occurred is exactly the same at the point the "set" call occurred.  If it's not, we assume that it's not safe to update the cache with the new tag value because something else has happened that has invalidated the entry in the meantime.


Fixes #10


полигон, а не точки


The `lift` one should be come it's own module `all-futures` and there should be a `any-futures`. `All` returns an array of all results and fails if one gets rejected, `any` returns the result of the first resolved future


и вот это нехорошо


Fixes #9


- > Змінна `value` виринає вгорі блоку і в тому ж місці відбувається ініціалізація .

-

> ...змінна `value` все **_ще_** доступна в блоці...


OK, I mistaken this for a different file. I can actually remove it and add as a new commit as it improves things (for example you get all html result files, not just the implicit one). Or even postpone it till we get the "avocado.app" vs. "avocado.plugins.xunit" support.


FWIW, there is PR #89 that 'conflicts' with this.


Why?


Did you mean ````registrar```` ?


Wouldn't be better to check null-pointer in line 1738?
i.e.:
`if(null == entity || code < 200 || code >= 300){`

I don't think we do need all staff under `else` statement.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#H-3z3dkaSQKldF-3u9GcXA


this test #2


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


we should test that the return value == 0 to make sure the command ran successfully.
e.g. `curl -H` will fail, but not raise a Python exception.


This cause Fall Damage everytime you fly down. I died several times because of that.


To be compatible with older Pythons: let's use {0} instead of {}.
cf. http://stackoverflow.com/questions/10054122/valueerror-zero-length-field-name-in-format-python


Please add *.py extension to all your files. In other case GitHub code highlighting will not work.


getting "No compatible version found: inky@'>=1.0.0 <2.0.0'" when running "npm i"


return x.left?


return x.right.min()?


no need to catch exception if you just throw it without processing.


У i2pd встренная поддержка конфиг-файлов. Думаю лучше их использовать, просто указав нужный конфиг в команде запуска параметром –conf=


Sorry, try it again.


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


Please add a check if there is something dangerous on ground.


Issue is that it should support random input. For example:
if you pass
`Observable(price=100, count=10)`
you will end up with
`TypeError: __init__()`
So it should work with random parameters. And now you support only limited number of params. You should change it from 'hardcoded' to more universal solution. But you are on the right way


Could you explain what it is? Is that height of tree? Why not `math.Floor(math.Log2(ivt.Len()))`?


I agree with your strategy of sticking to equality comparisons; I just think that using a timestamp as the value makes it harder to understand what is going on in the code if we're only doing equality comparisons.  Doubly so when we start artificially synthesizing the timestamp values.

I also agree with you that having the timestamp in there for troubleshooting is useful.

So maybe the thing to do here would be to keep the timestamp field in there just as a write-only field that could be used for debugging, and add a third field to the map which would contain the value from an atomicinteger... sort of a 'cache generation' id field?

I realize that that will take a non-trivial amount of extra work on top of this PR, so, depending on your thoughts about that idea, we can weight options as to short vs. medium term plans for this.


hello,

i am jes.

add time 2:57


Commented out line, would suggest removing


+2 - Approved - @kevinliao2016


> Часто новачкам у JavaScript розробці важко звикнути до того, як працює оголошення змінних (**_коми нетреба_**) і нерозуміння цієї неочевидної поведінки може призвести до помилок.


`IntervalTree represents...`?


@yehudasa Do you mean a situation when someone tries ```GET``` on a DLO through S3 API?


I am not sure where to go with this, so I guess I'll just put this on the newest commit. Will KSML support paintutils images?


Why comment that out?
It looks like we were trying to execute the function in DebugMode to check that everything was correct.


You derped again in TunerFragment....


It seems that this constructor conflicts with `Fake.TargetHelper.Target` when I have both `open Fake` and `open Fake.FscHelper`.


If you folks are going to introduce backwards-incompatible changes then please, _PLEASE_. add a CHANGELOG.rst to this repo. Changes are happening rapidly and I have no way to know what you've changed without sifting through tons of commits and reading the changes line-by-line.


Why skip? When will final cleanup happen?


If the issue is about `out_val` not being reused, then we can call `f(a_val, b_val)` without assigning the result, that would at least perform the check.


Same here.


Can it be done for other sources like CSV?


We want this for cpp codegen though, right?


Could you push tag v3.2.1?


I don't see where in the context of this class the hash algorithm defaults to sha1. I see it as a default in `Test.fetch_asset()`, but if a user decides to use this class alone, this will not hold true.


This should be cast to a string. Uncast finds in mongo are an injection vector.


yes, but in SoapUI empty origin tag is not allowed either for legacy system or asit.


Hello


A code review? What is this madness?!


It would be worth sanity checking this data against some validator. Right now this can be exploited to write and retrieve arbitrary data which could be used to DOS the server or hijack this storage mechanism for other purposes.


What about this?

    binder.bind(Event.class).toProvider(MockProvider.of(Event.class));

As suggested here: https://stackoverflow.com/a/10846095/14379

(Even that seems a bit redundant, because Event.class is repeated.)

Is there a test which uses events we could try?


I know, utter madness :-)


This is a markdwon file


Oh and also added sword health bar.


http://json-schema.org/ might be a decent choice for data validation server side.


Would love to see a clang version of uber :)


For final project.


You can make it with google's Allow peeking in Apps-Notifications-Phone


Dear Professor,

Dear Professor,
Thank you very much for your kind response.
I did some simulation by giving the parameters-
./waf --run "scratch/s1g-mac-test --NRawSta=256 --NGroup=8 --SlotFormat=0 --NRawSlotCount=4 --NRawSlotNum=32 --DataMode="OfdmRate1_3MbpsBW2MHz" --datarate=1.3 --bandWidth=2 --rho="50" --simulationTime=20 --payloadSize=100 --BeaconInterval=100000 --UdpInterval=1 Nsta=256 --file="./TestMac/mac-sta.txt" --pcapfile="./TestMac/mac-sta" --seed=1"
Number fo Groups	Number of Nodes	   Throughput
8	                                  256	              204.52
16	                                  256	              204.8
32	                                  256	              204.76
64	                                  256                      204.76
128	                                 256   	              204.72
256	                                 256                       204.8

I got the above results. I think I did not get the correct result. Because when we make more groups
the collision will be reduced but the above result does not show the better performance.

Please, give some comments.


This doesn't look like it correctly replaced the mixin's context update – it used to put all these within 'sources'.


Make sure to expand the condensed lines in grading.


Looks good. I'll get that in the next commit.


Also, maybe consider a timestamp on the record so that old payloads can be expired, or use a cache store like redis for this, which would also keys to be set to expire at write time. That would help mitigate unbounded growth of this data server side.


The above line is working on testing :)


Missing docstring for `cache_dirs`.


This doesn't look like it correctly replaced the mixin, which stored these all within 'sources'.


It's easier for people to just use a switch instead of going into phones permissions and disabling peeking


The above line here, however, is not working. It is doing the following:
It is setting genericval2 to 0
It is not clearing genericname2


→ https://github.com/andyearnshaw/Intl.js/pull/150


Getting and Cleaning Data Coursera Project

    The purpose of this project is to collect, work with and clean a data set.  A script, run_analysis.r, will prepare a tidy data set from the files obtained from this location:
https://d396qusza40orc.cloudfront.net/getdata%2Fprojectfiles%2FUCI%20HAR%20Dataset.zip
This project contains the following files:
•	run_analysis.r – a script that will read the necessary files and massage the data to create the tidy data set.
•	run_analysis_output.txt – the tidy dataset output
•	run_analysis_codebook.md – describes the data and the steps takes to create the tidy data set.
•	Run_analysis_codebook.md – this file you are reading.
Steps to understand this project
•	Read the run_analysis_codebook.md file to understand the data
•	Read the run_analysis.r file to understand how the data is manipulated.
•	Download the data from link above
•	Make sure your working folder is set to the location where you downloaded the data files.
•	Source the run_analysis file
•	Run the run_analysis.r file from within r or rStudio as run_analysis()


Is it semver?
Should be 0.1.0 imho then..


I think the issue is the AND within the SET section(should be a comma insead):
Try the following instead:
``````
UPDATE `patient_data` SET `genericval2` = '', `genericname2` = '' WHERE `genericname2` = 'Billing';


I'm a little concerned with the increase in both data storage and bandwidth used by this since we were originally doing whole thing on the client side without involving the server at all. We're currently sending ~500bytes per request for the data, but @fierydemise said he wants to expand the options quite a bit for 7.0. Are we going to risk hitting any sorts of limits with this?


I'm failing at trying to use the utility API (which I know, it's not the user's primary API, but it's available nonetheless). Example:

```
>>> a = asset.Asset('https://www.kernel.org/pub/linux/kernel/v4.x/linux-4.4.tar.xz', None, 'sha1', None, ['/tmp'])

EnvironmentError: Failed to fetch linux-4.4.tar.xz.
```

And various others tests failed. I see that there are unittests that exercise that API, so I'll take a better look tomorrow and check if/what I'm doing wrong.


Disk space and RAM (consumed by the DB indexes) is really the only concern. The biggest problem is that since these are unauthenticated endpoints, they are easily subject to abuse.

I think I'd recommend considering a Redis instance configured as an LRU cache for this data store, rather than Mongo. It offers transparent compression of the stored data on disk, and an LRU configuration would keep an attacker from exhausting server resources - the worst they could do is rapidly expire authentic cache records, which would be unfortunate, but wouldn't render the service unusable.


LGTM


OK


ha ba en voila un autre ^^, avant il afichait ecran noir pour le ingame et la magie ca c'affiche enfin, mais un peut buger comme tu voit, il est sur le psn tu sais quoi faire :p
![capture d ecran 2016-02-29 03 24 46](https://cloud.githubusercontent.com/assets/5261759/13384229/0b5d5d1c-de94-11e5-8279-40ff7b0f91d4.png)


Dear Professor,

Thank you very much for your kind response.

I did some simulation by giving the parameters-
./waf --run "scratch/s1g-mac-test --NRawSta=256 --NGroup=8 --SlotFormat=0 --NRawSlotCount=4 --NRawSlotNum=32 --DataMode="OfdmRate1_3MbpsBW2MHz" --datarate=1.3 --bandWidth=2 --rho="50" --simulationTime=20 --payloadSize=100 --BeaconInterval=100000 --UdpInterval=1 Nsta=256 --file="./TestMac/mac-sta.txt" --pcapfile="./TestMac/mac-sta" --seed=1"

I got the following results.
Number fo Groups   ----   Number of Nodes ------  Throughput
8      ---------------------------------- 256  ---------------------- 204.52
16    ---------------------------------- 256  -----------------------204.8
32    ---------------------------------- 256    --------------------- 204.76
64    ---------------------------------- 256    ----------------------204.76
128  -----------------------------------256    ----------------------204.72
256  -----------------------------------256    ----------------------204.8

I think I did not get the correct result. Because when we make more groups
the collision will be reduced but the above result does not show the better performance.

Please, give some comments.


unnecessary white line


this close issue #6


Edition version is wrong


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


better if we can check the return value


Yeah, no. This is unsafe, as it's breaking some code generation tools for creating data definitions.

Some of the less popular target languages like `scheme` or `lua`, `perl`, `tcl`, etc... rely on code generation to populate the services. Their naive implementations can't handle anything more complex than a single lined string with no special chars.

I think the best solution for now would be to avoid populating these target languages with the additional resource object. They already have access to `hook.params`. I will eventually go back around to the code generators for these languages and make improvements. Will have to read up on the SmallTalk documentation again. :-)


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


Another thought would be to use localStorage to keep the X most recent SHAs and payloads, so that the user can save and retrieve them entirely clientside (ie, if they want to step back and forth through their local history). The SHA should really only be needed for sharing a loadout, and using localStorage as the first-line data store and then backing it with the server-side store should allow users to continue to use the service without issues even during a resource exhaustion attack.

I've used https://github.com/Wisembly/basil.js/tree/master before for managing localstorage stuff, and it works really well. Nice an easy to use, should be as easy as dropping it in and integrating it into the save/load routines.


I'm having startup error with 'dotspacemacs-smooth-scrolling' set to nil:
```
(error "ad-disable-advice: `previous-line' is not advised")
```

Here's the relevant debugger output:
```
Debugger entered--Lisp error: (error "ad-disable-advice: `previous-line' is not advised")
  signal(error ("ad-disable-advice: `previous-line' is not advised"))
  error("ad-disable-advice: `%s' is not advised" previous-line)
  ad-disable-advice(previous-line after smooth-scroll-down)
  (if dotspacemacs-smooth-scrolling nil (ad-disable-advice (quote previous-line) (quote after) (quote smooth-scroll-down)) (ad-activate (quote previous-line)) (ad-disable-advice (quote next-line) (quote after) (quote smooth-scroll-up)) (ad-activate (quote next-line)) (ad-disable-advice (quote isearch-repeat) (quote after) (quote isearch-smooth-scroll)) (ad-activate (quote isearch-repeat)))
  spacemacs/init-smooth-scrolling()
  funcall(spacemacs/init-smooth-scrolling)
```

With this `unless` block commented out, I could start spacemacs without problem, and then ran `C-x C-e` to run each of the expressions within that block, and found that all of them raise error. It seems that since `smooth-scroll` is loaded conditionally, the functions below are not advised.


I think it better to change the expression like below. in case of some counts[node.Name] are negative number:
```go
fScore = 10 * (float64(counts[node.Name] - minCount) / float64(maxCount - minCount))
```


Task 1: it is working fine and includes ifmain statement. Also sum is computed using while loop.
Task 2: it is also working fine and computes the sum using for loop.
Task 3: it is also working fine.


Ignore that last comment, I forgot that jStorage was already in play for that exact purpose. :)


Uploaded the wrong file version. See 97d81bf for newest version.


Haha. Sorry for the quirky commit message.

This fix and feat covers:

feat:
* Export all reports to .csv
* Export a single report to .csv (including all of the reports from the same reporting)

fix:
* when the user inputs a comma in the notes/health, it **literally** _fucked up some commas_ and created cells that would not be there


hmmm... The issue here is the 'https'. I will fix that next PR.


The thing we were most concerned about with the URL length was that it's become impossible to paste URLs into IRC for support reasons. On top of that, determining what's in a URL is a painful process of scraping through that old code in `history.coffee`.


你这个 toxml 的方法支持多层的，比我的方案更好。


Ok agreed. iirc this came up at some point in a previous code review, but yes, you're right.


Yeah, definitely. I worked hard to get the URL to IRC-pasteable lengths when the tool was originally written, but the amount of things that have to be tracked now make it a lot less feasible to just store all the state in the hash.

One alternative might be to:

1. Save all history to localStorage rather than the server SHA endpoints
2. Add a "share this build" button which would post it to the server, get a unique ID for that payload, construct the URL, and offer it to the user

This would allow you to keep most work off the server while still allow for shorturl sharing of profiles. I'd still recommend schema validation and throttling of the write endpoints, but that would wholly alleviate the profile issue without necessarily having to save a copy of every single variation of a build to the server.


ack.


agreed.


Hi Brady
Unit testing ok


Task 1: it is working fine and includes ifmain statement. Also sum is computed using while loop.
Task 2: it is also working fine and computes the sum using for loop.
Task 3: I’m not able to run it in ipython.


Then SOAP UI is probably wrong.  The WSDL boils down to:

```xml
<xsd:element name="origin" type="xsd:string" minOccurs="0"/>
```

with no further restrictions; therefore, an empty string should be valid.

Regardless, in general, anything that work on prod should also work on ASIT, unless explicitly agreed otherwise.

Cheers.


In the case of `pThis->bIsActive == 1`, doesn't `thrdTerminate()` perform a `pthread_join()` unconditionally?  So why repeat that join?


  hello，my name is McCoyZ.
Kindly give us your advice, please.


Can't you just use:

```c#
        public static bool IsSpecialType(this ITypeSymbol symbol)
        {
            if (symbol != null)
            {
                switch (symbol.SpecialType)
                {
                    case SpecialType.System_Object:
                    case SpecialType.System_Void:
                    case SpecialType.System_Boolean:
                    case SpecialType.System_SByte:
                    case SpecialType.System_Byte:
                    case SpecialType.System_Decimal:
                    case SpecialType.System_Single:
                    case SpecialType.System_Double:
                    case SpecialType.System_Int16:
                    case SpecialType.System_Int32:
                    case SpecialType.System_Int64:
                    case SpecialType.System_Char:
                    case SpecialType.System_String:
                    case SpecialType.System_UInt16:
                    case SpecialType.System_UInt32:
                    case SpecialType.System_UInt64:
                        return true;
                }
            }
```

I can't tell what the difference is between your function and this one.


There were some updates about this point in the email discussion, just a note here, also to remind myself.

>> 5．The admission controller that limit topology key of anti-affinity to node level. I understand the background and consideration of this issue, and agree with the way to solve, just feel it won’t be a good implementation if simply match the topology key with a hard coded value (node). If user is not using “node” as a label key to specify a node (maybe “NodeID”, “MachineNo” etc. instead), the inter-pod anti-affinity feature cannot work. Maybe we should make it configurable for user to indicate the valid values of a topology key in the inter-pod anti-affinity case?

> We recently added a constant for hostname (LabelHostname in pkg/api/unversioned/well_known_labels.go), so you can just use that. If people don't want this feature, or want to use a different label name, they can disable or modify the admission controller.


>#gdb_id instead of #id

You used gbd_id here instead. A little confusing when I first encountered it.


# Caching Inverse of a Matrix


Doesn't this fail on n=2 and n=3?


@Cryptoslave is the comment "Singapore and Germany Seed Nodes" still accurate?


The first link gives a 404.


This could be replaced with `while i > 1`, etc. while you're at it.


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


trying comment


这样的测试用例只是做个样子，，，


Powered creeper can be repowered and it will die.


不要只是 print 出来看就好了，
因为测试的本质要满足自动化，没有测试驱不动，每次都需要人为再看看 log 输出，再回想一下，那个位置是什么代码，，，甚至还需要看代码才能确认是否真的通过了。


Sorry, i misplaced origin tag in soapUI request. Actually legacy system supports empty origin. So we need to fix this in asit. Thanks.


可以考虑把它集成成 Message 里面，替换我现在的 toxml 的实现。


how I see it now, nice catch :)

Funnily enough this is working thanks to the following line:
https://github.com/melonjs/melonJS/blob/master/src/level/TMXTileset.js#L38
as we then retrieve the corresponding object using the base name and not the extension

if you replace the `json` extension by `tsx`, it also works though, not sure this is really an issue though and if this really needs more code to fix it :)


Forgot to mention the related radar. It is rdar://24818500


executeTransactionAsync() is not declared in Realm class. Is this new or removed function?


I had actually commented this out because I moved from quill to trix


(I think) this comment stops the higher part of the loop from being triggered that does subTasksMainPidLoop which includes Blackbox and a couple of other bits.


We need to add @import "jquery-ui"; above. I added this in the last set of updates I pushed, it is a new addition and has to do with the drag and drop reordering of docs/categories.


naw dawg


Hey, I like this change. It's been a long-requested feature (le717/LDR-Importer#32). Would you like to submit it back to the plugin? With a little bit of work it could turn out really nice. :)


I thought we were reducing our use of this bootstrap code and moving things into CoreLib / the runtime?


You can remove the commented-out code as well.


Is the "if (!t.seen)" supposed to be inside "if (!base.scroll_up(t))"? Because it didn't used to be, but with the movement of the line from 89 then we've now got:

    if (!base.scroll_up (t))
        if (!t.seen)
            this.unread_count ++;


This seems to address issue #36.


it also does the removal, right? not only test if it can remove the node?


this also needs a test


This name is a little awkward: this encodes a forest and some additional state (`lookup`, `orphans`).


I would have like LLVM_GIT but not too hard on it.


Hi, I'm trying to create relationship between registered users in my app, any ideas on how to do it, I'm considering using peewee.


test case?


Why is that necessary? The error is supposed to happen just by calling `function(...)`.


**при** поміняй на **_в_**


A note on `tops` would help: perhaps "GUIDs that might be forest roots" or similar.


Sure thing. I'll add the change in my PR for the System.IO.Compression tests change.


I lied.  I decided just to fix the error in my fork and issue a pull request.  Hope that makes it easier on you.


Actually all your validations will fail. Because you are using regex in a wrong way.
`if (self.regex != self.text):`
this is just a comparison. So in your EmailValidator you pass `regex='[\w.]+[@][\w.][a-z]'`as a params and compare it with input(''ruslanpitula@yahoo.com" for example) and it gives you an error. Which is actually quite obvious. Because `'[\w.]+[@][\w.][a-z]' != 'ruslan.555@ukr.net'` :) Take a look better how re module works.


Fixes defines


It looks like the test was designed to use `test_cpy`. Does that not work?
If it does not, then we should at least put a comment that it does not, but someone believed it should.


Implement a custom exception and raise it here with your message here please.


Fixes Newtonsoft.Json targeting which has `portable-net45+wp80+win8+wpa81+dnxcore50`


Aquí es donde DNI puede (o debe) mapearse a un objeto complejo que tenga por lo menos el número y la letra. Recuerda que en Bussines encerramos la lógica y no hay nada malo en utilizar tipos complejos de dato


no you cant. subtract is static. that was a bug in the old version


update governor-general visiting


So just so you are aware why line 42 exists with the initial fetch. SwiftLint requires the source files to all exist in the root of the project. When we're ready to reenable SwiftLint, we can just remove the comment on like 43.


OK, you can remove the commented-out code.


@Nasafato Why is this here?


Hay que encapsular Address en un objeto complejo, tanto a nivel de VO como a nivel de BO


Fixes `please run dnu restore` on projects restored with `dotnet restore` when `running dnu build` because of how they output framework assemblies names.


@gregory38 I can boot games again with the atomic-fetch-correction branch.


typo


**більш_е_** схожа. бо вихоить збіг шиплячих **_шсх_**


Could you just use their actual names? My understanding goes by this chart found on Wikipedia
![image](https://cloud.githubusercontent.com/assets/8284194/13407325/ded20fe2-def5-11e5-9abb-0bd59807cb45.png)

Of course this can be simplified so the end player doesn't have to learn every little bit.


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


Space in the end of `LibraryRange.ToString()` was preventing `dnu build` restoration check from working correctly for framework assemblies.


thucydides has been renamed into serenity and and now lives here: https://github.com/serenity-bdd/serenity-core


I don't think so, it has to traverse the left branches until it finds a leaf.


@Nasafato why


This is okay, but its a very good practice to use exponential back off when doing remote service call retries.

Utility class: https://developers.google.com/api-client-library/java/google-http-java-client/reference/1.19.0/com/google/api/client/util/ExponentialBackOff

Example: https://github.com/googlegenomics/utils-java/blob/master/src/main/java/com/google/cloud/genomics/utils/grpc/GenomicsStreamIterator.java#L146


import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import javax.swing.JOptionPane;

public class ImportText {

 String allText=null;
 String line=null;

public  String readText() throws IOException{
     try {
     FileReader fileReader = new FileReader(new File("Enter your address to file"));
     BufferedReader file = new BufferedReader(fileReader);

    allText= file.readLine();
      while ((line = file.readLine()) != null) {
          allText = allText+" "+line;

                  }

     }
     catch(Exception e)
     {
        JOptionPane.showMessageDialog(null, "file not found");
     }
     return allText;
    }

public void lengthString(String text)
{
    if(text!=null){
    int lengthText = text.length();

    if(lengthText<9999){
    System.out.println("please enter more text");
    }
    else
    {
    System.out.println("your text =   "+ lengthText+ "   symbol");
    }
    }
    else
    {
    System.out.println("file is void");
    }
}
}


With new style config, is the expectation that user is responsible for explicitly mentioning  the additional OpenStack VMM domain needed in the config file?


Ten cuidado al commitear, has metido un tabulador inútil que ha generado un cambio inútil y que solamente guarrea el repo y hace más difícil ver el diff


Can you comment on the optional type here?  Structure connects parents and their children -- it encodes tree edges -- so I'm confused as to what `type` is.  It's not reflected in the structure table.  Are you accumulating the type of the child?  (Presumably you're confident the parent is actually a folder...)


Жодного


Default should be the boolean false, not the string false


can't you just drop the `return` and the `join`?


This looks like a program flow problem.  Would an `assert` be appropriate?


@rjmccall: Thanks for the template template tip - it's working nicely.

I've been using "Abstract" for the prefix but it doesn't look right to me. Do we have any conventions for these kinds of parameterized data structures? I was thinking maybe `TI` for "target-indepedent" or some other letter-y prefix.


Caca, utilizas el small para dar estilo. Prescinde por norma general de tags que cambien presentación y no contenido.


Может, тогда option conf вынести в опенврт-шный конфиг, а всё остальное в i2pd-шный?


не **тоді**, а **_то_**


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


just a nit: this constant is defined in three places in this commit.


yes, will fix comment


@prparmar77 the cluster created in this sample is configured for Unicast.


configuration for service "snc_redis.victoire_client" missing


makeFrame is an Action computation and in one round we collect information from the persistent. But true there are other ways to produce the same result.


This code is not actually doing a job. Did you test it?

- `class Lonely` - is an old style definition that was used in Python 2.1 and we are using 2.7 at least. New style - `class Lonely(object)`
- `lone_instance` is never created.
- male1 and female1 - what is the purpose of those variables?
I think you should better test code :) good luck


yes.


h


fjerna images/


This starts in the middle of an explanation... (see it live in: http://exercism.io/cli - and click Linux).


How do you end up with database rows that don't know their type?

I'm trying to understand this `StructureRow.type`, and I'm not getting it.


readme


Bolo to skor myslene ako popis chyby. Radsej by si tam mal typ oznamu namiesto nadpis detailu?


Oui, c'est pas DRY


Test note


Yeah, it's height. Will add comment. RB trees have a max height of 2*lg(n+1) since they're not perfectly balanced.


This change disagrees with the documentation. Borders are shown here. http://ionicframework.com/docs/components/#buttons

button-light in particular becomes very difficult to tell that it is indeed a button.


Through-out -- using `m` for tables that might not be the mirror might be mis-leading.  (I can't yet tell if you're trying to guide the reader to this.)


the else is unnecessary?

probably just
```
if len(s) == 0 {
    if len(sc) == 0 {
        return 0
    }
    return 1
 }

if len(sc) == 0 {
    return -1
}
```


unnecessary else


consistance: espace après la ,


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


Non vu qu'on appelle `.value` et `.value=`


:smile:


No idea.


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


lol


Got it. Right, I just found that in the book. Thanks.


That function may or may not be implemented in the AVR version of libc.


Part of IQSS/dataverse#2934


test


i disagree, im writing this tutorial mainly for coala contributors. if there's someone that wants to use it for another repo, he will simply replace coala with the repo he's using ;p


No, I mean this `it()` function. Seems to be linked to another function `describe()`.


Attach


#6064? Are we fixing future bugs now? :D

I guess it's for #6046


oki


> **Оскільки** `let` не перевизначає ідентифікатори, ..

після **якщо** забери кому.

> **з** **_таким самим_** ім'ям, як і в змінної,..


Ah, I get it, this is because the join can produce null rows.


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


The following doc(s) have been deployed to the [1407nginx](http://1407nginx-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/logs/nginx-access-log](http://1407nginx-static-docs.pantheon.io/docs/articles/sites/logs/nginx-access-log)


Thanks for the PR @aeke !


ah... `it` defines a jasmine test, see http://jasmine.github.io/2.0/introduction.html for details.


Not sure, giving it a go should tell you quickly


Aaah. Yeah, that's awful. Thanks for catching it.


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


Bringing HW3


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


don' tunderstand anything


expected 3


@njbbaer is `bootstrap` module required here?

If so, please use `npm install --save bootstrap` command to update package.json.
If not, please remove line 10, because it's breaking on machines what doesn't have `bootstrap` module installed globally.


Series alias is the prevailing method relaying the current series to user, since it's…
a) Also used as the input in many parts of the software.
b) It has less Terminal-unfriendly characters than full names might.
c) It can be changed by the user.


expected 2.


WTF!? =P


expected 1


Reprodicuble -> Reproducible


2094_DisableCloud (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


part 2 is not in use right now. I was experimenting with separating the two file stream processes.


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


**без проблем**


Look at using the hardware timer to generate an interrupt to accomplish this: http://efundies.com/avr-timer-interrupts-in-c/. See the [AT90CAN128 manual](http://www.atmel.com/images/doc7679.pdf) for more information on how the timers work.


Batoto is referred by its actual name rather than its URL. Otherwise, _Madokami_ references should be renamed to _manga.madokami.com_.


2094_DisableCloud (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I think the idea of having both a timestamp (for troubleshooting) and a "cache generation id" (for synchronization) seems reasonable.  Although I still get the sense that you don't like the idea of an entry being created as a side-effect of a "get-cache-generation-id" call in the case where no environment entry previously existed?  Is that the case?  If so, did you have a different proposal for how to do this part?

I'm thinking I could rework this to split out the cache generation id vs. timestamp with 1/2 day of work.  Not sure if we want to try to do that now, though, since we're getting pretty close to our intended release date.  What do you think?


without this line, you get only one set of logs about ril, thats all.
when you start logcat with -b and type of events, its start to log stuff, now if you dont add & its will run on same pid as your adb and die when disconnected and stop logging.
if you dont add > /dev/null you will see logcat lines randomly show in corrent adb session.  in adb shell.
so to prevent it null was used. logs recorded in 256kb buffer as should for all requsted filters.
please make sure you did all checks before chatting in github without a true reason. my 151 followers and there followers see this pointless chat.


nit: s/DataFlow/Dataflow/


This is pretty neat.


This is unfortunate. The Babel plugins aren't loading via `bower_components/foundation-sites/.babelrc`, so they're being loaded by the theme. There's probably a better solution for this.


If this fails, `err` is set and we'll abort later, right?  (You note the torture, but it's still hard to follow.)


Consider breaking the `SELECT` onto its own line so everything else aligns.


@AlbertoSONIC  Just readed a thing on a site,and it say that it will be possible to load decrypt9 via rxTools, does it is real?


> This cause Fall Damage everytime you fly down. I died several times because of that.

Worked fine for me.


Thanks!


Doh! We forgot to add a policy method external_report?  to offerings_policy.rb


В пунктах меню редко используется частица to :)
Да и вообще полезно уделить внимание английскому - хороший тон


@bkuhl you are catching the exception in your function, and not re-throwing it. Wouldn't it be wrong to say this function throws an Exception in your doc bloc?


closes #13


looks good!


- **оголошена**

- буде видалена **і** не буде доступна поза блоком.


@bkuhl if this condition is not met, the function will return true. Is this the desired outcome?