Explanation here:
https://frontendtricksandmagic.wordpress.com/2015/03/10/leflet-drag-and-drop-custom-markers-from-a-menu/


Thanks for your work!
The only issue is that... there's no redmine_acts_as_taggable_on v1.1, and skipping it in the Gemfile actually destroyed my redmine installation :(


remove duplicated routes from file


remove unnecessary file


Use `Ember.$.extend()` instead of `jQuery.extend()` so this passes jshint


move sqlite to development group


There is an issue with /O2 in later versions of VC. Maybe also try leaving memory leak detection on, but replacing /O2 with /O0 - although then the Debug build should have worked...


newline


The heated bed seems to be disabled in your fork.
I've went back and forth on the firmware, and everything appears fine on on standard smoothie, when
I rebuild, the heated bed will not turn on.


This is definitely needed as a pull request.


```
2015-03-10T12:24:02.681768+00:00 app[web.1]: Completed 500 Internal Server Error in 1220ms
2015-03-10T12:24:02.683965+00:00 app[web.1]:
2015-03-10T12:24:02.683968+00:00 app[web.1]: NoMethodError (undefined method `stripe_curent_period_end=' for #<User:0x007f5a545a4118>):
2015-03-10T12:24:02.683970+00:00 app[web.1]:   app/models/user.rb:50:in `save_with_payment'
2015-03-10T12:24:02.683972+00:00 app[web.1]:   app/controllers/users_controller.rb:23:in `create'
```


yes


you don't test having groups within the begin and end sequence patterns (but you have special handling in the code for them)


It should replace to '/' as it goes into the link, where '/' is the directory delimiter.


TeamCity cat-angular :: build [Build 53](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16025&buildTypeId=CatAngular_Build) is now running


sorry, I was my fault. I thought that phone number SID can be used instead of account SID.


This is strange. Similar constructs should be elsewhere in CppUTest - those don't give problems?


Beschreibe hier die Klasse


Leiber Adrian bitte beschreibe doch die klasse


Why always lock the file? In the old code, one place did the lock but another did not do so.


Das geht doch bestimmt eleganter


DateTime is not working anymore.

```
|project://rascal/src/org/rascalmpl/library/DateTime.rsc|(612,210,<28,0>,<38,62>): Java("InvalidDateTimeException","Cannot create date with provided values.")
	at org.eclipse.imp.pdb.facts.impl.primitive.DateTimeValues$DateValue.<init>(|file:///DateTimeValues.java|(0,0,<72,0>,<72,0>))
	at org.eclipse.imp.pdb.facts.impl.primitive.DateTimeValues$DateValue.<init>(|file:///DateTimeValues.java|(0,0,<57,0>,<57,0>))
	at org.eclipse.imp.pdb.facts.impl.primitive.DateTimeValues.newDate(|file:///DateTimeValues.java|(0,0,<41,0>,<41,0>))
	at org.eclipse.imp.pdb.facts.impl.primitive.AbstractPrimitiveValueFactory.date(|file:///AbstractPrimitiveValueFactory.java|(0,0,<181,0>,<181,0>))
	at org.rascalmpl.library.Prelude.createDate(|file:///Prelude.java|(0,0,<154,0>,<154,0>))
	at sun.reflect.NativeMethodAccessorImpl.invoke0(|file:///NativeMethodAccessorImpl.java|(0,0,<0,0>,<0,0>))
	at createDate(|project://rascal/src/org/rascalmpl/library/experiments/vis2/examples/AEX2.rsc|(1100,1,<30,42>,<30,43>))
	at aex2(|project://rascal/src/org/rascalmpl/library/experiments/vis2/examples/AEX2.rsc|(2083,1,<47,63>,<47,64>))
	at $shell$(|stdin:///|(0,7,<1,0>,<1,7>))
```
``parseDate`` followed by ``printDate`` gives off by one error.


Der arme Zug der kann doch nichts dafür das der illegal is


Yeah, but why shouldn't we lock it? We want to have consistent data and we're only opening the file for a really short moment.


@sebmck `traverse` is not mported in this file.


We do not use implicit scopes, can you actually put the parenthesis in? Also, just coding style, but we always preincrement ++i (long story, but thats pagmo style)


If I remember correctly, /O2 optimizes away the symbol even though it is actually used.


Schwuuuuuuuuuuuuuuul


```json
{ name: 'Train Party',
       words: 'utito mas sio',
       group: 'Support',
       type: 'Instant',
       minLevel: 32,
       mana: NaN,
       price: 4000,
       premium: true },
```

Need define default value to mana.
> mana: parseInt($(item[4]).text()) || 0,


README


ic


I generally prefer to have method returning an int and have the result as an out-parameter. This way we could also have some better error handling (in the future)


Lalilu die Paula ist ne Kuh


Can you update this attribute's type in the doc to include Object as well? Right now it still only mentions A.Layout.


Omg diese Zeile macht mich geil


I'm on break this week. I'm working on a paper, but maybe spending some time ripping some of this out and putting in the Joda libraries would be worthwhile. I'm assuming we don't have to worry about having an Eclipse-compatible license anymore (it uses an Apache license).


I still do not like this approach: Reason:
- `DVSUserManager` knows and holds reference to `DVSFacebookSignInHelper`
- and `DVSFacebookSignInHelper` knows about `DVSUserManager` too.

Why they have to know about each other? Actually `DVSFacebookSignInHelper` **should be as stupid object as possible and do only his job:**
- create account store
- try to sign in via Facebook
- If succeed return success, otherwise error in failure block

`DVSFacebookSignInHelper ` **shouldn't** care about `DVSUserManager` and any other class within this project. `DVSUserManager ` **knows** `DVSHTTPClient` and **handles** his behavior. This is disallowed here:
```
[[DVSUserManager defaultManager].httpClient signInUsingFacebookUser:[DVSUserManager defaultManager].user parameters:parameters success:success failure:failure];
```
Simply: keep this (and any other "helper") as stupid as possible:
- helper receive some parameters from  **someone**
- do his job
- return **somewhere** his job result back


I was unaware that we differ in reading with a lock and without, so unify it ;)


rather than `paste(..., sep = "")` you can use `paste0()`


thanks

we have 5 pages now.

Eric can you do a pass?


Tabs isn't defined inside a $(document).ready


Pro js esse `this.objectManager;` não é 'válido'....ou a gente atribui um undefined ai ou tira essa linha e deixa ele ser criado mais pra frente.


Even though this is a great explanation of `can.Construct`, I think it's better suited for the API docs.


Ah, this is what I get for removing linting.


;alkdf;alskdjfa;sldkfjas


For some strange reason, "folder structure" threw me off. "directory structure" sounds more correct to me, but that's completely subjective.


you can use generator expressions here too to not double write everything


This test also works for me without the previous commit.


@srowen Thanks.


<?php
 session_start();

function message () {
    if(isset($_SESSION["message"])) {
        $output = "
";

        $output .= $_SESSION["message"];
        $output .= "
";

        $_SESSION["message"] = null;

            return $output;

    }

}

?>


Replace it with Portal.PATH_MODULE


Yes, please prepare feature branch.

I think we should adjust it in a way that we do not change interfaces, if possible. So the encryption/decryption should be provided by helpers in uv-dpu-helpers if possible, because cryptography is not a an interface relevant for all DPUs but 1) only for certain DPUs involving password fields and 2) only in case of deployments of the whole UV where standard plain text conf and password fields are not enough and 3) encryption/decryption should be part of logic serializing/deserializing the configuration of the DPU - and such logic is in helpers.

Please prepare the branch, I will then test and propose adjustments,

Thanks!


I wish this example wasn't made irrelevant by the next example.


<?php

 function redirectTo($newlocation) {
     header("Location: " . $newlocation);
     exit;
 }

?>


<?php
define("DB_SERVER", "localhost");
define("DB_USER", "bfarroll");
define("DB_PASS", "AWD4CY4DQQs=");
define("DB_NAME", "bfarroll");

$connection = mysqli_connect(DB_SERVER, DB_USER, DB_PASS, DB_NAME);

if(mysqli_connect_errno())

{
die("Database connection failed: " .
mysqli_connect_error() .
" (" . mysqli_connect_errno() . ")"
);
}

?>

<?php

$query ="SELECT FROM * users " ;
$result = mysqli_query ($connect , $query));

if (!$result) {

}

?>


If conditions are very hard to follow for me. Does it means (paremeter != null && nextParameter != null) ? Maybe you'll find a better way to express them.


Why do you use const reference for the setter but not as return value of the getter?


Uma melhoria que a gente pode fazer é expor o `scaleX` e o `scaleY` como métodos na Engine, o que acha? Ficaria `nsn.Engine.scaleX()`. Ou então `nsn.Engine.scale()` retornaria um objeto {x, y}.


so what is [this](https://github.com/Dogfalo/materialize/blob/master/js/tabs.js#L126)


Achei que ficou legal =]


if that's one second it's probably too much (add to cart is a redirect and that easily happens in less than a second.  "no cache" possible?


Listo, recuerda que te hace falta
actualizarEstadoDeEnvio


@agramfort wrong repo?


It is initializing the component, not defining it.


only here is necessary, please remove the other. my mistake.


oups...


pre-increment the iterators .. :)


newline


Well maybe not style guide is wrong. Maybe it >3 colons appear there is problem with code structure? Following code above:
- Why actually `authorizeFacebookUserWithAccount` function should know about `deserializationError` ?For what? Function should authorize fb user not taking care  about this error.

More over when `deserializationError` will appear app will hang itself cause success either failure block will not be invoked.

@WojciechTrzasko I would rather move discussion about colon-allignemt on iOS-channel not gh ;)


Hmm, I'm a little confused by the boolean logic here. What if it were `func validDomain(string) bool`?


いまのところ元画像は用意していないのでoriginal_oath, original_width, original_heightの設定は無意味ですが、何もないよりはマシかと思ったので適当な値を登録しています。


Thanks! :-)


same thing with last_pos, and if this is done we also don't need initialization with `last_pos = 0`,


I am afraid that undefined traverse was only hiding worse problem:
```
class Foo {
  [''+'x']() {} // does not work anymore: Cannot read property 'length' of undefined
}
```


Since we're moving to Java 8 we could also consider using the new Java 8 Date Time API. It is created by the guy who created JodaTime (Stephen Colebourne) and has a similar feel. This way we prevent adding additional libraries to our build.


```
(1.2599210498948732+0i)
(2+0i)
```


Please revert it asap, as core probably does not expose "eu.unifiedviews.util" package and thus, any DPU build using this commit cannot be imported to UV.


Would that change dev/graphics/fb0 before the scaling engine ?


please specify atomicity


remove unnecessary file


I get notifications for pretty much everything that happens here.


please specify reference-type


This change makes sense, but if we need to change maxColspan to use `countDays` instead of `getDayOffset`, then we probably need to change this call as well, since the idea of what needs to be counted is the same, right?


wrong order `(copy, nonatomic)`


+1000000000 ;-)

Any chance this moves into "upstream" (wherever that is now)?


I think these files can be removed entirely.


Naprawione w 7fa01fbdc1231a02f4c0e84bc9698e19b368e3a0 ;) Dzięki


Could you use [rex](https://github.com/kevinushey/rex) to strip the whitespace.  The equivalent expression would be

```r
branch <- rex::re_substitutes(
  system("git rev-parse --abbrev-ref HEAD", intern = TRUE),
  rex::rex(list(start,spaces) %or% list(spaces, end)),
  "")
```

It might even be better to just add a trim function to [R/utils.R](https://github.com/stevepeak/covr/blob/805d4955d4f08b0c04da17a2b51de0b5f2bb6e07/R/utils.R)

```r
trim <- function(x) {
  rex::re_substitutes(x, rex::rex(list(start,spaces) %or% list(spaces, end)),  "")
}
```

Then
```r
branch <- trim(system("git rev-parse --abbrev-ref HEAD", intern = TRUE))
```


here also specify atomicity. Suppose should be `nonatomic`


Please, use more descriptive commit messages


OK, got it :)


I'd use DIC and GridFactory, to provide dependency inverdion.


@atironati thanks for spotting that! I'll whittle it down; damned SVG should be wicked small, but I suspect there's a fallback PNG buried in there…


@AngeloLupo forse non volevi committare questo!


Thanks, added it to [BZ-1199256](https://bugzilla.redhat.com/show_bug.cgi?id=1199256)


Cool :)


:heart: :checkered_flag:


Could you publish GridFactory?


This should be `SLIDE_WRAPPER_SEL`. (line 2074)


We don't need either of these submodules anymore.


Which version of nib did we have before? Which do we have now? Why did we upgrade? Would be good to know when looking at the message.


CCVertexIndexData.cpp no longer builds with this merge (Win32 at least.)
#include "base/ccConsole.h"
as the first include fixes the issue. (Not tested on other platforms.)


Very true


@neVERberleRfellerER How are you getting that? From trying to compile that code or importing `traverse`?


Right, I'm just not sure how soon we're moving. I just noticed the recommendation on the JodaTime site to use the `java.time` package for Java 8. So, when are we switching? :)


Quite right


That's a good suggestion. Will do.

I suspect `walCheckpointer` is not needed. Hopefully we'll remember to change `session_` back to a value when we remove that class.


Hey @v3n, I know this was a loooong time ago in a frenzied hackathon sprint, but I wanted to make sure that you were aware that checking things like this in to GitHub is a Bad Idea™. I tend to use something like [figaro](https://github.com/laserlemon/figaro) and keep it in an ENV variable.


J.O. - 10-MAR-2015 - Allow German characters to be typed in email input field


After this commit indentation of block end commands does not drop back a level when nested.


Variables will be changed to the PLL or LBL


Is there a difference by using `getActionIds` or `getActions` when registering the actions to the store?
I guess not but there are some examples using one or the other, so just wondering what would be the appropriate way to use them.

Thx :)


typo: "with *inherited* fds,"


In the function `createSlimScrollingHandler` you can replace the selectros `.fp-section` and `.fp-slide` as well.


And in `addVerticalNavigation` you can replace `.fp-section`.


Shgklkcffnwib


Is this any better @havardh ?

Alternatively:
```
#[cfg(feature = "dk3750")]
use dk3750 as kit;

#[cfg(feature = "dk3750")]
mod dk3750;

#[cfg(feature = "stk3700")]
use stk3700 as kit;

#[cfg(feature = "stk3700")]
mod stk3700;
```
This wouldn't require `allow(dead_code)`.

I don't know how to combine several lines under the same `cfg(feature = "...")`, or if it's even possible.


Ae


Also what happens here if you aren't in a git repository at all?

I would suggest using `system2` rather than `system` then checking the `status` of the command afterwards.

The following function should work, it can probably be put in [R/utils.R](https://github.com/stevepeak/covr/blob/805d4955d4f08b0c04da17a2b51de0b5f2bb6e07/R/utils.R) as well.

```r
local_branch <- function() {
  suppressWarnings(
    branch <- system2("git", "rev-parse --abbrev-ref HEAD", stderr = TRUE, stdout = TRUE)
  )
  if (!is.null(attr(branch, "status"))) {
    stop(branch, call. = FALSE)
  }
  trim(branch)
}
```


Ncthe


The


Wow, I like the result. Nice!


It initially confused me that these fields were simply removed, rather than explicitly being set to `dns.RcodeSuccess`.


Th


for what??


How will you test something if the source is not in the repository friend?


nice


I could do that, or I could keep putting a different version of expect_traces in each test-*.R file.

Either way is fine with me.


I haven't updated it yet. I'll take care of that this morning.


If you are referring to line 87/86... that is the migration code...as annotated ;)


Constants for the variables in the selection lists will begin with a "PLL_" (PickList Labels)


I have added `import traverse ...`, then `make buld-core` and after that, I can not compile anything.
Problem is in circular dependency. This:
https://github.com/babel/babel/blob/master/src/babel/traversal/path.js#L212
is source of problem.
It can be fixed by adding import after "t.TYPES = " in types/index.js


esto es un comment


Du elsker å merge du


Yes @GalaticStryder


You can remove the comma here.


Were you able to browse?, I am getting route error for having multiple home controllers, and you haven't specified the namespaces in the route.
Seems you have mastered MVC using editor templates


Here too.


коммент


"Since an int64 value is expected for seek and set position. When using a regular python integer I noticed set position wouldn't do anything when valid arguments were given, so following the dbus spec file and referring to the dbus example bash script in the omxplayer repo helped fix the issue." -- @jthuraisamy


ok :smiley:


I believe yes. Stock recovery and CWM works fine that way.


Thanks!


It would probably be a good thing to add a

if(!ishex(p)) goto bail;

for both the challenge check, and the binary check.   ishex() is a helper function declared in common.h


Is it worth adding a comment here on what the "3" means? Pretty sure this is the fd for the sync pipe according to the factory code, but it may demystify it for others to know why a raw integer is used here..


```
times = 5
1.4142135623730951 <nil>
0 cannot Sqrt negative number: -2.000000
```


Sure, will try to refactor this whole method, I don't like it either :)


É mesmo! E agora é `/dev/stderr`, talvez aí que possa ter algum problema.

Seria bom se você conseguisse isolar o problema na tua máquina, usando o clitest diretamente em um arquivo com algumas combinações de comandos:

```
$ printf ...; echo $?
$ echo ...; echo $?
$ printf ... >/dev/stderr; echo $?
$ echo ... >/dev/stderr; echo $?
```

E por aí vai, pra ver se encontramos qual a diferença pra minha máquina, onde tudo funciona.


Yeah I took a more radical approach. 98ca541fde0c1ec9f2b5e5222e0adf08740cb540


This breaks setting fonts in ApplySchemeSettings, can we get this reverted? Is there really any noticable performance impact that justifies breaking scripts? Can you explain where performance is lost, i think the fon't shouldn't change too often so the impact would be rather minimal. Would love to get an opinion on this.


Поздравляю!  :smile:
Успехов! :+1:


Ouch, that sux.  Spent a bit of time on this one, getting it right, and nice.  Too bad.  Cut down a LOT of excess crap, and simplified quite a bit.


Google Drive would work as it supports CORS. However, I have yet to find a high quality, Creative Commons licensed video to use.


If you use `Store#register()` or `Store#registerAsync()`, then no, it makes no difference. But internally, action ids are what is passed to the FB dispatcher, so they're provided for your convenience.


> Why set it to 16 max?

The reason is given in the commit message.
Ref: https://github.com/adafruit/adafruit-rpi-fbtft/blob/master/fbtft_device.c#L485


I almost approve. It would be better as:

```html
<div class="social-btn">
	<a href="https://www.facebook.com/selene.hakobyan">Student With The Best Mentor Ever</a>
</div>
```


Just pushed out 4.7.6, thanks @neVERberleRfellerER! Looks like #958 needs to be done ASAP.


TeamCity cat-angular :: build [Build 54](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16026&buildTypeId=CatAngular_Build) is now running


We need to add this to autoconf.


Aki nem jár suliba... ;)


I'm actually contemplating re-adding your patch now that the core stuff (which was trivial) is in there. I just need to get to the bottom of that $2b$ and $2y$ issue first.


Heated bed is working fine for me. I assume you updated your config?
(also it looks like 626Pilot didn't touch any of the heated bed sections)


Did you really mean Rsolr here?


Style bonus points:

```scala
(suspicious zip normal) map sigDif.tupled
```


esto es un comment2


TeamCity cat-angular :: build [Build 54](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16026&buildTypeId=CatAngular_Build) outcome was **FAILURE**
Summary: Gradle exception (new); exit code 1 (new) Build time: 00:03:02


Have a look at efarefactoring3 if you want. It's rebased against master now. I'll do some testing first and create a pull request when everything looks good.


Good idea.
Glad to help and thank you for fixing this!


Is this necessary? We know `defaults.options` exists and is an object no?


That is a clever way of testing it, but it's a good practice for unit testing.
Notice that this test also passes when the shifted problem does no transformation at all (i.e. applies a shift by a zero vector), so at the end ``shifted_prob3.get_best_x() == prob.get_best_x()`` anyways.

In principle, you want the unit tests to be simple and transparent in what they do, i.e, instantiate a problem, define two vectors: one with shift_vectors the other with expected_shifted_best_xs, and then compare those in a loop.


Hey @devcritter ive added teh auditSourceID to the dropdown options. I have added the UI for this as well. Just doing final tweaks before send PR


yes same controller giving errors for front end. Will change that later. However admin area is working fine now. Check  \Admin\Home\Index :-)


it was not the case when the configuration was not copied


Why are we loading all the products for the user?


@Amy-Xu should this be 0.1?


Oops... I messed up on the message for this commit. This commit was supposed to be an amendment to the previous commit, but I accidentally left a - and inserted a m on --amend, so it came out -ammend. Resulting in a normal commit with the message "mend"...


Nice fix, but to be completed, it has to be in the 2.0 file, too.
Greetings Jan


Regarding your question, you could create a div especially for the movie's detail, a unique one that you would use for all movies. For example ```#movie```, and just set its html with ```$('#movie').html(view1.render().el)```. But your implementation is very satisfying anyway.


Excellent work!


I have fixed the route issue with areas in the master.
Good you have done something.


Yes eventually. I had a quick look. First, here's unexpected lack of functionality:
```
$ icc -mmic -dM -E -x c /dev/null | grep -E "(MMX|SSE|AVX|XOP|_MIC)"
#define __MIC__ 1
#define __MIC 1
```
No MMX, no SSE, no AVX flags. Only the `__MIC__` tells us we have AVX-512.

Also, we can't just build jumbo without some prior efforts:
```
$ ./configure --disable-native-tests CC="icc -mmic" -host=x86_64
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-pc-none
checking whether to compile using MPI... no
checking for x86_64-gcc... icc -mmic
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... yes
(...)
checking for byte ordering according to target triple... little
checking for sqrt in -lm... yes
checking for deflate in -lz... no
configure: error: in `/home/magnum/src/john/src':
configure: error: JtR requires libz being installed
See `config.log' for more details
```


See: https://github.com/WebDevStudios/CMB2/pull/198#issuecomment-74569536

This needs to be reverted unless you have a good argument beyond what's already been hashed out there.


Whoops, it seems the `@user` who created the `@product` was able to add a review.


Looks like this hasn't yet been synced to the plugin on the WordPress.org repo.


Leave this blank for now as this might not be a valid user_id `54f81d51736f771880000000`.


Fixed :)


I think this might have broken automatically expanding the menu to the current page, for all pages except Phobos.


The log level doesn't currently get passed to the log or error functions. Did you have a change to station.js that added this?


@keradus that's what you wanted, right? looks funky, but I figured it shouldn't trim those trailing commas, there are other fixers for that.


Will that be easier if the common part gets named BF_std.* instead of BF_common_plug.*?


i have noticed it. i just didnt fix it properly. fuck if i knew that it was width.


@thabach The `CLOSE` will be invoked only once when the connection is getting closed, at least for the 'STREAMING' case? I think the `WEBSOCKET` works the same way, no?


Not testing the right scenario.


Can you break those tests into separate functions, i.e. ``test_shifted``, ``test_rotated``, and then in main use
```c++
return test_shifted() | test_rotated() | ...;
```
It's easier to keep track of what is going on (although you comment those very well).


Is it likely that the requestAuthenticator ever changes, once set? Could it be a parameter of Heimdalls `init` or passed as an additional argument to `authenticateRequest ` (using `HeimdallResourceRequestAuthenticatorHTTPAuthorizationHeader` as the default)?


Good bug catch!


Why is this required?


CSS+DIV练习


Ok thanks :+1:


I told them to put it in as a safety measure.  If the pot fails, there will be nothing to stop it other than hard mechanical stops.  Limit switches provide a fail back that might be added.


cool. thks


A list doesn't have a method `discard`.


What is this?


I eagerly await the pull request good sir! Keep up the good work! :smile:


Agreed, this is just trash left in the file, we "fixed" setPosition as the main method.


A bunch of indentation issues like this in the file.


why do you expose the ports here? that makes changing them in the backend much harder.


This is a cool idea. Please thoroughly test this and make sure everything works as expected, and I'll merge it. Thanks!


Is this too "minor"? hehe


You are missing a '/' here


give it back


With some luck, some GSoC intern will do most of the job for us :-)


hm. why not number >= table.Count?


![diagrama de contexto grupo 1](https://cloud.githubusercontent.com/assets/8311103/6575765/801c5532-c6f7-11e4-8e1f-f9b9417184fa.jpg)


0xA7 - DODF PATH REFERENCE
0x30 - DATA OBJECT (generic, might be used multiply in different locations)
0x04 - PATH


Definitely cleaner :-)


Change this to manywho.utils.IsEqual


Yea.....my bad. just fixed it


the variable {{cert}} contains the "port" already


For the Elevator Up and Down Buttons, and even maybe for the arms open/close operations, lets look at command based programming:

http://wpilib.screenstepslive.com/s/4485/m/13809/l/241892-what-is-command-based-programming

This would not be difficult to implement and get working in short order.  Let me know what you think.


This is only an icc thing?


Well done!


i rather have back the {{blocks}} here so this can be set from the index.html.


$('.datepicker') would be better as this.refs.whatever. If only so that the jquery plugin is initialized on this instance of the component only


for which reason do you include the JS of the not enabled login plugin?


yep. I know.I did update the config file.All the calibrate runs, hotend runs, all seem to be sane.
I'm adding debug now to try to find where its having fun.
Im on azteeg x5.

      From: Malcolm Williams <notifications@github.com>
 To: 626Pilot/Smoothieware <Smoothieware@noreply.github.com>
Cc: Glenn S West <glennswest@yahoo.com.sg>
 Sent: Tuesday, 10 March 2015, 21:14
 Subject: Re: [Smoothieware] Warn about memory and LCD controllers (23e6c2e)

Heated bed is working fine for me. I assume you updated your config?
(also it looks like 626Pilot didn't touch any of the heated bed sections)—
Reply to this email directly or view it on GitHub.


did you look at the rendered result? Never wrote markdown before?

Might want to look at https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet


oops!


You should call this.render from within the component (I think). this.forceUpdate() is the way to go


I'd use h3 (so \#\#\#) for the steps.


I thought about limit switches. The conclusion was that if there's a limit
switch, it won't stop anything, and it will probably just be thrown when we
don't want it to and keep the thing from moving when we want it to
On Mar 10, 2015 9:28 AM, "Michael J Coss" <notifications@github.com> wrote:

> I told them to put it in as a safety measure. If the pot fails, there will
> be nothing to stop it other than hard mechanical stops. Limit switches
> provide a fail back that might be added.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRC-3637-Daleks/Turing/commit/4048369b1fc7ea125421a55dae46efb63bb1f242#commitcomment-10120058>
> .
>


What broke it was moving large portions of code from a core file to a Jumbo file. Maybe the existing BF_std.* files can be used for OpenCL too, that would probably be fine. We can add tons of code like that (using #if..#endif as needed) without upsetting Git. Just avoid moving or dropping code.


saw approval from andrewjstone
at https://github.com/darach/riak_core/commit/3666df2b416db009e7b12686ff5a1acd51f2f7ae


merging darach/riak_core/de_add_is_stable_predicate_to_ring_manager = 3666df2b into borshop-integration-716-de_add_is_stable_predicate_to_ring_manager


darach/riak_core/de_add_is_stable_predicate_to_ring_manager = 3666df2b merged ok, testing candidate = bbd7758d


@malept If this works on real data, it seems like a better option.


Using glm::rotate will ensure use of a standard and optimized functionality.


Hello,

I try to have a dot based on 2 menus in Safari - Development (menu between Bookmarks and Window) but my applescript is nowhere and I don't know how to track/trigger the changes.
J = Disable Javascript
S = Disable Style
ON = with a checkmark.

green if J = OFF AND S = OFF
orange if J <> S
red if J = ON and S = ON

If you could give me an example or few tips.

Thanks a lot,

BTW, is it possible to change a dot from the Bash?
I am thinking to use it with https://github.com/felixhageloh/uebersicht

Thanks.


is it possible to use the reset button


Malcolm, can you provide your config.Maybe there a line missing somewhere in the Azteeg examples.(Assuming your using Azteeg)
      From: Malcolm Williams <notifications@github.com>
 To: 626Pilot/Smoothieware <Smoothieware@noreply.github.com>
Cc: Glenn S West <glennswest@yahoo.com.sg>
 Sent: Tuesday, 10 March 2015, 21:14
 Subject: Re: [Smoothieware] Warn about memory and LCD controllers (23e6c2e)

Heated bed is working fine for me. I assume you updated your config?
(also it looks like 626Pilot didn't touch any of the heated bed sections)—
Reply to this email directly or view it on GitHub.


Does this patch effectively stop advertizing any feature at all, or does passing a NULL there revert it to the default behaviour in Sofia SIP? At the time there was a pull request that explicitly disabled 100rel because it could lead to segfaults, so I want to be 100% sure that we're not bringing this back in.


There's still trailing whitespace in this line


Il y a du tabbing dans le fichier


This should be `SECTION_SEL`  (LINE 1142)
And `SECTION_SEL` (lINE 1145)


mal indenté


...and for code that is in BF_fmt.c, a change like this
```patch
diff --git a/src/BF_fmt.c b/src/BF_fmt.c
index bfa7235..903ef3c 100644
--- a/src/BF_fmt.c
+++ b/src/BF_fmt.c
@@ -116,7 +116,7 @@ static void init(struct fmt_main *self)
        sign_extension_bug = 0;
 }

-static int valid(char *ciphertext, struct fmt_main *self)
+int BF_valid(char *ciphertext, struct fmt_main *self)
 {
        int rounds;
        char *pos;
@@ -279,7 +279,7 @@ struct fmt_main fmt_BF = {
                fmt_default_done,
                fmt_default_reset,
                fmt_default_prepare,
-               valid,
+               BF_valid,
                fmt_default_split,
                BF_std_get_binary,
                BF_std_get_salt,
```
...would not cause much trouble either - and it would allow the OpenCL format to reuse that function.


Persisting a holiday request should not be an employee's responsibility.


No, by passing NULL all those features are disabled, so there is no Supported header sent out.


You're right - included to try to fix an error in tests but it was unrelated. I've removed.


Looks good


1.) BOM is fine with UTF, but not with JSON. For JSON it must be UTF-32BE, UTF-16BE, UTF-32LE, UTF-16LE or UTF-8. PHP is only working with UTF-8, so yes, it would be good to make PHP RFC conform. This is so far for PHP.
Another thing is that many ppl have trouble with different applications parsing JSON when broken editors added a BOM. Maybe I wrote it a bit unclear. Yes, BOM is wrong in a JSON file. And yes, an application checking for BOM in JSON and removing it could ease some trouble.
Editors and browsers which don't complain about a BOM are not agnostic, but they are RFC compliant. See rfc3629, sec. 6.
2.) I am using a text editor on a text file, and I expect it to insert text where I type it and remove text where I delete it. Not more and not less. And I expect it to not add anything I didn't type and I didn't want. That's what I was complaining about. Did you read my text?
3.a) OK, vi is avail to Windows, I didn't know that. And I never checked, because I never had the need for another editor. Notepad was sufficient for me.
3.b) I like to use Linux. Who told you I don't? Again, did you read what I wrote? Using GIT 2 times a year is easier with a GUI, at least for me. I find out the command line parameters I need, and 6 months later I have forgotten them and have to find them again. With the web GUI this is way easier. And yes again, the main problem is using the wrong editor. Sadly this is the editor Microsoft delivers with Windows. If I only had knew this before I used it, this would not have happened, at all.
4.) Again, pls read what I write. I asked you, who ran into trouble, and you didn't answer. Do you know anyone using this JSON from PHP? None of the problems I found regarding JSON & BOM were related to this specific JSON file.


Thanks for the clarification!


No I assumed it worked this way and did not test that.

So according the signatures of the two functions actually are `config.log(string msg)` and `config.err(string msg)`. Then, I don't understand how the `config.logLevel` influence the log level of the two functions: obviously if `config.logLevel === 'INFO'` the log level of `config.log()` passes from `WARNING` to `INFO`; but what of `config.err()`?

Also maybe it would be useful to order the log level by severity or specifying in which case which log level should be used, or linking the doc if it's already specified somewhere else.


The normal login plugin should _not_ be enabled? For me it is (additionally to TYPO3Login) and as far as I can tell it has always been...
Without this change the Piwik login page threw a JS error.

Should the normal Login plugin in fact be disabled?


No need, will come in with next merge.


@brendanashworth a better commit message would have been something like "fix indentation" :)


I have found a mistake in this commit.  gmake-qt4 is searched for instead of qmake-qt4.  I submitted a patch in the bug to be put in a future pull request or commit.

Patch is as follows:

--- configure.old	2015-03-09 21:25:16.565788946 -0400
+++ configure.old	2015-03-09 21:26:18.240586161 -0400
@@ -363,7 +363,7 @@
     echo "  QTDIR environment variable not set!"
     printf "  Checking for Qt..."
     for d in /usr/{lib,share,qt}/{qt-4,qt4,qt,qt*,4} /usr; do
-      for e in qmake gmake-qt4; do
+      for e in qmake qmake-qt4; do
         if test -x "$d/bin/$e"; then
           QTDIR=$d
           QMAKEEXE=$e
@@ -372,7 +372,7 @@
       done
     done
   else
-    for e in qmake gmake-qt4; do
+    for e in qmake qmake-qt4; do
       if test -x "$QTDIR/bin/$e"; then
         if test -e "$QTDIR/bin/$e"; then
           printf "  Detected Qt via the QTDIR environment variable..."


You don't need the braces here


Maybe I should build BGFX with my own custom bgfx.lua script and configure stuff from there?


figures--> figure


Zoom for clarity.

[image: Inline image 1]


Directly manipulating the state of the holiday request is fishy. Now the manager is strongly linked to the implementation details of the holiday request.
Consider HolidayRequest.Approve(), Reject(), Submit() and hide the implementation details.


More trash left from migrating from Lifter.  Note though that one of the nice things about a pot is that it doesn't require calibration.  It is always providing output as to the current position of the pot.  As it rotates it provides a voltage reading 0 (at one end) - MaxV (most likely 5V at the other end) depending on the amount of rotation.  So it get's placed in it's upright position with the pot reading 0V, and at the end of travel needs to be no more that the number of degrees of rotation the pot is capable of handling.  Normally you put a little slop on so 0 isn't 0 but more like 0.3.


Persistence of holiday request changes are not the manager's responsibility.
And the fact that this problem appears in the Employee and the Manager classes also tries to tell us something.


This should be `SLIDE_PREV ` instead of `class_prev` (Line 1352)


float m_roll = 0.0f.


`SearchController` 를 따로 두는 것에 대해서는 어떻게 생각해?


Looks good


Naming: EmailNotification sounds like a kind of Notification. It should be named EmailNotificationService probably.


Agreed I mentioned that this probably a bad way to do this


:shipit: Even mucher amaze


@jfarcand If you are going to write migration guide from 2.2 to 2.3, it would be nice to note that `framework` field is deleted but still accessible through `framework` method.

https://github.com/vibe-project/vibe-java-platform/commit/ce1e72c58ca8acb5b0a4c65e429e8d08265ef3b7


Naming: Maybe the NotificationMessage should be renamed to Notification?


@onli Ideally we should use a SQL command like "SHOW TABLES FROM DATABASE" to create an array with all tables, then remove all tables that do not match our dbPrefix, then remove all tables that belong to the core, then all those that belong to currently active plugins - and the remaining tables could then be listed. I think such a "blacklist" is better because it would also reveal tables that were initialized by plugins that got deleted from the presence already. Using a whitelist for unused tables could show too few...?!

(Thanks for working on this so quickly!)


You should define these as something like:
``#define THUNDER_PEMn_REG_BASE(n) (0x87e0c0000000UL | ((n) << 24))``


tested and working


I think you could simplify this and just do:
```
var zeroWidthSpace = previousNoteSegment.right();
var marker = zeroWidthSpace.right();
```


Another way to do same as above, with an even less intrusive patch
```diff
$ git diff -w
diff --git a/src/BF_fmt.c b/src/BF_fmt.c
index bfa7235..bb2cedf 100644
--- a/src/BF_fmt.c
+++ b/src/BF_fmt.c
@@ -116,7 +116,8 @@ static void init(struct fmt_main *self)
        sign_extension_bug = 0;
 }

-static int valid(char *ciphertext, struct fmt_main *self)
+#define valid BF_valid
+/* static */ int valid(char *ciphertext, struct fmt_main *self)
 {
        int rounds;
        char *pos;
```
This has pros and cons though. Not quite as easy to follow. But very easy to merge in the future.


Numeric levels are lame, why not `*` for public, and anything else describes an implementation-specific requirement for visibility.


ack


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


:fearful:


What is the purpose of this class? It is implemented only in the EF repository class.


you can use label: Ember.computed.alias('value') if you just need to have the same prop under another name


r+


main-header__main-menu - \это у вас независимый блок main-menu


Why not use File.join ?


Maybe yes; @wbsoft?
I also noticed that I didn't take care of the Windows problems in file_import/*.py like in convert_ly.py.
Will do later.


Hoje a noite eu faço o teste, e retorno.
Onde estou agora não tenho como testar, todavia em casa faço o teste em 3 máquinas diferentes.


здесь у каждого элемента списка идет main-menu__item а для активного еще добавляется main-menu__item main-menu__item--active


TeamCity cat-angular :: build [Build 55](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16027&buildTypeId=CatAngular_Build) is now running


saw approval from jdm
at https://github.com/zslayton/servo/commit/08ac0766eda2340008642e86799ea2cb1ef6e59f


merging zslayton/servo/master = 08ac0766 into auto


Кнопки независимые блоки btn btn--menu-close и btn btn--menu-toggle


zslayton/servo/master = 08ac0766 merged ok, testing candidate = d31e80f8


> Windows problems

I mean `env = util.bytes_environ()` vs `env = dict(os.environ)`.


I firmly believe 'get_if_exists' or 'getIfExists' is a much better name than 'maybe_get'. I personally refrain from using 'maybe' in a method name.


только не header а title


Why are you casting a ``void *`` to a ``uint64_t`` to a ``uint64_t *`` then accessing it directly? We have functions to access the data memory you should be using.


TeamCity cat-angular :: build [Build 53](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16025&buildTypeId=CatAngular_Build) outcome was **UNKNOWN**
Summary: Canceled (Gradle exception (new); exit code 1 (new); :npmInstall) Build time: 01:25:45


TeamCity cat-angular :: build [Build 55](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16027&buildTypeId=CatAngular_Build) outcome was **UNKNOWN**
Summary: Canceled (Gradle exception; exit code 1; :npmInstall) Build time: 00:01:19


вот этот блок у вас независимый, он повторяется много где с разными модификаторами


@n-west I have changed the glyphs to look uniformly across the site, but the one's associated with Doxygen Manual and Downloads doesn't seem to compliment well with the other icons. Our other option can be to use custom SVGs to make all icons look uniform.


@borshop merge


Изменил вложенность блоков. Заменил некоторые теги.


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


Does this need to be passed as a jvm-opt? Can it be an env var?


@LukeStonehm Do you want to keep the branch like this or a rebase is better? :)


и внтури после image можно col уже не приписывать


Cryptography can be potentially used also by other parts of UV, not just by DPUs, that is our business requirement. I need current develop branch as basis. Is it acceptable to create feature/cryptography from the current develop branch without reverting?


Good data makes for good code, and you have control over the data here.


тот же блок, то же класс col только к нему еще col--gray который меняет ему фон


The problem are the non-generic interfaces that are created. For example, the non-generic `Array` that is used so we can type `Array<Dynamic>` correctly. We could maybe solve this however using an attribute


Why you return Bad Request? This http status (400) manages erroneous request parameters, missing request headers, etc.


h2 внутри кнопки?


Same comment as above.


Why are these mapping the memory directly and not using bus_space? And why are they not getting the addresses from fdt/acpi?


I thought about that as well but decided to
- not add even more parameters to the initializer
- give the user the chance to change the `requestAuthenticator` at any time
- give unit tests the chance to change the `requestAuthenticator` at any time


просто social-links


social-links__item social-links__item--twitter


Abc


Instead of explicity return a new ReponseEntity object, you can directly add it on the annotation section, as for example:
@ResponseStatus(HttpStatus.OK)


Ah right, okay. Let's leave it as is for now.


Just created the repo "datasciencecoursera"


Then you can just name this var `config`. It's not a kidney stone.


Your commit has additional things added. I didn't even know about it, frankly. There is nothing in it, I feel. If you go back and check each commit, I have kept authorship wherever required. There are 1000s of commits that can be similar that doesn't mean each of them is kanged :)


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


Could you please comply with the max line width.


Milly Merged <3


D'oh. Of course!


Should `commands` be plural here? Seems like you're really running one command.


Think you don't need to check for edit rights here, becasue this tag is only for search engines.


How cute :+1:


Convenção do UNIX.

http://stackoverflow.com/questions/729692/why-should-files-end-with-a-newline

http://stackoverflow.com/questions/16222530/why-do-i-need-vim-in-binary-mode-for-noeol-to-work


@atironati fixed up in e17638d, thanks!


Fixed as per de0d35d. Sorry for being so sloppy!


@mmhelloworld Not sure why I got all these changes when I was trying to get the latest version.  Can you check that why these occurred and if they are appropriate please?


oh and in this commit you also want to put a link into the README.md in the getting_involved directory so people can browse this content easily


and don't forget to rebase before :)


Need to add the below validation in requirements
if len(c.Args()) != 0 {
		cmd.ui.FailWithUsage(c)
	}


I guess it was initially. I saw this in the gcc-5 release notes:

"
Specific for upcoming Intel Xeon Phi products:
* Run-time library.
* Card emulator.

"


What if a manager wants to reject a holiday request even if it is valid? For example an important release will be done on that date?
I think the manager (the real person using this code) should be able to decide if the request will be rejected or approved.
Or am I missing something?


Just to understand the issue:

1. One would expect to use `csvw:notes` as a predicate and not simply a copy of the predicate (as opposed to the usual common properties): I think you are right, the text should be amended accordingly
2. The fact of using a type `oa:Annotation` should probably be left to the author of the metadata. Indeed, I am not sure we want to restrict the CSVW notes to be exclusively an oa annotation. Or do we?


Missing produce section. If you pass a request body in a json format, it's needed to be specified. Please add it.


pls do change the Describe it should be user-info


You shouldn't need the udev.settle() call, it's part of StorageDevice._postSetup().

Is this going to cause a problem for devices that are are set up when this call occurs?


Juan Emilio Zurita Macías


You should not commit working/temp files. obj/bin/debug/release folders should be ignored


Er, sorry I didn't expect you to accept that pull request straight away as it wasn't building on appveyor. This bit snuck in as well from trying to get cpack+nsis to work together. I'd suggest either a very quick reset to HEAD^ and a force push or I'll fix it up in a new commit.


in case I merge wip/sils/links before which is fiddling also with the README


This results in an exception.  I think you want:

tmpDict["WMAgentID"] = int(results[i].get("WMAgent_JobID", 0))

?  Same problem in all above lines.


Yeah I urgently need to reformat IprotectionPlugin. I am aware that my plugin needs a cleanup.


use more conventional naming. If you have messageData array - use message_data_id key instead


That makes sense I'll have to think of a better alternative.


Should we ask for a CVE number?


Why not do this check at the top of the loop?  This way you don't process the whole job before deciding to ignore it.


Is it practical to keep `:strict-host-key-checking` enabled? If not, why not?


@supritashankar Hey! I am done with the code. Please take a look at it when you find time. Your assistance is greatly appreciated. Thanks!


@zdne

Leftover? Could you please remove it?


.. I'm not quite sure where this leads me but ....


nice, more flexible!


Yes, I will as all the `getDefault` method of factories has been removed. Note that `2.2.x` doesn't have such changes.


@skalk: Most of the ``type != NORMAL`` case seems to be the same as in ``base/thread_start.cc`` (all except getting the parent cap). What do you think about moving it to a generic back end function in ``base/bootstrap.cc``?


tested and working


This is where the input box is cleared out


You should not extract each property for user object. Just memoize user object and call it's name directly in the string. It's very important. If you have some calculations on it - define it this as class method on object.


As far as I can see, you can Submit a Rejected/Approved request.


Was this commit really what broke it?


A1 have a look


@tfarago: I didn't want to put this into the constructor to not cause any side effects. However, I strongly believe we should set all possible parameters to their default values.


search가 status만의 전유물이 아니라면 일리는 있음 ㅎㅎ

그런데 전 코멘트에서도 말했지만, 검색은 일종의 필터링인데 여태까지의 리소스 중심(모델 중심)으로 한 컨트롤러 구성과 아다리가 잘 맞는지 어떤지 모르겠삼 어떻게 생각해? 이를테면 유저 검색과 팀 검색 상태 검색이 모두 searchcontroller에서 이루어지는 거잖아


get_num_enrolled maybe? when I first saw this in the template I thought it was getting the enrolled list


how to use the inspector?


Does this really need to be a fixture? Seems like it might be better to bake it in to the `my-server` def, perhaps with a `delay`.


WTF?
You should loop through array of dialog ids. Use map() on Dialogs collection.


What a RequestConversation represents?


It seems the RequestConversation just adds the Status and some operations to the request.
Shouldn't they be part of the HollidayRequest class?


course_id=0 is totally temporary, right @aem1269 ?


I think it's more idiomatic to just use `(Integer. "123)`


Please send a new commit on top to fix... Appveyor sometimes fails things due to build timeout, it's Windows so it's opaque to me.


Will it still need to be `file-formula` on Tigerbrew, with its old Ruby?  Or was the change that allows it to be called `file` a change in Homebrew itself?


nit: let's remove these extra spaces.


And perhaps a timeout.


Content for non-telepathic humans ..... ;-)


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.2220](http://teamcity.monogame.net/viewLog.html?buildId=5646&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 444, ignored: 6 Build time: 00:14:05


Committing binaries, temp files, packages etc is bad.


Les lignes ajoutées dans MovingPlatform.py servent simplement à conserver la vitesse du personnage à 6 de plus que la plateforme. Car dans le cas d'une plateforme qui se déplace à une vitesse de 5, la différence de vitesse étant de 1, c'est comme si on ne bougeait pas lorsqu'on se déplace dans le sens de la plateforme, et inversément, c'est comme si on était "propulsé" de la plateforme lorsqu'on bouge dans le sens inverse. (nous = -6, et plateforme = 5, différence de 11, par conséquent difficile de se déplacer de manière précise)

L'ajout des lignes dans Character sert une fois de plus à "contrer" l'inertie de la plateforme.
I.e. dès le moment où on saute, que ce soit vers la gauche ou la droite, on aura une vitesse de 6. De cette manière on n'est pas propulsé à une vitesse de 12 si on saute d'une plateforme allant à vitesse 6 dans le même sens que nous, et inversément on ne sera pas collé sur place si on saute à contre-courant.


It would be great if we could take this opportunity, given we're refactoring quite a bit of this function, to split this code to an auxiliary function.


Replaced backslash \ to forward slash / in file name, otherwise this will show error.


osd_pool_get_choices_t instead?  as it is, sounds quite a bit like a method name.


This is just an observation - there's a little bit of implementation leakage here in the sense that we're exposing the fact that `runs` is a map of string to string.  I think in this case that leakage is pretty insignficant (e.g. we could change the impl to read/write an array of something in the future if we needed to without necessarily changing the `runs[]` API in this SDK. But just wanted to point it out as we work down the feature list and get to more complicated objects, the SDK will need to abstract out more of those details.


Yes, those dummy functions are all needed when register dma_buf_ops.  See the API `dma_buf_export_named`.


Can't we restore the ability to install mockup without the plone.mockup/mockup split? I do the the benefits of being able to install mockup as a bower package.


Is there another way to make this work?  I'm concerned about the general scalability. Imagine we support most if not all of the properties on a Service.  Extrapolating this pattern would yield 2 giant methods, __init__ and __toObject, instead of a bunch of getters and setters. In both cases, this file is going to get bigger, but I think a collection of getters/setters will be more managable from a maintenance point of view than 2 big methods.


I'm searching too but I find nothing yet:
https://www.youtube.com/results?sp=EgQgATAB&search_query=equirectangular&filters=hd%2C+creativecommons&page=2

What about this?
https://vimeo.com/64093648

Cheers! :)


Readability:ListOfRolesOfAnOrg  not  ListOrgsByRole, similalry for space related.


Why do need this object at all? Only Dialog and Messages. Move all properties starting "time" to message.
You should have Message like this:
{id, sender_id, dialog_id, created_at, text, read_status, delete_status}


totally realized its a default thanks to nick... but should we have a default?


어라 나도 "컨트롤러는 모델과 연관이 있어야 한다"고 생각해왔었는데 그게 단순히 편견이었던건가?


When I ran this example on my local, the output contained:
```
   "Runs": null,
```
But I expected something like:
```
  "Runs": { "foo": "bar"},
```

See note below about getters/setters


Add all the validations like no all org roles, no all space roles &  should contain []string{"User", "Org", "Space", "Role"},


true enough


이부분은 _search_form이라든가 하는 파셜 파일을 따로 만들어서 빼내는 쪽이 더 낫지 않을까?


Ah!! Really sorry about the indentation issues! I wrote it up with Vim on my server, moved to windows and fixed a line and moved it back to make sure it would still run! Sorry! And as for the discard, thanks, I actually didn't know! I'm just starting into python and had to so some research as I was going in! But will do for fixing the indentation and fix the removal in the list upon deletion of friend!


adam likes tables.


newline이 추가 안된걸 보니 웅철이 예~~~전에 말한 서브라임 설정이 적용 안된거라든가? ㅋㅋ


Perhaps the previous version was not the best at 5mb


Sounds interesting.  When will we see more?


This is how you create keyword arguments in python.

Unless you want to overcomplicate the method by putting a **kwargs in the
method declaration (which I will do if you really want) then course_id=0 is
the easiest way to define the keyword argument.

And yes, it has to be a kwarg, if you just put "course_id" the URL doesn't
work

On Tue, Mar 10, 2015 at 10:15 AM, Michael Washburn Jr <
notifications@github.com> wrote:

> totally realized its a default thanks to nick... but should we have a
> default?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aem1269/gr8/commit/ad1e76366c694c80a2ca255d3149565b21f36a93#commitcomment-10121083>
> .
>


I agree with Mike, but I like get_enrollment_count better


Why do you limit basic auth only to GET requests? This looks like a copy paste mistake. Other actions like DELTING an image should also be supported.


the super here sets the verb automatically (Command), so this whole initialize is unnec. the only real thing you might need at this point is the execute, but in theory, only if it does something special. Mokujin needs responses to attacks. That is probably how TrainingDummy's initialize should be expanded, with default responses. Also - a Being has lots that an Item does not. Being may be overkill unless you are planning something for Mokujin.


this fix only works for cases where the consolidate is triggered by ConsolidateDialogActivity. for example, it doesn't work when I delete a secret key where the consolidate is triggered as part of the deletion. :(


These `if` statements aren't needed at all anymore, I don't think. As we're using the translations or the main object, the built-in error highlighting works.


 #OSS
 PRODUCT_PACKAGES += \
+    timekeep \
+    TimeKeep \
     thermanager \
     addrsetup

in case you miss the blobs


This else could be removed


Ok, this will maintain consistency, because a user will be able to include both products and services in a PO. Thanks.


Same comment as https://github.com/mysociety/alaveteli/commit/9e1bd38867bafd1d1186ecc7b8a9c5238c8855a6#commitcomment-10121182


yeah perhaps having 2 copies of the source photos + the avatar all thin the image... then encoded into SVGs was the source of that : D


Ahhhhh ok. Python, man.

> On Mar 10, 2015, at 10:18 AM, Adam McCarthy <notifications@github.com> wrote:
>
> This is how you create keyword arguments in python.
>
> Unless you want to overcomplicate the method by putting a **kwargs in the
> method declaration (which I will do if you really want) then course_id=0 is
> the easiest way to define the keyword argument.
>
> And yes, it has to be a kwarg, if you just put "course_id" the URL doesn't
> work
>
> On Tue, Mar 10, 2015 at 10:15 AM, Michael Washburn Jr <
> notifications@github.com> wrote:
>
> > totally realized its a default thanks to nick... but should we have a
> > default?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/aem1269/gr8/commit/ad1e76366c694c80a2ca255d3149565b21f36a93#commitcomment-10121083>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


This if could be removed


How about get_number_of_students_that_are_currently_enrolled_in_this_course_section()


有道理 （其实我是有再看coffee最近。。 不过没push上来


Please, don't use `-D` but instead append your option to /etc/apache2/envvars as all the other options do above.


In all seriousness, I think it should be something like enrollment_count().


Why remove this line?


See https://github.com/mysociety/alaveteli/commit/9e1bd38867bafd1d1186ecc7b8a9c5238c8855a6#commitcomment-10121182


É uma boa! :+1: Esse stage.stage é dureza...


as a side note, it's incredibly annoying a duplicate switch-case for both the formatter and the plain-text output.  It was tolerable when we were relying on if-clauses for the output, but I think this feels a bit too intrusive.  I will not block the pull request on the grounds of this being annoying to me, of course, but if you can find a way to simplify the whole thing it would be greatly appreciated.


@bryanmtl Thanks for looking in to it. I'm not surprised that RABL has performance issues amongst its many flaws. Let me know what you find.


If you see my comment on line 39, you no longer need this switch here.


I think we are overthinking this now, as long as it shows its a count rather than a set of things it'll be good. Anything along those lines will do.

> On Mar 10, 2015, at 10:21 AM, Adam McCarthy <notifications@github.com> wrote:
>
> In all seriousness, I think it should be something like enrollment_count().
>
> —
> Reply to this email directly or view it on GitHub.
>


Thank you for bringing basic auth to the docker-registry-frontend. Please, see my comments on your PR.


if it is still relevant, I've created a docker image to easily create packages that can run on debian stable:
https://registry.hub.docker.com/u/bbinet/build-grafana/
https://github.com/bbinet/docker-build-grafana


it's a bad idea for such naming. Time in name langs is reserved word, e.g. Rails. You can use "created_at" for example. http://guides.rubyonrails.org/active_record_basics.html#schema-conventions


Why do we have the png images in the repository? Does not make sense to me


:+1: Much better than the current solution


строки 28, 29 я бы вынес в отдельный класс и регистрировал бы связки при старте application. Я переделаю.


Any chance the sysctl command ever comes back as a non-integer?


Very good idea ;)


@bbinet yea, me too :)


all tests pass:
success: http://build.servo.org/builders/linux1/builds/983
success: http://build.servo.org/builders/linux2/builds/982
success: http://build.servo.org/builders/android/builds/1017
success: http://build.servo.org/builders/mac1/builds/1198
success: http://build.servo.org/builders/mac2/builds/1199
success: http://build.servo.org/builders/gonk/builds/169


1: M111 unrecognized component name: base


The bug I found using example.py highlights how we need a little different approach to testing the SDK.
For Description, the test cases are:
- add a description to a service that had none
- remove a description from a service which already has one
- change a description for a service which already has one

In my testing of example, my input service had an empty `Runs` map. For `Runs`, there are test cases like
- add some Runs commands to a service that had none
- remove one or more commands from Runs
- add one or more commands to Runs
- replace all of the Runs with a different set (should work fine if the add and remove test cases work)


This comment could be removed


It is only going to work on iOS Cordova, so swipeBackEnabled is only effective on iOS cordova.


it's same asthatfilein folder


We should probably just move to using Mopidy from APT, along with it's service files and and it's code to setup all this stuff. I don't believe there is any good reason to still be using the package from PyPI.


 	"json" : function(cookies, url) {


We'd like some input from @hs0ucy on that one.


:-)
debian packages created from grafana-packer can run on debian wheezy? you don't have issue with the libc version anymore?


ОК


Sure, it's good to shave testing time whenever possible. Looks good.


HI Brady,
It is going to take some time to resolve these issues.  Not using extract is understandable but requires a significant overhaul as we are importing hundreds of variables each call.  Fundamentally I see exactly how it can be an issue, especially for POST and GET data.  But if all the doors are closed to sql injection on the front end, all the results coming out of the database fields should be clean, so theoretically extract should be safe for these.  It starts to remind me of C where you have to remember the space where variables exist in the stack, a concept I never can grasp, when you can just stack all the variables up willy-nilly in any old corner and use them when needed by calling their name, rather than their name within an associative array.  I will have to become more disciplined.
Ray


Right, this can be merged in the meanwhile, no?


fast-forwarding master to auto = d31e80f8


You have one line this `redirect_to home_url if logged_in?`.


This else could be removed


What if instead of checking for ABSPATH we did `function_exists( 'add_action' );` this would ensure that no fatal errors would occur if called too early as in the situation where it's auto-loaded at the top wp-config.php.

CMB already has robust checking to prevent multiple copies running simultaneously so if it's installed as a plugin and autoloaded after `add_action` is defined it will work just fine.

Does that make sense or am I missing something that could cause issues?


Rabl is taking, on average, 1200ms to generate the JSON. JBuilder does it in just over 400ms. Benchmarking AMS right now...


* The intention is not clear here to me. Should the value be `true` if cat fails?
* Recommending to add quotes.

~~~bash
TOKEN=""
if [ -r /tmp/passwdsrvrtoken ]; then
  TOKEN="$(cat /tmp/passwdsrvrtoken)"
fi
~~~

or

~~~bash
TOKEN=true
if [ -r /tmp/passwdsrvrtoken ]; then
  TOKEN="$(cat /tmp/passwdsrvrtoken)"
fi
~~~


extra blank line though?


A big +1 for this!


I am pretty sure this is still needed.


No, it was the only compilation error thanks to your abstraction of the safe functions.

Von: arstrube [mailto:notifications@github.com]
Gesendet: Dienstag, 10. März 2015 13:27
An: PaulBussmann/cpputest
Cc: PaulBussmann
Betreff: Re: [cpputest] Fix compilation issue with VS6: (378bc37)

This is strange. Similar constructs should be elsewhere in CppUTest - those don't give problems?

—
Reply to this email directly or view it on GitHub <https://github.com/PaulBussmann/cpputest/commit/378bc3711a0879f5fb2be90fa15cfbf71f98ca62#commitcomment-10119077> .  <https://github.com/notifications/beacon/AKas0ngRj40I4twpKmiWqhA1eYwCT3E8ks5nztqagaJpZM4DsQsw.gif>


I am not against this Business Requirements, the requirement is OK.

I am just saying that it has to be discussed FIRST and the 3 classes proposed in Plugins-devEnv will probably change anyway, so if you base other feature on top of that, you will have to anyway adjust them later on. For example, I need strong arguments for having Cryptography class (Cryptography.java) in the API.

So please create feature branch which contains the proposed interfaces, so that we can discuss there and then publish it to others.

Note: the commit https://github.com/UnifiedViews/Plugin-DevEnv/commit/a80d72cfa9af959cda6ea62acd61ded3956ed123 is also causing that DPUs build against this commit cannot be imported to Core being build before this commit.


This would be a good case to use argument defaults!

```js
function(options = {}) { ... }


Nice and big work, comrade!


oh good point. I'm about to change this function anyway to account for input while tilted, so I'll use glm functions when I do that


`login_user!` method is very cool, however I would just leave it to `session[:user_id] = user.id` and keep flash and redirect within `create`. Also `!` at method name signals something destructive usually, i would suggest to just leave it out.


Why did you rename path here? It seems a bit unnecessary and requires a lot of editing in other files.


Use argument defaults here, too.


add more spaces within the hash and you can write it as `order(created_at: :desc)` (as thats newer syntax)


I don't think these `if` statements are needed - should be handled by built-in validation


I want you to use similar way to render js templates https://github.com/janl/mustache.js#include-templates
But use <script type = "text/template"></script> and underscore.js internal templater. See how to show rendered template in the first link.
You may try to use http://handlebarsjs.com/. It's also a good lib.


@CarsomyrJ 검색이라는 작업이 각 모델들과 inherent한 관련이 있지 않기 때문에, 하는 일 혹은 책임 중심으로 나눈다는 측면에서 컨트롤러가 따로 있는 게 더 나은 구조. 지금이야 status 검색 밖에 없어서 `StatusController` 안에 넣는다고 해도 무리는 없는데, 다른 애들도 검색해야 하거나 혹은 통합 검색을 해야 하는 경우면 얘기가 달라지겠지?

@spilist 꼭 그렇지는 않음. 두 개 이상의 모델에 관여해야 할 때도 있고, 특정 모델과 관련 없는 컨트롤러도 있을 수 있음. 컨트롤러가 아주 커지면 액션 하나 하나를 서비스나 마이크로 컨트롤러로 빼내기도 하지. 그래서 위에서도 말했지만, 하는 일 혹은 책임 중심으로 나뉘게 됨.

트위터, 킥스타터, 깃헙, 페이스북 같은 다른 서비스들이 검색 기능을 어떤 url에서 제공하고 있는지 눈여겨봐봐. 검색 뿐 아니라 다른 여러가지 액션들도 보면 좋고. API 문서를 보고 어떤 endpoint들을 정의하는지도 보면 도움이 많이 되더라.

https://instagram.com/developer/endpoints/
https://dev.twitter.com/rest/public/
https://developer.github.com/v3/
http://strava.github.io/api/


i never had it enabled, IMHO the installer of piwikintegration atleast tries to disable it. Will test it, if it works out i will merge it.


This way - using an extended item attribute - I can set the selected-box CSS style verifying the actual cert port. If the port change I would have to change the port in the template as well to keep it being automatically selected.


Hmm, not sure, if defaults are possible here, because these are properties.


Well that is a bit annoying, because some of the referenced stuff in here is private/protected to Thread_base (e.g. _context, REINITIALIZED). In fact it is only the attach and id assignment that is the same. The detach for instance is not relevant for core.


It might just be very dated code, I can't remember now


> give the user the chance to change the requestAuthenticator at any time

I can see your point. My Issue with this is, that it is not obvious anymore which authenticator is currently used when calling `authenticateRequest ` especially if you have  some sort of "shared" Heimdall scenario. What about adding it as an additional argument to `authenticateRequest()`, maybe using `HeimdallResourceRequestAuthenticatorHTTPAuthorizationHeader` as its default?


You can use just `video` instead of `video_path(video)`


Trabalho 1


Liked to, but ESLint fails with the deconstruction from ES6. Same with classes and import… Wanted to do:

```js
const Geocoder = function({clientId = null, privateKey = null}) {};
```


@lol768 should return to index.md


You can use `= image_tag video.small_cover_url` but it is not necessary :)


Thanks!


Don't we need the condition to include after_callbacks?


Good job on Seeds!


@jshlee @gpetruc
I think the above line is inverted, should be
mappingPuppi[keyInOriginalCollection] = ic;


I'm not really sure - it was just a quick/dirty approach to get it working. I'll make sure to check it against other regions.


Az inAction egy index is a palyers lista adott elemére.
Az id mindig a listában való helyzetet adja meg.


You're right. So, it's not worth the effort.


I would recommend adding an empty line between specs to make it slightly more readable.


Yeah this line is lame. I'll split it out for sure.


A függvény neve lehetne egyszerűen getBigBlind()


You are describing a class method here. Use `describe ".search_by_title"`. `.` for class methods and `#` for instance methods.


This is wrong since `offer` manages this. This fill will only make the values `UNSET` the first round. See next comment in `seen`.


Yeah, I see Micky's point here.  The problem with `maybe_get` is the casual reader doesn't know the condition under which the get occurs.  It could be a coin flip.  It could the day of the week.  However `get_if_exists` tells the casual reader when the get occurs.


This doesn't look right, `zip` will clip to the shorter sequence. Maybe this instead? https://docs.python.org/2/library/unittest.html#unittest.TestCase.assertSequenceEqual


I could also use the {{cert}} string, but choose to use the port as the integer to apply it.


You should not manipulate the original request here. Removing `'` is only needed to prevent error in the geocoder.

Image you request `"cord d'azur"`: You'll expect the same string to be resolved.


This commit is against all rules. The revert must be done immediately.


An attempt to fix this issue which happens occationally:
```
Så kan jeg se kategorien i listen over lånerkategorier
features/step_definitions/patron_steps.rb:187

unable to locate element, using {:name=>"table_categorie_length", :tag_name=>"select"} (Watir::Exception::UnknownObjectException)
```


yea, those problems are solved, I now build binary, deb & rpm packages in a centos 6.6 container, tested the packages and binaries on ubuntu 12-14, debian stable, debian jessie , redhat/centos 5-7


You should validate against `itemsAhead` before getting the value from `arrayRef`


Remove print statement


I added some comments to the changes to `SequenceArray`.


need to have a test case for which the the fake GetRole , fake GetSpacerole return failure/error, so we need to add test cases & the fakes forthe same.


if yes, maybe you can update grafana production docker container to use debian:wheezy as a base image instead of debian:jessie?
https://github.com/grafana/grafana/blob/develop/docker/production/Dockerfile#L1


should be s|deployment|self|


please show only one Toast even when multiple files fail. Toasts are not stackable.
Also what if 1 fails but 20 succeed? Please indicate that in the Toast.


I like the idea of adding that as a parameter to `authenticateRequest`. However I think that someone using `Heimdall` in a singleton scenario should be aware that setting `requestAuthenticator` on the singleton instance obviously affects the singleton.

I still prefer the `var` approach because I think it's nice to configure the Heimdall instance once and the use it everywhere with as little configuration as possible.

We can talk about it when I'm back in the office :)


Nope: It's in the "command line" part of the annotator, so we'll allow it


Old commented code?


On a positive note, I think the function itself, regardless of its name, does clean up these two-line set-if-exists operations nicely.  I especially like not duplicating the text of the string.  It's a win.  Nice work.


hmmm, nothing came up on my end. We can look at it on Thursday.
> On Mar 10, 2015, at 8:15 AM, Felix Clack <notifications@github.com> wrote:
>
> This line, @user.favorites.where(post: @post) should return a collection of objects, that won't have a method after_create defined. So I would expect there to be a NoMethodError thrown here.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/clydiscope/bloccit/commit/6ccda536f53a0ef9ef274483e68dd57ba6d40c70#commitcomment-10113829>.
>


tested id with ? locally in FF and chrome, and it worked... :point_right:  probably not working in IE.


@ioanpocol  Can you please take a look at this implementation?


much enchancement so wow


Could you create a pull request with all your commits?


Hi @virasio

Not entirely sure what the issue is, but we have a few unpublished / internal branches, and I'm not sure what the state of Master is ATM.

We've rewritten a large part of the internal classes, which is why you see unused variables...

Note that while this repo is open, its not really for public consumption - we will be contributing developments back to Brad Larson's fork in the not too distant future, but its not ready for that just yet...


You might better place this inside of the promise to avoid duplicated request.

Not sure, but this might happen when requesting the same address twice: You'll access the cache before it gets populated.


Commented LOG, remove or uncomment


here are some changes made to this branch


Well, I'm not convinced it's a typo as we probably don't want to lock it. Running commands as the user mopidy has proven to be a useful debugging tool and I'm assuming locking the account will prevent us from doing that (please correct me if I'm wrong).


Поздравляю! :smile_cat:
Малаца! :+1:


I don't see that. How do I repro that?


Thanks, it's working now.


If we're doing this then, we want the copy semantics of sequence<DOMString> at least. DOMString[] is a mutable construct that I believe would allow modification into the track itself.

What's the motivation for this change? A dictionary guarantees non-recurrence of members, whereas an array does not. A dictionary test is a hash lookup, whereas an array is linear lookup. Seems like a lose lose.


Commented LOG. Remove or uncomment


Actually, looks like you can still do this by using sudo...


Added a "floor" to the scaledSize for iOS so, when scaling, the draw function doesn't perform an anti-alias operation at the right or bottom edge leaving a "white line" in the image.


It's locked by default on my PiMusicBox machine. Locking the account won't prevent su'ing to the user, you just won't be able to login as mopidy at the getty prompt, or through ssh.


I can't either, so let's drop for now, yep.


I think I'm done with it. And if you have time to look through a bit could you tell me if it is really ok this program?


T'as oublié un "A" :p


Also, the last time I introduced an array of strings into this spec, it wasn't popular. ;-)


Will the same ids be returned from the backend in the excluded_ids list?


No, they're present.


Absolutely leftover. Sorry will remove it and update fury.js


I'd be more comfortable if we just caught `boost::bad_lexical_cast`.  That's really the only problem we expect.  That way unexpected exceptions can slide past.


@catong LGTM. The feature's mostly self-explanatory, so this little blurb should be enough.


Yes :heart:


you could extract this to one method:
```coffeescript
@countBillable(team_members, true, @ui.billableCounter)
@countNonbillable(team_members, false, @ui.nonbillableCounter)
setCounter: (team_members, billable, $counter) ->
  ids = @roles
    .filter (role) -> role.get('billable') is billable
    .map (role) -> role.get('id')
  filtered = _.filter team_members, (user) -> user.get('role_id') in ids
  $counter.text(filtered.length)
```
also you can use underscore methods for collections on backbone collection
`@collection.filter` instead of `_.filter @collection.models`


More threads = more sources for bugs, correct? How difficult would it be to change this to a simple queue? (You would end up sending one packet at a time, even if it's for different destinations)


For reference, this part of the code is unused in the DAP dashboard, and was/is part of an original exploration of a more dynamic proxy.

There may come a time when we want to do more dynamic queries, and pull from a set of locally cached data (especially given Google Analytics API limits), in which case a persistence store will likely be needed. This isn't in the immediate plans.


sdf


Could you explain this change?


Pablo Gallardo Hiraldo


It's been a while since I've compiled mesos from source but I think that the python script is still called `test-executor.py`, not `example-executor`. The Java example that follows also appears to use a script named `example-executor` instead of `test-executor` -- whereas in the mesos source tree the script is called `test-executor.in` (and is probably transformed to `test-executor` by the build process).


We have net said anything about what notes can contain, other than they are treated as common properties. I'm not sure that's up to us, but it should be a separate issue. Still, that may place requirements on the value of notes, rather than the processing rules.


if you do not add documentation and test for this, i will revert it. this is unacceptable even for you. Also, please do PR when adding new module.


This change is available in the latest editors-pick commit c436748.


Hi,

 Today we tried the Coinbase .NET/C# Library in our web application. Initially we got one exception like the last discussed "No connection could be made because the target machine actively refused it 127.0.0.1:8888" .  we can get over this by modifying port number at client.Proxy = new WebProxy( "http://localhost.:54167", false ); with our localhost execution port number. It's OK BUT we are getting new issue like
![400](https://cloud.githubusercontent.com/assets/2277305/6577343/991b70ba-c713-11e4-84ee-f398b13c2cca.png)

 Please guide us to get over this issue.
Thanks in advance.


@Kelketek that's what I thought too :-) Thanks!


Use default for argument.


Can we pull this data out of this function?


we can, and probably should, kill this build flag


Muddies up the logic.


Mobile clients proxy servers list


@keiono,
please add text wrapping feature on your cytoscape desktop like this one.

![wrap](https://cloud.githubusercontent.com/assets/11401339/6577327/a1b27fc8-c76f-11e4-83f6-e6bccc10c33b.PNG)

at the moment, all edge label is shown horizontally, so when there are many edges, some edge labels will overlap each other.

you can see in this link. - https://github.com/cytoscape/cytoscape.js/pull/854

thanks.


looks good


Looks good


I don't think the fact that you can mutate the resulting array means you should expect the track itself to be changed — that wouldn't make much sense in this case any way. And I don't think the semantics of sequences help solve this either?

Re non-recurrence of members, it doesn't really matter anyway since that's an object provided by the browser which can certainly manage not to repeat entries.

Re linear lookup, sure, a Set would be better than an array. But I don't think it is such a big deal in practice (you're not going to call this very often in any case)

The main cost to having a new dictionary for this (i.e. if we don't re-use MediaTrackSetting) is that it makes a 3rd place where capabilities names have to be maintained; that doesn't make for a very nice extensibility.


You're already `refer`ing another function from this namespace, why not just use `:as`... `[clojure.string :as str]` ... `str/blank?`.


This is probably incomprehensible math that I'm hoping you'll straighten out. We need to figure out what distributional assumptions on the random term will work with our empirical specification.


구글 검색에서 나오는 참고자료에는 devise_parameter_sanitizer.for(:sign_up).append [:name]이 아니라  devise_parameter_sanitizer.for(:sign_up) { |u| u.permit(:email, :password, :password_confirmation, :firstname, :middlename, :lastname) }  이런 식으로 나온다. 즉, name이라는 걸 조건에 추가하려면 devise_parameter_sanitizer.for(:sign_up) { |u| u.permit(:name) } 이렇게 해야 할 것 같은데 이거랑 작성된 코드랑 같은 의미인지 아니면 다른 의미인지 궁금함


Valóban jobb helyen van itt, kényelmes lesz használni.

A this-re nincs szükség, de nem gond.


No problem! Thanks for writing it in the first place!


Or you could use lazy sequences... if you're man enough.


qidk what this does


No, what I want was to add a tests with comma after last item.
Because I saw no tests like that and I saw from implementation that fixer will remove that comma, when it should not do it.


`address` and `result` are more obvious here.


A CVE number might force all those running old MISP versions to finally update it..


It seems that stream extraction may be an expensive task. I am currently in the process of collecting performance information from this service. Further analysis required.


Is there no join operation? pthread, you had to join the thread back into the main thread. I could be missing something that QT automatically does though.. What are you trying to do overall?


Maybe put this in a `def` so you can add a docstring and a descriptive name `print-ip` or something.


Oh: I guess I should read the pdf rather than trying to look at the diff!

On Tue, Mar 10, 2015 at 10:55 AM, Kristy Buzard <notifications@github.com>
wrote:

> This is probably incomprehensible math that I'm hoping you'll straighten
> out. We need to figure out what distributional assumptions on the random
> term will work with our empirical specification.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kbuzard/political-uncertainty/commit/bf2f89be2aeae8bc326c261c9d9bc1be4e1a4ffb#commitcomment-10122004>
> .
>


I'm a belt and suspenders kind of guy, as long as the suspenders don't cost too much.  As the code is written there is no reason why `intVal` needs to be initialized.  My preference would be to initialize it anyway.  Then when the code is maintained in the future it will always have a known state.  And initializing an `int` is about as cheap as the suspenders can get.  Just my preference.  I won't hold up the pull request for it.


`address` and `result`.


i will, do not worry


Would `contains?` work?


Itt felhasználhattad volna a már elkészített gamestate.getCurrentPlayer() metódust


This breaks `make dev` with
```
# github.com/hashicorp/terraform/builtin/providers/cloudstack
builtin/providers/cloudstack/resources.go:44: not enough arguments in call to cs.Template.GetTemplateID
```
On a terraform build diverged from 23d90c0c02c10596eed79986e356b20bc6abb441


Why does the map need to be sorted?


Is this related to #1?


The wrath of Soumith is clearly not a force to be trifled with :-)


89639540-AirflowNetwork-Crash (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1647 of 1647 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)


Could pull this function out, DRY things up a bit.


89639540-AirflowNetwork-Crash (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-passing-green.svg)


89639540-AirflowNetwork-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1653 of 1653 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)


yea i get "test-" may be used in some places and not others etc - its more to improve (IMO) the naming conventions in the go bindings at least. "example" seems better named then "test" in an example package to me.


89639540-AirflowNetwork-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (637 of 637 tests passed)

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


Better use capital `I`: `getMetaInfo`


I like this.


Why yes, it is related to #1

On Tue, Mar 10, 2015 at 10:58 AM, Francis DiTraglia <
notifications@github.com> wrote:

> Is this related to #1
> <https://github.com/kbuzard/political-uncertainty/issues/1>?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kbuzard/political-uncertainty/commit/bf2f89be2aeae8bc326c261c9d9bc1be4e1a4ffb#commitcomment-10122103>
> .
>


@pmoust please do a `make updatedeps` or a specific `go get -u github.com/xanzy/go-cloudstack`


fast-forwarding develop to borshop-integration-716-de_add_is_stable_predicate_to_ring_manager = bbd7758d


I'm not sure this is a good idea to put this here.
Also, I wouldn't use a `<dl>` since it's not a list.
Finally, the term "linked to" doesn't mean anything to users.

I would remove the "program" information from the project page for now.


@gkellogg yes, I agree. But we should still say that the `notes` attribute in the metadata translates into `csvw:notes` in the generated RDF. This makes it a bit different from the common properties.


`enc`oding or `enc`ryption?


This change does not look right. This is the change I proposed: https://github.com/kaystrobach/TYPO3.piwikintegration/pull/30/files

What was merged here contains something else. This partly reverts https://github.com/kaystrobach/TYPO3.piwikintegration/pull/29 and so again introduces the errors, which pull request 29 was to fix.

I'll do a follow up.


will be done in a separate patch to replace all occurences


This function should be added to the `app` namespace.


I wouldn't use "Linked to" here. "Program" is enough.


To add to the `app` namespace.


Nice use of template specialization.


What about copyright! We're going to get our ass sued by the journal publishers! I'm shaking in my Wellington boots!


Function named `get...` probably expected to return a value - not to post.


No support for `numChannels` > 2


The same code is being run for `getBuffer()`.


free sonic boom and dragon rage in lc ubers


pls


broken ass mons, we should be able to use broken moves :^)


Can you make it so the select field is less wide?


What happens if you invoke `seen` before `offest`? Since the array contains zeros it might reply wrongly true, no?


Oh, we fully expect that folks _should_ set it to a value much larger than 5M -- we use 128M.  But if it is set to a value less than 5M, is is likely that the administrator has not bothered to configure MySQL; that is, we're using values less than 5M as "that's likely unchanged," not that we're trying to suggest that 5M is enough.

The reason we're doing it is because MySQL doesn't do a good job -- mysql will barf in your error logs, by which point you'll have lost the attachment, and many users will blame _RT_, not MySQL.  The purpose is to be proactive and catch these misconfigurations _before_ they cause people pain.


IIRC this was done due to multiple mains in the same package - @vladimirvivien would know more, i think we spoke about separate packages but i believe he settled on this approach.


okay thx ! Maybe we can find one app from play to simply showing actually refresh rate ? (just for fun xD)


I'll try to fix this. Thanks!


Nice :+1:


Hi Olivier,

thanks for the pull request!

I request some slight changes:

* Sat4J should be a class implementing the Prover interface, analogously to MiniSAT.
* There should be a FailSafeProver class implementing the Prover interface which calls MiniSAT if it is installed, and calls Sat4J otherwise.
* FailSafeProver should be used everwhere (?) where MiniSAT is currently used.

Before implementing these changes, I would like to do some refactoring to MiniSAT to make clear which parts of MiniSAT should be used as a "blueprint" for Sat4J. I will comment here again when the refactoring is finished.

Best,
Daniel


I think that's covered, as "notes" is a term, and a term is expanded by using the URI associates with it from [csvw-context].


updates


Well done!


Does the @[ ] construct work in Makefiles?  I'm pretty sure I've seen this fail before, but I could be remembering incorrectly.


參數char *fileName的確是要存出的檔名沒錯
至於ImgValue_t和ImgValue_s，原本應該是ImgValue_source 和 ImgValue_target
用來放原檔的值和要存檔的值，不過後來考量到要可以連續使用函式(ex: 轉灰階後作 downsample )，所以我後來的想法是把ImgValue_t當成一個暫存用的陣列，ImgValue_s用來存目前的值，之後每個函式都可以直接從ImgValue_s拿資料出來作處理
所以我建議可以在Conv2Gray()最後加一行ImgValue_s = ImgValue_t;
純粹我的想法，有錯請指證


The `unByKey` method is deprecated. Use `ol.Observable.unByKey(key)` instead.


Yes it is misspelled.  This query needs seeks to get the facility data for the current facility, which is '3' for me but not others.  Will have to track this down in Globals and insert here.


So if one jsfiddle can see my cameras then all jsfiddles can? http://jsfiddle.net is http, not https, so I think this is too permissive. I think it should be limited to the current page, like getUserMedia on HTTP is.


i like!


This fallback won't fix the value of androidhls. Should add something like
    androidhls = providers;

Also, since this is meant to be internal (and is only surfaced for easier testing), I would prefer a comment like
//TODO: JW6 plugin support, remove when plugins are updated


Just a small typo! `time t_stop` --> time `t_stop`


You don't really test anything here.
You should extend the tests so that you ensure that a user is able:
 - to change the program for a project,
 - to remove any relationship with a project.


Position of `{` is inconsistent with the rest of the code (here and below), it would be nice to use consistent style everywhere.


Related to #2


lc ubers is meant to be like pokebank lc w/ scyther tho


Can you add `{}` even around single line if statements, please?


Related to #4


@duvillierA I had upgrade tetra on legacy too


This looks to be the right thing to do for SQLite but I wonder what is SOCI level of support for the nested transactions in general. It would be nice to add tests for them to `common-tests.h` to see if the other backends pass them too...


@rapharezzi, esse teste não tava passando porque "bar" não é primo mesmo hehe.
My bad :facepunch:


@PCMan I had to changed this to make the panel find the status notifier plugin. I don't know why.


Great. Please make sure the existing createNode() method calls it (so we don't have two different codepaths for creating a node).


Did a M504, and it started working. Rerunning the calibration


@opdude Good catch, and yes, we should be checking for that in windows. Care to make that change/pull request? If not, we can grab it, too. Let me know so we don't lose track of this.


Setting fonts in ApplySchemeSettings is what causes the unnecessary infinite loop, you should not do that.

And I honestly doubt this breaks anything. All this commit does is it removes one pointless SetFontInternal call. You should use either DLabel:SetFontInternal or DLabel:SetFont outside of ApplySchemeSettings. ( SetFont never worked in ApplySchemeSettings because it always called ApplySchemeSettings )


which line is that on?


I added support for nii.gz files.
Have you considered adding the conversion from brainbrowser format to nifti?


See https://github.com/kaystrobach/TYPO3.piwikintegration/pull/33


89639540-AirflowNetwork-Crash (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1653 of 1653 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)


@skenqbx Can you sign the commit with your real name next time?  Thanks.


You are totally right, I just updated the branch with one more test and the change. Also, there is a test correction as well as some source formatting.


I just used it as a core.  Parts of it remain because I didn't understand them - I'm sure it can be furthered whittled down.  Because of the substantial re-purposing of this add_edit_issue.php file, I left the original alone and renamed this to a_issue.php and it is located in the eye_mag directory.  You'll see it on the next commit if I can figure it out on my end: the GitHub client for Mac is crashing so I am setting up the new repositories using another program combined with command line Git.  For me this is trial and error at its best, like pulling teeth, without dental school or anesthesia. Soldiering on.  Just hang on a bit 'til I can come to grips with all this code...


I never removed anything. Someone apparently added support for it from their own fork of this repo without me knowing, and my recent update has overwritten that. All development for pelican-bootstrap3 happens here: https://github.com/DandyDev/pelican-bootstrap3 The version in getpelican/pelican-themes is a copy that I will try to keep up to date. I should probably convert it into a git submodule to make that more clear, but I don't know how @justinmayer feels about that.


/O0 doesn’t help. I still get the same linker error.

Von: arstrube [mailto:notifications@github.com]
Gesendet: Dienstag, 10. März 2015 13:30
An: PaulBussmann/cpputest
Cc: PaulBussmann
Betreff: Re: [cpputest] Adding to CPPUTEST_MEM_LEAK_DETECTION_DISABLED to VS6 project to get around unresolved externals: (5cc6186)

If I remember correctly, /O2 optimizes away the symbol even though it is actually used.

—
Reply to this email directly or view it on GitHub <https://github.com/PaulBussmann/cpputest/commit/5cc6186ec6eaa88b428dce69c5b9cbe9bbfbdd12#commitcomment-10119122> .  <https://github.com/notifications/beacon/AKas0hQeHhD2_7o9C4EpkF530pUkQ28bks5nzttDgaJpZM4DsMmk.gif>


I suppose you meant to say "Clone the repo by issuing `git clone ....` on the command line".


Hm, does the patch below fix the issue?
https://gist.github.com/b275af5da780ca584025


That's a great thing to change in the next release. (Add a textbox that takes the repo name, with SeattleTestbed the default value).


Use `id` instead of `pid` here for consistency.


Great Roman!
Spot on.
This does exactly what I needed. Thanks a lot.

 foreach (var v in vertexes)
 {
 foreach (var kv in v)
 {
 Debug.WriteLine(String.Format("{0}: {1} ({2})", kv.Key, kv.Value, kv.Value.GetType()));
 }
 }

VS IntelliSense fooled me because when hovering over  "db.query()" it indicates a List<ODocument> output and all my efforts went towards obtaining fields from ODocument

Instead I notice that you achieved it with a simple KeyValuePair.
Marvellous

Thanks again.


@maplmi


Good catch, changed on master.


@OlegIlyenko I would remove the singletons as well. Too many small instances, I can't either see the benefit, but rather a problem. When you change this, we can merge.


I guess it might be worth ordering these memberships, perhaps by start date descending?


@obsoleted: Was it so that this callback is only called on Windows and on iOS, everything is handled behind that replyAcceptSession exec call?

The issue that I addressed in this commit was that in the Windows proxy JavaScript, we delete the AllJoyn message object, but for that, we have to wait until the replyAcceptSession completes. So changed it so that it is called in replyAcceptSession exec success callback.

Let me know if you think this change creates issues from iOS perspective.


@svanharmelen this was caught by our acceptance/CI system. It does `make updatedeps` each run before issuing a `make dev` to compile terraform.


Where does this function come from?

Also, what about existing solutions with likely higher degrees of abstraction, such as http://datatables.net/ or https://en.wikipedia.org/wiki/Help:Sorting ?


Awesome tutorial, thanks @tmcw!


@rallytime Yes made a PR already #21484


I mean, there are no deps fetched in the `$GOPATH`, runs in a container, completely fresh each time.


They are set to defaults when used with a parser. Do you want to use them standalone or why is this necessary? And if so I think there are better ways to do this (e.g. a seprate `Namespace` object) so that we don't mess with internal class attributes and arguments in the same place. E.g. we add a `sections` parameter to `SECTIONS`. Then what? We can do all internals private, then we forget, then we hate ourselves... :smile: Plus we have methods there, who cares about them when viewing all the "arguments"?


1: M111 unrecognized component name: assets


1: M111 unrecognized component name: travis


24:1: I201 Missing newline before sections or imports.
25:1: I101 Imported names are in the wrong order. Should be _request_ctx_stack, current_app
25:1: I100 Imports statements are in the wrong order. from flask should be before import six
25:1: I201 Missing newline before sections or imports.
27:1: I201 Missing newline before sections or imports.
29:1: I201 Missing newline before sections or imports.
235:9: B901 blind except: statement


21:1: I201 Missing newline before sections or imports.
23:1: I100 Imports statements are in the wrong order. from mock should be before import pkg_resources
24:1: I101 Imported names are in the wrong order. Should be ImportPathRegistry, PkgResourcesDirDiscoveryRegistry, RegistryProxy
24:1: I100 Imports statements are in the wrong order. from flask.ext.registry should be before from mock
24:1: I201 Missing newline before sections or imports.
29:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, nottest, run_test_suite
43:1: E731 do not assign a lambda expression, use a def
45:1: E731 do not assign a lambda expression, use a def
47:1: E731 do not assign a lambda expression, use a def
586:42: E231 missing whitespace after ','
586:44: E231 missing whitespace after ','
599:17: E127 continuation line over-indented for visual indent
599:48: E228 missing whitespace around modulo operator


It should be fine to call setup on a device that's already active.


The column header currently doesn't indicate that there exists any `onclick` functionality. How about saying "Type (click to sort)"?


I'm not sure jsfiddle is a good guide here (the jsfilddle model is at odd with the per origin security), nor that http and https have anything to do with it, nor that getUserMedia is a good comparison point (sharing label of media devices is hardly of the same scale as sharing media streams); but since I don't really care, I'll update the PR towards your model and we can revisit it if it proves too restrictive based on impl experience.


Thanks, I'll drop the udev call.


Sweet, :+1: !


Ah ok. I was the one who added it. Sorry for the misuse of the repository.
I however feel that that shall be clear somewhere, otherwise people contribute to the repository and then changes are silently overwritten. Some changes by @higebu have also been removed.

May we come up with a better solution? @justinmayer what do you think about it?

Thanks!


Combining 'generateUploadLink' and 'generateDownloadLink' into one function is no better than the origin.


Hi Pimm
Do mean to just include one section like PMH section?


New


Seems reasonable to me, but what if a user wants to use the netinst iso? Should there be a separate target for netinst?