Instead of comparing the elevation, you could check the id of the waypoints (e.g. `self.waypoint3.id`).


It's alright:

> ;domain=domain (e.g., 'example.com' or 'subdomain.example.com'). If not specified, defaults to the host portion of the current document location (but not including subdomains). Contrary to earlier specifications, leading dots in domain names are ignored. If a domain is specified, subdomains are always included.

I just tried it locally by editing my hosts file and it worked fine as well.


D-d-d-duplication? A kitten just died somewhere


another one


Hardcode!


да


tortue ninja !!!!!!!!!!!!!


We want this behavior be flexible.


Move it to resources


section +  wrapp
Данный контент можно вынести в отдеьную в секцию.


what if `offset` can not parsed to an int? maybe you could create a validator function like for the id:

https://github.com/c2corg/v6_api/blob/ffbf9e0db9f145f79ff62c10f99922bf9947c638/c2corg_api/views/__init__.py#L33
https://github.com/c2corg/v6_api/blob/ffbf9e0db9f145f79ff62c10f99922bf9947c638/c2corg_api/views/waypoint.py#L19


KOWABUNGA !!!!!!!!!!!!!!!


MIAM


I'd rather go with a dummy method which would have `return "Trello";` implementation, but it's not a big thing


#ifdef __cplusplus on installed headers of the series


Maybe a middleware plugin or something.


Schema name should not be something set in stone. Please make it a parameter.


@evvicente vaya chorrada de commit.


- Please use debugstr_*()
- Add spece between WARNING: and oplk_set....

````diff
-PRINTF("WARNING:oplk_setNonPlkForward() failed with error (0x%X)\n", ret);
+PRINTF("WARNING: oplk_setNonPlkForward() failed with \"%s\"\n(Error:0x%x!)\n", debugstr_getRetValStr(ret), ret);
````


Обернуть в тег header, post-preview__desc туда же


@runcom Need to fix docs also though?


Ah, I think I get it: like this? And what it's saving is having to re-build the top-level array `present`; it just replaces the references in each array element, right?

```
    present.map!(&:last)
    if present.include?("head") && present.include?("head block")
      problem "Should not have both `head` and `head do`"
    end
```

https://github.com/apjanke/homebrew/commit/cd8c73316bd07b98da410ca93022f72137a58c03


тег time, возможно чистовое значение, плюс возможность проставить дату


Adding this made travis completely crash before it could run anything...


This is not useful since we give the value to the token server it will return the ttl value.

What you can do is trust the duration rather than the ttl if you see some cases where the token server can have a different behaviorur


Please replace `_wsmc_add_uri_to_list(&(options->selectors), uri->query)` with `wsmc_add_selectors_from_str(options, uri->query)` to improve code quality.
Then I will happily merge your pull request ;-)


How do I do that. I'm new to github
On Oct 5, 2015 9:31 AM, "Eric Douglas" <notifications@github.com> wrote:

> @antonio015 <https://github.com/antonio015>
>
> can you add your profile in the issue #31?
>
> Thanks! [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-source-society/computer-science-and-engineering/commit/fa79ae65c2a9b607364f2da2b76eeafde00c445d#commitcomment-13593313>
> .
>


Please also do this fix in the embedded CN demo.


If the read() methods reaches the end of the file, an empty bytes object is returned. Methods rstrip() and decode("utf-8") have no problem with the empty bytes object. Method decode() can raise an exception in case there is a decode error, so I've added a errors="ignore" parameter into decode() which causes characters which cannot be decoded are silently ignored.


good point, thanks!


I wonder if on error cases the ``reprs`` output array should not be cleaned at this layer. It looks like ``_sol_oic_resource_type_handle()`` leaks data if this call fails.


Это ссылка:)


Hi @wizardbeard
Thanks for your comment. Some remarks:
1. This plugin is not active anywhere unless you deliberately want to use it.
2. Then if you want to use it, you read the doc of the plugin and eventually activate the "MOUNT_SUDO" option if you really need to. In the docs, it explains how you have to actually configure sudo to allow the usage of the mount command to apache. And nothing more.
3. From there all variables are sanitized and escaped.
If you have a better solution, please fill free to contribute by a PR to provide an auto-mount feature of a remote file system that does not require using sudo in any case.
Best
Charles


So i should remove some of the checks, so that it will raise a stack when provided null args?
Question, if a NPE is raised, does bukkit disable the plugin that raised the exception? Following that, if a NPE was raised in this point of the code and .banPlayer(Player, Ban) was called by another plugin, after the npe, which plugin would get disabled? This plugin, or the caller, or both?


@guptashvm This just happened--just FYI. You might want to pull. We also may need to produce facial adjaceny at some point.


Oops, sorry about that. The conflicts should be easy enough to resolve though :)


You better tested this huge code change intensively!


please revbump 1.7.3


nice code


Hardcode!


The spec allows any character to be escaped so that JSON is fine. Is it causing you any issues with other libraries or is purely for human readability?


:goat:


Will respond to this in about 40 minutes!


Do not write your code in README file atleast.


I see what you mean. The CourseOverview model uses a method from the LMS to get course image URLs. I'm unfamiliar with the course image URL code, but there appears to be a [similar method](https://github.com/edx/edx-platform/blob/master/common/djangoapps/util/parsing_utils.py#L7) in the `common` directory which doesn't rely on LMS settings. Maybe CourseOverview could use this method instead of the one from LMS?


It appears that `REGEX. nonCamelCase` is no longer being used as well, I'd suggest removing that unless you have plans for another util to make use of it.


@vilchik-elena changed in ed7e4f661c69fb86ee03de3ad61310a718bd9c92


izi ni nii mungu baba :smile:


@cehallen You can drop the images from the blocstagram repo now, since the images all come from Instagram.


@pynicolas it's possible with PHPSubscriptionCheck, i think. See https://github.com/SonarCommunity/sonar-php/blob/master/php-frontend/src/test/java/org/sonar/php/utils/SourceBuilder.java


@etarena

They may not be following the workshop tutorial.

So perhaps you could change "did you accidentally close all images?  no problem." to something like  "to generate the C0Z16 image do the following."


looks good


first ppt입니다


Looks good.


Yes! It’s like replacing `a = a + 1` with `a++` in Java.


Not `indico-team@cern.ch`?


**Test Coverage - Critical**

You should place your `beforeEach` inside a `describe` block.


interesting that you'd install purescript from hackage, not from npm.  any reason?  i think it just bit you, because it wants a more recent alex.  (-:


let's PEP8 this file ;)


@pynicolas ha! this is mistake in new grammar :) we should fix it


I guess we don't need a license header in here?


**Code Quality - Nitpick**

Try to `'use strict'` in all your JS files, in a **closure**.


*special_value_crits* are set of methods registered from different modules right now. They're used to replace values when you'll call `replace_special_values` on `$crits` object.
https://github.com/Telaxus/EPESI/blob/dev/modules/Utils/RecordBrowser/QueryBuilder.php#L26


@etarena

"did you accidentally close all images" -> "to generate the C0 image"

just in case they aren't following the tutorials in order


тут обычный div, по смыслу тут section не нужен, да же избыточен. Это некая мета информация для поста.
После него уже идет основной header статьи.


I haven't looked into the details of how the passthrough works, but I'm assuming that the entity is never unmarshalled, and is returned to the invoker as a JsObject.  If that's the case, then I believe that none of the paths in MethodConfigurationService require changes, because none of them will see the Rawls changes.


good code chitra!!!  :)


@vilchik-elena it seems to better reflect what the PHP interpreter accepts:
https://github.com/php/php-src/blob/PHP-5.6.14/Zend/zend_language_parser.y#L303


is this a `stack` thing?  if so, add a comment that sais so?


@pynicolas indeed. Let's leave it as it's then


Just use the second parameter then the check for `is_array()` is not needed.


"did you accidentally close all images?  no problem." could be changed to "To generate the test image..."


Не вижу сильных противоречий для его использования.

"Объекты, расположенные в section объединены общим смыслом.
Так же, служит для разделение текста на разделы (его часто вкладывают в article).Должен иметь заголовок (почти
всегда)"

и заголовки h2


Would be a good moment in time to add a documentation for the inserttags added by haste :)


the result would not be the same, we would get an array with one value (likely an empty string). Not what we want.


Agreed!


**Code Quality - Nitpick**

 - `isIndexCreated(…)` with-respect-to the function's naming, it's expected to return a `boolean`.
 - `isIndexCreated` and `getIndex` perform the same function.


Hello .. i just wanna know how did you make the vector ? can u tell me the whole procedure to convert png to vector(xml) please if you dont mind ?

Hope you reply ..

thank u


**Code Quality - Nitpick**

You should parse to JSON, the content of the jsonFile once, when the file is loaded, instead of having to do it in every function.


@vilchik-elena We need at least to make the scanTree method public (SourceBuilder is in the same package as PHPSubscriptionCheck). And I don't really understand why we need anything named Check for that: it's just about visiting, not about creating issues. If it's doable, I prefer to keep different responsibilities in separated classes.


Shouldn't need this require.


I don't think its possible for `^foo|bar$` to ever be generated as the regex. I did some playing around with different glob patterns and read through the code again, I think this is a non issue. I definitely see what your saying about the matching precedence though.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1373 of 1373 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)


It looks like this is undoing changes made in https://github.com/dotnet/corefx/pull/3615


post__desc, quote  и тд.. лучше не давать названия классов, а работать  сними каскадом. Иначе контент-менеджеру то же придется их проставлять добавляя текст. Что не очень удобно.


Nit: _headerBytes rather than headerBytes


Sounds good. Yeap ddl is in the building ;)


:question: Do you use send the token out in the response?


@houtanb Could you please revert this commit. It breaks steady state finding. We have to rethink our approach


@pynicolas ok


"You sure you got today's codes?" (the url didn't change...)


Corrected


Is this #if needed?  The contents are commented out, and presumably Console.WriteLine won't actually be used in any build.


Corrected.


cds-indico-extras?


more extras...


and more...


Is there any swagger doc in orchestration that would need to be updated after this change?


Please specify visibility.


:blush: missed some test debug code.


Same #if question... I'll stop commenting on this.


!mustfix Move this to the bottom of the file, that would make more sense.


Done with my changes, now about to merge. *Puts goggles on.*

Would be good to have a chat channel for FLIF, to better synchronize development.

I joined `#flif` on IRC just now. If you are not on IRC, I suggest creating a channel on matrix.org or gitter.im.


sorry was replying in the wrong place.


Consider extracting as constant at top of file rather than local.


I think I previously already asked this, but just in case, asking again: these strings don't need to be in the .resx?


Is this necessary?


build fixes do not need revbumps


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (835 of 835 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-24.31%25-green.svg)


:yellow_heart: This controller should be renamed to reflect its purpose.


Adenilton,

.Após a ultima atualização esta gerando um erro no Qrcode na nova função segue o retorno:

Erros da validação : The 'http://www.portalfiscal.inf.br/nfe:qrCode' element is invalid - The value

Funcão :infNFeSupl

Estou realizando o teste na NFCe quando desabilito gera a NFCe normalmente.


the method or the copying to the specs?


I don't think we need to checkin all the binaries, esp. twice.


:yellow_heart: `authRoutes` should be renamed to reflect its purpose.


no comment


Create a new issue?


Corrected


Not sure if its a good idea, to force kernel permisive ;)


begin


Consider logging here and exit?


Can anyone give a confirmed commit that it stopped working at? or a most recent commit where it was confirmed working?


:green_heart: You did an excellent job here (and in the other files)! The code is modular, cleaner and easier to debug.


:green_heart: Good security measure.


:smile:


@pmuchowski can you write a test for creating all enrollments?


omg the tabs


Alternative - slightly shorter image with thinner lines:
![Design Sprints](https://i.imgur.com/5OePzk6.png)


they do, if the dependency is part of RDEPEND. Otherwise a system cleanup might remove that RDEPEND, because it is not in VDB and the installed package breaks.


:yellow_heart: We may need to perform error-handling for the database connection. This could be abstracted into a database config file which will improve debugging by logging the database connection status.

You can place the database config file in the `config` directory.


Avoid parameter lists longer than 5 parameters.


!mustfix Might be better to extend `prompt.Dialog` to be able to handle nested structs.


:yellow_heart: The name of the method could be improved by defining the role is serves. For example, `authMiddleware` would help identify this method as middleware for authentication.


But in the 1.0.0-beta4 composer package it does not appear anymore when calling `bin/magento`. Maybe it will reappear in the next beta released


Fixing implies that the code was broken, since it wasn't you merely 'cleaned it up' (which is a coder way of saying 'my way is better' :P)


this was the merge issue


Copying to the specs. Why make a duplicate copy of DynamoLocal Ruby? You should be able to run directly from the gem just fine. Even if that somehow doesn't work, symlinking is faster than making full copies of everything.


Any idea when the 0.0.6 release will be out in the Maven repository?


Для некоторых локаций 30 км может не подойти. Лучше вытащить из JSONObject'а поля координат bounds.northeast и bounds.southwest и оценить радиус точнее.


You don't need to write a copyright notice to the generated file. But there should be a message, that this is a generated file and that it's  not supposed to be edited.


Atualizou seus shemas? Note que em https://github.com/adeniltonbs/Zeus.Net.NFe.NFCe/tree/master/NFe.AppTeste/Schemas foram incluídos e alterados alguns shemas.


test


life is bug


@Steinsplitter is German and Dutch attribution correct?


Instead of taking an option and modifying the behavior of the method on error, could you just let the method return true or false depending on success of the build, and then check in the calling code and fail or continue with a warning?


!mustfix Set to `salsaflow.modules.issuetracking.github`


It already logs what happens between trying to connect to the db. Any exceptions raised from buggery during the db connection would get raised. So you want it to Bukkit.shutdown() if failed to connect to db onEnable() ?


I don't think we should be listing all AEAD cipher suites in the core spec.  Sticking with what the working group considers MUSTs and SHOULDs would be best instead of including all ciphers.  I think this set is ECDHE_RSA and ECDHE_ECDSA, with AES and CHACHA20.


Niet nodig, als de parse functie niet werkt return hij een TeamPart.None. Die staat niet in de switch.


I suggest you create a directory `GenericPython` and put an `AST.hs` file in there.


!mustfix Use `tracker.updateIssues` instead.


> Learn C!

They said.

> It'll be **fun**!

They said.


I'm just curious:
Why was the coloring of the numerals removed? It made the context of numeral and sidenote more obvious for my understanding (and was *one* reason I got interested in this project).

I certainly respect your decisions … just curious and the commit message doesn't elaborate on the reasons.


There is a `parens` combinator that permits this to be written slightly nicer:

```
text  "for" <+> ppr i <+> text "in range" <> parens (ppr limit) <> text ":" </>
```


!mustfix Might make sense to use `tracker.updateIssues` as well...


Wouldn't it be better to check, if the command failed, instead of pattern matching the git revision?


It's not clear what's going on here. Did you squash these migrations?


Het sturen werkt, lekker! Deze code nog even implemeteren in een InteractionRequest zodat het async uitgevoerd wordt. Ik zal er vanavond naar kijken^^


sir plz send the last week codes


Boot up TWRP, mount the system partition, create a symlink for /system/vendor -> /vendor
Attempt to decrypt. If it works, then you do a ps | grep "recovery" to get the PID of the recovery process.
cat /proc/PIDgoesHere/maps and figure out what libraries are in use from the system partition. Those libraries will need to be added to the appropriate path in /vendor.

If it doesn't work then sometimes you have to mount some other partition like a firmware or modem partition. This will give you a rough idea of what files you will likely need to make it work:
https://github.com/TeamWin/android_device_lge_hammerhead/tree/android-5.0/recovery


It's well known that the files go to the /f/ directory. @rosskarchner your thoughts? It's been this way since I've been here. What's the reasoning behind putting everything in the /f/ dir instead of the WP created directories?


Interesting, wouldn't it be specific to the XS instance (which is crashed)? Either way I like that change.


This commit appears to do nothing other than undo [a previous commit](https://github.com/likeastore/ngDialog/commit/269c185413b5c805f68b6fc9bf3e4107f9c5fad6) whose changes I need to keep ngDialog working in IE. Surely this is a mistake?


DO NOT DELETE THIS BRANCH, @yasharora73 still needs to reference this information until all the accounts are moved to a dedicated server.


@jlhanl DWMR (done with my review). Nice code here. Done!


The german one is correct :-)

Date: Mon, 5 Oct 2015 08:34:56 -0700
From: notifications@github.com
To: dewkin@noreply.github.com
CC: steinsplitter-wiki@live.com
Subject: Re: [dewkin] grunt-banana-checker: enable 'requireMetadata' and make pass (4014280)

@Steinsplitter is German and Dutch attribution correct?

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


Have you guys tried on the latest mater? I was fairy sure this had been resolved


This may be poorly named. Its used to format the expiry string before its put into the db.
Renaming to getExpiryString.
Corrected.


cool. looking good. I'll try to convert my mark compact when i get back to figure out how to make mark() general to any objects.


listo


It describes a valuable piece of behavior. TBH, most of the text there is not that useful for an average user, but a backend author might find it interesting (see also #398).


This belongs to #187


Okay I fixed it in the tree.


Commented out for now. Will follow up later.


Thanks, will look into it.

Just tested the RB access QueryBuilder with SUPERADMIN account .
The access on one of the permissions (the current superadmin user does not have thatpermission) limited by specific field, e.g permission ACCESS:test and value on contact recordset set as "First Name" = Georgi.
Then the superadmin can see only records with first name Georgi. Traced this to `CRM_ContactsCommon::crm_clearance` where superadmin gets all clearances (including the limited one) so his results are limited.


@dkward2 The function is already vectorized at this point. Would the code be easier to read if a series of `if()` statements rather than `ifelse()`


:)  We probably should also create a log tarball (`*&.{out,err,log}`)?


Yeah, I don't know what happened here... :-/


@pixolin This commit, including that change, is related to specific requests from Mr. Edward Tufte.


Moved there.


Thank you. Thank you very much.


Hi @loicraux, you are right. That's a bug. Please feel free to add an overload for the event map syntax. Should be something like this:

    on(eventMap: {[index: string]: Function}): any;


Yep. Sorry about the ongoing brain fart there. :3


This is not going to work when `base` ends with `/`


You can use `$this->redirectToRoute()` method


Local TMs are also writable. The difference is that local TMs are automatically updated and external ones are not. Given the choice between `IS_LOCAL` and `AUTO_UPDATES` I prefer to stick with the former.


After this commit I'm now getting node errors on deployment to dokku (heroku-like environment), apparently because nan is out of date. Has anyone seen this? Is there a strong reason to to node 4?

```
$ git push dokku muni-census:master
Counting objects: 178, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (74/74), done.
Writing objects: 100% (178/178), 21.96 KiB | 0 bytes/s, done.
Total 178 (delta 145), reused 131 (delta 104)
-----> Cleaning up...
-----> Building muni-census from buildstep...
-----> Adding BUILD_ENV to build environment...
-----> Node.js app detected
-----> Requested node range: 4.1.x
-----> Resolved node version: 4.1.1
-----> Downloading and installing node
-----> Restoring node_modules directory from cache
-----> Pruning cached dependencies not specified in package.json
-----> Node version changed since last build; rebuilding dependencies

       > bcrypt@0.8.5 install /tmp/build/node_modules/bcrypt
       > node-gyp rebuild

       make: Entering directory `/tmp/build/node_modules/bcrypt/build'
       CXX(target) Release/obj.target/bcrypt_lib/src/blowfish.o
       CXX(target) Release/obj.target/bcrypt_lib/src/bcrypt.o
       CXX(target) Release/obj.target/bcrypt_lib/src/bcrypt_node.o
       SOLINK_MODULE(target) Release/obj.target/bcrypt_lib.node
       COPY Release/bcrypt_lib.node
       make: Leaving directory `/tmp/build/node_modules/bcrypt/build'

       > scrypt@4.0.7 install /tmp/build/node_modules/scrypt
       > node-gyp rebuild

       make: Entering directory `/tmp/build/node_modules/scrypt/build'
       COPY ../scrypt/scrypt-1.1.6/config.h
       SOLINK_MODULE(target) Release/obj.target/copied_files.node
       COPY Release/copied_files.node
       CC(target) Release/obj.target/scrypt_wrapper/src/util/salt.o
       CC(target) Release/obj.target/scrypt_wrapper/src/scryptwrapper/keyderivation.o
       CC(target) Release/obj.target/scrypt_wrapper/src/scryptwrapper/pickparams.o
       CC(target) Release/obj.target/scrypt_wrapper/src/scryptwrapper/hash.o
       AR(target) Release/obj.target/scrypt_wrapper.a
       COPY Release/scrypt_wrapper.a
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/crypto/sha256.o
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/crypto/crypto_scrypt-sse.o
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/scryptenc/scryptenc_cpuperf.o
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/util/memlimit.o
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/scryptenc/scryptenc.o
       CC(target) Release/obj.target/scrypt_lib/scrypt/scrypt-1.1.6/lib/crypto/crypto_aesctr.o
       AR(target) Release/obj.target/scrypt_lib.a
       COPY Release/scrypt_lib.a
       CXX(target) Release/obj.target/scrypt/scrypt_node.o
       In file included from ../scrypt_node.cc:28:0:
       ../node_modules/nan/nan.h:324:27: error: redefinition of 'template<class T> v8::Local<T> Nan::imp::NanEnsureHandleOrPersistent(const v8::Local<T>&)'
       NAN_INLINE v8::Local<T> NanEnsureHandleOrPersistent(const v8::Local<T> &val) {
       ^
```


lol no special reason just the one I thought of, and this was the problem #67. Feel free to add more domains :)


Thank you for this one! (This is what happens when you work too long with a chunk of text...)


Done. It will also provide some sample for users to tweak.


Thanks.. fixed it :)


@inikulin  Do you like the following?:
```js
import hammerhead from '../deps/hammerhead';
import clone from './clone';
...
//API
var positionUtils = clone(hammerhead.utils.position);

positionUtils.getIFrameCoordinates = function (iFrameWin) {
...
};
...

export default positionUtils;
```

where `clone.js` is
```js
export default function (obj) {
    var target = {};

    for (var field in obj) {
        if (obj.hasOwnProperty(field))
            target[field] = obj[field];
    }

    return target;
}

```


Haha


Didn't JurgenNL translate #1?


No, this is wrong. Ignore.


Kan je niet


r+


When the DOI has been registered, this should be a POST request to update the existing registration.  The expected response is 200 OK.


When the DOI has been registered, this should be a POST request to update the existing registration. The expected response is 200 OK.


The deploy URL needs to remain as Azure in our repo.


You should use  ```form_start()``` and ``` form_end()```


left pdb in there, don't take this commit.


Thanks for the patch. Unfortunately I get segfaults when training a model when specifying regularization parameters (e.g. -p c1=1.0 or -p c2=1.0). If I don't specify those arguments, training seems to be running and terminates as expected.


@jdubois just a crazy question,  we could have done `<appName>Properties` for this rite? Not trivial but just a thought


Oops, that seems to be a mistake on my part, good catch!


Please provide a meaningful description. Count of what?


Thank you for your fast reply. I guess, it was too obtrusive.
(Certainly not a loss for people with color blindness.)


i cancelled the original pull request, verified that github.com/thomas-couchbase/showfast/config.json does not have a change to port 8800, resubmitted the pull request but still see this as a pull request change.


why bother assigning to `exprs[n]` here?


Indent your sql would be nice


ditto


nothing has changed in this file, but the line endings were ^M - which does not follow openstack style guidelines. http://docs.openstack.org/developer/hacking/


Clearing my dokku cache fixed it, never mind :)


The name of the method is misleading. As we discussed better split up checking for the need to rebuild, testing for being in a git repository and the actual building. Methods should not have different behavior based on parameters, but have a clear task and return a clear value.


It wasn't mentioned anywhere in the javadoc if it did use time or not in its comparison.
Tested it, it in fact does.
Corrected


It's very difficult to review changes when every line in a file has change. This is likely due to incorrect line-endings. Please avoid such changes. If a file you are working with has incorrect line endings then provide a separate patch for those changes, indicating they are white-space only. Thanks.


'to a production' environment?


Changed so `--tm` usage is now optional.

Also refactored to ask for TM name (actually the TM server identifier in settings) instead of TM index is also done. So also removed `--tm-name` as a consequence since now we fully rely on the TM name being properly set in the settings for the specified TM (if any is provided, otherwise the default external TM server).


Well, looks shitty too :laughing: I think we need some bike shedding cc @churkin @helen-dikareva @AndreyBelym @miherlosev @LavrovArtem


href needs to point to Azure in our repo.


You can use `$this->redirectToRoute` method


@frenchcomp okay, see https://github.com/php/php-src/commit/35d0405c4790f0ce668c9e1b8b05197e55d29a05 ;-)

Thanks for pointing out that we actually can allow random $this. Was too focussed on scope initially to notice that $this isn't really dependent on that. Thanks :-)


@mkonicek thoughts on unifying:
1) AlertIOS / Dialogs(android) / Toast as Alert.
2) ProgressBarAndroid and ProgressViewIOS as ProgressBar
3) LinkingIOS and Intent. as Linking
4) ActivityIndicationIOS and (xx on Android) as ActivityIndicator


- 이거(`changedArray`) 안 써도 됨
- focus 된 채로 맨 위로 가면.......


... docker container. (More information about docker images can be found here https://docs.docker.com/userguide/dockerimages/. The instructions ...


As an option, we can avoid using shortcuts if there are conflicts,


preventing empty spans seems better to put in simplification to me. i don't feel strongly except for the increasing complexity and line count of this code, which makes it difficult to approach.


This was a very good one :)

Good job.


Why not use constants or a static method for retrieving the contract_types in the entity `Job` ?


Or use method shortcuts instead of util objects


There is probably a better name for this than s. Why not currentRoomId?


agree)

We can just get rid of the `hh-` vars and use `hammerhead.utils.someUtils`


this should be dummy_[13]?


oh duh. ok thx :)


Working with MongoDB 4 now


Magic Number, what color is this?


This feels like it should be worded along the lines that an exposing a port is something done on the container rather than it being exposed on the host.


More magic numbers?


“左键移动地图”


You should probably leave in the TODO regarding this.


Missing space between `~`
And don't add extra space to twig filter
http://twig.sensiolabs.org/doc/coding_standards.html


'as parameter' -> 'as a parameter'.


"In order to understand these tools, we must first understand their history."

I also like Haas's point that she mentions the history presented in her paper "does not in fact have to be the 'correct' history in order to support [her] larger point (143)." She goes on to mention that when she was interviewing people, their stories would often conflict. The point was that these objects **have** history, not that the history is 100% accurate (144).


oops this is no longer valid ;)


```
local last_value="$1"
shift
for value in "$@"; do
   if [[ $value != "$last_value" ]]; then
    found_many=1
    break
  fi
done
```
is both cleaner and faster


r-
 * that will create incompatible json output (so it need `analyze.py` and `parse_results.py` changes)
 * you should not perform flattening of the data during scan, that should happen in output function
* as said in comment, code can be cleaner


'good' -> 'powerful'


r-

again, this breaks json output and the switch shouldn't happen inside scan function


This was related to #10.


this looks great -- thanks for putting it together!


r-, same reasons


Is this transparent?


@prudhomm: @Trophime wonders why the `CMAKE_CXX_FLAGS_*` variables are overwritten.

@Trophime To use `git blame`:
```
git blame -L 165,+10 file.cpp
```


if its moved to another branch then must to save some bytes of git and must delete this branch :)


Glad you changed your mind, my friend!


"wasn't initialized"


This is as comment.


TeamCity SimpleConfig :: Deploy [Build 1.0.37.0](http://build.mikeobrien.net/viewLog.html?buildId=2474&buildTypeId=simpleconfig&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:00:48


First, let me say thank you for this. I'll comment inline once I've taken a look.


Do you think there would be value in parsing and exposing index metadata (now exposed via system.schema_indexes)?   Previously it was associated with the column.  That data was not parsed before either, so maybe a good future enhancement.


IMHO this would be cleaner:

```python
if user:
    return
if uuid:
    ...
raise ValueError(...)
```


specified


`$T.gettext(..)`


Properly flushing out the dependencies is a good idea but none of these are needed for the spl dkms build.  The libraries and utilities needed by zfs should be added in the zfs packages.  As for autoconf, automake, libtool aren't needed at all because the tarball contains the build products of these tools.


fixes #25


does not work.


Ok. I have removed the e.g. part completely. AFAIC, the paragraph makes some
sense. My limited knowledge of the company code base doesn't allow me to make a
more concrete judgments on this one. Shouldn't have touched it in the first
place.

>> On Mon, Oct 05 2015 08:44, Dmitry Gutov wrote:

> It describes a valuable piece of behavior. TBH, most of the text there is not that useful for an average user, but a backend author might find it interesting (see also #398).

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/vspinu/company-mode/commit/ee234585b3c507fcaf968c8a2bce90c5eb7d4ac1#commitcomment-13595124


i dont know the convention here, but i feel like i'd want the name to indicate the purpose rather than dereference the hex code.


what is this file for?


@jfoug I have no idea why you did this, but it created the very bug it says it fixes. I reverted it in 265c1c5. Maybe you had a problem with hybrid mask? If so, it's not this code, it's the primary format's `get_progress()`. This code is only for pure mask mode.


@Safihre seems like the most common request in #sabnzbd chat is to revert using a % to set the bw speed but a fixed speed. now that the scheduler accepts either.. perhaps offer a way to do that or gui option in glitter to change to the user preferred method to set the speed.


wtf?


is this really the way to go? why reuse a string rather than a dom element?


If in test case no any test "test_00_create_employee" then why create this method just creating one record ?


r-, same reasons


all good


@chirag04 We've been discussing similar points internally.
1) Have to look into how similar the AlertIOS and Dialog APIs are. `alert()` should work on both platforms.
2) Agreed, these seem pretty similar, I've been thinking ProgressIndicator.
3) Agreed, these might be very similar too.
4) On Android this is actually ProgressBarAndroid (it shows a "spinner" or a horizontal progress bar based on props)


pod repo push сделал, это я не ручками


Oh, yes. He translated it. :)


Atualizei aqui porem continua o mesmo erro


remove


lk


is this 80px because that's the navbar width? seems fragile to duplicate this. Is there a more declarative alternative?


Hi Francesc -- is a value of 2 generally enough to avoid all issues with overflow? If so, then this looks good to me. Maybe it's better to pull out the factor of 2 into a top-level define like

``#define MTS_SKY_OVERFLOW_WORKAROUND 2.0f``


Can you go a little more into detail by what you mean by the first part, or an example?
Logger is @ #12


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1373 of 1373 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)


Resolved "WARNING: Title underline too short."


To generate "test report" from maven,  the plugin is  Maven Surefire Report
Plugin
<https://maven.apache.org/surefire/maven-surefire-report-plugin/index.html>
that
generate HTML report and works well with maven site plugin.

However since such plugin works with DOXIA generator should be possible
generate also other format like Markdown

I've found this article
<http://gordondickens.com/wordpress/2012/08/02/maven-3-code-analysis-and-reporting/>
that
explains how generate test report in markdown. If it works you can publish
the generated report to confluence because, as you know, the confluence
plugin  now works also with markdown format

Let me know

Regards,
Bartolo

On Mon, Oct 5, 2015 at 1:47 PM raghuunt <notifications@github.com> wrote:

> Hi,
>
> Is there a way to publish Maven Test Results to Confluence ?
>
> -Raghu.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsorrentino/maven-confluence-plugin/commit/9927bccb13e07f30b9a2172b6933cf9cbcfc3c9f#commitcomment-13589996>
> .
>


Simply `if (user) { /* do stuff with user.* */ } else { /* do something else without a user */ }`.


it is in rdepend because there currently is no way to distinguish between:
"i need the headers and the config files in SYSROOT when cross compiling" (such as here)
and "i need it for building" (such as doxygen, gcc, etc.)

using RDEPEND for this case is the only way I've found to work properly...


@ewr So you would be fine with instead changing Article#asset to only return assets in the `top` scope?  Just so I understand correctly.


Can this really be correct? ``its.dpdu`` and ``its.dpdv`` need to be perpendicular to the normal (``its.geoFrame.n``), which depends on the orientation of the hair. But with this setup, it seems that ``its.dpdv`` is always perpendicular to the Z axis.


Why does the token change? I don't think there is a need for that, and I seem to remember (while testing) that the token stayed the same.


а ручные коммиты с ревертами как из форка сюда попали?


Ah ok, I thought it was either/or. I'll follow-up with a textMessage version.


Yes, much better.  Just "If both available" -> "If both are available".  And perhaps "will use Numexpr" -> "will use the Numexpr".


http://puu.sh
https://gyazo.com
https://www.google.com
http://www.deviantart.com/
http://giphy.com/
Those are all the domains I can think of right now.


нет


https://github.com/kugel-/geany/commit/883ac8bc2d82b1c4ca0fbb2ef4388e7547c77896#commitcomment-13420328

And IIUC it's still incorrect, as `proxy->cb_data` is not `plugin->cb_data` (or at least should not generally be)


import java.util.Scanner;
public class LoveHighschool {
	public static int count=0;
	public static void main(String[] args) {
        //System.out.println("Hello, World");
       String[] theLoveDocYouSays = {"You are dying to start your life with the other",
    		   "You want child with the other",
    		   "You want to bang her so bad",
    		   "You are so blind, you didn't notice she is ugly as fuck",
    		   "You are crazy in love with here, you could die",
    		   "Hello LOVE, Stupid in love",
    		   "You want annal with here, becuase the other has the booty you dream",
    		   "You are not eating and showring because of the other",
    		   "You have bought the house you both could live",
    		   "Please, the other who",
    		   "Oh shit, no you didn't",
    		   "oh lalalal, here I come...I am gonna find you...you can't hidde",
    		   "You want to do bad things to the other"};
        String[] theLoveDocOtherSays = {"The Other Loves you very Much",
        		"The Other Have no Idea Who you are",
        		"The Other Thinks You are cutie,but not a in a way you think about them",
        		"The other is dying to start a realtionship with you",
        		"The other is not ready for relation ship",
        		"The other thinks your ugly",
        		"The other thinks you are stupid",
        		"The other thinks, oh NOOOOOOOOO",
        		"Hell NO",
        		"Who The Fuck are You",
        		"Go head pls kill yourself"};
Scanner nameTxt = new Scanner(System.in);
System.out.println("Please Eneter Your Name");
String nameScreenYou=(nameTxt.nextLine()).toLowerCase();//Get the request from the you
System.out.println("Please Enter The Person's Name");
String nameScreenOther= (nameTxt.nextLine()).toLowerCase();//The person of interest
String[] nameYou=nameScreenYou.split("");
String[] nameOther=nameScreenOther.split("");
for(int n=0;n<nameOther.length;n++){
	System.out.println("index"+ n + "Character" + nameOther[n]);
}
        /*String[] nameYou = {"T","H","E","O","D","R","O","S"};
String[] nameOther = {"M","I","S","T","R","E"};*/
for(int i=0;i<nameYou.length;i++){
	for (int j=0;j<nameOther.length;j++){
		if(nameYou[i]==nameOther[j]){
			System.out.println("What the hell");
			count++;
			continue;
		}//comparison conditional statement
	}//inner for loop
}//outer for loop
//int theLoveDocYou=nameYou.length-count;
//int theLoveDocOther=nameOther.length-count;
System.out.println(count);

System.out.println("You Feel About The other " + theLoveDocYouSays[nameYou.length-count]);
System.out.println("The other Feel about you" + theLoveDocOtherSays[nameOther.length-count]);
    }
}


Anyone can `login()` with the email of someone else, but the `user` is only accessed if the email is verified, so in order for anything to work, you'll need the key sent to that email.

HSTS is a good idea, I'll try to add that soon.


GHX_Vert_IDD (mmatts) - x86_64-MacOS-10.9-clang: OK (1904 of 1905 tests passed)

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


I would also add tumblr.com


Fair enough. In that case, verifying that `user` is present is security-sensitive.


the `:31` is not needed. It was just used for the prior field to merge with a 1-bit field for a combined 32-bit field.


The commit messages still references `pluxy`, might be worth rewording those too (though really not a priority)


What is the intended use of these now with *FadingBanner* (which default and most themes with a traditional SSM use)?

_fallback, default, Simply Love, and I'm sure other themes currently use `scaletoclipped` in *BannerOnCommand=* under [ScreenSelectMusic], which now seems to result in a tweened scaling effect (from natural size to scaled size) when moving through the wheel with enough speed.


durring -> during


:+1: for testing!


вот этой вот ветки тут не было:
![screen shot 2015-10-05 at 19 23 50](https://cloud.githubusercontent.com/assets/1907011/10286437/5d94dffe-6b96-11e5-9a6e-0cdf31852259.png)


Super-fast comment: we probably need to take a little more care to thread the right model factory through in general. That is, a `MergedSQLiteBookmarks` should never give you a plain `SQLite…` `modelFactory`.

That'll work fine so long as the merged version always delegates correctly, which I believe is the case.

If you can give me a rebased branch, I can try to knock this out when you clock off for the day, or you can go run with it; lemme know!


Ok, I agree with you! So in that case, wouldn't be better to remove the whole if(false){...} block and leave only t=tvec?
Thx!


Maybe rewrite ``-1.0f/m_aspect / pixelAspect.y``-type expressions as ``-1.0f/(m_aspect * pixelAspect.y)`` everywhere?


Awesome. Love it when apis work out of box on both platforms.


Fixed: https://github.com/chasehd/EzidDOI/commit/7eca9aeaccaf99a0a3929d25bba3b071bc69791c


Check removed


PHPCS: Line exceeds 120 characters; contains 129 characters


Nitpick: the indentation of this line seems a little off.


Preventing empty spans in simplification is irritatingly difficult with the current code structure. Specifically due to the use of `Datum.Next()` in the `makeConstraints` and `makeSpans` functions.

The increasing complexity of this code is definitely an issue. I'm open to suggestions for simplification.


Got it.


There are a few options that are missing (most of these are from previous releases so not 3.0 specific):

* replicate_on_write (boolean) (2.1+, default true)
* populate_io_cache_on_flush (boolean, default false)
* memtable_flush_period_in_ms (default 0)
* default_time_to_live (2.0+, default 0)
* speculative_retry (2.0+, default NONE)
* index_interval (2.0 only, default 128)
* min_index_interval (2.1+, default 128)
* max_index_interval (2.1+, default 2048)
* crc_check_chance (3.0+, default)

Doesn't seem important to have these for the first 3.0 release.  Should I open up a separate ticket?


NOOOOOOO :cry:


What's up with moveTo in combination with svg paths, they always seem to end up at 0,0, no matter what you enter into the moveTo command?


Не совсем понятно создание экземпляра wtest


I will remove them.


table metadata is named `TableMetadata`, materialized view metadata is called `MaterializedView`.  Should we rename one of those for consistency?


Новая функция для реализации


Здравствуйте, Наталья Александровна!
Изменена вторая лаба:
https://github.com/artichoks/DoItBabe/commit/2d2dec28fc00195cd3d64fd0a9174e732e5cd6a0
Проверьте, пожалуйста.
С уважением, Лисьих Оксана, гр. 350502.

2015-10-04 20:12 GMT+03:00 trtpo <notifications@github.com>:

> Можно ещё доработать по моим замечаниям.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/artichoks/lab2_documentation/commit/ba368b30dc1fa0b44dca69302459fcdda434b6ba#commitcomment-13581207>
> .
>


What are the implications of it being security-sensitive? In all the code, verifying that `user` exists allows to show or not show user-specific content.


It's an auto-generated class. So I let the CS like it is. I think there is no value to update the CS here.
We only have to take care of the CS for `src` repository. (here it's `app/migration`)


Какой номер вернет функция если процессов несколько ( например несколько табов хрома )


You found a bug! This number matches the regex in N::P::StubCountry::AT for special rate, and according to your description on line 47 it ought to return true. I'll fix that myself.


@bwoebi Haha, thanks :) I understrand, and I am not always clear in my descriptions.

It's better, but sorry, there are always an issue, in this code (work on PHP5.4=>PHP7RC ;) )

                <?php

		class A
		{
			public function hello($lastname)
			{
				echo $this->name.' '.$lastname;
			}

		}

		class B
		{
			private $closure;

			private $name;

			public function __construct($name, $closure)
			{
				$this->closure = $closure;
				$this->name = $name;
			}

			public function __call($name, $arg)
			{
				$this->closure->call($this, ...$arg);
			}

		}

		$c = new ReflectionClass('A');
		$m = $c->getMethod('hello');
		$b = new B('first', $m->getClosure(new A));
		$b->hello('last');

The closure can access to private method of B (because $this is an instance of B) but it can not access with this commit.


Sorry. I tried several times to commit my changes to repository, but it
worked just when I wrote the wrong comment. And I don't remember all git
commands which I used.

2015-10-03 23:32 GMT+03:00 Pavel Kryukov <notifications@github.com>:

> (Excuse me for sending the message below to everyone, but GitHub
> broadcasts all comments to commits over the repository collaborators list.
> I don't understand how to fix it yet — P. K.)
>
> Hi VadimRT
>
> Please make commit messages more clear to the readers. For example, "Add
> my name to members.txt file".
> Also, what tool or flow (list of git commands) have you used? The merge
> seems to be ill-formed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MIPT-ILab/mipt-mips-2015/commit/0fc76748e5f551c3c523d02f191b83ef19047954#commitcomment-13577208>
> .
>


Sorry, guess I was being unclear: "e.g." was not the problem. The problem was "in other cases" coupled with "whenever makes sense".

But never mind, I'll change the wording myself.


Standby; have to locate the damage test links for this and exec when chao..
I found them on accident before. Of note destruction only affects base
weapon damage, modifiers like str dmg, zerk, fire etc don't count, that's
why people thought sword of destruction was broken, it actually works fine,
it's just so weak no one could tell. Let me find the articles and post the
numbers.

-Mike

Do we have a reference for the buff?

—
Reply to this email directly or view it on GitHub
<https://github.com/mikesa83/en/commit/c915bac6cb6b77a4cf9f59359ed6e9543d1f89a2#commitcomment-13585007>
.


done (on PR #23 )


Code Convention: функции назіваем с маленькой буквы


Not sure I 100% follow this part -- you're transforming (0,0,0) to camera space? Is that the image-plane position of the top left image corner? In that case, is 'shift' a fractional variable?

In any case, do you think that the 'shift' parameter is really required? IMHO something like this can already be done with the cropOffset parameter, even if you don't actually do any cropping.


"should be straightforward"?


And one more remark.
In this plugin in all locations import from Hotplug is included in the try.
I think that it is correct. Someone can change or remove Hotplug, maybe also in audiocdscan we should do the same?


Yep, done.


Done.


C’est quand même mieux… on est un peu loin du DRY, mais on fera avec :P


Achei o problema era o ID do CSC
Agora estou recebendo o retorno Rejeição: Falha no Schema XML do lote de NFe

Já atualizei os shemas


waaaah -- good catch!


I think this was done automatically by the `tin` tool. I tend to not sweat it for stuff like this, although for whitespaces on line endings we try to remove those if an editor adds them.


and if it should really be kept, would probably better get squashed


(y)!!!


getProcessIdByName, findProcessByName я бы вынес в отдельный класс какой-нибудь!
в wtest должны быть только статические методы возвращающие bool


The 'bug' I was seeing, was ETA was 'stuck', and only grew to 'complete' in the first 10-12% of the run, then simply ran to completion.  I did it time and time again (8 fork).

I do not remember the mode at the time (single?).   Yes, I do know that some modes ARE (or were correct prior to this change).  Unfortunately, I do not know the logic well for fork , or MPI all that well, nor the implications.

However, if you revert this, then it should be done in a proper way. The method I was using, REQUIRED this.  Once it was done this way things worked. It all was in setup. Whatever had done initial setup had removed the number of nodes part of the ETA already (at startup).  So adding that back in was double hitting it.

So I think what is 'really'wrong, is that some crack methods pre-remove the nodes, and others post remove them. So it is inconsistent.

Yes, go ahead and revert this, BUT lets look at auditing the entire set of execution pathways, so that they all are consistent, and work properly.


perfect, thank you


@bwoebi We can by pass with magics methods in PHP, but if you can manage this directly in the engine to avoid this tips, I'd be eternally grateful to you ;)


Ya the original author stanfe doesnt seem to have an active github.  I am just going to remove his commit from my local copy because this throws javascript errors and needs to be cleaned up I feel.


What happens if Team is not valid (and therefore `waitForParentToReceive() == false` ) ?  Will the child complete the collective as though it were successful?  I would assume that we would want to throw an exception in this case.


If I want to use any icons, I can just include icons.ts, rather than individually including each icon.


你也发现这个东西拉  用得怎么样？


Good point, I'll make this a variable.


This is the rebased version. I'm knocking off for the day. Feel free to go for it overnight, but I can pick up again in the morning if you like. There is a chance my brain might work better tomorrow....


Thanks. FIxed


Truth is we don't need to depend on `media-libs/mesa[opencl]`, heck, we don't even need to depend on `app-eselect/eselect-opencl`, thanks for pointing it.

As far as I know portage does not provide a way to protect you against it if you enable _dangerous_ stuff like jobs > 1.
No package warns you about this in the official tree, or does any? Can you point me to an example? I just did a few `grep`, but maybe I searched for the wrong thing. This is a delicate matter but it has to be dealt with one way or another.


Ah, you are right, now I remember. This 'was' hybrid mask mode.

But when I made this change, and re-ran same test, things 'worked' perfectly, grew to 100%, and as things got closer to the end, the percentage was more and more accurate (just like it should be).

Jim.


looks good. don't think it should effect fb


@grunenwald via ce bousin, le [mailCatcher](http://webmail.srvz-webapp.he-arc.ch/) est activé pour tous les groupes qui ont gagné une mise à jour de PHP au passage.


fValueに展開されたfloat型の値が入ってるはず


cool cool


Thx u will fix that ;)


I will remove them, slipped by in redoing all changes again.


Looks good, looking forward to seeing how it merges with ours!


nice


回数だけじゃない


ok


> However, if you revert this, then it should be done in a proper way.

It was totally incorrect before reverting, and obviously I tested it well after reverting: It now works like it should.

Now, some hybrid mode may well have the bug you describe, but again that's not this function at all. I'll do some hybrid testing later.


You should specify which issue this test fixes.


@maxiloc shouldn't you create a separate update installer to add the indices instead of adding in the same installer file. If the module is already installed then these indices will not be added to the table.


It's problematic if the "center" / "radius" -style initialization doesn't work anymore after this patch.


Tested DotCi-2.12.4-SNAPSHOT.hpi on local machine but still same issue so not ready yet to issue PR


Hey - it is on my list to actually clean this up, write a test or two and
submit a PR.

Do you have a preference though? I was thinking that there would be two
types of method parameters with this:

1) Method naming (the way it's setup in my branch now): pgJsonHasFieldValue
and pgJsonDoesNotFieldValue
2) Boolean closure argument - pgJsonHasFieldValue(String propertyName,
String jsonAttribute, propertyValue, Boolean equals = true) - and then
check the equals parameter and adjust accordingly (that way the current
method calls wouldn't change). Super simple this way I think.

Was slightly pressed for time, which was why I went with 1) above instead
of 2.

I am also working on some JsonArrayType/JsonbArrayType (we have an array of
Maps stored as json in our postgres install...) I'll let you know when I
get something ready for that too. =)

Also, thank you so much for setting up this json postgres ORM.

Tom M

On Sat, Oct 3, 2015 at 2:18 PM, Iván López <notifications@github.com> wrote:

> Hi. If you want, you can add some tests and submit a PR and I'll merge
> this functionality into master :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tmarthal/grails-postgresql-extensions/commit/87970e10d60236d70792ae32ca46c4bd4c5cb250#commitcomment-13577350>
> .
>


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 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-70.36%25-green.svg)


AnimatedTransform has a function that does exactly this: ``AABB getSpatialBounds(const AABB &aabb) const;``


I did it in the commit description


Could we make use of string interpolation here?


Make sure it returns .


see ``AABB getSpatialBounds(const AABB &aabb) const;``


下のおぷしょなるバインディングの値で判定すべき


An overview is already present in `docs/dev/README.md`: these notes should be integrated there.

In addition, the pad link is not related to this project and should not be included.


I know some combinations of wordlist + rules + mask will do a fairly poor job of reporting ETA. It's not trivial to get right in some edge cases (few words, many rules or vice versa, I can't remember).


「正確な数値」ってなんやねん


Thanks for the idea. Probably i have to select the right subtractive bonus to balance positive and negative history values.


As we're talking about only one container here, update to 'Remove existing container'


Thanks Grant! Just added it.

On Sat, Oct 3, 2015 at 6:31 PM, Grant McConnaughey <notifications@github.com
> wrote:

> This should be in setup.py under install_requires, too. Otherwise it
> won't get installed when django-sql-explorer gets installed and apps will
> fail to start on Python 2.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/epantry/django-sql-explorer/commit/ec2cfb2975e1b252c5a7dcede37f75a01e4071f5#commitcomment-13578198>
> .
>

--

Chris Clark │ ePantry │ 617-571-7327

1770 Union St, San Francisco, CA


Ah, so pretty now :-)


it's -> its

ran -> run ?


:+1: I hope it will be enough in order to close the related issue automatically ;)


You replace both `IsPathDirectory` and `IsPathFile` with `IsDirectory` here.


@danmar: this results in `c:\users\xmr\desktop\cppcheck\lib\symboldatabase.h(1078): warning C4018: '<': signed/unsigned mismatch` for Win x64.


Yes.


Yes it will


6ってなんか意味あんの？


It would be nice to support multiple shapes too


by the way

I still have the issue when renaming a file, that the XHR location href answer does not work and reload the page. Please only try this with a current master from now on. The last 2 weeks the renaming was broken on master and made a big mess in db, entries and files!
I do discover the same with 2.0.2.
Any ideas or verifications are welcome (maybe its my Mozilla or anything else?).


Line 518 of ext4_xattr:
		ret = ext4_block_get(xattr_ref->fs->bdev,
				&xattr_ref->inode_ref->block,
				xattr_block);

Probably You want to load &xattr_ref->block.


Why do you call it Daedulus?


未満だけどいいの？


ll


... start the new containers, switch the service over to the new contains, then wait for demand ...


Thanks for this


I'm working on ArrayList, I will be posting it tonight in GitHub
Karla

----- Original Message -----

From: "amazing-username" <notifications@github.com>
To: "amazing-username/CSC3610_Triathlon" <CSC3610_Triathlon@noreply.github.com>
Sent: Sunday, October 4, 2015 1:01:11 PM
Subject: Re: [CSC3610_Triathlon] Athlete_Two.java (6571152)

I didn't bother with making the other classes because the more efficent way of dealing with all this data is later on down the road. I made sure to include Lists, Stacks, Queues, Iterators, and Comparators.

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


@pranav01 so ur the culprit causing duplicate repos :P


This is a good start, but to make this work with all light source types, integrators, and potentially multiple lights per shapegroup, some changes will be needed. If you'd like, i could give this a stab next week (I'm traveling most of this week).


Oops, sloppy bug!  Fixed here: https://github.com/Maluuba/crfsuite/commit/9f52dfa84856b3e6b36614ee81ddcda67dd7936b


@blackwarthog
Привет! Как мы можем сделать так, чтобы наш filecontainerzip не мог читать сжатые файлы?
Я убрал проверку вот в этой строчке и теперь список файлов контейнера включает те файлы, для которых компрессия не равна нулю. Но я так понял из потока мы всё равно получаем сжатую информацию. Как сделать так, чтобы она распаковывалась?

P.S. Я пытаюсь сделать поддержку чтения kra-файлов. Это файлы Krita, он обычные zip-архивы. В каждом архиве есть файл mergedimage.png, который я пытаюсь прочитать. Пока не работает - импортёр png проверяет заголовок файла и говорит, что прочтённые из потока данные не являются png-файлом.
Проверка проходит вот здесь - https://github.com/morevnaproject/synfig/blob/8451907c7c315f6ca8e6a12e4497a726852b7fc0/synfig-core/src/modules/mod_png/mptr_png.cpp#L119
Думаю дело в компрессии.

На всякий случай прикладываю тестовые файлы, которые пытаюсь заставить работать -     http://cloud.morevnaproject.org/seafile/f/5fb0ce6c58/?raw=1


Is this a single meta data file or will one of these files exists for each message?  Aka how many times will the date_recorded entry be entered?

Also add version number of  the software to the data, and a data id field for the count of total messages and how many of this type of message has been made.


Huh?


r+


This is so named as it is a tool that (among other things) adds wiring.


saw approval from MonsieurNicolas
at https://github.com/StellarCN/stellar-core/commit/9fc3778f2ce1940d59c9df04f3e57dc205968b13


merging StellarCN/stellar-core/compilation-warnings = 9fc3778f into auto


StellarCN/stellar-core/compilation-warnings = 9fc3778f merged ok, testing candidate = 90d9a646


The edits in this commit are based on Ballot 151.


[Uploading 《软件设计与体系结构》实验指导书.pdf…]()


i hope someone else hasn't already made a m05 helmet that doest suck


+Yu Ma


@manish172 Yes that is part of the things I need to test :smile:


This check should be done earlier as @franzliedke stated.


@akhilnarang tomo! :+1:


`p` should be `unsigned` since that's what `pointer` is. Although, this fixes the aforementioned warning, another one will be shown since `size()` is used: `..\lib\symboldatabase.cpp(3766): warning C4267: '=': conversion from 'size_t' to 'unsigned int', possible loss of data`.


I am wondering whether we can avoid the `GetThread()` call here - it is relatively expensive because of it showed up in some of the micro benchmarks. Something like:
- Set `m_Next` to NULL in `Frame::Frame`
- Set `m_Next` to NULL in `Frame::Pop(Thread *pThread)`
- If `m_Next` is NULL here, early out and do nothing.

(This can be done as separate change.)


Good! :+1:


LGTM otherwise.


>Can you point me to an example?

Honestly I don't know what's the proper way to do this, that was just an assumption. Compilation may fail or user may run one more emerge in different console so we might end up in a wrong state. It should work though most of the time, so consider this just as suggestion.


Care to elaborate?


Accidentally pushed without PR. @sowmiya Please look over.


In this case, the easiest way I can see to insert raw html into the DOM is using mithril's m.trust which takes a string: https://lhorie.github.io/mithril/mithril.trust.html

The benefit of inserting the svg directly, vs using an image tag, is that we can style it with CSS.

This will all not matter once we get an icon font which Andrew is working on.


i think this should read

> Return types can be either on the same line as the last parameter, or put to next line with **4** space indent

that way the `:` will line up with the open paren


No it will not. The problem is that no indications were given to know if there were labels or not: by default the 1st line had to be labels. If we want to correct the reader so it can read the previous convention, I guess we would have to update the writer so that it stipulates that no labels exist, instead of stipulating (like I did here) that labels exist.


@chirag04 Updated https://facebook.github.io/react-native/docs/known-issues.html#platform-parity


should rather be `os.path.dirname(self.cfg_path)` -- if this ends up a real fix and not a hot quickie


Yo!


Poste aqui por favor o xml que vc está tentando enviar.


The `b` stander for `bool` but I'll update this.


I hope I don't bother you. :-)

This has a disadvantage of also applying the effect to permanents that were blinked between casting and resolving Blatant Thievery, because the target.getFirstTarget() has no zone change counter check build in.

You avoid that problem by creating a custom target class here.


Tag fault #215 had to be #214. Well, I'll fix that with the next push which will very likely come soon.


The token we have before email verification proves a different thing than the one we have after email verification.

For instance, the following scenario would yield issues without setting that token:
1. Alice logs in as alice@ with token A on computer 1
2. Alice verifies token A from alice@ on computer 2

You want to set token B on computer 2, and to acknowledge token A as valid.


Please rollback these changes to the POM file. The compile level must be set to 1.5.


> (or do not add it in the first place)

Haha! True, yes… But I think the important thing is that `git pull` doesn’t also update submodules. So you’re left with a stale version locally, and `git add . && git commit` causes that stale version to be committed.

> Is it a case of running git submodule update before committing?

Yes – exactly.


issue #1


Go ahead and delete these lines instead of commenting them out. They will continue to exist in the GIT history.


AddDataSetTestSupport (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1929 of 1934 tests passed)

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


Actually we just need [this] (https://github.com/liachmodded/Railcraft/commit/5fb8b763f8e2590ec697fed52cfbadc30cb0e851#diff-c197962302397baf3a4cc36463dce5eaR282) in #702.


ah, the comment now tells a lie. The following field is an array of KeySlice, so no need to be 16-byte aligned. Fixed the comment to be "8-byte aligned". BTW, the "slot" part of the data region is anyway 32-byte aligned because it starts from the end and grows backward.


Was there are reason for commenting out these lines? If source and target are not null when the LOS is initiated the last LOS will be shown, which is bad.


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


pandoc_scratchblocks was previously pulled in in the same way as the curriculum repos. I swapped it for a submodule. I now regret this decision (because lots of people hate submodules.)


Right, so allowing actions to be performed when `user` is null should always cause a failure, otherwise it may cause a security issue.


Thanks a lot! I will give that a try and come back.

Also Apologize for adding the comment in the wrong place(wasn't sure of the correct place)


**Eslint**: 'return' outside of function


we should also add info about the default value


Creating repository hello-world


now we have delegatecommand and relaycommand. should probably choose one and/or combine them


is the string really still valid after the decref?


I recommend using commit messages like `Fixes #54` so that Github automatically links the issue and the commit. See: https://help.github.com/articles/closing-issues-via-commit-messages/


In fact, it leaks even if it doesn't fail (considering text/byte strings). Will fix.


fast-forwarding master to auto = 90d9a646


Gotcha.


Hi

The right place is the issue space
https://github.com/bsorrentino/maven-confluence-plugin/issues

It is enough to create a new issue

On Mon, Oct 5, 2015 at 6:56 PM raghuunt <notifications@github.com> wrote:

> Thanks a lot! I will give that a try and come back.
>
> Also Apologize for adding the comment in the wrong place(wasn't sure of
> the correct place)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsorrentino/maven-confluence-plugin/commit/9927bccb13e07f30b9a2172b6933cf9cbcfc3c9f#commitcomment-13596725>
> .
>


Rejected because %%toolbar%% is already wildcarded, so would catch AVG toolbar. I'll manually add ABBYY FineReader. Thanks.


Thanks for the investigation. I think this one would be a good one to get fixed, but get to it when you can. We're still catching up on upgrading to the latest commit here.


```php
Parse error: syntax error, unexpected ')', expecting :: (T_PAAMAYIM_NEKUDOTAYIM) in php-jscrambler-master/includes/jscrambler.php on line 73
```


Why not just let systemctl do this?  `man systemctl` ->

```
If the suffix is not specified, systemctl will append a
       suitable suffix, ".service" by default, and a type-specific suffix in case of commands which
       operate only on specific unit types. For example,
```

Then pykickstart wouldn't have to maintain another list of systemd unit types that will get out of date.


@ThomasBurleson AFAIK it should be safe to always use `scope.$on('$destroy', ...)` for any kind of cleanup that needs to happen for a directive.


Let me know when you want me to test this :)


lgtm


I'm not sure of the convention either. At my last company we named them something descriptive (brick, sky, clay) - that way it was easy to identify both in mocks and the CSS.

The issue with purpose is that the same colors are often used all over. In the following shot, main green is used as:
- a part of the cockroach
- the "active icon color" in the navbar
- The color of the chart text
- The table header node count (but only for node count)
- The color of the circle indicating a node is healthy
and should possible be the color of the active secondary nav as well (since that color isn't specified in the UI-Kit)
<img width="238" alt="screen shot 2015-10-05 at 12 57 27 pm" src="https://cloud.githubusercontent.com/assets/3691886/10287386/ae054f68-6b60-11e5-8d64-fc24df04304a.png">


Yeah, I know what you mean, but this was the best idea I was able to come with. I was not able to find any ebuild in the main tree having to deal with a situation like that, and making the user do the job is a burden, live ebuild can update on a daily basis if upstream starts to have a lot of fun.


"The return value must be checked." -- @kugel- :]


I really like that!


i didn't mean using an image tag, i meant "why can't you construct a virtualdom element here?"


It's best not to add the IntelliJ project files to the repository as they will overwrite the local project file. Add to the .gitignore instead.


A possible compromise here would be to have functional variables aliased to the values in the palette/color scheme


can you just create a variable for each of these that has a value of "main green" or whatever?


LASda


Say ok


@dstoianov I've streamlined the process for importing the Excel spreadsheet with a python script and reprocessed the current export (August 2015). Future conversions should be quick.

@Marini06 The terribly denormalized multi-value "Market" column is the same as the input, sorry.


Det er godt nok grimt.


@rwaldron I'd like to get the Altimiter component and Multi controller added to this branch (tonight?).  Then I'll do a WIP PR just like the other sensor, and then you can test it.  Unit tests and diagrams after that.


What was the rationale for suggesting against aufs?


Hi Wenzel, I think 2 is enough but I will make more tests to make sure it is. The issue seems to be that with certain combinations of resolution, sun radius and sun direction, a pixel could end up accumulating too much samples.
Will also make it a define.


So all this is us tearing out foundation ?


I think you commented at the same time as my 2nd comment - sounds like we agree.


That sounds like a good idea. I'd like to do it separately.


What kind of assumptions are we making here?  GNU make?

Can we add some comments on what this Makefile is supposed to do?  And maybe how to use it if it's not just "make"?


I agree, I don't know why it was in that way in the code.
Changed.


I'm not sure what that would look like. Do you mean, using `m('svg' ...)` etc?


To be honest I have no idea ;D
I copied it from cylinder plugin and seemed to solve the Division by 0 errors. In my tests the render looks the same before and after the patch but faster with the patch.


cmake is required?


Or like, turing this into mithril with msx?


Σ( ° △ °|||)︴ the same FABCircleBehavior class...... you should tell me your progress or schedule....


Sure, will do that.


aggregate


@BenWiederhake, okay, that's fair. Unfortunately this PR is no longer a clean merge due to later changes. Do you have the time/inclination to branch+patch? I can get around to it but don't have the resources to make this a priority.


@lewurm
> that means you're relying on the fact that when the client finishes, the server is done with processing the client request.
Correct, I'm relying on that property. The client is expected to send some queries to the server, and then wait for the server to process them and get back to the client with the response. I believe this is a broad enough generalization which should cover most practical use cases (e.g. browser & web server, sql client & database, etc.) and can't think of a simple way to make it more generalized.
The fact that the client won't disconnect until receiving a reply from the server is something that should be checked, certainly.

> is that indeed the case for the EventStore benchmark?
Yes, the client sends a number of queries to the server, waits for the server to process all of them, shows some statistics, and exits.


Why a sed backup file with a suffix of "~t"?  And don't you want to put quotes around it?  If my shell is bash and there's a user named t, then ~t will expand to his home directory.  That seems bad.


maybe add a TODO to actually use awaitChecking somewhere?


LGTM


@skeptical Next round is ready :) I hope I didn't break anything by removing the `net.pms.medialibrary.dlna.RootFolder` stuff to allow refreshing like a normal `MediaLibraryFolder`


Already fixed in 5df54206a55c4ff64ee9d5261e5e155ffdd594b8


this is specifically about generated fcns so should stay with that bullet


Thanks for noting, it certainly would.


Please use camelCase for option names. That's a standard.


shift is in proportion of horizontal or vertical size depending on some blender parameters.
I just tested cropOffset and it doesn't seem to work. film.cpp checks that cropOffset is not negative and also that cropOffset + cropSize is not greater than film size.
I added shift to not touch cropOffset in case it had unintended effects but if it is safe to remove the checks on cropOffset it can work as a shift.


@garvinhicking

Please run addlang for me

Here is the german translation for the long one.
`
@define('BULKMOVE_HELP_DESC', 'Bitte setzen Sie ein Häckchen bei den entsprechenden Objekten oder Bildern und wählen hier den Ort zum Verschieben aus. <b>Achtung:</b> Diese Aktion tritt sofort in Kraft und wird nicht noch einmal gegengefragt, wie beim Löschen. Alle gewählten Dateien werden auch physisch an den neuen Ort verschoben und alle Blog Einträge nach entsprechenden Pfaden durchsucht und geändert. Mit Staticpage Versionen ab v.4.52(!) gilt dies auch für Statische-Seiten-Einträge.');
`


git init


I think we no longer need to create a copy of the list. `getNetworksFromStore` already does that.


any reason for this indentation?


```
PHP Parse error:  syntax error, unexpected 'Client' (T_STRING), expecting variable (T_VARIABLE) or '$' in ...
```
Workarounded doing this instead:
```php
$client = new JScrambler($accessKey, $secretKey);
```


When I tried to run the server with Node.js (version v0.12.7), I got the following error message:
```
const DEFAULT_CONFIG = "config.json"
^^^^^
SyntaxError: Use of const in strict mode.
    at exports.runInThisContext (vm.js:73:16)
    at Module._compile (module.js:443:25)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3
```

It seems that if you have strict mode, you cannot use "const" qualifier by default:
https://stackoverflow.com/questions/22603078/syntaxerror-use-of-const-in-strict-mode

Apparently, it would be possible to run Node.js with `--harmonize` to avoid the error, but you would also need to update `package.json` for that.

I would maybe leave the const keyword out for now, but I'm not that familiar with how other packages handle this. What do you think?


Oh my lord. What's a commit message?


Fuckin autobiography up in here


Can have fun with this sorting thing... Please go ahead and test!
Basically DUE tasks take precedence, followed by EVENTs (sorted by start time) and lastly FLOATs (sorted by name, regardless of start time  <------ any one wants to sort by start time for this one also? Because not all floating tasks have start time).


You're right.  The reason why I did this and didn't update VSVer was because I would need to update all of the vcxproj files to Visual Studio 2015 and stick them in vs14 folders.  This was a much faster, but dirtier solution.


Thanks. I knew this needed fixing, this was mainly because I hadn't made a decision on quite what the right way to do this was.

Firstly, think we should use scoped_ptr here instead of an auto_ptr, largely because people using this API may copy this code and accidentally copy the auto_ptr without realising.

The ideal solution I think would be returning a UniquePtr in C++11, but probably the next best option would be returning a raw pointer and getting the user to wrap it. Alternatively we return a shared_ptr and just live with the cost of the reference counting. Either way, think it makes sense to include a 'Ptr' style typedef inside the AttributeHandle.


OK. Sure. No problem. I think you may also have problems with the "let" keyword. I'll clean this up. Sorry about that. I was using 4.1.1.


Looks like we still do given the function return []*network not []Network


Yes, I planned to fix that but the Blender exporter does not use center or radius for spheres.
I was planning on adding a function similar to prependScale but for translations to add the center to the animated transform. Do you think it could work?


Don't worry about that. ;)

Is there any issues about using a single Target class for a spell that has multiple target instances with different targeting restrictions? Also, is there any already implemented card I can base myself on?


Forgive me if I am being dense, but I don't know what you're trying to tell me. I just added the comma.


This is weird. `RKey.Key()` is fishy in general (To go from `Key` to `RKey` we use `keys.Addr`; why is the reverse conversion an identity function?), and to cast the input to an `RKey` only to cast it right back to a `Key` doesn't make any sense. I would also expect CLI input to be in terms of Keys, not RKeys.


should still be `$entry->{key}`


Sorry……Because today is really boring, so I thought of the update code…I will continue to finish the feature of change dark theme


Can you update the comment too?


Not sure if cooldowns should be reset


s/want extend/want to extend/

I'm sorry I missed this in the initial review!


Instead of `TestSolutions` maybe `SharedSolutions`?


Hi @longhotsummer sorry for the hassle. Yes, it would have been a build cache issue.

We moved forward as that is how the Node ecosystem works - we all move forward :).


@hossome what is eventData?  I don't see it being defined in the controller


Should we have `RKey` versions of these constants? Since the mapping from `Key` to `RKey` is more clearly defined than the reverse it seems better to cast the `KeyMin` to an `RKey` than `key` to a `roachpb.Key`.


не `// Firefox`, а "кроссбраузерно"


Once we start retrieving them I would agree. If you think it would be already useful for debugging I would probably also agree :)


Then let's make it bigger and pull is the full illumos commit.  This is documentation update which should have happened some time ago illumos/illumos-gate@b3cff10.


Thanks!
Sure, it would be great if shapegroups supported emitters and instances allowed more than one emitter. It was a kick fix to allow mesh emitters to be animated and have motion blur and the exporter can handle the limitations and export shapes as separate objects to meet the requirement of a single shape.


Does it hurt to log the output?  It hurts to lose it when something's wrong.


You would want the resolver to leave that method with the same caching flag with which it entered, wouldn't you?  I'd suggest saving its value before the loop and restoring it afterwards.  To be really safe, that should probably be in an ensure block so that if an exception is thrown it still gets restored...but there are many return statements in there before which the original value would also need to be restored.  I suggest we extract a method with all those tests that conditionally return. This would make the task of restoring the original caching value much more manageable. I could probably write the code but would need someone else to test it.


@stephenplusplus Was there an issue with using SSE4_CRC32 directly? The version of fast-crc32c you are using depends on a much older version of SSE4_CRC32, which won't work with Node.js v4 and above.


@mrtracy might have some specific guidance.


Hoe bedoel je?


I will check that, thanks!


Can we add a catch all for associate_records that raises "You tried to create #{inspect(build_record)}. You cannot `create` a factory that is not an Ecto struct."

or something like that? I ran into that while testing this in Constable. It was an easy fix, but it took me a sec to realize what I was doing wrong


this looks easy to misread, but I ran into the 80-char limit.  Would it be better if I did a newline earlier, such as before the `tapename` ?


As this stands I will not be accepting this.  While I will use the trailing brace in others code, it is not the format I subscribe to.  I like lined up braces as it makes scanning complex scoped code easier.


Ooh ik snap 'm, kan een optie zijn, maar misschien is het makkelijker om de changelog berichten wat uitgebreider te schrijven


### [Codacy](https://www.codacy.com/app/caxaria/CssTests_2/commit?cid=13479322) detected an issue:
#### Message: `Unknown property 'background-colour'.`

#### Occurred on:
+ **Commit**:  cf405c14d0bb99c18a563dc643a28c2348df3cea
+ **File**:    [myFile.css](https://github.com/caxaria/CssTests/blob/cf405c14d0bb99c18a563dc643a28c2348df3cea/myFile.css)
+ **LineNum**: [8](https://github.com/caxaria/CssTests/blob/cf405c14d0bb99c18a563dc643a28c2348df3cea/myFile.css#L8)
+ **Code**:    `	background-colour: white;`

#### Currently on:
+ **Commit**:  b55b61ece4a2d33ebe7b8220f2130af19d4479a4
+ **File**:    [myFile.css](https://github.com/caxaria/CssTests/blob/b55b61ece4a2d33ebe7b8220f2130af19d4479a4/myFile.css)
+ **LineNum**: [7](https://github.com/caxaria/CssTests/blob/b55b61ece4a2d33ebe7b8220f2130af19d4479a4/myFile.css#L7)


He should have, it's broken.  @LiterallyWTH ti's supposed to look like our standard close button: http://i.imgur.com/po5Pzj7.png


hi there -- i just made you a committer on the original repo, if you want to merge this in and add other things as helpful. i dont really work with mongo at this time, so have at it!


@amenk it is available see (http://packages.magento.com/#!/deploy)


Oh, sry, the issue just said hide'n'seek.  I DID make it more better though!  I'll get back to it today.


I think we could move this to a separate method.


ryrtyrty


Kan daar wel een apart command voor maken misschien..?


Thank you.. I'll try my best


TeamCity pull requests :: netty-tcnative [Build 25](https://garage.netty.io/teamcity/viewLog.html?buildId=2050&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Remove before merging to master


@FelipePortela, o problema do NullPointerException é porque você não inicializou o objeto Questão. Basta dar um = new Questão() aqui nessa linha. :smile:


@igbasov, remove the commented code, please


TeamCity pull requests :: netty-tcnative [Build 25](https://garage.netty.io/teamcity/viewLog.html?buildId=2050&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:52


This also could be moved to a separate method.


:)
Just saying that your add_flashlight_tool doesn't need to follow this naming convention as the purpose of the flashlight tool won't be to add flashlights.


@hossome data/db/journal/lsn should not be in this commit.  We are not adding database files


Sounds good.


Uhm, no it can't. So the check is superfluous, but I guess it doesn't hurt either... I'll clean this up.


so we shouldn't have the whole bootstrap project here, this is already in public/lib/bootstrap/dist/


Sorry, done now


That happens with `+all`, but that should be ignored so it jumps to the `continue` right away.


This does not work


Hello


Thanks @chrisfsampaio!


This doesn't make sense to have an object that is restricted by an enum of strings. That's not how JSON Schema works.


This seems wrong to me: range descriptor keys are always constructed as `Keys`, not `RKeys`.

I think most of these key constants should be `Keys`, with the possible exception of the non-addressable `RangeIDKeys`.


Shouldn't this be version `3.3.0`?


super should be call at first, its convenient in your case
like:
res = super(AccountInvoiceLine, self).write(vals)
MembershipLine = self.env['membership.membership_line']


Короче вот как это получилось значит: Без ребейза пушфорс тебе смержили и получилась такая какашка


AddDataSetTestSupport (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1929 of 1934 tests passed)

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


да я понял


Yeah obviously… in A::hello() you should be bound to A.

I think this just shouldn't work. That code looks totally like it shouldn't work… If you need anything like this… why don't you just make the property public? That's what visibility is for?


Still broken for me.

I see clean as:

```
// Clean output directory
gulp.task('clean', function (cb) {
  del(['.tmp', 'dist'], cb);
});
```

I should of probably filed this under issues instead of under the specific commit that broke it.


search method gives you list of recordset so give proper name memebership_lines instead of ml_ids


AddDataSetTestSupport (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1935 of 1940 tests passed)

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


I think this sentence ("In addition to that...") is a little awkward. How do you feel about "Atom will also execute the code you add to your init file on startup" or even just "Atom will also execute the code in your init file on startup"


I'm pretty sure the client interfaces should *not* accept RKeys (or if they do they should be tagged as such so they won't be passed to `keys.Addr` again).


Tried a workaround in 3be8034


why not just `emails: []` ?


thanks! I totally noticed that just now. I dunno how it compiled for me.. it was late that night. :)

cheers


can we remove this method please? we have `external` attribute


@pengying mind opening up an issue for investigation?


I think that the option number 2 is more flexible, so I'd go with that.

Don't worry about submitting a PR. I was reviewing the forks the other days and I saw this, that's why I contacted you. I think that if other developers have some nice features it would be great to have that in the main plugin. It's good for the community.

Regards, Iván.


oops, typo. Fixed


Give proper name of variable because code reader should more understandable,
Like:
records = self.env.cr.fetchall()
for record in records:
    if record[0] and record[0] < 0:


Hi Qwasson, you might want to see the other recent additions to mcpe_viz :)  Some cool stuff.  Turns out height was sort of already stored in the leveldb.  It's height of first non-transparent block but still... Check it out.


@julianpoy A commit message is my diary <3


One


Why override _generateJournalArticleDom() ?  It seems to me you are primarily dropping _generateComponentListDom(), which could be done just by overriding that function as `return false;`.


Class AddUserPreferences does not have a Docblock comment.


https://hydra.nixos.org/job/nixos/trunk-combined/nixpkgs.musescore.i686-linux are these failures related?


Try to convert this SQLQuery into search or read_group method


The fix is good, but we'll need backward compatibility as well. I'll add that and then merge. Many thanks for fixing!


Don't use @api.one


AddDataSetTestSupport (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)


Oh dear. I didn't check the commit you referenced since I was on a phone and just assumed it was the right one. It wasn't -- you found an abandoned approach. Apologies for misleading you. The extended delegate just didn't work right due to numerous issues, including the one you just mentioned. It also presented an unnecessarily complicated interface to the app.

I thought you were referencing this: https://github.com/kgoodier/CocoaSPDY/pull/7. It's what I intend to merge into the main repo, after a rebase. It does not use any new delegates. Notifications back to the app are done via a global NSNotfication, and it is up to the app to filter the URL for ones of interest. CocoaSPDY only caches the pushed request for the lifetime of the parent request; the app must make a new NSURLRequest and "hook up" to the pushed request using normal Apple NSURL interfaces before the stream goes away.

Since you are both motivated and interested in this work, I'll get the rebase done and a pull request to the official repo posted soon, and you can try it out.


making jackson deps provided would require a change to:

https://github.com/hawkular/hawkular-agent/blob/master/hawkular-wildfly-monitor/src/main/resources/module/main/module.xml#L34-L36

so the jackson resourc roots are removed (<resource-root>) and added to <dependencies>


Завтра исправлю


So, this is just the curl command used to publish a stashbot.jar to palantir's internal artifact repo, we don't want to change this script or publishRelease.sh below. (or stash-comment.sh, probably).  The only place you need to add the "-k" is in the job formatter below, which looks good.


PHPCS: Each class must be in a namespace of at least one level (a top-level vendor name)


Arrange import as alphabetically like:  from openerp import api, fields, models,  _


remove the extra "-k"s from the bin/*.sh scripts, and I'll merge this.


Could have a method loadDistributedCache to load all the files and archives.


ок, спс


The condition is evaluated on every iteration and is most likely not going to be optimised away. So in theory this is faster, but since we're dealing with really small collections here it kind of seems like overoptimization to me. That said either way is fine for me.


Arrange imports as alphabetically like: from openerp import api, fields, models, _


Can you please move the html logs inside the if condition?

That way is clear when the conditions were true ;)


I agree that it could be confusing. will change the order.


Arrange import as alphabetically like: from openerp import api, fields, models, _


Hmm, why were the badges removed?


Yes true I was not aware of this. Thanks.


this variable name should be like : ResPartner = self.env['res.partner']


Please keep in mind PRs for updated translations should be opened in https://github.com/OpenRCT2/Localisation , not the main repository.


@taconaut :+1: I would call it "lateral" instead  of "backward" compataibility. And don't worry, after this you're free to blame me for any and all problems :).


great :+1:


I tested `../run/john test.in -form:nt -w -ru -mask:?w?a?a -prog:5 -fork:2` now, it works just fine with and without memory buffering.


The original problem was that single quotes were not being escaped, so I added ENT_QUOTES.

After considering the options further, I'd like to propose the use of PHP's `rawurlencode()` with the corresponding use of JavaScript's `decodeURIComponent()` in the ub.php JavaScript (so that it is self-contained).


ressources to resources :+1:  good changes


Доработал FileContainerZip. Сделал обёртку поверх файлового потока, не
совсем чистое решение, у нас был расчёт на то, что контейнер будет
работать без сжатия.

Для правильной работы нужно будет сделать сжатие/распаковку внутри
контейнера. В этом квикхаке некоторые функции отдают сжатый поток (почти
все), некоторые распаковывают. Путаница может быть.

> @blackwarthog
> Привет! Как мы можем сделать так, чтобы наш filecontainerzip не мог читать
> сжатые файлы?
> Я убрал проверку вот в этой строчке и теперь список файлов контейнера
> включает те файлы, для которых компрессия не равна нулю. Но я так понял из
> потока мы всё равно получаем сжатую информацию. Как сделать так, чтобы она
> распаковывалась?
>
> P.S. Я пытаюсь сделать поддержку чтения kra-файлов. Это файлы Krita, он
> обычные zip-архивы. В каждом архиве есть файл mergedimage.png, который я
> пытаюсь прочитать. Пока не работает - импортёр png проверяет заголовок
> файла и говорит, что прочтённые из потока данные не являются png-файлом.
> Проверка проходит вот здесь -
> https://github.com/morevnaproject/synfig/blob/8451907c7c315f6ca8e6a12e4497a726852b7fc0/synfig-core/src/modules/mod_png/mptr_png.cpp#L119
> Думаю дело в компрессии.
>
> На всякий случай прикладываю тестовые файлы, которые пытаюсь заставить
> работать -     http://cloud.morevnaproject.org/seafile/f/5fb0ce6c58/?raw=1
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/morevnaproject/synfig/commit/8451907c7c315f6ca8e6a12e4497a726852b7fc0#commitcomment-13596489


@svoinea Just did  `git pull` and the problem appears to be resolved :grey_question: :smile:


Should not have ${IMAGE} either


`LABEL STOP /usr/bin/docker kill -s HUP ${NAME}`


@timelyportfolio Thanks for your interest! Much of the work done here is to support a specific application of network visualization. There are somethings that might be worth merging back into the main project, but unfortunately I'm unsure if I can prepare a merge-ready branch in time for your next CRAN release.

I will see, however, if any of this work can be merged back in the future, as well as reviewing your issues list.


do we also need to bump the mathquill version number to get it to update?


Ok, so that passed?  Now we simply need to figure out the -test-full is causing opencl to crash, and fix it.   At least this gets us to where the CI's are not having spurious errors.


so should this be removed as well? https://github.com/runcom/atomic/commit/59efa58ae846d490b1c35ceb9babc6ad0a386b00#diff-ea1767c23694eb04f7a12e5bf8972719R28


when you wrap something in an `<ul>` , try to keep everything inside of it wrapped in an `<li>` If you don't want the bullets for the list, you can edit them out in CSS using `list-style-type: none;` .


Why is this only useful with `spyOn`?


Very clean code overall, nice job!


Since we pull mathquill from github, It should update cleanly. At least it did for me. We might need to remove the node_modules directory before running `npm install`


`spy = createSpy().andCallThrough()` doesn't really do anything.


Yeah, I'll have a look on super with AMD's CPU device and see if it crashes there too. Doesn't crash on my gear.


ok, well that is the same problem I was having with squire, it wouldn't update until I bumped the version.  But I was also specifying a separate branch than master


Not on mine either.  Strange ....

Wonder if it is the driver version.


will deploy now and check


thanks.


Done.


If value contain id then your variable name should be variable_id.
you have to option for above 4 variable
option 1 : self.company = self.env.ref('base.main_company')
option 2: self.company_id = self.env.ref('base.main_company').id
select your convenient way and change this for good code :)


Wow, cool that we found this :-)


@etarena on second thought I would also recommend tweaking the commit message for each of these.  For example "Ops_template: Document recommended image" could simply be "Document recommended image".  The "Ops_template" part is redundant.  It's OK for each of the four commits to have the same message.


As previous comments.
Action action = objectFactory.createACTION()
initializeAction(action)


But `createSpy(fn).andCallThrough()` will call `fn`


ah, right, i guess i assumed `createSpy(fn)` was a "non-official" way of invoking it :)


Arrange this variable for Quality code
like:
1.self.env[*.*]
2.self.env.ref(*.*)


This is intentional as this is the root exception class thrown from DocumentStore APIs.


I think IMAGE and SUDO* should be removed.  Looks like cut and paste errors from other man pages.


Not at all. This is removing Backbone events and implementing flux. Foundation would be in the views. Those classNames "small-12", "large-6", column, row, etc


Turns out you can't setup up system environments in a running docker container. This ENV method would probably work on standalone installations that have access to their host PATH environments. I'm thinking of piping the subprocess output and parsing the given strings for the variable I need (rfport)


Thanks for the patch @tgrall.

Could you please check my comment in the last file and update the pull request.


it can be useful when you want to create a spy for a regular function. `spyOn` is just a convenient way to spy on the property of an object


I am looking for a provision to set proxyhost, proxyport when creating new Consul client. Instead of me setting these as -D args, can the Consul.newClient() accept proxyhost, proxyport, user and password fields?


I tried that at first, to use simply an empty array, but then calling `to_yaml` produces `emails: []` instead of `emails:\n -`.


maybe should be
``` from website.mails import QueuedMail, NO_LOGIN_TYPE ```
or
``` + from website.mails.queued_mails import NO_LOGIN_TYPE ```
so that you can do
``` Q('email_type', 'eq', NO_LOGIN_TYPE)


hi, did you find this bug also in the original gcm-erlang? if so, would you mind opening a PR?


I see the problem in the debugger now, on super. Seems like you take a whole different code path and miss some initializations. When doing `--test=0` we are going through `benchmark_all()` but that's not happening with `--test-full=0`?


@rvonstange DWMR. Keep pressing ahead!


@bnorthan sure - i can rename them...


Give proper argument :options :--
assertTrue(self.pos_order_pos0.account_move, "If its false then print this message")
or
self.assertEqual(self.pos_order_pos0.account_move, True, "If bot not equal then print this message")


This fixes the problem, thx


Nice. :+1:


Attempt to improve in 4f8a7492d1e72571f20baab28d8c24dec3320834.


What about `utils.isPath`?

And so are we going to drop `utils.unixify` from here?


I'm not seeing any tweened scaling effect at all.  What I'm seeing is this: http://i.imgur.com/LZgzkLi.jpg
It's scaling a 16:5 image to a 3:2 area, then clipping off whatever hangs outside.  i.e. exactly what the name says it does, *and* what the documentation says.  (Old version: http://i.imgur.com/uQk96bB.jpg  also clips some off, but clips off a different amount, slightly changing the aspect ratio of the result.)

Personally, I think either version is the wrong thing to use on a banner because it clips off part.  Something that scales to whichever dimension requires less scaling would be better, because it would maintain the aspect ratio and not clip part off.


No that's not it. We do go through `benchmark_all()` but benchmark_time is set to 1 for some reason.


`emails: []` in yaml is fine


Extracted in 8df4753a2e4a7afbf9ef8df2df51f55a1fcadcfa.


We should see if the older code had this problem (when we had to test functions).  I may have missed something, or it could be that the original test-full simply was failing to do things the right way.


@flavour This comes pass the __init__ function of the cap_AreaRepresent, but does not pass other methods as lookup_rows and represent_row https://github.com/biplovbhandari/eden/blob/master/modules/s3db/cap.py#L2500 .
The area_id is foreign key. What did I missed here ?