Better `!frameState`?


Why is `rfi_filter_raw` moved here? Is this Crab specific?


Suggested usage line:

      sign_for <signer_address> <signer_private_key> <tx_json> [offline]


Ruby will let you return multiple things so you don't have to return an array:

```
tmpdirs, host_to_result_dir = find_tmp_dirs
```

and then have the method return those two things:

```
def find_tmp_dirs
  ...
  tmp_dirs, host_to_result_dir
end
```


Make this a bit more descriptive `dataModelAliases` (and use camelcase). Who knows how long this will be a temporary fix for.


This seems to imply the times and phases get calculated twice, which seems wasteful.  Also, Crab-specific, so perhaps better done in a separate PR.


I see you do not like my formatting....


Nu har vi ju två gitignore, ligger en i .meteor-mappen också. Den gäller väl också för hela projektet?


Precompiling IJulia (which uses JSON) is giving me:
```
WARNING: eval from module Parser to IJulia:
Expr(:import, :DataStructures, :OrderedDict)::Any
  ** incremental compilation may be broken for this module **
```
Why not just `REQUIRE` the `DataStructure.jl` module?  I don't understand the reason for the conditional import here.


Its not you .... its me ...


:+1: It finally happened. :+1:


Initial Project


Good point. Thanks. :)


Maybe just make a size var = 10, and use this for the sizes of the arrays, instead of .length and [0].length


@jordanschalm Can you rebuild `article-0.1.11.js` for production (just use `gulp`) and recommit? We don't want un-minified code making its way to production.


generators op


# This is the second test


Hm, this may need to be modified to be more careful about importing into its own module then?

DataStructures is bigger than JSON and the ordered feature is optional AIUI, though I didn't write the feature.


Oh how I laughed, I laughed so much :D


I don't think Bootstrap versus pure.css is a matter of what is better or easier to understand. I think each framework allows you to use CSS (and in the case of Bootstrap, JavaScript) in slightly different ways to do different things. (Although, from what I've seen on the pure.css website, pure.css has better official documentation/user scaffolding.) From what I read on the two websites, Bootstrap emphasizes mobile-first development, while pure.css focuses on being lightweight. I picked pure.css for its focus on being lightweight.


touche


@derekeder @cathydeng This was the culprit. Very, very sneaky and a non-obvious error (since it only manifested when debug was set to ``False``


validated, the code within quotes is solid.  yes i am on 0.9.9.8. so you want this command to run with exit 0? i am just trying to clarify, because i see the resource exec'ing every run.   wanted to make sure it shouldnt be onlyif


fixed, but future edits to autogenerated content will now get automatically overwritten due to #13162


I don't understand. Why isn't it included as a dependency by the generator, then?


@dexX7 A lot of BigDecimal -> Omni*Value, Coin in this commit.


Thanks for checking into it. From your description it sounds like it was unaffected by the change. Just wanted to make a note of it.


cool!


LowerCaseSymLink (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1896 of 1896 tests passed)

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


I like the text. But I don't like the `<br />`


I guess a question is if mathematically this is a degenerate case. I don't know enough about the details but I could think that interpolating between two points having the same value `n` on the first axis could produce a valid value with the interpolation on the other coordinate?
Again I could be totally off the mark


This change addresses both #15 and #8. However, it unfortunately introduces issues with CocoonJS.


Do you mean to say is the "date" that the base file calculates?


really?


For consistency, should we change this to `date --utc`?


NAH


I guess `a` cannot be more than 1. Nice use of greek symbols on the formula


I also had some trouble getting PureCSS to work for my page. Where did you find the stylesheet that you ended up using?

I was thinking of a stylesheet in editing as well. In both cases, the stylesheet sets out guidelines that will direct how the rest of the document is formatted or written.


去掉了警告信息； 不需要参考这个工程。


i can't even think of anything to comment on here. You seem to have a much better grasp of what's going on than I do.


:ok:


Could we make this just call `metdata-log --group=$tool_group --dir=$benchmark_run_dir int` instead of "`end`", where that logic above is handled in that script?  This would then help keep all the script behavioral details in the script instead of in a bench script.

What do you think?


LowerCaseSymLink (Myoldmopar) - 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)


简单的app导航页，传入图片和结束后要跳转的controller即可


Nice call on the interrupt.  Could we also record somewhere that the run ended due to an interrupt?


I don't use this package beyond fetching `newinstall.sh`, so I'm unsure of the history.  I suspect the table is/was being used to setup the env for `lsstsw`.


@kreatordxx, @btb,

Do you have any suggestions for improving the Mac OS X instructions?


@zicodxx, @Mako88,

Could you read this and critique any place where its instructions would not produce a working environment on your system, assuming you started from a base system without any development tools installed?


I agree that the CSS seems to mesh well with the html. Did you try any of the (I thought) trickier stuff that Duckett talks about with creating boxes and so on? In that situation, too, the CSS targets a section of the html to add formatting around it. I tried a box at one point but it didn't really work out the way I wanted it to. Similarly, were you able to adjust your page layout using Pure, or did it mainly just apply its own styles to the page?


In some cases the removal of severity seems to break other constraints within the same parent class.


It returns the values of the key.


Watch that you are analyzing the right file.  Your code looks good, though.


@nguyen-tuan-mulodo please check grammar and vocabulary when you write comments such as:
"bebfore delete posts" ==> need to change "before deleting posts"


This answer changes a little after you do Problem 6!


Why this change?


Are you using Python 3 or 2? You'll need parentheses around all your print statements for Python 3.


Thank you for comment. I've tested on ffmpeg 2.8 and it works fine.
```
# mediainfo 10_58_32.mp4
General
Complete name                            : 10_58_32.mp4
Format                                   : MPEG-4
Format profile                           : Base Media
Codec ID                                 : isom
File size                                : 721 KiB
Duration                                 : 1s 467ms
Overall bit rate mode                    : Variable
Overall bit rate                         : 4 028 Kbps
Encoded date                             : UTC 1904-01-01 00:00:00
Tagged date                              : UTC 1904-01-01 00:00:00
Writing application                      : Lavf56.40.101

Video
ID                                       : 1
Format                                   : HEVC
Format/Info                              : High Efficiency Video Coding
Format profile                           : Main@L3@Main
Codec ID                                 : hev1
Codec ID/Info                            : High Efficiency Video Coding
Duration                                 : 1s 467ms
Bit rate mode                            : Variable
Bit rate                                 : 4 019 Kbps
Width                                    : 640 pixels
Height                                   : 480 pixels
Display aspect ratio                     : 4:3
Frame rate mode                          : Constant
Frame rate                               : 15.000 fps
Color space                              : YUV
Chroma subsampling                       : 4:2:0
Bit depth                                : 8 bits
Bits/(Pixel*Frame)                       : 0.872
Stream size                              : 720 KiB (100%)
Writing library                          : x265 1.7:[Linux][GCC 5.1.1][64 bit]
Encoding settings                        : wpp / ctu=32 / min-cu-size=16 / max-tu-size=32 / tu-intra-depth=1 / tu-inter-depth=1 / me=0 / subme=0 / merange=57 / no-rect / no-amp / max-merge=2 / temporal-mvp / early-skip / rdpenalty=0 / no-tskip / no-tskip-fast / strong-intra-smoothing / no-lossless / no-cu-lossless / no-constrained-intra / fast-intra / open-gop / no-temporal-layers / interlace=0 / keyint=250 / min-keyint=15 / scenecut=0 / rc-lookahead=0 / lookahead-slices=0 / bframes=0 / bframe-bias=0 / b-adapt=0 / ref=1 / no-weightp / no-weightb / aq-mode=0 / aq-strength=0.00 / cbqpoffs=0 / crqpoffs=0 / rd=2 / psy-rd=0.30 / rdoq-level=0 / psy-rdoq=0.00 / no-signhide / deblock / no-sao / no-sao-non-deblock / no-b-pyramid / no-cutree / rc=crf / crf=18.0 / qcomp=0.60 / qpmin=0 / qpmax=51 / qpstep=4 / ipratio=1.40
Encoded date                             : UTC 1904-01-01 00:00:00
Tagged date                              : UTC 1904-01-01 00:00:00
```


restController


This is not necessary. Use `new File(libDir, resource)`


These need to be a raw count, not a ratio.


Last two agents

> Is it possible for remote baremetal nodes to host containers and manage resources inside them? ;-)

no


What is this for?


That's fair. It's certainly true that most editor somehow figure it out pretty well on their own (maybe it's already the default). Thanks for pointing out this resource in the docs.


It's better to use a standalone method `isNativeLibrary`


It's better to use a standalone method `isAsset`


Thanks for the update!


If there's no necessary zip entry in the zip file, it's fatal error we should make the app crash


Thanks for this nice reusable solution!

I'm curious if y'all considered using a restful solution first? I've often found myself in similar situations where there's too much logic associated with putting a back-end record in a certain state, and a non-restful action seems like the simple solution. But often there's a way to think about it in a restful way - e.g. in this case, you could create a Dismissal, which belongsTo your model. Then you'd be able to leverage all the conventions, and simply `store.createRecord('dismissal', {model: yourModel}).save()` and then deal with the logic in your backend `dismissals_controller`.

Just curious if you went through that and decided against it, and if so why. Thanks!


I see similarity between web browsing - the surface part that users see - and graphic user interfaces such as Windows. There is a lot happening that we just don't see. What's happening with browsers seems to me to have fewer layers to it than Windows, but that may just be because I'm not familiar with what goes in to creating Windows.


Come on!


Ok I did more testing on it. The removal of severity from an attribute seems to change the generated OCL to true; and the removal of severity from an association seems to completely remove the generated OCL for that association.


If the user supplies a directory path with a trailing slash, then we need to remove the trailing slash here for the sake of the comparison, not for +/+.

File.existsCaseSensitive("C:\\blah\\blah\\blah\\");


Space after `for`


Space after `if`


"These groups center around (8,0,8)." would be better - if that's even correct?


Anyway, I might drop and re-push this branch b/c of a typo in the comment (*shame*)


Tried this to no avail... can anyone help me with... see if it's my bad or something wonky.
1076742474   09  29

much appreciated.


ok


Watch out - what you want here is literally the AT count over the GC count.  Your atgc_count variable actually contains GC over ATGC.


this isn't actually accurate, but it's somewhat better than the old version for now, and while still trying to comprehend how it actually does work to more accurately describe it.


I think this will break if you have two different routes with the same set of parameters. I don't see a reason to change it from what it was before.


So that the `@Repro-Step-0x` is reset for each test.


Post made.  not a peep yet, but I would expect 'some' word or 2

http://www.openwall.com/lists/john-dev/2015/09/28/2


This matches the order of definitions in C++ CssErrors, and they appear to be grouped by functionality (database/table/key, etc.)


Better not to split the parameter name from the type annotation.


Does not really affect the test but while debugging I realized that I was passing the original acls as empty where as the original acls has the one acl that we added in the last step.


Yuchung Cheng is from Taiwan


I didn't find the stylesheet. The Pure website created it and downloaded it for me - like I said - automagic. I can show you in class if you want. The href is a link to a yahoo page.


Updated documentation for "Server Installation > Installing Keycloak Server as Root Context" to work with Keycloak 1.5


Just a personal style, not a real problem. I'll prefer:

```
String entryDir = "";
if (isNativeLibrary() {
    if(Build.CPU_ABI.equalsIgnoreCase("armeabi")) {
        entryDir = "lib" + File.separator + "armeabi-v7a" + File.separator;
    } else {
        entryDir = "lib" + File.separator + Build.CPU_ABI + File.separator;
    }
} else if (isAsset()) {
    entryDir = "assets" + File.separator;
}
ZipEntry entry = zipFile.getEntry(entryDir + resource);
````


I can't think of any way short of an explicit comment to clarify that within the code as it stands.  Basically, the protocol requires that any chunk of data be prepended by its total length noted as 4 hexadecimal digits.  The magic number 4 in the original code corresponded to the last 4 bytes of the final string, the `'0000'` part.

The magic number 15 corresponds to length of the strings `'# service='` and `"\n0000"`.  We then add the length of the `pack_type` string to 15 since that string will be part of the final advertisement string along with the former strings.

Perhaps this code would be more clear?

```ruby
str = "# service=#{pack_type}\n0000"
"%04x%s" % [str.size, str]
```

Honestly, I could go either way on this.  It's *slightly* less efficient than my first suggestion while being slightly more efficient than the original code, but it's more clear since the magic numbers are gone.


Why don't we set it back in `SetupIdeLogFolder` to be consistent to `testScreenshotIndex` which gets reset in Setup


I believe that order is important to swig, classes that are used by other classes should be processed first.


Please remove HTML tags from the description, it messed up the `<meta>` tag in the final HTML.


我赵日天并不服


Watch out - when you do Problem 6, the answer to this changes.


Commit message here is a bit excessive.


that will be nice, thanks :+1:


How about this?
```cpp
return osg::Matrix(
        tf(0, 0), tf(0, 1), tf(0, 2), tf(0, 3),
        tf(1, 0), tf(1, 1), tf(1, 2), tf(1, 3),
        tf(2, 0), tf(2, 1), tf(2, 2), tf(2, 3),
        tf(3, 0), tf(3, 1), tf(3, 2), tf(3, 3));
```


Commit message unhelpful


It's better to keep the parameter destDir for now. We may receive the request of customizing the loading path in the future.


Why mention about this man specially?  Why?
They're all from Google.  Good jobs, guys!


Removed all placeholder content here, and replaced `ul` with a `div`


You are correct.

Now I have to publish two plugins with slightly difference to npm.

cordova-plugin-admobpro, for those who use cordova/meteor/ionic CLI, can build android apk with gradle.

cordova-plugin-admob, for those who use PhoneGap Build/Intel XDK, use ant to build instead of gradle.

The javascript API and code is exactly same.


rax retry tests


Updating these dynamically added items took me a little while to figure out. I had to re-scan the entire document to find these in the dom. If there's a better approach, I'm totally open to suggestions.


@jodson, Do you will make this commit in BaseActionBarActivity?


Would something more specific like "Minimalism" be an improvement over "theoretical linguistics"?


For #149


Oh, osg and Eigen uses different order for matrices. The above code should be:
```cpp
  return osg::Matrix(
        tf(0, 0), tf(1, 0), tf(2, 0), tf(3, 0),
        tf(0, 1), tf(1, 1), tf(2, 1), tf(3, 1),
        tf(0, 2), tf(1, 2), tf(2, 2), tf(3, 2),
        tf(0, 3), tf(1, 3), tf(2, 3), tf(3, 3));
```


Why don't close the zip file?


これも好みなので勝手にいじってスミマセンという感じなのですが
全角記号（！？、。）の前後で半角スペースを空けると謎の空白感が…
半角記号（!?,.）であれば私もスペース入れる派です（こまかい…）


I was finally able to test this tonight. I'm still seeing the following repeatably and consistently:

1) Power on the FC with the transmitter off
2) Power on the transmitter and quickly arm the FC
3) Power off the transmitter within about ten seconds
4) The motors hold their throttle setting and never stop

That's with failsafe_delay=10 and failsafe_off_delay=10, S.BUS receiver.

If the transmitter is on for more than about ten seconds, failsafe works as it should.


This could be simplified to:

```
resourcesToAcls.map(ResourceToCommand).foldLeft(Array[String]())(_ ++ _)
```


LowerCaseSymLink (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (822 of 822 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-24.31%25-green.svg)


silly


OK, thanks.


[Data 150914.xlsx](https://github.com/jblasberg/Econ1970_Blasberg/files/4112/Data.150914.xlsx)


Just a comment, "xwalkcorelib" is a strange name. We are using "xwalkcore" or "xwalklib" in everywhere.


A lo mejor por las lambdas de la javafx...


Yes. I will.


What feature do we use that requires 3.0? (It'd be good to mention that.)


Shouldn't we be using http://www.goldencheetah.org/blank.html instead of https://goldencheetah.github.io/blank.html?
Who knows if in the future you are using GitHub to host the pages?


This is a markdown file


あー、全角記号の後でスペース入れるのは、原稿用紙に書いてた時代の名残です(懐かしい)
ツイートでも `！` の後にはスペース入れちゃってる程度にはただの癖なので、修正してもらって問題ないですよ。

個人的には、こういった「スペース入れる・入れない」とか「全角半角どっち使う」とか「codeで囲むか囲まないか」みたいなのは、組織や集団内ではある程度の統一をとっておいて欲しい派なので (コーディング規約と同じだよねそれと思ってる) この辺りの指摘は嬉しいです！


Readded .gitignore


Welcome back to the future :)


We should add corresponding instructions and samples in the class description.


Bump


I think I'd prefer a global update, so everyone's experience is the same


I agree but for cocoapod users is more comfortable samples and libs in the same project. They can use `pod try` and I think that's easier to maintain example updated.

To make easy track and review sample updated can be done in another branch or unique commit. What do you think about?


Was that so hard?


the date will be the start date and not show recurring. should update your date in the database. refer to CreateNewBlockSessionActivity saveOnClick method.


I just renamed satellite_new.py to satellite_new.py.test.

Not sure anyone is looking at this, but will leave it in git for now, but will no longer install it.


whAT og wHY


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1360 of 1360 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)


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18568/testReport/) for this commit**


yeah, I'll fix that


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18569/testReport/) for this commit**


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18570/testReport/) for this commit**


@h2o64 So how does one open menu then if theres no menu button?


Haha it also confused the linguist:

<img width="807" alt="screen shot 2015-09-29 at 1 04 06 pm" src="https://cloud.githubusercontent.com/assets/2547035/10176603/a0c4179a-66aa-11e5-839c-de8de98acd98.png">


Should mention that `Android Support Repository` is also needed to make framework resolving work. The image has it but it's not obvious that it is also required.


Thanks from here too, @simonmysun!


"With a dash" is what you wrote in your commit log, that's what I was referring to.


Yep. Otherwise, there may be conflicts or other weirdness.


That first '10' at line 203 could be put in a var, and so used for the rest, it's a bit messy having all those + 9 and + 10 - 9 don't you think? ^^' That wouldn't kill the memory :P


Sorry... I'll fix it now


Added a check to stop the script running from inside the installation folder as these causes quite a few subtle issues.


@davidfowl - I have not even noticed this but I committed this yesterday and this now has `std::endl`


When I read this it sounds like it's telling me Sandstorm only requires SNI for clients that don't support it, which is weird.

Maybe: "client-does-not-support-sni.sandstorm-requires-sni.invalid"?


That's strange. Why did the original code cause a crash?

I see an erase call on AuctionsMap, which is basically an std::map, and std::map's erase function only invalidates the iterators referring to the elements being removed by erase. Which is not a problem, because the C++ standard guarantees that all argument expressions will be fully evaluated before the function is actually called, and there is a **post**-increment operator here which means `itr` is incremented and the original value is returned.

So I believe this was perfectly valid code that should not crash with a decent C++ compiler.

But that said, I certainly don't disagree with the more elegant C++11 code.


Haha I think it's good to go now


LOL...

I used the wrong sign for scores at line 414 and after.
Should be -= because attackunits are "added" together,
then substracted from score.

I can't believe I spent 1.5 days "tuning" this crap.


This sentence looks like a TODO list ;-) Please remove it from here.


Should this be named "invalid-no-sni.key" or something? If I saw "0.key" in a directory I'd assume it was valid.


I'm not that excited about this change. This new terminology seems clunky, though I do agree that the
previous text wasn't that great either.

What I'd like to do here is have a stereotyped block either at the top or bottom of each extension that simply
indicates the extension status. e.g.,

Client: MUST send
Server: MUST send in ServerHello

or:

Server: MUST NOT send.
Server: MUST send in encrypted extensions.

This would replace all the text blocks.

I haven't decided exactly how I want this to look, but if you want to suggest a format, I'll be happy to take a look


No, I guess there's nothing in this release that required a major version bump.


For Synchronous it takes 1m46s
For Asynchronous it takes 7s :sunglasses:


What if environment variable doesn't exist?


update function is not working.. do some changes


+1 see latest commit


respuestas de las preguntas del taller 2


@Rocky640 : I believe mistakes are a sign that we are doing a good job. I hope this helps.


consider > considers


LGTM


Removed by accident?


Updated via https://github.com/winjs/winjs/commit/cb1adc56719c4cb4bfdfd81a18702ec52a09e01e.


It's happening on the r9b build too? The purpose of this commit was to fix the issue you mentioned.


que merda é esta?


download system > download system updates.


You open menu through the application option. Menu hardware is gone.


You can just do
```python
with self:
    etc
```


FIXME not a constant


Does the fact that your are passing a struct to a template that requires a class not throw everyone else's compiler for a loop?


Hi Daisuke,

I'm not sure I understood what you mean. Do the estimated values fall outside the [0...1] range because of the noise (e.g. they can be 1.03 or 0.98) or they are totally off (e.g. the range between 0 and 20)?

In the code, I tried to cope with possible scaling of the data, i.e. scl_slope and scl_inter parameters; then when we divide all the signal vector 'y' by the b0, this scaling should disappear.

Can you please provide me with more details so that I can reproduce the issue you found and understand what is going on?

Thanks a lot!
Ale


Discussed IRL; the directory name is hopefully enough clue.


@Usaga no complains since this release! thanks a lot for your work! :)


We should make this more common code as well. Most of this is already in robot_smach_states.util.startup I think. Which of those features are missing?


This seems to the effect that if any docs change we often end up rebuilding the Julia sysimage, which is unfortunate.


Can simplify this by removing the if check. `expanduser` already does this check and returns without action if there is no `~`.


The reason I changed the code was that it did not work. It did not copy the phantomjs binary with its execution permission (chmod +x) that was the only reason for switching to copy task because it has options mode true which preserves permissions


I would prefer to
```python
import robot_smach_states.util.designators as ds
```
and use
```python
ds.VariableDesignator(etc)
```


Oh, right, that was a mistake.


This really does'n make sense because user need to clone repo locally and perform all operations from root of project.


In the case below, can't you overlap?

```Python
In [1]: import numpy as np
In [2]: a = np.arange(10)
In [3]: a
Out[3]: array([0, 1, 2, 3, 4, 5, 6, 7, 8, 9])
In [4]: a = np.arange(9).reshape(3, 3)
In [5]: a
Out[5]:
array([[0, 1, 2],
       [3, 4, 5],
       [6, 7, 8]])
In [6]: b = [a[0], a[1], a[2]]
In [7]: a[1:] = b[:-1]
In [8]: a
Out[8]:
array([[0, 1, 2],
       [0, 1, 2],
       [0, 1, 2]])
In [9]: b
Out[9]: [array([0, 1, 2]), array([0, 1, 2]), array([0, 1, 2])]
```


its beautiful


theres a typo in this btw


Other boolean command line args use /arg+ and /arg-. This should be the same.


What compiler are you using? Passing on Travis and been building fine
locally since then


Ok, I checked again.
I think the problem is that the first alternative in

```scala
def `optional-cookie-pair`: Rule1[Option[HttpCookiePair]] = rule {
    (`cookie-pair` ~> (Some(_))) |
      (`cookie-name` ~ ws('=') ~ zeroOrMore((VCHAR -- ";\r\n" -- EOI)) ~> ((_: String) ⇒ None))
  }
```

is too broad. It matches "a=1" as a prefix of "a=1,b=2" even when it shouldn't.

If you change the first alternative to:

```scala
(`cookie-pair` ~ &(ch(';') | EOI) ~> (Some(_))) | ...
```

all your `Cookie` header tests pass.


That's not manageSieve POM...

We need to rename this commit ;-)


so this will be called every 5 minutes by [trimLabels() periodic task](https://github.com/jenkinsci/jenkins/blob/9c443c8d5bafd63fce574f6d0cf400cd8fe1f124/core/src/main/java/jenkins/model/Jenkins.java#L845-L850)


Not sure what you mean by passing a struct where a class is required, a
struct and a class are basically the same thing


Well or amend it !


THX! Now it's __perfect__!


Building fine on Linux using 4.9, but having trouble with Visual Studio 2015, not really good with windows.

Gotcha about that part, don't know too much about using templates with C++. But documentation says "class"


I think you need to add `n * \exp(\sigma)` to the loss.


Where is gradient? How do you compute dL/d \sigama?


I was actually wrong about this, sorry about that. This assumes that watchify's main is not in a subdirectory.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18576/testReport/) for this commit**


Locomotion is misspelled, but otherwise it looks good to me.


Yes, this was wrong.  I replaced it with another commit, hoping nobody would notice O=]

The updated commit is here: https://github.com/cmangos/mangos-classic/commit/b35a20a5137d10c44c26807c528013443ec56186 (note that the crash is not present in -wotlk).


:+1:  Heh.


'as basis' here is 'based on' or 'rely on'


Ok for the buffer check with ndarray but id() is not provided this way for others numpy_expr like numpy_gexpr which doesn't have an id() function but which can overlap with ndarray.


Do you mean just `exp(sigma)`. I've just added it. The `n` term is the same as `n_outliers`


the download is from master branch (subtitleedit-master.zip)

and in <b>AssemblyInfo.cs.template</b> I edit 2 lines for ex:

[assembly: AssemblyDescription("[GITHASH]")] to [assembly: AssemblyDescription(" b67fd3a")]

[assembly: AssemblyVersion("3.4.8.[REVNO]")] to [assembly: AssemblyVersion("3.4.8.552")]


I've created a PR on Lela to add this support (based on your commit) https://github.com/kif-framework/Lela/pull/17


nit: could you include the headers in alphabetical order


@jbecla I am a bit concerned that `sqlalchemy` is part of the public API of the `db` package. Do we want that to be the case? Are the `sqlalchemy` exceptions being tested here part of a private interface that is purely there for the internal testing or are we expecting end users to be catching these exceptions? If the latter then I worry that the encapsulation is wrong and you have no opportunity to easily change the internals of `db` again without affecting lots of user code.


Nice comment.  It helps.


Maybe the `space` between the `beginning single quote` and `http` makes this problem:
````
$ brew readall
Error: problem in /usr/local/Library/Taps/weareleka/homebrew-avr/avr-gcc.rb
invalid attribute: url (" http://ftpmirror.gnu.org/gcc/gcc-4.9.2/gcc-4.9.2.tar.bz2")
````
Also in execution of `brew upgrade --all` and `brew doctor`:
````
$ brew upgrade --all
Error: invalid attribute: url (" http://ftpmirror.gnu.org/gcc/gcc-4.9.2/gcc-4.9.2.tar.bz2")
Please report this bug:
    https://git.io/brew-troubleshooting
/usr/local/Library/Homebrew/formula.rb:190:in `validate_attributes!'
/usr/local/Library/Homebrew/formula.rb:153:in `initialize'
/usr/local/Library/Homebrew/formulary.rb:70:in `new'
/usr/local/Library/Homebrew/formulary.rb:70:in `get_formula'
/usr/local/Library/Homebrew/formulary.rb:165:in `get_formula'
/usr/local/Library/Homebrew/formulary.rb:205:in `factory'
/usr/local/Library/Homebrew/formulary.rb:223:in `from_rack'
/usr/local/Library/Homebrew/formula.rb:1008:in `block in installed'
/usr/local/Library/Homebrew/formula.rb:1006:in `map'
/usr/local/Library/Homebrew/formula.rb:1006:in `installed'
/usr/local/Library/Homebrew/cmd/doctor.rb:1018:in `check_missing_deps'
/usr/local/Library/Homebrew/cmd/doctor.rb:1277:in `block in doctor'
/usr/local/Library/Homebrew/cmd/doctor.rb:1275:in `each'
/usr/local/Library/Homebrew/cmd/doctor.rb:1275:in `doctor'
/usr/local/Library/brew.rb:133:in `<main>'
````


Nope, it wasn't that.  I'll fill you guys in over there JuliaIO/QuartzImageIO.jl#4


basisjs-tools going to do this, but not yet :)


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21319/testReport/) for this commit**


these 2 "fisheye" libs are removed from the base nvd3 repo... they don't add a graph we care about and they squash the default line and lineChart implementations.


Hi Mr. Huynh,

I just added you as contributor. Let me know if you need anything else.

Thank You
Vo, Phung


or? not and?


says compiler v140


In order for `*t` to be `NULL`, `pattern` would need to be `NULL`, right? Which isn't possible as we `assert()` on that above.


do you have 371f4748d47c429fdcec2a247f1f4f647f7d3835?


Is that the fault of this change? The sysimg should depend on helpdb whether or not you run gensdtlib, running it just ensures changes get written into the RST more immediately.


If this test was not failing, let's close it because we cannot reproduce it outside of sampler.


I can't parse this. What are you trying to do?


Yup.  I think this is a good change.  If we're going to distribute the checks in this way, we might as well go all in.


Same here.


Shouldn't you check for `terminator` here?


Hmm, where's the SVG??


@EdHurtig just an FYI, this has no effect, except to permit v0.12.0 of the `logstash` cookbook to be used. If a higher version < 0.13.0 is available, it will be used instead.


It can't overlap even if `E0 == broadcast<T0, T1>`


Not sure. Haven't quite tracked down why it's happening.


works now, thanks


I don't know, but the convenience fallback to `:` in case of `NULL` really causes more confusion on the caller side than being explicit, don't you think?


The rules exist to break them. ;-)


Спасибо, проверил. У вас раставлен уникальный ключ по полям lft rgt, как я понял, вот он и ругается. lft, rgt не должны быть уникальными индексами! Проверьте миграшки.


Not in the csproj?


@jeffbyrnes v0.12.1 wasn't in the supermarket at the time


@CatoTH Kannst du bitte die neusten Änderungen deployen?


@EdHurtig makes perfect sense, in that case :smile:


`v(s)`


lolz :smile:


Good observation... reverting to what it was


`func` -> FUNC


but users change their status


A `goto` statement that jumps backward is awkward. I'd much prefer a a `do { ... } while()` loop here.


l r? left right?


Are you sure about this -1?


Yes, I think no one of `Server` objects created in tests are get properly closed.


I would add a note mentioning the namespace, ie `YahooFinanza::WorkerName`


o, its pretty cool, tnx


So, if you run `/ban dualspiral blah`, this code will get the console to run `/ban dualspiral blah`. Because `/ban` is registered to this plugin, that will make the plugin run `/ban dualspiral blah`. And because `/ban` is registered to this plugin, that will make the plugin run `/ban dualspiral blah`... ad infinitum.

All this method does is mask the /ban command. Would it not be better if you DIDN'T implement this until the ban service is available?


every place you write to a pipe, you probably should use `print` and BYO "\n". `puts` isn't atomic, it writes the string and then the newline. For example, run something like `1.upto(100) { Thread.new { puts "hi" } }` in pry/irb and you'll get some lines with "hihi" and some with just newlines.


Hmm, we were doing this in the past, but without restricting the font-locking to the input, as the previous commit does, and it caused problems. Now it might make sense to enable this by default.


You  may just be able to call `parse` because you included it in the class as a module. Not 100% though, just a thought


But I like logs


Line 565 is not needed, that's common knowledge.
Line 566, perhaps better: 'Mapchunks of default chunksize 5 are offset by (-32, -32, -32) from the origin.
The offset is in 3D, so is your code, so (-32, -32, -32) is needed, and "center around (8,0,8)" is incorrect (it's actually (8, 8, 8).
I'm not sure how mapchunks of other sizes are offset, but the mapblocks themselves are always aligned by (0, 0, 0).
These 'groups' should be called mapchunks.


Yet another not


This isn't a line comment


This comment looks to be not related to the code.


@jezdez
Thanks for your answer!

While the [diff between master and 0.8](https://github.com/jezdez/django-configurations/compare/0.8...master) is quite large, most of the changes are of administrative nature or are bug fixes.
Apart from that, basically everybody with a decent version of Django has to use django-configurations from Git by now - so I would consider it quite stable.

Therefore I think a new release (0.8.1) is (still) much more appropriate to be done from (current) master than after merging your local changes and waiting for them to stabilize - the latter seem to be a better fit for a later 0.9 release anyway.

In case of helping out, me and @ticosax would be happy to help you out in this regard again, like we've done with envdir already.  Since you can do new releases fairly easy, you could keep that commit bit for PyPI for now, and just let us handle the issues/PRs for example.

Thanks! :octocat:


which makes this a little redundant: `vers("macro-compat")`


probably better to use `Process::WNOHANG`, test the return, and sleep if it's not dead yet.


These tests call `get`, every time. This is suboptimal. We should only call this once for each case because it slows the suite bigtime. One call for each quote type done total for all tests is where we should be.


groups -> mapchunks


`it's`


The naming here is ambiguous. `pop` generally implies a mutating action on an array/list. However here it means `popular`. But in the actual method definition it sounds like you are slicing quotes from the end of a pre-existing list and mutating its state, and this is not the case. I would rename this.


shouldn't that be set to true?


I don't understand the purpose of this change? You're adding `requirejs-tpl` as a peer dependency of itself?


We should consider pulling in ActiveSupport for its text transformation methods to normalize these to all lowercase snakecase


I'm not sure it's middleware, but extension..


y se podria discutir pero va a contra mano de como va todo el firmware :(. yo creo es mejor tener todo ordenado por tema, osea agrupados macros arriba, luego typos, luego datos lcoales, luego datos externos, luego funciones locales, luego funciones externas. pero alguien podria decir mejor juntar datos y funciones externas qeu esten relacionadas. entonces qeudarian datos funciones otros datos otras funciones. se entiende?


The one for you


Can you unzip the HIVE crypto wallet file directly into this them since it is workable for HTML5? It is in a zipped source file, and if I can indeed just input and unzip, it will save a tremendous amount of time!


@c-varberg i closed the 'css-to-bottom' branch last week. could you move this change to 'master' or to a new branch if necessary? then close down the 'css-to-bottom' branch when you're done. lmk if you have questions.


la validacion la haria en un archivo que NO sea de configuracion. sino alguien puede pensar en configurar los macros de chqueo que obviamente no es tu idea :) por ende diria que tenes que tener un _Cfg para que el usuario modifique y otro Usb_Check.h para los controles, te parece?


@snodnipper Good catch, thanks for contributing!


Seems like a good idea. I have experimented with other values and not seen the same speedup. I'll try to run some tests on it to see if the performance improvement is consistent.


Nothing goes unnoticed on the internet. But nice try! :wink:


a mi entender el const en el tipo no es lo que vos queres, lo que queres es el const en la definicion, osea este const lo pdoes dejar si no cmabias los datos pero importante es el const de la definicion.


es mucho más facil de seguir el codigo si usas un typedef Usb_ProbeFctType, al menos para mi. y además más prolijo ya que si luego hay que modificar modificas el tipo y no la estructura. y si a futuro se llega a  usar en otro lado es más facil :) a nadie se le ocurre hacer copy and paste sino que usa el tipo ya definido.


In other news, will you be on irc sometime soon?  I'd like to discuss the C++11 backport to -tbc and -classic with you :)


File have to be created using MINC2 API from master release (the develop branch now contains another speedup code: 78214a1ff62a726c984298969590136e9c5bf21d)


I'm not sure if all of these should be part of the plugin. These might be what catalysts is using and what version, but perhaps shouldn't be part of the plugin. Some projects might not use scalatest or specs2 or have macros.


On the fence, but I wonder if we could just use a tuple for this: `"dwijnand" -> "Dale Wijnand"`


As mentioned, this test should check the `::after` pseudo element to ensure that `right` is set to `auto`.


Not tabs please, we really should use spaces for indentation.


@torkelo Now you say it, it reminds me I saw that, but forgot to correct it. Sorry for this mistake. But, my comment was more related to the exec part than to the copy.
``
var exec = require('child_process').execFileSync;
var ph=exec(src,['-v'], { stdio: 'ignore' });
``
was intended to throw if the binary found from `npm` was not able to execute proprely on the current plateform, which could happen due to this [issue] (https://github.com/Medium/phantomjs/issues/376).


May I suggest something like the following. I've marked a couple of enum values with `/*VERIFY*/` because there is no direct correspondence between the historic `DokanResult` values and the present `NtStatus` values.

    /// <summary>
    /// Defines result status codes for Dokan operations.
    /// </summary>
    public enum DokanResult : long
    {
        /// <summary>
        /// The operation completed successfully.
        /// </summary>
        Success = NtStatus.Success,

        /// <summary>
        /// Incorrect function.
        /// </summary>
        Error = NtStatus.Error,

        /// <summary>
        /// The system cannot find the file specified.
        /// </summary>
        FileNotFound = NtStatus.ObjectNameNotFound,/*VERIFY*/

        /// <summary>
        /// The system cannot find the path specified.
        /// </summary>
        PathNotFound = NtStatus.ObjectPathNotFound,

        /// <summary>
        /// Access is denied.
        /// </summary>
        AccessDenied = NtStatus.AccessDenied,

        /// <summary>
        /// The handle is invalid.
        /// </summary>
        InvalidHandle = NtStatus.InvalidHandle,

        /// <summary>
        /// The device is not ready.
        /// </summary>
        NotReady = NtStatus.DeviceBusy,/*VERIFY*/

        /// <summary>
        /// The process cannot access the file because it is being used by another process.
        /// </summary>
        SharingViolation = NtStatus.SharingViolation,

        /// <summary>
        /// The file exists.
        /// </summary>
        FileExists = NtStatus.ObjectNameExists,

        /// <summary>
        /// There is not enough space on the disk.
        /// </summary>
        DiskFull = NtStatus.DiskFull,

        /// <summary>
        /// This function is not supported on this system.
        /// </summary>
        NotImplemented = NtStatus.NotImplemented,

        /// <summary>
        /// The data area passed to a system call is too small.
        /// </summary>
        BufferTooSmall = NtStatus.BufferTooSmall,

        /// <summary>
        /// The filename, directory name, or volume label syntax is incorrect.
        /// </summary>
        InvalidName = NtStatus.ObjectNameInvalid,

        /// <summary>
        /// The directory is not empty.
        /// </summary>
        DirectoryNotEmpty = NtStatus.DirectoryNotEmpty,

        /// <summary>
        /// Cannot create a file when that file already exists.
        /// </summary>
        AlreadyExists = NtStatus.ObjectNameCollision,/*VERIFY*/

        /// <summary>
        /// An exception occurred in the service when handling the control request.
        /// </summary>
        InternalError = NtStatus.InternalError,/*VERIFY*/

        /// <summary>
        /// A required privilege is not held by the client.
        /// </summary>
        PrivilegeNotHeld = NtStatus.PrivilegeNotHeld,

        /// <summary>
        /// The requested operation was unsuccessful.
        /// </summary>
        Unsuccessful = NtStatus.Unsuccessful,

        Undefined = long.MaxValue
    }


The "Download Windows buld" button doesn't seem to work right now, so I couldn't test it:
https://camo.githubusercontent.com/fe87bc4d79e427187f20fe3940334c122814aea6/68747470733a2f2f696d672e736869656c64732e696f2f62616467652f446f776e6c6f61642d57696e646f77732532304275696c642d677265656e2e737667

Will that build now contain a john-local.conf? I thought that shouldn't ever happen.


Outdated?


this is causing warnings:
```
[warning] The getter 'minScrollOffset' is not defined for the class 'ScrollBehavior' (/usr/local/google/home/ianh/dev/sky_engine/src/sky/packages/sky/lib/src/fn3/scrollable.dart, line 129, col 37)
[warning] The getter 'maxScrollOffset' is not defined for the class 'ScrollBehavior' (/usr/local/google/home/ianh/dev/sky_engine/src/sky/packages/sky/lib/src/fn3/scrollable.dart, line 129, col 80)
```


It appears you didn't actually check in your test.


nice one!


Code style nit: add a space after `{`


Code style: add a space before `=`.


:fireworks: Yay! :fireworks:


Ahem. To be found in your TOC ^^ needs to regenerate / update the table of contents again. E.g. by re-running the `doctoc` command.

Many thanks again for the reference to `dreamcat4/pipework` spin-off project.


Happy hacking!


Good point - esp. when /boot/efi is just the default. Removed in ac3a5c86e13a17426343f6f18d66d149a892baba .


Actually command output should be fixed :-/


Ignoring unforeseen events, I should be home all evening on Thursday. I can join IRC then.
And because we're an international team: where I say evening, I mean roughly 7-10 PM, UTC time.


:clap:


Metron listens to a UDP port, not TCP


The move constructor is always used when return optimizations are not performed. AFAIK, we shouldn't [care](http://en.cppreference.com/w/cpp/language/copy_elision) [about](http://stackoverflow.com/questions/4986673/c11-rvalues-and-move-semantics-confusion-return-statement) [that](https://llvm.org/bugs/show_bug.cgi?id=19458). Actually, compiling lxqt-panel with clang right now will give us some warnings:

```
/home/paulo/Workspace/lxqt/lxqt-panel/panel/panelpluginsmodel.cpp:99:12: warning:
      moving a local object in a return statement prevents copy elision
      [-Wpessimizing-move]
    return std::move(names);
           ^
/home/paulo/Workspace/lxqt/lxqt-panel/panel/panelpluginsmodel.cpp:99:12: note:
      remove std::move call here
    return std::move(names);
           ^~~~~~~~~~     ~
/home/paulo/Workspace/lxqt/lxqt-panel/panel/panelpluginsmodel.cpp:108:12: warning:
      moving a local object in a return statement prevents copy elision
      [-Wpessimizing-move]
    return std::move(plugins);
           ^
/home/paulo/Workspace/lxqt/lxqt-panel/panel/panelpluginsmodel.cpp:108:12: note:
      remove std::move call here
    return std::move(plugins);
           ^~~~~~~~~~       ~
2 warnings generated.
```

Those are lines 99 and 108 of [lxqt-panel/panel/panelpluginsmodel.cpp](https://github.com/lxde/lxqt-panel/blob/master/panel/panelpluginsmodel.cpp). We should not use move semantics in return values.


> Oraclefied

![Oh god why](http://i.imgur.com/7xLNBwV.png)


although this is moot if you define JAVA_HOME...

but still nice! will merge it now


Pourquoi est ce qu'on peut faire ça? On récupère ici le contenue de la gexpr d'avant pour en faire une nouvelle gexpr. Si on prend seulement une ref sur la gexpr d'avant (qui était pas une const& apparement sinon ça crasherait à la compilation), on a donc une ref sur le contenue de la gexpr précédent qui est maintenant supprimer donc c'est une dangling référence non?


I updated ide but didn't set indentation. will update on master after release


`schema.extractCsvAttributes()` always succeeds.  It will fill in missing values with default values.
One way to get around that is to add a sentinel value in the returned map that indicates if there were any flags in the schema file.  But that is a bit hackish.
Plus, one could argue that specifying the CSV attributes in code **should** override what is in the file  (if you wanted to force to use a field as an int even though it is defined as a string in the schema).
In either case, the csv attributes on reading is deprecated and it is only there for backward compatibility.


Benchmarks in https://github.com/jruby/jruby/issues/3355#issuecomment-144176035 if you haven't seen it.


Is the intent behind this to separate out the etcd clusters which loggregator uses vs what diego uses? If so, we should name this property "loggregator.etcd.machines"


me confundi y commitee todos los cambios en vez de solo la primera implementacion del logger.
este commit incluye:
- primera implementacion del logger
- servidor recibe e identifica todas las requets con su informacion
- tests agregados


Scary method name :) will this potentially cause data loss?


Then the user should proceed as they would have before, by entering the keys through the UI.


What if the value is null?


I should check it in! I suppose I could just link the svg instead of the png rendering?


Overall I'm fine with the change, but why are you doing it? The mixins, filter and sort-method can already be customized using their respective methods. Does access to them really help? What about the other ones? It might be nicer to provides customizer methods, than to make these variables `protected`


This I'll add to the locale file properly, tomorrow, but just want the tests to pass now.


Gute Arbeit  :+1:. Listen sollten noch parametrisiert und Null-Referenzen abfangen werden.
Optional kann noch einiges mit Streams ersetzt werden, wenn noch Zeit bleibt :).


typo, identier -> identifier


works for me.

that says memory usage is ~100MB doesn't it?

ENV var would be good.

a
On 30/09/2015 4:14 am, "Vladimir S. FONOV" <notifications@github.com> wrote:

> @rdvincent <https://github.com/rdvincent> @andrewjanke
> <https://github.com/andrewjanke> what do you think? This speeds up file
> access to MINC2 files created with MINC2 API dramatically, but uses about
> 10 times more memory:
>
> (old code)
> \time mincstats -sum library_stx_sym/local_model.mnc
> Sum: 2048604543
> 54.20user 0.39system 0:54.59elapsed 100%CPU (0avgtext+0avgdata
> 10540maxresident)k
>
> (new code)
> \time mincstats -sum library_stx_sym/local_model.mnc
> Sum: 2048604543
> 1.32user 0.05system 0:01.40elapsed 98%CPU (0avgtext+0avgdata
> 109812maxresident)k
> 0inputs+0outputs (0major+27689minor)pagefaults 0swaps
>
> $ mincinfo library_stx_sym/local_model.mnc
> file: library_stx_sym/local_model.mnc
> image: unsigned short 0 to 65535
> image dimensions: zspace yspace xspace
> dimension name length step start
> -------------- ------ ---- -----
> zspace 311 0.3 -78
> yspace 313 0.3 -110
> xspace 475 0.3 -72.8
>
> Should it be configurable with environment variable perhaps?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BIC-MNI/libminc/commit/08c0124648ad1411c8cbba6ab52862db86a770a5#commitcomment-13500107>
> .
>


Already done: https://github.com/dotnet/roslyn/commit/163f332ddebffae6c5fda4cc6cc948f85f3d2b05


Whoops.


If there's no other place where this is used we should also remove it from the database. Do we normally do that with a data migration or manually from the admin?


Does this mean that they worked?


Yeah I would, just use `<img src="thesvg.svg">` instead of this object tag stuff.


@rbvermaa, shouldn't `pkgs/tools/package-management/nixops/unstable.nix` be removed along with this change?


@xyz32 just make a PR.


This commit introduced some strange symbols in generated files that git now treats as binary: history.html, TODO.html ant e.t.c. In fact, when I try to re-generate the docs from pywinauto repo I also get these symbols. Is it related to sphinx version or docs-sources where changed ?


since this is a public API, we should `SOL_NULL_CHECK(type, NULL);` to protect against bogus `packet` (like NULL), that would return `type == NULL`. @cabelitos


Thanks Andrey!


What is the wallet supposed to integrate with?


Ну, с точки зрения русского языка тут наверно нужно двоеточие, а не тире. Не уверен.


@GrahamCampbell would you like to send this fixer into original repo?


It wasn't causing problems, I just noticed it because I'm about to try and shrink the size of the wx grid we're grabbing based on the forecast type. Looks like right now we always buffer 1 degree. Plotting a large number of wx model vectors is causing a problem in the phone app. Any reason not to do do that (shrink the size of the grid based on the model)?


If there are animation changes you'd like to see in the release, please try to get back with me before Oct 3.


Cache will use up-to 100Mb , probably less if file is smaller.


doctype "b i t c h"


Looks like Github lost my comment. Metron listens on UDP, not TCP.


Ha I found something! :smiling_imp: "accound"


You actually, need to choose the rotation axis and rotate it 180 degrees in the catch statement

There is an implementation for the same problem at Ogre3D wiki: "Sometimes my program quits and I get an Assert - Divide by Zero. What's up?" http://www.ogre3d.org/tikiwiki/Quaternion+and+Rotation+Primer


mmh not quite, instead do:

      polarization = Reference(reftype=PolarizationSubjectReference,
                                            backref=True,
                                            readonly=True,)


Nope, the commit looks good.  I just forgot to take out the 1 degree bound when I replaced it with the dynamic one that I cut and pasted incorrectly :)


@johnou no.. it just discard some already read bytes from the inbound buffer


Testing it now
On Sep 29, 2015 3:11 PM, "Jeff Taylor" <notifications@github.com> wrote:

> Does this mean that they worked?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Astroua/CARTAvis/commit/9028edad4a755c1b66098a502b1fd45fcb7baa60#commitcomment-13503708>
> .
>


It seems that `bpp` is only used in the library for the Cinema Profile warning (I guess the images components where in fact 12bits in the resulting jp2 file when you filed #609 ...)
The rest of the code in openjp2 uses `prec` (or so it seems doing a search on the project)
I don't know why the 2 co-exist... maybe @detonin have some insight on this. Documentation doesn't help.


@andrewkroh When can we get this into Packetbeat? Currently I am doing some manual tests there and my logs are just flooded with this entry because it cannot be disabled.


TeamCity pull requests :: netty-tcnative [Build 14](https://garage.netty.io/teamcity/viewLog.html?buildId=1841&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Perhaps we can state it more clearly as "bytes that are at risk for deletion." Also can we return the number of files that contain such blocks?


tested and working


TeamCity pull requests :: netty-tcnative [Build 14](https://garage.netty.io/teamcity/viewLog.html?buildId=1841&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:42


NIT: `allPartitions` is a bit confusing, it probably should be `allReplicas`.


Left a minor comment. Otherwise, LGTM!


> Lines 565, 566 are not really needed.

Well perhaps best leave them in since they are improvements to existing documentaton.


What does the equal sign do exactly?


As the plugin currently stands, the focus stays in the output window, so this changes the default. I'd prefer to keep the default the same so behavior doesn't change on users. I also think we should consider a name like `g:http_client_focus_output_window` (that defaults to 1) which to me conveys the intent more accurately.


@johjacb I added the code I changed to master. I'm not sure how to close the css-to-bottom


Maybe we could try to enable this if 1) MozActivity is undefined or 2) the activity throws an error event (e.g. the activity doesn't exist)?


Could this be better written as:

```julia
bswap{T<:WordIntTypes}(x::T) = box(T, bswap_int(unbox(T,x))
```
?


Esto esta bueno!!!


your heading in the right direction! keep up the good work! :+1:


Editing **AssemblyInfo.cs.template** is the problem. UpdateAssemblyInfo searches a line looking like:
```
[assembly:AssemblyVersion("3.4.8.[REVNO]")]
```
The three numbers can be different, and there can be some extra white space.
If there is no such line, an exception is thrown: (displayed when you run `build.bat`)
```
Malformed template file: '...\src\Properties\AssemblyInfo.cs.template'
```
And UpdateAssemblyInfo will exit with a value of 1 to indicate failure.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18591/testReport/) for this commit**


@bryanmtl just wanted you to be aware i've put these new stuff in from Lola:

1. subscription now stores its own email address (similar to Order) so users can change the email and order renewals will use that new email

2. skip history - and an on going dev that let them skip next renewal continously

3. consumption of store credits if available during renewal.

i'm also moving lola to use 2-4-stable


k


@steffengy fix


Off to a great start! :+1:


No trouble. I'd say use even more if it makes things faster. Chrome seems
happy to munch 2GB these days...

a

On 30 September 2015 at 07:15, Vladimir S. FONOV <notifications@github.com>
wrote:

> Cache will use up-to 100Mb , probably less if file is smaller.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BIC-MNI/libminc/commit/08c0124648ad1411c8cbba6ab52862db86a770a5#commitcomment-13503773>
> .
>


If it's not too much of a hassle, can you help me setup the build process and does the ramdisk need any modifications? Just for the kernel.


Also related to #315


I don't think this is part of the .Netcore, is it possible to work around this dependency?


Ah, right: https://github.com/djacobsmeyer/bloc-jams/blob/master/styles/main.css#L59.


looks good to me.  thanks for doing more testing.


I renamed the method here to be able to test it.


Car did not ask for mutator methods it wants you to rely on the constructor


It seems you've missed adding the third parameter to some calls of ShowTextFloater.


That is quite a big run() method you got there ;-)
Wouldn't it be better to have this as a method of MarkersAndTaxa and expose that to bbmerge, e.g. `smrt bbmerge --all` or something?


Since it's a hot fix, sure, do what you need.


You should always try to declare as close to initialization as possible I would move this inline with the statement on line   :36


https://github.com/MaPePeR/ProjectE/commit/bd58a596f0a2cfc0e8bb98dc99a55399f35dc765#diff-b2a61f21624781c2a79c5834ec847006L48

Sloppy. Produces nullpointerexceptions when you try to grab the proxy.


I suppose it's expected for `nixops` and `nixopsUnstable` to diverge again soon, at least on master.


Very nice!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.986.64654f3c5ff58c68e87e49ac928ded58792fe14b](http://build.palaso.org/viewLog.html?buildId=63871&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:03:08


Looks like you are only using it for ZipFile which should be ok.


You're completely right... thanks for the lesson.


These need access modifiers like `private` Fields are almost always `private`


Interesting approach it does reduce repetition but the declarations at the top could also have been in the body of the loop


It is not needed, but it is already there. Or you let it as it is, it is not bad, or you complete remove both  `saveRadar` and `savebutton`, anything else will likely increase code interdependency. In other words, if there is a GUI, the backend (pyart.io.write) shall also be there.


When we get things done, the windows build will NOT be taken from CircleCI anyway.  All of that build a .zip logic will get strip (probably should be stripped now).   @kholia and I worked a bit on getting things going, but did not get the final downloads 'working'.  I hope to link up with him at some time, and re-try.

I am building them on a Win box, under cygwin, and then will zip them and upload to a linode VM I own.

But we kind of have the plan in limbo at this time.

But you are 100% right, this logic change MUST be handled to not have that john-local.conf file ever 'live'.  It is being built in the ./src directory, and that is where the --test-full=0 is being run from, so I do not think there would be any way it would 'leak' into the binary release, even if it was being 'used'


I am confused with what is happening here. We are assigning itemNumber and then prompting the user for input to overwrite it. Was :39 - :41 vestigial?


Hot pink!

I'm starting to figure out that lots of the styles present in the couchtuner-companion are inherited from the CSS bleeding in from the actual site. I actually like most of the styles present over there, so I'm trying to get the sidebar to look the same on both sites.

Perhaps I should just create some common CSS doc or something, then I can use hot pink :D

Did you try it on Putlocker or something?


Lots of cleanup to do -> But performance first.


If you declared you RetailItem instances in the loop you would not have to make this decision and reduce some complexity.


Andrey,
I believe you also need to register MYSQLI_TYPE_JSON in ext/mysqli/mysqli.c.


LGTM


All assets were removed and new scene without ball! you shouldn't modify the master branch.


these one are failing on my ubuntu box

py.test -v test_cartavis.py::test_setColormap

py.test -v test_cartavis.py::test_setImage

py.test -v test_cartavis.py::test_invertColormap

py.test -v test_cartavis.py::test_reverseColormap

py.test -v test_cartavis.py::test_setColorMix

On Tue, Sep 29, 2015 at 3:17 PM, Alex Strilets <astrilet@gmail.com> wrote:

> Testing it now
> On Sep 29, 2015 3:11 PM, "Jeff Taylor" <notifications@github.com> wrote:
>
>> Does this mean that they worked?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Astroua/CARTAvis/commit/9028edad4a755c1b66098a502b1fd45fcb7baa60#commitcomment-13503708>
>> .
>>
>


this should be a pass, or something that works when filename is not there (don't just give up!)


actual

y.test -v test_cartavis.py::test_setColormap

py.test -v test_cartavis.py::test_setImage

py.test -v test_cartavis.py::test_invertColormap

py.test -v test_cartavis.py::test_reverseColormap
 last one passed

On Tue, Sep 29, 2015 at 3:31 PM, Alex Strilets <astrilet@gmail.com> wrote:

> these one are failing on my ubuntu box
>
> py.test -v test_cartavis.py::test_setColormap
>
> py.test -v test_cartavis.py::test_setImage
>
> py.test -v test_cartavis.py::test_invertColormap
>
> py.test -v test_cartavis.py::test_reverseColormap
>
> py.test -v test_cartavis.py::test_setColorMix
>
>
> On Tue, Sep 29, 2015 at 3:17 PM, Alex Strilets <astrilet@gmail.com> wrote:
>
>> Testing it now
>> On Sep 29, 2015 3:11 PM, "Jeff Taylor" <notifications@github.com> wrote:
>>
>>> Does this mean that they worked?
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/Astroua/CARTAvis/commit/9028edad4a755c1b66098a502b1fd45fcb7baa60#commitcomment-13503708>
>>> .
>>>
>>
>


The main contributor of slow speed CODECHEF just got faster. Spoj HackerEarth and HackerRank still left but mostly done. #12 solved


The RetalItem driver is very verbose. I see what you were doing here. You should consider what it would have been like to write these blocks as unit test methods instead of taking user input just providing your own basic input.


and tests for `None` is better to be explicit


Overall this is very good. I think the driver for RetailItem is too complex but your objects are nearly perfect and have only minor style issues so Good Job!


hi mom


looks like you are saving to /data directory that I do not have

On Tue, Sep 29, 2015 at 3:31 PM, Alex Strilets <astrilet@gmail.com> wrote:

> actual
>
> y.test -v test_cartavis.py::test_setColormap
>
> py.test -v test_cartavis.py::test_setImage
>
> py.test -v test_cartavis.py::test_invertColormap
>
> py.test -v test_cartavis.py::test_reverseColormap
>  last one passed
>
> On Tue, Sep 29, 2015 at 3:31 PM, Alex Strilets <astrilet@gmail.com> wrote:
>
>> these one are failing on my ubuntu box
>>
>> py.test -v test_cartavis.py::test_setColormap
>>
>> py.test -v test_cartavis.py::test_setImage
>>
>> py.test -v test_cartavis.py::test_invertColormap
>>
>> py.test -v test_cartavis.py::test_reverseColormap
>>
>> py.test -v test_cartavis.py::test_setColorMix
>>
>>
>> On Tue, Sep 29, 2015 at 3:17 PM, Alex Strilets <astrilet@gmail.com>
>> wrote:
>>
>>> Testing it now
>>> On Sep 29, 2015 3:11 PM, "Jeff Taylor" <notifications@github.com> wrote:
>>>
>>>> Does this mean that they worked?
>>>>
>>>> —
>>>> Reply to this email directly or view it on GitHub
>>>> <https://github.com/Astroua/CARTAvis/commit/9028edad4a755c1b66098a502b1fd45fcb7baa60#commitcomment-13503708>
>>>> .
>>>>
>>>
>>
>


realpath is not available on OS X :(


pd-from bottom to up. Initialize the last row. Every sum is the from the two elements next to it


and if the user changes his mind and cancel?


"primary" or "administrative" or something. We have heaps of mailing lists, and I agree that we shouldn't change the lists people are working on.

So the big question is if we should shift (in practice, cc minutes etc) all the administrivia announcements (what Art will call make-work) to html-admin which is specifically designed for the purpose, or figure out how to divide between more lists.


thanks, just what I thought.  But I saw a remote chance that this may have a part in dakcarto's build problem on OSX.


This can happen even without blocks with future generation stamps e.g. DN is restarted after a long downtime and reports blocks for deleted files. Perhaps we can leave this line out.


@jpopelka it would be helpful if you updated puppetlabs with this commit. I happened to re-create this exact commit on my own and came on github to fork it and merge request it. :)


Yeah probably. I have to look more into the new API tomorrow.


Tha password stands also for a token, for which you dont need username


sos un gil


This may be confusing. Perhaps state "If you are sure that the NameNode was started with the correct metadata files then you may proceed with '-safemode forceExit'."


Closes #24


@codepathreview thanks for the comments dirk :) Sorry for not creating an issue for this one - realized that one cannot create an issue on a forked repo a bit too late :)... This assignment was fun and helpful! I ran out of time towards the end but think I will go back and implement some optional features after this week's homework :)

thanks again.


Did you mean to prefix this `builtin cd` with `and`?


@indutny please review


Working in EE


comment from js


inline this variable


Test one two three


What appears to have happened in the dkms location for that spl/zfs.release file changed so this check is now always failing and nothing ever gets removed.  This clearly wasn't as robust solution as I hoped.  However, since we're now always bumping the version number and not just the release you fix should address the issue.


Nice commit. Xzibit approves.


Hello From EE.


This is a point of dispute, I know there are a lot of softwares that force/manipulate extensions (Windows). But, as a Linux programmer, if my user wants to write a "stupid" name without extensions, he should be free to do that. It is not responsibility of the software to tell the user what he should or should not do.


Ok, I didn't really understand why you changed the `Menu.saveRadar`, but please revise it to make sure all eventualities are covered.


@colinbruce see this commit. I think it covers 'most' cases.


BEST. UPDATE.


Suggestion for improving the English:
`In such a case, there are too many fields to inline (see ElementInjectorInlineStrategy)`


ok, but how can I put the <b>build version</b> without git tool??

before the code changes I always do this without problem


```
   grunt.registerMultiTask('nunjucks', `Renders Nunjucks' templates to HTML`,
                                       ^
Loading "nunjucks.js" tasks...ERROR
>> SyntaxError: Unexpected token ILLEGAL
```


The good side is that I just invented the "please give me a check" SF personality.
OK, its 50 ELO less strong, but sure it should be fun.


in circleCI failures:

test_saveFullImage - do not have /data directory

test_setColormap - Invalid colormap: cubehelix

test_invertColormap

test_reverseColormap

test_setColorMix

test_setDataTransform

test_setGamma

test_setImage

>           raise IOError("decoder %s not available" % decoder_name)
E           IOError: decoder zip not available

/usr/local/lib/python2.7/dist-packages/PIL/Image.py:420: IOError

Do I need to install zip python package, what is it called?

On Tue, Sep 29, 2015 at 3:33 PM, Alex Strilets <astrilet@gmail.com> wrote:

> looks like you are saving to /data directory that I do not have
>
> On Tue, Sep 29, 2015 at 3:31 PM, Alex Strilets <astrilet@gmail.com> wrote:
>
>> actual
>>
>> y.test -v test_cartavis.py::test_setColormap
>>
>> py.test -v test_cartavis.py::test_setImage
>>
>> py.test -v test_cartavis.py::test_invertColormap
>>
>> py.test -v test_cartavis.py::test_reverseColormap
>>  last one passed
>>
>> On Tue, Sep 29, 2015 at 3:31 PM, Alex Strilets <astrilet@gmail.com>
>> wrote:
>>
>>> these one are failing on my ubuntu box
>>>
>>> py.test -v test_cartavis.py::test_setColormap
>>>
>>> py.test -v test_cartavis.py::test_setImage
>>>
>>> py.test -v test_cartavis.py::test_invertColormap
>>>
>>> py.test -v test_cartavis.py::test_reverseColormap
>>>
>>> py.test -v test_cartavis.py::test_setColorMix
>>>
>>>
>>> On Tue, Sep 29, 2015 at 3:17 PM, Alex Strilets <astrilet@gmail.com>
>>> wrote:
>>>
>>>> Testing it now
>>>> On Sep 29, 2015 3:11 PM, "Jeff Taylor" <notifications@github.com>
>>>> wrote:
>>>>
>>>>> Does this mean that they worked?
>>>>>
>>>>> —
>>>>> Reply to this email directly or view it on GitHub
>>>>> <https://github.com/Astroua/CARTAvis/commit/9028edad4a755c1b66098a502b1fd45fcb7baa60#commitcomment-13503708>
>>>>> .
>>>>>
>>>>
>>>
>>
>


The extra define for this for gb builds can go now as well.


This might affect programs opening many time times in parallel, like mincaverage.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/18601/testReport/) for this commit**


Facebook, not gb -_-...


Hi @gbaydin, Do you have to target 4.4.0.0?  The recommendation for libraries is to target the lowest FSharp.Core feasible without making the development of the library too hard.  It seems for this that the core library should either be portable (profile 47) or target  4.3.1.0 to make sure you don't force dependencies on upstack users.

Applications and test programs using the library can target 4.4.0.0, including tests and apps in this same project.

If something else forced you to move to 4.4.0.0 I'd like to know about that too.

best regards
don


While, strictly speaking this isn't a misspelling, it is an invalid reference to the filename (which is LICENSE not the correct LICENCE :gb: ). So I guess I have to ok it.


Ethernet shield gets conflicted with Arduino digital pins 2 and 3 when you use these pins for LEDs. Experiment with pins 7, 8, 9 instead of 2, 3, 5 showed no conflicts.


comment from js


postTweet(String) method is added for future reference and use


[Data 150914.xlsx](https://github.com/jblasberg/Econ1970_Blasberg/files/4757/Data.150914.xlsx)


You can use ``bindingRedirect oldVersion="2.0.0.0-4.3.1.0" newVersion="4.4.0.0" `` instead of all these lines :)


Had to rework the contacts form due to that the design at the time included one general tab with all sections inside and whenever the code executed to hide a specific field it would hide another field on the same line.  New form design eliminated the general tab including multiple sections eliminating the hiding of the unintended field.
Due to the form rework, updates needed to be included in contactFormScript.js.


CacheValue might be null


likewise targeting .NET 4.0 or 4.5 seems to make sense


Sure thing, done.


Yeah, this is the only property we have for the converters and thought @Value is sufficient. Let me know if you think otherwise.


Small experiment - averaging 53 files from ICBM dataset:

old libminc:
\time mincaverage tal_001??_V1_t1w.mnc test_old.mnc
33.77user 0.27system 0:34.05elapsed 99%CPU (0avgtext+0avgdata 151784maxresident)k

new libminc:
\time mincaverage tal_001??_V1_t1w.mnc test_new.mnc
12.06user 0.66system 0:13.59elapsed 93%CPU (0avgtext+0avgdata 983496maxresident)k

so, it uses 1GB instead of 151MB


god bless mirin


@aeslaughter is this what you were after?


Yes its will. But you lock the limit only, not the scale range. So what is really done, is this. You set max freq to 2.6 via stweaks or synapse, that use the limitter to lock freq using sysfs that lead to cpufreq in arch.
Now user try to set 2.8ghz in some app, its set and go back instantly.  Like ping pong, but if user like to set 1.9ghz no problem. Just not above the set limit.
This way, no app or script or rom hacks, cant set higher freq then set in stweaks or synapse... as they dont know about the lock, that should be unlocked before setting higher freq.
Same for min...
I find this driver more protected from unwanted freq sets to high, or lower than user want.


Related to thymeleaf/thymeleaf#294


Faltó decir que está lista la función echo que invoca un retorno para comprobar la conexión con el arduino.


That is a drastic change and might be too much for real deployments (high performance hardware).


Updated about page link to be relative


typo: `of` should be `or`


running with all 152 files
old:
89.49user 1.34system 1:56.71elapsed 77%CPU (0avgtext+0avgdata 419616maxresident)k

new:
32.68user 1.70system 0:34.38elapsed 99%CPU (0avgtext+0avgdata 2777540maxresident)k


everything in a class body is strict. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Classes


Acá se están modificando archivos del proyecto y va a ser distinto para cada uno. Para evitar este cambio tenés que ponerle Ringo de nombre al virtualenv en PyCharm.


Hey Victor, I just took a look and I think what is giving you trouble is that the `UploadUsers` method in the `GitkitClient` class in `gitkitclient.py` uses a `map` function. In Python 2 map produces a list, but in Python 3 it produces a generator (a map object in this case).
This is causing the `assert_called_with` to fail since it is expecting a list, but in Python 3 it receives a map object.

I see three possible ways of fixing this:
1. Change the unit test so that it expects a map object instead of a list (for Python 3). You could do this by wrapping the list in a map function with a lambda function that does nothing.
2. Wrap the map function in `UploadUsers` in a list, which forces Python 3 to produce a list.
3. Change the map function in `UploadUsers` to list comprehension, which will produce a list directly instead of a generator.

There's no performance benefit in using the generator in this case, since the map object will be forced into a list by a JSON library downstream, so we should go for what is most readable. Personally, I prefer option 3.


This doesn't fully address the race condition (although it does clean up the seemingly unnecessary async waiting); I happened upon this today as well.  The proper fix is to have the sendWg just account for the actual HttpSender goroutines, not the individual buffers being sent, so all sendWg.Add() calls happen during the startup code path (as we spawn sender goroutines), and cannot happen concurrently with the waiting during shutdown.  I'll prep a pull request with the fix (along with some other races related to numErrors) shortly.


Все верно. v1.0.1 с корректировкой индексов индексов работает как надо. Вам спасибо.


It is constant time in respect to the algorithm not short-circuiting the linear comparison.


Нужно выделять модели. Используется rails angular resource


Need to extract model


By the way, default cache size in HDF5 is 1MB.


Need to use `ui-href` with `ui-router`


You might consider changing this to "an index call with anything other than a selector" maybe? (I think that's the issue here).


I did modifications in the previous one. I also, set the size of the matrix just to see if it gonna works.


Hate to nitpick, but it's "marshmallow" ;)


The components of the period class are vectors an these if will break.

I have a some local changes from a month ago when I tried to fix this issue. The fix is not quite an easy peasy.


It's better not spending efforts on the things like this -- Kotlin isn't stable enough to rely on suggested inspections.


it looks hacky to me to make `DEFAULT_TIME_AXIS_CONFIGURATIONS` public
does it make more sense to add a setter and getter for `DEFAULT_TIME_AXIS_CONFIGURATIONS` instead?


Manually from the admin.


can we add pid isolation too? last stand against zombies if our minion controller times out waiting for a kill to complete.


it looks hacky to me to make DEFAULT_TIME_AXIS_CONFIGURATIONS public
does it make more sense to add a setter and getter for DEFAULT_TIME_AXIS_CONFIGURATIONS instead?
(probably an implementation detail on 2824 instead of here


Removed WPF Chrome Tabs.

It was an idea I had, but at this point I don't see it fitting too well.


Can also use Promise.all() here, and generally try to keep promises sequential instead of nested. At end have some error handler.


nit: can you move this comment above the cores assignment, and indent it?


this can just be a `for ni := range nodeInfos {` statement. no need for the more complicated for/select loop.


Hi Claudia, I suppose you're talking about file `experiments/humanFixedBase/ID.m`. I remember looking at these lines back then with Naveen, but don't remember why this change was done (sorry Naveen if you've changed this because of any of my observations). Well, looking at it again, I would say that   `f{model.parent(i)}` is the correct variable to use:

`f{model.parent(i)} = f{model.parent(i)} + Xup{i}'*f{i};`

Since this is the contribution of each child node to the sum of wrenches on parent node.

and further at line 42, we should have:

`fReactionBase = Xup{1}'*f{1} ;`

(assuming `fReactionBase` is going from the floor to the base and is expressed on a frame attached to the foot sole).


Why?


I'd feel better if this was buffered with a size of `1`; the intitial executor Registered callback wouldn't be blocked by the kubelet run then.


Great...


Fixing down stream


Way ahead of ya :smile: - already replaced in f0505b02f0e5c3f67789c73070f58.


Irving puedes x favor agregar el script con datos de la bd para poder correr pruebas. (TNKS)
Estoy bajando un zip del branch master, es correcto vdd? es el unico que veo por default.


Also already fixed, this one in 51dd9f45d1d64f2a045552240c19d603792f8964.


Wouldn't it be possible to write this LUA script as a multi operation instead? (not that it is anything wrong with this, but the code gets simpler using multi).


nit: s/flag/mask


@Implementation


https://github.com/DiffSharp/DiffSharp/blob/master/src/DiffSharp/app.config

But, that is the case in the latest version. :)


Should this only be present on < 21?


Wouldn't be possible that delayedTimestamp becomes smaller than Date.now and if so, the guardian will never update the delay set?


I've never tried it.


can't we write this as a for loop? :)


it confuses me that this passes. I would think that pushing `6` would cause `shouldTransactionMerge` to return false, which would cause that transaction to be deleted - because of [this](https://github.com/vigetlabs/microcosm/blob/transaction-tree-tweaks/src/Microcosm.js#L67)


this function is too long


@karaxuna update the docs on this repo with this new enhancement


Not sure why I jumped to use LUA at first, but yeah it can be re-written in multi


"you are ok with data loss." might also be confusing. Perhaps we can say "if you are certain that the NameNode was restarted with the correct FsImage and edit logs."


No, thank you. I apologize. I have this completely in reverse.


Awesome, thanks!


Hi Don! The most important reason for us to move to 4.4.0.0 is the "implicit quotations" feature in F# 4.0. That is really going to make a huge difference for this library and probably will end up being the main way we are interfacing with user code. If you remember, our initial motivation with all this work was to base AD on F# quotations. This removed a big obstacle for doing it in a non intrusive way. (Thank you for this by the way, if I had one "wish" for a new feature in the language, it would have been this.)

Since we needed this, and since I thought we can benefit from bug fixes and improved performance, I made the move to 4.4.0.0.


it's better to mark instances first (perhaps add instances to a list) and then clean in a separate step. that way you can:
a. test for fishy things - e.g. all instances are bad
b. call stop_instances with multiple instance ids speeding up cleanup


How do we ensure that we also update other environments (e.g. stage and local dev)?


stopping instances and waiting should be a separate function.


`vfim` function in `convert.c` is.


Sure. I thought you didn't want it though, based on the discussion we had a few months ago.


I'll send a PR at the weekend. :)


I prefer the more explicit code rather than having the DoAsyncCall within tests. It explains the code better and doesn't require yet another lookup on what DoAsyncCall does.


why are you stopping the instances and only then terminating? wouldn't it be easier to store a list of ebs volumes to cleanup and just terminate the instances?


Thanks! better to find it now then later :)


Thank you for fixing it! It was causing some confusion for students with Debian based OSes.


Note that regardless of MSDN, this is actually true for the current Windows implementation that doesn't support skipping the callback.

A `//TODO` was left in the code to enable this now, since IOCP supports this on all CoreFX platforms.


Kaushik Krishnan September 29 2015, 6.17pm


Since I just merged my pull request and made some changes to this file as well, you may run into some problems if you were to merge this. If you make the following changes, it should be fine (shouldn't affect your code). This is just to deny invalid moves when dragging the pieces around.

Line 23-25: change to:

     Piece.transaction do
        if @piece.valid_move?([pos_x, pos_y])
          valid_move = true
          @piece.do_move!(pos_x, pos_y)
        else
          valid_move = false
        end
     end

Line 32:

     json_result = {valid: valid_move, active: @piece.active, captured: captured, not_color: @color}


what happens if i want to write several times in one fabric run? i would like to reuse the cloudwatch logs object


Nice, it's much better!


As mentioned above: I prefer this style. I also prefer having consistency throughout the tests since they are the best documentation for our code.


Sorry, shouldn't have deleted that comment. The sixth push returns true because there is a chain of transactions in the current pathway 6 deep. This returns true, so Microcosm merges in the left most (oldest) transaction.

This _is_ confusing however. `shouldTransactionMerge` indicates a specific transaction. We should rename this.


Argh, this one shouldn't have been changed: we do need to match the existing behavior for now on all platforms, and these asserts help ensure that. I'll undo this one.


no need to do err.Error() you can pass ``err`` directly in fmt.Errorf() internally formatters handle error interface to provide the proper string.


`shouldHistoryMerge`?


Whats the reason for the unrecognized character?


:+1:

Now I know that every time I wrote `@NonNull, @Nullable, etc` in a commit message — some GitHub users get notified :)


That's true. A simple print can be better


I went back and forth on this. No strong feelings; I'll change to `willRaiseEvent` etc.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21348/testReport/) for this commit**


@georgekola does it affect leaks?


The discussion was that Fabien won't like to move project into PSR-4 and some of us woudn't like to have fixer that rename class.
But as you can see I keep the file in interface and PSR0 as they is (oh, maybe disable it by default as it's risky :D)

Great to hear!


should this msg use gettext?


@indutny yes it affects leaks and seems like a race condition. Happens only on 8.4.x (tested on 8.4.1) when linked against iOS 9 SDK


Sorry about that.
https://github.com/flutter/engine/pull/1400


you need to upgrade to at least node 4.0.0


yeah, it should


What kind of race condition? I don't think that we access timers concurrently...


"embed_name" would be a better name than "review_type". There are different types of embeds - not all are reviews. And for the function, something like "parse_embed_from_datauri"

Only reviews are a review type, the others have nothing to do with reviews. Our system creates and supports embeds - that we then use for various items, like reviews, name/address data, best reviews, contract forms, attribution tags, etc.


Is this a placeholder for future functionality?


Regarding figure 2, see https://github.com/tc39/ecma262-6-src/blob/master/6.0-docs/readme-figure2.md

Note that figure 2 was created using the community (free) edition of the Astah UML editor.  That edition does not supporting generating svg output.  The paid edition does support svg output, if someone wants to step up and pay for it.


Yes, I can create an issue for that.
In my current project I can't debug deep because frameworks can't be debugged. So in development mode I get assert failure and sometimes in release I get blank screen (or app freezing). Actually I suppose that it's a socket timeout after several tests.

Unfortunately with GDC lib I get the same problem (iOS 8.x) but XWebView is really faster than it.
In my case restart connection might be very slow because I have a page controller with 3 or more webviews so I don't think that it is the best solution but in general words the socket is a a workaround to avoid WKWebView limitations for iOS 8.x. In this case, restart them can minimize this issue.


In this case also, the variable should be called something more appropriate, like "$embed_content" or related, as the content may or may not be a review.


nit:ws after method name


Multiple coins at same position


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21350/testReport/) for this commit**


`/ping` is actually command for debugging messages, so the call to `console.log` needs to be preserved.


The timers leak sometimes but not all the time without the weakSelf and the leak happens only on iOS 8.4.1. And there was a  certain path with triggered the leak but I was not able to exactly pin point it out


Thanks!


there's a chance `customLabels` may be a string depending on customer implementation at this point.

Let's move this logic beneath the line below (`if (is.string(customLabels)) customLabels = [customLabels];`)


Consider using a promoted data kind here instead:
```
data Nat = Zero | Succ Nat
```


The short story is this:

We started experimenting with deep learning models using this library and I had to make a few performance-oriented decisions after running many experiments. Supporting float32, adding native linear algebra backends (OpenBLAS by default, also GPU support soon). The best performance was with float32 on x64 with .NET 4.6 and RyuJIT. So we are using the x64 binaries of OpenBLAS and we made the move to .NET 4.6 because we want people to run our library with RyuJIT.

Do you think it is premature? I've been also following the tail call optimization issues with .NET 4.6 and thinking whether we should switch back to 4.5.


Then you can restrict `Plus` and `LT` to `Nat` kinds... optionally using closed type families, which make more sense with a closed kind.

```
type family Plus (n :: Nat) (m :: Nat) :: Nat where
  Plus Zero     n = n
  Plus (Succ n) m = Succ (Plus n m)
```


It should be sePtember as before.


@bjansen does this help with your performance tests?


It should be "sept." as was before.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21351/testReport/) for this commit**


Eventually I have a bit of a vague idea what you mean. It seems as if you want me to use cmake-gui to configure the source for the build of the binaries, right? The problem is, however, that I am using Gentoo and the package manager uses ebuilds - which are build scripts as well somehow - for doing the job of compiling.

So, which option do I have but to patch the CMakefile localy accordingly to my needs?


sorry I mispoke, `shouldTransactionMerge` would return true, you are correct. I'm still confused how 1 then merges into 2. Since the truthiness of `shouldTransactionMerge` just causes the latest one to get axed.

I do suppose `shouldHistoryMerge` sounds better for this use case of merging older transactions. Is that always the case? Maybe both should exist?


Nice :sunglasses:


Yes. :-) Is that too concise?


Quick note I forgot to add. You'll need to cd to the testing-deftly directory, and then run npm link to have the tdeftly command available globally. This allows me to use it on the ai-deftly tests while still piecing it together.


Did it have an unexpected side effect?


@pgavlin Please don't change them back! That's not what I meant...

While that's the behavior, it's just a limitation of the platform (it will always return true). Even if we would implement returning false, that wouldn't come with any timing guarantees.

The intent and MSDN documentation fits the changes in this PR. If anything, we should change the Desktop tests to match these.


I sometimes prefer using type aliases instead of type families when there is a single instance. It has some observable side affects in generated documentation, otherwise it's mostly personal preference.

```
type Triangle a b c =
  And (LT c (Plus a b))
      (And (LT b (Plus a c))
           (LT a (Plus b c)))
  :~:
  True
```


I agree about raising exceptions. But for
1. assert in `open` I think it isn't necessary because we have a boolean return
2. and 3. exceptions I put them there because assertions don't have description message and it was really difficult to debug.
Finally the case 3. is just unexpected. Server should never respond with status code >500 or <0 if it occurs I think app has to be terminated or exception be caught


My bad, it seems to work fine indeed. +1


Give me some time to look at it tomorrow...


hahaha most likely :smile:


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21352/testReport/) for this commit**


I never would have guessed ) Search engines can not index it?


That's fair. Please file a bug that tracks updating all of the tests to handle synchronous successful completion and assign it to me


Good catch, thanks!


I think part of what makes this confusing is how `clean` gets called. `clean` is run by `transactionDidClose`. When a transaction completes, it checks to see if it can clean up the transaction list. It does this by walking through the history graph from oldest to newest, merging in complete transactions. It stops when it hits an incomplete transaction (like a pending Promise).

This is here: https://github.com/vigetlabs/microcosm/blob/2ed48d356ae359a3a3c365fdeeb98520e4fda02f/src/Microcosm.js#L85

Transactions are merged into the cached `base` state from oldest to newest. We never want to merge an incomplete transaction, so that we can allow all future events to be re-executed when it does finally complete.


TeamCity pull requests :: netty-tcnative [Build 15](https://garage.netty.io/teamcity/viewLog.html?buildId=1842&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty-tcnative [Build 15](https://garage.netty.io/teamcity/viewLog.html?buildId=1842&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:45


Arf :confused:, you will have to rewrite this feature in the develop branch, because the next release (PR #12) changes a lot of things and hasn't been yet merged by @redox :


Generated files should not be in the source tree. Am I missing something here?


I think that is worth revisiting the nomenclature for all of this. These are all private API methods. There is no concern about changing them.


Thanks!


Why did escodegen get downgrade to 1.6? Was this a mistake?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21353/testReport/) for this commit**


Should this be a call to ensure_block instead() instead? ensure will create the chunk if needed and then return the block.


Why did escodegen get downgraded to 1.6? Was this a mistake?


:mindblown: i get it now. and agreed.


Why not just do `Value?.ToString() ?? “<null>"`


r+


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/8ca28fa23a50fc46ba6f938fe5ce5c7d175a6f61


merging MonsieurNicolas/stellar-core/scpIntervalFix = 8ca28fa2 into auto


MonsieurNicolas/stellar-core/scpIntervalFix = 8ca28fa2 merged ok, testing candidate = f5f32c32


We can add a check, that's easy. But would that ever happen? The `updateDelayTimer` method always set the delayedTimestamp to infinity when the timer is up


how can i contact you  im interested in your panel


comment from js


For turn-off tip, should we check {{getBytesInFuture}} after the threshold of reported blocks isreached? One potential issue is that the administrator may see this message and immediately run {{-forceExit}} even before block thresholds are reached.


If this doesn't exist then a file cannot be saved. I'll test Vradar for None instead


with this commit you can remove the tuple one, no? Don't even push it to the PR.


Some context here: http://ejohn.org/blog/javascript-micro-templating/


Okay, I'll buy that. It's nuked!


* now: useful for debugging.
* later: needed to skip `evaluate()` calls after null move.


I'm still trying to understand how the 3PL integration team is expecting the header to behave after the pairing/linking process has been completed. There's some thought that we'll need to remove the Sign Out and My Account options, and if the integration mode is used post-linking then there will need to be mobile navigation options for Revel.

Chatted about this with @mcaskey about this, we came up with a flow today which would work with just the basic mode at that point.

My thought right now is we follow the acceptance criteria above and revisit as we figure out how this works. I think Integration Mode may not be super stable though. Should this be notated somehow?


The etag is supposed to be quoted http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.19


This should indeed be reverted then, as the included code needs the variable.


i contact xoceunder@gmail.com @deadpooladdon


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/82836289


With the PRs I have open currently, MSVC 2015 can be used for the pieces of Folly that HHVM uses. There is no build system in place for MSVC in the main Folly repo.


Works! :+1:


Code in javascript for google spreadsheets


A small update, Seems that re-generating files does produce a better output. For example, *HISTORY.html* has an invalid 3-byte sequence of 0x000000 in a "PYWINAUTO TODO's" link instead of a valid apostrophe:
```title="PYWINAUTO TODO *0x00 0x00 0x00* s```, but after re-generating it's replaced with valid Unicode bytes: 0xe28099


Indentation


fast-forwarding master to auto = f5f32c32


issue5221_memoryAllocation (tsnouidui) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Hum, I think you swapped this line with L219, no?


Hmm?


Doesn't clang have a warning for this? Can we turn it on?


@PabstMirror gave the event the :banana:


Don't we now need to consider either actionFlags or actionsArray for equality?


should this be .push instead of .concat?


Remove debugger bp?


ditto


```C
if (!source)
  return;
```


Two comments, otherwise LGTM. Thanks!


Debug?


This should now be identical to the one in dotnet/master. (Manually updated.)


nit:EOL whitespace


Probably want to add

    private static final int LAST_LEGACY_STANDARD_ACTION = AccessibilityNodeInfo.ACTION_SET_TEXT;

    private static final int ACTION_TYPE_MASK = 0xFF000000;


Regardless of API level, first:

        if ((action & ACTION_TYPE_MASK) != 0) {
            throw new IllegalArgumentException("Action is not a combination of the standard " +
                    "actions: " + action);
        }


'action' can be a mask of multiple legacy actions.  So you need to replace the 21+ logic with something like:

int remainingIds = action;
while (remainingIds > 0) {
  final int id = 1 << Integer.numberOfTrailingZeros(remainingIds);
  remainingIds &= ~id;
  AccessibilityAction convertedAction = ReflectionHelpers.callStaticMethod(
    AccessibilityNodeInfo.class, "getActionSingleton", ClassParameter.from(int.class, id));
  addAction(convertedAction);
}


Note that my comments related to addAction(int) explicitly DO NOT apply to this removeAction(int) method based on the framework implementation. (Perhaps this is even worth a comment) I claim the framework implementation is actually incorrect, but the point is for RL to reflect that implementation, not implement it correctly.


if (actionId <= LAST_LEGACY_STANDARD_ACTION) {
  returnValue |= actionId;
}


s/possibly/possible


Can you add a comment about what this function does? It seems like .push is more appropriate here too.


Adding comment here, as it seems I can't comment on unchanged lines.

We need a revised implementation for performAction(int, Bundle) as currently, we rely on properties.


Good catch, thanks.


I haven't loaded this on my phone yet, but can you clarify some things about the code?


Not sure how this works. leaveSafeMode will just return {{if (isInStartupSafeMode() && (blockManager.getBytesInFuture() > 0)) \{}}


A newly obtained copy, I believe, should have it's original copy's actions.


LGTM! Thanks Joe!


lol, It didn't take long to remove C2.


Could you comment as to why this has been done before I add them back?


Nice job selecting your HTML5 elements here!


should comment come before the attribute?


*non-animating


Woah the display property! We didn't talk about that yet. Good job :+1:


Yes - I just don't remember what now, it was a while back.

+++ Jeff Johnson [Sep 29 15 15:39 ]:
>   Did it have an unexpected side effect?
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/gitit/commit/a9b449b1bfe6c91d808c5bc466fa1444f3e22d91#commitcomment-13505293


this is no longer a mock


The following errors should actually extend `ProgrammingError` I think. `InvalidRequestError` is orthogonal to the the `DatabaseError`/`DBAPIError` type, and I believe when used internally in sqlalchemy, they actually throw these errors before a query is ever dispatched.


Before assignment:

    if ((actionId & ACTION_TYPE_MASK) == 0 && Integer.bitCount(actionId) != 1) {
      throw new IllegalArgumentException("Invalid standard action id");
    }


ah, thats fixed in the next commit.


@igor-silverstripe There's a PR to fix this better with https://github.com/instaclick/php-webdriver/pull/67. Given it affects all our Behat projects, I hope this will be merged soon...


huh, is it right that one is CMD_FIGHT and the other is CMD.MOVE? as in, one with underscore, but the other with a dot?