Actually, I thought of another ambiguity
```ts
x as A<B>[]
```
Make sure this is a type assertion to an array type, and not nested comparison operators.


Here's what goes through my head when 4 separate character types are proposed:

* Woah, that's a lot of types
* That's going to require a lot of code (200-500 lines) just to give them basic functionality
* Plus at least that many tests
* So then the natural question is, "Really? Do we really need separate types here?"

That's where I'm pushing back, is trying to find some compelling reason for that much code complexity where my single most important goal with this play-package is finding something drastically *simpler* to what we have now. Now, that may *indeed* mean we should have 4 separate Char types for some performance/type stability reason, and I'm open to it; I just think the gains have *definitely* got to match the amount of complexity I perceive there. So far, I still haven't felt like there's been a compelling reason or concrete use-case or code snippets that really show, boom, having 4 character types = **this wonderful thing**.


src/create_ical_scenes.cpp does not exist. @drchrislewis


This seems like a huge change - allowing ```TX_MULTISIG``` as an allowed output type (where it previously was not).

IIRC there were major risk vectors in allowing multisig as an allowed output type (eg send a DEx payment as a multisig and then redeem to steal it back)


LGTM


OK. So this takes original idea and comments from here in an attempt to implement them: https://github.com/Flexget/Flexget/pull/471

With commit above and log here (http://pastebin.com/L0E0HJHL) you can see that the the debug print line loops through until files are found, and then continues on. This suggests the call back is functioning as expected.

Unfortunately from there it seems there are subsequent RPC calls failing in deluge. I'm having trouble tracing what subsequent calls to deluge would be failing and why.


Should we just stabilize these? cc @pnkfelix


The `reason` here is pretty funny :)

I'm fine with stabilizing this.


This change worries me slightly - is 0 really a special case that we can just ignore?   I prefer the IsCustom solution, but as was suggested by Shawn, making it private by calling it isCustom instead of IsCustom.


Pass the local `mNodes` into `Transactor::transact` and save a reference to it inside the transactor, so `mEngine->view()` becomes `view()` or some such.


I meant Set items. Right now it looks like any set entry with variables will be translated by the choice function, even if the variables are fixed by other lookups.


Thanks! Hotfixed it on master since I didnt see a PR


this enables for:
```
-|script|Foo Bar
{
    << SPELL LOGIC HERE >>
    end;

OnInit:
    registercmd "#foobar", strnpcinfo(0) + "::OnCall";
    end;
}


Does this not effectively make the moneyman address another Exodus marker?


For piping, `cat ~/.bash_history | cat nl` I couldn't get that to work in bash, `cat nl` is not recognized. But if I just run `cat ~/.bash_history | grep g++` It'll work like `history | grep g++`


Should we just stabilize these? It was planned for 1.1 IIRC.


same notes as for arc features


This is still not fixed. I don't see why checking membership in this set means anything useful in the places where it is used.


`cat nl` was a typo for just `nl`, which numbers the lines like the
`history` command does.  It's fine to not include the line numbers and go
straight to `grep`.

On Wed, Jun 10, 2015 at 5:18 PM, ZhuLeon <notifications@github.com> wrote:

> For piping, cat ~/.bash_history | cat nl I couldn't get that to work in
> bash, cat nl is not recognized. But if I just run cat ~/.bash_history |
> grep g++ It'll work like history | grep g++
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624040>
> .
>


This is kind of a bad idea. Pretty often modal box is larger then a screen, so one would never get the full content. Using JS approach looks much safer to me.


I've seen a few requests to just drop these.


I'm a helper!


hm actually I have a better idea. We could make the default label `OnInit` (only for global npcs of course) this way it would benefit all global npcs, not just spells:

```
-|script|#BarrelConfig
{
    end;

OnInit:
    setarray $@BarrelBits, (1 << 20), (1 << 21), (1 << 22), (1 << 23), (1 << 24), (1 << 25), (1 << 26), (1 << 27);
    << ... >>
    end;
}
```
VS.
```
-|script|#BarrelConfig
{
    setarray $@BarrelBits, (1 << 20), (1 << 21), (1 << 22), (1 << 23), (1 << 24), (1 << 25), (1 << 26), (1 << 27);
    << ... >>
    end;
}
```

===
For the spell in the comment above it would look like:
```
-|script|Foo Bar
{
    registercmd "#foobar", strnpcinfo(0) + "::OnCast";
    end;

OnCast:
    << SPELL LOGIC HERE >>
    end;
}
```


This is the same crash that was in build #1319 in which I replied to the group with the fix,
see https://groups.google.com/forum/#!topic/magarena/HPIUpak-hgQ
https://github.com/magarena/magarena/commit/f63ec901a70cda8c203f3392ed0634d83b2d23e1

The code was trying to get RefPlayer which was never passed into the event, what was passed into the event is enchanted, which is a permanent. Hence it caused a cast failure at magic.model.event.MagicEvent.getRefPlayer

I'm going to remove the redundant checks, groovy should not have to worry about valid, as it is too easy to forget to add these checks. As far as possible, groovy code should read like the oracle text, without lots of error checking or null handling as that would obscure the main flow.


`reconstruct` strikes me as a generic function rather than something specific to `LedgerEntrySet`. Maybe move this into beast name make the first parameter a template as well?


Anything blocking stabilization here?


Fix this up into the commit where you added it.


This commit message could be more specific.


I figured it out. Just needed a specific div.item around the rendered output:

``` js
return
    '<div class="item">' +
        '<div class="item-wrap-outer">' +
            '<div class="item-wrap-inner">' + escape(data[field_label]) + '</div>' +
        '</div>' +
    '</div>';
```

Seems to be working well. Thanks for your work on this.


Doesn't 0 point to the primary HDU as well?  Do you need to capture that?


Some of these can likely be stabilized as well.


Then I would say, do we really need `Bool`, `UInt8`, `Int8`, `UInt16`, `Int16`, etc.?
Do the gains from having all of those types match the complexity there?
I spent 29 years with a language with just a single type, string... (well, 3 types after orefs (object references) and IEEE doubles were added :grinning:, but that was later).   You don't *need* all the extra "complexity", but it sure can help make much better performing code!
I'll have to write some code snippets that clearly show the point, but in the meantime, just take a look at all the places in the current string handling code that does things like `< 0x80`, `> 0x7f`, `< 0x10000` or `> 0xffff`...
Why would it be 200-500 lines? Most all the code would be using AbstractChar, very little would really use one of the concrete types...  I don't think the testing would be that hard either...
Do you also get why these *validated* character types could be very useful with validated string types?


Produces NPE upon app start:

java.lang.RuntimeException: Unable to resume activity {net.osmand.plus/net.osmand.plus.activities.MapActivity}: java.lang.NullPointerException
	at android.app.ActivityThread.performResumeActivity(ActivityThread.java:3075)
	at android.app.ActivityThread.handleResumeActivity(ActivityThread.java:3104)
	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2475)
	at android.app.ActivityThread.access$900(ActivityThread.java:174)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1307)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:146)
	at android.app.ActivityThread.main(ActivityThread.java:5593)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(Method.java:515)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1283)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1099)
	at dalvik.system.NativeStart.main(Native Method)
Caused by: java.lang.NullPointerException
	at net.osmand.plus.osmo.DashOsMoFragment.onCreateView(DashOsMoFragment.java:66)
	at android.support.v4.app.Fragment.performCreateView(Fragment.java:1789)
	at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:955)
	at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1138)
	at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:740)
	at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1501)
	at android.support.v4.app.FragmentActivity.onPostResume(FragmentActivity.java:437)
	at android.support.v7.app.AppCompatActivity.onPostResume(AppCompatActivity.java:138)
	at android.app.Activity.performResume(Activity.java:5550)
	at android.app.ActivityThread.performResumeActivity(ActivityThread.java:3065)
	... 12 more


The point about alignment, is not for strings created by Julia, but rather, one of the neat things I *thought* you'd mentioned, being able to have these point into something from a file, without copying...


To reading such logs, normally only the last part is important. The key portion is
```
Caused by: java.lang.ClassCastException: magic.model.MagicPermanent cannot be cast to magic.model.MagicPlayer
        at magic.model.event.MagicEvent.getRefPlayer(MagicEvent.java:345)
        at Curse_Artifact$1.executeEvent(Curse_Artifact.groovy:20)
        at magic.model.event.MagicEvent.executeEvent(MagicEvent.java:659)
        at magic.model.MagicGame.executeEvent(MagicGame.java:775)
        at magic.model.stack.MagicItemOnStack.resolve(MagicItemOnStack.java:122)
        at magic.model.action.StackResolveAction.doAction(StackResolveAction.java:13)
        at magic.model.MagicGame.doAction(MagicGame.java:564)
```
So the failure occurred at getRefPlayer with the error "magic.model.MagicPermanent cannot be cast to magic.model.MagicPlayer". From this I infer, that getRefPlayer is called but the ref is actually a MagicPermanent so the casting of ref to MagicPlayer throws an exception.

Then going back to the groovy code, it is clear that not ref player is passed into the constructor. The ref for this event is enchanted, which is a permanent. This matches with the error, ref is permanent, groovy calls getRefPlayer, event tries to cast the enchanted (a permanent) to MagicPlayer and fails with a ClassCastException.

Hope this helps to explain the reasoning behind the fix.


O.K. I see.  You can disregard.


for my research. upload here for backing up.


site cedoc prograd


The fix commit is a bit large as I was also doing someother cleanup. The actual fix for the crash is to replace getRefPlayer with getPlayer as the upkeep player is the second argument after the event source, which means it is the player field. The ref field is the one after the choice.


Creio que seja melhor escrever como "largar".


囧，只改这一个还是用不了啊，其他很多字都是错的呢。
能恢复到这个版本吗？
https://raw.githubusercontent.com/acevery/ibus-table-wubi/3ea9821ed85c0a34c8a34257104f3f76836e0abc/tables/wubi86.txt
这个文件第一次提交还是对的，第二次提交就改错了，我刚才对比了一下。


"MoneyAddress" I found a little confusing, took me a mo to figure out where you were going with this - perhaps "ExodusCrowdsaleAddress" would make it easier to understand?


Hmm, I don't think we should be recommending that users place output files directly in their PyQLab folder for `AWGDir`. Could we do something different there?


BTW, I really didn't mean to hijack your wonderful `Strings.jl`!
I simply thought you could use the string conversion stuff I'd come up with, extended to use your neat Encodings...
I really see a lot of potential for what you are doing... it would be nice, IMO, to be able to get rid of ASCIIString, UTF8String, etc...
:grinning:


My PR #9378


What is the use-case of this? Can you provide an example?


Nice, can you commit the changes ? Also , have anyone using your ROM reporting any kind of screen suspend bug or device going off after call?


I ended up taking this pattern of `foo_extras` for small families of functions related to types, and here it's only applying to a few functions (all of which are likely to be stabilized or removed as a set). I do think, though, that I've been pretty inconsistent with naming, so I wouldn't say there's *not* a better name, I just couldn't think of them :(


Renaming to `iter_arith`


Sorry, bit busy at the moment @kanaka, I should be able to take another look at this next week or so.


1682_waterheater_stratified (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


Yeah I was hoping to deal with deprecations in a follow-up PR (at least the somewhat more controversial ones), but this is one of the first ones on my list!


Great explanation :+1:

The adjust for a better if that you quoted seems to be good. Another possibility can be a private method.


Yeah I think these are just unstable to be conservative, but after this lands I plan on making a follow-up PR to stabilize trivial APIs like this which probably should have been stabilized the first time around.


For `~/.bash_history` since this is a path, will I have to use getenv on it? or something else? Because I can't pass in a path.


I don't want to encourage the use of this idiom, its only a temporary work-around.


Hm, well maybe *you* are ok stabilizing this :)

More seriously, though, I wanted to hold off on any stabilization whatsover for a future PR (but coming soon after this) to make sure it's easy to see what's being stabilized and what the impact is.


So I am trying to get the `cat ~/.bash_history` to work in the pipe. Our program works with `cat ../file` but it's not working with any files in the home directory. I don't understand why it is not working.
I keep getting this error:
`cat: ~/.bash_history: No such file or directory`
I also tried it with the `~/.vimrc` file and that doesn't work either.


That's a big-ish change. Not sure it should go into this


r+


The import of NoSuchChildError was removed. It's not the right exception to use here.

In any case, why is this returning an exception object? That is almost never what you want.


Faltou o import da classe Curso.


The conditional inside the while loop could be refactored into a small private function. This function can then be reused on line 12.


No - the time constant is not important here, it is the attenuation off jitter


Keep this blank line; it's too easy to misread the `if` as an `elif` otherwise.


Oh hm 1.1 seems a little rushed to me (due to the "new" convention for static associated fns), but I'm totally fine stabilizing these (will come in a soon-to-follow PR).


:danger: `return treq` returns unidentified if the peer has not yet identified. Should either always return undefined (and possibly call forward with the treq), or always return a request.


@jkburges tidy yo' house


Doesn't require init - calculated further down and not used recursively


Random trivia: `!== null` doesn't match undefined, while `!=` matches `undefined` or `null`.  If you look at compiled coffee script, `!= null` is used for the existence operator `?`


@jkburges TODO on prod db


Error before patch:
```
christian@Christians-MacBook-Pro ~/P/v/tmp> vagrant up
[["/usr/local/bin", "/usr/bin"], "--version"]
/opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/util/subprocess.rb:29:in `block in initialize': undefined method `encode' for ["/usr/local/bin", "/usr/bin"]:Array (NoMethodError)
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/util/subprocess.rb:29:in `each'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/util/subprocess.rb:29:in `initialize'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.7.2/lib/vagrant/util/subprocess.rb:22:in `new'
```


Two things here:

- make i a local variable (right now it's global)
- rename the variable. single letter variables are confusing.


Hm yeah these I'd be mostly OK stabilizing, it does kinda depend on the subset though.


Adding to my list of banned abbrs for future review.


agreed


There are too many side bar items

I would reduce them to

* Fetch functions
* Fetch triggers
* Create a webhook
* Create a trigger
* so on and so forth


yes


Hah too funny, I basically just fixed this myself recently.  I think your fix here might fail the tests because it won't work if "file" is a StringIO, which we use for testing.


Oooooh! rainbow!!


None that I'm aware of, I think these were just unstable because they were new.

cc @Kimundi


Yeah, it hit me a couple of times, but usually helps finding uninitialized properties. I'd like to keep it as it is.


Addr is banned.


either is OK


Same comments as above.


Oh, bother. Precedent exists for exception when bound to scope (req, res).


80char limit would allow me to read this doc without scrolling. a tip for next time :)


I thought there might be an issue there — as I didn’t completely grok what might be returned from getFileHandle. You got to appreciate the cleaned up inheritance though?

> On Jun 10, 2015, at 5:43 PM, David Alexander <notifications@github.com> wrote:
>
> Hah too funny, I basically just fixed this myself recently. I think your fix here might fail the tests because it won't work if "file" is a StringIO, which we use for testing.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/extemporaneousb/pbcore/commit/143cef43ee09de3f8eebaf2647dd39ddf88a37bb#commitcomment-11624266>.
>


unnecessary, it is protected by the use of length and constraint on deltaTime


hi @brianlukoff - thanks for the contribution! We will test out your changes and pending everything being ok we will merge. Thanks


highlight them in some way


got *invalid* …


Cool @mercul3s , feel free to merge if you want.


Remove


Time to say "ouch". @mikewallace1979 @rnewson

This change introduced a weird/bad behaviour for non-interactive document update:
https://github.com/pouchdb/pouchdb/issues/3951#issuecomment-110952861

The problem is that proplist:get_value returns true when couch_util:get_value returns undefined. Because, couch_util version strictly expected [{K, V}] list while proplist implictly assumes V == true for [K] ones:
```
(node1@127.0.0.1)1> Options = [replicated_changes,full_commit,{user_ctx,{user_ctx,null,[<<"_admin">>],undefined}},{io_priority,{internal_repl,<<"_dbs">>}}].
(node1@127.0.0.1)2> proplists:get_value(replicated_changes, Options).
true
(node1@127.0.0.1)3> couch_util:get_value(replicated_changes, Options).
undefined
```

I believe that the others may also cause invalid behaviour.


remove this to keep identical behavior with original version of code


s/api/API


MessageFormDateと間違えて編集してしまいました。


:ship: has sailed, but Node.js URL.parse establishes precedent for the names "host" and "hostname", which usually helps people get past these bugs faster.

```
> require("url").parse("http://127.0.0.1:8080")
{ protocol: 'http:',
  slashes: true,
  auth: null,
  host: '127.0.0.1:8080',
  port: '8080',
  hostname: '127.0.0.1',
  hash: null,
  search: null,
  query: null,
  pathname: '/',
  path: '/',
  href: 'http://127.0.0.1:8080/' }
```


These tests look good, but I'd like to see a test that shows that the Exposure constructor gives the same MaskedImage as the MaskedImage constructor in all these cases.


yes


Add = 0.0f


You're completely right, however, I still think it's the right thing to do.

I checked the paths, and right now it's safe, because where it's important (coin selection, input selection and coin selection) it goes hand in hand with `ExtractDestination`.

We should probably be more specific, for example using `IsAllowedTypeForInput`, `IsAllowedTypeForReference` and `IsAllowedTypeForMarker` (with better names). What do you think?


consider rewriting

They allow you to build functions common to all platforms and can be changed without redeploying your app. Or something of this nature.


@brad999 It was working for me. I just had to make sure I changed the audio headers to include `{'Content-Type': 'audio/raw;encoding=unsigned-integer;bits=16;rate=8000;endian=little', 'Transfer-encoding' : 'chunked'}`.

In the end, I stopped using the live transcribe method though. I was expecting a speed boost with WitAI, but it didn't offer much of a bump and had a big impact on audio quality. Also, WitAI pushed an update a couple of days ago that significantly increased response time.


yeah I know, but we shouldn't be doing the same


Looking into it.


Triggers are commonly used to validate your objects before saving them. (dont say database, parse is database ?)


`~` is interpreted by bash and not by the operating system; that's why
`cat` isn't supporting it.

Again, you're free to choose a different example that would be easier.  You
just have to make sure it's an example that someone would actually care
about.

On Wed, Jun 10, 2015 at 5:40 PM, Lauren Boyd <notifications@github.com>
wrote:

> So I am trying to get the cat ~/.bash_history to work in the pipe. Our
> program works with cat ../file but it's not working with any files in the
> home directory. I don't understand why it is not working.
> I keep getting this error:
> cat: ~/.bash_history: No such file or directory
> I also tried it with the ~/.vimrc file and that doesn't work either.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624227>
> .
>


disregard what i said. im playing around with it and looks good!

i would just simplify...

@comment.post_id = Post.find(params[:post_id])


i would just skip this line.


Yes - will make static


I get where you're going with this, and I *want* to say yep let's go ahead and make Class C first vin.  I just want to make sure we've thought this through properly.

For example, let's take @CraigSellars thoughts on using SIGHASH_ANYONECANPAY to allow a third party to pay the fees - usually signatures apply to the whole transaction and thus inputs can't be reordered - but that's not the case for ANYONECANPAY because the default (SIGHASH_ALL) is no longer applied.

Perhaps I'm being overly nervous about this one - just want to make sure we get it right :)


GitHub mac という公式のものです．

`.gitignore`の中をみたところ，ちゃんと`/_build/*`と出ました．

terminalでコミットしたら，ちゃんと無視されるはずですね．


I find a class with a copy constructor and deleted `operator=` a strange beast. I'm OK with it as is, but here's an alternate design to consider: create a dummy class that wraps a `LedgerEntrySet const&` and define a function that returns the wrapper and a constructor and `operator=` that takes the wrapped type (and increments the sequence #). Calls would be: `lesEntries = vinniesWrapper(lesSource)` instead of `reconstruct(lesEntries, lesSource)`.


this should be json.


Review this


`peer.waitForIdentified` would make this more concise in the future.


Well below you're setting it to `null`, when semantically `its not passed in` on options, but it's quite possible it would be `undefined` (completely omitted) and later `getKeyRecoveryParam` will return `undefined`, instead of going ahead and searching ?


probably just have one output format.
function1, function2 are boring call them some thing else ? hello/foo/checkAddress/getName etc


Oh wait, it's practically always defined now, in the Signature class huh?


Hehe, sort of. That's the part I was wondering about too.

The relevant (old) parts:
```c++
// ### EXODUS MARKER IDENTIFICATION ###
    // ...
    if (ExtractDestination(wtx.vout[i].scriptPubKey, dest)) {
        strAddress = CBitcoinAddress(dest).ToString();
        if (exodus_address == strAddress) {
            ExodusValues[marker_count++] = wtx.vout[i].nValue;
        } else if (isNonMainNet() && (getmoney_testnet == strAddress)) {
            TestNetMoneyValues[getmoney_count++] = wtx.vout[i].nValue;
        }
    }
    // ...

// The MoneyAddress passes at this point, even without Exodus
if ((isNonMainNet() && getmoney_count)) {} else if ((!marker_count) && (!hasOmniBytes)) { return -1; }
```


:danger: This is a major breaking change. Please confirm.


> I checked the paths, and right now it's safe

So a m-of-n address cannot be the sender?  That's the crucial factor imo.

> We should probably be more specific, for example using IsAllowedTypeForInput, IsAllowedTypeForReference and IsAllowedTypeForMarker (with better names). What do you think?

This sounds like a much better approach than a generic "is type allowed for everything" kind of model.


I think it'd be better to write tests for data with a password and data without a password. But I'm not so great at the erlang yet, if this is desirable could someone give me a hand?


nit: the output :)


Looks like we still need to prevent the divide by zero.


Would it be okay if we have the user run `history > hist.txt` then they could search the file with `cat hist.txt | grep g++`? Would that still be a relevant example?


The host  == hostname:port precedent makes this confusing. `remoteHost` seems appropriate.


And Github still puts the bar at the top saying "18 commits ahead, 1 behind branch master." They're not connected, for dogs' sake!


This is attenuating jitter noise, so the ratio, not time constant is important


No need to pass it, it's created in the directive, well will be once I fix
it

On Wed, Jun 10, 2015, 6:05 PM Michael Stack <notifications@github.com>
wrote:

> You've removed title from the directive.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/projectjellyfish/api/commit/92849f0770a0868a71808d22d527c1ec005dcfb9#commitcomment-11622283>
> .
>


Faltou o Import da classe Usuario.


Let's use ```#ifdef __linux__``` to preserve the functionality and maybe prompt the user to use kextunload on osx?


Retested capture-previously-authed-amount with Ref Trans Id 2234918264.


The output ?


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


s/use you use/you use


should be json IMHO


i am ok skipping this


The output and results can have special sauce


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/733b5574010bbe6b968d8eeb8f42431a35848527


merging basho/riak_cs/bugfix/gh1162-backport = 733b5574 into borshop-integration-1163-bugfix/gh1162-backport


aaaa


basho/riak_cs/bugfix/gh1162-backport = 733b5574 merged ok, testing candidate = b7417f21


I would get rid of multiple responses everywhere, lets have one response format.


Glad you got it working @dcramble -- I'm running this plugin along with v0.12, so I knew it was possible


Those are the headers I have as well so I'm not sure the issue. However, if you didn't notice much of a speed increase then I won't even bother trying to get it working. Thanks for the quick response!


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


Names function1 function2 are boring to read, make them some imaginary thing a developer might have :)


I'm playing around a bit while I increase the coverage of the tests running on Travis. I will probably end up needing a custom config.json for testing as /my/path/to does not exist in Travis. Perhaps something under /tmp will work or I will need to build a custom tree for test. I'm writing (and plan to read from) in AWGDir from test_sequences. I'll come up with something better before making a pull request.


Confirmed offline. Breaking and necessary to fix.


Ok, I'll change it back and lower case the "i" to "isCustom".


Thanks. Same issue is present in these two sources (when building for 64-bit Windows):
crypto/evp/e_aes_cbc_hmac_sha1.c
crypto/evp/e_aes_cbc_hmac_sha256.c

Please see this PR about exact line numbers:
https://github.com/openssl/openssl/pull/299/files


Summarising our discussion today:
  - `POLLPRI` is distinguished from `POLLIN` and `POLLOUT`
      - Importantly enough that [it's a different `select()` set](lxr.free-electrons.com/source/fs/select.c#L386)
  - `POLLPRI` is different to `POLLRDBAND`... unless you're BSD
      - From freebsd and osx header: "Note that FreeBSD doesn't make much distinction between POLLPRI and POLLRDBAND since none of the file types have distinct priority bands - and only some have an urgent "mode"."

So, we're agreed that:
> [10:28:52] William Ahern: yeah. ok, so we'll just plow ahead with POLLPRI then. and p will translate to POLLPRI for Linux and Solaris.

Which just leaves the BSDs in question.

My reasoning there is:
  - `EV_OOBAND` maps cleanly to `POLLRDBAND`
  - `POLLRDBAND` is equivalent to `POLLPRI`
  - Therefore `EV_OOBAND` can be used with `EVFILT_READ` when we want `POLLPRI`


Oh wait, it is an orphan branch! Why does Github still compare it to master then?


Best.
Cheat.
Ever.


Postpone and rework for inclusion in U0.2


Fixed.


is Hongyun or Hongjun ??? :)

Date: Wed, 10 Jun 2015 09:29:48 -0700
From: notifications@github.com
To: addressList@noreply.github.com
Subject: Re: [addressList] houjun (1595ba2)

源码是我随便找的

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


Basically, everything that's not required by chatBot.js. They'll still be installed by default, however this way you can use [`npm install --no-optional`](https://docs.npmjs.com/cli/install) and install whichever few packages you need for your specific case manually.


Can you please add some logic to deal with the case that someone doesn't have a username / profile? In some Meteor apps the user may not have a profile object and/or name. Thanks


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


I see, so it's not really a change then - ideally there would be some separation between identifying an Omni transaction and identifying an Exodus purchase, but that might be too much of a departure at this point...


Review this deletion


These dependencies are fine for testing but should not be committed.


```
>> staff|/join battle-pu-239645114
<< >battlepu239645114
|deinit

```

Yet another  `toId` + `room` bug :| And it has slipped for so long...


So all these multiple unintended changes (`!` => `_!_`) are probably subject to be reverted?


`cat hist.txt | grep g++` is not a great example because `cat` isn't
needed; you could instead run `grep g++ hist.txt`.

On Wed, Jun 10, 2015 at 5:56 PM, Lauren Boyd <notifications@github.com>
wrote:

> Would it be okay if we have the user run history > hist.txt then they
> could search the file with cat hist.txt | grep g++? Would that still be a
> relevant example?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624401>
> .
>


I'm sorry, I really can't figure out how to initialize `char* []` with strings. I've tried googling but none of the solutions really work for me.


Is this import needed ?


Missing terminating newline


How could the endDt be specified in anything other than UTC if the startDt is UTC as set above?  Is this purely defensive programming, in case libical has a behaviour change in some library upgrade, or is it something we expect might occur "normally"?  If the latter, would there be any potential downside to simply setting the endDt to the appropriate UTC-timespec datetime?


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


Great. Will merge and test.

Curious about the authz not applying to the item show page.
https://github.com/vecnet/dl-discovery/commit/13c785ce16a4d654110a85b36c0634053c51b601#diff-e1a79d3515caaa892d3a82ad4a996223R14


allocate memory with `malloc` or `new`, then copy the string into the
buffer using `strncpy`

On Wed, Jun 10, 2015 at 6:11 PM, ZhuLeon <notifications@github.com> wrote:

> I'm sorry, I really can't figure out how to initialize char* [] with
> strings. I've tried googling but none of the solutions really work for me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624534>
> .
>


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

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


이런걸 만든거임?


Signature flags work fine, and each signature can have it's own flag (though the flags can have an impact on the whole transaction). Adding inputs to pay the fee would require that the first party signs with `ALL|ANYONECANPAY` (outputs locked, more inputs allowed).

The fee example is **excellent** by the way, and now I'm able to show an example, which less abstract, because it's the exactly the problem I'm trying to solve here! :)

Consider the following (slightly fictitious) example:

---

Bob wants to pay Alice' transaction fees, and Alice prepares the following transaction:
```
Vin 0:  0.00000546 BTC  [added by Alice]

Vout 0: 0.00000000 BTC OP_RETURN Simple Send  [added by Alice]
Vout 1: 0.00000546 BTC Reference Output       [added by Alice]

Fee: missing
```

Bob adds another input for the fees:
```
Vin 0:  0.00000546 BTC  [added by Alice]
Vin 1:  0.00010000 BTC  [added by Bob]

Vout 0: 0.00000000 BTC OP_RETURN Simple Send  [added by Alice]
Vout 1: 0.00000546 BTC Reference Output       [added by Alice]

Fee: 0.0001 BTC
```

---

What happens, if the sender..

- 1. is selected by contribution
- 2. is selected based on the first input

?


Our policy seems to be to avoid System.out.println() and friends and use a logger instead.


@scttl @hunterlang  ok so for the first draft what I'm going to do is POST JSON over HTTP then.
I'll write a demo HTTP server to make sure that everything is working, and then we can merge the demo code into the existing flask server.


yes I do.  Did you rebase your commit?  I will accept the change if it can apply to master


The behavior IS identical. The echo is captured as the method output on line 46. It's returning a string that gets read into an array.


See above comment.


Yes?


rax retry tests


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

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


@andyli has written a [blog post](http://blog.onthewings.net/2012/10/14/haxe-tips-everything-is-an-expression/) about that. I think the contents should somehow get on to the Haxe web site.


> POLLPRI is distinguished from POLLIN and POLLOUT
> Importantly enough that it's a different select() set

I just looked at the [NetBSD source](http://ftp.netbsd.org/pub/pkgsrc/current/pkgsrc/pkgtools/libnbcompat/files/poll.c)... and `POLLPRI` is part of the 'readable' set there.


Ffastmath isn't good for kernels. Had a long debate with ZaneZam about this. We found an article proving that fact too..:) while spying on you git I thought to share this with you.

Also -ffast-math -fsingle-precision-constant

They are the opposite to one another. Ffastmath removes precision, the other one enforces it. I believe here your ffastmath isn't even working because of that.. :p


What about the example `ps aux | "username"` so they can see what programs they are running? And they could run it on their partner's username or something like that?


> Adding inputs to pay the fee would require that the first party signs with ALL|ANYONECANPAY (outputs locked, more inputs allowed).

Perhaps I've got a misunderstanding there - I thought ANYONECANPAY also allowed *reordering* of the inputs - so in your example Bob's added input could be vin 0 if he chose to do so making Alice's input vin 1 and no longer the first...

Don't think I'm trying to be difficult mate :) I do support what you're trying to do here, I'm just trying my best to poke holes in it and/or find fault with it to do my best to make sure it's safe :)


非常感谢楼主 优化扩展的功能 ，对于pagecontrol的显示隐藏问题，我想更多人对 hidesForSinglePage 数据源个数为1的时候是否隐藏 的需求 比对整个pagecontrol隐藏设置的需求量更大一些，希望您考虑添加这个控制功能，另希望您能将该项目配置为同时支持cocoapods集成的方式，非常感谢！


@OliverZiegler what is the concern here, I'm not sure I follow? If you don't have cordova-ios 4.0 or greater installed, it won't install the plugin for the iOS platform even if you try.


"responseMatchingRules" and "requestMatchingRules" should be just "matchingRules".


This was my favorite hotplug on galaxy times... Does it work good?


Govenor *


fast-forwarding 2.0 to borshop-integration-1163-bugfix/gh1162-backport = b7417f21


Also, I needed to add this line to return out of the plugin immediately if multi-select, otherwise it was being activated on multi-selects and breaking the core functionality.
```
if (this.settings.mode === 'multi') return;
```


Accepting a `user_id` for event params means that any authenticated user can create an event for any other user as long as they know that users id all they'd have to do is modify the hidden field in the html before submitting the form

I'll leave you a comment on creating an event for the currently logged in user in `#create`


This is check isn't doing anything now. Simply having `$id = $permission['resource_id']` would be the same.


This is check isn't doing anything now. Simply having `$id = $permission->resource_id` would be the same.


@johnthepink Works now. Thanks!


This is check isn't doing anything now. Simply having $id = $permission->resource_id would be the same.


> So a m-of-n address cannot be the sender? That's the crucial factor imo.

Yup, only pay-to-pubkey-hash and pay-to-script-hash (if enabled).

> This sounds like a much better approach than a generic ...

Agreed. My main motivation was to reduce the number of assumptions and it-works-for-non-obvious-reasons-parts, but I guess I introduced another one of them. ;)

The part that caught my attention was the following:
```c++
if (!GetOutputType(wtx.vout[i].scriptPubKey, whichType)) validType=false;
if (isAllowedOutputType(whichType, nBlock)) validType=true;
if (ExtractDestination(wtx.vout[i].scriptPubKey, dest)) {
    // ..
} else {
    if ((whichType == TX_NULL_DATA) && (validType)) {
        // ...
    } else { // likeky a multisig <<<<<<<<
        // ...
    }
}
```


currently, all the examples have the incorrect matching rules. They all
need to change. How will this affect the other implementations?

On 11 June 2015 at 11:24, Beth Skurrie <notifications@github.com> wrote:

> "responseMatchingRules" and "requestMatchingRules" should be just
> "matchingRules".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bethesque/pact-specification/commit/48719b41dc50b6b7f745517505bf18fff6496762#commitcomment-11624637>
> .
>


UNKNOWN (typo)


Actually, there's no real need for the `$id` variable any more, references to it could just be switched to use the `resource_id` from the array/object.


Would a better way to do these be to make one function
 ```
mark_message = lambda x: prefix_match_text(t.rover_logs, x)
```
then define a bunch of constants like:
```
FLASH_SAVES = "INFO: Saved position to flash"
NEW_TRUSTED_EPH = "INFO: New trusted ephemeris for PRN"
NEW_UNTRUSTED_EPHS = "INFO: New untrusted ephemeris for PRN"
...
```
?

update: looking below, it looks like they don't all follow the pattern. I dunno, this just feels like a lot of functions to keep track of.


L1 cache size is 64 (16kb per core)
L2 cache size is 2048 (512kb per core)

Let me know if this helps on optimizations


另补充个人看法，typedef enum {
    SDCycleScrollViewPageContolStyleClassic,        // 系统自带经典样式
    SDCycleScrollViewPageContolStyleAnimated,       // 动画效果pagecontrol
    SDCycleScrollViewPageContolStyleNone            // 不显示pagecontrol
} SDCycleScrollViewPageContolStyle;  在这里您已提供了 SDCycleScrollViewPageContolStyleNone 这个配置了，可以考虑不提供showPageControl这个配置，改为提供hidesForSinglePage这个配置 跟SDCycleScrollViewPageContolStyle组合起来基本实现了所有对pagecontrol的需求了，当然共存也可以。


Nice catch on my forgetting to say "on Jackpot" :)
And yes, I use tech level since University level can be ambiguous.  (Even though that's what the field is in the empire table, but the empire call returns it as tech_level...)


Reviewed - make sure flight test verifies operation with a slow and fast throttle input and that we test operation in GPS failsafe by setting GPS_TYPE to 0 in flight


I've verified that tests pass. I'll look over the code next. Also, I'd like to have Nirmata weigh in as their business depends on this lib.


I'm assuming `#create` is only supposed to let the currently authenticated user create an event for themselves, if not you can ignore this comment

Let's say we had a reference to the currently logged in user `user`, we can use some rails magic to initialize an event for that user as `user.events.new(event_params)` also, since we just save the event right after we can combine the two lines into one as `user.events.create(event_params)`

We may not have a variable `user` that references the current user, but EventsController inherits from ApplicationController, which includes our SessionHelper. Awesomely enough SessionHelper has this method `current_user` https://github.com/artLopez/bucket_list/blob/master/app/helpers/sessions_helper.rb#L9

So `current_user.events.create(event_params)` guarantees we're creating the event for the currently logged in user, and ties initialization & saving into one statement


I'm curious: what is prompting this change?


lol


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


> I thought ANYONECANPAY also allowed reordering ...

Ahh, I see. Reordering (signed) inputs is never possible.

The following overview is pretty good:

- https://bitcoin.org/en/developer-guide#signature-hash-types

Each signature has it's own flag, and signatures are evaluates from bottom to top, so to speak, i.e. it's possible to create a mutable transaction, but then lock/seal it at some point later.

> Don't think I'm trying to be difficult mate :) I do support what you're trying to do here, I'm just trying my best to poke holes in it and/or find fault with it to do my best to make sure it's safe :)

The feedback is very helpful, and I really, really appreciate it!


Is it necessary to commit this file?


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


@dnahodil review these


Okay I got it, that fixed it! Thanks! Since `execvp` is called right after, I wouldn't need to then deallocate the memory would I?


> I just looked at the NetBSD source... and POLLPRI is part of the 'readable' set there.

That's not correct. I looked in the wrong place. it is in the priority set ( see sel_flag in http://ftp.netbsd.org/pub/NetBSD/NetBSD-current/src/sys/kern/sys_select.c )

These messages will self destruct now. (just replying for anyone following via email)


Good website, there is also another good website for you visit:
http://www.polyurethanesupplierschina.com/jp/products/China-Polyurethane-Components-Manufacturers-14-inch-tyres-China-Polyurethane-Components-Suppliers-so.html#.VXjlH_S4-tg


This change also updated the minimum password length back to 6 characters, which is much more user friendly.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


Correct, this is no longer needed. `debian/patches/add_kFreeBSD_in_supported_os.diff` should be deleted.


1682_waterheater_stratified (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2131 of 2214 tests passed)

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


Generally there are some properties on an object we want to verify exist before saving the object

In this case, having an event that doesn't have a valid `user_id` doesn't really make since, and an event should probably be required to have a date & title as well. Description & location could probably be more optional properties

Here's a link for validations in rails http://guides.rubyonrails.org/active_record_validations.html

If you do add some required fields, it's good to change your `#create` & `#update` controller actions to respond properly when the object in question fails to save


fix #357


when v_rms2 becomes 0 the program crashes


The show page authz is handled by the before_filter in the catalog controller.


Thanks Paul for all the suggestions and ideas, I'll merge my changes with yours, polish everything a little bit and make the pull again.


awesome, been waiting for this!!!


I don't understand the rationale for including "develop" in test but not in build, can you explain this @bertagaz ?


Sad to see it go.


you light up my life, Kenny. you bring me hope to carry on.


Couldn't the `AsyncBatchExecutor` type/instance be replaced by a single `TaskCompletionSource` instance? _(Task pub/sub)_


I agree but the right fix is not to make a wrapper to hide the ugliness but change the way `LedgerEntrySet` is used in that one place (PathState or RippleCalc I believe). This requires touching business logic and understanding how it works.


Just wanted to let you know, for some reason this setting on the N6 cause force closes. I don't know if it's different on the S6, but just wanted to tell ya.


Yeah, it's aweful, especially with the block height, but necessary to preserve testnet history.

`ExodusCrowdsaleAddress` sounds fine, thanks!


Just added a check here.


So, the existing code for doing this replacement doesn't work on Travis?

https://github.com/BBN-Q/PyQLab/blob/develop/config.py#L17


1682_waterheater_stratified (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


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

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


Commit message should read "Keep loading results if we get empty data."


maxChildrenPerSparseNode is rendered unused.


It's preferable to have separate commits for moving code and changing code. When looking at a diff of a commit that contains both, it's difficult to see which things are the actual changes. This is a special case of the [atomic commit convention](https://en.wikipedia.org/wiki/Atomic_commit#Atomic_commit_convention).


Generally if you're doing a "member" route and the resource doesn't exist, we'd throw a 401 or 404, so you shouldn't have to worry about the case where this view gets loaded and there isn't a proper user (because the controller should intercept that request & handle it)


Co to za commit do ciężkiej cholery, nazwa z dupy sugerująca że znajdę tu task a tu jakaś tona tekstu dziwnej treści.


Ten commit to żart? ... Commit zmieniający jedną nazwę?


It would be nice to maybe have the top bar reference the wiki or something. I had to dig to find any news on why it was going away.


Yes, ViewPagerOnTabSelectedListener works well. And I missed tab.getPosition(), I think this will work as well.


If you want to keep `@user_name` as a list of users I'd probably change the name to `@users`, but it seems that you always just access the first one. So I'd probably change this line to find the user with that id and store it as `@user` https://github.com/artLopez/bucket_list/blob/master/app/controllers/users_controller.rb#L25


Znalazłem klasę task scrolując...


Doesn't seem like this is being used anymore


Actually, try altering the test to pass the sig in as der bytes ? Got to
turn off phone ... damn flying


A long time ago we attempted to have a proper installable Python package.  It ended in relative import hell but we could resurrect the idea and have a ~/.pyqlab folder for the config.


Cool thanks -  let me test and get back to you.


https://github.com/Evgenii888Evgenii
![10998322_870930146283110_2574079707567678809_n 1](https://cloud.githubusercontent.com/assets/12538992/8098256/5299ad9c-100f-11e5-81ab-5afdf245f8b4.jpg)


makes sense


> ideally there would be some separation between identifying an Omni transaction and identifying an Exodus purchase

Yes, I agree. This would be a very reasonable next step.

The whole Exodus fundraiser could actually be wrapped into something like:

```c++
// 1. Select Sender
// ...

// 2. Exodus Purchase
for (unsigned int n = 0; n < tx.vout.size(); ++n) {
    CTxDestination dest;
    if (ExtractDestination(tx.vout[n].scriptPubKey, dest)) {
        if (CBitcoinAddress(dest) == MoneyAddress(nBlock)) {
            int64_t BTC_amount = tx.vout[n].nValue;
            if (0 < BTC_amount) {
                TXExodusFundraiser(tx, strSender, BTC_amount, nBlock, nTime);
            }
            break; // TODO: maybe continue and get all values
        }
    }
}
```

But it probably needs a bit more restructuring. I assume it would be a significant performance hit, if the sender selection is run twice, e.g. for the Exodus purchase, and another time for the actual transaction.


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

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


細長い方のバッジを"The source code for dit-rohm.com"の上に置こ


On 11/06/15 10:51, Andrew Yates wrote:
> We don't flag incomplete 5/3' at all? That sounds like a useful feature
> doesn't it?
GenomeLoader stores has_start_codon and has_stop_codon in the
translation_attrib table (for CDSs in fragments of sequence where the
CDS overhangs the ends).

--
Dan Staines, PhD               Ensembl Genomes Technical Coordinator
EMBL-EBI                       Tel: +44-(0)1223-492507
Wellcome Trust Genome Campus   Fax: +44-(0)1223-494468
Cambridge CB10 1SD, UK         http://www.ensemblgenomes.org/


@mbernat I mean that I am simply manually submitting all review requests that are accepted (Shit it!). The state is anyway propagated to the tracker via a hook, so it's already fragile in this respect...


@SublimeFrederic Nope, look again it has been changed to

    'compiled' => realpath(storage_path('framework/views'))


havana models also come with start_NF/end_NF flags for CDS and mRNA sequences, stored as attributes
Don't think they're used anywhere in the API though, only in the serializers


@tchap Shit It! :D You're right I didn't realize that it's fragile in the same way. In that case I'll also start closing accepted stuff.


Aaah, I see.


@kamcio96 Mavenizacja ;)


I think it's fixed, though I have no idea what that code is supposed to do (a comment would be nice!). `read -d ""` returns a non-zero exit code if EOF is reached, so the `|| true` is necessary.


@mbernat Edited the message, it was a typo... O:-)


test


Note that Sphinx docs aren't building correctly on this latest commit due to whatever is causing AttributeError messages.


DesignDaysGridView (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


Посмотри новый коммит(ValeraS@1e090656ce3fc962a725d224846b3b8f22610797), может так устроит?


there are a lot more add-on out there that are bad. Also a simple id change could work around that.


Nécessite la lib "httpful" https://github.com/nategood/httpful
http://phphttpclient.com/downloads/httpful.phar


Oh for. Right need to do better with this as it sounds like we've got 3 ways to do it

Magali Ruffier wrote:
> havana models also come with start_NF/end_NF flags for CDS and mRNA
> sequences, stored as attributes
> Don't think they're used anywhere in the API though, only in the
> serializers
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ensembl/ensembl/commit/3301ad54a15a12d39759e0546de03d77ffeee0d9#commitcomment-11630550>.
>


Hello, I have located the buggy code, and looking into it.


There’s no need to do that here. If the $_GET var somehow is populated with objects that implements the __toString() method, well … so be it. Even in that case, == is probably what we want.


On 11/06/15 11:07, Andrew Yates wrote:
> Oh for. Right need to do better with this as it sounds like we've got 3
> ways to do it

Yup, agreed. If anyone has any strong feelings, let me know...

--
Dan Staines, PhD               Ensembl Genomes Technical Coordinator
EMBL-EBI                       Tel: +44-(0)1223-492507
Wellcome Trust Genome Campus   Fax: +44-(0)1223-494468
Cambridge CB10 1SD, UK         http://www.ensemblgenomes.org/


This is the project I created for my graduation work. It's basically a instant messaging tool which enables user to send encrypted message to each other. It has not been maintained after graduation, and obviously a lot of docs need to be added.

If you are interested in make use of this project. I suggest that starting from the nativelib folder. It contains makefile for the JNI lib. You should be able to compile it after some minor changes to the makefile, like path of jdk. The you can import the two projects into eclipse: ibejnilib and jibe. ibejnilib is just the java part of the jni lib, and jibe contains OO encapsulation of the C-style jni functions.

After you have imported the projects into eclipse, you can try to run the JUnit test cases under both projects. Just take them as examples and you can use the encryption lib in your own project.

There are three projects under server folder. They are applications of the encryption lib.
* key_dist_server: A java web project which allows a user to register and retrieve encryption keys.
* key_gen_server: A EJB project which generates encryption keys for a user.
* msg_relay_server: A PHP application that relays messages between two users.

There is an iOS app project in client/iOS, but it was based on iOS 5, which is quite old now. It requires some efforts to make it work with current release of iOS and xcode.


Hm, I can't remember now - possibly an earlier version of this modified one of the nested dictionaries? Anyway, I'll replace it with `.copy()` now.


All the other election specific URLs start with `/election/`, any reason this one is different?


This should be the normal id.


guid string infact include 'creature' prefix for creatures, entry id and low guid part that can be used for lookup data in entry/guid based tables.


See https://bitbucket.org/anthony_tuininga/cx_freeze/issue/127/collectionssys-error


Yeah, long story short the global balance arrays will only contain the total *spendable* balance now to avoid confusion.  There are extra benefits too:

* Properties are now added to this new list directly from the ```TokenMap``` based on whether an address has previously transacted in the property rather than if the balance is non-zero (so if you receive 10 tokens, and spend 10 tokens, the balance is shown as zero instead of not at all)
* Properties where you have no spendable balance but are watching an address with a balance are added to this list so those properties can still be selected from the balance property selector

I'll try and put together some notes on my proposed changes to handle watch-only a bit better once I'm finished with them bud :)


If this form is on a page with election in the URL, better to take the election entry from that rather than a form field that can be changed?


Checking the variable is faster that running is_user_logged_id(). If the first part before the logical and is false, the second part will not be evaluated. Thus will is_user_logged_id() not run if the $tokenId is not empty().


done


yes, I am not sure whether we will maintain this patch or drop it.

>Also a simple id change could work around that.

yes -- but the plan is not to prevent install, just warn the user.


Should probably use https://github.com/geonetwork/core-geonetwork/blob/3.0.x/web-ui/src/main/resources/catalog/js/CatController.js#L11 ?


I see.


Did you check formatting with JSLint - could you run mvn clean install -Pjslint in web-ui module to format ?


I think you might avoid to add commons-io dependency by using java7 try-with-resources and streams.

It would be more reasonable in terms of jar footprint


You are right.


Line 6 probably isn't, and line 9 isn't unless you are using the TimeAgo() function.

(Was messing around with the network graph and saw this branch :stuck_out_tongue:.


The Mithril home page downloads the gzipped mithril.min.js v0.2.0 at 8.2kb
according to the network panel of Chrome Devtools


The problem just comes from x = np.matrix([1, 0, 0]).T where the vector is made with integers instead of doubles.
Solution : x = np.matrix([1., 0., 0.]).T

;)


Thanks!


On Wednesday June 10 2015 19:24:39 Mika Lindqvist wrote:
>The above block doesn't work. It is missing FAIL_REGEX "not supported".

With cmake 3.2.2 it does work and detects that my compiler supports the attribute?!


Ага, так лучше.
Изучаю.


You are correct. They didn’t cause any harm, but they were not necessary. Removed in [ad8c9349b0761ddfc0df71fd0ee2fee50b50734e]


This will be tricky stuff to do as we want to have old functionality working and new one commented. I will have to rethink how to test it.


@sultanxda
Hi Sultan, today I import the framebuffers and mdp from ivanich's pyramid kernel, however it also doesn't boot. The error logs are similar to your villec2 kernel.....So could you tell me how to boot it? It seems like a NT35590 mipi driver error:
http://www.pastebin.ca/3024413
Or is this related to ivanich's gamma hacks? I see "mipi_dsi_cmd_dma_tx: dma timeout error" in the last_kmsg.
Thanks very much.


Hmm, can you please change the commit msg to say this is the catalog translation? i.e. the subject should be something like "catalog: zh_TW translation" or so?


>Процедура ПакетныйРежимЗапуска_ПослеЗагрузкиТестов(Результат, ПараметрыЗапуска) Экспорт

Имя параметра `Результат` нечитабельно и непонятно.
Если он не юзается, предлагаю поменять, например, на `НеиспользуемыйПараметр`


Because I needed a replay-1 functionality, and AsyncSubject (as a higher-order Observable) doesn't provide that when I mergeAll() the AsyncSubject. Another way of saying it is: why AsyncSubject?


wow so test


@darealshinji: shouldn't we build-depend on ```dh-python``` after that?


@lukasgeiter
``
realpath(storage_path('framework/views'))
``
Strange, this line return false for me, i have to add the / for Laravel 5.1


>Процедура ПакетныйРежимЗапуска_ПослеПолученияСвойствФайлаОтчетаТестирования(РезультатОбработкиФайла, ПараметрыЗапуска)

Аналогично `РезультатОбработкиФайла` не говорит о сути параметра, предлагаю поменять на СвойстваФайла


initialized


Аналогично `РезультатОбработкиФайла` не говорит о сути параметра, предлагаю поменять на СвойстваФайла


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


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


It is a filter, so if we’re not actually doing anything with the value we should return what we retrieved. Fixed in [afc5ae982c1b3b3ce57861a866a63e6b2e962bd0]


this is a comment


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


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


Another approach would be to modify the string (or array) returned by the get operation, so that it can store the original JSON object.  They original key should also be added.  Then, the wait_next could use the value of the index in the JSON when waiting.  This could be equally applicable to strings, and arrays, and maintains the simplicity and useability of the current interface.  Later, when users desire more access to the internal structure of the KV pair, such as flags or the lock, methods could be added to this extension module to allow access to these other parts of the stored JSON.


Нам невидимость нужна где-нибудь? для `.git` где-нибудь юзается?


`this` to `window`


функция `СвойстваФайлаПослеПолученияТолькоЧтения` зачем нам нужна?


This is doing a MapIt area ID lookup with a post_id, so I guess something a bit awry here still?


I'd expect this wouldn't need the `.to_s` check on line 88. This alternative would likely work and be a little more conventional ruby (assuming `nil` is returned when there is no match, `nil` is falsy in ruby):

```ruby
if match
```


R E F A C T O R B O Y S


All aboard the refactor train


No, this is wrong. Note that window_free() internally, via window_unlink() actually removes the window you specify from the FileDescriptor object's window list that is assigned to.

Or in other words: f->windows is a linked list, and window_free() not only frees a Window object but also removes it from the list. Since we keep freeing/removing the first item until all are gone, things are safe.


Also see `Array#empty?` method for an alternative that reads a little clearer: http://ruby-doc.org/core-2.2.0/Array.html#method-i-empty-3F


done


Since this is a non-urgent fix, perhaps I should park this branch for now. It's an ugly fix at best.


Thanks for spotting the error, Python strikes again!


THis is not a bug either. window_free() calls window_unlink() internally, which will actually unset m->last_unused to NULL if it is pointing to the window.


The `while entries.length` check can be skipped. `entries.each` will handle an empty array without issue.


I've changed the properties in this line as well: from start/end to min/max. Github's diff does not highlight this line, I don't know why.


Do we need to check these calls are succeeding, so we can rollback or error (`result = NO; return`, as where the migration fails)?


Nice.


There look to be a few areas of the code that are still assuming post=mapit ID, if I'm reading it right. Isn't mentioned in the PR todo, so thought it's worth mentioning here, e.g. get_area_from_post_id, gss_code setting (guess that's UK specific anyway), ToggleLockForm, ResultEvent.


This is a regression if you consider that pull can lead to merge conflicts. That was the main reason behind using reset.


Dur, this is clearly UK specific anyway, given the 2015.


Shouldn't this be "Could not achieve ..." (note the additional "not" and the remove "r")? :)


Minor points above - overall excellent work :+1:


@frank-trampe anything I can do to test this on mac?


Link didn't work for search-page. Should be /title-search?


I think this change is worth a bit of explanation in the commit message - MySQL's laxity regarding `GROUP BY` isn't obvious.


maybe a format too, I haven't run one for ages I always forget :p


Yes, it's throws exception as before.
But for example: when I use Zendesk API Client PHP in a way like this:
```php
    try{
        //... some code skipped
        foreach ($identities->identities as $identity) {
            if($identity->type == 'email'){
                // $this->client - instance of Zendesk\Api\Client
                $this->client->users()->identities()->delete(['user_id'=>$id,'id'=>$identity->id]);
            }
        }
        //... some other code skipped
    }catch (\Exception $e){
        // some reaction here: write error to log, and throw exception up (or not, it depends)
    }
```
Before this pull request, my workflow just listen for exceptions and catches it.
But, in sometime, it start fails. Output was broken (it return not valid XML, JSON for example).
And the reason is ZendDesk error text, flushed to output buffer.

Solve in my owned code is add one more output buffer level, like this:
```php
    try{
        //... some code skipped
        foreach ($identities->identities as $identity) {
            if($identity->type == 'email'){
                ob_start();
                // $this->client - instance of Zendesk\Api\Client
                $this->client->users()->identities()->delete(['user_id'=>$id,'id'=>$identity->id]);
                ob_end_clean();
            }
        }
        //... some other code skipped
    }catch (\Exception $e){
        // some reaction here: write error to log, and throw exception up (or not, it depends)
    }
```

It was repaire my code, and stops flush to my output, but this change was sudden for me.


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


You forgot to modify this. It should read
```
# Wouter Horlings (Achilleshiel)
```


This breaks the other example agents because they only connect to one player and this variant of the Trainer.py assumes all players are connected by agents.


run example error


Just asking: How do you add you extension onto php.ini for hhvm so?


`type` should probably be `image/png` rather than `image/x-icon`.


Ok, no problem. I'll fix README.md tonight.


You can see here...

https://github.com/laravel/framework/blob/5.1/src/Illuminate/Foundation/helpers.php#L528

that the directory separator is automatically prepended.

I'm sure that you know, but "false" means that realpath cannot find the <storage>/<path>/framework/views on your machine...

http://php.net/manual/en/function.realpath.php#refsect1-function.realpath-returnvalues

Could you bring up artisan tinker & run ```storage_path('framework/views')``` to see what is returned, and then try to cd into that path returned.


:cake: Really, really nice work!


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


BTW, don't we have the GWT logo in ICO format in the GWT sources?


This line (and similar lines elsewhere in this commit) still appears to be indented incorrectly.


Title not generic


'yunbaTestCookie'与'testCookie'不相等，还有在禁用cookie时会不会抛异常？


primeiro commit


Indentation should be 4 spaces per level.


Nope the code comes from the website


I just wanted to show, that user can call functions from String explicitly (strip) and implicitly (first)


Again, would be nice to test something on the page :)


This works great. I reckon we should add a filter before the redirect so it can be disabled again (similar to the post-login redirects and post-registration redirects. I've attempted to add it in e2a7968c3389ca03fad9c7add9d2c43a7b20b3b8 but please review and let me know if there's a better way to do it.


Two quote styles on the same line!?


Ca c'est du commit comme on les aimes :P


Review?


有错别字：从git上拿来了第二个例子


What about this?

```ruby
g.remote.fetch
g.reset_hard("origin/#{branch}")
```


type changed


iif -> if


Ah right. Might be good to add JarJar later on to only keep the required classes, but if it works that way, go ahead.

Maybe add a comment in the POM then about why those dependencies have the default `compile` scope and will be deployed, contrary to "standard GWT practices" (maybe just copy your comment above as a comment in the POM).


I thought the `key` column was the hex-encoded key, which means this keyData will just be the bytes from the hex-encoded data rather than the real digest key for the file (as stored in `attachments`). Doesn't that mean that below, where you create a TDBlobKey, it isn't really a TDBlobKey?


This doesn't save?


Right, I'll fix that and create a pull request. Sorry - I am quite new in elixir world.
Should create PR fist instead pushing to master - my bad.


Why the finally clause? If we get an exception here, don't we go down in flames anyway?

I don't object to it as a bit of defensive behaviour, but thought I'd sanity check.


@unknownbrackets :)


Looks good! Only two questions:

Should we create a readme file where we collect the instructions for testing, where we list for instance the command "json-server --watch db.json"?

This commit removes the schema but doesn't add one. Why is this not needed? Shouldn't the server be capable of providing the schema?

Thanks!


TeamCity YaccConstructor :: YC.Core General Build [Build 446](http://teamcity.codebetter.com/viewLog.html?buildId=193040&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Still says 2015 line below.


Why are you doing you requires inside the class instead of at the top of the file?


Probably not the effort to make this work: spaces in install-path; so instead just bail out.
bash praise: it can handle nested spaces ;) it has a match operator like the perl one ;)


If this will be the main testing fixtures, we might want to add more corner case data as well, such as newlines (\n) or HTML entities, special characters, etc.?


I tried to decipher it a week ago, and it seems it's supposed to collect all directories containing so-files and add them to RPATHs.


It's 43
```
32 / 3 * 4 = 42.666666666666664
```


d


Is this a nice mixin to be used in any of the views that now have election in the URL?


Without FAIL_REGEX it detects incorrectly that my compiler supports the attribute because it is not fatal error due to cmake using -Wattributes.


I'd have thought blobFilename needs to be UNIQUE, and NON NULL?

In the sync-android version, I didn't create a PRIMARY KEY, but instead declared both columns UNIQUE NON NULL and created indexes on both -- is there a reason you don't index the filename?

Also, pretty sure this is the case, but can you confirm PRIMARY KEY implies creating an index, otherwise we'll have slow lookups.


We probably ought to allow "T" as well - and in fact we should be tougher and reject values that we don't understand (if somebody puts "7" in there, we should reject it, so the only valid values are T/F, 1/0, true/false, yes/no - and if you do anything else you get an error logged).


Change method name to ```active_page_class```


Let's use "session." since this is for sessions


As the method is only meant for use during the initial migration, I'd consider inlining it in the migration code.


I think this if...else block is not required. Please re-factor.


I still think you need to unique the filename col?


Cool.


Not totally understanding the management command changes, but as long as it produces the same/right results :)


Multiple if...else conditions creates confusion. Use ```switch...case``` instead of


The global tunnel device is not auto-created, but precreated by something which explicitly has to have called at some stage in setup.

Also, why not put the tunnel name in config but default it at this point?


@teoliphant @asmeurer Why do we need to check if the package already exists? It is preventing us from building new or modified version of the package.


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


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


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


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


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


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


Oh, forget about it. :ok_hand:


This logic becomes much easier if the insert fails when you try to insert an existing filename, you don't have to do the generate/select loop, see:

https://github.com/cloudant/sync-android/blob/47761-use-db-for-blob-fnames/sync-core/src/main/java/com/cloudant/sync/datastore/AttachmentManager.java#L511


Isn't this maybe too rigorous? I see that we can lose quite some species here; For example all species which only share sequences with one of the exemplars but not the other, they would be excluded (as far as I see)?


I think that "bird_ip" is a very misleading name for this, and we should change it, even if it requires changes to `calico-docker`.


analysis


We discussed this on IRC on the 4th June. Check your logs.


ditto


Why do we only have IPv4 tunneling? Surely we should (at least in principle) consider the possibility of IPv6 tunneling too, and then send stuff through the splitter (which is a bit more natural than short-circuiting here)?


No need to pass blank values for actions. You will get in view helpers using ```action_name```. Read [this](http://stackoverflow.com/questions/1244921/rails-controller-action-name-to-string)


why we don't have same test for jvm?


Adding the version to the dmg is a good idea - without this, it's possible that providing a different `MAJOR_OS_VERSION` key after a run has already completed with a different version makes AutoPkg check the dmg's timestamp and consider it the correct download, but then merges in incorrect version info.

If this happens because AutoPkg checks the timestamp first (and then returns without checking that the ETag might not match), it might be something we need to examine and decide if URLDownloader should instead check ETags first. I haven't investigated that bit closely, but it seems keeping each cached version unique is at least a workaround for now.


Maybe produce mana when wiggling


TeamCity YaccConstructor :: YC.Core General Build [Build 446](http://teamcity.codebetter.com/viewLog.html?buildId=193040&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (11).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\work\4138ce7c6332a78c\tools\Build.Tools\Core.fsx:line ... Build time: 00:12:34
### Failed tests
```
CommonTests+Components loader tests.All generators: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.SIMPLE right recursion: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Calc test - simple for translator: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Counter test - simple for translator: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.If Then Else: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Parse empty string: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Resolvers: <no details avaliable>

RNGLRParserSimpleTest+RNGLR parser tests with simple lexer.Translate with Attributes: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.Calc no operand in the end: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.Primitive translate: <no details avaliable>

RNGLRParserErrorRecoveryTest+RNGLR error recovery tests.PrintErrorInfo: <no details avaliable>

##### there are 1 more failed tests, see build details
```


useful


@perliedman Use one of mine:
pk.eyJ1IjoidGhlbWFyZXgiLCJhIjoiODgxZTkxMDA1MjkyMjE5N2RhMjc4NjllNTNlYTQ3OTgifQ.lVhSKTZj4JlBY66m5ykTQQ

Just FYI we can see who uses those tokens. :-) Also we should rotate this once a month or so. Just mail me when the rotation comes up.


Starting from here - new stuff.


wiggle signal!


Checks if


Doing it in run-time was too slow? What if I install the `unaccent` extension after having run this?


You might want to square up your version with the current master - there were some [changes I added](https://github.com/autopkg/jessepeterson-recipes/commit/7567ee9a7bbf3812577a853889d9a27640e4d49a) since this PR was opened that have an Input variable substituted in here.


See comment below, after line 140.


Not needed on the homepage -> add a yield in this file and move the line to common.blade.php


training while space


analysis


No.


I don't understand how this comment relates to the code below it.


off?


@trupedia done ;)

Sorry for the delay


registered


nit: may as well check the length of the refresh_token as well, since we've gone to the trouble of checking it for the access_token.


Yes, I need to clean up this file and refactor since we don't need actual callbacks for the tests to run.
We could additionally check stuff in onload events where applicable in followup patches .

After I minimize the needed code, I will see what can indeed be reused.


stray `console.log`?


Issues, Useful


ditto


So far i haven't been able to compile and execute any feel++ example, i have tried hard to do it on my own and failed @vhuber @aancel


Yes, as long as it works time won't be an issue (and the error was caused by unit testing so skipping this step obviously got rid of the error).
Thanks again for your explanation ^^.


This is new - the resource request "tracker".


Yup, that's the issue I ran into. First checked for 10.8 (=> CUDA 6.5) and then checked for 10.10 (=> CUDA 7) and it did not download the latter.


Will do that. Any other suggestions?


Wouldn't it be cleaner only to pass the hosts set if IP_IN_IP_ENABLED was true, thus removing part of the check?


"prints registered pmemcheck statistics" sounds better


Could you add a comment noting how this situation could come about, it's not clear, given we generated the dstPath filename above, why we'd have the situation where the file exists already?


and what extensions do feel++ files carry?


True, not all resource types are yet supported. So that's what I meant by "needs more work" :-)
spec.src.json is suppressing a lot of resources currently.

If you mean the blockable vs optionally-blockable, I would avoid sub-categorizing here.


homeworks of bruce


Changing all files in single commit is absolutely wrong. It's impossible to know what really changed.
What's the purpose of this -single commit- release?


God please look at the darn code! There is no trailing whitespace! The line ends in a star that is aligned to stars of the other lines!

Doesn't matter anyways, because the PR is closed.


@mattsoutherden @lostie: I've got a slightly different approach in 48c506a.


I *think* Readers will raise either ValueError or IOError now.... but definitely something we could tighten up.

FormatError is dead and buried!


We could run just that step manually. The related indexes can't be handled
on the fly anyway

On June 11, 2015 12:59:37 Pedro Ferreira <notifications@github.com> wrote:

> Doing it in run-time was too slow? What if I install the `unaccent`
> extension after having run this?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/indico/indico/commit/816b10cff1da697535ce7da7fa75fc96905b55b4#commitcomment-11631386


I'm going to submit that patch to the source repository.


great job


Менше — http://sum.in.ua/s/menshe


I added some inline notes (mostly typos and one clarification request).


This merge is broken. Actually both `+` and ` ` (space) should be allowed in a username.

Seems like someone added the `+` in 4.1, but did not include a spec ^^


DesignDaysGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Deneme comment


neden


Agreed & updated.


It might be, yes. I guess the correct implementation would make sure that the exemplars are present in the supermatrix, not necessarily in each individual alignment, and that all taxa in the supermatrix are transitively reachable from all others.


Done.


Here also! :-)


Sold on sed


Really, we already have.


Yep, done


а зачем проверять невидимость каталога `.git`?


You should open an [issue](https://guides.github.com/features/issues/) per question.
Without that, you won't be able to collect data for your report at the end of the internship.


About that, I guess we should discuss this section a bit more:
http://www.w3.org/TR/mixed-content/#requirements-forms


To #3


Hum, I'm not sure. The GS doesn't have the notion of height (very painful). The only constraint is to start at a page (only for target and zbuf not tex neither clut). It is allowed to upload a small texture in the remaining half page. It is really up to the game designer where it will render. The GS memory is just a big fat blog, and texture/rt/zbuf/clut only pointer. Overflow is a built-in feature


Maybe update the docs. (Or at least add some examples for the new use) I had some issues, where I needed to fix a bug but without updated documentation and examples :(


Hi I am facing issue : when client try to checkout in opencart version 2.1.1 he gets an fatal error..
cal a member function trigger() on a non object in catalog/ model /account/customer.php on line 4

please help


I believe this was intended for testing <picture>, but at this point it's only a note-to-self about covering this scenario.
Notice that I suppress all of these until I determine what to keep or move out.


En ulempe med denne patchemåten er at Kohas egne tester ikke kjøres etter at vi har patchet Koha. Vi kan innføre feil som Kohas tester ville fanget opp.


Litt spent på hvordan det blir å vedlikeholde disse patchene om de patchede filene endres. Men det finner vi ut etterhvert.


Surely an insert call would be more natural and readable? I'm not going to get too over-excited since the code is correct, but it seems it could be a bit clearer.


Yes, we should ...


I'd be tempted to names these something like: `resourceName, sourceFolder, targetFolder`. Doesn't have to be those, I just had to check usage before I understood what they are so wondering if this could be a bit more literal.


What is ``build.gradle`` is this a sample user can use to know how to configure the plugin?


What is this used for?


Interesting - this starts with calico, not felix. I think it should be consistent (though we have an issue that the ipsets and chains probably ought to start with calico, one that I agree with). And you need to be a little careful that our tag cleaning up code doesn't hit it there too.


@NobodysNightmare thanks for the heads-up. Will fix.


I'm reading this as one parameter rather than two because they're stacked vertically until this point. Could be PEP has trained me this way but might be worth stacking them all.


Shouldn't this also be `...Handler` like the others? For consistency, but also because federation in itself is more of a concept, not a concrete thing...


Do we need an intermediate forms_wanted variable here?


Shouldn't this be ``j2ObjCPreBuild``instead of ``j2objcPreBuild``?


why do we need this property?


just added kmod-nf-nathelper-extra


Add copyright banner to the top


`Fixes issue 2748` is not really very descriptive since it adds "lookup on github" as an abstraction


@VivienBarousse: The comment here has been duplicated from line 21 and doesn't seem to relate to your change.


@ParthivPatel-BTC @PragatiDoshi-BTC Shall we not use `flash.now`?


* TODO: Define a better validator for the repository, should ensure it is a valid URL and has .git at the end.


* TODO: Check for input, make sure it is a valid gitlab hook, check repo and branch are correct


* TODO: Use a decorator pattern here


* TODO: Don't allow this to run if there is already a pending deploy or no servers


$server->first = (count($output) === 0); // FIXME: Let backbone.js take care of this


* TODO: rewrite this as it is doing way too much and is very messy now.


// FIXME: Handle this situation as it is then unclear what went wrong


* TODO: Should this be an attribute?


* TODO: Translate


I came up with idea that we don't need themes manager for the initial release. At the beginning the mod may have only two hard coded themes: "Vanilla International" and "Vanilla American". This will be enough for beta release, in my opinion. What do you think?


Makes sense.


Built successfully, yet have an error with message like this:

nnnuser@nnnuser:~/ns-allinone-3.23/ns-3.23$ sudo ./waf --run fd-nnnsim-v24
Waf: Entering directory `/home/nnnuser/ns-allinone-3.23/ns-3.23/build'
Waf: Leaving directory `/home/nnnuser/ns-allinone-3.23/ns-3.23/build'
'build' finished successfully (1.682s)
assert failed. cond="uid != 0", msg="Assert in TypeId::LookupByName: ns3::nnn::ConsumerCbr not found", file=../src/core/model/type-id.cc, line=567
terminate called without an active exception
Command ['/home/nnnuser/ns-allinone-3.23/ns-3.23/build/scratch/fd-nnnsim-v24'] terminated with signal SIGIOT. Run it under a debugger to get more information (./waf --run <program> --command-template="gdb --args %s <args>").

Need check on ns3::nnn::ConusmerCbr.


```
Функция ЗагрузитьТесты(РодительскаяСтрока, ОбъектФайл)

	Если ОбъектФайл.ЭтоКаталог() Тогда
		Если ОбъектФайл.ПолучитьНевидимость() и ОбъектФайл.Имя = ".git" Тогда
			Возврат Истина;
		КонецЕсли;
```
чтобы не грузить тесты из репозитария Git, т.к. там могут файлы-обработки (например, из precommit1c - v8Reader.epf)
https://github.com/xDrivenDevelopment/xUnitFor1C/issues/364


Thanks Maria.


>`Если ОбъектФайл.Имя = ".git" Тогда // ОбъектФайл.ПолучитьНевидимость() и `
Увидел у тебя в УФ.
Можно и так.
Но тогда код по получению невидимости и `только чтение` вообще не нужен :(


IIRC the reason was to ensure the branch pointer was correctly set, as we were using that to test that the branch switch worked correctly. We couldn't check for a particular revision being checked out since the state of repositories isn't fixed during tests, as we pull actual live ones.

But I believe a mix of both approaches, with a checkout followed by a hard reset should work.


There are still some jslint warnings I am not sure how to solve. Some docs
missing it says... on the jsonix automatically generated files.

On Thu, Jun 11, 2015 at 1:21 PM, François Prunayre <notifications@github.com
> wrote:

> Thanks Maria.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geonetwork/core-geonetwork/commit/bc9c640296c891cdc379284b8861119ae2f746db#commitcomment-11631728>
> .
>


Please put static files in the correct folder. "/static/"


:fearful:


Так невидимость этого каталога нужно проверять?


Yo lo veo bien, salvo el tema de que no vas a poder acceder a los métodos protegidos del portlet que hacen cosas (como sendRedirect y otros). También faltaría lo de poder usar un parametro personalizado en lugar de mvcPath para indexar el comando.


Yes, instructions are a good idea. I'll take care of that. The schema/UI schema aren't needed anymore since they won't be delivered by the backend anymore. It'll be the responsibility of the client to provide the appropriate schemas.


Good catch, thanks for pointing it out. I just removed it in 7acb985694.


So what about `if resp.is_a?(Net::HTTPSuccess)`? Does it turn out to superfluous?


We don't, will remove it


Since `book` and `book.add_entry`  are setup.. I tend to put these in a `before` or `lets` block.  In this case it doesn't make a huge difference since you have have on example (`it` block) but I still think its a good best practice.


I should stop working on two projects at the same time.


I don't think you need this.

Also since nothing here should be published to the `npm` registry, you should declare it private.
Simply add  `"private": true`


Ok, will type it out


OK, I'll move the IV/CTR saving code to the .S file.


webhook


I think you can put it in `devDependencies` (so no `dependencies` at all)


Merge this to master.


Maybe "Plateforme de jeux open source" will be better. More than one game so: jeu --> jeux.


Где-то ошибка.
На ТимСити пропущенные/pending тесты стали обозначаться как fail :(

```
 6 tests failed ( 6 new)
   Group by:
 All tests
 Tests: CommonApp (1)
  ТестыГенератораДанных.Тест_ДолженПроверитьИзменениеПредопределенногоСправочникаПоГуид

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Отсутствует реализация тестового метода (Тест.Тест_ДолженПроверитьИзменениеПредопределенногоСправочникаПоГуид();).

 Tests: selftests (5)
  Tests_CmdLine.Протестить вывод лога в формате TeamCity (Тонкий клиент)

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Отсутствует реализация тестового метода (Тест.Тест_не_реализован_из_за_особенностей_командной_строки_тонкого_клиента_1С_ВыполнитьОдинТестЧерезЗапуск1СПредприятия(ПараметрТеста);).

  ТестПроверяетНезависимостьТестов.ТестДолжен_ИзменитьСостояниеОбработки

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Независимость тестов путем создания разных экземпляров обработки-теста больше не поддерживается

  ТестПроверяетНезависимостьТестов.ТестДолжен_ИзменитьСостояниеПеременной

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Независимость тестов путем создания разных экземпляров обработки-теста больше не поддерживается

  ТестПроверяетНезависимостьТестов.ТестДолжен_СравнитьСостояниеОбработки

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Независимость тестов путем создания разных экземпляров обработки-теста больше не поддерживается

  ТестПроверяетНезависимостьТестов.ТестДолжен_СравнитьСостояниеПеременной

First failure:  	feature…ent-466	#168	Changes (242)	11 Jun 15 13:29
[Pending] Независимость тестов путем создания разных экземпляров обработки-теста больше не поддерживается
```


doesnt need to be verbose. "hacks ReplyToOriginator intent to work with callbacks"??


Shouldn't you do something with those responses? If they were just meant to be thrown away, the server wouldn't send them.

Take a look at [MailKit](http://github.com/jstedfast/MailKit) as an example of what to do to properly handle these untagged responses.


И еще несколько тестов я не стал указывать.


We avoid using ifconfig elsewhere (it's deprecated, though still widely used) - more consistent to use `ip link set <tunnel_name> up`.


It would be better to duplicate the options before deleting anything from them, as we don't want to change them in place in case that would screw up other uses of the hash being passed to the method. So, something like:

    opts = options.dup
    resp = download(url, opts.delete(:limit))
    File.write(path, resp.body, opts)

Or if you can think of a more concise way to achieve the same.


Нет, давай не будем, просто название каталога и все.


主要是每次拷到u盘里忘了带u盘，登了邮箱会忘退出，其实带个键盘记录的软件.... yeah


Thanks @emmenko! I was not sure about this, since Travis seems to need anyway the CLI version... I'll change it.


deleted
fixed


yes
removed


fixed


You are missing `@JsonProperty("excluded_quantity")`.


I didn't event think about using this data for unit testing yet, but I guess that would make sense at some point :) How about we extend the data issue-driven, i.e. when trying to address issues related to things such as those you mentioned?


If it's only for travis, simply install it globally in travis (e.g.: https://github.com/sphereio/sphere-node-sdk/blob/master/.travis.yml#L7)


https://github.com/opf/openproject/commit/a6d80496dbf1398c521e4e829e115e987cc4765e.

CommitPolice! :police_car:


> Yes, instructions are a good idea. I'll take care of that.
Thanks a lot!

> The schema/UI schema aren't needed anymore since they won't be delivered by the backend anymore. It'll be the responsibility of the client to provide the appropriate schemas.
I see, alright!

Regarding my inline comment:
> If this will be the main testing fixtures, we might want to add more corner case data as well, such as newlines (\n) or HTML entities, special characters, etc.?
Do you think it makes sense to introduce corner-cases? Doesn't have to be with this commit, but maybe we should open a bug on that to look up what the corner-cases in json files may be and add them to the fixtures right away to force us to consider them from the beginning on?


Could you think of an alternative way to do this validation?
(There is nothing wrong with your implementation.. just for a thought exercise)


I also thought about that.

Advanced users can edit the XML file.


sdfsdfs


https://github.com/opf/openproject/pull/3121 fixes the `+` and adds missing specs for both `+` and `@`


asdasda


@strangerweather check my stuff on gerrit :v


This file was created from the GitHub web interface


@mitchellh can you review this before I start the behavior change work? want to make sure the strategy is sound.


Looks good just use same spacing.


Did I miss a change in with_tempfile? Why is it not "TEMPDIR" anymore?


It would be nice to list the keys of the `event` dict like done for `get_releases_by_discid()`.
I know, this isn't done for other entities yet and this is also not done in the WS/2 wiki, but it certainly helps.

The only upstream "documentation" is this:
https://github.com/metabrainz/mmd-schema/blob/master/schema/musicbrainz_mmd-2.0.rng


It seems html doc was not synced after changing xml version. (There are TODOs in html version which were filled in xml version)


It would be nice to have a unit test for this regex


I think the tutorial said to do this.


Ladies and Gentlemen,

I request a license and copyright agreement under MIT license for American Sundown@ https://github.com/daukantas.

Thank you!


I'm planning making a "ploppable growables" mod this weekend. It will feature a toolbar with live previews/thumbnails. And later it may be reused for adding assets to themes by adding "+" button to each thumbnail. This may be Milestone 2.
And Milestone 3 will feature full-fledged Themes Manager.

BTW, what time is it where you live? Are you available all day long or only at nights or mornings?


Thanks. That's better to check the array.

Would you write this as
```ruby
if @address_book.entries.empty? == false
```


> why not simply _builtSetMultimap?

I like the current naming :p


1) Change class name to ```not_records_found_h3```
2) Change margin to ```margin-top: 5px``` in all index pages.


Followup commit coming...


@clefebvre: please merge https://github.com/linuxmint/mintupdate/pull/106 then.


I've meant "Vanilla International" ("Vanilla American") and "Vanilla European" in previous comment :)


Should be perl(Crypt:SMIME)


Use common code to avoid code duplication.


Yep, that's what we have: https://github.com/sphereio/sphere-sunrise-design/blob/master/.travis.yml#L5
That was the only way to make it work. Probably I left the Grunt dependency there before finding the solution.
But it is fixed now, thanks!


We should remove the key instead.


Doesn't "-geometry 256x200" require software-scaling/stretching of the framebuffer? Shouldn't we rather stay with "-1" here?


I think that this use of OutputFile property will return a different filename from the one used by the zipAction ( two lines above).


note to self: check what happens when changing the code so it should fail to build (e.g. `visibility("hiden")`)


Good find - can't we just insert $elem.off(); above here?


This should confirm https://github.com/laravel/framework/issues/9206


unnecessary named parameter


Note how all the plugin tasks use lowercase `j2objc` as the prefix, e.g. `j2objcCycleFinder`, `j2objcTranslate`, `j2objcTest` and so on. This is important for allowing the gradle shorthand, for example, `gradle shared:jPre` which triggers `j2objcPrebuild`. You're required to use all the uppercase letters. If you had to write out `gradle shared:jOCPre` then it's not as elegant.


Good find! Wouldn't it be better to just insert:
$elem.off();
after this line.


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


3: L102 copyright year is outdated, expected 2015 but got 2014


1: L101 copyright is missing


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


This aids publishing to Gradle's plugin directory: https://plugins.gradle.org/plugin/com.github.j2objccontrib.j2objcgradle

See the link on line 17 which explains more about the process.


Yes. I had only added that option a few commits earlier. There shouldn't be anything that relies on it.


`any`?


use zerop


Иправил


Good point Jeff. This change come potentially ripple all the way up to calling applications! Needs further review in my opinion. At a minimum the header file needs additional comments specifying the calling requirements.


- refactoring : js and css
- modify examples layout


You're right, Alec. It is best to avoid it. I'll update the PR to not use _get_plugin_filepath.


this should be the default since it is the current interpretation