Looks interesting! Is it possible to #include some more things (after line 19) so that we can use ETH_P_IP and IPPROTO_ICMP on lines 52 and 57?


To avoid messing around with ext4 partition we could also provide "empty" disk image file. Both partitions would be already formatted and storage partition would already contain resize file. Then mkimage script would only copy kernel, system and bootloader files. Also bootloader would be installed.


Эти картинки не будут резиниться, так что лучше им задать Height в html тоже.


I accidently committed this to the wrong branch. This was the first fix, but this caused other issues. In order to fix the other issues I thought i had to do a lot of rework, hence the branch.


Merk at CSV import fortsatt brukes i featuretesting, ved oppretting av testbrukere o.a.


> Would that make you happier?

Don't get me wrong, it's not about "making me happier", there is nothing personal here :O. It is just that there are tons of arguments on the benefit of making smaller tests, such as the one [stated here](https://github.com/impress/impress.js/pull/510#discussion_r52167929) about debugging.

Also I am not saying that all tests should be less than 10 lines long or something like that. It is just that smaller tests are easier to reason about, review and maintain. Whenever possible, a test should test one thing and one thing only.


this is already fixed


@bbondy what is the purpose/use-case of `this.webview.focus` here anyway? I can't see where it makes a difference, and removing all of it still works for #361


limit was removed in https://github.com/xbmc/xbmc/commit/e31277442bf624d3ac2480e9a457cd5fb23bb7e5


note to self: the grep command incorrectly reports CPUS=2, becuase the string "processor" appears twice, once giving the number of the processor and once in the processor's description.


Clear. Thanks.

On Tue, 9 Feb 2016, 04:04 Ryan Lindeman <notifications@github.com> wrote:

> I will take a look at the repo manifest call and fix the crash. After
> thinking about my implementation I think I want to all for -u and -o to
> coexist together. This would allow for reading from the current
> manifest.xml file and saving to the specified other.xml file. The expected
> use case for this work was to create a feature/release/topic/support branch
> and then perform repo manifest -u -b (peg to current branch if not already
> pegged to a SHA1 revision) across all XML files while also preserving the
> original format. I'm not familiar with lxml, I was purposely trying to keep
> my changes to a minimum against the original repo code base as was
> previously decided. I welcome your comments on adding lxml as a dependency
> (is it common enough to be a non-issue?).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GatorQue/git-repo-flow/commit/48ce38e575c9be467e0aa6f71dd98cf440ac6abd#commitcomment-15968124>
> .
>


@adam-collins pointed out that the check is still happening in the MediaStore.save. Please ignore.


Kan godt prøve å enable den igjen, den feilet kun på kohadev


My bad.  Need to update myself.


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


Could this be passed as an argument on the command line? Would be easier than having to modify the script every time


Thanks. May you please also clarify how to read the `value` field within the `srtp_ssrc_t` struct?:

```c
/**
 * @brief An srtp_ssrc_t represents a particular SSRC value, or a `wildcard' SSRC.
 *
 * An srtp_ssrc_t represents a particular SSRC value (if its type is
 * ssrc_specific), or a wildcard SSRC value that will match all
 * outbound SSRCs (if its type is ssrc_any_outbound) or all inbound
 * SSRCs (if its type is ssrc_any_inbound).
 *
 */

typedef struct {
  srtp_ssrc_type_t type;  /**< The type of this particular SSRC */
  unsigned int     value; /**< The value of this SSRC, if it is not a wildcard */
} srtp_ssrc_t;
```


I did not want to make too many changes, will submit a PR to make this change post 16.2.


because the node doesn't have corresponding topology key


@NatsuTheGreat you also edited `PlayerCell.js`?


Which constructors do you mean?
If you mean the constructors in HttpSender class, I think this is the way it's supposed to be done. They are meant to be used like that..


Спасибо, все поправлю. Похожесть шрифтов я определяю на глаз?)


This probably shouldn't be here?


1.) The previous code could handle a NULL token. The current code would throw a NPE.

2.) The method name is "isQuoteCharacter" therefore this method should only check if the token is a quote character what the previous code did. The current code checks if the incoming token begins or ends with a quote character. The current code does not check if the token begins and ends with a quote character. So the question is what is the meaning of the new method? The method name should be changed to the new meaning.


Они использовались когда движение перса осуществлялось с помощью кнопок, так что они ненужны, ибо нигде не используются)


@mrtnzlml yes... good catch!


It's bad, that this code is modified second time couple of commits down the road :( Especially that this implementation is indeed worse


Эти картинки не будут резиниться, так что лучше им задать Height в html тоже.


import java.util.*;

public class Win32RegKeyTest{

  public static void main(String[] args){

   Win32RegKey key = new Win32RegKey(Win32RegKey.HKEY_CURRENT_USER, "Software\\JavaSoft\\Java Runtime Envoronment");

   key.setValue("Default user", "Harry Hacker");
   key.setValue("Lucky number," new Integer(1));
   key.setValue("Small primes", new byte[] { 2, 3, 5, 7, 11});

   Enemuration<String> e = key.Names();

   while (e.hasMoreElements())
   {
        String name  = e.nextElement();
        System.out.printl(+ "=");

        Object value = key.getValue(name);

        Object value = key.getValue(name);

        if (value instanceof byte[])
        for (byte b : (byte[]) value) System.out.print(b & 0xFF) + "");
        else

         System.out.print(value);

         System.out.println();
      }
    }
  }

   }


Seems to be fixed automatically :+1:


T will also be used here in Exp4P
https://github.com/ntucllab/libact/blob/master/libact/query_strategies/active_learning_by_learning.py#L103


s/satisfys/satisfy/


What about moving it directly below the assignment of `text_str`? I think it makes more sense there. TEXT_STR and TEXT_LEN should be denoted as constants I guess.


s/satisfies/satisfy/


@dougwilson it starts


This feels a little weird, but not a deal breaker.  I would have expected something like `ExHal.Client.new`.  Just a nitpick.


Estudo Exercicio Algoritmo IF,ELSE


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


unnecessary space?


watch out for little mistakes like this. `Margin-top` -> `margin-top`

Stephen


Just curious, what are the upsides of removal of support for Qt4?


Remove the 'GeUIVersion()' code.


`Semibold` -> `semibold`


remove if unused :)


j


unused


This should not be necessary for Closure Compiler, as we have Array.from in our externs: https://github.com/google/closure-compiler/blob/a482d260edf5baadcaf42e945d5459dcb75ca2ef/externs/es6.js#L1403

If you found it to be necessary then you might be on an outdated version.


Just to note here as well...this is an internal class, so not used outside NSB NHibernate context, which is serializing Dictionary<string, string> and List<OutboxOperation> which is like a poco class.


no more `inline-styles` add this to your `style.css` whether by adding a class or scoping to select this element properly.


Yeah, it was originally generated against the commits in node-private and I forgot to update them when everything pulled over.


canola typo


yeah I must be on an outdated version (: ... this was the version off of the download page from 2 weeks ago.


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


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


Has this been cherry-picked into the 2.0 branch? I didn't see it in there, or the 2.0.x release notes.


I just know it was confusing to a group of newbies we were teaching here, as the Arduino IDE creates the file with the ".bin" extension ... and they didn't realize why the script was failing when they put the files on the server.  So we had to edit the script.

I do understand your thought @rherget on how you get to choose the filenames, but in that case there should not be *any* extension in the script ... and the array should contain the full filename and extension.

The PHP script seems to insinuate there is a ".bin" extension ... but then it fails to include the "." and so it breaks.


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2107 of 2107 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

![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)


I don't have a strong preference, but I lean toward marking things explicitly if I definitely feel they should be inlined. Like the other one, this one's up to you, though. :-)


We have a similar one: https://github.com/ably/ably-python/blob/35446180719433da3e339954e1ebdf0e7c42baa9/test/ably/resttoken_test.py#L132-L136


Good idea. I will submit a PR for both `dev` and `RCBugFix` enabling this by default in all configurations.


Oh, so you wrote this one in Python?


No the first one is correct (NavbarController) what I meant was the word after "as". It will be the name of an instance of the controller. So it should start with lowercase letter (navbarController), whereas the controller name is the definition of the controller not an instance name and that's why it starts with an upper case (NavbarController).
So in summary, you should always have your controller instances defined this way: "NavbarController as navbarCtrl".


When will the refactoring work land?


Testing mention again @sktguha


that's what `--force` is for!


Looks good.


Ah, I figured it out, it's to supress the 'no handlers for yapsy'  because yapsy plugins get loaded before the main function is called which initializes the logging facility.


Hate this code so much :)


Hey Rita,

In general I think you are very close. Some nit-picky things are,
- remove files (eg normalize.css) and code that are unused since it adds size to your data.
- keep your indentations clean and neat (you can try utilizing a codeformatter plugin in your sublime).
- keep in mind that any parent that has floated elements as its children, you would do declare a CSS property called `overflow: auto` or `overflow: hidden` for the parent to account for its children's height.

Other than that, :100:

Stephen


Update 3


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


fisher-yates shuffle
```
function shuffle(&$items, int $seed)
{
    random_int($seed);
    $items = array_values($items);
    for ($i = count($items) - 1; $i > 0; $i--)
    {
        $j = random_int(0, $i);
        $tmp = $items[$i];
        $items[$i] = $items[$j];
        $items[$j] = $tmp;
    }
}
```


Just use `HW_PERIODIC` here.  Don't think that the intermediate variable helps.


the `--force` is strong in that one


I think you mixed up the pluralities here ;)


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


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


You're passing Plurality.SINGULAR to both methods, this one should be plural!


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


I don't understand this call.  Why do we have anything other than a `snd_activate` equivalent?  Maybe a relic of this being outdated by our previous conversation on slack.


Hey there. Is this change taken from a patch on d.o? If not, would you mind submitting a PR here or uploading a patch to d.o so that it can be reviewed? Thanks.


Be careful when using quotes in the meta tags, quotes should be either double quotes like this " " or single quotes like this ' '. Quotes in this format “ ” are not allowed.


@JonathonReinhart as I said in my [previous](https://github.com/SUSE/Portus/commit/1e77fee92ef4f77272e1525505d8c2d36d329fba#commitcomment-14991374) comment, we did not include it because it added a new dependency and we were releasing 2.0. Patch-level releases should not change dependencies either. Sorry for the inconvenience.


added


The font link tag should come before your css stylesheet. In this case it would come before style.css


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


I'd leave off the `:- schema/Bool` and just use `when-let` here, since `nil` is "falsey" and a `?` at the end of the name makes it obvious it's predicate.  However that is not a convention or anything formal, so this is fine.


Hi @Halkreen , maybe you can help us translating this file? Thanks in advance


Overall good job on the hw. Please refer to the comments for best practices.


user is undefined. Change to steamIds[index].nickname!


this is unused


Consider using a span tag in this case instead of the div tag.


@mssola Ah okay, I thought you meant just 2.0.0, not 2.0.x. Not a problem; I will move my test environment over to using RPMs built from `master`, and look into doing it for production also. Thanks for the quick response.


fucjq


bitch


Can't return without post.


[pubnub] Upgrade to pubnub-3.8.0


Overall good job on the hw. Please view the comments for best practices. One thing to keep note is to use classes as much as possible when selecting elements in css


What is the purpose of this? This is a very subtle change to a core bit of code.


- create a volume-specific interface


아.. 제엔트로피설정이 너무크게되어있어서그랬습니다. 뭐지금은해결했습니다.


Isn't the HW cap quite small?  Wouldn't it be 16B?


Natural context of the function called by this formatter will be the current object on which it was defined. If model is
```javascript
{
  obj: {
    name:'foo',
    getName: function(){ return this.name }
  }
}
```
And a template
```html
{ obj.getName | call }
```
The result must be `foo`. So `getName` context has to be `obj`. The code is here to guarantee it. It works with this [code](https://github.com/mikeric/rivets/commit/02cfbddf7dcd195051551f0e40468a8d1eaf422c#diff-439efcdcbce24442e08410304419ce97R68)
Hope it's clear


hgfrtyi


Wait, if the function name is "printParam", then shouldn't it be print x after all instead of spit x, cause spit x is just returning the x?


fuck u


Consider using a class attribute instead of an id. It is best practice to use id's for special circumstances.


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.59](http://localhost:9090/viewLog.html?buildId=556&buildTypeId=CommonProvider_CoreCiRelease&guest=1) is now running


[A Course for Confidence - Responses.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/123998/A.Course.for.Confidence.-.Responses.txt)
[A Course for Confidence.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/123996/A.Course.for.Confidence.txt)
[Week 1 Anchor Point.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/123997/Week.1.Anchor.Point.txt)
[Week 1 Anchor Points - Responses.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/123999/Week.1.Anchor.Points.-.Responses.txt)
[Week 1 Call to Action - Success! - Responses.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124000/Week.1.Call.to.Action.-.Success.-.Responses.txt)
[Week 1 Call to Action - Success!.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124001/Week.1.Call.to.Action.-.Success.txt)
[Week 1 Development.txt](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124002/Week.1.Development.txt)
[NovakBrandy_Time Estimation.xlsx](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124003/NovakBrandy_Time.Estimation.xlsx)
[NovakBrandy_BurnUpList.pdf](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124004/NovakBrandy_BurnUpList.pdf)
[NovakBrandy_Project.pdf](https://github.com/blnovak/Novak_Brandy_Portfolio/files/124005/NovakBrandy_Project.pdf)


These are commented out in the frontend controller. Let's be consistent and comment them out here also.


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


Consider using a class attribute instead of an id. It is best practice to use id's for special circumstances. Also id's have a higher specificity than classes meaning that they can override any class or link styling.


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.59](http://localhost:9090/viewLog.html?buildId=556&buildTypeId=CommonProvider_CoreCiRelease&guest=1) outcome was **SUCCESS**
Summary: StyleCop violations: 229, Tests passed: 68 Build time: 00:00:51


This example doesn't belong in /examples. It belongs in docs/user-guide. It also needs a README.md


Week One - Project and Portfolio I: Development Activities


113360729_HumidifierOASys (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1526 of 1526 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.7%25-green.svg)


y


Wow, I didn't know you could draft a Github Issue template like this. Totally going to use it.


Overall good job on the hw. Please view the comments for best practices. A couple of things to keep note: consider the use of id(s) in special circumstances and use classes as much as possible to select elements in css.


block


Redid pretty much the entirety of CreationGUI, not finished but the layout is somewhat getting there.


What about monitor operations that find the connection active?


Fixed this to get files automatically.


"With that in mind, the solution we have been using to some extent here at Gilt is to provide deployment notifications in a configurable way, as a service, by talking directly to Codedeploy" - would be great to get this statement earlier in the article, so that the reader knows what the article is really about.


Will do this soon.


notebook -> system


...the notebook's JavaScript...


This is unused. rebase leftover.


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


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.60](http://localhost:9090/viewLog.html?buildId=559&buildTypeId=CommonProvider_CoreCiRelease&guest=1) is now running


Started the remaining work needed for #8


should be replicasets


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.60](http://localhost:9090/viewLog.html?buildId=559&buildTypeId=CommonProvider_CoreCiRelease&guest=1) outcome was **SUCCESS**
Summary: StyleCop violations: 229, Tests passed: 68 Build time: 00:00:35


- create a volume-specific interface


sasdadasda


Overall good job on the hw. One thing to keep of note is to consider how specificity affects styling in css. Sometimes it is best to assign a class to an element inside of using nested selectors - your footer styling would be a good example to apply this practice.


Is this repo going to exist for the foreseeable future? because deleting and re-forking repeatedly is a garbage workflow for anybody trying to use your repos.


Shouldn't this be a coroutine?


zip


I thought scale didn't work yet?


And these two as well?


Overall great job on the hw. Keep up the great work.


Is kubectl scale implemented using the scale subresource, or does it just update the resource?


5429_ConstructionInternalSource (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2061 of 2062 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.

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


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


Yes.


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (973 of 973 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-28.21%25-green.svg)


This was tested. I ran $.consoleLn to ensure that things were set behind the scenes even at boot-up of the bot as well, which I took out of course before checking in.  Also, tested changing command values with user in chat, had attached output from bot.


Your google fonts link should come before your css stylesheet. In this case it should come before the style.css link tag.


I don't think we need to retest general expose functionality here.


Coool

On Feb 9, 2016 7:35 PM, "David Vandenburgh" <notifications@github.com>
wrote:
>
> Yes.
>
> —
> Reply to this email directly or view it on GitHub.


@havocp @mattpap if there is a better way to mark this without code, I am open to ideas


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


We're eating up a lot of instruction cache footprint here.  I think that at the very least, we can just have an unconditional `jmp` after we call the C function to a single location that restores registers, and returns.

You can add this later as it is low priority.  The code is quite clean now.  It will just use more memory than we want.


If [`sd_Info`](https://github.com/pLeBlanc93/ArcREST/blob/5df14923fe99abebeeda40229c3d1380da06274a/src/arcresthelper/publishingtools.py#L1445-L1453) is `None` (occurs when the output from arcpy.mapping.CreateMapSDDraft() has [errors](https://github.com/pLeBlanc93/ArcREST/blob/5df14923fe99abebeeda40229c3d1380da06274a/src/arcrest/common/servicedef.py#L98-L106)), then attempting to access keys from `sd_Info` will fail.

Unsure if the [message given a few lines later](https://github.com/pLeBlanc93/ArcREST/blob/5df14923fe99abebeeda40229c3d1380da06274a/src/arcresthelper/publishingtools.py#L1470-L1477) that does check for `None` needs to be updated to reflect this.


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.61](http://localhost:9090/viewLog.html?buildId=573&buildTypeId=CommonProvider_CoreCiRelease&guest=1) is now running


uo


TeamCity Common Provider :: CommonProvider => CI [Build 2.0.61](http://localhost:9090/viewLog.html?buildId=573&buildTypeId=CommonProvider_CoreCiRelease&guest=1) outcome was **SUCCESS**
Summary: StyleCop violations: 229, Tests passed: 68 Build time: 00:00:38


Thanks!


Overall good job on the hw. Please refer to the comments for best practices.


@xfurry small bump to this:

(...) i have working part with npc plays random sound everytime wee'll use item(to wake him up)
 i fixed it using eventAI for event play sound on spellhit ...

- At the moment this wont work because sd2 script exist for this npc so:

•can we update sd2 script... that we can use sd2 and EAI together?
•or you prefer to do that directly by sd2??


small note:  line up the `=`s


@hoffmabc So, for me the app is bombing on the return in line 82, saying can't return outside of a function. I wonder if this is happening for others?

If I comment out the line it works. I believe I have actually returned from a module before, but only for debugging purposes, so I would think it's possible, but not totally sure. Maybe the fact that it's happening in the entry point is problematic?


Also the sequence of commands to insert the space could be factored out, I guess, but I'll await your decision re: this choice first.


![coachuphomepage](https://cloud.githubusercontent.com/assets/17131542/12935595/f3fbc566-cf64-11e5-8aa2-01e51e4eae92.png)
Homepage of CoachUp.com


Wrong selector. The rs used `app: guestbook` and `tier: frontend`.


@thinkyhead
How did you find this? I was almost sure it vanished during one of these 'clean ups'.


This needs to be compared to the compile-time value of `CFG_BOOTSTRAP_KEY` ala [https://github.com/rust-lang/rust/blob/master/src/librustc/session/config.rs#L1176]. Ensuring they are equal prevents people from setting `BOOTSTRAP_KEY` to any value to activate the feature.

Can you also mention in the comment that we're preventing its use out-of-tree because it's needed in-tree but not ready for production?


New version of CoachUp homepage with auto match link.
![homepage w post a job flow](https://cloud.githubusercontent.com/assets/17131542/12935613/1a90e3f0-cf65-11e5-814c-6bd7cba9420d.png)


JSON --> YAML


For now, yes, lets do it here.  We need to talk to Luke about the right way to design this.


Понял. Это невнимательность моя.


I think `accept` makes more sense -_-


Perhaps in all examples we can replace rc with rs as the new "replicated nginx" kind of things.


Sexy...


This line should be changed to if (error && error.error === 'invocation-failed') to allow the test to pass.


Thank


Do we need the GetIsDynamic check here? Is this just looking for the async function's home scope?


I see you are attempting to license your project on a different license than rswier/c4 (GPLv2).  As AMaCC contains substantial rswier/c4 code, please change your licence to GPLv2.


Thanks for the tip.  Though I've been messing around with Python for quite some time, I'm very new to Git.  Gitignore files were not included in any of the git tutorials I looked at.  Seems rather critical, since Eclipse generated over 200 files, and I imagine other IDEs have similar behaviour.

I might delete and re-fork the repository to avoid having all that crap in any commit and merge I'm associate with, though it looks like the project has 2 contributors that have a lot more coding time than myself!


Since the function returns `FALSE` to indicate the buffer was too small, this means it'll fail if the value is larger than the initial buffer (not that a user profile directory should ever be bigger than 512 but not a nice bug to have regardless).


This only holds if the users don't set `--selector`, right?


Can you update the usage of `selector` flag too?
```go
	cmd.Flags().String("selector", "", "A label selector to use for this service. If empty (the default) infer the selector from the replication controller.")
```

Since it's not replication controller only.


This is actually _extremely_ dangerous, as if Windows returns a value other than 0 or 1, you now have an invalid enum instance which can trigger undefined behavior.


Oh gosh, my mistake.


tmrudick@


Nice to see `systemd-activate` get new features.


I really need to make this script smarter


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


fixed in a85b918e9f3a870e4de7c90617531d1782519d59


We need to set `sudo: false` to use container based infrastructure (and caching). It depends on when the repository was enabled on Travis. See https://docs.travis-ci.com/user/workers/container-based-infrastructure/#Routing-your-build-to-container-based-infrastructure


Maybe rename this to something like frontend-replicas.yaml or frontend-replicasets.yaml?


Same question for the file name here.


5452-controllerWaterCoil-IP-units (MatthewSteen) - i386-Windows-7-VisualStudio-12: OK (2061 of 2061 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

![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)


Sometimes it's more effective to be condescending than mean. Effective at what I'm still not sure...


what about the following idea:

		//opacity -= 0.125;
		opacity -= 1 / keys.calendar.maximumEntries;

Works better than your attempt (limits calendar entry to 8(1/8=0.125)


This looks great, @ryanjhodge! Any chance we could change the text input to a select? Or is that not a constraint we can impose?


You are using a non installer version so that could be it. I will test and fix.


I guess gray is ok :)  ++


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


Is this needed though?
I thought one of the things I'd done was to have the same thing happen from within the proxy code.


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


@drewrygh hey! check out this: https://github.com/driftyco/ionic/pull/5363


refactored to qs and abstracted out keys to gitignored file


refactored


not `remote_node_up()`?


will handle as we integrate to front-end


I think in memory solution with an additional persistent disk storage solution would work great.

So it seems like, if we used Postgres, we don't need to have a special process for invalidation server.

Am I correct?


all comments addressed


Nice, why do you think it reduced the strong cubemap reflection?


Thanks @topaz. But, please, satisfy my curiosity... in which environment do you have to use such an old Perl? Or are you suggesting this just because it's a weak dependency?


Personally I think it is good to have it be a separate service so that it can be scaled independently, but I can totally imagine a situation where you can reasonably run it in-process.


There's a couple more places this needs to be done; they're all of the form `next(x.__iter__())` for some `x`


Can you elaborate on why the instantaneous value won't work for HPA? It seems like the monitoring shouldn't be doing smoothing as it hides potentially valuable data. Rather, the consumer (HPA) of the data should apply any necessary transforms? Also, I'll schedule a VC to work out any remaining issues tomorrow morning. /cc @vishh


Status should not be exposed anywhere. I think it's used in some places internally, because I didn't want to change more than needed to get the API right, but it's not used at the API level, as it is not defined the spec.


The test runner already has a -v argument for this, i.e.:
mono appdomain-unload.exe -v


test


FWIW since this is in BSS quiet will be zero anyway, but initializing it anyway makes clear you intend to use it so no big deal.


It's computing the right envIndex for the async function's outer function home scope.  The dynamic check is because FindScopeForSym will return when it hits a dynamic scope, but I want it to go all the way to the symScope.

Perhaps this logic should be pulled out into a ComputeEnvIndex function for clarity?  And maybe that function can copy the code in FindScopeForSym but do away with the dynamic scope early out?


5429_ConstructionInternalSource (mitchute) - i386-Windows-7-VisualStudio-12: OK (2061 of 2062 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: ZSZ big diffs.
 * 1 test had: Table big diffs.

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


I did not understand the instructions for this step. Thanks for the feedback!


That's how it was initially, actually. I want the top left and top right
displays to be as symmetrical as possible. Day & Date = Wind & Sun, Time =
Current Weather, Traffic = Verbose Weather, Calendar = Forecast. There are
a few exceptions, like Apparent Temperature, Traffic displaying only at a
certain time, but for the most part, it's symmetrical. By matching the
opacity steps of the Calendar to the Forecast, it furthers the symmetry,
e.g. the second Calendar entry has the same opacity as the second forecast
entry, regardless of the number of Calendar entries. That's my thinking
anyway, I encourage everybody to customize theirs as much as possible.
On Feb 9, 2016 7:50 PM, "Dominic Spatz" <notifications@github.com> wrote:

> what about the following idea:
>
>     //opacity -= 0.125;
>     opacity -= 1 / keys.calendar.maximumEntries;
>
> Works better than your attempt (limits calendar entry to 8(1/8=0.125)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/leadlag/MagicMirror/commit/9e4c9505196a27a113a17a6b4c5c5eafab91c14a#commitcomment-15993960>
> .
>


The following doc(s) have been deployed to the [envconfig](http://envconfig-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/environment-specific-config](http://envconfig-static-docs.pantheon.io/docs/articles/wordpress/environment-specific-config)


- [x ] Use Fisher-Yates Shuffle


This will never be faster than simply "return fabsf(i);" on modern compilers/CPU. There are two issues with this:

1) because of aliasing two different types to same memory location, it will actually impact compiler optimizations. Compiler will optimize worse the code that uses this function.

2) On modern CPU can get absolute value on floating CPU register with one instruction easily (andss/andps for SSE reg), compiler will do that automatically. But if you cast to int type, compiler will move float register to regular CPU int register to perform your and. And then back again to float. You'll loose performance this way.

Here's simple example demonstrating issue: https://gist.github.com/mmozeiko/d8a16d0adc393059a1a6
In real code it will be much worse. Because sometimes compiler can join fabsf operation with some other code that uses it. But if you use this kind of casting, it won't be able to do so, because compilers typically give up if too much aliasing happens.


Ya orange is not very good for that. I tried with orange first.


Much better thanks


@cbattista you can take a look at these now (10 bins, < 20 rt, no blank answers)


np thx


Derp, yeah, the check for symScope supersedes the dynamic check.  I'll fix that up :)


5452-controllerWaterCoil-IP-units (MatthewSteen) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2093 of 2093 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

![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)


I think you mean *preposterously* old Perl.  Old CentOS 5 machine.  Fixed because the dependency was weak and easy to remove.


...didn't I write most of the stuff in this file? I could've sworn I did.


Use `cookie: { secure: true }` only if you are using https


Hi @fallenby,

Commented on [here](http://mattermost.uservoice.com/forums/306457-general/suggestions/11632551-see-who-s-online-in-a-channel) but realized it might be easier to comment directly in GitHub, would you be able to add some screenshots so we can take a look?

We're thinking about adding something similar and it would be interesting to see the design you went with.


It's basically the best thing ever.


we don't want to use the Lollipop MMS APIs for 21 any more?


This is incorrect.


oops.. so apparently want to use something like:
git update-index --assume-unchanged example.txt


password changed - so config file is not accurate


Don't export IFS.


5452-controllerWaterCoil-IP-units (MatthewSteen) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Without the sed transformation of %2f the terminal title isn't set correctly.


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


I think "full domain" is confusing here, "default root domain" worked in my opinion.  For example, "cfapps.io" might be the default root domain you want associated with your apps, then when you push "myapp" it will get the route "myapp.cfapps.io" associated to it.


```hideDefaultChrome = _previews.Any(p => p.HideDefaultChrome);``` ?


2 of my radials work, and one does not, grrrr. Must find mistake!!


coming along well - would things look totally insane with we matched the x
axis width?  or perhaps even better, overlay the sp timed/sp untimed/vs in
the same plots?  just trying to think of some ways we could quickly compare
the distros.

On Tue, Feb 9, 2016 at 5:01 PM, Jonathan Nicholas <notifications@github.com>
wrote:

> @cbattista <https://github.com/cbattista> you can take a look at these
> now (10 bins, < 20 rt, no blank answers)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StanfordCosyne/math-intervention-api/commit/4931864aa5649f1ee66dcf2727d5c583f622a097#commitcomment-15994108>
> .
>


You like beer, I like to drink beer! What a match! ;)


I like switching to this numbering scheme.  For consistency can you do this for the vsphere, bosh-lite, and openstack deploying landing pages as well?


You do not need bosh-init for this setup step.


Thats a good idea.
As you said, everybody should do the customization on his own.

I am really impressed by the traffic feature. I might implement this to my repository.
(some features of my smartmirror/magicmirror: Configpage,Localization,notifications from my smart home)

But i got a question, that you might answer:
Do you think it could be possible to crop a certain calendar, showing only the next 2 events, or only a few days in advance.
The reason is, that i don't want to see every entry of my waste calendar, provided by the city.


The following doc(s) have been deployed to the [gitfaqsau](http://gitfaqsau-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/git-faq](http://gitfaqsau-static-docs.pantheon.io/docs/articles/local/git-faq)


This could  still be made faster by caching the most recently-checked domains


Do not ever change values of variables that you're not supposed to.


I'm done importing the masters DB. I made a copy in /home/ubuntu.


It's not correct to say installing CF requires this many VMs.  There are an infinite number of ways to deploy cloud foundry, the instructions here give one "standard" deployment topology, which happens to require more VMs.


Why do this?


It's still missing the dev doc, so no PR, but @dcleao can you have a quick look?


If the shooter isn't ready to shoot, you probably want to turn off the intake


Let's definitely *not* be in the business of duplicating docs from a different site!  What happens when their docs change?  We should not add this page, and simply explain on the main landing page that they should follow the instructions on bosh.io, but not follow the AWS setup steps.


misspelled!


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


To be honest, as an engine developer, code that specifically calls out a platform like this is never a good sign. Ever. Prefer handling things like this in a data driven way through an external platform configuration or something. What happens when react native decides to support a new platform? do we do a massive search of all instances of the string "ios" everywhere? Code like this implicitly forks the entire codebase for one platform, and in a way that is opaque and hard to reason about.


Interestingly enough, these code paths could not be hit when using `vfs.dest` because `prepareWrite` assigns a `fs.Stats` object if it is falsey.


THis is my first repository


@erikkemperman @piranna is this correct behavior?  It seemed weird that we would error if we can't open the file since the open is only to update metadata.


Let's say "with a PEM-encoded RSA private key" and "the corresponding PEM-encoded RSA public key"


why not just change the above two to

`taper = Variable('taper', taper, '-', 'Taper ratio')`
`Lmax  = Variable('L_{max}', Lmax, 'N', 'Maximum load')`
? why are these coming in as Variables?

@bqpd need help on design intent here


The `while...end` isn't needed as far as I can tell. It definitely isn't needed for the only use case that currently exists (converting $PWD). It also doesn't make any sense to give it more than one line of input in any case as it drops the newlines which probably isn't correct even if we allow more than one line.


These blanket impls will prevent people from manually implementing `MirPass`, I believe?


I delete this part because its repetitive when I run the Java


It does if you just negate the yaw. That's how it's done in Transform.


no worries -- update in the code if you haven't already.


You could have just done:
```
$adapter = $this->driver->getAdapter();

if (! $adapter instanceof AwsS3Adapter) {
	throw new RuntimeException('This driver does not support retrieving URLs.');
}

return $adapter->getClient()->getObjectUrl($adapter->getBucket(), $path);
```


these changes are modeled after @jread-usgs 's changes (https://github.com/krlmlr/r-appveyor/compare/master...jread-usgs:master)


This makes me nervous. It's important for correctness that this runs at the end -- so why would we leave it to chance? I feel like I'd rather have the passes just arranged in a fixed order, perhaps with some "meta-passes" inserted that run passes registered from plugins. I guess I'm not really sure if any particular pass can decide on an absolute priority all by itself. I expect people will be jiggering these things constantly to achieve a particular ordering.


重写


hnnnnnggggggg

```
                    ##        .
              ## ## ##       ==
           ## ## ## ##      ===
       /""""""""""""""""\___/ ===
  ~~~ {~~ ~~~~ ~~~ ~~~~ ~~ ~ /  ===- ~~~
       \______ o          __/
         \    \        __/
          \____\______/

```


Shouldn't this be 你好?


I just cleared code style warning. `!_.isEmpty` is the same as `_.nonEmpty`, is not it?


Your three_steps.py seems to be named incorrectly.


The google fonts link should come before your css stylesheet. In this case it would come before the style.css link tag.


fix-missed-date (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1525 of 1525 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.7%25-green.svg)


is this reactive


lol


5452-controllerWaterCoil-IP-units (MatthewSteen) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (959 of 959 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-28.69%25-green.svg)


@jdwarner This unit test still fails.  That is because as soon as the input is passed into the `input` property, it is fuzzied.  When the read is done on this line, a defuzzification is occuring.  Not sure if a fuzzy variable should resolve back to the crisp value originally used to define it.


Now you can store your data in the butt!


Avoid using inline css. All of your css styling should be in your external css stylesheet.


Consider using a class instead of an id in this case.


# THE FUTURE IS NOW


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


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


Fuck off. If your settings don't default to what we have in mtndew.dll then we're just going to change your source code anyway before we release.


quality


Using a fileReader to get a dataUrl did not work. The dataUrl was somehow broken.


Removed AHA Number references, not required for Australia


if you are using the git bash (most people recommend) then you should still use init.sh as far as I know.


This should be drivetrain.solenoid.driveShifters.port.a and drivetrain.solenoid.driveShifters.port.b


This is my new repository


Overall good job on the hw. Please refer to the comments for best practices. A thing to keep not is that you should consider styling using classes as much as possible which will allow you to avoid nested selectors. Your aside styling would be a good example as to when you should apply this strategy.


Make sure you change these to match my above comment as well


Hi @elagarrigue, I'll try to translate it all ! What if the french version is longer than the English version ? Will you have displaying issues ?
Cheers


This comment on this commit


shouldn't this be
git **clone** https://github.com/rmff/laravel-homestead-32bit.git Homestead32Bit
?


Just to note, I deleted a poorly named jpg file from the repository and had to pull to my local machine.


It is now always doing exit 0 because ImagicalMine.phar is not removed before the build is started.
Thats why it was using the $version.
You have two options here, revert back to use $version or remove ImagicalMine.phar before it is going to build a new one.

I prefer the first option, then you can hold some old builds and if there is a problem, you can easy go back to a earlier build.
Thats why I do not use this build script and use my own. It adds a date/time to my $version. Yet I go implement the latest GIT commit as reference and make a ImagicalMine-$version.txt with the link to the github commit so I know when I build the phar and when I can avoid a problem and can go back to a older phar that I want before a problem :-)

_As in Mr Spock's mind: Think logical._


Change this to upshiftButton.whileHeld


I would rename this upshiftButton


doesn't \w allow many more characters than you want to test for? I usually just do [a-zA-Z0-9] to be explicit.


1) This should be a button on the left joystick, not on the gamepad
2) I would just name it button.shift


I don't understand this part. Why would we want to concatenate the same name multiple times? Wouldn't it be better to rename `--fdname=` to `--fdnames=` and allow multiple colon-separated values (that was possible before, but undocumented, IIUC)?


We haven't named any of the other buttons with "button" on the end, so for consistency let's keep it `upshift`. But yes, `whileHeld` is the correct action instead of `whenPressed`.


Your "first flight:" key needs to be one token.  That is to say, we need to remove the space between "first" and "flight".  I'll accept the PR to get litreview1 in, but if the space comes back in, I'll remove it again and try generate my own pull request to bring the change into your fork.


Yeah, I will do that.


Would you mind sending me your email? I don't want to publish mine out to
the world
On Feb 7, 2016 3:14 AM, "leadlag" <notifications@github.com> wrote:

> In the Keys.js file, did you retain the single quotations that surround the
> API keys, address, etc? Did you retain the place_id: prefix for the origin
> and destination for traffic? If so, it's likely some other formatting issue
> with your keys.js file. If you're still having problems, could you either
> send me your keys.js file, or a screenshot of it?
>
> On Sat, Feb 6, 2016 at 10:51 PM, Scott Rodgers <notifications@github.com>
> wrote:
>
> > I have tried to run the latest version of this code and am having issues
> > with 2 ajax calls (one going to weather.php and one going to
> traffic.php) I
> > was able to recreate successfully the call that should be going out to
> the
> > google directions api and had good results with the values in my Key.js
> > file. Any ideas?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/leadlag/MagicMirror/commit/9e4c9505196a27a113a17a6b4c5c5eafab91c14a#commitcomment-15936309
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/leadlag/MagicMirror/commit/9e4c9505196a27a113a17a6b4c5c5eafab91c14a#commitcomment-15937490>
> .
>


Cool thanks. :)


"stop"


LGTM


I'm not sure what this sentence means. Were you going to add another clause?


Hi Kevin,

Actually, although I still think this PR is worthwhile, I made a mistake. We are using `pan.xapi`, not `pan.http`. I thought `pan.http` was a transport layer for the everything in the library but I later realised that `pan.xapi` has its own http code.

The important thing we need to support is using a client cert. We are interfacing to Panorama using a reverse proxy, which is protected by a client cert. I saw that `pan.xapi` supports passing in an ssl context, which can be used to set a client cert.

Although I am personally interested in your library, at this stage the team implementing our Panorama integration look to be preferring to make their own requests using python requests.


"reloads"


configuration


:+1: great job!!!


fix-missed-date (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2075 of 2075 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.

![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)


Or two more lines shorter :smile:

```php
if (! ($adapter = $this->driver->getAdapter()) instanceof AwsS3Adapter) {
    throw new RuntimeException('This driver does not support retrieving URLs.');
}

return $adapter->getClient()->getObjectUrl($adapter->getBucket(), $path);
```


:-) whole jenkins is slow... think about move the service to a better country.... I can't open/download from jenkins at all.


Can you try and rewrap this?  The newlines got pretty borked


Please use spaces instead of tabs. Soft tabs are always preferable to hard tabs. In cloud9, enable the soft-tab option and have it set to 4 spaces.


forman


Or how else can we share the file. I did include the single quotes and
place id though
On Feb 9, 2016 8:46 PM, "Scott Rodgers" <srodger@g.clemson.edu> wrote:

> Would you mind sending me your email? I don't want to publish mine out to
> the world
> On Feb 7, 2016 3:14 AM, "leadlag" <notifications@github.com> wrote:
>
>> In the Keys.js file, did you retain the single quotations that surround
>> the
>> API keys, address, etc? Did you retain the place_id: prefix for the origin
>> and destination for traffic? If so, it's likely some other formatting
>> issue
>> with your keys.js file. If you're still having problems, could you either
>> send me your keys.js file, or a screenshot of it?
>>
>> On Sat, Feb 6, 2016 at 10:51 PM, Scott Rodgers <notifications@github.com>
>> wrote:
>>
>> > I have tried to run the latest version of this code and am having issues
>> > with 2 ajax calls (one going to weather.php and one going to
>> traffic.php) I
>> > was able to recreate successfully the call that should be going out to
>> the
>> > google directions api and had good results with the values in my Key.js
>> > file. Any ideas?
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/leadlag/MagicMirror/commit/9e4c9505196a27a113a17a6b4c5c5eafab91c14a#commitcomment-15936309
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/leadlag/MagicMirror/commit/9e4c9505196a27a113a17a6b4c5c5eafab91c14a#commitcomment-15937490>
>> .
>>
>


Why has this been removed?


sudo gem install foreman


Why was this moved above `execute`? I think it's nice to have methods in alphabetical order, for easy searching.


dd


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


LGTM


Please remember to also make a new branch for each feature/pull request you wish to merge. Don't worry about coding time, he best way to learn anything is to use it, and that's exactly what you'll be doing by contributing.


I wasn't sure if it were ever possible to override the "type" name in the parent scope with a variable.  I tried my best to always prefix things with a _G.  I suppose since these are helper methods at the very beginning of a file, there is less of a risk.


Also, what I was intending to do is, in our code inherit from the `PanHttp` class and override the `_http_request_requests()` method so that certain kwargs could be injected before using `super()` to call the library version of `_http_request_requests()`. Can I also say, it seems odd that both urllib and requests are implemented.


Updated.


I'm leaving it in as a dummy cause someone told me the command name got locked in, keep me posted with updates.


This is correct


@cbattista @TeresaI @smitsven
Pretty clear trend here -- timed < untimed < vs
Also the number completed (see previous commit) wasn't as bad as I thought overall.


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


At this point I hesitate to comment because I know there is a pretty significant change in building the docs with 7.1.beta2. But this is in prefix and the pdf-doc fails. So excuse the noise if it's known.
```
[manifolds] writing... done
[manifolds] copying TeX support files...
[manifolds] done
[manifolds] build succeeded.
[notebook ] reading sources... [ 65%] sagenb/notebook/interact
/bin/sh: 1: all-pdf: not found
[tensor_fr] reading sources... [ 52%] sage/tensor/modules/free_module_automorphism
[repl     ] reading sources... [ 71%] sage/repl/rich_output/backend_sagenb
[homology ] reading sources... [ 52%] sage/homology/examples
[graphs   ] reading sources... [ 12%] sage/graphs/base/graph_backends
Error building the documentation.
Traceback (most recent call last):
  File "sage_setup/docbuild/__main__.py", line 2, in <module>
    main()
  File "/storage/strogdon/gentoo-prefix/var/tmp/portage/sci-mathematics/sage-9999/work/sage-9999/src-python2_7/sage_setup/docbuild/__init__.py", line 1620, in main
    builder()
  File "/storage/strogdon/gentoo-prefix/var/tmp/portage/sci-mathematics/sage-9999/work/sage-9999/src-python2_7/sage_setup/docbuild/__init__.py", line 284, in _wrapper
    getattr(get_builder(document), name)(*args, **kwds)
  File "/storage/strogdon/gentoo-prefix/var/tmp/portage/sci-mathematics/sage-9999/work/sage-9999/src-python2_7/sage_setup/docbuild/__init__.py", line 491, in _wrapper
    x.get(99999)
  File "/storage/strogdon/gentoo-prefix/usr/lib/python2.7/multiprocessing/pool.py", line 567, in get
    raise self._value
RuntimeError: failed to run $MAKE all-pdf in /storage/strogdon/gentoo-prefix/var/tmp/portage/sci-mathematics/sage-9999/work/sage-9999/src-python2_7/doc/output/latex/en/reference/manifolds
```
A `make` or `make all-pdf` in the manifolds folder produces `manifolds.pdf`. Actually I copied that folder elsewhere and the `makes` worked there. I find the `/bin/sh: 1: all-pdf: not found` worrisome. Is something not using my prefix `bin/sh` or is something else up? Of course parallel building probably obscures things.


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


2045-draw-dual-ducts (kbenne) - 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)


I'm curious, what sort of GitHub feature lead you to my WIP branch?  I doesn't even compile yet :grinning:


boost 1.60 not seem to have those


Please include the code for servo motor in gcodecncdemo2axis


Nope, can’t do that: other plugins read the value for different purposes, because it is guaranteed to be there.


Ideally these two lines of additional whitespace would be in a separate commit.


Doing addcslashes($exp, '/')


after thinking about it little more, I got into conclusion that you are missing the 'user' as parameter of this method.


change the last line with **** and author.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.674.899956424b34854311d6a3d8912135d7c2181030](http://build.palaso.org/viewLog.html?buildId=69053&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:07:53


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1040.899956424b34854311d6a3d8912135d7c2181030](http://build.palaso.org/viewLog.html?buildId=69052&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:08:02


@tvsonar this is a bit disturbing, I'd rather see a method named `IsInEqualsOverride()`. At first I thought this call is redundant due to the `IsMethodDefinedOnObject()` call above.


Always having fun :) doing addcslashes($pat,'/')


@sils1297 Actually I wanted to do it that way only but the comment size was greater than 50, so I escape the spaces. Secondly I thought that the shortlog message should be same as that of the issue name. But I will keep all these things in mind from the next time.


@sb8244 do you have another example that is not ui-select? If you have one, then it would be nice to get one. The reason I am asking for another example is that ui-select is broken as it made some assumptions that were false even before Angular released 1.0. Here http://plnkr.co/edit/sRIW4LJ2VpSBJZAnODvY?p=preview is the proof that ui-select breaks with Angular 1.4.

> Whether the core team thinks [that isn't how it should be used] does not impact how people are using it and expect it to work.

I find that statement very unfair:
* The behavior is documented https://docs.angularjs.org/api/ng/service/$compile#-templateurl-. Developers are free to read or not to read the documentation, but when there are discrepancies between what the assumed to be true and how things work, they are not free to ignore that the behavior __is__ documented
* This change happen in a new version release, not in a patch release. Some breaking changes are needed to make things faster and to fix some bugs that need some underlying changes
* People are free to upgrade to the latest release or stay with an older version, the same goes for all the other libraries that developers are using

Now, the Angular core team is doing many underlying changes trying to improve the experience for all Angular apps while keeping the API and behavior stable. When things break, effort is made to fix it. When some behavior is not properly documented, we do so. The issue you are raising does no fall into any of these categories as it was possible to trigger this behavior before Angular 1.5 and the behavior is documented.


Is there a strong (domain-related) reason you went for the longer method name rather than just `resident_in_third_country?`. I'm not convinced the longer name adds anything and I think it reduces the readability of the code.


Should this also go into the xcomm_zynq branch?


Working


ah nope, an issue can be related to multiple commits so there's no such obligation : )


ze nemusis ratat, na ktorom riadku si


잘햇네


ak robis aj v if aj v else VLAN=VLAN+1, radsej to urob mimo if/else.


Presumably all the regression tests were passing at this point. If so, I think it would be worth saying so in the commit note.


yeah. in this case it still counts as "one line" to me :D


Missed a spot. See a97414e335c24a5b10b1a48538afacf37fcb892f.


Thx for updates. Looks good


s/existes/exists/


Roland, why are these packages implicitly imported?


s/the we/we/


Done


Sorry if my question is a bit dumb. Why does this work? the tests are not supposed to be in the import path correct?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.684.899956424b34854311d6a3d8912135d7c2181030](http://build.palaso.org/viewLog.html?buildId=69054&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:05:58


@rdoc101 feel free to add more specific classes when it simplifies css targeting


:+1: :)


Thanks for keeping an eye on it!


A couple of ideas that might help with this in the future:

1. Maybe the calculator/policy should "own" the questions and their possible responses, the flow should only be concerned with the order in which they are presented, and the templates should only determine how the question is presented. Also perhaps the actual response should be more closely associated with the question, rather than just on some generic state/calculator object.

2. I think it would be useful to constrain the value of a response to a multiple choice question, perhaps by using some kind of "enum" type e.g. [ruby-enum](https://github.com/dblock/ruby-enum).


Ah, you already fixed it in .fr.   I just did in .hu.


whoops... totally missed the fact that this PR is about bringing them into the import path. sorry for the noise!


@danielgindi what do you think of this?

Example use case can be found here: https://github.com/PhilJay/MPAndroidChart/blob/master/MPChartExample/src/com/xxmassdeveloper/mpchartexample/BarChartPositiveNegative.java


You forgot to close the container div in
app/views/users/edit.html.erb


<img width="815" alt="screen shot 2016-02-10 at 11 10 59" src="https://cloud.githubusercontent.com/assets/6759734/12944401/084c752e-cfe7-11e5-9f5b-87570a619331.png">


Yes, but first I prefer it to pass all the tests and commit it there after.


Great stuff….It worked…..Thanks a ton Amar for your prompt help.

Regards
Simran

From: Amar Zavery [mailto:notifications@github.com]
Sent: Wednesday, 10 February, 2016 01:48 AM
To: amarzavery/AutoRest <AutoRest@noreply.github.com>
Cc: Singh Simranjeet <Singh.Simranjeet@rsystems.com>
Subject: Re: [AutoRest] added retry policy and modifief redirect filter, optimized the service client constructor (876f74b)

@RSsimranjeetsingh<https://github.com/RSsimranjeetsingh>
Hi Simran,
We add an exponentialRetry filter to the client by default. You can disable the default retrypolicy by setting
options.noRetryPolicy = true in the client constructor. Check this link<https://github.com/Azure/autorest/blob/master/ClientRuntimes/NodeJS/ms-rest/lib/serviceClient.js#L29>. All the clients that are generated with "NodeJS" flavor are children of the ServiceClient class in the (ms-rest) clientruntime.

At the same time we offer an option to add your own filters. We will add those filters to the request pipeline.
Check this link<https://github.com/Azure/autorest/blob/master/ClientRuntimes/NodeJS/ms-rest/lib/serviceClient.js#L24>. So you can create your custom retry filter (may be reuse our retry filter with different default values) and add it as a filter in options.filters array to the client constructor.

Hope this helps.

Thanks,
Amar

—
Reply to this email directly or view it on GitHub<https://github.com/amarzavery/AutoRest/commit/876f74b86c9b9b3ffe876e9d6de64c50a0a2e499#commitcomment-15988250>.


Yes: http://docs.overviewer.org/en/latest/building/#osx

Note: I've tried pip install both PIL & Pillow to avoid this step. No dice /o\


$usergroupid==0 then empty($usergroupid)===true;

see http://php.net/manual/function.empty.php


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


Your idea does win if the code rejects based on the namespace - here we'd just make one simple query in the ``__contains__`` method. However, on a successful access we still construct the ``DBSeqRecord`` and must make the other select statements anyway.

The downside of my approach avoiding the pre-check ``"SELECT COUNT(name) FROM biodatabase WHERE name=%s;"`` in favour of a post-record creation check, is it would incur a *much* higher overhead when rejecting a record due to the namespace (multiple SQL queries, multiple wasted object instantiation).

In my own usage the successful access route has been typical, and the name space ``KeyError`` exceptional, so my bias is to focus on that. In practice I have not tested or bench marked the two approaches, and suspect the difference is marginal and usage dependant.


ok, thanks.


Nothing to comment on, looking good.


nie widzę nigdzie abyś zatrzymywał animacje ;) przy obrocie ekranu w trakcie animacji będzie wyciek pamięci. Aczkolwiek nie udało mi się doprowadzić do crasha :P


Bash script looks great, good work!

Recommend adding jvm_build, x (not sure what this is?) and instrumented_build to the .gitignore on your next commit to keep the repo clean.


I don't agree with these formal definitions. That should not be the number of 'xxx system'. Besides, by doing so, 'culm number', 'stem number' and 'leaf number' are equivalent (they are defined the same way) while they are not.


Done changed back to "/\"/"


What about `decode012` ?


Further remark for this line: MySQL did not accept '1970-01-01 00:00:01'. After some research, I realized that I did not take time zones into account. I think I have to set the time zone to UTC before creating the table (`SET time_zone = UTC;`) and then set it back afterwards,
Would this be acceptable (keeping the `TRADITIONAL` SQL mode), or would you prefer a default value of `NULL`?


Windows line endings?


@rdjedjig  Comme tu préféres mais faut fixer et écrire les choses sinon on va se poser les questions à chaque fois.

"ENTNG" n'existe pas c'est vrai.  Si on veut faire court on met "ONG"

fr.ong   ou fr.openent  ou fr.openentng

Ca se tient.


This logic is missing in the design. But for anti affinity topology key should not be there on the node as label key....
so in this case how to handle? just need to consider the node.Name??


add another comment with the option redirect


If you use the 2.1 branch of libarcus with master curaEngine it should work.


	@@-0,0 1@@
+ Khách hàng cho Asus Download Master - donloader cho các trung tâm mạng Asus như RT-N66U.


this.params.data.username = username || undefined


LGTM.


Yes, this is copying files other than .so. Sounds like you need to install the Qt QML packages via apt-get, but before you do, try running Musescore.  You'll see that it runs fine without QML but the plugins do not load. (They won't appear under the plugin menu even after enabling them from the plugin manager.)

To get the right QML packages look at the `yum install` commands in [my recipe](https://github.com/shoogle/MuseScore/blob/portable-linux-appimage/build/Linux%2BBSD/portable/Recipe), but be aware that the package names might be slightly different in Debian/Raspian.


Hi, please offer the same for jsr354-ri. We cannot have both RIs deviate by doing this only in one.
Thanks


Agree working on it


ack


@wei2912 any updates?


Errors:

1.In the "if(x[0]==X && y[0]==Y)" brackets, "x[2+index2]=x2;y[2+index2]=y2" should be replaced by  "x[3+index2]=x2;y[3+index2]=y2", otherwise you will see that when eating an apple, the program doesn't work as you imagine(there will be strange rectangles).

2.when judging lose, index i should be "for(int i=1;i<3+index2;i++)" to judge whether the snake touch itself instead of "for(int i=1;i<3;i++)" for the length of snake grows.

------------------------------------------------------------------------------------------------

Suggestions:
1.If you want to control time, "for(int i=0;i<100000000;i++)" can be replaced by "StdDraw.show(500)" which means stop 500 milliseconds

2."if(a=='1' || (a!='w' && a!='a' && a!='s' && a!='d') || index1==0)" can be replaced by "if(index1==0)"

3.x1,y1 is not necessary in the program, which makes it hard for other programmers to read the code, actually the code can be replaced by
 +      if(a=='w' && y[2]!=y[1]+1)
 +			{y[0]++;index1=1;}
 +
 +			if(a=='a' && x[2]!=x[1]-1)
 +			{x[0]--;index1=1;}
 +
 +			if(a=='s' && y[2]!=y[1]-1)
 +			{y[0]--;index1=1;}
 +
 +			if(a=='d' && x[2]!=x[1]+1)
 +			{x[0]++;index1=1;}
 +
 +			if(index1==0)
 +			{x[0]=x[1]+(x[1]-x[2]);y[0]=y[1]+(y[1]-y[2]);}"
another advantage of the last sentence is "x[0]=2\*x[1]-x[2];y[0]=2\*y[1]-y[2];" is that it can avoid overflow, just like what we have learned in BinarySearch "mid=lo+(hi-lo)/2" instead of "mid=(lo+hi)/2".


hey, this should work without it being indented as a code block, if it doesn't something's wrong


<!DOCTYPE html>


I'll change that tonight.


@sils1297 It doesnt
Even in rdt it works only if this is there


The anonymous method used as a  callback in all the load methods is the same, it can be refactored


Try to set tabs to 4 spaces


ERROR: Missing a Javadoc comment.


ERROR: Utility classes should not have a public or default constructor.


FATAL: [MissingTranslation] "`dialog_action_ok`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


FATAL: [MissingTranslation] "`dialog_error_title`" is not translated in "hi" (Hindi), "ro" (Romanian), "zh" (Chinese), "zh-CN" (Chinese: China)
 If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.

If the string should *not* be translated, you can add the attribute `translatable="false"` on the `<string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore="MissingTranslation"` attribute.

By default this detector allows regions of a language to just provide a subset of the strings and fall back to the standard language strings. You can require all regions to provide a full translation by setting the environment variable `ANDROID_LINT_COMPLETE_REGIONS`.

You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale="languageCode"` for the root `<resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)


WARNING: [Typos] "Ok" is usually capitalized as "OK"
 This check looks through the string definitions, and if it finds any words that look like likely misspellings, they are flagged.


WARNING: [ButtonCase] The standard Android way to capitalize Ok is "OK" (tip: use `@android:string/ok` instead)
 The standard capitalization for OK/Cancel dialogs is "OK" and "Cancel". To ensure that your dialogs use the standard strings, you can use the resource strings @android:string/ok and @android:string/cancel.


Good point. Fixed in 2cb0620


hi @Grz3s
Sorry for the late response. I've been quite busy lately and traveled a lot, so I still need some time to catch up with what is happening here.

On your question:
We can't have both SD2 and EAI scripts on the same creature (with a few particular exceptions). So I think it's better if you can send me your update and I will add it directly in the dummy effect spell in SD2.


Бежите впереди программы :) Это генератор списков. Я буду рассказывать о них завтра, если успею. Вы воспользовались им неправильно. Кроме того, незачем записывать результат в переменную, если вы дальше её же и возвращаете. Можно было сразу сделать
`return  [element, kwargs for element in elements if fn(element, **kwargs)]`

Правильно бы это выглядело следующим образом:
```
def my_filter(fn, iterable, **kwargs)
     return [element for element in elements if fn(element, **kwargs)]
```


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 285](http://ci.infinispan.org/viewLog.html?buildId=35263&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for c6125793f862e6297a7282bea36fbdd187c55a20
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 1


Hi @sgeulette

this broke my usecase because I call a view on an object I can see but I am not able to access the entire path to the object (some sublevels like folder 'Members' are private) and restrictedTraverse breaks.

What is the usecase still of using restrictedTraverse instead of getMultiAdapter?

I would suggest to use unrestrictedTraverse if do not use getMultiAdapter, or come back to getMultiAdapter, what do you think?

Gauthier


@mbatchelor, Sonar is complaining about DimensionLookupDialog and DeleteDialog. There's a checkstyle violation. Refer to the Sonar report for details.


I moved the matching options from `ExhaustiveMatching` to `MatchingBase` because they should be the same for all matchers.


forgot to change the name here


ack


add new functionality for update unsubscribe user status in SIB.

Please check and merge the code.


I did git pull beforehand, that is strange. Well. This is now a larger screwup, I don't have the time to check into that. Can somebody revert my commit somehow?

We need addlang.txt to contain linebreaks after the define, I didn't see this wasn't the case.


That's a huge change. Kudos


ended


Check Style violation, please fix @akhayrutdinov


I just renamed `ROR_VERSION_STRING_WIN` to `ROR_VERSION_STRING_RESOURCE`. I think the only place this is used is inside `source/main/main_sim/icon.rc`. I also assume that this version information is shown in the properties panel of RoR.exe. I haven't tested it though.


Check style violation, please fix


would be good to change the single quotes to doubles for consistency


+1 to what Roland said


I use linux. I will remove the line endings


the name is too generic


yup thanks


Ты теперь поняла, почему это неправильно?


Naming should be consistent, but it doesn't matter if the recipe doesn't run on x86 Jessie as long as the x86 AppImages built on CentOS work on x86 Jessie.

I'd grep for "Raspbian GNU/Linux 8 (jessie)".


ustawiasz to na LinearLayoucie...a gdzie orientacja? :P


to samo co wyżej


I think using `%lu` would be better here.
`[NSString stringWithFormat:@"%lu", intNumber];`


Ты можешь назвать мне ошибки в этой строке?


@flavour Here I am trying to show only the active alerts (ie those in the datalist) by adding this filter but this does not seems to be working, I see all the alerts but if I try the url `(/cap/alert.geojson?~.id=...)` then I get geojson of only the specified alert id?
What am I missing here?


rather name it `force_linebreaks` or so then


@flavour Please review...seems to work fine for me :)


still the installer status here? why not `Crowbar::Backup::Restore.status`?


Зачем эти две строки?


Зачем это?


There is no version locking here, so executing `docker build` will assume you actually want `java:latest` I would put version lock just to make docker build process consistent.


It makes sense to describe that this is your solutions to the exercises in Think Python book with the link to the book


Other than version locking, LGTM :+1:


better to call it `item`, not `i`


I'd use `%li` and `%lu`instead of creating NSNumber for building NSString format


can you do a fixup in the last commit (Move node upgrade status to last finishing step) please?


There's an unused import in this file, StepMetaInjectionInterface on line 52.

Even though it was not introduced in this commit, you should've fixed it because you'd have caught the issue when running check style before issuing the pull request.

Please review, @AlexanderBuloichik


Can you figure out non-recursive solution?


Странно, у меня работает :)
Мы же ведь сначала генерируем новый элемент, а потом предыдущий отображаем


Interesting - I will test it.

If this works well, I think it should be included/enabled by default and not reside in the Customizations.js file

I will report back after my tests. Please be a bit patient.


Nice!  If there are any event that has a weird name in the FTrace object please report it in trappy.  It's confusing and it doesn't buy as anything, so we should stop doing it.


Java/Ch 08. Recursion and Dynamic Programming/Q8_02_Robot_in_a_Grid/Point.java

Consider returning the expression in the if statement, for simplicity:
```Java
return ((o instanceof Point) && (((Point) o).row == this.row) && (((Point) o).column == this.column))
```

(would fix the formatting also)


Right I see


In general hardcoding IDs here is a last resort (usually only used to show a single record):
* Not dynamic (Layer refreshes on a timer - this won't show new Alerts, or hide newly-expired alerts)
* Not scalable (GET URLs are limited to a certain number of chars, so a long list of ids can blow that limit)


I would prefer to have it not at a fixed url, instead set it at the discovery
(like OpenID Connect: http://openid.net/specs/openid-connect-discovery-1_0.html#ProviderConfig )


Il faut utiliser un nom domaine sous le contrôle de l'association Open ENT.   cf http://maven.apache.org/guides/mini/guide-naming-conventions.html.

donc fr.openent me parait être bien adapté.


`~max_parallel_atoms:1`? This looks like you're limiting the actions to be serial. If this isn't the case, this parameter needs a bit of renaming.


Check style violation. Please fix, @matthewtckr


Since all this is commented, this won't work in S3LocationSelectorWidget
Not a massive issue, but surely better to just comment the part which creates the CIRCULAR STRING?


Java/Ch 08. Recursion and Dynamic Programming/Q8_02_Robot_in_a_Grid/QuestionB.java

Please consider using the parent types where possible, so that they become interchangeable in the future (i.e. `List<Point> path` and `Set<Point> failedPoints`, etc.


Alright, I will do some testing with the new maps first and merge it in.


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


This style is a bit off... I realise you have to cram it in like this because you're inside the list construction but you can pull some stuff out above where the `let vbds = ...` is.


It is as good as any rule. None will be fully satisfactory. What happens if you find none though?

```
class Tuple<V1, V2> {
    // no V2 field
    public V2 notAGetterV2() { ... }
}

@Valid
Tuple<String, @Min(1) Integer> nameAndAge = ...;
```

Assuming we ignore method validation, we have a few options:

* ignore that case
* leave it explicitly undefined in the spec
* make the spec throw an exception


Hi @funkyproject , I see you have fixed bugs and cleaned up things here. can you please submit a pull request to ng-joyride ?


@flavour Can you suggest a work around for this? this varies according to datatable id so I think datatable id is dynamic so this should be, isn't it?
anyway can you point to more generic one?


Видимо тогда надо все же заменить arr[len - 1] на arr[i - 1] в строке 11?
А так решение хорошее


Collaborators


changed processUpdates to return a response, and updated caller to merge. But we're more "ask" than "tell" now.


I think better to name it as gxbb. Everyvhere in kernel source code it is referred as gxbb.


`unit1604.exe` fails to link for me on MSVC:
```
unit1604.obj : error LNK2019: unresolved external symbol _sanitize_file_name referenced in function _test
```

AFAICS, this program depends on `$(UNITFILES)` which do *not* include `./src/tool_doswin.c`. Or how is this test supposed to pull in the function `sanitize_file_name()`? If I add `tool_doswin.c` to the list, I get more unresolved symbols.


even in the anti-affinity case, you still need to take topology key into consideration. Making topology key only to node is a runtime requirement, otherwise there would be to much hard coding.


No, it's all good, my bad. The version committed in c3db29a036d2b15b9d941f69d8e4e98bf0600040 is great, I was confused because the discussion was “attached to” *this* commit, but that is not *your* commit. Sorry.


Wrong repo, you want this one: https://github.com/CC-Hive/Turtle


Do we ever have a case with a radius & no WKT?
I think this is unnecessary


errorneous -> erroneous

Also this is related to a bug, right? Can you add the `Fixes <full_bug_url>` note to the body?


See this @tawashley?


The Typesafe Open Source Position is now out of date:

>  Slick Extensions is a perfect example: open source users may freely use Slick with open source databases (like MySQL or Postgres). To use Slick with commercial databases (like Oracle or SQLServer) requires Slick Extensions, which is a commercial module.


Would you like this to be merged with my version?


Maybe is better to add a option/variable something like AMLOGIC_SOC or AMLOGIC_FAMILY or the like to minimize case statement in the future, as new projects may be added.


@sils1297 Yes!


The spec was originally expressed as `ImportedHitsFile.should have(1).file`, transpec turned that into `expect(ImportedHitsFile.size).to eq(1)` and this made it break because AR::Base objects don't have a `size` method.


Done, reverted to previous pattern with delimiter escaped


@srisankethu cool :)


What happens here if we have a point?
I suspect the JS might crash?


@pamval, sure, I was not expecting this PoC tol be merged as-is :)
I'll also replace my todos with proper comments


I read this as: Integrator are what other projects might call committers. If that's right, we could clarify that here.  My first reading of "Integrators" was like ["System Integrators"](https://en.wikipedia.org/wiki/Systems_integrator).


Are we waiting here for 20 minutes per parallel operation to finish or fail? What if they all hang? It looks like the timeouts wills stack? 255x20m = 85hours? Won't this get caught by the `queue_operation_and_wait`?


Note that the notion of linking an extractor (or extracted value in the alternative extractor proposal) to a specific type parameter is still required. Otherwise we don't know what constraint applies to the extractor/extracted value.
So in essence, this problem is no longer a problem for either proposals.

In this proposal though, the fact that an explicit extractor is present on a given type parameter, should it inhibit the idea of looking for gettters and fields?
For example, if I have a tuple V2 extractor, should I also apply the constraints on the `getV2()`?

My initial reaction is not to consider getters/field when explicit extractors are present.


На случай, если появятся промежуточные статусы. Чтобы нумерация была по порядку.


@lauraluiz Removing `bottom: 0` does the trick ;)


No need to restructure this now, but ideally this change, and any others that are just whitespace should go in a separate commit  so that reviewers can more easily see the meaningful changes.


> Or should I open an issue?

yes please - thx


Also, what happens when one of the sub-operations failed. Will the others be un-done or are we relying on Xapi to do that?


Yep that's very clear, thanks


thanks for doing the site, I really suck with design/css


TODO: reference the extractor proposals from Proposal 1 instead of this outdated one. I guess they can be shared.


s/implemention/implementation/


Ah I see now that you have a 10 here. This patch needs to be squashed into the previous otherwise the commit message of the parent patch is wrong.


commit message: `Changes` -> `Correct`? We use imperative tense as a convention. (I can see how this sometimes may be hard especially for non native english speakers - which all of us are :))


TeamCity Selenium POM :: CI [Build 55](https://teamcity.sponte.uk/viewLog.html?buildId=1060&buildTypeId=SeleniumPom_Ci) is now running


Check your editor isn't changing whitespace automatically (unless you think this is an important change for some reason).


I'm glad that on Python3 you don't loose data in responses anymore.


@lexaux Shall we add some event (button,restart) manage simulation from gui?


TeamCity Selenium POM :: CI [Build 55](https://teamcity.sponte.uk/viewLog.html?buildId=1060&buildTypeId=SeleniumPom_Ci) outcome was **FAILURE**
Summary: Tests passed: 39; exit code 1 (new) Build time: 00:00:42


With this change, if I screw up the dagr options, but don't provide a pipeline name, the output changes from telling me what I screwed up, to complaining about not providing a pipeline name.  I'm not sure that's more helpful.  E.g. try running `java -jar target/scala-2.11/dagr-0.1.0-SNAPSHOT.jar --confit=my.conf`


I'm actually wondering if hiding one level is a good idea. We can refine that later, though but I am having second thoughts.


'dynamic' only on server-side refresh currently...should be on client-side refresh


TeamCity Civic Assembly :: Build backend + frontend [Build 90](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=926&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Now I'm curious - is there some reason a long pipeline name is different than a short pipeline name?


commit message shortlog (i.e. first line) `LICENSE : chnage file permissions ` - I'd rather change that to something like `LICENSE: Remove executable permission`, as it's more precise. (Also note: no space before the colon, first char after the colon upper case)


Хватит писать по-русски. Пиши имена коммитов и все комментарии везде по-английски.


Ja dit is duidelijk.


sad


yes!


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


Geneace dump files possibly effected

dump_2016-02-09_A.10.ace
dump_2016-02-09_A.11.ace
dump_2016-02-09_A.12.ace
dump_2016-02-09_A.13.ace
dump_2016-02-09_A.14.ace
dump_2016-02-09_A.15.ace
dump_2016-02-09_A.16.ace
dump_2016-02-09_A.17.ace
dump_2016-02-09_A.18.ace
dump_2016-02-09_A.208.ace
dump_2016-02-09_A.20.ace
dump_2016-02-09_A.211.ace
dump_2016-02-09_A.212.ace
dump_2016-02-09_A.213.ace
dump_2016-02-09_A.214.ace
dump_2016-02-09_A.215.ace
dump_2016-02-09_A.216.ace
dump_2016-02-09_A.217.ace
dump_2016-02-09_A.218.ace
dump_2016-02-09_A.219.ace
dump_2016-02-09_A.21.ace
dump_2016-02-09_A.220.ace
dump_2016-02-09_A.221.ace
dump_2016-02-09_A.222.ace
dump_2016-02-09_A.223.ace
dump_2016-02-09_A.224.ace
dump_2016-02-09_A.225.ace
dump_2016-02-09_A.226.ace
dump_2016-02-09_A.227.ace
dump_2016-02-09_A.228.ace
dump_2016-02-09_A.22.ace
dump_2016-02-09_A.23.ace
dump_2016-02-09_A.262.ace


Fixed in https://github.com/rib/gputop/commit/d5657d9e54e14746606733c4e504aef669cced7d


This is a good improvement Alberto! A slightly more informative commit message would have helped me understand and review the code quicker but the actual changes look good.


У тебя уже есть функция `calculate`, которая осуществляет ряд операций над рядом чисел. Не надо дублировать её здесь. Эта функция должна заготовить аргументы для `calculate` и вызвать её.


Commite -a


Embed the `.kd-middleellipsis-container` container here. Dont make the clients use this class. When you embed the container here you would not have to use .closest method. The only requirement for clients would be to be expandable (most likely `display: block`).


I still don't understand the issue. How does shared lock helps? Is rename on Windows not atomic?


Re `clostest`: See my inline comment.


the commit message shortlog needs some love though it already is imperative tense, which is good :) try to be more specific while keeping it short - update is pretty generic and doesn't give me much information. Also the ClangCloneDetectionBear is more of a context which usally goes into a so called tag into the message. One could refactor this into something like

ClangCloneDetectionBear: Add % to clarify message` or so?


reack


@radeksimko I honestly cannot remember why I chose 100 - I think it was a bit of mentalness at the time and should more than likely be changed :)


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


This looks fine to me, but you could also do this more compactly as:
```scala
val extraMsg = msgOption map { text => s"\nmessage: $text" } getOrElse ""
```


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 287](http://ci.infinispan.org/viewLog.html?buildId=35272&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for c6125793f862e6297a7282bea36fbdd187c55a20
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:03

Process exited with code 1


TeamCity Civic Assembly :: Build backend + frontend [Build 90](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=926&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 148 (1 new), passed: 24, ignored: 4 Build time: 00:07:10
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455101842036: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

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


And in its irony, this commit message does not live up to the commit message guidelines it corrects. @digitalcraftsman -- are you merging these?


This should be a typo.


Does a thumbsup mean ok to merge?
In that case how do you deploy? With `git push heroku master` ?


Nice Job Everyone!


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


Add new Keywords for Arduino IDE


I just noticed that `list($toWrite[$file], $toWrite["$file.meta"]) = $this->generate(...)` will be broken on PHP 7 (regardless of platform) because the evaluation order [has changed](https://3v4l.org/vDKN9).


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 288](http://ci.infinispan.org/viewLog.html?buildId=35282&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for c6125793f862e6297a7282bea36fbdd187c55a20
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 1


See the description of the commit. Well-formed tests are all passed.

Another note: The `useBinnedData` seems to slow down the  dense case with cross-sectional mean quite significantly.


Is there a reason why this isn't in the PluginStore?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 289](http://ci.infinispan.org/viewLog.html?buildId=35287&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for c6125793f862e6297a7282bea36fbdd187c55a20
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:01

Process exited with code 1


I think this could do with a better name in general since `option` tends to sound like `arg` or `parameter`.  How about `choices` or `possibleValues`?


Otherwise :+1:


Was definitely on autopilot when reading the eslint errors and fixing this all up, I probably just thought thats a daft thing to do and didn't grasp the bigger context


I've only a rough idea of it. I do believe Forthers need a IDE to communicate wit all the Forths out there. Something like DrRacket with an editor and a REPL area. Maybe also an area to display parameter stack.

I need some Forth words to specifiy the data to be transfered from the Forth side. And at least a word to tell Forth to transfer data.

I'll start with a data format taken from  MTConnect:

http://static1.squarespace.com/static/54011775e4b0bc1fe0fb8494/t/55a95b3ee4b07afa0a97a170/1437162302749/MTConnectWorkshop_2015_Wickelhaus_BuildingAnAdapter.pdf


Avoid unused local variables such as '$response'.


PHPCS: Expected 1 newline at end of file; 0 found


Il faudrait ne pas commit ce fichier.


@muzaffaryousaf fixed


Umm, what now?


Un test unitaire pour illustrer le problème et faire une PR à edx ?


This issue is treated in the main CMakeLists
https://github.com/poparteu/openMVG/blob/a3fe68214b69d8bb0f74d844193300015dfca772/src/CMakeLists.txt#L399

You can compile openMVG with `-DBOOST_NO_CXX11=ON` and `BOOST_NO_CXX11_SCOPED_ENUMS` will be added to the definition


Now that this is a case, we could probably add a more helpful error for `case Seq()`


We need to replicate this:
FS("info.expires") >= request.utcnow
As a URL filter, right?
Docs: http://eden.sahanafoundation.org/wiki/S3/S3REST/URLFormat#URLExamples
So:
info.expires__gt=YYYYMMDDTHH:mm:ss
I'll leave it to you to work out how to get the appropriate time format from JS & see if this updates dynamically.
Even if it doesn't update dynamically upon client-side refresh as-is, it's better than a hardcoded list of IDs...but there must be a way to update the timestring upon client-side refresh...maybe need to add a hook to this function:
https://github.com/flavour/eden/blob/master/static/scripts/S3/s3.gis.js#L2068


ce n'est pas justement le correctif qu'a ajouté loic dans le commit:
https://github.com/jeedom/plugin-openzwave/commit/cbc4a3c1e1684a04049f90a6e9f3b884e9eac7ea

pour corriger ce bug:
https://www.jeedom.com/forum/viewtopic.php?f=34&t=15391&p=279303#p278472

?


Great =D


oui, j'attends la reponse sur le ticket pour aviser


Having to put `case None => Unit` bothers me.  I think this could be done more simply as:
```scala
byName.get(name) foreach { n =>
  throw new CommandLineParserInternalException(s"$name has already been used.  Conflicting arguments are: '${arg.name}' and '$n'"
}
```


Thanks. It misses the locale in the defaultResources.nopres.xml file. And it breaks "Configuration" menu when you hover it.

We're still thinking whether you implement it right now because we're going to re-design our admin area very soon (as described at #34). So let's move to the new UI first (most probably https://almsaeedstudio.com/preview)


Perhaps you can say like "An iterator over a collection whose elements need to be fetched remotely" instead of the "Iterator that involves server communication, and as such can throw".

Also, please add method level java docs as this is an exposed interface.


FUCK BROWSING!
Who even browse?

![](http://cdn.meme.am/instances/500x/66211586.jpg)