If the descriptor is being canceled, that means it's being removed from the polling queue entirely. Unless the thread also has a timeout, it could sit there forever. See also my comment re POLLPRI, above.


Btw, since S6 doesn't do hotplugging, when is this update_nr_running_count() called?


lolz <=


Well... I'm not exactly sure what the performance problem is, but I *do know* that:

1. `LockWindowUpdate` is wrong, ( ["It's not surprising that naughty gives the best results for the person being naughty[...] Of course, you also screw over everybody else"](http://blogs.msdn.com/b/oldnewthing/archive/2011/01/24/10119211.aspx#10119636)) so it doesn't really need to be faster to be changed.
2. I haven't seen the profile of the performance problem, so I can't expect to know why it isn't faster - but I do know that `LockWindowUpdate` doesn't prevent the actual redraw work, it just forces a failure. [`WM_SETREDRAW` has internal optimizations to reduce the amount of work that's actually done](http://blogs.msdn.com/b/oldnewthing/archive/2011/01/24/10119211.aspx).

Related: [Using WM_SETREDRAW to speed up adding a lot of elements to a control](http://blogs.msdn.com/b/oldnewthing/archive/2014/04/07/10514610.aspx)


When the device's screen is off it hotplugs the cores and starts this thing to turn them back on when there's some load that warrants it. You can monitor this via adb for example.


Looks good!


991757360-DocIssues-4956-4997 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (744 of 744 tests passed)

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


97903100_SolarCollectors (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


I can't remember what I was thinking. Maybe I was confusing POLLPRI with POLLHUP, or maybe I was worried about issues with OOB data causing a busy loop.


IIUC `this.notifyCurrentValue(“dir”)` was for tricking `computeProperties()`/`refershRendering()` to make them think that `dir` is changed. Given the new `initializeRendering()` code runs `computeProperties()`/`refreshRendering()` with the widget instance as the argument, it has the same effect (`”dir" in oldValues` will be `true` there). Does it address your concern?


That's what I've been wondering with 7420.
It keeps hotplugging cores on and off continuously on screen-off.
I'd rather keep just 1 A53 core on with A57 cluster completely off, as I see no reason why to use hotplugging on screen-off.

Ohh how the table has turned, we used to do the exact opposite lol.


Another reason Github should support and verify GPG signed commits and tags, and permit repos to reject unsigned commits.


It's worth to hotplug in stuff to be able to get faster to sleep / race-to-idle.

In any case this patch improves the behaviour a lot as it'll only plug in when needed.


:-1: The code is not python3 compatible. python2 was put there explicitly for that.


Me too, se for example the abuse of `only` into OFF :-)

These tests has been too much quickly...


looks good! I'm happy to accept this. the naming conventions make sense and I'm able to figure out the intent of the names now.


I think it'll be better to go suspend a bit later and *not* hotplug like crazy on screen-off.
Mine hotplugs something like every 3 seconds even the CPU usage is less than 3%.


You'll want this `return this.addBowerPackageToProject('eonasdan-bootstrap-datetimepicker', '~4.14.30')`


you'll want all this as well


PlantOperationSchemes (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2210 of 2232 tests passed)

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


@mayaijung  can we please combine both .mq-sup selectors as bother are in the same scope ie the one at line 353


:tada: :+1:


Hmm.  `with urlopen(...)` is broken on Python 2 as urlopen doesnt return a context manager:
```
AttributeError: addinfourl instance has no attribute '__exit__'
```


Now let me buy a ticket. ;-)


相当于 strings.ToUpper，不过 strings 包针对 unicode  以及会产生新对象。

所以这么写的目的是仅使用栈对象 & 只针对 byte 处理，没有 benchmark 过，不过应该比 strings 快。


Should this be defined contiguously with the Placement class (just a few lines higher in this file)?


You are completely right. Today (I hope) I'll try to provide a complrte set of interfaces to test a more strictly factory pattern.

I have one question: if I understand right Rouson's examples the factory object (that is concrete not abstract) has a method returning a polymorphic object (in our case the abstrac `wenointerpolator`) that is *then* instantiated with a concrete object. The `create` method that I used for *initializing* the concrete object should on the contrary **returns** an inizialized concrete object. Something like

```fortran
type factory
  contains
     procedure :: create ! => should return a concrete extension of class(weno_interpolator)
end type
```

Is this right?


i think you need to put this file to ug, otherwise you'll need to use import-from-derivation and it will haunt you :)


I'm guessing that you're going to make this take a Player.t argument next in order to handle the silver setup?

Also, as we've sketched out in the API draft, we're going to want support for handicap games or even EEE-style setups. Can you add in a simple boolean flag (maybe "allowPartialSetup: Boolean") to allow the caller to disable the length = 16 check?


I had this patch locally, then I did "work on my computer".


What's the use case? At this point I'm OK with building a static binary and using it on my machine.


doing `float()` after the integer division doesn't fix the integer division problem
try it out:
```python
1/2 # =0
float(1/2) # =0.0
float(1)/2 # =0.5
1.0/2 # =0.5
```


Double check how you deal with division in Python.
Remember that Python does floor division with integers.
Check this out:
```python
1/2 # =0
```
but if you convert one of the integers to a float, then it will act as expected:

```python
float(1)/2 # =0.5
1.0/2 # =0.5
```

hint: this is the correct answer if the run took 43 minutes exactly.


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Yes, thank you. I'm trying to use the FQNS for built-in functions where possible.


@notspiff, could you help with this cmake depend? my netplay add-on uses protobuf messages, but I'm completely stuck trying to get the add-on to compile on osx and windows

On linux the protobuf messages compile using the system protoc compiler, so everything's good there.

On windows, protobuf v2.6 has a VS project, protobuf v3.0 has a cmake system. I'm not sure how to use cmake to build a VS project, so I tried upgrading to v3.0 and using the cmake system. It looks like it fails when it tries to invoke autoreconf? http://pastebin.com/uzcMMBCA

On OSX, v2.6 successfully compiles the messages but fails to link against libstdc++: http://pastebin.com/Kpf6my0E. Stack Overflow suggest adding list(APPEND DEPLIBS stdc++) (doesn't work) or setting the compiler from gcc to g++ (how is this done?)

thanks


Tried game Stolen and it seems to work ok even with 7. White screen and flashing is gone and game seems to be playable now.

Here is a windows sse2 build if anyone wants to test(try harder =9).
http://www.filedropper.com/gsdx-sw-memflush


This line here is an excellent start.


This doesn't look like a rename...


Permitting cqueue:cancel to accept a list of file descriptors was a poor API choice. It should have only accepted a single descriptor or object. The reason being that it's really messy (and currently not implemented well) to return information about multiple errors.

More importantly, cqueue:cancel needs to be relatively bullet-proof because it's necessary for maintaining the integrity of the internal polling state. If the application closes the descriptor while the controller still has events installed, bad things happen and there's no reliable, race-free way to recover other than to discard the entire controller state. That's why the current code keeps chugging along on an error; it tries to cancel everything it can.

Also, the only failure mode for fileno_signal is when delayed initialization of the alert descriptor is enabled. Delayed initialization violates RAII, multiplying the number of failure paths :(

Ultimately cqueue_cancelfd should be changed to accept a stackinfo argument so it can generate more useful error information. Or, preferably, change the implementation to enforce passing only a single object/descriptor; or maybe add a separate _cancelfds to deal with the case of multiple fds and multiple error conditions.


Maybe, but it doesn't seem essential. The build is rather simple, and I don't want to complicate it with _too_ much abstraction.


You probably have something odd running in the background, mine isn't doing anything.

Play with the threshold if you want ```echo 500 2 > /sys/power/hotplug_nr_running```.


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I'm wondering if we should wrap and tag parsing errors so that, for example, we know a piece parsing error occurs as part of a placement parsing error via the exception message. I'll consider doing this in a cleanup pass later. For now, don't worry about it yourself. At least in local testing, it's not a problem since we have a stack trace, but it would be useful for other cases (such as if we want to use the 'apply' version of the parsing for user code).


whoops. sorry.


А почему RATING_DRIVER_DB хранится как строка, да еще и цифра, а не какое-то вменяемое название?


IOException from getFileSystem.delete... this is a test class so I think we can be aggressive here


А что мы будем делать, если у нас redis упадет? Мы потеряем все плюсики?


Thanks for the review—I appreciate it. Still helps to have another set of eyes on the API and docs. I've fleshed out the doc.go example so that it uses a template and the requisite tag to make it clearer.

Going to push to master now.


why?


the use case is using upcast from ug sdk; we put all other packages there anyway


I can't for the life of me figure out what I'm doing wrong here. Please have a look and advise.
<img width="1757" alt="screen shot 2015-08-05 at 12 23 32 am" src="https://cloud.githubusercontent.com/assets/11021075/9078194/5ae20746-3b08-11e5-92ab-a8a5fbbab969.png">


If I may ask, what is the purpose of this commit?


Can you add newlines at the end of your new files?


Just as a note, not anything that needs to change here:

For convenience, I'm happy with allowing functions of the form apply(s: String) to raise exceptions. However, is it possible we can limit it to that?

One invariant in the that the coding conventions that we use at work is that unless you're communicating with the outside world in an obvious way (parsing, network connections, IO, spawning processes, making other kinds of system calls, etc.), functions should never throw exceptions unless explicitly documented to. The reason this invariant of the codebase is useful is because it limits the amount of nonlocal control flow. Barring errors that you usually can't do anything about at runtime (out-of-memory), you know that your code will never unexpectedly jump somewhere, making it easy to reason about correctness and integrity when you have mutable state.

In Java, a convention like this is a bit impractical. But in Scala, it seems that there is enough support in the standard library and in the type system (via Try[T] and Option[T]), that this is actually possible. Searching online also finds a possible plugin might work with Scala to force exceptions to be checked rather than unchecked. Any thoughts?


This should be `mInflater`. Does it really make a performance difference?


@dlenski

FWIW, this is change is failing for me on CentOS 7 with the following error:

    -bash: /home/hatzopoulos/bin/wtf.py: python2: bad interpreter: No such file or directory`

more info ...

```bash
$ type -P python python2
/bin/python
/bin/python2

$ whereis python python2
python: /usr/bin/python2.7 /usr/bin/python /usr/lib/python2.7 /usr/lib64/python2.7 /usr/include/python2.7 /usr/share/man/man1/python.1.gz
python2: /usr/bin/python2.7 /usr/bin/python2 /usr/lib/python2.7 /usr/lib64/python2.7 /usr/include/python2.7 /usr/share/man/man1/python2.1.gz
```


The Protect function signature unnecessarily complex.  I suggest changing it to:

    func Protect(h http.Hander, authKey []byte, opts ...func(*csrf)) http.Handler {
           cs := parseOptions(h, opts...)
           ... same code as before
           return *cs
   }

It's not common to use value methods on large values.  I suggest changing ServeHTTP to:

     func(cs *csrf) ServeHTTP(w http.ResponseWriter, r *http.Request) {

You will need to make a compensating change at the end of the Protect function.

This code clears the Gorilla context.  Is that safe to do?

I suggesting using conventional Go names for things instead of adding comments explaining the unconventional names.


To login, all a user needs to do is go to /auth/provider and they will get redirected to Sessions Controller create method. Does that make sense?


Did you mean to change this message to something else?


This does not work for me. The slider doesn't scroll at all when an item is tapped, and the selected indicator is not activated for the selected item.


:+1:  Thanks for keeping so on top of this!


991757360-DocIssues-4956-4997 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


300, 400 とグループがある場合の性能が気になりますが、とりあえずは LGTM。


what if you may not


@bcomnes, you are welcome.


I will die of confusion.
I'm trying to get oot3dhax to build, saw your commit and tried applying it. It just broke.


can you be more specific about how it broke ? wont compile ? what's the error ?


asdasd


It's complaining that the immediate value of 268435452 is "out of range" and "too large for field of 2 bytes at 3586"


ESLint is complaining about multiple spaces here...


I was originally trying to find a way for the oot3d save program to stop finding the crc16 invalid


k; good to remove this comment then?


oh, woops.

can you try the new commit ? should work fine.


У редиса базы по номерам. И нужен именно инт.
В настройках строка, чтобы не путаются при переключении драйвера.


Surely,

here are the slide from a workshop

http://fr.slideshare.net/linaroorg/using-gcc-autovectorizer


No, still complaining. For some reason it's thinking the value being moved into r1 should be 2 bytes, which is absurd.


I am a nice comment


Редис периодически сохраняет свой дамп. Вероятность потерять часть данных тем меньше, чем чаще идёт сброс дампа. В конфиге все настраивается.


Todo: think about this one


can i see the error output


If I change my shebang line to use `#!/usr/bin/env python2 -u` I get a different error: `/usr/bin/env: python2 -u: Permission denied`. And when I searched I found it may have to do with a **one** argument limit with `env` on some systems and it being used up with `python2`. See http://stackoverflow.com/a/16549517/881551

So I changed mine to `#!/bin/python2 -u` and that seems to work.

Although I was hoping there would be another way that wouldn't involve a hard path to the python executable/symlink.

In my case I do not have root on the systems where this is going to be installed, to make high level configurations.


bumped priority of https://github.com/openshift/origin/issues/1870


wow this is really great


I was playing around a little at the bottom there oops


@dead2k is relativizing here what you would expect?


@deads2k and here


hiddenColumns will be better :)


you clearly dont have the latest version of the code...
https://github.com/smealum/oot3dhax/commit/1cdaf49581f9f99bf1c6bc79ca7725dc936f2ca3


use English


Oh god I'm so sorry, I forgot to hit the save button. It's still failing, but instead of saying 268435452 it says 268435456.


Thanks


97903100_SolarCollectors (macumber) - i386-Windows-7-VisualStudio-12: OK (2211 of 2241 tests passed)

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


asario


![1435859812935 51](https://cloud.githubusercontent.com/assets/13653968/9078430/3eb8c236-3b0c-11e5-93a0-943ff0919620.jpg)


https://github.com/smealum/oot3dhax/pull/1/files#diff-451bdfc57cc788fd9f2d9be239f9c6fb

On Tue, Aug 4, 2015 at 9:51 PM, Ian McWhorter <notifications@github.com>
wrote:

> Oh god I'm so sorry, I forgot to hit the save button. It's still failing,
> but instead of saying 268435452 it says 268435456.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/smealum/oot3dhax/commit/f4e2d05b7ccafed357c294b29bd57356ff8e3fd8#commitcomment-12532207>
> .
>


agario


Semicolons please );


It is also using wrong CounterType here.


@keithbostic, please take a look -- I'm happy for this to move or to address this in another way, but I had the patch so figured it was simplest to push it.


What FPGA is that for, Spartan-6?


man, what's about the 0.2 release's changelog?


argh


Ok that got it to build.


Note that the `Protect` function is the way it is to allow for use with middleware chaining libraries and to simply wrap other `http.Handler` types. The `func(http.Handler) http.Handler` pattern is fairly common across Go HTTP applications.

If you change it to `func Protect(h http.Handler, authKey []byte, opts ...func(*csrf)) http.Handler` then you cannot (easily) use libraries like Alice and Negroni, nor simple wrappers like `func use(h http.Handler, mw ...http.Handler) http.Handler` without wrapping the middleware in another closure.

You can still use it in the "really simple" case with `CSRF := csrf.Protect(...)` and then `CSRF(yourServeMux)` as well.

> It's not common to use value methods on large types. I suggest changing ServeHTTP to:

Yes, agreed. This should have been a pointer receiver from the start.

>  The comments will not stop people from running golint and submitting PRs.

Note that the two exceptions to golint re: `HttpOnly` are explicitly related to the http://golang.org/pkg/net/http/#Cookie field of the same name. I've elected to stick with the convention, even if that means the occasional golint related PR. If they manage to submit a PR without reading the comment on the line above, well...


Makes sense.
The differences would be that -funroll-loops is different from developers
manually unrolling loops or using -funroll-all-loops, as GCC determines
which loops to unroll.

But anyways, I do agree that Auto Vectorizer is more powerful, I totally
forgot -O3 was using it. lolz

On Wed, Aug 5, 2015 at 1:43 PM, Maxime Poulain <notifications@github.com>
wrote:

> Surely,
>
> here are the slide from a workshop
>
> http://fr.slideshare.net/linaroorg/using-gcc-autovectorizer
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2b32ac8627030749f47dd228d1bc584d84885a1c#commitcomment-12532159>
> .
>


97903100_SolarCollectors (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Looks good to me. The indentation is probably because you used tabs to indent, not spaces (naughty, naughty).


If you call the `Except` first, and then the filtering, then the `Comparer` will run on less elements.


merge branches


oops, i meant #536


In this raw output both events and nodes are represented as vertices, which can be a bit confusing. To figure out which vertices are nodes and which are events you need to look at the types ("Color" is just another word for type. The word "color" is commonly used when identifying canonical forms of graphs, which is one crucial step in identifying temporal motifs.). The additional type 1 you are seeing is the default type for events in case event types are not given in the input data. This is what the help of the main program says about this:

"The input event data has one event per line, with the first four columns corresponding to starting time, duration, and the id's of the two nodes involved. A fifth column may also be used to denote event type by an integer. When omitted, the type is assumed to be 1.”

So "[0:1 1:2 2:3] 0,2 1,0” means graph [1] -> [0] -> [2], but since we know that node 0 is an event (because it has type 1), this means a single event (of type 1) from node of type 2 to node of type 3. The last row in your example has three events and four nodes. It is generally quite hard to see what kind of motif this line represents without making a drawing, which is why the python plotting library is included also (drawing those by hand get a bit tedious after a while).

The reference to "motif in column(10)" means exactly the columns "N [node:color ...] edges ...".  The number in column N gives the total number of vertices in this graph, e.g. N[events] + N[nodes].

For a thorough discussion of the concepts and algorithms, please see sections 1, 2 and 3 in this article (should be available free of charge). Figure 2 explains the mapping of events to vertices and why it is needed: http://iopscience.iop.org/1742-5468/2011/11/P11005

Hope this helps!


Does this happen? Are partial itinerary results returned from the API?


> The public API uses unexported types. Try to avoid that.

The only two private types are the 'csrf' and `options` types - which should be private as both are modified via their associated constructors or options functions (respectively).

Making them public would make it less clear/more error prone: do you modify the struct fields directly? Or use the constructor `Protect` function? Why does `MaxAge(age int)` exist if the struct itself is public? I'd be open to making these public if there were valid uses cases for doing so, but I can't see any obvious ones.


окей) я жду))


hehe no problem, my concern was more oriented towards a possible regression due to two optimization algorithm applied on loops. I don't consider myself good enough to judge if O3 unrolling loops should specify for example no vectorization.

Btw, did you ever try bb-vectorizer flag for clang compiler ?

https://archive.fosdem.org/2014/.../AutoVectorizationLLVM.pdf

I noticed that on the master branch, android is built more and more with clang, clang binary is even at 3.6.1 (I THINK).


If you look at the response JSON and find `buildServiceWarnings` you'll get something like:
```json
{
  "status": 0,
  "age": 0,
  "message": "No such file or directory - old dSYM file cannot be overwritten:\t\told: '/Library/Developer/XcodeServer/Integrations/Caches/c40f3384eed2ea14647e2236a117d0a8/DerivedData/Build/Products/Debug-iphonesimulator/HeatChat.app.dSYM/Contents/Resources/DWARF/HeatChat'",
  "commits": [],
  "integrationID": "12993d83910012e7cb7fc57023069f3b",
  "type": "warning",
  "issueType": "Build Service Error"
}
```
As you can see, `type: "warning"`, `issueType: "BuildService Error"` and since `enum` must have unique associated values I can't do anything about this than leave it and treat `buildServiceWarnings` as regular `warnings`. Hope it's clear now?


Non-ssh urls would be prefered. Makes it a pain to checkout on systems without ssh keys registered to github


404 bro :P

On Wed, Aug 5, 2015 at 2:02 PM, Maxime Poulain <notifications@github.com>
wrote:

> hehe no problem, my concern was more oriented towards a possible
> regression due to two optimization algorithm applied on loops. I don't
> consider myself good enough to judge if O3 unrolling loops should specify
> for example no vectorization.
>
> Btw, did you ever try bb-vectorizer flag for clang compiler ?
>
> https://archive.fosdem.org/2014/.../AutoVectorizationLLVM.pdf
>
> I noticed that on the master branch, android is built more and more with
> clang, clang binary is even at 3.6.1 (I THINK).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_build/commit/2b32ac8627030749f47dd228d1bc584d84885a1c#commitcomment-12532295>
> .
>


Yeah I have update the link to the pdf!


PHP = PHP Hypertext Processor.  It's a bacronym


And BTW we're getting `commits` back! Check this out:
``json
{
"_id": "99e84a22b20df344df2217d5e413f43e",
"_rev": "3-7dfc0aa57a8119c025cd2d86143589f2",
"message": "Expected declaration",
"type": "error",
"issueType": "Swift Compiler Error",
"commits": [
  {
    "XCSCommitCommitChangeFilePaths": [
      {
        "status": 4,
        "filePath": "XcodeServerSDK/Server Entities/Bot.swift"
      },
      {
        "status": 4,
        "filePath": "XcodeServerSDK/Server Entities/Trigger.swift"
      }
    ],
    "XCSCommitMessage": "Break regular code\n",
    "XCSBlueprintRepositoryID": "A36AEFA3F9FF1F738E92F0C497C14977DCE02B97",
    "XCSCommitContributor": {
      "XCSContributorEmails": [
        "cojoj@icloud.com"
      ],
      "XCSContributorName": "cojoj",
      "XCSContributorDisplayName": "cojoj"
    },
    "XCSCommitHash": "1a37d7ce39297ad822626fc2061a1ba343aed343",
    "XCSCommitTimestamp": "2015-08-03T08:23:17.000Z"
  }
],
"target": "XcodeServerSDK - OS X",
"documentFilePath": "XcodeServerSDK/XcodeServerSDK/Server Entities/Bot.swift",
"documentLocationData": "YnBsaXN0MDDUAQIDBAUGIyRYJHZlcnNpb25YJG9iamVjdHNZJGFyY2hpdmVyVCR0b3ASAAGGoKQHCBobVSRudWxs2gkKCwwNDg8QERITFBQVFhcYGRgVW2RvY3VtZW50VVJMXxAUc3RhcnRpbmdDb2x1bW5OdW1iZXJfEBJlbmRpbmdDb2x1bW5OdW1iZXJfEBFjaGFyYWN0ZXJSYW5nZUxlbl8QEGxvY2F0aW9uRW5jb2RpbmdWJGNsYXNzXxAQZW5kaW5nTGluZU51bWJlcll0aW1lc3RhbXBfEBJzdGFydGluZ0xpbmVOdW1iZXJfEBFjaGFyYWN0ZXJSYW5nZUxvY4ACEAQQABABgAMQHIAAXxCbZmlsZTovLy9MaWJyYXJ5L0RldmVsb3Blci9YY29kZVNlcnZlci9JbnRlZ3JhdGlvbnMvQ2FjaGVzLzk5ZTg0YTIyYjIwZGYzNDRkZjIyMTdkNWU0MDNmM2U0L1NvdXJjZS9YY29kZVNlcnZlclNESy9YY29kZVNlcnZlclNESy9TZXJ2ZXIlMjBFbnRpdGllcy9Cb3Quc3dpZnTSHB0eH1okY2xhc3NuYW1lWCRjbGFzc2VzXxAXRFZUVGV4dERvY3VtZW50TG9jYXRpb26jICEiXxAXRFZUVGV4dERvY3VtZW50TG9jYXRpb25fEBNEVlREb2N1bWVudExvY2F0aW9uWE5TT2JqZWN0XxAPTlNLZXllZEFyY2hpdmVy0SUmVHJvb3SAAQAIABEAGgAjAC0AMgA3ADwAQgBXAGMAegCPAKMAtgC9ANAA2gDvAQMBBQEHAQkBCwENAQ8BEQGvAbQBvwHIAeIB5gIAAhYCHwIxAjQCOQAAAAAAAAIBAAAAAAAAACcAAAAAAAAAAAAAAAAAAAI7",
"lineNumber": 29,
"integrationID": "99e84a22b20df344df2217d5e413adf7",
"age": 0,
"status": 0
}
```
So either they forgot something or it matters for a specific case or I don't know...
![](https://encrypted-tbn3.gstatic.com/images?q=tbn:ANd9GcQSsiJE8ooOSza9DaihJxGUuAhe876972juxzaknlZPxljif5wW)


hehe


I'm +1 to keeping those private for now.

Also since setStore is not exported yet, do we need to expose the Store interface?


Does Negroni use func(http.Handler) http.Handler pattern? I don't see any use the documentation.


How do you think about to [reuse a configuration check like "AC_PROG_INSTALL"](https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Particular-Programs.html#index-AC_005fPROG_005fINSTALL-270 "Which install command will be appropriate?")?


Untested, but all corners and edges go around in the same direction (except foundations somewhat), so a switch with 4 cases seems a bit overkill. Something like below perhaps?
if (GB(slope, PATHBIT_NE + edge, 1) != 0) edge = INVALID_EDGE;


Cause of the difference it is used in OverlayRenderer.cpp and OverlayRendererGUI.cpp - but was just a question, thx for the answer.


Nah, this is how it's written these days... 😎

Will fix it in next commit!


How about adding a background-color so that it doesn't look bad when it's disabled?


so Play starts? Nice :)


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


There are still places were chan->err is accessed outside the context of a spinlock, so if there is in fact a race condition then it still exists.

What exactly is the race condition?  I don't see one in xilinx_dma_start_transfer()- if chan->err were to have been set between the old check and the moment the spinlock is acquired, the calls to dma_is_running(), dma_is_idle, and dma_halt() would all work correctly and the error would be caught at line 374.  The only thing you gain by protecting chan->err is that you save some time waiting for the channel to halt.


Great point! **Fifty points to Gryffindor!**


Maybe add a "LOOPEND" as a alternative to "LOOPLENGTH"?


I chose a display font to make the headings stand out. Averia Libre has a nice rounded stroke, is sans-serif and has the weight to distinguish itself at different font-weights.


Thank you :)


The one hour off is a bug and needs to be investigated. Maybe their this something wrong with time conversion the addon or your system is misconfigured. We'll see.

Min/ Max duration display seems to be broken in tvh 4.1. Works for me in tvh 4.0.5. There, the durations are not shown as plain numbers, like in your screenshot, but as strings like "45 mins". I guess there got something broken in tvh with the localization stuff recently introduced (and not in 4.0.x)


Hi


97903100_SolarCollectors (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Auto Vectorization on LLVM is much more advanced than GCC, and it's
actually one of the advantages of using iOS.
Superb for personal projects, but building entire Android with that might
end up funky. so.. :)

On Wed, Aug 5, 2015 at 2:05 PM, 박주형 <qkrwngud825@gmail.com> wrote:

> 404 bro :P
>
> On Wed, Aug 5, 2015 at 2:02 PM, Maxime Poulain <notifications@github.com>
> wrote:
>
>> hehe no problem, my concern was more oriented towards a possible
>> regression due to two optimization algorithm applied on loops. I don't
>> consider myself good enough to judge if O3 unrolling loops should specify
>> for example no vectorization.
>>
>> Btw, did you ever try bb-vectorizer flag for clang compiler ?
>>
>> https://archive.fosdem.org/2014/.../AutoVectorizationLLVM.pdf
>>
>> I noticed that on the master branch, android is built more and more with
>> clang, clang binary is even at 3.6.1 (I THINK).
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/temasek/android_build/commit/2b32ac8627030749f47dd228d1bc584d84885a1c#commitcomment-12532295>
>> .
>>
>
>


Just added a new clarification at the end of the readme. Thanks for the question!


Why?


@kisielk I might just make the whole interface private for now. Technically a breaking change but this has only been pullable for < 1 hour! I also don't expect any demand for a "Redis backed CSRF session store" because the disadvantages of cookies (size limit, can't forcibly expire, etc) don't make a big difference when you're just storing 64 byte tokens (+ gob overhead).

@avrtno Negroni is odd (I find it pretty unwieldy myself) but it can. A few of the other popular frameworks easily accept the above signature too. I guess I don't see a clear benefit in moving to returning just a `http.Handler`. Users who want to selectively wrap routers or even just the basic case where you `wrap(your(handler(MyHandler)))` can no longer share a CSRF instance - e.g.

```go
func main() {
	CSRF := csrf.Protect([]byte("32-byte-long-auth-key"))

	r := mux.NewRouter()
	// The two below lines are now invalid.
	r.HandleFunc("/signup", CSRF(ShowSignupForm))
	r.HandleFunc("/signup/post", CSRF(SubmitSignupForm))

	http.ListenAndServe(":8000", r)
```

This is obviously a very basic use-case, but it's not uncommon.


@garrett-schlesinger I had to add this hack to make twilio calls work. Any clues on why it was required?


Why this change?


Woops, forgot to change second to last line to Mary Jo


WithSearchPaths?


WithBaseDirectory?


no module named urllib2


我好爱你


Wrong date! Today is 5. August :smile:


来了


Helloo


The morse code on 712 means ASK and the code on 724 means DO. There is no particular reason why morse but it seemed like a fun idea that collar and updater are communicating in morse code! Bleeep Blip Blip ... Bleeep! :speak_no_evil:

One update message needs confirmation the other will happen right away, either "AFK first" or "DO it right away".


It is intentional that the method is not checking if the target file exists, unless it's needed.


<img width="480" alt="2015-08-05 1 36 33" src="https://cloud.githubusercontent.com/assets/8899510/9078815/2ec9468a-3b7e-11e5-9aee-2ed65370cbc9.png">

You Must upload Image!!!


Need to `channel-proxy` this chan


GAC resolver is supposed to resolve references from GAC. NuGet resolution is completely unrelated.


This doesn't quite seem to be working yet - https://noosa-eproperty.t1cloud.com/NOOEPRPROD/P1/eTrack/eTrackApplicationSearchResults.aspx?Field=S&Period=TW&r=P1.WEBGUEST&f=$P1.ETR.SEARCH.STW for example should be "submitted this week" and shows some detail, but 0 results showing up on morph.

However you might be in luck; this superficially looks like my council's system - https://github.com/CloCkWeRX/port_adelaide_enfield/blob/master/scraper.rb might help


This would have sufficed, however I think we need to fix it in proper way after all - by making the members unsigned long - there's no reason not to - probably something I shd have done in the first place !


Why are we not using a library to do this?


Sorry!  I didn’t even look at the edsurge jobs.  Assumed they were the same as last week.  For most of them we say “apply now” and it goes to the person — for this one we put MJ’s email in the blurb.  Not sure that’as needed.

Amy

> On Aug 4, 2015, at 10:23 PM, Tony Wan <notifications@github.com> wrote:
>
> Woops, forgot to change second to last line to Mary Jo
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/edsurge/careers.edsurge.com/commit/cf6db0082875536b7e65026c12b715ea024cb22d#commitcomment-12532462>.
>


Did you remove the println statements in a later diff?


remove whitespace!


could you change from AsEnumerable() to paging(ExecuteNextAsync)? it is more performant.


93651760-RowPerObjectReport (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1795 of 1795 tests passed)

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


The logo looks great Linda!


I commited yesterday and forgot to push
I think I will release today


97903100_SolarCollectors (macumber) - Win64-Windows-7-VisualStudio-12: OK (2223 of 2241 tests passed)

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


there should never be a case where the dRepeatInterval is zero, correct? If that's so, why not just throw an error for this case rather than gating the decrement?


Looks good! Did you test it?


Would there be any merit in sending an AudioListener event to inform the backend of the fact that a GST error occurred that has stopped playback?

This would make it possible to trigger the current skip-to-next-track-on-error behaviour, and possibly recover from situations as explained here: https://github.com/mopidy/mopidy-beets/issues/12.

Just emitting an end of stream event instead of doing a hard stop might also be an alternative, in order to allow the rest of the tracklist to continue playing.


93651760-RowPerObjectReport (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


1 more reason for Linus for ditching Github


See #121.


@steakknife honestly it seems that GitHub totaly lost a momentum. A lot of features, and enhancements random people are constantly whining about (IPv6 for example) yet to be implemented. There is no visible issue tracker for that. It's just not possible to ask GH staff for anything technical. I'm afraid with @mojombo 's retirement GitHub started to deteriorate. The only (and quite big btw) valuable asset left intact of GitHub is people but who knows for how long.


J'ai refais un gitignore dans mon vendor qui ignore tout sauf le dossier friendsofsymfony. Vu que j'ai modifié certain truc du bundle existant j'me suis dit qu'il devait apparaître sur mon dépôt.

Envoyé de mon iPhone

> Le 4 août 2015 à 17:46, Jean-François Jung <notifications@github.com> a écrit :
>
> Pourquoi t'as viré vendor du gitignore ?
>
> —
> Reply to this email directly or view it on GitHub.
>


This also needs to check that result[settings.displayField] is not undefined


One more thing... let's unexport CookieStore too


Will do for now, but I think we'll need a more sustainable strategy than doing this manually.
In Noda Time I have a SetVersion tool that fixes up all the AssemblyInfo and nuspec files in one go - do you think we should have something similar? Or at least a sed script?

(I'm not too bothered about version numbers for test assemblies, to be honest. Anything we ship is a different matter.)


@ArsenShnurkov happens :)


@tamasvajk - yes, I think it happens. I've saved 6 responses from SkyScanner into the unit-test project, around which I buld the logic. Indeed, new itineraries are added and existing itineraries seem to get more pricing options ( updates ).

Good point in your use case, it's not very user friendly in that sense. For me - I'll be doing all that in JavaScript. I'm going to have signalR push the diff down the wire wire and inside Angular add new ones and update existing ones. Still - it'll be easier to make explicit what's new and what's updated, as you suggest.

So how about I create a new type `interimItinerraryCollection` with `updated` and `new` itineraries inside - and provide that with the event?


So no support of old browsers ?


Will do - thanks!


I think this(`timePerMileInSeconds`) is better described as time per mile in hours.
Each mile takes `0.116725` of an hour.
```python
0.116725 * 60 # time per mile in minutes
0.116725 * 60 * 60 # time per mile in seconds
```


This just looks wrong!!


What will people use as the example?


No, because it is not just a simple change. It might be just a single line of code, but in order to build the source to an executable, you need extensive knowledge of several development tools, and a complicated setup on your computer.


Or should we keep `.env` but just let `.env.override` be the one developers can define and put their own credentials?


This looks wrong too, why do it twicw=


:heart:


I think `#token_refresh!` returns the token itself. That means when this is called the first time, `@client` will be a token?


Actually, I'm not sure what you did.
If it were 43 minutes the answer would be 8.66.


Yeah, my bad somewhere.  It's actually working locally (and worked initially in my morph account), but seems to have gotten stage fright, since being moved to "live". :(  I'll be looking at it tonight.  Will also check out your scraper as well, if the current approach resists cooperation.


Contrary to the title of this commit, tests are only finished for #allocate action.  Future commit finishes #packing_list controller.  Future future commit (as yet unwritten) will capture the #pack_it action


There's a significant and undocumented issue here that these indices *do not* correspond to the indices of the constraints at the MathProgBase level. At least for nonlinear problems, we order the linear constraints first, then quadratic, then nonlinear. This needs to be clarified for #508.


access token expire, right?


One way to set AssemblyVersion for all projects at once it to have a single file shared (added as a link) by all projects. This is what gRPC is doing currently.
https://github.com/grpc/grpc/blob/master/src/csharp/Grpc.Core/VersionInfo.cs

To make the version propagate to nuspec files shouldn't be too hard. Btw, nuspec files can contain tokens  like $version$ that can be set from commandline. (I am using that in gRPC because I have quite a few packages).

I think it wouldn't be too hard to make the build_packages.bat script retrieve the version from the VersionInfo file - then we would have single source of truth for the version.


Not a big deal, but you can do:

- `expect(response).to_not be_completed`
- `expect(response).to_not be_error`


If jQuery 2 then no, otherwise ok. But if jQuery 2 even the bootstrap website is not ie8 or 9


IMO the confusing thing here is that two different classes are located in a single file: COverlay and CRenderer. Subclasses of COverlay do not need "COverlay::" (EType is defined in this class) while CRenderer does.


I'm not quite sure what you did, although someone else did have this answer initially before arriving at the correct answer. We'll go over this tomorrow.
The correct answer is `Average Speed in MPH: 8.56714499893`


Also there will be an options request here


@mehulkar how do i render notes with line breaks accounted for? for some reason, when notes have a line break, it shows in markup but doesn't create `<br/>`'s or new `<p>`s

![screen shot 2015-08-05 at 1 51 11 am](https://cloud.githubusercontent.com/assets/1022843/9079113/7c0d0c0c-3b14-11e5-8ea1-7606e4b59afa.png)


Why @vvo, it's a simple GET without headers, isn't it?


Yup - another option is to expose constants from the core project (which everything else refers to anyway) and use those. I'm generally not a fan of linked files, but I can see how it would work for this one. It's not immediately urgent, but we should probably do it at some point.


Oh boy, I was actually looking for this feature. Can't wait for the next release :+1:


@WooD1k add Pods folder to gitignore


Nice! :+1:


Ahh I see why - I was using a conversion of 1.6 km/mi, as opposed to 1.61 km/mi

Updated answer: 8.56714499893

Thanks for pointing it out!


@WooD1k Create tests before implementation


A CSRF instance is a key and slice of option functions. Because the option functions reference an unexported type, it's impossible for the application to declare and initialize a slice of these functions.

If the API is fixed to avoid unexported types, then the application can share options. Here's one way to do it:

    type Option struct { f func(*csrf) }

    func Path(p string) Option {
	return Option{func(cs *csrf) {
		cs.opts.Path = p
	}}
    }

With this change, applications can share keys and options:

    opts := []csrc.Option{csrf.Path("/"), csrf.Secure(true)}
    r.HandleFunc("/signup", csrf.Protect(ShowSignupForm, key, opts...))
    r.HandleFunc("/signup/post",csrf.Protect(SubmitSignupForm, key, opts...))


Nice, I will test the batch approach on the different clusters I have access, this way I can generate different queue systems templates and share them. I will integrate the binary search on the the parallel version using mcapply.


These constants are added for Cortex-A57, mt8173 doesn't need them. I remove the constants in next patch.


修改内容


I want to keep the changes and possible side effects as minimal as possible.


What happens when this is saved and loaded in rct2? Do we want to break compatibility just for entrance styles?


I'm not an expert on these time conversions functions and glad about any advise, but I thought the most important part about conversion to UTC is function gmtime() which you've removed now? Don't understand this.


^ this seems more confusing than the `multiplayer_item` and `port_item` that were there before (and are still here)


It will show "Preview is currently disabled. You can enable it from the
option panel." instead of the preview with this commit.


Why is this one the only line that doesn't use loop_menu?


Yes, but I meant maybe change the background color too.

As for the wording, "option panel" seems weird. How about "you can enable it in MPC-HC's Options"?


> Yes, but I meant maybe change the background color too.

Hmm ok, what color would you like?

> As for the wording, "option panel" seems weird. How about "you can enable
it in MPC-HC's Options"?

Ok. About about the warning? Is it clear enough?


(pinaillage) Ajouter un saut de ligne avant (comme les autres au de-dessus)


Il faut pas faire ça. Du tout. L'idée de séparer les vendor c'est pour
pouvoir les mettre à jour (et une mise à jour de sécurité du truc qui
gère les utilisateurs, tu as très envie de la faire)

Du coup si t'as besoin de faire des modifs symfony propose une approche
très cool :
http://symfony.com/fr/doc/current/cookbook/bundles/inheritance.html
---

Cordialement,

Jean-François Jung
Responsable Pédagogique ItechSup
02.49.62.16.02

Le 05.08.2015 07:39, Mcouteau a écrit :

> J'ai refais un gitignore dans mon vendor qui ignore tout sauf le dossier friendsofsymfony. Vu que j'ai modifié certain truc du bundle existant j'me suis dit qu'il devait apparaître sur mon dépôt.
>
> Envoyé de mon iPhone
>
>> Le 4 août 2015 à 17:46, Jean-François Jung <notifications@github.com> a écrit :
>>
>> Pourquoi t'as viré vendor du gitignore ?
>>
>> --
>> Reply to this email directly or view it on GitHub.
>>
>
> --
> Reply to this email directly or view it on GitHub [1].

Links:
------
[1]
https://github.com/ItechSup-Ariane/Quest-MCU/commit/a4b318a9633a09d939dc2062492b44da879e20c2#commitcomment-12532605


Wow, your first suggestion greatly simplifies things. Thanks! :+1:

Rebasing...


C'est normal que tu n'aies pas internationalisé le message "Pas de changements." ?


Looks good!


93651760-RowPerObjectReport (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (747 of 747 tests passed)

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


I'll have a closer look later today and ping you with a patch.

The warning, maybe we should mention where it's actually being used in WebUI or is it too much?


Le `if` est-il nécessaire ?
Je n'ai jamais eu de cas où `a_blob.path != b.b_blob.path` mais je me trompe peut-être (un git mv ne fait pas apparaître de fichiers ici mais dans les autres sections chez moi).


Hi @alicechiu, I'm guessing you got stuck while writing `reverse_method`.  Maybe if you add some comments we'll be able to understand your thinking here and give you some pointers.  And if not, it's ok for now, we'll talk about it Wednesday in class.


Si pero entonces no se necesita este archivo sino que usas directamente en tus files

ciaaPOSIX_stdint.h (los tipos estan aca) y tambien include a
ciaaLibs_Matrix.h

o falta portar algo mas? Y claro matrix vera si implementa en c o en assembler las operaciones sobre las matrices... :) te parece?


Noticed the default cipher removed from the previous commit was `MCRYPT_RIJNDAEL_128`, changed here to `AES-128-CBC` and then later to `AES-256-CBC`. What was the reasoning behind this change, is it just a more secure cipher?


Can you double check this one?


done


No activity indicators or titles. Twitter shows very low resolution images. Cells not properly sized. No way to refresh or fetch more data. UI not aligned properly.


Yeah, this IS reasonable ;)
I updated my repo on atmosphere, too.


I was mistaken about not overriding renderOrderByElement() in SQLServer2012Dialect. The following should remain in SQLServer2012Dialect.


The following should be added to SQLServer2008Dialect.


Hotpink is not my favourite colour as a background.


Cool.


the `type()` of the calculated value is an `int`


https://code.google.com/p/android/issues/detail?id=180462

TabLayout이 왜 안뜨는지 분명히 코드상의 로직 문제가 아니었는데.

Design Support Library에 이런 심각한 버그가 있었다고 합니다. 결국 22.2.0으로 다운그레이드해서 사용하게 되었습니다.


If you are going for visibility I think alternating between two colours rapidly would be a better choice. I suggest CSS3 animations.


My first thought is to make a wrapper, something like:

    #!/bin/sh
    python2 -u $(dirname $0)/wtf.py


also data structure


Double check that you read the steps:
```python
>>> delimiter = '.'
>>> delimiter * 5
'.....'
>>> type(delimiter * 5)
<type 'str'>
```


Do you know how you arrived at this answer?
```python
>>> 1 + 2 * 5
11
>>> type(1 + 2 * 5)
<type 'int'>
>>>
```


How do we solve this?

On Wed, Jul 22, 2015 at 6:24 PM TysonKerr <notifications@github.com> wrote:

> Both notepad++ and sublime are aware of the newlines \r\n and \n. Basic
> notepad is not. Non-programmers probably wont have notepad++ or sublime,
> and will use the default program.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gikeymarcia/Collector/commit/f07cf35f61ce1dfa3baffd318846f217d6bcf7a6#commitcomment-12311692>
> .
>


Hmm...
```python
>>> 1 + 2 * 5
11
>>> type(1 + 2 * 5)
<type 'int'>
>>>
```
Do you know what happened?


It might be overkill, people try to enable this option will probably have
read the message in the webUI I guess.


Extra class is not necessary for anywhere other than the InstructorComments page.


97903100_SolarCollectors (macumber) - i386-Windows-7-VisualStudio-12: OK (2213 of 2241 tests passed)

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


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.15](http://pixelated.design/build/viewLog.html?buildId=84&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 292 Build time: 00:00:16


And so are these two icons. Don't change the behavior here. I know there are so many inconsistencies between different FRCs with regards to which icons/... get shown, but just let it be for now.


TeamCity FluentMigrator :: FluentMigrator [Build 335](http://teamcity.codebetter.com/viewLog.html?buildId=196622&buildTypeId=bt82) is now running


странно. у меня чтобы заработало нужно убрать /restservice


Corregido.


TeamCity FluentMigrator :: FluentMigrator [Build 335](http://teamcity.codebetter.com/viewLog.html?buildId=196622&buildTypeId=bt82) outcome was **SUCCESS**
Summary: Tests passed: 3113, ignored: 60 Build time: 00:01:40


I am still facing this issue from your library. Why did you close this issue if its really not solved.


TeamCity FluentMigrator :: FluentMigrator [Build 336](http://teamcity.codebetter.com/viewLog.html?buildId=196623&buildTypeId=bt82) is now running


It might be more trouble than its worth, but we could add a .gitattributes file to the repo, and within that, we can set a value
`text eol=crlf`
which would turn all newlines into windows-style `\r\n`. However, I dont know if this would mess up Mac users.

More info here:
https://help.github.com/articles/dealing-with-line-endings/


TeamCity FluentMigrator :: FluentMigrator [Build 336](http://teamcity.codebetter.com/viewLog.html?buildId=196623&buildTypeId=bt82) outcome was **SUCCESS**
Summary: Tests passed: 3115, ignored: 60 Build time: 00:01:27


@DanWahlin these two properties were breaking Visual Studio Code's ability to detect the TypeScript files without the `///<reference...` tags. I didn't test other editors. I also didn't see any issues with removing these options (they are gulp-typescript) settings so we could restore them in the gulp file if needed.


Not able to switch between tabs at random times. UI lagging too much. Links in tweets not clickable. Flickr screen layout fuzzy on iPad.


Wrapping in a
 tag would work but you will have to specify font-family
again. There is also a CSS property for this. `white-space` or something.

------
Mehul Kar
@mehulkar

On Tue, Aug 4, 2015 at 10:51 PM, Ankit Shah <notifications@github.com>
wrote:

> @mehulkar <https://github.com/mehulkar> how do i render notes with line
> breaks accounted for? for some reason, when notes have a line break, it
> shows in markup but doesn't create
's or new
s
>
> [image: screen shot 2015-08-05 at 1 51 11 am]
> <https://cloud.githubusercontent.com/assets/1022843/9079113/7c0d0c0c-3b14-11e5-8ea1-7606e4b59afa.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TeaWithStrangers/tws-on-rails/commit/1b0c767b0cd63398b051fa5eae0f5a85a97e1e24#commitcomment-12532720>
> .
>


@dsawa What do you think about: `_.merge(category_names: [@props.category], _.serializeTopic(topic))`


Are you sure this is ok?

`thread.join()` will block until the thread finishes, that way we can be sure that the result will be there when we test it.

On the first test, this wouldn't be so bad, because we only expect that the function has been called, but on the second one, we rely on the thread putting a value on the queue - although `queue.get()` will block if the queue is empty.


True ... better to put the fix inside.


Thanks, sorry I didn't see that you'd added this.


Well done.


I figured out how to solve that Deity issue I told you about a while back. Fix was alot easier then I originally thought it would be. Though I'm not sure if it a temp. fix or perm. fix. Anyways the 3 Steps to fix it is as follows:
1) In cell A1 on the Deities Sheet change the Formula to:
=IF(ISERROR(B1),A3,LEFT(A3,B1-1))
2) The Formula in B1 change that to:
=FIND(" (",A3)
3) Save and walk away!!!

Although Choosing a Deity 1 step away from your Alignment still doesn't work properly (ie You cant be True Neutral and select a Deity that not also True Neutral (ie a Neutral Human Druid cant select Meilikki (FRCS) who is Neutral Good without unchecking the "Limit deity list based on alignment & race" check box))


How can there be so many final commits? And by final commit, what should I perceive as work done?


This one is a bit of a hard slog, but **absolutely necessary** before beginning work with a framework, to achieve a more flat and modern look.


Thanks Rekha for your contribution!
Your solution is correct, To keep things simple, I have added only the insertion code( 1 line of code) into the Module1.vb


@stefanpenner I thought you removed delete here I thought we just set to undefined?


Change java version on own computer.


Hi Rekha! This is the correct solution. I will discuss in the class and then add into the module.
Thanks!


Jo, kan lage en partial for link (samme som er gjort for de ulike input-typene)


En fait, pour que ça apparaisse, il faut faire une manipulation un poil tordue : imagine que tu a un fichier `text.md` qui contient deux lignes de texte:

```
texte 1
texte2
```

Eh bien si on fait

```bash
git rm text.md
echo "ligne 1" >> fichier1.md
git add text1.md
echo "ligne 2" >> fichier2.md
git add text2.md
# + édition du manifest.json, bien entendu
git add manifest.json
git commit -m "split"
```

alors ..

```
[master bca2708] split
 2 files changed, 2 insertions(+), 2 deletions(-)
 rename text.md => text1.md (50%)
 create mode 100644 text2.md
```

Et

![screenshot from 2015-08-05 08 36 47](https://cloud.githubusercontent.com/assets/7889675/9079679/277cf858-3b4d-11e5-873c-0bb6769e3827.png)

Bon, c'est tordu et il faut y penser, sauf que c'est exactement ce que fait la fameuse "prcédude de migration des article" de "avant ZEP-12" à "ZEP-12" (en français dans le texte, on découpe les articles en extraits). Donc oui, ça arrive et non, c'est probablement pas un bug.


It's early and pre coffee, but I don't follow: what is this about?


I'll put this into microgram in a separate PR (against microgram, of course)


EDIT: je sais pas faire des copier-coller.


I'm getting an error right here as the lazy seq is evaluated. It's in `seg-uidx`, complaining that the third arg is a number and not a `[col ci si]` vector. I'm pretty sure it's where `sources-connected-from` maps this fn over the output of the internal `sources-connected-from`. Are you hitting this? I've tried on the `fixed-seqs` and `second-level-motor` demos. Have you changed comportex locally?


Many servers use "&" for admin, so is it possible to have both "&" and "!" show as admins on Shuo?


couldnt you have just created some symlinks?


please use "game"! "mod" basicly is wrong...


If it was _that_ simple I would be able to read it. Sadly it is little more than squiggly lines on a page.


Non :-°


Ok :)


Add demo


I already fixed it)


Ok


We could get rid of the `LANGUAGE_SETTING` define now that we have a proper constant.


Unrelated change.


应该是 convert to UTF-8 encoded byte string


This is unicorns all the way down ;-)


"alias" may be better than "abbreviation" to be consistent with the words used above.


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


@eeree oh wow, didn't see that. My bad :(


Uppercase. "repository" -> "Repository"


From the end peeragogues user point of view the comments system in github does not convince me much. We should try find other more practical alternatives @holtzermann17

http://www.google.it/imgres?imgurl=http://cdn.meme.am/instances/500x/61377547.jpg&imgrefurl=http://memegenerator.net/instance/61377547&h=375&w=500&tbnid=pyT4PyYnSkEyfM:&docid=0dz57qjRI2GigM&ei=ybDBVYfqOoSrU_nrt5gN&tbm=isch&client=ubuntu&ved=0CCAQMygAMABqFQoTCMfy1t6rkccCFYTVFAod-fUN0w


yep thought I caught all. sed sometimes modified the eof.


It's right that the data structure changed in  2a72726. But before DSpace 2a72726 I think all authors where put in one metatag like:
```xml
<meta content="Nachname1, Vorname1; Nachname2, Vorname2;..." name="citation_authors" />
so there is no need to apply the fix for version 4.x
See: https://jira.duraspace.org/browse/DS-2309?jql=text%20~%20%22google%20scholar%22


Why did you go with `const char*`? `ISetting::GetId()` returns `const std::string&` and all `CSettings` and `CSettingsManager` methods expect a `const std::string&` and you even had to throw in some manual casts to `std::string`.


Could not see background fixed. Could not see UI fixed for rotation.

Need to see longer texts on cell.


Ohhh, yeah, sorry. Remember now. A method was always broken, i fixed and
forgot to push.

My plane is taking off now.

On Wednesday, August 5, 2015, Marcus Lewis <notifications@github.com> wrote:

> I'm getting an error right here as the lazy seq is evaluated. It's in
> seg-uidx, complaining that the third arg is a number and not a [col ci si]
> vector. I'm pretty sure it's where sources-connected-from maps this fn
> over the output of the internal sources-connected-from. Are you hitting
> this? I've tried on the fixed-seqs and second-level-motor demos. Have you
> changed comportex locally?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/floybix/comportexviz/commit/010a0d077a54bef15d2c5c95d3c1fa3ad2d020e0#commitcomment-12533150>
> .
>

--
Felix Andrews / 安福立
http://www.neurofractal.org/felix/


@duncanspumpkin Glitches in stable build, will update the result of vanilla rct2 later.
![image](https://cloud.githubusercontent.com/assets/13048125/9079940/dfc2fa64-3b81-11e5-9ba5-1b3200b703d4.png)


Those are some sweet improvements!


LGTM


I just updated winnfsd plugins.

Good job boy ;)


I'm curious why this was based on Turkmen rather than Turkish. It sort of instinctively makes sense to me but what rules specifically make it a more suitable base?

I'm looking at another variant of Turkish used in Bulgaria, Ukraine, etc. that is written using Cyrillic. On the plus side, I can actually read it which makes toying with support for it a lot easier than all these Arabic scratchings. Transposing it to another alphabet is reasonably simple (there are enough exceptions that you couldn't spell check on the results but there are few enough that it's likely easy to adapt hyphenation rules for it). On the other hand hyphenating it appears to use a mix of Turkish and Russian rules.


Oops plz ignore it. It's a message from 2014. I was ripping out incremental compiler from sbt/sbt using git filtering and Github apparently recognizes it as a brand new commit.


This is close.
But.
1. Don't forget that Python uses floor division for integers.
2. If your division was using floats, then your `totalRunTime` would be correct
3. Did you check it by hand? I think your `convertToOneHundred` is messing with things.


reverted


> If the descriptor is being canceled, that means it's being removed from the polling queue entirely. Moreover, in the common case the descriptor is being closed. Unless the thread also has a timeout, it could sit there forever. So the thread needs to be woken up by moving it to the pending queue.

Sure, but isn't that why `fileno_signal` is called with *all* of the possible events? i.e. `POLLPRI|POLLOUT|POLLIN`. `fileno_signal is called from more places than just `cqueue_cancelfd`.


Teşekürler.


You are right ! That is wrong. Thanks for reporting this
The goal is to call ImageService.Initialize with whatever you want and then it is passed to InitializeIfNeeded


I'm rather new to typescript but won't this commit break this file?
I'm getting errors, since hotkeys doesn't exist on the ng module since it's not renamed to angular.


Did you see the hint in line 7?
Recall that Python does floor division w/ integers:
```python
1/2 # =0
1/2.0 # =0.5
```


Can you post sample code or a code snippet? I'd be glad to take a look.


@Usaga:

please avoid usage of wx in all non-gui classes/files. some of the code can/will be reused in spring where wx can't be used.

this applies to all files not in src/gui


This allows us to generate maps like this:
<img width="1075" alt="screen shot 2015-08-05 at 12 05 52 am" src="https://cloud.githubusercontent.com/assets/2423263/9080137/cef1ff68-3b05-11e5-8f0a-61bda73a3e91.png">


Ah true. Nevermind, then. :)


Oh, ok


Check this again.


Did you check by hand?


Tu aurai pas oublier par hasard d'ajouter des .x_blob pour les noms de fichiers renommés ?


Why the check fw.length > firmwareVersionMajor ?
From what I can tell the firmwareVersionMajor isn't the last byte in the firmware.


Hello


Why don't you return them by value?
```c++
QList< QPair< QString,QString > > gatherChannelAndPasswordPairs()?
QStringList gatherQueryNames();
```


Solves bug #25


i've tested this already, also (0,4): passed STC (reconfirmed by your test) and failed LTC.


Double check.


@ashimokawa perhaps the file-reading related changes may be of interest also for your code that reads pbw/pbz files.


Non. Là, dans ce cas là, ça fonctionne. GitPython est bizarement conçu :p (et leur doc pas claire, j'ai du aller lire le code). Maintenant, je peux le faire si ça t'amuse, j'imagine que ça fonctionne :)


a-ha, got it. Nevermind the comment on the last commit :)


Why do we need an interface ?
Isn't `MessageListView` the only class that needs it ?


Such clever.
Very hack.
Wow.


Seems off by 60 cents.


dlaczego jest pusta linia - prosze usun


Changed HTML version to HTML 4


Checkmarx scan results 2015-08-05T07:12:20Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
14
22
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


I already identify with the MC.


Infinite loop?


Proszę usuń ten tekst


In Python terminology, the `type` of this is a string.
You can test it:
```python
>>> delimiter = '.'
>>> type(delimiter * 5)
<type 'str'>
```


@VMBindraban

May you please add either a simple spec/test for this.

I will appreciate as it will set a ground for us to work with association
custom error messages.

Thanks.


I will fix the name. I don't suppose it will directly affect performance but as it was easy enough we might as well not pollute the GC.


This is a bit more than I had.


Sure thing!


1. You seemed to have an extra `5`.
2. Remember that Python does floor division on integers:
```python
1/2 # = 0
1/2.0 # = 0.5
```


std::string supports a ctor + operator for `const char*` and we don't need to put it in the header and the implementation if we use `constexpr`.

I've also got a build-error where we use the id as const expression in initialization and thought it's better to use `const char*` instead of `std::string`, less code, same result.
```
VDPAU.cpp:55:1: error: cannot convert ‘const string {aka const std::basic_string<char>}’ to ‘const char*’ in initialization
```


final version


I update Mhi driver from note4 kernel
https://github.com/shminer/LG-F460-Kernel/commit/3029dcf74c0940939bec7341e43051d63f1561e0


Making a public Option struct with a function type member could be
useful—good example. I'll look into it tonight. I'd like to genuinely lock
down the public API very shortly (next day or two). Thankfully, this
specific suggestion won't break it in any practical way.

I don't foresee changing the signature on Protect to accept a http.Handler
as an argument  (rather than return a func) though: still not sold on that
for the reasons I've described above.

On Wed, Aug 5, 2015 at 1:57 PM avrtno <notifications@github.com> wrote:

> A CSRF instance is a key and slice of option functions. Because the option
> functions reference an unexported type, it's impossible for the application
> to declare and initialize a slice of these functions.
>
> If the API is fixed to avoid unexported types, then the application can
> share options. Here's one way to do it:
>
> type Option struct { f func(*csrf) }
>
> func Path(p string) Option {
> return Option{func(cs *csrf) {
>     cs.opts.Path = p
> }}
> }
>
> With this change, applications can share keys and options:
>
> opts := []csrf.Option{csrf.Path("/"), csrf.Secure(true)}
> r.HandleFunc("/signup", csrf.Protect(ShowSignupForm, key, opts...))
> r.HandleFunc("/signup/post", csrf.Protect(SubmitSignupForm, key, opts...))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gorilla/csrf/commit/21dad7a81a52b69de043cd46eb1e603cf5d1c6d1#commitcomment-12532770>
> .
>


@wasabeef Thx:) :beers:


space


We should figure out what CPU architecture we are using before downloading. right?


a bug in 103 line


Okay.


Wasn't this on purpose for Celtic pagans ?


I think you gave a 60% discount. Maybe?


I planned on removing them later.


this changed since last time I saw it. funny, I was going to suggest changing std::string to const char*. However, the more I thought about it, I realized that std::string might be more efficient as otherwise the cstring will have to be copied to the heap every time it's implicitly cast to std::string


这是一个测试工程 branch 2 不错呢


Never use fps to calculate time.
fps is not reliable due to non linear timed frames (0.033, 0.033, 0.066) and timeline problems.
You should use the pts (its in seconds) of the last frame minus the pts  of the first frame


I'll have to give connection.cpp a closer look, but I agree, received data should only be abled to cause soft fails only, no crashes.


Hello


Sure it has a ctor and a comparison operator but that means that the compiler will have to call those on every usage of these constants which doesn't sound ideal to me. I'm not sure if the benefit of `constexpr` outweigh the drawback of it having to be `const char*`.
Personally I'd prefer `static const std::string` because that matches the type used by the settings library.


Oops, Turkmen was a mistake. Shall fix that. Basically Uyghur is a Turkic language using the Arabic script as an alphabet, but hyphenating according to Turkish-style conventions. Which is unusual for Arabic, to say the least...


I don't have an issue if we prefer `std::string` over `const char*` here, let me know and i'll change it back.


If you run 6.21 miles in 43.5 minutes, you probably are not running 16 miles in an hour.
I'll go over these in class.


aaaa


ok np, will change it back than and need to adjust the parts where we use it in initializations.


init


Minor nitpick because you seem to want to contribute more than just one commit: I'd refactor your shortlog

`expose the possibility to override decimal digits`

to

`Numbers: Allow overriding decimal digits`

Rationale: http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits/#how-to-write-good-commit-messages

Note that this is no official guideline so feel free to neglect this suggestion.


I believe all requirements change were on purpose from the original ACR.


Layout not fixed. Unable to assess.


Good job.


Good job. I hope your interview went well.


This seems like a separate (though useful) change, that would be best put into a separate commit (or at least documented in the commit message)


looks good to me. Can you temporarily log parameters list to make sure the right ones are included?


It's a POST no? So it need the content-type: application/x-www-form-urlencoded, multipart/form-data


@tbeu I just committed an updated version b5854f8b8948bcb16fe2ee3b672882f2d3b15f1e taking your comments into account:

- External trigger is also in the packager since it is possible to trigger packages from there, too. So it seems to be consistent to have it at both places.
- I have now two trigger examples that use another trigger source
- Removed the "(default!)" hint
- Renamed conditionalExternalTrigger to trigger


IMO the name doesn't fit the description well. How about something like `show_digits` with appropriate semantic changes of course? Not fully sure of that though... other suggestions welcome.


Use double `` to mark it as code (single ones will make it italics)


No indentation for .rst (in comparison to .md), indent will make it a quote.


I wish everyone here would stop whinging and learn how to use gpg to sign tags (thusly history). https://git-scm.com/book/tr/v2/Git-Tools-Signing-Your-Work


Mistake or otherwise it might actually have some merit. Does Uyghur draw on local words of Russian in the way that, for example, Kazakh does? If so Turkmen might be actually be a better base, but you would want to transliterate to Täze Elipbiý rather than the Turkish alphabet to use those rules. (Which is potentially more interesting because Turkmen was written using an Arabic alphabet too in much more recent history that Turkish was.)


I'd move this comment above, replacing the comment on line 106


Yes Shaun, I don't need help in OpenEdge side , I just need break the inertial force in C#.net version community 2013: here is the info :

Microsoft Visual Studio Community 2013
Version 12.0.31101.00 Update 4
Microsoft .NET Framework
Version 4.5.51209

Once I can test your SDK, and maybe add something more,   this code will go straight away to the community forum for share or I can open a project that can be used as a guide for other OpenEdge ABL developer to integrate MYOB.


thanks!

i see the benefits of using try/catch but it will some time until i will use it in a nice way i guess...


This is FuXiong


dRepeatInterval's value comes from mRepeatInterval in Action, which can be zero if the user hasn't set a repeat interval yet. Would it be better to increment zero values when passing the parameters to setDefaults in RepeatPickerFragment?


this is theoretically a better idea to keep the result of detection around,

- but in real world, `syncfs` is supported by recent linux kernels[0,1] and some popular recent linux LTS releases . and i'd like to favour the recent kernels instead of kernels older than 2.6.39. please note that ubuntu 12.04 ships with linux 3.11. albeit the kernel shipped with rhel6 is 2.6.32. but it includes the syncfs patch[2].
- also, since this condition statement is sitting in the io path. i don't want to add an extra `if` statement in it.

that's why i didn't change it into a member variable in #5395. what do you think?

---
[0] http://thread.gmane.org/gmane.linux.file-systems/51497/
[1] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=b7ed78f56575074f29ec99d8984f347f6c99c914
[2] https://bugzilla.redhat.com/show_bug.cgi?id=691267


I have increased this because the default value 2 seconds is too low. Specially on flush, in which not receiving the pubsub messages means they will be lost forever. Here we could still keep it as `pubsub_timeout_ms` but I would still keep it as the window one, because we know the connection can be hold for that long.


This change causes a crash when you try to move a file to other folder.
R.id.progressBar doesn't exist in files_folder_picker.xml


Not it's a GET :)


Wow! Thank you!


No, normally not work ...


Replace with multicatch


Use SLF4J logger


@TheSooth Pods were committed intentionally, i thought that i will work alone on that project and there is no potential conflicts in Pods, but if needed i will remove them from repository and will add to gitignore


As a conclusion: now that we identified the cause of the differences, I will redo the multipliers that need to be updated in this commit.


I'm curious: why do we need to store something in the scope and not in the controller itself ?


I modify the code so that it only copy mutable In instances( SharedVariable with update). This should simplified the logic and fix the problem. I will try to improve testcases.


Use `LOG.error("Something's wrong", ex)` to save stacktrace in log


lgtm


I think it is bad idea use sprintf with data from external source. Check for xss is needed.


Why `lft.getLoadBase` here, instead of `view().fees().units`?


Yes sorry, $.ajax defaults to GET even when data is here and we accept GET searches (js client always uses POST, dunno why)


Must be removed or replaced with `trace` and more precise message.


Why do this?


I don't like that we call `script_error` now every time, not just the name doesn't fit the new functionality anymore (should be more like `script_error_if`), but also because we pass things, like the pointer to the function name, or the lua context, that aren't used most times, when we have no error, two times through the stack.

I think the `PCALL_RES` macro should make the extra call only if there is an error. This way we spare many needless jumps.


I do not sure that this if statement is necessary because it contains code to delete all dashboard's components.
I think we should not use it with new life cycle.

And the same opinion for "this.init(noAutoAutoSubmit);" statement (https://github.com/krivera-pentaho/pentaho-platform-plugin-common-ui/blob/680620ba451b7fd236af9b6fc6fc9be0bf31fbf9/package-res/resources/web/prompting/PromptPanel.js#L639) because "init" method contains logic for creating all new components.


I think  this part can be modified to simply mention wdm because there nothing of significant difference here, even better would be just to find a way to context into the Windows docs because there is really nothing here that isn't mentioned there and differs from Linux and Mac.


True, I had forgotten this since the last time I wrote documentation. Redoing..


Reorganizing imports are not very useful


I shouldn't have committed the comment in the first place, my bad


There's no need for `;` after closing the if statement. It is not syntactically wrong, but it's unnecessary.


:+1: nice one!


The first thing I noticed is that you are using a HashSet of String, don't know if you are familiar with HashMap. Basically it looks like this HashMap<Integer, String>


Very nice for quick and dirty :+1:


aaaaaaaaaaaabbbbbbbbbbbbccccccccccccc


I like the trick that we take the method name now, too.


I will upgrade to that, shouldn't be anyone below 6.4 by now ;)


For some it's "executable bit" but for most it's probably "exectuable attribute" so I would phrase that "needs to have the exectuable attribute set" rather than "exectuable rights."


Greeting, you @Reisyukaku, for being very glad to see the code on github.

* Since it has a repo now, you may want to find somebody to help you with its bugs/features. Don't forget to show other this repo.. *However whom i could find in china are mostly gamers not programmers for this*
* You may need to use a more descriptive git commit notes, so whenever something goes wrong, at least easier to find than it is currently. *Yes, just `git stuff` or says, `add sub module:mset`, or other.*

Hope your progress goes well. ~~I've seen someone booted N3ds emunand 9.5 from rxtools on gbatemp.~~


I was considering maybe we needed to make false _dom_node but it looks like no, we don't need to do so.


Yep, I'm really bad at formatting, I'm blind to it. One of the main points of going more open source is to get me better at this.


Brackets is needed, because if first condition will be true, then it will not check !$byEmail condition.


actually i tested it using `(-1,4)+(0,5)`. Passed `(-1,4)` failed `(0,5)`. Makes sense to run `(0,4)@LTC` now.

I would like this patch to be committed. One non scientific reason is that it would improve tactics. But a more important reason, is that lowering the depth means it kicks in more frequently at 15" game play, which makes further SE tweaks more mesurable.


Yes; it passes all the corner cases I could think of.


> * Returns empty object if row wasn't found, null expected
thats the case of a bug :)


Good catch, I tend to use for managed objects so normally "flat" classes.


`email = email[0] if email else 'NoEmail@mail.com'`
or
`email = email and email[0] or 'NoEmail@mail.com'`

Have the exact same behavior.


Are you sure that it should find in Channel entity, not User Entity?


Yep, I think thats a good thing since name spacing in obj-c is "limited"


The bit about "chmod" is highly irrelevent without additional information IMO.  I can already infer as much as you just forced me to read,  perhaps link  to the GNU coreutils page for chmod.  http://www.gnu.org/software/coreutils/manual/html_node/chmod-invocation.html#chmod-invocation


@herrniemand
> I just think that word "nigger" that is offensive in US, is not offensive in Russia, in oppose of word black WHICH IS OFFENSIVE IN RUSSIA.

while fun thing to use in the movies (such as "brother", part 2 :) this is bending the truth. the actual words are негр and черномазый, while you are talking about нигер and чёрный, which are not the same words :stuck_out_tongue:


Yep Safari also.

Wro4j is complaining about
```
2015-08-05 09:51:01,753 ERROR [geonetwork.wro4j] - Error occurred during a wro4j request handling
ro.isdc.wro.WroRuntimeException: Cannot build valid CacheKey from request: /geonetwork/static/angularjs-color-picker.min.map
```

Do you have that also ?


Would you mind removing the manual require and instead add the subnamespace to the autoloader?
(https://github.com/FluidTYPO3/flux/blob/development/Tests/bootstrap.php#L19)


Information, about the creation and authors of this website.


Should it not read; ```item.setAttribute('value', titles[i]);``` since the index is no longer being passed through? Or have I missed a trick?


I thought I might want to change the block at some point. but I think its cleaner to not allow that, and I've never had to do it.


```
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
```


Yes a Spartan6. I'll add a comment or eventually rename it to s6rgmii.py (not sure it will be easy to have something generic for rgmii). Thanks.


@makc Well. Fine. Not the best example. *) ЛОЛ. Этимология блеять*)


This is a workaround, not a solution.  The solution is "bundle update" on your local system (which isn't a valid solution on Windows) and check-in the new Gemfile.lock or don't check-in the Gemfile.lock at all. The workaround is to simply delete the Gemfile.lock as you suggested.  I think we should encourage people not to check-in the Gemfile.lock and learn to properly use dependency locking inside of their Gemfile but that's for another route, can we just suggest they don't check-in Gemfile.lock at all?


It is because of SelectionUtilsTest that I moved as a unit test, where SelectionUtils is, then it executes some ruby/python... scripts


awesome :smiley_cat:


Are you mean for syncfs we add a judgement 'm_filestore_syncfs'?  For most case, syncfs implemented and we don't need check 'm_filestore_syncfs.


You're right. I just re-changed the encoding with 5f3cb392de4042643e83256593281f4b35c184a0 . Thanks for the hint!


I normally just call `make html` and optionally `make clean` directly. This script was originally created for travis where we start with a clean folder anyway. But also there it does not hurt to clean before build.
Would it make sense to call `make clean` instead? That way we only maintain the clean-code in one single place.


Sorry for my dimness, but how this connects to the local installation ?


Do not use sprintf with external data.


Spotted a slight issue I think(?) but otherwise nice move towards removing jQuery.


Thanks for the link. I'm convinced :+1: I will change the URLs.


$nameFromEmail = explode('@', $this->email);
+        $internalUserName = $nameFromEmail[0];


(That's a very useful image.  Quite relevant for me these days because it shows what reviewers think of a couple recent papers I've submitted.  OK, fair enough.)

I think it would be useful to connect the outstanding tasks (mentioned on this introduction page) with the "next steps" described in the pattern catalogue -- maybe the bigger requirements relating to things like how to deal with comments will become more clear if we think of the big picture.

A quick thought is that hypothes.is could be easier for new users.  Yes, it requires a sign-up, but that's easy to do.  And when it's turned on, you can just use the mouse to highlight passages and add comments.  The workflow is very simple.  Should we install it and see how that works out?

***
# Fitting ideas from http://peeragogy.github.io/introduction.html to the framework from the patterns.

*Peeragogy.* We intend to revise and extend the patterns and methods of peeragogy to make it a workable
model for education.

- *Translations.*  People need to be able to read the book to be able to use it for "education".  I don't think we should insist on everyone using English all the time.  There's a lot more to do than just translating the text we have, but it could be a start.

*Roadmap.* If we sense that something needs to change about the project, that is a clue that we might need to
record a new pattern.

- *Next steps? What’s the future of the project?* The Peeragogy Charter could show us what the major next steps are.

*Reduce, reuse, recycle.* We’ve spun off the pattern catalog from the Peeragogy Handbook into this paper,
sharing it with a new community and gaining new perspectives. Let’s look for other parts of the handbook we can
spin off!

 - *Maintaining a list of useful resources*.  We don't want to have to develop everything ourselves.

*Carrying capacity.* Making it easy and fruitful for others to get involved is one of the best ways to redistribute the
load (compare the NEWCOMER pattern).

- *<<< making it easy to comment could fit here.*  If people have comments or questions about anything in the book, this would make it easy for them to share their comments.  In the text of the intro it says that the adventure begins when they pick up a pencil -- but they could also just make comments on the website.

*A specific project.* We need to build specific, tangible “what’s next” steps and connect them with concrete action.
Use the SCRAPBOOK to organize that process.

- *Workflow for the 4th edition*.  Let's try to begin with why.  Also, it would be clearer if we got version 3 out the door.

*Wrapper.* We have prototyped and deployed a visual “dashboard” that people can access to immediately get an
idea of what work is ongoing in the project with links to ways to get further engaged. Let’s improve it.

- *Crafting a visual identity* seems related to this.

*Heartbeat.* Identifying and fostering new HEARTBEATS and new working groups is a task that can help make the
community more robust. This is the time dimension of spin off projects described in REDUCE , REUSE , RECYCLE .

*Newcomer.* A more detailed (but non-limiting) “How to Get Involved” walk-through in text or video form would be
good to develop. We can start by listing some of the things we’re currently learning about, including: business
issues relevant to the Peeragogy project, how to run a MOOC, and hot-syncing our website from Git.

- *Developing a really accessible DIY tool-kit*  to make it easy for people to get started right away.

*Scrapbook.* After significantly pruning back our pattern catalog, we want it to grow again: new patterns are
needed. Reviewing the contents of the SCRAPBOOK will be one place to look for inspiration, but there are many
others.


Why did you remove the empty line at the end of file? You should have an empty line at the end of the file here, some discussions about it [here](http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline)


You should make sure that this is also done in the versions to be potentially included into the MSL.


> Also, the only failure mode for fileno_signal is when delayed initialization of the alert descriptor is enabled

There's always more and rarer corner cases. e.g. some syscall (e.g. `write`) might fail due to seccomp/tame.

> If the application closes the descriptor while the controller still has events installed, bad things happen and there's no reliable, race-free way to recover other than to discard the entire controller state. That's why the current code keeps chugging along on an error; it tries to cancel everything it can.

Should this be done somewhere? that is, call `:reboot` if certain functions error?


So we need to keep this in mind and place pot files generated with `textdomain "country"` in the i18n repo, isn't it?


> @piedoom: So true. Amazing how people attempt to justify their behavior by blaming something else. I don't see one person in this thread explaining why saying "retard" is completely acceptable and justifiable, just a bunch of angry dudes pointing out that "git" - an insult with no history of prejudice and hate- is also bad, for some reason in attempt to move any wrongdoing away from themselves.

Multiple people explained both throughout the course of this thread, actually. So I'm not really sure what point you're trying to make here.

> If you don't like how Github runs their company, you're free to leave. Some people here sound like the sovereign citizens of the internet. "It's not illegal, so it must be fine!" If your biggest argument is that it's not illegal, that's not a very good argument.

> Relevant xkcd, with emphasis on "host you while you share it".

This stops applying when you become monopoly-sized, and an entire ecosystem relies on participation in your services. That comes with a social responsibility.


Yes, it makes sense. Thanks for the suggestion.


yeah. i'd prefer keeping the way it is, i.e., print a debug(0) message when `syncfs(2)` is not supported, to adding `m_filestore_syncfs`and checking it every time we syncfs.


@piedoom "If you don't like how Github runs their company, you're free to leave" :smile: when the girfriend does not like her boyfriend in some way, she is also free to leave, but instead she decides to make a scene. some may say she should not, but we all know she only does that because deep inside she hops her boyfriend would actually listen to her for once.


we can leave it false, not a good practice to use daemon threads all around... wdyt?


Seems the recent fix has revealed another bug indeed. I guess this would have failed before this commit too, if you had left out the quotes.


The scan failed.
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Hi, nice patch. Were you going to submit a PR?

pcgenorg.atlassian.net is our JIRA issue tracker. We have this tracker open:
DATA-2496
[Paizo] Advanced Class Guide Errata
Link: https://pcgenorg.atlassian.net/projects/DATA/issues/DATA-2496?filter=allopenissues

Send me a name, and I'll assign you to the issue tracker (Assuming you're already part of the system). Otherwise, sign up or email me "help@pcgen.org" with a user name and full name.

Cheers,
Andrew


Didn't know about this one, nice


A flag indicating that not all chars could be encoded/decoded would be nice. In this case the problematic chars could then be omitted from the result?


@joepie91 "If you don't like how Github runs their company, you're free to leave". Wait. I am paying for my account, that is not considering the people I invite and general "Pushing mainstream". We as community have full rights to have fucks about such an idiotic behaviour.


Why didn't you use the 'silent' option of setValue? Seems simpler / as designed. Do I miss something (I haven't tested).


Jetpack tests are run using jpm, which means we shouldn't ignore the test files because when the tests are run, this file won't be included and the tests will fail, at the same time, we don't want the tests to be included in the packaged XPI.

So should I omit it ?


Good, these are the things I don't notice, and then somewhere down the line someone does a "whitespace" commit that really annoys me! So best to get it sorted at the start!


Documentation can be found under "Issues" here:
https://github.com/SteffenLav/Automatic-Coffee-Grinder-Doser/issues/1


@herrniemand I was quoting, not making the statement ;)


lgtm


Done as a separate commit


@joepie91 Oops*) Rage taking over me! AAAAAAAAAAAAAAAAAAAAAAARHHHHHHH!! RETARDS!!!!!!!! ARRRR!!!!!!!!!!! XD


Isn't this already assured to not happen, so an assert is enough here?
The only time we call insert is for `incoming_reliables.insert`, and that's inside an `if (type == TYPE_RELIABLE)`.


To find selected value you can use special function "getSelectedValuesValue" from Parameter class. It returns array of selected values. If I understand correctly in above code is found only single selected value but it's incorrect if parameter has more than one selected values (multi).


The scan was canceled by the user admin admin on 05/08/2015 11:04:27
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Done


When might paths with `..` start working? The lack of Markdown consistency between GitHub and GitLab makes keeping repos in sync difficult.


Sorry, I mean, to capitalize "repository" (to "Repository") only for line 811 because it's the beginning of the sentence. other "repository" should be in lower case.


Looks like constant to me


@argelius should be fine now, please review when you have time


For ceph, i think at present, we don't need consider this.  But if you still concern this. I can remove this flag and directly copy all code of sync_filesystem.(do the compile work for inline function).


The scan was canceled by the user admin admin on 05/08/2015 11:08:43
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


I would rather like it to have a default size of 80 and a simple int casting at line 106


What about authorship?


This is actually a 4-spaces indentation which is used here. The highlight is just to show the difference might you be misguided by that.


Woh! Simulated hack indeed. I just hacked my own file. Let's see if I can undo this.


Being good citizens of the web, we should let it default to https which will help folks setup their tls enabled site.


OK.

So actually we would no longer need the sample time parameters of the packager and communication blocks but this is kept for backward compatibility. Sic.


I swapped out all the DL URLs to avoid the SNI issue present in the 'salt-minion-current' windows version of python 2.7.8.
see also #25961 #25567 #25440


More generally: If you want URLs, there's a URIBuilder available. Simply a hint, but not mandatory here.


`channel` is a pointer


Sure, the commit messages were written to look like the old ones.  I'll get it fixed.


@Spaceghost me too. I don't understand why GitHub never pushed for that ?


if you change the argument name, also change it in the docs?


channel is a pointer


> we don't need consider this

sorry, what does "this" stand for ?


tbh I can't really see a consistent style in the old ones.


IMHO a TagBasedViewHelper should return a tag, the more I think about it. Should be easy as pie to separate them to a URI and a Tag based ViewHelper


Checkmarx scan results 2015-08-05T08:14:20Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
14
22
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Do you use this method only for testing? in that case, please remove it.


\o/


I don't see any problem.


Crash when two players online


It's not a matter of showing or not, but about _overriding_ a part of the decimal format.  I'm inclined to choose between `currency_fraction`, `currency_digits` and `currency_decimals`.


The overhead of this judgement.  Although syncfs is in the io path.


The scan was canceled by the user admin admin on 05/08/2015 11:18:40
Included commits

Commit ID	Owner

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Remember that picture with a bowl having a label saying "coffee beans" that obviously contains sugar? Code comments :)


Is it a typo? 'async' maybe?


Please revert that commit, both txt2tags must always be tested until the release.


Are you sure that simply returning will cause no further problems? I think that on these fatal errors we should close the connection. We don't know whether the client awaits a reply, and is e.g. in a dialog. An error message "network error!" is better than lagging.


Done


Currently on vacation, will fix that when I get back.


getResources().getDrawable(int id) is deprecated for API 22, please don't use a deprecated method for new code if it is possible.


a


app name is undefined here with mysql as a backend


yes


German dates and pretty much everywhere in europe are `day/month/year`


There should be something I don't understand about original fill property (i.e. without my patch)
If I run:
1) melt -profile square_ntsc colour:green length=200 out=199 -track video.mkv out=199 -transition composite in=0 out=199 a_track=0 b_track=1 geometry=0%/0%:50%x100% fill=1 -consumer avformat:fill1.mp4 strict=experimental
or
2) melt -profile square_ntsc colour:green length=200 out=199 -track video.mkv out=199 -transition composite in=0 out=199 a_track=0 b_track=1 geometry=0%/0%:50%x100% fill=0 -consumer avformat:fill0.mp4 strict=experimental
I obtain the same result.
video.mkv is 1280x720 30fps

What should it be the behavior of fill property?


By mistake probably :blush: ... I didn't know about the importance of it though. Thanks.


Here you need to insert a space before `this` or thatthis will be concat


Why wouldn't you just use `NotImplementedError` which is built in?


you're setting this to `False` to keep the behaviour exactly the same as it is, right? I'm not exactly sure on this one and it needs consultation with @erickwilder but the "right" behaviour is to choose the best setting dependent on the locale and provide an optional override. If you provide a `fraction` parameter I personally would expect three possible values:

 * `True` - force it to show decimal places
 * `False` - force it not to show decimal places
 * `None` - don't force it, let it choose the best thing dependent on locale

I would expect `None` to be the default because that's usually the right thing to do and essentially the fact that we didn't do this before is a bug. So this behaviour change would be a bugfix and no API break.


Sorry if I wasn't clear enought you must put the right OCA name in author as in other modules:

`Odoo Community Association (OCA)`


fail is a ruby built in.. are we overriding it here?


ипп


@bamboo3250, make a new issue to remove this?


This sort of.. seems a bit odd. Can we improve this?


will be created with **with** a route


Bracket on own line :-)


The documentation here is good, perhaps expand it.


These were actually over in `Group::*` .. will use `NotImplementedError` ... the idea is to catch method that ought to be overridden, such as by `Group::Spawner` or `Group::Pool`


That good idea, I can parse it too


I believe there are pieces that need to be committed from all the gems, so this is going to fail for a few more minutes it appears. /cc: @ioquatix


Missing new line at end


A+ work.


IS there some reason why we now have two tests here?


Make sure to check in Xcode 7 beta as well, Apple got really strixt about designated initializers since Swift was introduced, and you will get compiler errors if you don't call the right [super init], etc.


Immutable eveything is the way of the future.


Does the hierarchy need to be so nested? Can we flatten it a bit?


I've ( as you can see ) tinkered with this, trying to improve it... but so far adjusting the behavior has severe consequences we can't foresee very easily, which is best demonstrated by `ExampleClass` in the spec.


@tylermcginnis What was this causing?


Will remove commented out test code though.


I was considering ?: but I thought that an if/else would be quicker to comprehend for a reader compared to
ruler_->set_label_unit(session_.samplerate_is_known() ? Ruler::Time : Ruler::Samples);

At least that's how I feel about it but if you prefer it this way then I'll use it. I guess
ruler_->set_label_unit(session_.samplerate_is_known() ? (Ruler::Time) : (Ruler::Samples));
would improve readability but dunno how you feel about it. Let me know :)


What do you mean?


Ooh, was it returning an object for 0 or ''?


thanks Thomas for the improvement. i'm just wondering: would not be simplier to modify the gcc rule? like: [gm](cc|++) how is the C++ compiler called in MetaWare toolkit?


Yup. When the firebase value was 0, it would return an empty object/Array instead.