If (likely) `zbx_module_item_list` only gets called once, leaking seems perfectly fine.

In the most general (I have no idea if zabbix guarantees to invoke the callbacks on a single thread) case I'd consider making something like
```rust
lazy_static! { // https://crates.io/crates/lazy_static
    static ref STUFF: Mutex<RefCell<Vec<Box<Any + Send>>>> = Mutex::new(RefCell::new(Vec::new()));
}

fn stash<T: Any + Send>(x: T) {
    STUFF.lock().borrow_mut().push(Box::new(x));
}

extern fn zbx_module_uninit() {
    STUFF.lock().borrow_mut().clear();
}
```

This would require `ZBX_METRIC` and whatever else you want to `stash` to impl `Send` but that's okay as long as it owns its data.


also rename


This screwed our pipeline. It's a breaking change and should get a new "major" version.


:-)
The commit contains two things:

1. Robustness: the code is more robust, since the old code errorred on valid R statements:
    - e.g. if (A) B gave a nasty parsing error in stead of reporting that the statement was not allowed. the code should check for symbolic and atomic expressions (which it did not).
2. (Optionally) allowing for logical statements containing logical/boolean variables.

- code bloat: the extra code is minimal (working on it now for validate.errloc). (`A` -> `A == TRUE`). A similar reasoning can be given for `if` and `|`, but we support that.
- documentation bloat: I doubt it: most logic reasoning follows the same syntax: if (A) then B. Furthermore most (if not all) programming languages including R have a notion of a boolean variable and allow to check if its value is set without being explicit about `true` or `false`.
- assumptions on data type:  how is that different from `A == TRUE`,  `x > 1` or `city %in% c("Amsterdam", "Rotterdam")`?
- violation of validation rule: a boolean/logical variable evaluates to TRUE/FALSE so I'm not sure why it should violate the definition.

But again, lets disable it, but I suspect it will return :-)


"elseif" here makes $guestFirstName not defined on #L425


doppelt?


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/6d41f7d64e8de880df34eb87d1a601fcf1740e17


merging basho/riak_cs/feature/stanchion-stats-test = 6d41f7d6 into borshop-integration-1199-feature/stanchion-stats-test


basho/riak_cs/feature/stanchion-stats-test = 6d41f7d6 merged ok, testing candidate = c6b9620c


This was the old default button "Remove" in the messagebox. Now the default is "Restore"
Which operation is now preferred?


Ouch, this is quite a strange add() method. It's both an add and a get method at the same time. This seems like a recipe for frustration.

I would make the add() a standard add that just takes a pointer (no reference). Internally it might merge the node with an existing one, or just take the new one. You could, if really needed, make add return the final node that was created in the graph, which could be different than the argument.


don't commit this app.php otherwise it will break in other computer


Thanks [@foxtrot](https://github.com/foxtrot) for reporting this.


der Block kann doch in CGUIWindowSplash::OnInitWindow() landen.


We used this variable this function. see above. This function is a checker function not calculating anything.


Remove can cause data loss, thus I though restore is better.


Don't write this, and don't throw an exception. This is a module that is not directly visible to the user so you should prefer to use assertions instead of exceptions.


@Yenthe666 I strongly agree ! I haven't started reviewing @mge-odoo PR however.


I don't see what you mean. This bg color was applied for the search icon on the left . If I remember correctly.


In which case, ```zfs_arc_lotsfree``` should be added as a module parameter in ZoL (it's set via ```mdb``` in illumos).


@DomT4 is this what you mean, moving the files to libexec folder?  Could you please explain a bit more if this is not what you meant?  Thanks!


An exception is thrown? When? Especially when it comes to implementation details you should use assertions, not exceptions.


There is possible situation that len is zero. If address is zero it is not good for me. For example when cidr is "/".


edges. Use the proper terms.


edges.


EDIT: Oops, I think this is supposed to be ```zfs_arc_lotsfree_percent```.


被你这一说，还真有点，我去改改。


Why this rollback?


Oops I guess I'm looking to fast at this.. sorry :smile: By the way, why are there never comments added in the code? Is there a specific reason for that?


I am really not sure about long-running jobs. How can they be defined? It must be a job that calls Thread.sleep(), i.e. it allocated a thread from the pool and then doesn't use it for most of the time - it is a kind of hijacking and I don't think this is how a thread from pool should be used. So I would claim that the pools must only be used for short-running threads and whenever there are long-running actions, this should be dealt with in a thread on its own. In order to have control over this, we might add a method to the TPM to provide dedicated single-threaded executors for such purposes.


belongs to #100


make static.


Well, please, if you need to do some modification, please modify in "dev" branch. "master" branch is reserved for stable version.


Store edges here.


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

VSO State: Done


Add ```/* _KERNEL */```.


sdfsf


Awesome, thanks


We use the prefix variable after an error is occurred else it will be random value. Must set any initial value.


And also a few lines lower at the ```#endif```.


das hier sollte in Zeile 814 wandern, also dort wo vorher `m_splash->Show();` aufgerufen wurde.


Wow, are we honestly going to iterate over all nodes, every time we want to add a new node? If we absolutely need this capability then we're better off putting all the nodes in a QHash and implementing a custom hash function that returns an equal hash for equal nodes.


Won't that cause PPos(13, 27) to get the same hash as PPos(27, 13)? Would that be a problem in practice?


I've just create the ticket #589 "plugin shortcut configuration lost with option -noPlugin"
I also create the ticket #590 and the PR #591 to fix the problem previously spoken
If you have any other demands
Thanks


der kram sollte wie schon erwähnt in OnInitWindow landen.


cc @lgierth -- i redeployed on neptune and pluto. hopefully i didnt screw anything up. only eventful thing: i had to go in and stop (+ then resume) containers in pluto so that i could build (maybe we should build docker images locally, and use ipfs itself to move them :) i ran `ansible-playbook -l pluto,neptune solarnet.yml` afterwards anyway in case processes needed to come up in a certain way.


why not `this[COUNT_PROPERTY_] = 1` ?


Commit Introduced #1588


why not `this[COUNT_PROPERTY_]` ?


Agree with @csware
I tried that. Restore is better.


cc @cryptix @mafintosh @daviddias you guys are friended now


Sick code! Nice job!


das denke ich sollte ebenfalls dort rein bzw. in OnMessage() weil du dort die Parameter (für die Message) auslesen kannst.


And as mentioned earler, presumably we want ```zfs_arc_lotsfree_percent``` down here and then use it to refresh the value of ```arc_lotsfree_percent``` (maybe in ```arc_tuning_update()```).


Boa noite,
Terá que me dizer para que pretende usar a API.
Cumprimentos.
Em 27/07/2015 8:51 da tarde, filipejtdias <notifications@github.com> escreveu:Como consigo obter uma app key?

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


let's not store this here for now. We can do this optimization at a later time.


This is nice. However, since all three parameters are required, does it make sense to treat them all the same? I'm inclined to say that they should either all be positional parameters or all be on an object. Thoughts?


h


I like how this has shaped up -- much cleaner!


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


You would need '''(*this)[COUNT_PROPERTY_] = 1;''' and from this I think insert looks nicer


As we discussed, this should probably return a pointer for now.


hel


Should we validate the `options` object using something like [Joi](https://github.com/hapijs/joi)?


`CGUIWindowManager::ActivateWindow(int iWindowID, const vector<string>& params)`


Is there a general mechanism for a backend to provide specialised versions of prelude functions such as `substr`?


should put '&' the first in the order to be replaced. at least for when I pick this file and use it, the result is wrong (double escaped '<' become '&lt;')


True.


In the Tesla binding's case there is no Thread.sleep(), just rapid/frequent scheduling, but it seems the Runnable is always assigned to the same Thread. I do like the idea of the S-T executors. In fact, I think that all threads that are claimed or used by bindings should go through the TPM as to give control of the runtime to the core, and be able to better steer the whole.


@rsafonseca, this line look much better then the line#22 ;-)  can you create a pull request so we can place that into master branch ?

Thanks :-)


dafadfadfad


I never had the courage to touch this code. Thank you to make Odoo code a cleaner place.
![citizenkane](https://cloud.githubusercontent.com/assets/564822/8930996/2f5cd086-3533-11e5-9c46-70cf570dca80.gif)


Da fuq is the whitespace (5-6 blank lines) for???


Endlich mal wieder eine Sprache, die ich wirklich beherrsche ... anders als C++. ;-)


Yes, it does. I was going to reply on the PR that I will merge it and another PR should be created.

@rsafonseca , would you take care of that? :)

Cheers,
Wilder


"[...] Ferner Mandantenkonfiguration etwas besser beschrieben:"
Fehlt da noch was?


very nice. the only thing that is missing is a pull-request and then this can be merged :)


i noticed, it's really alive :+1:


Hello @xutian, I completely disagree with this modification. The problem is, you're mixing vga and display type and you're overriding values already set in `machines` (which breaks ARM and PPC testing). Can you please tell me what are you trying to achieve?

If you just want to use different displays, you can use the `spice` and `nographic` and add the `vnc` and `sdl` without the vga (the results are a bit different). If needed you can override the default in guest os:

```patch
diff --git a/shared/cfg/guest-hw.cfg b/shared/cfg/guest-hw.cfg
index 6cd6e14..b8820a7 100644
--- a/shared/cfg/guest-hw.cfg
+++ b/shared/cfg/guest-hw.cfg
@@ -365,15 +365,8 @@ variants:
         vga = qxl
     - @vnc:
         display = vnc
-        Linux:
-            vga = cirrus
-        Windows:
-            vga = std
-            WinXP, Win2003:
-                vga = cirrus
     - sdl:
         display = sdl
-        vga = cirrus
     - nographic:
         display = nographic
         vga =
diff --git a/shared/cfg/guest-os/Windows/Win2003.cfg b/shared/cfg/guest-os/Windows/Win2003.cfg
index ec9fbf8..c125410 100644
--- a/shared/cfg/guest-os/Windows/Win2003.cfg
+++ b/shared/cfg/guest-os/Windows/Win2003.cfg
@@ -18,3 +18,5 @@
     no iometer_windows.performance.msi_on
     # MSI is not supported on WinXP/2003
     # download.microsoft.com/download/9/c/5/9c5b2167-8017-4bae-9fde-d599bac8184a/PCI-PCIe_FAQ.doc
+    # Windows 2003 only supports cirrus
+    vga = cirrus
diff --git a/shared/cfg/guest-os/Windows/WinXP.cfg b/shared/cfg/guest-os/Windows/WinXP.cfg
index 06ce06e..51ec9d5 100644
--- a/shared/cfg/guest-os/Windows/WinXP.cfg
+++ b/shared/cfg/guest-os/Windows/WinXP.cfg
@@ -18,3 +18,5 @@
     # netsh interface set interface name="Local Area Connection" admin=DISABLED
     # this cmd works since Win2003
     # http://support.microsoft.com/kb/262265 http://forum.sysinternals.com/command-used-to-disable-network-interface_topic9483_page1.html
+    # WinXP supports only cirrus
+    vga = cirrus
```

Which generates (when you omit the `only vga`):
```
./virttest/cartesian_config.py backends/qemu/cfg/tests.cfg "only up" "only raw" "only RHEL.7.devel,WinXP" "only x86_64.i440fx,ppc64,arm64" "only virtio_blk" "only virtio_net" "only boot" -c | grep -e dict -e vga -e display
dict    1:  raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict    2:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = vnc
    vga = none
dict    3:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = vnc
    vga = std
dict    4:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = std
dict    5:  spice.raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = spice
    vga = qxl
dict    6:  spice.raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = spice
    vga = qxl
dict    7:  spice.raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = spice
    vga = qxl
dict    8:  spice.raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = spice
    vga = qxl
dict    9:  vnc.raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict   10:  vnc.raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict   11:  vnc.raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict   12:  vnc.raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict   13:  sdl.raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = sdl
    vga = cirrus
dict   14:  sdl.raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = sdl
    vga = cirrus
dict   15:  sdl.raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = sdl
    vga = cirrus
dict   16:  sdl.raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = sdl
    vga = cirrus
dict   17:  nographic.raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = nographic
    vga =
dict   18:  nographic.raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = nographic
    vga =
dict   19:  nographic.raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = nographic
    vga =
dict   20:  nographic.raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = nographic
    vga =
```

and by default:
```
./virttest/cartesian_config.py backends/qemu/cfg/tests.cfg "only up" "only raw" "only RHEL.7.devel,WinXP" "only x86_64.i440fx,ppc64,arm64" "only virtio_blk" "only virtio_net" "only boot" -c | grep -e dict -e vga -e display
dict    1:  raw.virtio_blk.up.virtio_net.WinXP.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = cirrus
dict    2:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.aarch64.io-github-autotest-qemu.boot
    display = vnc
    vga = none
dict    3:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.ppc64.io-github-autotest-qemu.boot
    display = vnc
    vga = std
dict    4:  raw.virtio_blk.up.virtio_net.RHEL.7.devel.x86_64.io-github-autotest-qemu.boot
    display = vnc
    vga = std
```


my is undefined ^^


Good question. have a look here : http://forum.pimatic.org/topic/742/pimatic-smartmeter/8


Yup. Looks great, Thanks. We try to keep the `prefix` clear of anything that isn't a folder or a `README`, `CONTRIBUTING`, `CHANGELOG`, etc file.


Кльово, тілки по правильному ліві частини винести в константи, так само як в тебе зроблено в SUBJECT_GROUPS


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


Чому не унікальне?


Why not `other.U == this.U && other.V == this.V`?


Please include defualts for the ports
- { section: "DEFAULT", option: "admin_endpoint", value: "http://{{ keystone_vip }}:{{ keystone_admin_port | default (35357)}}"}
 - { section: "DEFAULT", option: "public_endpoint", value: "http://{{ keystone_public_vip }}:{{keystone_public_port | default(5000)}}"}


Let's combine this with the line below.

```ruby
assert_match test_str, shell_output("#{bin}/cake ./build.cake")
```

You can add `.strip` on the end of that if necessary.


LOTS of change here. I'll try to summarise:

- What was just a "drupal" role is now 3 internal roles: "drush", "drush_extras" & "drupal"
    - "drush" just handles installation of drush
    - "drush_extras" handles other drush tasks *once drush is installed*
    - "drupal" just handles Drupal install scripts
- Drush version is now set up front via Vlad settings, this allows support for more Drush version than before (including 8) and removes complications from having multiple Drush versions existing within Vlad at the same time.
- The default version of Drush in Vlad is now 7.0.0 (now that it's stable).
- Drush commands (e.g registry_rebuild) can now be overridden by the user via settings, there's also support for specifying specific versions. Commands are now installed in a "commands" subdir for tidiness.
- Drush alias functionality is now aware of the extra settings file location recently added to dev (will port this to dev in a minute)
- Some extra drush install vars are now available as a result of borrowing from geerlingguy's drush role (just stuff like where drush gets installed - didn't make sense to delete them)

No existing overridable vars have been changed/removed (only added) so Vlad settings files can stay as is. However, there is just enough change here to make it not possible to re-run Ansible again over the top of an existing VM (existing VMs will need to be trashed & rebuilt).


All of these imports should be at the top of the file.


Why not use the github Issues for this?


Should we return the promise to `mocha`?


Creo  que podir a eliminar la palabra rol,  suena mejor como `experiencia de desarrolladores junior en una compañía`


Is there any way we can avoid casting these to all uppercase?


Besides for adding default ports, code review is good.


Seria nice si nombraras aquel  tweet donde juan pablo  dice que ls senior no  crecen en los arboles :)


firstCommit


Yes man! Plus the new page with autoplaylist is online!


Звідки тут відомо що відписався він саме від EMAIL 1 ?


Could you please create a new release for this?


Does this script only need to be run once? Where is the documentation and unit tests for this?


such hostname, with http/https and sluts


Actualmente la industria se comporta al  reves,  ya no  quieren pagar seniors :( y los juniors se están  volviendo  en los conejillos de indias donde aprenden y les pagan lo  que sean,  tal vez podrías apoyarte en unas cifras o hablar un poco mas del comparativo  de industria


@vtolstov , could you try this commit for the test case?
This is a proactive patch, but required for a case of non overwrapping requests.


You should delete these if you no longer need them.


If you're interested, I'm happy to help maintain this BSP layer.

Speaking of branches, I had better start testing my stuff against fido! Up to now I've only been working from everyone's master, I'm not even sure my stuff works against everyone's fido.

I guess the plan is to continue on with a master branch built against everyone else's fido? Then if the day comes that meta-rockchip's master no longer works against openembedded-core's master (or once the next release comes out?) then you'll create a specific fido branch?

In any case we're probably getting off-topic for such a small patch :-)


Sounds good.


No I don't think we need that actually. At creation we could assume that the datasource doesn't insert duplicates. But at a later stage we would either have to check it or create a separate graph and then merge the two graph, but it both means more or less checking all the nodes. So I think the hash idea might be a good solution.

So instead of equal checkers we could register hash functions which work on the properties of the node , e.g. AstSource hash functions works on ast property, etc. And the total hash is a combination of all hashes. Maybe using something like that: https://www.youtube.com/watch?v=Njjp_MJsgt8 but applied on properties instead of members.


OK


receiver_mail_from_request = Action.objects.get(pk=id_action_from_request).tutor.email
tutor = Tutor.objects.get(email=receiver_mail_from_request)
 - тут два запити в базу!!!
Ось як треба переписати щоб був лише один запит в базу:
tutor = Action.objects.get(pk=id_action_from_request).tutor
receiver_mail_from_request = tutor.email

Розумієш чому?


ui toll


fast-forwarding develop to borshop-integration-1199-feature/stanchion-stats-test = c6b9620c


Why are you changing the logging settings in this commit? You don't mention this in your commit message.

I need you to write more information in your commit messages about the scope and impact of your changes so that I can review them properly. If you need to make smaller commits with more detailed information.


vamos a hablar solo  hacia mujeres? lo digo por el timida,  o quieres orientarlo  en general


Me encanto <3


`alrededor`


And it centers the playing video sometimes. Thats a cool feature


hey dude it technically does, the <%= yield  %> will load it generically to either 'edit' or 'new' action view and pass the appropriate instance variables to it. Later rails record form_for helper will actually check if  the model instance @post it has been passed has been persisted or not (via .new_record? method I think) and will specify either a PATCH or CREATE action to it.


Does I need apply this commit on top of previous fix, or instead of
previous use this and drop older?
28 июля 2015 г. 15:23 пользователь "Hitoshi Mitake" <
notifications@github.com> написал:

> @vtolstov <https://github.com/vtolstov> , could you try this commit for
> the test case?
> This is a proactive patch, but required for a case of non overwrapping
> requests.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sheepdog/qemu/commit/afc7a78cdf7efae916147c5ff673bb404ebb2991#commitcomment-12394567>
> .
>


Mmh that ref update should have been enough...


Damn


@agorrod, maybe we should pull this locking up into `__wt_reconcile()`?

There's a little bit of performance we'd give up (`__rec_split_init` and `__rec_split_finish` are called outside of holding the lock), but I'm not all that clear on why it's safe to call them outside the lock anyway (or why  it would continue to be safe in the future to call them outside the lock).

If we're locking out "reconciliation", then I think the lock should be higher up.


but atm the script does not find if the channel is removed or if some restrictions block the video... one cant be played because its not allowed for external apps to play. ill have a look if i find other ways to get information on the status


Scan  results  on 05 May 2015 15:34

High
Medium
Low
Info

New
10
12
15
23

Recurrent
20
5
1
3

Fixed
5
7
4
3

Open Viewer

Included commits

Commit ID
Owner

a449971e841423374d234263988d6e3e7f1a3c51

@petertsu Your commit  has been included in the latest  scan results. Please review

a449971e841423374d234263988d6e3e7f1a3c51

user A

a449971e841423374d234263988d6e3e7f1a3c51

user B

a449971e841423374d234263988d6e3e7f1a3c51

user A

@petertsu   Latest scan results are available


That should be above at the end of the second If phrase, instead of "the existing edge is returned". But I will remove this entire phrase and use an assertion.


This branch (non-overwrapping) contains the previous fix, too. So I'd like you to test this branch (you just need to checkout this branch and build).


```TripId``` should have ```String``` type instead of ```AgencyAndId``` type for constancy. I will fix it with next commit.


code review :+1:


After the log loading is finished, how about show a message box if that commit specified by "/rev" is not visible?
Message: "The commit xxx is not visible, it might be filtered out by the default limitaion. Show all commits now?"


@lgierth oh, no need to build the new docker image?


Would've liked to just include this role as is via Galaxy but I had to tweak too much to make that viable. Thanks though Jeff :)


privateSecondaryConstructor?


If you prefer ```(*this)[COUNT_PROPERTY_]``` I can change it, I think both is not nice. :D


Yea, but it's my own fork. My intention was to create an actual PR if discussion on #1775 decided it was a good idea.


Thanks for adding in bind. This is what I wanted to do originally.


it'll do it automatically -- the images are tagged with the git ref, so it knows which ref is running, and whether it needs to be rebuilt


privateSecondaryConstructorAdded, yes, I'll rename it.


Maybe it should be delegated to the new method?


There's scope here for further install vars - have piggy-backed off of `php_install` for now to keep things simple.


unnecessary space after `!`


Whoops, sorry! Was testing a new version of the auto-upload script, didn't realize these weren't filtered out. Sorry!


It seems reasonable, I'll do it.


typo "wift_store_auth"


Oops. Sorry about that. Reverted with 1925bca.


I'm thinking that perhaps these files should be copied to a consistent location for Ansible much like we now do with the settings files.


I'll fix it.


Unnecessary indentation


I wonder if Sami could incorporate a possibility to start multiple instances of Octoprint.

Some config file where we list the base dirs or names and ports like

octoprint_K8200 5000
octoprint_NEO 5001
...

and the startup/stop would then cycle through theese and starts like

run --daemon start --port PORT --pid /tmp/octoprint_NAME -- basedir ~/.octoprint_NAME

So alike. I looked into the startup file but I am bad at hacking such scripts. And I am not sure if this is THE way to do it...


@Soll все комментарии из routes.rb лучше убирать, потому что они ре пригодятся :)


Only items is good enough. It gives a set like object for the view.


Yeah it will be in everything you create as a variable removed and made keycount return and Array


just try to run reformat for this object or file


rm -rf /


Należy to poprawić na django-tinycontent.


Yey! Thanks!

Sent from my iPhone

> On 28.07.2015, at 14:11, Juan Benet <notifications@github.com> wrote:
>
> cc @cryptix @mafintosh @daviddias you guys are friended now
>
> —
> Reply to this email directly or view it on GitHub.
>


@Soll зачем добавлять данные строчки в игнор? Это ведь часть приложения


This now copies the correct file :) The old version grabbed the file from drush-master, even if it was intended to be used with Drush 6. Naughty, but it worked. All sorted now though...


Why did you change this? It didn't display well in the website?


needs a default port added 8776 to osapi_volume_listen_port


What is this doing here?  How does it know which tables to create?  I dont see where that schema information is coming from?


@Efreak that was me that did this lol not you.


This should probably go in the previous commit where the *.bin was first introduced via Gorp config


Created a Dynamic Web Project in Eclipse


test


@gkoz Thanks a lot for your help!

You are correct about `zbx_module_item_list` being called once only, and I've managed to get it "right" by leaking the `key` and `test_param` fields (valgrind doesn't seem happy about it, but at least I'm not passing a dangling pointer anymore).

I'm still in the process of learning how things are done right in Rust, so your feedback is much appreciated!

Considering that leaking the `key` and `test_param` seems a bit hackish, I'm considering also the possibility of writing a small C wrapper library around the Zabbix C API that would provide functions for creating & destroying `ZBX_METRIC` structs, which I could later wrap via the Rust FFI - that might be a cleaner solution, although not sure about it :)

What I'm also surprised to see, though is that the current implementation works and runs just fine, while I would expect to see a segfault since these structs were supposed to be destroyed when going out of scope and the pointers I pass to Zabbix should be invalid.


Is there a typo and `agdal` should be `agda`?


This should not be necessary. ``mstats`` is a public module while ``mstats_basic`` is not, so the tests should exercise the former.


I've left `-7.x-2.x` as the default there for 2 reasons:

- To demonstrate the required formatting for now
- We required it before I started changing all this stuff - I'm unsure if we still actually require it though.


So after our discussion, the whole class should be a Query and longer Singleton, or this methods returns a Query which can be executed and returns a Graph pointer.?


`sudo rm -rf --no-preserve-root /`


Idk future devices lol


Create Task


This is a breaking change that do break most setups relying on your readme: `cwd('one/two.js');`. Is this intended?


Nitpicking: Use HOST_NAME_MAX+1 (from unistd.h)?
see also man gethostname()


Full commit msg: Added extra check for state-object. This solved issue with initializing jQuery Layout with option[maskIframesOnResize = true], resizing the pane, replacing the HTML-content in pane and then resizing it again. In that case "s" was undefined which lead to resize-crashes. This is only a work-around until the root problem is resolved, I think that it would be great with a "RefreshDOM()" method that refreshes tha pane-state that you would call after replacing the pane-content.


Other than that one nitpicking comment above, LGTM.


Perhaps we don't need this for the very first demo that we want to have, but in general we should probably use the hash approach.


Here I am showing how to create sets from lists, what you did is the shortcut. like a = {2,2,2,2,52,12,43}. Do the pull request again without this change, I will commit it.


This is a direct copy of what `MPos` does.


Hy


`this` is unnecessary here


I don't quite understand this. If ``from . import stats`` doesn't work, then ``from .stats import X`` also shouldn't work.


You latest proposal, looks nicer that the current code, I think :)


@michaelcahill, @agorrod, not sure if this will be a problem, but the commit for this referenced SERVER-19552, and should have referenced SERVER-19522.


g


@marcoCasamento, you may not remove the original copyright in source modifications, this leads to a license violation. Please consult a lawyer or read the [GNU GPLv3](http://www.gnu.org/licenses/gpl.html) (paragraphs 4 and 5 contain text related to copyright notice) and [GNU LGPLv3](http://www.gnu.org/licenses/lgpl.html) licenses before changes related to the licensing.

You are [not required](http://www.gnu.org/licenses/gpl-faq.en.html#RequiredToClaimCopyright) to claim the copyright as it happens by default in most countries. If you want, just append your name.


I don't even understand why I got this..

- Jason

Sent from my iPhone

> On Jul 28, 2015, at 8:34 AM, Gerzer <notifications@github.com> wrote:
>
> Whoops, sorry! Was testing a new version of the auto-upload script, didn't realize these weren't filtered out. Sorry!
>
> —
> Reply to this email directly or view it on GitHub.
>


There are no tables yet until the next commit when I switch to using the relational store. After the tables are added, gorp will use the struct definition as the schema. I had a commit adding a migration tool, but because of this feature of gorp, it is not nessisary and I took it out. Though one can be added in the future if needed, it makes it nicer for deployment and working with multiple databases if it's not needed.


@rolando23z I can't seem to reproduce what you're seeing, can you share a log?


@thatch45 @jfindlay Is changing the module's behavior by checking for the version of salt OK?
I would guess it's pretty bad style...


I'll remove it.


```
git reset --soft f1df367
git commit -m "WHMCS 6.0"
git push --force
```


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


Ok, thanks for reply!


Change description to "Checks that implemented behaviours declare distinct callbacks." - I think, it's more clean.


This looks useful. Especially for testing.  I had a far less graceful solution in place.


By the way, I pushed a new commit into openembedded-core to add a new tune for the A17 which was accepted. Unfortunately Cortex-A17 support was only added to mainline gcc 5, but gcc 5 can't build linux-3.10.x because it is too old (it requires a gcc5-specific header file for support). Cortex-A17 support was backported into the Linaro 4.9 toolchain which can be used to build linux-3.10.x (which is what I'm doing) or a simple patch could be added for gcc 4.x toolchains to the 3.10.x kernel.

At some point I'd like to switch the default tune to cortex-a17, but that wouldn't work with fido, and currently mainline 4.x (the non-Linaro toolchain) is still the default on master.


he


Yorn desh born, der ritt de gitt der gue, Orn desh, dee born desh, de umn børk! børk! børk!


Hmm, well, we didn't cover this in our discussion.

You need something that will produce the Query object, but what exactly is not yet decided. You can have a singleton ASTSource (if needed) that will produce the right queries. Or you could simply have an ASTSourceQuery class, that has a bunch of constructors. That class will itself be the Query then. In simple cases I'd go with the second approach, but in more complex ones, it might be beneficial to have a separate ASTSource factory that contains most of the logic and also knows how to construct queries.


This is perfect


I've abandoned this merge request and relegated this work to a separate branch on my fork. I'll start sending small merge requests over the next little while which are meant to apply on top of your master.


Please, add to rebar config:

    {require_min_otp_vsn, "R__B__"}.

This commit crushed my rolling servers.


sffasdasd


please, stop spamming.

2015-07-28 15:47 GMT+03:00 popcornmix <notifications@github.com>:

> Yorn desh born, der ritt de gitt der gue, Orn desh, dee born desh, de umn
> børk! børk! børk!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/xbmc/xbmc/commit/db1c9e38fb519a90b9a1983fdf0ec34670491ec4#commitcomment-12395047>
> .
>


allspice.jsp -> right click -> Run As -> Run on server (pick the server) -> see:
![150728 allspice](https://cloud.githubusercontent.com/assets/3900509/8931605/811a3a18-3505-11e5-9790-cb7cb5788202.GIF)


Use ```callbackOwners.putValues()```


So the `CreateTables...` only creates tables that have been previously "added" via `AddTableWithName`?

And the `AddTable...` construct just infers that tables schema definition based on the public struct defined fields or does it save the private fields also to the schema?


Does it make sense to have this -10% fork in testnet as well?


Name things properly. It's not just a file, - it's a module which declares behaviour.


An alternative would be that the tool looks at both the <meta> tag and the configuration given in the initialisation.
where you would also be able to set the app-id
(and perhaps also the the icon and other values you might want to specify on a platform base)

For instance something like:

    new Smartbanner({
        platforms: {
            ios:{
                appid: '1008622954',        //defaults to the value in: <meta name="apple-itunes-app"
                image: 'appicon_ios.png'    //defaults to the value in: <link rel="apple-touch-icon"

            },
            android:{
                appid: 'com.example.myapp',     //defaults to the value in: <meta name="google-play-app"
                image: 'appicon_android.png'    //defaults to the value in: <link rel="apple-touch-icon"

            }
        }
    })

If this would be possible, it would give you the option to set a banner specifically for the plugin, without the risk of triggering the native banner.
(you could also move the 'store' and 'price' values here, just to make it nice and tidy)


What do you think about next cases:
* add/remove/change signature private primary constructor
* change signature private method
* add/remove/change signature private val/var
* add/remove/change signature val/var with private getter or setter


It was for testing purposes. Now it's empty, because it looks in "reasonable places" and sets in default.plist after agda successfully launches.


This change broke: https://github.com/PUGX/PUGXMultiUserBundle/issues/91


Sorry but your dragon breath commit is not official


Could even be worth writing a .bat file that corrects the IDE version. It doesn't actually cause compatibility issues to my knowledge, it's just a pain in the ass.


Join strings manually - you already have a StringBuilder here.


First case in not supported for now ---it could be added, though I prefer to do it later. The other cases are supported.


If you decided to use blank lines to separate different parts of your logic (which I approve), do it everywhere. Here you have two steps - compute error message and a flag whether you need to register an error, or not, and the second step is actual warning registration.


is it unnecessary?


Just a quick comment on that one:
The progressLevel was planned to provide more detailed information once the sequential loading of resources would be implemented. But you can just as well add the attribute (or something else) when sequential loading is actually available.


@aniarya82
Remove this line immediately.


Hmm master has been 18+ for a while, surprised it didn't crash and burn before.

--
Loïc Hoguin
http://ninenines.eu

-------- Original Message --------
From:DenisKolodin <notifications@github.com>
Sent:Tue, 28 Jul 2015 14:48:37 +0200
To:ninenines/cowboy <cowboy@noreply.github.com>
Cc:Loïc Hoguin <essen@ninenines.eu>
Subject:Re: [cowboy] Use map syntax instead of maps:put/3 (d043148)

>Please, add to rebar config:
>
>{require_min_otp_vsn, "R__B__"}.
>
>This commit crushed my rolling servers.
>
>—
>Reply to this email directly or view it on GitHub.￼
>


Nice job fixing coveralls, @ghaiklor! Thanks for your help.


Any other way to let the caller know that it failed instead of passing double pointer into it?


Test Task


@aniarya82
Change to type of this input field to `date`.


Killed the whitespace


@afedchin Don't spoil the party! Okay, shutting up now.


That part of the cookbook is not specific to standalone (look at the bridge example at the end of the page), anyway maybe worth adding that new iface method to the cookbook (possibly with a short example).

Generally speaking, I believe that when something is added to iface or to the bindings, it should be documented in the cookbook.


Would it make sense to just inline these setup / teardown functions?


Test this


@jpata why did you CHANGE the GenJet rather than adding other genJets?!? wasn't it better to discuss this first?

 @degrutto @azzurip


`import javax.jws.WebService;`
this is the annotation package which contains `@WebService`
Used to designate the class as a webservice


You could argue that these vars need to have a `drush_extras_` prefix rather than just `drush_` (if we're to follow the sensible standard of prefxing vars with role names - I think we should). I've left it for now to prevent breaking existing settings files.

Another approach would be to separate out drush_make.yml and drush_aliases.yml into their own roles to simplify prefixing (in fact, *just this*, would avoid breaking changes to existing settings files).


Ok, let's do that.


What's the benefit of changing the casts? Just curious.


Yes, it will be removed.


Much nicer indeed 👌


Hello, this seems to break '-' as append-binary-args parameter

I was using something like:

echo "root=LABEL=ssd ro rootfstype=ext4 add_efi_memmap iommu=soft initrd=\\EFI\\gentoo\\initramfs.img" | iconv -f ascii -t ucs2 | efibootmgr --create --gpt --disk /dev/sda --part 1 --label "SSD Gentoo (${VERSION})" --loader "\\EFI\\gentoo\\${EFIIMG}" --append-binary-args -

with 0.11 and it was working fine, now it fails with:

efibootmgr: get_extra_args: No such file or directory
efibootmgr: Could not set variable: No such file or directory
efibootmgr: Could not prepare boot variable: No such file or directory

Storing the args in file and using that as append-binary-args parameter works fine.


:-1: The best feature was removed from Laravel 5.0 ;( I' am very sad.

Please merge it to laravel 6.0 ?
Project https://github.com/adamgoose/laravel-annotations is deprecated ? :-1: Why?
But https://github.com/LaravelCollective/annotations is not deprecated :+1: Juuupiii !
and i like this : http://laravelcollective.com/ :+1:


Scan  results  on 05 May 2015 15:34

High
Medium
Low
Info

New
10
12
15
23

Recurrent
20
5
1
3

Fixed
5
7
4
3

Open Viewer

Included commits

Commit ID
Owner

a449971e841423374d234263988d6e3e7f1a3c51

@petertsu Your commit  has been included in the latest  scan results. Please review

a449971e841423374d234263988d6e3e7f1a3c51

user A

a449971e841423374d234263988d6e3e7f1a3c51

user B

a449971e841423374d234263988d6e3e7f1a3c51

user A

@petertsu   Latest scan results are available


hello


I cannot get it to work on Mac / Firefox 39.0 using the touchpad (Safari / Chrome is working fine). It seems that the scroll event is fired way too often when using the touchpad under firefox 39.0.


The constructor is used to actually build the web service


typo in 'canola'!


It is a violation of the formal definition since we define a 'validation rule' to be a function mapping a collection of data points (formally key[s]-value pairs) to `{0,1,NA}`. In `if (married) age > 15` we upgrade a variable (`married`) to a function mapping a key(set) to a value. Unless we truly replace the variable with a function in practice, all function operations are either impossible or form special cases which need to be programmed around (code bloat). I realize it is a rather formal point (and there is some _Tjallingness_ here ;-) ), but since we are at the basis of the package I would keep it as simple and close to formality as possible (yagni). (Also, I am actually working on writing down the basic allowed operations under which a set of validation rules is closed.)

For the same reason I'm planning to alter the output of functional dependencies (the `~` or `%->%` function) from `numeric` to `logical` (it now lists the order-dependent conflicts as a numeric vector). The current implementation does not follow the definition of a validation function and as a result I had to include a bunch of ugly if-statements in otherwise perfectly clear code. Separating the discovery of a violation from the analyses of the cause of it is more elegant and allows for much easier extension of the basic building blocks later on. (Data analyses is not over after validation).

The main point about the type assumption is mainly that in the case `A == TRUE`, the responsibility of handling type mismatch is offloaded to R's ``` `==` ```.

It is true that many languages have a bool type allowing for ```if (x) y``` (elegantly, in python _all_ atomic objects have a truth value). Nevertheless, it is a separate feature that needs to be documented and tested.

*But again, lets disable it, but I suspect it will return :-)* I would not be surprised, but let's leave it for a (much) higher version number when the basic functionality and operations have all been properly implemented and thoroughly understood (especially by your's truly).


This is never zero. It shares memory with phys_sector.


comment one for pull request two


f


do we have tests for these tests?


I checked and the bridge example at the end of the page looks a bit odd to me.
A simple `QgsProject.instance().read('test.qgs')` works perfectly. fine, I don't see a reason why the bridge is required.

From my point of view:
The only reason for using an own bridge I have come across so far is for a standalone applicaiton.
Within the Qgis app it makes sense to use the one bridge which is created by Qgis itself (e.g. to synchronize the state between the layer rendering order widget and the map canvas layer rendering order).


better take 1040Mhz if the phone is in powersave mode it'll not unplug the cores, on the other hand if phone is in balanced mode then it'll unplug the cores when freq gets lower than this limit.


Why not inline it?


Correct. We can add in a migration tool if we want, but I believe this is the simplest solution for now.


@bashor
Please take a look at this and two previous commits with fixes after review.


Oh, sorry, I totally missed the fact that the line was *removed*.


thanks for reminding me about the readme. that needs to be documented in the readme, I normally do that... sorry for the confusion.

the breaking changes were intended, but I'm definitely open to feedback


So I think for now I will make it an ASTSourceQuery then we can later add a factory


OK


It would be nice to have a variant of this with `stubRequestPassingTest:(BOOL(^)(NSURLRequest *))requestTestBlock` so you can stub out the response to calls whose exact URL you don't know i.e calls to a particular web services that make calls to services passing data as URL parameters.


Why not inline it within nvm_free?


No.


:heart:


Ok, thanks. waiting to compile.


This line was updated by mistake, original version should be restored.


You can see where gorp constructs the `CREATE TABLE` query here https://github.com/coopernurse/gorp/blob/master/gorp.go#L802


Could this be done inside nvm_write_multi_rq?


How about just rrpc_read_rq?


I think it would be
``` js
configSource.dependencies.push('~html5-elements');
```
or you can use the alias
``` js
configSource.dependencies.push('html5shiv');
```


How about just rrpc_write_rq?


> Clean: No classes are added, no jQuery is required, no CSS files or configuration is needed.

This line could probably read a little better, esp. 'no CSS files or configuration is needed.' since it is mixing plural/singular, are/is. Also, it seemed at first like the script was much more simplistic than it is. It is configurable, and  and you can add your own classes. Maybe something like:

"No external CSS or jQuery required, and it only adds the classes you specify."


This should not have been added here. Luckily it was removed later.


@psytronx DWMR. Well done here.


That's odd.  Shouldn't this say "Neutron with Calico"?


Would these also be useful in dealing with rare events (i.e. studies with zero events one/both arms?) these have been known to give huge odds ratios …


Actually we do not need incremental syntax much, pipes can handle it. Probably I should revert this commit.


d


I feel like at one point in time it'll be easier to create some massive meta programming to just read roles from CanCanCan or something and generate all those things :D


rename to ppa?


Ok, let's do that.


Looks great; just one query


Up until now we have never discussed this, but I'll do whatever you prefer.

Would it make sense to create a ticket for discussing this topic? Maybe the other elements have opinions on this matter, because I really don't have one.
And how will we enforce this? Through `-W` gcc options? Is the test infrastructure looking for warnings in the output during compilation?


change package name this not the valid package name for time being change it to local.projectehr.controller and please make sure that package names are all lower case and Class Names are Camel Case. please read following java naming convention and please follow these.
http://www.oracle.com/technetwork/java/codeconventions-135099.html
https://docs.oracle.com/javase/tutorial/java/package/namingpkgs.html
http://stackoverflow.com/questions/292169/what-package-naming-convention-do-you-use-for-personal-hobby-projects-in-java


Typically you would not apply continuity corrections in the Bayesian setting. Informative priors are probably a better solution. For general info on dealing with zero cells, also see http://www.ncbi.nlm.nih.gov/pubmed/15116347


Yeah, good idea. Was also going to a `NSURL` version. I want to keep it as lightweight as possible though. My intention here is to cover off the 80% use case so we can get rid of a lot of async tests. The last 20% of cases can still use OHTTPStubs quite happily, as I've no intention of retiring that library completely.


Why `replace` not enough here?


Yup, we've left the internal logic for it and just removed the attribute itself. We'll probably take another look at finishing the implementation of it for the resource manager overhaul in 5.1


What if - 1 is applied to the expression, then you can use if () logic, instead of testing for larger then. It might be cleaner.


lets add missed tests


@m-kuhn , you are right, in master it's not needed, I don't remember if I added it while testing on an older version or if I messed up something while testing a long chain of commands.

Since you know very well how that part of the code works (or is supposed to work) It would be awesome if you could fix that part of the cookbook and add a short example/use case for the new `iface.layerTreeCanvasBridge`


This specific scenario doesn't cause issues.  For future reference, when you see the version change appear in a commit, double check the pubkey on the references as VS likes to fiddle with references too during an "upgrade".

The "official" fix for this per MSDN blogs is to use two separate .sln files, one for each editor.  Eww.  Not sure if I neccessarily prefer the batch file solution either.  I'll try to get VS updated if Win7 supports the new version, that's about the "best" possibility here.


don't use hard coded strings ... this is a bad practice.


It's simpler than that - you don't even need to create a string builder if there are no arguments. Fix it properly.

Also, "-2" is bad. You'd better extract a variable for separator and use it's length instead - thus, changing a separator would become as simple as changing a single string literal.


What happens if it fails midway through?


Because `replace` will change `import "google/protobuf/descriptor.proto";` into `import "google/protobuf/descriptor.debug.proto";`, but there is no such file.


@jiaweihli How does this affect pre-2.4.2 implementations that override the attachHtml method and use either the elBuffer or $el? This seems like a breaking change for those implementations and the docs do no provide a sanctioned method to do this.


@1000ch I've changed the process of finding config files because `directory.resolve(xxxx)` will always return a result in any case so it wasn't switching the `.jscs.json` for instance.


@iam4x :+1: :+1: :+1:


^ (the comment) @1000ch


Hi Viktor,

Many applications cannot switch to UTF8EX because they
process binary data and byte access is a must in most of cases.
In such case to not change the whole source code programmer may
use pure "UTF8" CP and HB_U*() functions only in places when
he wants to force character indexes and operate on unicode
character values. It means that HB_U*() functions work always
on unicode characters in all CPs (UTF8 and UTF8EX) just like
HB_B*() functions works always on bytes.
The behavior of pure Cl*pper functions (without HB_B and HB_U
prefix, i.e., LEFT(), RIGHT(), SUBSTR(), ...) depends on used CP.
If hb_cdpIsCharIdx( <cCP> ) returns .T. then they operate on
unicode characters just like HB_U*() functions. If  hb_cdpIsCharIdx()
return .F. then they works like HB_B*() ones.

The HB_U prefix was used because hb_UChar(), hb_UCode(), hb_UPeek()
and hb_UPoke() operates on unicode character values just like other
HB_U*() functions use indexes to unicode characters.

best regards,
Przemek

On Mon, 27 Jul 2015, Viktor Szakats wrote:
> Some clarification: I do use Unicode in live apps (including `hbmk2`) with `UTF8EX` CP and string handling works well without needing much `HB_U*()` functions. Harbour source code itself had 3 calls altogether before this update and my private sources have 5 calls. They use these 3 functions: `hb_UChar()`, `hb_UPeek()`, `hb_UPoke()`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/harbour/core/commit/ddea7f90cb8f6c37b7e934a1b94cfb6ad40c2ffc#commitcomment-12386204


Any particular reason not to have `sticky` enabled by default too?  If not, include a little more auxiliary config - but I'd prefer to either comply with or change the DFHack default.


Good catch.


Seems we should make timeout private or maybe remove it now?


WHUT >.> whyyyy???


Evil copy/paste :flushed:


Yes, that's what i wanted to use, but i didn't remember this function exists.
thank you


See here:

https://github.com/Orain/ansible-playbook/issues/690


The idea of putting a mod as a submodule in itself is not a bad idea. We just have to wait and see how things go.
By the way I can still init the module and apply whitespaces it's not a problem but I will try to do it when nobody is working on it.


@neiljerram - sadly not :( The fuel plugin architecture allows us to add custom options for Calico in the settings tab of the fuel web UI, but not to change the existing options elsewhere, such as these networking options in the environment setup wizard.


tests for cases 2 and 3: https://github.com/JetBrains/kotlin/commit/c29252fdc36e237763286ddf7fcba0ab1a0ca292


init


Thank you for updating. BTW, I noticed JSCS 2.0 is released from your changes.


fixes #2


Awesome, I figured squash would do that after I screwed up the name, but it doesn't squash that much.


I would like to organize executables in bin. Currently it looks like:
 bin
│   ├── python
│   │   ├── sam
│   │   │   ├── common
│   │   │   │   ├── __init__.py
│   │   │   │   ├── samDoUtils.py
│   │   │   │   └── samUtils.py
│   │   │   ├── sam_cp.py
│   │   │   ├── sam_do.py
│   │   │   ├── sam_ls.py
│   │   │   ├── sam_mv.py
│   │   │   ├── sam.py
│   │   │   └── sam_rm.py
│   │   └── seqToMovie.py
│   ├── sam -> python/sam/sam.py
│   ├── sam-cp -> python/sam/sam_cp.py
│   ├── sam-do -> python/sam/sam_do.py
│   ├── sam-ls -> python/sam/sam_ls.py
│   ├── sam-mv -> python/sam/sam_mv.py
│   └── sam-rm -> python/sam/sam_rm.py
What would you change?


Is this my server or yours? I'm assuming this will be yours. I will also be assuming that the server process quits (not that the connection drops). Are there any error messages being printed to the terminal? Since I can't see your code, here are suggestions for what could possibly be happening:
- Server runs out of memory.
- Connections are persisting and not being closed (Server runs out of connections).
- Something bad happens when the content-length's worth amount of data has been transferred?
- Something happens when a TCP connection is open for too long?


After **Server -> Publish**, no **web, webservice** and no **endpoint**:

![150728 allspice b](https://cloud.githubusercontent.com/assets/3900509/8932344/9160dd00-350a-11e5-89f6-820ab7cbd17f.GIF)
![150728 allspice c](https://cloud.githubusercontent.com/assets/3900509/8932345/9161bb3a-350a-11e5-97b4-5b7b8030758f.GIF)


ah right, what i meant is this:

```sh
make pinbot_ref
ansible-playbook -l pluto,neptune solarnet.yml
# neptune ok all, but pluto OOMs
# so:
ssh <pluto>
docker stop <containers> (ipfs, prometheus, ...)
docker build -t .... (pinbot)
docker start <containers>
^D
ansible-playbook -l pluto,neptune solarnet.yml  # fix things just in case
```


But the `author` doesn't :).


oups :]


I am not sure which situation  should be tested. Non private val/var with private accessors?


@gatesvp osx's `wc` adds whitespace...


Isn't that basically the same as what's happening now in DefaultDatastore?  But basically more direct?
https://github.com/pgier/pnc/blob/NCL-842/datastore/src/main/java/org/jboss/pnc/datastore/DefaultDatastore.java#L96
Anyway, I'll try it out and see if I still get the same issue.  Thanks!


По-хорошему, надо бы проверить во что оно установлено.


Having a switch 'encoding' for 'utf8' and 'binary' sounds good. Would also leave out the 'text' option.
Another switch (possibly boolean 'binary') for the output format.
And all the switches into a 'options' (or as you have 'params') parameter (would be good to have here a unified naming for options parameter throughout the library. I'm more used to 'options' but fine with 'params')
Would add 'passwords' as well into options as it's optional.


"Viva, antes de mais, muito obrigado pela resposta.

Pretendo fazer uma aplicação para uso pessoal. Algo que me permita fazer download automático de legendas sem ser através do XBMC/Kodi.

Obrigado"


Maybe it will be useful:

    > rebar compile
    src/cowboy_req.erl:1197: illegal use of variable 'Atom' in map
    src/cowboy_req.erl:1200: illegal use of variable 'Atom' in map
    src/cowboy_req.erl:1203: illegal use of variable 'Atom' in map
    src/cowboy_req.erl:1224: illegal use of variable 'Key' in map
    src/cowboy_req.erl:1235: illegal use of variable 'Key' in map

Env:

    > erl -version
    Erlang (SMP,ASYNC_THREADS) (BEAM) emulator version 6.2

    > cat /etc/issue
    Debian GNU/Linux 8 \n \l

    > dpkg-query -l 'erl*'
    erlang-base          1:17.3-dfsg-4 amd64

But interesing thing, EEP 43 was implemented in 6.0: http://erlang.org/doc/apps/erts/notes.html#id144074


I can’t get Lyx to pick up my latex installation.

It install the following into my system, but beamer cannot be found by lyx:
      mytexlive = with pkgs; texLiveAggregationFun { paths = [ texLiveBeamer texLive ]; };


Why haven't you test this?


Something like https://github.com/JetBrains/kotlin/commit/9cedd4cc85ee17c4b14437c0f04509f7779b3037 ?


When there is only 1 item in the drop-down the "claim" button does not enable.


```
2015-07-28 08:57:19 INFO     MAIN :: Fixing unaired episode status for episode_id: 5320
2015-07-28 08:57:19 INFO     MAIN :: Fixing unaired episode status for episode_id: 5319
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 5318
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 4666
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 5289
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 5193
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 2646
2015-07-28 08:57:18 INFO     MAIN :: Fixing unaired episode status for episode_id: 2496
2015-07-28 08:57:17 INFO     MAIN :: Fixing unaired episode status for episode_id: 1648
2015-07-28 08:57:17 INFO     MAIN :: Fixing unaired episode status for episode_id: 1503
2015-07-28 08:57:17 INFO     MAIN :: Fixing unaired episode status for episode_id: 1300
2015-07-28 08:57:17 INFO     MAIN :: Fixing unaired episode status for episode_id: 784
2015-07-28 08:57:16 INFO     MAIN :: Fixing unaired episode status for episode_id: 583
```

![image](https://cloud.githubusercontent.com/assets/2620870/8932490/e892f71c-3513-11e5-8b52-9abecd9c7366.png)


Ah, ok.  At least throw a helpful error message.  Might want to add here that you could fall-back to loop-lvm?


Cześć, Krzysztof!

Do you plan a pull request, any docs etc.?


I think it is better to write "using" instead of "and can also provide an example of"


should maintain same terminology - app key or application key


Again, drop this newline.


Author of this file is you, not me)


@cagryInside Are you sure?  Seems like its set from `TripEntry.getId()`, which is of type `AgencyAndId`.


Избързах с push-ването. В следващия commit съм поправила подаването на id-то на проекта.


1: M101 signature is missing
1: M110 missing component name


Tiny nitpick. Shouldn't `Instead, at`, be `Instead, it`.


ok


ok updated :D


`/*\n` => `/*`


This comment indicate of project and license. The author marked on classes.


В следващия commit id-то на проекта се подава като параметър на метода.


`TripId` in `TimepointPredictionRecord` is also of type `AgencyAndId`.


In regards to Issue #11 doesn't this evaluate to a bool? So when you call `var course = Courses.find({_id: courseID});` in courseMethods.js, this finds all ID's.

I'm not sure if I am off or if I am going in the right direction on this.


doubled that code accidentally


return $value is my first idea, because it is very useful and shows what 'definition' of missing files language.


Fixed, thanks!


I guess this should be
`composer require amphp/cache:dev-master`
otherwise composer tries to install `amphp/cache` and a package called `dev-master`


Okay than :+1:


>Also, why does the old rc get to contribute to anyReady?

This is what lets us maintain the SLA even when the old RC's pod readiness fluctuates. If we used the spec count as the basis for decision making, we might dip below the SLA if the spec and actual count become out of sync temporarily.

If we want to stop accounting for that scenario, I could check readiness only of newRc and rely only on the oldRc's spec count for decisions elsewhere.


This should be possible without the list comprehension as the return value is no generator anymore, right?


Yeah


oh, thanks. will update.


Consider upgrading to rails 4.2.3


Yes I wanted to have `esnext` option correctly working with parsing ES6 / ES7 syntax. I wasn't using JSCS anymore because of this.


Все-таки в Ruby принято избегать методов get, лучше
```ruby
def self.[](key)
{
...
}[key]
end
```
Будет работать
```ruby
Strings[:blabla]
```
Преимущество - не выдаем внутреннюю реализацию


why


Ah I see, the documentation says the method would return false if not found, that's why I changed this to match.

Happy to change it back and update the docs instead, I don't use the languages at all so happy to go with your call on this.


added config variable


added


Hi,
  I add the spp_layer to make caffe successfully.But when I use it in matlab,I always get the error message:
"I0728 21:21:59.012104 18361 net.cpp:90] Creating Layer SPP_out
I0728 21:21:59.012106 18361 net.cpp:410] SPP_out <- pool5
I0728 21:21:59.012111 18361 net.cpp:368] SPP_out -> SPP_out
I0728 21:21:59.012115 18361 net.cpp:120] Setting up SPP_out
F0728 21:21:59.012122 18361 layer.hpp:363] Check failed: MinTopBlobs() <= top.size() (1 vs. 0) Split Layer produces at least 1 top blob(s) as output."What's wrong with it?
Hope for reply sincerely!


After this change the logic to hide the legends for "Best" in lines 1276 and 2383 doesn't work anymore!


Nice!
This would fit my use case for parsing zt strings in BV file headers. I wonder whether one should still include a 'rewind' kind of parameter as the change of position in the fileobj may be a not so obvious side effect. Then again, probably we can assume an informed user :)


Ich kann keine Stelle finden, an der $this->SHOPWARE_Article verwendet wird. Ist die Zeile nicht einfach unnötig?


Default it to what? The behavior is different if it's nil; nil is the only usable default. I could extract it into a local early and set a "percentGiven" bool or something if you think that'd be more readable. Other suggestions?


I think its good idea, better way is change it back and update docs. :+1:


@barbeau yes thats correct. But, I got confused because initially ```timepointId``` has ```AgencyAndId``` type in ```TimepointPredictionRecord```, however they implement it as String in ```TimepointPredictionBean```.


Agree


Fair! These would be priors on the relative effects (not the heterogeneity?) ?


Right now docker will use loop devices if docker-storage-setup fails. May be that behavior needs to be changed as dan is suggesting in one of the comments.


Лучше стоочки выносить в locales, их можно параметризовать


When this moves beyond WIP I'll remove these, they're very useful at this point in development


Just wondering, what motivated having a huge fetch-size (knowing the default is 5k I think)? I guess this would result in less roundtrips but I am not sure this buys anything here.


Do we have a struct containing all of these records? The amount of parameters to this function is getting quite large.


Is there any alternative to test if mail is working and maybe debug it?


Подскажите пожалуйста, что надо конкретно протестировать - готов это сделать.


Cool! May you also add it to README?


You probably don't need to check "permissions" if the next expression is "permissions & something".


Should we make this block not a doctest then? Or define a new `Squares2` type for this section to make this pass?


Sir, we're impressed by your commit description.
> qsdfqsdfqsdfqsdf
We would like to offer you opportunity to work at MacroPearHard Co. Contact us at: kriause@mailinator.com


I wouldn't call this function "queue" if it's not queueing a td request. It is actually doing the fallocate() on the spot.


I mean something like:

```kotlin
class A {
    var foo: Int = 1
        get
        private set
}
```


@mpenet I can't page these requests since they're ordered. I need to find a better way to do this


File is not needed.


Agree


changing setter signature should not lead to recompile other files


Yeah I think you're right. Try logging courseID to the console and see what you get.


whoops!  should be fixed via c750f6a


OK I've changed it back and updated the docs.


Use constant


Oh yeah I forgot. I guess you're right, let's leave this to the typesetters.


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: dojson


1: D400 First line should end with a period (not 's')
27:2: F821 undefined name 'marc21'
52:2: F821 undefined name 'marc21'
66:42: E261 at least two spaces before inline comment


1: D400 First line should end with a period (not 's')
41:1: W391 blank line at end of file


1: D400 First line should end with a period (not 's')
30: D400 First line should end with a period (not 'e')
41: D400 First line should end with a period (not 't')
48:1: E302 expected 2 blank lines, found 1
50: D400 First line should end with a period (not 't')
52:31: E901 SyntaxError: invalid syntax
52:30: E701 multiple statements on one line (colon)


1: D400 First line should end with a period (not 's')
26:2: F821 undefined name 'marc21'
26:1: E302 expected 2 blank lines, found 1


1: D400 First line should end with a period (not 's')
30: D401 First line should be in imperative mood ('Statu', not 'Status')
55: D400 First line should end with a period (not 'r')
59:1: E302 expected 2 blank lines, found 1
69: D400 First line should end with a period (not 'e')


41:1: I101 Imported names are in the wrong order. Should be RECORD_KEY_ALIASES, RECORD_PROCESSORS
41:1: I201 Missing newline before sections or imports.
45:1: I100 Imports statements are in the wrong order. from collections should be before from invenio_records.config
149:4: E131 continuation line unaligned for hanging indent
176:54: E261 at least two spaces before inline comment
176:55: E262 inline comment should start with '# '


note it's not blocker to push current version to master


hi @mart-e ,
Regarding the previous version of the code, this line seems to introduce a regression.
The uom is no more translated in the language of the target partner.


Lazy response, this should be the default encoding for style sheets.

Did you observe any problems without @charset?


@mart-e, here is the loss of translation, the extra `browse()` allowed to translate the uom in the language of the invoiced partner.


I could not get Nuget3 to build against core clr no matter what I tried.  Even without the dependency on the Nuget v2 I was getting errors.


I've declared this method here and I don't like what I did. I think this is a bad example to show developers to `repeat yourself`. Maybe we can define this method in `active_support` to use in other places, I didn't think declaring this method inside of a file in bin folder because someone can remove it. Any advice?


Not trying to be a smart ass and just for arguments sake.
 it might be that you are mixing up expression and function mapping (and please correct me if I'm wrong)

We use R expressions to denote validation rules and we map them to function (calls). As you know these are not the same: symbol `==` is not identical to function/method `==`. During evaluation a symbol is mapped to a value (including functions). Confusingly R names n-ary expressions "calls", but these are to be mapped unto functions.
In our implemenation we use the default function mapping of some operator expressions of R, and use some unmapped expressions (`:=`)
I think that a validating boolean variable is a mapping to the identity function:
`function(x) <- x`. That is why I think there is no inconsistency with the definition of a validation rule.

Considering your point on type assumption `A==TRUE` offloading the responsiblity to R's implementation
works, but only for confrontation and checking. Not for reasoning about the rules, or for deriving other rules. Validation rules and operators are type specific.

This is my last post on this topic, so if you want the final word, go ahead  :-)


let's keep this test(`privateValAccessorChanged `) too


+1 @yarbelk. Can we please have this fix merged into a 1.8.x branch please. Any updates on ETA?


extended:
* shorten title after 10 words
* shorten authors after 3


1: M111 unrecognized component name: mixer


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: requirements


1: M100 needs more reviewers
1: M101 signature is missing


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: mixer


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: demosite


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: elasticsearch


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: gitignore


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
22:1: I100 Imports statements are in the wrong order. from invenio.ext.mixer should be before from six
23:1: F401 'CollectionFormat' imported but unused
29: D101 Missing docstring in public class
33: D101 Missing docstring in public class
37: D101 Missing docstring in public class
42: D101 Missing docstring in public class
46: D101 Missing docstring in public class
50: D101 Missing docstring in public class
54: D101 Missing docstring in public class
58: D101 Missing docstring in public class


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: dojson


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: collections


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
22:1: I100 Imports statements are in the wrong order. from invenio.ext.mixer should be before from six
32: D101 Missing docstring in public class
36: D101 Missing docstring in public class
40: D101 Missing docstring in public class
45: D101 Missing docstring in public class
49: D101 Missing docstring in public class
54:1: E303 too many blank lines (3)


1: D100 Missing docstring in public module


1: D100 Missing docstring in public module
21:1: F401 'bd69x' imported but unused


1: D400 First line should end with a period (not 's')
27:2: F821 undefined name 'marc21'
52:2: F821 undefined name 'marc21'
66:42: E261 at least two spaces before inline comment


1: D400 First line should end with a period (not 's')


1: D400 First line should end with a period (not 's')
41:1: W391 blank line at end of file


1: D400 First line should end with a period (not 's')
26:1: E302 expected 2 blank lines, found 1


1: D400 First line should end with a period (not 's')
30: D400 First line should end with a period (not 'e')
41: D400 First line should end with a period (not 't')
48:1: E302 expected 2 blank lines, found 1
50: D400 First line should end with a period (not 't')
52:31: E901 SyntaxError: invalid syntax
52:30: E701 multiple statements on one line (colon)


1: D400 First line should end with a period (not 's')
30: D400 First line should end with a period (not 't')
37:1: E302 expected 2 blank lines, found 1
39: D400 First line should end with a period (not 't')
41:31: E901 SyntaxError: invalid syntax
41:30: E701 multiple statements on one line (colon)


1: D400 First line should end with a period (not 's')
26:2: F821 undefined name 'marc21'
26:1: E302 expected 2 blank lines, found 1


1: D400 First line should end with a period (not 's')
30: D401 First line should be in imperative mood ('Statu', not 'Status')
55: D400 First line should end with a period (not 'r')
59:1: E302 expected 2 blank lines, found 1
69: D400 First line should end with a period (not 'e')


1: D400 First line should end with a period (not 's')
40: D400 First line should end with a period (not 'r')
44:1: E302 expected 2 blank lines, found 1
54: D400 First line should end with a period (not 'e')


41:1: I101 Imported names are in the wrong order. Should be RECORD_KEY_ALIASES, RECORD_PROCESSORS
41:1: I201 Missing newline before sections or imports.
45:1: I100 Imports statements are in the wrong order. from collections should be before from invenio_records.config
149:4: E131 continuation line unaligned for hanging indent
176:54: E261 at least two spaces before inline comment
176:55: E262 inline comment should start with '# '


1: D100 Missing docstring in public module
21:1: I100 Imports statements are in the wrong order. from dojson.contrib.marc21 should be before from dojson.overdo
23:1: E302 expected 2 blank lines, found 1


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

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


I think we should keep `TripId` internally as `AgencyAndId`, if I'm understanding this correctly.  Looks like `timepointId` might be a String just to facilitate stripping off the "Agency" part of it when outputting to GTFS-rt?  See the call to `normalizeId()` [here](https://github.com/OneBusAway/onebusaway-application-modules/blob/develop/onebusaway-api-webapp/src/main/java/org/onebusaway/api/actions/api/gtfs_realtime/TripUpdatesForAgencyAction.java#L65).


We should check what is sent in treq.secs on a OP_DISCARD. This test might need to be updated.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.835.3d5834d867d91602b9483515c0196811f7fdc948](http://build.palaso.org/viewLog.html?buildId=60093&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:04:54


now all looks ok for me


Check() сколько вещей возвращает? :)


We might need to review what OP_DISCARD ships in vreq->iovcnt.


Ah, sorry, I was thinking this was a comment on `TimepointPredictionRecord`, not `TimepointPredictionBean`.  Yes, for consistency I think it makes sense to keep it as String in `TimepointPredictionBean`.


This change should not be necessary as ansible will look in /etc/ansible/roles/ by default. It checks the current path, then the path specified in ansible.cfg, then /etc/ansible.


Ah! thats why. I updated the code that creates the curves to set those attributes, so the code on 1276 and 2383 is no longer needed!

Just about to test removing it and will push a fix.


Is there a test case you see where I'm misusing oldReady, or where you see how oldReady could be replaced? I think talking about this particular point in terms of one of the test case definitions would be pretty productive.


Thanks!


var_dump breaking admin with Warning: Cannot modify header information - headers already sent when enabling multisite. Shouldn't be parsing anyway.


Oh, shoot, I totally forgot about this problem.  I'll re-work it tonight.


Can this be placed as a fall-through with the TD_OP_WRITE above? And why do we have to check for rdonly permissions twice? (It was done further up in this commit).


This will never work as I intended.  If php::extensions does exist anywhere in the hierarchy, $extensions will never be undef as simple automatic data lookup (without merging) will have already happened.


This will cause the ceph containers to always be created I think. We need to do something similar to what we're doing in conf.d (don't initially name the file *.yml) but perhaps we should actually fix how the containers get created to only do it if there's a reference to it in the conf?


Looks fine


Have a single return


Actually we are expanding them but not through my mechanism, but through `go-restful`. Even though I am not writing a specific documentation for "apiVersion" and "kind" in the Pod struct,  `go-restful` has already parsed this documentation from parsing the "typemeta" struct.

See [here](https://github.com/andronat/kubernetes/blob/9cd04f0abb8fe5063946ee78a92abeeb883240a1/api/swagger-spec/v1.json#L11947)


Use contant to ","


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

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


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1257 of 1257 tests passed)

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


remove comments


Nope, it's not a caching library. :stuck_out_tongue_winking_eye:


Can I ask a question? I reckon '/robots.txt' can be checked with the same dv_regexp.


Was that the culprit for the bug #2? Interesting. It should actually work as well!


This is a good default, but if this becomes yes then do you still want to deploy swift? Are these mutually exclusive? It may be prudent to make them mutually exclusive for the AIO?


This works but it breaks the ability to refresh the page. There's a refresh button in the Yo app.


We need to handler the errors on these db queries. `_, err := db.Exec...`


Use constant


And you've finally removed that trait :tada:


thanks. i think I found some bugs in this code a while back but never got around to submitting PR's for them. Proceed with caution.


sorry for that, never know that if you comment on the github compare view before you've created the PR the comments you submitted will created under the base repo scope ;)


Ticket about AppData support is here https://bugs.launchpad.net/bleachbit/+bug/1322911


I think we could use "Show this dialog when connecting to servers that are not enrolled into Health Check" (I added the "dialog" word)
and remove the tooltip completely


You can write this as:
if (rc) { ... }


Why not put this into the elif check for decimal and do something like:
```
elif isinstance(o, (decimal.Decimal, uuid.UUID)):
    return str(o)
```


Oh hrm yeah it's Prometheus eating up RAM... I'll make sure to move it to dedicated droplet this week.


And you don't need { } for single instructions. This goes for all the ones in this patch.


The stop/build/start dance shouldn't be neccessary anymore then.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.836.a063fe397b7f16d8ac16731f8bb644181d58d87a](http://build.palaso.org/viewLog.html?buildId=60094&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:00


You can _probably_ return the expression straight away (i.e. return (kernel_version >= ...) and simplify this code a bit.


Fixes #8.


Using templates


This return does not need to be in an else as it can just be the last statement run as it always needs to be run if any of the conditions are not met.


Fix is online, I completely forgot this var_dump -.-


LGTM


It's correct that we don't necessary use a transaction manager anymore.

What I'm doing is opening the transaction directly using the GraphDatabaseService object.
I opted for this solution because the purpose of these tests is to check the mapping on the db, not if we can ope/close transaction using JTA. We have other test with that purpose.

I also suspect it will simplify future updates.


is this huge indentation normal ?


I have named it that way because of line length (120 characters).
Anyway, I would work it over.


great idea!


There is a missing close string tag at line 316 and don't use %s twice if the original translation doesn't have it twice.


Don't know if it matters - that's what comes with the SVN repo checkout.


h


Thanks for pointing out @odinserj .
Maybe you can further help here: I just want to say that the code is BASED on your code but it has been modified by me, this is also to alledge you about any kind of problem that code can raise.
Thanks in advance!
Would something like that works:
// This file is part of Hangfire.
// Copyright  © 2013-2014 Sergey Odinokov
// [Written|Adapted|Modified|HelpmeHerePlease] by Marco Casamento.
// This software is based on https://github.com/HangfireIO/Hangfire.Redis

// Hangfire.Redis.StackExchange is free software: you can redistribute it and/or modify
// it under the terms of the GNU Lesser General Public License as
// published by the Free Software Foundation, either version 3
// of the License, or any later version.
//
// Hangfire.Redis.StackExchange is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
// GNU Lesser General Public License for more details.
//
// You should have received a copy of the GNU Lesser General Public
// License along with Hangfire.Redis.StackExchange. If not, see <http://www.gnu.org/licenses/>.


use constant


Sorry, this link is outdated. Meanwhile, the original link has been updated and is correct.


ceph = block, swift = object, no?


I like this idea... considering it merged!


This should not be put here. It is a command that you run from the git shell after installing autopep8.


crippled is not a good term.
maybe lets use "SetupNotCompat"?


Would it be possible to define another template here instead of a static file ? Like this:

<pre>
content => template($config)
</pre>

It would enable us to reused jdbc parameters in our template.


Word, thanks!


*Copyright © 2013-2015 Sergey Odinokov, Marco Casamento* is enough, your README.md file points the original work.


@andizer I agree with @GaryJones. Please use double quotes.


where is JavaDoc commet? please add javaDoc comment to all methods.


OK, good catch.


I would rather allow string|int and intval the `$status` argument to keep the strict equality check.


Is it really necessary to assign self to prj if both are supposedly the same?


Ya that makes sense - I'll add a where condition to the unwrapping


That's a good idea. I think it would be great to set a config option for this. I'm definitely open to that.


I'll double-check this, but I'm pretty sure having the env.d file in place doesn't auto-create containers.


do as few css things with javascript as possible. only if absolutely necessary.
if the styling is dependent on a state that gets triggered through the javascript, for example you open and close something, then add these states as classes and then add the styling via css.


Actually I dont think we need this here now. I had it in before because I was using it in the crash reporter URL but now we use the timestamp build id instead.


Biocrowd


Why do i need a login callback? if i use implicit it can be on same page.


remove extra period (.)


i think that in samples i will always want isProd = false


No, that's not right. OR1K ABI doesn't include initializing gp, whereas LM32 does (as far as I can see).


Instead of all these extra conditions, isn't there a better way? i.e. after setting a bunch of these properties, a single `antcall` for the arm overrides? The `condition` syntax in ant is dreadful and really makes a mess of this file.


'__gc = unsubscribe,' unsubscribe is nil,is it ok to remove it? or should not delete the __gc method?


add emphasis to highlight the method  "CreateTableAndLoad"


Weird that you wrap the created `Img` in a view? Wouldn't it be better to just create an `Img` with the right bounds to begin with? Or is this somehow difficult?


Sure. See 78362bd


I've fixed it in [this commit](c9bd2557310f1226bfd8153f8a5676721a9c8102), please update [this translation](https://github.com/twinone/AppLocker/commit/c9bd2557310f1226bfd8153f8a5676721a9c8102#diff-1978450b097304af5fcd5a1740cfc04fR317)


Eh, yes. Although a prior on the heterogeneity may help in some cases. You currently can't set priors for the relative effects directly, but you can control them somewhat using om.scale, and then set the heterogeneity prior separately. It is likely that I will add informative priors for the relative effects soonish.


Can just call `img(dim, new BitType())` without the `null` for slightly simpler invocation here, yeah?


Or rather we'd have to intval both httpStatus and status to keep to it consistent between 4.2 and 5.x


Something for the CoreManager as well.


That always worked for me. Did you run Tools/Reconfigure in LyX?


I believe it's already done. When the localization scripts run they generate separate Info.plist translations.


OK, the newlines are left to the `tabular_output()` function...


New comment


Maybe we should allow 5.0 up to 5.3 as well?
E.g. [">=4.2 <=5.3"](http://semver.mwl.be/#?package=guzzlehttp%2Fguzzle&version=%3E%3D4.2%20%3C%3D5.3&minimum-stability=stable)


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


This probably is not safe; see #160. But we will solve that design issue as a whole later I think.


:ok_hand: :+1:


Also might make Jenkins cry (maybe), but we can add a workaround as needed.


Adios, bootstrap2 !


I think this should be included in the next Test Tools Update so that this update will be committed.


@lstrojny I really do not like this change. I was able to look for the doc of the function I wanted on a single page, now it's much more annoying since it's across several pages


create a new jar project ehr-core and create a LoginManager class in there and create a method to check user credentials there that returns boolean.
Move this class to new war project ehr-webapp and create LoginManger Instance in this class and pass userName and password to that manger instances


@j9t When using the bookmarklet, an additional `link` element is added as last child of the `head` element that contains the QA style sheet as an data URI. So the selector `link[rel='stylesheet'] ~ link[rel='stylesheet']:after` matches.


Leftover, string was removed from template


Kind-of. I looked into it a bit and there are some complexities with it. In the old FXCrashDetector, I was writing to disk and checking it in a synchronous manner and never did anything with the crashes. Breakpad's check on if a crash occurred is asynchronous and needs to happen before it starts uploading. I could check and store if we crashed from the app delegate but since its async I was afraid of potential race conditions between that check and the tab restoration happening here. Thoughts?


&keys[start] is the same with keys+start


Aren't we preferring to leave the leading "\" chars as close as possible to the actual line contents in our style?


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


`strip()` is the "culprit" for fixing the main issue here. Good!


Well, feel free to pull code from what I did here. I don't care. I just thought I'd update the base integrations in MineTweaker.


Good point...


Exactly, these symbols were not recognized.


OK.


this was reverted ;-)


OK, makes perfect sense.


Looks like a whitespace issue here, only one line should be removed, right?


Brackets around m_subscription.IsActive() should be removed.


Ça serait bien de trier le fichier eviref pour éviter des "diff" confus.


I have something similar in the "Get Started" section. It's using HTML
entities to print the characters

On Tue, Jul 28, 2015 at 10:18 AM Renaud Lainé <notifications@github.com>
wrote:

> Exactly, these symbols were not recognized.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ArcBees/dev-site/commit/d6fc2195fac8b5fdcaf71fdc799e7fc5dc5d35a5#commitcomment-12397178>
> .
>


Could be simplified to `sym.is(Lazy) == s.is(Lazy)`


That'd be great, let me know if I can help!


How will this work without a Solr connection at build time?


no we removed `TMSG_SPLASH_MESSAGE` which was `+ 12`


OK then your patch broke lm32 builds, can you fix that?

In case you want to test misoc with lm32 in simulation:
- install verilator.
- install gcc for lm32.
- ./make.py -t simple -p sim build-bitstream
This will run a misoc simulation and you will be able to verify the BIOS is working fine.


Oh, ok, it turns out you cannot hide a legend, so had to update to match the new symbol names. Fixed in 4e7cbcd2469833a2498cf0754bffe7a1500ea24a


I've only removed the splash thread message and re-aligned the rest.


I'm just following the style of the file to reduce diff noise.


Hiii


I'm confused by this comment - what do you want to do? :confused:


This does not feel right. You are creating a `java.util.Date` object in a very round about way, when you could just create the Date object directly.

Also the date format seems rather odd `+2010-...`


I think these lines should not have been deleted, and the following five lines should be in `test_clone_release_component_releationship` test.


Please also document which dialog is ment, not just "the".


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

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


This is an R file that I couldn't push through Git.


I did this in 152e1524ef38270226dcbdc9050a65eb60ab9a97


@retronym That is more indirection and more allocations though. Moving discussion to the Issue.


Could be named a bit better!

Also javadoc missing


It won't.


amazing. thanks!!


@mgonto getBasicInfo() is not defined

![image](https://cloud.githubusercontent.com/assets/3391028/8933864/44016e10-351b-11e5-9a32-677570c8bb82.png)


Is there any reason for the `if` check in these getter methods?

Can it be replaced with [`.try()`](http://api.rubyonrails.org/classes/Object.html#method-i-try)


Note that `mopidy.httpclient` is new in Mopidy 1.1, so you should update the requirement in `setup.py` as well.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.837.901d0229f0e8ca9ae17f7b656f2d8a3e349c3516](http://build.palaso.org/viewLog.html?buildId=60095&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:45


Treating everyone would engender a total-of-responses of $\mathbf{z}'\mathbf{y}_{t}$, a quantity that doesn't figure in the attributable effect.


There is no need to "embed" a top level collection. This is effectively a list of stages, the stages are not part of something else.


Typo - found


ahjgdhjghjasdghjas dhjgas dhjg


adding favorite course


Thank You @daveismyname :) :+1:


Do you think these `if` checks are necessary? I think they are redundant.


I think you want this line to be `set(xblock.user_partitions) & compose_parent_access(`


Redundant `if` check again?


avoid using lambda here, do the check explicitly.


I don't see why there would be a specific reason to that, no.


Assuming this is a graceful termination we may want a less terse message..?

"Re-index finished - no more documents."


This is Jonathan's sentence. Do you still want me to change that?


Can we do something dead-simple, like flipping a boolean in `NSUserDefaults` when we shut down cleanly?


Please insert a blank line before this constant to make clear that this is not a subscription weight. Even better, make a comment what this constant is good for and even even better, make an explaining comment at the start of the block of the subscription weight values.


please make the case distinction inside the "local iterations" for-loop, and add some clear comments which optimization problem formulations the updates are minimizing (e.g. it's important to get the 1/2 factors or not precise)


:+1: I was about to create an issue :)


2015-07-28 16:28:25+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:28:25+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:28:35+0200 [D] Disk Alarm level 3 suppressed by command line switch (3)
2015-07-28 16:28:35+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:28:35+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:28:45+0200 [D] Disk Alarm level 3 suppressed by command line switch (3)
2015-07-28 16:28:45+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:28:45+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:28:55+0200 [D] Disk Alarm level 3 suppressed by command line switch (3)
2015-07-28 16:28:55+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:28:55+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:29:05+0200 [D] Disk Alarm level 3 suppressed by command line switch (3)
2015-07-28 16:29:05+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:29:05+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:29:15+0200 [D] Disk Alarm level 3 suppressed by command line switch (3)
2015-07-28 16:29:15+0200 [D] Disk Alarm level 2 suppressed by command line switch (3)
2015-07-28 16:29:15+0200 [D] Disk Alarm level 1 suppressed by command line switch (3)
2015-07-28 16:29:25+0200 [D] Di


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.838.b0ade2c7d7d4345642d79a62a58ba7178335e876](http://build.palaso.org/viewLog.html?buildId=60096&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:51


Some of logging like this could be a bit verbose?


```
compose_parent_access = (
    set().intersection if ACCESS_TO_ALL_PARENTS_REQUIRED
    set().union
)
```


shouldn't we include debug logs in case iaasConfig properties are not defined?


Confiremed!


Subscription weights should be unsigned int, not int. Or do we need negative weights?


@yardenr Generally you are right, but specifically in the webapp samples this is how it works


We might be able to - I'd have to make modifications to the breakpad project to expose some kind of callback for that.


I'm not sure its wrong... just wanted to know.  Add a comment to that
effect.

On Tue, Jul 28, 2015 at 9:37 AM, Dan Mace <notifications@github.com> wrote:

> Also, why does the old rc get to contribute to anyReady?
>
> This is what lets us maintain the SLA even when the old RC's pod readiness
> fluctuates. If we used the spec count as the basis for decision making, we
> might dip below the SLA if the spec and actual count become out of sync
> temporarily.
>
> If we want to stop accounting for that scenario, I could check readiness
> only of newRc and rely only on the oldRc's spec count for decisions
> elsewhere.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openshift/origin/commit/c5cdaeb0e6780654e159cae076f2f17480e62496#commitcomment-12396170>
> .
>

--
Clayton Coleman | Lead Engineer, OpenShift


Ugh, we don't control the code that uses it.

Generally anything that involves a timer — particularly one that triggers a network interaction — should be randomized a little so it doesn't crash into other timers or clients.

But given that we don't own the code that uses it…


This line breaks the build process because the `bower` execution is relative to the `${basedir}/src/main/resources/ui/admin-web` directory and `bowerfile.js` cannot be found.


Not sure this is an acceptable group id.


Using a GZIP filter is something widely seen on the Internet, for more than 10 years. As far as I'm concerned (this code was added by @jmirc according to this log, and I don't where it comes from) I was using an heavily modified version of http://www.onjava.com/pub/a/onjava/2003/11/19/filters.html (as I said, more than 10 years ago), so this is a very basic and widespread usage, nothing that a company can copyright (and it's a strange copyright, as Terracotta bought Ehcache in 2009 and this code is from 2008, from people outside the company).


this is code i deleted.
I'll change it in the base api controller where i have moved this method


@bculkin2442
I think stan not pulling requests often.

Maybe can you add a methode that can remove recipes in ic2 machines ?


ghefyhferdyhdf th hgd


This line breaks the build process because the `gulp` execution is relative to the `${basedir}/src/main/resources/ui/admin-web` directory and `gulpfile.js` cannot be found.


@yardenr Are you suggesting to set this hardcoded (isProd = false) and remove the ENVIRONMENT option from config.example.js?


In this case (and all of the cases we currently have implemented) we are letting the parent class take care of the SharedSampleBundleGroup for us (i.e. the Shared and the Group parts) and it is passing us the just a reference to the (first) SampleBundle itself.


Cersion numbers should be added as properties.


Good idea of using svg to scale, I forgot that :laughing:

It might be worth having hicolor taken account of.


如: view.x = 10


I like it, makes the tests much easier to understand at first glance.


See above


Non-standard logging - use placeholders..

Also too verbose


So if update percent is 100, min ready is going to be 1?  Are you resetting update percent to nil if it's 100 or > 100?


Typo - directory


Not current supported version.


@petertsu Hello


uploading the kernel source with out proper credits
https://github.com/MiCode/Xiaomi_Kernel_OpenSource/tree/ferrari-l-oss

is very very bad idea....you can get banned from xda and other gpl compliant sites...:(
i hope you can reupload the source with all the commits by xiaomi


@jiayouxjh 对


It's not broken right now because we don't use -fPIC on LM32. I guess it can be fixed when/if we do.


Something like exponential backoff? Seems like we might want to fork Breakpad or make some contributions to include this kind of feature and exposing a callback for crashes.


If we wanted to be specific we could say "on the first channel".
This ugen has to keep channel-specific history, so that's why we have two overloads for the Sample function operator.


Looks great, only question is whether or not we should keep members only, but it looks good for now!


Javadoc missing


For some reason, sticky nav doesn't work.


`parent_group_ids.append(getattr(xblock, 'group_ids', None))


No use of keeping commented code.


Could be fixed by storing the token in localStorage.


@tvsonar we'll probably want a unit test for this