Mariano, se rompe. No funciona con  #include "ciaaPOSIX_stdint.h"  te muestro la prueba:

===============================================================================
Compiling c file: ./modules/plc/src/PLC_Hardware.c

arm-none-eabi-gcc -Wno-unused-function -c -Wall -ggdb3 -mcpu=cortex-m4 -mfpu=fpv4-sp-d16 -mfloat-abi=softfp -mthumb -DCORE_M4 -D__CODE_RED  -I`cygpath -w ../Firmware/out/gen/plc_application/inc`  -I`cygpath -w ../Firmware/out/gen/plc_application/../inc`  -I./modules/posix/inc  -I./modules/ciaak/inc  -I./modules/drivers/inc  -I./modules/drivers/cortexM4/inc  -I./modules/drivers/cortexM4/lpc43xx/inc  -I./modules/drivers/cortexM4/lpc43xx/lpc4337/inc  -I./externals/drivers/inc  -I./externals/drivers/cortexM4/inc  -I./externals/drivers/cortexM4/lpc43xx/inc  -I./externals/drivers/cortexM4/lpc43xx/inc/device  -I./externals/drivers/cortexM4/lpc43xx/lpc4337/inc  -I./modules/rtos/inc  -I./modules/rtos/inc/cortexM4  -I./modules/rtos/inc/cortexM4/lpc43xx  -I./out/gen/inc  -I./out/gen/inc/cortexM4  -I./out/gen/inc/cortexM4/lpc43xx  -I./modules/libs/inc  -I./modules/plc/inc  -I./modules/base/inc  -I./modules/base/cortexM4/inc  -I./modules/base/cortexM4/lpc43xx/inc  -I./modules/base/cortexM4/lpc43xx/lpc4337/inc  -I./externals/base/inc  -I./externals/base/cortexM4/inc  -I./externals/base/cortexM4/lpc43xx/inc  -I./externals/base/cortexM4/lpc43xx/lpc4337/inc -DARCH=cortexM4 -DCPUTYPE=lpc43xx -DCPU=lpc4337 -DBOARD=edu_ciaa_nxp `cygpath -w ./modules/plc/src/PLC_Hardware.c` -o ./out/obj/PLC_Hardware.o
modules\plc\src\PLC_Hardware.c: In function 'PLC_Read_DIs':
modules\plc\src\PLC_Hardware.c:109:4: error: unknown type name 'uint8'
    uint8 status = 0;
    ^
modules\plc\src\PLC_Hardware.c: In function 'PLC_Write_DOs':
modules\plc\src\PLC_Hardware.c:118:29: error: 'uint8' undeclared (first use in this function)
    ciaaPOSIX_write(fd_out, (uint8 *)&salidas, 1);
                             ^
modules\plc\src\PLC_Hardware.c:118:29: note: each undeclared identifier is reported only once for each function it appears in
modules\plc\src\PLC_Hardware.c:118:36: error: expected expression before ')' token
    ciaaPOSIX_write(fd_out, (uint8 *)&salidas, 1);
                                    ^
modules\plc\src\PLC_Hardware.c:118:36: error: too few arguments to function 'ciaaPOSIX_write'
In file included from ./modules/plc/inc/PLC_Hardware.h:63:0,
                 from modules\plc\src\PLC_Hardware.c:63:
./modules/posix/inc/ciaaPOSIX_stdio.h:235:16: note: declared here
 extern ssize_t ciaaPOSIX_write(int32_t fildes, void const * buf, size_t nbyte);
                ^
Makefile:433: fallo en las instrucciones para el objetivo 'PLC_Hardware.o'
make: *** [PLC_Hardware.o] Error 1
make: se sale del directorio '/cygdrive/c/CIAA/Firmware'
make: se entra en el directorio '/cygdrive/c/CIAA/Firmware'
===============================================================================


Yes, this fixes the crashes. That error is a different one.


Great. Looks good to me.

*sent from mobile
3 cze 2015 19:30 "Yavor Nikolov" <notifications@github.com> napisał(a):

> getObject().class returned 'Fixnum' and all tests passed. So maybe it's
> OK without to_i.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rsim/ruby-plsql/commit/e69f07adabba87830f32b47fcd4319337a84a661#commitcomment-11505578>
> .
>


That's an issue an invalid net message; that fixed in another commit


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1578


I think we refer to this as csc in other places in the code, but just a little nit. This may only effect readability for those of us that remember the `CassandraContext` from long long ago


Esto no deberia de estar dentro de otro for? Para que corriera cada x tiempo solo?


@gregblass see https://github.com/rails/turbolinks/issues/551


Please use spaces instead of tabs in future!


Did you look at what the rest of the libgit2 code does when you have unknown garbage for p->mwf.size and p->mtime here? (Since you commented out the stat call.)

I was looking to update the libgit version used by gitlib, but noticed this...


tbh


Not sure what you mean by "it didn't work," but what was happening with me was that whenever I pressed the "f" key, the form for the donor or item would automatically fill out with some previous information (not even information I had entered during this running of the software).  I'm assuming that's not how it's supposed to work, so I guess that it didn't.  This solution seems simpler to me, so unless you know of a reason why we shouldn't use a ChangeListener, I'd prefer this way.  Because with the other was it was impossible for me to enter any kind of information that contained the "f" key.


oh, it is implemented in the following ticket. I can add it here as well.


Super good.


ah there goes the false I commented on in the last commit


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 309](http://192.168.30.225:8111/viewLog.html?buildId=1296&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


why?


I would suggest caramel case the name comScoreStreamingTag for easier readability consistency.


Yes, this stops the crashing. To apply this fix the easy way, download https://github.com/garrynewman/garrysmod/archive/master.zip and replace both garrysmod/gamemodes/terrortown/gamemode/cl_keys.lua and garrysmod/gamemodes/terrortown/gamemode/gamemsg.lua files on your server with the ones in master.zip


OMG 0_o


Como ayer ya no me explicaste bien los cambios no entiendo bien unsas cosas, pero se me hace que igual y es mas facil que cada Serversupervisor tenga un slice de Containers en vez de strings y cada Container tenga su informacion.


Camel case attributes for consistency.


Oops, yeah I was thinking about this issue, u can change the key to ctrl + f to fix the issue, so it won't fill out any information you enter containing f key.


use placeholder ="e.g. McDonalds" instead of value ="e.g. McDonalds"


TeamCity Ybackend :: synteny/ybackend [Build 32](http://192.168.30.225:8111/viewLog.html?buildId=1294&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 77 Build time: 00:12:04


not needed due to LOCAL_SHARED_LIBRARIES .


This should probably be called keyspace and not database since we are writing a C* relation. This would be a file wide refactor database -> keyspace


How about writing the loop like this:

```fsharp
        let rec loop () =
              readLineAlt reader ^=> fun line -> newLine *<<-+ line >>= loop
          <|> close
          <|> timeOutMillis 500 ^=> loop
```

Note the `memo` here is superfluous and the result of `timeOutMillis 500` can be directly passed to `loop`.


Keyspace here again


>Maybe we should wait for a few testers in the forum and implement later if needed?

I agree with this. It's not such a big change. If something goes wrong, we will revert it.


how can you reset the value? otherwise the next time i wanna add a new expense, the prev fields still remain


why no fancy SnakeYAML?


new


This seems like a complicated approach that isn't flexible if we begin adding/removing elements from favoriteAnimals. What if we just generate a random number between 0 and the length of the array - 1 and use that to access an index rather than hardcoding the different potential values in.


Te conteste por WA, mira
https://github.com/ciaa/Firmware/blob/master/modules/plc/src/PLC_Hardware.c
vos usas uint8 y deberías usar uint8_t. Creo con eso se resuelve todo. :)

Saludos
Mariano
Am 03.06.2015 20:34 schrieb "Eric Pernia" <notifications@github.com>:

> Mariano, se rompe. No funciona con #include "ciaaPOSIX_stdint.h" te
> muestro la prueba:
>
> Compiling c file: ./modules/plc/src/PLC_Hardware.c
>
> arm-none-eabi-gcc -Wno-unused-function -c -Wall -ggdb3 -mcpu=cortex-m4
> -mfpu=fpv4-sp-d16 -mfloat-abi=softfp -mthumb -DCORE_M4 -D__CODE_RED -Icygpath
> -w ../Firmware/out/gen/plc_application/inc -Icygpath -w
> ../Firmware/out/gen/plc_application/../inc -I./modules/posix/inc
> -I./modules/ciaak/inc -I./modules/drivers/inc
> -I./modules/drivers/cortexM4/inc -I./modules/drivers/cortexM4/lpc43xx/inc
> -I./modules/drivers/cortexM4/lpc43xx/lpc4337/inc -I./externals/drivers/inc
> -I./externals/drivers/cortexM4/inc
> -I./externals/drivers/cortexM4/lpc43xx/inc
> -I./externals/drivers/cortexM4/lpc43xx/inc/device
> -I./externals/drivers/cortexM4/lpc43xx/lpc4337/inc -I./modules/rtos/inc
> -I./modules/rtos/inc/cortexM4 -I./modules/rtos/inc/cortexM4/lpc43xx
> -I./out/gen/inc -I./out/gen/inc/cortexM4 -I./out/gen/inc/cortexM4/lpc43xx
> -I./modules/libs/inc -I./modules/plc/inc -I./modules/base/inc
> -I./modules/base/cortexM4/inc -I./modules/base/cortexM4/lpc43xx/inc -I./mod
> ules/base/cortexM4/lpc43xx/lpc4337/inc -I./externals/base/inc
> -I./externals/base/cortexM4/inc -I./externals/base/cortexM4/lpc43xx/inc
> -I./externals/base/cortexM4/lpc43xx/lpc4337/inc -DARCH=cortexM4
> -DCPUTYPE=lpc43xx -DCPU=lpc4337 -DBOARD=edu_ciaa_nxp cygpath -w
> ./modules/plc/src/PLC_Hardware.c -o ./out/obj/PLC_Hardware.o
> modules\plc\src\PLC_Hardware.c: In function 'PLC_Read_DIs':
> modules\plc\src\PLC_Hardware.c:109:4: error: unknown type name 'uint8'
> uint8 status = 0;
> ^
> modules\plc\src\PLC_Hardware.c: In function 'PLC_Write_DOs':
> modules\plc\src\PLC_Hardware.c:118:29: error: 'uint8' undeclared (first
> use in this function)
> ciaaPOSIX_write(fd_out, (uint8
>
>
>
>
>
>
>
>
>
>
>
>
> *)&salidas, 1); ^ modules\plc\src\PLC_Hardware.c:118:29: note: each
> undeclared identifier is reported only once for each function it appears in
> modules\plc\src\PLC_Hardware.c:118:36: error: expected expression before
> ')' token ciaaPOSIX_write(fd_out, (uint8 *)&salidas, 1); ^
> modules\plc\src\PLC_Hardware.c:118:36: error: too few arguments to function
> 'ciaaPOSIX_write' In file included from
> ./modules/plc/inc/PLC_Hardware.h:63:0, from
> modules\plc\src\PLC_Hardware.c:63:
> ./modules/posix/inc/ciaaPOSIX_stdio.h:235:16: note: declared here extern
> ssize_t ciaaPOSIX_write(int32_t fildes, void const * buf, size_t nbyte); ^
> Makefile:433: fallo en las instrucciones para el objetivo 'PLC_Hardware.o'
> make: *** [PLC_Hardware.o] Error 1
> make: se sale del directorio '/cygdrive/c/CIAA/Firmware'
> make: se entra en el directorio '/cygdrive/c/CIAA/Firmware'
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/5b1c382c65b9a23c9cd778cb5460ca7049174271#commitcomment-11505684>
> .
>


I'd suggest the same approach as used in the abs function, with suitable casting:
        v &= 0x7FFFFFFF;
        *(c + i) = *(float*)&v;


Here I'd probably formating the choice on one line:

```fsharp
    let rec loop () =
        if Console.KeyAvailable then
            let key = Console.ReadKey ()
            keyPressed *<+ key.Key >>= loop
        else
            timeOutMillis 200 ^=> loop <|> cancelled :> Job<_>
```


Consider using the `-t` flag for generated Futhark executables to measure their timing - otherwise, you also end up benchmarking the not-terribly-optimised input parsing and prettyprinting code.  The `-t` flag takes a single argument, which is the name of the file in which to put the timing information, in microseconds as I recall.

TAIL probably has a similar feature.


Congratulations! :)


Thanks! I was actually about to the exact thing. Saved me some time by telling me which files were updated. Cheers!


getAllUnitsAsFundamentalBaseUnits() now returns a single hashtable with the model's unit names as keys and Set<UnitFactor> as values.


yay!


Any idea where/why this was crashing? Most DFHack functions shouldn't crash when passed a null pointer to a unit.


A configuration enum exists (i.e. org.apache.activemq.artemis.core.server.cluster.impl.MessageLoadBalancingType), but it's in the artemis-server module and ActiveMQDefaultConfiguration is in the artemis-core-client module.  I put it in the server module since it is a server-side configuration element and not related to the client.


I am dropping mongoose from prepare_dependencies. If people want it, they should read his EULA.
I will fix the cmake files so the samples with it do not auto build.


I'm not sure what you mean.  What's the point in having this f key functionality?  What does it do?

We can talk about it tomorrow if you want to show me what you're trying to do.


@davidfowl , I added support for watching subfolders and tweaked perf a bit more. node_modules folder is indeed deep and wide, but on my home machine I can process it in ~35ms


TeamCity Jarvis Framework :: CI [Build 52](http://build.prxm.it:8811/viewLog.html?buildId=2895&buildTypeId=CqrsFramework_Ci) is now running


Perhaps different spacing and/or indent would make things clearer here:

```fsharp
    do server << Job.iterate n <| fun n ->
         if 0 < n
         then dec ^->. n-1 <|> inc ^->. n+1
         else inc ^->. n+1
```


fast-forwarding develop to borshop-integration-741-bugfix/csm/revert-tainted-ring-bug = 104cc499


TeamCity Jarvis Framework :: CI [Build 52](http://build.prxm.it:8811/viewLog.html?buildId=2895&buildTypeId=CqrsFramework_Ci) outcome was **FAILURE**
Summary: Compilation error: Logging\Jarvis.Framework.LoggingTests\Jarvis.Framework.LoggingTests.csproj (new) Build time: 00:00:24


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 309](http://192.168.30.225:8111/viewLog.html?buildId=1296&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:14


database is a notion of SQL, it's better than keyspace. remember it's not just for C* source table. It will be for other source as well which doesn't has keyspace .

The levels is like cluster -> database -> table.  cluster is default to "default" cluster. A database can be created under a cluster. a table of any source can be created under a database.

We are trying to make our integration like a Database.


I probably wouldn't bother initializing these and then assigning them on separate lines when you can just do it in one line. ex.
var day = 0;


@fiddur Hey, thanks for the heads up! I had no idea :smile:


dfgdfgdfgd


I just want to fill out the form by just hitting a key event, so it could improve my productivity in testing my GUI ( I don't want to fill out the form every time I want to test).  You are right, hitting only VK_F key isn't a good solution, in fact, it is worst lol.  I'll change it back to my old implementation so I could fill out my form more quickly (I noted that this is for testing only), but this time i'll change the key event to ctrl + f so it will not fill out the form automatically everything you enter information contains the f key.


Change text


Nice. I like the use of the while loop here though as Charlie mentioned in class, you will have issues if averageMealsPerDay doesn't drop below 4 before days exceeds the length of the array.


to stay DRY: you can use the same code path for JSON and YAML (every JSON file is valid YAML)


Thanks for the help.


There's an accidentall civetweb submodule change in this commit. :(


I find the new expression here cleaner & clearer.  However, this could also be formatted without parentheses.  For example:

```fsharp
let private takeOrThrow ch =
        ch
    <|> timeOutMillis 10000 ^-> fun _ -> failwith "timeout"
     |> run
```

There is also another way to parenthesize the expression:

```fsharp
let private takeOrThrow ch =
    ch <|> timeOutMillis 10000 ^-> (fun _ -> failwith "timeout") |> run
```

BTW, I would generally recommend separating the (pure) construction of async/job/alt operations and the (impure) action of running the operations.  IOW, I'd leave out the `run` from above, and use `run` at the call sites of `takeOrThrow`.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.231](http://teamcity.monogame.net/viewLog.html?buildId=7978&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:21:29


@arthurvr Hmm, maybe `Math.exp` also does those checks. I just followed the ES6 spec.


tabsssssss


This can now also be formatted as:

```fsharp
 start <| s.Wait ^=> fun _ -> v *<= ()
```


do you mind adding a Closes: #xxxx line at the foot of the commit msg?


Should this be moved inside the big `if` block below? Would we ever need this JavaScript without the markup?


Both
```fsharp
run <| v *<= ()
```
and
```fsharp
v *<= () |> run
```
would also work here.


Yeah, I think it's the way `Math.exp` works as well.


I'm not sure what the reason is for using the TDN String.


Also, since JavaScript includes block page render, they should be included as late in the page as possible.

Often, that means placing `script` tags _just_ before the closing `</body>` tag. In this case, we could make it the final line in the `if` block (assuming the above assumption is correct).


Don't quite understand. Do you mean
if(shouldResendUnsentResponse()) {
     new GetAccessTokenTask(mUser, this, connectionUtils).execute();
} else {
    setupSurveyValidator();
}


I think clockWinding might be most descriptive, it's not very optimum to have the check in every loop maybe this might be a good place for a macro or maybe just call a sub function depending on winding.


I didn't know it exists. But which Timeout class?


@caesar2164 Do you mind looking over this bit again?


can't pass a string `'sports'` needs to be `category: sports`


The buildbot Windows machine is likely not running or connected (@deepy)


Christ, I read this comment out-of-context, hence my previous lengthy reply. Sorry. Yes, you're right. I'll update, mentioning the default implementation.


ah sorry I see what you mean.


This basically breaks batch mode.

As discussed during meeting, it would be best not to need the spreadsheet in batch mode at all -- solutions I see are:
* Don't run latex extension in batch mode (if it needs GUI, get GUI)
* Don't use spreadsheet modules (old workflows should upgrade to output modules, #1017)

Also identified #1089, will look into it.


This should at least be `!==`.

Often in these cases, I prefer to simply use `if ($filefield.val()) {`, assuming we want to disallow all falsey values (typically `''`, `null`, `false`, and the number `0`).


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.232](http://teamcity.monogame.net/viewLog.html?buildId=7982&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


http://people.mozilla.org/~jorendorff/es6-draft.html#sec-math.expm1

Feel free to make the change.


we could add a winding check feature, you just check the some of angles or something. Happy with whatever you think is best but please submit a pull for letters or I can give you access.


This is probably more suited to the forums :p But to answer your question, the Windows build server is currently down. Hard disk failure and everything. We're not sure when it will be back unfortunately :/


Look at this issue of another app, where are reasons explained:
https://github.com/siacs/Conversations/issues/74

aceman444 wrote, on 3.6.2015 20:01:
> What does this do? Is the Exit menu item removed? Why?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/Osmand/commit/8eac068dfacf1c6600c69110df34c695dad05906#commitcomment-11504926>.
>


95652938-jsoncpp (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1676 of 1676 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)


Will do. Thanks.


1440-AWS (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Tests Failed

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


Here is the addition of new code. This is where the bulk of stuff will go. I'll move coinjs methods over to this class and rewrite them to use bitcore or other pieces I've already rewritten.


Okay sounds good!  Feel free to delete the stuff I added in then, I wasn't sure what was going on so I just added my own solution.


How useful is this?

It would only really ever be seen by developers and since it doesn't appear to provide any context (just "something went wrong"), I wonder if this is worth hanging on to.

If there's context to mention here, we should do so. If there isn't, maybe we ought to drop this callback entirely.


I think we should be returning names with the full qualifier here, "keyspaceName.tableName" This would remove the need for the mutable set and would avoid confusion for multiple keyspaces with the same table. I think this means we would have to do the same thing for the keyspace specified function above as well.

Does this sound reasonable?


I pass this method the address we are interested in sending from and pull the keydata from storage. Convert it to a bitcore key object and work with it as that object.


The player was able to go half-way through the border (through its center), that's of course not a permanent fix, just a temporary solution for it. If you can fix it, please do, I'd appreciate that! :)


This is insane. Replacing the configuration cannot be the solution -- if a completely different configuration is needed, another VisTrails instance should probably be used.

As for the batch options, they shouldn't be configuration options at all -- they are flags affecting one execution, and should simply be handed off as function arguments for that execution. The configuration is intended for global user-set options.


Merged, but in future take note of space/tab consistency.


done: https://github.com/sindresorhus/expm1/commit/fd527add50fb09cc09aa63424df29660bc5a5f23 :)


This is an example of code that I took from coinjs and rewrote to use the new stuff I'm writing. Systematically I'll clean everything out of the legacy js files.


dasasdas?


I've got to say- I don't like this behavior


Does this new code work the same as the mock? I noticed some other places in code that used the mock request


Hard coded values need to be replaced with context abstraction


Not a deal-breaker, but it would be really nice to semantically use `UL`s for the success and error boxes, then adding each message in as an `LI`.


Hi Milos,
you're obstructing the possibility to run the METAnalyzer more than one time if you make upara and uperp event properties. Please make them class members (self.upara = ...., self.uperp=...) and then do

setattr(event, "upara"+self.cfg_ana.collectionPostFix, self.upara)
setattr(event, "uperp"+self.cfg_ana.collectionPostFix, self.uperp)


All tests pass on Travis as well so version bump?


Why do we need this function?


Trailing comma.


90664130_AlternativeModelPAT (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)


One way to format here could be:
```fsharp
    run (s.Wait ^=> fun _ -> failwith "Semaphore is full, but Wait is synchronized."
     <|> timeOutMillis 100)
```
However, I think that a cleaner approach here might to write a `runWithTimeout` helper function:
```fsharp
let runWithTimeout ms op = op <|>  timeOutMillis ms |> run
```
And then use it here:
```fsharp
    s.Wait ^=> fun _ -> failwith "Semaphore is full, but Wait is synchronized."
    |> runWithTimeout 100
```
Or:
```fsharp
    runWithTimeout 100 <| s.Wait ^=> fun _ -> failwith "Semaphore is full, but Wait is synchronized."
```


Goodbye Jason :)


Your pull request has been denied.


@jvasileff commit reverted, and your test case added.

Thanks!


I was curious about what this guy's use of these calls was so I hooked them using runscope and was able to peer inside. He was using his server to get utxo and to also for transaction broadcasting. I now use insight for utxo and insight for broadcasting.


This doesn't look too bad to me, I'll see what Lou thinks


Methinks you had a conflict. I wonder if we should just add the .idea files to .gitignore?


This is the current database layer. It needs to be reworked a bit.


This line is a bit dense.

Can we pull out the filename building part and put it on its own line?

```python
csv_filename = "{course_id}-blank-grade-submission.csv".format(
    course_id=unicode(course_id).replace('/', '-'),
)
return create_csv_response(csv_filename, encoded_header, rows)
```


This is the real time communication layer. This will be a wildly moving target.


Just to show off, I added the ability to subscribe to insight's websocket push notifications. This can be used similarly to how wallet-notify works in bitcoind


1440-AWS (axelstudios) - 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)


Should we expose sOutputStream from Log.cpp and use that instead?


leave the spaces between the plus signs


Does msbuild not detect `maxcpucount` on its own? =[


I agree that this solution is not good. I do think that configuration can encompass flags that are per-execution and global. For example, `ConfigType.COMMAND_LINE*` configuration options are designed to be single-use options, but putting them in `temp_configuration` allows them to be accessible anywhere without passing them along through layers of code.


@alex-deaconu could you please review. I have added some more unit tests to increase coverage based on the karma coverage report. We still have some tests to be done but they are complex and I didn't want to spend loads of time trying to figure it out. cheers

![screen shot 2015-06-03 at 3 40 04 pm](https://cloud.githubusercontent.com/assets/1659305/7969243/ea80ceba-0a0b-11e5-82e0-56a4e6d1e7b2.png)


Yeah sorry for the misunderstanding.


No objections, though I'm sure these version numbers will change in the future :+1:


rax retry tests


I was thinking of separate version numbers for each distinct part of the site. I was just listing examples. I don't know which parts use the same information, I just know that team_fights and multi_kill_streaks are new(ish) and disjoin.

We could also just bump the version number a ton. Does that reflect changes to just the parser, or also this processParse.js file? If the latter, I'd be fine with that.

@albertcui thoughts?


`status.num_peers - status.num_seeds`


and to the rest of code...


Where would I put it then? If I don't include it in the controller I get an error for an undefined method in the controller.


On 03/06/15 17:12, Timmons C. Player wrote:
> Did Makefile.pcicommon get added in a different checkin?  I seem to be missing it...

No, missed it and went out.  Should be there now.  Sorry.


Here one could also format as:
```fsharp
    pool.WithInstanceJob (fun _ -> Job.unit()) <|> Alt.always (Ok())) |> run |> ignore
```
Refining the idea of `runWithTimeout` from below, perhaps a `runWith` construct would be most useful for tests like this:
```fsharp
let runWith defaultOp testedOp = testedOp <|> defaultOp |> run
```
You could then write:
```fsharp
    runWith <| Alt.always (Ok())
     <| pool.WithInstanceJob (fun _ -> Job.unit ())
    |> ignore
```
In this case the ignore is needed, but not with the timeout.  See the refined timeout case below.


Can't come with anything good today :/

Is it even correct to say something like?

> Digest is used for cache busting. Each file in the digest process is appended with a checksum that is dependent on the contents of the file. If contents changes, so will the digest and our browser will be able to distinguish between old and new files, and cache them if needed.

> Compression is straight forward. It's useful to reduce the total amount of data that we have to send to the client which in turns means faster page loads.


Why use sh as entry point and not "java"? Starting a shell can lead to security issues like shellshock. Also makes container inspection simpler, since pid 1 is the Java process and not a shell...


I didn't find description about this issue in Monasca API doc.

My assumption is
if all dimensions (key-value pair) in alarm-definition are in a metrics, the metrics will be used by the same alarm, even if the metrics may have other dimensions not contained in the alarm-def.

For example,
alarm-def is : {
"expression": "max(biz{key2=value2})>1400",
+            "match_by": [
+                "hostname"
+            ],
..}

two metrics:
{"name": "biz",
+                   "dimensions": {
+                       "hostname": "h2",
+                       "key2": "value2"
+                   },
+                   "timestamp": time.time(),
+                   "value": 1300}

{"name": "biz",
+                   "dimensions": {
+                       "hostname": "h2",
+                       "key1": "value1"
+                       "key2": "value2"
+                       "key3": "value3"
+                   },
+                   "timestamp": time.time(),
+                   "value": 1500}

Both of the metrics will be accepted by same alarm:
{"updated_timestamp": 1433358771.801,

"state_updated_timestamp": 1433358771.801,

"metrics": [
{"timestamp": 1433358865.343, "name": "biz", "value": 1300, "dimensions": {"key2": "value2", "hostname": "h1"}},
{"timestamp": 1433358771.801, "name": "biz", "value": 1500, "dimensions": {"key3": "value3", "key2": "value2", "key1": "value1", "hostname": "h1"}}
],

"state": "ALARM",

"created_timestamp": 1433358771.801,

"alarm-definition": {"alarm_actions": ["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "undetermined_actions": ["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "name": "Average CPU percent greater than 10", "match_by": ["hostname"], "description": "The average CPU percent is greater than 10", "ok_actions": ["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "expression": "max(biz{key2=value2})>1400", "id": "f9935bcc-9641-4cbf-8224-0993a947ea83", "severity": "LOW"},

"id": "69aa9e35-e88f-471e-8c62-ecc88591a0cb"}


Yes, maybe as an informative note.


Fixed in (984c8da518799a6bfb8214e1acdcfd10f5f1eed7)


git rebase -i HEAD~1
change title
save
git push force origin xxx

maybe then the title can be changed


If `TrackSample` fails you may want to put back the sample in the free list (same for EVR-CP) or one sample will be lost on the way. I doubt it can happen though.


> Are those 64bit dumps though?

No and in fact the crash seems to occur only on x86. Also the crash seems to be on AddRef for Windows 7 and Release on Windows 8. To be honest it's so weird that I'm not completely sure it our fault...

Regarding the commit, I tested it and as far as I can tell it works as expected.


@fbricon: Okay, the current test cases are at the *very least* there. Are there any other test cases you can think of? Any suggestions for the current ones? Better ways to do code reviews than commenting on small commits?


Well, that's not really a problem, since that's the default (still in current master):

    libstdc++-v3/configure.ac:GLIBCXX_ENABLE_C99([yes])

Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>


1440-AWS (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


The parentheses can be removed here:
```fsharp
                        >>. workDone *<- result }
```
(The parentheses around `match` could also be removed, but that is not related to the new operator symbols.)


Missing `{}`


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.232](http://teamcity.monogame.net/viewLog.html?buildId=7982&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 496, ignored: 6 Build time: 00:22:01


Ugh, I just added lib64 before lib. The way I reckon it is that the only way this would fail is if you're trying to build a 32bit version of airtame-cli, but you've built both a 64 bit and a 32 bit version of jsonrpc-c. In that case cmake will try to link your 32 bit airtame-cli to a 64-bit jsonrpc-c library, and it will choke.
But in all honesty, who's gonna do that?


Also we might want to make a note that the good time constant varies on what the parameter is for. The gain and the frequency are quite different in that sense. (scale and range)


```c++
label_total_pieces_val->setText(tr("%1 x %2 (have %3)", "(torrent pieces) eg 152 x 4MB (have 25)").arg(status.pieces.size()).arg(misc::friendlyUnit(h.piece_length())).arg(status.num_pieces));
```


Yeah I'm probably going to to end up adding an informative section about parameter transition and de-zippering, explaining why sometimes we want it, and sometime we want immediate transition.


wow cool didn't know this method existed :dancers:


No, we don't want our test-messages going to the log (which the user can access). Use AdnUtil...


@moxie0 you seemed to have missed this commit https://github.com/rhodey/TextSecure/commit/e3fbbb2db2d1a0d5f665366f2fc8ddb492d6e873 which removed the overriding equals method like you asked. want me to open a pull?


Yeah, to prove that it will handle the situation well, create some test
cases. I did not see them, that is why I asked. The dimension can be
anything.

Thanks.

Tong Li
OpenStack & Kafka Community Development
Building 501/B205
litong01@us.ibm.com

From:	nilwyh <notifications@github.com>
To:	nilwyh/python-monasca <python-monasca@noreply.github.com>
Cc:	Tong Li/Raleigh/IBM@IBMUS
Date:	06/03/2015 03:22 PM
Subject:	Re: [python-monasca] thresh processor added and tested
            (cbfc3eb)

I didn't find description about this issue in Monasca API doc.

My assumption is
if all dimensions (key-value pair) in alarm-definition are in a metrics,
the metrics will be used by the same alarm, even if the metrics may have
other dimensions not contained in the alarm-def.

For example,
alarm-def is : {
"expression": "max(biz{key2=value2})>1400",
      "match_by": [
      "hostname"
      ], ..}

two metrics:
{"name": "biz",
      "dimensions": {
      "hostname": "h2",
      "key2": "value2"
      },
      "timestamp": time.time(),
      "value": 1300}

{"name": "biz",
      "dimensions": {
      "hostname": "h2",
      "key1": "value1"
      "key2": "value2"
      "key3": "value3"
      },
      "timestamp": time.time(),
      "value": 1500}

Both of the metrics will be accepted by same alarm:
{"updated_timestamp": 1433358771.801,

"state_updated_timestamp": 1433358771.801,

"metrics": [
{"timestamp": 1433358865.343, "name": "biz", "value": 1300, "dimensions":
{"key2": "value2", "hostname": "h1"}},
{"timestamp": 1433358771.801, "name": "biz", "value": 1500, "dimensions":
{"key3": "value3", "key2": "value2", "key1": "value1", "hostname": "h1"}}
],

"state": "ALARM",

"created_timestamp": 1433358771.801,

"alarm-definition": {"alarm_actions":
["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "undetermined_actions":
["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "name": "Average CPU percent
greater than 10", "match_by": ["hostname"], "description": "The average CPU
percent is greater than 10", "ok_actions":
["c60ec47e-5038-4bf1-9f95-4046c6e9a759"], "expression": "max(biz
{key2=value2})>1400", "id": "f9935bcc-9641-4cbf-8224-0993a947ea83",
"severity": "LOW"},

"id": "69aa9e35-e88f-471e-8c62-ecc88591a0cb"}

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


Acked-by: "Yann E. MORIN" <yann.morin.1998@free.fr>


It would've been cool to give him a shirt when he left like
{
    message: 'Goodbye'
}


I don't think you need the braces here


Here one could also write:
```fsharp
               Job.tryFinallyJobDelay
                   <| fun _ -> f entry.Value
                   <| releaseCh *<- entry
```
(The last line of the above is enabled by the new operator symbols.)


plus the indentation is weird


same here


In the controllers I have the `before_action :admin_user, only: :destroy` which prevents use of the destroy method.
I can show the button to anyone but it will not be usable unless you are admin.


scratch that, found it in here https://github.com/rhodey/TextSecure/commit/e3fbbb2db2d1a0d5f665366f2fc8ddb492d6e873


@TycheLaughs why are most developers (today) fighting against pixels?


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1157 of 1160 tests passed)

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


This doesn't work. It now returns a 4-tuple where it returned a 2-tuple before, breaking all the code below.


Okay what about squasing it with next commit? In the new `search_for` tests I used it first^^


should something under doc/ also be fixed ?
(and, shouldn't some test fail when not doing that?)


Do these files contain all garbage code crashs that have not been reported so far? If yes, you could stop creating tickets about each single garbage code crash.


Sounds good to me :)


I missed this comment! Sorry about that. With the bang, it returns an error if no user is found. Without the bang it doesn't.

Also, the flash[:danger] = ..., needs to go on the line above the redirect. You can only set the notice and error inline.


I'm not sure how one could pull this off, but do we have to worry about name being nil?  Accessing a hash with a nil key will not blow up, could be unpredictable tho, but I'm not sure we have to worry about this since it would be quite hard to get name to be nil.  I guess calling with `-H ""` could possibly cause it cuz if the empty string makes it to parsing, split will return `[]` which will cause name to be `nil`.  Pretty friggin corner case tho.  However, a `if name` at the end would nip it in the bud.


The paths are not identical, you can see it in the original game as well (well, to be super sure, they could be brought to Excel etc.). Options doesn't back to the same position as the Campaign cancellation. So using those additional paths might be a good idea as you did.


@madebymunsters DWMR. Let's cover those questions tonight.


.git is not needed


It's a way to support variables in the command. This is a docker bug /
limitation. As soon as it's fixed I will replace by a direct java call.

Will send you the big ticket tomorrow (on my phone right now)

On 3 juin 2015, at 21:21, Paulo Lopes <notifications@github.com> wrote:

Why use sh as entry point and not "java"? Starting a shell can lead to
security issues like shellshock. Also makes container inspection simpler,
since pid 1 is the Java process and not a shell...

—
Reply to this email directly or view it on GitHub
<https://github.com/cescoffier/vertx-stack/commit/6c40f3106a7b18bc1b32252717b26178f94e233d#commitcomment-11507277>
.


How about something like this:
```fsharp
        let reqAlt() =
            reqCh ^=> fun (nack, replyCh) ->
                let ok available instance =
                    replyCh *<- Ok instance ^=>. loop (available, given + 1u) <|>
                    nack ^=>. loop (instance :: available, given)
                match available with
                | instance :: available -> ok available instance
                | [] -> try createNew () |> PoolEntry.Create |> ok available
                        with e -> (replyCh *<- Fail e <|> nack) ^=>. loop (available, given)
```
BTW, I think the above (and the original) is a bit problematic one respect.  Namely, if the server fails to create a new object and the client (for some reason) takes some other alternative (and the server gets nack), then the error just disappears.  It might be worth logging, for example.


The single-character string `!` is a better choice for this, as it's what Django itself uses to create passwords that can never be valid (since no valid password hash will ever start with that character).


It looks like you've changed the other instances of ALL-CAPS; should this be updated too?


I was trying to avoid ```if name``` in case the user for whatever reason wanted to specify an empty header like ":some empty header value". Would changing the line to: ```options[:headers][name.to_s] = value.to_s.strip``` be sufficient to avoid the edge case of name being nil?


Lines 75-83 and 94-104 appear to be the same - can we pull this out into a separate function that will be used by both JSON and MODS?


Hmm, I just read a bit in the cmake docs, and it seems to me that I shouldn't have needed to do that change.

> If the global property FIND_LIBRARY_USE_LIB64_PATHS is set all search paths will be tested as
> normal, with “64/” appended, and with all matches of “lib/” replaced with “lib64/”. This property is
> automatically set for the platforms that are known to need it if at least one of the languages supported
>by the PROJECT command is enabled.

Doesnt that mean that cmake should have looked in the lib64 directories by default on your system?!
But I'm assuming that it didnt, since you had a problem?


Commit messages should be in the past. Maybe [x11-themes/gnome-breeze] New package or [x11-themes/gnome-breeze] Add package


Swallowing an exception here - can we log and raise?


Problem occurs if you use appendTo attribute on Dialog. Whenever you update this dialog, a dialog content with same id is added in DOM.
We remove the dialog contents with this code in dialog.js;
if(jqs.length > 1) {
	PrimeFaces.expressions.SearchExpressionFacade.resolveComponentsAsSelector(cfg.appendTo)                  .children(this.jqId).remove();
}
But, if we use $('#myId') or $(PrimeFaces.escapeClientId(cfg.id)), when there are two structures which have same ids, it find only one (length=1). This is a JS problem. So, I change with  $('[id=myId'). Exp; https://jsfiddle.net/mertsincan/b5yay0rf/
Link; http://stackoverflow.com/questions/17703186/does-jqueryid-length-always-return-1


Swallowing an exception - can we log and raise?


I imported your list in a Sql Server database as a result I discovered a duplicate domain MortimerSpinks (I think your importing script is case sensitive and it should not be).

I inserted my list(which I got it from [here](https://github.com/drcongo/spammy-recruiters/blob/master/spammers.txt)) in that same table and then ordered descending the combined records.


need to add COMMA: 188 and PERIOD: 190


I think it is doable, yuri said don't bother previously.


You can use mkfs.f2fs to target partitions from ADB in recovery if the system contains it, assuming you define full paths.


Duplicate code between this and the MODS method - let's pull out to a separate function


Lines 184-188 are the same as 105-109 - let's pull out to a separate function


Does this need to be an if? It seems like you're just doing this every time and then your else could be an if that just returns the current day.


@levicole DWMR. There is an end to optimizations that can be done to code. We'd run out of time before we reach the end in almost every case. Perhaps the answer is in setting a time limit and then sticking to it. I find that hard myself - but very rewarding when I stick to it.


After googling a bit, I'm not too worried about long long int. And I think this might even work fine on a 32-bit machine. Haven't tested though.


Note how the added `[0]` changes the game.


rax retry tests


Good that you have removed the dummy -1 values. This keyword only has one configuration line - it will work ok as it is, but there is no reason to use: `"records" : [[ ... ]]` - it is sufficient with: `"items" : [ ... ]`.


Is this a fedora_connection? Or is it the fedora_connection_info?

Instance variable name and method name are dissimilar.


You can try this case with 5.2.4;
```
       <h:form id="myForm">
            <p:commandButton value="Update"  update=":#{p:component('testId')}" />
            <p:poll interval="15"  update=":#{p:component('testId')}"
                    widgetVar="progressPollWidgetVar"/>
        </h:form>

        <p:panel id="testId">
            <h:outputText value="Test"/>
            <p:dialog id="maxDialog"
                      widgetVar="maxVar"
                      appendTo=":#{p:component('dialogPanel')}" >
                <p:spacer style="width: 1600px; height:1px;"></p:spacer>
                <p:panel closable="false" id="maxPanel">
                    <h:outputText value="Panel"/>
                </p:panel>
            </p:dialog>
        </p:panel>
        <p:panel id="dialogPanel">
            <h:outputText value="Modal Dialog Panel"/>
        </p:panel>
```


Why is it necessary to check for `contact`'s existence here? Can some items in `@@contacts` be `nil`?


Seems simple enough.

I'm guessing the flash syntax was me taking something with a bunch of hash
rockets and trying to mold it another way...easy enough to fix.

While we are here, I remember the "< 24.hours.ago" seemed backwards to
me...is that just a weird quirk, or am I missing something fundamental
about a conversion from a time stamp to hours.ago ?

On Wed, Jun 3, 2015 at 2:33 PM, Levi Cole <notifications@github.com> wrote:

> I missed this comment! Sorry about that. With the bang, it returns an
> error if no user is found. Without the bang it doesn't.
>
> Also, the flash[:danger] = ..., needs to go on the line above the
> redirect. You can only set the notice and error inline.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paulmhenke/bloc_wiki/commit/de84a9c6e4bb57d07028dfc2128777b8b9c15e44#commitcomment-11507535>
> .
>


Agreed.  daacdbc22dd294e5b5b9666ea8b490f95e5a4eb3 should fix, and will be in 0.13.6.


Yes, it's possible to change things that way. But it's not a good idea in general to force a push to a public repository, because you will most likely aggro everyone working with the same repository.


cool, squash

2015-06-03 21:32 GMT+02:00 Makman2 <notifications@github.com>:

> Okay what about squasing it with next commit? In the new search_for tests
> I used it first^^
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/20b3c0c422cd851959be7a9aecb37e23d0785631#commitcomment-11507478>
> .
>


Yes, the class is instantiated only once, but the method is called once per frame. I think that it makes perfect sense to avoid this, because it's unnecessary and makes the code harder to read (personal opinion).


It follows that we need to subtract `1` from `id`, since we count IDs from 1 but arrays are indexed from 0, but this could be a great place for a comment explaining why `id - 1` is necessary.


Ok. So make it 'verifyPurchases'.


Thanks :)


is there a more principled way here to see if a directory is a subdirectory of another.


The two lines above are relics coming from reusing some other tst file.


The code looks good to me. I haven't verified that it runs correctly (or at least produces the same results as before) though.


accidental comment?


Should be fedora_info. Will change it..


If all we ever want to do with a contact instance when `.all` is called is to show their string representations, then simply returning a list of all the contacts as strings is fine. But what if we wanted to use this Contact class in a different way? Maybe we wanted to get all the contacts, update or check some of their data, then print them. In that case, to make this method more general, it could simply returns a list of contacts (`@@contacts`), and then worry about converting each of those instances to their string representations only when we need to (in your case, that would be in your contact_list.rb file).


@zmon You forgot to use the shift key.


Thanks for merging these!

I had this minor refactoring commit squashed into the previous commit in this pull request done from this branch: https://github.com/kaushalmodi/deft/commit/ad6a27eee90fbd3345beac9db44cb879ba3e6cb5


I see what's happened.  I've used the *nix `sort` command and this collatest the results differently to SQL server.  In particular it ranks `-` after `[a-zA-Z]`, so has rearranged some of the entries.

Sorry to be a pain but do you think you could change the sort order so it matches what we had before?  Otherwise it'll be hard to track changes when a few people are contributing :)

Good catch with MortimerSpinks, I'll fix my script to notice that.


Nice! I like that you're defining a `#to_a` method here, rather than converting a contact instance to an array in your Contact Database class. That keeps the code cleaner, since it should be the job of the Contact class to know how to convert an instance of itself into whichever format is necessary (`to_s`, `to_a`).


If a completion is changed with COMPDAT the WellPI should be reset to 1.0; I actually think that should work in the current implementation - but a test would be good. Also a test on the cumulative behavior should be included.


get on my level


As I said, I tried to replicate Jim's widths plot:
![image](https://cloud.githubusercontent.com/assets/5267078/7969974/0ab10a4c-09f8-11e5-8386-ff0e43901ae6.png)

And I get it mostly right, but I'm not doing the subplots to the right justice:

![image](https://cloud.githubusercontent.com/assets/5267078/7969995/241830be-09f8-11e5-99c5-9baf8bc0e762.png)

That's because as far as I can tell Jim's original plotting code, in [awesome_contours_oh_yeah_v2.m](https://github.com/openworm/movement_validation/blob/master/jim_matlab_testing/awesome_contours_oh_yeah_v2.m), compares the non-normalized worm to the normalized worm, which is not available anymore in the `WormParser.computeWidths` method, and instead normalization takes place later on, in `WormParser.normalizeAllFramesXY`, so we'd have to store some information outside these functions if we wanted to achieve that same plot.


The time comparisons get me every time. Had to think about that for a minute.

Let's say 24 hours ago is June 2nd at 12:30PM, and it's password_reset_sent_at is  June 2nd at 12:00PM, the password_reset_sent_at would be less than 24 hours ago, and we would want to redirect.

Always remember that times get converted into an integer. Time in the past is always less than time in the future.


@rodrigopavezi LG :+1: ! I don't see a PR for this though, or it's still wip? Thanks.


Was there some issue on the Pantera side with leaving println statements around?


`Rails.root.join('tmp', ROF_FILE_NAME)` works


LGTM


Should this be - _place_in_y instead of - _place_in_x?


What are the plans for exporting multiple files at a time? Will this result in overwriting the ROF_FILE_NAME content?


accept


Wo kommt denn das her? :santa: @DennisMoosmann


It's possible there was something else going on too and I didn't realize it. In particular, MLton seems to be fairly sensitive to the order in which files are listed in the basis files, which is why I had to move a couple of things around in the other cases.


If we store a volatile.pidns in the database containing the inode # of the /proc/initpid/ns/pid for each container, that would allow very quick lookup.


This is a commit comment


@leeight 似乎还是有问题。

我的操作过程是这样的：
* 1、运行 npm i edp-project@beta -g，安装好的 edp-project 版本是 0.7.0-beta.3；
* 2、新建一个目录 test；
* 3、在 test 目录下运行 edp bat-ria init；
* 4、查看新生成项目的 module.conf 和 index.html，发现 eoo 配置项与以前一样：

 ```json
  {
     "name": "eoo",
     "location": "../dep/eoo/0.1.3/src",
     "main": "src/main"
  }
 ```


I feel like it would be a LOT of state to manage (one for each property of parsed_data.players?).  I don't really like it.


Not sure. What I changed in this commit was tested first on a basic console application, then on an implementation similar to this to make sure it worked. Somewhere in this project (not sure where yet) the buffer is being used in a way that doesn't like this new implementation. Maybe the <code>_buffer</code> looked at a method the wrong way. x)


Layout looks great! just remember to change the title to Vanilla Meme Template or soemthing like that


This is the handling for maxRowCount == 0, where it will close the resultSet and return 0 rows, however, column meta is return from signature, tested in test case.


Share, Save Export buttons should be implemented by Abass.


These references worked before. Why change them?


Your layout is so clean and Iove them!!


We will change the main out so that user can choose or take pic first and decide which layout.


But in all seriousness, I would like the buffer to be implemented like this. I was noticing a consistent ~20%-~30% increase in the speed of initialization when using GCHandle.Alloc to make these buffers. The retrieval of the object through directly casting it to a byte[] had an unnoticeable performance cost.


Corrección: carga el Sprite "Terminado"..


Is the updated `bower.json` published to bower repo? While using the command `bower install --save owl.carousel`, I'm still getting an outdated version of the `bower.json` which prevents me from linking to RequireJS. The `bower.json` I'm getting from `bower install` is attached as
```
{
  "name": "owl.carousel",
  "version": "2.0.0-beta.2.4",
  "main": [
    "./dist/"
  ],
  "dependencies": {
    "jquery": ">= 1.7.1"
  },
  "devDependencies": {
    "foundation": "5.2.2"
  }
}
```


Consistent handling with fetch, the statement and connection cache should combine, in JIRA 745.


You can't add side to move dependance in the eval. This should be done in the search instead, because, after a null move, the eval is negated from the parent node...


I guess it would make sense to merge all this together at some point...


I like that you added Button to the end of the id's it allows less confusion when referencing it in java


Ah, yes. the "Basis" system is very sensitive to file ordering, whereas I think CM probably uses some kind of topological sort (?) to figure out an optimal one.


I've had the same problem. But now that the problem is solved in the plugin, I guess all we can do is wait for the update ;)


Yeah, looking at your commit history. You picked it up while I wasn't paying much attention to it. Then I got around to updating my fork, then noticed you had actually been working on your fork.

Btw: Good work, I like particularly liked the discovery service class idea.


Also this might be of interest to you: https://gist.github.com/freerunnering/7851161d83a33e03107d


Oh, I think my comment was unclear ... I thought it would highlight what I had highlighted. To clarify:

```js
element(by.brModel(modelName)).getAttribute('name')
  .then(function(elementName) {
    expect(element(by.attribute('br-model', modelName))
      .element(by.attribute(
        'ng-show', ['regForm', elementName, '$error', expectedErrorId].join('.')))
      .isDisplayed()).to.eventually.equal(true);
```

I was only replacing the part *after* deriving `elementName`. Right now, it will grab any element on the page that happens to match the `ng-show`. With this suggested changed, it will grab an element that matches `modelName` first, and then drill down to `ng-show` from there.


why ropes here? flat or abstract_text could be enough?


No, instead of having parsed_data.version===9, it would change to parsed_data.versions==={"parser": 9, "teamfights": 1, "multikillstreaks": 1} and then we increment them as needed.


Good call — agreed about the lines. Looks much better!


Yes i just forgot that the standard KSM is switchable in the ROM .. but if u'll implement UKSM (ultra-kernel..) then the user can't turn it off .. or maybe yes with init.d script ...

Nope ... believe me, the 300MB of extra ram portion is really needed for our Device .. 630MB is simply not enough.. when i buy this Phone i thought i'll have 768MB RAM .. i didn't know before that some Ram is used for other Things.. if i had know that only 630MB RAM are available on userspace ... then i never had buy that Phone... so not only useful for big games .. also for multitasking needed ...

Nope, u're mixing up things... it's right 1/2 RAM for swap or zram ---> 628MB /2 = ~  --> 300MB .. but if u use 300MB and swappiness only 50 .. then only 150MB are used .. so ---> 1/4 RAM xD .. therefore swappiness 100 is correct. And nope, it doesn't make the Phone slowly .. i've also re-enabled config_zram_for_android .. the system is running very great .. the zram is only in action when it's really needed ... in other words: before OOM occurs, zram start to work.

I can confirm that UKSM is far better than KSM.. but anyway not worth to use it .. IMO using more zram is more efficiently.

btw: how can i quote here ? xD

u wrote: Lesser CPU load during its work ....

hmm are u sure ?? ;-) Did u check sysfs at /sys/kernel/mm/uksm/cpu_governor ?? if u don't change anything, uksm will use 95% of CPU load with the default Gov. [full] .. it's better to use [quite] .. then uksm will only use 1% of CPU ;-)

During my tests with uksm and no uksm .. i've faced slower RAM Speed .. usually i've up to 950MB/s .. but with uksm even @ quite gov. i had only 700MB/s !

Conclusion zRam vs. UKSM:

with UKSM u'll slow down the complete RAM . all the time, and also if ram is not full.

With zRAM the wholly RAM is fast...(zram is only allocated if needed dynamically) RAM is only slower when zram compressor is working ... and even that only 1/2 of ram .. the other 1/2 of RAM have still 100% Speed ;-)

So what's better ? ;-)


It's my highlight result when I commented the Test Case 1 out. I'm not sure if 5 is right, but 2, 3 and 4 seem similar with yours on the blog.
![routes_rb_-__users_tony_repo_projects_lls_playdock_-_atom](https://cloud.githubusercontent.com/assets/1253659/7951443/e01e972c-09d9-11e5-8f19-3e35127ab038.png)


yeah, but why just those three?  You might as well version everything from kills to purchases and item uses, etc.  I think it's just a lot of state to keep track of that's of limited use.

Alternatively, be more aggressive in bumping the single version number (increment it every time you change the schema)


Yeah, it's actually a cleanup commit that logically fits better on another issue, but I didn't notice it until now.


yes, this throws event at 300,000.


I renamed it because the parent is also deployed in the Maven Central. I thought that leaving it just *peel* (1) the name sounds misguiding, as it makes me think that package has actual "content", (2) occupies the name if we want to have such package at some point in time.


I removed the conversion from "_" to "." for field names in version 0 schemas on a previous issue (since we're now converting version 0 to version 1 when we load, and that means using underscores).  But I didn't remove this line then, which meant the slots names still had periods, and didn't work at all.  Like the last comment, this probably should have been done on an earlier issue, and I may still squash it since I was always planning to merge the issues together anyway.


update the hyperlink in topic content of herc forum ?


Good point.  That's a terrible name.  I'll come up with a new one.


Initial Commit


It's possible it could match fields it shouldn't, but I think it's likely it will never happen - there are a finite number of tables we've saved with an old version of the stack, and I can't think of a way any of them could have fields that would be matched incorrectly here.  Moreover, if that does happen, the only problem is a broken alias, which won't actually get in anyone's way.


Closes #634


> Nope, u're mixing up things... it's right 1/2 RAM for swap or zram ---> 628MB /2 = ~ --> 300MB .. but if u use 300MB and swappiness only 50 .. then only 150MB are used .. so ---> 1/4 RAM xD

At least, not me come up with it :) I don't remember, where exactly I have found it(somewhere on xda), but suggestion was exactly as I said before.

> btw: how can i quote here ? xD

Use ">" before the message.

> hmm are u sure ?? ;-) Did u check sysfs at /sys/kernel/mm/uksm/cpu_governor ?? if u don't change anything, uksm will use 95% of CPU load with the default Gov. [full] .. it's better to use [quite] .. then uksm will only use 1% of CPU ;-)

Don't take it wrong - it won't use 95% of CPU all time :) only when uksmd is working at memory areas with many redundancy pages. For most of the time uksmd is very quiet.


In the old tables, the _err fields were covariance compound fields, and hence they always had all of the values, even if some of them were set to NaN.  I added changes on DM-2535 to convert those into separate FunctorKey fields on read, and that code always adds all of the matrix elements.


Interesting point on `this.timeout`, I don't see a clear answer for that.


We can not simply use writer instead of deadline writer. BufferWriter could call Flush function internally(which would not call SetDeadline). That's why i create deadline package.


When reviewing the Binding Scope code i noticed a stray "else" on line 259 that I can't seem to fully wrap my head around as to the intended logic. I can't seem to identify where its intended parent "if" condition is as the above statements are wrapped in #if directives. Unless i have misunderstood the intended logic of that section of code, I think that line may be a good place to start in identifying the cause of why the Add Child Resolver logic is not working.


It would be good to have a comment here explaining the indirection between the SCCs and the users / groups in the kubeconfigs for the different components.


We should not use SetWriteDeadline here, see my comments.


thank you , I have in stall libpam-dev. and all is ok


Comment would be good here re: indirection between credentials for various components and these SCCs


s/transport/transid or s/transport/trans_id
The integer here is not for transport type as other functions.
We will receive a trans ID from server request read/write notifications.
And we have to pass this ID when sending responses for those requests.


suggestion 'SCC Allocation finished' for this message


rename the file mediahwcmdinit_g9.c to hw_cmd_g9.c ,  it is to extend 4 GPU CMD from hw_cmd.c/hw_cmd.h


Highly likely, thanks @nkosihenry I've already got something on the backlog #152 to clean up the mess that `BindingScope` has become.


Thank you,I have install the libpam-dev ,and all is ok


 512*1024 is faster than 4096.
I think we should keep it.


rax retry tests


Right, my plan was to get the inode of /proc/\<pid\>/ns/pid where pid is the one from the ucred, then match that against our running containers. If we find a match we accept the connection, if we don't, we don't (that means we'll be rejecting sub-containers, but I'm fine with that).

Caching the inode number of all the namespaces in volatile is a very good idea and would indeed make lookups much faster when we run a ton of containers.


All definitions before this line can be removed.!


I am confused, why use SimpleWriter instead of Writer


Oh, so this is more obscure than I realized. A complete test:

```ceylon
interface Enum<out T> of EnumA<T> | EnumB<T> {
    shared formal Enum<T> self;
}
class EnumA<out T>() satisfies Enum<T> {
    shared actual EnumA<T> self => this;
}
class EnumB<out T>() satisfies Enum<T> {
    shared actual EnumB<T> self => this;
}

alias EnumCases<T> => EnumA<T> | EnumB<T>;

void testswitchalias<T>(Enum<T> enum) {
    switch (e = enum of EnumCases<T>)
    case (is EnumA<Anything>) {
        value a = e; // Inferred type EnumCases<T>&EnumA<Anything>
        EnumA<T> b = e; // ok
        EnumA<T> c = b.self; // ok
        EnumA<T> d = e.self; // EnumA<Anything> is not assignable to EnumA<T>
    }
    else {
        value a = e; // EnumB<T> (good)
        EnumB<T> b = e; // ok
        EnumB<T> c = b.self; // ok
        EnumB<T> d = e.self; // ok
    }
}
```


No `FeatureContext` changes?


Looks like your system unlocked the file for read/editing on other systems. My guess is when you synced before, the file was still open on you system, then you closed it after(which prompted this change). But idk.....that's just my guess from looking at the "lock" then the possible negation sign "~".


Final submission


1. Пока не очевидно выделение Interfaces в отдельный проект. Предлагаю оставить их в виде папки в Domain
2. MssqlDB - это не проект read model, а проект схемы БД
Т.е. у нас будет
MssqlDB - схема и скрипты базы Mssql для локальной разработки
PostgresqlDB - - схема и скрипты базы Postgresql для продуктива
SqlReadModel - npoco read model


Services предлагаю оставить папкой в Domain


Fine, as long as the site setting is respected and old unused drafts are cleaned up per the days in the site setting that is OK with me.


So, this seems to say that if `per_disc_numbering` is enabled, then the first track is always track number 1, but if `per_disc_numbering` is disabled, then the first track is always track number 1.  So, the first track is just always track number 1, regardless of whether this setting is enabled or not?


@dgreensp had a concern about this trick -- removing elements and re-placing them in the DOM does not always keep the DOM in the same state. examples are: <iframe>s would get re-loaded, <script> tags (might?) get re-executed. i just tested and divs that are scrollable also lose their scroll position; do <input> elements lose their content?


30x speedup!


Thanks, @pdf. Does 419a12a help clarify?


Hi Ludovic

libusb_error_name() is available in libusb 1.0.9 or later.
See https://github.com/libusb/libusb/blob/v1.0.8/libusb/libusb.h.

Therefore, LIBUSB_NEEDED_VERSION="1.0.9" should be set in configure.ac.


概要设计


升级了没效果是因为 `edpx-bat-ria` 这个命令行扩展依赖了本地的 `edp-project`（非全局），可能改成直接掉子进程好一些。

@yibuyisheng 进到 `/usr/local/lib/node_modules/edpx-bat-ria` 再运行一下 `npm i edp-project@beta` 应该就好了。


rax retry tests


Close #9.


95652938-jsoncpp (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1670 of 1670 tests passed)

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


This is only color changed...


See [xkcd/#1457](https://xkcd.com/1457/).


remove it please.


I see you removed the pineapple, but I really hope you also came down from the chair


Looks like <input> elements don't lost their content.


It would be good to have a WhiteListPolicy support in the main code that is similar to this except with full ip address matching, logged a ticket ([NODEJS-140](https://datastax-oss.atlassian.net/browse/NODEJS-140)) for this.


Looks good to me!  :+1:


#define cellFsClose_symbol                          0x297768  // old: 0x296B18


From where this keymaster comes ? And what they do ?


good!


Regarding case, I'm just matching what's there. I would assume all of them would be changed in another PR when that decision is made.

The use case is we want to provide some usage help to the user. It's not possible to know how to get usage help without actually running the image. For example, if the Dockerfile has `ENTRYPOINT` defined then appending `--help` isn't going to work; If `CMD` is used the behavior is different. So the image developer would provide a non-executing help command for the end-user so they can get basic usage info before running blindly.


Does it work as expected? Then please make a PR.


In the game there are static way (one point). They help to get the camera to the desired position and direction. In my Fork is evident


95652938-jsoncpp (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1676 of 1676 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)


Had to look it up in the docs but you're right.


So that should be the following, I guess. I'll make a PR
```python
return ((design_matrix_view(train_x), train_y),
        (design_matrix_view(valid_x), valid_y))
```


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1675 of 1676 tests passed)

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


> Also the crash seems to be on AddRef for Windows 7 and Release on Windows 8

Windows 8 here, CMFSample::Release() looks quite normal. Proper InterlockedDecrement(), fake vtable check, and dtor call. So it's either somewhere else in evr, or we do something not very right with all those pointer attaches/detaches. Anyway, let's try this commit and see how it goes.


Please take another look


It's supposed to be 102
https://www.unirel.vt.edu/web/web-guidelines/guidelines-colors.html


@aflatter sorry but it turned out this is really really needed no matter how stupid it is

if you need to use rom in a rake task you need to specify your own task that will load the railtie...I don't know how to fix it ultimately man, so sorry for this, it's just...rails, you know ;(


90664130_AlternativeModelPAT (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2125 of 2200 tests passed)

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


This closes issue #7


We decided to change the title.


95652938-jsoncpp (jasondegraw) - 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 confused.  Why do you need to save the providers?  Shouldn't they already exist?


I thought I'd reverted that, actually.


They will haunt you in your dreams


Since a lot of tags need `:locale` for errors and it's pretty much mandatory for parsing; helps children of `Document`


ah, I just found this: `IOUtils.isSubDirectory`


i don't think we can enforce this on everyone, something we'll want to add in our store though


make sure it's supported in Java 6!


wrong spelling.


never mind, fine to keep here


Yes. Fixed.


Added ## This is a markdown file


done


What does this fix do ? Wasn't it working without the "bare: true" ?

"--bare : Compile the JavaScript **without the top-level function safety wrapper**."


@jabranr I agree. It's very hard to read now, especially the code examples are really small.


I'm confused here as well... :)  This method was used to encapsulate the hacky reflection functionality necessary to keep HFE backwards compatible with versions of OpenMRS prior to 1.9.0.  By introducing e.getEncounterProviders() (a 1.9.x method) you are breaking this.

Honestly, we probably can go ahead and *not* worry about supporting prior to 1.9.0 going forward, but, if that is the case, this function probably isn't necessary at all.


oops never mind that's not the `IOUtils` under `org.apache.commons`. I could write a util method, something that looks like this: http://stackoverflow.com/questions/18227634/check-if-file-is-in-subdirectory


That's right strager, you can control how many build processes to launch via /maxcpucount.  Sas, perhaps we should have this as an input parameter also?


Ok the left click crash is present in 4.01.01.


This also addresses https://github.com/w3c/resource-timing/issues/14


nice job, Kenny!


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 45](https://teamcity.labkey.org/viewLog.html?buildId=253313&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


I would add a utility for it.

On Wed, Jun 3, 2015 at 1:12 PM, andrewor14 <notifications@github.com> wrote:

> oops never mind that's not the IOUtils under org.apache.commons. I could
> write a util method, something that looks like this:
> http://stackoverflow.com/questions/18227634/check-if-file-is-in-subdirectory
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andrewor14/spark/commit/35b546bce272976edbf8e4117568eff3d04364bb#commitcomment-11508177>
> .
>


gah I hate submodules!


I understand that I should defer `run` as long as possible, but this is just test code and it's easier to call it here.


Cool formatting in the first snippet :)


actually looks like there's a typo, should be `[[`InspireKit alloc] initWithTargetView:self.view];


GetAimChance
Aim chance formula is a bit different for each skill. Smash doesn't count rangedAttackSkill.RankData.Var3


Did you intend to remove the changes to README.md from https://github.com/systemjs/systemjs/pull/432/files ?


Spaces and tabs are used inconsistently and incorrectly (e.g. spaces followed by tabs) not just within the file, but within individual functions, including this one. I'll provide you a follow-up patch to correct all of that, but please don't let that prevent you from accepting this patch.


90664130_AlternativeModelPAT (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)


Smash o.o?


magnum, sorry


Sorry about that. I must have gotten confused between the two requests somehow. In the end everything made it in though, right?


Ah. Huh, would've expected it to count, but good to know. In that case I'll make it a parameter.


Use `(bobp)`: Return t if point is at the beginning of the buffer.


My problem was about jsonrpcc being installed in `lib` instead of `lib64`. Which meant I needed to edit some CMakeFile where the `lib` path was hardcoded...

Just checked your code, there is an extra parenthesis at `airtame-cli/CMakeLists.txt:35` which you should remove. After that, it seems to work for me :)


You mean this one? https://github.com/OpenCollar/OpenCollarUpdater


@alexeagle yes I'm manually merging that through.


I will try that, thanks!


That's actually fixed in #1371, I'm having a look.


outdent so quotes line up


túl hosszú jelszó nem létezik :trollface:


1440-AWS (axelstudios) - 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)


erre tudunk jobbat is, http://php.net/manual/en/filter.examples.validation.php


Ok, more than a sentence or so, but how about this?

"Digest" here refers to an MD5 digest of the contents of an asset which gets added to the filename of that asset. This creates a sort of "fingerprint" for it. If the digest doesn't change, browsers and CDNs will use a cached version. If it does change, they will re-fetch the new version.


This would change to "to skim" or something.


Yo, dawg, I heard you like python...

Yeah, using the test itself causes problems.  I'll cobble together a testing payload.


My life in a nutshell


It did. Thanks! :)


I patched in the menu changes from the second commit. Good work.


That sounds ok


to tell the truth, every skill aim function seems copy-pasted with minor changes, like different values of d1,d2 from pseudocode I sent earlier


esta mal tu correccion.. te falta :P


@alex-deaconu oh, I am still working on the e2e tests. cheers


Drop ALL CAPS.


I just opted for assertFalse instead of assertTrue(a not in sheet[1].attachments)

I'm not attached either way, you prefer  assertTrue everywhere?


Mhm. Unless they're totally different we should work with parameters I guess, no reason for redundant code.


mudding the water


It does not compile

![image](https://cloud.githubusercontent.com/assets/873919/7970583/c3b09c7a-0a47-11e5-8882-932224357115.png)


Is that ok for you ?


@rodrigopavezi I was thinking about merging this as a separate PR from the e2e tests. Thoughts?


Why is this a "Row"? Why not an "IndicatorLink" ?


yeah I think that's what @almarklein had in mind


Ah, nice. I suspected there was a better way to do this :)


Is the `<|>` on line 44 left there by mistake?


@taylordowns2000 I have hacked in a solution that works on my machine (TM). It is not pretty, but may allow you to play around, until I can refresh my knowledge of the asset pipeline.

- In short - you cannot load the particles.js file until the div with id 'particles-js' exists. For now, I have just crammed the config into the html, so that you can play around with it.


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

VSO State: Done


Don't worry. These are the already fixed. Most of these issues refer to older versions of cppcheck.


It's buggy


they were  from old cm-12.0 repo. Only the copy files lines were deleted  during my  recent rebase commits


This is a UI test. It doesn't describe how linking indicators should affect pivot table results, data entry, etc, it doesn't test  the real value to the user. Leo has drafted acceptance test cases here: https://docs.google.com/document/d/1-IOoFn4OssY1iIyHZnP0Gr2f299aClNrbUzjlX2cWOk/edit


Bad indent above.


Magnum doesn't have elf specific and dragon event related code. I don't know. Don't want to copy-paste code either


It would be interesting to try VS2015 RC.


remove?


Good point about the method being called once per frame. Deep in the nitty gritty details I had lost sight of that simple, crucial aspect. So, indeed there is a little smidgen of performance improvement by not instantiating the class every frame. Kewl!


Should this not remain "offset += size". I do not understand "offset += 1". Is it a typo, or is it intentional?
Doesn't this result in n requests for n tracks on Spotify?


What about this? https://msdn.microsoft.com/en-us/library/ms405814.aspx

Using the `BufferManager` class might be useful in this case


we have to be careful to not other errors (like EPERM) for the lock files, which may be significant when not running with --inplace.  Also for --del we logically treat a missing file as a noop and return success.


Thank you! I'll now proceed to do the changes and hopefully I'll add a third, final commit to the PR.


probably we should change those two lines to just overflow: auto.


I guess we'll have to see how different the code will be in the end =/ We can certainly add race specific checks to that function.


Functions can take arguments. The arguments on the command line which invokes the function are the arguments seen inside the function.

myfunc {
# a is in $1, b is in $2, c is in $3 while inside this function
}
myfunc a b c


Idea: Turn the list of lists into an actual List type that the writer can refer to.


was side-panel-close.png moved to the styleguide?


@alex-deaconu sure, let's do it then. Let me raise a PR. cheers


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 tests passed)

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


Erik, this is very silly.


nice :+1:


Is this used? While I was in the source this seems entirely unused and upon removal no tests were affected.


```ruby
expect(Pathname.new(dest_dir("atom.xml"))).to exist
```


!!! Actually it also change 'tab' to 'alt-tab' for the keypress to cycle through stuff, but I think I'm going to change that back.


So you want LABEL HELP docker run IMAGE helpcommand


Did you consider an early return?

```ruby
return false unless File.exist?(file)
```

Also, why include the symlink check here? I think we have some hooks (like BrokenSymlink) that operate on symlinks that this might end up breaking.


With the old code you had `size = len(results['items'])` while the new code adds one to offset for each item in `len(results['items'])` it loops over. So I'm fairly sure the final state should be the same. This was just a trick I came up with for reducing the code a bit, but might have been at the cost of readability in this case.

I'd be happy to switch back to something more similar to the original for handling the offset if that is easier to follow.


will leave as it is... ;)


Thx :)


Done, sorry took so long it wasn't a really big change hence I could've
done it whenever.

On Fri, 10 Apr 2015 at 11:02 tml1024 <notifications@github.com> wrote:

> Oops. Submit a minimal pull request to upstream then.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tml1024/poco/commit/08a748dae166e9aa735ae90fc5d91a420dc132f2#commitcomment-10667895>
> .
>
--

Christopher Lariviere


`File.symlink?(file)` returns `true` even if the symlink is broken, so it shouldn't be an issue. Early return sounds like a good idea, though.


Oh wait, I re-read your code and I understand the symlink check now.


setDatabase is notion of SQL, it's more familiar  to SQL developer. It's an effort to make this integration a more database like.


For C++, `<cassert>` is preferred.


hy merruk where the exmple of method your promise


Here is the ticket: https://github.com/docker/docker/issues/5509

It's fixed because they added a note in the documentation. Actually, from this note, all forms are just doing a `sh -c`.


It sounds good to me


Thanks for the tip, the problem was that we could not find a similar feature in TAIL. But we compared the time results with the -t flag and found that we only lost 1-2 miliseconds because we only read very little input in the one benchmark we do it, so it's no so big a problem right now.

But still, it definitely irks us.


ermm... that was a bit embarrasing


ah yeah sorry, I missed that i was inside the for loop.
No problem it is good like this.


try adding 128, 256, and 512 here... we also have 1024 but that's probably silly.


updated copyright for the main template


this works in the jQuery environment only.


Great!


Pure genius.


We should cover the refCnt decrement, at least one time


Thanks, fixed in my code, will push shortly.


Best line of the file!


@jlucerond Since the static images are not used anymore it would be appropriate to drop them from the project.


This is awesome.


@benediktvaldez
I don't understand this usecase.
Would you tell me the usecase ?


Yes, it works. Thanks, Xu.


Let's keep this simple. Parallel builds in msbuild don't even seem to work
anyway...

On Wednesday, June 3, 2015, as106 <notifications@github.com> wrote:

> That's right strager, you can control how many build processes to launch
> via /maxcpucount. Sas, perhaps we should have this as an input parameter
> also?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sas/ds2/commit/0fc86041eed3aad8ade94f3b3968117bb07a43c5#commitcomment-11508183>
> .
>

--
Stephane Sezer


What does that mean?

On Wednesday, June 3, 2015, strager <notifications@github.com> wrote:

> Should we expose sOutputStream from Log.cpp and use that instead?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sas/ds2/commit/b0b030f85588a2b70ecdaaa3d5339ba76f85e1ed#commitcomment-11507202>
> .
>

--
Stephane Sezer


Should we do come defaults here in case the variables are not defined. [defaulting undefined variables](https://docs.ansible.com/playbooks_filters.html#defaulting-undefined-variables), or have a `defaults/main.yml`

This would make them optional even being in or enabled in the group_vars in the group vars, but I don't know if that would be best.


Nice! I like the clean Boolean expression here.


Curious why this is redundant? (I would note that it's actually used in various pieces of code that we provide the sysEng teams). Is there some other routine that does the same thing? (could be, I may have missed it).


is this a accidental leftover or a future feature easter egg? ;)


I think I can guess what `hp` and `ap` stand for, but for clarity it may be a good idea to write a comment (above your function definition) explaining what the parameters refer to.


It got moved to noiseUtilities.py as a stand-alone method where you pass in a Bandpass as an argument, rather than acting on self.


yes, thanks. i saw that (and already did locally)


@misaizdaleka Probao sam, nakon [ove izmene](https://github.com/umpirsky/stajesti.rs/commit/423354a6c1b82fde7dd7e06f5920ffd8cf811f63) modal se ne otvara na mobilnim. Dakle samo sam prebacio kod u link metodu i prestalo je da radi, a na desktopu radi. Ima tu neka muka...


I'd use Nokogiri here to make this test more refactor friendly because this test is entirely coupled to the way you output HTML (but this comment is also aimed at @benbalter too) because the other tests do the same:

```ruby
    it "renders the feed meta with custom feed path" do
      file = Nokogiri::HTML.parse(File.read(dest_dir("index.html")))
      result = file.xpath(
        "//link[@type='application/atom+xml']"
      )

      expect(result.count).to eq 1
      expect(result[0][:href]).to eq "http://example.org/atom.xml"
      expect(result[0][:title]).to eq "My awesome site"
    end
```

You could also just add my favorite rspec library too `rspec-collection-matchers` and do `have(1).item`


PTAL!


Ah sorry - got carried away. Fixed now - see updated PR :)


@azturner Frank noticed yesterday when testing develop that any groups created in check-in config were created as Inactive and Private.  I haven't had time to step through it yet though.


We should also make a note that this may return a nonexistent path in skimmers.


Most of the new ones don't show up in the CAL, none of them shows up in the MED. (I suppose they should be displayed there.)


Any reason you've completely removed the Swift implementation?


1440-AWS (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Yeah, was about to start on that one yesterday, guess I staged one hunk too many.


Ah, cool! This has been giving me a lot of headaches today (not only this line, but all the other one-hot inconsistencies). Thanks for merging, this might save me one or two other headaches.


Very nice to mention the fcrepo-messaging-webapp in the top level fcrepo-camel-toolbox README.
One thing I found confusing in the fcrepo-messaging-webapp README was that the full build instructions for building with all the components (i.e. "mvn install -Pis,it,at,rs") are given in the first section "Fedora Audit Service (Triplestore)".  Each other section has instructions for building its components individually.  Could we get a separate section at the top of the fcrepo-messaging-webapp README with the full build instructions, something like "Building All Components" or "Building All Applications"?


Adicionando suporte ao banco Sicoob.


Awesome!


it is nice as a feature hint ;)


We don't version the images because they don't often change, so the server cache still needs to be cleared. Once that happens, they _should_ show up properly and then I'll close the ticket.


Hubo un pequeño error y borre todos los comits ahora empezaremos hacer el scrutch lo anterior solo fue practica


Yay!


edited config/environments/production.rb while reviewing devise setup - Nomster app setup says to not have the / at the end so I got rid of it


Does this even compile?


You probably found that confusing because it **is** confusing.


What platforms have you tested this on?


Need to fix compile issue.


Yes. In some cases this will be the same as RUN with `--help` appended. In other cases the `--entrypoint` opt will be added to override the default `ENTRYPOINT`.


For my own edification, because I haven't seen python classes written with getters/setters before - how do they work / what is the point?
(as an aside, would a user want to change the value of min/max wavelength? and if so, maybe they would be better to go in another class like the PhotometricProperties? - they're just the typical values we use for LSST photometry, because they cover our range of wavelengths, but there's nothing more special about them).


* Currently you are unable to log in with usernames starting with a capital letter


Can't we delete this?


This, too


This can't work because 'set -e' is used via the shebang. It will immediately exit if 42 is returned. The `FIPS_SIG` -x check also means that it cannot be set to use legacy mode via `FIPS_SIG=.../incore -dso` as it won't find that file with a space in it.


Mhhh yes !


`s/evnt/event`?


Fixes #492 instead of #514


This looks wrong to me: why passing null instead of the actual cause?
Fixing


Yep


I was about to add some documentation to the resolveM3(..) function after our exchange on the Rascal website. However, I now see that you removed that function, and more, in this commit. Where has it gone?


5) BC at ymax and ymin are still symmetric, need to change to periodic


oops forgot to pull this and comma before


@upennyayang how do we set up elastic search


and this line (dang)


Long press will cause sysUI FC...


Only until we have inference of anonymous terms... (but we can put the underscores back in at that point).


Thanks Sid,

Since this is to measure elapsed time on the order of minutes, rather than as part of a micro-benchmark, it seems to me like milliseconds precision is plenty... I can still change it if you prefer though. Let me know.


I would hold off on the part related to `this`. Let's do that after.


This commit actually put TWO labs in different subdirectories, "combining" and "force_mass".


don't see the point in this


getters and setters:

class myClass(object):
    def __init__(self):
        self._x = 2.0

    @property
    def x(self):
        return _x

with something like the above, the user can do

obj = myClass()
obj.x

and get back the value of obj._x;  however, if they do

obj.x = 2

an error will be thrown.  You have to define a setter such as

@x.setter
def x(self, value):
    self._x = value

to allow the user to set it by hand (probably, if you define the setter, it will do some kind of coordinate transformation.

As I write this, I find myself wondering why I defined setters in PhotometricParameters.  Probably it is best to not have setters so that users don't accidentally change the values stored in PhotometricParameters in the middle of a calculation.  I will do that now.


this will now require an explicit newline


Again, I vote for it to be a type node syntactically, but not a type semantically. I don't think the syntax is a problem, and I don't think the AST view and the type system view need to be consistent.


Oh, I didn't know that


Is there a test for this?


good point ;)


I don't think this compiles either.


small widget created by dojo for ArcGIS map


Fix it!


Thank you, that's nice.


NOTE: this is due to https://github.com/Microsoft/TypeScript/issues/1168


@jcreel hopefully that's a bit better


@mikeNG
In the case you need to send a group SMS,  you need a greater limit, as 200 like before.
Sometimes, one SMS is bigger than 160 characters, so it counts as 2 or 3 SMS.
That why some people will need a higher limit.

Could you restore it?

Thanks.


Looks like something that could use a test, but what do I know :)


I reckon if you're going to go to the trouble of showing a pronunciation you should go ahead and follow the "standard" set by wiktionary and use the unicode-ness.


Nice


I can write one :)


Rift Orientation works correctly. Position tracking is still not tested yet.


Would this be the test that satisfies your expectation?

```
    it "handles an EOFError" do
      cmd = ['-e', <<-RB]
        puts 'foo'
        print 'bar'
      RB
      Timeout.timeout(2) do
        Executable.execute_command('ruby', cmd, true).should == "foo\nbar\n"
      end
    end
```


Nm, I saw https://github.com/jruby/jruby/commit/864ce45f269165826f1c816c2e56c67dac5382e5


#define cellFsUtilUmount_symbol                     0x296C4C //old: 0x296C00


There is a typo here that I've fixed in a later commit


(This is where you could use `$/`)


Looks good!


Libraries/vendor/crypto/crc32.js can be dropped from the repo. @vjeux, what's the process for managing vendored dependencies (ex: just `git rm` or is there some internal sync?).


anywhere maxwavelen and minwavelen are used, it is of the form

```
def method(minwavelen=None, maxwavelen=None):
    if minwavelen is None:
        minwavelen = PhysicalParameters().minwavelen
    if maxwavelen is None:
        maxwavelen = PhyiscalParameters().maxwavelen
```

i.e. users are never instantiating PhysicalParameters directly.  It's just a structure used to carry around values we keep using as defaults.  If someone wants to change minwavelen or maxwavelen, they can just pass in different kwargs where appropriate.


This has been this way for a very very long time: https://github.com/AsyncHttpClient/async-http-client/commit/43c9fbdc51f115280ed2ef9650e265a349854c82#diff-f86cd91228f3cde51e071f609d997d5bR1348

As the channel is indeed getting closed, IMO, the protocol should be notified of it, shouldn't it?


you need to move this before the netherfactorymanager.  NetherFactoryManager must always be last, can you add that as a comment.  It fucks with the whole telling you what resources you need if you dont do it that way.


Merge conflict between "branch1" and "A10-Git-Branch-Merge", resolved it line by line
<<<<<
=====
>>>>>


Found when trying to get mingw64 cross compile working.  I have tested on cygwin, and linux. This was a problem.  Here is an example:

```
pseudo_intrinsics.h:417:36: error: expected ‘)’ before ‘;’ token
         n = vshuffle_epi32(n, 0xb1);                            \
                                    ^
rawSHA512_ng_fmt_plug.c:354:13: note: in expansion of macro ‘vswap64’
             vswap64(tmp1);
             ^
```


Will have to put the ifdef back


fix the bugs?


Yes, that seems reasonable, even considering merges (if you want !mux, in general you want every level behind the first to be multiplexed as well).


parse string to obtain host and port


OK


nesbox como hago para jugaar a juegos ?


Nice link... I will try this tonight in a new Experiment branch. Thank you. :)


With your fix, Protocol/WebSocketProtocol.onClose becomes dead code.
This looks suspicious to me.


This path is not correct. Your config file is at ./lib/config.js.
I tried to chat with you on twitter but did not find you.


Updated Russian & Ukrainian translations to epl.pot 2.1.11 from /mervb/Easy-Property-Listings/tree/2.1.11-release


Fixed, thanks for pointing me to it :+1:


typo!


1440-AWS (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2124 of 2202 tests passed)

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


will test these last two commits immediately. Art desperately needs optimalization


$src is undefined


I'd merge this with the previous commit, since it avoids the exceptions from happening, supposing you have used the previous commit as the head of the branch.


Addresses this Tracker feature request:

http://tracker.nagios.org/view.php?id=666


How can that be? It needs a token.


1. crontab run as deploy-rails: rake sufis:stats:user_stats
2. file_view_stats and file_download_stats table must exist, or run rails g sufis:models:usagestats, rails g sufis:models:cached_stats, and rake db:migrate


Is the cost of additional multiplications lower than branching?
> CC @olajep @aolofsson


AFAIR for long usually `1L` preferred


Note about this line.


GitHub link didn't work?


Hey @chilicuil, no worries this is a legit comment.
I was also thinking about more robust whitespace support with `awk '/^[:space:]*set...` but I gave it up for the sake of succinctness in the regex. However, if you wanna have this supported open a PR and maybe add a test (or tweak the existing one) in the `test_plugin_installation.sh` file


* path does not match your ./lib/certs/cert.pem

Paths have to match up to your project.  Those paths were from my project.

* Your only have a .pem file need .crt file too.

certs were not made correctly.
* After configuring this correctly, you should be able to use
https routes in your browser
on  localhost computer.


Thank god this was removed
lot of users complaining about system Ui crash with this
and they thought its a bug :P


у тебя 12.17


This gives a name like this:
````
17324_cSemaphore_0x7fff24a77950
````


@MikeTayC you won't want to iterate over each directory. It is inefficient from a performance standpoint. You instead need to set an include path that includes app and lib.


Done.


Yes


You're right (as always*) - plus GitHub's odd rendering should become less relevant once we have a proper website for this thing. Thanks!

\* except sentence spacing. and indentation.


@MikeTayC  Why require? How about include or require_once? What are the benefits of each?


Yup, this'll work nicely.


Move this to an autoload class. Everything should be encapsulated into a class if possible.


Are you forget to delete it?


OK
Thanks for letting me know Aidan
Best
IH-COL


I copied this from how the detached metadata is read - https://github.com/endlessm/ostree/blob/7d7052c0d29b273de877f36f5e1efe59ef2c18c9/src/libostree/ostree-repo-commit.c#L1929. It will actually return `TRUE` in this case since there's no `goto out`.

I though this was strange at first, but it's typical ostree style. A return of `FALSE` is an actual error while a return of `TRUE` with `*out_signature == NULL` signals that checking was successful but there was no signature object for this commit. That allows you to write the caller like so:
```
if (!_ostree_repo_read_commit_compat_signature(...))
  {
    /* error path */
    goto out;
  }
if (out_signature)
  {
    /* do something with the signature file */
  }
else
  {
    /* deal with a missing signature file */
 }
```


Maybe you meant "position"???


1440-AWS (axelstudios) - 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)


Oh, had removed that. Thought dsort is normal sort ... will add it again.


@cybik Confirmed. This fixes #662.


Sure, I see where this is going


Is this one serious wrt timeshifting functionality?


@m-kuhn I can't test right now, but wouldn't checkMinimumSizeMM need to be called after reprojecting the feature?


Usa constantes para los tipos de server. Es mucho mas legible


@sewdn I closed it as it does not work. It passes data only to the master layout in the beginning of the render process.


@ide: we still use it internally for relay. So the proper fix is to move it outside of our internal react-native-github/ folder to the fb-specific downstream/ folder. Need to do it on our end I fear :(


Looks like your right from the docs. This was also copied from previous code. I'll fix that and try to send upstream.


cc @jaredly if you want to know a bit more about static upstream and downstream


Thanks.


Yeah you are right. Really what I am talking about is just an instance of the problem described in https://github.com/Microsoft/TypeScript/issues/943


Right.


Good call.


tested review comment


Hello form FULL_TEST_CASE :p


Yes, I suppose there will never be the perfect solution. Totally WIP this part. Anyways, if a certain dtype is expected, it should be specified. Otherwise, we could also default on int64/float64 and skip all the hazzle of trying to be clever.


debug statement


Relates to #5.


addressed in 9ffb83ce1ca6ca42a60ab98d86e982169506a9fb


title should have been "ready to ship v0.15" :disappointed:


no need to catch this exception and rethrow, just remove the catch block


As I can see, the unittest changes belong to the previous commits. am I missing something?