This needs to be rebased on top of kubernetes-connect branch and needs to get the http api object from KubernetesClient.connect().

@mvollmer So much stuff is blocking each other here, so I'm taking a bit of a breather until some of the pull requests get merged before doing more rebases.


v2.1.0 released!


sup


That's a good point. We can make the dimensionality check here.


I just discovered that when calling private methods (like `get` in this case) it's better to use `self`: see http://3v4l.org/RqcEC
So, in this specific case is better to keep `self` (otherwise we could mark `get` as `protected` instead of `private`).


I mean,
``self._base_vect = np.array((base_vect[0], base_vect[1], base_vect[2]), dtype=np.float)``


I don't quite get what the original problem was, as the commit message is too terse.

Please, be verbose in the commit messages. Take a look at the Linux kernel development. :) Sometimes they have many paragraphs of explanation for changes of just a single line. :)

That is probably an excess for our little project (since we are not dealing with anything as sensitive as deadlocking the whole system, like the Linux kernel), but I regard the commit messages as one super rich form of documentation.

This is just the first commit that I found:

https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=5c1ac56b51b9d222ab202dec1ac2f4215346129d


I this documented somewhere ?
If not, any particular reason ?


We should use `findmnt` rather than plain ol' `mount`. It has a nicer tree-like and whitespace-aligned format that is easier to read.


No I believe it first news the void function, thereby returning an instance of f. Then it calls the result of that.

You can change the return type if you like. You will still get an error from the type checker about newing a non-void function.


I though I tested it, I didn't. I chose `quit-restore-window` because sometimes `quit-window` wasn't deleting the window and the docstring of `quit-restore-window` seemed nice. Reverted to `quit-window` again. Perhaps I have to set the `quit-restore` parameter, but haven't figured out to what.


Nope;  it's an experimental option that most people don't need.  It is,
however, not going away anytime soon.
On 15 May 2015 16:36, "Victor Berchet" <notifications@github.com> wrote:

> I this documented somewhere ?
> If not, any particular reason ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DylanLacey/zone.js/commit/d4c019bb8d4cfcbb7471881aa11f20c51ca45a34#commitcomment-11208579>
> .
>


Named tuples could be useful also, but this is OK.


See https://adc.luxoft.com/jira/browse/REVSDL-832


No, it's just one. I will change the name of the variable.


You can only call this if the client is Transmission, as you only added the functionality to the Transmission client adapter.


This is necessary for me. Otherwise I get "No name in module" for once for every symbol we use.


Git never forgets. :fearful:


Name of my branch REVSDL-834 so see https://adc.luxoft.com/jira/browse/REVSDL-834


This means that we are killing the other user agents, right? I'm OK with that and edX people would certainly prefer that, as per one previous comment from @pmitros...


Moreover, ``base_vect`` and ``origin`` are also read-only properties (according to my understanding, and the current implementation) ...


Ah, names instead of cryptic magic numbers. That's great!


@Ron-e  Also may want to add an 'if' statement in case the file does not exist it can revert back the the default provided by the module.


Hi @marmbrus
will this be released in the 1.3.x releases or is it scheduled for 1.4?

rgds


       ☝
       ＼＼
         ＼＼
           ＼(ಠ益ಠ)
            /     \
           /     へ＼
         /     /   ＼＼
        /    ノ      ヽ_つ  377ayy lmao
       /   / \
      /   / \  \
     (   /   \  \
     |  |      \ ＼
     | 丿       ＼⌒)
     | |         ) /
     /  )        Lﾉ
    |  /
    Lﾉ


> This may end up leaving a fair number of people silently stuck at that version

When I next make a mailing list announcement I'll highlight the need for Windows users to reinstall the package, which might help a bit.


有翻译记得积极提交，不然别人重复翻译了你的内容就不好了。。
还有你有没有做过测试，你的翻译的title，description，icon，EFFECT这几个单词感觉很可能会是系统关键字，不能随便翻译成中文。


thanks, yes it would, had not seen QPalette yet …
should be easy to convert.

best a

> On 15/05/2015, at 08:14 , Julian Rohrhuber <notifications@github.com> wrote:
>
> would it make sense to use http://doc.sccode.org/Classes/QPalette.html <http://doc.sccode.org/Classes/QPalette.html> for these things instead?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/supercollider/supercollider/commit/e8525d378f4c4793c31f38caab4561ea080702df#commitcomment-11208394>.
>


To revert, Causing issues with DATA Ill need to look into this


Actually you're right. Numpy array construction does not care if the initial data is in a tuple or in a list. We can force 3D here as well.


You also need to update the sql where tv_episodes table is created in InitialSchema @ line 205, and change the db version to 43 at line 216.


ok so Kohei your .gitignore file and my .gitignore file is the same right
i.e from origin/master right ?
why you need me to add .project file ?

On Thu, May 14, 2015 at 10:47 AM, Kohei Sugimoto <notifications@github.com>
wrote:

> You can probably commit your .project file, just make sure it uses
> variables instead of absolute file paths.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plcapital/coagency/commit/a3cb309b6afcb7c0074e56ef61ba2086fa115555#commitcomment-11189109>
> .
>


LGTM. I haven't checked all the details that carefully, but I like the gist of the changes.

What I don't really like is this: `w.h3.a.string`. In my opinion, we should get rid of that (if possible) so that we can use meaningful names instead of cryptic and rigid structures that may break if edX changes their page layout to include other features.


OK.


Is there a good reason why you are trying to generate the Debian control files from the build system? AFAIK the usual way is to have a ```debian/``` directory directly in the repository, including the control files.

The only reason I can think of is substituting the version string into the package info, but there must be a nicer way to do this than spitting out text files during the build, which only serve to build the package again.


Intersection-finding code was copied from here:
http://www.java-gaming.org/index.php?topic=22590.0


Well, concerning the disabled auto-upgrade code, the usual way with most GNU packages would be to pass some option during the build, e.g.
~~~
./configure --disable-auto-upgrade
~~~


这个地方可以用helper.getPrimaryClassName()来获取主要样式类


Thanks for the feedback. I greatly appreciate your touch-ups here and there in these navigation updates, especially your lua binding updates.

Indeed "filter type" is pretty vague. Thanks for highlighting that, because post release I'm going to start on marking flags in the mesh polys (it's there in the off-mesh connections, but you have to manually seek it  out), which is really going to bring "filters" to the forefront and require much more doc wise.

The short-term objective is marking flags and flags/area in shapes other than AABBs, events for CrowdAgents regarding off mesh connections and flags, and alternative a more detailed FindPath variant that includes these details about the path.

If you want to get abreast of things in navigation feel free to arrange an IRC time with me or some other form of contact like Skype/etc.


This causes a **really** weird behaviour of creating *two* different cpio images (one uncompressed and another one compressed). Is this really intended?


I just added something similar in 52e2dd1658ee5c8dcf7030e711c3035cceae300c.


Congrats with your first release! :dancers:


Super. Ich mach heute Abend was zu Zero Cost Abstractions: Traits, wenn Du nichts dagegen hast :-)


L83 index cannot be 2D because it is index of a LatticeNode, which is already forced to 3D in the constructor.


MPManager server and client reset methods need to uninitialize first before initializing it again


Ate Prey, paki-document naman po para mas madali ako makapag-contribute sa backend. Thanks!


i think here label itself no need ,you can use button title r8


什么鬼


这个是不是用$(' >ul >li ', this.getMain())更好些？


HYPE! This is what we want <3


「当映射里处理器时，会当作空操作处理。」应为「当没有对应的处理器时，React 则简单地忽略掉这个事件」


我看到下边`fill()`方法里有给`autocomplete-item`的样式，是不是这样更好些呢？

```js
return $(
  '.' + this.textbox.helper.getPrimaryClassName('autocomplete-item'),
  this.getMain()
);
```


i've been switching anything called `location` to `sql_location` if it's a `SQLLocation` object. is pretty annoying having to figure it out by inspection


由于`$.each`会把prototype的属性一并给遍历，所以建议使用`lib.each`


Just noticed a mistake in this last commit.  Stand by for a correction.


[𠮩	aa	liu](https://github.com/safetychinese/rime-jutjnyu_gaulaupin_dangjunbikwaa/blob/master/jutjnyu_gaulaupin_dangjunbikwaa.dict.yaml#L4708)
這行格式錯誤，第三列不是詞頻。是多音字？


The element picker can block things that aren't visible; if passed a URL it will offer sensible suggestions for a generalised pattern from it. If the URL corresponds to a visible element, that element will be highlighted as picked, but if not, creating the rule still works.


can you put this in a new module called `locations.dbaccessors.users_by_location_id`? see https://github.com/dimagi/commcare-hq/pull/6622 for reasoning


只校验第一个元素？这是为什么呢？不懂


Why? path isn't used after this point.


@AlexVallat hmm — I was almost certain it disabled the *Create* button when no elements matched. I'll have to check again; perhaps I screwed up.


Well, it is a massive and gross hack, like everything in span_utils. In fact it is a hack around another hack. The root cause is that the AST representation of qpaths is horrible. It treats `<A as B>::C` as a path with two segments, `B` and `C` and notes that there is also a self type `A` at position 0. Because we don't have spans for individual idents, only the whole path, we have to iterate over the tokens in the path, trying to pull out the non-nested idents (e.g., avoiding `'a`). So we end up with finding the first ident, `Pattern`, but the span info we have is from the start of the ident `Pattern` to the end of the path, i.e., we get the span for `Pattern<'a>>::Searcher` (because we don't know about the `<>` around the qpath bit. Then we run this method and of course the brackets are imbalanced.

This is revoltingly gross. We could probably handle it better by noticing the presence of the qpath earlier and special casing handling of that kind of path. The long term fix is to have a neater AST for paths and spans for each segment. Given that I'm keen on that long term fix, I don't want to investigate the medium term slight improvement (also because this is a semi-side project and I don't have much time for it).


Okay, after looking at b4fedf5315ecad1e0c54052f3e927ab43803a2fa, I can see the reason as to allow very early microcode loading. Do you think that also solvable by passing multiple initrds to the bootloader?


errno should still be valid from the open() call at this point (free() won't change it), so this isn't strictly necessary.


since you did repeat "api/projects" by adding this unit test, I still don't see the point of having a public constant for the endpoint.
You would have the same amount of repetition by having a constant in the unit test and a constant in the WS (ie. String "api/projects" is written once in the WS and once in the unit test)


@AlexVallat just tested, and it certainly seems like the element picker disables the "Create" button when there are no matching elements (tested in Firefox and Safari). Let me get a fix on where that particular check is going on.


どんまい


乾燥とはっっw


@AlexVallat [Looks like it happens right here](https://github.com/chrisaljoudi/uBlock/blob/master/src/js/element-picker.js#L590).


FYI: the version symbol is actually now backwards \< -> \> , and should be >= 1.7.0-beta2  (or perhaps actually 1.7.0-alpha ... some earlier version.)


Any reason you don't just stat() the file?


fixed in eed9c2594bcd7ad309b5aa00bb47aa39eaed8c51


Yes, you are right (even better)!


지금 올려주신 소스 대로면 툴팁을 없애신거 같은데, 그럼 데스크탑에서 툴팁이 안나오지 않나요?


`est`提供了一个mixin: `.ellipsis()`，可以直接用


どんまいw


(> <) ww


Huh, you're quite right, and it's not some new behaviour either. Can't believe I missed that, I guess when I was testing it I only actually created filters that were for visible elements!

I can't think of any good reason not to allow users to create rules from arbitrary block strings, given that we also allow direct editing of the string.


Hmmm... we're potentially exposing a file descriptor that we might close in an error path. Pretty minor, but you might want to think about restructuring the code so that the file descriptor is only added to the afu struct once we are sure that we are going to return success.

Not really concerned about this though, if you wanted to leave it as is that is fine too.


Why this change?


我觉得`Popup`宽度这里的处理可能会有一些问题。一般来讲，这里的浮层宽度处理可以有两种可能：

1. 和输入框一样宽
2. 内容宽度（或者是内容宽度有最小值）

对于1，除了需要样式支持内容截断之外，还需要给`item`元素设置`title`属性。否则截断的内容用户没办法用。

对于2，我们可以再看看这个需求是不是很强烈吧，讨论下。


@AlexVallat me neither. I locally modified the element picker to always enable the Create button, and it seems to work as one would expect.

I'm trying to figure out, though, whether there's some way to convey matching non-visible resources, as well.


but... maybe we need this someday!


If you have the file descriptor, why not just fstat() it here rather than caching this?

Or are you concerned about performance issues due to the context switch in fstat()? If so, I wouldn't - the AFU error buffer is unlikely to be required in a hot path (if it is, someone has done something very silly) so I wouldn't bother caching here.


The macros from line 80 to 194: Where do you use them? Can you move some macros to the sources where they belong??


look good to me


Thanks @phillipj! :)


Thanks, @DavideD. Seems Fongo is a bit more restrictive here.


You have defined a few new APIs, but they need to also be added to the symver.map file to be exported and usable from the library.

I'd also appreciate if you add some manpages for them while you are at it.

Also see the inline comments above.


@Sword-Smith , husk at "if (boolsk udtryk) return true else return false" kan reduceres  til
"return (boolsk udtryk)" :ear_of_rice:


thank you so much


> The MAC is hashed before sent so it can not be used for tracing.

Shouldn't you add some salt to impede dictionary “attacks”? (I'm aware of the fact, that regrettably [*metrics*](https://github.com/atom/metrics/blob/master/lib/metrics.coffee) isn't doing it, either.)


this comment and the one above need rephrasing


nobody needed it for more than 3 years! :)


がんばれ


Does ENABLE_CACHE come from Altera toolchain? If not, rename it please to e.g. ALTARM_CACHE_ENABLE or so.


I guess there are two use-cases for this. Either the user has directly blocked a resource from the net monitor or logger, in which case they already know they're blocking a non-visible resource, or they have edited the block string (or chosen a suggestion). If they've done that, then I think the most important bit of information to know is whether the new block string works (blocks the offending resource), and if it is too general (blocks other resources).

Without reloading the page I don't think there's any general way to preview the effect an arbitrary block might have. I'd suggest that the most useful measure would be a count of how many resources the rule would have blocked, but even that might not be possible without reloading the page...


it should be possible to use the database functions here for the len and sum, please try that


x2CodeGenerator for x2 0.5.4.0 update


If you don't have any file support, please use OBDCDC_DISABLE_FILE_SUPPORT macro (like for Nios and Microblaze).


I'm not totally sold on the idea of keeping this fd around. Offline you mentioned performance concerns, but really this is an error path and performance should not be a consideration - if it's in anyone's hot path they are doing it wrong.


@vais That's right, but the real problem is that we shouldn't have minitest in the stdlib

That's where the missing files are required


Please add some info what should be fixed...


I don't see any OPLK_IO_xxx() macros (please refer to targetdefs/nios2.h)


@AlexVallat correct. Keeping a backlog of the URLs of the all the requests that occurred on a page is arguably expensive.

For non-visible resources, is there a reason we're sending people from the net monitor to the element picker instead of the "My Filters" tab in uBlock's preferences, and have them edit the rule there?


Use OPLK_IO_WR8()


Use OPLK_IO_RD8()


Actually, for that matter, why not just stat() the path (if it exists) rather than opening the file descriptor here? An application could conceivably query this in their init path then permanently have a fd open as a result.

I think it would be better to delay opening the fd until the application actually tries to read it.


revork


Do you need this block? Otherwise correct it to while (...);


Should be "fixes #4".


ok


Great! It seems all right.


The macros are used in stack/src/arch/altera_arm/*.c files and also some apps/common/src/**/*_cyclone-arm.c files. So I moved it from the source files to the target header to reuse.


this was a ordereddict before, are you sure an unordered one makes more sense?


Do you need those braces around the pointers?


Wow that works? I would think that the list concatenation operator only works on lists, and Travis didn't build anything...


Please use prefix 'p' for pointers!


It is used in the stack only. Okay will rename.


嗯


The suffix '_l' refers to local variables according to our coding rules. Please don't use '_l' and '_h' to refer to high and low, instead use full words:

    pGlbTimerRegCntBaseHigh
    pGlbTimerRegCntBaseLow


没有打算去校验全部的数据，期望用户提供的数组项格式是一样的，所以就只校验了第一项


@mgruner To disable Pre-filters for some templates you could do
```
            # check for heavy templates
            next FILTER if first{ $_ eq $TemplateFileWithoutTT } qw(Header Footer ...);
```

Not all pre-filters add data that can't be cached (like restructuring a template). So a flag "IsCacheable" could run the filter after installation and cache the result...

```If used the wrong way they might harm the system.``` is not a valid argument. Then you have to prohibit every extension because if you do anything in the wrong way it might harm the system. Again, this is something the developer and/or the OTRS admin is responsible for...

```Next time you a struggling with such kind of problem, open up a discussion with examples to test on and lets try to find a solution together. Just an idea, no offense.```  100%ACK


Yes, sorry this one was a huge one, I will try to do smaller and more detailed commits.


是的~


- Wrong cast
- No space between cast and function call

    return (UINT32)getTimerCurrentScaledCount(...);


@acolomb Because official method requires pretty big magic with deb* utils and running on Debian-based distro, what I don't have nor need as requirement. I don't have `./configure` script either, but it should be possible to add flag to `setup.py`. I'll check on that.


안녕하세요.

네, 그렇네요.

데탑에서 okky 사이트 이용 할 때는 툴팁이 중요한데, 그걸 없애 버렸네요.

확인 감사드립니다.

2015년 5월 15일 오후 4:06, 정병태 <notifications@github.com>님이 작성:

> 지금 올려주신 소스 대로면 툴팁을 없애신거 같은데, 그럼 데스크탑에서 툴팁이 안나오지 않나요?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iamssin/okky/commit/0851c2b83d39592349453a63d51585dff3c2554b#commitcomment-11208840>
> .
>


I would recommend to handle cache also here (disabling, flush, invalidate or so)


No need to invalidate/flush cache?


Use INT instead of int


I will add OPLK_IO_XXX() and OPLK_MEMBAR()


Just one more thing :)

```
diff --git a/codec/gen.go b/codec/gen.go
index ec5600f..7e3e4e3 100644
--- a/codec/gen.go
+++ b/codec/gen.go
@@ -582,8 +582,10 @@ func (x *genRunner) encStruct(varname string, rtid uintptr, t reflect.Type) {
        firstVarname := genTempVarPfx + "first" + i

        x.line(sepVarname + " := !z.EncBinary()")
-       x.line("var " + firstVarname + " bool")
        tisfi := ti.sfip // always use sequence from file. decStruct expects same thing.
+       if len(tisfi) > 1 {
+               x.line("var " + firstVarname + " bool")
+       }
        numfieldsvar := genTempVarPfx + "q" + i
        ti2arrayvar := genTempVarPfx + "r" + i
        x.linef("const %s bool = %v", ti2arrayvar, ti.toArray)
@@ -712,7 +714,9 @@ func (x *genRunner) encStruct(varname string, rtid uintptr, t reflect.Type) {
                        // x.line(varname + "." + t2.Name + " != " + genZeroValueR(t2.Type, x.tc) + " {")
                }
                if j == 0 {
-                       x.linef("%s = true", firstVarname)
+                       if len(tisfi) > 1 {
+                               x.linef("%s = true", firstVarname)
+                       }
                } else {
                        x.linef("if %s { r.EncodeMapEntrySeparator() } else { %s = true }", firstVarname, firstVarname)
                }
```
Otherwise I get errors about variable declared and not used .


Do you really need the inline?


i think this should be no problem. i'm adding the values with specific keys (the value of the answer, so numbers from 1 to 5). afterwards i'm iterating over these numbers and look up their values. the order should be of no interest, or is it?


- Please use UINT64 (hope it is available)
- Please use UINT32


Why is this necessary? Most SQL parsers can handle bool arguments without requiring any additional comparison.

```
~ $ psql postgres
psql (9.3.5)
Type "help" for help.

postgres=# select 'yes' where true;
 ?column?
----------
 yes
(1 row)

postgres=# select 'yes' where false;
 ?column?
----------
(0 rows)

postgres=# \q
~ $ sqlite3
SQLite version 3.7.13 2012-07-17 17:46:21
Enter ".help" for instructions
Enter SQL statements terminated with a ";"
sqlite> select 'yes' where 1;
yes
sqlite> select 'yes' where 0;
sqlite>
```
(I know this for certain to be the case for T-SQL too, haven't tested on mysql though).


- Do you really need the inline?
- Please use UINT64 and UINT32


Needs docs in nixpkgs


idont know


Oh, is the point that I deleted too far?


no space between cast and value:

    var = (UINT64)UINT32_MAX;


Okay.


Also here the suffix '_l' is not according to our coding rules!


Is there any particular reason why this z-index (and the one further) is so high? The toolar has a z-index of 1000, so when you scroll the page the conversation author and icons are on top of it.
![conversation-toolbar-zindex](https://cloud.githubusercontent.com/assets/9311307/7648645/94b0c478-fb28-11e4-83bc-1926a5ab3676.jpg)


Your copyright is missing!


Hi @ossobuffo

I will try using this will the code for SmartDocs module and see if there are any issues.

Regards,
Sudheesh


What's the use case here? `src = list ...` wasn't enough?


I guess you don't need care for Xilinx/Zynq specific variables, right? (I'm not sure)


:heart: permalink police


Also here... they are all Xilinx specific!? Can you please check twice, optionally add some "FIXME/TODO" comment


I haven't but others have https://github.com/thoughtbot/dotfiles/pull/373


`quit-window` calls `quit-restore-window` (and really does nothing else), so it's not like using the latter helped you with anything.

`quit-restore` is set in `display-buffer` automatically, you probably shouldn't mess with it.


Do you still need the short enum flag? (same issue like on Xilinx ARM??)


I agree, the parameter is quite arbitrary I tested with 10 and then moved to 20 most because my tests were downloading huge courses, but I think 10 would be probably a good average for users. I don't think this must be an user option since it is quite technical.


Please add info what phy is used on the board (Renesas)


It would be nice to test it in a more constrained machine to see if 20 works ok.


!? There is no 88E1111 phy on that board !?


Sorry, I thought you was already using .NET 4.6!


Do you really need to do all this configuration? I'm asking because it is the same file like for the INK board!


you do not need this function it can be simplified


this change make the variable disp in the following line undefined and the code won't compile anymore.


Right this the idea.


Niet logischer om geen phpdoc te zetten, weinig toegevoegde waarde


yes, I have to confess I don't know how to avoid this noise, can you give me any reference about what i should have done ?


Misschien ook korte handleiding/blogpost schrijven waar we naar kunnen verwijzen over hoe ze een applicatie kunnen aanmaken en dergelijke


Maybe you should add additional file Release.txt to repository with this text?


```You can see that every single uncached file adds at least 20-40ms. If caching is disabled completely the request becomes > 50% slower.``` When a post-filter has to use a heavy regex (with lots of backtracking) to achieve the same it could be worse than that. Did you port all your pre-filters and checked the numbers?

But I guess, that adding IDs to every div/span/... and adding HTML comments when a block is rendered might help to keep the regexes simple.


Klopt dit?


Since there is no host interface present please remove line 60 and 61


A question: will the ``self.index`` be a tuple when assigned in this way?


Niet beter om dit toch nog te voorzien? Of is hier geen nieuwe versie van die met v3 werkt?


@cdjackson : This file seems to be nasty - look at its url: It shows srctest instead of src/test. There seems to be a %90 character between src and test, which causes a lot of trouble. See e.g. Github showing a "?" in the patch when browsing the repo and Hudson is blocked by it as well: https://hudson.eclipse.org/smarthome/job/SmartHomeDistribution/doWipeOutWorkspace
```
java.io.IOException: Unable to delete /home/hudson/genie.smarthome/.hudson/jobs/SmartHomeDistribution/workspace/bundles/config/org.eclipse.smarthome.config.xml.test - files in dir: [/home/hudson/genie.smarthome/.hudson/jobs/SmartHomeDistribution/workspace/bundles/config/org.eclipse.smarthome.config.xml.test/src�test]
```
Do you have any idea what might have happened here?


Is it always USB-1 ? Do you really need a specific cable selection?

Please unconnect all other Altera USB blasters (e.g. INK board) from your PC while working with the C5 SoC board!


Oops, should have been `SITE_URL`.

But google does not accept .dev domains, so I had to hardcode this to localhost to be able to couple this.

Only localhost or real ltd's are supported.


What does this configuration is used for?


It is good please, you have to add the name of the test in the CMakeLists.txt


Staat in de locale, die wordt getoond als er nog geen koppeling is.


`autopep8` is not included in python package for my distribution or available as a separate package tyvärr. Alternatives?


True, I could have explained better this one, the thing is that python can do parallel collections with processes and threads, Processes can use the available processor cores while threads run just in the current running core. So processes in principle are faster, however when you try to kill the script using with Ctrl-C, as there are many processes running you have to deal with the signal in all of them, and this made the script more complex, so I run a benchmark on the thread solution and I didn't see an enormous time difference, and it solved the issue, so I changed it.


In fact I faced the issue with a real Swift server (version 1.12) when doing a HEAD request on an empty file (not a manifest). I dont know if newer versions of Swift have the same behaviour.


Add your name following my format


name again


Hvad vis man godt kan lide snask?


This looks like debug output, you may want to delete this line.


Good work but it can be already simplified


fixed in 73ca505


Does it refer to use dualprocshm?? Then please use CFG_DUALPROCSHM instead (like done for APC2100 driver!)


True, I just tried to homogenize the types, so I continued with dicts.


asa man ka nikopya sa imong folder structure? haha
lahi lage kag folder structure? /<module>/controller?  one controller one folder?


Is niet nodig, maar was de code niet properder geweest als je onderstaande had gedaan:

```php
date('Y-m-d', $endDate),
implode(
    ',',
    array(
        'ga:pageviews',
        'ga:users',
        ...
```


Yes, the checker-only tool `pep8`. Please include that here, unless `autopep8` is a standard tool of Python haxx0rs.


Yes, and I wanted to get rid of this because there was not even an option to choose it.


Ah, ik heb eigenlijk geen idee wat die display advertising is, en heb het nog nooit gebruikt, dus ik dacht dat dat eigenlijk niet echt een gebruikte feature was.


Merci :)


The element picker offers functionality for suggesting sensible generic rules from a specific URL, and for previewing the result of the rule. Given that I didn't realise at the time that the Create button would be disabled if it didn't detect matching elements, there were no downsides to using it and only benefits for those cases where they applied.

I think the element picker would be a nicer experience than just being dropped into the My Filters tab with a new line added (and presumably highlighted in some way).


TeamCity NEventStore :: Raven :: Build [Build 9](http://build.prxm.it:8811/viewLog.html?buildId=2615&buildTypeId=NEventStore_Raven_Build) is now running


Ik vind da persoonlijk eigenlijk niet properder


True, I didn't want to touch such details for the moment since my commit was mostly to organize functions plus some short fixes. But in a future we should do it.


I'm not so sure about this.

The `doneCb` is called after each test step, synchronously. This code runs before the browser acts. Causing logging of a "passed" while the browser didn't even act yet.


Niet beter om een wrapper rond $this->analytics->data_ga->get te schrijven die de profile altijd insert + die een arrat van metrics aanvaard?


TeamCity NEventStore :: Raven :: Build [Build 9](http://build.prxm.it:8811/viewLog.html?buildId=2615&buildTypeId=NEventStore_Raven_Build) outcome was **SUCCESS**
Summary: Tests passed: 58 Build time: 00:01:01


fixed in 5d0ec49


Is it possible to move that file out of the quartus directory? Instead I would prefer to refer to the Zynq board structure, where the linker script is in sdk directory...

Maybe move this file to altera-c5soc/mn-soc-shmem-gpio/ds5 or so?


można zamiast jednego if'a zrobić tak
`@name = resources['name'] unless resources['name'] == nil`
wtedy nawet jak przekażesz tylko jedna wartość w resources to nie wywali reszty


Hmm, mss wel, goed idee.


Yes, self.index will be a tuple by this kind of assignment.


Niet logischer om die onder Common te steken?


line too long (148 > 79 characters) (PEP8 E501) :tongue:


A number of other engines are using the option "originalsaveload" for this.


Don't include that as it's not portable, just use g_path_get_basename() :)


chyba zapomniałeś wyrzucić tego puts'a?


Maybe also these DS5-specific files should be moved from quartus subdir to some other place?


Ok, I revert it back to original.


I would expect this to deadlock, as you hold self->lock and the property setters also take that lock


ok


Also setting the suburi will call this function here again, giving you an infinite loop


Are you experiencing this behaviour?


line too long (95 > 79 characters) (PEP8 E501)


Thanks. :[


ok nog beter


line too long (86 > 79 characters) (PEP8 E501)


negeer maar voorlopig


*GUID should be *ole.GUID


fixed in 5c14445


k


Although the same issue occurs here, as well—"Passed" is printed before the step even executes.


maakt niet uit, gaat gefixed zijn met de wrapper ;-)


The stopping, position remembering and seeking should be done depending on the current target state. And you should recover the target state again after seeking (i.e. if we were playing before, go to playing again)


Okay.


This line on the subsequent 3 lines:

block comment should start with '# ' (PEP8 E265)


I slight confusion here also on my part: I prefer that the for-loop inside the if-statement on L83 is replaced by an explicit comparison of the 3 components (like in L69).


element-type annotation missing. Also maybe this should just be a NULL-terminated gchar** ? That's easier to handle for bindings


修改部分关键字


Get the factory name here, not the element name


It's only executed for unit tests.


And here keep the name at NULL


Not related to http. This piece of doc needs an update.


const


TeamCity NEventStore :: Raven :: Build [Build 10](http://build.prxm.it:8811/viewLog.html?buildId=2616&buildTypeId=NEventStore_Raven_Build) is now running


In fact, should we allow negative index coordinates in ``LatticeNode`` (is this specified somewhere)?


Actually const was not a good idea, it makes usage of the GList API a bit awkward


:clap:


tu chyba mialo byc
`@places = Places.new(resources)`
ale można zrobić tak:
`@places = Places.new({"name" => name, "latitude" => latitude, "longitude" => longitude})`


TeamCity NEventStore :: Raven :: Build [Build 10](http://build.prxm.it:8811/viewLog.html?buildId=2616&buildTypeId=NEventStore_Raven_Build) outcome was **SUCCESS**
Summary: Tests passed: 58 Build time: 00:00:40


line too long (124 > 79 characters) (PEP8 E501, which you probably know by now)


`OSTYPE` is not defined by default on every shell.
Use `OSTYPE ?= $(shell uname -msr)` if its not set.


See http://cgit.freedesktop.org/gstreamer/gst-plugins-base/tree/gst/playback/gstdecodebin2.c#n1023

The stuff with the cookie. We should always ensure that the list we return is the very latest one, and if necessary update it


is the extra `+` needed?


Also the list can be a global variable, and this function have no parameters. It's going to be the same for every GstPlayer instance.


Same here, not related to http


Maybe instead of just the name, we should also remember the GST_ELEMENT_METADATA_DESCRIPTION and return a struct with both to the application


Same here


fixed in deaa48c


Usually I do: for (l = features; l; l = l->next) { ... }


test


Honestly I don't really like it, any better solution? The goal here is to take the engine FormMapping Object and change the processDefinitionId from long to String. See the FormMappingItem web object.


It will always be non-NULL


Okay.


All GstPluginFeatures you get here *are* always GstElementFactories. You can just cast them:
factory = GST_ELEMENT_FACTORY (feature);


Again it can be simplified see my comment on the log, in both case good work we try to finalise that for the pull request and the merge of this two branches in the master


The python bindings look for `libclang.so`, but the symlink is not automatically created^^


I think we can allow negative indices, if the user really wants to construct such LatticeNode by hand. However, factory functions, and Lattice constructor never produce negative indices so a Lattice does not contain LatticeNodes with negative indices.


Fourslash doesn't use the `CompilerHost` provided here and instead uses the one in `services.ts`'s `synchronizeHostData` (which is in turn called from `getProgram` on language service).


Nice to see this checked (here and in other places in the test)


I am experiencing this behavior, yes.


I'm now making in a change in the new branch called datascience-readmes


:D


Could I be running unit tests instead of functional tests by mistake?


It would be nice to have this as two separate commits, i.e. the pristine version of gce.py and gce.ini from upstream and then an additional commit to show the changes we made.


This call to `hostCache` instead of `sys.fileExists` is because of fourslash using this instead of its own host. Investigating a better method


If have start_session: false then yes.

On Fri, May 15, 2015 at 10:04 AM, Shahar Or <notifications@github.com>
wrote:

> Could I be running unit tests instead of functional tests by mistake?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/beatfactor/nightwatch/commit/c4289e632e5f7244f310e633efd3454660e82fc2#commitcomment-11209537>
> .
>


This is a comment on the commit from the datascience-readmes branch.


So I guess the point of this PR is to simplify implementations of HttpClientMetrics and friends, by giving them the guarantee of a single thread calling the callback methods?


You can get it from here: https://pypi.python.org/pypi/autopep8/
It depends on python and pep8, and it has a single source file.


Ok, let's then keep the if statements in the ``Lattice`` class (preferably without the for-loop) ...


Deguns.net is the best place to buy firearms, handguns, guns, optics, ammunition, magazines, and many other shooting accessories online.


Well, I leave it to implicitly be `true`, which according to the docs is supposed to be the default. But in line 72 above here, it resolves to undefined so I guess this is it.


Why are we using * for some items and not for others? e.g.

`"{{ deploy_env }}-tsuru-db"`
`"{{ deploy_env }}-tsuru-gandalf*"`


Please fix your Makefile instead. The reason for that is that there will be a conflict between libev and other  event libraries. However I will take a few picks from this commit and merge them.


Okay.


For now this works, but it's not very smart at the moment. For instance, I would like to have a back-off mechanism when connections fail. Main use case: let's say my SK server is on but my wifi router isn't (yet) on. I don't want the SK server repeatedly try to connect, but I do want it to connect at some point, after turning on the router.


No, remnants of typecasting. Will remove. Thanks.


I very like this comment please keep it in the simplification


I didn't even specify the `selenium` property object in the configuration. When I did now specify it as just `{}`, it seems that the issue went away. So perhaps the defaults of the `selenium` property object only get applied when it is at least specified?


Same in MySQL:
```
mysql> select 'yes' from dual where true;
+-----+
| yes |
+-----+
| yes |
+-----+
1 row in set (0.00 sec)
```


just return exp(x*ln10) here you do not need the selector


how does your nightwatch.json look like?

On Fri, May 15, 2015 at 10:13 AM, Shahar Or <notifications@github.com>
wrote:

> I didn't even specify the selenium property object in the configuration.
> When I did now specify it as just {}, it seems that the issue went away.
> So perhaps the defaults of the selenium property object only get applied
> when it is at least specified?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/beatfactor/nightwatch/commit/c4289e632e5f7244f310e633efd3454660e82fc2#commitcomment-11209635>
> .
>


I agree that the whole Debian packaging stuff is rather complicated, mainly because of the traditional relationship of relying on the "holy upstream tarball". But generating the `debian/*` files during `go build deb` vs. including them in the repo does not affect the need for special Debian package building tools.


Please revert changes done to mn_pcp.qsys due to Nios II instruction set limitation (call, jmpi) what is not handled by compiler!


@hastalafiesta- Maybe this? https://jira.cyanogenmod.org/browse/NIGHTLIES-1224


Using ``2>&1`` won't work on Windows


I spent some time on this matter because I didn't want to break previous behaviour, but there was a need for new cases. It might be that some generalization/refactoring is still needed with what I wrote.

The previous usage of authenticate was to check if someone is authenticated, but failed (when `shouldNotCatch` is false) indifferently if credentials where incorrect or absent. And if it was true, it was always forwarding the request.

The modifications in this commit are two :
  * introduction of `authenticateIfPresent` which forces *a priori* `shouldNotCatch` to true when no basic auth header is present, which allows to handle requests where authentication is optional. In the case of cookies, an invalid cookie should be considered as no authentication, so should not throw a 401 error unless authentication is mandatory (in that case use `authenticate` with `shouldNotCatch == false` as before). So case 1 of your list here, *before checking authentication*
  * in `authenticate` check if the authentication header is still present : it means that `preserveCredentials` was true either for the site or for the authenticator and so needs to be forwarded unconditionally (but this statement might be wrong). So a 4th case here : an unsuccessfully checked `Authorization` header (maybe also an invalid cookie) but the request to forward the header to the upstream server (with `preserveCredentials`)

Does your question come after some problem setting a case (which one?), or just on code review ?


Please try to avoid such changes which have no functional effects!


AAAAAAAAAAAA
GLOBAL VARIABLE


Hmm, warum ```console.warn``` und nicht ```Scandio.debug.warn```? Ist das mit der console nicht gefährlich? Wir haben ScandioJS in Confluence bei Kunden laufen deren Browser kein ```console.warn``` hat. Oder haben wir da irgendwo einen Polyfill?


Also these changes should be avoided.


You removed the |htdocs| directory in 040bdf45.
https://github.com/skylines-project/skylines/blob/master/config/default.py#L46
should answer your question what it is.

On 14.05.2015 14:21, Tobias Bieniek wrote:
> what is |htdocs| and why is it ignored?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/skylines-project/skylines/commit/97dc897a96689f308c65e4afb5d8eaad40d551e6#commitcomment-11194061>.
>

--
Man sollte niemals einer Frau trauen, die einem ihr wirkliches
Alter verrät. Eine Frau, die einem das erzählt, würde einem auch alles
andere erzählen.
		-- Oscar Wilde


As a start, it looks like this:
![screenshot_2015-05-15_11-16-20](https://cloud.githubusercontent.com/assets/635591/7649246/d68fa51a-faf3-11e4-8ed6-12e4bba8a304.png)

I relied on the implicit defaults to help me out.

Previously, the `selenium` property wasn't defined, as I mentioned.


Okay.


TeamCity NEventStore :: Raven :: Build [Build 11](http://build.prxm.it:8811/viewLog.html?buildId=2617&buildTypeId=NEventStore_Raven_Build) is now running


Please revert version back to 13.0 for now. When we move all Altera designs to the same version we can update the version here!


This code looks almost the same. Could you use inheritance there?


TeamCity NEventStore :: Raven :: Build [Build 11](http://build.prxm.it:8811/viewLog.html?buildId=2617&buildTypeId=NEventStore_Raven_Build) outcome was **FAILURE**
Summary: Class failed: NEventStore.Persistence.RavenDB.Tests.when_getting_paged_commits_and_a_subsequent_page_throws_an_error: NEventStore.Persistence.StorageException : Object reference not set to an instance of an object.
---- System.NullReferenceException... Build time: 00:00:34
### Failed tests
```
NEventStore.Persistence.RavenDB.Tests.dll: NEventStore.Persistence.AcceptanceTests.when_a_payload_is_large.can_commit: <no details avaliable>

```


In this file it is okay, that it is specific to Cyclone V!


This looks a bit "random" I know that you got here because GCE was the first platform you targeted but it looks a bit strange as part of this commit though I guess it's ok because it ensures that the commit does not break anything.


$ man help gitignore
       ·   A leading slash matches the beginning of the pathname. For
           example, "/*.c" matches "cat-file.c" but not
           "mozilla-sha1/sha1.c".

Or the other way round: "lib/" matches "/lib/" and "/skylines/lib".

On 14.05.2015 14:21, Tobias Bieniek wrote:
> what does "fix" mean?! this worked quite well without the fix too
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/skylines-project/skylines/commit/e63b1c6753d87d9ed0b0d5cdbbf1a3ab8bbd8de1#commitcomment-11194056>.
>

--
Schön ist's, wenn wir Freunde kommen sehn.
Schön ist es ferner, wenn sie bleiben.
Und sich mit uns die Zeit vertreiben.
Doch wenn sie schließlich wieder
gehn, ist's auch recht schön.
		-- Wilhelm Busch


I think this commit happened when I tried to git pull a previous commit (78193552f84adacf59f9bd4a888c99715b1a033e) that caused a conflict with mine (6fc5b82e14d4b1356259127c959d30a1f4eb16a3).

However, the commit 8ec68d015a474c0a102135e7d9718a0a3242ccd9 (the one just before the one you mention) should be the commit performing the merge... so I'm a bit confused.

Do you have other clues?


Please remove this line in the .qsf:

    set_global_assignment -name PROJECT_OUTPUT_DIRECTORY output_files

... then the files are generated into the quartus project dir, and you don't need to workaround this in the scripts!


Nope, but that's fine, C#6 is quite outstanding!


Two much "value"s:

    value (...) value


Ok, it has been changed later. Accepted


Webfaction should be WebFaction


Why is this macro empty? (Please add comment)


Ah sorry, the name's wrongish. This test actuallyt tests both gpsd compatibility and the stream module.


missing locator


In WebFaction, you need to do three additional steps from a normal WebFaction Django Deployment: Install gcc>=4.6, compile and install ``libsass`` and use the ``compress`` command to bundle static files.


Binary exploit clean up.


You can find how __to__ do this here


this should get some comments about why it exists in the header :+1:


Please add into the commit message where the file is from (Altera installation, Altera support/FAE, ...???)


the point is to be very explicit about the thread/context provided when calling the metrics SPI and use the most appropriate synchronisation when implementing the SPI.

Here is a simple example with http server metrics:  when it is created it will be the http server thread, then the method are called with the same thread, it implies that:

- you can expect different threads to call the `createMetrics` method for http server so if you have a data structure to store state for this http server metrics it needs to be synchronized
- you can expect the same thread to call the methods of the `HttpServerMetrics` object, so if you modify state you don't need to perform synchronization on state used inside the `HttpServerMetrics` only by these methods


opo ae ad


No worries!


There's a missing semicolon here.


consider using [zenhub](https://www.zenhub.io/) maybe? it's directly integrated into github


nyobak za -,-


Run `pip install libsass`


It's recommended that you use a `var` statement when creating variables.


This should probably be in the previous commit https://github.com/alphagov/tsuru-ansible/commit/fa60bdaacec6d5d58e60dfb0081ab4939b6d97f8 ? Perhaps this could have been staged better i.e. one commit handles the host names and the other handles the IP addresses?


You should not mix absolute and relative imports : `from pybb.something` and `from .something`.
Absolute import should be used :pep8 says :
Relative imports for intra-package imports are highly discouraged. Always use the absolute package path for all imports. Even now that PEP 328 [7] is fully implemented in Python 2.5, its style of explicit relative imports is actively discouraged; absolute imports are more portable and usually more readable.


commands


Basically, `shouldNotCatch` still means the same, but is force in some obvious cases. It could be that we can decide to remove the parameter, but this would impact existing configurations.


You have duplicate import of django here.


Yep that's the reason.


__This__ is very important. If you don't collect the static files with compressor, Wagtail's admin won't have any style; compressor uses ``libsass`` to compile SASS files and Wagtail's admin styles are SASS files.


The rest is a normal Django deployment


Don't use assignments inside a conditionals.


Perhaps this could have been squashed into a previous commit?


Is => If

What's doug?


Sent a new [PR](https://github.com/jonathantorres/construct/pull/34) dropping the error redirection.


I don't understand what this sentence means. Is this the script you must run to do the installation?


Indeed, even though the clone() method seems the best place, there are the 2 problems:
* the fact they're not components
* the dependency issues

So in the end I feel the best is to introduce this Copier interface (to make it different from clone()).

Now we still need to decide about the location of it...

What do others think? Thomas?


Oh, you mean that the if's should be removed completely (perhaps for performance reasons) and let the user take responsibility of not feeding negative indices to update_node and get_node? That's how it was originally implemented, but if's were added, I think, to make it more clear to the user that negativity is not supported. :)

At least for now, I can just remove the for loop, as it probably is slower than an if-or-or chain.


Queda definir cómo se debe implementar la memoria, ¿como un diccionario? los array de Python no aceptan clases como tipos, sólo enteros y caracteres.


What is the difference between DUALPROCSHM_KERNEL_SOURCES and DUALPROCSHM_DRIVER_SOURCES and/or the source files set in the variables??

Maybe it makes sense to have a dualprocshm-pcp.c which is stored in the board directory?
-> hardware/boards/altera-c5soc/mn-soc-shmem-gpio/src/dualprocshm-pcp.c
or
-> hardware/boards/altera-c5soc/common/dualprocshm/dualprocshm-pcp.c


Marko why is this so ugly


La clase Memory debería ir acá, pero tuve problemas para armar una jerarquía, cosas que no entendí de la sintaxis de super y herencia en Python.


COMMIT #10000!


Can you avoid this if construct (line 59 - 80)?


Why did you escape quote character after src tag?


EN MERGE! JUHUUUUU!

Det eneste der kunne slå en merge ville være en revert.


the CFG_HOSTINTERFACE variable should be empty/not set at all.


It may be easier to always use  `permissions.perms` in all views or to have a `get_perms_instance` function. With that, we could only update permissions.perms in tests and be sure it's updated everywhere.


Please use elif and check if the node is MN, uses openMAC and dualprocshm.


Just a little typo.
"porviders" --> "providers"


should be want_sles12 here and above


Maby a file "_bases.py" file would be more helpful ? We could add the future abstract classes inside it.


You don't need this anymore, because the drivers.sh script adds that inlcude already!


"If in doubt" ?


Good idea. I'll just add to master?


Hm, given that the same test requires multiple files, and tests are organized into groups (each test file contains multiple tests), I think that naming the data as the test file is hard, but I could prefix them with the test name


Haashub+program


Replacing the for-loop with if-or-or is good for now.

I only meant that if the ``LatticeNode`` does not accept negative indices, we do not
need to test them in the ``update_node`` method of ``Lattice`` class.

But I think keeping ``LatticeNode`` and lattices separate, like you proposed, is a good idea: any indices are allowed for the "stand-alone" nodes, but the lattices which we implement only deal with non-negative indices
(perhaps this should be documented in the relevant doc-strings).


我的


fixed


Yeap, I will add that today thanks


switched to a cygwin-based shell on windows running on top of zsh so all this stuff's actually pretty outdated


thanks tho :p


The enable disable functions handle cache init/maintenance operations.


Okay.


Haha hey :p

Yeah I'm aware Yggdrasil has died again, I went away for a week (due to my missus moving in and sorting out a lot of stuff. I did inform the league.). When I returned the league turned into Rain league and rebased on Infinite, as did Lotus (despite both of them practically having a free personal server).

I added the http node package because I was working on the code with a test environment and figuring out why the files wouldn't create. All the solutions I found used it so I thought it couldn't hurt to try. It's all working no so don't really have any intentions of touching it again.

Even though Ygg has died again, I thought I'd carry on working on this project incase we end up working on code which could be useful for you to use on other servers etc.


I already have some automatic import problems that could be resolved by having "protected files" : `_attachement.py` instead of `attachement.py` etc. This would also discourage people to import models from it's own file instead of the models init file.


Doh I didn't realize those were lists :sweat_smile:.


Ah, here it is - was waiting for this workaround :)

Please remove the line in Quartus qsf as I recommended, then you don't need to check ther version here!


"Niels G. W. Serup" <notifications@github.com> writes:

> EN MERGE! JUHUUUUU!
>
> Det eneste der kunne slå en merge ville være en revert.

Jeg havde selv lidt håbet på det ville være en skamcommit.

--
\  Troels
/\ Henriksen


I will add a disable cache in case it was enabled by us. But do you think it makes sense?


What's the use case for this? I have no idea what prepending a file to an initrd does.


@tonifisler check for `window.console` before, to avoid "console is undefined" error.

and I hope this is at the very end of js files because is not so useful for our visitor but it's fun :chestnut:


Works great :)


How do I do thos


Lot of activity lately,nice !


it is called from the target_enableGlobalInterrupt() which is a little redundant and repetitively. So for optimization, I have added inline.


wrong reference ?


Okay I will change it to INT. I liked the looks of int :(.


True
Instead of making stuff FOR a league itself, we can always make stuff that's meant to be like, a starter pack for leagues.

Btw, I figured something. We should move these files to the config folder, because we basically need to refresh the config.js file for the avatars to be there after refreshing. Soo...I guess we should move this file a bit later. Whaddya think?

Btw, could I meet you on Sora? :3


I'm afraid this introduces a regression, as the default image for those users without a registered gravatar is not the mistery man anymore (`d=mm`). Please fix that or consider reverting.


_cellClassGenertor_ -> cellClassGenerator


Removed in https://github.com/cloudera/spark-dataflow/commit/228edbb76714b1e71943e8f44f01ec7780e0acc5.


I do program.screen


rect???


It actually makes her feel more agnostic than atheist but it works to show that she isn't religious. Still have to have Claire argue/talk to her sister and end up escaping to answer the door where the assailant will be waiting for her.


Lasa doua spatii dupa randurile din blockquote care nu sunt ultimul rand.
De asemeni in engleza, I, adica - eu - se scrie cu I mare oriunde in propozitie. O alta varianta este C mare intors orizontal, dar nu stiu cum se face de la tastatura.


because I deleted class `News` and Rspec was complaining about it


@typeinference
Please take a look at this commit.


Добавить LTE


Cmd support(-c, --command)


That being said, Nicole is religious and a little bit preachy but also loving and caring for her sister. This has to come across in the argument/ talk with Claire. Nicole has to be a character in this story, not just a plot device.


Why does hipache_host_internal_lb in AWS contain -int, but not in GCE?


Maaan, I sure miss all the guys back in Ygg... =/


@jbcrail Thanks for your feedback! I fixed lines 45-46; however, it seems that the method split_whitespace() does not exist any more in 1.0.0-beta.5, as you can see in the documentation:
http://doc.rust-lang.org/1.0.0-beta.5/std/string/struct.String.html
while it seems ok in this page:
http://doc.rust-lang.org/std/string/struct.String.html#method.split_whitespace
What does it mean? Should I wait for the implementation or should I implement the method by myself?


PYTHONCMD="python"
 24 +OPTS="$@"
24 25
25 26  function doProgram {
26 27    # process input from pipe and store as pickled file
27  -  $PYTHONCMD "$BASEDIR/src/processInput.py" "$@"


Okay.


Are you converting into Pythonic counting here? See also discussion on https://github.com/biopython/biopython/issues/486 about similar values in Stockholm files.


Good spot!


GCE does not have internal load balancers, so the routing for SSL traffic has to be handled by using an external load balancer instead.


Should be a full change log at changelog.txt and the latest 2-3 changes in the code.
Not everyone getting the files directly from here, also if someone want to quickly compare at a glance two code files he can easily do so with 2-3 latest changes.


That why you can lost the previous  value of `default_currency`. Also this can be problem when you use this in parallel. Consider to use `refinements` to override the default_currency instead assign new value by initialization.


Not really. The commit message looks like a stash. That is strange. That is why I asked in case you may know what produced such a commit.

That was just by curiosity. If that does not happen too often, so need to worry about that.


Why function is invoking?


Keep the context in mind @uNiversaI. But as your referenced label is used as a media type it should work out in this case.


:+1:  Nice commit :-)


Not a bad idea.

Would they work in a sub-folder in config for just put the files directly into the config folder?

I won't be able to come on as I'm at work, but I'm free all evening (in about 8 hours) if you're about then?

As do I, was a shame that they decided to abandon the server.


filed https://github.com/coala-analyzer/coala/issues/531


I will update this to maxTimeStampLow and maxTimeStampHigh


`UpgradableTrait<WithSpriteBodyInfo>` already exposes a public `Info` field, so this is not necessary.


This should be at least `get; private set` if not outright private.


test


The aim of this function is to cleanup the target. In my view it is similar to having malloc and free pairs :)


string ID 291 is better suited, as it's also a media type like music above


Merged.


I suggest extracting the facingFunc stuff back out to here so that other subclasses can have the flexibility to do their own thing wrt facings.


ああああ


あああ


- Usually the compiler is already that smart to inline it automatically. If possible check the disassembly...
- :)


sureeeee


/**
 * @objMethod - jQuery扩展对象方法
 * 注意     : jQuery插件,必须引入jQuery
 *
 * author : junyu
 * email  :
 * 内容     : refrshBalance          - 刷新余额
 *          tab    -            tab切换
 *          toggleselect       - 模拟下拉选择框
 *          selectfont          -  下拉框赋值
 *          shining         - 元素闪烁效果
 *          positionMove    - 让元素按数组数据进行定位移动
 */

;(function($) {
        $.fn.extend({
            refrshBalance : function(){
            	$.post('refrshBalance?token='+token+"&math=" + Math.random(),function(data){
        			if(data.code == 1){
        				$("#left_acc_p_balance").text(data.content.balance.toFixed(2));
        				$("#left_free_balance").text(data.content.freezeBalance.toFixed(2));
        			}
        		});
            },
            "tab" : function(options){
                options = $.extend({
                    leftCssname  : "",    //左边LI 样式名
                    rightFathername : "",    //右边影响的DIv名字
                    rightFatherElement       : ""     //右边影响DIV的子元素
                },options);
                $(options.rightFathername).children(options.rightFatherElement).hide();
        		$("#"+$(this).attr("div_view")).show();
        		console.log(options.leftCssname)	;
                $(this).siblings().removeClass(options.leftCssname);
                $(this).addClass(options.leftCssname)
            },
            "toggleselect" : function(options){
                options = $.extend({
                    select  : ".option",    //下拉名字
                },options);
                event.stopPropagation();
          		$(this).find(options.select).toggle();
          		$(this).find(options.select).style.display;
            },
            "selectfont" : function(options){
                options = $.extend({
                	select_txt: ".option",    //选择框文本
                	inputval: ".option",    //input val
                },options);
                var questionId=$(this).attr('questionId');
        		$(options.inputval).val(questionId);
        		$(options.select_txt).html(''+$(this).html());
        		$(document).click();
        		return this;
            },
            shining : function(options){
                options = $.extend({
                    opacity_t : 0.1,   //变化后透明度
                    opacity_f : 1  ,   //原始透明度
                    timer     : 1000,  //变化时间(一次循环)
                    state     : 1      //状态:1-鼠标over暂停;2-鼠标over开始;3-一直闪烁
                },options);
                var that = $(this);
                var f = function(){
                    that
                        .fadeTo(options.timer/2 , options.opacity_t)
                        .fadeTo(options.timer/2 , options.opacity_f);
                }

                switch(options.state)
                {
                    case 1 :
                        var s = setInterval(function(){f()} ,options.timer);
                        that.hover(
                            function(){clearInterval(s);},
                            function(){s = setInterval(function(){f()} ,options.timer);});
                        break;
                    case 2 :
                        that.hover(
                            function(){s = setInterval(function(){f()} ,options.timer);},
                            function(){clearInterval(s);}
                        );
                        break;
                    case 3 : var s = setInterval(function(){f()} ,options.timer); break;

                }
            },
            posMove : function(options){
                options = $.extend({
                    speed  : 1000,
                    step   : [],
                    unit   : "px"
                },options)
                $(this).css("position","relative");
                var getPosX, getPosY;
                for(var i=0; i<options.step.length; i++){
                    getPosX = "+=" + options.step[i][1] + options.unit;
                    getPosY = "+=" + options.step[i][1] + options.unit;
                    switch(options.step[i][0]){
                        case "l": $(this).animate({left:getPosX},options.speed);break;
                        case "t": $(this).animate({top:getPosY},options.speed);break;
                        case "lt":$(this).animate({left:getPosX,top:getPosY},options.speed);break;
                        default:"";
                    }
                }
            }
    })(jQuery);


Now that looks strange i.e. using the `gce_private_ip ` with `when: ip_field == "ec2_private_ip_address"`


That's exactly what was needed. Thanks!


:+1: really nice to have this one fixed!


please lowercase "resampling"


That was a cut 'n' paste error... fixed in a later commit..


Shall we add a bot, or would that be pushing it a bit?


Well... about the peeps who were in the Ygg you and Alzzy recreated, I didn't find that league as fun, to be honest. The only time it was really fun was when either you, Alzzy or Deyi were around. xD
But that isn't the case anymore.

I don't know how to add bots, lol.


Yes, just some clean up though.
Btw: I integrated NeoSharp into the main repository and it is working on Windows now.


And uh.. I guess you could move this file directly to the config folder. =P

Btw, I wasn't able to tell you this earlier, but I ended up joining Sora 4 or 5 days after Rain was created.


Somehow it is possible to have that char there...

> Am 14.05.2015 um 23:15 schrieb BigXplosion <notifications@github.com>:
>
> The "®" doesn't seem to be supported, so probably not in MC too...
>
> —
> Reply to this email directly or view it on GitHub.
>


In fact i think a more accurate description would be : "Calculate acoustic fingerprints and try to use them to identify the files by the actual music, even if they have no metadata"


have you checked the context? Are these "channels" referring to TV channels or audio channels?


@dereksmart I can't figure out where this commit is, it looks like it's a detached thing, right? Those changes are handled by #2119 and #2135, I think we can remove it.


I think I wouldn't test the order here, it might change in the internal implementation of PouchDB. You could map all objects to their ids, then sort, and then check that all IDs are present.

Also, why is it `syncedObjects[0]` an array? It should be an object.


srcs list would work fine if we had the ability to also specify a goPackagePath alongside the source.  This works around that limitation and allows you to specify multiple gopaths to just bring in from source. Handy when you have packages which are codependencies.

@domenkozar Working on the docs now.


Agreed. Comment was too long before to want to isolate the new statement that far from the other code. Have shortened the comment now so it can go before the new statement and it's not so isolated now.


If your intent with this is to keep a reference to the int at index 0 it wont work. irFrontRight will be set to the value of  infrared[0] at initializitation, resulting in and will forever remain 0.


much better


is the "for DSP" really relevant here? We would already have a string "cpu usage"


can we do [`remoteObj1`, "test3"] here? So it has one local, one remote object?


Bitte bestehende Konventionen einhalten, sonst verwirrst du die Autoren. Bei diesem Projekt besteht ein Mail aus zwei Text-Dateien. (Siehe zipfelchappe/emails).
Man kann später schon auf diese Methode umstellen, es sollte aber alles einheitlich sein.


Done


@gborri Also, you're right about not needing to check if its transmission, as you have a default
`def remove_torrent_downloaded(self,hash)` in generic.py which just returns true, and that is properly done.


Consider to use more simple solution like `/^to_[a-z]{3}$/.match(name).present?`


oh how I hate this "..." suffixes to indicate that there is additional functionality behind. We really need to find a better way for this kind of stuff throughout the core


Yes or How can I just hack on elm-compiler without care about other components? I don't want to build the whole elm-platform.


Okay, I have added a disable cache in the cleanup section.


Yeah I agree with you. Yggdrasil didn't turn out as we expected.
Had a lot of chaos (yes a lot of it was brought on ourselves).

I have another server in testing which has a bot create by CreaturePhil, TalkTakesTime and Stevoduhhero. Guess I could implement that into this project?

If I leave the file redirecting to you since you know what you mean XD

How are things on Sora? :)


I actually called it Call, because its blocking without wx.


When I first wrote file_reader I didn't know about the new api just the FS_ routines.
I wish we had a list of deprecated classes with what the replacement is.
Even if we don't mark things deprecated a list is essential!
Deleted file_reader.


Done.


Jaume, no sé si és intencionat però has llevat el text per idnicar el "fitxer analitzat".


This is not appropriate: there is no feedback to the user while the indicators are loading, and if the request fails, the UI will be left in a broken state with no way forward except refreshing the page. What you need is a Loader / Proxy so that the indicator nodes are loaded as the forms are expanded. See IndicatorTreePanel for one example.


Agreed. Delete. Hangover from a previous idea that I didn't go with.


@nelanelanela Update version to 1.0.14 please. Because [Carthage determines which versions of your framework are available by searching through the tags published on the repository](https://github.com/Carthage/Carthage#tag-stable-releases).


requested is the accumulated demand - so we can roll over Long.MAX_VALUE


Pain be gone - a3fb2c002c1b8a0b815d8d8ae1e8e08b0cf6d3f3


你的提交记录太有趣了^_^。。。
防止冲突出现可以尽量在每次又该前先同步然后修改，修改好后就上传，不过目前你上传了但是没有提交（Pull requests）。
还有文件与文件之间并没有关系，可以以文件为单位分开上传，这样至少保证了没有冲突的文件可以正常上传。
最后就是看到你翻译了那么多请尽快Pull requests让我合并到主线，避免造成和别人的冲突。


See earlier notes about deprecation. Have removed all va calls now in this file.


#21 Updated README.md.


Is this dependent on the ordering of the clauses generating during `$and` expansion? Iirc, the text clause should _always_ be after the simple clause because we handle the simple clauses first?


#21 Removed commandline variables and updated configurations.


你这里漏了“Mode = 3”


言いたかったのはscrollではなくthrottle


Abgesehen davon; diese Methode ist nicht so toll. Das Subject muss auf die erste Zeile, auch wenn es noch `{% load %}` Tags braucht. Die Implementation hier vorziehen:

https://github.com/feinheit/feincms-in-a-box/blob/master/fbox/%24PROJECT_NAME/tools/mail.py


I really hope it passes. Gains is one of those few useless things that somehow managed to survive the simplification purge.


I sense a disturbance in the force with this commit!


这些条目不要乱碰，危险啊！！！！


I mean, we don't have to. We could even move it to its own header file.


lgtm


Nice


Didn't know ee3's mod id was the same as qLib :P


I like to have a constant in my web service class to directly see the endpoint. It's true it could be private and add another private constant in my test with the path. I agree it's not worth it (add a test to test the path)


Yes, you are correct in that aspect.
This was to explicitly force it, being extra cautious :). As per my knowledge, it does not copy the code inside anymore, just applies some optimized call stack. I will check the disassembly and let you know, if I need any help in reading the matrix. ;)


Was supposed to go and find out what the mod id was just ran out of time
;-) my bad
On 15 May 2015 10:14, "Modmuss50" <notifications@github.com> wrote:

> Didn't know ee3's mod id was the same as qLib :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TechReborn/TechReborn/commit/55a3acff14c32fc359b11e9ded2c079b0b4164ea#commitcomment-11210537>
> .
>


Done.


What happen when API will be crash ?


Good observation. Fixed.


创建Create pull request提交把，不然我要替你创建了^_^


How you raise exception when user attempts a conversion before the rates are fetched ?


A lot of duplication here not sure if you could(or should) include a common.yaml to import all the shared stuff e.g.  service ports


Please add tests for those two things.


It generally would. However, the media type has undergone quite significant changes so that the current implementation doesn't really match the current spec anymore. Before we publicly advertise it even more we need to upgrade to the current spec version.


You somehow managed to modify the modes of many files here. You set it from 644 to 755, which is not desired. Have you this for a specific purpose or by accident?


Yeah, that's all right. I totally keep forgetting diamonds were 7 and not 8, because I was introduced to them with 8. (They are 7 right?)


this should be `syncedObjects = syncedObjects.concat(obj.change.docs)` like in pull & push


an* object


Well we really only did this not to duplicate logic in the other drivers.  I will mention it would also let you accomplish a throttled forEach / into.


Und dann kann auch das `render_mail` Geschwurbel [hier](https://github.com/feinheit/zipfelchappe/blob/master/zipfelchappe/emails.py) entfernt werden ;-)


You can use `default` section to avoid duplication.


Shouldn't the id be in the message.receiver already?


This commit should probably directly followed https://github.com/alphagov/tsuru-ansible/commit/2b481cebd8fd6a82700545ebbdbdf323b70365d7


Our host naming scheme within terraform is inconsistent between GCE and AWS... on GCE multi-host names are suffixed with an instance count number, but not on AWS where we are allowed multiple VMs with the same tag Name. I would ideally have liked to use '-*' on all hosts, but that means we need to tweak our Terraform AWS deployment. For now, I kept wildcards to minimum for now to try to avoid unexpected matches.


I do not need this, Because it is being set and used in the main CMake project file of dualprocshm, I unset it to make sure, it does not produce wrong configuration.


resolves #54


啊啊啊啊


I think you should also use `cp` command, like this: `cp config/secrets.yml.sample config/secrets.yml`


Yeah, [here](http://docs.oracle.com/javase/7/docs/technotes/guides/language/type-inference-generic-instance-creation.html) is information about it, and the URL includes `/7/` (meaning it's for Java 7) and it mentions "Java SE 7" in the page in a few places.


@wchyumo2009  仅自己可见啊。。


Please add ChangeLog file


Okay, I will remove these.  makes sense.


古古怪怪


Habe ich nicht :). Ich werde dann mal mit ADTs und pattern matching anfangen,


This has nothing to do with contributing docs!


Similar issue. The option is still there in the toolchain file. But it is disabled from the board settings.cmake.


Not done because this message isn't usually indicative of something being wrong so I didn't use warning. If your changing files you expect differences. It's only a warning if you weren't changing files. There is no way to know the users intention.
It would be nice to have a HIGHLIGHT flag to focus things that are more important but not indicative of any warning or error.


Why not to modify .gitignore to eliminate possibility of pushing secrets to github?


Typo in “output”


Yup, sorry, but as we're just merging stuff to get it done...


Поправь отступы


и здесь


you are releasing this dict (or rather, a copy of it....?) into the wild in the return statement.


Millions of ansible runs which cried out in error were suddenly silenced by switching to the tag_Name format.


发噶人


功夫


És que ho estic provant en diferents contextos. Ho deixo aquí com estava, amb el nom del fitxer, perquè pugui ser codi comú.


Done for the shaderpath_enabled variable that Amanieu asked for. Could not get the shaderpath variable to work with cvar. I think Cvar does not appear to support init type variables or I was doing it wrong. So used the existing methods.


globals.yml is reinstated for truly 'global' values in a later commit.


meh


how about you calculate total_count upfront or even pass it as parameter into the function, and then just do this in one step
```
for answer_counter in answer_counters:
        distribution[answer_counter.answer] = float(answer_counter.count) / total_count * 100.0
```


Please don't add local stuff like this. You can easily ignore it on your machine for all repos by creating a global gitignore as follows:

```bash
git config --global --add core.excludesfile ~/.gitignore_global
echo '/nbproject/' >> ~/.gitignore_global
```


Done.


This commit has broken a repo of mine.

"Fatal error: Class 'FPDF' not found…"

Any ideas?


一


please remove the comments. also, i think the `if value` is not needed here, or is it?


Thanks for the info.


no, MaxLuck is the max your weapon can be lucked or cursed in general - however this applies to bene luck only - which on official is max 7 (yet max luck / curse is still 10)


@andres-montanez  please release it .. just discovered your project and have to patch manually :)


Volgens mij moet deze in je `DATABASES` array: https://docs.djangoproject.com/en/1.8/ref/settings/#atomic-requests


The CFG_HOSTINTERFACE variable is used while building stack libraries, to check whether it is host-intarface or dualprocshm.


This is actually how it was working the entire time.


C'est moche, tu compiles dans le dossier src ? J'ai enlevé tous les fichiers .class


Would probably move the "(" to the next line, also not sure about indentation.


Я не заливал.
Потому что это пока разбирательства.

2015-05-15 2:23 GMT+03:00 Alex Ozerin <notifications@github.com>:

> А код то где? :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fogside/the-pipe/commit/be24240dc81443b3d74c15913489bf60276e3b4c#commitcomment-11205413>
> .
>


why this change? at least the template stuff like inserting the user's name and iterating over some courses should stay here, shouldn't they?


One of our dependencies dropped off packagist; I've made a fix, just need
to push it
On 15 May 2015 10:32, "James Furey" <notifications@github.com> wrote:

> This commit has broken a repo of mine.
>
> "Fatal error: Class 'FPDF' not found…"
>
> Any ideas?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/M1ke/fpdi-ext/commit/30c7c1c5d360cb40405596baef046f5b223dedbf#commitcomment-11210789>
> .
>


Thanks


I wanted to do it from ages :D


No need to remove comments over setters.The getter comments look good.


we could just assume that every participant votes and display the vote count as participant count, what do you think?


Just a though, not sure if suggestion would make it better.
``` (self.properties.game not in self.nif_export.texturehelper.USED_EXTRA_SHADER_TEXTURES)``` could be pushed onto the next line as its x and (longer complex bit).


Yes it is always USB-1 when I dont have any other development board connected. HPS UART uses one USB too.


Done.


We should think about offering constructors for the PointType that takes Java primitive types directly...


Would one line the string.


Oups... The query param is not `allow-tour` as stated in the commit message but `enable-tour`.


Shouldn't this be a CONFIGURATION_ERROR?


The last 3 could be combined.


there were quite a few reasons tbh. as poison doesn't take over expOwner automatically if someone else has owner - the mob gets poisoned then the owner runs away, the poison won't do damage until the owner times out.

because poison now saves on logout (and doesn't clear) but the Exp owner clears - if you leave this in and the caster logs out and in, the poison stays but cannot ever do any damage again (as the poison owner cannot save).

also i came the conclusion that you shouldn't really be stopping giving damage in code because a poison is too long, instead you should be balancing the poison length so it's not an issue.


This was added from the PCIE module configuration that are being done with NIOS. This differentiates between using PCIe interface or internal bus between host and PCP for the dualprocshm interface.


We can't leak ShaderExceptions exceptions? They aren't on the heap.


Done


関数名が `SGD` となってますが、この関数がやってることは行列をシャッフルして分割してるだけなので、関数名が合ってないように思います。SGDは最適化アルゴリズムの名前です。


Do you mean only the semihosted files or all linker files?


file() is already relative to the project directory
See [Project.file](http://gradle.org/docs/current/dsl/org.gradle.api.Project.html#org.gradle.api.Project:file%28java.lang.Object%29)


mao mana proper structure gd, haha
domain/domainName/appName/moduleName/layers of the module(controller, service, dao, etc)/name of layer
com.cebedo.pmsys.systemuser.controller.UserController
unsa man diay?hahaha


Ah, okay.

Any eta on your push?

And the possibility of a tagged branch version?

:)


you're right I've missed that


@wchyumo2009  额……那个sql文件夹里有最新的数据库……


doesn't has_enough_questionnaires return True for single results?


shouldn't the complete th and the td below be inside the if?


@lfzark 怎么覆盖？以前不是有数据库了，再也导入不进去了！！- -


Here you rename `__nonzero__` to `__bool__`. Note that we already have `__bool__` and that `__nonzero__` needs to stay, because we still support Python 2.


this will be deleted  by mistake


It would be simpler to have the semihosted and unhosted files in the same directory, as they are differentiated by names and both can be used from ds5. Only the semihosted permits full memmory view debug.


I'm not really bothered about the speed yet as this is just a debug tool. The performance does not appear to be causing any problems. The files are small. If I get time or this becomes a problem I will look at it then.


Ran 'rubocop -a' which autocorrects common errors like spacing and using double quotes instead of single ones and deprecated method names etc.


This double `__bool__` was introduced in this same PR.


@xuwei-k ご指摘ありがとうございます。
https://github.com/lambdasawa/scala-quiz/commit/e0305b55d6eb5e1d0d4c3e65d006a1672ffa70d2 この点に関してはこのコミットで修正しました。
https://gist.github.com/lambdasawa/88aba95f201e71756962 簡単な物ですが、計測はこのように行いました。


This should remain http://nuget.localtest.me


Both files can be used from ds5. In the semihosted, the separate uart wont be used and ds5 would provide some memory debug views.


ok will do. Question: when all feedback is addressed and PR is ready to be merged, will you want changes like this one to be squashed? If so, I assume it is fine to just append commits now?


Actually scratch that, I think it's ok with this name


maybe some comments here to make things clearer?


I really think we should move flush_handlers before 'wait for leader', because if restart occurs we need to wait for leader election


This needs to be in dependencies (too) for phantomJS / your addon to properly work with ES6


Sorry, I guess I wasn't clear.  I think the official correct style is.

```
case "$1" in
        start)
                do_start
                ;;
       <etc>
       *)
esac
```


did you use this Toast for testing purposes? if so that's a great idea.


Commenting this line removes the possibility to change the language immediately after saving the user profile. Instead, you have to logout from the platform now to get the new language. I'll try to fix that with a little change to the global $_user


Yeah probably, thanks.


Hi! :smile:


I think we only take the first EQ condition for a column and discard the followings. a = 1 and a =3 won't returns anything. Good catch though, I can create a test to see what's going on here.


why do we need the `rescue` on the other cuke but not this one?


oh actually this commit message explains the `rescue` I think.


Nah it's even more crazy.

I'm building the actual packages using web2exe on ubuntu (via Wine :D )
Then I spin up a mac VM, since it's not possible to build a proper .dmg
file on any other platform (yay for virtualisation)

No windows neccesary anymore, just for testing uTorrent (but I can do that
one of the laptops as well)

On Sat, May 16, 2015 at 1:57 AM, garfield69 <notifications@github.com>
wrote:

> \o/ you are starting to develop a farm. What you got, a windows machine, a
> linux one, now a mac, all building supporting platform specific versions of
> DuckieTV? ;-D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SchizoDuckie/DuckieTV/commit/8daf6a1743580d1bd0037056ffb8f7f302a15e58#commitcomment-11223874>
> .
>


overall things are well-organized not too confusing. awesomesauce :+1:


I think this can be formatted in a clearer way. Currently it's very hard to read this many boolean clauses in a row, especially with the negations


Looks good.


yea, so maybe it would be better to keep data in the values reference folder


Turns out my previous fix made the child loop exit immediately no matter what, which causes infinite zombie child processes and breaks mutex locking.

@isegal why does this loop allow a break? I have no idea what this error is that it's checking for, it's not documented anywhere on the internet.


@JohanTan


npm please ;D


I've added a fixup commit under the assumption that we'll squash later after more feedback.


The `restart consul` and `reload consul` handlers both notify `wait for consul to listen`. So when flush_handlers is complete the leader will have been elected.


Looked for references in golang standard packages. Found different behaviors.
But I also found in net pkg the method func InterfaceByName(name string) (*Interface, error), which is the closest to EndpointByName() and NetworkByName().
Here they follow the convention @cpuguy83 and @LK4D4 were suggesting, that is returning a custom error for the case of not found.

Will change libnetwork methods accordingly.
Thanks.


I was going to use this when the user chose the wrong button but it stayed in the screen for too long so I deleted it lol


really? I'm confused with my code haha. Thanks! I will try to make it clearer! I should add more comments


Can you make the style here match the rest of the functions in utils? Specifically instead of `ARGUMENTS` you should have `Parameters` (i.e. see the `run_antechamber` function.). Also, to format this like that, you would have:

```
    Parameters
    ----------------
    top_filename : str
          Topology path/filename
```
etc. (In other words, change how these are formatted, and specify what data type they are supposed to be - string, integer, float, etc.). Also `LIMITATIONS` can get replaced by `NOTES`.


Does this really fix anything?


Let's remove the portion inserting this specific water model into the topology file. Does the `water_model` variable specify a water model that will also be understood here? If so we should use the same variable to fill in the appropriate include file. Otherwise we probably need the user to specify what include file they want (and you can have an optional argument for this). What you do NOT want is to have the specific force field and water model specified here, as it means that no one can use this function for another water model or force field.


Yes, see the change on line 2738/2737. The other changes are just because the dict keys should always exist, and if there is a case where they do not, this would only hide the cause of the error.

It fixes issue #275.


I don't think ensure_forcefield is a function we have here and I don't see that you've added it. If you want to call it you'll have to add it.


I added a couple line notes. Also it looks like you will need to check that the output .gro file is created and not empty (and/or parse the output of your GROMACS commands to look for errors). What we're trying to achieve is that we never have a function which runs and silently fails. In other words, if it doesn't work for any reason, it needs to tell you that it didn't work.


Is the auto invite page included?


If this happens within a transaction (as the case for our code), there is no need to manually unpersist traces of this transaction. You could just let the transaction bubble and remove all that rescuing altogether.


For the record, you can update this PR simply by making new commits/pushes to your branch.


Backward incompatible change with no documentation as to why?


removing IncludeResources namespace breaks compilation


@mbj yeah, our code has stronger constraints that "stock" spree, so I was trying to maintain code that would continue to work the same as before.

I do think we should eventually push some of our transaction ideas to spree itself, but I see that coming later.


> I do think we should eventually push some of our transaction ideas to spree itself, but I see that coming later.

I'm close to factor out a commit that adds transactions around *any* spree action, so our fix for the deadlocks can make it to upstream. Also our performance fixes depend on the concurrency semantic guarantees transactions give.


These can probably just be uploaded to the admin!


So for now: I think its fair you assume all actions are wrapped inside a transaction. I'll upstream your fix after the transaction is in spree master. I'm pretty sure I can convince them.


updated, thanks!


One suggestion to remove clutter from your Navigation Drawer, make a submenu for all the zodiacs you listed in the Navigation Drawer or find a way to navigate through the zodiac signs in a separate fragment instead.


If the field will only ever be an integer, it's preferred to set the field as such. It uses less memory in the system.


What is the rationale behind this patch, and what is its source? Is this patch applied somewhere in upstream boost?

I'm currently experimenting with upgrading the boost version installed by the ``./depends`` system, and I don't yet understand what happened to the underlying code in boost's history, nor do I yet understand the rationale for this patch.


Not now... But I've enabled the ability to set dimensions to 0... Which is the same that yours do.


You need an introduction of some sort, but not in headers.


"margin-left" and "margin-right" are deprecated since version 3.12 ;-)
where "xpad" and "ypad" are set to same value you can replace them with one "margin" attribute


[note] [/note]


For semantic clarity, you could have called your model "Car" instead of "Garage." This still works though


lol that 2 cool 4 const! :D


I used the pod to install the 3.x branch.


No need for the '>'


It gets set [in a continuation](https://github.com/sharwell/openstack.net/commit/5042a321ee4182aa11fb0bba36d4db86229754c4#diff-3d8fa280f2ae597e1b51be56a0262d8aR217) after `PrepareRequestAsyncFunc` returns.


I like the way you organized all the info in the resources/string.xml file, it keeps everything in one place in case you want to change it later. One small suggestion is to add the word "description" or "info" to the names of the zodiac strings containing their description/info. For ex: "capricorn_info" instead of just "capricorn"


test


You could also use the rails shortcut "root 'garages#index" to define the app's root or "/" path


Is note? [note][/note]


Is there no way to make local sockets the default? :frowning:


This test will never pass on Travis. It depends on analizo 1.18 which can't be installed on Ubuntu 12.04 (Travis' version). See: https://github.com/mezuro/prezento/commit/e79554cd669bf46afa87ff13cd0232e5385f51de


Shouldn't this be rfind, to support ipv6 addrs as hosts?


for i in $(ls /etc/apache2/sites-available/sites-enabled); do a2enmod $i; done;

/me walks away ;)

No, but for documentation use, this is probably fine.


Might be good to specify that it's the NIX_REMOVE env var that's to blame


I think for now I'm going to leave this code as is, assuming there is no transaction.

When this code is rebased on top of something that wraps a transaction, then it's a different matter.


build.lua 在 miktex 下都不能用，现在就把 windows 脚本删除了，岂不是 miktex 下就无法编译了？ #179


I'm fine with that.


Agreed that it seems a little arbitrary when you're thinking about moving average or convolution.

However, the "whatever is in buffer when end of stream is reached" language can be interpreted this way too:
```javascript
[1, 2, 3].slidingBuffer(3, 1) == [[1, 2, 3], [2, 3]]
```
Here's the logic. The first buffer ends at `3`, so it's emitted. At the same time, a new buffer is started with `[2, 3]` in it, since `skip == 1`. Then the stream ends and the current buffer is emitted, which happens to be `[2, 3]`. I think you'd agree that this behavior makes even less sense. If we don't allow that, then we need to add extra language to clarify this situation.

Let's say we add language that says we don't start a new buffer until the *next* element is found. So in the above situation, there's no buffer started since there's no next value.

Applying this to the general case means
```javascript
[1, 2, 3, 4, 5].slidingBuffer(3, 2) == [[1, 2, 3], [3, 4, 5]]
[1, 2, 3, 4, 5, 6].slidingBuffer(3, 2) == [[1, 2, 3], [3, 4, 5], [5, 6]]
```
This seems weird to me. Are we not allowed to emit a single-element array when `n > 1`? Why not?

Now, consider the logic from RxScala. You start a new buffer every *skip* elements and emit one every *n* elements. A consequence is that when `skip < n`, there are *multiple buffers* all accumulating at once. So the "whatever is in buffer when end of stream is reached" could be interpreted as, "we will emit all currently accumulating buffers when the stream ends."

This behavior has a few things going for it:
1. If `slidingBuffer` returns extra elements, we can easily filter out the extra. The sliding window that you want could be `slidingBuffer(n, 1).filter(function (xs) { return xs.length === n; })`. It's harder to put in extra elements.
2. This is how RxScala implements their `slidingBuffer`, and if we're going to provide a similar transform, I don't think we should change the semantics unless there's a very good reason.
3. A view with multiple concurrently accumulating buffers is easier to communicate, in my opinion, than the gymnastic we have to go through to explain the current behavior. Obviously, there's no need to actually implement the transform this way and waste memory, but it is easier to explain.


Also, "the old init process manager" ... Eh, it's not "old". And I'd refer to it as the "init daemon"


What am I supposed to do? Make a troubleshoot section ;) Oh, or if you have one, make steps <3


ss -pau? ;)


With ai_family set to unspec, why do we need V4MAPPED? From POSIX:

> The AI_V4MAPPED flag shall be ignored unless ai_family equals AF_INET6.


Check out the link_to helper: http://apidock.com/rails/ActionView/Helpers/UrlHelper/link_to


uh, your indenting is weird. Fix it <3


Thanks! Wonderful!


https://, please!


Make sure you read the documentation for resource references. This still isn't quite right.


The ruby standard indentation is 2 spaces. You can change the default in your sublime text editor.


That's a long header. Maybe not doing headers would be betters.


Why not just return here and unconditionally throw the error outside the for loop? And why the double assignment to fdSocket?


It would be great for this to be pushed to npm


$ ./codespell
  File "./codespell", line 41
    'data', 'dictionary.txt')
    ^
IndentationError: unexpected indent


Of course not.


Consider changing this route to "dogs/:id". That is a standard format you'll see on many sites. It allows you to scale up your application (For instance, what if you wanted to add a cats controller?)

This change could also be applied to your "new" route


A+. Would add.


very nice and wish have more comments :)


Told you! :wink:


Done!


This has been removed in latest commit. Session HiveConf is used instead to pass compression configurations to RowSet.


This is fine since the latest official Hive also exposes the SessionManager publicly. Seems this piece of HiveServer2 code is not really stably designed yet and is still under change.


Arquivo obrigatório para o login do google, NÃO ALTERAR!


By the way, I realize that this behavior isn't exactly what you wanted, so we could also add a `slidingWindow` transform that means one of the two.
```javascript
// Emit only size n buffers.
[1, 2, 3].slidingWindow(3) == [[1, 2, 3]]
[1, 2, 3].slidingWindow(4) == []
// OR
// Sliding behavior at the front too.
[1, 2, 3].slidingWIndow(3) == [[1], [1, 2], [1, 2, 3], [2, 3],[3]]
```

Both would be very easy to implement in terms of `slidingBuffer`. I would lean towards the latter.


Sorry for my stupidity. Of course a MetricFu test has nothing to do with Analizo's version and the commit refers to a different test.


Hi Achim,

If you try

UG_DrawLine(0,10,20,0,C_BLUE);

which is an upward slope line, your code will keep the X coordinates and swap the Y coordinates.

What the code then actually draws is:

UG_DrawLine(0,0,20,10,C_BLUE);

which is a downward slope line and different from what was intended,

Regards,

Marco


3 concerns, in increasing severity:

1. There is no way to start up an inet server without socket activation
2. The security model of "control access to the daemon by controlling access to the socket" *can* be translated to the inet domain (with firewalls), but it seems risky. I'd prefer to see either a) some explicit acknowledgement required by the user before the daemon will listen on inet (e.g. through some nix config option `i-have-a-firewall-setup-i-swear`) or b) application-level auth. B seems much preferable to me.
3. No encryption over the wire.


This all seems like too much boilerplate.  Let's see if we can't simplify.


These came from later commits in upstream boost. If you're looking at bumping to 1.58, you can probably just drop them.


Can you change this to !lhs.getDataType().isCoercibleTo(PChar.INSTANCE)


Can you change this to !rhs.getDataType().isCoercibleTo(PChar.INSTANCE)


suggested extension: Koop is an entirely open source ETL Engine for APIs that makes it easy to combine data from different sources into common formats.


In the index function defaulting the user_id to None will make the API not work.  See the docs https://developer.enphase.com/docs
Under the heading "The API Key and User ID"


Checked in code developed in 2014/12, rather experimental.  Passes basic tests, but not tested very much.  Not being actively pursued.


Just would like to point out that the case has changed for these object names, and is no longer backwards compatible. I also wanted to make sure it was intentional and it won't be changed back?


:octocat: :100:


Did you notice, too, that some ``help="stuff"`` params start with capitals, and some don't. And some sentences end with full-stop characters, while others don't

I'm just thinking out loud here... Given the number of sub-commands, and command options this program has, maybe it's worth thinking about a language/style guide for this kind of thing.

I'm sure there is prior art/inspiration for this kind of idea somewhere.


Oh yeah. It's a mess. Compartmentalization coming in a little bit. I just ripped all the commands out.


As a user, I prefer to enable the "Show pointer" option and leave it on. I do so to aid in pointing out topic of discussion in screenshots as well as to indicate that the device is actually working if the screen is not otherwise responding.

I also use the screenrecord functionality and find it bothersome to have to manually re-enable the "Show pointer" option after I use it. I feel it would be much better if the original setting was remembered and restored when the screen recording is finished. That is what this change accomplishes.

Thank you for your consideration.
Paul M Edwards


@tbielawa Also, style guide. Yes.


Done in https://github.com/dkorolev/Sherlock/commit/0ed07c1c4d009fcdfd111970ec9e4dd55f40772f


(C:


This actually would be redundant: `copy_free` is always a const reference for a `struct` or `class`, and `T_ENTRY` is always one.


Thanks again @mzhurovich -- made all the changed in https://github.com/dkorolev/Sherlock/commit/0ed07c1c4d009fcdfd111970ec9e4dd55f40772f


@petterroea It should actually be $database not $con. So change all occurances of $con with $database instead :)


rax retry tests


I'm learning to port ROM for xperia sp :)