I'm in the process of pushing projections (pull request) which
rewrites so much logic it will obliterate any changes you make.  It
seems to work for me correctly in all modes.

On 7/23/15, didier <notifications@github.com> wrote:
> Yes,
>
> Replacing GetSize/SetSize with GetClientSize/SetClientSize help for the
> missing 2 lines I'm seeing here and it seems to make DC mode works but it
> badly breaks in opengl mode if the vp height is adjusted, even the piano
> rollover has issues!
>
> Moreover a lot of place assume that canvas size and vp size are the same (
> grid/bar scale and so on) but currently vp height modif change the scale.
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/OpenCPN/OpenCPN/commit/355cca196d9a1f7768f6c3eee3e1ccc4d6902cd9#commitcomment-12308245


What's the purpose of this? Even though runtime errors might error if this is not followed, please state any references as to the reason why this is needed (in order for other developers to be knowledgeable about it).


$_SERVER is an array containing information such as headers, paths, and script locations. The entries in this array are created by the web server. There is no guarantee that every web server will provide any of these; servers may omit some, or provide others not listed here.

I would definitely use isset() test before reading the values as they might not be set.

And this is not detecting chrome only but all browsers.


Sorry, I thought you wrote that line (I badly read the commit diff). ^^


Ahah no worries! ^^


In previous version of this pull request I asked you to do something about
```java
    ErlangCompositeElement warningHolder = PsiTreeUtil.getChildOfAnyType(file, ErlangAttribute.class, ErlangModule.class);
    if (warningHolder == null) return;
```
Please, address this issue.


started adding row name


I think this method should be renamed to `createMockPersonUsingNumber()`, so as to indicate that the number is just a cosmetic property for the mock object.


code is good


双方都


You've changed the same string in two places. Why not extract a constant?


Merging in the last bug fix.


@SumOfUs/developers 'should' doesn't add anything to our test descriptions.

See http://betterspecs.org/#should


View->Raw


second time am leaving my comments


fast-forwarding develop to borshop-integration-1198-refactor/build-misc = 7a9fea4b


In that case, this branch will always be true, always requesting next frames.. So possibly change this also to ```fade < this.options.opacity```


`version = float(platform.release().split('-')[0])` and check `version >= 10.0` seems preferable for this case.


Valid point


sehr gueti änderig


Yes what? I couldn't start ghl at all like this.
On 23 Jul 2015 09:44, "Oliver Faircliff" <notifications@github.com> wrote:

> Yes
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/github-listener/commit/ab5a3288d568797e9387267901e6924bb45abfd7#commitcomment-12315943>
> .
>


part of this fixes #172


This test starts with empty line, but the third doesn't.


Too much empty lines.


@andresgutierrez
1 By default secure and httpOnly is false
2 I think We need to discuss this in Zephir, via it's not good to convert


@prijindal: Please correct the spelling of faculty, and change the title for the HTML page VITacademics from VIT Academics.


انا كنت محتار مابين هذا الفضول وهذه الرغبة
وكتبتها كدة عشان القارئ ميحسش اني بكرر الكلمات
بس ماشي عدلتها خلاص :)


Message is specific to each location, not global to the issue.


مالها كلمة القبلية؟
طب ما نكتب المتطلبات القبلية وجنبها
prerequisites? :D


We should use a precompiled header to speedup compilation. Especially since compiling the clang stuff is slow.


Remove all unnecessary empty lines.


j（和谐）8被无情的抛弃了_(:зゝ∠)_


ممكن المتطلبات المسبقة لو عايز


I use this syntax
https://github.com/cms-sw/cmssw/pull/10233/files#diff-5cf45e3eec9fd7ebe3a7adac487723d7R35


اكيد طب احط لينك وهمي يعني لحد ما تجهز؟


`context` blocks would help here with organisation. For example:

```ruby
context 'searching'
  context 'by text'
    it 'returns matching pages'
  context 'by tags'
    it 'returns matching pages'
    it 'returns empty array when no match'
```


k dùng set. để init


set


From here http://www.gladman.me.uk/ you can check the reachable URL for now:
http://173.254.28.24/~brgladma/oldsite/cryptography_technology/sha/sha2-07-01-07.zip


تمام


Пробел


لول انا كنت سكران


Compiler doesn't allow for that.


Done.


pls replace the "long" string with the corresponding constant


Ее надо создавать при создании пользователя


I've moved all the reordering to a separate commit.


@ibuclaw gcc <= 4.9 doen't have a `hash-set`. Is has got a `pointer-set` though. Do you know if `depmod->srcfile->name->str` returns the same pointer for identical strings?


Ha, lol.


no bicycles `npm prune`?


test priority was here, now set to real


I must have nicked the phpdoc originally from there. The code in this middleware is nothing like watson, lol.


no, it's all wrong, that prevents zoom levels from being cleaned and so on. I'm digging further


why not use the `column` version here?


can be static


阿西吧


Yeah, those were already broken. todo: fix


Any reason not to apply this on master?


This fixes  IngoScholtes/pyTempNets#4


Укажешь тип сразу, чтобы потом не париться?


Done


Please revert this or fix formatting. Ummmm. Also, I just woke up and I'm working on other stuff (aswell as eating xD) so can you change all the $this->getServer()->broadcastMessage to Server::getInstance()->broadcastMessage


We need another description.


Use JS code highlighting here and in other places


At present, I do not see any reason to ignore zero-length file for file dup tests. Do you have any use case where you have to ignore zero-length files ?

The other use case, only to treat files with a length greater than for example 100K, is mentioned somewhere in the code (when using the command line version, this is easy).

So what I suggest is:

* GUI change: add a file size parameter (default= 0) meaning "treat only files having at least (greater equal) xx bytes". In other words: ignore files having less than xx bytes.

Again, I don't have a case at hand, where the (present) ignoring-zero-length makes sense.


And also it should be

```php
if($pName instanceof Player){
```


Move to API section, add `Install`, `Testing`, `Author` sections. In `Testing` section describe how to run mocha tests and playground (describe why we need playground as well).


The property column name is added through `appendComponenCollectionPath()`. I am not sure why it was needed here before, but the generated query looks alright now (as does the mapping in the store).


Image is exported and there is a getter that does not return just image, that's confusing.


Alright, that's a valid way of doing things...
I guess it saves RAM and some cycles rather than parsing it from the api_start2.json (because let's face it, JSON isn't the best thing ever)


Fixes this https://github.com/MicrosoftDX/Vorlonjs/issues/172


這邊就不用另外寫 80。


I think you were thinking about this commit https://github.com/MicrosoftDX/Vorlonjs/commit/a442cd5f1ab65b8c1436ca40d84ad6e94aec3d21, about the issue :)


Use the hash literal syntax when your keys are symbols:

`{ title: 'test page', search: [content_search: test_text] }`

See https://github.com/bbatsov/ruby-style-guide#hash-literals


WHY DID YOU REMOVE
```php
$this->saveResource("config.yml", false);
```
WTF DO YOU EVEN KNOW WHAT IT DOES?


Add README


It stops the config from overwriting everytime the server starts


這個應該可以用 `node[:opsworks][:applications][0][:slug_name]` ？


It's pushed to master, not staging. Github sometimes shows the branch info in this way, no idea why. (Perhaps it's due to the fact that master was merged to staging recently.)


:+1: short, descriptive methods. Good work!


@LaChal The latest dev I've pulled fails to load up. After showing this the software exits automatically.
![default](https://cloud.githubusercontent.com/assets/11222265/8847601/ef3dc5e2-3162-11e5-93ca-024ac8afac46.png)


The Haskell version has three terms because `let x : Nat = Z in x + x` parses to something like `PLet (UN "x") (PRef (UN "Nat")) (PRef (UN "Z")) (PApp (UN "+") [PRef (UN "x"), PRef (UN "x")])`


這是opsworks預設的app name這樣？


This plugin has been renamed: https://github.com/dcendents/android-maven-plugin/blob/master/README.md

So this line should be:

`classpath 'com.github.dcendents:android-maven-gradle-plugin:1.3'`


Fixes needed after http://review.motechproject.org/#/c/5416/ merge.


super


typo  :  'oin' not critical 'cause in description string.


I would remove this unref, because unrefing (therefore even destroying) the parameter can be a unexpected side effect (the caller of the function can be unaware of this), and if this happens, investigating the bug might not be trivial.


I'd suggest moving the guard clause onto line 37 to try and keep the rules at the same level.
`before_validation :clear_inapplicable_local_advice_methods, if:  remote_or_local_advice == :remote`


Same as in line 126.


Ah, no, my bad :-/ I forgot to push the updated master before merging it to staging.


١- تمام حاقسمه ٣ مقالات
٢- الموضوع ده مهم فعلا
صورة الكمبيوتر دي من ويكيبيديا ومكتوب ورخصتها رخصة المشاع الإبداعي.
CC BY 3.0
 يتهيألي لازم نكتب ان احنا جايبين الصورة من ويكي كومنز ونحط لينك صح؟ صاحب الصورة سايب اسمه وموقعه.
صورة الناقل العام وفون نيومان نفس الكلام
صورة اللوحة اﻷم انا كنت جايبها زمان من حتة ما... بس لاقيتها دلوقت على ويكي كومنز وصاحبها حاطط اسمه
https://commons.wikimedia.org/wiki/File:IBM_PC_Motherboard_(1981).jpg
صورة المكونات بتاعة الكمبيوتر دي انا اللي عاملها بإيدي كنت عاملها قبل كجزء من مشروع تعليمي بيني وبين زمايلي
وبما إنها بتاعتي وبما إنها كانت مفتوحة اي حد ممكن يعمل فيها اللي هو عايزه فأعتقد ان من حقي استخدمها تاني
اما الـ
screenshots
دي انا اللي واخدها من حاسوبي.

٣- حاضر حاعمل الـ
author


asdfasdf


@@ -47,6 +47,9 @@ Just type in ./SWSiOSResign and you will be prompted to enter the parameters


Lol, dont know how i could get this comment wrong, thanks @cubitouch


These aren't equivalent right?

In the older interface you can score *one* scorable for each model all at once and in the second you can score *many* scorables for *one* model. So, what this means is that if we are scoring 4 models you'll have to do 4 potential remote calls instead of just one.

Now, I don't really see a fundamental problem with it but you might have to re-evaluate perf.


:moneybag:


LGTM


jbl,


Yes getting
```
Error during instantiation of LocationStrategy! (Navigation -> Navigation -> Router -> function (_platformStrategy, href) {
        var $__0 = this;
        this._platformStrategy = _platformStrategy;
        this._subject = new EventEmitter();
        var browserBaseHref = isPresent(href) ? href : this._platformStrategy.getBaseHref();
        if (isBlank(browserBaseHref)) {
          throw new BaseException("No base href set. Either provide a binding to \"appBaseHrefToken\" or add a base element.");
        }
        this._baseHref = stripTrailingSlash(stripIndexHtml(browserBaseHref));
        this._platformStrategy.onPopState((function(_) {
          return $__0._onPopState(_);
        }));
      } -> LocationStrategy). ORIGINAL ERROR: TypeError: baseElement.attr is not a function
```

in alpha-32 also


Picked to master now af2ab0b91. Thanks for notifying me; I'm sorry for the non-standard merge pattern this creates.


qqqqq


There isn't anything that is particularly hard in it. However, this will be a huuuuge PR, so I wanted to it in a separate PR.


Answered below.


Okay, this is a fairly obvious fix, but please let someone review the code before you commit next time.


233333，早就抛弃了只不过一直没有改这个图片，因为jxxx8不支持安卓（扶额）


神！！！！！KAMI！！！！！GOD！！！！


-The purpose of this script is to simplify the resigning of an iOS Secure Work Space app. Apps need to be re-signed after they are wrapped with BES. Wrapping an app invalidates it's previous signature. The manual process for wrapping an app is documented on [developer.blackberry.com](http://developer.blackberry.com/devzone/develop/enterprise/install_android_or_ios_work_space_app.html) and the [BES Universal Device Service Documentation](http://docs.blackberry.com/en/admin/deliverables/62506/BES10_v10.2.1_UDS_Advanced_Admin_Guide_en.pdf)


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

Offending files: templates/structure/row_stats_table.phtml


RHS_UPDATE is a purely internal variable and probably should not be exposed via the API. Is the BC you are changing just the stim value? If so then you shouldn't need to do anything. If it is changing Vm i.e., the dependent var then you will need to set RHS_UPDATE. You could reset the UPDATE flags in the pre-control loop. This would ensure that when problem solve is called again after you change the dependent field it will re-assemble the matrices/vectors the first time it goes through. Now, the downside to this is that if you haven't changed anything it will uncessarily reassemble things. If this is a problem we can add a restart variable to the problem and have an API for setting this variable for warm/cold restarts etc. This would allow a more general problem solve.


@SBoudrias Should I remove this devDep ? I add this for ESLint plugin in editor


+The purpose of this script is to simplify the resigning of an iOS Secure Work Space app. Apps need to be re-signed after they are wrapped with BES 10+. Wrapping an app invalidates it's previous signature. The manual process for wrapping an app is documented on [developer.blackberry.com](http://developer.blackberry.com/devzone/develop/enterprise/install_android_or_ios_work_space_app.html) and the [BES Universal Device Service Documentation](http://docs.blackberry.com/en/admin/deliverables/62506/BES10_v10.2.1_UDS_Advanced_Admin_Guide_en.pdf)


Changed to EtcdStorage (hopefully you're fine with that one).

About package structure - I think that we should make the final decision where we would like to be and make it instead of moving some files multiple times. I'm happy to do it, but in my opinion it shouldn't be part of this PR.

Does it sound ok?


The sample code for this application is Open Source under the [Apache 2.0 License]


As far as I can see (in the play store) Webview/Chrome 44 is for android still in beta. Do you have a source for that this version is considered stable?


I found that too on the internet, but wasn't confident-enough in myself to try it, and went with more "conventional" recursion. Since the functionality for looping over a tuple is already in your PR, could it be (eventually) moved to some common area to be used also elsewhere (to avoid proliferation of different for-each-tuple implementations)?


@emmettu The American flag has *50* stars.


Has this bugfix been released yet?


Awesome you have cleaned it up and added the transaction :).
One suggestion, Replace this line with:
```csharp
using (var dbContextScope = _dbContextScopeFactory.CreateReadOnlyWithTransaction(_isolationLevel))
```
and then you can remove the second using below on line 84. Makes the code a bit more tidy!


I believe ```update!``` method (versus ```update```) will raise an exception if the model does not validate properly. In that case you would need to use a ```rescue``` statement rather than the ```if ... else``` statement.

I'd recommend writing a test that attempts to update a record with invalid data and check that it does redirect to the edit page as expected.


It doesn't, but since `jpm` defaults to `index.js` we might as well move it there. Do you mind if I steal the README instructions from your commit?


@emmettu Perhaps the transpose would be better :bulb:.


I remember Toontown Reloaded I think. You announced it back in August 2013 right?


Ничего серьёзного, просто более компактный код.


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

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


this was bad


:shipit:


lol, this was to prepare for .jar


Forgot about all this...


Vecs don't allocate if they're zero-length. Is it possible to remove the `Option` here and save a word? (If not, that's totally fine, just checking.)


ya i totally understand will RT


de que es esto?

2015-07-23 19:37 GMT-03:00 Javier Pájaro <notifications@github.com>:

> @danielpadula <https://github.com/danielpadula> @dbleyenda
> <https://github.com/dbleyenda> recuerden ademas que pueden recuperar del
> request directamente la preferencia que necesiten. Puede tambien que tengan
> el account_id disponible, y si recuperan el account, quiza no hace falta
> llamar al get_featured_accounts sino que directamente pueden sacarlo del
> objeto.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datal-org/datal/commit/223be0ab58cb58210e5759663ea578c616ef0790#commitcomment-12332163>
> .
>


@enebo truffle get threads!


@annevk and @metromoxie the latest changes extract the test file so it can be used by `npm-sri-toolbox` to reverse engineer the hashes we use here.

Still no `fetch()` tests that is next on my list (I suggest perhaps merging these first to clear the decks once this is OK with everyone - I'll work in parallel in another issue).

The latest tests seem to prove that:
- No integrity support in fetch request
- The script is not checked for a correct response coming out of the service worker


2578: if not choose_variable in integer_variables or bi and bi.is_integer():
2584: 'solution'
2588-2589: can be deleted
2592: if not eligible_source_row(choose_variable, b[index]):
2596: variable_list = copy(list_B)


Hardcoding atom ids doesn't work - need to do an intern as commented-out above. (I assume it will work to cache the atom for the duration of the program)


the `-P` needs to be escaped from `sh`

```
sudo sh -s -P <etc>
```
should be
```
sudo sh -s -- -P <etc>
```


That is lame.


Nice


Well the entire flag integration team seem to be on PTO.


- You do not need the "use strict" statement at the top of every function. You can instead put it at the top of the file, and every function within will also be run in strict mode. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode?redirectlocale=en-US&redirectslug=JavaScript%2FStrict_mode
- The .prototype.methodName = function() syntax is good, and I use it frequently in my code as well. Check out some examples from my snips project: https://github.com/lzilioli/snips (in the lib/ directory)
- .charAt() is not usually necessary for string access, unless you are working with unicode characters. Since in this case your alphabet is all ascii characters, you can get away with using [] to access your string rather than .charAt https://mathiasbynens.be/notes/javascript-unicode
- at the bottom, you dont need to declare a typewriter array, just declare var typewriter = new Typewriter("abcde") and use your typewriter variable
- The way this is designed, you need to define a new Typewriter everytime you want to try to type out a new phrase. This is fine given the way your typewriter is implemented, since it uses a lot of internal state to keep track of what its doing. I understand that you did it this way to get practice with OOP, but just wanted to mention that typically the same problem would not be solved via OOP in practice.
- in your answer method, you do not need to build up the story_so_far variable. Since you already know how many characters you have successfully matched (stored in best_guess), you can simplify that entire method down to one line: `return ('found ' + story.substring(0, this.best_count) + ' after ' + this.guess_count + ' guesses');` (if you do this, it exposes a bug in your code: `best_count` does not get incremented on the final attempt when it successfully finds the correct story. There is below feedback on how to refactor `match_advanced` to eliminate this bug.
- In general, its good to avoid naming variables such that their english name contains a negative, and instead negate your variable in your if statements. It makes it difficult to reason about the variable's value. eg. `incomplete = true` VS `complete = false`. Which is more straightforward to think about? Consider `!incomplete` vs `complete`. `!complete` is a much more straightforward way to represent completion.
- a few places in the code, you increment a variable by 1 via `variable += 1`. You can shorten that to `variable++`;
- in `match_advanced`, you only increment the `guess_count` variable in the else case. Since `match_advanced` means that a match was found, you can safely increment `guess_count` at the top of the method. If you make that change, you can remove the -1 from the if statement. The flow for that method can be simplified to:
    - increment `guess_count`
    - if `guess_count > best_count`
        - assign `guess_count` to `best_count`, and `console.log(this.answer())`
        - check if `guess_count` is the same as the length of the story, and if it is set complete to true
 with the above change to `match_advance`, you can remove the `console.log(typewriter.answer())` after your while loop


Your comments are very good thanks!

On the other hand, It's a little bit tricky hehe.. seems a tricky way for forcing a given maven reactor order. As using the test scope the jboss modules plugin is skipping this dependency to generate the modules and their dependencies, so I the resulting modules should be fine.

But what i'm missing now is... if you are forcing to build the bpms modules layer build before the brms, the parallel build itself makes sense? The dependency should block the thread building the brms until the bpms one is finished, isn't it? Just for my clarification too, probably i'm wrong :)

Thanks @psiroky


99% done. Should be finished this weekend

On Thu, Jul 23, 2015 at 5:38 AM -0700, "Steven Degutis" <notifications@github.com> wrote:

Wait does this mean it's done?

—
Reply to this email directly or view it on GitHub.


Adding `me_fields.MapField: drf_fields.DictField` to django-rest-framework-mongoengine.serializer.py gives you last_read_times.


`Option<Vec>` has the same size as `Vec` because of the Unique/NonZero optimization.  Do you still want this change for any other reasons?


That is because you're summing real numbers each of which have small rounding errors. You need to use loose comparisons like [numpy.allclose](http://docs.scipy.org/doc/numpy/reference/generated/numpy.allclose.html#numpy-allclose) to make sure it is within epsilon of  1.0, e.g. this does not throw an exception
```
def get_random_haplotype():
    haplotypes = pop.keys()
    frequencies = [x/float(pop_size) for x in pop.values()]
    if not np.allclose(sum(frequencies), 1.0):
        raise Exception("frequencies don't sum to 1.0")
    return np.random.choice(haplotypes, p=frequencies)
```
There is a similar accumulation of errors in `get_snp_frequency` because as written, it is accumulating real numbers.  You'd never notice as a practical matter, but it would be more numerically accurate and efficient to do the division only once, e.g.
```
def get_snp_frequency(site, generation):
    minor_allele_count = 0
    pop_at_generation = history[generation]
    for haplotype in pop_at_generation.keys():
        allele = haplotype[site]
        if allele != "A":
            minor_allele_count += pop_at_generation[haplotype]
    return minor_allele_count / float(pop_size)
```


@emmettu Did you water their plants?


Course.


Very nice.


'ny' -> 'by'


This would be a 10-point by (cost is 5 x new rank), but it looks like you accounted for it as a 5-point buy.


Ah... I see. Thanks for the explanation.


In future, place a blank line between the script description and the copyright.


Remove this line.  `_namespace' initializes to `null` by default.


Thanks @Redjik , we will put this feature on hold until we have a use case for the specification pattern - likely a more business logic specific case.


Test


Testing


@PhilipShilling Only handle events async, marked "async" such as AsyncPlayerChatEvent. And, once we get a scheduler system, there will be no need for ServerTickEvent.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1247 of 1247 tests passed)

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


Tests


@misterdjules is there any reason this was done over `while (new Date().getTime() < stopTime);`? (porting this to io.js)


Looks good. Will commit this to the codebase. For the future, for your commit title, do something like:
security fix CWE-89 fix #1(and give each a separate number; start it at 10 since I am going to do this for your prior commits)
(we do this so we don't make the security vulnerability obvious to the public)


The performance will not be an issue here if we only retrieve subset array information from database.


The only problem I see with this is that there is no validation done for this field.


hermano la base de datos donde esta gracias


93653360-ResetVarsAtEndOfWarmup (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1237 of 1237 tests passed)

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


This case comes up at https://github.com/FStarLang/FStar/blob/6fd00de934914874c2bfcb5eee49862ce48d1e97/lib/heap.fst#L48

The error is:

$ fstar st3.fst
extracting: Prims
extracting: FunctionalExtensionality
extracting: Set
extracting: Heap

Unexpected error
Unexpected meta in type ((Meta {:pattern (contains h0 a)} ((mem (Ref a) refs) ==> ((not (contains h0 a)) /\ (contains h1 a))))) at ../../lib/heap.fst(48,56-48,112)

   at Microsoft.FStar.Backends.ML.ExtractTyp.extractTyp(env c, syntax`2 ft) in C:\cygwin\home\t-aban\FStar\src\backend\extracttyp.fs:line 179


Put this test back in in the new format.


Er, what new format? The function only takes one arg now.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1247 of 1247 tests passed)

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


this could be on same line as the err, but a nit so whatever


I havent tried moving it around in the take turn order, but I dont expect
that will make much difference.

On Thu, Jul 23, 2015 at 2:55 PM, John c <greencardinal4@gmail.com> wrote:

> This is very similar to what ive been working on. I had the player give a
> clue about reservables only if it had a clue match greater than one, if
> thats true I then have it find the highest amount of matches possible
> between different reservables and their colors and numbers. it bumped down
> our average to 19.06. I havent tried changing its place in the take_ turn,
> and I also wanna try and filter it to only give those clues when the
> reservable cards age is high.
>
> I can work on this a bit more today, and show you what ive come up with
> later if you wanna take a look tonight.
>
> On Thu, Jul 23, 2015 at 1:30 PM, Mark Spurlock <notifications@github.com>
> wrote:
>
>> I added (privately, this isn't pushed yet) an inferred_reserved thing
>> that was basically analogous to inferred_playables, but it wasn't working
>> out. Too easy to either (a) fail to infer the intention of the clue, or (b)
>> misconstrue it as implying the card is playable.
>>
>> So I'm thinking of scrapping that (for now) and just relying on the
>> "players try not to discard stuff they have info about" behavior. So at the
>> point where we decide to give a "reserve this card" clue, all we have to do
>> is make sure the clue can't be misconstrued as "play this card". So the
>> clue would have to meet one of two conditions: (a) the clue is about more
>> than one card, meaning the receiving player won't try to infer anything
>> about it, or (b) if it is just one card, the receiving player will try to
>> infer that it's playable but there won't be any playable cards that match
>> it. In that case, he won't infer anything about it, but he'll be less
>> likely to discard it since he has information about it.
>>
>> The other part of this is knowing how much to prioritize "reserve card"
>> clues. I think it's generally better to tell a player "play this" as
>> opposed to "reserve that", but we may need to experiment to find the sweet
>> spot. And in general, the decision making process that goes on during
>> take_turn() probably needs to get smarter.
>>
>> I was planning on doing this work tonight, but it sounds like you might
>> be toying with some similar ideas, so if you'd like to take a crack at it
>> instead just say the word.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Spurlock/Hanabi/commit/b1b24eea3bddb244e16e8a821e23d803ec1a400d#commitcomment-12326037>
>> .
>>
>
>

from __future__ import division
from pprint import pprint
import random
import sys

NUM_PLAYERS = 3
HAND_SIZE = 5 if NUM_PLAYERS < 4 else 4
COLORS = ['pink', 'blue', 'white', 'yellow', 'green']
CARD_COUNTS = {1: 3, 2: 2, 3: 2, 4: 2, 5: 1}
MAX_CLUES = 8
NUMBER_OF_GAMES = 50
final_scores = []
in_game_prints = False

# Current 50 Game Scores:
# Best: 22
# Worst: 16
# Average: 19.32000

class Game:

    def __init__(self):
        self.game_over = False
        self.remaining_fuses = 3
        self.remaining_clues = 8
        self.whose_turn = 0
        self.last_turn = 0
        self.unseen_cards = self.build_deck()
        self.graveyard = []
        self.turn_taken = False
        self.table = {color: [] for color in COLORS}

        self.deck = self.build_deck()
        random.shuffle(self.deck)

        # Deal initial hands
        self.players = [Player(i) for i in xrange(0, NUM_PLAYERS)]
        for player in self.players:
            for i in xrange(0, HAND_SIZE):
                player.hand[i] = self.deck.pop()
                player.hand_age[i] = i
            if in_game_prints is True:
                print "player %d's hand: %s" % (player.number, player.hand)

    def build_deck(self):
        deck = []
        for color in COLORS:
            for key, count in CARD_COUNTS.iteritems():
                cards_to_add = [Card(color, key) for i in xrange(0, count)]
                deck.extend(cards_to_add)
        return deck

    def get_playable_cards(self):
        playable_cards = []
        for color, card_list in self.table.iteritems():
            if len(card_list) < 5:
                playable_cards.append(Card(color, len(card_list) + 1))
        return playable_cards

    def get_useless_cards(self):
        useless_cards = []
        # all cards that have already been played
        for color, card_list in self.table.iteritems():
            useless_cards += card_list

        # cards that can't be played because there are no remaining copies of one of its unplayed prereqs
        for color in COLORS:
            color_discarded = [card for card in self.graveyard if card.color == color]

            # for each rank, if the number discarded equals the total number, all higher cards are useless
            for rank in xrange(1, 6):
                color_rank_discarded = [card for card in color_discarded if card.number == rank]
                if len(color_rank_discarded) >= CARD_COUNTS[rank]:
                    for higher_rank in xrange(rank + 1, 6):
                        useless_cards.append(Card(color, higher_rank))
        return useless_cards

    # Returns any card that isn't useless and only has one left of its type
    def get_reserved_cards(self):
        reserved_cards = []
        card_types = []
        useless_cards = self.get_useless_cards()
        for color in COLORS:
            for key in CARD_COUNTS:
                card_types.append(Card(color, key))

        for card in card_types:
            if card not in useless_cards:
                if self.graveyard.count(card) == CARD_COUNTS[card.number] - 1:
                    reserved_cards.append(card)
        return reserved_cards

    def mark_turn_taken(self):
        if self.turn_taken:
            sys.exit("Tried to take an extra turn!")
        self.turn_taken = True

class Card(object):

    def __init__(self, color, number):
        self.color = color
        self.number = number

    def __repr__(self):
        return "%s %s" % (self.color, self.number)

    def __eq__(self, other_card):
        if self.color == other_card.color and self.number == other_card.number:
            return True
        return False

class Player:

    def __init__(self, number):
        self.hand = [None] * HAND_SIZE
        self.hand_age = [None] * HAND_SIZE
        self.knowledge = [Card(None, None) for _ in xrange(0, HAND_SIZE)]
        self.private_knowledge = []  # What I know
        self.public_knowledge = []  # What everyone knows I know
        self.number = number
        self.infered_playables = [None] * HAND_SIZE # [Blue 2, Green 2]

    def take_turn(self):

        # Play a card from my hand if one is known to be playable
        my_playable_cards = self.get_known_playable_cards(self)
        if len(my_playable_cards) > 0:
            self.play_card(my_playable_cards[0])
            return

        #plays infered playables.
        for index, card_list in enumerate(self.infered_playables):
            if card_list is not None:
                self.play_card(index)
                return

        # If clues remain and next player has a playable card, gives clue about the card
        next_player = game.players[(self.number + 1) % NUM_PLAYERS]
        next_player_playable_cards = self.get_playable_cards_for_player(next_player)
        players_after_next = self.get_players_after_next()

        #TODO: give clues to keep players from discarding reserved cards
        if game.remaining_clues > 0:
            clue_up = None
            if len(next_player_playable_cards) > 0 and len(self.get_known_playable_cards(next_player)) == 0:
                for playable_card_index in next_player_playable_cards:
                    card_knowledge = next_player.knowledge[playable_card_index]
                    if card_knowledge.number is None:
                        clue_up = next_player.hand[playable_card_index].number
                    elif card_knowledge.color is None:
                        clue_up = next_player.hand[playable_card_index].color
                if clue_up is not None:
                    self.give_clue(clue_up, next_player)
                    return

            #in 100 games, the reserved > 0 and useless < 0 conditions are never met
            #so for now the check for useless < 0 is off
            if len(self.get_reserved_cards_for_player(next_player)) > 0: #and len(self.get_known_useless_cards(next_player)) < 0:

                match_bar = 5 #only matches greater than or equal to the bar pass
                age_index_bar = 4 #only matches less than or equal to the bar pass

                #mostly no improvements changing the bars around, but there are a few exceptions...
                #with match_bar at 4 and age bar between at 1,2,3, or 4, or match bar at 3 and age bar at 3 or 4,
                # average improves up to .04 points. not much at all.
                #with match bar at 2 and age bar at 1, best game goes up by one, and with match at 2 and age bar at 2...
                #high game and best game go up by one (22-23 and 16-17). average is lower though (19.22 at age 1 and 19.00 at age 2)

                highest_match = 0
                for index, reservable_card_index in enumerate(next_player.hand_age):
                    if index <= age_index_bar and reservable_card_index in self.get_reserved_cards_for_player(next_player):
                        card_knowledge = next_player.knowledge[reservable_card_index]
                        reservable_card = next_player.hand[reservable_card_index]
                        if card_knowledge.number is None:
                            number_match = 0
                            for card in next_player.hand:
                                if card is not None and getattr(card, 'number') == reservable_card.number:
                                    number_match += 1
                            if number_match >= match_bar and number_match > highest_match:
                                clue_up = reservable_card.number
                                highest_match = number_match
                        if card_knowledge.color is None:
                            color_match = 0
                            for card in next_player.hand:
                                if card is not None and getattr(card, 'color') == reservable_card.color:
                                    color_match += 1
                            if color_match >= match_bar and color_match > highest_match:
                                clue_up = reservable_card.color
                                highest_match = color_match
                if clue_up is not None:
                    self.give_clue(clue_up, next_player)
                    return

            # If the next player already knows they have a playable card, gives clue to the player after them
            for player in players_after_next:
                if len(self.get_playable_cards_for_player(player)) > 0 and len(self.get_known_playable_cards(player)) == 0:
                    for playable_card_index in self.get_playable_cards_for_player(player):
                        card_knowledge = player.knowledge[playable_card_index]
                        if card_knowledge.number is None:
                            clue_up = player.hand[playable_card_index].number
                        elif card_knowledge.color is None:
                            clue_up = player.hand[playable_card_index].color
                    if clue_up is not None:
                        self.give_clue(clue_up, player)
                        return

        #useless_cards = game.get_useless_cards()
        my_useless_cards = self.get_known_useless_cards(self)
        #reserved_cards = game.get_reserved_cards()
        my_reserved_cards = self.get_known_reserved_cards(self)

        if not game.turn_taken:
            if len(my_useless_cards) > 0:
                self.discard(my_useless_cards[0])
            else:
                discardables = [index for index, card in enumerate(self.knowledge) if card.color is None and card.number is None]
                if len(discardables) < 1:
                    discardables = [index for index, card in enumerate(self.knowledge) if card.color is None or card.number is None]
                if len(discardables) < 1:
                    discardables = [index for index, card in enumerate(self.knowledge) if index not in my_reserved_cards]

                for age in self.hand_age:
                    if age in discardables:
                        self.discard(age)
                        return

        if not game.turn_taken:
            print "*** WARNING! Using stupid discard! ***"
            self.discard(0)

    def lose_card(self, index):
        lost = self.hand[index]
        game.unseen_cards.remove(lost)
        self.hand_age.remove(index)
        drawn_card = None

        self.remove_from_public_knowledge(lost)
        self.remove_from_private_knowledge(lost)

        if len(game.deck) > 0:
            drawn_card = game.deck.pop()
            self.hand[index] = drawn_card
            self.hand_age.append(index)
        else:
            self.hand[index] = None

        self.knowledge[index] = Card(None, None)
        self.public_knowledge[index] = [card for card in game.unseen_cards]
        self.private_knowledge[index] = [card for card in game.unseen_cards]
        # private knowledge about the drawn card accounts for other players' hands
        for player in game.players:
            if player is not self:
                for card in player.hand:
                    if card is not None:
                        self.private_knowledge[index].remove(card)

        self.infered_playables[index] = None

        # other players see the card you just grabbed and update their private knowledge
        if drawn_card:
            for player in game.players:
                if player is not self:
                    player.remove_from_private_knowledge(drawn_card)
        return lost

    def play_card(self, index):
        if self.hand[index] is None:
            sys.exit("Error: Tried to play a nonexistent card.")
        game.mark_turn_taken()

        played = self.lose_card(index)

        if in_game_prints is True:
            print "Player %d plays a %r" % (current_player.number, played)
            if self.hand[index] is not None:
                print "Player %d draws a %r" % (current_player.number, self.hand[index])
            else:
                print "The deck is empty"

        if played in game.get_playable_cards():
            game.table[played.color].append(played)
            if played.number == 5:
                game.remaining_clues = min(game.remaining_clues + 1, MAX_CLUES)

            #Removes inference that has a card that matches played card
            for player in game.players:
                for index, card_list in enumerate(player.infered_playables):
                    if card_list is not None and played in card_list:
                        player.infered_playables[index] = None
                        break
        else:
            game.graveyard.append(played)
            game.remaining_fuses -= 1
            print "FUSE BURNED!!!!!!!!!!!!!"

    def discard(self, index):
        game.mark_turn_taken()
        discarded = self.lose_card(index)

        if in_game_prints is True:
            print "Player %d discards a %r" % (current_player.number, discarded)
            if discarded in game.get_reserved_cards():
                print "Discarded an irreplacable card!!!"
            if self.hand[index] is not None:
                print "Player %d draws a %r" % (current_player.number, self.hand[index])
            else:
                print "The deck is empty"

        game.remaining_clues = min(game.remaining_clues + 1, MAX_CLUES)
        game.graveyard.append(discarded)

    def give_clue(self, clue, receiving_player):
        game.mark_turn_taken()
        if receiving_player is self:
            sys.exit("Error: Can't give yourself a clue")

        receiving_player.receive_clue(clue)

        if in_game_prints is True:
            print "Player %d gives Player %d a clue about %ss" % (current_player.number, receiving_player.number, clue)

    def receive_clue(self, clue):
        game.remaining_clues -= 1
        clue_type = 'number' if type(clue) == int else 'color'
        matches = []
        for index, card in enumerate(self.hand):
            if card is not None and getattr(card, clue_type) == clue:
                setattr(self.knowledge[index], clue_type, clue)
                matches.append(index)

        if len(matches) == 0:
            sys.exit("Error: It's illegal to give a clue that the receiving player has zero of something")

        for i in xrange(0, HAND_SIZE):
            if i in matches:
                self.public_knowledge[i] = [card for card in self.public_knowledge[i] if getattr(card, clue_type) == clue]
                self.private_knowledge[i] = [card for card in self.private_knowledge[i] if getattr(card, clue_type) == clue]
            else:
                self.public_knowledge[i] = [card for card in self.public_knowledge[i] if getattr(card, clue_type) != clue]
                self.private_knowledge[i] = [card for card in self.private_knowledge[i] if getattr(card, clue_type) != clue]

        # When a clue only matches one card, a list of playable cards from the
        # private knowledge index of that card is added to that index of infered playables
        if len(matches) == 1:
            self.infered_playables[matches[0]] = []
            playable_cards = game.get_playable_cards()
            for possible_card in self.private_knowledge[matches[0]]:
                if possible_card in playable_cards:
                    self.infered_playables[matches[0]].append(possible_card)

    def get_cards_in_list(self, player, card_list):
        if player is self:
            sys.exit("Can't call get_cards_in_list on self. Use get_known_cards_in_list instead.")

        player_card_matches = []
        for index, card in enumerate(player.hand):
            if card is not None and card in card_list:
                player_card_matches.append(index)
        return player_card_matches

    def get_known_cards_in_list(self, player, card_list):
        player_card_matches = []
        knowledge = player.private_knowledge if player is self else player.public_knowledge
        for index, possibilities in enumerate(knowledge):
                hits = [True for possible_card in possibilities if possible_card in card_list]
                if len(hits) == len(possibilities):
                    player_card_matches.append(index)
        return player_card_matches

    def get_playable_cards_for_player(self, player):
        playable_cards = game.get_playable_cards()
        return self.get_cards_in_list(player, playable_cards)

    def get_useless_cards_for_player(self, player):
        useless_cards = game.get_useless_cards()
        return self.get_cards_in_list(player, useless_cards)

    def get_reserved_cards_for_player(self, player):
        reserved_cards = game.get_reserved_cards()
        return self.get_cards_in_list(player, reserved_cards)

    def get_known_playable_cards(self, player):
        playable_cards = game.get_playable_cards()
        return self.get_known_cards_in_list(player, playable_cards)

    def get_known_useless_cards(self, player):
        useless_cards = game.get_useless_cards()
        return self.get_known_cards_in_list(player, useless_cards)

    def get_known_reserved_cards(self, player):
        reserved_cards = game.get_reserved_cards()
        return self.get_known_cards_in_list(player, reserved_cards)

    # returns players in order of turn after next player, up to the current player
    def get_players_after_next(self):
        players_after_next = []
        number = (self.number + 2) % NUM_PLAYERS
        for i in xrange(0, NUM_PLAYERS - 2):
            players_after_next.append(game.players[number])
            number = (number + 1) % NUM_PLAYERS
        return players_after_next

    def init_knowledge(self):
        self.public_knowledge = [game.build_deck() for _ in xrange(0, HAND_SIZE)]

        private_deck = game.build_deck()
        other_players = [player for player in game.players if player is not self]
        for player in other_players:
            for card in player.hand:
                private_deck.remove(card)

        self.private_knowledge = [list(private_deck) for _ in xrange(0, HAND_SIZE)]

    def remove_from_private_knowledge(self, card):
        for card_list in self.private_knowledge:
            if card in card_list:
                card_list.remove(card)

    def remove_from_public_knowledge(self, card):
        for card_list in self.public_knowledge:
            if card in card_list:
                card_list.remove(card)

# Prepare to start playing games
random.seed(0)

for game_number in xrange(0, NUMBER_OF_GAMES):
    #TODO: study worst game (number 9) for strategy ideas
    #if game_number == 9:
        #in_game_prints = True
    #else:
        #in_game_prints = False

    # Start a game
    game = Game()
    turn = 1

    for player in game.players:
        player.init_knowledge()

    # Take turns loop
    while not game.game_over:

        if in_game_prints is True:
            print "*****"
            print "Starting turn %d of game %d" % (turn, game_number)

        game.turn_taken = False
        current_player = game.players[game.whose_turn]

        if in_game_prints is True:
            print "Player %d's turn" % current_player.number

        current_player.take_turn()
        if not game.turn_taken:
            sys.exit("Player failed to take an action!")

        if in_game_prints is True:
            print "Game Table:"
            pprint(game.table)
            for player in game.players:
                print "player %d's hand: %s" % (player.number, player.hand)
                print "Player %d knows: %s" % (player.number, player.knowledge)
            print "Clues remaining: %d" % game.remaining_clues

        #if game isn't over, prepare for next turn
        if game.remaining_fuses > 0 and game.last_turn < NUM_PLAYERS:
            turn += 1
            game.whose_turn = (game.whose_turn + 1) % NUM_PLAYERS
            if len(game.deck) <= 0:
                game.last_turn += 1
        else:
            game.game_over = True

        # Validates every player's public and private knowledge.
        for player in game.players:
            for index, card in enumerate(player.hand):
                if card:
                    if card not in player.public_knowledge[index] or card not in player.private_knowledge[index]:
                        sys.exit("Error: Player %d believes a card in his hand (%r at %d) is impossible" % (player.number, card, index))

    color_scores = [len(card_list) for card_list in game.table.values()]
    final_score = sum(color_scores)
    print "Game %i's Final Table:" % game_number
    pprint(game.table)
    print "Game Score: %d" % final_score
    print ""

    final_scores.append(final_score)

best_score = max(score for score in final_scores)
worst_score = min(score for score in final_scores)
average_score = sum(final_scores) / NUMBER_OF_GAMES
print "*****"
print "Out of %i Games:" % NUMBER_OF_GAMES
print "*****"
print "Best Score: %i" % best_score
print "Worst Score: %i" % worst_score
print "Average Score: %f" % average_score


Yes, but there's no EXPECT_FALSE(RDMUtil_RequiresResponse(OTHER_UID)); check


An simple frontend for GNU GDB Linux.


spurious change


looks good


This should really be erased. In the meantime, just ignore the pattern and translate the underlying type.

Sent from my Windows Phone
________________________________
From: Abhishek Anand<mailto:notifications@github.com>
Sent: 7/23/2015 17:44
To: FStarLang/FStar<mailto:FStar@noreply.github.com>
Cc: nikswamy<mailto:nswamy@live.com>
Subject: Re: [FStar] some more debugging with Abhishek (6fd00de)

This case comes up at https://github.com/FStarLang/FStar/blob/6fd00de934914874c2bfcb5eee49862ce48d1e97/lib/heap.fst#L48

The error is:

$ fstar st3.fst
extracting: Prims
extracting: FunctionalExtensionality
extracting: Set
extracting: Heap

Unexpected error
Unexpected meta in type ((Meta {:pattern (contains h0 a)} ((mem (Ref a) refs) ==> ((not (contains h0 a)) /\ (contains h1 a))))) at ../../lib/heap.fst(48,56-48,112)

   at Microsoft.FStar.Backends.ML.ExtractTyp.extractTyp(env c, syntax`2 ft) in C:\cygwin\home\t-aban\FStar\src\backend\extracttyp.fs:line 179

---
Reply to this email directly or view it on GitHub:
https://github.com/FStarLang/FStar/commit/6fd00de934914874c2bfcb5eee49862ce48d1e97#commitcomment-12333759


Tailing "]"


Trailing "rsync=/usr/bin/rsync" when using debug mode.


@jython234 Alright. :P


Missing a "$" in "$user@host".


That's because it doesn't make sense now. The precondition is that RDMUtil_RequiresAction() is true, which wouldn't be the case for OTHER_UID.


What you need to do before checking out stuff:

1. bundle install
2. go to your config/application.yml and add two fields FACEBOOK_APP_ID and FACEBOOK_APP_SECRET.

For example if the config/application.yml was an empty file, it would now have these two new lines:

FACEBOOK_APP_ID: "whatever_your_app_id_is"
FACEBOOK_APP_SECRET: whatever_your_secret_is

3. The Users table has been revamped completely. So that means any and every migration pertaining to the old Users table should be dropped. Once the dropping is done, run rake db:migrate.

4. That's it! Enjoy.


97903100_SolarCollectors (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Add a name


Exciting!


97903100_SolarCollectors (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


This is a style change. You just added {}.


Just style change.


97903100_SolarCollectors (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


This looks correct to me.  It makes me wince to do this twice.  But it is what it is.


Just style change though I would like to accept this change.


Don't remove static modifier.
In a parser, please don't define global variables. Just define file static variables.


style.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (352 of 352 tests passed)

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


ECB related fix?
This should not be part of this  PR.


style


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (352 of 352 tests passed)

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


Not sure if my `to_delimited` function is the best way to go about it. Open to feedback.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (352 of 352 tests passed)

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


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/3cae2aed1d936b3149430612030f3866e0cb5973/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/3cae2aed1d936b3149430612030f3866e0cb5973/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/3cae2aed1d936b3149430612030f3866e0cb5973/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/3cae2aed1d936b3149430612030f3866e0cb5973/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/3cae2aed1d936b3149430612030f3866e0cb5973/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


> or I'm ok t disable dump log for all cases in default

For my case, dump-command in ik fail is OK and it would be nice to disable dump-command in ik success by default.
How about to introduce some kind of mode for `dump-command`?
- `dump-command` is `nil`, do not dump command for ik success and ik fail.
- `dump-command` is `t`, always dump command
- `dump-command` is `:fail-only`, do not dump in ik success and dump in ik fail


Thank you but I would like receive this in another PR or another commit in this PR.
I'm very sorry you spening time.


You meant backendPort here?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1247 of 1247 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (352 of 352 tests passed)

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


Haha, ok, I was confused there apparently.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (352 of 352 tests passed)

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


return setDefault(v, tag.defaultVal)


Actually, we can just use DictField.


https://www.python.org/dev/peps/pep-0008/#id17
https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Imports_formatting#Imports_formatting

- So all stdlib imports first (that's all there are here)
- Sorted lexicographically (basically alphabetically by package name, not from vs. import)
- Also, we'll probably need `subprocess.check_output` and possibly `check_call` later.


Learning something new every day. Cool, either way works for me.


Can we not just set it to false by default and if the server admin wants to enable it we assume they have Towny?


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (352 of 352 tests passed)

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


result won't be null


You already checked this above; I'd drop one of the two.


Awesome, I'm glad I could contribute!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (352 of 352 tests passed)

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


https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=Indentation#Indentation

Two options here. See above, per the "Aligned with opening delimeter-dict" and "4-space hanging indent in a dictionary" options.


There is no seperate SQL file for this.

Thank you,


@jvlomax this may fix or be related to the issues you were seeing with `self.master` when switching to new-style classes!


It's better to move  `Thread.sleep(waitTimeMs)` after `result = Some(body)` so that it doesn't wait in the normal case.


Can't you use the existing valueOf to do most of this work?


No you still need both classes for users who dont want to use stem


Also, FYI, for *me*, I like to always end string lines that have to be continued with a space so that the *next* line starts with an actual character, as you have done here. Bravo!


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (352 of 352 tests passed)

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


The performance will not be an issue here since we only retrieve subset array information from database.

________________________________
From: Zhou Ji [notifications@github.com]
Sent: Wednesday, July 22, 2015 11:54 AM
To: geworkbench-group/geworkbench-web
Cc: You, Min
Subject: Re: [geworkbench-web] fix for mantis bug 3343 (c540ef0)

this commit is problematic in multiple ways

—
Reply to this email directly or view it on GitHub<https://github.com/geworkbench-group/geworkbench-web/commit/c540ef07f9b7ef25f0e053fa65c1d03050fa0a57#commitcomment-12301391>.


Let's get all the way nerdy: https://www.python.org/dev/peps/pep-0257/


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


Normal case starts with initial waitTimeMs of 0. But I guess its more intuitive to move it down.


Also, we'll see how long this construct lasts. I had been thinking about them and wanted to try it. Not sure if this is ultimately the best way to express this-_and_ how many times do intend on using this value? If only for one conditional block this may be overkill / extreme-fancy-pantsness.


Uh huh. See what you did there. Nice eyes.


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (351 of 352 tests passed)

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


Does that mean having a check of "use_stem" at this point and it True, then call the StemEvent(x) otherwise, just call Event(x)?
This would mean that StemEvent class will be inheriting the Event class but will override the got_update method.


Feel free to try out the APK (Android) or the runnable JAR file.

This is a clone of the game Drop7 that I programmed to get some experience with LibGDX.
Gameplay is limited to the "hardcore" mode, which in my opinion is by far the best mode.
High Scores and options all work, but sound effects & music are basic/placeholder quality.

I'll consider releasing the source if anybody requests it.


`cursor(**behavior)`


Nope, go ahead


https://google-styleguide.googlecode.com/svn/trunk/pyguide.html?showone=TODO_Comments#TODO_Comments

Do we care about putting *who* is going to  work on the TODO's? e.g.
`# TODO(Shea): Add --plist argument to header info up top.`

(I think yes-makes it super easy to grep for myself and my TODO list.


Too long!


It works perfect thanks


Actually, you also dont want to wait later when you have exceeded max retries as well. So a better option is to do
```
while (result.isEmpty && !isTimedOut && !isMaxRetryDone) {
      if (retryCount > 0) {  // wait only if this is a retry
        Thread.sleep(waitTimeMs)
        waitTimeMs *= 2  // if you have waited, then double wait time for next round
      }
...
```


I'd love to see all the different types tickled here.
Also would be good to check a field that has a default but has been set.


This is only true on cpython2.7 and higher

For cpython2.6, and jython2.7, sys.version_info still returns a tuple.

In pkg_resources this is *actually* implemented as ``platform.python_version()[:3]`` which is consistent across versions


Talk to me about the name-mangling of variables here and elsewhere. I'm interested to understand the motivation-I usually don't use these, so I'm interested in learning!

I *do* frequently use the convention of preceding private variables, methods, and classes with an underscore.

Also, if you try to use certain reserved python syntax as variables names it will let you, but it's not good. The one I always get stuck on is `id`, which is a builtin. Instead, it's better to use `id_`.


#2 fixed


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (351 of 352 tests passed)

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


Making changes


Ultimately, `main` should read as a flowchart of how the entire program executes, from a very high level. As each level of abstraction is drilled into, it should still be fairly readable, and brief-hopefully no more than a full screen's worth of text. Of course, I have my retina 15" MBP on itty bitty 10pt font, but we're really talking about one-old-skool terminal worth of vertical real-estate, plus or minus.

I will continue to refactor and get us there, once I have slept and recaffeinated. And I'm throwing the python-jss update out into the wilds of production tomorrow.


I am not sure that code after merging will be more readable in this case.


I'll extract logic to function.


The sqrt_controller makes it so the response to an error isn't linear.  It's close to linear when the errors are small but as they get very large the response is relatively weak.


Looks like it's probably unnecessary


Nice make fu


Remove `only`


I'd leave this on a single line


LGTM


I've just built this analyzer for windows, 32 bit and 64 bit, targeting Saleae Logic Software 1.2.2+ (dynamically linked runtime, not compatible with previous releases)

You can get the binaries here:
32 bit:
http://downloads.saleae.com/betas/1.2.3/RFFEAnalyzer+32+bit+Windows+1.2.2%2B.zip
64 bit:
http://downloads.saleae.com/betas/1.2.3/RFFEAnalyzer+64+bit+Windows+1.2.2%2B.zip

In testing in debug mode, the debug iterators from Logic indicated that there could be an ordering problem with how frames are added, but when running with the production software, these iterator checks are not performed. Does not seem to be a problem for now though. I haven't investigated yet.


All azure locations are allowed for Application Gateways.


Nevermind.  It is done later.


what is addmessage


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


You've removed the `git-link-github` function: https://github.com/carlesso/git-link/blob/9a09806251fa88c51864b28aca7b815bd1c730f1/git-link.el#L169-179


That doesn't work on GitLab: https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.rubocop.yml#L50-55


Would it make sense to make this lower case as well?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1247 of 1247 tests passed)

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


IIRC, this should be in devDependencies because it only aids in development, not in the running of actual code.


You're right. Thanks.


Shouldn't this be Educational Public License?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@toobulkeh this is progress on the tagging, but right now, i'm getting an error on creating resources. the `collect` action of the resources controller seems to be involved in responding to the POST to `/resources` (where `create` should be responding). Haven't been able to track down why yet. Any ideas?


idk


Fixed.


PR: https://github.com/OData/odata.net/pull/218


Let's move this function to the top scope with the rest of the the helpers.


no particular reason - just forgot. will add it, thanks!


Let's check `if (err.code === 'ENOENT')` to see if it is actually not found. Also maybe some validation to see if it is indeed of the format `MDTM YYYYMMDDhhmmss filename`.


minor: spaces around operators


What if date didn't parse here... would it be null or NaN?


style: no space after function keyword


Looks good. Just a few minor things I noted inline. Also, you can get rid of the original code that's commented out. Much appreciated!


Hey @oddityoverseer13 looks like your text editor is using tabs. Plz switch to spaces

See http://wiki.smashingboxes.com/articles/backend#indentation


Does this accomplish the same as this: https://github.com/yawkat/AlwaysOnline/commit/20afb6362792e13133e2a03694dbe38f535f6172


Instead of `GIT_PROMPT_USE_PYTHON`, should that be `GIT_PROMPT_EXECUTABLE`?

The _README.md_ file mentions setting `GIT_PROMPT_EXECUTABLE` before sourcing this file, but despite doing so line 8 above overwrites `GIT_PROMPT_EXECUTABLE` unconditionally.


After looking through all the tutorials, I think I understand. I didn't realize that there were so many screenshots! I just pushed some changes that make it so that on pages like _getting started_, which have lots of text and an image every few paragraphs, we can include the inline version that floats next to the text. But on pages that we don't modify to switch the include, which I expect will be most of them, they will be horizontally centered on their own line like before.


Is something like this a viable solution?

```js
Util.IS_NODE = !!(process && process.browser !== true);
```

Relying on the explicit value of this key seems possibly a bit less fragile, though I may be missing context for other considerations.


Looks good to me!


y u change my version numbers?!


Juanjors el futuro lo tienes en tus manos, que es lo que se puede esperar de el? todo, pero solo pasara si tu eres el articife de el !!!FUTURO........ , elige, camina no te pierdas en los objetivos, aunque si debes fijarlos y perseguirlos con tesón , audacia y sin temor pero siempre sin perder y disfrutar del camino que a ellos te lleve y sin sacrificar nunca el presente que  en si es el que sucede en todo momento a nuestro alrededor, vive disfruta siente y nunca olvides que si ahora el esfuerzo te lleva a conseguir tus grandes dotes en la programación seguira dándote y cada vez mas satisfacción.
jpetis http://jpetis.blogspot.com.es/


Why not?
```php
env()->get('slack_app_name', 'app-name');
```


Nice, good catch


copy-pasta!


The testpypi.python.org is the old version, need to be updated.


Review 完工。


Could you explain why moving the lock makes binary addons work?


Did someone tested both Netty versions on affected files what is the extact difference in network communication which breaks streaming of specific files? :+1:


got confused with master where we just moved the close from the runtime to ScriptingContainer and then I came across this JavaEmbedUtils and wanted to have it consistent.

right we should do this reflectively here and use this method in ScriptingContainer#tearDown as well.

I am bit tied today and over the weekend . . .


Where do you see the use of the `dataset` API? :)


Probably good to add:
  - several bugfixes and workflow improvements of Script running and dialogs (e.g. https://github.com/openmicroscopy/openmicroscopy/pull/3859 https://github.com/openmicroscopy/openmicroscopy/pull/3888  and https://github.com/openmicroscopy/openmicroscopy/pull/3894


Whoops, will do


Oops.


Implemented "answer_on_next_confirm($answer)" method to react on each confirmation dialogue individually.

Changed to three-argument form of "open" on line 318 (according to 'Perl Best Practices' page 207) and added error messages.


That would be it from me.


thanx for taking care of this finally. I saw it the problem before and the last time there was a prerelease it worked. but better handle it explicitly ;)


\o/


You could use state resolves here as used in the contacts states? This would also remove the use of $scope.$apply() as the data would already be loaded. It could also be a good opportunity to try using the loading bar service recently merged?


bdfdn rfr


hjytkrktytyy


ejtkuklyxdtmkymkhmkytmktykyky


Maybe, I didn't understand you properly.


Let's put it in a separate directory


> I think we should still always show the most recent timestamp

It still does, at least for me on chrome.

> Using :hover is risky as you have to hover in precisely the right place
and can only see one timestamp at a time.

We can use :hover to show all timestamps when hovering over the
messagepanel. Currently it shows the timestamp when you hover over any part
of the whole message tile, so im not really sure that counts as "precisely"
the right place.

> I'd suggest that we do onMouseEnter...

What's the advantage of using onMouseEnter? I find CSS for this clearer and
easier to implement and change

On Fri, 24 Jul 2015 10:26 Matthew Hodgson <notifications@github.com> wrote:

> I think we should still always show the most recent timestamp. Using
> :hover is risky as you have to hover in precisely the right place and can
> only see one timestamp at a time. I'd suggest that we do onMouseEnter for
> the whole messagepanel to show them, as we do on angular.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vector-im/vector-web/commit/c726257e9b0ba30a4fabb7c3a6bbfaee23c1fe16#commitcomment-12339353>
> .
>


LGTM!


That was a typo, my bad. The following commit renames this correctly (in this pr)


will rename plat_def.h to mt8173_def.h in next patch.


No data updated ?


Add a comment in the empty sections.


No need to write if it's empty


Where did you use this class ?


All echos should go in simulator


This seems working to me:
![screenshot-gitlab com 2015-07-24 12-12-19](https://cloud.githubusercontent.com/assets/207895/8872434/45f6f1de-31fd-11e5-9b72-11717d484e40.png)

while if we visit https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.rubocop.yml#L50-L55 nothing is highlighted


:+1: Thank you!


Remove this file from git


Don't commit commented out code.


Sure.


I agree with @a-detiste for releasing versions 2.15. About `native` vs `quilt` issue, I don't really know what that means and would be the correct thing to do in our case...


Thanks Dave. could you add some building blocks resp. requirements that you see fitting?


Indentation ? Simple "=G" command in vim


DB handler not used yet


It was a bit early to bump to v1 so I have no choice but to bump a major again


Please, what is the pass for de 7z archive. I would like to compare. Thanks.


empty loop ?


Thank You.  $currencies->calculate_price does not deal with multi-currency shops.


Could this have an optional `assurance` parameter. The two variations of this pattern are:

--

> - item
> - item
> - item

--

> - item
> - item
> - item
>
> Assured by independent validation of assertion


users -> users'


Camel notation


Can we do it in the simulator?


Data updated but store data only for single post/photo


[work in progress]


This should go in the simulator


started doing this because it wasn't picking up the layout with the same name inside `layout-land` folder.
had to rename the second layout with the `_land` prefix


$model->load($modelData) - does not work.


Doesn't this change mean that if a socket is closed, with a non-6 byte binary string, then the stream is still left open but self.connected is now reporting that it is *not* connected?

Also the APNS docs says that in the case of a "10" response the stream should be dropped and re-connected, and that doesn't look it happens

(I'm not normally a python developer, so I'm hoping I've read this right...but let me know if I haven't)


Should also be in `prepareForDisplay()`, seems it got lost there during editing. Thanks. :)


can't this part be executed after print_records?


by sambhav


The 'event' probably was not correct name in the first place, as it was describing an operation with certain duration, not a single point in time. The interface with getStarted() and getEnded() suggests that; if you want the history log used this way, I'd suggest having single getTime() and getType() that would return enum { START, END }.


It's already there. :smiley:


line comment


This change makes no sense, let's drop this commit while merging.


OK to write the test like this given the chances are very, very, very low to find a file named after the UUID.

It is possible though to make it more deterministic: just create a random directory inside temp, and then use this directory instead of serverConfig. You're 100% sure you'll never find any file in this one.


ok... but i have done every thing i a single file for now!


This is great! It will also be useful for highlighting desugarings.. Right now that's normalizing, which is clearly too much effort.


This is a fucking lovely commit! :star2:


@ferki There is a bug i just detected. This creates an object of the wrong class.


TeamCity xBIM :: XbimScripting :: Build [Build 26](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2128&buildTypeId=XBIM_XbimScripting_Build) is now running


TeamCity xBIM :: XbimScripting :: Build [Build 26](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2128&buildTypeId=XBIM_XbimScripting_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:37


UIUnityRendererをunity5.1で使わせていただこうと考えているのですが、unity5.1には対応していますでしょうか。
対応していなければ、いつ頃対応予定か教えていただけますか。
よろしくお願い致します。


Do we need this given that we're only interested in versions >= 1.3 ?


Isnt this at the wrong place?
https://github.com/abcminiuser/lufa/blob/master/Projects/USBtoSerial/USBtoSerial.c#L236

Dont you have to place it here?
https://github.com/urjaman/fast-usbserial/blob/34dc4f0e1c752130034ed8c361e069f4323dae0a/fast-usbserial.c#L278


Ah, sorry, my mistake.


Good addition! One remark, as this is a heavy action (deleting a subweb), maybe it's good to add a Force parameter. See https://github.com/OfficeDev/PnP/blob/dev/Solutions/PowerShell.Commands/Commands/Principals/RemoveGroup.cs for an example.

Also don't forget to commit the project as it otherwise doesn't see the new class.
You can just add that code, and sync your branch to your repository. It will automatically update the pull request.


Shouldn't the above be Representation.REF?


Too harsh! Make it a bit nicer?


De donde sale este id?


Using interface here is very important - otherwise Weld will call DefaultCacheManager default constructor!!!


@flavour Please merge


1. Real - time usage of Linux 不应该翻译为 实时 Linux 吧。
2. 系统的相关消息 -> 系统的相关信息。


avoid infinite loops


@musamusa minor stuff, i think we should use a suitable name for these:
      `"kmOUT": "60331",
      "kmIN": "60550",
      "estimatedKM": "110",`

like distanceBefore, distanceAfter, estimatedDistance


We need to get rid of this... It calls manager.stop behind FileStoreCacheManagerProvider's back


looks like there is a misspell


Reverting Changes


Wonderful! I can stop swearing everytime I add a new site and provision only to discover a DB was wiped!


@czechboy0 please check this... Dunno why but lately I have some really hard times thinking while programming 😑


The formatting in this file does not look right. Is it gihub that messes it up? :)


@erichuanggit  Fixed in 7b005ff.


1. 不同提案 -> 各种新的建议 （Various new proposals）


Is there any reason why we include retired encounter roles?


1. 软中断 -> 软中断线程 ？


We usually discourage wild card imports https://wiki.openmrs.org/display/docs/Coding+Conventions


I don't see why you added this directory and left 'behaviour' empty.


This calls `CacheManager#stop` behind `FileStoreCacheManagerProvider`'s back.


:D Iestājās lielais P*ĢEC.


It is extremely important to use an interface here. Using concrete implementation will result in calling default constructor. This in turn will register a `DefaultCacheManager` in JMX tree. Note that we don't own this unwanted instance...


```javascript
if !(isNil "_sound") then {
    playSound _sound;
};
```


1 CacheManager per app


こちらUnity5.1,NGUI3.9で動作しております。
よろしくお願いいたします。


```javascript
_type = if (count _this > 1) then {_this select 1} else {"INFO"};
```


It's better this way than to break semver.


```javascript
_sound = if (count _this > 2) then {_this select 2} else {nil};
```


Fabric should get options for connecting knext

This is not Functional style

What is it?:
```javascript
knex.cache = {};
```


Makes sense.


honor the 80 char width, please.
```python
        base = self._cmd.cli.base
        self.assertResult(base, itertools.chain(
              base.sack.query().installed(),
              dnf.subject.Subject('lotus.x86_64').get_best_query(base.sack)))
```


 - potrzebna baiej jest metoda
boolean CanMove(double x,double y,int ruch)


```javascript
[
    format ["Please wait %1 seconds before you spawn another vehicle", (_unit getVariable ["sos_vehicle_spawn_delay", 0.0])],
    "SILENT",
    "sos_warning"
] call SOS_fnc_showMessage;
```

we can format this so we don't need _message


Btw, I think this should be referencing https://github.com/NRGI/resourceprojects.org-frontend/issues/105 not issue 100


The logic looks solid, I'm just not sure about all those casts. Lately I changed my opinion and now I think we should really minimize the places where we force unwrap optionals. I think it's better to ignore a value (and print an error) than to crash the whole app. So here, when you're casting, I'd do something like

```
for (key, value) in json {
            guard let blueprintID = key as? String, let commitsArray = value as? [NSDictionary] else {
                Log.error("Couldn't parse key \(key) and value \(value)")
                continue
            }
            resultsDictionary[blueprintID] = commitsArray.map { Commit(json: $0) }
}
```

(Not sure this compiles, please try it :laughing:)


```javascript
if ([] call SOS_fnc_isMember) then {
```


Any reason for including retired encounter roles?


1.1 Ajuste de tamaño del iframe que alberga el form del sistema de reservas y redondeo de los bordes para continuar con la estetica de la web.


Hey @rheoli, I just started to look into balder, found this fork, and am wondering if you have any plans to open up a PR on espen/balder with Rails 4 support. If not, I might use a few pieces of your migration if you don't mind. Nice work here! Migrating between major versions of Rails is often a pain. :disappointed:


thanks for adding the overloads!


没有发起pull request


paket zalej na repo


An update on the user profile page.
<img width="1280" alt="screen shot 2015-07-24 at 11 52 19 am" src="https://cloud.githubusercontent.com/assets/10073273/8873035/779d0ab4-31fa-11e5-83d5-860fe24d9cf4.png">


+lppl+


I thought you are simply asserting that the encounter role is not retired. Did you intentionally include name in the above assert?


Because I'm doing one more behaviour inspection now and I need one more level here. Yes, now I understand it would be better to change this in another commit.
Should I change this commit?


ref #61


This padding makes things a bit too spaced out.
If I remove it the "Full Name" text looks off, but that gets fixed if .accounts-priviledged gets a padding-bottom of 5px


Opisz dokładniej co ma robić ta metoda i opisz argumenty


hotfix, self ack


Yes.


Almost there. The file extension needs changing to `.cql` as well


sdfgsdfgsdfg


Yes, definitely +1. license plugin's defaults for java-ish languages are very unfortunate.


These variables are just icky and disgusting in size. Next thing to do is to yank (YY) it out and move it into some ruby helper/controller like thing. Because yuck.


It is required otherwise URL would be separated via `\`


How about having something like:

import time

obelix = None

def _get_obelix():
    if obelix:
        return obelix
    else:
        return time.time()

obelix = _get_obelix()

and then we just do everywhere: from invenio.obelixutils import obelix


i need somw help with setting this all up please anyone??


Yuck. Look at this wet usage of commandline foolishness. There's probably a better way to do this.


And the irony of all this is that this could sit inside the markup


use two empty string to separate import and consts blocks


Don't bother too much with the quests right now, I'm greatly improving the mod on a personal fork, and I'll merge it once it's done


People should check out this Band caleed Red Line Chemistry or Candle Light Red


I also installed both pip and pip3 using apt-get on ubuntu 15.
Applying the above patch to both:
/usr/lib/python3/dist-packages/pip/__init__.py (line 299)
/usr/lib/python2.7/dist-packages/pip/__init__.py (line 299)
enabled `pip3 list` and `pip list` to work


Elle est reponsive, elle s'adapte au écrans


And it kinda does... in it's own special way!
TODO: Perhaps alternate the background colour and name the paths with something a little more meaningful than 'd'. And 'Page-1' etc...


"Configuration" has for me a little wider meaning. Configuration is a composition of preferences which can be changed by user. User cannot change logging configuration (I mean XML files with config) DIRECTLY. So, user can choose between three MODES of configuration, but configuration details like rotation pattern etc. are INTERNAL and user "has no power" to change them. For me It is just a bit confusing - user is not choosing configuration (he SET configuration), but he is choosing MODE, which has inside some configurable values. What you think?


Very good reasoning :+1:


And, as mentioned in chat, I guess if the `brew` script is broken then `brew tests` will be too.


Hi, this issue seems to be affecting rolling_std and rolling_var also (at least, rolling_mean seems to be okay). Can you please check?


@tsegismont a small nitpick: could you plz move `<groovy>` behind `<gradlew>` so that that the alphabetic ordering is kept?


http://docs.oracle.com/javase/7/docs/api/java/lang/Enum.html#toString%28%29 - "An enum type should override this method when a more "programmer-friendly" string form exists", but here there isn't exist.


do you mean pure olive oil?


Hi @ramondominguez, I am surprised that you are not getting support from the API support team. Are you sending your emails to developers@myob.com?

We also have a forum that you can post and ask API specific questions on - http://community.myob.com/t5/AccountRight-and-Essentials-API/bd-p/AccountRightAPIquestions.

I am looping in @keranm and @devjack who should also be able to help with the dev keys that you only need to talk to the cloud hosted API, otherwise you only need a username/password that you use to access the company file.

As for this SDK library - you can build it yourself or you use nuget to pull the library - this candidate is a beta that I am testing locally

https://www.nuget.org/packages/MYOB.AccountRight.API.SDK/2015.3.53-rc1

Shaun


V takovém stavu už je to snad dobré nebo bys tam ještě něco upravil?


We will need to add this jar to saved artifacts in our build configuration:
https://teamcity.jetbrains.com/admin/editBuild.html?id=buildType:bt345


I'm not really sure but it might cause side effects on `Gita add`, `Gita rm`, `Gita reset`, or so on?


you dont think its worth mentioning where or rather who you got this code from? and its not just this commit there are three others too. seeing you as a dev for pacaur i assumed you knew better.


![screenshot_doom_20150724_080117](https://cloud.githubusercontent.com/assets/6380147/8873253/0b9c4150-31db-11e5-95c5-ac0237f81925.png)
![screenshot_doom_20150724_080406](https://cloud.githubusercontent.com/assets/6380147/8873252/0b9a1aa6-31db-11e5-8a5a-148780089de1.png)
![screenshot_doom_20150724_075923](https://cloud.githubusercontent.com/assets/6380147/8873254/0b9ea1c0-31db-11e5-88dd-c99c34f2b8a3.png)

No bugs is testing, everything working perfect. bout mech turrets shoting on me is only a small give ismech.


np - suffered my way through :) but I can not deal with [one issue](https://github.com/jruby/jruby/issues/3053) as it's a bunch of `ClassNotFoundException` from Tomcat (on master) and I'm not sure where to start looking. although no biggie I do not think it's a jruby-openssl regression so I'll probably go ahead and release 0.9.8 soon-ish


Ransac is registration as well... Better call it ICP parameters and ransac parameters


use two empty strings to separate imports and consts blocks


Nope.
1. Why have this API for FIFOs and not for other cores? If the problem is the ``ClockDomainsRenamer`` syntax is too heavy then that syntax should be fixed. Adding a kludge on top of it is the way to autoconf-style software.
2. Factory functions are problematic as they break inheritance.


hi!
please update phpbrew with CLIFramework 2.6.
I get this error when I work with PHP 5.3.29

currently I have
phpbrew - 1.20.0
cliframework core: 2.5.3

thanks


Widzę, że projekt się rozwija :-)


use 'RE' suffix instead of 'REFEXP' like here: https://github.com/superroma/testcafe-browser-natives/blob/master/src/installations.js#L9


x i y to obecne polożenie
a ruch to numer stalej
Będzie w klasie Vehicle:
public static final int LEFT,RIGHT,UP,DOWN;


Also removed stupid whitespace


I'm not sure exactly how this works. But it sounds like the comment should be something like this instead:
```php
// Add relaxed settings config object as a dependency
```
Isn't that more correct? Because we're not adding any actual cache tags here I believe?


Vypada to pekne


Strategy in design pattern means some many ways ("algorithms") to achieve some goal. This interface is used to grab requested values from different constructed XML logback configuration files. It is strictly connected with MODE of used connfiguration (type, module or type and module) not for configuration itself. So I think adding "mode" is a better solution, more clear for readers


numer stalej w jaką strone będzie się poruszać


This pattern is not good because you are doubling the number of map look-ups. It's better to check that the result is `null`. Having said that, I don't know what is the general approach used by the rest of the code, so it would be good to have input from one of the committers or authors of this code.


make private each things, so that other can't excesses it


Does this make such a large difference that it needs a parameter?


It is common method for all implementation of ConfigurationStrategy. So maybe change ConfigurationStrategy to abstract class with less number of abstract method?


@StephanHoyer: Unless you object, I plan to fold this into master and npm publish 0.2.1


@adaur  It makes the releases correctly detectable ,ie this release ```Mr Robot S01E05 HDTV x264-KILLERS[ettv]``` without this it would detectar the release group ```KILLERS[ettv]``` ,in this case thr torrents from cpasbien havê a prefix called [www.Cpasbien.**] this strips it to get the corrrect release


Maybe just good, not "very good", because you are telling the cons. Or something like "So far I've been trying different apps and Pomotodo is my favorite one..."


@dickolsson Yes, you are right, I'll update the comment.


Can you describe with more comments here why exactly this is needed? I have a slight feeling that we're doing something wrong if we need this... I just read this change record https://www.drupal.org/node/2513810 and it sounds like if we only return `Resource` objects we should not need to do this.

Is there a documentation page, a drupal core issue, or a change log entry that describes this "early rendering" concept better?


lets write
```js
import viewport from 'viewport-list';
```
like in the [Usage section](https://github.com/kevva/viewport-list#usage)

and
```js
var getViewportSize = Promise.denodeify(viewport);
```


What is the difference between  new WriteCommandAction<Object> and ApplicationManager.getInstance().runWriteAction(ThrowableComputable)?


Yes, jobs are deleted right away. But the status is stored separately and must be either deleted or expired depending if the job was completed or not.


Please provide version number. Follow guide http://definitelytyped.org/guides/contributing.html


Take a look at WriteCommandAction's source and you'll be able to answer this question yourself.


Hey @ramondominguez,
I've just started with the team.  If you get in touch via developers@myob.com with details on the error you're getting (include the HTTP uri, headers and body) and we can take a look into it for you.


And I just wanted to remember how to commit changes


Adjusted comment to make this a bit more clear. 6d738e75176d399bfb346212081631b8ff0269c4


Ja już przestałam to ogarniać! Jest dobrze! :D


@dickolsson We need this just for user entity type and just when we have the user as an entity reference - I implemented this to avoid the `\LogicException()` from `Drupal\Core\EventSubscriber\wrapControllerExecutionInRenderContext()`. I also couldn't find any documentation that describes this concept better.


This the full URL was unfortuatly necessary, since the Anaconda build server first fetches a shallow clone of the repo. That means that conda build can't clone that repo, so we need to get the original one instead.


the project page says the current api version is 1.17 and that's what the constant above says however that's not what is in use unless this is changed as well.


This could go in 8660 common couldn't it?


Can it be separate follow-up commits?


晕.....好多代码被误格式化了......


@UsabilityEtc :+1:


@stuarthodgson FYI this is the general plan, not working yet but if you're curious worth a look...


@LukasReschke why did you remove oracle unit tests?


BTW, toplevel gcc patches should be regenerated with ``autogen -T Makefile.tpl Makefile.def && autoreconf2.64``

I can backport this pull to other branches though...


@inikulin Should we use this way to define a function inside the other one everywhere in our code?


комментарий


еще один комментарий


Хороший код)


Good point. Optimized. Thanks for review.


If it's short event handler, then it's prefered way (arrow function). If you have two function which mutually use each other you shouldn't use functional expressions (eslint will warn you). In general, you should avoid declaring named functions/f-expression inside functions. With the exception then you can't avoid closure on current context (as it's done here).


TBH, I don't think I'd ever put results of `querySelectorAll` to const in the real code, but I suppose context matter.


@lluismf do you mind sharing the reason behind removing this method? :)


Regards Razvan for this Implementation!!!


Cool


Done

On 07/24/2015 10:53 AM, Denis Rouzaud wrote:
>
> I did not look in details, but would Date/time widget
> (QgsDateTimeEditWidget, QDateTimeWidget) or relation reference
> (QgsRelationReferenceWidget, QWidget) would be candidates?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/de547adc19cad7b7bba9613538389b85a6912f61#commitcomment-12338727>.
>


tools/nuget/NuGet.exe → .nuget/NuGet.exe


On Thu, Jul 23, 2015 at 3:11 PM, KC Sivaramakrishnan <
notifications@github.com> wrote:
>
> If we have a caml_is_marked(obj) function which returns true if obj is
marked, would conditional marking just be
>
> if (caml_is_marked(obj))
>   caml_darken(val,0);

Ah, that's not what I meant, my comment was a bit misleading.

Conditional marking in that sense is already done, since `caml_mark_object`
returns 0 if the object was already marked and caml_darken is a no-op. I
prefer this to exposing a `caml_is_marked` function, since it's very easy
to use `caml_is_marked` incorrectly.

In particular, it is tempting to use caml_is_marked in the write barrier:
after all, there's no need to log the store if only unmarked objects were
being mutated. However, this scheme is racy, since the property of being
unmarked becomes false at arbitrary times. So, I wanted to only allow read
access to the mark bits by code that has committed to marking, hence a
`caml_mark_object` function that returns a boolean but no `caml_is_marked`.

What I meant by the comment in 23e5bf0 was that it might be good to have
a quiescent phase for the GC. In the current design, the next GC starts as
soon as the previous one finishes, but it might possibly be good to have a
quiet period where no GC is done. During such a period, the write barrier
could be a no-op.

I avoided this, because the benefits are slight and theoretical, but the
drawbacks are some horrible race conditions around phase transitions
between quiescent and normal marking. In particular, getting all domains to
agree on what phase the GC is in is a bit painful.


Great! Together with #49 this solves #33


Whoops, GistReporterTest was meant to be part of my next commit!


thanks!


`'/apsis/ui-components*/'` isn't found on OS X.


merged


Добавить новую строку


Okay, now I will know. :smirk:


Hi  SCiite, du hast nicht mehr den aktuellen Stand von dem Dokument, wir haben schon einiges verändert und umstruktuiert. Ich würde dir empfehlen den neuen Stand zu ziehen oder einfach dem Master-Branch in deinen zu mergen.

Gruß Stura


@incuna/backend Review?


I don't know exactly where do you want to point, but if you want to reach the Monitoring Manager IP you should use  something like:
     ```$brooklyn:component("monitoring").component("child","manager").attributeWhenReady("NAME_OF_THE_SENSOR_YOU_WANT_TO_READ")```

In order to know which is the right sensor, please deploy the current SeaClouds YAML and check which returns the information that you need.

Thank you!


looks like you have spell error


reack


just `var size`


Fixed in 0d42e3b9654a33b610f3d1df7265f167ef2b75ae, thanks !


Because job execution takes about 1hour then and sometimes fails randomly :speak_no_evil:


I'm not sure if I understand why it should be a `Lint` cop. The `EndAlignment` and `DefEndAlignment` cops are in the `Lint` category based on the idea that an incorrectly indented `end` could be a semantic mistake where the user has aligned it with the wrong thing. I guess this could happen for `rescue` and `ensure` too, but is seems less likely. If we put it in the lint category, it follows that we should also set `AutoCorrect: false` as we do with those other cops. I suggest that we put this cop under `Style`.


Probably wants `sudo: false`


I don't think that this needs to be so far down this file as of django 1.8


Actually it seems a bug of `Method#to_proc`.


@a-detiste, @dbent please review the copyrights in the section to make sure I was correct.
Feel free to commit updates to them if I'm wrong somehow. Thanks.


This check will always fail. There must be:
if not 'instance' in data['ansible_gce']:


Is it possible to not build it on each `ant dist`? It takes precious time


I think with `Django 1.8` it's not required anymore to create a main database


I decided to re-license my piece of code under MIT to avoid problems with license imcompatibilities.


Sí, ya habíamos hablado eso ayer.


They got 3 email unanswered, ok , let me send another email, thank you all folks


Y acá no hay revs.


I agree, competence is so overrated. Who decides if someone is competent or not? Experience is also useless, a junior developer with 3 months of experience should be able to tell a senior architect with 15 years of experience that he's wrong, regardless of who's right or wrong.


Oh nevermind this is the test database


okay - as long as the :hover applies to all timestamps and is triggered on any message tile i'm happy :)

> On 24 Jul 2015, at 11:10, Erik Johnston <notifications@github.com> wrote:
>
> > I think we should still always show the most recent timestamp
>
> It still does, at least for me on chrome.
>
> > Using :hover is risky as you have to hover in precisely the right place
> and can only see one timestamp at a time.
>
> We can use :hover to show all timestamps when hovering over the
> messagepanel. Currently it shows the timestamp when you hover over any part
> of the whole message tile, so im not really sure that counts as "precisely"
> the right place.
>
> > I'd suggest that we do onMouseEnter...
>
> What's the advantage of using onMouseEnter? I find CSS for this clearer and
> easier to implement and change
>
> On Fri, 24 Jul 2015 10:26 Matthew Hodgson <notifications@github.com> wrote:
>
> > I think we should still always show the most recent timestamp. Using
> > :hover is risky as you have to hover in precisely the right place and can
> > only see one timestamp at a time. I'd suggest that we do onMouseEnter for
> > the whole messagepanel to show them, as we do on angular.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/vector-im/vector-web/commit/c726257e9b0ba30a4fabb7c3a6bbfaee23c1fe16#commitcomment-12339353>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


@oryband this is a bug - not checking `SuccessEnabled`


Not sure if the section is important, though someone may be interested in the program's background.


What's the reasoning for this?

`originalBuilder` no longer means `originalBuilder`.


@oryband comments should be switched


I did and it crashed :-) But i tested on OS X. Will give it a go on linux someday soon.