Yup, looks good to me.

On 03/01/2016 12:12 PM, David Cavins wrote:
> Thanks, Boone. Fix is here:
> https://github.com/dcavins/buddypress-docs/commits/folder-updates-feb29
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boonebgorges/buddypress-docs/commit/fc47ffd7fab2b6a957e0caf3f992dc82c3accd33#commitcomment-16425292>.
>


Otimização de queries


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


Note that I've made the analyzer the default with a recent change because I want to get some testing going for the upcoming release candidate. There's now `-D no-analyzer` to disable it.


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


Should I start merging some of these changes into 1.9.x? (From the branch: https://github.com/dcavins/buddypress-docs/commits/folder-updates-feb29)


I'm not sure about the `- 1` here, i think `currMax` is a value at which it is known to work so `avg`


Could you be more exact?


This conditional isn't necessary unless functionality from ADM-79 is implemented.


Beautifull :+1:


ok


You are right about peewee, i decided to use SQLAlchemy. Based on your advice, i did a little research and found more tutorials that deal with SQLAlchemy relationships.
Thanks for the help


Good catch! Looks good to me now.


The following doc(s) have been deployed to the [1400roles](http://1400roles-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/organizations/change-management](http://1400roles-static-docs.pantheon.io/docs/articles/organizations/change-management)


It would be good to have a permanent link that is independent of the version. Does anyone know how to do this?


I thought about asking the `[LPTouchUtils applicationWindows][0]` window if it responded to that selector, but decided this was a reasonable way to test for availability.  I am, on the other hand, suffering from vertigo and feeling super muddled.  @sapieneptus What say you?


Agreed that the test should be simplified; I've reduced it to the minimum required to verify the logic.


Please, fix indentation here.


hmm, that didnt go well :) kernel crash, fixing...


WTF???


I hope it will be the last minor fix before merging :)


Closes #18 by removing this functionality.


Hmm. I catch it, good trick :+1:


Boolean(r)?


Hey love this but it's creating some problems. Can you clean it up?


Blank line. Great thinking.


I'll put id and label in then.


This just needs updated to use single-quotes instead of double-quotes because all strings are non-interpreted strings and should maintain code consistency.


@mibby I'll update to support 1.9 as soon as possible. Thanks for letting me know.


I don't know - it could also be a problem to continue benchmarking against old upstream because we might be optimizing around a certain function being slow, when it isn't in the latest upstream.

Its always possible to re-run benchmarks of an old commit, so I think using the latest upstream and just re-running the history when it is updated would be most useful for development. If we are planning to publish these results as a reference though it would make sense to pin a specific version (but then we also need to decide on a reference machine, etc).


Mongo stopped working due to permissions errors when we added in accounts


@msigley As far as I can tell it is all legit.


I'd prefer the original syntax `self.log.info("%s: %s" % (key.upper(), stats[key]))` so we can make sure the number of arguments matches the number of strings to be converted.


I'm not seeing mention of this vulnerability at that info link, except for 1.3.2. Am I missing something?


great.


Which version is this coming out for?


awd


selurvedu/robot_tests@5e4d152
Блін. :smile:


ABASKINAN TIIN


Nice Work! Keep up the contributions!


I think it might be better to raise some kind of specific exception here.


Ettim sanırım


Here you're keeping the syntax. Good.


### Instructions for project

The purpose of this project is to demonstrate your ability to collect, work with, and clean a data set. The goal is to prepare tidy data that can be used for later analysis. You will be graded by your peers on a series of yes/no questions related to the project. You will be required to submit: 1) a tidy data set as described below, 2) a link to a Github repository with your script for performing the analysis, and 3) a code book that describes the variables, the data, and any transformations or work that you performed to clean up the data called CodeBook.md. You should also include a README.md in the repo with your scripts.
This repo explains how all of the scripts work and how they are connected.
One of the most exciting areas in all of data science right now is wearable computing - see for example this article . Companies like Fitbit, Nike, and Jawbone Up are racing to develop the most advanced algorithms to attract new users. The data linked to from the course website represent data collected from the accelerometers from the Samsung Galaxy S smartphone.
A full description is available at the site where the data was obtained:
http://archive.ics.uci.edu/ml/datasets/Human+Activity+Recognition+Using+Smartphones

Here are the data for the project:
https://d396qusza40orc.cloudfront.net/getdata%2Fprojectfiles%2FUCI%20HAR%20Dataset.zip

You should create one R script called run_analysis.R that does the following.

1. Merges the training and the test sets to create one data set.
2. Extracts only the measurements on the mean and standard deviation for each measurement.
3. Uses descriptive activity names to name the activities in the data set
4. Appropriately labels the data set with descriptive variable names.
5. From the data set in step 4, creates a second, independent tidy data set with the average of each variable for each activity and each subject.


... and here, not :/


Adding_Baseline_Pdrop_Adjustment_to_AirLoopHVAC (jmarrec) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1349.079336ccf11ad8d1f9fc0840e2cae5ecd4571eb3](http://build.palaso.org/viewLog.html?buildId=69788&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1022, ignored: 12 Build time: 00:06:41


Большое спасибо!))))))


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 126](https://teamcity.labkey.org/viewLog.html?buildId=306648&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 22 (3 new), passed: 274, muted: 1 Build time: 01:39:05
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

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

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

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

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

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

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

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

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


Don't you need vendor/pdalboost as an include for thest test subdirs?


Ok, great job here. Looks awesome. Some minor adjustments, but overall seems quite ready to go.


Yes, but not before plugins. I'm about to test another that only enables testing prior to adding plugins.


@jdauphant no response from you in the PR


can confirm this is a bug and breaks the plugin


Re: `is_trait`, it was either (a) rename `has_type` to `is_trait`, or (b) rename the concept `Trait` to `HasType`.  (b) makes me want to vomit, so I went with (a). :-)

Re: `strict_and`, `fast_and` doesn't describe its semantics, and begs the question why one would ever want to call something that wasn't "fast". `strict_and` carries more semantic weight.


Shoot.  I copy/paste and replaced the lines of code from another file I was working on as a temp file.  That messes up the tracking on GH.  Won't do that again.


Descriptions should be self-contained. What exactly is the issue and what is the plan to fix it?

You can also provide an ignore reason as parameter to `@Ignore`:

    @Ignore("reason")


Aii yes - indeed this is a good question.
I will look into this.


Adding_Baseline_Pdrop_Adjustment_to_AirLoopHVAC (jmarrec) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


I don't see how this diff matches the commit message.


I need to be able to use a cross-id so that I can reference both f# and c# in a `insertbefore` declaration. Thoughts?


Yes.  Remember that stuff being fiddly.  I remember reading the docs on enable_testing, and it wasn't too helpful, so I guessed.  Apparently incorrectly.


@ncborges Could you be more exact?


How important is it to you that the language be changed from machine to server? I sincerely inquire on this because in reading your update proposition here I feel that there could be some slight sentiment towards ethical treatment and tactful communication regarding machines. Overall, for what it is worth I do not find it helpful to the documentation to remove the abbreviation VM so long as the first instance of the abbreviation occurs as Virtual Machine (VM). Then every other call to VM is implied. The **machine** to **server** changes though, they truly do interest and inspire me. Wishing you the best.


You should use `BibtexEntryAssert.assertEquals(bibis, entry)` as this is a) more readable and b) also falls back to the CanonicalBibtexEntry.

Side comment: You should not have mixed `toString` and `CanonicalBibtexEntry.getCanonicalRepresentation`. Both have the same effects, but mixing them makes the code hard to read.


He-he! That was my first variant.
Exactly without that `powerful` :smile:


@neilb I need to fix something in PathTools. Was going to base it off your work, but the url in this commit is 404. What's the latest HEAD these days and where do I get it from?

Cheers


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


Commit is reporting a large number of deletions because I rearranged the order my functions/variables appeared in default.js. I will keep this format moving forward and will refrain from doing so again.


s/single nodes/single node/g


This is indeed a functional update.


Looks like Chrome just fixed this bug :-)


No test file contains a DOI. Please add DOI support.

Please also remove this comment line. It is an implementation comment, which should not be visible to users.


What version of GCC are you using?

On Tue, Mar 1, 2016 at 12:35 PM, Víctor Fernández <notifications@github.com>
wrote:

> GCC throws an error while compilation because:
>
> M68k/devices/Timer.cpp:91:14: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
>    if (mask > TIMER_IRQ)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BSVC/bsvc/commit/456926e72922535d9b347716496f2237efeeb674#commitcomment-16424492>
> .
>


Is this `[@CompileStatic]` rendered properly ?

I can check locally, of course.


Please remove this line at the export - or replace jabref.sf.net by www.jabref.org.


nice


It seems that SMod does not send smod_ver until cgame has been loaded, so it will look like they don't have SMod on the very first connection to a server. Since we don't want every single player to use /setu autodl 1 in their autoexec, this patch cannot go live on PUG servers for now since the majority uses SMod.

In short, this commit will be pulled upstream when:
- SMod is updated OR is considered legacy after an up-to-date PUG mode is released
- Known clients that support HTTP autodl (jkclient, jaMME) either send autodl 1 or make their autodl happen before UDP autodl
- Map makers who were using bad practises with different BSP versions (thinking about siege using dark/bright maps or the patched CTF manaan) use .patch files instead

Either way, basejka users will always be fine. It's only client mods that need to update their design.


Another option is to instead assert (false); before returning m_unknown.


New repo url


sorry but i was focused on iconcarousel and gamecarousel on my tests and i did not noticed any problem when switching to icongrid, maybe because of having only one page to select in icongrid :)


The closing parenthesis is missing here. Causes webpack to fail parsing.


Revision 1 of this package produced a hang: https://github.com/manjaro/pamac/issues/104


Go for it

On 03/01/16 12:27, David Cavins wrote:
> Should I start merging some of these changes into 1.9.x? (From the
> branch:
> https://github.com/dcavins/buddypress-docs/commits/folder-updates-feb29)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/boonebgorges/buddypress-docs/commit/fc47ffd7fab2b6a957e0caf3f992dc82c3accd33#commitcomment-16425638>.
>


That's a good point here. We shouldn't lose support for old versions.


ahh, so this is related to some other '250 looptime' changes that aren't in this commit. ok.


Yes! This is what I actually have to do in nengo_ocl, and I was wondering if it would make sense to have it in nengo as well. It would certainly clean things up on the nengo_ocl side, and probably a bit on this side too. The only caveat is in nengo_ocl I have a dedicated TimeUpdate operator, are people fine with that? I think I had a reason for that instead of doing it in terms of base operators, but I can't remember exactly what it was.


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


There is a double quote bug in this line, it was causing me trouble trying to edit quotes subpanel in contacts module through the studio.


nngraph no longer allows nodes which do not connect to the output of the gModule. This PR fixes this in the relevant unit test and updates the corresponding example in README.md.


Yuck


Hi @mgruner, these use statements was already there. Now there are twice in all affected files.


Definitely makes sense for Nengo as well! I think it was at one point like this; I forget why we changed it... perhaps the same reason why a dedicated op is needed? But yeah, if it can be done without additional ops that'd be better, otherwise a new one is fine.


What is different from the original file?
Did you erased the file, committed, replaced with original contents and then committed again?


That's not fully correct phrase: the Groovy code 3is compiled to the bytecode anyway. And that is fully Java code.
The only difference with `@CompileStatic` that we have a **direct method invocation** instead of dynamic resolution and conversion, like we bypass SpEL in case of direct `MH`.

Maybe `plain Java code` would be better ?


@marianogonzalez my bad. Here is just a commit to put the message in the right place. It was wrong in previous PR.


Ok I agree, let us give it a shot.

But please note: The use case I talk about is this, and there all
destinations really "disappear":
- Tap anywhere on the map, then on the context menu hit the blue
"Directions to" disk
- While or after the route is being calculated, tap another pint on the
map, hit the blue (now intermediate dest.) disk again, and select "Set as
first intermediate destination"
- Repeat this again to assemble any number of intermediate destinations.
- Finally: Hit the blue route arrow button (to the right of the menu
button) and on the Configure route dialog select (X) to cancel the route.
Destination including all intermediate destinations will be GONE ( with no
marker left behind, not even where the final destination was...).

On Tue, Mar 1, 2016 at 3:48 PM, vshcherb <notifications@github.com> wrote:

> They don't completely disappear there will be a marker at final
> destination. In case of intermediate destinations it is advisable to have
> map markers anyway. That's why we don't have add waypoint from map or
> whatever in waypoints. I don't see a big issue here cause it might take a
> while to realize that map markers is a proper way but then it is smooth
> way. We will work to remember previously selected map markers for
> navigation and next time it will be using them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/Osmand/commit/fdedcfb1d33003b66d699b3fa09b3125f4990f83#commitcomment-16419386>
> .
>


containerChainName and containersChainName feels like a bug waiting to happen; can you make these two SLIGHTLY more different in wording?


You might want to extend the tile.style object here, using `Object.assign` or something like that, because I'm getting a `ChangeDetector` error, using this in an async router loaded component.


Capybara is supposed to handle that automatically in most cases, depending on the methods you use to poke at the page contents. In general, things like `find` and `expect(page).to have_content` will wait up to several seconds for the expected values to be there. Other methods, like `first`, do not have that waiting behavior, though.


@ribasushi I forked it off rjbs's repo, but if I remember right, after I did a PR Rik deleted his repo, as it's upstream blead, so you'll need to do a blead patch.

/cc @rjbs


I have added a check for `originalEntityData`, but not for `entityIdentifiers`: `entityIdentifiers` is supposed to be empty anyway because the `CmsUser` has no identifier. Iterating over entityIdentifiers (making no assumptions over the size of it) is not possible, because the iterator variable would reuse the hash instead of `$user`, so the `persist` would theoretically not fail anyway anymore.


http://www.snagajob.com/ is the webpage we will similarly build


procura guardar tu API_KEY en un lugar seguro y no publicarla en github, hay personas que se aprovechan de esto, puedes usar variables de sistema por ejemplo:

http://stackoverflow.com/questions/21173826/environment-variable-in-settings-gradle-not-working-with-android-studio

Otra forma que puedes implementar es guardar el api_key en el archivo strings.xml al igual que el baseURL para que en un futuro no tengas que buscar la url en cada archivo si es que el API cambia


prueba finalizada


Sure, completely right. I added a comment with a rough explanation in #142.


`handler` is doomed to crash (referenced before assignment) on invalid log level in `--store-logging-stream`.


Generally looks good. Can you open a pull request on the DCIB to add the default CONTAINERS rule, along with the forwarding to it and the conntrack entries?


ngModule was modeled after ngApp. ngApp does support all those different variations.

Except for the plurar variation, I did that because you can use multiple modules and I wanted it to be clear when it was being read. I have reservations about that decision, might add more confusion than clarity.


@cfelton A unit test should first fail ;-) (and travis cannot be tested locally). I only started the exercise when I judged it should be possible to get it to work. Not straightforward, as it requires custom install. But in the mean time it works.

The purpose is to see if improved VHDL 2008 support in ghdl may be useful to ongoing efforts.


Whoops. Sorry.


Why the rename?


You should add StamBoom as well at the Spigot Resource page in the credits ;)


This is tricky. I could not make it work:
~~~
[apahim@hurricane avocado]$ avocado run passtest --store-logging-stream all
JOB ID     : ebf7f6951ba69d9f912602bc73f3140c3ccae233
JOB LOG    : /home/apahim/avocado/job-results/job-2016-03-01T15.59-ebf7f69/job.log
TESTS      : 1
 (1/1) passtest.py:PassTest.test: PASS
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0
JOB HTML   : /home/apahim/avocado/job-results/job-2016-03-01T15.59-ebf7f69/html/results.html
TIME       : 0.00 s

[apahim@hurricane avocado]$ cat /home/apahim/avocado/job-results/job-2016-03-01T15.59-ebf7f69/all.INFO
[apahim@hurricane avocado]$
~~~

Please add some tests for this feature - if it's not ahead in next commits :)


Yes...


To be honest, I didn't actually test this without, but I knew it needed to run first to re-compile with the bootstrap popover.


This also fixes #265 btw


> $user = "users".$i;;

Pensez à virer les deux points virgules :+1:


https://github.com/tailhook/vagga/blob/44790c3194df0473f50383ba3589d606094f84d1/examples/django_alpine/vagga.yaml#L77

Вот тут сделано на `.lock`-файлах. Чуть изящнее.


Curious — why was demo changed from ES6 to CommonJS?


This looks like it might make sense to backport to branch-1-8, any objections?


So silent will suppress stderr? I cannot say I like it, but seems like it was discussed before.


I've filed https://github.com/dotnet/roslyn/issues/9344 to track this.


No problem. It was easy to fix keeping your feature. ;)
And yes, bugs only happened if more than one page and last selected game not on first page.


Ok, this is great. lgtm.


Testing comments **-skanaan**


The same is here.
Even if it is scrip, it is compiled to Java class(es) on the first invocation. And it is re-compiled on the each refresh.

The `static` word there means exactly that we won't have **dynamic** behavior, but normal Java style, when we are forced to do standard Java steps, like `cast` to `String` in case of the Splitter test:
```
<groovy:script compile-static="true"><![CDATA[
	((String) binding.variables.payload).split('\\s+')
]]></groovy:script>
```
With regular dynamic groovy we are in the "hell" of invokers and coercions even around so simple script code.


@brianary It's mentioned in the linked page:

> ngSanitize: blacklist the attribute usemap as it can be used as a security exploit (234053fc)


The following doc(s) have been deployed to the [1172tempf](http://1172tempf-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/files/temp-files](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/files/temp-files)
- [docs/articles/sites/code/unsupported-modules-plugins](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/code/unsupported-modules-plugins)


If you test equals/hashcode against generated code then your are mostly testing the algorithm of the code generator and not your code. Is it worth it? similarly I typically don't unit test setter and getter of code generated by an idl, nor do I unit jdk apis...


Better later than never :)
Looks good to me.


Yes, true: Here is where the dualism comes in:
- We have 2 separate (but partially overlapping) histories now:
Destinations go to the Search history. map markers go to the map marker
history. (and after deletion also to the Search history). Destinations, on
the other hand, will only end up in the map marker history if they were map
markers before their deletion.

And: The first map marker (if any) automatically becomes the final
destination if "Directions to" is tapped with no other destination set. But
the same is not true for additional map markers, they do not become
intermediate destinations.

I am not saying all of this is bad or I have a great improvement idea
already, but you have to admit it is complicated now ... :-)  That's what I
meant by "dualism".

On Tue, Mar 1, 2016 at 3:50 PM, vshcherb <notifications@github.com> wrote:

> I don't think there is such dualism. There is dualism Navigation and non
> Navigation. In Navigation markers become less relevant and Destinations are
> the most important. Without navigation there should be no destinations at
> all, otherwise it leads to confusion. You don't know what do you deal with
> (waypoints or markers and that's a nightmare). That's why it is very
> important to delete destinations after navigation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/Osmand/commit/723063ab2da7bb63e737d81719eae4d1517dbb92#commitcomment-16419450>
> .
>


I think it can be simplified as net.ParseCIDR will do both for you if the string is put back in a CIDR format (add "/32" if no / found) if not present.


gcc version 4.8.4 (Ubuntu 4.8.4-2ubuntu1~14.04)


- I think we could remove `checkIndex = true` from `setup()`, but I don't think it hurts. :)
- the observer field is assigned only in setup (that's how it was before), so it can't be `val`, unless I manage to initialize it in the constructor.... but I can make them private, I think


click</a>


Yeah... It is an `interpretation` for high-level users. For us, as Java developers, it is still `compiled` bytecode...


This needs to read `if (this.isotope.options.equalheight)`. `this` is the context of `FitRows`.


What do you think of making the demo ES6 and using moduleserve's babel transformer?


Same as above `if (this.isotope.options.equalheight)`

Great work though, really handy!


Also useful: [`App#addLabel(String, String)`](https://github.com/willroden/marathon-client/blob/master/src%2Fmain%2Fjava%2Fmesosphere%2Fmarathon%2Fclient%2Fmodel%2Fv2%2FApp.java#L219).


Okay, I just reproduced and pushed a fix. Thanks for reporting it!!

On Tue, Mar 1, 2016 at 2:05 PM, Víctor Fernández <notifications@github.com>
wrote:

> gcc version 4.8.4 (Ubuntu 4.8.4-2ubuntu1~14.04)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BSVC/bsvc/commit/456926e72922535d9b347716496f2237efeeb674#commitcomment-16426444>
> .
>


This change causes an ascii encoding error on my install, when flushing the config file (http://puu.sh/nr3P6/d18c2c5489.png) and the seen database. I reversed the change on the new line 160 and the error goes away.


Needs `preprocessedModuleBody  = code`


Ada kesalahan


can you change this angle too? relative to vision.range


Looks like we've got quite a few obsolete lines commented out. If they are not going to be used, shall we remove them altogether?


I guess if you did it with `ElementwiseInc`, you could accumulate floating point errors in `step`.  If we changed `self.sig['common'][1]` to be an int instead of a float we should get integer addition though (assuming that doesn't cause any problems anywhere else).


Shouldn't this be Rect.fromCircle()? It seems like Rect.boundingCircle() would return a circle.


Great solution !


Shouldn't this entry be removed altogether?


License text used is obsolete. Please fix.


Have a newline between the class signature and the declaration of the first variable.


This is not JP stylized.  .config(theme)  function theme.  Also needs a spec file.


Because putting all the babel-transpiling into distfs makes it easier to reuse the build artefacts. I didn't feel like setting up a mechanism for exempting dist/ files from moduleserve's transform, so I just made the tiny remnants of the demo ES5


Leave a newline just before the end of the class definition.


merge these 2 in one line


This is the Web site folder for the Project "Ayubowan". DON'T COMMIT to the MASTER branch


Yes, they are.


These %zu make my builds crash when they're used.


I think the bigger problem is that I need the operator to do an "update", not a "write", to ensure proper ordering and stuff (at least that's what I've got in OCL). Currently our only operator that does updates is SimSynapse, so we'd have to modify `DotInc` to do updates again (I think it used to), and I'm not sure if that's worth it.


Would be great if we can add some class level comment explaining what this is used for.


The changes we did for 1.7 compatibility also make it easier to upgrade to 1.9.


Shouldn't this be a debug log?


What about moving 5 to a macro?


you could move this method to the utilities class


this too?


This can't be the case, as a number of files (notably `Changes`) are not present [in blead](http://perl5.git.perl.org/perl.git/tree/HEAD:/dist/PathTools).

@rjbs Please clarify ;)


Oh! Yes. That's correct.
Thank you!

We can go a bit further and say something like this yet:
```
.... and overrides the `compileStatic` option.
If `CompileStatic` is still requirement, a `new ASTTransformationCustomizer(@CompileStatic)` should be manually added into the `CompilationCustomizers` of that custom `compilerConfiguration`.
```
WDYT?


My fault, fixed in 8b86a813a9c9e124bc487ffd811d259ac5496e49. Thanks for reporting.


I think mot of the code base uses `<<` instead of `push`


Yikes, I have been writing too much JS.


Right, makes sense. I'd favor a new op over modifying `DotInc`.


Let me know if I should pick up your changes and provide the further fix into the PR according this our discussion.


*FINALLY* happy with the text layout.
Sorry about that  :P


why is_active True?


that friction is not necessary: you typically write the class first without the static factory then ctrl+s (eclipse) or ctrl+shift+F9 (idea) then write the static factory. compile all the way.


Shouldn't this line be removed?


maybe we should check for the name NOT matching something like ftl_xx_xx and automatically delete the screenshot in that case.


@hspencer77  I don't think this call would cause any significant load to the cloud. The only use of get_public_ip() is for access logging thread, whose interval is at least 5 minutes. When the IP is determined using metadata, it is cached in the in-memory variable (i.e., metadata is called only once in the lifetime of the ELB).


Это ваше `vagga` не умеет в банальные зависимости штоле?


if you define is_active = False, you don't need this line


License text used is obsolete. Please fix. Also, reformat the contents.


I believe I found the error. I have pushed a change that should fix the error.

The problem occurs when TA configure finds/uses MADNESS that is built outside TA. If you are expecting TA to build MADNESS, cmake may be finding MADNESS in the TA install directory. The configure script should print the path to the MADNESS libraries if this is the case.


I am not sure this solves the uploading in the proper order.


I think, better return {'is_new': False, 'user': user}


Why do we have to modify the default contents of the carbon.xml and repack?


The following doc(s) have been deployed to the [1172tempf](http://1172tempf-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/unsupported-modules-plugins](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/code/unsupported-modules-plugins)
- [docs/articles/sites/files/temp-files](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/files/temp-files)


@ikelca See [sockeye44/instavpn](sockeye44/instavpn).


Yes!


this is not necessary anymore as we provide the accountId as part of the event


As a general comment - applies not only to this patch : we should not modify libnetwork in the vendor directory. The right way to do this is to for docker/libnetwork and vendor it in docker.


You can't have a no user, user variable haven't default value


If this is supposed to be a comment, make it more meaningful.


is to fork docker...


Looks great


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


This comment is incorrect


You like this TC? As stable as SM 4.9?


Should this be retained as commented out configs? Or can we remove them altogether?


This only works in the trail's home region. Also this doesn't tell you that the trail is in the recording state.


License text is missing.


The `echo 'Build success!'` won't work on Windows, will it? Does it need a similar "windows" override?


inconsistent whitespace


Thanks!


I'm not sure if it's just me, but I've got the impression that to get packet value I'd need this 28 line function... Maybe explain its code and/or rename it, as it does way more than get packet value...


Reformat the licensing text. Also fix licensed year.


Update terbaru, semoga bener


Ah damn. I see. will fix probably end of tomorrow.


Checking if I'm following correctly: `filter` here means `nozzle.filter` right?


Fantastic!  Thank you so much!


License text used is obsolete. Please fix.


...and `generic_filter` is `nozzle.generic.filter`?


Only broke linux builds. I am looking at why.


Fix license year.


the concept is more "Value object" not "final class". "final class" is an implementation details.
Also the the example should mention adding a package private no-arg constructor to avoid accidental sub-classing.


+1
Looks good.


Actually I don't want to abuse you with this code. The problem is that I
don't know this part and I cannot say what is internal state and what
should be exposed outside an instance. Just as example checkIndex is public
and I'm not sure if I should search for its usages (now I know that not)
and actually changes state only once? And looks like this `setup` is more
like a jamming in code, its obfuscation. All for better readability :).
Sorry again, anyway I think it;s good enough
LGTM

2016-03-01 20:05 GMT+01:00 Iulian Dragos <notifications@github.com>:

>
>    - I think we could remove checkIndex = true from setup(), but I don't
>    think it hurts. :)
>    - the observer field is assigned only in setup (that's how it was
>    before), so it can't be val, unless I manage to initialize it in the
>    constructor.... but I can make them private, I think
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dragos/scala-search/commit/6d295918057d50baebde7bd034be4bbd9eefa75f#commitcomment-16426447>
> .
>


Sure.


License text is missing.


Hola, no doy soporte para listas, únicamente me encargo de mantener ocasionalmente el programa.
Saludos.


deleted the file


great thanks! :+1:


En este espacio se inicia la creación de proyectos con la herramienta Jupyter y el Kernel Python 3, con la librería para comunicaciones Scikit-rf, además de las típicas...


Fix licensed year.


IMHO DemuxStream should be responsible to format the correct string for displaying, because only Demuxstream knows best about itself and of changes made in future.

Forming a reasonable string could be more then displaying width and height, for inputstream.mpd bandwidth and other identifiers must be set to make a stream unique. Beside the goal should be to form a string wich does not look different regardless wich Demuxer you use.

Searching all over the kodi places where a string is formatted makes no sense in my eyes, I like to change it at one place (where I expect it, at the root - at this example it is DemuxStream).


Remove unnecessary newlines.


`echo` should work on windows too, though I haven't tested it.


yay


Can we declare `option` local. Otherwise it becomes visible outside the function.


//
var s1 = "hello";
var s2 = "hi";
function longest (s1, s2) {
  if (s1.length > s2.length) {
    return (s1)
  } else (s2.length > s1.length)
  {
return (s2)
};
}
console.log (longest ("hello", "hi"));


On Mar 1, 2016, at 9:51 AM, John W. Peterson <notifications@github.com<mailto:notifications@github.com>> wrote:

Nice catch. My compiler did not bitch about this even in --disable-cxx11 mode.

No problem!

Even on CentOS-7, I’ve installed the gcc-compatibility compilers, which is 4.4 based, and that was old enough to trip this.

One of these days we’ll require 2011 I guess, but we better make that decision consciously with our eyes wide open!

-Ben


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


That's what everybody says after writing a bit of javascript.  :)

On Tue, Mar 1, 2016 at 1:11 PM, Ryan Mulcahy <notifications@github.com>
wrote:

> Yikes, I have been writing too much JS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lambdas-of-love/kademlia/commit/bfb29d59de887d358dbe5cf1c10594585847dc04#commitcomment-16426629>
> .
>


Add a newline between the two method definitions.


Cool, thanks! :)


What about creating a new issue here on Github so that we don't forget about these high ULPs?


Always use brackets together with "if" clauses as a commonly followed practice.


good job!


Same goes here as well.


should be removed


```
Evaluate an expression and ignore certain types of errors by converting them to ``null``.

In cases ...

The following errors are ignored by ``TRY``:

...


Hola, los plugins de Kodi/XMBC no son compatibles con Rosadin ya que son dos plataformas distintas.
Lo que podes intentar es usar un analizador de trafico (conocidos como Sniffer) para intentar obtener lo que te interesa de ese Plugin.


License text is missing


Move this to `functions/conditional`


License text is missing.


This is a drawback of google autovalue, but other generator based on visitor pattern (derive4j/ad4j) do not have it.
By the way, I think a chapter on the use-cases and benefits of the vistor pattern (for closed disjoint unions aka sum types) would be a great addition!


Shouldn’t this be `if (this.isotope.items[i].size.isBorderBox) {` instead? You want to subtract the padding, if `border-box` was set, right?


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


var compliance ...


That's done before the try-catch-block, but I admit it isn't clear. I'll change it.


this should be in a helper function


Hey @fujimo-t :)

As you said yourself, several languages have no actual spaces in their sentences. Thus, I'd rather had some API to the system manager checking whether the language is a no-space language, and had support for a setting in the font.lua file. I'll do that right after merging your patch, for you to know.


It'd be good to have a reason this `.get` can't fail.


Oh yeah, I guess Windows does indeed echo.

![image](https://cloud.githubusercontent.com/assets/2192439/13438972/a30f55f6-dfa8-11e5-813b-acb5970c307c.png)


Hi, the threshold is still important. We removed the threshold here, because we developed an user interface where session records can already be prepared. In this ui the session threshold can be configured. The ui enables the user to specify the input raw logs files and then to manually define how the required session data is extracted from the raw logs to the session log. When the ui is ready (we hope soon) we also want to upload it to this git.


Remove unnecessary newline.


provide rounded corners to maintain uniform look.


That's pretty funny.  I think it's trying to check that the explain plan contains an Append-only Scan step but whoever wrote it seems to have just blindly copied the output as the answer file.  I'll update this part properly.


In VSCode, where does this echo get output?


Remove unnecessary newline


The following doc(s) have been deployed to the [1172tempf](http://1172tempf-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/files/temp-files](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/files/temp-files)
- [docs/articles/sites/code/unsupported-modules-plugins](http://1172tempf-static-docs.pantheon.io/docs/articles/sites/code/unsupported-modules-plugins)


Formatting issues. Please fix.


hey, if its easier to input the card data into a spreadsheet, I can hook you up with my set xml compiler.  When I typed card sets out by hand it as always easier putting them into a spreadsheet then doing it manually into the set xmls.


These items have not been added as we do not wish to impose or provide any implementations for message repositories:
-	Created a new table OXA_MESSAGES to store the messages in the DB instead of a file system
-	Added a new column with the message uid for the link with the table
-	Added project oxalis-jdbc-message-repository

All proposed changes related to SOAP (START-protocol) have been ignored, as the START protocol is now obsolete and removed from Oxalis.

The following items have been reviewed and implemented by hand in the "release4" branch:

- [x]	Datasource name should now be included the prefix of the property. The hard-coded "java:comp/env" in OxalisDataSourceFactoryJndiImpl.java is not compatible with Weblogic
- [ ]	Added setRedirectStrategy(new LaxRedirectStrategy()) to As2MessageSender.java in order to solve a HTTP 302 error

  *Postponed as we have upgraded the httpclient library and will rewrite this part of the code.*

- [ ]	Added weblogic.xml (required for Weblog, ignored by other application servers)

  *Not added. Needs to be reviewed as the SOAP stuff has been removed*

- [ ]	Modified web.xml with weblogic settings to avoid performance issues in case of high load

 *Same as above item*

- [ ]	Added ws-addr.xsd (the compilation process was complaining if it wasn't present)

 *Same as above item*

- [x]	Changed logback-oxalis-server.xml to make it compatible with all computers
- [x]	Changed OcspValidatorCacheTest.java with different values. 20ms was apparently too short on our environment.
- [x]	OxalisHomeDirectoryTest.java : ignore a test because it requires an existing folder '$USER_HOME/.oxalis' and I think that this should not be a requirement for a unit test
- [x]	GlobalConfiguration.java:
  * Added support for http(s) proxy
  * Added ability to load the configuration from the classpath because it isn't easy on our servers to create environment variables. However, it's really easy to put the oxalis properties file in a folder that is included in the classpath.
- [x]	From time to time, there was an issue when adding the security provider with the command Security.addProvider("BC"). I changed that to Security.addProvider(new BouncyCastleProvider()) and it solved the issue. (example in SignedMimeMessageInspector.java)
- [x]	Renamed TransmissionRequestBuilderTest to TransmissionRequestBuilderIT : it is actually an integration test with an access to an external SMP. It should run in a different maven phase (integration-test / cf failsafe maven plugin). Our continuous integration platform has no internet access so this test was failing.

  *Should work now as the entire scheme of unit and integration testing has been reviewed*


sucks


I think we want to pass EXTRA_DOCKER_STORAGE_OPTIONS in case of all drivers. This is not specific to devicemapper driver only. Though right now overlay does not support any options, but it is possible to support some later. So can we add this for overlay options too.


Agregue iconos


This is unnecessary


It goes in the same panel as the compiler output / errors.
I thought it felt weird to see nothing, but maybe I missed something.


Since this is duplicated from _print_list_episodes, perhaps you could extract it and use it in both places?


Thanks, I'll try that as take 3.
However, I would like to make the check more specific: only give a penalty if the rook can only move on the same rank. But I still have to figure out how to code this.


Fix formatting issues.


Tested, and it works!


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


Indeed, done.


Yes; please pick up my changes and make corrections - revert to static java code is ok; thanks!


Remove unnecessary newline


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


This isn't necessary


Pro-tip:  `git mv` will let git know that you're renaming file.   (But yeah, I often forgot to use it too.)


Fix formatting isues.


Formatting issues. Please fix.


You can achieve the same thing more cleanly by using sorted():

    for episode in sorted(episodes.values(), key=key_func):
        ...


Why do you need `"" +`  up front?


subscriber, does that make sense?


what does this do?


Does the whole
if (x < guessSquared) {
           return guess - 1;
 }

really ever happen? I don't think so.


possibly still have some bugs in this parser function


License text is missing.


Just my thought about renumber label ID's: we shouldn't do that.
I few like this action will break any previous _.lang_ file that has been already translated.


useless???


Duplicated code. The request can be pre-filled before the async call.


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


Remove unnecessary newline


How is `contentEditableChildrenArray` different from `contentEditableChildren'? As to me it looks like you are just copiyng all the values from one to the other.


Remove unnecessary newlines


Initialize code not implements


Remove non-standard comment


mapping to a correct types


See the cloudtrail sample


Make this comment more meaningful and something that provides a clear idea around the intended context.


Hi i just merged this change into the master branch


Get source value is not implements yet


Probably low-risk, but I notice that all of the `launch_server` groups belong to `t1` and the `launch_stack` groups belong to `t3`. Having `get_all_groups` accidentally pass this test if it filtered by tenant would be a very strange, unlikely bug, but it'd be easy to prevent by, say, changing the 2nd group's tenant to `t3`.


Thanks! This was a great suggestion. It caught an error that I had missed. I did this simply because that is how build_display_list is implemented. I will investigate removing the default implementation for build_display_list as well.


Projeto Simples para aprender Window Builder
Direitos autorais de Leandro Colevati


I'm pushing a change that makes them private and removes the `setup` method. I agree it's obfuscation. A lot of this code has been over-designed and should be simplified


Fix formatting issues.


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


Remove unnecessary newline.


I need to re-evaluate. If you want, I can upload it later.
ATM there is some memory issue which leds to force close of apps like FB. I need to check that first what causing this.


It doesn't look like that's checked in the current code -- not that this change is three years old.


Curious, were they specific problems with letting babel handle ES6 to ES5 transpilation? With webpack2 (currently in beta) now supporting tree-shaking via imports rather than require (enabled via babel plugin) having the babel as the intermediary has definite advantages (smaller bundles for one). Just my 2 cents.


See the comment in the CloudTrail sample


Is there no way to avoid explicit SQL coding and using beans instead?


Make this comment more meaningful and something that gives users a clear understanding as to what it is all about.


You may use something like this.

if (b == (b & rank_bb(s)))
{
   //The rook has no file mobility. This can only happen if the rook is squeezed between
   // a friendly pawn (or a King) on one rank,
   // and a pawn protected  opponent pawn on the other rank.
}


Please use brackets always.


Stack Overflow says there's an easy way to split on whitespace but keep quoted strings together: `shlex.split(str)`
https://docs.python.org/2/library/shlex.html
Seemed to work as expected in my brief experimentation.


I decided not to do a video on how to link tables for two reasons:
1.  The information would be very specific to the Massachusetts APCD
2.  The APCD tables are largely denormalized, so users can often get all the data they need from a single table.


You're missing a call to `done()` here..


It would be awesome to have automated testing around this area, it would make it much more reliable


ack 4e29a81


License text is missing.


_Crash?_  Yikes.

They're a C99 thing, which I thought would be safe, but apparently they were only officially added to C++ in C++11.

As I understand it, _all_ printing of vector indices should be using `%zu`, so we should fix that if/when we ever bump to C++11, but for now I'll just drop this to `%lu` like everywhere else.


multiItemTene**r** :)


Remove unnecessary newlines.


you don't need to mention the sha on my commits since I'm not using the fork, you need the sha if you do the comment on the PR so gitmate knows where to associate them to


When do you see this "Build Success" output? This file (`tasks.json`) hasn't been edited since I pulled the repo from Dan, and I haven't had problems with this extension running in Linux. Is this just a relic file, or does it get used? cc: @pleclech


Wouldn't it be simpler to use array.filter here? Something like:

    var allElementsArray = allElements.filter( function (el) {
        return !contentEditableChildrenArray.indexOf(el);
    });


Remove unnecessary newlines.
 Add a line note


Remove unnecessary newline.


The algorithm seems to disallow this:

```
s = context.createBufferSource();
b = context.createBuffer(...)
s.buffer = null;  // This causes buffer set flag to become true.
s.buffer = b; // Illegal because new buffer is not null and buffer set flag is true
```
Is this what we want?

I think what we want is to set the buffer set flag only if the buffer is ever assigned to a non-null value.  Then any future attempts to set the buffer to any new non-null buffer should throw.  Setting the buffer to null can be done as many times as desired without throwing.


Please add the big (OPTIONAL) note like the one we have in the GLX ABI.

On the question, whether/how much we should care about these broken programs I think we all know the answer :)


How is Thunder performing? Up to par with MSM and Intelli?


Remove unnecessary newline


License text is missing


They patch it in a non-compatible way?


how is this true for pure JSR-269 (part of JavaSE) code generators?


Нельзя, это extern static class variable.


@luisnaranjo733 I had a problem where the fragments I was adding were overlapping instead of replacing. I found this solution online. But I will try to figure out why they are overlapping in the first place and see if I can remove this.


Yeah, looks good. Is it true that you can recover from a syntax error without killing the webpack process altogether and restarting? If so, very cool.


Good point.


I take it that this is to prevent potential issues with the already released binary driver ?

Side note: can we bump the package version (minor and/or patch) in the same commit that changes changes the GL/GLX API/ABI. This allows you some to have some sanity checks at configure time, as opposed to things exploding all over the place.


good catch, reverting this in https://github.com/galaxyproject/galaxy/pull/1830


Не хочу спорить, поправлю.


Control structure keywords MUST have one space after them; method and function calls MUST NOT.


I see you're trying to make a mock for the plugin. However there are many parts missing here; project, core, activeNode, rootHash, etc. Why not use the boilerplate test approach?


Absolutely. I'll go through the UI and figure out which params are good for tool tips. Then we'll have to go through the text to add.


Weren't we going to extract this out into a named fun? And capture the things we need in a much smaller one to call into the named fun?


Is the following example (from the docs) a valid comparison with nil?
```
query = from c in assoc(post, :comments), where: c.title != nil
```
From:
https://github.com/elixir-lang/ecto/blob/6464da626276073495a8dcb61d37b3baafcde834/lib/ecto.ex#L337


I was not able to do that today due to Hangout not working at first, so let us keep that in mind for our next session.


The closing ?> tag MUST be omitted from files containing only PHP.


@EstherDeLoof Good to go!


Why is this in here?  If we need this at all, shouldn't this be a separate commit?


Not required, as you've got it in the register command annotation. (Nice one for picking these annotations up so quickly though!)


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 127](https://teamcity.labkey.org/viewLog.html?buildId=306849&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:00


Should test if group exists and is a valid export node.


closes https://github.com/jharris119/set-game/issues/17


@jprvita On the server side, I thought the group name was `DEFAULT` in all caps. No?


But this is not specific to code generators, this is also true for any library (JDK or otherwise) you use, even the compiler have bugs.


If indirect variable access is considered a warning, then this and the `YieldInExpressionContextVisitor` should probably emit warnings too. This yield usage is completely legal, it just has different meaning in PHP 7


Can we at least add a comment here that we should better do this via the log profiler output?


This is a bit cumbersome - if `fix_authors` is not used anywhere else, it might be a good idea to just revise `fix_authors` in order to *always* return a list (even if it only has one element), and probably even extend it to take care of the strip and replace at 192-193?


Is it worth handling the `ValueError` exception raised by strptime if the string could not be parsed?


No, in that case you couldn't assign generic materials from another groups that are not defined in this blender file, as Ozkan was requesting.


Thanks for commenting the tests.


Renaming this from metadata has affectively broken those that were already clearing this out and doing just what you are saying we could do
`Rake::Task[:metadata].clear
desc "Validate metadata.json file"
task :metadata do
  begin
    require 'metadata_json_lint'
    sh "metadata-json-lint metadata.json --no-strict-license"
  rescue LoadError => e
    warn "Skipping metadata validation; the metadata-json-lint gem was not found"
  end
end
`


Sorry to only come up with this observation this late, but I have a question and proposal on this.

Wouldn't it be better to have `cachedirs` at the job level? I have just tested a single file, with 2 tests, and the cache isn't good across them, and IMHO, that's against the whole concept of caching.

The per-test caching is currently good acrross variants, because their "workdir" is the same. But they're are different across different tests, say `SleepTest.test` and `SleepTest.test2`:

```
JOB ID     : 140ec90e62642a2f6a8c038f9a9a4eac6271faee
JOB LOG    : /home/cleber/avocado/job-results/job-2016-03-01T16.34-140ec90/job.log
TESTS      : 8
 (1/8) examples/tests/sleeptest.py:SleepTest.test.variant1: PASS (1.73 s)
 (2/8) examples/tests/sleeptest.py:SleepTest.test.variant2: PASS (1.00 s)
 (3/8) examples/tests/sleeptest.py:SleepTest.test.variant3: PASS (5.01 s)
 (4/8) examples/tests/sleeptest.py:SleepTest.test.variant4: PASS (10.01 s)
 (5/8) examples/tests/sleeptest.py:SleepTest.test2.variant1: PASS (1.61 s)
 (6/8) examples/tests/sleeptest.py:SleepTest.test2.variant2: PASS (1.00 s)
 (7/8) examples/tests/sleeptest.py:SleepTest.test2.variant3: PASS (5.01 s)
 (8/8) examples/tests/sleeptest.py:SleepTest.test2.variant4: PASS (10.01 s)
RESULTS    : PASS 8 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0
JOB HTML   : /home/cleber/avocado/job-results/job-2016-03-01T16.34-140ec90/html/results.html
```

tests 1 and 5 downloaded the asset (look at their wall time).

Also by moving it to the job level, we remove another attribute from the test class (`cachedirs`), which I think we don't want to support, neither I see the need for it. For tests, `fetch_asset()` seems to be a suffient interface.


Dashes, not underscores.  It's literally one method call.


Hola Alesis :dancer: :dango: :+1:  :D


not true for pure JSR-269 code generators: any IDE that support JavaSE should support them out of the box. (or it is a IDE bug/limitation). I mean, the processor only need to be in the classpath of javac, like any other library you use.


maybe something like https://github.com/stefansaraev/TB/commit/ba793b90670096d888c4611f866015c1a0d64c95 can be useful for you, too.


Start with upper case.


@stubailo should some of these names be updated since it looks like @avital & @yyx990803 might not be with MDG any more?


ack :D


Great use of pattern matching


Funny enough I started doing this the other day. I think we (I) could communicate clearer what's WIP and the remaining TODO list.

I'm all for the patch, although personally I would split it
- update the API/ABI + bumping the package version, always return 0 for the glXAdd.*Mapping functions
- correctly wire up/propagate error handling


Change `o torna-o` to either `o torna` or `torna-o` (I think `o torna` is best here)


You made good use of pattern matching above. I think you could use it here as well.


@gizm0bill you shouldn't really be using any components from here; they're all just experimental prototypes from somewhere mid-alpha. The real Angular 2 Material components are being built here: https://github.com/angular/material2


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


Okidoki!


Consider -

"Check if the queens can attack each other"


Where did `len` go? isn't `f` a string, which will error when passed to struct pack?


I'd prefer if this second console.error is executed only when ```error.code``` is defined.

(or in case it stays, we can fix the typo: ```speficied -> specified```)


Fixed :D @brunoasantos


This and the following tests have the "_"'s left over from minitest?


`a_duration.instance_of? Fixnum #=> true` which leads to `eql?` behaving wrongly as `==`. `1.eql?(1.second)` should be `false` and `1 == 1.second` should be `true`.


Hm, sorry, but I had to revert this. This may well be a great idea, but as far as I can tell, it hasn't been reviewed at all. For something as central as the NixOS module system, that's really not a good idea. Could you please resubmit this as a PR?


"would look like"


Sure. Actually I was wondering which is better and someway I ended up choosing a per-test-cachedir, since a global cachedir can be set in config file. But yes, now you pointed it out, seems like it makes more sense it to be in job level.


Ah, right, I see what you mean.  Looks good then.


Please bump to 1.7 as the behaviour of the lib stays completely the same. 2.0 should be released once #5 and #6 are implemented.


This could be split into two commits: one that adds everything that should've been there before and a second one that adds the `NODE_ENV` entry in `env` and the buildpacks array. (The current commit message for this commit implies it only does the latter of those.)


Typo: nonExistantDir -> nonExistentDir
(here and the following two in the same file)


Именование переменных это важно. На самом деле если обдумывать названия, то код становится намного понятнее. А вообще стоит подумать над этой задачей еще немного. Она решена правильно, но ее можно упросить)


actually was just thinking about this and the `MAIN_TABLE` will have many other properties besides just case properties such as `row.number` etc.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 125](https://teamcity.labkey.org/viewLog.html?buildId=306851&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed to start: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:01


I think you'll find that branching logic is already in
FormProcessorInterface


Was about to send a patch that actually renames it to {Add,Remove}Vendor... like everything else, but if you guys don't use/depend on it that's even better.
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1350.bcc5e44a6cd3181b0aa49356e9b46f817abcd025](http://build.palaso.org/viewLog.html?buildId=69789&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1022, ignored: 12 Build time: 00:06:19


Lol

Je ne cherche pas a voir les erreurs, je les trouves^^
Par contre ce que je souhaiterai c'est une version stable "sans bug"  apparent sur github, histoire de déceler les petites coquilles^^


this makes 1 extremely long line. is that preferred over what I have?


I've created a PR for this: https://github.com/elixir-lang/ecto/pull/1286


ah yes, will change


This parameter could now be `enum _hy_goal_actions` instead of `int` right?


The "automatic" cache dir is always created, even when there are other RW ones:

```
START sleeptest.py:SleepTest.test
Asset /home/cleber/avocado/cache/stress-1.0.4.tar.gz not found.
Asset /var/tmp/avocado_4luBpZ/sleeptest.py_SleepTest.test/cache/stress-1.0.4.tar.gz not found.
Looking for a writable cache dir.
Using /home/cleber/avocado/cache as cache dir.
Downloading from http://people.seas.harvard.edu/~apw/stress/stress-1.0.4.tar.gz.
Fetching http://people.seas.harvard.edu/~apw/stress/stress-1.0.4.tar.gz -> /home/cleber/avocado/cache/stress-1.0.4.tar.gz
```

My current config:

```
datadir.paths.cache_dirs                ['~/avocado/cache', ]
```

Is that intended? It looks like it'd make more sense to look for RW directories first, and then, if needed, create the automatic ones.


Máš pravdu. Já občas trpím tím, že v angličtině se řekně prostě performance a nedokážu k tomu najít český výraz, který to 100% vystihne. Je to neduh toho, že celý den mluvím anglicky :smile: díky za připomínku


Would be nice if these tests were commented too...


Thank you for the pull request. This is a great step to address the issue.

One concern I have is that this may lead to a situation where some records in the database will never get sent out and will be indefinitely retried later times.

Instead, we should look for specific kinds of failures (e.g. NSURLErrorDomain with specific codes such as NSURLErrorNetworkConnectionLost) in the `- submitRecordsForStream:records:partitionKeys:putPartitionKeys:retryPartitionKeys:` implementation and handle accordingly. What do you think?


Proyecto de comunicaciones analógicos en jupyter y el kernel pythonj, con la libreria principal scikit-rf


no, you need to call explicitly the init code. this is less magic, amd like and also in non-amd environments a better approach. thus its not backward compatible and needs a major version bump - according to semantic versioning.


You forgot to return :D


You should also refer to github docs:
https://help.github.com/articles/configuring-a-remote-for-a-fork/
and
https://help.github.com/articles/syncing-a-fork/


Store regex in config.yml so that you can always edit it whenever you want


Added precice; resolves #4


I think this is the same issue with radio button. It also has margin-left: inherit which causes issue when minified.


It will compile with PyQt5 if the build system is modified.  The changes to the c++ files are minimal.  It won't run without many changes to the python files, but nearly all of it is covered with
s/PySide/PyQt5/
and changing from traditional style connector syntax to new style, which is also compatible with pyside.  Ref: https://wiki.qt.io/Signals_and_Slots_in_PySide

The changes to the build system are minimal as well.  After detecting PyQt5 in the top level CMakelists, s/PYSIDE/PYQT/ covers the rest of the changes.


may remove later


perhaps, though dplyr and we would want to have a `.default` method problably, so probably would still collide, can you open an issue?


This is a consequence of using floats. Without knowing how to clear floated elements, there are odd effects on your layout. Nice use of floats though.


may remove later. This is used to count complexity.


@lstanii-magnet Please review. This is what I did for infinite scrolling on the Support page.


+ 1
0 additions and 6,540 deletions. Lol


master still gives some dimension mismatch error while running attention code. I am working with a branch corresponding to the pre-thin-stack merge state.


:clap:


this error


Gotta watch that Atom!


Ok, I didn't revert. I wound up re-setting top of tree. Please don't re-commit anything yet. I want to verify things are back to normal. Then I'll see about committing what you mean to commit. Then we can chat. :)


@weisskrispies Its done. Just get the latest code for the new func i added in readme, hope it helps


This needs to be changed back. We're going to end up snapshotting the wrong files here, as `new SimpleFileCollection(collection.getFiles).resolveAsFileTrees()` is not the same as `collection.resolveAsFileTrees()`, in the presence of patterns or resources that are backed by an archive.


I don't know the issue at hand, but is there a reason not to run at head? fat_classifier.py should maintain all of the old behaviors of classifier.py, including working attention (at least if there aren't any lurking merge glitches).

> On Feb 29, 2016, at 6:09 PM, abhirast <notifications@github.com> wrote:
>
> master still gives some dimension mismatch error while running attention code. I am working with a branch corresponding to the pre-thin-stack merge state.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hans/rembed/commit/4352c8b0c099ff5c49e26083d808afea0cf6e35f#commitcomment-16407589>.
>


これどういうこと？


deck = user毎のお気に入り　みたいな認識でおけ？


I was trying to provoke Garrett!!!!


Yeah, we attempt to run it all with deferred constraints. I like to not tempt fate if there's also a valid order!


てなると、ここら辺いらないんじゃないかと思う


書いてたらspeakingの練習になんなくない？
http://qiita.com/inouet/items/2c9e218c05f547bb6852
こんなのとか使ってみてよ　笑
(使えるかは分からんけど、任せる)


Yes, it would be good to add to integration tests, however there are few issues with it:

* Though this change adds an option to enable/disable version in listing it actually only matters if the
bucket is configured with version (false by default).
* Even if we enabled object versioning (lets say by creating another bucket with versioning set to true),
the test would be a weak test. Google Cloud Storage listing functionality is eventual-consistent, which
makes already existing similar tests weak.

Regardless, I will add something to be consistent with what we already have.


excellent use of rgba


sp. names -> named


deckの役割がよくわからんから、教えて！
役割次第では中間テーブルで全て補える気がする


Keep an eye on your indentation. Good use of the em tag.


If we went the route of setting the values automatically, I think we'd want to use `(-DBL_MAX, DBL_MAX)` because min would give essentially 0.


Great job using Google Web fonts!


Watch your indentation.


This is excellent work. You've covered all the requirements and added several nice extras.


I think it’s probably better to let the engine crash until this extra parameter is evaluated and properly implemented, rather than to make it silently pass through with a TODO that someone might accidentally overlook. Happy to solicit others’ opinions on this though!

(Also, a third parameter is definitely *not* handled by SQ6/SCI2.1mid, so the signature should probably reflect this instead of being `SIG_SCI32` with an extra optional?)


Ok, got it


I applied this patch.
Is there an additional patch for user can select higher clocks?
It seems higher clocks not appeared in clock tables.


With this change, UI will initially be populated with the mailboxes in the local database. Once that is complete, a separate thread will calculate the differences between the server's mailbox list and the local mailboxes. Both the database and the UI will be updated when there is a difference.


The muffling is really in `applyResult` — `SyncResult` is `Deferred<Maybe<SyncStatus>>`, and that gives us the ability to say `.NotStarted` and to add more states like that in the future.

There's some omitted richness here and elsewhere for when we're partially downloaded, when the green light isn't illuminated, etc.


Was this supposed to be committed?


Some sort of code refactor has changed all of the occurrences of the word `form` and replaced it with `forms`. This is a breaking change that affects many areas, including renaming this endpoint.


The original color should be fine here. We'll look at color values in CSS, but colors will need to start with `#` or follow one of the other color value types.


You are the MAN! Thanks so much, it's working perfectly.


Your indentation is mostly good, but there's a ton of whitespace. Too much whitespace limits the amount I can see on the screen at one time.


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


Keep an eye on your indentation. This makes it seem as if the h1 is a child of the ul.


These items shouldn't be needed other than the fix you made to `float`.

Elements can have more than one class and both will apply.

We'll look at the "Cascading" part of "CSS" to explain more about how this works.


Did you get this?


The link here is broken


doesn't seem like this should be called for `did-navigate-in-page`. actually i'm not sure why `loadEnd` is called for `did-navigate-in-page` at all.


This function doesn't have any parameters.


accidentally deleted earlier comment by rebasing + force-pushing...
@Memphiz , what are these lines needed for? Which paths do we have to sync here?


It's good that you added a new style sheet, but you'll avoid repeating yourself if you reuse your css when you can.


leads to an infinite spinner w/o


@Aeusman See #132 here. You need to tie this into that. We need to have a single place in ndio where the server name is set.


Because these img tags are related to the li's, it makes sense that they should be inside the li tag. If you're trying to get the img to break to the next line, try playing with the display property in your css.


@Aeusman These 2 functions are what we need to make it less involved for the user. We can discuss how to implement these offline.


To remove personal voice, could the final sentence be

> In Version 6 and above, use the explicit immediate dispatch API instead.


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


Nice selector


nice use of floats.


I believe these are the defaults that the controls currently report now as min/max values. But I will double-check tomorrow.


The clear property doesn't do anything in this case, but you'll learn about this soon. Again, good use of floats.


@Aeusman Again see #132. I think with this you can remove the dev tag you are passing around and have only a single URL. Also, I think you can talk with @j6k4m8 and tightly couple this with build_url function in neurodata.py


typo


Good job figuring out this property.


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


There's probably some fat we can trim in this, but so far so good.


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


Do we need a constructor that instantiates the title, description and prerequisites?


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


When `archs` is not specified, this code breaks. In addition, nobody with the last name Moore is on the [Cordova committers list][committers]. I'm reverting this commit because it's breaking the build, and I will follow up on the private mailing list.

[committers]: https://people.apache.org/committers-by-project.html#cordova


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


Great


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


2055_CommasInNames (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


This is a solid submission. Pay attention to the small details like indentation and typos but overall everything looks good.


Just continue with your part of the use cases, and as we go we can figure out exactly *what* can be trimmed.  If you need help, just ask.


@Aeusman this is a flawed assumption. There might be a case where the user can have an existing project and an existing token but wants to add a new channel. We cannot do a blanket check for token and reject this. Here we will have to something more complex which is comparing dataset(backend) to dataset(provided by user) and more. Another alternative is to skip this as I anyways do this on the backend.


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


2055_CommasInNames (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I'm just adding this so everyone can work on their use cases.


Thanks man. How did I miss these?!?


Did you test this one? I doubt the exact numbers are the same :/


This would only occur with gateware bugs. Where do we stop checking for this sort of things? Should we also raise an exception e.g. when the SDRAM controller fails to ack a transaction?


IntelliJ is fine here but compiling it, maven went psycho, so I'm not sure on this...


You could just return true here and return false after the for loop.


def max(e, d, v):
    if e == d:
        return v[d]
    else:
        m = (e+d)/2
        maxe = max(e, m, v)
        maxd = max(m+1, d, v)
        if maxe >= maxd:
            return maxe
        else:
            return maxd
# O professor desculpa não consegui fazer o min_max recursivo só consegui fazer o max


yes, you are right, maybe we should change it to LLVM_GIT and
LLVM_GIT_PROTOCOL

On Mon, Feb 29, 2016 at 8:01 PM, Khem Raj <notifications@github.com> wrote:

> I would have like LLVM_GIT but not too hard on it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/walkthetalk/meta-clang/commit/13ec20a3c20f0045f593e1b9a37ccc7f45d1581c#commitcomment-16399167>
> .
>


AndroidRecordVideo(安卓录制短视频)


Line 73 was added with .upper() before. So, to generalize the code, cutting site was turn into upper case too.


It's not about size, it's about ergonomics for the user:
```rust
Source {
glsl_120: Some(include_bytes!("shader_120.glslv")),
glsl_150: Some(include_bytes!("shader_150.glslf")),
.. Source::empty()
}
```
This would be too verbose, for no strong reason. I realize that `EMPTY` is slightly not idiomatic to Rust but it really wins in terms of usability, and we aren't really losing any safety here.


Sure


Send a PR when you're ready.  And please clarify what the fields mean; actually, are these based on any samples?


What other models would need a votes member variable?


Yup, good call.


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


Making lists here and testing for items in those lists could raise annoyances later when styling. For now though, I think it looks good and works well.


There is already a system notification for this.


To answer the question in your test file, yes, we are using the singleton pattern for the ServerFacade. Thus the constructor should be left private and not be changed to public. In order to get an instance of the ServerFacade simply call the getServerFacade static function.


Watch your indentation. As it is, it looks like the p is a child of the h2.


** I originally performed an informal review of this commit on Saturday, but only posted comments in Taiga. This is a copy of that comment. **
Verified buttons have been added to the dialog appropriately. Event handlers are not implemented, but those were not required for this task and need the other tasks done before implementing.


We covered this before class, and I'm sure I'll see it in the next commit, but you don't have to qualify class selectors with a tag selector (i.e. you can use .bold instead of p.bold or li.bold)


@gavinbunney gonna fix up runMadcow.bat too or hate windoze ?


The structure of this list is perfect.


** I originally performed an informal review of this commit on Saturday, but only posted comments in Taiga. This is a copy of that comment. **
Tasks do not always have an end date set. The template should take this into account and be able to handle null values when setting values. See new unit test from https://github.com/cst316/spring16project-Ada/commit/ac5604a6b2520998212a0d638ebbb5f90abb2d5f. One way to represent no end date would be to return a -1 for each value in the difference array.


It's not the prettiest thing I've ever seen, but it hits all the requirements. If you're interested in making better layouts, I can dig up some resources on web typography for you. Nice work!


Conceptually, a `TripEvent` represents the state of the attraction for a particular trip (votes, where it is in the itinerary, etc). `Attraction` is the actual place itself, independent of a trip. On the attraction details view, we also surface the current state of what people think of the attraction (vote counts), so it will help to have the information accessible from the `Attraction` model.


Can you please update this for PHP7?


@DanHickstein fantastic, and laborious,  work on your part. The only downside it the loss of ascii art from the actual program files, where many users would first look:
```
-                   ∞
-                   ⌠
-               -1  ⎮   g'(R)
-       f(r) =  ─── ⎮ ──────────── dR      Eq. (2a)
-                π  ⎮    _________
-                   ⎮   ╱  2    2
-                   ⎮ ╲╱  R  - r
-                   ⌡
-                   r
```
has become:
```
.. math:: f(r) = \frac{1}{\pi} \int_{r}^{\inf} \frac{g^\prime(R)}{\sqrt{R^2-r^2}} dR,
```
Identical, (and clever that LaTeX formatting is used), but not as instantly recognisable.

No big deal, though.


this is causing problems in rails 5 beta 3 if no filename is specified. it will raise ```no implicit conversion of nil to string```.

any idea where ```@tmpname``` is coming from? I fixed this for the time being by replacing ```@tmpname``` with ```path```. It seems to do the trick but I couldnt find any references to the ```@tmpname``` variable and trying to figure out the best way to solve this problem


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


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


Interesting! I will figure out what is wrong. Thanks


it manually inlines `error`. this can get pushed to master at any time, it just currently landed on top of my WIP


It should be a    "*.c" file , We will update the instruction.


I thought this would add a separate history entry for every anchor in a page but looks like that's not happening, so lgtm


`s/thrown/throws/`


thats for sd card stuff?


I think it would too, not sure why not.


nvm..um yea ill look into it


lgtm

unrelated bug (it was happening before this commit): before i ran `npm clean-session-store`, this line in `urlBarSuggestions.js` was erroring because `formatTitle(site)` was undefined:
```
return data.findIndex(x => formatTitle(x).toLowerCase() === formatTitle(site).toLowerCase()) === index
```

Probably need to add an undefined check in case it happens when people update to 0.8


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


Still on the to-do list:
* Problem by Problem analysis for 20, 200, and 1000 item count problems
* Does number of generations make a difference analysis
* Analysis of the crazy tree at the end
* Conclusions


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


I like this approach. Using a temporary variable like this can make debugging easier.


it looks like to me the rmt_storage handels special devices for the kernel. Which im guessing is kernel specific hence probablly why it didnt work.


adf


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


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


Thanks.


Heads up: your changes won't be merged if you change the icon names from `ic_abc` to `abc` -- the `ic_` prefix is to denote that they are icons. Any `drawable` without that prefix is **not** an icon.


Do you have a screenshot of this? I'd love to see it!


2 space indentation


Можно было так и оставить {% include 'news/news.html' %}
Код в обоих шаблонах пока одинаковый, он выводит N новостей. Разница только в количестве новостей, которое определяется в views.py.


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


This should be prefixed by "For non JSR269 code generator, " ;)


Ahh. Thanks for clarifying.


use javascript instead of JSON here otherwise it will all be red.


We renamed the package from `nix` to `nixio` yesterday. Sorry.


Stupid text replace, that should of been lowercase but whatever... We'll get to it as it's a compile error.


`nix` -> `nixio`


added the link and spelled out the acronym


Jejda. Tak to jsem vážně poletl. To mohla být chyba jak blázen. Opravím


See: https://www.drupal.org/node/2676752


More precisely, it freezes and the OS ends up aborting it with the "<Program> has stopped working" dialog, with a BEX (buffer overflow exception). Running in in a debugger shows that it performs a fatal error in the runtime library.


Hmm, OK. This still seems like the best option for getting all the case
properties if we can find a way to filter out meta properties.


lines 105 to and including 111 can probably removed. Anybody who is installing a python package should be able to start an interpreter ;)


Давай все это вынесем в файл .properties , отдельный для production, и отдельный для dev, и грузить нужные проперти как это сделано для настроек БД


With the February release, we're adding a Shell command to add code.exe to the OS X $PATH environment variable (see https://github.com/Microsoft/vscode-docs/blob/vnext/release-notes/latest.md#os-x-shell-command).
Removing this section from the docs because I don't believe it will be needed anymore.
@rtorr please let me know if this is incorrect.


I really like this approach


Excellent, .. It will be nice to hear the story behind these decisions, but I like this setup, much needed flexibility ..


I'd prefer `self.assertIsNotNone` here.


oh sorry, I just wanted to use this grid directive and I found the sourcecode via google


That'll  only work on Container public builds. mac is running a vm.


0.2.2 hasn't been released yet.

my standard practice is to bump right after release so that we never trample an existing release.  that's why it's at 0.2.2 now :)


Alright, I'm on it.
Need to reset last commit.


>I like to change it at one place

It is at one place now ;)

>(where I expect it, at the root - at this example it is DemuxStream)

Yes, maybe you're right. But for me information representation and formatting is rather related to gui than to some levels further down. Thinking more about it, you want to do it there because you don't want to propagate all that identifiers and stuff up to the gui, right?


`assertTupleEqual` might give a better error message in the failure mode.


As mentioned [previously](https://github.com/AppFolioOnboarding/perkins_eidolon/commit/c3023a3b32c05e0ba005223edd082780b7657bcf#commitcomment-16403929), this commit should either be squashed or removed.


and I forgot closes #16030


Yep. Angular 2 will not support this, but then that will be an entirely different animal anyway.


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


Consider modernizing the for loop next time.


OK @elieserdejesus. Só mantenha o parâmetro "-Dvisualvm.display.name=Portugol-Studio", pois ele faz com que o nome seja exibido corretamente ao depurar a aplicação no JVisualVM e no profiler do NetBeans.

Além disso, pra isso funcionar será necessário incluir este parâmetro no projeto que inicializa o Portugol Studio. Pode deixar isso comigo.


Thats a good plan. In this case, me bumping the number actually wasn't really needed. But as I make breaking changes and implement features in the Asthma app that rely on those changes, we'll have to bump it and tag it. We'll also have to decide when/how those things will land in the AsthmaApp. Keep it in a branch/fork until the changes land in a released CocoaPod?


I guess you didn't make it `const`, because `mc_search` takes non-const arguments and it's annoying to go ahead and constify the whole call path?
```
const char * const syntax_type = edit_get_syntax_type (edit_widget);
```


for the homebrew tap only?


именование, остальное ок)


Adding a new method to an interface in a patch release should be considered a backwards incompatible change and *not* following semver. This breaks the cakephp/migrations plugin (boo) with the 0.5.2 release (I guess we'll implement it soon). Boo :(


These changes are not taking place through `npm install passport-23andme`.

Still using `getProtectedResource`


Nice dude!


Should I change `NIX` here, to `NIXIO`, too?


Öh, das wird wieder rausgelöscht, ich denke einmal Angular einbinden reicht.


решена верно


• Added ragdolls for Cyberdemon, Hellknight, and Imp
• Added proper mass values on heavier monsters


very nice :+1:


Все операции проводим с деньгами осуществляем в  BigDecimal


Bah. Suffers from the same problem (just worse now since there's only one parent process…)


Well, that's a good point. I'm adding the fallback cachedir as the last `cache_dirs` list item in line 210 above. This keeps the utils.asset module simple by not expecting a fallback_cachedir argument and also not needing to probe the fallback cachedir in utils.asset from a core module, which would be a module boundary infringement.


@BnJam
```
src/ItemHUD.cpp: In member function ‘virtual void ItemHUD::displayHUD()’:
src/ItemHUD.cpp:30:44: error: invalid use of member function (did you forget the ‘()’ ?)
  dmg = ItemHandler::getInstance().getWeapon->getDamage();
                                            ^
src/ItemHUD.cpp:30:44: error: base operand of ‘->’ is not a pointer

```


must `conform to`


Souhlas :smile:


I think this is for the municipality data, right?

So maybe "Municipality_Zika" as the report_name - shorter and no spaces?
unit should be cases

Others look good!


Masking broke here. (See issue #215 )


Good point. Going to notify the appropriate people.


Sorry, wrong issue no. it should have been #612


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


Poderia colocar de melhor maneira os pontos e tem alguns erros de digitação também.


How about something along the lines of

```
std::size_t offset = family.find("_");
if (offset == std::string::npos) {
   // PANIC
}

const char* szIRMeasure = family.c_str() + offset + 1;
```


Zkusím to nějak upravit. "Už brzy nebudeme muset řešit 300ms zpoždení prokliku." také není ono, protože to v podstatě budeme muset řešit tím, že budeme muset použít v CSS `touch-action`. Něco vymyslím.


It looks like a bug. Since the Python 3 branch is commonly not executed it is possible nobody noticed before. Can you please create a PR which fixes this.


Допер,не увидел прошлого письма. Учту,спасибо за замечание. На самом деле привычка пошла с acmp.ru . Там ,чем меньше код,тем выше ты в общем рейтинге. Буду исправлять привычку)--Отправлено из мобильной Яндекс.Почты01.03.2016, 23:09, "boikov" <notifications@github.com>:именование, остальное ок)—Reply to this email directly or view it on GitHub.￼


However from my experience, there can be friction when the annotation processor fails to generate the code, most often due to other part of the project not compiling properly. Then the static factory above will not compile. This often  acceptable because limited to a single class/method.
What is not acceptable is when a generated type is used in method signature all over the project. Then a large portion of the project might not compile due to the code generator having failed.

For this reason I try to avoid JSR269 code generator that expose new type for general use.
Ideally, JSR269 generated classes should never appear in public APIs. Thankfully this is the case with google autovalue (and also derive4j).


It kept getting out of sync because I'd forget to tag it und update it, so mostly my own laziness. I can add Bower support back if you need/want it, but IMO npm has mostly wiped out bower usage.


[R/pols_501.R:184:1:](https://github.com/UW-POLS501/r-uwpols501/blob/92499871425531d92216068f024b4f72052ed1fe/R/pols_501.R#L184) *style:* **Trailing blank lines are superfluous.**
```r

^
```


Unfortunately I'm not familiar too with Thumb ISA.

Can one build a 'universal' binary - one that uses thumb, yet runs on ARMv7 that lack the instruction set ? And I'm not talking about having two separate code paths and choosing the right one at runtime.

If the answer is no, then one can nuke the "thumb vs not" distinction. This is an implementation detail which should not be exposed, but controlled at build time by distributions. Sort of like softfloat and hardfloat builds for ARM CPUs.

Same goes for the x86/x86_64/ARM split - one can build only against a single platform. It's not like we can build against ARM, and try to use x86 at runtime ;-)

On the TLS vs TSD topic, if we want to keep them, it should be an explicit configure/build time decision and one should be able to query it (store it in the libglvnd.pc ?).

Afaict TLS is available on pretty much every Linux/BSD/Solaris? supported distro or am I missing something ?


lol just made a similar commit with pull request seconds ago


Для турниров это ок, но наша задача это уровень для реальной работы, а в реальной работе коллеги должны понимать твой код) правильное именование это важно)


There is. to an artilce I've been putting off authoring for months. I didn't realize that was still in there.


Yo, I’m here.


Я понял,впредь буду называть нормально)--Отправлено из мобильной Яндекс.Почты01.03.2016, 23:18, "boikov" <notifications@github.com>:Для турниров это ок, но наша задача это уровень для реальной работы, а в реальной работе коллеги должны понимать твой код) правильное именование это важно)—Reply to this email directly or view it on GitHub.￼


Couple of problems with this approach.

There may have been an issue with communication, this is build on top of BitcoinClassic 0.11.2.
I'm not sure if that makes sense.

The second problem is that this shows a dialog that requires an answer. For a commandline version (bitcoind) this looks like a sub-optimal solution.

For the immediate future, the 0.12.0 release (we don't plan any 0.11 based releases) we ended up with a simpler solution that leaves users that have obfuscation to keep it. New users won't get obfuscated.

In the future a standalone app that cleans the chainstate from obfuscation may be useful, Pedro started one but not sure what the state of that is.

Thanks!


Actually the change proposed in previous comment to use a per-job fallback cachedir can cover this as well. I will inform the path, bot not create it in advance. It will then be created only if no other rw dir is available. Thanks.


I think we can and should be a lot more aggressive about overwriting old metadata with new metadata. The current cautious came from a time when importing metadata from an OPDS feed would potentially overwrite _all_ other metadata about a book, from _any_ source. Now importing metadata from an OPDS feed will only overwrite other metadata from the original source of the OPDS feed.

I've created this helper class to contain an application-specific policy for metadata, so that I have a place to put these kinds of experiments.


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


If you leave the version tag out of the `bower.json` file, Bower will automatically use the Git tag as the version.


Come to lab 115 we need you to work as a group

Sent from my iPhone

On Mar 1, 2016, at 12:14 PM, Daniel Guenther <notifications@github.com<mailto:notifications@github.com>> wrote:

@BnJam<https://github.com/BnJam>

src/ItemHUD.cpp: In member function 'virtual void ItemHUD::displayHUD()':
src/ItemHUD.cpp:30:44: error: invalid use of member function (did you forget the '()' ?)
  dmg = ItemHandler::getInstance().getWeapon->getDamage();
                                            ^
src/ItemHUD.cpp:30:44: error: base operand of '->' is not a pointer

-
Reply to this email directly or view it on GitHub<https://github.com/danthemango/Kitten-Pirateer/commit/2799a757906133438684b1ff251eb58dce82142b#commitcomment-16428295>.


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


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


Bachir Niang : je pense que quelques explications s'imposent.. .
Tout d'abord Ngagne tes fichier config la je ne pige pas ! tu peux trouver un moyen de nous faire parvenir l'exemple que tu suis ! en lien
merci


why have you removed the 'é' here (L705)?


Does it really need to be a hard coded relative link?


See comment above.


Cool! If you get to solve both issues at once it'd be awesome!


creo que no necesitamos `$scope`, o sí?


Oh wow, well that makes it just another file chilling in the repo I don't have to touch. I can go ahead and publish to bower if you'd like. Is there any reason in particular you aren't using NPM, though? I'm just curious. I'm actually kind of curious how you'd go about building a SPA w/o NPM + browserify/webpack. Seems like a headache.


voy a consultar por que se lo usa para el ionicModal.. no investigue a fondo eso