RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2174 of 2215 tests passed)

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


That's some good regex there bud gg


@chaityabshah yes :)


fixes joaoneto/angular-bootstrap-select#31


Yeah, switched to that. Takes forever to build 5.22 and blead, but at least it's comprehensive. Thanks for the pointer.


Ok, egyelőre úgyis más „mókát” találtam.
Remélhetőleg most már mindenhol, ahol lehet, van u2d.
Esetleg még a problémás elérhetőségű forrásokat kellene valahogy leszedni, eltárolni.
KDE-ből pedig, ha minden jól megy, lesz 5-ös. Már elkezdtem összerakni.


Why you like this? I would go by the style it is already there. It is part of python documentation


I think this might be backwards...

By default, hbck doesn't modify anything, so it should be lockless. If any of the -fix options are passed in, then it should take the lock to prevent simultaneous fixing and turn off the balancer because it doesn't need things changing underneath it.

If you only want one hbck running at a time, maybe there is a good use for an -exclusive flag that also sets the lock and turns off the balancer? But is there any time you'd want to mutate the database in parallel? You could just delete the lock...


8 line for just 1 function call? It makes it very hard to read. i think the original is fine.


same as the top


i like inline with the first param. much clear


geez and I broke the build. I can't get these tests to run successfully locally so I can't fix properly. Any ideas?


this is fine


I got two dialogues when I tried to shrink the image.
Some commits later, I chose another solution (redo action should remember users first-time choice).


El key hay que hacerlo configurable, lo dejamos para despues, pero nada mas para tenerlo presente


Thanks for the feedback. I updated the pull request with some tests and
fixed the interpretation of empty lists.

On Thu, Jul 30, 2015 at 1:09 PM, Mihir Kedia <notifications@github.com>
wrote:

> also, can you update the text of partials/admin/project-settings.html:70
> to reflect the new syntax (and that "blacklist trumps whitelist")?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/msolo-dropbox/changes/commit/e03e477d3a81607bee0225f2de48a7f4cc6ae6e4#commitcomment-12453370>
> .
>


I actually mentioned that 6 days ago in a review:
* https://github.com/PacificBiosciences/pbcommand/pull/7#commitcomment-12352811
* https://github.com/mdsmith/pbcommand/commit/286d474e41f609f857d373e43e21996a1a0ad437#commitcomment-12352811


One possible reason for the delay is that a client try to request null video from mediaq. Rectangle query can return video location, time without actual video (i.e., the video_url is null) when users have not uploaded the video yet.

I tested the newer version but no video shows up. Below is the log.

![screenshot 2015-07-30 16 18 14](https://cloud.githubusercontent.com/assets/3415436/8997487/a824bb1a-36d6-11e5-90c1-5f43901423fd.png)


MemberArray_Reduce (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1261 of 1261 tests passed)

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


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I like this. Thanks


PietCreator crashed when trying to pop a command from empty command stack.


this is okay. But for function i like Aligned with opening delimiter. http://legacy.python.org/dev/peps/pep-0008/#indentation


> doesn't work

Can you give more details?


I feel like it is much easier to read if I know the xml_content_header is part of the util method. I would keep the original


I like the method to be like the original


same as above


Undo of shrinking did not restore lost pixels.


I also appreciate the feedback and knowing that this has been helpful! Going through the history of your changes is helpful for me to breakdown the review into more manageable chunk. What I lose by repeatedly reviewing lines that change multiple times, I gain by not being paralyzed by a huge chunk of code to review at once.


Cada vez que renderemos un campo de un formulario, primero checamos si este campo esta entre los `customized_fields` que ya definiste, entonces si vemos el archivo https://github.com/codn/adminpanel/blob/master/app/views/adminpanel/shared/_form_fields.html.erb

cuando llaman a renderear el parcial del campo custom tenemos varias variables a nuestra disposición (dentro del parcial)
* `f`: que es el objeto del formulario
* `properties`: que es lo definido en form_fields (a la derecha del `=>`)
* `attribute`: el nombre del campo, atributo, propiedad
* `is_modal`: si el formulario va a ser rendereado en un modal o no

Lo que tu necesitarias es cambiar `:map` por `attribute`


opss


full name please


utilizar una clase en vez de un id para que tenga soporte de varios mapas


@willsalz that approach will be really hard to scale.

Your basically creating one big fat ringpop for all services in uber.

Any `sk` concerns should be done by ringpop+tchannel; hyperbahn just does forwarding.


this is much better


**Regarding the array initialization**
You can't define the questions the way you do. You're going to end up with an array full of undefined values:

    [undefined, undefined, undefined, undefined, undefined]

This is because the variables aren't defined as their final objects yet. When you store those variables in the array, they're stored by reference, meaning the reference to the "object" `undefined`. Later, you change the reference of `q1`, `q2`, etc. to a different object at a different place in memory. Don't worry, you'll get the hang of this later. If you move this line to the bottom of this file, it will work.

**Regarding object initialization types**
I think you're right, the factory or constructor patterns are the easiest. I always use the constructor method of creating objects, but you do what you understand best. I eat prototypal inheritance for breakfast now, but it's painful to learn.

Something to note here is the unwieldiness of passing in so many arguments to a function. Imagine coming back to your initialization code on lines 43-47 and trying to remember which argument was which. What if the order needed to change?

A common method of handling this is to accept a single parameter that is an object:

````javascript
function createQuestion = function (params) {
  validateParams(params) // Also a good idea
  var obj = {
    questions: params.question,
    // etc...
  }
  return obj
}
````

Then you can initialize like:

````javascript
var questions = [
  createQuestion({
    question: "Some question?",
    selections: [
      "Answer 1",
      "Answer 2"
    ],
    // etc...
  }),
  createQuestion({
    question: "Some question?",
    selections: [
      "Answer 1",
      "Answer 2"
    ],
    // etc...
  }),
]
````

**Regarding constructor functions**
The Thinkful lesson doesn't mention this, but you can do the same thing with it as you do with the factory pattern regarding functions:

````javascript
function Question(answer) {
  this.getAnswer = function() {
    return answer
  }
}
````

Then you can call

````javascript
question1 = new Question("answer")
answer = question1.getAnswer()
// => "answer"
````


Redo of insertImage should not ask the user again how to handle too large images.


I wrote my code is properly. I dont know why it is showing wrong format.


The current aok.yml patch works fine on the branch, but when I applied it to the most recent dev build it broken because 'cloud_controller' is no longer the last entry in oauth/clients/cf


According to the [docs](https://developers.braintreepayments.com/javascript+node/start/hello-server#install-and-configure), it doesn't look like `new` is required here.


good one


what is resp


`$time` will be a Unix timestamp obtained from git blame raw output. With `date -r`, I convert this timestamp into a human-readable date in "YYYY-MM-DD" format. This has only been tested on OS X. If this doesn't work on your system, please provide me with more details so I can fix this. Thanks!


this are static variable, not part of the object


Ok np. Lets look at it tomorrow.


This line is a bit confusing. Doesn't seem to be necessary. If it is, it's unclear what `key` should be.


I don't know the intention, but I did not like it when the palletes had selected colors that were different to the active color. And I did not like it, when two colors were selected (a "real" color as well as black or white).


Please read the comment and fix based on the comment


Ok


thank you


MemberArray_Reduce (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1777 of 1777 tests passed)

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


Wasn't for incoming, was for outgoing-- if I'm missing something, please do
share.  Probably, I'm missing something.

Thanks!
-Jake

Jacob Gadikian
E-mail: faddat@gmail.com
SKYPE: faddat
Phone/SMS: +84 167 789 6421

On Fri, Jul 31, 2015 at 3:34 AM, Aaron Judd <notifications@github.com>
wrote:

> can't you just expose these on the run command? why do you need incoming
> traffic on these?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/faddat/reaction/commit/e8c3dddd2a247f53afad99680d2d62a17da32db5#commitcomment-12453920>
> .
>


Instead of the loop, how about:
```C#
Array.Copy(buffer, offset, bufferSegment, 0, length);
```
?


Changes I forgot in the prior "KColorCell commit".


Gracias! se me habia pasado por alto.


When exactly is a licence considered 'old' and its header not needed any more?
I thought the Apache License sticks, even if a file is used many years later and modified?

This is a sincere question, I want to understand a bit more about licensing and how to apply licenses to my own projects and saw this commit while getting the newest version of this nice little plugin ;)


Yes, the grains/core.py file could use a great amount of work, but it will be very difficult if not impossible to disturb without exhaustive testing in place.


This is my bad for not checking your follow-up commits. There's definitely no need to leave any comments if the line is intended to be removed right afterwards.


Hmm, I only just saw this comment... this actually isn't true with ```Dictionary<>``` but it is with ```Hashtable```.  I thought I'd seen that all usage of _domainTable, including all reads, was being done under the lock.  If that's true, then just delete this comment.  If that's not true, and the code actually was relying on being able to have concurrent readers with a single writer, unfortunately it'll probably need to change back to being a Hashtable, or reads will need to be locked, or some other synchronization scheme will need to be put in place.  Apologies if I led you astray here with my initial Dictionary suggestion.


With this version I'm now getting an error when the script runs:

`Unknown options: "/dev/xvda", }-backup-2015-07-30, "Devices":`


TeamCity Bender :: Deploy [Build 3.0.22.0](http://build.mikeobrien.net/viewLog.html?buildId=2458&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3249, ignored: 5 Build time: 00:03:44


fixed


They probably should be though?


Not currently, but this could be moved out to on crates.io. In general this doesn't have the static guarantees one might want (e.g. `T` and `U` are the same size)


```rake db:reset``` would continue to work through each of these migration/schema changes because you kept all of the code valid, and starting from scratch there remained a way to update the schema for each migration.

Glad to hear you're more comfortable using migrations now! I'd very much recommend preferring ```rake db:migrate``` over ```db:reset``` going forward, as you won't uncover migration issues until deployment otherwise.


This shouldn't be necessary, as TryGetValue is documented to set the out value to the default for the type if the key can't be found, so this if block can just be:
```C#
_domainTable.TryGetValue(cookie.DomainKey, out pathList);
```


+1 this change - `LOG_INFO` is the proper level for regular httpd access log lines.

Cf., [*syslog*(3)](http://man7.org/linux/man-pages/man3/syslog.3.html)


Ditto


I'd choose name, unweighted and weightings because everything else is implicitly available.


I recommend: `done: not @cursor.hasNext()`

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Iterators_and_Generators


well you don't want to tell me that only the name is important for the repr of a section?


I remember seeing a whole file of SecureString-related helper, which I'm assuming was SecureStringHelpers.  Is that still needed, or can the file be removed now?


fvvf


origin and from_cli are information one might need. For debugging one also might often need strip_whitespaces and list_delimiters - I'm not sure we really want custom repr's here instead of showing all members.


dat commit


@fasterthanlime I believe Ubuntu (which Travis uses) comes with the Boehm GC libs (just not the headers) by default in `/usr/lib`. Try something like:

```
sudo rm -f /usr/lib/libgc*
sudo rm -f /usr/local/lib/libgc*
```

(I wouldn't be saying this if this weren't a VM... :)


Why `ENDED`? This block is for `touchesCount > 0`, so if you still have touches (fingers on the screen) why would you end it?


`.wrap()` and `.once()` are in `bem-xjst@1.1.0`. PTAL


Well we have prafulls thread on the forum, i did ponder looking at what code it blocks out to see if it was fixable


@hunyeti https://www.youtube.com/watch?v=l58NESfWDmQ


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2211 of 2232 tests passed)

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


Sadly there's not much convention for it. In general I find that the decision is made based on subjective things like "code feel"...

If your function ends up being a series of twenty very similar ```if``` statements in a row, it represents an impediment to understanding, and can potentially hide bugs because each ```if``` statement is entirely open in terms of what code it can contain.

When you move to having a statically specified structure like the ```CATEGORIES``` map you later move to, you're more explicitly specifying the relation between the category name and the creator title, there's less room for misunderstanding and any errant code in there is more likely to be recognized as a bug by either Ruby or a programmer.

That transition from "if logic" to "static data structure" is primarily about "data volume" -- how much data your code actually needs to manage. Moving from there to having additional databases columns/tables is more a decision about "data velocity" -- how often your data changes, and consequently who is responsible for changing it.

Conservatively, the choice to move this data structure into the database should only be made when the project requirements say that the data is going to be changed frequently enough that it would be too slow / too wasteful to have a developer make the change themselves. Instead a UI should be built which allows other team members to make any changes directly.


It's curious that you've initialized this variable to `None`. It seems OK, just odd.


When you say notification, do you mean email?


good


Seems like it's not really fixed: #4177 :)


This is a good commit.


Is it really needed to put "Menu" everywhere? Could we not just have a similar structure as the menu with the same item names?

Also, if a change like this is made which would leave other toolboxes in a bad state, it should be made across all toolboxes. Currently, there are some things in "Raster Menu" and others in "Raster". Thanks.


Spencer is secretly god in disguise.


> no longer the last entry in oauth/clients/cf

What does this have to do with this commit?

> I changed it to this:

```
 -      scope: uaa.none
 +      scope: uaa.none,scim.userids
```

I'm not understanding. This is the same change as this commit. So if the commit didn't work, why did this change fix it?


@jlipps here `{deviceId, emPort}` is more suitable?


Delays seem to be too short for the ADC to read dht values. Will need to increase this to 10ms and test


If you don't want to assume UNIX lowest-first behaviour, just open once, dup2() it onto STDOUT, STDIN and STDERR, then close it if it's not equal to one of them.  Otherwise, just close them, open once and dup twice.


Hm, shouldn't this work for long form canonical App Store identifiers as well? Most links I see to apps are formatted like `https://itunes.apple.com/us/app/tappd-apps-worth-talking-about/id823376869` rather than with the `appsto.re` host.


The newline at the end of this file shouldn't be altered.


Although, I actually prefer what you've done. Nevermind.


Discussion at https://groups.google.com/d/msg/cryptopp-users/BiKubx4K2Us/5LcuqXxMFwAJ.


I was bothered by the fact that the user could type in "no" for the first answer and then be given ingredients for a drink with no liquid in it, i.e. a rasher of bacon and a slice of orange.  This was the only way I could think of to add in some "weak" alcohol, in case the answer was "no", without asking another question.


ref https://github.com/FortAwesome/Font-Awesome/commit/7cde41ea934f43e66dc123ea6411d8535b50a4f7


Lewis,

The kernel submodule SHA1 was not updated to include the new changes.

If you update your tree the ONL linux submodule will now include the
changes for the 5712.

Jeff

On Wed, Jul 29, 2015 at 7:43 PM, Lewis-Kang <notifications@github.com>
wrote:

> @jnealtowns <https://github.com/jnealtowns> It's strange that we got
> latest from ONL github yesterday and "make onl-x86", the kernel config is
> still not including the needed USB config (though we can see the USB config
> is already set via browsing the github). Do you know what may go wrong?
> Does the building process do not get the latest kernel config? Thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworklinux/linux/commit/c004248a9c8d87c4e3a8b8a298b3ba8573769abc#commitcomment-12435304>
> .
>


ref https://github.com/twbs/bootstrap/commit/55336e6a9e93d8861c68d7df67ecea3a33efccbf


This field doesn't exist on dealer registration, so I wrapped this call in an if-statement.

Also, it's generally considered bad form to query the DOM before the document has finished loading, so I moved this into the onready function.  (In this case it didn't matter but sometimes it does.)


Updated exit code handling by changing the case statement. Also looking to implement an offline update for clamav.


Here is example of the fix I am proposing. https://jsfiddle.net/frpeqfmL/


lol first the Code of Conduct, and now this. fuck github, and to think i was gonna actually purchase a plan here. hello bitbucket


Using statement.


250ms per measurements apparently. Will need to find alternative sensors for production version


额, 你确定? 不需要做成可配置的? 貌似会影响前端jquery的代码吧?


Consider implementing this with the following approach -
Create a class Student with required properties.
Loop through objects of that class to create an XML file.


generated


removed this try/catch because runCommand already handles that


Danni: this sounds like a lot of workflow effort


try/finally?


There used to be a bug in the driver where it did not pass along the readPreference. that has since been fixed


Perhaps the one with the bar underneath it?


DAMN, this is so cool.


maybe break this line


@borshop retry


So you're sacrificing performance?


Need to remove the e parameter and echo (the e parameter has been deprecated and is not supported in the newer xl* functions:
``````
alert("<?php echo xls('End date must be later than Begin date!'); ?>");


This should be one constant that is translated:
``````
<title><?php echo xlt('Automated Measure Calculations (AMC) - 2014 Stage I'); ?></title>


This should be one constant that is translated.


This should be one constant that is translated


I think the standard notation is the triple-equals sign


document what parametrs your method expects


But it's `--host` parameter already.


Noted that the related change in interface/reports/report_results.php were not made.


nihao


You are so stupid


@jonsterling I think it's coming nicely :D This theorem isn't quite proven but it's looking very possible. Just needed to switch gears for the night.


As of right now this code lets you at least query the "drive" without a bunch of fail. More fixes to come.

Here's some output from the OS once this starts running:

**
sh# smartctl -ax /dev/sdb -d scsi
smartctl 5.43 2012-06-30 r3573 [x86_64-linux-2.6.32-220.23.1.el6.x86_64] (local build)
Copyright (C) 2002-12 by Bruce Allen, http://smartmontools.sourceforge.net

Vendor:               AMI
Product:              Virtual Floppy
Revision:             1.00
User Capacity:        1,474,560 bytes [1.47 MB]
Logical block size:   512 bytes
Serial number:
Device type:          disk
Local Time is:        Thu Jul 30 03:30:53 2015 UTC
Device does not support SMART

Error Counter logging not supported
Device does not support Self Test logging
Device does not support Background scan results logging
scsiPrintSasPhy Log Sense Failed [unsupported scsi opcode]

sh# fdisk /dev/sdb

WARNING: DOS-compatible mode is deprecated. It's strongly recommended to
         switch off the mode (command 'c') and change display units to
         sectors (command 'u').

Command (m for help): p

Disk /dev/sdb: 1 MB, 1474560 bytes
1 heads, 3 sectors/track, 960 cylinders
Units = cylinders of 3 * 512 = 1536 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes
Disk identifier: 0xc9d0c9d0

This doesn't look like a partition table
Probably you selected the wrong device.

   Device Boot      Start         End      Blocks   Id  System
/dev/sdb1   ?     9628704   212845771   304825601   c4  DRDOS/sec (FAT-16 < 32M)
Partition 1 has different physical/logical beginnings (non-Linux?):
     phys=(1022, 40, 32) logical=(9628703, 0, 2)
Partition 1 has different physical/logical endings:
     phys=(964, 8, 33) logical=(212845770, 0, 2)
Partition 1 does not end on cylinder boundary.
/dev/sdb2   ?   394518778  1470859346  1614510853+  30  Unknown
Partition 2 has different physical/logical beginnings (non-Linux?):
     phys=(262, 19, 51) logical=(394518777, 0, 1)
Partition 2 has different physical/logical endings:
     phys=(787, 228, 13) logical=(1470859345, 0, 3)
Partition 2 does not end on cylinder boundary.
/dev/sdb3   ?   441817288   836407705   591885626+  46  Unknown
Partition 3 has different physical/logical beginnings (non-Linux?):
     phys=(899, 70, 6) logical=(441817287, 0, 2)
Partition 3 has different physical/logical endings:
     phys=(131, 200, 1) logical=(836407704, 0, 3)
Partition 3 does not end on cylinder boundary.
/dev/sdb4   ?   464871450   464878562       10668+  45  Unknown
Partition 4 has different physical/logical beginnings (non-Linux?):
     phys=(843, 94, 3) logical=(464871449, 0, 2)
Partition 4 has different physical/logical endings:
     phys=(325, 82, 14) logical=(464878561, 0, 2)
Partition 4 does not end on cylinder boundary.

Command (m for help):
**

Getting closer...


Happy to. Give me a few days. :)


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (743 of 743 tests passed)

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


This is a good example of how you have done xla() and xlt() correctly. Note that xla() is used within html attributes (hence the a) and that xlt() is done when not within html attributes.


To not break prior use of the amc_2014 rules and reports, should also allow the amc_2014 token here and in all the other above places where this is done.


To not break prior use of the amc_2014 rules and reports, should also allow the amc_2014 token here and in all the other places below where this is done.


I don't think these are created equal. Maybe separate these cases (JoinedUser does not exist, root_user does not exist)


hi，idlesign，I sent you an e-mail, please check。
I want translate you project(pythonz)  into Chinese.and i need you help~:)


Looks like I should save this to a SNIFFING_INTERFACES variable.  Without this, it starts ossec_agent, but doesn't start any IDS, FPC, or other processes that listen on the wire..


This seems really odd. Why not just use encounter and then place this in the rule denom itself (since this is only used in one rule). Also, for this rule, why is only Office Visit accepted?


What sends the "message"?


I would advise two things:
1) take out the variable params in get_all_sets, it's confusing and I don't think get_all_sets should be multipurpose
2) write a function (and put it in an appropriate util) that takes two set lists and combines and sorts them and use it here.


@ppbustamante en este commit eliminaste la referencia a org-eclipse-jgit.jar del classpath. Aunque el código compila con el build.xml eclipse no encuentra la referencia (por no estar en el classpath). Yo lo acabo de solucionar, por favor haz pull para que tomes el fix.


Ariel Update


separate this
if (cond)
elif (cond2)
else


Here's an example of what the header should look like. All the new scripts will need to have an acceptable header:
``````
+/**
+ *
+ * AMC 302f 2 STAGE1 Denominator
+ *
+ * Copyright (C) 2015 Ensoftek, Inc
+ *
+ * LICENSE: This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://opensource.org/licenses/gpl-license.php>;.
+ *
+ * @package OpenEMR
+ * @author  Ensoftek
+ * @link    http://www.open-emr.org
+ */


nice


These three commits look good to me.  Consider them reviewed and push to master given that the tests pass.


For example, can't you just use what you have in library/classes/rulesets/Amc/reports/AMC_302m_STG2/Denominator.php


If this is a script, you should:

1) write a function recalculate_trueskill()
2) call that function in main. see parse_challonge


I found him, but I'd rather somebody more socially "ept", like you, contact him!
He's on LinkedIn, https://www.linkedin.com/in/johnlgustafson


also allow amc_2014


check_trueskill is not the correct function name. maybe reset_trueskill?


all sets


@mcostalba: why don't you open a pull request? this patch should be merged, it's a no brainer.


do you not have a  backref for winner/loser in set?


Combine the two constants:
xlt('AMC-2014 Stage 1')


Combine the two constants


warning message does not seem correct.


why not have a BETA and TAU as well if these are constants?


still think the name could be improved. reset_trueskill?


This file is auto-generated by `Pandoc.brew`-ing https://github.com/Rapporter/pander/blob/master/inst/README.brew


@latobarita: retry


saw approval from matschaffer
at https://github.com/graydon/stellar-core/commit/4c21eb2f073306c425c5b5f2c7da8baf90663b26


merging graydon/stellar-core/switch-testnet-keys = 4c21eb2f into auto


graydon/stellar-core/switch-testnet-keys = 4c21eb2f merged ok, testing candidate = 3223baec


@ryanb elixircasts next?


Typo?  "the h"


does it use the local branch at all? I use these branches regularly with
git, because it lets me namespace my local work.

I see that tests are failing, and now I see the same failure on the remote
that i saw locally.

On Wed, Jul 29, 2015 at 7:08 PM, Mark Waite <notifications@github.com>
wrote:

> You might try naming your local branch something like
> "master-jk-submodules" so that there is no "/" in the branch name. The
> tests may have some form of strange dependency that the local branch must
> not include a "/".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435023>
> .
>


This has nothing to do with regression in the statistical sense


Do NOT do the above UPDATE commands. Instead need to modify the original entries.


Review is complete.


That comment was in regards to the disassembler, not the decompiler. I'm still using my compare_shader_bins.py to repair missing precision in disassembled shaders (which is working better than expected, but is not perfect), but now that Flugan has given us a disassembler function we should switch to using it instead of Microsoft's so that all freshly disassembled shaders will have full precision.


So which category would you suggest?


Out of curiosity, what is the OpSim catalog?  I didn't think you were doing anything at all with OpSim


@Faiz7412 lol


mmm coffee!


When you go through the next revision, please respond to each github comment (whether complete or why not or answer the question). Then can avoid reviewing the same issues over and over again.


I'd rather it wasn't here at all to be honest, it was just some ideas to share with friends at this stage.


Also on this commit (omitted from the commit message): Operator-specific sensor max and min values were added to the SensorConfig class. These values can be acquired from the TelebotSensorConfiguration program (this will be incorporated into the TelebotMasterArmTCP component later).


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


Sure, fixed.


ReadVarsSmallDiffTest (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1776 of 1777 tests passed)

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


Thanks!


It will be nice if empty (0) collections are skipped, for example, when:

     picker.mediaTypes = @[@(PHAssetMediaTypeVideo)];

![empty](https://cloud.githubusercontent.com/assets/6657/8975608/b1705440-36b3-11e5-8a06-11a1fdf158d4.jpg)


We are not supposed to put spaces around *, /, and %: https://confluence.lsstcorp.org/pages/viewpage.action?pageId=16908737#C++LayoutandComments-WhiteSpace


Curious how this one works exactly, is this for whether players are ignoring another player or if they have messages globally disabled?


fast-forwarding master to auto = 3223baec


Same as above.  I'm all for consistency.  I think it should probably be changed to adhere to one convention or the other.


@Lukas-W, reverted via https://github.com/tresf/lmms/commit/95f0b29bd0b311007f9d2360b27819be3413f6bd


This outputst Unrefracted_RA a second time, and worse it has hardcoded values.


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/223c7ba8062badc06160cb83343bcf5bb0cf09a4


merging basho/riak_cs/bugfix/copy-object-bugs = 223c7ba8 into borshop-integration-1200-bugfix/copy-object-bugs


basho/riak_cs/bugfix/copy-object-bugs = 223c7ba8 merged ok, testing candidate = 8eaf49f2


commented code should be made to work or removed.


is node ever nil?


lib, see tsconfig.json


yaml loader會轉成json格式然後還要用json loader才可以讀到檔案的樣子OAO


It seems like a more natural argument would be the density of stars, not the number.


@khan0405 마지막 @jar이 없어도 기본적으로 jar인것 같은데 aar만 쓰는걸로 봤는데


@mmikeww Somebody remarked earlier that BitBucket is apparently no better: https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12425361

I guess we're now all collectively realizing what a mistake it was to centralize a decentralized version control system (Git). This is what happens when you trust private corporations with your critical infrastructure.


The test is also failing for me.  The failure is not related to local
branch name, since I pulled your changes into a branch name using a common
pattern (for me).  If I discard the last of the three changes, then the
compile and test succeeds for me.

Mark Waite

On Wed, Jul 29, 2015 at 10:00 PM Jacob Keller <notifications@github.com>
wrote:

> does it use the local branch at all? I use these branches regularly with
> git, because it lets me namespace my local work.
>
> I see that tests are failing, and now I see the same failure on the remote
> that i saw locally.
>
> On Wed, Jul 29, 2015 at 7:08 PM, Mark Waite <notifications@github.com>
> wrote:
>
> > You might try naming your local branch something like
> > "master-jk-submodules" so that there is no "/" in the branch name. The
> > tests may have some form of strange dependency that the local branch must
> > not include a "/".
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435023
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435875>
> .
>


@Pluu aar과 함께 배포된 경우엔 디폴트로 aar로 불려져 옵니다. 이 경우 @jar를 명시해야 가져오더라구요:)


@khan0405 그렇군요. 하나 배우고 갑니다.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1424


sweet.


Since the ulClass has changed from '=' to '@', it cannot support passing object to ngClass any more. I have explained the usage in  #40. Shall we change it back to '='?


:+1:
Why not push a new release? It do solve the problem :)


It's probably because the tests depend on each other, which is really bad.
Removing the one test breaks the others because it has inter-dependent
portions.

On Wed, Jul 29, 2015 at 9:19 PM, Mark Waite <notifications@github.com>
wrote:

> The test is also failing for me. The failure is not related to local
> branch name, since I pulled your changes into a branch name using a common
> pattern (for me). If I discard the last of the three changes, then the
> compile and test succeeds for me.
>
> Mark Waite
>
> On Wed, Jul 29, 2015 at 10:00 PM Jacob Keller <notifications@github.com>
> wrote:
>
> > does it use the local branch at all? I use these branches regularly with
> > git, because it lets me namespace my local work.
> >
> > I see that tests are failing, and now I see the same failure on the
> remote
> > that i saw locally.
> >
> > On Wed, Jul 29, 2015 at 7:08 PM, Mark Waite <notifications@github.com>
> > wrote:
> >
> > > You might try naming your local branch something like
> > > "master-jk-submodules" so that there is no "/" in the branch name. The
> > > tests may have some form of strange dependency that the local branch
> must
> > > not include a "/".
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435023
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435875
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jacob-keller/git-client-plugin/commit/272b1216bad65a4eccefab5d673aeb249a4bb17c#commitcomment-12435990>
> .
>


this is a blank


line note


A pod name of all numbers is pretty user-hostile. I don't think we want to set that example. Please use a descriptive name of the form that generateName would produce, which would be appropriate for a one-off busybox pod.


In this case I would add `class="post"` to each post `<div>`. So you would have, e.g.,
```
<div id="post1" class="post">
...
</div>
```
Then, in your styles you can just do this:
```
.posts .post {
  width: 515px;
  ...
}
```
To apply the styles to all elements with `class="post"` under elements with `class="posts"`.

Even more description HTML structure might be:
```
<div id="posts-list">
  <div id="post1" class="post">
  ...
  </div>
  <div id="post2" class="post">
  ...
  </div>
</div>
```
Then styles:
```
#posts-list .post {
  width: 515px;
  ...
}
```


At a glance, this looks fine, but please be extra careful to ensure that it's consistent with the other container-centric commands, esp. exec.


The new command looks reasonable to me. I need someone else to do the code review.


@EugeneButusov, it's better if you put the logic to the method with the maximum number of params, and delegate in others, providing default values.


fast-forwarding develop to borshop-integration-1200-bugfix/copy-object-bugs = 8eaf49f2


a comment


I like it too. Thanks a lot guys!

Can we add to the commit message the api level? Otherwise, for example
these two commits look identical from the title but api levels are
different:

https://github.com/opengapps/arm/commit/b8f9df7778896e3310a1100fdca1f73a4166f333
https://github.com/opengapps/arm/commit/74f1923d406a112f11a57fee714549760bb59902

On Wed, Jul 29, 2015 at 3:39 PM, mfonville <notifications@github.com> wrote:

> I am happy that it works well :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/all/commit/3a812beab1f83986f361ef0cd3e62d2570087902#commitcomment-12430568>
> .
>

--
<http://www.facebook.com/djalik>  <http://twitter.com/#!/DJAlik>
<http://www.linkedin.com/in/alexrusinov>
<http://plus.google.com/114580313768137070684>
<http://www.mixcloud.com/djalik/>
<http://psnprofiles.com/DJAlik>


Do you want to define this method as test_has_gift_message?


Switched order of these two functions (so the one that gets called by the other comes first)


Some of these vars are never used


node was not liking const for whatever reason - maybe it's because I have an older version


console.log('here') is ugly - I'll remove in a bit.


I only changed this because I originally deleted all console.log lines. No particular reason to have this version over yours.


Camel case


Nit: the command doesn't need 2 ways to specify the pod. Imo, since pod is required, it should be an arg.


Comments above


@krainboltgreene maybe you should actually contribute to the discussion than spew nonsense


ESLint is a code checker I use for javascript code. It works with ES6, JSX, and vanilla JS. This is the start of a config file for it.


啦啦啦


yeah I like that


I think this will need rewritten as one triggered ability, as it is on the card. A card like Stifle should be able to counter the whole effect.

Maybe something like:
Ability ability = new BeginningOfUpkeepTriggeredAbility(new GainLifeEffect(new PermanentsYouOwnThatOpponentsControlCount()), TargetController.YOU, false);
ability.addEffect(new DrawCardSourceControllerEffect(new PermanentsYouOwnThatOpponentsControlCount()), TargetController.YOU, false));
this.addAbility(ability);

Thanks, super excited for Zedruu finally!


add 'self.' to be in the trend )


seems throttle_ops.should_wait() returning true ensures max_ops != 0


Perdão pelo erro ** divulgação de vocês no facebook **
Segue o link para Debug do facebook:
https://developers.facebook.com/tools/debug/

Quando você colocar as imagens e sua hospedagem, use o link acima para atualizar o caminho da imagem, entre outras "meta" do seu site...

Forte abraço!


Ah, ok. That's professional bartending!


same above, check of max_bytes not necessary


Note that since the values of `like` are already booleans, you can simplify this to:

```python
if like[key]:
    ....
```

That would be how most Python programmers would write it.


grub-install --target=x86_64-efi --efi-directory=/mnt/linux/boot/efi --bootloader-id=KOTOHANA --recheck --directory=/mnt/linux/usr/lib/grub/x86_64-efi/ --root-directory=/mnt/linux/ /dev/sda5


@EugeneButusov, which of them we need for "oauth2"?
I guess `access_token` will be enough.
Or did you see anybody in "answers" asked for others params as well?


Contribution: Give up and leave.


LGTM!!


ReadVarsSmallDiffTest (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1776 of 1777 tests passed)

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


@EugeneButusov, please, keep changing binaries in the separate commits, and it's better to commit them at the final stage, after passing the code review. They make repositories huge. I hope in the future we'll clean up them from here.


I don't get you. Check this out: https://github.com/Circa75/dropplets/blob/master/dropplets/style/style.css#L146-L155

As per [documentation](https://developer.mozilla.org/en/docs/Web/CSS/@font-face), the `@font-face` rule defines a font face with the given `font-family` name and allows the browser to use one of the listed font files.

The json file is **not a font file**, and also not in that list. I am still of opinion that you should remove this huge `json` file. Please let me know if I'm still missing something.


@SchizoDuckie I can't figure out how to use angular-locale's plurals for days hours etc.
So unless you want me to hold off (because you are going to sort it out), then I am going ahead and use a directive solution (i'll call a function for the day value, and return the translation for day/days in the appropriate language).
Please reply with:
- [ ] Hold Off, SchizoDuckie to the rescue,
 or
- [ ] Go Ahead with your solution Garfield69.
thanks.


There is more net45xx stuff here also.


Should add CONFIG_EXTCON=y to arch/arm/configs/sunxi_defconfig otherwise CONFIG_PHY_SUN4I_USB doesn't get enabled properly.


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 9](https://build.fubar-dev.com/viewLog.html?buildId=70&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:11


sorry. I will definitely do that in the PR.
You definitely should send a PR to https://github.com/mdo/github-wide - it is working after all.


This seems complicated. Would modifying the Google Calendar RSS parsing code to provide a Date object instead of a string help?


test


Err, why?


Here's one idea we might use:

    #ifdef NDEBUG
        #define gc()
        #define GC()
    #else
        struct Might_GC { char unused; };

        #define gc_ struct Might_GC might_gc_, // <-- notice comma
        #define GC_ might_gc_, // <-- notice comma

        extern const struct Might_GC gc_ok_;
        #define GC_OK_ gc_ok_, // <-- notice comma
    #endif

Then make the signal dispatcher that runs recycle take a `struct Might_GC` parameter.  Now awareness of garbage collection is something weaving through the call graph, perhaps the prototype to Do_Next could look like:

    REBCNT Do_Next(gc_ REBSER *block, REBCNT index, REBFLG op);

You might call it like:

      Do_Next(GC_ blk, index, op);

This would have more teeth than calling it `Do_Next_Might_GC`, because the names are kept in line, transitively through the call graph.  You'd kick it off at the top level with GC_OK_.  Then if we turn unused parameter warnings on, you'd even get told when you didn't need the `might_gc_` anymore if the usages were removed.

While a little bit ugly, it might pay off...and it's not too disruptive.  Maybe worth a shot to look into.


No need to clear it (same for blacks)


留名。。。


The changes are not proper


修正ありがとうございました。
ただちょっと意図が違いました。
details(); という関数を作って欲しかったです。
なので、使い方としては以下のような想定です。
```java
Dcm dsc = new Dcm();
....
hoge = dsc.result();
fuga = dsc.detail():
```


Guess it's better to ignore @krainboltgreen, their constant whining/denial isn't doing anything.


It looks to me like [`tests/generate.py`](https://github.com/libgit2/libgit2/blob/master/tests/generate.py) is the only python code in the `libgit2` repository, and I just got it to run successfully using python3.


I think this has improved the server.
![vultr 30-7-15](https://cloud.githubusercontent.com/assets/11748913/8976360/787f87e0-36be-11e5-9f81-d5510461f13c.JPG)


seconds seconds  0 256
 1 291 35  2 333 42  3 389 56

133

example of extra render time due to adding in up to 3 weight functions on a
mandelbox in my temporary Mandelbox103 setup. (By comparison using the
Mandelbox tab , render time was  217 seconds, therefore  my  19 "enable
checkBoxes" are adding about 2 seconds each ),

On Thu, Jul 30, 2015 at 12:56 PM, carbuncle grim <mclarekin@gmail.com>
wrote:

> I am currently experimenting with mandelbox, where the  weight function
> appears to be much more expensive than with the mandelbulb ( 4 weight
> functions almost  doubled my render time).  This needs to be investigated
> further and maybe " enable weight checkBox" may be required.  Weight
> function often works well up to 1.3, consider allowing spinbox and/ or
> slider from 0 to  2.0 range. Weight function may need to be applied to  aux
> DE for spherical fold.
>
>
> On Thu, Jul 30, 2015 at 8:09 AM, Sebastian Jennen <
> notifications@github.com> wrote:
>
>> yes, looks like auto generated, see hotkeys under menu > image
>> -> autohotkeying tries to find first occurence of not yet used capital
>> letter in any surrounding.
>> Not really desired, although it could be quite handy if we got ctrl +
>> ??? as hotkeys for common things like save / load / settings / ...
>> You can add a shortcut as a property directly to an action in the
>> designer. This will also write the hotkey next to the entry.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/buddhi1980/mandelbulber2/commit/c0f01842c1044417f63e5ab549a0bbb39fffe77f#commitcomment-12429963>
>> .
>>
>
>


TABS!!!!!!!!


@krainboltgreen offends me.  Luckily, github has a mechanism for dealing with anything anyone at all finds even slightly offensive.  I suggest people use it.


demo comment


Nit: capitals and punctuation.


LGTM


We can delete this now.


@dchan272, Can you make this function call Charts.viewEVanalysisChart similar to what is on the forwardBackward branch?

It will be easier to test that way.

https://github.com/a-team-comp354/project-manager-app/blob/forwardBackward/UI/ateamcomp354/projectmanagerapp/ui/util/Charts.java#L18


We should not use sandbox url here since it is possible there could be more than one sandbox instances. Currently we are using sandbox1. When sandbox1 gets filled we might use sandbox2. It is best to get instance url from instanceUrl property using the function records.getDataresource().


hello


When `payload` is blank you receive

```shell
NoMethodError:
       undefined method `each' for "":String
```


更新ありがとうございます。
"|" の両サイドに半角スペース入れてもらえますか？


@hrjaco This was only a problem in the code that unset the read callback and had outstanding writes that errored out. Normally, a writeErr() callback would be followed by a readEOF() which would also close the connection, unless the read callback happened to be uninstalled at the time.

This could in theory happen with mcrouter if the server hit the limit of outstanding unreplied requests (maxInFlight) so that it would pause the connection and happened to receive a writeErr at that time, but we never observed this issue in production. This bugfix was necessary because of another project that shares the same server code internally - that project calls pause() on the connection manually and much more often than mcrouter does.


AS已炸，我可是选了用ta b缩进

发自我的 iPhone

> 在 2015年7月30日，13:28，Peter Cai <notifications@github.com> 写道：
>
> TABS!!!!!!!!
>
> —
> Reply to this email directly or view it on GitHub.
>


@krainboltgreene
Take your own advice, troll


We should remove pointers and use structs to return information. I have created #159 to track it.


აქ მე მგონი სწორად იყო ყველაფერი. არ უნდოდა ამას გადაკეთება.


Perfect thank you!!


我不小心把这个删了


For DMTCP 2.5.0, let's remove this and other such deprecated APIs that have been in the same state for a while now. I created #160 to track it.


I'm unsure whether we want to admit that an attachment exists but not under the given attachment id.


@johnsimons I've made this change to use a bit more of the available real estate on the diagram nodes. It looks like this now.

![node-saga](https://cloud.githubusercontent.com/assets/132117/8976555/0e866e26-36d2-11e5-8aba-398b92a7d1a9.png)


@einatr  i see this and not _this here - please recheck the code...


Sorry Patrik, I had forgotten adding NatPredicates.lidr. It is now in
the repository. Best, Nicola

PS: I still think that |Rational|, |NonNegRational|, etc. should be
ideally expressed via type classes. But, after the last attempt at using
type classes to specify |ContainerMonad| failed, I am trying to stay
away from them.


to be removed once #30 is resolved


really closes #8


Can you tell me how to report the issue that I linked above?


Hello, I'm not sure how else to contact you, so I'm leaving this here on the latest commit...

I recently pulled and installed LTPV to help me do some analysis on some multi-gpu software I'm writing. Here is the result of the following sequence of commands, ran from the root directory of the LTPV repo (I put it in ~/LTPV):

$ ./waf configure
Setting top to                           : /home/tiger/LTPV
Setting out to                           : /home/tiger/LTPV/build
Checking for 'gcc' (C compiler)          : /usr/bin/gcc
Checking for 'g++' (C++ compiler)        : /usr/bin/g++
Checking for header dlfcn.h              : yes
('Mode: ', 'debug')
('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
'configure' finished successfully (0.128s)

$ ./waf
Waf: Entering directory `/home/tiger/LTPV/build'
('Mode: ', 'debug')
('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
[3/4] cxx: src/opencl-hook.cc -> build/src/opencl-hook.cc.1.o
[4/4] cxxshlib: build/src/common.cc.1.o build/src/init.cc.1.o build/src/opencl-hook.cc.1.o -> build/libltpv.so
Waf: Leaving directory `/home/tiger/LTPV/build'
'build' finished successfully (0.816s)

$ sudo ./waf install
[sudo] password for xxxxxx:
Waf: Entering directory `/home/xxxxxx/LTPV/build'
('Mode: ', 'debug')
('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
+ install /usr/local/bin/ltpv (from bin/ltpv)
+ install /usr/local/include/ltpv.h (from src/ltpv.h)
+ install /usr/local/lib/libltpv.so (from build/libltpv.so)
+ install /usr/local/share/ltpv/inc/favicon.png (from share/ltpv/inc/favicon.png)
+ install /usr/local/share/ltpv/inc/functions.js (from share/ltpv/inc/functions.js)
+ install /usr/local/share/ltpv/inc/getQueryString.js (from share/ltpv/inc/getQueryString.js)
+ install /usr/local/share/ltpv/inc/jquery.min.js (from share/ltpv/inc/jquery.min.js)
+ install /usr/local/share/ltpv/inc/jquery.xml2json.js (from share/ltpv/inc/jquery.xml2json.js)
+ install /usr/local/share/ltpv/inc/main.js (from share/ltpv/inc/main.js)
+ install /usr/local/share/ltpv/inc/minus.png (from share/ltpv/inc/minus.png)
+ install /usr/local/share/ltpv/inc/minus2.png (from share/ltpv/inc/minus2.png)
+ install /usr/local/share/ltpv/inc/plus.png (from share/ltpv/inc/plus.png)
+ install /usr/local/share/ltpv/inc/plus2.png (from share/ltpv/inc/plus2.png)
+ install /usr/local/share/ltpv/inc/profiling_model.xml (from share/ltpv/inc/profiling_model.xml)
+ install /usr/local/share/ltpv/inc/shortcut.js (from share/ltpv/inc/shortcut.js)
+ install /usr/local/share/ltpv/inc/sorttable.js (from share/ltpv/inc/sorttable.js)
+ install /usr/local/share/ltpv/inc/style.css (from share/ltpv/inc/style.css)
+ install /usr/local/share/ltpv/index.html (from share/ltpv/index.html)
Waf: Leaving directory `/home/xxxxxx/LTPV/build'
'install' finished successfully (0.181s)

Great. Now when I run my program with $lptv "./program", it segfaults somewhere in this function.

void OclEnv::NewCLCommandQueues()
{
  this->ocl_device_queues.clear();

  for (uint32_t k = 0; k < this->ocl_devices.size(); k++)
  {
    std::cout<<"Create CommQueue, Device: "<<k<<"\n";

    this->ocl_device_queues.push_back(
      cl::CommandQueue(this->ocl_context, this->ocl_devices[k]));
  }
}

I guessed correctly that it crashes when it tries to use your overloaded "clCreateCommandQueue" because the normal output looks like

...
Create CommQueue, Device: 0
Create CommQueue, Device: 1
...

and when running w/ ltpv I see

...
Create CommQueue, Device: 0
Segmentation fault
...

Here's the gdb dump:

Create CommQueue, Device: 0

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7bb9f2c in std::_Rb_tree<int, std::pair<int const, ltpv_t_stream*>, std::_Select1st<std::pair<int const, ltpv_t_stream*> >, std::less<int>, std::allocator<std::pair<int const, ltpv_t_stream*> > >::_M_begin (this=0x20)
    at /usr/include/c++/4.8/bits/stl_tree.h:502
502          { return static_cast<_Link_type>(this->_M_impl._M_header._M_parent); }
(gdb) bt
#0  0x00007ffff7bb9f2c in std::_Rb_tree<int, std::pair<int const, ltpv_t_stream*>, std::_Select1st<std::pair<int const, ltpv_t_stream*> >, std::less<int>, std::allocator<std::pair<int const, ltpv_t_stream*> > >::_M_begin (this=0x20)
    at /usr/include/c++/4.8/bits/stl_tree.h:502
#1  0x00007ffff7bbc382 in std::_Rb_tree<int, std::pair<int const, ltpv_t_stream*>, std::_Select1st<std::pair<int const, ltpv_t_stream*> >, std::less<int>, std::allocator<std::pair<int const, ltpv_t_stream*> > >::lower_bound (
    this=0x20, __k=@0x7fffffffd980: 15337424) at /usr/include/c++/4.8/bits/stl_tree.h:879
#2  0x00007ffff7bba2dd in std::map<int, ltpv_t_stream*, std::less<int>, std::allocator<std::pair<int const, ltpv_t_stream*> > >::lower_bound (this=0x20, __x=@0x7fffffffd980: 15337424) at /usr/include/c++/4.8/bits/stl_map.h:864
#3  0x00007ffff7bb8d22 in std::map<int, ltpv_t_stream*, std::less<int>, std::allocator<std::pair<int const, ltpv_t_stream*> > >::operator[](int&&) (this=0x20, __k=<unknown type in /usr/local/lib/libltpv.so, CU 0x0, DIE 0x1aece>)
    at /usr/include/c++/4.8/bits/stl_map.h:481
#4  0x00007ffff7bb6c1d in ltpv_addStream (idStream=15337424, idDevice=6601488, name=0x7fffffffd9f0 "Queue 1")
    at ../src/common.cc:121
#5  0x00007ffff7bc1392 in clCreateCommandQueue (context=0x64bcb0, device=0x64bb10, properties=0,
    errcode_ret=0x7fffffffda6c) at ../src/opencl-hook.cc:342
#6  0x00000000004076b1 in cl::CommandQueue::CommandQueue(cl::Context const&, cl::Device const&, unsigned long, int*)
    ()
#7  0x000000000040511d in OclEnv::NewCLCommandQueues() ()
#8  0x00000000004021e2 in main ()

So it fails in your code starting at opencl-hook.cc:342 which is:

        ltpv_addStream((size_t) queue, idDevice, queueName);

then it goes into ltpv_addStream until line 121 which is:

    ltpv_devices[idDevice]->streams[idStream] = newStream;

and after that point it fails in stl stuff.  Please help! I want to use your code!


if we're seek to the live point we'll see that we're on the live edge or not? i think we should set the live edge by the time and not by click the live button what do you think?


thanks, got that and one other


I've never built used protobuf for anything else but linux so not sure why OSX fails.

Concerning using a cmake buildsystem you can try two different approaches:
* simply use `add_subdirectory(cmake)` to call the `CMakeLists.txt` from protobuf
* use a different call to `externalproject_add()` but you shouldn't have to provide any configure/build/install commands as `externalproject_add()` will automatically default to looking for and using a cmake buildsystem if nothing else is specified. You just need to make sure that it can find the proper `CMakeLists.txt` (because it isn't in the root folder and we copy your custom `CMakeLists.txt` there).

@notspiff can probably provide you with a working answer instead of (my) guesses.

BTW are you sure that the patching doesn't work because we use it in multiple audiodecoder addons (see e.g. https://github.com/notspiff/audiodecoder.sidplay/tree/master/depends/windows/sidplay2) just fine.


I test and found a small issue is that the video only plays once.
That If I click on another video or click on the same video again, it does not play.


Fixes #2


I'm not internet famous anymore! :(


ruh oh typo


Closes #2


I think OPTIONS should also be included. JS UI running under different hostname will send CORS preflight requests, which are OPTIONS. Also, should this list be moved to a top-level constant? It probably does not make sense to be in settings.


how satisfying, there are enough tears here to salt my bucket of popcorn


单位是jiffies,不会小于微秒(根据配置,一般是1-10ms)
53位的数据,即使按照微秒算的话
2^53/1000000/86400/365 ~~  284年


I had thought that my code had broken react-titanium, but I now know that my code did not break it. I think this change does break it.  I'll more rigorously test this tomorrow.


Line 62-86: CUI. :smile:


@vedi, thanks for the notice, I missed it.


TODO: Test this!


Fixes #7


@snoyberg: Looks like this is causing the [stackErr ["build"]](https://github.com/commercialhaskell/stack/blob/20f6386ec5086d610429d5e8b79d17345aa304ed/test/integration/tests/sanity/Main.hs#L11) integration test to fail (since `stack build` no longer fails outside a project).  But there's a good chance that we'll turn it back into an error in fixing https://github.com/commercialhaskell/stack/issues/687, so not sure if it's worth adjusting the integration test at this point.


LGTM


//Class


This should not be done here because it's not a "gulp way". `gulp-phpcs` uses custom reporters for output results so you can create one for files.


copy


cool


Should I fix title handling in src/skins-qt/main.cc too?
I couldn't find any trace of this code execution.


"WinAPI Error"


i wouldn´t use id ... i think name is better


@vedi, I didn't found any requests about another extra fields, so you're right, thanks.
I just did it with a margin :-)


forget (: my fault


@ariccio, hypherion2 made it at https://github.com/notepad-plus-plus/notepad-plus-plus/pull/585


@bagder can you clarify, is it acceptable for options that only have a single arg value to pass an `int` instead of a `long`? I know it can be a problem for options that take multiple arg values. As I said I can see in the code that the flags used as values for some options that specify long values are basically ints (ie 1<<1) which would imply that it's acceptable.

My concern is the user may not pass the correct type. Although you could argue user error here I'd say it's generally assumed that when you pass a value 1 to functions that take a long that value becomes a long instead of staying as an integer as is the case here due to va args. For example take [CURLOPT_MAXREDIRS](http://curl.haxx.se/libcurl/c/CURLOPT_MAXREDIRS.html) which says in its description:

> Set it to -1 for an infinite number of redirects.

I think without some sort of epxlicit notice in that option stating that the type *must* be `long` that you'd get users that do `curl_easy_setopt(curl, CURLOPT_MAXREDIRS, -1);` which is incorrect.

I experimented with explicitly tacking on the `long` literal-suffix `L` to all the numbers used to specify value in the options that take `long`. You can see that at https://github.com/jay/curl/commit/ca7c8da. The more I look at it the less I like it.

I wonder though, assuming the type must be `long` and not `int` or badness will happen, that maybe something different should be done, like for each option have a new section `IMPORTANT` and say something like

> It is important that you specify the data types exactly as specified. If the data type is long, you must not pass an int. For more detail please read the overview of xyz.


Scroll Rect はコンポーネント名（クラス名）なので そのまま大文字でお願いします！ #144


Shouldn't it be "origin_(origin)" instead of "origin_(origin_)"? I think this might have been a typo, but since I don't have a computer I don't want to just go and fix this without testing.


ანუ kmlში რაიონია?


That seems to require c9/smf.core to be on github, and accessible to sdk users?

Just read the code so far, didn't try to update my local sdk install yet.


რეგიონი ყოფილა...

On Thu, Jul 30, 2015 at 10:06 AM, Nick Shvelidze <notifications@github.com>
wrote:

> ანუ kmlში რაიონია?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kakheti/pathfinder/commit/0d501ffe4fbbe37bc5251a5c1a17de046ab27fd0#commitcomment-12436728>
> .
>


ok, accecpt


Students/Ghiurau A. Alexandra Cristina/Assignment 2.1/src/mathematics/Functions.java

this will never fix an IOB exception


Students/Ghiurau A. Alexandra Cristina/Assignment 2.2/src/MainClass.java

Why `instance`? instance of what? why not something like `matrixOperations` or `operations`. Everything is an instance!


what is this :))


Students/Ghiurau A. Alexandra Cristina/Assignment 2.2/src/MatrixOperations.java

well done


This property marked as readonly and doesn't have mutator method, so we can change this value only using ivar.
Please, let me know if we should remove readonly modifier.


fair enough, although you do not need a tread-safe implementation


this should be `private` !!


why static


ქ/ს მუნიციპალიტეტი-ა დავიბენი მოკლედ


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 228](https://teamcity.labkey.org/viewLog.html?buildId=263567&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


Heard there's a party kickin' in here?


@EugeneButusov, how does it affect this? (Sorry, maybe I'm not so good in ObjC.)


Have you noticed that this is the 666th commit? You are replacing unicorns with the devil! :D


set config.id = 'habitatInner'. This can be done when object is created in show.gsp.


Seems like this could all be one if:

```js
if (subscription && !subscription.isUnsubscribed) {
  subscriber.next(this.value);
}
return subscription;
```

If `subscription` is "falsey" it's presumably a `null` or `undefined`. I doubt it will make a difference.


void SetGameWorldViewer(GameWorldViewer* viewer)
vs
void SetViewer(GameWorldViewer* gwv){ this->gwv = gwv; }


did you forget to uncomment this?


I think this needs clarification somewhere, I attempted to set this up this evening and ended up putting it aside.

The log message makes it appear as if there are two separate modules that need to be installed mongo-logs and ui-mongo-logs. mongo-logs is found in github as mod-mongo-logs, as expected, however the ui module does not appear to exist.

Further examination seems to indicate that the mongo logs module presents multiple facets, one to the broker to store the logs, and a second to the webui to display the logs, However it isn't clear how to configure both as it would appear that you could just configure it as 'mongo-logs' and the reference it from both the broker and the webui, however the message here appears to indicate a structure more similar to that of the graphite modules, one instance to write to graphite, and a second separate one to retrieve the graphs.

Honestly, Though I think this log message may be misleading, I believethis is more a documentation problem, not certain if the documentation should be in the webui, the mongo-logs module, or both.

or I'm entirely wrong, and I just haven't found the second module :)


Got it, closing this question. Thanks.


سلام
خسته نباشید
گوشی من اچ تی سی دیزار اچ دی هست و مشکل فاصله حروف داره برای اس ام اس میتونم از میراسل استفاده کنم اما بقیه برنامه ها مثل واتساپ نمیتونم .لطفا راهنمایی کنید


ფიდერშიც მუნიციპალიტეტია


I have nothing against `'@@iterator'` – even if it is non-standard, it was around for a year or so – so some libraries/polyfills may have adopted it.

But at any case `Symbol.iterator` should be the default. So please flip the `if` and `else if`.


can you pls change httpbin with mockbin to keep consistency across kong? thx.


Hi, 这里依然有一个BUG。今天在PHP Version 5.5.24 我 改成always_populate_raw_post_data=On之后依然在接口验证收到异常。

**具体原因:**
1. 接口验证使用的是GET 请求。所以$xmlInput 必然为空
2. 接口验证的时候$_REQUEST['signature'] 刚好非空。所以就抛异常了。
3. 建议修改xmlInput 获取的代码逻辑如下``` $xmlInput = file_get_contents('php://input');
        if (empty($xmlInput) && !empty($GLOBALS['HTTP_RAW_POST_DATA'])) {
            $xmlInput = $GLOBALS['HTTP_RAW_POST_DATA'];
        } ```
4. 异常的抛出应该首先判断是否是GET 请求，然后签名，然后post的表单数据。

看看有没有什么问题？


this should probably be `let index = 0` and `index++ < len`. Nitpicky, I know, but this seems more consistent to what people find in other libraries.


I’d mention http://www.ecma-international.org/ecma-262/6.0/#sec-tolength in a comment. This puzzled me :)


i cannot download it on my system


i cannot download data on my system.


only this method is static, I wanted to see the difference, I can chance it.


了解しましたー、以後、コンポーネントは 大文字とします！


I know, I can use the 'classical' implementation.


Weeell, I can do this in main, I just create two identical matrices with this


this will [perform much better](http://dvolvr.davidwaterston.com/2013/06/09/restating-the-obvious-the-fastest-way-to-truncate-an-array-in-javascript/) here:

```js
if(events.length > size) {
  events.length = size;
}
```


I know, that's why in my divide method I get an IOB exception, buut, I'm pretty sure I said this isnt to be checked :)) However, I am not sure how to solve this problem, as I don't have a constructor that gives the polynomial with the degree and coefficients (as I don't have a degree-field)


@rathanbpoojary remove id from ehre.


There must be a better name than `window` here? I found it confusing below like "Why is he comparing this number to `window`?".


change made


Can you please bring it along plot profile button  and a button instead of a link.


我真不知道发生了 什么，这些文件我连打开都没有打开(我是在用记事本写代码)


ここも Mask でお願いします #149


That’s a good idea – I now understand what you meant in <https://github.com/studio-b12/array-from/pull/3#issuecomment-125663542>.

I’d rather name the file `./polyfill` however. While `require('array-from/arrayFrom')` looks weird, `require('array-from/polyfill')` is quite explicit – even if the words “polyfill,” “shim,” etc. [mean different things](https://github.com/ljharb/array-includes/pull/1#issuecomment-123411923) [to different developers](http://toddmotto.com/polyfills-suck-use-a-featurefill-instead/).


Thanks @chriscool :D


@blesh that would slice events off the end of the buffer, not the front.


No difference. Yes


ここも Slider です #151

私の方で書き換えちゃいますー


我已经强制回滚了还是这样。。没救


update_energy is call once per player every 5 seconds, if there are lots of player, it would be saving lots of time.
not only that, it would only save if the player stamina is less than 0.65, which imo is bad.

the saving should place under the for loop in the global step so that it would only save once for all the player per update.


I haven't tried tweaking the numbers for this force layout, but the collection of nodes and links for the layout looks good to me.

Just for the record, if tweaking these numbers starts to become tedious, then I think it might be worth it to ask Max, Ian M, or the social studies PAs for help. Usually, I think getting non-engineers involved in engineering tasks is a bad idea, but I in this case, it could be productive. The best way to do this might be to make a temporary controller which allows tweaking of each force layout parameter.


HA! good catch. It's late.

Still, `splice` should be a little better than the `while` loop:

```
if(events.length > size) {
  events.splice(0, events.length - size);
}
```


are you sure? my MatrixOperations instance is private..


Whant's this file?  Do we have modification on this?


IIRC from C, ++ prefix is 1 less asm op, so it's been my habit for years. not married to it.


:+1: I’m also that kind of perfectionist :)


@Pike thanks for noticing. removed smf.core as it is unnecesary in sdk https://github.com/c9/core/blob/master/package.json


window is what it's called in Rx


I would name this one ExplorationContextWithScore, and the other one ExplorationContext


Too many not related changes for our binding I think.


ხო, არეული ყოფილა


could this key be an enum, i.e. can we already define which attributes defined an algorithm?


Are you sure this will not cast `Symbol.iterator` to a string? I’ve never used it directly, to be honest.


@iflamed


maybe keep suffix of 'Definition' for all things describing stuff


How about adding a spec for an object supporting `Symbol.iterator` natively – such as [`Map`](http://devdocs.io/javascript/global_objects/map)?


This shouldn't occur normally, but ends up causing issues when we run Mocha.


这个 commit message 。。。


How about separate into few patches for each feature?


this looks like consistency1. Maybe it should just be renamed consistency_level. I'm ok to merge it this way too


1: M111 unrecognized component name: legacy


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
31: D101 Missing docstring in public class
89: D101 Missing docstring in public class
104: D101 Missing docstring in public class


haha! I can delete it if you want. Or feel free to do it yourself...


`windowSize`?   `bikeShedWindow`? My bike shed needs a window, Paul.

But seriously, you understand my confusion in the lines below right?


Thanks for keeping me sharp.


Meh, it's probably fine.


这个文档的地址还是 bigquery


Hi,
"net_event_xxx" aren't used in cfg file.
replacing 'net_event_xxx'  with 'event_xxx' is good point.

but you should attach the reason in commit information,
which is useful for review.


this is a really bad idea, there are some other implementations than requirejs, which provides a module.exports in the browser. Instead you should check if XMLHttpRequest exists already.


[Uploading 注意.txt…]()


I think this change has broken the unit tests for `test_release` as this method is now not implemented on the `StubReleaseVCS` class.

I have lost push access to github.com at the moment (firewall) so can't make a pull request.  However I resolved this locally by adding the following to the `StubReleaseVCS` class.:

    def tag_exists(self, tag_name):
        data = self._read_stub()
        if "tags" not in data:
            return False
        return tag_name in data["tags"]

And then updating `test_release.py` so that when [`create_build_process`](https://github.com/nerdvegas/rez/blob/resources2/src/rez/tests/test_release.py#L89) is called `ignore_existing_tag=True` is also passed through.


しょうもない指摘ですが mutipart (multipart) の誤字がパッチコメント中にあります。


DebianDistribution and UbuntuDistribution [inherit](http://bazaar.launchpad.net/~deity/python-apt/debian-sid/view/head:/aptsources/distro.py#L421) from Distribution: `isinstance(distro, aptsources_distro.Distribution)` is sufficient (besides the second parameter of `isinstance` could be a tuple of class).


of course I'm sure. if it's protected than it is accessible to:
- subclasses
- classes in the same package

You want to have the constructor accessible only to this specific class because only here are you actually creating an instance of it!


Instead of implementing pause-resume logic, I think we should make sure that the revision objects inside the changes (RevisionList) do not include JSON body instead (at least that is the current iOS implementation).

There is already the code that loaded the JSON body when it is needed here https://github.com/couchbase/couchbase-lite-java-core/blob/feature/issue_android_629_outofmemory_push_repl/src/main/java/com/couchbase/lite/replicator/PusherInternal.java#L470.


Intel 和 Marvell 都不翻译或者都统一翻译吧。


Travis runs on 5.5.9 as minimum for 5.5, so we can't test earlier versions.

Also, Symfony 3.0 also has 5.5.9 as minimum, so when Laravel upgrades to Symfony 3, that shouldn't be a problem.


@overtrue  测试了一下通过。可以正常接口验证了。


Control Plane 的翻译看是否可以再调整下呢？

--> 控制面，舵面，制导机


yeah i should get rid of all of this extra vms crap before someone tries to use it :\


:+1:  ( ・∀・)ｲｲ!!


> is it acceptable for options that only have a single arg value to pass an int instead of a long?

On non-windows 64bit systems that will generally lead to badness, exactly as you say. I've tried to write that a long must be passed in all descriptions of numerical vararg options but I'm sure it can be emphasized better and as you say, showing just "-1" is fairly misleading or downright wrong. Although "-1" is of course not exactly stating if it is long or int.

As definitions are just numbers I've tried to not imply that they are integers or longs so that implementations can choose to use them however they want, it is only passing them as arguments to libcurl vararg functions that need to have them as longs.

(as a side-note I could mention that I believe this int/long confusion is among the worst design decisions I've tricked us into having...)


user could/should be parameterized so it's easier for extensions like this to change it


1: M111 unrecognized component name: elasticsearch


1: M111 unrecognized component name: annotations


Thanks, I simplified the test in the next commit according to your suggestion.


I personally think that if the current Laravel version doesn't require it, then don't require it. When it's required you can require it.

And if you want to encourage people to use latest versions of php, why 5.5.9 and not newer?

Anyways, turns out media temple actually runs 5.5.21 so at least for me it doesn't matter.


Sorry, don't understand why do you want to change this. Can you please elaborate. What exactly is the conflict you mention in the commit msg and why does decreasing the weights by 10 solve it?


Those are legion exclusions. The vanilla calls would try and build all the Legion units and then spam the logs with entries about how it wants to build the thing but it doesn't have access to the thing, regardless of any tests in that particular block. Basically, having a builder listed against an item it can't possibly build is bad.

The Uberwhatevers are the vanilla calls but excluding units with Custom1, meaning they're vanilla only. This avoids the log spam and hopefully aids performance.


Gave it a quick try and the version of protobuf that you are defining in depends (version 2.6.1 from 21 October 2014) does not have the CMake buildsystem yet which was added on 31 May 2015.


Why?


github code review test.


Seems to be reasonable.


        if ($options) {
            $this->options = array_replace_recursive($this->options,$options);
        }


s/reccomended/recommended/


But I want my constructor to be visible in my Main Class as well, in order to use it. What am I missing?


Now you will most likely return a wrong value if a format has FMT_8_BIT set, bot the GPU driver/OpenCL library prints some arbitrary output, see https://github.com/magnumripper/jtrTestSuite/issues/57


Thank you for such a great update!
It was surprise. It's working great. It's simpler to write and completion working better.


presentation


<code>PVR_TIMER_TYPE_SUPPORTS_END_TIME     | </code> is missing here.


Only the led notification which only works when you are plugged on the USB? but since the stock rom doesn't have led notification we could consider it not a bug


;p

Your work is still in.


CDMA is now working in LTE but still a bit unstable


@DJAlik I will add it to the commit message of the script.


it's going f9


@rlaalstn5383 이 커밋에 대해서 어떻게 생각하시는지?? ^^


work on the input required


Not sure exactly what you mean. I called the function that because I took out the line at the end that called "record.save" and moved it to another function (so that I could have a version that just returned the record, which I would save after modifying).


Awesome


I found the reason for the PushSimpleTest failure, is because the repository didn't have enough history built up so "origin/master^" fails to find a commit and then it tries to interpret it as a path.

We can fix the incorrect interpretation by explicitly adding -- to the arguments.

origin/master^ works in the test with the previous NoRefSpec push test being there because it creates the commit that was missing before.

As for why PushNoRefSpec doesn't work, it seems it has to do with configuring "push.default = upstream" which the default "simple" appears to work for this case, but upstream does not.

it appears that upstream breaks it because we change the remote and then it doesn't automatically include the branch name. I'm wondering if we should fix the CLI implementation to properly set "push.default" settings as I don't believe we can fix the test to do so?

Basically "upstream" is just how I use my local setup for normal development, so I can work around this and drop this patch, since it appears we need a previous push to exist in order for the next set of pushes to successfully run...

Alternatively we can change this test to just explicitly include the branch name. At any rate, I have re-worked this patch description and change my branch name to jk- instead of jk/ as suggested above.


Ummm...can I ask what happened here?

1. The commit message doesn't even relate. Was this even meant to be committed? Can you just rebase and drop it?

2. What's the problem with `mktemp` for you? This is supposed to be creating a _scratch space_ unique to each run of the program so that they don't clobber each-other. The only safe way to do that is with the system supplied tools for doing so. Not only do they guarantee you get setup with the right permissions in temp file systems (or RAM disks) this also generates unique names. Having each scratch space in a unique directory per run allows you do parallelize the script and run as many simultaneous processes as your system will compute. This will make an order of magnitude difference when it comes time to generate a full set of files in  production. Without this you don't allow for that possibility and you guarantee inconsistent results between runs.

   If you to run it a fixed location for debugging purposes and manage that space yourself you can by passing it any known directory. But if you are going to manage the space yourself it expects you to actually do that. Running `mkdir` like this is not safe and will encourage incorrect usage down the road and people will not catch their mistake usage mistake because it fails silently.

   The three usages are:

   1. Set nothing and it will do the right thing by the system.
   2. Set custom path in a template format (a string including at least three X's as randomness placeholders) and let it do the right thing by the system but using a user's preferred path or name format.
   3. Set a path that exists (such as a previously auto-created but not cleaned up scratch space from a previous run or a custom location you've prepared yourself for the purpose).


This is wrong, but I fear you squeezed this into another commit? Not good. Always taylor commits the way that they could be merged without functionally breaking things. Making stuff compile again is not sufficient.


Doing this on every tick seems suspicious to me.

First, unless I am mistaken, it looks like labelHolders[i] is not modified when we center a node about its reference point. This seems suboptimal since two labels could be assigned disjoint positions by the force layout algorithm, but collapse to the same position when they are centered (and therefore overlap when they are displayed).

Also, if we did updated labelHolders[i] to update the fact it has been centered after every iteration of applying forces to the nodes, then I think the centering behavior might undo some of the good work the forces are doing to separate the nodes.

For these reasons, I would favor removing this behavior, unless the PAs really insist upon it. Let me know if I have misunderstood anything, or if you want to talk about this.


That is the whole purpose of using the singleton pattern. You no longer create the object in this way
`ClassName firstInstance = new ClassName();
because this means that you can also write
`ClassName secondInstance = new ClassName();`
-> two different instances of the same class! The point of the singleton pattern is to force you to have only one instance of this class everywhere and to limit your creation of instances to just one.

Whenever you type everywhere outside of the `MatrixOperations` class like
`MatrixOperations m = new MatrixOperations();` -> this should give you a compilation error. You should not be able to create an instance of this class anywhere outside the class.

This is why you have the `getInstance()` method. The way you instantiate an singleton object is by calling this method.
`MatrixOperations m = MatrixOperations.getInstance();`

Inside the `getInstance()` method you see that it checks whether or not the current object (`instance`) is null. If it is, it create a new one, if not, it simply returns the one already created. That means that it will only get to instantiate the `instance` variable ONCE! Regardless of how many times you call this `getInstance()` method.


Review done.


@mgechev Loading templates for unit test is referenced issue in Millestone [alpha.33](https://github.com/angular/angular/milestones/alpha-33) and is available [here](https://github.com/angular/angular/issues/1610).


if you want to find out if it's NodeJS use this:
`if (typeof process !== "undefined" && process.title !== '"browser") {//NodeJS}`
but in your case this feels much better:
`if (typeof window === "undefined") {//NodeJS}`


სად რომელია ვერ ნახავ?

On Thu, Jul 30, 2015 at 10:43 AM, Dimitri Kurashvili <
notifications@github.com> wrote:

> ხო, არეული ყოფილა
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kakheti/pathfinder/commit/0d501ffe4fbbe37bc5251a5c1a17de046ab27fd0#commitcomment-12437117>
> .
>


this breaks apps with browserify and webpack.
If you want to use `NodeFS.readFile` in line 41, then check if exactly this function does exist.

you should should never assume NodeJS environment for that condition
see: https://github.com/mudcube/MIDI.js/commit/4111d818b87d22731fb6518a313fde299698286d#commitcomment-12437218


I think this make to nosetests failed on Travis.


got it! thank you.


멋져용~


[WIP] Add docker support


No problemo


 1. Что означает текст GitHub в ссылке?
 2. Почему не использовано более распространенное расширение файла .md?
 3. По возможночсти следует использовать относительные ссылки (relative links): https://github.com/blog/1395-relative-links-in-markup-files


we could check the options first, if options in undefined, it call the api with undefined without throwing error.


@kytwb yeah, but the activity has kinda slowed down :(


Please use `logger.error` instead of `logger.exception`. There is no exception in this context.


> ユーザーに別の UI 要素を移動させるときに使うもので
> 並行軸および垂直軸で制御します
> 通常表示されるのは大きなイメージや
> 別の UI 要素のパネルです よくある用途としては
> 一列に並んだボタン群や大量のテキストブロックがあります

もうちょい柔らかくならないかなーと思いました。

様々な UI 要素をユーザーが移動させる時に使うもので
水平／垂直に動かすことが出来ます
表示するものは大きな画像や
複数のボタン、量の多い文章などです。


> 様々な UI 要素をユーザーが移動させる時に使うもので
様々な UI 要素をユーザーの手で移動させる時に使うもので


If you pass error_on_warning=True to yara.compile() it should supress it. With this patch if you don't pass error_on_warning you won't get any error notification at all, because you are not setting the compiler callback.


@FRosner Can you please restart the job? Some compiling issue occurred: https://travis-ci.org/FRosner/spawncamping-dds/jobs/72306002


@schisma Fu, I'm always that guy who's late at parties. Whatever, back to code then.


The pose label represented as the Camera position(x, y, z) on sphere is used for training data arrangement, So the network is enhanced for seperate different poses from the same object.


Please use the same message for log and exception. I guess we can use a separate method to encapsulate the logic to either just log or throw an exception.


А нафига такой импорт?


Oh, sounds good. Then we can wait for a few days.


I didn't even know that rsh exist thanks ;)


@okdave,

okay thanks for your answer :)

You could have a DefaultClient ?

So the previous api does not get broken. And people who don't really need multiple clients dont have to refector tons of code :p


Thanks :)


Тут есть циклическая зависимость между list.py и iterable.py. Вот это — самый простой способ с этим бороться.


You forget to change comment above this code, quote:
```
    Sends a GCM notification to a single registration_id.
	This will send the notification as form data if possible, otherwise it will
	fall back to json data.
	If sending multiple notifications, it is more efficient to use
	gcm_send_bulk_message() with a list of registration_ids
	A reference of extra keyword arguments sent to the server is available here:
	https://developers.google.com/cloud-messaging/server-ref#downstream
```
So,
```
    This will send the notification as form data if possible, otherwise it will
	fall back to json data.
```
is incorrect now


we can reorder the check like
first check: options.width && option.height anyhow we are checking it in "options.width && !option.height" if bothe match it will return from there. instead of checking others. In real scenario both has to be present

```js
if (options.width && option.height)
                return root + "resizenp/" + options.width + "x" + options.height + "/" + url;
            else if (options.width && !option.height)
                return root + "width/" + options.width + "/" + url;
            else if (!options.width && option.height)
                return root + "height/" + options.height + "/" + url;
            else
                return url;
```


А не проще тогда принимать один опциональный аргумент?


DI error fixed as I was calling angular.module('name',[DI]) twice
Point to note : You should call a setter (module name and dependencies) before calling any getter.
A setter can be called only once, where as a getter can be called as many times as needed.


I'm not for this, backlog and threads can manage this in a clean way, a sub process is overkill for such a simple usecase.

This will only degrade perfs for lot of people instead of fighting the root cause (backlog) and locking can be solved with a thread.


Yep maybe, I don't know all about cmake, but using only prefix will install not correctly for packaging.


Why remove the pickle part? What is people where using it ? Will is just fail now for them?


If documenting the type params, adding `@tparam A the Java8 temporal type to be considered: LocalDateTime, ZonedDateTime, Instant` could be nice.


And here we see GitHub going full retarded.


> 基本的な事柄しか紹介しません
基本的な機能しか紹介しません。がいい気がしました


Хм, а cmp куда дел?


Проблема в том, что дефолтным значением не может быть None, потому что .count(None) — валидная операция. Но раз я уже ниже запилил NO_ARGUMENT_PASSED, то надо и тут так сделать, да.


I'm not for this default value. It was true, so should keep the same default to not change previous installation behavior.


И ваще, почти тут то лист возвращается, а не Iterable?


pls use "Loading ......" replace of loading gif to pass the license issue firstly.


Гвидо не велит :)


Note: one other way is to allow UDP. Really works well, unless network is down, but then even your tcp will just make growing data andforce you to drop or backlog.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.428](http://teamcity.monogame.net/viewLog.html?buildId=9217&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Win10) Build time: 00:00:00


Отступы в коде не стоит ломать.


Ah, missed the caret there, so yeah, please a comment ;)


Файлы с исходниками на любом языке программирования должны оканчиваться переводом строки, старое требование, скорее ставшее признаком хорошего тона сегодня:
http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline


Why remove the buffer part? Now it will add latency for each check, so for very large env it will just explode (unless UDP).

I prefer to have te tick behavior/buffer back, ino a thread if need, than just add altency for every one (that will make large env size problem even worse).


> ゲームオブジェクトに scroll rect のスクリプトを追加するには

ここスクリプトって言っているんですね...
本当は 「Scroll Rect コンポーネントをアタッチするには」が正しいです


Это что? Отступ ТАБом?
Плохо, принято использовать 4 пробела за 1 отступ,  и в этом проекте будем придерживаться такого правила.


I have changed what you've told me and implemented twist2.


@MikroAcse Thank you, do you use spigot 1.8.7? Someone told us this fix doesn't work with latest versions, but it's impossible :/


@Qubad786 you should not add your own PyCharm Cache folder and .idea folder on the github. Please remove those folders from your repo and make sure to add the .gitignore file, add these folders in that file so it won't be able to add on github again.


Hash#try_convert does the trick, see https://github.com/cucumber/cucumber-ruby/commit/4e51bafa7a405421acb0145e5223f770b36505c6 and https://travis-ci.org/cucumber/cucumber-ruby/builds/73328950


Ah, I understand, any ASM that is generated like the ShaderCache files ought to go through his disassembler to avoid the problems with precision.

That was one spot I hadn't quite finished (mostly is BinaryToAsmText), we have two different spots where we call d3d_disassemble.  I'll see if I can't unify those into one routine that uses his.


Also, make sure not to commit .pyc files. these are just compiled files and will cause trouble for other members of the team. add .pyc in .gitignore as well.


По PEP8 методы и атрибуты класса разделяются одной строкой: https://www.python.org/dev/peps/pep-0008/


MemberArray_Reduce (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1777 of 1777 tests passed)

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


Add sys prop back here and in the next 2 files


@kytwb
It might kick back up tomorrow morning in North America


TeamCity Subsonic :: Debug [Build 14](http://109.103.226.38:8512/viewLog.html?buildId=34671&buildTypeId=Subsonic_Debug) is now running


type is a reserved word. you should not use this as your model field.


Предлагаю поле с зарплатой сделать необязательным, на первое время, по крайней мере, пока не придумаем более расширенных средств указания потенциальной зарплаты.


Это что же, циклическая зависимость?


The model class representing that it is a product. no need to add "product_" before your field name


TeamCity Subsonic :: Debug [Build 14](http://109.103.226.38:8512/viewLog.html?buildId=34671&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:01:12


I like this field name. well done. (y)


Она самая.


MemberArray_Reduce (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Еще раз замечание про перевод строки.


@mvasilkov Correction: "Here we see GitHub users going full retard"...


@krainboltgreene as a white straight cis-male you have absolutely no right to tell people what to do, you should educate yourself before coming telling people what to do.

"You're going to be alone and cynical in your little MRA "free speech" camps so busy being offended by the concept of offensiveness that you wither"
As a black lesbian transgender could you PLEASE tell me why in the hell I would be "alone and cynical in my little MRA 'free speech'" ? If you want to defend G**Hub by making that kind of assumption tell how better you are than anyone on this thread ?
"You don't agree ? You are a nazi scum." is basically what you just said


suggestion:  instead of writing date with the field name, you can write this field name as "added_on"


TeamCity Subsonic :: Debug [Build 15](http://109.103.226.38:8512/viewLog.html?buildId=34672&buildTypeId=Subsonic_Debug) is now running


Снова перевод строки отсутствует.


Excellent.


TeamCity Subsonic :: Debug [Build 15](http://109.103.226.38:8512/viewLog.html?buildId=34672&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:00:30


Try to create a separate css file. try to avoid inline css.


browsers doesn't


Try to avoid inline css and make a separate css file.


every question


Вот тут я могу заблуждаться, но разве return внутри генератора это нормально? Может лучше
```python
raise StopIteration()
```


update the form fields accordingly. added comments on the model class.


Built-in sorted для слабаков?


not the time


Пропуски строк были вставлены намеренно.
Сначала идут стандартные джанговские apps, затем сторонние, затем проектные.
Верни обратно, пожалуйста, и в будущем, если что-то уже сделано каким-то образом сначал поитересуйся зачем. Или, по крайней мере, уведоми, что изменил уже имеющуюся систему.


I fixed some stuff and it is working know. I'll merge the code once I finish some tests.
Again, thanks for your PR :)


comprehensive (at the end)


you can write this date as "ordered_on"


С отступом беда - пробел + ТАБ, нужно 4 пробела.


@spicyj One question. It makes very hard to catch some errors, since they are not rethrow an app with random exceptions in any React Cycle could just blows up and still "works". Any idea to tackle this problem? Thanks.


successfully


This will crash if no product with given id found.


nice. clean code.


If `needSend` is `False`, `mnext` is not used. Is this intentional?


Should call __Sequence::getItems()__ no?


django gives you a built-in method to use instead of filter and [0], and that method is get()
If you are certain that the product is already there, please use get() instead of filter() with [0]


This looks neat - would be good to get append API merged back :)


Warning: indentation...


nice nice...


patch-one-1 branch add some new information to readme


I have removed the check for the spatial filter


Isn't libcec only available in the jessie sources?


Well how do you want to control what is allowed to be read and what is not.  Some system files are required by java, so we can't just block those out completely, it would be a lot of nitpicking. Maybe we could deny access to my documents etc, but then they are in different parts of the system too, depending on OS.. Do you know how professional plugin based software does it? I just removed all that, because like you said, a lot of stuff was being denied.


As long as the reports are not usable, I would remove them and possibly add them later once they bring something useful.

@mcalmer: Will the SCC report change when forwarding is implemented or additional API is missing?


`return` внутри генератора — это нормально. Он именно `raise StopIteration()` и делает.


这个merge是在干嘛……

尽量用git rebase 不要直接merge


No need to store the values in separate variables. these values are not been used anywhere in the code. just assign them directly to the object from the form.
Example,
user = request.user
user.username = form.cleaned_data.get('name')
............


Forgot to comment last week. I could replace org.json with Jackson, but right now I don't think it's necessary. I created a test for performance which times out when parsing of json file with 1000 items takes longer than 200 ms. I don't think that is too slow, but I guess we will see when the code is used. Another benefit of Jackson would be that it takes care of parsing in a more portable way as you say. The problem here is that (as far as I know) I would have to create another class (like ProjectVersionRef) but with dependencies (so something like ProjectWithDependencies) because current solution would require a structure that is not typeble in Java (something like Map<String, String OR List<String>>). I might eventually create such a class if it is necessary but right know I am not sure if it's a good idea.


@MexinaD , I Believe it will read the contents of the second file if it exists


T-that's kinda cute. I like it.


Да, про iIterable вкурил, когда ниже увидел, что такое list(). А про cmp не понял, в стандартном sort же есть такой ключ?


Why do you need to define this function instead of keeping __Sequence::getItems__ ?
Get items of an item is strange...

Your code should seem like that :
```c++
if(item.getType() == eTypeSequence)
    return item.getSequence().getItems();
else
    return item // return it as array
```


sorry man, i'm on a little vacation ..

still following your work, i'll be back soon!!
i didn't abandon you! aaahahha

cheerssssss


`cmp` есть во втором питоне, но выпилен в третьем.


thinks


Nice catch, Simon :-)
 Will do...


files?


Maybe better to add a directory path argument to getFiles:
`return seq.getFiles(_path.parent_path())`


Wouldn't lowering the connection-timeout help? Deep down it's just a socket waiting for a connection...


Ah damn. The changes related to logging and Promise rejections were not meant to be in this commit. :-(


No Hiro robots are named `nx*` IMO.


If you're going to go the same way with all of these, I'd recommend send instead of reply.


Done


\o/ thanks


@andkirby commenting on a 3 year old commit is not the right way to start a discussion on the code. The only way to know about this discussion is through the notification we get (if we haven't disabled them), and there is no way to find the discussion again easily (looking at all commits to find the ones with active discussions is not a viable solution). If you want to start such discussion, opening an issue is much better


select first


I don't actually know the difference.

Is there one?


You should not fix compiled assets. Patch the LESS source file instead.

https://github.com/octobercms/october/blob/master/CONTRIBUTING.md#pull-requests (point 3)


It's impossible to enable TCP_FASTOPEN on Mac OS X:

https://github.com/shadowsocks/shadowsocks-libev/blob/master/src/common.h#L25


@robespier

под Windows содержимое файла `post-merge` такое:
```
#!/bin/sh
npm install
```


T-thanks, y-you too.


I don't think there's anything to release, you probably just need to pull the template repo.


If you use reply the bot directly answers the person with the trigger by prefixing the message with their name. Send just sends the message. So not much of a difference, send is just cleaner.


Danke, dass ist ein Fehler. SetViewer war wahrscheinlich schon da und ich habs nicht gesehen. SetGameWorldViewer mit dem check ist die richtige


It seems you have forgotten the signed-off-by footer on this, Simon. Do you want to add it or do you want me to make the changes manually (not a problem since it is not a big change anyways ...)


It will be too trivial work if you want to do few branches.
Another option is keep all commits we have generated


Now that we don't support Scala 2.9, you can use string interpolation like so:

`error(s"Haven't been able to send leader and isr requests, current state of the map is $leaderAndIsrRequestMap")`