Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150623 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150623") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


I thought that for multi-line ones we'd keep the curly braces. It looks odd otherwise


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150629 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150629") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-150635 "deployed-2-env-Prod-20170915-150635") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


develop (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (2156 of 2156 tests passed, 0 test warnings)

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


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150644 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-150644") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


For reference, the errors looked like this:

```
--------------------------------------
History with id 244c246e3628fe4f in error - summary of datasets in error below.
--------------------------------------
| 3 - blastp rhodopsin_proteins.fasta vs 'four_human_proteins.fasta'
 (HID - NAME)
| Dataset Blurb:
|  error
| Dataset Info:
|  Fatal error:
|  /tmp/tmplgB1aH/job_working_directory/000/96/task_0:
|  Traceback (most recent call last):
|  File "./scripts/extract_dataset_part.py", line 17, in <module>
|  import galaxy.model.mapping  # need to load this before we unpickle, in order to se
| Dataset Job Standard Output:
|  *Standard output was empty.*
| Dataset Job Standard Error:
|  Fatal error:
|  /tmp/tmplgB1aH/job_working_directory/000/96/task_0:
|  Traceback (most recent call last):
|  File "./scripts/extract_dataset_part.py", line 17, in <module>
|  import galaxy.model.mapping  # need to load this before we unpickle, in order to setup properties assigned by the mappers
|  File "/home/travis/build/peterjc/galaxy_blast/galaxy-master/lib/galaxy/model/__init__.py", line 20, in <module>
|  from six import string_types
|  ModuleNotFoundError: No module named 'six'
|
--------------------------------------
```


develop (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


1. Add license.
2. Don't use pragma. Some compiler can not understand.


Same


Ok, I've swapped in `htmlspecialchars`: https://github.com/pkp/citationStyleLanguage/commit/c73fed0bf921326ee37e272e63cd32dfe98a3024

I think `citeproc-php` expects to receive and preserve HTML. It spits out an HTML string at the end and probably expects to receive `<i>` or `<em>` tags within citation titles that it wants to preserve.


Is fps supposed to be done with a fraction instead of a decimal number?


Can you specify better what you mean?


servlets poti config mai usor in AppGuiceServletContextListener.getInjector() cu serve(), stie guice


Is it different between struct tm and this?


Mohs and I worked to make this not necessary, is there any way to avoid it here? The app's hash change listener should cause the re-render.


License


I dunno. The hash change isn't triggering a re-render.


Enabling quad mode for the ISSI is25lp256d is a non-volatile setting in the status register so it's better to write it only if it's not already enabled.


nonono, leave this in there, just change the class definition...

You don't need to derive from `EB_icc` explicitly, that's already covered by depending on `EB_ifort`

But you still need this `elif icc` and call to `EB_icc.*`...


Is RN 0.46 for v5.1 a mistake? Both v4.0 and v5.1+ shares RN 0.46


-1 WHY START WITH DOUBLE LINKED LISTS???


This should resolve #3 .


How would that help this problem?


For readability, let's change like below.
```
if (xx < 0 || xx > 255) {
```


In other places, tabs are used, but this line indents with spaces.


GAYYYYYY


Whoops, ok


Good catch!


Created multidev environment [continuity-pro#pr-feature](https://dashboard.pantheon.io/sites/f4b4b00a-1cc1-4f25-a05d-4dce395bff80#pr-feature).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-feature-continuity-pro.pantheonsite.io/)


Couple privates I have are undetected AFAIK.
Not sure what changes exactly make them UD, perhaps something stupid like the font.
I am still thinking which way we should go in terms of anti-VAC


Oh, sorry, I misread the variable being freed. Nevermind. Middle of the night.


Typo in the `sign` -- double `n`.


@adzweig as it says in `README.md` "the only supported build configuration is cross-compiling from Linux to Windows", i.e. using a Linux computer to generate DLLs that only work with Windows. I'm happy to add proper support for running on Linux, but I'll need to do some work to the makefiles, and I'm not sure how long it will take.


the qty gets decremented with each order (i missed that in the first commit, added the line to do so in the second commit), so the listing could exist but be out of stock


Yeah, I ran yarn - wonder what went wrong. I'll pay closer attention to that when releasing.


@kaileu Hah. Completely forgot about that. I'll have to look at it.

That doesn't solve the schload-of-network-requests-on-startup problem, but does fix one of the bugs.

Thanks

---Alex


I think your mixing of ids for different things is not very elegant.


although it is redundant with the block expiration check which will always be 0.. maybe that's what you meant but the intent of that check is to make sure the listing isn't expired. seems like having these two separate might make sense just for understandability


the `algid` parameter should be explained here (it is also missing in the previous function `SEC_DerSignData`).


No I needed to clean it up, I was just reluctant because then I would have to build & test again.  Thanks for pushing me to update it.  :)

On Sep 14, 2017, at 5:40 PM, Landon Lunsford <notifications@github.com<mailto:notifications@github.com>> wrote:

TY - my inner OCD feels better now.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/SmarterApp/RDW_Reporting/commit/d772729287d1309d36842dd3160e5b8c0d6588f7#commitcomment-24315170>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AFgPbKhCPNkZELe70XE6e7hC75kJood-ks5siceKgaJpZM4PYWdz>.


You ought to put these magic numbers at the top of the contract, `uint constant dayNum = 14` and explain the rationale.


The comment still does not explain why you substract the last two characters. Please add an example.


мощно


Not very legible 😂


Use a `keypress` event inside the `events` property. Call a `keyCheck` or similar method to check the code then add the speaker.

Shouldn't be binding any DOM events in the constructor.


Removed!


TODO:
- JFX UI
- Upload to imgur


Done.


I was thinking to say that keeping orders in the contract was a waste of space, but I see that you enforce logic based on them, so you win.


Fix for issue #77.


Yes, I just realized the note in events methods :)


Euh, now it fails on my mac
```
cuberite/src/Root.cpp:93:22: error: implicit conversion loses integer precision: 'long'
      to '__darwin_suseconds_t' (aka 'int') [-Werror,-Wshorten-64-to-32]
                        Timeout.tv_usec = static_cast<long>(PollPeriod.count());
                                        ~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
make[2]: *** [src/CMakeFiles/Cuberite.dir/Root.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [src/CMakeFiles/Cuberite.dir/all] Error 2
make: *** [all] Error 2
```


Created multidev environment [continuity-pro#pr-feature](https://dashboard.pantheon.io/sites/f4b4b00a-1cc1-4f25-a05d-4dce395bff80#pr-feature).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-feature-continuity-pro.pantheonsite.io/)


I guess zero could make sense, the motivation for a non-zero value is that it's basically pointless refreshing the cache 10 seconds after it's done.


Hmm, I don't think so. We're talking about installed apps, if an app here is not installed, then we shouldn't really add it or we'll get it trouble later...
The only case where this could happen is in case of a race (apart from a bug in Flatpak) but I still think it's good to skip the app.


@carlocamilloni did you change this by hand or using the `./header.sh` script? In principle it should detect dates from git log. I also noticed this was changed (by me, with the script) in cef88ba424a80d6c4343a1f3c186ea2b2b19e9c6 and I was wondering which is the correct year for this file..


I think this should happen higher in the code. Check out line 117 and around.


hmmmmmm i just figured i'd really never run out of 256 bits and with blockchain where the transactions are necessarily sequential it'd be fine to use a single autoincrementing id for all my structs (thereby saving myself a *whole* 2 words of space in the market contract). it was easier for me to think about this where the structs were basically tables in a relational database where the id is the primary key because that's what i'm used to lol


lol oops.. need helper methods like vendorOf(uint orderId) 👍

or a better way to model the data....


i thought potentially an address would use multiple vendors, but i'm not sure why a single address would need to do that, except maybe for branding?


For consistency with the rest of the code, we use `ApiHelper` to do these conversions. But because this will change soon once the headless server is ready, I don't think it's worth worrying about now.


Example given and remove substr.


@zebastian any insight what needs corrected here?


Thank you!


Added new return array with duration (seconds)


FixExtraOutputVariables (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


@snicolet: thank you. But this patch uses pawn value for all attackers, am I right?


Fixes issue #473
When will this make it into a release please?


But why? did you get start getting bored of :trollface:?


tv_usec is not always a long :(


@Jakuje: this is just a clone repo that we use for workflow purposes.

NSS bugs get filed at:
https://bugzilla.mozilla.org/enter_bug.cgi?product=NSS&component=Libraries

Patches, CLs, etc. get posted at:
https://phabricator.services.mozilla.com/


Right of course, it will work for 5, but it won't work for 6. Generally I prefer future proofing the code, but I also would love to just remove or absorb replicate in the future. Maybe just add a deprecation warning for 5.2 saying that it will stop working in 6?


Neither the author nor the commit contents are correct. Pathetic kang


Where is the spec :)


wrong indent.


Perhaps `os.environ.get('DATADOG_SERVICE_NAME', 'celery')`


bb


@Jakuje I'm ashamed to confess that I don't quite know how (my git-fu is very weak). A short example would help a lot.


Are you sure? Seems like pretty drastic?


FixExtraOutputVariables (JasonGlazer) - i386-Windows-7-VisualStudio-14: OK (2759 of 2761 tests passed, 4 test warnings)

Messages:

 * 6 tests had: AUD diffs.
 * 2 tests had: Table big diffs.
 * 3 tests had: MTD diffs.

Failures:

regression Test Summary
 * Passed: 604
 * Failed: 2

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


It's highly essential security fix, man!


Looks like a bug: `activity` should be `action.activity`.


Was following the established pattern from other libs.

```
./ddtrace/contrib/falcon/patch.py:    service = os.environ.get('DATADOG_SERVICE_NAME') or 'falcon'
./ddtrace/contrib/flask/patch.py:    service = os.environ.get('DATADOG_SERVICE_NAME') or 'flask'
./ddtrace/contrib/pyramid/patch.py:    service = os.environ.get('DATADOG_SERVICE_NAME') or 'pyramid'
```


The following doc(s) have been deployed to the [2879backup](https://2879backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/backups](https://2879backup-static-docs.pantheonsite.io/docs/backups)


The simplest way is probably (since these are two latest commits)

    git rebase -i HEAD~~

 * it will open a vim. Replace the "pick" on the second line with "s" (squash), close vim (`:wq`).
 * It will open once more vim, where you adjust a commit message (you can skip this by writing `f` in previous step).
 * and you are done


Linux  4.8.0-53-generic #56~16.04.1-Ubuntu SMP Tue May 16 01:18:56 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux


Doing `or 'celery'` also helps to protect against an empty environment variable, e.g.:

```
$ DATADOG_SERVICE_NAME="" python3 -c "import os; print(repr(os.environ.get('DATADOG_SERVICE_NAME', 'celery')))"
''
```

vs

```
$ DATADOG_SERVICE_NAME="" python3 -c "import os; print(repr(os.environ.get('DATADOG_SERVICE_NAME') or 'celery'))"
'celery'
```


Add `infobae.net`

Test link, scroll down and click on `Paste` , and click on blue buttons.
http://www.megawarez.org/ciguena-apuros-dvdrip-720p-1080p-latino/

![una_ciguena_en_apuros_2017_en_dvdrip _720p _1080p__2017-09-15_17-29-20](https://user-images.githubusercontent.com/27441315/30491146-97403560-9a3c-11e7-91c9-da8b06e0299e.png)


Hi Gustavo,

First thanks for your work on this plugin.

Then, do you have any idea why this file is not the same as here ?

https://cdnjs.cloudflare.com/ajax/libs/lz-string/1.4.4/lz-string.js

Is this commit not supposed to be zithin 1.4.4 version ?

thanks


Yes, the main app is the last one. Now that I think of it maybe just comparing the app pointers... Let me change that.


sa


Hi Gustavo,

First thanks for your work on this plugin.

Then, do you have any idea why this file is not the same as here ?

https://cdnjs.cloudflare.com/ajax/libs/lz-string/1.4.4/lz-string.js

Is this commit not supposed to be within 1.4.4 version ?

thanks


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/other-peoples-private-conversations-in-suggested-messages/70111/4


@heavysink 请在提交中写你能够收到邮件的地址，这样 lilac 才能在打包失败时给你发通知。


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154419 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154419") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154424 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154424") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-154430 "deployed-2-env-Prod-20170915-154430") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154442 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-154442") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


`parents:` → `:parents` ?


Hmm, TravisCI still failed - I would have thought this would have grabbed the newest commits from Galaxy's master branch, but I don't see an easy way to check that.


Here too https://cdn.jsdelivr.net/npm/lz-string@1.4.4/libs/lz-string.js

Did you forget to increment version number zhile qdding this export maybe ?


Added.


It is already fine, you don't need to change anything. `getFramerate()` decides, if we use mlv framerate or overrated framerate. So both cases are fixed with that. FBongCam did the test for me.

Would be cool to have both - ffmpeg and AVFoundation - as export option... I am really waiting for that! :-)


I think the problem is due to the use of `--no-create-venv` at line 114 and `--skip_venv` at line 126.


Well, you can still wrap `rw` into `unique_ptr` by using a custom deleter. And it would be the preferred approach.


Let's add a test case for this to make it clear what this commit is trying to fix?


Sorry did not see it was a PR.

@pieroxy maybe ? thanks


yes:
- qmake needs to be corrected
- cmake needs to be corrected for linux
- osx needs to be checked
- cross compiler build scripts need to be reviewed
- packaging scripts need to be reviewed


Isn't this a potentially costly change since we'll now be copying vectors of data? Or does Vector contain a SEXP?


Strange that that works on https://github.com/peterjc/galaxy_mira/blob/master/.travis.yml and https://github.com/peterjc/pico_galaxy/blob/master/.travis.yml but likely down to the different datatypes somehow...


Incomplete sentence.


อันนี้ไม่ใช้ใช่ไหม


@DServy really?


Opdatér også gerne afhængighederne i dokumentations-READMEen.

!


ช่ายๆๆ  รอตอ ทำให้ อยู่ 55555


@IIvec Could you please point to the code where you think this happens?

In this first attempt the handling of pawns was not consistent with how it is defined for pieces, since for pieces `kingAttackersCount` counts the attacking pieces and `kingAdjacentZoneAttacksCount` counts the number of attacked squares, whereas my extra term adds the number of squares attacked by pawns to `kingAttackersCount` (which roughly but not always exactly equals the number of attacking pawns). I will perhaps retry the idea with a more consistent implementation.


Great jobs :smiley:


Fíjate lo que nos pide aquí, que el último parámetro se lo pasamos null y ya está, es fácil.
.
![image](https://user-images.githubusercontent.com/23742881/30491776-a0cdfbd8-9a3e-11e7-99a6-981db8c8cc31.png)


Could you use here widely accepted parameter name for threads number? Many tools use "--threads" :
* samtools: -@, --threads INT
* fastqc: -t --threads
* bowtie: -p/--threads
Also I don't see changes in ZinbraCLA which add this param support.


I think this should have been a separate pull request.  I have some questions about the order-item spec.  I know this is not finished but I am hesitant to approve this commit.  I think we should go through this together at some point.


@DStaleyCCC Awwwww fuck, will remove


It's just that `extract_dataset_parts.sh` (used when `<parallelism>` is active) assumes the virtualenv is at `GALAXY_HOME_DIR/.venv` if `GALAXY_VIRTUAL_ENV` is not configured.


`Rcpp::Vector` contains an `SEXP` (ref-counted via `R_PreserveObject()` and `R_ReleaseObject()`). No copies or duplication involved. It does fix the problem, but a more sustainable fix involves changing references to pointers in the affected constructors, which would make passing temporary objects much harder.


kenapa dijadiin tampan?


revert for a while, will be back with correct solution


## Aquí una explicación de este método. Parece fácil
.
![image](https://user-images.githubusercontent.com/23742881/30491907-1651eb76-9a3f-11e7-9fa5-9b822a59d8d6.png)


FixExtraOutputVariables (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1547 of 1547 tests passed, 0 test warnings)

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


FixExtraOutputVariables (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


FixExtraOutputVariables (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Is this needed at runtime because llvmdev is linked statically?


@odony your [comment above](https://github.com/odoo/odoo/commit/961b7820deabaddfd01fbb7d380feb854f354b6d#commitcomment-24324033) is totally reasonable. :+1:


PEP-8 violation


@ekr, yes, I asked @Jakuje to review this patch series informally, before submitting.  Thank you for the info anyway; I didn't know phabricator.


The following doc(s) have been deployed to the [2879backup](https://2879backup-static-docs.pantheonsite.io/docs) Multidev environment:
- [/docs/backups](https://2879backup-static-docs.pantheonsite.io/docs/backups)
- [/docs/unsupported-modules-plugins](https://2879backup-static-docs.pantheonsite.io/docs/unsupported-modules-plugins)


Y aquí un poquito de cómo está hecho notifiChange().
.
![image](https://user-images.githubusercontent.com/23742881/30492059-8bfb3ca6-9a3f-11e7-8803-f246016725b5.png)


Crude attempt to "solve" this but explicitly installing ``six`` https://github.com/peterjc/galaxy_blast/commit/d1f13052a9dde4cd02b844489b9764e985c55a5e https://travis-ci.org/peterjc/galaxy_blast/builds/275952811


I will release a new version once I have time to release it. Would you mind to use dev-master for the time being in your project(s)? Thank you for your understanding.


You'll go in a rabbit hole with this approach. Galaxy really wants its own virtualenv.


Me faltó agregar también que se quito el banner de imagenes al crear una nueva factura por cuestión de comodidad


Hi there,
I’m trying to have a create a target area (a red square) and a cancel button inside of the ZxingScannerView, to allow me to focus on the QR Code or cancel the activity.
Here is my Xaml.

<StackLayout Padding="20">
    <Label Text="Welcome to Xamarin Forms!"
           VerticalOptions="Center"
           HorizontalOptions="Center" />

    <Button Text="Scan" Clicked="Button_OnClicked"></Button>

    <Button Text="Scan2" Clicked="Button2_OnClicked"></Button>

    <forms:ZXingScannerView x:Name="ZXingScannerView1" IsScanning="True"                     OnScanResult="ZXingScannerView1_OnScanResult"
     HorizontalOptions="FillAndExpand" VerticalOptions="FillAndExpand"
     AutomationId = "zxingScannerView"
     IsAnalyzing="True" IsTorchOn="True" IsVisible="True">
        <forms:ZXingDefaultOverlay ShowFlashButton="True" BottomText="Scanning will happen automatically"                          TopText="Hold your phone up to the barcode"
                   FlashButtonClicked="ZXingDefaultOverlay_OnFlashButtonClicked"                                            AutomationId="zxingDefaultOverlay" IsVisible="True">

        </forms:ZXingDefaultOverlay>
    </forms:ZXingScannerView>
</StackLayout>

Any insights on how to create a target area or add a cancel button to the ZXingScannerView.
Thanks.


Ek sal nou nou daar wees sien nou nou

Sent from my iPhone

> On 15 Sep 2017, at 10:56 AM, git-for-windows-ci <notifications@github.com> wrote:
>
> The administrative script of Git for Windows identified a problem with this commit while running t1408-packed-refs.sh:
>
> make: Entering directory '/c/BuildAgent/_work/2/s/t'
> *** t1408-packed-refs.sh ***
> Initialized empty Git repository in C:/BuildAgent/_work/2/s/t/trash directory.t1408-packed-refs/.git/
> expecting success:
> 	test_tick &&
> 	git commit --allow-empty -m one &&
> 	one=$(git rev-parse HEAD) &&
> 	git for-each-ref >actual &&
> 	echo "$one commit	refs/heads/master" >expect &&
> 	test_cmp expect actual &&
>
> 	git pack-refs --all &&
> 	git for-each-ref >actual &&
> 	echo "$one commit	refs/heads/master" >expect &&
> 	test_cmp expect actual &&
>
> 	git checkout --orphan another &&
> 	test_tick &&
> 	git commit --allow-empty -m two &&
> 	two=$(git rev-parse HEAD) &&
> 	git checkout -B master &&
> 	git branch -D another &&
>
> 	git for-each-ref >actual &&
> 	echo "$two commit	refs/heads/master" >expect &&
> 	test_cmp expect actual &&
>
> 	git reflog expire --expire=now --all &&
> 	git prune &&
> 	git tag -m v1.0 v1.0 master
>
> ++ test_tick
> ++ test -z ''
> ++ test_tick=1112911993
> ++ GIT_COMMITTER_DATE='1112911993 -0700'
> ++ GIT_AUTHOR_DATE='1112911993 -0700'
> ++ export GIT_COMMITTER_DATE GIT_AUTHOR_DATE
> ++ git commit --allow-empty -m one
> [master (root-commit) 76a57ce] one
>  Author: A U Thor <author@example.com>
> +++ git rev-parse HEAD
> ++ one=76a57ce2bf33b96e5824acab4b2827d4cddce759
> ++ git for-each-ref
> ++ echo '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master'
> ++ test_cmp expect actual
> ++ mingw_test_cmp expect actual
> ++ local test_cmp_a= test_cmp_b=
> ++ local stdin_for_diff=
> ++ test -s expect
> ++ test -s actual
> ++ mingw_read_file_strip_cr_ test_cmp_a
> ++ local line
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ line='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ eval 'test_cmp_a=$test_cmp_a$line'
> +++ test_cmp_a='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ test -z ''
> ++ break
> ++ mingw_read_file_strip_cr_ test_cmp_b
> ++ local line
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ line='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ eval 'test_cmp_b=$test_cmp_b$line'
> +++ test_cmp_b='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ test -z ''
> ++ break
> ++ test -n '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ test -n '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ test '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> ' = '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ git pack-refs --all
> ++ git for-each-ref
> ++ echo '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master'
> ++ test_cmp expect actual
> ++ mingw_test_cmp expect actual
> ++ local test_cmp_a= test_cmp_b=
> ++ local stdin_for_diff=
> ++ test -s expect
> ++ test -s actual
> ++ mingw_read_file_strip_cr_ test_cmp_a
> ++ local line
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ line='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ eval 'test_cmp_a=$test_cmp_a$line'
> +++ test_cmp_a='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ test -z ''
> ++ break
> ++ mingw_read_file_strip_cr_ test_cmp_b
> ++ local line
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ line='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ eval 'test_cmp_b=$test_cmp_b$line'
> +++ test_cmp_b='76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ :
> ++ IFS=$'\r'
> ++ read -r -d '
> ' line
> ++ test -z ''
> ++ break
> ++ test -n '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ test -n '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ test '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> ' = '76a57ce2bf33b96e5824acab4b2827d4cddce759 commit	refs/heads/master
> '
> ++ git checkout --orphan another
> Switched to a new branch 'another'
> ++ test_tick
> ++ test -z set
> ++ test_tick=1112912053
> ++ GIT_COMMITTER_DATE='1112912053 -0700'
> ++ GIT_AUTHOR_DATE='1112912053 -0700'
> ++ export GIT_COMMITTER_DATE GIT_AUTHOR_DATE
> ++ git commit --allow-empty -m two
> [another (root-commit) 169e1dc] two
>  Author: A U Thor <author@example.com>
> +++ git rev-parse HEAD
> ++ two=169e1dcee4718eeffd6beeb0d76c0d4299bc7d4f
> ++ git checkout -B master
> Switched to and reset branch 'master'
> ++ git branch -D another
> error: error replacing .git/packed-refs: Permission denied
> error: Error deleting branch 'another'
> + test_eval_ret_=1
> + want_trace
> + test t = t
> + test t = t
> + set +x
> error: last command exited with $?=1
> not ok 1 - setup
> #
> #		test_tick &&
> #		git commit --allow-empty -m one &&
> #		one=$(git rev-parse HEAD) &&
> #		git for-each-ref >actual &&
> #		echo "$one commit	refs/heads/master" >expect &&
> #		test_cmp expect actual &&
> #
> #		git pack-refs --all &&
> #		git for-each-ref >actual &&
> #		echo "$one commit	refs/heads/master" >expect &&
> #		test_cmp expect actual &&
> #
> #		git checkout --orphan another &&
> #		test_tick &&
> #		git commit --allow-empty -m two &&
> #		two=$(git rev-parse HEAD) &&
> #		git checkout -B master &&
> #		git branch -D another &&
> #
> #		git for-each-ref >actual &&
> #		echo "$two commit	refs/heads/master" >expect &&
> #		test_cmp expect actual &&
> #
> #		git reflog expire --expire=now --all &&
> #		git prune &&
> #		git tag -m v1.0 v1.0 master
> #
> make: *** [Makefile:49: t1408-packed-refs.sh] Error 1
> make: Leaving directory '/c/BuildAgent/_work/2/s/t'
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160254 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160254") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160300 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160300") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-160309 "deployed-2-env-Prod-20170915-160309") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160318 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-160318") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


I will check again, but it work with my Pi on Raspbian Jessie (2017-01-11-raspbian-jessie-lite).
What is the behavior ?


Indeed, not sure what I was thinking 😄


注意一下代码风格，else不要写一行啦不然很难看


I had it later... I usually run rebase interactive with a command to compile after each commit but I totally forgot.
I will do that after checking all your comments.


ok i will close https://github.com/buddhi1980/mandelbulber2/issues/336.
we can start a new dialog with these TODO's


Θέλετε να λαμβάνεται πρώτοι ενημερώσεις για ό,τι συμβαίνει στο http://freddoespresso.gr ;


:+1: thanks


I now added the message originator as a requirement, does it qualify as a challenge?


Add a failure callback that invokes `onstop` just in case.


why is this still here... thought i overwrote history


This looks like a sane fix/workaround to an annoying problem. :-)

Perhaps we should stop calling parse_shows() in the init and only actually call it if the user selects the NFL Network section? That way, we won't slow down anything unless this information is actually being needed.


I'm not sure what the utility is to having a hard-coded default install location that doesn't respect the setting from --with-plugindir; it would seem better to me to change https://github.com/cyrusimap/cyrus-sasl/blob/2ea0649f5cafce4f0b597d82f7acca7721e96cc8/plugins/Makefile.am#L66 to set sasldir to the same value as plugindir. People who need the current behaviour would be able to override sasldir with e.g. `make sasldir=/usr/lib/sasl2` (an approach we currently use to make sure they're installed to plugindir) but it would be more difficult for people to end up with a broken install.


裸写字符串！


I did, since messages are always signed by their author on Ethereum, I added msg.sender as a requirement, Please feel free to let me know if that is satisfying (I was hesitant to use a method like ecrecover)


Ok, great.


I do not think of malicious action when crafting rules, so that makes sense. A revision of all regexps with two gready * or + operators may be needed. This is a too laborious task, so can the regexp engine we use be limited so that we had + or * as a defined arbitrary number, like, say 100 or 200?

On another note, the main performance hog in [pt] is hunspell, particularly it's suggestions engine.
Using oddball test cases, if you run a foreign test in portuguese you can get up to 200x longer checking time. This is noticeable in the standalone tool. I believe it doesn't affect the server since it times-out.


was just looking at that (mutex)

On Fri, Sep 15, 2017 at 9:04 AM, joshrosso <notifications@github.com> wrote:

> Merging "matt-deboer/sync_upstream"'s AWS sync
>
> - Adds AWS sync on an interval (defaulting to 3 minutes)
>
> - Brought @matt-deboer 's work into alignment with our most recent code
> changes.
>
> - TODO: I believe we're in need of some form of mutex. Based on my
> reading of the code, it is possible for the controller to be
> mid-reconcile and have it's ingress swapped out from an AWS sync.
>
> - Resolves: https://github.com/coreos/alb-ingress-controller/issues/161
>
> --
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub:
> https://github.com/coreos/alb-ingress-controller/commit/
> 7e3dfa9a4c1440d197839ac2a0c9f100b9c190aa
>


@cescoffier - the release build is failing with your last push, can you please check it , so that I can merge this for our release


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/9cef1303-33f3-4ed3-8af5-6c6cfb5de726 "kartees-s3-monitoring"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/acd998c4-a5f2-49c3-945d-6f2893384550 "kartees-s3-monitoring") deployed [kartees-s3-monitoring](https://github.com/mgzeitouni/kartees-s3-monitoring/releases/tag/deploy-ibm-yp-us-south-Kartees-dev-20170915-161152 "deploy-ibm-yp-us-south-Kartees-dev-20170915-161152") to [dev](https://console.bluemix.net/apps/e713c8b0-fae3-4e0d-9b83-8301b63fea7c?env_id=ibm:yp:us-south "ibm:yp:us-south:Kartees:dev"), including this commit


[Jenkins vg tests](http://jenkins.cgcloud.info/job/vg/539/) complete for merge to master.  View the [full report here](http://cgl-pipeline-inputs.s3.amazonaws.com/vg_cgl/vg_ci/jenkins_reports/branch/master/index.html).

9 tests passed, 0 tests failed and 10 tests skipped in 6995 seconds

Tests produced 7309 warnings. 7309 were for lower-than-expected alignment scores


Initial commit


Good catch, however we don't guarantee exact number of threads used.


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161222 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161222") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


That with the current season for null seems to be a good workaround to reduce the network usage to a minimum.

Mit freundlichen Grüßen
Kai Leutner

> Am 15.09.2017 um 18:10 schrieb emilsvennesson <notifications@github.com>:
>
> This looks like a sane fix/workaround to an annoying problem. :-)
>
> Perhaps we should stop calling parse_shows() in the init and only actually call it if the user selects the NFL Network section? That way, we won't slow down anything unless this information is actually being needed.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161227 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161227") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-161233 "deployed-2-env-Prod-20170915-161233") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161242 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-161242") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


como podemos hacer un prefetch_related(Libros)  para precargar los libros del autor en un solo query en vez de hacer un query por cada libro?


第n次裸写字符串==


Given the amount of changes next Crystal version IMO should be more like `0.24.0`.


should be named init


CSDL文件由赵祥伟提供！


agregamos la carpeta static y modificamos settings.py
agregamos links a los autores y libros
¿como podemos utilizar prefetch_related(Libros) en la vista AuthorDetail?


We should force to `true` in that case. Because it's not a user error to provide an unknown field, there's nothing he can do about that.


No. It returns a _FlatpakInstalledRef_ and I am almost sure I saw a warning which made me do this cast.


FixExtraOutputVariables (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2795 of 2801 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: Table big diffs.
 * 3 tests had: MTD diffs.

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


The cleanup of `sigitem.data` disapeared for normal RSA operations.
```
        PORT_Free(sigitem.data);
        sigitem.data = NULL;
```
It should be probably cleaned also while the failures in PSS branch.


Created multidev environment [continuity-pro#pr-feature](https://dashboard.pantheon.io/sites/f4b4b00a-1cc1-4f25-a05d-4dce395bff80#pr-feature).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://pr-feature-continuity-pro.pantheonsite.io/)


    bootstrap-mgmt.sh - Install/Config Ansible & Deploy Code Snippets
             examples - Code Snippets
               README - This file ;)
          Vagrantfile - Defines Vagrant Environment


Ok, I will revert this.


In this case, even replacing the `+` with `{1,3}` wasn't enough to get it fast. Maybe we could have some internal performance monitoring, so we notice these cases early.

About the speller, most languages use `MorfologikSpellerRule` instead of `HunspellRule` exactly because of this issue. Some more details can be found at http://wiki.languagetool.org/hunspell-support#toc1


Uploads and vendors ignored @.gitignore
Phpstorm's update @workspace.xml
Default bundles removed
Encryption key @defaults.json
New errror lines @errors.json
New encryption and converter class implemented @ActiveClass.php
Converter class added (incl. private fielded object to array func.) @Core
Encrypter class added @Core
New quote sign used @Database.php
Manual loading canceled. it replaced with automatic loading @Loader.php
New model interface added for common repository options @Core
Possible bug reason fixed@Session.php
New classes included @index.php

Reference
-----------
@.gitignore: ./.gitignore
@workspace.xml: ./.idead/workspace.xml
@defaults.json: ./core/Config/defaults.json
@errors.json: ./core/Config/errors.json
@ActiveClass.php: ./core/System/ActiveClass.php
@Core: ./core/System
@Database.php: ./core/System/Database.php
@Loader.php: ./core/System/Loader.php
@Session.php: ./core/System/Session.php
@index.php: ./public/index.php


由好朋友MikesWei（https://github.com/MikesWei）指导完成！


I think nobody guarantees ;)


SettingFragment <Activity>에서 삭제하기


Already fixed in 737d065092fbfa06883158c282b63486b42f5754


Lần commit này dán code lộn rồi.


No, RN 0.46 for v5.1 is not mistake - we've restored backwards compatibility in v5.1


of challenge -> of the challenge/of a challenge?


**Whoops!**

There were *1 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/updates/2017/09/autoplay-policy-changes.md#L153`](https://github.com/Google/WebFundamentals/blame/c72923bff27777cb70c965088cb465865ba86e20/src/content/en/updates/2017/09/autoplay-policy-changes.md#L153) - '<script> tags are generally not allowed, please double check.


Separates routes by v1 designation as data, apps, or dbtables


Yes, it'll be `0.24.0`. This says *after* 0.23.1, not sure what the issue is.


Preview.


oh i was still doing it wring lol! thanks for the tips.


I am aware of it. I have considered it recently and I have seen the different implementations LT uses.
My main issue is that I maintain the dictionary mostly as my system and LO use. This allows me to efficiently develop and test the dictionary.
Having another task is not on my plans, yet. Ideally, another person to handle the spelling but, lately, there seems to be a lack of volunteers.


FixExtraOutputVariables (JasonGlazer) - Win64-Windows-7-VisualStudio-14: OK (2759 of 2761 tests passed, 4 test warnings)

Messages:

 * 6 tests had: AUD diffs.
 * 2 tests had: Table big diffs.
 * 3 tests had: MTD diffs.

Failures:

regression Test Summary
 * Passed: 604
 * Failed: 2

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


**Whoops!**

There were *1 warnings* that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.

**WARNINGS**
[`src/content/en/updates/2017/03/chrome-58-media-updates.md#L215`](https://github.com/Google/WebFundamentals/blame/8420493e74303cf8a2d19917c85f63f394d4d5d7/src/content/en/updates/2017/03/chrome-58-media-updates.md#L215) - '<script> tags are generally not allowed, please double check.


lol


The check in Line 120 is something completely different - it checks if `item` is actually a subfolder of `directory` assuming that `directory` refers to an existing folder.
#659 adds an additional check if the function did get some non-directory value for `directory`.


Por defecto, una vez declarado el Provider, el mismo puede proveer de contenidos tanto para el acceso a los datos como para la modificación también, por parte de otras apps.
El mismo AndroidStudio advierte de esto y te sugiere, demás de lo generado automáticamente(lo mínimo) para declarar el provider, que le metas otro argumento diciendo si deseas o no que lo anterior suceda, que **## pueda el contenido de los datos ser o no EXPORTABLE**. Mira.
.
![provider exported](https://user-images.githubusercontent.com/23742881/30493235-a834da54-9a43-11e7-9f56-c723ec3693e8.PNG)


We're not talking about this.


For me, that is the issue....I need sasldir and plugindir to match, but
there is no way to set sasldir, so I had to patch it to be settable.

My patch also checks that the 2 match, INSTEAD of comparing to something
hard-coded.  Having them match anyway would fix that.

On 9/15/2017 9:10 AM, flowerysong wrote:
>
> I'm not sure what the utility is to having a hard-coded default
> install location that doesn't respect the setting from
> --with-plugindir; it would seem better to me to change
> https://github.com/cyrusimap/cyrus-sasl/blob/2ea0649f5cafce4f0b597d82f7acca7721e96cc8/plugins/Makefile.am#L66
> to set sasldir to the same value as plugindir. People who need the
> current behaviour would be able to override sasldir with e.g. |make
> sasldir=/usr/lib/sasl2| (an approach we currently use to make sure
> they're installed to plugindir) but it would be more difficult for
> people to end up with a broken install.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/cyrusimap/cyrus-sasl/commit/2ea0649f5cafce4f0b597d82f7acca7721e96cc8#commitcomment-24326253>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AWZUI8FNnnQBYCtKaDik6XMgWmrxhFkzks5siqExgaJpZM4PZMes>.
>

--
Jan Parcel, Software Developer
Oracle Systems Server & Cloud Engineering


> In this case, even replacing the + with {1,3} wasn't enough to get it fast. Maybe we could have some internal performance monitoring, so we notice these cases early.

Even though I have looked at the performance logs before, I have reinstated the performance logging in my build, but it has varying results, since they are dependent on the system performance at runtime.


yea


Indeed... my cache got stale as I was still living in belief that the last version is `0.23.0`, sorry for the noise! :man_facepalming:


Hello,

Its been a few years since you touched this code. Still, I'm wondering if you remember if the error with convertToSignedPrimitive() was a compiler error, or a logical error? If you switched compiler / os / optimization level, did the code work?

Best Regards
Melvyn


[R/get_fire_season.R:2:3:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/R/get_fire_season.R#L2) *style:* **Trailing whitespace is superfluous.**
```r
#'
  ^
```
[R/mask_crop_subset.R:29:7:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/R/mask_crop_subset.R#L29) *style:* **Commented code should be removed.**
```r
    # message("Masking raster over polygon")
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/mask_crop_subset.R:42:1:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/R/mask_crop_subset.R#L42) *style:* **Trailing whitespace is superfluous.**
```r

^~~~
```
[R/plot_obs_vs_forecast.R:119:54:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/R/plot_obs_vs_forecast.R#L119) *style:* **Commas should always have a space after.**
```r
        raster_mean_matrix[i, j] <- round(perc_layer,2)*100
                                                     ^
```
[R/plot_obs_vs_forecast.R:119:56:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/R/plot_obs_vs_forecast.R#L119) *style:* **Put spaces around all infix operators.**
```r
        raster_mean_matrix[i, j] <- round(perc_layer,2)*100
                                                      ~^~
```
[tests/testthat/test-get_percentile_raster.R:28:3:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/tests/testthat/test-get_percentile_raster.R#L28) *style:* **Variable or function name should be snake_case.**
```r
  probs_maps_1_B <- get_percentile_raster(probs = 50, input_raster = s)
  ^~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:43:3:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/tests/testthat/test-get_percentile_raster.R#L43) *style:* **Variable or function name should be snake_case.**
```r
  probs_maps_2_B <- get_percentile_raster(probs = c(50, 75), input_raster = s)
  ^~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:62:3:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/tests/testthat/test-get_percentile_raster.R#L62) *style:* **Variable or function name should be snake_case.**
```r
  probs_maps_1_A <- get_percentile_raster(probs = 50,
  ^~~~~~~~~~~~~~
```
[tests/testthat/test-get_percentile_raster.R:80:3:](https://github.com/ecmwf/caliver/blob/3ed9a34890cafda2f624c1202fdbc47de15c0a9f/tests/testthat/test-get_percentile_raster.R#L80) *style:* **Variable or function name should be snake_case.**
```r
  probs_maps_2_A <- get_percentile_raster(probs = c(50, 75),
  ^~~~~~~~~~~~~~
```


IDE fails?


Nothing wrong with the code here, but I *strongly* recommend coming up with a better naming convention for all customization Objects (fields, searches, lists, etc) across your organization. It is a *nightmare* as a developer trying to keep these custom field IDs straight when they are just numbers. It's much better to read something like `custrecord_status`.


Note that Bool is special, in that all of the methods have to be placed into it manually.

So it doesn't get this iterator. (or the `.WHICH` change, but that probably isn't necessary.)


My point is that if you remove `_changeMenu()`, this function does nothing and you could also remove the other checks, or even the whole function :-)


También, propone, de entrada, cerrar la etiqueta de una forma que no es la que me han enseñado y el mismo AndroidStudio, que es el que me la propone, no la ve apropiada. Me refiero a esto:
.
![image](https://user-images.githubusercontent.com/23742881/30493373-2ae8526e-9a44-11e7-9057-e5678523af6d.png)


Why wait so long to do this check if it's the key to whether you send the email or not? I'd move this as early as possible, and do something more like:

```javascript
function emailSolution(context) {
    var status = context.currentRecord.getValue({fieldId: "custrecord37"});
    if (status !== "3") {
        return;
    }
    // rest of function...
}
```


OK, well I've got a form that shows up now, but no Account info gets saved anywhere as far as I can tell.. Trying to dig into the code and sort out what I should be doing, but I'm struggling!


On 9/15/2017 9:10 AM, flowerysong wrote:
>
> I'm not sure what the utility is to having a hard-coded default
> install location that doesn't respect the setting from
> --with-plugindir; it would seem better to me to change
> https://github.com/cyrusimap/cyrus-sasl/blob/2ea0649f5cafce4f0b597d82f7acca7721e96cc8/plugins/Makefile.am#L66
> to set sasldir to the same value as plugindir. People who need the
> current behaviour would be able to override sasldir with e.g. |make
> sasldir=/usr/lib/sasl2| (an approach we currently use to make sure
> they're installed to plugindir) but it would be more difficult for
> people to end up with a broken install.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/cyrusimap/cyrus-sasl/commit/2ea0649f5cafce4f0b597d82f7acca7721e96cc8#commitcomment-24326253>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AWZUI8FNnnQBYCtKaDik6XMgWmrxhFkzks5siqExgaJpZM4PZMes>.
>

For me, that is the issue....I need sasldir and plugindir to match, but
there is no way to set sasldir, so I had to patch it to be settable.

My patch also checks that the 2 match, INSTEAD of comparing to something
hard-coded.  Having them match anyway would fix that.

-       @if test "$(plugindir)" != "$(prefix)/lib/sasl2"; then \
+       @if [ "X$(sasldir)" != "X$(plugindir)" ] ; then \

Here's what I do in Makefile after patching:

SASL2.32 = $(USRLIBDIR)/sasl2
SASL2.64 = $(USRLIBDIR)/sasl2/$(MACH64)

CONFIGURE_OPTIONS.32 += --with-plugindir=$(SASL2.32)
CONFIGURE_OPTIONS.64 += --with-plugindir=$(SASL2.64)
CONFIGURE_OPTIONS.32 += --with-sasldir=$(SASL2.32)
CONFIGURE_OPTIONS.64 += --with-sasldir=$(SASL2.64)

That's a lot of trouble to go to just to get them to install and be
looked for in the same place.

--
Jan Parcel, Software Developer
Oracle Systems Server & Cloud Engineering


nu gasesc


Yeah, I think it is OK to unconditionally remove all the directory files. Let's stop deleting any desktop files from this function.


FWIW, there was some discussion about this commit: https://irclog.perlgeek.de/perl6-dev/2017-09-15#i_15168387 and https://irclog.perlgeek.de/perl6-dev/2017-09-15#i_15168719

```


nu eu am facut aplicatia asta, astept raspuns!!!


It would be a good exercise for you to move this value out to a Script Parameter. If you're not familiar with these, check out the Help doc "SuiteScript 1.0 Creating Script Parameters (Custom Fields)" as a starting point. Although it's 1.0 content, the basic concepts still apply. Once you're ready to read the parameter in code, you'll want to check out the `N/runtime` module.


The details are fuzzy; but, if I remember correctly, the issue was not a compiler error, but a valid compiler design decision. Changing compilers "fixed" the issue, but left open the possibility that a future version would have the same problem.

Sent from my iPad

On Sep 15, 2017, at 12:37 PM, melvyniandrag <notifications@github.com<mailto:notifications@github.com>> wrote:

Hello,

Its been a few years since you touched this code. Still, I'm wondering if you remember if the error with convertToSignedPrimitive() was a compiler error, or a logical error? If you switched compiler / os / optimization level, did the code work?

Best Regards
Melvyn

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<https://github.com/kurmasz/bigint/commit/a1ecc8c6da543476e882bf25d87863014bfb954f#commitcomment-24326650>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAYQpCOWIPlTFajUlPLMYnM8tJMXUCcVks5siqeogaJpZM4PZOdY>.


Please tag properly as `v0.3.14`.


"Functions and sessions and database connection. Not for users"


I understand


From an architectural perspective, I would suggest moving this functionality to a User Event's `afterSubmit` event. That is usually the best place to handle sending out notifications after a record update. Doing this on the client side will bog down the time it takes to save a record.


Really good start!!


Thanks for the quick response!


Would it be better to scrape https://github.com/ruby/www.ruby-lang.org/blob/master/_data/releases.yml? That's where the Releases page info comes from.


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164924 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164924") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164931 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164931") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-164937 "deployed-2-env-Prod-20170915-164937") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164947 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-164947") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


lgtm thanks


@behdad I just hit an AssertionError at this line while testing some test fonts I got. If `axisPoints[axis]` is a set, why do we have to `assert value not in axisPoints[axis]` before calling `.add(value)`? The set is a collection of unique items anyway, calling add is idempotent.


These are not necessarily Chromium, since this function was adapted to also work for Chrome.


This now allows `sparse[1.0]` - is that deliberate?


Yeah, I've actually got the following change sitting locally waiting for Jenkins to commit so I don't mess that up, to remind us to look into both that file and `downloads.yml` as well:
```diff
diff --git a/update.sh b/update.sh
index 3d86d98..efc43e6 100755
--- a/update.sh
+++ b/update.sh
@@ -10,7 +10,8 @@ fi
 versions=( "${versions[@]%/}" )

 releasesPage="$(curl -fsSL 'https://www.ruby-lang.org/en/downloads/releases/')"
-newsPage="$(curl -fsSL 'https://www.ruby-lang.org/en/news/')" # occasionally, releases don't show up on the Releases page
+newsPage="$(curl -fsSL 'https://www.ruby-lang.org/en/news/')" # occasionally, releases don't show up on the Releases page (see https://github.com/ruby/www.ruby-lang.org/blob/master/_data/releases.yml)
+# TODO consider parsing https://github.com/ruby/www.ruby-lang.org/blob/master/_data/downloads.yml as well

 latest_gem_version() {
        curl -fsSL "https://rubygems.org/api/v1/gems/$1.json" | sed -r 's/^.*"version":"([^"]+)".*$/\1/'
```


Perhaps we can just omit the browser name and just say `error while parsing $path`


I wonder if self.time_window_length should be in range(150, 400)?


No fetches are necessary here, it just picks up the number from the page because in the line:

```
const countEl = select('.notification-center .count', notifications);
```

`notifications` is undefined so it just looks in the current page.


Please use camel case for variable names


These two try/catch use inconsistent style for whether a space is put before the paren.
gnome-shell is generally a bit inconsistent, but at least let's keep it consistent across the file.


[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](http://test.pantheonsite.io/)

This is a test


/cc @sdodson


/cc @kargakis


Good to know. Thank im change ma package.json config to fixed number 0.14.0  my old config ^0.13.0 is to risky


It's sanity check.  You can hit the assert if you have two or more masters at the same location.

Why are you commenting here though, instead of opening a new issue?! :)


And another test


I find that `switch`es like this to be very verbose and usually unnecessary. In JS, you can pass around references to functions, and I use that to accomplish this same thing more like this:

```javascript
function displayItemInfo(request, response) {
    var infoName = request.getParameter("infoType");
    var itemId = request.getParameter("itemId");

    var displayFn = {
        "specs": itemSpecs,
        "included": includedItems,
        "download": downloads
    };

    displayFn[infoName](itemId, response);
}
```

To break it down a little further:

1. `displayFn` simply maps an `infoName` value to the appropriate `function`.
1. `displayFn[infoName]` is thus a *reference* to the appropriate function
1. `displayFn[infoName](itemId, response)` is thus *invoking* the appropriate function.

You could also add some checks to make sure that `infoName` is found in `displayFn`:

```javascript
if (displayFn[infoName]) {
    displayFn[infoName](itemId, response);
}
```


Thanks.


FYI this breaks

```sh
mvn surefire:test -Dtest=SomeTest\#method
```

used for example by the NetBeans IDE to run individual tests quickly, when they have already been compiled. Is there a way to default `@{argLine}` to blank, in case `jacoco:prepare-agent` has been omitted? Or put the whole `jacoco` execution into a profile activated by default only when the `test` property is unset, or even not activated by default and only by `-Pjacoco` in the `Jenkinsfile`?


Is it really that awesome??


Fix #3


I think the removal of this function may belong to another commit?


This will not work without driver changes: https://github.com/oblitum/Interception/issues/25#issuecomment-204277598


Alphabetical order


# Really?? Meh..


Is it your actual password?


Actually, I don't remember why. @gulios may know. I think where should be a PR for this (see "Copy nested directories")


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165830 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165830") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165836 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165836") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-Prod-20170915-165841 "deployed-2-env-Prod-20170915-165841") to [Prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "There was a problem retrieving the space name"), including this commit


Bluemix toolchain: An unknown tool deployed [testRepo-Traceability](https://github.com/idsdev/testRepo-Traceability/releases/tag/deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165850 "deployed-2-env-ibm-yp-us-south-user-ibm.com-prod-20170915-165850") to [prod](https://console.bluemix.net/apps/6e3bc311-c83d-4cd1-a457-99d9a5f20f19?env_id=ibm:yp:us-south "ibm:yp:us-south:user@ibm.com:prod"), including this commit


I find it incredibly annoying that `nlapiSearchRecord` returns `null` for empty results. Instead of needing these `null` checks everywhere, I avoid them altogether by doing this whenever I call `nlapiSearchRecord`:

```javascript
var searchResults = nlapiSearchRecord(searchType, null, filters, columns) || [];
```

The important piece here being the `|| []`.

The `||` operator can be used as a progressive fallback, to provide default values when the primary value is `null` or some other unacceptable value. This way, when `nlapiSearchRecord` returns `null`, the `||` operator falls back to an empty Array instead.

Now, I can simply avoid the `null` check and write a `for` loop to process my results. Since the Array is empty, we'll just never enter the loop and the code will move on.


ここで固定されてしまってる可能性


You could actually let the server handle this instead by providing a `sort` directly on the search column.

```javascript
new nlobjSearchColumn('custrecord_item_spec_name_id').setSort(false); // false for ascending, true for descending
```


[Jenkins vg tests](http://jenkins.cgcloud.info/job/vg/541/) complete for merge to master.  View the [full report here](http://cgl-pipeline-inputs.s3.amazonaws.com/vg_cgl/vg_ci/jenkins_reports/branch/master/index.html).

9 tests passed, 0 tests failed and 10 tests skipped in 6660 seconds

Tests produced 7344 warnings. 7344 were for lower-than-expected alignment scores


Is it like a chain of custody?


NoClassDefFound?


Bluemix [toolchain](https://console.bluemix.net/devops/toolchains/9cef1303-33f3-4ed3-8af5-6c6cfb5de726 "kartees-s3-monitoring"): [Delivery Pipeline](https://console.bluemix.net/devops/pipelines/acd998c4-a5f2-49c3-945d-6f2893384550 "kartees-s3-monitoring") deployed [kartees-s3-monitoring](https://github.com/mgzeitouni/kartees-s3-monitoring/releases/tag/deploy-ibm-yp-us-south-Kartees-dev-20170915-170328 "deploy-ibm-yp-us-south-Kartees-dev-20170915-170328") to [dev](https://console.bluemix.net/apps/e713c8b0-fae3-4e0d-9b83-8301b63fea7c?env_id=ibm:yp:us-south "ibm:yp:us-south:Kartees:dev"), including this commit


I would feel more comfortable if there was another line with a counterbalancing `-=` on another field.


Looks like this is repeated


OK, all done. But you've diverged now.


あーこれformを使うの試したときの名残ですね直さないと


Here too, being `msg.sender` should be enough to confirm.
That would require you to change your data structure, but it would be more elegant.


Test post, please ignore


yep.


Why is it an `if` and not a `require`?


@jbrooksuk This broke the initial setup process for me. Clicking on next does nothing. Browser console gives this:

```
manifest.d41d8cd98f00b204e980.js:1 Uncaught TypeError: Cannot read property 'call' of undefined
    at n (manifest.d41d8cd98f00b204e980.js:1)
    at Object.<anonymous> (all.933ef52c701c02556f3b7fa32b0d5f5d.js:1)
    at n (manifest.d41d8cd98f00b204e980.js:1)
    at Object.<anonymous> (all.933ef52c701c02556f3b7fa32b0d5f5d.js:1)
    at n (manifest.d41d8cd98f00b204e980.js:1)
    at Object.<anonymous> (all.933ef52c701c02556f3b7fa32b0d5f5d.js:5)
    at n (manifest.d41d8cd98f00b204e980.js:1)
    at Object.<anonymous> (all.933ef52c701c02556f3b7fa32b0d5f5d.js:5)
    at n (manifest.d41d8cd98f00b204e980.js:1)
    at Object.<anonymous> (all.933ef52c701c02556f3b7fa32b0d5f5d.js:1)
```


A great alternative to `for` loops is the Array's `map` function: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map

If I were writing this same functionality, it might look something like this instead:

```javascript
function itemSpecs(itemId, response) {
    // ...

    // instead of for loop:
    var tableBody = searchResults.map(resultToHtml).join("");
    var specTable = "<table class='specsTable'>" + tableBody + "</table>";

    // ...
}

function resultToHtml(result) {
    return "<tr><td>" + result[0] + "</td><td>" + result[1] + "</td></tr>";
}
```


Ah! I was thinking of _changeMenu(), not _onMenuSourceEnter(). Sorry for the confusion, it's clear now


Boom!


If you don't like escaping quotes, JavaScript allows you to mix quote styles (single vs double) in order to display quotes inside of strings, e.g.

```javascript
var specsTable = "<table class='specsTable'>"; // => <table class='specsTable'>
// or
var specsTable = '<table class="specsTable">'; // => <table class="specsTable">
```

See https://developer.mozilla.org/en-US/docs/Learn/JavaScript/First_steps/Strings#Single_quotes_versus_double_quotes for more


@giovannibussi yes this was the result of ./header.sh, but 2016,2017 is the correct one
I don’t know why it is different if applied from master, lets see what happen one we merge isdb into v2.4

C

> On 15 Sep 2017, at 17:22, Giovanni <notifications@github.com> wrote:
>
> @carlocamilloni <https://github.com/carlocamilloni> did you change this by hand or using the ./header.sh script? In principle it should detect dates from git log. I also noticed this was changed (by me, with the script) in cef88ba <https://github.com/plumed/plumed2/commit/cef88ba424a80d6c4343a1f3c186ea2b2b19e9c6> and I was wondering which is the correct year for this file..
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub <https://github.com/plumed/plumed2/commit/0d145b1c5991fb6e75a9d3e05fa30ab3db7e4985#commitcomment-24325419>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEXl9uVa_W1495JRs7Maw-eb_qzyx3wRks5sipZKgaJpZM4PZIr7>.
>


I think 0.01 is a stronger choice, here, since that is near where k=8 is optimal.


Can't this be null here as long as numBytes != 0?


Can't this be null here?


compile-time constants are written in all caps usually, like "MAX"


;P


You can leave off the Math.log calls.

Also, can you explain to me the pros and cons of choosing max BF size of min BF size or 0 when m < 0?


Not double, int. This is wrong here and elsewhere.


I don't think you mean "the column value to be compute". I think you mean the value to hash.


I meant that if `listing.id == 0`, then surely you also have `listing.qty == 0`. So no need to check `.id == 0`, only check for `.qty == 0`.


This can be very slow because of the synchronization if many Binary values are being hashed in a multithreaded setup. Is there another better way?


Watch out for singular "validate" on custom validations versus plural "validates" for Rails built-in validations!


I'm not sure you want the `document.write` in here; I don't believe that's necessary. `response.write(specTable);` s/b sufficient.


Actually this service is also used by gnome-software. We should split it in a separate commit.


might want to change the 4.13 requirement to 4.12 as that's when the first case of that breakage started
as it is now cobalt won't compile with 4.12 kernels without modifying that line


Why does this need to be async?


Typical JavaScript practice is to use the Array literal `[]` instead of the `new Array` constructor, and the same is true for Objects:

```javascript
var filters = []; // instead of new Array()
var obj = {}; // instead of new Object()
```


gonna try a little something


Why 300 if I may ask? I'd always thought power-wise it might be a good idea too keep this as low as possible, as long as no negative side-effect is noticed...
Thanks!


FYI @tmassman


I don't think this needs to be here; IIRC the `AppLauncher` object is only used for the `endlessm-app` stuff, and I don't believe anything calls this method to launch the old app store.


You should actually be able to get this via your initial search. Add a column with a join:

```javascript
columns[2] = new nlobjSearchColumn("itemid", "custrecord_download_item_parent");
```

Then when you want to retrieve it, you can do `searchResults[i].getValue("itemid", "custrecord_download_item_parent");`


![](https://github.trello.services/images/mini-trello-icon.png) [Update Sublime plugin to support cmd+a code search](https://trello.com/c/bx9R1JfX/19-update-sublime-plugin-to-support-cmda-code-search)


Common


## What is this for???

Please explain this line better


Noted, I'll add a simple int to string conversion function to replace to!string.


## Commit Comments

Looking at how this commit was created.


I prefer to not mess with specific indices like this; it becomes a real pain for larger sets of columns, or searches that change over time. Instead, I prefer to declare the Array all at once:

```javascript
var columns = [
    new nlobjSearchColumn("custrecord_download_name"),
    new nlobjSearchColumn("custrecord_download_file")
];
```


Awesome, thanks Rob!


![](https://github.trello.services/images/mini-trello-icon.png) [V1.0.0](https://trello.com/c/D87xHFZw/11-v100)


File for coursera


Yes, but each listing or order has a non-continuous list of ids. That's where the elegance is lost to me.
You save 1 word of storage space, but you save gas in the constructor only, not in other function calls. It is better to save gas than reduce the number of storage slots.


`vendorOf(uint orderId)`? Go on... give yourself an opportunity to hit a classic mistake ;)


![](https://github.trello.services/images/mini-trello-icon.png) [Increase hit circle radius to 120](https://trello.com/c/lyWcsx4k/9-increase-hit-circle-radius-to-120)


You *might* be able to get the URL through your very first search as well (I'm not sure). This would save you a ton of time and governance because you wouldn't need to load every single File associated to your download records. Try adding the following join column in your initial search to see if it works:

```javascript
new nlobjSearchColumn("url", "custrecord_download_file");
```
and retrieve the value url with
```javascript
var fileUrl = searchResults[i].getValue("url", "custrecord_download_file");
```


I've just followed the Google's guide lines, didn't exactly tested it though


@kjschiroo how did you generate this list?


Ok, it is more a matter of specs then.
On the other hand, if you make your code simpler, you reduce the scope for errors. If you indeed make the code simpler, then you let the off-chain world deal with the different accounts.


committed using `git commit -am "commit message"`


Asking me who sent a transaction in the past is not a challenge. `web3.eth.getTransaction(hash).from`, boom.


Added a basic lap-timer.


I think it's fine if `.say for Seq.new: Order.iterator` works in this new way.

I'm less sure if `.say for Order` should blindly call `.iterator` on a non-Iterable values


Alphabetical order


This breaks any external applications which use cudev.


This import is added here but nothing in this patch uses it


Nevermind. https://meta.wikimedia.org/wiki/Research:Wiki_Ed_student_editor_contributions_to_sciences_on_Wikipedia


merged undocumented opcodes


Please don't use C-style comments in JS files


Ok


This change was from 2006, so this behavior (deliberate or not) has been around for a long time.

Currently, `sparse[1.0, 2.0]` indexing works for LIL and DOK but not CSR or CSC.


Checking whether the recipient is the intended one is correct, although this does not fall under the _challenge_ category. The challenge would be "crack that private key", which is a general math challenge, or "steal that private key", which is...
In the exercise description, we say that Bob needs to get a password from Carol. That's his off-chain challenge. How can he prove on-chain that he has solved this challenge?


2017 now


yes, that's what I'll do


Sounds like a candidate for deprecation in the near future


I'd probably squash this with the previous commit


Oops! Yes, this belongs to the one removing the dash


I fixed the bug here


Gracias @pablogrod voy a reubicar estos documentos a una sección particular donde almacenar este tipo de archivos, asi organizamos mejor el contenido 👍


Thanks for the link, lots of good stuff there. Probably makes sense... :)


Pretty sure this is upstream already too.


Link for this: http://virtualenvwrapper.readthedocs.io/en/latest/install.html#basic-installation


Is this something we can bring upstream?


Nitpick: does not look like we really need the separate variable here


Created multidev environment [build-tools-756#ci-1](https://dashboard.pantheon.io/sites/51afe6c0-0222-47fd-8ba0-fdfcf2925e90#ci-1).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-1-build-tools-756.pantheonsite.io/)


![](https://github.trello.services/images/mini-trello-icon.png) [Update Atom plugin to support cmd+a code search](https://trello.com/c/K0QXrgtB/17-update-atom-plugin-to-support-cmda-code-search)


> In this case, even replacing the + with {1,3} wasn't enough to get it fast. Maybe we could have some internal performance monitoring, so we notice these cases early.

Another thing, since this rule does not match anywhere the `\n`, and with  `{1,3}` it would be limited to three line returns, how can you pin-point the problem to this rule. Those results show that the problem comes from else where.

I will reactivate PostReformPortugueseDashRule.java.

http://forum.languagetool.org/t/checking-portuguese-slow/1669/7

If the times spike again, I would say that it is worth checking the optimizations made to the base Dash rule, since that rule is being used at the moment in other languages.


Este es mi segundo Commit desde GITHUB


@sreyemnayr you'll need to call form.save() in form_valid in your FormView, otherwise it just redirects.

Something along the lines of this (I have not run this but have something similar in my project working):

```
def form_valid(self, form):
    try:
        form.save()
    except:
        if form.errors:
            # means we've converted the exception into errors on the
            # form so we just redisplay the form in this case
            return self.form_invalid(form)
        else:
            # some untranslatable error occurred, log it and
            # inform user you're looking into it
            pass
    else:
        # success
        pass
    # redirect to success url
    return super(self, CreateBankAccountView).form_valid(form)
```

If the submission is successful, you should have an Account instance available associated with the current user.


An additional note - we only have to catch errors thrown by form.save() in this case as we hit Stripe's API - so while we've validated things as best we can in our form, it may still fail due to a validation error of some kind that Stripe finds. We defer as much of the validation to Stripe's API, actually.

Be sure to experiment with entering wrongly formatted account numbers etc.


@scchan @whchung @AlexVlx Seems that this is fixed in these branches:
https://github.com/RadeonOpenCompute/hcc/tree/feature_remove_whitelist
https://github.com/RadeonOpenCompute/hcc-clang-upgrade/tree/feature_remove_whitelist
https://github.com/RadeonOpenCompute/clang-tools-extra/tree/amd-hcc
https://github.com/RadeonOpenCompute/llvm/tree/feature-amd-hcc-remove-erasenonkernel
https://github.com/RadeonOpenCompute/compiler-rt/tree/amd-hcc
https://github.com/RadeonOpenCompute/lld/tree/amd-hcc
https://github.com/RadeonOpenCompute/ROCm-Device-Libs/tree/master


got this error when try to compile new code with this change

Error:(572, 33) java: cannot assign a value to final variable forCounter
@draknyte1


@SmartFinn Thanks, noted. :)


What is the best practice for keeping these secrets private when sharing code base on GitHub?


please deploy to the cloud max 🥇


That is a bug unique to NetBeans (does not occur in Eclipse IDE or IntelliJ).
See Prior Issue #1779


What was the reason for this? Why move away from twist messages? Only the precision of the numbers seems to be different.


It is unlikely that we'll remember to remove this. Would be good to have a tracking item for it. Better yet, don't modify this html at all, and instead return a custom header on the response.


1)Создаем миграцию для добавки полей в таблице
 php yii migrate/create alter_user_table

а)вставляем нужные поля в миграцию

б)Инициализируем миграцию
php yii migrate

в)в моделе frontend/models/User.php добавляем доки.описание
* @property integer $status....

2)frontend/config/main.php создаем правило
  'profile/<nickname:\w+>' => 'user/profile/view',

3)меняем в контролере ProfileController  $id на $nickname

4)profile/view.php подставляю данные и экранирую


You probably want to avoid creating a new default handler primitive every call (unless port-read-handler guarantees that every call produces a distinct default object from previous calls).

The JIT will not like this, as primitives are promoted by identity when they are invoked.

What you probably want to do is use the `make_procedure` decorator in expose.py and wrap that around the definition of `default_read_handler`.


> how can you pin-point the problem to this rule?

Just create a text "foo bar" followed by a thousand `\n`, then run a Portuguese check on it. It will hang if the original `SPACE_BEFORE_PUNCTUATION` is active, and adding some System.out.println to `RegexPatternRule` will show which rule is hanging. I found out by using `jstack` on the live system and then even running a debugger in the live system.


@grabear go [here](https://github.com/datasnakes/Datasnakes-Scripts/commit/91ec12397cab6460ce57478c8b95308178f249d2) to genbank.py


Created multidev environment [continuity-pro#ci-52](https://dashboard.pantheon.io/sites/f4b4b00a-1cc1-4f25-a05d-4dce395bff80#ci-52).

[![Visit Site](https://raw.githubusercontent.com/pantheon-systems/ci-drops-8/0.1.0/data/img/visit-site-36.png)](https://ci-52-continuity-pro.pantheonsite.io/)


Ok


Could be one liner: `return (V && V->hasOneUse()) ? *V->user_begin() : nullptr;`


赚了


king


yes you are


and compare with [this](https://github.com/datasnakes/Datasnakes-Scripts/commit/91ec12397cab6460ce57478c8b95308178f249d2)


why you make an anticheat in 1.2 you can remove him


I think you're right. Also, this snippet was in the wrong place anyway (should be below, and reference `appName`), so I'll remove it


wciecie


@sunghan-chang  alc5658_workerthread, alc5658_senddone , alc5658_sendbuffer and alc5658_returnbuffer which were a part of workerthread have been removed.
Rest of the changes add the functionality the worker thread was doing.

@btheosam Could you clarify more on this?


niepotrzebna pusta linia


formatowanie w calej tej funkcji


Good work


niepotrzebne wciecie


wewnatrz tej funkcji wszystko powinno byc wciete


I assume it's because it can take a while and you don't want to block the caller?


as a professional coder, I am very impressed.


la función utiliza el color de la manera antigua, debe cambiarse para que verifica según los nuevos parámetros y se esta utilizando una función aun no agregada "confirmarEnemigo" en las lineas de if's


using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace memoria_dinamica
{
    class Program
    {
            void imprime_binario(int n)
        {
            if (n >= 2)
            {imprime_binario(n / 2);
             Console.Write("{0}", n % 2);}

            else
            {Console.Write("{0}", n);}
        }
        static void Main(string[] args)
        {
            Console.Write("Alimenta un Numero Entero: ");
            int numero = Int32.Parse(Console.ReadLine());

            //Console.Write("Alimenta un Segundo Entero: ");
            //int numeroA = Int32.Parse(Console.ReadLine());

            Program p = new Program();

            Console.Write("\nNumero Entero: {0}, en Codigo Binario: ", numero);
            p.imprime_binario(numero);

            Console.ReadLine();
        }
    }
}


"+" caused a merge manifest conflict.


This commit has been mentioned on **Discourse Meta**. There might be relevant details there:

https://meta.discourse.org/t/notifications-in-user-profile-are-broken/70105/8


Yes, that's right


Agreed.  If I manage to merge that change then I'll add a test that all maps meet this criteria.

Unfortunately the change currently breaks the existing Android client (it initially displays the map incredibly small for some reason), so I need to come up with a new Android layout that works with both before this can be deployed.

Consequently I think leaving this manipulation in here is a good idea! :+1:


666666
How's it going recently


The only thing I did differently was add the rooms, scenes and shades which
was I missed on the first attempt. Seems to work very well. Tested it with
a goodnight scene and it worked well. I added the blinds to my
"actiontiles" and cannot seem to add the blinds individually. They show up
as a group but as an on/off switch. Therefore cannot set them as a % like
within the smartthings app. Would be nice but probably would require them
to act as a "dimmer"

thanks for the amazing program.

On Wed, Sep 13, 2017 at 8:39 PM, clang13 <notifications@github.com> wrote:

> Glad to hear it is working for you! Do you have any idea what you did to
> get past that exception? If you run into any others, please paste the logs
> here so I can try to debug.
>
> Please let me know any other feedback as well. I haven't spent much time
> lately on this project, as it is working well enough for me. But I'd like
> to incorporate feedback and fixes from others as well.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/clang13/hunter-douglas-powerview/commit/376a4d20ad7ae2e6a3f9d8af9f5b1291d5a1f129#commitcomment-24290377>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AbqZR2ZTm4NmFGyqGxLjecdnSSEHqIlnks5siHWogaJpZM4PVep_>
> .
>


In fact the build failed because the script is called after activating the Conda environment (see https://travis-ci.org/peterjc/galaxy_blast/jobs/275952812#L3380 ), so the `six` from `/home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages` is not visible.
I bet that https://github.com/peterjc/galaxy_blast/commit/5f91d28364883174af4ea74f9231abeb39add885#commitcomment-24325873 would solve it.


Hi,

I follow your step to try the sample application but I get the following error
The remote server returned an error: (400) Bad Request
My server url is http://localhost
My redirect url is http://localhost:49676/token
The resulting error is on this page
http://localhost:49676/token?code=<a lengthy characters here>
Could you please advise? I am not a web programmer and new to asp.net.
Thanks a lot.
Leo


@taylorotwell is there any reason why `ConvertEmptyStringsToNull` doesn't include exceptions like `TrimStrings` does?


I didn't say it clearly, I meant that the verification function requires the passwords to be sent by an authorized contract.


Used Materalize
https://trello.com/b/cRVdCid0/wdi-12-project-1


```if (lock == tokens[recipient].lock && msg.sender == tokens[recipient].AuthorizedContract)
		sender.transfer(tokens[recipient].remittableAmount);```


Fixes issue #80.


`if (lock == tokens[recipient].lock && msg.sender == tokens[recipient].AuthorizedContract)
		sender.transfer(tokens[recipient].remittableAmount);`


makes sense?


I was just lazy 😛


This resolves Issue #11


The chain here is:
```
User U  -> cast to ConstantArray -> CA
CA -> getSingleUser -> CS
CS -> getSingleUser -> Temp
Temp -> cast to GlobalValue -> VTable
```
Note that `getSingleUserOrNull` is applied twice, this looks a bit suspicious since we are depending on the exact structure. Would move the second call to the same line and extend the comment to make this more apparent.
```
// ConstantArray -> Struct containing the array ->  VTable ??
auto *Xyz = getSingleUserOrNull(getSingleUserOrNull(CS));
```


agregue los mensajes en ingles


Nice job!


@benguhin you can also rename this file with two underscores in the start of the filename to turn it into a draft, should you ever need to re-open this position.


Only for testing purposes


@williamd4112 what's the new feature?


This works?


> On 15 Sep 2017, at 19:21, Brad Gilbert <notifications@github.com> wrote:
>
> I think it's fine if .say for Seq.new: Order.iterator works in this new way.
>
> I'm less sure if .say for Order should blindly call .iterator on a non-Iterable values

I’m afraid it would get very difficult to get one without the other.

Please note that this *only* occurs if it is the only thing being iterated upon:

$ 6 '.say for Order'
Less
Same
More

$ 6 '.say for Order,Order'
(Order)
(Order)

So I would think that if you want to iterate over multiple Enums, you can still do that.


Since `getSingleUserOrNull(Value *V)` takes a value, this line is not needed to call it. However, it ensures that we exit early if we can determine that this could never be a VTable.
I would remove this exit-early optimization (I don't think there is the possibility of false positives with the code below) or turn it into an if to make the exit-early explicit:
```
if (!isa<ConstantArray>(U)) return false;
```


sfsfsfsf


You just need

    compile ('org.influxdb:influxdb-java:2.2')

The snapshot has been updated in the public Maven repository.


Creo que seria mejor que vos mismo elimines y re-subas estos documentos en la sección "Archivos" asi el sistema deja registrado que fueron tus aportes :)


Sure, the `isa<>` sounds good. We know the structure of a vtable and depend heavily on it, so I don't think there's much reason to not make an isa<> check here. Checking all users of some random value could be expensive (although it should return pretty early when finding something that isn't definitely a vtable use).


is the change in tag format intentional (V -> v)?


Very important nit: could you merge these two lines? :-D


Whooopsiedaisie! Thanks for taking care of this 💟  I with we could just include a `.ruby-version` file into the README :)


If you follow the current procedure, it is not clear how to add devices


looks like mongodb-core versions are wrong in HISTORY.md for versions 2.2.31 and 2.2.30


When is this change getting deployed for GA? How do I pass an array of strings in the override parameters section?


I am afraid the script will rewrite again your change... You can try the following:
- merge current v2.4 (or master), since I just fixed the script in order to sort years
- add a fake commit to your file, just adding an empty line somewhere, with
````
  git commit --date="Jan 1 2016"
````
- try `./header.sh`. Your file should get the years that you like.

Clearly, this is cheating git log :-)


I'm really not a big fan of data driven testing for that reason. It's easy to grow a single test into a monstrosity that tries to test every corner case, but he's extremely hard to understand or even "debug". Tests shouldn't have corner cases themselves. Can you write a different test failure?


Actually, I think you need to add `re.UNICODE` as a parameter, otherwise the encoding is messed up in the output file (my bad)


That is a bug and should be fixed.


@varunturlapati If you added `ConnectionOpts` remove `TestConnectionOpts`. We should only have one.


resolve cant take target 8.8.8.8. Only google.com


Para esta primera inserción me ha hecho falta declarar el CONTENT_URI en el contract, que hasta ahora no lo había hecho. Supongo que se me pasó o lo dejé a drede para ver para qué servía.
El caso es que cuando iba a introducir los dos parámetros que hay que pasarle al Resolver para que los complete el Provider, tenía claro que uno era el ContentValues y el otro era una URI con la dirección de la tabla PERO... dónde estaba?? jajaj, no estaba definido, lo he tenido que hacer en este paso, se ve en el archivo modificado siguiente en este mismo comit.
Aquí dejo una captura de cómo se compone esa dirección de la tabla:
.
![image](https://user-images.githubusercontent.com/23742881/30497438-2ac6ec96-9a53-11e7-9640-076fade49b47.png)


wtf


not so sure about this


Hello, I try to import  **AngularFire**  but it doesn't work!!
``import { AngularFire } from 'angularfire2';`
Any idea to what is happening?
Thanks.


You can use a `struct` and merge these 2 mappings.


@edolstra, @nbp: Would it make sense / be possible to pretty print Nix attrsets differently in the manual? It seems a bit pointless to make fixes like I just did here :-)


@bagage possible regression here, can you please take a look? https://josm.openstreetmap.de/ticket/15244


Hello,

I do not think this is included with the newest version of Mark-A-Spot that installs with composer as written in the documentation. I went and made these changes manually, but others may still be having the issue with reports not saving because of the polygon verification.

Thanks


Normally an unbounded array is not ok, because you can get locked out of your contract when you hit the block gas limit. In your case, it is less serious because you fail to create a contract. Unless the existence of this Question was crucial for some other logic, it is not the end of the world.


Pretty sure that it's buggy. I've two different app nodes in the same AZs


We have another bug report here: https://josm.openstreetmap.de/ticket/15184


Per my comments, this is improved. We still want to consider the default settings for the hardcoding, but using an exponential backoff in the SDK library is better than a static solution.


Agreed, but this commit is just moving text around, so I'll fix it separately.


excellent work


Oops!


You could make a small `constant` function that takes a `string` and returns the hash. With it, the UI could check that the `string` matches the hash in here. That improves checks and balances.


highly questionable


I would say that this line should be in the `for` loop of the constructor, not here.


In fact, `userBalances(address user, bytes32 answer)` was already created for free because you made the mapping `public`.


Aria2这一段代码也太多了……过于复杂了


Is this feature (branch) completed? If so, please submit a pull request, so it can be merged into master.


congrats for the 100000th commit! :)


You see, here you have the opportunity for malfeasance from the answerer. How about using a block number at which the contract flips to `Waiting`? Then anyone making a call would flip to the proper state depending on the block number. That would be rules visible by all, not dependent on the goodwill of answerer.


This is going to be a problem with some Linux distros switching over to python 3...

2 ways to solve this are either run `python path/to/main.py` or changing this back on your local box.

Gotta love python2/3 fighting for the same spot.


Try Code Badges 1-4; 6


Sounds good


Not my code, as usual (did I ever write anything? :-)), but will fix it. Agree it's bad to mix these things


yeah, discussed with rob on slack. i'm probably going to change this and add another Question state where the question has been opened and the answer choices are still being added.

the model i have here is bad because the moving gas limit makes the behavior of this constructor unpredictable


interesting. why did you choose to move it to the table?


True, will remove it


Dear @LCraftPE,
I am sorry to inform you that your submitted release, "UltimateMute" (v1.0.0), for the project [UltimateMute](https://poggit.pmmp.io/ci/LCraftPE/UltimateMute/UltimateMute) on 2017-09-09 12:54:21 has been rejected.

- Cleanup scattered debugging messages

Please resolve the above-listed issues and submit the updated plugin again.


If you _have_ to do this, it is much better to use [git submodule](https://git-scm.com/docs/git-submodule) rather than copy the source. That will make keeping in sync with the upstream modules easier.

Should also update the platformio.ini to use these libraries to ensure consistency between the builds generated by PlatformIO and Arduino IDE.


i assume you mean in addition to having it here?


Associating with #3 with this commit.


Ok


ah, didn't know i that `public` gave access to nested mappings as well


@zachvanduyn I think it should just point to Releases page (it always displays the latest release), we will forget to update this link for sure.


This should be renamed:
"Communication - Kamstrup Multical 401 [TESTING]"


jaja que odna con eso? sobra


TA TUDO ERADO


What is the best way to fix it? Something like this?

```
CWaitableLock lock(csBestBlock);
ENTER_CRITICAL_SECTION(cs_main);
bool keepRunning = chainActive.Tip()->GetBlockHash() == hashWatchedChain && IsRPCRunning();
LEAVE_CRITICAL_SECTION(cs_main);
while (keepRunning) {
    …
}
```


The test was added, but I forgot to perform "git add" command for test folder. Actually, the equals method was added in this commit.


only concern with that is that the block times aren't super predictable. we wouldn't want people betting on the answers they already know is correct. the solution is to always pass in a conservative duration because it's fine if the question closes early


Make it a "commission" to the creator.


`PATCH`?


<details>

<summary>No TaskCluster jobs started for this pull request</summary>

```js

The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the
default branch) does not allow starting tasks for this pull request.
```

</details>


Can this be written to avoid code duplication, e.g. something like
```
Seq(ClioDocument.UpsertIdFieldName, ClioDocument.EntityIdFieldName).map(replaceAll("([A-Z])", "_$1").toLowerCase) ++ Seq(dateField()...)
```


You are welcome to judge all you want. If I was "kanging" why would I tag? Anyways. This was done by mistake and I have already reset back to a previous commit. Cheers


aff


sdf


You're right: https://git.gnome.org/browse/gnome-shell/commit/?id=2e1e00c3de747b9208fcd8fd6a1b713d99adc932

I guess I missed that one while rebasing, will pick it up and move it to the beginning, together with the other patches that I'm cherry picking from upstream


جرررررر دادی خودتو ها


remove extra space here


not sure how to track the remainder in the contract, though. only thing i can think of is have a `Closed` state where withdrawals can no longer be made and the question asker can withdraw all remaining balance (unwithdrawn winnings + any division remainders)


No me gusta que comenten las lineas comentadas.


What;s this?


see that you found it https://github.com/deployphp/deployer/pull/1072
I remember that we have a problem with coping nested directories....


yes, or drop a table


Первый вид функционального каркаса для методов хранящихся в объекте. Такой "ФК" дает возможность расширять возможности методов обладающих им, или даже переопределять существующий сценарий.

Метод получает два метода в качестве колбеков. Первый колбек "до" вызывается в условной конструкции и его возвращаемое значение проверяется на истину (нестрого). Если колбек "до" вернул значение приводящее к истине то основная логика метода выполняется. Затем выполняется метод "после".

**Плюсы:**

1. Два постоянных колбека "до" и "после", которые позволяют делать ветвления сценария.
2. Простота (с точки зрения понимаемости), быстродействие (две условных конструкции).

**Минусы:**

1. Наличие лишних методов в объекте, по 2 на каждый основной метод.
2. Каждый колбек можно использовать лишь единожды.
3. Риск возможности перезаписать существующий колбек новым.


+1 for only supporting Python 3 in Odoo 11 and onwards. It's already the time!


remove unused comments