Would it be simpler to make `core.counter.counter_t` a part of the interface?


Very good top level design of this code!


add attribute:
default="specified"


Shouldn't syntax highlighting be implemented in EditorCell?


`groupBy`?


I think it would be more correct to check the Kind == "ImageStreamTag" as well. (same below)


I am concerned that the licensing addition somewhat bypassed any discussion. When I created the empty repository, I did so to allow for a collective decision about licensing. Please remove the MIT license, and indicate any licensing preference in issue #2.

Respectfully,
Brylie Christopher Oxley


Yeah, I had a play around with that a couple of days ago.

The problem I had is that, unlike MediaSession, MediaSessionCompat demands that you provide it with a component (the mediaButtonEventReceiver parameter), which looked to be a lot more invasive than simply appending "Compat" in a few places.

I suspect the right thing to do is to recast some of Squeezer's internals so that it looks a bit more like https://github.com/googlesamples/android-UniversalMusicPlayer. But I need to do a bit more investigation of those APIs and see how well they match up to our use case (at first glance they might assume that they're in direct control of the music playing and that, for instance, nothing else can pause playback, which is not the case with players connected to an LMS).


makeImageStreamKey


Thank you for the answer :rainbow:


Damien Cassou writes:

> not sure because I think this is part of `auth-sources` API.

We use the `&allow-other-keys`, to I don't see the problem.


We should probably add "--strip-trailing-cr" here to prevent diff from choking on line endings.


remove 'related' ... since it accepts all of them


我更改这个了


same here


My pleasure, thanks for doing the hard work in the first place :)


yes


But Arc lookup did not work.  I think the bug is actually elsewhere and has
to do with the fact that arc is first in the array.  If you used the script
to calc all crcs ARC would show nil.  (Might also be an issue with length
of the name arc in the script.)
But since they are one and the same, I just swapped it.
On Jun 11, 2015 5:32 AM, "Iceman" <notifications@github.com> wrote:

> In the docs of reveng, it say ARC is often the definition of CRC-16. But
> its name is still ARC?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marshmellow42/proxmark3/commit/5afe9838eb6c911ab136acdca101830e42b069fb#commitcomment-11629973>
> .
>


When I took out the while loop, it only deleted one entry, not all of them. Is there something I am missing?


Thanks to @jack-webster for the proposed fix.


I think this is good


Peter C and I had some discussions by email on how the "placeholder" works and should work. I'm not convinced we have an ideal solution for that.

To generalise the _errors it would be nicer to have the markup on the _errors definition. As in: I refer to a dataset of my name with the _errors stripped. As opposed to add "I can have errors" to everything. For the moment "placeholder" is worse, it just says "there will be something somewhere that includes my name". Not nice if you have a "t" axis.

"specified" vs "any" looks good though.


Done in songkick/oauth2-provider#84.


这里的修改，有遇到过什么实际的场景卡在这里么？我的stack有大量线程卡在这里：at java.util.Hashtable.get(Hashtable.java:333)
        -  blocked on java.util.Properties@3c64ba34
        at java.util.Properties.getProperty(Properties.java:932)
        at java.lang.System.getProperty(System.java:653)
        at com.alibaba.fastjson.util.ASMUtils.isAndroid(ASMUtils.java:28)
        at com.alibaba.fastjson.parser.JSONScanner.subString(JSONScanner.java:179)
        at com.alibaba.fastjson.parser.JSONScanner.stringVal(JSONScanner.java:172)
        at com.alibaba.fastjson.parser.DefaultJSONParser.parseObject(DefaultJSONParser.java:394)
        at com.alibaba.fastjson.parser.DefaultJSONParser.parseArray(DefaultJSONParser.java:1071)

fastjson版本 1.2.3，但是不知道原因


I'd avoid sort if it possible


Commit [569d364eb712537b5d3d627d0517d5a201a88869] makes the entire file conform to the WordPress Coding Standards


Use ```render :change_password```. Please change in all controllers


aha,  now I see.  True,  it said nil in the script.  So by making the name long it works?!?  Very strange.
LUA is one-index based..  Could have something to do with that... scripting.c...  ( i-1 )... error..


My rating for this project
```
Name - 1/10 animu disgusting
Commit name - 10/10
Readme.md - 11/10
```


in pls_.py / PLSRegression / predict()
the default (copy==True) was not to divide X by std
this commit changes this behavior to always divide by STD and breaks backwards compatibility for me.
can we add an option to not do this normalization?
what should be the default?


excellent commit!!!


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Missing documents, but will add it.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 537](http://build.prxm.it:8811/viewLog.html?buildId=2973&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 537](http://build.prxm.it:8811/viewLog.html?buildId=2973&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:06


Standardize to use `P` instead of `p`?


Dear committers. Why on Earth do you allow files that differ in capitals only?

Please don't do that.


Remove commented code


To show error messages in new line, i have added <br/> and to render it in html, i have used html_safe.


Use haml tag


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 538](http://build.prxm.it:8811/viewLog.html?buildId=2975&buildTypeId=Jarvis_DocumentStore_CI) is now running


@Raesu DWMR. Press on.


I remove reading in KGenTauJet objects from 8TeV skims, since they are not available in the committed samples. In case somebody needs them, we need to find a solution that works for everybody (e.g. correctly specify the full nick of the sample where the jets are available).


Relates with issue #411 .


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 538](http://build.prxm.it:8811/viewLog.html?buildId=2975&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Compilation error: src\Jarvis.DocumentStore.JobsHost\Jarvis.DocumentStore.JobsHost.csproj (new) Build time: 00:00:31


Rad! Thanks.


Worth keeping a reference to first (or all) exceptions and throw afterwards?


:beers:


refactoring js and styles


I want to change `struct link` in `link.h` like so:

```
struct link {
   // this is a circular ring buffer, as described at:
   //   http://en.wikipedia.org/wiki/Circular_buffer
   struct packet* packets[LINK_RING_SIZE];
   struct {
-    double txbytes, rxbytes, txpackets, rxpackets, txdrop;
+    struct counter *txbytes, *rxbytes, *txpackets, *rxpackets, *txdrop;
   } stats;
```


翻墙


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 29](http://ci.reaver-project.org/viewLog.html?buildId=519&buildTypeId=mayfly_TestBuildWithClang&guest=1) is now running


TeamCity Reaver Project :: Mayfly :: Test Build with GCC [Build 27](http://ci.reaver-project.org/viewLog.html?buildId=517&buildTypeId=mayfly_TestBuildWithGcc&guest=1) is now running


Originally, wasn't sure I didn't need name, but it clearly isn't used now.


I would normally use 'double' instead of _real64. I'm not very familiar with _real64 and I'd guess it's not in all compilers.


No pull request? :cry:


TeamCity Reaver Project :: Mayfly :: Test Build with Clang [Build 29](http://ci.reaver-project.org/viewLog.html?buildId=519&buildTypeId=mayfly_TestBuildWithClang&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:00:32


TeamCity Reaver Project :: Mayfly :: Test Build with GCC [Build 27](http://ci.reaver-project.org/viewLog.html?buildId=517&buildTypeId=mayfly_TestBuildWithGcc&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 28 Build time: 00:00:36


![Uploading 1-www_soideas_cn12041Q94947_big.jpg…]()


Could you further simplify with the following?

```
option.values[value.position] ?= {value: value, variants: []}
```


That seems reasonable although I wonder if there is a simpler way. Should we make the `counter` module operate on a simpler type like `uint64_t*` instead so that no definition needs to be made and shared?


I've updated the commit in the PR.


Oh, great - I didn't know about `closest`


@surayashivji DWMR. Good work here. Press on.


_real64 -> double


LGTM


good catch, fixed in 7781609104915910702d3340bcaa10bde827581c


Thanks @jimmypuckett, i have not see the difference in the commit...


using an anonymous function is the best solution in my opinion


Why does chpl_tuner_fini return chpl_bool?
What does the return value indicate? What do you expect a Chapel program to do with the result?
(I'd expect chpl_tuner_fini to get called automatically/in a destructor somewhere and in that case it would be hard to respond to the return value. And if it's returning whether or not there is an error, there's not enough information to respond).


This is surely some complex and convoluted undertaking – I have no idea why I am messing with it


After testing...
The         run("KEEPCOUNT=%s;find /projects/keeplatest -maxdepth 1 | grep %s | sort -rn | tail -n +$KEEPCOUNT+1 | while read folder; do rm -rf \"$folder\" ; done;" % (params['releases'], params['path'], params['name']))
 command gives error...


Misspelled canola


global variables like this should have a chpl_ prefix


What's evil about it?


Yep, agreed :+1:


Honestly I noticed my C has gotten quite rusty, I don't feel immediately comfortable with the (INSANE) syntax and semantics anymore. I personally think `struct counter` is "good enough" because this is something that can be changed easily later on. The counter type should be chosen based on practical considerations (e.g. the `od` trick you mentioned).


This function should be called chpl_tunable (for C namespace reasons) and then your Chapel module can create an (inline) function called tunable that calls chpl_tunable.


Fixed


@mattsoutherden I'm not entirely sure that this solution works. I have attempted to deploy FCA with this changes and the issue seems to persist.

I then attempted to deploy FCA with @sgerrand [changes](https://github.com/songkick/oauth2-provider/pull/84) and that seems to have worked.


It's perl? *ducks*


I'd consider just adding one thing here "TUNER_DATA" or something, but if you think the 3 options here are useful that's fine too.


aaaadfasfsafsd


@levicole DWMR. :+1:  Press on!


sdfsdsafsdafsaffffffff dasfdssd


Hi !
Visual studio compiler does not support variable length arrays. So this line of code does not compile on Windows.
Greetings,
Laura


First commit should probably have read 2nd, I know!!! Took a short cut and got caught out. :)


No PMD, Checkstyle, or Findbugs introduced


I don't understand why the compiler needs to know about dtTuneID. I would expect this type to only be used from module code - and in that case, the extern type mechanism should be sufficient. Do you have some reason why you want the compiler to know about this type?


wtf??


But we can not rollback these `INSERTs`, the block that executes this code does not provides a `rollback` parameter . Also, [a few lines above](https://github.com/cloudant/CDTDatastore/commit/8d934124dcbb252cfc15bb6f7cf9f424a8777d7f#diff-8e8b4e1d26db1376fbf88a600a1a7a1fR481), the `replicator` table is update and we are not checking the result (I know it is not a good excuse).


TeamCity Azmq :: Build (Linux) [Build 111](https://78.46.71.115/viewLog.html?buildId=533&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 142](https://78.46.71.115/viewLog.html?buildId=534&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 95](https://78.46.71.115/viewLog.html?buildId=535&buildTypeId=Azmq_BuildOsx&guest=1) is now running


as in I remove paymentTypeDescription?


This breaks API compatibility.
It's possible to add default value = 0, then in new methods we can check connection memeber for null and return "not implemented" error in that case.

I'm not sure if it worth it.


Oh noes!! :+1:


TeamCity Azmq :: Build (OSX) [Build 95](https://78.46.71.115/viewLog.html?buildId=535&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:29


Is it necessary to re-define elfeed's variables here?


Not sure if I grok this, but it seems to me that the _minlength_ and _maxlength_ attributes won't actually get any values in the HTML element here, according to https://github.com/SteveBenner/hyper_admin/commit/38bf3daa1697af8747138c89c4a4a7dc254454c0#diff-8eee3caf1b1b159064fc92bf7f96dd5bR52, where it splits those two attributes. Shouldn't these also end up something like this:

```html
<input minlength="15" maxlength="20" ng-minlength="15" ng-maxlength="20">
```

Or maybe I'm reading this wrong. :stuck_out_tongue:


TeamCity Azmq :: Build (Windows) [Build 142](https://78.46.71.115/viewLog.html?buildId=534&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:54


TeamCity Azmq :: Build (Linux) [Build 111](https://78.46.71.115/viewLog.html?buildId=533&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:07


Fixed on 76b9c17


hi, which browsers support?


Ok, the tutorial's been updated, here's the latest code for reference:

```ruby
   def search_entries
 # #9
+    print "Search by name: "
+    name = gets.chomp
 # #10
+    match = @address_book.binary_search(name)
+    system "clear"
 # #11
+    if match
+      puts match.to_s
+      search_submenu(match)
+    else
+      puts "No match found for #{name}"
+    end
   end
```


Ah, well, if I took your suggestion, resource ends up being only the key, not the entire part of the graph, which I need access to. I can simplify by removing ".keys()" though.


done


YES YOU ARE -- M. BACCONNIER


Please send these changes in together with SnabbTop and we can check it out in context.

I only have the micro-concern that `counter` is such a trivial type that it seems a bit verbose to dedicate a header file to it and have to import that. However, I don't want to hold you up, and I can always propose a type refactoring change on top later to see if that makes sense.


I'd argue things about people's specific working environments shouldn't be in our project-level gitignore files but, rather, in their own user-level gitignores? (This goes for `.DS_Store` later, too.)

That way, we're using gitignore to document the stuff that *our project* creates that we don't want to be committable.


TeamCity Azmq :: Build (Windows) [Build 143](https://78.46.71.115/viewLog.html?buildId=537&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 96](https://78.46.71.115/viewLog.html?buildId=538&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 112](https://78.46.71.115/viewLog.html?buildId=536&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 112](https://78.46.71.115/viewLog.html?buildId=536&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:09


TeamCity Azmq :: Build (OSX) [Build 96](https://78.46.71.115/viewLog.html?buildId=538&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:15


TeamCity Azmq :: Build (Windows) [Build 143](https://78.46.71.115/viewLog.html?buildId=537&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:17


@KsyMC 야자 안하고 이거만 하셨나 ㅋㅋㅋ


Viens donc me le dire de profil !


Awesome!


Where is the lowercase `p` used?


For big 3rd party packages, we use one of two patterns:
1) put a tarball in the Chapel repo and untar it or
2) bring the source into revision control as you did here.

Generally though, we only do 2 if we expect to have Chapel-specific patches to the library. I don't imagine that's the case here - so I would probably prefer the tarball approach.

Note that including Active Harmony in Chapel releases will require that somebody verify that Active Harmony is license compatible with Chapel. It looks like it's LGPL which I think is OK, but I'm not the one who checks these things.... I would expect that the license check step will take time, and so your pull request might get slowed down by it. For that reason, I'd suggest that initially you write the Makefiles and scripts to download a tarball (or do a checkout) of Active Harmony from another source.

Once the license for Active Harmony has been checked out/approved, we can move that to be in-tree (either as a tarball or as the source code).


comment ile backup alinmaz :) sil onlari


This may be a regression for invalid timestamp `$value`s


indentation!!!


hai


Guzel. Sql ile sunucu yukunu bolmek akillica :)


Ah that's interesting. What's happening is that the `entries` array is being modified at the same time as its being iterated.

Modifying a collection while looping through it can lead to unexpected or buggy behaviour and is best avoided.

Avoid looping altogether and call `entries.clear`. See http://ruby-doc.org/core-2.2.0/Array.html#method-i-clear and scan through the other available methods on `Array` as they're worth being familiar with.


Bir onceki commit ile ayni mesaj. Bu commit in mesajinin comentleri kaldir <-- Emir kipi olmasi daha dogru olurdu


my bad. it looked like a small P in the diff! :D


indent


Bu commit mesaj da bir oncekinin aynisi. Olmaz


Indeed it does :D


Overall, this set of patches looks like good work to me and I'm excited to see it move forward.
In order for it to get merged with master, you will need:
 - contributor agreements (ICLA and CCLA)
 - verification of the Active Harmony license (or only commit scripts to download it)

Also, it may just be an artifact of this particular version, but the "compare files"
feature on GitHub is showing many changes that aren't authored by you. You'll
need to reapply your patches to a clean, recent master branch in order to
get a clean Pull Request. (Or fix it another way).


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.265](http://teamcity.monogame.net/viewLog.html?buildId=8224&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Thanks for this. Should we have a new release with this fix?


Shouldn't this block in the task itself, in `Callable.call()`? That would seem to be the pattern of the other latches, cf. `AbstractSoftwareProcessDriver.start()`.


Good catch, I fixed that in commit a6167e4, in this [line](https://github.com/cloudant/CDTDatastore/commit/a6167e4d1b679c883e21c296022428c6cf2e178c#diff-64cf4ed96700d9bd22b060e7ef6ee938R113)


magic :1234:


Use the local variable `entries` you created on line 163:

```ruby
if !entries.empty?
```

or

```ruby
if entries.any?
```

Comparing booleans in `if` conditions with `== false` and `== true` can always be omitted.


This fixed a lot of issues for us.  Thanks Chris.  If our XML string was too long, curl would only send the first 1024 characters.  Now it will send as many characters as you want.


hi, i have a code written in delphi7. it was called the functions "cvCreateCameraCapture" , "cvQueryFrame" to grab frames as IplImage*.  my code works fine with opencv 2.4.11 but
because of the changes that cvCreateCameraCapture function don't call VideoCapture_DShow my code don't work. i want to ask if i must try to find a solution or keep using opencv 2.4.11


This change, as far as I can tell ONLY affects the ZIP file distribution, not the Neo4j Desktop EXE distribution.

@akollegger Currently, the powershell scripts can completely replace the functionality of the Batch files.  Like Ben mentioned,  they are a shorter way of invoking of specific powershell functions. They do provide backwards compatibility for existing installation scripts so people can migrate from 2.x to 2.3 with no change in installation code.

The tricky part comes with OS support.  Does Neo support operating systems which that vendor no longer supports i.e. Does Neo Tech. support Neo4j on Windows XP and Server 2003 (post July 2015)?  It certainly doesn't sound like something it should do.  Windows 7 and Server 2008 R2 come with Powershell 2.0 installed in the OS. That only leaves Vista and Server 2008 that will be unable to manage a Neo4j instance that is deployed via the ZIP files, without first installing Powershell.  There was a request for querying the UDC data to find the adoption of these two OSes but I don't have that answer yet.  It may not matter much but Vista, 2008 and 2008 R2 are all in extended support.

I do like the idea of changing the text to "Did you know that a Powershell ...."

@benbc If you know the name of the Neo4j windows service, then net.exe (old school) or sc.exe or Stop-Service (powershell) are fine.  Use the tools in the OS.  Same with GUI, use services.msc.  The advantage of using my powershell module is you won't need to know the name of the windows service, it's read from the configuration file e.g. ` "c:\neo4j\neo4j-community" | Stop-Neo4jServer `


:-1: this change is wrong, because trying to destroy an inactive network will trigger an exception and then not undefine it


Sure, good catch! I'll fix it ASAP. Thanks!


:joy: :joy_cat: Ok ok :stuck_out_tongue_winking_eye:


I've already uploaded a new version 0.9.348.


Ember needs all requests to go to index.html for routing to work.


@LorenzMeier  I wrote the code working on a project that used off-board local position setpoints from ROS to command drones. I flew it on an IRIS, and an M690L align. It has around 100 test flights behind it in offboard mode. Without this fix, the vehicle will hit the maximum allowed pitch/roll angle, I saw the Iris going at 25 m/s trying to follow offboard points about 100 m apart and with altitude varying by 60 m. It was the most impressive thing I have ever seen a quadrotor do, but listening to the velocity saturation is definitely an improvement :-).


It's a slave node.


return $this, or return $instance?


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 539](http://build.prxm.it:8811/viewLog.html?buildId=2976&buildTypeId=Jarvis_DocumentStore_CI) is now running


owwwww nice


Instead of importing this huge node.js dependency, I'd rather let a simple Python script like this update the contributors list:

```python
import httplib
import json

URL = 'api.github.com'
REQ = '/repos/matze/mtheme/contributors'

HEADERS = {"User-Agent": "curl/7.13.2 (i386-pc-linux-gnu) libcurl/7.13.2 OpenSSL/0.9.7e zlib/1.2.2 libidn/0.5.13"}

conn = httplib.HTTPSConnection(URL)
conn.request('GET', REQ, None, HEADERS)
resp = conn.getresponse()

if resp.status == 200:
    data = json.loads(resp.read())
    print("\\begin{itemize}")

    for name, url in ((c['login'], c['html_url']) for c in data):
        print("   \\item \\href{%s}{%s}" % (name, url))

    print("\\end{itemize}")
else:
    print("Couldn't get contributors")
```


Typo VM


Is this a typo? :)


Yeah, it's still a todo - it's one of the things mentioned in the pull request message


aqui se veran mis conocimientos


W razie czego, to zmienne środowiskowe też można dodać w UI: https://travis-ci.org/PartiaRazem/mapy/settings/env_vars, w ten sposób nie zaśmieca się .travis.yml i nie ma problemu jak ktoś używa forka.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 539](http://build.prxm.it:8811/viewLog.html?buildId=2976&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Compilation error: src\Jarvis.DocumentStore.Jobs\Jarvis.DocumentStore.Jobs.Tika\Jarvis.DocumentStore.Jobs.Tika.csproj (new) Build time: 00:01:21


otra linea


otro coment


`current_branch` is working fine. I can't catch the error. Here's what happens:
```
[shark0der@laptop:~/repo on master]
% ggpnp
From bitbucket.org:shark0der/repo
 * branch            HEAD       -> FETCH_HEAD
Already up-to-date.
fatal: remote part of refspec is not a valid name in
[shark0der@laptop:~/repo on master]
% ggl                                                                       128 ↵
From bitbucket.org:shark0der/repo
 * branch            master     -> FETCH_HEAD
Already up-to-date.
[shark0der@laptop:~/repo on master]
% ggpnp
From bitbucket.org:shark0der/repo
 * branch            master     -> FETCH_HEAD
Already up-to-date.
Everything up-to-date
[shark0der@laptop:~/repo on master]
%
```
The subsequent `ggpnp` calls work fine. If I close the terminal and open it again (or just invoke `zsh` in the current terminal) - `ggpnp` fails again. If I don't close it but just cd around, it works fine. Note that the second part of `ggpnp` fails (the `ggp` I guess) and it fails fast (i.e. no connections to origin are initiated). Calling `ggl` or `ggp` alone work without issues.


Thanks for the comment. Will move the block to the constructor.


PS: The remotes are set correctly (or else it would fail on `ggl` / `ggp` as well).


I never got to reviewing this branch, but now I noticed this.  Why are we doing `shell=True` with a command line here and `shell=False` with arguments (the recommended way) on the initial build?


I'll note it's odd that we construct `mSet` by calling the default constructor and then call `init`. Add a constructor to `TransactionMetaSet`?


Why do you CreateNoWindow = false? Wouldn't true be better here?


@tomspilman Was originally aiming to target Xam branch, but just ran this commit by @dellis1972 to make sure it was ok. Otherwise, if you prefer to revert the change I'll setup a PR instead.


Good spot to pick up Spritely.Recipes.SecureString :)


@yjun2 DWMR. We can cover any topics here when we speak next.


You are ~~un~~welcome


wat? 6?


Could possibly do with rewording.  "Metadata" includes code to process the meta data.


@64FF00 야자안합니다 ㅋ


I may be being a bit over-ambitious, and I don't know how long it'll be if it happens, but I suspect the next release might be version 8.0 and bring SpellForge and MonsterForge back into the fold. I don't currently have copies of those though, so I'd need to track them down.


Okay, I understand your concern. I had added them in case we need them later. But as all the attributes are public, I will remove the getters and the setters.


What's the point of this?

Should SendFile return a bool for success/failure?


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


fixed it

> Op 11 jun. 2015, om 14:46 heeft Franco Fichtner <notifications@github.com> het volgende geschreven:
>
> Is this a typo? :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/opnsense/core/commit/ddceb45417787643f161d1841f087cf38a65210c#commitcomment-11632941>.
>


Tried to debug it:
```
[shark0der@laptop:~/repo on master]
% ggp () {
function> [[ "$#" != 1 ]] && b="$(current_branch)"
function> echo "params: $@"
function> echo "b: $b"
function> echo "something: " "${b:=$1}"
function> git push origin "${b:=$1}"
function> }
[shark0der@laptop:~/repo on master]
% ggpnp
From bitbucket.org:shark0der/finance
 * branch            HEAD       -> FETCH_HEAD
Already up-to-date.
params:
b:
something:
fatal: remote part of refspec is not a valid name in
```


+ ＃ DEVICE_RESOLUTION：= 720x1280


When I run `get_leaderboard( ..., All=TRUE)` it still returns only 200 queries from leaderboard table. Do you know if its API limit and if there is a way around it to return the entire leaderboard?


I have copies of the last updated version of both spellforge and
monsterforge, how would you like it delivered?
On Jun 11, 2015 15:54, "Heliomance" <notifications@github.com> wrote:

> I may be being a bit over-ambitious, and I don't know how long it'll be if
> it happens, but I suspect the next release might be version 8.0 and bring
> SpellForge and MonsterForge back into the fold. I don't currently have
> copies of those though, so I'd need to track them down.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Heliomance/HeroForge-Anew/commit/63d148b69bf9b9ce61dd24df1d0795e8269bfdd3#commitcomment-11633059>
> .
>


Fluentmigrator migration version numbers are of type long (Int64). The aspnet runner treated the version number as an int. We had problems using it because of this, our version numbers do not fit into Int32.


LGTM


Woops, I meant "any" suburl


You can make a more robust test.

Maybe by adding a loop inside your module and put this in output:
```
6
28
496
```


不小心傳錯
標題和內容不符><
其實這個只有在keydown的地方加上e.Handler=true;


use the method `.eq( 0 )` avoid custom pseudo selectors


Thanks


Fixed in 221aba1


text라는 이름이 너무 generic하지 않나? body 정도면 적절할거같은데


That could be part of another refactor - this is certainly not the last word on refactorings.


And finally I tried using the old function definition:
```
[shark0der@laptop:~/repo on master]
% function current_branch() {
function> ref=$(git symbolic-ref HEAD 2> /dev/null) || \
function cmdor> ref=$(git rev-parse --short HEAD 2> /dev/null) || return
function> echo ${ref#refs/heads/}
function> }
[shark0der@laptop:~/repo on master]
% ggpnp
From bitbucket.org:shark0der/finance
 * branch            HEAD       -> FETCH_HEAD
Already up-to-date.
params:
b:
something:
fatal: remote part of refspec is not a valid name in
```
I can't understand what's happening here...


I don't get it?
1. Why empty vector elements should have any name?
2. Why we iterate through all elements, when function name is addUnknownItem(int id) ? If we need names (because reason, I don't understand), we need iterating only between old and new size, not whole container.


아 처음에 :body로 했다가 Story#body랑 겹쳐서 헷갈릴까봐...상관없을까? 오히려 맞춰주는게 나을라나?


Make it a link


Did you try to use hardware support of in-order custom blending?

AFAIK almost all mobile GPUs support EXT_shader_framebuffer_fetch, it is just easy to use. Do you support OpenGL ES?

On the desktop side, there is INTEL_fragment_shader_ordering and NV_fragment_shader_interlock (on maxwell). We just need to get AMD and mesa to also implement such an extension :D


before overwriting previous optimization  please talk..
The problem was that previous code never had exit from the loop (the same problem comes back with reverted change)!

The method: Collections.min(Arrays.asList(offset+limit, uris.size()))):new ArrayList<String>()
does not loop through the list of URIs, but through provided list of integers, to calculate the size.
See http://stackoverflow.com/questions/1484347/finding-the-max-min-value-in-an-array-of-primitives-using-java for more details.


If you'd like to know the number of pages which are already generated,
`list.metrics.pages.length` is enough.
Is there any reason why you use `getOwnPropertyNames()`?


I have copies of both if you want them. Also I realized something The Animal Companion Sheet is good and all but their are 2 things that are missing for it "besides fixing skills".
1) place for choosing feats gained through Advancement.
2) place for buffs spells.


There's a findbugs build failure:
```
M B Eq: org.jruby.truffle.runtime.core.RubyMatchData$Pair defines compareTo(RubyMatchData$Pair) and uses Object.equals()  At RubyMatchData.java:[line 176]
```


Both are in the lookup table,  CRC-16 is longer and not first in the lookup
table.  And it works...  Haven't looked further.
On Jun 11, 2015 8:07 AM, "Iceman" <notifications@github.com> wrote:

> aha, now I see. True, it said nil in the script. So by making the name
> long it works?!? Very strange.
> LUA is one-index based.. Could have something to do with that...
> scripting.c... ( i-1 )... error..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marshmellow42/proxmark3/commit/5afe9838eb6c911ab136acdca101830e42b069fb#commitcomment-11632294>
> .
>


Это лишнее. В конце любого js/php файла должна быть ровно одна пустая строка.


No reason. Copy&Paste from somewhere if I had to guess :-)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.265](http://teamcity.monogame.net/viewLog.html?buildId=8224&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:25:31


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.266](http://teamcity.monogame.net/viewLog.html?buildId=8228&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


왜 이것만 let!으로 한거야?


I also have Mountforge if you want that one as well.

On Thu, Jun 11, 2015 at 8:00 AM, TwistedFaith84 <notifications@github.com>
wrote:

> I have copies of both if you want them. Also I realized something The
> Animal Companion Sheet is good and all but their are 2 things that are
> missing for it "besides fixing skills".
> 1) place for choosing feats gained through Advancement.
> 2) place for buffs spells.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Heliomance/HeroForge-Anew/commit/63d148b69bf9b9ce61dd24df1d0795e8269bfdd3#commitcomment-11633194>
> .
>


Works for me.


Why does git think these files are binary? Are you using some funky file encoding or something?


no because then I need the rds database


Actually, IIRC, this works:

``self.logger.info('{} {}', x, y)`` ... at least I am fairly sure I tried that lately :-)


first write


主要研究 JBoss Drools


fixed in f1e95ef


you need to translate the script comment in Chinese
and the README.md in Spanish

otherwise, nice addition to the script


Yuck


...shouldn't it be for source *of*?


As far as spellforge goes I think torrasque666 is supposed to be updating it. I was going to help him but I cant cause of family and life problems that very recently came up. So I'm not sure if he is still going to work on it or not.


let! 이 테스트 전에 미리 실행해두는거고 ! 없으면 lazy하게 호출될 때까지 뻐기고 있는데, 밑에 comments_count 개수 체크할때 미리 실행되지 않고 comment생성하는 변화량까지 change에서 걸려버림 ㅠ


Oops! -tabs strike again...


just checked, it works — though previously, it wasn’t using it and even mixing `%s` and `{0}`.


Indeed this looks very weird. Could you open an issue and sum up your findings? when you do, ping me by mentioning my GitHub handle (starting with @).


More yuck...


즉
원래 1 -> 삭제시도 실패 -> 그대로 1 이어야 하는데
원래 0 -> 코맨트 생성 1, 삭제시도 실패 -> 1 -> 변했다 fail 이렇게 됨


음 나는 오히려 괜찮을거같아


I have added a comment about the format, but not to document it.


@Raesu DWMR. :+1: Press on!


이거 그냥 sql쓸필요 없이 user.stories + user.stories_of 이케 하면 될텐데?


Aquí le quité un import que lo hace imposible de ejecutar sin tkintertables.
También le agregué la posibilidad de ver las contraseñas guardadas y de copiar cada una de ellas al portapapeles.
La parte que crea la base de datos de cero todavía está mala.
La próxima vez escribo esto directo en el commit. Todavía estoy aprendiendo a usar Git :P

I removed one unnecessary import for tkintertables which is not used.
I also added the option to see the passwords saved in the database and to copy any of them to the clipboard.
The part where a new sqlite database is created is still broken.
Next time I'll write this directly in the commit. I know. I'm still learning git :P


This line of code leaves a space in the field name so that if you use a + in the sort operation it will always fail.
This logic works in my testing. I will get you a pull request with the fix shortly.

        sort_criteria = {field: unformat_key(sort[1..-1]).to_s}
        if sort.start_with?('-')
          sort_criteria[:direction] = :desc
        else
          sort_criteria[:direction] = :asc
        end


Try [this](https://github.com/jashkenas/coffeescript/issues/new?title=the%20way%20you%20extend%20this%20is%20BAD&body=see%20https://github.com/AtomLinter/linter-plus/commit/36c97bb1b8e088d1a136b87ae2aee03aa632cda9) see if that helps


신기하네. likes랑, followships의 destory spec에서는 왜 그런 문제가 없었지?


@cmherd DWMR. Looks good. Nice job.


Probably what I meant to type, yes, but muscle memory from just having type "for" probably kicked in. It looks fine of now, everyone knows what I was thinking for.


Hi,
I have tested it with couple of desktop and mobile browsers , for desktop I have checked it and working IE9 and above, chrome , firefox, safari. For mobile in android chrome and ios safari.

Is there any browser you are trying and not working ?


Why do you have a bunch of StyleCop files in here? One at the root should be enough.


For all you Forge's needs (Most of the Forges)
https://github.com/TwistedFaith84/DND_Gaming_Tools

For SpellForge
https://github.com/torrasque666/Spellforge-Anew


ㅇㅇ 그런데 그렇게하면 sql 쿼리 두개를 쓰잖아


I wondered where that bug was coming from -


I am not sure about this commit, isn't that WebView version still in Beta?


I had added this method to ensure the consistency of of the data. So, if somebody changes one of the attributes in the class, others also get updated based on it. I guess this is not the correct way to do that. Rather, I found a post talking about the same:
http://stackoverflow.com/a/14916491/2606411

Please confirm if this is the correct way to proceed.


OK, sorry, reverted. :fried_shrimp:


It's not evil, it's just inconvenient that I had a relatively short SQL query and then I had to spend half an hour figuring out how to express it using DBIx::Class.


go rachel!


@zinigor Could that change be merged into `master`? It would solve quite a few issues we've been having with Publicize Next (see https://github.com/Automattic/io/issues/430 )


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 540](http://build.prxm.it:8811/viewLog.html?buildId=2978&buildTypeId=Jarvis_DocumentStore_CI) is now running


그래서 잘 보면 걔네들은 미리 생생하는 부분을 before를 씀  (lazy로딩 안하는놈)

여기서는 왜 before 안썼냐면 아래 두 context에서 사용하는 comment 객체가 동일해서 코드 중복이라고 생각해서임 (likes, followship에서는 위쪽 context - 내 소유가 아닌 relationship destroy 체크 할때는 아얘 relation 생성을 안함 1:1 이라)


why do you do that especially in this commit? I liked the former version more readable. The execution of `Base.srpm_path` could change in the future giving the exact path as other `Base.*_path`.


Thanks! :+1:


One activity corresponds to one node in the execution graph


Where does this connect to? Will this test run in AppVeyor?


I think the same can be said with squashString function, but I coded with intent on re-using those 2 methods elsewhere within the module. I thought it was a standard thing to do. I dont use it much now so yeah you suggestion makes sense.  Do you think the 'var function name = function definition ' approach is a little verbose?


I wander if calling .replace complicates the transformation step, and thus enabled the whole expression to be confined in a function? I know at some point there was a bit more logic in there as we shifted ideas about string uniformity. I wanted to be able to isolate and test too.


Apologies, I tend to over explain.  I should just say 'url.startsWith(string)' since that's what the regex does. The definition here can actually be a bit more explicit by placing the regex within the function parameters and by renaming it to trimTransform or something that bares > meaning in what it does.


```ruby
entries.any?
```
sounds good!


I've been calling it an associative array for the longest time.  Thanks you just made my day. =)


It is stored in the workunit.  Workunits are currently stored in Dali, but soon also in cassandra


I apologize I missed your mention of this aspect.  And thanks for the catch. I'll repair this.


nit: could use `creating` instead, as what we are doing is to skip the `if (s.state & PG_STATE_CREATING)` when `creating` is true, easier to understand than ... when `nocreating` is false.


what is it?


Should this be configurable from the outside?


Thanks, mapping up front is faster and more readable.

I Needed to avoid mpromise ( trying to stick to one Promises dialect ), and I didnt want to continue with callbacks.
I guess I could promisify this function like findDuplicate.. but then I'd be dealing with callbacks again.  Trying to get out of that habit.


I"ll stop using underscores in JS. lol  nonstard.


comm


Thats my reflex of not wanting to hit the DB so much kicking in.  I, we and I think many on the back-end avoid DB lookups at all costs, and this way allows me just to query once ( if I couldn't solve the problem in a small, concise db query ) .  I hope you understand where I am coming from.  Its not usual to loop database reads - you still have to make a round trip to the network even if data is already in memory. We've already been able to lean over a firebase server, so imagine our mongodb cluster :violin:


`mNodes` is not *really* optional. All the code paths assume a valid `mNodes`. What we have now is a way to easily reconstruct plus `assert` for valid in debug mode. Those are wins, but I'm glad @nbougalis has ideas to remove this member.


![Uploading Youre_awesome_gif.gif…]()


I was attempting to hint the arity of a callback func (sometimes fn(e), sometimes fn(), sometimes fn(e,o) ).  The standard node callback contract should always apply when using callbacks so all thats moot now, Im way past it.


Pas nécessaire


It's an OR condition on equality,  or did I get my logic wrong? I intend on it finding at least one of those conditions in the whole collection. This way, the user will have to correct and solve for the next condition if there is one.


Yes, there's your ```puts match.to_s``` in the tutorial.


because it's normalized that way? I err on the side of not trusting what comes out of the username field and since _un needs to be escaped when doing match which might include escape characters?  Its just to remain consistent in escaping?  A little OCD, yes.


This paragraph is unclear. Looks like // 'shared' build.gradle should come before the plugin... part. Please make this more clear.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 540](http://build.prxm.it:8811/viewLog.html?buildId=2978&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 198 Build time: 00:04:34


AHahah ^^


That is right, it is only used in one place but it also uses the private class method `TD_Database:appendExtensionToName:`. This method does not do much (it simply appends the extension to the filename) but I prefer *all this knowledge* inside this category.


Hi Kostis,

this change looks good to me. I will merge it with the upstream of oftc/dsa-nagios.git though, which is http://anonscm.debian.org/cgit/mirror/dsa-nagios.git/ though.

Cheers,
Martin


Can you check there if `if key == "file": ...` and maintain the autoadding Command feature?


You could probably gain some efficiency here by copying ranges at a time... that said it might be an over optimization as this code is probably a lot easier to read than the alternative chunking code.


```
src/Control/Biegunka/Biegunka.hs:125:3:
    Illegal constraint: c (Const (Traversed () GHC.Conc.Sync.STM))
      (Use ConstraintKinds to permit this)
    When checking that ‘go’ has the inferred type
      go :: forall t l (c :: (* -> *) -> Constraint).
            (c (Const (Traversed () GHC.Conc.Sync.STM)),
             Logger.HasLogger c l) =>
            l -> t -> IO ExitCode -> IO ExitCode
    In an equation for ‘confirm’:
        confirm
          = I go
          where
              go c _ ks
                = catchJust
                    (guard . IO.isEOFError)
                    (do { k <- prompt "Proceed? [Y/n] ";
                          k })
                    (\ _ -> do { ... })
                where
                    prompt message = fix $ ...
```


:+1: the best


This should be there. This is correct.


That's an ancestor of this commit, though?


Why `copy`? :) It sets it back to person.standing_in afterwards, so I was more meaning is there a reason you can't just update person.standing_in directly.


LGTM


Oh, yes, it was totally broken. It's just that it's better to use it that way because then the format() can't fail.


The problem is you're using a CoffeeScript class in a way that it wasn't designed. You're supposed to call `new BottomTabProject`.

`this.__super__` is stored on `BottomTabProject` but you're bypassing it and just plucking out the `.prototype`.


I like the simplicity of package_builder in this patch. Should spec.py change be part of this commit?


Thanks for sending this along, and I'm very sorry to have been on hiatus for so long. I'm going through outstanding pull requests, finally, and this looks good. I'm accepting it.

Thanks again!


Pretty slick


Sigh, GitHub listing them in the wrong order on the PR page...


@KsyMC 와 개부럽다 ㅋㅋㅋ 저희는 인문 사립이라 야자 무조건 하게 되어있는데 ㅜㅜ


On 11 June 2015 at 09:58, Jérôme Forissier <notifications@github.com> wrote:
>
> Because unless I'm mistaken, ctr_encrypt() (in core/lib/libtomcrypt/src/modes/ctr/ctr_encrypt.c) handles the first block by itself, i.e. it will not call accel_ctr_encrypt() (this function) for the first block.

Well, i suppose your validation testing would have caught any
problems. It is just a bit surprising that tomcrypt does not use the
accelerated routines for the first block.


but its probably better to use Math.min which i will change right now.


Yeah, I always forget that you can do that when I've been doing Java for a few months.


Likely.


No need to add class for history div. Please use theme CSS classes.


For what we need this? Are we using virtualenv anywhere?


Suggestion: GenerateGetCall() or something


I tested this script on a local environment with ~180K rows, and it took under 10s, so I guess that the same conversion on a production environment should be quite painless.


Doesn't really matter because we use -noblit, -nodraw, and run with the dummy SDL video driver.


Fix is done by alihalabyah


It was a debug test that I left in just to show usage (it's commented out at the attribute...).


ack


Probably should have it in the constructor as optional params...


Комментарии следует оформлять в стиле [JSDoc](http://usejsdoc.org/) на **русском** языке.
Комментарий к функции должен сразу отвечать на вопрос *Что делает?*. Например:
```
/**
 * Возвращает время начала следующего события.
 *
 * @param {Number} column Номер текущего столбца.
 * @param {Number} row    Номер текущей строки.
 *
 * @return {Date|Boolean} Время начала следующего события,
 *                        либо false, если следующее событие отсутствует.
 */
```


You did it again :D


Add `(fix #124)` to commit message, please


@brayhoward You committed to master.


Yeah i had that originally but it was just gnarley to look at...


This explanation doesn't quite make sense to me. The goal of object safety is to ensure that the object type `Foo` can implement the trait `Foo`. The reason that `Self: Sized` is problematic is because the type `Foo` is unsized, and hence if `Foo: Sized` then `Foo: Foo` is not possible. That said, I'm not sure how best to phrase this explanation for this document.


My first impression on the RunScriptSessioned was that it was going to Begin/End session internally. Code might read better as RunScriptUnderSession or encapsulate a Session object and then do session.RunScript(...). Incidentally 'Object' seems redundant in sessionObject.

Assuming you choose to encapsulate a 'session', it might take advantage of the Dispose semantics using (var session = new Session(runspace)) { ... }. Now end is implicit and the nice thing is Code Analysis can detect if someone forgets to close a session.

Alternatively, a functional variant might work that encapsulates the begin/end and calls back for the user to perform their actions within a session.


@kciredor what is this +1 for ?


O O


Why eradicate?


오 글쿤. 그리고 return type이 array라 where같은 activerecord함수도 못쓰네 ㅠ 내가 레알못이었네...


A哥nb


Your comment was confusing, can you create a workable version of the example posted on the issue.


Dont know figured it was a way of hiding them on githubs side...


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.266](http://teamcity.monogame.net/viewLog.html?buildId=8228&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 499, ignored: 6 Build time: 00:26:35


In general, I wonder if it's worth covering some of the reasons these restrictions exist.

For example, the restriction against using the `Self` parameter is because, in the case of an object type, the `Self` type is unknown (that is the whole point of objects).

Similarly, the restriction against methods having generic type parameters is because we need to generate a vtable, and we could not do so without knowing the values of those type parameters in advance.

I'm not sure how much detail is warranted, but I have found restrictions feel less onerous if you understand the motivation behind them better.


The reality, PCSX2 doesn't run on mobile stuff. I drop openGL ES because it was useless (vulkan).

On desktop side, Nvidia is only limited to 1% of the market (my recent GPU doesn't support it). And Intel 0% because Win driver sucks and Mesa doesn't support it. As you said AMD is 0% too... So yes, I know about those extensions but it is a long term future. Tons of issues can be fixed without those extensions anyway so it isn't blocking.


Almost, but not quite. What you want is to match consecutive characters that compose a symbol and then only change a symbol that is in the list, so for example `<<-` should not be partially changed into `<` plus Unicode for `<-`.


lol... thought you valued this rule


Yes, you are right, the filename is UNIQUE NOT NULL.

As you already saw, I created an index for the filename [here](https://github.com/cloudant/CDTDatastore/commit/70e1bbb1fe5cfa0f27253691fdc253f3da5e7d98#diff-64cf4ed96700d9bd22b060e7ef6ee938R40).

About the PRIMARY KEY, after checking the documentation, I have found that a *UNIQUE and PRIMARY KEY constraints are implemented by creating a unique index in the database* (click [here](http://www.sqlite.org/lang_createtable.html) & read title **SQL Data Constraints**, also [question 7](https://www.sqlite.org/faq.html#q7) in FAQ).

I'm going to define both columns as `UNIQUE NOT NULL` and remove the code to create the index, it is not neccessary.


http://stackoverflow.com/questions/9540801/combine-two-activerecordrelation-objects
이런 이야기를 죽 따라가다보니
https://github.com/oelmekki/activerecord_any_of
이런 젬이나 (원래 rails pull request였다가 gem으로 빠짐 https://github.com/rails/rails/pull/10891)
http://stackoverflow.com/questions/7976358/activerecord-arel-or-condition
Arel (ruby용 relation algbra 구현) 까지 나오네


Really not! I'll remove this uppercase.


This idea looks a bit too smart for my taste. It is worth to be more explicit in tests about what is and what is not tested as in case of failure error messages are incomprehensible.

Also we would like to test mixture of unicode and non-unicode, if you replace everything then it is not testing a mixture.


Supporting the demand for a release, it would help our team as well. Meaning: thanks ;)


Now the connection string can be loaded from configuration instead of specified on the command line


Yeah i've got a task for that in asana, i've got two or three things that I need to make recipes also, just time...


Hm, it looks as if one could do this with:

```diff
diff --git a/mysite/settings.py b/mysite/settings.py
index 36fa92c..34cf348 100644
--- a/mysite/settings.py
+++ b/mysite/settings.py
@@ -15,6 +15,9 @@ import os
 import re
 import sys
 import yaml
+
+from debug_toolbar.settings import PANELS_DEFAULTS
+
 BASE_DIR = os.path.dirname(os.path.dirname(__file__))

 from .helpers import mkdir_p
@@ -173,20 +176,8 @@ WSGI_APPLICATION = 'mysite.wsgi.application'

 DEBUG_TOOLBAR_PATCH_SETTINGS = False

-DEBUG_TOOLBAR_PANELS = [
-    'debug_toolbar.panels.versions.VersionsPanel',
-    'debug_toolbar.panels.timer.TimerPanel',
-    'debug_toolbar.panels.settings.SettingsPanel',
-    'debug_toolbar.panels.headers.HeadersPanel',
-    'debug_toolbar.panels.request.RequestPanel',
-    'debug_toolbar.panels.sql.SQLPanel',
-    'debug_toolbar.panels.staticfiles.StaticFilesPanel',
-    'debug_toolbar.panels.templates.TemplatesPanel',
-    'debug_toolbar.panels.cache.CachePanel',
-    'debug_toolbar.panels.signals.SignalsPanel',
-    'debug_toolbar.panels.logging.LoggingPanel',
-    'debug_toolbar.panels.redirects.RedirectsPanel',
-    'template_timings_panel.panels.TemplateTimings.TemplateTimings',
+DEBUG_TOOLBAR_PANELS = PANELS_DEFAULTS + [
+    'template_timings_panel.panels.TemplateTimings.TemplateTimings'
 ]

 INTERNAL_IPS = ['127.0.0.1']
```

... but `debug_toolbar.settings` does a `from django.conf import settings` and you get an error about `SECRET_KEY` not being set. I'm not sure if there's an easy way around that.


i dont do it this way anymore...


This was just the initial implementation it's all an object collection now.


Whoops, well spotted.


I like the simplification of projects here


Looks good.


@chelmuth: please apply above commit to staging


Yes, I'm going to clean up this branch and get people to review it ASAP.


I only caught this one because I was running everything in valgrind trying to find the double free in volume_key


Dude i was telling suraj the other day that since we're removed it i find myself using the available ordering ALL the time.  was definitely a good call to take it out!


前几次提交的comment有误，对应的章节应该是58.1、58.2等，由于参考了最新的文档，不慎写错，在此声明。


How can this be tested?


Formerly package, but I think that protected make more sense ATM.


Good idea, probably won't get to that for a bit but I see what you mean about the name being misleading.  It was created to not have to create a new session to send the file 'nibbles' for performance...


Thank you very much @thomashug


remove white spaces


Вызовы `console.log()` нужно удалять пред коммитом.


Строк и столбцов в расписании может быть любое число, не обязательно 4 строки и 5 столбцов.


@kciredor this has already been released...


Yeah, it's the right thing to do here. GHC 7.10 has become stricter and it asks to enable the extensions needed by the inferred type signatures. (The constraint is a bit crazy, but that's because I haven't yet figured out how to be polymorphic in the choice of optic (`Lens`/`Traversal`) and not to have gross type signatures at the same time.)


`.to_sentence` usually


oh, my bad, it hasn't yet. Will see if I can find some time this week end for it.


Overkill repeating Naos yet again this filename.


New column names


Лучше явно возвращать `firstDate.valueOf()`, иначе это больше похоже на опечатку.


Please forward your contributions robrichards/xmlseclibs


939px: ![screen shot 2015-06-11 at 14 38 29](https://cloud.githubusercontent.com/assets/282788/8108307/8bdd2272-1047-11e5-89f3-ed1ce48ca35d.png)


If you use `emplace` you can lose the `make_pair` (here and below)


 Do you want to stand out in your job interview? Do you want to learn how to handle job interview questions? Do you want to look confident during interview? Do you want to crack your interview?  I would like to draw your attention towards one of the very important points. If you want to look confident, ask as many as questions you can.
Most of the people are afraid of asking questions. Why? Most commonly there are 2 reasons:-

    People think “what interviewer will think if I ask this question?” This though holds people back from asking questions.
    Mostly people don’t know what to ask.

Let me solve these 2 issues for you.

When you ask questions you will send very positive signals to interviewer. Your questions will not be considered negatively if you know what to ask. In fact, you will be considered as a person who is

     Curious to learn
    Interested in the company
     Focused
    Wants to grow
     Competitive
     And so on...

If you are NOT asking questions, you are missing on above mentioned qualities. Is that you want? I guess no. In your next interview, go ahead and ask questions boldly.

What kind of questions should you ask? You can ask questions to know more about

    Company
    Products & services
    Major competitors
    SWOT (strengths, weaknesses, Opportunities and threats)
    Your role and major responsibilities
     Major challenges of your job profile

Likewise, you can ask many other questions. Because of time and space constraint I can’t explain it further. However; you can visit www.rideforpride.in to learn more about interview. You can also use “mention your promo code’ as promo code to get attractive discount on paid services. You can also visit www.youtube.com/InterviewRockers to see videos on handling interview. I wish you all the best.


AFAIK there's no need to add those resolvers there.


Cheers


let's take it as is for now and I'll raise an issue to review master password file handing


Are you asking for the [sample that I compiled](http://coffeescript.org/#try:class%20BottomTab%0A%0Aclass%20BottomTabProject%20extends%20BottomTab%0A%20%20initialize%3A%20(%40linter)%20-%3E%0A%20%20%20%20super(%22Current%20File%22)%0A) to see how it works?


I think if it gets killed during the install... all bets are off. We would not be able to `postResult()` back to the calling app, so it does not matter.


Cuando intento instalar el proyecto mediante `npm install` y ejecuto `grunt` me elimina todas las carpetas.
Si usas sass puedes evitar repetir la estructura de `.menu`, `.menu ul` y `.menu ul li` por un bloque de código anidado:
````
.menu {
   ul {
      ...
      li {
        ...
      }
   }
}
````


Значение `callFuncNext = 0`, заданное по умолчанию, никогда не будет использовано, так как ниже оно переопределяется `callFuncNext = nextTime - now;`


good catch


I added them because I couldn't resolve s3://snapshots.era7.com/ohnosequences/aws-scala-tools_2.11/0.13.0-SNAPSHOT/aws-scala-tools_2.11-0.13.0-SNAPSHOT.pom


Takie cos było ale zmieniłem na tym branchu obydwie tabelki zeby zamiast ajaxa były updatowane przy pomocy notyfikacji SSE. musiałem wiec przenieść kod ktory był ale dzialal tylko dla datatable.

Generalnie chce doprowadzić do sytuacji kiedy z widoku monitorowania eksperymentu nie bedzie żadnych periodycznych zapytań a wszystko bedzie szło z serwera.

Do tego brancha zmergowalem jeszcze zmiany dot. komend SSH o ktorych kiedyś pisałem. Niestety nie mogłem zmergowac tak powstałego kodu do brancha development bo tam juz sa jakies zmiany w shellcommands ktore tez zmieniałem a ktore docelowo chce wywalić.

Jak skończę z notyfikacjam to dam Ci znac i będziesz musiał recznie rozwiązać konflikty.

Pozdrawiam,

> On 11 Jun 2015, at 02:02, Jakub Liput <notifications@github.com> wrote:
>
> Cześć, czy mógłbyś napisać na czym polega ta zmiana? Po opisie widzę, że chodzi o wyświetlanie simulation runów z tabeli progress - a już takie coś mamy.
>
> —
> Reply to this email directly or view it on GitHub.
>


Did it work?


Этот TODO-комментарий нужно удалить.


You are leaking the configuration sname here? The semantics of "add" with -s isn't clear.


Deberías de usar un clearfix más completo como el de (Nicholas Gallaguer)[http://nicolasgallagher.com/micro-clearfix-hack/] ya que te cubre muchos más casos:


Yes, thanks.


This is localhost from the perspective of the remote machine - correct?


space after `:`


I think this change fixed a problem i didn’t get around to reporting.
In night mode there appeared some odd gaps between some of the answer (ease) buttons.
The gaps are gone now, so thanks.


Очень много глобальных переменных (**15 штук**). Нужно постараться свести их количество к минимуму.


`Constants.CURRENT_CATROBAT_LANGUAGE_VERSION` is set to `0.95f`, now you set the `CCLV` to `0.95` twice.


Question: should we but that in the general `Utilities.jl` like

`asnull(T, d, key) = Nullable{T}(convert(T, d[key]))`


Yes, the responsibility for error reporting was moved to the callers. When pacemaker starts up, it calls crm_ipc_connect() to verify that an instance isn't already running. So before, this would result in messages that looked like errors in the syslog, which can make sysadmins nervous. Also, with the messages here, they were always logged with info severity, when sometimes they were important errors and sometimes not. Now each caller gives a more appropriate message.


Evita las mayúsculas. Puedes utilizar `text-transform: uppercase`


Works for me without them, the whole ivy cache obliterated


Yeah, sorry, fixing all the `<title>`s is another known todo


Опечатка?


The equivalent feature in MySQL is [online DDL](https://dev.mysql.com/doc/refman/5.6/en/innodb-create-index-overview.html).  It will work automatically in 5.6/InnoDB, and you can use the optional syntax `LOCK=NONE` to assert that the operation will be online.


Need to require ThemeManager up here as well.


Такие конструкции следует оформлять так:
```
if (breforeMarked) {
    beforeMarked.setAttribute('class', '');
}
```
После `if` должен быть один пробел.


Let the callback handle the error, which means that if we have an error then the call should be `cb(err, null)`


Un elemento de bloque si no está flotado *siempre* va a ser `width: 100%` por defecto


Fixed in 304c4a9


I sent some bitcoins to my electrym wallet they arrived but then I sent them to a wallet on alphabay they have not showed up yet. It has been about 18 hours since I sent then any suggestions would be great.


No, I think you'd need to avoid the `class` keyword and build the inheritance manually or switch this to ES6


Remove one of these, its redundant.


 - move scheduling config into config file
 - move your email address into config :)
 - make location of schedule.db configurable
 - put SQLite into requirements.txt
 - could the dimension2 stuff be refactored into a config option? The schedule could have filter = "ga:dimension2 =@ 'e3'" or something?


We were on Travis, but now I am slowly removing all of this, do not worry
I'll get there.
On Jun 11, 2015 14:23, "Vladimir Sapronov" <notifications@github.com> wrote:

> For what we need this? Are we using virtualenv anywhere?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/syncloud/platform/commit/c0d27b6f77d303c9c62fa56b2742caefcef1aef8#commitcomment-11633673>
> .
>


I meant electrum wallet


I'm not sure what you mean...
If you run a command using `//command`, the first `/` is removed by bukkit and `/command` remains as the command/message. Is that what's confusing you?


Or do you mean `//up` is never matched because the command is just `/up`?


This line can be simplified to:
``` java
if (Looper.myLooper() != Looper.getMainLooper())
```


Do you have other container-like objects such that you need to clarify that this is a 'Details' element? Container seems sufficient to me. Then you also get to rename ContainerId to Id.


```
var dateColumn = headerInfo[i];
dateColumn = dateColumn.innerHTML;
dateColumn = dateColumn.split('.');
```
можно заменить на:
```
var dateColumn = headerInfo[i].innerHTML.split('.');
```
Читаемость кода от этого не пострадает.


Yes - I read this https://groups.google.com/forum/embed/?place=forum/orient-database&theme=default&fragments=true&showsearch=true&showtabs=true&hideforumtitle=true&hidesubject=true&parenturl=http%3A%2F%2Forientdb.com%2Factive-user-community%2F#!topic/orient-database/dzL1FShbUaI

I meant that System.exit(1) is too serious punishment:)

In any case, what to do if there is a structure of DBs? A over B over C? And more over: it seems that OrientDB partitioned pool doesn't support autoregistering a db.


What is the difference between a Location and a ContainerLocation? Perhaps that'll clarify for me what a 'ContainerDetails' is. Or maybe there should be an additional nested Container object with Id/Location properties.


Для таких задач следует использовать функцию `parseInt()`


Nope!


В файле много лишних пустых строк, их нужно убрать.


saw approval from bkerley
at https://github.com/holidayworking/riak-ruby-client/commit/c0d5b80e77d9d12ab5d837e5e715b153179fbcef


merging holidayworking/riak-ruby-client/add_support_yokozuna_result_as_inputs_for_mr_jobs = c0d5b80e into borshop-integration-232-add_support_yokozuna_result_as_inputs_for_mr_jobs


holidayworking/riak-ruby-client/add_support_yokozuna_result_as_inputs_for_mr_jobs = c0d5b80e merged ok, testing candidate = a4231be8


Hi,
I was the main translator of Ethercalc in french, 3 years ago, for the Framacalc instance. Yes, it's ok to publish the translation under Artistic License 2.0.
Thanks for your great work!
Martin


Good. I do it at once in my branch.


I debated that with myself, but thought it best to make as few assumptions as possible about what would be done with the output data.  Especially since the relevant `GVariant` format strings are public symbols.  A caller might just want to dump the `GBytes` to disk, for example.

But I could be persuaded...


Hmm right, for some reason I was thinking it was > 0.

-[Unknown]


It seems redundant for callers to write myPackageManager.GetFileContentsFromPackage(myPackage, ...) - the 'FromPackage' is implicit in the containing type as well as in the package parameter you pass.


@mcamara Thanks :)


Glad I could help that much! It's a tricky problem to solve!


comment why only linux for now


@poettering How can I do this using git?


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (2187 of 2213 tests passed)

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


detail errors should be lower case, like all Errorfs.  Open a follow up issue to do this post 3.0


Ditto:
myPackageManager.DownloadPackage(myPackage...)

This reads nicer IMO:
myPackageManager.Download(myPackage...)


Hello @GeekOnAcid

I haven't work with this project or the Strava API for months so I'm not sure if it has changed but as far as I remember the get_leaderboard uses the get_pages function, which when used with "All=TRUE" calculates the total number of pages to request (total number of items/200) and checks if you have enough API request left to do so. If you do, it starts a loop for retrieving each page one after the other.

Later when I have some free time I'll play with it to see if it works as It should or if there's something wrong. Could you send me more info on how were you using the function and at what segment to see if I could replicate the problem?


I'm guessing these two readers should be using the "using" pattern to be properly disposed?


GetInstancesWithPackages?


Opps forgot to add a comment. Moving to a frameless window so we can remove the maximise button.


Revert it and commit again?


Is this both for the backbone and the clade markers?


Yup, and a bunch of custom reports.  I spoke with Sheel about it, and we agree that it makes sense to change all case reports in the same way.


Hmm, good point.  Not *super* critical for this command, but I suppose we should anyway.


Noted comments.

Math.random() is uniformly distributed and normal therefore crypto is not necessary for the game.

Will look into a cleaner single reply.


Yes, we use the same subroutine for this.


if so, return false


, return false


Shouldn't that be `>= 4000`? That looks suspicious given the ranges used below...


check if


Get all the events related to


Follow up: "expected directory %q permissions to be %s, but was %s"


Perhaps when I read the implementation this name will make more sense, but it throws me a bit for a loop as to what it is supposed to do on first read. You comment seems more informative - perhaps RemoveInstance or StopTrackingInstance will convey its purpose more clearly.


Add a comment in here (I think you can add it to the end of line)


Pretty sure I've tested this now and the answer is yes.  Not sure if there's a real test though, I feel like we ought to have tests for more of these assumptions (even though they're reasonable.)

-[Unknown]


I actually think that this was wrong before as `journal_class_name(type)` returns "[TYPE]Journal" whereas "[TYPE]" makes way more sense.


Add a comment in here about what this is doing


Add a comment


according to the direction


Maybe it also makes sense to have current action virtualenv_path here too?

You remember that ansible case with hardcoded `/opt/stackstorm/virtualenvs/ansible/bin`:
https://github.com/StackStorm/st2contrib/pull/210/files#r31071877


I wouldn't add `.viminfo` to the repository. It might contain sensitive information.


current one


Dude, WTF? :-)


Do I have to wait?


goodgoodgood


Yes it should be.


We're both still learning all the internals of cmake... Otherwise I would have done this completely myself when I started working on this file...


"Details" is a bit of a noise word to me - is 'InstanceConfiguration' a more accurate description of the object in question here?


is it ever used?


The sample app does a check for `![error.domain isEqualToString:PTPusherErrorDomain]` to see if it should try to reconnect, should that be changed to `PTPusherFatalErrorDomain`? Or should it perhaps give up when it's either of these two?


@knocte - Yes, that is a different problem. I was not looking at that :)


TrackInstance?


OK I think that's ok 😊


This particular call?  No, this gets a list of reporting and case_sharing groups to add to the filter.  This behavior is in the old version as well, I just moved it over here so the normal version only gets `"reporting"` groups, and this one gets both `"reporting"` and `"case_sharing"` groups.  (Prior to this change that happened via a flag, which was a bit confusing)


test


UpdateTracking?


Thanks! 😀


On top of that now it only copies the actual password to the clipboard instead of everything.


IIRC, it retains TType because the generator know how to deal with them,
and there are some important TTypes - like Null, Ref, etc

On Thu, Jun 11, 2015, 04:53 Simon Krajewski <notifications@github.com>
wrote:

> @waneck <https://github.com/waneck>: This strikes me as weird: The
> function skips (non-core-type) TAbstract but retains TType. That can't be
> right, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Simn/haxe/commit/5ffda1ded11d4ffabe8cbe442861410951b13aa7#commitcomment-11628438>
> .
>


@coryvirok thank you for this, it's awesome, very nice work.

Two questions:

* Why exactly we need to change to `uglifyjs` in the Makefile?
* Do you think we should open a PR for the official repo?


I prefer to do all development in a separated branch


*bump* how thoroughly has this been tested under the circumstances outlined by @kinglozzer ?


@isolver I think this commit fixes the memory leak that people have been finding with MovieStim2

The issue was that there was a circular reference with the vlc audio stream (moviestim stores a reference to it and it stores a reference to MovieStim's EOS callbacks). I now pass a separate function as a callback and a `weakref` to the original MovieStim2 instance that the callback can then use to signal the EOS


This pull makes sense.  It keeps 1016 working like it was written (changing the documentation to match what was there), and adds a new format.

Was I the one that made 1016?


1). How is it faster?

2). Where is the distance estimator now? (I know it's global, but where is it defined?)


MigrationVersion would be a more precise name.


@Notsosof1 DWMR here. :+1:


done


Да, спасибо за наводку.


Nope, you did this one, lol

https://github.com/kholia/JohnTheRipper/commit/08e578ad2841e7e03794df56f6849888c9129f70


@derickr I would stand for more compatibility, thus for reverting this. Not disputing the correctness of the change itself - obviously it gets PHP closer to having clean APIs. However in this case, the massive breakage of the user code nullifies the benifit, or even negates it.

Plus, by @nikic suggestion, this same can be done without breakage with 7.1. This means obviously just a delay, but not a rejection.

So I'd suggest to revert this before the alpha2 /cc @Tyrael @KalleZ

Thanks.


On Thursday June 11 2015 07:13:27 Mika Lindqvist wrote:
>We're both still learning all the internals of cmake... Otherwise I would have done this completely myself when I started working on this file...

Apparently :)

I'm writing up something that guesses the appropriate compiler flags to add for the arch-specific tests, and that also lets the user override them.


I would modify the mesage, saying something like "Invalid signature request message, send by me"


@baspeters Not sure why this would be necessary. Would you ever have a different Schema / Prefix / Charset / Collation in different environments?


I just added it. Could you please check if it is ok now?


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/356


Controllers are conventionally named with the plural version of the resource in the name. See `PostsController`.


Oh, sorry that was a refuse. Already fixed in repository. About auto registering, could you elaborate it more?


Considering we're already [storing the FB ID](https://github.com/zanematthew/zm-ajax-login-register/blob/master/controllers/abstract.php#L370), is there a reason to store this again?  (I know I should of prefixed the original with an _, but that was done some time ago :0)


Aye; I've wanted somewhere to start adding this stuff and I think the lab idea makes the most sense at the moment.   If there are some protocol specs for something common these type of apps would use for networking it would be good to support it.  I haven't captured but a few weak NOAA passes myself so far but would like to build a QFH antenna this summer to try to get something usable.

As I said just a minimal working code that decodes something should be enough to get started and I can integrate it into the 'lab' format and we can keep tinkering from there -- it should be more suitable for these type of experiments.   I can see a few possibilities for classifying the signal using spectrum and constellation plus some of the gain / symbol sync tools in liquid to match it up so that should be doable too.

As for the output I think having an symbol log/display in the UI with a few formats it can copy/save to as well as the block output devices from Issue #73 will be good for most use cases.


This would also be plural conventionally, see the route declared for `:posts`.


You still have to keep `5.5.9`. Like this;

```php
- 5.5.9
- 5.5
- 5.6
```


Is that because we run the testthatQuantity code 3 times for every call to `source`?

In that case, can we just run the thatthatQuantity code once for every call to `source`? That way, we could get one timing for each test_that block, and one timing for the entire test file at the same time.

And then to get multiple timings you could just call `source` multiple times.


Yeah, good call. I just saw the webview blog post. I'll revert


Wouldn't an instance have a 'DriveType' and the 'Default' would be defined elsewhere and simply be applied?


nevermind, you already reverted!


I think it's enough to simply `import Data.Foldable (Foldable(foldMap))` here.


*Prashant


I'm having second thoughts about this one, actually. iscsi-initiator-utils just split their python libraries into separate packages, so python3-iscsi-initiator-utils should be the anaconda dependency, and iscsi-initiator-utils added by lorax for the actual utilities. Oh well, that'll be another patch.


list_box's margins in gtk3 as in gtk2


can you explain why is this line needed?


Why are these methods not in the Database object?


@sachinchitta I deleted this file from the tutorials directory because an improved version of this very hacky node is in industrial_calibration/intrinsic_cal/src/create_ical_scenes.cpp. The newer version has a bunch more parameters, but generally still requires one to edit the code and recompile to output exactly what one wants for intrinsic calibration. If you have any questions about any of this code, feel free to call or email me directly.


Are you configuring something with this object? Or is it 'Info' about something that already exists?


because we just check qpid's version on linux


I just rebased my testing branch so it isn't so much behind the current development branch... It's still full of experimental stuff, but now it's easier to cherry-pick new commits to test...


This ought to be `question_id` (similar to show `post_id` is a column in the `comments` table)


This ought to be `question_id` too.


Why exactly do you think it should be reverted?

Also about the crashes, are you saying that it might not be caused
something we do?


Why did you change the ionization energy.
`E_i = 13.598 eV = 2.17896e-18 J`


ack


Hehe this was not supposed to be a "Macism" :laughing:

I simply considered the fact that every player widget has a minimize button but not always a close button, in which case the "columns" would be misarranged, i.e. minimize buttons might not be on top of each other.

But I see your point: Users expect the close button to be in the top right border :+1:


Seems like this should be an 'InvalidConfigurationException'


please add a newline here


All I have to say is: Ask @sfmskywalker ... :)


I'd expect a 'DeploymentException' to be thrown when I try to deploy and something goes wrong. In this instance I'd expect an 'InvalidConfigurationException'.


fast-forwarding master to borshop-integration-232-add_support_yokozuna_result_as_inputs_for_mr_jobs = a4231be8


Wow! This is awesome! I'm running an OA campaign and we're going to rebuild my tables characters tonight because of this. Thanks for all your hard work!


It was originally Question then I was looking at the specs and they wanted Questions. So it changed.


@diegobeauroyre no es de buena educación meter dos cosas completamente distintas en un mismo commit

un commit debería ser "agrega imágenes para historia cap", otro ""agrega ícono de datos.gob.mx", "agrega historia de slp"


g


Nice use of a macro.


Hello, this doesn't seem to work in FF still. Every time you click, even with right click, it closes the dropdown menu.


Could you also add the `MANUAL_INCLUDE` to the `conf/default.conf` file and add some proper description about its usage


Consider also removing the `randomDrawOpts` function from the `drawUtils` module. I believe it was only used here.


I smell a copy-paste error ;)


I realize we talked about this, but maybe keeping the Optional (via Guava) would be better.


Another test I would suggest here would be to check that the predicted values (probabilities or classes) on the training data are the same from glmnet and from SAG.


Never thought of doing that before.  Good idea!


It'd be plural - `QuestionsController`, however the column name is always singular, `question_id`. If you rebuilt the database from scratch with this column name, I suspect there will be an error.


SSE4.2 code can be added here.  I did not do that yet.  NOTE, there is no code that uses this code at this time anyway.


Add setting log level to log4j for EbodacEventListener and ReportController to INFO


the problem with this is that memory consumption of large arrays is not something Ruby does very well. I'd allow delegation of this to different adapters. in CSV, for instance, you can just print every row by itself, and then discard it, letting the GC throw it out. This way, a very large collection (think 100K or millions) would need to enter all items into memory. Ruby is notoriously bad at doing that.


merge pull request


Making this (these) variable global was one thing needed to get the inline #define macros to work.  Those inlines are fast.


FIxed in ca278bc


Commit title is wrong, actually a memory leak was fixed.


I mean that with new logic, pool should call activateOnCurrentThread() every time when it returns db. What do you think?


Awesome, thank you! :rainbow:


It's not broken, advanced handling like this has just never been supported. As you know I'm working around the problem by using a bunch of multi-pass rules. But before I go and improve those rules I'd like to know if Mike's branch has new opcodes tailored for this.


@unorthodoxgeek handling it in adapters is a good idea. Could you open an issue for that, please?


Yeah doing the dapper thing here like i did on FluentUri...


You can use inheritance instead of `case..when..`. For example:
```rb
class Sentence
  def initialize(sentence)
    ...
  end

  def match?
    true
  end

  def content
  end
end

class Question < Sentence
  def match?
    words.first == "how"
  end
end

class Definition < Sentence
  def match?
    ...
  end
end

class Record < Sentence
  def match?
    ...
  end
end

class UnknownSentence < Sentence
end

class Identifier
  def initialize(sentence)
    ...
  end

  def identify_result
    sentence_types.each do |type|
      result = type.new(sentence)
      if result.match?
        return result
      end
    end
  end
end
```


好问题... 这个是个麻烦事儿... 所以他们带过了 应该解释一下


I expect the API of a serializer/deserialize to just read 'Deserialize'. DeploymentConfigurationSerializer.Deserialize(input).

If you have multiple things you need to deserialize on that type and thus have to distinguish one from the other then you might consider splitting the type so each time it is obvious and implicit.


responsible for


I don't think so because if people get just the file instead of the package then there wouldn't be any context...


Thanks, you are right. Will release with 2.30


> btw, where did you find that BossAI::EnterCombat() does the same as _EnterCombat()? I see just this https://github.com/TrinityCore/TrinityCore/blob/6.x/src/server/game/AI/ScriptedAI/ScriptedCreature.h#L360

that answers your question, BossAI::EnterCombat() calls only _EnterCombat() so it does exactly the same as directly calling _EnterCombat(), with the difference that
```
void EnterCombat()
{
    BossAI::EnterCombat();
    DoSomeCustomEnterCombatStuff();
}
```
is read by a C++ developer as
>call the implementation of the base class;
execute some more custom code;

It's common design when dealing with polymorphic classes in C++ especially where the derived class extends the base class.
You can take a look for example at how
Player::CleanupsBeforeDelete() calls
Unit::CleanupsBeforeDelete() which calls
Unit::CleanupsBeforeDelete() which calls
WorldObject::CleanupsBeforeDelete()

@Nyeriah there's no reason to keep any standard just because it already proliferated across all scripts, that shouldn't be taken into account at all. Most of scripts were (are) written by starter C++ developers who mainly copy-paste existing scripts without fully understanding what each line is doing and if that's the most elegant and maintainable way to write it. To me a BossAI:: prefix is much more clear than a _ prefix.


Please add copy constructor


I forget - why LinkedHashSet?


Hmm, any reason why this shouldn't be deleted instead?


You can use hash instead.


uhm is this the only way? Can't we make a result object where the client can fetch single information via methods? Isn't that clearer maybe?


arggghh :disappointed:


No, this is localhost on the executing machine, you can't run a WinRM command against a remote computer unless that remote computer's address or name is added to your localhost list of 'TrustedHosts'.


I think this will work


essence of a call like that is basically this.

switch to guava optional:
pros: folks who are using guava already, or are already having to convert
out of guava optional
cons: folks who use a functional language (scala or java 8) or different
functional library have to convert out of guava, typically meaning a dep or
utility code. Guava itself is a very hairy dep with many libraries still
pinned to guava 11 despite 18 being current.

leave as nullable:
pros: all functional languages and libraries can make an Option(al) out of
null, annotation processors can read and handle Nullable directly.
cons: even when using Nullable annotation, it is less explicit to users
than Optional. those who are using Optional as a function (ex. transform,
fallback etc), miss out. Many are dealing with guava anyway, and those
folks won't have the cons of a guava dep just for optional.

your call: FWIW, finagle has tons of deps already, including guava 16


mmm... interesting - you have to infer the type based on the members. I wonder if it would be useful to add a 'type' member that can be used to determine the correct strategy type.


Get this error after update sails-permisson.  Request by sails io.socket
```
Sending 500 ("Server Error") response:
 TypeError: Object #<MockIncomingMessage> has no method 'get'
    at sanitizeRequestUrl (/home/hank/git/node/backend-tracking-marketing/node_modules/sails-permissions/api/policies/AuditPolicy.js:26:15)
```


I've go the following solution from the dev:
https://github.com/driftyco/ionic/issues/3875#issuecomment-107995904


@artbear Хотел уточнить, этот код выполняется для строк с типом отличным от `ТестовыйМетод`, на этом уровне параметры отсутствуют. Может узел `properties` должен быть в узле `testcase`?


Annoying.


This looks fantastic.  Lets keep up the amazing work.


This is awesome! @zadsas too bad I can't read it!


Why the change from Throwable to Exception?


Rational: LinkedHash* doesn't cost more than Hash*, yet retains insertion
order. This helps when JREs switch iteration order, which breaks tests when
updating. It is easier to wildcard to LinkedHash than have to go back and
fix flakey builds due to oracle vs openjdk vs whatever. These bugs usually
show up in things like json tests.


I don't have great names for these.  Basically a location == AWS Region like 'us-east-1', container location == AWS Availability Zone like 'us-east-1a'.  You need both for creating an instance, I could technically infer it but that seemed more brittle...


Thought you were against these sort of project file updates.


Hmm...you mean with piecewise_construct? I'm loath to change it in this change set


[Again][1], I prefer idempotent deletes, but this approach is ok if you don't.

[1]: https://github.com/swcarpentry/amy/pull/357/files#r32225405


Please add ", ' brackets type.

Please handle:
1. {}()
2. {.......a.......}
cases


I'm not sure if this is ok, because if we're in the threadpool then we might need to switch to the iothread.


Is this correct for VS RC?


because it didn't seem sensible to wrap an Error (such as OOM) in a
RuntimeException. Looked like an accident, but yeah it is a separate issue
and I can revert.


I have InstanceConfiguration AND InstanceDetails; again not great names but there are two separate ideas being tracked.


For VS RC, this was still 8.2, wasn't it?


1 issue, can't run the script from any directory


Я вообще не помню, что это за `property` :(
Возможно, @pumbaEO знает?

Я сейчас заливаю последние правки.


indeed - I used a lot of test instances as I tried to be sure that the solution is broad enough - but I will be very soon at the point of trimming a lot the fixtures and the code test


good call on the ordering, I'll fix that up.


I think You like this variable ? :)


Hi @ptdrow
Thanks for a quick reply, really appreciate your time and also thanks for making those great functions! I'm confident I had enough API requests so that shouldn't be a problem. Here is how I used it with a specific segment:
`get_leaderboard(stoken, 772649, All=T)`

Whats more, if I use `nleaders` to define number of retrieved used and go above 200 - it crashes with `client error: (400) Bad Request`:
`get_leaderboard(stoken, 772649, nleaders=201, All=F)`


Nice job!


word missing?


Please move i inside for cycle.


Cascading deletes will also remove associated `Award`s (grepping for `ForeignKey.*Event` in `workshops/models.py`).


Wohoo!


Here I’ve got to apologize for the lack of documentation, because this is definitely the most cryptic part of the code and my intent was to push some more docs up and possibly more comments in this part of the file too. I will do that later today, and for now try to succinctly explain:

Based on the exact options you pass to validation methods in your Rails model file, the value yielded from `TRANSLATIONS` is going to result in either one or more AngularJS directives, or as in the case we’re discussing, a `Hash` of HTML attribute key/value pairs. You can see in [the `Article` model file of my demo app][1] (which my screenshots are taken from) that `minimum => 4` is given, so when `translate_rails_validation` is called, the proc for `length` is run and the resultant `{'ng-minlength' => v.options[:minimum]}` is passed to Rails’ own tag generator method. The value is set within `field_with_validations_for`, which is iterating through the attributes and pulling the validation data from actual models like `Article`, passing it to `translate_rails_validation` as the `validation` argument, where procs can access it dynamically. Ergo `v.options[:minimum]` gets turned into the HTML below:

![trust-me-im-a-code-doctor](https://cloud.githubusercontent.com/assets/2619000/8109653/bd9712cc-100c-11e5-9ff9-884515a69364.png)

As you might discern from the screenshot, the code is behaving as expected, though `minlength`/`maxlength` HTML attributes are not included as I did not witness them being used in Angular code examples (I take it that `ng-minlength` is meant to replace the HTML5 validation attr). If a `Range` value was given in the model, then you would see both min and max attrs. I haven’t checked 100% of these cases myself, which is why I admit to my latest push being ’not final’…

OK, that took a long ass time to explain, so again, I would consider this a great candidate for refactoring if code complexity is prohibitive, though I would prefer to rely on documentation efforts. I’ve been mindful of the fact we need more clarity here and will add more docs soon. By the way: none of this would ‘flow’ if not for the way Rails internal API works or how elegantly Hyper builds atop the model system. Ruby is ***so. freakin. sweet.*** Just have to say it again!

[1]: https://github.com/SteveBenner/hyper-admin-app-demo/blob/master/app/models/article.rb#L4


Nevermind, ProcessInstanceCreation doesn't take that anymore (it's in the object...).


你这不是偷了我的 idea 么！！泪目啊~~~


oh yeah another pro to guava optional: in scala or other languages
that have implicit conversions, you can implicitly convert guava
optional to something else. Not sure if anything supports implicit
conversion based on annotations or not.


we like semi colons, maybe terminate that return


'Exactly' to me implies a version match. Also, distinct implies a some sort of filter type operation whereas this is a boolean question. Perhaps this method should be something like 'AreMatchingPackageIds(a, b)'


any echo's not allowed in the boot script


Oh right… forgot about these.


Done.


@kbourgoin @emmett9001 just a heads-up: it seems that producing via `kafka-console-producer.sh` tends to push all the messages onto the same partition (not the same one every time, but all messages we send in one go in the test setup do end up together). That's not a problem unless your tests assume they're randomly assigned all over, which is how it bit me :)


this is the way Douglas Crockford likes it, but you know, our team style should be consistent, so can we make changes to this IIFE so that the parens are outside?

```
(function() {

})();
```


great work!! thanks so much for doing the generator, but can you check out the comments i made, and make the appropriate changes?

k,thx,bai


hello world


Yup, in fact for the consumer tests I had to use our producer to get a good distribution. Which sucks, because you end up running the risk of the producer breaking also breaking the consumer tests and making it much harder to debug.


is it not more possible to use RELEASE_CONFIG_DIR?


let me fix this


Why not just return false in the loop here and get rid of the tAllNumbers variable?


The opening and closing of a function (or any block) should not start/end with an empty line.


These are AWSisms. Code probably belongs in your AWS package.


@eregon There was an MRI test regression in one of the latest commits:
```

  1) Error:
TestMonitor#test_cond:
TypeError: Truffle doesn't have a case for the org.jruby.truffle.nodes.core.ConditionVariableNodesFactory$WaitNodeFactory$WaitNodeGen node with values of type  ConditionVariable(org.jruby.truffle.runtime.core.RubyBasicObject) Mutex(org.jruby.truffle.runtime.core.RubyBasicObject) NilClass(org.jruby.truffle.runtime.core.RubyBasicObject)
    /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:110:in `wait': Truffle doesn't have a case for the org.jruby.truffle.nodes.core.ConditionVariableNodesFactory$WaitNodeFactory$WaitNodeGen node with values of type  ConditionVariable(org.jruby.truffle.runtime.core.RubyBasicObject) Mutex(org.jruby.truffle.runtime.core.RubyBasicObject) NilClass(org.jruby.truffle.runtime.core.RubyBasicObject) (TypeError)
    	from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:110:in `wait'
    	from /Users/brandonfish/Documents/jruby/test/mri/monitor/test_monitor.rb:123:in `block in test_cond'
    	from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:211:in `mon_synchronize'
    	from /Users/brandonfish/Documents/jruby/test/mri/monitor/test_monitor.rb:120:in `test_cond'
    	from /Users/brandonfish/Documents/jruby/test/mri/lib/test/unit.rb:888:in `run_test'

  2) Error:
TestMonitor#test_timedwait:
TypeError: Truffle doesn't have a case for the org.jruby.truffle.nodes.core.ConditionVariableNodesFactory$WaitNodeFactory$WaitNodeGen node with values of type  ConditionVariable(org.jruby.truffle.runtime.core.RubyBasicObject) Mutex(org.jruby.truffle.runtime.core.RubyBasicObject) NilClass(org.jruby.truffle.runtime.core.RubyBasicObject)
    /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:110:in `wait': Truffle doesn't have a case for the org.jruby.truffle.nodes.core.ConditionVariableNodesFactory$WaitNodeFactory$WaitNodeGen node with values of type  ConditionVariable(org.jruby.truffle.runtime.core.RubyBasicObject) Mutex(org.jruby.truffle.runtime.core.RubyBasicObject) NilClass(org.jruby.truffle.runtime.core.RubyBasicObject) (TypeError)
    	from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:110:in `wait'
    	from /Users/brandonfish/Documents/jruby/test/mri/monitor/test_monitor.rb:165:in `block in test_timedwait'
    	from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/monitor.rb:211:in `mon_synchronize'
    	from /Users/brandonfish/Documents/jruby/test/mri/monitor/test_monitor.rb:159:in `test_timedwait'
    	from /Users/brandonfish/Documents/jruby/test/mri/lib/test/unit.rb:888:in `run_test'

```


Add line above for clarity (like you have in `Deduplicate()`).


不要慌~我就下周二讲课的时候拿这个演示 WebGL 的图像处理！


Hey @toftat, awesome changes, a couple of questions.

1) In `includes/oauth2callback.inc` I do not think that we want to `return ''` as that would cause Drupal to render a full page, which will take a little extra time as well as potentially confuse the user. The goal of that oauth2 calback is to update the parent window as quickly as possible. What do you think?

2) In the .info file, I think we should remove the reference to "Docs" and just have "Spreadsheets".

3) Instead of the admin path lying at `admin/config/content/`do you think a better place might be either `admin/config/content/webform/*` as a local task (tab) or at `admin/config/services/*` since we're connecting Drupal with a 3rd party web service?

Thanks for working on this! I really appreciate the revisions to install / uninstall, the requirements check, etc.


awesome job!


The thing is that Optional (Java 8) was already use. I think it will be less disruptive to use Guava's Optional than to not have it at all. Guava is already a dep for Workflow.


果然还是忍不住改 eval 呢


It's hard to imagine not using SSD's, i did it this way in case I supported overriding via the volume definition of the DeploymentConfiguraiton in the future in case it came up...


ahh, good catch on the other semicolons, we def want those!!

:+1:


Changed InstanceDetails to InstanceCreationDetails, that's really what it's for.


这里提交是没有用的。建议提交review
提交pr,是不会被merge进主分支里面的。


This doesn't work outside of the deobfuscated dev workspace. Please fix so that NovaGradle works...


Since only this value is being returned you can just do:

```go
return &LogMap{
    ...
}
```


How so?  Can you not point to the test_user_file.csv if not in the same directory?


注释的代码应该删掉


This is the information to setup the credentials in the PackageManager to communicate with the private gallery.


You probably should do a filter on instanceTypes to eliminate whitespace only/null members and then do you check to make sure you've got something valid here.


We should definitely document that.


Ah yes I need to handle `nil`, thanks!


还好我读研了，否则跟你同一年找工作被完虐了！


Can you wrap the body of the commit message... usually 72 characters long is the standard.  Wrapped commit messages help improve the experience with 'git log' and other git commands, especially when the user's console is narrow.


That's weird. I'm not sure then where they expect you to call those variables. Could be we just have to add them back in.


Copy-paste from setCCList :wink:


That is definitely a good idea. I had no idea how easy the GitHub API is in this case. Just made a quick Google search and took the first solution I found and didn't care about the language. I will adjust your solution that it writes to a file instead of the console and add it.


> 果然还是忍不住改 eval 呢

分分种注入攻击


let's go ahead and assign resp.getWorkunit to a local var so we can avoid 2 extra function calls.


Just like with struct fields, arrays and slices should be multi-line for readability:

```go
key := [6]string{
    fact.Entity.Domain,
    fact.Entity.Name,
    ...
}
```


Well done my babyli

xxx


Hmm what will break step 4.? Above I'm mean change only:

3. Let attribute be the result of getting an attribute given name and the context object.
to
3. Let attribute be the first attribute in context object's attribute list whose name is name, and null otherwise.

all other steps will be the same.


ack


Sorry i didn't even mean to comment yet. I am still testing


return decode(&buffer) == DECODE_SUCCESS would avoid an unnecessary assignment operation.


Since `exists` is only being used once this should be simplified to:

```go
if _, exists := fm.facts[key]; exists {
    return true
}
```


Hey, thanks dude. There's a bit too much happening for a single conditional. Could you think of a way it would be expressed more clearly? I'd do a rewrite myself but you're already in the zone.


2nd fstab? I dnt understand you


You're instantiating a base pagination model. How can it know that the context is a blog?


But basically what I was saying there is that I like being able to run "docs/web_services/examples/groovy/add_user.groovy" instead of "cd docs/web_services/examples/groovy; ./add_user.groovy"


RELEASE_CONFIG_DIR not exists